JCS dependency tries to load log4j2 backend

Describe the bug
dependencycheck depends on jcs (caching library) which seems to use some facade for logging. It seems it uses service loader to find log backend implementation, and the log4j2 backend seems to be included, but as the log4j2 classes are not present, JCS output a full stack trace as a debug message before trying the next backend
Version of dependency-check used
The problem occurs using version 11.1.1 of the maven plugin
Log file
https://gist.github.com/laurentgo/cb678fb2bdfc40efdf57693f90f7a644
To Reproduce
Steps to reproduce the behavior:

Run a plugin goal with Maven debug logs enabled

Expected behavior
No debug message
Additional context
it's a debug message, so the issue is minor but it may be a sign that log configuration need to be checked.

Hey Would Like to pick up this issue and fix the bug you are facing here

We use jcs3-slf4j. How this works is documented on the JCS side here: https://commons.apache.org/proper/commons-jcs/UpgradingFrom2x.html#Logging_Abstraction
The LogFactories are loaded here: https://github.com/apache/commons-jcs/blob/c01db5a678d24f72840186bc392d4e3778038e8e/commons-jcs3-core/src/main/java/org/apache/commons/jcs3/log/LogManager.java#L48
Somehow I doubt the fix is in the dependency-check project itself.

We use jcs3-slf4j. How this works is documented on the JCS side here: https://commons.apache.org/proper/commons-jcs/UpgradingFrom2x.html#Logging_Abstraction
The LogFactories are loaded here: https://github.com/apache/commons-jcs/blob/c01db5a678d24f72840186bc392d4e3778038e8e/commons-jcs3-core/src/main/java/org/apache/commons/jcs3/log/LogManager.java#L48
Somehow I doubt the fix is in the dependency-check project itself.

Yes, thanks for pointing it out, it looks like the core module has both the JUL and log4j2 LogFactory classes so it doesn't seem possible to avoid the error message without having to remove the classes from the core jar.
Removing log4j2 is already done, and jcs will iterate over the LogFactory providers independently of setting the system property or not, so I don't think there's anything dependencycheck can actually do to address the issue.
Sorry for the noise, closing the ticket


Add Cloud Billing Info for projects.

Thanks for opening a Pull Request!
Here's a handy checklist to ensure your PR goes smoothly.

[x] I signed Google's Contributor License Agreement
[x] My code conforms to Google's python style.
[x] My PR at a minimum doesn't decrease unit-test coverage (if applicable).
[x] My PR has been functionally tested.
[x] All of the unit-tests still pass.
[x] Running pylint --rcfile=pylintrc passes.

These guidelines and more can be found in our contributing guidelines.

Note that the cloud billing API must be enabled on the project by the deployment or this will fail.

Adding cla: yes label, as the PR & commit authors are googlers at the time of this PR was created.


Cannot share file uploaded by same API user if username is overriden

(Filling out the following details about bugs will help us solve your issue sooner.)
Reproducible in:
pip freeze | grep slack
python --version
sw_vers && uname -v # or `ver`

The Slack SDK version
slack-sdk==3.20.2
Python runtime version
Python 3.10.7
OS info
ProductName:		macOS
ProductVersion:		13.2.1
BuildVersion:		22D68
Darwin Kernel Version 22.3.0: Mon Jan 30 20:38:37 PST 2023; root:xnu-8792.81.3~2/RELEASE_ARM64_T6000
Steps to reproduce:
import slack_sdk
import os

token = os.environ["SLACK_API_TOKEN"]
client = slack_sdk.WebClient(token=token)

file = client.files_upload_v2(
    filename="terraform-plan",
    content="test",
    title="Terraform Plan",
    alt_txt="",
    snippet_type="java",
)
permalink = file["files"][0]["permalink"]
client.chat_postMessage(
    channel=self.channel,
    text="test",
    username="Terraform",
    blocks=[
        {"type": "divider"},
        {
            "type": "context",
            "elements": [
                {"type": "mrkdwn", "text": permalink}
            ],
        },
    ]
)

Expected result:
The expectation is to see Terraform posting a message, containing a functioning link, for which a preview is rendered. Example:

Actual result:
When username is overriden, the file is not shared with the channel and the url does not function.

So, it seems that files_upload_v2, when channel is not set, uploads the file as private (seems to be the intention as per API docs). When trying to share the file using the chat_postMessage method, if the username is overriden, the file wont be shared with the channel.

Hi, @sidekick-eimantas! Thanks for submitting this question! 🙌
After doing some research, this appears to be expected behavior and a current limitation of the files upload endpoint. Since there is no username field for that API call, it indicates that the API does not support being able to customize the username, although chat.postMessage does support it.
I recommend checking out some of the workarounds listed in this reply from an older issue! Hopefully one of those workarounds will suit your use case.
I hope this helps, and let me know if there are further questions! 😄

Hi, @sidekick-eimantas! Thanks so much for that clarification! 🙌
Are you uploading the file in a private or public channel? I found this excerpt in the files.upload documentation:

By default all newly-uploaded files are private and only visible to the owner. They become public once they are shared into a public channel (which can happen at upload time via the channels argument).

Additionally, for the "File Not Found" error - are you logged into the Slack workspace in your browser? I wonder if the link isn't working because it will only allow members of your workspace to see the file.

Hi, @sidekick-eimantas!
Have you tried using the files.sharedPublicURL API method to pass in your uploaded file to enable public / external sharing? I suspect with the username updating not working to share the file preview via the permalink, it could be some permissions or access issue for the file since it's being posted from a "different user" and it's defaulting to a private setting. You can view more information on that API method here!

Unfortunately that API endpoint requires a User token
@sidekick-eimantas If you are unable to generate and use a user token for your app for that API call (available through your App Configuration page), then I recommend taking a look at this suggested approach for manually constructing a URL for your file to see if this mitigates the issue and allows you to preview / access your file. However, I'm not sure if the listed approach will work without using the files.sharedPublicURL API method, but it's worth a shot!

I have a similar problem. When I upload a file in Slack without channel ID and try to send it via PostMessage, I get this error:
The server responded with: {'ok': False, 'error': 'invalid_blocks', 'errors': ['downloading image failed [json-pointer:/blocks/0/image_url]'], 'response_metadata': {'messages': ['[ERROR] downloading image failed [json-pointer:/blocks/0/image_url]']}}
I have tried all the above solutions + this hack and also with the old file_upload method and the files_sharedPublicURL method. Everything fails. Slack can not recognize the own uploaded files no matter what URL format.
The only thing I want to do is style my text using blocks or other richtext options to style the text in the images accordingly.


Fixes building on Windows

os.rename fails on Windows if the destination already exists.

So this fixes building on Windows with the following commands since cloning the repo includes an empty secp256k1-zkp folder.
git clone https://github.com/grinventions/secp256k1-zkp-mw.git
cd secp256k1-zkp-mw
pip install .

However I just looked at the contents of the Python package with pip download secp256k1-zkp-mw and it doesn't include an empty secp256k1-zkp folder, so this might not fix installing on Windows with pip install secp256k1-zkp-mw.

I was able to get this all figured out. The Visual Studio C/C++ compiler doesn't work when files with long path names are used, and pip uses %TEMP%\pip-install\... as it's default temp location when installing modules which easily exceeds the compilers path length limitation. So to install this module on Windows you'll need to change the TEMP environmental variable to be something with a short path, like C:\temp, and also create that temp folder. This PR adds that information to the readme.
That's not all. The module also fails to install on Windows because of the absolute paths in the module's secp256k1-zkp-mw-0.1.0/secp256k1_zkp_mw.egg-info/SOURCES.txt file. So you'll need to change them from this:
/Users/marek/Development/open-source/grinventions/tmp3/secp256k1-zkp-mw/secp256k1-zkp/contrib/lax_der_parsing.c
/Users/marek/Development/open-source/grinventions/tmp3/secp256k1-zkp-mw/secp256k1-zkp/src/secp256k1.c

to this and republish the module with that change.
secp256k1-zkp/contrib/lax_der_parsing.c
secp256k1-zkp/src/secp256k1.c

This has been a great reminder of why I don't use Windows lol.

Thank you @NicolasFlamel1 for investigating this.


[Enhancement] Unordered parallel graph walk.

Summary of changes

Changes introduced in this pull request:

Benchmark to test the performance of unordered graph walking with cids extracted in parallel.

Reference issue to close (if applicable)

Work on #3314
Other information and links

Change checklist

[ ] I have performed a self-review of my own code,
[ ] I have made corresponding changes to the documentation,
[ ] I have added tests that prove my fix is effective or that my feature works (if possible),
[ ] I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

The current results show quite a performance gain, based on multiple single runs.
forest_snapshot_mainnet_2023-05-31_height_2908403.car.zst
  `unordered-graph-traversal` traversed 87.43 GiB at 310.22 MiB/s in 00:04:48
  `car-streaming --inspect` traversed 87.43 GiB at 270.14 MiB/s in 00:05:31

forest_snapshot_mainnet_2023-05-31_height_2908403.car
  `unordered-graph-traversal` traversed 87.43 GiB at 507.92 MiB/s in 00:02:56
  `car-streaming --inspect` traversed 87.43 GiB at 412.18 MiB/s in 00:03:37

Now that we're on the same page regarding the definition of a "graph traversal", let's close this PR.


Reweighting replicas

Added rayleigh_replica_estimator, and associated functions, for performing reweighting on general operator overlap data, using multiple replicas.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

