unable to row validate hash with Teradata

I get the following error when error when trying to run a validate row using hash all.
 [Teradata Database] [Error 3706] Syntax error: Data Type "concat__all" does not match a Defined Type name.
Here is an example of the command
data-validation validate row -sc td-conn -tc bq-conn -tbls expdw_ete_dwh.DWH_D_ORG_LOC_GRP_LU=EXPDW_POC_DWH.DWH_D_ORG_LOC_GRP_LU --primary-keys LOC_GRP_KEY --hash LOC_GRP_KEY
If i try to specify columns (rather than --hash '*') by creating a configuration file and editing it, I get the same error.
part of hte error shows the created SQL statement which gets run in Teradata:
SELECT LOC_GRP_KEY, LOC_GRP_ID, LOC_GRP_DESC, RCD_INS_TS, RCD_UPD_TS, RCD_CLOSE_FLG, RCD_CLOSE_DT, cast__LOC_GRP_KEY, ifnull__cast__LOC_GRP_KEY, rstrip__ifnull__cast__LOC_GRP_KEY, upper__rstrip__ifnull__cast__LOC_GRP_KEY, concat__all,        hash_sha256("concat__all") AS "hash__all" FROM (   SELECT LOC_GRP_KEY, LOC_GRP_ID, LOC_GRP_DESC, RCD_INS_TS, RCD_UPD_TS, RCD_CLOSE_FLG, RCD_CLOSE_DT, cast__LOC_GRP_KEY, ifnull__cast__LOC_GRP_KEY, rstrip__ifnull__cast__LOC_GRP_KEY, upper__rstrip__ifnull__cast__LOC_GRP_KEY,          ARRAY_TO_STRING(["upper__rstrip__ifnull__cast__LOC_GRP_KEY"], ',') AS "concat__all"   FROM (     SELECT LOC_GRP_KEY, LOC_GRP_ID, LOC_GRP_DESC, RCD_INS_TS, RCD_UPD_TS, RCD_CLOSE_FLG, RCD_CLOSE_DT, cast__LOC_GRP_KEY, ifnull__cast__LOC_GRP_KEY, rstrip__ifnull__cast__LOC_GRP_KEY,            upper("rstrip__ifnull__cast__LOC_GRP_KEY") AS "upper__rstrip__ifnull__cast__LOC_GRP_KEY"     FROM (       SELECT LOC_GRP_KEY, LOC_GRP_ID, LOC_GRP_DESC, RCD_INS_TS, RCD_UPD_TS, RCD_CLOSE_FLG, RCD_CLOSE_DT, cast__LOC_GRP_KEY, ifnull__cast__LOC_GRP_KEY,              rtrim("ifnull__cast__LOC_GRP_KEY") AS "rstrip__ifnull__cast__LOC_GRP_KEY"       FROM (         SELECT LOC_GRP_KEY, LOC_GRP_ID, LOC_GRP_DESC, RCD_INS_TS, RCD_UPD_TS, RCD_CLOSE_FLG, RCD_CLOSE_DT, cast__LOC_GRP_KEY,                coalesce("cast__LOC_GRP_KEY", 'DEFAULT_REPLACEMENT_STRING') AS "ifnull__cast__LOC_GRP_KEY"         FROM (           SELECT LOC_GRP_KEY, LOC_GRP_ID, LOC_GRP_DESC, RCD_INS_TS, RCD_UPD_TS, RCD_CLOSE_FLG, RCD_CLOSE_DT,                  CAST("LOC_GRP_KEY" AS STRING) AS "cast__LOC_GRP_KEY"           FROM expdw_ete_dwh.DWH_D_ORG_LOC_GRP_LU         ) t4       ) t3     ) t2   ) t1 ) t0
and so I ran this in Teradata studio and it appears the error is twofold:

Teradata doesn't have a ARRAY_TO_STRING() function, so that fails. A working alternative would be concat() i think.
Teradata doesn't have a hash_sha256() function. A working alternative would be hashrow()

I hope this makes it easy to implement and fix and release.

Thanks for reaching out @ajg12345 it seems like there are two separate issues here

The incorrect concatenate function is being called on Teradata. We should be able to override the default implementation without too much issue

The hash_sha256() replacement is a bit more involved. Teradata does not support a native sha256 hashing algorithm so a udf needs to be deployed and registered under that namespace. I see this is missing in the documentation and I will resolve. Using the native hashrow() function would be the preferred approach but as far as I can tell the algorithm is proprietary and/or undocumented. Since this tool is designed for heterogeneous database comparisons we need the same algorithm implemented on all platforms.

I will update the docs and look at the concatenate issue. Thanks for raising and please let me know if you have any questions.

We are running into the same issue with ARRAY_TO_STRING function on teradata.

the follow changes are implemented on teradata to get the query working
hash_sha256("concat__all") -> hash_sha256(concat__all)
ARRAY_AGG("upper__rstrip__ifnull__cast ->  concat("upper__rstrip__ifnull__cast

I have a PR open for this that is under review. Once that is cleared this will get merged into develop

sounds good! thank you for the update

Updated to resolve the issue.

@renzokuken  and so this is part of a PR now?

Yes this should be resolved with PR #457 and #482 . Thanks!


Window is too small when it starts

Window is extremely small and jumps in size when first resized.  This is true for all images.

Resolved with GUI update.


Added LiveForm Jetpack Module

Added LiveForm for Jetpack Compose. This module was extract from the Live View Native Client Jetpack Repo in the PR 478.

@bcardarella the Jetpack library is being published on Jitpack. But seems like the CI is failing in the Swift part... @carson-katri can you help us with this?

Waiting for https://github.com/liveview-native/liveview-native-live-form/pull/29 be merged...

@nelson-glauber it's merged 👍


Ventura RC version bumps

Will bump Monterey to Xcode 14.1 after stable release.

Review period skipped due to critical label.


Add: pagination support for issue requests

A pr to support pagination when we fetch issues.
Working locally on summary values for total submissions over time, etc

this pr is actually working really well now! its "failing" because of a decrease in test coverage not because of failing tests.

I am going to merge this. I think the tests for these api calls are more involved and i'd like to do it in a separate pr. i'll open an issue for that so we have the functionality for this now.

Closes #140


Device did not appear with the intermediate firmware error

I am getting the 'Device did not appear with the intermediate firmware' error while attempting to flash Smart Breaker Switch with model no scw nf101. I am using Raspbbery Pi 4 with Raspian buster. Below is the link of the device
https://www.aliexpress.com/item/32961129650.html
smarthack-mqtt.log
smarthack-psk.log
smarthack-udp.log
smarthack-web.log
smarthack-wifi.log
Attached are my logs. I appreciate the assistance.

How did you get to the logs.  Unfortunately, I'm very new with Linux...


Bump and tweak Alpine image version

Bump to building with golang 1.20.5 to placate CVE scanners
Tweak to use a less complex image version so that Renovate will pick this up in the future.

I highly doubt your changes are linked to this but still, one check of the eventchecker failed, could this be an arm64 specific flake?:
I0629 14:37:39.978785   10001 dumpinfo.go:48] "Dumping test data" dir="/tmp/tetragon.e2e.TestLabelsDemoApp.991831803"
E0629 14:37:40.890404   10001 dumpinfo.go:68] "Failed to extract json events" err="failed to run kubectl cp -c tetragon kube-system/tetragon-97chk:/var/run/cilium/tetragon/tetragon.log /tmp/tetragon.e2e.TestLabelsDemoApp.991831803/tetragon.tetragon-97chk.json: exit status 1"
E0629 14:37:40.914667   10001 portforward.go:407] an error occurred forwarding 8118 -> 8118: error forwarding port 8118 to pod d765e8e9e461f27416bda416d97a04ba068fa63061d7b82d8e71189e6cdc7ee0, uid : failed to execute portforward in network namespace "host": failed to connect to localhost:8118 inside namespace "d765e8e9e461f27416bda416d97a04ba068fa63061d7b82d8e71189e6cdc7ee0", IPv4: dial tcp4 <IP_ADDRESS>:8118: connect: connection refused IPv6 dial tcp6 [::1]:8118: connect: connection refused
E0629 14:37:40.927435   10001 portforward.go:233] lost connection to pod
I0629 14:37:40.961888   10001 dumpinfo.go:299] contacting gops agentaddr<IP_ADDRESS>:8118
E0629 14:37:40.963795   10001 dumpinfo.go:303] "failed to dump heap profile" err="failed to dump heap profile: dial tcp <IP_ADDRESS>:8118: connect: connection refused" addr="<IP_ADDRESS>:8118"
E0629 14:37:41.622452   10001 dumpinfo.go:71] "Failed to extract previous tetragon logs" err="failed to run kubectl logs -c tetragon -n kube-system tetragon-97chk --previous: exit status 1"
E0629 14:37:43.553726   10001 dumpinfo.go:371] "failed to run bpftool" err="failed to run bpftool: unable to upgrade connection: container not found (\"tetragon\")"

Same failure on second run, is it bad luck or a real failure??

@mtardy really not sure on this one – I can't think of an obvious reason why those errors would be arm-specific. I'll try a third time, will do some experimenting if it fails again.

Ah tragic, I'm not able to re-run these tests myself. I'll leave it up to you to decide how you would like to proceed.

@mtardy really not sure on this one – I can't think of an obvious reason why those errors would be arm-specific. I'll try a third time, will do some experimenting if it fails again.

Yes indeed it makes no sense but it's again another flake to investigate... I just restarted the workflow.


File localpath staging conflict

I haven't tried to make this happen, but noting as I think it is likely.
When multiple sites are involved and staging sets localpath, then when File objects are shared between sites - for example, Globus staging the same file to two different sites, or staging a file out from one site and then in to the next - then I think that the File.localpath attribute may be improperly shared between those sites.
File.localpath represents the local path of the file on a particular site (perhaps, eventually, even for a particular task, if per-task workdirs come into existence #89) - but staging code mutates File.localpath on the submit side without regard for any other conflcting staging action that would want a different values in that attribute.
Probably the correct way to deal with this is to replace no-local-path File objects with modified copes, rather than mutating them.

This also happens even in the single site case in the case that the staging provider runs in such a way that it modifies the submit side File object - so that the conflict occurs between the local_path set for the remote site, and the submit side request for the submit-side path.
I've encountered this for real in a (post-#1201, in development) locally-running staging provider, and will put in a PR to test for this happening.

This was fixed by #1272


add es6 version

Not sure if a PR is appropriate...but had to tweak a couple things to get this working with ES6 classes

Sorry for the late response. Honestly I feel like a fork is more appropriate (since this basically implements the component in a different language/react api target).
react-countdown-timer-es6 maybe?

@rloomba  spent couple of hours changing it over to es6 syntax with classes found your es6 code afterwards. :)


Share matrix CSS theme with community

Hello,
This is not really an issue, however i wanted to share this if anyone interested.
Feel free to close it, if it is posted at the wrong place.
It is a matrix style skin for authelia.
authelia_matrix_dist_final.zip
It looks like this:

