minor - update packages

:tada: This PR is included in version 3.0.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Issue related with "talkative" word

There some issue with this word ?

@Marlysson: What do you mean, exactly?

Nothing @pvdsp , already I solved :D


Repeat icons modification

Now the repeat icons are vertically aligned.

U da real MVP

Ahaha thanks @KeitIG ♥


Create Play Store Account

Google Play Developer account has been opened. I invited @dburka, @pratul, and @saket.

The final decision on the package name is org.simple.clinic.


Terminal error message: Access denied

Brief description of your issue
I know this bug has been posted in issues before this, but I still haven't solved my problem after scrutinizing those issues, and my winget still doesn't work, so I'm reopening a new issue, so I hope you guys can understand me.
My current version of winget is "1.21.3482.0", which is the latest version of winget, and I have manually added "%userprofile%\AppData\Local\Microsoft\WindowsApps" to my system environment variables. But I can't use any commands of winget in terminal, it will tell me that access is denied.
My Windows version information is as follows:
Version: Windows 11 Professional
Version: 23H2
OS version: 22631.3227
Experience: Windows Feature Experience Pack 1000.22687.1000.0

Steps to reproduce
$ winget

Expected behavior
I don't know.
Actual behavior
程序“winget.exe”无法运行: 拒绝访问。所在位置 行:1 字符: 1
+ winget
+ ~~~~~~。
所在位置 行:1 字符: 1
+ winget
+ ~~~~~~
    + CategoryInfo          : ResourceUnavailable: (:) [], ApplicationFailedException
    + FullyQualifiedErrorId : NativeCommandFailed

Environment
Name              : Microsoft.DesktopAppInstaller
Publisher         : CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US
Architecture      : X64
ResourceId        :
Version           : 1.21.3482.0
PackageFullName   : Microsoft.DesktopAppInstaller_1.21.3482.0_x64__8wekyb3d8bbwe
InstallLocation   : C:\Program Files\WindowsApps\Microsoft.DesktopAppInstaller_1.21.3482.0_x64__8wekyb3d8bbwe
IsFramework       : False
PackageFamilyName : Microsoft.DesktopAppInstaller_8wekyb3d8bbwe
PublisherId       : 8wekyb3d8bbwe
IsResourcePackage : False
IsBundle          : False
IsDevelopmentMode : False
NonRemovable      : True
Dependencies      : {Microsoft.UI.Xaml.2.7_7.2208.15002.0_x64__8wekyb3d8bbwe, Microsoft.VCLibs.140.00.UWPDesktop_14.0.3
                    3519.0_x64__8wekyb3d8bbwe, Microsoft.DesktopAppInstaller_1.21.3482.0_neutral_split.language-zh-hans
                    _8wekyb3d8bbwe, Microsoft.DesktopAppInstaller_1.21.3482.0_neutral_split.scale-100_8wekyb3d8bbwe...}
IsPartiallyStaged : False
SignatureKind     : Store
Status            : Ok

@JohnMcPMS @yao-msft @ryfu-msft @msftrubengu @Trenly @denelon @florelis @jedieaston @ashpatil-msft @AmelBawa-msft @jsoref @sreadingMSFT @mapill-msft @KevinLaMS

@SobreroD @Scarjit @bckelley

Don't......don't do that....

Don't......don't do that....

This bug that I can't use winget still exists at the moment, can you please tell me how to fix this bug.

@nianBroken Just so you know, when you @ mention people, they get an e-mail.
Here are a couple of additional options:
Check you App Execution Alias

Have you gone through the troubleshooting steps on the troubleshooting page?


Is skorch.Dataset needed?

I have several questions regarding skorch Dataset.
First, the documentation says:

However, the Datasets provided by PyTorch are not sufficient for our usecase; for instance, they don’t work with numpy arrays.

