Ubuntu >= 15 with latest docker version

When upgrading to version 0.14.6, AZK automatically installs the latest version of docker. AZK does not always works 100% with the latest docker version. I think it would be better to leave a fixed version, instead of taking the last update.
Selecting previously unselected package docker-engine.
Preparing to unpack .../docker-engine_1.8.1-0~vivid_amd64.deb ...
Unpacking docker-engine (1.8.1-0~vivid) ...

@renanmpimentel I'm sorry for this.
In azk latest version (v0.14.6), we've added a Docker API version lock, as this PR: https://github.com/azukiapp/azk/pull/499. However, it seems this won't be enough because even Docker supporting older API versions, it doesn't support the features from an older Docker version.
We're working in a new azk version (v0.15.0) in which we're going to add a Docker version lock and features to support Docker 1.7 or newer.

But the AZK will persist with the behavior of installing the latest version of docker?
Because from what I noticed, the version (v0.14.6) does not need docker installed on the machine.

@renanmpimentel actually it does. Docker has changed its default repo and package name from lxc-docker to docker-engine, and both deb and rpm packages require it.
Since in this new Docker repo only the latest Docker version is available, we have assumed the behavior you described.

Nice! :+1:


Fix the too-many-nested-blocks pylint issues

Please review the changes carefully.

Codecov Report

Merging #3469 into master will decrease coverage by <.01%.
The diff coverage is 88.63%.

@@            Coverage Diff             @@
##           master    #3469      +/-   ##
==========================================
- Coverage   71.04%   71.04%   -0.01%
==========================================
  Files         419      419
  Lines       25993    25994       +1
  Branches     3930     3927       -3
==========================================
  Hits        18468    18468
- Misses       6378     6379       +1
  Partials     1147     1147

Impacted Files
Coverage Δ

...s/azure-cli-interactive/azclishell/az_completer.py
60.09% <100%> (ø)
:arrow_up:

...cli-core/azure/cli/core/commands/_introspection.py
94.8% <100%> (ø)
:arrow_up:

...cli-vm/azure/cli/command_modules/vm/_validators.py
72.67% <62.5%> (-0.18%)
:arrow_down:

src/azure-cli-core/azure/cli/core/commands/arm.py
85.71% <87.23%> (+0.03%)
:arrow_up:

...etwork/azure/cli/command_modules/network/custom.py
62.41% <90%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Ping @tjprescott


Don't have s immediately step out of the current frame

I'm considering having s step to the return statement rather than immediately out of the function if there's no call to step into available in the current frame. Thoughts? @JeffBezanson ?

Stepping out one frame at a time seems easier to follow.

Not sure I understand which option you're advocating for. The distinction is between
julia> gcd(100,25)
16  function gcd{T<:Union{Int64,UInt64,Int128,UInt128}}(a::T, b::T)
17      a == 0 && return abs(b)
18      b == 0 && return abs(a)
19      za = trailing_zeros(a)

About to run: Base.==
1|debug > s
228
229   =={T<:Number}(x::T, y::T) = x === y
230    <{T<:Real}(x::T, y::T) = no_op_err("<" , T)
231   <={T<:Real}(x::T, y::T) = no_op_err("<=", T)

About to run: Base.===
1|debug > s
16  function gcd{T<:Union{Int64,UInt64,Int128,UInt128}}(a::T, b::T)
17      a == 0 && return abs(b)
18      b == 0 && return abs(a)
19      za = trailing_zeros(a)
20      zb = trailing_zeros(b)

About to run: (==)(25,0)

vs
julia> gcd(100,25)
16  function gcd{T<:Union{Int64,UInt64,Int128,UInt128}}(a::T, b::T)
17      a == 0 && return abs(b)
18      b == 0 && return abs(a)
19      za = trailing_zeros(a)

About to run: Base.==
1|debug > s
228
229   =={T<:Number}(x::T, y::T) = x === y
230    <{T<:Real}(x::T, y::T) = no_op_err("<" , T)
231   <={T<:Real}(x::T, y::T) = no_op_err("<=", T)

About to run: Base.===
1|debug > s
228
229   =={T<:Number}(x::T, y::T) = x === y
230    <{T<:Real}(x::T, y::T) = no_op_err("<" , T)
231   <={T<:Real}(x::T, y::T) = no_op_err("<=", T)

About to run: retrun false
1|debug > s
16  function gcd{T<:Union{Int64,UInt64,Int128,UInt128}}(a::T, b::T)
17      a == 0 && return abs(b)
18      b == 0 && return abs(a)
19      za = trailing_zeros(a)
20      zb = trailing_zeros(b)

About to run: (==)(25,0)

I agree with making this change. It's better to err on the side of skipping over less. When stepping you don't want to feel you're missing anything.

Skipping over less is what I was advocating.


Tag ="transition-group", vue3.2.33 Warning

[Vue warn]:  children must be keyed.
Extraneous non-props attributes (class) were passed to component but could not be automatically inherited because component renders fragment or text root nodes.

Same issue as https://github.com/SortableJS/vue.draggable.next/issues/138 and https://github.com/SortableJS/vue.draggable.next/issues/140

使用 vuenextdraggable.   fix it.
https://www.npmjs.com/package/vuenextdraggable
在使用 tag="transition-group" 的时候，需要至少两层 html 标签
<template #item="{ element }">

{{ element.name }}


Fix #563 - Add breadcrumbs to single method docs.

Need a review... there's some weirdness with this:

ACL methods are weird. They show Storage > Acl > acl.add
Default ACL methods are even weirder: They show Storage > acl.default.add

Also I'm not 100% happy with the code, but not sure how I could make this better. Guidance would be really helpful.....

Yeah this can be resolved by putting acl on Bucket.prototype instead of initialized in the constructor of Bucket. (I quickly fooled around with it and that seemed to resolve it, but left the code in a non-working state). @stephenplusplus can perhaps describe the reason why this was done this way. Seems lots of this scope magic prevents us from moving these methods around more freely I believe.

Another issue is http://localhost:8000/docs/#/docs/master/datastore/dataset?method=allocateIds showing DatastoreRequest as its parent. Breadcrumbs sounded nice at first, but I wouldn't mind ditching them in favor of the sidebar being the "hierarchy of truth". It's also a little odd why the breadcrumbs are only used on one page, and not all.

@ryanseys The main issue is to keep Acl generic, it needs information that we don't have until instantiation of our modules.

Yeah makes sense. I think sidebar of truth sounds okay to me. Breadcrumbs are messy when we don't use a very specific style of object hierarchy.

