autopilot hashed sender

Context
During liquid stake and forward, the autopilot "receiver" of the inbound transfer becomes the "sender" of the transfer back to the host. However, downstream applications shouldn't trust this new "sender" so we need to use a generated address instead.
To be clear, using the original sender would not introduce an attack vector on Stride. However, it could introduce an attack vector on a different zone, if they were to trust the sender. The hashed sender is used to make the assumption more explicit that new zones should not trust the address.
This bug appeared in PFM (if more context is needed):

Issue
PR fix

Design Considerations
There wasn't an immediately obvious way to implement this. The complexity arises in that the address used for the inbound transfer doesn't always line up with the address used in the autopilot action. Additionally, if a bank send is required after the transfer (in the event that we transferred to an intermediate recipient), some scenarios would require us to build the IBC denom hash ourselves (not impossible, but adds a lot of unnecessary code).
There didn't seem to be a clean way to refactor the callback such that we use a different receiver based on the action type (i.e. use "hashed" for LS&Forward, but "original" for Claim). As such, I think the only two options were as follows:

Send inbound transfer to original receiver

✅  Claim: No changes
✅  Liquid Stake: No changes
❌  Liquid Stake and Forward: Requires us to bank send to hashed receiver (but this is straightforward since the token is not an IBC denom)

Send inbound transfer to hashed receiver

❌❌ Claim: Would require a bank send to the original receiver (we'd need to copy over all the IBC hashing code from ibc-go in order to handle both native and non-native denoms)
❌  Liquid Stake: Would require a bank send to the original receiver (but this is straightforward since the token is not an IBC denom)
✅ Liquid Stake and Forward: No changes

I do think using the hashed receiver address as the inbound recipient and actor for each of the autopilot actions, does feel slightly more correct. However, I don't think it's a significant enough argument to justify the additional changes that would be required. Additionally, going with option 1 demands no changes to the existing autopilot functions that have been live on mainnet.
TLDR: I went forward with option 1 (send transfer to original receiver) - but very open to other ideas if one of you can think of a better option!
Brief Changelog

Added GenerateHashedAddress to generate the hashed address from the channel Id a previous sender (body of function taken from PFM)
Renamed PacketForwardMetadata to AutopilotMetadata
Added a bank send to the hashed address in LS and forward
Set the hashed address as the outbound sender during LS and forward
Restricted packets to being either autopilot or PFM (but not both)
Cleaned up the variable names to avoid confusion with all the data structures (open to better names though!)
Moved structs out of parser.go

Just talked this one through with Sam and think I agree that (1) is a good approach.
My original intuition would have been to opt for approach (3): on the inbound IBC transfer, use a different receiver based on the action type (i.e. use "hashed" for LS&Forward, but "original" for LS and for Claim). Was thinking that at the stage of the IBC middleware stack where the transfer occurs, we could check the memo and decide whether to transfer to "hashed" or "original" at that stage. But it sounds like that might require some refactoring.
And (1) is easy to think about because it puts all the LS&Forward logic in liquidstake.go functions, rather than intermingling it in the IBC middleware stack.

@asalzmann @riley-stride
I now remember why I thought option (3) would be messy. it's because we call the same TryLiquidStake function regardless of whether we're doing just LS, or LS&Forward. Which means we have to have a clunky switch statement to check whether we want to use the hashed receiver or original receiver.
I implemented option (3) here to make things more concrete. (Funny enough option (1) and (3) are the exact same number of LOC haha).

I personally think option (1) is a bit cleaner than (3) because we only have action-specific stuff in one place instead of two. And more importantly, we don't duplicate logic between the two [ex: here and here].
That said, I'm still open to option (3) if that's your preference!

Chatted on line - decided on option 3 and will refactor the autopilot schema to cleanup the switch statement

should we add our rationale for it into the comments? so that if we ever want to change it in the future that context isn't lost.

Great idea! Updated here


How to use Sessions?

Does Vibora support sessions yet?
If so, an example would be helpful.
Thanks

How familiar are you with Redis? In the past I have leveraged Redis to store session related data and has worked well. Oh and setting an EXPIRES on the key is also helpful if you want a session to end at a certain time and have the browser and/or web app check.

Found this https://github.com/vibora-io/vibora/tree/master/samples/sessions

DanielSokil
Do you use a session with Redis?
I can not find the AsyncRedis class anywhere
ImportError: cannot import name 'AsyncRedis' from 'vibora.sessions'

@Dexus77 I never got this to work either.


Duration.ofMillis()

When it comes to Coding (same as with Bridge Building), Cultural Marxist garbage will not produce well written code or a stable bridge.  Java's main purpose is to conduct Virtue Signaling and propagate fringe subculture into mainstream, not to produce manageable and provable code

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

ID: 70a492fe-4ea3-2972-e4bd-e25f95b210bf
Version Independent ID: 31b848a4-c9ce-bf05-d3c3-c171d7696741
Content: Reliable Actors timers and reminders
Content Source: articles/service-fabric/service-fabric-reliable-actors-timers-reminders.md
Service: service-fabric
GitHub Login: @vturecek
Microsoft Alias: vturecek

@LastTribunal I don't understand your feedback. What are you looking to have added to this specific document?

I don't understand why MS needs to bond with the Java subculture..
But if you are looking for a contribution, find where timerIntervalInMilliSeconds is declared.  And then compare to the C# code.  Actually dont bother, because Java is not measurable by merit, but by "how it makes you feel"

I still don't understand the issue with the document. I have ran through it and I do not find any issues. If you have any specific feedback on items in the docs that are not working or you are having issues with we are happy to assist.
Simply inform us the problem you are seeing, any repro steps and we can take it from there. Else, if you just have general feedback on a doc that also works. Im closing this as there is no action needed on the doc. If you have any additional feedback feel free to let me know and I will take a look.


Next Up indicator shows up for Scaled Tokens

When I prepared pixies and scaled the token, the next up indicator showed up and spins for the latest token. There is no combat.

It happens after I scale a token. Copy/Paste a scaled token doesn't make this bug happen. Also, the starting indicator also shows up


[Parameter Request] Please expose RulesetFile

Not all rules that are checked by the code analyzers always need to apply, especially when developing a PTE.
To be able to disable certain rules or to lower severity, the parameter RulesetFile of Run-AlPipeline needs to be added to .AL-GO/settings.json

Will look at this

Shipped in v0.2


Unable to use plugins installed within a project

I attempted to add "esformatter-quote-props" to my .esformatter, after properly running npm install --save-dev esformatter-quote-props (which was successful).
Upon trying to format a .js file, I received this error within Atom's dev tools:
/Users/ckknight/.atom/packages/esformatter/index.js:27 Error: Error: Cannot find plugin 'esformatter-quote-props'. Make sure you used the correct name on the config file or run `npm install --save-dev esformatter-quote-props` to add it as a project dependency.

I also attempted this with the esformatter-spaced-lined-comment plugin, with a similar error.

This is already documented: https://github.com/sindresorhus/atom-esformatter#config It's just easier to bundle all plugins.
https://github.com/sindresorhus/atom-esformatter/commit/1acafce6efa49c0062caa44e9a7a1f11cda81622

Hi, I have faced the same issue when I attempted to add esformatter-ignore to my .esformatter. I have installed the plugin by npm install --save-dev esformatter-ignore but still doesn't work. Is this issue fixed or need your help to add the plugin?
Thanks.

By "faced the same issue", you mean not reading the documentation too, then you're indeed correct:

Some plugins are bundled. PR welcome for more.

Sorry about that, I just misunderstood the document. I have added a plugin, thanks for merging.

I think it would be beneficial for atom-esformatter to use esformatter and its plugins from the project in preference to what is shipped with the package. The benefits of this approach that I see are:

The same esformatter version can be used regardless of whether I use atom, another text editor or call it from the console.
Upgrades to esformatter and upgrades/installs of new plugins can be done quicker since I can continue to use npm install and don't have to create a PR.
If I have a project that relies on a specific version of esformatter and plugin I can easily control this via npm.

Shortly I will submit a PR that looks for esformatter in the project. Let me know what you think.


test: add type tests

Codecov Report

Merging #18 (b08407a) into main (7db10c4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage   94.21%   94.21%
=======================================
  Files           6        6
  Lines         121      121
  Branches       37       37
=======================================
  Hits          114      114
  Misses          7        7

Flag
Coverage Δ

unittests
94.21% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


#285: Allow users to include/exclude rules via config file

Users can now enable/disable specific rules by using only/ except options in the .tailor.yml configuration file.
TODO:

[x] Document feature in README
[x] Add tests

Resolves #285.

:+1:

:+1:


Speedup forward pass by more than 10x

Change for loop by single-line command. Works for both float and cuda.

Thank you. Please, check my comment and force-push. Then I'll merge it.

Thank you! Before merging, could you name your commit DEV - <description> and remove the ,_?

Done.

Thank you, @melgor!


buttonInteraction.js:7:32

TypeError: Cannot read properties of undefined (reading 'get')
const button = client.buttons.get(interaction.customId);
is throwing an error every time a button is pushed even if the button doesn't belong to the current bot

Remember to define client.buttons in index.js
client.buttons = new Collection()


Create Setup

Description:
Describe your changes.
Related issue:
Add link to the related issue.
Check list:

[ ] Mark if documentation changes are required.
[ ] Mark if tests were added or updated to cover the changes.

Hello @420TOKER. For now I'm going to close the pull request because it looks like a spam.

Please explain+


Fulcio certs: Encode the CI/CD "environment" in an extension?

Problem / value statement
The current Fulcio X.509 extensions include Runner Environment, which corresponds to a GitHub Actions infrastructure scope, e.g. github-hosted or self-hosted or similar.
However, there's another kind of "environment" in GitHub Actions (and possibly other CI/CD providers) as well: job within workflows can configure deployment environments, which have user-controlled names and can be mapped to deployment protection rules.
For example, here's an example of an environment (named pypi) used in an individual workflow job:
https://github.com/trailofbits/pypi-attestations/blob/464bd18b42b7ba207d6a77e9cca468116aa0406b/.github/workflows/release.yml#L39-L41
When a job references an environment, its OIDC claims include references to that environment. In particular, the sub claim contains the environment:$NAME clause, and there's also a separate environment claim that contains the environment name directly. Examples of this can be seen in GitHub's OIDC docs.
At the moment, it looks like Fulcio issued certificates don't reflect these user-controlled deployment environments, probably because there's no dedicated OID/extension definition for them. For example, this recent log entry has no reference to the pypi environment it came from. This is true even for the subject, since Fulcio rewrites the SAN away from the sub claim provided by the GitHub Actions OIDC credential.
It would be useful to include the deployment environment's name in the Fulcio extensions for a few reasons:

A deployment environment can correspond to deployment protection rules, making it possible to confirm (by the repo owner at least) that a workflow ran with explicit user approval or other checks.
Other OIDC applications, like Trusted Publishing, use the deployment environment as a binding piece of metadata for the publisher's "identity". It'd be nice if these identities could fully align with what Fulcio can provide.

Proposal

Define a new OID + extension for Deployment Environment or similar (maybe there's a better name)
Define Deployment Environment as corresponding to the environment claim on GitHub Actions and GitLab CI/CD (where it appears to have similar semantics)
Begin emitting Fulcio certs with the new extension where appropriate

CC @haydentherapper @segiddins as interested parties 🙂

TIL, I wasn't aware of this feature! This is neat, would be a nice way to enforce multi-party review for releases.
I'm supportive of adding this as an extension.
cc @kommendorkapten @feelepxyz for GitHub's perspective. Let's also loop in the other CI providers on the documentation update.

Interesting scope with multi-party review for releases! We have discussed the usage of the environment before, but not decided to pursue it, the use-case have been unclear and no real demand for it (yet).
If we can document this properly, I think it may add value for more advanced users. The GitHub Actions environment feature is really good for managing releases and deployments, I'm using that for a ton of things. Getting the feature carried over to a signing certificate can definitely help in some scenarios.
For things like OIDC based trusted publishing (or internally build and deployed software), a feature like environment makes a ton of sense, as the trust model is managed completely by a "single entity", which knows about the expected values. The downside of the environment claim for the general OSS use-case is that it's not as clear how it should be used during verification (as it's just an opaque producer defined string), but providing this option does not hurt, as the producer can specify what information is expected OOB to the consumers.

For things like OIDC based trusted publishing (or internally build and deployed software), a feature like environment makes a ton of sense, as the trust model is managed completely by a "single entity", which knows about the expected values. The downside of the environment claim for the general OSS use-case is that it's not as clear how it should be used during verification (as it's just an opaque producer defined string), but providing this option does not hurt, as the producer can specify what information is expected OOB to the consumers.

