[WIP] proof for issue#1506

@ see https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/1506

any chances for a fix?

any chances for a fix?

not really :(

no need to close it then ;)

Closing due to #1559
Thanks @SpacePossum !

thanks @keradus :)


Missing input validation on firecracker console settings

There are  a couple issues with console handling in firectl.

There's no sanity checking at all on the value passed to --firecracker-console. E.g.

ulap:firecracker-go-sdk‹master›$ ./cmd/firectl/firectl --firecracker-binary=/usr/local/bin/firecracker --kernel=~/bin/vmlinux --root-drive=~/src/firecracker/images/image-debootstrap.img --kernel-opts="console=ttyS0 noapic reboot=k panic=1 pci=off nomodules rw init=/sbin/init" --vsock-device=root:3 --firecracker-console=xyz
INFO[0000] Called startVMM(), setting up a VMM on ./firecracker.sock
INFO[0000] VMM logging and metrics disabled.
INFO[0000] refreshMachineConfig: [GET /machine-config][200] getMachineConfigOK  &{CPUTemplate:Uninitialized HtEnabled:true MemSizeMib:512 VcpuCount:1}
INFO[0000] PutGuestBootSource: [PUT /boot-source][204] putGuestBootSourceNoContent
INFO[0000] Attaching drive /home/ANT.AMAZON.COM/nmeyerha/src/firecracker/images/image-debootstrap.img, mode rw, slot 1, root true.
INFO[0000] Attached drive /home/ANT.AMAZON.COM/nmeyerha/src/firecracker/images/image-debootstrap.img: [PUT /drives/{drive_id}][204] putGuestDriveByIdNoContent
INFO[0000] startInstance successful: [PUT /actions][204] createSyncActionNoContent
^CINFO[0001] Caught signal interrupt
WARN[0001] firecracker exited: signal: interrupt
FATA[0001] startVMM returned error signal: interrupt

I would expect firectl to report an error in that case, and never actually try to spawn a VM. This issue has likely existed forever.

If --firecracker-console=file is specified, but no filename is provided, the behavior is confusing:

ulap:firecracker-go-sdk‹master›$ ./cmd/firectl/firectl --firecracker-binary=/usr/local/bin/firecracker --kernel=~/bin/vmlinux --root-drive=~/src/firecracker/images/image-debootstrap.img --kernel-opts="console=ttyS0 noapic reboot=k panic=1 pci=off nomodules rw init=/sbin/init" --vsock-device=root:3 --firecracker-console=file
FATA[0000] Failed to build command: open : no such file or directory

I would expect firectl to complain clearly about a missing argument. This is new as of #17

Firectl

No longer lives in this repository.
No longer handles output.


Card component

Zeplin: https://zpl.io/agrvXAD https://zpl.io/amynYpe

Essentially card is style component with slots for its different parts.
You MAY use existing componenent as a base. You'll need to dispose carefully of bootstrap styles and perform style checks if you decide so.

References

vuetify - https://vuetifyjs.com/en/components/cards

Confirm design as required.

Task is not remotely close to be finished at the moment.


Missing support for adaptiveCard/action type invoke in TeamsActivityHandler

Versions
SDK Version: 4.15.0
NodeJS Version: 12.18.3
Browser Version: Chrome
OS: Windows 11
Describe the bug
While writing a JS Bot for Teams, we would be over-riding the handleAdaptiveCardAction method inside the botbuilder/src/teamsActivityHandler.ts file for adaptiveCard/action invokes. However this type of invoke is not being returned in the onInvokeActivity method of the same file and this is affecting the ability to handle such invokes in the latest bots being developed.
Expected behavior
Would need the adaptiveCard/action type of invoke to return an invoke response using the above method.
Screenshots
The following are the requested changes to help override the existing methods:

@stevkan @ramfattah can you pls help looking into this as it'll unblock our customers building JS bots

Hello @ashwinvc, we are investigating.

Hi @ashwinvc
looks like the adaptiveCard/action type of invoke is already checked in the base class /activityHandler /, in method onInvokeActivity.
https://github.com/microsoft/botbuilder-js/blob/e7d845eeacb32824af93652d07e46aabf30e800e/libraries/botbuilder-core/src/activityHandler.ts#L508
Perhaps the customer could use ActivityHandler to return an invoke response; example using ActivityHandler.onAdaptiveCardInvoke ?

Update: In .NET SDK, the handleAdaptiveCardAction method is not available in TeamsActivityHandler class. Only in JS SDK.

Sure thing. Will work with the ActivityHandler methods. But wouldnt it be easier to add the changes mentioned in the description, since it enables for ease to work with! @ramfattah


When can Islandwood apps be submitted to the Win App store?

hi, is it possible to submit apps to the win app store now if they are buit w/ the current version of the "project islandwood" WinObjC porting tool?  If not, what is the estimated calendar time we can submit?

Yes, if your app builds and runs using the bridge, you should be able to submit it to the store.  In fact, there are apps in store that are built using winobjc.  Please note that the bridge is work in progress, so you may not have all the APIs that you need.

that is great, thanks.  is there a way you could tell us some example apps that are in the store built w/ winObjC so we can examine the results?

here is a blog post on an app that used the bridge.

Hi again, is there a list of iOS classes that are supported at a high level?
In particular I am looking for CoreGraphics (CGContext and related "Quartz 2D" drawing functions).
I am not seeing it here:  https://github.com/Microsoft/WinObjC/wiki/Roadmap
  From: Raj Seshasankaran<EMAIL_ADDRESS>