I modified the css, and for the totp register i had to add a small white border for it to be recognised by the qr scanners. Also added a js file for the matrix style, which is a bit CPU heavy, beware :)
Hope you like it, and I can share sources if you like.

That should scare away any unwanted visitors 😃
I'm going to have to give it a go when I get 10mins.

Hello @BankaiNoJutsu , this theme looks great! We should definitely find a way to share it with the community. I think we should add a "themes" directory with subdirectories for each new theme.
There is now way to easily switch theme dynamically in Authelia yet but we might find a way in a near future. In the meantime let's create a directory called "themes", a "matrix" subdirectory holding the css and let's create a tuto explaining how to update the theme in a README under "themes".
Sound good? Can you do the PR?

Hello @BankaiNoJutsu , would you like to contribute with your theme?

Hey @clems4ever, good idea to already create the folder structure. I am finishing some minor things on the theme, and will create a pull request once all is ready.
Also doing the readme for quick change of themes. Sorry it may take a while, quite busy those days.


feat(datasets): Add ToyDatasetEyeLink

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (08d93dd) 100.00% compared to head (4320aac) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #343   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           53        54    +1
  Lines         1803      1848   +45
  Branches       386       358   -28
=========================================
+ Hits          1803      1848   +45

Impacted Files
Coverage Δ

src/pymovements/plotting/tsplot.py
100.00% <ø> (ø)

src/pymovements/dataset/dataset_files.py
100.00% <100.00%> (ø)

src/pymovements/datasets/__init__.py
100.00% <100.00%> (ø)

src/pymovements/datasets/toy_dataset_eyelink.py
100.00% <100.00%> (ø)

src/pymovements/utils/parsing.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add WebGL fallback

WebGPU is nowhere nearly as widely adopted as webgl 1 and 2. This means that maybe 15% of people out there with internet connection can run the project. I myself am using the latest version of Firefox and I can't run it!
There should be some sort of fallback to webgl when webgpu isn't detected. Otherwise, the project won't be useable for at least a solid 5+ years until webGPU adoption grows enough

Unfirtunately WebGL's programming model is limited and cannot enable some fo the optimizations needed.
Good news is that Firefox is also working to land WebGPU https://bugzilla.mozilla.org/show_bug.cgi?id=1841346

WebGL is not worth at all for simple reasons as culling.
WEBGPU is the way to go and will be greater when we move out the webassembly 4GB cage.
Totally out of topic. @tqchen did you had a chance to review TVM with wgpu?

WebGL is not worth at all for simple reasons like culling. WEBGPU is the way to go and will be greater when we move out the webassembly 4GB cage.
Totally out of topic. @tqchen did you had a chance to review TVM with wgpu?

ah yes, nothing like buying a car you can't drive anywhere because the roads dont support it.
Makes for a beautiful paper weight

WebGPU isn't supported by most browsers

Can you name a few?
Chromium based browsers supports WebGPU right now. To name a few Chrome, Opera, Edge and of course and the most important Electron... When I say supported is that it comes enabled by default not  having to enable the flag. Safari and others supports it with the flag.
Everybody is moving from WebGL because of (sight) many issues, only float16, culling... almost every single framework do not support all the operators thats why wasm has succeded, and of course the effort... Its simply not worth at this point.

@DavidGOrtega I meant by most browser versions being currently used. As you can see
https://caniuse.com/webgpu
https://www.stetic.com/market-share/browser/
Only about ~30% of chrome users, ~7% Edge, ~1% of opera, 0% firefox and safari can use webGPU currently. And thats based on the statics of that one website, some others point to even LESS.
Now consider this https://www.khronos.org/blog/webgl-2-achieves-pervasive-support-from-all-major-web-browsers being exactly what I said(took webgl 2 about 5 years to be supported on all browsers, and for most versions being actively used)

@peq42 stats improves when you watch desktop, for mobile you have to go straight with TVM mobile.

And this discussion is a bit of futile, not because Webgpu is supported or not, Its just because WebGL has so many issues that failed to deliver, its simply not the right tool to do DL in the browser. Thats why many operators are not implemented in many frameworks.
I any case, TVM is open source, you can of course contribute and enjoy the unpleasant WebGL way 🙂


PHP Version 7.3.9 with sqlserver 2012 connection

i following this link:
https://www.youtube.com/watch?v=hxfdbnpOqSI
PHP Version 7.3.9
C:\xampp\php\ext\php_pdo_sqlsrv_73_ts.dll
C:\xampp\php\php.ini    extension=php_pdo_sqlsrv_73_ts.dll
http://localhost/phpinfo.php
PDO
PDO drivers  = mysql, sqlite
can you help me?

hi @Romit4309 I only watched the first minute of the video. It was about some old releases.
Depending on your choice (thread safe or non-thread safe? 64 bit or 32 bit?), the latest for PHP 7.3 is 7.3.17, and you can download one of the four zip files from https://windows.php.net/download
Make sure you download the right PHP zip file and the corresponding pdo_sqlsrv driver.
Also, in your C:\xampp\php\php.ini you may need to specify the extension_dir
For example, you can do that:
extension_dir=ext\
extension=php_pdo_sqlsrv_73_ts.dll

driver downloaded this link :
https://docs.microsoft.com/en-us/sql/connect/php/download-drivers-php-sql-server?view=sql-server-ver15
PHP Version update 7.3.9 to 7.4.4
already set
extension_dir = "C:\xampp\php\ext"

Hii,Jenny Tam, yitam,my issues solve.
Thanks for help.
use PHP Version 7.1.13
SQL Server driver downloaded this link :
https://www.microsoft.com/en-us/download/details.aspx?id=55642
following this link:
https://www.youtube.com/watch?v=w3L3G-TMmJA&t=192s


The function "minimum" may have the wrong name or wrong parameters, depending on which function in _prototype it corresponds to.

In Pyclesperanto_prototype, we had three functions:

minimum_box(source, destination, radius_x, radius_y, radius_z)
minimum_sphere(source, destination, radius_x, radius_y, radius_z)
minimum_images(source1, source2, destination)

And an alias:

minimum which points at minimum_images.

In pyclesperanto/clic, there is minimum, with these parameters:

minimum(input_image, output_image, radius_x, radius_y, radius_z)

Does it make sense to rename it? E.g. "minimum_filter" would match scipy.ndimage.minimum_filter

The same issue existis with maximum obviously.

ok for renaming to minimum_filter (same for max)


Fix present method on macos

cc @EPashkin @sdroege

@sdroege Examples failed build second time. Any idea?
error: custom attribute panicked
  --> src/bin/gio_futures_await.rs:23:1
   |
23 | #[async]
   | ^^^^^^^^
   |
   = help: message: unsupported character _

Bug in nightly of today, see https://github.com/rust-lang-nursery/futures-rs/issues/992

Can you explain why this is necessary and correct? And please link to the relevant GTK bug in Bugzilla, or create one if none exists. This looks very ugly and needs at least an explanation in the code

@sdroege So we can ignore it now, Thanks.

Can you explain why this is necessary and correct? And please link to the relevant GTK bug in Bugzilla, or create one if none exists. This looks very ugly and needs at least an explanation in the code

Because on OSX, this function doesn't bring the window to the foreground whereas it should. As simple as that. I found the bug in bugzilla but can't it again...

please find it again so we can properly link it here :) It's important so that we can remove this ugly workaround once we depend on a new enough GTK, and so our futures selfes can later understand how this disaster of code happened

For reference, bug is referenced here.


Fix typo in Js Objects in Rust

Summary
Fixed the following typo:

Because of the "stack-y" nature of this sceheme...

to

Because of the "stack-y" nature of this scheme...

Fixes #1293

Thanks!


Tweak Dockerfile for official image.

Replaced ADD with COPY and ENTRYPOINT to CMD per best practices article.
Yes, this means that you'd need to type an extra "swarm" on the CLI:
docker run swarm swarm <args>.  Removed unnecessary Godeps install,
and added a .dockerignore to shave off ~23M.
Signed-off-by: Peter Salvatore<EMAIL_ADDRESS>

I don't like the change ENTRYPOINT to CMD

Commented on the reasoning behind it over in the official-images PR.  Happy to update the PR with whatever gets decided over there to be consistent.

OKOK, the Docker Gods have spoken.  I've updated the PR to use ENTRYPOINT/CMD.  :cat:

This PR isn't for official image changes any longer, since that is shaping up to be an independent thing, so feel free to merge or close and pick out whatever you like. ;-)

@psftw thanks!


absorb rust-lorawan lorawan-device's trait impl locally

Typically, a trait implementation is not committed to the upstream repo. Was there a specific reason this one was?

This should probably be done on embassy-rs/lora-phy and not on this fork

Yea I didn't realize it was a fork. I'm just going to maintain this impl in the rust-lorawan crate for now.


samples: net: lwm2m: use real temperature values instead of dummy

This changes the use of a static and fixed temperature value to using a
read_callback which will either return the previous dummy value or
gather real temperature data from a supported sensor (for now only
FXOS8700 which is available e.g. on a FRDM-K64F).
Signed-off-by: Daniel Egger<EMAIL_ADDRESS>

Not sure how to fix the shippable error. IS_ENABLED is preferred to checking #defines but for this still barfs for DT symbols. Of course I could also combine both and define the DT label if it is not defined but yuck...

IS_ENABLED is preferred to checking #defines but for this still barfs for DT symbols. Of course I could also combine both and define the DT label if it is not defined but yuck...

IS_ENABLED() is indeed preferred but in some cases is not possible to do it like that. Just do
#if defined(CONFIG_FXOS8700_TEMP)
		dev = device_get_binding(DT_NXP_FXOS8700_0_LABEL);
#endif

in this case.

@therealprof love the change -- I need a few days before my time opens up to review (and 1.14 LTS rolls out)
In our sources we make all of the HW temp device names the same via these settings:
https://github.com/foundriesio/dm-lwm2m/blob/master/Kconfig#L72
https://github.com/foundriesio/dm-lwm2m/blob/master/boards/frdm_k64f.overlay#L11
and then generically use them:
https://github.com/foundriesio/dm-lwm2m/blob/master/src/main.c#L28
https://github.com/foundriesio/dm-lwm2m/blob/master/src/main.c#L82

In our sources we make all of the HW temp device names the same via these settings:

So you're creating custom labels using the device tree and use them in the application? I'm not sure that approach really scales for general samples in Zephyr as we probably don't want to create a separate overlay per potential hardware device which could be used which also this gets very messy if a user attaches an external sensor to a generic board.

Haven't forgotten about this!  Just got back from a small Spring vacation, clearing out the work items from 1.14 and I'll get to this soon (along with some new client changes as well).

@mike-scott Gentle monthly ping 😅


Update use cases

Routine use cases updated, based on https://github.com/apache/incubator-skywalking/issues/443
Also, add a new list of all use cases.

Coverage increased (+0.004%) to 24.586% when pulling 23292c5f819f8f8be5bf8c1e524a1f0364c59cc8 on users into 6753478a7769f22ff82aaedbdec79da3a38058fc on master.


