Custom request headers

Many APIs that I would like to use as an action target require header-based authentication. For example, it would be great to directly push events generated by inquery to Segment (HTTP Tracking API) to pipe the events to marketing tools and our data warehouse.
More generalized: Being able to set custom request headers (key/value similar to Postman) would be a great add-on to inquery to reduce the need to proxy the request when trying to integrate with more targets.

Might also be useful if the actual authentication key used in the header could somehow be referenced from an environment variable, or retrieved feom the db.


最新SDKflutter2.2.3 创建一个Demo 执行flutter build web  iOS设备浏览器可以打开网页 Android设备打不开是啥原因

最新SDKflutter2.2.3 创建一个Demo 执行flutter build web  iOS设备浏览器可以打开网页 Android设备打不开是啥原因
[✓] Flutter (Channel stable, 2.2.3, on macOS 11.2.2 20D80 darwin-x64, locale
zh-Hans-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.57.1)
[✓] Connected device (3 available)

waitForActivation(reg.installing ? reg.installing : reg.waiting);
这样就解决了

Hi @caofuqing
Using Google trnaslate
The latest SDKflutter2.2.3 creates a demo and executes flutter build web. The browser of the iOS device can open the webpage. What is the reason why the Android device cannot be opened?
waitForActivation(reg.installing? reg.installing: reg.waiting); So solved
Can you please provide a complete reproducible minimal code sample, and complete flutter build web --verbose logs building the minimal code sample?
Thank you

The latest SDKflutter2.2.3 creates a demo and executes flutter build web. The browser of the iOS device can open the webpage. What is the reason why the Android device cannot be opened?
waitForActivation(reg.installing? reg.installing: reg.waiting); So solved
Can you please provide a complete reproducible minimal code sample, and complete flutter build web --verbose logs building the minimal code sample?
Thank you

你好,我新建了一个Demo 没有做任何更改  直接使用命令编译了web,然后把build/web 里面的文件放到了服务器上,现在可以在手机端浏览器上访问
访问地址:  https://webapp.xinling001.cn/  (使用Android手机浏览器访问该地址白屏, iOS手机浏览器访问正常)
demo测试仓库:https://github.com/caofuqing/flutter_test.git
demo编译后的文件(build/web):https://github.com/caofuqing/flutter_test_build_web.git
我的操作日志:
talking@talkingdeMac-mini flutter_app111 % flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.2.3, on Mac OS X 10.15.7 19H15 darwin-x64, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] Connected device (1 available)
• No issues found!
talking@talkingdeMac-mini flutter_app111 % flutter build web --web-renderer html
💪 Building with sound null safety 💪
Compiling lib/main.dart for the Web...                             21.8s
talking@talkingdeMac-mini flutter_app111 % open .
我的手机请求情况(iOS正常显示,Android显示白屏): https://webapp.xinling001.cn/doc/image.jpg