61 of 75 (81.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 90.052%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/StatsTools/reweighting.jl
61
75
81.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
3983

Relevant Lines:
4423

💛  - Coveralls


Update scroll-reverser to 1.7.6

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as fd62cd6ae847ce7596b46d881cca2662e857e544. Your contribution is still credited to you.


allow to enable/disable plugins

it should be possible to activate/deactivate individual plugins

resolved 2.x.x


Add more tests for ipywidgets

Recently identified that a PR broke k3d, but everything else worked.
I.e. it looked as though everything worked, except just one feature.
Highlights complex nature of ipywidgets, messaging..
What's to prevent this from happening again.

Add tests for k3d, qgrid, ipyleaflet.
Can we add tests for nglview (this is another one that didn't work when everything else except output didn't work).
Confirm the required messages are sent from UI to kernel, sick as message hooks, comms channels and buffers.

@DonJayamanne Do you think we need this now before we ship? Or can we just test manually for now?

Perhaps manually, with with notebooks in src/test/datascience/uiTests/notebooks folder.
Oh yes, we'll need to add comments to know what to look for in the test.
I.e. someone might load the ipynb and see yup a canvas is printed out with an image, when in fact it needs to support some kind of animation or it should be a 3d model (i.e. some details for tester to know what to look for).
I found that the test notebooks i put in there didn't have enough comments and soon forgot what i was testing.

Okay I won't work on this one now then. Just doing more manual testing

@DonJayamanne do we still need this? Nlgview isn't tested at the moment, but do we think it's necessary?

Nope, lets close this.


Docs problem. Notebook not found

Hello, I am new to tensoflow and right now I am reading your gides, particularly guide about tf.function. There you have a link to guide for "eager execution", but I don't have access to it. Is it my problem or you have broken link?
Guide for tf.function: https://colab.research.google.com/github/tensorflow/docs/blob/master/site/en/guide/function.ipynb#scrollTo=J122XQYG7W6w
Broken link, which is in the first block of code:
"In TensorFlow 2, eager execution is turned on by default."
https://colab.research.google.com/github/tensorflow/docs/blob/master/site/en/guide/eager.ipynb
Upd.
Also there is broken link here
https://colab.research.google.com/github/tensorflow/docs/blob/master/site/en/guide/function.ipynb#scrollTo=ocxX-HVk7P2o
(See the Transformer and Deep Dream tutorials for example).
first link does not work
Upd. 2
Another broken link
https://colab.research.google.com/github/tensorflow/docs/blob/master/site/en/guide/function.ipynb#scrollTo=JeD2U-yrbfVb
To learn more, see the tf.data: Build TensorFlow input pipelines guide.

Hi @Tialo, Thank you for reporting the issue. The issue will move to closed status once the #2223 is merged. Thanks!

Also it would be nice if you would fix those links at the end
https://colab.research.google.com/github/tensorflow/docs/blob/master/site/en/guide/function.ipynb#scrollTo=IKyrEY5GVX3M
Further reading
To learn about how to export and load a Function, see the SavedModel guide. To learn more about graph optimizations that are performed after tracing, see the Grappler guide. To learn how to optimize your data pipeline and profile your model, see the Profiler guide.

Also it would be nice if you would fix those links at the end https://colab.research.google.com/github/tensorflow/docs/blob/master/site/en/guide/function.ipynb#scrollTo=IKyrEY5GVX3M
Further reading To learn about how to export and load a Function, see the SavedModel guide. To learn more about graph optimizations that are performed after tracing, see the Grappler guide. To learn how to optimize your data pipeline and profile your model, see the Profiler guide.

@Tialo I've added these changes in this commit. Thank you!

Thanks, on it.

Using https://www.tensorflow.org/guide/function as the reference.

"In TensorFlow 2, eager execution is turned on by default."

This should now take you to https://www.tensorflow.org/guide/basics or the corresponding Colab/notebook

(See the Transformer and Deep Dream tutorials for example).

Same here: https://www.tensorflow.org/text/tutorials/transformer and https://www.tensorflow.org/tutorials/generative/deepdream

To learn more, see the tf.data: Build TensorFlow input pipelines guide.

And same here: https://www.tensorflow.org/guide/data
Feel free to reopen the issue if you see any other links that need to be fixed. Thank you.


Add deploy to staging with terraform

Features

Deploy to staging with one command
Fix a some bugs in the minifying process

Pull Request Test Coverage Report for Build 40

6 of 9 (66.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 29.99%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/Playground.tsx
6
7
85.71%

src/utils/registerServiceWorker.ts
0
2
0.0%

Totals

Change from base Build 30:
0.03%

Covered Lines:
1062

Relevant Lines:
3106

💛  - Coveralls


Add http request example to rp

After myself and 2 others had the same question in 3 days on matrix: Maybe You want an example?
This is not strictly an embassy thing, but it took me (= new to Rust and fairly new to embedded) quite some time & a lot of kind help on matrix to figure out. So my guess is others might benefit.
(thanks @jamesmunns especially!)
Word of caution: Binary size grew > 10xxkb and flashing that somehow corrupts the device and the code fails. The exact same code copied into my own project works just fine. So I added a secction to cargo.toml to reduce binary size:
[profile.dev]
lto = true
opt-level = "z"

That way the code now runs without issue, with less than half the original binary size.
-> if that is not okay, then the entire thing should not go into the repo without some other solution. As a newbie I would then need help to do it differently.

I tested some random other examples and it does not look like I broke stuff.


refactor: lexer file reorganization

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Reorganized lexer files and changed reference approach to avoid circular imports. This change will make it easier to add lexer tokens and modes without needing to develop around the possibility of circular imports.
What is the current behavior? (You can also link to an open issue here)
Adding additional tokens and modes that reference those already existing have a tendency to create circular imports. When building, some import references are accessed before they are defined, which causes accesses to undefined during runtime.
What is the new behavior (if this is a feature change)?
The file structure has been changed so that circular imports are no longer a concern when extending the library.
Other information:
Most of the work here existed on a separate branch which was created prior to my previous pull request.  I used git cherry-pick to pull the specific commit (a074ed7) that had the changes I needed to resolve this issue.
fixes #7

Force push was to amend the merge commit to comply with the conventional commits format.

This looks good, thanks!


fix: Account tabs overflow

Description
Resolves #548
I got inspiration from referenced explorers
Demo
https://github.com/aeternity/aescan/assets/15363559/b85e4eae-0fc8-489c-8969-b02102f6329a
Checklist:

[x] I have read and followed the Contributing Guide
[x] add scrollbar to tabs

The build stopped working reopening here https://github.com/aeternity/aescan/pull/602


Create Page 'Admin Home' Mockup to Project

Please do work for this task in a branch named issue-06.

Merged onto main.


Check in license files for 1.6.0

Pull Request Test Coverage Report for Build 7954

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 63.932%

Totals

Change from base Build 7950:
0.0%

Covered Lines:
11977

Relevant Lines:
18734

💛  - Coveralls


country not appearing in address block

I included the Country "Canada" for respondent, but it does not appear either for respondent or their parent (same address):

Note, the interview knows the country:

It's just not displaying because it doesn't display by default. I think we may need to set up a special field for country if we want it because otherwise it adds an extra line and messes with the font size of the addresses.
Documentation links:
https://docassemble.org/docs/objects.html#Address
https://assemblyline.suffolklitlab.org/docs/components/AssemblyLine/al_general#aladdress-objects

Note, we just don't allow international address in LHI, so this might be considered nonessential.

Amy said fine to go ahead, OK to just add the flexibility.
However, will be nontrivial amount of time to get this going, so I think I'll shelve it for now and we can consider adding later.
Not too hard to add for user address (note that user address has default country code set, and most other addresses don't):
    - users1_address_block: |
        % if county_choice == "Oakland" and wants_confidential_contact_details:
        Confidential
        % else:
        ${ users[0].address.line_one() }
        ${ users[0].address.line_two() } ${ users[0].address.country if users[0].address.country != "US" else "" }
        % endif

But haven't tested extensively yet and running into more problems with the other_parties addresses.

Decided not to add international address capacity for now since it would be an enhancement and am running into difficulties.  Instead, remove country fields to match LHI.

Note, when using AL address_fields() method, it shows dropdown for state if "country" field is not shown, and fill-in-the-blank for state if "country" field is shown. Since not allowing country choice now, to make address screens match, I have made the other address screens (which don't use that method for various reasons) also dropdowns for state for now.


Correction for raw affiliation CBGP, INRAE, CIRAD, IRD, Montpellier SupAgro, Univ Montpellier, Montpellier, France

Correction needed for raw affiliation CBGP, INRAE, CIRAD, IRD, Montpellier SupAgro, Univ Montpellier, Montpellier, France
raw_affiliation_name: CBGP, INRAE, CIRAD, IRD, Montpellier SupAgro, Univ Montpellier, Montpellier, France
new_rors: 05q3vnk25;051escj72;003vg9w96;03rnk6m14;05kpkpg04;05h7ddb14
previous_rors: 05q3vnk25;051escj72;003vg9w96;03rnk6m14;05kpkpg04
works_examples: W4237749085;W4393420590
contact: 5fa35b216bfbdbbca7be2fb4c905ed81:7dad86ec6616eaa1a39b04e1cae96af0bff7 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-18. The new affiliations should be visible within the next 7 days.


msd: fix warnings in test

Why this PR?
It's very minor, but we have some warnings in our own tests and they pollute the pytest results. This PR fixes them.

I found them: https://github.com/lumicks/pylake/runs/7730132855?check_suite_focus=true#step:5:91. In general, it's good to mention which warnings are being fixed as part of the commit message/PR description.
Would it be a good idea to turn on warnings as errors on CI?

Yes, it would. I'll do that in a future PR. We still plan to clean some of the mpl warnings. I'll turn on warnings as errors when doing that.


[tracking] Alt e2eshark TODO's

Please feel free to contribute to this project by picking up one of the TODO items for alt_e2eshark

[ ] look into using pytest or another testing framework.
[ ] generate command-line reproducers for ease of debugging individual stages
[ ] come up with a better name for the directory than "alt_e2eshark".
[ ] identify functionality present in e2eshark that is currently lacking and should be migrated over. E.g. Chi's comment about the --report flag.
[ ] add more type hints to make it easier to track down definitions.
[ ] add (much) more to the README about adding tests and running them with various command line options.
[ ] look into removing torch-mlir as a python dependency entirely (perhaps optionally, so torch-mlir developers can still use the test suite for debugging op conversions).
[ ] Consider alternative options for storing cache directory paths, log directory paths, onnx model paths, etc. OnnxModelInfo is currently a bit clunky in needing to sometimes infer the log directory from the model path, etc.

Ci Setup:

[ ] flesh out report feature (maybe ignore time report)
[ ] add a report merge feature
[ ] testfile intake / registry
[ ] register all of the tests we want to run
[ ] maybe setup json configurable tests (and also automate the registry of such tests)
[ ] have a option to report with smaller stages (5 from e2eshark)


Add support for long arguments

If the argument for the console program gets too long, you run into issues like:
Error: fork/exec /app/console: argument list too long

Solution: first compress with http://www.faqs.org/rfcs/rfc1950.html and then base64 encode

fixed in 1.0.3


Run WMAgent validation for 2.1.2 production release candidate

Impact of the new feature
WMAgent
Is your feature request related to a problem? Please describe.
By-monthly task
Describe the solution you'd like
We need to release a new WMAgent stable version in September, so this issue is to keep track of the full validation activity to be carried out in the first weeks of September, once we have a final 2.1.2 tag. Note that this agent will likely be the first version to run CouchDB 3.2.2, so extra attention during the validation is required.
In addition to the validation results, WMAgent release notes and the validation check-list twiki need to be created as well.
Describe alternatives you've considered
none
Additional context
none

WMAgent has been validated and no patches are required. So version 2.1.2 is ready to go to production.
Here is the usual validation twiki: https://twiki.cern.ch/twiki/bin/view/CMS/WMAgent212Validation
Release notes - compared to the latest stable version 2.0.4 - can be found at:
https://github.com/dmwm/WMCore/releases/tag/2.1.2
and
https://github.com/dmwm/WMCore/releases/tag/2.1.1
and
https://github.com/dmwm/WMCore/releases/tag/2.1.0
Last but not least, there is one CMSMonitoring patch being discussed here:
https://github.com/dmwm/CMSMonitoring/pull/164
this is meant to silence the stomp.py library, which is extremely verbose even in INFO log level.


⚠ Bofrost.es has degraded performance

In f5a8259, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 4444 ms

Resolved: Bofrost.es performance has improved in aabe8d7 after 13 minutes.


Strong name?

Hello,
I was looking at using Markdig as a replacement for CommonMark in one project and MarkdownDeep in another, however Markdig isn't strongly named. Does you have any plans on strong naming the Markdig library (or perhaps offer two packages, one signed, one unsigned (the way YamlDotNet does it), which will satisfy both stubborn-as-mules camps)?
Thanks;
Richard Moss

You can always use Strongnamer: https://github.com/dsplaisted/strongnamer

Oh, nice this strongnamer, thanks @jeremydmiller
@cyotek If you can live with strongnamer that would be great... I dislike strong naming specially for a library that provides a plugin API, as you have either to deal with assembly redirect or to "trick" your assembly/file version that will be different to the NuGet version to avoid assembly redirect binding (which is typically not great for minor versions, as you would still want your clients to work in the same way, without having to recompile anything)

Thanks for the replies. I tried that strongnamer and can confirm it worked (at least as far as the example goes) but it's not a solution I'd like to use. At the same time I understand the reluctance of some library maintainers to add strong names (and I'm just as stubborn about using them it seems :))

From a library author to another, please don't go the route of strong naming. I'd rather we strong arm the community.

:-1: for strong naming the project

Would be convenient for those of us with strong name requirements if there were 2 NuGet packages available. MarkDig, and MarkDig.Signed. Should satisfy everyone, no?

Would be convenient for those of us with strong name requirements if there were 2 NuGet packages available. MarkDig, and MarkDig.Signed. Should satisfy everyone, no?

This causes problems if there are other NuGet packages that depend on MarkDig.  Each of those NuGet packages would have to also publish a signed and nonsigned version, or else the version of MarkDig that you want to use might not work with packages that depend on it.

Would be convenient for those of us with strong name requirements if there were 2 NuGet packages available. MarkDig, and MarkDig.Signed. Should satisfy everyone, no?
This is a really great idea, we also want to use MarkDig but are currently blocked only because this is not signed

@gsaralms  Yes, I will add a Markdig.Signed nuget package.

@xoofx : thanks a lot, it will help us a lot

So the NuGet package Markdig.Signed should now provide signed assemblies.

Thanks a ton @xoofx :)


Car not on track despite the amazing training results(accuracy)

I have trained the car to 90% training and 60% testing accuracy. However, the car is not able to run successfully on the track. Here are the videos: 1. https://photos.app.goo.gl/25NyShp9mrt47B0J3
2. https://photos.app.goo.gl/TDsIPjCbnqeLMhPS2
Any insights?

Sorry bro I can't answer your question, but may I know how do you manage to let the car move? Mine jus won't move after opening the rc_driver.py, although i only have a test accuracy of 30%, does that related? Thank you in advance!

same problem here also

@avisrivastava254084 Slow down your Car's speed. By adding more gears or using variable resistor cut down access voltage.


Added javaslang-benchmark module with microbenchmarks.

Progress towards #725: Micro benchmarks implemented for:

head()
tail()
get(int)
update(int, T)
prepend(T)
append(T)

For the following Javaslang colections:

List
Stream
Vector
Queue

Current coverage is 95.36%

Merging #1187 into master will not affect coverage as of ba71424

@@            master   #1187   diff @@
======================================
  Files           78      78
  Stmts         9170    9170
  Branches      1713    1713
  Methods          0       0
======================================
  Hit           8745    8745
  Partial         91      91
  Missed         334     334

Review entire Coverage Diff as of ba71424

Powered by Codecov. Updated on successful CI builds.

Hi Miguel, that's really great! I'm thrilled to let the tests run this evening an I'm really looking forward to the comparison with Scala. That will be worth a nice blog post, maybe as guest post to reach more people :-)

@danieldietrich I'm happy to contribute to the blog post. That being said I wouldn't be comfortable to make a blog post with the current use cases - they're a start but we need more insights before we try to have a meaningful discussion. Posts on benchmarks are likely to fuel flame wars :). I would prefer to add composite operations (say, groupBy, distinct elements etc) also. What do you think?

@malduarte yes, we should get a clear picture. Yesterday I also thought about adding more complex measures, e.g. combined operations like adding and removing elements in one test. Also it makes sense to add real-world use-cases like solving project Euler problems. I'm looking forward to have side-by-side measures with Scala. We will take our time to understand the results.


CupertinoTextField should not accept requestFocus when disabled

Description
CupertinoTextField should not accept requestFocus when disabled
Related Issues
Fixes #65191
Tests
I added the following tests:
See files.

@chunhtai Please review at your convenience, thanks.

@HansMuller Ready for review :)

@justinmc @LongCatIsLooong please review, thanks.

Google testing failure doesn't seem related. Rerunning.


Selling solutions

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

@Mady1 please do not create spam issues


Send: 5 confs warning is shown after success, which is confusing

Feedback from Giacomo's students:

No serious bug to report, no serious UX confusion either (except maybe that, after a cooperative send, the error-like alert about "5 confs being necessary to send cooperatively" confused some of them a bit, since it seemed the send had failed, when it has succeded...probably a success message would help).

We could change the warning to an info or something like that, if a send was just successfully made. The warning is more important for first-time users, i.e. for the first send.

Already tackled here https://github.com/joinmarket-webui/jam/issues/516#issuecomment-1382865893 waiting for development.
Bens Tutorial already brought it to light. But good that it’s coming up more, we are on the right path

I think this can be closed.

A success message is shown after a transaction has been made.
Jars are not preselected anymore (so the message is only shown, after a jar has been chosen).

What do you say @editwentyone ?

exactly, closing it


publishing Issue

Cannot set a constant field.
[FieldAccessException: Cannot set a constant field.]
System.Reflection.RtFieldInfo.PerformVisibilityCheckOnField(IntPtr field, Object target, RuntimeType declaringType, FieldAttributes attr, UInt32 invocationFlags) +0
System.Reflection.RtFieldInfo.InternalSetValue(Object obj, Object value, BindingFlags invokeAttr, Binder binder, CultureInfo culture, StackCrawlMark& stackMark) +255
System.Reflection.RtFieldInfo.SetValue(Object obj, Object value, BindingFlags invokeAttr, Binder binder, CultureInfo culture) +36
Serenity.Data.RowFieldsBase.Initialize() +1609
Serenity.Data.RowExtensions.Init(TRowFields rowFields) +12
MovieTutorial.Northwind.Entities.CategoryRow..cctor() +44
[TypeInitializationException: The type initializer for 'MovieTutorial.Northwind.Entities.CategoryRow' threw an exception.]
MovieTutorial.Northwind.Entities.CategoryRow..ctor() +17

Probably running under low trust. Try removing readonly from all field declarations. But you might still have issues.

What is the best way to publish this type of project

I will try but same result.

Ask your hosting provider for medium trust, high trust. Or change it.

Thanks
You Save My Life and my 3 months working.

.NET doesn't allow reflection in low/medium trust level. These fields are initialized through reflection. What can i do about this. If this is saving your life, i'm glad, good luck with your overly protective hosting provider that no-one elses uses.


Follow results in a request pending approval

Unsure why this is happening. Is it the bridge or is there something PodcastAP can do to resolve the request.
Found if you cancel and then try again on the active server, the request can go through
https://podcastindex.social/<EMAIL_ADDRESS>

If it gets stuck indefinitely, the problem is somewhere between receiving the Follow activity and sending out the Accept Follow reply.
If you've implemented this synchronously, could it be that your inbox endpoint fails with a status code 2xx, 4xx or 501 except 401, 408 or 429? Seeing one of these causes Mastodon not to retry delivery. (It doesn't indicate permanent delivery failure.)


Remotes, install.R potential improvements

Currently running the install.R; into hour 5 and up to VETransportSupplyUse. This is on Windows, over VPN, so not the fastest test, but still quite slow. A few changes should improve this:

The following should be added to the Getting Started page, rather than a link to the install.R page on GitHub:

source("https://raw.githubusercontent.com/gregorbj/VisionEval/master/install.R")

VESimHouseholds, VECommercialTravel still have @develop remote
VEEnergyAndEmissions still has @storagetest remote
Every module that imports VESimHouseholds (such as VELandUse) also therefore downloads @develop
Each module still installs framework from source. This seems to be driven by the imports option in the DESCRIPTION; this is probably unavoidable if we stay with the single repository and use install_github. Possible that install_local from a downloaded zip of the repository would not have this issue.

If time allows, consider testing a method like this in the install.R script to download once and then install_local:
VE_modules = c(
                "VESyntheticFirms",
                "VESimHouseholds",
                "VELandUse",
                "VETransportSupply",
                "VETransportSupplyUse",
                "VEHouseholdVehicles",
                "VEHouseholdTravel",
                "VERoadPerformance",
                "VEEnergyAndEmissions",
                "VETravelCost",
                "VEReports",
                "VECommercialTravel"
                )

# inspired by https://github.com/r-lib/devtools/blob/master/R/install-github.r
  dest <- "~/ve_temp"
  request <- httr::GET("https://api.github.com/repos/gregorbj/visioneval/zipball/master")

  writeBin(httr::content(request, "raw"), paste0(dest, ".zip"))

  unzip(paste0(dest, ".zip"), exdir = dest, overwrite = T)

  devtools::install_local(path = file.path(dest, "/sources/framework/visioneval"))

  for(i in 1:length(VE_modules)){
    cat(rep("<<>>", 10), "\n Installing", VE_modules[i], "\n")
    devtools::install_local(path = file.path(dest, "/sources/modules/", VE_modules[i]))
    )
  }

I updated https://github.com/gregorbj/VisionEval/wiki/Getting-Started as well

Sorry, one more..
https://github.com/gregorbj/VisionEval/blob/master/sources/modules/VEHouseholdTravel/DESCRIPTION


staging-next 2022-05-04

workflow docs: nixos.org/manual/nixpkgs/unstable/#submitting-changes-commit-policy
constitutents: hydra.nixos.org/job/nixpkgs/staging-next/unstable#tabs-constituents
jobset: hydra.nixos.org/jobset/nixpkgs/staging-next
nix-review reports: https://malob.github.io/nix-review-tools-reports/
previous staging-next: https://github.com/NixOS/nixpkgs/pull/169965

It looks like this broke gfortran (and probably gcc) on aarch64-darwin. My guess is that this patch needs to be updated or removed after #170121: https://github.com/NixOS/nixpkgs/blob/3c36acf988e6f7e104adea166124fa8451e9abfe/pkgs/development/compilers/gcc/11/default.nix#L72
@bobrik initially introduced it, maybe they know how to update it? Otherwise I will try tonight. Is it acceptable to send a PR directly to staging-next to fix this?

Yes, regressions introduced in master..staging-next are sent directly to staging-next.

@bobrik initially introduced it, maybe they know how to update it?

Previously I waited for @fxcoudert to update his fork and created a patch from the tags diff there. Looks like there's no update for 11.3 thought. Homebrew puts 11.2 as 11.3:

https://github.com/Homebrew/homebrew-core/commit/5849ebff85a5684a4f67459cd2ab771aa3663143

Homebrew puts 11.2 as 11.3

The release of gcc 12.1.0 will happen on Friday, so I deemed the effort of rebasing a 11.3 port for two weeks not worth it. You can have a look at this branch by the darwin GCC maintainer, if you want: https://github.com/iains/gcc-11-branch/tree/gcc-11-3-apple-si
It contains the diff you can apply on top of 11.3

Regression fix from 22f5d327101dbf13c072761417f460ed413725e8: #171832

More https://github.com/NixOS/nixpkgs/pull/164163 -related fixes:

[ ] https://github.com/NixOS/nixpkgs/pull/171836

There is an issue with firefox on staging-next. Crashes instantly with exit status 255.

No surprise, this is related to https://github.com/NixOS/nixpkgs/pull/164163.
The culprit seems to be this part of the firefox wrapper:
https://github.com/NixOS/nixpkgs/blob/384d446910801ac29070cac4765761caba15fcd0/pkgs/applications/networking/browsers/firefox/wrapper.nix#L216-L219
It rewrites the target of the gapps wrapper from
/nix/store/1qg729mv21pk0i7hylx4qczirdfybv8d-firefox-release-bin-unwrapped-100.0/bin/.firefox-wrapped

to
/nix/store/gyybbi7d2jwr23f673ascrmq6c5wh2yy-firefox-bin-100.0in-unwrapped-100.0/bin/.firefox-wrapped

[ ] @rszibele: curlcpp broke down, maybe due to curl update.  (you're in meta.maintainers)

[ ] https://github.com/NixOS/nixpkgs/pull/171985


feat: added retention controller. Fixes #5369

This PR aims to fix: https://github.com/argoproj/argo-workflows/issues/5369. It is currently WIP and is thus set to 'draft'.
Tips:

Maybe add you organization to USERS.md.
Your PR needs to pass the required checks before it can be approved. If the check is not required (e.g. E2E tests) it does not need to pass
Sign-off your commits to pass the DCO check: git commit --signoff.
Run make pre-commit -B to fix codegen or lint problems.
Say how how you tested your changes. If you changed the UI, attach screenshots.
If changes were requested, and you've made them, then dismis the review to get it looked at again.
You can ask for help!

@NikeNano  Retention is controller level policy to make sure given time  X number of workflow should in the cluster. what bases will workflow delete like creation _at/finishied_at? How are we going to handle the multi-tenant use case? if the user wants to configure # of workflow per namespace.

@NikeNano Retention is controller level policy to make sure given time X number of workflow should in the cluster. what bases will workflow delete like creation _at/finishied_at? How are we going to handle the multi-tenant use case? if the user wants to configure # of workflow per namespace.

@NikeNano Retention is controller level policy to make sure given time X number of workflow should in the cluster. what bases will workflow delete like creation _at/finishied_at?

Do you mean how we should handle the retention controller in relation to TTL controller? I dont fully follow here, could you elaborate further @sarabala1979 ?

How are we going to handle the multi-tenant use case? if the user wants to configure # of workflow per namespace.

Good point, I will check how we handle this today in the TTL controller further and come back. I think the ideal solution should allow for setting on namespace level.

I think there is already a GC controller we can extend?

When doing the design I looked at how we do for the TTL, which is a separate controller, and tried to stay consistent with it. Do you refer to the workflowGarbageCollector and that we move this functionality there?

I think users will want to know why their workflow was deleted. I'm not sure how to tell them. Maybe a Kubernetes event?

Sounds good to me, will have to read up a bit on it and update the code.

I meant the TTL controller, it is responsible for deleting old workflows. It might be a good place to extend.

I meant the TTL controller, it is responsible for deleting old workflows. It might be a good place to extend.

Since the retention controller will need to loop over all workflows to select which to keep based upon freshness and completion/error/fail I think the logic will be very different from the ttlcontroller which treats each workflow separately. Combining them will combine all different type of clean up (retention and ttl) to one place but the logic is very different between the two flows as I see it.  Keeping them seperatly makes more sens to me since they do the clean up very differently and dont share any bigger parts of logic. What do you think @alexec ?

I was thinking there is a deletion queue in the TTL controller where we’d want to centralise deletion, so any events or logging is consistent.

I was thinking there is a deletion queue in the TTL controller where we’d want to centralise deletion, so any events or logging is consistent.

Ok, I can see the advantage with having one deletion queue. Will move the code to the TLL controller(and rename it).

changes requested - dismiss AND re-request review when complete

Have had a lot at work, have set aside time tomorrow to hopefully have time to fix all the changes.

I have updated to use a Heap now instead @alexec, it would be awesome if you could give some early feedback on it. I will look in to and fix the tests as well.

@simster7 could you take a last look(as I understand the requested changes are blocking the auto-merge?), the changes you requested should be resolved mostly due to redesigning the solution, thanks!

@alexec who should we move this forward?

I think your changes have been addressed.


ESP32-SX Have alternate partition scheme with no OTA support

Though OTA is cool, I'd hazard to guess that fewer CircuitPython users would use OTA in general, and even be confident enough to guess hardly any use it all.
I'd like to see, not only a second UF2 option for ESP32-SX boards that has no OTA partitions laid out at all, and just a much bigger user file system partition, but also have that be the new default partition layout/binary and have OTA as a second option for those that want to use OTA.
I've spoken a few times with @tannewt about this, and it has to happen at the UF2 bootloader side, as that is what enforces the partition scheme, so we'd need this integrated as alternate builds for ESP32 (and any other platforms that could/might) use this in the future.
I am constantly getting asked questions like...

or folks just letting me know their board they just received is faulty because it's only got 2MB of flash on it instead of 4MB or 8MB etc like advertised.
I don't really mind how this is implemented - but I think it's really important to give users more flash if it's available on the chip, and the choice to use it or choose OTA instead if that suits their needs.
@hathach Thoughts?

Note that a solution is now released in Circuitpython 8. This PR: https://github.com/adafruit/circuitpython/pull/7000 thanks to MicroDev, lets you reformat the CIRCUITPY drive so that it extends into the OAT1 partition.
https://docs.circuitpython.org/en/latest/shared-bindings/storage/index.html#storage.erase_filesystem

An empty flash is formatted in extended mode by default.
It disables the dualbank module (it's still there but raises an exception).
Reformatting an existing drive preserves the [non] extended nature of it.

An existing drive can be extended like this (save your data first of course).
import storage
storage.erase_filesystem(extend=True)

Just to be clear, it's actually "extended", rather than "extend":
import storage
storage.erase_filesystem(extend=True)


Should be able to set type on functions

TypeScript allows to set a type on a function

CC should allow that too
This will allow to define a function as a type in one file and then use it in implementation.
I understand everyone's very conservative here I remember I opened a similar issue I got an explanation that functions should not be annotated with @type but I think they should. This way I can create interfaces for functions, import them in implementations and have type checking done by the compiler. Otherwise, I have manually add the parameters to the function which I don't want to do because it's wasting time since I already have function's types in a separate place. TypeScript in vs code does it properly as you can see on the screenshot.

OK so i started to assume that it's not possible to set types like that because my typechecking wasn't done when trying to set app.anything because i expected an error there saying that this property wasn't define it app's constructor but it didn't. when i removed the type and returned params the compiler still didn't say anything so i don't know what the problem was. the issue can then be converted into the following one
/**
 * @type {_idio.session}
 */
export default function(app, opts = {}) {
}

src/index.js:31: WARNING - [JSC_MISPLACED_ANNOTATION] Misplaced type annotation. Type annotations are not allowed here. Are you missing parentheses?
export default function(app, opts = {}) {
               ^^^^^^^^^^^^^^^^^^^^^^^^^^

i.e., not being able to annotate type on default export which shouldn't be an obstacle.


Performance Regression or Improvement: cogbk_python_batch_load_test_reiterate_4times_2MB_values:runtime

Performance change found in the
test: cogbk_python_batch_load_test_reiterate_4times_2MB_values for the metric: runtime.
For more information on how to triage the alerts, please look at
Triage performance alert issues section of the README.
Test description: CoGroupByKey Python Load test - reiterate 4 times 2 MB values python | coGBK | reiteration 2MB value Test Link - https://github.com/apache/beam/blob/5e38decf9e723a385057131b01bbd33d8c60bda3/.test-infra/jenkins/job_LoadTests_coGBK_Python.groovy#L128 Test Dashboard - http://<IP_ADDRESS>/d/fK0U4JqWz/cogbk-load-tests?orgId=1&var-processingType=batch&var-sdk=python&viewPanel=5

timestamp: Fri Sep 29 14:22:49 2023, metric_value: 324.00 <---- Anomaly
timestamp: Fri Sep 29 12:37:24 2023, metric_value: 111.00
timestamp: Thu Sep 28 12:34:22 2023, metric_value: 79.00
timestamp: Wed Sep 27 12:37:28 2023, metric_value: 78.00
timestamp: Tue Sep 26 12:42:04 2023, metric_value: 71.00
timestamp: Mon Sep 25 12:38:57 2023, metric_value: 77.00
timestamp: Sun Sep 24 12:39:03 2023, metric_value: 137.00
timestamp: Sat Sep 23 12:37:51 2023, metric_value: 77.00
timestamp: Fri Sep 22 12:39:30 2023, metric_value: 77.00
timestamp: Thu Sep 21 12:45:09 2023, metric_value: 198.00
timestamp: Wed Sep 20 12:38:37 2023, metric_value: 78.00

This is a bug. We need not to report the issue if the tool reports on the most recent run.


Leverage new differ autoload.

Builds on rspec/rspec-support#181. This avoids the
cost of loading diff/lcs, pp, etc for cases where
we don’t need to print any diffs.

Appveyor has gone away for some reason but this LGTM so merging regardless.


Write a C# program to find area of a square

Description
Write a C# program to find area of a square
Area of square = Side^2
Side : 2

Area : 4

How to contribute

Save the solution in FindAreaOfASquare.cs file
Add FindAreaOfASquare.cs file in find-area-of-a-square folder

!assign

Hey @Mohitt0,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)


[✨FEAT✨]: Heart beat node for making sure the car is alive

Is there an existing enhancement request for this?

[X] I have searched the existing feature requests

Description
Have a message be sent to the motec every so often so that the motec knows that the computer is stull running.
Additional Information
No response

Could this be generalised to include all safety critical nodes

This is more or less just talking to the motec bc the motec dose not care what brakes on the computer as long as it is still working and so all internal problems should be solved internally and if somthing completly brakes internally then the computer should shutdown and thus the motec will shutdown

How would you propose we geeneralise this to include all safety crit nodes?

@Jack17432 I think I'm thinking more of #20. Like a node that sends heartbeat and checks which ones send back a response.


Comply with bash being installed in a different path

First of all thank you a lot for the script. It saves me a lot of time and works pretty reliably. Nevertheless I have two suggestions.
Bash may be installed in a different path. On Mac OS it lives in /bin/bash for example.

I've used that in the past for my perl scripts (and did for this project as well before the switch to bash, I think) just neglected to do it for this version. Thanks.


fix: validate provider urls before use

This fixes validation, like #133, except for all providers that accept a hostname/base URL variable.

@lamchau can you cast eagle eyes over this?

thanks for the advice @lamchau will revise impl after clarification on https://github.com/block-open-source/goose/pull/147#discussion_r1802101239

sorry was awol on medical leave. sporadic this week, but will adjust this and others soon

@codefromthecrypt no worries! hopefully you're doing well/on the mend!

@lamchau PTAL I have consolidated what I could into the same check, and refactored things around it.


Error when starting kit on Windows

Error: spawn C:\Users\xenox\AppData\Local\Programs\kit\resources\fallbacks\windows\clipboard_x86_64.exe ENOENT
    at r (C:\Users\xenox\AppData\Local\Programs\kit\resources\app.asar\main.prod.js:2:543688)
    at n (C:\Users\xenox\AppData\Local\Programs\kit\resources\app.asar\main.prod.js:2:543859)
    at ChildProcess.e.emit (C:\Users\xenox\AppData\Local\Programs\kit\resources\app.asar\main.prod.js:2:544000)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:291:12)

main.log

Kit 1.40.62
Electron 22.0.0
win32 10.0.19045
Locale: en-US

Edit: Changed the title - I read that there was an alternative to the windows installer (which gave this error), which was the npx command. This installed the kit fine and everything worked. So did a few commands.
I then had to restart my PC, and after restarting, kit gave the same error as the installer did. It tries to start kit, but then it gives this error and then just hangs in the tray on "starting".
I tried rebooting again and such, to no avail.
I then tried looking for the file at "C:\Users\xenox\AppData\Local\Programs\kit\resources\fallbacks\windows\clipboard_x86_64.exe" - however I found no resources directory.


Blender file for minotaurs

The .blender file for Minotaurs is here : https://github.com/Kalabasa/minetest_defense/tree/master/mods/defense/models
also the post on the forum : https://forum.minetest.net/viewtopic.php?p=177856#p177856
Now we need to release the Minotaaur (without rotate problems :))

Tweaked and 100% over. Thanks to Mg :)


