App Configuration Spring 4.0 - Draft

Description
Updating App Configurations Spring Library to use the 4.0 Spring Design.
Changes

Changed From azure-spring-cloud to spring-cloud-azure
Moved Code into implementation folder for revapi
Cleaned up code, some pre azure sdk code was still around
Renamed some classes for clarity, none of which are part of the public API
Feature Flags now have a Selects API similar to configurations
Feature Flags can now come from multiple sources without issue
Decoupled Feature Flag and Configuration Property Sources
Watch Keys aren't loaded if monitoring is disabled on startup

Blockers

[ ]  Geo Replication https://github.com/Azure/azure-sdk-for-java/pull/30084
[ ] App Configuration SDK Client Regression, also blocks Geo Replication https://github.com/Azure/azure-sdk-for-java/issues/30536
[ ] SDK throws uncatchable errors, doesn't actually block but makes catching the issues sub optimal https://github.com/Azure/azure-sdk-for-java/issues/30518
[ ] Dynamic Features, this feature branch updates the Feature Management code to the 4.0 style. Can't update the starter until both are done. https://github.com/Azure/azure-sdk-for-java/tree/feature/azconfig-spring/DynamicFeature

API change check
API changes are not detected in this pull request.


Closing this issue.


New package: DLFlib v0.0.9

Registering package: DLFlib
Repository: https://github.com/emsig/DLFlib.jl
Created by: @kerrykey
Version: v0.0.9
Commit: 2ba26c72eddaf6396c499c31d19248c90cbe3d0e
Reviewed by: @kerrykey
Reference: https://github.com/emsig/DLFlib.jl/commit/2ba26c72eddaf6396c499c31d19248c90cbe3d0e#commitcomment-55053704

Please block this pull request. We're revising our multi-language package distribution based on feedback on our first pull request as LibDLF.


Fix / Unit tests test_violin and test_pbmc3k

Description
As briefly outlined in #1420, using seaborn==0.11 causes the unit tests scanpy/tests/test_plotting.py:test_violin and scanpy/tests/notebooks/test_pbmc3k.py fail as the violin plots are no longer vertically oriented.
Changes

Rely on seaborn.catplot and seaborn.stripplot instead of seaborn.FacetGrid in scanpy/plotting/_anndata.py

Related issues
Closes #1420.

@fidelram, from seaborn==0.12 on, the only valid positional argument will be data which may cause failures or unexpected behaviour. This is relevant at least here and here. Should we add / specify the keyword argument x in this PR or open a separate issue and PR?

@WeilerP Thanks for finding about this. Yes, please add explicitly x=x in this PR 👍

Thanks. Any idea why the images are now bigger (which I assume is the reason for the very small fonts)?

Hm, no sorry, I don't. My best guess would be that seaborn.catplot and seaborn.FacetGrid use different default figure sizes which in turn makes the font look smaller?

I noticed, however, that the tests pass even when removing the if stripplot: part. Any idea on why this is happening and how to prevent it?

The tests have a tolerance parameter that is set high. The problem is that the stripplot shows different results each time. Also, different versions of matplotlib and seaborn have slight differences.

I wonder why the tests are not working now?

I wonder why the tests are not working now?

Sorry, I forgot to update violin.png after the latest changes to _anndata.py. Let's see if the tests pass now.

The tests have a tolerance parameter that is set high. The problem is that the stripplot shows different results each time. Also, different versions of matplotlib and seaborn have slight differences.

Ah yes, I see. The stripplot result could be fixed by setting a seed with np.random.seed. I doubt it will fix the difference due to the used version, though.

Hi Developers,
I got the same issue when using seaborn==0.11, and fortunately I got the issue solved by replacing the annotate.py with the modified version. Howerver, I was wondering why I tried to use 'pip install scanpy' to update the scripts, it failed? Is there any other easier method to modify the script, not to locate the file and replace it?

@hhhh1230511, this PR is not part of any release yet (the latest version scanpy==1.6 was released August 15, 2020). If you want to have the latest version from GitHub you can follow the instructions for a developer installation here in the documentation, for example. Once a new release is available on pip, you can install it via pip install --upgrade scanpy
In general, you should avoid modifying the code by e.g. simply copying and pasting. This will either easily cause conflicts when updating the package or cause problems when functions from other files which depend on the content you changed but were not updated accordingly.
Hope this helped and clarified things.


Add a Pause button to the 2D and 3D editor

Salvaged from https://github.com/godotengine/godot/pull/31070. Thanks @Chaosus :slightly_smiling_face:
This can be used to decrease CPU/GPU usage if particles or shaders are being displayed in the editor.
Note: Since low-processor usage mode hasn't been reimplemented yet in master, this won't decrease CPU/GPU usage significantly in the master branch, only in 3.2 if it's backported there.
Don't merge yet as pausing will break freelook in the 3D editor and scrolling in the script editor if smooth scrolling is enabled (related to https://github.com/godotengine/godot/issues/28385, partially addressed by https://github.com/godotengine/godot/pull/42704).
I can make a 3.2 version of this PR once it's ready.
This closes https://github.com/godotengine/godot-proposals/issues/942.
Preview
Freeze icon is at the bottom.

Salvaged from #31070.

Yeah, as I remember the pretensions for that PR in IRC was: "it was not customizable enough and better to be placed in the menu" and "it takes a valuable place". Perhaps the views are changed from that time...

Perhaps the views are changed from that time...

I think the current problem at hand is more important than saving a bit of space in the menu. It's making people not use Godot when they could.

Yeah, as I remember the pretensions for that PR in IRC was: "it was not customizable enough and better to be placed in the menu" and "it takes a valuable space". Perhaps the views are changed from that time...

At this point I'd take that pause button over most others, haha. To maybe prevent confusion with the pause scene button on the top right, I'd suggest changing the tool name to "Freeze." I've made a 14x14 icon here: ToolFreeze.zip

@molevision I amended my PR to use the icon you provided, thanks :slightly_smiling_face:

Just a note that I think although pausing might be useful for timelines, a more suitable fix for the problem of constant editor updates might be to have a slight modification to the system for requesting updates.
At the moment it is very broad, you request an update in the editor no matter whether this is caused by an animating texture, or something that really needs updating (like an update to the UI). Most of the time speaking for myself on a fanless PC, I don't want these constant updates the moment I use TIME in a shader or somesuch.
I'm wondering whether we can change the request_update function to have an argument such as a bool (perhaps with a default) to indicate whether the update is an essential update or not. Then we could have a switch in the UI to turn the non essential updating on and off.
I've been meaning to have a look at this but it might be something you might be interested in @Calinou.

Is this likely to make it to 3.2.4?

Is this likely to make it to 3.2.4?

Probably not, since 3.2.4 is nearing release.

Probably not, since 3.2.4 is nearing release. You can still cherry-pick this pull request and build it locally if you need it.

I haven't ever been able to get Vulkan Godot to work properly on my machine, the viewport is always completely black. Is there some way to build this PR with 3.3?

Is there some way to build this PR with 3.3?

You can cherry-pick the PR's commit to the 3.3 branch, but you'll have to resolve merge conflicts in the way.
You can also apply the .patch file using git am, but I don't know how to resolve conflicts in an interactive manner with git am.

Rebased and tested again, it works successfully.

Congratulations on the 3.3 release! Can this please make it in to 3.3.1? 😅
This PR was intended as a stopgap awaiting specific fixes to a lot of people's technical problems with the viewport updating. I can't add animations to my characters or use post-processing without it, the editor load is just too high. I have not been able to use Godot for about a year because of this. It would make a big difference to everyone in #39758 to have this merged.
(Plus, Vulkan in 4.0 using the MoltenVK backend to access Metal might just make the problem go away, for Mac users at least.)

This can be closed, functionality implemented by #53463.


[SUGGESTION] Allow downloads to be picked up by external download managers

The title says it. On batch selecting files and downloading, Files get downloaded from default chrome download client even though I try to force it with IDM using the force keys for IDM.
So it'll probably be great if it lets external download managers to pick up downloads.

It's not possible to do so, because the chrome downloads API works only for
the default DM. Moreover, if I were to hard-code this, IDM would have to
have a message listener that I could send from one to another. Closing the
issue.
On Mon 20 Aug, 2018, 1:14 AM Rishav Rungta<EMAIL_ADDRESS>wrote:

The title says it. On batch selecting files and downloading, Files get
downloaded from default chrome download client even though I try to force
it with IDM using the force keys for IDM.
So it'll probably be great if it lets external download managers to pick
up downloads.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Presto412/Enhancer-for-VIT-Vellore-Academics/issues/1,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZJyxxlT2Jtqe5vZ6krf8W9ihBrJZsDWks5uScAigaJpZM4WDF4a
.


[docs] update create-coin docs to 2024 grammar

Description
[docs] update create-coin docs to 2024  grammar
Test plan
How did you test the new or updated feature?

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:

Thew CI failed on this but the issue was addressed.


[stable/coredns] Update coredns to 1.3.0

https://coredns.io/2018/12/15/coredns-1.3.0-release/
Signed-off-by: Manuel Rüger<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped

/assign @andor44

/assign @davidkarlsen

/ok-to-test

/lgtm


Code alignment issues

I think the "else" in this code should align with the first "if".(better readability?)
code location :  https://github.com/scipy/scipy/blob/main/scipy/signal/init.py

in python for loops can end with else blocks to run when the for loop is exhausted without breaks. Hence the else block belongs to for and not if https://docs.python.org/3/tutorial/controlflow.html#break-and-continue-statements-and-else-clauses-on-loops


Add FFT analysis pathway in estats etc for looking at oscillations in the models.

SVD is very fast and working well.


$configContext.ToString()

$configContext.ToString() for returning System.String instead of System.DirectoryServices.PropertyValueCollection

Pull Request (PR) description
$cdpContainer = [ADSI]('LDAP://CN=CDP,CN=Public Key Services,CN=Services,{0}' -f $configContext) returns an error:
format-default : The following exception occurred while retrieving member "distinguishedName": "An invalid dn syntax has been specified.
"
+ CategoryInfo          : NotSpecified: (:) [format-default], ExtendedTypeSystemException
+ FullyQualifiedErrorId : CatchFromBaseGetMember,Microsoft.PowerShell.Commands.FormatDefaultCommand
This Pull Request (PR) fixes the following issues:

Task list:

[ ] Change details added to Unreleased section of CHANGELOG.md?
[ ] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[ ] Examples appropriately updated?
[ ] New/changed code adheres to Style Guidelines?
[ ] Unit and (optional) Integration tests created/updated where possible?

This change is

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

A great catch there @amrowicki ! And thank you for submitting the fix!
Are you able to add an entry to the README.MD file Unrleased section mentioning this fix? https://github.com/PowerShell/DscResources/blob/master/CONTRIBUTING.md#fixing-an-issue
E.g. to this part of the file:
https://github.com/PowerShell/xCertificate/blob/dev/ReadMe.md#unreleased
Thank you

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

The entry has been added. Thanks!

@amrowicki @PlagueHO Please make sure you are merging into dev instead of master next time

Sorry abouth that @KwirkyKat ! I completely missed that. Will keep an eye on it. Thanks for picking this up.


Need to restart propagate on Talia brain

Propagate for one of the channels ("ch1") of [myToken] appears to have failed
http://openconnecto.me/ocp/ca/[myToken]/ch1/xz/1/3000,4000/7000/0,1000/window/0,600/
It's propagate status has been "1" for at least a week
http://openconnecto.me/ocp/ca/[myToken]/ch1/getPropagate/
I don't have the permissions to restart propagate.  When I try to set it to "0" so that I can restart propagation....
http://openconnecto.me/ocp/ca/[myToken]/ch1/setPropagate/0/
I get the following error...
Cannot set this value. Project is under propagation.

@kkutten1 I have restarted this.

Propagated.


fix: type the return of xml.parse to fix compilation problem with 4.3.0

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

fast-xml-parser v4.3.0 was released yesterday. Looks like stronger typing requirements were added to .parse which is breaking the build. See: https://github.com/NaturalIntelligence/fast-xml-parser/pull/594


External Clinical Sites / non-member / without users

In weekly meeting 2021-12-07, the need to support Clinical Sites that are not part of the group was added as a new request.
Clinical Sites that are not part of the group, will not have registered users. (Clinical Site and user on-boarding is still a manual process at v0.10.x). But there will be a need to display clinical trials that are recruiting on clinical sites that are not on-board.
Options in discussion: For those "external" clinical sites,

inhibit the "CONTACT SITE" button and display manual/external contact information ?
send an email to a clinical site contact email for every application
Other...

Meeting 2021-12-14 Flora: This requirement will not be part of the 1st real-world pilot.


RBAC support

IoT VK provider needs to support RBAC. Currently the VK can only be deployed to Kubernetes clusters without RBAC or you need to do modifications afterwards to make it running under Kubernetes clusters with RBAC enabled.
Please add RBAC support as we have it today for the ACI VK.
Thanks!

Added pull requests #13, #14, #15, and #16 for RBAC support

Support checked in. Thanks @neumanndaniel #17 #16 #15 #14 #13


Added .m2 to cache directory.

This should speed up builds.

Coverage remained the same at 100.0% when pulling 4ccf6d9c01d9b5c4e6150668e84c47120388df25 on feature/cache_maven into 16915fc50a7a613bf0b90aac3288925278059baa on develop.


Add Author Column to Recent Commits

Is your feature request related to a problem? Please describe.
I find myself frequently wanting to see the author when viewing Recent commits, but most of the times I'm too lazy to press enter on the commit to inspect it's details.
Describe the solution you'd like
I would like the author of the commit to also be included in Recent commits as a separate column so I can quickly see who authored the commit.
Describe alternatives you've considered
Start exercising, change my diet, work on my relationships, all in a futile effort to overcome my laziness.
Additional context

The place to embed that information is over here: https://github.com/NeogitOrg/neogit/blob/3d63d76f04dc7102c83a057ded30a93822b59dec/lua/neogit/lib/git/log.lua#L273
Or do some custom parsing for the commit in render_section, over here: https://github.com/NeogitOrg/neogit/blob/3d63d76f04dc7102c83a057ded30a93822b59dec/lua/neogit/status.lua#L193
Not sure what the best approach on that is. I'm guessing we'd want to set recent commit formatting based on a config flag.

I have it working on this branch. No flags to opt in or out, but this is how it looks:

@treatybreaker you're an absolute legend. If you PR this i'll be more than happy to review and approve.

Yep, definitely going to PR. Writing some tests for it currently 😎.


Docs missing instructions for "Choose Package Products"

When using Add Package in XCode with this repo, how do I know which boxes to check here? The docs don't mention as far as I can tell.

nevermind i see the pic now 👯


Improvement of TimeStructure types

The aims of this PR is to prevent problems in the construction of TimeStructures. So far, it was possible to create, e.g., an inconsistent Twolevel instance as we did not provide any internal constructors for identifying problems.
This PR solves this through incorporation of internal constructors.
In addition, it:

adds constructors to RepresentativePeriods to be consistent with other TimeStructures,
significantly improves the docstrings, and
test the individual constructors.

This PR partly closes #17, although there are still open questions regarding the default value of the keyword argument  op_per_strat as well as removal of certain constructors. This would however result in breaking changes.
All changes were tested with EnergyModelsBase v0.8 and EnergyModelsInvestments v0.7.

The formatting change is caused by the 80 character line length specified. It seems in my situation a bit to narrow, but that should be a separate PR.

I updated the PR ased on the comments. There are few things to note: We have to use frequently Number instead of Real as the CalenderTimes values do not work with Real. I have not done a thorough check of it, but it is something we have to think of before changing everything.


Port predefined workflow tests to derecho environment

We need the workflow software to run on the Derecho HPC.

The runtime environment differs from the Cheyenne environment
a. The spack-stack modules are different
b. The compile toolchains are different
c. The Cylc package is not installed on derecho (as of 11/27/23)
d. We will port the software to work with Cylc 8.x (the cylc package on Cheyenne is 7.x)
e. Some precompiled modules used by the predefined tests must be recompiled to run on Derecho
f. Some cylc job configuration files need to be updated to use cylc8 syntax.

The modules we need to load for the intel toolchain are described in mpas-bundle issue 154
The modules we need to load for the gnu toolchain are described in mpas-bundle issue 156

Steps 1.a-e are complete. Step 1.f has a separate issue, 267


404 on requesting css.

Did everything in order and added the

I selected 'title" instead of "path", now it works.


readonly={true} has no effect inside  element (SolidJS v1.0.6)

When doing <input type="text" value="123" readonly={true}/>, the readonly attribute does not appear inside the input element on SolidJS v1.0.6.
Further reading:
https://www.w3schools.com/tags/att_input_readonly.asp

Just discovered readOnly={true} works. (capital O)


security/acme-client: DOS line-endings in generated config lead to operational failure

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] The title contains the plugin to which this issue belongs

Describe the bug
A clear and concise description of what the bug is, including last known working version (if any).
Acme Client generates config files with DOS line endings, which results in settings gaining a carriage-return character when interpreted, resulting in things like new certificates being placed under /var/etc/acme-client/home^M, instead of /var/etc/acme-client/home, and parts of the activity being logged to /var/log/acme.sh.log^M instead of /var/log/acme.sh.log. Note: on the console, the ^M is displayed as a ?.
Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)
To Reproduce
Steps to reproduce the behavior:
Unclear, unfortunately. The specific setup was pushed through all updates from 19.x to 22.1.6 in quick succession via the Web GUI.
I think, the issue is in this line of code in the update migration path, but I also realize that the code is two years old, and, so, would've expected to find more People reporting this issue.
Expected behavior
A clear and concise description of what you expected to happen.
Acme client would continue to function.
Screenshots
If applicable, add screenshots to help explain your problem.
We are seeing /var/etc/acme-client/accounts/.../account.conf files generated that have DOS line-endings, reading like this:
CERT_HOME='/var/etc/acme-client/home'^M
LOG_FILE='/var/log/acme.sh.log'^M
ACCOUNT_KEY_PATH='/var/etc/acme-client/accounts/5d1c791e804ff9.78199304_prod/account.key'^M
ACCOUNT_JSON_PATH='/var/etc/acme-client/accounts/5d1c791e804ff9.78199304_prod/account.json'^M
CA_CONF='/var/etc/acme-client/accounts/5d1c791e804ff9.78199304_prod/ca.conf'^M
<EMAIL_ADDRESS>LOG_LEVEL='1'
SYS_LOG='6'
USER_PATH='/sbin:/bin:/usr/sbin:/usr/bin:/usr/games:/usr/local/sbin:/usr/local/bin'

Relevant log files
If applicable, information from log files supporting your claim.
Happy to answer questions, but not providing acme.sh.log, as they're full of sensitive information.
It is noteworthy that there exist both /var/log/acme.sh.log^M and /var/log/acme.sh.log files...
Additional context
Add any other context about the problem here.
As a workaround, we edit the generated account.conf files manually.
Environment
Software version used and hardware type if relevant.
e.g.:
OPNsense 22.1.6-amd64
FreeBSD 13.0-STABLE
OpenSSL 1.1.1n 15 Mar 2022

Cannot find DOS line-endings on my firewalls. Even account.conf files created way back in 2017 are still OK. So whatever caused this is either extremely rare and/or already fixed.

@fraenki FWIW, the PR #3032 looks correct

Hm, that commit fixes a almost 3 year old migration. Not sure if this fix will ever affect any users.
@cluck, if you submit a PR I'll merge it.


"severity levels" link generates 404

The link "severity levels" references "https://github.com/PagerDuty/incident-response-docs/blob/master/before/severity_levels.md" which appears to be missing.

https://github.com/PagerDuty/incident-response-docs/pull/122 will fix this I think.

This should now be fixed.


Code for nuScenes Dataset

Thanks for your great work! The paper also provides nuScenes results in supplemental materials. May you upload codes for training in the nuScenes dataset? Thanks again.

Thanks for your interest in our work. The result we provided in the supplemental materials was from a simple version of CenterFormer, which only aims to demonstrate the improvement of our transformer structure. I still need to test it with some features in our paper (deformable attention/ multi-frame fusion) and find a more suitable training setting. I will add the codes for nuScenes once I complete those experiments.

@edwardzhou130
您好，有个问题请教一下哈。
怎么用nuscene数据集去调试，centerformer代码。手中暂时没有waymo数据集

