chore: AI chat window UI changes

Description
Contains cosmetic changes to AI window.

Removes AI window dimensions configuration from editor components
Disables "Ask AI" slash command in query/API editor pages.

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Chore (housekeeping or task changes that don't impact user perception)

How Has This Been Tested?

Manual

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=6ad2e76

/build-deploy-preview env=release

/ok-to-test sha=6ad2e76

/ok-to-test sha=906c5b9


Run basic emulator tests on Travis/Linux

@tvolkert out of curiosity, did you find a good set of flags for creating a device? I'd like to extend flutter emulators to support creating but all the possible flags look like a minefield (ideally I'd like to just create a "standard device" without the user having to provide any real settings).


Open With not working for scripts

When I "open with" a shell script with Bash, I get this error message:
/bin/bash: D:\Path\To\Scripts\dostuff.sh: No such file or directory

Which is understandable, because Linux uses a different way to indicate paths.
Expected results
I'd expect it (not sure what "it" would be) to convert the supplied path to a path that is usable for bash, e.g.
D:\Path\To\Scripts\dostuff.sh -> /mnt/d/Path/To/Scripts/dostuff.sh
Steps to repro

Make sure you've got a shell script (a .sh file). Any will do.
Right click -> Open With -> Choose another app
More apps -> scroll down -> Look for another app on this pc
Type C:\Windows\system32\bash.exe (or whereever your Windows install is, you know the drill)

Now Bash will open very briefly with the above error message. Steps 2 to 4 can now be replaced with a simple doubleclick on a shell script, still resulting in the same error message.
Windows 10.0.14393 x64

You need to do bash.exe -c, I believe.

@fpqc -- I believe that's true but I don't think it's sufficient?  Don't you also need to pass in a valid Linux path?

@aseering bash.exe does automagic path translation iirc.

@fpqc Relative paths should work. We do not translate absolute paths in bash.exe

@benhillis yeah, that was my experience when I tested this out to get it working for a stackexchange question.

I tried playing with this the other day. I've made a VBS script that can act as a drop-target from Explorer for a bash script, though it would be quite nice if the bash.exe program itself would recognize a Windows path (beyond relative) and convert it.
I tried various ways to drop a script onto a shortcut in Explorer (for bash.exe -c "somescript"), to pass the dropped path along to "somescript" to process; but I couldn't find a way to do it. I think bash.exe is only allowing a certain number of arguments through into the WSL environment, at least from dropping onto a shortcut. I believe if the argument is inside the quotes ( -c "somescript argument") it is passed along, but when dropping onto a shortcut it would end up appended after the -c quotes, bash.exe doesn't pass it in, and "somescript" never gets the argument. Maybe a Windows guru can tell me how to make a desktop shortcut that will pass the argument?
(Does that make sense?)
Anyway, the use case of making a shortcut to run one specific script in bash.exe is straightforward: figure out what the linux path would be, and use it with -c.  But the use case for dropping a script onto a shortcut (or using it with Send to, or even Open) does not seem to be possible at this point.

@benhillis I'm sure relative paths work, but that's just not how "open with" works in Windows. It always supplies a full path.
No matter how this issue is twisted, in the end something needs to translate a Windows-path to a Linux-path.
So! I made a crude script that does exactly that. Like the VBscript thing @rodrymbo made, I made a CMD script, which is about 100% less likely to set off virus alerts :)

@thany I understand that.  Currently bash.exe takes its command line arguments and passes them to /bin/bash as-is (with the exception of the ~ argument which must be the first argument).  I think supporting Open-Wish (perhaps even registering bash.exe as a "handler" for .sh files) is a good suggestion and can look into what changes we'd have to make to support that.

For Everyone who has the same problem, I found a way with regedit instead of a script.
Change (Standart) in
HKEY_CLASSES_ROOT\Applications\bash.exe\shell\open\command
to
"C:\Windows\System32\bash.exe" -c "$(wslpath '%1')"
Then select .sh file in explorer an choose open with: "C:\Windows\System32\bash.exe"

Now all that Microsoft needs to do is add this as a default context menu action for .sh files if WSL is installed.


Small spelling: artefact to artifact

thank you!

Sure. Since it changes the artifact directory name, do make sure that if you have any config in AppVeyor that you are aware!

The directories are defined in separate parameters file: https://github.com/AMVSoftware/NSaga/blob/master/parameters.cake#L4 I could not push into this branch, so quickly fixed it in master -)

Gotcha, not sure how I missed that. Thanks!


Missing weights-py3.p file

I don't think this was posted before, but the weights-py3.p file was missing when I pip installed Instamatic. This prevents data acquisition: on attempting to collect data, the following error is returned:
**Traceback (most recent call last):
File "c:\python36\lib\site-packages\instamatic\gui\gui.py", line 71, in run
func(self, kwargs)
File "c:\python36\lib\site-packages\instamatic\gui\red_frame.py", line 110, in acquire_data_RED
from instamatic.experiments import RED
File "c:\python36\lib\site-packages\instamatic\experiments_init_.py", line 1, in
from .autocred import experiment as autocRED
File "c:\python36\lib\site-packages\instamatic\experiments\autocred\experiment.py", line 29, in
from instamatic.neural_network import predict, preprocess
File "c:\python36\lib\site-packages\instamatic\neural_network_init_.py", line 1, in
from .neural_network import predict
File "c:\python36\lib\site-packages\instamatic\neural_network\neural_network.py", line 6, in
with open(Path(file).parent / 'weights-py3.p', 'rb') as p_file:
FileNotFoundError: [Errno 2] No such file or directory: 'c:\python36\lib\site-packages\instamatic\neural_network\weights-py3.p'
Not sure why this file is not downloaded with the package but it can be found here. So, a simple fix is to download the weights-py3.p file and put it in the appropriate folder.

Thanks, apparently this only shows up in the distribution from pypi, and not when installing the package locally. Turns out this was because of a typo in manifest.in.
Fixed by https://github.com/instamatic-dev/instamatic/commit/a0fdbca20da44e82c4a75edb706cb6050fe80810


New logo/icon proposal

Good day sir. I am a graphic designer and i am interested in designing a logo for your good project. I will be doing it as a gift for free. I just need your permission first before I begin my design. Hoping for your positive feedback. Thanks

Hey! This will be great!!!
Sure.

Hi sir, I have design logo for this project. what do you think ? do you like it ? which one do you like it ? let me know if there is something you want to change.

how about the logo. I really hope for a positive response from you.


Having trouble configuring quill editor

When I give a property option of 'type: richtext' and I try and pass along toolbar options: example:
answer: {
type: 'richtext',
custom: {
modules: {
toolbar: [['bold', 'italic'], ['link', 'image']],        <--- These options do not seem to be getting through
},
},
},
I am trying to add image functionality to the toolbar but its not working.
It doesn't seem to be taking my quill custom options when compiling or bundling. The only way for me to get past this is to go into node_modules/admin-bro/lib/frontend/assets/scripts/app-bundle.development.js and edit this file directly to get the result that I want which does not seem ideal for production.
I was wondering if maybe there is a way I need to rebundle admin bro or recompile somehow? I have tried uninstalling and reinstalling adminbro with npm and still having the issue.
Any suggestions? Thanks!

closing as it duplicates https://github.com/SoftwareBrothers/admin-bro/issues/603


Query: 5-methyltetrahydropteroyltriglutamate-homocysteine S-methyltransferase activity (GO:0003871)/Methionine synthase ?

Term to be merged: Provide GO ID and term label

GO:0008705 Methionine synthase
methionine synthase activity
Catalysis of the reaction: (6S)-5-methyl-5,6,7,8-tetrahydrofolate + L-homocysteine = (6S)-5,6,7,8-tetrahydrofolate + L-methionine.
appears to be the same as
GO:0003871 5-methyltetrahydropteroyltriglutamate-homocysteine S-methyltransferase activity (GO:0003871)
Catalysis of the reaction: 5-methyltetrahydropteroyltri-L-glutamate + L-homocysteine = L-methionine + tetrahydropteroyltri-L-glutamate.
BOth terms are used to annotate "methionine synthase"

CHAT -GPT
Yes, the reaction you described:
5-methyltetrahydropteroyltri-L-glutamate + L-homocysteine → L-methionine

tetrahydropteroyltri-L-glutamate 5-methyltetrahydropteroyltri-L-glutamate+L-homocysteine→L-methionine+tetrahydropteroyltri-L-glutamate

