Kconfig choice symbols could not be assigned in Kconfig.* files

When a Kconfig symbol is defined in a choice directive, it cannot be defined in Kconfig.* file.
This is blocking for particular cases where _defconfig files cannot be used to by-pass this situation (#6837)
For instance:
choice
	prompt "Trigger mode"
	depends on HTS221
	default HTS221_TRIGGER_GLOBAL_THREAD
	help
	  Specify the type of triggering to be used by the driver.

config HTS221_TRIGGER_NONE
	bool
	prompt "No trigger"

config HTS221_TRIGGER_GLOBAL_THREAD
	bool
	prompt "Use global thread"
	depends on GPIO
	select HTS221_TRIGGER

config HTS221_TRIGGER_OWN_THREAD
	bool
	prompt "Use own thread"
	depends on GPIO
	select HTS221_TRIGGER

endchoice

When trying to set HTS221_TRIGGER_NONE in Kconfig.defconfig:
warning: default on the choice symbol HTS221_TRIGGER_NONE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:43) will have no effect

or Kconfig.shield:
warning: the choice symbol HTS221_TRIGGER_NONE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:43) is selected by the following symbols, which has no effect:
 - SHIELD_X_NUCLEO_IKS01A1 (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.shield:8)

Hi @ulfalizer , I guess you would have a valuable opinion on this.

@erwango
Hello,
If I'm understanding it right, you want to have a different symbol be the default selection of the choice depending on some condition. To do that, you put conditions on the defaults:
choice
	prompt "Trigger mode"
	depends on HTS221
	default HTS221_TRIGGER_GLOBAL_THREAD if FOO && BAR
        default HTS221_TRIGGER_OWN_THREAD if BAZ
	help
	  Specify the type of triggering to be used by the driver.

...

endchoice

If you want those defaults to live in a separate file, there's an obscure Kconfig feature you could use: named choices.
First, give the choice a name:
choice TRIGGER_MODE
	prompt "Trigger mode"
	depends on HTS221
	help
	  Specify the type of triggering to be used by the driver.

...

endchoice

Then, in your Kconfig.* file, list it again with the same name, and add any properties you want. You can leave the body empty.
if SOME_BOARD_OR_SIMILAR

...

choice TRIGGER_MODE
	default HTS221_TRIGGER_GLOBAL_THREAD
        depends on HTS221

endchoice

...

endif

Zephyr's Kconfiglib and C Kconfig tools have a custom patch that makes Kconfig prefer defaults defined later, to enable those Kconfig.* overrides. Vanilla Kconfig prefers earlier defaults.
It seems choices still prefer early defaults though. I could patch those too if needed.
The logic for calculating the value of a choice symbol completely ignores defaults and selects by the way. Having to define what it means for a choice symbol to have a default (pretty weird) or be selected would make choices much more complex, and they're pretty complex already. There are many cases you don't think about until you try to implement them.

Thanks @ulfalizer for these propositions.

If you want those defaults to live in a separate file

This is the case indeed.

there's an obscure Kconfig feature you could use: named choices.

I already tried to use this approach once as proposed. Problem was the dependency introduced on the board  defining default value for the named choice, so I reverted it:
https://github.com/zephyrproject-rtos/zephyr/pull/5636
Ideal solution would be to use named choice with a default value that doesn't introduce dependency
on a specific board. I haven't been able to figure out how to do this.

@erwango
What do you mean "the dependency introduced on the board"? Only the default itself should depend on the board. What was the warning?
The following two definitions should be perfectly equivalent:
All in one place
choice
	bool "choice"
        default CHOICE_SYM_A if BOARD_A
        default CHOICE_SYM_B if BOARD_B

config CHOICE_SYM_A
        bool "A"

config CHOICE_SYM_B
        bool "B"

endchoice

Split into many locations
choice NAMED
	bool "choice"

config CHOICE_SYM_A
        bool "A"

config CHOICE_SYM_B
        bool "B"

endchoice

...

if BOARD_A

choice NAMED
        default CHOICE_SYM_A

endchoice

endif

...

if BOARD_B

choice NAMED
        default CHOICE_SYM_B

endchoice

endif

As a side note, a choice without defaults will default to the first (visible) symbol in the choice.

I'm not able to retrieve the warning message, but as I understood, when using sensor on one board w/o specific configuration, you were referring to the default configuration made in another board, which was not acceptable, hence the warning.
So the first solution you propose is not possible as it would mean updating driver Kconfig each time you add a board likely to use it.
Your second proposition looks like what I did initially, but let me have another try.

@erwango
I just looked into defaults on choices btw, and the C tools in Zephyr haven't been patched to prefer later defaults there either, so it's a bit inconsistent: Later defaults override earlier defaults for symbols, but earlier defaults override later defaults for choices.
That means that the "base" choice definition (where the symbols are defined) can't have any defaults (with satisfied conditions). Otherwise, those will take precedence over the defaults in the Kconfig.* files, which are included later.

@ulfalizer : So I tried proposal and it seems to work
https://github.com/zephyrproject-rtos/zephyr/pull/6966
I don't actually quite understand the difference vs what was causing troubles earlier, but ok.
I'll check warnings in non reg.
Now, I'm trying to apply this to #6837 and I get a lot of warnings:
Merging /local/mcu/zephyr/zephyr-project/samples/sensor/hts221/prj.conf
warning: HTS221_TRIGGER_MODE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a2/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:36) defined with type unknown
warning: the default selection HTS221_TRIGGER_NONE (defined at /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:54) of HTS221_TRIGGER_MODE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a2/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:36) is not contained in the choice
warning: the default selection HTS221_TRIGGER_NONE (defined at /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:54) of HTS221_TRIGGER_MODE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a2/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:36) is not contained in the choice
warning: LIS3MDL_TRIGGER_MODE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:14) defined with type unknown
warning: LIS3MDL_TRIGGER_MODE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:14) defined without a prompt
warning: the default selection LIS3MDL_TRIGGER_NONE (defined at /local/mcu/zephyr/zephyr-project/drivers/sensor/lis3mdl/Kconfig:65) of LIS3MDL_TRIGGER_MODE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:14) is not contained in the choice
warning: HTS221_TRIGGER_MODE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a2/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:36) defined with type unknown
warning: the default selection HTS221_TRIGGER_NONE (defined at /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:54) of HTS221_TRIGGER_MODE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a2/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:36) is not contained in the choice
warning: the default selection HTS221_TRIGGER_NONE (defined at /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:54) of HTS221_TRIGGER_MODE (defined at /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a1/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/boards/shields/x_nucleo_iks01a2/Kconfig.defconfig:10, /local/mcu/zephyr/zephyr-project/drivers/sensor/hts221/Kconfig:36) is not contained in the choice

Though, it rather looks like shields/*/Kconfig.defconfig files are parsed unconditionally, which might be related with my changes.

I get a lot of warnings:

Solved after moving source of shields/*/Kconfig.defconfig after drivers Kconfig parsing
So, seems fine!

That means that the "base" choice definition (where the symbols are defined) can't have any defaults (with satisfied conditions). Otherwise, those will take precedence over the defaults in the Kconfig.* files, which are included later.

Ok, hence this is why default should be removed from choice. Got it, thanks.

@erwango
Glad it worked out.
That warning could be improved. It shouldn't matter whether the instance of the choice that actually has the default symbol comes later. It might be a non-trivial fix though (it turns out that you need to look at dependencies to even know if a symbol is a choice symbol or not, because of implicit submenus, which makes things a bit messy).

Turned out to be a pretty simple fix, luckily: https://github.com/ulfalizer/Kconfiglib/commit/e8408a06c68d87485a9d45817dfdd60e722a6f1c
Those warnings won't get generated anymore once that gets into Zephyr's Kconfiglib fork.

@ulfalizer @erwango #6969 is merged now. Can we close this?


Improve UX

Add event to complete/doing tasks

Add loader when list isn't available

PR Feature ux #22


In the query interface, add some examples of how to search for a specific user and group of users

@qjhart can you provide some nice examples and I will add them to the page?

Query Examples
In the following examples, these are called from the query page.  For shorthand, the form is specified as
Demand Queries

form
value

Table
Demand

Watershed
EEL RIVER

Date
5-1-2014

With the Query changing.

Desire
Query

One Application
SELECT * WHERE application_number='A001719'

Large Riparian Demands, Descending
SELECT * WHERE riparian = true and demand > 20 ORDER by demand desc

Latest Appropriative rights
SELECT * WHERE water_right_type='Appropriative' and priority_date  > '1990-01-01'  order by priority_date desc

For a specific HUC
SELECT * WHERE huc_12 = '180101050802'

Examples are up.

This looks great; just one further request to clarify:
I had trouble understanding this line: "With the form is specified as:". I had to go back to Quinn's example above to figure it out. I think this would be more clear: "With the above form specified as:" (dropping the is and adding the above).
Also, I would change the period after "With the Query changing." to a colon so that it matches the other instructional line.

@brmagnuson push changes.  Let me know if this is good to close.

Looks good to close, other than I'm not seeing any of the changes in Chrome--does that just take another push later on once all the issues are closed?

@brmagnuson The changes were pushed yesterday.  Can you refresh your browser and let me know if you see the updates?

That finally worked; I had to clear the cache, apparently. Thanks!


3493 fix cocoon JS not working in Safari

Resolves #3493. It works now! Cocoon is unfortunately old and unmaintained. We should probably look into removing our usage of it or vendoring the whole thing into our repo if we want to keep it.

I did a quick check on local and it seems to work with my Safari - so if @awwaiid is good with this so am I

I plan to do some more testing Saturday afternoon.  Hoping this is the last stopper.


Blender missing "requests" python library for add-ons