(edit: can't believe I gave a localhost link)

oh man :see_no_evil:
@jgeewax You ok with closing this due to the complexity of this fix given the circumstances noted above? Neat idea, but hard in practice.

:( Makes me sad, but if it's not worth the effort, so be it.
It just seems weird that we can detect, for every method, what module and class should light up in the sidebar, but we can pick up that same thing and just put in the class (Bucket, File, ...).
I'll close out this PR but leave the main issue open. Maybe there's a better way.


Failures on Test for avoiding exponential tree traversal of DAG constants...[vm/ffi] Document marking struct fields as external

There are new test failures on Test for avoiding exponential tree traversal of DAG constants...[vm/ffi] Document marking struct fields as external.
The tests
standalone_2/io/socket_close_test/0 Crash (expected Pass)

are failing on configurations
dartk-tsan-linux-release-x64

Log is here. The /0 version of this test is with --enable-isolate-groups.
/cc @mkustermann

This failure is flaky and unrelated to the CL. It happens for the multitest that opt-in via --enable-isolate-groups in JIT mode (which is still under development).

This failure is flaky and unrelated to the CL. It happens for the multitest that opt-in via --enable-isolate-groups in JIT mode (which is still under development).

This is unrelated to --enable-isolate-groups per se, it's already tracked here: https://github.com/dart-lang/sdk/issues/45641


chore: remove explicit module manifest account/region mappings

Changes

remove explicit module manifest account/region mappings from fmops-qna-rag

Checklist before requesting a review

[x] I updated CHANGELOG.MD with a description of my changes
[x] If the change was to a module, I ran the code validation script (scripts/validate.sh)
[x] If the change was to a module, I have added thorough tests
[x] If the change was to a module, I have added/updated the module's README.md
[x] If a module was added, I added a reference to the module to the repository's README.md
[x] I verified that my code deploys successfully using seedfarmer apply

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: a7b3ec73e167396fc36fbad3587d79c8bcea538e
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: 081c0375b0cf7759898a8974fe3766b0bda6e9a6
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: 333b0d5858aefc1c711a399256a422c474793874
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Adding CPU/GPU distribution to the logs and reports

While checking your result statistics on https://llm.aidatatools.com/ I always missed a notification if the model was completely loaded into a GPU or if it runs in a mixed environment.
Implementing such a check could be a low-hanging fruit, as ollama keeps the last model running after closing the request at
run_benchmark.py:
75:  result = subprocess.run([ollamabin, 'run', model_name, one_prompt['prompt'],'--verbose'], capture_output=True, text=True, check=True, encoding='utf-8')
If you add another call subprocess.run([ollamabin, 'ps'], capture_output=True, text=True, check=True, encoding='utf-8')
you can still gather the utilization.
e.g.:
`
$ ollama ps
NAME      	ID          	SIZE  	PROCESSOR	UNTIL
qwen2:1.5b	f6daf2b25194	1.8 GB	100% GPU 	4 minutes from now
$ ollama ps
NAME      	ID          	SIZE 	PROCESSOR      	UNTIL
llama3:70b	786f3184aec0	41 GB	79%/21% CPU/GPU	4 minutes from now
`
Based on the ollama documentation, it will be possible to have several models loaded at the same time. So you need to expect that in future, ollama ps will report several rows of models. Adding a filter to the model_name of the ollama ps output should be future-proof .
At the end: it would be great to see that usage/distribution on your results pages.

I suggest to add "WSL" (Windows Subsystem for Linux) category or just a tag in Linux category also. to detect whether a Linux distribution is in WSL, just check whether the output of uname -r command or platform.uname().release ends with -WSL2:
<IP_ADDRESS>-microsoft-standard-WSL2

But I'm not sure the ending for WSL1, maybe -Microsoft (found from this file)

@nuffin You have great points, would you please create a separate ticket? I don't want to mix two different issues in one ticket.

@nuffin You have great points, would you please create a separate ticket? I don't want to mix two different issues in one ticket.

sure. I'm creating it.


acme/Watch: dump first command

This makes sure closing and reopening acme relaunches at least the first command. Other commands aren't dumped.

I was also having the same issue, can confirm this patch fixes it. Thanks a lot for the patch, I really appreciate it :-)


Repo sync for protected CLA branch

The pull request is created from main637907383839253351 to main to fix git push error for protected CLA branch

@opbld15 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Config.parse() is not parsing the mcl.Config.Line structs

Trying to parse an existing config file only parses correctly for line_names.

I'm not really sure where the root of the problem is. I did not change the Config struct nor the parse function so was this always a problem?

it was a typo


Change error messages everywhere.

Error messages when command fails, or messages that show syntax

Message while using the view command on a task should be improved


Gradle on Windows failing with file system error in CI

For example here:
https://ci.chromium.org/ui/p/flutter/builders/prod/Windows_android flutter_gallery_win__compile/10419/overview
[2023-06-05 21:40:13.182166] [STDOUT] stderr: [!] Gradle threw an error while downloading artifacts from the network.
[2023-06-05 21:40:13.182166] [STDOUT] stdout: Retrying Gradle Build: #1, wait time: 100ms
[2023-06-05 21:41:10.425980] [STDOUT] stdout: Running Gradle task 'assembleRelease'...
[2023-06-05 21:41:10.425980] [STDOUT] stdout: Parameter format not correct -
[2023-06-05 21:41:18.124144] [STDOUT] stderr:
[2023-06-05 21:41:18.124144] [STDOUT] stderr: FAILURE: Build failed with an exception.
[2023-06-05 21:41:18.124144] [STDOUT] stderr:
[2023-06-05 21:41:18.124144] [STDOUT] stderr: * What went wrong:
[2023-06-05 21:41:18.124144] [STDOUT] stderr: Execution failed for task ':app:packageRelease'.
[2023-06-05 21:41:18.124144] [STDOUT] stderr: > A failure occurred while executing com.android.build.gradle.tasks.PackageAndroidArtifact$IncrementalSplitterRunnable
[2023-06-05 21:41:18.124144] [STDOUT] stderr:    > java.io.FileNotFoundException: C:\opt\s\w\ir\x\w\recipe_cleanup\tmp1w76bp6n\flutter sdk\dev\integration_tests\flutter_gallery\build\app\outputs\apk\release\app-release.apk (Operation did not complete successfully because the file contains a virus or potentially unwanted software)
[2023-06-05 21:41:18.124144] [STDOUT] stderr:

Possibly related: https://github.com/flutter/flutter/issues/123519
Marking P1 for triage since this is causing some tests to be above the flake threshold: https://github.com/flutter/flutter/issues/128394

@reidbaker do you know what may be happening inside gradle that can be causing this?

cc: @yusuf-goog
Yusuf is disabling cloud protection on the windows bots in order to try and mitigate this. The theory is that the cloud protection defaults to marking the artifact as dangerous, and then later changing it to safe after analysis.

Lowering to P2 as this is not blocking the tree anymore

@yusuf-goog Did you happen to disable cloud protection? I can also try to do it, I wasn't sure if there was a specific account needed or something.

Protections have been disabled on all physical devicelab windows hosts by @kentnguyen99

Do we believe this is resolved?

Yes, closing it.


Vscode

@nilehmann here's the pure vscode stuff ... except i've made the dbg thing emit the statement events whenever the dump_checker_trace is on (not just debug-builds).


Setup the semantic release process

Describe the bug
Setup semantic release process

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Authentication using FireBase Auth

https://firebase.google.com/docs/auth/where-to-start

was able to link FireBase auth with some of the responsibilities included in creating the user database


Skip feature detection if image is an SVG

Fixes #11172
The error described occurs as a result of trying to run feature detection on SVGs. This isn't currently possible. This PR adds a check for SVGs to the get_suggested_focal_point method, returning None in that case.
To confirm this fix, upload an SVG on a Wagtail build with SVGs enabled and WAGTAILIMAGES_FEATURE_DETECTION_ENABLED = True. The image should be uploaded without issue.
Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[x] For Python changes: Have you added tests to cover the new/fixed behaviour?

Merged in 47d048e242dfad3e7754b14a6a42ee46e451e334 (main) and c36cf20dd4ae8b30776df18a8e6b4974925838bd (stable/5.2.x)


[build-script] Clean submodules where applicable

When updating the project with the clean option, ensure that sub modules are cleaned as well. This issue can be tested by adding a line to swift-corelibs-libdispatch/libkqueue/README.md - if this file is changed then the reset hard on the libdispatch project will fail silently before this fix. As a result the libdispatch project checkout can become out of date.

Resolves SR-2524.
When building Swift from a non-clean checkout, the git reset that
occurs on subprojects that include submodules may fail if the submodule
is deemed to have been changed (for example, by running the autoconf
scripts).
When cleaning the projects, ensure that git submodule is used to clean
submodules and reset them to their original state as well, so that the
project can be reset successfully.
Issue: SR-2524

Is it possible to just not build those dependencies in-tree? The intent is that all build products and intermediates end up in build/.

The problem is that the libdispatch project includes those modules as git submodules; and if one of those files is inadvertently changed then it prevents the parent project from being "git reset" to head. So it's not actually about the binaries or whether these should be rebuilt or not, but whether it prevents the libdispatch project from being correctly updated with the right code.
Note that the problem occurs when running the "update-checkout" script and is before the build occurs.
You can see the problem in action by doing:
(cd swift-corelibs-dispatch; git reset --hard HEAD^)
echo "Prevent dispatch from being updated" >> swift-corelibs-dispatch/libkqueue/README.md
utils/update-checkout --clone --clean
The problem is that after this step the swift-corelibs-dispatch is still pointing to HEAD^ and isn't updated, because a change to the source files in the submodule results in the git reset failing (silently).

Ah, sorry. I thought this was just about building because of the "build-script" in the title.
@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test Linux platform

Thanks, Alex!


minSdkVersion of cwa

Your Question

Documentation File:
Google test app
https://github.com/google/exposure-notifications-android/blob/master/app/build.gradle
cwa (SAP/Telekom)
https://github.com/corona-warn-app/cwa-app-android/blob/master/Corona-Warn-App/build.gradle

Line / Paragraph:

Question:
In the Google test app on github the minSdkVersion is 21. This refers to Lollipop.

In your source minSdkVersion 23 (Marshmallow) is used.
The market share of Lollipop is 8.5 % and Marshmallow is 10.7%
Why don't you follow the recommendation from Google?

Since this question is very specific to the android app, we moved it from the documentation repo.

Dear @herionhonig,
thanks for reaching out! You are correct, a minimum API level of 21+ was required, however this has changed. Please have a look at the official documentation for the Exposure Notification API which can be found here: https://static.googleusercontent.com/media/www.google.com/en//covid19/exposurenotifications/pdfs/Android-Exposure-Notification-API-documentation-v1.3.2.pdf
In the revision history, the min API level was raised from 21 to 23, therefore we had to adjust our configuration as well. Hope this answers your question, please close your issue if it does.
Thanks and best,
Marc


Consolidate --show-* options

The --show-ref and --show-resources cannot easily be used together (since they have different formats), and they cannot be used in the same command as a download. They're primarily intended for debugging and automation. Find a way to support a single command to download files while still having a method to capture metadata useful for external tools.
Maybe introduce a --print-info-like option with a machine-friendly output format like below. It would be helpful to be support sending this to a non-STDOUT/STDERR descriptor to continue to maintain a nice user output:
origin	resolved<EMAIL_ADDRESS>origin	service	github
origin	server	github.com
origin	owner	dpb587
origin	repository	gget
origin	ref	v0.4.0
metadata	tag	v0.4.0
metadata	commit	d75361c1316c6004cde13c04460c57552a839d57
resource	gget-0.4.0-darwin-amd64
resource	gget-0.4.0-linux-amd64
resource	gget-0.4.0-windows-amd64.exe

Probably requires introducing a dedicated --no-download-like option to allow external tools to continue gathering info before an actual download.

I added --no-download, and went with --dump-info= since it kind of matches curl's --dump-header. The --show-ref and --show-resources flags are now hidden from the help text and I intend to remove them before a 1.x version (after which commands will error with the flags being unrecognized).
I'm still thinking if I want to change the resource prefix/format of files in case other info might need to be shown per file at some point (e.g. size).
@whalehub, I'm typically not too concerned about backwards compatibility before a 1.0, but figured I'd call it out since I think you're relying on --show-ref. I'll keep around those old options for a couple minor versions. Let me know if you have feedback on the new options or output. The below are some examples which may help you switch once they're available (note: it'll probably be a week before I get back to this and potentially cut a new release).
# detect if --dump-info is supported
$ gget --version='>=0.5' --quiet 2>/dev/null

# getting the resolved version
$ go run . github.com/dpb587/gget --no-download --dump-info=- \
  | awk '{ if ( $1 == "origin" && $2 == "ref" ) { print $3 } }'
v0.4.0

# listing resource names
$ go run . github.com/dpb587/gget --no-download --dump-info=- \
  | awk '{ if ( $1 == "resource" ) { print $2 } }'
gget-0.4.0-darwin-amd64
gget-0.4.0-linux-amd64
gget-0.4.0-windows-amd64.exe

@dpb587 I don't have any objections to the new format since all it takes is a small change in my script from
gget --show-ref "${!REPO}" 2>/dev/null | sed '2d;s|tag[[:blank:]]||g;s|v||g')"

to
gget --no-download --dump-info=- "${!REPO}" 2>/dev/null | grep metadata | sed '2d;s|.*tag[[:blank:]]||g;s|v||g')"

before everything is back to normal again:
whalehub@pdh:~# ginstall.sh -c gget
The latest version of gget is v0.4.0.

I'll wait for you to cut a new release before I publish the next version of ginstall.sh since these changes will require bumping the minimum required version of gget for my script.

@dpb587 Just tried out gget v0.5.0 and it works quite nicely. Thanks for your hard work! 🎉
VERSION_CHECK="$(gget --no-download --export plain "${!REPO}" 2>/dev/null | grep metadata | sed '1,2d;s|.*tag[[:blank:]]||g;s|v||g')"
if [ -z "${VERSION_CHECK}" ]; then
  APP_VERSION="$(gget --ref-stability=pre-release --no-download --export plain "${!REPO}" 2>/dev/null | grep metadata | sed '1,2d;s|.*tag[[:blank:]]||g;s|v||g')"
else
  APP_VERSION="${VERSION_CHECK}"
fi

whalehub@pdh:~# ginstall.sh -c gget
The latest version of gget is v0.5.0.
whalehub@pdh:~# sudo ginstall.sh gget latest
Found 1 file (11.1M) from<EMAIL_ADDRESS>√ gget-0.5.0-linux-amd64 done (sha256 OK; executable)
The latest version of gget (v0.5.0) was successfully installed to /usr/local/bin.

One other thing I changed since my last comment was the export, as you found, but it also now supports jsonpath-ish. It should be a bit easier to extract the ref (and use fewer API requests than plain in a future optimization). Something like the following should be equivalent:
for stability in stable pre-release ; do
  APP_VERSION="$(gget --ref-stability="${stability}" --export jsonpath='{.origin.ref}' --no-download "${!REPO}" 2>/dev/null | sed 's|^v||')"
  [ -z "${APP_VERSION}" ] || break
