OTA upgrade fail (IDFGH-10956)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
must have v5 but "git describe --tags" returns error. Command "idf.py" in VScode returns v1.0.3. Esspresif IDF Extension says v1.6.4
Operating System used.
Windows
How did you build your project?
VS Code IDE
If you are using Windows, please specify command line type.
CMD
What is the expected behavior?
It is supposed to verify the certification file
What is the actual behavior?
I have the same problem with issue https://github.com/espressif/esp-idf/issues/6833 I get
"esp-tls-mbedtls: Failed to verify peer certificate!"
Steps to reproduce.
I created a new example program from the IDF examples "advanced_https_ota" and setup the menuconfig for a WROOM board
Build or installation Logs.
No response
More Information.
Hi
I have the same problem with issue https://github.com/espressif/esp-idf/issues/6833
I have enabled EXAMPLE_SKIP_COMMON_NAME_CHECK, in menuconfig but still got the same problem

@Jimis333
Simplest option here is to enable the certificate bundle feature in the advanced_ota example, please see the code below:
https://github.com/espressif/esp-idf/blob/82cceabc6e6a0a2d8c40e2249bc59917cc5e577a/examples/system/ota/simple_ota_example/main/simple_ota_example.c#L95
This ensures that most commonly used root certs are bundled in the firmware and hence it does not need any additional certs.
Alternatively, you can share the endpoint and cert that you using and then we can evaluate what could be the reason of failure in the certificate.

Hi mahavirj,
Thanks for the reply. I have already enabled the bundle as shown in picture.
I had to include "esp_crt_bundle.h" for the crt bundle to work but it did.Thanks a lot!
Below is the result that gives an error of version but that's another problem
I (5693) advanced_https_ota_example: Starting Advanced OTA example
I (5693) advanced_https_ota_example: OTA started
I (5703) main_task: Returned from app_main()
I (6443) esp-x509-crt-bundle: Certificate validated
I (8013) esp-x509-crt-bundle: Certificate validated
I (8733) esp_https_ota: Starting OTA...
I (8733) advanced_https_ota_example: Connected to server
I (8733) esp_https_ota: Writing to partition subtype 16 at offset 0x120000
I (8743) advanced_https_ota_example: Reading Image Description
I (8903) advanced_https_ota_example: Running firmware version: 1
W (8903) advanced_https_ota_example: Current running version is the same as a new. We will not continue the update.
E (8903) advanced_https_ota_example: image header verification failed
I (8913) advanced_https_ota_example: OTA abort
E (8913) advanced_https_ota_example: ESP_HTTPS_OTA upgrade failed
I have a few questions

why do I need to add this line since I already do it in menuconfig
why the cert.pem file  from the example doesn't work ? I have also tried making a cert file using the command
echo ""

| openssl s_client -showcerts -connect raw.githubusercontent.com:443 | sed -n "1,/Root/d; /BEGIN/,/END/p" | openssl x509 -outform PEM >ca_cert.pem

I attach my config and files
url : https://github.com/Jimis333/Test/blob/37b44668bb625586653054c6fc7b48fa8017df42/hello_world.bin?raw=true

I guess I found the answer to my 1st question. Following line is missing from sdkconfig. They have updated simple_ota but not advanced_ota example
CONFIG_EXAMPLE_USE_CERT_BUNDLE=y

@Jimis333

Please see the cert bundle API usage documented here
I think you are embedding the certificate for raw.githubusercontent.com endpoint but using firmware URL for github.com. Please change the config CONFIG_EXAMPLE_FIRMWARE_UPGRADE_URL to https://raw.githubusercontent.com/Jimis333/Test/main/hello_world.bin and it should fix the issue

It worked thanks !

I re-open the issue cause I still have a problem. If I sign my build I get a signature verification fail.
So here are the details
I have uploaded the same code in git but changed the version number. So both codes are signed using the same key. the version check passes but I get a signature fail. Why is that ? Below is the output
I (0) cpu_start: App cpu up.
I (508) cpu_start: Pro cpu start user code
I (508) cpu_start: cpu freq: 160000000 Hz
I (508) cpu_start: Application information:
I (513) cpu_start: Project name:     advanced_https_ota
I (519) cpu_start: App version:      <IP_ADDRESS>
I (524) cpu_start: Compile time:     Sep  1 2023 13:33:37
I (530) cpu_start: ELF file SHA256:  f58c44250bb2e505...
I (536) cpu_start: ESP-IDF:          v5.1.1
I (541) cpu_start: Min chip rev:     v3.0
I (545) cpu_start: Max chip rev:     v3.99
I (550) cpu_start: Chip rev:         v3.1
I (555) heap_init: Initializing. RAM available for dynamic allocation:
I (562) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (568) heap_init: At 3FFB7CE0 len 00028320 (160 KiB): DRAM
I (574) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (581) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (587) heap_init: At 40095488 len 0000AB78 (42 KiB): IRAM
I (595) spi_flash: detected chip: gd
I (598) spi_flash: flash io: dio
I (605) app_start: Starting scheduler on CPU0
I (607) app_start: Starting scheduler on CPU1
I (607) main_task: Started on CPU0
I (617) main_task: Calling app_main()
I (617) advanced_https_ota_example: OTA example app_main start
I (637) example_connect: Start example_connect.
I (647) wifi:wifi driver task: 3ffbf944, prio:23, stack:6656, core=0
I (657) wifi:wifi firmware version: ce9244d
I (657) wifi:wifi certification version: v7.0
I (657) wifi:config NVS flash: enabled
I (657) wifi:config nano formating: disabled
I (657) wifi:Init data frame dynamic rx buffer num: 32
I (667) wifi:Init management frame dynamic rx buffer num: 32
I (667) wifi:Init management short buffer num: 32
I (677) wifi:Init dynamic tx buffer num: 32
I (677) wifi:Init static rx buffer size: 1600
I (677) wifi:Init static rx buffer num: 10
I (687) wifi:Init dynamic rx buffer num: 32
I (687) wifi_init: rx ba win: 6
I (697) wifi_init: tcpip mbox: 32
I (697) wifi_init: udp mbox: 6
I (697) wifi_init: tcp mbox: 6
I (707) wifi_init: tcp tx win: 5744
I (707) wifi_init: tcp rx win: 5744
I (707) wifi_init: tcp mss: 1440
I (717) wifi_init: WiFi IRAM OP enabled
I (717) wifi_init: WiFi RX IRAM OP enabled
I (727) phy_init: phy_version 4670,719f9f6,Feb 18 2021,17:07:07
I (827) wifi:mode : sta (80:64:6f:13:a1:e4)
I (827) wifi:enable tsf
I (827) example_connect: Connecting to Emphasisdw...
I (837) example_connect: Waiting for IP(s)
I (3247) wifi:new:<1,0>, old:<1,0>, ap:<255,255>, sta:<1,0>, prof:1
I (4167) wifi:state: init -> auth (b0)
I (4177) wifi:state: auth -> assoc (0)
I (4187) wifi:state: assoc -> run (10)
I (4227) wifi:connected with Emphasisdw, aid = 4, channel 1, BW20, bssid = a4:88:73:43:b1:20
I (4227) wifi:security: WPA2-PSK, phy: bgn, rssi: -67
I (4227) wifi:pm start, type: 1
I (4297) wifi:AP's beacon interval = 102400 us, DTIM period = 1
I (4737) wifi:idx:0 (ifx:0, a4:88:73:43:b1:20), tid:6, ssn:2, winSize:64
I (5637) example_connect: Got IPv6 event: Interface "example_netif_sta" address: fe80:0000:0000:0000:8264:6fff:fe13:a1e4, type: ESP_IP6_ADDR_IS_LINK_LOCAL
I (5737) esp_netif_handlers: example_netif_sta ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>
I (5737) example_connect: Got IPv4 event: Interface "example_netif_sta" address: <IP_ADDRESS>
I (5747) example_common: Connected to example_netif_sta
I (5747) example_common: - IPv4 address: <IP_ADDRESS>,
I (5757) example_common: - IPv6 address: fe80:0000:0000:0000:8264:6fff:fe13:a1e4, type: ESP_IP6_ADDR_IS_LINK_LOCAL
JIM connected...........
I (5767) wifi:Set ps type: 0, coexist: 0
I (5777) advanced_https_ota_example: Starting Advanced OTA example
I (5777) advanced_https_ota_example: OTA started
I (5787) main_task: Returned from app_main()
I (5797) wifi:idx:1 (ifx:0, a4:88:73:43:b1:20), tid:0, ssn:0, winSize:64
I (6837) esp_https_ota: Starting OTA...
I (6837) advanced_https_ota_example: Connected to server
I (6837) esp_https_ota: Writing to partition subtype 16 at offset 0x120000
I (6847) advanced_https_ota_example: Reading Image Description
I (6847) advanced_https_ota_example: Running firmware version: <IP_ADDRESS>
I (6857) advanced_https_ota_example: Verifying chip id of new image: 0
I (35857) esp_image: segment 0: paddr=00120020 vaddr=3f400020 size=27230h (160304) map
I (35907) esp_image: segment 1: paddr=00147258 vaddr=3ffb0000 size=03630h ( 13872)
I (35907) esp_image: segment 2: paddr=0014a890 vaddr=40080000 size=05788h ( 22408)
I (35917) esp_image: segment 3: paddr=00150020 vaddr=400d0020 size=9b0ech (635116) map
I (36097) esp_image: segment 4: paddr=001eb114 vaddr=40085788 size=0fd00h ( 64768)
I (36117) esp_image: segment 5: paddr=001fae1c vaddr=00000000 size=051b4h ( 20916)
I (36127) esp_image: Verifying image signature...
I (36127) secure_boot_v2: Take trusted digest key(s) from running app
E (36137) esp_image: Secure boot signature verification failed
I (36137) esp_image: Calculating simple hash to check for corruption...
W (36357) esp_image: image valid, signature bad
E (36357) advanced_https_ota_example: Image validation failed, image is corrupted
E (36357) advanced_https_ota_example: ESP_HTTPS_OTA upgrade failed 0x1503
I (36357) advanced_https_ota_example: OTA finish

@Jimis333
Unfortunately this information is not sufficient to comment on the issue. Please provide detailed steps that you followed, complete console log and the efuse summary of the device. Thanks.