getRtlScrollAxisType detects incorrect RtlScrollAxisType for chrome

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
I recently upgraded to Angular ~16.2.10, and noticed that cdk-virtual-scroll-viewport is not working in RTL mode for horizontal scroll. Initial data is rendered but then on scroll subsequent records are not rendered.
On investigations I came to know that getRtlScrollAxisType is returning 2 (Inverted) instead of 1 (Negated). If I make it return 1 during debug, things start to work fine.
Reproduction
StackBlitz link: https://stackblitz.com/edit/angular-yehn5j?file=src%2Fapp%2Fcdk-platform-overview-example.html
Steps to reproduce:
Notice that at the bottom of information, getRtlScrollAxisType is 2

Expected Behavior
getRtlScrollAxisType() should return 1 for chrome.
Actual Behavior
getRtlScrollAxisType() returns 2 for chrome
Environment

Angular: ~16.2.10
CDK/Material: ~16.2.10
Browser(s): Chrome 119.0.6045.160, Chrome 119.0.6045.200
Operating System (e.g. Windows, macOS, Ubuntu): Windows 10 Pro

Hmm, it does return 1 as the scroll axis type for me with Chrome on Mac. This might be specific to Chrome on Windows. The logic for calculating the axis type doesn't look at the user agent or anything, it just experimentally determines by adding a hidden scrollable element, so I'm not really sure why it would get it wrong only for Chrome on Windows.
Also nothing has changed about the code recently, so I'm not sure why it would have changed only when updating to Angular 16.2.10.

You are right, nothing changed code wise, as far as I could tell, but If I still run above blitz on Chrome for Windows, from a separate machine with chrome reinstalled, it is still returning 2.

Is there any update on this?


Error on /signup page

Cloning and starting the repo, then I get this runtime error on the web:

Any idea what's the issue?

I managed to fix this by running npx expo install --fix inside the packages/app dir and the apps/expo dir. There are some other deprecated functions but this gets past the main issue

https://github.com/xerxiastudios/CUA-SUPA/compare/main...bpmct:CUA-SUPA:main

Thanks! The npx expo install --fix in those two directories worked like a charm for me


Add clone() back to request_id to fix compiler error in juniper_warp

It's possible that I don't understand the original reasoning behind commit 54be035 removing .clone() here but this PR adding it back allows compilation to succeed and completes the fix of #705 via #707 for me. With this change multiple subscriptions now work on master.

Thanks! I'll add this to CI to prevent further breakage.


With index methods

Implementation of methods need improving in Iterable extensions. ScalaDoc comments need improvement here and in SeqGen.

I think this can be closed as done.


parsePlayerInfo performance

Hi,
I'm trying to get players info from number of demos with parsePlayerInfo
However I noticed it's much less performant, than parsing some event say round_announce_match_start and then one tick from it, to get the same data.
I'm not sure if there's issue with parser or just it is what it is.
This is the result I get when parsing 44 demos:
parsePlayerInfo: 13.306s
parseEventAndTick: 6.574s
I'm running JS script on 6 worker threads and demos are on SSD.
Script:
demoParserTest.zip

Ah you are right, the parsePlayerInfo is unreasonably slow. This could be fixed but my plan is to move away from these separate functions into simple game events. So you would use something like "player_connect" event in the future.

https://github.com/LaihoE/demoparser/pull/201
player_info does not parse entities anymore and should be very fast now.


Update troubleshooting instructions for antrea-agent API

A bearer token is required for local access now that the API is
externally accessible (in order to expose Prometheus metrics). The
instructions to connect directly to the antrea-agent API therefore
needed to be updated.
Thanks to @srikartati for pointing out this issue to me.

/skip-all


issue with keychain

In KeychainItemWrapper.m in writeToKeychain method the app crashes with -25299 error

This should no longer be the case. Keychain helper has been refactored. If that still happens, please reopen the issue.


Add command option to filter based on amount of transaction

Add command option to filter based on amount of transaction. Would be nice to filter on value. Whether it's only export values over a specific amount. Or values under a specific amount.

Added in v0.3.1


Logging formatter for stdout log records

Description
Can logfire provide a logging format configuration with access to LogRecord Attributes for logs written to stdout? The API can be as simple as adding an argument like logfire.configure(format="<format_string>").
The stdout logger is not configurable by means provided to the end user (as far as I can tell) and it really doesn't provide enough information to be useful (filename, line number, etc.). Integrating logfire into the stdlib logger via LogfireLoggingHandler might allow for custom formats but, per the documentation, using logfire directly is recommended.
current:
import logfire

logfire.configure()

logfire.info("log this message") # 23:04:47.846 log this message

proposed:
import logfire

log_format = "%(asctime)s - %(filename)s -%(lineno)d - %(message)s"
logfire.configure(format=log_format)

logfire.info("log this message") # 2024-10-08 16:49:45,896 - file.py - 16 - log this message

I agree that we need to make console logging more flexible and customizable in a user friendly way. Unfortunately it's not easy. The objects we have are OpenTelemetry spans, not LogRecords. Here's a rough demo of what you can do now:
import datetime

from opentelemetry.sdk.trace.export import ConsoleSpanExporter, ReadableSpan, SimpleSpanProcessor

import logfire

def format_span(span: ReadableSpan):
    timestamp = datetime.datetime.fromtimestamp(span.start_time / 1_000_000_000)
    filename = span.attributes.get('code.filepath', '')
    lineno = span.attributes.get('code.lineno', '')
    message = span.attributes.get('logfire.msg', '')
    return f'{timestamp} - {filename}:{lineno} - {message}'

logfire.configure(
    additional_span_processors=[
        SimpleSpanProcessor(
            ConsoleSpanExporter(
                formatter=format_span,
            )
        )
    ],
    console=False,
)

But the filename and lineno won't always be available, and this will log two lines for every non-log span (the first being for the pending span), and there's probably other complications.


Add a StatefulDoFn test that sets event time timer within allowed lat…

…eness
Please add a meaningful description for your change here

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

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

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @kennknowles

Are there any suites expected to fail?

Could this be merged?

I believe this is reproducing a bug, so that some (maybe all) runners ValidatesRunner suites would go red in postcommit. So I think please choose some suites to run and/or sickbay this case from those runners where it doesn't work yet.

Run Java Dataflow V2 ValidatesRunner

Run Java Dataflow V2 ValidatesRunner Streaming

Run Dataflow Streaming ValidatesRunner

Run Java Dataflow V2 ValidatesRunner Streaming


ansible script to deploy helloworld-msa to a regular OpenShift 3.2 cluster (not CDK)

