populate_db : Add non-ASCII and non-BMP characters to usernames and stream names.

Previously, we had only plain text for our user and stream names
on populate_db.  with this PR, populate_db will have non-ASCII and
non-BMP characters on both stream names and user names.
Related issue: #14991
Testing plan:
Tested using ./manage.py populate_db --extra-users 400 --extra-streams 30
GIFs or screenshots:
Stream Names:

User Names :

Thanks for the merge @timabbott and sure I would work on adding this to default streams too :) !.


Bug: focus doesn’t move from page navigation

If you tab to the ‘on this page’ navigation section and then click enter on one of the links visually it scrolls to the appropriate section. However, the focus remains on the link pressed to when you try to continue tabbing it jumps back up to the ‘on this page’ section instead of focusing on the next interactive element in the chosen section.

@muditchoudhary Looks like a solvable issue. Try it out

@mud1tx Looks like a solvable issue. Try it out

@ArslanYM @EmmaDawsonDev giving this issue a try.

@EmmaDawsonDev Is this issue still open?

Yes, would you like to be assigned @mud1tx ?

@EmmaDawsonDev yes I can work on it, but can you please explain the issue.

@EmmaDawsonDev If you wait for 2 sec after clicking the link everything is working don't know why it is taking that much time. as I click the link on navbar it should instantly move to that page but there is a delay of atmost 2sec.

@EmmaDawsonDev I think it works like that only don't know how to solve this issue.

Hmm, That's strange. I'm not home right now so can't investigate. I'll take another look when I'm home next week

Check https://www.w3.org/WAI/fundamentals/accessibility-intro/ in this they have the behaviour where the focus is also shifting to the appropriate section.
Solution:
Moving the focus to the appropriate section requires a small code change we can wrap the H2 (section heading) with a tag and this will solve this issue. In addition to shifting the focus we should also do some visual style change to indicate the focus has changed (especially for large screens)
@EmmaDawsonDev

Check https://www.w3.org/WAI/fundamentals/accessibility-intro/ in this they have the behaviour where the focus is also shifting to the appropriate section.
Solution:
Moving the focus to the appropriate section requires a small code change we can wrap the H2 (section heading) with a tag and this will solve this issue. In addition to shifting the focus we should also do some visual style change to indicate the focus has changed (especially for large screens)
@EmmaDawsonDev

Can confirm that this solves the issue! Also, can you please hint at what visual changes you are suggesting for the focus change?

I've taken a look at the code from the w3c above and they are not wrapping anything in anchor tags, they are just using ids to reference links in the same way that it is being done in this repo so I'm not really sure why this is not working. There must be something slightly wrong in the code somewhere

I think I've worked out the problem and it's due to using the NextJS Link component instead of just an anchor tag.
To fix this in the NavPage component, change line 18 from <Link href={item.href}>{item.linkName}</Link> to {item.linkName} and see if that solves the problem.
@mud1tx would you like to have a go or is anyone else interested?

I can take this issue @EmmaDawsonDev

Thanks Neha!

Hey, @EmmaDawsonDev any style change I need to add to the focused section? Eg: outline or background colour that will fade after a few seconds?


Problems replicating data from NAV to BC when using Traefik

PLEASE DO NOT INCLUDE ANY PASSWORDS OR TOKENS IN YOUR ISSUE!!!
Describe the issue
We face some issues replicating data from our NAV18 to BC 21.2 using an ISV solution.
This has been workin as inteeded earlier when using BC 20.1 with Traefik  "tobiasfenster/traefik-for-windows:v1.7.34", after upgrade to BC 21.2 this stopped working.
We have tried to make a similar setup with local DB (outside container) and BC 21.2 but without Traefik, and this is working as intended.
One thing we noticed is when trying to create new Traefik container with (Setup-TraefikContainerForBcContainers) it always uses version 1.7.34 even when newer version is available for tobiasfenster/traefik-for-windows.
Scripts used to create container and cause the issue
Setup-TraefikContainerForBcContainers -PublicDnsName bc.kvik.com -CrtFile "C:\crtfile.crt" -CrtKeyFile "C:\pemfile.pem" -IP **.*.*.** -Recreate

Screenshots

Additional context

does it happen all the time?
Yes it does happend all of the time
did it use to work?
Is used to work in earlier versions of BC (20.1) with traefik

@tfenster do you know why this might happen?

@tfenster
The replication is made as a httpwebrequest sending an XML header and body through SOAP, using basic authentication with username/webservicekey.

OK, how big are those files? And has that changed between 20 and 21? Because there is a (configurable) size limit in Traefik, maybe that is the reason

We've seen instability meaning the higher the file size the more errors. And seems to have hit a dead end where even file sizes with 1 record file size should be a few KB's are also failing.
As far as i can see the default maxRequestBodySize is 0, but maybe we should investigate further.

Was this issue resolved?

This might very well be the same problem as https://github.com/microsoft/navcontainerhelper/issues/2945 - as this problem surfaced in 21.
Will close this one, problem should be solved in 21.7


Move CLEU to it's own frame handler

Can you move CLEU to it's own frame and event handler. I did this to my local copy and it improved my frame rate by at least 20.
    local frame = CreateFrame("Frame")
    frame:RegisterEvent("COMBAT_LOG_EVENT_UNFILTERED")
    frame:SetScript("OnEvent", function() addon:COMBAT_LOG_EVENT_UNFILTERED() end)

is all you need in the OnEnable()

Not sure I follow that reasoning.
Aside from using native frames instead of AceEvent wrapper, there shouldn't be a difference.

Brought this up with another dev, he traced it down to

it's not AceEvent it's CBH
2x next() calls, 1x xpcall

Thanks for bringing this to my/our attention, I'll make the optimization today

Also your CLEU handling needs optimization.
With out the addon loaded, I can get ~75fps in Dalaran. With it turned on I'm at ~35fps.
The order of operations inside CLEU should bail as early as possible so that it's not doing a lot of useless checks before realizing the player is idle out of group.
CLEU is a very very chatty event and it shouldn't be causing such a FPS loss.


Removed sensor_msgs::msg::PointCloud references because is deprecated

Related to this PR https://github.com/ros2/common_interfaces/pull/254

Linux
Linux-aarch64
Linux-rhel
Windows

Linux
Linux-aarch64
Linux-rhel
Windows

@clalancette what do you think about this one ?

Linux
Linux-aarch64
Linux-rhel
Windows

@clalancette what do you think about this one ?

See my comment in https://github.com/ros2/common_interfaces/pull/254#issuecomment-2213790380 ; I'm not totally convinced we should do this right now.


Ask about the relevant knowledge of find-leaks API.

The result of find-leaks can be output to a detailed file, or type'memlab analyze detected-DOM --snapshot./test1.heapsnapshot --output text 'detailed content is output in text form

I want to write a plugin to use find-leaks to process parsed data. Is there any idea for this?

Looking forward to your answer！！！

@boliangleung

The result of find-leaks can be output to a detailed file, or type'memlab analyze detected-DOM --snapshot./test1.heapsnapshot --output text 'detailed content is output in text form

Are you suggesting adding a new option flag to specify the output file?

I want to write a plugin to use find-leaks to process parsed data. Is there any idea for this?

I don't quite understand your question. Could you provide more details about this?

Closing this issue due to lack of clarification. Please feel free to reopen with more details.


Extending the Key Loader

I have an ops requirement to load my key from an envar. This looks easy to do, I just need to implement the KeyLoaderInterface and have my class injection into the JWSProvider
I could make the change to my keyloader in a compiler pass, but it would be nicer (in my eyes) to set it in the config (like the encoder service). Should I make the config change and submit a PR? Or is there not enough interest to warrant this as a feature?

Hey @MisterGlass,
I plan to review a bit the "encoding" part of this bundle for a v3, things are going too far, we've somehow rewritten the JOSE library this bundle depends on to correct some bad/missing behaviours, we should have changed the default JOSE library instead, which is what will happen in 3.0.
So I propose to keep this open and keep using a compiler pass for solving this need until I start to rework this part of the bundle so that we have more visibility on what should be done (should happen very soon, keep you informed here).
Thanks

Thanks. I actually ended up using a server level workaround for my problem, but this seems like a worthwhile feature.

Está bueno el script

loadstring(game:HttpGet("https://raw.githubusercontent.com/alan11ago/Hub/refs/heads/main/ImpHub.lua"))()

Hmm

cómo consigo la key?🐍


harmonise task - PMID 33983923

1021 studies with empty harmonised files.

These files have been formatted but the raw files only contain chr, bp and p-value. Formatting has only added a lot of NA columns. Harmonised files are empty.
I will remove the harmonised directories. Should not re-enter harmonisation pipeline as not all required headers exist.

done.


Refactor Product improvements into a button (per mock)

What

[ ] Refactor the snackbar Product improvements into a discreet button on the product page, per mocks

[ ] https://github.com/openfoodfacts/smooth-app/commit/d41a0b1d55097371c6b2fd5a421455160b775e83
[ ] The buttons have to be ventilated across panels (if Nutri-Score, on the health panel, if Eco-Score on the environment panel, if Ingredients on the Health panel). For the summary card, we need to be inclusive: "Add a category for Eco-Score and Nutri-Score".

[ ] On button click, let's open as a first the Web View edit mode (and we'll plug the merged novelties from #505 a bit later)

Mockup

@stephanegigandet how should we do this one ? Should BE tell the client what improvement can be done for the product in a knowledge panel ? Or should the client figure it out on it's own.


Integration tests improvements

added hvac service integration test
added external component config for vs.code tasks and containers (prerequisite_settings.json)
vs.code tasks improved, added integration test (local) for dapr testing and integration test(containers) to test with released containers
removed --task argument checks for vs.code task scripts (run-*.sh)
removed specific versions from requirements*.txt
added custom volume for dbc2val (original config does not work with databroker)
fixed metadata bug in vdb_helper causing ValueErrors with VEHICLEDATABROKER_DAPR_APP_ID set
fixed docker container testing (integration_test/it-* scripts)
it-setup.sh start made lazy by default (if containers are started, do not re-start them)
added some timeout after container startup, needed for feedercan datapoint registration

[WIP]


Overlord incremental select worker strategy