@mahavirj
I run the advanced_https_ota example from the ESP-IDF examples. I have included the sdkconfig file with my settings
I sign the build with my secure_boot_signing_key.pem
I take the advanced_https_ota.bin  and upload it to git (url in sdkconfig)
I change the version of the project to a lower one so that it can be upgraded from the .bin from git
(I change the version in the cmakeLists.txt like this :  set(PROJECT_VER "<IP_ADDRESS>") )
Now when I run my program it's supposed to call the url and download a lower version of itself
Upgrade stops in the verification process (full output in text file included) :
I (38857) esp_image: segment 5: paddr=001fad28 vaddr=00000000 size=052a8h ( 21160)
I (38867) esp_image: Verifying image signature...
I (38867) secure_boot_v2: Take trusted digest key(s) from running app
E (38877) esp_image: Secure boot signature verification failed
I (38877) esp_image: Calculating simple hash to check for corruption...
W (39097) esp_image: image valid, signature bad
E (39097) advanced_https_ota_example: Image validation failed, image is corrupted
E (39097) advanced_https_ota_example: ESP_HTTPS_OTA upgrade failed 0x1503
I (39097) advanced_https_ota_example: OTA finish
output.txt
sdkconfig.txt

I just found out from the last answer to my other opened issue what was the problem !
I signed by mistake using ECDSA key instead of RSA. (https://github.com/espressif/esp-idf/issues/12170) which is not supported by my chip EC3.


[changelog] Missing Test category in conventional-commits preset

Reported by @jponge at https://github.com/jreleaser/jreleaser/discussions/1608

Fantastic


diagnostic: automatically tap homebrew/core.

Otherwise, this just dies with Error: No available tap homebrew/core..

Review period skipped due to critical label.

brew doctor tests not yet complete: https://github.com/Homebrew/brew/actions/runs/2933746419
Probably fine, though.


hg38 + multiallelic calls?

I feel that it is important to move to hg38 as soon as possible, due to mapping issues caused by paralogs not present in hg19. Will there be support for hg38? A request for future versions is also handling handling of multiallelic calls.
Thanks for making this great tool!

I completely agree and these are a part of our plan.  We will likely address multiple allele support first, as we need to come up with a detailed plan for supporting build 38 given that many annotations are only available for build 37.

Multi-allelic support is now in master and will be part of the forthcoming release.
hg38 support is in the works but will take several months.


index.md

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

I closed/re-opened in an attempt to clear Travis cli error.
LGTM

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


[Feature] HTML Reporter

This implementation extends ReporterBase and generates the report in a single HTML file.
The code is formatted using Google Code Prettify, with a custom style for readability.
The html uses as favicon the new logo created in PR92
Configuration
reporter = "html"
reportfile = "luacov.report.html"

Sample report
luacov.report.html.zip

New template Sample report
luacov.report.html.zip

New template Sample report
luacov.report.html.zip

Cool! Thank you for the contribution! I will send you an invite to give you commit access so you can push any updates to the HTML reporter and any other features/fixes you'd like!


FAQ Update (fixes #118)

Add instructions to solve the failure when provisioning F5 Workshop from mac.


Crashes when I try to play a track (Error reading the stream. (code 18) (StandardError))

OSX 10.9.3, ruby 2.0.0p247 (using rbenv/ruby-build installed on Homebrew)
I found the debug log
Warning: Xing stream size off by more than 1%, fuzzy seeking may be even more fuzzy than by design!
/usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/models/player.rb:51:in `new': Error reading the stream. (code 18) (StandardError)
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/models/player.rb:51:in `length_in_seconds'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/models/player.rb:58:in `load'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/models/player.rb:34:in `play'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/controllers/player_controller.rb:61:in `play'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/application.rb:38:in `block in initialize'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/events.rb:13:in `call'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/events.rb:13:in `block in trigger'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/events.rb:12:in `each'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/events.rb:12:in `trigger'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/controllers/track_controller.rb:21:in `block in initialize'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/events.rb:13:in `call'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/events.rb:13:in `block in trigger'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/events.rb:12:in `each'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/events.rb:12:in `trigger'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/application.rb:74:in `handle'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/application.rb:49:in `block in main'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/application.rb:47:in `loop'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/application.rb:47:in `main'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000/application.rb:65:in `run'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/lib/soundcloud2000.rb:15:in `start'
        from /usr/local/var/rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/soundcloud2000-0.1.0/bin/soundcloud2000:5:in `<top (required)>'
        from /usr/local/var/rbenv/versions/2.0.0-p247/bin/soundcloud2000:23:in `load'
        from /usr/local/var/rbenv/versions/2.0.0-p247/bin/soundcloud2000:23:in `<main>'

Just found this app today and love the idea! Unfortunately as-is it is quite unstable and prone to crashing. Looks like it isn't under active development anymore? Perhaps I could help out?


webgl tests are flaky on dart2js-linux-chromeff bots

tex-image-and-sub-image-2d-with-video_t01 flaked on dart2js-linux-chromeff-4-4-be.
WebGLContextEvent_t01 did on dart2js-linux-chromeff-1-4-be
I tried them my linux workstation - both test passed.

Same for co19/LayoutTests/fast/canvas/webgl/tex-sub-image-cube-maps_t01
https://build.chromium.org/p/client.dart/builders/dart2js-linux-chromeff-2-4-be/builds/5644/steps/dart2js-chrome-co19-fast-startup tests/logs/stdio

The following test is also flaky, although the error message is different:
co19/LayoutTests/fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data-rgba5551_t01
Exception: Expect.fail('shouldBe [0, 255, 0, 255]: at (0, 0) expected: [0, 255, 0, 255] was 0,0,0,0')

A few other tests flaked with the 'Unable to fetch WebGL rendering context for Canvas' message:
co19/LayoutTests/fast/canvas/webgl/tex-image-and-sub-image-2d-with-image-data-rgba5551_t01
co19/LayoutTests/fast/canvas/webgl/tex-image-and-sub-image-2d-with-video_t01
co19/LayoutTests/fast/canvas/webgl/tex-image-and-sub-image-2d-with-array-buffer-view_t01
co19/LayoutTests/fast/canvas/webgl/uniform-location-length-limits_t01
co19/LayoutTests/fast/canvas/webgl/tex-sub-image-2d_t01

@whesse: Can we filter those errors? Should we disable all the WebGL tests? Who is looking into this?

Duplicates #29961.
But to answer the questions: Not right now; Yes, we'll move them to an FYI bot; Me.


Add an optional parameter for MsBuild so we can use the latest version

This is needed for any applications that target .NET 4.6 and will later be useful for any newer .NET versions

Thanks Kenneth, this is good. Just a minor niggle: the naming convention for properties is camelCase, not PascalCase. Please fix this and also update the version number in the nuspec file.


Adjusted initial bls12_381 branch

This is a new branch that contains the changes requested in the previous PR.
Changes:

commits specific for bls12_381 cherry-picked and squashed
added comments
updated README
added licence to choice.sw
created a separate testing folder where all test folder can go into
moved choice.sw into utils library so it can be reused easily
cleanup and changes along the way

This branch works on all comments in the previous PR, except for the last one about assembly versions of the computational heavy operations (whether that would be more efficient than pure Sway). I think it would be better to try that out in a separate branch.
@man2706kum shared that for him there occur some errors in compilation. For me on the other hand; I can compile and run the tests in testing/tests_bls12_381 for Fp succesfully. Might be some versioning issue, would be interesting to know whether you can compile @Mikerah.

Made the last fixes as suggested, will merge now

:rocket:


Database#close(): Promise doesn't handle SessionPool#close() properly

Database#close() has Promise interface.
https://github.com/googleapis/nodejs-spanner/blob/main/src/database.ts#L627
But I found that await database.close() or database.close().then() doesn't wait SessonPool#close() properly.
https://github.com/googleapis/nodejs-spanner/blob/f915bd16cf7e817243e46a319b3e6f270b24bf68/src/database.ts#L634
I think this code should be handle callback as followings:
if (typeof callback === 'function') {
  this.pool_.close(callback);
} else {
  return new Promise((resolve, reject) => {
    callback(err => {
      if (err) {
        reject(err);
      }  else {
        resolve();
      }
    });
  });
}

Hi @nabeix Can you please share an example or repro steps for us to reproduce this issue ?


Storage drive cloning PowerToy

Description of the new feature / enhancement
Looking at the third party options available for storage drive cloning to a larger capacity drive, there are quite a few that involve a convoluted process or offer a limited feature set. Would love to see a PowerToy built by Microsoft that is certified for use with Windows to perform this exact function.
Scenario when this would be used?
This would be used when upgrading any storage drive to a larger capacity storage drive (i.e. upgrading the boot drive, or switching from an HDD drive to an SSD drive).
Supporting information
Some of the best options out there require payment for essential features, or require booting from a flash drive. I just want to be able to clone my boot drive in a straightforward manner, and then easily swap in the new drive with no hassle. Any extra features can be left to all those other third party apps, but a PowerToy that does just this should be the only thing needed for any PC owner looking to upgrade their boot drive to a higher capacity.

I could see us doing something like synctoy / robocopy gui but a true cloning i don't see us doing.  /dup #26


Semver adherence

Deprecated or not, this commit & subsequent release broke Semver convention by releasing breaking changes on a patch version bump!
https://github.com/angular/angular.js/commit/1daa4f2231a89ee88345689f001805ffffa9e7de
Please refer to the npm versioning page: https://docs.npmjs.com/getting-started/semantic-versioning
A minor version would have been enough - a patch version number is a bit mean.

Two things:

AngularJS has always included breaking changes in the minor version. Never in the patch version. Yes, that means we do not adher to semver.
The commit you mentioned has not been released in any version. It's only part of the master branch

Apologies, I assumed it had gone out in the release yesterday. Something in 1.6.3 has broken our builds which we've failed to lock to specific versions.


Update AppVeyor configuration

Remove Clang/C2, this toolset is no longer supported
Fix path to avoid Chocolatey shims breaking mingw-w64 build
Remove Clang path workaround, this issue has been fixed
Enable Clang targeting MSVC
Use CMAKE_VERBOSE_MAKEFILE

Moved to staging.


Imp From trait on EthernetAddress

Why?

@whitequark
just for friendly api.
and i also add Hash trait on EthernetAddress .

So what's "friendly" about it? I don't see the use case.

@whitequark
let hwaddr = EthernetAddress::from([1,2,3,44,5,6]);

let hwaddr = EthernetAddress::from([1,2,3,44,5,6]);

This is longer than using the constructor directly:
let hwaddr = EthernetAddress([1,2,3,4,5,6]);

yeah, got it.
now just add Hash trait.


After click button, hotkey no longer working

Reproduction: check the https://github.com/gengjiawen/react-shortcut-key example folder.

Before you click the hi button in top, shortcut works fine.
After click the button, shortcut will break.
Click other area in this page, it will work again.

I find a fix, in React Component
  componentDidUpdate = () => {
    console.log(this);
    document.activeElement.blur()
  }

But I can't figure out the reason why this occur.

It's a feature however. Browsers has its own shortcut keys act on some elements, such as <button> or <input>. Imagine the case if the browser is focusing on an <input> element, and then you press ctrl + v, what will happen? It's no doubt there will be a conflict. So I prefer preventing keyborad events on these element to resolve this problem.

The crtical line is here https://github.com/Dafrok/react-shortcut-key/blob/master/src/index.js#L127

By the way, I find the key of keymap is meaningless at all. Being the only user of my lib, how do you think If I change the API like this in 2.0 version?
// Before
const keymap = {
  hello: {
    key: 'enter',
    ctrl: true,
    fn: componentOnHello
  }
}

// After
const keymap = {
  'ctrl+enter': componentOnHello
}

Hmmm, I think it will be good. 2.0 version has been released.

I think this issue should be mentioned in README.md.
I have upgrade to 2.0.2, works fine :)

OK, just remember you have to prevent default key events by yourself in 2.0 if needed.

And put the demo link and License info in README.md will be more user friendly.

Finished :)


Adding GitHub Copilot Adventures

Adding GitHub Copilot Adventures session details

@toddanglin - I've uploaded the assets here.


Make C* compile and run on Java 11 and Java 8

patch by Robert Stupp; reviewed by Jason Brown for CASSANDRA-9608
reworked for scylla and 3.11 code base.

Note: Testes in so far as running the daemon + tools + stress etc with both jdk 8 and jdk 11. But on my machine, on my build.
I would very much suggest merging into a test branch and run whatever set of comprehensive tests you can think of with jdk 11 before fully adopting it.

@haaawk - need your help with this one - can you give your blessing - I can try a BYOD run to get some validation as well


Select only one field  from related model on has:one relation

On struct like this
type User struct {
ID string
AuthenticationProvider *AuthenticationProvider `bun:"rel:has-one"`
}

type AuthenticationProvider struct {
	bun.BaseModel    `bun:"authentication_provider,alias:authentication_provider"`
	UniqueID         string `json:"-" bun:"-"`
	AuthenticationID string `json:"authentication_id"`
	UserId           string `json:"-"`
	ProviderName     string `json:"-"`
}

When I fill values in User struct var and print it , it comes like this:
{
id: "jkdojd",
AuthenticationProvider: {
authentication_id: "jjhkjjlk"
}
}

but I want it like this:
{
id: "jkdojd",
authentication_id: "jjhkjjlk"
}

Right now I am achieving this by changing user struct to like this:
type User struct {
ID string
AuthenticationProvider *AuthenticationProvider `bun:"rel:has-one"`
AuthenticationID       string                  `json:"authentication_id" bun:"-"`
}

Is there any direct way  to achieve same result without adding  AuthenticationID field to User struct?
and how does I handle foreign key in structs?

Is there any direct way to achieve same result without adding AuthenticationID field to User struct?

Perhaps you want belongs-to relation - it has the layout you need.

and how does I handle foreign key in structs?

Bun does not use information about foreign keys. But you can create foreign keys when creating a table - https://bun.uptrace.dev/guide/queries.html#creating-tables

will try , if  there is no more doubt or problem , I will close this issue

belongs-to give me same response layout, and probably that's how it should be given that type is struct so layout should also be struct.
{
      "id": "c0068408-9c59-4a19-b701-ed2f12181ebe",
      "organisation_name": "",
      "first_name": "Yas",
      "last_name": "Kal",
      "email_id"<EMAIL_ADDRESS>      "email_verified": true,
      "country_code": "+78",
      "phone_number": "0010000000",
      "created_at": "2021-07-08T16:40:01.535251Z",
      "updated_at": "2021-07-09T03:58:05.547421Z",
      "authentication_id": "",
      "AuthenticationProvider": {
          "authentication_id": "60ae10e6cb1fe7006ac2ebd7"
      }
  }


Fix Open Graph image location

og:image isn't configured correctly for the distribution builder.

Actually this is a known bug in vite. Let's wait and see if it gets fixed before release.


Is there any Huggingface url link for the model？

I appreciate it.

tks

@dsp6414 We have no trained weights for this architecture yet. If anyone from the community is willing to help, it will  be great. I can train on my GPU after 2 weeks.

@athenasaurav Thank you for the offer. I will email you as soon as the initial run of this code is bug-free. Thanks again!

Sure @p0p4k looking forward to it.


Links are not being decoded causing issues for links with # and other symbols

For example, this Jason will result in 404 for the linked page because # is not decoded on the webview
{
"type": "label",
"text": "Visit Website",
"href": {
"url": "http://simplydo.com/cryptography/#/about",
"view": "Web"
}
}

Fixed this on https://github.com/Jasonette/JASONETTE-iOS/commit/c476aee9390e33a94dec832e33427ab7eb1c6d34 Please pull from dev and it should work.
Please reopen if this still doesn't work!

@sbashyal actually i should have confirmed it works before closing. Could you confirm it works when you get a chance @sbashyal ? Thanks!

@gliechtenstein It is working now. The issue has been resolved. Thanks much for quick fix.


<EMAIL_ADDRESS>update to 2.4.7

Fixed issues regarding templates not being found

[x] I have read the Contributing Guide.

/verify


operator equal on Index should behavior similarly to Series

currently on master (after the PR https://github.com/pydata/pandas/issues/9785), we have the following behavior when comparing two Indexes with ==:
In [3]: index_a = Index(['a', 'b', 'c'])
In [4]: index_b = Index(['a', 'b', 'c', 'd'])
In [5]: index_a == index_b
Out[5]: False

Firstly, == should be element-wise comparison, so it really shouldn't return just a scalar bool. That should be the case for Index.equals() instead.
Secondly it is also not consistent with the behavior with Series
In [6]: series_a = Series(['a', 'b', 'c'])
In [7]: series_b = Series(['a', 'b', 'c', 'd'])
In [8]: series_a == series_b
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-8-97f84083a0cd> in <module>()
----> 1 series_a == series_b

/Users/mortada_mehyar/code/github/pandas/pandas/core/ops.py in wrapper(self, other)
    576             name = _maybe_match_name(self, other)
    577             if len(self) != len(other):
--> 578                 raise ValueError('Series lengths must match to compare')
    579             return self._constructor(na_op(self.values, other.values),
    580                                      index=self.index, name=name)

ValueError: Series lengths must match to compare

This PR makes the behavior consistent by checking the Index lengths. Also fixes a few unit tests which seem to incorrectly assume .equal() should return the same output as ==

What happens if you compare an Index to a Series? Or to a numpy array?

with the example above (without applying this PR) we would get:
In [8]: index_a == series_a
Out[8]: array([ True,  True,  True], dtype=bool)

In [9]: index_a == series_b
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-9-3ca722762b7b> in <module>()
----> 1 index_a == series_b

/Users/mortada_mehyar/code/github/pandas/pandas/core/ops.py in wrapper(self, other)
    583         elif isinstance(other, (np.ndarray, pd.Index)):
    584             if len(self) != len(other):
--> 585                 raise ValueError('Lengths must match to compare')
    586             return self._constructor(na_op(self.values, np.asarray(other)),
    587                                      index=self.index).__finalize__(self)

ValueError: Lengths must match to compare
In [10]: index_a == series_b.values
Out[10]: False

It seems like the length mismatch will be caught when comparing Index to a Series, but not to a numpy array

This was done to provide == to work on an Index vs a MultiIndex. I guess this changed the default behavior, which should be to raise (though this does differ and IMHO is a bit unexpected from .equals. But since this is numpy it just raises).
@mortada can you change back w/o breaking the fix in #9875  and maybe add some tests for Index vs Series as well to validate.

Update the same release note as #9875 and just add this issue number on

@jreback I added tests for Index vs Series and np.array, also added tests for MultiIndex, please take a look, thanks

@jreback

I guess this changed the default behavior, which should be to raise

What do you mean exactly? I think the current behaviour (returning False when comparing non-equal-length Index object/numpy arrays) is the 'default' behaviour for at least going back to 0.12 (and I suppose this is the case as this is the numpy behaviour to return False instead of raise if the two arrays can't broadcast).
Nonetheless, I think it would be good to have a more consistent behaviour across Index and Series. But I also think we should wait with this untill 0.17, as this is a clear and an on-purpose API change.

BTW, unequal length Index and Series don't always raise (so here it seems to follow numpy broadcast behaviour):
In [26]: index_a = Index(['a'])

In [27]: index_b = Index(['a', 'b', 'c', 'd'])

In [28]: index_a == index_b
Out[28]: array([ True, False, False, False], dtype=bool)

In [29]: index_a == series_b
Out[29]: array([ True, False, False, False], dtype=bool)

In [30]: index_a == series_b.values
Out[30]: array([ True, False, False, False], dtype=bool)

This issue is that this is pretty inconsitent (I guess this has always been the case, if so they should for sure wait for 0.17.0; I thought the related PR changed this).
In [1]: Series([1,2])==Series([1])
ValueError: Series lengths must match to compare

In [2]: Index([1,2])==Index([1])
Out[2]: array([ True, False], dtype=bool)

@mortada status?

@jreback with this PR the behavior is now consistent between Series and Index, in that unequal lengths would raise in both cases:
In [2]: Series([1,2]) == Series([1])
ValueError: Series lengths must match to compare

In [3]: Index([1,2]) == Index([1])
ValueError: Lengths must match to compare

Note that broadcasting still works if the comparison is against a scalar
In [4]: Series([1,2]) == 1
Out[4]:
0     True
1    False
dtype: bool

In [5]: Index([1,2]) == 1
Out[5]: array([ True, False], dtype=bool)

if this is the behavior we want this PR is already doing the above (although I'm certainly happy to add more test cases etc)

@mortada Can you add to your tests also the case of a index/series/array that is of a broadcastable length? As now you only include of length 3 and 4.
Further:

can you add a whatsnew section explaining the change (API changes section)?
this should also be explained somewhere in the docs I think (or is this is explained somewhere, this should be updated)

And to summarize the api changes/differences with numpy, there are two ways pandas differs:

unequal lenght but not broadcastable:

pandas: raises (with this PR)
numpy: just returns False

unequal length but broadcastable:

pandas: raises (for series, and for index broadcast now but raise after this PR)
numpy: broadcast and compare elementwise

Ideally, I think we stay as close as possible to the numpy behaviour. Although for the unequal length, I agree it is very strange to just return False and I like raising an error more. But for the broadcast issue, there is maybe less a reason to deviate from numpy behaviour?

numpy is wrong in returning False at the very least it should be NotImplementedError so the caller can do something. So this is a bug IMHO (though not really sure why they do this).
Futher, unequal lengths but can broadcast is an API issue in numpy in that they allow things like wrap-around (e.g. they will broadcast even if it 'wraps' around to the beginning of the array BY DEFAULT). I don't really remember the example.
We should follow numpy rationale behavior. Both of these case are irrational IMHO. (though to be fair, these prob aren't much of an issue in numpy the way you use it).

@jorisvandenbossche I've added a section to explain the change in basics.rst and also the whatsnew for v0.17.0
I think we all agree that returning False is a very strange behavior, especially in the unbroadcastable case. In the broadcastable case I feel less strongly about it, however I think it's still slightly more consistent to raise

@jorisvandenbossche forgot to say, also added more test cases, please take a look

Yes, maybe that was not fully clear, but I also agree completely we want to raise instead of just returning False.
It was only for the broadcastable case that I was doubting a bit. But no strong opinion on that. In any case, it will only for very minor cases, namely a 1-element Index (as Indexes are always 1D). So OK to raise then as well.

Nice docs and whatsnew note!

@jorisvandenbossche Thanks! I've updated according to your comments, please take a look

@mortada ping when updated / green. I have a PR that I am going to push in after this one.

@jreback sure will ping you soon

@jreback updated, will ping you when green

@jreback it's green now

Thanks!

@mortada
these tests should really be under the Base object in index tests, so that they tests for ALL sub-classes as this is a general operation.
Can you do a PR to move (this will involve changing the way the tests happen a bit).

@jreback sure I'll do that in another PR

@mortada thanks. FYI, I just merged #10569  which touched some of this (the issue was I think the comparisons used np.testing.assert_array_equal which bomb when nans are there (and is not a standard pandas test method), so now use assert_numpy_array_equivalent.


Update README.md

Problem
What is being fixed?  Examples:

Fix crash on startup.If you update the branch to the original version of v4.3, you will get an error.

Change overview
What's in this PR
Testing
How was this tested? (at least one bullet point required)

If unit tests were added, how do they cover this issue?
If unit tests existed, how were they fixed/modified to prevent this in future?
If new unit tests are not added, why not?
If integration tests were added, how do they verify this change?
If new integration tests are not added, why not?
If manually tested, what platforms controller and device platforms were manually tested, and how?
If no testing is required, why not?

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

The first time to use a pull request, the submission may not be very standard, but in fact, this is the case. It is not possible to use V4.3, so it should be changed


[REQUEST] Possibility to upload PDF

Possibility to upload PDF with the image uploader.

Hello,
Bumping on this, @dignajar would you consider adding this feature to the v4 roadmap?
Thanks!


Aliased docker images with docker image ls in docker images

When I use docker images it doesn't show the alias command docker image ls or docker image list.
- What I did
I added the alias for the docker image ls  and docker image listcommand to docker images command
- How I did it
I add cmd.Aliases to the function that returns a new docker images command.
- How to verify it
I ran this command in a docker container per the contributing guidelines.
- Description for the changelog
Aliased docker images command to docker image ls and docker image list
- A picture of a cute animal (not mandatory but encouraged)

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "docker-images-alias" git@github.com:troy0820/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

My commit is signed, it has the verified on the commit message. Is there something I'm not doing?

looks like  commit https://github.com/troy0820/docker/commit/eb2081a63aa33b0fd8dfeec015a5e5bad4c96a35 doesn't have a sign-off; If you squash your commits, validation should pass

Yeah I rebased and forced up 👍 It looks like it's passing now.  Thanks @thaJeztah

awesome!

/cc @dnephin

I looked into this in #26859. I'm not sure this is quite right because normally aliases are listed with just the single level command alias (ex ls, ps). But these will show up with the full command path.
The Aliases field is actually used by cobra for routing, so if someone were to run docker "docker image ls" as a command, that would actually be valid.

I see what you are saying @dnephin.  My question is this, not from a design decision, but the command docker image ls has the following aliases ls, images, list.  From an alias standpoint, you can run docker image images, docker image ls ,docker image list and get the same result as docker images.  Can you in fact split up the new docker image ls command and then alias docker images to docker image ls because the route to each other?

Yes, they are already split up. newListCommand() is the function which returns the command for docker image ls.
Aliasing to commands that are under different sub-commands is not supported by cobra. We did it by just adding the commands where we needed them, but that leaves us with them missing from the help text.

Gotcha, should we just close this PR?

I guess so. I think any solution to this will have to involve changes to cobra (which we have forked already ).

No problem 👍🏽


different settings on device

Hey, I have a question: I need to have a slightly different plugin settings on mobile and desktop.
In detail, I want mobile to have adaptive height of slides and on desktop fixed.
I tried set it via window.resize, but it breaks my events on slides change.
Is there any way to do that you suggest?
Thanks in advance

Check out the docs, there's an option called "responsive" where you can define options at different break points. There's an example implementation on the demo page. Make sure you define any option that will change at every breakpoint.
Also see this about issue submission. I find it's helpful to search old issues to see if there's ever been a question about it.
Hope that helps. Thanks

Sure, sorry :)
The problem was that the responsive crashed the animations. I uploaded version 1.5.5 and it works great now.


Ban in video game?

Hey, is it possible that I get a ban in a video game for using this? Like downloading the video game with that or having that thing on my computer? Or even having it opened when playing a game? If yes or no, why?

I wouldn't try running it while a game is open and in a match (beyond just the fact that you shouldn't be trying to edit windows while playing a game, just do one or the other), mostly because I would assume some anticheats are just dumb and might not like powershell touching anything that the game might be accessing (i highly doubt the script does but with anticheats, you never know what they look for), but i have not had any issue using this script on every windows install i have and I play CSGO Faceit and Valorant which both have kernel level spyware i mean anticheat *cough*
TL;DR: The script should have absolutely no affect on anticheat, but with anything that touches registry or system files or runs in powershell, you are best to not be running or doing anything with it while in a anticheat heavy game. (some have been autobanned for stupider stuff)

Alright but is it 100% sure that it will not get me banned from games for
running it at the same time as the game?
Le mar. 14 févr. 2023 à 16:35, lavavex @.***> a écrit :

I wouldn't try running it while a game is open and in a match (beyond just
the fact that you shouldn't be trying to edit windows while playing a game,
just do one or the other), mostly because I would assume some anticheats
are just dumb and might not like powershell touching anything that the game
might be accessing (i highly doubt the script does but with anticheats, you
never know what they look for), but i have not had any issue using this
script on every windows install i have and I play CSGO Faceit and Valorant
which both have kernel level spyware i mean anticheat cough
TL;DR: The script should have absolutely no affect on anticheat, but with
anything that touches registry or system files or runs in powershell, you
are best to not be running or doing anything with it while in a anticheat
heavy game. (some have been autobanned for stupider stuff)
—
Reply to this email directly, view it on GitHub
https://github.com/ChrisTitusTech/winutil/issues/603#issuecomment-1430408648,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5W54FQZIWC7MVUE6GVK74TWXP3BVANCNFSM6AAAAAAUTKIIYY
.
You are receiving this because you authored the thread.Message ID:
@.***>

And are you an employee or someone that is in the right place to know that
type of information
Le mar. 14 févr. 2023 à 17:51, Mat 6 @.***> a écrit :

Alright but is it 100% sure that it will not get me banned from games for
running it at the same time as the game?
Le mar. 14 févr. 2023 à 16:35, lavavex @.***> a
écrit :

I wouldn't try running it while a game is open and in a match (beyond
just the fact that you shouldn't be trying to edit windows while playing a
game, just do one or the other), mostly because I would assume some
anticheats are just dumb and might not like powershell touching anything
that the game might be accessing (i highly doubt the script does but with
anticheats, you never know what they look for), but i have not had any
issue using this script on every windows install i have and I play CSGO
Faceit and Valorant which both have kernel level spyware i mean
anticheat cough
TL;DR: The script should have absolutely no affect on anticheat, but with
anything that touches registry or system files or runs in powershell, you
are best to not be running or doing anything with it while in a anticheat
heavy game. (some have been autobanned for stupider stuff)
—
Reply to this email directly, view it on GitHub
https://github.com/ChrisTitusTech/winutil/issues/603#issuecomment-1430408648,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5W54FQZIWC7MVUE6GVK74TWXP3BVANCNFSM6AAAAAAUTKIIYY
.
You are receiving this because you authored the thread.Message ID:
@.***>

And are you an employee or someone that is in the right place to know that type of information Le mar. 14 févr. 2023 à 17:51, Mat 6 @.> a écrit :
…
Alright but is it 100% sure that it will not get me banned from games for running it at the same time as the game? Le mar. 14 févr. 2023 à 16:35, lavavex @.> a écrit : > I wouldn't try running it while a game is open and in a match (beyond > just the fact that you shouldn't be trying to edit windows while playing a > game, just do one or the other), mostly because I would assume some > anticheats are just dumb and might not like powershell touching anything > that the game might be accessing (i highly doubt the script does but with > anticheats, you never know what they look for), but i have not had any > issue using this script on every windows install i have and I play CSGO > Faceit and Valorant which both have kernel level spyware i mean > anticheat cough > > TL;DR: The script should have absolutely no affect on anticheat, but with > anything that touches registry or system files or runs in powershell, you > are best to not be running or doing anything with it while in a anticheat > heavy game. (some have been autobanned for stupider stuff) > > — > Reply to this email directly, view it on GitHub > <#603 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/A5W54FQZIWC7MVUE6GVK74TWXP3BVANCNFSM6AAAAAAUTKIIYY > . > You are receiving this because you authored the thread.Message ID: > @.***> >

I'm not an employee, no. this project is a collaboration effort and is open source, there are no employees...
 Usually, as long as no program is touching the games memory while it is running, or having funky dll's doing stuff while game is running you should be fine. but again, as the script touches registry and system settings, there is always a 1 in a million chance some poorly coded anticheat that is overly sensitive will not be happy.
I play Valorant and Faceit CSGO which have such invasive anticheats that they are also given the nickname of "spyware", and I have had no Issues for the years I have been using this script in production.
Whenever making system changes or running any type of script, you shouldn't have a game running. you are either doing maintenance/admin stuff, or playing a game.
 But anticheats have false positives sometimes.  There was a time where Valorant would ban you for plugging a samsung phone into your pc while playing due to a bug picking up the phone driver as malicious.

Alright but does ChrisTitusTech e  ou nodes utility interacts with game’s
files?
Le mar. 14 févr. 2023 à 18:02, lavavex @.***> a écrit :

And are you an employee or someone that is in the right place to know that
type of information Le mar. 14 févr. 2023 à 17:51, Mat 6 @.
> a écrit : … <#m_-3696123870391741478_> Alright but is it 100% sure that
it will not get me banned from games for running it at the same time as the
game? Le mar. 14 févr. 2023 à 16:35, lavavex @.> a écrit : > I wouldn't
try running it while a game is open and in a match (beyond > just the fact
that you shouldn't be trying to edit windows while playing a > game, just
do one or the other), mostly because I would assume some > anticheats are
just dumb and might not like powershell touching anything > that the game
might be accessing (i highly doubt the script does but with > anticheats,
you never know what they look for), but i have not had any > issue using
this script on every windows install i have and I play CSGO > Faceit and
Valorant which both have kernel level spyware i mean > anticheat cough

TL;DR: The script should have absolutely no affect on anticheat, but
with > anything that touches registry or system files or runs in
powershell, you > are best to not be running or doing anything with it
while in a anticheat > heavy game. (some have been autobanned for stupider
stuff) > > — > Reply to this email directly, view it on GitHub > <#603
(comment)
https://github.com/ChrisTitusTech/winutil/issues/603#issuecomment-1430408648>,
or unsubscribe >
https://github.com/notifications/unsubscribe-auth/A5W54FQZIWC7MVUE6GVK74TWXP3BVANCNFSM6AAAAAAUTKIIYY
. > You are receiving this because you authored the thread.Message ID: >
@.***> >

I'm not an employee, no. this project is a collaboration effort and
is open source, there are no employees...
 Usually, as long as no program is touching the games memory
while it is running, or having funky dll's doing stuff while game is
running you should be fine. but again, as the script touches registry and
system settings, there is always a 1 in a million chance some poorly coded
anticheat that is overly sensitive will not be happy.
I play Valorant and Faceit CSGO which have such invasive anticheats
that they are also given the nickname of "spyware", and I have had no
Issues for the years I have been using this script in production.
Whenever making system changes or running any type of script, you
shouldn't have a game running. you are either doing maintenance/admin
stuff, or playing a game.
 But anticheats have false positives sometimes.  There was a time where Valorant would ban you for plugging a samsung
phone into your pc while playing due to a bug picking up the phone driver
as malicious.

—
Reply to this email directly, view it on GitHub
https://github.com/ChrisTitusTech/winutil/issues/603#issuecomment-1430506623,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5W54FWX5NJVIOBDRREPEGTWXQFJPANCNFSM6AAAAAAUTKIIYY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Alright but does ChrisTitusTech e ou nodes utility interacts with game’s files? Le mar. 14 févr. 2023 à 18:02, lavavex @.> a écrit :
…
And are you an employee or someone that is in the right place to know that type of information Le mar. 14 févr. 2023 à 17:51, Mat 6 @. > a écrit : … <#m_-3696123870391741478_> Alright but is it 100% sure that it will not get me banned from games for running it at the same time as the game? Le mar. 14 févr. 2023 à 16:35, lavavex @.> a écrit : > I wouldn't try running it while a game is open and in a match (beyond > just the fact that you shouldn't be trying to edit windows while playing a > game, just do one or the other), mostly because I would assume some > anticheats are just dumb and might not like powershell touching anything > that the game might be accessing (i highly doubt the script does but with > anticheats, you never know what they look for), but i have not had any > issue using this script on every windows install i have and I play CSGO > Faceit and Valorant which both have kernel level spyware i mean > anticheat cough > > TL;DR: The script should have absolutely no affect on anticheat, but with > anything that touches registry or system files or runs in powershell, you > are best to not be running or doing anything with it while in a anticheat > heavy game. (some have been autobanned for stupider stuff) > > — > Reply to this email directly, view it on GitHub > <#603 (comment) <#603 (comment)>>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/A5W54FQZIWC7MVUE6GVK74TWXP3BVANCNFSM6AAAAAAUTKIIYY > . > You are receiving this because you authored the thread.Message ID: > @.> > 1. I'm not an employee, no. this project is a collaboration effort and is open source, there are no employees... 2.  Usually, as long as no program is touching the games memory while it is running, or having funky dll's doing stuff while game is running you should be fine. but again, as the script touches registry and system settings, there is always a 1 in a million chance some poorly coded anticheat that is overly sensitive will not be happy. 3. I play Valorant and Faceit CSGO which have such invasive anticheats that they are also given the nickname of "spyware", and I have had no Issues for the years I have been using this script in production. 4. Whenever making system changes or running any type of script, you shouldn't have a game running. you are either doing maintenance/admin stuff, or playing a game.  But anticheats have false positives sometimes.  There was a time where Valorant would ban you for plugging a samsung phone into your pc while playing due to a bug picking up the phone driver as malicious. — Reply to this email directly, view it on GitHub <#603 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A5W54FWX5NJVIOBDRREPEGTWXQFJPANCNFSM6AAAAAAUTKIIYY . You are receiving this because you authored the thread.Message ID: @.***>

NO, they shouldn't touch any game files. If you use the "remove microsoft store apps" option, it will remove any games from the Microsoft store/Game pass, but that is it.
Just don't use powershell while playing a multiplayer game... this whole situation doesn't make any since to me as to why you would be playing a game and running a script in the first place...

I just fear that I might forget to close it, can you explain what is
powershell btw?
Le mar. 14 févr. 2023 à 18:09, lavavex @.***> a écrit :

Alright but does ChrisTitusTech e ou nodes utility interacts with game’s
files? Le mar. 14 févr. 2023 à 18:02, lavavex @.
*> a écrit : … <#m_4243479123698571014_> And are you an employee or
someone that is in the right place to know that type of information Le mar.
14 févr. 2023 à 17:51, Mat 6 @. > a écrit : … <#m_-3696123870391741478_>
Alright but is it 100% sure that it will not get me banned from games for
running it at the same time as the game? Le mar. 14 févr. 2023 à 16:35,
lavavex @.> a écrit : > I wouldn't try running it while a game is open and
in a match (beyond > just the fact that you shouldn't be trying to edit
windows while playing a > game, just do one or the other), mostly because I
would assume some > anticheats are just dumb and might not like powershell
touching anything > that the game might be accessing (i highly doubt the
script does but with > anticheats, you never know what they look for), but
i have not had any > issue using this script on every windows install i
have and I play CSGO > Faceit and Valorant which both have kernel level
spyware i mean > anticheat cough > > TL;DR: The script should have
absolutely no affect on anticheat, but with > anything that touches
registry or system files or runs in powershell, you > are best to not be
running or doing anything with it while in a anticheat > heavy game. (some
have been autobanned for stupider stuff) > > — > Reply to this email
directly, view it on GitHub > <#603
https://github.com/ChrisTitusTech/winutil/issues/603 (comment) <#603
(comment)
https://github.com/ChrisTitusTech/winutil/issues/603#issuecomment-1430408648>>,

or unsubscribe >
https://github.com/notifications/unsubscribe-auth/A5W54FQZIWC7MVUE6GVK74TWXP3BVANCNFSM6AAAAAAUTKIIYY
https://github.com/notifications/unsubscribe-auth/A5W54FQZIWC7MVUE6GVK74TWXP3BVANCNFSM6AAAAAAUTKIIYY
. > You are receiving this because you authored the thread.Message ID: >
@.> > 1. I'm not an employee, no. this project is a collaboration effort
and is open source, there are no employees... 2.  Usually, as long as no
program is touching the games memory while it is running, or having funky
dll's doing stuff while game is running you should be fine. but again, as
the script touches registry and system settings, there is always a 1 in a
million chance some poorly coded anticheat that is overly sensitive will
not be happy. 3. I play Valorant and Faceit CSGO which have such invasive
anticheats that they are also given the nickname of "spyware", and I have
had no Issues for the years I have been using this script in production. 4.
Whenever making system changes or running any type of script, you shouldn't
have a game running. you are either doing maintenance/admin stuff, or
playing a game.  But anticheats have false positives
sometimes.  There was a time where Valorant would ban you for
plugging a samsung phone into your pc while playing due to a bug picking up
the phone driver as malicious. — Reply to this email directly, view it on
GitHub <#603 (comment)
https://github.com/ChrisTitusTech/winutil/issues/603#issuecomment-1430506623>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5W54FWX5NJVIOBDRREPEGTWXQFJPANCNFSM6AAAAAAUTKIIYY
. You are receiving this because you authored the thread.Message ID: @
.**>

NO, they shouldn't touch any game files. If you use the "remove microsoft
store apps" option, it will remove any games from the Microsoft store/Game
pass, but that is it.
Just don't use powershell while playing a multiplayer game... this whole
situation doesn't make any since to me as to why you would be playing a
game and running a script in the first place...
—
Reply to this email directly, view it on GitHub
https://github.com/ChrisTitusTech/winutil/issues/603#issuecomment-1430512534,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5W54FTOQXBPSCJBUOXCLMLWXQGBFANCNFSM6AAAAAAUTKIIYY
.
You are receiving this because you authored the thread.Message ID:
@.***>

I just fear that I might forget to close it, can you explain what is powershell btw? Le mar. 14 févr. 2023 à 18:09, lavavex @.> a écrit :
…
Alright but does ChrisTitusTech e ou nodes utility interacts with game’s files? Le mar. 14 févr. 2023 à 18:02, lavavex @. > a écrit : … <#m_4243479123698571014_> And are you an employee or someone that is in the right place to know that type of information Le mar. 14 févr. 2023 à 17:51, Mat 6 @. > a écrit : … <#m_-3696123870391741478_> Alright but is it 100% sure that it will not get me banned from games for running it at the same time as the game? Le mar. 14 févr. 2023 à 16:35, lavavex @.> a écrit : > I wouldn't try running it while a game is open and in a match (beyond > just the fact that you shouldn't be trying to edit windows while playing a > game, just do one or the other), mostly because I would assume some > anticheats are just dumb and might not like powershell touching anything > that the game might be accessing (i highly doubt the script does but with > anticheats, you never know what they look for), but i have not had any > issue using this script on every windows install i have and I play CSGO > Faceit and Valorant which both have kernel level spyware i mean > anticheat cough > > TL;DR: The script should have absolutely no affect on anticheat, but with > anything that touches registry or system files or runs in powershell, you > are best to not be running or doing anything with it while in a anticheat > heavy game. (some have been autobanned for stupider stuff) > > — > Reply to this email directly, view it on GitHub > <#603 <#603> (comment) <#603 (comment) <#603 (comment)>>>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/A5W54FQZIWC7MVUE6GVK74TWXP3BVANCNFSM6AAAAAAUTKIIYY https://github.com/notifications/unsubscribe-auth/A5W54FQZIWC7MVUE6GVK74TWXP3BVANCNFSM6AAAAAAUTKIIYY > . > You are receiving this because you authored the thread.Message ID: > @.> > 1. I'm not an employee, no. this project is a collaboration effort and is open source, there are no employees... 2.  Usually, as long as no program is touching the games memory while it is running, or having funky dll's doing stuff while game is running you should be fine. but again, as the script touches registry and system settings, there is always a 1 in a million chance some poorly coded anticheat that is overly sensitive will not be happy. 3. I play Valorant and Faceit CSGO which have such invasive anticheats that they are also given the nickname of "spyware", and I have had no Issues for the years I have been using this script in production. 4. Whenever making system changes or running any type of script, you shouldn't have a game running. you are either doing maintenance/admin stuff, or playing a game.  But anticheats have false positives sometimes.  There was a time where Valorant would ban you for plugging a samsung phone into your pc while playing due to a bug picking up the phone driver as malicious. — Reply to this email directly, view it on GitHub <#603 (comment) <#603 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A5W54FWX5NJVIOBDRREPEGTWXQFJPANCNFSM6AAAAAAUTKIIYY . You are receiving this because you authored the thread.Message ID: @ .> NO, they shouldn't touch any game files. If you use the "remove microsoft store apps" option, it will remove any games from the Microsoft store/Game pass, but that is it. Just don't use powershell while playing a multiplayer game... this whole situation doesn't make any since to me as to why you would be playing a game and running a script in the first place... — Reply to this email directly, view it on GitHub <#603 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A5W54FTOQXBPSCJBUOXCLMLWXQGBFANCNFSM6AAAAAAUTKIIYY . You are receiving this because you authored the thread.Message ID: @.***>

Not to be rude, but if you have no clue what powershell is, you should not be touching github or even running the script. you may be learning, and that is fine, but when you dont even know the name of the program you have to open to run the script you are asking a question about, i am slightly worried.
I will no longer respond to this thread as I have given up trying to explain this to you

it doesnt do anything that would get you banned just dont run the scripts while playing an MP game, run them before and/or after.  Simple


proof: relax parsing in GetDecDisplay

If an old asset didn't have this field, then we can just assume that the value is zero, rather than error out.
Fixes https://github.com/lightninglabs/taproot-assets/issues/1001

That return was used to enter this case:
https://github.com/lightninglabs/taproot-assets/blob/5236deae2e3b3c73610063062643c5f32ab10ca4/proof/meta.go#L284
Which would trigger adding the display to the JSON. So that should get updated as well.


task: yield

In cooperative multitasking, a process does not stop running until it voluntary decides to do so.The act of a process voluntarily suspending itself is called yielding. Ideally, processes yield often, giving each runnable process a decent chunk of the processor, but the operating system cannot enforce this.The shortcomings of this approach are manifest: The scheduler cannot make global decisions regarding how long processes run; processes can monopolize the processor for longer than the user desires; and a hung process that never yields can potentially bring down the entire system.
https://student.cs.uwaterloo.ca/~cs452/F23/assignments/kernel.html
Test it with the creation of multiple tasks on the same priority level

The task needs to be moved to the end of the priority list.


feat: Removed sub sections of Tutorial from documentation dropdown

Can you update the PR to indicate why we are removing it (conversations, links, etc), for anyone in the future doing maintaince


stable download URL and version?

In https://github.com/hayguen/clipgrab, i've added explicit check for available version of yt-dlp on github, by parsing __version__ line of https://raw.githubusercontent.com/yt-dlp/yt-dlp/master/yt_dlp/version.py
It's simple to compare against the output of yt-dlp --version and allow an update by retrieving e.g. https://github.com/yt-dlp/yt-dlp/releases/latest/download/yt-dlp  for Linux.
Despite reading issue https://github.com/yt-dlp/FFmpeg-Builds/issues/10,  i'm still confused howto find out the right URL for stable releases, cause those contain the version in the filename, as in https://github.com/yt-dlp/FFmpeg-Builds/releases/download/latest/ffmpeg-n5.1-latest-linux64-gpl-5.1.tar.xz
and the Links at https://github.com/yt-dlp/FFmpeg-Builds#downloads do only provide the latest non-stable master downloads.
Can someone give me a hint, how to automatically determine the latest stable online version and the corresponding URLs?
In addition, it should be possible the retrieve the same version string or just the iso-date running sort of ffmpeg --version-date.

Use GitHub API https://api.github.com/repos/yt-dlp/FFmpeg-Builds/releases/latest

@pukkandan very interesting. thank you!
thus, i should be able to check the name and the browser_download_url of the assets list, filtering out names containing master and find the right platform/architecture.
but, the created and updated fields look useless!?
how would i compare with the output of the installed local binary?,
when ffmpeg -version outputs:

ffmpeg version N-108893-g17987b371b-20221029 Copyright (c) 2000-2022 the FFmpeg developers

but name is

ffmpeg-n5.1-latest-linux64-gpl-5.1.tar.xz

there's no '5.1' in 'N-108893-g17987b371b-20221029' or vice versa!
hmmmm, '17987b371b' looks to be a commit/sha-id. would be great, if this would be part of the download name!
but the above commit/sha-id is really new:
https://github.com/FFmpeg/FFmpeg/commit/17987b371bf8ebc284950115a7e2987baa2f2685
why is that commit contained in a stable build?
it should be possible to detect/avoid download, just cause of an automatic rebuild without any change.

You will have to ask these upstream. We only add the patches to the builds - not modify the build system or versioning

@pukkandan : thanks again for fast reply.
sorry, i've to ask again, cause now realizing https://github.com/BtbN/FFmpeg-Builds
should i aske there? or, somewhere at https://ffmpeg.org/ ?

I meant BtBN, but I cannot know if they will further redirect you to ffmpeg. Somewhat related: https://github.com/BtbN/FFmpeg-Builds/issues/199

@pukkandan Thank You

see https://github.com/BtbN/FFmpeg-Builds/issues/210 for the result


Mergify configuration

This PR adds Mergify configuration to automatically merge approved PRs once CI succeeds as well as automatically merge dependabot PRs.
Test Plan
Merge this and :crossed_fingers: that the dependabot PRs automatically merge.

@fleupold Does Mergify need to be enabled somewhere as well?

Yes, we need to add it in the Gnosis repo settings: https://github.com/organizations/gnosis/settings/installations/10251815
I just added it for this repo.

Note, that mergify doesn't run on files that change the mergify.yaml


Remove the remaining closure in the src/core/function.js file

Given that the code is written with JavaScript module-syntax, none of this functionality will "leak" outside of this file with these changes.
By removing this closure the file-size is decreased, even for the built pdf.worker.js file, since there's now less overall indentation in the code.

/botio test

Thank you for doing this!


Fix label on Zone network setup step

If a zone holds a storage network then the following label issue happens:

After fixing the label:

@blueorangutan package

@GabrielBrascher a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/897 (JID-3743)


Fix typos

Found by running 'codespell -S .git,dist -L ons'.
Signed-off-by: Mateusz Gozdek<EMAIL_ADDRESS>

Ping @deitch

Thanks for the cleanup.


Add support of int8/uint8/int16/uint16 for tf.subtract

This fix adds support of int8/uint8/int16/uint16 for tf.subtract (on CPU only)
This fix fixes #12571.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Jenkins, test this please.

I tested locally for //tensorflow/python/kernel_tests:matrix_solve_ls_op_test and it passes without any issues. I think the Jenkins timeout failure (matrix_solve_ls_op_test) is not related to this PR.

Yes, that failure is unrelated. Thanks!


Shortcut Headline problem

I use the shortcuts ⌘ + 1…6 to create a headline. When I do this before entering text the Ursine theme somehow moves the hashtags for the headline a line below or above the cursor input.
It’s hard to explain. Here are some screenshots before hitting the shortcut and after (left side normal view, right side source code view).

I have no clue why this is happening or if it’s just me. Other themes seem to have no problem.

Same here.

(Pardon the late reply, college semester got real busy this time around)
But it's also happening on my end, but I honestly have no idea what might be causing it. It's definitely the theme, because every other theme works fine with me. How odd that a stylesheet is causing this...

I think it is due to the position: absolute in h1::before (Line 423)
I had a look at some other Themes that outdent the headline information. They use ::before pseudo-element and negative margin. I managed to change it in the the generated css for myself. I am sure you had something in mind with the absolute position and I don’t know for sure if my “solution” does not break anything else.

Yeah, it was that! I think back then I had an unorthodox way of styling that part, which is why I used ::after; but I forgot about it afterwards. Konfido's PR was enough to fix things and it'll be up in the next release once I get things together


remove  that snuck in

Signed-off-by: Felix Kronlage-Dammers<EMAIL_ADDRESS>

Thanks for spotting!


Premint - show how to decode creator attribution event

Current dependencies on/for this PR:

main

PR #253

PR #244

PR #260   👈
PR #255

PR #256

PR #258
PR #257

PR #246

PR #247

PR #245

This comment was auto-generated by Graphite.


Should the back button close open modals?

If so, how would that work? Should it be part built-into the component? Or do consumers set that up?
(Thumbtack employees can refer to FE-1589 for previous context.)

Specifically this is the expected behavior on mobile devices

Trying to clarify expected behaviour here. Here's what I have so far

User can specify a custom hash for their modal, for example #comments
When the modal is opened, this hash is added to the URL
When the modal is closed (by any method) this hash is removed from this URL
a. Close button
b. Browser back button
c. ESC key
Pressing back while the modal is open closes the modal (and removes the hash, see 3)
Pressing forward again when the modal has been previously opened and closed will open it again (but if it has never been opened, it will do nothing).
Pressing back while the modal is closed returns to the previous page
Pasting the URL with the hash (eg. #comments) causes the modal to open itself when the page loads. If the hash is not present the page loads with the modal closed.

Here's a simple demo that supports most of the core assumptions. Back, forward, and updating the URL all work as expected. The only thing missing is #7 (opening the URL with the hash already present does not cause the modal to appear. But this could be easily added with a componentDidMount check.
You will need to preview the demo here rather than inside codesandbox preview window, to get the real browser button behaviour.
Dan and I have chatted and after outlining the assumptions above, we are leaning towards recommending that people who want this behaviour implement it as shown in that demo in their parent component, and we will close the issue for now.
If this same problem comes up multiple times, and there are many places in website that need this behaviour, we can look into extracting out the functionality into a shared component or utility that wraps the default modal. We can also raise this in the FE committee if you think it warrants further discussion.


deploy creates zip wrongly with folders

There is a bug when it creates a zip with a path in the settings. Amazon wants the contents of the zip, not the folder zipped.
See here: http://docs.aws.amazon.com/lambda/latest/dg/walkthrough-s3-events-adminuser-create-test-function-create-function.html
For example:
myfolder
|-myfunc.js
|-node_modules
|--morestuff
zip should look like:
myfunc.js
|-node_modules
|--morestuff

How are you specifying the path value in the config file?

I tried both
path: s3resize/
and
path: s3resize
I feel like i should mention I'm on windows.

Same here. Tried with the default setup, and amazon would not find the module "AutoStart"
autoStart/AutoStart.js

with config
lambda:
  name: AutoStartLalala
  zipfile_name: AutoStart.zip
  description: Start ....
  path: autoStart/
  handler: AutoStart.handler
  runtime: nodejs
  memory_size: 128
  timeout: 3
  mode: event
  test_data: sample_event.json
  event_sources:
    -
      arn: arn:aws:s3:::bucketName
      events:
        - s3:ObjectCreated:*

after manually zipping only the contents of the autoStart folder and reuploading, it worked.

btw, I installed kappa using pip and got version 0.1.0

Could confirm logic working with latest develop branch

Ok, I will look into this today.  Sorry for the slow response.

Did you try it with this config?
lambda:
  name: AutoStartLalala
  zipfile_name: AutoStart.zip
  description: Start ....
  path: autoStart
               ^^^^ - note no trailing / character

Oh, I just saw your last comment.  Are you saying that it is working correctly in the latest develop branch?

I just tried it on a Windows instance using the develop branch of the code and it seems to have worked.  I'm going to fix up a few other things and then do a new kappa release.

I just pushed out a 0.2.0 release.  Please give that a try and let me know if it clears up your issue.

Yes, it worked on the develop branch for me. Will pull the 0.2.0 today

I'm going to close this issue.  AFAIK, it is working correctly now.


Incomplete documentation for HRMC MTD VZAT web request headers format

In Set up application-specific parameters for MTD VAT web request headers format there is no example/suggestion given to the users as to what URL to set in Gov-Vendor-Public-IP & Gov-Client-Public-IP in order to collect the IP Address.
It should be added an example section similar to the one mentioned in the Business Central documentation page:
https://learn.microsoft.com/en-us/dynamics365/business-central/localfunctionality/unitedkingdom/fraud-prevention-data
E.g.:
IP addresses
Your company can sign up for any of the following endpoints:
https://api.db-ip.com/v2/free/self
https://www.cloudflare.com/cdn-cgi/trace
https://api.ipify.org/
https://jsonip.com/

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

ID: 13949dac-f9ce-5110-9f01-129964a5ee41
Version Independent ID: 52d74b51-fdc8-301c-49c3-cd77a1480c82
Content: Prepare your environment to interoperate with HMRC's MTD VAT web service - Finance | Dynamics 365
Content Source: articles/finance/localizations/emea-gbr-mtd-vat-integration-setup.md
Service: dynamics-365-finance
Product: ****
Technology: ****
GitHub Login: @AdamTrukawka
Microsoft Alias: atrukawk

@vbrisan, thank you for your feedback. We have passed this feedback onto the product team for consideration.


Add unboxed vector instances

I suggest, analogous to the Vector instance:
import qualified Data.Vector.Unboxed as VU
import qualified Data.Vector.Unboxed.Mutable as VUM

instance (VU.Unbox a, Store a) => Store (VU.Vector a) where
    size = sizeSequence
    poke = pokeSequence
    peek = VU.unsafeFreeze =<< peekMutableSequence VUM.new VUM.write
    {-# INLINE size #-}
    {-# INLINE peek #-}
    {-# INLINE poke #-}

We do actually generate all of the different instances for unboxed vectors - https://github.com/fpco/store/blob/4f65518c2a136f513d6fa07dfb65b45d7df11d81/src/Data/Store/TH/Internal.hs#L319
It's written this way for efficiency, so that we can use memcpy.  That said, it seems like the instances module takes quite a lot of memory to compile - https://github.com/fpco/store/issues/91 .  Might be worthwhile to go for a simpler implementation with less compilation.

@mgsloan Your answer made sense to me, and I can see these instances in the haddocks of Data.Store.Internal:

(Have to hover them to see that the Vector is an Unboxed one here.)

A few years later, another question comes up though:
How does one do Store instances of Unboxed vectors of one's own data types?
For example, I have this neat data type with some "phantom" parameters:
data EstimatedPose from to = EstimatedPose
  { epRotation :: M from to
  , epTranslation :: V to
  , epTriangulatedPoints :: VU.Vector (V from)
  , epInlierIndices :: VU.Vector Int
  } deriving (Eq, Show, Generic)

Trying to make an derived
instance Store.Store (EstimatedPose from to)`

I'm confronted with:
No instance for (Store.Store (VU.Vector (V from)))

What's the correct way to provide such instances?

Or perhaps for a simpler example:
data MyVector = MyVector (VU.Vector V4)
where V4 is from linear and has Unbox and Storable instances.

My current workaround: Roundtripping via Storable (for which e.g. V3 also has instances):
import qualified Data.Vector.Generic as VG
import qualified Data.Vector.Storable as VS
import qualified Data.Vector.Unboxed as VU

data MyVector = MyVector (VU.Vector V3)

instance MyVector => Store.Store MyVector where
  size = contramap VG.convert Store.size
  peek = MyVector . VG.convert <$> Store.peek
  poke (MyVector vu) = Store.poke (VG.convert vu :: VS.Vector V3)

Hmm, I don't know of a straightforward way to do this with current store functionality.
The main approach that comes to mind is to create a function similar to the function for deriving instances for unboxed vectors, which takes a list of types to derive (Store (VU.Vector ...)).  It would then find the matching data family instance, and derive store for the constructor on the rhs.


Fixes #171 - Adjust pytest logging of Proton for clarity

If I understand correctly, the change in _init.py (to set proton logging level to INFO) makes sure that the proton logging is not too verbose. That makes sense.
The change to tox.ini.in (e.g. log_cli_level) is to control the log level of pytest logging records ? Are these two changes related ?


Documentation for FileInfo.Exists and File.Exists is misleading

Documentation for FileInfo.Exists and File.Exists is misleading
File.Exists "returns" currently reads:
true if the caller has the required permissions and path contains the name of an existing file; otherwise, false. This method also returns false if path is null, an invalid path, or a zero-length string. If the caller does not have sufficient permissions to read the specified file, no exception is thrown and the method returns false regardless of the existence of path.
Should read:
true if the caller has the required permissions and path contains the name of an existing file; otherwise, false. This method also returns false on an invalid filename or if any error prevents determining if the file exists or not.
FileSystemInfo.Exists "returns" currently reads:
true if the file or directory exists; otherwise, false.
Should read:
true if the file or directory is known to exist; otherwise, false. This method returns false if an error prevented determining if the file exists or not. If FileSystemInfo.Exists returns false, call FileSystemInfo.tLastWriteTimeUTC to throw the exception for the error. If LastWriteTimeUTC returns at all, the file or directory is known to not exist.
Remakrs on FileSystemInfo.Exists should contain this blob:
FileSystemInfo reports on existence based on a snapshot of the file in question that may be invalid by the time FileSystemInfo.Exists() returns. Therefore, the following code can still throw FileNotFoundException: var fileinfo = new FileInfo(pathstring); if (fileinfo.Exists()) using (var f = new FileStream(fileinfo.FullName, FileMode.Open) { /* ... */ } else {var ignoredvalue = fileInfo.LastWriteTimeUTC; }
General Commentary (why these specific changes):
Not checking if the result of FileInfo.Exists is trustworthy is almost always a bug, and is always a bug in batch processing (and therefore all library modules). File.Exists() has no use for the reason of there being no way to check if false was a successful false or an error. Just calling fileInfo.LastWriteTimeUTC != new DateTime(1601, 1, 1, 0, 0, 0) as FileInfo.Exists won't work because that's a valid time that somebody could have set a file to. (And I don't trust that value's right on Unix anyway but I digress.)
Most of the time, FileInfo isn't what you want anyway. The static file provider in MVC has some bizarre corner case bugs that would not exist had it checked for a file's existence by opening it and catching FileNotFoundException.
Original Discussion: https://github.com/dotnet/corefx/issues/26893

Please note the XML comments as viewed by Visual Studio also need to be updated.

The key thing is that the API returns false if the file doesn't exist or if it is unable to determine the file's status (which can be a number of reasons).
A common mistake should be called out, and that is making assumptions based on the results of this or any other I/O API is dangerous. I/O is transient, and as such you always need to catch errors when attempting to access or modify files (reading/writing/copying/etc.). For example: If File.Exists() returns true it was true at one point during the call to File.Exists(). It may no longer be true even by the time the API returns. Some other process may have moved or deleted the file, remote or removable media may become disconnected or unreachable. Even if the risk of the file being deleted/disconnected is low, permissions can fail for a variety of reasons, including interaction with things such as antivirus programs and indexers.

Thanks @jhudsoncedaron and @JeremyKuhne. The way I see this, we need to update the API docs to change the description/remarks and some of the IO conceptual articles to explain the transient nature of I/O operations. The IntelliSense gets updated with the content fixes we make here on a certain cadence.

@mairaw The way I ended up caring about this was a transient network failure causing a spurious false from File.Exists() causing huge problems in production and to me that's what needs documenting the most.

Thanks for the example @jhudsoncedaron. That helps.


added topic deletion on by default

As I believe this image is intended for quick development, I think it's safe to have topic deletion enabled by default.

Thanks! Actually this meant to be the default and only behavior for fast-data-dev, so it is a bug we had it disabled. I am not sure how we missed it for this long.
I will merge your request later today. Cheers!

Merged. I renamed DISABLE_TOPIC_DELETE to TOPIC_DELETE, by default enabled.
Thanks! I gave you a shout out in the news box for this, hope you don't mind. :D

Thanks heaps ! As antonios knows I'm a big fan of your image and heavily
advertising it. Happy to contribute :) really looking forward to v 3.2.0
On 22 Apr. 2017 7:53 pm, "Marios Andreopoulos"<EMAIL_ADDRESS>wrote:

Merged. I renamed DISABLE_TOPIC_DELETE to TOPIC_DELETE, by default
enabled.
Thanks! I gave you a shout out in the news box for this, hope you don't
mind. :D
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Landoop/fast-data-dev/pull/23#issuecomment-296362020,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT4rads7vdA7dSfE4Kom-GYVj3ItJSqOks5ryc4HgaJpZM4NE-aJ
.


Include the log file path when saying to See the log for details

After executing actions in chocolatey, there is a message that says See the log for details. - but where is that log file?
I suggest including the path in that message to make it easier:
See the log for details (C:\ProgramData\chocolatey\logs)

For more information: https://groups.google.com/forum/#!topic/chocolatey/UVncL7PxXRg

Moved to choco repo


removed unused import from reactRouterDom

Description
Deleted unused import statement from reactRouterDom.
Fixes # (issue)
How to test?
npm run code:lint
Checklist

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] This PR is ready to be merged and not breaking any other functionality

Good job @soujanyapolavarapu :)