for a demo we're building we've stood up a (shared) OpenShift cluster. To deploy helloworld-msa to it, we've built the folllowing ansible script: https://github.com/wrichter/hailstorm/tree/master/ansible/roles/layerX_openshift_demo_redhatmsa_on_devclient/tasks
This role is applied to  a RHEL system with software collections providing maven and nodejs which connects to the openshift cluster. This system has all the preregs (bzip2,git,bash-completion,atomic-openshift-clients,java-1.8.0-openjdk-devel,rh-maven33,rh-nodejs4) installed via yum and has two OS users called "developer" and "admin". On the Openshift cluster, the admin user has cluster-admin permissions.
This script is almost idempotent and can be run multiple times without failing.
Since we never forked from this repo, I cannot send a pull request :-( but maybe it's still good for inspiration.

It seems that it does the same thing of this ansible script but using different approaches. Am I right? https://github.com/redhat-helloworld-msa/helloworld-msa/blob/master/ansible/helloworld-msa.yml

a little bit less (doesn't deploy jenkins), more focus on idempotency, no assumptions  doesn't modify cluster-wide things such as the restricted scc

Given the status of the current Ansible script, we will keep using the current version.


Readability enhancements, clarified support and environmental URLs...

for external audience, updated permid.org PNGs

Dan can you take care of the conflict so we can merge your changes.


Support JDT external annotations files

See the external annotation file (.eea) feature for JDT described here:
https://wiki.eclipse.org/JDT_Core/Null_Analysis/External_Annotations
This is relevant for:

BEF, such that JDT Java Editor can show these NPE cases as errors (but only if the file is open in the editor)
Bazel builds that have the Eclipse Java compiler installed as the Java toolchain, such that command line build will emit the error

FYI http://www.lastnpe.org


apply dependency fix

as we discussed before — see https://github.com/brunch/brunch/issues/916 and https://github.com/stephenhutchings/jade-static-brunch/pull/1
up to you whether or not to apply while we discuss fixing that on the brunch level :smile_cat: but figured better to have the pull request around in case you decide you do want it.

@lalomartins This is how jaded-brunch was doing this [in early 2013][old], but the feature for requesting brunch to skip storing the file was added to brunch afterward due to an issue where I had a project that had a lot of static files which resulted in a large amount of unnecessary JavaScript in the output.

The blunt answer is that Brunch was not designed to provide this capability, so all these solutions are inherently hacky workarounds with some drawbacks. I pointed out in brunch/brunch#916 where someone could look to tweak the workflow for a better fix, but it's not something I'm going to personally work on.
I have ideas for Brunch 2.0 which would make this a non-issue, but I don't honestly know when I'll work on that either.

@es128 Maybe I'm missing something, but if it's an accepted enhancement in brunch then doesn't that imply that it's a use-case that is going to be supported? If it wasn't expected to work properly then I don't know why the enhancement would be accepted.
Either way, I'm asking because I want to dig in and see if I can fix it. That may not work, because brunch's code is not very helpful but I'm planning on giving it a shot if there's some expectation that there is going to be a release sometime soon.
I just don't want to spend time on it, submit a PR, and then a release never happens.

A PR that solves the problem would be accepted and published

I didn't mean in my earlier comment that the desired behavior here is being deliberately not supported. It just wasn't designed this way from the beginning, and as we've seen problems have arisen from trying to find ways to slide it in. There probably is an easy enough solution to the remaining problem, but at this point I think the better approach is to encourage community participation in figuring it out than waiting for me to personally focus on it.

Thanks for clarifying, @es128. I'll attempt a fix sometime later this week :)


HelmInstaller install wrong version

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: HelmInstallerV0
Environment

Server - Azure Pipelines

If using Azure Pipelines, provide the account name, team project name, build definition name/build number: https://dev.azure.com/T-Systems-MMS/phonebook/_releaseProgress?_a=release-environment-logs&releaseId=116&environmentId=178

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name: Hosted Ubuntu 1604

Issue Description
The specified Verion in the Task: Install Helm 2.14.2 is 2.14.2 but Helm Version 2.15.0 is installed from cache.
Why??? Few days ago it was working as expected.
Task logs
Failing Task:
https://dev.azure.com/T-Systems-MMS/phonebook/_releaseProgress?_a=release-environment-logs&releaseId=116&environmentId=178
Working Task:
https://dev.azure.com/T-Systems-MMS/phonebook/_releaseProgress?_a=release-environment-logs&releaseId=114&environmentId=174
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs

Well ok, seems like the checkmark overrides the setting... Very unintuitive.
So I am proposing to show the version input only if the checkbox is not set.

ok, the 1.0-preview has it implemented.

When will you release the "official"  version of the task? @chshrikh

@vithati please check this


FileNotFoundError: [Errno 2] No such file or directory: 'data-mscoco/images/val2017/000000532481.jpg'

I encountered this problem while i was trying to run the evaluation

Hi @khoa127,
You have to unzip the MS COCO data and also you would need to download the corresponding WholeBody annotations as explained here. If you do not have the images and the annotation files in the default location you can also specify the location via CLI. You can check all CLI options for the eval module with python -m openpifpaf.eval -h. These are the four CLI options to specify the image folder and annotation file locations:
data module wholebody:
  --wholebody-train-annotations WHOLEBODY_TRAIN_ANNOTATIONS
                        train annotations (default: data-
                        mscoco/annotations/person_keypoints_train2017_wholebody_pifpaf_style.json)
  --wholebody-val-annotations WHOLEBODY_VAL_ANNOTATIONS
                        val annotations (default: data-
                        mscoco/annotations/coco_wholebody_val_v1.0.json)
  --wholebody-train-image-dir WHOLEBODY_TRAIN_IMAGE_DIR
                        train image dir (default: data-mscoco/images/train2017/)
  --wholebody-val-image-dir WHOLEBODY_VAL_IMAGE_DIR
                        val image dir (default: data-mscoco/images/val2017)


Populate Dataverse with Contoso or Northwind data?

 If not, any tools or projects that will do that for me? 500 or so records is fine. Demo only.
I've seen this one https://www.sqlbi.com/tools/contoso-data-generator/, but it creates a SQL db.
Thanks.

@evogelpohl I do not know what project you are referring to as you do not mention it in your issue statement. However, you can populate an environment with sample data. See Add or remove sample data


ADD should behave consistently with relative paths and URIs

fetch file before running dockerfile
wget https://cpan.metacpan.org/authors/id/X/XE/XENO/Pod-Spell-1.17.tar.gz
FROM busybox:latest
MAINTAINER Caleb Cushing<EMAIL_ADDRESS>WORKDIR /tmp

ENV MIRROR https://cpan.metacpan.org/authors/id/X/XE/XENO/
ENV FILENAME Pod-Spell-1.17.tar.gz

ADD $FILENAME local/
RUN ls -1 local

ADD $MIRROR/$FILENAME remote/
RUN ls -1 remote

IMHO, the 2 ls's should print the same thing, however the second does not extract a tarball.
ADD foo.tar.gz, ADD http://...bz2, ADD ftp://...tar.xz , etc, should act consistently.
Docker version 1.8.1, build d12ea79

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

ahh its like pouring salt on a wound :-)

see #14934 & #11174

This indeed would be breaking change. We have couple of unfortunate design decisions in dockerfile handling and it's sad that we can't fix them now because some people use them as features.


CK65 - Disabling Physical Scanner?

The commands to start & stop scanning do not disable the physical scanner on my CK65.  Based upon what I can tell, when you stop the scanning, the logic simply does not process the actual value scanned.
This seems like a very non-user friendly, confusing situation.  For all practical purposes, the scanner is still functioning, the value is scanned, so it appears to the user that something should happen.
Is there a way the physical hardware can be turned off & on?

Start scanning uses property TRIGGER_SCAN_MODE_CONTINUOUS, so what this does is just to start scanning and wait for something to scan. When you use the physical button you override this behavior with a TRIGGER_SCAN_MODE_ONESHOT so the scanner stops continuos scanning and delivers whatever it scanned.
If you have some use case in which you need to disable physical scanner button, check the docs referenced in the README.md, BarcodeReaderProperties.java and BarcodeReader.html and search for the property to disable scanning, presumably some TRIGGER_CONTROL_MODE_XXX... anyway search for all the properties having *TRIGGER* and read docs and there should be what you need.
Important: note that all the properties I mentioned above like TRIGGER_SCAN_MODE_CONTINUOUS are really the Android const name for the property... the property value is what you need when using it from dart code... for instance:
    public static final String TRIGGER_SCAN_MODE_CONTINUOUS = "continuous";

What you should use as property name from dart is continuous.

Thank you!
I did scan that document previously and did not see anything that made sense, however I will look again.


Set a limit on the number of highlighted changed words in comments

E.g. this is too many:

We now require at least 50% unchanged.


chore: add more tests for empty and non-present

Add a few more test cases around empty and non-present bodies for https://github.com/unjs/h3/issues/197.
Some of the tests are currently failing as readBody/readBodyRaw for a request with Content-Type: 'text/plain' and body: ''  return undefined now. Is this the expected behavior? #197 was about distinguishing empty vs non-present bodies, but both return undefined now.

Thanks <3


Incorrect 64-bit Counter Behavior on nRF52840DK and nRF52833DK Devices Using counter_get_value_64()

Hi Zephyr Community,
Describe the bug
We conducted hardware counter tests on the nRF52840DK[1] and nRF52833DK[2]. During testing, we found that calling counter_get_value_64() on these devices returns an incorrect counter value (it appears to remain static).
To Reproduce

Select an external clock device (e.g., timer, rtc) from the device tree file of the nRF52840DK or nRF52833DK.
In the test code, call counter_get_value_64() and pass the external clock device as the argument.
Build and flash the application to the board using the following commands:

west build -p -b nrf52840dk/nrf52840 <path/file>
west flash

Observe the output.

Expected behavior
The counter value of the external clock device should vary over time.
Impact
This issue could affect applications that rely on the 64-bit counter functionality.
Logs and console output
The following screenshot shows that the output value of the counter remains constant.

Environment:
OS: Linux
Toolchain: Zephyr SDK 0.16.8
Commit: 318b4957
Additional context
[1] https://docs.nordicsemi.com/bundle/ps_nrf52840/page/keyfeatures_html5.html
[2] https://docs.nordicsemi.com/bundle/ps_nrf52833/page/keyfeatures_html5.html

Where did you start the counter?

Hi nordicjm,
Before calling the function counter_get_value_64, we ensured to start the counter by calling
counter_start once.
Further testing on the nrf52840dk board showed that calling counter_get_value_64 returns -ENOTSUP, as defined in counter.h, indicating unsupported functionality.
Here is our experimental code:
main.c:
#include <zephyr/kernel.h>
#include <zephyr/sys_clock.h>
#include <zephyr/drivers/counter.h>
#include <stdio.h>

static const struct device *external_sensor_clock = DEVICE_DT_GET(DT_CHOSEN(zephyr_sensor_clock));

int main(void)
{
	uint64_t cycles = 0;
	uint64_t delta_ns = 0;
	int err;

    	counter_start(external_sensor_clock);

	while (true) {
		k_sleep(K_MSEC(1000));

		err = sensor_clock_get_cycles(&cycles);
		if (err) {
			printf("Failed to get sensor clock cycles, error: %d\n", err);
		}

		printf("Cycles: %llu\n", cycles);

		delta_ns = sensor_clock_cycles_to_ns(cycles);

		printf("Nanoseconds: %llu\n", delta_ns);
	}

	return 0;
}

int sensor_clock_get_cycles(uint64_t *cycles)
{
	__ASSERT_NO_MSG(counter_is_counting_up(external_sensor_clock));

	int rc;
	rc = counter_get_value_64(external_sensor_clock, cycles);

	return rc;
}

uint64_t sensor_clock_cycles_to_ns(uint64_t cycles)
{
	uint32_t freq = counter_get_frequency(external_sensor_clock);

	if (freq == 0) {
		printf("Sensor clock %s has no fixed frequency\n", external_sensor_clock->name);
		return 0;
	}

	return (cycles * NSEC_PER_SEC) / freq;
}

nrf52840dk_nrf52840.overlay:
/ {
	chosen {
		zephyr,sensor-clock = &timer0;
	};
};

&timer0 {
	status = "okay";
};

&rtc0 {
	status = "okay";
};

prj.conf:
CONFIG_SENSOR=y
CONFIG_COUNTER=y
CONFIG_RTC=y

Then the function is not supported, this is not a bug


Tuple for associated values in enum case is deprecated

Code generated with AutoEquatable and AutoHashable templates is raising a deprecation warning
Using a tuple to access associated values in the enum case is now deprecated since Swift 5.2
Update the 2 templates accordingly to put the exhaustive list of associated values, and avoid the warning
#842

thanks for contributing🙇


update bus lua profiles to accept highway=busway

Is this something that is present upstream?

No, OSRM default does not include any bus specific routing profile. But in Transition, we must allow buses to route on busways


upgrade to codemirror v19

Signed-off-by: Augustin Husson<EMAIL_ADDRESS>This PR is going to upgrade codemirror to the last version. There are some breaking changes in lezer and it looks like the behavior of the parser changed.
Some tests in the autocompletion failed and need to be fixed before thinking to merge this PR.

this PR once it is finished will fix https://github.com/prometheus-community/codemirror-promql/issues/150 and fix https://github.com/prometheus-community/codemirror-promql/issues/149

and don't know if there is a link, but upgrading codemirror lead to have a node_modules folder in the react-app which is not the case currently on the main branch

Looks good so far. Weird that it would create a node_modules dir that wasn't there before, would be good to find out why.

should be good for a review now @juliusv I fix the code of the autocompletion.
The behavior changed for the expression like http_requests_total{method="GET"} off. Before it was identified as a Binary expression. Now it's just a correct Expression + an error node.
For the new node_modules folder that appears in the react-app, I think it's coming from a conflict on the babel deps. Don't know where is coming from so far ... That's just my best supposition of why it appeared.

:+1: besides the comments


New ESX server is getting DHCP by default why it shouldn't

Environment info

Version: 2.6.0-2c95897
Deployment: AZURE
Customer: QA

Actual behavior

When we installing fresh new ESX server, it gets DHCP by default

Expected behavior

We should not get any ip from the DHCP by default.

No longer relevant due to OVA deployment support removal


Add dependencies in setup.py

Currently, Fedora's automatic Python run-time dependency generator fails to generate dependencies because it uses package metadata (as recorded in installed *.dist-info directories) to determine what the package depends on [1]. Add dependencies in setup.py so dependencies can be contained in *.dist-info.
Note this package also depends on cryptography. Add this missing dependency as well.
[1] https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_run_time_dependency_generator

LGTM

PS I'll setup packit for this so it syncs the spec through to Fedora


Incorrect convolution computation in a specific case on Xtensa HiFi4

Hi, I'm not sure if TFLM supports Xtensa HiFi4 cores, because the optimized kernels have preprocessing conditions like:
#if defined(HIFIMINI)
      ConvEvalHifiMini(ConvParamsQuantized(params, op_data.reference_op_data),
                       op_data.reference_op_data.per_channel_output_multiplier,
                       op_data.reference_op_data.per_channel_output_shift,
                       tflite::micro::GetTensorShape(input),
                       tflite::micro::GetTensorData<int8_t>(input),
                       tflite::micro::GetTensorShape(filter),
                       tflite::micro::GetTensorData<int8_t>(filter),
                       tflite::micro::GetTensorShape(bias),
                       tflite::micro::GetTensorData<int32_t>(bias),
                       tflite::micro::GetTensorShape(output),
                       tflite::micro::GetTensorData<int8_t>(output));
#elif defined(FUSION_F1) || defined(HIFI5)
      ConvEvalHifi(context, node, params, op_data, input, filter, bias, output);
#else
      return ConvReferenceEvalInt8(context, node);
#endif

But if it does (I used a small workaround), then it computes convolution incorrectly in one specific case. When I compare the outputs from HiFi4 and reference implementations, I see some differences:

I created a unit test which reproduces the issue.
conv_test.txt
On Xtensa Fusion F1 core the computation works fine.

The hifimini implementation is not intended for use with Hifi4.
The naming was unclear previously but that has recently been improved. Please see this comment for the current way to build TFLM with Hifi4 optimized kernels:
https://github.com/tensorflow/tflite-micro/issues/296#issuecomment-886868989

Sorry, I was not clear which configuration I used, but I used the same for HiFi4 as for Fusion F1. HiFi Mini would not compile with the HiFi4 compiler configuration (different inline assembler). I stepped through the code and it uses the HiFi4 NN library.

@cad-audio

Thanks for the clarification. We'll look into this along with the nnlib authors (@cad-audio)

@advaitjain, below PR should resolve this issue.
https://github.com/tensorflow/tflite-micro/pull/330

@lvadke, if you can confirm that your issue is resolved once #330 is merged, that would be great.
Please note that to get the updated xannlib, you will need to delete what is currently downloaded prior to building the code:
rm -rf tensorflow/lite/micro/tools/make/downloads/xa_nnlib_hifi4

I'm going to leave this github issue open until we add a new test case to the conv kernel tests based on the file attached with https://github.com/tensorflow/tflite-micro/issues/329#issue-954051795

Yes, update of the HiFi4 NN library to version 2.4.1 fixed the issue. Thank you for very quick resolution.

@lvadke Closing this issue as it is working as expected with 2.4.1 lib


[Feature]: Add a Jest architecture Blog

🚀 Feature Proposal
I would like to add a small write up on the internal architecture of Jest, this can be potentially released as a blog in the blog section and can acts helpful to anybody who is curious around the internal.
For now i have think of explaining what is jest, explaining the Jest architecture  and including some external links and talks describing the same
also thinking of code diagrams to explain the whole flow
this is a small POC for the same

Motivation
Users and contributors both can benefit as they can get a overall idea of how Jest works under the hood
Example
N/A
Pitch
N/A

@SimenB thoughts on the same!


Combine sub-TOCs into standard TOC

Combine the following sub-TOCS into the /standard/ TOC:

attributes
collections
cross-platform
events
exceptions
garbage-collection
generics
io
parallel-programming
serialization

Delete/combine a few dispensable top-level topics

LGTM, my only question is why the Security sub-TOC wasn't incorporated into the TOC?

Thanks for the review. Good question - there were no security articles with page views over 5000, so I'm not currently planning to add any security articles to the landing page. Also, this PR was taking longer than I'd hoped!


Update rust-toolchain to 1.70.0

Rust version of AtCoder is 1.70.0.

I just remembered  #118. Other platforms such as yukicoder still use <1.70.0.

We must continue discussing what is the appropriate MSRV on #118 , and this issue might as well be closed for now.

@qryxip
Sorry for my slow response.

I just remembered #118. Other platforms such as yukicoder still use <1.70.0.

I think that we just need to focus on AtCoder platform. It's a tough to support all platforms.

@qryxip
Could you approve this PR if it's ok to you? what else code should I fix?

I'm going to ignore tests that are not run and merge this PR, and then stop requiring 1.42.0-* tests to pass because they are stale.


todo example no longer works with 5.9.1

Hi Ben,
i still have an older version of quickflux (commit 65d79cfedf61ca0a071cb528e09280af494f9e0b)  and the todo app still adds new entries there (using 5.9.1, on osx ) but the new version of the todo example doesn't do anything when i check off items or enter new items in the list and press the "Add" button.
is this a known issue ? or has something broken with the latest version of Qt ?
let me know if it is a new issue and i can collect more details if you need them, otherwise let me know and i'll delete this issue. don't want to double up. (i searched but didn't see any other report like this)
thanks,
tom

hi @clogwog ,
Thanks for reporting. I have checked the example program and found that it is a silly mistake caused by one of the middleware. I have just committed a fix. If you still have any problem, please feel free to contact me.


%20 Removal

So, my name uses multiple words, and I was just wondering if there is a way to remove the "%20" that appears on the page showing your brews between each word.

I don't know what you are talking about?

My username has multiple words (Eye of Horus). On the page that shows the
list of my brews, it says "Eye%20Of%20Horus's Brews."
On Sun, Jan 22, 2017 at 12:22 AM, Scott Tolksdorf<EMAIL_ADDRESS>wrote:

I don't know what you are talking about?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stolksdorf/homebrewery/issues/293#issuecomment-274314275,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AX-d3xhmw4os-06roPdl8Tzmm3H73CWDks5rUwPTgaJpZM4LkE7K
.

--
"People are idiots."
-Braden Roberts

%20 is the ASCII code for a space, so if the name was auto-generated when you first saved it might have stuck those in there. You can change the name of your file manually by opening it up and clicking the small hamburger button (the ☰ icon) across from the snippets bar. Then you can type in the new name.

@Brancor the %20 is just a graphical hiccup. Some of the components on that page use the URL to grab the username. This will be visually fixed in v3.

This appears to have been fixed, as evident here:
https://homebrewery.naturalcrit.com/user/spacey macfacespace
Closing.


Add support for styling edge FC by cost attribute

 Add support for receiving paths as parts (individual edges) instead of aggregated paths/geometries

 Visualize path parts with different colors based on noise/AQ


revert(convert): remove convert from file subcommand

Codecov Report
Patch coverage has no change and project coverage change: -0.01 :warning:

Comparison is base (d19b622) 33.97% compared to head (be75a6e) 33.96%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #975      +/-   ##
==========================================
- Coverage   33.97%   33.96%   -0.01%
==========================================
  Files         100      100
  Lines       11906    11909       +3
==========================================
  Hits         4045     4045
- Misses       7468     7471       +3
  Partials      393      393

Impacted Files
Coverage Δ

cmd/convert.go
0.00% <0.00%> (ø)

cmd/root.go
0.91% <0.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Implemented gcapc-path configuration.

This closes #31.

@tobiastom thank you! Same as in PR #34 - could you change the commit message to something like:

feat: add gcapc-path option

Updated the commit message. :)

Cool! Thank you for merging it!
Any idea when you might release it?

Version 1.5.0 is release - you should be able to download it (also via composer) - thank you for your contribution :)


