Backport goodies for the code editors (2.1)

Refactor duplicated code (from 0159e4f96918990ee7bc3e9616ba073e566ad6e4)
Add line length guideline to code editors (from d9c1729a8f1d3eceb259ef540b378b70beb55f24)
Allow turning off zero-padding for line numbers (from 00b3af246b03bc789a7edc45c11b000d7d63ad27)
(In 3.0 zero-padding is off by default, but for 2.1 I'm setting the default to be on because it's how it always worked.)
Fixed line lenght guideline drawing with color option (from @Paulb23's 6b42cd5fe637d6d0fe30fa397eca659d295ad956)

While you are at it, add distraction free mode toggle button in f10d2bd3944f3a520cf9f131b801f403c834ea6e please :)

You're making my life harder for when I will actually get to cherry-picking relevant commits from master into 2.1 ;)
But I don't complain, at least someone else than me does the job and I trust you for resolving potential merge conflicts efficiently :)

Though actually I'd prefer a PR where each of those commits is cherry-picked with git cherry-pick -x hash like I do usually. Otherwise it will start being really hard for me to see what was cherry-picked and what was not (when cherry-picking I usually compare the two branches starting at the last big cherry-picking I did, and grab all relevant master commits one by one).

I'll try to do it the same way as you from now on.
I always document the source commit(s) by hand. I hope that alleviates the
"friction".
El 9 feb. 2017 9:20 a. m., "Rémi Verschelde"<EMAIL_ADDRESS>escribió:

Though actually I'd prefer a PR where each of those commits is
cherry-picked with git cherry-pick -x hash like I do usually. Otherwise
it will start being really hard for me to see what was cherry-picked and
what was not (when cherry-picking I usually compare the two branches
starting at the last big cherry-picking I did, and grab all relevant master
commits one by one).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/7719#issuecomment-278575746,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALQCtv6aRNj3JZVRJby33Zcdw8maXosyks5raswygaJpZM4L3LZI
.

I always document the source commit(s) by hand. I hope that alleviates the
"friction".

Yeah that's fine, but that makes it harder to just compare visually the commit logs or by grepping :)
But it's no big deal either as if I try to cherry-pick a commit you already backported in another differently named bigger commit, the cherry-pick will just tell me there is nothing to do anyway.


Ignored "shared" action

It seems that the get-current-version action @lachmanfrantisek moved to make it shared is ignored/not executed.
You can see it by looking at https://copr.fedorainfracloud.org/coprs/g/oamg/convert2rhel/builds/. The pull request builds have version 0.25 (coming from the shared get-current-version action, as expected), but the main branch builds have version v0.25 (likely coming from git describe - the packit default - unexpectedly).
I wonder why is the get-current-version action not executed even for the main branch builds. My suspicion is that the main branch build has a job-specific action and it overrides the shared action. I haven't found in the docs https://packit.dev/docs/actions/ whether this behavior is to be expected.

Hi Michal @bocekm!
You are right. The problem is in the partially overwritten actions key.
Our overriding workflow works for all the keys in the same way and the values are handled as one unit independently of the type -- so, if you set the actions key in the job, everything from the top level is ignored.
Is this behaviour ok, if we make this really clear in the docs? Or, should we try to do the overwriting/inheritance single-action-specific => make a union of the top-level and job-level actions.

I'm for the option "make a union of the top-level and job-level actions". Our packit config is an example of why it can be useful - two jobs sharing a single action to avoid duplication and one of the jobs requiring one action on top of that.
However I'm leaving it up to you. Duplicating the actions on a job-level is also an option - that's what I'll do now. I can revert it back if you decide for the "union".

For some reason, making the previously shared action per-job again did not help. Or better to say, the behavior is now switched:

the PR builds have version v0.25, e.g.

PR https://github.com/oamg/convert2rhel/pull/458, copr build https://copr.fedorainfracloud.org/coprs/g/oamg/convert2rhel/build/4293648/

the main branch builds have version 0.25

e.g. copr build https://copr.fedorainfracloud.org/coprs/g/oamg/convert2rhel/build/4221844/

So now what seems to be ignored is the get-current-version defined under the job triggered by pull_request: https://github.com/bocekm/convert2rhel/blob/d320550cef86be0a8c7e9c1db275e160b018bfa8/.packit.yaml#L22.

@bocekm Sorry for the late response. I was on a PTO...
Thanks for the detailed investigation. Sadly, you hit an issue with the first job picked for SRPM build in Copr. Laura is fixing that here: https://github.com/packit/packit/pull/1564
You can change the order of the jobs to make it work in the meantime -- Use the tests one as the first one to not be affected by that.
Sorry about that. As usual, the fix should be on the production instance on Monday if it is ok for you.

It works now as expected. Thanks!


Musl libc support

Description
for now, all prebuilts are linked to glibc; it would be nice to also see some musl builds, especially that currently it requires a few patches to even build

I would like to see some musl-linked libraries too


update go mod, github actions

Codecov Report

Merging #18 (a1b0475) into main (58cfa1f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage   86.37%   86.37%
=======================================
  Files           9        9
  Lines         543      543
=======================================
  Hits          469      469
  Misses         61       61
  Partials       13       13

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


feat: enhancements for july '23  release

Summary

Updates for HRM module July release
Enhancements:

Updated format for Notifications.json file to make it readable GH-60

Documentation:

Updated README.md and update version and dist files for release

Chore:

Added CHANGELOG.md and removed changelog from send-data-to-vrops.py file

Type

[ ] Bugfix
[x] Enhancement or Feature
[ ] Code Style or Formatting
[x] Documentation
[ ] Refactoring
[x] Chore
[ ] Other
Please describe:

Breaking Changes?

[ ] Yes, there are breaking changes.
[ ] No, there are no breaking changes.

Test and Documentation

[x] Tests have been completed.
[x] Documentation has been added or updated.

Issue References

Closes #60
Additional Information

Addressed the changes.


feat(router): Support return_full_text

@Narsil,
This is a bit annoying as return_full_text default value is not the same between CausalLMs (return_full_text=True) and Seq2SeqLMs (return_full_text=False).
What do you think should be the optimal solution here?

My vote would be for the server to abstract this detail and have the default the same irrespective of the type of model, so that the external behaviour is equivalent.

This is a bit annoying as return_full_text default value is not the same between CausalLMs (return_full_text=True) and Seq2SeqLMs (return_full_text=False).

What do you mean ? The default in transformers is always True. It just happens that for seq2seq the initial text is usually [BOS] so it gets discarded anyway during decoding. So return_full_text=False really only has an impact for decoder only.
(To be perfectly honest the param is not implemented for text2text-generation since the two pipelines are actually separate in transformers)

Given how this server abstracts over the two kinds of models, might it make sense to prepend the encoder input when return_full_text is specified as True? Perhaps the parameter could be named include_input or something to avoid confusion with the transformers parameter?

I think alignment with transformers  is desirable.
I think for encoder-decoder, it doesn't make sense to include the prompt beforehand. Because the prompt is encoded, and is only used as guidance for the decoding part. (So there can be 1 or more encoding, which can even be text or different modality).
The real previous text is the decoder_start_ids  if you want (usually bos but sometimes more complex like [bos, <language_en>, <start_of_translation>]. But enabling users to specify the start with maybe some readable tokens can be useful too.


Products tab needs to show file sizes

Products tab needs to show file sizes. This would help for users deciding if its worth the effort to download certain files. This is really important for some of the larger files like DSM’s and DEM’s

This is completed with the caveat that it will only show for data processed going forward.


Update JSFiddle examples in blog post

Looks like the new API in 0.5.0 broke your excellent blog post.
I would suggest either pinning the JSFiddles to a commit, or reworking them to use mobservable.ObservingComponent. I have done so in this update.

Awesome, thanks! I had it as pending todo, but then I lost my internet connection and forgot about it. Thanks for fixing :)


How can I deploy webpage from WGLMakie?

I have create an interactive web app from WGLMakie + JSServe.
I want to deploy it as a standalone html (maybe + javascript) webpage so that I can deploy it.
How can I do that? I cannot find any tutorials on the document.

You could do this via Franklin, there's a demo here: https://franklinjl.org/demos/wgl/ the source page is https://github.com/tlienart/Franklin.jl/blob/master/demos/wgl.md
Alternatively you could just write the code in the same form, remove the ~~~ stuff and put that in a .html file, that's what they do here https://makie.juliaplots.org/stable/documentation/backends/wglmakie/#export

Can you give some more context? E.g. do you need to have a connection to a running Julia session, or do you want to export a static page?
I wanted to prepare a demo for this soon...

I just found there are some examples in JSServe.jl. Here is for a standalone static page, and another serves a Julia session.
Both of them requires to change some widgets into JSServe's widgets. It's a bit hacky.
For my current project, I am developing a physical animation. There are some sliders for users to control some parameters.
Some 2D and 3D plots are going to refresh on the screen simultaneously. It can be a standalone web page or running a Julia session.
For future, a Julia session backend would require to calculate some values and read/write to files or database.


PR: Install pyls locally without modifying site-packages

Description of Changes
This PR installs pyls on a external directory different to site-packages, by doing so, Spyder won't remove any user-installed pyls

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @andfoy

/show binder

@andfoy Can you please explain what this means for us, developers (or point to an existing description)? I never really understood all the magic that bootstrap.py is doing with pyls and spyder-kernels; I just know that it usually works, but sometimes it does not ...

@jitseniesen, basically Spyder starts with its own development version of pyls, due to the difficulty and complexity of producing releases each time a critical feature/bug is introduced there (pyls is controlled by Palantir and we have limited permissions). So basically we have a local version of pyls independent of any other version that would exist on site-packages, which point to the latest develop/branch commit that we require on Spyder

@andfoy Sounds good, thanks for the explanation!

That's the big picture @jitseniesen. Regarding this PR, it changes the way we use the subrepo present in external-deps/python-language-server from bootstrap.py to not remove the current version present in site-packages and reinstall it again in development mode (which is the solution I came up with in PR #12389).
Instead, @andfoy found a brilliant solution: install that subrepo in development mode in a temporary directory and patch sys.path to use it (this and removing that temp dir after closing Spyder is handled by bootstrap.py now). That way we don't need to touch the PyLS version present in site-packages anymore, which won't break people's environments from now on.


how do we track the checked status of OptionsModel elements

I am using the multi select module with ng2-smart-table module and wish to keep a tab on the status of the checkbox in the multi-select drop-down for performing Function.
Kindly share the details by which we can access the isChecked property inside our components.

interested in this as well


🐛 Fix imgs with neurips template + export to .tex

Fixes issues when exporting with the neurips template (it was missing the graphicx package).
Also found that the neurips template referred to academic.tex rather than neurips.tex so I fixed it.

Thanks!


Redux example error while running build

Redux example
next build
next start

