Accent colors and other related things could be loaded on startup as environment variables, so apps can just check a file in /run (or however environment variables are handled properly), but a fallback checking the config file will be necessary for compatibility. Also, an option to switch Accent colors on a timer like RGB motherboards and keybords.

I just think loading the Accent and theme (background) color names into RAM on startup as a file and/or environment variable will make it easier to later have apps update that color, both for testing and for if people want to override something with their own script without touching the config files. This may present an issue for the lockscreen though, but maybe that can just load from the config file, and the script can load things into RAM properly on login or something. Not accessing the storage for this may help some apps load slightly faster, but it might not help much other than being simpler for testing or if the user wants their accent color to change between every color of the rainbow (or colors in set themes that the user can create or will be shipped as choices, but maybe that should be in an Extras package when everything is split up) without constantly writing to their storage. Actually, that's a really useful idea, then on top of that, add a delay setting that lets how often it changes be set (with a warning message that pops up if an amount that's so fast it could induce a seizure is chosen by accident or on purpose). To help with this, there could be a list of colors that the user wants/doesn't want to have it switch to, and a way to add more than one custom color to the list (also the main list, since someone may want to keep more than the last 5 colors that were set). There should also be a way to fade between colors like many RGB motherboards and keyboards instead of instantly switching between them, if someone wants that feature.
Loading from the config file will still have to be done if the environment variables aren't set, so apps can be run outside Retiled (just fancier stuff won't be supported at all or as easily, unless the script is manually run but with the caveat that it could break stuff in an environment it's not intended for).