Handling cases where a collection's index is missing or renamed

Describe the bug
isOccurrence collection used to have an index named byPkgIDArtIDJust which was renamed to byPkgIDArtIDOriginJust in September 2023 under this commit. The change also includes addition of the field origin to the index.
However, since guac creates indices only if the guac graph doesn't already exist, the whole process of creating new/renamed indices is skipped when we already have a database and are not starting from scratch.
This issue is not just about creating new collection indices but also about managing the old ones. Should we delete all the existing indices that are not in our current map? Should we edit them? Or are there better options?
To Reproduce
Steps to reproduce the behavior:

Clone the repo and switch to the 80eab75 commit state
Run make container to create the local-organic-guac image
Switch to the main branch
Update the guac-graphql container in arango.yaml file to refer to the image created in step#2
In Arango DB UI check that the index byPkgIDArtIDJust is created
Ingest a provenance and verify the corresponding entry shows up in the isOccurrence collection
Update the guac-graphql container image to local-organic-guac image built out of the main branch
Check the indices of the isOccurrence collection

Expected behavior
isOccurrence collection should have only byPkgIDArtIDOriginJust index
Ingested document(s)
Any sample slsa provenance
Can you share the documents that are used to reproduce the ingestion errors or showcase the data quality issues.
No

@semmet95, I can do some testing to see but until we reach a v1.0 for GUAC, we should be deleting old (unused) indexes manually (if arango is persistent). Once we have determined the proper indexes needed, we should not need to change them for the particular collection.
As for adding/updating indexes, it should be fairly straightforward via edit them if guac graph exists within arango. I will take a look and provide a solution or steps to remediation.


Errors in the source code of Space Station 14

Hello! I found several errors in your code with the help of the PVS-Studio analyzer.
Issue 1
private void OnStateChanged(...., MobStateChangedEvent args)
{
  if (   args.NewMobState != MobState.Dead
      || args.NewMobState != MobState.Critical) // <=
  {
    return;
  }
  ....
}

Here's the link to the code fragment with the error.
The conditional expression is always true here: if 'args.NewMobState' equals 'MobState.Dead', it can't be equal to 'MobState.Critical', and vice versa. It is probably worth using the '&&' operator here instead of '||'.
Issue 2
public GasTankWindow(GasTankBoundUserInterface owner)
{
  ....
  _spbPressure = new FloatSpinBox
  {
    IsValid = f => f >= 0 || f <= 3000, // <=
    ....
  };
  ....
}

Here's the link to the code fragment with the error.
This error is similar to the previous one.  Perhaps it's better to replace the '||' operator with '&&'.
Issue 3
private void OnSmokeSpread(....)
{
  if (.... || args.NeighborFreeTiles.Count == 0)
  {
     ....
     return;
  }
  ....
  if (   args.NeighborFreeTiles.Count == 0
      && args.Neighbors.Count > 0
      && component.SpreadAmount > 0)
  {
    ....
  }
}

Here's the link to the code fragment with the error.
The 'args.NeighborFreeTiles.Count == 0' check is performed twice. However, if it will be true for the first time, a return from the method will be performed. Thus, the second check is always false, and the code in the second 'if' block will be unreachable.
Issue 4
protected override void Dispose(bool disposing)
{
  MenuBody.OnChildRemoved -= ctrl =>            // <=
    _uiController.OnRemoveElement(this, ctrl);
  ....
}

Here's the link to the code fragment with the error.
public void OnRemoveElement(ContextMenuPopup menu, Control control)
{
  ....
  element.OnMouseEntered -= _ => OnMouseEntered(element); // <=
  element.OnMouseExited -= _ => OnMouseExited(element); // <=
  element.OnKeyBindDown -= args => OnKeyBindDown(element, args); // <=
  ....
}

Here's the link to the code fragment with the error.
In this code fragment, newly created anonymous functions are used to unsubscribe from events. This will not work. Even if the event handler body and one of these functions’ bodies are the same, they are still different objects.
Issue 5
public ThresholdBoundControl(...., float value, ....)
{
  ....
  _value = value;
  ....
  CBoundEnabled.Pressed = _value != null;
  ....
}

