Sudoku Solver using Vanilla JS

Pull Request for WebMasterLog 💡
Issue Title: Sudoku Solver using Vanilla JS

Name of the project : Sudoku Solver
Category: Vanilla-JS
Name: Sohan RoyChowdhury
GitHub ID: https://github.com/SohanRC
Idenitfy yourself: JWOC 2024

Closes: #56

Describe the add-ons or changes you've made 📃
This is a Sudoku Solver project where the website automatically solves a sudoku based upon the inputs provided by the user !!
Type of change ☑

What sort of change have you made:

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested? ⚙
Tested on local system and it worked perfectly !
Checklist: ☑

[X] My code follows the Contributing Guidelines & Code of Conduct of this project.
[X] This PR does not contain plagiarized content.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly wherever it was hard to understand.
[X] I have made corresponding changes to the documentation.
[X] My changes generate no new warnings.
[X] I have added things that prove my fix is effective or that my feature works.
[X] Any dependent changes have been merged and published in downstream modules.

Screenshots 📷

@Avdhesh-Varshney Please review this PR now !! I have resolved the conflicts !!

@Avdhesh-Varshney Please review it now !


”ERROR: [v++ 60-398] cf2sw failed“ when run "p2p_bandwidth" with SmartSSD U.2

I configured the environment according to the instructions UG1382.
Then I run the vitis_accel_examples,after passed software/hardware eumulation,I make all=hw....but failed.
I also get the same wrong result when run the helloworld example.
I would like to know what causes this?
details:
[17:07:35] Starting bitstream generation..
[17:12:37] Creating bitmap...
[17:14:27] Writing bitstream ./level0_i_ulp_my_rm_partial.bit...
[17:14:27] Finished 6th of 6 tasks (FPGA bitstream generation). Elapsed time: 00h 06m 52s
[17:14:27] Run vpl: Step impl: Completed
[17:14:27] Run vpl: FINISHED. Run Status: impl Complete!
INFO: [v++ 60-1441] [17:14:28] Run run_link: Step vpl: Completed
Time (s): cpu = 00:00:29 ; elapsed = 01:06:37 . Memory (MB): peak = 2064.074 ; gain = 0.000 ; free physical = 24402 ; free virtual = 122272
INFO: [v++ 60-1443] [17:14:28] Run run_link: Step rtdgen: Started
INFO: [v++ 60-1453] Command Line: rtdgen
INFO: [v++ 60-1454] Run Directory: /home/embed/DYL_CSD/Vitis_Accel_Examples-2021.2/host/p2p_overlap_bandwidth/_x.hw.xilinx_u2_gen3x4_xdma_gc_2_202110_1/link/run_link
INFO: [v++ 60-991] clock name 'blp_s_aclk_kernel_ref_clk_00' (clock ID '0') is being mapped to clock name 'DATA_CLK' in the xclbin
INFO: [v++ 60-991] clock name 'blp_s_aclk_kernel2_ref_clk_00' (clock ID '1') is being mapped to clock name 'KERNEL_CLK' in the xclbin
INFO: [v++ 60-1230] The compiler selected the following frequencies for the runtime controllable kernel clock(s) and scalable system clock(s): Kernel (DATA) clock: blp_s_aclk_kernel_ref_clk_00 = 300, Kernel (KERNEL) clock: blp_s_aclk_kernel2_ref_clk_00 = 500
INFO: [v++ 60-1453] Command Line: cf2sw -a /home/embed/DYL_CSD/Vitis_Accel_Examples-2021.2/host/p2p_overlap_bandwidth/_x.hw.xilinx_u2_gen3x4_xdma_gc_2_202110_1/link/int/address_map.xml -sdsl /home/embed/DYL_CSD/Vitis_Accel_Examples-2021.2/host/p2p_overlap_bandwidth/_x.hw.xilinx_u2_gen3x4_xdma_gc_2_202110_1/link/int/sdsl.dat -xclbin /home/embed/DYL_CSD/Vitis_Accel_Examples-2021.2/host/p2p_overlap_bandwidth/_x.hw.xilinx_u2_gen3x4_xdma_gc_2_202110_1/link/int/xclbin_orig.xml -rtd /home/embed/DYL_CSD/Vitis_Accel_Examples-2021.2/host/p2p_overlap_bandwidth/_x.hw.xilinx_u2_gen3x4_xdma_gc_2_202110_1/link/int/copy_kernel.link.rtd -o /home/embed/DYL_CSD/Vitis_Accel_Examples-2021.2/host/p2p_overlap_bandwidth/_x.hw.xilinx_u2_gen3x4_xdma_gc_2_202110_1/link/int/copy_kernel.link.xml
ERROR: [CF2SW 83-2179] Unable to resolve addressing information on axi_interconnect_0, sptag bank0.  Either address assignment failed, or the PFM.AXI_PORT property on axi_interconnect_0 indicates a "memory" option that was set to an slave component and/or slave segment that are not reachable from axi_interconnect_0.  Currently instance = ddrmem_1 and segment = C0_DDR4_ADDRESS_BLOCK.  Please correct the platform design.
ERROR: [v++ 60-398] cf2sw failed
INFO: [v++ 60-1652] Cf2sw returned exit code: 255
INFO: [v++ 60-1442] [17:14:30] Run run_link: Step rtdgen: Failed
Time (s): cpu = 00:00:02 ; elapsed = 00:00:02 . Memory (MB): peak = 2064.074 ; gain = 0.000 ; free physical = 24401 ; free virtual = 122272
ERROR: [v++ 60-661] v++ link run 'run_link' failed
ERROR: [v++ 60-626] Kernel link failed to complete
ERROR: [v++ 60-703] Failed to finish linking
INFO: [v++ 60-1653] Closing dispatch client.

Hi @sdudyl ,
Can you please post your query to forum to get help from support team?
https://support.xilinx.com/s/topic/0TO2E000000YKYAWA4/vitis-acceleration-acceleration?language=en_US

Hi @sdudyl , Can you please post your query to forum to get help from support team? https://support.xilinx.com/s/topic/0TO2E000000YKYAWA4/vitis-acceleration-acceleration?language=en_US
Thank you!I have solved this problem.


fix(alert): show scrollbars not by default - Fixes #14178

Short description of what this resolves:
y Scrollbar will be showed to alert and action sheet although it's not needed. So I convert it to auto so the scrollbar will only be shown if needed
Before

After

Fixes: #14178

Has this been tested on the scrollable options in Action Sheet? I'm not able to scroll them with this PR: http://localhost:3333/src/components/action-sheet/test/scrollable-options
You may need this PR to reproduce: https://github.com/ionic-team/ionic/pull/16789

@brandyscarney I am not able to scroll with the master branch. So my PR is not the cause that there are no scrollbars ;)
Master branch:

So I'm seeing scrolling working fine on master in a browser (tested in chrome & safari):

However, I am seeing some weird things happening with press events interfering with scrolling in iOS and it sometimes not working.

@brandyscarney Yeah scrolling is also fine with this PR, I thought you missed the scrolling bars

Merged. Thank you!


Set container name to a deterministic value when launching

When launching an Aspire app, the created containers are just given the default random docker names:
CONTAINER ID   IMAGE                  COMMAND                  CREATED         STATUS         PORTS                       NAMES
5ce3d9b00a89   plenary/shell:latest   "/shell/entrypoint.sh"   2 minutes ago   Up 2 minutes                               nice_swirles
166f578bc2a6   postgres:latest        "docker-entrypoint.s…"   2 minutes ago   Up 2 minutes   <IP_ADDRESS>:55839->5432/tcp   friendly_kirch

I have configured my app with a "shell" container to allow invoking dotnet tools like dotnet-ef in the context of a service, and I need to docker exec into it in order to run those commands. It'd be helpful if the containers had a consistent name like [app]_[resource] or similar, much like how docker-compose sets names. Then I could avoid having to look up the container name and could more easily script interactions with running containers (such as execing in to them).

Also worth following this issue https://github.com/dotnet/aspire/issues/398

Yep, I think the container name change is somewhat orthogonal to my specific case, so I filed this. I can see utility in exec-ing in to the running containers for various reasons. I'll chime in on the other issue with specific thoughts for EF tooling.

@davidfowl @karolz-ms @mitchdenny would changing the container naming logic require changes just to the app model or in DCP too?

Container banking is handled by DCP.

@DamianEdwards I was actually going to send a PR but I quickly found that it was handled by DCP 😢

@mitchdenny short of better devault in DCP, try passing --name desired-container-name via Container.Spec.Args; I believe that should do the trick https://github.com/dotnet/aspire/blob/main/src/Aspire.Hosting/Dcp/Model/Container.cs#L42

DCP side of this in Preview 2.

cc @JamesNK

I looked into this, and it is working today without any changes required in hosting. DCP now automatically sets a container name unless one is explicitly specified from hosting.

(Docker requires unique names. The random postfix on each name is a safety measure to prevent duplicates)
I'll update our DCP DTOs to include the new property, but we don't need to use it.

Are we adding support for setting the desired container name from an AppHost project?

I guess we need another issue?

#1855

(Docker requires unique names. The random postfix on each name is a safety measure to prevent duplicates)
I'll update our DCP DTOs to include the new property, but we don't need to use it.

The hostname changes with each restart of debugging for containers managed by the AppHost. This breaks RabbitMQ queue persistence (with fixed volume mounts configured as well) as queues are bound under a hostname directory. RabbitMQ's internal RabbitMQ naming can be changed by setting the NODENAME environment variable on the container which overrides this behavior - but this can be a source of frustration as no indication as to why queues disappear appear in any log.
The following works to override this behavior - but the  .WithEnvironment("NODENAME", "rabbit@localhost") statement should perhaps be a default when using the AddRabbitMQContainer method?
 var docGenRabbitMq = builder
     .AddRabbitMQContainer("rabbitmq-docgen", 9002, password: rabbitMqPassword)
     .WithEnvironment("NODENAME", "rabbit@localhost")
     .WithVolumeMount("rabbitmq-docgen-vol", "/var/lib/rabbitmq", VolumeMountType.Named);

@pedershk Could you create a new issue? That will ensure it will get looked at.

@pedershk Could you create a new issue? That will ensure it will get looked at.

Yes, sorry, didn't even notice that this one was closed. I blame lack of coffeee.
#2247


Hard dependency on yanked refinery version?

When setting up a new project with migrations through refinery, cargo resits adding the dependency because klickhouse depends on refinery 0.8.13, which has been yanked.
It looks like the issue that caused it to be yanked probably isn't anything to worry about, so I was able to add the dep with
`cargo +nightly  -Zunstable-options update --precise 0.8.13 refinery`

but would it be possible to update the dependency to 0.8.14?