Description
I was trying to use existing fillCapacity select worker strategy in an effort to implement MiddleManager autoscaling, but since MM instances that I use are created by k8s statefulsets it wasn't consistent enough. Proposed strategy fills capacity based on worker's hostname, with assumption that host is k8s FQDN - eg. druid-middlemanager-0.druid-middlemanager.druid-test.svc.cluster.local:8091.
Similar problem was discussed in https://github.com/apache/druid/issues/8801.
I'm not sure if this feature is desirable and this is my first java admission, so I'll take every bit of critique here. I can add tests if it will appear that this feature might be useful and we want to add it to codebase.
Key changed/added classes in this PR

FillIncrementallyWorkerSelectStrategy
FillIncrementallyWithAffinityWorkerSelectStrategy
FillIncrementallyWithCategorySpecWorkerSelectStrategy

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[x] been tested in a test Druid cluster.

thank you for your contribution. can you elaborate a bit on following

but since MM instances that I use are created by k8s statefulsets it wasn't consistent enough.

what is not consistent?

thank you for your contribution. can you elaborate a bit on following

but since MM instances that I use are created by k8s statefulsets it wasn't consistent enough.

what is not consistent?

For statefulset to scale down we need to take into consideration that:

When Pods are being deleted, they are terminated in reverse order, from {N-1..0}.

But with existing fillCapacity overlord strategy it doesn't care on what host ordinal index it accumulates tasks - it just checks which middlemanager instance has highest currCapacityUsed value. So in scenarios when there are multiple middlemanagers with same amount of currCapacityUsed it seems like this strategy picks middlemanagers at random.

@jakubmatyszewski, thank you for your contribution! Are you interested in reviving this PR?


[WIP] major refactoring of architecture

Q
A

Bug Fix?
yes

New Feature?
yes

BC Breaks?
yes!*

Deprecations?
no

Fixed Tickets
#26

License
MIT

This a major refactoring of the search system architecture.
In summary the following has changed;
ConditionOptimizers
Formatters are renamed to ConditionOptimizers, and have an optional priority for sorting.
Input processor changes
View to normalized value transformation now happens in the Input processor.
The SearchConditionBuilder does not use this technique.
Range bounds are now validated when processing, and not as an optional process afterwards.
To make InputProcessors reusable, configuration must be provided as first parameter of process().
And last, all processors when testing now share the same assertions to ensure a correct result.
Values
To make locking the SearchCondition state easier and enforce a stricter OOP architecture value objects are now immutable and marked final.
PatternMatch::getViewValue() is removed as this method doesn't make sense for a string-only value object.
ValueGroups no longer keep track of errors, but ask the ValuesBag's and nested groups instead.
Symfony validator
The Symfony validator extension is finally moved to its own repository and package.
As constraint validation is no longer needed for validating range-bounds.
TODO

[x] Add support for locking the ValuesGroup and FieldSet.
[ ] Check documentation for inconsistent usage of terms.
[x] Make tests for exporters uniform (same like input).
[ ] Look for missing tests and documentation.
[x] Allow removing errors from a ValuesBag.
[x] Run php-cs-fixer and remove deadcode

A new inspection was created.

A new inspection was created.

The inspection completed: 20 new issues, 74 updated code elements

The inspection completed: 20 new issues, 74 updated code elements

Move to back Doctrine lexer. https://github.com/doctrine/lexer/pull/6 now that modifiers are configurable.

A new inspection was created.

A new inspection was created.

The inspection completed: 22 new issues, 92 updated code elements

A new inspection was created.

The inspection completed: 22 new issues, 93 updated code elements

Missing tests for:

DuplicateRemover with ranges
InputProcessor

with invalid ranges (and make sure values are properly quoted)
with invalid value (failed transformation)
values merging/counting with alias-field
root-group logical OR

A new inspection was created.

The inspection completed: 36 new issues, 94 updated code elements

A new inspection was created.

The inspection completed: 22 new issues, 94 updated code elements

A new inspection was created.

The inspection completed: 21 new issues, 95 updated code elements

A new inspection was created.

The inspection completed: 21 new issues, 95 updated code elements

A new inspection was created.

The inspection completed: 12 new issues, 95 updated code elements

All this refactoring decreased the scrutinizer score, I need to look into making these parts better again.
But on the positive site, some code was actually improved :+1:

A new inspection was created.

The inspection completed: 7 new issues, 98 updated code elements

A new inspection was created.

The inspection completed: 7 new issues, 102 updated code elements

A new inspection was created.

I cleaned the commits up to be more precise and not contain unrelated changes.

The inspection completed: 34 new issues, 99 updated code elements

A new inspection was created.

The inspection completed: 34 new issues, 100 updated code elements

A new inspection was created.

The inspection completed: 34 new issues, 108 updated code elements

A new inspection was created.

The inspection completed: 15 new issues, 105 updated code elements

A new inspection was created.

The inspection completed: 13 new issues, 105 updated code elements

A new inspection was created.

The inspection completed: 12 new issues, 105 updated code elements

Merged


Unique iterator doesn't work with BoltDB

With BoltDB the graph.Value can be two different pointers with the same contents. This causes the unique iterator to fail.

Fixed with d42aa90c4c64542ff8bc982925c68bb802f9683f. But thanks anyway.


How do I set the selected values?

Lets say something happens that makes me want to set the selected values. How do I do that?

Please, check this function, if I understood you correctly, it might do what you request for: https://package.elm-lang.org/packages/inkuzmin/elm-multiselect/latest/Multiselect#populateValues


Will you run fletcher on aliyun F3 platform ?

This is the platform :https://www.aliyun.com/product/ecs/fpga?spm=5176.12418109.aliyun-offcial-website-list-react.1.307729b36PzL2b

As I'm not sure who you mean to ask this, I will answer: "Will Fletcher run on Aliyun F3?"
I would assume that it will, were someone to implement a platform library for it.
Fletcher is known to run on other UltraScale+ devices already, like the Xilinx Alveo cards and the AlphaData ADM-PCIE-9H7.
I can't read Chinese, but from the Google translate version of that page it seems like a platform similar to AWS EC2 F1?
If you would like to implement a platform implementation for it, you could take a look at the following link for inspiration:
https://github.com/abs-tudelft/fletcher-aws
Closing this assuming the question is answered.


[Bug]: Backup not working | connect: cannot assign requested address

Error Message and Logs
I have setup the DB backup to be on CloudFlare R2, it was working until I upgraded to v4.0.0-beta.349 and v4.0.0-beta.350. I'm getting this error:

mc:  Unable to initialize new alias from the provided credentials. Get "https://CLOUDFLARE-R2-LINK/probe-bsign-12birz1zdcqsil54zjl1kfa0v9cvvi/?location=": dial tcp [2a06:98c1:58::ec]:443: connect: cannot assign requested address.

Steps to Reproduce

Set up R2
Set up DB/Project auto backup
Back up the DB/Project

Example Repository URL
No response
Coolify Version
v4.0.0-beta.350
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
Alpine Linux
Additional Information
No response

This looks like a connection error to R2 on IPv6.
Can you please downgrade to the previous version where it worked and check if it working there?

It works on version v4.0.0-beta.348

I migrated to the latest version, here is the full error:

mc:  Failed to copy /data/coolify/backups/databases/root-team-0/postgres-db-ss8c0swo0o4c0gs4ww08kkos/pg-dump-workleen-1728691202.dmp. SigV2 authorization is not supported. Please use SigV4 instead.

@Hexacker We have upgraded a lot of packages in the latest version, is this error still happening for you?

Glad it works now. I will close this issue then.


Change grid to allow 5 connectors per row

@bumi wanted to have all on one row instead of having 1 connector by itself.

whoot. you're fast.
@johnsBeharry are you OK with that. and then we think what do do once we have Galoy in?

We won't have Galoy, as that's just the tech from what I understand. So it will be BitcoinBeach, and I think Bitcoin Jungle. @vindard is that correct? OR do those two apps share a database?
Either way I reckon that list will continue to grow, and I'd opt we go for a more App Store style card. Similar to what Auwal did

Check out the #434 branch. It is a Galoy connector.
I like the app store style cards...those can hold enough information.

but are you OK with this PR for this release?

So it will be BitcoinBeach, and I think Bitcoin Jungle

I guess tbd because the only thing that separates these two is the api link we point at.

Should we create a new issue to track the connector re-design?

:+1: please do.

On 25. Jan 2022, at 18:04, secondl1ght @.***> wrote:
Should we create a new issue to track the connector re-design?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.


Set default pydeck notebook width to 700px

For #2929

Background

Change List

Coverage remained the same at 79.908% when pulling 400e5171391a9ed7341d21ac5ae0f48151ce9f6b on ajd/pydeck-default-width into adbda77b25a3b8cd0af2974197b70bde934580f8 on master.


Add meta-kf6 and meta-qt6