Thankyou tithi 😊


What is the status of this fork?

Hi,
I came here from your toml repository. This repo claims:

The module passes the TOML test suite which is a fork of BurntSushi's TOML test suite.

This worries me: why can't there be a single test repo?
And more importantly: does it mean there are incompatibilities?
Is this fork competing with the base repo? Isn't it possible to merge the changes back to the main repo? I didn't see any PRs from this repo to the other.
Since this fork is not just temporary (because it is advertised on your toml project), I'd prefer if there was at least a section at the top of the README explaining why you keep a long lived-fork and what are the main differences/incompatibilites.
It would greatly help to assess the situation when looking for an interoperable format.
/cc @uiri

Hi Charles,
For my toml library, this repo is the single test repo. There are no incompatibilities with the main repo (that I recall). This repo exists solely to merge in additional tests which I want to use for my toml library that have not yet been merged in the main repo. I link to this repo rather than the main repo so that contributors to my toml library know that there are additional tests that they need to run their changes against.
Actually, I do view this fork as temporary. Alas, there is nothing quite so permanent as a temporary solution so this fork has lived for quite a bit longer than I would have liked when I initially created it. It should be very easy to merge this repo back into the main one.
@BurntSushi could you please comment on the current status of the main toml-test repo? If you no longer wish to maintain it, I would be willing to step in as I am still using it.