[Bug]: VSCode extension does not compile ligo smart contract through command palette

What happened?
When you run the command to compile a ligo smart contract in the command palette in vscode you run into the error "The @taqueria/plugin-ligo plugin experienced an error when executing the proxy.". When running the same command through the CLI (taq compile) there is no issue. This may be due to an error when installing the ligo plugin though the command palette in vscode which just returns "There was a problem running taq."
What should have happened?
What should have happened is running the latest 0.0.8 release on a linux system/subsystem we should be able to install the ligo plugin through the command palette and then add a ligo contract and compile it through all the vscode extension.
Steps to reproduce the bug
Steps to reproduce the error:

Download a taqueria binary from the releases page (0.0.8 at the time of writing this) - https://github.com/ecadlabs/taqueria/releases and add it to the path on the machine that will be running the vscode extension
Install the taqueria vscode extension in your vscode instance through the marketplace (at the time of writing 0.0.8 is in pre-release)
Initialize a taqueria project through the command taq init {your_project_name}
cd into the directory for your taqueria project
Initialize it as a npm project through running npm init -y
Open up vscode to the taqueria project
Open the command palette and select the command "Taqueria: Install Plugin", select ligo. This is where the first error happens where it says that there was an error running taq. There is no output beyond that
If you install the taq plugin through the command line through taq install @taqueria/plugin-ligo then there is no issue with installation
Once the ligo plugin is installed then add a contract to the contracts directory named increment.jsligo with the following inside of it:

type storage = int;
type parameter =
| ["Increment", int]
| ["Decrement", int]
| ["Reset"];
type return_ = [list <operation>, storage];
/* Two entrypoints */
let add = ([store, delta] : [storage, int]) : storage => store + delta;
let sub = ([store, delta] : [storage, int]) : storage => store - delta;
/* Main access point that dispatches to the entrypoints according to
   the smart contract parameter. */
let main = ([action, store] : [parameter, storage]) : return_ => {
 return [
   (list([]) as list <operation>),    // No operations
   (match (action, {
    Increment: (n: int) => add ([store, n]),
    Decrement: (n: int) => sub ([store, n]),
    Reset:     ()  => 0}))
  ]
};

Open the command palette and select "Taqueria: Compile (LIGO)". This is where the error with the proxy comes in while running the compile command through the CLI has no issue

Relevant log output
No response
Screenshots and images
Screenshot and images:
Installing the ligo plugin
https://user-images.githubusercontent.com/29528321/153069076-999c911c-21cc-4d45-9e54-d55fd417c17d.mp4
Trying to compile after installing the ligo plugin through the CLI
https://user-images.githubusercontent.com/29528321/153069332-370a13e5-72de-42c0-aacd-ad93a4c9d730.mp4
Code of Conduct

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

@alexzbusko is this still an issue?