flutter build web --web-renderer html --verbose.  log日志
talking@talkingdeMac-mini flutter_test % flutter build web --web-renderer html --verbose [  +93 ms] executing: sysctl hw.optional.arm64 [  +16 ms] Exit code 1 from: sysctl hw.optional.arm64 [        ] sysctl: unknown oid 'hw.optional.arm64' [   +9 ms] executing: [/Users/talking/Documents/flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H [  +16 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H [        ] f4abaa0735eba4dfd8f33f73363911d63931fe03 [        ] executing: [/Users/talking/Documents/flutter/] git tag --points-at f4abaa0735eba4dfd8f33f73363911d63931fe03 [  +19 ms] Exit code 0 from: git tag --points-at f4abaa0735eba4dfd8f33f73363911d63931fe03 [        ] 2.2.3 [   +8 ms] executing: [/Users/talking/Documents/flutter/] git rev-parse --abbrev-ref --symbolic @{u} [  +10 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u} [        ] origin/stable [        ] executing: [/Users/talking/Documents/flutter/] git ls-remote --get-url origin [  +13 ms] Exit code 0 from: git ls-remote --get-url origin [        ] https://github.com/flutter/flutter.git [ +101 ms] executing: [/Users/talking/Documents/flutter/] git rev-parse --abbrev-ref HEAD [  +13 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD [        ] stable [   +6 ms] executing: sw_vers -productName [  +15 ms] Exit code 0 from: sw_vers -productName [        ] Mac OS X [        ] executing: sw_vers -productVersion [  +15 ms] Exit code 0 from: sw_vers -productVersion [        ] 10.15.7 [        ] executing: sw_vers -buildVersion [  +13 ms] Exit code 0 from: sw_vers -buildVersion [        ] 19H15 [  +68 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update. [        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update. [        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update. [        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update. [   +2 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update. [        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update. [        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update. [        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update. [        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update. [        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update. [        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update. [  +50 ms] Artifact Instance of 'MaterialFonts' is not required, skipping update. [        ] Artifact Instance of 'GradleWrapper' is not required, skipping update. [        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update. [        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update. [        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update. [   +1 ms] Artifact Instance of 'FlutterSdk' is not required, skipping update. [        ] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update. [        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update. [        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update. [        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update. [        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update. [        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update. [        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update. [        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update. [        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update. [        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update. [        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update. [        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update. [        ] Artifact Instance of 'FontSubsetArtifacts' is not required, skipping update. [        ] Artifact Instance of 'PubDependencies' is not required, skipping update. [  +62 ms] Skipping pub get: version match. [ +127 ms] Generating /Users/talking/Desktop/未命名文件夹/flutter_test/android/app/src/main/java/io/flutter/plugins/GeneratedPluginRegistrant.java [  +69 ms] 💪 Building with sound null safety 💪 [  +47 ms] Compiling lib/main.dart for the Web... [   +5 ms] Initializing file store [  +17 ms] Done initializing file store [  +74 ms] Skipping target: web_entrypoint [   +3 ms] Skipping target: gen_localizations [ +253 ms] Skipping target: dart2js [  +56 ms] Skipping target: web_release_bundle [  +11 ms] Skipping target: web_service_worker [   +1 ms] Persisting file store [   +8 ms] Done persisting file store [  +23 ms] Compiling lib/main.dart for the Web... (completed in 444ms) [   +2 ms] "flutter web" took 855ms. [ +262 ms] ensureAnalyticsSent: 258ms [   +1 ms] Running shutdown hooks [        ] Shutdown hooks complete [        ] exiting with code 0 talking@talkingdeMac-mini flutter_test %

Hi @caofuqing
I tried to replicate the issue using your demo, executed flutter build web --web-renderer html --verbose to build web app, and hosted the build files on Netlify and I can load the web app on Android and iOS
https://hungry-hodgkin-91ffa3.netlify.app/
I also tried your web page link  https://webapp.xinling001.cn/, it takes 1-2 seconds to load but your pages also load on my Android device

Preview

Closing, as this doesn't look like an issue with Flutter itself If you disagree, please write in the comments and I will reopen it.
Thank you


Dataset to test loopclosures

I've been experimenting with Kimera and while the state estimation works great I was wondering  how I could best test the loop closure detection and PGO. As when I run Kimera (I use the ROS Wrapper) with the euroc dataset and verbosity set to print out loop closures it doesn't seem to find any. Is there any dataset available to test this better or a publicly recommended dataset that I could use?
Due to the current lockdown situation I'm in testing with real sensors is not possible for a while :)

Hi @WouterJansen, the Euroc dataset is the only one we support right now for loop closures.
Kitti would be nice, but it doesn't have real IMU data.
Alternatively, we could upload a rosbag doing a big loop-closure in a simulator (given current lockdown it might be the safest/quickest).

Hi @ToniRV ,
I have been testing Kimera with intel real sense D435i hardware.Just wondering if Loop closures will work on live captured data or not?

See #72 for I think the same issue I'm experiencing. Perhaps it's not a dataset issue but more an issue with the LCD.


deps: bump libcurl to 8.8.0

Description

Fix crash on macOS Sonoma: https://github.com/curl/curl/issues/11893
Disable more curl features.
Debug build do not enable sanitizers anymore.

Before merging

[ ] Assess binary size impact.

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

Project coverage is 94.48%. Comparing base (bb5358e) to head (449983c).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #135   +/-   ##
=======================================
  Coverage   94.48%   94.48%
=======================================
  Files          71       71
  Lines        3716     3716
=======================================
  Hits         3511     3511
  Misses        205      205

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


Solarized Dark - Find and Replace is largely unreadable

Several issues:

Text on "Find in Files" and "Replace in Files" is black on dark background
Text in boxes is mostly light gray on white
On hover/focus, the text in the search box becomes unreadable. In this screenshot there's "abcdef" with "def" selected and "abc" completely unreadable. The selected text is not that readable in itself.

I can take a swing at a fix if you could let me know what elements should I be looking for in the theme editor.
Microsoft Visual Studio Community 2019
Version 16.11.0
VisualStudio.16.Release/16.11.0+31605.320
Microsoft .NET Framework
Version 4.8.04084
Color Themes for Visual Studio 1.0.11

Did you ever make any progress on this?

Latest VS 2022 comes with Solarized Dark out of the box, so get rid of. The extension, select from options & you're good to go! 😉

For those of us stuck with VS2019, you can change the CommonControls.TextBoxBackground color using the old Color Theme Editor (or the new Theme Designer) to something more Solarized. I suggest FF003847.
I did not manage to find how to change the dark Tab text nor the purple Tab hover, but I can live with that. It was harder to tolerate the unreadable textbox.

While I appreciate your efforts & findings, I'm pretty much the opposite regarding being (forced to) stuck with 2019 - nobody should interfere with software engineers demanding the latest VS release. Nobody. Ever.


Update outdated spelling for Quenya

Replace 'k' with 'c' and 'kw' with 'qu'.

And 'q[^u]' with 'qu'.

And "ks" with "x".


KAFKA-8855; Collect and Expose Client's Name and Version in the Brokers (KIP-511 Part 2)

This PR implements the second part of KIP-511:

Connection Registry
Integration in the Processor and the KafkaApis
Integration with SaslServerAuthenticator
New Metrics
Enrichment of the RequestLog

Committer Checklist (excluded from commit message)

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

The PR is based on #7381. I will rebased it on trunk once #7381 will be merged.

retest this please

retest this please

Failed test:
JDK 8 and Scala 2.11

kafka.network.DynamicConnectionQuotaTest.testDynamicConnectionQuota

I've noticed, as a client, the client must pessimistically assume that the broker it is talking to has a max version of 2. Otherwise, with flexible versions, if the client assumes the latest version, the request will be written compact and an old broker will not understand the request and will close the connection. Compact requests can only be written if the client knows the broker is at least 2.4.0.
This seems to break the spirit of ApiVersions, which historically has been able to be handled even if the broker does not know the version.
When requesting with flexible metadata v3 and a short client name / version:
org.apache.kafka.common.errors.InvalidRequestException: Error parsing request header. Our best guess of the apiKey is: 18
Caused by: org.apache.kafka.common.protocol.types.SchemaException: Error reading field 'client_id': Error reading string of length 1131, only 15 bytes available

My question is: should the client behavior going forward be to

open a connect, pessimistically assume max v2, issue ApiVersions v2
see if the response indicates support for at least v3, and if so, reissue the request with the latest version (including the client software name / client software version)
?

If yes, I think the KIP should be updated to describe this new required behavior. If no, then I do not think ApiVersions request v3 should be flexible, as well as versions going forward. If not that, then I am not sure about this flexible dilemma.

@twmb You are right, the ApiVersionsRequest must remain backward and forward compatible. We have fixed this here: https://github.com/apache/kafka/pull/7479. Could you try with this patch and let us know if it does not work?

Ah, that's yet another reasonable fix! That makes sense. Sorry I missed that; I followed the links in KAFKA-8855!

I have opened a new PR with uses a different approach: https://github.com/apache/kafka/pull/7749 Closing this one.


[LMS-220] Generate and post Canvas data

It's running successfully in the VM as well, so ready for review.


[FEATURE REQ] Setup dashboard for build telemetry data

The dashboard is setup for build telemetry in Azure Data Explorer
https://dataexplorer.azure.com/clusters/ddazureclients/databases/AzureSDKJava


deps(bump): zcash_primitives, zcash_note_encryption and orchard deps at once

Motivation
We want to bump this 3 dependencies at the same time. This PR will replace the other 3 if we go this way:

https://github.com/ZcashFoundation/zebra/pull/5497
https://github.com/ZcashFoundation/zebra/pull/5498
https://github.com/ZcashFoundation/zebra/pull/5500

It also works around the fact that address() is now private.
I am pushing this now as we might want to add it to the release candidate.
Solution

Upgrade the 3 dependencies.
Fix address issue.

Review
Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?
[ ] How do you know it works? Does it have tests?

This PR looks great. I was working on a similar one, but my workaround for address() wasn't as smooth.

Apologies for working on something you were already working on. I didn't saw https://github.com/ZcashFoundation/zebra/pull/5500#issuecomment-1295563898

@oxarbitrage I have updates that make Zebra compatible with https://github.com/ZcashFoundation/zcash_script/pull/41, should I push them as a PR into this one, or would it be better to make a standalone PR?

That is great, i think we might want to have all the ECC or our own dependencies up to date in the release candidate but i am not totally sure if it is that important.
If your changes are not too big i think we can try to push to this one and merge all the dependencies alltogether.

@oxarbitrage I have updates that make Zebra compatible with https://github.com/ZcashFoundation/zcash_script/pull/41, should I push them as a PR into this one, or would it be better to make a standalone PR?

I just realized it would be best if we wait until we merge https://github.com/ZcashFoundation/zcash_script/pull/41, and then do the rest of the updates in Zebra, so a standalone PR will be a better solution.

Apologies for working on something you were already working on.

No worries at all, the only blocker was the address() function being private.

I just realized it would be best if we wait until we merge https://github.com/ZcashFoundation/zcash_script/pull/41, and then do the rest of the updates in Zebra, so a standalone PR will be a better solution.

I actually pushed a draft PR with the updates #5506 without https://github.com/ZcashFoundation/zcash_script/pull/41 being merged so that we can see what changes are needed.

I am pushing this now as we might want to add it to the release candidate.

Yes, we do, using outdated or duplicate cryptographic dependencies is a security vulnerability. (And it's a pain to audit.)

error: linking with cc failed: exit status: 1
https://github.com/ZcashFoundation/zebra/actions/runs/3357475081/jobs/5563308196#step:15:463

This is only happening in the GitHub Actions Linux build, so I cleared this PR's GitHub Actions caches, and the main branch GitHub Actions caches.


CLOUD: Add pricing underneath Edge Virtual Data Center

Here is the information needed, based on the pricing component from example site repo.
https://secure.threefold.me/code/#/2/code/edit/XIq3pN7rcghl3WOSE8F8KfLK/

[x] Kristine adds component to cloud site in correct spot
[x] Kristine OR Sam fills in data (@Vilnite feel free to give this to me once created)

Well detailed issue, thank you. Added.


Opacity resets after changing keyboard layout

Windows Terminal version
1.13.10336.0
Windows build number
10.0.22000.0
Other Software
No response
Steps to reproduce

Change opacity with Ctrl+Shift + wheel
Change keyboard layout ( default is Win+Space )

Expected Behavior
The opacity level should not change to the level specified in the opacity parameter.
Actual Behavior
The opacity is reset to the value specified in settings.json or up to the default value if it is not specified.

/dup #11522


[do not merge] cfrantz watchdog

@cfrantz i made a couple of minor edits and it looks like wdog can now reboot the device.
we just needed to enable it inside pwrmgr, you can ignore some of the other stuff, it was done mainly to allow the test to run in dv also.

Thanks Tim, I've incorporated your changes into my original PR an have updated the functest to test the behaviors I want tested.


Lower Accuracy on RAF-DB Dataset

Hi，
I attempted to replicate your results on the RAF-DB dataset using your notebook but achieved only about 88.07% accuracy, despite making no changes except for dataset replacement. This was consistent across tries, including on Kaggle with TensorBoard 2.12.
Could you offer any advice or adjustments to reach the expected accuracy?
Thanks for your work and help in advance.
Here is my notebook.
result

@cyinen Hi, have you tried enabling the sample_resizing? It could be due to this as the samples might not be using the full expression from the MobileNet with it still based on 224x224.

Sorry, what's sample_resizing means? I have resized the samples to 224x224 before input to the network.
Can you provide more details?
Thanks!

@JLREx Hello, my accuracy rate on the RAF-DB data set is also 88%, I don't know what the problem is, no code changes have been made

try experimenting with different seed values for shuffling ur train dataset and see if the results vary. It was the case for me.


Merge comments and guide history tabs

https://trello.com/c/CHVGwoTP/82-better-events-history
Also, style edition action buttons.

:+1:


Debug slow hot reload times in the frontend

@mattijsstam @MariaBee3 It turns out I am experiencing this as well:
wait  - compiling...
event - compiled client and server successfully in 19.2s (543 modules)

This is weird... I can make a change in a much larger Next.js app and get this:
wait  - compiling...
event - compiled client and server successfully in 657 ms (2008 modules)

It was nearly two weeks ago that I last ran the app properly, so whatever it is must have happened since then. Would you like me to take a look at it? I could do so early next week.


Add Tail Option to kit dev logs Command

Describe the problem you're trying to solve
Currently, the kit dev logs command retrieves and displays logs from the last or current development session. However, it lacks the capability to continuously stream logs in real time. Developers often need to monitor logs live to track the progress of ongoing tasks, identify issues as they occur, and ensure that their development environment is functioning correctly.
Describe the solution you'd like
Introduce a --tail (or -f) flag to the kit dev logs command that, when used, continuously streams and displays new log entries as they are generated. This behavior mimics the functionality of the Unix tail -f command, providing a seamless and integrated way to monitor logs in real time.
Behaviour: When the flag is used, kit dev logs enters a streaming mode, continuously fetching and displaying new log entries. The tailing continues until the user manually terminates the command (e.g., via Ctrl+C).

Will get this fixed today.

As there is no log rotation, I've added custom tailing.


Set up travis ci support

@doujiang24
Would you have time to review this change?
Also, it requires you to enable travis-ci for doujiang24/lua-resty-rsa.

It is worth mentioning that all tests were passed in this pr:
https://travis-ci.org/spacewander/lua-resty-rsa


Fix for iPhone X

Source: https://forum.ionicframework.com/t/iphone-x-and-ionic/105197/3

thanks :)


[BACKLOG-29232] - removing usage of guava Maps because of conflict on…

… cdh514 spark clusters.

Master PR https://github.com/pentaho/pentaho-kettle/pull/6564
8.3 PR https://github.com/pentaho/pentaho-kettle/pull/6566


Tree view is a little broken

Description
When you try to click (left or right click) on a file or folder, it clicks the next one below him (only if there is one on the same tree level).

Platform: windows
Atom version: 1.16.0-beta0
Seti-icons version: 1.4.3
UI Theme: One Dark

By the way, when I disable seti-icons it works everything as expected.
Mouse hovering .dev-elements folder with seti-icons active

Mouse hovering .dev-elements folder with seti-icons disabled

By the way, not always happens.
If the pointer is over the file/folder name, it will click that file/folder. If it's over the icon, it will open the one below only if that one has a custom icon.

I'm not sure I follow this one 100%. Does this happen on the latest stable? I noticed you are using a beta. Would this be something you could send over a PR for?

Sorry, but I never did anything on atom. I don't know where to start debugging things to at least write a test about it.
BTW, today I tried it on Atom 1.15.0 (latest release) and the bug is still there.

Please check the latest version of both Atom (1.17.0) and Seti Icons (1.5.0) and let me know if the issue persists.

I can confirm is fixed.
Now the icons are wrong placed :P

I'm using Atom 1.18.0-beta0 (I know, it is a beta, but is good to know right?).
Thank you @wyze.

Still the same theme? I am thinking it is Windows being weird, because it looks fine for me.

Yep, same theme

Just updated to 1.5.0 and I'm also seeing icons off in the tree view:

I'm using Windows 10, Atom 1.17.0 x64, One Dark theme

+1, this is happening to me too :/


Move toc-toggle to the right

The offset between the toc and the toggle is now constant
This also fixes #331, since the button is no longer placed where the
scrollbar is on windows.

Perhaps make these two distances the same?

Yep!
Now it aligns better with the article headline as well.


test, verify onClient=true

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


Alert Status - Cancel - cancels all caps

When the alert status is set to cancel, it will cancel all the CAP events associated with the alert
Acceptance Criteria

[ ] Changing the alert status to cancel will close the alert
[ ] Caps associated with the alert will be cancelled
[ ] When submitting the changed on the frontend component should display a warning letting the user know that disabling the alert will also cancel and CAP events that are associated with it
[ ] When submitted, the api will find associated caps, record their current status in history, proceed with cancelling the cap.

PR: https://github.com/bcgov/nr-rfc-alertauthoring/pull/116


Secret rotation doc update

Reason for Change:

This PR adds note about setting flags to enable this feature from azure provider helm chart.

Requirements

[x] squashed commits
[ ] included documentation
[ ] added unit tests and e2e tests (if applicable).

Issue Fixed:

fixes: #815
Does this change contain code from or inspired by another project?

[ ] Yes
[x] No

If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers:

Codecov Report

Merging #819 (c7274b6) into master (3528425) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #819   +/-   ##
=======================================
  Coverage   59.44%   59.44%
=======================================
  Files           7        7
  Lines         868      868
=======================================
  Hits          516      516
  Misses        316      316
  Partials       36       36


Open another profile in same directory

Description of the new feature/enhancement
Imagine I have a terminal open at some directory like c:\project\something
Now I want to open another terminal in this directory(wsl, or my custom added mingw terminal).
It would be good if I able to this by holding somthing and pressing that terminal profile hotkey(e.g instead of ctrl+shift+3 I go with ctrl+shift+alt+3 and new terminal startingDirectory is now same as where I am)
Little hard for wsl terminals I guess, but for the rest it can be helpful

Thanks for the request! This is a /duplicate of #3158.


Refactor gaps code

See #48.

Should gaps be a plugin?
If gaps functionaiity is part of a plugin, then the plugin should do all the work.
What to do, in general, if a file is requested that is produced by a plugin that is inactive. Should the plugin be activated? Should there be a general mechanism for this?

Fixed in 0.7.0


AerSimulator(method='statevector') is not statevector simulator

Summary
Fix #7010.
AerSimulator(method='statevector') is not a state vector simulator, but rather qasm simulator with state vector method.
If one wants to get the state vector, they should use SaveStatevector in AerSimulator.
Details and comments

Just as a comment - it was convenient in the past for a user to chose ideal result with a statevector simulator, or behave like a noiseless device (just shot noise) with a qasm like simulator. Algorithms would then tailor themselves to the statevector or qasm way according to the backend passed. If all one does now is pass in an Aer simulator then somehow else it would need to be specified. In general such backend/transpile etc config was in QuantumInstance - a central place for common logic that algorithms needed and could use etc.
I will note that switching to include_custom=True changes what is tested. No longer is the operator converted to matrix and the matrix expectation logic used - which would work with BasicAer statevector too - rather it ends up using custom expectation logic for Aer.

I think it would be better to deprecate statevector_simulator in the long run, and we need to deprecate the algorithm that depends on statevector_simulator first, or rewrite it to use SaveStatevector instruction.

@manoelmarques Do you have any comments given you had made changes in this area when Aer changed?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
15 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.03%) to 82.458%

Files with Coverage Reduction
New Missed Lines
%

qiskit/extensions/quantum_initializer/uc.py
2
88.65%

qiskit/pulse/library/waveform.py
3
90.38%

qiskit/utils/quantum_instance.py
10
66.76%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
49591

Relevant Lines:
60141

💛  - Coveralls

The algorithm user may not have access to the circuit prior to running it in order to add the SaveStatevector.  Currently QuantumInstance adds the SaveStatevector during the execute method (in case it is not present) if the backend name is aer_simulator_statevector or starts with statevector or is an instance of StatevectorSimulator:
https://github.com/Qiskit/qiskit-terra/blob/205c7fef9cbe7a663425cf776a2d61f6618fdff8/qiskit/utils/quantum_instance.py#L443

I understand the historical context, but there is a performance problem as original issue said: I believe only MatrixExpectation uses SaveStatevector, which is extremely slow and not worth using now. If there is an algorithm that needs SaveStatevector, I think it needs to be fixed on the algorithm side to make StatevectorSimulator deprecated.
Should the Optimization side be modified as well?
Or, do you have another approach to this issue?

If there is an algorithm that needs SaveStatevector, I think it needs to be fixed on the algorithm side to make StatevectorSimulator deprecated.

As I commented above the selection of backend caused the algorithm to adapt to it rather than the other way around. The only time we did things that way was with the AerPauliExpectation which was limited to VQE. Each algorithm did look at the backend that it was provided and alter itself to suit. Having to configure each one though to behave one way or another seems like a lot of duplicate configuration for the end user which as I also mentioned we centralized via QuantumInstance where the user could set up things and algorithms would check/adapt. Going forwards though, with the change to expectations etc and algorithms around such primitives this should certainly be revisited.


Trap and wrap exceptions with a Neo4jClientException

The Neo4jClient has an OnOperationCompleted event which can be subscribed to in order to get QueryText and TimeTaken on successful completion of a query.
When a cypher query throws an exception however, this event data isn't available.
I propose:

Wrapping the IRawGraphClient.Execute* internals in a try catch and throwing with a new type Neo4jClientException.
The InnerException property would be set and this type would have a  QueryText property with the query that caused the problem. QueryText wouldn't be in the ToString() so as not to leak potentially sensitive data into the logs.
Consumers can then in their app exception handler trap Neo4jClientException and optionally log the QueryText that caused the issue. It would also be useful in debugging failing unit tests.

Would a PR in this vain be accepted or is there another way to achieve what I intend?

The original intention was that OnOperationCompleted fires after all events. Otherwise, it should be called OnOperationalSuccessful. If that's not the current behaviour, I'd call it a bug.
The other ideas sound generally sensible to me.
There's already a NeoException type that gets thrown to represent server-side exceptions. The naming there is pretty generic because we don't know too much about what went wrong.
I'd be hesitant about a generic type like Neo4jClientException. I'd prefer something more specific, like CypherParseException where we do know, or QueryExecutionException where we don't.

The original intention was that OnOperationCompleted fires after all events

To double check my reading, here is the call chain:
A consumer calls CypherFluentQuery<TResult>.Results say
which at line 51 calls GraphClient.ExecuteGetCypherResults
which at line 936 calls ExecuteGetCypherResultsAsync<TResult>(query)
 which makes the HTTP call [at line 964] (https://github.com/Readify/Neo4jClient/blob/master/Neo4jClient/GraphClient.cs#L964) and fires OnOperationCompleted at line 973
If however the HTTP call throws then the catch at line 941 triggers and OnOperationCompleted will never fire.
In this case say, I'd wrap up the aggregate or unwrapped Exception into a QueryExecutionException
...and IRawGraphClient.ExecuteCypher(CypherQuery query) would get similar treatment at line 997 which is used by CypherFluentQuery.ExecuteWithoutResults and so on.

If that's not the current behaviour, I'd call it a bug.

I don't follow how that event could ever fire if there is a throw somewhere?

I'd be hesitant about a generic type like Neo4jClientException

Sounds fair - but I'd make a base class Neo4jQueryException which contains the QueryText parameter. Then my app can try, catch, log.fatal on that while the actual exception is still more specific. Would that be OK?

I don't follow how that event could ever fire if there is a throw somewhere?

That'd be the bug. :) I'm not suggesting it ever worked. Just describing the original design intent and naming.

I'd make a base class  Neo4jQueryException  which contains the  QueryText  parameter

Concept makes sense. Neo4j in a type name in .NET irks me. Considering it's query specific, how about CypherQueryException? (I don't imagine you were going to bother with touching the gremlins in the Gremlin code?)

cool - will do. No wasn't going to bother with Gremlin :)

Gentlemen,
Feedback requested:
I thought I would try to solve this issue of capturing query text during exception by honouring @tathamoddie 's intention that the OnOperationCompleted event should fire even during a throw event.
In the current source, this block of code is repeated several times unnecessarily in each Execute* method (1,2,3,4)
            CheckRoot();
            var policy = policyFactory.GetPolicy(PolicyType.Cypher);
            CheckTransactionEnvironmentWithPolicy(policy);

so I made a private ExecutionContext class and use that so as to DRY.
See line 1042 for an example of where it cleans up the code nicely. In the catch, just before throwing I fire the event with the exception attached to it.
So far so good.  It looks ok but then this test started failing because the code dropped through to Assert.Fail without throwing beforehand. This is strange since the exception does throw correctly, which I proved by rebranching off master (removing my changes) and testing for the exact exception message. That passes, so merging it back in the unit test passes demonstrating the exact right exception throws even though the code manages to reach Assert.Fail somehow.
I would like to
a) Check to see if you are happy with this pattern
b) I haven't done something horrible wrapping the async in a try / catch and modifying that test which curiously was redundantly checking for exception via the attribute and assert.fail. Maybe it was redundant...maybe it was some critical async test which I don't get.
c) Any preferences on where that private class ExecutionContext lives?
Full diffs
a1fd60916834f8aa159ea3d2ee6f40c5d7cac879 First commit with ExecutionContext added
7d20d26910772f2e8721e028cbeca6bd056bf004 Updating the unit test
Note - if this all looks good then I'll add more tests and dogfood before PR.
cc: @arturosevilla as the author of the transaction code