@rockywind You can refer to CenterPoint's configs for the training in nuScenes. What I did was simply switch the neck and bbox head to CenterFormer.
Some notes: 1) Some modules and parameters that work well in Waymo may not work in nuScenes due to the difference in the data and evaluation method, like the IoU loss. https://github.com/TuSimple/centerformer/blob/0408b29fa728950baead73354e21240174ce5b6b/configs/waymo/voxelnet/waymo_centerformer.py#L59 2) Our released code currently does not support multi-task in the bbox head.

@edwardzhou130
您好，按照这个教程安装，运行会报这个错。
  File "/DATA2/xiaoquan.wang/centerformer/det3d/models/detectors/voxelnet_dynamic.py", line 20, in __init__
    super(VoxelNet_dynamic, self).__init__(
  File "/DATA2/xiaoquan.wang/centerformer/det3d/models/detectors/single_stage.py", line 24, in __init__
    self.backbone = builder.build_backbone(backbone)
  File "/DATA2/xiaoquan.wang/centerformer/det3d/models/builder.py", line 35, in build_backbone
    return build(cfg, BACKBONES)
  File "/DATA2/xiaoquan.wang/centerformer/det3d/models/builder.py", line 21, in build
    return build_from_cfg(cfg, registry, default_args)
  File "/DATA2/xiaoquan.wang/centerformer/det3d/utils/registry.py", line 65, in build_from_cfg
    raise KeyError(
KeyError: 'SpMiddleResNetFHD is not in the backbone registry'

@edwardzhou130 您好，按照这个教程安装，运行会报这个错。
  File "/DATA2/xiaoquan.wang/centerformer/det3d/models/detectors/voxelnet_dynamic.py", line 20, in __init__
    super(VoxelNet_dynamic, self).__init__(
  File "/DATA2/xiaoquan.wang/centerformer/det3d/models/detectors/single_stage.py", line 24, in __init__
    self.backbone = builder.build_backbone(backbone)
  File "/DATA2/xiaoquan.wang/centerformer/det3d/models/builder.py", line 35, in build_backbone
    return build(cfg, BACKBONES)
  File "/DATA2/xiaoquan.wang/centerformer/det3d/models/builder.py", line 21, in build
    return build_from_cfg(cfg, registry, default_args)
  File "/DATA2/xiaoquan.wang/centerformer/det3d/utils/registry.py", line 65, in build_from_cfg
    raise KeyError(
KeyError: 'SpMiddleResNetFHD is not in the backbone registry'

You need to install spconv1.2.1


Add some missing fields

The MarkLine requires a precision configuration. According to the ECharts documentation at this link, the default value for precision is 2. If a higher precision is needed for plotting, it is currently not achievable. (Amend: add silent field.)
Most series allow for custom tooltips, but currently, this functionality is missing. I have added the missing parts based on the ECharts documentation, which allows different series to customize their respective tooltip display content. As it stands, tooltips can only be defined globally at the Chart root.

Looks good as usual, ty


💡 [Feature]: Update ReadMe and Docs preview features after latest update of VS Code

🎯 Aim of the feature
Since the latest update of VS Code it is now possible to use GitHub chat participant in standard version of VS Code. We should:

[x] update the preview features section to standard feature
[x] update wiki

📷 Images (if possible) with expected result
No response
🤔 Additional remarks or comments
No response

done ✅


Update content (internal SharePoint): SDM

@myp192
Did we do this last week?

Not sure if this issue is related to this but all the material was moved to Office 365 and links in the internal SharePoint have been updated.

Forget my previous comment, I understand what it means now. I'll work with @myp192 on this.


Project Kat - Paper Lily Prologue

SteamDB link to the game
https://steamdb.info/app/1698960/
What should it be detected as?
Engine.Godot

Sadly this game can't reasonably be detected with file based heuristics. It doesn't include clear Godot markers that would not lead to false positives.
https://steamdb.info/depot/1698961/

It bundles the main game data pack in the .exe file, so there's only ProjectKat.exe, not ProjectKat.exe + ProjectKat.pck. The latter combination would confirm it's a Godot game, but without it we can't be sure.
It does include additional .pck files which are loaded on demand (translations, etc.), but this extension isn't specific to Godot.
It uses Mono, so it has a data_<Project Name> folder, but the actual project name differs from the game / .exe name they chose, so even that couldn't be reliably matched here (I don't think we try to match these though). I don't think any of the files in the data_<Project Name> folder can reasonably infer that it's Godot, Unity C# games typically have a similar structure.
Doesn't use popular GDNative/GDExtension addons like GodotSteam, which would add a godotsteam.dll which can be a good marker.
No macOS build, which is usually a good fallback when the Windows build embeds the PCK file, as this can't be done for macOS so the GameName.pck file would be detected.


Update baseline spike

Description
Try net472

/azp run

Abandoning this in favor of PR that will be based on branch that is directly on this repo and not on a fork.


Pack URL bug

The new pack:/// url feature seems to be buggy.
From time to time Chromium is unable to read random packed files (I run the same executable each time, without rebuilding, and sometimes it happens). For big files it happens more frequently. Add something like 30K-line javascript to reproduce it. Unexpected end of input, unexpected token, etc., each time on a random line.

If I click build.js:1 to go to that line, dev tools close. The affected files are missing Sources tab. It I click Application tab, dev tools close.
Maybe the contributor @lhyqy5 knows what's wrong.

Could you share the corresponding project/file?

I'll make up a sample project to reproduce it.

Must be related to the copying of data between the streams: PackUriResourceHandler.cs
Related ChromoumFX docs.

@o2genum , I just corrected it. It was one nasty issue due to the way of ChromiumFx manage memory (using weak references) that may lead to hard to reproduce bug.

Ok. I suspected the copying of data in PackUriResourceHandler.cs and it is coded a bit awkwardly, indeed. It only works because ResponseLength is set.
I corrected it according to the ChromiumFX docs (the code below), maybe you can include this fix, too. This is unrelated to the issue, though.
Will you make a release for this fix?
            ReadResponse += (s2, e2) =>
            {
                if (resInfo != null)
                {
                    var buffer = new byte[e2.BytesToRead];
                    var bytesRead = resInfo.Stream.Read(buffer, 0, e2.BytesToRead);
                    System.Runtime.InteropServices.Marshal.Copy(buffer, 0, e2.DataOut, bytesRead);
                    e2.BytesRead = bytesRead;

                    if (bytesRead != 0)
                    {
                        e2.SetReturnValue(true);
                    }
                    else
                    {
                        e2.SetReturnValue(false);
                    }
                }
            };

            ProcessRequest += (s3, e3) =>
            {
                e3.SetReturnValue(true);
                e3.Callback.Continue();
            };

Are you refering to this alteration:
   if (bytesRead != 0)
   {
         e2.SetReturnValue(true);
   }
  else
   {
         e2.SetReturnValue(false);
   }

?
The new version should be available shortly.
Could you comunicate what kind of application you are creating with Neutronium?
Would you be ok to share some screenshots on Neutronium documentation to ilustre aplication made with Neutronium?

This alteration, also unnecessary Continue() in ReadResponce, also wrong order (Continue(), then SetReturnValue()) in ReadRequest and ProcessRequest.
Ok, I'll share the app description and screenshots when I got it working (we're in process of switching to Neutronium-based UI).

Ok Thanks.
I will take a look at the alteration, and commit them after review.

I updated handler based on this two references:
https://bitbucket.org/chromiumfx/chromiumfx/src/25c83dc7ef6f204e85645725bf27fdfcd639e176/ChromiumWebBrowser/WebResource.cs?at=default&fileviewer=file-view-default
https://bitbucket.org/chromiumfx/chromiumfx/issues/21/how-do-i-add-a-custom-schema
I did not find any reference on e2.SetReturnValue(false);

Ok. I only looked at what the docs say.
e.SetReturnValue(false) is for signalling end of input, but when e.ResponseLength is set, this is not necessary (Chromium stops reading before the end of input is met). So I think it's ok as it is.

Thanks for the link. I made another alteration to the original base on this doc.


The operation couldn't be completed (OSStatus error -10875)

having the same issue
but change name file, it's worked
In case you.mp3 -> In_case_you.mp3

i found this on thier community it working but u have to modifiy the native ios

Hi all, I have a question why I need to modify this in order to play the sound in my react-native application project. Otherwise, the sound does not be played. Any idea? Thanks.
sonoy/react-native-sound@cb6e4d3

@natuanorg I already have the filenames without spaces and I still have the same problem in iOS. Does it work for you still?

@natuanorg I already have the filenames without spaces and I still have the same problem in iOS. Does it work for you still?

I fixed it with white space


fix: the behavior of the FreezeCoreTransformer when a charge is present

Summary
The FreezeCoreTransformer had a bug when applied to molecules with a charge present.
Details and comments
Thanks to @Anthony-Gandon for spotting this!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.452%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
9402

Relevant Lines:
10751

💛  - Coveralls


Add dependencies to setup.py

When I went to run the example test using MDR_test_main.py, I needed these dependencies which were missing when I setup my virtual machine using the pip install mdr-library command.

Great spot @lauracbell! These packages are updated in the requirements.txt file but not in the setup.py. I will merge your changes later, but first I'll write a Github Action that updates the "pip install" package for every commit/pull request.

Included these changes in commit b6a5db0d7c3575f00e18328032174ab0d5605a5f


Analyzer Feedback from IntelliJ

Version information

IDEA AI-192.7<IP_ADDRESS>92135
2.9.0-8.2.beta
AI-192.7<IP_ADDRESS>92135, JRE 1.8.0_212-release-1586-b4-5784211x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.15.4, screens 1440x900

As it taken up from the os build directly during build, so closing it


[5] c++ code for finding longest palindromic substring in a string

solved problem #182
i added a solution for finding longest palindromic substring in a given subtring in c++ language

can you plz merge it or if you want to some changes then plz tell me

hii i changed the code with leetcode tested solution plz check it and merge it


[FLINK-10569] Remove Instance usage in FailoverRegionTest

What is the purpose of the change
Temporarily, use SimpleSlotProvider to get rid of usage of Instance. However, SimpleSlot is a legacy resource concept. A follow up should refactor SimpleSlotProvider to TestingLogicalSlotProvider or something.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector:(no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

cc @tillrohrmann

Why can't we refactor the test to ``TestingLogicalSlotProvider` now? What is blocking this effort?

@zentol Actually I did the refactor. The only possibly problematic change is I expose AllocatedSlot to public. Since it has a public constructor I guess it could be OK. If not so, we are going to add a testing SlotContext with similar functions as AllocatedSlot.

oooooki now that I see it this TestingLogicalSlotProvider business is out-of-scope for this PR. Please open a separate JIRA for that.
I will merge the first commit though.

OK I file a JIRA FLINK-11710. Is there anything I should do with thi pr? In my understanding we merge the first commit and close this pr. The second commit will be in another pr.

you don't have to do anything to this PR, I'll take care of it.


using return bool + fallback functions

https://github.com/jbaylina/eip820/blob/master/contracts/ExampleImplementer.sol#L5
pragma solidity ^0.4.18;

contract ExampleImplementer {
    function canImplementInterfaceForAddress(address addr, bytes32 interfaceHash) view public returns(bool) {
        return true;
    }
}

If I unlock the fallback function and provide some default behavior, then it would break those checks because of strange behavior by design in solidity if a method does not exist it will instead execute the fallback function, and if the fallback function does not raise an exception it will return 1 causing the check to pass.
The only solution that comes to my mind is to use some magic numbers.  Maybe we should return uint which should be more than 0.

@rstormsf nice catch.

This was actually closed in 78eb6fa62f942b36983be31772874b320415aea0.


Revert metrics changes from #843

Getting 1.4.x and master back in sync (per #932)

NetflixOSS » Hystrix » Hystrix-pull-requests #206 SUCCESS
This pull request looks good


first try with recent plugins (especially suspected spyder-terminal)

Description
What steps will reproduce the problem?

launch spyder
a loud bang occures, after 6 kernel deaths, with the window

    plugin.register_plugin()
    self.report_widget.set_html_from_file(html_path)
    with codecs.open(filename, encoding="utf-8") as file:
    file = builtins.open(filename, mode, buffering)
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\famille\\AppData\\Local\\Temp\\welcome.html'
WARNING:traitlets:kernel died: 6.001707315444946

not sure what may or may not work after that, as spyder started

What is the expected output? What do you see instead?
Please provide any additional information below
plugin.register_plugin()
self.report_widget.set_html_from_file(html_path)
with codecs.open(filename, encoding="utf-8") as file:
file = builtins.open(filename, mode, buffering)

FileNotFoundError: [Errno 2] No such file or directory: 'C:\Users\famille\AppData\Local\Temp\welcome.html'
WARNING:traitlets:kernel died: 6.001707315444946
Version and main components

Spyder Version: 3.2.0
Python Version: 3.6.2rc2
Qt Versions: 5.9.1, PyQt5 5.9 on Windows

Dependencies
pyflakes >=0.6.0 :  1.5.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.20.3 (OK)
numpy >=1.7      :  1.13.1 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.10.5 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
nbconvert >=4.0  :  5.2.1 (OK)
sympy >=0.7.3    :  None (NOK)
cython >=0.21    :  0.26 (OK)
qtconsole >=4.2.0:  4.3.0 (OK)
IPython >=4.0    :  6.1.0 (OK)
pylint >=0.25    :  1.7.2 (OK)

if I remove spyder-terminal, kernel dies two times, and I get a:
    plugin.register_plugin()
    self.report_widget.set_html_from_file(html_path)
    with codecs.open(filename, encoding="utf-8") as file:
    file = builtins.open(filename, mode, buffering)
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\famille\\AppData\\Local\\Temp\\welcome.html'
WARNING:traitlets:kernel died: 6.001142263412476

@rlaverde seems related to spyder-reports

checking by uninstalling spyder-reports ...

maybe shall I try to recompile spyder-reports from master ? https://github.com/spyder-ide/spyder-reports/issues/36

ok, it works with spyder-reports master of today

spyder-terminal works \o/ , amazing feat !

That has been @andfoy's feat! Thanks for your incredible work on spyder-terminal @andfoy!

Thanks @andfoy ! you did something stated as "impossible".


Auto expand option for context

I looked through the doc but didn't see this option available. I'd like see if there is a possibility for a contextAutoExpand option. In the tests I'm working on, the screenshot images are important references, and visually it will make more sense it they are auto expanded.

This looks like a duplicate of https://github.com/adamgruber/mochawesome/issues/217

Indeed this is a duplicate of adamgruber/mochawesome#217 I was only searching issues within this repo. Closing the issue.


Implement feedback

Coverage increased (+5.2%) to 70.213% when pulling d13c468440d7ad3cfdb4647d5575db13fcb80215 on implement-feedback into c624709c849c561e3b06aed184869dcde02197ea on develop.


raidboss: ASS/S Slippery Soap trigger calls wrong target

Description
The ASS/S Slippery Soap triggers have a bug which causes the getInFrontOfPlayer and getInFrontOfPlayerKnockback output strings to substitute the boss name (Silkie) instead of the targeted player.
Additional information
This is due to the Slippery Soap triggers being changed to use the 0x775E (ASS) and 0x7781 (ASSS) ability IDs to fix a previous bug with the trigger not firing on some Slippery Soap casts.  The changed ability IDs are self-targeted (the boss targets itself), which breaks the player callouts.
re: #4992
cc: @wexxlee

Sounds like we should go back to the previous ids and maybe just drop source entirely for that trigger if the mob name is not reliable.

That's probably the easiest fix.  I'll take a look at logs today to confirm that dropping source won't result in accidental misfires of the trigger.  Can probably submit a PR later today unless @xiashtra gets to it first.


how to read all packet from

i want to analyze the packet from tap/tun device, but only BROADCAST packet is recieved from the tun/tap device when i use iface.read method.
code as below.
package main
import (
// "encoding/json"
"fmt"
"log"
"os"
"os/exec"
// "strings"
// "fastvpn/common"
"github.com/songgao/water"
"golang.org/x/net/ipv4"

)
const (
BUFFERSIZE = 1500
MTU        = "1500"
)
func checkFatalErr(err error, msg string) {
if err != nil {
log.Println(msg)
log.Fatal(err)
}
}
func runIP(args ...string) {
cmd := exec.Command("/sbin/ip", args...)
cmd.Stderr = os.Stderr
cmd.Stdin = os.Stdin
cmd.Stdout = os.Stdout
err := cmd.Run()

if err != nil {
	log.Fatal("Error runing /sbin/ip:", err)
}

}
func main() {
config := water.Config{
DeviceType: water.TAP,
}
iface, err := water.New(config)
checkFatalErr(err, "Unable to allocate TUN interface: ")
log.Println("Interface allocated: ", iface.Name())

runIP("link", "set", "dev", iface.Name(), "mtu", MTU)
runIP("addr", "add", "<IP_ADDRESS>/24", "dev", iface.Name())
runIP("link", "set", "dev", iface.Name(), "up")

packet := make([]byte, BUFFERSIZE)

for {
           // read packaet
	plen, err := iface.Read(packet)
	fmt.Println("readed")
	if err != nil {
		fmt.Println("erro read")
		fmt.Println(err)
	}

	header, _ := ipv4.ParseHeader(packet[:plen])
	dstIP := header.Dst.String()
	srcIP := header.Src.String()
	fmt.Println(dstIP)
	fmt.Println(srcIP)
}

}
only the broadcast packet is recieved when a excute the command, as below.
ping -c1 -b <IP_ADDRESS>
my question is how can i read all packet from the tap device,everything send to the tap,like ping <IP_ADDRESS>
Tks.

yes,you are rigth,the method i test packet capture is wrong,i should ping <IP_ADDRESS>(any ip  in the subnet range except 85.).if i ping <IP_ADDRESS>，the iface.read method is work well, could be recieve packet beacuse the kernel will pass the packet to the tap device according to the route table.
thanks very much.
it's nice golang lib for develop some tool use tun/tap device.


Add support for ONLY in index creation

By default when creating an index on a partitioned table,
it will also recurse and create indexes on all the underlying
partitions. This commit adds the option to only create the index
on the partioned table. Then the partitions can be indexed individually
and attached to the parent index.
I could not find support for this in MySQL or MSSQL. In Postgres it was
added in version 11.
See also https://www.postgresql.org/docs/11/sql-createindex.html
and best practices https://www.postgresql.org/docs/14/ddl-partitioning.html#DDL-PARTITIONING-DECLARATIVE-BEST-PRACTICES

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Strange error message when compiling a function returning None

This works ok:
>>> def f():
...    print(1)
...    return 2
...
>>> ff = numba.jit(nopython=True)(f)
>>> ff()
2

but leaving out the return statement gives this:
>>> def f():
...     print(1)
...
>>> ff = numba.jit(nopython=True)(f)
>>> ff()
Traceback (most recent call last):
  File "/home/antoine/numba/numba/lowering.py", line 280, in lower_block
    self.lower_inst(inst)
  File "/home/antoine/numba/numba/lowering.py", line 316, in lower_inst
    val = self.loadvar(inst.value.name)
  File "/home/antoine/numba/numba/lowering.py", line 659, in loadvar
    ptr = self.getvar(name)
  File "/home/antoine/numba/numba/lowering.py", line 656, in getvar
    return self.varmap[name]
KeyError: '$0.5'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/antoine/numba/numba/dispatcher.py", line 157, in _compile_for_args
    return self.compile(sig)
  File "/home/antoine/numba/numba/dispatcher.py", line 277, in compile
    flags=flags, locals=self.locals)
  File "/home/antoine/numba/numba/compiler.py", line 547, in compile_extra
    return pipeline.compile_extra(func)
  File "/home/antoine/numba/numba/compiler.py", line 293, in compile_extra
    return self.compile_bytecode(bc, func_attr=self.func_attr)
  File "/home/antoine/numba/numba/compiler.py", line 301, in compile_bytecode
    return self._compile_bytecode()
  File "/home/antoine/numba/numba/compiler.py", line 534, in _compile_bytecode
    return pm.run(self.status)
  File "/home/antoine/numba/numba/compiler.py", line 191, in run
    raise patched_exception
  File "/home/antoine/numba/numba/compiler.py", line 183, in run
    res = stage()
  File "/home/antoine/numba/numba/compiler.py", line 492, in stage_nopython_backend
    return self._backend(lowerfn, objectmode=False)
  File "/home/antoine/numba/numba/compiler.py", line 447, in _backend
    lowered = lowerfn()
  File "/home/antoine/numba/numba/compiler.py", line 438, in backend_nopython_mode
    self.flags)
  File "/home/antoine/numba/numba/compiler.py", line 684, in native_lowering_stage
    lower.lower()
  File "/home/antoine/numba/numba/lowering.py", line 246, in lower
    self.lower_block(block)
  File "/home/antoine/numba/numba/lowering.py", line 285, in lower_block
    raise LoweringError(msg, inst.loc)
numba.lowering.LoweringError: Failed at nopython (nopython mode backend)
Internal error:
KeyError: '$0.5'
File "<stdin>", line 2

This seems to have been fixed somewhere along the way.


bug: special workspaces and multi monitor scaling

What
When a window is bound to a special workspace and started on a lower resolution monitor, the scaling doesn't update when opening the workspace on other monitors.
Steps

Assign an application to start in a special workspace via hyprland.conf
start the application on a lower resolution monitor
toggle the special workspace on a higher resolution monitor
You can see the window is upscaled from the lower resolution, instead of updating to the new monitors resolution

Env

hyprland-nvidia-hidpi-git from aur @ 0aeb61a95af38ad20dd4d0f8a4ec1d485d73352f

  $ cd ~/.cache/yay/hyprland-nvidia-hidpi-git/hyprland/
  $ cat packed-refs | grep "`cat HEAD | cut -d " " -f 2`"
    0aeb61a95af38ad20dd4d0f8a4ec1d485d73352f refs/heads/main

Did the patch work?

god knows, will close as inactive as the patch will defo be outdated


Write / set parameter support

It should be possible to set parameters that can be set by the API.

Implemented in 0.3.0


Possible deployment error

I installed the library from pub.dev: langchain: ^0.0.1-dev.1
I am getting a compile error, with call (and generate) not recognised as methods within the second line of:
  final OpenAI llm = OpenAI(apiKey: openaiApiKey, temperature: 0.9);
  print(await llm.call(prompt: 'x'));

I drilled through to langchain.dart, which contains only:
library

  export 'src/langchain_base.dart';

which in turn contains only:
class LangChain {
  bool get isAwesome => true;
}

The code I cloned from the 'main' branch, langchain.dart contains:
/// Build powerful LLM-based Dart/Flutter applications.
library;

export 'src/llms/base.dart';
export 'src/schema.dart';

and src/langchain_base.dart' contains code for BaseLLM etc.
Seems like what was deployed to pub.dev is behind the repository code

Just realised, you are operating just the main branch at this stage.  So it may just need a +2 release

Indeed, I just published the package in pub.dev to claim the name, but the current version is way behind master.
Maybe in the future, we can deploy nightly dev builds automatically. But for now, I would suggest you use master:
dependencies:
  langchain: ^0.0.1-dev.1
  langchain_openai: ^0.0.1-dev.1

dependency_overrides:
  langchain:
    git:
      url: https://github.com/davidmigloz/langchain_dart
      path: packages/langchain/
      branch: master
  langchain_openai:
    git:
      url: https://github.com/davidmigloz/langchain_dart
      path: packages/langchain_openai/
      branch: master

I've just pushed a new version to pub.dev (0.0.1-dev.2) with the latest changes in master, so that there's at least a working version published.


Fixed Readme containing confusing instructions for Rust 2018

the new Rust 2018 notation does not remove the need for extern crate. However the example shown does not represent that.

It's been a little while now so I'll go ahead and close this one. The current example is correct for edition = 2018 crates depending on log, but I'll try clarify the text a little more in the readme.
Thanks again @Gilgamesh22


Release nodejs-translate v2.1.1

This pull request was generated using releasetool.

Release is at https://github.com/googleapis/nodejs-translate/releases/tag/v2.1.1


reload: 'on_disable' is not documented

Hi,
This option shoud be documented, right?
https://github.com/orestbida/cookieconsent/blob/cdc934601b0df30db36ae3118c53dbd73327c4e3/demo/demo_gtm/cookieconsent-init.js#L130C30-L130C30

Sorry I was looking for it using browser search (in the Readme), but It does not work with collapses sections.


App insight cookies feature

Add a brief description of the purpose and contents of the pull request

Changes
Add a summary of the changes in this pull request
Screenshots of UI changes
Before
After
Checklist

[ ] Deploy this branch to the test environment for manual testing once comments have been resolved and checks have passed
[ ] Attach this pull request to the appropriate user story in Azure DevOps
[ ] Update the ADR decision log if needed

Superseded by #267


Use configured shell to update plugins

Request to do so in #440
However, this requires the user to have npm installed (#381) ...
Based on https://github.com/sindresorhus/shell-env/pull/3

@mike-engel Could you test if this works for your setup? 👼

Just tested again. Nice work! 👍

@rauchg have a look 👼

We should wait until https://github.com/sindresorhus/shell-env/pull/3 was merged. Because loading something directly from a GitHub branch is way too risky!

<EMAIL_ADDRESS>with ability to specify shell is out.

Thanks @sindresorhus ❤

Pingeling @leo @rauchg have a look 🦄

Thank you all for your patience and the changes made! 😘


Allow the component to be mounted multiple times

Experienced the same issue as described in https://github.com/patw0929/react-intl-tel-input/issues/73.
Removing EventListeners during component unmount seems to fix this problem.

Thanks! Merged manually in v2.2.4.


fix amp-bind sample

Fixes #1816

@sebastianbenz  Can you please merge it. Need this merge very badly. Thanks


Run github cron workflow to update deps

Hi @gerrymanoim, I've just updated the dependencies in #227.
tbh, I'd question if employing dependabot to maintain the requirements files is worthwhile. # 227 updated 16 or so dependencies - if dependabot were to have managed it that would have been (I believe) 16 separate commits clogging up the commit history.
Manually updating the dependencies is a simple enough affair - just commit the changes from the following:
$ pip-compile --upgrade --output-file=etc/requirements.txt pyproject.toml
$ pip-compile --upgrade --extra=dev --output-file=etc/requirements_dev.txt pyproject.toml
$ pip-compile --upgrade --upgrade-package pandas==1.1 --extra=dev --output-file=etc/requirements_minpandas.txt pyproject.toml

Maybe just use dependabot for github actions and update the requirements manually once a month or so?
Perhaps it would even be possible to write a github actions workflow to run the above and raise a PR once a month with the resulting changes?
Originally posted by @maread99 in https://github.com/gerrymanoim/exchange_calendars/issues/217#issuecomment-1236887435

I'm now thinking the ideal would be to do as above, although additionally have dependabot set up for just the requirements.txt dependencies (not the dev). I wouldn't suggest merging the PRs that dependabot raises (rely instead on the above), but setting up dependabot for just the main requirements gives an immediate heads up if a dependency upgrade has broken exchange_calendars (the tests for the dependabot PR will show as failing). Otherwise we could go a month without any reason to suspect that the package is broken.
@gerrymanoim, as I think you've mentioned before, looks like setting up dependabot for just the requirements.txt would necessitate having this file in a separate directory to the rest of the requirements. Perhaps move requirements.txt out of etc/ and into the root directory? (Would need to revise all the workflows that use the requirements files.)

Note that dependabot now supports grouped updates by just dropping a config file into the repo - blog post - docs - example


Issue 1131

Types of Changes
Prerequisites
Please make sure you can check the following two boxes:

[x] I have read the CONTRIBUTING document
[x] My code follows the code style of this project

Contribution Type
What types of changes does your code introduce? Put an x in all the boxes that apply:

[x] Bug fix (non-breaking change which fixes an issue, please reference the issue id)
[x] New feature (non-breaking change which adds functionality, make sure to open an associated issue first)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[x] I have added tests to cover my changes
[x] All new and existing tests passed

Description
The PrettyMarkupPrinter now accepts one parameter which is: nodes of the document that their text format shall be kept. Also did some minor refactoring, all tests have passed succesfully.  The last

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: dramlian:x: DamianDamian seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

So now it asks for CLA from my unauthorized commits, well another PR it is then and I am closing this one.


Containerize infrastructure development environment

Description
Currently the DevOps environment; azure CLI and Terraform are installed locally on the developers machine. This can lead to variations in versions, and slower on ramp for new developers
Proposed solution
Create a minimal container which provides a consistent infrastructure development environment for the team.
Additional context

@rin-skylight @jdorothy Thoughts on prioritizing and fleshing out additional details for this work?

Discussed at engineering sync on 2/10. Given that we don't want to be running TF apply from outside deployments anyways, it's probably not worth investing our time in this.


xRemoteDesktopAdmin: Add Unit Test

Pull Request (PR) description
This PR adds the Unit test from the abandoned PR #18
This Pull Request (PR) fixes the following issues
None
Task list

[ ] Added an entry under the Unreleased section of the change log in the README.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[ ] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #25 into dev will increase coverage by 97.87%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             dev    #25       +/-   ##
========================================
+ Coverage   2.12%   100%   +97.87%
========================================
  Files          1      1
  Lines         47     47
========================================
+ Hits           1     47       +46
+ Misses        46      0       -46

Impacted Files
Coverage Δ

...urces/xRemoteDesktopAdmin/xRemoteDesktopAdmin.psm1
100% <0%> (+97.87%)
: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 c3fd03e...f6ba73f. Read the comment docs.

Superseded by PowerShell/ComputerManagementDsc#227


[HELP] how to fix incorrect word spacing like this?

I got incorrect word spacing with unRaid(6.11.5) which uses ttyd provide the webterminal window.
And the build-in ttyd version is 1.6.3
relate softwares and version:

unraid: 6.11.5
ttyd: 1.6.3
browser: firefox 124.0.1

I also encountered the same problem.

https://github.com/tsl0922/ttyd/wiki/Client-Options

I fixed it by use noMeiryoUI change win11 system font from Microsoft YaHei UI to  PingFang.


Custom queries location in Linux

Two ways to add custom queries:
/root/.config/bloodhound/customqueries.json
While in Neo4j, drop down the main menu > Queries > at the bottom, click the pencil icon (opens the customqueries.json in a text editor) > paste the json contents and save > refresh the Custom Queries list with the circular arrow icon.

Your first method should more generally be described as:
~/.config/bloodhound/customqueries.json
In this way, it works for more than just the root user.


Kill-switch broken?

I understand that the kill-switch is a way to protect checkwatt's servers, which is great, as stated in #69. I have the same issue as #78 though, and checking the https://checkwatt.se/ha-killswitch.txt link, it returns a php-page. Not a 1 or a 0. Can checkwatt accidentally have broken it? And is it individual or the same for all of us using ha-checkwatt?

Kill-switch is generic, happens before login. If file is absent or content is 1, pyCheckwatt stops.
The fix for this is to remove check for killswitch and release pyCheckwatt and hs-checkwatt.
https://github.com/faanskit/pyCheckwatt/blob/f874a99429a4f1efd784652008e4d962a5ddcc39/pycheckwatt/__init__.py#L230
Or...contact Checkwatt to ask them to bring the file back

Checkwatt confirmed thet removed it by mistake last week. They will re-insert it.


[Question] Static Field Values

Is your feature request related to a problem? Please describe.
When I decompile and compile the same apk and compare with a diff tool, some smali files have static field values removed. Can this be fixed?

Describe the solution you'd like
There must be no changes between the same apk files after recompilation and decompilation

Thanks for report.
I couldn't replicate this. I even wrote a test to double check if was platform related. I get the values on each disassemble. Linked PR of adding into test suite.
Closing.


Add support for Sombra

Data required:

abilities
achievements
hero_sub_roles
heroes
rewards

Hero added in #32 and abilities in #33

Rewards added in #34


Fix HTML5 errors in theme templates

Fixed #54

Changes Unknown when pulling d0ad901af491dfbdc9a6777ee547465799233ac2 on issue/54 into ** on master**.


Automatically changing track and car for the Hot-Lap-Challenge

Is your feature request related to a problem? Please describe.
Right now the link to the leaderboard is hard-coded and needs to be changed manually. This should be automated with !changeTrack Link. This feature could also include an image of the new track and/or car
Describe the solution you'd like
Instead of going into the code and changing the link manually there should be a command which automatically reads out the car and track from the link and changes the info using the required info
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

This feature was added here: https://github.com/Akashic101/theGameMaster/commit/0887fd68d5fff77363d69cf30dfcc44a7f2e9ccf
The user can now change the Hot-Lap-Challenge using  .updateHLC link. The required info gets automatically grabbed and is displayed in an embedMesssage for easier visualization


Cannot configure TX2 images: "Partition not found: 12."

https://sentry.io/resinio/cli/issues/297830328/
This seems to be an issue in resin-device-init, or resin-imagefs: the image does have a partition 12 (according to fdisk), but we aren't finding it, probably something to do with GPT.

Further notes:

resin-image-fs right now does not support GPT
It does partition lookup using partitioninfo v2
partitioninfo v3 does support GPT, but is a breaking change, and we'll need to upgrade
Once that's done, finding the partition itself should be ok: not clear yet whether resin-image-fs will be able to read it once it's found it though, tbc.

Fixed in resin-device-init 4.2.0, which should be picked up by all new CLI installations.


Improve error message for already registered members

This now also shows the name. It was tedious to find the conflicting add call without this.

Current coverage is 89.25% (diff: 80.00%)

Merging #48 into master will not change coverage

@@             master        #48   diff @@
==========================================
  Files            28         28
  Lines          2923       2923
  Methods         879        878     -1
  Messages          0          0
  Branches        304        305     +1
==========================================
  Hits           2609       2609
  Misses           56         56
  Partials        258        258

Powered by Codecov. Last update bb9d814...e80dda1

Thank you.


Easier installation

Monarch seems to be entirely written in dart so why not use the dart package ecosystem. Instead of the long and cumbersome tutorial on https://monarchapp.io/docs/install. Why not publish a monarch package that can be installed with pub global activate monarch?

Monarch is a combination of applications: a CLI executable, a native desktop app, and a Flutter app. There is also the monarch flutter package. The native desktop apps are written in Swift for macOS, C++ for Windows, and probably will be written in C++ for Linux. pub.dev is not meant to distribute desktop app executables.
In the future we will look at simpler options like Homebrew, Chocolatey or the App Stores of each platform. We couldn't do Homebrew because Homebrew doesn't distribute brand-new projects. Also, flutter desktop was so new that the App Stores were not an option.
Stay tuned though, making the install easier is in our roadmap.

Alright thanks for the info. Awesome work so far keep it up!


Added required star to FormLabel.

I think this is a good feature but will understand if it is not wanted.  When the "required" field is specified in uibinder then a star icon is displayed following the label.
I like the way the icon is added better than the HelpBlock implementation but it is a more complicated implementation.
Here is a screenshot:

Let me know what you think.

Can you make it optional? It might not be in some people's UI/UX to have that, maybe give a way to opt out?

Yeah...its optional in the fact that if you don't include "required" in uibinder then it defaults to not required. Is that what you meant?

Oh I meant as an additional flag like "showRequiredIndicator". Some might want to have the required flag for validation but not show the asterix.

Oh.  No worries there.  It applies only to the FormLabel and is
independent of validation.  You could have a star and no validation if
you wanted or no star and validation. Should be very flexible.
On 02/12/2015 05:44 PM, Joshua Godi wrote:

Oh I meant as an additional flag like "showRequiredIndicator". Some
might want to have the required flag for validation but not show the
asterix.
—
Reply to this email directly or view it on GitHub
https://github.com/gwtbootstrap3/gwtbootstrap3/pull/312#issuecomment-74184930.

IMPORTANT: This email does not constitute a contract or an offer or acceptance of an offer to enter into a contract.  Further, this email may not be used to modify, supplement, novate, or waive any rights with respect to an existing contract or other binding commercial terms.

I think I will change the name to showRequiredIndicator like you said to avoid confusion.
On February 12, 2015 5:44:39 PM MST, Joshua Godi<EMAIL_ADDRESS>wrote:

Oh I meant as an additional flag like "showRequiredIndicator". Some
might want to have the required flag for validation but not show the
asterix.

Reply to this email directly or view it on GitHub:
https://github.com/gwtbootstrap3/gwtbootstrap3/pull/312#issuecomment-74184930

--
Sent from Kaiten Mail. Please excuse my brevity.
IMPORTANT: This email does not constitute a contract or an offer or acceptance of an offer to enter into a contract.  Further, this email may not be used to modify, supplement, novate, or waive any rights with respect to an existing contract or other binding commercial terms.

Should be good to go.


Content Security Policy missing

https://www.dareboost.com/en/report/d_15d773f2190b99843ddb4b550?reportIds=d_15d773f2190b99843ddb4b550

@dhellmann @iranzo I'm not 100% on what needs to be done for this. Are either of you familiar with it?

Nothing on your side, sorry! it's server-side


Why string instead of []byte?

Is there any reason why the parser's Unmarshal function takes a string instead of a []byte like other Go marshalers?
I think there might be a performance penalty with using a string.
Also, would it ever make sense for Marshal to return an error? I haven't read the sdp spec, but I imagine there are some length limits or something.

@maxhawkins I think the reason I did it as string was because of backward compatibility with the existing code. This was done a while back.


vagrant: test on f31

Fixes #181
Just making sure that bender works on f31. Since F31 has cgroup v2, the vagrant
VM needs to be rebooted and switched back to v1 so we can run all the tests. I
fear this won't be possible to do in CI.

/packit test

/packit test

Merging, the tests are passing for me locally in a clean vagrant VM


Multi-Plant AEP Analysis using Spark Map

This pull request represents the multi-plant AEP analysis work that was completed for Q1 FY23. Features include:

Support for computing the AEP for multiple wind plants in parallel with new entr toolkit function entr.aep_spark_map
03_multiple_plants_using_map_entr.ipynb notebook demonstrating this functionality.
Updated 02_plant_aep_analysis_entr to use OpenOA V3 api.

Merging this PR for now. I will create issues that correspond to Eric's comments.


pull request for e2e test at 12/30/2016 02:45:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add primitive list support

What does this PR do?
Add support to dataspec to split batches consisting of primitive types, e.g. a list of strings as prompts.
What issue(s) does this change relate to?
CO-1686

while you're here, can you also add a test (and support if necessary) for having a single string in the batch?
e.g.
{'input_ids': <tensor with batch dimension>, 'labels': <tensor with batch dimension>, 'metadata': <single string>}

was on my list to check if that errors or not/add support

Looks good. Just had a couple nits 😉

I will defiNITely approve the PR once those are fixed


Smarter way to export CSL date override

for biblatex: Test whether string in {:issued: …}, {:accessed: …} etc. matches a biblatex (=EDTF Level 0) date or date range. (Biblatex manual: “The date fields … require a date specification in yyyy-mm-dd format. Date ranges are given as yyyy-mm-dd/yyyy-mm-dd. Partial dates are valid provided that date components are omitted at the end only. You may specify an open ended date range by giving the range separator and omitting the end date (e. g., yyyy/).”) When matching, export to the biblatex date field; if not, export to the biblatex year field (which, unlike date, can be used for literal/free-form date strings).
for “Pandoc JSON”: Same test. When matching, reformat date or date range according to CSL specs, e.g., { "issued": { "date-parts": [ [ 2014, 12, 31 ], [ 2015, 01, 01 ] ] } }; if not, export to a “literal” element, e.g., { "issued": { "literal": "13th century" } }. See citeproc-js specs.

For Pandoc JSON I simply export the CSL-JSON that Zotero generates, with the added change that the fields extracted from the 'extra' field are set on the generated JSON. I should perhaps just not do that for the CSL-style fields; that is perhaps better handled by the CSL processor itself.

… that is perhaps better handled by the CSL processor itself.

No, since pandoc can only deal with CSL JSON that strictly follows the specs; it neither understands the ad hoc {:blah: blah} elements in “Extra”, nor "issued": "2014-12-31/2015-01-01". It’d have to be { "issued": { "date-parts": [ [ 2014, 12, 31 ], [ 2015, 01, 01 ] ] } } and { "issued": { "literal": "13th century" } }.

@nickbart1980: I can find the part on the date field formatting, but not the part that says "otherwise, use the year field". The bibLaTeX manual says that "year" (and even "month") are legacy fallbacks to "date", implying the same rules go for those.

WRT pandoc: alright, then I'll parse them out myself.

BTW, I meant: the CSL engine inside Zotero already does this kind of thing (albeit not perfectly, see https://github.com/zotero/zotero/issues/845#issuecomment-141894703), but I can do the parsing myself as a workaround.

Re: biblatex date vs. year fields: A well-kept secret … but just try it, literal stuff in the date field just won't work, but it does when in the year field. The biblatex-chicago manual is aware of this: “year – Standard biblatex field. It usually identifies the year of publication, though unlike the date field it allows non-numeric input, so you can put “n.d.” (or, to be language agnostic, \bibstring{nodate}) here if required, or indeed any other sort of non-numerical date information. If you can guess the date then you can include that guess in square brackets instead of, or after, the “n.d.” abbreviation.”

I need to take into account #245; the Zotero date parser is quite enthusiastic in that it will do its darnedest  to get out a date of whatever has been put in. I don't want to demand people input dates in EDTF always, but I'd also like to acknowledge the difference between [year/month] (most likely literal) year/month and date/date. The latter two are ambiguous of course, and I haven't yet found a good heuristic/marker.

(I mean 11/2012 is ambiguous between november 2012 and a date range)

Wait, the brackets is an official thing? Would [1802]/[1833] be a meaningful year value?

Would [1802]/[1833] be a meaningful year value?

No, not at all. In biblatex you should use date and origdate; in CSL issued and original-date – and let the style decide on formatting (brackets or whatever).

(I mean 11/2012 is ambiguous between november 2012 and a date range)

That’s why I’m all for using EDTF Level 0 exclusively when dealing with dates in Zotero’s “Extra” field.

Oh yeah sure in the extra field, but I'm also thinking about stuff entered in the regular date fields.
But [1802]/[1833] would be a date-range, right? Not an date/origdate.

The brackets being an official thing would mean I could take out the configurable I put in place for #245.

If I wanted to express a date range, i'd use [1802/1833].
But the real problem is that Zotero's current date parsing is so weird. [1802]/[1833] in the Date field is parsed and exported as 1802; 2014-12-31/2015-02-03 as 2014-12-31.
My hope was that with {:issued:2014-12-31/2015-02-03} in the “Extra” field, BBT might be able to export a date range to CSL JSON so that it can be used with pandoc – but if I understand you correctly and BBT might also export Zotero's “Date” field in a more sensible manner (at least if it contains EDTF Level 0 dates and ranges) that'd be fantastic.

… the brackets is an official thing?

Ok, now I read https://github.com/ZotPlus/zotero-better-bibtex/issues/245, and I'd still say, no, brackets are never “official”, neiter in biblatex nor in a CSL context. However, if a user enters brackets, or anything else that does not match a well-defined date spec into a date field, the field content should be treated as a literal string, which is possible in both biblatex and CSL. Zotero’s parsing, of course, is clearly overzealous here, but if you have found a way of bypassing this and having BBT parse/export date fields more sensibly, that's great.
Now, if BBT managed to export Zotero's "Date" field to CSL JSON as described earlier, date-parts if it's an EDTF Level 0 date or range, literal otherwise, we could even forget about dealing with {:issued:} in Zotero's "Extra" field altogether since that'd no longer be necessary.

The EDTF date/ranges for the date field is easy, so I'm going to do that regardless. I'm pondering how to do it more generally, given the plethora of ways dates find their way into Zotero.
The Zotero date parser does its best. Can't fault them for that. The assumption behind it is that you entered a date, and it tries to find it.

The EDTF date/ranges for the date field is easy, so I'm going to do that regardless.

Fantastic, looking forward.

Some at Zotero have expressed a strong dislike of the {:csl-variable: content} format. They prefer a simpler, newline-separated csl-variable: content format for the ‘Extra’ field, so you might want to cater for both of these.
Examples:
Foo bar {:original-date: 1888-09-18} {:medium:DVD} baz

and
Foo bar
original-date: 1888-09-18
medium: DVD
baz

This already works, no? Not all CSL vars are present, since I add them when I encounter them and understand their mapping, but a lot of then are in already. Except for pandoc-CSL, where that list is less complete; for this I've created #354.

No, the second, newline-separated csl-variable: content format is not parsed out. This is with ‘pandoc CSL’ and BBT 1.4.6.

It depends on the CSL variable. Which are you missing (please add on #354)

Ah, I see, sorry. I tried original-date. WIll continue on https://github.com/ZotPlus/zotero-better-bibtex/issues/354.


Source.setMinimumTileUpdateInterval() is not working

Describe the bug
As MapTiler doesn't provide the Cache-Control HTTP response header for every tileset, I wanted to override it in the code in my Android app. What seems to be made for that is Source.setMinimumTileUpdateInterval(), but it doesn't seem to work. After setting this value to 14400000, the MapLibre still makes duplicate network requests every time I pan/zoom the map.
To Reproduce
Steps to reproduce the behavior:

Set up MapLibre on Android (I didn't test on iOS) with any style source that doesn't have a Cache-Control HTTP response header, for example, this style has a source like that: https://api.maptiler.com/maps/openstreetmap/style.json?key={api_key} (I don't know if it was on purpose or not, but I wrote to them about that, so they may update it in the future), you can get the free API key on maptiler.com
Change minimumTileUpdateInterval after the style is loaded:

style.getSourceAs<VectorSource>("openmaptiles")?.apply {
    minimumTileUpdateInterval = 14400000
}

Pan around, zoom in, zoom out (to capture the same tiles multiple times), and check logs. Look for:
[HTTP] Request was successful (code = 200).
[HTTP] This request was cancelled (request url)
It keeps requesting the same tiles every time you view this tile, so it seems like minimumTileUpdateInterval is 0.

Expected behavior
After setting minimumTileUpdateInterval to 14400000, MapLibre should keep tiles in the cache for 14400000 milliseconds and not make any more requests for the same tiles until this time passes
Platform information (please complete the following information):

OS: Android
Platform: Android
Version: API 29, API 25 (emulator)
Device: Xiaomi Redmi Note 7, Pixel 7 Pro (emulator)

Just had a quick look, if this method is called every time a tile is needed
https://github.com/maplibre/maplibre-native/blob/8c940cb7a65ad4c865cc51d1b8ce88dd425c52b8/platform/default/src/mbgl/storage/online_file_source.cpp#L478-L485
Then a Cache-Control HTTP header would be needed to use this functionality.

Is it supposed to be like that? Seems like this function is made to bypass configuration set by tileserver, but it doesn't in this case.

Yes, I think so. From the changelog:

Introduced Source.minimumTileUpdateInterval to override the expiration time set via HTTP headers with a longer expiration time on the client.


copy frida_input_data to uncertainty dashboard data folder

model errors out with out this file present

on a related note, we should ask the chrisses to provide more years in this file, for running the model to further into the future


Add Coq to the list of programming languages.

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[ ] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[x] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[x] Content (and my changes are in index.md)

I am a maintainer of the Coq project, but the contributing guide seems to imply that the "no affiliation" criterion is only relevant when creating a new topic or collection. Note that the Coq topic already exists: https://github.com/topics/coq
I chose the position to insert Coq in the list based on the number of stars.

Hi @Zimmi48 ,
Thanks for starting this pull request. In order to review it for acceptance, we need to make sure that all of the prerequisites are satisfied. I see that you didn't check this:

I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee)

This is a requirement so that we can maintain a high level of independence in this project. Please update if you are able to verify that you meet that requirement.
Thanks!

I am a maintainer of the Coq project, but the contributing guide seems to imply that the "no affiliation" criterion is only relevant when creating a new topic or collection. Note that the Coq topic already exists: https://github.com/topics/coq

You should probably rephrase your contributing guide then.

@Zimmi48 thanks for the reply. You raise a good point and I think we should be more clear with our guidance in the contribution docs.
In light of that, I think I'll open a PR to update it to explicitly only disallow new topics, since the addition of collections doesn't really apply to projects (they are broader than a single project).
Could you edit this PR to add the entry to the end of the list and I can get it merged in once CI passes.
Thank you!

@kenyonj I've moved the entry down the list. However, I'd like to highlight the fact that the ordering that is displayed in https://github.com/collections/programming-languages currently feels weird:

The ordering is the same as the one in the collection file.
Which appears to have initially been based on the number of stars of each project (#484), but numbers have varied since then.

So this results in a list like this:

Go (63000 stars)
Swift (49000 stars)
TypeScript (53000 stars)

And later on:
10. CoffeeScript (15300 stars)
11. Elixir (15800 stars)
12. Crystal (13800 stars)
13. PowerShell (15400 stars)

The ordering is the same as the one in the collection file.

That is correct, we currently display only based on the listing order. Collections can contain a few different item types, not just repos. It's on our roadmap to look into a more dynamic way of displaying/sorting the items, but for now, it's only based on the list order.


Feature/wait cursor

Coverage decreased (-0.03%) to 81.447% when pulling ceba3a6be39d84064a0400ad85c8af599a6ff4ed on numberpi:feature/waitCursor into 90547fc4c7edfe6f071220d4205f72992dee799f on HPI-SWA-Teaching:master.


Make configure show it's looking for "pixman", not "xrdp_pixman"

The pixman library in not part of xrdp. If users see that "xrdp_pixman"
is not found, they might assume it is some xrdp specific library.
The Automake conditional is still XRDP_PIXMAN, but the pkg-config
variables become PIXMAN_LIBS and PIXMAN_CFLAGS.

+1 I think this can be included to 0.9.1.

+1


Add initial code structure

This PR creates a basic skeleton of the projects based on two modules:

API
spec

Where I did not create TCK project/module yet; however, it will be as a module.
This PR solves those PRs:

https://github.com/jakartaee/data-api/issues/3
https://github.com/jakartaee/data-api/issues/6
https://github.com/jakartaee/data-api/issues/8

:warning: it is a baby-step the repository issue is still open

@njr-11 could you review and if it makes sense approve it?

@njr-11 I merged your commits on this PR.
There are several TODO but this PR is already big enough.


Text node types

In addition to graph types, individual text nodes should allow a type to be specified. Maybe hidden by default.

Implemented this a while back.


Attaching to a remote process from Windows to Linux

Hey, I am trying to attach render doc to my remote linux machine to try out the debugger. However, I have not done any sort of remote debugging before. Can you explain to me what I need to do to get remote debugging working? Thanks.

This setup is a bit unfriendly and possibly buggy since it hasn't really been supported in the past.
It's something I'm actively looking at improving and smoothing out the workflow though, so there will be some proper documentation and instructions arriving soon.

The workflow for this has been implemented in v0.31 - see the documentation or a short video I recorded showing how it fits together.


added support for carrier gateway

Added

Added support for carrier gateway

APIs Added

CreateCarrierGateway
DescribeCarrierGateway
DeleteCarrierGateway
CarrierGatewayNotFound - Exception

Terraform Test Passed

TestAccAWSEc2CarrierGateway
(Sub Test Cases)

TestAccAWSEc2CarrierGateway_basic
TestAccAWSEc2CarrierGateway_Tags
TestAccAWSEc2CarrierGateway_disappears

Codecov Report

Merging #4280 (84834bc) into master (99c6617) will decrease coverage by 0.07%.
The diff coverage is 37.68%.

@@            Coverage Diff             @@
##           master    #4280      +/-   ##
==========================================
- Coverage   94.97%   94.90%   -0.08%
==========================================
  Files         486      487       +1
  Lines       54814    54883      +69
==========================================
+ Hits        52059    52085      +26
- Misses       2755     2798      +43

Flag
Coverage Δ

servertests
53.04% <37.68%> (-0.02%)
:arrow_down:

test_responses
32.74% <37.68%> (+<0.01%)
:arrow_up:

unittests
94.80% <37.68%> (-0.08%)
:arrow_down:

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

Impacted Files
Coverage Δ

moto/ec2/models.py
94.09% <33.33%> (-0.58%)
:arrow_down:

moto/ec2/responses/carrier_gateways.py
37.50% <37.50%> (ø)

moto/ec2/utils.py
92.24% <50.00%> (-0.22%)
:arrow_down:

moto/ec2/exceptions.py
97.60% <66.66%> (-0.38%)
:arrow_down:

moto/ec2/responses/__init__.py
100.00% <100.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 99c6617...84834bc. Read the comment docs.

@bblommers, this pr is good ready to merge.


Algorithm parameters lost on fallback

I have noticed that the algorithm parameters get lost when using the fallback to encrypt/decrypt mode. This doesn't work for example with RSA OAEP when non default parameters are used.

Thanks for the patch, nice pickup! Merged.

This is now up in https://www.bouncycastle.org/betas 171 b04 or later.


Fixed and tidied combinatorics.list_permutation function

Hi, hope this is okay.

The problem
I noticed an error in the implementation of the combinatorics.list_permutation function, when provided with lists containing duplicate items. For example (printing each permutation on a new line):
// (Old behaviour)
// [1, 1, 2] |> list_permutation
[1, 2]
[1, 2]
[2, 1]
[2, 1]

Clearly, none of those are permutations of [1, 1, 2]! It did work correctly when given lists with unique elements:
// [1, 1, 2] |> list.unique |> list_permutation
[1, 2]
[2, 1]

The issue was the use of filter on line 357. This removed from the remaining list all duplicates of x, when it shouldn't have.

New behaviour
When fixing this, I considered two reasonable behaviours for list_permutations:

A: Treat duplicate elements as distinct for the purposes of permutations. For example, the list [1, 1] has two permutations... [1, 1] (the same way round) and [1, 1] (the other way round). With this behaviour all lists of the same length will have the same number of permutations, namely factorial(list.length(whichever_list)).
B: Treat duplicate elements as identical for the purposes of permutations. In this case, the list [1, 1] has only one permutation... itself. With this behaviour lists with duplicate elements will have fewer permutations than other lists of the same length.

Both behaviours are plausibly useful. I have decided, for this pull request, to implement behaviour A. Partly because it was a simpler change to write, but also because I think it is slightly more versatile: if you need behaviour B it is as simple as calling the function with behaviour A, and then piping the result to list.unique:
// (New behaviour)
// [1, 1, 2] |> list_permutation
[1, 1, 2]
[1, 2, 1]
[1, 1, 2]
[1, 2, 1]
[2, 1, 1]
[2, 1, 1]

// (Simulating behaviour B)
// [1, 1, 2] |> list_permutation |> list.unique
[1, 1, 2]
[1, 2, 1]
[2, 1, 1]

To implement this new behaviour, I switched from filter to pop. This always removes exactly one instance of x, leaving its duplicates in the remainder of the list. One difficulty is that pop returns a result in case the element it's searching for isn't in the list. But it always will be (as x is drawn from the list in the line above), so we can be sure to get an Ok(...) and I use a let assert expression to destructure this.
This is not exactly the most efficient way to write this function, but it has the same performance as what was there before, and is fairly readable. I would be happy to write a more efficient version if needed.

Other changes
I made a few other code changes at the same time. These do not have any effect on the behaviour of the function.

I noticed we were using local definitions of flat_map and concat for the implementation of list_permutation (and nothing else). As these are standard library functions (in the list module, which we already import) we don't need to duplicate them locally, so I got rid of them.
I rewrote the call to flat_map on line 354 to use use. Personally, I find this more readable.

To do
I have not yet:

Added the above examples (or similar) as tests.
Updated the documentation to explain the behaviour with duplicates.

Hey Daniel thanks for this! I've converted the PR to draft, please can you address the todos (tests and docs).
I'll defer to @NicklasXYZ on whether this change is sensible but it seems reasonable to me. We could also consider making the behaviour configurable with a second parameter:
combinatorics.list_permutation([1, 1, 2], ignore_duplicates: False)
// [1, 1, 2]
// [1, 2, 1]
// [1, 1, 2]
// [1, 2, 1]
// [2, 1, 1]
// [2, 1, 1]

combinatorics.list_permutation([1, 1, 2], ignore_duplicates: True)
// [1, 1, 2]
// [1, 2, 1]
// [2, 1, 1]

This might make the implementation a bit clunkier but it would definitely be more performant than suggesting folks iterate the list again with list.unique. I don't feel super strongly about this - maybe we could wait and see if/when Gleam gets optional args.

This is not exactly the most efficient way to write this function, but it has the same performance as what was there before, and is fairly readable. I would be happy to write a more efficient version if needed.

If a more efficient version is fairly simple to implement I think we should favour it over readability: perf wins for number-crunching stuff is likely to be quite high impact!

To implement this new behaviour, I switched from filter to pop. This always removes exactly one instance of x, leaving its duplicates in the remainder of the list. One difficulty is that pop returns a result in case the element it's searching for isn't in the list. But it always will be (as x is drawn from the list in the line above), so we can be sure to get an Ok(...) and I use a let assert expression to destructure this.

Might be good to just leave this as a comment above the assertion too so folks in the future know this assertion is safe!

I am sorry for keeping you waiting, I will get to this very soon :sweat_smile:

Thank you @DanielSherlock, thank you for fixing this and @hayleigh-dot-dev  for already having a look.
I think behavior A that you are describing is a reasonable default choice to go with, but in the future it could be cool to have both behaviors implemented as @hayleigh-dot-dev also points out.
If you could update the tests and documentation I'd be happy to approve this.
With respect to the performance of the implementations, of course if things can be improved in that department as well, I would not turn down such improvements ;)

No worries, I'm not in any rush (and I hope you aren't too)!
Actually, I thought after posting this pull request: Oh before I start writing tests and docs I should go and look at the standard (or standard-ish) libraries of other languages to see how they handle this case. Then I got distracted. And now it is now.
As far as I can tell, it's like this:

Language
Behaviour A
Behaviour B
Iterator

Javascript (Deno)
permutations

Ruby
permutation

✅

Scala

permutations
✅

Swift
permutations
uniquePermutations
✅   (Sequence)

Julia
permutations
multiset_permutations
✅

Rust
permutations

✅

Nim

nextPermutation
✅   (mutating)

C++

next_permutation
✅   (mutating)

Haskell
permutations

✅   (everything is lazy)

Python
permutations

✅

Clojure

permutations
✅   (lazy-seq)

(I had hoped to find a list-permuting function in the standard (or standard-ish) libraries of Erlang or Elixir, and prioritise similarity to that. But no luck, not even Elixir's Math library has this. Other languages that seem to have nothing (semi-)official include Java, Elm, and Lobster. Though perhaps I wasn't looking hard enough.)
Personally, I like the way Swift handles it. Plus, their documentation is clear (with examples) and also has some hints for possible efficient implementations. The Clojure implementation seem nice and clean too, and might be easy to port. Not all other documentation was similarly clear...
Another thing to note: Almost all of the languages I surveyed returned an iterator (or equivalent thing in that language), even when given a list/array/etc. If we are going to be worried about performance, we might consider doing the same. This after all exactly the sort of function where iterators make a lot of sense:

It does not take a particularly large input list for the output list to be very very large indeed.
The state needed to generate the next permutation is small.

I think we could discuss returning an iterator in a separate PR. It'd be a breaking change and just generally beyond the scope of what's already here. Better to merge incremental improvements to the library than spend a lot of back-and-forth making a single PR cover everything all at once!

Thanks a lot for the Table Daniel. It's a really nice overview and you are right. Returning an iterator would be the way to go for sure :)
I think what @hayleigh-dot-dev  suggests sounds very sensible.  It would be a larger change and thus probably more suitable to discuss in another PR.

Very practical suggestions, thanks.
I've added tests and docs and an explanatory comment to the current implementation.

[x]  I ran the tests and it seems to work fine.
[x]  I built the docs locally and nothing looks wrong with them.
[x]  I ran gleam format but it didn't do anything, so I hope that means it's all fine.

When running this, I do get loads of warnings - it seems from Gleam's recent type import syntax change. I've not gone and fixed all of those in this PR.
I'm happy to create separate issues for:

[ ]  Another function implementing behaviour B.
[ ]  A perhaps slightly more efficient implementation.
[ ]  The possibility of returning an iterator.

Looks good to me. I'd be happy, if you could add the other issues :)!


"microk8s enable kubeflow" command failing

Hello, I have tried to deploy kubeflow on microk8s on a g4dn.2xlarge on AWS.
The command fails. See screenshot below.

@xxl007: Can you run microk8s inspect, and attach the tarball here that it generates?


Render templates in immutable sandbox

Chat templates are rendered with ImmutableSandboxedEnvironment in transformers so no need to do otherwise here.

Heh, what a fluke with dual PR, just noticed the discrepancy when double checking rendering with transformers...
BTW, the security PR left loader=jinja2.BaseLoader() in, it doesn't really make a difference, but it's not necessary since we're always using .from_string() which will not invoke a loader.


Allow searching by seed?

There's no wiki yet, I don't want to have to update 2 documents when adding features.
There is in-app documentation for that.
https://github.com/RupertAvery/DiffusionToolkit/blob/master/Diffusion.Toolkit/Tips.md#geting-started
Basically add seed:  in your query or seed: - to search a range of seeds