% symbol interpreted as URL escape characters in Postgres password

2015/08/22 21:40:43 [W] Register Mail Service: Mail Service is not enabled
2015/08/22 21:40:43 [...s/routers/install.go:66 GlobalInit()] [E] Fail to initialize ORM engine: Fail to connect to database: parse postgres://gogs:2wPb*%tBQyjP@<IP_ADDRESS>:5432/gogs?sslmode=disable: invalid URL escape "%tB"

(I've obviously changed my password.) The characters "%tB" in my password caused me to be unable to connect after upgrading from v0.60 to v0.65.

Woops, wrong project sorry!


Search on enter

scrolls to top on search
removes text in input on search
removes default browser behavior on enter click while input is focused
gets matches when enter is pressed while input
Solves issue #52

Thank you, @20jasper

Thanks guys. That was quick.


What about docs?

Let's work on the documentation with the following suggestions:

[ ] #7
[ ] #8
[ ] #9
[ ] Decide if we need something more complex

Hey there, O wonder what we should do for documentation in this project, any thoughts

So what you're saying is that we should

[ ] Create a docs folder
[ ] Create a new index.md document
[ ] Turn on github pages
[ ] Decide if we need something more complex

THanks I created the issues for the first three suggestions


@yuki24: remove exhibitor count on mobile

This is removes the exhibitor count from the fair microsite on mobile. It is the microgravity equivalent of https://github.com/artsy/force/pull/1218

LGTM :+1:


Migration of GCPMachineClass to MachineClass

What this PR does / why we need it:
This PR allows migration of GCPMachineClass to MachineClass objects. It also updates any references to old objects.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Check out the commits for ease of review
Release note:

Migration of `GCPMachineClass` to `MachineClass` CRs is implicity. Don't use old GCPMachineClass CRs post migration.

/needs review

/invite @hardikdr @ggaurav10 @ashwani2k . Checkout commit-wise changes for ease of review.

cc: @AxiomSamarth . Similar changes for azure might be needed.


support for xcode 8 GM

@zhangao0086 Tried to update the syntax, but line 139 (override fileprivate var asset: DKAsset!) in DKAssetGroupDetailVC.swift file triggered "Command failed due to signal: Segmentation fault: 11". So i am not sure how to inherit private property in super class with beta 6.  Maybe you want to take a look.  Thanks much.

The Xcode Beta 6 came out so fast. I would wait until the stable release of Xcode.

Beta 6 has major changes but there shouldn't be much more left in Swift 3's accepted proposals

Sorry about the late reply...
I appreciate it!!! You are wonderful!!!


⚠ Bofrost.at has degraded performance

In b52c37c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1410 ms

Resolved: Bofrost.at performance has improved in 13be5f5 after 6 minutes.


Create .npmignore

@felixge please consider merging this. It'd be good to get this in before the next publish. :)


Closing other <InfoWindow>

Hi
I have few markers with a <InfoWindow> on it. How can I close the open <InfoWindow> when I click on the map or open other <InfoWindow>?
Thanks :)

Hi @ralbear
You'll have to manually control the info windows through refs.

Thanks for the response, will give a try!

I give a few tries to this but can't find a working approach, is there any similar example I can base on? I try to find it few times without luck.
Thanks in advance!

friend after trying to solve your same problem this is my solution I hope it can help you.

@ssantss This works perfect at first attempt! Muchas gracias!😃

<InfoWindow :ref="(el) => (infoWindows[i] = el)"> This line uses an anonymous function to assign each InfoWindow to an element in the infoWindows array at their corresponding index. The reference of each InfoWindow is stored in the infoWindows array so that we can access them later.
const infoWindows = ref([]);  This line creates a reactive array called infoWindows that will store the references to the InfoWindows.
const clickMarkerEvent = (i) => { closeInfoWindows(i); }; The clickMarkerEvent function takes the index of the marker . It then calls the closeInfoWindows function passing in the index of the marker.
const closeInfoWindows = (i) => { infoWindows.value.forEach((ref, index) => { if (index !== i) { ref.infoWindow.close(); } }); };
The closeInfoWindows function closes all InfoWindows except the one at the provided index i. This is achieved by iterating through all the InfoWindow references stored in infoWindows and closing each of them except the one at index i.
This solution takes into account the opening of InfoWindows and closing of others when clicking on a marker. It also closes all InfoWindows when clicking anywhere on the map, as it still uses the @click="closeInfoWindows" event on the GoogleMap component  <GoogleMap @click="closeInfoWindows" >

The clickMarkerFunction function is never called. What is its purpose?

<InfoWindow :ref="(el) => (infoWindows[i] = el)"> This line uses an anonymous function to assign each InfoWindow to an element in the infoWindows array at their corresponding index. The reference of each InfoWindow is stored in the infoWindows array so that we can access them later.
const infoWindows = ref([]);  This line creates a reactive array called infoWindows that will store the references to the InfoWindows.
const clickMarkerEvent = (i) => { closeInfoWindows(i); }; The clickMarkerEvent function takes the index of the marker . It then calls the closeInfoWindows function passing in the index of the marker.
const closeInfoWindows = (i) => { infoWindows.value.forEach((ref, index) => { if (index !== i) { ref.infoWindow.close(); } }); };
The closeInfoWindows function closes all InfoWindows except the one at the provided index i. This is achieved by iterating through all the InfoWindow references stored in infoWindows and closing each of them except the one at index i.
This solution takes into account the opening of InfoWindows and closing of others when clicking on a marker. It also closes all InfoWindows when clicking anywhere on the map, as it still uses the @click="closeInfoWindows" event on the GoogleMap component  <GoogleMap @click="closeInfoWindows" >

this approach worked for me! ChatGPT gave me the approach of conditionally displaying info windows based on a ref of marker.id
That approach required a double click on markers to get the window to open. Probably some kind of race condition setting the info window marker id ref after the info window is set to display.
Thank you @ssantss !


Sketch Library updates

[X] bug
[ ] feature request
[X] enhancement

Expected behavior

[x] updated UI Libraries (light, dark)
[x] updated Icon Library
[ ] update version numbers on download buttons - on Get Started page

Updates

[x] Add Button Dropdown
[x] Add Button Dropdown DT
[x] Button color changes #1939
[x] App-Level Alerts - resizing issues
[x] App-Level Alerts - resizing issues DT
[x] App-Level Alerts - add close icon
[x] App-Level Alerts - add close icon DT
[x] App-Level Alerts - add pagination version
[x] App-Level Alerts - add pagination version DT
[x] Vertical nav resizing issues #2098
[x] Vertical nav resizing issues #2098 DT
[x] Resizing issue on Dropdown Divider override
[x] Resizing issue on Dropdown Divider override DT
[x] Add vol 11 icons to Icon Library #2101
[x] Update icon instances in Vertical Nav symbol to use symbols from Icon Library
[x] Update icon instances in Vertical Nav symbol to use symbols from Icon Library DT
[x] New Forms updates

@adityarb88 the updates are complete. Here are links to the files:
UI Library 0.12.1
UI Library - dark 0.12.1
Icon Library 0.12.1

Updates are complete. Here are links to the files:
UI Library 0.12.2
UI Library - dark 0.12.2
Icon Library 0.12.2

Sketch has been updated on the website for 0.12.2


v1.2.2 is slower than v1.2.1

I conducted multiple tests and found that v1.2.2 is 2 to 3 seconds slower than v1.2.1. After filling in the specific IP and port in "endpoint" and starting the software, v1.2.1 can connect successfully in almost 1 seconds, while v1.2.2 takes 3 to 5 seconds to connect successfully. This isn't a big deal, but wanted to give some feedback.
Thank you very much for your selfless contribution.

thanks :heart:


Support for OpenBSD & Nim 1.2.0 | or at least help to modify for Nim 1.2.0

For several years, I have wished native_messenger was compatible with OpenBSD, and I had assumed that it was merely a limitation of knowledge that prevented this from happening. Recently, I was successful in setting up firefox to communicate with a different messenger application, so I figured I should resurrect my work on native_messenger and try again.
Since the installation script installs native_messenger as an Executable Linux File (ELF), it would not be able to run on a native unix(bsd) system, which means it is going to have to be compiled. Nim 1.2.0 is available in the OpenBSD ports collection, and has been installed. Unfortunately, the minimum requirement to compile native_messenger appears to be Nim 1.4.2. Ignoring this fact, I attempted to compile anyway, and as you could guess, it failed.
Frankly speaking, it is unpleasant having to download Nim 1.4.2, attempt to compile it, learn how to write code in Nim, troubleshoot all potential errors, possibly contact other developers to mitigate those encountered errors, maybe arrive at a resolution, and then attempt to compile Nim again. All just to then attempt to compile native_messenger again. So, I was wondering if there is anyway I could get some help to allow native_messenger to compile using Nim 1.2.0? The differences between 1.4.2 and 1.2.0 are possibly not too great.
Right now, compilation with 1.2.0 is breaking on line 145 in native_main.nim, with a type mismatch error. I will post the full output below to help troubleshoot the error.
        ... /home/user/native_messenger/src/native_main.nim(145, 32) Error: type mismatch: got <Process>
        ... but expected one of:
        ... iterator lines(f: File): TaintedString
        ...   first type mismatch at position: 1
        ...   required type for f: File
        ...   but expression 'process' is of type: Process
        ... iterator lines(filename: string): TaintedString
        ...   first type mismatch at position: 1
        ...   required type for filename: string
        ...   but expression 'process' is of type: Process
        ... iterator lines(s: Stream): TaintedString
        ...   first type mismatch at position: 1
        ...   required type for s: Stream
        ...   but expression 'process' is of type: Process
        ... expression: lines(process)