Adds missing meta-kf6 and meta-qt6 layers (https://github.com/jhnc-oss/yocto-build/issues/68).

I'm merging here as it unbreaks main.


fix(spaceward): Add intents styles

Squashed and merged in https://github.com/alex-nax/wardenprotocol/pull/3/files#diff-32b5d335e3107e003482d13b97a48bb62d246109d7f76c87f9d395a88c81b0c1


Status bytes

Status A, Status B, and Status C are returning incorrect bytes. The bytes being returned are the 3 bytes before the checksum, 0d, 0a, 03. The status bytes are the 3 bytes, of a v4B return, just preceding the date/time bytes.

We are now taking them from the end of the 56 byte block before time.  If this is not correct please reopen.


Tokens not getting saved after refreshing

Describe the bug?
credential.refreshIfNeeded is not updating Credential.default after token gets refreshed successfully.
What is expected to happen?
up on token refresh Credential.default should gets updated with the latest token value
What is the actual behavior?
credential.refreshIfNeeded is not updating Credential.default after token gets refreshed successfully.
Reproduction Steps?
Login
Get the latest tokens
Close the app open again we will get new pair of tokens but Credential.default will have the previous token only.
Close the app again and open. Now Credential.default has the same OLD token only which is invalid by this time and user is stuck now because those tokens are expired.
Additional Information?
No response
SDK Version(s)
3.0.4
Build Information
No response

Hi @Sushree-nayak-2022, I found another edge-case where this scenario can occur, and am working through the code path to find the root cause. My guess is that it's a race-condition, and I'll update this ticket when I find out more.

Internal ref OKTA-562398

@Sushree-nayak-2022 I've just released okta-idx-swift 3.0.5 which explicitly pulls in the new SDK version as a dependency.

Thank you very much @mikenachbaur-okta . It is working as expected now.


Timeline doesn't show file saves changes

VSCode Version: 1.45.0
OS Version: Fedora 32

Steps to Reproduce:

Open a git workspace
Create a new file (without committing it)
Write some content in the file and save it
Edit the file (delete some of the content or add other content) and save it again
Open the Timeline view and instead of seeing the file history you will just see Uncommitted Changes
Open the More Actions ... menu
Only Include: Git History is available for selection

I don't know if this is the normal behavior, but because in the changelog it was documented that the timeline works also with file saves I'd expected it works and show changes for each saves.

The Timeline view is now out of preview and enabled by default. This is a unified view for visualizing time-series events (for example, Git commits, file saves, test runs, etc.) for a file.
Quote: https://code.visualstudio.com/updates/v1_44#_timeline-view

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

This is expected behavior as the timeline doesn't currently show file saves, just git history. The note in the release notes was about the things the timeline view could be used for visualizing.
Will treat this as the feature request for file save history.

/duplicate https://github.com/microsoft/vscode/issues/26339


Make distinction between root finding algorithm and function clearer

At the moment the x^3 - x^2 + 2 function and its derivative are hard to distinguish from the root finding algorithm. It would be nice if the function + derivative could be moved to their own code block
So it is clearer we are trying to find the root of the  x^3 - x^2 + 2 function with the Newton-Raphson algorithm.

And rename func to equation and derivFunc to derivative.

also rename find to solve.


Implement local channel/server credentials

Local channel/server credentials will be used by local UDS endpoints to create a secure channel. UDS connection is almost as secure as other types of connections secured by SSL/ALTS as it satisfies both peer authentication and channel confidentiality. With this implementation, call credentials can be sent over UDS channels which is not possible before since UDS channels were treated as insecure channels, and call credentials are not allowed to be sent over insecure channels. For comparison, gRPC Java/Go allow non-google secure tokens (call credentials) to be sent over insecure channels.

 All comments are addressed.   Comments are minor.

Reviewed 27 of 29 files at r1, 2 of 2 files at r2.
Review status: all files reviewed, 5 unresolved discussions (waiting on @jiangtaoli2016 and @yihuazhang)

src/core/lib/security/security_connector/local_security_connector.cc, line 37 at r2 (raw file):

#include "src/core/tsi/local_transport_security.h"

#define GRPC_UDS_URL_SCHEME "unix"

Please include a ':' at the end of this string, so that we don't accidentally match URIs whose schemes start with "unix" but contain additional characters (e.g., "unixa").

src/core/tsi/local_transport_security.cc, line 195 at r2 (raw file):

    handshaker_destroy,
    handshaker_next,
    nullptr, /* shutdown */

Shouldn't we have a shutdown method, even if it's a no-op?

test/core/end2end/fixtures/h2_local.cc, line 76 at r2 (raw file):

}

static int fail_server_auth_check(grpc_channel_args* server_args) {

This can return bool.
Also, please document.

Comments from Reviewable

Review status: all files reviewed, 5 unresolved discussions (waiting on @jiangtaoli2016 and @yihuazhang)

src/core/tsi/local_transport_security.cc, line 195 at r2 (raw file):
Previously, markdroth (Mark D. Roth) wrote…
Shouldn't we have a shutdown method, even if it's a no-op?

transport_security.cc will handle shutdown if it is no-op.
https://github.com/grpc/grpc/blob/master/src/core/tsi/transport_security.cc#L217

Comments from Reviewable

Review status: all files reviewed, 5 unresolved discussions (waiting on @jiangtaoli2016 and @markdroth)

src/core/lib/security/security_connector/local_security_connector.cc, line 37 at r2 (raw file):
Previously, markdroth (Mark D. Roth) wrote…
Please include a ':' at the end of this string, so that we don't accidentally match URIs whose schemes start with "unix" but contain additional characters (e.g., "unixa").

Done.

src/core/tsi/local_transport_security.cc, line 44 at r1 (raw file):
Previously, jiangtaoli2016 (Jiangtao Li) wrote…
I thought we agreed to delete peer_identity here. It is not used in local TSI.

Done.

src/core/tsi/local_transport_security.cc, line 195 at r2 (raw file):
Previously, jiangtaoli2016 (Jiangtao Li) wrote…
transport_security.cc will handle shutdown if it is no-op.
https://github.com/grpc/grpc/blob/master/src/core/tsi/transport_security.cc#L217

Thanks Jiangtao for the response. I was about to reply the same :)

test/core/end2end/fixtures/h2_local.cc, line 76 at r2 (raw file):
Previously, markdroth (Mark D. Roth) wrote…
This can return bool.
Also, please document.

Done.


how to deploy in vercel? could you please explain, thanks!

It would bit challenging to deploy directly to the vercel from what I know so far (please correct me if I am wrong), but streamlit probably is working on working with this type of issue. My suggestion is to migrate to typescritpt first. Thx!
-Nick


Nice to have a "streaming" diagnostics

Often you have different types of diagnostics in your LS.
Some are very quick, some are slow. However, the LSP supports only all diagnostics in a fell swoop. It's possible to send all diagnostics we have so far on each update, but it might quickly get slow, i.e. if we send addition of one diagnostic message, we could get to O(n^2).
Consider the following categories of diagnostics:

simple syntactic checks - almost immediate
reference resolve - a bit slower
complex inspection - might be very slow

Would be nice to have an ability to send diagnostics incrementally. The API might look like this:

startDiagnosticStream() - returns diagnostic stream id
sendDiagnostics(id, diagnostics) - the fastest diagnostics
sendDiagnostics(id, diagnostics) - less fast diagnostics
sendDiagnostics(id, diagnostics) - very slow diagnostics
closeDiagnosticStream() - closes everything
What do you think?

You can create different diagnostic stream by using the following approach

use the new pull model
use a different identifier for different streams.

I will close the issue. Please ping if this doesn't work for you.

@dbaeumer Thanks a lot for the pointers!

@dbaeumer Thanks a lot for the pointers!


[BUG]: Condition with Date as input in sql template fails

What version of drizzle-orm are you using?
0.30.1
Describe the Bug
With a column
timestamp('createdAt', { mode: 'date', withTimezone: true }).notNull(),

using a where condition like
gte(sql`(${product.createdAt} AT TIME ZONE ${timeZone})`, date)

we see the following error:
TypeError [ERR_INVALID_ARG_TYPE]: The "string" argument must be of type string or an instance of Buffer or ArrayBuffer. Received an instance of Date

Changing the the timestamp mode to string or date has not effect.
The only workaround is to convert the date to string when inserting in the where condition:
gte(sql`(${product.createdAt} AT TIME ZONE ${timeZone})`, date.toISOString())

This issue did not occur in 0.29.3 and below.
Expected behavior
The Date object should be accepted as input in the where clause.
Environment & setup
Postgres

Duplicate of #1993 ?

Closing in favor of #2009.

Closing in favor of #2009.

This was the issue #2009 itself. It was likely meant to be closed in favor of issue #1993.

This was the issue #2009 itself. It was likely meant to be closed in favor of issue #1993.

Correct. My bad.


context to pydantic_object

Feature request
while parsing LLM output with pydantic_object it would be nice to send to the parse function context object , like pydantic docs does here: https://docs.pydantic.dev/latest/usage/validators/#validation-context
Motivation
In order to do validation on LLM output rather than just parse it, in some cases to do the validation we need some context.
Your contribution
This change may require to support pydantic v2 not sure about backword capabilities.
in PydanticOutputParser.parse function insted of parse_obj we should use model_validate then we could send context object(it can be optional) to model_validate
from pydantic:
    @typing_extensions.deprecated(
        'The `parse_obj` method is deprecated; use `model_validate` instead.', category=PydanticDeprecatedSince20
    )
    def parse_obj(cls: type[Model], obj: Any) -> Model:  # noqa: D102
        warnings.warn('The `parse_obj` method is deprecated; use `model_validate` instead.', DeprecationWarning)
        return cls.model_validate(obj)

here is a workaround that worked for me
def parse_and_validate(pydantic_object, text: str, context=None):
    # taken from langchain. see issue: https://github.com/langchain-ai/langchain/issues/10034
    context = {} if context is None else None
    try:
        # Greedy search for 1st json candidate.
        match = re.search(
            r"\{.*\}", text.strip(), re.MULTILINE | re.IGNORECASE | re.DOTALL
        )
        json_str = ""
        if match:
            json_str = match.group()
        json_object = json.loads(json_str, strict=False)
        return pydantic_object.model_validate(json_object, context=context)

    except (json.JSONDecodeError, ValidationError) as e:
        name = pydantic_object.__name__
        msg = f"Failed to parse {name} from completion {text}. Got: {e}"
        raise OutputParserException(msg, llm_output=text)


[Actions] source env.sh to run run-fe-ut.sh

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

@mergify backport branch-2.4

@mergify backport branch-2.3

@mergify backport branch-2.2

@mergify backport branch-2.1

@mergify backport branch-2.0


parallelize detection and classification

@magland I am finding mountainsort5 to be excellent but a bit slow for long (2+ hour recordings) with many channels. Is there a reason why the detection and classification in scheme 2 cannot be threaded or parallelized with multiprocessing? If its simply the case that nobody has had the time to implement this, I will take a stab - but I just was wondering if there was something I was missing that prevented this from parallelization.

Hi @jbmelander
That would be great if you could take a stab at parallelizing this!
I should mention that many parts are already multi-threaded by numpy. So I think it would be important to identify the bottleneck steps before endeavoring to do this. I haven't thought too much about this, but I think it may be non-trivial to optimize.
But anything you can do would be great!

@magland Thanks for the feedback. There is at least one place that I think could benefit from spawning multiple processes, but you never know if the effort is futile until you try. I'll give it a shot over the next few days and report back here.

Sounds good!

I was able to speed up scheme2 significantly but in the end it looks like this had less to do with the parallelization and more to do with increasing the chunk size. Might I suggest an argument for scheme2 parameters that enables users to dictate this?

I was able to speed up scheme2 significantly but in the end it looks like this had less to do with the parallelization and more to do with increasing the chunk size. Might I suggest an argument for scheme2 parameters that enables users to dictate this? I can submit a PR if that helps.

Thanks @jbmelander it would be great if you could submit a PR for that. I think the default should be None, which would use the default calculated chunk size as is now. Also, the name of the parameter should be specific enough to distinguish it from the other durations (it's okay if it's a long parameter name). Feel free to suggest a change with the default as well -- I think this mostly concerns memory usage.

Sure, I'll submit a PR later today. It's useful for users with computers with significant RAM and agree the default should be what is currently set. Thanks! I'm still using MS on an experimental basis until I can solve a few more issues, but once I do, I'm happy to revisit the parallelization code, which did contribute to some speed improvements (but negligible compared to utilizing more memory).