Error in browser console:
VM3895:2 You are currently using minified code outside of NODE_ENV === 'production'. This means that you are running a slower development build of Redux. You can use loose-envify (https://github.com/zertosh/loose-envify) for browserify or DefinePlugin for webpack (http://stackoverflow.com/questions/30030031) to ensure you have the correct code for your production build.
Is there a way to use NODE_ENV === 'production' with Next.js build/start scripts?

Same issue here, this error message (You are currently using minified code outside of NODE_ENV === 'production'.) were printed on the console in a production environment.

Have tried the following but no lucks:
  "scripts": {
    "dev": "next",
    "build": "NODE_ENV=production next build",
    "start": "NODE_ENV=production next start"
  },


New version: Google.Chrome.Canary version 99.0.4764.0

ARP entries are correct.
Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 1047
Microsoft Reviewers: Open in CodeFlow

/azp run


Move all of document to READEME

I think  swag don't have a lot of document for needs. And a lot of PPL asking how to use'swag and they don't to read our another website.  So for more clarity, I prefer to add all document at  https://github.com/swaggo/swag/blob/master/README.md instead of outside document. /cc @pei0804

I will consider moving.


[Python] Allow ints to be considered as floats/complexes and floats as complexes

Currently, python Beam transforms will complain about type incompatibilities if you try to supply ints for floats/complexes or floats as complexes. It's not hard to work around this but it comes up fairly often and is a bit tedious to fix over and over when ints work fine in place of floats/complexes.
PEP484 states that ints should be compatible with floats/complexes and floats as complexes, so this change just updates our type check to allow for these special cases
Dev mail thread on this

(Looks like there's a couple more tests to update.)

bump, I think this should be all set


Update deps

Install this would downgrade a lot of packages. Are fixed versions in compat necessary?

Well, that the versions I tested against.
But it could also be the result of my limited knowledge on how to specify the versions in the file. I got some errors messages from the package registry, that at the time push me into these fixed versions numbers.
This being said,  I could have look into it and check that the test passes with other versions as well.
Could you share which packages are particularly problematic from your perspective?

I made some changes that you can test by using the version associate with this issue.
In the Julia package manager you can use the following command to add this version:
add https://github.com/ericqu/LinearRegressionKit.jl#16-update-deps
let me know if this solve your problem or not.

This should be fixed in version 0.7.11, which will become the default soon.


Error: Enable to run Lora - Adapter files are empty

Describe the bug
when trying to run mistral server with a lora repeatedly getting Error: Adapter files are empty. Perhaps the ordering file adapters does not match the actual adapters?
have tried this on different systems and also changing to different file paths and checking - in any permutation combination still getting the same issue
Folder Dir:
Directory structure for: \test_lora
├── config.toml
├── mistralrs-server.exe
├── ordering.json
└── [sample_data_adapter]
    ├── README.md
    ├── adapter_config.json
    ├── adapter_model.safetensors
    ├── special_tokens_map.json
    ├── tokenizer.json
    └── tokenizer_config.json

ordering.json file:
{
    "order": ["sample_data"],
    "layers": {
        "model.layers.0.self_attn.q_proj": 0,
        "model.layers.0.self_attn.k_proj": 1,
        "model.layers.0.self_attn.v_proj": 2,
        "model.layers.0.self_attn.o_proj": 3,
        "model.layers.0.mlp.gate_proj": 4,
        "model.layers.0.mlp.up_proj": 5,
        "model.layers.0.mlp.down_proj": 6
    },
    "base_model_id": "meta-llama/Llama-3.2-1B-Instruct",
    "preload_adapters": [{
        "name": "spiritual_love",
        "adapter_model_id": "sample_data_adapter"
    }]
}

cmds tried:
./mistralrs-server -i lora --model-id meta-llama/Llama-3.2-1B-Instruct --adapters-model-id ./sample_data_adapter --order ./ordering.json
mistralrs-server -i toml -f lora_config.toml  (when using toml, only base model ran without the lora)
both for linux and windows, mistralrs-server was built from source
Latest commit or version
running version 0.3.2/git pull as of 22_11_24

Hello again, just reverting back on this - if anyone could check and help me out - Please and thank you..


Include statement incorrect

In the SparkFun_MPL3115A2.cpp file it says #include "SparkFunMPL3115A2.h" when I believe it should be #include "SparkFun_MPL3115A2.h". I am receiving errors because it says "SparkFunMPL3115A2.h" does not exist.

You are correct. I thought I fixed all of the includes when I changed to the underscore in the name but missed this one. It should be fixed now. Thanks!


Migrate from rollup-plugin-userscript-metablock to rollup-plugin-userscript

What are these changes?
Replace rollup-plugin-userscript-metablock with rollup-plugin-userscript.
Why are these changes being made?
A recent update of<EMAIL_ADDRESS>caused this build to fail.
[!] SyntaxError: The requested module 'rollup-plugin-userscript-metablock' does not provide an export named 'default'
file:///app/rollup.config.js:4
import metablock from "rollup-plugin-userscript-metablock";
       ^^^^^^^^^
    at ModuleJob._instantiate (node:internal/modules/esm/module_job:171:21)
    at ModuleJob.run (node:internal/modules/esm/module_job:254:5)
    at ModuleLoader.import (node:internal/modules/esm/loader:475:24)
    at getConfigFileExport (/app/node_modules/rollup/dist/shared/loadConfigFile.js:471:17)
    at Object.loadConfigFile (/app/node_modules/rollup/dist/shared/loadConfigFile.js:430:59)
    at getConfigs (/app/node_modules/rollup/dist/bin/rollup:1727:39)
    at runRollup (/app/node_modules/rollup/dist/bin/rollup:1698:43)

(this appears to have been resolved as of<EMAIL_ADDRESS>rollup-plugin-userscript seems like a better alternative to rollup-plugin-userscript-metablock:

It's owned by the Violentmonkey organization on GitHub.
It uses TypeScript.
The repository is half the size, and seems a lot less convoluted.
It has an automated deployment script, and doesn't store build artifacts in the repo.

coverage: 90.646%. remained the same
when pulling 96e3c00dc5bad34500a4729b8e629f8ccc285430 on rollup-plugin-userscript
into 946e82746473d9dcc0d8c002ce8334c4a98a9051 on mainline.

LGTM


vscode commandline switches not working as per the vscode cli url https://code.visualstudio.com/docs/editor/command-line

VSCode Version: 1.44.2
OS Version: Windows 10 Enterprise, Version id: 1803

Steps to Reproduce:
Based on the url https://code.visualstudio.com/docs/editor/command-line
1. Download the zip file based VSCode for windows (regular or insiders) and unzip to a folder.
2. cd to unzipped folder in a command window.
3. Run code --version  (or code --help)
4. Expect it to show 1.44.2 or 1.45 (for insiders) (or for code --help the cli options)

Does this issue occur when all extensions are disabled?: Yes/No Yes

Any one else facing this ?
I need this to work so I can automate vscode installs within our network..
-N

That's because you are launching Code.exe directly and it opens application.
On the other hand  VS Code installation adds .../Microsoft VS Code/bin to PATH and therefore bin/code.cmd will be used when you launch it from command line, which correctly handles CLI arguments.

That's because you are launching Code.exe directly and it opens application.
On the other hand VS Code installation adds .../Microsoft VS Code/bin to PATH and therefore bin/code.cmd will be used when you launch it from command line, which correctly handles CLI arguments.

Awesome Sir.  I tried running the .cmd file this time and spot on. It worked.
Thank you so much for your help, time and attention.  Appreciate that very much.
-N


Rider Plugin Update

I cannot install the Rider Plugin because it is not compatible with the current Rider version

Rider plugin is maintained by Rider, not us. Please consider opening the issue in this repo or Rider bug tracker.
https://github.com/ForNeVeR/AvaloniaRider


Convert tabs in multiline strings to 4 spaces

In the past this repo used tabs instead of 4 spaces for .kt files.
Ktlint doesn't convert tabs in multiline strings to spaces.
Please take a look at the following issue.
pinterest/ktlint#575
Therefore, the following command has been used for conversion.
find ./ -iname '*.kt' -type f -exec bash -c 'expand -t 4 "$0" | sponge "$0"' {} ;

./ is recursively searching the given directory
-iname is a case insensitive match .kt files
type -f finds only regular files (no directories, binaries or symlinks)
-exec bash -c executes the following commands in a subshell for each file
expand -t 4 expands all tabs to 4 spaces
sponge soaks up standard input (from expand) and writes it to the same file

Source:
https://stackoverflow.com/a/43523362

Thanks for doing this!


Increase number of char possible in inter: method

Increase the number char possible in the field "method", currently varchar(35). Some networks have various method, so we need an increase of the number of char. (maybe varchar(50)?)

Done in commit 32b1f95, convert to VARCHAR(255)


Add "Toggle capture" button and menu item, toggle capture with 'X' from everywhere

Add "Toggle capture" button at the top of the "capture" tab
Add and use QAction to toggle capture with 'X' from everywhere
Also add such action to the "File" menu.
Also add Ctrl+S and Ctrl+O shortcuts to save and open a capture.
Bug: http://b/156245865

A (somewhat) more sophisticated attempt than #543, that I closed.

Screenshot:

Another comment: I would also not add a menu item, the button and shortcut are enough and you just clutter the menu with an item that doesn't really belong there.

Thanks, I think it's important we add this, just two quick comments:

I would like to change the wording on this to be "Start capture" and once you press this, switch to "Stop capture". Makes it much clearer which state you are in.
Buttons that extend over the entire length of a view like this are really ugly. I don't have a specific suggestion how to place this, but we should refine this.

I agree and I definitely thought about it, but in the end I went for this for simplicity. I'll look into it.
The point is that I don't really know where to place the button. If I leave it in the same place but have it as wide as the text (assuming I'm capable of that, not exactly a Qt expert here), then it might still be just as ugly with all that empty space on the two sides.

Another comment: I would also not add a menu item, the button and shortcut are enough and you just clutter the menu with an item that doesn't really belong there.

I don't really agree that it doesn't belong, but either way, the reason for that was that it was the way way I found to have a QAction that can respond to 'X' from anywhere (apparently I can't attach it to the button itself). There's probably a better way but that was the first one I found.

I don't really agree that it doesn't belong, but either way, the reason for that was that it was the way way I found to have a QAction that can respond to 'X' from anywhere (apparently I can't attach it to the button itself). There's probably a better way but that was the first one I found.

I would not make UI choices based on "that's how I was able to implement it". Could you make this a hidden menu item as a workaround? (That way we don't have the menu item, but the implementation would still work.)

I don't really agree that it doesn't belong, but either way, the reason for that was that it was the way way I found to have a QAction that can respond to 'X' from anywhere (apparently I can't attach it to the button itself). There's probably a better way but that was the first one I found.

I would not make UI choices based on "that's how I was able to implement it". Could you make this a hidden menu item as a workaround? (That way we don't have the menu item, but the implementation would still work.)

Of course, but the two things seemed to go hand in hand with each other for me, as I said it seemed fitting to give the main action of the program a place in the menu. But if you don't want it to be there I'll find another way.

Hi Davide, I'm not sure about the 'X' from anywhere feature, don't we risk triggering unwanted captures when entering text thinking that a filter has focus for example?
Concerning the button, I was thinking about making a start/stop capture button in the OpenGl window, much like the "triangle button" to collapse tracks work.  I would have placed it in the top dark grey bar, probably in place of the "press H for help" text.  I was thinking of a red circle that would transform to a red square when pressed and vice versa.  This could be a good opportunity for you to get into the rendering code if you wanted btw, I could also make a prototype tomorrow.

Hi Davide, I'm not sure about the 'X' from anywhere feature, don't we risk triggering unwanted captures when entering text thinking that a filter has focus for example?

That's a good point. I think all we need is a button. This addresses discoverability on how to start/stop a capture, which is the primary concern.

I'm not sure about the 'X' from anywhere feature, don't we risk triggering unwanted captures when entering text thinking that a filter has focus for example?

Well, I agree, this was because of http://b/156245865. In this pull request I had made it so that it would work when any tab is selected, but even restricting it to only when the "capture" tab is selected the problem would be the same. Also what if you're navigating the trace with the keyboard and instead of hitting for example 'S' you hit 'X' by mistake? Maybe we should change the shortcut to Ctrl-X or something like that? Or maybe the the button itself is enough?

Concerning the button, I was thinking about making a start/stop capture button in the OpenGl window, much like the "triangle button" to collapse tracks work. I would have placed it in the top dark grey bar, probably in place of the "press H for help" text. I was thinking of a red circle that would transform to a red square when pressed and vice versa. This could be a good opportunity for you to get into the rendering code if you wanted btw, I could also make a prototype tomorrow.

I think there should be text like "Start capture" and "Stop capture" to make it very clear. A red circle may mean "press here to start recording" or "we are already recording". Other tools use a "Play" button (triangle pointing to the right), though I also don't really like it.

Or maybe the button itself is enough?

My vote is for just a button.

I think there should be text like "Start capture" and "Stop capture" to make it very clear.

Fair enough, in this case, I think using ImGui would be the easiest way to go if we want to place the button in the top bar of the capture window, using less vertical space.  Let me create a prototype real quick... won't be long.

Well, I agree, this was because of http://b/156245865.

Apologies for not being explicit about how to actually resolve the issue raised in that bug (that users have to click into the capture field before they can press 'x' to capture).
In general, we should explicitly discuss all user visible changes in more detail and how to resolve them between at least @pierricgimmig and myself. IMO, the "discussion" can also be a PR and prototype, sometimes that's easier than other methods, but the downside is that some work will likely get thrown out.


Change the default CallbackUrlResolver for Azure

LGTM. Thanks!

Do you think a patch release might become available to include azure-related changes before August 17?

@mmoayyed I think we could backport our minor changes on the 3.0.x branch and I could release a 3.0.3 version this week. Sounds good to you?

I cherry-picked the two commits on the 3.0.x branch: can you ensure everything is ok on the CAS side when using the version 3.0.3-SNAPSHOT? If so, I'll cut the release...

On the CAS side, it actually turns out that both master and the 5.3.x branch are based on 3.1.0-SNAPSHOT. When do you think might be a good time to cut 3.1.0?

I see only one thing blocking the 3.1.0 release: https://github.com/pac4j/pac4j/issues/1154
Let's see if we can tackle that before the deadline...


people_count_app.py

Hi Akhtar,
how do you use people_count_app.py ?

@raylooi  it's just integration of  same code with Flask so just run python people_count_app.py and using any other Flask API or Postman  to hit '/People_Count' API  which define in people_count_app.py to give path of any video path or IP camera path.

closed


Problem with external Dll

Hello good day.
I'm using a project and I saw some problems, but this only occurs in the local version>
When I import an external dll (Serilog.AspNetCore for example), on the site it does not return results. (Follows images)

In its published version, it returns.

Could you tell me what changes in the published version but the git version?
Thank you

U must run in your computer.
And resolve serilog package!

Something like refs.Add(MetadataReference.CreateFromImage(byte[] image));

No guide no explanation how to use monaco with cs. It is sad!

Ive downloaded repo, updated nugets.
Add Serilog package from nuget.
Set to run from kestrel!
And voila.

I guesse differnce with published version, that published version included Serilog package.
When U ve pressed Build, it resolves packeges that all ready loaded with Your solution!
Its Roslyn Baby.

I guesse differnce with published version, that published version included Serilog package.
When U ve pressed Build, it resolves packeges that all ready loaded with Your solution!
Its Roslyn Baby.

Hello, i resolved incluing Serilog package and worked. Thank you!

I guesse differnce with published version, that published version included Serilog package.
When U ve pressed Build, it resolves packeges that all ready loaded with Your solution!
Its Roslyn Baby.

Hello, i resolved incluing Serilog package and worked. Thank you!


860022: Fix password replacement bug in setup-tomcat-ssl-cert.sh

Replacing the sed command used to replace passwords in setup-tomcat-ssl-cert.sh with awk, as the sed command failed when the password contained the @ symbol.
https://internal.almoctane.com/ui/entity-navigation?p=131002/6001&entityType=work_item&id=860022

A developer build has not yet been created for this branch. Click here to go ahead and create the build...

CI Build Link:
http://sou-jenkins2.swinfra.net:8080/job/CAFapi/job/CAFapi~opensuse-tomcat-jre11-image~860022~CI


Start multiple services in the same process, zookeeper Only one service was registered

func main()  {
go Test("1")
Test("2")
}
func Test(name string)  {
reg := zookeeper.NewRegistry(func(options *registry.Options) {
options.Addrs = []string{"localhost:2181"}
})
srv := micro.NewService(
	micro.Name("service.core.test"+name),
	micro.Registry(reg),
	micro.Transport(grpc.NewTransport()),
	micro.BeforeStart(func() error {
		return nil
	}),
)

srv.Init()

srv.Run()

}
only register service.core.test1, how to regsiter more service in same process

You must provide an entirely new Server to each new service you create otherwise it makes use of a default server.


Default date issue in the React big calendar

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/funny-booth-hsub7o?file=/src/App.js
Bug description
The event is not showing on the calender when I pass the defaultDate={new Date()}. But when I pass the date with this format defaultDate={new Date(2022, 12, 27)} It shows up. Can you help me out with this issue @jquense . Thanks
Expected Behavior
<Calendar
onNavigate={onNavigate}
events={myEventsList}
defaultDate={new Date()}
localizer={localizer}
step={15}
// timeslots={8}
defaultView='day'
views={views}
style={{height: 600}}
/>
Actual Behavior
<Calendar
onNavigate={onNavigate}
events={myEventsList}
defaultDate={new Date(2022, 12, 27)}
localizer={localizer}
step={15}
// timeslots={8}
defaultView='day'
views={views}
style={{height: 600}}
/>
react-big-calendar version
1.5.2
React version
18.2.0
Platform/Target and Browser Versions
Chrome
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a react-big-calendar issue and not an implementation issue

Would you like to open a PR for this bug?

[x] I'm willing to open a PR

This is not a bug in react-big-calendar.
new Date(2022, 12, 27) returns the 27th of January 2023!
If you are already importing moment, you will probably want to also use it to manipulate dates.
You don't have to pass defaultDate={new Date()}, because by default the calendar opens up at the current day.
See also understanding dates for how to use the defaultDate prop properly.


[BUG] MAC OS X command + c broke all

After single press command + c HumanInput thinks that I hold c
After 5000ms pseudo hold HumanInput react on watchers and thinks that already c holded
After some timeout HumanInput starts work normaly, in DEBUG no info
http://jsbin.com/bexojekapi/1/edit?html,js,output
http://imgur.com/DZtvWpp

After pressing only Enter the same

What browser/version are you using?

Chrome 52.0.2743.116 (64-bit)

I just commandeered my wife's Mac and can confirm this behavior.  It appears to be a bug in Mac OS X itself since it is exhibited in Chrome, Firefox, and Safari.  Essentially, the browser does not fire the keyup event if the command key is held at the same time.  The only keyup event you'll get is for the command key itself.
I'll see if I can implement a workaround that resets the state of all keys held during a command+ keystroke on Macs (to prevent the hold event from misfiring) but I can't work around the fact that on Macs you'll never get that keyup event in these situations.  So something like HI.on('command-c', doStuff) would work but if you also had a keyup:c event at the same time it would never get triggered if the command key was held during a combo.
Interestingly, this bug doesn't appear to effect the other modifier keys.  So command-shift would work without issue.

Ok

I just pushed a commit that should fix this issue.  Can you pull the latest code and confirm/deny?

Yep, 1.1.13 fix this
http://jsbin.com/nodofoguza/edit?html,js,output

But

After pressing only Enter and HI.pause() - HumanInput thinks that I hold Enter

Is still here

Try it http://jsbin.com/kahusajica/1/edit?html,js,output

OK I've finally got around to trying to test this on a Mac and I cannot reproduce it.  Is this still happening?  Here's what I've been using to test:
function onLeft() {
    console.log('LEFT')
}

function onEnter() {
    HI.pause();
    console.log('ENTER PAUSE');
}
HI.off('left'); HI.off('enter');
HI.on('left', onLeft);

HI.on('enter', onEnter);

Then I check HI.state to see if there's anything in the down or downAlt arrays but every time I test it they're both empty (as they should be).  It doesn't think that the enter key is still held down.
I tested in Chrome and Safari on a Mac (no idea what version it was--only had it for a few minutes).  What browser were you using?


Add Licence and Copyright to Repo

Related #232
We likely should have Copyright and License files in our repo as well as headers in the source files.
I believe we should be using Apache 2 just like the tests and build repos.
https://github.com/AdoptOpenJDK/openjdk-build/blob/master/LICENSE

I have added the Apache2 License to the repo

@gdams Please review, might be able to close this.


[InputUnstyled] ownerState is a required field for componentsProps and is appended to a DOM element

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
Currently, ownerState field is required as per type definition of componentsProps
componentsProps?: {
    root?: React.ComponentPropsWithRef<'div'> & {
        ownerState: InputOwnerState;
    };
    input?: React.ComponentPropsWithRef<'input'> & {
        ownerState: InputOwnerState;
    };
};

And the ownerState prop itself is being appended to input DOM element:

Expected behavior 🤔
ownerState is not a required field as per this guide and is not appended to en input DOM element
Steps to reproduce 🕹
Steps:

Import InputUnstyled component
Defined componentsProps property of component
See that ownerState is required
Define ownerState
See that ownerState is being appended to DOM element

Context 🔦
The issue makes development more complicated, as the need to understand some undescribed props appears and to understand it you need to dive deep into type definitions of components. In addition, unnecessary data is being appended to a DOM elements
Your environment 🌎

`npx @mui/envinfo`
    System:
    OS: Linux 5.13 Ubuntu 20.04.2 LTS (Focal Fossa)
  Binaries:
    Node: 16.14.2 - ~/.nvm/versions/node/v16.14.2/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 8.5.0 - ~/.nvm/versions/node/v16.14.2/bin/npm
  Browsers:
    Chrome: 99.0.4844.74
    Firefox: 99.0
  npmPackages:
    @mui/base: ^5.0.0-alpha.79 => 5.0.0-alpha.79
    @mui/types:  7.1.3
    @mui/utils:  5.6.1
    @types/react: ^18.0.5 => 18.0.5
    react: ^18.0.0 => 18.0.0
    react-dom: ^18.0.0 => 18.0.0
    typescript: ^4.6.3 => 4.6.3

tsconfig:
{
  "compilerOptions": {
    "target": "es5",
    "lib": [
      "dom",
      "dom.iterable",
      "esnext",
      "es2021.intl",
      "ES2017.Intl",
      "ES2018.Intl"
    ],
    "baseUrl": "src/",
    "allowJs": true,
    "skipLibCheck": true,
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
    "strict": true,
    "forceConsistentCasingInFileNames": true,
    "noFallthroughCasesInSwitch": true,
    "module": "esnext",
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "isolatedModules": true,
    "noEmit": true,
    "jsx": "react-jsx"
  },
  "include": [
    "./src/**/*.ts",
    "./src/**/*.tsx"
  ]
}

Hi @gabunia-nick ,
Can you provide a CodeSandbox showing the issue? It would really help us investigate.

Hi @ZeeshanTamboli
Here is CodeSandbox showing the issue: https://codesandbox.io/s/sad-bassi-cvdk08?file=/src/App.tsx

Hi @gabunia-nick ,
It will be fixed when #32491 is released. There are quite a few type changes there.
I can confirm with the preview package: https://codesandbox.io/s/mui-material-ui-forked-315rvr?file=/src/App.tsx.


script exits after starting first epoch

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: NA
TensorFlow installed from (source or binary): binary
TensorFlow version: 2.5.0
Python version: 3.6
Installed using virtualenv? pip? conda?: pip
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: CUDA : 11.2, cuDNN : <IP_ADDRESS>
GPU model and memory: GEFORCE RTX 3090

Describe the problem
python code is exiting after starting first epoch with the exit code error
"Process finished with exit code -1073740791 (0xC0000409)"
when I debugged it is observed that code is exiting when
function
tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
inputs, attrs, num_outputs)
executing
Provide the exact sequence of commands / steps that you executed before running into the problem
just installed all the packages and updated the script for tensorflow 2.5.0 and using pycharm started running the script
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
2021-08-13 14:28:59.820107: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library cudart64_110.dll
3670
Found 3670 files belonging to 5 classes.
Using 2936 files for training.
2021-08-13 14:29:02.684243: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library nvcuda.dll
2021-08-13 14:29:02.767638: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1733] Found device 0 with properties:
pciBusID: 0000:15:00.0 name: NVIDIA GeForce RTX 3090 computeCapability: 8.6
coreClock: 1.695GHz coreCount: 82 deviceMemorySize: 24.00GiB deviceMemoryBandwidth: 871.81GiB/s
2021-08-13 14:29:02.768344: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1733] Found device 1 with properties:
pciBusID: 0000:2d:00.0 name: Quadro P2000 computeCapability: 6.1
coreClock: 1.4805GHz coreCount: 8 deviceMemorySize: 5.00GiB deviceMemoryBandwidth: 130.53GiB/s
2021-08-13 14:29:02.769004: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library cudart64_110.dll
2021-08-13 14:29:02.815082: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library cublas64_11.dll
2021-08-13 14:29:02.815441: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library cublasLt64_11.dll
2021-08-13 14:29:02.820867: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library cufft64_10.dll
2021-08-13 14:29:02.823516: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library curand64_10.dll
2021-08-13 14:29:02.836259: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library cusolver64_11.dll
2021-08-13 14:29:02.840952: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library cusparse64_11.dll
2021-08-13 14:29:02.842131: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library cudnn64_8.dll
2021-08-13 14:29:02.842526: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1871] Adding visible gpu devices: 0, 1
2021-08-13 14:29:02.843140: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX AVX2
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2021-08-13 14:29:03.199398: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1733] Found device 0 with properties:
pciBusID: 0000:15:00.0 name: NVIDIA GeForce RTX 3090 computeCapability: 8.6
coreClock: 1.695GHz coreCount: 82 deviceMemorySize: 24.00GiB deviceMemoryBandwidth: 871.81GiB/s
2021-08-13 14:29:03.200104: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1733] Found device 1 with properties:
pciBusID: 0000:2d:00.0 name: Quadro P2000 computeCapability: 6.1
coreClock: 1.4805GHz coreCount: 8 deviceMemorySize: 5.00GiB deviceMemoryBandwidth: 130.53GiB/s
2021-08-13 14:29:03.200791: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1871] Adding visible gpu devices: 0, 1
2021-08-13 14:29:04.474905: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1258] Device interconnect StreamExecutor with strength 1 edge matrix:
2021-08-13 14:29:04.475271: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1264]      0 1
2021-08-13 14:29:04.475507: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1277] 0:   N N
2021-08-13 14:29:04.475734: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1277] 1:   N N
2021-08-13 14:29:04.476211: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1418] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 18786 MB memory) -> physical GPU (device: 0, name: NVIDIA GeForce RTX 3090, pci bus id: 0000:15:00.0, compute capability: 8.6)
2021-08-13 14:29:04.478811: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1418] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:1 with 3625 MB memory) -> physical GPU (device: 1, name: Quadro P2000, pci bus id: 0000:2d:00.0, compute capability: 6.1)
Found 3670 files belonging to 5 classes.
Using 734 files for validation.
['daisy', 'dandelion', 'roses', 'sunflowers', 'tulips']
2021-08-13 14:29:05.999112: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:176] None of the MLIR Optimization Passes are enabled (registered 2)
(16, 180, 180, 3)
(16,)
0.0 0.9627812
Model: "sequential"