As far as I can tell, this is no longer the case. A commit introducing numpy array support landed in PyTorch master more than a year ago, and the code has matured since then (see it's current state).
Second:

skorch Dataset must have a device argument, which determines whether the returned data should be transferred to a specific computation device

Why is this needed? Both NeuralNet.infer() and NeuralNet.get_loss() explicitly call to_tensor() to transfer input and target to self.device.
Third:

Dataset applies a transform [...]. By default, it casts the data to a PyTorch Tensor and replaces y by a dummy variable in case it is None.

Again, the default_collate function of PyTorch already does this. However, it's true that it does not handle None.
With all of this in mind, I wonder if skorch Dataset is needed? If the ability to convert None to a dummy variable is the only value added, then maybe it's easier to just add this to PyTorch Dataset and drop skorch Dataset? As a PyTorch/Skorch beginner I should admit that despite detailed documentation, the existence of two is still confusing.

You're making good points. If we had the choice to get rid of our custom Dataset, we would gladly take it, since our goal is to provide as little friction with PyTorch as possible.
That being said, there are still data types that we would like to support and that wouldn't work otherwise. For instance, we often have X as dictionaries or DataFrames. The magic is in the multi_indexing part. You are right that we should highlight this better in the docs.
Regarding the casting to tensor, we probably really don't need it anymore. This is a relic from the time when PyTorch had tensors and variables. I'll open an issue for that.
Finally, I think there is value in having a transform method on Dataset after all the business with indexing is done. This way, a user has a dedicated transform method (instead of having to deal with everything in __getitem__).

we often have X as dictionaries or DataFrames.

PyTorch handles dictionaries, right? There is definitely no branch for pandas, but again, if this is (one of the few) raison d'être for Dataset, wouldn't it be easier to merge this upstream?

PyTorch handles dictionaries, right

It's not about the DataLoader; none of PyTorch's Datasets can deal with dictionaries, DataFrames or None.

wouldn't it be easier to merge this upstream?

Ideally, yes, but I suspect that this isn't high on the priority list of the developers.

It's not about the DataLoader; none of PyTorch's Datasets can deal with dictionaries, DataFrames or None.

I do realize that the functionality to convert numpy arrays and dicts is in PyTorch DataLoader, not in Dataset. Perhaps I'm missing some important point, but I don't understand why would we want this conversions to be a part of the Dataset class given that the data are anyway passed through DataLoader before being consumed by the network.

Say we have an X like this:
X = {'key0': numpy_array0, 'key1': numpy_array1}
At the critical point, _DataLoaderIter does this:
batch = self.collate_fn([self.dataset[i] for i in indices])
This would raise a KeyError instead of indexing into the numpy arrays. I don't see how we can make this work except by having a custom Dataset that knows how to index into the values of the dictionary.
We're open to suggestions how to treat this differently. The tests for what we expect should work are there, if you find a better way, make a PR. But let me warn you, the DataLoader code is very messy :)

Thanks for taking time to explain, now this makes sense.


Regression in cull_opacity_perf__timeline_summary 90th_percentile_frame_build_time_millis & other benchmarks

also affects the cull_opacity_perf__timeline_summary 90th_percentile_frame_rasterizer_time_millis, 90th_percentile_vsync_transitions_missed, 99th_percentile_vsync_transitions_missed, average_vsync_transitions_missed.
Very clear signal at engine roll https://github.com/flutter/flutter/pull/56162.

Looks like flutter_gallery_ios32__transition_perf 90th_percentile_frame_rasterizer_time_millis is affected too.

I confirmed locally that the regression is caused by https://github.com/flutter/engine/commit/4e2973633d6b8e0bf0c26b33fa61254e09fd1384. Will revert that in the engine and wait for the engine roller to fix this regression.

The revert has been rolled in https://github.com/flutter/flutter/pull/56378


[fix] setTimeout example

setTimeout returns timeoutId which will be saved as state

When you use arrow function with body (a, b) => { ... } nothing will be returned by the function, to return something you will need to use an explicit return statement.
So, this action return nothing, that will not result at the merge issue you are referring.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions#Function_body

@Swizz that's right. Thank you


Fix md output format

Fixes: #660

need to release the updated package after this push,As of now current previous version output is unusable.


[rv_dm,dv] fix typo in seq

This typo causes dv smoke test failure.
Not sure how this slipped into the repo.

I'm terribly sorry: this was my fault. I merged #21192 and didn't notice that it had failed CI.

For @BasitMehmood: The change that broke things come from your PR #21192. I'm feeling silly for merging it without checking CI. But please create PRs without some basic testing at your end: I don't believe the code could have worked on any simulator.

I'm terribly sorry: this was my fault. I merged #21192 and didn't notice that it had failed CI.

Good to know it catched it. My other PR had another ci fail in similar way :(


Remove unsupported rules from rule_exists

There is a list of rules in rule_exists and only one of these is actually supported by the rule engine plugin: pep_api_data_obj_get_post. The others should be removed to prevent a SYS_NOT_SUPPORTED error in exec_rule as well as to ensure the proper execution of the post-pep should another configured rule engine support it.

The "unsupported" rules are actually needed for the storage tiering pluging to work. The real problem is that we are returning a SYS_NOT_SUPPORTED for all operations except pep_api_data_obj_get_post in exec_rule even though there is just a special case for that operation in which restaging to a lower tier occurs due to data access.
The error returned should not be an error because it is not necessarily handling an unsupported operation: https://github.com/irods/irods_capability_storage_tiering/blob/19c229a2e62af29db34c5ed97a092ed4260bddba/libirods_rule_engine_plugin-storage_tiering.cpp#L85..L88


feat(fingerspelling): add continues fingerspelling animation

Fixes #39
For American Sign Language, based on https://fingerspell.net/, I added the transitions for fingerspelling.
The old version (all letters, then a word: "journalizations")
old.webm
journalizations
The new version (all letters, then a word: "journalizations")
new.webm
journalizations
I kind of like the a-z in the old version more, but spelling a word is nicer in the new version.
What do we like more?

I cannot tell what the exact difference is: did you make the transition smoother and the overall speed quicker?

the second "journalizations" is a bit confusing for me because it looks like starting with an "i"

I cannot tell what the exact difference is: did you make the transition smoother and the overall speed quicker?

I made it so there are only transitions, and each possible transition is coded (A->B, A->C, ....) - that way the spelling is smooth, in place, and faster.
Previous version treated every character as separate a sign, and concatenated them. transitions are generated on the fly


Add rsqrt method to Float trait

rsqrt is a widely used math function in game development, and is faster than combining the two functions: recip(sqrt(x)) thanks to x86 SSE instructions RSQRTSS, RSQRTPS, RSQRTSD, and RSQRTPD. Should we consider add rsqrt to Float trait?

x86/x86_64 SSE: _mm_rsqrt_ps (note that RSQRT instruction is approximate, less accurate than SQRT)
ARM Neon VRSQRTE
PowerPC Altivec: vec_rsqrte

This should probably be proposed for the standard library first.  So far we don't have any arch-specific code -- I'd rather just have num's impl Float for f64 forward to an optimized/intrinsic call in std.

I agree; it makes sense to offer this in libstd.

Reasonable, I'll propose it in rust-lang/rust.
I'll keep the issue open as reminder.

This issue was moved to rust-num/num-traits#1


Support multiple Gerrit instances per JIRA server

Like the plugin, but 1 Gerrit instance per JIRA server is too limiting -- it would be great if we could connect a JIRA instance to multiple Gerrits.

Is this plugin abandoned? It looks promising, but the lack of multi-gerrit-server support means it won't work for my environment.

@bekriebel it's not abandoned, this feature is just not something that my company has a need for, and so it has not been high on my priority list.
There have been other people who have shown interest in adding support for that, but I do not know the status of those integrations (as no pull requests have been created for it).  Looking at the network graph https://github.com/MeetMe/jira-gerrit-plugin/network, I see only one fork that appears to have the intention of creating multiple-server support (by @barisinanc), but it does not look like any progress has been made yet toward that goal.

Thanks, @jhansche. I may be trying to implement it soon and will submit a pull request if I can get it to work well.

@bekriebel, Might want to hold off for a couple days. I just received an email today that there is a working implementation of this in another fork. Once that is cleaned up and tested further, he plans to submit a pull request for the feature.

@jhansche @bekriebel I made a fork for multiple instances and added to my todo list But currently I have no time for it.

@jhansche Have you heard any more on the status of multi-instance support? I looked through the fork list and don't see a public fork that has it implemented.

It was someone else, not someone participating in this ticket.  Last I heard, he got the green-light from his company to contribute the patch back to the project, so as far as I know he's just preparing the patch and should be submitting a pull request when it's ready.

@jhansche I hope this guy will get the approval! Would be useful.

@jhansche i'm also interested by that feature.

@jhansche is there any news from this patch?

Hi All
Still working on code review and hopefully will be able to publish it in near future.
Regards,
Areg

Aha, so he is participating in this thread!  Good to know :+1:  ;)
I will be happy to merge it once it's ready.
Thanks again @vrtareg for taking this on!

In case you're not aware, the gerrit-trigger plugin has multi-server support, so it might be worth aligning said code with their implementation so that perhaps such common functionality can be shared via a common library.

@electrofelix this plugin already makes use of the gerrit-events artifact from the Jenkins Gerrit Trigger plugin (what I assume you're referring to).  That does not magically mean that the multiple-server support is automatically inherited.

@jhansche aware of that, however I believe that it would require a refactoring of the gerrit trigger plugin to turn that piece into a separate artifact. It might however be easier for it to happen if  the code is being reused elsewhere.

Hi,
are there any news or a "time horizon" for this feature to handle more gerrit-server?
Thx for reply

@vrtareg Any updates on this?

Hi
Unfortunately no. We are trying to sort out some internal procedures.
I can publish some information what we have done so far.
Regards,
Areg
On Wed, 4 Nov 2015 9:01 pm Joe<EMAIL_ADDRESS>wrote:

@vrtareg https://github.com/vrtareg Any updates on this?
—
Reply to this email directly or view it on GitHub
https://github.com/MeetMe/jira-gerrit-plugin/issues/2#issuecomment-153864126
.

--
Regards,
Areg

@vrtareg does that mean that you won't be able to contribute what you have done?  Or you just need more time?
Either way, yes I think having some additional information about what you've done so far, in case someone else chooses to jump in, would be helpful.  This is by far the most requested feature, and I haven't had the time to work on it myself. :(

Hi @jhansche
I hope that we will be able to sort it out.
Yes we need a bit of time to have a resource to finalise this.
Regards,
Areg

Hi @vrtareg, any update?  I've been contacted by several other large companies looking for this support, but I've been holding off (and telling others to hold off), because I know that you have already completed the implementation.

My company has also the need to integrate JIRA with multiplt Gerrit servers. I have been following this thread for some time now and am hoping for some progress. I think this is a normal use case that there be be multiple Gerrit servers used within a company. Therefore adding this support would be welcome.

@vrtareg bump -- any news?
@mlarborn it's interesting to me that so many people have this workflow.. In my company, there's just one gerrit server for all of engineering.  The netops/sysops/devops team has their own gerrit server as well, but I never understood why, since the master gerrit server should work fine even for them.
It would probably be worth investing some time refactoring the plugin for better support (maybe even cloud support, but that would might be difficult since I don't know what kind of sandboxing JIRA cloud does). The problem is I don't really have the time to do that during the day due to $DAY_JOB

Hi
Really no news.
I can’t do much as my hands are tied with legal process which is still in place to check the code to publish.
I will try to put together the list what we did so far.
Regards,
Areg

Hi,
are there any news  for this feature to handle more gerrit-server, we are very interested, too?
Thx for reply

Interested in this feature. any updates?

Also interested in this. Would be really cool, as the stock Gerrit hook tends to spam our comment feed with Gerrit updates. (I believe its-jira works the same way too).

I am wondering if we'll ever get some updates from @vrtareg after soon to be two years since he announced that he has some code.

Hi
It is working in our internal version of the PlugIn which is based on this code with our changes.
Unfortunately there are the legal problems to share the code.
We are hired external developer to polish changes and possibly will be working to get legal work done to push changes back.
I really don't know how long it can take.
Regards,
Areg

@vrtareg Thanks for the update. It seems that I am luckily to work for an employer that fully embraces open-source and where I don't need to involve legal doing small contributions to open-source projects. That an assumed part of my job, like sending an email to someone outside :)

Hi.
Joining the question. Multiple gerrit support is a nice feature to have. Are there any good news?

any good news on the horizon?

This would be a great feature to add and there is definitely a demand for it just purely based on the amount of comments here.
@vrtareg is there perhaps any update from your Legal Dept to the work that was done by you & co?
If not, is there perhaps anyone else here that has a solution that works?
Looking forward to some feedback

hi,
I am watching this since many months :-(.
We also have more than 1 gerrit server, so for us this is a critical requirement.
Isn't there any news about this feature?

so a greate feature,I want that too.Is anybody to achieve it?

Hi,
Is there any news regarding this feature?


Changes to the reference section of the TOC to create a new section called Creator indoor maps and move creator articles into this new section.

TOC Updates to Reference section:
Reference

Remove all next steps
Add a top level section called “Creator Indoor Maps”

Move drawing-error-visualizer.md under “Creator Indoor Maps” from how-to section

Move drawing-requirements.md under “Creator Indoor Maps”

Remove all next steps except the tutorial

Move schema-stateset-stylesobject.md under “Creator Indoor Maps”
Move creator-facility-ontology.md under “Creator Indoor Maps”

Other Tasks

Add a learn more to every article

Learn more by reading:

[!div class="nextstepaction"]
Creator for indoor maps

@anaharris-ms
@rbrundritt
@DCtheGeek
@eriklindeman
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


[Warhammer 4e Character Sheet] - More fixes in talents and localisation improvements

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Fixed visibility of Strider Talent
Divided DISTRACT into DISTRACT-TALENT and DISTRACT (item quality) due to localisation problems

Character Sheet Info Roll20 Internal Use only.

@Djjus Found some more bugs and create more localization tags because DISTRACT has two polish translations depending on context.

Great. I will look at adding logic to the now visible Talents, also Beat Blade is not on list yet either.
One small correction to the previous update, remove the max="1" for Wealthy Talent if you can.

Great. I will look at adding logic to the now visible Talents, also Beat Blade is not on list yet either. One small correction to the previous update, remove the max="1" for Wealthy Talent if you can.

Ok, also you can click on "Changed Files" and comment on my changes in specific lines so would be easier for me to locate what to change. If you "review changes->request changes" merging will be blocked so no one will accidentally merge code with errors.

Cool. Thanks for your contributions, really apricate it. (I'm not Mad! ;))


Night mode in Mod Mail Pro does not get saved

When selecting Night mode in Mod Mail Pro in the settings sets the modmail to night mod. Despite the display box saying the setting is saved and verified, mod mail opens up without night mode enabled and you have to go to the settings again to set it.

Try setting it from reddit proper for now.


where is command line now?

Steps to reproduce
vim --clean -c 'set ruler'
// pls check the bottom of the window, where is the command line now?

Expected behaviour
a regression, the cmd line lost, wish be back soon.
Version of Vim
v9.0.669
Environment
linux
xfce-term
Logs and stack traces
No response

What's wrong? Everything seems fine.

cmd line should be under 'ruler' (stl) line when cmdheight is 1 (which is default).
but now it is same line.

That depends on the value of 'laststatus'.

cmd line should be under 'ruler' (stl) line when cmdheight is 1 (which is default). but now it is same line.

ruler combines the cmdline and statusline-like info into a single line. So it looks ok for me.
...
If a window has a status line, the
ruler is shown there.  Otherwise it is shown in the last line of the
screen.
...


Regridding functionalities (powered by xESMF)

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR addresses issues #68 #180 #182 #215 and follows up #196
#141 #168 will have to be addressed again at a later point

[x] Tests for the changes have been added (for bug fixes / features)
[x] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes) -> Will be added in a release preparing PR
[x] I have added my relevant user information to AUTHORS.md

What kind of change does this PR introduce?:

Extending #225 for all data_vars and coords of the xarray object (requires cf-xarray >= 0.7.5)
Adding jupyter notebook to showcase remapping functionalities
Adding regridding functionalities (powered by xESMF):

clisops.ops.regrid.regrid one line remapping function orchestrating below functions and classes
clisops.core.Grid class:

create xarray.Dataset holding description / coordinates of a regular lat-lon grid in CF compliant format:

from xarray.Dataset/DataArray (optionally of adaptive resolution, if source is not a regular lat-lon grid)
via grid_instructor (creating regional or global grid)
selecting a pre-defined grid (https://github.com/roocs/roocs-grids)

reformat grids (SCRIP, CF, xESMF formats)
detect extent, shape, format, type of the grid
detect collapsing or duplicated cells
create hash, compare grid objects
save to disk
exchange attributes and non-horizontal coordinates between datasets
calculate bounds (for regular  lat-lon and curvilinear grids)
re-define data_vars and coords of xarray.Dataset for xESMF application

clisops.core.Weights:

orchestrate creation of weights with xESMF
holds xesmf.Regridder object
read from or store to local remapping weights cache (lock file mechanic for contemporary weights creation)
generate hash to identify similar weights

clisops.core.regrid:

application of remapping weights on xarray.Dataset/DataArray
optionally transfer attributes and non-horizontal coordinate variables from source to target dataset
set new attributes related to the remapping operation

Adding lock file mechanic (from source)

Does this PR introduce a breaking change?:

adding regrid operator in ops
adding regrid function, Grid and Weights classes to core
the remapping makes use of xesmf, which is already a dependency
adding dependency for roocs-grids (might be removed with resolving #168)
potentially TBA

Other information:

Future planned PR(s) specific for remapping:

Support for manually provided masks
Support for out-of-domain masking for nearest neighbour (likely better put to xesmf)
Deal with the periodic attribute of xesmf (xesmf resets periodic to False for conservative remapping, but probably should not)
Support datasets with shifted longitude frames (eg. ranging from (-60, 300) degrees_east) - base work already done in previous PR
Support reformatting from/to further formats
Support grid type detection for other formats
Support reading / reformatting / using weight files from other tools like nco, cdo, ...
Calculate nominal_resolution of the target grid, if not present
Set up central / web based remapping weights cache and synchronizing cron job
Find solution for vector variables / variables defined on cell edges
Support vertical interpolation (eg. with xgcm)
(Support other remapping backends than xesmf in the far future)

Future planned PR(s) generally for clisops:

Unify the detect_coordinate functions
Attribute a new tracking_id / PID (general requirement for clisops)
Support datasets with missing missing_value / _FillValue attribute that feature missing values (add fix in dachar?)

Hey @sol1105, I'm wondering what the state of this PR is? Do you need reviewers?

Hey @sol1105, I'm wondering what the state of this PR is? Do you need reviewers?

Hey @Zeitsperre, I'm sorry I wasn't able to continue to work on it in the past weeks, I try to get it into a reviewable state soon. Thanks for keeping it up to date with the master branch all this time :-)

@Zeitsperre @cehbrecht @agstephens I did all the changes I wanted to make to the PR. It would be great, if you gave me some feedback. It is not urgent, however. In the background I will fix the tests that are no longer running through.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

604 of 955 (63.25%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-6.6%) to 72.396%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

clisops/ops/base_operation.py
23
30
76.67%

clisops/utils/common.py
14
22
63.64%

clisops/utils/output_utils.py
24
37
64.86%

clisops/utils/dataset_utils.py
133
165
80.61%

clisops/ops/regrid.py
20
57
35.09%

clisops/core/regrid.py
388
642
60.44%

Totals

Change from base Build<PHONE_NUMBER>:
-6.6%

Covered Lines:
1689

Relevant Lines:
2333

💛  - Coveralls

@sol1105
I wanted to know if you're still interested in finalizing the regridding implementation? I've adapted your PR to the newest package layout and will be seeing what I can do about roocs-grid (i.e. PyPI). Please let me know!

@Zeitsperre @huard Thank you for all the help and feedback so far. I know there is still quite a lot of work to be done with regards to the remapping functionalities, but despite that I would like to merge this PR, so there can soon be a clisops release with regridding capabilities. From there I would keep on working on the regridding functionalities (incl. working on your above suggestions) in future and smaller PRs.
Would you be ok with this or would this conflict with your use case or plans with clisops? Is there anything crucial for me to do so a merge is possible?
Problematic for your use case may be that two bugs (pydata/xarray#7794 and xarray-contrib/cf-xarray#442) prevent using the more recent xarray and cf-xarray releases. Another topic was that the roocs-grids repo (holding a set of pre-defined target grids) should be released as pypi and conda package before merging this PR.

Hi Martin, works for me !

@Zeitsperre @cehbrecht  I removed python 3.8 from the CI checks because of the cf_xarray problem you adressed in roocs_utils using the following lines in the requirements.txt:
cf-xarray>=0.3.1,<=0.8.4; python_version == '3.8'
cf-xarray>=0.3.1; python_version >= '3.9'

I am not sure how such a setting can be added in the dependencies entry of the pyproject.toml, in case python 3.8 is still required. Would it simply be:
dependencies = [
  "bottleneck>=1.3.1",
  # cf-xarray is differently named on conda-forge
  "cf-xarray>=0.8.6;python_version>='3.9'",
  "cf-xarray>=0.7.5,<=0.8.0;python_version=='3.8'",
...

@sol1105

I am not sure how such a setting can be added in the dependencies entry of the pyproject.toml, in case python 3.8 is still required. Would it simply be:
dependencies = [
  "bottleneck>=1.3.1",
  # cf-xarray is differently named on conda-forge
  "cf-xarray>=0.8.6; python_version>='3.9'",
  "cf-xarray>=0.7.5,<=0.8.0; python_version=='3.8'",
...

That's exactly what you would need to do to maintain Python3.8 support. Everything is using pip under-the-hood. The only thing I'm not certain about is that I believe cf_xarray up until v0.8.4 is compatible with Python3.8 (at least, that's what I indicated in other places).

@Zeitsperre I leave the last words to you :) We would like to merge this PR and make the new clisops release with the regrid operator. I have prepared daops and rook already for the new operator and it works.

@Zeitsperre Thanks a lot for the review and your contribution.  I addressed the outstanding issues you found

Well done. Feel free to merge whenever you're ready!


Feature/add crate io version badge

Adds an automatically updated version badge for this library (see the README as it appears on my fork)
Badge:

Thanks!


add anndataset class

Codecov Report

Merging #52 into master will increase coverage by 0.57%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
+ Coverage   88.58%   89.16%   +0.57%
==========================================
  Files          31       32       +1
  Lines        1367     1394      +27
==========================================
+ Hits         1211     1243      +32
+ Misses        156      151       -5

Impacted Files
Coverage Δ

scvi/dataset/anndata.py
100% <100%> (ø)

scvi/dataset/__init__.py
100% <100%> (ø)
:arrow_up:

scvi/train.py
90.24% <0%> (+0.81%)
:arrow_up:

scvi/metrics/stats.py
100% <0%> (+5.97%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix error EmptyStaticCreds: static credentials are empty

Extract credentials from "default" section of ini file

Thanks

@delfer -- I ran into a few more case in my own world.  Ended up redoing some of the session credential fixes.  Would you mind given the current master branch another test to make sure it's working for you still?

@koblas now I found https://github.com/minio/mc and it's very very nice!


FragalysisFactory for generation of prep input Target and Ligand Schema from Fragalysis

FragalysisFactory for generation of prep input Target and Ligand Schema from a fragalysis file system
This will looks something like the below in use.
from asapdiscovery.data.schema_v2 import *

targets: List[Target], ligs: List[Ligand] = FragalysisFactory(fragalysis_path)

OR we can also combine these into a combined schema, which may be neater.
from asapdiscovery.data.schema_v2 import *

complexs: List[Complex] = FragalysisFactory(fragalysis_path)

@kaminow would this be of interest to you / would you have bandwidth?

yeah definitely! I’m at ICML this week though and won’t be super productive, so if this is going to block other stuff then it might be better for someone else to take it


Fix ShardedDataParallel has no attribute require_backward_grad_sync

What does this PR do?
Fixes #6804
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Hello @awaelchli! Thanks for updating this PR.

In the file tests/plugins/test_sharded_plugin.py:

Line 304:5: E101 indentation contains mixed spaces and tabs
Line 304:5: W191 indentation contains tabs

unrelated (?) tests are failing now for unknown reason :(
it was passing earlier this afternoon
it's very surprising to me


Adds interface for sim to Node

Adds an interface for the simulators into Node.

It might be worth making a special API package where you don't need to actually include the core simulation code, so that other mods know how to interface with stuff properly. Thing is, that API would also need to present MNA primitives because we currently don't really have a great abstraction layer for when that isn't handled by the MNA we have; it's relatively tightly implemented and we may need another layer here.

Cameron and I talked about this and we think that we want to change how this abstraction happens. Going to rewrite.


code command not creating a file in wsl2

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

Open up vscode in wsl2
code somefile.txt

The code command does work for opening up a directory, but not creating a new file.

I am facing the same issue. This in fact doesn't work for any remote server in general. I was having this problem while using ssh on vscode

Same here using ssh remote server
Version: 1.62.3 (system setup)
commit: ccbaa2d27e38e5afa3e5c21c1c7bef4657064247
Date: 2021-11-17T08:11:14.551Z
Electron: 13.5.2
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.18363

The code command in integrated terminal does open up folders or existing files, but cannot create and open new files.

I can reproduce.

run out of source
open test resolver
in the integrated terminal type code.sh|bat newFile.txt
nothing happens

Debugged it down to
https://github.com/microsoft/vscode/blob/3a81c25c7abd5b1ebf76fb10e8b3b7a27aef9000/src/vs/workbench/common/editor.ts#L1157
The type is undefined as the file does not exist.
Change got introduced by https://github.com/microsoft/vscode/commit/e31a18ea8c6cf53e205e481c98f290e6d8c20460
FYI @lszomoru @bpasero

Thanks @aeschli for the repro steps. Will take a look at this today.


Seasons dropdown should only have relevant seasons on each page, or have an empty page not a 404

Perhaps a different visual treatment of the seasons where there are/are not entries.

This would involve querying the other seasons.  Potential perf issue.

15eb6a21 puts in "empty page" for archetypes. We already have it for people and cards. I think we're done here. We'll put all seasons on the dropdown but have a sane response if you click a season where (for example) there were no Necro Control decks.


Abort if current and target Node versions are the same (closes #78)

I'm trying to handle #78.
The implementation is very simple and probably doesn't cover edge cases.
But it works for me.
I'd be glad to expand upon this, but first wanted to get some traction going.
Also some tests are failing when I run them locally, not sure what's the deal with that.

@wbyoung Hope you're still active here.

I understand the point of wanting to optimize this, but I'm reluctant to make this change for a few reasons:

The process of activating a version of node could change something besides just simply the version number (I thought I left this comment somewhere before, like on #78, but guess I forgot when that was first opened). For instance, it could change things about which global npm modules are available (similar to RVM gemsets).
The file may not be just plain text and may not stay that way in the future (see #10).
This doesn't seem to be the right solution to solving a performance problem. The performance problem should be addressed rather than adding completely to this system.
Shell support & fragility is being increased by adding this complexity. One motivation of the current design is to keep as much as possible written in Node to avoid shell specific issues.

If we can resolve these concerns, then I think this is a go. But I'm skeptical at this point.
Also, I honestly don't use this project any more. I've tried to find time to finalize converting it to Babel & more modern JS, but haven't even completed that. So if you're interested in contributing, someone helping tackle and maintain complexity could help push things like this forward.


Cache style file content to improve ssr performance

getInlineStyleTags(extraProps = {}) {
    const mainAssets = this.getMainAssets('style')
    const promises = mainAssets.map(asset =>
      assetToStyleTagInline(asset, extraProps, this).then(data => data),
    )
    return Promise.all(promises).then(results => joinTags(results))
  }

🚀 Feature Proposal
getInlineStyleTags use node fs.readFile to get style file content every time a request comes. Can we cache style file content to improve ssr performance?
Motivation
Improve server render performance.
Example
Cache style file content into memory.
Pitch
Why does this feature belong in the Loadable Component ecosystem?
User of loadable component has no way to customize or rewrite getInlineStyleTags logic.

OS level file cache should work as well.


Non-finite internal branching

We currently restrict internal branching to finite branching, indexed by Fin types.
This is a clear restriction, and an immediate blocking point in experimental applications investigated by @nchappe.
Two possibilities:

Allow for indexation by either a Fin type, or by nat
Allow for indexation by an arbitrary type. In this case, the indexes used by a given computation must become a parameter of the datatype, similar to E for external interactions.

Exposing statically the indexes used in the second solution is mandatory if any notion akin to interpretation of internal non-determinism is to be defined in the future (and it shall!).

See the branch choice-gen by @nchappe which addresses this issue.
We plan on merging it to master eventually, but probably not before a first 1.0 without it is stable.


Write Java Unit Test for: DELETE /api/organization/delete/{id}

Assign task on controller, not on opearation


Add GtkImageMenuItem support

You should open issues at https://github.com/gtk-rs/gtk-rs instead. And GtkImageMenuItem was deprecated in gtk 3.10 while gtk-rs supports only gtk 3.14+

You should open issues at https://github.com/gtk-rs/gtk-rs instead. And GtkImageMenuItem was deprecated in gtk 3.10 while gtk-rs supports only gtk 3.14+


There is some problematic content in the datalake

While browsing the datalake snapshot I found some entries that contain explicit descriptions of sex between children. All of them share the submitter id of c75e6147-2b47-4273-9edc-0209c986751d.

I've stumbled upon this content too, while i was browsing the datalake visualization on https://gpt4all.io/index.html


Ampersand problem

My project is in UTF-8 (hopefully, not sure how to check this.)
As far as I can see,  all german characters are fine but there is a problem when displaying an ampersand. Please see the attached screenshots.
Pic 1) I create some content with a bladewind input, like Miller & Sons
Pic 2) in my grid and also in the database (using Table Plus) it looks good
Pic 3) when displaying the same content for editing it's wrong
Pic 4) same problem when trying to select that record

@mkocansey
please see my comment on your commit. Thank you.


Remark + Hugo

Hi everyone,
I've just made some remark.js templates, which can be accessed via Hugo shortcodes.
Demo:
https://eueung.github.io/hugo-remark-minion/
Repo:
https://github.com/eueung/hugo-remark-minion
Hope it'd be useful and fun for some of you.
rgds,
EM

Added here, thanks! https://github.com/gnab/remark/wiki/Using-with-Hugo


[IS 5.5.0] [SCIM2] filter attributes should not be case-sensitive

per Section <IP_ADDRESS> of RFC 7644:
Attribute names and attribute operators used in filters are case
   insensitive.  For example, the following two expressions will
   evaluate to the same logical value:

   filter=userName Eq "john"

   filter=Username eq "john"

however, when attempting to query a local installation of WSO2 IS 5.5.0:

curl -k https://localhost:9443/scim2/Users?filter=username+eq+admin returns {"schemas":"urn:ietf:params:scim:api:messages:2.0:Error","scimType":"invalidValue","detail":"Not a valid attribute name/uri","status":"400"}
curl -k https://localhost:9443/scim2/Users?filter=userName+eq+admin returns {"totalResults":1,"startIndex":1,"itemsPerPage":1,"schemas":["urn:ietf:params:scim:api:messages:2.0:ListResponse"],"Resources":[{"meta":{"created":"2018-06-27T11:12:31Z","lastModified":"2018-06-28T15:37:38Z"},"name":{"givenName":"admin","familyName":"admin"},"groups":[{"display":"Application/impression-DS"}],"id":"a450e086-ce5e-4e38-b048-d397e6190ac3","userName":"admin"}]}

this can cause some compatibility issues with SCIM2 client libraries.

thanks!  Is there currently an estimate for 5.6's final release?

The official 5.6.0 GA release announcement will be on 16th July.

this is still failing for me in 5.6.0:

curl -k https://localhost:9443/scim2/Users?filter=userName+eq+admin returns {"totalResults":1,"startIndex":1,"itemsPerPage":1,"schemas":["urn:ietf:params:scim:api:messages:2.0:ListResponse"],"Resources":[{"emails":[{"type":"other","value":"admin@wso2.com"}],"meta":{"created":"2018-07-16T17:51:30Z","lastModified":"2018-07-16T17:51:30Z"},"name":{"givenName":"admin","familyName":"admin"},"groups":[{"display":"Application/local-8443"}],"id":"f2d980d6-cd5f-4c2d-890f-94bc9cf39705","userName":"admin"}]}
curl -k https://localhost:9443/scim2/Users?filter=username+eq+admin returns {"schemas":"urn:ietf:params:scim:api:messages:2.0:Error","scimType":"invalidValue","detail":"Not a valid attribute name/uri","status":"400"}


can't see libbgs.pyd in my C:\bgslibrary\build path.

I tried to install：Wrapper: Python
I've run it successfully：cmake -D OpenCV_DIR=%OpenCV_DIR% -D BGS_PYTHON_SUPPORT=ON -G "Visual Studio 14 Win64" ..
But at the end, I don't see libbgs.pyd file in my C:\bgslibrary\build path.
My environment is
win10+opencv:3.4.2+python3.6;。
this is my generated directory:

I don't know if it is caused by the following reasons：
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/x86_amd64/cl.exe - skipped
Why the output is "skipped" instead of "works"?

I've solved my problem. I have this conclusion:

Be sure to use: git clone -- recursive https://github.com/andrewssobral/bgslibrary.git . DON'T downloading the zip file directly. This causes the pybind file to be lost.
Everyone's. PYD file name is different.
Pybgs needs to adjust environment variables to work properly. I still have some confusion about this, such as I can't use it in other directories.
In addition, pybgs seems unable to adjust the parameters of different methods at present. It's a pity.

I've solved my problem. I have this conclusion:

Be sure to use: git clone -- recursive https://github.com/andrewssobral/bgslibrary.git . DON'T downloading the zip file directly. This causes the pybind file to be lost.
Everyone's. PYD file name is different.
Pybgs needs to adjust environment variables to work properly. I still have some confusion about this, such as I can't use it in other directories.
In addition, pybgs seems unable to adjust the parameters of different methods at present. It's a pity.


Cloud-Provider(aws): Update the example policy with additional ec2 actions

Need to update the policy for IAM Role with additional ec2 actions like  create  and attach  the  volume .The sample example refers attach and detach but creation of volume  not mentioned .
As end-users are  referring this docs  to createVolume, its better to update the doc with below allowed  ec2 actions.
"ec2:DeleteTags",         "ec2:CreateTags",         "ec2:CreateVolume",         "ec2:DeleteVolume",         "ec2:CreateSnapshot"
Steps:

Open https://rancher.com/docs/rke/latest/en/config-options/cloud-providers/aws/
Check the Example Policy for IAM Role.
Right now we have added attach ,detach volume ,so please update example with above mention ec2 actions(createVolume, deleteVolume etc).
JFYI:
refer  this page  for your reference: sample ec2 actions

Thanks for addressing the issue. So closing this issue as resolved.


An error occurred (InvalidClientTokenId) when calling the ListStacks operation: The security token included in the request is invalid

I get this error when I attempt to test the lambda function.
{   "errorMessage": "An error occurred (InvalidClientTokenId) when calling the ListStacks operation: The security token included in the request is invalid",   "errorType": "ClientError",   "stackTrace": [     [       "/var/task/lambda_function.py",       180,       "lambda_handler",       "drift_region(profile, region)"     ],     [       "/var/task/lambda_function.py",       169,       "drift_region",       "dd = DriftDetector(profile,region)"     ],     [       "/var/task/lambda_function.py",       74,       "__init__",       "self.detections = self.check_drift()"     ],     [       "/var/task/lambda_function.py",       98,       "check_drift",       "for stack in self._get_stacks():"     ],     [       "/var/task/lambda_function.py",       84,       "_get_stacks",       "resp = self._cfn_call('list_stacks',{'StackStatusFilter':ACTIVE_STACK_STATUS})"     ],     [       "/var/task/decorators.py",       39,       "f_retry",       "return f(*args, **kwargs)"     ],     [       "/var/task/lambda_function.py",       166,       "_cfn_call",       "return getattr(self.cfn_client,method)(**parameters)"     ],     [       "/var/task/botocore/client.py",       508,       "_api_call",       "return self._make_api_call(operation_name, kwargs)"     ],     [       "/var/task/botocore/client.py",       911,       "_make_api_call",       "raise error_class(parsed_response, operation_name)"     ]   ] }
This is the log output
`START RequestId: 2a53a505-518e-4584-a6b6-17d4e5789505 Version: $LATEST
An error occurred (InvalidClientTokenId) when calling the ListStacks operation: The security token included in the request is invalid, Retrying in 3 seconds...
An error occurred (InvalidClientTokenId) when calling the ListStacks operation: The security token included in the request is invalid, Retrying in 6 seconds...
An error occurred (InvalidClientTokenId) when calling the ListStacks operation: The security token included in the request is invalid, Retrying in 12 seconds...
An error occurred (InvalidClientTokenId) when calling the ListStacks operation: The security token included in the request is invalid: ClientError
Traceback (most recent call last):
File "/var/task/lambda_function.py", line 180, in lambda_handler
drift_region(profile, region)
File "/var/task/lambda_function.py", line 169, in drift_region
dd = DriftDetector(profile,region)
File "/var/task/lambda_function.py", line 74, in init
self.detections = self.check_drift()
File "/var/task/lambda_function.py", line 98, in check_drift
for stack in self._get_stacks():
File "/var/task/lambda_function.py", line 84, in _get_stacks
resp = self._cfn_call('list_stacks',{'StackStatusFilter':ACTIVE_STACK_STATUS})
File "/var/task/decorators.py", line 39, in f_retry
return f(*args, **kwargs)
File "/var/task/lambda_function.py", line 166, in _cfn_call
return getattr(self.cfn_client,method)(**parameters)
File "/var/task/botocore/client.py", line 508, in _api_call
return self._make_api_call(operation_name, kwargs)
File "/var/task/botocore/client.py", line 911, in _make_api_call
raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (InvalidClientTokenId) when calling the ListStacks operation: The security token included in the request is invalid
END RequestId: 2a53a505-518e-4584-a6b6-17d4e5789505
REPORT RequestId: 2a53a505-518e-4584-a6b6-17d4e5789505	Duration: 26768.29 ms	Billed Duration: 26769 ms	Memory Size: 128 MB	Max Memory Used: 70 MB	Init Duration: 353.78 ms
`
Not sure what I did wrong.
I attempted to put in my secret key and access key into the test event parameters, but that didn't work.

This is the error I get when I attempt to test the lambda function. I have Admin access permission, but I still get this error.

Can you send me the test event that you're using?

are you referring to the event rules, or the lambda function?

I'm just going to send you what I got and go from there.

this is the event rule in the eventbridge
this is the lambda role

I think I have a general idea of what caused this error. I know it's something to do with the regions because I am working on the AWS gov cloud and they have different regions. I tried to update the regions in the lambda_function.py file, however I am still running into this issue. Any ideas?

I would try to update line 32 in the lambda to only include the gov regions or just the regions you care about.
https://github.com/dejonghe/stack-drift-notifier/blob/master/lambda/drift_detection/lambda_function.py#L32

I tried that's how I got the intial error. It didn't work when I change the region to the gov cloud regions.


web3 migrate command

I'm not entirely sure how this could work, but imagine wanting to move to a new contract if you wanted to upgrade it (and didn't use an upgradeable contract). Or if you wanted to move from Ethereum to GoChain (or vice versa). This would move all the data and allow you to either use the existing contract code (if moving blockchains) or deploy new contract code with the same data.

@benbjohnson you might have some thoughts on this after doing the upgrade feature?

Given that it is not possible to e.g. iterate over map keys, is it possible to even know the full state of a contract without first replaying it's history? Could we replay the full history in an offline simulation, and then deploy that result via a single tx? Would it have to be a relatively costly constructor?
Is it reasonable to expect users to reuse the same accounts on both chains? Or would we need to support some kind of mapping?

I believe you could get code with eth_getCode and potentially all the storage via eth_getStorageAt (guessing). Now I have no idea how you'd write that to the new contract storage though....

It looks like getStorageAt only retrieves individual values, so you'd have to know all the keys for a map for example.

Alternatively, we could do it for contracts that do support such an iteration.
https://docs.openzeppelin.org/docs/token_erc721_erc721token#tokenByIndex
Too bad ERC-20's don't typically have something like this for all token holders.

Sorry for the late reply. I agree with @jmank88 that we wouldn't be able to fetch the data without replaying history. I also agree that data relating to addresses isn't going to migrate well across chains.


Support fsspec 2023.10.0

Fix #6333.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

I think https://github.com/huggingface/datasets/pull/6334 fixes it already no ?

Yes, @lhoestq, you are right. I think we cross-send fixing PRs in a 15 minute interval... :sweat_smile:
I would say the code in this PR is simpler and easier to understand, but feel free to ignore it.

I think the correct way it to check if "file" in in the tuple if it's a tuple (in case someone adds another protocol name for the local filesystem)


Phase1 types and params

Does this look ok?
Should I try to adjust tests for phase 1 stuff? Do we have some other ideas how to test those type generators?
resolves #1458

LGTM :)

LGTM :)

Didnt read pr description, eh? : 😁

@wemeetagain I manage to make type generation full type safe so it don't actually need those wierd unit tests since typescript checks whether we have ssz definition for every interface and if they match.
Do you wan't me to change phase0 types in separate PR?


[BUG] The .d.ts declaration file is not valid

Setup a simple TypeScript project adding this module as dependency, import this module, and running tsc results in following error:
node_modules/@lenml/tokenizers/dist/main.d.ts:308:69 - error TS2339: Property 'progress_callback' does not exist on type 'undefined'.

308     static from_pretrained(pretrained_model_name_or_path: string, { progress_callback, config, cache_dir, local_files_only, revision, legacy, }?: PretrainedTokenizerOptions): Promise<PreTrainedTokenizer>;
                                                                        ~~~~~~~~~~~~~~~~~

Which seems to be caused by PretrainedOptions from hub.js being undefined.
Copying the definition from upstream should be able to fix it, but I'm not sure if there are other issues might occur:  https://github.com/xenova/transformers.js/blob/f43d3dd348fd7b293008802590bb3a1afa218dc7/src/utils/hub.js#L14-L27`

Thanks for the feedback!
In addition to the issue you mentioned, I found two other type issues and attempted to fix them as well. You can try the new version @1.1.2.
By the way, these type issues might originate from transformers.js. There are actually some simple workarounds, like adding xxx as any; in the appropriate places.
Fixed in 3e08f12dd02ad86ac4136b5909305a6981d15de9.

The new version works perfectly, thanks a lot!


Remove deprecated Ember.K

This PR was generated automatically using ember-k-codemod

wow, thanks!


RPA-Python can‘t use browsers other than chrome

But TagUI does support other chrome。
According to the issue:https://github.com/aisingapore/TagUI/issues/946，
Can you please add this feature in RPA-Python?

TagUI supports Chrome, Edge, Firefox old version 59. But Python rpa package's goal is only to automate repetitive tasks. So it only aims to support the most commonly used browser Chrome. Chrome is similar to Edge, both are using Chromium browser underneath.
For more browser options, consider using Selenium which is designed for testing and it supports all browsers I believe.

Closing issue for now but please feel free to reply and share your thoughts!


feat: disable contribute button if user unathenticated

Fixes Issue
Closes #82
Changes proposed
Added a check to disable contribute button

A tooltip should be present. Because he or she needs to be told to log in to see it.
__
  Summary: :hammer_and_wrench: This code needs a fix

A tooltip should be present. Because he or she needs to be told to log in to see it.
__        Summary: 🛠 This code needs a fix

already created an issue for tooltip - https://github.com/rohitdasu/projectmate/issues/89
I believe adding tooltips can be done in seperate PR

@rohitdasu I have an idea like we can show the contribution button as it is only change the function depending on user is signed in or not if not we can show the same modal of sign in and user can sign in then ..


SyntaxError: Unexpected token

Red Hat EL7, node 6.17.1
/usr/lib/node_modules/corona-cli/index.js:36
(async () => {
^
SyntaxError: Unexpected token (
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:549:28)
at Object.Module._extensions..js (module.js:586:10)
at Module.load (module.js:494:32)
at tryModuleLoad (module.js:453:12)
at Function.Module._load (module.js:445:3)
at Module.runMain (module.js:611:10)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:160:9)

Before filing a bug it's important that you check if you are using the latest version of the CLI or not. I'm pretty sure you're on an older deprecated version.
UPDATE corona-cli by running:
npm i -g corona-cli

Peace! ✌

I did try this, thank you... however encountered the same error after.  Looks like corona-cli 3.8.6
[~]: sudo npm i -g corona-cli
/usr/bin/corona -> /usr/lib/node_modules/corona-cli/index.js
/usr/bin/corona-cli -> /usr/lib/node_modules/corona-cli/index.js
/usr/lib
└──<EMAIL_ADDRESS>delgreco@abby3 [~]: corona
/usr/lib/node_modules/corona-cli/index.js:36
(async () => {
^
SyntaxError: Unexpected token (
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:549:28)
at Object.Module._extensions..js (module.js:586:10)
at Module.load (module.js:494:32)
at tryModuleLoad (module.js:453:12)
at Function.Module._load (module.js:445:3)
at Module.runMain (module.js:611:10)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:160:9)

Before filing a bug it's important that you check if you are using the latest version of the CLI or not. I'm pretty sure you're on an older deprecated version.
UPDATE corona-cli by running:
npm i -g corona-cli

Peace! ✌

You are using an old version of Node. I don't support Node below 12 since those not maintained. Update you Node version so it can support features like async/await.


Clarifying fullName

The description of 'fullName' was not clear to users

@pilor : Thanks for your contribution! The author, @DCtheGeek, has been notified to review your proposed change.


Close connection

I am working with 2 clusters, and I am having issues with closing connections to A in order to use B cluster.

Can you describe it in detail?

In before route (using fatfree framework) I call cluster A, and produce message. Other class S should call another cluster B and produce message there. And it seems every time, I hit topic/partition is not found, because it always referrers to cluster A.
Class S extends controller with before route.

@phm3
You can use two produce instance, cluster A corresponding Instance A, cluster B corresponding Instance B, Or you can derict use  \Kafka\Protocol\Encoder and \Kafka\Protocol\Decoder send produce request, example can refer :https://github.com/weiboad/kafka-php/blob/master/example/SimpleProduce.php

If use \Kafka\Protocol\Encoder and \Kafka\Protocol\Decoder you need maintenance connect self

Thank you, I haven't tested yet \Kafka\Protocol\Decoder (Encoder)

I recomend you use tow instance solve this issue, because use  \Kafka\Protocol\Decoder (Encoder) method need you get kafka metadata mapping partion and broker, use \Kafka\Produce simply

The issue can be closed, I suspect it is a problem of the framework I use.


Surface error message when player is too far close to the webcam

If a player's body is too close to the webcam, they won't be able to perform the required poses. We should warn them to move back in an easy to understand way.

looks good to me!

The approach I am considering is still the same: before/in/after drawing, check if some essential landmarks are empty. If, for example, abdomen landmarks do not exist, it means the abdomen is not in the camera frame, with the assumption that we do not store data from prev states.
Regarding the question about which landmarks do we check, I think it is best to check the boundaries of the human body: face, shins, and hands. If these three parts are not shown on camera, the rest of the body is not shown on camera.
If I have access to those landmarks in Story.js, it might be efficient to check there? I need to look into mediapipe/holistic for more information.

I like that general approach, @likeketchup


Update pytorch_lightning.py callback

Issue Type

Bug fix

Description
Use new trainer property, and fix callback function arguments.
Motivation and Context
Deprecated property in lightning.
https://github.com/Lightning-AI/lightning/pull/12072/files#diff-667a2513c158c2b01d73b479e4dea75587d8531f9cd286891d34570a2fd145dcR2117

Hello @lkhphuc, thank you for your contribution to this PR.
My preference would be to retain this callback for now, and potentially consider deprecating it or marking it as defunct. This is primarily due to the fact that this code has been in place for several years, making it challenging to cater to a broad specter of pytorch-lightning or lightning versions. I believe a more suitable approach would be to encourage users to utilize the first-party callbacks maintained by the Lightning-AI team. An example could be the lightning.pytorch.callbacks.DeviceStatsMonitor callback. I would appreciate your thoughts on this.
It's important to note that nvitop doesn't rely on pytorch-lightning or lightning. As such, we don't have a release schedule that's aligned with Lightning-AI/lightning. This autonomy means we occasionally struggle to keep up with upstream API alterations. Here's a list of regressions in the Lightning API you may find helpful: Lightning-AI/lightning/docs/source-pytorch/upgrade/sections.

Hi.
For me the default DeviceStatsMonitor gives me 122 scalars chart just for "on train_batch_end" step and it's pretty much useless. That's why I came to nvitop in the first place.
I understand your concern about keeping up with Lightning's upstream. It's indeed should better be incorporated in their library if they ever want to.
One options might be to just provide the API forget_gpu_stats and move all the other callbacks into an example folder, maintained by community. These callbacks are not very complicated but some boilerplate to get it quickly running would be very helpful from user's perspective.
For now the simple changes in this PR is working fine for me, but feel free to close it.

For me the default DeviceStatsMonitor gives me 122 scalars chart just for "on train_batch_end" step and it's pretty much useless. That's why I came to nvitop in the first place.

@lkhphuc Have you ever tried nvitop.ResourceMetricCollector, it will give you a detailed list of metrics for host, device and process metrics.

One options might be to just provide the API for get_gpu_stats

You can access get_gpu_stats via:
from nvitop import Device
from nvitop.callbacks.utils import get_gpu_stats

gpu_stats = get_gpu_stats(Device.cuda.all())

You can also try to nvitop.take_snapshots implement your own get_gpu_stats-like APIs:
from nvitop import take_snapshots

print(take_snapshots())

I think ResourceMetricCollector may be better to fit your needs. Here is a minimal example to create a background monitor:
import time

from nvitop import Device, ResourceMetricCollector

def on_collect(metrics):
    print(metrics)
    return True

collector = ResourceMetricCollector(Device.cuda.all(), interval=1.0)
daemon = collector.daemonize(on_collect, interval=5.0)

time.sleep(3600)  # do something

For more examples, you can find them in the README or the online documentation.

For me the default DeviceStatsMonitor gives me 122 scalars chart just for "on train_batch_end" step and it's pretty much useless. That's why I came to nvitop in the first place.
It's indeed should better be incorporated in their library if they ever want to.

The pytorch-lightning callback in nvitop.callbacks is a port version that refactored with nvitop APIs rather than nvidia-smi calls. I think it will not give more metrics than the original GpuStatsMonitor (now DeviceStatsMonitor). I can submit a PR to the Lightning upstream to add more metrics if you want to.

Thanks. I will take a look at the ResourceMetricCollector.
For Lightning if you have time to contribute upstream that would be great. My issue with it is not that it doesn't log enough but that it logs too much, like this active_bytes /inactive_split_bytes,  small_pool/large_pool, peak / free / allocated ... to a total of 122 charts for a step.

I'm going to close this and roll my own callback using your suggestion for now.


show grid has turned itself on

v0.18.0
after the upgrade, starting node-red has show-grid on.
you have to go into settings, turn it on, then turn it off, in order to turn it off

We now store your editor config on the server side rather than in localStorage of the browser you happen to use. This means you get a consistent experience when accessing it from different browsers.
It should have migrated your existing settings over - but maybe not.
Fixing the migration is going to offer diminishing returns as more users upgrade.

ah, OK - that would also explain why snap to grid turned itself off, then :)
As you say, diminishing returns.

when you say server-side, though, do you mean in the flow file currently in use?
Notwithstanding the move to projects, wouldn't that mean inconsistent experience when you move between flow files, as it would drop back to the defaults (grid on, snap off) the first time you use an existing flow file after migration to 0.18 ?
Or maybe you now have a secret stash somewhere else server-side?

It is stored in the runtime config file: .config.json

good :) in that case, the only remaining problem is that the "show grid" checkbox in settings doesn't accurately reflect the current state, as the grid will be on (the new initial default) but the checkbox shows it's off. So you have to turn it on, then off to turn the grid off.

I'm sure you've got far bigger fish to fry ;)

Going to flag this as 'wontfix' given my comments above about diminishing returns. Sorry!


Add logformatter

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.73%. Comparing base (f985d7f) to head (2120cf6).

:exclamation:  There is a different number of reports uploaded between BASE (f985d7f) and HEAD (2120cf6). Click for more details.
HEAD has 5 uploads less than BASE

Flag
BASE (f985d7f)
HEAD (2120cf6)

6
1

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
- Coverage   96.53%   87.73%   -8.80%
==========================================
  Files          15        4      -11
  Lines        1790      473    -1317
==========================================
- Hits         1728      415    -1313
+ Misses         45       41       -4
  Partials       17       17

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


names should be unique

for hubs, signals, etc

and required, also for all other models

There is already a case of two different initiatives with same name (Voto Joven). Platform will allow identic names to cover these cases.


Fixed packages index in url

Description
Fixed packages index in url
Type of change
What is it?

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I run npm run lint:js:fix to check that my code is properly formatted

Codecov Report

Merging #291 (848eb87) into master (8281aa8) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #291   +/-   ##
=======================================
  Coverage   31.23%   31.23%
=======================================
  Files         102      102
  Lines        2110     2110
  Branches      620      620
=======================================
  Hits          659      659
  Misses       1330     1330
  Partials      121      121

Impacted Files
Coverage Δ

src/Routes/ImageManagerDetail/ImageDetailTabs.js
0.00% <0.00%> (ø)

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 8281aa8...848eb87. Read the comment docs.


Possible Inappropriate Implementation in s2-naip_dataset.py (Reshape Issue)

Hi Authors,
Thanks for make it public! I am following up on your work and have found an inappropriate code implementation that may invalidate some conclusions and findings about the number of Sentinel-2 images in your research paper. Specifically, at lines 190-191 in s2-naip_dataset.py, when it reads and reshapes the data, the result of s2_img (in line 191) blends the channels and time series together, which makes the s2_tensor unusable for time-series split with indices. Thus, the model always uses all the time-series data but only a few channels.
Here is an example: Say you have an RGB image with dimensions 3x128x32, which becomes 4x3x32x32 (TxCxHxW) after reshaping (T: Time, C: Channel, H: Height, W: Width). The desired output after reshaping should be like this:
-- For each T_i in the first dimension (each 3-channel image 3x32x32), the value is from (T_i^R, T_i^G, T_i^B) as a Sentinel-2 image;
However, the current implementation will result in the following outcomes:
-- For the 1st T_i (i=0), its 3x32x32 is from (T_0^R, T_1^G, T_2^B);
-- For the 2nd T_i (i=1), its 3x32x32 is from (T_0^G, T_1^B, T_3^R);
-- For the 3rd T_i (i=2), its 3x32x32 is from (T_0^B, T_2^R, T_3^G);
-- For the 4th T_i (i=3), its 3x32x32 is from (T_1^R, T_2^G, T_3^B);
In this case, a 3x32x32 array is not a Sentinel-2 image for a specific time, but a non-meaningful channel combination.
The correct way to address this could be - first reshape it to CxTxHxW, then move axis using permute to TxCxHxW:
Replace
s2_img = torch.reshape(s2_img, (-1, s2_img.shape[0], 32, 32))
with
s2_img = torch.reshape(s2_img, (s2_img.shape[0], -1, 32, 32)).permute(1,0,2,3)

Hi, thank you for bringing this to our attention! I ran the experiments for the paper in my research codebase, which used skimage for reading in images, so the way we loaded and shaped input was different than it is here. When writing the cleaner codebase for releasing to the public, I switched to using torchvision for reading in images because I found some memory leak issues with skimage, and this bug slipped in.
This explains why outputs have not been as high-quality for some users as the ones from the paper and Satlas website.
I have pushed the fix. Thanks again!


QDM 1.2.0: Upgrading to OCSF 1.2.0

What?
This PR brings QDM up to OCSF 1.2 without breaking anything.
Several events in the discovery category were renamed between OCSF 1.1 and 1.2. The old names have been preserved but deprecated. Attributes in the old events in QDM but not in the new events in OCSF have been added to the new events.

This should be merged after merging https://github.com/query-ai/ocsf-schema/pull/18, hence marking it as draft


Got 404 Not Found when accessing https://git.io/weave-kube

After changing pod_network_type to 'weave', got error as below:
TASK [kube-cni : Apply the weave podnetwork] ***************************************************************************************************** fatal: [kube-master]: FAILED! => {"changed": true, "cmd": "kubectl apply -f https://git.io/weave-kube > /tmp/podnetwork-apply.log\n", "delta": "0:00:02.412719", "end": "2020-01-14 06:28:05.748333", "msg": "non-zero return code", "rc": 1, "start": "2020-01-14 06:28:03.335614", "stderr": "error: unable to read URL \"https://git.io/weave-kube\", server reported 404 Not Found, status code=404", "stderr_lines": ["error: unable to read URL \"https://git.io/weave-kube\", server reported 404 Not Found, status code=404"], "stdout": "", "stdout_lines": []}

Without kube-ansible and kubernetes, https://git.io/weave-kube just returns 404 error.
This means 'https://git.io/weave-kube' is no longer used. Could you ask weave community to get the latest (and valid) URL?
This issue is closed because it is not a kube-ansible issue.


Add support for configuring High Availability feature.

What does this PR do?
Adds support for configuring the High Availability feature in the Agent.
A new top-level configuration section, ha, has been added which allows specifying a number of settings required for the High Availability/HAMR feature. Additionally, we've allowed this section to be exposed through the new internal configuration API, as it must be queried by an adjacent Agent subprocess.
Motivation
This functionality is necessary to allow for the Agent to drive configuration of Agent Data Plane (ADP) -- the aforementioned adjacent Agent subprocess -- so that all credentials and site/endpoint configuration are sourced primarily from the Agent, rather than configuring the same information in two places. It also sets us up for the ability to allow Fleet Automation capabilities to naturally compose with these settings, rather than having ADP itself also have to talk to FA.
Additional Notes
We've also made a small tweak to the route handler for the configuration API to send back a proper a proper content type header, indicating the payload is JSON and not simply plaintext.
As well, this is marked as not needing a changelog as the functionality is not yet ready to be shown to users, as this is only being used for internal development of the High Availability/HAMR feature.
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
The configuration settings added are straightforward enough to validate on their own, but the real meat is whether or not they are emitted via the configuration API.
This can be checked by enabling the configuration API endpoint (as simple as setting DD_AGENT_IPC_PORT=9999 when running the Agent) and then querying the endpoint, like so:
curl -k -H 'Authorization: Bearer <auth token>' https://<IP_ADDRESS>:9999/config/v1/ha

This should return a simple JSON payload:
{"enabled": false, "failover": false}

Updating the relevant settings -- DD_HA_ENABLED=<true/false>, DD_HA_API_KEY=<random string>, and so on -- should produce a corresponding changes in the JSON output.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[x] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[x] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[x] If applicable, the need-change/operator and need-change/helm labels have been applied.
[x] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[x] If applicable, the config template has been updated.

/merge

/merge

/remove

/merge

/merge


[AKS] Add cluster upgrade settings options to aks update

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Queued a live test pipeline to verify the change.

AKS


Current "Using git sources" example invalid

It's not possible to use the current example command for "Using git sources" as git can't clone into a non-empty directory.
2016/07/27 14:42:02 cannot git clone into ., directory not empty

Very much possible. If your Caddyfile is not in root.

Oh sorry. I see what you mean. I will look for an alternative.
Thanks.

Fixed.
Thanks.


Add null check for newComment paramter from request.

My original comment was to replace the .length() == 0 with .isEmpty(), you still need the null check.


Resolver does not support @/ paths

What happened?
It's common in projects to have settings like this (from tsconfig.json):
"baseUrl": ".",
    "paths": {
      "@/*": [
        "src/*"
      ]
    }

In other words, using @/ as a "source-relative" reference. This works in TypeScript, ESLint, Babel, swc, Rollup, etc, as long as all are configured correctly.
However, @rnx-kit/tools-node explicitly forbids this, with these lines in package.ts. Meaning that the symlink resolver can't resolve modules the way any other module resolver would.
Therefore, this throws an error:
import { useGlobalState } from '@/context/globalContext'

Affected Package
@rnx-kit/tools-node
Version
latest
Which platforms are you seeing this issue on?

[ ] Android
[X] iOS
[ ] macOS
[ ] Windows

System Information
System:
    OS: macOS 11.6
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 909.99 MB / 32.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 18.10.0 - /usr/local/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 8.19.2 - /usr/local/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.11.3 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
    Android SDK: Not Found
  IDEs:
    Android Studio: Not Found
    Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
  Languages:
    Java: 17.0.1 - /usr/local/opt/openjdk/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.1.0 => 18.1.0
    react-native: 0.70.0 => 0.70.0
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to Reproduce
Try writing this:
import { foo } from '@/bar'

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @matthew-dean, thanks for reporting this! We don't have support for the extended module resolution semantics in tsconfig. The parsing code you pointed out is looking for a package-scope, and is blissfully unaware that it might be looking at an alias of some kind.
Are you hoping to see that support added? Or do you have something more tactical in mind?


Update to new Docsearch v3 and fix native search when docsearch is disabled

This change

Updates Algolia Docsearch to v3
Fixes the issue with search not filtering results according to the current PrestaShop version (read note below)
Re enables the native search when docsearch is disabled (eg for prestashop-specs)

Here's what it looks like now with Algolia Docsearch:

Note: the shortcut "keys" are added by Algolia's script. I guess they change according to the user's platform automatically.
Here is what it looks like when searching:

Note about broken search-by-version
Currently, the search is not filtering results by PrestaShop version and displays results twice (one for each version). This is also fixed with this PR because I updated Algolia's index but it only works with Docsearch v3.

Thank you @eternoendless


Fixed typo in channel.c

Changed "preend" to "prepend".

Codecov Report

Merging #4262 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4262      +/-   ##
==========================================
+ Coverage   79.75%   79.77%   +0.01%
==========================================
  Files         106      106
  Lines      141579   141579
==========================================
+ Hits       112923   112941      +18
+ Misses      28656    28638      -18

Impacted Files
Coverage Δ

src/channel.c
83.54% <ø> (ø)
:arrow_up:

src/gui.c
57.88% <0%> (-0.52%)
:arrow_down:

src/term.c
69.3% <0%> (-0.06%)
:arrow_down:

src/window.c
86.36% <0%> (+0.03%)
:arrow_up:

src/terminal.c
80.46% <0%> (+0.03%)
:arrow_up:

src/message.c
76.63% <0%> (+0.04%)
:arrow_up:

src/ui.c
53.58% <0%> (+0.07%)
:arrow_up:

src/ex_cmds.c
81.77% <0%> (+0.09%)
:arrow_up:

src/screen.c
80.7% <0%> (+0.1%)
:arrow_up:

src/syntax.c
79.74% <0%> (+0.12%)
:arrow_up:

... and 2 more

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 7f27976...06dc7fd. Read the comment docs.

Thanks.  I'll not make a separate patch for this, since it's just a comment.  It will be included in another patch for this file later.


CUDA error message handling

We generally wish to avoid print statements inside CUDA kernels.
For now, the SurfaceFluxes code contains @printf outputs. Currently there is an @printf screen output if the SurfaceFluxes conditional upon non-convergence of the iterative root-solver for Monin-Obukhov length. Error handling needs to be addressed when we begin enabling GPU tests.
cc @sriharshakandala

Currently the error message is displayed only when the iterative solver has reached a "breaking / crashing" non-convergence condition. We have a fallback option for no-print + crash output with an error_on_non_convergence flag.

Duplicate issue. Issue #129 captures the requirements.


Added plopfile for generating examples in example directory

https://www.loom.com/share/0a3064907006444b97e969069eafbd00

Finally got around to testing this out; LGTM


UI Improvements - Perk Levels / Icons

Some guidelines:

Avoid very vivid colors (such as pink or piss-yellow)
Have some sort of system where people can tell at a glance who's the 'higher level' player
Have lower tiers go up only to 3 stars, and higher tiers go up to 5 stars
We know that the outlier players are going to be ~lv50 on sharpshooter specifically, so it's a good end-point
Add some kind of shading to the icon textures, like a border or bevel with a darker color

Example:
Tier 0 - Basic

Standard KF icons (red)

Tier 1 - Green

Fill: #19d000
Border/Bevel: #16b500

Tier 2 - Blue

Fill: #3f93ff
Border/Bevel: #3578d9

Tier 3 - Purple

Fill: #a335ee
Border/Bevel: #8b2dcd

Tier 4 - Gold

Fill: #ffd700
Border/Bevel: #ccac00

Tier 5 - Orange

Fill: #ff8c19
Border/Bevel: #cc7015

Tier 6 - Platinum

Fill: #f3f6f4
Border/Bevel: #c2c4c3

Have some sort of system where people can tell at a glance who's the 'higher level' player

No idea what this would mean. Sounds like a bad idea.

Have lower tiers go up only to 3 stars, and higher tiers go up to 5 stars

Weird inconsistency for no particular reason. Sounds like a bad idea.

Implemented in b06f7952dadf441cf1e0dfcf3ad41057bcb67eab.