Fixed, released 0.13.2


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit 132e9e6:
:clock10: Full build: the average full build time is 6 min(s) 5 sec(s), this is based on the last 478 build(s) for this repository.

Docs Build status updates of commit 132e9e6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Job not running when expected (every 10 minutes)

I'm attempting to run the job every 10 minutes.  I believe I have the cron set correctly for every 10 minutes (shown in picture).  But the last enqueued shows it doesn't execute every 10 minutes.
I'm running...

Rails 5.1.6
Sidekiq 5.1.3 (upgraded to 5.2.2 after posting - same results)
Sidekiq-Cron 1.0.0 (upgraded to 1.0.4 after posting - same results)

Hello,
So, the job doesn't execute every 10 minutes. It was enqueued once and you took a screenshot 11 minutes after that.
Anything showing up in the logs? Errors, enqueue attempts?
If it doesn't execute every 10 minutes, does it execute at all? Are you seeing no executions or sporadic executions, not every 10 minutes?
Thanks in advance for giving this issue tracker a clearer view on your problem.

Yes you have that right.  I've checked in the sidekiq.log and it shows when I manually kick it off through the UI with the Enqueue  button.

2018-10-24T21:28:03.450Z 397 TID-11a7m9 CreateOrUpdateDariExamJob JID-d10c6c5100b89534df431593 INFO: start
2018-10-24T21:28:07.419Z 397 TID-11a7m9 CreateOrUpdateDariExamJob JID-d10c6c5100b89534df431593 INFO: done: 3.969 sec

I also see this in the development.log
[ActiveJob] [CreateOrUpdateDariExamJob] [33c53033-9f2c-4a11-8546-6146df213d2b] Performed CreateOrUpdateDariExamJob (Job ID: 33c53033-9f2c-4a11-8546-6146df213d2b) from Sidekiq(import_queue) in 3967.74ms
However if I wait for 10 minutes it never shows up.  And thanks for your help I'll try/provide anything that will help. Thanks!

Here are some other files that I felt might be relevant...
sidekiq.yml
---
:concurrency: 25
:pidfile: ./tmp/pids/sidekiq.pid
:logfile: ./log/sidekiq.log
:queues:
  - default
  - [high_priority, 2]
  - [search_queue, 2]
  - [import_queue, 3]
  - [user_queue, 4]
  - [small_queue, 5]
  - [rss_parser, 6]
:daemon: true

initializers/sidekiq.rb
schedule_file = "config/schedule.yml"

if File.exist?(schedule_file) #&& Sidekiq.server?
  Sidekiq::Cron::Job.load_from_hash YAML.load_file(schedule_file)
end

Also the "last enqueued" column is now saying "in 3 hours".  Not sure how that factors into this if at all?

Have you tried with a trivial "hello world" job every minute (* * * * *) ?

I think it all has to do with me misunderstanding my deployment strategy, and how the sidekiq processes operate.  As I had setup sidekiq with capistrano a long time ago, I had forgotten some of the ins-and-outs.
I've removed the comment in my initializers/sidekiq.rb file.  So that the "&& Sidekiq.server?" is placed back.  And it has corrected to the time it's being asked to run at (~every 10 minutes).  I had commented it out originally due to this issue #224 because I couldn't add new jobs to my schedule.yml and have it reflected on the /cron web ui.  By commenting out the Sidekiq.server? condition it would add the job when I deployed.

I found a workaround. This doesn't trigger a job (every 15 minutes):
0/15 * * * *

However, adding a range makes it work (run every 15 minutes within 0 and 59 minutes):
0-59/15 * * * *

I also had the issue that @exAspArk had.
I found out that this has been fixed in the cron parser (fugit) in version 1.4.6.
However, this project seems to be locked on 1.1.*


ENV variable passing in bin/ci-build.sh could be shortened

.. to just the ENV keys

OTOH, shortened versions do not validate the presence of of the ENV variable - this way it's be left to Docker, and I'd guess Docker treats the values as optional..


[Release] Remove live demos from old versions

I've made this part of the other PR because it makes more sense there.


Fixing 3rd party guide link

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Thank you for fixing this. Good to see someone is reading our documentation ;)

You're welcome :)


Make typesafe entry point for main

The two valid type signatures for main are as follows:
main := λ(: argc U64)(: argv String[]);
main := λ;

Return type of main is Nil because the program should exit immediately if there is a hard error.


Allow same-slot attestations to affect the fork choice

Problem
The fork choice rule does not consider attestations from the same slot. Apart from the fact that some attacks may benefit from this, it's unclear why the fork choice rule should not consider same-slot attestations.
An attacker may strategically release attestations from previous slots in such a way that it could tip over honest attestors from the current slot to vote for a different head of the chain. There may be scenarios where this only works because the fork choice is disregarding attestations from the current slot.
Proposed solution
Allow attestations from the current slot to be considered by the fork choice.

Special thanks to @adiasg and @barnabemonnot for discussions/help.

This change looks correct to me. 👍
Side note for myself, not an issue:

In on_attestation docstring:

An attestation that is asserted as invalid may be valid at a later time, consider scheduling it for later processing in such case.

In the networking spec: https://github.com/ethereum/eth2.0-specs/blob/dev/specs/phase0/p2p-interface.md#beacon_attestation_subnet_id

[IGNORE] attestation.data.slot is within the last ATTESTATION_PROPAGATION_SLOT_RANGE slots
(within a MAXIMUM_GOSSIP_CLOCK_DISPARITY allowance) --
i.e. attestation.data.slot + ATTESTATION_PROPAGATION_SLOT_RANGE >= current_slot >= attestation.data.slot
(a client MAY queue future attestations for processing at the appropriate slot).

Due to (2), the future attestations (current_slot < attestation.data.slot) should have been ignored in the networking layer, so the client should not need to consider scheduling the future attestations.

Some (strong) counterarguments to the proposal of this PR:

An adversary who controls many nodes across the topology of the p2p layer may abuse same-slot attestations: Say the adversary hears about an honest block proposal very early on in the propagation process. The adversary could then flood the network with alternative blocks or attestations (using all the nodes it controls in the p2p network) from all the nodes it controls. Using sophisticated flooding strategies can make the adversary's attestations being "more powerful", i.e. being heard relatively more when they can still tip a close race between different blocks into a desired direction. While the adversary can still use the tactics described above with attestations from previous slots, enabling same-slot attestations gives the attacker more room to pull this off successfully. This is based on gossiping strategies described by @joachimneu in this ethresear.ch post.

It should be noted that this PR would affect the power of proposer score boosting (#2353): Allowing same-slot attestations implies that it is easier for an adversarial actor to "over-power" the score boost by using saved up attestations as well as attestations from the current slot (especially if the adversary uses sophisticated flooding strategies on the p2p layer).

I think that we want to move to only considering new attestations in batches at slot boundaries regardless of from this or previous slots. This combined with the proposer boost is a promising avenue to shore up the fork choice prior to a deeper redesign
Closing this for now


Not working on Asus GL504GM

Hi,
I just tried with compiling rogauracore and running it on a GL504GM.
If I run rogauracore single_static ffffff it returns Could not open ROG Aura Core keyboard: LIBUSB_ERROR_ACCESS.
So I run sudo rogauracore single_static ffffff. In this case no error is returned but the backlight color does not change.
This is my lsusb output:
Bus 002 Device 001: ID 1d6b:0003 Linux Foundation 3.0 root hub
Bus 001 Device 002: ID 13d3:56a2 IMC Networks
Bus 001 Device 004: ID 8087:0aaa Intel Corp.
Bus 001 Device 003: ID 0b05:1866 ASUSTek Computer, Inc.
Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub

May I provide more information on my system that could be helpful for you?
Thank you!

Hi @nico87
Thanks for the ticket!  As you have rightly deduced, you need to have superuser privileges to modify the keyboard RGB state.
Support for the 0x1866 keyboard in your machine was added by PR #2 and works on the GM501, so I'm a little puzzled why it doesn't work on the GL504.
One thing we could try is looking at the verbose output of rogauracore.  If possible, try running this:
sudo rogauracore -v single_static ffffff

Cheers, Will

Thanks. This is the output. Seems there are no errors, bu unfortunately the keyboard backlight is still in rainbow mode.
parse color ffffff
interpreted color 255 255 255
args:
color1 255 255 255
color2 240 54 127
color3 0 0 0
color4 0 0 0
speed 0
single_static
constructed 1 messages:
message 0: 5d b3 00 00 ff ff ff 00 00 00 00 00 00 00 00 00 00
Initialising libusb
Initialised libusb.
Found 5 USB devices.
Checking device 1d6b:0003, address 1
Checking device 13d3:56a2, address 2
Checking device 8087:0aaa, address 4
Checking device 0b05:1866, address 3
Found ROG Aura Core keyboard.
Opened USB device.
Auto detach kernel mode set.
Claimed interface 0.

Yeah, that all looks normal.  Maybe the keyboard is somehow different even though it has a product ID we know about,  Can you try:
sudo lsusb -v -d 0b05:

Another thought: Asus Aura is the name used for a bunch of different control systems.  On Asus motherboards and graphics cards, for instance, it controls RGB lighting in the case (and external LED strips); that hardware isn't connected on USB, so Aura control is via SMBus.  I know Asus laptops also have an internal SMBus, but I've not heard of it being used to control the keyboard lighting.  However, maybe that's what's going on with your machine.  Do you have any RGB lighting that's not on the keyboard?  Multicolour LEDs on the case or the lid perhaps?

Yes. It has a front led and a led logo on the monitor cover.
Bus 001 Device 003: ID 0b05:1866 ASUSTek Computer, Inc.
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               2.00
  bDeviceClass            0
  bDeviceSubClass         0
  bDeviceProtocol         0
  bMaxPacketSize0        64
  idVendor           0x0b05 ASUSTek Computer, Inc.
  idProduct          0x1866
  bcdDevice            0.02
  iManufacturer           1 ASUSTeK Computer Inc.
  iProduct                2 N-KEY Device
  iSerial                 0
  bNumConfigurations      1
  Configuration Descriptor:
    bLength                 9
    bDescriptorType         2
    wTotalLength       0x005b
    bNumInterfaces          3
    bConfigurationValue     1
    iConfiguration          0
    bmAttributes         0xe0
      Self Powered
      Remote Wakeup
    MaxPower              100mA
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        0
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      1 Boot Interface Subclass
      bInterfaceProtocol      1 Keyboard
      iInterface              3 (error)
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.10
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      67
         Report Descriptors:
           ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x81  EP 1 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        1
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      1 Boot Interface Subclass
      bInterfaceProtocol      1 Keyboard
      iInterface              1 ASUSTeK Computer Inc.
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.10
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength      65
         Report Descriptors:
           ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x82  EP 2 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        2
      bAlternateSetting       0
      bNumEndpoints           2
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      1 Boot Interface Subclass
      bInterfaceProtocol      1 Keyboard
      iInterface              1 ASUSTeK Computer Inc.
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.10
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength     167
         Report Descriptors:
           ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x83  EP 3 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x04  EP 4 OUT
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               1
can't get device qualifier: Resource temporarily unavailable
can't get debug descriptor: Resource temporarily unavailable
Device Status:     0x0001
  Self Powered

So here's my keyboard:
Bus 001 Device 005: ID 0b05:1869 ASUSTek Computer, Inc.
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               2.00
  bDeviceClass            0 (Defined at Interface level)
  bDeviceSubClass         0
  bDeviceProtocol         0
  bMaxPacketSize0        64
  idVendor           0x0b05 ASUSTek Computer, Inc.
  idProduct          0x1869
  bcdDevice            0.03
  iManufacturer           1 ITE Tech. Inc.
  iProduct                2 ITE Device(8910)
  iSerial                 0
  bNumConfigurations      1
  Configuration Descriptor:
    bLength                 9
    bDescriptorType         2
    wTotalLength           34
    bNumInterfaces          1
    bConfigurationValue     1
    iConfiguration          0
    bmAttributes         0xe0
      Self Powered
      Remote Wakeup
    MaxPower              100mA
    Interface Descriptor:
      bLength                 9
      bDescriptorType         4
      bInterfaceNumber        0
      bAlternateSetting       0
      bNumEndpoints           1
      bInterfaceClass         3 Human Interface Device
      bInterfaceSubClass      1 Boot Interface Subclass
      bInterfaceProtocol      1 Keyboard
      iInterface              0
        HID Device Descriptor:
          bLength                 9
          bDescriptorType        33
          bcdHID               1.10
          bCountryCode            0 Not supported
          bNumDescriptors         1
          bDescriptorType        34 Report
          wDescriptorLength     263
         Report Descriptors:
           ** UNAVAILABLE **
      Endpoint Descriptor:
        bLength                 7
        bDescriptorType         5
        bEndpointAddress     0x81  EP 1 IN
        bmAttributes            3
          Transfer Type            Interrupt
          Synch Type               None
          Usage Type               Data
        wMaxPacketSize     0x0040  1x 64 bytes
        bInterval               4
Device Status:     0x0001
  Self Powered

There are some differences to your output: I've only got one interface, whereas you have three.  This could be a good start.   The endpoint under my interface with bInterfaceNumber 0 looks very much like yours, but we already know rogauracore is talking to your bInterfaceNumber 0, and it's not doing the trick.  The second interface on your keyboard (with bInterfaceNumber 1) also looks very similar; I wonder if we should try talking to that endpoint?  Your third interface looks different, it has two endpoints, so that seems less promising.  I'm intrigued by the iInterface 3 (error) on your first interface, though I guess that's probably not all that important.  Would be curious to know if this value changes, e.g., after a system reboot, but probably it's always 3.
If you think the front LED and lid LED can change colours, that might also support for the idea that your machine uses a different bus to manage RGB.  Here is an issue I filed against OpenAuraSDK while I was thinking that my laptop might support the full Asus Aura spec (like allowing the keyboard flash in time to music and so on).  Sniffing in this direction, you'd want to find the SMBus and modprobe the driver (probably Intel hardware, so it'd be sudo modprobe i2c-i801), and then do i2cdetect to see if it looks like there's an RGB controller on the bus.
At this point, I would bet that the most promising avenue would be to try the USB approach again, but attaching to the second interface instead of the first.  I'll try to push a few changes to the Python prototype of rogauracore tonight that you could try out.