To: Microsoft/WinObjC<EMAIL_ADDRESS>Cc: brian k<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Sent: Tuesday, July 5, 2016 4:19 PM
Subject: Re: [Microsoft/WinObjC] When can Islandwood apps be submitted to the Win App store? (#607)
here is a blog post on an app that used the bridge.—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Here is another my app used Windows Bridge for iOS:
https://www.microsoft.com/en-us/store/apps/typography-insight/9nblggh4qrs5
http://typeinsight.org

@cre8ivepark BTW, do you have any plans for bringing this app to Windows Mobile?

Thanks, this is pretty interesting, looking fwd to checking out this app.
  From: cre8ivepark<EMAIL_ADDRESS>
To: Microsoft/WinObjC<EMAIL_ADDRESS>Cc: brian k<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Sent: Wednesday, July 6, 2016 3:36 PM
Subject: Re: [Microsoft/WinObjC] When can Islandwood apps be submitted to the Win App store? (#607)
Here is another my app used Windows Bridge for iOS:
https://www.microsoft.com/en-us/store/apps/typography-insight/9nblggh4qrs5
http://typeinsight.org—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Closing this issue since the question was answered.


Improve __repr__

__repr__ could include the file name, samplerate and number of channels, and possibly the open mode.

OK, looks good to me.
I don't see why we need format should require subtype and endian, but if you insist on it, we'll do it that way. I'd rather have always too much information than a different amount of information for different files.

I inadvertently added this to the merge commit message, I actually wanted to post this comment here:
OK, I'm merging this.

I don't see why we need format should require subtype and endian, but if you insist on it, we'll do it that way. I'd rather have always too much information than a different amount of information for different files.

If we get annoyed by the large amount of information, we can still remove the format/subtype/endian triple at a later point.
Also, let's see if the implementation of #58 sheds a different light on this.


[Feature][E2E] Setup E2E under JDK 11

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
No response
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I am interested in this issue and create a PR for fixing it.
feel free to comments if any concern.


Update README.md

Github -> GitHub

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Fixed conflicts with crafts styled select field

This fixes issues where when resizing the craft control panel the select2 field gets 'funky'
https://www.dropbox.com/s/gfhz93821256thx/Screenshot 2016-09-27 22.24.56.png?dl=0

Thanks for this :) Kept meaning to fix that issue.


Do not re-fetch transactions that are already confirmed in getAddressHistory func

Will be resolved once the migration branch here is merged. Beignet will continue checking until 6 confirmations.


strange hour ahead persistence at Bend OR PV

Hard for me to explain this kind of behavior. Perhaps bad system metadata.

Forecast
Report (UA org only): https://dashboard.solarforecastarbiter.org/reports/d900481a-c5eb-11ea-96c5-0a580a820020

I checked if this is still an issue since I thought we had addressed some inconsistencies in SRML problems. Still an issue, though less of one due to season changes. Guess I misremembered.


e2e tests : enable keycloak and quay test cases

Description
Enable keycloak and quay test cases and add/update the dynamic config required for plugin test cases.
Update the pipeline to run plugin test cases (keycloak and quay)
Please explain the changes you made here.
Which issue(s) does this PR fix
https://issues.redhat.com/browse/RHIDP-859

Fixes #?

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

@davidfestal can you review this pr? thanks!


Any plans to support Expo?

Hi,
Great library, I would love to see this supported in Expo. Do you have plans to do it?

I'm curious about this as well.

Hey @frossi85 ,I'm not able to run this app in Android
I'm getting "RNPdfScanner was not found in the UIManager".
Any solution for this. Please let me know.
Using React<EMAIL_ADDRESS>


Fix #726, handle different numpy blas engines

lgtm, I assume it works with your mkl setup?

ok, the approach is not good, see https://travis-ci.org/serge-sans-paille/pythran/jobs/282234192
any thought?

looks like the mkl config is filled even if mkl is not present? there's a field in pythran config to force the blas egine, we should base on that

Closing this as it's obsoleted by #729


Dask doesn't play well with interrupts

This is on Python 3.5.2, dask 0.10.1, and jupyter-notebook 4.2.1, all installed via conda on a 64-bit Ubuntu machine.
 See https://asciinema.org/a/2yqxbhn1patwenwy024m2316l for a video.
 Sometimes, dask will mysteriously keep printing out the progress bar from the last session, even after restarting the kernel (!).
EDIT: When this happens, dask will also leave around python processes still running. See https://asciinema.org/a/8x1liiyuiw7910mugj9tscher for an example (particularly the extra python process at the end).

Notably, I can only reproduce this in the python repl (and ipython). When I run it as a script (python test.py), 3 KeyboardInterrupt are enough to exit cleanly without leaving traces of itself anywhere (see https://asciinema.org/a/eitz5acubx6ot7a9cmm1sdz2l)

...contrary to my previous assumption, this doesn't have anything to do with the progressbar. I just thought it did because the phantom progressbar on a jupyter notebook was how I found out about it.
Everything above can be replicated without the with ProgressBar(): line.
Sorry for the noise.

I noticed that behavior when using dask in a jupyter notebook: the progressbar running in background after a stop and also orphan processes even after killing all kernels.
FWIW, I used ProgressBar().register() at the beginning of the notebook.

I've so far been unable to reproduce this without a progressbar.
I have been able to reproduce this with the synchronous single-threaded scheduler

Possibly resolved in #1444

Confirmed that it was resolved. Thanks!

Thanks for confirming @alanhdu and for raising in the first place.

Something like this still occurs for me with the distributed scheduler running on a LocalCluster. When I work in a jupyter notebook, and the notebook is executing a blocking .compute() method, and then hitting CTRL+C - the blocking method indeed returns, but all the worker processes are killed (not very gracefully). The end result is that there are zombie python processes and the scheduler reports 0 workers.
So far I haven't found a good way to recover from this without restarting the jupyter kernel, because issuing Client.restart() doesn't restart the workers, and trying to execute Client(LocalCluster()) again gives port 8787 is already in use.
If anyone has a good tip on how to workaround that could really help.


Fix chttpd auth cache log errors

Overview
In situations where the chttpd authenticaion db (usually named "_users")
does not get created, or gets deleted, the auth cache still attempts
to start the change listener, which results in notice and error level
logs in the system like:
[notice] 2018-12-03T22:04:44.299392Z<EMAIL_ADDRESS><0.374.0> -------- chttpd_auth_cache changes listener died database_does_not_exist at mem3_shards:load_shards_from_db/6(line:395) <= mem3_shards:load_shards_from_disk/1(line:370) <= mem3_shards:load_shards_from_disk/2(line:399) <= mem3_shards:for_docid/3(line:86) <= fabric_doc_open:go/3(line:39) <= chttpd_auth_cache:ensure_auth_ddoc_exists/2(line:195) <= chttpd_auth_cache:listen_for_changes/1(line:142)

[error] 2018-12-03T22:04:44.299585Z<EMAIL_ADDRESS>emulator -------- Error in process <0.3180.0> on node<EMAIL_ADDRESS>with exit value:
{database_does_not_exist,[{mem3_shards,load_shards_from_db,"_users",[{file,"src/mem3_shards.erl"},{line,395}]},{mem3_shards,load_shards_from_disk,1,[{file,"src/mem3_shards.erl"},{line,370}]},{mem3_shards,load_shards_from_disk,2,[{file,"src/mem3_shards.erl"},{line,399}]},{mem3_shards,for_docid,3,[{file,"src/mem3_shards.erl"},{line,86}]},{fabric_doc_open,go,3,[{file,"src/fabric_doc_open.erl"},{line,39}]},{chttpd_auth_cache,ensure_auth_ddoc_exists,2,[{file,"src/chttpd_auth_cache.erl"},{line,195}]},{chttpd_auth_cache,listen_for_changes,1,[{file,"src/chttpd_auth_cache.erl"},{line,142}]}]}

This changes the auth cache so that it
• Only starts change listener if auth db exists
• Only retains EndSeq if auth db still exists
Fixes https://github.com/apache/couchdb/issues/1354
Fixes https://github.com/apache/couchdb/issues/1949

Testing recommendations
make eunit apps=chttpd suites=chttpd_auth_cache_test

Related Issues or Pull Requests

Checklist

[x] Code is written and works correctly;
[x] Changes are covered by tests;
[ ] Documentation reflects the changes;

@jaydoane One unintended side effect of this is that there is no logged warning that the system databases are missing anymore. We should log an error once at least.

+1 for at least one log

@jaydoane  any plans to get this in? I am preparing to mass-close old PRs that never got merged.

@jaydoane any plans to get this in? I am preparing to mass-close old PRs that never got merged.

@wohali oh no, I completely forgot about this one 🙀
It seems like the main issues left are including more test cases, and adding some logging for missing _users db? If there's general agreement that this is a desirable PR, I'm happy to make those changes and get it finished.

Yes please.

I took another look at this, but noticed this commit from Jan already addressed the issue of stack traces in the logs, and the repeating error does actually seem like a good -- albeit somewhat spammy -- feature. So I think we should just close this in favor of his more elegant fix.

I took another look at this, but noticed this commit from Jan already addressed the issue of stack traces in the logs, and the repeating error does actually seem like a good -- albeit somewhat spammy -- feature. So I think we should just close this in favor of his more elegant fix.


Can not execute deployment in migration

Issue
Can not execute this migration script with CoralX v0.1.0.
const Greeting = artifacts.require("Greeting");

module.exports = async function (deployer) {
  await deployer.deploy(Greeting);
};

Steps to Reproduce

coralX compile
coralX execute --path migrations

Expected Behavior
Migration succeeds.
Actual Results
% coralX compile

Compiling your contracts...
===========================
> Compiling ./contracts/Greeting.sollc-bin. Attempt #1
> Compilation warnings encountered:

    Warning: Visibility for constructor is ignored. If you want the contract to be non-deployable, making it "abstract" is sufficient.
 --> project:/contracts/Greeting.sol:5:3:
  |
5 |   constructor() public {
  |   ^ (Relevant source part starts here and spans across multiple lines).

> Artifacts written to ~/my-smart-contract/build/contracts
> Compiled successfully using:
   - solc: 0.8.17+commit.8df45f5f.Emscripten.clang
% coralX execute --path migrations
1_deploy_greeting.js

~/.nvm/versions/node/v18.9.0/lib/node_modules/coral-x-fe/dist/coralX.build.js:685
        const ABI = artifacts.getTmpABI();
                              ^
TypeError: artifacts.getTmpABI is not a function
    at Object.deployContract (~/.nvm/versions/node/v18.9.0/lib/node_modules/coral-x-fe/dist/coralX.build.js:685:31)
    at Array.<anonymous> (~/.nvm/versions/node/v18.9.0/lib/node_modules/coral-x-fe/dist/coralX.build.js:662:65)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at _Queue.processQueue (~/.nvm/versions/node/v18.9.0/lib/node_modules/coral-x-fe/dist/coralX.build.js:1115:28)
    at ~/.nvm/versions/node/v18.9.0/lib/node_modules/coral-x-fe/dist/coralX.build.js:678:13

Environment

Operating System: macOS
Ethereum client: N/A
CoralX version (coralX version): 0.1.0
node version (node --version): 18.9.0
npm version (npm --version): 8.19.1

Adding a second argument { gas: 250000 } resolved the issue, but this error message is too obscure.


feat: introduce addon upgrading condition

Signed-off-by: ashish<EMAIL_ADDRESS>This PR introduces upgrading condition to the operator. The upgrading condition is set to true when the controller observes that the spec.version is not equal to status.ObservedVersion. The condition then transitions to false when the controller sees a new CSV which is different from the last CSV it saw (from status.LastObservedAvailableCSV).

/lgtm


Remove superfluous dataclasses dependency

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Removing the dataclasses dependency as that only ever applied to Python <3.7 and Python 3.8 should be the minimum supported Python now. I believe Python 3.8 is the minimum version conda-forge would want to support too.

@conda-forge-admin, please rerender

I was beginning to doubt myself regarding the Python 3.8 minimum but I checked the generated packages and yes, no conda packages for end of life versions of Python (3.8 at present time) are generated so I'm confident the dataclasses dependency is unnecessary.

Sorry I somehow missed this PR. It's addressed in the latest 2.9.0 release (https://github.com/conda-forge/mpire-feedstock/pull/24). Previously, 2.8.1 was still listing it in its setup.py, so I think at that point it was premature to remove it.


feat: add ci and impacted service to alert burn rules

Purpose of pull request
closed #615

Superseded.


Document package building

Thanks for your review @Vincinator. Not merging today on purpose to allow others to look at it, will merge after some time.


README and Standard Demo Fixes

Fixes to demo script, hw contract, and operator code.

@stevennevins fyi


Exposing rails path helpers in React components

Context
What I'm working on won't be a SPA (at least not in the initial version). I want Rails to handle routing and rendering React components (with turbolinks). What I'm envisioning is a shared store approach and one route will have a couple of block components server side rendered which all get the store and that's where all app state is injected. I wanted this so to not mix app state through component props + store state, and keep data in a single place (this might not be the best way).
The problem
I'm not sure how to expose URLs from the rails app to the client. Of course the idea would be to have "prop serialisers" or "store state serialisers" computed in Rails that use the path helpers to expose all the component needs, but I imagine that there are subtle situations where this can get out of hand.
I mean, when you think about an isolated component props makes sense, but if the component that actually needs the URL is somewhere down the chain, you will want to but that in the store somehow so that you avoid the "big ball of props" from your top-level component. And if you want to put these URLs in the store you need to decide where they go. Dynamic URLs are resource related, that means that if you want to have a "resource" entry somewhere in the state tree you most probably have a resource serialiser that will include the URL, but then you're thinking, hmmm what if a page only cares about the URL for a small component, should I serialise the whole resource in the state tree to keep it consistent, or start hacking things around.
This was a bit of a brain dump sort of description, so I hope it isn't too hard to follow but what I'm after is basically is to see how other people are handling this.

I suggest using this features to make the relevant paths available in all the JS code:
https://github.com/shakacode/react_on_rails/blob/master/README.md#customization-of-the-rails_context


Admin flair

This add some of the fixes suggested in #89.
A admins command list all admins.
When listing users in room, admins have a star before their names.

What do you think of the changes I made?

Looks like this now:

Thanks for the PR I love this


Remove PlaceholderInExtend linter from SCSS

This is a nice linter. It suggests the use of a placeholder selector rather
than a class selector, as it would generate less code [1]. However, since we use
Twitter Bootstrap as a baseline and Twitter Bootstrap does not support
placeholder selectors (due to issues with media queries [2]), this linter raises
errors that we cannot fix. This relates to admin apps (e.g. content-tagger.
By removing this linter we suppress errors we couldn't otherwise fix.
[1] https://github.com/brigade/scss-lint/blob/master/lib/scss_lint/linter/README.md#placeholderinextend
[2] https://github.com/twbs/bootstrap-sass/issues/721
Edit: I could disable the linter in place where needed, but I was wondering if it could be disabled globally here (assuming it doesn't impact anything else).

Extending classes is generally bad practice anyway. What’s your usecase here? Can you link to some code where you’re using it?

@robinwhittleton here is an example: https://github.com/alphagov/content-tagger/blob/master/app/assets/stylesheets/application.scss#L48-L52 I guess what this is avoiding is adding those classes to the tables. Would that be preferred in this case?

If it’s possible to do that then yes, generally it’d be preferred. Extends can lead to very unwieldy rendered CSS (which is why that rule is there in the first place). We generally see them as a last ditch attempt to get things working.

I will move those classes into the markup instead and close this PR. Thanks @robinwhittleton


[issue-2176] Added escape/unescape functions

[x] New feature: functions for escaping/unescaping values wrapped in {{}}
[ ] BC breaks
[ ] Deprecations
[x] Tests added
[x] Changelog updated

This commit adds two new functions:

escape(string): string
unescape(string): string

Once the value has been processed by the escape function, then the values wrapped in {{}} like {{foo}} will not be parsed by the parse function.
The escape function works by replacing the {{ with \\{\\{ and }} with \\}\\}.
The unescape function reverses the process.
The parse function has been updated to run the unescape afterparsing has been completed, to ensure that the commands are being executed normally.
Fixes #2176

So unescape will be run on each command? Hmmm. What are consecutions? Is there a way it will break something?
Maybe we should think a little about placeholders?
Now there is one special placeholder secret (which will not be printed to stdout nor logs):
run("fjoy -p '%secret%' -fk", ['secret' => getenv('CI_SECRET')]);

What about adding something similar?
run('mysql --silent --raw --skip-column-names --execute %query%', ['vars' => [
    'query' => $query,
]]);

Also, I'm concerned with the name escape. It may confuse people what this is for escaping CLI arguments (deals with quotas, but it does not).

This might be even more complicated than I've previously thought.
The {{foo}} is being replaced everytime the parse method is being called, meaning it'll not only be executed as a part of run but also as a part of get.
Maybe the best approach would be simply to do nothing in \Deployer\Configuration\Configuration::parseCallback when the "match" is not defined in configuration values?

So what do you think about placeholders?

Placeholder will work if the value is being set explicitly, like:
$query = 'select * from {{foo}}';
run('mysql --silent --raw --skip-column-names --execute %query%', ['vars' => [
    'query' => $query,
]]);

if however the code is like:
set('foo', 'select * from {{foo}}');
run('mysql --silent --raw --skip-column-names --execute %query%', ['vars' => [
    'query' => get('foo'),
]]);

then the code will try to parse that {{foo}}.
If we assume that all we want to handle here is run then I guess I could proceed with the different MR ;)

set('foo', 'select * from {{foo}}');

What is the point if this?) From my understanding case is next: a user has some $variable with {{}} in it. What to pass it to run. Placeholder will work in this case.

There is no point ;)
The problem is that we do not know where the value that will be used with run and placeholders will be coming from. If it comes from get then it'll still be parsed. But I guess that's a problem that has not yet happened ;) It can be dealt with once (if ever) it appears ;)
I'll get to the new MR then ;)