Issue description
Blender is missing the Python requests library which is a dependency of at least one popular add-on
Steps to reproduce
Open Edit > Preferences > Add-ons and select 3D View: BlenderKit Online Asset Library and this Python backtrace is presented:
Traceback (most recent call last):
  File "/nix/store/v8ykwxys87dfh60s414ds70ckbgyl9yx-blender-2.83.1/share/blender/2.83/scripts/modules/addon_utils.py", line 351, in enable
    mod = __import__(module_name)
  File "/nix/store/v8ykwxys87dfh60s414ds70ckbgyl9yx-blender-2.83.1/share/blender/2.83/scripts/addons/blenderkit/__init__.py", line 51, in <module>
    from blenderkit import asset_inspector, search, download, upload, ratings, autothumb, ui, icons, bg_blender, paths, \
  File "/nix/store/v8ykwxys87dfh60s414ds70ckbgyl9yx-blender-2.83.1/share/blender/2.83/scripts/addons/blenderkit/search.py", line 33, in <module>
    from blenderkit import paths, utils, categories, ui, colors, bkit_oauth, version_checker, tasks_queue, rerequests
  File "/nix/store/v8ykwxys87dfh60s414ds70ckbgyl9yx-blender-2.83.1/share/blender/2.83/scripts/addons/blenderkit/categories.py", line 27, in <module>
    from blenderkit import paths, utils, tasks_queue, rerequests
  File "/nix/store/v8ykwxys87dfh60s414ds70ckbgyl9yx-blender-2.83.1/share/blender/2.83/scripts/addons/blenderkit/rerequests.py", line 28, in <module>
    from blenderkit import ui, utils, paths, tasks_queue, bkit_oauth
  File "/nix/store/v8ykwxys87dfh60s414ds70ckbgyl9yx-blender-2.83.1/share/blender/2.83/scripts/addons/blenderkit/ui.py", line 34, in <module>
    from blenderkit import paths, ratings, utils, search, upload, ui_bgl, download, bg_blender, colors, tasks_queue
  File "/nix/store/v8ykwxys87dfh60s414ds70ckbgyl9yx-blender-2.83.1/share/blender/2.83/scripts/addons/blenderkit/ratings.py", line 30, in <module>
    import requests, threading
ModuleNotFoundError: No module named 'requests'

Technical details

system: "x86_64-linux"
host os: Linux 5.7.7, NixOS, 20.03.git.a41b1a288c7 (Markhor)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.6
nixpkgs: /home/luke/git/nixpkgs

Workaround (fix?) seems to be
nix-shell -p python3Packages.requests --run blender

Proposed solution https://github.com/NixOS/nixpkgs/pull/97388.


Investigate what format to use to store embeddings+id

Current format :

Numpy matrix + parquet for IDs (ordered collections)
parquet with embeddings+id

Numpy+parquet :
Benefit:

Fast to read numpy alone
fast to read parquet alone
Drawback:
Ordered collections are distribution friendly. Even though it is possible to keep ordering when doing distributed processing, it makes things significantly more complex

Parquet with embeddings :
Benefit:

No ordering needed
Easy to filter / use normal processing tools (parquet, beam, pandas)
Drawback:
Embeddings in parquet are represented as an variable length array, this is not efficient
slow to read
It doesn't make sense to use a columnar format to read data sequentially with all columns

What alternative exist to store embeddings+id ?

I may give parquet with embeddings one more try to check if it's really that slow
Alternative formats i can think of:

Tfrecord
orc ?
Some SQL database?

Direction to look into:

Format supported by spark or beam readers
Format supported by pandas or pyarrow
things like zarr


Defining new api with yaml file: setting publicAPI or publicServer to true does not work

I'm using Apiman 1.2.1 on Wildfly 9, and the latest version of apiman-cli (d22e6ea48a9044a26bd06ca0d1ec3def2e9c900f). I'm using this simple yaml:
---
system:
  gateways: []
  plugins: []
org:
  name: "myorg"
  description: "myorg descr"
  apis:
    - name: "someapi4"
      description: "someapi descr"
      initialVersion: "1.0"
      published: true
      config:
        endpoint: "${myApiEndPoint}"
        endpointType: "rest"
        #note the following line:
        publicAPI: true
        gateway: "TheGateway"
      policies: []

and use it with:
./apiman apply --server http://localhost:8080/apiman -f ../apiman_settings.yml -P myApiEndPoint=http://myapi:8080/ --serverVersion v12x

The api is created, but not set public (checkbox on "Make this API public" is not set). Publishing therefore fails, with a stacktrace:
INFO  Loaded declaration: ../apiman_settings.yml
INFO  Org already exists: myorg
INFO  API already exists: someapi4
INFO  Configuring API: someapi4
ERROR Error in Apply declaration
io.apiman.cli.exception.CommandException: Error applying declaration
        at io.apiman.cli.core.declarative.command.ApplyCommand.performAction(ApplyCommand.java:105) ~[apiman-cli.jar:?]
        at io.apiman.cli.command.AbstractCommand.run(AbstractCommand.java:126) [apiman-cli.jar:?]
        at io.apiman.cli.command.AbstractCommand.run(AbstractCommand.java:155) [apiman-cli.jar:?]
        at io.apiman.cli.Cli.main(Cli.java:37) [apiman-cli.jar:?]
Caused by: io.apiman.cli.exception.DeclarativeException: Unable to publish API 'someapi4' in state: Created
        at io.apiman.cli.core.declarative.command.ApplyCommand.publish(ApplyCommand.java:401) ~[apiman-cli.jar:?]
        at io.apiman.cli.core.declarative.command.ApplyCommand.lambda$null$27(ApplyCommand.java:235) ~[apiman-cli.jar:?]
        at java.util.ArrayList.forEach(ArrayList.java:1249) ~[?:1.8.0_71]
        at io.apiman.cli.core.declarative.command.ApplyCommand.lambda$applyApis$28(ApplyCommand.java:222) ~[apiman-cli.jar:?]
        at java.util.Optional.ifPresent(Optional.java:159) ~[?:1.8.0_71]
        at io.apiman.cli.core.declarative.command.ApplyCommand.applyApis(ApplyCommand.java:219) ~[apiman-cli.jar:?]
        at io.apiman.cli.core.declarative.command.ApplyCommand.lambda$applyDeclaration$17(ApplyCommand.java:138) ~[apiman-cli.jar:?]
        at java.util.Optional.ifPresent(Optional.java:159) ~[?:1.8.0_71]
        at io.apiman.cli.core.declarative.command.ApplyCommand.applyDeclaration(ApplyCommand.java:124) ~[apiman-cli.jar:?]
        at io.apiman.cli.core.declarative.command.ApplyCommand.performAction(ApplyCommand.java:103) ~[apiman-cli.jar:?]
        ... 3 more

Note that I've used publicAPI, as found in io.apiman.cli.core.api.model.ApiConfig, instead of publicService as still mentioned in the README.md. (Using publicService doesn't work either.)

Moved to: https://issues.jboss.org/browse/APIMAN-948

@bramvonk it looks the YAML in the README was wrong; it should be public rather than publicApi or publicService.
I've fixed the example in the README now.
When I make the change, the YAML now applies correctly:
./apiman apply --server http://localhost:8080/apiman -f /tmp/api.yaml -P myApiEndPoint=http://myapi:8080/ --serverVersion v12x
INFO  Loaded declaration: /tmp/api.yaml
INFO  Org already exists: myorg
INFO  API already exists: someapi4
INFO  Configuring API: someapi4
INFO  Publishing API: someapi4
INFO  Applied declaration


[Enhancement]: Remove Contact Us section and prevent page reload on clicking QR button

Description of the bug

Remove the contact us route and link from navbar
Prevent the page from reloading when we click the QR button. You can achieve this by giving an e.preventDefault( ) function in the QR code function

Please add screenshots (if applicable)
No response
Additional information
No response
Code of Conduct

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

Improvements have been done


OutOfRangeError (see above for traceback): FIFOQueue '_1_create_inputs/batch/fifo_queue' is closed and has insufficient elements (requested 10, current size 0)

