samples/subsys/usb/mass/usb_device_next fails on frdm_k64f

Describe the bug
The sample.yml contains three samples.
sample.usb.mass_ram_none
sample.usb.mass_ram_fat
sample.usb_device_next.mass_ram_none

but the only one fails.
sample.usb_device_next.mass_ram_none
To Reproduce
twister --device-testing --hardware-map <path_to>/map.yml -T ./samples/subsys/usb/mass/ -v
Expected behavior
Passes.
Impact
Annoyance.
Logs and console output
Renaming output directory to <path_to>/twister-out.12
INFO    - Using Ninja..
INFO    - Zephyr version: zephyr-v3.3.0-3578-g1f43e126a841
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report <path_to>/twister-out/testplan.json

Device testing on:

| Platform   | ID                                               | Serial device                                                                                |
|------------|--------------------------------------------------|---------------------------------------------------------------------------------------------------|
| frdm_k64f  | 02400b0132044e4500417009997b00316781000097969900 | /dev/serial/by-id/usb-ARM_DAPLink_CMSIS-DAP_02400b0132044e4500417009997b00316781000097969900-if01 |
| frdm_k64f  | 02400b0140214e45001b1018c89200349e11000097969900 | /dev/serial/by-id/usb-ARM_DAPLink_CMSIS-DAP_02400b0140214e45001b1018c89200349e11000097969900-if01 |

INFO    - JOBS: 8
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - 5/7 frdm_k64f                 samples/subsys/usb/mass/sample.usb.mass_ram_none   PASSED (device 3.925s)
INFO    - 6/7 frdm_k64f                 samples/subsys/usb/mass/sample.usb.mass_ram_fat    PASSED (device 5.516s)
INFO    - 7/7 frdm_k64f                 samples/subsys/usb/mass/sample.usb_device_next.mass_ram_none  FAILED Timeout (device 64.573s)
ERROR   - see: <path_to>/twister-out/frdm_k64f/samples/subsys/usb/mass/sample.usb_device_next.mass_ram_none/handler.log

INFO    - 7 test scenarios (7 test instances) selected, 4 configurations skipped (4 by static filter, 0 at runtime).
INFO    - 2 of 7 test configurations passed (66.67%), 1 failed, 0 errored, 4 skipped with 0 warnings in 69.99 seconds
INFO    - In total 3 test cases were executed, 4 skipped on 1 out of total 567 platforms (0.18%)
INFO    - 3 test configurations executed on platforms, 0 test configurations were only built.

Hardware distribution summary:

| Board     | ID                                               |   Counter |
|-----------|--------------------------------------------------|-----------|
| frdm_k64f | 02400b0132044e4500417009997b00316781000097969900 |         1 |
| frdm_k64f | 02400b0140214e45001b1018c89200349e11000097969900 |         2 |
INFO    - Saving reports...
INFO    - Writing JSON report <path_to>/twister-out/twister.json
INFO    - Writing xunit report <path_to>/twister-out/twister.xml...
INFO    - Writing xunit report <path_to>/twister-out/twister_report.xml...
INFO    - -+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
INFO    - The following issues were found (showing the top 10 items):
INFO    - 1) samples/subsys/usb/mass/sample.usb_device_next.mass_ram_none on frdm_k64f failed (Timeout)
INFO    -
INFO    - To rerun the tests, call twister using the following commandline:
INFO    - west twister -p <PLATFORM> -s <TEST ID>, for example:
INFO    -
INFO    - west twister -p frdm_k64f -s samples/subsys/usb/mass/sample.usb_device_next.mass_ram_none
INFO    - or with west:
INFO    - west build -p -b frdm_k64f -T samples/subsys/usb/mass/sample.usb_device_next.mass_ram_none
INFO    - -+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
INFO    - Run completed

but the handler.log contains information like this.
[00:00:00.000,000] <inf> main: No file system selected
[00:00:00.002,000] <err> usbd_msc: Cannot register LUN SD
[00:00:00.002,000] <err> usbd_init: Failed to initialize class instance
[00:00:00.002,000] <err> usbd_init: Failed to init configuration 1
[00:00:00.002,000] <err> main: Failed to initialize device support
[00:00:00.002,000] <err> main: Failed to enable USB

It seems like this sample is prepared for check the external card reader but frdm_k64f does not support this feature.
Environment (please complete the following information):

OS: Linux
Toolchain: Zephyr SDK 0.16.0
West version:  v0.13.1
Latest main: 68d1d7d

Could be the same issue as seen here: https://github.com/zephyrproject-rtos/zephyr/issues/57654#issuecomment-1541718040

@gbarkadiusz I ran the sample using west
west build -b frdm_k64f zephyr\samples\subsys\usb\mass
The sample works as expected.

@mmahadevan108 , this case is failed for very long time, you need use this config
west build -b frdm_k64f zephyr\samples\subsys\usb\mass -- -DCONF_FILE=usbd_next_prj.conf -DCONFIG_LOG_DEFAULT_LEVEL=3

and with below config it is ok
west build -b frdm_k64f zephyr\samples\subsys\usb\mass -- -DCONFIG_LOG_DEFAULT_LEVEL=3

@hakehuang You're right, after changing this parameter in the sample.yml, it's passing.
Before.
...
sample.usb_device_next.mass_ram_none:
    min_ram: 64
    depends_on: usb_device
    platform_allow:
      - nrf52840dk_nrf52840
      - frdm_k64f
    extra_args: CONF_FILE="usbd_next_prj.conf"
    extra_configs:
      - CONFIG_LOG_DEFAULT_LEVEL=3
    tags:
      - msd
      - usb
    harness: console
    harness_config:
      type: multi_line
      ordered: true
      regex:
        - "No file system selected"
        - "The device is put in USB mass storage mode."
 ...

After
...
sample.usb_device_next.mass_ram_none:
    min_ram: 64
    depends_on: usb_device
    platform_allow:
      - nrf52840dk_nrf52840
      - frdm_k64f
#    extra_args: CONF_FILE="usbd_next_prj.conf"
    extra_configs:
      - CONFIG_LOG_DEFAULT_LEVEL=3
    tags:
      - msd
      - usb
    harness: console
    harness_config:
      type: multi_line
      ordered: true
      regex:
        - "No file system selected"
        - "The device is put in USB mass storage mode."
...

So, maybe we need to prepare the PR for this change?

@gbarkadiusz , no this is an issue to this config, and as it is for usbd_next_prj.conf so maybe usb maintainer need check this issue.
@tomaszduda23

USBD_MSC_LUNS_PER_INSTANCE is one thing, but for some reason the device does not seem to respond with the USBS to Mode Sense(6) on LUN 0.

The way endpoint stall in Mode Sense(6) is handled seems to differ between nRF and K64F.
If the bulk in endpoint stall at
https://github.com/zephyrproject-rtos/zephyr/blob/98810bc7a1649b4a8ccffa55176559f81512e283/subsys/usb/device_next/class/usbd_msc.c#L508-L515
is removed, then everything works fine. The stall is handled correctly on nRF52840DK therefore I think it is K64F udc driver. Note that just removing the stall is not really a solution.
On K64F, the CSW enqueued in msc_send_csw() doesn't make it to the bus. Note that this is the first queued IN data after IN endpoint STALL in the whole communication since USB reset (therefore it seems unlikely that Mode Sense(6) command is the culprit but rather a first victim).


Require multiline constructor [parameter] signature when long

  <rule ref="SlevomatCodingStandard.Classes.RequireMultiLineMethodSignature">
    <properties>
      <property name="minLineLength" value="80"/>
      <property name="includedMethodPatterns" type="array">
        <element value="/__construct/" />
      </property>
    </properties>
  </rule>

Constructor parameter list can get very long, especially with promoted properties.
As required, this rule is autofixable.

/ https://github.com/kubawerlos/php-cs-fixer-custom-fixers#multilinepromotedpropertiesfixer

Also needs <rule ref="SlevomatCodingStandard.Functions.RequireTrailingCommaInDeclaration" /> in order to enforce trailing comma in declaration.
This is autofixable.

6 / 6 votes for


export interfaces on defineXFactory

First of all, really like the look of this library. Hoping to switch to it from our current solution.
However having an issue with creating reusable factory templates and exporting them (e.g. for overriding defaults across several different test files). For example:
export const factory = defineUserFactory({
  defaultData: async () => ({
    displayName: faker.name.fullName().slice(0, 30),
    email: faker.internet.email().slice(0, 100),
  }),
});

Current fails to compile with Exported variable 'factory' has or is using name 'UserFactoryInterface' from external module "<dir>/src/factories/__generated__/fabbrica/index" but cannot be named.. This would be solvable by being able to import UserFactoryInterface, however the generated factories don't export these interfaces.
Is it possible to either:

Export the related interfaces for each defineXFactory, so they can be imported into another file?
For me to do this in a better way that doesn't require that above?

Thanks for your time!

@Pete-Hamlin
Thanks for reporting this issue. I fixed this and published as v1.0.2


openjdk{16,18,19,20}, openjfx{20,22}: update

Description of changes
Split from #286267.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 313920 run on x86_64-linux 1

1 package marked as broken and skipped:

minecraftServers.vanilla-1-17

29 packages built:

autopsy
corretto19
corretto19.debug
cryptomator
jabref
openjfx19 (javaPackages.openjfx19)
openjfx20 (javaPackages.openjfx20)
openjfx21 (javaPackages.openjfx21)
openjfx22 (javaPackages.openjfx22)
jdk19 (openjdk19)
jdk19.debug (openjdk19.debug)
jdk19_headless (openjdk19_headless)
jdk19_headless.debug (openjdk19_headless.debug)
jdk20 (openjdk20)
jdk20.debug (openjdk20.debug)
jdk20_headless (openjdk20_headless)
jdk20_headless.debug (openjdk20_headless.debug)
maptool
mcaselector
moneydance
mysql-workbench
ns-usbloader
olvid
pattypan
pdfsam-basic
quark-goldleaf
scenebuilder
sparrow
sparrow-unwrapped

@ofborg build openjfx19 openjfx20 openjfx21 openjfx22


Updated smirky sampler with commandline tool