I just tried to reproduce, and I can't even get the VS Code Extension to install the LIGO plugin...

I just tried to reproduce but the VS Code Extension fails when I try to install the LIGO plugin...

Assigning to @MichalLytek for further investigation

@AlirezaHaghshenas please verify if this is still an issue?

It works now if the contract has been added to the registry, meanwhile there is no command to add contract to registry

@AlirezaHaghshenas just confirmed that there is a PR to add a command to the command palette. This issue can be closed.


Decision List Learner implemented

Hi,
The implementation of DecisionListLearner was incomplete (discussed in #948). I've implemented the algorithm, written tests and updated the corresponding notebook.
Here's what I've done:

DLTest class represents tests in the decision list. A DLTest has attr_values to hold attributes and corresponding values to test them against. It has some member functions like calling matches with an example as input checks whether the example passes the test or not, calling matched_examples or unmached_examples with inputs as a list of examples returns a subset of examples which pass and don't pass the test respectively.

create_DLTests creates a list of all possible DLTests which are used in creating Decision List. (Decision List Learner repeatedly loops on these tests so as to find a test that agrees exactly with some subset of the training set.) It takes as input attribute_count which is the maximum number of attributes to consider while creating a test. As of now attribute_count=1 is supported.

DecisionListLearner is the implementation of the of Decision list learner as mentioned in the book. Member function decision_list_learning is the python implementation of the pseudocode

I've added test for DecisionListLearner here.

I've updated the learning.ipynb to include DecisionListLearner. I've also added an image in its description.

The code is well documented, let me know if you need any clarifications regarding the code.

Any updates on this PR?


Added XCRC (aka CRC32) support

Hi,
i added CRC32 support cause most of the FTP servers i use only supports this.
To be honest i didn't create the crc32 class by myself. I used this  as base. I hope this not a problem (due license or someting else) and you can accept this pull request.

n0ix

This is awesome, thanks!

Released. Thanks again! This was a major missing feature and you really helped bring it up to speed!
https://www.nuget.org/packages/FluentFTP/30.2.0

Btw see https://github.com/robinrodricks/FluentFTP#contributors


Document backups

This PR attempts to document a shared definition of what a backup is and our current use of backups.
📖 Rendered page
https://trello.com/c/8TESDxDu/3129-write-a-document-of-our-current-use-of-backups-2

Thank you for your invaluable input ❤
I pushed some minor tweaks (missing commas, full stops) and I'm going to merge once the CI is green ✅


Publish a document targeting maintainers explaining how they can get package changelogs auto-pulled into Dependabot

I noticed a recent Dependabot PR of a popular project is missing the changelog and release notes sections:

https://github.com/dependabot/dependabot-core/pull/6218

This happened to be a major version bump, indicating breaking changes. While hunting down the release notes, I happened to notice that the maintainer had temp closed the issue tracker to reduce folks filing tickets because things don't work anymore:

https://github.com/PyCQA/flake8/issues/1760

Having the release notes / changelog front and center in the Dependabot PR's helps not only our users, but also maintainers because users are more likely to see the notes before they file a new issue. For example, contrast the above PR with this which makes it super easy to see release notes / changelog:

https://github.com/dependabot/dependabot-core/pull/6219

I was going to email the flake8 maintainer, when I realized we should really have a public doc we can link to that shows maintainers how to more easily expose their projects metadata in a way we expect.
The metadata fetch process will vary by ecosystem, so ultimately we should have a few notes and then link to that ecosystem's package index doc on how to expose metadata... but I expect some package index docs don't have a doc like that, so it's an opportunity for us to work with them to create one. That will help those entire ecosystems, not just Dependabot.

also document the changelog for docker that @mctofu just landed... requires maintainer to add OCI tag...


[Bug]: There's a warning that ctg still uses v9 style data access

Expected Behavior
no error-warning
Current Behavior
error-warning
Steps to Reproduce
load up a world with ctg active
Context
I already have a PR going on to fix it, it's a super easy fix.
Version
1.4.3
Foundry VTT Version
v10.284
Operating System
Windows 10
Browser / App
Native Electron App, Chrome
Game System
dnd5e (2.0.2)
Modules Disabled

[X] I can confirm that disabling all of my other modules doesn't fix this issue.

Thanks for the PR! I will have to close it for now because I'd like to preserve v9 compatibility until at least v11 comes out.


helm installed elastalert but no pods found

i tried to install elastalert via helm install elastalert codesim/elastalert --set writebackIndex=elastalert  on my ubuntu VM running k8s and also on my raspberry pi k8s cluster.
it seemed successful on both
NAME: elastalert
LAST DEPLOYED: Tue Jun 16 15:24:44 2020
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:
1. Elastalert is now running against: :80

however when i run kubectl get po --all-namespaces, i am unable to find any pods running elastalert and so far it only affected elastalert helm installation.
is there any config that i might be missing?

Hello, Elastalert needs to know the hostname of the ES cluster to connect to. Make sure you specify the elasticsearch hostname value, which is blank by default. The reason the pod is missing is because the deployment.yaml does not attempt to deploy a pod when the elasticsearch host is blank.

I just pushed a change to the values.yaml to have it use more sensible default values that should avoid this confusion.


Gluing slippery rotating bodies on the ground

Hello !
I am currently using MatterJS with Phaser 3 for a game project. It is basically the same concept as the "draw climber" game.
The idea is to move a character on a solid ground (generated from a SVG path) defined as follows:
isStatic: true, friction: 0.8
I then allow the player to draw the "legs" of the character, which are dynamically added to the character via a constraint:
this.constraintA = scene.matter.add.constraint(mainBody, legA.body, 0, 1, {
    pointA: { x: 0, y: YOffset }
});

These legs are based on a set of vertices. They rotate at a fixed speed and at an opposite angle, using the Matter.Body.rotate function:
Matter.Body.rotate(legA.body, Math.PI/60);

However, I cannot get these legs to "stick/glue" to the ground. With the rotation, they slide on it. More precisely, I would like the character to be able to move forward properly. Here is a visualization of the problem:

When a collision between a leg and the ground is detected, I tried to calculate the normal vector from the current leg angle in order to force the body to move forward, but the character then makes strange and non-consistent jumps. Here is a handmade example of this normal vector according to the leg angle:

I also tried to modify the parameters related to friction and mass, without any good result.
Finally, is there a way to make the leg "stick" to the ground and thus allow the player to move forward? Or maybe a much simpler solution proposed by MatterJS for this type of problem? Thanks!

Regarding "sticking" the foot to the ground, did you change the friction of both the ground and the feet.

Hi! Thank you so much for your help!
I have indeed changed both frictions. I finally realized that the problem was due to the fact that the "legs" were slightly going through the solid ground (even though the density is very high), due to the rotation. The solution was to limit the velocity of the legs in the "beforeUpdate" function of the MatterJS engine.
Now, I can define a high leg friction (>1) and use the initial ground friction (0.7).

Did you get this working? I'd say to experiment with body.frictionStatic (this is roughly how 'sticky' the body is) as well as body.friction and then try tuning the body.slop (how much bodies can sink into each other for traction) of both the legs and the ground.

Hey @Seragonia could you please share more info? I have the exact same problem, it's being really difficult to nail the correct values, specially if the object is small


Added HTML5 importable sources

I don't know if I should fork this repo, or somewhere within jquery/jquery itself, but it would be very very handy to have HTML5 importable version of jquery (for the sake of Polymer and any future Web Components modules).
Currently every module importing jquery via script tag will re-download it and re-run it. HTML5 Imports will download the source once as long as the URL path does not change. This helps a lot and will make more and more developers use jQuery in their web components.

Hi there, either there is an error, or I misunderstand the CLA thingy. I've signed it both with my full name and public email, and with my username and commit email (which appears on "check the status" page). Can you guys validate it manually or hint me what did I do wrong?
Cheers,
Dan

The CLA check requires using a full name (with at least two words) instead of a nick. If you have such a name and agree to use it, please modify your commits to be signed by your full name & re-sign the CLA. In other cases please notify us so that we can verify the signature manually. Thanks!
As for the PR itself, all the files in this repo are automatically generated by the release script at https://github.com/jquery/jquery/blob/master/build/release.js (which uses jquery-release) so this is where all modifications need to happen.

Got it. Could you please verify that signature manually? I prefer not to use my full name in commits.
PR-wise, if I fork the jquery-release, when could I expect to see a new release of jQuery using my fork (in case it's accepted)?

Could you please verify that signature manually? I prefer not to use my full name in commits.

@scottgonzalez @dmethvin @csnover what's the procedure here?

if I fork the jquery-release, when could I expect to see a new release of jQuery using my fork (in case it's accepted)?

In the next jQuery version in the 3.x line. Depending on when it'd land it could be 3.0.0, 3.0.1, 3.1.0... I can't provide reliable estimates, I'd expect it would happen within a month or two at the most. But yeah, this needs to be discussed as well.

Great, thanks for the info :). I'll try to do the fork when I get some free time, need to get familiar with it.


No License.

Add a license to begin with, this might be the CC BY NA 4.0 License.

License added.


Add --help flag to turbo-ignore

This matches the setup for create-turbo package

Closing this down, @tknickman is going to take over


chore(pkg): upgrade version 1.9.2

PR Checklist
If you have any questions, you can refer to the Contributing Guide
What is the current behavior?
Please describe the current behavior and link to a relevant issue.
Issue Number
Example: #123
What is the new behavior?
Please describe the new behavior or provide screenshots.
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Specific Instructions
Are there any specific instructions or things that should be known prior to review?
Other information

@Red-Asuka Speed 🫡


PROD: show team coach details on club game reports

See mail from Sven

Dont understand - league reports show coach name in the contacts list already


Ambient (and sidecar TPROXY mode) does not work with GKE intranode visibility

Context https://istio.slack.com/archives/C041EQL1XMY/p1723039587363189
https://github.com/GoogleCloudPlatform/netd/blob/a2d69d0b4bb3e52b1c7e61e9408aa0bc60a31609/pkg/options/options.go#L42
https://github.com/GoogleCloudPlatform/netd/pull/177#issue-1441380837
They set src_valid_mark=1 which breaks TPROXY. Proposed solution is to unset this in the pod network namespace.
cc @bleggett

A hotfix for existing deployments with Istio 1.23 was discussed here:

Set AMBIENT_TPROXY_REDIRECTION=false on the CNI daemonset (e.g. with kubectl  -n istio-system  set env ds/istio-cni-node AMBIENT_TPROXY_REDIRECTION=false)
Restart workloads

Just leaving this here for historical purposes and in case someone faces the same issue till the fix is available.
P.S. v1.24 will also have env exposed in Helm chart for CNI which allows to apply no-cli fix.

A backport was done for 1.23 but has not been merged yet: https://github.com/istio/istio/pull/53629

A backport was done for 1.23 but has not been merged yet: https://github.com/istio/istio/pull/53629

Nice! Would be great to have it with 1.23 as well.

cc @istio/release-managers-1-23 :point_up_2: can we get this merged?

Approved backport for 1.23, patch is small and no change to default behavior so this seems fine.

Approved backport for 1.23, patch is small and no change to default behavior so this seems fine.

thanks!


Adding CI integration that runs tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@28763a9). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             master     #3   +/-   ##
=======================================
  Coverage          ?   100%
=======================================
  Files             ?      1
  Lines             ?     10
  Branches          ?      0
=======================================
  Hits              ?     10
  Misses            ?      0
  Partials          ?      0

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 28763a9...37e2172. Read the comment docs.


RSS Feed for each year

We have pages per year:
all: https://developers.google.com/web/updates/
2017: https://developers.google.com/web/updates/2017/
2018: https://developers.google.com/web/updates/2018/
all: https://developers.google.com/web/updates/rss.xml
2017: https://developers.google.com/web/updates/2017/rss.xml
2018: https://developers.google.com/web/updates/2018/rss.xml
@petele says https://github.com/google/WebFundamentals/blob/master/gulp-tasks/build.js#L238 is the place to look

This is done.


feat(nvim-tree): remove deprecated options

open_on_setup mechanisms has been deprecated by nvim-tree.lua#2122.
I received warning message after upgrading nvim-tree:
[NvimTree] unknown option: ignore_ft_on_setup | [NvimTree] unknown option: ignore_buffer_on_setup | [NvimTree] unknown option: open_on_setup | [NvimTree] unknown option: open_on_setup_file | see :help nvim-tree-setup for available configuration options

These options should be removed.

tks, but it's already in the branch 0.9


No possibility to use Service Principal when Workload Identity is enabled in the Pod

Problem
We have been encountering the issue in our pipelines, when we try to run kubectl commands in self-managed runners.
When we have Workload Identity variables in our Pod (AZURE_CLIENT_ID, AZURE_CLIENT_SECRET, AZURE_FEDERATED_CREDENTIALS), we've no choice to use another type of Azure AD object like Service Principal, even if we have converted the kubeconfig by using kubelogin convert-kubeconfig with Service Principal credentials.
When we migrated from deprecated AAD Pod Identity, we had to do many workarounds to not interrupt working of each of our pipelines/scripts. For example, we had to override the AZURE_CLIENT_ID variable only for the time of running kubectl command, which is redundant, time consuming and not convenient.
We noticed that even if you are logged in via az login with SP credentials, running the az aks get-credentials and then kubelogin convert-kubeconfig -l spn with Service Principal arguments --client-id <AZURE_CLIENT_ID> --client-secret <AZURE_CLIENT_SECRET> --tenant-id <AZURE_TENANT_ID>, the kubeconfig file will anyway get Workload Identity client ID in the kubelogin get-token command as the CLI argument.
Moreover, even if we override AZURE_CLIENT_ID to use SP one only during running kubelogin convert-kubeconfig -l spn command, the next kubectl command run still uses the Workload Identity.
AZURE_CLIENT_ID environment variable always takes precedence over cli arguments - I don't know if it's intended, but for sure is not obvious.
There should be an option to authenticate to the AKS cluster by using Service Principal, even if Workload Identity's variables are injected into the Pod. The fact that the command line arguments in the kubeconfig are always overriden by AZURE_CLIENT_ID variable doesn't give us any flexibility.
How to reproduce
Prerequisites

Self-managed runners created via Gitlab runner helm chart
Workload Identity associated with Gitlab runner Pod's Service Account

Steps
Run the Gitlab job which will do the following:

Login via az login as the Service Principal
Get kubeconfig via az aks get-credentials command
Convert kubeconfig with kubelogin convert-kubeconfig command and Service Principal parameters specified as arguments --client-id <AZURE_CLIENT_ID> --client-secret <AZURE_CLIENT_SECRET> --tenant-id <AZURE_TENANT_ID>
Run any kubectl command (best to run against some Kubernetes resource the SP has and WI doesn't have access to, so you will see the object id of the logged in AAD identity)

Expected output
kubectl command runs with Service Principal credentials
Real output
kubectl command runs with Workload Identity credentials

@daleksandrowiczgd can you share what your kubeconfig looks like after kubelogin convert-kubeconfig -l spn? you only need to capture the exec plugin part:
      exec:
        apiVersion: client.authentication.k8s.io/v1beta1
        command: kubelogin
        args:
          - get-token
          - --environment
          - AzurePublicCloud
          - --server-id
          - <AAD server app ID>
          - --client-id
          - <AAD client app ID>
          - --tenant-id
          - <AAD tenant ID>

besides, in your repro step you seem to miss -l spn? not sure if this is intentional or not?

Non intentional, I missed it this in my message, so please just assume it should be there.

can you also share the actual relevant environment variables in your runner environment? There is no reference to AZURE_FEDERATED_CREDENTIALS in any repo I can find.

Actually, I just mentioned all environment variables set by Workload Identity, but it seems like only the AZURE_CLIENT_ID variable generates conflicts for kubelogin tool.

can you share what your kubeconfig looks like after kubelogin convert-kubeconfig -l spn? you only need to capture the exec plugin part:

Yes, let me even share you few cases with exact commands I run and results of the kubeconfig file after running each of them.
In all cases the following env variables will be set (I will use the same name to identify the same values in kubeconfig):
export SP_CLIENT_ID="<SP_CLIENT_ID>"                    # Client (Application) ID of the Service Principal
export SP_CLIENT_SECRET="<SP_CLIENT_SECRET>"            # Client secret of the Service Principal
export SP_TENANT_ID="<SP_TENANT_ID>"                    # Tenant ID of the Service Principal
export CLUSTER_RG="<CLUSTER_RG>"                        # AKS cluster resource group
export CLUSTER_NAME="<CLUSTER_NAME>"                    # AKS cluster name
export CUSTOM_KUBECONFIG_PATH="${HOME}/.kube/config"    # Path to the kubeconfig

Additionally, for better understanding of the output I will specify WI_CLIENT_ID (Workload Identity client ID), instead of the real value.
Pass Service Principal Client ID as the CLI argument in the kubelogin command (desired usage)
az login --service-principal -u "${SP_CLIENT_ID}" -p "${SP_CLIENT_SECRET}" -t "${SP_TENANT_ID}"
az aks get-credentials --resource-group "${CLUSTER_RG}" --name "${CLUSTER_NAME}" --file "${CUSTOM_KUBECONFIG_PATH}"
kubelogin convert-kubeconfig --login "spn" --kubeconfig "${CUSTOM_KUBECONFIG_PATH}" --client-id "${SP_CLIENT_ID}" --client-secret "${SP_CLIENT_SECRET}" --tenant-id "${SP_TENANT_ID}"

Kubeconfig:
exec:
  apiVersion: client.authentication.k8s.io/v1beta1
  args:
  - get-token
  - --login
  - spn
  - --server-id
  - 6dae42f8-4368-4678-94ff-3960e28e3630
  - --client-id
  - ${WI_CLIENT_ID}
  - --tenant-id
  - ${SP_TENANT_ID}
  - --environment
  - AzurePublicCloud
  - --client-secret
  - ${SP_CLIENT_SECRET}
  command: kubelogin
  env: null
  installHint: |2

    kubelogin is not installed which is required to connect to AAD enabled cluster.

    To learn more, please go to https://aka.ms/aks/kubelogin
  provideClusterInfo: false

$ kubectl get pod --kubeconfig ${CUSTOM_KUBECONFIG_PATH}
# RESPONSE 401 Unauthorized
# --------------------------------------------------------------------------------
# {
#   "error": "invalid_client",
#   "error_description": "AADSTS7000215: Invalid client secret provided. Ensure the secret being sent in the request is the client secret value, not the client secret ID, for a secret added to app '${WI_CLIENT_ID}'. Trace ID: 1d7b8101-9172-4b1e-8ac6-ac840f666301 Correlation ID: 9d204593-95f3-460e-81e6-fb6a94e4d959 Timestamp: 2024-01-03 10:45:10Z",
#   "error_codes": [
#     7000215
#   ],
#   "timestamp": "2024-01-03 10:45:10Z",
#   "trace_id": "1d7b8101-9172-4b1e-8ac6-ac840f666301",
#   "correlation_id": "9d204593-95f3-460e-81e6-fb6a94e4d959",
#   "error_uri": "https://login.microsoftonline.com/error?code=7000215"
# }
# --------------------------------------------------------------------------------
# To troubleshoot, visit https://aka.ms/azsdk/go/identity/troubleshoot#client-secret
# Unable to connect to the server: getting credentials: exec: executable kubelogin failed with exit code 1

Conclusion: --client-id argument in kubeconfig was overriden by Workload Identity. kubectl command doesn't work correctly - return 401 error.
Override AZURE_CLIENT_ID environment variable only for the kubelogin command
az login --service-principal -u "${SP_CLIENT_ID}" -p "${SP_CLIENT_SECRET}" -t "${SP_TENANT_ID}"
az aks get-credentials --resource-group "${CLUSTER_RG}" --name "${CLUSTER_NAME}" --file "${CUSTOM_KUBECONFIG_PATH}"
AZURE_CLIENT_ID="${SP_CLIENT_ID}" kubelogin convert-kubeconfig --login "spn" --kubeconfig "${CUSTOM_KUBECONFIG_PATH}" --client-id "${SP_CLIENT_ID}" --client-secret "${SP_CLIENT_SECRET}" --tenant-id "${SP_TENANT_ID}"

Kubeconfig:
exec:
  apiVersion: client.authentication.k8s.io/v1beta1
  args:
  - get-token
  - --login
  - spn
  - --server-id
  - 6dae42f8-4368-4678-94ff-3960e28e3630
  - --client-id
  - ${SP_CLIENT_ID}
  - --tenant-id
  - ${SP_TENANT_ID}
  - --environment
  - AzurePublicCloud
  - --client-secret
  - ${SP_CLIENT_SECRET}
  command: kubelogin
  env: null
  installHint: |2

    kubelogin is not installed which is required to connect to AAD enabled cluster.

    To learn more, please go to https://aka.ms/aks/kubelogin
  provideClusterInfo: false

$ kubectl get pod --kubeconfig ${CUSTOM_KUBECONFIG_PATH}
RESPONSE 401 Unauthorized
--------------------------------------------------------------------------------
{
  "error": "invalid_client",
  "error_description": "AADSTS7000215: Invalid client secret provided. Ensure the secret being sent in the request is the client secret value, not the client secret ID, for a secret added to app '${WI_CLIENT_ID}'. Trace ID: 854ccce1-bdf3-4122-8453-8e3ba73f9700 Correlation ID: 1371d7e8-053f-4f32-8f7b-9859bc27f635 Timestamp: 2024-01-03 11:00:37Z",
  "error_codes": [
    7000215
  ],
  "timestamp": "2024-01-03 11:00:37Z",
  "trace_id": "854ccce1-bdf3-4122-8453-8e3ba73f9700",
  "correlation_id": "1371d7e8-053f-4f32-8f7b-9859bc27f635",
  "error_uri": "https://login.microsoftonline.com/error?code=7000215"
}
--------------------------------------------------------------------------------
To troubleshoot, visit https://aka.ms/azsdk/go/identity/troubleshoot#client-secret
Unable to connect to the server: getting credentials: exec: executable kubelogin failed with exit code 1

Conclusion: --client-id argument in kubeconfig got correct Client ID of the Service Principal. kubectl command doesn't work correctly - return 401 error, again with the Workload Identity client ID in the error (WI_CLIENT_ID).
Override AZURE_CLIENT_ID environment variable only for kubelogin and kubectl commands
az login --service-principal -u "${SP_CLIENT_ID}" -p "${SP_CLIENT_SECRET}" -t "${SP_TENANT_ID}"
az aks get-credentials --resource-group "${CLUSTER_RG}" --name "${CLUSTER_NAME}" --file "${CUSTOM_KUBECONFIG_PATH}"
AZURE_CLIENT_ID="${SP_CLIENT_ID}" kubelogin convert-kubeconfig --login "spn" --kubeconfig "${CUSTOM_KUBECONFIG_PATH}" --client-id "${SP_CLIENT_ID}" --client-secret "${SP_CLIENT_SECRET}" --tenant-id "${SP_TENANT_ID}"

Kubeconfig:
exec:
  apiVersion: client.authentication.k8s.io/v1beta1
  args:
  - get-token
  - --login
  - spn
  - --server-id
  - 6dae42f8-4368-4678-94ff-3960e28e3630
  - --client-id
  - ${SP_CLIENT_ID}
  - --tenant-id
  - ${SP_TENANT_ID}
  - --environment
  - AzurePublicCloud
  - --client-secret
  - ${SP_CLIENT_SECRET}
  command: kubelogin
  env: null
  installHint: |2

    kubelogin is not installed which is required to connect to AAD enabled cluster.

    To learn more, please go to https://aka.ms/aks/kubelogin
  provideClusterInfo: false

$ AZURE_CLIENT_ID="${SP_CLIENT_ID}" kubectl get pod --kubeconfig ${CUSTOM_KUBECONFIG_PATH}
No resources found in default namespace.

Conclusion: --client-id argument in kubeconfig got correct Client ID of the Service Principal. kubectl works correctly, because AZURE_CLIENT_ID was overriden only during running this command.
Override AZURE_CLIENT_ID environment variable only for the kubectl command
az login --service-principal -u "${SP_CLIENT_ID}" -p "${SP_CLIENT_SECRET}" -t "${SP_TENANT_ID}"
az aks get-credentials --resource-group "${CLUSTER_RG}" --name "${CLUSTER_NAME}" --file "${CUSTOM_KUBECONFIG_PATH}"
kubelogin convert-kubeconfig --login "spn" --kubeconfig "${CUSTOM_KUBECONFIG_PATH}" --client-id "${SP_CLIENT_ID}" --client-secret "${SP_CLIENT_SECRET}" --tenant-id "${SP_TENANT_ID}"

Kubeconfig:
exec:
  apiVersion: client.authentication.k8s.io/v1beta1
  args:
  - get-token
  - --login
  - spn
  - --server-id
  - 6dae42f8-4368-4678-94ff-3960e28e3630
  - --client-id
  - ${WI_CLIENT_ID}
  - --tenant-id
  - ${SP_TENANT_ID}
  - --environment
  - AzurePublicCloud
  - --client-secret
  - ${SP_CLIENT_SECRET}
  command: kubelogin
  env: null
  installHint: |2

    kubelogin is not installed which is required to connect to AAD enabled cluster.

    To learn more, please go to https://aka.ms/aks/kubelogin
  provideClusterInfo: false

$ AZURE_CLIENT_ID="${SP_CLIENT_ID}" kubectl get pod --kubeconfig ${CUSTOM_KUBECONFIG_PATH}
No resources found in default namespace.

Conclusion: --client-id argument in kubeconfig was overriden by Workload Identity. kubectl works correctly, because AZURE_CLIENT_ID was overriden only during running this command.
So as you can see, always the AZURE_CLIENT_ID variable takes precedence over CLI arguments defined in the kubeconfig:

if the wrong --client-id was put inside the kubeconfig (Workload Identity one), you can override AZURE_CLIENT_ID variable only during running kubectl/helm command, but this is not a deal.
if the correct --client-id was put inside the kubeconfig (Service Principal one), the kubectl/helm command will fail with 401 error, what gives no possibility to use Service Principal, when there is already Workload Identity injected into the Pod.

Hope everything is understandable.

@daleksandrowiczgd does directly overriding the env field help as a workaround?
exec:
  apiVersion: client.authentication.k8s.io/v1beta1
  args:
  - get-token
  - ...
  command: kubelogin
  env:
  - name: "AZURE_CLIENT_ID"
    value: "value for client ID"

what is your environment? do you know how AZURE_CLIENT_ID is set?

@weinong, our environment:

Self-managed runners created via Gitlab runner helm chart
Workload Identity associated with Gitlab runner Pod's Service Account

We migrated from deprecated AAD Pod Identity to the Workload Identity. In our Gitlab CI jobs we want to give us the opportunity to switch between Service Principal or Managed Identity credentials, if needed (e.g. sometimes we have issues with Azure API throttling, so we need to have the option to use SP in such cases). After migration to Workload Identity, we encountered the problem that I described in the issue description.
In general, how the runners config related to the Workload Identity looks like and how those variables are injected:

Custom ServiceAccount with the annotation azure.workload.identity/client-id: <SP_CLIENT_ID> (<SP_CLIENT_ID> - client ID of the Service Principal)
Gitlab runner config modified with two additional fields:

serviceAccount: <SA_NAME> (<SA_NAME> - the name of the above ServiceAccount)
Label azure.workload.identity/use: "true"

With the above configuration, the newly created runner Pod that runs the Gitlab job will have injected the following 4 variables (but I think only AZURE_CLIENT_ID interfere with kubelogin) :

AZURE_CLIENT_ID
AZURE_TENANT_ID
AZURE_FEDERATED_TOKEN_FILE
AZURE_AUTHORITY_HOST

Right now after migration to Workload Identity, it's impossible to use Service Principal nor Managed Identity with different client ID, because AZURE_CLIENT_ID environment variable injected by Workload Identity always takes precedence in the kubelogin commands.

@enj yes, this workaround works, thanks for an idea. But we still need to modify every pipeline and script to manually update the kubeconfig file everywhere we want to use the Service Principal credentials.
Maybe it will be good to add AZURE_CLIENT_ID env variable to this exec part in the kubeconfig file, when Service Principal is chosen in the kubelogin convert-kubeconfig command (--login "spn"), wdyt?

We just hit this issue in Azure DevOps too, while trying to migrate our self-hosted runners to use workload identity instead of aadpodidentity, and on a quick skim of the docs I don't see any way to manipulate the environment for these commands to implement a similar workaround.
In general the idea that environment variables override provided cli arguments is quite surprising, most tooling I'm used to interacting with uses envionment variables as a fallback if cli arguments aren't provided. If the kubelogin tool used the environment as a fallback for missing cli arguments then there would be no problems here at all.


Feature Request: Alphabetical Sort of Buffers

Hey there! Still continuing to use this plugin and it's been fantastic so far!
As I've progressed in use, i've found that in our orgs code base that I might often need to have 2-3 different file types up, with various extensions on them. An example would be like this in the buffer manager:
Reviews.tsx
Comms.test.tsx
Reviews.loader.tsx
Reviews.test.tsx
Comms.loader.tsx

And you can imagine that as you're going to and from buffers, deleting them, working, etc, that this can become somewhat chaotic. I do know that you have an option to move buffers around with remaps, but i'm curious if it would be much work to extend the manager to do some kind of default sort when its updated?
I think this takes some agency away from the user, which can be great, but when I am in the thick of things and exploring around, I find that it would actually be really helpful if these were presorted.
Would this be a feature that is difficult to introduce? I envision that a user could set the sort order as Ascending or Descending as part of the configuration. Thoughts on this suggestion?

Thanks for your feedback. I think it is a good idea. I have not had time to look into it. I hope to have time to implement it by the end of the year.

I wonder if something like this would be good enough here -
vim.keymap.set({ 't', 'n' }, '<localleader>b', function()
  require('buffer_manager.ui').toggle_quick_menu()
  vim.defer_fn(function() vim.cmd'%sort' end, 50) -- %sort! for descending
end, { noremap = true })

Hello. I have created the branch order-buffers with this feature. If you could give it a try to see if it works as expected, I would appreciate it.


Prtesting

:warning: Validation status: warnings

File
Status
Preview URL
Details

ops_testing/index.md
:warning:Warning
View
Details

ops_testing/index.md

Line 2: [Warning] Invalid file link:(~/../nonexisted1.md).
Line 3: [Warning] Invalid file link:(~/../nonexisted2.md).

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.


How to install Ubuntu 14.04 x64

I dont see anythink installation guide ? anyone help ?

I do not recommend using this script, if you have 0 knowledge in nodeJS.

I am the Python director and i know this is simple guide if you wanna i little tell you
sudo apt-get update sudo apt-get install -y curl libcurl3 libcurl3-dev curl -sL https://deb.nodesource.com/setup_6.x | sudo -E bash - sudo apt-get install -y nodejs sudo apt-get install -y build-essential npm i -g forever forever start index.js
?

sudo apt-get update
sudo apt-get install -y curl libcurl3 libcurl3-dev
curl -sL https://deb.nodesource.com/setup_6.x | sudo bash -
sudo apt-get install -y nodejs
sudo apt-get install -y build-essential
npm i -g forever
forever start index.js

Would work yes

but folder doesnt work steam/auth not found the server i upload the server folder /var/www

That means u wrongly configured the script.
steam/auth is an endpoint, not a folder
/var/www doesn't matter, it's a node script, it can run anywhere
This script is not meant for people who don't know what they're doing

I know it's not enough for me but i try and learn i want to little help. Just give me lead ?

We are the developer ,we learn as we share i believe this.


Meme Battle Game [GSSOC'23]

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below- In this project , I have created a Meme Battle Game using HTML/CSS/JS
In this game, 2 players can play together and battle on the basis of memes generated in their respective containers
All the Memes used - are collected from the net, User can add their own memes in the players folders. The game works on the principle that it generates a random situation and the person who generates a meme related to it wins or if that's not possible then the person with 5 memes wins and the game restarts.
Fixes #2004

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

After the player wins - the meme generates=0 and popup is seen of the Winner

Thank you soo much for contributing to our repository 💗

Thank you @sahaycodes ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @sahaycodes , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @sahaycodes,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


License

Please, kindly provide a license to give us a legal statement to use your code, Sir.

Please, kindly provide a license to give us a legal statement to use your code, Sir.

Sure. Thank you for your remind.


Direct Velocity or Position Control

iS IT POSSIBLE TO BYPASS THE CARTESIAN CONTROLLER AND CONTROL IN JOINT SPACE DIRECTLY.

Yes.
Environments typically start with joint control, Cartesian control is added on top of that.
For example, in RGB stacking, we start with a joint velocity controller, then a 6-DoF cartesian controller is made that delegates to the joint controller, finally, a 4-DoF cartesian controller is created that delegates to the 6-DoF cartesian controller.
The 4-DoF controller is the exposed to the agent by setting it as the arm_effector in the StandardRobot initializer.
If you want to change your action space, change this effector.  You could use a joint velocity controller or create your own effector (E.g. a PD based position controller) the api is mainly action_spec (input names and ranges) and set_control which decides how actions are interpreted,

Also note, you can use create your own ActionSpace if you want to change actions without using an effector.  But since the choice between cartesian vs joint space control is made by effector choice (for RGB stacking) that would be the place to start for that task.
ActionSpaces are widely employed for things like casting actions to be of the same type, clipping or limiting action values, changing the control frame and others

wow, beautiful interface, what do you advice with respect to reducing the substeps. if i use the direct velocity control. i don't expect to have to need 100 substeps, as must suite i used in the past works well even for 4 substeps.

@shacklestone

Sorry, could you elaborate what you mean by substeps? Do you mean the control rate?

yes so adjusting the control timeteps and physics timesteps. if i adjust it to reduce the substeos. so to make CONTROL_TIMESTEPS/PHYSICS_TIMESTEP smaller.

Here is what you can achieve:

By changing the control timestep you will change at which frequency the agent is able to send a command. You can easily play around with this.
By changing the physics timestep you will make the simulation go faster overall and have less physics timestep. However, this is not advised as it might render the simulation less stable

Thank you


hack/patch: fix some typos in README, make cherrypick.sh executable

@jpbetz This is successfully run to create https://github.com/coreos/etcd/pull/8947.
My git remote -v looks like this:
$ git remote -v

gyuho	git@github.com:gyuho/etcd.git (fetch)
gyuho	git@github.com:gyuho/etcd.git (push)
origin	git@github.com:coreos/etcd.git (fetch)
origin	git@github.com:coreos/etcd.git (push)

lgtm

Witnessed "/bin/etcd: Permission denied" error with latest release (v3.1.11).


fix(multicast.spec): typo fix

results1 repeated twice. changed one to results2 to check both
results1 and results2.

Description: Bug in tests (misprint?)
Related issue (if exists): https://github.com/ReactiveX/rxjs/issues/2698

Coverage remained the same at 97.734% when pulling 2266dd4f4777fecc262b80e5e34d3a257521b9a8 on piq9117:test-typo into 136944bdb1d57d52d3fff792acd9f54d262660db on ReactiveX:master.


Checkout upstream/stable instead of LATEST

From the private mail conversation:

Hi all,
It often happens that someone finds a typo in the docs or other Ddoc
weirdness. At the moment, it takes about a month for this fix to
reach dlang.org.
We already have the version chooser with master/stable and the last
releases at docarchives.dlang.io.
So what do you think about building from upstream/stable instead of
the latest release?
Any thoughts on this one? As mentioned it would be cool to see fixed typos going live immediately and not after one month.
Sorry for the late answer. I think that's a fine idea. -- Andrei

This also brings it in more accordance to the version chooser:

nice

Something has gone terribly wrong. Just a short excerpt from the VERY LARGE log file:
.generated/stable_dmd-2.077.1/dmd2/linux/bin64/dub(22666:4805)[error]: Declaration expected

Just a short excerpt from the VERY LARGE log file:

Ouch 143M log file??
Fixed & this is passing now, but it seems that the assert->rewrite pipeline isn't working correctly anymore :/

but it seems that the assert->rewrite pipeline isn't working correctly anymore :/

Luckily I have been working on a separate PR that simplifies this pipeline: https://github.com/dlang/dlang.org/pull/2069 - this should help this PR too.

Argh. I realized a big problem with this. We can't do this within the two weeks before release where master got merged into stable, but didn't get fully released yet.
We could check that with https://raw.githubusercontent.com/dlang/dmd/stable/VERSION == git ls-remote --tags https://github.com/dlang/dmd | sed 's/.*refs\/tags\/\([a-z0-9.-]*\).*/\1/' | sort -r | head -n1 though ...


Breaking change, Tempblob.ToBase64String()

Good day,
breaking change in deleting the tempblob table from platform 15. The replacement codeunit "Temp Blob" does not have a ToBase64String-function.
.

The functionality has been moved out of the TempBlob on purpose. There is now a Base64 Helpers Codeunit that is doing the same based on Streams.  See an example of usage bellow
var
    Base64Helpers: Codeunit "Base64 Helpers";
    TempBlob: Codeunit "Temp Blob";
    ....
begin
    TempBlob.CreateInStream(InStream);
    Base64Text := Base64Helpers.ToBase64String(InStream);

Thank you. In my opinion all of the changes that break existing solutions should primarily be avoided, but if necessary to do, they should be mentioned proactively in the BREAKINGCHANGES.md - not just after someone runs into trouble.

Thank you I will add them

I have the following code and I don't know how to transform it so as not to use the TEMPBLOB table

var
    TempBlob: Record TempBlob temporary;
    Out: OutStream;
    RecRef: RecordRef;
    Report: Report "Order Confirmation";
...
begin
...
    TempBlob.Blob.CreateOutStream(Out);
    Report.SaveAs('', ReportFormat::Pdf, Out, Recref);
    exit(TempBlob.ToBase64String());
end

Any ideas?

I have this code and I don't know how to transform it so as not to use the TEMPBLOB table
var
TempBlob: Record TempBlob temporary;
Out: OutStream;
RecRef: RecordRef;
Report: Report "Order Confirmation";
...
begin
...
TempBlob.Blob.CreateOutStream(Out);
Report.SaveAs('', ReportFormat::Pdf, Out, Recref);
exit(TempBlob.ToBase64String());
end
Any ideas?

maybe something like
local procedure fuu():text;
var
Base64: codeunit "Base64 Convert";
TempBlob: Codeunit "Temp Blob";
Out: OutStream;
InStr: InStream;
RecRef: RecordRef;
Report: Report "Order Confirmation";

begin
TempBlob.CreateOutStream(Out);
Report.SaveAs('', ReportFormat::Pdf, Out, Recref);
TempBlob.CreateInStream(InStr);
exit(Base64.ToBase64(InStr));
end