Here's the link to the code fragment with the error.
'_value' is a meaningful type and is not Nullable. Therefore, it cannot be null. The true value will always be assigned to the 'CBoundEnabled.Pressed' property here.
Issue 6
public SuitSensorStatus? GetSensorState(....)
{
  ....
  // check if sensor is enabled and worn by user
  if (   sensor.Mode == SuitSensorMode.SensorOff
      || sensor.User == null
      || transform.GridUid == null)
  {
    return null;
  }
  switch (sensor.Mode)
  {
     ....
     case SuitSensorMode.SensorCords:
       ....
       if (transform.GridUid != null)
       {
         ....
       }
       else if (transform.MapUid != null)
       {
         coordinates = new EntityCoordinates(transform.MapUid.Value,
                            _transform.GetWorldPosition(transform, xformQuery));
       }
       else
       {
         coordinates = EntityCoordinates.Invalid;
       }

       status.Coordinates = coordinates;
       break;
   }

   return status;
}

Here's the link to the code fragment with the error.
Note that the 'transform'GridUid == null' condition is always checked at the beginning of the method. If it is true, a return from the method is performed. So, the 'transform.GridUid != null' check, which is stored in 'switch', will always be true. And the code that is stored in the 'else if' and 'else' blocks of the same condition will be unreachable.

The code is perfect.

yeah im sure theres only 6 bugs in the entire codebase gj

yeah im sure theres only 6 bugs in the entire codebase gj

I am surprised it found so few


Pages 404 on page refresh using Github Pages

Version
v2.11.0
Reproduction link
https://github.com/jbratcher/nuxt-gh-pages-404-reproduction
Steps to reproduce
Demo Link

Navigate to About Page
Refresh Page

What is expected ?
Page is refreshed
What is actually happening?
Page give 404 error
Additional comments?
I have a project using Nuxt and Github Pages and noticed when I refreshed static pages other than the home route, they would give a 404 error.
A similar but not exactly the same issue may be #2240
This issue can be fixed by setting generate.fallback to true.  However, I'm really not sure why/how this works.  As I understand it, generate.fallback is used to control what is rendered when the page isn't found.
In nuxt.config.js
export default {
  generate: {
    fallback: true
  },
}

The fix branch contains this code for the fix.
Any guidance on what is happening here?  Thanks!