Implement RFC 6824

[✓] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[✓] I have searched the existing issues and I am convinced that mine is new.

Is your feature request related to a problem? Please describe.
No my feature request is not directly related to a problem
A clear and concise description of what the problem is including your motivation for the request,
i.e. "For the purpose of [...] I am missing a solution that will [...]."
Motivation for my request is having access to multiple WAN uplinks as well as a VPS with enough bandwidth to service both links fully with low and stable latency. I would like to be able to use a wireguard vpn setup to bond these uplinks using MPTCP so that a single tcp connection/flow can utilize both links fully. i have been able to verify using a linux machine that this can work but ideally i would be able to do it right in the router so my entire lan can transparently have that boost.
A clear and concise description of what you want to happen.
have RFC 6824 support added to the kernel so it can be used on the system.
Describe alternatives you considered
I have considered just setting up a linux router, and indeed have one currently for testing but ideally i would be able to keep using opnsense.

Please check with FreeBSD where this has been added?


Issues deploying the helm chart for spark-on-k8s-operator using argocd

I'm attempting to deploy the latest version of the spark-on-k8s-operator using argocd 1.7.6, and I've encountered a few problems.
Helm lifecycle hooks
The webhook-cleanup-job annotations specify they should run on pre-delete and pre-upgrade, this is interfering with the way argocd deploys helm charts and causing the spark-on-k8s-operator to never deploy, as the cleanup job first attempts to mount the service account before it has been created. I've found if I configure the values files locally to remove the pre-upgrade, the helm chart deploys and is useable, but whether this is a suitable long term solution I'm not sure. The change to this section of the helm chart just opens that section up to be configured without changing the default helm hooks that was there originally.
It looks like several people have had this issue before judging by the end comments in this issue, and this is the only helm chart with that issue exposed: https://github.com/argoproj/argo-cd/issues/355
Spark CRDs malformed
The creation timestamp would be better left out, as it that line is created automatically when the CRD is deployed, and the conditions and storedVersions are better set to the empty list instead of null, as they currently fail validation.
I've managed to get the helm chart deployed using some local changes, and am happy to provide them if you wanted to merge the changes in.

My PR was merged, so can close this

My PR was merged, so can close this


Integrity Constraint Violations Fix - Introduction of Duplicate Entries in DB

We were facing integrity constraint violation errors when trying to delete customer account, with order creating data still intact inside the account.

Basic Idea for Commerce

The Order is independent including addresses, line items, payment, and more
An order is only created when the customer places it, not before, the checkout process does not create an "empty" order when it starts
The cart is basically an order that has not been placed and it contains the same data and things but it is more actively modified and typically contains addresses, line items and payments as reference and not as copy
When creating and order, the cart is "converted" into an order which has states such as new, failed, created. The states are part of the final processing, because the payment is done after an order exists and might fail. Or we fail due to inventory not being available (inventory does not yet exist in Posters).
Anonymous Carts expire when the session is expired (we keep it simple for now)
Carts are attached to registered on user when the log on or when the cart is created for a session with a logged on user (which replaces the previous cart they might had)

