Add date-range filtering to Adjustments#index

Summary
We currently have the ability to filter by date range on Donations and Purchase. We'd like to extend this functionality to other resources as well. You should be able to copy it over, though it will require you to look in a few locations.
Our filtering stack involves model, view, and controller for each resource. It's always done in the :index action. Look to Donation and Purchase for how it's done. There should also be a concern or partial that you can re-use.
Ref #1238
Criteria for Completion

[ ] Adjustments#index (on :created_at)
[ ] Test coverage that it works

I can add this.


lots of "defined in the headers but not the wiki!" warnings

E.g.: WARNING: Mix_AllocateChannels defined in the headers but not the wiki!
Seen in the CI logs too.

I think we need someone to go through and update the wiki with all of our changes.
Any volunteers?

These are caused by this line:
https://github.com/libsdl-org/SDL_mixer/blob/14f1eb25603cc75624e9d2c92a5b0eebcf93a015/.wikiheaders-options#L15
It is set to 0 in SDL, but to 1 in the satellite libraries.
I asked here: https://github.com/libsdl-org/SDL/issues/7514#issuecomment-1608560156 (but I don't think it got answered)

I think we need someone to go through and update the wiki with all of our changes.
Any volunteers?

The manpage generator script always generates the wiki/man pages from scratch.
So when first generating the docs, all pages are guaranteed to be missing.

Oh, okay. So is it fine just to remove the warning?

I think the warning makes sense when updating the sdl wiki, but not when generating the headers.
So perhaps it should be changed into a cli argument-only option.

https://github.com/libsdl-org/SDL_mixer/pull/552 seems to have fixed this.

BTW, the SDL2 branch has the same warn_about_missing = 1 but I don't remember getting the warnings.


Update proposal cli command needs to include a field for cost model

Resolved by https://github.com/input-output-hk/cardano-node/pull/3888


[docs] Update the summaries for ksonnet-related Github repos

Since these are pretty much all public.
I don't have permissions to change some of these. Open to suggestions/rewording, just want to make sure this gets done, for a polished look.
Currently:

Repo Title
Needs change?
Summary

ksonnet
N (?)
A CLI-supported framework that streamlines writing and deployment of multi-cluster Kubernetes configurations.

parts
N (?)
A collection of ksonnet registries, libraries, and prototypes that enable reuse of common Kubernetes configurations

ksonnet-lib
Y
~(technical preview) Simplify working with Kubernetes~Codegen for Kubernetes-API-compatible Jsonnet libraries

homebrew-tap
N
Homebrew tap for installing ksonnet software

I think I need @hausdorff 's help for ksonnet-lib and @jbeda 's help for homebrew-tap. Thx!
Screenshot so you're clear on what i'm talking about:

I think homebrew-tap may be OK as it exists, because it'll likely be extended to install more than ks (i.e. if kubecfg ever needed a homebrew recipe, it'd probably go there)

ah ok, that makes sense @jessicayuen ! We can leave that as is

Yeah -- I'm thinking we'll help get kubecfg into homebrew soon-ish.  Will reach out to @anguslees to see what he thinks.

Sg. There's been a homebrew recipe floating around for ages, I just don't
know what I'm supposed to do with it (not a Mac user).
On Mon, 4 Dec 2017, 13:48 Joe Beda<EMAIL_ADDRESS>wrote:

Yeah -- I'm thinking we'll help get kubecfg into homebrew soon-ish. Will
reach out to @anguslees https://github.com/anguslees to see what he
thinks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ksonnet/ksonnet/issues/184#issuecomment-349083680,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANr4IxpPRJd-SC3RHEHrobm4MXJYCk8ks5s9EyMgaJpZM4Qxweu
.
Message protected by MailGuard: e-mail anti-virus, anti-spam and content
filtering.
http://www.mailguard.com.au/mg
Report this message as spam
https://console.mailguard.com.au/ras/1Sdg3x7M46/5cNJqZduLqAR2n4ZYLsMhQ/3.3

Hi. This seemed like an appropriate issue to piggyback on. Please explain the relationship between https://github.com/ksonnet/ksonnet and https://github.com/ksonnet/kubecfg in the documentation.


How to compile scripts with g++

Hi
Basic question from a newcomer to GRSISort. I downloaded the leangrsi branch of the code and now I am trying to compile my own script "myanalysis2" (based on examples in the GRSISort directories) to produce histograms etc.
I would have attached my code but GitHub doesn't accept any of my files, even .txt files. If anyone is interested I can email the file.
I use the following to compile:
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ root-config --cflags --libs
But I get a few error messages (the 'undefined reference' type). Again I can email the script and a transcript of the errors if required.
Best,
Mike Bowry

The easiest thing to do is put it in the myanalysis directory with a cxx extension and just type make. All the libraries will be linked and includes included automatically.

On Sep 25, 2015, at 6:30 PM, MikeBowry<EMAIL_ADDRESS>wrote:
Hi
Basic question from a newcomer to GRSISort. I downloaded the leangrsi branch of the code and now I am trying to compile my own script "myanalysis2" (based on examples in the GRSISort directories) to produce histograms etc.
I would have attached my code but GitHub doesn't accept any of my files, even .txt files. If anyone is interested I can email the file.
I use the following to compile:
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ root-config --cflags --libs
But I get a few error messages (the 'undefined reference' type). Again I can email the script and a transcript of the errors if required.
Best,
Mike Bowry
—
Reply to this email directly or view it on GitHub.

Also in case you are wondering we have a grsi-config that can be used the same as the root config.

On Sep 25, 2015, at 6:30 PM, MikeBowry<EMAIL_ADDRESS>wrote:
Hi
Basic question from a newcomer to GRSISort. I downloaded the leangrsi branch of the code and now I am trying to compile my own script "myanalysis2" (based on examples in the GRSISort directories) to produce histograms etc.
I would have attached my code but GitHub doesn't accept any of my files, even .txt files. If anyone is interested I can email the file.
I use the following to compile:
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ root-config --cflags --libs
But I get a few error messages (the 'undefined reference' type). Again I can email the script and a transcript of the errors if required.
Best,
Mike Bowry
—
Reply to this email directly or view it on GitHub.

Or use the grsi-config --cflags --libs --root. Instead of the root config.
On Sep 25, 2015 9:32 PM, "Ryan Dunlop"<EMAIL_ADDRESS>wrote:

The easiest thing to do is put it in the myanalysis directory with a cxx
extension and just type make. All the libraries will be linked and includes
included automatically.

On Sep 25, 2015, at 6:30 PM, MikeBowry<EMAIL_ADDRESS>wrote:
Hi
Basic question from a newcomer to GRSISort. I downloaded the leangrsi
branch of the code and now I am trying to compile my own script
"myanalysis2" (based on examples in the GRSISort directories) to produce
histograms etc.
I would have attached my code but GitHub doesn't accept any of my files,
even .txt files. If anyone is interested I can email the file.
I use the following to compile:
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x
-I$GRSISYS/GRSISort/include/ root-config --cflags --libs
But I get a few error messages (the 'undefined reference' type). Again I
can email the script and a transcript of the errors if required.
Best,
Mike Bowry
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/512#issuecomment-143389283
.

Hi guys. Thanks for answering so quick.
These seem like sensible suggestions, though at the moment neither appear to work. In the version of grsi that I downloaded the myAnalysis directory is empty, so there is no makefile. Should there be? Maybe I'm missing some files.
Also changing to grsi-config seems to generate a longer list of errors.
Cheers
Mike

There is now only one make file in the main directroy, it figures out what
to do from there.
As for the grsiconfig, if you post the errors, we can be a lot more helpful.
On Sep 25, 2015 6:45 PM, "Michael Bowry"<EMAIL_ADDRESS>wrote:

Hi guys. Thanks for answering so quick.
These seem like sensible suggestions, though at the moment neither appear
to work. In the version of grsi that I downloaded the myAnalysis directory
is empty, so there is no makefile. Should there be? Maybe I'm missing some
files.
Also changing to grsi-config seems to generate a longer list of errors.
Cheers
Mike
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/512#issuecomment-143390092
.

You make from the main directory. It is empty as it is reserved for everyone's analysis scripts and got ignores all files in this location.

On Sep 25, 2015, at 6:47 PM, pcbend<EMAIL_ADDRESS>wrote:
There is now only one make file in the main directroy, it figures out what
to do from there.
As for the grsiconfig, if you post the errors, we can be a lot more helpful.
On Sep 25, 2015 6:45 PM, "Michael Bowry"<EMAIL_ADDRESS>wrote:

Hi guys. Thanks for answering so quick.
These seem like sensible suggestions, though at the moment neither appear
to work. In the version of grsi that I downloaded the myAnalysis directory
is empty, so there is no makefile. Should there be? Maybe I'm missing some
files.
Also changing to grsi-config seems to generate a longer list of errors.
Cheers
Mike
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/512#issuecomment-143390092
.

—
Reply to this email directly or view it on GitHub.

Ah, ok, now I'm with you. Taking 'MakeMatrices' as the example on the grifuser account, this can simply be called in the terminal after the make (MakeMatrices [root file]). Looks like this will work with my script.
Cheers,
Mike

Yep. Make matrices is a bit archaic as far as GRSISort goes, so it might not be the best example. Exanalysis or lean matrices might be the go to examples.

On Sep 25, 2015, at 7:15 PM, Michael Bowry<EMAIL_ADDRESS>wrote:
Ah, ok, now I'm with you. Taking 'MakeMatrices' as the example on the grifuser account, this can simply be called in the terminal after the make (MakeMatrices [root file]). Looks like this will work with my script.
Cheers,
Mike
—
Reply to this email directly or view it on GitHub.

@MikeBowry, what kind of errors did you get when you tried using grassy-config? Last time I tried that I got some errors as well which were resolved by using --all-libs instead of --libs. I haven't checked into this any further, but if you got similar errors (complaining about unresolved symbols), I might look into this.

Hi Vinzenz, I experienced the same thing  - see error report below. The addition of all-libs seems to fix it.
Thanks, Mike
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis

g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ grsi-config --cflags --libs --root
/home/mbowry/packages/GRSISort/GRSISort/libraries/libTGRSIFormat.so: undefined reference to TGRSIOptions::AddInputCalFile(std::basic_string<char, std::char_traits<char>, std::allocator<char> >&)' /home/mbowry/packages/GRSISort/GRSISort/libraries/libTGRSIFormat.so: undefined reference to TGRSIOptions::AddInputMidasFile(std::basic_string<char, std::char_traits, std::allocator >&)'
collect2: ld returned 1 exit status
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ grsi-config --cflags --all-libs --root
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis

Sorry I quickly scanned the email. And missed the part where it's fixed. My make stuff still is true though :p

On Sep 28, 2015, at 9:38 AM, Michael Bowry<EMAIL_ADDRESS>wrote:
Hi Vinzenz, I experienced the same thing - see error report below. The addition of all-libs seems to fix it.
Thanks, Mike
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ grsi-config --cflags --libs --root
/home/mbowry/packages/GRSISort/GRSISort/libraries/libTGRSIFormat.so: undefined reference to TGRSIOptions::AddInputCalFile(std::basic_string<char, std::char_traits, std::allocator >&)'
/home/mbowry/packages/GRSISort/GRSISort/libraries/libTGRSIFormat.so: undefined reference toTGRSIOptions::AddInputMidasFile(std::basic_string, std::allocator >&)'
collect2: ld returned 1 exit status
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ grsi-config --cflags --all-libs --root
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
—
Reply to this email directly or view it on GitHub.

I also noticed that your path is a little strange. GRSISYS should already point into the GRSISort directory, not one up from it.

On Sep 28, 2015, at 9:38 AM, Michael Bowry<EMAIL_ADDRESS>wrote:
Hi Vinzenz, I experienced the same thing - see error report below. The addition of all-libs seems to fix it.
Thanks, Mike
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ grsi-config --cflags --libs --root
/home/mbowry/packages/GRSISort/GRSISort/libraries/libTGRSIFormat.so: undefined reference to TGRSIOptions::AddInputCalFile(std::basic_string<char, std::char_traits, std::allocator >&)'
/home/mbowry/packages/GRSISort/GRSISort/libraries/libTGRSIFormat.so: undefined reference toTGRSIOptions::AddInputMidasFile(std::basic_string, std::allocator >&)'
collect2: ld returned 1 exit status
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ grsi-config --cflags --all-libs --root
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
—
Reply to this email directly or view it on GitHub.

Is GRSISort completely compiled? Will the script compile with make? If so it just means there is a missing library not being linked with your grsi config command. I still am suggesting using make because it uses all of the compiler flags we have found to be useful. It also automatically copies your executable into bin which should be part of your PATH.

On Sep 28, 2015, at 9:38 AM, Michael Bowry<EMAIL_ADDRESS>wrote:
Hi Vinzenz, I experienced the same thing - see error report below. The addition of all-libs seems to fix it.
Thanks, Mike
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ grsi-config --cflags --libs --root
/home/mbowry/packages/GRSISort/GRSISort/libraries/libTGRSIFormat.so: undefined reference to TGRSIOptions::AddInputCalFile(std::basic_string<char, std::char_traits, std::allocator >&)'
/home/mbowry/packages/GRSISort/GRSISort/libraries/libTGRSIFormat.so: undefined reference toTGRSIOptions::AddInputMidasFile(std::basic_string, std::allocator >&)'
collect2: ld returned 1 exit status
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ grsi-config --cflags --all-libs --root
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
—
Reply to this email directly or view it on GitHub.

My internet is terrible here so I'm sorry about all of the messages and the potentially random order in which they are appearing.

On Sep 28, 2015, at 9:38 AM, Michael Bowry<EMAIL_ADDRESS>wrote:
Hi Vinzenz, I experienced the same thing - see error report below. The addition of all-libs seems to fix it.
Thanks, Mike
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ grsi-config --cflags --libs --root
/home/mbowry/packages/GRSISort/GRSISort/libraries/libTGRSIFormat.so: undefined reference to TGRSIOptions::AddInputCalFile(std::basic_string<char, std::char_traits, std::allocator >&)'
/home/mbowry/packages/GRSISort/GRSISort/libraries/libTGRSIFormat.so: undefined reference toTGRSIOptions::AddInputMidasFile(std::basic_string, std::allocator >&)'
collect2: ld returned 1 exit status
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
g++ myanalysis2.cxx -o myanalysis2 -std=c++0x -I$GRSISYS/GRSISort/include/ grsi-config --cflags --all-libs --root
mbowry@grsmid01:~/packages/GRSISort/GRSISort/myAnalysis
—
Reply to this email directly or view it on GitHub.

Oh yes, I think this happened when I first downloaded grsisort. In the GRSISort directory, I changed the version to leangrsi and it made a second directory. I correct for this in the GRSISYS path though so it shouldn't be a problem.

No worries. We're just having the conversation in reverse :-)

Ok, the easiest way to select a version is by sourcing the thisgrsi.sh script in the respective directory. This is what root does, so we tried to mimick it. Also switching versions is as easy as checking the branch out (git)

On Sep 28, 2015, at 10:28 AM, Michael Bowry<EMAIL_ADDRESS>wrote:
Oh yes, I think this happened when I first downloaded grsisort. In the GRSISort directory, I changed the version to leangrsi and it made a second directory. I correct for this in the GRSISYS path though so it shouldn't be a problem.
—
Reply to this email directly or view it on GitHub.

Ok, thanks. Things are working at least, even with the extra directory.

With the way you have things set up, if you type make or make clean from
the base directory (the first GRSISort directory) very strange things will
happen.  I do not recommend try it.  If you have already done that, I would
consider removing it (all GRSISort directories) reclining and re compiling.
On Sep 28, 2015 8:49 PM, "Michael Bowry"<EMAIL_ADDRESS>wrote:

Ok, thanks. Things are working at least, even with the extra directory.
—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/512#issuecomment-143912897
.


feat(UI): Add config to preferDocumentPictureInPicture

Closes https://github.com/shaka-project/shaka-player/issues/5689

LGTM

@beaufortfrancois can you review the focus issues reported on https://github.com/shaka-project/shaka-player/issues/5689#issuecomment-1737748455 ? Thanks!


Binding error with ref attributes within child html templates

Greetings,
When navigating to a route that contains nested html element with a ref attribute, and then navigating to another route, console logs
Uncaught (in promise) TypeError: Cannot read property 'parent' of null(…)

Here is the repo and the "culprit" line
ref-attribute-issue
Issue occured after the latest updates.

hey @Markusxmr- thanks for letting us know- what's the stack trace for that error?

On the repo I linked console logs
aurelia-binding.js:1265 Uncaught (in promise) TypeError: Cannot read property 'parent' of null
    at AccessScope.evaluate (http://localhost:9000/jspm_packages/npm/aurelia-binding@1.0.0-beta.1.2.2/aurelia-binding.js:1265:21)
    at AccessMember.assign (http://localhost:9000/jspm_packages/npm/aurelia-binding@1.0.0-beta.1.2.2/aurelia-binding.js:1306:34)
    at NameBinder.unbind (http://localhost:9000/jspm_packages/npm/aurelia-binding@1.0.0-beta.1.2.2/aurelia-binding.js:4965:29)
    at View.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:1045:23)
    at Controller.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:2961:21)
    at View.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:1050:26)
    at Controller.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:2961:21)
    at View.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:1050:26)
    at ViewSlot.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:1349:24)
    at View.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:1055:23)

and in the app where I encounterd the issue
aurelia-binding.js:1265 Uncaught (in promise) TypeError: Cannot read property 'parent' of null
    at AccessScope.evaluate (http://localhost:9000/jspm_packages/npm/aurelia-binding@1.0.0-beta.1.2.2/aurelia-binding.js:1265:21)
    at AccessMember.assign (http://localhost:9000/jspm_packages/npm/aurelia-binding@1.0.0-beta.1.2.2/aurelia-binding.js:1306:34)
    at NameBinder.unbind (http://localhost:9000/jspm_packages/npm/aurelia-binding@1.0.0-beta.1.2.2/aurelia-binding.js:4965:29)
    at View.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:1045:23)
    at Controller.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:2961:21)
    at View.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:1050:26)
    at Controller.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:2961:21)
    at View.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:1050:26)
    at If.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating-resources@1.0.0-beta.1.1.3/if.js:68:17)
    at Controller.unbind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.1.4/aurelia-templating.js:2965:26)

Try running jspm update and see if that fixes it for you. We recently had a patch release covering a bug in this exact method.

jspm update to latest aurelia modules, did not fix the issue. It took me some time track it, and by refactoring the code, bypassed the issue. But original issue does remain

Can you please create a repro on the skeleton of us? If you can attach the
src folder to this ticket we can probably get it fixed.
On Mar 6, 2016 8:05 AM, "Marko Rendulić"<EMAIL_ADDRESS>wrote:

jspm update to latest aurelia modules, did not fix the issue. It took me
some time track it, and by refactoring the code, bypassed the issue. But
original issue does remain
—
Reply to this email directly or view it on GitHub
https://github.com/aurelia/binding/issues/335#issuecomment-192885495.

I did, in my first post. Gues "ref-attribute-issue" is not most intuitive name :smile:
skeleton-repo

Oops. Sorry :) We will take a look shortly.
On Mar 6, 2016 8:38 AM, "Marko Rendulić"<EMAIL_ADDRESS>wrote:

I did, in my first post. Gues "ref-attribute-issue" is not most intuitive
name [image: :smile:]
skeleton-repo
https://github.com/Markusxmr/Binding-behaviour/blob/master/src/modules/users/components/sub-element.html#L2
—
Reply to this email directly or view it on GitHub
https://github.com/aurelia/binding/issues/335#issuecomment-192890146.

No prob. Eh, sundays :smile:

Simple repro based on @Markusxmr's example:  https://gist.run/?id=421475231db224f6d18cdf4c2b16e843
The ref is being put on a bindable property.  Not sure if this is a supported scenario.

Closing this- not a supported use-case for ref binding.

Ok.


Add strategy support for STFX vesting tokens

Adds an addition strategy to strategies which supports STFX vesting tokens.
Changes proposed in this pull request:

Adds an addition strategy to strategies which supports STFX vesting tokens.

Closing - The strategy does not pass all unit tests.


Allow not specifying a package name

Converts the repo into a go module, makes it so that a package name isn't explicitly required. When no package is available it uses the file name as a package name. Have built and ran this locally and it appears to work for schema generation.

Apologies, I meant to raise this PR on a fork. If you would like this functionality please feel free to reopen