Hullo @neutmute - thanks for putting the effort into this!
Soooo.....
a) - I like that pattern, the ExecutionContext class is a nice addition and certainly tidy's up that area.
b) - I think you're right - and that it is a redundant check to have the Assert.Fail there as well.
c) - Private class should presumably be in the GraphClient. Normally I'd wrap in a #region and put at the bottom of the file, I thought about a partial class route with a new file GraphClient.ExecutionContext.cs but I think that implies the class is public, or at least internal and might give the wrong impression.

cool - will amend the (c) by putting at the bottom in a region.
re: partials graphclient.cs is a 1400 line beast. I'd be tempted to split it into a few different sections
graphclient.cypher.cs, graphclient.gremlin.cs, graphclient.main.cs say. Maybe even subfolder it - depending how strict you are with file location and namespace.

I would leave it as a single file for now - it'll be easier to look at the diffs when pulling the request in, which I think will make the process faster.
I think the addition of the partial files especially if nested is nice, but maybe would be better in another subsequent pull request? Whaddya think?!

yes agree it should be separate PR where the reorg is the only change
On Fri, Aug 14, 2015 at 11:27 PM, Chris Skardon<EMAIL_ADDRESS>wrote:

I would leave it as a single file for now - it'll be easier to look at the
diffs when pulling the request in, which I think will make the process
faster.
I think the addition of the partial files especially if nested is nice,
but maybe would be better in another subsequent pull request? Whaddya
think?!
—
Reply to this email directly or view it on GitHub
https://github.com/Readify/Neo4jClient/issues/106#issuecomment-131106221
.