Important: Cart references to addresses and products are not DB references! This would makes maintenance impossible. This are references by text aka ID, so you say, product A with (internal) ID 69ffd249-4750-457d-b904-c205e803f27b is part of the cart but this is not a foreign key relation but the ID is a a "text" and when you display or calculate the cart, you have to check all of this weak references and if needed, drop it as well as read the updated data.


feat(wallet): improve fee estimation

Description:
Resolves #1972

Types of changes:
Enhancement

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

@mrfelton updated

Sorry, unit tests

Coverage decreased (-0.03%) to 20.326% when pulling fb053258cde31c7c992226c4c6e4839f11d2e8d0 on korhaliv:feat/improve-fee-estimate into de5a0d42e53878ecec8fd0c422fa37ff5f5356fb on LN-Zap:next.

@mrfelton done


Fix documentation for Command.option

Examples were using param instead of option.

Thank you 🙂 Super nice to see you here 🙂


Adding new appendPartWithFormData method with Content-Type parameter in AFMultipartFormData

Adding new appendPartWithFormData method with Content-Type parameter in AFMultipartFormData

Could you sync this back up with master to get the fixes for the expired certificate? That should allow the test suite to pass.

Could also please add test cases? Thanks!

Thanks for the PR! However, due to its age it will not be merged. If the issue is still occurring feel free to open a new PR.


unslick not worked

unslick - not working. How to disable the carousel on desktop and enable it on mobile?
responsive: [
  {
    breakpoint: 9999,
    settings: 'unslick'
  },
]

this code working  Vue Slick Carousel npm page.

Hi @bondarenkobv,
Thanks for raising this issue. I didn't originally plan to support the unslick option from the vue-slick-carousel library because I didn't see it fit for a carousel component. But if I were to add an option like that, would it be fine if it just hides all the dots and arrows, and disables all slide navigation events, effectively making the carousel a static display of the slides?
If so, I think I could patch this in the upcoming two days.
Thank you!

Hi @bondarenkobv,
Thanks for raising this issue. I didn't originally plan to support the unslick option from the vue-slick-carousel library because I didn't see it fit for a carousel component. But if I were to add an option like that, would it be fine if it just hides all the dots and arrows, and disables all slide navigation events, effectively making the carousel a static display of the slides?
If so, I think I could patch this in the upcoming two days.
Thank you!

Hi and thank you! Yes, this option is really needed to avoid using workarounds like v-if, v-else. I’d really appreciate it if you could add this!


[bug]

I encountered a bug when depositing money into the myst app: Money deposited, 0 myst
I donloaded the app, created an identity, topped up for 11.43 the payment failed from the side of the myst payment provider but went through on my side (revolut). I restarted the app and got into the panel with the vpn locations, however, 0 myst on the top.

ID: 6c59ebe6-76ac-4f0f-b270-f1317e584be4
Address: 0xecdfe24c49971679bd86437eabbe9bdf203ac97c

@AlexTechGR Thanks for reporting this! So sorry that it didn't work out for you as intended. I was able to find and validate this transaction - it went through just fine, but our payment backend failed to update the order status.
Would you like to receive MYST that you ordered addressed to the ID 0xecdfe24c49971679bd86437eabbe9bdf203ac97c or a refund to your card?

I've issued a full refund


<EMAIL_ADDRESS>hash check failed

Hi, I was trying to install Defraggler, but failed:
Installing 'defraggler' (2.22.995) [64bit]
dl.7z (48 B) [==============================================================================================================================] 100%
Checking hash of dl.7z ... ERROR Hash check failed!
App:         extras/defraggler
URL:         https://www.piriform.com/defraggler/download/portable/downloadfile#/dl.7z
First bytes: 3C 4D 45 54 41 20 48 54
Expected:    2edfa974fc2e8e34c0b8c152077da9c3a4ab28cad81a4ced463f0cf6634a0f01
Actual:      fb249f429180a78992375a0f736343a8cd1f6bffe5e940411f6cca448e019471
Please try again or create a new issue by using the following link and paste your console output:
https://github.com/lukesampson/scoop-extras/issues/new?title=defraggler%402.22.995%3A+hash+check+failed

2edfa974fc2e8e34c0b8c152077da9c3a4ab28cad81a4ced463f0cf6634a0f01 is correct
The hash in the manifest (the “expected” hash above) is correct. Please try again.


question: proper options for parsing a TSV file, retrieving exact contents of each field (including quotes)

Is your feature request related to a problem? Please describe.
question:  is it possible to disable special handling of quoted text, so that when csv-parse is  reading from a stream, it simply splits each line (according to record_delimiter) on delimiter ?
background:

reproduced issue with csv-parse version 4.8.8
parsing a TSV file, where fields may have single or double quotes inside each field
potentially variable number of fields between different lines
can guarantee TSV separators are correct for each line of text

desire:

read each row based on record_delimiter, default looks fine for EOL:  \n, \r\n, or last line i file EOF
read each field strictly on delimiter , no special parsing of " included

issues:

while using the input text example and options/code listed below for reading from a file stream, output doesn't match our desired output noted above
e.g. for line 2, csv-parse strips the " on either side of Q2 double quoted.  --> desired output for first field on that line is "Q2 double quoted"
e.g. for line 3, csv-parse will make "field 1":  all of line 3 (because it starts with ", through the \r\n, and up to the next " on line 4 --> desired output for first field on that line is Q2.3 ending one double quote char"

input text example:
Q1 normal text	A1.1	A2.1	A3.1
"Q2 double quoted"	A1.2	A2.2	A3.2
"Q2.2 beginning one double quote char	A1.2.2	A2.2.2	A3.2.2
Q2.3 ending one double quote char"	A1.2.3	A2.2.3	A3.2.3
quote "Q3 in middle of text" q	A1.3	A2.3	A3.3
quote "Q3 ending text"	A1.3.2	A2.3.2	A3.3.2
"Q4 quote beginning text" q	A1.4	A2.4	A3.4
 "Q5 begin/end with whitespace" 	A1.5	A2.5	A3.5
 "Q6 begin with whitespace and one double quote	A1.6	A2.6	A3.6
Q6.2 end with one double quote and whitespace" 	A1.6.2	A2.6.2	A3.6.2
Q7 "one double quote	A1.7	A2.7	A3.7
Q8 'one single quote	A1.8	A2.8	A3.8
'Q8.2 beginning with one single quote	A1.8.2	A2.8.2	A3.8.2
Q8.3 ending with one single quote'	A1.8.3	A2.8.3	A3.8.3
Q9 'pair of single quotes' case	A1.9	A2.9	A3.9

code example:
      const parser = parse({
        delimiter: '\t',
        'relax_column_count': true,
        'relax': true,
      });

     // ...

      parser.on('readable', function() {
        for (;;) {
          let record: Array<string> = parser.read();

          if (!record) break;

          // do not want blank columns, so remove all of those.
          record = record
            .map(column => column.trim())
            .filter(column => column.length > 0);

          const firstColumn = record.shift();
          const restOfColumns: Array<any> = [];

          // REF ISSUE

          while (record.length > 0) {
            restOfColumns.push({
              text: record.shift(),
            });
          }

        // ...

        }
      });

Describe the solution you'd like

Understand whether there's an option which will give us the output we desire, as noted above

Describe alternatives you've considered

attempted to set escape to null, but upon reading docs further, realized this is only used within quoted text, implying that quoted text smart logic is still being applied
attempted to set relax=true, but that yielded the two issues from line 2 and line 3 of test input example

Additional context
None that i can think of, thank you so much for your help.

If I understood well, your solution is to disable quote. Here is a working script written in CoffeeScript:
parse = require './lib/sync'
assert = require 'assert'

records = parse """
Q1 normal text	A1.1	A2.1	A3.1
"Q2 double quoted"	A1.2	A2.2	A3.2
"Q2.2 beginning one double quote char	A1.2.2	A2.2.2	A3.2.2
""", quote: '', delimiter: '\t'

assert.deepEqual records, [
  [ 'Q1 normal text', 'A1.1', 'A2.1', 'A3.1' ],
  [ '"Q2 double quoted"', 'A1.2', 'A2.2', 'A3.2' ],
  [ '"Q2.2 beginning one double quote char', 'A1.2.2', 'A2.2.2', 'A3.2.2' ]
]

hi @wdavidw , wonderful -- do i pass that in as a parameter, instead of 'relax': true, ?

apologies, i completely missed the option on the help page:
quote (char|Buffer|boolean)
Since early days
Optional character surrounding a field as one character only; disabled if null, false or empty; defaults to double quote.

I think the reason i missed that is i searched the page for literal ", as i figured the explicit character would be listed.
Thank you for your help !
suggested update for quote option help text:
Optional character surrounding a field as one character only; disabled if null, false or empty; defaults to " (double quote).

I will update the document. Do not use relax, it is not related to your use case.

really appreciate your quick help on both guidance, and updating the documentation.  thank you !


Support for collection.name={$topic}

Hello!
I used another Kafka-Mongo sink connector a few months ago, and it has the feature to set the collection name to sink as "{$topic}", and i could listen to several topics and sink them to multiple collections.
If i'm not wrong, this connector do not have that feature. Is this doable? Would be a great help in my projects.

Hello Rafael!
First of all, I'm glad to hear that you are trying to run my connector :) Concerning your comment, you are right that the current implementation does not offer the feature you are mentioning.
Nevertheless, I would like to learn more about your use case and the "project(s)" you are referring to.
As a side node, a very naive form of this feature could be easily added but in general it would not help a lot of users that I discussed this with already in the past. If you need this urgently I guess your best bet is to fork the repo and do a quick & dirty hack yourself.
Also may I ask which other sink you were using. It would also be interesting to know why you looked for something else, like my connector, and moved away from the other one.

@hpgrahsl Thanks for the quick reply!
I've been working with some sensors to measure energy consumption in my IoT startup and i found myself in need of a connector since i'm using MongoDB to store the data & Kafka as the main broker. I'm sending different types of data from the sensors to different topics in Kafka, and wanted them to end up in specific collections in the database. I started off using this connector, that had the feature i mentioned. I had some problems with data conversion and serialization and found out about your implementation, that solved those problems and gave me way better customization options (congratulations by the way!).
I really want to keep using the connector and this feature is a must-have to my architecture. I can play a little by forking the repo and trying to code it myself for now =)