is catalyzed by the enzyme methionine synthase.
Methionine synthase is responsible for the transfer of a methyl group from 5-methyltetrahydrofolate (a form of folate, also referred to as 5-methyltetrahydropteroyltri-L-glutamate) to L-homocysteine, resulting in the formation of L-methionine and tetrahydrofolate (tetrahydropteroyltri-L-glutamate).
( I need to look further into this but I don't want to lose my place!)

Which term do you propose this term merge into? Provide GO ID and term label

Reason for merge

Reference(s)
PMID:nnnnnn

Any other information

GO:0003871 5-methyltetrahydropteroyltriglutamate-homocysteine S-methyltransferase activity
Catalysis of the reaction: 5-methyltetrahydropteroyltri-L-glutamate + L-homocysteine = L-methionine + tetrahydropteroyltri-L-glutamate.
GO:0008705  methionine synthase activity
Catalysis of the reaction: (6S)-5-methyl-5,6,7,8-tetrahydrofolate + L-homocysteine = (6S)-5,6,7,8-tetrahydrofolate + L-methionine.

Actually they both have Rhea,
RHEA | 11172 GO:0008705 methionine synthase activity
need to figure out  if methionine synthase (yeast) performs both reactions

CHATGPT
The term "methyltetrahydropteroyltriglutamate-homocysteine S-methyltransferase" is more specific about the substrate involved (polyglutamated form), while "methionine synthase" is a broader term encompassing all enzymes that catalyze the conversion of homocysteine to methionine using any form of 5-methyltetrahydrofolate
So I am still not sure how to resolve this. It seems that
GO:0003871 5-methyltetrahydropteroyltriglutamate-homocysteine S-methyltransferase activity
is a type of
GO:0008705 methionine synthase activity?
I don't feel confident to resolve this  correctly so I'm going to unassign myself.

GO:0003871 5-methyltetrahydropteroyltriglutamate-homocysteine S-methyltransferase activity XREF EC:<IP_ADDRESS>, RHEA:21196
GO:0008705 methionine synthase activity XREF EC:<IP_ADDRESS>, RHEA:11172
The source of CH3 appears different. Need sources to backup ChatGPT, I think.

More background:
Background:
At PomBAse, historically we  annotated methionine synthase/homocysteine methyltransferase
https://www.pombase.org/gene/SPAC9.09 Met26
using
GO:0003871 5-methyltetrahydropteroyltriglutamate-homocysteine S-methyltransferase activity
Definition
Catalysis of the reaction: 5-methyltetrahydropteroyltri-L-glutamate + L-homocysteine = L-methionine +5-methyltetrahydropteroyltri-L-glutamate.
and GO:0003871 is used in my model:
http://noctua.geneontology.org/workbench/noctua-visual-pathway-editor/?barista_token=sasgrjd2nx44ugmbfn3w&model_id=gomodel:66a3e0bb00001342
Then I came across
https://www.ebi.ac.uk/QuickGO/term/GO:0008705 methionine synthase
Catalysis of the reaction: (6S)-5-methyl-5,6,7,8-tetrahydrofolate + L-homocysteine = (6S)-5,6,7,8-tetrahydrofolate + L-methionine.
so I wondered why this was not used instead? It seems that I could use either, I was not sure if I should annotate to both reactions?
apparently the difference is
(6S)-5-methyl-5,6,7,8-tetrahydrofolate (5-MTHF/L-methylfolate.): This is the simplest form of the reduced, methylated folate molecule with a single glutamate residue attached.
5-methyltetrahydropteroyltri-L-glutamate: This is a polyglutamated form of 5-MTHF with three glutamate residues attached. The additional glutamate residues are added by the enzyme folylpolyglutamate synthetase, increasing the molecule's retention and functionality within cells.
The other difference I have found is that
https://enzyme.expasy.org/EC/<IP_ADDRESS> (methionine synthase) is cobalamin-dependent
but
https://enzyme.expasy.org/EC/<IP_ADDRESS> 5-methyltetrahydropteroyltriglutamate--homocysteine S-methyltransferase (Alternative name cobalamin-independent methionine synthase) is cobalamin independent.
Yeast cobalamin synthase is not cobalamin-dependent
https://www.yeastgenome.org/reference/S000086090
so I  think the original term I am using is the correct one.
However, these EC/<IP_ADDRESS> / EC/<IP_ADDRESS> are both "methionine synthase" and should have a grouping term in GO.
(It might even be clearer to call GO:0003871 5-methyltetrahydropteroyltriglutamate-homocysteine S-methyltransferase activity  "cobalamin-dependent methionine synthase" as a primary term label)
The current arrangements is difficult to figure out from a curation perspective:

Another problem is excessive grouping terms, both parents of these 2 terms are single inheritance, even though the biologically relevent grouping term is missing.


Add placeholders for dataset overviews

Description
Checklist
Please confirm that this pull request has done the following:

[x] Documentation added (where applicable)
[x] Changelog item added to changelog/

@durack1 @vnaik60 what do you reckon?

Relevant pages for doc reviews:

https://input4mips-controlled-vocabularies-cvs--146.org.readthedocs.build/en/146/usage-data-user/
https://input4mips-controlled-vocabularies-cvs--146.org.readthedocs.build/en/146/dataset-overviews/

Couldn't hurt to point out that we also have the CMIP6-era datasets available alongside the CMIP6Plus-era datasets.. Makes me think about how relevant it is to provide the historical context of these data and their evolution


Resolve MyPy errors in Cosmos pre-commit

Description
Resolves all MyPy pre-commit errors. "type: ignore" comments used sparingly and only when needed, i.e. a MyPy solution currently does not exist or deeper code changes need to be made.
Closes: #301
Checklist

[ ] I have made corresponding changes to the documentation (if required)
[ ] I have added tests that prove my fix is effective or that my feature works

@abhi12mohan, thank you very much for addressing the feedback and creating #382 to follow up on the issues that were ignored in this initial support to MyPy.
Once we rebase and resolve the conflicts, we can merge this PR!


kiss: implement tsort-based dependency resolution

Fixes: #29
Fixes: #30
Superseded: #32

@E5ten @cemkeylan @aaronNGi @dilyn-corner

Amazing work. I can't really comment on the algorithm but the syntax and style looks good. Good to merge in my opinion.

When a build package already exists, and it is going to rebuild, the missing (make)dependencies are not resolved.
E.g for building xorg-server the package xtrans is not installed.

Amazing work. I can't really comment on the algorithm but the syntax and style looks good. Good to merge in my opinion.

Thanks! For now, it's just a draft. There are some things need to be fixed before it can be merged.

When a build package already exists, and it is going to rebuild, the missing (make)dependencies are not resolved.
E.g for building xorg-server the package xtrans is not installed.

Thanks for testing! Will work on it.

Closed in favor of Dylan's solution.


Expose Body::GetBodyCreationSettings

It would make it easy to copy bodies between multiple physics systems.

I added it.


Implement activeChain module

Coverage increased (+0.9%) to 39.509% when pulling 0370abbf208b4335fb8cfa7c8c1da32813054c49 on switching-chains into 40173cb845a4fb201d33aabe79d96a49d4c77687 on dev.


style: rename functions to make them more readable

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

Project coverage is 87.79%. Comparing base (8b1dfe6) to head (ec24976).

:exclamation:  There is a different number of reports uploaded between BASE (8b1dfe6) and HEAD (ec24976). Click for more details.
HEAD has 5 uploads less than BASE

Flag
BASE (8b1dfe6)
HEAD (ec24976)

6
1

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
- Coverage   96.52%   87.79%   -8.73%
==========================================
  Files          14        4      -10
  Lines        1438      377    -1061
==========================================
- Hits         1388      331    -1057
+ Misses         39       35       -4
  Partials       11       11

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


Knots Conversion

According to this website 1 knot is 1.852 km/h but this library is using 1.852 to convert knots to miles, and something else for km/h:
    ...
    case gps_speed_kph:     return (gps_float_t)(sik * 0.5144L);
    case gps_speed_mps: return (gps_float_t)(sik * 1.852L);
    ...

Confirmed, fixed, thanks. Values are opposite here.


Add Bitbucket example

Currently there is no precise bitbucket example (unlike others like GH and Gitlab)
Ill try to work on this myself as im currently doing this integration

Thanks @DrFaust92 - much appreciated! I suppose that can be added to this page.
In case it's helpful for the Atlantis integration, we have a bitbucket pipeline example but really infracost comment bitbucket is the only interesting part of it.


dev express dx-drop with angular2

I'm trying to port this example to angular2
http://plnkr.co/edit/aMAHCZC2hDwgABHu2CmW?p=preview.
But the event dx-drop

 or
``
doesn't fire.
The drag work, i can bring the appointment outside the scheduler.
All the handler of the scheduler work:
`onAppointmentRendered(e) {
var appointment = e.appointmentElement;
var draggableAppointment = appointment.dxDraggable("instance");
draggableAppointment.option("area", "body");
    e.appointmentElement.on("dxdragstart", function (evt) {
        this.appointmentData = e.appointmentData;
        e.element.find(".dx-scrollable-content").css("overflow", "visible");
        e.element.find(".dx-scheduler-work-space").css("overflow", "visible");
    });

    // return overflow: hidden
    e.appointmentElement.on("dxdragend", function (evt) {
        e.element.find(".dx-scrollable-content").css("overflow", "hidden");
        e.element.find(".dx-scheduler-work-space").css("overflow", "hidden");
    });
}`

Here my destination box:

Drop items here
       <repeater [repeaterArray]="movedBooliList" class="slotTimeMainWrapper">         <ng-template let-row="item">           <app-book [showtime]="row"></app-book>         </ng-template>       </repeater>

Here the handler in the momponent:
dropped() {         debugger;     }
How could i solve?

Events like dxdrop, dxdragstart, dxdragend are jQuery events. You need to handle them using the jQuery on method.

This issue is closed because of being inactive. If you need further assistance, leave a comment here.

Is possible to implement drag & drop from external source to scheduler in angular2+?


Support uploading caption files

Would it be difficult to add support for uploading caption files?

We can do that, based on the documentation
https://developers.google.com/youtube/v3/docs/captions/insert


Model for llama-3-8B, EP: cpu, precision: int4 generated using onnxruntime-genai/src/python/py/models/builder.py has issues

I have used llama-3-8B on hugging-face model to generate ONNX model using builder.py in onnxruntime-genai. When I try use that model and run onnxruntime-genai\examples\python, I am getting this error:
onnxruntime_genai.onnxruntime_genai.OrtException: Load model from .\llama-3-8B_cpu_int4\model.onnx failed:Invalid model. Node input '/model/layers.0/attn/k_proj/repeat_kv/Transpose_2/output_0' is not a graph input, initializer, or output of a previous node.

This is the same error as this issue. Can you try the steps specified there? Alternatively, you can upgrade to the latest RC version and try again.

In the earlier case (issue #459), I have generated Gemma model using  onnxruntime_genai.models.builder. Because that is giving error and you suggested doing onnxruntime-genai/src/python/py/models/builder.py, that fixed that issue, I have used your suggestion. This is for Llama. It is giving the error reported.

The fix is the same as the linked issue. If you substitute google/gemma-2b-it with meta-llama/Meta-Llama-3-8B and re-generate the ONNX model using the "from source" version, the error should go away.
Example:
# Your original command (i.e. "from wheel" version):
$ python3 -m onnxruntime_genai.models.builder -m meta-llama/Meta-Llama-3-8B -o ./llama3_8b -p int4 -e cpu

# New command to try (i.e. "from source" version):
$ git clone https://github.com/microsoft/onnxruntime-genai
$ cd onnxruntime-genai/src/python/py/models/
$ python3 builder.py -m meta-llama/Meta-Llama-3-8B -o ./llama3_8b -p int4 -e cpu

With the version of ONNX Runtime GenAI that you have installed, this error will keep happening when trying to generate any INT4 CPU model where num_attention_heads != num_key_value_heads using the "from wheel" command. The error has been fixed by this PR. Once the latest RC version is released (0.2.0rc7), you can upgrade your ONNX Runtime GenAI version and try the "from wheel" command again.

@jmopuri Can you try with onnxruntime-genai version 0.2.0?

@jmopuri I'll mark this as completed. Please comment/reopen the issue in case you're still encountering issues.

Agreed!
On Sun, Jun 9, 2024, 7:50 PM Baiju Meswani @.***> wrote:

@jmopuri https://github.com/jmopuri I'll mark this as completed. Please
comment/reopen the issue in case you're still encountering issues.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/onnxruntime-genai/issues/462#issuecomment-2157094220,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGAVAY2F2T3AYTDCZAZQBDZGUH5RAVCNFSM6AAAAABHYWUGM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJXGA4TIMRSGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add support for a maximum distance constraint

This PR:

[x] Closes #330.
[x] Adds and passes relevant tests.
[ ] Has well-formatted code and documentation.

TODO:

[x] Define relevant field on VehicleType;
[x] Add relevant members to pyvrp::search::Route, tests;
[x] Add relevant members to pyvrp::Route and pyvrp::Solution, tests;
[x] Add penalty for distance violations, tests;
[x] Update operators, tests.

Benchmarks
This PR (315e273a1f329d9fe1d5d8f7bf63682b64dcdf43; job ID 7554747; 1h VRPTW):
                MEAN       MIN       MAX   COV
      inst
obj   C1     41362.4   41334.9   41382.0   0.0
      C2     16200.2   16194.9   16204.6   0.0
      R1     47272.5   47228.8   47309.9   0.1
      R2     27798.2   27771.0   27837.8   0.1
      RC1    44273.4   44240.5   44307.4   0.0
      RC2    23283.4   23271.9   23303.0   0.0
iters C1     94931.9   84807.5  103873.3   7.2
      C2    133950.4  112353.7  156002.5  10.1
      R1     45582.7   38853.3   52118.4  10.4
      R2     84074.9   72281.8   93146.3   8.7
      RC1    46366.3   41211.2   50371.4   8.8
      RC2    88961.4   80244.7   97845.4   8.2

Main from a few days ago (from #510):
               MEAN      MIN      MAX   COV
      inst
obj   C1    41381.2  41367.2  41399.3   0.0
      C2    16195.6  16194.1  16197.6   0.0
      R1    47332.3  47293.2  47392.1   0.1
      R2    27792.7  27773.0  27818.7   0.0
      RC1   44326.8  44292.6  44371.6   0.1
      RC2   23282.4  23264.8  23307.9   0.1
iters C1    49953.8  36474.2  57625.8  11.0
      C2    77654.1  56661.7  92990.9  12.1
      R1    23704.5  17219.9  31143.7  14.2
      R2    48094.9  34347.2  59268.9  12.8
      RC1   25269.4  19147.6  29493.1  10.8
      RC2   51344.8  35158.9  61453.1  13.0

Notes:

It is essential that you add a test when making code changes.
This keeps the code coverage level up, and helps ensure the changes work as intended.
When fixing a bug, you must add a test that would produce the bug in the master branch, and then show that it is fixed with the new code.
New code additions must be well formatted. Changes should pass the pre-commit workflow, which you can set up locally using pre-commit.
Docstring additions must render correctly, including escapes and LaTeX.

Benchmark looks good, no meaningful differences in gaps. Iters changed a lot due to #511, but gaps are largely unchanged between this PR and one a few days ago.


Search url updates

I fixed and issue with the url for one of the links and added the ability to specify a different search engine in case someone didn't want to be so troll-y
Updated version to 0.1.2
Tested locally in Otterbot

Checking in to see if these changes could be reviewed


Correct Kedro starters GH link for PySpark

Description
The page had the incorrect link for the starter repo on GitHub (pointing at ./quantumblack/kedro instead of ./quantumblacklabs/kedro).
Development notes
Added the correct GH URL, just not sure about the branch.
Checklist

[x] Read the contributing guidelines
~- [ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress~
Fixes #
~- [ ] Updated the documentation to reflect the code changes~
[ ] Added a description of this change and added my name to the list of supporting contributions in the RELEASE.md file
~- [ ] Added tests to cover my changes~

Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

I don't see the changed link...just a change to the release.md to add contributors.

Sorry, that was me solving a merge issue. I am still having newbie-issues 😅
The PR correction is here: https://github.com/quantumblacklabs/kedro/pull/491/commits/07013dd44a9bb6f3f0606edeb3fe603fad774b60

Should I close this PR too @laisbsc? It doesn't point to the fix...

Should I close this PR too @laisbsc? It doesn't point to the fix...

Just closed. Thanks for the heads up.


Update chapter14.txt

@theAtropos4n6 is this a stale PR now? Should we just close this?

Yeap, please do not approve this commit. I replaced the file completely


spinach add in

I have added, require 'capybara-screenshot/spinach'  to the env file and I get:
cannot load such file -- capybara-screenshot/spinach (LoadError)

Well it's definitely there, https://github.com/mattheworiordan/capybara-screenshot/blob/master/lib/capybara-screenshot/spinach.rb
We will need more info to work out what is wrong, are you sure the capybara-screenshot gem installed?

Yes it is installed.
gem list --local | grep capy
capybara (2.5.0)
capybara-email (2.4.0)
capybara-screenshot (1.0.11)

What happens when you require "capaybara-screenshot"

Interesting, I get:
Saved file /Users/chabgood/projects/me/registeryourhorse/tmp/capybara/screenshot_2016-01-18-19-34-34.324.html
Rack::Test capybara driver has no ability to output screen shots. Skipping.
All of the images are missing in the saved file.

That's because you are using the Rack driver i.e. not a headless browser, so not sure what you expected to capture?  Also, not sure how this relates to the spinach issue unfortunately.  Can you try require "capybara-screenshot/capybara-screenshot/spinach" and also require "capybara-screenshot/lib/spinach" to see if those work?

Thanks, neither one of those commands worked, it errored out.
cannot load such file -- capybara-screenshot/capybara-screenshot/spinach (LoadError)
cannot load such file -- capybara-screenshot/lib/spinach (LoadError)
FYI, 'require 'capybara-screenshot''  works it is just that when i open the file all of the images are missing, a bit odd.

FYI, 'require 'capybara-screenshot'' works it is just that when i open the file all of the images are missing, a bit odd.

You mean the HTML file?  If so, that is not a capybara-screenshot issue, we only call the driver's create screen shot or save HTML method, it is up to the driver to ensure that the HTML has relative paths or the PNG etc is valid


Serialization Issue

I'm currently building a custom form input and I'm running into a JSON.parse issue that is somehow trying to parse an undefined variable.
I'm missing the smarts to make NPM work with a local copy of react-form so I'm not 100% sure what is going on there. The object that I'm passing into the setValue call is not undefined. This makes me suspect that either a context property is not getting passed correctly or something else.
Full Component
    console.log('updateItem.items', oldItems);

    /*
    Uncaught SyntaxError: Unexpected token u in JSON at position 0
      at JSON.parse (<anonymous>)
      at v (index.js:formatted:1)
      at index.js:formatted:1
      at dispatch (index.js:formatted:1)
      at Object.dispatch (index.js:formatted:1)
      at t.value (index.js:formatted:1)
      at Object.setValue (index.js:formatted:1)
      at PackagesWrapper.updateItem (index.js:formatted:1)
      at PackageSelection.updateChannel (index.js:formatted:1)
      at Select.setValue (index.js:formatted:1)
    */
    /*
    oldItems = [
      {
        slug: 'omc-global',
        channel: 'master'
      },
      {
        slug: 'elasticsearch',
        channel: '2.4'
      }
    ]
    */
    this.props.fieldApi.setValue(oldItems);

Is it because I'm trying to call setValue with an array of data?

i found the dynamic form example, which is what I really needed.


When was the disable OneDrive Profile creation introduced?

When was the disable OneDrive Profile creation introduced?  We had been looking for a solution like this for years.

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

ID: be31bf07-922b-7307-a30e-f09d00fb79f3
Version Independent ID: a42d0699-1acf-808c-0d86-dc3516f4dfaf
Content: Manage user profiles in the SharePoint admin center - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/manage-user-profiles.md
Service: sharepoint-online
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Websocket section should warn about billing impact

Websocket support is great but it also means the container likely will not sleep anymore if a user decides to use they. This has a big impact on expected pricing and should probably be mentioned.

This is extensively documented but I'll add a sentence.


Update sbt-release to 1.0.12

Updates com.github.gseitz:sbt-release from 1.0.11 to 1.0.12.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.gseitz", artifactId = "sbt-release" } ]

labels: semver-patch

Codecov Report

Merging #732 into develop will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #732      +/-   ##
===========================================
+ Coverage    89.28%   89.32%   +0.04%
===========================================
  Files          114      114
  Lines         8965     8965
  Branches       325      325
===========================================
+ Hits          8004     8008       +4
+ Misses         961      957       -4

Impacted Files
Coverage Δ

.../main/scala/com/twitter/algebird/Successible.scala
87.5% <0%> (-4.17%)
:arrow_down:

.../src/main/scala/com/twitter/algebird/ExpHist.scala
95.55% <0%> (-1.12%)
:arrow_down:

...src/main/scala/com/twitter/algebird/Interval.scala
77.39% <0%> (-0.87%)
:arrow_down:

.../main/scala/com/twitter/algebird/Approximate.scala
88.7% <0%> (ø)
:arrow_up:

.../main/scala/com/twitter/algebird/HyperLogLog.scala
92.96% <0%> (+0.78%)
:arrow_up:

.../main/scala/com/twitter/algebird/BloomFilter.scala
95.57% <0%> (+1.32%)
:arrow_up:

.../main/scala/com/twitter/algebird/Applicative.scala
61.76% <0%> (+2.94%)
:arrow_up:

...main/scala/com/twitter/algebird/monad/Reader.scala
55.55% <0%> (+5.55%)
:arrow_up:

Continue to review full report at Codecov.

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


Error while adding a new Show

This issue is being closed due to inactivity. Please feel to re-open.


[tvOS] App crashes when going into fullscreen 'Can't add self as subview'

Current behavior
When entering fullscreen the app is crashing (screen turns black with loading indicator showing)
Reproduction steps
This is the sample code that I used:
    <Video
        ref={player => this.player = player}
        style={{ flex: 1 }}
        resizeMode="contain"
        paused={this.state.paused}
        onLoad={() => {
          // this.player.presentFullscreenPlayer();

          setTimeout(() => {
            this.player.presentFullscreenPlayer();
          }, 5000)
        }}
        source={require('./videotest.mp4')}
    />

Platform
tvOS<EMAIL_ADDRESS>Video sample
I have tried multiple sources but feel free to use this:
https://s.wahlswiper.de/bundestagswahl-2017/01-grundeinkommen/index.m3u8

@mxmtsk I've had a similar issue.
Try to invoke fullscreen onReadyForDisplay with all the same setTimeout. If it doesn't help right away, try to next <Video> deeps in your tree. It might help. Also, pls take a look at https://github.com/react-native-community/react-native-video/pull/1165 <= without this PR you won't get a proper video playback on tvOS. Well, at least for now.


Auto Mode no longer working in script - HP04

I'm a bit of a new user for Home Assistant but I had managed to coble together some easy scripts to control my Dyson HP04
Up until recently and for most of the last year this very simple script was working :
Config : https://i.imgur.com/yoNpIJk.png
Visual Editor : https://i.imgur.com/ZXHFtQZ.png
For some reason it will no longer apply the auto setting once triggered, this is happening for all my scripts.
I then tried applying the switch directly (as I had seen closed issues from middle of 2021 about a move to switches from profiles
Config : https://i.imgur.com/cfAj7wv.png
Visual Editor : https://i.imgur.com/qCAHl9K.png
To me even though taking a different route through the visual editor the config is identical for both and both are using the recommendation ( switch.pure_hot_cool_auto_mode ) from this closed ticket : https://github.com/shenxn/ha-dyson/issues/49
Am I doing something wrong? Not sure what I broke in my script, it was working for most of the year.
Any help would be appreciated

Experiencing the same here, and to be more specific both auto mode and front airflow are now showing as unavailable for Dyson Pure Cool.

I have exactly the same issue.

Thank you for the report, it's clearly a regression from 0.15, I added it to the 0.17 roadmap

Appreciate it.
Is there somewhere we can view the future roadmaps?

Exactly the same issue, looking forward to the fix. Thanks.

Hi!
I've just installed the integration for TP04 and there are no controls to enable auto mode or front airflow:

Is it the same issue?
Also, when the TP04 is in AUTO mode and I switch it off and on again it starts with the fan setting set to 2 and not with the previous setting. Is it expected or is it a separate issue?

Pretty sure it's the same thing we are all experiencing. I have to manually set my Dyson using the app again for a few weeks now.

Since this is mostly due to my changes that went into 0.16 — this is expected behaviour following Home Assistant best practices.
Auto mode and direction of fans is set on the control of the fan itself, rather than as separate toggle switches:

The fan.turn_on service provide access to the additional parameters:

For scenes this means that you'll have to remove and re-add the device for it to be started in the configuration you want.

Using this I was able to fix my scripts.

I can't believe I didn't try to click the click the Fan entity, I just thought it was a simple toggle ☺
All good now, thanks for looking into this @Flameeyes!

I'll add a section in the README regarding this issue.

Auto mode and direction of fans is set on the control of the fan itself, rather than as separate toggle switches:

Is it possible to get these toggles directly in a card? (avoiding the need to click into the entity to get them)

You should be asking that upstream, I'm afraid.
I would generally be in favour of a "Fan" card view that would contain the settings, but I'm not sure if this is the right place to implement (particularly as we intend to merge the integration into core.)

Auto mode and direction of fans is set on the control of the fan itself, rather than as separate toggle switches:
 @Flameeyes Is it possible to get these toggles directly in a card? (avoiding the need to click into the entity to get them)

Now that the integration is to be merged in the HA core, the best thing would be to steer away this project from these kind of features as it should become a minimal working base on which you could build upon.

Apologies about digging this back up, however I am still seeing a problem but now in Node Red when trying to recreate the above script.
After being able to sort out the Auto Mode I started building the automatons out in Node Red but I can't get it to trigger Auto Mode using switch.pure_hot_cool_auto_mode with fan.turn_on.
Am I missing something? It works fine using it in Home Assistant this way.

https://i.imgur.com/okZSg1v.png

Your entity should be the fan entity, since the switch entity is no longer provided.

Ok, I thought we moved Auto Mode to switches so that's my bad, however these are the options I get for entities using Fan. Using fan.pure_hot_cool,  i get the error "Call-service error. The preset_mode auto is not a valid preset_mode: ['Auto']"

I guess it needs to be capitalized.

hahahaha I hate it here hahaha
Capital A worked on Auto
Thank you

@Flameeyes, what is the "canonical" way to disable Auto mode, other then setting a percentage speed? I wonder if modelling this as a preset is really the right way, or maybe it is, but there needs to be another preset manual (and Auto should really be auto).

@mundschenk-at the Home Assistant canonical way to handle this is indeed having a single Auto preset_mode and switching it off by setting a percentage.
From the documentation:

Preset modes should not include named (manual) speed settings as these should be represented as percentages.
Manually setting a speed must disable any set preset mode. If it is possible to set a percentage speed manually without disabling the preset mode, create a switch or service to represent the mode.

"Auto Mode" is not orthogonal to the speeds: if auto mode is enabled, speed control is disabled, and vice-versa. You can't have "Speed 4 + Auto Mode."

It is, sort of, in that when you use the Dyson app or the IR remote control, the device remembers the last set speed percentage. If I switch on auto mode while speed is set to 10 (100 %), then auto will reduce speed if there are no particles etc. If you now switch off auto mode, the fan will return to full blast.

Yes the Dyson app/firmware have a recall-last-speed — but that is not really exposed by Home Assistant. Since the plan for this component is to be merged upstream (or at least that's what @Kakise was hoping), I'd strongly suggest keeping it compliant with Home Assistant's semantics.
You can build your own helper switch, by saving the state of the fan entity before turning the preset to Auto, and recalling it after turning it off (this does require https://github.com/home-assistant/core/pull/62794 to be merged though), but I don't think that breaking the HA model to match the Dyson app is the right answer here.

Sure, getting the integration merged upstream would be best. OK, I see, this is one architectural purity decisions. Well, can't be helped then.


Question/Doubt: how does nvm handle the include files distributed by node?

How does nvm handle the include files distributed by node?
Using nvm will node/npm/yarn/pnpm find and use them correctly?
I'm assuming their purpose has something to do with npm packages that are written in C/C++; but I could be wrong here.
Forgive me if this is not the place to post this.

It doesn’t; node handles that. Yes, all those things will work correctly, or else nvm wouldn’t be very useful :-)
Is there a specific use case you have that you’re concerned about?


Issue parsing strings that end with a backslash.

As an example
"some text\\"

should parse to a JsonValue with the string value of
some text

But it seems that the \" is taking precedence over the \\ causing the end of the string to not be found.

@nvaghari here is where I'll track this issue.

@gregsdennis thank you a lot

Bug was in the StringParser class.  The loop kept track of when it found an escape character (\) but didn't clear the flag on the next character, so it thought the ending double-quote was supposed to be escaped.

@nvaghari, please download Manatee.Json v5.2.1 when it becomes available.


Paste value with comma in query array parameter cause split into array

Hi,
We have an issue when we try to paste some value with comma in a query parameter of type array.
"/test": {
      "get": {
        "parameters": [
          {
            "name": "criterions",
            "in": "query",
            "required": true,
            "type": "array",
            "items": {
              "type": "string"
            },
            "collectionFormat": "multi",
            "example": "{\"crit1\":\"null\",\"operator\":\"Equal\",\"Value\":\"null\"}"
          }
        ]
      }
    }

The value is parsed in an array and in our case we doesn't want that. A value can contain a comma (JSON).
It will, also, be great if it can be possible to edit an item after adding it : in our example we don't have real life values so client must edit it to try (more for schema structure)...

I don't think we have a easy way for this .
we allow pasting an array and we must use some delimiter  to break the items. And the most intuitive one is the comma. If you values contains one then dont use it while pasting ...just add it manually.  The rest of it you can still paste it.
I dont think we we make any changes to this behaviour. if you feel strongly about it and have a better idea, them submit a PR and we can then review it


version conflicts

[!] Unable to satisfy the following requirements:  - Alamofire (~> 2.0) required by Podfile - Alamofire (~> 1.3) required by AlamofireObjectMapper (0.7)

I've released a update of AlamofireObjectMapper today (v8.0). You should be fine after updating.


Distributing Zipline as Python Executable

This depends on OS, but might be a good try if user don't have to type python zipline.py blablabla manually anymore.
See here for more details:  https://packaging.python.org/en/latest/overview/#packaging-applications

Instead of using freezers or PEX(as those are too much), use setuptools instead.
-> https://setuptools.pypa.io/en/latest/


build: extend default locales with es_ES, es_MX, pt_PT and pt_BR

The Issue
Extend default locales with some spanish and portuguese variants.
Manual Testing Instructions
$ docker build containers/ddev-webserver -t ddev-locales
...
$ docker run --rm ddev-locales locale -a
C
C.utf8
POSIX
de_AT.utf8
de_DE.utf8
en_CA.utf8
en_GB.utf8
en_US.utf8
es_ES.utf8
es_MX.utf8
fr_CA.utf8
fr_FR.utf8
ja_JP.utf8
pt_BR.utf8
pt_PT.utf8
ru_RU.utf8

Automated Testing Overview
This area is not tested afaik.
Release/Deployment Notes
No.

Agree that as the 2nd and 6th most spoken languages based on https://en.wikipedia.org/wiki/List_of_languages_by_number_of_native_speakers, and with a strong ddev user base, this would be helpful.
Just a reminder that a workaround is adding webimage_extra_packages: [locales-all]. See https://ddev.readthedocs.io/en/stable/users/extend/customizing-images/#adding-locales.
If this goes in, we need to update docs/content/users/extend/customizing-images.md too.

This increases the size of ddev-webserver by less than 1MB.


NTR: capsule fruit width

Def: A capsule fruit morphology trait (TO:0001078) which is the width of a fruit capsule (PO:0030091). It is measured from the edge of a carpel (PO:0009030) to the edge of the other carpel.
Ref: DOI: 10.1007/978-3-319-98098-0_2
Exact syn: capsule width
SubClassof: TO:0001078 capsule fruit morphology trait
ORCID: 0000-0001-9832-772X

*capsule fruit width (TO:0001131): A capsule fruit morphology trait (TO:0001078) which is the width (PATO:0000921) of a capsule fruit (PO:0030091).

synonym: capsule width (exact)


Restructure style setprop

Pull Request Test Coverage Report for Build c813d48f5d8fa73988d1561a5581e322292dbf6f-PR-6

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 98.752%

Totals

Change from base Build 079cf42f94f61ea9c68587d5955d0386c9031cc2:
-0.004%

Covered Lines:
526

Relevant Lines:
528

💛  - Coveralls


Proposal: Kaia Connect: Coffee & Code Conversations

Description:
Marlon (Brand Ambassador) and Erick (Developer Ambassador) of Kaia Chain are moving forward with plans to host small-scale meetups in Pontianak, Indonesia, at a local coffee shop. We aim to have 7-9 attendees per session, but we are now requesting to accommodate more participants, such as an entire class in the SMK Immanuel Pontianak (Immanuel Vocational Highschool), Indonesia and Institute Technology and Business Sabda Setia Pontianak, but we still waiting for the acceptance from the teachers and lecturers to conduct there. These meetups will create a friendly space for attendees to learn about Web 3 concepts and gain an introduction to Kaia Chain.
Our goal is to facilitate discussions about blockchain fundamentals, share insights, and foster community connections in a relaxed, open environment that encourages conversation and networking. To show appreciation, participants will receive Kaia Chain merchandise. However, we wish to inform the KAIA Team that, unlike the Vietnam meetup, we will not have a Kaia Chain banner for this event.

Cool. Approved but can you modify the proposal to be for each meetup.
For example, i plan a meetup on x date at y location. then post photos on social to claim.
make sure you open a BUIDL on dorahacks too.

Banner has been printed out successfully~

Banner has been printed out succesfully~

Update Estimation Cost:

Drink for 40 Institute of Technology Sabda Setia Students and Lecturers (We providing 250ml milk with each cost around 0.30 USD) : 12.13 USD
Drink + Snacks for 14 Immanuel Vocational Highschool Students and Teachers in La Vica Cafe (Local nearby coffee shop): roughly cost around 30 USD

Documentation for Kaia Hangouts with Immanuel Vocational Highschool students and Teachers!

Total spending for drinks and snacks cost for IDR 416.000, roughly USD 26.5

Total spending for Drinks and Snacks for the 2 session is
12.1 USD + 26.5 USD = 38.6 USD

Approved for 850 KAIA bounty.

closing


[Executore] CpuFreq parameters tuning is hard-coded only for ondemand

Devlib already exposes a method to configure each parameter supported by a governor.
We do not use that method mainly because it takes a long time especially on systems with many CPUs.
Still we need to remove the hard-coded support which is limited to just one governor and one of its parameters.

This has been fixes by 9883f9edb93e5f8e818992e21f2518340135aa8f


Segmentation fault when dragging view

Description
I get a segmentation fault when dragging the view:

[2023-08-22T08:34:26Z INFO  re_sdk_comms::server] Hosting a SDK server over TCP at <IP_ADDRESS>:9876. Connect with the Rerun logging SDK.
[2023-08-22T08:34:28Z INFO  re_sdk_comms::server] New SDK client connected: <IP_ADDRESS>:51564
Rerun caught a signal: SIGSEGV
Troubleshooting Rerun: https://www.rerun.io/docs/getting-started/troubleshooting
Report bugs: https://github.com/rerun-io/rerun/issues
1: re_crash_handler::install_signal_handler::signal_handler
2:
3: _ZN4llvm7CmpInst6CreateENS_11Instruction8OtherOpsENS0_9PredicateEPNS_5ValueES5_RKNS_5TwineEPNS_10BasicBlockE
Rerun caught a signal: SIGSEGV
Troubleshooting Rerun: https://www.rerun.io/docs/getting-started/troubleshooting
Report bugs: https://github.com/rerun-io/rerun/issues
Segmentation fault (core dumped)

Reproduce

Run the following code snippet:

use rerun::{
    components::{Point3D, Radius},
    MsgSender, RecordingStreamBuilder,
};
use anyhow::Result;

fn main() -> Result<()> {
    let recording = RecordingStreamBuilder::new("Events")
        .connect(rerun::default_server_addr(), None)?;
    let v: Vec<Point3D> = Vec::new();
    MsgSender::new("points")
        .with_component(&v)?
        .with_splat(Radius(0.001))?
        .send(&recording)?;
    Ok(())
}

Run rerun with rerun.
Wait until rerun connected and shows the colored background.
Click and drag the colored background
Rerun crashes with SIGSEGV

Expected behavior
Rerun should not crash
Desktop (please complete the following information):

OS: Ubuntu 22.04.1

Rerun version
rerun-cli 0.8.1 [rustc 1.71.0 (8ede3aae2 2023-07-12), LLVM 16.0.5] x86_64-unknown-linux-gnu, built 2023-08-22T08:13:01Z

_ZN4llvm7CmpInst6CreateENS_11Instruction8OtherOpsENS0_9PredicateEPNS_5ValueES5_RKNS_5TwineEPNS_10B

I'm fairly sure this indicates a crash in llvmpipe, a software implementation of Vulkan. Can you try running with OpenGL by running with WGPU_BACKEND=gl? Generally, we don't support software rasterizers at this point, you should install a graphics driver.

(likely related to #2918)

Yes seems pretty similar to #2918.
I forgot to write that but I'm running Ubuntu 22.04 in a VirtualBox VM. This might explain the issue.

Can you try running with OpenGL by running with WGPU_BACKEND=gl?

This fails like it does in #2918:
$ WGPU_BACKEND=gl rerun
[2023-08-23T07:18:41Z INFO  re_sdk_comms::server] Hosting a SDK server over TCP at <IP_ADDRESS>:9876. Connect with the Rerun logging SDK.
[2023-08-23T07:18:41Z WARN  wgpu_hal::gles::egl] 	EGL context: -robust access
[2023-08-23T07:18:41Z WARN  wgpu_hal::gles::egl] Re-initializing Gles context due to Wayland window
[2023-08-23T07:18:41Z WARN  wgpu_hal::gles::egl] 	EGL context: -robust access

thread 'main' panicked at 'Error in Surface::configure: Validation Error

Caused by:
    Requested usage is not supported
', wgpu-0.16.3/src/backend/direct.rs:734

   6: core::panicking::panic_fmt
             at core/src/panicking.rs:67:14
   7: wgpu::backend::direct::Context::handle_error_fatal
   8: <wgpu::backend::direct::Context as wgpu::context::Context>::surface_configure
   9: <T as wgpu::context::DynContext>::surface_configure
  10: wgpu::Surface::configure
  11: egui_wgpu::winit::Painter::configure_surface
  12: egui_wgpu::winit::Painter::resize_and_generate_depth_texture_view_and_msaa_view
  13: pollster::block_on
  14: <eframe::native::run::wgpu_integration::WgpuWinitApp as eframe::native::run::WinitApp>::on_event
  15: eframe::native::run::run_and_return::{{closure}}
  16: winit::platform_impl::platform::wayland::event_loop::EventLoop<T>::run_return
  17: eframe::native::run::with_event_loop
  18: eframe::native::run::wgpu_integration::run_wgpu
  19: eframe::run_native

Troubleshooting Rerun: https://www.rerun.io/docs/getting-started/troubleshooting
Report bugs: https://github.com/rerun-io/rerun/issues

WINIT_UNIX_BACKEND=x11 rerun Also does not help. (Same crash as just running rerun)

gathering all crashes of this kind now on #3089 and will take it from there (first getting a good error message, then potential follow-up for actual support)


[Feature Request] TorchRL to gym API

Motivation
Allow any torchrl env to be executed as a gym environment.
Solution
Proposed API:
class _TorchRLGymnasiumWrapper(gymnasium.Env):
    def __init__(self, torchrl_env_cls, **kwargs):
        self.torchrl_env = torchrl_env_cls(**kwargs)

    def step(self, action):
        ...

    def reset(self):
        ...

    def seed(self, seed):
        ...

@classmethod
@implement_for("gymnasium")
def gym_register(cls,
    env_name: str,
    reward_threshold: float | None = None,
    nondeterministic: bool = False,
    max_episode_steps: int | None = None,
    order_enforce: bool = True,
    disable_env_checker: bool = False,
    additional_wrappers: tuple[WrapperSpec, ...] = (),
    vector_entry_point: VectorEnvCreator | str | None = None,
    **kwargs,
):
    import gymnasium

    kwargs.update({"torchrl_env_cls": cls})

    gymnasium.register(
    id=env_name,
    entry_point=_TorchRLGymWrapper,
    reward_threshold=reward_threshold,
    nondeterministic=nondeterministic,
    max_episode_steps=max_episode_steps,
    order_enforce=order_enforce,
    disable_env_checker=disable_env_checker,
    additional_wrappers=additional_wrappers,
    vector_entry_point=vector_entry_point,
    kwargs=kwargs,
)

At runtime it will look like
PendulumEnv.gym_register(env_name="TorchRLPendulum-v0", batch_size=torch.Size([4]))
env = gym.make("TorchRLPendulum-v0")

We will need to adapt this for each gym version as well as gymnasium.
cc @albertbou92 @BY571

Here's a gist to follow the progress
https://gist.github.com/vmoens/dc6eb3026561b27e4c027a1d91fb65c2


AppBar's show/done button is not pressable on the device

Overview
On HL2 device, App Bar's Show/Adjust/Done buttons are not pressable.
In the Unity editor, everything works fine.
Expected Behavior
Works with hand interactions
Actual Behavior
Does not work with hand press interaction
Steps to reproduce
Build and deploy HandInteracitonExamples scene.

Testing now on device. I'm finding that if you interact with the buttons using far interaction, then they become pressable. Once pressed, buttons can no longer be pressed until they've been triggered via far interaction again. Very strange.

Fixed by #3832


Master160

This is my branch preparing for imgui 1.60 release

Super cool, thanks @sonoro1234 !
Correct me if I'm wrong, but that PR is meant to be the fixes necessary to update the underlying imgui core, but doesn't expose the new 1.60 functions.
I had a look, here's what I listed from diffing imgui.h between 1.53 and 1.60

flags ImGuiBackendFlags, ImGuiConfigFlags
flags ImFontAtlasFlags
enum ImGuiDir
SetNextWindowBgAlpha()
ArrowButton()
InputDouble()
IsItemFocused(), IsAnyItemFocused()
IsAnyMouseDown()
SetAllocatorFunctions()
ImFontConfig layout/size has changed > need to add asserts to compare all C/C++ sizes to catch those
need to expose Flags in ImFontAtlas functions

Minor: the new lines are using tabs instead of spaces. Not super important as few people would wander there but it still shows depending on your IDE configuration and it shows on github:

You are right @sonoro1234 it would be sane to consider looking at a way to convert structure automatically. It could probably be done with script. I think in the future if this was fully automated we could consider making this part of the imgui repo?
Any specific reason for not using typedef the structs, e.g. typedef struct ImVec2 ImVec2 ? My standard C is rusty but would that cause problems?

This PR was not ready. Your last question was the same I did to Extrawurst.
https://github.com/Extrawurst/cimgui/issues/49


Adding / removing curation labels is very slow

It can take anywhere from 1-30 seconds or so to apply or remove a label.

This was particularly an issue when running the webapp at home.

I couldn't reproduce the problem on my end (it always takes 1-2 seconds for me). I added a latency test button to the gui. Could you try that out after upgrading sortingview and restarting the task backend? Screenshot:

Thanks. I get 1.87 seconds, but it takes ~3-4 seconds for the labels to appear / disappear from my home machine and about 2 seconds on the work machine. Note that the message
"Finished task: workspace_sorting_curation_action.1”
appears almost immediately, but the label takes longer. Is that due to polling?

On Jul 5, 2021, at 3:03 PM, Jeremy Magland @.***> wrote:
I couldn't reproduce the problem on my end (it always takes 1-2 seconds for me). I added a latency test button to the gui. Could you try that out after upgrading sortingview and restarting the task backend? Screenshot:
https://user-images.githubusercontent.com/3679296/124520725-461f2080-ddbb-11eb-9e52-f0c9cff0f72e.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/magland/sortingview/issues/10#issuecomment-874346146, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV4PSJXKM6HSAZWURNFGC3TWIT3DANCNFSM47Z6UJUQ.

Okay thanks. I have just added a new latency test that includes both a query to the backend and appending a subfeed message. See screenshot below. This second task takes around 1.6-2 seconds for me. In the future we can speed this up, but for now it would be helpful to know whether or not you are getting around the same timing.
This requires upgrading sortingview python package on the backend and restarting.

Here’s what I get for the work computer and about the same at home:
I’m using a VPN to get in to UCSF, so perhaps that is why mine is slower…

On Jul 5, 2021, at 6:28 PM, Jeremy Magland @.***> wrote:
Okay thanks. I have just added a new latency test that includes both a query to the backend and appending a subfeed message. See screenshot below. This second task takes around 1.6-2 seconds for me. In the future we can speed this up, but for now it would be helpful to know whether or not you are getting around the same timing.
This requires upgrading sortingview python package on the backend and restarting.
https://user-images.githubusercontent.com/3679296/124529161-eaaf5b80-ddd7-11eb-9d6e-c875b9208b8b.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/magland/sortingview/issues/10#issuecomment-874396669, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV4PSMH2NEMFMRAQZZYBOTTWJL37ANCNFSM47Z6UJUQ.

@lfrank, was there supposed to be a screenshot? I don't see any timings.

Not sure what happened there, as I see the screenshot in my email, but here it is again:
And I’ll try upgrading sortingview shortly...

On Jul 5, 2021, at 6:49 PM, Jeremy Magland @.***> wrote:
@lfrank https://github.com/lfrank, was there supposed to be a screenshot? I don't see any timings.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/magland/sortingview/issues/10#issuecomment-874402613, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV4PSMPUNTBCZ6OIAJRL4DTWJOMDANCNFSM47Z6UJUQ.

Unfortunately I don't see these screenshots in the emails, and they don't appear on github. I think you may need to paste the image on github website issue tracker for it to go through.

Trying again...

Okay thanks. Seems to be around 1 sec slower than mine for both tasks. I'm not sure how to explain that -- but let's keep an eye on it. Ultimately we can move to an offline-first model where the curation steps will be instantaneous, but this slower method is safer during development.

Closing for now, will revisit.

Hi all, just adding to this conversation that I have still been experiencing long latencies for adding labels. Just reading this issue thread now, so I have not used the latency timer tool discussed. That said, my experience is that the labeling is much slower when interacting with larger numbers of units at a time. This is especially prevalent when working with larger sorts (such as probes, rather than tetrodes, in the Frank lab). I also have found that the time to add a label can be pretty inconsistent. Occasionally, labels add basically instantly, especially when only a few (say, 1-5) units are selected at a time. However, if I am adding or removing labels from many units - say, 20+, in a sort with ~240 clusters - then oftentimes the labels won't ever display in the units table no matter how long I wait, until after fully refreshing the page in the browser and reopening the units table. Refreshing the page has very reliably solved this, but this solution currently also means reopening views and reselecting units that were previously open before refreshing.

Sorry - just following up on this - the latency to select units can also be quite slow. Can open as separate issue if useful. Would there also be a way to track/monitor this latency? Have only ever noticed it on large sorts with, say, 240 clusters total. As I try to select/highlight more and more units, it can be (but is not always) very slow for the display to catch up with my unit selections. Thanks for any advice.

Alison - can you include the url so Jeremy and Jeff can look at this directly?

On Aug 8, 2021, at 4:52 PM, acomrie @.***> wrote:
Sorry - just following up on this - the latency to select units can also be quite slow. Can open as separate issue if useful. Would there also be a way to track/monitor this latency? Have only ever noticed it on large sorts with, say, 240 clusters total. As I try to select/highlight more and more units, it can be (but is not always) very slow for the display to catch up with my unit selections. Thanks for any advice.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/magland/sortingview/issues/10#issuecomment-894876041, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV4PSM4MB2N4XZF7DSWBR3T34KDTANCNFSM47Z6UJUQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

Absolutely, here's an example I've been interacting with recently. Thank you! [https://sortingview.vercel.app/workspace/sorting/R-c5089c48b72f/S-816a8e175985?workspace=e5c306c436a9f75fd97a20d832cd54d8243d516da53cbfe86270f11e98ee8b26&channel=franklab]

@acomrie could you please open two new separate issues for this? One for slow-adding-labels and one for slow-selecting-units? You can reference this issue. Thanks!


Smart Smoke Sensor Heiman HS1SA - Zigbee2mqtt 1.14.4

After update 1.14.4 my Smart Smoke Sensor (Heiman HS1SA) is not visable anymore in HA, and unable to find with automatic discovery.

This looks like a problem with zigbee2mqtt and should be reported on that repo. The add-on just wraps that functionality. Please report here: https://github.com/Koenkk/zigbee2mqtt


format SciML Style

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 65 (32.31%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 63.694%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/utils.jl
0
2
0.0%

src/maps.jl
20
62
32.26%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
100

Relevant Lines:
157

💛  - Coveralls


deployment: handle deployment templates and parameters with json comment and metadata

So far, CLI will fail on loading such files
For context, CLI 1.0 leverages json-minify package for better error reporting, and that package is able to strip away comments during the parsing. So coincidentally, CLI 1.0 is able to do better than 2.0.

Here's a equivalent package for python we can use:
https://github.com/getify/JSON.minify/tree/python
Essentially, using this package, we should be able to strip away JS-style comments from the contents of a file before parsing it into JSON by doing:
json.loads(json_minify(contents))

So why was this closed?
Why does it make sense that New-AzResourceGroupDeployment handles my azuredeploy.json file fine but az deployment sub doesn't?

@onionhammer Hi, New-AzResourceGroupDeployment command is used to deploy resources under the resource group scope, but az deployment sub is used to deploy resources under the subscription scope. If you want to use CLI to deploy resources under the scope of resource group, please use az deployment group.
If you still have any questions, please feel free to let us know~

@zhoxing-ms so you're saying az deployment group does support comments in JSON?

@onionhammer Yes, all the scope deployments of CLI now support comments in JSON.
The reason why CLI cannot deploy your azuredeploy.json successfully is that the command used is incorrect: when deploying under the scope of resource group, you should use the command az deployment group

@zhoxing-ms Just tried it and see this:
az deployment group validate -g next-c2c --template-file azuredeploy.json
Failed to parse 'azuredeploy.json', please check whether it is a valid JSON format
resources> az --version
azure-cli                          2.7.0
command-modules-nspkg              2.0.3
core                               2.7.0
nspkg                              3.0.4
telemetry                          1.0.4
Extensions:
azure-devops                      0.17.0

@onionhammer Could you please email me the template file azuredeploy.json you used? My email address is<EMAIL_ADDRESS>thanks~

@onionhammer Thanks for your email~
This problem is actually caused by the mutil-line string in the ARM template (such as resources.Configuration.WebTest )
This is a known issue. Issue #12909 is already following up on this problem.
It is recommended that you change all the mutil-line strings to single line first.

@onionhammer By the way, May I ask is the configuration of mutil-line string in the resources you successfully deploy by PowerShell valid? (such as such as resources.Configuration.WebTest)
If PowerShell does not modify the ARM template and send it directly to the service ( It should be so in design) , I think the parsing logic might be break due to the newline character added by the Service.
For example:
In ARM template:
"meta": {
    "type": "string",
    "metadata": {
        "description": "meta of the network security group."
    },
    "defaultValue": "ARM
    template
    test
    newline"
}

After service processing:
"meta": {
    "type": "String",
    "value": "ARM\r\n      template\r\n      test\r\n      newline"
}

@zhoxing-ms Yep, the web tests work fine on deployment via powershell.

In fact, I found out through the debug mode of PowerShell that PowerShell already adds line breaks to a multi-line string when requesting a Service, Such as:
WebTest becomes
"WebTest": "[concat(\r\n                '<WebTest Name=\"Health check\" Id=\"', guid(concat('health check-', variables('partnerInsights'))), \r\n                '\" Enabled=\"True\"

I think modifying the ARM template of customers before requesting a Service is not in line with the design and destroys the customers's input to some extent
So waiting for the reply from the service team in #12909

@zhoxing-ms what's the status on this? It's been a few months, but this is still definitely an issue. Is this discrepancy a "wont fix"? Is Microsoft's preferred solution an 800+ character long string on a single line filled with variables when a user wants to create a WebTest in ARM?

@onionhammer Because this is mainly a Service issue, not a CLI problem, I reminded relevant personnel to pay attention in this issue. #12909 (The reason why PowerShell can support it is that they have pre-processed the ARM templates, but this does not conform to our design)

Consistency of developer experience is presumably more important than some principle of not wanting to do preprocessing in the CLI tool

@onionhammer Hi, this issue has been fixed by #15502
We will publish it in the new CLI version on 12-29, please try to update CLI and have a try at that time~


how to use the component twice in same screen ?

if i used the component twice in the same screen when i select the first one the second update as well , i need them to be separated

Hi, you need use difference array of datasource, if is has same datasource, clone it to another array.
Tip: you can using cloneDeep method of Lodash library.
ex:
const datasource = [
    {id: 1, name: 'OJAM'}
];
const datasource1 = cloneDeep(datasource);
const datasource2 = cloneDeep(datasource);

render() {
    <View>
        <Select2 data={datasource1}/>
        <Select2 data={datasource2}/>
    </View>
}


[Bug] the arrows dont damage other players in RFV2

Video: https://www.youtube.com/watch?v=7Q-tkN1TWDw&feature=youtu.be
Cardinal Version: CardinalPGM version 1.0-SNAPSHOT-git-CardinalPGM-48aa316

Can anyone else confirm this?


can't update some 3rd-party apps

I can't update applications like malwarebytes or other 3rd party apps that have internal updates. Which setting needs to be reversed to make that possible again?

I don't have this problem.
Take a look at your firewall, syst-pref-security.
Download LockDown from https://objective-see.com
You will be able to easily switch on or off the options you want.


Add a launch configuration for Streamlit

Streamlit is becoming increasingly popular, and it would be great to add a built-in debug/launch configuration for it, similar to the experience for Flask/Django/FastAPI. In searching online, there are many users confused by how to do this properly.
  "configurations": [         {             "name": "Python Streamlit",             "type": "python",             "request": "launch",             "program": "/path/streamlit",             "args": [                 "run",                 "file.py"             ]         }

Thanks for the feature request! We are going to give the community 60 days from when this issue was created to provide 7 👍 upvotes on the opening comment to gauge general interest in this idea. If there's enough upvotes then we will consider this feature request in our future planning. If there's unfortunately not enough upvotes then we will close this issue.


fix(learn): Fixed comment for ES6: use destructuring ... function's parameters

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Comment line not showing as green and therefore confusing. Moved comment  line to a new line to make the challenge easier to read.

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


Can we please get the wifi_signature in python

Is it possible to get wifi_signature code done in python using tshark or scapy.
I am working on a project Distributed-Wifi-Capability-Collector and would love to add support for this but cant make any sense of the C code used in wifi_signature.

Though it would be possible to do so in Python, I suspect it would end up looking very much like the C code. It walks through the Information Elements in the Probe Request and Association Request, storing what tags are present and some of the bitmasks from various fields.
If your intention is to gather signatures of different kinds of devices to add to the database here, I have an alternate suggestion: at this point we're not adding just the text signatures alone, we also add a pcap file in https://github.com/NetworkDeviceTaxonomy/wifi_taxonomy/tree/master/testdata/pcaps as a unit test for that signature. We do this for a couple reasons:

We detect aliasing, where a new signature breaks an older signature without us noticing.
We've updated the content of the text signature a few times already. Having pcap files means we can re-evaluate all existing signatures if we ever update it again.

There are ~1700 pcap files in https://github.com/NetworkDeviceTaxonomy/wifi_taxonomy/tree/master/testdata/pcaps already (note that github truncates the listing at 1000 files, but git clone will get them all). We export a pcap containing:

The Beacon from the AP which the client is going to join.
The Probe Request from the client (this one is crucial).
The Probe Response from the AP, and the Authentication packets from AP and client.
The Association Request from the client (this one is crucial).
The Association Response from the AP, the WPA2 Key1/2/3/4 frames, and a few more Action frames just to get a flavor of how that client behaves (which might be useful in the future).
If the client implements Beamforming reports or Radio Resource Management, we include those frames as well.

If you're interested in having Distributed-Wifi-Capability-Collector gather signatures of different types of clients, I'd request that it do so in the form of a pcap file we can check in. We have tools to anonymize the pcap files, zeroing out the lower bytes of the MAC address, 'X'ing out the SSID, etc.

I would like to add support in DWCC(Distributed-Wifi-Capability-Collector) so that I can stats on the devices models and makes for the networks that this system watch. Some of the networks that I will be using this on are Southern California Linux Expo(3200 people) and South East Linux Fest (800 people).
Right now the nodes collect all 802.11 management frames using pcaps. I think with so testing I can add support to capture action frames too.
What I hope could happen is I pass the pcap to wifi_taxonomy, if it can match any models then it would print back model and mac address. If wifi_taxonomy cant match then it will pull the packets out and write them to a new pcap, then return model not found.  The new pcap could then be giving to you.
I will have to do more testing but in my first test I gave  wifi_signature c version a pcap with 50+ clients in it but it only gave back the sig for more client. I will do more testing with the c version of wifi_signature.

Ok, that isn't something I'd thought about and you're right that it doesn't do anything rational. I believe wifi_signature will pick out the last Probe Request and Association Request it finds in the file, even if they are from two different stations, and will output a signature from those two packets. It was originally written as part of the unit test for the pcaps in testdata/pcaps, where there is only one station.
Would it work to add a MAC address argument (-m 00:11:22:33:44:55), the address of the client you want it to print a signature for? Do you know the address at the time you want to get information about it?
I'd prefer not to rewrite it in Python because some portions of it wouldn't translate. In particular, it anticipates a day when we get submissions of pcaps from third parties whom we might not completely trust, and so it installs a Linux SECCOMP policy to limit the damage were someone to find and exploit a buffer overflow in the app code or in libpcap.


Attr enabled already exist

I'm using both this library and MaterialSeekBarPreference
https://github.com/MrBIMC/MaterialSeekBarPreference
and there is an issue with attribute "enabled" already being defined, which result in me not being able to use this library

I encountered the same issue

You can solve this problem by downloading the source code and replicating the library folder in your application source code. This will increase the app size a bit, but that is the only solution i could find.
Consult: https://stackoverflow.com/a/16639227


Fix typo in error message

What this PR does / why we need it:
Not a huge deal, fixes a small typo in the error message
Which issue this PR fixes
Special notes for your reviewer:
Release note:

NONE

/kind cleanup

/retest

There was an issue with our CI before 😅
/test pull-cert-manager-e2e-v1-22


added localstorage for timer

Timer now persists after page refresh

What's up with this PR?
Is this repo stil being maintained?

What's up with this PR?
Is this repo still being maintained?


/info JSON output of docker swarm is not machine friendly

Here is sample output given by "/info" api
{
    "ID": "",
    "Containers": 83,
    "Driver": "",
    "DriverStatus": [
        [
            "\bRole",
            "primary"
        ],
        [
            "\bStrategy",
            "spread"
        ],
        [
            "\bFilters",
            "affinity, health, constraint, port, dependency"
        ],
        [
            "\bNodes",
            "2"
        ],
        [
            "dockerhost2",
            "<IP_ADDRESS>:2376"
        ],
        [
            " └ Containers",
            "51"
        ],
        [
            " └ Reserved CPUs",
            "0 / 2"
        ],
        [
            " └ Reserved Memory",
            "0 B / 1.851 GiB"
        ],
        [
            " └ Labels",
            "executiondriver=native-0.2, kernelversion=3.16.0-4-amd64, operatingsystem=Debian GNU/Linux 8 (jessie), storagedriver=aufs"
        ],
        [
            "dockerhost3",
            "<IP_ADDRESS>:2376"
        ],
        [
            " └ Containers",
            "32"
        ],
        [
            " └ Reserved CPUs",
            "0 / 2"
        ],
        [
            " └ Reserved Memory",
            "0 B / 1.851 GiB"
        ],
        [
            " └ Labels",
            "com.example.environment=production, executiondriver=native-0.2, kernelversion=3.16.0-4-amd64, operatingsystem=Debian GNU/Linux 8 (jessie), storagedriver=aufs"
        ]
    ],
    "ExecutionDriver": "",
    "Images": 7,
    "KernelVersion": "",
    "OperatingSystem": "",
    "NCPU": 4,
    "MemTotal":<PHONE_NUMBER>,
    "Name": "0ac63565be62",
    "Labels": null,
    "Debug": false,
    "NFd": 0,
    "NGoroutines": 0,
    "SystemTime": "2015-09-15T19:21:59.258278674Z",
    "NEventsListener": 0,
    "InitPath": "",
    "InitSha1": "",
    "IndexServerAddress": "",
    "MemoryLimit": true,
    "SwapLimit": true,
    "IPv4Forwarding": true,
    "BridgeNfIptables": true,
    "BridgeNfIp6tables": true,
    "DockerRootDir": "",
    "HttpProxy": "",
    "HttpsProxy": "",
    "NoProxy": ""
}

Instead of providing JSON array of docker hosts with nested information, the API provides a flat output which is cumbersome to parse. The parser needs to be aware of position of elements within JSON array which is not a good design
A much cleaner output will look like this-
{
    "dockerHosts": [
        {
            "dockerhost2": {
                "Containers": 4
            },
            "dockerhost3": {
                "Containers": 32
            }
        }
    ]
}

@gauravphoenix yes but the docker cli wouldn't display these information

@vieux  I agree. This is a problem for those who are trying to consume the docker swarm API.

I think this is a more general issue regarding the Swarm API that is discussed in #735. As of now we offer the same subset than the Docker remote API, but Swarm should somehow extend this capacity to provide more useful informations and capabilities. /info is a good example.

+1, I am integrating Swarm into my app and just ran into this issue.

:+1:

+1

shouldn't this issue be placed in docker core, too or why is it swarm specific?

FWIW, in the meantime, to be able to retrieve at least some metrics from the swarm, I wrote a simple Groovy script that will parse /info output (and push it to graphite, though one can easily modify it to push elsewhere). HTH
https://gist.github.com/batmat/e67e661656c3d31ed36f

This is not perfect, but it was improved in https://github.com/docker/swarm/pull/1721 since the \b were removed

@vieux Problem is not in \b symbols only. Docker/Swarm change this output quite often. For example entries number between Node entry and its host/IP was changed several times.
It would be much easier to handle if API responds in a predictable way.
If this information is not so useful for docker clients Swarm can add it as its own endpoint because it is useful information for Swarm clients.

+1
We need machine friendly output.
It better to have not just array,  but structured info.
What about adding another endpoint with machine friendly output if you don't want to change existing one?
Maybe we can have some script distributed with swarm, that convert output of /info to machine friendly form?

+1
Just ran into the same problem while integrating swarm into my app. /info/json would do it, no need to change /info.

Adding to what has been already said, to return structured information per-node instead of a single list, please, return numbers instead of strings for all quantities. Right now there is memory: instead of a string with a unit (10 Gib) return the number of bytes.
Also, split reserved and total memory/cores in two different fields.

Shift between nodes entry "\bNodes" and each node declaration  ["dockerhost2","<IP_ADDRESS>:2376"] changes almost every new version of swarm/docker. We had to add configurable shift value into the app to change it instead of code on each update of docker/swarm.
This is crazy )

@garagatyi this is not supposed to happen. Can you provide more details for that?

Example:
Before docker introduced labels swarm returned smth like
"DriverStatus": [
        [
            "\bRole",
            "primary"
        ],
        [
            "\bStrategy",
            "spread"
        ],
        [
            "\bFilters",
            "affinity, health, constraint, port, dependency"
        ],
        [
            "\bNodes",
            "2"
        ],
        [
            "dockerhost2",
            "<IP_ADDRESS>:2376"
        ],
        [
            " └ Containers",
            "51"
        ],
        [
            " └ Reserved CPUs",
            "0 / 2"
        ],
        [
            " └ Reserved Memory",
            "0 B / 1.851 GiB"
        ],
        [
            "dockerhost3",
            "<IP_ADDRESS>:2376"
        ],
        [
            " └ Containers",
            "32"
        ],
        [
            " └ Reserved CPUs",
            "0 / 2"
        ],
        [
            " └ Reserved Memory",
            "0 B / 1.851 GiB"
        ]
    ],

so the distance between
 [
            "dockerhost2",
            "<IP_ADDRESS>:2376"
        ],

and
 [
            "dockerhost3",
            "<IP_ADDRESS>:2376"
        ],

is 3 entries of JSON array.
But after addition of labels distance became 4
"DriverStatus": [
        [
            "\bRole",
            "primary"
        ],
        [
            "\bStrategy",
            "spread"
        ],
        [
            "\bFilters",
            "affinity, health, constraint, port, dependency"
        ],
        [
            "\bNodes",
            "2"
        ],
        [
            "dockerhost2",
            "<IP_ADDRESS>:2376"
        ],
        [
            " └ Containers",
            "51"
        ],
        [
            " └ Reserved CPUs",
            "0 / 2"
        ],
        [
            " └ Reserved Memory",
            "0 B / 1.851 GiB"
        ],
        [
            " └ Labels",
            "executiondriver=native-0.2, kernelversion=3.16.0-4-amd64, operatingsystem=Debian GNU/Linux 8 (jessie), storagedriver=aufs"
        ],
        [
            "dockerhost3",
            "<IP_ADDRESS>:2376"
        ],
        [
            " └ Containers",
            "32"
        ],
        [
            " └ Reserved CPUs",
            "0 / 2"
        ],
        [
            " └ Reserved Memory",
            "0 B / 1.851 GiB"
        ],
        [
            " └ Labels",
            "com.example.environment=production, executiondriver=native-0.2, kernelversion=3.16.0-4-amd64, operatingsystem=Debian GNU/Linux 8 (jessie), storagedriver=aufs"
        ]
    ],

We need list of nodes to implement some operations on the nodes, so we parse /info output to get it. But since docker adds new entries in the DriverStatus we have to use configuration parameter of the distance between nodes declaration in the output.
We would love to avoid such a strange parsing but swarm doesn't provide endpoint to get list of nodes in machine friendly format.
It would be better if swarm returns smth like:
{
    "Nodes": {
        "dockerhost2": {
            "Address": "<IP_ADDRESS>:2376",
            "DriverStatus": [
                [
                    " └ Containers",
                    "51"
                ],
                [
                    " └ Reserved CPUs",
                    "0 / 2"
                ],
                [
                    " └ Reserved Memory",
                    "0 B / 1.851 GiB"
                ],
                [
                    " └ Labels",
                    "executiondriver=native-0.2, kernelversion=3.16.0-4-amd64, operatingsystem=Debian GNU/Linux 8 (jessie), storagedriver=aufs"
                ]
            ]
        },
        "dockerhost3": {
            "Address": "<IP_ADDRESS>:2376",
            "DriverStatus": [
                [
                    " └ Containers",
                    "51"
                ],
                [
                    " └ Reserved CPUs",
                    "0 / 2"
                ],
                [
                    " └ Reserved Memory",
                    "0 B / 1.851 GiB"
                ],
                [
                    " └ Labels",
                    "executiondriver=native-0.2, kernelversion=3.16.0-4-amd64, operatingsystem=Debian GNU/Linux 8 (jessie), storagedriver=aufs"
                ]
            ]
        }
    },
    "ID": "",
    "Containers": 83,
    "Driver": "",
    "DriverStatus": [
        [
            "\bRole",
            "primary"
        ],
        [
            "\bStrategy",
            "spread"
        ],
        [
            "\bFilters",
            "affinity, health, constraint, port, dependency"
        ]
    ],
    "ExecutionDriver": "",
    "Images": 7,
    "KernelVersion": "",
    "OperatingSystem": "",
    "NCPU": 4,
    "MemTotal":<PHONE_NUMBER>,
    "Name": "0ac63565be62",
    "Labels": null,
    "Debug": false,
    "NFd": 0,
    "NGoroutines": 0,
    "SystemTime": "2015-09-15T19:21:59.258278674Z",
    "NEventsListener": 0,
    "InitPath": "",
    "InitSha1": "",
    "IndexServerAddress": "",
    "MemoryLimit": true,
    "SwapLimit": true,
    "IPv4Forwarding": true,
    "BridgeNfIptables": true,
    "BridgeNfIp6tables": true,
    "DockerRootDir": "",
    "HttpProxy": "",
    "HttpsProxy": "",
    "NoProxy": ""
}

@nishanttotla This happens because DriverStatus is a list, it is not a new problem, this bug is open since September.
Every new Swarm version something in /info changes and we have to make our code uglier and uglier. Please, try to use this endpoint in any language of your choice and extract a list of how many cores are free on each Swarm node. It's a nice exercise.
I want to stay constructive, so I will not make fun of the \b and └ characters in a machine readable API.

@garagatyi @dvenza as far as the \b characters are concerned, we removed them for newer API versions, keeping the output same for old ones to not break things. What API client version do you pass when you use the Swarm remote API?

My client do not pass client version to swarm and I don't know what is the behavior of swarm in that case.
But removal of \b and └ won't help me. I'm suffering from JSON structure, not this symbols.

You can see the desired by me structure of that JSON

Yes, The first and foremost problem is that DriverStatus should be a map/object/dictionary, not an array.
One per node would make the most sense.
The UI characters can be dealt separately, they are funny, but do not require specialized parsing code like the DriverStatus array.

@garagatyi @dvenza I agree with your comments. The reason for why this is the way it is right now is because the docker cli needed to be compatible with Swarm, while keeping the two decoupled.
Anyhow, let us see if this can be meaningfully redesigned.

Closing due to lack of activity. Please reopen if you wish to continue discussing it.


New mode : "gazette"

Hey there, stumbled on this tool by accident while looking for something else. Took a few minutes to fiddle around and decided to share a mode I created.
Anyways thanks for the tool!

This and manticore95 are sweet! I really appreciate you checking out the project and adding a little of yourself to it. ❤

@BjrInt I'll make sure to update the readme examples next release 😉

@BjrInt What info would you like to be used for the contributions section in the package?
I typically do: Name <email> (https://github.com/BjrInt) -- email is totally optional.


zipfile.BadZipFile: File is not a zip file

I'm getting the following error:
zipfile.BadZipFile: File is not a zip file
And I followed the following steps:
First I installed their docker image by this command:
docker run -it -p 8888:8888 urw7rs/spiralpp:latest /bin/bash

And here is the output of ls from inside the Docker image:
(spiralpp) root@8c0404e74c57:/src/spiralpp# ls
CODE_OF_CONDUCT.md
LICENSE
dataset.lock
libtorchbeast.egg-info
pyproject.toml
setup.py
third_party
CelebAMask-HQ
README.md
demo.ipynb
nest
requirements.txt
spiral-envs
torchbeast
Dockerfile
build
libtorchbeast
plot.ipynb
scripts
tests

And then after running this command:
python -m torchbeast.polybeast \
     --dataset celeba-hq \
     --env_type libmypaint \
     --canvas_width 64 \
     --use_pressure \
     --use_tca \
     --num_actors 64 \
     --total_steps 30000000 \
     --policy_learning_rate 0.0004 \
     --entropy_cost 0.01 \
     --batch_size 64 \
     --episode_length 40 \
     --xpid example

I got the following error:
Traceback (most recent call last):
  File "/root/miniconda3/envs/spiralpp/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/root/miniconda3/envs/spiralpp/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/src/spiralpp/torchbeast/polybeast.py", line 59, in <module>
    main()
  File "/src/spiralpp/torchbeast/polybeast.py", line 52, in main
    run_learner(flags)
  File "/src/spiralpp/torchbeast/polybeast.py", line 31, in run_learner
    polybeast_learner.main(flags)
  File "/src/spiralpp/torchbeast/polybeast_learner.py", line 865, in main
    train(flags)
  File "/src/spiralpp/torchbeast/polybeast_learner.py", line 585, in train
    dataset = utils.create_dataset(flags.dataset, grayscale)
  File "/src/spiralpp/torchbeast/utils.py", line 88, in create_dataset
    dataset = CelebAHQ(root="./", split="train", transform=tsfm, download=True)
  File "/src/spiralpp/torchbeast/core/datasets.py", line 53, in __init__
    self.download()
  File "/src/spiralpp/torchbeast/core/datasets.py", line 122, in download
    os.path.join(self.root, self.base_folder, "CelebAMask-HQ.zip"), "r"
  File "/root/miniconda3/envs/spiralpp/lib/python3.7/zipfile.py", line 1258, in __init__
    self._RealGetContents()
  File "/root/miniconda3/envs/spiralpp/lib/python3.7/zipfile.py", line 1325, in _RealGetContents
    raise BadZipFile("File is not a zip file")
zipfile.BadZipFile: File is not a zip file

The error is coming from /src/spiralpp/torchbeast/core/datasets.py line 122:
with zipfile.ZipFile(
            os.path.join(self.root, self.base_folder, "CelebAMask-HQ.zip"), "r"
        ) as f:
            f.extractall(os.path.join(self.root, self.base_folder))

And here is the output of ls from inside the CelebAMask-HQ directory:
(spiralpp) root@8c0404e74c57:/src/spiralpp/CelebAMask-HQ# ls
CelebAMask-HQ.zip
list_eval_partition.txt

Hi @urw7rs, @heiner could you please help me to solve this issue?

CelebaHQ sometimes can't download the files from google drive. This is a known issue. Try downloading the file manually and set download=False at https://github.com/urw7rs/spiralpp/blob/6f1d6130e99cb44f8dbc4c0e291f194f3f84a12c/torchbeast/utils.py#L84 line 84.

Alright @urw7rs so I downloaded the dataset manually from here: https://github.com/switchablenorms/CelebAMask-HQ
And now running this command:
python -m torchbeast.polybeast \
     --dataset celeba-hq \
     --env_type libmypaint \
     --canvas_width 64 \
     --use_pressure \
     --use_tca \
     --num_actors 64 \
     --total_steps 30000000 \
     --policy_learning_rate 0.0004 \
     --entropy_cost 0.01 \
     --batch_size 64 \
     --episode_length 40 \
     --xpid example

But I need your advice here:
What do you think is preferable to run this image on the cloud or it's okay to run it locally in my machine?
My machine resources:

RAM Type: DDR4 SO-DIMM
RAM Size : 16 GB
Processor Speed (GHz): 2.9 GHz
Graphics: 6 GB NVIDIA GeForce RTX 2060

And what could be the expected time for the model building according to these ressouces?
Thank you in advance!


TestScheduler bug delay subscription time by 1 tick when 0 is given

There is a problem with the TestScheduler from the testing framework.
When i call the Start method with 0 as the subscribed param, it call the ScheduleAbsolute method and the dueTime = Clock + 1 trigger.
Because of that, in my test case, every messages are delayed by one tick.
I think that the dueTime = Clock + 1 was implemented because of this.
So, I guess that changing the comparison with >= and removing the dueTime = Clock + 1 would fix that.
The problem doest not occur with a subscribed time > 0.
using System;
using System.Reactive.Linq;
using System.Reactive.Concurrency;
using Microsoft.Reactive.Testing;
using Xunit;

namespace Cycle.Net.Sample.Test
{
    public class Test : ReactiveTest
    {
        [Fact]
        public void TestX()
        {
            var scheduler = new TestScheduler();

            var source = scheduler.CreateColdObservable(
                OnNext(100, 1),
                OnNext(200, 2),
                OnNext(300, 3),
                OnCompleted<int>(500)
            );

            /*
                This will work with subscribed > 0
                but trigger exception with 0 as shown with this test,
                because our subscription get delayed.

[xUnit.net 00:00:00.7707121]       Expected: [OnNext(1)@100, OnNext(2)@200, OnNext(3)@300]
[xUnit.net 00:00:00.7707640]       Actual..: [OnNext(1)@101, OnNext(2)@201, OnNext(3)@301]
             */
            var subscribed = 0;
            var observer = scheduler.Start(() => source, 0, subscribed, 500);

            ReactiveAssert.AreElementsEqual(new[]
            {
                OnNext(100 + subscribed, 1),
                OnNext(200 + subscribed, 2),
                OnNext(300 + subscribed, 3)
            }, observer.Messages);
        }
    }
}

Yes, probably a mistake started by that wrong comparison. The change looks reasonable but it will fail 164 unit tests that relied upon that 1 tick drift in some form and others prevent the test run from completing entirely.


Throwables Do No Damage

The three explosive throwables seem to do no damage to dark fog units or buildings (combustible, explosive, crystal explosive). The AOE does not aggro them either, they seem to completely ignore the explosions.
This was tested on a standard sol generated run, with 48 stars.

Does it work on the 5 closest planets to the star?

I tested this on the starting earth planet, which is third closest, so no.

Should be fixed in 2.14.2 #229