Thanks for the reply.
It's good to know that you are careful about not fracturing the ecosystem.
Given your work on the Python lib, I would also consider that the best outcome would be to merge your changes of even grant you write access to the base repo.
If it's not possible, could you add your comment above somewhere in the README? It's a good reply.

I don't see myself maintaining toml-test any time in the near future. @uiri How do you want to handle this? I'd be willing to transfer ownership to you or just add you as a contributor to the repo where it is now.
IIRC, there was at one point a plan to move it to the TOML organization itself, so it might be good to keep that in mind for the future.

@uiri Done! Thanks. :-)

Closing this issue now that you have access to the main repo. Thank you all.


regarding tests

I believe generic integration test sets could be written in such ways:

partition midi file
input midi file
expected output midi file

each set in a separate directory
running the library on 2 first inputs and perform a diff on the expected output midi file and the output given by the library.
a bijective "midi file <-> text file" format would help keeping it all in plain text to ease with displaying the diff.
just a suggestion :)

nice idea :)
we've been wondering about integrating MIDI capabilities directly into the library, but for the moment it only consumes and produces some non MIDI standard structs.
@rblard already made an ossia score midifile performer plugin, including a midifile parser which works pretty well :)
We could indeed use a MIDI library as a submodule, write adapters for the MIDI messages, and run some tests on MIDI files from this repo ...
To be investigated