thanks for your geat work. I encount the follow problem when I run the code , thank you very much.
D:\Anaconda3\python.exe "D:\PyCharm Community Edition 2017.2.3\helpers\pydev\pydev_run_in_console.py" 51787 51788 D:/pyprojects/tensorflow-deeplab-resnet-master/train.py
Running D:/pyprojects/tensorflow-deeplab-resnet-master/train.py
import sys; print('Python %s on %s' % (sys.version, sys.platform))
sys.path.extend(['D:\pyprojects\tensorflow-deeplab-resnet-master', 'D:/pyprojects/tensorflow-deeplab-resnet-master'])
Backend Qt5Agg is interactive backend. Turning interactive mode on.
2017-12-10 14:58:35.976057: W c:\l\tensorflow_1501918863922\work\tensorflow-1.2.1\tensorflow\core\platform\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE instructions, but these are available on your machine and could speed up CPU computations.
2017-12-10 14:58:35.976473: W c:\l\tensorflow_1501918863922\work\tensorflow-1.2.1\tensorflow\core\platform\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE2 instructions, but these are available on your machine and could speed up CPU computations.
2017-12-10 14:58:35.976857: W c:\l\tensorflow_1501918863922\work\tensorflow-1.2.1\tensorflow\core\platform\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE3 instructions, but these are available on your machine and could speed up CPU computations.
2017-12-10 14:58:35.977247: W c:\l\tensorflow_1501918863922\work\tensorflow-1.2.1\tensorflow\core\platform\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.1 instructions, but these are available on your machine and could speed up CPU computations.
2017-12-10 14:58:35.977635: W c:\l\tensorflow_1501918863922\work\tensorflow-1.2.1\tensorflow\core\platform\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use SSE4.2 instructions, but these are available on your machine and could speed up CPU computations.
2017-12-10 14:58:35.978033: W c:\l\tensorflow_1501918863922\work\tensorflow-1.2.1\tensorflow\core\platform\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX instructions, but these are available on your machine and could speed up CPU computations.
2017-12-10 14:58:35.978417: W c:\l\tensorflow_1501918863922\work\tensorflow-1.2.1\tensorflow\core\platform\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use AVX2 instructions, but these are available on your machine and could speed up CPU computations.
2017-12-10 14:58:35.978802: W c:\l\tensorflow_1501918863922\work\tensorflow-1.2.1\tensorflow\core\platform\cpu_feature_guard.cc:45] The TensorFlow library wasn't compiled to use FMA instructions, but these are available on your machine and could speed up CPU computations.
2017-12-10 14:58:42.310260: W c:\l\tensorflow_1501918863922\work\tensorflow-1.2.1\tensorflow\core\framework\op_kernel.cc:1158] Not found: Can not get size for: ./dataset/VOCdevkit/SegmentationClassAug/2008_003519.png : ϵͳ�Ҳ���ָ����·����
2017-12-10 14:58:42.310260: W c:\l\tensorflow_1501918863922\work\tensorflow-1.2.1\tensorflow\core\framework\op_kernel.cc:1158] Not found: Can not get size for: ./dataset/VOCdevkit/JPEGImages/2008_003519.jpg : ϵͳ�Ҳ���ָ����·����
--- Logging error ---
Traceback (most recent call last):
File "D:\Anaconda3\lib\site-packages\tensorflow\python\training\queue_runner_impl.py", line 238, in _run
enqueue_callable()
File "D:\Anaconda3\lib\site-packages\tensorflow\python\client\session.py", line 1063, in single_operation_run
target_list_as_strings, status, None)
File "D:\Anaconda3\lib\contextlib.py", line 88, in exit
next(self.gen)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\framework\errors_impl.py", line 466, in raise_exception_on_not_ok_status
pywrap_tensorflow.TF_GetCode(status))
tensorflow.python.framework.errors_impl.NotFoundError: Can not get size for: ./dataset/VOCdevkit/SegmentationClassAug/2008_003519.png : ϵͳ\udcd5Ҳ\udcbb\udcb5\udcbdָ\udcb6\udca8\udcb5\udcc4·\udcbe\udcb6\udca1\udca3
[[Node: create_inputs/ReadFile_1 = ReadFile_device="/job:localhost/replica:0/task:0/cpu:0"]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "D:\Anaconda3\lib\logging_init.py", line 994, in emit
stream.write(msg)
UnicodeEncodeError: 'utf-8' codec can't encode character '\udcd5' in position 195: surrogates not allowed
Call stack:
File "D:\Anaconda3\lib\threading.py", line 884, in _bootstrap
self._bootstrap_inner()
File "D:\Anaconda3\lib\threading.py", line 916, in _bootstrap_inner
self.run()
File "D:\Anaconda3\lib\threading.py", line 864, in run
self._target(*self._args, **self._kwargs)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\training\queue_runner_impl.py", line 254, in _run
coord.request_stop(e)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\training\coordinator.py", line 222, in request_stop
compat.as_str_any(ex))
File "D:\Anaconda3\lib\site-packages\tensorflow\python\platform\tf_logging.py", line 82, in info
_logger.info(msg, *args, **kwargs)
Message: 'Error reported to Coordinator: %s, %s'
Arguments: (<class 'tensorflow.python.framework.errors_impl.NotFoundError'>, 'Can not get size for: ./dataset/VOCdevkit/SegmentationClassAug/2008_003519.png : ϵͳ\udcd5Ҳ\udcbb\udcb5\udcbdָ\udcb6\udca8\udcb5\udcc4·\udcbe\udcb6\udca1\udca3\r\n\n\t [[Node: create_inputs/ReadFile_1 = ReadFile_device="/job:localhost/replica:0/task:0/cpu:0"]]')
Traceback (most recent call last):
File "D:\Anaconda3\lib\site-packages\tensorflow\python\client\session.py", line 1139, in _do_call
return fn(*args)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\client\session.py", line 1121, in _run_fn
status, run_metadata)
File "D:\Anaconda3\lib\contextlib.py", line 88, in exit
next(self.gen)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\framework\errors_impl.py", line 466, in raise_exception_on_not_ok_status
pywrap_tensorflow.TF_GetCode(status))
tensorflow.python.framework.errors_impl.OutOfRangeError: FIFOQueue '_1_create_inputs/batch/fifo_queue' is closed and has insufficient elements (requested 10, current size 0)
[[Node: create_inputs/batch = QueueDequeueManyV2[component_types=[DT_FLOAT, DT_UINT8], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](create_inputs/batch/fifo_queue, create_inputs/batch/n)]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "D:\PyCharm Community Edition 2017.2.3\helpers\pydev\pydev_run_in_console.py", line 37, in run_file
pydev_imports.execfile(file, globals, locals)  # execute the script
File "D:\PyCharm Community Edition 2017.2.3\helpers\pydev_pydev_imps_pydev_execfile.py", line 18, in execfile
exec(compile(contents+"\n", file, 'exec'), glob, loc)
File "D:/pyprojects/tensorflow-deeplab-resnet-master/train.py", line 254, in
main()
File "D:/pyprojects/tensorflow-deeplab-resnet-master/train.py", line 243, in main
sess.run([reduced_loss, image_batch, label_batch, pred, total_summary, train_op], feed_dict=feed_dict)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\client\session.py", line 789, in run
run_metadata_ptr)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\client\session.py", line 997, in _run
feed_dict_string, options, run_metadata)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\client\session.py", line 1132, in _do_run
target_list, options, run_metadata)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\client\session.py", line 1152, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.OutOfRangeError: FIFOQueue '_1_create_inputs/batch/fifo_queue' is closed and has insufficient elements (requested 10, current size 0)
[[Node: create_inputs/batch = QueueDequeueManyV2[component_types=[DT_FLOAT, DT_UINT8], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](create_inputs/batch/fifo_queue, create_inputs/batch/n)]]
Caused by op 'create_inputs/batch', defined at:
File "D:\PyCharm Community Edition 2017.2.3\helpers\pydev\pydev_run_in_console.py", line 82, in
globals = run_file(file, None, None)
File "D:\PyCharm Community Edition 2017.2.3\helpers\pydev\pydev_run_in_console.py", line 37, in run_file
pydev_imports.execfile(file, globals, locals)  # execute the script
File "D:\PyCharm Community Edition 2017.2.3\helpers\pydev_pydev_imps_pydev_execfile.py", line 18, in execfile
exec(compile(contents+"\n", file, 'exec'), glob, loc)
File "D:/pyprojects/tensorflow-deeplab-resnet-master/train.py", line 254, in
main()
File "D:/pyprojects/tensorflow-deeplab-resnet-master/train.py", line 141, in main
image_batch, label_batch = reader.dequeue(args.batch_size)
File "D:/pyprojects/tensorflow-deeplab-resnet-master\deeplab_resnet\image_reader.py", line 179, in dequeue
num_elements)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\training\input.py", line 919, in batch
name=name)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\training\input.py", line 716, in _batch
dequeued = queue.dequeue_many(batch_size, name=name)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\ops\data_flow_ops.py", line 457, in dequeue_many
self._queue_ref, n=n, component_types=self._dtypes, name=name)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\ops\gen_data_flow_ops.py", line 946, in _queue_dequeue_many_v2
timeout_ms=timeout_ms, name=name)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\framework\op_def_library.py", line 767, in apply_op
op_def=op_def)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\framework\ops.py", line 2506, in create_op
original_op=self._default_original_op, op_def=op_def)
File "D:\Anaconda3\lib\site-packages\tensorflow\python\framework\ops.py", line 1269, in init
self._traceback = _extract_stack()
OutOfRangeError (see above for traceback): FIFOQueue '_1_create_inputs/batch/fifo_queue' is closed and has insufficient elements (requested 10, current size 0)
[[Node: create_inputs/batch = QueueDequeueManyV2[component_types=[DT_FLOAT, DT_UINT8], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](create_inputs/batch/fifo_queue, create_inputs/batch/n)]]

I have the same problem.Are you solve it?

I have the same problem！！Are you solve it?

I solve this problem，and after training I used the saved model to predict，the outputs image are all black.I don't know why，just follow the author's ReadMe.md.

the problem was solved, thank everyone above.
the problem will solved if you change the paths of file in the file 'train.txt' to your own paths.
thanks again!!

I used the voc2012, but after training ,all of the outputs image are black!!! Im crazy .......

I have changed the paths of file in the file 'train.txt' to my own paths, but "2008_003519.png" does not exist in the voc2012.  The version I downloaded is VOCtrainval_11-May-2012.tar

No. I just download the already converted models.


preview

Hello
I just found your great mmal wrapper, thank you!!! and i would like to use it to do a simple app.
Im not able to get a preview and no errors on console, im doing something like.. but i get no preview:
Im doing something wrong?
RekkonCamControl camera;
camera.open();
camera.setStillPreviewSize(1280,720);
camera.startStillPreview();

Hello,
The start function doesn't display any image by itself. Once the preview is started, the buffer will be updated each time the camera pipeline is processed. You have to grab the image in the buffer and retrieve it. Here is how I do it :
            if (m_cam_control.grab())
            {
                if (m_cam_control.isVideoPreviewOpened())
                    m_frame->create(m_cam_control.getVideoPreviewHeight(), m_cam_control.getVideoPreviewWidth(), CV_8UC3);
                else m_frame->create(m_cam_control.getStillPreviewHeight(), m_cam_control.getStillPreviewWidth(), CV_8UC3);

                m_cam_control.retrieve(m_frame->ptr<uchar> ( 0 ));
            }

I use an opencv mat (m_frame) as opencv is a library that has display functions.

Hello
Ok, i understand, sounds great!!
Thank you


exclude traffic_ops_golang vendor dependency in rat report

rat report shows go-sqlmock is not properly licensed,  but it is documented in the LICENSE file.  Adds to .rat-excludes so rat won't report it.
@rob05c -- your first merge?

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/35/

test this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/37/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-traffic_ops-test-PR/7/


when array keys are Chinese characters, atom-alignmen is not working fine

$lang['我的帐号']  = 'My Account';
$lang['登陆']  = 'Login';
$lang['登出']  = 'Logout';
Big But !!! I does not meaning it's a issue, issue is i'm using Chinese characters as array keys.
It's better if atom-alignment can support this one
Thanks

Alignment is based on the number of characers, not the width of individual characters (since they should all be the same width in a typical monospaced font).
This issue is actually caused by the chinese glyphs being wider than any other character in a monospaced font (~1.7x in Source Code Pro at 12pt for example). It's simply not possible to properly visually align anything involving such characters.


UML-1437-change-lpa-details

Purpose
As a user I need to know what to do when I see a mistake on my LPA summary
Fixes UML-1437
Approach
Created a new route, handler, and twig template for new page. Added link to the bottom of the LPA summary to direct to the new page. Added welsh translations and a UI test
Checklist

[x] I have performed a self-review of my own code
[x] I have added tests to prove my work
[x] I have added welsh translation tags and updated translation files
[x] I have run an accessibility tool on any pages I have made changes to and fixed any issues found
[ ] The product team have tested these changes

@cooperaj  - request a review please


[FEATURE REQUEST] Add dimensions to label in app

Not sure what the default is but would be helpful to know if it's mm or otherwise in the labels if the app. Also helps to cross check when importing into a slicer
i.e.
Width (mm)

Added in 5847563

@czM1K3 thanks for taking the time to put this in!


Update README.md

Updated dead links, various grammar/spelling fixes, please review before merging.

@EthanHindmarsh thanks for the changes, appreciate you efforts.


Series created from a list of string has 'category' dtype instead of 'object'

System information

OmniSci branch: intel-go/omniscidb/gal/fix_pytests
Modin branch: intel-go/modin/omnisci-on-ray
Code we can use to reproduce:

import os

os.environ["MODIN_EXPERIMENTAL"] = "True"
os.environ["MODIN_ENGINE"] = "ray"
os.environ["MODIN_BACKEND"] = "omnisci"

if __name__ == "__main__":
    import modin.pandas as pd
    import pandas

    data = ["some", "string"]

    md_ser = pd.Series(data)
    pd_ser = pandas.Series(data)

    print("pd_series:\n", pd_ser, sep="") # dtype=object
    print("md_series:\n", md_ser, sep="") # dtype=CategoricalDtype

Output
pd_series:
0      some
1    string
dtype: object

md_series:
0      some
1    string
dtype: category
Categories (2, object): [some, string]

<!--
You can obtain the Modin version with
<p>python -c "import modin; print(modin.version)"
-->