done

I'd encourage using the origin ref instead of metadata tag since that's what gget is technically referring to and prefers. Also I've learned I need to namespace those metadata keys and include some additional metadata in an upcoming version; will make an issue.

The jsonpath format allows for a much more elegant solution indeed - thanks!


periodic flushing

add a periodic flush counter. this forces any buffered data to be flushed to disk on a regular interval.
important when capturing very low bandwidth traces, and using the push features.

need to modify the current inactivity flush to be programable

documentation updated
http://fmad.io/en.fmadio10-manual.html#config-flush
Released in FW: 5186


Precomputed header bytes

Encode all the standard preamble once for all requests

705,348.96 rps -> 1,030,679.42 rps
/cc @davidfowl @halter73

That's some SRS RPS :+1:

1,069,312.35 on the 3rd wrk run (with Kestrel warmed up)

If SocketOutput used MemoryPool2 internally and there was a Write(+Async) overload that took (MemoryPoolBlock2 block, int length, ...) and which took ownership of the pooled block (and Released it at write callback); then the copy step at the start of public Task WriteAsync could be skipped for the header preamble.

New numbers from SmurfLab with this PR, up from ~497k previously:
Running 10s test @ http://<IP_ADDRESS>:5001/plaintext
  32 threads and 256 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency    10.36ms   23.70ms 316.86ms   93.90%
    Req/Sec    18.86k     3.02k   35.23k    77.74%
  5912136 requests in 10.10s, 744.25MB read
  Socket errors: connect 0, read 0, write 337, timeout 0
Requests/sec: 585370.49
Transfer/sec:     73.69MB

Try non pipelined? I'm also seeing an improvement there.

Added precomputing Http Version bytes also

Wow. With the addition of 371b2ad we break 600K:
Running 10s test @ http://<IP_ADDRESS>:5001/plaintext
  32 threads and 256 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency    11.26ms   25.12ms 265.05ms   92.61%
    Req/Sec    19.57k     3.08k   43.89k    82.55%
  6115424 requests in 10.10s, 769.84MB read
  Socket errors: connect 0, read 0, write 300, timeout 0
Requests/sec: 605502.09
Transfer/sec:     76.22MB

And non-pipelined results:
Running 10s test @ http://<IP_ADDRESS>:5001/plaintext
  32 threads and 256 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     1.52ms    1.35ms  56.63ms   96.06%
    Req/Sec     5.68k   667.38     8.41k    83.08%
  1778313 requests in 10.10s, 223.86MB read
  Socket errors: connect 0, read 0, write 552, timeout 0
Requests/sec: 176086.22
Transfer/sec:     22.17MB

Have one last additional item for Server and Date; just need to sort out tests

Latest results, getting higher:
Running 10s test @ http://<IP_ADDRESS>:5001/plaintext
  32 threads and 256 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency    10.50ms   23.15ms 230.54ms   92.07%
    Req/Sec    19.83k     2.93k   35.72k    85.37%
  6204867 requests in 10.10s, 781.10MB read
  Socket errors: connect 0, read 0, write 587, timeout 0
Requests/sec: 614363.71
Transfer/sec:     77.34MB

Added Server and Date precomputation; while still allowing swap out/clearing

Wow, Travis is really unhappy, going to give up on it for a bit; hopefully something is updating and not just broken.

Keep it coming :smile:
Running 10s test @ http://<IP_ADDRESS>:5001/plaintext
  32 threads and 256 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     8.42ms   19.76ms 293.82ms   93.96%
    Req/Sec    20.22k     3.08k   40.68k    86.43%
  6302045 requests in 10.10s, 793.33MB read
  Socket errors: connect 0, read 0, write 549, timeout 0
Requests/sec: 623966.09
Transfer/sec:     78.55MB

Also rebased

Also passed Travis first time, it must like it...

I really like this, but to go the whole way with it SocketOutput really needs to be memory-pool-block based instead of byte[] segment based... Do we have that change queued up anywhere?

Have a commit that passes the memoryblock through and the SocketOutput held onto it and freed it as well as using memoryblocks internally - however the results were indeterminate. I believe from putting a much higher pressure on memorypool.
However, the results of this are conculsive; and can work out the improved pass through later - @davidfowl gets upset when I put too many changes in a PR ;-)

@davidfowl gets upset by a lot of things :)
I'm putting together a "for instance" commit on top of your branch, but yeah SocketOutput needs to become block-aware to make this change happen all the way

@lodejard various test repositories https://github.com/benaadams/KestrelHttpServer/tree/lockless-socket-output and https://github.com/benaadams/KestrelHttpServer/tree/socket-memorypool2

Hello, here's a commit I was going to send as a PR to your fork's branch:
https://github.com/lodejard/KestrelHttpServer/commit/7a265d94a8539c70c1e023ed318666465001d2df
but you've already rebased and made changes on top of it :) so I'm just gonna send this as reference material instead of trying to replant it on top of your new changes
it removes a lot of the special-case code in the generated response headers, but also accomplishes what you are working towards for blitting well-known byte[] material wherever possible

Added another commit - fixing a silly mistake in SetRawXxx which wasn't setting the _rawXxx field as well
See: https://github.com/aspnet/KestrelHttpServer/compare/dev...lodejard:faster-headers

There are a couple bonuses there; on CreateResponseHeader you've gone async?
@davidfowl criticised me for invoking the async state machine in a different PR; which I felt was unfair as he was using my own arguments against me; to address this I had to go a third way https://github.com/aspnet/KestrelHttpServer/pull/391/files maybe something similar can be used?

To be honest I suspect the "async" is will be temporary - @halter73 is also looking into making a change to SocketOutput to make it more like SocketInput in that the buffers will be head/tail block references. The Frame will be able to call methods like ProduceStart/ProduceComplete in order to access the tail's end iterator and blit directly onto the unused space of the output buffers already present. At that point it wouldn't need to await any Tasks anymore, because it's the SocketOutput's job to return blocks after they're written.
So just like SocketInput, it can be a rolling series of packed/pinned buffers. Should help the pipeline throughput test scenario in particular because a number of 'hello world' responses can probably fit into each 4k page.

I tried to get your modifications running @lodejard but the performance is atrocious and wrk reports thousands of errors. I suspect I've done something wrong trying to fetch your changes into a local branch though, so I'll try again with a fresh clone of your fork.

OK tried a fresh clone of your fork @lodejard and switched to the faster-headers branch and I still see the same result, so less likely it's my fault :smile:

No guarantees, :), more of a for-instance commit

@lodejard @halter73 sounds really interesting! Looking forward to it

@lodejard should I copy the changes in?

@lodejard am merging

@DamianEdwards should be good to test

Needs tweaks

May behave better on  #362

Latest results. Note that these are down a littler from the earlier results, likely due to the change to be multi-loop by default now:
Running 10s test @ http://<IP_ADDRESS>:5001/plaintext
  32 threads and 256 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency    11.73ms   24.73ms 267.14ms   91.27%
    Req/Sec    19.01k     1.73k   31.57k    79.63%
  6094081 requests in 10.10s, 767.15MB read
Requests/sec: 603365.93
Transfer/sec:     75.95MB

Rebased on #410 doing integration now
/cc @halter73

I just filed https://github.com/aspnet/Common/issues/64 we might want to consider.
It's related to this PR in that it would open a channel from app to server for pre-computed ascii byte[] response headers to be written.

There's some duplication here with the CopyFrom logic. This looks much cleaner with @halter73 's changes

Seem to be spitting out the right length data but in lots of zero bytes; not sure I've integrated right.

5 tests to fix

Yay!

Rebased on SocketOuput changes; fingers crossed for performance

Looks like it might be a good one, al lot of the allocations drop out
Before

After

Bug in the copy (hence the System.Exception.ToString having high allocs)

Adding #411, #408 the allocations change to the following

RPS isn't great; but CPU is low, so I think there is high lock contention - making some minor adjusts.

Reduced the in lock work; but also turned out I had receive side scaling turned off. When I turn that on... Oh my...
  32 threads and 256 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency    11.67ms   34.29ms 657.95ms   90.26%
    Req/Sec    40.62k     4.81k   76.06k    77.62%
  77224953 requests in 1.00m, 9.49GB read
Requests/sec: 1284937.56
Transfer/sec:    161.75MB

Nice. I'm merging this in now. I will plan to make a few more changes, like changing the signature of ProducingComplete to not take a count, but that can be done in another PR.

:checkered_flag: time to rebase all the things...


 You are still assigned, so I was not sure if more was needed. The URL change seems OK.

@javitonino yep, you requested changes in the last review

Seems to work fine. Waiting for tests to run, and then :rocket:


DOMException: Failed to execute 'transaction' on 'IDBDatabase': The database connection is closing.