If you think the front LED and lid LED can change colours, that might also support for the idea that your machine uses a different bus to manage RGB.

Yes! Definitely it's like that. The lid and front LEDs are just like the keyboard.
The following is the i2cdetect -l output:
i2c-3	i2c       	i915 gmbus dpb                  	I2C adapter
i2c-1	i2c       	Synopsys DesignWare I2C adapter 	I2C adapter
i2c-6	i2c       	i915 gmbus dpd                  	I2C adapter
i2c-4	i2c       	i915 gmbus dpc                  	I2C adapter
i2c-2	i2c       	Synopsys DesignWare I2C adapter 	I2C adapter
i2c-0	smbus     	SMBus I801 adapter at efa0      	SMBus adapter
i2c-7	i2c       	DPDDC-A                         	I2C adapter
i2c-5	i2c       	i915 gmbus misc                 	I2C adapter

Thanks for your help!

OK, so the 0xefa0 address for the i801 doesn't seem to match up with OpenAuraSDK.  Nonetheless, you could try:
sudo i2cdetect 0

Maybe the i2c device looks more familiar.
In the mean time, let's try the USB again, but talking to interface 1 instead of interface 0.  There's a script you can download at https://raw.githubusercontent.com/wroberts/rogauracore/nico87/prototype/rogauracore.py .  You'll need Python3 and python3-usb (sudo apt install python3 python3-usb), and then you'd do something like sudo python3 rogauracore.py single_static ff0000.

The i2cdetect output is:
     0  1  2  3  4  5  6  7  8  9  a  b  c  d  e  f
00:          -- -- -- -- -- 08 -- -- -- -- -- -- --
10: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
20: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
30: 30 -- -- -- -- 35 36 -- -- -- -- -- -- -- -- --
40: -- -- -- -- 44 -- -- -- -- -- -- -- -- -- -- --
50: 50 -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
60: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
70: -- -- -- -- -- -- -- --

Running the python script returns an exception:
Traceback (most recent call last):
  File "rogauracore.py", line 262, in <module>
    main()
  File "rogauracore.py", line 256, in main
    raise e
  File "rogauracore.py", line 253, in main
    dev.ctrl_transfer(bmRequestType, bRequest, wValue, wIndex, message)
  File "/usr/lib/python3/dist-packages/usb/core.py", line 1034, in ctrl_transfer
    self._ctx.managed_claim_interface(self, interface_number)
  File "/usr/lib/python3/dist-packages/usb/core.py", line 102, in wrapper
    return f(self, *args, **kwargs)
  File "/usr/lib/python3/dist-packages/usb/core.py", line 167, in managed_claim_interface
    self.backend.claim_interface(self.handle, i)
  File "/usr/lib/python3/dist-packages/usb/backend/libusb1.py", line 811, in claim_interface
    _check(self.lib.libusb_claim_interface(dev_handle.handle, intf))
  File "/usr/lib/python3/dist-packages/usb/backend/libusb1.py", line 595, in _check
    raise USBError(_strerror(ret), ret, _libusb_errno[ret])
usb.core.USBError: [Errno 16] Resource busy

i2c output looks just like mine, so I guess SMBus isn't the answer here.  Sorry about the USB error, I don't quite understand that; I'll need to go and figure out why we can't connect to interface 1.  Let's try interface 0 again, but using the python code.  Updated script at https://github.com/wroberts/rogauracore/blob/nico87/prototype/rogauracore.py

Thanks for your help.
The new script doesn't show any error but the keyboard backlight nor the other LEDs didn't change. :-/
I'm a dev myself, so if there's something I can try by editing the code you linked, let me know. It could be faster.
Thanks!