On the lockscreen, it might not be able to cycle through the colors without loading the set color cycle theme and manually changing the Accent color on a delay, but not sure about the Action Center. Maybe there'll just have to be a static, main color for that. But then, the Action Center will be able to change when logged in, so maybe it could do the same thing as the lockscreen, but once it sees that the environment variable is set, it switches over, or something.
When the screen is locked, the cycling needs to be paused to save battery life after fully switching to the next color (if fading is set, but to be safe it'll go to the next color anyway), and it can continue when unlocked. With Battery Saver, it switches to the next color and then pauses it until Battery Saver is off, similar to locking and unlocking above. Maybe a static Accent color can be done by saving the last Accent color before shutting down, that way the one on the lockscreen can still be dynamic every startup, even if it doesn't cycle until login.
An option to pause the dynamic color cycling should be added, with an item for it in an Accessibility menu in the Action Center. It'll also have a button in the Settings page where the dynamic color cycling is set up, plus a shortcut back to the Accent colors page (might be cluttery to add a button for pausing dynamic color cycling here, but if dynamic color cycling is a subpage, it could have its own link) from possibly the Color Accessibility settings page.

The page that allows dynamic color cycling to be configured will have to have radio buttons that let people choose whether to use a slider control with a minimum value that should be safe for anyone susceptible to seizures (would 5 seconds be too short?), and a maximum value that's not way too long (maybe an hour max?) so it takes forever to change the colors; or to manually set the exact amount of time (with a units dropdown) if the slider doesn't work (a time picker would be easier than a textbox, so I could use that instead). Note that both options have the same minimum and maximum limits, because I don't want to give anyone seizures by accident just because they can't read a warning message.
Also, when loading this setting from the config file, it will be automatically reset to the safe minimum setting if it's below that, then it'll be loaded, and this all happens in a script. This script will be written in Python, and it'll tell an SH (would be Bash, but that's less compatible) script which colors to cycle through from the command line with the requested delay in seconds (because that's easier than trying to do time in different units). To be safe, that script will also make sure the requested delay is 5 seconds or more, and if not, it'll set the delay to 5 seconds. (All this safety is to ensure someone can't just modify someone's phone's configuration to give them a seizure without doing a lot of extra work.)
I think the script that sets the colors when cycling could just sleep for the requested delay then set the environment variable and sleep again. Sleeping should start after the first color is set so it doesn't look like it's broken immediately, causing the user to turn it back off, though.


BUG: Remove wrong Python wrap file.

Remove Python wrap file accidentally committed in 6c736b7.

@jhlegarreta I enabled the CI services for this repository. Please try re-pushing this branch to trigger the builds with something like
git checkout RemoveWrongWrapFile
git commit --amend --no-edit
git push -f origin HEAD

Done.

Thanks @jhlegarreta .
I fixed the CircleCI build configuration so it builds forked pull requests.
Other CI errors are unrelated, so merging.

Thanks @thewtex  !


[CRD] versioning support in StarrocksCluster

Describe the current behavior
Currently, CRD: StarrocksCluster has a fixed version apiVersion: starrocks.com/v1, without proper versioning and capability guarantee, the risk to upgrade CRD is unknown.
Describe the enhancement
Support versioning in CRD: StarrocksCluster, and add capability verification during CRD upgrade.
Additional context
Add any other context or screenshots about the enhancement here.
cc @dengliu @yandongxiao

If we are just marking the version information of CRD, can we add the corresponding version information to the annotation or label of CRD?

It's good to stick using one version in CRD, if we're able to keep backward/forward compatible crossing the released CRDs:

the old CRs can be parsed by the new operator, this eliminates the migration effort when upgrading operator/crd;
the new CRs can be parsed by the old operator, this ensures it's safe to rollback operator upgrade;

Technically we can use webhook to rewrite CRs when compatibility is not guaranteed. I don't suggest to move that far until necessary, adding logic in operator to handle unset behaviors is much simpler.

Some more information in https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definition-versioning/


Proof Key for Code Exchange is required for cross-origin authorization code redemption

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

After updating the authConfig.ts file with the latest changes, if you have added the Client ID as an authorization in Azure WebApp, you may encounter the following error during login:
AADSTS9002325: Proof Key for Code Exchange is required for cross-origin authorization code redemption.

client id app registration is added as authentication in Azure WebApp

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

According to this SO post, you need to add a Web section in addition to the Single Page section
https://stackoverflow.com/questions/64692600/aadsts9002325-proof-key-for-code-exchange-is-required-for-cross-origin-authoriz
Does this solve your issue? I will try to reproduce this as well

@mattmsft
It worked after updating the settings to below in client app.

and

@vrajroutu  thanks, are you OK to close this issue?

Why is this not added into the Readme file?

I'll add it in a PR today!

https://github.com/Azure-Samples/azure-search-openai-demo/pull/2224 has various updates for the login doc.


Generated scripts can be written in a bogus encoding

https://github.com/pypa/setuptools/blob/118edbb2b715c96620b51018c1d28e81f2318053/setuptools/command/easy_install.py#L836-L837
open's encoding argument, when unspecified, uses locale.getpreferredencoding , which is not necessarily UTF-8. However, the generated script doesn't include an encoding cookie and can contain non-ASCII data if, say, the shebang path contains non-ASCII components. Either always open the file with encoding='utf-8' or write out an encoding cookie.
This has resulted in an issue filed on a different project: https://github.com/pypa/pipenv/issues/1265

raise this issue in pip?

Looks like it's already done.

Is this related to #1400?

@pganssle AFAICT #1400 is a duplicate of this issue yes.


Parallelising cubes concatenation

In ESMValTool, we have some recipes that requires concatenating a long list of cubes.
This is done with CubeList.concatenate, which, as far as I understand, loops over the cubes, identifies cubes with compatible signatures, and check whether the coordinates of matching cubes are equal (in proto_cube.register):
https://github.com/SciTools/iris/blob/f8a45bed68982eb741c493aac49b99f56a208dad/lib/iris/_concatenate.py#L335-L353
As in #5743 , we observe significant slowdown when we try to act on a list of cubes with lazy coordinates. These coordinates, in fact, need to be computed (read from disk) in order to compare them with the ones of other cubes, and this is now carried out sequentially. Of course we could skip the auxiliary/derived coordinate comparison (e.g. check_aux_coords=False), but we would like to keep all checks to make sure the concatenation is robust.
We are thinking to possible approaches to speed up the concatenation for similar use cases, i.e. long lists of cubes with lazy coordinates. One way to do this would be to "realise" all coordinates, which could be done in parallel for all cubes- but this would bring quite some memory usage as a disadvantage.
Another potential strategy (suggested by my colleague @bouweandela) would be to load coordinates, hash them, and only store hashes, to be used later on for comparisons between cubes. By running the coordinate loading & hashing for all cubes in parallel one could get quite some performance improvement without significantly increasing the memory footprint.
What are your thoughts on this? How would you see the (optional) usage of hashes for the coordinate comparison when concatenating cubes?

I may be out of my depth here, but is a [direct] hash of floating point values really useful? I imagine that some rounding, possibly user-adjustable, would prevent unwarranted down-to-last-bit-agreement requirement.

Yes, I afraid that is the case (without knowing exactly how it is done), which has bitten us several times (e.g. here. Why enforce that level of precision when comparing floating point values?

Aah -- this sounds promising :-)) pinging @ljoakim for info


Enhance the ExecutionDomain

We'll need to discuss enhancements to the analysis server's ExecutionDomain. Right now it's essentially hardcoded to know about dart cli apps and dart web apps. We'll need to make this mechanism more flexible to cover application types that the analysis server may not be aware of.
For example, a project may have several dart libraries with main() methods. Some of these libraries may be regular dart CLI apps, and some may be flutter entry-points. Atom will need to launch these two libraries differently. The only way we'll be able to decide this is by looking at the transitive closure of the imported libraries for each entry-point.
This is similar to what's done for web apps (looking for dart:html). We'd recognize a flutter entry-point by the transitive import of package:flutter. We'll want a general purpose solution that lets clients self-serve when they need to support new launch types.

Also note that configuration specific imports will make the transitive search become less reliable.

/cc @lrhn

From discussion, we may be able to self-serve on this one, if we run a regex over the file with main(). We'd have to be ok with a ~low false negative rate.

From discussion, we may be able to self-serve on this one, if we run a regex over the file with main(). We'd have to be ok with a ~low false negative rate.

Is this sufficient or do we want an analysis query for the the transitive closure of imports?

New AnalysisDomain request for the transitive closure of reachable sources in flight:
https://codereview.chromium.org/1491013002/
This should solve the specific problem for flutter (e.g., "is this a flutter entry-point?") but generalizes to other cases (like web apps, tests, and beyond).

AnalysisDomain.getReachableSrouces request landed in 43f027fc05f25a40049dee0a71418570f6f20d10.

:metal:


[ADD] DevOps

Category Name
DevOps
Subcategory Name
CI, CD, Containerization, Orchestration
Additional Context
DevOps is widely used in the present Software Industry for faster and more reliable software development and deployment through improved collaboration, automation, and efficient processes. I believe adding a DevOps category in Linkshub would be a perfect fit
Subcategories would be
CI/CD - Jenkins, GitLab, Azure pipelines
Containerization - Docker, Kubernetes, Google Cloud
Orchestration - Kubernetes, Docker Swarm, Mesos, Nomad
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I have checked that this category does not exist
[X] I have checked that this subcategory does not exist
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

@saikiranreddy201 assigned to you, you may start working on it now


Authentication dialog opens for .NET Core when using Active Directory Interactive Authentication

I have been using Microsft.Data.SqlClient in .NET Framework with Active Directory Interactive authentication and it worked great:

When running on Azure, the App Service's identity was used to connect to the database
When running locally, my Microsoft account was used to connect to the database
I had nothing to do it just worked

I have seen that Active Directory Interactive authentication is now available for .NET Core as well but it does not behave at all the same way. Each time I do a connection.OpenAsync() it launches an authentication dialog in my browser for me to log in with my account. Then my code can successfully make request to the database but I don't want to do a manual action each time my code runs locally.
Am I missing something ? My database connection string has the following format : "Server=tcp:sql-dev-myserver-data.database.windows.net,1433;Initial Catalog=db-dev-kpis;Persist Security Info=False;MultipleActiveResultSets=False;Encrypt=True;Authentication=Active Directory Interactive;"
Is there a way to avoid this dialog and that my code connects to the database silently with my account already logged in in Visual Studio as it was doing in .NET Framework ? The solution I was using until now for .NET Core was to retrieve an access token and set it directly in the connection but that is something I was hoping to avoid now that Active Directory Interactive is supported in .NET Core.

@TechWatching
"Active Directory Interactive" authentication mode by-design performs authentication interactively with a dialog window.
It seems like you want to make use of Managed Identity when authenticating.
In case of .NET Framework application, you've probably configured your application to use a custom authentication provider that's not inbuilt in driver, but implemented by AppAuthentication library, as documented here. That provider is not what we own, so you'd have to reach out to https://github.com/Azure/azure-sdk-for-net team for .NET Core support regarding that.
On the other hand, we'll be introducing a different authentication mode for Managed Identity based authentication specifically, to be used as "Active Directory Managed Identity" (Feature #616 - WIP #730).

Indeed you are totally right, I forgot I was using this library. Thanks for the information. Will this Managed Identity based authentication work for local debugging in Visual Studio as well (using the identity of the account logged in Visual Studio) ?

@TechWatching
I'm not sure what you mean by "Identity of account logged in Visual Studio".
Identity based authentication works only with Managed Identities in Azure (System-Assigned and User-Assigned).
They are not attached to any user account.
Can you point me to any official documentation that discusses this approach?
Furthermore, if you want to authentication with active directory user account, you can always use Active Directory Integrated/Interactive modes.

To clarify these two behaviors you need working:

When running on Azure, the App Service's identity was used to connect to the database
You will be able to use "Active Directory Managed Identity" as authentication mode in connection string after #730 is merged and released.

When running locally, my Microsoft account was used to connect to the database
You can use "Active Directory Integrated" or "Active Directory Interactive" authentication modes (supported since M.D.S v2.0.0+ for .NET Core)

Yes sure I am referring to [this documentation][https://docs.microsoft.com/en-us/azure/app-service/app-service-web-tutorial-connect-msi#modify-aspnet)

That's every thing you need to connect to SQL Database. When debugging in Visual Studio, your code uses the Azure AD user you configured in Set up Visual Studio. You'll set up SQL Database later to allow connection from the managed identity of your App Service app.
I don't want to have to change my connection string depending on whether I a running the code locally or on Azure. I want the connection string to be specified in my app settings file and the code to use the identity appropriate to my context.
It is currently the case in .NET Framework with  "Active Directory Interactive" and the library you mentionned : on Azure the Managed Identity from my App service is used, and locally my Microsoft account is used. It feels a little bit like magic but the developement experience is great. I would love if it could look like that in .NET Core.

By the way, I did not manage to make the "Active Directory Integrated" work for me maybe because the Azure SQL Resource in on a different tenant (from a client company) than my Microsoft account (of my consulting company).
"Active Directory Interactive" .NET Core works fine but having to manually connect each time I launch the debug is not a very convenient experience.
Most of my code now runs in .NET Core but I have the feeling the interaction with the dabase was smoother when my code ran in .NET Framework, once configured everything just worked without manual actions (to log in or to change a connection string).

Yes sure I am referring to [this documentation][https://docs.microsoft.com/en-us/azure/app-service/app-service-web-tutorial-connect-msi#modify-aspnet)

That's every thing you need to connect to SQL Database. When debugging in Visual Studio, your code uses the Azure AD user you configured in Set up Visual Studio. You'll set up SQL Database later to allow connection from the managed identity of your App Service app.

I don't want to have to change my connection string depending on whether I a running the code locally or on Azure. I want the connection string to be specified in my app settings file and the code to use the identity appropriate to my context.
It is currently the case in .NET Framework with "Active Directory Interactive" and the library you mentionned : on Azure the Managed Identity from my App service is used, and locally my Microsoft account is used. It feels a little bit like magic but the development experience is great. I would love if it could look like that in .NET Core.

Interesting! I will look more into it and get back to you!

Hi @TechWatching
I did discuss this internally and it may be possible improve the overall design of Managed Identity authentication experience to match that of AppAuthentication library. As of now, it won't be part of next release since we cannot take dependency on Azure.Identity for the same.
Keeping issue open to address this in future.

@TechWatching
In the meantime, there's another solution you could implement that will bring back same experience for you with .NET Core too, as I mentioned here: https://github.com/dotnet/SqlClient/pull/730#issuecomment-705307179


Does ceph ansible support mon public addr?

This is used to tell the node the public address it cannot find from the interface. If it is not there, ceph clients cannot connect.
2017-05-03 15:46:33.011447 xxxxxxxxx 0 -- - >> xxxxxxxx:6789/0 conn(0x558fbfed7cd0 :-1 s=STATE_CONNECTING_WAIT_BANNER_AND_IDENTIFY pgs=0 cs=0 l=1)._process_connection connect claims to be <IP_ADDRESS>:6789/0 not x.x.x.x:6789/0 - wrong node!
On further testing. I'm not sure what is wrong.

The host has an internal network that is not accessible to the public and it's the only public network available, since the other nodes in the cluster are able to access the internal ip. There's no point in using tinkering further.


exclude some Files or directory t

Hello. I test this plugin but i wondering if function are available to exclude some folder os file extension ?
My backups is taking a lot of size when it backup. This is because  of it backup Uploaded sub folder that i have link to my Backup storage with a  lot of Gcode files and STL and 3MF files and so on..

There is not, I suggest you use a separate folder for gcode and just use that for sync and define the folder in the plugin's settings.


It hides all files of repo and breaks code navigation

Please ensure:

[X] The bug is caused by Refined GitHub. It doesn't happen if I disable the extension.
[X] The bug happens after clearing extension cache. The "Clear cache" button below can also be found at the bottom of the options page.

Example URLs
https://github.com/refined-github/refined-github

Extension version
22.3.3
Browser(s) used
Brave
Description
Can't see files listed on a github repo as seen in screen shot. It seems to be an issue loading fonts.

Include in this issue:

[X] Screenshots/video/gif demonstrating the bug, if it’s visual
[X] Console errors, if any

Can't see files listed on a github repo as seen in screen shot

You can toggle the files list with this button:

@Ashtonian It looks like you used toggle-files-button and the feature remembers the last state it was on. To fix it, click the button again.

https://github.com/refined-github/refined-github/issues/3588


Add security providers in kafka security config

Adds a new custom security provider class configuration.
The configured classes are added to the Security provider via Security.addProvider api
KIP - https://cwiki.apache.org/confluence/display/KAFKA/KIP-492%3A+Add+java+security+providers+in+Kafka+Security+config

How do we handle a scenario that some providers require more information for installing providers?
For instance, Bouncy Castle(BC) provider requires input parameter "fips:BCFIPS" for enabling FIPS mode.
Example:
Static Configuration in java.security file:
security.provider.1=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider  fips:BCFIPS
Dynamic Installation
Security.addProvider(new BouncyCastleJsseProvider(“fips:BCFIPS”))
So I suggested we might consider providing more info for the new config property, example like:
security.provider.info=classname of provider/name of provider/initial parameters,
Example for BC case:
security.provider.info=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider/BC/fips:BCFIPS,sun.security.provider.Sun/SUN,
Basically info for each provider will consist of three pieces information: name of class, name of provider(for unit testing purpose),initial parameter for instantiating class.
Still use comma ","  for separating each provider info.
Also new class handling installing provider should consider any other component could use this if it run in different Java vm.
Jeff Huang,

How do we handle a scenario that some providers require more information for installing providers?
For instance, Bouncy Castle(BC) provider requires input parameter "fips:BCFIPS" for enabling FIPS mode.
Example:
Static Configuration in java.security file:
security.provider.1=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider fips:BCFIPS
Dynamic Installation
Security.addProvider(new BouncyCastleJsseProvider(“fips:BCFIPS”))
So I suggested we might consider providing more info for the new config property, example like:
security.provider.info=classname of provider/name of provider/initial parameters,
Example for BC case:
security.provider.info=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider/BC/fips:BCFIPS,sun.security.provider.Sun/SUN,
Basically info for each provider will consist of three pieces information: name of class, name of provider(for unit testing purpose),initial parameter for instantiating class.
Still use comma "," for separating each provider info.
Also new class handling installing provider should consider any other component could use this if it run in different Java vm.
Jeff Huang,

Hi Jeff, the BouncyCastleJsseProvider example you've provided is one case taking in an constructor parameter, apart from this, there are lots of other provider Constructors which take in complex arguments like a map or an object. Addressing just the string case will be solving for a specific use case leaving out more generic cases. The base "security.Provider" currently doesn't provide any configure methods as of now; if so, we might have evaluated sending in a configuration map. So, probably the best way to handle this with the limited functions provided by "security.Providers" class is making the provider read from environmental variables or from a properties file.

How do we handle a scenario that some providers require more information for installing providers?
For instance, Bouncy Castle(BC) provider requires input parameter "fips:BCFIPS" for enabling FIPS mode.
Example:
Static Configuration in java.security file:
security.provider.1=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider fips:BCFIPS
Dynamic Installation
Security.addProvider(new BouncyCastleJsseProvider(“fips:BCFIPS”))
So I suggested we might consider providing more info for the new config property, example like:
security.provider.info=classname of provider/name of provider/initial parameters,
Example for BC case:
security.provider.info=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider/BC/fips:BCFIPS,sun.security.provider.Sun/SUN,
Basically info for each provider will consist of three pieces information: name of class, name of provider(for unit testing purpose),initial parameter for instantiating class.
Still use comma "," for separating each provider info.
Also new class handling installing provider should consider any other component could use this if it run in different Java vm.
Jeff Huang,

Hi Jeff, the BouncyCastleJsseProvider example you've provided is one case taking in an constructor parameter, apart from this, there are lots of other provider Constructors which take in complex arguments like a map or an object. Addressing just the string case will be solving for a specific use case leaving out more generic cases. The base "security.Provider" currently doesn't provide any configure methods as of now; if so, we might have evaluated sending in a configuration map. So, probably the best way to handle this with the limited functions provided by "security.Providers" class is making the provider read from environmental variables or from a properties file.

Jeff Huang.

How do we handle a scenario that some providers require more information for installing providers?
For instance, Bouncy Castle(BC) provider requires input parameter "fips:BCFIPS" for enabling FIPS mode.
Example:
Static Configuration in java.security file:
security.provider.1=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider fips:BCFIPS
Dynamic Installation
Security.addProvider(new BouncyCastleJsseProvider(“fips:BCFIPS”))
So I suggested we might consider providing more info for the new config property, example like:
security.provider.info=classname of provider/name of provider/initial parameters,
Example for BC case:
security.provider.info=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider/BC/fips:BCFIPS,sun.security.provider.Sun/SUN,
Basically info for each provider will consist of three pieces information: name of class, name of provider(for unit testing purpose),initial parameter for instantiating class.
Still use comma "," for separating each provider info.
Also new class handling installing provider should consider any other component could use this if it run in different Java vm.
Jeff Huang,

Hi Jeff, the BouncyCastleJsseProvider example you've provided is one case taking in an constructor parameter, apart from this, there are lots of other provider Constructors which take in complex arguments like a map or an object. Addressing just the string case will be solving for a specific use case leaving out more generic cases. The base "security.Provider" currently doesn't provide any configure methods as of now; if so, we might have evaluated sending in a configuration map. So, probably the best way to handle this with the limited functions provided by "security.Providers" class is making the provider read from environmental variables or from a properties file.

Hi, This is my question.
I can statically configure BC providers in java.security with following format, JVM can load the provider with additional parameter "fips:BCFIPS" and it works fine. Why we cannot provide same configuration format in Kafka to allow specify additional parameter in addition to class name for Provider.
The following configuration is tested and working. JVM can load and handle.
security.provider.1=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider  fips:BCFIPS
So I think in Kafka, we might follow same pattern like static config in java.security file, so new config property would be:
security.providers=provider1  ,provider2
thanks!
Jeff.

These are my suggestions based on discussion with Rajini @rajinisivaram and on my current project relate to FIPS using Bouncy Castle(BC) provider which is compliant and certified provider and used by many company want to do business with government.

The new security config would be:
"security.providers=provider1 class  ,provider2". Format would fellow same pattern in java.security.
security.provider.1=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider fips:BCFIPS
Create a new class such as SecurityProvidersInstaller or SecurityProvidersLoader. This class will implement a default function addConfiguredSecurityProviders or we might call configure() ? without consider operational parameter.
Any customized special handling like BC could extend the class and overwrite the function addConfiguredSecurityProviders.
This class will be needed by both broker and clients such as producer, consumer, so we need put this class into appropriate package.
Only install all providers once per JVM.

Thanks!
Jeff Huang.

These are my suggestions based on discussion with Rajini @rajinisivaram and on my current project relate to FIPS using Bouncy Castle(BC) provider which is compliant and certified provider and used by many company want to do business with government.

The new security config would be:
"security.providers=provider1 class  ,provider2". Format would fellow same pattern in java.security.
security.provider.1=org.bouncycastle.jsse.provider.BouncyCastleJsseProvider fips:BCFIPS
Create a new class such as SecurityProvidersInstaller or SecurityProvidersLoader. This class will implement a default function addConfiguredSecurityProviders or we might call configure() ? without consider operational parameter.
Any customized special handling like BC could extend the class such as BCSecurityProvidersInstaller and overwrite the function addConfiguredSecurityProviders.
This class will be needed by both broker and clients such as producer, consumer, so we need put this class into appropriate package.
Only install all providers, such as for TLS, SASL, once per JVM.

Thanks!
Jeff Huang.

Ok, these suggestions make sense and sound reasonable. We'll go ahead with the following new changes

Creation of a new SecurityProvidersLoader interface
This interface contains a method configureProvider(or a better name) which takes in a Map<String, ?> properties object
Any configuration of Providers is expected to be implemented in the configureProvider method
This method is passed the properties passed to the broker/producer/consumer

security.providers

Will the new config property be "security.providers"?
Thanks!

security.providers

Will the new config property be "security.providers"?
Thanks!

We can go ahead with "security.provider.classes" as mentioned in the KIP, unless there is some problem with it.

security.providers

Will the new config property be "security.providers"?
Thanks!

We can go ahead with "security.provider.classes" as mentioned in the KIP, unless there is some problem with it.

Actually the main idea for this KIP will allow dynamically configure security providers. so I still think "security.providers",  from long term view and flexible aspect,  better than "security.provider.classes" to represent the main idea even both need classes name. Also using "security.providers" will be consistent with format in static configuration in java.security file.
Thanks!
Jeff.

@jeffhuang26 please review

HI Saisandeep,
@saisandeep
Any update on my comments?
Jeff.

HI Saisandeep,
@saisandeep
Any update on comments and plan for moving forward? I have one project depends on this PR. :-}
thanks!
Jeff.

Ok, will put this as a priority

@jeffhuang26 please review

Any update on this PR?


withAlias adds field name to alias name

q.withAlias("alias").field();
produces the graphql:
field_alias: field
I'd expect it to produce
alias: field
See startField()
https://github.com/Shopify/graphql_java_gen/blob/master/support/src/main/java/com/shopify/graphql/support/Query.java#L66

Don't you mean the alias name would end up being field__alias in your example.
It preserves the original field name so that it can be obtained on the response to parse it with this library, with the assumption that the alias is there so the field can be queried more than once.

If that's by design, then maybe we can have a variant of withAlias that generates the alias as requested?

That would break response parsing, which isn't done with access to the original request.
Do you have an actual use case for being able to avoid the field name prefix?


Get column values as an array

Is your feature request related to a problem? Please describe.
Given the example in the documentation:
const dfd = require("danfojs-node")

let data = {"A": [-20.1, 30, 47.3, -20],
            "B": [34, -4, 5, 6],
            "C": [20, -20, 30, -40]}

let df = new dfd.DataFrame(data)

console.log(df.values)

And the output
[
  [ -20.1, 34, 20 ],
  [ 30, -4, -20 ],
  [ 47.3, 5, 30 ],
  [ -20, 6, -40 ]
]

It is useful to obtain the values by rows, but I would like to obtain the values by columns, i.e. to recover the original arrays.
Describe the solution you'd like
It'd be great to have something like Pandas: df['A'].to_list() with output:  [-20.1, 30, 47.3, -20]

@iserranoe You can do the following for column as well:
console.log(df["A"].values)


Scenario app's screenshots are not stable enough on CI

h/t to @yjbanov for noticing this:

As a first step I want to verify these screenshots manually and document how they are supposed to work: https://github.com/flutter/flutter/issues/143350.

So the bottom image is part of the nav bar. Its odd that it is missing.
But the top could be explained by us sometimes capturing frame N or N+1. From looking at the golden harness, there isn't actually any synchronization between rendering and readback.

So the bottom image is part of the nav bar. Its odd that it is missing.
But the top could be explained by us sometimes capturing frame N or N+1. From looking at the golden harness, there isn't actually any synchronization between rendering and readback.

If you look at the Skia gold link, you'll see we generate basically infinite different images, so makes sense...

Here is something interesting:

API v34 is stable across all backends

API v28, on Skia, is very unstable

I wonder if there is something about API v28 emulators that are different, for example they didn't hide the status bar by default, but the newer versions do?

This is no longer occurring. I will re-open if the problem persists.


Serialising patch requests not working correctly

Not really sure whether you would consider this a bug or a feature but I thought I'd raise it in case I can save other people some pain. I am using 0.5.0-RELEASE2 which I upgraded to so that we could use the new PATCH functionality.
The issue I am seeing is this:

I have an entity being served up by a Spring Data REST service that contains a linked resource.
I modify the entity in a web-app to update the linked resource to point to a different resource.
When the request is serialised to send to the SDR service it always serialises the 'old' linked resource.

After some investigation this appears to be because of a combination of two things:

When the resource setter method is called it does actually update the underlying instance BUT the linked resources contained in the LinkedResourceMethodHandler are never updated.
The handler chain has the LinkedResourceMethodHandler in the chain BEFORE the getter handler which means that it returns the resource from the list in the LinkedResourceMethodHandler which is the original linked resource.

Would be interested to get your thoughts on this and if you had any ideas for a possible work around / fix.
I am aware of #2 but that obviously relates to PUT operations rather than PATCH.
Thanks, Andy.

Been looking at this a little bit more and I do consider it a bug mainly because there is the LinkedResourceUriSerializer class in the JacksonClientModule class file. This class is trying to write out the URI for the linked resource on any serialisation of the entity but as the instance has been updated I think it is actually writing out the wrong one.
Also, after looking more closely at things I was wondering about the possibility of using LinkedResource annotation on setters as well as getters? If we did this then the LinkedResourceMethodHandler would be triggered for setters as well, then in the invoke method we could look at whether it was a setter or getter before deciding what to do? If it were a setter method being called we could replace the value in the map instead of calling the service for the result?
Happy to do the work on this if someone can provide a little guidance?

Hi, thanks very much for reporting this!
I agree with you - it's definitely a bug, and a pretty massive one at that! Your analysis of the problem seems spot on. I don't think it's PATCH-specific - basically any time you mutate a @LinkedResource property of a proxied response and re-serialise, the original value is marshalled instead, because the handler uses the version that's cached within it.
I'm not sure the best way to fix this yet but I think you're on to something with your second comment - I'm not sure it makes sense to split up handlers based on whether they're getting or setting - you kind of want them to share the same instance state regardless. It may well make more sense just to define handlers based on what annotation (or lack of) that they're targeting. Looks like quite a big refactor though: it's certainly making me question whether my splitting of the old massive monolithic MethodHandler into a chain of little ones was necessarily a good idea!
Would you be interested in raising a PR to address all this? Once deciding on a strategy if need be?
Regarding a temporary workaround - all I can think of is wondering whether there's some way you can 'clone' the response state before re-submitting. Maybe you could use some JavaBeans-property-aware lib like BeanUtils, or maybe you could use Jackson but with a new ObjectMapper that doesn't register the Bowman module?

Hi Ryan, I actually have a working patch in test at the moment that seems to do the trick. Without copying the whole thing here, the salient parts are:
class LinkedResourceMethodHandler extends AbstractContentDelegatingMethodHandler {
.
.
.
    @Override
    public boolean supports(Method method) {
        return method.isAnnotationPresent(LinkedResource.class);
    }

    @Override
    public Object invoke(Object self, Method method, Method proceed, Object[] args)
            throws InvocationTargetException, IllegalAccessException {

        if (isSetter(method)) {
            super.invoke(self, method, proceed, args);
            linkedResourceResults.put(getGetterFromSetter(method).getName(), args[0]);
        }
        else {
            if (!linkedResourceResults.containsKey(method.getName())) {
                linkedResourceResults.put(method.getName(), resolveLinkedResource(self, method, proceed, args));
            }
        }
        return linkedResourceResults.get(method.getName());
    }

    private boolean isSetter(Method method) {
            return Arrays.stream(getContentBeanInfo().getPropertyDescriptors())
                    .map(PropertyDescriptor::getWriteMethod)
                    .anyMatch(method::equals);
    }

    private Method getGetterFromSetter(Method method) {
            return Arrays.stream(getContentBeanInfo().getPropertyDescriptors())
                    .filter(pd -> method.equals(pd.getWriteMethod()))
                    .collect(Collectors.toList()).get(0).getReadMethod();
    }
.
.
.
}

There are a couple of other really small changes that are required to support this but ommitted here for brevity. I'll create a fork later and apply the patch to and raise a PR and we can discuss it and modify as required.

Brilliant, thanks!

Fixed by #22


fix(kit): InputDateRange fix showing same month twice when min is later than given month

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Build or CI related changes
[ ] Tests related changes
[ ] Documentation content changes

What is the current behavior?
Closes #4544
What is the new behavior?

something is wrong when default view month is selected
https://taiga-ui--pr4569-calendar-wqvhfz4i.web.app/components/input-date-range/API?defaultViewedMonth$=2&min$=2&max$=4

something is wrong when default view month is selected
https://taiga-ui--pr4569-calendar-wqvhfz4i.web.app/components/input-date-range/API?defaultViewedMonth$=2&min$=2&max$=4

oh sorry, its reproduced on main. probably a separate bug


New angular-cli generated project has webpack error

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
1.2.2
Repro steps.
Generate new project
The log given by the failure.
After projects gets generated I do this: "npm ls --depth=0"
That shows:
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>Desired functionality.
No errors
Mention any other details that might be useful.

@angular/cli dependency should be updated to<EMAIL_ADDRESS>

Revert back to 1.2.1 for the time being, the issue is due to version 1.2.2 using webpack 3.3.0
https://github.com/angular/angular-cli/blob/1.2.x/packages/%40angular/cli/package.json#L85
Rather than 2.4.0
https://github.com/angular/angular-cli/blob/v1.2.1/package.json#L99

Just wanted to chine in as a maintainer and highlight @RicardoVaranda's comment. This was a mistake on our part.

Visit the angular webpack website, I have mentioned direct link https://v5.angular.io/guide/webpack

Update the package.json
After that remove node_modules rm -rf node_modules/
Install npm install
run the command as you mentioned above - npm ls --depth=0

├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

I have re-created this problem, as I can see the version issue. So just visit the website and update the version.


Add mvnw and Github actions

The PR adds mvnw to download maven and GitHub actions for 8 and 11 java and different platforms

Tks, nice feature.


Add unit tests for Java

I take this on. I plan to turn it into a minimal Maven project and will create a pull request in a short while.

Opened pull request #152 for this.


[13.0] ISPN-14053 Use WildFly OpenSSL 2.2.5

https://issues.redhat.com/browse/ISPN-14053

@tristantarrant is causing some failure with
Caused by: java.lang.UnsatisfiedLinkError: /home/infinispan/workspace/Infinispan_PR-10453/server/router/target/tmp-18165555667644039469openssl/libwfssl.so: /lib64/libc.so.6: version `GLIBC_2.34' not found (required by /home/infinispan/workspace/Infinispan_PR-10453/server/router/target/tmp-18165555667644039469openssl/libwfssl.so)

Lovely

No. it's unrelated

merged! thanks @tristantarrant !


AWS provider gets out-of-sync when submitting batch changes using the TXT registry

When using the AWS provider with the TXT registry, the limit change set logic is not able to differentiate between desired A/CNAME records and accompanying TXT records as shown below: https://github.com/kubernetes-incubator/external-dns/blob/fd3bed1f879c7af2c64a4865415b4b06a47473b0/provider/aws.go#L290-L311
Scenario:
I have set the --aws-max-change-count to 50 and have 100 ingresses which need to have their DNS records and TXT records created from scratch (empty zone). External-dns discovers to correct records to create, update and delete and passes it to the registry, which then appends the appropriate TXT records to the create list. This is then passed to the aws apply changes function which then applies the changes in batches of 50.
Problem:
The first set of 50 is selected from the change list using:
https://github.com/kubernetes-incubator/external-dns/blob/fd3bed1f879c7af2c64a4865415b4b06a47473b0/provider/aws.go#L363-L393
and this is them applied on the first run. The issue is, is that this first list can be a set of A/CNAME records or TXT records, and not necessarily the matching pair. This will then result in having out-of-sync registry data which it is unable to fix.
Fix:
I think a batch change should be considered individual calls to the aws api rather than the amount of changes to be made in a single sync. This would mean that after a sync we know that the state in route53 matches that of the desired state at the start of the sync. To do this we would need to loop on the change set, batch the calls and have some sort of lock, stopping a subsequent re-sync happing during the batch update.

Below shows the logs external-dns outputs showing this issue in more detail:
$ ./external-dns --source=ingress --provider=aws --zone-id-filter=MYROUTE53ZONE --aws-max-change-count=2 --registry=txt --txt-prefix=my-prefix_ --interval=5s --log-format=json
...
{"level":"info","msg":"Created Kubernetes client http://my.kubernetes.client","time":"2018-09-06T16:32:54+01:00"}
{"level":"warning","msg":"Initial change batch count is 20","time":"2018-09-06T16:32:55+01:00"}
{"level":"warning","msg":"Limited change batch count to 2","time":"2018-09-06T16:32:55+01:00"}
{"level":"info","msg":"Desired change: CREATE my-prefix_test-1.my.domain TXT","time":"2018-09-06T16:32:55+01:00"}
{"level":"info","msg":"Desired change: CREATE my-prefix_test-2.my.domain TXT","time":"2018-09-06T16:32:55+01:00"}
{"level":"info","msg":"Record in zone my.domain. were successfully updated","time":"2018-09-06T16:32:55+01:00"}
{"level":"warning","msg":"Initial change batch count is 20","time":"2018-09-06T16:32:59+01:00"}
{"level":"warning","msg":"Limited change batch count to 2","time":"2018-09-06T16:32:59+01:00"}
{"level":"info","msg":"Desired change: CREATE my-prefix_test-1.my.domain TXT","time":"2018-09-06T16:32:55+01:00"}
{"level":"info","msg":"Desired change: CREATE my-prefix_test-2.my.domain TXT","time":"2018-09-06T16:32:55+01:00"}
{"level":"error","msg":"InvalidChangeBatch: Tried to create resource record set [name='my-prefix_test-1.my.domain.', type='TXT'] but it already exists\n\tstatus code: 400, request id: my-request-id","time":"2018-09-06T16:32:59+01:00"}

In this example, I have created 10 test ingresses in my cluster which need DNS to be created for them. As well as their A records, I also need to have the accompanying TXT records created (you can therefore see that it wants to create 20 records). I have capped the batch change to a total of 2 records (arbitrarily low but will outline the bug) which means that external dns will take the first two changes alphabetically. What we see is that due to the aws provider not being aware of the TXT records themselves (as we are using a custom prefix), it starts to create disjoint pairs of records. Once it has completed this first batch all subsequent batches fail as it is not aware of the newly created TXT records.
To keep the separation between a registry and a provider, I think it would be best to have external-dns fully sync all records at part of each sync period, thus having >=0 batch per sync rather than <=1.


Sendmail Analyzer

I am thinking of integrating this into docker-mailserver. Would that be merged to upstream?
https://github.com/darold/sendmailanalyzer

You should be able to notice a log rotation by checking the files, perhaps? Something similar to the check for changes script but external that sees if the files change? If the main log file is smaller than it was the last iteration it must have been rotated. Or if there is a rotated file that is new (use find command with modification time and pattern).

Sounds like good ideas. Since this has no urgency for me at the moment, this month is pretty stuffed and also https://github.com/darold/sendmailanalyzer/issues/72 did not create any interest, I will put that on my backlog for now...

Grafana with ELK and prometheus / blackbox_exporter

Do you happen to have a docker-compose file for that config that you could share?

Have a look at this:
https://github.com/deviantony/docker-elk

This is another good idea of yours @mindrunner. Feel free to re-open this if you feel like there is more to this issue. I will close this due to age and lack of activity. Good luck tough!


pygsti changes matplotlib default settings

Describe the bug
When importing pygsti several default settings of matplotlib are changed. This causes problems for various other packages using matplotlib for plotting.
To Reproduce
Steps to reproduce the behavior:
# pip install deepDiff matplotlib pygsti
import matplotlib as mpl
from deepdiff import DeepDiff
print(DeepDiff(mpl.rcParams, mpl.rcParamsDefault))
from pygsti.modelpacks import smq1Q_XYI
print(DeepDiff(mpl.rcParams, mpl.rcParamsDefault))

In particular the patch.edgecolor is changed from black into white, which is a problem since usually backgrounds are white.
Expected behavior
Importing pygsti should not override the matplotlib default settings.
Environment (please complete the following information):

pyGSTi version: 0.9.10.post94
python version: 3.8
OS: Windows 10

Additional context

I can confirm this was a bug, due to unnecessarily importing seaborn in one of pygsti's files.  It's been fixed in d54fea2121775b9eb627d28043472188e07b3ead, and will be in the upcoming v<IP_ADDRESS> release.

Closing now that version <IP_ADDRESS> is released.


[FEATURE] Videos and Projects Pages Need a Loader

Description
The Loader (That stuff that rolls when you go to a page that hasn't fully loaded)  should be conditionally rendered. The Loader should only be removed when all the resources have been fetched.
Screencast from 14-12-2022 14:41:43.webm
Screenshots

Anyone wants to do this??

Assign it to me Toby :)

Awesome @Schleidens!! Have fun!!

How far have you gone with this @Schleidens??

I added the loader it works pretty well. But the Youtube component still taking time to render.
I'm trying another way, i'll reach you in the discord server.
Thank you for taking care @tobySolutions 👐🏽😌

I added the loader it works pretty well. But the Youtube component still taking time to render.
I'm trying another way, i'll reach you in the discord server.
Thank you for taking care @tobySolutions 👐🏽😌

Sure thing @Schleidens!! 💙💙

I added the loader it works pretty well. But the Youtube component still taking time to render.
I'm trying another way, i'll reach you in the discord server.
Thank you for taking care @tobySolutions 👐🏽relieved

Sure thing @Schleidens!! blue_heartblue_heart

Done @tobySolutions you can close this now

No, this issue is still very much open @Schleidens.

I think you're right @Schleidens.
Fixed by #447


User tweets (python)

I am trying to get tweets from a specific user, but my Python-code won't return any tweets. Is there an error in my code?
Example:
tweets = query_tweets(user='usernamehere', begindate='2019,1,1', enddate='2020,1,1', limit=1000, lang='english')
Error code:
TypeError: query_tweets() got an unexpected keyword argument 'user'

Yes, there's an error in your code - the query_tweets function doesn't take a user argument. Instead, you could put the desired username in your query string:
tweets = query_tweets('from:usernamehere', begindate='2019,1,1', enddate='2020,1,1', limit=1000, lang='english')

Yes, there's an error in your code - the query_tweets function doesn't take a user argument. Instead, you could put the desired username in your query string:
tweets = query_tweets('from:usernamehere', begindate='2019,1,1', enddate='2020,1,1', limit=1000, lang='english')

Perfect! Thanks a lot

I also got the error message when I try to get tweets with user profiles. How can i fix it?
Error code:
TypeError: query_tweets() got an unexpected keyword argument 'profiles'

I also got the error message when I try to get tweets with user profiles. How can it be fixed?
Error code:
TypeError: query_tweets() got an unexpected keyword argument 'profiles'

dspencer12 solution above worked for me.
To get tweets from a certain profile do:
query_tweets('from:profilename', ....)

That works for me too:)
Instead of a single user's profile, I am looking for the profile information of users from a queried list of tweets. Like the mentioned section in README.rst :

2.5 Scraping for User Profile information
By adding the argument --profiles twitterscraper will in addition to the tweets, also scrape for the profile information of the users who have written these tweets. The results will be saved in the file "userprofiles_".


Use Result objects to have less exceptions in converters

Add some result objects for when exceptions should be thrown.  This results in a huge speed increase
Messages no longer have a stacktrace though:

https://github.com/specklesystems/speckle-sharp-connectors/pull/456 superceeds this for now


return null for number and percentage if input is empty

To be compliant with the angular input[number], the ui-number-mask
and ui-percentage-mask should parse an empty value to null and not
to undefined (or an empty string). This has an advantage, that performing
calculations with null is possible (e.g. 5 + null === 5). If
undefined or a string would be returned, then the model
has an illegal value and an explicit check for undefined is required
in all calculations...

This would really simplify working with input fields that are not required/mandatory.

Thank you for your pull request.
There is only one thing to do before merging this PR: formatting the commit message using conventional changelog

I hope the commit message is now according to the convention.

It is. Again thank you for this pull request.


Improve Jitify performance

Blocked by my upstream PR NVIDIA/jitify#82, which allows reusing the already-constructed header map (in C++). Also, I eject all input code from this map to only keep headers. Together, this helps reduce the runtime for both the first shot compilation as well as a long-running program (such as running the test suite).

Jenkins, test this please

Jenkins CI test (for commit 93eacc686a62fea09bdb6e4368cc4cd2ab6d4c13, target branch master) succeeded!

The upstream PR NVIDIA/jitify#82 is merged, and the submodule is updated accordingly. This is ready now @emcastillo.

Jenkins, test this please

Jenkins CI test (for commit c586aa141469760c57997c637abeb97c792a6385, target branch master) succeeded!


Add HTMLHyperlinkElementUtils.

https://github.com/purescript-web/purescript-dom/issues/143
This API is required to get the href attribute from an anchor element. I just threw this together to get a concrete proposal for this. I think it's working in my experiment project.

Bump - this one is reviewable again, possibly good to merge.

Sorry, thanks!


use cancellation token for shutdown

This refactors worker shutdown to use a cancellation token to signal shutdown. By doing so, we can avoid managing an atomic bool. It also makes it possible to provide addtional methods, such as a shutdown method, on the worker itself to signal shutdown.

/cc @kirillsalykin I ended up using a cancellation token here instead (Notify could also work, but it's not cancel safe and requires pinning to work as I'd want it).
From here we could add shutdown or similar to the worker. Then the job could wrap that. Likewise with the scheduler I think.
Let me know what you think.

LGTM


fix(graphql-searchable-transformer): formatting domain name

Description of changes

Created a new resource helper function that will generate the domain name for open search domain
Referred to this function in the CDK
Completed the local testing

CDK / CloudFormation Parameters Changed
No parameters have been changed

Issue #, if available
Solves the issue: https://github.com/aws-amplify/amplify-category-api/issues/907

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[x] Relevant documentation is changed or added (and PR referenced)
[x] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[x] Any CDK or CloudFormation parameter changes are called out explicitly

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I think the change makes sense to me. I'll need to investigate if this could have ill effects on existing apps.
There is a failing test on the mock util. cd packages/amplify-util-mock then yarn e2e to see the failure.
Also, can you re-run yarn extract-api? Seems to just be a formatting issue.

The domain name is an attribute which requires replacement, during which process the new search domain with the given name is generated and the old one being deleted as follow up. This is definitely a breaking change for the existing users. Thus this PR will be held until a migration plan is figured out.
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-opensearchservice-domain.html#cfn-opensearchservice-domain-domainname
As a workaround for the linked issue, the amplify override api can be used to change the naming of search domain. The example override.ts is as following.
import { AmplifyApiGraphQlResourceStackTemplate, AmplifyProjectInfo } from '@aws-amplify/cli-extensibility-helper';

export function override(resources: AmplifyApiGraphQlResourceStackTemplate, amplifyProjectInfo: AmplifyProjectInfo) {
  resources.opensearch.OpenSearchDomain.domainName = `amplify-opensearch-${amplifyProjectInfo.envName}`;
}


Parsing model.json

Are there any tools available for parsing the contents of the model.json file? We'd like to create SQL tables based on the data and meta-data found in our CDM folders, but generating the required CREATE TABLE commands based on the meta data is proving tedious without a proper tool to parse the file.

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

ID: ce95c65c-2b6d-cca3-ff37-db6d5bfe818d
Version Independent ID: 2e2cd63e-ae99-811d-3b09-ec3e54b700f2
Content: Metadata format - Common Data Model - Common Data Model
Content Source: common-data-model/model-json.md
Service: common-data-model
GitHub Login: @theresapalmer
Microsoft Alias: tpalmer

Thanks, @ptansey. You might find what you're looking for here.

Thank you for your reply. We had to create our own app using the Newtonsoft.Json library to parse the model.json file.
In the end I asked the wrong question. I now realize we want to know how to read the model.json file and CDM folders with Azure Data Factory. I'll ask that question in a more appropriate space, though.


Add support of password authentication (AUTH)

When a password is configured it will be sent right
after a connection to a Redis instance is setup,
via the AUTH command.
Added API:
redisClusterSetOptionPassword(cc, "password");

Mostly identical to vipshop/hiredis-vip#22


Handle Windows UNC paths as Windows paths

Fixes https://github.com/webpack/webpack/issues/12305
UNC paths, of the form \\hostname\path\to\file are defined in section 2.2.57 of https://winprotocoldoc.blob.core.windows.net/productionwindowsarchives/MS-DTYP/[MS-DTYP].pdf. See also https://docs.microsoft.com/en-us/dotnet/standard/io/file-path-formats#unc-paths, for example.

What kind of change does this PR introduce?
bugfix
Did you add tests for your changes?
I don't know where the tests for these filesystem functions are. I'm happy to add tests if someone points to where they should go.
Also, I'm writing this a bit blind--I don't have a windows system set up to test this. This is very similar to the fix reported to work in practice from https://github.com/webpack/webpack/issues/12305, except it fixes the code given there to check for two backslashes instead of just one.
Does this PR introduce a breaking change?
No.
What needs to be documented once your changes are merged?
I'm not sure anything needs to be documented (except a changelog entry, maybe?)

Let's fix linting

Let's fix linting

Done.

Could use !path.posix.isAbsolute(p) && path.win32.isAbsolute(p). Call to the posix one could be saved to a variable and reused across if statements.

Could use !path.posix.isAbsolute(p) && path.win32.isAbsolute(p). Call to the posix one could be saved to a variable and reused across if statements.

Nice! Could the webpack folks confirm that all of these functions are only called with absolute paths? (I don't want to assume that "rootPath" is always going to be an absolute path without confirmation). Or are you saying that that is true too, that these functions are only called with absolute paths?

!path.posix.isAbsolute(p) && path.win32.isAbsolute(p)

I would prefer the cheap checks over the full check for absolute path.

!path.posix.isAbsolute(p) && path.win32.isAbsolute(p)

I would prefer the cheap checks over the full check for absolute path.

path in node is pretty optimized, check it out: https://github.com/nodejs/node/blob/master/lib/path.js
Yes, it has the cost of validating the parameter as string, but it already handles many cases properly (UNC paths) without the developer having to worry about it.
I wouldn't bet my chips the .startsWith calls here work faster than node's implementation... and there's always the question of readability and maintainability burden that comes with these string assertions.
You could go either way, of course. I personally try to avoid implementing functionality already available to me in Node.

Ok, they look good. Then let's use them

it has the cost of validating the parameter as string

This might get optimized away with v8 inlining and it's knowing that only strings are passed.

For reference these Node functions are defined at posix.isAbsolute and win32.isAbsolute

Thanks @AviVahl for pointing out these functions. I'm also +1 on using the builtin functions rather than rolling our own incomplete checks.

Thanks


Questions about trainting strategy

What are your iteration counts on the CelebA-HQ dataset? Why does my model perform well on the training set but poorly on the test set? Could you please provide your training strategy?

Actually the mask dataset has a huge influence on the outcome. I trained about 400 epochs on CelebA-HQ with a fixed learning rate from scratch, and changed the mask dataset in the middle. At the beginning training with small mask and then turning to huge mask for better generation performance. Some of the masks are generated in a way like LaMa: Resolution-robust Large Mask Inpainting with Fourier Convolutions, you can check that.

Thanks for your answer. Do you use --model 3 directly during training?

Thanks for your answer. You are so kindly~


Old dataset that never successfully "built" appear in search

@zelima commented on Mon Dec 18 2017
If you visit search page and click on around first 10 packages all of them will result in 404. Those are older packages that do not have any successful revisions.
Acceptance Criteria

[ ] no metadata exists in search DB if dataset failed to build
[ ] If dataset is in search clicking it should result with nice showcase page

Tasks

[ ] Identify brocken datasets (make list) - @Mikanebu
[ ] delete from ElastiSearch manually (???)

@zelima could you explain why we end up with metadata in the metastore - if there is a good reason we may want to keep this but just ensure they don't show up in search ...

@rufuspollock I'm not sure I know the exact reason, but I guess at the beginning, even before going alfa, we cleared out some unnecessary data from database for some reasons, but forgot about elasticsearch.

@Mikanebu think we deleted these datasets right? can you double check?

@zelima yes, it is FIXED is this issue https://github.com/datahq/pm/issues/92


Add Ottoman Empire Flag

I think you might be mistaken? This repository does not deal with historical countries, and we use emoji for flags


Different languages in designer and executed application

Expected Behavior
Same languages
Actual Behavior
Different languages
Steps to Reproduce the Problem

VS 2017 English lang.
System (Windows 7 x64 sp3) - system lang. - russian
Add control to WPF app.

Wpf localization use system language (normally) not vs language.  What is your system codepage language ?

Russian.
So i try to set Language property to "en-US"  but it doesn't work

You can force control in constructor by adding a line of code before initialisecomponent()

Or delete Russian language in solution


Simple limit order should not get filled immediately

Issue type

[x] bug

Maybe I'm missing something but so far it looks like a bug
Brief description
I want to create an order at current_price + 100, so that when it goes there it gets filled. But it seems to get filled as soon as I create it and the price is still below.
Steps to reproduce
Full scriot using btc_candle_data.json from the examples folder
import sys
sys.path.append('../')

from hfstrategy import Strategy

ENTERED=False
SUB_TICKER_ID = "tBTCUSD"

# Initialise strategy
strategy = Strategy(
    symbol='tBTCUSD',
    exchange_type=Strategy.ExchangeType.EXCHANGE,
    logLevel='INFO'
)

@strategy.on_enter
async def enter(update):
    global ENTERED
    if not ENTERED:
        await strategy.open_position_limit(0.1, update.price - 100, update.mts)

from hfstrategy import Executor
exe = Executor(strategy,  timeframe='1h')

# Backtest offline
exe.offline(file='btc_candle_data.json')

Additional Notes:
On the plot it adds an arrow assuming the order was executed:

Also we can see from the logs that it gets filled automatically:

amount_filled=0.1

[HFStrategy] [INFO] New Position opened:
[HFStrategy] [TRADE] OPENED Order <'tBTCUSD' amount_orig=0.1 amount_filled=0.1 mts_create=1533859740000 status='EXECUTED @ 6448(0.1)' id=1590337494321>
[HFStrategy] [POS_STATUS] Symbol tBTCUSD | Amount_filled 0.1 | State OPEN | Orders 1
			  Net P/L -0.6448 | Gross P/L 0.0 | Vol<PHONE_NUMBER>000001 | Fees 0.6448
[HFStrategy] [INFO] Setting new exit position: ExitOrder <amount=-0.1 stop=None target=None stop_type='MARKET' target_type='MARKET'>
[HFStrategy] [INFO] Removing position exit order.
[HFStrategy] [INFO] Setting new exit position: ExitOrder <amount=0 stop=None target=None stop_type='MARKET' target_type='MARKET'>
[HFStrategy] [INFO] Position closed:
[HFStrategy] [TRADE] CLOSED Order <'tBTCUSD' amount_orig=-0.1 amount_filled=-0.1 mts_create=1533919680000 status='EXECUTED @ 6374.5(-0.1)' id=1590337494332>
[HFStrategy] [POS_STATUS] Symbol tBTCUSD | Amount_filled 0.0 | State CLOSED | Orders 2
			  Net P/L -1.9197000000000002 | Gross P/L 0.0 | Vol 1282.25 | Fees 1.9197000000000002
[HFStrategy] [TRADE] CLOSED_ALL 1 open positions.

Backtesting complete:

+---------------------+---------+-----------+--------+--------+------+---------------------+---------------------+
|         Date        |  Symbol | Direction | Amount | Price  | Fee  |         P&L         |        Label        |
+---------------------+---------+-----------+--------+--------+------+---------------------+---------------------+
| 2018-08-10 01:09:00 | tBTCUSD |    LONG   |  0.1   |  6448  | 0.64 |          0          |                     |
| 2018-08-10 17:48:00 | tBTCUSD |   SHORT   |  -0.1  | 6374.5 | 1.27 | -7.3500000000000005 | Close all positions |
+---------------------+---------+-----------+--------+--------+------+---------------------+---------------------+

[HFExecutor] [INFO] Net P/L -9.27 | Gross P/L -7.35 | Vol 1282.25 | Fees 1.92
[HFExecutor] [INFO] Min P/L -7.35 | Max P/L 0 | Avg P/L -3.68
[HFExecutor] [INFO] Losses 1 (total -7.35) | Gains 0 (total 0)
[HFExecutor] [INFO] 1 Positions | 2 Trades

Any help highly appreciated, thanks!

I could be wrong - I dont think anyone else is going to answer- with the offline backtester any market order will be filled straight away and there is no slippage - if you try using the data server backtester (bitfinexcom/bfx-hf-data-server) - it should behave as expected.
good luck.
eg:
exe.with_data_server(then, now)

I am also facing the exact problem. Furthermore, I don't even understand the purpose of this library if it lacks the concept of balance/margin and if it does not verify we in fact can issue the requested order (and have enough balance/margin to fulfil it.)
I am confused by how and where this is implemented, if it all.
@dy66 your "exe.with_data_server(then, now)" matches no code here.
And how are we suppose to use the data server if it is implemented in node js, and this library in python?
Is there, anywhere in the Honey Framework, an actual implementation of the order/position manager of bitfinex?


Stop using OS-specific file paths for Shader.load()

The output of os.path.join() (an OS-specific file path) is being passed to Shader.load(). This breaks loading shaders on Windows.

Fixed by 62c45d8e5a8e3c910f98a1942242ae8d5b380e0c


Disabling software write-protect and clearing the WP range   chromebook c630

i'm using the MrChromebox script to install bios

int the  end step ,there is an error:
Disabling software write-protect and clearing the WP range

Error clearing software write-protect range; unable to flash firmware.

i have do something:
flashrom --wp-disable

echo "wp false" > /dev/ttyUSB0
echo "wp false atboot" > /dev/ttyUSB0
echo "ccd reset factory" > /dev/ttyUSB0
gsctool -a -I
crossystem wpsw_cur

but it is not work

i'm using the MrChromebox script to install bios

no, you're not. You're using someone else's fork of my work, as evidenced by the fact that your device isn't recognized, and that it says it's downloading "from wkkun's mirror" - whoever that is.
Run only the commands listed on my site, and if you have issues then come back and re-open the issue. You do not need to run any flashrom commands manually!!!


panic: runtime error: index out of range [0] with length 0

Hey there,
Just installed this on my arch linux system via the following package:
https://aur.archlinux.org/packages/keylight-control
When I try to run 'keylight-control' from the terminal, I get the following error:
panic: runtime error: index out of range [0] with length 0

goroutine 37 [running]:
main.main.func2()
        github.com/mschneider82/keylight-control/main.go:78 +0xca8
created by main.main
        github.com/mschneider82/keylight-control/main.go:63 +0x310

Any insight into what might be going on?

i think it didnt found any lights, thats why it panicd, i added a skip for this kind of failure, but i think you may not find any lights. Is keylight-systray working for you too?
Maybe you have to setup your keylights in the same network (maybe with the mobile phone app first..)
You can compile the new version

do you have libnss-mdns installed?

i think it didnt found any lights, thats why it panicd, i added a skip for this kind of failure, but i think you may not find any lights. Is keylight-systray working for you too?
Maybe should first setup your keylights in the same network (maybe with the mobile phone app first..)
You can compile and try the new version

Hey thanks for the quick reply.
When I start keylight-systray, it shows my keylight listed, and its the correct one that is setup in the mobile app.
But when I try "power toggle all" nothing happens, and when I click on "settings" it crashes.
As mentioned I have had it setup through the mobile app, and its on the same network as my pc.

do you have libnss-mdns installed?

libnss-mdns is not available for arch, but I did install nss-mdns as noted on the readme

Does it work with the new version?

We've made some progress. I have a window open up now, but it doesn't show anything. It is the same window that opens up when I hit "settings" in keylight-systray.
After I close the window I get the following message in the console:
Xlib:  extension "NV-GLX" missing on display ":0".

But I'm not sure whether that's relevant or not.

i still think it doesnt detect the lights
try to follow this: (nsswitch shuold be correct)
https://github.com/mschneider82/keylight-control/issues/1#issuecomment-1047483497
in the mobile app the lights work?

Hey there, just wanted to follow up on this.
I have been tinkering in my free time with setting up mDns. In the end I believe it was a mis-configured nssswitch.conf file. I tinkered with it and it is now working.
For others that may stumble across this post here is what my full nsswitch.conf file looks like:
# Name Ser0vice Switch configuration file.
# See nsswitch.conf(5) for details.

passwd: files systemd mymachines
group: files [SUCCESS=merge] systemd mymachines
shadow: files systemd
gshadow: files systemd

publickey: files

hosts: mymachines mdns4_minimal [NOTFOUND=return] resolve [!UNAVAIL=return] dns files mdns4 myhostname files
networks: files

protocols: files
services: files
ethers: files
rpc: files

netgroup: files


Additional deletions of unused code

Description
Further deletes cruft accumulated from the integration of accord.

Codecov Report
Attention: Patch coverage is 2.77778% with 35 lines in your changes are missing coverage. Please review.

Project coverage is 24.27%. Comparing base (eb4da54) to head (9fee9fb).

Files
Patch %
Lines

...sis/SailDiff/Statistics/StatsCore/Ops/MatrixOps.cs
0.00%
12 Missing :warning:

...s/StatsCore/Distributions/EmpiricalDistribution.cs
0.00%
10 Missing :warning:

.../Analysis/SailDiff/Statistics/StatsCore/Ops/Ops.cs
0.00%
4 Missing :warning:

...StatsCore/Distributions/NonCentralTDistribution.cs
0.00%
3 Missing :warning:

...lysis/SailDiff/Statistics/StatsCore/DoubleRange.cs
0.00%
2 Missing :warning:

...sh/Analysis/SailDiff/Statistics/StatsCore/Range.cs
0.00%
2 Missing :warning:

.../Distributions/UnivariateContinuousDistribution.cs
0.00%
1 Missing :warning:

...Analysis/SailDiff/Statistics/StatsCore/IntRange.cs
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #160       +/-   ##
===========================================
- Coverage   44.14%   24.27%   -19.87%
===========================================
  Files         335      285       -50
  Lines       12712    10342     -2370
  Branches     1782     1402      -380
===========================================
- Hits         5612     2511     -3101
- Misses       6775     7690      +915
+ Partials      325      141      -184

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Function rem-calc() converts em to rem

Expected Behavior

Function rem-calc() should return em unprocessed, so it is possible to use in a loop, where the value is either a px, rem or em.
Current Behavior

Converts em to rem
Possible Solution

Return em value unchanged
Test Case and/or Steps to Reproduce (for bugs)

How to reproduce:

Set value in $header-styles map to 2em

Context

Your Environment

Foundation version(s) used: 6.6.0
Browser(s) name and version(s):
Device, Operating System and version:
Link to your project:

Checklist

[x] I have read and follow the CONTRIBUTING.md document.
[x] This is a bug report or a feature request.
[ ] There are no other issues similar to this one.
[x] The issue title and template are correctly filled.

Hi @petterpetter

Converts em to rem

This is what what it is expected to do.
See https://github.com/foundation/foundation-sites/blob/develop/scss/util/_unit.scss#L20


Sea ice extent indicator

Description

In DACCS we planned a couple of sea-ice indicators. We could have:

Arctic/Antarctic sea ice area
Arctic/Antarctic sea ice extent
Sea ice fluxes

In cases of area we may want to see if we can implement a gridcell-area utility calculation in order to be able to properly average areas via a weighted mean

xref: pydata/xarray/pull/2922

@mathause This looks great.
Do know of a method for calculating grid-cell weight values that would be used? E.g. similar to CDO Grid area weights https://code.mpimet.mpg.de/projects/cdo/embedded/index.html#x1-140001.3.1

I often use cos(lat) which is simple and precise enough (for me).

cos(lat) is a bit dangerous since some global datasets do not use the same spacing for latitudes and longitudes. Map projections are getting more "exotic" so grid cell-area need to be computed properly when not provided by the areacella (atmosphere, land) or areacello (ocean) variables.

Started working on it. The current indicators require that area be passed explicitly. Agreed it would be useful to have a function to reconstruct grid cell areas.

cos(lat) is a bit dangerous since some global datasets do not use the same spacing for latitudes and longitudes. Map projections are getting more "exotic" so grid cell-area need to be computed properly when not provided by the areacella (atmosphere, land) or areacello (ocean) variables.

@cothel any idea how prevalent the areacella, areacello variables would be for CMIP / CORDEX datasets...  Best case would be getting the "official" values directly from modeling centres as much as possible

For CMIP5 I think I have the vast majority of them.

WIP in branch ice

les valeurs pour areacella peuvent se trouver à l'intérieur des fichiers des autres variables ou bien être fournies dans un fichier à part (comme nous le faisons pour MRCC5). C'est une obligation de le fournir. Même chose pour areacello.


Don't leak transactions on explicit commit/rollback

Fixes #409

Looks good to me. Thanks for the fix.


Update Dockerfiles and Vagrant with new public apt repos

Signed-off-by: Ryan Beck-Buysse<EMAIL_ADDRESS>

After this is merged we probably want to blow away the repo at https://repo.sawtooth.me/ if nothing else is using it. Probably wait till after the hackathon though, so people there don't have issues.


Es output docs

enhancement
Release notes

[rn:skip]
What does this PR do?

Exposes required setting for the elasticsearch output when using the plugin
Why is it important/What is the impact to the user?
Without the configurations exposed in this PR an user trying to use the plugin along with an elasticsearch output plugin will fail to ingest to es with the following error:
Could not index event to Elasticsearch. status: 400, action: ["index", {:_id=>nil, :_index=>"...", :routing=>nil}, {"@version"=>"1", "message"=>"...", "aws"=>{...}, "tags"=>[...], "_id"=>"...", "log"=>{...}<EMAIL_ADDRESS>"cloud"=>{...}}], response: {"index"=>{"_index"=>"...", "_id"=>"aXlKQ4sBCqmxnW6hpvO3", "status"=>400, "error"=>{"type"=>"document_parsing_exception", "reason"=>"[1:331] failed to parse field [_id] of type [_id] in document with id 'aXlKQ4sBCqmxnW6hpvO3'. Preview of field's value: '...'", "caused_by"=>{"type"=>"document_parsing_exception", "reason"=>"[1:331] Field [_id] is a metadata field and cannot be added inside a document. Use the index API request parameters."}}}}

Checklist

~- [ ] My code follows the style guidelines of this project~
~- [ ] I have commented my code, particularly in hard-to-understand areas~

[x] I have made corresponding changes to the documentation
~- [ ] I have made corresponding change to the default configuration files (and/or docker env variables)~
~- [ ] I have added tests that prove my fix is effective or that my feature works~

Author's Checklist

[ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

@yaauie I've committed your suggestions, please review, thanks :)

@karenzone your suggestion were welcome :)
please check if everything is fine

Here's what the new info looks like in my local build after formatting tweaks:

@yaauie @girodav Did you have additional comments?  @aspacca has given me the go-ahead to bump versions and merge.

@logstashmachine bump patch

@aspacca Thanks for your work on this. I assigned myself to get these reviews on my project board for tracking.

Published to rubygems.org: v0.1.4

@karenzone when v0.1.4 will be published on https://www.elastic.co/guide/en/logstash-versioned-plugins/current/input-elastic_serverless_forwarder-index.html? thanks :)

Most likely, today. Stay tuned! :-)

@aspacca, here it is: https://www.elastic.co/guide/en/logstash-versioned-plugins/current/v0.1.4-plugins-inputs-elastic_serverless_forwarder.html.  Thanks for all of your great work on this.