On the latest master I'm getting the following output:
pd_series:
0      some
1    string
dtype: object
md_series:
0      some
1    string
dtype: object

Closing as not reproducible. If you could reproduce the issue, please reopen.


When I open the app, it takes few minutes for Magic Keyboard commands to work

When I open the app, it takes few minutes for Apple Keyboard commands to work properly.
When I open the app, no keyboard commands work at all. After a short moment keyboard commands work, if I hold the buttons down. After few minutes, also tapping the brightness controls work like it should. It works all the time if I change the brightness with mouse using the brightness selector.
What could fix this? I have selected the listen only brightness and start at login.
My display is LG 27UK850-W.

Same issue here. Display responds to Apple Magic Keyboard 2 input several minutes after launching the app.
My display is Dell U2417H.

Similar issue here (Mojave 10.14.4/Apple Magic Keyboard 2/ASUS PB278 display/MonitorControl 1.3.0):
The brightness gets stuck after a while. The brightness keys become useless and I have to move the menu slider to wake up the app.
Previously, MonitorControl was working great on MacOS High Sierra.

The bluetooth debug/reset seems to help a bit:
https://perfectfitcomputers.ca/wireless-magic-keyboard-special-function-buttons-not-working/

Are you sure this is related to the Magic Keyboard? Have you tried testing this with either the default macbook keyboard or a different keyboard? Because there is currently an issue where the app is unresponsive at start for specific displays that we are working on.
Also, are you getting the volume/brightness HUD and not working? Or is it just showing nothing?

Is this still an issue? Could you try lowering the polling mode in advanced settings and see if that has any effect?

Closing due to inactivity, feel free to reopen.


✨ Update README.md

Added a proper README.md file.

Coverage remained the same at 100.0% when pulling 8c6f3631803b294e3378736202f266775664c786 on AileenCGN:readme into 8aa271da842e2ae5fcb85b3e9e68a64d81bfeb17 on jbhannah:master.

Coverage remained the same at 100.0% when pulling 8c6f3631803b294e3378736202f266775664c786 on AileenCGN:readme into 8aa271da842e2ae5fcb85b3e9e68a64d81bfeb17 on jbhannah:master.


Bump versions of wasm-tools crates

Primarily also release the wit-parser and wit-component crates to crates.io.

Also, I parked wit-component on crates.io, so I'll need to transfer that over to the BCA.

Indeed! I'll wait for that to merge first before landing this.


Modified project structure

Some elements were not created in PR #1 as the standar way, among these are:

Screen directory wasn't created
types.tsx file wasn't created

@EstebanOlmedo antes de que empieces a tirar código, creo va a ser necesario darle merge a esto a main, porque hice el initial dev con las patas, meper?


Unable to use ServiceClient with embedded Graph API