yes agree it should be separate PR where the reorg is the only change

PR 108 now raised, closing


Readme updates

@adunndevster I made some edits to the Docs Authoring Pack and DOCS YAML readmes, please review. So far I haven't touched the feature list in the DOC YAML readme. Can you review those, especially the "New Features" section? I think it's out of date.

@adunndevster thanks - I made that change and some minor copyedits in the feature section. Did the features look correct? Nothing to add?

@meganbradley - I made one more copy edit suggestion, but to my knowledge this readme covers the features.

@adunndevster thanks, please approve so I can merge, then release at your convenience!


[fix] Add top margin to base twitter iframe

Twitter iframe should now align with main title
Before

After

Good call.


Cache busting of service worker + cache-control

Because the service worker is in the root right now we are not cache busting it nor adding the right cache-control header. We should find a way to do this.

@molant The cache-control header is set to no-cache in the sonarwhal-worker. What's the "right" cache-control header value that it should be changed to? It doesn't seem right to me to set a max-age for the service worker since the content is mutable.

I think that if you move the scripts to /static/scripts the gulp task should take care of updating the file names and the html that points to it.

Here's a couple of pointers, I hope they'll be of help if/when you decide to reintroduce service workers again:
For webhint-worker.js,

Should live in the root folder, enables full scope of the site.
Don't set a hash in the url, it may cause issues (sw always updated if the file is modified anyways):
→ https://developers.google.com/web/fundamentals/primers/service-workers/lifecycle#avoid_changing_the_url_of_your_service_worker_script
If use of the Cache-Control header is still desired (since the Chrome 68 changes, see link provided below) you will need to opt-in by setting updateViaCache to the preferred value. If that is the case, no-cache and/or max-age should suffice. (I suppose max-age=0 is recommended maybe because some browsers tend to default to max-age=15~ for resources where max-age is not specified.)
→ https://developers.google.com/web/updates/2018/06/fresher-sw

For sw-reg.js:

Can be placed anywhere, just point the registration to /webhint-worker.js.
Cache it, but revalidate w/  no-cache or update hash in file name. Should be safe since webhint.io is revalidating HTML pages.


Bc/downgrade errors to warnings

Summary
Treat reporting issues as warnings as they fail gracefully and can recover. Reserve errors for programmer errors such as misconfiguration and incorrect object types.

LGTM, thanks!


Move to Go 1.15.5

Type of change
Improvement (currency)
Description
Update to Go 1.15.5

Missed the Makefile: https://github.com/hyperledger/fabric-ca/blob/master/Makefile#L57

Missed the Makefile: https://github.com/hyperledger/fabric-ca/blob/master/Makefile#L57

👎


Add a new Issue and Pull request template

Currently the project is not having an issue and pull request template.
Adding one helps in having standardized information when submitting a pr or an issue.
I would like to work on this

ok, please file a PR :)


Fix StochSystem(::CoupledODEs) method

The parameter field became Vector{Vector{...}} after using the StochSystem(::CoupledODEs) method.

Thanks for spotting and fixing this @oameye!


MQTT Support

It would be cool if MQTT Support could be added as well. This makes the possibilities and links endless.
Thanks

Hi there,
I am sorry but I am not planning to add MQTT Support.

Hi there,
I am sorry but I am not planning to add MQTT Support.


fix(Cache): always invoke lake to fetch ProofWidgets

Always invoke lake to fetch the latest ProofWidgets cloud release on a lake exe cache get.  This requires lake because cache has no simple heuristic to determine whether the ProofWidgets JS is out-of-date.

bors merge


[Subtask] Support python code style and  code style format

Describe the subtask

support code style check
support format code style automatic
pylint or flake8 + yapf

https://flake8.pycqa.org/en/latest/
https://github.com/google/yapf
Parent issue
#2113

+1, for Python we should have this. Thanks Peidian for creating this issue.


the taskbar is bugged since the last windows update including news and fields of interest

Édition	Windows 10 Professionnel pour les Stations de travail
Version	21H1
Installé le	05/04/2021
Build du système d’exploitation	19043.1023
Expérience	Windows Feature Experience Pack 120.2212.3740.0
the taskbar is bugged since the last windows update including news and fields of interest

Replica of https://github.com/ChrisAnd1998/TaskbarX/issues/649

I was able to get around this bug between 21H1, News and Interests, and Taskbar X by using the <IP_ADDRESS> pre-release and selecting "Show Icon Only" for the News and Interest toolbar. If either of the other options (Hidden or Show Icon and Text) is selected, everything gets jumbled up on the right and often even the entire system tray disappears.
This approach even works after reboots, etc. so I thought I'd share my findings with everyone. I hope it helps while we await a bug fix (and/or the ability to turn off this News toolbar entirely). :)
Note that my OS build is x64 21H1 19043.1023. YMMV.

I was able to get around this bug between 21H1, News and Interests, and Taskbar X by using the <IP_ADDRESS> pre-release and selecting "Show Icon Only" for the News and Interest toolbar. If either of the other options (Hidden or Show Icon and Text) is selected, everything gets jumbled up on the right and often even the entire system tray disappears.
This approach even works after reboots, etc. so I thought I'd share my findings with everyone. I hope it helps while we await a bug fix (and/or the ability to turn off this News toolbar entirely). :)
Note that my OS build is x64 21H1 19043.1023. YMMV.

How I install this new release? Help me, please

How I install this new release? Help me, please

https://github.com/ChrisAnd1998/TaskbarX/releases/tag/<IP_ADDRESS>

How I install this new release? Help me, please

Assuming you have the file grumpydev provided, the steps I use are as follows:
Hit the Right Mouse Button over your own TaskbarX Configuration icon from your previous installation.
Select "Open File Location"
That folder contains the current install. I move all those files into a New Folder, which I rename "(old)"
I then unarchive the new version and copy all of those files into this now empty folder for the previous install.
I hope that makes the process clear to you. :)

I get it. Thanks for answer it

Please make that an official Release for the Microsoft Store.

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


KAFKA-4943: Make /config/users with SCRAM credentials not world-readable

cc @junrao

@junrao When it comes to security issues like this one, the default position should be to backport unless there's a reason not to. It's true that there is a workaround, but people may not realise that they have to do it until it's too late.

I'd like to see it in 0.10.2 unless there is special complication in cherry-picking.

@ijuma @junrao @gwenshap Thank you for the review. I have made the updates.

@rajinisivaram @gwenshap Can you please check https://github.com/apache/kafka/pull/2829


Error when running example code

Hi,
When running the example code on the front page, I get

Exception ValueError: "Buffer dtype mismatch, expected 'DTYPE_t' but got 'long'" in 'skgpuppy.UncertaintyPropagation2.UncertaintyPropagationApprox.propagate_mean' ignored
Traceback (most recent call last):
File "test.py", line 58, in
out_mean, out_variance = up.propagate_GA(mean,Sigma)
File "UncertaintyPropagation2.pyx", line 266, in skgpuppy.UncertaintyPropagation2.UncertaintyPropagationApprox.propagate_GA (skgpuppy/UncertaintyPropagation2.c:6461)
File "UncertaintyPropagation2.pyx", line 289, in skgpuppy.UncertaintyPropagation2.UncertaintyPropagationApprox.propagate_GA (skgpuppy/UncertaintyPropagation2.c:6250)
File "UncertaintyPropagation2.pyx", line 259, in skgpuppy.UncertaintyPropagation2.UncertaintyPropagationApprox._get_sigma2_and_variance_rest (skgpuppy/UncertaintyPropagation2.c:5718)
ValueError: Buffer dtype mismatch, expected 'DTYPE_t' but got 'long'

What can I do?
Thanks.

Unfortunately, I have no time right now to actively maintain this project and to take care of that bug. I suggest it might be an issue with Cython. You can try to modify the setup.py and set USE_CYTHON manually to false. Then an alternative pure python implementation will be used. (Which is much slower unfortunately). (There is also a deprecated scipy weave implementation.)

Thank you very much, I understand. I was able to overcome this by directly working from the files without the installation.
However, the output of the propagation is one dimensional although the state space is 2-d. Perhaps you have any directions on that?
Thanks again.


FileUpload: Fix top margin CSS

Description

Fixes #6480's the margin issue by removing the top margin.

How Has This Been Tested?

Visually
Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Why does it have label "breaking change", but the type of changes says the opposite?
Can we merged this or not?

@ScarletKuro it's because it might break layouts due to the CSS change. I will update the text to reflect this.