Maybe a textual representation of this non standard struct works the same for this purpose?
I think it makes adding tests and adjusting them better experience as it doesn't involve changing the test code.
It is a bit of upfront work, but generally makes the test codebase more resilient, especially to refactors on the actual code under test, etc.

I just added libremidi to Catch2 tests and it seems to work.
Getting closer to easy testing of midi files ...


[E2E] Fix file manage e2e error

Purpose of the pull request
https://github.com/apache/dolphinscheduler/runs/4963908433?check_suite_focus=true

Codecov Report

Merging #8232 (19b2201) into dev (07a15b7) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #8232   +/-   ##
=========================================
  Coverage     44.99%   44.99%
  Complexity     3955     3955
=========================================
  Files           672      672
  Lines         26218    26218
  Branches       2826     2826
=========================================
  Hits          11798    11798
  Misses        13292    13292
  Partials       1128     1128

Continue to review full report at Codecov.

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


Cached response content is inconsistent with original response

The problem
Cached result is inconsistent with what originally was loaded from server.
Expected behavior
Cached response text should exactly match original response.
Steps to reproduce the behavior

create a file containing empty json array (echo "[]" > empty_arr.json)
start a server pointing to that file (python3 -m http.server 9000)
ensure content is loaded properly on the first loading or bypassing cache:

with requests_cache.disabled():
    tmp = requests.get('http://localhost:9000/empty_arr.json')
print("loaded text:", tmp.text)
print("loaded json:", tmp.json())

# loaded text: []
# loaded json: []

Trigger another load, ensure it's done from cache:

tmp = requests.get('http://localhost:9000/empty_arr.json')
print("loaded text:", tmp.text)
print("loaded json:", tmp.json())

# loaded text: (empty!!!)
# loaded json: JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Environment

requests-cache version: 1.2.0
Python version: 3.12.1
Platform: MacOS 14.3.1

I'm not able to reproduce this locally. Previous versions of requests-cache didn't handle an empty JSON array correctly, but I believe that was fixed in 1.1. If you have cached responses older than that, that could cause what you're describing.
What results do you get if you clear your cache or create a new one? If you still get the same results, please post a full example including any options you're passing to install_cache().
Examples
Here's what I see with a new cache created with requests-cache 1.2 and python 3.12:
Example with patching:
import requests
from requests_cache import install_cache

install_cache()
r = requests.get("http://localhost:9000/empty_arr.json")
print(f"Text: {r.text.strip()}")
print(f"JSON: {r.json()}")

Output:
Text: []
JSON: []

Example with CachedSession:
from requests_cache import CachedSession

session = CachedSession()
r = session.get("http://localhost:9000/empty_arr.json")
print(f"Text: {r.text.strip()}")
print(f"JSON: {r.json()}")