Sounds great, and don't hesitate to star the repo if you find it useful. :)

:) Definitely. I will say that ms5 seems to give the best sorting results I've seen yet on my data - in terms of identifying real cells. I'm mainly trying to find parameters that doesn't miss smaller amplitude cells. On a separate note, do you know of any users that have applied MS4 or 5 to a Neuropixels dataset? I'd love to ask them what kinds of preprocessing and parameters were used. You can email me at<EMAIL_ADDRESS>if you want to continue this in private.

Glad it's working well for your data! I don't think anyone has been able to use MS4 for neuropixels data, since the implementation is not suited for very large channel counts. I haven't yet heard of any reports yet about folks using MS5 for NP, although I tested 128-channel subsets of NP datasets during development.

Well, if I have any success I will let you know :) Can you explain why it isn't suited for large channel counts?

There are some parts of the implementation (not inherent to the algorithm) which are quadratic time in the number of channels. Feel free to open an issue on that repo if you want to get into more detail.

I have a number of observations that are worth sharing - and that have ultimately led to some beautiful results on my NP datasets. Should I make separate issues for each before putting in a PR? Or just put them all in one PR and you can review from there?


Fixed typing exception throwing issues with JIT

As per #3027, some JIT issues were introduced in #2860 with typing, that weren't caught initially by tests.
This PR addresses the issues related to the densenet module.
This PR should close #3027

I'm still getting the following error with mypy:
torchvision/models/densenet.py:74: error: Name 'forward' already defined on line 70  [no-redef]
        @torch.jit._overload_method  # noqa: F811
         ^
torchvision/models/densenet.py:80: error: Name 'forward' already defined on line 70  [no-redef]
        def forward(self, input: Tensor) -> Tensor:  # noqa: F811
        ^

But I'm not very knowledgeable on JIT overlead decorator. Should we accept that some mypy tests fail? Or is there a way around this without having JIT issues? @fmassa @pmeier

Should we accept that some mypy tests fail?

Sort of. I suggest we simply exclude densenet.py from the type check for now and only re-enable it if the issue with torchscript is fixed.

Alright, it should be good to go now :ok_hand:

@frgfm Do you want to have another go at this as https://github.com/pytorch/pytorch/pull/51675 fixed the behavior of type: ignore comments for torch.jit?

@pmeier Thanks for the mention! Yes, it would be bad karma for me to leave this unfinished haha

@pmeier Here it is https://github.com/pytorch/vision/pull/3395 !


Enable pre and post content to be added