@ScarletKuro as I seen you checking other changes for v7, this should also be included/looked at.

Merged into v7. People upgrading from v6 might experience layout breaks.


Add to the docs how to DRY on multiple crontab lines and use a shell script instead

When setting up multiple lines in crontab, repeating $(command -v docker) run --rm --env-file=/home/bob/.bitcoin-dca ghcr.io/jorijn/bitcoin-dca:latest [arguments...] is annoying and error prone.
Proposed solution
I'd recommend to add to the Tips section of the docs file cron examples how to create a shell script file to reference in the crontab and pass the arguments from the crontab to this shellscript.

I'm liking this idea. Thanks! I will pick it up when I have some time. If anyone wants to do this in the meantime, please! :)

Yes I am interested in this too!


Add signal class and helper function

For now, this is just a proof of concept.
Alternatives are very welcome.
And suggestions for improvements, too.
For now, there are no doc changes yet. I'll do that if we decide to go that way ...

What is supposed to happen when a function receives multiple signals with different sampling rates?

raise an error?
perform automatic resampling?

@trettberg Both behaviors are possible choices, we can decide once we actually have a function that takes more than one signal. We could even add an argument that specifies if resampling should be used or not.

My line of thought was something like:
If different sampling rates are not handled automatically, we don't have to store them with the signals.
The constant in defs might be enough?

@trettberg I agree with your edited comment. The sampling rate could be implicit (only handled by defs), but I think this has a high potential for errors.
I think it's best to make it explicit and obligatory. This is the case in this very PR.
However, a signal class could be created without explicit sampling rate and an explicit sampling rate could be used without a signal class.
As I said, IMHO the sampling rate should always be explicit, but there could still be a better alternative for the signal class proposed here.
Are there any counter-proposals?

From what i've tried so far, this works nicely for keeping track of processing delays.
I'm with it!
A suggestion for later:
IMHO this begs for overloading the class's arithmetic operators.
For example, adding driving signals for two virtual sources with d1 + d2 should take care of time offsets (and sample rates).

I'm not sure if overloading the operators is the right way to go ... but I also don't know if it will be even needed, we'll see about that in the future.
I don't think it's a good idea to build too many features into the Signal class which would in the end lead to something like http://www.briansimulator.org/docs/reference-hears.html#sounds.
We should definitely consider using free functions instead of methods and overloaded operators.
If we use methods, we create a lock-in for our users, since they are forced to convert to a Signal object before they can do anything useful with it.
Overloaded operators might have to deal with some strange edge cases like:
somedata = ...
sig1 = somedata, 44100
sig1 = sfs.util.as_signal(sig1)
sig2 = sig1 + ([1], 44100, 0.007)

I don't think this is the way we want to go, but it is definitely a possibility ....
Consider, OTOH, using a (hypothetical) free function:
somedata = ...
sig1 = somedata, 44100
sig2 = sfs.util.add_signals(sig1, ([1], 44100, 0.007))

I think this is better, but it would probably be even better not having to add signals at all.
Since the main feature of the proposed signal class is the delay, we should probably rename it to something more specific?
What about DelayedSignal and as_delayed_signal()?
This should show that it is not a generic signal class (with many features) but rather a very specialized thing.

I switched to DelayedSignal and updated the documentation.
Any comments? Suggestions for improvements?

Superposition of signals is necessary e.g. to generate driving functions for data-based SFS.
Acoustic room modeling (think image sources) is another application.
I do not suggest to stuff arbitrary functionality in the class but to get the math right:
For ease of illustration, assume all signals have

the same number of channels C,
a sampling rate of 1,
integer delays.

These signals live in the same (C x N)-dimensional vector space, for some N large enough.
The delay offset is merely an implementation detail to avoid lots of zeros.
Compare to the sparse matrix classes which (of course) do matrix arithmetics.

OK, we might need signal superposition at some point, but is it something that has influence on the current PR?
I think we can decide later if we use operator overloading or not.
But the decision if we use integer or fractional delays indeed has a lot of influence here.
As far as I understood, we want fractional delays, right?
The relation to sparse matrices is very interesting, but as you say, it's only valid for integer delays.
And even then, there is the difference that we need a way to align the signal data on the time axis.
Another detail is that our signals (currently) cannot have "holes". If you add a signal that is non-zero in the first second to another signal that only plays in the 10th second, you'll have to fill the time in-between with zeros.

The delay offset is merely an implementation detail to avoid lots of zeros.

No, it's not.
It would be if we only used positive delays, but it is actually required for negative delays. Or we would need to set some arbitrary maximum negative delay somewhere.
Or do you know a different way how we could implement this?
Another possibility would be to use one delay value per channel instead of one global delay for all channels.
Instead of (data1, fs, -0.007), we could have something like (data2, fs, [0.02, -0.005, 0.006, 0.001, -0.007]) (assuming 5 channels).
We could even make it optional: A scalar value would be applied to all channels, a sequence would have to have the right length and its element would be applied column-wise.
What about that?
But then we also might think about having numeric weights per channel instead of applying the weights destructively ...

OK, we might need signal superposition at some point, but is it something that has influence on the current PR?

Go ahead!

The relation to sparse matrices is very interesting, but as you say, it's only valid for integer delays.

Mathematically it's the same fractional delays (even with multiple sampling rates).
All our signals live in the same vector space.
(Implementation is a different thing though, that's true.)
I think we should stick with a simple DelayedSignal class, that is a single delay per signal:

The data part can always be used "raw" with consistent time alignment.
Nobody would be forced to use the custom functionality.
For the same reason, it's probably a good thing that signals cannot have "holes".
When using multiple signals, manual time alignment is IMHO much more tedious than manual weighting. So I don't see why DelayedSignal should care about individual channel weights.

We should go ahead with the class and check its application in the toolbox.

OK, I've merged it, thanks for all the comments!
Of course this is not set in stone, we can still change any aspect of it ...


fix nixos service:

also bumps iohk-nix to rev with alpha1/alpha2
Issue

This PR results/does not result in breaking changes to upstream dependencies.

Checklist

[ ] This PR contains all the work required to resolve the linked issue.

[ ] The work contained has sufficient documentation to describe what it does and how to do it.

[ ] The work has sufficient tests and/or testing.

[ ] I have committed clear and descriptive commits. Be considerate as somebody else will have to read these.

[ ] I have added the appropriate labels to this PR.

bors r+


Bump version to 0.3.2

@umezawatakeshi
I am doing following the release procedure.
https://github.com/cybozu-go/nyamber/blob/main/docs/release.md


Reverts to using Union for typing

As | is not supported for typing in Python < 3.10 references to this have been replaced with Union.
Closes #141

Thanks, works fine with the default Python on a new Ubuntu 20.04 VM.


autocomplete did not work

Correcting the attribute. tested and working :)

Coverage remained the same at 100.0% when pulling baf7b713b01a90e311669f9a2f8d405a36b79c1f on jimnordb:patch-2 into c1cd6d7dfcadc2223a770d796ae2e9cdfb66a78a on charliekassel:master.


Typology

Hi @hannah-rae @jacobsn thinking we could start to consolidate some of the fields already, in particular the 'types'.
The 'creation_method' is the same concept as was intended for the 'determination_method' in core, perhaps you could comment on the allowed values in the enum? The list was based on the outcome of AgGateway WG17: Field Boundaries Use Cases and Definitions. I expected there'd also be more method-specific detail (e.g. the ML model and version) so good you have that in the AI extension but you may want to rename it to match.
I think we could also propose to put the 'boundary_type' into core somehow. For reference, this is what AgGateway WG said on this concept:

A field boundary can fit into one, or even all four categories below, it is not related to how the boundary was created but what it is used for.
Conceptual Field Boundary: How the grower thinks of the field, this is the boundary they would share with service providers to allocate information at the highest level of the field concept within their operation

This is a Field per the definition, with a boundary. (Field: "A named and farmer-accepted physical space where production agriculture takes place used to partition and identify data.")

Operational Boundary: The boundary for a specific set of field operations as defined by or for the grower.

Management area for the purpose of sharing with service providers for analysis or recommendations
Area for a specific field operation for a specific machine

Economic Defined Boundary: A boundary used to define, plan, and analyze a field for business purposes as defined by or for the grower.

GHG/Sustainability/Traceability
Ownership/Splits
Billing

Administrative Received Boundary: A boundary used to organize data that is defined by some other authority and is generally not easy to change:

Governmental programs
Insurance
Legal land description

Notably, it was seen as not mutually exclusive (i.e. a boundary can have multiple types...)
Personally I feel that there is something not captured here which is what is the thing on the ground the geometry represents, which is not quite the same as its purpose - for example does it represent a physical boundary feature, or a single crop.

Thanks for getting the discussion started.
I'm wondering if we should separate this into two notions:

A universally agreed-upon definition of a given boundary, which we would move to the core.
A dataset-specific definition of a label for the region, which we keep in the extension. This is nice because it grants flexibility to the person building a dataset.

FWIW we've been using https://github.com/Kitware/kwcoco on another project to represent similar data for training ML models. Perhaps there are some lessons to be drawn from that.


Gui Optimizations

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Other information:

Status