OK, so the Python version of rogauracore doesn't work either; that's not too surprising, since the C is just translated from the Python.  Too bad, I was hoping we would get lucky.
To sum up our story so far: On the GL504GM, and, as of this morning, on the GU501G (#9), rogauracore runs without error messages, but the keyboard colours don't change.  These machines both use the 0b05:1866 USB keyboard, which is also included on the GM501.  I have a report from @thorhop that rogauracore works on that laptop (#2), and that all colour modes are functional.
At this point, I don't have a solid hunch for why things are working on one laptop model but not on the two others.  Asus's website says the GL504GM supports Aura Sync, which my GL503 does not support, so there might be a different control system or a different protocol being used.  The GL504GM also has a programmable RGB light bar along the front of the laptop chassis, presumably not controlled by the keyboard.  I can't find any current information on the GU501.
I've muddied the waters a little bit by talking about i2c and SMBus; this thinking was in relation to the OpenAuraSDK project, which aims to support Asus's Aura and Aura Sync products under Linux, but which at the moment mostly seems to be focussed on RGB control of RAM modules on Asus motherboards.  The hardware targeted by OpenAuraSDK is a dedicated Aura control chip on the motherboard, which is addressable over SMBus.  As far as I can see, this SMBus seems to live at address 0x0b00 or 0x0b20 on AMD hardware, and possibly at 0xf000 on Intel.  If you query the bus, you'll see a list of devices on the bus that are present on the machine.  The Aura chip might live at 0x4e.  You can see RAM sticks under 0x5 (0x50 is the first RAM stick, 0x51 the second, etc.).  Where there is colour control of the RAM, there will be corresponding Aura controllers on 0x70, 0x71, etc.  There is also the OpenPyAura project that is working on similar themes, might be interesting to check out.  These projects are quite young, but are more sophisticated in terms of reverse engineering the Aura protocol, documenting different registers and settings, and supporting a much wider range of hardware than rogauracore.
To be clear, rogauracore only communicates on USB, and does not do anything with i2c or SMBus.  In this respect, it's similar to other projects to control RGB keyboards such as the Razer Chroma and Corsair K70.  KeyboardVisualizer is, I think, the project where I stumbled onto some of this material.
All of this is just background to our central question, which we still don't have an answer to: how is it that the 0b05:1866 USB keyboard can be controlled on the GM501 but not on the GU501 or GL504?  Do these two machines use USB for RGB control?  If so, do they use a different interface or a different protocol than the GM501?  If not, do they use an Aura control chip like Asus desktop motherboards do, and is this controlled over the SMBus?
I suggest we engage in some sanity testing here.  Questions:

Do any of the colour commands in rogauracore work?  We have established that single_color doesn't work; how about the others?

Do you have a Windows dual-boot?  If so, can you install Lighting Services and Aura Core (or Aura Sync?) on the Windows, and does it control the RGB there?  If yes, can you list the different colour modes and parameters that the software allows you to control?  Are they the same as the set that rogauracore supports?

Thanks for the clarification. I'll give OpenPyAura a shot later.

Do any of the colour commands in rogauracore work? We have established that single_color doesn't work; how about the others?

None of them work

Do you have a Windows dual-boot? If so, can you install Lighting Services and Aura Core (or Aura Sync?) on the Windows, and does it control the RGB there? If yes, can you list the different colour modes and parameters that the software allows you to control? Are they the same as the set that rogauracore supports?

Unfortunately, I don't have a dual boot. I don't have the possibility to mount another nvme ssd either and try it :-(
I need Linux for my daily work so I cannot erase it and start from scratch.

I'm going to take a look at manipulating it from within VirtualBox to see if I can get any info.

I wanted to follow-up. I'm not having any luck getting it to work via virtualbox. I'm not sure how to get it to actually see the USB keyboard -- I'm assuming user error for that. I'm not sure how to proceed, but I"ll keep an eye on this thread.

To be able to connect USB devices, make sure you installed the vbox
additions and that your user is in the vboxusers group. Then, logout and
login again to make sure changes are effective.
Il sab 11 mag 2019, 15:22 Stuart Templeton<EMAIL_ADDRESS>ha
scritto:

I wanted to follow-up. I'm not having any luck getting it to work via
virtualbox. I'm not sure how to get it to actually see the USB keyboard --
I'm assuming user error for that. I'm not sure how to proceed, but I"ll
keep an eye on this thread.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wroberts/rogauracore/issues/8#issuecomment-491537516,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKMVZRL5EWN43UR5Y6K4BTPU4MIBANCNFSM4HK6QNAQ
.

Something strange happened to me, and I thought maybe sharing it would help.
This is on the Strix GL703GM.
OS is Arch Linux.
I had the same problem as every other currently open issue on this topic. Then I loaded Vbox and edited my lights from within there.
Sidenote for Arch (maybe other?) users: I had to install virtuabox-ext-oracle (and reboot after) and also on the Windows image in Virtualbox go to Preferences -> USB -> checkbox on USB 2.0, before it worked.
Once I changed the lights a bit from the box, I downloaded Wireshark to start snooping the USB traffic, but just randomly also ran the rogauracore command again on my host machine. Suddenly, the lights changed colour...
I have no idea why, but it works now... so yay?
The output from the command is the exact same as it was before, when it didn't work.
sudo ./rogauracore -v single_static ffffff

parse color ffffff
interpreted color 255 255 255
args:
color1 255 255 255
color2 26 174 127
color3 0 0 0
color4 0 0 0
speed 0
single_static
constructed 1 messages:
message 0: 5d b3 00 00 ff ff ff 00 00 00 00 00 00 00 00 00 00
Initialising libusb
Initialised libusb.
Found 6 USB devices.
Checking device 1d6b:0003, address 1
Checking device 13d3:56a2, address 3
Checking device 248a:8566, address 2
Checking device 8087:0aaa, address 5
Checking device 0b05:1866, address 4
Found ROG Aura Core keyboard.
Opened USB device.
Auto detach kernel mode set.
Claimed interface 0

I hope this helps with the detective work.

I had a similar experience to @VesterDe except I couldn't get the AuraSync software installed in my VM - simply having booted the Windows VM with the device passed through seemed to enable something and allow rogauracore to work.

I'm noticing a similar behavior.  It's as though it has to be switched into "accept lighting commands" mode. When rebooting, the rainbow comes on and stays on, with animation, right until GRUB loads up. Then it's dead. There must be some message that turns the system on. How did you uncover the formats you've already documented, Will?

Never mind. It was just the brightness. I added brightness controls to the application and was able to use it exactly as you had it.

Hi can i used it for Arch ( Manjaro KDE ) ?

@wroberts why was this issue closed?
It was not fixed as far as I understood from the messages and I myself have a GL504GM with keyboard 0b05:1866 and running any command results in a correct execution but no light changes on the keyboard.
sudo rogauracore -v black
args:
color1 48 7 12
color2 160 60 127
color3 0 0 192
color4 10 12 160
single_static
constructed 1 messages:
message 0: 5d b3 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
Initialising libusb
Initialised libusb.
Found 6 USB devices.
Checking device 1d6b:0003, address 1
Checking device 13d3:56a2, address 3
Checking device 045e:076c, address 2
Checking device 8087:0aaa, address 5
Checking device 0b05:1866, address 4
Found ROG Aura Core keyboard.
Opened USB device.
Auto detach kernel mode set.
Got configuration descriptor.
Found 3 interfaces on the USB device.
Claimed interface 0.
Successfully sent all messages.
Sent SET message.
Sent APPLY message.

Hi,
I had the same issue on UBUNTU 20.04 with a GM501
I did chmod +s /usr/local/bin/rogauracore
Afterwards I did
rogauracore initialize_keyboard
followed by
rogauracore single_static ff0000
All works!

Hi,
I had the same issue on UBUNTU 20.04 with a GM501
I did chmod +s /usr/local/bin/rogauracore
Afterwards I did
rogauracore initialize_keyboard
followed by
rogauracore single_static ff0000
All works!

Can confirm. Works on my Zephyrus M GM501GM as well.


fix wallet total balance component

Task linked: CU-3af9nfw While loading the icon is shown as unknown


Enable clap help message text wrapping

As discussed in #202, re-enabling the text wrapping feature for clap help messages. This was the default in clap v3, but is now an opt-in feature in clap v4.

Thanks again!

Thank you for adding a regression test :heart:


IddCxSwapChainReleaseAndAcquireBuffer can take up to 5000ms to return.

Within the video/IndirectDisplay IddCxSampleDriver, the 'RunCore" while loop in the sample has stuttering. IddCxSwapChainReleaseAndAcquireBuffer() normally takes around 6ms to return. Every second or two, it can take up to 5000ms to return, this results in some consistent and terrible stutters, that is completely unreasonable.
This isnt an issue with my own code (which obviously still has this issue), but an issue with IndirectSampleDriver.
Things I have tried. . .
-Windows 11 latest, Windows 11 Insider, Windows 11 Dev
-Windows SDK Versions: 19041, 20348, 22000, 22572 (on insider and dev)
-IddCx Versions: 1.4, 1.6, 1.8 (on insider and dev), 1.9 (on dev)
-UMDF Version 2.25, 2.33
-Changing AvSetMmThreadCharacteristicsW to "DisplayPostProcessing"
-Setting AvSetMmThreadPriority to AVRT_PRIORITY_CRITICAL
-Setting DXGI SetGPUThreadPriority to 7
-Disabling WPP Tracing
-Building x64Release and x64Debug
-Updating Nvidia Drivers
-Increasing the DISPLAYCONFIG_VIDEO_SIGNAL_INFO vSyncFreq numerator
-Increasing the DISPLAYCONFIG_VIDEO_SIGNAL_INFO hSyncFreq numerator
-Increasing the DISPLAYCONFIG_VIDEO_SIGNAL_INFO pixelRate
-Using different EDID data
-Changing the Version within IDDCX_ENDPOINT_DIAGNOSTIC_INFO for IddCxAdapterInitAsync
Things I havent tried yet . . .
-Somehow telling the sample driver to use WDDM 3.0. I have no idea how to get this to work. According to dxdiag, the virtual display spun up by the sample is using WDDM 1.3 (while my main display uses 3.0). Maybe this is causing an issue? I really dont know.
-Going back to Windows 10, I would do this, but I need Windows 11.
My most recent test showed 7 frames of 2,000 that went above 16ms (for a 60fps monitor) for the return time for IddCxSwapChainReleaseAndAcquireBuffer(). The times for them where 2033.26, 1932.26, 897.88, 348.074, 299.568, 41.8258 and 20.3827. The last 2 frames arent that big of a deal only missing a single frame or two. I also believe the frame that took 897.88ms is due to my shutting down the system. The second to last frame before shutting down the Software Device always seems to take forever.
I am using a Ryzen 5800X and a RTX 3090.

Discovered this is due to windows simply not updating the desktop


New log collector module

What is the purpose of the change
Embedded log collector plugin for Flink and Spark Streaming application
Brief change log

Define a new module for collecting log of Streamis
Enhance the restful api of job to receive heartbeat request from Spark Streaming.
Define a log server to listen and store the log event.

Verifying this change
This change added tests and can be verified as follows:
run the test unit StreamisLogAppenderTest.java
Does this pull request potentially affect one of the following parts:

Dependencies: no
Anything that affects deployment: no
The Core framework, i.e., JobManager, Server.: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? not documented

LGTM！


feat: Added Text component

:tada: This PR is included in version 1.0.0-alpha.25 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Ensure BinaryData responses are durable in polling operations

Description
Fixes #36564
This fixes a bug where BinaryData.toReplayableContent could be called but the underlying content isn't made replayable until the BinaryData returned was accessed for data. This only affected BinaryDatas using FluxByteBufferContent as the backing content source, this issue was that the synchronous replayable content API was coded that the underlying Flux<ByteBuffer> wasn't copied until the BinaryData's accessor APIs we called to get content from the BinaryData. Now, the synchronous API eagerly makes the underlying Flux<ByteBuffer> replayable.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Consider adding a useful ToString()

For generated models, it would be useful to have a ToString() that prints out all the fields in a deterministic order. For example:
public partial class Value
{
	[JsonProperty("A")]
	public string A { get; set; }

	[JsonProperty("B")]
	public string B { get; set; }

	[JsonProperty("C")]
	public string C { get; set; }

	override public string ToString()
	{
		return string.Format("[{0} A={1}, B={2}, C={3}]",
			GetType(),
			A,
			B,
			C);
	}
}

It may be reasonable to implementEquals, Hashcode, and ToString... I'd certainly be fine to start with:
In the short term, I'd be fine with:
	override public string ToString() => ToJson(this);

Hopefully without the arrow syntax but sure! ;)


Fix and update ICU calculation and methodology text.

Following @HenkPoley's comment I've fixed the logical mistake with the ICU calculation (fractions are hard), updated the constant used, and also updated the method of calculation (to be a ratio of the reported active cases and not a ratio of the estimated cases).
The table looks considerably redder and worrisome now :(

I notice the model is still seems fairly far off for anything about The Netherlands.
We discover about a 1000 cases per day. Over 5 days that would be 5000. This model expects 58546. ~11x more.
From China's 45K cases report 13.8% severe + 4.7% critical = 18.5%. Let's assume we only find those in The Netherlands, everybody straight to hospital. That means there were 5k/0.185 ~= 27k total infected ~10 days ago.
So even with a strong under counting assumption, the model here predicts ~2x more.
We currently have ~150 deaths a day. 5 days * 150 people / 0.045 ~= 17k total infected ?? days ago.
So with that assumption, the model here predicts ~3.5x more active cases.
Currently the counts in The Netherlands seem to be plateauing / in a steady state. Maybe the model is forced to assume a stronger growth?
btw, potential ICU capacity has been scaled up to 2400 beds. Since we went past the 1150 beds on the 31st of March. And if you keep in mind the 575 reserved for 'normal' patients, we went over the remaining beds on the 23rd, and the COVID-19 patients section has a capacity of 2400-575 ~= 1800-1900.
E.g. ~10.9 per 100k population.

Also: If actually 1.5% of the country was infected we would have in the order of 800k deaths by now. We would know.

FYI on a related note, I updated all dashboards that contain a predictive model with a more stern warning

😅

@HenkPoley thanks for that analysis.

We discover about a 1000 cases per day. Over 5 days that would be 5000. This model expects 58546. ~11x more.
So even with a strong under counting assumption, the model here predicts ~2x more.
We currently have ~150 deaths a day. 5 days * 150 people / 0.045 ~= 17k total infected ?? days ago.
So with that assumption, the model here predicts ~3.5x more active cases.

This is due to estimating the "testing bias" from deaths. The reported fatality rate is very high (unfortunately) at 10.7%, and the estimation of actual cases is trying to calculate the testing bias from deaths relative to lagged cases. In this case the fatality rate for current deaths over lagged cases (8 days ago): is 2256 / 12668 = 11.8%. It is than compared to the fatality rate that was found in countries where testing was comprehensive - 1.5%. And this ratio is 11.8 / 1.5 gives an estimate of testing ratio of 7.8. So this number is used to adjust the reported case numbers. While the exact numbers in this estimation might not be totallly correct, it seems reasonable that there is significant under counting if the fatality rate (relative to reported cases) is so high.
These "total cases" are including the asymptomatic cases as well.

Currently the counts in The Netherlands seem to be plateauing / in a steady state. R0 is estimated to wobble near 1 now (probably below). Maybe the model is forced to assume a stronger growth?

The growth rate is calculated from the last 5 days data, so if that data is reliable and consistent, this should reflect in the calculation. The growth rate I see in the table is 5.9%, which is consistent with nearly a plateau (since 5% is about the size of just the recovery rate), so active cases should be rising very slowly at this point (6% added daily, and 5% removed daily). Perhaps in a few days the projections will level out completely, when growth rate estimation catches up.

epiforecasts 90% confidence max. is about 8300 new cases in 5 days: https://epiforecasts.io/covid/posts/national/netherlands/

The link was 404, but this https://epiforecasts.io/covid/ worked so I'll definitely have a look there.

btw, potential ICU capacity has been scaled up to 2400 beds..

That's great. I imagine most countries' capacities are increasing. Not sure how to factor that into the capacity estimations.. and it's probably going to be more important as time progresses. Any ideas for data sources? Maybe just multiply by some constant factor, any ideas for data to estimate that "scaling" factor from?

Also: If actually 1.5% of the country was infected we would have in the order of 12k deaths by now. We would know.

With the fatality rate assumed in this model (for asymptomatic cases included in totals) I get 17000000 x 0.015 (total assumed infected) x 0.015 (fatality rate) = 3825, which is not the current number of expected deaths, but the number expected in 8 days. With the current number of deaths being 2256, and the new deaths during last 5 days being 765, this might not be too far off unfortunately. Does this make sense? Or am I missing something about your calculations?
Thanks again for fact checking this vs reality!

0.015 (fatality rate)

Ah, your fatality rate estimate is rather low. I'm wonder though why/how would be missing about 11 out of 12 (91.3%) infected cases.
Maybe you are not taking into account that The Netherlands is on average older than China? I believe the scaling factor for the CFR for Greece could even be ~4x, just looking at the population pyramid.

btw, potential ICU capacity has been scaled up to 2400 beds..

Maybe just multiply by some constant factor, any ideas for data to estimate that "scaling" factor from?

There will be no further scaling of ICU, it has already gone from 2 beds per medic to 4 beds per 1 medic. Unless of course over many many quarters (e.g. full but shortened education cycle for new intensivists).

epiforecasts 90% confidence max. is about 8300 new cases in 5 days: https://epiforecasts.io/covid/posts/national/netherlands/

The link was 404, but this https://epiforecasts.io/covid/ worked so I'll definitely have a look there.

Epiforecasts somehow dropped The Netherlands overnight 🤷♂.

Re: scaling of CFR with population pyramid: https://twitter.com/CJEMetcalf/status/1229792572513669121

@HenkPoley

Ah, your fatality rate estimate is rather low. I'm wonder though why/how would be missing about 11 out of 12 (91.3%) infected cases.

Re: scaling of CFR with population pyramid: https://twitter.com/CJEMetcalf/status/1229792572513669121

Adjusting the expected CFR by country would be a great improvement, I agree.
One point about the 1.5% CFR that I've used, is that it's not based on China's CFR, but instead on Cruise Ship, South Korea and Singapore at the time. Which are not that young, so I don't think it would be a factor of 3x / 4x but hopefully more around 2x (from the population-pyramid data). I think the main difference for these countries (bringing the CFR so low) was the fact that asymptomatic cases are accounted in it as well (more so than in countries that are testing only the suspected cases). To me, this is a more important measurement since for estimating future growth asymptomatic cases are very important.

Just oggling CFR's here it seems to corroborate that: https://avatorl.org/covid-19/?page=CaseFatalityRate

That's a great resource, thanks! How do you find these?

@HenkPoley
Another point about CFR and testing bias relationship to age is perhaps to consider Japan and Germany, which have old populations, but in which the CFR is around 2%.

it's not based on China's CFR, but instead on Cruise Ship, South Korea and Singapore at the time.

Do you have a reason to believe these people were all tracked properly, and not transferred to their respective home countries and thus 'lost' in the statistics ?
South Korea's naive CFR is not 1.5% (nor Resolved CFR). And it is still rising.
Singapore is still low, yes. But don't they have the majority of infections under travellers and foreign workers (both a healthy group) ?

That's a great resource, thanks! How do you find these?

Loosely based on how Christopher Hitchens was reading newspapers. He would grab a paper and skip reading everything but the name of the reporters, and if he knew it was a good reporter he would read the article. With social media you don't need to do the first step, you just only/mostly follow the people who seem to know things, and seem to be able to filter out nonsense.
Maybe go down my most recent fresh followers 🤷♂: https://twitter.com/henkpoley/following
I've studied Bio-informatics years ago, so I have some feeling with the field.
A resource that was good very early on, but not specifically for your statistics: http://virological.org/c/novel-2019-coronavirus/ncov-2019-genomic-epidemiology/36
People who gather (and link to) there are people who seem to know things.
At the PowerBI site (avatorl), the linked resource BNOnews is also a person who seems to know things.
There are also people who only seem to know things, but who do not have proper filters. For COVID-19 specifically, Eric Feigl-Ding comes to mind. But people who cluster around Nassim Nicholas Taleb are also strong on this factor. Very right- and very left-wing politics is also smell for this.
Basically: I spend too much on Twitter 😋

Basically: I spend too much on Twitter yum

@HenkPoley Thanks for those resources, I definitely do not spend enough time on twitter, looks like a great source of info.. I did however find this conventionally, although problematically, named list.
About the true CFR. I find the age theory compelling, but difficult to reconcile with some counter examples:

The cruise ship population was fairly old, and here they estimate the CFR (accounting for the lag) to be 2.3% (which is higher than what I'm using, and it might be worth adjusting it already to that figure).
In addition to the cruise ship data, it is hard to reconcile some additional counter examples of how the CFR can be so low in some countries. I'm calculating the lagged fatality rate with a lag of 8 days (that is also likely on top of the implicit testing lag of around 5 days and also the implicit likely lag of symptoms onset of another several days - so should be of 2-3 weeks). Indeed for South Korea it is 2.1%, Germany is 3.2% and Japan is at 4% but some countries are below 1%, and there are even more below 2%, and Australia, Israel, New Zealand are not particularly young countries, neither are they likely to misreport deaths:

                      Cases.total  Cases.new  Deaths.total  lagged_fatality_rate
Country/Region
New Zealand                  1283        244             2              0.003759
Qatar                        2512        908             6              0.007368
Singapore                    2108        799             7              0.007619
Australia                    6215        528            54              0.010748
Israel                      10408       1978            95              0.013998
United Arab Emirates         3360       1561            16              0.016585
South Africa                 2003        348            24              0.017088
Thailand                     2473        304            33              0.018124
Chile                        6501       2030            65              0.019383
South Korea                 10450        213           208              0.020948
Croatia                      1495        313            21              0.021739
Norway                       6314        627           113              0.022145
Malaysia                     4346        684            70              0.022778
Saudi Arabia                 3651       1249            47              0.025451
Russia                      11917       6528            94              0.026768
Pakistan                     4695       1538            66              0.027663
Austria                     13555       1504           319              0.028751
Estonia                      1258        161            24              0.029104
Czechia                      5732       1145           119              0.031096
Finland                      2769        842            48              0.032258
Germany                    122171      22048          2767              0.032643
Japan                        5530       2391            99              0.040064
China                       82941        339          3340              0.040530
Portugal                    15472       4194           435              0.048257
Canada                      22059       6303           557              0.049446
Panama                       2752        951            66              0.050835
Slovenia                     1160        163            45              0.051225
Switzerland                 24551       3451          1002              0.053272
Turkey                      47029      19960          1006              0.055525
Moldova                      1438        574            29              0.059289

I think there is good reason to adjust the CFR by age, but it seems from some counter examples, that adjusting it to 4% for the "old" countries might be a big overestimation still. I just wouldn't expect to see developed countries with 1% lagged CFR if that would be the case.
In other news, I've came across some real data about current ICU occupancy for Germany and for Israel and they both seem pretty close to what the calculations estimate (within 10%-20% percent), which might be a lucky coincidence with all the crude assumptions, but still might be a sign that for other countries it might be of the right order magnitude. (The ICU calculations don't involve that "true CFR" btw, so unrelated to the main point we discussed, but are related to what started the dicussion)

Japan's Resolved CFR is more like 12-13%, btw

Recent serological tests of blood donors in The Netherlands showed 3% were infected. So maybe.. you are actually lowballing the figures 😉 🤷♂
You can read it in English in this excellent scientific journal ;)
In dutch, from the horses mouth (Sanquin): https://www.sanquin.nl/artikelen/nieuwsberichten/2020/04/sanquin-ongeveer-3-van-donors-heeft-corona-antistoffen

@HenkPoley Thanks for that.

On one hand, It's a bit hard to know what the biases in that study are (are these random blood donors, or self-selected) without the paper. Also publication bias may have an effect here (especially if it made to popular-press).
On the other hand, I have seen that mentioned in other places that in a lot of Europe's heavily hit countries there is under-attributing of deaths to COVID as patients who do not make it to the hospital (e.g. die in the nursing home), aren't eventually tested and so are not counted. So it's possible that it's 3%, but that may mean that there could be 2-3 times more deaths than actually reported. It is possible, but I'm not sure how to try and account for that.

There is a graph of "over-mortality" in The Netherlands: https://www.rivm.nl/monitoring-sterftecijfers-nederland
Yes, I'm also concerned that the test would pick up 10% (of the ones who tested positive now) in older blood samples.
Viewed from the outside it might as well be that nobody had anti-bodies, and they essentially went on a "p-hacking fishing expedition" that showed people sometimes have anti-bodies for the common cold coronavirus.

Yesterday around 16:00 UTC+2 Ernst Kuipers (an ICU official, of 'Landelijk Netwerk Acute Zorg') told that current models (I guess by RIVM.nl) tell that The Netherlands is at 4% total affected population. Which is roughly in line with your model.

I've also recently updated the lower bound CFR to be 0.72%, which bumped up a lot of European countries to quite high percentages.
I wonder how much of a big factor excess deaths  could be hiding, might even be 2x or so..

In The Netherlands the excess mortality was (is?) 2x of the known COVID-19 mortality, yes.

FYI, The Netherlands just published their own dashboard: https://coronadashboard.rijksoverheid.nl/


WM_CLASS not set

Hello! Thanks you very much for taking this project! You're doing an amazing job updating this app.
This issue is more a mix between a problem and an enhancement.
If you're using  tiling manager like i3 and/or bspwm (on linux), you can set rules to automatically assign application on specific desktops.
This require the wm_class info to be defined inside the app.
This is a common requirement, most of the app (all of them?) have this data set on linux.
The rules to assign desktops are:
bspc rule -a Vlc desktop=^2
bspc rule -a messengerfordesktop desktop=^6

The problem is that the Messenger app doesn't have the requirements, so the rule can't work.
When using the xprop command, i get the following infos
_NET_WM_USER_TIME(CARDINAL) = 74813799
_NET_WM_DESKTOP(CARDINAL) = 5
WM_NORMAL_HINTS(WM_SIZE_HINTS):
   	program specified location: 20, 62
WM_NAME(UTF8_STRING) = "Messenger"
_NET_WM_NAME(UTF8_STRING) = "Messenger"
_NET_WM_ICON(CARDINAL) =
XdndAware(ATOM) = BITMAP
_MOTIF_WM_HINTS(_MOTIF_WM_HINTS) = 0x2, 0x0, 0x1, 0x0, 0x0
_NET_WM_WINDOW_TYPE(ATOM) = _NET_WM_WINDOW_TYPE_NORMAL
_NET_WM_PID(CARDINAL) = 2539
WM_LOCALE_NAME(STRING) = "en_US.UTF-8"
WM_CLIENT_MACHINE(STRING) = "hostname"
WM_PROTOCOLS(ATOM): protocols  WM_DELETE_WINDOW, _NET_WM_PING

And for Vlc I get the following data:
XdndAware(ATOM) = BITMAP
_MOTIF_DRAG_RECEIVER_INFO(_MOTIF_DRAG_RECEIVER_INFO) = 0x6c, 0x0, 0x5, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x10, 0x0, 0x0, 0x0
_NET_WM_NAME(UTF8_STRING) = "Multimedia Player VLC"
WM_CLIENT_LEADER(WINDOW): window id # 0x2e00005
WM_WINDOW_ROLE(STRING) = "vlc-main"
_NET_WM_PID(CARDINAL) = 2835
_NET_WM_WINDOW_TYPE(ATOM) = _NET_WM_WINDOW_TYPE_NORMAL
_MOTIF_WM_HINTS(_MOTIF_WM_HINTS) = 0x3, 0x3e, 0x7e, 0x0, 0x0
WM_PROTOCOLS(ATOM): protocols  WM_DELETE_WINDOW, WM_TAKE_FOCUS, _NET_WM_PING, _NET_WM_SYNC_REQUEST
WM_NAME(STRING) = "Multimedia Player VLC"
WM_LOCALE_NAME(STRING) = "en_US.UTF-8"
WM_CLASS(STRING) = "vlc", "Vlc"
WM_HINTS(WM_HINTS):
		Client accepts input or input focus: True
		Initial state is Normal State.
		bitmap id # to use for icon: 0x2e00012
		window id # of group leader: 0x2e00005
WM_NORMAL_HINTS(WM_SIZE_HINTS):
		user specified location: 20, 62
		program specified location: 20, 62
		user specified size: 1880 by 998
		program specified size: 1880 by 998
		program specified minimum size: 489 by 368
		window gravity: NorthWest
WM_CLIENT_MACHINE(STRING) = "hostname"
WM_COMMAND(STRING) = { "vlc" }

Hum thats strange, I thought it was fixed a while ago, I might take a look this weekend... If you can find where it is in the config files, pls PR (I really dont have a lot of time right now :/)


Remove Py 2 - image size

I think there are still two Python versions shipped with ACS.
We can do two things to reduce the size if the image.

remove the Py 2 version (probably very simple)
remove actually both Py versions, which are shipped with ACS .. and use the system python of Centos ...

Also I think ACS installs scipy ... which is not needed by ACS .. but is nice to have for Python developers ...

In my current testing branch https://github.com/dneise/acs_test/tree/2020.6_release
I can confirm that Python3 really works.
However I find working with pyenv here really annoying and I wonder if getting rid of all the Python2 in the container would be a good thing.
On the other hand ALMA I guess still has a lot of Py2 sources which they had no time to convert to Py3 yet, so for them removing Py2 would be a killer.


✏ Reword and clarify text in tutorial docs/en/docs/tutorial/body-nested-models.md

It currently says "list of items", and since the entity in the example is called Item, this was a little confusing for me. Avoiding the word "item" in this sentence is probably better.

Great! Good point, thanks @papb! :rocket:


Update alert toaster to handle 'detail' errors

This came up while testing attempting to sync EDA projects when Redis is down.  The API responds with a 409 error and a body that looks like:
{
  "detail": "Redis is required but unavailable."
}

This change allows us to show the contents of the error message to the user like this:

Rather than the current UX:

This got a little bit messier than I thought it would but I looked for instances where we're using errorToAlertProps and the majority of them are here in EDA.  There are a couple in Hub that I don't think will be impacted as only the error object is a required parameter.

@lgalis I abandoned the errorToAlertProps approach and rolled a small utility function to parse the error and display the string.  That utility function could probably be much more robust but I think it'll get the job done here in the short term.


JetBrains now stopped all sales in Russia and Belarus

https://blog.jetbrains.com/2022/03/11/jetbrains-statement-on-ukraine/

Today we are announcing that we will be suspending sales and R&D activities in Russia indefinitely. Sales in Belarus are also suspended indefinitely.

Couldn't post it in https://github.com/vshymanskyy/StandWithUkraine/issues/11 since it is locked.
/cc @azjezz

I just saw this after creating https://github.com/vshymanskyy/StandWithUkraine/pull/28

It seems nonsensical to me to boycott JetBrains. They made their position clear and are following up with actions, including closing their offices in Russia.

This is enough to be removed from the list. As you see this was announced just today (March 11, 2022).
Thanks for letting us know

Thanks @vshymanskyy! :slightly_smiling_face:


Implement middleware as a different format

Describe the issue
Currently, the middleware is implemented in a different format and needs to be converted into a list.
Additional context
It allows for improved compatibility, modularity, and easier integration within the system.

✅ Done


[Infineon] SoftwareDiagnostic update

Problem
Infineon CYW30739 doesn't support DGSW.S.F00(Watermarks)
Change overview
Add DGSW.S.F00(Watermarks) support
Testing
./chip-tool softwarediagnostics reset-watermarks 1234 0

Optional feature not being implemented is ok. This change is not needed (and likely harmful by claiming an unimplemented feature works).

These are perfectly allowed in the SDK, they're just not mandatory. Approving

@bzbarsky-apple We just added DGSW.S.F00(Watermarks) support to pass #3 "./chip-tool softwarediagnostics read feature-map 1  0" of TC-DGSW-1.1, or it would read the value "0" not expected "1".

@AlvinHsiao That test only expects 1 if the relevant PICS is claiming that the device supports watermarks, no?

@bzbarsky-apple  Yes. But it seems no option to pass that test if the device not supports watermarks.

@bzbarsky-apple Yes. But it seems no option to pass that test if the device not supports watermarks.

The test is wrong. If the spec says "Optional" you are allowed and spec compliant to not implement it.

But it seems no option to pass that test if the device not supports watermarks.

No option in what sense?  How exactly are you running the test?

The test plan expects that the FeatureMap attribute has a value 1, but we got value 0 if we don't add "bool SupportsWatermarks() override { return true; }"

The test plan expects that the FeatureMap attribute has a value 1

If the test plan expects that, the test plan is broken and that is what we should fix.  I filed https://github.com/CHIP-Specifications/chip-test-plans/issues/1571 on the test plan.


update FSRS-rs to 1.3.3

What's Changed between FSRS-rs v1.1.3 and FSRS-rs v1.3.3

Fix/smooth stability after training by @L-M-Sherlock in https://github.com/open-spaced-repetition/fsrs-rs/pull/212
Feat/check_and_fill_parameters by @L-M-Sherlock in https://github.com/open-spaced-repetition/fsrs-rs/pull/214
Check parameters are valid numbers by @dae in https://github.com/open-spaced-repetition/fsrs-rs/pull/217
Fix/simulator crashes if no history by @L-M-Sherlock in https://github.com/open-spaced-repetition/fsrs-rs/pull/216
Feat/fine-tune the sample size for CMRR by @L-M-Sherlock in https://github.com/open-spaced-repetition/fsrs-rs/pull/222
Fix/filter_outlier outputs dataset in arbitrary order by @L-M-Sherlock in https://github.com/open-spaced-repetition/fsrs-rs/pull/223
Fix/filter rating_stability to match rating_count keys by @L-M-Sherlock in https://github.com/open-spaced-repetition/fsrs-rs/pull/225
Feat/float next interval by @L-M-Sherlock in https://github.com/open-spaced-repetition/fsrs-rs/pull/213
R_MIN = 0.70 by @Expertium in https://github.com/open-spaced-repetition/fsrs-rs/pull/229
Fix/update FSRS-4.5 param into FSRS-5 properly by @L-M-Sherlock in https://github.com/open-spaced-repetition/fsrs-rs/pull/230
derive Copy for trivial struct by @sineptic in https://github.com/open-spaced-repetition/fsrs-rs/pull/236
Make simulate iterate by card instead of by day. by @Luc-Mcgrady in https://github.com/open-spaced-repetition/fsrs-rs/pull/235
Fix/sample_size should be continuous over learn_span by @Expertium in https://github.com/open-spaced-repetition/fsrs-rs/pull/238

Full Changelog: https://github.com/open-spaced-repetition/fsrs-rs/compare/v1.1.4...v1.3.3

Thanks! I'm not sure why the auto-update CI failed. I'll have to keep an eye on that. Please feel free to open an issue or @me  if you notice something wrong.


securityadmin.sh overwrites config

There MUST be something done with your documentation guys. Really, why would you tell users to re-run securityadmin.sh if you effectively destroy all existing configuration and upload the one that's defined by configs. Why not just try to merge it instead?
Why not put a red warning in docs that running this script will erase all their config?
I had to modify admin user config to include a snapshot restore role (why not include that by default?!) and reran the script which erased hours of work.

ouch, cc: @aetter

@celesteking security plugin provides REST API to add (merge) role to the existing roles, there is no need to re-run securityadmin.sh for that use case. The securityadmin is mostly designed for backup and restore use case.
@aetter Please close this issue after updating documentation.

I'm in complete agreement that we should offer some sort of merge option on the script. Blowing away the existing configuration without a word of warning in the script itself flies in the face of user expectations, and it's the only way to add new reserved or hidden users. Until we improve the script, I've added an extensive warning section here: https://github.com/opensearch-project/documentation-website/pull/135

We are doing some "spring cleaning in the fall", and to make sure we focus our energies on the right issues and we get a better picture of the state of the repo, we are closing all issues that we are carrying over from the ODFE era (ODFE is no longer supported/maintained, see post here).
If you believe this issue should still be considered for current versions of OpenSearch, apologies! Please let us know by re-opening it.
Thanks!


update Contributors directive to merge to main

@lstein - Assuming we're having people merge to main (still calling out doing that AND having an auto-merge is high risk), updating directive to do so.

@lstein - Assuming we're having people merge to main (still calling out doing that AND having an auto-merge is high risk), updating directive to do so.

The answer is for the reviewing team to be careful before approving a PR and not do it unless they're absolutely sure it is good to go.
@lstein I wonder if it is possible to set 2 mandatory approval requirements for new or non approved contributors and 1 for the rest?

I don't think that there's a way to require more approvals for one type of
contributor than another.
The auto-merge feature is an option, and I suggest that we only use it for
very small fixes. Reviewers are able to set and unset it as well, and I
have been setting it to off for anything I have any uncertainty about.
Happy to discuss.
Lincoln
On Wed, Dec 14, 2022 at 8:11 AM blessedcoolant @.***>
wrote:

@lstein https://github.com/lstein - Assuming we're having people merge
to main (still calling out doing that AND having an auto-merge is high
risk), updating directive to do so.
The answer is for the reviewing team to be careful before approving a PR
and not do it unless they're absolutely sure it is good to go.
@lstein https://github.com/lstein I wonder if it is possible to set 2
mandatory approval requirements for new or non approved contributors and 1
for the rest?
—
Reply to this email directly, view it on GitHub
https://github.com/invoke-ai/InvokeAI/pull/1984#issuecomment-1351330298,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA3EVP2NFC65ZMKIQEQXJ3WNHBRHANCNFSM6AAAAAAS55ONMI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Lincoln Stein
Head, Adaptive Oncology, OICR
Senior Principal Investigator, OICR
Professor, Department of Molecular Genetics, University of Toronto
Tel:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
@.***
E**xecutive Assistant
Michelle Xin
Tel:<PHONE_NUMBER>
@.*** @.**>
Ontario Institute for Cancer Research
MaRS Centre, 661 University Avenue, Suite 510, Toronto, Ontario, Canada M5G
0A3
@OICR_news
https://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftwitter.com%2Foicr_news&data=04|01|Michelle.Xin%40oicr.on.ca|9fa8636ff38b4a60ff5a08d926dd2113|9df949f8a6eb419d9caa1f8c83db674f|0|0|637583553462287559|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=PS9KzggzFoecbbt%2BZQyhkWkQo9D0hHiiujsbP7Idv4s%3D&reserved=0
| www.oicr.on.ca
Collaborate. Translate. Change lives.
This message and any attachments may contain confidential and/or privileged
information for the sole use of the intended recipient. Any review or
distribution by anyone other than the person for whom it was originally
intended is strictly prohibited. If you have received this message in
error, please contact the sender and delete all copies. Opinions,
conclusions or other information contained in this message may not be that
of the organization.

I think its okay to stay. We can be a tad bit careful and I think it'll have its advantages in the long run. So we don't sit around waiting for checks to pass on PR's that we have fully approved and cleared.

Case in point on this PR. I reviewed and approved it, and all checks had passed, but because main had been updated since this PR was submitted I had to do an "Update from main" operation before I could do the merge. This of course triggered the CI tests, meaning I would have to come back to this PR in about half an hour to do the actual merge.
Rather than having to come back, I then set auto-merge to true. So now if the CI tests pass (which they should!), the merge will happen without this additional work.

And meanwhile, despite automerge being enabled its... stuck in an "out-of-date" with base branch status. 😂


Question: Any way to pick up intermediary results?

Hi there. Loving the package, keep it up!
Have a question: is there a way to pick up result values from steps before the end of the survey? The use case I have is that I would like to compute some intermediary scores based on values picked in questions, and then dependent on those scores show the right final question. I could add Navigation Rules where I pick up the value and move to next step, but I see it more of a hack and also an annoyance when having a lot of questions. So, any other way to do it?

You can use the SurveyController with its three methods:

onNextStep()
onStepBack()
onCloseSurvey()

With those three methods, you can observe the current state of the survey.


File APIs and minor changes to NNNTensor.

Added File APIs to write entire storages, arrays, and tensors.
Changed how Tensor.Storage was used.
Changed Tensor.Data to return IntPtr.
Changed Tensor.NumElements to read-only property.
Added unit tests.

This branch mine started showing up with errors. I'll pull back and submit a partial PR.


Update Import/export doc with regard to the suggested number of workers

Also this doc need to be updated?
https://cloud.google.com/bigtable/docs/importing-sequence-files


refactor(Rating): refactor component

WIP
This PR:

splites Rating's icon to separate internal component;
updates docs to unified format;
minor refactors.

I'll update tests tomorrow, so we can merge it soon 👍

Current coverage is 100% (diff: 100%)

Merging #640 into master will not change coverage

@@           master   #640   diff @@
====================================
  Files         119    120     +1
  Lines        1890   1891     +1
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
+ Hits         1890   1891     +1
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 3fb0ca7...c391428

I added tests for RatingIcon, it was easy.
I started updating of Rating tests for use RatingIcon instead of i.icon, but get stuck there. I added todos where test might be updated.
I don't understand how I can convert this to component test:
icons.find('.active').last().simulate('click')
icons.every('.icon.active')
  .should.equal(true, 'Some icon lost its "active" class')

If I correcty undestand enzyme docs, I can convert it to this?
icons.findWhere(n => prop('active', true)).last().simulate('click')
icons.everyWhere(n => prop('active', true))
  .should.equal(true, 'Some icon lost its "active" class')

You can also use the object property selector:
icons.find({ active: true })

You can also use the object property selector:

According to airbnb/enzyme/#582 - no, in mount mode, so I used findWhere there.
All tests are working with RatingIcon now, we're ready for final review there. Please, pay attension to Rating tests.

Restored messages to asserts and added some new. It's enought? Or I need add message to each assert?

Released in<EMAIL_ADDRESS>thanks once again!


grid-cols-[xxx] parsed incorrectly to CSS

First off, love all the work you guys do :)
Currently using the latest version of UnoCSS via Vitesse.
Attempting to create a grid layout as so:

            Menu 1