Output:
Text: []
JSON: []

Seems like this problem is specific to filesystem backend, there is full setup:
# echo "[]" > empty_arr.json
# python3 -m http.server 9000

import requests
import requests_cache

requests_cache.install_cache(
    "http_cache",
    backend='filesystem',
    serializer='json',
    decode_content=True
)
requests_cache.clear()

# Make initial request, ensure it came directly from server and not cached
r_orig = requests.get('http://localhost:9000/empty_arr.json')
assert r_orig.from_cache == False

original_response = r_orig.text
original_json = r_orig.json()
print("original response:", original_response)
print("original json:", original_json)

# Make the same request again, make sure cached response is returned
r_cached = requests.get('http://localhost:9000/empty_arr.json')
assert r_cached.from_cache == True

cached_response = r_cached.text
print("cached response:", cached_response)
assert original_response == cached_response, "Cached response is not equal to original response"

cached_json = r_cached.json()  # throws error
print("cached json:", cached_json)

The main problem here has been fixed in v1.2.1.
However, the comparison original_response == cached_response will still fail. The original response body will have a trailing newline, but since decode_content=True saves the content in a plain JSON file instead of a binary format, any leading or trailing whitespace won't be preserved.
This is unavoidable, and the Content-Length response header is updated accordingly; in this case you'll see a value of 2 bytes instead of 3. Hopefully that won't present a problem in practice, but if it does, you can continue to use  decode_content=False.