luci-app-dawn: Rewrite of Dawn Hearing Map in Table Form

Doing a rewrite of the luci dawn hearing map in fancy table form.

Thanks to @PolynomialDivision for the work

Looks awesome! Thank you.
Please merge, since I do not have merge permissions.

@JonnyTischbein
Has this been tested with all LuCI standard themes?
Why do we still use deprecated table-syntax with hardcoded styles?
I would suggest to better use the common div-structure with a separate css file, e.g.

<%:...%>

<%:...%>

<%:...%>

[...]

Originally I wanted to use something like bootstrap grids but I didn't know that there are classes like that.
I'll to replace it

Works good so far.
@dibdot Do you know if there is an equivalent to rowspan ? I didn't found something like that in the css file.
Is there documentation for the div table ?

Current changes with div tables
I don't know which other themes I should test. Where can I choose them ?

@JonnyTischbein that looks much better - many thanks!
The theme selection is placed in the system menu, at least you should test Bootstrap (most probably already done), OpenWrt & OpenWrt2020.
Thanks again for your efforts!

Thanks! In my build only bootstrap is availalble and was used for develpoping.

Just install the other themes with opkg (luci-theme-*)

Looks awesome! :)

OK then, let's merge it.

Let's see if I can convert the other page to "div-structure with a separate css file", too. ;)


add a sweet homepage

i.e. http://sourcejs.com/

Do you mean use sourcejs or make a homepage like sourcejs.com?

this seems to a rival product of sorts which i randomly came across. I was thinking more about creating an instructions site and demo pages that look much better than we currently have.. taking a steer from sourcejs


Fix Doc

Hello Mark, blz
Thanks for excellent job in package!
I tried to register the components separately and how was documentation.
Vue.use(VueMaterial.MdCore); //Required to boot vue material
Then the components that need, e.g.
Vue.use(VueMaterial.MdToolbar);
But I looked at your code and the property only worked, at least in my project, with property starting with a lowercase letter, for example.
Vue.use(VueMaterial.mdCore);
Vue.use(VueMaterial.mdToolbar);
It may be a problem with the name resolution in my project, but it will be what happens to anyone else?
If so, just change the doc, thanks
Or, if you are normal for most, please disregard

Ahhh, in my project I'm using SSR, can possibly be so.
I'm waiting

@ktquez Yeah. You're right. This is really a  small problem in my package.  I'll change this ASAP.
I never tried those components with ssr, but I think that they will work. Can you give me a feedback about it if you try this before me?

@marcosmoura
Using with SSR had some problems but mainly by manipulating the DOM, but fix it using
the jsdom-global, e.g.
require('jsdom-global')(htmlMarkup, { url: 'http://host:port' })

This fixes me.


Update cancun.md

Based on ACDE 162
https://github.com/ethereum/pm/issues/781

Codecov Report
Patch and project coverage have no change.

Comparison is base (a620b0a) 74.01% compared to head (e483185) 74.01%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #763   +/-   ##
=======================================
  Coverage   74.01%   74.01%
=======================================
  Files         570      570
  Lines       31557    31557
=======================================
  Hits        23356    23356
  Misses       8201     8201

Flag
Coverage Δ

unittests
74.01% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


+[TapSenseAds getSDKVersion] returns previous version

I've downloaded your SDK v.3.2.1, but a call to the mentioned method returns '2.3.1'.  Please fix.

Thanks for downloading our SDK!
Tried to reproduce the issue:

Tested with[TapSense Base SDK.zip] (https://s3.amazonaws.com/ts-sdks/ios-pub/tapsense_ios_publisher_sdk_3.2.1_base.zip)
Tested with [TapSense Full SDK.zip] (https://s3.amazonaws.com/ts-sdks/ios-pub/tapsense_ios_publisher_sdk_3.2.1_full.zip)
Tested using CocoaPods integration

In all cases, [TapSenseAds getSDKVersion] returned 3.2.1.
Please try re-downloading the SDK, and feel free to email us at<EMAIL_ADDRESS>if you have any specific questions. Thanks!


parallelize dbpersister event playback hydration

Credit to @ericvolp12 for most of the work here
I couldnt manage to easily rebase the last PR so i grabbed the diff and just applied it on top.
I then made hydrateBatch send out events as it gets them instead of waiting and collecting everything up.

replaces #188

I don't remember the context on this one too much but as long as the event ordering looks good I don't have any concerns here :)
You should be able to attribute me via Co-authoring the commit if you want -  https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors#creating-co-authored-commits-on-the-command-line


404 Page

Checked with client and design team and both approved :)

I think this needs to be hooked up, doing something like this leads you here instead of the custom 404:

Oh I think it only does that in develop mode since the build mode/netlify preview works. I looked into the docs  and it says that when you use develop "Gatsby uses a default 404 page that overrides your custom 404 page. However, you can still preview your 404 page by clicking “Preview custom 404 page” to verify that it’s working as expected"
Tried looking up overriding the development 404 too but it seems like you can only preview?

Got rid of padding and box shadow passed props for button. Made a GetInvolvedButton component that uses a standardized padding


Plugin Support

Hello again,
There's any plan in the roadmap to create a plugin (or app) system so we can extend the functionality of the CMS?
Maybe you can use the project tab to define next ideas 😄
Cheers.

Yes, after 1.0 stable. I'm using milestones at the moment but will likely use projects for the next phase of dev.


CalloutClickedCommand not working

I was wondering if the CalloutClickedCommand is bindable since the documentation doesnt say its bindable?

CalloutClickedCommand
Is getting executed when the callout of a pin gets tapped. On Android the whole callout is clickable, on iOS there will be a disclosure image visible on the right top.

I am attempting to bind this command but the function never gets executed when i click the callout. I have also attempted to attach the event directly with no success.

Binding the event
map.SetBinding(TKCustomMap.CalloutClickedCommandProperty, "CalloutClickedCommand");

Function when binding
private async Task Callout_Clicked() {     ... }

Directly setting the event
map.CalloutClicked += Callout_Clicked;

Function when attaching event
private void Callout_Clicked(object sender, EventArgs e) {     ... }

Check IsCalloutClickable on your pin.

@TorbenK Thank you this was the issue.

is like a decade after but Im having the same exact issue. I don't have that property IsCalloutClickable, only have access to the CalloutClickedCommand is that the same? cause I click the pin and it only raises the title,not the event on the command.


[Blazor] Default type for @typeparam

Creating generic components, it would be helpful to allow a default type for @typeparam for components that have optional generic params, or the typeparam is not being using for parameters (therefore no way to infer).
Describe the solution you'd like
Allowing something like this
TestComponent.razor
@typeparam TValue = string

Has Value: @(OptionalParameter != null)

@code {
    [Parameter] public TValue OptionalParameter { get; set; }
}

Usage
@* TValue is int *@
<TestComponent OptionalParameter="5" />

@* TValue is string (instead of compilation error)*@
<TestComponent />

Thanks for contacting us, @stavroskasidis.
This is not something we can support.

May I ask why? Is this a technical reason ? It seems to me that it should't be that hard to implement.
From the top of my head, and sorry if this is completely nonsensical, what about this approach:
This line here
@typeparam TValue = string

would generate an attribute on the generated class, something like
[DefaultTypeParam("TValue", typeof(string))]
public class TestComponent<TValue> : BlazorComponent

And then in the context of a .razor file, the razor compiler would generate an automatic inference based on the attribute.
Is what I am proposing completelly non-feasible?


What data format expect?

I'm developing a iPhone app by Unity.
And I using this asset.
https://github.com/jerrytron/neverending-story
It sends a byte array.
My device(Lolin32) seems to be receive some data.
But it shows up different number.
Do you have any Idea for solution?
Thanks
#include <BLEDevice.h>
#include <BLEUtils.h>
#include <BLEServer.h>
#include <BLE2902.h>

#define SERVICE_UUID        "4fafc201-1fb5-459e-8fcc-c5c9c331914b"
#define CHARACTERISTIC_UUID "beb5483e-36e1-4688-b7f5-ea07361b26a8"

BLECharacteristic *pCharacteristic;

bool deviceConnected = false;

class MyServerCallbacks: public BLEServerCallbacks {
    void onConnect(BLEServer* pServer) {
      deviceConnected = true;
    };

    void onDisconnect(BLEServer* pServer) {
      deviceConnected = false;
    }
};

class MyCallbacks: public BLECharacteristicCallbacks {
    void onWrite(BLECharacteristic *pCharacteristic) {

      char sNum[20] = "55";
      std::string value = pCharacteristic->getValue();

      if (value.length() > 0) {
        Serial.println("*********");
        Serial.print("New value: ");
        for (int i = 0; i < value.length(); i++)
          Serial.print(value[i]);

        Serial.println();
        Serial.println("*********");
      }
    }
};

void BLEInit() {
  BLEDevice::init("wind9");
  BLEServer *pServer = BLEDevice::createServer();
  pServer->setCallbacks(new MyServerCallbacks());
  BLEService *pService = pServer->createService(SERVICE_UUID);
  pCharacteristic = pService->createCharacteristic(
                      CHARACTERISTIC_UUID,
                      BLECharacteristic::PROPERTY_READ   |
                      BLECharacteristic::PROPERTY_WRITE  |
                      BLECharacteristic::PROPERTY_NOTIFY |
                      BLECharacteristic::PROPERTY_INDICATE
                    );

    pCharacteristic->addDescriptor(new BLE2902());

  pCharacteristic->setCallbacks(new MyCallbacks());

  pCharacteristic->setValue("Hello World");
  pService->start();

  pServer->getAdvertising()->start();
}

void BLEUpdate(float windDegree) {
    if (deviceConnected) {
      char sNum[20] = "55";
      dtostrf(windDegree, 5, 2, sNum);
      pCharacteristic->setValue(sNum);
      pCharacteristic->notify();
      //Serial.println(sNum);
    }
    delay(200);
}

Howdy.  I understand you are developing an iPhone app using the Unity framework.  You say ... "it sends a byte array" ... should I assume that it is sending it over BLE?  You say that your ESP32 receives some data.  You then say that it "shows up different number".  Does that mean the length of the data received is not what you expected or that the content of the data is not what you expected?