[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Needs input from @jstarpl @ianshade


update raster source url?

How do I update the url in a raster source?
This works fine when the layer source is GeoJSONSource:
m.set_data(JSON_LAYER_ID, new_data)  # works great, but only for GeoJSON source
is there an equivalent for raster layers?  Maybe something like:
m.set_raster_url(RASTER_LAYER_ID, new_url)   # no such method
I tried this, but it doesn't do anything useful:
m.set_layout_property(RASTER_LAYER_ID, 'url', new_url)   # nope
Thank you.

You have to remove and re-add the layer with the new url.

You are right. There are not mappings for all JavaScript Map methods but with the powerful Map.add_call method you can trigger any maplibregl.Map method. The first argument is the name of the JavaScript method followed by the arguments of the method itself.
In your case you have to take a look at Map.removeLayer. As you can see the only argument ist the layerId, so in Python it looks like this:
m = Map()
# ...
m.add_call("removeLayer", layer_id)

That works great!  Thank you.
BTW, I removed both the layer and the source, then added them both again.  Was that necessary?
def layer_update(params):

    m.add_call("removeLayer", LAYER_ID)
    m.add_call("removeSource", SOURCE_ID)

    new_url = f"https://whatever.com/{z}/{x}/{y}?{params}.tiff"

    raster_tile_source = RasterTileSource(
        tiles=[new_url],
        ...
    )
    m.add_source(SOURCE_ID, raster_tile_source)

    wx_layer = Layer(
        type=LayerType.RASTER,
        source=SOURCE_ID,
        id=LAYER_ID,
    )
    m.add_layer(wx_layer)


STYLE: Prefer python39+ coding style recommendations

Only python3.9 and greater are supported, so remove
conditional code supporting <python3.9 statements.
Prefer native-supported type hints rather than
separate imports from the typing library.
Remove the redundant "r" designation for the default
file open behavior.
PR Checklist

[x] No API changes were made (or the changes have been approved)
[x] No major design changes were made (or the changes have been approved)

only mac builds have a single python test failure:
Traceback (most recent call last):
  File "/Users/runner/work/1/s/Modules/Filtering/Smoothing/wrapping/test/MedianImageFilterTest.py", line 69, in <module>
    assert Sequence[int] in get_args(radius_hints)
AssertionError
itkTestDriver: Process exited with return value: 1

Double checking to ensure that this is a transient issue.

I verified that this is a regression on Mac with Python builds.
It seems like there is a problem with the expected structure of type hints.  Will look at this soon.


SteamVR dashboard UI

Add On and Off btns

I do not know why this may be necessary and I am not going to implement it in the near future.


Clean up migration systems

This issue is imported from pivotal - Originaly created at Jul 5, 2017 by Thijs Kinkhorst
There are several database migration systems in place in EB. Also they contain a long history of database changes. The database currently has tables DATABASECHANGELOG, DATABASECHANGELOGLOCK, db_changelog and migration_versions.
This adds a lot of noise to the system. Also when installing a new version, a user will be creating stuff that gets dropped in a later migration again. This should be cleaned up.
There's no need to support direct upgrades for users e.g. running 0.1 to 5.4.
I propose the following approach:

Pick an EB version X.Y in which the newest migration system was already in use and that has a reasonably clean db schema;
Remove any older migration systems, consolidate the schema of X.Y into the migration so that a new install will start from that schema.
Document in the release notes that if upgrading from a version < X.Y, you need to upgrade to X.Y first and then to the current version.

The migration system that used the tables DATABASECHANGELOG and DATABASECHANGELOGLOCK was last used in 2011, the source code contains no reference to those tables and they can be safely removed. I included a migration that removes those tables if they exist.
EB currently uses dbpatch and doctrine migrations. I've rewrote the last parts of EB that depend on dbpatch (db_changelog) to use doctrine and doctrine migrations. DbPatch can now be removed so doctrine is the one and only tool to manage the schema.
Part of the release notes should be: if you upgrade from EB <5.0, upgrade to EB5.0 before upgrading to EB5.7.
PRs:

https://github.com/OpenConext/OpenConext-engineblock/pull/493
https://github.com/OpenConext/OpenConext-deploy/pull/166 (Joris Steyn - Feb 19, 2018)

Todo: discuss and fix failing build in https://github.com/OpenConext/OpenConext-deploy/pull/166 (Joris Steyn - Feb 19, 2018)

The following tables will be dropped in EB5.7 (if they exist):

DATABASECHANGELOG
DATABASECHANGELOGLOCK

This table is part of DBPatch and will be dropped in EB5.7:

db_changelog

These tables are part of EB5.x but were never actually used. They will too be dropped in EB5.7:

allowed_connection
sso_provider_roles
saml_entity

The following tables are and will continue to be part of EB:

consent
migration_versions
saml_persistent_id
service_provider_uuid
sso_provider_roles_eb5
user
(Joris Steyn - Feb 21, 2018)


Zombies with Weapons

Well, Silvercatcher once began to do this, but he didn't finish it.
Now it's my turn to honour his work and finish it ^^

Added in e5583d6b25243a8fcd6ca195f030bc87b224189b
Could be buggy, but didn't experience any bugs myself for now ^^


Ingress example not working with current NetworkPolicies

What happened?
After migrating to newest version (commit https://github.com/prometheus-operator/kube-prometheus/commit/73bbec9659941e95301d684bd54d39b9644b21b5) I lost all ingress connectivity to grafana, alertmanager, and prometheus.
Did you expect to see some different?
Services are accessible when used with Ingress example
How to reproduce it (as minimally and precisely as possible):
Run stack with Ingress example
Environment

Prometheus Operator version:
Insert image tag or Git SHA here

Kubernetes version information:
kubectl version

Kubernetes cluster kind:
insert how you created your cluster: kops, bootkube, tectonic-installer, etc.

Manifests:

insert manifests relevant to the issue

Prometheus Operator Logs:

Insert Prometheus Operator logs relevant to the issue here

Prometheus Logs:

Insert Prometheus logs relevant to the issue here

Anything else we need to know?:

Additionally seems like grafana cannot reach prometheus and alertmanager

Additionally seems like grafana cannot reach prometheus

We have a community contribution attempting to fix this at https://github.com/prometheus-operator/kube-prometheus/pull/1710, but I agree that it's a quite serious bug so we might fix it quickly 😬

Adding (import 'kube-prometheus/addons/networkpolicies-disabled.libsonnet') does not seem to be disabling the networkpolicies. Is there something I'm missing?
local kp =
	(import 'kube-prometheus/main.libsonnet') +
        (import 'kube-prometheus/addons/all-namespaces.libsonnet') +
        (import 'kube-prometheus/addons/networkpolicies-disabled.libsonnet') +
	(import './config.libsonnet');

Adding (import 'kube-prometheus/addons/networkpolicies-disabled.libsonnet') does not seem to be disabling the networkpolicies. Is there something I'm missing?
local kp =
	(import 'kube-prometheus/main.libsonnet') +
        (import 'kube-prometheus/addons/all-namespaces.libsonnet') +
        (import 'kube-prometheus/addons/networkpolicies-disabled.libsonnet') +
	(import './config.libsonnet');

This does work for me, now.

The latest release v0.11 introduces restrictive network policies that do not allow access to any of the pods from anywhere but internal use. This results in timeouts (HTTP 504) from load balancers.
In order to get access, relevant network policies need to be added.
In my case, I want to expose Prometheus, AlertManager and Grafana from load balancers - pods ingress-nginx in the namespace ingress-nginx. (Load balancers are restricted to be accessible only from VPN.) And I want the NetworkPolicy configuration to be part of the main Jsonnet file so that the network policies are generated together with the rest of the kube-prometheus stack.
The following configuration in Jsonnet does the trick (example redacted to contain only relevant configuration):
local kp =
    (import 'kube-prometheus/main.libsonnet') +

    // ... some other configuration here

    alertmanager+: {
        networkPolicy+: {
            spec+: {
                ingress+: [{
                    from: [{
                        podSelector: {
                            matchLabels: {
                                'app.kubernetes.io/name': 'ingress-nginx',
                            },
                        },
                        namespaceSelector: {
                            matchLabels: {
                                'kubernetes.io/metadata.name': 'ingress-nginx',
                            },
                        },
                    }],
                    ports: [{
                        port: 'web',
                        protocol: 'TCP',
                    }],
                }],
            },
        },
    },

    prometheus+: {
        networkPolicy+: {
            spec+: {
                ingress+: [{
                    from: [{
                        podSelector: {
                            matchLabels: {
                                'app.kubernetes.io/name': 'ingress-nginx',
                            },
                        },
                        namespaceSelector: {
                            matchLabels: {
                                'kubernetes.io/metadata.name': 'ingress-nginx',
                            },
                        },
                    }],
                    ports: [{
                        port: 'web',
                        protocol: 'TCP',
                    }],
                }],
            },
        },
    },

    grafana+: {
        networkPolicy+: {
            spec+: {
                ingress+: [{
                    from: [{
                        podSelector: {
                            matchLabels: {
                                'app.kubernetes.io/name': 'ingress-nginx',
                            },
                        },
                        namespaceSelector: {
                            matchLabels: {
                                'kubernetes.io/metadata.name': 'ingress-nginx',
                            },
                        },
                    }],
                    ports: [{
                        port: 'http',
                        protocol: 'TCP',
                    }],
                }],
            },
        },
    };


Feature / Hover Link Style

This PR adds a new link style called hover. With this style a link is only underlined when hovering over the text.

Very nice – thank you! This will need to be documented at some point.


Setup service monitor for prometheus and alertmanager

Fix #278

Test this please
Just to check the assets pipeline.

Test this please

Test this please

test this please

@mxinden I already tested this, it's the same as I have in my  cluster

@gianrubio Sorry for that, tried to trigger our Jenkins CI, but somehow the hook seems to be broken.
LGTM. If no other concerns I will merge it.

@gianrubio Sorry for that, tried to trigger our Jenkins CI, but somehow the hook seems to be broken.

So let's wait the test pass, so I'll not break other developers :)

test this please

@gianrubio Thanks for the contribution! Sorry for the trouble and the notification spam.


Potential regression in peak memory for test_join_big[0.1-tasks]

https://github.com/coiled/benchmarks/actions/runs/4888526166/jobs/8735019095#step:5:23
Saw this one failing twice in a row, difference seems to be small.
cc @hendrikmakait

There's an issue for that already: https://github.com/coiled/benchmarks/issues/711.


Data API - /account/balance

what /account/balance needs to include the following information

unlocked balance
total balance (including both unlocked balance and locked balance)
vesting schedule
account nonce

why this is essential for our Custody team to be able to show the client their account details.
Note: we will followup with where exactly we want to place these information. But we will be leveraging SubAccounts to expose these information in the API
References: MsigGetAvailableBalance returns the unlocked balance of a Msig, the actual “balance” of the Msig actor is the total balance (including locked). There isn’t an endpoint to get the actual vesting schedule, but StateReadState will get you the StartEpoch and UnlockDuration (it’s always linear vesting).

request
--header 'Content-Type: application/json' \
--data-raw '{
    "network_identifier": {
        "blockchain": "Filecoin",
        "network": "testnet"
    },
     "account_identifier": {
        "address": "t0128015",
        "sub_account": {
            "address": "LockedBalance",
            "metadata": {}
        },
        "metadata": {}
    }
}'