This pull request has a lot of changes including updates to setup.py to install the command line tool smirky (addressed in issue #128 ).
While it does not actually create the proposal engine addressed in issue #110 the create_new_environment function in sampler_smirky.py is significantly more modular than before so much of the code should be reusable.
The sampler attempts to address detailed balance, only removing things it can create in one move as discussed in issue #99, but this will likely remain a concern any time we make changes to the way moves are made.
This is now a separate tool from smarty with it's only driver (cli_smirky.py) and is callable with smirky --options
I haven't updated anything in the main README, but there are two existing examples at examples/smirky/default_angle and examples/smirky/dummy_torsion_sampler
It should be noted that while testing I created and destroyed a few different log files causing the number of deletions to be a bit ridiculous.
There are a lot of changes to environment.py and sampler_smirky.py. I updated the tests and made the new examples, but it's possible there may be problem with older examples using these, i.e. @cbayly13 's ipython notebooks.

@davidlmobley when you have time could you take a look?
@cbayly13 you will probably want to see what this is doing so far and maybe comment on the odds used for move making inside the sampler.
I'm going to make a more complicated example tomorrow with specified decorator/odds files that will hopefully make the moves smarter.
I do calculated the probability of making each move when a child is created, but then I don't do anything with it. I can print it to the log file if we want.
On the topic of log files - they are the same format as the printed comments in the original sampler, but looking at the torsion file is painful because most lines don't fit on a computer screen so I will likely re-think some of that structure

Note - (mostly to myself) I need to add more tests for environment objects for the new methods, planning to do that tomorrow.

I'll review relatively soon. I'm not sure this is highest priority though so maybe we can talk about that very briefly today. Thanks.

Well, I removed the WIP because I thought it was ready to merge, but it looks like tests are failing because there is no module called "smarty"?
I'll try to look into it, but if any of you know why that might be happening please let me know

It looks like travis is getting stuck "fetching packages". I don't know why this happens, but I've seen it before. @jchodera , do you know why this happens and what we can do about it?

Thanks David, the good news is the tests are passing on the computer so I don't think I've broken anything so this should be good once we address that issue.

@bannanc -- @jchodera just fixed the tests in a separate PR. (Thanks, John!)

Thanks @davidlmobley I had a small python3 compatibility I needed to change, but otherwise this seems to be working well.
The sampler found all the AlkEtOH bonds in about 600 steps, here . I'm still playing around with odds for angles and torsions, which may end up including built in odds within sampler_smirky.py
Also, for @camizanette , in order to get the commandline tool working, I made some changes to setup.py and moved cli.py to cli_smarty.py, but didn't actually change anything inside cli.py so it shouldn't affect anything you're working on, but the script names on your branch may need to be changed to match the two tool format.

@davidlmobley is there something else you think this needs before it can be merged?

Well, someone should probably review it since it's a rather major change -- I was going to as soon as I get past the deadlines. Or you could recruit someone else.
Also you should update to master.

It probably makes sense to wait and have you review when you have time, with Camila out of town and Chris leaving I don't think there's a good alternative. I'll pull from master again.

@bannanc - I'm going to review now. But can you also bring this up to date with master?

First place I went, @bannanc , was to your smirky custom_Bond example, which does not run for me. I get an error like:
Attempting to parse file 'atom_OR_bases.smarts'
Attempting to parse file 'atom_OR_bases.smarts'
Traceback (most recent call last):
  File "/Users/dmobley/anaconda/bin/smirky", line 9, in <module>
    load_entry_point('smarty==0.1.2', 'console_scripts', 'smirky')()
  File "/Users/dmobley/anaconda/lib/python2.7/site-packages/smarty-0.1.2-py2.7.egg/smarty/cli_smirky.py", line 138, in main
    atom_OR_bases = smarty.utils.parse_odds_file(option.atom_OR_bases, verbose)
  File "/Users/dmobley/anaconda/lib/python2.7/site-packages/smarty-0.1.2-py2.7.egg/smarty/utils.py", line 226, in parse_odds_file
    for l in input_lines:
UnboundLocalError: local variable 'input_lines' referenced before assignment

It looks like some sort of issue with exception handling -- it seems like it's not finding the file .
I think it's a typo in the relevant function in utils.py, where it has input_ddlines = ifs.readlines() where it should be input_lines = ifs.readlines(). UPDATE yes, fixing that makes it run.
So, one small bug to fix so far.

I also get an error when it finishes, @bannanc :
  File "/Users/dmobley/anaconda/bin/smirky", line 9, in <module>
    load_entry_point('smarty==0.1.2', 'console_scripts', 'smirky')()
  File "/Users/dmobley/anaconda/lib/python2.7/site-packages/smarty-0.1.2-py2.7.egg/smarty/cli_smirky.py", line 177, in main
    if verbose: print("Final score was %.3f%%" % frac_found*100.)
TypeError: can't multiply sequence by non-int of type 'float'

@davidlmobley I'll fix the bug in utils and take a look at the second error

@davidlmobley I tried pulling from master, it said this is up to date, let me know if you think that is wrong for some reason.
I fixed the input_ddlines bug in smarty/utils.py, I'm surprised that didn't come up earlier for me, but it's fixed now.

File "/Users/dmobley/anaconda/bin/smirky", line 9, in
load_entry_point('smarty==0.1.2', 'console_scripts', 'smirky')()
File "/Users/dmobley/anaconda/lib/python2.7/site-packages/smarty-0.1.2-py2.7.egg/smarty/cli_smirky.py", line 177, in main
if verbose: print("Final score was %.3f%%" % frac_found*100.)
TypeError: can't multiply sequence by non-int of type 'float'
I cannot figure out what is going on with this error, the frac_found should be a float... always.
The relevant method is write_type_matches in sampler_smirky.py. I will try to replicate the error.

@bannanc - I don't think you'd pushed (or maybe committed?) since the last time you updated from master, as your last commit brought in a bunch of changes with it.

I had it print the frac_found in the cli_smirky.py script to see if I could figure out the problem.
0.577833125778
that was printed, but then I asked for the type of frac_found and got
<type 'float'>

Typo above?

I think I fixed it I changed this line:
if verbose: print("Final score was %.3f%%" % frac_found*100.)
to
if verbose: print("Final score was %.3f %%" % (frac_found*100.0))

@bannanc - that works for me now, but there still seems to be a print of the type left in your code that should be removed.
Your Python2 test is also currently failing, but I don't see a good reason for it and the Python 3 tests are passing, so it seems possible it's some sort of sporadic failure. We'll see what happens with the next commit.

I removed print statements, I'm running the tests on my computer to see if the python2 passes there.

The things failing here say "no module openeye.oechem" do I need to import openeye first for the Travis tool to work for some reason?

@bannanc -- it looks like the first error in your Travis log for python 2 is this:
  Could not find a version that satisfies the requirement OpenEye-toolkits-python2.7-ubuntu-12.04-x64==2016.10b3 (from openeye-toolkits) (from versions: 2016.10b1, 2016.10b2)
No matching distribution found for OpenEye-toolkits-python2.7-ubuntu-12.04-x64==2016.10b3 (from openeye-toolkits)

It looks to me like they just came out with a new OpenEye beta version (10b3) which is not yet available for the python 2.7 ubuntu12 platform, so Travis is failing to install the openeye toolkits which results in everything else failing.
This is OK, actually. We can merge your PR when it is ready to merge (assuming python 2.7 passes locally) because:
a) it's a problem which should go away as soon as the 10b3 is up for ubuntu 12
b) it works locally
c) the other python versions pass
etc.

@bannanc : 👍  on all the comments you just made. Sounds like you're going to be pretty close (once you make some updates to the overall README.md) to having a merge-able version.

@davidlmobley I think this is updated, I added the help documentation from smirky to the primary README file. I ran the tests again locally to double check and they are still passing on python2.7
The smarty part of the README seems a little out of date, it includes decorators I don't think we're using any more, Camila would be more qualified to comment on that, maybe she can update it when she gets her next pull request ready.

This looks fine to me, @bannanc . The one thing I'd probably add (but it's not necessary at this point if you're in a rush to get this off your plate) is to explain in the README how the scoring is done, and that it is the analog of SMARTY but extended to these other "types" or environments.
There's also a minor typo on one of the new lines the README (just a jumbled up word).
I'm stepping out for a minute, but ping me after you see this, when the tests pass, or at least when the Python 3 tests pass, and I can merge.

slight updates on the README.


Empty Email

received emails with changed ip are empty, there is no new ip.

I just set everything up on a fresh virtual machine, using gmail as the sending service, and was successful in receiving a populated email from the system.
I went ahead and fleshed out the installation guide in README.md a bit more.  Try following it and see if it resolves your issue.


Add page for sponsors

@infernocloud - is the goal to have a separate /sponsors url path with this list?

Callcap
Signlab
Terracon Foundation

If you have logos of these sponsors please send it my way, else I will look it up from their website.

Hey! The current "Who we are" page lists these sponsors for right now. https://www.lakeafton.com/who-we-are/ You could probably grab the logos from their linked websites and work them into that page for a quick update.
When adding this a year ago I think we expected the Observatory board would more actively look for new sponsors and so we would want to display some info on existing sponsors and information on new sponsors joining. I'm not sure what the board of directors thinks about new sponsors anymore. @DJSethDuncan thoughts on this one?

Yeah I don't think we're gonna go with a single page for sponsors at the moment. If we outgrow the Who we Are page, we'll reconsider. Thanks!


New version of feathers-vuex.

Wondering if you are interested in updating your repo with the new version of feathers-vuex?

This is now done!  https://github.com/silvestreh/feathers-nuxt/pull/26/files