①I'm sending it over BLE.
②The content of the data is not what I expected.
When I send
Value = (byte)201 or (byte)0x00C9
Result is like this symbol  → �
When I send
Value = (byte)55
Result is  → 7
Here is my sending part of Unity(C#).
`
void SendByte (byte value)
{
byte[] data = new byte[] { value };
	BluetoothLEHardwareInterface.WriteCharacteristic (_connectedID, _serviceUUID, _writeCharacteristicUUID, data, data.Length, true, (characteristicUUID) => {
		BluetoothLEHardwareInterface.Log ("Write Succeeded");
	});
}

`

Howdy ... sorry for the delay ... but I think I see the problem.
The data you are receiving in an array of bytes ... a byte is 8 bits of binary data ... decimal 0 to decimal 255.  When you code:
Serial.print(value[i]);

we need to look at the data type of value and we will find it is an array of bytes that are treated as characters.  If the byte value is decimal 55 then it will be printed as ASCII character '7' (see the following table  http://www.asciitable.com/)
If what you want to do is print the decimal value of the byte, then try the following:
Serial.print((int)value[i]);

This should cast/convert the byte/character value to an integer which is then printed in decimal (by default).

Thanks for the assistance!
It works well perfectly.


Switch to useActiveNamespace

📝 Description

use plugin SDK useActiveNamespace instead of getting this value from the session storage.
refactoring the useLastNamespace including the ns path (removing all the conditions around the code base)

What do you think guys @metalice  @glekner @yaacov @hstastna @pcbailey  @vojtechszocs @avivtur ?

/lgtm

/hold

/unhold


[BUGFIX] Line highlight on hover refactor using ECharts dispatch

TBD - need to fix flicker issue due to limitation described in: https://github.com/apache/echarts/issues/18495
Overview
Early on in the Perses project, we decided to build a custom tooltip with React so that we could have full control. This allows us to only show nearby series in the tooltip, better performance with 1000+ series, and ability to easily format the tooltip differently depending on the number of focused series. Since we went down this path, that also means we need to build custom highlighting behavior since the number of focused series can vary depending on what is set in our yBuffer param in the getNearbySeries function.
This PR improves the implementation by using EChart's dispatch API so we can trigger custom highlight and downplay actions depending on seriesIndex conditions and which series are currently in the currentFocusedData array.
Now hover styles are applied correctly so that only the focused series showing in the tooltip are emphasized!
The blur workaround in #1107 is also no longer needed.
Screen Recordings
Loom demo showing changes and ECharts dispatch API docs
Local example 1
https://user-images.githubusercontent.com/9369625/232504585-8ee1b0fd-41da-4f00-8238-4d4435668c71.mp4
Local example 2
https://user-images.githubusercontent.com/9369625/232507723-7a4c1088-2283-49ae-8cac-7b6fe33c3755.mp4
Checklist

[x] Pull request has a descriptive title and context useful to a reviewer.
[x] Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, IGNORE.
[x] All commits have DCO signoffs.

UI Changes

[x] Changes that impact the UI include screenshots and/or screencasts of the relevant changes.
[x] Code follows the UI guidelines.

It's sort of separate from your change here, but I find our current approach to be kind of confusing in some cases like this one where you get this tooltip flicker of some items that occasionally have peaks near where you're moving the mouse left/right. I wonder if we need to have some chats with UX about the ideal behavior for tooltips/highlights/etc., so we can identify an ideal behavior for tooltip/emphasis/etc. we want to work towards.
https://user-images.githubusercontent.com/396962/232570754-5aba4ec5-2b4d-48e4-abb4-257d8b8ad2f5.mov

Separate from my previous note, the lines seem extra flicker-y with this change. My guess is that the highlight & downplay events you're triggering are still fighting with the highlight/trigger events from the axis pointer.
https://user-images.githubusercontent.com/396962/232577280-c98be6ff-a7a4-4d00-ac19-a06ee27ccaad.mov

Separate from my previous note, the lines seem extra flicker-y with this change. My guess is that the highlight & downplay events you're triggering are still fighting with the highlight/trigger events from the axis pointer.

Using the nightly build, your fix in ECharts is definitely helping with the flicker!
One downside is when we set axisPointer.triggerEmphasis to false, the points that move with your cursor, stop showing. To adjust for this, I decided to bring back symbols when the chart renders, but only when there aren't a ton of datapoints (roughly correlates to when the dashboard time range is 15 minutes or less).
This was something I wanted to do anyways and it makes sense to go out with the highlighting improvements, since having better highlighting makes up for cases when symbols don't render (note: HIDE_DATAPOINTS_LIMIT condition is also important since rendering too many symbols can hurt performance).

you get this tooltip flicker of some items that occasionally have peaks near where you're moving the mouse left/right

I've adjusted variables that control how many series show and added a new SHOW_MORE_NEARBY_SERIES_LIMIT condition that will help with this. Now when there are 5 or less series, it will almost always show all 5, but when there are more series than will fit in the tooltip (w/o pinning and scrolling), the yBuffer area is smaller.
It doesn't completely address your concern, but between that and fixing the line color flicker using triggerEmphasis, it is a big improvement. I am planning to add panel options to override default tooltip behavior in the future. I need to sync with Mohit again, but these options could make sense:

Single Series
Nearby Series (Default)
All Series

Still seeing a bit of flickering when you hover directly over lines. Any idea of that's something that can be worked around easily with some of the series emphasis settings? If not, I don't think it's too bad and can be revisited later.
https://user-images.githubusercontent.com/396962/235540424-ff9e2670-48d3-4e08-ae74-d7da5bb32e58.mov


override popupShare not working

override popupShare not working  because this function not open
Overriding non-open instance method outside of its defining module
can you solved this

Can be open func, I'll update in next version.


Enhancement: Use oauth token in push

I'm a little weak on the specifics (if this is possible). Since hub records your OAuth token, it could prevent entering credentials on for example hub push $REPO $BRANCH.

Step 3 looks relevant in this guide:
https://help.github.com/articles/caching-your-github-password-in-git/

Good idea, but the reason why this isn't feasible is outlined in https://github.com/github/hub/issues/1150#issuecomment-214737629. Sorry!


Line height on Input component is not set correctly

Type: minor
Describe the bug
The Input.lineHeight setting that applies to all input components is not being set on the actual <input> but rather on the wrapping div, which has no effect.
To Reproduce
Steps to reproduce the behavior:

Change Input.lineHeight to something like '24px'
Find a text field (any textfield)
Inspect element and see that the line-height: 24px CSS is being applied to the div and not to the input element.

Expected behavior
The line-height attribute should be applied to the actual <input> element.
Screenshots

If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Mac
Browser: Chrome
Version: 70

Additional context
The reason this is a problem is because with the default font Source Sans Pro, the line-height of the input gets automatically set to 20px. But when I change the font to Roboto, the line-height gets automatically set to 18px. And because this is less than the height of the icon, the input field flickers and changes height when an icon appears.

This looks to be happening on the TextInput component but not on the Phone Number component. Not sure about others.

This should be easy to fix, could you elaborate if any other components have the same issue?


Constrain boto3 to our specific package

Closes #22

@perrygeo this is ready for final review and merge.


[zstd] pass level param through to compress/zstd encoder

I'm also open to explicitly enumerating the encoder categories as was done with gzip, but decided to do the sync.Map approach instead for simplicity in case the number range changes meaning down the road and we need more or fewer numbers, etc.
https://pkg.go.dev/sync#Map says it's okay to use it as it fits (1) the entry for a given key is only ever written once but read many times, as in caches that only grow

closing/reopening to possibly get CLA status to refresh?

@lizthegrey thanks for these changes, I'm happy to approve as I think we should be exposing the level
FWIW we previously had a PR from someone (https://github.com/Shopify/sarama/pull/1869) suggesting that sync.Once locking was somehow expensive and that's why the setup had been moved to func init(), but we probably should have pressed for more benchmark evidence at that time because it seems like it would only ever have effected startup time and wasn't worth optimising for.
Aside, whilst it doesn't strictly need it at the moment (because we're not exposing any options), could you mirror the use of a sync.Map in the decode case too so that the decoder is only initialised on-demand in a similar way? I know people have been interesting in customising the encoder and decoder so I think this would be a good step toward exposing more options going forward.

Aside, whilst it doesn't strictly need it at the moment (because we're not exposing any options), could you mirror the use of a sync.Map in the decode case too so that the decoder is only initialised on-demand in a similar way? I know people have been interesting in customising the encoder and decoder so I think this would be a good step toward exposing more options going forward.

Do you prefer use of a Pool or of a single returned zstd.Reader? I worry there's no meaningful map key to use, so might the Pool be a better construct to use here?

Aside, whilst it doesn't strictly need it at the moment (because we're not exposing any options), could you mirror the use of a sync.Map in the decode case too so that the decoder is only initialised on-demand in a similar way? I know people have been interesting in customising the encoder and decoder so I think this would be a good step toward exposing more options going forward.

Do you prefer use of a Pool or of a single returned zstd.Reader? I worry there's no meaningful map key to use, so might the Pool be a better construct to use here?

Or we could create a CompressionOptions{} struct that could be used as the map key I suppose. still leaves open why the other mechanisms use a Pool but zstd does not.

@lizthegrey thanks for these changes, I'm happy to approve as I think we should be exposing the level
FWIW we previously had a PR from someone (#1869) suggesting that sync.Once locking was somehow expensive and that's why the setup had been moved to func init(), but we probably should have pressed for more benchmark evidence at that time because it seems like it would only ever have effected startup time and wasn't worth optimising for.
Aside, whilst it doesn't strictly need it at the moment (because we're not exposing any options), could you mirror the use of a sync.Map in the decode case too so that the decoder is only initialised on-demand in a similar way? I know people have been interesting in customising the encoder and decoder so I think this would be a good step toward exposing more options going forward.

Done, there's now both ZstdDecoderOptions and ZstdEncoderOptions and lazy-creation when a decoder or encoder is requested with a new set of parameters.


cycle action editor "insert selected action in action window" broken since 6.47

The cycle action editor is kind of broken since the update to 6.47. Might have to do with the action overhaul. Any action I try to add from the action list to a cycle action is not recognized anymore.
Here I am inserting the infamous no-op action (ID 65535) into a cycle action and it's not recognized. Is it only on my computer ?
[url=https://ibb.co/hH6MFJy][img]https://i.ibb.co/hH6MFJy/juyhtgrfeds.png[/img][/url]
Edit : Ok so you can actually add actions in cycle actions, but you have to enter manually the command ID. Not that bad but new.

Seems it breaks here: https://github.com/reaper-oss/sws/blob/4ca0494874c3b38cac5372bc56505835e46150ac/SnM/SnM_Window.cpp#L501
(lParam no longer containing the command ID since the action list overhaul).
No idea how to fix (yet), I know there's 'new' native API PromptForAction() but it's not the same as it always starts with an all actions unselected action list).