Yep, this aligns with what I was thinking! It's essentially just an opaque string from the verifying side, but when used in conjunction with Trusted Publishing it's the missing piece towards aligning the two "identities" completely 🙂


We should do a better job of explaining homeservers signing requests

Now that we have multiple APIs which can be signed outside of s2s, we should generalize the concept

For the record, the algorithm used in the IS is the same as that in the S2S API (see https://matrix.org/docs/spec/server_server/r0.1.4#request-authentication). That means that any implementation capable of receiving such requests needs to be capable of retrieving the requesting server's signing keys, which in turn means implementing the federation routing algorithm, complete with SRV lookup.


skip failing jmx metrics test

Looks like it's been a while since anyone's looked at https://issues.apache.org/jira/browse/CASSANDRA-10845, so I figured I'd skip this for the time being. Any objections?

+1

+2


Update groups to passwordfile in authentication.md

ref to https://github.com/kubernetes/kubernetes/pull/39651

This change is

@devin-donnelly this is for 1.6.  Is master open for that yet?

@deads2k PTAL

lgtm


Move Shared object from tfx_bsl

R: @rose-rong-liu @katsiapis
I would like to move Shared from (https://github.com/tensorflow/tfx-bsl/blob/master/tfx_bsl/beam/shared.py) to Beam, since it is probably generally useful for other Beam users. Does this move sounds reasonable to you?
The change is mostly a copy paste, change to copyright headers, unittest framework, and typehints.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Thanks Ahmet! I think it would be very nice to have it commonly available in beam.
On tfx part, as we have other libraries (e.g. TFMA) that depends on the shared object, and all of them have google3 as the source of truth, therefore it will be a google3 change.

R: @tysonjh

Run Python PreCommit

LGTM - thank you!

Optional comments:

is there an associated BEAM jira issue that can be used for the commit?
consider adding this to the CHANGES.md.

Run Python PreCommit

Run Python PreCommit

Optional comments:

is there an associated BEAM jira issue that can be used for the commit?
consider adding this to the CHANGES.md.

Done. Thank you!

Run Python PreCommit

Run Python PreCommit

Run Python2_PVR_Flink PreCommit


Add error object in task

Pull Request
What does this PR do?
Fixes #1877
PR checklist
Please check if your PR fulfills the following requirements:

[x] Update error test
[x] Remove flattening of errors during task serialization

bors merge


module/powermenu: Cannot open unexisting menu level

I'm using Arch Linux and have installed the Git version of polybar, is there something I am missing in my config?  When I click the power button I get this error:
warn: module/powermenu: Cannot open unexisting menu level '1342171104'
This is my config:
https://pastebin.com/ZCsZWAhs

The menu indices start at zero. You'll need to define menu-0-0 etc

Hi Dobbie03,
I think you have all actions defined under one index.
Here is my powermenu;
`
menu-0-0 = reboot
menu-0-0-exec = menu-open-1
menu-0-1 = power off
menu-0-1-exec = menu-open-2
menu-1-0 = cancel
menu-1-0-exec = menu-open-0
menu-1-1 = reboot
menu-1-1-exec = sudo reboot
menu-2-0 = power off
menu-2-0-exec = sudo poweroff
menu-2-1 = cancel
menu-2-1-exec = menu-open-0`
You see that each action has a defined index;
reboot=0
cancel=1
power off =2
You just need to follow the same rules.

Got it thanks chaps.
Closing this one.


Broken Link

fix broken link to post training quantization doc

Nagging Reviewer @MarkDaoust: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.


How to identify dataset

I did some thinking and encountered the following problem:
For datasets identification, we have the following cases:

Dataset connected to experiment
Dataset connected to object
Dataset anywhere identified by PermID
The first two cases have the same identifier "/SPACE/PROJECT/CODE". However, PyBis can't distinguish these two cases. We could solve this in two ways:

Ask the user to specify whether the data is on a collection or on an object
Try searching for both cases and take the one that gives a result. In case of ambiguity, give an error message.

I favor the second solution. What do you think?

I think that inside openBIS there cannot be an ambiguity for a dataset access path. That's my intuition as for the design principles.
pyBIS does not seem to worry about the potential ambiguity of paths. So I think we should not either.
Anyway, will try to ask Swen or Adam...

While I agree that there's no ambiguity inside openbis, we don't know a priori - given the identifier only - whether this identifier points to a collection or to an object. That's the ambiguity i refer about. I don't know what's the best way to resolve it.


Fix/17 replace mappers

Q
A

Branch?
6.2.x

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes

Fixed issues
chameleon-system/chameleon-system#17

License
MIT

With this PR we can handle incomplete mapper configuration. I'd prefer to avoid invalid data being written to database, but that would mean quite a bit of effort.

@rnuernberg I think logging isn't the right thing to do here. The log message is a very indirect way of telling the user that there is a problem; the log could even be read by another person as the one that introduced the wrong config (apart from cluttering the log with semi-relevant information). A better solution would be to validate input on save and present an error message there; as time is short, this fix simply leaves invalid input as is and writes invalid entries back.

As I now know, having an entry that only contains a name without further config is required for legacy modules in a theme. Therefore this is not a configuration error and should not be logged or aborted.
Line 85 should also be OK - when this line is called, the config line only consists of the name so that no information is lost.


Fix nltk.downloader warning

Fix nltk import warning at command line in GitHub workflow

coverage: 0.0%. remained the same
when pulling 22fdb01d8c49e0f7ad2d6d40e82d70b433f422e1 on bact:fix-nltk-download
into 0874f2eac1686d17912ecbc3280e0b39d6c72999 on PyThaiNLP:dev.


Add cloudformation helper scripts and a few improvements

Uninstall py-pip without creating unnecessary docker image layers. py-pip is not required at runtime.
Add cloudformation helper scripts consists of well-known cfn-init, cfn-signal, etc to be used for deepder integration with cloudformation. To be used for e.g. https://github.com/coreos/kube-aws/issues/49.
apk add --no-cache for not populating /var/apk/cache to reduce image size

ref https://github.com/coreos/kube-aws/issues/55
succeed to https://github.com/coreos/kube-aws/pull/56

Btw this change adds only less than 2%(0.4 MB) of size to the docker image!
Thus, I assume there's no issue regarding its size.
https://github.com/coreos/kube-aws/issues/55#issuecomment-260241753

LGTM
I'm going to say the && of expressions is fundamentally better for our use case now than separate RUN blocks (we're not interested the intermediary build layers in all likelihood)


Removing hardcoded version numbers from c# telemetry events.

Summary of the Pull Request
This removes the Hardcoded version # from certain telemetry events, by including the version # in BaseEvent.cs
PR Checklist

[x] Applies to #3040
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

Adds version information by leveraging calling into PowerToysInterop to get the product version.
Removes hardcoded version strings.
Updates the wxs file so that the FileExplorerPreview can find the PowerToysInterop.dll

Validation Steps Performed

Built PowerToys in Release.
Built PowerToysSetup in Release.
Installed the msi from \installer\PowerToysSetup\x64\Release
Locally updated the Fields In TelemetryBase to view events in the Diagnostic Data viewer.
Verified that the version is present in the expected telemetry events

We should actually create one assemblyInfo.cs which has the current version and all projects should add it as a link (not a copy, or make a post build script which will update all assembly infos in all projects) then all modules will have the same version controlled from one place and they can only query their own version when doing telemetry or anything else. All assemblies would then have the same version as the release.

@martinchrzan / @enricogior .  I've updated to get the version information from the assembly directly.  Not forcing a dependency on the PowerToysInterop.

This looks OK now, I will make sure if ColorPicker.exe has a proper version set in, but in general this is the way to go!

@martinchrzan
it doesn't

while PT Run does


Question - generation of SPL image (preloader) for Cyclone V

Hi robseb,
I'm in the process of using the socfpgaPlatformDesigner.py script for generating the SD image for Cyclone V.
I can see that the script has separate settings for the three Intel SoCs; Cyclone V, Arria V and Arria 10, regarding generation of SPL (preloader). I believe that I'll need to generate the SPL (preloader) image file since the SDRAM likely differs from the one that you are using in the default SPL file. Would you like to provide guidance reading what a developer like me need to set the "generate_spl_image_file" and "u_boot_bsp_qts_dir_list" to if the hardware peripherals differs from your designs?
`

Run the bootloader filter script (Cyclone V)

Cyclone V    |  Arria V     | Arria 10
run_filter_script =[True, True, False]
u_boot_bsp_qts_dir_list = ['/board/altera/cyclone5-socdk/qts/', '/board/altera/arria5-socdk/qts/',
' ']

Generate the bootable SPL image file

Cyclone V    |  Arria V     | Arria 10
generate_spl_image_file = [False,False,True]
#`

This link explains a lot of my concerns:
https://rocketboards.org/foswiki/Documentation/BuildingBootloader


Add directory scope

There needs to be a way to run different scripts per folder with one command. E.g. litr test executes different scripts on different folders.

One idea is using categories, defining sub commands that can run for specific glob patterns:
[commands.install]
script = "jest"

[commands.install.cpp]
script = "some install script"
pattern = **/cppRocks/*

Sub-commands and folder restrictions are added to https://github.com/krieselreihe/litr/wiki/Setup


RenderingException: The following has evaluated to null or missing

When creating a temporary e-mail address via API, the following exception may be logged. Note that I erased the real API token and address from the log.
2019-09-30 15:59:03 ERROR n.NinjaDefault:232 - Emitting bad request 500. Something really wrong when calling route<EMAIL_ADDRESS>(class: class controllers.BoxHandler method: public ninja.Result controllers.BoxHandler.createTemporaryMailAddress(java.lang.String,java.lang.String,java.lang.String,ninja.Context))
ninja.exceptions.RenderingException: The following has evaluated to null or missing:
==> emailAddress  [in template "views/BoxHandler/createTemporaryMailAddress.ftl.html" at line 6, column 46]

----
Tip: If the failing expression is known to be legally refer to something that's sometimes null or missing, either specify a default value like myOptionalVar!myDefault, or use <#if myOptionalVar??>when-present<#else>when-missing</#if>. (These only cover the last step of the expression; to cover the whole expression, use parenthesis: (myOptionalVar.foo)!myDefault, (myOptionalVar.foo)??
----

----
FTL stack trace ("~" means nesting-related):
	- Failed at: ${emailAddress} auto-escaped  [in template "views/BoxHandler/createTemporaryMailAddress.ftl.html" at line 6, column 44]
	~ Reached through: #nested  [in template "views/layout/defaultLayout.ftl.html" in macro "xcLayout" at line 57, column 11]
	~ Reached through: @layout.xcLayout title=i18n("editUser...  [in template "views/BoxHandler/createTemporaryMailAddress.ftl.html" at line 1, column 105]

The code performs some checks and if they fail, it is intended to return generic HTTP error responses (400, 401, 403). However, this is not done correctly so that Ninja still tries to render the template, but fails to do so because of missing input data, which in turn leads to a 500 response.
Now the standard Ninja error results are returned in these cases. They also support different formats (HTML, JSON, XML) as accepted by the client.


KAFKA-7462: Make token optional for OAuthBearerLoginModule

OAuthBearerLoginModule is used both on the server-side and client-side (similar to login modules for other mechanisms). OAUTHBEARER tokens are client credentials used only on the client-side to authenticate with servers, but the current implementation requires tokens to be provided on the server-side even if OAUTHBEARER is not used for inter-broker communication. Tokens should be optional for server-side login context to allow brokers to be configured without a token when OAUTHBEARER is not used for inter-broker communication.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@rondagostino Can you review, please? Thank you!

@rajinisivaram Before I review, is it possible simply to mark the LoginModule as OPTIONAL in the case where it is one of multiple login modules on the broker side but it is not used for inter-broker communication?  The Javadoc (https://docs.oracle.com/javase/7/docs/api/javax/security/auth/spi/LoginModule.html#commit()) says this means the module is ignored, and I confirmed in the debugger that it is in fact the case (see, for example, https://github.com/frohoff/jdk8u-dev-jdk/blob/master/src/share/classes/javax/security/auth/login/LoginContext.java#L779).

@rajinisivaram Ah, wait, I think I see the issue.  If we want to add a listener, then it may be the only LoginModule listed.  Is that correct?  In the meantime, I will review.

@rondagostino We need one login module that succeeds. With the sasl.jaas.config option, we only have one login module (because it is a per-mechanism config). With static JAAS config, we could have multiple modules if on the same listener, but I think OAuth requires a single login module (https://github.com/apache/kafka/blob/09f3205e44a0b1251e88edb73faa04b0c73521a2/clients/src/main/java/org/apache/kafka/common/security/oauthbearer/internals/unsecured/OAuthBearerUnsecuredValidatorCallbackHandler.java#L115). And as you said, you could have just OAuth on a listener. Thanks for reviewing!

retest this please

Merging to trunk and 2.0.

@kirktrue, @rajinisivaram Hi!
Looks like the same fix is needed for the new secured org.apache.kafka.common.security.oauthbearer.OAuthBearerLoginCallbackHandler
It also requires clientId/clientSecred even if oidc listener is not user for inter-broker communication.

@AntonSmolkov would you mind filing a new Jira to track? Thanks!


Would LIke To Connect

I have a question as to how to implement this code in my app. Just need a litle kick start and wondering if you could help.
I do not know where to contact you, so I thought I might try here. I did look at your x post, but says you are not accepting messages.
If you did want to connect my email is<EMAIL_ADDRESS>

Hi!
oh nice that you are interested in it :) Would loved to help you but I'm currently quite busy until at least next week. Left you an email.
My Twitter/X account is completely deserted, not using it anymore so good that you did approach me here 😁
Have a nice weekend!


lil sleep

"hmm, yeah I hadn't touched anything
I only found out because I was running both bots at the same time
and at the exact same time the one stopped working, the other one displayed a message "can't connect to steam server" then a minute later reconnected and kept going
steam literally only went down for a couple of seconds"

there

That code doesn't even work

this does work smh
i just tested it


Add FromRequestUnmarshaller in akka-http module

Hello!
It seems  that in order to have an Akka-http endpoint that accepts a request with xml in the body and decodes it we need a FromRequestUnmarshaller available, as it is expected from entity and as in the example bellow
(post & path("request") & entity(as[Request])){ req =>
      complete(OK, req)
}

It look like that the implementation can be the same as for the FromResponseUnmarshaller.
Can we add this? I can open a PR if that is okay

Hi!
That makes sense. Feel free to contribute this instance.
I wonder, if we can replace FromResponseUnmarshaller with FromEntityUnmarshaller? I think, it'll be a better option. If it does the trick, of course. I haven't touched this code for a while, so now I don't remember why it's only FromResponseUnmarshaller, not FromEntityUnmarshaller

Using FromEntityUnmarshaller makes sense and seems to work, I will see to open a PR.
Thank you for your reply 🙏

PR opened https://github.com/Tinkoff/phobos/pull/221

Merged and published in 0.16.0


ModuleNotFoundError: No module named 'solana.publickey'

code ：from solana.publickey import PublicKey

ModuleNotFoundError: No module named 'solana.publickey'

I have tried reinstalling the env

This module hasn't existed for almost two years. Are there out of date docs somewhere that told you to use it?
What you want is in Solders https://kevinheavey.github.io/solders/tutorials/pubkeys.html


Valence tick

Description
Describe the changes you've made. Link to any issues this PR fixes or addresses.
Adapted the bevy_time implementation to be used with Ticks as usize instead of Time as Instant/Duration, allows you to run systems with a fixed tick interval.
Quick note: This does (still) use CoreSchedule::FixedUpdate and I am not sure that it should, I looked briefly in how to add a custom Schedule thing, but that went way over my head.
What I am missing, (also from bevy itself) is how to make multiple FixedTick to run separate systems, I don't think it's possible, I would prefer to assign a system to a schedule, where you can have systems run at different intervals without needing to resort to something like https://github.com/valence-rs/valence/pull/377/files#diff-50197256120b6495759f6a7d5ff7276b9716bc0deddc48b62e174c0762e4ac00R68
But that might just be a bevy generic limitation from what I can tell.

Could be useful for #440

@Bafbi I don't think so. The behavior we'd want for rate limiting is more dynamic than just "send x packets every n ticks"

@Bafbi I don't think so. The behavior we'd want for rate limiting is more dynamic than just "send x packets every n ticks"

@dyc3  What do you mean? we would just not be able to the delay per entity (which is not really an indispensable feature) but would get the behavior wanted .
I would like to know what do you mean by dynamic ?

This is getting off topic for this PR, I'll respond in #440

Regardless in this PR's current state, it's only possible to have one FixedTick. Ideally, we'd have something that allows for more than 1 time interval to run systems at.

With only one FixedTick I don't see this having much use. Perhaps what you're looking for is something analogous to bevy's Timer?


docs: add link to nodejs/examples in initiatives

Adds a link to nodejs/examples now that the repo is up.

I've cleaned up the conflict. Ready to merge.

Thank you @joesepi!


Failed Assertion in _HttpClientResponse.listen

Code is at https://github.com/timsneath/dadjokes
With Dart 2 enabled, snapshot.hasError is true with an HTTP GET request; stepping into the debugger, I get an assertion failure.
Here's the result:

This same code works successfully with Dart 2 disabled (snapshot.hasError is false, snapshot.data contains a valid response.)

Here are the results of flutter doctor:
[✓] Flutter (Channel master, v0.1.5-pre.19, on Mac OS X 10.13.3 17D102, locale en-US)
    • Flutter version 0.1.5-pre.19 at /Users/timsneath/flutter
    • Framework revision 37ffb2d730 (3 hours ago), 2018-02-21 15:01:46 -0800
    • Engine revision ead227f118
    • Dart version 2.0.0-dev.28.0.flutter-0b4f01f759

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/timsneath/Library/Android/sdk
    • Android NDK at /Users/timsneath/Library/Android/sdk/ndk-bundle
    • Platform android-27, build-tools 27.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.4.0

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] VS Code (version 1.20.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Dart Code extension version 2.9.0-beta.4

[✓] Connected devices
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 8.1.0 (API 27) (emulator)

• No issues found!


Add CAA support to Azure Provider

Currently it is not supported, is it something on the roadmap?

Currently it is not supported, is it something on the roadmap?

Looks like support was added to Azure itself in mid-november. It should be relatively straightforward to add the new type to the provider.
/cc @h-hwang for thoughts

Ack. I'll put this on our backlog.

Just started setting up octodns and found that CAA and SOA records weren't supported. Any idea when these might get added?

SOA probably isn't going to be supported due to the inconsistencies in what, if anything, providers support editing. More info on that is in https://github.com/github/octodns/issues/39#issuecomment-303738180.
CAA is supported in general, but not (yet) by all providers. This issue is tracking adding CAA support to Azure, others already support it. There's a table in the README with the support matrix.

Another vote for CAA support in Azure. Thanks.

PRs accepted. I don't have access to Azure's DNS or time to dig into it really.


Globe show skirts option

This PR adds a new property globe.showSkirts to allow showing/hiding terrain skirts dynamically. The main use case is to hide skirts when viewing terrain from below the surface like in the clipping planes screenshot below.
The two main changes in this PR are:

Change the draw command count when skirts are disabled so that skirts don't get rendered.
For heightmap terrain, force skirt indices to be added after grid indices rather than interleaved. This brought along some optimizations like being able to generate and cache edge indices outside of the web worker. Quantized mesh terrain didn't require as many changes since skirt indices are already after grid indices there.

Local sandcastle

CC @fredj @gberaudo

Thanks for the pull request @lilleyse!

:heavy_check_mark: Signed CLA found.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

The PR looks good. I submitted my own commit with minor code changes including a restructuring of how skirts are generated by HeightmapTessellator. @lilleyse and I decided to move this to the February release to give it more time to breathe.
Even though this PR is done, there are some ideas for further skirt improvements:

Ability to reverse the direction of skirts when underground. One approach would be to generate skirts pointing below and above, and render the correct set depending on where the camera is.
Move skirt creation to a common function. Right now heightmap, quantized mesh, etc do it themselves but the logic is roughly the same.
Make skirts contiguous in the index buffer by ordering west indices south to north, north indices west to east, east indices north to south, and south indices east to west). This could allow rendering skirts as a triangle strip, though it might not be worth the extra draw call.
Create skirts on the corners of tiles.

None of these are high priority at this time.


Update documentation

Closes #736

Let's wait for #781 before continuing here

rebased onto current master. @willclarktech could you have a look?

Added 4 commits to address the feedback. Do they make sense to you @willclarktech?


add synchronicLinkLabeled

synchonicLinkLabeled:applying: message could be added to bootstrap component supplier in the same way that asynchonicLinkLabeled:applying: is implemented

Fixed by #213


x/sys/unix: unix.Connect cause uninterruptible sleep

I didn't find related section where this package belongs, please ignore and close this if it's not relevant. Also if you can point me to the right direction.
What version of Go are you using (go version)?

go version go1.17.8 linux/amd64

Does this issue reproduce with the latest release?
Didn't try with 1.18
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/loow/.cache/go-build"
GOENV="/home/loow/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/loow/development/go/pkg/mod"
GOOS="linux"
GOPATH="/home/loow/development/go"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.17.8"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build52921490=/tmp/go-build -gno-record-gcc-switches"

What did you do?
package main

import (
	"strconv"
	"strings"
	"syscall"

	"golang.org/x/sys/unix"
)

func main() {
	var addr = "54:88:2D:EE:EE:EE"
	fd, err := unix.Socket(syscall.AF_BLUETOOTH, syscall.SOCK_STREAM, unix.BTPROTO_RFCOMM)
	if err != nil {
		panic(err)
	}
	socketAddr := &unix.SockaddrRFCOMM{Addr: str2ba(addr), Channel: 6}
	if err := unix.Connect(fd, socketAddr); err != nil {
		panic(err)
	}
}

// str2ba converts MAC address string representation to little-endian byte array
func str2ba(addr string) [6]byte {
	a := strings.Split(addr, ":")
	var b [6]byte
	for i, tmp := range a {
		u, _ := strconv.ParseUint(tmp, 16, 8)
		b[len(b)-1-i] = byte(u)
	}
	return b
}

What did you expect to see?
It would be nice if I get an error if the particular device isn't available.
What did you see instead?
I got an uninterruptible sleep in case of the device determined in the address isn't available.
$ px aux | grep go-build85
XXX       35538  0.0  0.0 703112  1676 pts/2    D    10:59   0:00 /tmp/go-build851823850/b001/exe/main

$ sudo kill -9 35538

$ px aux | grep go-build85
XXX       35538  0.0  0.0 703112  1676 pts/2    D    10:59   0:00 /tmp/go-build851823850/b001/exe/main

All the Go code is doing is making the system call.  If the process locks up and becomes immune to SIGKILL, that sounds like a kernel bug.  There is nothing that Go can do to become immune to SIGKILL.
I imagine that the same thing would happen for an equivalent C program.
I'm going to close this because I don't think there is anything we can change in Go to affect this.  Please comment if you disagree.


Add indentation rule for closing tag of multi-line jsx

For issue #1206:
This rule validates that the closing tag for a multi-line jsx component matches the indentation of the opening tag.

@ljharb I think I've addressed your concerns. The one situation this doesn't fix is the following:
<App
  foo="bar">baz</App>

It will fix it to:
<App
  foo="bar">baz
</App>

When ideally it would be fixed to:
<App
  foo="bar">
  baz
</App>

I opted not to handle it in this rule because I imagine multiline children location should be handled in a separate rule entirely (maybe it already is?)

I agree with you that it should be handled in a separate rule.

Pardon my ignorance (this is the first eslint rule I've written), but isn't that what the output attributes of the invalid tests are for?

@rsolomon Would this rule also cover the situation below?
// Considered a warning
<App>
  { condition && <Component>
    ...
  </Component> }
</App>

// Not considered a warning
<App>
  { condition && (
    <Component>
      ...
    </Component>
  ) }
</App>


⚠ Bofrost.at has degraded performance

In c211c3d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1289 ms

Resolved: Bofrost.at performance has improved in 94dda98.


not found: type HttpRoutes

I am getting the above error when using using the http4s framework in Routes.scala with guardrail 0.41.2. It does indeed look like HttpRoutes is not being imported in.

ah yeah sorry, 0.18.12.

@SpyMachine It looks like the last guardrail version before the 0.19.x update for http4s was v0.41.0

I try to keep the breaking changes to minor version bumps, but this one seems to have slipped past


Add support for JavaScript in component tests

Feature request
Currently, component tests cannot run JavaScript. This means that components that use JavaScript cannot be tested until they are used in the system, at which point a system test can be used.
However, if the component's JavaScript is incorrect, this is too late to find this out — it would have been more useful to reject the PR implementing the component, rather than merge it and then find out when the component is used that it is wrong.
Would it be possible to add a documented way to use tests in JavaScript? I'm not sure whether simply rendering the preview would be enough, though this would need a way of specifying which preview, and also a way of preventing the test from interacting with the preview chrome. (Is there a way to render the preview without the surrounding chrome, perhaps?)
Motivation
I would like my employer to be able to require developers to provide tests for components that exercise all functionality of the component, including the JavaScript, so I do not find the components break when I have to use them on a page. Particularly if fixing it then falls to me, rather than the original implementer.

@unikitty37 the best way to do this is to test the components in a system tests with a preview


Must I use a domain name prefix?

Q
A

Bug?
no

New Feature?
no

Framework
Laravel

Framework version
5.8.26

Package version
2.3

PHP version
7.3

I used IP as the site to access, this is to avoid the confusion of DNS resolution in China, my project will not use the domain name

You can leave the API_DOMAIN blank, but you will need to set the API_PREFIX, yes (in which case, you can use it with any domain, so long as the prefix is correct). One of those two must be set though.


[Bug]: Command timed out, killing process.. when trying to use ssh

Is there an existing issue for the same bug?

[X] I have checked the troubleshooting document at https://opendevin.github.io/OpenDevin/modules/usage/troubleshooting
[X] I have checked the existing issues.

Describe the bug
I try to use bash ./evaluation/swe_bench/scripts/run_infer.sh gpt-4o  CodeActAgent 1 based on the document. But the program  stuck on the SSH connection.
v2024-05-31 21:02:00,300 - INFO - SSHBox is running as root user with USER_ID=1000 in the sandbox
2024-05-31 21:02:00,316 - INFO - Using existing Docker container
2024-05-31 21:02:00,317 - INFO - Connecting to root@localhost via ssh. If you encounter any issues, you can try `ssh -v -p 63710 root@localhost` with the password '123456' and report the issue on GitHub. If you started OpenDevin with `docker run`, you should try `ssh -v -p 63710 root@localhost` with the password '123456 on the host machine (where you started the container).
2024-05-31 21:06:01,863 - ERROR - Command timed out, killing process...
2024-05-31 21:06:01,863 - ERROR - Command timed out, killing process...
2024-05-31 21:08:24,866 - INFO - SSHBox is running as root user with USER_ID=1000 in the sandbox
2024-05-31 21:08:24,885 - INFO - Using existing Docker container
2024-05-31 21:08:24,886 - INFO - Connecting to root@localhost via ssh. If you encounter any issues, you can try `ssh -v -p 63710 root@localhost` with the password '123456' and report the issue on GitHub. If you started OpenDevin with `docker run`, you should try `ssh -v -p 63710 root@localhost` with the password '123456 on the host machine (where you started the container).
2024-05-31 21:12:26,384 - ERROR - Command timed out, killing process...
2024-05-31 21:12:26,384 - ERROR - Command timed out, killing process...

I try to  ssh -v -p 63710 root@localhost and input password "123456" then I login in successfully.
Current OpenDevin version
ghcr.io/opendevin/opendevin:0.5

Installation and Configuration
my config.toml is:

[core]
max_iterations = 100
cache_dir = "/tmp/cache"
sandbox_container_image = "ghcr.io/opendevin/sandbox:latest"
sandbox_type = "ssh"
ssh_hostname = "localhost"
sandbox_timeout = 120
ssh_password = "123456"
persist_sandbox = false

# SWEBench eval specific
use_host_network = false
run_as_devin = false
enable_auto_lint = true

# TODO: Change these to the model you want to evaluate
[gpt-4o]
model = "gpt-4o"
api_key = "sk-xxx"
temperature = 0.0

[eval_some_openai_compatible_model]
model = "openai/MODEL_NAME"
base_url = "https://OPENAI_COMPATIBLE_URL/v1"
api_key = "XXX"
temperature = 0.0

Model and Agent
No response
Operating System
wsl2 ubuntu 22.04
Reproduction Steps
No response
Logs, Errors, Screenshots, and Additional Context
No response

2024-05-31 21:06:01,863 - ERROR - Command timed out, killing process...

Seems plugin initialization is timed out.

Closed by https://github.com/OpenDevin/OpenDevin/pull/2172
Please let me know if you meet any other issue!

Closed by #2172
Please let me know if you meet any other issue!

Thanks! It acctually solved my problem and it ran smoothly now.


Allow functions

This is a feature request, can we allow the support of functions? I know this have been raised in the past, and the solution was gettered, but here is a getter:
 get sayHello() {
            console.log('saying hello...');
            return `echo "Done"`
 }

And the getter is ending up being called three times.
➜  nps sayHello
saying hello...
saying hello...
saying hello...
nps is executing `sayHello` : echo "Done"
Done

Also maybe supporting functions that return a string, or even functions that return a promise that resolves to a string for asynchronous code - could be great for different use cases.
###USE CASE:
I have a project that needs to run some commands and update a file when the command is called, so I want to be able to use fs when calling the script.

Hey Liron, I couldn't reproduce your issue with the sayHello getter being called three times. Could you show how it's imported to package-scripts?

For your second part regarding functions that return strings, especially async, it's a feature that we've discussed in the past and don't want to implement to avoid adding further complexity to nps. The example with fs can be referred to in #184. Sorry, but thanks for the issue!

@kevjin Thank you for your time.
scripts.js
module.exports = {
  scripts: {
     get sayHello() {
            console.log('saying hello...');
            return `echo "Done"`
 }
  }
}

and calling it with nps v5.9.3 installed globally:
nps sayHello

@kevjin
I checked the possibility to add a function instead of just a string.
I can do it in 5 lines of code worth of change.
And I'll write the tests for it.
Can I fork and make a PR? or is it useless and you honestly don't want this feature?

@kevjin
Made a PR, it's kinda big but its 90% tests can you please review it? do I need to add anyone specific as a reviewer here?
https://github.com/kentcdodds/nps/pull/197

Hey @liron-navon, I saw the PR you made and appreciate your enthusiasm for supporting this feature and nps. We closed most of the issues you referenced in your PR because supporting functions is something we explicitly stated we do not want to do, for the reason of too much complexity as I mentioned above

Too bad, please close this issue.

Hey @liron-navon, I saw the PR you made and appreciate your enthusiasm for supporting this feature and nps. We closed most of the issues you referenced in your PR because supporting functions is something we explicitly stated we do not want to do in our roadmap, for the reason of complexity as I mentioned above. I think @kentcdodds would agree with me regarding this issue and PR.
Sorry about this, but again, I really appreciate you taking your time to write this. Thanks!

Hey Liron, I couldn't reproduce your issue with the sayHello getter being called three times. Could you show how it's imported to package-scripts?

I found out why the triple-log issue showed up for Liron but not others: the issue only shows up when the getter is "nested" inside a group.
So this has the issue:
module.exports = {
  scripts: {
     get sayHello() {
            console.log('saying hello...');
            return `echo "Done"`
     }
  }
}

// in terminal
nps sayHello

But this is fine:
module.exports = {
  scripts: {
     get sayHello() {
            console.log('saying hello...');
            return `echo "Done"`
     }
  }
}

// in terminal
nps sayHello

I found out why the triple-log is occurring by the way.
The reason is that the scripts.sayHello entry is having a copy of the entry added under the key say-hello, from this line: https://github.com/sezna/nps/blob/57989a24ff6876b3d5245f7e00b76aaf39296d31/src/get-script-to-run.js#L9
Now that exports.scripts has two keys (sayHello and say-hello) with the same accessor/getter function, it causes it that getter to run twice once this line is hit: https://github.com/sezna/nps/blob/57989a24ff6876b3d5245f7e00b76aaf39296d31/src/get-script-to-run.js#L12

@kevjin By the way, you might be happy to hear that I found a way to achieve your feature request by adding some monkey-patching within the package-scripts.js file. It's kinda hacky, but it works, and doesn't take that much boilerplate:
const Dynamic = commandStrGetter=>{
	const result = new String("[placeholder for dynamically-evaluated command-string]");
	result.commandStrGetter = commandStrGetter;
	return result;
};
const join_orig = Array.prototype.join;
Array.prototype.join = function(...args) {
	// If we're concatenating the script-entry with its args (just before execution)...
	//	...and we find a String object produced by the Dynamic function above (rather than a primitive string like normal)...
	//	...then intercept and replace the String object with the result of its commandStrGetter().
	// Here is the relevant location in the nps source code: https://github.com/sezna/nps/blob/57989a24ff6876b3d5245f7e00b76aaf39296d31/src/index.js#L59
	if (this[0] instanceof String && this[0].commandStrGetter != null) {
		this[0] = this[0].commandStrGetter();
	}
	return join_orig.apply(this, args);
};

exports.scripts = {
	sayHello: Dynamic(()=>{
		console.log('saying hello...');
		return `echo "Done"`
	}),
	otherScript: "echo otherScriptDone",
};

With the package-scripts.js above, here are the results: (confirmed just now in fresh project)
> nps sayHello

saying hello...
nps is executing `sayHello` : echo "Done"
"Done"

> nps otherScript

nps is executing `otherScript` : echo otherScriptDone
otherScriptDone

(Note that saying hello... is logged for the first case, but not for the second. 🙂)


fix: correctly config multiple address for frontend

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

Comparison is base (5718739) 38.21% compared to head (5c50fc1) 38.21%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #186   +/-   ##
========================================
  Coverage    38.21%   38.21%
========================================
  Files           16       16
  Lines         1285     1285
========================================
  Hits           491      491
  Misses         689      689
  Partials       105      105

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


Updated WeaverCore and fixed IKG invincibility bug

lotta commits and merges

Guess I messed something up when creating the request, since most of those commits are pretty old


Multilingual Text-to-Speech and Speech-to-Text Functionality

This feature enables users to interact with the website using text-to-speech and speech-to-text functionalities in multiple languages. Users can select their preferred language for both text-to-speech output and speech-to-text input, catering to a diverse user base and enhancing accessibility. This feature will integrate with the backend to support language-specific models and ensure accurate and efficient processing of text and speech data.
Use Case
Problem Statement:
Many users face language barriers and accessibility challenges when interacting with websites. Non-native speakers and users with disabilities may find it difficult to read or type in a single language interface, leading to frustration and decreased engagement.
Use Case Description:
Objective: To provide a personalized, accessible, and inclusive user experience by offering multilingual text-to-speech and speech-to-text capabilities.
Multilingual Text-to-Speech:
Current State: The website lacks the ability to read text aloud in different languages, limiting accessibility for users who prefer auditory content consumption.
Future State: Users can select their preferred language from a list of supported languages, and the website will read aloud the text in the selected language. This is useful for users with visual impairments, language learners, and those who prefer listening over reading.
Multilingual Speech-to-Text:
Current State: Users can only input text through typing, which may not be convenient for everyone, especially those with physical disabilities or those who prefer speaking over typing.
Future State: Users can speak into their device’s microphone, and the website will convert their speech into text in their selected language. This feature will support voice commands and dictation, enhancing usability and efficiency.
Benefits:
Enhanced Accessibility: Improves the experience for users with visual impairments or physical disabilities, making the website more inclusive.
User Convenience: Offers a more natural way for users to interact with the website, particularly for those who prefer auditory input/output.
Language Support: Accommodates a diverse user base by supporting multiple languages, making the website accessible to non-native speakers and language learners.
Improved Engagement: By catering to individual language preferences and accessibility needs, users are more likely to engage with the content and services provided by the website.
User Story:
As a user, I want to listen to the website content in my preferred language, so I can easily understand and consume information.
As a user, I want to use speech-to-text in my native language, so I can interact with the website more comfortably and efficiently.
Implementation Goals:
Language Selection: Provide an easy-to-use interface for users to select their preferred language for both text-to-speech and speech-to-text.
Backend Integration: Connect with backend services that support multiple languages for text-to-speech synthesis and speech-to-text recognition.
Accurate Processing: Ensure high accuracy in language processing to provide a smooth user experience.
Responsive Design: Make the feature accessible across various devices, including desktops, tablets, and mobile phones.
Security and Privacy: Implement robust security measures to protect user data during voice interactions.
By incorporating multilingual text-to-speech and speech-to-text functionalities, this feature aims to create a more inclusive, accessible, and user-friendly platform, ultimately enhancing user satisfaction and engagement.

Assign me this issue under GSSoC'24

@ShampaShaw this project currently not having any books for reading purpose?
So, you have to go with the language conversion model.


Sometimes I can't write the update to ID3V2

Hi,
Sometimes I can't write the update to ID3V2, but ID3V1 works.
I check out that there're permission to decide which ID3 version be writable.

Could it be modified before writing with the script? Thanks!

Using the python library, I've experienced a similar problem. I update values for several tags, including the band tag. When I print the tags after saving them, all the other changes are shown but there's no entry for band.
But if I then open the mp3 file in a GUI editor, say like MP3Tag, the band tag change that I made is shown there.
I've encountered this error when trying to set a value for band when there's not been one set. If there's already a value for band and I just change it, printing the tags displays the change.


feat(cli): append env key to flag's usage automatically

What this PR does
this is an cli example

Usage:
  example [flags]

Flags:
  -a, --auto string   (env $EXAMPLE_AUTO)
  -h, --help          help for example
  -l, --log string    (env $LOG) (default "test")

why we need it
Special notes for your reviewer:

Coverage increased (+0.8%) to 54.318% when pulling 325c6c6696f32e643896822ff839feac1ac6d82d on zoumo:cli into 0fd9cc75a205a7770b77aeb2453c947e5e03e22b on caicloud:master.

this is an cli example

this is a cli example

Coverage increased (+0.8%) to 54.318% when pulling a6fd09cdf18aa2deafb54e4026a2a912d8ef7531 on zoumo:cli into e1555353a4da96f72996c0dd7a537bfceafe672d on caicloud:master.

🚢


How can I get a background colour to use it in a case definition

Hi,
I'm loading the doom-themes using use-package and I'm trying to customise some faces. I currently have this
:custom-face
  (tab-line ((t (:inherit mode-line :box (:line-width 6 :color "#323845")))))

but I would line to get the :color definition from the background of the mode-line face defined by the theme
Is that possible?

but I would line to get the :color definition from the background of the mode-line face defined by the theme
Is that possible?

Sorry for the late response. You can get the mode-line background with either:

(doom-color 'modeline-bg)
(face-background 'mode-line)

Hope that helps!


Can't get an email message when two searchParameters added

Hello,
when I use sentTo and subject parameters at the same time:
cy.mailosaurGetMessage(serverId, {
    sentTo: emailAddress,
    subject: 'Aktivierung Ihres Zugangs',
}, { timeout: 30000 }
).then((email) => {
    const confirmationCode = /invitationCode=(\d{6})/.exec(email.html.body)[1];
});

then I get really often this error:

And what is strange when I change to:
cy.mailosaurGetMessage(serverId, {
    sentTo: emailAddress,
}, { timeout: 30000 }
).then((email) => {
    const confirmationCode = /invitationCode=(\d{6})/.exec(email.html.body)[1];
    expect(email.subject).to.equal('Aktivierung Ihres Zugangs');
});

I don't get the error The exception appears but only from time to time.
Do you have any idea why it doesn't work or what I do wrong?
Edit:
Timeline
13:42:18 - click register button (send confirmation code via email)
[execute cy.mailosourGetMessage]
13:42:21 - email appears in mailosaur mailbox
13:42:48 exception thrown (screenshot above)

Closing as not related to the client itself, discussion continuing in support channel.


[Bug]: jaeger-operator v1.57.0 query.base-path is broken when strategy is production and storage.type is memory

What happened?
I tried to deploy a jaeger instance with strategy=production and custom query.base-path.
apiVersion: jaegertracing.io/v1
kind: Jaeger
metadata:
  name: jaeger
spec:
  strategy: production
  query:
    options:
      query:
        base-path: /jaeger

I couldn't access the query UI through web browser. I notice two things that may explain why it doesn't work.
[1] The jaeger container command argument misses --query.base-path=/jaeger.
...
spec:
  containers:
  - args:
    - --sampling.strategies-file=/etc/jaeger/sampling/sampling.json
    env:
    - name: SPAN_STORAGE_TYPE
      value: memory

[2] The ingress doesn't have the expected rules for path=/jaeger.
...
spec:
  defaultBackend:
    service:
      name: jaeger-query
      port:
        number: 16686

Steps to reproduce

kubectl apply -f jaeger.yaml (contents shown in the above section)
review the generated deployment and ingress. (also shown in the above section)

Expected behavior
The command argument includes --query.base-path=/jaeger
Relevant log output
No response
Screenshot
No response
Additional context
No response
Jaeger backend version
No response
SDK
No response
Pipeline
No response
Stogage backend
No response
Operating system
No response
Deployment model
No response
Deployment configs
No response

Hi @yanqi27. Thanks for reporting. Would you like to send a pull request?

@iblancasa
This happens only for storage.type==memory from my test. Therefore, it might be a minor issue. Though I am interested in fixing it, I am not familiar with the repo and I am currently tied with other works.

I am experiencing the same problem even with the default strategy (all-in-one), query base path is not being set therefore the UI is not working since it is trying to pull the static files from the wrong path.


[video_player] add setBitrate function

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
List which issues are fixed by this PR. You must list at least one issue.
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[ ] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Hey @apieceofcode1801 do you need help with this feature? I'm interested in bringing these things in chewie to allow the user to pick the quality they want.

Hey @apieceofcode1801 do you need help with this feature? I'm interested in bringing these things in chewie to allow the user to pick the quality they want.

I implemented setBitrate function in this PR, but I have no experience in creating PR for Flutter plugin so many checks are not passed. For these, I have not yet time to continue so I'm waiting to continue this feature later.
If you can help to review, it's appreciated.

@apieceofcode1801 Are you still interested in working on this one? If so, you may be able to get help by asking on #hackers-new in our Discord server (see link in the contributing guidelines). If you would like a review, marking this as ready for review (as opposed to draft) will bring it onto the radar of the people doing code review.

Since this is marked as a draft and hasn't been updated in several months I'm going to close it to clean out our review queue. Please don't hesitate to submit a new PR if you decide to revisit this. Thanks!


[Enhancement]: Improve the color selector styling

Do you want to have the enhancement of existing game ? 😀 Describe yourself..

i can improve the styling of this color panel, i can style it according to the theme of the game, also once it opens up, it does not close back unless the page is reloaded.
Describe the solution you'd like
Solution steps
Improve styling, adding borders and more enhancing styles to the buttons to make it look more interactive.
Select program in which you are contributing
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Please assign me this issue.

@lassmara
Please raise any other issue, i'll assign you
If you have any doubt you can directly send message on our discord (https://discord.gg/rZb46cCMmK), we have created separate channals for all projects...
Thank you :)

@AditiPrabhakar Don't forget to maintain responsiveness


Provide armhf variants

Provide duplicated versions of the two templates which end with "-armhf" as the name.
This is to add better support for our ARM users.

@alexellis: I believe this issue can be closed. I must have missed the "closing #" annotation in the PR :neutral_face:


Equivalent of printf()?

Is there an equivalent of printf() for GNU Coreutils or C in Liquid?  If not, how difficult would it be to add one?

I don't think there is one. Adding a wrapper around sprintf should not be too hard.


Use ConcurrentHashMap to fix flaky test 1031

Fix https://github.com/gradle/gradle-private/issues/1031
This PR replace original Map in DefaultListenerManager with ConcurrentHashMap
to avoid concurrent access issue. A test is added to reproduce the issue.
@eskatos Since you're familiar with the context, could you take a look at it?

@blindpirate, I'm off till Wednesday, if you need a quick review of this then maybe ask @melix, I'd say he has more context than me in this area, otherwise I'll pick that up on Wednesday


[go_router] handle alt+rightArrow or alt+leftArrow keys to navigate back and forth in the current history stack

copied from https://github.com/csells/go_router/issues/267
Discussed in https://github.com/csells/go_router/discussions/212
Originally posted by Agondev November 25, 2021
By wrapping the MaterialApp's builder's child with CallbackShortcut and passing various options of popping route, like
Navigator.of(context)

or
context.pop()

or
GoRouterDelegate

or
_goRouter.pop()

or
Router.of(context).routerDelegate.popRoute()
etc.
I get exceptions like
Router operation requested with a context that does not include a Router.
and
No GoRouter found in context.
Only thing that seems to kind of work is appRouter.navigator?.pop(), and that requires 2 attempts after the first time. With Auto Route it's enough to call _autoRoute.pop() from anywhere in the app using a shortcut of my choosing for it to pop the closest encompassing route.
Can the same not be done in GoRouter?
Example of my latest draft of MaterialApp
return MaterialApp.router(
  routeInformationParser: _router.routeInformationParser,
  routerDelegate: _router.routerDelegate,
  builder: (context, child) {
    CallbackShortcuts(
        bindings: {
          LogicalKeySet(
                    LogicalKeyboardKey.escape,
                  ): () => appRouter.navigator?.pop(),
                    LogicalKeyboardKey.escape,
                    LogicalKeyboardKey.alt,
                  ): () => appRouter.pop(), // or context.pop()
        }
        child: child!,
    )
  }
);

To reiterate, the first binding works in GoRouter after two attempts, the second one just throws. But it does work in Auto Route with no problems.

Hi @noga-dev, Are you still experiencing the same error?
I tried with the below code sample and I am unable to reproduce the issue.
The exception I get is this
"have popped the last page off of the stack; there are no pages left to show

which is intended since we are popping all routes.

logs
Restarted application in 147ms.
══╡ EXCEPTION CAUGHT BY SERVICES LIBRARY ╞══════════════════════════════════════════════════════════
The following assertion was thrown while processing the key message handler:
Assertion failed:
file:///Users/mahesh/.pub-cache/hosted/pub.dartlang.org/go_router-3.0.7/lib/src/go_router_delegate.dart:177:12
_matches.isNotEmpty
"have popped the last page off of the stack; there are no pages left to show"

When the exception was thrown, this was the stack:
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 251:49  throw_
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 29:3    assertFailed
packages/go_router/src/go_router_delegate.dart 177:21                         pop
packages/go_router/src/go_router_delegate.dart 695:11                         <fn>
packages/flutter/src/widgets/navigator.dart 4900:18                           pop
packages/new_app/main.dart 23:23                                              <fn>
packages/flutter/src/widgets/shortcuts.dart 1047:25                           [_applyKeyBinding]
packages/flutter/src/widgets/shortcuts.dart 1062:20                           <fn>
packages/flutter/src/widgets/focus_manager.dart 1687:25                       [_handleKeyMessage]
packages/flutter/src/services/hardware_keyboard.dart 828:16                   [_dispatchKeyMessage]
packages/flutter/src/services/hardware_keyboard.dart 875:73                   handleRawKeyMessage
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54            runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 123:5            _async
packages/flutter/src/services/hardware_keyboard.dart 854:51                   handleRawKeyMessage
packages/flutter/src/services/platform_channel.dart 77:49                     <fn>
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54            runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 123:5            _async
packages/flutter/src/services/platform_channel.dart 76:58                     <fn>
packages/flutter/src/services/binding.dart 380:35                             <fn>
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54            runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 123:5            _async
packages/flutter/src/services/binding.dart 377:98                             <fn>
lib/_engine/engine/platform_dispatcher.dart 1121:13                           invoke2
lib/ui/channel_buffers.dart 25:12                                             invoke
lib/ui/channel_buffers.dart 65:7                                              push
lib/ui/channel_buffers.dart 130:16                                            push

code sample
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:go_router/go_router.dart';

void main() => runApp(App());

class App extends StatelessWidget {
  App({Key? key}) : super(key: key);

  static const title = 'GoRouter Example: Declarative Routes';

  @override
  Widget build(BuildContext context) => MaterialApp.router(
      routeInformationParser: _router.routeInformationParser,
      routerDelegate: _router.routerDelegate,
      builder: (context, child) {
        return CallbackShortcuts(
          bindings: {
            LogicalKeySet(LogicalKeyboardKey.escape): () {
              _router.navigator!.pop();
            },
            LogicalKeySet(LogicalKeyboardKey.shift, LogicalKeyboardKey.keyZ):
                () {
              _router.navigator!.pop();
            },
          },
          child: child!,
        );
      });

  final _router = GoRouter(
    routes: [
      GoRoute(
        path: '/',
        builder: (context, state) => const Page1Screen(),
      ),
      GoRoute(
        path: '/page2',
        builder: (context, state) => const Page2Screen(),
      ),
    ],
  );
}

class Page1Screen extends StatelessWidget {
  const Page1Screen({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(title: const Text(App.title)),
        body: Center(
          child: Column(
            mainAxisAlignment: MainAxisAlignment.center,
            children: [
              ElevatedButton(
                onPressed: () => context.go('/page2'),
                child: const Text('Go to page 2'),
              ),
            ],
          ),
        ),
      );
}

class Page2Screen extends StatelessWidget {
  const Page2Screen({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(title: const Text(App.title)),
        body: Center(
          child: Column(
            mainAxisAlignment: MainAxisAlignment.center,
            children: [
              ElevatedButton(
                onPressed: () => context.go('/'),
                child: const Text('Go to home page'),
              ),
            ],
          ),
        ),
      );
}

flutter doctor -v (mac)
[✓] Flutter (Channel stable, 2.10.4, on macOS 12.3 21E230 darwin-arm, locale en-IN)
    • Flutter version 2.10.4 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c860cba910 (13 days ago), 2022-03-25 00:23:12 -0500
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.65.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.39.20220405

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 535317A4-921C-488F-B234-C2CC6D4AE5A1 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5
      (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.3 21E230 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 100.0.4896.75

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

the default shortcuts cmd+arrow keys left/right works fine too

Mac? Web? Haven't tested on either, and I assume that's where you mean it works? I meant on desktop (win), and maybe mobile too using Bluetooth keyboard (haven't tested).
As for the other issue, the shortcut will still need to be pressed twice following the 1st time in order to work. So the 1st time you press the shortcut it works. Any subsequent attempts will necessitate doing so twice.
Note that you shouldn't use escape as a shortcut since it has a default behavior so your example is problematic.
This is what I use on desktop:
          LogicalKeySet(
            LogicalKeyboardKey.arrowLeft,
            LogicalKeyboardKey.alt,
          ): () {
            if (appRouter.navigator != null &&
                (appRouter.navigator?.canPop() ?? false)) {
              appRouter.navigator?.pop();
            }
          }

@maheshmnj


Deeplink error

Deeplink when you don't have the app installed doesn't take you to the store!
basically i need to have the app installed for deeplink to work.
WalletLink.createWalletLinkUrl({
  universalLink: app.universalLink,
  requestAppId: getBundleId(),
  callbackUrl: 'hotspottyconnectscheme://',
  appName: 'Hotspotty',
})

https://user-images.githubusercontent.com/37663993/173549582-e9b6f079-abe9-4ab0-8ab2-8461a525f43a.mp4

PR #50


Removing vertical gutters from default and adding them as modifiers

This addresses #2.
@davidberner can you review?

Lgtm


Cannot red OS Specific Keyfiles

[ x ] If you are using gdb

[ x ] gdb --version >= 7.7.1
[ x ] it works on the command line with gdb
[ x ] cwd and target are properly set

Hi, i am trying to create a configuration .json that can be used by both my Linux Machine and my Windows machine at home.
I tried using something that Microsoft recommended, namely breaking up the variables by OS as such:
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "type": "gdb",
            "request": "launch",
            "name": "Launch Program (SSH)",
            "target": "./build/debug/file1",
            "cwd": "${workspaceFolder}/",
            "ssh": {
                "host": "seas.upenn.edu",
                "cwd": "/mnt/rrio1/home/tabedzki/vscode-cpp-remote-debug",
                "windows":{
                    "keyfile": "C:/Users/tabedzki/.ssh/id_rsa"
                },
                "linux": {
                    "keyfile": "${env:HOME}/.ssh/id_rsa"
                  },
                "osx": {
                    "keyfile": "${env:HOME}/.ssh/id_rsa"
                  },
                "user": "tabedzki"
            }
        }
    ]
}

However, this ends up with the error Error: All configured authentication methods failed. However, if I explicit refer to the keyfile path (on both machines, Linux shown) as such
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "type": "gdb",
            "request": "launch",
            "name": "Launch Program (SSH)",
            "target": "./build/debug/file1",
            "cwd": "${workspaceFolder}/",
            "ssh": {
                "host": "seas.upenn.edu",
                "cwd": "/mnt/rrio1/home/tabedzki/vscode-cpp-remote-debug",
                    "keyfile": "${env:HOME}/.ssh/id_rsa",
                "user": "tabedzki"
            }
        }
    ]
}

then it starts debugging normally without any errors with respect to the authentication method.
Can you see if there is a way to modify the extension to accept these OS specific arguments? Thanks in advance!
~tabedzki

Hey @WebFreak001,
I am closing this because this is probably the way that it should be configured (since it works this way):

        {
            "type": "gdb",
            "request": "launch",
            "name": "Linux-Launch Program (SSH)",
            "target": "./build/debug/file1",
            "cwd": "${workspaceFolder}/",
            "linux": {
                "ssh": {
                    "host": "rrlogin.seas.upenn.edu",
                    "cwd": "/mnt/rrio1/home/tabedzki/vscode-cpp-remote-debug",
                    "keyfile": "${env:HOME}/.ssh/id_rsa",
                    "user": "tabedzki"
                }
            },
            "windows": {
                "ssh": {
                    "host": "rrlogin.seas.upenn.edu",
                    "cwd": "/mnt/rrio1/home/tabedzki/vscode-cpp-remote-debug",
                    "keyfile": "${env:USERPROFILE}/.ssh/id_rsa",
                    "user": "tabedzki"
                }
            },
        },

huh if that works ok


Offline Bubble Chart Using Cufflinks Shows All Bubbles in One Color

Hello,
According to this example:
https://plot.ly/ipython-notebooks/cufflinks/
I should have bubbles with different colors.  I have created an issue at the plotly.py github repo, but I think this is a cufflinks issue.

Hi, you need categories for this.
cf.datagen.bubble().iplot(kind='bubble', x='x', y='y', size='size', text='text',categories='categories')


improve commands output

adding |head -1  because without it the output is confusing:
ldemailly-macbookpro:istio-0.1.5 ldemailly$ kubectl get configmap istio -o yaml | grep authPolicy
    authPolicy: MUTUAL_TLS
      {"apiVersion":"v1","data":{"mesh":"# Uncomment the following line to enable mutual TLS between proxies\nauthPolicy: MUTUAL_TLS\nmixerAddress: istio-mixer:9091\ndiscoveryAddress: istio-manager:8080\ningressService: istio-ingress\nzipkinAddress: zipkin:9411"},"kind":"ConfigMap","metadata":{"annotations":{},"name":"istio","namespace":"default"}}

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed


3.x)Additive blending fix

I'm writing up a detail pull request and accidentally hit submit I think? SMH. How frustrating.

That fact that it is more like a feature proposal made me want to make a really good case for this PR so I am very embarrassed having this be accidentally submitted before it's ready. ;_;

Thanks for your help!

Sorry my mistake. I don't think I can reopen a PR.

I'm writing from my mobile rn. Maybe I can reopen it from my desktop.

This PR has 1500 commits so there's not much point reopening it as is, it would be better to redo it with only the relevant commit.

This PR has 1500 commits so there's not much point reopening it as is, it would be better to redo it with only the relevant commit.

Ill make another, properly.  Is that a good method of catching up my branch with the most recent 3.x changes or is there a better way?

There's already a PR that makes animation blending additive in 3.x: #53893

Cool! Should I make a PR for my solution or just discuss my solution in that thread?

Cool! Should I make a PR for my solution or just discuss my solution in that thread?

See that comment first: https://github.com/godotengine/godot/pull/53893#issuecomment-1172947478
If your implementaion is indeed different, then leave a comment in https://github.com/godotengine/godot/pull/53893.


Support for hcp

Include support for installing hcp packages. Eg : Metron
Ref : https://docs.hortonworks.com/HDPDocuments/HCP1/HCP-1.4.0/bk_installation/bk_installation.pdf

Hi there, for metron/hcp deployment with ansible please follow https://github.com/apache/metron/tree/master/metron-deployment/ansible


ED-1227: Add oci oss oss support for kp/flinkjobs -- superseed #695 , #713

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Type of change
Please choose appropriate options.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes in the below checkboxes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Ran Test A
[ ] Ran Test B

Test Configuration:

Software versions:
Hardware versions:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@pallakartheekreddy this is the PR for kp jobs

@pallakartheekreddy i tried the suggestion, but the circle ci failed at Installation of imagemagick


Loading indicator

Hello, I have a listener for when the range changes, during which I fetch some events for timeline items.  However, when I drag left or right to change the range, it does not show the loading indicator.  Any ideas?
Thanks

When you drag the timeline left or right are you getting the range change event on the server side? Can you reproduce the issue with the demo?

Yes I am getting the event, and yes on the demo it's not showing any indication it's working either...

So if I understand correctly, the issue is that you're not seeing the Vaadin loading indicator when your performing a long operation as a result of the range change event?
I tested this with the demo and I see the loading indicator if I slow down the work done in response to the range change event. For example, if I add:
       try {
          Thread.sleep(5000);
        }
        catch (InterruptedException ex) {
          // ignore
        }

to the RangedChangeListener#rangeChanged call I get the loading indicator for a few seconds. Is it possible that your operation is just completely so fast that you don't see the indicator?

No my operation takes some time.  In your demo, change the amount of data items to 1,000,000 and zoom out until you see about 8 days.  Then drag to the left...

I think the issue you're seeing is that the browser is spending a ton of time in JavaScript calls marshaling and unmarshmaling the JSON required for that many items. In that case you won't get a progress indicator because JavaScript is single threaded so if the thread is busy handling the UIDL request to/from the server it can't display or update the progress indicator. There isn't much that can be done about that other than using fewer items so the browser can render it efficiently. I don't know how optimized the vis.js Timeline is but I'm guessing a few hundred items would be about all you want on the screen at one time or it is going to spend a lot of time layout out items and drawing plus the overhead of marshaling to/from the server.
You can debug this by looking at the developer tools network console in your browser and seeing how much time is spent waiting for the server reply vs how much is spent in JavaScript processing the JSON and redrawing the timeline.

Sorry for the delay in my response.  The majority of the time is spent on the server side...

I haven't been able to reproduce the issue if it is all server side. If I put a sleep in the range change event I get the Vaadin spinner client side. If I send thousands of items to the client I don't get the spinner but that is most likely because the client is busy processing the returned UIDL and updating the items in the timeline. If you're sending a ton of items you might want to scale it back to avoid flooding the client side with JSON to be processed.
I can look at it more if you can reproduce the issue in the demo with limited items but a long server side operation.


Make sure the build repo is pulled in docker-build stage

Signed-off-by: Peter Zhu<EMAIL_ADDRESS>Description
Make sure the build repo is pulled in docker-build stage.
Directly use agent node will have consequences where git repo is not pulled.
Running on docker container will force the pull every time.
Issues Resolved
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2060 (02c1262) into main (ec6c37c) will increase coverage by 5.89%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #2060      +/-   ##
=============================================
+ Coverage     94.10%   100.00%   +5.89%
=============================================
  Files           198         6     -192
  Lines          3902       105    -3797
  Branches         29        19      -10
=============================================
- Hits           3672       105    -3567
+ Misses          224         0     -224
+ Partials          6         0       -6

Impacted Files
Coverage Δ

src/build_workflow/builders.py

src/manifests_workflow/component_opensearch_min.py

src/test_workflow/perf_test/perf_test_cluster.py

.../assemble_workflow/bundle_opensearch_dashboards.py

...sts/jenkins/jobs/ArchiveAssembleUpload_Jenkinsfile

.../ci_workflow/ci_check_gradle_properties_version.py

src/manifests/component_manifest.py

tests/jenkins/jobs/Messages_Jenkinsfile

src/build_workflow/build_args.py

src/test_workflow/test_args.py

... and 182 more

Continue to review full report at Codecov.

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

See OS build 2626 vs 2629 for docker build trigger comparison.


feat(tutorial): complete step 5 - Cambridge

https://carbon-tutorial-hilljona.mybluemix.net/#/

:+1 Looks good. Congratulations on completing the tutorials!


upgraded Gradle version and Jenkins version and plugins

please merge this PR
AFTER #736
otherwise it will create all FDB again without being disabled


Traffic Stats shouldn't panic when it cannot talk to Traffic OPs

If Traffic Stats cannot talk to Traffic Ops, like to get servers, it panics and dies.  Traffic Stats should be able to handle this situation without panicking.
Last login: Thu Jul 26 14:56:45 2018 from <IP_ADDRESS>
panic: Error getting server list from https://trafficops.cdn.net: Get https://trafficops.cdn.net/api/1.2/servers.json: net/http: request canceled
goroutine 9 [running]:
panic(0x7b3bc0, 0xc820193060)
/usr/lib/golang/src/runtime/panic.go:481 +0x3e6
main.getToData(0xc82000f9e8, 0x6, 0xc82000fa00, 0xe, 0xc8200c0ae0, 0x1e, 0xc82000fa10, 0xa, 0x0, 0x0, ...)
/tmp/trafficcontrol/rpmbuild/BUILD/traffic_stats-2.2.0/src/github.com/apache/incubator-trafficcontrol/traffic_stats/traffic_stats.go:447 +0x63b
created by main.main
/tmp/trafficcontrol/rpmbuild/BUILD/traffic_stats-2.2.0/src/github.com/apache/incubator-trafficcontrol/traffic_stats/traffic_stats.go:129 +0x280
[FAILED]

@dneuman64 looking at the code for this, it seems like it's supposed to panic only when it's initializing. Once it's initialized, it looks like it just logs an error and continues working. Is that the desired behavior? Or do we want TS to not panic on startup/initialization if it can't reach TO? Perhaps it should retry on startup instead?


Add block decoder

Relates to #11
Overview

Copy and slightly modify the CBOR block decoder from the prototype.
Add some tests using mainnet blocks.
This branch takes the Cardano.Wallet.Primitive module from #25, so that should be merged first, then this branch rebased.

Todo

[x] Remove TODO line from decoders.
[x] Add test for a testnet block.
[ ] ~Add test for Yoroi style addresses~ (actually is ok, because addresses aren't decoded).
[x] Wait for #25 to be merged.

Rebased on top of latest master :+1:


Clarify open source license for YOLOv8 patch training modifications

Could I ask the author if you can open source the code for the YOLOv8 adverse patch

Hi @dkyman,
It wouldn't hurt to check with them, however the author(s) of the upstream for this project released their code under the MIT license (see LICENSE file). This is a permissive license which allows us to modify and redistribute with minimal restriction, as long as our version contains the same license. That also means that all code under this project is open source under the same permissive terms.
We're still fixing some issues with this repo for patch training with YOLOv8, it may be worth checking back soon though

We probably should clarify the terms of the components we added to this project repo.
The Airbus dataset is under CC-SA 4, which is generally compatible with MIT.
The yolov8 model we trained (best.pt) using transfer learning from the pretrained COCO weights is most likely AGPL-3.0, since that weights file we trained from was in a repo marked as AGPL-3.0. Only the file best.pt would be under the restriction imposed by this permissive copyleft license.

Hi @dkyman,
It wouldn't hurt to check with them, however the author(s) of the upstream for this project released their code under the MIT license (see LICENSE file). This is a permissive license which allows us to modify and redistribute with minimal restriction, as long as our version contains the same license. That also means that all code under this project is open source under the same permissive terms.
We're still fixing some issues with this repo for patch training with YOLOv8 (#2), please check back with us soon though!

Dear author, can I understand that you will soon open source the engineering code for YOLOv8 patches?

This is the code, it's currently a work in progress


Appveyor and NUnit console issue

In the Appveyor output we get this text
NUnit-Console version <IP_ADDRESS>50
Copyright (C) 2002-2012 Charlie Poole.
Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A. Vorontsov.
Copyright (C) 2000-2002 Philip Craig.
All Rights Reserved.

Runtime Environment -
   OS Version: Microsoft Windows NT 6.2.9200.0
  CLR Version: 2.0.50727.8009 ( Net 3.5 )

ProcessModel: Default    DomainUsage: Multiple
Execution Runtime: net-3.5

This is of course wrong, we are on .Net version 4.5.2.
We are experiencing this issue. This affects some async tests - we have had spurious fails on appveyor of tests that pass locally, and we likely have spurious passes (i.e. if they started failing locally, appveyor would not notice).
Using NUnit 3.0 may be the answer according so some discussions.  We don't use NUnit 3.0 yet.
JustSaying my be a good trial for NUnit 3.0?

Just a note NUnit 3.0. - There's a slight syntax change to Assert.Throws in that it does not support async code, instead they have added Assert.ThrowsAsync. I am ok with upgrade to NUnit 3.0, should be no major issues.

Yep, I believe that the Assert.AreEqual syntax is also deprecated in favour of Assert.That. We can update to deal with these minor issues.

We are on NUnit 3.0 now.