Undefined method `where' for Mongoid::Userstamp::User:Module

After adding the gem, everytime I call my user model
User.where(...

I get the this namespace error. Quick to fix is to call
::User.where(...

But how can we solve that globally ?

I have fixed this issue. Take a look https://github.com/atherplero/mongoid_userstamp


basename for use on GitHub Pages

Hi, I love the idea behind Redux Little Router, I have a little problem getting it to work with GitHub Pages. I am using Create React App and its deploy script to publish to GitHub pages. My app lives at  https://user.github.io/Repo so I set basename: Repo in the hopes that would make the app work, but somehow that didn't solve the issue. On my lokal development server, the app still starts at http://localhost:3000 instead of http://localhost:3000/Repo, Once I click a link it works correctly, navigating to http://localhost:3000/Repo/route and the pathname in the state is page. But when I reload the page on http://localhost:3000/Repo/route the pathname in the state becomes Repo/page instead of just page and hence nothing is matched.
Am I misunderstanding how basename is supposed to work, or is this a bug?

I realized that GitHub Pages has some general problems with browserHistory, but even (this workaround)[https://github.com/rafrex/spa-github-pages] had no effect. I have the feeling that there is a bug in the implementation in how URLs are parsed, that it doesn't take care of the basename.

Having a similar issue. I was able to reproduce it on the demo app.
I just added basename: '/something' in demo/client/app.js. i.e.:
const {
  routerEnhancer,
  routerMiddleware
} = routerForBrowser({ routes, basename: '/something' });

Notice how clicking any of the links works fine, but if I refresh the page (or go directly to http://localhost:4567/something/cat), none of the routes match anymore.
Any clues on what I may be doing wrong or is this a bug?

This is a bug in how the state is initialized. If you look at https://github.com/FormidableLabs/redux-little-router/blob/master/src/link.js you'll notice that it manipulates basename for the location that ultimately gets pushed. However https://github.com/FormidableLabs/redux-little-router/blob/master/src/store-enhancer.js initialization of the location doesn't take basename into account.

Should be fixed in v13.0.0! Ping here if not 😬


Incompatible with Dangerclose on the latest 1.19.2 forge ver of DangerClose

Incompatible with Dangerclose on the latest 1.19.2 forge ver of DangerClose because campfire priority when liting entities.
Still crashes when disabling the feature in forgery confing.
https://gist.github.com/SoimuBadass/b20f5344215f2241ab2678ff43f4111a

features still load when disabled you have to ban them or have limit runtime configs on.
I just tested it and dangerclose and forgery aren't in conflict, also at a quick glance dangerclose really dosn't do anything that would cause this.
this crash really should never happen the feature should just get automaticly disabled.
please narrow down what mod forgery is conflicting with. i cannot reasonably download and debug 170 mods

please re-open if you findout what's causing it


Failing to manage multiple profiles

Describe the bug
AwsSession fails to manage multiple profiles for QPU-related tasks. It only works if I move the credentials to the default profile.
QPUs = AwsDevice.get_devices(types=["QPU"], aws_session=aws_session) is failing but simulators = AwsDevice.get_devices(types=["SIMULATOR"], aws_session=aws_session) is working. We created aws_session using the instruction provided in the docs . The only workaround is to move the credentials to default profile.
Error message:
ClientError: An error occurred (UnrecognizedClientException) when calling the SearchDevices operation: The security token included in the request is invalid

Hi, thanks for documenting this issue!  If we can see how you're setting up the credentials and setting up the sessions, that will help with debugging. Please redact credentials of course

Yes, the setup looks good, just wanted to make sure I'm debugging the same scenario you're facing. Getting QPUs searches over multiple regions and involves copying the session, whereas simulators is more straightforward, so that gives a good start to where this edge case is cropping up. I'll take a look and see if I can get this fixed quickly. Thanks for sharing!

My profile is setup for only us-east-1. Its weird when it works fine for the default profile.
Thank you!

Thanks for identifying this bug! Here is a PR with my fix: https://github.com/aws/amazon-braket-sdk-python/pull/446. It is not ready to merge as I need to update some of the adjacent unit tests, but feel free to install from source to unblock your development in the meantime.

@ajberdy its working! Thank you very much!


take merge views from corsix branch

This makes Winograd twice as fast (to compile) on M3

@corsix I remember you mentioned having a better merge views, I found it
This is the fix I need for Winograd, but it seems like it breaks something else.

This should also be able to share a lot of code with reshape I think. The reshape thing is just a specialized version of this where the second view has to be contiguous.

FWIW, taking out the tinygrad/shape/view.py change seems to put it in a better place (said change was only there because merge_views was very occasionally too good in synthetic tests, and that View.create change was an easy way to make the competitor equally good).

Too good? I didn't really understand that change, is there a bug with/without it?

I didn't really understand that change

There's long-standing code in View.create to canonicalise strides: if any dimension has size 1, then its stride can be canonicalised to 0. The change added another case of canonicalisation: if any dimension has size >1, but is masked such that only one index in the dimension is unmasked (e.g. a dimension with range from 0 through 10, but with <4 and >4 all masked off, leaving 4 as the only unmasked index), then its stride can also be set to 0, albeit with a corresponding adjustment required to the offset (e.g. continuing the example, if said dimension had stride X, we'd need to add 4*X to the offset).

Too good?

The context was a unit test that compared the following:

A: Start with some shape X, applied a sequence of elementary transforms to it to create Y, then applied another sequence of elementary transforms to create Z.
B: Start with shapeof(Y), apply the 2nd sequence of elementary transforms to that, then do merge_views between Y and that.

The expectation was that (B) would create something complicated, then the merge_views would simplify it back down, ideally getting as simple as (A). In practice, it sometimes created something simpler than (A), because (B) naturally canonicalised away masked-to-single-element dimensions as a consequence of removing masked elements then canonicalising then re-applying padding. In those cases, to get the results of (A) and (B) to be identical, (A) also needed to canonicalise away masked-to-single-element dimensions, which is the view.py change.

is there a bug with/without it?

It slightly increased the number of cases where (A) gave identical result to (B). At least in the tests I was looking at, in all cases (A) and (B) gave equivalent results, just not identical results. That said, there is clearly some bug with it based on the earlier test failures in this PR.

Is this merge_views complete?

It isn't sufficient to make the aforementioned unit test always give identical results for (A) and (B); testing showed various cases where either (A) was slightly simpler, or (B) was slightly simpler, or (A) and (B) were equally complex but decided to express the complexity in slightly different ways. Is there some weaker sense in which it is complete? Perhaps.

Huh, so the test_shrink_const_then_cast is fixed it seems by the canonicalization.

I have a different fix for this in #3277

Ah, I see you identified the bug as the handling of symbolics, as comparison operators on symbolics end up false-y if the comparison is always false, but end up true-y if the comparison is potentially true. Both new calls to any in view.py are affected; one fix as you've done is instanceof checks (though the 2nd any also needs them), the other (more subtle) approach is to invert the direction of the condition and wrap it in a not (e.g. b+1 >= e -> not (b+1 < e)).

Ahh yea, the second any probably caused the issue with llama on master.
@chenyuxyz perhaps symbolic should assert on those comparisons?

FWIW, this behaviour of symbolics is why the revised merge_views has things like if not (merged_term >= merged_size) and not (merged_term < 0):, rather than the more obvious way of writing that without nots.

@corsix @geohot

comparison operators on symbolics end up false-y if the comparison is always false, but end up true-y if the comparison is potentially true.

Comparison operation on symbolics will return True only if the condition is true for all values of the symbolic between min and max. Analogously, return False if false for all.
The problem occurs when for some values between min and max, the condition is true and for others it is false. In this case, the comparison operation will return LtNode instead of a bool; but since you have put the condition in an if-statement, python will try to convert the LtNode into a bool by calling LtNode.__bool__(). But guess what, a LtNode will always get converted to True.
I have a more readable solution for this problem: explicitly checking if the comparison returned True or it was LtNode or False. So, you can replace pattern if x < y with if (x < y) is True and pattern if not x < y with if (x < y) is False.
see also: https://github.com/tinygrad/tinygrad/issues/2469#issuecomment-1867856487 and https://github.com/tinygrad/tinygrad/issues/1924#issuecomment-1742153891 for original.
To catch these kind of bugs, why not just put an always failing assert in LtNode.__bool__()?
The assertions in this patch does not fail when running test/test_symbolic_ops.py and test/test_symbolic_jit.py.
diff --git a/tinygrad/shape/view.py b/tinygrad/shape/view.py
index acdd4b9b..3bb0a2f7 100644
--- a/tinygrad/shape/view.py
+++ b/tinygrad/shape/view.py
@@ -85,6 +85,16 @@ class View:
     contiguous = offset == 0 and mask is None and strides == strides_for_shape(shape)
     # if any dimension has size >1, but is masked such that only one index in the dimension is unmasked
     # then its stride can also be set to 0, albeit with a corresponding adjustment required to the offset
+
+    if mask:
+      explicit = [(b+1 >= e) is True for b,e in mask]
+      flip = [not (b+1 < e) for b,e in mask]
+      assert explicit == flip
+
+      explicit = [(b >= e) is True for b,e in mask]
+      flip = [not (b < e) for b,e in mask]
+      assert explicit == flip
+
     if mask and any(elim := [not (b+1 < e) for b,e in mask]):
       if any(not (b < e) for b,e in mask):
         strides, offset, mask = (0,) * len(shape), 0, ((0,0),) * len(shape)


Update command.rst to include newly proposed CLI options

Add upgrade group sub-commands
Update help message of plan and upgrade

Example output for the updates in help message:
Usage: cou plan hypervisors [options]

Show the steps for upgrading nova-compute machines.
This is possible only if control-plane has been fully upgraded,
otherwise an error will be thrown.

Note that only principal applications colocated with nova-compute units
that support action-managed upgrades are within the scope of this command.
Other principal applications (e.g. ceph-osd) and subordinates
can be upgraded via the data-plane subcommand.

Options:
  -h, --help            Show this help message and exit.
  --model MODEL_NAME    Set the model to operate on.
                        If not set, the currently active Juju model will be used.
  --backup, --no-backup
                        Include database backup step before cloud upgrade.
                        Default to enabling database backup.
  --force               Force the plan/upgrade of non-empty hypervisors.
  --verbose, -v         Increase logging verbosity in STDOUT.
                        Multiple 'v's yield progressively more detail (up to 4).
                        Note that by default the logfile will include standard logs
                        from juju and websockets, as well as debug logs from all other
                        modules. To also include the debug level logs from juju and
                        websockets modules, use the maximum verbosity.
  --quiet, -q           Disable output in STDOUT.
  --machine MACHINES, -m MACHINES
                        Specify machine id(s) to upgrade.
                        This option accepts a single machine id as well as a stringified
                        comma-separated list of ids, and can be repeated multiple times.
                        This option cannot be used together with [--availability-zone/--az].
  --availability-zone AVAILABILITY_ZONES, --az AVAILABILITY_ZONES
                        Specify availability zone(s) to upgrade.
                        This option accepts a single availability zone as well as a
                        stringified comma-separated list of AZs, and can be repeated
                        multiple times. This option cannot be used together with
                        [--machine/-m]


Random - Should this be zero-indexed?

Describe the bug
The 'Random' function is not zero indexed.
To Reproduce
Use Message($"{Random[1.0]}");
Expected behavior
If our arrays are zero-indexed, the returned index should (intuitively) be zero-indexed.
Screenshots
System information

OS: Windows 10
Version of affected NuGet or PyPI package or packages - Not sure

Additional context

Hold up - this is more complicated than I expected. Run this method using the resource simulator:
           Message($"{Random([1.0, 2.0])}");
I'm getting 0, 2 as my values?!

With the following code:
           Message($"{Random([1.0, 2.0, 2.0])}");
I'm getting 0, 1, 3 - I'm guessing that the final element is erroneously increased by 1?
Additionally, this bug only happens on the ResourceEstimator - it's fine using the QuantumSimulator.

Hm, this is very odd indeed. Thanks for reporting, we'll go on and investigate.

I think I may have found the issue; it looks like the resources estimator uses a different implementation of Random than other simulators. That said, investigating it looks like this it is a qsharp-runtime issue so I'll transfer there. Thanks for the report!


Garbled screen on NVIDIA GeForce GT 630

          It filled up successfully, just got the Garbled screen error.

https://bsd-hardware.info/?id=pci:10de-0f00-1462-8a90
Originally posted by @louies0623 in https://github.com/helloSystem/ISO/issues/523#issuecomment-1712517143
Describe the bug
#524 (comment)
Same bug
Version (please complete the following information):
Live build 0I42

Does it work properly on NomadBSD?

Yes, it can perform, it's just very choppy and laggy.

Hello system
2  0xffffffff83200000  db7e30  nvidia.ko
1  0xffffffff8310b000  e6dd0  nvidia-modeset.ko
nomadbsd
2  0xffffffff82c00000  db7dc0  nvidia.ko
1  0xffffffff839b8000  e6d40  nvidia-modeset.ko

@probonopd What do you think about temporarily using Nouveau as the driver?

Nouveau is Linux-only, and even there it never has worked as well as the official Nvidia driver for me.

Can we use nomadbsd driver on the system?

I think we are using the same?
If you can find the differences, I can try to make them the same.

Ghost bsd /usr/local/etc/X11/xorg.conf.d/00-video-initgfx.conf There is no file.
Nomadbsd  :

I have a guess that our system may have problems processing nvidia graphics cards, but I don’t have a newer graphics card to verify.
I only have a MacBook with Nvidia as the GPU for testing. This is the information of that laptop.
hello system

nomadbsd

So, looks pretty identical to me. Can you find out what exactly we need to change?

I only know temporarily that this is not a driver problem.
https://github.com/helloSystem/ISO/issues/542#issuecomment-1738249445

@probonopd We can celebrate now, we finally found that we need to add linux_enable="YES" to our /boot/loader.conf and it will be fixed.

With the changes, what does /usr/local/etc/X11/xorg.conf.d/00-video-initgfx.conf say?

When the graphics card driver has not been installed yet.
https://bsd-hardware.info/?probe=5d1dbf86d9

After install nvidia-driver-390
and add linux_enable="YES" to /boot/loader.conf
https://bsd-hardware.info/?probe=e587bca33a

I recently learned how to install HW probe and USB tethering on NomadBSD. This is the NomadBSD Hardware Probe https://bsd-hardware.info/?probe=8816b1ac4a

Update: In fact, it can be used without Linux enable =yes


Is it possible to manually raise the video bitrate?

Hi! Thank you for providing the amazing SDK and I'm grateful for your kind support!!
I have some questions about manipulating video bitrate.
It's sort of picky (and apologize for that), but I need to handle live streams held with an unstable internet connection. While setting up the IVS broadcast session, I configured the initial bitrate, max bitrate, target fps, and size for the video setting.
IVSBroadcastSession.applicationAudioSessionStrategy = .playAndRecord
let config = IVSBroadcastConfiguration()
try config.audio.setBitrate(128_000)
try config.video.setInitialBitrate(4_000_000)
try config.video.setMaxBitrate(8_500_000)
try config.video.setTargetFramerate(30)
try config.video.setSize(CGSize(width: 1920, height: 1080))

To handle the situation like a user stream with low bandwidth, I didn't set the min bitrate, so the live stream won't get disconnected even if the user streams with a temporally unstable internet connection. (since I didn't set the min, 300_000 is applied by default min bitrate). Since the broadcast SDK supports automatic bitrate adjustments, I manually change the network bandwidth from iPhone's developer setting and streamed for about 90 min, as below. However, after I lower the bandwidth and the video bitrate gets low, and then revert it to the original bandwidth (turn off the network conditioner), it takes a while to get back to high video bitrate.

Start with disabling the network link conditioner and streamed for around 30 min -> the bitrate was around 4,000,000 to 8,500,000, which is great
Turned on the 2Mbps network link from the iPhone's developer setting for around 5 min -> the bitrate got low to around 1,00,000, which is also fine
turned off the network link conditioner -> the video bitrate is steady with around 1,000,000 to 2,000,000 for around 30 min, then back to a higher bitrate. It's good that the bitrate recovers, but I want it to happen a little bit more quickly. (maybe like within 5 min or something?)

So I want to confirm things below

Even after the network condition gets better, the automatic bitrate adjustments take a while to recover the bitrate?
Is this an expected behavior?
Is there a way to raise the bitrate a little bit more quickly?
While the video bitrate is low, I want to tell the user something like "the streaming video might get pixelated due to the lower bitrate". Is there a way to check the current video bitrate out? I checked the IVS broadcast document and I think broadcastQualityChanged can be used, but I saw it returns one of the following values, 0.0, 0.25, 0.5, 0.75 while I was testing. With low video bitrate, like around 1,000,000 (I think) it takes a while to get 0.25+ after getting 0.0. So it would be better, if I can find out the current video bitrate or the way to raise the bitrate if the network connection gets recovered.

Again, so sorry for the picky question, and thank you so much for the support!!
(I hope all users use wired connections while broadcast... 😅)

@yuya-h-29 hello, thank you for the detailed issue. I have good news!
I'll answer your 4 questions for v1.5.2 (the latest available today), and then again for an upcoming release.
For 1.5.2

It takes a while to recover, the bitrate increase is gradual to prevent congestion and starvation.
Yes it is expected.
There is not a way to increase the recovery rate.
That API (broadcastQualityChanged) is the best way to display a message like that today.

For Upcoming Release

It will still be expected that recovery takes time, we do not do bandwidth estimation to make large bitrate jumps, only gradual increases.
Same answer as 1.5.2
There will be a new API that controls bitrate recovery aggressiveness. Today it takes ~4.5 minutes to double the bitrate after dropping due to low bandwidth, with the new API it can be as low as 36 seconds if the network is strong. So recovery can be ~8x faster.
A new API will be introduced that allows monitoring the outgoing bitrate directly. More details will be in the release notes and updated API reference documentation.

I can not confirm a release date for the upcoming release, but keep an eye on our Release Notes page for details. I will try my best to remember to comment on this issue again when we release.

@bclymer
Thank you so much for the quick response and detailed explanation, Brian!!
Looking for the board cast SDK get updated soon :)

Hey @yuya-h-29 good news, 1.7.0 has been released today with the new APIs I was talking about.
Here is autoBitrateProfile, and here is transmissionStatisticsChanged:. I hope those help out your use case!
You can view the full release notes here.


Test Case update for public jira APIMANAGER-3381

Please review and merge the pr for public jira
APIMANAGER-3381

This PR can't be merged as it's outdated. Please update accordingly. Thanks.


removed leftover asset files

some precompiled asset files were left over and can be removed

LGTM :stuck_out_tongue:


The Red Hand Files failed with error 0

Just noticed that this is failing and no one else has reported it.
HttpException: cURL error HTTP/2 stream 0 was not closed cleanly: PROTOCOL_ERROR (err 1): 92 (https://curl.haxx.se/libcurl/c/libcurl-errors.html) for https://www.theredhandfiles.com in lib/contents.php line 308

index.php(7): RssBridge->main()
lib/RssBridge.php(15): RssBridge->run()
lib/RssBridge.php(92): DisplayAction->execute()
actions/DisplayAction.php(133): TheRedHandFilesBridge->collectData()
bridges/TheRedHandFilesBridge.php(13): getSimpleHTMLDOM()
lib/contents.php(368): getContents()
lib/contents.php(147): _http_request()
lib/contents.php(308)

Query string: action=display&bridge=TheRedHandFilesBridge&format=Atom
Version: dev.2023-03-22 (git.master.6e1048c)
Os: Linux
PHP version: 7.4.33

reminds me of https://github.com/RSS-Bridge/rss-bridge/pull/3249
@Mynacol

PHP 7.4.33 (cli) (built: Feb 22 2023 20:07:47) ( NTS ) Copyright (c) The PHP Group Zend Engine v3.4.0, Copyright (c) Zend Technologies     with Zend OPcache v7.4.33, Copyright (c), by Zend Technologies
Using:
php -r 'foreach (get_loaded_extensions() as $extension) echo "$extension: " . phpversion($extension) . "\n";'|grep curl
I get
curl: 7.4.33

I think I am well overdue to upgrade to php8.x as was as libcurl. I will try that and close the ticket with a note detailing if that worked.

looks like switching to php8.2 plus a current libcurl was enough to fix this.


DOCUMENTATION links are wrong

Links to documentation on e.g. this page:
https://github.com/rancher/rancher/blob/release/v2.8/README.md
are pointing to e.g. https://rancher.com/docs/rancher/v2.8/en/installation/
whereas I was only able to find the real documentation under
https://ranchermanager.docs.rancher.com/v2.8/pages-for-subheaders/installation-and-upgrade
Not sure if the links must be fixed or the documentation path of the real pages.

Not sure why this was transferred, as it pertains to a page in rancher/rancher...

Imo this issue shouldn't have been transferred since it relates to the rancher/rancher readme having a broken link


Joining Repetion of Rule into One

Good day,
First of all, thank you for a great library,
I'm trying to parse a C++ string splitted into multiple lines ex:
return "First"
"Second"with middle quote"
"Third";
Is it possible you parse this to get "FirstSecond"with middle quoteThird"
Thank  you very much,
Nelson

Multi-part C/C++ string literals like this are relatively easy to parse, there is also an unescape action for C-strings that can be used. What exactly does your input look like, do you need to parse the strings in arbitrary C++ source?
PS: It appears that you forgot a backslash before one of the quotation marks, it should read "Second\"with middle quote".

You're right about the missing slash, the input looks like I've posted, nothing too fance, if I can get the parser to apply escape substitution would be great, otherwise I' can do after.
Right now I've come up with, which parses, but gives me multiple lines in the parse tree, I would like to join them and use substitute the escaped characters
struct Semicolon : one<';'>
{};
struct Quote : one<'"'>
{};
struct EscapedQuote : seq<one<'\'>, Quote>
{};
struct Return : TAO_PEGTL_KEYWORD( "return" )
{};
struct String : star<not_at, sor<EscapedQuote, any>>
{};
struct Strings : if_must<Return, star<not_at, pad<seq<Quote, String, Quote>, space>>, Semicolon>
{};

There are multiple ways to achieve what you are looking for, but I can't decide which one is best for you. The fundamental problem, however, is that by default the PEGTL actions as well as the parse tree nodes only reference portions of the input. If the input contains foo\"bar, we can not reference the string foo"bar. You'll have to assemble it yourself or use a convenience helper.
That said, you mentioned that you are working with a parse tree. So here's a small example I wrote:
#include <iostream>
#include <string>

#include <tao/pegtl.hpp>
#include <tao/pegtl/contrib/parse_tree.hpp>
#include <tao/pegtl/contrib/parse_tree_to_dot.hpp>

using namespace TAO_PEGTL_NAMESPACE;

namespace example
{
   // the grammar

   // clang-format off
   struct Semicolon : one<';'> {};
   struct Quote : one<'\"'> {};
   struct EscapedQuote : seq<one<'\\'>, Quote> {};
   struct Return : TAO_PEGTL_KEYWORD( "return" ) {};

   struct String : star<not_at<Quote>, sor<EscapedQuote, any> > {};
   struct Strings : if_must<Return, star<not_at<Semicolon>, pad<seq<Quote, String, Quote>, space> >, Semicolon> {};
   // clang-format on

   template< typename Rule >
   using MySelector = tao::pegtl::parse_tree::selector<
      Rule,
      tao::pegtl::parse_tree::store_content::on< any, String >,
      tao::pegtl::parse_tree::remove_content::on< Strings, Semicolon, EscapedQuote, Return > >;

}  // namespace example

int main( int argc, char** argv )
{
   if( argc != 2 ) {
      std::cerr << "Usage: " << argv[ 0 ] << " EXPR\n"
                << "Generate a 'dot' file from expression.\n\n"
                << "Example: " << argv[ 0 ] << " \"(2*a + 3*b) / (4*n)\" | dot -Tpng -o parse_tree.png\n";
      return 1;
   }
   argv_input in( argv, 1 );
   try {
      const auto root = parse_tree::parse< example::Strings, example::MySelector >( in );
      parse_tree::print_dot( std::cout, *root );
      return 0;
   }
   catch( const parse_error& e ) {
      const auto p = e.positions.front();
      std::cerr << e.what() << std::endl
                << in.line_at( p ) << std::endl
                << std::string( p.byte_in_line, ' ' ) << '^' << std::endl;
   }
   catch( const std::exception& e ) {
      std::cerr << e.what() << std::endl;
   }

   return 1;
}

After compilation, you can call it with
my_example 'return "foo" "b\"ar";' | dot -Tsvg -o parse_tree.svg && eog parse_tree.svg

(well, on my Linux machine, anyways).
You can then walk the parse tree and collect the node content one-by-one. It's not efficient, but it should help to get you started.

The fundamental problem, however, is that by default the PEGTL actions as well as the parse tree nodes only reference portions of the input. If the input contains foo\"bar, we can not reference the string foo"bar. You'll have to assemble it yourself or use a convenience helper.

This was the piece I was missing, and it makes perfect sense, since the rules just reference the underline buffer, thus we can't remove the "Quote spaces Quote" the connects the two lines.
So I'll have to check how to use said convenience helper.
Thank you very much for you support


Refactor the stats website by using new data structures from the web app.

In the stats website code, all tokens-related data are identified/classified by the tokens' symbol. I think it makes frontend code requires frontend devs' development time when adding new token/new pair liquidity data. Using tokens' addresses as a key property is logically correct and it helps scalability for the stats website and reduces adding token/pair time.

Import constants/tokens and constants/pairs files from the web app.
Use the CurrencyLogo component from the web app instead of using the CurrencyIcon component.
Refactor all data used tokens' symbol as a key to data structure token address as a key
Remove tokens' SVG files in this repo. Instead, use the asset repo.

@hetfly has plans to refactor when doing the redesign


SCB-411 SC can not check the reduplicate endpoints when register with ID

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run go build go test go fmt go vet to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #310 into master will decrease coverage by 0.71%.
The diff coverage is 56.91%.

@@            Coverage Diff             @@
##           master     #310      +/-   ##
==========================================
- Coverage   70.33%   69.61%   -0.72%
==========================================
  Files          17       17
  Lines        3452     3443       -9
==========================================
- Hits         2428     2397      -31
- Misses        853      874      +21
- Partials      171      172       +1

Impacted Files
Coverage Δ

server/service/util/instance_util.go
47.16% <41.66%> (-1.93%)
:arrow_down:

server/service/instances.go
67.28% <58.55%> (-3.84%)
:arrow_down:

Continue to review full report at Codecov.

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

Coverage decreased (-0.6%) to 73.351% when pulling 89b62fd58a8d3f9ca22e88ea5d94b500f697faaa on little-cui:master into f8b3e7b95ef63709cfb933a3688a4714271ec518 on apache:master.


libSDL2_gfx.so: cannot open shared object file

When trying dart run example/fps_manager.dart I get the following error:
Unhandled exception:
Invalid argument(s): Failed to load dynamic library 'libSDL2_gfx.so': libSDL2_gfx.so: cannot open shared object file: No such file or directory
#0      _open (dart:ffi-patch/ffi_dynamic_library_patch.dart:12:43)
#1      new DynamicLibrary.open (dart:ffi-patch/ffi_dynamic_library_patch.dart:23:12)
#2      dylibOpen (package:sdl2/sdl2/dylib.dart:25:25)
#3      libSdl2Gfx (package:sdl2/sdl2/generated/lib_sdl_gfx.dart:8:26)
#4      libSdl2Gfx (package:sdl2/sdl2/generated/lib_sdl_gfx.dart)
#5      sdlInitFramerate (package:sdl2/sdl2/generated/lib_sdl_gfx.dart:15:42)
#6      FpsManageerPointerEx.init (package:sdl2/sdl2/ex/gfx/fps_manager.dart:9:5)
#7      main (file:///home/jake/workspace/source/dart-sdl2/example/fps_manager.dart:34:7)
#8      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:297:19)
#9      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:192:12)

My Environment
OS: Windows 11 - WSL2 Ubuntu 22.04
Dart: Dart SDK version: 2.18.6 (stable) (Unknown timestamp) on "linux_x64"

Hello, jakeklassen.
sdl2-gfx is not an official library, so it needs to be installed separately.
sudo apt-get install -y libsdl2-gfx-dev

Hello, jakeklassen.
sdl2-gfx is not an official library, so it needs to be installed separately.
sudo apt-get install -y libsdl2-gfx-dev

@sansuido ah right, thanks that worked perfectly.


Image loading placeholders could use some margins

What happened?

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@prury woho love the paintings

caveman universal issue description


Medication Schedules Website

Title and Issue number

Title : Medication Schedule Website
Issue No. : #435
Code Stack :  HTML, CSS, JS
Close #435

Video (mandatory)

https://github.com/apu52/METAVERSE/assets/71439585/015cf701-6166-46e5-b99a-af393e271040
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have updated the projectData.json file for my contribution
[x] I have added the img of my project in the assets/img section

Additional context (Mandatory )
Are you contributing under any Open-source programme?

Yes, Codepeak 2023

added a landing page with some really cool animations also


rename version to 0.1.0

there is no way to test pulling from quay - the make targets all assume the images the user wants come from the internal registry. I would like to be able to tell make to pull from quay. I'll add that to this PR.


fixed validation issues in product & seller files

The changes in the model file was to change the "nullable" attribute from "False" to "True" because it was obstructing me from successfully creating new products and it wasn't being passed as a parameter [to the "post" function. If this is not okay with the management though, I can revert the change and pass it as a parameter.
We need to look into rolling back failed commits to avoid invalidated data into the database.
And finally for the "update" functions, we seem to have supply all the information instead of just supplying what we need to update. That should be looked into as well.
Thank you.

Weldone Brother, you did an excellent job. Any changes to the database have to pass through rebuntu team. However, the info will be relayed to them to they can look into it.
"And finally for the "update" functions, we seem to have supply all the information instead of just supplying what we need to update. That should be looked into as well." That's a good observation, you can go ahead and correct that


Add jitsi integration to Gather

It would be awesome if we could enable some kind of community-wide video and phone conferencing into Gather. Would it be possible to enable something with jitsi? Then we could post community meeting events on a common calendar.

I hear Jitsi doesn't work so great with more than 8-10 people. My current recommendation is a Zoom subscription and then you can add the personal meeting room as a resource in Gather. I do have a plan to add a URL field in the resource form so you can add a URL for virtual resources.

I'm fine with closing this as the functionality will be (mostly) covered by #742. I heard jitsi can handle up to 35 people, but I haven't tried it. I despise Zoom, and hate the idea of the community having to pay them money.

Well let me know what kind of luck you have with Jitsi or another
alternative. I'm not against it, I have just heard about the Jitsi
scalability issue from many people.
On Sun, May 24, 2020 at 11:18 AM Leila Tite<EMAIL_ADDRESS>wrote:

I'm fine with closing this as the functionality will be (mostly) covered
by #742 https://github.com/gather-community/gather/issues/742. I heard
jitsi can handle up to 35 people, but I haven't tried it. I despise Zoom,
and hate the idea of the community having to pay them money.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gather-community/gather/issues/745#issuecomment-633246475,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFLO4ASQTWIUYBA73MJLM3RTE3E7ANCNFSM4NIP7UWQ
.


Invalid code generation for anonymous enums/namespace

For anonymous enums, Chimera generates invalid code such as dart::common::ClassName::(anonymous)::EnumValue. Instead, it would be nice to print a warning and skip the code generation for the anonymous enums.

This also happens for an anonymous namespace. The generated code looks like dart::math::(anonymous namespace)::MyClass::foo().

Anonymous enum is now ignored by #121 as a workaround. A better solution would be defining the elements of an anonymous enum in its parent scope with valid binding code.


DueDate blank values - View Formatting

Important
Use the following form to submit an issue only if it's related to samples in this repo. If you have an issue related to the SharePoint Framework or its documentation, please submit the issue at https://github.com/SharePoint/sp-dev-docs/issues/new. This will help us respond to your issue faster.

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you.
Category

[* ] Question
[ *] Bug
[ ] Enhancement

Expected or Desired Behavior
Is there a way to use View Formatting with blank values.  I tried [$DueDate]==''but no success.
Steps to Reproduce
Applied this json to a list where there are empty values in the DueDate column
{
  "schema": "https://developer.microsoft.com/json-schemas/sp/view-formatting.schema.json",
   "additionalRowClass": "=if([$DueDate] <= @now, 'sp-field-severity--severeWarning', ''"
}

Thanks for your contribution! Sharing is caring.

@sympmarc Very creative!! 👍
@sohailmerchant This also works for me...does it work for you? :)
{
  "schema": "https://developer.microsoft.com/json-schemas/sp/view-formatting.schema.json",
   "additionalRowClass": "=if(toString([$DueDate]) == '', 'sp-field-severity--severeWarning', ''"
}

Looks like this:

Is that what you are looking for?

toString - brilliant!
{
  "schema": "https://developer.microsoft.com/json-schemas/sp/view-formatting.schema.json",
  "additionalRowClass": "=if(toString([$DueDate]) == '', '', if([$DueDate] <= @now, 'sp-field-severity--severeWarning', if([$DueDate] > @now, 'sp-field-severity--good', '' )))"
}

@PopWarner and @sympmarc  Thank you very much this work.  I think it is worth documenting this especially on the sample provided.  Happy to do a PR.

@sohailmerchant Always worth providing the PR to offer an update for more clarity. :)
Glad all worked out for ya. Should we go ahead and close this issue as fixed? :)

@lordneeko Same as before. We'd love to help you, but I'm not entirely sure what you are asking. This is also a closed issue. Please open a new issue with a few more details.
Specifically, what you are trying to do, the format you are using (the code above), and what happens currently. Since this doesn't appear to be about one of the samples in this repository, please open your issue on https://aka.ms/spdev-issues
You can always reference an existing issue by copying and pasting a link to it. Feel free to tag me in the new issue as well, thanks!


Fix list alignment

Hi @bevacqua,
Thanks for your open source work (horsey and insignia, in particular)!
This PR removes the padding from .sey-list, fixing the alignment with the field
Before:

After:

Published as 1.0.5

Thanks!


Handle drag-and-drop from Chrome download bar

Users may want to drag from non-Finder/Explorer sources, such as their browser's download bar.

I'm gonna merge this manually since the change is only two lines and this merge affects a lot of files.
Thanks for the fix and figuring this out.

No problem, thanks for a great library. :)


Potentially unsafe get/set usage warning

I've got this strange warning:

[flow] [flow] Potentially unsafe get/set usage (Getters and setters with side effects are potentially >unsafe and disabled by default. You may opt-in to using them anyway by putting >unsafe.enable_getters_and_setters=true into the [options] section of your .flowconfig.)

Can you clarify what kind of side-effects my plain setter/getter cause?
// @flow
class Git {

  constructor(path: string) {
    this.gitBinary = 'git';
    this._path = path;
  }

  get path(): string {
    return this._path;
  }

  set path(path: string) {
    throw new Error('Path is readonly.');
  }

  get gitBinary(): string {
    return this._gitBinary;
  }

  set gitBinary(value: string) {
    this._gitBinary = value;
  }

}

Well, your example is entirely invalid, because you try to set this.path, but don't have a setter, and in getter you cause infinite recursion.

@vkurchatkin yeah right. So I back my properties with some private variable, still get the same warning.

It does not really matter how I define properties, even simple getter that returns constant still emits warning.
class Git {

  static get master() {
    return 'master';
  }

}

I'm not 100% sure, but I think Flow always warns about this, because getter or setter with side effects can potentially invalidate type refinements, but Flow doesn't actually check if your getter/setter has side effects

So Flow considers all setters/getters as an error regardless of their implementation? That's pretty frustrating.

If you prefer to use them without the warnings, you can opt to enable the feature using the configuration option listed in the warning message: unsafe.enable_getters_and_setters=true

It would be nice to disable the warning on a per getter/setter basis.  That way, a human could evaluate whether or not a particular getter/setter had side-effects as opposed to blanket acceptance.

This seems like a pretty clear bug in Flow... Any plans on getting getters/setters working in Flow without having to pass in unsafe flags? It seems that is a bad idea since flow could actually catch issues rather than just completely ignoring getters/setters...

@jdalegonzalez I think you can use // $FlowFixMe right above the getter to suppress the error

Unfortunately // $FlowFixMe disables all checks vs. enabling only unsafe.enable_getters_and_setters for that line as an interim workaround until Flow can understand side effects.

Is there any movement on this? From my perspective, this is a pretty big bug in that Flow essentially doesn't work for setters and getters.


make test_psm3.py robust to API overuse errors

[x] Closes #870
[x] I am familiar with the contributing guidelines
[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

test_psm3.py has been dealing with an random failures due to a dependency on a general API key from NREL's Development Network. This PR implements two features to deal with this.

Implement a pvlib-python specific NREL API key for testing.
Calling get_psm3() inside a try/except statement that repeats the call if an "OVER_RATE_LIMIT" error occurs.

For 1, Implementing the NREL API key requires an admin of our Azure Pipelines to create the appropriate variable for this to work. The azure-pipelines.yml exports this value into the environment before pytest is run. There is a fallback to the general API code of "DEMO_KEY" that allows for tests to still be run locally. This only alleviates part of the problem and doesn't fix NREL's protections that deny multiple API calls from the same IP address.
Feature 2 works to repeat requests that return an HTTPError caused by an "OVER_RATE_LIMIT" and increases the sleep time between calls in the case of multiple errors. This should allow even local testing without a registered NREL API key to avoid these errors although they may be more susceptible to these tests taking longer.
The only thing I can think of that might cause an error from the API after implementing these strategies would be if there were more than 1000 calls by it within a day. I don't believe we need to accommodate for that possibility now but I just want to highlight it for future reference.

Left some suggestions for marking tests with @pytest.mark.remote_data from pytest-remotedata in reference to #878. This disables the tests unless including --remote-data from the command line when starting the tests.

Thanks for pointing this out to me. This addresses #878 more than #870 to be clear but I'll try to include it.

There's good news is that I have the nrelApiKey secret working for PRs. This was due to a setting buried in Azure Pipelines that allows secret variables to be accessed by PRs. I can see how this would be good to have off for other keys but should be fine for our case.
After a small hiccup dealing with conda's version of pytest-rerunfailures being >Python3.6 I believe this is ready for review.
I'm opting for putting pytest-remotedata in another PR since this one has so many moving parts between pytest and Azure Pipelines. It could be done now but this is currently working and would create another debug iteration that could open up another can of worms.

thanks @CameronTStark @kanderso-nrel @mikofski !


Add negation location to negex outputs

It would probably be a good idea to show not just which entities where negated, but what was the trigger for the negation. This should not be too much of a problem since the matcher returns the position and all that is needed is to persist that.

Great point. There is a way to get this by looking at an entity's modifiers, although it's not based on specific attributes. Does the snippet below get you the info you're looking for?
doc = nlp("There is no evidence of pneumonia.")
for ent in doc.ents:
    print(ent, ent._.is_negated)
    for modifier in ent._.modifiers:
        print(modifier, modifier.category)

Great point. There is a way to get this by looking at an entity's modifiers, although it's not based on specific attributes. Does the snippet below get you the info you're looking for?
doc = nlp("There is no evidence of pneumonia.")
for ent in doc.ents:
    print(ent, ent._.is_negated)
    for modifier in ent._.modifiers:
        print(modifier, modifier.category)

Yeah, I realized I was looking at the old version of medspacy with the original negex instead of the context. Thanks!

Yeah, I realized I was looking at the old version of medspacy with the original negex instead of the context. Thanks!


Generalize API initialization requirements

This would allow subclassers to easily change the list of required metadata using a class attribute.
I'm thinking something like:
class DataAPI(object):

    DatabaseName = 'MyDatabase'
    DataTableName = 'DataFiles'

    TimestampFormat = '%Y%m%d-%H%M%S'

    RequiredUserData=['username','contact']

    def __init__(self, download_priority=None, upload_services=None, **userdata):
        for d in self.RequiredUserData:
            if d not in userdata:
                raise KeyError('{} argument "{}" required'.format(self.__class__.__name__, d))

        self.userdata = dict(userdata)

        self.manager = None
        self.services = {}

        self._download_priority = download_priority
        self._upload_services = upload_services

This would also require changing all references to api.username and api.contact throughout the package.

completed with #53


Bugged

something in glslify is breaking dFdx / dFdy
... will investigate ...

fixed in glsl-tokenizer 2.0.2


feat: Add support for Vault Path in authentication

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

| License         | Apache 2.0
What's in this PR?
Add support for specifying the vault path for vault kubernetes auth.
Why?
If you have multiple clusters in kubernetes but fewer Vault's this allows you to have a shared vault.
Additional context
Checklist

[ ] Implementation tested
[ ] Error handling code meets the guideline
[ ] Logging code meets the guideline
[ ] User guide and development docs updated (if needed)

To Do
N/A

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

There is at least one issue with this PR which would require some changes. Anyways, this will be part of a larger security change in the operator (addressing not only Vault), so closing it for now.


Testing pointpillars pre-trained on kitti dataset on nuscenes data

How can i test pointpillars pre trained on kitti (64 layer lidar )on different lidar data like 32 layer lidar of nuscenes dataset or custom 128  layers lidar data

Please refer to the documentation for demo. You can basically get a sample data from nuScenes and follow the way given in the demo doc.

I have tried the documentation demo but the problem in the nuscenes point cloud size not match with the kitti pretrained model expected input i tried to upsample  the point cloud by padding and the demo run but the detector did not work correctly

What do you mean by "did not work correctly"? BTW, the generalization of 3d point cloud detector is not very good, especially when the property of input such as point density changes. It's reasonable if you get bad prediction results when testing 64 layer trained model on 32 layer data.

Yes ,the model predict a random and and not correct bounding boxes , i will try to fine tune the model with 32 layer lidar data first,thank you very much

There are many underlying problems when you decide to transfer a pretrained model to a new dataset. Apart from different lines of LiDAR, the intensity of point cloud data on nuScenes is also very different from that on KITTI. Its fourth dimension corresponds to timestamp currently, so this can be one of the important reasons that make the adaptation further worse.

Feel free to reopen this issue if you have any further questions :)


Update Postgres DB password to come from a docker secret

@ManoMarks @spara
PTAL -- implements using docker swarm secrets (which docker-compose fakes locally) for the postgres password
first step towards also using for the payment processor token for our OX demo too

devsecrets will be a "developer secret" (i.e., not actually secret) folder checked into version control
In the prod version, a folder prodsecrets will be used (which is in .gitignore to avoid accidentally checking into version control)


✨ feat: Added website header #1

Description
Added a website header also included a logo, navigation list with one of it (Solutions) has dropdown option and a contact us button.
It is resposive to the different width.
Fixes #1
Type of change
Please delete options that are not relevant.

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

Screenshot (optional)
Desktop

Tablet

Mobile

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] When rendered the component is pixel-perfect compared to the design above.
[x] The header adapts naturally to different screen widths

I worked on this with @Tugi-Ngunjiri

@paulkahura
I have updated the issue and its ready for review

@paulkahura I have updated the issue and its ready for review

Kindly add the new screenshots

@paulkahura
I have updated the screenshots as requested

@JAPHETHNYARANGA
I have updated this PR and its ready for review


CreateAsync return type fixed in sample console app

Hello,
it seems that sample console app is not up to date, which might be confusing to some users:
http://salesforce.stackexchange.com/q/95884/13265
Here's my small fix.

This is what I came up with as well.  check that the samples compile during the build

I have removed this sample app. Closing this PR.


Add AsinhMSELoss and AsinhSmoothL1Loss

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

Project coverage is 100.00%. Comparing base (7e2471f) to head (743eaa0).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #86   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           50        51    +1
  Lines          913       939   +26
  Branches        71        71
=========================================
+ Hits           913       939   +26

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


Replace handmade dropdowns with bootstrap dropdowns

Basic support for dropdowns. Not pretty yet but position-wise they are much nicer. And they are JS-based (using click events) rather than CSS-based (using hover). Closes #81

Coverage decreased (-0.4%) to 76.184% when pulling ec137a7bec2e9535e079f1dce34866c1de19f1a4 on eatonphil:master into 00a2d764fc5b24a48ee603c74d75841cb60397b2 on linode:master.

Coverage decreased (-0.4%) to 76.184% when pulling 9eab58953e023d14c3f71ee8ef1e220735f9fb96 on eatonphil:master into 00a2d764fc5b24a48ee603c74d75841cb60397b2 on linode:master.

That dropdown was oriented incorrectly.

Coverage decreased (-0.4%) to 76.184% when pulling 9eab58953e023d14c3f71ee8ef1e220735f9fb96 on eatonphil:master into 00a2d764fc5b24a48ee603c74d75841cb60397b2 on linode:master.

Actually I'd like to get the colors sorted out in #140 and fix the button colors here afterward.

Coverage decreased (-0.4%) to 76.184% when pulling 68b813e6264a308bfb1fc11b80acf29f81c8ab65 on eatonphil:master into 9ca4922d34f8d53707edd1508d3c1b567c2dc07c on linode:master.

Coverage decreased (-0.4%) to 76.184% when pulling 68b813e6264a308bfb1fc11b80acf29f81c8ab65 on eatonphil:master into 9ca4922d34f8d53707edd1508d3c1b567c2dc07c on linode:master.

Coverage increased (+0.3%) to 76.842% when pulling dc64937f37917af037c10387d80d12c48b7d44a2 on eatonphil:master into 9ca4922d34f8d53707edd1508d3c1b567c2dc07c on linode:master.

Tests look good. Ready for merge.

Coverage increased (+0.2%) to 76.76% when pulling 0082ae856d08893ad1af020b266bb04d0ec55ae0 on eatonphil:master into 9ca4922d34f8d53707edd1508d3c1b567c2dc07c on linode:master.

Coverage increased (+0.2%) to 76.76% when pulling 825c4f96d42ac110c69d3c504d9431b422bbde6a on eatonphil:master into 9ca4922d34f8d53707edd1508d3c1b567c2dc07c on linode:master.


Serialize - DeSerialize between similar types.

If we are passing data over the network, we are only having schema but not original types. With wire we are facing issues while serializing or deserializing between this kind of types. As Wire is some how storing the original value (namespace).
Here is simple example.
#I "../../packages/"
#r "Wire.0.8.2/lib/net45/Wire.dll"

open Wire
open System.IO

type User1 = {
    Name : string
}

type User2 = {
    Name : string
}

let opt = SerializerOptions(preserveObjectReferences=false, surrogates = s)

let serializer = new Serializer(opt)
let p =
    use stream = new MemoryStream()
    serializer.Serialize({User1.Name = "Kunjan"},stream)
    stream.ToArray()

let d1 =
    use stream = new MemoryStream(p)
    serializer.Deserialize<User2>(stream)

printfn "%A" d1.Name

Here d1 is not working. While it will work with any JSON serializer. Is there any options I am missing?

@kunjee17 this is result of having polymorphic deserialization. This means that each serialized payload knows it's own type, so you don't have to provide it manually upon deserialization.
I.e. if you have an array new Animal[] { new Cat(name:"Sam"), new Dog(name:"Max") } you could deserialize it with serializer.Deserialize<object>(stream) and you'd still have valid types produced with no need to make casts between them. No JSON serializer would be able to do that without embedding type names in its content.
So this is deliberate decision, deeply set into the library itself.

@Horusiath yeah. Seems like that. But what if I am trying to send data over the wire. It is having type details with it. But on other project, types are not exactly the same. (namespace is different to be precise ). Is there any way to do it in that case?

@kunjee17 it's a matter of use the right tool for the right problem.
Wire is as @Horusiath says, a polymorphic serializer, meaning it will encode type information in the payload.
While it could in theory be possible to have some form of type replacement support, that feels very much like the wrong way to go.
Use a serializer that fits your needs, it's all about trade offs.

Thanks @rogeralsing and @Horusiath for details explanation. I am closing the issue.


chore(bbs): make module public

Make @aries-framework/bbs-signatures public

Codecov Report

Merging #1411 (a76486f) into main (7459509) will decrease coverage by 3.37%.
The diff coverage is 89.83%.

@@            Coverage Diff             @@
##             main    #1411      +/-   ##
==========================================
- Coverage   85.34%   81.97%   -3.37%
==========================================
  Files         788      795       +7
  Lines       19435    19597     +162
  Branches     3159     3169      +10
==========================================
- Hits        16587    16065     -522
- Misses       2841     3525     +684
  Partials        7        7

Impacted Files
Coverage Δ

.../repository/AnonCredsCredentialDefinitionRecord.ts
100.00% <ø> (ø)

.../anoncreds/src/repository/AnonCredsSchemaRecord.ts
100.00% <ø> (ø)

packages/askar/src/wallet/AskarWallet.ts
3.49% <0.00%> (-62.54%)
:arrow_down:

packages/core/src/error/index.ts
100.00% <ø> (ø)

packages/indy-sdk/src/wallet/IndySdkWallet.ts
46.69% <0.00%> (-0.39%)
:arrow_down:

...kages/indy-vdr/src/dids/IndyVdrIndyDidRegistrar.ts
94.50% <ø> (ø)

...re/src/wallet/error/WalletExportPathExistsError.ts
50.00% <50.00%> (ø)

...ages/core/src/storage/migration/UpdateAssistant.ts
86.32% <60.00%> (-3.20%)
:arrow_down:

...e/src/storage/migration/updates/0.3.1-0.4/cache.ts
87.50% <87.50%> (ø)

...ages/anoncreds/src/updates/0.3.1-0.4/linkSecret.ts
89.47% <89.47%> (ø)

... and 13 more

... and 14 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


PDF, CSV export feature for the entity list table

PDF, CSV export feature for the entity list table

Written by @xqliu
Created at 2014-12-09T00:45:14.000+0800
Link to http://betterlife.atlassian.net/browse/BE-32

http://ui-grid.info/docs/#/tutorial/206_exporting_data

Written by @xqliu
Created at 2014-12-09T00:45:58.549+0800

As a User I want to export PDF, CSV from the entity table so that export data from server to offline.

Written by @xqliu
Created at 2015-03-07T12:29:51.615+0800


Log when we see an OPTIONS without AccessControlRequestMethod

Right here, we ignore an OPTIONS request without the AccessControlRequestMethod header, and treat it as a non-preflight request.
We should log specifically when this happens, it's important to know why in this case the request is being treated as non-preflight.
What will typically happen is that your CORS-ing in front of an API, and that API won't support OPTIONS, so you get a 404.

This issue was moved to aspnet/Home#2375


Examples fail....

The XAML in the example file
WpfAnimatedGif-master\WpfAnimatedGif.Demo\MainWindows
fails to compile with
error MC3072: The property 'ImageBehavior.AnimateInDesignMode' does not exist in XML namespace 'http://wpfanimatedgif.codeplex.com'.
The XAML in the example file
WpfAnimatedGif-master\WpfAnimatedGif.Demo\TestRelativeUri.xaml
fails to compile with
The property 'ImageBehavior.RepeatBehavior' does not exist in XML namespace 'http://wpfanimatedgif.codeplex.com'
I tried replacing
http://wpfanimatedgif.codeplex.com
with
http://github.com/XAMLAnimatedGif/WpfAnimatedGif
and then
http://github.com/ThomasLevesque/WpfAnimatedGif
but the same problem occurs.
I downloaded the package with NuGet manager into Visual Studio 2019.
Sorry, but this code just does not work in this application.

Note: I also read the note for the same problem
https://github.com/XamlAnimatedGif/WpfAnimatedGif/issues/19
but the link to the solution failed. It also said it should not happen if you use the NuGet package manager, which I tried, but no success.

Hi @GrantGussie,
Sorry you're having trouble. But I just checked, and the project compiles fine for me... There might be something wrong with your Visual Studio installation.

I downloaded the package with NuGet manager into Visual Studio 2019.

I'm not sure I understand this remark. The demo project isn't in the NuGet package...

I tried replacing
http://wpfanimatedgif.codeplex.com
with
http://github.com/XAMLAnimatedGif/WpfAnimatedGif
and then
http://github.com/ThomasLevesque/WpfAnimatedGif

No, the workspace is fine. It's the former URL of the project, before I migrated it to GitHub. I didn't change it when I migrated because it would have been a breaking change. But it's just an identifier for the namespace, the URL isn't actually accessed.

Thank you Thomas for the prompt reply.
I realize my original post reads as being abrupt and curt, rather than concise, which I was aiming for. Sorry about that... it was a long day and I was getting tired.
To clarify my comment about downloading the package with NuGet manager... the note about the same problem I found (#19) mentioned that if you download the package with the NuGet manager you should not have this problem. So I tried to add the package to the demo manually that way. The add failed. And I still can't get the demo to compile.
However, my own application did accept the Nuget package and now compiles and runs just fine... and frankly that's good enough for me.
However, I am including the output created when I tried to compile the demo (as downloaded from github) with Visual Studio 2019.
As you can see, numerous errors occur, starting with "System.Exception: Failed to run Git". Is this is reference to a github extension (https://github.com/Haacked/VisualStudio)?
1>------ Rebuild All started: Project: WpfAnimatedGif, Configuration: Debug Any CPU ------
1>
1>Unhandled Exception: System.Exception: Failed to run Git. Git may not be installed on the system. ---> System.ComponentModel.Win32Exception: The system cannot find the file specified
1>   at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
1>   at System.Diagnostics.Process.Start()
1>   at MinVer.Lib.GitCommand.TryRun(String args, String workingDirectory, ILogger log, String& output) in /home/runner/work/minver/minver/MinVer.Lib/GitCommand.cs:line 32
1>   --- End of inner exception stack trace ---
1>   at MinVer.Lib.GitCommand.TryRun(String args, String workingDirectory, ILogger log, String& output) in /home/runner/work/minver/minver/MinVer.Lib/GitCommand.cs:line 36
1>   at MinVer.Lib.Repository.TryCreateRepo(String workDir, Repository& repository, ILogger log) in /home/runner/work/minver/minver/MinVer.Lib/Repository.cs:line 21
1>   at MinVer.Lib.Versioner.GetVersion(String workDir, String tagPrefix, VersionPart autoIncrement, String defaultPreReleasePhase, ILogger log) in /home/runner/work/minver/minver/MinVer.Lib/Versioner.cs:line 36
1>   at MinVer.Lib.Versioner.GetVersion(String workDir, String tagPrefix, MajorMinor minMajorMinor, String buildMeta, VersionPart autoIncrement, String defaultPreReleasePhase, ILogger log) in /home/runner/work/minver/minver/MinVer.Lib/Versioner.cs:line 9
1>   at MinVer.Program.<>c__DisplayClass1_0.b__0() in /home/runner/work/minver/minver/minver-cli/Program.cs:line 59
1>   at MinVer.Program.Main(String[] args) in /home/runner/work/minver/minver/minver-cli/Program.cs:line 31
1>
1>Unhandled Exception: System.Exception: Failed to run Git. Git may not be installed on the system. ---> System.ComponentModel.Win32Exception: The system cannot find the file specified
1>   at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
1>   at System.Diagnostics.Process.Start()
1>   at MinVer.Lib.GitCommand.TryRun(String args, String workingDirectory, ILogger log, String& output) in /home/runner/work/minver/minver/MinVer.Lib/GitCommand.cs:line 32
1>   --- End of inner exception stack trace ---
1>   at MinVer.Lib.GitCommand.TryRun(String args, String workingDirectory, ILogger log, String& output) in /home/runner/work/minver/minver/MinVer.Lib/GitCommand.cs:line 36
1>   at MinVer.Lib.Repository.TryCreateRepo(String workDir, Repository& repository, ILogger log) in /home/runner/work/minver/minver/MinVer.Lib/Repository.cs:line 21
1>   at MinVer.Lib.Versioner.GetVersion(String workDir, String tagPrefix, VersionPart autoIncrement, String defaultPreReleasePhase, ILogger log) in /home/runner/work/minver/minver/MinVer.Lib/Versioner.cs:line 36
1>   at MinVer.Lib.Versioner.GetVersion(String workDir, String tagPrefix, MajorMinor minMajorMinor, String buildMeta, VersionPart autoIncrement, String defaultPreReleasePhase, ILogger log) in /home/runner/work/minver/minver/MinVer.Lib/Versioner.cs:line 9
1>   at MinVer.Program.<>c__DisplayClass1_0.b__0() in /home/runner/work/minver/minver/minver-cli/Program.cs:line 59
1>   at MinVer.Program.Main(String[] args) in /home/runner/work/minver/minver/minver-cli/Program.cs:line 31
1>
1>Unhandled Exception: System.Exception: Failed to run Git. Git may not be installed on the system. ---> System.ComponentModel.Win32Exception: The system cannot find the file specified
1>   at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
1>   at System.Diagnostics.Process.Start()
1>   at MinVer.Lib.GitCommand.TryRun(String args, String workingDirectory, ILogger log, String& output) in /home/runner/work/minver/minver/MinVer.Lib/GitCommand.cs:line 32
1>   --- End of inner exception stack trace ---
1>   at MinVer.Lib.GitCommand.TryRun(String args, String workingDirectory, ILogger log, String& output) in /home/runner/work/minver/minver/MinVer.Lib/GitCommand.cs:line 36
1>   at MinVer.Lib.Repository.TryCreateRepo(String workDir, Repository& repository, ILogger log) in /home/runner/work/minver/minver/MinVer.Lib/Repository.cs:line 21
1>   at MinVer.Lib.Versioner.GetVersion(String workDir, String tagPrefix, VersionPart autoIncrement, String defaultPreReleasePhase, ILogger log) in /home/runner/work/minver/minver/MinVer.Lib/Versioner.cs:line 36
1>   at MinVer.Lib.Versioner.GetVersion(String workDir, String tagPrefix, MajorMinor minMajorMinor, String buildMeta, VersionPart autoIncrement, String defaultPreReleasePhase, ILogger log) in /home/runner/work/minver/minver/MinVer.Lib/Versioner.cs:line 9
1>   at MinVer.Program.<>c__DisplayClass1_0.b__0() in /home/runner/work/minver/minver/minver-cli/Program.cs:line 59
1>   at MinVer.Program.Main(String[] args) in /home/runner/work/minver/minver/minver-cli/Program.cs:line 31
1>C:\Users\Owner.nuget\packages\minver\2.0.0\build\MinVer.targets(39,5): error MSB3073: The command "dotnet "C:\Users\Owner.nuget\packages\minver\2.0.0\build../minver/MinVer.dll" --auto-increment "" --build-metadata "" --default-pre-release-phase "" --minimum-major-minor "" --repo "D:__WpfAnimatedControl\WpfAnimatedGif" --tag-prefix "" --verbosity "" --version-override """ exited with code -532462766.
1>Done building project "WpfAnimatedGif.csproj" -- FAILED.
1>C:\Users\Owner.nuget\packages\minver\2.0.0\build\MinVer.targets(39,5): error MSB3073: The command "dotnet "C:\Users\Owner.nuget\packages\minver\2.0.0\build../minver/MinVer.dll" --auto-increment "" --build-metadata "" --default-pre-release-phase "" --minimum-major-minor "" --repo "D:__WpfAnimatedControl\WpfAnimatedGif" --tag-prefix "" --verbosity "" --version-override """ exited with code -532462766.
1>C:\Users\Owner.nuget\packages\minver\2.0.0\build\MinVer.targets(39,5): error MSB3073: The command "dotnet "C:\Users\Owner.nuget\packages\minver\2.0.0\build../minver/MinVer.dll" --auto-increment "" --build-metadata "" --default-pre-release-phase "" --minimum-major-minor "" --repo "D:__WpfAnimatedControl\WpfAnimatedGif" --tag-prefix "" --verbosity "" --version-override """ exited with code -532462766.
1>Done building project "WpfAnimatedGif.csproj" -- FAILED.
2>------ Rebuild All started: Project: WpfAnimatedGif.Demo, Configuration: Debug Any CPU ------
2>D:__WpfAnimatedControl\WpfAnimatedGif.Demo\MainWindow.xaml(5,9): error MC3072: The property 'ImageBehavior.AnimateInDesignMode' does not exist in XML namespace 'http://wpfanimatedgif.codeplex.com'. Line 5 Position 9.
2>D:__WpfAnimatedControl\WpfAnimatedGif.Demo\TestRelativeUri.xaml(13,32): error MC3072: The property 'ImageBehavior.AnimatedSource' does not exist in XML namespace 'http://wpfanimatedgif.codeplex.com'. Line 13 Position 32.
2>Done building project "WpfAnimatedGif.Demo.csproj" -- FAILED.
========== Rebuild All: 0 succeeded, 2 failed, 0 skipped ==========

Ah, I see what's going on. MinVer is a package used by the library to compute a version number based on Git versioning. If MinVer can't find Git, it fails, so the library fails to build, and since the demo project depends on the library, it also fails...
How did you get the code? Did you clone the Git repo or just download the code? Is Git in your PATH?

Hi... I just downloaded the code and unzipped the file.
Then I opened the .sln file with VIsual Studio 2019.  The ,sln file updated to 2019 without problem.
Then the only thing I did before trying to compile it is change the "Solution | Properties | Start Up Project" from "WpfAnimatedGif" to "WpfAnimatedGif.Demo".
And GIT is not in my build path.
Cheers!
G

Glad we were able to sort that out. Yeah, you absolutely need Git to build the project... I never realized it could be a constraint, since I use Git for all my projects.


fix: ensure the module does not instantiate an empty condition block

Guard the creation of the condition block on the length of the optional
listener_http_header_conditions.
Fixes #42
what

Make the condition blocks dynamic, requiring a non-empty list for the http headers

why

Without this, the module attempts to create an empty condition, and fails

references
Fixes #42

/test all

/test all


Labels layer: Zooming while brush is activated

🚀 Feature
It would be cool if the mouse wheel allows zooming in and out while the brush tool is activated on a labels layer.
Motivation
While annotating small structures on a large image, I'm zooming in and out between drawing lines. It would be cool if this was possible without switching between brush and zoom tool.
Pitch
When I use the mouse-wheel while the brush is activated, nothing happens. Thus, I presume we could use this input for zooming without harming any preexisting workflow.
Alternatives
For annotating, we could use the right mouse button while the zoom-mode is on.
Additional context

Might #4814 be a solution for this issue? Holding 5 will switch mode for a moment.

When I use the mouse-wheel while the brush is activated, nothing happens. Thus, I presume we could use this input for zooming without harming any preexisting workflow.

fwiw I and I think a few other people have set it up so that scrolling with the brush active modifies the brush size which I find to be very convenient. If I want to zoom then I press and hold space and scroll.

fwiw I and I think a few other people have set it up so that scrolling with the brush active modifies the brush size

That's cool, too. How can I get this?

If I want to zoom then I press and hold space and scroll.

Awesome! I'm learning new things here. Where in the documentation can I find this? Are there more tricks of that kind?

Awesome! I'm learning new things here. Where in the documentation can I find this? Are there more tricks of that kind?

On the bottom bar of napari when activating brush.

But other shortcuts are not working and need to be treated from settings.

That's cool, too. How can I get this?

Feel free to use this callback: https://github.com/Hekstra-Lab/napari-labeller/blob/74913dce72c773df2ec94e1cb3798dd40fedf219/src/napari_labeller/_keybinds.py#L59-L66

On the bottom bar of napari when activating brush.

Wow. Indeed. I have never noticed that! Thanks again. And if more people complain that this is not obvious / accessible enough. Consider putting this hint in a more prominent place.

Feel free to use this callback: https://github.com/Hekstra-Lab/napari-labeller/blob/74913dce72c773df2ec94e1cb3798dd40fedf219/src/napari_labeller/_keybinds.py#L59-L66

How about making this part of napari-core? I mean at the moment, the mouse wheel has no function attached. It would be cool if it had. It would make napari more user-friendly

I try to understand how napari/vispy change wheel behavior base on layer mode. But still have no idea how it works.
I end in place when it comes to rendering under the mouse.

I agree.
It doesn't make sense that you need to use a different method to zoom for image layers vs other layer types. And especially so because the mouse wheel isn't even given another purpose, it just does nothing.
It would make much more sense if the zoom user interface was consistent across all layer types and modes. I also support the idea of adding to core napari a way to use the scroll wheel plus a modifier key to adjust the brush size when a user is in paint mode.

There was a design audit done on napari a quite a while back. My understanding was that it was intended in part to find these kinds of inconsistencies. Perhaps it's worthwhile revisiting that?

See also:

https://github.com/napari/napari/issues/2060
https://www.napari-hub.org/plugins/napari-mouse-controls

I'm happy to contribute code from the linked napari plugin to napari core if a core developer guides me where to put it

I may help, but I do not have an idea how should look interface for this (understanding that we will implement these controls in the image control area)