update: https://forum.cockos.com/showpost.php?p=2527570&postcount=12


Incorporate test results into Cucumber Pro

Desired Behaviour
We've moved from publishing the acceptance tests in Relish to Cucumber Pro. This provides the opportunity for registered stakeholders to collaborate on the tests. Another feature being developed in Cucumber Pro is displaying a summary of test results.

Our end-to-end tests (yarn run e2e) show test results locally

Explore if there is value in integrating test results with Cucumber Pro to share test results publicly.
As Cucumber Pro is in Beta, provide feedback on:

what additional test result information we want to share publicly
the version test summary feature
the document test summary feature (although the doc mentions Cucumber-Java, Cucumber-JS support is in development)

What's coming in Cucumber Pro?

Instructions to do this a available on https://app.cucumber.pro/projects/data-curator/settings (secured)


Create a version of handleActions which supports async actions too

We want to be able to support handlers in the form of:
{
    begin: (state, action) => {},
    end: {
      next: (state, action) => {},
      throw: (state, action) => {},
    }
  }

In order to do so, we have to inspect themeta field of the action and switch on the async field there. We also have to make sure that the handlers receive all arguments the original action receives, so that we can do optimistic updates easily.
In case an async handler receives a non-async action it should just handle it the same way it would handle the end status of an async action.

I have a branch https://github.com/acdlite/redux-actions/tree/sequence that solves this pretty well. I'll update the docs soon and get it merged in.

@johanneslumpe @acdlite
Here is my idea
Example here:
const handler = (state, action) => newState

const reducer = createReducer()
  .when([ACTION_FOO, ACTION_BAR], handlerForBothActions) // share handler for multi actions
  .when('BAZ', handler)    // optimistic update here if you need
  .done(handler)               // handle 'BAZ_COMPLETED'
  .failed(errorHandler)      // handle 'BAZ_FAILED'
  .build(initValue);

I don't know why async isn't even mentioned in the docs. Making at least one async call in any real web app seems unavoidable. Am I missing something? Currently looking at this library to handle it.


#unable to run 'search_lib.py' file

C:\Users\HP PC\anaconda3\python.exe" "C:/Users/HP PC/openDiagram/physchem/python/search_lib.py"
[nltk_data] Downloading package stopwords to C:\Users\HP
[nltk_data]     PC\AppData\Roaming\nltk_data...
[nltk_data]   Package stopwords is already up-to-date!
Traceback (most recent call last):
  File "C:/Users/HP PC/openDiagram/physchem/python/search_lib.py", line 11, in <module>
    HOME = os.path.expanduser("~")
NameError: name 'os' is not defined

Process finished with exit code 1

os is a very basic Python library.
Suggest you see if os works in your REPL.
python
Python 3.8.3 (default, Jul  2 2020, 11:26:31)
[Clang 10.0.0 ] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import os
>>>
>>> print( os.path.expanduser("~"))
/Users/pm286


CHIP credential serialization

Problem
We're lacking support to adequately parse elements from CHIP and non-CHIP certificates.

Summary of Changes

Added method to retrieve CHIP certificate value from certificate.

Enhanced OperationalCredentialSet class with Serialize/Deserialize methods. User can now serialize/deserialize a specific Trusted Root ID

Testing

This was manually tested using Android controller (chip-tool) and x86/arm64 linux device (lighting-app)
Unit tests have not been changed, existing unit tests for CHIP certs are used.

@jpk233 , maybe we can just use the DER formatted cert, and store it as is. If the KVS doesn't support storage of binary data, we can convert it to base64 string and store that instead.

For extracting Pubkey, how about converting X509 cert to CHIPCert, and get the value from there? CHIPCert is TLV formatted, and you can use kTag_EllipticCurvePublicKey to get the pubkey.

@jpk233 , maybe we can just use the DER formatted cert, and store it as is. If the KVS doesn't support storage of binary data, we can convert it to base64 string and store that instead.

It's being serialized as base64 currently

For extracting Pubkey, how about converting X509 cert to CHIPCert, and get the value from there? CHIPCert is TLV formatted, and you can use kTag_EllipticCurvePublicKey to get the pubkey.

This change is meant to be applicable to both CHIP and non-CHIP certs, which is why it's not converted to CHIPCert format.

Per the updated template, can you update the PR here?
#### Problem
What is being fixed?  Examples:
* Fix crash on startup
* Fixes #12345 12345 Frobnozzle is leaky (exactly like that, so GitHub will auto-close the issue).

#### Change overview
What's in this PR

#### Testing
How was this tested? (at least one bullet point required)
    • If unit tests were added, how do they cover this issue?
    • If unit tests existed, how were they fixed/modified to prevent this in future?
    • If integration tests were added, how do they verify this change?
    • If manually tested, what platforms controller and device platforms were manually tested, and how?
    • If no testing is required, why not?

Thanks, updated with testing details.

Oh, and please document the lifetime assumptions involved....

Oh, and please document the lifetime assumptions involved....

Updated CHIPOperationalCredentials.h to document assumptions for the new methods.

/rebase


Question: Is strtotime Algorithmically Equivalent to the PHP Version?

Hello!  I recently discovered this repository while working on an article/tutorial about javascript date libraries. It looks pretty useful.  I have a question about one of the implementations and I wasn't sure where/who to ask -- my apologies if this is an abuse of the issues format.
I'm curious about the relationship between this repository's implementation of strtotime and PHP's native implementation.  Specifically, is the locutus implementation of strtotime algorithmically equivalent to the C code that powers PHP strtotime, or is the locutus implementation an alternate implementation that returns the same results?  Or is the relationship murkier with the C implementation inform some of of what's going on in the javascript code, but the language differences means locutus is effictivly it's own thing?
I started trying to answer this myself and was quickly reminded why it's so rare for folks to regularly program in raw C these days.  I figured/hoped the author would be able to give me a high level answer before I burn cycles deciphering a C based scanner.

Hi. First of all, please consider locutus implementation of strtotime a WIP. It still lacks some features that native strtotime offers (especially timezone support and several complex date/time formats).
To answer your question, locutus implementation is based on PHP implementation, but they're not 100% identical. It's done by trying to understand what's happening in C code and trying to write it in JS-way. This is one reason why the implementation is still incomplete.
As for details, the pattern matching follows same rules, but some regexes and their order differ for various reasons, it will be aligned in near future.
The time computation also follows PHP logic, but uses JS APIs instead of calculating "by hand" wherever possible. This may result in different output timestamps in some corner cases.
To summarize, at the moment you can't count on high compatibility, but we're trying to get as close as possible.
If you have additional questions, please feel free to ask here on contact me directly.

Thank you @kukawski, that's a perfect answer to my question.  I'll say hello if I have more to ask or run into any edge cases.  Thanks for contributing this code! (closing)


Anchore engine kills itself in docker or with docker-compose

Is this a request for help?: YES

Is this a BUG REPORT or a FEATURE REQUEST? (choose one): BUG

Version of Anchore Engine and Anchore CLI if applicable:
anchore-cli, version 0.2.3
Name: anchore-engine
Version: 0.2.3
What happened:
When starting Anchore engine, either with docker start or docker-compose up, it kills itself randomly (but far more often than not killing itself). Key logs are:
anchore-engine_1  | [anchore-catalog] [anchore_manager.cli.service/terminate_service()] [INFO] Found old pid for service: anchore-catalog. Terminating it
anchore-engine_1  | [anchore-catalog] [anchore_manager.cli.service/terminate_service()] [INFO] killing old twistd pid: 18
anchore-engine_1  | /usr/bin/anchore-engine: line 11:     7 Killed                  /usr/bin/anchore-manager --configdir ${CONFIGDIR} service start $REST
anchore_anchore-engine_1 exited with code 137

What did you expect to happen:
Just start without failing. I don't see the check of PID in a container world, it does not make sense to me (am I missing something though?).
As a workaround, in the container, any running PID should just be ignored, since it's probably leftovers from the last execution.
Any relevant log output from /var/log/anchore:
...
[MainThread] [anchore_manager.cli.utils/connect_database()] [INFO] DB connection configured: True
[MainThread] [anchore_manager.cli.utils/connect_database()] [INFO] DB attempting to connect...
[MainThread] [anchore_manager.cli.utils/connect_database()] [INFO] DB connected: True
[MainThread] [anchore_manager.cli.utils/init_database()] [INFO] DB compatibility check: running...
[MainThread] [anchore_manager.cli.utils/init_database()] [INFO] DB compatibility check success
[MainThread] [anchore_manager.cli.utils/init_database()] [INFO] DB post actions: running...
[MainThread] [anchore_manager.cli.service/start()] [INFO] DB version and code version in sync.
[MainThread] [anchore_manager.cli.service/start()] [INFO] Starting services: ['anchore-catalog', 'anchore-simplequeue', 'anchore-api', 'anchore-kubernetes-webhook', 'anchore-worker', 'anchore-policy-engine']
[anchore-catalog] [anchore_manager.cli.service/startup_service()] [INFO] cleaning up service: anchore-catalog
[anchore-catalog] [anchore_manager.cli.service/terminate_service()] [INFO] Found old pid for service: anchore-catalog. Terminating it
[anchore-catalog] [anchore_manager.cli.service/terminate_service()] [INFO] killing old twistd pid: 18
/usr/bin/anchore-engine: line 11:     7 Killed                  /usr/bin/anchore-manager --configdir ${CONFIGDIR} service start $REST

What docker images are you using:
anchore/anchore-engine                                     latest              7f60d4a6fdb7        3 weeks ago
How to reproduce the issue:
Run anchore engine with the docker-compose given in GitHub, wait until the vulnerabilities are fetched, and try stopping/starting the anchore-ending container.
I could not find any context that could change the behavior. I had the issue after a system reboot between stop and start, and I also had no issue between 2 fast restarts... However, it seems that time has globally a positive impact on the next try.
Anything else we need to know:

Hi @Karreg - thank you for the detailed report!  I looked at the logic, and I do see a way for this condition to arise (when new anchore service processes are getting the same PIDs as previous, when the previous   service PIDs are shut down with a hard kill (-9)) - I'll investigate further and will report status in this ticket, I believe the solution will be to improve the anchore manager service restart logic to be more specific about determining if an existing service is still running.

Thanks for the fix. I'll be glad to test it as soon as a new image is available :)

Is there any workaround until the new image is available? It's kinda blocking me to advance on the other issue ;)

hi @Karreg - I can think of two workarounds currently if this condition is cropping up a lot:

use the process

docker-compose down
docker-compose up -d

instead of 'stop, start', which will result in a clean pidfile state each time.

enable 'auto_restart_services: True' in your config.yaml - this should instruct the anchore_manager to automatically restart the service processes if they unexpectedly fail, and should result in all the services up and running.

If you're in a testing environment only, the final workaround could be to try the dev container, which is auto-built from the latest code in master (which has the fix!), but caveat there is that we cannot guarantee upgrades/stability for dev containers, generally.  That container is always available at anchore/anchore-engine:dev on dockerhub.
Thanks!
-dan

OK, thanks for the workaround!


Fix Rollbar::Notifier#report_custom_data_error

Ensure that we receive a Hash object so we can take the uuid from
there. In any other case just return an empty Hash.
[branch #5660]

:+1:


Undefined array key "transient"

I constantly get this error from Whisper API:

Does anyone know how to fix it?

Are you using the latest version of our package?

Currently I have v0.4.2. Will upgrade and check again. Thank you)

@nunomaduro I got this error on 0.5.2, upgraded to 0.5.3 but I am still getting it.

Upgrading to 0.6.3 solved issue for me

Ah, openai-php/laravel didn't want to upgrade past  v0.5.3. Not sure how tags vs releases work, but maybe the Laravel wrapper requires some releases @nunomaduro

openai-php/laravel also has v0.6.3 version which should require 0.6.3 of this library. You can require/upgrade them both with:
composer require -W "openai-php/client:^v0.6.3" "openai-php/laravel:^v0.6.3"

You guys need to simply use the latest version of both the API Client and Laravel integration: v0.6.


Please include examples of the bad old ways and what to do about them instead, generally.

I wish this information about the client side vs server side evaluation was somehow louder, because I got the exception for the first time without know why it happened, and finding this documentation was difficult. Stackoverflow doesn't have a good history of handling this issue, since it's relatively new.
I wish there were more examples of what used to be "acceptable" but poor performance, and what is now totally blocked - as well as examples of what to do instead of the now totally blocked processes.
I have a particular issue that uses the .Where() clause with an .Include(), and I can't find examples of what to do instead. I'm also sure that there are plenty of other places that an example would go a long way to helping people figure out what to do instead of the bad old ways :-)
Thanks!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: fd35b912-445c-e801-7f70-de73839db80d
Version Independent ID: e558a081-5397-1e3f-1d58-abea3f72cad6
Content: Client vs. Server Evaluation - EF Core
Content Source: entity-framework/core/querying/client-eval.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @smitpatel
Microsoft Alias: avickers

@rhires prior to 3.0, EF Core used to automatically switch to client evaluation whenever something wasn't translatable to the server (SQL); all these cases are now blocked (except for top-level projections). Since this encompasses a very wide range of scenarios, it simply isn't practical (or particularly useful, I think) to document comprehensively. Assuming a user is aware of the change to disallow (most) client evaluation - which is mentioned in the release notes - it seems to me that the exception is explicit enough in saying the expression could not be translated, and additional docs wouldn't necessarily help.
Now, whether client evaluation is "acceptable but poor performance" is entirely dependent on your specific scenario and requirements. For example, a Where clause that cannot be translated previously caused the entire table to be pulled back for client-side evaluation; if the table happens to be small, that's fine, but if it's huge, this is likely to be a real perf-killer. So once again, I'm not sure what we could add in the documentation that would be helpful.
I'm also not sure what you're referring to with using Where and Include - AFAIK that should work just fine in most scenarios.
To summarize, it would good to understand very concretely what you're proposing we add to the docs etc.