Reading the comments for #45 there's a suggestion to add prefix and postfix properties to enable to input content on either left (pre) or right (post) side of the input field.
To me this seems like a reasonable approach to enable users to add e.g. icons, buttons and similar things (that's often requested). This way we're not changing any code of the actual component.
A downside to this is of course that the CSS could break due to changing the HTML, but that is a risk I think is OK.
I'm interested in hearing people's opinions about this (especially @arqex's).

Hey @simeg
I had thought something even more generic. We can have a renderInput prop that accepts a function like:
function renderInput( inputProps ){
 return <input { ...inputProps } />
}

That would print the input just right like it is now, so it would be our default renderInput prop. But that is so flexible, people could customize it so much that they could render whatever they like, not even an input there.
We could do the same with the views, renderView would receive the type of the view to render:
function renderView( viewType, ViewComponent, viewProps ){
 return <ViewComponent { ...viewProps } />
}

Again the developer could completely customize every view (years, months, days, time), changing it slightly or building a completely new view.

This was implemented in #464, looks like we didn't have to wait for v3!


Incompatible with spree_auth_devise

Hello,
I'm attempting to use your alchemy_spree plugin with spree commerce version 2.3.2. There seems to be a incompatibility between alchemy-devise and spree_auth_devise. Alchemy-devise depends on devise "~> 3.0.0", and alchemy_spree depends on '~> 3.2.3', I can get bundler working correctly if I remove spree_auth_devise, but then I lose my /login route, and thus the login capability for my customers. Is there an easy way to get the two working together that I'm not seeing?
Thank you for your time!

Will be fixed with with #20

Update: Will be fixed with #21

Thanks to @lethjakman and @tomg65 for providing your code. I made a branch that should fix this hard coded alchemy-devise dependency.
Could you please test the allow-spree-user branch and report if this fixes your issues?
Thanks.

I put the allow-spree-user branch in a copy of my existing application. (Custom user authentication with devise and rolify -> model = User )
It works good - I found no issues until now. Spree administration works in alchemy backend
I had to implement an additional method to the user model:
  def admin?
    if has_role? :admin
      true
    else
      false
    end
  end

Best
Thomas

Thanks @tomg65
The method is an implementation error and will be fixed.
Thanks for trying out.


Request samples should not contain readOnly fields

How to use this? I still have those fields in my request samples.

@sixers Which version of ReDoc do you use?

@RomanGotsiy I'm using the latest one:
https://rebilly.github.io/ReDoc/releases/latest/redoc.min.js

@sixers can you share your spec or at least minimal reproducible sample?

Yes, sample Swagger YAML
# this is an example of the Uber API
# as a demonstration of an API spec in YAML
swagger: '2.0'
info:
  title: My API
  version: "1.0.0"
# the domain of the service
host: myhost
# array of all schemes that your API supports
schemes:
  - https
# will be prefixed to all paths
securityDefinitions:
  Authorization:
    type: apiKey
    name: Authorization
    in: header

basePath: /apis
produces:
  - application/json

paths:
  /api/v1/endpoint:
    post:
      parameters:
      - name: body
        in: body
        description: New object
        required: true
        schema:
          type: object
          properties:
            data:
              $ref: '#/definitions/MyObject'

      responses:
        200:
          description: created object
          schema:
            $ref: '#/definitions/MyObject'

definitions:
  MyObject:
    type: object
    properties:
      id:
        type: string
        example: 2a850262-afe9-4a13-b1fa-73cd25280d51
        readOnly: true
      user_id:
        type: string
        example: 3efca1ed-64bd-476a-ba30-aa1c57827d9c
        readOnly: true
      created_at:
        type: string
        format: date-time
        example: "2017-02-14T01:12:12Z"
        readOnly: true
      updated_at:
        type: string
        format: date-time
        example: "2017-02-15T06:41:25Z"
        readOnly: true
      published:
        type: boolean
        example: true
      conclusion:
        $ref: '#/definitions/Conclusion'

  Conclusion:
    type: object
    properties:
      link:
        type: string
        example: "asd"
      title:
        type: string
        example: "asd"

Request body schema contains only published and conclusion, request sample is as follows:
{
  "data": {
    "id": "2a850262-afe9-4a13-b1fa-73cd25280d51",
    "user_id": "3efca1ed-64bd-476a-ba30-aa1c57827d9c",
    "created_at": "2017-02-14T01:12:12Z",
    "updated_at": "2017-02-15T06:41:25Z",
    "published": true,
    "conclusion": {}
  }
}

I think it happens for nested objects, when I removed data envolope it looks good.

@sixers ahhh! Yes. I was able to reproduce it.
Thank you @sixers for your findings.
This fix for this issue will be shipped in the next release


ok so this is the error i get does anyone else have this issue

MCEdit version: 2.0.0alpha-767
Python version: 2.7.9 (default, Dec 10 2014, 12:24:55) [MSC v.1500 32 bit (Intel)]
Platform: win32
System version: Windows-post2012Server-6.3.9600
Processor: AMD64 Family 22 Model 48 Stepping 1, AuthenticAMD
Obtained a GL context with this format:
Valid: True
Version: 4.3
Hardware Accelerated: True
Depth buffer: True, 24
Double buffer: True
Driver info:
GL_VERSION: 4.3.12682 Compatibility Profile Context 13.302.1101.1004 (4, 3)
GL_VENDOR: 'ATI Technologies Inc.'
GL_RENDERER: 'AMD Radeon(TM) R2 Graphics'

Unhandled Exception
Traceback (most recent call last):
File "", line 207, in
File "", line 203, in main
File "out00-PYZ.pyz\mcedit2.util.profiler", line 66, in _wrapper (self is a Profiler)
File "", line 177, in startup
File "out00-PYZ.pyz\mcedit2.editorapp", line 282, in init (self is a MCEditApp)
File "out00-PYZ.pyz\mcedit2.worldlist", line 211, in init (self is a WorldListWidget)
File "out00-PYZ.pyz\mcedit2.worldlist", line 305, in reloadList (self is a WorldListWidget)
File "out00-PYZ.pyz\mcedit2.worldlist", line 342, in showWorld (self is a WorldListWidget)
File "out00-PYZ.pyz\mcedit2.worldview.minimap", line 152, in init (self is a MinimapWorldView)
File "out00-PYZ.pyz\mcedit2.worldview.worldview", line 149, in init (self is a MinimapWorldView)
File "out00-PYZ.pyz\mcedit2.worldview.worldview", line 190, in setDimension (self is a MinimapWorldView)
File "out00-PYZ.pyz\mcedit2.worldview.worldview", line 228, in createSceneGraph (self is a MinimapWorldView)
File "out00-PYZ.pyz\mcedit2.worldview.minimap", line 161, in createWorldScene (self is a MinimapWorldView)
File "out00-PYZ.pyz\mcedit2.worldview.worldview", line 224, in createWorldScene (self is a MinimapWorldView)
File "out00-PYZ.pyz\mcedit2.rendering.worldscene", line 222, in init (self is a WorldScene)
File "out00-PYZ.pyz\mcedit2.rendering.worldscene", line 76, in init (self is a SceneUpdateTask)
File "out00-PYZ.pyz\mceditlib.worldeditor", line 713, in blocktypes (self is a WorldEditorDimension)
File "out00-PYZ.pyz\mceditlib.worldeditor", line 415, in blocktypes (self is a WorldEditor)
AttributeError: 'AnvilWorldAdapter' object has no attribute 'blocktypes'

Fixed in 9ec45459c73215afbc8dd4ecf676a25e3e7afc0f

It's not the place to write MCEdit2 issues, just Unified ones but it seems like CodeWarrior fixed it in MCEdit2 already :)

Oh geez, I didn't even notice. :pig:


Coding best practices

This is a totally random list of things that are currently not best practice in this repository that we eventually want to resolve with #27  by fixing #17 .

The package has too many dependencies and some of them are difficult to install. Ideally we don't depend on users to install packages in a specific order nor having to use Conda

To resolve this we should double check every dependency. Do we really need it and can we get rid of it? Why are there installation issues for some of them on different platforms? Is there a way to get rid of all Conda dependencies? If not, can we create a conda package for sparcs?

The package should be trivially installable.

Get it on PyPI and Conda

Separate CLI and library. This brings a lot of complexity and weird code structure. Don't do it.

We create sparcs and sparcs-cli. Eventually both of them are on PyPI and Conda. The CLI package has sparcs and optionally click/whatever as dependencies. This helps us resolve weird code structure

Doc structure is off.

There should be a single docs folder with potentially a source folder. We'll just use what the scverse templates provides us with.

We'll use the Ruff + pre-commit setup of the scverse template. We'll use Google docstring style and configure Ruff to check it.

setup.py is legacy

Adopt the pyproject.toml setup from the template. The files defined in the MANIFEST should be picked up and shipped automatically. However, larger files should potentially be downloaded lazily on request.

The README is overwhelming.

Keep it minimal and remove the complex installation instructions. We'll just make it so easy so that this won't be necessary anymore Ha!

No unit tests.

We should have unit tests so that at least every public function is tested. An integration test for the whole pipeline would also be good.

documentation was automated and moved to correct location in #98


fix: Enable autostart for control, compute and infra VMs

Signed-off-by: Klaus Smolin<EMAIL_ADDRESS>

This is great @smolin-de! Could you add a commit to this PR for the bootstrap too? It doesn't have autostart on either and we might as well get them all done at the same time.

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


Unmarshalling error logs contain Slack user data

Describe the bug
When the dependency (nlopes/slack) can't marshall a message event, an error is logged.
The log contains information about the event, which contains much too much information on the Slack user.
To Reproduce
Steps to reproduce the behavior:

Go to Slack
Click on the huddle icon; do it in a really public channel just to annoy lots of other people with your test.  Some of them might join to say hi!  Please tell them I said: "hello".
Dump the error logs of your botKube pod (k logs botkube-blah)
See error

Slack unmarshalling error: RTM Error: Received unmapped event "user_huddle_changed": {all my info about me}

Expected behavior
Don't log things that don't matter.
Screenshots
n/a
Additional context
nlopes/slack has moved to slack-go/slack, and you should probably also update your dependency, but the problem with using this tool is that it listens to every possible slack event.
(ref: https://github.com/slack-go/slack/blob/0e14c9d4a15c91e845bc371fe91ee58763a3813a/websocket_managed_conn.go#L494)
Which means BotKube listens to every event
Which makes this tool a really frightening kind of slack spy program that's running in my kubernetes cluster and outputting employee information in error logs.

Thanks for reporting this. We will prioritize this for the upcoming release

Hi @richgerrard,
Thanks for report. I've checked that, and you're absolutely right—it is caused by the slack-go/slack library. I've prepared a PR with fix directly to the repo: https://github.com/slack-go/slack/pull/1067
Also, to avoid being blocked, I've created a PR with a workaround on Botkube side: https://github.com/infracloudio/botkube/pull/583. I hope it will be merged soon 🤞
Cheers!


Area Devices + Major code refactor

This PR introduces Area Devices, a feature requested by the community that makes each area have it's own devices and allowing users to manage all Magic Areas entities for that area through that device.

This will enable a future change in the Meta-Areas behavior where current it tracks it's child areas' entities instead of the "magic entities". Now that MA is able to know which entities are theirs (by checking the device info) we can track them directly.
Secondly, there's a MAJOR code refactor going on. Up to the current release, Magic Areas code has been piled up with different levels of knowledge (from myself) since "I have no idea what I'm doing" to today's "I have some idea what I'm doing". Given that, the code wasn't particularly efficient nor abiding by the components standards, a lot of stuff was being done dumbly and I wanted to take some time to go through all of it and make it better. This refactor will (hopefully) have no impact in users but it is increasing the quality of the integration by a lot.
I've tried to break down the code into more reusable components and things should be more consistent and stable throughout.

Bug with the features entity loading is fixed. Was a race condition when the area was not ready. Wrapped all entity creation into a listener that listens for their area "area ready event" then proceeds.
This bug was introduced when I moved the platform setup out of the area initialization code (it should be where the whole integration is setup). Fortunately that's sorted out, on to the next things!
I'll spend tonight going over whatever else is can be optimized and merge so I can work on the other bugfixes.

All features tested over the past 2 days, everything seems to be working. Merging.


Multi-Agent with Visual Studio Test task is not working

We are using TFS 2017.2

using Visual studio Test Version 2.*
when running tests on single agent, tests runs with pass or failed results.
when using Multi-Agent, we get System.IndexOutOfRangeException

##[error]ExecutionTaskStateModel.ExecuteTask : An exception occured while executing the task. So the task is not executed. System.IndexOutOfRangeException: Index was outside the bounds of the array.
at Minimatch.Minimatcher.MatchOne(IList1 file, IList1 pattern, Boolean partial)
at Minimatch.Minimatcher.Match(String input, Boolean partial)
at Minimatch.Minimatcher.IsMatch(String input)
at System.Linq.Enumerable.WhereEnumerableIterator1.MoveNext() at Microsoft.VisualStudio.TestService.VstestAdapter.MiniMatchSourceFilter.FilterSources() in E:\v2.0\A5\_work\1\s\Ta\Agent\VS.TestService.VstestAdapter\MiniMatchSourceFilter.cs:line 68 at Microsoft.VisualStudio.TestService.VstestAdapter.RunDetailsStore.NormalizeSettings() in E:\v2.0\A5\_work\1\s\Ta\Agent\VS.TestService.VstestAdapter\SliceDetailsStore.cs:line 154 at Microsoft.VisualStudio.TestService.VstestAdapter.RunDetailsStore.set_CurrentSliceDetails(TestAutomationRunSlice value) in E:\v2.0\A5\_work\1\s\Ta\Agent\VS.TestService.VstestAdapter\SliceDetailsStore.cs:line 57 at Microsoft.VisualStudio.TestService.VstestAdapter.VstestAdapter.ExecuteAsync(TestAutomationRunSlice sliceDetails, CancellationToken cancellationToken, IProgress1 progress) in E:\v2.0\A5_work\1\s\Ta\Agent\VS.TestService.VstestAdapter\VstestAdapter.cs:line 52
at Microsoft.VisualStudio.TestService.AgentExecutionHost.ExecutionTaskStateModel.GetExecutionTask(CancellationToken cancellationToken) in E:\v2.0\A5_work\1\s\Ta\Agent\VS.TestService.AgentExecutionHost\ExecutionTaskStateModel.cs:line 351
at Microsoft.VisualStudio.TestService.AgentExecutionHost.ExecutionTaskStateModel.ExecuteTask(CancellationToken cancellationToken) in E:\v2.0\A5_work\1\s\Ta\Agent\VS.TestService.AgentExecutionHost\ExecutionTaskStateModel.cs:line 189
##[error]ExecutionTaskStateModel.AbortSlice : System.IndexOutOfRangeException: Index was outside the bounds of the array.

@ggenetec  Thanks for reaching out. Two points:

Currently we do not support UNC paths for "Search Folder" input. The error you see comes up because the filter on task fails to process a UNC path correctly. Please use a File copy task before VsTest task to copy the files to a local agent location and then use that path in your task.

To utilize the multi-agent scenario in VsTest task, please update to Update 3 of TFS 2017. You will an Advanced Options in the task post that which lets you customize on how you wish to distribute your tests.

@ggenetec  Are you unblocked?

I changed the path a simple name of a dll in "test assemblies" but it is still not working :(
"folder' does not contain any assemblies that match the specified criteria: 'Genetec.Media.Tests.dll'.
I checked  and the dll is there. I tried with 'Genetec.Media.Tests*.dll' still same error

Did you change your input to "Search Folder" field? As I mentioned earlier, UNC paths are not supported in that field.

changing to Agent.ROOTDIRECTORY solved the problem. Thanks

Thanks for reporting back.


[hi] Add  content/hi/docs/tasks/tools/included/optional-kubectl-confi…

Added one file
localize optional-kubectl-configs-bash-linux.md to Hindi.
issue

hey @anubha-v-ardhan can you review this PR ?

@Garima-Negi, @anubha-v-ardhan   I have updated the file, can you review this file?

@divya-mohan0209 , @Garima-Negi , @anubha-v-ardhan I have updated the file, Can I squash the commit?

@divya-mohan0209 can you merge this PR? I have updated this file.

/label tide/merge-method-squash
/assign
@ashish-jaiswar If you know how to squash commits down to 1 commit, Can you do it? Its okay if you don't too.

@anubha-v-ardhan  I will squash the commit and merge it into a single commit.

@anubha-v-ardhan can you squash the commit?

/lgtm
/approve
Thank you @ashish-jaiswar


Multithreaded LSP Wrapper

Adds a multithreaded mode to LSP wrapper.
Overview:

A new interface makes asynchronous message processing possible. You instantiate multithreaded LSPWrapper with a lambda. Then, you call send() methods that send a set of messages to the server. Responses to these messages are passed to the lambda.

This functionality is not available in single-threaded mode. LSPWrapper will throw an exception if you try to use send in single-threaded mode.
We could hack in a codepath for this to work in single-threaded mode if it seems beneficial.

The original interface continues to work in multithreaded mode. You can give LSPWrapper a set of messages, and it will return all messages created in response to that message.

Implementation involves a new sorbet/fence message that waits for all prior messages to clear out.
This is primarily useful for sharing test code between single/multithreaded use cases. I see no good reason for a multithreaded user to not use the new async interface.

LSPWrapper now contains factory functions rather than multiple constructors. This change let me mark many fields on LSPWrapper as const. I couldn't make the constructor private, though, as unique_ptr needs to be able to construct it.

Motivation

Required to easily test the preemptive slow path end-to-end.
Test plan

See included automated test.

This is now ready to review!

https://github.com/sorbet/sorbet/pull/2200#discussion_r348722470
https://github.com/sorbet/sorbet/pull/2200#discussion_r348722632
are the ones I'd prefer addressed, all the others are minor

I believe I've addressed all feedback.
There are now subclasses of LSPWrapper: One is single-threaded, the other is multithreaded. They have different APIs to reflect that reality.
The multithreaded wrapper no longer accepts a lambda. Instead, it exposes a blocking read method. As a bonus, this lets me use the same LSPOutput class for both versions of LSPWrapper, and avoids the state machine I had earlier.

I'm fixing the build error right now; ~WorkerPool assumes that logger is still alive, but it gets destructed with LSPConfiguration.

Rebased on master; this should be ready to review now!


Inquiry Regarding Bandwidth Limitation in Polaris Project

Hy There,
I am reaching out to inquire about a potential limitation I've encountered while using the Polaris project for streaming audio services.
Upon scanning my files, I've noticed that the maximum throughput I can achieve is 100Mb/s. This limitation has posed some challenges for my intended use case. I am curious to know if it's feasible to modify or remove this bandwidth restriction within the Polaris project.
Could you kindly provide some insights into whether such modifications are possible or if there are alternative approaches to achieving higher throughput within the project?
Thank you for your attention to this matter. I look forward to your response and appreciate your efforts in developing the Polaris project.
Best regards,
Simis

Related discussions:
https://github.com/agersant/polaris/pull/96
https://github.com/agersant/polaris/pull/86 (older)
Polaris indexing is already faster than its competitors (last I checked) and currently has no known way of becoming faster.


fix: broken link to kind install guide

Is there a related GitHub Issue?
No
What is this change about?
Fixing a broken link in the docs

Hi @kirederik
Thanks for the fix! Could you please sign the CLA so that we can merge it?

@danail-branekov done!


swiftmailer TLSv1.1 and TLSv1.2 support

Does swiftmailer currently support tlsv1.1 and tlsv1.2? If not, what is the ETA for this feature?
Thanks

PCI compliance is forcing us to turn off TLSv1.0, initially it seems like swiftmailer does not support TLSv1.2 Any confirmation on this?

Swiftmailer does not implement TLS on its own. So this depends on what your PHP version supports (which actually depends on the openssl version against which PHP is linked AFAIK)

PHP 5.6.20 (cli) (built: Apr  2 2016 08:28:23)
OpenSSL 1.0.1e-fips 11 Feb 2013
Both PHP and OpenSSL support TLSv1.2
Swiftmailer starts throwing:
[19-May-2016 08:58:01 America/Chicago] PHP Fatal error:  Uncaught exception 'Swift_TransportException' with message 'Connection could not be established with host localhost [Connection refused #111]' in ...
I'll see if I can create a complete minimal verifiable example.

Can you paste the full message, i.e. including the stack trace ?

[19-May-2016 08:56:02 America/Chicago] PHP Fatal error:  Uncaught exception 'Swift_TransportException' with message 'Connection could not be established with host localhost [Connection refused #111]' in /site/vendor/swiftmailer/swiftmailer/lib/classes/Swift/Transport/StreamBuffer.php:265
Stack trace:
#0 /site/vendor/swiftmailer/swiftmailer/lib/classes/Swift/Transport/StreamBuffer.php(62): Swift_Transport_StreamBuffer->_establishSocketConnection()
#1 /site/vendor/swiftmailer/swiftmailer/lib/classes/Swift/Transport/AbstractSmtpTransport.php(113): Swift_Transport_StreamBuffer->initialize(Array)
#2 /site/vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mailer.php(79): Swift_Transport_AbstractSmtpTransport->start()
#3 /site/public_html/includes/mail.php(67): Swift_Mailer->send(Object(Swift_Message))
#4 /site/includes/common.php(215): in /site/vendor/swiftmailer/swiftmailer/lib/classes/Swift/Transport/StreamBuffer.php on line 265

i had forgotten about this and after forcing tlsv1.2 in postfix config of mailserver, swiftmailer can't send mail. "connection reset by peer" b/c the protocol swiftmailer is trying to use isn't supported on mail server. i had to reduce supported tls version to 1.0 before swiftmailer could mail again. i'm using archlinux with php php 7.0.7-1 and openssl 1.0.2.h-1 on both the web server and mail server so i doubt my php and openssl versions are the issue. i also use tlsv1.2 with my sites' certs on said webserver and have for a while.  My servers supported tlsv1.2 when i posted this issue over a year ago.
thanks

Temporarily try changing https://github.com/swiftmailer/swiftmailer/blob/5.x/lib/classes/Swift/Transport/StreamBuffer.php#L95 STREAM_CRYPTO_METHOD_TLS_CLIENT to STREAM_CRYPTO_METHOD_TLSv1_2_CLIENT.
http://php.net/manual/en/function.stream-socket-enable-crypto.php#119122
Now, be careful because since PHP 5.6.7, STREAM_CRYPTO_METHOD_TLS_CLIENT (same for _SERVER) no longer means any tls version but tls 1.0 only (for "backward compatibility"...).

@sstok thanks a lot! that worked. I guess i'll leave this issue open until this workaraound is not necessary.

Great 👍 A simple way to fix this is using STREAM_CRYPTO_METHOD_SSLv23_CLIENT. But that will make the system vulnerable for downgrade attacks (forced lower version by a man in the middle).
We can change the tls option to accept eg. false (default), true (STREAM_CRYPTO_METHOD_TLSv1_0_CLIENT|STREAM_CRYPTO_METHOD_TLSv1_1_CLIENT|STREAM_CRYPTO_METHOD_TLSv1_2_CLIENT when available) or a specific value (the constant value).

Working here as well. Thank you for the update.

@sstok the STREAM_CRYPTO_METHOD_SSLv23_CLIENT  option leaves a bad taste in the mouth, even though sslv2 and 3 are disabled on my mailserver. Being able to choose tls protocol instead of generic "tls" option sounds like the best option (out of the two) to me. A "use highest protocol supported by openssl and lock" option would be future proof, though maybe more difficult to implement.

As Swift_Transport_StreamBuffer::startTLS() is hard coded to use STREAM_CRYPTO_METHOD_TLS_CLIENT, can anyone suggest a neat way to override it to use STREAM_CRYPTO_METHOD_TLSv1_2_CLIENT, without editing its source file?  PCI compliance requires that only TLS v1.2 be used from July onwards.

I couldn't figure out a way to do it. I edited the source. Kind of bad because I have to remember to either not update that package or reapply the fix. IMO, tlsv1_2 should be the new default, there shouldn't be much reason not to use tls v1_2 at this point in time.
I'd love to hear how your PCI 3.1 compliance is going. If you'd be willing to send me an email it's my username at gmail. Ours has been difficult at best.

Overriding the transport, and updating the dependency map
Le 17 juin 2016 8:35 PM, "mkaatman"<EMAIL_ADDRESS>a écrit :

I couldn't figure out a way to do it. I edited the source. Kind of bad
because I have to remember to either not update that package or reapply the
fix. IMO, tlsv1_2 should be the new default, there shouldn't be much reason
not to use tls v1_2 at this point in time.
I'd love to hear how your PCI 3.1 compliance is going. If you'd be willing
to send me an email it's my username at gmail. Ours has been difficult at
best.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/swiftmailer/swiftmailer/issues/598#issuecomment-226847492,
or mute the thread
https://github.com/notifications/unsubscribe/AADCfcXa2SDkpmnvJSq1acyhGcoOljrxks5qMukBgaJpZM4D8Bfq
.

Isn't overriding the transport, and updating the dependency map just as bad as editing the source, in the sense that you've still got to manually merge any changes in the master code to yours?  (It's also using a sledgehammer to crack a nut!)
Fortunately, my code is running on the same linux host as the mail server, so I've worked around this problem by switching to the sendmail transport.  The disadvantage, of course, is not getting the same exception feedback.

Any hope of some change on this? Our app needs to be able to make SMTP connections to a wide variety of servers (our clients enter their SMTP connection info, and our app sends mail using their accounts), so we need it to try TLSv1.2, but fall back to TLSv1.0 if that's all the client's SMTP server has. But I can see others wanting to lock in TLSv1.2... so I guess @sstok second idea sounds the best?

@Rotzbua, @sstok, How will the code in #935 play out? If the server supports 1.2, it will use that? Else, it will use 1.1, and if it doesn't support 1.1 it will support 1.0? Or will it default to 1.0?

If the server supports 1.2, it will use that? Else, it will use 1.1, and if it doesn't support 1.1 it will support 1.0?
@malikkotob yes, it would use the highest possible tls version.

Great, thank you for the clarification @Rotzbua!

When will this fix be merged into the master?  Thanks.

While we wait for this fix to be merged, is there anyway to explicitly specify TLS version in the config (i.e. I provide host, username, password, TLS version, etc.)? Reading the doc and browsing the code tells me no, but, maybe there is a way that I'm not seeing.
Side note, the manual fix of editing source code isn't really a fix for containerized apps. I have to add custom code that, after composer pulls in the proper packages, edits that source code to make this update. That's just bad.

@fabpot Thanks for the update! While you're at it, do you want to add TLSv1.3? https://github.com/swiftmailer/swiftmailer/commit/4c4b333c9164b74aaceb7ff9bef2d55f7a5514b8


Feature Request: Better errors when refresh token fails

I currently utilize login2fa to generate refresh tokens to authenticate with ALKS.  My persistent grant seems to expire every 30 days, which is fine and a good security practice.  What is not fine is how alks behaves when my refresh token ceases to work.
 $ alks sessions open
ALKS v3.0.5
Error: Failed

Of course, after a bit of flailing I finally remember the --verbose flag.
 $ alks sessions open --verbose
ALKS v3.0.5
[sessions-open]: getting developer
[sessions-open]: getting auth
[developer]: checking for access token
[sessions-open]: getting accounts
[developer]: retreiving alks account
[developer]: using cached auth object
[api:injectAuth]: getting refresh token
Error: Failed

At this point running alks developer login2fa and going through the process of regenerating my refresh token resolves the issue.

Just ran into this myself. I was on ALKS 3.0.3, thought I just needed to upgrade and did so. When I got the error again I came here and after some reading I figured it out but it was more painful than desired :/

This has been fixed in this PR: https://github.com/Cox-Automotive/alks-cli/pull/159


Update http4s-async-http-client, ... to 0.23.5

Updates

org.http4s:http4s-async-http-client
org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-client
org.http4s:http4s-core
org.http4s:http4s-dsl

from 0.23.1 to 0.23.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Superseded by #926.


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
d9eb236f17212f4e93cf7b633082b921475fac8d
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Update cpal dependency to 0.7.0

A first step towards moving to cpal 0.8 and fixing #160.
I tried to test the changes by compiling the crates that depdend on rodio >= 0.6, as listed on crates.io.
They are,

ggez: compiles fine, tests pass, but examples keep failing on my machine due to #160.
impose: compiles, but has no tests
audact: compiles, tests pass
macromusic: compiles, tests dont pass neither before or after my changes.

Thank You!


Should add Tidying up after Pull Requests section

I like to suggest after a Pull Request has been merged delete the old branch should add as the last step of tutorial. as explained here The GitHub Blog.

Fixed in #5397


Why YouTubePlayerView Eating so many Memory?

Could anyone help me with this? How could I reduce the memory for this?

I'm unable to reproduce this. Closing for now. Please reopen if you can provide concrete steps to reproduce the issue.

I have faced the same issue. With every video open +20mb of memory are used.
Steps to reproduce
youTube player as property
      self.player = YouTubePlayerView(frame: self.view.frame)
      self.player.delegate = self
      self.player.loadVideoID("wQg3bXrVLtg")

Open a video. Close it. Open up another one (in different VC). Check memory usage.
After youTube view is dismissed memory usage remains the same (does not decrease).

Here is an instruments file, if it can help.
https://www.dropbox.com/s/jevnzb584wwq8ij/Instruments.trace.zip?dl=0

playerView.clear() before playing a new video, this solved the problem for me, but Battery usage remains high and very high when loading video


TypeError: value.toArrayBuffer is not a function

I'm running into the exception that toArrayBuffer is not a method of value on line #68. I wonder if this code in #L67-L68 is correct and what the purpose of it is? I really have no clue, for now I just disabled #L67-L69 and "It works for me".

Hm, the point of that line is that IndexedDB supposedly accepts Uint8Array objects, but not "Buffer" objects (which are a shim for the Node.js "Buffer" object). If you can provide a test case to reproduce and let us know which browser it failed in, that would be helpful. :)

I've written a new approach where native types are supported if asBuffer is false. I'm interested in feedback on it, see #46.


Revert "Clearly callout the behavior for listening ports."

Reverts grpc/grpc#19529

Thank you for the quick approval! :-)


Run merge moderation script as part of the production release of OS Hub

Execute a bulk merge via scripts in preparation for Beta launch.
Target merge facilities and source facility(ies) to be merged into the target will be identified and cluster IDs will be added to each merge grouping.
File containing merge information will be added to the ticket prior to submission for sprint planning.

@mariel-oar This ticket is updated for sprint planning. Merge cluster listing

This ticket should be sequenced such that it happens on or before with the OAR database migration to OS Hub (said another way: this ticket is for merging duplicates that exist in the current OAR database that we want to resolve in the version of the database that gets copied to OS Hub).

I labeled this client action needed because the OS Hub Team will need to provided the merge list just priory to launch.

@jwalgran, might this be a good issue for @KlausGPaul to be involved in? The idea being that we could run these more often if it's something our team can control.

We'd have datasets for Indian companies (some 1.2 million), and a synthetic dataset of Turkish addresses (the addresses seem to be real, but names are synthetic), 100K or 1000K. I will make a plot of the spatial distributions, then we see if this buys us something.

Merge moderation listing

@obrienad, we have been working with the understanding that this ticket would be picked up for next sprint. Based on that, @vrwOAR is working to finalize the list by Sept 29 (Thursday). Is this information sufficient to remove the CAN label?

Great, thanks! Let's keep it on and @vrwOAR can reply in thread when it's good to go and I'll remove the label then!

Hi all - @obrienad this ticket is ready to be picked up. Thanks

The merge script has been run successfully. Halfway through attempting to run it the first time, an API Block was applied; after assigning myself a grace limit, I was able to complete the script successfully. Note for the future that we should ensure the token used for this script has unlimited API requests assigned.
On the first attempt, the following errors ocurred:
('Status:', 400, 'Content:', '{"target":["Facility IN2021271XS1AKT does not exist."]}', 'Target ID:', 'IN2021271XS1AKT', 'Merge ID:', 'IN2019181A16JXD')
('Status:', 400, 'Content:', '{"merge":["Facility TR2019083ETMJ3E does not exist."]}', 'Target ID:', 'TR20193173JXFWT', 'Merge ID:', 'TR2019083ETMJ3E')
('Status:', 400, 'Content:', '{"merge":["Facility IN20211597CNK4A does not exist."]}', 'Target ID:', 'IN202218625HBK2', 'Merge ID:', 'IN20211597CNK4A')
('Status:', 400, 'Content:', '{"merge":["Facility IN2019181Q2G965 does not exist."]}', 'Target ID:', 'IN2021271G3BRNH', 'Merge ID:', 'IN2019181Q2G965')
('Status:', 400, 'Content:', '{"merge":["Facility TR2022027WJTDER does not exist."]}', 'Target ID:', 'TR2019083QKH765', 'Merge ID:', 'TR2022027WJTDER')
('Status:', 400, 'Content:', '{"merge":["Facility TR202030139YZF6 does not exist."]}', 'Target ID:', 'TR2019083QKH765', 'Merge ID:', 'TR202030139YZF6')
('Status:', 400, 'Content:', '{"merge":["Facility TR20203512KKF8J does not exist."]}', 'Target ID:', 'TR2019083T3GQ78', 'Merge ID:', 'TR20203512KKF8J')
('Status:', 400, 'Content:', '{"merge":["Facility TR20200983VJ4Q6 does not exist."]}', 'Target ID:', 'TR2019083T3GQ78', 'Merge ID:', 'TR20200983VJ4Q6')
('Status:', 400, 'Content:', '{"merge":["Facility TR2019178KXQERF does not exist."]}', 'Target ID:', 'TR2019083XMDKT8', 'Merge ID:', 'TR2019178KXQERF')
('Status:', 400, 'Content:', '{"merge":["Facility TR2021159TEKK2A does not exist."]}', 'Target ID:', 'TR2019083XMDKT8', 'Merge ID:', 'TR2021159TEKK2A')
('Status:', 400, 'Content:', '{"target":["Facility IN2022007MDS525 does not exist."]}', 'Target ID:', 'IN2022007MDS525', 'Merge ID:', 'IN2019181FFP3JP')
('Status:', 400, 'Content:', '{"target":["Facility IN2022007MDS525 does not exist."],"merge":["Facility IN2022063T4Y9MZ does not exist."]}', 'Target ID:', 'IN2022007MDS525', 'Merge ID:', 'IN2022063T4Y9MZ')
('Status:', 400, 'Content:', '{"merge":["Facility IN2022007F2WBB7 does not exist."]}', 'Target ID:', 'IN2021342TFV77Q', 'Merge ID:', 'IN2022007F2WBB7')
('Status:', 400, 'Content:', '{"merge":["Facility IN2022012MZMV8A does not exist."]}', 'Target ID:', 'IN2021342TFV77Q', 'Merge ID:', 'IN2022012MZMV8A')
('Status:', 400, 'Content:', '{"merge":["Facility TR2020070NV9G82 does not exist."]}', 'Target ID:', 'TR20203513VKQ72', 'Merge ID:', 'TR2020070NV9G82')
('Status:', 400, 'Content:', '{"merge":["Facility TR2021182X2Z952 does not exist."]}', 'Target ID:', 'TR2020204WNSQZG', 'Merge ID:', 'TR2021182X2Z952')
('Status:', 400, 'Content:', '{"target":["Facility TR2019259DCC209 does not exist."]}', 'Target ID:', 'TR2019259DCC209', 'Merge ID:', 'TR2019172ZYZNYB')
('Status:', 400, 'Content:', '{"target":["Facility TR2019259TQN4M0 does not exist."]}', 'Target ID:', 'TR2019259TQN4M0', 'Merge ID:', 'TR20210861BXSJD')
('Status:', 400, 'Content:', '{"target":["Facility TR2019259TQN4M0 does not exist."]}', 'Target ID:', 'TR2019259TQN4M0', 'Merge ID:', 'TR2019083G5XWS4')
('Status:', 400, 'Content:', '{"merge":["Facility IN2020085EHVSJD does not exist."]}', 'Target ID:', 'IN2021162RH4X6N', 'Merge ID:', 'IN2020085EHVSJD')
('Status:', 400, 'Content:', '{"merge":["Facility TR2020191QKY8MS does not exist."]}', 'Target ID:', 'TR2019172ZVAW3Z', 'Merge ID:', 'TR2020191QKY8MS')
('Status:', 400, 'Content:', '{"merge":["Facility TR2019259QK9CTV does not exist."]}', 'Target ID:', 'TR2020053K07BF6', 'Merge ID:', 'TR2019259QK9CTV')
('Status:', 400, 'Content:', '{"merge":["Facility IN2020239FZYMHT does not exist."]}', 'Target ID:', 'IN2021336X8JTXP', 'Merge ID:', 'IN2020239FZYMHT')
('Status:', 400, 'Content:', '{"target":["Facility IN2020053842XZD does not exist."]}', 'Target ID:', 'IN2020053842XZD', 'Merge ID:', 'IN2021340TRKHJB')
('Status:', 400, 'Content:', '{"merge":["Facility BD2020021177SB0 does not exist."]}', 'Target ID:', 'BD20211599W2XNK', 'Merge ID:', 'BD2020021177SB0')
('Status:', 400, 'Content:', '{"target":["Facility TR20191725RKFC7 does not exist."]}', 'Target ID:', 'TR20191725RKFC7', 'Merge ID:', 'TR2020330841RSE')
('Status:', 400, 'Content:', '{"target":["Facility TR20191723D9751 does not exist."]}', 'Target ID:', 'TR20191723D9751', 'Merge ID:', 'TR2021336AFDR2Q')
('Status:', 400, 'Content:', '{"target":["Facility TR20191723D9751 does not exist."]}', 'Target ID:', 'TR20191723D9751', 'Merge ID:', 'TR20191761HSFA4')
('Status:', 400, 'Content:', '{"merge":["Facility IN20190833ATEHM does not exist."]}', 'Target ID:', 'IN2020053QN9B47', 'Merge ID:', 'IN20190833ATEHM')
('Status:', 400, 'Content:', '{"target":["Facility IN20202160SCSQS does not exist."]}', 'Target ID:', 'IN20202160SCSQS', 'Merge ID:', 'IN2022063J25YMH')
('Status:', 400, 'Content:', '{"merge":["Facility IN2019083R8ETYM does not exist."]}', 'Target ID:', 'IN202001054KF07', 'Merge ID:', 'IN2019083R8ETYM')
('Status:', 400, 'Content:', '{"target":["Facility IN2021102HB5PA0 does not exist."],"merge":["Facility IN20213154ZFVTC does not exist."]}', 'Target ID:', 'IN2021102HB5PA0', 'Merge ID:', 'IN20213154ZFVTC')

On the second attempt, the facilities which had been merged successfully all returned 'does not exist' errors (as expected), so the error response was mostly too noisy to be useful. These are the errors that were thrown in the second attempt that were not 'does not exist' errors:
('Status:', 400, 'Content:', '{"target":["Cannot be the same as merge."],"merge":["Cannot be the same as target."]}', 'Target ID:', 'CN20190830X6JJF', 'Merge ID:', 'CN20190830X6JJF')
('Status:', 400, 'Content:', '{"target":["Cannot be the same as merge."],"merge":["Cannot be the same as target."]}', 'Target ID:', 'IN2019172EWGPVR', 'Merge ID:', 'IN2019176HE1SCP')


Adding File

@helenchanars file/lib/domains/ca/arsdayschool.txt: Your request has an incorrect file path format, please ensure the request follows     the repository rules set out at the bottom of this page and change     it accordingly or start a new one.     If you are unable to make a pull request with the instructions provided, please send an email to<EMAIL_ADDRESS>mentioning the URL of this request in the message.

@helenchanars I renamed file/lib/domains/ca/arsdayschool.txt to lib/domains/ca/arsdayschool.txt.
For the record, domain ownership proof is at http://www.arsarmenianschool.ca/index.php/news-a-calendar/ars-news/103-back-to-school-letter-2012-2013


Allow injecting EmbeddingStore with the type parameter

If this makes sense I'll also apply it in the (upcoming) Pinecone store processor

Does this also need a docs update?

The examples in the docs inject the concrete implementation class to avoid ambiguity (well, they will after https://github.com/quarkiverse/quarkus-langchain4j/pull/64) so not necessarily


Add option to invert Y axis

Adds an option that inverts the Y-Axis in gameplay, without affecting menus and other parts where the player would use the control stick. Tested with all 3 vehicles, and both rails and 360.

Looks nice to me but can you change the var name to yAxisMult and make it a bool?

I messed up the commits accidentally. Wanted to sync the main branch in my fork and ended up syncing it into the invert-y branch. Shouldn't break anything as far as I can tell though


1.7.1

Made debounce equal number of components in form json.

The 500 is the delay for the debounce. What is the reason for this change? Is it taking too long before triggering change? Having a few milliseconds may not be long enough.

Yes, it is. After updated to react 16.x, it is taking too long before triggering email input change. I sent video demonstration on your email address. The video shows huge the delay, in the console I call the console.log to each field changes.

Would it be better to decrease the delay to 100ms or something? The component length seems rather arbitrary for the delay.

Maybe 50ms?

What will i have to do? New commit with current version or something else?

Hi Randall. Can you push my changes? My boss requires to close task but i
cant't because need to change delay. Merge plz...
вт, 31 дек. 2019 г. в 16:59, Randall Knutson<EMAIL_ADDRESS>
Would it be better to decrease the delay to 100ms or something? The
component length seems rather arbitrary for the delay.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/formio/react-formio/pull/278?email_source=notifications&email_token=ANTLFZBGQ6YF4P7FZB4Z4WTQ3NF5HA5CNFSM4J4HZS22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH4HZFA#issuecomment-569932948,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANTLFZAVMUO4LKEZB7OVKWDQ3NF5HANCNFSM4J4HZS2Q
.

This is published as version 1.7.2 since there was already a 1.7.1


Improved comment

Comment suggested argc was being discarded.. then it's used in the very next line

@kierenj  Thanks man :+1:


Fix "invalid integer value" for ruby 3.2

closes #104

@mico can you please help fix the merge conflict and merge it? Thanks.

Done.


MSAL Guard should properly support hash routing and non-root base urls

This PR addresses the issues raised in #1452:

Properly construct the redirectStartUrl if hash routing is used.
Properly construct the redirectStartUri if the application uses a different root url than /

It also fixes the behavior in msal when redirectStartUri itself contains a hash. If this occurs, msal will process the response hash immediately, then redirect to the redirectStartUri. This is to prevent both hash values from being concatenated together, which confuses both msal and the Angular router.
Unfortunately, this may lead to redirectStartUri not receiving the handleRedirectCallback response (this would happen if redirectUri and redirectStartUri are not the same base url, e.g. /redirect and /profile#edit). I wasn't able to find a graceful way to handle this scenario, given the changes that were made in #1358.
Fixes:

#1447
#1422

Another thing to consider; In the case where a 2nd redirect is needed and a hash routing strategy is used, the 2nd redirect will throw an error due to no state being present in the hash. This is something we will want to also address in conjunction with #1413 where state is especially important in determining whether to process the hash at all

Another thing to consider; In the case where a 2nd redirect is needed and a hash routing strategy is used, the 2nd redirect will throw an error due to no state being present in the hash

I don't think that is true here, as we are clearing the response hash, processing it, then navigating to the original url (which has the Angular specific hash values, which won't pass MSAL's checks, which look for known properties in the hash).

Another thing to consider; In the case where a 2nd redirect is needed and a hash routing strategy is used, the 2nd redirect will throw an error due to no state being present in the hash

I don't think that is true here, as we are clearing the response hash, processing it, then navigating to the original url (which has the Angular specific hash values, which won't pass MSAL's checks, which look for known properties in the hash).

I stand corrected. I didn't dig into the urlContainsHash function and assumed it was just a basic check whether or not the url contains a hash but it is in fact inspecting the hash for known properties. So this should be fine. I take back my original concern


Strict equal (===) mistakenly scoped as assignment operator

The strict equal is currently being scoped as keyword.operator.assignment rather than keyword.operator.relational like other comparison operators.

@seanCodes Want to PR? :D

Fix was easy enough :) PRs are always welcome!

Thanks @simonzack, I'll submit a PR next time. :)


Breaking new substrate API

It is possible that I broke something.
If you find something please paste it here so I can fix it as soon as possible
Known bugs:

Identicon is making a ton of bugs: https://github.com/polkadot-js/ui/pull/668

commits:

:package: updated @polkadot packages
:zap: api url minin that knows current websocket
:coffin: :mute: legacyyy API code
:zap: api constants use agnostic impl kodadot/packages#31
:snake: BSX create collection is using new api
:snake: BSX create token is using new api
:memo: new usage of sub-api
:memo: useApi
:zap: useApiMixin
:zap: upgraded metatxmixin to new api
:snake: available action API::new
:memo: useApi
:zap: you wont believe that offers useApi]
:dolphin: subscription returned to BSX gallery item
:zap: onApiConnect from sub-api
:construction: new apin in components/bsx/specific/DestroyCollection.vue
:construction: new apin in components/rmrk/Create/Admin/AdminPanel.vue
:construction: new apin in components/rmrk/Create/Admin/SendHandler.vue
:construction: new apin in components/rmrk/Create/CreateCollection.vue
:construction: new apin in components/rmrk/Create/CreateToken.vue
:construction: new apin in components/rmrk/Create/CreativeMint.vue
:construction: new apin in components/rmrk/Create/SimpleMint.vue
:construction: new apin in components/rmrk/Gallery/Appreciation.vue
:construction: new apin in components/rmrk/Gallery/AvailableActions.vue
:construction: new apin in components/rmrk/Pack/PackSaver.vue
:construction: new apin in components/rmrk/Profile/IdentityForm.vue
:construction: new apin in components/shared/format/Identity.vue
:construction: new apin in components/unique/Collection/Item/CollectionItem.vue
:construction: new apin in components/unique/Collection/Item/TransferCollectionModal.vue
:construction: new apin in components/unique/Create/CreateToken.vue
:construction: new apin in components/unique/Gallery/Item/AvailableActions.vue
:construction: new apin in components/unique/Gallery/Item/DangerModal.vue
:construction: new apin in components/unique/Gallery/Item/GalleryItem.vue
:construction: new apin in components/unique/apiConstants.ts
:construction: new apin in pages/teleport.vue
:construction: new apin in pages/transfer.vue
:top: kodadot1-sub-api-0.1.0-alpha.1.tgz
:broom: cleaned up configs
:zap: chain mixin is using config instead of reading data from chain
:bug: since 5b955bab7425d21c7c9dc7b6bf6399d5f6548514
:zap: wallet modal uses chain mixin
:zap: ss58 of the chain
:zap: vuex plugin to change ss58 of the logged user
:mute: chainPropListOf
:package: sub-api-0.1.0-alpha.1

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[x] Feature
[x] Refactoring

What's new?

[x] PR closes #3364
[x] PR closes #3312

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[ ] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

I went to https://deploy-preview-3500--koda-nuxt.netlify.app/bsx/gallery/515729876-6
connected wallet
I see UNIT

It seems needs to bump polkadot/util?

I achieved this couple of times in console by flipping between RMRK (kusama) / BSX, or more specifically following these steps:

log in with bsx wallet
switch to rmrk
switch back to bsx
go to explore
switch to rmrk
redirected back to home page, console looks like this

Gonna resolve the bugs tomorrow

It seems needs to bump polkadot/util?

https://github.com/polkadot-js/extension/issues/1113

I achieved this couple of times in console by flipping between RMRK (kusama) / BSX, or more specifically following these steps:

Thanks for the good eye! It was fixed in kodadot/packages@986447e4ac881c8d243510794c2f3c3749f65675

@petersopko @Jarsen136 all issues should be fixed now.
If you will be able to crack it again I would be super happy

Let's make a release https://github.com/kodadot/nft-gallery/releases/tag/untagged-08c2fed23fc63c6921d9 and then we can merge this 👌

Tried flipping between rmrk / bsx again, landed at bsx. My balance seems to be zero everywhere I look (even after refreshing at BSX couple times. No visible issue in the console for me:

links to check:

https://deploy-preview-3500--koda-nuxt.netlify.app/bsx/create
https://deploy-preview-3500--koda-nuxt.netlify.app/transfer

Tried flipping between rmrk / bsx again, landed at bsx. My balance seems to be zero everywhere I look

d57f2c4a6de364f2812c9ccf934744086f63e0b5

classic RMRK mint repeatedly fails

Would be nice to push this to main 👀

I think It would be nice to have a last check and merge cc @petersopko

tested on https://deploy-preview-3500--koda-nuxt.netlify.app/transfer
I could see an incorrect balance after I switch from BSX to RMRK

After I reload the page, it works well as usual.

@Jarsen136 fixed in 19b1984

@Jarsen136 fixed in 19b1984

Seems that I could still reproduce it and there is no any error related in the console.

Let's open an issue for that and merge this