response
{
    "code": 42,
    "message": "unable to get locked balance for address",
    "retriable": true
}

@qiwu7, Filecoin team have recently provided me with better way to get the state of a multisig account.
I'm adding that fix on the next PR.


[953] Pure attribute to avoid clones

Fixes #953

@stepan-beresnev Thanks for submitting this PR. We will review ASAP.

Hey guys! When can we expect this to get mainlined?

Good idea. However I think this should be reviewed in more details, because this is not standard .NET behavior, and merging it could cause problems down the line with people using this feature.
A few points to consider:

Usually a struct that you don't want cloned could be a class instead.

As of C# 7.2-8.0 there has been done quite a lot of work in the language standards and compiler to deal with this:

a) Immutable structs can be declared as "readonly struct" to enforce immutability and tells the compiler it can safely be passed around as a reference because it can't change (no cloning required).
b) Properties and methods in a struct that don't change it can also be declared as "readonly". This avoids the need to clone the struct before invoking that method/property, because compiler knows it won't change it.
c) If you want to mutate a struct given as an argument to a method, you can pass it with the "ref" keyword. Tells the compiler not to clone the struct, but pass it as a reference which the method may or may not modify, similar to the example in the PR.
d) If you don't want to mutate, nor have to clone it, on a per-method basis that accepts a struct parameter, use the "in" keyword.
My opinion is that it's not a good idea to merge this, but instead try to implement these new C# features that serve the same purpose.


EmrEtlRunner: replace elasticity by the aws-sdk

The official sdk is better supported and provides more control.

apparently we can configure retries https://docs.aws.amazon.com/AWSRubySDK/latest/AWS/Core/Configuration.html thanks @jbeemster


Benchmark latency test needs to be asynchronous

An asynchronous subscriber needs to handle the return message from the server.  The test erroneously assumes the flush will return before the published message arrives back to the subscriber.

Fixed by https://github.com/nats-io/csnats/pull/36.


Support 3.12 and drop 3.7

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

Comparison is base (e808fc8) 100.00% compared to head (a7d9b87) 97.11%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##            master      #99      +/-   ##
===========================================
- Coverage   100.00%   97.11%   -2.89%
===========================================
  Files            5        5
  Lines          381      381
===========================================
- Hits           381      370      -11
- Misses           0       11      +11

Files
Coverage Δ

src/objprint/objprint.py
100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

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


Using Neo4jOperations save(enitity) method runs multiple DELETE cypher queries.

I am trying out the neo4j-ogm-embedded driver to make POJO's of Node Entities and also updating their relationships. Also i am using the spring-data-neo4j for API. When i create a relationship entity between nodes that already have multiple relationships and try to persist the relationship object using the save() method without specifying the depth, multiple delete statements were being run and subsequently the relationship was recreated. However the penultimate delete relationship was not recreated.
The Work-around:- I saved the object as save(entity, 1) i.e a depth of 1 and it worked.
Still curious about why this happens.
For example, if a group of persons(NODE ENTITIES) connects(RELATIONSHIP ENTITY) with every other person in the group, it should create a star type graph in neo4j. But the last connect relationship is deleted and never reestablished again when the connects object is persisted using `save(connects).

HI @kk19ag,
Would you be able to supply a zip file or a test that can replicate this issue? Or at the very least some psuedo code of what you are calling so we can see what's going on?

Closing this issue due to inactivity. If this is still an issue: try it with the latest version and if it's still not working please add a test case and reopen.


Wider acceptance of currency codes for Register API

cc @jhankin @pyricau @amscher please take a look when you get a chance

[ ] Check in Gemfile.lock
[ ] Register World tests


⚠ GitLab Server has degraded performance

In 77b15da, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 7149 ms

Resolved: GitLab Server performance has improved in 5da9e31.


[RESTEASY-1805] Add support for priorities and fix issue with local R…

…esteasyProviderFactory instances

@chkal FYI


configure prettier and tslint

Changelog

Install prettier for formatting the code
Configure tslint for better code
Prettier and lint all existing codes

Coverage remained the same at 100.0% when pulling cfda1190eac2168bf3025f1177930130625f6d80 on install-prettier into 664d72bdbe899ef21cb34871debde4bc5db058b7 on master.

Coverage remained the same at 100.0% when pulling 9fd2c5e0bcfd00bb2161d27c51437a1f45476fb4 on install-prettier into 664d72bdbe899ef21cb34871debde4bc5db058b7 on master.


Y24-028 [BUG] Targeted NanoSeq - Fix metadata insert issue preventing creation of additional child plates from the LTN AL Lib plate

Describe the bug
It is not currently possible to create any additional child LTN AL Lib Dil plates from an LTN AL Lib parent plate in the Targeted NanoSeq pipeline due to a limitation in the metadata creation code. It throws an exception when it detects a duplicate key on attempting insert. This prevents users from looping back to the LTN AL Lib plates and creating additional child plates (desired normal behaviour).
RT Ticket Number
NA
To Reproduce
Steps to reproduce the behaviour:

Attempt to make more than one child plate from an LTN AL Lib plate in Limber
Get errors regarding duplicate metadata
In addition, child plate is made anyway, but binning is done incorrectly so needs to be cancelled

Expected behaviour
Having discussion about this, see research story https://github.com/sanger/limber/issues/1623
Short term fix for this story:
Allow request metadata to be overridden. See code in the create_request_metadata method in file app/models/labware_creators/pcr_cycles_binned_plate_for_t_nano_seq.rb which expects to insert fresh keys. We should check and overwrite the value if the key already exists.
This is not ideal as it wipes out the record for previous runs if they loop without a new submission, but users are Ok with that for short term. Longer term better to have a new submission with different requests for follow up runs, on just the samples that they want to move forward, plus allowing partial transfers in the labware creator. See research story above.
Possibly in the short term we could store history by re-writing each of the metadata to be overwritten with a prefix key e.g. date. Example 'original_plate_barcode' could become '20240321_original_plate_barcode'.
Screenshots
NA
Additional context
We currently make 9 metadata records for each well with a sample on the plate. This is already slow so adding additional queries to check whether each metadata key exists before creating or updating it will slow it further. Check it's not going to time out for a full plate.
Longer term we should only be uploading data for rows they want to take forward, and only recording metadata for those rows (a partial transfer to the child plate), but this is not the case at the moment and the code is writing metadata for all rows with samples.

Also see integration suite PR here
https://gitlab.internal.sanger.ac.uk/psd/integration-suite/-/merge_requests/123


ask for help

Hi, can you provide complete reference data and code

Hi, can you provide complete reference data and code

Not only you have the same wish.....

Actually this work maybe not easy to reproduce.....

Actually this work maybe not easy to reproduce.....
thanks

Would you mind giving me an email or other IM.e.g. wechat ....
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2022年3月24日(星期四) 上午10:40
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [geekfeiw/wifiperson] ask for help (Issue #2)
Actually this work maybe not easy to reproduce.....
thanks
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

good evening, my email is<EMAIL_ADDRESS>

good evening, my email is<EMAIL_ADDRESS>

reproduce

Have you got any datasets so far？

Hi, can you provide complete reference data and code
hello, Have you got any datasets so far?

Hi, can you provide complete reference data and code

hello, Have you got any datasets so far?

Hi, can you provide complete reference data and code

hello, Have you got any datasets so far?

no,I couldn't reproduce it.I gave up.


Darker style --diff formatting

I couldn't find any infos on that so it is propably not possible, but it would be nice for older projects etc. to be able to introduce
linting and formatting slowly like with darker for black, would be cool if ruff had it built in

Thanks! There's more discussion around this and related ideas in https://github.com/astral-sh/ruff/issues/1149 so I'm going to merge into that.


Launchpad Integration

Hi,
did you think about a SAP Launchpad integration? What would be the best place to load and init the error collector if we dont have an index.html?
Plugin Development?
Best regards Richard

Hi @richardbruenning
You should be able to import it in your Component.js and use it from there.


Non-existing transactions shown

Description of the Issue
Within Transactions list, weird transactions are shown.
Expected behavior: show list of onchain and lightning transactions (both outgoing and paid invoices). On click, show transaction details in right pane.
Current behavior:
Within transaction list, 3 separate outgoing lightning payments are shown to 2 nodes I did transactions with. All 3 have the same amount 3 244 570 453 sat.
Upon click on tx, no tx detail pane is shown, and main pane with tx list dissapears. Side issue: wrong transaction date shown.
Check on my LND node:
lncli listpayments does not show such outgoing payments (fortunately).
lncli listchannels shows that channels open to concerned nodes have total capacity lower then transactions amount shown.
Steps to Reproduce
Click on transaction list.
Description of Your Setup
Browser & version: Chrome
Operating system: Windows 10
Node software & version: Remote node accessed through LAN running LND 0.5.0-beta
Console Logs / Screenshots

lncli listpayments response

That's really strange, thanks for the report. If you could do me a few favors to help debug, that'd be really appreciated:

When you click on the faulty transactions, do you get any errors in the console?
When you load the transactions tab, can you get the network responses for the 3 network requests it makes? The erroneous data is probably in  /v1/payments but looking at all 3 may help.

You can open the console by right clicking the icon, and clicking "Inspect popup". There's also a network tab in there.

Here it is:

Console on loading tx tab looks OK - api calls return correct responses from my LND instance:

Console after having clicked on invalid tx:

OK, this is definitely much more to work with. Can you actually click on "payments" in the network tab, and paste the response from it in full (feel free to omit any sensitive data!)

Hi, I checked them, they look OK. I searched for numeric strings in responses like 3244* and there are not. Really strange :)


Trying out new GraphQL queries

While I sit and wait for my GPT-3 access... might as well ask @maraoz to run a little experiment on my behalf :)

This was the output. Is it correct? I don't know GraphQL

This is very interesting..

this is graphQL.  so +1 just for that!
of course the schema doesn't have "season" but this might well have a lot of queries going thru a database and see which one will go with the schema that is there.

Amd about GPT3 created:

will list the episodes where Luke Skywalker is a hero, and for each friend the name of the episode they (but not the name of the friend)

... all the season related filters make no sense, but I'm sure that all we need is more examples and probably will start making more sense.
havent said all that: Is pretty impressive that just creates  queries by not knowing GraphQL at all!

Well it probably does know GraphQL from parsing half the internet :)
Closing the PR because the prompt doesn't seem to be working well, though.


Cannot use Delete with ReviewerEmail

The document states that you can enter a value for ReviewerEmail if the outcome is Delete of KeepAndDelete. This is incorrect, as a Reviewer can only be specified for KeepAndDelete. If you try and add a Reviewer to Delete then it will fail validation
"Validation failed in the provided CSV at row 1 column "ReviewerEmail". Error: ReviewerEmail can only be applied when retention action is KeepAndDelete."

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

ID: d4465ca1-9ee2-e551-aff2-9fc8e4a5eae3
Version Independent ID: 2cbc9375-d055-edf8-0f89-b5794f7aa7a7
Content: Overview of file plan manager - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/file-plan-manager.md
Service: o365-seccomp
GitHub Login: @cabailey
Microsoft Alias: cabailey

@bloodytourists  Thanks for flagging this.  Yes, you must have a retain setting for disposition review, not supported for delete-only.  Fixing in the docs and should be live later today.

Fix is live now, closing.


Add Screencast video in ReadMe

Add video of  user doing the installation!
Screencast
https://github.com/CatsInTech/GitHub-ReadMe/assets/112793075/61141527-c855-4221-a671-512ded82b532

Please assign me this issue


Hexadecimal display in NumericUpDown

Please wait before merging.

@punker76: I think this can be merged.
I have refactored a bit the TextBoxHelper class to avoid duplicate code. The content in the Files changed tab looks a bit confusing but it should not break anything.
Please double-check if there are any weird behaviors.

@punker76  :+1:


This error I have when try to build libmodsecurity

On Ubuntu 16.04
this is prerequisites which is installed:
apt-get install apache2-dev autoconf automake build-essential bzip2 checkinstall devscripts flex g++ gcc git graphicsmagick-imagemagick-compat graphicsmagick-libmagick-dev-compat libaio-dev libaio1 libass-dev libatomic-ops-dev libavcodec-dev libavdevice-dev libavfilter-dev libavformat-dev libavutil-dev libbz2-dev libcdio-cdda1 libcdio-paranoia1 libcdio13 libcurl4-openssl-dev libfaac-dev libfreetype6-dev libgd-dev libgeoip-dev libgeoip1 libgif-dev libgpac-dev libgsm1-dev libjack-jackd2-dev libjpeg-dev libjpeg-progs libjpeg8-dev liblmdb-dev libmp3lame-dev libncurses5-dev libopencore-amrnb-dev libopencore-amrwb-dev libpam0g-dev libpcre3 libpcre3-dev libperl-dev libpng12-dev libpng12-0 libpng12-dev libreadline-dev librtmp-dev libsdl1.2-dev libssl-dev libssl1.0.0 libswscale-dev libtheora-dev libtiff5-dev libtool libva-dev libvdpau-dev libvorbis-dev libxml2-dev libxslt-dev libxslt1-dev libxslt1.1 libxvidcore-dev libxvidcore4 libyajl-dev make openssl perl pkg-config tar texi2html unzip zip zlib1g-dev
I tried with this steps:
$ cd /opt/
$ git clone https://github.com/SpiderLabs/ModSecurity
$ cd ModSecurity
$ git checkout -b v3/master origin/v3/master
$ sh build.sh
$ git submodule init
$ git submodule update
$ ./configure
$ make
$ make install
After make I get this error:
/opt/ModSecurity/src/variables/xml.cc:83: undefined reference to xmlXPathNewContext'
/opt/ModSecurity/src/variables/xml.cc:99: undefined reference to xmlXPathRegisterNs' /opt/ModSecurity/src/variables/xml.cc:127: undefined reference to xmlXPathFreeObject'
/opt/ModSecurity/src/variables/xml.cc:146: undefined reference to xmlXPathFreeContext' /opt/ModSecurity/src/variables/xml.cc:116: undefined reference to xmlXPathEvalExpression'
/opt/ModSecurity/src/variables/xml.cc:142: undefined reference to xmlFree' /opt/ModSecurity/src/variables/xml.cc:135: undefined reference to xmlNodeGetContent'
collect2: error: ld returned 1 exit status
Makefile:540: recipe for target 'modsec-rules-check' failed
make[2]: *** [modsec-rules-check] Error 1
make[2]: Leaving directory '/opt/ModSecurity/tools/rules-check'
Makefile:477: recipe for target 'check-recursive' failed
make[1]: *** [check-recursive] Error 1
make[1]: Leaving directory '/opt/ModSecurity/tools'
Makefile:954: recipe for target 'check-recursive' failed
make: *** [check-recursive] Error 1`