There is an issue with automating an entire environment you want to deploy with Dynamics 365 ServiceClient. It will work if I manually add the app registration both at portal.azure.com and here: (https://admin.powerplatform.microsoft.com/environments/YOURENVIRONMENTIDHERE/appusers)
I have managed to automate app registration for portal.azure.com but I cannot automate the dynamics 365 registration to allow creations of entities via ServiceClient. Is there a way to create entities without ServiceClient or automate the registration itself in Dynamics 365 CRM like I can in Azure?
This is required because I am making apps that need to be able to be deployed in full with nothing setup originally. This way, people can just run my installer, and everything is setup for them.

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

ID: 746d24e7-1f0a-9760-4ded-ea1d4820b97d
Version Independent ID: 31597e5d-d181-5f0f-5cda-43e124e4f5a0
Content: Register daemon apps that call web APIs - Microsoft Entra
Content Source: articles/active-directory/develop/scenario-daemon-app-registration.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

@WayneAASPLLC, did you consider using multi-tenant applications?
I'm not familiar with the dynamic portal. do you think it uses Microsoft graph under the hood?

We want everyone to have their own instance in full.

I may have found a way to do this with HttpClient... I'll keep you posted but I am still open to suggestions.

@WayneAASPLLC
Thanks for your feedback! We will investigate and update as appropriate.

I have tried the following to mimic the working one, but it still will not POST(0 errors):
        JObject newAppUser = new JObject();

        newAppUser["accessmode"] = 4;
        newAppUser["firstname"] = "#";
        newAppUser["address2_shippingmethodcode"] = 1;
        newAppUser["setupuser"] = false;
        newAppUser["incomingemaildeliverymethod"] = 1;
        newAppUser["internalemailaddress"] = "TESTDataverseAPI_" + appID + "@" + dynamicsOrgID + ".com";
        newAppUser["identityid"] = 93;
        newAppUser["domainname"] = "TESTDataverseAPI_" + appID + "@" + dynamicsOrgID + ".com";
        newAppUser["isintegrationuser"] = false;
        newAppUser["windowsliveid"] = "TESTDataverseAPI_" + appID + "@" + dynamicsOrgID + ".com";
        newAppUser["fullname"] = "# TESTDataverseAPI";
        newAppUser["address1_addresstypecode"] = 1;
        newAppUser["displayinserviceviews"] = false;
        newAppUser["caltype"] = 0;
        newAppUser["applicationid"] = appID;
        newAppUser["azurestate"] = 0;
        newAppUser["address1_shippingmethodcode"] = 1;
        newAppUser["defaultfilterspopulated"] = false;
        newAppUser["outgoingemaildeliverymethod"] = 1;
        newAppUser["address2_addresstypecode"] = 1;
        newAppUser["lastname"] = "TESTDataverseAPI";
        newAppUser["preferredphonecode"] = 1;
        newAppUser["issyncwithdirectory"] = false;
        newAppUser["preferredemailcode"] = 1;
        newAppUser["userlicensetype"] = 3;
        newAppUser["deletedstate"] = 0;
        newAppUser["organizationid"] = dynamicsOrgID;
        newAppUser["defaultodbfoldername"] = "Dynamics365";
        newAppUser["isemailaddressapprovedbyo365admin"] = false;
        newAppUser["invitestatuscode"] = 0;
        newAppUser["yomifullname"] = "# DataverseAPI";
        newAppUser["emailrouteraccessapproval"] = 0;
        newAppUser["isdisabled"] = false;
        newAppUser["preferredaddresscode"] = 1;
        newAppUser["islicensed"] = false;

        var response = await client.PostAsJsonAsync("https://orgafac2fa3.crm.dynamics.com/api/data/v9.2/systemusers", newAppUser);

Thoughts?

I managed to get an error out of this. What is the proper POST for systemusers?
{"error":{"code":"0x0","message":"Error identified in Payload provided by the user for Entity :'systemusers', For more information on this error please follow this help link https://go.microsoft.com/fwlink/?linkid=2195293  ---->  InnerException : Microsoft.OData.ODataException: An unexpected 'StartArray' node was found when reading from the JSON reader. A 'PrimitiveValue' node was expected.\r\n   at Microsoft.OData.Json.JsonReaderExtensions.ValidateNodeType(IJsonReader jsonReader, JsonNodeType expectedNodeType)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightPropertyAndValueDeserializer.ReadPrimitiveValue(Boolean insideJsonObjectValue, IEdmPrimitiveTypeReference expectedValueTypeReference, Boolean validateNullValue, String propertyName)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightPropertyAndValueDeserializer.ReadNonEntityValueImplementation(String payloadTypeName, IEdmTypeReference expectedTypeReference, PropertyAndAnnotationCollector propertyAndAnnotationCollector, CollectionWithoutExpectedTypeValidator collectionValidator, Boolean validateNullValue, Boolean isTopLevelPropertyValue, Boolean insideResourceValue, String propertyName, Nullable1 isDynamicProperty)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightResourceDeserializer.ReadEntryDataProperty(IODataJsonLightReaderResourceState resourceState, IEdmProperty edmProperty, String propertyTypeName)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightResourceDeserializer.ReadPropertyWithValue(IODataJsonLightReaderResourceState resourceState, String propertyName, Boolean isDeltaResourceSet)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightResourceDeserializer.<>c__DisplayClass9_0.b__0(PropertyParsingResult propertyParsingResult, String propertyName)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightDeserializer.ProcessProperty(PropertyAndAnnotationCollector propertyAndAnnotationCollector, Func2 readPropertyAnnotationValue, Action2 handleProperty)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightResourceDeserializer.ReadResourceContent(IODataJsonLightReaderResourceState resourceState)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightReader.StartReadingResource()\r\n   at Microsoft.OData.JsonLight.ODataJsonLightReader.ReadResourceSetItemStart(PropertyAndAnnotationCollector propertyAndAnnotationCollector, SelectedPropertiesNode selectedProperties)\r\n   at Microsoft.OData.JsonLight.ODataJsonLightReader.ReadAtStartImplementationSynchronously(PropertyAndAnnotationCollector propertyAndAnnotationCollector)\r\n   at Microsoft.OData.ODataReaderCore.ReadImplementation()\r\n   at Microsoft.OData.ODataReaderCore.InterceptException[T](Func1 action)\r\n   at System.Web.OData.Formatter.Deserialization.ODataReaderExtensions.ReadResourceOrResourceSet(ODataReader reader)\r\n   at System.Web.OData.Formatter.Deserialization.ODataResourceDeserializer.Read(ODataMessageReader messageReader, Type type, ODataDeserializerContext readContext)\r\n   at System.Web.OData.Formatter.ODataMediaTypeFormatter.ReadFromStream(Type type, Stream readStream, HttpContent content, IFormatterLogger formatterLogger)."}}

I have gotten the error down to but just adding ToString to newappuser. PostAsJsonAsync apparently wanted it to be a string instead of a jobject:
{
  "error": {
    "code": "0x80040530",
    "message": "Unable to retrieve attribute=businessunitid for entityLogicalName=systemuser. Entity has Attribute Count=32. AttributeNames= address1_shippingmethodcode, islicensed, caltype, domainname, address1_addresstypecode, preferredphonecode, displayinserviceviews, lastname, incomingemaildeliverymethod, address2_addresstypecode, isemailaddressapprovedbyo365admin, setupuser, accessmode, internalemailaddress, userlicensetype, firstname, outgoingemaildeliverymethod, preferredaddresscode, azurestate, address2_shippingmethodcode, emailrouteraccessapproval, applicationid, windowsliveid, isdisabled, isintegrationuser, preferredemailcode, invitestatuscode, defaultfilterspopulated, systemuserid, issyncwithdirectory, deletedstate, isactivedirectoryuser"
  }
}

another one after fixing all of that:
{
  "error": {
    "code": "0x0",
    "message": "CRM do not support direct update of Entity Reference properties, Use Navigation properties instead."
  }
}

I have figured out how to fix all these errors and more, no longer needed. However, it takes around 300+ lines of code when ServiceClient should be able to handle all this. At this point, I recommend it as a feature, so the standard way mentioned in docs works as is, issue is resolved though.

I did find a way with this in ServiceClient but it is the way everyone does not recommend for some reason. I'm using username/pass, but it has a login prompt, so the password is not saved. It is very secure, and the person just needs to be a system user which every company will likely already have.


bugs in whiten features?

There might be a bug in whiten feautres.
We should pass **self.conf['whiten_params'] to the whiten_features function, rather than self.conf['whiten_params']. Ohtherwise, this parameter is actually passed to l2norm, and the parameters will not take effect.

Good catch. Given the default params, this only changes the whitenp from 0.25 to 0.5, which should have minor impact. I will reevaluate and update (the new retrieval feature anyway needs more doc).

Fixed in https://github.com/cvg/Hierarchical-Localization/commit/99e0fd88ae78acc8c09c0fbb6d61f1805b0777f5.


Poor quality finishing touches

Couple of things... the "All Products" and "Watch Video" buttons near the top of the page are not aligned. And second, the fonts at the bottom (phone number etc) are far too small (on Ubuntu at least).

@Resonanz ,
thanks for your concern. i made an update for this template, hope you like it.


Run executable tests on python -m ssort as well as plain ssort

Coverage for the coverage gods.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 97.003%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
1230

Relevant Lines:
1268

💛  - Coveralls


Add "Missing" section

First entry: Color management and HDR
Also removed stray  tag.

color management is a big deal, it should be included on the list


Dark mode fonts are blurry

When in dark mode, font is blurry, after zoom-in it looks like two fonts overlap:

Fixed for content, not header and footer:


Use Pure kube wait.go to wait for pipelinerun to finish instead of tkn logs

It currently shows like this as unkown :

porobably need elevated to have logs while we don't really need it yet, so may as well do this in pure k8s

implemented in https://github.com/openshift-pipelines/pipelines-as-code/blob/eebe6c2/pkg/kubeinteraction/wait.go


Add double and timestamp MATLAB annotations functions to the table (rebased onto dev_5_0)

This is the same as gh-1059 but rebased onto dev_5_0.

This Pull Request adds the new annotations utility functions and a link to the WriteData example to the MATLAB developer page.

--rebased-from #1059

Looks fine on staging


Loading animation for beforeAll & afterAll

Clear and concise description of the problem
Just started using Vitest and some of my tests have asynchronous preparations and cleanup functions mainly via beforeAll  and afterAll hooks that may take some time before finish, the testing reporter (or logger) freezes, I had to add console logs but it still isn't as nice as a loading indicator.
Suggested solution
Loading indicator for beforeAll and afterAll as an example:
 ❯ tests/demo/instructions.test.ts (4)
   ❯ demo instructions (4)
     [L] beforeAll hook
     · tx with echo instruction
     · tx with add instruction
     · tx with transfer instruction
     ↓ tx with donate instruction [skipped]
     · afterAll hook

 ❯ tests/demo/instructions.test.ts (4)
   ❯ demo instructions (4)
     ✔ beforeAll hook
     ✔ tx with echo instruction
     ✔ tx with add instruction
     ✔ tx with transfer instruction
     ↓ tx with donate instruction [skipped]
     [L] afterAll hook

Replace [L] with animated loading indicator
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

@antfu Contribution welcome? This sounds like a good issue to get acquainted with the codebase.

Hey! If this issue is not already taken, can I work on this please? If so, where should I begin in the code?

Hey! If this issue is not already taken, can I work on this please? If so, where should I begin in the code?

I took a brief look at this a few days ago but haven't gotten around to working on it (please, go ahead!).
You'll want to look at the reporters in the vitest package. getStateString and getStateSymbol largely handle the stdout formatting. From a cursory glance at the code, I can see that this ticket might be a bit tricky because no metadata about the hooks exists on the task objects that pass through the reporters (such was the case when I looked at the code a few days ago, that is).

I'm not able to understand how different parts of the codebase are interacting with each other. To me, it seems that its an event based architecture, creating worker pools to execute the test files. But not able to understand how the reporters module is able to access the state of these tests.
I've never worked on such a project, so forgive my ignorance. Is there a better/easier way to understand this? Are there certain tools I should know/learn?


Adding Feature 3.html and Feature 3.css

Pull Request Template

Please provide all of the details below. Pull requests missing any of the information below will be closed!

Sumarise what this PR fixes: < enter details >
Fixes #: < Enter Issue ID/URL >
Checklist:
(Add an x between the brackets to check the items)

[x] I have sumarised what this PR fixes above
[x] I have provided the issue ID/URL above
[x] This PR fixes only the issue mentioned above (create seperate PRs for seperate issues)
[x] I was the first one to claim the issue or I created the issue

Hey @aayushi02agarwal
I am not sure which [Issue Number](https://github.com/zero-to-mastery/Keiko-Corp/issues?page=2&q=is%3Aissue+is%3Aopen] this PR refers too as the ID was not provided in the description:

Managing tasks via the issues tab makes maintaining the project alot easier and ensures that we keep the issue list clean and upto date. Feel free to provide the relevant information and reopen the PR.
Regards
Matt


Implement UserState

The main contribution is to remove the LoginUser singleton, and introduce a new concept called UserState. The UserState manages the login information and state of any client (master, worker, client). There are specific implementations for UserState depending on the security type, so every implementation has its own class.

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3584/
Test FAILed.

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3585/
Test PASSed.

@bf8086 @ZacBlanco

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3586/
Test PASSed.

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3599/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3600/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3602/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3616/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3621/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/3675/
Test PASSed.

alluxio-bot, merge this please


Theme issue

flutter_paystack-1.0.7/lib/src/widgets/checkout/checkout_widget.dart:191:64: Error: The getter 'headline1' isn't defined for the class 'TextTheme'. getting this issue while running the application.

I solved with this in my pubspec.yaml
dependency_overrides:
flutter_paystack:
git:
url: https://github.com/I-driod/flutter_paystack.git
ref: master
Thank to I-driod

I solved with this in my pubspec.yaml
dependency_overrides: flutter_paystack: git: url: https://github.com/I-driod/flutter_paystack.git ref: master
Thank to I-driod

Thanks this is something I want

I solved with this in my pubspec.yaml
dependency_overrides: flutter_paystack: git: url: https://github.com/I-driod/flutter_paystack.git ref: master
Thank to I-driod

Is this works for IOS?

I solved with this in my pubspec.yaml
dependency_overrides: flutter_paystack: git: url: https://github.com/I-driod/flutter_paystack.git ref: master
Thank to I-driod

Is this works for IOS

This problem is not platform-specific; it's due to Flutter not being able to reach the package, so we explicitly provided it. If you are facing the same issues, it should work.

Got this error after adding the above code
Error on line 67, column 8 of pubspec.yaml: Invalid version constraint: Could not parse version "https://github.com/I-driod/flutter_paystack.git". Unknown text at "https://github.com/I-driod/flutter_paystack.git".
`environment:
sdk: '>=3.3.4 <4.0.0' # '>=3.3.0-273.0.dev <4.0.0'
versions available, run flutter pub outdated.
dependencies:
flutter:
sdk: flutter
amplify_authenticator: ^2.1.2
amplify_flutter: ^2.4.0
amplify_auth_cognito: ^2.4.0
amplify_storage_s3: ^2.4.0
amplify_datastore: ^2.4.0
json_annotation: ^4.8.1
flutter_paystack:
dependency_overrides:
http: ^0.13.5
intl: ^0.18.0
flutter_paystack:
git:
url: https://github.com/I-driod/flutter_paystack.git
ref: master`


Persist the watch list

The watch list that watches for changes on specific addresses should be persisted, it will be removed if user exists or restarts the extension (background process).
This means that an transaction performed might not be reflected correctly in the state of the account.

This was done with the introduction of the AddressWatchStore.


NaN Loss

There are so much NaN in loss. The train script, skip optimizing whenever NaN is in the tensor. What causes these NaN values?

When do you meet NaN? Training Stage C or Stage B?

Stage B


Add jupyterhub sizes ConfigMap

This commit moves the 'odh-jupyterhub-sizes' ConfigMap from odh-manifests
to deployer. This required so that users can configure sizes without the ConfigMap
getting overwritten by the operator
RHODS-982
odh-manifests PR: https://github.com/red-hat-data-services/odh-manifests/pull/128

Can you add a link to the corresponding jira ticket to the commit message?

@LaVLaS You good with this version?

/lgtm


float() argument must be a string or a real number, not 'Timeout'

Problem
Invoking crash without any parameters trips with that error message.
float() argument must be a string or a real number, not 'Timeout'

References
Apparently introduced by this other patch, but unnoticed so far.

GH-421

Ad hoc solution
Will yank the releases 0.31.0 and 0.31.1 from PyPI.
Real solution
Run a bugfix release.

I've tried with different versions of urllib3 (1.26.16, 2.0.7, 2.2.0), but I have not been able to reproduce the error. So, I've un-yanked the releases on PyPI again.

To work properly, it needs crate>=0.35.0, see https://github.com/crate/crate-python/commit/af27cb18aa84.

This issue has been fixed with GH-429.


uaac options parser doesn't deal with spaces in option values

My regular uaac 3.1.0 (not from this package) allows for uaac curl commands with spaces in the options, eg the space after the colon in Content-type:
uaac curl /oauth/clients -k -H "Content-type: application/json"

Running the same command from this package outputs
Too many command line parameters given.

  curl [path]                      CURL to a UAA endpoint
                                   -X | --request <method>, request method type, defaults to GET
                                   -d | --data <data>, data included in request body
                                   -H | --header <header>, header to be included in the request
                                   -k | --insecure, makes request without verifying SSL certificates

This is a problem when specifying -d request body data that may need to contain spaces.

Hmm I wonder how we support this with the uaac shell wrapper. My shell-fu isn't sure right now. Ideas?
On Mon, Jul 27, 2015 at 8:51 AM, Will Tran<EMAIL_ADDRESS>wrote:

My regular uaac 3.1.0 (not from this package) allows for uaac curl commands with spaces in the options, eg the space after the colon in Content-type:
uaac curl /oauth/clients -k -H "Content-type: application/json"

Running the same command from this package outputs
Too many command line parameters given.
  curl [path]                      CURL to a UAA endpoint
                                   -X | --request <method>, request method type, defaults to GET
                                   -d | --data <data>, data included in request body
                                   -H | --header <header>, header to be included in the request
                                   -k | --insecure, makes request without verifying SSL certificates

This is a problem when specifying -d request body data that may need to contain spaces.
Reply to this email directly or view it on GitHub:
https://github.com/cloudfoundry-community/traveling-cf-admin/issues/7

Oh I see what you mean: https://github.com/cloudfoundry-community/traveling-cf-admin/blob/master/packaging/wrappers/uaac.sh#L13

The last comment on this issue explains the need for wrapping $@ in quotes. I'll make a PR.


Adding spec for ListView and GridView design updates

You may want to consider updating your ListView examples in the new style, to also include then secondary text and avatar/PersonImage - to better reflect the same content in both examples, and also to better delineate those examples from NavigtationViewItems, which look almost identical.

@mdtauk Thanks for the feedback! I will look into creating some mockups that have similar content before and after.
As for your question about Card content, I believe that the styles proposed in this spec would handle that rather well. Note that GridView and ListView items will be receiving a rounded grey backplate on all non-rest states. For GridViews that contain full-bleed images, the backplate won't be shown. However, for card-type items I think that backplate will help reinforce their structure and shape in a more subtle way than a border. To make a long story short, I don't know of any plans for a separate card control and we tried to make sure that these new designs would support the many use cases for a GridView, including the card use case 😊.

@MikeHillberg I see that some of your feedback looks like it was left on an outdated doc. I pushed updates yesterday morning, just want to make sure you've seen the latest updates.

@MikeHillberg I see that some of your feedback looks like it was left on an outdated doc. I pushed updates yesterday morning, just want to make sure you've seen the latest updates.

@anawishnoff  I noticed you also agree with rounding the focus visuals...
https://github.com/microsoft/microsoft-ui-xaml/issues/3568

@anawishnoff  I noticed you also agree with rounding the focus visuals...
https://github.com/microsoft/microsoft-ui-xaml/issues/3568

@mdtauk Yes, for this spec I'm specifically proposing that we round corners on focus rects for ListView and GridView items to make sure they match the new rounded corners design :)

@mdtauk Yes, for this spec I'm specifically proposing that we round corners on focus rects for ListView and GridView items to make sure they match the new rounded corners design :)

@mdtauk Yes, for this spec I'm specifically proposing that we round corners on focus rects for ListView and GridView items to make sure they match the new rounded corners design :)

I think this should be the default for all FocusVisuals unless there is a specific need for there to be no corner rounding.

@mdtauk Yes, for this spec I'm specifically proposing that we round corners on focus rects for ListView and GridView items to make sure they match the new rounded corners design :)

I think this should be the default for all FocusVisuals unless there is a specific need for there to be no corner rounding.

Does the selected item need to have a darker backplate (in Light Theme) - The LayerFillColorDefault value of #09FFFFFF would appear to give a more aesthtically pleasing result, and more clearly distinguish between the Hover and Pressed states, which are more subtle now than they used to be.
I used that value in my proposal for NavigationView Changes

Does the selected item need to have a darker backplate (in Light Theme) - The LayerFillColorDefault value of #09FFFFFF would appear to give a more aesthtically pleasing result, and more clearly distinguish between the Hover and Pressed states, which are more subtle now than they used to be.
I used that value in my proposal for NavigationView Changes

@MikeHillberg Can this be merged?


[Magica Logia / inSANe] UI Modify

[Magica Logia] Character Sheet Modify
Changes / Comments
Character Sheet UI Modify.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[O ] Does the pull request title have the sheet name(s)? Include each sheet name.
[X] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.
마기카로기아 캐릭터 시트 UI 수정
기존 기능은 그대로 유지한 채, UI만 수정 되었습니다.

+inSANe Korean Language Modify


Add extra tests (to nudge coverage slightly upwards)

Current coverage is 78.39% (diff: 100%)

Merging #492 into master will increase coverage by 0.13%

@@             master       #492   diff @@
==========================================
  Files            31         31
  Lines          7213       7235    +22
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           5645       5672    +27
+ Misses         1568       1563     -5
  Partials          0          0

Powered by Codecov. Last update cbdd9b6...14768d3


[WIP] New content screen prototype

refs https://github.com/TryGhost/Ghost/issues/7860

remove preview pane from content screen
add basic post status filters
replace custom infinite scroll with ember-infinity and increase trigger threshold for improved scroll behaviour

TODO:

[x] fix tests
[ ] add tests for filters

Coverage decreased (-0.3%) to 80.913% when pulling 7b128b2700591571e956cf077e6c8027008a86ae on kevinansfield:new-content-layout into 1189c1b505e134762b3d9140e93f3567fb8a1c76 on TryGhost:master.

Coverage decreased (-0.1%) to 81.122% when pulling e06a8b721124e5b1780e4bd6bbc9bfdc6eb7ae7b on kevinansfield:new-content-layout into 1189c1b505e134762b3d9140e93f3567fb8a1c76 on TryGhost:master.

Ready for review!


ERROR: Burp Piper Extension input expected: list index out of range

When using knoxnl with the Burp Piper Extension, I encounter the error: ERROR: Burp Piper Extension input expected: list index out of range.
Attached are screenshots of my Burp Piper settings and the error message for reference.

Please help in fixing.

Sorry, I missed Check the Pass HTTP headers to command check-box point. Its working fine now. Hence, closing the issue.


update version number to string

When setting the version number to something like 2.10, it gets truncated to 2.1.
The version number should instead be handled as a string to avoid accidental truncation.

@guimou FYI


Download default Apalache release on init

As discussed in #243 , we want to download a default version of Apalache upon running atomkraft init (of course, only if it does not exist already on the system)

The intention was fixed (now we have modelator apalache info and modelator apalache get


Firebase : Read firebase Options from environment

have a function with signature
FirebaseOptions.fromEnv()

Already available, maybe just a
fun FirebaseOptions.from(map: Map<String,String>)?

created firebaseOptionFrom(map: Map<String,*>)


[meta] Deis v2 Alpha

This issue is a WIP, this notice will be deleted when everything is finalized.
Meta issue for Deis v2 alpha; as specific issues are filed in other repos, we will link them back to this issue.
Alpha Goals
Primary goal is to complete the integration of all components necessary for the Deis workflow to function. This includes git push deis master with the control plane and data planes running fully within a Kubernetes cluster.
Component pieces of the Deis workflow may be implemented as services that aren't ready for prime time, including single-node, non-ha and non-recoverable. For example, object storage may be off-cluster via S3, Google Storage, or implemented on-cluster as a single-node instance of minio.
Outcome of the alpha will be a concrete set of goals and implementation details for the Beta phase.
Non-functional requirements

Scale target: none
Availability target: none
Recoverability target: none
Dogfooding target: one internal service, long-running
Installation target: basic case via helm, manual installation ok
Infrastructure targets: AWS via coreos-kubernetes & GKE

Components
Controller and client

deis client cli + lift-and-shift python app

Database

deis/postgres#5

Etcd

deis/etcd#7

Registry

single-instance, no ha

Object storage

off-cluster, minio single-instance

Router

basic http routing to services

Builder

basic git-push functionality
slugbuilder / slugrunner / git gateway

Logger

log aggregation will be addressed in beta

sounds good for the alpha release

Released via deis/charts#54

Released via deis/charts#54

:confetti_ball:


Reschedules are not treated

We start the syncs in Fivetran mainly with the Airflow Operator of Fivetran and monitor them with the corresponding sensor. This way we can plan different DAGs depending on the syncs.
When a sync is rescheduled by the Fivetran engine, the sensor detects this. The status is also indicated on the Fivetran interface.
Extract from the Airflow log:
[2023-01-20, 03:03:11 CET] {fivetran.py:280} INFO - Connector "hepatitis_ligament": sync_state = rescheduled
Fivetran UI:

According to the entry on the interface, I expect the Fivetran engine to restart the sync at 3:07 AM. However, this was not the case. The sensor is now in a continuous loop until the next regular sync is triggered by AIrflow or someone intervenes manually.
I have already opened a Fivetran support ticket for a similar case. They said that the rescheduled sync in this case should not be triggered by the Fivetran engine but externally. In my opinion, the resync should be triggered by the Fivetran engine, otherwise it is a UI bug.

Created internal ticket for reported issue.


Feature Request - Textbox

When adding metadata fields, right now one of the types is just "Text" which is a single line of text.
I would like to utilize an option for "Textbox" that is a multi-line text box where we can add lots of information and have linebreaks.

This feature is available in the latest release


Data setup

Hi,
this seems like a very specific website design using a very specific framework (Bootstrap).
This is not what gulp-nunjucks-sass-template is about, so I'm closing the PR. If you have any questions though, feel free to reopen.


How to convert DirectionDestination / TypedDestination to Route

I want to override root destination at run time.
In startRoute parameter it accepts Route object.
I have DirectionDestination / TypedDestination.
How can I pass this in startRoute

Hmm Route is superclass of both NavGraph and Destination. So.. they’re already Route, no need to convert anything, you can just pass it in there 😅

@UKMIITB
Hope that was enough to unblock you, let me know if you need anything else, happy to help!

@raamcosta
In case if I have a destination which needs some parameter, then I am unable to convert those to Route type.
Can you please help on this


feat: allow slim_check to do work in MetaM

This opens the possibility to e.g. run norm_num inside a Testable instance.
Ideally we would actually add a Testable instance that demonstrates this. :-)
Some hurdles:

norm_num can't decide all equalities in the reals. Does it just respond with success if it can't decide? We could add an extra case to TestResult called inconclusive, but I think the code paths would be the same as for success.
alternatively, we could have a new type class MetaDecidable, for propositions with decision procedures in MetaM. There would be a lot of new design there.
Until we merge #3114 it's annoying to work on slim_check.

@semorrison what's the status of this PR? If you'd still like this to be reviewed, could you resolve the merge conflicts

@semorrison I'm going to close the PR since it has been stale for a while and it's not immediately clear to me if it would be worthwhile for somebody else to adopt it.
Please reopen it anytime, for example as "draft" or the "please-adopt" label if you feel like this would be better!


Can't bind to options since it isn't a known property

Hi,
I am getting this error when trying to use this package. Please help

Here is my code
 <ng-select [options]="countries">         </ng-select>
of course after importing it and including it in my module. One more thing worth mentioning is that the app is lazy loaded.

Here is my code
<ng-select [options]="countries">
of course after importing it and including it in my module. One more thing worth mentioning is that the app is lazy loaded.
`import {SelectModule} from 'ng-select';
@NgModule({
imports: [
SelectModule
]})
`

Anyone knows how to resolve this?

Is any body knows how to resolve this ?


Enable jdk21 reproducible comparison builds.

Close #767
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

run tests quick

alpine build just had a GPG connection issue, re-build was fine


Validate all web pages(HTML, CSS, and JS) for syntax errors.

Validate HTML and CSS for syntax errors (make sure it is compliant with the standards)
Validate JavaScript through code quality tools like JSHint and JSLint

All good.


Fix dashboard acceptance tests

Weren't up-to-date with the defined API resulting in the following errors:
# command-line-arguments
integration/dashboards_test.go:167: cannot use "warm" (type string) as type *string in assignment
integration/dashboards_test.go:192: cannot use 99.9 (type float64) as type json.Number in field value
integration/dashboards_test.go:196: cannot use 99 (type int) as type json.Number in field value
integration/dashboards_test.go:200: cannot use 99 (type int) as type json.Number in field value
FAIL    command-line-arguments [build failed]

As a result of #62 which changed the datadog.GraphDefinitionRequest to use a pointer for Style, it seemed prudent to break this out into a separate struct to make it easier for people to initialize (just fixing the above compilation errors ended up causing a null pointer panic in TestCreateAndDeleteAdvancesTimeseriesDashboard.

LGTM, and Dashboards integration tests pass 👍
=== RUN   TestCreateAndDeleteDashboard
--- PASS: TestCreateAndDeleteDashboard (84.66s)
=== RUN   TestCreateAndDeleteAdvancesTimeseriesDashboard
--- PASS: TestCreateAndDeleteAdvancesTimeseriesDashboard (83.98s)
=== RUN   TestUpdateDashboard
--- PASS: TestUpdateDashboard (64.30s)
=== RUN   TestGetDashboards
--- PASS: TestGetDashboards (64.26s)
=== RUN   TestCreateDashboardWithCustomGraph
--- PASS: TestCreateDashboardWithCustomGraph (67.98s)


Check possibility for decoder/thread reuse

First-off - very cool plugin, works off the bat and I like how well it automatically blends in with the Qt image system.
But I found it to be fairly slow.
I noticed in the debugger, that for every image being loaded, 20 threads get launched and then exit again, which seems to me an unnecessary overhead, that may cost more time than it speeds up.
20 threads is the number of logical CPUs on my PC, so that's a match.
The problem is, that Qt will create and destroy an instance of the image handler for each and every picture loaded.
So I wonder, if it shouldn't be possible to create a single, static instance of the avifDecoder and have it keep the threads throughout runtime.
From the libavif explanation of maxThreads, I can't tell how much the avif decoder will actually benefit from multiple threads, if at all. The main use seems to be to parallelize multiple images, which doesn't happen anyway in this current use-case.

Single/shared decoder instance would be a problem for animations, where the decoder exists for a longer period of time. Threads are created and destroyed by embedded libraries, I guess that re-using the instance is not going to make desired difference.
In my benchmarks on Linux, lossy-compressed AVIF often decodes faster than traditional JPG.
Performance highly depends on how libavif is built. The best performance is achieved when dav1d decoder is available and libyuv is present too.
Maximum thread limis is 64, you may lower the limit in the source.

Yes, that's sort of my point.
Your implementation probably doesn't benefit from multiple threads at all, but creates a sh*tload of them.
My scenario: I load around 100 images at once, give or take (and partially in parallel), so a total of 2000 threads are being created and terminated, 1900 of which are then discarded without ever having burned any fuel.
It seems, that when I limit maxThreads to 1 or 2, performance is best (2 seemed slightly - ever so slightly faster, but that may have been coincidence).
So I assume decoding individual images happens in a single thread anyway. There's no point in having the system puke up up to 64 threads over and over again, that will not ever be used.
My concern was basically regarding the overall overhead, not only the threads. Whether having an instance of the decoder up and running instead of loading, initializing, allocating threads, ... for every single image.


nni cannot use tensorboard

When I use NNI to conduct hyperparameter searching, I cannot use tensorboard.
As I selected the experiment and click tensorboard button, it will quickly pop up a python window (fig.1), and then the tensorboard page shows error (fig.2).
for the path in trail source code, I use
log_dir = os.path.join(os.environ["NNI_OUTPUT_DIR"], 'tensorboard')
Indeed, I can see tensorboard log file in each exp directory, but I cannot open the file via nni platform.
However, when I open terminal, and use the tensorboard from there,  open the file in the trail output dir, the tensorboard can work well.

Environment:

NNI version: v2.8
Training service (local|remote|pai|aml|etc): local
Client OS: windows
Server OS (for remote mode only):
Python version: 3.9.12
PyTorch/TensorFlow version: pytorch
Is conda/virtualenv/venv used?: conda
Is running in Docker?: no

Configuration:

Experiment config : hyperparameter tuning

Did you still have this issue? I really hope that you can try the latest version to test it.

Thank you, after I moved to Ubuntu, all the issues were disappeared.
Thanks,
Han
From: Lijiaoa @.>
Date: Monday, 8 May 2023 at 4:23 PM
To: microsoft/nni @.>
Cc: Han WANG @.>, Author @.>
Subject: Re: [microsoft/nni] nni cannot use tensorboard (Issue #5064)
Did you still have this issue? I really hope that you can try the latest version to test it.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/nni/issues/5064#issuecomment-1537954656, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASE5ULP753UUCQ5ASU36OCLXFCUOHANCNFSM56LDCQZQ.
You are receiving this because you authored the thread.Message ID: @.***>


bash syntax errors in the ctl scripts

Looks like a bash syntax error made it into the ctl helper script:

https://github.com/cloudfoundry-incubator/etcd-release/blob/master/jobs/etcd/templates/etcd_bosh_utils.sh.erb#L51

Fixed here https://github.com/cloudfoundry-incubator/etcd-release/commit/a240183ec876c48359c833d088f698e28ce5c88f


Restore integer dictionary keys dumped by RuntimeEncoder

Summary
By default json.dumps converts any integer dictionary keys to strings. json.loads, however, does not convert them back (since JSON doesn't really support integer keys). This PR adds support for restoring dictionary integer keys.
Details and comments
Fixes #42
Replaces #18

Looks like you're trying to convert all dictionaries and not just settings. This is fine, but you need to document edge cases. For example, a dictionary with mixed integer & integer-like string keys will have their string keys converted to integers. Another concern would be performance if you're traversing any dictionary twice.

@jyu00 For this ticket I actually want to handle it only for settings dict, since I am concerned with performance too when generalizing it and if we are providing a general solution then we need to handle other built-in containers too. (was trying not to delete some of Ryan's code, but it's probably not needed to fix this issue)


UMS 1.62 on Xperia Z5: bug (or feature request?)

Hello, I use a Xperia Z5C and would love to use USB MSEnabler but alas nothing in the options I alter seems to work. The drive F: shows up on Windows PC, as a Mass Storage device, all good, but it's empty when I click on it (please insert media). It's like the microSD card isn't mounted properly.
I found another app called "USB Share"[ os.devwom.usbsharereval , that needs the android device date's to be changed to jan 2017 otherwise trial period is expired :S) that allows me to mount the microSD card as a virtual image and share it as Mass Storage device, and it works. On PC I can access it as Mass Storage device. But there are 2 issues, the microSD card is mounted as read only, and having to set the time to Jan 2017 for the app to run isn't ideal either :/
What I notice though is that this "USB Share" app is mounting the microSD card as:
/mnt/media_rw/1471-1AEA
So I tried to replicate this in UMS 1.62 but I just cannot select 1471-1AEA ....
At /mnt/media_rw/ when browsing from inside UMS 1.62, the media_rw folder is just empty...
Android 7.1.1, SELinux set to permissive, UMS 1.62 has root, everything seems to be in order but it just doesn't/cannot mount my microSD.
I'll be happy to provide any detail you require / test any builds you throw at me..
Please help :)
Thanks.
edit: Attaching a file which may help making UMS 1.62 compatible with Sony Z5 family series (I believe anyway ^^). It was generated by the "USB Share" app mentionned above.
Z5C.txt

Oops, wrong command ^^;
Anyway more infos that may help pinpoint the source of the issue in the form of images attached.

Please help, Gokul..^^;

GokulNC, really could use your help..

I don't remember how  "USB Share"[ os.devwom.usbsharereval] works, but as far as I can tell, it actually creates something like a fuse mountpoint of the folder that you choose, and connects it to the file driver of Android kernel. The reason it's in read-only mount is because by default if I remember right, USB-Share app uses the CD_ROM file driver to do this.
Referring to the txt file that you attached, it's probably /sys/devices/virtual/android_usb/android0/f_cdrom/lun which my app isn't display. Referring to the 2nd screenshot that you have attached, in the top right corner, click the 3-dots menu and check if there's an option like "Display All options". This should list all the LUNs that you can use.
Now try the CD-ROM if it turns up there. If it works, good. If it doesn't I'm sorry I can't help.
It's been almost 3-4 years since I have stopped the development of this app. It's so long that I almost forgot Android development. And BTW, I'm sorry for my really late reply.


Change agent user context from "ddagentuser" to "LOCAL_SERVICE"

What does this PR do?
Use the built in account "LOCAL_SERVICE" instead of self-made account "ddagentuser".
Motivation
Self made account creation has proven less reliable that we'd like
Additional Notes
Anything else we should know when reviewing?

Superceded by https://github.com/DataDog/datadog-agent/pull/4179
Had diverged too far from master for clean PR


inv import-db does not shut down nautobot service before importing

Environment

Python version:  3.11
cookiecutter-nautobot-app template version:  Latest
database: psql

Observed Behavior
IF nautobot service is running and you run inv invoke-db psql raises an error due to an active connection from the nautobot service

Expected Behavior
All services are stopped prior to import and import is successful

Steps to Reproduce

inv start
inv import-db

Self inflicted pain


Update to Swift 3

The goal is to update the repo to work with Swift 3 and to adopt the cosmetical changes in Swift 3 (e.g. camelCase enum values).
Please drop a comment if you are interested so others know that you are working on this issue.
Have fun coding! 😄

on it!

Great :)

@felton If you change your mind let me know; I'll take a stab at it.

Since @serejahh is apparently not working on fixing the PR #2 feel free to give it a shot @felton @rudedogg


Adding vagrantfile and modify .gitignore to include .vagrant

Creating a Vagrantfile for a working Vagrant environment with file sharing.
add .vagrant to git ignore.
fix https://github.com/redhat-beyond/Xpense/issues/1

Spelling mistake in commit: "and modife"
Also, adding some details to the commit message about the content of the Vagrant file (or why you ignore .vagrant), might be helpful for reviewers to review your change.

when your commit msg include the word "and" it means your commit is not atomic. (ie - doing mulitple things) and in most cases (like this) it is better to split the commit

Spelling mistake in commit: "and modife" Also, adding some details to the commit message about the content of the Vagrant file (or why you ignore .vagrant), might be helpful for reviewers to review your change.
+1
in general the commit msg should explain the why (the what can be observed from the diff..)

@ronyyosef I took a look on the PR, nice work rony 👍

@ronyyosef Nice work Rony.
Please pay attention to Yuval's comment, it is recommended to split the PR when you use the word "and" in order to describe the commit.

Opening two PRs

Sorry, I deleted my comment because I miss read Yuval's comment. He meant two commits and not two PRs.
If you already did the change, leave it.

Hey @adiabramovitch , I already closed this PR.
That was really funny to do a pr with one line of git ignore

for future reference, you dont need to close the PR, just update it.


USB methods disable interrupts for too long(?)

I'm working on a SAMD21 based USB device, which does the USB work in the USB ISR, along the lines of the usb_echo example.  The USB interrupt is set to a lower priority (higher number) than another interrupt which needs to be serviced fairly quickly, but sometimes that doesn't happen quickly enough.  The issue seems to be that UsbBus's methods do all their work in interrupt free contexts.  For example, these.
Might it make sense for UsbBus to have an "in use" flag in the mutex rather than the RefCell, so that the interrupts only need to be disabled for setting/checking that flag?

Can you elaborate on 'too long'? We tried to do as little as possible in the UsbBus inner implementation.

The USB interrupt is set to a lower priority (higher number) than another interrupt which needs to be serviced fairly quickly

Depending on how many priority bits your chip has, could you boop the USB to have less priority than your other CSR?

Might it make sense for UsbBus to have an "in use" flag in the mutex rather than the RefCell, so that the interrupts only need to be disabled for setting/checking that flag?

I'm not sure i follow, can you give me an example?

Can you elaborate on 'too long'?

Maybe more importantly than the specific timing constraint, I was surprised to find that the entirety of read() and write() is in an interrupt free block since they call copy_to_nonoverlapping().  Isochronous endpoints can have up to 1023B per read/write, so there would be a pretty wide range of time that interrupts will be disabled.
Specifically, I've got about a 16us window to respond to a DMA interrupt, and I believe that response usually takes about 3us so I should be able to tolerate about 12us of interrupts being disabled.  Admittedly I haven't put much effort in to profiling - an early test showed a case where the DMA interrupt wasn't handled when the processor was in the USB ISR, so I went digging to see how that could happen...

could you boop the USB to have less priority than your other CSR

Yep, I'm pretty sure that is the current arrangement, but perhaps got something wrong.  Will investigate further.

I'm not sure i follow, can you give me an example?

Sorry, what I wrote was a bit unclear, and thinking about it more I may be misunderstanding the point of the mutex.  Will try to catch you on matrix to discuss, but for completeness I was imagining adding an AtomicBool field to UsbBus to indicate that work is being done on the RefCell contents.  It might look something like below, where swap is along these lines:
fn write(&self, ep: EndpointAddress, buf: &[u8]) -> UsbResult<usize> {
    if swap(self.busy_flag, true) {
        // self.inner was already in use
        Err(ComputerSaysNo)

    } else {
        // We have atomically set busy_flag, so are the only user of self.inner

        // write() proceeds with interrupts enabled
        let result = self.inner.borrow().write(ep, buf);

        // release the flag
        swap(self.busy_flag, false);

        result
    }
}

Ooooh I have an idea. Instead of turning off interrupts globally, what if we use mask and unmask to just turn off the IRQ lines for USB (USB_OTHER, USB_TRCPT0, and USB_TRCPT1on samd5x, just one on d21 iirc).
That way we can be happily preempted by more urgent interrupts while servicing, as long as they dont take too long that we miss a frame.

Something like this maybe:
struct UsbCS {
    other_active: bool, // USB_OTHER irq
    trcpt0_active: bool, // USB_TRCPT0 irq
    trcpt1_active: bool, // USB_TRCPT1 irq
}

impl core::ops::Drop for UsbCS {
    fn drop(&mut self) {
        use cortex_m::peripheral::NVIC;
        use crate::target_device::interrupt;
        unsafe {
            if self.other_active {
                NVIC::unmask(interrupt::USB_OTHER);
            }
            if self.trcpt0_active {
                NVIC::unmask(interrupt::USB_TRCPT0);
            }
            if self.trcpt1_active {
                NVIC::unmask(interrupt::USB_TRCPT1);
            }
        }
    }
}

impl UsbCS {
    fn mutex_cs(&self) -> cortex_m::interrupt::CriticalSection {
        unsafe { cortex_m::interrupt::CriticalSection::new() }
    }
}

/// Disables IRQ lines related to USB, returning a token that resets the
/// masking state when dropped.
fn usb_critical_section() -> UsbCS {
    use cortex_m::peripheral::NVIC;
    use crate::target_device::interrupt;

    let cs = UsbCS {
        other_active: NVIC::is_enabled(interrupt::USB_OTHER),
        trcpt0_active: NVIC::is_enabled(interrupt::USB_TRCPT0),
        trcpt1_active: NVIC::is_enabled(interrupt::USB_TRCPT1),
    };

    NVIC::mask(interrupt::USB_OTHER);
    NVIC::mask(interrupt::USB_TRCPT0);
    NVIC::mask(interrupt::USB_TRCPT1);
    cs
}

That way we can be happily preempted by more urgent interrupts while servicing, as long as they dont take too long that we miss a frame.

Is this the reason we're currently disabling interrupts?  If so, it probably hasn't been working well in uses like this
#[interrupt]
fn USB() {
    usb_bus.poll(&mut [a_usb_device]);

    // this is a window for other interrupts

    a_usb_device.do_some_reads_and_writes();  // likely contains windows for other interrupts
}

I guess we'll want a method along the lines of interrupt_free() that uses a usb_critical_section, maybe that should be in UsbBus?
#[interrupt]
fn USB() {
    usb_bus.critical_section(|cs| {
        // Would poll(), read(), write(), etc need a reference to cs, the UsbCS?
        usb_bus.poll(&mut [a_usb_device]);
        a_usb_device.do_some_reads_and_writes();
    }
}

Is this the reason we're currently disabling interrupts?

I think it was just to prevent us being interrupted by a different USB interrupt while servicing a USB interrupt.
I don't think there is anything wrong with being preempted by a higher priority interrupt as long as its not USB-related, but theres only one way to find out xD

That way we can be happily preempted by more urgent interrupts while servicing, as long as they dont take too long that we miss a frame.

Thinking about this more - I don't know if it's really what is needed.  Imagine there's a GPIO interrupt that triggers a USB write - we wouldn't want it to stomp on an in-progress write...
I've just yanked out the interrupt::free(), Mutex, and RefCell - it seems to work this state on SAMD21 (which only has one USB interrupt) without doing any masking, so gets around my immediate problem.  This is obviously sketchy though, there still needs to be a mechanism to only allow one "thread" from accessing the UsbBus at a time.
Since the methods where we had interrupt::free() are implementing usb-device, I guess we can't make them require a CriticalSection.  In case of contention, should the second call to a UsbBus method block, or return an error - maybe InvalidState?

I've just yanked out the interrupt::free(), Mutex, and RefCell - it seems to work this state on SAMD21 (which only has one USB interrupt) without doing any masking, so gets around my immediate problem

Can you share the diff somewhere? Im curious how you did this

Sure, https://github.com/ianrrees/atsamd/tree/usb-interrupting .  It's not pretty :).

seems to work ... without doing any masking

This was premature, it definitely needs a mechanism to prevent USB interrupts while already processing one.

I think there's a fundamental issue here, but feel a bit out of my depth.
Imagine that we have an implementation of mutex that allows work on the inner struct without disabling interrupts for the duration of that work.  When a method tries to access the inner struct through that mutex, the mutex will either have to block or return an error.  Since we don't have an OS, the blocking option doesn't make sense (the interrupted code that holds the lock won't make progress until the interrupting code is done), so the locking attempt must return an error.  But, we don't have a way to get that error out of several methods required by usb_device that need exclusive access to the inner struct, for instance reset().
A secondary question is how to handle an interrupt that fails to get the lock.

Sorry for my lack of follow-through on this, the above hack is working well enough in the meantime ;).

This was premature, it definitely needs a mechanism to prevent USB interrupts while already processing one.

Can't remember what prompted me to write this, but think it might be incorrect; AFAICT any particular ISR on SAMD21 won't ever interrupt itself.
I'm becoming convinced that although the mutex in the UsbBus implementation may be necessary in the current implementation of usb-device, with some hopefully-straightforward changes to usb-device (see https://github.com/mvirkkunen/usb-device/issues/9) it won't be.  Currently, a typical use-case involves multiple structs that refer to the same UsbBus, but if we instead had one struct that owned all the USB stuff, then Rust's ownership guarantees should be sufficient to ensure that only one execution context can access it at a time.
There might still be a mutex required to mediate access to the USB stuff, but it would be easier to implement one that only blocks interrupts for a few instructions, for example.

dirbaio on Matrix suggested that the stm32-usbd doesn't have this particular issue, perhaps it's possible to make UsbBus and/or Inner not by Sync, so that the user either picks their own mutex or only accesses the USB from one particular context.

@ianrrees, did you settle on a solution here? Does this need a change to the HAL or a change upstream?

Currently I'm using a branch that removes the mutex/critical section (CS), it works but isn't ideal.  Some possible ways we could resolve this, all involve changes to the HAL:

usb-device changes along the lines of point 2 at https://github.com/mvirkkunen/usb-device/issues/9 , which I think would then mean the HAL implementation wouldn't need the CS to begin with.
It might be possible to push the CS down closer to the hardware, so that the external API stays the same but there are only a couple instructions at a time in a CS.  From memory, this would be a fairly involved refactoring.
Make the mutex implementation configurable via feature flag, and provide one that either takes the mutex or panics.  I've found it's really not difficult to only access the USB methods from a USB interrupt context, and would expect that a bad access pattern with a lock/panic mutex would result in a panic pretty quickly.  So, this isn't really the cleanest solution, but it might be a good enough "pressure relief valve" for situations where people need to use USB along with lower-latency interrupts.

for 3, you can check if you're in interrupt context with this cortex-m function and panic if you're in thread context

I think option 3 is the best at the moment. Should be fairly simple and that way you either access USB methods from different context safely or get maximum performance at the cost of panics if it's used from multiple contexts (which I think wouldn't be too difficult to avoid). Maybe mutexes should be opt-out to make the safer option the default.
Another option would be to let the user provide a mutex with the current one as the default.

@ianrrees is there any update needed to this issue as a result of the recent USB PRs? Or is it the same story

Recent changes are unrelated, I was thinking about picking this back up though...
Any opposition to the approach in 3 from a few comments back?

Nope, I think it makes good sense


Instructions alignment

Describe the bug
After signed in webpage instructions should be aligned

LGM SOC 21

@PrathmeshMutke I don't think, we should the font style for Instructions page, fonts for all page should be similar, as its looks good and uniform.


docs(readme): add npmjs link

Coverage remained the same at 100.0% when pulling 0b314caf85d05f4b3806e90134e95fc3bcd79049 on curbengh:npmjs into 8bcc617e7189b235b52718c777427c49e0641479 on hexojs:master.