This bug report is available on Nuxt community (#c10479)

This is intended. When generating pages, you have to tell your platform/web server (GitHub pages in your case) in case you want to handle 404 errors "through your own application". This is usually not wanted when building non-SPAs but highly recommended for SPAs (to avoid the mentioend reload - 404 - issue).
When enabled, generate.fallback creates a 404.html which is essentially an SPA fallback page. GitHub pages picks it up automatically and redirects all "not found URLs" there.
However, generated URLs (with mode: universal) should not need a redirect. In your case, the capitalization of About.vue is giving troubles here. Lowercasing it and it should work fine 👍🏻
For the future, it might be an idea to unify pages, no matter how they are capitalized (e.g. About.vue should behave the same like about.vue or aBoUt.vue, as long as no drawbacks are present (cc @pi0).
Closing here 😋


:bug: fix the placement decision clusters have random order issue

Summary
Related issue(s)
Fixes https://github.com/open-cluster-management-io/ocm/issues/233

/assign @qiujian16
/cc @zhujian7

/approve
/assign @zhujian7

/lgtm

/lgtm


Cannot overwrite read only files anymore

On previous versions of VSCode (1.47) you received a message when attempting to save a file that was marked read only:

As of 1.48, you receive a different message that lacks an overwrite option:

When a file is simply marked as read only, retrying as admin is not an appropriate option and fails to resolve the issue.
Please restore Overwrite functionality as it is very useful for us that use archaic source control systems that rely on making files read only.

VSCode Version: 1.48.0
OS Version: Windows 10 (1903)

Steps to Reproduce:

Open a file with the read only flag set
Make a change to the file
Save the file

Does this issue occur on the latest insiders build?: Yes
Does this issue occur when all extensions are disabled?: Yes

Thanks, a regression we can probably fix for recovery.

Was just about to post this as well. This changed about 3-4 weeks back in VSCode Insiders as well if that time frame helps.

Works great, I have verified the fix on windows with read-only files.

I just went on my Twitter and the same picture was on my profile filling me with no friends and no tweets I report and blocked
Sent from my iPhone

On Aug 14, 2020, at 8:07 PM, James Taylor<EMAIL_ADDRESS>wrote:

Was just about to post this as well. This changed about 3-4 weeks back in VSCode Insiders as well if that time frame helps.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


%directory% does not work anymore.

Bug description
%directory% does not work anymore.
Steps to reproduce
Use the renamer with theses settings :
I got this :
Expected behavior
The same name into the origin, and destination field. %directory% was supposed to store the path of the file, I worked by the past, but doesn't.
Context
Logs does not have information for the rennaming tool.
System information

OS: Windows 11 21H2
Grabber version: Nightly 7b129b87 (Just checked in ad68aae9, don't work either.)

It's been renamed to old_directory and old_filename to prevent collisions with existing tokens.
See issue #2417. It was for %filename% but since I had to rename one I just renamed both.


[Important] Update Baileys's URL

After adiwajshing took down Baileys, the community re-uploaded it under WhiskeySockets

Thank you!


[icu] update to 59.1

This also switches icu to autotools based buildsystem, enabling all flavors of desktop builds.

Thanks for the update!
The issue with the path was most likely due to find resolving to C:\windows\system32\find.exe instead of GNU find. Instead of modifying the MSYS2 enlistment (and possibly leaving it in a bad state if the build fails!) I changed the portfile to insert the path in the middle.
Could you comment on why you chose to do several individual calls to bash instead of using a separate script (like in ffmpeg)?

Nothing special, just thought the logs will be a little cleaner this way.


Feat: Support All AWS Partitions

This PR extends support of moto to all AWS Partitions.
One can see all Partitions, Regions, and DNS Suffixes with: curl https://raw.githubusercontent.com/boto/botocore/develop/botocore/data/endpoints.json | jq '.partitions[] | [.partitionName, .partition, .regions, .dnsSuffix]'

I realized I was querying an old endpoints.json, so I just added the most recent DNS Suffixes.

Hi @joel-aws!
Another followup question/issue around this feature:
The get_available_regions() will be potentially deprecated by the AWS SDK team - see https://github.com/aws/aws-sdk/issues/206#issuecomment-1471354853
The suggested alternative way to get this region-information is to use the SSM parameter store instead, and I am looking into a feasible implementation for Moto.
The main problem I am running into is that these ISO regions are not exposed in the SSM parameter store. The US-gov and China regions are exposed, so it would make sense to me if the ISO-regions are also part of it.
Is that something you can raise internally, to see if that  information can be included/exposed? I understand that it's probably an entirely different team responsible for this, but I'm assuming that a request from an internal employee would hold more weight than a random support ticket from a regular user :slightly_smiling_face:

@bblommers -- apologies for the delay in responding!
I am now posing this question internally... On the flip side, we value customer input via service tickets. If you haven't already opened one, could you do that and share the ticket # ?

No problem @joel-aws - thanks for getting back to me.
I haven't been on top of this either, to be honest - I only just opened a ticket now, nr<PHONE_NUMBER>00169.


SDK 'Microsoft.Build.Sql/0.1.3-preview' specified could not be found

Issue Type: Bug

open a sql database project
in 'database projects', right-click projectname and choose 'Build'

=> OUTPUT
	[ Build ]
    > "C:\\Program Files\\dotnet\\dotnet.exe"  build "c:\\Users\\m.yusername\\OneDrive - mycompany\\DBProjects\\DatabaseProjectTESTDB\\DatabaseProjectTESTDB.sqlproj" /p:NetCoreBuild=true /p:SystemDacpacsLocation="c:\\Users\\m.yusername\\.azuredatastudio\\extensions\\microsoft.sql-database-projects-0.18.0\\BuildDirectory"
    stdout: Microsoft (R) Build Engine version 17.2.0+41abc5629 for .NET
    stdout: Copyright (C) Microsoft Corporation. All rights reserved.
    stdout:
    stdout:
    stdout: c:\Users\m.yusername\OneDrive - mycompany\DBProjects\DatabaseProjectTESTDB\DatabaseProjectTESTDB.sqlproj : error : Unable to resolve 'Microsoft.Build.Sql (= 0.1.3-preview)' for '.NETStandard,Version=v0.0'.
    stdout:
    stdout: c:\Users\m.yusername\OneDrive - mycompany\DBProjects\DatabaseProjectTESTDB\DatabaseProjectTESTDB.sqlproj : error MSB4236: The SDK 'Microsoft.Build.Sql/0.1.3-preview' specified could not be found.
    stdout:
    stdout:
    stdout: Build FAILED.
    stdout:
    stdout:
    stdout: c:\Users\m.yusername\OneDrive - mycompany\DBProjects\DatabaseProjectTESTDB\DatabaseProjectTESTDB.sqlproj : error : Unable to resolve 'Microsoft.Build.Sql (= 0.1.3-preview)' for '.NETStandard,Version=v0.0'.
    stdout: c:\Users\m.yusername\OneDrive - mycompany\DBProjects\DatabaseProjectTESTDB\DatabaseProjectTESTDB.sqlproj : error MSB4236: The SDK 'Microsoft.Build.Sql/0.1.3-preview' specified could not be found.
    stdout:     0 Warning(s)
    stdout:     2 Error(s)
    stdout:
    stdout:
    stdout: Time Elapsed 00:00:00.48
    stdout:
    >>> "C:\\Program Files\\dotnet\\dotnet.exe"  build "c:\\Users\\m.yusername\\OneDrive - mycompany\\DBProjects\\DatabaseProjectTESTDB\\DatabaseProjectTESTDB.sqlproj" /p:NetCoreBuild=true /p:SystemDacpacsLocation="c:\\Users\\m.yusername\\.azuredatastudio\\extensions\\microsoft.sql-database-projects-0.18.0\\BuildDirectory"    … exited with code: 1
	>>> "C:\\Program Files\\dotnet\\dotnet.exe"  build "c:\\Users\\m.yusername\\OneDrive - mycompany\\DBProjects\\DatabaseProjectTESTDB\\DatabaseProjectTESTDB.sqlproj" /p:NetCoreBuild=true /p:SystemDacpacsLocation="c:\\Users\\m.yusername\\.azuredatastudio\\extensions\\microsoft.sql-database-projects-0.18.0\\BuildDirectory"   … errored out: Process exited with code 1

Extension version: 0.18.0
Azure Data Studio version: azuredatastudio 1.38.0 (dff93c4736269ef2a7e8a225737a0d8354bf0246, 2022-07-26T23:56:16.217Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8565U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.81GB (8.15GB free)

Process Argv

Screen Reader
no

VM
0%

@PowerDBAKlaas Did building this project used to work and recently start failing? Can you please share the sqlproj for this project?

Same happens to me.  Builds fine in full Visual Studio Community edition IDE.  I have both VS 2022 and VS 2022 Preview installed, which I suspect is the root of the problem … kind of the risk one takes with previews. :neckbeard:

@kisantia it is the first time I try to build a proj, so I have no idea since when this problem exists
I can share the proj later (on holiday now) but it is nothing complicated;
I created a new testDB with just 1 table with 4 columns

<?xml version="1.0" encoding="utf-8"?>
<Project DefaultTargets="Build">
  <Sdk Name="Microsoft.Build.Sql" Version="0.1.3-preview" />
  <PropertyGroup>
    <Name>DBA_DB</Name>
    <ProjectGuid>{8FF4226E-F67B-4C5E-BBA1-176896CD6165}</ProjectGuid>
    <DSP>Microsoft.Data.Tools.Schema.Sql.Sql150DatabaseSchemaProvider</DSP>
    <ModelCollation>1033, CI</ModelCollation>
  </PropertyGroup>
  <Target Name="BeforeBuild">
    <Delete Files="$(BaseIntermediateOutputPath)\project.assets.json" />
  </Target>
</Project>

@zijchen have you seen this error before or have any ideas on why Microsoft.Build.Sql can't be found? Maybe nuget.org needs to be added as a source or something like that?

nuget.org should be included by default. @PowerDBAKlaas do you have any additional Nuget sources configured? You can run dotnet nuget list source to check.

@zijchen
"No sources found."

nuget.org should be listed like this:

Can you try adding nuget.org as a source like in the first example here?

dotnet nuget add source https://api.nuget.org/v3/index.json -n nuget.org
fixed everything!
Thank you @kisantia !!!
should that have been executed when adding the extension? or any reason this could have been removed from my system(s)?
maybe useful to add this on the 'dependencies' tab?

nuget.org should be included by default, as Zi mentioned. Have you used chocolatey or PowerShell’s Install-Package command at some point before building this project? It might be the same problem described here: https://developercommunity.visualstudio.com/t/missing-default-nuget-package-source/1453751

I have used choco and winget and MS Store to install packages, but can't remember in what order or time frame.
Can you clarify with what nuget.org should be default installed? Is that with Windows setup, Azure Data Studio install, .Net SDK, .... ?


controller.provisioning	Failed to bind pod to a node in highly concurrent cluster (100s nodes and 1000s pods)

Version

Karpenter:v0.5.1

Kubernetes: Server Version: version.Info{Major:"1", Minor:"20+", GitVersion:"v1.20.7-eks-d88609", GitCommit:"d886092805d5cc3a47ed5cf0c43de38ce442dfcb", GitTreeState:"clean", BuildDate:"2021-07-31T00:29:12Z", GoVersion:"go1.15.12", Compiler:"gc", Platform:"linux/amd64"}
Expected Behavior

per the docs "Karpenter schedules pods that the Kubernetes scheduler has marked unschedulable. After solving scheduling constraints and launching capacity, Karpenter optimistically creates the Node object and binds the pod. "
Actual Behavior

With 120 nodes and 7000 pods, I see many
controller.provisioning	Failed to bind default/appupdate-5b8485cd6d-bz2gz to ip-192-168-142-135.us-west-2.compute.internal, Operation cannot be fulfilled on pods/binding "appupdate-5b8485cd6d-bz2gz": pod appupdate-5b8485cd6d-bz2gz is already assigned to node "ip-192-168-157-11.us-west-2.compute.internal"	{"commit": "6984094", "provisioner": "default"}

Steps to Reproduce the Problem

Create a deployment and and scale it to ~7000 pods with ~100s nodes

How many of these? This is a natural and self healing mechanism where the kubescheduler is scheduling pods as capacity comes online. Perhaps the message should say "Ignoring pod, already bound to capacity by the kube scheduler". You shouldn't be seeing this on a large percentage of your scale up, but it definitely can happen.

I've seen this occur when binding is throttled on a subset of nodes being launched which, as Ellis mentioned, results in kube-scheduler beating the bind call and then getting duplicate binds.  Full logs would definitely help if you have them or can reproduce.

That does not happen to me anymore because my load stabilized. Should we close it and reopen when I see the issues again?

We've made some improvements in this area, so I'm hoping it's resolved. Feel free to reopen.


Adds a crossing with a lane turning to the right.

🎉 New feature
Closes #29
Summary
Solves the following map:

Test it
Following the instructions in the readme, you can reproduce the map.
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[x] Added example and/or tutorial
[x] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if it affects the public API)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Thanks for creating this! I'll test it ASAP.

One extra comment, you can add the generation of this map to the tool script: https://github.com/maliput/maliput_xodr/blob/main/tools/update_resources.sh

Thanks for creating this! I'll test it ASAP.

@liangfok , I pushed some fixes mainly related to the traffic direction rules. For testing use the latest files that I now pushed.

Got it. I just grabbed the latest revision. Thanks for the note.

Gently ping to check if this PR is mergeable as it is. @liangfok

This is still undergoing the initial review, but based on work in #37, I will likely have to re-generate the map using our downstreawm tooling once our downstream tooling supports stop lines with and without stop signs. I'm going to wait for #37 to pass initial testing before refocusing on this map.

This is still undergoing the initial review, but based on work in #37, I will likely have to re-generate the map using our downstreawm tooling once our downstream tooling supports stop lines with and without stop signs. I'm going to wait for #37 to pass initial testing before refocusing on this map.

Friendly ping @liangfok . Based on this result, is it necessary to update something here?

I continue to wait for our downstream users to evaluate this map. This particular map is a lower priority for them, so it might be a while before they have the cycles to evaluate it. They are just starting to use the map in 54a70d4.

Great! Thanks for the update! Let's wait for the feedback then.

I need to fix minor conflicts due to the last PRs that got merged. However, let me know if this PR is mergeable @liangfok

Will do. Currently our downstream users are focused on other maps, but will definitely return to this one in the future.

Let's rename this map "dual_opposing_dedicated_right_turn_lanes." This is to highlight the fact that the dedicated turn lanes are right turn lanes rather than left turn lanes, which is more important than the fact that they are oriented North/South.

Let's rename this map "dual_opposing_dedicated_right_turn_lanes." This is to highlight the fact that the dedicated turn lanes are right turn lanes rather than left turn lanes, which is more important than the fact that they are oriented North/South.

Done.

Ready to review @liangfok

Just a quick note that @max-shaan is helping out with the downstream usage of this map.

I confirm we want to model both dedicated right-turn lanes, see attached screenshot.

Side note: No need to annotate any lanes as having oncoming traffic. We can add that later when the need arises.

@liangfok

Requested changes were addressed:

Extended lead-in and lead-out to 300m


media query tests / fixes

I added some basic tests, one of which currently fails but I'm not sure I know babel well enough to solve the issue. It seems that you would need to evaluate the template literal and look for "@" to transform differently. I may go at it more and try to fix it on this PR but I figured I might as well initially open it now.

Excellent! This is a great start to fixing this and super helpful setup. Let me know if  you investigate further. I was planning to sneak in some work on this plugin this weekend and this issue was on my todo list to investigate.

Let me know if you see any changes needed! I updated the testMatch string since it didn't work for me on Windows. The new one works on both my Ubuntu and Windows 10 machine.


fix build with samples on linux

fixes: https://github.com/Azure/azure-sdk-for-cpp/issues/3723

/azp run cpp - core

/azp run cpp - identity

/azp run cpp - core

/azp run cpp - identity

/azp run cpp - core

This looks to stop the sample from ever building.  What am I missing.  Also I see its still failing the core builds.

This looks to stop the sample from ever building. What am I missing. Also I see its still failing the core builds.

It was never a sample. It is a helper library for samples.  But it is just a header without any .cpp files,  so the library was created as an INTERFACE, which somehow works on Windows, but makes no sense for linux.
All we need is to make this header available for building samples

Also I see its still failing the core builds.

false positive.  I have triggered re-run failed.

closing on the benefit of https://github.com/Azure/azure-sdk-for-cpp/pull/3728


Add elementof test

This PR introduces an implementation for an elementof test.
In conjunction with the selectattr filter (or rejectattr for negation) this test can be used to extract values from one sequence having attributes that match at least one element of another sequence. It is a thin proxy for Python's in operator – like equalto is for ==. While it is trivial to implement this using a custom test by Jinja's users i'd argue that the addition of this test to the Jinja core provides feature parity with existing functionality with respect to equalto.
Use case:
When writing Ansible plays (which uses Jinja2) I hit a wall trying to filter a list of dicts on their name attribute inside of a Jinja2 expression.
When your rendering target is text (e.g. website templates) you can achieve something similar (not interchangable) using {% for x in seq1 %}{% if x.attr in seq2 %}{# access x or its attributes here #}{% endif %}{% endfor %}. However you can't use the latter construct when the consuming environment calls for an actual sequence being returned from Jinja.
Side notes (not related to Jinja, but the concrete use case inside Ansible playbooks):
It is possible to resort to external logic as a workaround (e.g. Ansible's when statement or dynamically create restructured, intermediate data representations).
Ansible can use a variation of the aforementioned for-if-template by incorporating the joiner() helper to create a YAML flow-style list but this creates a somewhat awkward contraption:
# […]
  - debug: "msg={{item}}"
    with_items: "[{% set j = joiner(',') %}{% for x in s1 %}{% if x.name in s2 %}{{ j() }}{{ x }}{% endif %}{% endfor %}]"

Example code (using this PR):
>>> import jinja2
>>>
>>> t = e.from_string("""\
... {{ list_of_dicts|selectattr("name", "elementof", selection_seq)|list }}
... """)
>>>
>>> t.render(list_of_dicts=[dict(name="foo", x=1), dict(name="bar", y=2), dict(name="baz", z=3)],
...          selection_seq=dict(foo=1, bar=42))
"[{'name': 'foo', 'x': 1}, {'name': 'bar', 'y': 2}]"

I hope this hasn't been discussed here previously. I'd like any opinions and criticism on this PR and alternative approaches for achieving the presented behavior using existing features in Jinja2,

hm.. why not call the test in? sounds more natural to me for what it does

I agree with @ThiefMaster here. I really would just call the filter in since it mirrors the in operator.

Agreed, I was probably trying to mimic equalto a little to much here. Naming the test in is probably for the best for its discovery.
Since I don't know the your preferences on the organizational side of PRs:
Shall I edit the PR and its OP (annotating the edited passages) by force-pushing or add a rename commit on top?

You can force push, or we can squash, whatever you're comfortable with.

Force-pushed the commit and edited the PR description to match it.


P9K crashes zsh when autocompleting ffmpeg

Describe Your Issue
When i type ffmpeg and press tab a few times, zsh fully crashes and terminal emulator no longer responds to any keypresses. It has to be fully closed and reopened again.

This is also accompained by weird cursor and screen flickers that were not there before.
Have you tried to debug or fix it?
I have tried to comment out most of my .zshrc, so my P9K settings, and any other zsh plugins, functions and aliases. This has stopped happening only after commenting out loading of P9K.
I did perform git bisect on this, and it seems to be caused by this commit: 4ccfd9dae5b34a053f8080ba8fc109d68ac611ca
And sure, after commenting out the lines that were added in this comit, the issue was fixed.
Environment Information
This information will help us understand your configuration.

What version of ZSH are you using? You can use zsh --version to see this.
zsh 5.7.1 (x86_64-pc-linux-gnu)
Do you use a ZSH framework (e.g., Oh-My-ZSH, Antigen)?
Oh-My-ZSH, Antigen
How did you install P9k (cloning the repo, by tarball, a package from your OS, etc.,)?
Installed via Antigen
What version of P9k are you using?
Lastest commit on the next branch. As of writing this issue, it is 2de4b99844bc9eacdb47a6f19cf8a7766fd96061
Which terminal emulator do you use?
Yakuake

FWIW, background_jobs segment works correctly in Powerlevel10k without trapping signals. Maybe give it a try?


Embedding forces HTTPS

When visiting embed.plnkr.co over http protocol, the iframe's source points to run.plnkr.co using https protocol. This causes mixed content errors for requests which are hardcoded to http. I'de suggest using a protolless url for iframe source //run.plnkr.co instead of https://run.plnkr.co.
Was thrown for a loop when my urls without protocol started using https while i was using http and the assets i was trying to load failed because https for them wasn't available. :)

Good

@iH8 can you confirm that this is appropriately resolved?

Sorry for the late response, works like a charm now @ggoodman, thanks a million


Allow individual api group and version for each config resource kind in Pilot config store

Currently all CRDs in Pilot share the same IstioAPIGroup: config.istio.io and IstioAPIVersion: v1alphav2
In this PR, we allow each resource kind to be able to define their own config group with a common suffix istio.io and version.
This allows

each feature to advance API versions at different pace.
the CRD group name to be reverse order mapping to proto package, as suggested in istio/api#343
separate public/internal storage of CRDs #3457

Given this is a config model change and deep refactor, the work is split into steps to avoid overly large PRs

[ ] Add Group/Version per each schema.
[ ] Change crd/Client to becomes a wrap of clientsets, each for one apiversion.
[ ] Update config store to identify schema by group version type instead of type.

Not until the last step is done, Pilot still continue to work with one config.istio.io and same version.

This is mostly done. The only remaining API resources in config.istio.io are Mixer policy and the HTTPSpec and QuotaSpec for mixerclient.

It makes much more sense to do this when Galley becomes the default config ingestion pipeline. Creating new API Group handlers, and deprecating old ones is trivial in Galley, but it is complicated for each component.


The AWS security group rules array/list in the examples section are empty.

The AWS security group rules in the examples are empty. Can there be example how the provider will add this security groups ?

@Nikoolayy1 example has been added to chapter  Feature specific examples at: https://github.com/f5devcentral/terraform-xc-aws-ce?tab=readme-ov-file#feature-specific-examples


Add support for easier math expressions in mp-units/math.h

Time for 2 more suggestions! <3
I began to adapt this utility:
      template<QuantityOf<isq::length> Distance>
      struct lognormal
      {
        /// @brief Dispersal location kernel parameter
        struct param_type
        {
          /// @brief Scale parameter homogeneous to a distance
          Distance a;

          /// @brief Shape parameter determining the shape of the curve (weight of long-distance events).
          double b;
        };

        /// @brief Probability density function
        /// @param r The distance radius from the source to the target
        /// @param p Parameters of the distribution
        /// @return The value of the expression \f$ \frac{1}{ (2\pi)^{3/2}br^2} exp( - \frac{log(r/a)^2}{2b^2} ) \f$
        static constexpr auto pdf(Distance r, param_type const& p)
        {
          Distance a = p.a;
          double b = p.b;
          assert(a > 0. * m && b > 0. && r >= 0. * m);
          return (1./(std::pow(2.*M_PI, 3./.2)*b*r*r)) * mp_units::exp(- (mp_units::log(r/a)*mp_units::log(r/a) / (2.*b*b) ) );
        }
      };

But I encountered two problems:

I had to pinpoint the library to use: std::pow when double are manipulated (mp_units::pow does not allow dimensionless types), and to use mp_units::pow when dimensions quantities are manipulated. This mix made the code a bit weirder than it should be, as I expected to be able to use mp_units::pow everywhere.
The mp_unit::log function seems to have been forgotten during the implementation ;)

I have been super happy with v2.0 so far! Great great job! I don't know exactly why but I find it easier to use than v1 :p

You need to explicitly mark b as a dimensionless quantity with q * one.

Neat! Thanks! Although it will not solve the non-existing mp_units::log problem, right?

It seems like some are marked constexpr, and others not.

I had to pinpoint the library to use: std::pow when double are manipulated (mp_units::pow does not allow dimensionless types), and to use mp_units::pow when dimensions quantities are manipulated. This mix made the code a bit weirder than it should be, as I expected to be able to use mp_units::pow everywhere.

You should actually use pow everywhere in an unqualified way. The correct namespace should be found with ADL. Additionally, as we do not have CPOs for math functions (see https://youtu.be/eUdz0WvOMm0?si=jGXAHLXXk97ejNJf&t=4332) you should do using std::pow; to bring the default implementation for fundamental types.

The mp_unit::log function seems to have been forgotten during the implementation ;)

math.h is known to be incomplete. We add things there one by one when they are needed. Please submit the PR with log and possibly all other functions that you might need.
When we have more time, we will probably extend the math.h but for now we have other priorities to scope on.

On a side note, i am under the impression that mp-units math functions are not constexpr, correct ?

All math functions aside the trigonometric ones are constexpr but they might not work at compile time for most of the compilers as the std ones are getting constexpr only from C++23 and C++26. See: https://wg21.link/P0533R9 and https://wg21.link/P1383R2.

Amazing! Thanks for the explanations 👍🏽 I will try to put together a PR 🤔

Hi mp-units team!
I am still having some problems regarding the math function. I tried marking b as dimensionless with  *one as in :
 template <QuantityOf<isq::length> Distance = mp_units::quantity<mp_units::si::metre>> struct MyStruct {
    // ...
    static constexpr auto pdf(Distance r, param_type const &p)
    {
        Distance a = p.a;
        double b = p.b;
        assert(a > 0. * m && b > 0. && r >= 0. * m);
        return b / (2. * std::numbers::pi * a * a * tgamma(2. / b)) * exp(- pow(r, b*one) / pow(a, b*one));
    }
 };

But I get a error: no matching function for call to 'pow(mp_units::quantity<mp_units::si::metre()>&, mp_units::quantity<mp_units::one(), double>)'

mp_units::pow takes the exponent ratio as a pair of template arguments.

@Becheler, it will always help us to answer you faster and more accurately if you provide a short repro in the Compiler Explorer.

Calling pow() on a quantity changes its type. This is why an exponent can't be a runtime variable. We have to know it at compile time to return a correct type (e.g., length, area, volume, etc.).

@mpusz @JohelEGP Indeed with a CE example it makes more sense.
Here it is : https://godbolt.org/z/cbvsezYos

Maybe you wanted to just do pow() on a numerical value of the quantity and not on the quantity itself? I do not know if it makes sense, but in such a case the quantity type will not change.

#483 rears it head?

It's not just about the unit but, first and foremost, about the quantity type.

Yeah, that's the first bullet of https://github.com/mpusz/mp-units/issues/483#issuecomment-1694675566.

Thanks guys!
At least in this expression, calling pow on the numerical values of the quantites does not change the dimension of the resulting expression (1/m^2). I will have to look at others distributions to make sure of it though (what defeats a bit the automation of dimension analysis but oh well, mp_units rocks as it is! 🥇
Here is a working example: https://godbolt.org/z/5n8dreTez


[Improvement] Merge spi.utils into common.utils

Purpose of the pull request
Code improvement, reduce the util class of the same name, and then we can reduce maintenance costs.
Related pr: #12321
Brief change log
Merge spi.utils into common.utils
Verify this pull request

Manually verified the change by testing locally.

Codecov Report

Merging #12502 (16c4f08) into dev (70aef3e) will decrease coverage by 0.01%.
The diff coverage is 17.18%.

@@             Coverage Diff              @@
##                dev   #12502      +/-   ##
============================================
- Coverage     39.38%   39.37%   -0.02%
+ Complexity     4214     4175      -39
============================================
  Files          1040     1037       -3
  Lines         39158    38958     -200
  Branches       4482     4458      -24
============================================
- Hits          15423    15340      -83
+ Misses        21958    21874      -84
+ Partials       1777     1744      -33

Impacted Files
Coverage Δ

...in/alert/dingtalk/DingTalkAlertChannelFactory.java
98.79% <ø> (ø)

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...r/plugin/alert/email/EmailAlertChannelFactory.java
98.82% <ø> (ø)

...olphinscheduler/plugin/alert/email/ExcelUtils.java
81.63% <ø> (ø)

...olphinscheduler/plugin/alert/email/MailSender.java
51.03% <ø> (+0.34%)
:arrow_up:

...ugin/alert/email/template/DefaultHTMLTemplate.java
79.24% <ø> (ø)

...plugin/alert/feishu/FeiShuAlertChannelFactory.java
97.36% <ø> (ø)

...dolphinscheduler/plugin/alert/http/HttpSender.java
47.69% <ø> (ø)

...eduler/plugin/alert/pagerduty/PagerDutySender.java
66.07% <ø> (ø)

...lphinscheduler/plugin/alert/slack/SlackSender.java
25.00% <ø> (ø)

... and 169 more

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


New event to enable follow again.

On the current flow, when you follow a profile (A), you receive an NFT as a  prove of following. If you transfer this NFT to a new address, this new address automatically follows the profile(A). To fix that, on the backend side we need to "mute" the current follow on transfer, but we need a way to enable it again.
With this new event, we create a new flow where the user can enable the follow again without losing the followNFT.

This greatly improves things and removes spam to a level. Now you can get back to it fully on-chain with events.
Top work @damarnez this solves a lot of things!


Initial commit of built in worker based upon the Ruby beaneater client.

This implements a basic was of creating a worker. Based loosely upon the the implementation in the Ruby Beaneater client.

Thanks for submitting this, but I think it belongs in a separate repository.
The scope of Pheanstalk is provide a PHP interface to the beanstalkd network protocol. Anything beyond that belongs elsewhere, and can use Pheanstalk as a dependency.

Thanks I will do this.


FEATURE: Add Cursor Tail Effector for Enhanced User Experience ✨

Feature Summary
I want to add a blue Cursor Tail Effector...
I propose adding a cursor tail effector to improve the overall user experience. This subtle but visually appealing effect will make interactions feel more dynamic and engaging for users, enhancing the visual feedback as they navigate through the site... 🔵✨
PA please assign me this task.
Description
,.
Proposed Solution
No response
Alternatives Considered
No response
Screenshots/Logs
No response
Additional Information

[X] I have searched for existing feature requests
[X] I am willing to help implement this feature
[X] I can provide more details or clarification if needed

@YadavAkhileshh


Fix Interface name validation

What this PR does / why we need it:
Reject non unique interface names by the virt-api webhook validation.
Special notes for your reviewer:
Just open a new pull request because the force push disable the option to reopen my preview PR
Release note:

Fix duplicate interface name.

ok to test