Seems like the compiler recognizes cols-[1fr,2fr] and turns it into grid-template-columns: 1fr,2fr, but incorrectly not remove the , between 1fr and 2fr which results in a CSS error. Attempting to write cols-[1fr 2fr] results in the compiler not recognizing it.
Would this be considered a bug in the preset? I've not added any other presets or anything btw

Use cols-[1fr_2fr] instead, _ will be replaced to space.


Endpoints_resolution_error when using ROPC flow in msal-node (B2C)

Core Library
MSAL Node (@azure/msal-node)
Core Library Version
1.3.1
Wrapper Library
Not Applicable
Wrapper Library Version
None
Description
I'm trying to acquire a token from our Azure AD B2C authority via the ROPC flow (using acquireTokenByUsernamePassword) but always get an endpoints_resolution_error. I played around with the authority url quite a bit but couldn't get it to work, although the .well-known url in the error description is returning the meta data properly.
Note: Retrieving the token using the same endpoint is working fine via manual get request as described here.
Error Message
ClientAuthError: endpoints_resolution_error: Error: could not resolve endpoints. Please check network and try again. Detail: ClientAuthError: openid_config_error: Could not retrieve endpoints. Check your authority and verify the .well-known/openid-configuration endpoint returns the required endpoints. Attempted to retrieve endpoints from: https://<tenant>.b2clogin.com/<tenant>/b2c_1_ropc_auth/v2.0/.well-known/openid-configuration
    at ClientAuthError.AuthError [as constructor] (...@azure/msal-common/dist/index.cjs.js:477:24)
    at new ClientAuthError (...@azure/msal-common/dist/index.cjs.js:754:28)
    at Function.ClientAuthError.createEndpointDiscoveryIncompleteError (...@azure/msal-common/dist/index.cjs.js:791:16)
    at Function.<anonymous> (...@azure/msal-common/dist/index.cjs.js:6921:47)
    at step (...@azure/msal-common/dist/index.cjs.js:79:23)
    at Object.throw (...@azure/msal-common/dist/index.cjs.js:60:53)
    at rejected (...@azure/msal-common/dist/index.cjs.js:51:65)
    at processTicksAndRejections (internal/process/task_queues.js:97:5) {
  errorCode: 'endpoints_resolution_error',
  errorMessage: 'Error: could not resolve endpoints. Please check network and try again. Detail: ClientAuthError: openid_config_error: Could not retrieve endpoints. Check your authority and verify the .well-known/openid-configuration endpoint returns the required endpoints. Attempted to retrieve endpoints from: https://<tenant>.b2clogin.com/<tenant>.onmicrosoft.com/b2c_1_ropc_auth/v2.0/.well-known/openid-configuration',
  subError: '',
  correlationId: ‘<correlationId>’

(Obfuscated the tenant name)
Msal Logs
No response
MSAL Configuration
{
  auth: {
    clientId: "<appId>",
    authority: "https://<tenant>.b2clogin.com/<tenant>.onmicrosoft.com/B2C_1_ROPC_Auth",
    redirectUri: "http://localhost:3001",
    knownAuthorities: ["<tenant>.b2clogin.com"],
    validateAuthority: false,
    postLogoutRedirectUri: "/"
  },
  cache: {
    cachePlugin
  },
}

Relevant Code Snippets
const pca = new msal.PublicClientApplication(msalConfig);

let scope = "openid https://<tenant>.onmicrosoft.com/<appid>/access_as_user offline_access";
let user = "<user>";
let pass = "<pass>";

const usernamePasswordRequest = {
  scopes: [scope],
  username: user,
  password: pass,
};

let authResult;
try {
  authResult = await pca.acquireTokenByUsernamePassword(usernamePasswordRequest);
} catch (error) {
  console.log(error);
}

Reproduction Steps

Create a PlublicClientApplication using the specified config
Build a request
call acquireTokenByUsernamePassword

Expected Behavior
A authorization Token is received within the authentication result succesfully.
Identity Provider
Azure B2C Basic Policy
Browsers Affected (Select all that apply)
None (Server)
Regression
No response
Source
External (Customer)

@birneamstiel Can you please email me a fiddler trace. You can find my email on my profile. Thanks!

Thanks for coming back to this @tnorling. I don't see any related request showing up in fiddler, seems like the logic is failing before any request is made...

@birneamstiel This error is thrown as a result of the GET request to https://<tenant>.b2clogin.com/<tenant>.onmicrosoft.com/b2c_1_ropc_auth/v2.0/.well-known/openid-configuration failing. Is it possible your fiddler configuration is filtering out the relevant requests?
Alternatively, you can also provide me with a link to a small sample repo that reproduces this issue.

@birneamstiel This will be addressed in teh linked PR, and should be included in the next release.


Could Database be Clone?

I noticed that both fields of Database are Arcs, and was wondering if the Database could implement Clone, since it should be cheap, and since creating read and write transactions don't rely on borrowing the Database, it should be safe.

No. compact() takes a &mut self
On Mon, Apr 1, 2024 at 12:55 AM Casey Rodarmor @.***>
wrote:

I noticed that both fields of Database are Arcs, and was wondering if the
Database could implement Clone, since it should be cheap, and since
creating read and write transactions don't rely on borrowing the Database,
it should be safe.
—
Reply to this email directly, view it on GitHub
https://github.com/cberner/redb/issues/789, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGNXQHRPBFFYYDV4SMW6YLY3EHHVAVCNFSM6AAAAABFRHJOY6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGIYTONZVGUYTOOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Ahh, darn.

So can the database be shared throughout an app? If yes, then is it assumed we should wrap it in our own Arc?

Yes, ideally you can just pass &Database around in your components as the methods to start transactions take &self and Database is Send + Sync.
This could mean that you have to wrap it inside an Arc, for example if the value is not part of a larger struct and has its lifecycle determined by that.


Speed up remapping of labels

https://github.com/BlueBrain/atlas-annotation/blob/13574c24f01958bcf08e399712bb28d0d4183950/src/atlannot/ants/__init__.py#L61-L70
We stumbled upon 2 issues:

The remap_labels seems to be very slow (it is both inside of register and transform)
Both register and transform require the user to pass volumes of type float32, however, is  that really necessary when dealing with atlases? 2 reasons why we might need to rethink

We even stumbled upon "overflow" issues when casting large integers to float32
If the remap_labels does not work in-place there is no need to enforce float32 inputs to it

Closed with 118af9b.


Reference to the component's properties are being lost

Hi,
The module is working fine, but there is one thing that I noticed with the update from ng2-owl-carousel to ngx-owl-carousel.
If I used a reference to the component's properties in order to update their values in my event callbacks with the old ng2-owl-carousel, they actually got updated and the new value was shown in the template.
The same thing however does not work when I use this new ngx-owl-carousel module.
I wrote my callbacks directly in carousel's options like so:
export class MyComponent {
public carouselOptions: any = {
items: 1,
dots: false,
navigation: false,
onInitialized: (event) => {
this.myVar1 = xxxx; // a value to be updated
},
onDragged: (event) => {
this.myVar2 = xxxx; // a value to be updated
}
};
}
In template:
<owl-carousel
[options]="carouselOptions">
....

{{myVar1}}
{{myVar2}}
With the new version of the module (ngx-owl-carousel) this does not work.
Regrads

Possibly same issue #23

It does in fact bind the Component class to the method, but the problem persists, the values of the properties of the Component class are not being updated.
I noticed also the same case as before, it does not work with ngx-owl-carousel Module, but it does however work with the old ng2-owl-carousel Module

I've implemented owl carousel in angular 6, carousel seems to be working fine.
I wanted to change the sliderOptions, so I had given sliderOptions property in my component for it to work on responsiveness and number of items to be shown in a view.
Could anyone help me with this?
Thanks.
<owl-carousel [options]="sliderOptions" [items]="content" [carouselClasses]="['owl-theme', 'row', 'sliding']">
sliderOptions: {
margin: 25,
dots: false,
navigation: true,
nav: true,
autoplay: true,
loop: true,
autoplayTimeout: 2000,
autoplayHoverPause: true,
lazyLoad: true,
responsive: {
0: { items: 1 },
600: { items: 1 },
960: { items: 1 },
1200: { items: 1 }
}


Add ggtitle to plotPrevalenceByLocation

See also #32

How is this issue different from #32 ?

Whoops, I must have copied the wrong function name.  There's another plotting function that could use a title, lmk if you can't figure out which one.

Got it! I think it's plotAllLineagesByLocation. Thank you!


[move-prover] Update script_incorrect.exp

Fix the baseline file for the "script_incorrect" test which was broken from https://github.com/libra/libra/pull/3669

Motivation
To fix the broken baseline file
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
cargo test

@bors-libra r=wrwg

:pushpin: Commit 9294d51 has been approved by wrwg

:hourglass: Testing commit 9294d51ed12d927975ed11bd407366b16b71302e with merge 7ed092f3803799b7e94f9c5c2734dfb0e6e0d1d2...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: wrwg
Pushing 7ed092f3803799b7e94f9c5c2734dfb0e6e0d1d2 to master...


Make HasUnit units non-Optional

Depends on #228
Note that this triggers warnings in PyCharm in shared/definitions/custom/py. This appears to be a manifestation of https://youtrack.jetbrains.com/issue/PY-60352/False-positive-Inherited-non-default-arguments-warning-with-multiple-inheritances
Setting a default value for unit in HasUnit fixes the above warning, but we don't want that.

merged as part of #243


Removes the image && video check to enable the IE11 fallback

https://github.com/googlevr/vrview/issues/266

@andrebonon This looks good, please sign the CLA and I will merge.

@lincolnfrog, done!


Add "Runs on host" info

Admin user can link-through to Cockpit running on a host with particular VM.
Fixes: https://github.com/oVirt/ovirt-web-ui/issues/117

See the hyperlink in the status filed:

For longer host name:

and narrow screen:

What about rendering the (on [HOST_NAME]) separately as a first entry of the second column (right above the Consoles).
Sort of Runs on:      [LINK_TO_HOST]

Anyway, we shall avoid additional field

What about this? @jelkosz , @jniederm
How to rephrase?

For non-admin user:

@mareklibra

maybe instead of "On" I would put "Runs on host: ..."
the hostname should look like you can click it so you know it you can click it

@mareklibra 👍

I have doubts about the tooltip solution. As a new user to webui, how am I supposed to find it? Ctrl+F couldn't work. I'd prefer regular additional name-value widget.

Let's try it this way. We can change to name-value solution or something even better later.
The reasons for preferring the per-click context information now are:

it's a pattern in web-ui to use blue-style of <a href> to route and gray-underlined-style to denote local context-aware additional information
so the user knows the [status] can be clicked since the notation is used on other places as well
I don't think the user will search (like Ctrl+F) for host name on the VmDetail page. It's for a single VM.
We might add new Hosts tab listing all VMs defined on particular host (+ more host info), if needed. Would be linked from the VmPage as well.
This cockpit-link is rendered for admin users only (not the "typical" kind of users)
context information of the [status] is used for other kind of information, like status reason - see later commit in this PR. So it's not single-purpose.
the page shall provide sufficient amount of information but not too much to keep it understandable at the first glance. This is sort of "advanced" information and adding extra field for it is overwhelming in my opinion.
Maybe we will be forced to split the recent layout of VmDetail page to "sort of subtabs or so" in the future. Anyway, I would like to avoid/postpone it as much as possible.

The web-ui is focused on standard, non-admin users. We have to keep the presented information at minimal useful level. For more details, there's webadmin.

Enriched for status reason (requested in https://github.com/oVirt/ovirt-web-ui/issues/247):


Use Coveralls GitHub Action

This uses the Coveralls GitHub Action in place of coveralls-reborn as recommended in the gem README https://github.com/tagliala/coveralls-ruby-reborn#github-actions and as discussed here https://github.com/ruby-grape/grape/pull/2265. The parallel test steps are taken from the Action README https://github.com/coverallsapp/github-action
Examples on my fork:

Job for a merge showing individual Ruby version & Gemfile matrix https://coveralls.io/builds/50032540
Master after a merge https://coveralls.io/github/duffn/grape

Much better! Add a changelog with something like “Fixed code coverage” and we are good to go.


fix: Issue #861  Removed improper Flickering of elements

Title and Issue number
Title : Removed improper flickering of div elements
Issue No. : #861
Description
Before:
when a client mouse over on the divs at Frequently Asked Questions Section are getting flickering due to improper sizing of divs and  hovering.
https://github.com/apu52/Travel_Website/assets/108344062/3c1c714c-4fe4-4a79-9c3d-7f37157927e2
After:
Now I have removed the flickering effect and I adjusted the toggle button in such a way that there is no overlapping between the FAQ title(Questions)
https://github.com/apu52/Travel_Website/assets/108344062/0fc62e85-57b4-456a-96a9-23a294b97f24
Type of PR

[X] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

[X] I have performed a self-review of my code.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have gone through the  contributing.md file before contributing

##Are you contributing under any Open-source programme?

[X] I am contributing under GSSOC'24
[ ] I am contributing under VSOC'24

Hi @apu52
Please go through the PR and add appropriate labels to it


Add a notification about the missing token

Fixes https://github.com/sourcegraph/sourcegraph/issues/57373.
Note this is a notification part only! Not a solution to the root cause.
Demo:
https://github.com/sourcegraph/jetbrains/assets/19799111/b279dab5-6d07-4e1f-85fb-215a95691ec0

Looks great :+1:
@mkondratek, please note that commit message has fixes so we will need to re-open linked issue after merge.


[Type checker] Don't merge dictionary key type variables with different kinds

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

…nt expression kinds.
The constraint generator's optimization to eagerly merge type
variables for different keys in a dictionary literal was too eager,
merging the type variables for (e.g.) a string literal and an integer
literal. This prevented us from properly inferring AnyHashable key
types in dictionary literals. Fixes the rest of rdar://problem/27661580.

@please smoke test and merge

@swift-ci please smoke test and merge