@SkiaBr23 thx for clarifying your use case by providing some more details.
The feature you suggest is in fact a bit more involved than it looks at first sight. I already discussed something similar with other users in the recent past. Find a quick summary of my thoughts below:

Some general connect specific settings can only be defined once for an instance e.g. number of tasks, key and value converters, using the schema registry or not => so from what I've seen you cannot configure this on a topic level, yet different topics might need different settings for these as well and you would then have to run multiple connect instances anyway.

Coming back to my mongo sink and its specific settings: from my own experience (and also based on feedback from other users) only a handful of them (if at all) would be similar across different topics. This means that we would need to allow a large majority of the config settings to be defined individually for each and every topic, e.g. topics might need different id generation strategies, they would highly likely need completely different post processing chains etc.

Of course such behaviour can be implemented and it shouldn't be too hard but IMHO allowing this would lead to a long, complex and bloated single configuration file for typical real world scenarios. Other than that, it would also mean that if the connector fails with some unrecoverable exception for a specific topic it might not be able to happily continue its work for the other topics, which are potentially unaffected by the exception in question.
Although I'm aware that you might save some CPU/RAM resources by running fewer instances, I don't see any real issues with running multiple connect instances, effectively one for each topic... if it weren't for hundreds or thousands of topics of course ;-)
What do you think?

@hpgrahsl thanks for the detailed answer and tips! I ended up using multiple instances of the connector for now and it's been a good fit for the project.

@SkiaBr23 great to hear that what I suggested is helpful and works for your current use case!
further I wanted to ask if you maybe would like to share your concrete use case so that I could feature it under users / customer testimonials like the other two which are already present in the README. just let me know, would be highly appreciated to have an IoT related use case :)