Layer (type)                 Output Shape              Param #
rescaling_1 (Rescaling)      (None, 180, 180, 3)       0

conv2d (Conv2D)              (None, 180, 180, 16)      448

max_pooling2d (MaxPooling2D) (None, 90, 90, 16)        0

conv2d_1 (Conv2D)            (None, 90, 90, 32)        4640

max_pooling2d_1 (MaxPooling2 (None, 45, 45, 32)        0

conv2d_2 (Conv2D)            (None, 45, 45, 64)        18496

max_pooling2d_2 (MaxPooling2 (None, 22, 22, 64)        0

flatten (Flatten)            (None, 30976)             0

dense (Dense)                (None,<PHONE_NUMBER>

dense_1 (Dense)              (None, 5)                 645
Total params: 3,989,285
Trainable params: 3,989,285
Non-trainable params: 0

Epoch 1/10
2021-08-13 14:29:08.463012: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library cudnn64_8.dll
Process finished with exit code -1073740791 (0xC0000409)

@sujithsudhi,
Please share a simple standalone code to expedite the process. Thanks!

Tried with same code from tensorflow 2.5.0 examples, but got the same
import matplotlib.pyplot as plt
import numpy as np
import os
import PIL
import tensorflow as tf
from tensorflow import keras
from tensorflow.keras import layers
from tensorflow.keras.models import Sequential
import pathlib
dataset_url = "https://storage.googleapis.com/download.tensorflow.org/example_images/flower_photos.tgz"
data_dir = tf.keras.utils.get_file('flower_photos', origin=dataset_url, untar=True)
data_dir = pathlib.Path(data_dir)
image_count = len(list(data_dir.glob('/.jpg')))
print(image_count)
roses = list(data_dir.glob('roses/'))
PIL.Image.open(str(roses[0]))
PIL.Image.open(str(roses[1]))
tulips = list(data_dir.glob('tulips/'))
PIL.Image.open(str(tulips[0]))
PIL.Image.open(str(tulips[1]))
batch_size = 16
img_height = 180
img_width = 180
train_ds = tf.keras.preprocessing.image_dataset_from_directory(
data_dir,
validation_split=0.2,
subset="training",
seed=123,
image_size=(img_height, img_width),
batch_size=batch_size)
val_ds = tf.keras.preprocessing.image_dataset_from_directory(
data_dir,
validation_split=0.2,
subset="validation",
seed=123,
image_size=(img_height, img_width),
batch_size=batch_size)
class_names = train_ds.class_names
print(class_names)
import matplotlib.pyplot as plt
plt.figure(figsize=(10, 10))
#for images, labels in train_ds.take(1):
for i in range(9):
ax = plt.subplot(3, 3, i + 1)
plt.imshow(images[i].numpy().astype("uint8"))
plt.title(class_names[labels[i]])
plt.axis("off")
for image_batch, labels_batch in train_ds:
print(image_batch.shape)
print(labels_batch.shape)
break
AUTOTUNE = tf.data.AUTOTUNE
train_ds = train_ds.cache().shuffle(1000).prefetch(buffer_size=AUTOTUNE)
val_ds = val_ds.cache().prefetch(buffer_size=AUTOTUNE)
normalization_layer = layers.experimental.preprocessing.Rescaling(1./255)
normalized_ds = train_ds.map(lambda x, y: (normalization_layer(x), y))
image_batch, labels_batch = next(iter(normalized_ds))
first_image = image_batch[0]
Notice the pixels values are now in [0,1].
print(np.min(first_image), np.max(first_image))
num_classes = 5
model = Sequential([
layers.experimental.preprocessing.Rescaling(1./255, input_shape=(img_height, img_width, 3)),
layers.Conv2D(16, 3, padding='same', activation='relu'),
layers.MaxPooling2D(),
layers.Conv2D(32, 3, padding='same', activation='relu'),
layers.MaxPooling2D(),
layers.Conv2D(64, 3, padding='same', activation='relu'),
layers.MaxPooling2D(),
layers.Flatten(),
layers.Dense(128, activation='relu'),
layers.Dense(num_classes)
])
model.compile(optimizer='adam',
loss=tf.keras.losses.SparseCategoricalCrossentropy(from_logits=True),
metrics=['accuracy'])
model.summary()
epochs=10
history = model.fit(
train_ds,
validation_data=val_ds,
epochs=epochs
)
acc = history.history['accuracy']
val_acc = history.history['val_accuracy']
loss = history.history['loss']
val_loss = history.history['val_loss']
epochs_range = range(epochs)
plt.figure(figsize=(8, 8))
plt.subplot(1, 2, 1)
plt.plot(epochs_range, acc, label='Training Accuracy')
plt.plot(epochs_range, val_acc, label='Validation Accuracy')
plt.legend(loc='lower right')
plt.title('Training and Validation Accuracy')
plt.subplot(1, 2, 2)
plt.plot(epochs_range, loss, label='Training Loss')
plt.plot(epochs_range, val_loss, label='Validation Loss')
plt.legend(loc='upper right')
plt.title('Training and Validation Loss')
plt.show()

@sujithsudhi,
Formatting in the above code is not clear, and can you share a colab gist for the same? Thanks!

I just used the script from the TensorFlow examples as a test script
https://www.tensorflow.org/tutorials/images/classification

@sujithsudhi,
I am able to run the code fine with colab and take a look at the gist here. Can you try installing the latest stable version of Tensorflow 2.6 and let us know if the issue still persists? Take a look at these links for your reference link1, link2. Thanks!

@sanatmpa1,
I have update the Tensorflow to 2.6 but still the same error. when I use Qadro P2000 (with its dependencies) training is fine.
anybody is using GEFORCE RTX 3090.?

@sujithsudhi,
Can you please take a look at this similar issue #45285, also you can checkout this #46673, #45170 which discussed about the similar problem. let me know if it helps. Thanks!


Add support for ATSAMD21E15B

Hi @shumatech
I am trying to program an ATSAMD21E15B and I get an error, is the chip supported?
How can I help to add support?
Thanks

I get the next error
`Set binary mode
readWord(addr=0)=0x20000ffc
readWord(addr=0xe000ed00)=0x410cc601
readWord(addr=0x41002018)=0x10011427
version()=v2.0 [Arduino:XYZ] Jun 27 2018 12:18:39
chipId=0x10010027
Connected at 921600 baud
readWord(addr=0)=0x20000ffc
readWord(addr=0xe000ed00)=0x410cc601
readWord(addr=0x41002018)=0x10011427
Atmel SMART device 0x10010027 found
write(addr=0x20000800,size=0x34)
writeWord(addr=0x20000830,value=0x10)
writeWord(addr=0x20000820,value=0x20001000)
Device       : ATSAMD21E15B
readWord(addr=0)=0x20000ffc
readWord(addr=0xe000ed00)=0x410cc601
readWord(addr=0x41002018)=0x10011427
Chip ID      : 10010027
version()=v2.0 [Arduino:XYZ] Jun 27 2018 12:18:39
Version      : v2.0 [Arduino:XYZ] Jun 27 2018 12:18:39
Address      : 8192
Pages        : 384
Page Size    : 64 bytes
Total Size   : 24KB
Planes       : 1
Lock Regions : 16
Locked       : readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
readWord(addr=0x41004020)=0xffff
none
readWord(addr=0x41004018)=0
Security     : false
Boot Flash   : true
readWord(addr=0x40000834)=0x7000a
BOD          : true
readWord(addr=0x40000834)=0x7000a
BOR          : true
Arduino      : FAST_CHIP_ERASE
Arduino      : FAST_MULTI_PAGE_WRITE
Arduino      : CAN_CHECKSUM_MEMORY_BUFFER
Erase flash
chipErase(addr=0x2000)
done in 0.112 seconds
Write 9880 bytes to flash (155 pages)
write(addr=0x20000c00,size=0x1000)
writeBuffer(scr_addr=0x20000c00, dst_addr=0x2000, size=0x1000)`

Set binary mode
version()=v2.0 [Arduino:XYZ] Jun 27 2018 12:18:39
Connected at 921600 baud
readWord(addr=0)=0x20000ffc
readWord(addr=0xe000ed00)=0x410cc601
readWord(addr=0x41002018)=0x10011427
write(addr=0x20000800,size=0x34)
writeWord(addr=0x20000830,value=0x10)
writeWord(addr=0x20000820,value=0x20001000)
version()=v2.0 [Arduino:XYZ] Jun 27 2018 12:18:39
readWord(addr=0x41004018)=0
readByte(addr=0x804001)=0xc7
readByte(addr=0x804001)=0xc7
readByte(addr=0x804006)=0xff
readByte(addr=0x804007)=0xff
Device       : ATSAMD21x15
Version      : v2.0 [Arduino:XYZ] Jun 27 2018 12:18:39
Address      : 0x0
Pages        : 512
Page Size    : 64 bytes
Total Size   : 32KB
Planes       : 1
Lock Regions : 16
Locked       : none
Security     : false
BOD          : true
BOR          : true
Erase flash
chipErase(addr=0x2000)

Done in 0.112 seconds
Write 9880 bytes to flash (155 pages)
[                              ] 0% (0/155 pages)write(addr=0x20000834,size=0x1000)
writeBuffer(scr_addr=0x20000834, dst_addr=0x2000, size=0x1000)```

I had to comment on the line
https://github.com/shumatech/BOSSA/blob/master/src/Samba.cpp#L123
to work correctly

Above is a quick workaround for a bug that I suspect is hanging the flashing process. BOSSA assumes a writeBufferSize of 4096B but the chip only has that much ram in total. Relevant lines
https://github.com/shumatech/BOSSA/blob/d480a1ed34f4d08ac4822fe3aa5d1149b53b40c8/src/Samba.h#L85
https://github.com/shumatech/BOSSA/blob/d480a1ed34f4d08ac4822fe3aa5d1149b53b40c8/src/Samba.h#L89
If the sizes are suppose to be the same then may I also recommend refactoring them into a constant.

I have had the same is with a SAMR21S16 I reduce the buffer size to 2048 and it worked #110


Optional validation

Question

I have a schema that looks like this:

const schema = Nope.object().shape({
    name: Nope.string().required(),
    in_use_from: Nope.date().required(),
    address: Nope.object().shape({
        type: Nope.string().required(),
        virtual: Nope.object().shape({
            url: Nope.string().required(),
        }),
        physical: Nope.object().shape({
            site: Nope.object().shape({
                suite: Nope.string().required(),
                street: Nope.string().required(),
                suburb: Nope.string().required(),
                post_code: Nope.string().required(),
                state: Nope.string().required(),
                phone: Nope.string().required(),
            }),
            post: Nope.object().shape({
                suite: Nope.string().required(),
                street: Nope.string().required(),
                suburb: Nope.string().required(),
                post_code: Nope.string().required(),
                state: Nope.string().required(),
                phone: Nope.string().required(),
            })
        })
    })
})

address.type will either be a string 'physical' or 'virtual'.
I want to have the address.virtual object to be validated if address.type == 'virtual', and address.physical to be ignore.
If address.type == 'physical' then I want to have address.physical validated, while the address.virtual part of the validation is ignored.
How can I achieve this with Nope?
Were the docs unclear about it?

yes, there are not enough examples on the more complex uses

Hi @supagu, thanks for sharing 🐛
Unfortunately, what you want, the way you are doing it, is not possible. However, you can do this in a few ways, but they are all complex or "dirty" (using validateAsync or when for each nested field. Which doesn't seem useful at all given the number of fields you have.
I created a more simplified alternative, I understand that it may not be what you expected, but it works the way it should and I hope it meets your expectations. While exploring the problem, I realised that it might make sense to implement the when method for Object, this would make it easier for cases like yours.
const {Nope} = require('nope-validator');

const addressTypes = ['physical', 'virtual'];
const addressTypeError = 'Address type must be either "physical" or "virtual"';

const basePhysicalAddressSchema = Nope.object().shape({
  suite: Nope.string().required(),
  street: Nope.string().required(),
  suburb: Nope.string().required(),
  post_code: Nope.string().required(),
  state: Nope.string().required(),
  phone: Nope.string().required(),
});

const baseSchema = Nope.object().shape({
  name: Nope.string().required(),
  in_use_from: Nope.date().required(),
  address: Nope.object().shape({
    type: Nope.string().oneOf(addressTypes, addressTypeError).required(),
  }),
});

const physicalAddressSchema = Nope.object().shape({
  physical: Nope.object().shape({
    site: basePhysicalAddressSchema,
    post: basePhysicalAddressSchema,
  }),
});

const virtualAddressSchema = Nope.object().shape({
  virtual: Nope.object().shape({
    url: Nope.string().required(),
  }),
});

const MOCK_BASE_PHYSICAL_ADDRESS = {
  suite: 'A',
  street: 'B',
  suburb: 'C',
  post_code: '123-456',
  state: 'LS',
  phone: '9090',
};

{
  // address.type === 'physical'
  const data = {
    name: 'physical address',
    in_use_from: new Date(),
    address: {
      type: 'physical',

      physical: {
        site: MOCK_BASE_PHYSICAL_ADDRESS,
        post: MOCK_BASE_PHYSICAL_ADDRESS,
      },
    },
  };

  const baseSchemaErrors = baseSchema.validate(data);
  const physicalAddressSchemaErrors = physicalAddressSchema.validate(data.address);
  console.log(
    'address type is "physical", has errors?',
    baseSchemaErrors !== undefined,
    baseSchemaErrors,
  );
  console.log(
    'address type is "physical", has errors?',
    physicalAddressSchemaErrors !== undefined,
    physicalAddressSchemaErrors,
  );
}

{
  // address.type === 'virtual'
  const data = {
    name: 'virtual address',
    in_use_from: new Date(),
    address: {
      type: 'virtual',

      virtual: {
        url: 'http://www.google.com',
      },
    },
  };

  const baseSchemaErrors = baseSchema.validate(data);
  const virtualAddressSchemaErrors = virtualAddressSchema.validate(data.address);
  console.log(
    'address type is "virtual", has errors?',
    baseSchemaErrors !== undefined,
    baseSchemaErrors,
  );
  console.log(
    'address type is "virtual", has errors?',
    virtualAddressSchemaErrors !== undefined,
    virtualAddressSchemaErrors,
  );
}

{
  // address.type === 'invalid'
  const data = {
    name: 'invalid address',
    in_use_from: new Date(),
    address: {
      type: 'invalid',
    },
  };

  const baseSchemaErrors = baseSchema.validate(data);
  console.log(
    'address type is "invalid", has errors?',
    baseSchemaErrors !== undefined,
    baseSchemaErrors,
  );
}

You can run this code using this URL -<EMAIL_ADDRESS>


Timeout in waiting for login success!

Hi.
Adapter can't capture RegistrationToken header:
ERROR Timeout in waiting for login success!
ERROR Screenshot saved at: /opt/hubot/login-failure.png
ERROR Error: SkypeWeb adapter failure in initial login!

I tested different configurations: CentOS docker image with Node.js 0.10.36/0.10.41/4.2.3, Fedora 23 docker image, Fedora 23 desktop, different PhantomJS builds (from source, your build, etc). Adapter successfully connects only on one Fedora 23 desktop (Node.js 0.10.36, PhantomJS from source), but can't connect on similar Fedora 23 desktop or docker image.
This is login-failure.png screenshot from CentOS docker image:
Account status is offline. It seems what account not automatically connected after sign in.
What I can try to do to make this adapter work?

Just tested with another Skype account: adapter works fine.

After some time I get the same error with new Skype account. Patch by nagodon fixes the problem.


jellyfin: 10.4.2 -> 10.4.3

Motivation for this change
https://github.com/jellyfin/jellyfin/releases/tag/v10.4.3
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @nyanloutre @minijackson

This change is

@GrahamcOfBorg test jellyfin

Hey @filalex77 thanks for the update, but I think it was already taken care of in  #75145 ^^

@minijackson Didn't see it, sorry :)
Also, I think I pushed an extra commit by mistake :thinking:

No worries ^^


unable t trigger a dag

I am getting below error in response when i am triggering a dag via API:
"stderr": "/home/miniconda/lib/python2.7/site-packages/airflow/models.py:2160: PendingDeprecationWarning: Invalid arguments were passed to DataProcPySparkOperator. Support for passing such arguments will be dropped in Airflow 2.0. Invalid arguments were:\n*args: ()\n**kwargs: {'region': 'us-west1'}\n  category=PendingDeprecationWarning\n"
Below is GET request i am sending
http://XX.XX.XX.XX:8080/admin/rest_api/api?api=trigger_dag&dag_id=MLPipeline&run_id=dummyUser1_2018-06-07 13:21:49&conf=%7B%22customerName%22%3A%20%22dummyUser1%22%7D
What I am missing here?

Can you confirm what version of Airflow you're running?

Hello @aashishjoshi-ekryp,
It looks like this is actually working fine and the Error you're seeing is only a Depreciation warning. Can you confirm:

The exit code
If the dag_run is actually created or not
What you get from the stdout


Cannot install sidecar injector with heml

Downloaded 0.8 candidate.
helm template install/kubernetes/helm/istio --name istio --namespace istio-system --set prometheus.enabled=true > $HOME/istio.yaml
Followed instructions from here for sidecar injector with helm:
https://preliminary.istio.io/docs/setup/kubernetes/sidecar-injection.html#automatic-sidecar-injection
Got:
helm install --namespace=istio-system --set sidecar-injector.enabled=true install/kubernetes/helm/istio
Error: release loopy-stoat failed: configmaps "istio-mixer-custom-resources" already exists

delete step does not work if helm template was used , only if helm install was used:
helm delete --purge istio

cc @linsun

I can see how the docs are a little confusing.
There are two alternative workflows - one uses helm install / kubectl create - the other uses helm install (with no kubectl create).
The exact 3 steps that must be followed for istio with injection are:
https://preliminary.istio.io/docs/setup/kubernetes/helm-install.html#alternatively-use-helm-and-tiller-to-manage-the-istio-deployment

@andraxylia as sdake states the change was already made by @Tahler
It removes the confusing instructions.

Removing release blocking, if you think its not sufficient please suggest changes. Thanks.

ok to close


[DONT MERGE] [CI/Build] Reorganize models tests

To avoid timeout error, split up the models tests as per https://github.com/vllm-project/vllm/issues/7439#issuecomment-2301422056
I still haven't implemented the core_model flag yet.
cc @khluu

For some reason, splitting them up actually causes the multimodal tests to take longer than before...

That didn't seem to reduce the test time. @khluu any ideas?

The longer test time also occurs for other PRs. Hope that this PR didn't mess up the HF cache somehow...

Seems to be a temporary network issue, since the test times are back to normal now. Let's merge this before working on core_model.

LGTM in general, thanks for the huge efforts!
My only question is w.r.t. the fork of process for every test.

Nice reorganization


Unsupported Snapchat/OS version

Exception: You're using a version of Snapchat or operating system that's no longer supported. Please upgrade your device's operating system and update to the newest app version to use Snapchat. Thanks!

Just got that on mine as well.

I would recommend looking at this commit that seems to fix it https://github.com/mgp25/SC-API/commit/fc02c6b8449afe1e418f2e048bfca3ba26fc7922

I implemented what SC-API did here: https://github.com/matthazinski/snapy/commit/b32b9f4a90fb69f9344946d16426cd003e646120
They're sending client credentials over plain HTTP to http://casper.io though so I wouldn't recommend using it.

Well, Casper is an Android app that works well. Does that URL work with
https?
On Aug 31, 2015 5:02 PM, "Matt Hazinski"<EMAIL_ADDRESS>wrote:

I implemented what SC-API did here: matthazinski@b32b9f4
https://github.com/matthazinski/snapy/commit/b32b9f4a90fb69f9344946d16426cd003e646120
They're sending client credentials over plain HTTP to http://casper.io
though so I wouldn't recommend using it.
—
Reply to this email directly or view it on GitHub
https://github.com/tatosaurus/snapy/issues/4#issuecomment-136513244.

I think I'll implement it until i figure out a better way of doing it.
Thanks for your help! - tato
On Mon, Aug 31, 2015 at 5:02 PM, Matt Hazinski<EMAIL_ADDRESS>wrote:

I implemented what SC-API did here: matthazinski@b32b9f4
https://github.com/matthazinski/snapy/commit/b32b9f4a90fb69f9344946d16426cd003e646120
They're sending client credentials over plain HTTP to http://casper.io
though so I wouldn't recommend using it.
—
Reply to this email directly or view it on GitHub
https://github.com/tatosaurus/snapy/issues/4#issuecomment-136513244.

Nope, it's HTTP-only.


Increase range of retrieved forecast data

Acceptance Criteria

The range of retrieved forecast data is T+5 days (rather than T+1-5)

Test Checklist:

[x] Test Analysis & discussion with developer
[x] Write Test Cases / Charters
[x] Test Cases & Charters Reviewed
[ ] Test Cases / Charters Executed
[x] Regression tests executed
[ ] Write Automation tests for regression

Test Analysis
To test:
Test Case required:

Verify retrieving the forecast data generates the correct range of documents for a city

assert 41 documents for a city
assert 3hr increment on forecast_valid_time matching forecast_range

Candidate for automation: Yes (when there is a deployed env)
Test Charter
Explore retrieved forecast range using different cities to discover if range is implemented correctly
Regression candidate:

Yes

Only regression tested forecast db as in_situ is too unstable

Test case PASS

forecast_base_time
forecast_valid_time

2024-05-27T12:00:00.000Z
2024-05-27T12:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-27T15:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-27T18:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-27T21:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-28T00:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-28T03:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-28T06:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-28T09:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-28T12:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-28T15:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-28T18:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-28T21:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-29T00:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-29T03:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-29T06:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-29T09:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-29T12:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-29T15:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-29T18:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-29T21:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-30T00:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-30T03:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-30T06:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-30T09:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-30T12:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-30T15:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-30T18:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-30T21:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-31T00:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-31T03:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-31T06:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-31T09:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-31T12:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-31T15:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-31T18:00:00.000Z

2024-05-27T12:00:00.000Z
2024-05-31T21:00:00.000Z

2024-05-27T12:00:00.000Z
2024-06-01T00:00:00.000Z

2024-05-27T12:00:00.000Z
2024-06-01T03:00:00.000Z

2024-05-27T12:00:00.000Z
2024-06-01T06:00:00.000Z

2024-05-27T12:00:00.000Z
2024-06-01T09:00:00.000Z

2024-05-27T12:00:00.000Z
2024-06-01T12:00:00.000Z


Build(deps-dev): Bump @typescript-eslint/eslint-plugin from 5.33.1 to 5.34.0 (#3930)

Pull request opened by the merge tool on behalf of #3930

Merged in dc443efe8165b1df5961583a1b17ae4dd5de8505


include code links in documentation

Similar to the <> notation used in d3's docs:

Issue moved to d3plus/d3plus-dev #1 via ZenHub


Added support for the deployment scanner subsystem

SWARM-1415:  Add support for the deployment scanner subsystem

Thanks @Ladicek . I've moved it into the unsupported profile


Table parameter data types

Expected behavior
I want table to not only support string data, but also int and bool. I don't know how you achieve this in the current version, but I find it quite frustrating that you can't represent data in different types.
As a workaround I introduce an extra type column describing data types.
    | field              | value                | type           |
    |--------------------|----------------------|----------------|
    | pid                | PID-123              |                |
    | profile_level      | newbie               |                |
    | name               | James Bond           |                |
    | allow_bonus        | true                 | boolean        |
    | front_end_id       | 1234567              |                |
    | score_adjustment   | 1                    | integer        |

and convert types by hand.
My proposal should be to use quotes for strings. This would help recognize truthy values like true or false and use numbers.
    | field              | value                |
    |--------------------|----------------------|
    | pid                | "PID-123"            |
    | profile_level      | "newbie"             |
    | name               | "James Bond"         |
    | allow_bonus        | true                 |
    | front_end_id       | "1234567"            |
    | score_adjustment   | 1                    |

Actual behavior
All data in table parameters are returned as string type.
Steps to reproduce

Define step table parameter with values in different types.
Try read table data.
All data is represented in strings.

Gauge version
Gauge version: 0.5.0

Plugins
-------
html-report (2.1.1)
js (1.2.1)

This would be hard to implement in a way that's backward compatible and same across languages that's supported by Gauge.


Add support of Base Name in JSON decoding.

Tentative fix for #73
This breaks the integration test BootstrapTest::bootstrapSecure(). I'm not familiar enough with Leshan code to find the root cause.
Signed-off-by: David Navarro<EMAIL_ADDRESS>

Actually BootstrapTest::bootstrapSecure() test is a pass if I increase the timer. I guess my virtual machine is slow...

Bootstrap tests is ok for me.
It seems you have some formatter issue. This is not a big deal I can reformat it on my side before to integrated it. But if you want to easily configure eclipse to use the right formatter you can follow this instructions.

I made the suggested improvements.
I also removed the part checking that the base name is Under te requested UIT. This would prevent linked objects.
Sorry for the formatting. I can not install Eclipse IDE on this virtual machine.

Sry but why do you think the checking part in no more necessary ?
(no problem about formatting issue, I will do it)

In the LWM2M TS, look at the Object Link example in Appendix C. If the server does a Read on object A, the returned payload will be the one in the last example of chapter 6.3.4 JSON. Thus the requested path is "/A/0" and the returned base name is "/". With the previous check, this case would have failed.
Regards,

I think the basename path can be shorter than the request path but should match it.
I mean if request path is /1/0 a basename equals to /1/ or / is ok but not /2/.
It seems your code handle it correctly as you do the check only if the path is not /.
I will re-add the tests if you are ok ?

I think you are right...

This is integrated in master (commit 42f8a3923bad44a8b75b24bd0490dc5511078f77)
Thx @dnav


Add support for apollo-link-batch-http

We'd like to use apollo-link-batch-http in our app as it seems like it'd work well with Ember Routes and reducing requests as our app loads.
Obviously, this will by default mean it's included in the vendor bundle even if you don't use it, so before we go and add a PR for this, I thought I'd throw out the idea of adding some build time configuration to specify which packages from Apollo are included?

Yes, would absolutely love to see build time config to specify which packages are bundled. My only request is that there is a thoughtful default set to keep the getting started experience simple.
After Apollo Client 2, the core became so much smaller and it became easier to require only the parts you need. But that is not yet the case in this addon.
Excited to see this!

@bgentry I work with Ivan - just submitted a PR with a basic implementation of this feature. Please let me know what you think!

This issue can be close now 🎉

Closed by #109!


Use golang.org/x/sys/unix provided ioctl functions

Tries to fix #86

Please let me know if this is not the correct way to use these functions, I'm not 100% sure about them.

Thanks @maddie for the PR!
We recently had a similar issue in our macOS builder for the arduino-cli, so this PR is very welcome and comes at the right time.
At a first glance, the PR looks good and ready to be merged, BTW give me some time to do some tests on real hardware to see if the syscalls are still working as expected.


Add reactive binder for PubSub

A placeholder issue to investigate a reactor+pubsub integration - if I can I will take a stab at it. But before then let’s see if it makes sense :)

@smaldini Will this be addressed by #1461, or did you have a Cloud Stream binder in mind?


Confusing description of expected values for 'since' parameter on the direct methods

As an example, the description for since in the table of parameters for the GetModuleLogs method says (confusing part in bold):

Only return logs since this time, as a duration (1 d, 90 m, 2 days 3 hours 2 minutes), rfc3339 timestamp, or UNIX timestamp. If both tail and since are specified, the logs are retrieved using the since value first. Then, the tail value is applied to the result, and the final result is returned. OPTIONAL.

This same description appears in other places for both since and until on other direct methods.  The confusion contributed to issue https://github.com/Azure/iotedge/issues/6464.

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

ID: 3dedace3-fcb7-45b7-8bc1-11b5485d033a
Version Independent ID: 1af36bac-1b08-f493-285f-23b64f12c500
Content: Retrieve IoT Edge logs - Azure IoT Edge
Content Source: articles/iot-edge/how-to-retrieve-iot-edge-logs.md
Service: iot-edge
GitHub Login: @PatAltimore
Microsoft Alias: patricka

@micahl Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thanks, @micahl ,
I've clarified the descriptions. The change will be published soon.
Thanks,
Pat
#please-close


Terminal chat widget in task terminals

I keep seeing this show up in my build terminal after awhile, but no consistent repro steps

@meganrogge could this not be fixed yet or just not in insiders?

it's probably fixed in tomorrow's insider's

/duplicate #207367


Feature request - Filter commands by site

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.07.09. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.07.09

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Feature request - Filter commands by site
A command line option filter that runs commands based on the site being used. e.g.
--match-site youtube.com --output template
would run any commands that come after it only if the site being used is youtube.com.
Multiple --match-site commands could be used to make templates for different sites (e.g. different --output commands for youtube.com and vimeo.com, etc.).
--match-site unlisted
would apply commands to sites not specifically listed by --match-site.
Support for any url structure.
Or add the same support options to --match-filter

This is not how it's work. Explicit match site option does not make any sense since it's already filtered by input URL.

It makes plenty of sense because there's a config file. Putting custom --output templates in the config file for different sites would make downloading videos a more streamlined process than having to change my --output configuration every time I download from a different url.


Multi-fabric broken (Ethernet device, minimal mDNS, TC-MF-1.21)

Problem
Multi-Fabric tests (TC-MF-1.21) fails after adding 6..9 additional controllers.
This seems to be mDNS related. Observed during TE9 with our DUT, which currently using the minimal mDNS for development.
Reproducible on TE9 tag with the example lighting-app as well as on current master (a777a801c6f0637ee54f83a63d02f662338e8b51).
chip-tool output:
[1652870021.531285][6077:6078] CHIP:DMG: Received Command Response Data, Endpoint=0 Cluster=0x0000_003E Command=0x0000_0008
[1652870021.531311][6077:6078] CHIP:CTL: Device returned status 0 on receiving the NOC
[1652870021.531332][6077:6078] CHIP:CTL: Operational credentials provisioned on device 0x7f5bc0001960
[1652870021.531352][6077:6078] CHIP:TOO: Secure Pairing Success
[1652870021.531373][6077:6078] CHIP:CTL: Successfully finished commissioning step 'SendNOC'
[1652870021.531392][6077:6078] CHIP:CTL: Commissioning stage next step: 'SendNOC' -> 'FindOperational'
[1652870021.531411][6077:6078] CHIP:CTL: Performing next commissioning step 'FindOperational'
[1652870021.531430][6077:6078] CHIP:CSM: FindOrEstablishSession: PeerId = 20014E4F04A750D8:0000000000000001
[1652870021.531451][6077:6078] CHIP:CSM: FindOrEstablishSession: No existing OperationalDeviceProxy instance found
[1652870021.531542][6077:6078] CHIP:DIS: Operational node lookup already in progress. Will NOT start a new one.
[1652870021.531577][6077:6078] CHIP:DMG: ICR moving to [AwaitingDe]
[1652870021.531601][6077:6078] CHIP:EM: Sending Standalone Ack for MessageCounter:9615839 on exchange 59506i
[1652870021.531632][6077:6078] CHIP:IN: Prepared secure message 0x7f5bc4a99180 to 0xFFFFFFFB00000000 (0)  of type 0x10 and protocolId (0, 0) on exchange 59506i with MessageCounter:2930314.
[1652870021.531657][6077:6078] CHIP:IN: Sending encrypted msg 0x7f5bc4a99180 with MessageCounter:2930314 to 0xFFFFFFFB00000000 (0) at monotonic time:<PHONE_NUMBER>E6A9D1 msec
[1652870021.531695][6077:6078] CHIP:EM: Flushed pending ack for MessageCounter:9615839 on exchange 59506i
[1652870021.731146][6077:6078] CHIP:DIS: Checking node lookup status after 200 ms
[1652870036.531581][6077:6078] CHIP:DIS: Checking node lookup status after 15000 ms
[1652870036.531678][6077:6078] CHIP:DIS: Operational discovery failed for 0x0000000000000001: ../../examples/chip-tool/third_party/connectedhomeip/src/lib/address_resolve/AddressResolve_DefaultImpl.cpp:174: CHIP Error 0x00000032: Timeout
[1652870036.531882][6077:6078] CHIP:CTL: Device connection failed. Error ../../examples/chip-tool/third_party/connectedhomeip/src/lib/address_resolve/AddressResolve_DefaultImpl.cpp:174: CHIP Error 0x00000032: Timeout
[1652870036.531912][6077:6078] CHIP:CTL: Error on commissioning step 'FindOperational': '../../examples/chip-tool/third_party/connectedhomeip/src/lib/address_resolve/AddressResolve_DefaultImpl.cpp:174: CHIP Error 0x00000032: Timeout'
[1652870036.531927][6077:6078] CHIP:CTL: Failed to perform commissioning step 18
[1652870036.531945][6077:6078] CHIP:CTL: Going from commissioning step 'FindOperational' with lastErr = '../../examples/chip-tool/third_party/connectedhomeip/src/lib/address_resolve/AddressResolve_DefaultImpl.cpp:174: CHIP Error 0x00000032: Timeout' -> 'Cleanup'
[1652870036.531972][6077:6078] CHIP:CTL: Performing next commissioning step 'Cleanup' with completion status = '../../examples/chip-tool/third_party/connectedhomeip/src/lib/address_resolve/AddressResolve_DefaultImpl.cpp:174: CHIP Error 0x00000032: Timeout'
[1652870036.531996][6077:6078] CHIP:CTL: Successfully finished commissioning step 'Cleanup'
[1652870036.532022][6077:6078] CHIP:TOO: Device commissioning Failure: ../../examples/chip-tool/third_party/connectedhomeip/src/lib/address_resolve/AddressResolve_DefaultImpl.cpp:174: CHIP Error 0x00000032: Timeout

lighting-app does not show anything suspicious with the TE9 tag, but the following on master:
[1652869752.438360][426445:426445] CHIP:ZCL: OpCreds: successfully created fabric index 7 via AddNOC
[1652869752.438372][426445:426445] CHIP:DMG: Decreasing reference count for CommandHandler, remaining 0
[1652869752.438379][426445:426445] CHIP:EM: Piggybacking Ack for MessageCounter:6433823 on exchange: 64301r
[1652869752.438402][426445:426445] CHIP:IN: Prepared secure message 0x55ad4f9e6b30 to 0xFFFFFFFB00000000 (7)  of type 0x9 and protocolId (0, 1) on exchange 64301r with MessageCounter:3112835.
[1652869752.438410][426445:426445] CHIP:IN: Sending encrypted msg 0x55ad4f9e6b30 with MessageCounter:3112835 to 0xFFFFFFFB00000000 (7) at monotonic time:<PHONE_NUMBER>220A33 msec
[1652869752.438441][426445:426445] CHIP:DMG: ICR moving to [CommandSen]
[1652869752.438445][426445:426445] CHIP:DMG: ICR moving to [AwaitingDe]
[1652869752.438517][426445:426445] CHIP:EM: Received message of type 0x10 with protocolId (0, 0) and MessageCounter:6433824 on exchange 64300r
[1652869752.438522][426445:426445] CHIP:EM: Found matching exchange: 64300r, Delegate: (nil)
[1652869752.438528][426445:426445] CHIP:EM: Rxd Ack; Removing MessageCounter:3112834 from Retrans Table on exchange 64300r
[1652869752.438530][426445:426445] CHIP:EM: Removed CHIP MessageCounter:3112834 from RetransTable on exchange 64300r
[1652869752.438548][426445:426445] CHIP:DIS: Insufficient parsers to process all SRV entries.
[1652869752.438553][426445:426445] CHIP:DIS: Insufficient parsers to process all SRV entries.
[1652869752.438567][426445:426445] CHIP:DIS: Discovered node without a pending query
[1652869752.438572][426445:426445] CHIP:DIS: Discovery does not require any more timeouts
[1652869752.438576][426445:426445] CHIP:DIS: Discovered node without a pending query
[1652869752.438579][426445:426445] CHIP:DIS: Discovery does not require any more timeouts
[1652869752.438595][426445:426445] CHIP:DIS: Insufficient parsers to process all SRV entries.
[1652869752.438599][426445:426445] CHIP:DIS: Insufficient parsers to process all SRV entries.
[1652869752.438611][426445:426445] CHIP:DIS: Discovered node without a pending query
[1652869752.438615][426445:426445] CHIP:DIS: Discovery does not require any more timeouts
[1652869752.438619][426445:426445] CHIP:DIS: Discovered node without a pending query
[1652869752.438622][426445:426445] CHIP:DIS: Discovery does not require any more timeouts
[1652869752.438638][426445:426445] CHIP:DIS: Insufficient parsers to process all SRV entries.
[1652869752.438642][426445:426445] CHIP:DIS: Insufficient parsers to process all SRV entries.
...
[1652869752.439191][426445:426445] CHIP:EM: Received message of type 0x10 with protocolId (0, 0) and MessageCounter:6433825 on exchange 64301r
[1652869752.439197][426445:426445] CHIP:EM: Found matching exchange: 64301r, Delegate: (nil)
[1652869752.439206][426445:426445] CHIP:EM: Rxd Ack; Removing MessageCounter:3112835 from Retrans Table on exchange 64301r
[1652869752.439211][426445:426445] CHIP:EM: Removed CHIP MessageCounter:3112835 from RetransTable on exchange 64301r
[1652869752.439325][426445:426445] CHIP:DIS: Discovered node without a pending query
[1652869752.439330][426445:426445] CHIP:DIS: Discovery does not require any more timeouts
[1652869752.439333][426445:426445] CHIP:DIS: Discovered node without a pending query
[1652869752.439335][426445:426445] CHIP:DIS: Discovery does not require any more timeouts
[1652869752.439354][426445:426445] CHIP:DIS: Discovered node without a pending query
[1652869752.439357][426445:426445] CHIP:DIS: Discovery does not require any more timeouts
...
[1652869752.439486][426445:426445] CHIP:DIS: Discovery does not require any more timeouts
[1652869752.439502][426445:426445] CHIP:DIS: Insufficient parsers to process all SRV entries.
[1652869752.439518][426445:426445] CHIP:DIS: DNSSD parse error: ../../third_party/connectedhomeip/src/lib/dnssd/IncrementalResolve.cpp:316: CHIP Error 0x0000000B: No memory
[1652869752.439523][426445:426445] CHIP:DIS: DNSSD parse error: ../../third_party/connectedhomeip/src/lib/dnssd/IncrementalResolve.cpp:316: CHIP Error 0x0000000B: No memory
[1652869752.439527][426445:426445] CHIP:DIS: DNSSD parse error: ../../third_party/connectedhomeip/src/lib/dnssd/IncrementalResolve.cpp:316: CHIP Error 0x0000000B: No memory
[1652869752.439530][426445:426445] CHIP:DIS: DNSSD parse error: ../../third_party/connectedhomeip/src/lib/dnssd/IncrementalResolve.cpp:316: CHIP Error 0x0000000B: No memory
[1652869752.439533][426445:426445] CHIP:DIS: Discovered node without a pending query
[1652869752.439535][426445:426445] CHIP:DIS: Discovery does not require any more timeouts
[1652869752.439538][426445:426445] CHIP:DIS: Discovered node without a pending query
[1652869752.439541][426445:426445] CHIP:DIS: Discovery does not require any more timeouts
[1652869752.439554][426445:426445] CHIP:DIS: Insufficient parsers to process all SRV entries.

Out of memory can be ruled out.
Configured via:
gn gen out/host --args="target_os=\"linux\"
        chip_inet_config_enable_ipv4=false
        chip_enable_openthread=false
        chip_enable_ota_requestor=false
        chip_config_network_layer_ble=false
        chip_mdns=\"minimal\"
        chip_enable_ble=false
        chip_enable_wifi=false
        treat_warnings_as_errors=false"

This cannot be tested with the Avahi-based mDNS, as this seems to be completely broken at the moment, cf. #18548.

Might be related/duplicate of #18256

@sstruchtrup  could you describe testing steps here? How can I reproduce the issue and what commands do I need to run to reproduce it.
Was this tested on latest or TE9? The latest changes to minmdns that affect device discovery were merged only recently.

@andy31415 This was discovered during TE9 with the TE9 tag, but is also present on current master (commit a777a801c6f0637ee54f83a63d02f662338e8b51).

Then I configure the example lighting app (linux):
gn gen out/host --args="target_os=\"linux\"
        chip_inet_config_enable_ipv4=false
        chip_enable_openthread=false
        chip_enable_ota_requestor=false
        chip_config_network_layer_ble=false
        chip_mdns=\"minimal\"
        chip_enable_ble=false
        chip_enable_wifi=false
        treat_warnings_as_errors=false"

Initial pairing:
chip-tool pairing ethernet 1 20202021 3840 fc00:832::1 5540

Then I use the following script for the additional pairings:
#!/bin/bash

# Change this to the chip-tool you are using
chip_tool=chip-tool-te9

fail()
{
	echo "$*"
	exit 1
}

pair()
{
	n=$1

	(${chip_tool} administratorcommissioning open-basic-commissioning-window 500 1 0  --timedInteractionTimeoutMs 1000 > ocw-${n}.log) || fail "Open commissioning window failed"

	let id=n+10

	(${chip_tool} pairing onnetwork 1 20202021 --commissioner-name ${id} > comm-${n}.log)|| fail "commissioning failed"
}

for s in `seq 1 20`;
do
	echo "---> ${s}"
	pair $s || exit 1
done

Retested against today's master (45f623b73cbdc4a304579882aeb00163f28fd192) with verbose mDNS logging. This somehow fails on the 5th additional pairing.
Full logs for the lighting-app:
log.txt
Failed commissioning starts at timestamp<PHONE_NUMBER>.

I can get a pcap tomorrow if needed.

For some reason MDNS packets are received with several operational data answers. We limit parallel processing to 2, so operational SRV records fail beyond that. Not sure if there is an ordering issue or something else.

What is unclear to me is what queries are being made - I see MINMDNS reporting received packets, however I never see a log of "MINMDNS: Sending query" which I would expect to see what chip-tool is asking for (and to figure out why it gets so many replies ... because maybe we are asking for the wrong thing).

Unassigning this to give priority to cert blockers for now.

Not observed anymore on the pre-SVE commit 425f1fb916ca154deeb75cd82cd5196778aba152.


fix(error-handling): adjusted sample error handler md reference

Hi again 😅,
when trying to navigate to the sample error handler, you'll get a 404.
See link https://1dsag.github.io/UI5-Best-Practice/error-handling/sampleerrorhandler.markdown

I adjusted this accordingly.

I have read the CLA Document and I hereby sign the CLA


nixos/tests/nextcloud: Update redis configuration

This resolves the services.redis.enable has been renamed to services.redis.servers..enable warning.
Description of changes

Rename services.redis.enable to services.redis.servers.nextcloud.enable

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Will be merged in #190646.


NetworkManager[880]:   [1665551312.2723] sup-iface: failed to cancel p2p connect: P2P cancel failed

in /var/log/syslog
NetworkManager[880]: <warn>  [1665551312.2723] sup-iface: failed to cancel p2p connect: P2P cancel failed

https://forums.debian.net/viewtopic.php?t=149517
nmcli
wlp1s0: connected to 자동 ivs-office-5g
       "Qualcomm Atheros QCA6174"
       wifi (ath10k_pci), E0:0A:F6:40:07:95, hw, mtu 1500
       ip4 default
       inet4 <IP_ADDRESS>/24
       route4 <IP_ADDRESS>/0
       route4 <IP_ADDRESS>/24
       route4 <IP_ADDRESS>/16
       inet6 fe80::d5:641:2da1:1ee3/64
       route6 fe80::/64
p2p-dev-wlp1s0: disconnected
"p2p-dev-wlp1s0"
wifi-p2p, hw
lo: unmanaged
       "lo"
       loopback (unknown), 00:00:00:00:00:00, sw, mtu 65536
DNS configuration:
       servers: <IP_ADDRESS> <IP_ADDRESS>
       interface: wlp1s0

nmcli connection add type wifi-p2p ifname p2p-dev-wlo1 wifi-p2p-peer [mac address of the device you want to connect to]

link : https://unix.stackexchange.com/questions/370123/wifi-direct-between-linux-android
other : https://unix.stackexchange.com/questions/420640/unable-to-connect-to-any-wifi-with-networkmanager-due-to-error-secrets-were-req


Support custom labels for Loki pushes

#117
Signed-off-by: Frank Jogeleit<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 8 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 93.775%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
2109

Relevant Lines:
2249

💛  - Coveralls


Questions about TOOLING.md

Hey!
I was looking into what was required to make an intero plugin for the Atom editor, and was wondering if someone could expand a bit on the tooling.md?
From what I can see:

You build stack for the current project with stack build intero
You start a process running stack ghci --with-ghc intero to load the project (what is the difference between that and just running stack exec intero?)
You send input to the process on its stdin, such as :type-at Settings.hs lineStart colStart lineEnd colEnd functionName (e.g. :type-at Settings.hs 171 1 171 15 combineScripts)
You catch the output from running that and display it to the user in a neat way

Some immediate questions (I'm sure there'll be more later on):

Are the any documentation on the arguments the various functions expect?
How do you currently handle things like docker processes? An example being if the docker container has a name, then intero can't run if the container is already running

What is the difference between that and just running stack exec intero?

stack ghci sets up the load paths and targets and such.
If you run :type-at it replies with the required args:

Are the any documentation on the arguments the various functions expect?

<no location info>: Expected a span: "<module-name/filepath>" <start line> <start column> <end line> <end column> "<sample string>"

How do you currently handle things like docker processes? An example being if the docker container has a name, then intero can't run if the container is already running

stack itself handles the docker stuff. I don't know much about it.


Enable Drag Feature of Mini-map Navigator

Task
Enable drag feature of Mini-map navigator.
Demo
Update
Temporarily be able to drag the navigator, but haven't completed the bindings because the implementation of demo is very different from Texera. Need more discussion and suggestions.

Update
Able to drag the navigator.

Remaining Problems

There are some bugs on moving. The offset/position is not very accurate.
Haven't decided the limit/boundary.

Update
Modify the movement, now the navigator can moves the canvas correctly.

To-Do
Still need to set the limit of the navigator (when moving the canvas paper, the navigator may exceed the mini-map region).
Also need to improve the zoom in/out feature because the origin of the jointJS paper would change.

Update
Modify the movement, now the navigator can moves the canvas correctly.

To-Do
Still need to set the limit of the navigator (when moving the canvas paper, the navigator may exceed the mini-map region).
Also need to improve the zoom in/out feature because the origin of the jointJS paper would change.

Update
Add the limit check. Before moving the canvas paper, it would first check whether the event would make the navigator exceed the mini-map region.


rename helm imageKey to image to match starter workflows

Description
The recommended generated key names in draft don't line up with the  keys in the starter workflows, so this PR changes the imageKey key to just image so that they are all consistent
This is a breaking change for consumers of the IMAGENAME variable, as it will no longer can contain both an image name and tag.
Previously, alpine:latest was a valid IMAGENAME, but now the IMAGETAG variable (which defaults to "latest" following docker and kubernetes convention) should be used to pass a tag.
Usages of IMAGENAME that include a tag will now need to use the IMAGETAG variable for passing a tag, as passing an image name like alpine:latest will result in double-tagging such as alpine:latest:latest

[x] Breaking Change - new variable IMAGETAG and changed usage of IMAGENAME
[x] Bug fix (non-breaking change which fixes an issue)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. Is it a  breaking change which will impact consuming tool(s).

[x] existing integration tests pass

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] 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
[x] Any dependent changes have been merged and published in downstream modules

needs tag

Can we bump the version and add a breaking change comment to the release notes when cutting the release?

Can we bump the version and add a breaking change comment to the release notes when cutting the release?

yes for sure! atm the flow is to have a dedicated release PR with the changelog entry, so I'll put it there