Which one is the right manpage?

There are pms.1 and manpage.md. I guess the former is outdated, because the example for configuring the topbar does not work anymore. Is there a reason why the old one is still included during the build process?

You are correct, the former is outdated and should be removed. It is still included because it is undecided whether the auto-generated pms.1 (from manpage.md) should be part of the repository or not. If not, compiling and installing will require having pandoc installed.

If pandoc is too heavyweight, you could try md2man or possibly some other tool for the conversion.

Thanks for the suggestion. I don't really consider pandoc too heavyweight. The requirement should probably be that pms.1 is generated when running make dist, and I need to figure out how to configure autoconf and automake to do this.


Drill still not found on fm-transmitter

I am using the test-fm-transmitter branch to figure out what is going on with this. I added zero:'L'  to the options like you suggested but it's still not working.

Ah that was my bad, I forgot that I found two problems with fm-transmitter. It:

Does not specify zero-suppression and uses leading-zero suppression by default (Excellon default is to keep leading zeros and suppress trailing zeros)
Uses coordinates with 2 integer places and 5 decimal places, the Excellon spec says files in inches should use 2 integer places and 4 decimal places

NC drill files have no mechanism to express this format other than comments that some CAD programs add, and gerber-parser will catch them and use them
This drill file has no such comments

To get this to render properly, you need: {zero: 'L', places: [2, 5]}

Had messed up the option input, works with those options, thanks!

Is there any way we could detect a situation like this where additional options are required?
I just had a look at the viewBox without the right options and it's way out from the rest of them of course. Can we set some sensible thresholds there? I am not sure though why the out, tss, and bcu are so different from tcu and the sm layers and what that would mean for the thresholds.
tcu [164.685, 65.51, 795.3149, 490.87]
drl [18425.19, 12025, 72600.81, 37052]
out [10, 10.51, 1000, 570]
drw [0, 0, 0, 0]
tss [10, 10.51, 1000, 570]
tss [10, 10.51, 1000, 570]
bcu [25, 25.51, 970, 540]
tsm [161.685, 62.51, 801.31489, 496.87]
bsm [161.685, 75.01009, 801.31489, 460.99982]

Yeah I've been thinking about this as well. Given some arbitrary standard-deviation threshold, we should be able to throw a warning (not that we currently have a warning interface). We could even do some basic reasoning with what the problem is:

Wrong units if viewbox is different by a factor of about 25.4,
Wrong number of decimal places if different by a factor of about 10
Wrong suppression or combination of problems if viewbox is just wildly different

On a side note, did you generate these Gerber's yourself? They've got the same problem as the Bus Pirate gerbers where the bottom silkscreen has a file ending that's usually used for top silkscreen (hence your two tss logs above).