Hello @hpgrahsl I can give you a use case if I may.
For example I have 10 devices (or more doesn't matter) and those devices send data to kafka somehow it doesn't matter but i have for example 5 different types of values (1. batteryVoltage, 2. tankFuelLevel, 3. energyConsumed, 4. solarLoad, 5. windProduced energy) each parameter I want to store in a different collection/table (database) for each parameter from those 5 cases I will create a topic so will be 5 topics on broker. And this feature will help to automapping 5 topics to 5 collections/tables.
In that case I wont need to create 5 instances for each collection/table. Report will be 1:1 (topic to collection) on one single instance.
I hope you understand better why we need ${topic) feature. I'm waiting for your feedback


Don't override color settings from print dialog

When we put print-color-mode as a default attribute, it always overrides
settings from print dialog. We need to respect those settings and
transform the known PPD options into print-color-mode options.

@michaelrsweet thx Mike! Sorry for taking so long...


Animate/visualize the ilumination problem

Inspired by Numberphile - The Illumination Problem - https://www.youtube.com/watch?v=xhj5er1k6GQ .
write a P5.js program that explores how a room gets illuminated.

wrong repo


Update https://dry-rb.org/gems/dry-cli to Version 1.0

Describe the bug
Latest documentation (0.6) is not aligned with latest release (0.7). As if this e.g. this page is outdated: https://dry-rb.org/gems/dry-cli/0.6/file-utilities/
To Reproduce
Open https://dry-rb.org/gems/dry-cli
Expected behavior
https://dry-rb.org/gems/dry-cli shows documentation for 0.7. https://dry-rb.org/gems/dry-cli/0.7/file-utilities page is gone.

Congrats to 1.0 release! Unfortunately the docs are still outdated. On top, when I open https://dry-rb.org/gems/dry-cli/main/commands-with-subcommands-and-params it downloads the file instead..


<IP_ADDRESS> a ssl listener setup

    No, it does not solve the mqtts problem. In <IP_ADDRESS> a ssl connect looks like this:

❯ openssl s_client -port 8883 -host mqtt.localdomain
CONNECTED(00000005)
4380345900:error:140040E5:SSL routines:CONNECT_CR_SRVR_HELLO:ssl handshake failure:/AppleInternal/Library/BuildRoots/b6051351-c030-11ec-96e9-3e7866fcf3a1/Library/Caches/com.apple.xbs/Sources/libressl/libressl-2.8/ssl/ssl_pkt.c:585:
---
no peer certificate available
---
No client certificate CA names sent
---
SSL handshake has read 0 bytes and written 0 bytes
---
New, (NONE), Cipher is (NONE)
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : 0000
    Session-ID:
    Session-ID-ctx:
    Master-Key:
    Start Time:<PHONE_NUMBER>
    Timeout   : 7200 (sec)
    Verify return code: 0 (ok)

but it should look look like this:
❯ openssl s_client -port 8883 -host mqtt.localdomain
CONNECTED(00000005)
depth=2 C = US, O = Internet Security Research Group, CN = ISRG Root X1
verify return:1
depth=1 C = US, O = Let's Encrypt, CN = R3
verify return:1
depth=0 CN = mqtt.localdomain
verify return:1
---
Certificate chain
 0 s:/CN=mqtt.localdomain
   i:/C=US/O=Let's Encrypt/CN=R3
 1 s:/C=US/O=Let's Encrypt/CN=R3
   i:/C=US/O=Internet Security Research Group/CN=ISRG Root X1
 2 s:/C=US/O=Internet Security Research Group/CN=ISRG Root X1
   i:/O=Digital Signature Trust Co./CN=DST Root CA X3
---
Server certificate
-----BEGIN CERTIFICATE-----
[....]
-----END CERTIFICATE-----
subject=/CN=mqtt.localdomain
issuer=/C=US/O=Let's Encrypt/CN=R3
---
No client certificate CA names sent
Server Temp Key: ECDH, P-256, 256 bits
---
SSL handshake has read 4515 bytes and written 322 bytes
---
New, TLSv1/SSLv3, Cipher is ECDHE-RSA-AES256-GCM-SHA384
Server public key is 2048 bit
Secure Renegotiation IS supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : ECDHE-RSA-AES256-GCM-SHA384
    Session-ID: 81497B9CB586F8822E600B8B95BA51FAFD766EAAEF2D6AC8017AED53B3EABD3B
    Session-ID-ctx:
    Master-Key: CF45890E1887C98A883AE7C282DB43F7CEF897C76BEDC97C8ADC9C51C279C716C88F31A5AF0B348F59DB607F838DA432
    Start Time:<PHONE_NUMBER>
    Timeout   : 7200 (sec)
    Verify return code: 0 (ok)
---
quit
closed

Config is 100% the same.
SSL certificates are configured like shown above using env vars.
<IP_ADDRESS> does load the certificates correctly!?
Originally posted by @kettenbach-it in https://github.com/vernemq/vernemq/issues/2061#issuecomment-1367585244

Looks like the first one was compiled in OSX with Apple LibreSSL?

I have the same issue:
have deployed to k8s cluster using helm chart, pod started fine and conf looks like this:
########## Start ##########
allow_register_during_netsplit=on
allow_unsubscribe_during_netsplit=on
accept_eula=yes
allow_publish_during_netsplit=on
listener.ssl.default=<IP_ADDRESS>:8883
listener.ssl.keyfile=/etc/ssl/vernemq/tls.key
listener.ssl.cafile=/etc/ssl/vernemq/ca.crt
listener.tcp.localhost=<IP_ADDRESS>:1883
allow_anonymous=on
listener.ssl.certfile=/etc/ssl/vernemq/tls.crt
allow_subscribe_during_netsplit=on
listener.tcp.default = <IP_ADDRESS>:1883
listener.ws.default = <IP_ADDRESS>:8080
listener.vmq.clustering = <IP_ADDRESS>:44053
listener.http.metrics = <IP_ADDRESS>:8888
########## End ##########
but pod logs shows this error:
20:13:35.558 [error] can't reconfigure mqtts listener({127,0,0,1}, 1883) with Options [{max_connections,10000},{nr_of_acceptors,10},{mountpoint,[]},{cafile,"/etc/ssl/vernemq/ca.crt"},{depth,1},{certfile,"/etc/ssl/vernemq/tls.crt"},{eccs,[sect571r1,sect571k1,secp521r1,brainpoolP512r1,sect409k1,sect409r1,brainpoolP384r1,secp384r1,sect283k1,sect283r1,brainpoolP256r1,secp256k1,secp256r1,sect239k1,sect233k1,sect233r1,secp224k1,secp224r1,sect193r1,sect193r2,secp192k1,secp192r1,sect163k1,sect163r1,sect163r2,secp160k1,secp160r1,secp160r2]},{keyfile,"/etc/ssl/vernemq/tls.key"},{require_certificate,false},{tls_version,'tlsv1.2'},{use_identity_as_username,false},{allowed_protocol_versions,[3,4,131]},{allow_anonymous_override,false}] due to {already_started,<0.461.0>}
I tried using " openssl s_client -connect :8883 -key tls.key.pem -cert tls.cert.pem", got back the info below.
CONNECTED(00000003)
write:errno=54
--- no peer certificate available
--- No client certificate CA names sent
--- SSL handshake has read 0 bytes and written 287 bytes
--- New, (NONE), Cipher is (NONE) Secure Renegotiation IS NOT supported Compression: NONE Expansion: NONE No ALPN negotiated SSL-Session: Protocol : TLSv1.3 Cipher : 0000 Session-ID: Session-ID-ctx: Master-Key: Start Time:<PHONE_NUMBER> Timeout : 7200 (sec) Verify return code: 0 (ok)

@ioolkos were you able to figure out the root cause. If so could you please post the update here

@sivaNbalusu5
as a general rule I recommend to use "named" configurations, as in the following example (with listener name "default")
listener.ssl.default=<IP_ADDRESS>:8883
listener.ssl.default.keyfile=/etc/ssl/vernemq/tls.key
listener.ssl.default.cafile=/etc/ssl/vernemq/ca.crt
listener.ssl.default.certfile=/etc/ssl/vernemq/tls.crt

Also, your OpenSSL call is wrong as you don't want to give client cert and key. (you do not require a client cert from the broker side).
Try a call like openssl s_client -connect localhost:8883 first.
The logged error about listener reconfiguration is a minor issue but not related. (it's related to default listeners given; will fix this)

:point_right: Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
:point_right: Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.

For MQTTS listeners in Docker images, notice this.
https://erlangforums.com/t/latest-vernemq-not-allowing-tls-traffic-with-self-signed-ca-certificate/2394/11?u=afa
A lot, if not most of the SSL issues in the Docker image come from the configuration of the path to the CAfile.
If the 'default' listener name creates clashes, use different listener names for the listeners you'd like to configure. (example listener.tcp.my_listener = <IP_ADDRESS>:1883)

:point_right: Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
:point_right: Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.

The listener error is fixed by: https://github.com/vernemq/vernemq/pull/2113

:point_right: Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
:point_right: Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.


Update README and build scripts to recommend Vagrant >= 2.2.2

Operating System Version: Ubuntu 18.10
Provider (VirtualBox/VMWare): Virtualbox
Vagrant Version: 2.0.3
Packer Version: N/A
Is the issue reproducible or intermittent? Reproducible

Description of the issue:
README recommends Vagrant >= 1.9.2. Recommend updating that to 2.2.2. Looks like earlier versions, at least to 2.0.3, don't like the default_nic_type setting in the Vagrantfile.
Tried spinning the lab up on an Ubuntu machine. The current Ubuntu Vagrant package is 2.0.2.
This runs into an issue with fog-core. Manually updating to 2.0.3 as recommended solves that issue.
However, after running vagrant up with 2.0.3, I get the following error:
VirtualBox Provider:
* The following settings shouldn't exist: default_nic_type

Manually updating to 2.2.2 from Vagrant fixes the issue.

Hey @6r0k3d - thanks for the report! I'll update the README and build scripts to ensure version 2.2.2 or greater is being used.


TypeError: unsupported operand type(s) for +=: 'float' and 'list'

hello
I got the typeerror while running text_style_transfer example.
This is terminal error

File "main.py", line 179, in main
eval_epoch(sess, gamma, lambda_g, epoch, 'val')
File "main.py", line 133, in _eval_epoch
avg_meters.add(vals, weight=batch_size)
File "C:\Users\Bahar\Anaconda3\envs\envm\lib\site-packages\texar\tf\utils\average_recorder.py", line 222, in add
self._recorders[name].add(val, weight=weight)
File "C:\Users\Bahar\Anaconda3\envs\envm\lib\site-packages\texar\tf\utils\average_recorder.py", line 65, in add
self._sum += record * w
TypeError: unsupported operand type(s) for +=: 'float' and 'list'

Please help me

Sorry for the late reply. We will take a look at it.

@gpengzhi
Thank you.
Is there any answer for this problem? I can't solve it.

pls see my response in https://github.com/asyml/texar/issues/276#issuecomment-641545303


Y24-289 - [BUG] Exception 29/08/2024 at 15:28

Email subject line
[Psd-exceptions] [Sequencescape Production] searches#index (ActionView::Template::Error) "uninitialized constant Barcode::FormatHandlers:...
Error body
An ActionView::Template::Error occurred in searches#index:

  uninitialized constant Barcode::FormatHandlers::AkerBarcode
  app/models/barcode.rb:222:in `const_get'

Acceptance Criteria

[ ] Look into this exception and fix the code to handle the exception sensibly.
[ ] Document the root cause and solution in a comment on this issue.

Additional information
The id of the user who triggered this error is often shown in the exception email, in case you need to communicate with them.

@wendyyang More context. This is happening because someone is running a Sequencescape search  /searches?type=&q=7747.
It looks like they are doing a search for study 7747 but because they are not specifiying a type it is actually looking for everything with an id of 7747.
We probably don't want it to fail but it is worth a check on how it is handled.
It will be difficult to reproduce in UAT as we might not have any Aker barcodes in the table.

This exception is raised in the Sequencescape console when calling labwhere_location on a Labware object that has an aker_format (id = 4) barcode
e.g.
l = Labware.find 23061777
l.labwhere_location
/var/www/sequencescape/releases/20240920142713/app/models/barcode.rb:222:in `const_get': uninitialized constant Barcode::FormatHandlers::AkerBarcode (NameError)
I've come across this when writing reports in Ruby and have handled this with:
class Labware
  def find_location()
    if self.barcodes.map(&:format).include?("aker_barcode")
      location = nil
    else
      location = self.labwhere_location
    end
    return location
  end
end

barcodes = Barcode.where(format:4)
labware  = barcodes.map(&:asset)

using the above method
labware.map(&:find_barcode)
=> ["NT1457915H", "DN537747W", "DN537868H", "DN537869I", "DN537870B", "DN538319K", "NT1462400Q", "DN538443N", "DN552057Q"]

DN537747W is the barcode being matched in the search


[WIP] Create CODEOWNERS

@grugnog i've added a team for each docs/[folder].  Each team is a subteam of the main CivicActions team, and a list of team with their members can be seen here: https://github.com/orgs/CivicActions/teams/civicactions-team/teams
This approach to governance adds github team management as an ongoing responsibility of the Docs working group. I've also added the subteams to our onboarding template here: https://trello.com/c/I5L6gPiQ/174-add-to-github
If you think this is the right approach, let's add people to the appropriate subteams before we merge this in.  I'll also write this out in the docs/00-contributing/docs-governance.md


chore(deps): update termion to v3.0

This PR updates termion to its latest version. There is only one breaking change https://gitlab.redox-os.org/redox-os/termion#200-to-300-guide.
We don't really use this but our users do in when initializing the terminal. I'm not sure then if this is a breaking change. I'd say no because users can probably still specify termion 2 right?

Perhaps link to the changes in the commit to make it easy for users.

Perhaps link to the changes in the commit to make it easy for users.

Good idea, done.


Issue with obs submission

From Zach Guy at CBAC: Hey Charlotte, we’ve had several people tell us that when they try to submit an observation using the app, nothing happens. We have yet to get an observation, and I just tried it this morning and had the same issue. When I fill out the form, and then hit the submit button, nothing happens. I’m heading into the field right now, but happy to troubleshoot tomorrow or whenever! Thanks for your help.
I tried to repro and for me the location selection isn't work which is a required field so I can't get submit to go through.

I can't reproduce the issue getting location selection to work on any platform (attempted on iOS, Android, both on the production app, preview app, and Expo Go). I can see that the "submit" button does nothing, need to look into this. I hope the new architecture didn't break our background controller.


Create acceptance.md

Here are manual acceptance steps for Indy SDK.

Can you please DCO sign your commit?  See https://github.com/hyperledger/indy-sdk/pull/2281/checks?check_run_id=1378952060
Thanks

(ci) please test

ci test this please

CI test this please

@VladimirWork can you please update this PR to the latest code in the main branch?  I'd like to merge the PR and I don't have access to update the branch myself.  Thanks!

Somehow I also cannot update this branch so I created another PR from my fork and it is in sync with current master: https://github.com/hyperledger/indy-sdk/pull/2285

Thanks @VladimirWork  I'll merge the new PR


[Bug]: REPORT 09: All Cities in Region

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Name
Bobby Walters
Description of the bug?
1.Output column titles need formatting to ALL CAPS
2. When ehe region , Eastern Asia was entered, the cities for the Caribbean are displayed.
Version
Develop Branch
Steps to Reproduce the Error
Enter Region
Eastern Asia
All Cities by Region: Eastern Asia

City                                Country                                  District                       Population
La Habana                           Cuba                                     La Habana                      2256000
Santo Domingo de Guzmán             Dominican Republic                       Distrito Nacional              1609966
Port-au-Prince                      Haiti                                    Ouest                          884472

Code of Conduct

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

This Report and UC14 are Similar and can be written in One Class


Feat: Wallet button

Wallet button
Wallet popover
Fiat for every wallet
Full accessibility for wallet management

Task linked: CU-8692u8ag6 Update WalletSelect component

Task linked: CU-862kcp3zd Update wallet tab (navigation to wallet details)


Backoff on reestablishing watches when Unavailable errors are encountered

this is a targeted fix to the predominant CPU consumer when an etcd server becomes unavailable to a client with established watches. this picks cleanly back to 3.3.x and 3.2.x
This took our apiserver from 700% cpu when etcd was temporarily down to 10% cpu
xref #9578 #9740

/cc @gyuho @jpbetz if we use the new balancer, we will have a builtin retry mechanism?

@xiang90 @liggitt
@jpbetz has implemented new retry in interceptor layer (still in development branch for testing), but not for this watch routine. We will see if we can incorporate those. But, if we need this be backported for Kubernetes, something simple like this would suffice as a transitional solution.
Defer to @jpbetz


Update Node's MAC address to the Node's annotation for direct routing

To bypass Windows host network when forwarding Pod egress traffic in noencap mode, antrea-agent needs to know peer Nodes' MAC addresses so that it can configure Openflow rules which route the packets to underlay network via uplink interface directly.
To discover the MAC addresses, the PR makes each antrea-agent report its uplink's MAC address to the Node's annotation, then other agents can get it when the NodeRouteController configures route/flows to this Node.
Signed-off-by: Quan Tian<EMAIL_ADDRESS>For #2157

/test-all

/test-all

/test-networkpolicy

/test-windows-conformance
/test-e2e
/test-all-features-conformance

/test-e2e /test-conformance /test-networkpolicy

@antoninbas @lzhecheng I rebased on main and added unit test for initNodeLocalConfig. Could you approve again?

/test-all

/test-all

/test-all


mutable objects with appended methods not recognized as constants

Hi,
I don't know if this is the proper place to report this issue, it is in between Rubocop and Rubocop Rails. I did not report to Rubocop because with_indifferent_access is not a Ruby method
I've noticed that mutable objects chained with with_indifferent_access / stringify_keys do not trigger a Style/MutableConstant offence
Is this an expected behavior?

Expected behavior
Get a Style/MutableConstant warning
Actual behavior
No warnings
Steps to reproduce the problem
# frozen_string_literal: true

class User
  CONSTANT = {
    external: 'external',
    internal: 'internal'
  }.with_indifferent_access # without `.with_indifferent_access` I'm getting the warning
end

RuboCop version
1.48.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.27.0, running on ruby 3.0.4) [x86_64-darwin22]
  - rubocop-capybara 2.17.1
  - rubocop-performance 1.16.0
  - rubocop-rails 2.18.0
  - rubocop-rspec 2.19.0

That's the default behavior, yes. If you want all constant expressions to be frozen, configure the cop in strict mode:
https://docs.rubocop.org/rubocop/cops_style.html#configurable-attributes-59
Style/MutableConstant:
  EnforcedStyle: strict

Sorry, thanks, that is what I was looking for!


bump: alloy core and use PrimitiveSignature

Bump alloy/core and use PrimitiveSignature

Have locally run devnet-5 tests,
Running tests in ../../tests/pectra_devnet5/state_tests/...
███...███ 170/170Finished execution. Total CPU time: 1.580341s
All tests passed!


Bluetooth: Mesh: Verify network buffer max len

As network_decode needs a target buffer for decoding, the max PDU length
must be checked to prevent overflow on the target buffer. When receiving
a proxy configuration message with excessive length, there's no previous
check for this.
Also pulls the NET PDU length defines out into net.h, so they can be
used when defining the target buffers.
Signed-off-by: Trond Einar Snekvik<EMAIL_ADDRESS>

Alright, I'll open an issue then. I think this is a serious enough issue that we should ensure it goes in.


add RBAC to 2020-10-01-preview autogen

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify

Adding specs to package tag that were previously left out

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
N/A

When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
Swagger is already public

If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[x] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[x] Updating API in a stable or public preview version with Breaking Change Validation errors
[x] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Most of the changes in swagger files are of moving parameters to common-types file or adding type information to the objects. Signing off.

/azp run


Ts action bootstrap

#6 Turned into a GitHub action that can be referenced from other repositories

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


Import / Export: does not export tenant data other than admin

Steps to reproduce the problem:

Tenant admin with templates, devices and flows created
Create user from tenant other than admin
Log in with this user
BD empty. Export the data.
Verify that the admin data is exported.

Create template "t" and device "d". Export the data.
Verify that only the admin data is exported.

Affected Version: 0.3.0-nightly_20181227

Test OK.


Allow use hotkey directive as shorcut to click element

Like this:
<button ng-click="doSomething()" hotkey="ctrl+q">

Why? :

Today, it is writing like this:

<button ng-click="doSomething()" hotkey="{'ctrl+q': doSomething}">

Besides of code duplication and verbose syntax, we have to manage click availability problem inside code, for example:

<!-- click not trigger when 'exepression' is true, but with hotkey still working! -->
<button ng-click="doSomething()" ng-disabled="exepression" hotkey="{'ctrl+q': doSomething}">

+1 i need this too

+1

+1

+1


Bump DALI version. Adjust ThreadPool constructor

Bumps DALI to 1.16. Adjusts ThreadPool constructor to new version.

CI MESSAGE: [5355869]: BUILD PASSED

CI MESSAGE: [5442890]: BUILD STARTED


hb_correction overrides manual rbc:m ratio

I've entered an RBC:M ratio in the config file using
self.rbc_m_ratio = 0.401

When I enter a Hb value in the config file using the following code, the RBC:M ratio printed on the report is not the value I enter manually
# set what kind of hemoglobin correction to use
self.hb_correction_key = constants.HbCorrectionKey.RBC_AND_MEMBRANE.value
# define subject hemoglobin in g/dL
self.hb = 12.5
```

However, when I comment out the Hb correction from the config file, the RBC:M is correctly reported as 0.401.

Other than how to fix the issue, I'm wondering if this is a reporting problem or is the wrong RBC:M propagated through the calculations?

Thanks as always,
Yonni

Hi Yonni -- I think this is actually the expected behavior, assuming it's not going completely off the rails with your RBC:M value. The Hb correction affects both the RBC and the M values, and the Hb-corrected version is what's used in the report.
A Hb value of 12.5 is a bit lower than normal, so we'd expect the correction to bring the RBC signal UP and the membrane DOWN. The RBC:M thus gets a double whammy and gets driven upward to some degree. It shouldn't huge for an Hb of 12.5 but definitely noticeable.
Is that what you're seeing?
Note the original RBC:M is still used for the Dixon decomposition. The Hb-corrected version just gets output in the report.

No problem! Yeah I have to think it through every time to remember which way the corrections go...
Closing this bug.


[Issue] Change Remote Containers badge in templates to Dev Container

Remote containers was rebranded Dev Containers.
Our templates have

Would be good to update to this style
[![Open in GitHub Codespaces](https://img.shields.io/static/v1?style=for-the-badge&label=GitHub+Codespaces&message=Open&color=lightgrey&logo=github)](https://codespaces.new/azure-samples/openai-plugin-fastapi)
[![Open in Dev Container](https://img.shields.io/static/v1?style=for-the-badge&label=Dev+Container&message=Open&color=blue&logo=visualstudiocode)](https://vscode.dev/redirect?url=vscode://ms-vscode-remote.remote-containers/cloneInVolume?url=https://github.com/azure-samples/openai-plugin-fastapi)

@rajeshkamal5050 Super low hanging fruit

@vhvb1989 can we get the readme's updated?


Lots of orphaned progress notifications after interactive window fails to open

Issue Type: Bug
Repro
Same initial steps as #14509:

Install python extension
Run something in the python terminal
Now open python interactive window using run in interactive window
See alert about installing ipykernel
Click button to install

The install should never complete. Now:

Click the cancel buttons on the install notifications.
See some error notifications
The interactive window closes
Click on VS Code's notifications

Bug
There are still progress tasks running:

Specifically Connecting to IPython kernel: Fetching Kernels and Executing Cell. I don't seem to be able to cancel or dismiss either of these
Extension version: 2020.9.114305
VS Code version: Code - Insiders 1.51.0-insider (7a3bdf4ee9588755d447aa1c3b5db4a123fc11a9, 2020-10-23T07:32:22.213Z)
OS version: Darwin x64 19.6.0

Thanks for reporting this @mjbvz. The 'Connecting to IPython kernel' message is created on a progress indicator that isn't cancellable, so at minimum we should pass the cancelToken if any here to createProgressIndicator:
    protected async createNotebookInstance(
        resource: Resource,
        identity: vscode.Uri,
        disableUI?: boolean,
        notebookMetadata?: nbformat.INotebookMetadata,
        cancelToken?: CancellationToken
    ): Promise<INotebook> {
        traceInfo(`Creating raw notebook for ${identity.toString()}`);
        const notebookPromise = createDeferred<INotebook>();
        this.setNotebook(identity, notebookPromise.promise);

        const progressReporter = !disableUI
            ? this.progressReporter.createProgressIndicator(localize.DataScience.connectingIPyKernel())
            : undefined;

Same with:
Version: 1.51.0-insider (user setup)
Commit: e5c0007ce0e9a3816679cba674d03376c03726e3
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.20226
Python Extn : v2020.9.114305
Pylance: v2020.10.2


Installation issue

Error<EMAIL_ADDRESS>is missing from the TypeScript compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.

@rogsmith it's not very clear how to reproduce it.
Please provide some steps to follow. We'll configure it.

So I just ran npm install in my Angular app, and then tried to start my app.

@rogsmith, I'll provide you with working angular example by the end of the day.
We are updating our angular library at this moment for easier integration.

@revolist perfect thank you.

@rogsmith
Documentation has been updated:
https://revolist.github.io/revogrid/guide/framework.angular.overview.html
The sample has been updated:
https://codesandbox.io/s/angular-datagrid-overview-d5i0b?fontsize=14&hidenavigation=1&theme=dark&file=/src/app/app.module.ts
Github initial Repo provided:
https://github.com/revolist/angular-datagrid-test


Add CodeQL

add codeql-analysis.yml which automatically enables CodeQL code security and quality scanner. It executes on every push commit, PR, manually and every day at 8:00AM UTC. A scan check can be viewed on commits and PRs. All alerts and fix suggestion can be viewed at Security tab -> Code scanning alerts -> CodeQL

Resolves #1562

It looks like @aleks-ivanov hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement here.
Once you've signed reply with [clabot:check] to prove it.
Appreciation of efforts,
clabot

It looks like @aleks-ivanov hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement here.
Once you've signed reply with [clabot:check] to prove it.
Appreciation of efforts,
clabot

[clabot:check]

@confluentinc It looks like @aleks-ivanov just signed our Contributor License Agreement. :+1:
Always at your service,
clabot


[analysis_server] provide a fix to disable a lint rule in analysis_options.yaml

Description
Provide a fix to disable a lint rule in analysis_options.yaml that is similar to ignore diagnostic fixes. The ignore diagnostic fixes can either ignore a line or an entire file by adding a comment above the line to be ignored or the top of the file for the entire file to be ignored (e.g. // ignore:  public_member_api_docs for a line).
A fix to disable a specific lint rule would instead disable the lint rule in the analysis_options.yaml file such as:
include: package:very_good_analysis/analysis_options.3.0.1.yaml
analyzer:
  exclude:
    - build/**
linter:
  rules:
    public_member_api_docs: false  # <~~~~~~~ add here by the quick fix

Additional Context
This feature request is necessary for an downstream feature request in the VS Code extension repo which can be found here: https://github.com/Dart-Code/Dart-Code/issues/4079

Is there any news about this?

Is there any news about this?

No, I'm sorry, but there isn't. While we agree that this would be a good feature to have, it isn't currently at the top of the priority list.

CL submitted at: https://dart-review.googlesource.com/c/sdk/+/352220
Though unlike the example I provided above, the CL adds ignore fix  to the analyzer.errors label such that:
analyzer:
  errors:
     public_member_api_docs: ignore

I believe the above works for any kind of warning/error.  For instance, unused_import cannot only be ignored when added analyzer.errors.
I hope I am not missing anything out and any feedback/review is appreciated.


Breaks on server side rendering

/node_modules/react-motion-ui-pack/lib/get-vendor-prefix.js:9
if (!document) return prop;
^
ReferenceError: document is not defined

same as #25

This has been fixed https://github.com/souporserious/react-motion-ui-pack/commit/22a2776a2fe5b62932ea40c7c15a67fd964773fa I will push out an update ASAP

Many thanks for the quick reply

Fixed :) https://github.com/souporserious/react-motion-ui-pack/commit/991400d35dc5329ff0c74fbe3b866dc8251fecbc


Add payload config for dubbo:service/

What is the purpose of the change
Fixes a part of #10773
Brief changelog
Support to specify payload limit for <dubbo:service/>. User can configure a different payload with the payload in <dubbo:protocol> or <dubbo:provider> for <dubbo:service/> if it is needed.
Verifying this change
Run the related unit tests.
Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #10931 (a25e6cb) into 3.1 (dce5507) will decrease coverage by 4.13%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                3.1   #10931      +/-   ##
============================================
- Coverage     68.99%   64.86%   -4.14%
+ Complexity      467       14     -453
============================================
  Files          1364     1421      +57
  Lines         72448    58892   -13556
  Branches      13045     8646    -4399
============================================
- Hits          49983    38198   -11785
+ Misses        17945    16674    -1271
+ Partials       4520     4020     -500

Impacted Files
Coverage Δ

...n/java/org/apache/dubbo/config/ProviderConfig.java
85.05% <ø> (-0.50%)
:arrow_down:

...org/apache/dubbo/config/AbstractServiceConfig.java
97.46% <100.00%> (-0.90%)
:arrow_down:

...luster/router/script/ScriptStateRouterFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...zookeeper/curator/CuratorZookeeperTransporter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...pc/cluster/router/file/FileStateRouterFactory.java
0.00% <0.00%> (-80.96%)
:arrow_down:

...o/rpc/cluster/router/script/ScriptStateRouter.java
0.00% <0.00%> (-73.34%)
:arrow_down:

...mmon/serialize/fastjson2/FastJson2ObjectInput.java
0.00% <0.00%> (-62.50%)
:arrow_down:

...ting/zookeeper/curator/CuratorZookeeperClient.java
15.38% <0.00%> (-61.73%)
:arrow_down:

...mon/serialize/fastjson2/FastJson2ObjectOutput.java
0.00% <0.00%> (-56.25%)
:arrow_down:

...org/apache/dubbo/remoting/api/pu/DefaultCodec.java
33.33% <0.00%> (-33.34%)
:arrow_down:

... and 484 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

The essential cause of this problem should be caused by connection reuse. Maybe we can add some conditions to make the scope of connection reuse smaller.


Cluster/Agent name is missing in grafana

From grafana unable to filter the events based on cluster name.
Proposal:
Agent sending events to server add cluster/agent name and in grafana add filter for cluster name.

Working on it

Done as part of https://github.com/kube-tarian/kubviz/commit/4f430a893a236f925d1ecf5f71de9cd1719c8e01


EAC Stop

is this okay to use this mod ?
https://www.nexusmods.com/eldenring/mods/90
And after I start the game normally using Steam and i'm using your mod after.
Is it ok this way ?

This app will start the game without EAC, there is no need for the mod. Start the game from the app, not steam, and EAC will be disabled.


zh-translation: /docs/root/configuration/http/http_conn_man/header_sanitizing.rst

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes: /docs/root/configuration/http/http_conn_man/header_sanitizing.rst
Release Notes:
[Optional Runtime guard:]
[Optional Fixes #Issue] #280
[Optional Deprecated:]

非常棒的翻译。


Back to the fixtures

Coverage decreased (-0.2%) to 86.505% when pulling f9c6b870bb3211b6f4baa5137d44f529402ac269 on back-to-the-fixtures into 7723c19297455065dbc38ca2eb5c91f9530197f3 on master.


service mock

service mock needs to be separated just like load balance

Sorry, because this issue was raised very early, we have closed it temporarily. You can verify whether this problem still exists in the latest code of the develop branch. If this problem still exists, please reopen it at any time.


Jeff Minter Collection

Hi there, do you plan to add the key for the recently released Jeff Minter collection?

Hi, thank you for letting me know about this release.
I will download the game and add the key.

Please see release 0.5.0 for support of Llamasoft: The Jeff Minter Story.