Additionally, when I've looked for examples of what to do with a query like
```

var aspNetUsers = await _context
.Users
.Where(m => m.Id = userId)
.Include(u => u.UserRoles)
.Select(m => new AppUserViewModel
{
StaffId = m.StaffId,
Id = id
});
They simply take out the Where() clause, do the projection, then do the filter...which happens client side. Everyone says that this can cause performance problems when datasets are large, but no one seems to have a solution for how to do server side filtering, especially for large datasets, to avoid the performance issue. I know it can't be done in every case, but is there a solution for this?

They simply take out the Where() clause, do the projection, then do the filter...which happens client side. Everyone says that this can cause performance problems when datasets are large, but no one seems to have a solution for how to do server side filtering, especially for large datasets, to avoid the performance issue. I know it can't be done in every case, but is there a solution for this?

@rhires can you point to where the above query is said to be problematic? Everything in the above query should get translated to SQL and execute efficiently at the server, with only the StaffId column getting sent to the client for rows with the matching Id. Have you executed the query and seen something else in the generated SQL?
It's also not clear why you're including UserRoles, as it isn't used later.

Ok - while I wasn't trying to get support for this query, and it was an example of a "bad" query that would generate an exception, I re-ran it the way I thought it was giving me errors, and it no longer is. So I'm quite puzzled by that. I also just posted an abbreviated example. I was getting UserRoles to associate them with the users in my admin interface on the UI side, it's just not shown in my example. I'll remember to post a more complete example next time. Sorry about that.
But my main request... It would be nice to have examples of the previously allowed but now disallowed situations, and what the "replacement" query might be. The text in the doc is well written, and explains the rationale pretty well about why the change, but it doesn't say what to do instead now that the change is in place.
I realize that not all situations can be documented, but one of the first things would be to have a specific page for the error "The LINQ expression could not be translated. Either rewrite the query in a form that can be translated, or switch to client evaluation"...and then some explanations of what might be wrong with the query. It might also be useful to change the message to be closer to what the error really is: some kind of evaluation that EF is doing not at the top level, but a level or two below the top level which it can't send down to the server.
Here's also an different example from stackoverflow to maybe better illustrate what I'm talking about. The exception has the user scratching their head at what that even means, and then the person answering taking a guess about how to fix the issue.
So there must be a small list of these kind of "second tier" evaluations/projections that are common that would generate an exception like the one on SO as shown above, that would go on a page similar to the page by a third party that better explains what the error means, and what could generate it.

@rhires Are you asking how to use AsEnumerable to cause client evaluation? Or are you asking how to rewrite queries so they don't need client evaluation?

No, not asking how to do that explicitly. I'm looking more for the "rewrite queries so they don't need client evaluation", but in the form of examples - examples of what used to work pre-.net 3.1 (3.0?), and now won't work because of the new logic. And then how they would now look with the new logic in place.
I suspect that there are common patterns that devs use that will throw the exception, that could be documented as the "old way", and then shown with the "new way".
So we could have a page with the name of the exception in the title, explaining what "cannot be translated" means - that LINQ cannot convert the LINQ as written into SQL and passed down to the server for evaluation. Then we have an example (as on the third party page linked above), to show how throwing a method into the middle of a LINQ statement causes a problem, since it can't be converted to SQL (but its result can, so make it a stand-alone statement).
Then we could have another example of something that used to be allowed in say, 2.1, but isn't any longer. And then a few more after that. Two columns: "old way" | "new way".
Similarly, but not the same, there are a few pages in the docs somewhere that show quite a few LINQ simple examples for the different operators, but these examples don't have an old vs new.
I'm sorry I'm having such a hard time communicating this out to you. I appreciate you trying to understand what I'm talking about.

@rhires I think I understand what you're asking. However, given that it would be quite a big effort to write this, what do you see are the exact advantages of this, given that users can simply run their queries and see them fail? Our "could not translate" exception messages are probably not perfect, but they do point to the specific part of the query which cannot be translated. I agree this doesn't tell users how to fix the issue, but from our experience, users generally figure it out (e.g. they execute some local .NET function inside a Where clause, and understand that this cannot be translated).
So I'm not saying we're perfect here, but overall I'd rather we invested in tailoring exception translation failure messages rather than attempting to cover everything via docs.

I think that’s probably the better option: updating the exception translation message to be a little less cryptic, such as you mention below. Basically, I couldn’t immediately figure out what the exception message meant, since I don’t think of LINQ as something that gets “translated”.
Also, I don’t believe I saw which specific part of the query gets pointed to that is causing the issue in the exception or the stack trace. I probably just missed it, because I didn’t understand the “headline" of the exception.
Thank you - I guess that’s all I have. Cheers!

On Nov 17, 2021, at 6:09 PM, Shay Rojansky @.***> wrote:
@rhires https://github.com/rhires I think I understand what you're asking. However, given that it would be quite a big effort to write this, what do you see are the exact advantages of this, given that users can simply run their queries and see them fail? Our "could not translate" exception messages are probably not perfect, but they do point to the specific part of the query which cannot be translated. I agree this doesn't tell users how to fix the issue, but from our experience, users generally figure it out (e.g. they execute some local .NET function inside a Where clause, and understand that this cannot be translated).
So I'm not saying we're perfect here, but overall I'd rather we invested in tailoring exception translation failure messages rather than attempting to cover everything via docs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dotnet/EntityFramework.Docs/issues/3566#issuecomment-972243279, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC76XZMI7IIJCAFD7YNA4GDUMQY35ANCNFSM5IEQ5WDQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

since I don’t think of LINQ as something that gets “translated”.

Yeah, that's basically the main role of EF Core when it comes to queries - translate LINQ queries to SQL.
In any case, thanks for engaging and don't hesitate to reach out with other issues.


Generalize Sudoku to size N

The first implementation of the Sudoku was a 2x2 board just to make it simpler to implement and be able to implement WFC quickly. However, 2x2 is way to simple and people wouldn't really want to play that game.
The prefabs for the "board" and the "board cells" are now statically defined for the 2x2 case, but we want the board and the cells to be dynamically generated based on the input parameter N, so:

Each cell will have $N * N$ possible numbers (e.g. N=3, classic Sudoku, you'd have 9 options to select from)
The board wil have... $(N * N)^2$ cells? For N=3, that'd be 81 cells (imagine for greater values of N! 😮)

This would probably be a bit crazy for values of N greater than 3, but at least generalizing what was already implemented would let us reuse most of the functionality implemented for 2x2

Actually, the generalization could have independent number of rows and columns e.g. 2 x 3 (6 numbers and "boxes" would be rectangles)


Fix Formatting in Displaying Lists in API Docs

Description:
There is a formatting issue as shown below in displaying the list in [1], which is displayed properly in [2].

[1] https://ballerina.io/learn/api-docs/ballerina/io/index.html
[2] https://ballerina.io/learn/api-docs/ballerina/io/functions.html#sprintf
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Need to verify the fix.

Closing since this has been verified when sending the PR.

Reopening as this is not fixed yet. Now, the list is not rendered at all in the index page[1] as shown below.

[1] https://ballerina.io/learn/api-docs/ballerina/io/index.html

Reopening as this is not fixed yet. Now, the list is not rendered at all in the index page[1] as shown below.

[1] https://ballerina.io/learn/api-docs/ballerina/io/index.html

This is due to a new logic introduced in 1.2.2 release to limit the descriptions in tables to first sentence only. This was introduced as there were some formatting issues due to descriptions containing code snippets.
However, the above is only applicable for the 1.2.2 branch and is fixed on the master branch.

Closing as per the above comment.


chore: use etcetera instead of directories

Resolves #138
Hi, this project is very cool. I found issue which I may solve so I try to.
About possibility to change of cache dir
In my mac, cache directory was changed like below by this change. I can't test on Windows and Linux, but there is a possibility that cache dir on these OS can changed. Is this acceptable?

Before
After

${HOME}/.cache/update-informer-rs
${HOME}/Library/Caches/update-informer-rs

✔ Checklist:

[x] Commit messages have been written in Conventional Commits format;
[ ] Tests for the changes have been added (for bug fixes / features).

@kyu08 👋
Thank you for your help 🚀


AWS AMPLIFY CANT GET SUBDOMAIN FOR THE APP

Before opening, please confirm:

[X] I have checked the custom domains troubleshooting guide.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
No response
Region
us-east-1
Step
Domain Activation
Status
Failed
Issue or question
I try to setup my domain on godaddy with AWS amplify I have a link to the full reproduction https://www.youtube.com/watch?v=he3T9Jo5wmA
Error message
Error message: [CNAMEAlreadyExistsException] The following CloudFront distributions or Amplify Apps are already using one or more of your domains: [Distribution: *******EBGIRLL in AWS Account: ******231235]. For distributions you own, please remove the domains from them first and then retry adding your domain. For Amplify apps you own, you may already have associated this domain to another Amplify App in another account or region. You can also check our troubleshooting guide for more help: https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html
Additional information
full video reproduction, for some reason ui-preview works flawlessly https://www.youtube.com/watch?v=he3T9Jo5wmA

AWS AMPLIFY CANT GET SUBDOMAIN FOR THE APP

Expected Behavior

the root and subdomain hooks up to AWS amplify site

Current Behavior

I get a Error message: One or more of the CNAMEs you provided are already associated with a different resource. error
[CNAMEAlreadyExistsException] The following CloudFront distributions or Amplify Apps are already using one or more of your domains: [Distribution: *******EBGIRLL in AWS Account: ******231235].
when I have no cloudfront distributions

Possible Solution

manually go into application interanls and see whats going on

Steps to Reproduce
Youtube url
Environment
Angular CLI: 16.1.4
Node: 18.16.1
Package Manager: npm 9.5.1
OS: win32 x64

Hi @windmillcode0 👋 , thanks for raising this with us and for providing detailed information on the steps you performed to troubleshoot/debug this issue.
The CNAMEAlreadyExistsException occurs because the subdomain www.findmyrole.co is currently associated to a CloudFront distribution ending in *******EBGIRLL in AWS Account: ******231235.
Can you confirm if you are the owner of the AWS Account ending in ******231235 ? If yes, you can disassociate the subdomain from the CloudFront distribution ending in *******EBGIRLL from within the CloudFront console on your end. Detailed steps are outlined here.
If you are not the owner of the AWS Account ending in ******231235, kindly reach out to me via a DM on our community discord server. My username is Jay Raval and I can advise on next steps.

Alright i just got the user name and pass I will take a look


Add link to workshop video

Later it would be nice to have the video uploaded to youtube, but for the time being, I think it makes sense for people to have this link handy.

+1
Please merge, this was helpful and it isn't immediately obvious to look to PRs for a link 😇

agreed. figma slides aren't quite enough to intuit the solutions from 😅


[FEATURE] Created the features section in the Website

Fixes issue

Fixes #281
Changes proposed

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

I uploaded a folder called features-images and there uploaded flat icon images. If something needs to be changed feel free to tell me, and I will change it as it needs to be. Also about the background color, I didn't ask about that and the color is bg-var. So if it needs to be changed also tell me, it's not a problem. This is my first collaboration so I know that everything can not be perfect, but I would want to make it such.

@anevski-stefan It would be better if you do not commit the package-lock.json file.


Update wartremover to 2.4.1

Updates org.wartremover:wartremover from 2.3.7 to 2.4.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "org.wartremover", artifactId = "wartremover" }]

Codecov Report

Merging #103 into series/0.6.x will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           series/0.6.x     #103   +/-   ##
=============================================
  Coverage         61.18%   61.18%
=============================================
  Files                11       11
  Lines               304      304
  Branches             21       21
=============================================
  Hits                186      186
  Misses              118      118

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 3b06a54...d628fc0. Read the comment docs.


PSP migration guide.

Adds a how-to guide explaining how users can migrate their Kubernetes PSPs to Kuberwarden policies.
Fix #88

@jvanz , @divya-mohan0209 : I think we can merge this PR now, as the last thing before we migrate to docusaurus


Create Tessellation and switch some callers to it

I adapted the idea in your branch and started this transition. This nearly rids us of polygons that don't have valid outer/inner rings.
I think the next steps are now:

Look at the last two Polygon::precomputed callers -- circles and polylines. They send in valid rings. There might be a perf impact to using earcutr. So maybe geom::Polygon optionally stores some indices sometimes. The approach in your branch was to change the API to put out a Tessellation instead of a Polygon, but this will affect lots of code, and I'm not sure it's what we want. A Polygon is "stronger" than a Tessellation.

Be more careful about converting geo::Polygon to geom::Polygon. The current From implementation can crash -- and does, in a few maps that intersect OSM areas with the boundary polygon in a particular way. I want to more carefully go through all the polygons coming from a georust operation.

Change the internal implementation of Polygon. Probably to start, ditch points and indices and always have the Rings. It'd be neat to just wrap geo::Polygon, but that'll mess with serialization since we trim the precision of Pt2Ds.

Thanks for the review!


Publish a release?

💬 Questions and Help
Hi maintainers; last month my PR got merged, and some other helpful PRs have been merged since the last release. Could you publish a new release containing these PRs? Thank you!

Would be great...! Anything for which you need support, to ship a release?

I have just done this, sorry about the delay


Question about version specific dependency handling

Let's say I do:
    if sys.version_info[0:2] < (2, 7):
        project.depends_on("ordereddict")

Does anyone know if this will propagate to the generated setup.py? My guess would be no, but I can't say for sure...

Well yes and no. The dependency won't be added if you build it on 2.6. But then the install target doesn't matter.
In short it's used at build time, not at install time.

hmmm, I am just wondering about this because I may need to make an OS-Specific dependancy. This should obviously propagate to the setup.py that will then end up on PyPi. It's not clear yet, if I really do need to make that right now, but I am sure it is just a matter of time until me or someone else hits this snag.

The pre/postinstall scrips for the setup.py give you a lot of control. But there is no good primitive for dealing with dependencies there. So we might end up having to support some kind of lazy precondition on dependencies that is added to the setup.py

Yeah, this looks like a bigger feature which probably requires some design work before hitting the keyboard.

I am no longer interesting in contributing this feature.

I am no longer interesting in contributing this feature.


How to do lazy loading of records? +300000 records

TL;TR
What's the approach to use this plugin for tables with +300000 records.
I'm trying to implement a report using this plugin, but the problem I'm facing, is that seems like try to retrieve all records in once.
My "idea" is to only load records of each page and avoid to request all records because scripts crashed because out of memory.
This is what I use on my controller. I'm using Query builder and skipPaging as suggested on an older issue). Just for try, I tested doing some filtering too, and then works, but without filtering don't.
return datatables(DB::table('mytable'))->skipPaging()->toJson();
This is my JS side:
var table = $('#powersearch-table').DataTable({
           processing: true,
           serverSide: true,
           pageLength: 50,
           deferRender: true,
           destroy: true,
           ajax: {
               url: "/powersearchajx",
               dataSrc: "data",
               type: "GET"
           },
           buttons: [
              'pageLength', 'csv', 'excel', 'pdf', 'print'
          ],
           columns: [
                mycolumns
           ],
       });

     $('#powersearch-table thead tr').clone(true).appendTo( '#powersearch-table thead' );
     $('#powersearch-table thead tr:eq(1) th').each( function (i) {
         var title = $(this).text();
         $(this).html( '<input type="text" placeholder="Search '+title+'" />' );

         $( 'input', this ).on( 'keyup change', function () {
             if ( table.column(i).search() !== this.value ) {
                 table
                     .column(i)
                     .search( this.value )
                     .draw();
             }
         } );
     } );

Hello devgiu,

I have passed the additional parameters using the with function from controller like below snippet-
->with([ "cities" => app(\App\Http\Controllers\HelperController::class)->cityListing(), "leadCounters" => $this->leadCounters ]) ->make(true);
Now, how can we extract these keys values in the view side?
Here, in the attachment. I am getting the response.

@devgiu don't skip the paging unless you want to implement your own paging. Just remove ->skipPaging().


statespace constructor fix

...other changes related, the deleted setup.py wasn't used anywhere as far as I could see.

The intent of allowing StateSpace(sys) was as a copy constructor, as described in the function documentation.  This change would remove that functionality (also present in several other classes).

I can add a .copy() function if that would help? If you want I can change the other classes too, to reflect this change.

It seems to me that we are fixing a non-problem.  I suggest we drop this pull request and leave the current functionality.

You don't see a problem with help(control.statesp.StateSpace) showing the user (self,*args) as valid arguments for __init__ and the docs saying the the way to create a statespace is to call matlab.ss(*args) and not to use the StateSpace contructor?

I am not a core dev with python-control but use it often. My rule of thumb is to not break backwards compatibility unless absolutely necessary. The amount of code you may break and potential amount of frustrated users is almost always not worth the change.
How about improving the documentation and examples of use for the class instead of changing the functionality? Then it is is a win win for everyone.

Agree with @moorepants that a good approach is to update the documentation.  I'll try to take a cut at that in the coming days and will close out the PR if issue is fully addressed.

I have created a new PR (#171) that fixes the documentation for the class constructors.


Added License files and comments

Added License Headers

PR Rejected. Kindly use MIT license for all codes across all SRA repositories and raise PR again

PR Rejected. Kindly use MIT license for all codes across all SRA repositories and raise PR again


Bug report

Describe the bug
Run docker/run_docker.sh -r -d /data/disk returns error shown in the ppic

Environment
Docker Windows 10(Home Edition), Docker 19.03.12

Maybe you can try to config git :

Not working.. Still getting the same error. I'm using Git Bash btw

@hooklee2000  It is recommended to use centos7

The run_docker.sh script have not been tested in windows platform.


Create folders recursively on rocksdb_open

There's some problems with the libpath dependency, I had to patch those changes to be able to bypass the compiler errors reproduced on CI:


400: Invalid label name - invalidArgument

The issue tracker is for reporting product deficiencies. How do I questions should be posted to the discussion forum at https://groups.google.com/group/got-your-back. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GYB release from https://github.com/jay0lee/got-your-back/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/got-your-back/wiki

Full steps to reproduce the issue:
1.
2.
3.
Expected outcome (what are you trying to do?):
Actual outcome (what errors or bad behavior do you see instead?):

400: Invalid label name - invalidArgument
How do I fix this...

400: Invalid label name - invalidArgument
Command:
gyb --email<EMAIL_ADDRESS>--action restore --local-folder "c:\gyb\LastNameFirstIntial" --service-account

Probably a duplicate of #392.

What is the command to estimate labels

This is closed?

I just need the command to see the estimation for a
label


/home/{user}/.fzf/shell/key-bindings.zsh:emulate:35: unknown argument -o

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
I have followed the instruction to install fzf using git way.
I have selected to use key binding.
After installation and source ~/.zshrc, I have this error:
/home/{my-user-name}/.fzf/shell/key-bindings.zsh:emulate:35: unknown argument -o

And it keeps popping up every time I login to the host.

zsh --version?

zsh 5.7.1 (x86_64-pc-linux-gnu)

I can't reproduce the problem, tested on zsh 5.4 and 5.8
$ echo $ZSH_VERSION
5.4.2
$ source ~/.fzf.zsh
$ bindkey | grep fzf
"^I" fzf-completion
"^R" fzf-history-widget
"^T" fzf-file-widget
"^[c" fzf-cd-widget

I have this correctly:
~ ➜ bindkey | grep fzf
"^I" fzf-completion
"^R" fzf-history-widget
"^T" fzf-file-widget
"^[c" fzf-cd-widget

But I just keeps getting the error when logon to the host:
/home/{user}/.fzf/shell/key-bindings.zsh:emulate:35: unknown argument -o

I too see the same error whenever I do login:
/home/$USER/fzf/shell/key-bindings.zsh:emulate:35: unknown argument -o

I too see the same error whenever I do login:
/home/$USER/fzf/shell/key-bindings.zsh:emulate:35: unknown argument -o

+1 . I see this error too. Any workarounds ?

Same issue.
zsh --version: zsh 4.3.11 (x86_64-redhat-linux-gnu)
fzf is the latest.

Same issue. zsh --version: zsh 4.3.11 (x86_64-redhat-linux-gnu) fzf is the latest.
$ cat /etc/centos-release CentOS release 6.9 (Final)

I have the same problem withe the exact same output.

I can't reproduce the problem, tested on zsh 5.4 and 5.8

Confirmed, with the default zsh version or the latest brew version the issue is not reproducible on macOS.

Building zsh from source

instructions: https://sourceforge.net/p/zsh/code/ci/master/tree/INSTALL

clone

git clone https://git.code.sf.net/p/zsh/code zsh-code
cd zsh-code

pre-configure

[!IMPORTANT]
If the configure script does not already exist -- e.g., if you've got
a snapshot of the bare sources just checked out from a git repository
-- some things need to be built before the configuration can proceed.
Run the script ./Util/preconfig to do this.

./Util/preconfig

Configure

define the location where the binary shall be placed

./configure --prefix=$HOME/.local/zsh-test

check out the desired commit/tag

git checkout zsh-4.3.17

Make the binary

make install.bin

Optionally add the binary to your PATH

# .zshrc
export PATH="$HOME/.local/zsh-test/bin:$PATH"

Run the binary

zsh --version
# zsh 4.3.17 (x86_64-apple-darwin19.6.0)

I can produce the error by running the following command with zsh version 4.3.17 non-interactively.
zsh --version
# zsh 4.3.17-dev-0 (x86_64-apple-darwin19.6.0)

# prints the current options flags set for that shell session
zsh -c 'echo $-'
# 569Xd

zsh -c 'source /Users/paria/Developer/fzf/shell/key-bindings.zsh'
# /Users/paria/Developer/fzf/shell/key-bindings.zsh:emulate:35: unknown argument -o

❯ git tag --sort="-creatordate" --format="%(creatordate:format:%d/%b/%y) - %(refname:short)" | grep -C 2 'zsh-4.3.17'
# 16/Dec/12 - zsh-5.0.0-test-1
# 21/Jul/12 - zsh-5.0.0
# 15/Jul/12 - zsh-4.3.17-test-2
# 23/Feb/12 - zsh-4.3.17
# 20/Feb/12 - zsh-4.3.16
# 17/Dec/11 - zsh-4.3.15

When bisecting the zsh source code the issue was fixed with zsh / Code / Commit [fa8a0e] (7/Mar/12)
# Commit [fa8a0e]
git checkout fa8a0e >/dev/null
make install.bin >/dev/null
zsh -c 'source ~/Developer/fzf/shell/key-bindings.zsh'

# Commit prior to [fa8a0e]
git checkout fa8a0e^ >/dev/null
make install.bin >/dev/null
zsh -c 'source /Users/paria/Developer/fzf/shell/key-bindings.zsh'
# /Users/paria/Developer/fzf/shell/key-bindings.zsh:emulate:35: unknown argument -o

the line was added with #1944

merged in b1b916ce157df4fe1299edfdd012676419e2263c

'emulate' 'zsh' '-o' 'no_aliases'

@romkatv can you help here ?

workaround

if you use the version prior to the merged commit, I assume you will not have the issue
History for shell/key-bindings.zsh - junegunn/fzf

wget -qO key-zsh-old.sh https://raw.githubusercontent.com/junegunn/fzf/18261fe31cf1270f9aa783f99bbdbca343f89479/shell/key-bindings.zsh

zsh --version
zsh -c 'source key-zsh-old.sh '

possible fix, check if interactive ? 😬
--- a/shell/key-bindings.zsh
+++ b/shell/key-bindings.zsh
@@ -32,7 +32,7 @@ else
   }
 fi

-'emulate' 'zsh' '-o' 'no_aliases'
+[[ -o interactive ]] && 'emulate' 'zsh' '-o' 'no_aliases'

 {

the line was added with [zsh] Ensure that fzf code always parses the same way #1944

merged in b1b916c

'emulate' 'zsh' '-o' 'no_aliases'

@romkatv can you help here ?

The -o flag was added to emulate in zsh 5.0.2. If you want to support zsh older than that, you'll have to avoid this flag and use setopt to set options.
'builtin' 'emulate' 'zsh' && 'builtin' 'setopt' 'no_aliases'


Show the PayPal's PayerDetails.City (the payer's city) in the Magento's backend «Payment Information» block

A PayPal's PaymentStatus API request, and a response to it
See also similar tasks for other PayPal's PaymentStatus fields:

#72: Country (the payer's country)
#73: Custom1 (the payer's account confirmation status)
#74: Custom2 (the payer's email)
#75: CustomerFirstName (the payer's first name)
#76: CustomerName (the payer's last name)
#77: State (the payer's state/region)
#78: Street1 (the payer's street address)
#79 Zip (the payer's postal code)

Implemented in 0.6.0:


emoji_picker: Allow filtering emojis by unicode characters in emoji picker.

Tested on mobile browser by opening the emoji picker with the
reaction button, selecting an emoji via the native keyboard, and
ensuring the selected emoji appears in the emoji picker’s search
result.
Screenshot testing on mobile device

Testing with multiple emojis

Fixes #2714.

Screenshots and screen captures:
Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Just a note that we've merged #21765, which reverts some of the changes that this depends on. I don't expect @Fingel to work on that PR further, so I'd encourage you to extend this PR with a version of the changes removed there that address the problems that motivated that revert PR. (E.g. tracing the code to figure out whether indeed the function ever gets called without a full emoji object)

Just a note that we've merged #21765, which reverts some of the changes that this depends on. I don't expect @Fingel to work on that PR further, so I'd encourage you to extend this PR with a version of the changes removed there that address the problems that motivated that revert PR. (E.g. tracing the code to figure out whether indeed the function ever gets called without a full emoji object)

Thank you for the update here @timabbott. Indeed the function did sometimes get called without a full emoji object. However, it will always contain the emoji_code so long as it's not a realm emoji. What was excluded however, is the reaction_type which I have now added and restored the previously reverted commit


:recycle: [iOS & Android ContributorsScreen]Added loading indicator since it is missing.

Issue

None.

Overview (Required)

I modified the UIState so that it can display a loading screen.

Movie (Optional)

Before
After

The indicator display on the iOS side cannot be confirmed because there is a problem where the contributor screen crashes when there is no network connection on the iOS side. 🙏
#954


{packager} has been archived on CRAN

https://github.com/IndrajeetPatil/awesome-r-pkgtools/blob/10e6ad57ece77049e02f2632e814239182b019ac/README.qmd#LL44C1-L44C62
Alternate location on GitLab https://gitlab.com/fvafrcu/packager

Thanks!


SetTopAppBarColor

Change the top bar colour to black with looks nice

Set top app bar colour

i am working on this @Lucky-chelani i created a pull request for this one