Thank you for your time and investment.

#56 should do it, unless there are other issues that are specific to BSD.

Eureka! Like butter, absolutely like butter. Thanks.

After modifying the installation script to copy and not curl, and adding execution permissions to unveil.main, my mind was completely blown when I saw:
# Native messenger is correctly installed, version 0.3.6

Bosanova! Super!


ISPN-6563 JCache remote should pick HR props file

https://issues.jboss.org/browse/ISPN-6563

For situations where JCache cannot be instantiated directly, it should
pick Hot Rod client properties file to enable it to be configured only
declaratively.

CI has one failure related to this.

@tristantarrant Fixed

Pushed to master and 8.2.x. Thanks


build-infer.sh fails with opam 1.2.0

Related to #554 (specifically this https://github.com/facebook/infer/pull/554#issuecomment-271283944)
Confirm that build-infer.sh fails on opam 1.2.0 due to the changed name of the flag as of https://github.com/ocaml/opam/commit/9ccabc03f75b1908ce5fda4c676ddd16d9d0c7ce#diff-906c973c07f8c64e8dd7866de2b40e35. Error is:
initializing opam...
opam: unknown option `--compiler'

Would either need to support both --comp and --compiler flags in build-infer.sh so Debian stable continues to work, or increase minimum opam version to 1.2.1 in INSTALL.md

Thanks @3flex. As I said in the comment you link to, I'd rather bump the minimum version of opam we support that periodically fix the build for ancient opam versions. Would that make sense for your use-case too?

Yes, I was only playing around with the Docker build, so bumping the minimum version would have no effect on me.


abi: type name conflict in root_schema

Currently, if a contract uses two types that are similarly named and are defined in different modules, schemars collapses that to one entry in the root_schema.
#[derive(JsonSchema, Serialize, Deserialize)]
pub struct MyType {
    field: u8,
}

mod other_crate {
	#[derive(JsonSchema, Serialize, Deserialize)]
    pub enum MyType {
        Variant1,
		Variant2,
    }
}

#[near_bindgen]
pub struct MyContract {}

#[near_bindgen]
impl MyContract {
    fn method(a: MyType, b: other_crate::MyType) -> String { .. }
}

The Associated ABI looks like this:
{
  "schema_version": "0.1.0",
  "metadata": {
    "name": "adder",
    "version": "0.1.0",
    "authors": [
      "Near Inc<EMAIL_ADDRESS>    ]
  },
  "body": {
    "functions": [
      {
        "name": "method",
        "params": [
          {
            "name": "a",
            "serialization_type": "json",
            "type_schema": {
              "$ref": "#/definitions/MyType"
            }
          },
          {
            "name": "b",
            "serialization_type": "json",
            "type_schema": {
              "$ref": "#/definitions/MyType"
            }
          }
        ],
        "result": {
          "serialization_type": "json",
          "type_schema": {
            "type": "string"
          }
        }
      }
    ],
    "root_schema": {
      "$schema": "http://json-schema.org/draft-07/schema#",
      "title": "String",
      "type": "string",
      "definitions": {
        "MyType": {
          "type": "object",
          "required": [
            "field"
          ],
          "properties": {
            "field": {
              "type": "integer",
              "format": "uint8",
              "minimum": 0.0
            }
          }
        }
      }
    }
  }
}

Related upstream issue: https://github.com/GREsau/schemars/issues/177

What we could do for now is just generate a compilation error if we are trying to insert a new definition with the same name, but with a different schema (assuming that schemars is generating consistent schemas for the same type which is most likely true).


nydus-service: split service framework from nydus into nydus-service

Split service framework from nydus into nydus-service crate, to reduce dependency when used by other project like image-rs.
Signed-off-by: Jiang Liu<EMAIL_ADDRESS>

Looks good.


Refactor define before use class

What kind of change does this PR introduce?
Refactoring.

Did you add tests for your changes?
Not necessary .

Does this PR introduce a breaking change?
No.

What needs to be documented once your changes are merged?
Nothing.

Thanks


3D integrations page improvement

Description
PR fixes #1190
This PR includes following fixes

Adjust all integration cards onto screen in middle for all different views (Table, Sphere, Helix, Grid)
Position loader inside container containing integration cards.
Improve UI of View buttons
Define boundaries for 3d interaction area for better user experience. Currently there is no defined boundary visible to user, so he/she might struggle scrolling the page( user wants to scroll but being unable to find area outside interaction container). I have always struggled.

See below screenshots for Before/After comparison 👇 👇

Before
After

Adjust all cards on screen

Fix loader position (position it inside integration container)

Button UI and position improvement

Fix overlapping cards in sphere view, fix Grid view (grid cards overlapping text)

Define boundaries for user interaction (user can interact with the 3D structure anywhere inside the area with dashed border). The 3D structure won't overflow outside the defined region

Dark Mode

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Yay! 🙂

Great work on the PR @GaganpreetKaurKalsi
There's one teeny tiny thing that I thought of, and would love to hear everyone's thoughts on this.
Not that it matters much, but usage of font-weight to hover felt just a little unnatural to me,
with the buttons and everything my mind expected a background and shadow combination effect on hover instead.

Great work on the PR @GaganpreetKaurKalsi
There's one teeny tiny thing that I thought of, and would love to hear everyone's thoughts on this.
Not that it matters much, but usage of font-weight to hover felt just a little unnatural to me, with the buttons and everything my mind expected a background and shadow combination effect on hover instead.

Yeah, I agree. I was trying that with the existing colors available in rootvariables.scss but didn't work out. Guess will add some new ones.

Does this look good, @8rxn ?? Let me know if you have some better color suggestions?
SPHERE is the hovered button

Merge conflicts... @GaganpreetKaurKalsi

This is good work @GaganpreetKaurKalsi , have you thought on how to improve the page performance because right now due many images in single data.js is giving very load to site in integrations page

Yeah, the performance is poor. Haven't thought about how to improve it yet.

Yes, yes, yes, and yes. @GaganpreetKaurKalsi 👏


Windows support

With PR https://github.com/clarkx86/papyrusjs/pull/15 Windows support is now possible in theory. Quoting the PR summary:

The problem is that Mapnik dropped Windows support a while ago and I've to find a library that is nearly as fast as Mapnik for stitching images.

If anyone has a good suggestion for a comparably fast module such as Mapnik that compiles on Linux, Windows, macOS x86 & x64 (and optimally ARM), I would be more than happy to investigate and implement it.
As a kick start, my benchmarks showed that jimp is way to slow for this use case and sharp is currently not nearly as fast as Mapnik for stitching images.
Thanks,
clarkx86 :-)

Small update: Rendering on Windows works now with https://github.com/clarkx86/papyrusjs/commit/d503529cd688b56fc7e088e91afbdaf856df2bc4, but it's suuuper slow (13 cp/s instead of 300 cp/s on my system), due to the FFI, I guess... An alternative would be fastcall, but it hasn't been updated in quite some time and doesn't compile at all on node.js 12. Also, I'm not too sure if it would even make too much of a difference in performance.


feat: publish events to SNS instead of Rabbit

Events which require sending to TIS Core are sent via RabbitMQ, however we want a separate service to handle publishing to Rabbit centrally.
Update COJ signing to publish to an appropriate SNS topic instead of direct to Rabbit.
Add event publishing when GMC details are provided by a trainee, again using SNS instead of rabbit.
TIS21-6292
TIS21-6348

Published Message Examples
Note that raw message delivery is disabled on the subscription as we'll need additional information to determine which rabbit routing key to apply when forwarding the message to TIS Core.
Conditions of Joining Signed
{
   "Type":"Notification",
   "MessageId":"57f15004-0d5f-5730-8eda-0f4ff0238b0e",
   "SequenceNumber":"10000000000000003000",
   "TopicArn":"arn:aws:sns:eu-west-2:430723991443:tis-trainee-preprod-coj-signed.fifo",
   "Message":"{\"programmeMembershipTisId\":\"0460eb0d-2797-4078-ab6f-060b2ae6a18e\",\"conditionsOfJoining\":{\"signedAt\":\"2024-08-30T17:02:49.235885300Z\",\"version\":\"GG9\",\"syncedAt\":null}}",
   "Timestamp":"2024-08-30T17:02:52.519Z",
   "UnsubscribeURL":"https://sns.eu-west-2.amazonaws.com/?Action=Unsubscribe&SubscriptionArn=arn:aws:sns:eu-west-2:430723991443:tis-trainee-preprod-coj-signed.fifo:c0731151-a440-4154-a4dd-0f6bb69438c4",
   "MessageAttributes":{
      "id":{
         "Type":"String",
         "Value":"b7c2fbde-16c5-1342-4194-57d8d853123a"
      },
      "contentType":{
         "Type":"String",
         "Value":"application/json"
      },
      "timestamp":{
         "Type":"Number.java.lang.Long",
         "Value":"1725037371758"
      }
   }
}

GMC Details Provided
{
   "Type":"Notification",
   "MessageId":"11d27838-1bd8-5b1f-a900-2773e61091df",
   "SequenceNumber":"10000000000000003000",
   "TopicArn":"arn:aws:sns:eu-west-2:430723991443:tis-trainee-preprod-gmc-details-provided.fifo",
   "Message":"{\"gmcDetails\":{\"gmcNumber\":\"1234567\",\"gmcStatus\":\"Registered with Licence\"},\"personId\":\"47165\"}",
   "Timestamp":"2024-08-30T17:04:08.727Z",
   "UnsubscribeURL":"https://sns.eu-west-2.amazonaws.com/?Action=Unsubscribe&SubscriptionArn=arn:aws:sns:eu-west-2:430723991443:tis-trainee-preprod-gmc-details-provided.fifo:7645891b-bc96-43f2-8cb0-24704bcd6a16",
   "MessageAttributes":{
      "id":{
         "Type":"String",
         "Value":"52dd8ae3-fb12-17cc-2d70-ac2500624cd1"
      },
      "contentType":{
         "Type":"String",
         "Value":"application/json"
      },
      "timestamp":{
         "Type":"Number.java.lang.Long",
         "Value":"1725037448499"
      }
   }
}


Various fatal error panics when using in goroutines

Hey there. I've been trying to use this snowball stemmer implementation and with linear code it works great. But, unfortunately when I'm trying to wrap it up into goroutines I get these seemingly random panics with different sorts of errors. Perhaps, I'm doing something wrong, but the issue is really easy to reproduce, which I've done by cloning project and writing the following example next to the existent one:
import (
	"fmt"
	"sync"

	"github.com/tebeka/snowball"
	"github.com/zhexuany/wordGenerator"
)

func ExampleStemInGoroutines() {
	var wg sync.WaitGroup

	stemmer, err := snowball.New("english")
	if err != nil {
		fmt.Println("error", err)
		return
	}

	words := wordGenerator.GetWords(500, 20)
	stemmas := make([]*string, len(words))

	for i, word := range words {
		wg.Add(1)

		go func(i int, word string) {
			defer wg.Done()
			stemma := stemmer.Stem(word)
			stemmas[i] = &stemma
		}(i, word)
	}

	wg.Wait()
	fmt.Println("test")

	// Output:
	// test
}

Notice: on a smaller number of words the issues won't reveal themselves, so I had to use github.com/zhexuany/wordGenerator to demonstrate when and how it emerges.
Here are just some errors I'm getting:
fatal error: unexpected signal during runtime execution
snowball.test(2553,0x700001f1f000) malloc: *** set a breakpoint in malloc_error_break to debug
[signal SIGSEGV: segmentation violation code=0x1 addr=0x5afffea pc=0x7fff70574b49]

runtime stack:
runtime.throw(0x416fb4b, 0x2a)
        /usr/local/Cellar/go/1.14.1/libexec/src/runtime/panic.go:1114 +0x72
runtime.sigpanic()
        /usr/local/Cellar/go/1.14.1/libexec/src/runtime/signal_unix.go:679 +0x46a

snowball.test(4038,0x70000950a000) malloc: Region cookie corrupted for region 0x9c00000 (value is 7277)[0x9c0407c]
snowball.test(4038,0x70000950a000) malloc: *** set a breakpoint in malloc_error_break to debug
SIGABRT: abort
PC=0x7fff704c633a m=7 sigcode=0

panic: runtime error: gobytes: length out of range

goroutine 121 [running]:
github.com/tebeka/snowball._Cfunc_GoBytes(...)
        _cgo_gotypes.go:63
github.com/tebeka/snowball.(*Stemmer).Stem.func4(0x5c04338, 0xfffffffe, 0xc000099a00, 0x9, 0x5c04338)
        /Users/smileart/Sync/Projects/snowball/snowball.go:73 +0x59
github.com/tebeka/snowball.(*Stemmer).Stem(0xc00008e020, 0xc000099a00, 0x9, 0x0, 0x0)
        /Users/smileart/Sync/Projects/snowball/snowball.go:73 +0xcb
github.com/tebeka/snowball_test.ExampleStemGoroutines.func1(0xc000098020, 0xc00008e020, 0xc0000cc000, 0x1f4, 0x1f4, 0x56, 0xc000099a00, 0x9)
        /Users/smileart/Sync/Projects/snowball/example_test.go:41 +0x8b
created by github.com/tebeka/snowball_test.ExampleStemGoroutines
        /Users/smileart/Sync/Projects/snowball/example_test.go:38 +0x19a
exit status 2
FAIL    github.com/tebeka/snowball      0.207s

And so on. Also I don't provide all the stack traces cause the issue is really consistent and easy to reproduce with the code I provided above, so I guess it'd be easier for you to get them yourself. I'm not a C expert by any means, but I've seen somewhat similar symptoms discussed in many places (here are just a couple of them: link, link), so my guess would be that either it's my usage or C code which is wrong.
And also here's my Go ENV, JIC:
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/smileart/Library/Caches/go-build"
GOENV="/Users/smileart/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/smileart/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.14.1/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.14.1/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/smileart/Sync/Projects/snowball/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/3x/b7d_4f997rxfrv1qb81hscp80000gn/T/go-build699026935=/tmp/go-build -gno-record-gcc-switches -fno-common"

Thanks in advance. And thank you for the project. 👌🙏

Thanks for reporting, I'll try to have a look soon.
IMO it'll be safter to create a stemmer per goroutine or use them in a sync.Pool, I'm not sure the underlying C library is thread (and goroutine) safe.

I moved the stammer creation inside the goroutine and the code works. I don't have plans to make stemmer goroutine safe but I will add this fact to the documentation.

Sure, but if I provide some library which uses this one internally and creates Stemmer in method New, reusing it later on, I'd have to ask the users to avoid using it in goroutines as well. ¯_(ツ)_/¯ Anyway, thanks for the answer and help. 🤝


README example doesn't work out of the box

Hi!
First of all, I'm really liking your web framework! I just tried it (with a small fix) and it looks nice. It looks compact to write and has some things that made me stop and think "wait, how did he do that?".
I tried the example from the README and doing require "amethyst" didn't work, it can't find the view macro. Doing require "amethyst/all" did work, though, but that's not what the example says.

Hello!I really happy that you liked it, so I can say I like Crystal as well :). Thank you for feedback. This error is due to the fact that I forgot to include Sugar::View implicitly into the Base::Controller class. I'll fix it right now.
P.S. Amethyst is very young, so there are a lot things to do.

@asterite, can you tell me about a "small fix"? It is due require "amethyst" or not?

Oh, the small fix was to use require "amethyst/all". Sorry, that was a bit confusing from my part :-)
Everything's working well now. Thanks!


Added promises from @google

Project URL

https://github.com/google/promises
Category

eventbus
Description

Promises is a modern framework that implements the aforementioned synchronization construct in Swift and Objective-C.
Why it should be included to awesome-ios (optional)
Received >1100 stars in 17 days since it was born at Github!
Simple: The framework has intuitive APIs that are well documented making it painless to integrate into new or existing code.
Interoperable: Supports both Objective-C and Swift. Promises that are created in Objective-C can be used in Swift and vice versa.
Lightweight: Has minimum overhead that achieves similar performance to GCD and completion handlers.
Flexible: Observer blocks can be dispatched on any thread or custom queue.
Safe: All promises and observer blocks are captured by GCD which helps avoid potential retain cycles.
Tested: The framework has 100% test coverage.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 6 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

5
Dupe
https://gitter.im/vsouza/awesome-ios

18
301
https://goo.gl/hNadrZ redirects tohttps://try.instabug.com/awesomeios/?utm_source=awesomeios&utm_medium=spon&utm_content=header

20
301
https://goo.gl/q93Qtd redirects tohttps://try.instabug.com/awesomeios?utm_source=awesomeios&utm_medium=spon&utm_content=learn-more

21
301
https://goo.gl/A74z8Q redirects tohttps://instabug-ga.appspot.com/UA-41982088-6/github/awesomeios?pixel

21
Dupe
https://instabug.com

2705
503
https://boombox.io/

  Generated by :no_entry_sign: Danger


improvement: select "has one" record

then the next level would be handling "has many" and "many to many"

Duplicate of #7


zsh autocompletions break the helper

I am running zsh and autocompletion plugins and this seems to break the helper, error thrown is:
.autocomplete__key-bindings:33: terminfo[kcbt]: parameter not set\n
I ran the binary with xcode debugger and the token is actually correctly returned, but the error message causes downstream failure. May be some kind of environment stripping for the executed command could help?

Thanks a lot for raising this and providing this context!
With the (currently a little hacky) way the helper is evaluating the exec config, any chance you can spot any obvious gotcha that you think could make it work with your setup?

I’m doing some hacking to see if I would be able to come up with at least something dirty, but I am mostly amateur with swift so … it takes time, but I’ll update the issue if I come up with something.

Thank you! If you figure out the correct way of running the command shell-wise, I'm super happy to help with the Swift part! 🙂


Adding Issue_Templates

Will like to provide .yml files for issue templates which will bring better organization and readability!

Fix


Bump ubuntu version for Latex devcontainer

Ubuntu 22.10, Kinetic has reached its end of life, therefore the packages are no longer available.
Bump version to LTS Ubuntu 22.04.x with jammy.

this was actually covered by #3, but thanks for opening this PR! 🙂


Determine transition id before trying to close a ticket

Currently, the transitionID is set to 21. However, this could deviate in a Jira project.

0.6.1


How to predict a result of object detection from a dataset only containing pictures?

I trained a model with my own dataset with annotations in COCO format, now I want to use it to predict a result of another dataset without annotations, and save the result as json files. How can I realize it?
I tried test part in mmdetection, whereas it requires annotations.

Search for the format of test annotation in COCO dataset. Just write a simple script to generate test annotation from pictures.


Using Chartist.js and need to move position of Y axis

I need to move Y axis to 0 and give display few negative fields and few positive .How Can we move Y axis or get expected output shown in Image .In Image red are positive value legends and blue negative value .

Hi @Shreya1771 👋
It can be achieved by catching the draw event and adding necessary classes to elements with positive and negative values. And then styling them. Here's a small example to demonstrate.


Add CMake support for generating Reflex targets

This PR addresses #203, adds a cmake function to create Reflex targets from the specified sources. It aims to provide easy-to-use CMake interface for users wishing to use Reflex as lexer generator. The product of the function is self-sufficient target. In case of a library the users can freely target_link_libraries against it.
The signature of the function is straightforward:
add_reflex_target(
    TARGET_TYPE <EXECUTABLE|LIBRARY>
    TARGET <target_name>
    LIBRARY_TYPE <whataever add_library accepts as type>
    FILES <list of reflex source files>
    INCLUDE_DIRECTORIES <list of public include directories>
    DEPENDENCY_TARGETS <list of targets to put inside target_link_libraries(<target_name> PUBLIC )>
)

I would like to add an example that creates a library and then the library is linked into final executable. CMake has a concept of libraries and executables, but with bison it seems the headers are inside executable target, while the lexer library would depend on that. It makes creating a target rather awkward, so I thought I could get your opinion on other bits first. I also looked at the github actions, but it seems like it does not use cmake at all.

I would like to add an example that creates a library and then the library is linked into final executable. CMake has a concept of libraries and executables, but with bison it seems the headers are inside executable target, while the lexer library would depend on that. It makes creating a target rather awkward, so I thought I could get your opinion on other bits first. I also looked at the github actions, but it seems like it does not use cmake at all.

Sorry, my working knowledge of cmake is limited to help you out. Perhaps check SO, Quora etc for answers? A similar SO Q&A: https://stackoverflow.com/questions/19398113/cmake-and-flex-bison
Perhaps create a couple of example cmake files? E.g. one for just using the reflex library, one for just using a scanner and one for using a scanner and Bison parse?

I have looked at the examples and assuming they are how people usually use it, my code unfortunately will not be helpful as the cmake I wrote would require some valid (not necessarily useful) headers to be generated from reflex. Also it requires a bit more rigid naming with headers, which most people might not find useful. It came against my own usage cases, which is why I will have to abandon this PR for now. Please feel free to incorporate/modify/remove my code as you see fit.

OK. It is a always good sign of a grown-up attitude to give something up if it doesn't meet expectations in the end. Don't get me wrong, I appreciate your work on this. I will keep your suggestions in mind!


Fix spawn protection

Apparently spawn protection can't be set to zero... I need to investigate.

Nevermind, this can be disabled from bukkit.yml


Investigate tradeoffs between redirects and copying files [to correct paths]

Per https://github.com/netlify/next-on-netlify/pull/75#discussion_r549917745 on i18n PR:

I would love to get some insight about impact on HTTP request duration and about impact on build pipeline duration for two repos, one with 10K redirects and one with 10K pages à 100KB each. So that we can make an informed decision about the copy vs redirect approach for the naked routes without locale. I thought of this here, because this "hack" is needed because of the redirect approach.

We can either do something like /some/path   /en/some/path   200 or we copy the file located at .next/serverless/pages/en/some/path.html to both out_publish/some/path.html and out_publish/en/some/path.html. After some discussion during i18n, we decided to stick with the redirect approach that was originally used at the start of the i18n work.

i think the conclusion at the moment is that minimizing build times are a bigger priority and copying files is probably more costly than having many redirects. this is tangentially related to https://github.com/netlify/next-on-netlify/issues/123. we can re-address if redirects become a real issue.

i think the conclusion at the moment is that minimizing build times are a bigger priority and copying files is probably more costly than having many redirects. this is tangentially related to https://github.com/netlify/next-on-netlify/issues/123. we can re-address if redirects become a real issue.


Split sendCreateCommentAction as two parts, one for update comment related informations, another for actions

As title.

Codecov Report

Merging #8784 into master will decrease coverage by 0.02%.
The diff coverage is 47.61%.

@@            Coverage Diff             @@
##           master    #8784      +/-   ##
==========================================
- Coverage   41.27%   41.24%   -0.03%
==========================================
  Files         544      544
  Lines       69839    69855      +16
==========================================
- Hits        28824    28812      -12
- Misses      37328    37359      +31
+ Partials     3687     3684       -3

Impacted Files
Coverage Δ

models/review.go
58.46% <0%> (-1.31%)
:arrow_down:

models/issue_comment.go
47.86% <55.55%> (-0.12%)
:arrow_down:

modules/indexer/indexer.go
44.73% <0%> (-10.53%)
:arrow_down:

modules/avatar/avatar.go
48% <0%> (-6%)
:arrow_down:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

models/repo_indexer.go
66.54% <0%> (-1.82%)
:arrow_down:

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

routers/repo/view.go
37.71% <0%> (-0.88%)
:arrow_down:

modules/sync/unique_queue.go
89.28% <0%> (+10.71%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 72aa5a2...3e88406. Read the comment docs.


helm chart upgrade issues

during an upgrade from chart 0.61.2 to latest I've hit the error
Error: UPGRADE FAILED: template: opentelemetry-collector/templates/service.yaml:37:28: executing "opentelemetry-collector/templates/service.yaml" at <include "opentelemetry-collector.serviceInternalTrafficPolicy" .>: error calling include: template: opentelemetry-collector/templates/_helpers.tpl:154:44: executing "opentelemetry-collector.serviceInternalTrafficPolicy" at <eq .Values.service.enabled true>: error calling eq: incompatible types for comparison

I'm deploying the collector as a statefulset so I'm not supposed to be required to explicitly set service.enabled.
To get around this I've enabled it anyway, another deploy attempt and now I hit
Error: UPGRADE FAILED: template: opentelemetry-collector/templates/NOTES.txt:44:42: executing "opentelemetry-collector/templates/NOTES.txt" at <eq .Values.service.internalTrafficPolicy "Cluster">: error calling eq: incompatible types for comparison

I'm going to keep attempting to fix this, but either the config in _helpers has some bugs or the documentation in Values.yaml needs updating for new requirements.
For reference this is (most) of my custom values I'm setting.
mode: "statefulset"

presets:
  logsCollection:
    enabled: false
    includeCollectorLogs: false
    storeCheckpoints: false

config:
  processors:
    batch: {}
    k8sattributes:
      pod_association:
      - sources:
          - from: resource_attribute
            name: k8s.pod.ip
      # below association matches for pair `k8s.pod.name` and `k8s.namespace.name`
      - sources:
          - from: resource_attribute
            name: k8s.pod.name
          - from: resource_attribute
            name: k8s.namespace.name
    # endpoint ignore list
    filter/spans:
      spans:
        exclude:
          match_type: regexp
          attributes:
            - key: http.target
              value: (/ping|/health|/prometheus/metrics)
    # If set to null, will be overridden with values based on k8s resource limits
    memory_limiter: null

    resourcedetection:
      detectors: [gcp]
      timeout: 10s
  receivers:
    otlp:
      protocols:
        grpc:
          endpoint: ${MY_POD_IP}:4317
        http:
          endpoint: ${MY_POD_IP}:4318
    prometheus:
      config:
        scrape_configs:
        - job_name: common
          scheme: http
          scrape_interval: 60s
          scrape_timeout: 30s
          metrics_path: /prometheus
          kubernetes_sd_configs:
            - role: pod
              namespaces:
                names:
                  - myService

  service:
    telemetry:
      metrics:
        address: ${MY_POD_IP}:8888
      # logs:
      #   level: debug
    extensions:
      - health_check
      - memory_ballast
    pipelines:
      logs:
        receivers:
          - otlp
        processors:
          - k8sattributes
          - memory_limiter
          - batch
        exporters:
          - otlphttp/honeycomb
      metrics:
        receivers:
          - otlp
          - prometheus
        processors:
          - resourcedetection
          - k8sattributes
          - memory_limiter
          - batch
        exporters:
          - otlphttp/metrics
      traces:
        receivers:
          - otlp
        processors:
          - k8sattributes
          - memory_limiter
          - batch
          - filter/spans
        exporters:
          - otlphttp/honeycomb

image:
  repository: gcr.io/xmatters-eng-mgmt/otel/opentelemetry-collector-contrib
  pullPolicy: IfNotPresent
  tag: "0.76.1"
  # When digest is set to a non-empty value, images will be pulled by digest (regardless of tag value).
  digest: ""

# OpenTelemetry Collector executable
command:
  name: otelcol-contrib
  extraArgs:
    - --config=file:/vault/secrets/honeycomb.yaml
serviceAccount:
  create: true
  annotations:
    iam.gke.io/gcp-service-account: opentelemetry-collector@{{ .Values.global.googleProject }}.iam.gserviceaccount.com
  # The name of the service account to use.
  # If not set and create is true, a name is generated using the fullname template
  name: ""

clusterRole:
  create: true
  rules:
  - apiGroups:
    - ""
    resources:
    - endpoints
    - events
    - namespaces
    - namespaces/status
    - nodes
    - nodes/proxy
    - nodes/spec
    - nodes/metrics
    - pods
    - pods/status
    - replicationcontrollers
    - replicationcontrollers/status
    - resourcequotas
    - services
    verbs:
    - get
    - list
    - watch
  - apiGroups:
    - apps
    resources:
    - daemonsets
    - deployments
    - replicasets
    - statefulsets
    verbs:
    - get
    - list
    - watch
  - apiGroups:
    - extensions
    resources:
    - daemonsets
    - deployments
    - replicasets
    verbs:
    - get
    - list
    - watch
  - apiGroups:
    - batch
    resources:
    - jobs
    - cronjobs
    verbs:
    - get
    - list
    - watch
  - apiGroups:
    - autoscaling
    resources:
    - horizontalpodautoscalers
    verbs:
    - get
    - list
    - watch

resources:
  requests:
    cpu: 256m
    memory: 2048Mi
  limits:
    cpu: 512m
    memory: 3072Mi

podAnnotations:
  cluster-autoscaler.kubernetes.io/safe-to-evict: "true"
  vault.hashicorp.com/auth-type: gcp
  vault.hashicorp.com/auth-path: auth/gcp
  vault.hashicorp.com/auth-config-type: gce
  vault.hashicorp.com/role: "opentelemetry-{{ .Values.global.environment }}"
  vault.hashicorp.com/agent-inject: "true"
  vault.hashicorp.com/agent-pre-populate-only: "true"
  vault.hashicorp.com/preserve-secret-case: "true"
  vault.hashicorp.com/agent-inject-secret-honeycomb.yaml: "secrets/opentelemetry/{{ .Values.global.environment }}/honeycomb"
  # the old otlp/metrics and otlp/honeycomb endpoint was `api.honeycomb.io:443`
  vault.hashicorp.com/agent-inject-template-honeycomb.yaml: |
    {{`{{- with secret "secrets/opentelemetry/`}}{{ .Values.global.environment }}{{`/honeycomb" -}}
    exporters:
      otlphttp/honeycomb:
        endpoint: {{ .Data.data.endpoint }}
        headers:
          x-honeycomb-team: {{ .Data.data.apikey }}
      otlphttp/metrics:
        endpoint: {{ .Data.data.endpoint }}
        headers:
          x-honeycomb-dataset: kubernetes_metrics
          x-honeycomb-team: {{ .Data.data.apikey }}
    {{- end }}`}}

service:
  enabled: true
  type: ClusterIP
  annotations:
    consul.hashicorp.com/service-name: otel-collector
    consul.hashicorp.com/service-port: "4317"
    consul.hashicorp.com/service-sync: "true"
    consul.hashicorp.com/service-tags: common

I've fixed the second bug, the default for service are
service:
  # Enable the creation of a Service.
  # By default, it's enabled on mode != daemonset.
  # However, to enable it on mode = daemonset, its creation must be explicitly enabled
  # enabled: true

  type: ClusterIP
  # type: LoadBalancer
  # loadBalancerIP: <IP_ADDRESS>
  # loadBalancerSourceRanges: []

  # By default, Service of type 'LoadBalancer' will be created setting 'externalTrafficPolicy: Cluster'
  # unless other value is explicitly set.
  # Possible values are Cluster or Local (https://kubernetes.io/docs/tasks/access-application-cluster/create-external-load-balancer/#preserving-the-client-source-ip)
  # externalTrafficPolicy: Cluster

  annotations: {}

  # By default, Service will be created setting 'internalTrafficPolicy: Local' on mode = daemonset
  # unless other value is explicitly set.
  # Setting 'internalTrafficPolicy: Cluster' on a daemonset is not recommended
  # internalTrafficPolicy: Cluster

With mode: statefulset these no longer work, I had to uncomment enabled and internalTrafficPolicy
mode: statefulset
service:
  enabled: true
  internalTrafficPolicy: Cluster

@jacob-bresciani-everbridge when I've seen these errors in the past it was due to an old helm version.  What version of helm are you running?

updated the deploy job to use helm 3.12.3 thanks for point out my procrastination :)

Did it solve your problem?

just confirmed, yes it did. Thanks again and this can be closed, "user failed to read requirements"


infra - reduces print() noise and kicks off export after serge sync

Why?

Want to make sure translated files export back to google drive
Reduce the noise in wordpress export print statements that make it harder to find issues
Add a try/except block to capture and print error messages when something goes wrong

Thanks!


Improved onboarding experience

Description
I'm submitting a ...

feature request - please describe the behavior you want and the motivation.

Overview
It would be nice if Yew had an onboarding experience like Elm.
I remember I stumbled across Yew a few months ago, but did not realize what it was about.
About two weeks ago I learned about Elm (mainly through the web page elm-lang.org), and thought to myself "I wish Rust had something like this".
Then, yesterday, I read a blog post about someone comparing web frameworks who mentioned Yew and Elm, and only then (after I knew what Elm was about) I came back to this page and could appreciate what Yew was really about.
Current Problems
Here are some problems and thoughts I had as a first-time user:

Yew's documentation (README.md) starts with implementation details, comparing itself to Elm and WebAssembly. I think it's good to have that, but unless I know both, I don't really know anything.
In contrast, elm-lang.org, starts with user (developer) values: JavaScript Interop, No Runtime Exceptions, Great Performance. I'm not saying that has go into the README, but whatever is considered the main entry point of Yew should probably have it.
For the actual README (or whatever is considered the getting started), a super-simple "hello world" should be provided, along with "out of the box" instructions how to compile and run it. As of today, I have to either clone the repository to follow the instructions, and / or piece these things together manually (e.g., the README says to run $ ./ci/run_tests.sh; what if I don't have Bash)?
After I successfully passed all of the above – I have my first "hello world"; how do I use it from a real  page? How to interface with parts of an existing page should be next to the hello world.

I've added documentation to https://yew.rs/docs/ and https://yew.rs/docs/getting-started let me know if they have the material you had in mind! Thanks for the issue!

The docs look great and I found them easy to follow and the actual landing page /docs is a clear improvement over the previous README.
I still think its narrative speaks primarily to a specific subset of Rust developers (those familiar with React, Elm, actix, async-await, wasm-bindgen; as the landing page primarily explains along those concepts).
I'd argue the page could be even better if it instead emphasized values that matter to web developers and their customers.
Elm
Elm sets the base line here:

No Runtime Exceptions
Great Performance
Enforced Semantic Versioning
Small Assets
JavaScript Interop

These are 5 strong points I care about as a developer, and I care about them for my customers. Even more, I can use the same arguments to convince my manager or team (which might not even care about Elm or Rust) why this technology should also matter to them.
Yew
Right now the page lists:

Architecture
Concurrency
Safety
JavaScript
HTML
Why Rust?
Why WebAssembly?

As a Rust developer I'd expect to find all of these terms on a Rust client web framework, but I'd have to actually read them to understand what they mean. For example:

Architecture - Does that mean I can build a good architecture, or Yew has a good one? Oh, it says it's architecture is inspired by Elm and React. Again, I might not know these, so I'd have to look them up first and understand what makes their architecture great.
Concurrency - Talks about async-await and actor models. Certainly interesting for Rust users which know about them.
...

In contrast, what if Yew could tie in its own strengths with those of Rust, maybe in two adjacent paragraphs. One addressing primarily people concerned with web values (why should I chose Yew over any other web framework), the other people concerned with Rust values (why should I chose Yew over any other Rust web framework):
Yew For Web Developers:

Bare-Metal Speed (wasm ...)
From User Interfaces to IO (actor model, concurrency)
Reliable Code (no exceptions, ...)
Seamless Interop (works well with existing HTML / JS if you have to) -- (TODO: does it?)
Most Loved Language (no more JavaScript if you don't want; 1000's of Rust libraries to pick from)

Yew for Rust Developers:

Modern Foundation (wasm-bindgen, async/await...)
One Mental Model (actix actors, ...)
Best of Both Worlds (fuses React model with Rust model)
...

I am not saying it has to be any of those, but something like that could be helpful on a landing page to immediately address a visitor's question "why should I care?"

Great feedback, I've updated the docs, let me know your thoughts! Thanks in advance :+1:

Looks much cleaner, great improvement! I like that it focuses on a handful of strong points, and they resonate with me.
That pretty much covers my "onboarding" issues, thank you so much!
A few smaller design changes could be done later, but I don't think they are essential and probably just my taste:

Maybe consider having a small diagram or "visual" code snippet for each of the 3 points (components, performance, interop) might look nice, similar to how React, or Angular  do it.
The points on the lower half are also good, but I find them a bit verbose.

I tried to create a PR but still had trouble getting the site to render properly.
Things I was experimenting with:

Side-by-side images for "What is Yew" section turned out to be difficult in the current flow. For this style of page it's probably fine.
Added a "Created with Yew" instead to lighten up flow, just an idea, can probably live without it.
Added a more prominent "Getting Started" image
Offloaded "Still not convinced" section to other page, incl. "Alternatives?"
Changed flow to "What is Yew" -> "Ready to dive in" -> "Join Us"

Screenshot:

Again, not saying it has to be like that; more for myself to get a feeling for the page.

More compact version:

Hi, guys
Just a suggestion: It'll be great if the examples folder is shortly advertised more inside the first few paragraphs so that people who prefer to read code AND who are not used to Rust codebase convention are informed that there are the examples folder. @jstarry helped me close an issue just now and it made me realize that yew has a lot of gems inside the examples folder. It's just hard to notice right now, at least for me.
And, another rough suggestion, I think yew will need the paradigm introduction like react does with https://reactjs.org/docs/thinking-in-react.html

There is a link to the examples in the sidebar, do you think this does the trick?

That pretty much covers my "onboarding" issues, thank you so much!

Agreed.

Just a suggestion: It'll be great if the examples folder is shortly advertised more inside the first few paragraphs so that people who prefer to read code AND who are not used to Rust codebase convention are informed that there are the examples folder.

yew.rs has an example page within the Getting Started section now so I think this is covered too.
I think this can be closed now - documentation will never be "perfect" but I think this issue has served it's purpose and any more tweaks wanted/required can come in another issue :)


upgrade clusterinfo crd

Signed-off-by: Zhiwei Yin<EMAIL_ADDRESS>story: https://github.com/open-cluster-management/backlog/issues/11698

/lgtm


Issues with Certificate (Wildcard/SAN)

Hi all,
I'm currently facing the issue that docker does not accept my Wildcard Certificate, which has a Subject Alternative Name (DNS Alternative) additionally.
Let me explain:
First case:
My private docker registry (in this case an artifactory) is provided through TCP/443 with CN=*.sub.domain.tld and my client is able to pull the images from this repository.
Second case:
I'm requesting a new certificate, while I'm leaving the CN=*.sub.domain.tld, but I add a SAN-Entry for sub.domain.tld (why this is necessary can be provided if needed).
In this case my docker login tells me:
docker login repo.sub.domain.tld
Username: admin
Password:
Error response from daemon: Get https://repo.sub.domain.tld/v1/users/: x509: certificate is valid for sub.domain.tld, not repo.sub.domain.tld

Is this behaviour by design and is the SAN more weighted than the CN?
My openssl-certificate request:
openssl req -new -newkey rsa:4096 -nodes -sha256 -subj "/C=DE/O=PRIVATE/OU=Test/CN=*.sub.domain.tld" -out /docker/ssl/sub.domain.tld.csr -keyout /docker/ssl/sub.domain.tld.key -config <(
cat <<-EOF
[req]
default_bits = 4096
default_md = sha256
req_extensions = req_ext
distinguished_name = dn
[ dn ]
[ req_ext ]
subjectAltName = @alt_names
[alt_names]
DNS.1 = sub.domain.tld
EOF
)

Output of docker version:
Client:
 Version:      1.11.2
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   b9f10c9
 Built:        Wed Jun  1 21:23:11 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.2
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   b9f10c9
 Built:        Wed Jun  1 21:23:11 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 6
 Running: 1
 Paused: 0
 Stopped: 5
Images: 7
Server Version: 1.11.2
Storage Driver: devicemapper
 Pool Name: docker-202:1-469779312-pool
 Pool Blocksize: 65.54 kB
 Base Device Size: 10.74 GB
 Backing Filesystem: xfs
 Data file: /dev/loop0
 Metadata file: /dev/loop1
 Data Space Used: 9.868 GB
 Data Space Total: 107.4 GB
 Data Space Available: 97.51 GB
 Metadata Space Used: 10.49 MB
 Metadata Space Total: 2.147 GB
 Metadata Space Available: 2.137 GB
 Udev Sync Supported: true
 Deferred Removal Enabled: false
 Deferred Deletion Enabled: false
 Deferred Deleted Device Count: 0
 Data loop file: /var/lib/docker/devicemapper/devicemapper/data
 WARNING: Usage of loopback devices is strongly discouraged for production use. Either use `--storage-opt dm.thinpooldev` or use `--storage-opt dm.no_warn_on_loop_devices=true` to suppress this warning.
 Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
 Library Version: 1.02.107-RHEL7 (2015-12-01)
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: null host bridge
Kernel Version: 3.10.0-327.18.2.el7.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 3.451 GiB
Name: ip-172-16-0-126.eu-central-1.compute.internal
ID: KD6M:PLXA:PIVD:J45T:CKF6:53YE:3BKW:5OCP:TNVA:PNRK:CNHF:2MYM
Docker Root Dir: /var/lib/docker
Debug mode (client): false
Debug mode (server): false
Registry: https://index.docker.io/v1/
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled

Cheers,
Matthias

/cc @dmcgowan

This behavior does appear intentional, see https://golang.org/src/crypto/x509/verify.go?s=12675:12734#L433. I do not know the reason behind this design since this was a decision by the Go team, but it does seem easy to accommodate the desired functionality by adding the wildcard also as a SAN.

thanks!


activePane is deprecated

closes https://github.com/fcharron/last-cursor-position/issues/18

:rose:

I should be the one thinking you for the fix ;)


Add gpx support

This PR adds support for GPX loading. It's based off this old PR https://github.com/CesiumGS/cesium/pull/2939.
Test files for waypoints, tracks and routes have been added, there is also a SandCastle  with a dropdown menu for different gpx files here: http://localhost:8080/Apps/Sandcastle/index.html?src=GPX.html.  ( test it online )
On top of the rebasedPR we added some symbology options: clampToGround, trackColor and waypoint and track images. Also path symbols have been removed in favor of polyline symbols. The main reason is that path symbols does not support clamp to ground so it was done for the sake of consistency within the symbology options.

Thank you so much for the pull request @jtorresfabra! I noticed this is your first pull request and I wanted to say welcome to the Cesium community!
The Pull Request Guidelines is a handy reference for making sure your PR gets accepted quickly, so make sure to skim that.

:x: Missing CONTRIBUTORS.md entry.

Please add yourself to the contributors file!

:x: Missing CLA.

Please send in a Contributor License Agreement (CLA) and comment back here to let us know to check this!

:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

I've added you to Cesium GS's corporate CLA. Don't forget to add yourself to CONTRIBUTORS.md under Cesium GS!

@OmarShehata thanks for the review!
I tried to address all the comments. But there are still some concerns:

About clamp to ground: I implemented an option to pass clamp to ground to billboards and polylines, but unfortunately  PathGraphics don't support clamp to ground (https://github.com/CesiumGS/cesium/issues/7133). So I'm not sure if that could be misleading for users that won't see their dynamic lines clamped to terrain. I could be missing something, though.

I'm going to look for better data and implement more tests.

About clamp to ground: I implemented an option to pass clamp to ground to billboards and polylines, but unfortunately PathGraphics don't support clamp to ground (#7133). So I'm not sure if that could be misleading for users that won't see their dynamic lines clamped to terrain. I could be missing something, though.

Ok, this is a bit tricky. Clamping some things but not everything is definitely going to be confusing. I think the best options are:

Nothing is clamped. Not confusing, but not ideal in practice.
Just use polyline instead of path. So you'll see the full line and we give up the dynamic path, and just see the marker moving along that line.

My preference is (2). It is nice to have the animated path, but it seems more useful to be able to correctly see the full path on the ground in many cases.

I changed the code to use polylines instead of paths so we always can clampToGround.
Also I implemented more tests and added my own gpx track to the examples. I checked the other files and they are really into the public domain, no license though, but gpx code was there to copy-paste. I removed some of them not used though.
The good point about those files is that they show different features of gpx (waypoints, tracks and routes).
Do you think it is ready to change from Draft PR to "Real" PR?

Great!
Yeah feel free to change it to "ready for review" if you feel like it's ready. I'll take another pass before bumping it to a final reviewer.

Tests are failing, but master is also failing for me.
cc @OmarShehata

Looks great @jtorresfabra ! I pushed some minor cleanup things. I have just a few more things minor things. Once these are cleaned up please bump to Matt. It would be good to update the PR description to make it easier for the reviewer, with a summary of the changes and decisions we've discussed here, like that the GpxDataSource support passing in images, uses polyline instead of path etc. As well as any relevant instructions for testing (such as a link to the Sandcastle for this branch for convenience: http://cesium-dev.s3-website-us-east-1.amazonaws.com/cesium/feature/rebased-gpx/Apps/Sandcastle/index.html?src=GPX.html).
Also, I noticed you force pushed a few times since the last review. It would have been good to retain the commit history so we could see the initial rebase commit followed by the commit(s) where we changed the implementation from the initial PR. Something to keep in mind for the future.

The GpxDataSource doc page doesn't seem to document clampToGround ?
The doc page for .load says "An object with the following properties:" but doesn't list any properties?
For the Sandcastle, should the last "GPX - Route" also have a clampToGround: true option?
I think the viewer.homeButton.viewModel.command(); call in each one isn't necessary and causes an extra camera flyTo
It would be nice if one of the GPX files added in the Sandcastle shows how to use one of the options, like how to change the track color

Thanks @OmarShehata!
I think it is now ready for Matt review.
CC: @mramato please have a look when you have the time. Thanks in advance.

We are also very interested in this feature !

Do you have any timeline / plan to review / merge it? Thanks a lot in advance!

Thanks again for your contribution @jtorresfabra!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @jtorresfabra!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @jtorresfabra!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @jtorresfabra!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @jtorresfabra!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @jtorresfabra!
No one has commented on this pull request in 90 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 90 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

Thanks again for your contribution @jtorresfabra!
No one has commented on this pull request in 90 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 90 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

@eleu This PR fell off our radar but we're close to getting this feature in. We'd appreciate any feedback or sample data you may have.

Also please merge in main when you get the chance. A new eslint rule has gone in.

Thanks @sanjeetsuhag and all those involved! I think this is in a good place and we're ready to merge, exciting!


tour: introduce collapsible hint sections on page

Some recent CLs to the tour added hints to certain problems
where the user used to have to think for themselves.
I suggest that we add collapsible hint sections to hide these
spoiler contents by default.

I'd prefer not to do this. Experience has shown that many tour users font
read the text when it is right in front of them, hiding it would be counter
productive.
On Fri, 9 Oct 2015 11:22 Minux Ma<EMAIL_ADDRESS>wrote:

Some recent CLs to the tour added hints to certain problems
where the user used to have to think for themselves.
I suggest that we add collapsible hint sections to hide these
spoiler contents by default.
—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/12885.

I would prefer to write more text to go inside collapsable sections, rather
than hiding any existing text.
There's a lot more explanation we should give, and we should give it to
people who are interested.
On 9 October 2015 at 11:28, Dave Cheney<EMAIL_ADDRESS>wrote:

I'd prefer not to do this. Experience has shown that many tour users font
read the text when it is right in front of them, hiding it would be counter
productive.
On Fri, 9 Oct 2015 11:22 Minux Ma<EMAIL_ADDRESS>wrote:

Some recent CLs to the tour added hints to certain problems
where the user used to have to think for themselves.
I suggest that we add collapsible hint sections to hide these
spoiler contents by default.
—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/12885.

—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/12885#issuecomment-146725036.

Yeah, the intention is not hide any existing text. The new collapsible
sections will contain more detailed explanations and when new
questions arise, we can answer them in the collapsible sections so
that the page is not cluttered with unnecessary text.


fixes https://github.com/tensorflow/playground/issues/161

I have signed the Google Individual Contributor License Agreement.

可以拉取請求合併嗎謝謝


Implement user profile ActivityView

I've merged the current implementation to unlock the next steps, but we have to clean-up the code in parallel #33


Sample weight for XGBoost

In Python XGBoost one can provide weights for each row of the data, see http://xgboost.readthedocs.io/en/latest/python/python_api.html#xgboost.XGBClassifier.fit. I tried to look for a way to specify such weights in SharpLearning, but could not find it. Is this possible?

Hi @petterton ,
Currently there is no support for providing sample weights. I have a few tasks for adding this to all the learners, however, I have not completed these yet.
I might add it to the XGBoost learners as a starting point, without adding the option to the general learning interfaces. I will see if I can fit it in during the coming weeks.
Best regards
Mads

@petterton, I was trying add sample weight support for the XGBoost learners today. However, it seems that the DMatrix implementation in XGBoost.Net, which I am using to interface with the native xgboost lib, does not support setting the sample weights yet. This makes things slightly more complicated, so adding this will take more time, and depends on when the guys at XGBoost.Net have time to add this. I have added an issue for it: https://github.com/PicNet/XGBoost.Net/issues/26

Thanks for your efforts @mdabros! I will follow developments on the issue at XGBoost.Net.


Revisit b5929e7

Just a reminder. We will retest when the next OpenDSS release is out anyway.

SVN r3477 adds some changes related to this. We will reapply and retest whenever new official  OpenDSS binaries are available (their latest release is affected by the issue r3477 mentions).

Changes reapplied. Before closing this, we should retest and check, then prepare a new release.


Session hijacking buggggggggg

Hi. why laravel doesn't prevent session hijacking....?!!!!!!!!!!!!!!!!!!!!!!!!!!!!

i injected generated session in another browser cookie and booooooooooooom i logged in...!!!!!!

@saeedvaziry
This is the standard behavior, you can perform periodic session id regeneration, but this may cause unexpected behavior on concurrent requests,
So the best idea would be using HTTPS to prevent man in the middle attacks which may increase the probability of session hijacking.

is there any way to prevent this?

is there any way to prevent this?

@reshadman already give you a way "So the best idea would be using HTTPS to prevent man in the middle attacks"

So the best idea would be using HTTPS to prevent man in the middle attacks which may increase the probability of session hijacking.

+1 this and avoid XSS issues in your website.
You could also check in a middleware to ensure the user agent is still the same but it is not bulletproof and I would not recommend doing that either as it can have side effects.

@crynobone @reshadman
HTTPS is not 100% secure...! if you have a powerful server , you can decrypt it...!

Nothing is 100% secure...

@kamui545 sure...

There is no reliable way of doing this, ip and user-agent can be forged.
Best you can do is on your own application, ask for re-entering of password after certain time passed since last activity, regenerate session after significant actions (change password, update details etc.) and invalidate previous cookie.

For those who wat a tad bit more security for session hijacking:
function setStrictCookie($name, $value='', $maxage=0, $path='', $domain='', $secure=false, $HTTPOnly=false, $strictMode=null)
    {
        $ob = ini_get('output_buffering');

        // Abort the method if headers have already been sent, except when output buffering has been enabled
        if ( headers_sent() && (bool) $ob === false || strtolower($ob) == 'off' ) {
            return false;
        }

        if ( !empty($domain) )
        {
            // Fix the domain to accept domains with and without 'www.'.
            if ( strtolower( substr($domain, 0, 4) ) == 'www.' ) {
                $domain = substr($domain, 4);
            }
            // Add the dot prefix to ensure compatibility with subdomains
            if ( substr($domain, 0, 1) != '.' ) {
                $domain = '.'.$domain;
            }

            // Remove port information.
            $port = strpos($domain, ':');

            if ( $port !== false ) {
                $domain = substr($domain, 0, $port);
            }
        }

        // Prevent "headers already sent" error with utf8 support (BOM)
        //if ( utf8_support ) header('Content-Type: text/html; charset=utf-8');

        header('Set-Cookie: '.rawurlencode($name).'='.rawurlencode($value)
                                    .(empty($domain) ? '' : '; Domain='.$domain)
                                    .(empty($maxage) ? '' : '; Max-Age='.$maxage)
                                    .(empty($path) ? '' : '; Path='.$path)
                                    .(!$secure ? '' : '; Secure')
                                    .(!$HTTPOnly ? '' : '; HttpOnly')
                                    .(is_null($strictMode) ? '': '; SameSite='.$strictMode)
                                    , false);
        return true;
    }

and then set the session cookie with
$cookie = new Cookie(
        session()->getName(),
        session()->getId(),
        time() + (7 * 24 * 60 * 60),// 7 days
        '/',
        $domain, // your domain
        false // http secure
        );

setStrictCookie(
        $cookie->getName(), // get the name
        $cookie->getValue(), // get the value
        $cookie->getExpiresTime(), // get the expire time
        $cookie->getPath(), // the cookie path
        $cookie->getDomain(), // the cookie domain
        true, // only serve cookie when on https(recommended. get free https certificate at https://letsencrypt.org/)
        true, // http only flag to protect against session hijacking via xss
        'lax' // only send cookie when the url in address bar actually changes.

When a user logout should the session not expire or be revoked?

HTTPS is not vulnerable to this, provided you set the correct header security.

@GrahamCampbell But... when the user logout, the cookie session should be revoked or not? To prevent Session hijacking I use HTTPS; but I dont understand why my session cookie still works after user logout.

prevent Session hijacking I use HTTPS;

You also need to set the cookie to be "secure" and "http-only".

Yes, I know that; but when I logout, this old cookie session, can be used to login...
Should old "revoked" cookie session works after logout?

Yes, I know that; but when I logout, this old cookie session, can be used to login...

That's not a security problem though, actually, since nobody can access that.
NB If you're using a session driver such as redis, they will actually be garbage collected for you, after they expire.

I believe that it is a security proble; someone with temporary physical access to the computer could extract the cookie and use it whenever he wanted to be logged in. (Im a little paranoid, but I think this is possible).
I will check redis session driver, but... Do you know any way to prevent the use of old ("expired") cookies on laravel 5.4?
Thanks.


Adding auto-QC to Snakemake pipeline to filter out samples with bad quality in the middle of the pipeline steps

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

My question is similar to this one: Conditional execution of multiplexed analysis with snakemake

I tried the dariober's gist answer with "if-else" in shell, but the else sentence output empty file or folder would always been deleted by snakemake and snakemake would report error (I even added "|| true" for each shell command to prevent non-zero exit code but still doesn't help). If I run the if-else shell outside of snakemake, files and folders would be generated though.
I then tried to write python script instead of shell in "run" section, something like:
rule G_move_7:     input:          passORnot_file = '{sample}/XXX_RData/{sample}.passORnot.txt',         sample_folder = '{sample}',         filtered_bam = '{sample}.Aligned.out.filtered.bam',     output:          move_log = '{sample}.log',     output     run:         with checkpoints.F_XXX_withCheck_6.get(sample=wildcards.sample).output[2].open() as f:             if f.read().strip() == "PASS":                 os.system("mv -f" + input.filtered_bam + " ./PASS/ > " + output.move_log)                 os.system("mv -f" + input.sample_folder + " ./PASS/ >> " + output.move_log)                 os.system("echo " + input.sample_folder + " ./PASS/ >> " + output.move_log)             else:                 os.system("mv -f" + input.filtered_bam + " ./NOT_PASS/ > " + output.move_log)                 os.system("mv -f" + input.sample_folder + " ./NOT_PASS/ >> " + output.move_log)                 os.system("echo " + input.sample_folder + " ./NOT_PASS/ >> " + output.move_log)
This works but after getting all the passed sample to the folder "PASS", I have to run the downstream steps with a new snakemake file to use wildcards for each samples in the PASS folder. So now my pipeline was cut to two parts. I have to run first piece and then the second snakemake file. This is fine with only one "pass or not" conditions, but if we add more quality check steps, this would become endless and not doable.
Just wonder is there any more efficient and elegant way to do such auto-QC and filtering-out bad samples in snakemake? (or any other alternatives?) It's better that it can write all QC and filtering within one pipeline files. Thanks.

This is now answered on stack overflow and will soon also land in the upcoming Snakemake paper.


AmbiguousMatchException with PresentationDocument

Hello, testing with a very simple PPTX and getting a AmbiguousMatchExpression exception with the call to GenerateSourceCode()
 at System.RuntimeType.GetMethodImpl(String name, BindingFlags bindingAttr, Binder binder, CallingConventions callConv, Type[] types, ParameterModifier[] modifiers)   at System.Type.GetMethod(String name)   at Serialize.OpenXml.CodeGen.OpenXmlPartExtensions.BuildEntryMethodCodeStatements(IdPartPair part, NamespaceAliasOptions opts, IDictionary2 typeCounts, ISet1 namespaces, OpenXmlPartBluePrintCollection blueprints, KeyValuePair2 rootVar)   at Serialize.OpenXml.CodeGen.OpenXmlPartExtensions.BuildEntryMethodCodeStatements(IdPartPair part, NamespaceAliasOptions opts, IDictionary2 typeCounts, ISet1 namespaces, OpenXmlPartBluePrintCollection blueprints, KeyValuePair2 rootVar)   at Serialize.OpenXml.CodeGen.OpenXmlPackageExtensions.GenerateSourceCode(OpenXmlPackage pkg, NamespaceAliasOptions opts)   at Serialize.OpenXml.CodeGen.OpenXmlPackageExtensions.GenerateSourceCode(OpenXmlPackage pkg)
Is this is known issue or would it be related to running in LinqPad?

Hi, sorry for the delay.
This shouldn't be related to linqpad.
Looking at the error msg, it looks like there is a conflict with the type names being used. Most likely from 2 types in 2 different namespaces but with the same name that i was not aware of. This scenario is mostly common with pptx and docx files.
Can you share the file in question so i can debug and see what I can do?  Also, are you using the latest source or what is in nuget today? There may be a possibility that i haven't pushed this fix to nuget yet.

samplepptx.pptx
Here is a very simply pptx I got off the internet that has the same ambiguous exception

Ok, the most current release is 0.3.2-alpha here and in nuget.  You may have not seen it in nuget because it is still considered a prerelease.
Also,
Looking at the project changelog, it looks like I fixed an ambiguous exception issue with the 0.3.0-alpha release.  I'm going to make sure that the latest release is available here (looks like I haven't updated github in awhile, my fault).  Can you check your office document file against the 0.3.2-alpha release and see if the exception still occurs?

This release - v0.3.2-alpha should fix this issue.  Please let me know what you find.

I can confirm that the AmbiguousMatchException does not occur with the latest version, 0.3.2-alpha, using the referenced file in this issue. I will close this issue later today unless there are any further followups.

Closing issue because fix has already been applied to main branch.

yes that fixed the issue.
I was trying to make sense of what it generated and forgot to reply, sorry


Add missing isLinux checking to the whichOs detection

Changeset:

Add missing isLinux checking to the whichOs detection
Cover with UT
Update changelog

👍

👍


Make test can not run a single test

Tell us about your request
Run make test to be able to test a single test, like termination. For now, using TEST_FILTER env doesn't work.
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Tried exporting TEST_FILTER with file name, test name and keywords. But none of them work.
TEST_FILTER=github.com/aws/karpenter-core/pkg/controllers/termination make test
TEST_FILTER='Termination Reconciliation node that could not be drained [BeforeEach] should delete nodes that have timed out drain' make test
TEST_FILTER='timed\ out' make test
Are you currently working around this issue?
No
Additional Context
No response
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

FOCUS "Scheduling" make test works for me!

FOCUS="Scheduling" make test

is not obvious + is not documented
produces lots of noise

can we make go test github.com/aws/karpenter/pkg/controllers/interruption work ? (so regular go devs can run tests without having to lookup specifics)

Lots of ways to skin this cat. I played with a couple approaches and landed on this one. As a developer, I just run go test github.com/aws/karpenter/pkg/controllers/interruption if I want to test that directly.
❯ go test github.com/aws/karpenter/pkg/controllers/interruption
ok  	github.com/aws/karpenter/pkg/controllers/interruption	11.203s

for me this always results in:
[BeforeSuite]! [PANICKED] [0.145 seconds]
[BeforeSuite]
/Users/mgrosser/Code/tools/karpenter/pkg/controllers/interruption/suite_test.go:90

  Test Panicked
  In [BeforeSuite] at: /Users/mgrosser/go/1.19.0/pkg/mod/github.com/samber/lo@v1.33.0/errors.go:53

  unable to start control plane itself: failed to start the controlplane. retried 5 times: timeout waiting for process etcd to start successfully (it may have failed to start, or stopped unexpectedly before becoming ready)

  Full Stack Trace

but somehow make test works, so I though this is some setup issue where only make test is supported

export KUBEBUILDER_ASSETS ?= ${HOME}/.kubebuilder/bin

this worked:
export KUBEBUILDER_ASSETS=${HOME}/.kubebuilder/bin

... can we make this either the default or fail when it's set incorrectly ?

This is unfortunately buried in the testenv library. I'd recommend just setting it up in your bashrc.

can the tests or some helper fail when it's not set to make this easy to discover ?
... alternatively is there maybe a good place for "development" docs ?

Check out https://karpenter.sh/v0.18.1/contributing/development-guide/


Batch_size ignored

Hi, I am using the following model.
# preprocess input
x_train = preprocess_input(x_train)
x_val = preprocess_input(x_val)

# define model
model = Unet(BACKBONE, encoder_weights='imagenet')
model.compile('adamax', loss=bce_jaccard_loss, metrics=[iou_score])

# model.summary()
# fit model
model.fit(
    x=x_train,
    y=y_train,
    batch_size=16,
    epochs=200,
    validation_data=(x_val, y_val))

However, no matter what batch_size am using, model always trains as if the batch size was set to one even though it is 16. That is, the number of steps per epoch is equal to the number of samples in the training data set. I would appreciate any help on that issue.

It might be because the steps_per_epoch variable isn't set, have you given that a try?
steps_per_epoch: Integer or `None`.
                Total number of steps (batches of samples)
                before declaring one epoch finished and starting the
                next epoch. When training with input tensors such as
                TensorFlow data tensors, the default `None` is equal to
                the number of samples in your dataset divided by
                the batch size, or 1 if that cannot be determined.

See here for Keras' training.py


fix aliased variables being emmited twice

Hi there, this is just a quick fix for a bug I had.
Aliased tokens with different modes were emitted twice (with two modes) as there were two place in the code where modes were taken into account.
This removes the loop in the processAliasModes fn, just keeping L98 which is already looping over modes.
I didn't tested it with more than two modes for now.

Thanks for the plugin, I'm doing research for implementing a multi-brand design system and I'll maybe use it, could contribute more if it is the case and you're open to it.

Hey @MonsieurMan this makes sense.
Could you explain a way to test this to me?
How to I reproduce the issue with double variables? What setup do I need to create this? Afterward I can test this.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 6 (16.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 66.667%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/utilities/getVariables.ts
1
6
16.67%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
496

Relevant Lines:
728

💛  - Coveralls

@JackHowa this would be great. All we need is a way to test this and than we can merge it.

Cool, working on it now.

I like this attempt and appreciate the movement on it. But I tested this locally and it still mistook which mode was referring to which reference token by loading the in-development manifest

This works as expected for me locally. I will work on trying to unit test extractTokens. But will involve mocking figma's api

@lukasoppermann @MonsieurMan thanks for your work on this and patience. I have added some tests in a separate pr off the default that shows the alias modes emitting two values to show https://github.com/lukasoppermann/design-tokens/pull/299/files#diff-18b59c301a30dbea8cef574b8e83d81e812fa126a021df0860ff518f3dd21035R17

@JackHowa can you please resolve conflicts? I think you know better what is correct.

@JackHowa can you please resolve conflicts? I think you know better what is correct.

sorry, this isn't my pr. looks like my pr was just merged! https://github.com/lukasoppermann/design-tokens/pull/300 🥳

@JackHowa sorry, I completely missed this the whole time. 🤦

@JackHowa sorry, I completely missed this the whole time. 🤦

hahah no worries at all. thanks for the quick replies. your plugin is a lifesaver for my org


Display "is typing"

Typing uses EChatEntryType.Typing = 2
Official client emits this only once every 20 seconds but this timer resets once the message is sent.

[x] display "is typing" coming from other users
[x] send "is typing" to other users

Took a while but finally done.


Syntax highlighting is broken

Custom components are now highlighted weirdly (only the first letter is highlighted with proper color), but should be like this:

Weird behavior in html too:

Custom components (MutationMatchBadge and BiomarkerMatchBadge) should be red, but after removing Volar and installing Vue official plugin they became yellow. The craziest part is that highlighting depens on scroll for some reason. You can see it on minimap:
Out of view:

In view:

It can be seen while scrolling fast up and down.

Duplicate of https://github.com/vuejs/language-tools/issues/4070

And https://github.com/vuejs/language-tools/issues/3935


Gitops 1323 config dynamic plugin

What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
GITOPS-1323, pull gitops-plugin image into operator.
Have you updated the necessary documentation?

[x] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #? GITOPS-1323
Test acceptance criteria:

[x] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:

Create a docker image base off this branch code and install it to a cluster
Install pipelines-operator
Apply argocd manifests
Observe the Dynamic Plugin is available on Developer/Environment - Dynamic

/test v4.10-e2e

/test v4.10-e2e

/test v4.9-e2e

/test v4.8-e2e

/test v4.8-e2e

/test v4.8-e2e

/test unit

/test v4.10-e2e

@ciiay you will have to have all your commits signed off on to pass the DCO check

/test v4.8-e2e

/test v4.8-e2e


Update README to include prerequisite .NET 4.6.2 Developer Pack for Windows

This fixes a bug with Lab05 when using a fresh install of VS Community Edition 2017.
The dotnet run -f net462 command would fail on Lab05 with a message
about needing the 4.6.2 framework. I reproduced the error on a fresh
Windows 7 install with VS2017 Community. Before installing, the error
occurred. After installing and opening a fresh command window, the
app runs successfully.

@micahyoung : Thanks!


Use DoAsync in other queries WIP

WHY:
I want to run async queries and get the task id for other queries as well (not only reindexer)

Thanks for contributing to Elastic. I will review and see what we can do...

I added DoAsync to both UpdateByQuery and DeleteByQuery. However, I decided to implement it with my own version. While extracting common code is generally a good idea, it has backfired with Elastic quite often as ES changed things in a way that breaks the common basis. So I tend to copy & paste in order to be prepared for future changes.

thank you!


There should be a list (somewhere) of the correspondence between Calc versions and features

so there is a reference for the values in :requires.

Running c versions will give a nicely formatted list of version number and comment of what changed.


Updates the min go version to 1.17.

This change updates the minimum go version to 1.17, and removes the pinned version of grpc-gateway.  It was pinned because later versions required 1.17.
TODO: The GitHub action needs updating to 1.17, but I can't push a change to the workflow.  This doesn't affect the generation steps.

Hey @MadVikingGod , any updates on this PR? We are blocked by our security systems due to vulnerabiltiies in grpc-gateway