I get this exception occasionally for the amp.dev serviceworker. I'm not 100% sure how to reproduce this. Loading the page -> clearing cache in DevTools -> reloading a few times sometimes triggers it.
amp-sw.js:1 Uncaught (in promise) DOMException: Failed to execute 'transaction' on 'IDBDatabase': The database connection is closing.
    at https://cdn.ampproject.org/sw/amp-sw.js:1:8225
    at new Promise (<anonymous>)
    at r.transaction (https://cdn.ampproject.org/sw/amp-sw.js:1:8188)
    at async r._call (https://cdn.ampproject.org/sw/amp-sw.js:1:8591)
    at async r.put (https://cdn.ampproject.org/sw/amp-sw.js:1:7118)
    at async i.setTimestamp (https://cdn.ampproject.org/sw/amp-sw.js:1:9212)
    at async y.updateTimestamp (https://cdn.ampproject.org/sw/amp-sw.js:1:11953)
    at async i.cacheDidUpdate (https://cdn.ampproject.org/sw/amp-sw.js:1:14642)
    at async Object.put (https://cdn.ampproject.org/sw/amp-sw.js:1:17694)

I've experienced this serveral times (months ago) when testing Workbox in the PWA plugin for WordPress. I think it's a browser issue, where after clearing browser data it sometimes takes IndexDB to become available again. That's just what I assume is going on.


Synchronously send error response on message decoding failure

Replaces two logs since those don't result in response messages anymore
Adds a func for sending messages synchronously, to get the two messages out before fatalError

@swift-ci please test

@swift-ci please test

@swift-ci please test

Done. Thanks for the learning opportunity 😊

@swift-ci please test

16:01:54 Test Case '-[LanguageServerProtocolJSONRPCTests.CodingTests testMessageDecodingError]' started.
16:01:54
/Users/buildnode/jenkins/workspace/swift-sourcekit-lsp-PR-macOS/branch-main/sourcekit-lsp/Tests/LanguageServerProtocolJSONRPCTests/CodingTests.swift:161: error: -[LanguageServerProtocolJSONRPCTests.CodingTests testMessageDecodingError] : failed - expected error not seen
16:01:54 Test Case '-[LanguageServerProtocolJSONRPCTests.CodingTests testMessageDecodingError]' failed (1.289 seconds).

Oh, I guess this is now a decodable error response message:
{"jsonrpc":"2.0","error":{"code":-32000,"message":""}}

So decoding that produces no error in CodingTests.swift:161. Should I remove the test or adjust it? I'm not sure what kind of LSP violation the test is testing for ...

Sounds good to me

@swift-ci please test

done. this time, regenerating the Linux tests had no effect, just so you know 🤷🏼♂


Meraki AP Device tracker

Description:
Track devices with Meraki AP. More info here
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#4044
Example entry for configuration.yaml (if applicable):
device_tracker:
  - platform: meraki
    secret: secret
    validator: validator

Checklist:
If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[X] New dependencies have been added to the REQUIREMENTS variable (example).
[X] New dependencies are only imported inside functions that use them (example).
[X] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[X] New files were added to .coveragerc.

Maybe cleanup error messages before PR?
2017-12-01 16:55:59 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
File "/usr/lib/python3.6/site-packages/aiohttp/web_protocol.py", line 416, in start
resp = yield from self._request_handler(request)
File "/usr/lib/python3.6/site-packages/aiohttp/web.py", line 323, in _handle
resp = yield from handler(request)
File "/usr/lib/python3.6/site-packages/homeassistant/components/http/ban.py", line 58, in ban_middleware
return (yield from handler(request))
File "/usr/lib/python3.6/asyncio/coroutines.py", line 213, in coro
res = yield from res
File "/usr/lib/python3.6/asyncio/coroutines.py", line 213, in coro
res = yield from res
File "/usr/lib/python3.6/site-packages/homeassistant/components/http/init.py", line 430, in handle
result = yield from result
File "/config/custom_components/device_tracker/meraki.py", line 93, in post
yield from self._handle(request.app['hass'], data)
File "/config/custom_components/device_tracker/meraki.py", line 101, in _handle
accuracy = int(float(i["location"]["unc"]))
ValueError: cannot convert float NaN to integer

Can you give me request body. I use it for more than 2 months and haven't received this error

After i set logging to debug error never appered but then i remembered i cleared all my known hosts. And probably thats why its not appearing... i will keep my eye open. Sorry bout that.
Setting track_new_devices: false   does nothing for me... I am guessing having that should only scan and update the hosts already in known_hosts?

I haven't updated the documentation. Sorry about that, little busy these days. I misunderstood the option track new devices, so it's removed. May be should be added as feature for all platforms.
No. I don't use discord

Yes, without such feature home assistant gets cluttered with so many entities that are irrelevant (passersby) 5 hours have me 200 entities when really I just want to keep track of my own.

If possible add me on gmail-chat.

2017-12-03 17:44:51 DEBUG (MainThread) [homeassistant.components.websocket_api] WS<PHONE_NUMBER>: Sending {'id': 2, 'type': 'event', 'event': {'event_type': 'state_changed', 'data': {'entity_id': 'device_tracker.0055da5137b3', 'old_state': <state device_tracker.0055da5137b3=home; source_type=router, latitude=x.xxxx, longitude=x.xxxx, gps_accuracy=26, manufacturer=IEEE Registration..., ipv4=/<IP_ADDRESS>, seenTime=2017-12-03T16:44:25Z, ssid=Lovein - Home, os=Generic Linux, friendly_name=0055da5137b3, custom_ui_state_card=state-card-custom-ui @ 2017-12-03T05:55:03.151612+01:00>, 'new_state': <state device_tracker.0055da5137b3=home; source_type=router, latitude=59.1404482, longitude=x.xxxx, gps_accuracy=0, manufacturer=IEEE Registration..., ipv4=/<IP_ADDRESS>, seenTime=2017-12-03T16:44:25Z, ssid=Lovein - Home, os=Generic Linux, friendly_name=0055da5137b3, custom_ui_state_card=state-card-custom-ui @ 2017-12-03T05:55:03.151612+01:00>}, 'origin': 'LOCAL', 'time_fired': datetime.datetime(2017, 12, 3, 16, 44, 51, 247281, tzinfo=)}}
2017-12-03 17:44:51 INFO (MainThread) [homeassistant.components.http] Serving /api/meraki to <IP_ADDRESS> (auth: True)
2017-12-03 17:44:51 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
File "/usr/lib/python3.6/site-packages/aiohttp/web_protocol.py", line 416, in start
resp = yield from self._request_handler(request)
File "/usr/lib/python3.6/site-packages/aiohttp/web.py", line 323, in _handle
resp = yield from handler(request)
File "/usr/lib/python3.6/site-packages/homeassistant/components/http/ban.py", line 58, in ban_middleware
return (yield from handler(request))
File "/usr/lib/python3.6/asyncio/coroutines.py", line 213, in coro
res = yield from res
File "/usr/lib/python3.6/asyncio/coroutines.py", line 213, in coro
res = yield from res
File "/usr/lib/python3.6/site-packages/homeassistant/components/http/init.py", line 430, in handle
result = yield from result
File "/config/custom_components/device_tracker/meraki.py", line 93, in post
yield from self._handle(request.app['hass'], data)
File "/config/custom_components/device_tracker/meraki.py", line 101, in _handle
accuracy = int(float(i["location"]["unc"]))
ValueError: cannot convert float NaN to integer
2017-12-03 17:44:53 INFO (SyncWorker_8) [homeassistant.components.device_tracker.nmap_tracker] Scanning...

How much coverage percentage is there with these tests? If it's above ~ 95 % you should remove the module from .coveragerc.

Please also add tests for the cases at lines 87 and 91:
https://coveralls.io/builds/14503965/source?filename=homeassistant%2Fcomponents%2Fdevice_tracker%2Fmeraki.py

Great!

Wait, there's an unrelated commit in this PR history that should not be there:
https://github.com/home-assistant/home-assistant/pull/10749/commits/1d3b7c396d60869e5ac2b7847c801840696be210
Please rebase the changes that are relevant in this branch on top of latest dev.
Then we can merge.

I merged remote dev in this branch and that's why this commit is in it.

That commit doesn't exist in upstream dev. Your commit in your PR to update hbmqtt was squashed and merged as:
https://github.com/home-assistant/home-assistant/commit/bd5a16d70b3eb432af5bc46cdbb9c9d8bb07ea14
You should start from a new branch based on newly synced dev when you start your work for a PR.
Can you rebase from https://github.com/home-assistant/home-assistant/pull/10749/commits/1a264da164756252a123c238b7ddb48510d0d178 to HEAD, onto fresh synced dev?

Closing this PR. New one is here https://github.com/home-assistant/home-assistant/pull/10971


observable decorator not working with decorators function

Here is a reproduction: https://gist.run/?id=db11dd4fd9e4dd1908fc88778a60aa5d.

@EisenbergEffect is this supposed to work?

@jdanyow We should probably make it work in order to support pure ES6. The decorator implementation will need to be updated to account for this scenario.

Is this being worked on? I would like to develop without a transpiler.

PRs are welcome.


gitignore: ignore coverage.txt files

I was running make ci and noticed my working git directory picked up a bunch of unstaged coverage.txt files.
I don't think there's any value in tracking them with git if they aren't checked into the repo, so this PR adds *coverage.txt to the .gitignore
Signed-off-by: Riyaz Faizullabhoy<EMAIL_ADDRESS>

LGTM


Fix compression path for attachBlob ops

Description
The recently-merged batch compression PR caused an ICM incident because of an incorrect assumption in the batch continuation path. The continuation path checked for metadata === undefined rather than metdata.batch === undefined and ops with metadata weren't correctly detected as a batch continuation.
I've also added an end to end test that would've caught this.

@justus-camp, @andre4i , do we need to port it to main? Ideally, we do all this development (in non-breaking way) in main

@vladsud I've got a draft PR open for porting to main (and have cherry-picked this commit on top of it). I've got some questions regarding what's breaking vs. what's not (and what changes need to be made if there are some breaking changes). Do you have time to take a look?


Option to disable coloured output

Clear and concise description of the problem
When running tests on CI (in our case Bamboo) the default output is in colour which causes some ugly output when viewing the raw logs:

Suggested solution
Peraps disable colours when CI=true or add a new CLI option to do so.
Alternative
No response
Additional context
We're running tests for CI like this:
CI=true vitest run --coverage

Validations

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

Oh perfect, thanks @Demivan !

If this works you can close the issue


Add chart v1.5.0

ChangeLogs:

Add chart for release v1.5.0.
Also includes #84, #88 changes.

Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist:

[ ] I have bumped the chart version according to versioning
[ ] I have updated the chart changelog with all the changes that come with this pull request according to changelog.
[ ] Any new values are backwards compatible and/or have sensible default.
[ ] I have signed off all my commits as required by DCO.

Changes are automatically published when merged to master. They are not published on branches.

This version should be stable. PTAL /cc @hormes @FillZpp

/lgtm
/approve


Support for the webview_flutter_platform_interface

I'm not familiar with the interface of webview_flutter_platform_interface and not of the internals of this widget, but I think it would be cool if that interface could be implemented in order to use webview_flutter on windows using this implementation.
How are the chances that this can be done?

@jnschulze are there any plans to add this as an extension to the default flutter webview>


Integration fails to load

Describe the bug
Seeing the following errors in the HA logs and the integration fails to load.
I am able to telnet to this port without issues.
Expected behavior
Modbus Solaredge integration loads correctly.
Inverter is working, I've had this error a number of times, but most recently updated the integration version maybe a week or so back but didn't have time to verify correct operation.
Logs
Logger: homeassistant.setup
Source: setup.py:192
First occurred: 9:02:18 AM (1 occurrences)
Last logged: 9:02:18 AM
Setup failed for custom integration solaredge_modbus_multi: Unable to import component: cannot import name 'ModbusTcpClient' from 'pymodbus.client' (/usr/local/lib/python3.10/site-packages/pymodbus/client/init.py)
2022-11-13 09:02:18.321 ERROR (MainThread) [homeassistant.setup] Setup failed for custom integration solaredge_modbus_multi: Unable to import component: cannot import name 'ModbusTcpClient' from 'pymodbus.client' (/usr/local/lib/python3.10/site-packages/pymodbus/client/init.py)
2022-11-13 10:39:10.189 ERROR (SyncWorker_7) [pymodbus.client.sync] Connection to (<IP_ADDRESS>, 502) failed: timed out
2022-11-13 10:47:38.306 ERROR (SyncWorker_1) [pymodbus.client.sync] Connection to (<IP_ADDRESS>, 502) failed: timed out
2022-11-13 10:50:46.937 ERROR (SyncWorker_3) [pymodbus.client.sync] Connection to (<IP_ADDRESS>, 502) failed: timed out
2022-11-13 10:51:07.959 ERROR (SyncWorker_1) [pymodbus.client.sync] Connection to (<IP_ADDRESS>, 502) failed: timed out
Debug Logs
Debug logs are not always necessary, but may be required for more complex issues. You can enable debug logs by adding the following to your configuration.yaml file and restarting Home Assistant.
logger:
    default: warning
    logs:
        custom_components.solaredge_modbus_multi: debug
        #pymodbus: debug #only if needed

Debug logging will generate a large amount of data: it is recommended to configure it, collect the data needed, then remove debug logging during normal operation.
Home Assistant (please complete the following information):

Home Assistant Core Version:
solaredge-modbus-multi Version:

Additional context
Add any other context about the problem here.

2022-11-13 09:02:18.321 ERROR (MainThread) [homeassistant.setup] Setup failed for custom integration solaredge_modbus_multi: Unable to import component: cannot import name 'ModbusTcpClient' from 'pymodbus.client' (/usr/local/lib/python3.10/site-packages/pymodbus/client/init.py)

This means that your Home Assistant installation does not have a compatible version of pymodbus loaded. How are you running Home Assistant?

2022-11-13 09:02:18.321 ERROR (MainThread) [homeassistant.setup] Setup failed for custom integration solaredge_modbus_multi: Unable to import component: cannot import name 'ModbusTcpClient' from 'pymodbus.client' (/usr/local/lib/python3.10/site-packages/pymodbus/client/init.py)

This means that your Home Assistant installation does not have a compatible version of pymodbus loaded. How are you running Home Assistant?

After a restart yesterday, the Solaredge integration started working but I received this error assumably being the inbuilt modbus?
Setup failed for modbus: Unable to import component: No module named 'pymodbus.client.sync'
November 13, 2022 at 11:11:44 AM – (ERROR) setup.py
This isn't a new integration, it's been previously running for ages.
I'm not sure what you mean by how am I running HA, I'm running HA OS on a Pi4.

Sorry, I read it wrong. Yes the "setup failed" is the core modbus integration which can't be loaded at the same time the integration is loaded. See issue #162 for more info. Basically if you are using the core modbus integration you have to stay on v2.2.5 until Home Assistant gets updated to work with newer versions of pymodbus.

Is there an easy way to rill back the version?

Remove 2.2.6, reboot, and reinstall 2.2.5 is what I suggest.
https://github.com/WillCodeForCats/solaredge-modbus-multi/issues/162#issuecomment-1304011425


Unable to POST more than 70 nodes to service discovery endpoint.

Do you already have an issue opened with F5 support?
Github Issues are consistently monitored by F5 staff, but should be considered as best effort only and you should not expect to receive the same level of response as provided by F5 Support. Please open an case with F5 if this is a critical issue.
ISSUE TYPE

Bug Report

AS3 BUILD/ VERSION
v3.14.0-4
BIGIP VERSION
v13.1
SUMMARY
After configuring service discovery EP, unable to POST more than 70 endpoints to service discovery endpoint in same go.
STEPS TO REPRODUCE

Configured Service discovery for BigIP
Using POSTMAN we tried to post data in following format with the url below :

Service discovery url : https://<IP_ADDRESS>/mgmt/shared/service-discovery/task/~Part_11~Virt_server~web_pool/nodes
Method : POST
Data : ( per node ) - the scaled up post had 70+ nodes in below format.
[
{

    "ip": "<IP_ADDRESS>",

    "id": "<IP_ADDRESS>"

}

]
Observations :
75 nodes ( and above): 400 BAD REQUEST: 1m 28s

It was observed that the BigIP went unreachable for a few minutes and looks like the services restarted.

Error response from bigIP :
{
"code": 400,

"message": "remoteSender:<IP_ADDRESS>, method:POST ",

"originalRequestBody": "[\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\": \"<IP_ADDRESS>\",\n        \"id\": \"<IP_ADDRESS>\"\n    },\n    {\n        \"ip\"",

"referer": "<IP_ADDRESS>",

"restOperationId": 1470018,

"errorStack": [

    "java.util.concurrent.TimeoutException: remoteSender:<IP_ADDRESS>, method:POST ",

    "at com.f5.rest.common.RestWorker.logAndFailExpiredOperation(RestWorker.java:3099)",

    "at com.f5.rest.common.RestWorker.checkForExpiredOperations(RestWorker.java:3088)",

    "at com.f5.rest.common.RestServer.checkAndExpirePendingWorkerOperations(RestServer.java:1457)",

    "at com.f5.rest.common.RestServer.access$400(RestServer.java:48)",

    "at com.f5.rest.common.RestServer$4.run(RestServer.java:1430)",

    "at com.f5.rest.common.ScheduleTaskManager$2$1.run(ScheduleTaskManager.java:123)",

    "at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:473)",

    "at java.util.concurrent.FutureTask.run(FutureTask.java:262)",

    "at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)",

    "at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)",

    "at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1152)",

    "at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:622)",

    "at java.lang.Thread.run(Thread.java:748)",

    "Suppressed: com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Expected BEGIN_OBJECT but was BEGIN_ARRAY at line 1 column 2 path $",

    "\tat com.google.gson.internal.bind.ReflectiveTypeAdapterFactory$Adapter.read(Unknown Source)",

    "\tat com.google.gson.Gson.fromJson(Unknown Source)",

    "\tat com.google.gson.Gson.fromJson(Unknown Source)",

    "\tat com.google.gson.Gson.fromJson(Unknown Source)",

    "\tat com.f5.rest.common.RestOperation.getBody(RestOperation.java:1586)",

    "\tat com.f5.rest.common.RestOperation.setErrorResponseBody(RestOperation.java:2544)",

    "\tat com.f5.rest.common.RestOperation.fail(RestOperation.java:2485)",

    "\tat com.f5.rest.common.RestWorker.logAndFailDrainedOperation(RestWorker.java:1318)",

    "\t... 13 more",

    "Caused by: java.lang.IllegalStateException: Expected BEGIN_OBJECT but was BEGIN_ARRAY at line 1 column 2 path $",

    "\tat com.google.gson.stream.JsonReader.beginObject(Unknown Source)",

    "\t... 21 more\n"

],

"kind": ":resterrorresponse"

}
Logs observed in restnoded.log :
Tue, 10 Sep 2019 09:29:06 GMT - finest: [LoaderWorker] '/var/config/rest/iapps/f5-service-discovery/nodejs/lib/util.js' is not a worker, skipping
Tue, 10 Sep 2019 09:29:07 GMT - config: [RestWorker] /shared/service-discovery/task has started. Name:RestWorkerTask
Tue, 10 Sep 2019 09:29:07 GMT - info: [f5-service-discovery: Task.js] Task ~Part_9~iqeo~ZX4o4oGCS2BOuss4iSIndzIPQAb7h1T2dkLfvjg3D failed: The var "nodes" must be instantiated and be an array for comparison
Tue, 10 Sep 2019 09:29:14 GMT - warning: [appsvcs] {"message":"Unable to register device for telemetry: Error: getaddrinfo EAI_AGAIN product.apis.f5.com:443\n    at Object.exports._errnoException (util.js:907:11)\n    at errnoException (dns.js:32:15)\n    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:78:26)","level":"warning"}
Tue, 10 Sep 2019 09:29:14 GMT - severe: [appsvcs] {"message":"Error: Command failed: ls /var/config/rest/iapps/f5-appsvcs/encryptedDeclaration\nls: cannot access /var/config/rest/iapps/f5-appsvcs/encryptedDeclaration: No such file or directory\n","code":"2","level":"error"}
Tue, 10 Sep 2019 09:29:14 GMT - config: [RestWorker] /shared/appsvcs has started. Name:RestWorker
Tue, 10 Sep 2019 09:29:14 GMT - info: [f5-service-discovery: Task.js] Task ~Part_10~7f4tRBFgFs7bLLQqR45Br6wnXUXUb4rBBsXB9WHKsLI3D failed: The var "nodes" must be instantiated and be an array for comparison
Tue, 10 Sep 2019 09:29:17 GMT - info: [f5-service-discovery: Task.js] Task ~Part_10~N8uhvRBWFLRIQOyHRzzyeon~7dLuRDFqy2BdP8~kqMEc3D failed: The var "nodes" must be instantiated and be an array for comparison
Tue, 10 Sep 2019 09:29:20 GMT - info: [f5-service-discovery: Task.js] Task ~Part_10~tujdLZIvPRE8fMVto4xP9THbgO2BkFWshKS8CDZPRPD43D failed: The var "nodes" must be instantiated and be an array for comparison
Tue, 10 Sep 2019 09:29:24 GMT - info: [f5-service-discovery: Task.js] Task ~Part_11~WXU5SyPaTDUeHplEOULUYgitgNEa9LEZ6Lo2~74amQg3D failed: The var "nodes" must be instantiated and be an array for comparison

Thank you for reporting this issue. I have added it to our internal product backlog as AUTOTOOL-969.

Found during testing integration with F5 ACI ServiceCenter

I believe this issue should be fixed in AS3 3.24.

@kedargslab Does the latest release resolve this issue for you?

Closing for inactivity. If you are still encountering issue, please reopen this ticket for a file a new one.


Send email verification to an existing account

We are trying to implement email verification for our users. I've enabled sendVerificationEmail: true and it works fine for new creating account. However, existing users that have signed up before get a Please verify your email first. Check the email and follow the link! Is there a way for me to send the verification link to those?

Lets have a look at [Accounts.sendVerificationEmail](https://docs.meteor.com/#/full/accounts_sendverificationemail
It should be enough to cycle over your users at server startup and send out a new verification email for each one...
You can also check whether an email address is already verified by looking at the user.emails field which is an array of objects like this:
user: {
  emails: [
    {address<EMAIL_ADDRESS>verified: false},
    {address<EMAIL_ADDRESS>verified: true},
  ]
}

@splendido worked fine, thanks :)


style: auto-fix formatting issues with Rustfmt, make CI pass

Disables configuring import_layouts, this is an unstable feature and only available on the nightly channel, but this library is specifically for the stable version of Rust.

LGTM !


FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory

`<--- Last few GCs --->
3129122 ms: Mark-sweep 1348.8 (1404.3) -> 1350.1 (1405.3) MB, 996.2 / 0.0 ms [allocation failure] [GC in old space requested].
3130130 ms: Mark-sweep 1350.1 (1405.3) -> 1351.4 (1404.3) MB, 1007.1 / 0.0 ms [allocation failure] [GC in old space requested].
3131090 ms: Mark-sweep 1351.4 (1404.3) -> 1354.3 (1404.3) MB, 960.6 / 0.0 ms [last resort gc].
3132096 ms: Mark-sweep 1354.3 (1404.3) -> 1357.8 (1404.3) MB, 1005.4 / 0.0 ms [last resort gc].
<--- JS stacktrace --->
==== JS stack trace =========================================
Security context: 0x11ed9e6cfb51
1: parseModifiers [/home/mario/WebstormProjects/proyectsMad/laplaya/node_modules/typescript/lib/typescript.js:~12756] [pc=0x7e919e18b3](this=0x11ed9e6e60e1 ,permitInvalidConstAsModifier=0x11ed9e604381 <undefined)
2: arguments adaptor frame: 0->1
3: parseTypeMember [/home/mario/WebstormProjects/proyectsMad/laplaya/node_modules/typescript/lib/typescript.js:~10277] ...
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
1: node::Abort() [gulp]
2: 0x10a3b3c [gulp]
3: v8::Utils::ReportApiFailure(char const_, char const_) [gulp]
4: v8::internal::V8::FatalProcessOutOfMemory(char const_, bool) [gulp]
5: v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationSpace) [gulp]
6: v8::internal::Runtime_AllocateInTargetSpace(int, v8::internal::Object__, v8::internal::Isolate_) [gulp]
7: 0x7e8f9079a7
Aborted (core dumped)`

node --max_old_space_size=4096 ./node_modules/webpack/bin/webpack.js

<--- Last few GCs --->
6457436 ms: Scavenge 1398.5 (1456.3) -> 1398.5 (1456.3) MB, 13.8 / 0 ms (+ 2.1 ms in 1 steps since last GC) [allocation failure] [incremental marking delaying mark-sweep].
6458470 ms: Mark-sweep 1398.5 (1456.3) -> 1396.7 (1456.3) MB, 1034.6 / 0 ms (+ 3.3 ms in 2 steps since start of marking, biggest step 2.1 ms) [last resort gc].
6459491 ms: Mark-sweep 1396.7 (1456.3) -> 1396.4 (1456.3) MB, 1020.0 / 0 ms [last resort gc].
<--- JS stacktrace --->
==== JS stack trace =========================================
Security context: 0x2da8187e3ac1
1: addDeclarationToSymbol [/Users/mariomalagon/WebstormProjects/laplaya/node_modules/typescript/lib/typescript.js:~14623] [pc=0xe728a219b73] (this=0x2da8187fd081 ,symbol=0x27ba73c40ef9 <a SymbolObject with map 0x2b8769c3f041>,node=0x702566bd961 <a NodeObject with map 0x2b8769c3efe9>,symbolFlags=512)
2: declareSymbol [/Users/mariomalagon/WebstormProjects/laplaya/node_mo...
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory
Abort trap: 6

The problem is the bundle is too heavy for node to handle with their ram, For that reason now. It is splitting into three chunks easier to handle.
https://github.com/madgoods/madseed/commit/a6155cd21df2de866866def815f2600b68255bda


How can I resume a failed download?

Hi @tungwaiyip. We prefer to keep our github issues for code changes and discussion and general questions about using the app should go to our support team (mobile-support@edx.org).


Would raw kv interface support atomic operations like CAS(compare and set)?

Feature Request
Is your feature request related to a problem? Please describe:
Nope
Describe the feature you'd like:
I'd like to use the raw kv interface for better performance, but certain operations require compare-and-set and other atomic support.
Describe alternatives you've considered:
I might consider some other distributed kv storage.
Teachability, Documentation, Adoption, Migration Strategy:
e.g. I have a pair SomeK -> SomeV, when updating SomeK concurrently, by first read SomeV, then modify and update SomeV, I need CAS to ensure the correctness of the program.

Sorry, might be a duplicate of https://github.com/tikv/tikv/issues/3015
But I emphasize more on raw kv interface because for txn kv, I can use transaction.

Duplicated issue. We've been internally discussing the feature, and you can add comments to https://github.com/tikv/tikv/issues/3015.


Is it possible to add the ability to upload files?

I need a file upload function for mobile phone and computer to share files in the LAN. Do you have upload function

感觉翻译的不对劲

I just want local file sharing, or can you recommend a good file sharing system, one deployed on a local LAN

File uploading is out of the scope of DirectoryLister and will not be added (see: #263, #231).
Maybe look into something like Syncthing for file syncing or NextCloud if you want a managed web interface to upload/download files.


[Bug] showSubtechniques in SVGConfig does not show the sub techniques in the SVG

Expected Behavior
Expanded sub techniques in SVG
Actual Behavior
SVG is not showing sub techniques, while the attack navigator does
Steps to Reproduce the Problem

create a json with:
layout: "expandedSubtechniques": "all"
techniques "showSubtechniques": True
"selectSubtechniquesWithParent": True
Validate that the json contains:
layout: "expandedSubtechniques": "all"
techniques: "showSubtechniques": true
"selectSubtechniquesWithParent": true
Validate that the json works in attack explorer
Import: from mitreattack.navlayers.exporters.to_svg import ToSvg, SVGConfig
svgconf =SVGConfig(width=118.9, height=84.1, unit="cm", showHeader=False, fontSize=12, showSubtechniques="all")
export with
exporter = ToSvg(domain='enterprise', source='local', resource=resource_path, config=svgconf)
exporter.to_svg(layerInit=layer, filepath="Coverage.svg")

Possible Solution
Review > mitreattack.navlayers.exporters.to_svg

Hi @KustoKing,
Thanks for opening a request! You are right. There is a bug in the code used to expand the subtechniques in the SVG. We do plan to address this issue. As a workaround, I suggest you follow the steps below -

Open the layer in ATT&CK navigator
Select expand sub-techniques in the navigator toolbar
Download the layer
Then you can create the SVG using the new layer file and the mitreattack-python library

Alternatively, if you want to strictly make use of the mitreattack-python library, ensure that ALL techniques have been included in the layer file with showSubtechniques: true.
Let us know if you encounter any issues.


Generated properties & DateTime vs. current public docs

According to this: http://ef.readthedocs.org/en/latest/modeling/generated-properties.html#data-annotations
We're supposed to be able to use Created or Modified DateTime -fields with entities. With rc1-final and base class like:
public abstract class EntityBase
{
    public long Id { get; set; }

    [DatabaseGenerated(DatabaseGeneratedOption.Computed)]
    public DateTime Created { get; set; }

    public long CreatedBy { get; set; }

    [DatabaseGenerated(DatabaseGeneratedOption.Computed)]
    public DateTime LastModified { get; set; }

    public long LastModifiedBy { get; set; }

}
And derived class like:
public class Customer : EntityBase
{
public string Name { get; set; }
public string Address { get; set; }
public string Code { get; set; }
public List Companies { get; set; }
}
I got the following error while saving changes:
info: Microsoft.Data.Entity.Storage.Internal.RelationalCommandBuilderFactory[1]
Executed DbCommand (4ms) [Parameters=[], CommandType='Text', CommandTimeout='30']
SELECT [c].[Id], [c].[Address], [c].[Code], [c].[Created], [c].[CreatedBy], [c].[LastModified], [c].[LastModifiedBy], [c].[Name]
FROM [Customer] AS [c]
info: Microsoft.Data.Entity.Storage.Internal.RelationalCommandBuilderFactory[1]
Executed DbCommand (2ms) [Parameters=[@p0='?', @p1='?', @p2='?', @p3='?', @p4='?'], CommandType='Text', CommandTimeout='30']
SET NOCOUNT OFF;
INSERT INTO [Customer] ([Address], [Code], [CreatedBy], [LastModifiedBy], [Name])
OUTPUT INSERTED.[Id], INSERTED.[Created], INSERTED.[LastModified]
VALUES (@p0, @p1, @p2, @p3, @p4);
fail: Microsoft.Data.Entity.DbContext[1]
An exception occurred in the database while saving changes.
Microsoft.Data.Entity.DbUpdateException: An error occurred while updating the entries. See the inner exception for details. ---> System.Data.SqlClient.SqlException: Cannot insert the value NULL into column 'Created', table 'fina-vnext-dev.dbo.Customer'; column does not allow nulls. INSERT fails.
Obviously this doesn't work for DateTime -fields as they are not getting inserted in the first place and the columns won't support nulls. This kind of functionality would be greatly appreciated since handling modified & created stamps for magnitude of entities isn't actually business logic. What's the future of DatabaseGenerated attribute and/or fluent api? Is this kind of functionality going to make to RTM? Or if not, is there any plans for hooks or events which one could attach common operations related to entity lifecycle?

You could use Shadow Properties for this: http://ef.readthedocs.org/en/latest/modeling/shadow-properties.html

Shadow properties would do the trick, but I'd fancy keeping this kind of information on the base class and sort of forgetting it there until it's needed for trouble shooting etc. If DatabaseGenerated properties would function as described in the article. Some kind of official statement would be nice to know - that is - will the generated properties support documented features or not.

@mkontula for DateTime properties the SQL Server doesn't automatically setup value generation.. by calling ValueGeneratedOnAdd etc. you are just letting EF know that a value will be generated, but you still need to setup the actual value generation. I have a change out for review at the moment to clarify this in the docs you referred to https://github.com/aspnet/EntityFramework.Docs/pull/88.
You could either do this in the database (with default values and triggers) or client side by overriding SaveChanges.

@rowanmiller Thanks a million! This : https://github.com/aspnet/EntityFramework/issues/3955 pretty much solved my case. Override of SaveChanges serves the purpose perfectly.


Licensing implications of UserType = Guest

I understand that if the UserType is Guest, then the External Identity pricing applies.
If not, then the normal Azure AD pricing applies.
Can this be explicitly confirmed in the documentation please?

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

ID: c6eb6697-4ace-adb6-c922-df8a9a485283
Version Independent ID: fd4ce34d-d55e-92af-d71a-625210ad3a63
Content: Properties of a B2B guest user - Azure Active Directory
Content Source: articles/active-directory/external-identities/user-properties.md
Service: active-directory
Sub-service: b2b
GitHub Login: @msmimart
Microsoft Alias: mimart

@si-chan
Thanks for your feedback! We will investigate and update as appropriate.

@si-chan
I have created a PR to update the documentation and will now close out this issue. Please allow some time for the changes to reflect and if you don't see the document update, please let me know.
I will be more than happy to re-open this issue and look into what happened.
Thank you for your time and patience!


History of Advisor's or Principal's Position Assignment(s) is Not Available

When a new advisor/principal needs to be assigned to a position, the common response is "They [the incoming person] is taking so-and-so's position."  The problem is, if the outgoing advisor/principal is no longer assigned to the position, then the previous position can't be found unless each vacant position in the organization is clicked on and the "Previous position holders" is reviewed.
This problem is more prevalent when the outgoing advisor leaves.  Their profile is set to inactive.  Then the incoming advisor arrives but the previous person's position is unknown.
Just like the position page has the "Previous position holders" section, could the person page have a history of the positions assigned to?  This would also be useful for tracking principals.  Because the principals names are so similar (and even the same), it would be easier to distinguish the principals by their work history.

Related to the sticky layout work @maradragan has been doing. We need to dig a bit more how to do that well in IE11, we will then address this issue


V2 Move varlink home

move cmd/podman/varlink/* to pkg/varlink to support podmanV2 refactor
update Makefile
reformatted all impacted code

This does NOT include a port to the latest varlink release with Context I/O support.
Signed-off-by: Jhon Honce<EMAIL_ADDRESS>

LGTM
I like this change...

One question, otherwise LGTM once the tests are happy

/lgtm


Node 18, pkg updates, more relaxed parsing

Probably requires a major version update

Yes, the version in package.json needs to be updated.
And also the node version in the work flow files.

@max-leuthaeuser done

Thanks. I think we need to tag this commit manually and trigger a release with this version once it's in.

@max-leuthaeuser When can you merge and release this?

Sorry, totally forgot about it. Done now.


onPlayerInteractEvent should not be cancelling event during sneaking

https://github.com/McJty/RFTools/blob/1.11/src/main/java/mcjty/rftools/ForgeEventHandlers.java#L102-L120
As discussed on IRC, cancelling the PlayerInteractEvent.RightClickBlock to avoid item activation is violating the event contract. The proper way to prevent a player from using the item in hand is to simply use the following code in your event handler
event.setItemResult(Result.DENY)
Javadocs explain it here
https://github.com/MinecraftForge/MinecraftForge/blob/1.11.x/src/main/java/net/minecraftforge/event/entity/player/PlayerInteractEvent.java#L189
Once this fix is added, SpongeForge should work properly with RFTools latest builds.

Implemented next version


Close pop-up

Description
Needs a feature to close the pop-up after an account is selected.

Seems like this is a Firefox specific issue, should be fixed in 1.8.6.


node version too new using v8.11.3

Maybe the workshop can stick to the LTS release which is not often changed?
LTS version has both freshness and reliability.
The official also recommend it for most of users.
So the ./exercises/am_i_ready/vars.json can have the node options like:
"node": {
      "min": "8.0.0", // keep the major release
      "max": "8.11.3" // the newest LTS version
},

fixed by https://github.com/workshopper/goingnative/commit/8e9a0284b2e582b86248a8c45ae22aa41efbb5fc


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=648141510
https://github.com/Azure/azure-cli-extensions/commit/136b57a07a5f2bd8500c60b9d18a8d53bffc7ded

Docs Build status updates of commit cb1267a:
:clock10: Full build: the average full build time is 7 min(s) 9 sec(s), this is based on the last 415 build(s) for this repository.

Docs Build status updates of commit cb1267a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

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

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


Make the regex less permissive

@rickharrison my understanding is that some would like to see it rewritten to use an actual syntax aware parser instead of regexes, but that it hasn't yet been a high enough priority for anyone to put in the leg work.

@rwjblue thoughts?

This fixes #20

@rwjblue I've updated this for v3. Though ultimately I'd love to see this back ported to v2 or have broccoli-asset-rev updated to work with v3 also.

@rwjblue I've updated this for v3. Though ultimately I'd love to see this back ported to v2 or have broccoli-asset-rev updated to work with v3 also.


resample versus decimate methods

I have a question about the resample method for mne.source_estimate.SourceEstimate. why do we have resample here and how is it different from the decimate method that we have the Epochs? Is it generally safe to resample the SourceEstimate if it is doing internal filtering?

cc: @reddigari

Decimate doesn't do any filtering, just subselects samples. It's appropriate if you know your data are already low-passed, and doesn't introduce edge artifacts.
Resampling, on the other hand, does apply a low-pass filter of its own before sub-selecting samples. This therefore can introduce edge artifacts.
Make sense? If so, please open a PR to the FAQ page about the differences between decimation in mne-python, and resampling if you have time. If you don't please re-title this issue to be about adding such a section.

thanks @Eric89GXL. next question would be, should we have the decimate method added to the SourceEstimate as well?

It's not as safe because for the SourceEstimate objects we have no knowledge of lowpass, whereas for Epochs we do in info['lowpass'].

But I think you can get what you want by using stc.bin

ah, I didn't realize that this wasn't in its info. actually, I just discovered that it has no info.

@teonbrooks you updating the FAQ section? If not, I can tackle it -- there are a few things I'd like to add.

I can't at the moment. please feel free to jump in, thanks :)


do not overwrite server.yaml

Hi,
could you please enhance rpm install script to not overwrite /etc/ntfy/server.yaml configuration. Instead of it create server.rpmnew in the usual way?
Thanks.

I think noreplace can be used or specify them according docs...
      # These config files will not be replaced by new versions if they were
      # changed by the user. Corresponds to %config(noreplace).
      config_noreplace_files:
        path/to/local/bar.con: /etc/bar.conf

I can check it if you provide a test rpm for x86_64.

Pretty sure that https://github.com/binwiederhier/ntfy/commit/53e08988e7821996686bcc12e3d7b70f2745a05a will do it, though can provide a test rpm for you shortly (probably after dinner).

nvm, here it is: https://phil.nopaste.net/2w23NWuEDS?d=1&f=ntfy_v1.17.1-next_linux_amd64.rpm&a=yy7uex5F6b

Works fine now. Thanks!


need help continue button

The "need help" continue button needs fixed immediately. Right now, it does not link anywhere. this is an old bug that was never fixed.

The continue button on Missouri Tenant Help Feedback Form doesn't work for either "I'd like to tell you something" or "I'm looking for more help"  I could not replicate this on the lemma legal server (I copied in the question from GitHubFeedbackForm to avoid the config debug.


ci: lint presubmit scans entire repo (not filtered for the PR)

The lint presubmit job seems to be checking all code sample directories even if they're not part of the PR.
I recently saw a failure on a Bigtable PR due to a GAE lint issue.

More context: It looks like this changed three days ago starting with build https://fusion2.corp.google.com/ci/kokoro/prod:cloud-devrel%2Fjava%2Fjava-docs-samples%2Flint%2Fpresubmit/activity/b527e351-8134-4f77-981a-1eed92efe69f/log. This coincides with an update with an update to the test container https://pantheon.corp.google.com/gcr/images/cloud-devrel-kokoro-resources/global/java11, but that seems unrelated.

This problem should be fixed now, see lint runs in https://github.com/GoogleCloudPlatform/java-docs-samples/pull/7890


feat(module): expose some internal classes

coverage: 67.541% (+0.03%) from 67.516%
when pulling ab1f99dade757375a57d555a73d0c8bc2d8b02d4 on carlosmiei:fix-async-client-import
into cc28d8e7b0f1a03b6c2c0f83d9a0de030439e6df on sammchardy:master.


added behaviour to allow for copying of wordpress config file to avoid issues with local changes on deploy.

Team Approved


0.1.9 version number not incremented in script or tracked in history

The version number in tomdoc.sh still reads 0.1.8.
Additionally, we're missing history for 0.1.9 in HISTORY.md!
Just bringing this to the maintainer's attention, because I'm testing out a new feature and would like to have correct versioning as I work on it.
Thanks!

Also I just noticed that for the latest 0.1.9 tag, the missing the "v" prefix is breaking the git describe version retrieval in the Makefile.

I added notes to HISTORY, tagged v0.1.9 and v0.1.10 in the repository, and finally deleted tag 0.1.9. I believe this should solve all of the issues.
Thank you very much for reporting these problems to me!


[ROS2] "OSError: [WinError 193] %1" When Running "ros2 run rqt_console rqt_console"

Describe the bug
I am learning about ROS2 on Windows 10 today,
and when I run ros2 run rqt_console rqt_console,
an error appeared as:

Traceback (most recent call last):
File "E:\dev\ros2-eloquent\ros2-windows\Scripts\ros2-script.py", line 11, in
load_entry_point('ros2cli==0.8.7', 'console_scripts', 'ros2')()
File "E:\dev\ros2-eloquent\ros2-windows\Lib\site-packages\ros2cli\cli.py", line 69, in main
rc = extension.main(parser=parser, args=args)
File "E:\dev\ros2-eloquent\ros2-windows\Lib\site-packages\ros2run\command\run.py", line 72, in main
return run_executable(path=path, argv=args.argv, prefix=prefix)
File "E:\dev\ros2-eloquent\ros2-windows\Lib\site-packages\ros2run\api_init_.py", line 61, in run_executable
process = subprocess.Popen(cmd)
File "c:\python37\lib\subprocess.py", line 800, in init
restore_signals, start_new_session)
File "c:\python37\lib\subprocess.py", line 1207, in _execute_child
startupinfo)
OSError: [WinError 193] %1 不是有效的 Win32 应用程序。

To Reproduce
Steps to reproduce the behavior:

Start a terminal and source the ROS2
Run ros2 run rqt_console rqt_console

Then the error appeared.
Expected behavior
After running ros2 run rqt_console rqt_console, rqt_console program start normally.
Environment information:

OS version: winver-- 1909
Chocolatey Versions: choco list --local-only

Chocolatey v0.10.15
asio 1.12.1
chocolatey 0.10.15
chocolatey-core.extension <IP_ADDRESS>
chocolatey-windowsupdate.extension 1.0.4
cmake 3.17.1
cmake.install 3.17.1
cunit 2.1.3
eigen 3.3.4
KB2919355 1.0.20160915
KB2919442 1.0.20160915
KB2999226 1.0.20181019
KB3033929 1.0.5
KB3035131 1.0.3
log4cxx 0.10.0
python 3.7.5
python3 3.7.5
tinyxml-usestl 2.6.2
tinyxml2 6.0.0
vcredist140 14.25.28508.3
vcredist2013 12.0.40660.20180427
20 packages installed.

If you install eloquent via the Microsoft chocolatey package (https://ms-iot.github.io/ROSOnWindows/GettingStarted/SetupRos2.html), rqt will start.
We have identified a new issue with rqt due to a libssl update. We'll be fixing it, but in the mean time you can do set path=C:\opt\python37amd64\DLLs\;%path% which allows rqt to load the libssl from the python path.


This fixes issue #34. Using the wrong hostname.

When registering as a slave we are using the wrong hostname. It should
be our hostname, not the hostname name of the server we are connecting
to.
Fixed issue with using the wrong variable.

In https://github.com/siddontang/go-mysql/pull/35#issuecomment-173796573 @siddontang said:

Hi @gdey , I guess if we don't register slave, we can still sync binlog from mysql master.
Maybe we can remove register slave later?

I'm not sure what you mean? I know I need register slave for what I'm doing. I do want to register as a slave; and receive the events. (Not just look at the binlog files.) However, what I discovered was that the name in the show slaves tables was wrong. It was showing the hostname of the master instead of the hostname of the slave. This is because we are sending the wrong hostname. This pull request fixes that. I not very good at naming things. And there were two ways I could have gone with this pull request. I took the more complicated route.

Just grab the hostname from the os; using os.Hostname. In the writeRegisterSlave routine.
Setup a variable that people can set if they want something different, and if the variable is the zero value (i.e. "") then use os.Hostname.

I went with option two just because it offers a bit more flexibility without too much added complexity. But willing to change it.

Hi @gdey
What I mean is that we can sync mysql binlog too even we don't send register slave command.
So I am not sure whether to support this or not.  :-)

Well, the current implementation of register slave is wrong according to the docs. As it should be sending the slave's hostname, and not the hostname of the server it's connecting to. (or empty if we don't want to register as a slave).
For me; I'm connecting as a slave, and so it's useful to have it register properly.
from https://dev.mysql.com/doc/internals/en/com-register-slave.html:

COM_REGISTER_SLAVE:
register a slave at the master
Payload
1              [15] COM_REGISTER_SLAVE
4              server-id
1              slaves hostname length
string[$len]   slaves hostname
1              slaves user len
string[$len]   slaves user
1              slaves password len
string[$len]   slaves password
2              slaves mysql-port
4              replication rank
4              master-id
Note that the value of [15] is hexadecimal; in decimal, this would be 21.
Fields
server_id (4) -- the slaves server-id
slave_hostname (string.var_len) -- see --report-host, usually empty
slave_user (string.var_len) -- see --report-user, usually empty
slave_password (string.var_len) -- see --report-password, usually empty
slave_port (2) -- see --report-port, usually empty
replication_rank (4) -- ignored
master_id (4) -- usually 0. Appears as "master id" in SHOW SLAVE HOSTS on the master. Unknown what else it impacts.
Returns
OK_Packet or ERR_Packet

https://dev.mysql.com/doc/refman/5.7/en/replication-options-slave.html#option_mysqld_report-host

LGTM

请问一下，LocalHost可以不设置吗？默认的也不设置

@lwfofgit
您应该能够使用NewBinlogSyncer.SetHostName将其设置为所需的内容。 如果未设置，它将返回“ os.Hostname”返回的内容。
我用google translation来翻译，所以不清楚。

@lwfofgit 您应该能够使用NewBinlogSyncer.SetHostName将其设置为所需的内容。 如果未设置，它将返回“ os.Hostname”返回的内容。
我用google translation来翻译，所以不清楚。

BTW there is no function SetHostName. So how can I set LocalHostname?

Seems it have been moved into a config item
https://github.com/go-mysql-org/go-mysql/blob/c4a17aa5b49fdadb524de0d33a5024ae32b0e019/replication/binlogsyncer.go#L46

@lwfofgit 您应该能够使用NewBinlogSyncer.SetHostName将其设置为所需的内容。 如果未设置，它将返回“ os.Hostname”返回的内容。
我用google translation来翻译，所以不清楚。

BTW there is no function SetHostName. So how can I set LocalHostname?

Seems it have been moved into a public config item
https://github.com/go-mysql-org/go-mysql/blob/c4a17aa5b49fdadb524de0d33a5024ae32b0e019/replication/binlogsyncer.go#L46

I use this config implicitly when creating syncer in Canal, here: https://github.com/go-mysql-org/go-mysql/blob/master/canal/canal.go#L435
Can we add this option to canal config and init it here https://github.com/go-mysql-org/go-mysql/blob/master/canal/canal.go#L452?
Smth like:
Localhost: c.cfg.Localhost ?

Hi @lance6716 , i opened related pull request https://github.com/go-mysql-org/go-mysql/pull/777


Gantt and sequence diagram do not render

Thank your for making this awesome library ! I'm working on a google docs integration. I've encountered some difficulties along the way so i'll report them in separate issues.
I cannot render gantt charts and sequence diagrams, while the same code works for flow charts. Here's a test case reproducing the problem :  https://codepen.io/renanlecaro/pen/VJZZge
Am i pulling a version of mermaid that does not have those capabilities ? What am i doing wrong ?
Cheers

Turns out using mermaid.init instead of mermaid.render solved this, so i guess we can close it

Great, good luck!


Messed up & missing recorded media

Checklist:

[*] I updated to the latest version available
[*] I cleared the cache of my browser

Last working release (if known):
5.2.0
Browser and Operating System:
iOS HA App & HASS.Agent via Edge on Windows 10
Description of problem:

Frigate card diagnostic information
type: custom:frigate-card
cameras:
  - camera_entity: camera.gatecam1
    live_provider: go2rtc
    go2rtc:
      modes:
        - webrtc
      stream: GateCam1Lo
    webrtc_card:
      entity: camera.scrypt_gategam1_lo
    frigate:
      url: http://<IP_ADDRESS>:5000
      camera_name: GateCam1Lo
    triggers:
      occupancy: true
menu:
  style: overlay
  position: bottom
  buttons:
    substreams:
      enabled: false
    snapshots:
      enabled: false
    recordings:
      enabled: false
    download:
      enabled: false
    camera_ui:
      enabled: false
    fullscreen:
      enabled: false
    cameras:
      enabled: false
    frigate:
      enabled: false
    clips:
      enabled: true
    mute:
      enabled: true
    microphone:
      alignment: opposing
      enabled: true
      type: momentary
  alignment: left
view:
  default: live
  dark_mode: auto
  camera_select: live
media_gallery:
  controls:
    filter:
      mode: left
live:
  microphone:
    always_connected: false
  preload: true
  auto_mute: all
  auto_unmute: selected
  controls:
    title:
      mode: popup-top-left
    thumbnails:
      mode: none
      show_favorite_control: false
      show_timeline_control: false
      show_details: false
      show_download_control: false
    builtin: false
timeline:
  style: stack
  media: all

It would help if you could describe the issue -- i.e. what does "messed up" mean? Do the recordings play? Do  regular clips play?
Just to rule in/out whether the issue is the card: Do the recordings work okay in the HA Media browser? ("Media" tab in the Home Assistant sidebar).

To be clear the message in the card's media gallery is "No media to display". This is the same for clips, snapshots & recordings.  It's the same across all browsers & apps however the HA media browser loads and plays fine.
I should also mention that when I use the configuration below the issue resolves but leaves me struggling with smooth playback (local & remote) and two-way audio

type: custom:frigate-card
cameras:
  - camera_entity: camera.gatecam1
menu:
  style: overlay
  position: bottom
  buttons:
    substreams:
      enabled: false
    snapshots:
      enabled: false
    recordings:
      enabled: false
    download:
      enabled: false
    camera_ui:
      enabled: false
    fullscreen:
      enabled: false
    cameras:
      enabled: false
    frigate:
      enabled: false
    clips:
      enabled: true
    mute:
      enabled: true
    microphone:
      alignment: opposing
      enabled: true
      type: momentary
      icon: mdi:phone
  alignment: left
view:
  default: live
  dark_mode: auto
  camera_select: live
media_gallery:
  controls:
    filter:
      mode: left
live:
  microphone:
    always_connected: false
  preload: true
  auto_mute: all
  auto_unmute: selected
  controls:
    title:
      mode: popup-top-left
    thumbnails:
      mode: none
      show_favorite_control: false
      show_timeline_control: false
      show_details: false
      show_download_control: false
    builtin: false
timeline:
  style: stack
  media: all

I also forgot to mention: camera.gatecam1 is a frigate entity.

Here is the diff between the version that works and doesn't:
--- /tmp/broken	2023-11-01 21:41:08
+++ /tmp/works	2023-11-01 21:41:31
@@ -1,18 +1,6 @@
 type: custom:frigate-card
 cameras:
   - camera_entity: camera.gatecam1
-    live_provider: go2rtc
-    go2rtc:
-      modes:
-        - webrtc
-      stream: GateCam1Lo
-    webrtc_card:
-      entity: camera.scrypt_gategam1_lo
-    frigate:
-      url: http://<IP_ADDRESS>:5000
-      camera_name: GateCam1Lo
-    triggers:
-      occupancy: true
 menu:
   style: overlay
   position: bottom
@@ -41,6 +29,7 @@
       alignment: opposing
       enabled: true
       type: momentary
+      icon: mdi:phone
   alignment: left
 view:
   default: live

Since it works without the camera_name variable, my guess is that is not actually a Frigate camera name. When you don't specify it, the camera_name is automatically detected from the Frigate entity. To test, simply take your working config and remove the camera_name.
PS: Since you are using the go2rtc provider, you do not need the webrtc_card section.

Thanks.  This has worked a treat.


Test CNI plugin with rkt in CI

This should fairly simple to implement and much quicker to test then Kubernetes.
I've got provisioning scripts in weaveworks-guides/weave-net-docker-and-rkt, which can be adapted.

We do have https://github.com/weaveworks/weave/blob/master/test/830_cni_plugin_test.sh which tests at the pure CNI level.
I guess it would be nice to test with rkt as a parallel to Docker.


py improvements

[ ] -X like lua for external script
[ ] typed variants of the row
[ ] row class should cast
[ ] plural separator for lists?
[ ] row should be able to index with tuple like in casanova
[ ] import some stuff by default

https://pyo3.rs/v0.18.1/building_and_distribution.html#embedding-python-in-rust

error: The `auto-initialize` feature is enabled, but your python installation only supports embedding the Python interpreter statically. If you are attempting to run tests, or a binary which is okay to link dynamically, install a Python distribution which ships with the Python shared library.

Embedding the Python interpreter statically does not yet have first-class support in PyO3. If you are sure you intend to do this, disable the `auto-initialize` feature.


[Mobile] There is no way to exit Add Bank Account page when the pop-up is loading

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
PRECONDTION: Network connect may need to be slow, so app takes long time to connect to Plaid

Go to Payment setting > Add Payment Method
Select Bank Account

Expected Result:
While the content is loading users should be able to exit Add Bank Account page/ popup (whether by adding the X button or minimize the popup.)
Actual Result:
There is no way to exit Add Bank Account page when the page/pop-up is loading
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[x] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.33-4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/788c3cda-cd02-4677-a555-b087fc905798
Expensify/Expensify Issue URL:
Issue reported by: @tranvantoan-qn
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687623883584849
View all open jobs on GitHub

There is a small delay with Plaid loading (even in good connection) but I don't know if this is worth fixing now as I think it would be a feature request - asking the team - https://expensify.slack.com/archives/C01SKUP7QR0/p1687906374015349

Talking this over with the team - this is a feature request.
@tranvantoan-qn if you feel passionate about this update, feel free to repost it as a feature request following the steps in either of these GHs.

https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md#raising-jobs-and-bugs
https://github.com/Expensify/App/blob/main/contributingGuides/HOW_TO_CREATE_A_PLAN.md