Duplicate with https://github.com/SpiderLabs/ModSecurity/issues/1685.
Issue was fixed as of https://github.com/SpiderLabs/ModSecurity/commit/854a661a2a86357326b5c6be9799e9ab943b61e3


Export dictionary from other than ~/.dragonfly

It would be helpful to be able to export a dictionary that is not located in ~/.dragonfly, but rather is specified as a switch or argument to export.py. This would allow easy conversion when you have access to a json dictionary, but not an exported version of it.

You could do this like so right now:
cp my_new_dict.json ~/.dragonfly/foo.json
python3 ./scripts/export foo

Yes, that's how I did it (so it's not particularly urgent).

You can make up whatever language codes you want (like 'foo') in my example so you don't need to rename files. If you wanted, you could have multiple dictionaries (sin1.json, sin2.json) and switch between them when running dragonfly.
But yes, I can add this.


Retry to use Arc<Statistic> in PartitionedFile again

Which issue does this PR close?

Closes #11885
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

See also https://github.com/apache/datafusion/pull/11893


snap store needs updating

https://snapcraft.io/kubescape

lists @HollowMan6 as developer, not Kubescape
lists armosec.io as URL, not kubescape.io
only edge has the latest build: how do we make this filter down?

/cc @HollowMan6

lists @HollowMan6 as developer, not Kubescape

Since I uploaded that package, it shows my name. I'm not sure if there's any way to transfer the ownership (don't think they have the option, only adding collaboration).

lists armosec.io as URL, not kubescape.io

Updated.

only edge has the latest build: how do we make this filter down?

You need to manually go to https://snapcraft.io/kubescape/releases and change the channel release version. This process can't be automated. I just helped update everything to 2.9.0

looks like we're updated, thanks @HollowMan6 !


Fix typo in releases directory

https://github.com/zhangyuan/google-authenticator-extractor/issues/3

https://github.com/zhangyuan/google-authenticator-extractor/pull/16


All Interfaces not Showing

I am not sure if this is a bug or feature request.
An interface which is not configure but is available does not show on the interface API request. Should they be there or am I missing something?

Figured I would add this as well to better clarify. Is there a means through the API to get all available network ports?

Currently, no. You might be able to accomplish this by using the /api/v1/diagnostics/command_prompt endpoint to call ifconfig to read all interfaces and something like grep to search for the specific interface IDs.
I'll keep this open as a feature request for now. This should be simple enough to squeeze into a patch in the near future.


shared_audits: add nuclear to GITHUB_PRERELEASE_ALLOWLIST

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[ ] Have you successfully run brew man locally and committed any changes?

Required for https://github.com/Homebrew/homebrew-cask/pull/91310 to pass audit.

Thank you @issyl0.


feature request: query without having to provide a jobid ?

i'm reading the google-cloud-python docs and the bigquery rest api reference [2], the job insert api is the async query job, I wonder why the python api is designed to have to provide a jobid? since bigquery server side is archiving the job id for a long time can't re-use same id, that has imposed client side a burden to maintain every jobid to be unique, although that can be easily archived by a time and random based id generator, but the point is: if the bottom layer rest api reference [2] doesn't require jobid to be provided, in the case when jobid is omitted, bigquery server will allocate a unique jobid; why the Python api becomes requiring a jobid?

https://googlecloudplatform.github.io/google-cloud-python/stable/bigquery-usage.html#querying-data-asynchronous
https://cloud.google.com/bigquery/docs/reference/rest/v2/jobs

@c0b thanks for the report.  AFAICT, the docs for the jobReference element of a job resource say that jobId is a required field.  The "managing jobs" docs state that supplying a job ID is best practice:

If you call jobs.insert without specifying a job ID, BigQuery will create a job ID for you, but you will not be able to check the status of that job until the call returns. Moreover, it may be difficult to tell whether the job was successfully inserted or not. If you use your own job ID, you can check the status of the job at any time and you can retry on the same job ID to ensure that the job starts exactly one time.

The client library can totally solve for this by generating a job ID (e.g. str(uuid.uuid4())), saving it to the object, and using it.

the docs for the jobReference element of a job resource say that jobId is a required field.

the jobId is only required if jobReference is present; but the jobReference itself is optional;

Tracking this internally as part of GA redesign.


Convert to signed 16 but integer format

Continue to use Uint16 for encoding
Santise numerical inputs and calculations to perform twos-complement transformations

Transforms not necessary in the end - the numbers only need be interpreted as signed during output or in ADI (which they already are). https://github.com/francisrstokes/16bitjs/commit/c80a2667e008679aeb50fc1940e3f1e413377001 addresses this issue