Windows: Fix windows to match linux after compile break

Signed-off-by: John Howard<EMAIL_ADDRESS>@swernli
This PR is part of the proposal described in docker/docker issue 10662 to port the docker daemon to Windows.
@lk4d4 This PR updates server_windows to match the change made to server_linux in #13559 which wasn't applied to the Windows variant causing Windows daemon compilation to break

@icecrime @jfrazelle @tianon This is why we need a Windows daemon compilation verification in the build! Sooner, rather than later ;)

Please PR for 1.7. Thanks :)

LGTM

ping @jfrazelle @vieux to check if this should be in 1.7 (see https://github.com/docker/docker/pull/13609#issuecomment-106991861)

when can we start compiling windows I can set up ci for that, can we add to hack/make.sh as a target

@jfrazelle Yup, I already have an unsubmitted PR waiting in the wings which will do exactly that on make cross - it will do an interim verification check that Windows builds with the daemon tag, but will still only make the client only binary at the end of it. (You can see the code change here: https://github.com/docker/docker/compare/master...Microsoft:10662-hackcross). However, the daemon won't compile on Windows quite yet as there are other PRs waiting to be merged still. Once those are in, I'm hoping we can get the compilation check PR merged ASAP to avoid future breaks.

Ah cool super excited! You guys move super fast!! Also it's Saturday hahaha
you should stop working ;)
On Saturday, May 30, 2015, John Howard (Microsoft<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle Yup, I already have an
unsubmitted PR waiting in the wings which will do exactly that on make
cross - it will do an interim verification check that Windows builds with
the daemon tag, but will still only make the client only binary at the end
of it. (You can see the code change here:
master...Microsoft:10662-hackcross
https://github.com/docker/docker/compare/master...Microsoft:10662-hackcross).
However, the daemon won't compile on Windows quite yet as there are other
PRs waiting to be merged still. Once those are in, I'm hoping we can get
the compilation check PR merged ASAP to avoid future breaks.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/13609#issuecomment-107073329.

Ha, the weekend is the only time I can catch up with you guys - you move way too fast! ;)

Jess, the easy solution would be to modify the existing windows worker
build script to compile both in series -- I can take a look at doing that
later today if you'd like. :smile:

LGTM. Not sure why the requirement for 1.7.0 though?

i dont think we need in 1.7.0
On Mon, Jun 1, 2015 at 5:04 PM, Arnaud Porterie<EMAIL_ADDRESS>wrote:

LGTM. Not sure why the requirement for 1.7.0 though?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/13609#issuecomment-107750001.

Reason is I'm trying everything I can do to get the daemon to compile on Windows and match any Linux changes which break it. I think there's still 2, maybe 3 PRs outstanding before compilation will succeeds Reaching status quo and a reliable compiling Windows daemon would be a huge milestone for 1.7 IMO

No question that there's value in a Windows compiling daemon, but what does having it in 1.7.0 adds to having it in master?


Powertoys run crashing randomly when trying to search

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
WinGet
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
I happens very frequently, but I have no clue how to reproduce it.
All that I do is use my hotkey (alt + space) and after seemingly allowing me to type the first letter it gives me this message:

It should be noted that I can just close the message box and use my hotkey again without any issues
✔ Expected Behavior
to be able to search with powertoys run
❌ Actual Behavior
a crash window
Other Software
No response

Having similar issues as well, left a comment on #24214 as there are a few of us having the same or similar issues.

This should be fixed with .69. Please upgrade.


[Bulk] Validation: fixed missing descriptions II 5/30

This PR fixes 29 missing descriptions.

Docs Build status updates of commit 0672c05:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav/Sizing-a-Control-Add-in-on-a-Page.md
:bulb:Suggestion

Details

dynamics-nav/SETVALUE-Function--TestPage-Field-.md
:white_check_mark:Succeeded

dynamics-nav/SETVIEW-Function--Record-.md
:white_check_mark:Succeeded

dynamics-nav/SETVIEW-Function--RecordRef-.md
:white_check_mark:Succeeded

dynamics-nav/SETVIEW-Function.md
:white_check_mark:Succeeded

dynamics-nav/SID-Function--Database-.md
:white_check_mark:Succeeded

dynamics-nav/SIFT-and-Performance.md
:white_check_mark:Succeeded

dynamics-nav/SIFT-and-SQL-Server.md
:white_check_mark:Succeeded

dynamics-nav/SKIP-Function--Report--XMLport-.md
:white_check_mark:Succeeded

dynamics-nav/SKIPSYSTEMTRIGGERS-Function--Debugger-.md
:white_check_mark:Succeeded

dynamics-nav/SLEEP-Function.md
:white_check_mark:Succeeded

dynamics-nav/Setting-.NET-Framework-Types-to-Target-the-Microsoft-Dynamics-NAV-Windows-Client-or-Server.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Database-Owner-and-Security-Administration-Permissions.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Colored-Indicators-on-Cues-by-Using-Codeunit-1.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Colored-Indicators-on-Cues.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Cues.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Dynamics-CRM-Integration.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-the-Home-Button-and-Home-Items.md
:white_check_mark:Succeeded

dynamics-nav/Setting-up-Hyperlinks-in-Word-Report-Layouts.md
:white_check_mark:Succeeded

dynamics-nav/Setting-up-Office-Add-Ins-Outlook-Inbox.md
:white_check_mark:Succeeded

dynamics-nav/SetupComplete2.cmd-File-Example.md
:white_check_mark:Succeeded

dynamics-nav/ShortCutKey-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowAsTree-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowCaption-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowFilter-Property.md
:white_check_mark:Succeeded

This comment lists only the first 25 files in the pull request.

dynamics-nav/Sizing-a-Control-Add-in-on-a-Page.md

Line 16, Column 73: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/search/index?dataSource=previousVersions&search=System.Drawing.Size' will be broken in isolated environments. Replace with a relative link.
Line 19, Column 109: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/search/index?dataSource=previousVersions&search=Microsoft.Dynamics.Framework.UI.Extensibility.WinForms.WinFormsControlAddInBase.ApplySize(Microsoft.Dynamics.Framework.UI.Extensibility.WinForms.DisplaySize,Microsoft.Dynamics.Framework.UI.Extensibility.WinForms.DisplaySize)' will be broken in isolated environments. Replace with a relative link.
Line 25, Column 35: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/search/index?dataSource=previousVersions&search=Microsoft.Dynamics.Framework.UI.Extensibility.WinForms.DisplaySize' will be broken in isolated environments. Replace with a relative link.
Line 59, Column 18: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/search/index?dataSource=previousVersions&search=Microsoft.Dynamics.Framework.UI.Extensibility.WinForms.WinFormsControlAddInBase.ApplySize(Microsoft.Dynamics.Framework.UI.Extensibility.WinForms.DisplaySize,Microsoft.Dynamics.Framework.UI.Extensibility.WinForms.DisplaySize)' will be broken in isolated environments. Replace with a relative link.
Line 59, Column 479: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/search/index?dataSource=previousVersions&search=Microsoft.Dynamics.Framework.UI.Extensibility.WinForms.WinFormsControlAddInBase.CreateControl' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 387961e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav/SETVALUE-Function--TestPage-Field-.md
:white_check_mark:Succeeded

dynamics-nav/SETVIEW-Function--Record-.md
:white_check_mark:Succeeded

dynamics-nav/SETVIEW-Function--RecordRef-.md
:white_check_mark:Succeeded

dynamics-nav/SETVIEW-Function.md
:white_check_mark:Succeeded

dynamics-nav/SID-Function--Database-.md
:white_check_mark:Succeeded

dynamics-nav/SIFT-and-Performance.md
:white_check_mark:Succeeded

dynamics-nav/SIFT-and-SQL-Server.md
:white_check_mark:Succeeded

dynamics-nav/SKIP-Function--Report--XMLport-.md
:white_check_mark:Succeeded

dynamics-nav/SKIPSYSTEMTRIGGERS-Function--Debugger-.md
:white_check_mark:Succeeded

dynamics-nav/SLEEP-Function.md
:white_check_mark:Succeeded

dynamics-nav/Setting-.NET-Framework-Types-to-Target-the-Microsoft-Dynamics-NAV-Windows-Client-or-Server.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Database-Owner-and-Security-Administration-Permissions.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Colored-Indicators-on-Cues-by-Using-Codeunit-1.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Colored-Indicators-on-Cues.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Cues.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Dynamics-CRM-Integration.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-the-Home-Button-and-Home-Items.md
:white_check_mark:Succeeded

dynamics-nav/Setting-up-Hyperlinks-in-Word-Report-Layouts.md
:white_check_mark:Succeeded

dynamics-nav/Setting-up-Office-Add-Ins-Outlook-Inbox.md
:white_check_mark:Succeeded

dynamics-nav/SetupComplete2.cmd-File-Example.md
:white_check_mark:Succeeded

dynamics-nav/ShortCutKey-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowAsTree-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowCaption-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowFilter-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowMandatory-Property.md
:white_check_mark:Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 86fb3ae:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav/SETVALUE-Function--TestPage-Field-.md
:white_check_mark:Succeeded

dynamics-nav/SETVIEW-Function--Record-.md
:white_check_mark:Succeeded

dynamics-nav/SETVIEW-Function--RecordRef-.md
:white_check_mark:Succeeded

dynamics-nav/SETVIEW-Function.md
:white_check_mark:Succeeded

dynamics-nav/SID-Function--Database-.md
:white_check_mark:Succeeded

dynamics-nav/SIFT-and-Performance.md
:white_check_mark:Succeeded

dynamics-nav/SIFT-and-SQL-Server.md
:white_check_mark:Succeeded

dynamics-nav/SKIP-Function--Report--XMLport-.md
:white_check_mark:Succeeded

dynamics-nav/SKIPSYSTEMTRIGGERS-Function--Debugger-.md
:white_check_mark:Succeeded

dynamics-nav/SLEEP-Function.md
:white_check_mark:Succeeded

dynamics-nav/Setting-.NET-Framework-Types-to-Target-the-Microsoft-Dynamics-NAV-Windows-Client-or-Server.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Database-Owner-and-Security-Administration-Permissions.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Colored-Indicators-on-Cues-by-Using-Codeunit-1.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Colored-Indicators-on-Cues.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Cues.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-Dynamics-CRM-Integration.md
:white_check_mark:Succeeded

dynamics-nav/Setting-Up-the-Home-Button-and-Home-Items.md
:white_check_mark:Succeeded

dynamics-nav/Setting-up-Hyperlinks-in-Word-Report-Layouts.md
:white_check_mark:Succeeded

dynamics-nav/Setting-up-Office-Add-Ins-Outlook-Inbox.md
:white_check_mark:Succeeded

dynamics-nav/SetupComplete2.cmd-File-Example.md
:white_check_mark:Succeeded

dynamics-nav/ShortCutKey-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowAsTree-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowCaption-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowFilter-Property.md
:white_check_mark:Succeeded

dynamics-nav/ShowMandatory-Property.md
:white_check_mark:Succeeded

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@buck1ey
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you’d like to see. Thank you!


Clean Code to release v0.3.0

It is necessary to review the whole project to remove unused code.

Reviewed


query timeouts not honored if broker takes extra time

if all historicals (and peon) respond within the query timeout specified and it is broker spending time doing the merge (usually in the case of merging lots of sketches) then timeout wouldn't be honored irrespective of however long broker takes to merge things.
on the client side, clients typically use the same timeout to close the connection... so broker doing all the work is eventually wasted.
one possibly simple (maybe crude) fix is that broker puts query processing startTime in query context and all things taking time to process the query continuously watch the time spent and fail if it exceeds. I'm imagining probably in QueryToolChest.mergeResults(..) impl and may be in the QueryResource where it is streaming the results to response.
another alternative to startTime would be to do query processing in a thread pool and cancel the future on query timeout .... druid code that takes time in processing the query would continuously check if the thread is interrupted.

Related to https://github.com/druid-io/druid/issues/1415

duplicate of #1415


Keep track of recently finished requests

This is only used so we don't log an error when receiving a CancelRequestNotification for a request that has just returned a response.

@swift-ci Please test


DevTools 2.20 in the Dart SDK was released with experimental features enabled

The version of DevTools currently in the SDK was built in profile mode instead of release mode, which resulted in all of the DevTools experimental features being enabled. Many of these features are incomplete or broken and should not be shipped with the 2.19 release.
We should create a new release and have it cherry picked in to the outgoing stable release.
FYI @mit-mit @itsjustkevin @kevmoo @vsmenon @a-siva

Tracking issue on the DevTools repo: https://github.com/flutter/devtools/issues/5073

The commit that will need to be cherry picked is up for review now: https://dart-review.googlesource.com/c/sdk/+/279335

lgtm for cherry pick.

Converted to a CP request issue, because the CP approval is on this issue already.
CP to beta changelist: https://dart-review.googlesource.com/c/sdk/+/279383

This is in 2.19.0.


python3Packages.pysonos: 0.0.46 -> 0.0.48

Motivation for this change
Update to latest upstream release  0.0.48
Change log: https://github.com/amelchio/pysonos/releases/tag/v0.0.48

Related Home Assistant change: https://github.com/home-assistant/core/pull/50798
Target Home Assistant release: current, only dependency update no changes

Things done

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

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

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

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

3 packages built:

home-assistant
python38Packages.pysonos
python39Packages.pysonos

Result of nixpkgs-review pr 123537 at 96819d28 run on aarch64-linux 1

3 packages built successfully:

home-assistant
python38Packages.pysonos
python39Packages.pysonos

@ofborg build home-assistant python38.pkgs.pysonos python39.pkgs.pysonos


HeroCard list not showing up as a list in teams desktop app

Hello. I'm trying to make so my bot displays a multi-select options list. I was able to follow this tutorial https://docs.microsoft.com/en-us/power-virtual-agents/advanced-bot-framework-composer-example2 on the official documentation and the list shows up fine in the test environment from pva.

But when I publish my bot to Teams desktop app, the list gets rendered as a card or something like that, is this intentional behavior or do I need to change something else?
This is how it shows on the Teams desktop app:

@ZagoSV Thanks for your message! The best place troubleshooting is the Power Virtual Agents forum. If you repost your question there an engineer, project manager, or someone with a similar issue/solution should be able to help you.

@v-alarioza Got it. Thank you.


Certain Dockerfile volumes preventing pod start

We've found certain volume paths defined in the Dockerfile prevent the Pod from starting with Kata. One such example is /sys/fs/cgroup.
The following Dockerfile can be used to replicate:
FROM ubuntu

VOLUME /sys/fs/cgroup

We receive the following errors:
Error: failed to create containerd task: Could not run process: container_linux.go:348: starting container process caused "process_linux.go:402: container init caused \"rootfs_linux.go:58: mounting \\\"/var/lib/containerd/io.containerd.grpc.v1.cri/containers/sleep/volumes/a45869c2ff90df3649c035b429e23e8b935804e90cd5cfd997778ff0d64d693c\\\" to rootfs \\\"/run/kata-containers/shared/containers/sleep\\\" at \\\"/sys/fs/cgroup\\\" caused \\\"stat /var/lib/containerd/io.containerd.grpc.v1.cri/containers/sleep/volumes/a45869c2ff90df3649c035b429e23e8b935804e90cd5cfd997778ff0d64d693c: no such file or directory\\\"\"": unknown

Unfortunately we do not have control over the contents of the Dockerfiles/images and need an elegant solution for handling this in Kata.
The following paths have been found to prevent the pods from starting:

/sys/fs/cgroup

(I will add more as I find them)
I think this issue is similar to https://github.com/kata-containers/runtime/issues/1561 as both are related to the volumes defined in Docker images/OCI image specs.

@amshinde @bergwolf can you take a look?

Thanks @awprice - OOI, do we know what that VOLUME over-ride is trying to achieve? It feels very strange... (I know, you didn't write it, but wondered if you had some insight?)

@awprice this issue was fixed in kata 1.6.2

@grahamwhaley Yep it is very strange and we've run into a number of headaches with it :(
I found this comment by a Docker maintainer from a while back that was really helpful for my understanding of what they are used for:

Theres more reasons for having a volume than just keeping the data around; an "unnamed" volume for example, can be for performance reasons (i.e., writing data to the union/layered filesystem of a container has a performance penalty - writing to a volume prevents that).
When not naming your volume you generally mean: I don't "care" about its data, it's ok to remove after the container is removed.

It's from https://github.com/moby/moby/issues/30647, which is definitely worth a read.

@awprice What version of runtime are you using? I am not able to reproduce this with the latest runtime.
@egernst @mcastelino Can one of you give this a shot as well?

We are using the following:
Runtime - https://github.com/kata-containers/runtime/releases/tag/1.7.0-alpha1
Containerd 1.2.6
Kubernetes 1.13.5
I believe you might not run into this issue if you are using Docker.


Rename search_paths to _directories

In progress.  Now I've broken a lot of code by renaming.

I think it's done.


[opt](inverted index) performance optimization for need_read_data in compound #35346

Proposed changes
pick from master #35346
Issue Number: close #xxx

run buildall

run buildall


The wagl Gadi migration should bump the second version digit, not the third

Each metadata version number has three digits:  eg: "3.0.0".
The Previous PR bumped the third digit (ie. "3.0.1") for new gadi data. But the third digit is the version of each dataset, not the collection. It's intended to be bumped when a dataset is reprocessed.
This changes the bump for our gadi software change to be the second digit, ie: "3.1.0"
I'm also adding the processing day as the dataset version (ie. "3.1.20190711"), to match what's done for L1 and USGS datasets. Currently wagl packages do no tracking of the dataset version, so reprocesses will all have the same number. This is better default behaviour, and it is backwards compatible with existing datasets.

Coverage increased (+0.005%) to 86.376% when pulling 9cc2a731ec33fcdf90fbede3ccaace421df00d61 on jeremyh:eodatasets3 into 70210fe0a29dde67bcd828c9d20c4e848ef02f7e on GeoscienceAustralia:eodatasets3.

The logic makes sense when the concept of the final number is dataset version.  I missed that part.  I was looking at the collection number as a whole, and thinking this is a patch to the collection, but completely ignoring everything at a dataset level.

Will be good to confirm this with @simonaoliver


Operations with loop variables

I have been playing around with this package, and it looks like there are many unimplemented corner cases involving operations with loop variables.
With version 0.6.27:
using LoopVectorization
function rshift_i!(out)
    n = length(out)
    @avx for i in 1:n
        out[i] = out[i] << i
    end
end

n = 512
out = Array{Int}(undef, n)
rshift_i!(out)

MethodError: no method matching <<(::VectorizationBase._MM{4,Int64}, ::VectorizationBase.SVec{4,Int64})
Closest candidates are:
  <<(!Matched::VectorizationBase.Static{N}, ::Any) where N at /home/kose/.julia/packages/VectorizationBase/OOwYB/src/static.jl:89
  <<(!Matched::VectorizationBase.Mask{W,U} where U<:Unsigned, ::Any) where W at /home/kose/.julia/packages/VectorizationBase/OOwYB/src/masks.jl:34
  <<(!Matched::VectorizationBase.SVec{W,T}, ::VectorizationBase.SVec{W,T}) where {W, T<:Union{Int128, Int16, Int32, Int64, Int8, UInt128, UInt16, UInt32, UInt64, UInt8, Ptr}} at /home/kose/.julia/packages/SIMDPirates/thWlM/src/SIMDPirates.jl:80
  ...

Stacktrace:
 [1] macro expansion at ./gcutils.jl:91 [inlined]
 [2] rshift_i!(::Array{Int64,1}) at ./In[78]:3
 [3] top-level scope at In[80]:3

function one_plus_i!(out)
    n = length(out)
    @avx for i in 1:n
        out[i] = 1 + i
    end
end

n = 512
out = Array{Int}(undef, n)
one_plus_i!(out)

MethodError: no method matching vstore!(::Ptr{Int64}, ::VectorizationBase._MM{4,Int64}, ::VectorizationBase._MM{4,Int64})
Closest candidates are:
  vstore!(::Ptr{T}, !Matched::Tuple{Vararg{VecElement{T},W}}, ::I, !Matched::Val{Aligned}, !Matched::Val{Nontemporal}) where {W, T, I, Aligned, Nontemporal} at /home/kose/.julia/packages/SIMDPirates/thWlM/src/memory.jl:329
  vstore!(::Ptr{T}, !Matched::Union{Tuple{Vararg{VecElement{T},W}}, VectorizationBase.AbstractStructVec{W,T}}, ::VectorizationBase._MM{W,I} where I<:Number) where {W, T} at /home/kose/.julia/packages/SIMDPirates/thWlM/src/memory.jl:474
  vstore!(::Ptr{T}, !Matched::Union{Tuple{Vararg{VecElement{T},W}}, VectorizationBase.AbstractStructVec{W,T}}, ::VectorizationBase._MM{W,I} where I<:Number, !Matched::Unsigned) where {W, T} at /home/kose/.julia/packages/SIMDPirates/thWlM/src/memory.jl:474
  ...

Stacktrace:
 [1] vstore! at /home/kose/.julia/packages/VectorizationBase/OOwYB/src/vectorizable.jl:197 [inlined]
 [2] macro expansion at ./gcutils.jl:91 [inlined]
 [3] one_plus_i!(::Array{Int64,1}) at ./In[72]:3
 [4] top-level scope at In[74]:3

I added these examples to the test suite.
Please file new issues as they come up!
PRs are welcome too.
For reference, the actual fixes were in SIMDPirates.jl.
Eventually, it'll need a refactoring with a focus on better type promotion behavior.

I cannot reproduce the test failures on Macs on travis.
I have neither a Mac, nor a CPU without AVX2. The latter is probably the problem. AVX2 added the SIMD shift operations.


Store patterns in sparse vectors/matrices instead of sets

I want to try this but it might require larger code changes than I thought.
A first hurdle is myempty: for a set it doesn't require further information, but for a sparse array we need to give the ambient size.

I'll do this once #135 is merged.

Ah sorry, I just read your comment in #135.
Let me retry to add pattern types first, then we go back to this and #135.

Do you still want this or can I close the issue @gdalle?
I can't imagine it being faster than DictHessianPattern since SparseVector and SparseMatrixCSC are based on Vectors of indices and values.

I think for small and medium problems using a BitMatrix to store the sparsity pattern in shared (Walther) hessian mode would crush the competition

So yeah I'd like to keep this issue open at the least


Is it possible to close a connection when no messages are received?

I have a simple use case where i'm running more tasks concurrently, every task is a different websocket connection that receives messages and is long running. Sometimes it can happen that a connection might not receive messages anymore, and in that case i would like to close the task handling that connection.  So when there haven't been messages for x second -> close the connection.
Now i just got started to this library, so i'm sorry if this question is too basic, but how can i do that? Or do i have to write my own implementation for this? I thought that using ping_timeout would do it, but it didn't work for me. Here is how i create a connection:
async def connect():
	async with websockets.client.connect('MY_URI',  ping_timeout=3) as ws:
			await ws.recv()

You need to combine ping_interval and ping_timeout. The maximum wait time is the sum of both.
But unless you really know you have a resource exhaustion problem (e.g. lack of file descriptors), I wouldn't reduce either of these variables and just rely on the default ping and TCP connection timeout. Reducing the interval means more packets exchanged which means more load. That needs to be carefully weighed up with the benefits (and verify that there are in fact benefits). This kind of sounds like premature optimisation. :slightly_smiling_face:

Fair enough! This is clear now, thank you a lot for the response @lgrahl !

As long as the other endpoint responds to pings, the connection will remain alive, even if no messages are exchanged.
Put a timeout on ws.recv and, if the timeout elapses, exit the with connect(...) block to close the connection.
See https://docs.python.org/3/library/asyncio-task.html#timeouts for ways to apply a timeout with asyncio.


Solidity's model checker basic options

Not sure if this is wanted, but I think it would be cool to have direct access to Solidity's model checker via the Foundry options. This is basically what I'm running locally.
The options are opt-in and not there by default, so it shouldn't bother the default case and Foundry users who don't explicitly add it.
Description
This PR adds the minimum required options to foundry.toml (as opt-in) that enable Soldity's model checker.
Motivation
Solidity has a built-in model checker that can be easy to use on OSX and Linux (the latter needs z3 <=4.8.14 installed system-wide) via compiler options.
Solution
The options were added based on a subset of the tools options. For now only the essential options were added, but the rest (such as divModNoSlack and invariants) can easily be added afterwards.
This PR depends on its corresponding ethers-rs PR (https://github.com/gakonst/ethers-rs/pull/1258), and the first commit here points to that PR's branch to show how the integration works, and would be removed if/when that PR is merged.
Example
The feature this PR enables is represented below.
Let src/Contract.sol be the following code, which you want to verify:
// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.13;

contract Contract {
	uint x;

	function inc() public {
		++x;
	}

	function inv(uint y) public view {
		require(y > 2);
		assert(x <= y);
	}
}

This is added to foundry.toml:
[default.model_checker]
contracts = { '/path/to/project/src/Contract.sol' = [ 'Contract' ] }
engine = 'chc'
timeout = 10000

Result of forge build:
[⠆] Compiling...
[⠑] Compiling 2 files with 0.8.13
[⠘] Solc 0.8.13 finished in 332.19ms
Compiler run successful (with warnings)
warning[6328]: Warning: CHC: Assertion violation happens here.
Counterexample:
x = 4
y = 3

Transaction trace:
Contract.constructor()
State: x = 0
Contract.inc()
State: x = 1
Contract.inc()
State: x = 2
Contract.inc()
State: x = 3
Contract.inc()
State: x = 4
Contract.inv(3)
  --> /path/to/project/src/Contract.sol:13:3:
   |
13 | 		assert(x <= y);
   | 		^^^^^^^^^^^^^^

just fixed fmt


Permission issue running with podman

Right off that bat, I realize that this is meant to work with docker and singularity. However, I have read that podman is supposed to be a drop-in replacement for docker. I have a server running RHEL 8 and docker is not supported, but podman is. If this issue cannot be replicated with docker, it's probably not an issue with fmri prep.
That said: when I try to run the container, I am immediately getting a permission error:
podman run --rm -it \
    -v /usr/local/freesurfer/license.txt:/opt/freesurfer/license.txt:ro \
    -v /data/chriscox/MRI/ds000157-download:/data:ro \
    -v /data/chriscox/MRI/ds000157-prep:/out poldracklab/fmriprep:1.4.0 \
    /data /out participant

Traceback (most recent call last):
  File "/usr/local/miniconda/bin/fmriprep", line 10, in <module>
    sys.exit(main())
  File "/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/cli/run.py", line 303, in main
    sentry_setup(opts, exec_env)
  File "/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/utils/sentry.py", line 77, in sentry_setup
    if dset_desc_path.exists():
  File "/usr/local/miniconda/lib/python3.7/pathlib.py", line 1329, in exists
    self.stat()
  File "/usr/local/miniconda/lib/python3.7/pathlib.py", line 1151, in stat
    return self._accessor.stat(self)
PermissionError: [Errno 13] Permission denied: '/data/dataset_description.json'
Sentry is attempting to send 0 pending error messages
Waiting up to 2.0 seconds
Press Ctrl-C to quit

This is my first time working with a container, so I am not sure how to debug/get any more useful information about what is going on while things are running or what the environment within the container looks like.

What are the permissions of /data/chriscox/MRI/ds000157-download and /data/chriscox/MRI/ds000157-download/dataset_description.json? I don't know how podman handles permissions, but the Docker container runs fMRIPrep as a non-root user. You may be able to get it to use your user with -u $UID, or perhaps by giving more liberal permissions to your data directories.

My permissions are set so that I own both as a non-root user:
[chriscox@relearn0 MRI]$ ls -ld ds000157-download/
drwxrwxr-x. 34 chriscox chriscox 4096 May 28 13:06 ds000157-download/
[chriscox@relearn0 MRI]$ ls -l ds000157-download/dataset_description.json
-rw-rw-r--. 1 chriscox chriscox 423 Jul 16  2018 ds000157-download/dataset_descr                                                               iption.json

When I run with -u chriscox I get:
podman run --rm -it -u chriscox -v /usr/local/freesurfer/license.txt:/opt/freesurfer/license.txt:ro -v /data/chriscox/MRI/ds000157-download:/data:ro -v /data/chriscox/MRI/ds000157-prep:/out poldracklab/fmriprep:1.4.0 /data /out participant
unable to find user chriscox: no matching entries in passwd file

I would try it with your actual $UID, not $USER, but that might still fail some kind of /etc/passwd check.
But anyway, if it works like Docker, you should be able to enter the container and poke around:
host$ podman run --rm -it \
    -v /data:/data:ro --entrypoint=bash \
    poldracklab/fmriprep:1.4.0

container$ ls -l /data{,/chriscox{,/MRI{/ds000157-download}}}

Thanks for iterating on this with me!
                                                         File "/usr/local/miniconda/bin/fmriprep", line 10, in <module>
    sys.exit(main())
  File "/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/cli/run.py", line 303, in main
    sentry_setup(opts, exec_env)
  File "/usr/local/miniconda/lib/python3.7/site-packages/fmriprep/utils/sentry.py", line 77, in sentry_setup
    if dset_desc_path.exists():
  File "/usr/local/miniconda/lib/python3.7/pathlib.py", line 1329, in exists
    self.stat()
  File "/usr/local/miniconda/lib/python3.7/pathlib.py", line 1151, in stat
    return self._accessor.stat(self)
PermissionError: [Errno 13] Permission denied: '/data/dataset_description.json'
Sentry is attempting to send 0 pending error messages
Waiting up to 2.0 seconds
Press Ctrl-C to quit

If I use the command you provided above, I am able to enter the container. It looks like the permissions on the /data directory are being set to root, and the group is nogroup.
[chriscox@relearn0 MRI]$ podman run --rm -it \
>     -v /data:/data:ro --entrypoint=bash \
>     poldracklab/fmriprep:1.4.0
root@7da8257a25be:/tmp# ls
root@7da8257a25be:/tmp# cd ../data/
root@7da8257a25be:/data# ls
ls: cannot open directory '.': Permission denied
root@7da8257a25be:/data# ls -ld .
drwxrwx---. 7 root nogroup 4096 May 24 17:50 .

I suggested mounting /data so that you could see at what point the permissions stop you from accessing contents. It looks like the problem is that /data doesn't have the o+x permission. My suspicion is that if you had that for all directories above yours, it would work.
I would suggest debugging further with your admin. I don't think there's anything we can do in fMRIPrep to work around these limitations.

Thank you so much. It turns out, podman does not behave exactly like docker. I came upon the solution while reading the following issue thread for podman. In short, if I add the --privileged flag, it works as expected.
[chriscox@relearn0 MRI]$ podman run --rm -it --privileged -v /data/chriscox/MRI/ds000157-download/:/data:ro --entrypoint=bash poldracklab/fmriprep:1.4.0
root@500e48676e27:/tmp# ls /data
CHANGES                   participants.json  sub-04  sub-09  sub-14  sub-19  sub-24  sub-29
README                    participants.tsv   sub-05  sub-10  sub-15  sub-20  sub-25  sub-30
annex-uuid                sub-01             sub-06  sub-11  sub-16  sub-21  sub-26  task-passiveimageviewing_bold.json
dataset_description.json  sub-02             sub-07  sub-12  sub-17  sub-22  sub-27
derivatives               sub-03             sub-08  sub-13  sub-18  sub-23  sub-28

Thank you for all your help with this!

Might've been a SELinux issue. What worked for me in similar situation was -v /data/chriscox/MRI:/data:Z. The Z tells podman to relabel.

Might've been a SELinux issue. What worked for me in similar situation was -v /data/chriscox/MRI:/data:ro,Z. The Z tells podman to relabel.

Hello from GPT4 era, even new Bing can't give this solution. Many thanks to you.


Actions with showOnTableRow() aren't shown on table row

I have nothing in my filters/actions, they're both moved within MegaFilter. However actions with the method ->showOnTableRow() only appear in the megafilter, and not inline on the row.
This is a relatively new core method. And for the time being, I'm moving my actions back to actions().
Doesn't work:
          'actions' => [
               (new Actions\AcceptKey)->showOnTableRow(),
          ],

Works:
    public function actions(Request $request)
    {
        return [
              (new Actions\AcceptKey)->showOnTableRow(),
        ];
    }

But that's a bit weird, how to show an "inline" action from this package, if the action is inline I think it should be removed from the MegaFilter actions picker, somewhat be "ignored"

Agreed, but that specific action is also one that can be mass assigned, so yes, it's valuable inline, but also if someone sees multiple rows that need assigned.
Maybe something to point out in the documentation, with a possible solution of two actions, one dedicated to inline actions, the other to mass assignment. :thinking:
I'll have a play around with it tonight.

This package currently doesn't support conditionally show/hide actions as it does with filters, but if I were to add this functionality for actions too then I think it would make sense having inline actions that only appears once a certain column is active.. so you could have something like:
Column::make('Image')->addAction(DownSampleImage::class),


list/tuple/sequence: implement Index

See #1667

Yeah, I though about it, but that runs into API consistency again: do we want to panic on invalid indices or not?
In the end, I didn't (and don't) care about it enough to make the change.


seprated db and api schemas

What does this pull request do? Explain your changes. (required)
Separated database and API schemas. Instead of one schema, we now have two schemas, the api-schema.yaml is the schema for API references, and db-schema.yaml is similar to API schema except it has keys that open API doesn't validate but it is needed for db such as readOnly, table.

@victorges This PR is just for the separation of the schemas,  I will open two other follow-up PRs for including other APIs that are missing here and another one for API references

Specific updates (required)

Separated database and API schemas

Does this pull request close any open issues?
Linear - DX 123
Checklist

[X] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

I will take a look on why test are failing as well :)

Does this pull request close any open issues?
Linear - DX 123

you can edit the title of your PR to start with [DX-123] and linear should automatically pick up correct reference.

Oh, that is cool! thanks @hjpotter92

Hmm, it seems like the build is also failing. Is the merged schema coming out with the exact types as the original schema? Maybe one flow that you can follow is:

run yarn compile-schemas from master (from inside packages/api)
copy the generated schema.json file
change to your branch and run compile-schemas again
compare the generated schema.json with the one that came from master. Make sure only docs change

(need to fix tests)

Looks great! Good to merge @suhailkakar?

Almost!, just pushing a final change :)

Almost!, just pushing a final change :)

great. feel free to merge afterwards

I believe it is now completed, @victorges can you please confirm?

@victorges heads up not sure if you're aware but looks like checks are failing after merging.

Yeah that was fixed later on #1720. I think this branch wasn't rebased


Update wallet description

Hello, is it possible to update our 2 wallets description.
Thanks
talao.json
altme.json

Hi @ThierryThevenet ,
yes, just fork the repository, make the changes to your files and open a pull request. Let me know when you need any help.

Hello
Here is the pull request @cre8 : https://github.com/openwallet-foundation/digital-wallet-and-agent-overviews-sig/pull/133

Got merged, thank you for the update!


Cannot disable endpoint deviceauthorization

I was hardening the identityServer4 and closing some endpoints.
Is there a reason that in EndpointOptionsExtensions.cs that devicecodeauthorization endpoint is not mentioned, I suspect this is a bug.
Issue / Steps to reproduce the problem

In AddIdentityServer disable the deviceauthorizationendpoint at start-up:

var builder = services.AddIdentityServer(o =>
{
  // Remove endpoints not in use
  o.Endpoints.EnableDeviceAuthorizationEndpoint = false;
}

Check if the endpoint /connect/deviceauthorization gives 404 after disabling the endpoint in the IdentityServerOptions (Personal hardening unit test)

// Act
var response = await client.GetAsync("http://localhost/connect/deviceauthorization");

// Assert
Assert.IsFalse(response.IsSuccessStatusCode);
Assert.IsTrue(response.StatusCode == System.Net.HttpStatusCode.NotFound);

However it gives a 400 as a response. Indicating it is still open.

I suspect this is a bug.

Perhaps more of an omission, than a bug.

True. As a workaround I manually put the handler out of the servicecollection for the moment so my test passes.

Hi @jaanclaeys @brockallen 👋
Please find above a PR to resolve this issue!


Request for SublimeLinter-cmakelint

cmakelint is a linter that checks coding style of cmake files such as CMakeLists.txt, FindXXX.cmake, and it could be obtained by
sudo pip install cmakelint
I attempted myself to do it based on SublimeLinter-cpplint but failed due to my lack of knowledge in Python.
Thanks in advance to whoever knows python well that makes it happen!

Figured out a way to make it work
https://github.com/jasjuang/SublimeLinter-cmakelint


MultiTenancy - Admin user not able to designate tenancy

I think you tackled this with someone before but even using google translate I had some difficulty following what you were telling them to do so I ask here in English with hopes of a solution.
I've implemented the MultiTenancy per the guide and I've ran into two issues. Firstly, I designated the restriction on the UserDialog,cs regarding showing TenantID only when the admin user logs in.
protected override List GetPropertyItems()
{
var items = base.GetPropertyItems();
        if (!Authorization.HasPermission("Administration:Tenants"))
            items = items.Where(x => x.Name != UserRow.Fields.TenantId).ToList();

        return items;
    }

This make the TenantID field disappear for secondary tenant (great) but it's also disappearing for the admin user.
Secondly, the restriction upon the Tenancy table is present for the second user but it is still displaying the Tenant table on the left nav bar for the secondary user though it will break when the secondary user attempt to access the table "{"Authorization has been denied for this request!"}".

Your HasPermission method in script side should be like this:
         public static bool HasPermission(string permissionKey)
          {
            return
                UserDefinition.Username == "admin" ||
                UserDefinition.Permissions[permissionKey];
        }

For navigation problem, set permission Administration:Tenants for row, page and endpoint

This is also changed in latest version.


Update Collections docs - Add ListNode examples

As per Issue #3132.
After discussion with @SeanTAllen on Zulip[1], notes were added to flag that some functions are only intended to be called by Collections.List
[1] https://ponylang.zulipchat.com/#narrow/stream/189985-beginner-help/topic/Prepending.20.2F.20appending.20to.20ListNodes

Thanks @andrewharmellaw!


word_tokenize() fails with a misleading error message if you give it an invalid language

If you call word_tokenize() and pass a language that is not supported by punkt, it returns an error message saying that punkt could not be found, instead of the language.
word_tokenize() should probably fail with a different error that indicates an invalid language or that a language was not found in this case. With the current error message, one could be led on a wild goose chase to find out why nltk is not recognizing a download.
If I filter out the language that is missing a punkt model, everything runs normally otherwise.
I'm running Windows 10, Python 3.7.0, and nltk 3.3
Example:
>>>nltk.tokenize.word_tokenize('Abc cde def, abc?', language='esperanto')
[I'll put the full traceback in a comment]
LookupError:
**********************************************************************
  Resource punkt not found.
  Please use the NLTK Downloader to obtain the resource:

  >>> import nltk
  >>> nltk.download('punkt')

  Searched in:
    - 'C:\\Users\\roger/nltk_data'
    - 'C:\\nltk_data'
    - 'D:\\nltk_data'
    - 'E:\\nltk_data'
    - 'C:\\Users\\roger\\Miniconda3\\nltk_data'
    - 'C:\\Users\\roger\\Miniconda3\\share\\nltk_data'
    - 'C:\\Users\\roger\\Miniconda3\\lib\\nltk_data'
    - 'C:\\Users\\roger\\AppData\\Roaming\\nltk_data'
    - ''
**********************************************************************

@rfilmyer nope, there's nothing wrong with that. The word_tokenize currently requires the sent_tokenize so it requires the punkt models.
Following the message to download punkt and afterwards word_tokenize() would work.

Proposed improvement to the error message to make it more obvious that a specific language failed to load: https://github.com/nltk/nltk/pull/2179


Add var.managed_iam_policy_enabled to use a managed IAM policy or inline IAM policy

what

Add var.managed_iam_policy to use a managed IAM policy or inline IAM policy
Maintains backwards compatibility by defaulting this to true

why

Most of the time people do not need a managed IAM policy which are useful when an IAM policy should be reused.
If a single purpose EKS IAM role is created, the IAM policy most likely does not need to be reused and so the var.managed_iam_policy can be set to false to switch to an inline IAM policy.

references

N/A

/test all

/test all

/test all

Can this be merged?

@nitrocode friendly ping -- Looks like this lost traction and has conflicts now, but still seems valid. Want to get it updated and I'll give you a final review?

Ah yes I was looking for this feature today... would be nice!


HOW TO SOLVE THIS PROBLEM in train

when run "train.py --seed 1 --bS 16 --accumulate_gradients 2 --bert_type_abb uS --fine_tune --lr 0.001 --lr_bert 0.00001 --max_seq_leng 222"
HOW TO SOLVE THIS PROBLEM
this is the error :
/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/torch/nn/functional.py:1386: UserWarning: nn.functional.sigmoid is deprecated. Use torch.sigmoid instead.
warnings.warn("nn.functional.sigmoid is deprecated. Use torch.sigmoid instead.")
Error closing cursor
Traceback (most recent call last):
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/engine/result.py", line 1268, in fetchone
row = self._fetchone_impl()
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/engine/result.py", line 1148, in _fetchone_impl
return self.cursor.fetchone()
sqlite3.ProgrammingError: Cannot operate on a closed database.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 1333, in _safe_close_cursor
cursor.close()
sqlite3.ProgrammingError: Cannot operate on a closed database.
Traceback (most recent call last):
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/engine/result.py", line 1268, in fetchone
row = self._fetchone_impl()
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/engine/result.py", line 1148, in _fetchone_impl
return self.cursor.fetchone()
sqlite3.ProgrammingError: Cannot operate on a closed database.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "train.py", line 605, in
dset_name='train')
File "train.py", line 310, in train
cnt_x1_list, g_ans, pr_ans = get_cnt_x_list(engine, tb, g_sc, g_sa, sql_i, pr_sc, pr_sa, pr_sql_i)
File "/home/quh/pythonwork/nl2sql/nl2sql_baseline/sqlova/sqlova/utils/utils_wikisql.py", line 1652, in get_cnt_x_list
g_ans1 = engine.execute(tb[b]['id'], g_sc[b], g_sa[b], g_sql_i[b]['conds'])
File "/home/quh/pythonwork/nl2sql/nl2sql_baseline/sqlova/sqlnet/dbengine.py", line 29, in execute
table_info = self.db.query('SELECT sql from sqlite_master WHERE tbl_name = :name', name=table_id).all()[0].sql.replace('\n','')
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/records.py", line 195, in all
rows = list(self)
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/records.py", line 126, in iter
yield next(self)
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/records.py", line 136, in next
nextrow = next(self._rows)
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/records.py", line 365, in
row_gen = (Record(cursor.keys(), row) for row in cursor)
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/engine/result.py", line 946, in iter
row = self.fetchone()
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/engine/result.py", line 1276, in fetchone
e, None, None, self.cursor, self.context
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/engine/base.py", line 1466, in _handle_dbapi_exception
util.raise_from_cause(sqlalchemy_exception, exc_info)
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/util/compat.py", line 399, in raise_from_cause
reraise(type(exception), exception, tb=exc_tb, cause=cause)
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/util/compat.py", line 153, in reraise
raise value.with_traceback(tb)
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/engine/result.py", line 1268, in fetchone
row = self._fetchone_impl()
File "/home/quh/.conda/envs/pytorch/lib/python3.7/site-packages/sqlalchemy/engine/result.py", line 1148, in _fetchone_impl
return self.cursor.fetchone()
sqlalchemy.exc.ProgrammingError: (sqlite3.ProgrammingError) Cannot operate on a closed database.
(Background on this error at: http://sqlalche.me/e/f405)

I solved this problem by  "pip install records==0.5.2"
but got another problem...

Thanks so much!


Avoid copying over data in the write_vio into response_buffer

At Http2Stream::update_write_request(), data in the write_vio is copied to response_buffer and consumed immediately . But we don't need it.
Also, this could be fix of #2530. Because the response_buffer is removed.

Notes: This PR includes commits of #2837. Please look at last commit. I'll rebase after #2837 is merged.

This PR can remove the need for checking the size of the response buffer in #2534.  However, #2534 also checks the size of the chunked_handler buffer.

@bryancall I rebased this PR on latest master (includes #2534) and removed only response_buffer size check.
It'd be great if ChunkedHandler or HttpTunnel have dechunked_buffer size check. But it looks out of scope of this PR.

@zwoop I'll rebase this PR when #2837 is merged. The first two commits will disappear.

[approve ci autest]

[approve ci CentOS]
[approve ci Ubuntu]

Rebased on latest master (c802ea2f95af2360520fc75d57266586271e0196)

Ah, I rebased on broken master. Rebased latest master again.

[approve ci autest]

[approve ci autest]

This breaks larger downloads (they stall). I think anything > 1MB will fail over H2 now with this, I backed out this change from Docs, and it makes things work again.


Fix --output option to dump the model in a specific directory

I've implemented this some time ago and it worked relatively well for normal models, however for transformers we realised it was not taken in account. With the new update to TF2 it seems to work fine for sequenceLabelling, except for the n-fold cross-validation.
In particular the problem seems to be related to the fact that the store of the model is hidden within the cross-validation. In my opinion we should call the model.save() after the n-fold cross-validation which will save either just one model (the best) or all of them (e.g. in case of ensemble).
My proposal is to give the wrapper a working/temporary directory and then explicitly save the model using model.save() and passing either the --output path or the default path within the data/models.

My suggestion would the following:

for train we should keep it as it is, the model is saved in the proper directory when calling model.save()
for train_eval, in particular n-fold cross-validation the model will be saved in a temporary directory and placed in the correct directory when calling model.save(), notice that the behaviour here could be to copy just the best model or all the models (e.g. envisioning a ensemble classification approach).
model.save() will default under data/xxx/model_name in case no parameter is specified

With this approach the wrapper will just need to know the tmp directory and if something fails delft will not pollute the data/ directory
What do you think?

I think to clarify, there is no problem currently with --output option afaik.
The dir_path parameter in save() for the two wrappers is working fine in n-fold usage, both with and without transformer, in previous version and current one. The --output parameter in grobidTagger.py is working too - this is the only "applications" script using the --output parameter (it can be used to save directly a Grobid model under grobid home, with the right compatible Grobid model name).
Using the tmp folder will modify the saving mechanism to address #126, but the --output option is working without it in my tests.

Maybe the current version is good now.
When I opened this issue it was the version 0.2.6 I think and at that time it was not working only in the case of scibert + 10fold or training (I can't remember exactly) because only config and preprocessor were saved correctly in the output directory, while the rest was saved in the default directory data/models/....
Since there is a new version we can close it from my side 🎉


Unable to install plugin on ES 2+

Hi, due to the changes made to ES on 2+ this plugin can no longer be installed.
I've created a fork at https://github.com/bomberby/elasticsearch-concatenate-token-filter/tree/es-2.2 that allows me to install on 2.2.0, and https://github.com/bomberby/elasticsearch-concatenate-token-filter/tree/es-2.2.1 for 2.2.1(versions I personally needed).
I think you probably need to switch to versioning of the plugin to support newer ES, Is there anything i can do to help?

Would your fork be backward compatible? I'm really not an expert on the elasticsearch plugin environment, but if that would be the case I would be very glad to accept a pull request!

Unfortunately, it is not, ElasticSearch changed variables names and the way plugins install.
One option i can think of is to create a version meant for 1.X(the current one), and another for higher versions, but i'm not sure if we can have one that will fit all 2.X, or will need to generate one for each version.

Oh that's unfortunate. What do you think would be the best thing? Keeping two separate repositories? Would you maintain the 2.X version?

I think having under this repository multiple versions would probably be the best, and people coming to use this plugin will be able to see it straight away. Installation could be performed via
sudo bin/plugin install francesconero/elasticsearch-concatenate-token-filter/2.2.0 for example

So just have a 1.X and a 2.X branch?

This will be the fastest solution, your 1.X is known to work, and the 2.X could probably use more testing, but it's working for me on 2.2, this will enable users to enjoy both worlds.

Great, so I should open a 2.X branch so that you can use it as a target for your pull request, correct?

Yes, Please open 2.2.0 Branch, as i'm not sure if my fork works on 2.0 and 2.1

I've created the 2.2.0 branch and waiting for your pull request!

Thank you, created at #3

Closing after accepting #3

@bomberby would you like to be added as a collaborator to the repo?

Sure, if you want to, but don't count on my Java skills.

Done, at least you seem to be more up to date with current releases of ES (I've stopped using it temporarily). Thanks for the PR!


Safari - Mentee cannot apply, getting errors

After last Pull Request, a Mentee cannot apply, this seems only happening in Safari
Tested in dev, getting error in header.  info below.  in chrome works fine, but safari stopped working.

Console

Network
{
    "message": "This user is already registered",
    "result": null,
    "success": false
}

Network.headers
3
9
88 B
1.05 KB
0
Summary
URL: https://mentee-dev.herokuapp.com/api/application/new
Status: 422 UNPROCESSABLE ENTITY
Source: Network
Address: <IP_ADDRESS>:443
Initiator:
xhr.js:258

Request
POST /api/application/new HTTP/1.1
Accept: application/json, text/plain, */*
Accept-Encoding: gzip, deflate, br
Accept-Language: en-GB,en-US;q=0.9,en;q=0.8
Connection: keep-alive
Content-Length: 561
Content-Type: application/json
Cookie: _ga_HSJ2934X33=GS1.1.17075726<IP_ADDRESS>7573<IP_ADDRESS>; _ga=GA1.1.277300148.1707363500
Host: mentee-dev.herokuapp.com
Origin: https://mentee-dev.herokuapp.com
Referer: https://mentee-dev.herokuapp.com/application-form
Sec-Fetch-Dest: empty
Sec-Fetch-Mode: cors
Sec-Fetch-Site: same-origin
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.2.1 Safari/605.1.15

Response
HTTP/1.1 422 UNPROCESSABLE ENTITY
Access-Control-Allow-Origin: https://mentee-dev.herokuapp.com
Connection: keep-alive
Content-Length: 76
Content-Type: application/json
Date: Sat, 10 Feb 2024 13:57:37 GMT
Nel: {"report_to":"heroku-nel","max_age":3600,"success_fraction":0.005,"failure_fraction":0.05,"response_headers":["Via"]}
Report-To: {"group":"heroku-nel","max_age":3600,"endpoints":[{"url":"https://nel.heroku.com/reports?ts=1707573457&sid=e11707d5-02a7-43ef-b45e-2cf4d2036f7d&s=Hb7iuGwH%2F1RxTADh1xgIq5q%2BTlC8YU6V0mDay61SJxc%3D"}]}
Reporting-Endpoints: heroku-nel=https://nel.heroku.com/reports?ts=1707573457&sid=e11707d5-02a7-43ef-b45e-2cf4d2036f7d&s=Hb7iuGwH%2F1RxTADh1xgIq5q%2BTlC8YU6V0mDay61SJxc%3D
Server: gunicorn
Vary: Origin
Via: 1.1 vegur

Request Data
MIME Type: application/json
Request Data:

Nothing clear in logs
Feb 10 05:56:23 [mentee-dev](https://my.papertrailapp.com/systems/mentee-dev/events?focus=1692972764468592646&selected=1692972764468592646) [heroku/router](https://my.papertrailapp.com/events?q=program%3Aheroku%2Frouter&focus=1692972764468592646&selected=1692972764468592646) at=info method=POST path="/api/application/new" host=mentee-dev.herokuapp.com request_id=[3e730b4c-ce30-4f93-9c14-7d23caf077af](https://my.papertrailapp.com/systems/mentee-dev/events?q=%223e730b4c-ce30-4f93-9c14-7d23caf077af%22&focus=1692972764468592646&selected=1692972764468592646) fwd="[<IP_ADDRESS>](https://my.papertrailapp.com/systems/mentee-dev/events?q=%22<IP_ADDRESS>%22&focus=1692972764468592646&selected=1692972764468592646)" dyno=web.1 connect=0ms service=5ms status=422 bytes=316 protocol=https
Feb 10 05:56:23 [mentee-dev](https://my.papertrailapp.com/systems/mentee-dev/events?focus=1692972764875497520&selected=1692972764875497520) [app/web.1](https://my.papertrailapp.com/events?q=program%3Aapp%2Fweb.1&focus=1692972764875497520&selected=1692972764875497520) [2024-02-10 13:56:23,216] DEBUG in manage: POST /api/application/new 422 b'{"message":"This user is already registered","result":null,"success":false}\n'
Feb 10 05:56:23 [mentee-dev](https://my.papertrailapp.com/systems/mentee-dev/events?focus=1692972764875497522&selected=1692972764875497522) [app/web.1](https://my.papertrailapp.com/events?q=program%3Aapp%2Fweb.1&focus=1692972764875497522&selected=1692972764875497522) [2024-02-10 13:56:23 +0000] [8] [DEBUG] POST /api/application/new 422 b'{"message":"This user is already registered","result":null,"success":false}\n'
Feb 10 05:56:23 [mentee-dev](https://my.papertrailapp.com/systems/mentee-dev/events?focus=1692972764875497526&selected=1692972764875497526) [app/web.1](https://my.papertrailapp.com/events?q=program%3Aapp%2Fweb.1&focus=1692972764875497526&selected=1692972764875497526) 2024-02-10 13:56:23,216 [<IP_ADDRESS>](https://my.papertrailapp.com/systems/mentee-dev/events?q=%2<IP_ADDRESS>%22&focus=1692972764875497526&selected=1692972764875497526): requested https://mentee-dev.herokuapp.com/api/application/new: DEBUG in [manage: 15]: POST /api/application/new 422 b'{"message":"This user is already registered","result":null,"success":false}\n'
Feb 10 05:56:23 [mentee-dev](https://my.papertrailapp.com/systems/mentee-dev/events?focus=1692972764875497529&selected=1692972764875497529) [app/web.1](https://my.papertrailapp.com/events?q=program%3Aapp%2Fweb.1&focus=1692972764875497529&selected=1692972764875497529) [<IP_ADDRESS>](https://my.papertrailapp.com/systems/mentee-dev/events?q=%2<IP_ADDRESS>%22&focus=1692972764875497529&selected=1692972764875497529) - - [10/Feb/2024:13:56:23 +0000] "POST /api/application/new HTTP/1.1" 422 76 "https://mentee-dev.herokuapp.com/application-form" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.2.1 Safari/605.1.15"
Feb 10 05:56:28 [mentee-dev](https://my.papertrailapp.com/systems/mentee-dev/events?focus=1692972786509611009&selected=1692972786509611009) [heroku/web.1](https://my.papertrailapp.com/events?q=program%3Aheroku%2Fweb.1&focus=1692972786509611009&selected=1692972786509611009) source=web.1 dyno=heroku.199150821.[8906a0ab-5dbc-450f-9c70-2e065b507834](https://my.papertrailapp.com/systems/mentee-dev/events?q=%228906a0ab-5dbc-450f-9c70-2e065b507834%22&focus=1692972786509611009&selected=1692972786509611009) sample#load_avg_1m=0.00 sample#load_avg_5m=0.00 sample#load_avg_15m=0.00
Feb 10 05:56:28 [mentee-dev](https://my.papertrailapp.com/systems/mentee-dev/events?focus=1692972786509611010&selected=1692972786509611010) [heroku/web.1](https://my.papertrailapp.com/events?q=program%3Aheroku%2Fweb.1&focus=1692972786509611010&selected=1692972786509611010) source=web.1 dyno=heroku.199150821.[8906a0ab-5dbc-450f-9c70-2e065b507834](https://my.papertrailapp.com/systems/mentee-dev/events?q=%228906a0ab-5dbc-450f-9c70-2e065b507834%22&focus=1692972786509611010&selected=1692972786509611010) sample#memory_total=185.62MB sample#memory_rss=161.95MB sample#memory_cache=23.67MB sample#memory_swap=0.00MB sample#memory_pgpgin=200pages sample#memory_pgpgout=0pages sample#memory_quota=512.00MB

Update, besides the error user gest, the user seems to be created as Admin can see it

But no email is sent to the users.


Output / Current cell not scrolled into view on shift-enter

I'm using data-sciencetutuorial-for-beginners from kaggle.
This doesn't always repro, but seems to be pretty much 100% after I do a clear outputs command from the toolbar.

Clear outputs from top toolbar (I've seen this other times, but seems to help the repro)
Select a cell near the bottom
Shift enter

Expected: either the output is on screen or the next cell with my selection context.
Actual: nothing is scrolled.

Dupe of #7270


common transform toBoolean added

fixes #3518  , changes all the cell data that doesn't match with "true" as false . similar to what the "edit" shown on each cell does when "to boolean" option is chosen.

@wetneb  yeah of course it had to be boolean data ,sorry my bad ! updated

i have added a submenu to choose the language ,for now javascript sql and python , (now the data is being transformed to true or false based truthy values as documented in that particular language.
please let me know if it works well

Ah sorry I didn't see the discussion on the linked issue. It looks like people have legitimate concerns about adding those.

@singhakshita unfortunately I think we need to close this PR - sorry for the work you invested into it! We should have checked the discussion on the related issue more carefully.


SBOM tools test suite

I would be useful to have a way to see the outputs produced of various SBOM tool so we can:

Compare accuracy (breadth of detection or completeness / fidelity / metdata included)
Compare semantics (each tool translates realty in code differently into a SBOM)
Compare component ids (each tool uses generates different package names/identifiers)

@joshbressers  as said in today Security Tooling meeting  I am working on repository containing  various package definition files (pom.xml, package.json, etc) which will be scanned using (FOSS) SBOM tools every 24 hours using GitHub actions.
My intent is that one can always compare latest state of the tools to make it easier to choice one that fits their business needs.


Weird buffer size limit-like behavior

Description
Trouble with sending a long base64 string, usually gets limited to 2072 characters (bytes) but occasionally also sends 3108 or 1036 characters, the full string is over 10 thousand characters. The server is written in C# but the issue cannot be caused by that as i have tried everything server-side i could, buffer size is not a limitation on the server.
Only thing i can think of is a buffer size limitation in react-native-tcp-socket
Steps to reproduce
Sending a very long string in one write() - string long over 10 thousand chars.
Current behavior
Sent data limited to 2072 bytes (usually)
Expected behavior
Buffer size only limited by memory or the network itself
Relevant information
| OS | Android 11  |
| react-native | 0.64.3 |
| react-native-cli | 2.0.1 |
| react-native-tcp-socket  | 5.5.0 |

@Aensdottir, have you tested your implementation with a NodeJS script? This library's behavior should match Node's net API. If they do not match, please, provide a minimal reproducible code so I can run it in both React Native and NodeJS in order to match the expected behavior.

@Rapsssito
I have created a quick NodeJS script running very similar code to my React Native application and it worked flawlessly. The string i'm sending (an image in b64) is long over 5 million characters.
NodeJS Client code - Working
console.log("Start");

const net = require("net");
const fs = require("fs");

var image = base64_encode("F:/TEST.jpg");
console.log(image.length);

Connect(image);

function Connect(command) {
  const client = net.createConnection({ port: 10144 }, () => {
    console.log("connected to server!");
    client.write(command);
  });
  client.on("data", (data) => {
    console.log(data.toString());
    client.end();
  });
  client.on("end", () => {
    console.log("disconnected from server");
  });
}

function base64_encode(file) {
  var bitmap = fs.readFileSync(file);
  return new Buffer(bitmap).toString("base64");
}

React Native Implementation
function SendImage() { // RN Package for photos
    const imgOptions = {
      mediaType: "photo",
      includeBase64: true,
    };
    launchImageLibrary(imgOptions, callback);

    var image = "";
    function callback(props) {
      image = props.assets[0].base64; // Base64 String
      TcpConnect(image);
    }
  }
function TcpConnect(command) {
  // Connect
  const client = TcpSocket.createConnection(options, () => {
    client.write(command); // Sends only a small portion of the string
  });

  // On data received
  client.on("data", function (data) {
    console.log(bin2String(data));
    client.destroy();
  });
  client.on("error", function (error) {
    console.log("error:", error);
  });
  client.on("timeout", function () {
    console.log("socket timeout");
  });
  client.on("close", function () {
    console.log("Connection closed!");
    client.destroy();
  });
}

@Aensdottir, thanks a lot. I will try to match the behavior ASAP. In the meantime, a workaround could be sending the data in smaller slices.

@Aensdottir, I have been testing your code in a both NodeJS server/client config and the buffer size limitation is still present. However, NodeJS implementation has a larger buffer and has a more direct connection with the OS so, in your case, this results in the sending pipe not filling up and the data being "perceived" as one large chunk.
In RN, I have more limitations since I have to use the bridge between JS and the native OS, so I am forced to use smaller chunks. Since TCP is stream oriented protocol, this should not be an issue, you will receive the same information but in smaller chunks. In your protocol, you can indicate the start and end of the file with any special bytes and concatenate all the chunks as they are received. react-native-tcp-socket uses the drain event for this kind of data transfers.

@Rapsssito, Thank you for the clarification, i have looked into the drain event and slicing data into chunks however i am not able to get this implementation to work with neither a C# server nor a NodeJS server, it seems the drain event is triggered but never resumes writing.
Would you, please, be able to provide me with some minimal code or further guidance/explanation as to how this would work?

@Aensdottir, I have uploaded a new example code. You can find it here.


New version: Baidu.BaiduWenku version 1.1.4

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.1 schema?

Microsoft Reviewers: Open in CodeFlow

😕

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot run

https://github.com/microsoft/winget-cli/issues/2057

🤔

@SpecterShell The installer here doesn't seem to have an unattended mode - I wasn't able to find the switch to activate it, nor could I find switches to load a help page.

@SpecterShell The installer here doesn't seem to have an unattended mode - I wasn't able to find the switch to activate it, nor could I find switches to load a help page.

It is a NSIS installer so the silent switch is /S (and it works).
There is nothing wrong with the manifest. The validation pipleline fails due to https://github.com/microsoft/winget-cli/issues/2057


Build status

I was checking Travis CI configuration earlier, and fired a build of the last commit to the master branch by mistake. https://travis-ci.org/colszowka/phantomjs-gem/builds/43432489
Since RSpec resolves to 3.x these days, and the build failed. I apologize for this mixup. https://github.com/colszowka/phantomjs-gem/pull/45 should fix this issue.

The status seems fixed.


How can I enable rules in Java?

I'm trying to follow this link (http://stackoverflow.com/questions/6613321/form-validation-library-for-android) without success.
I have a situation where I create views dynamically inflating them from a xml file. Is it possible to do that using annotation? If not, how can I do that in Java?

I would suggest you to use v1 for this particular scenario, because v2 allows only annotations. However, I am keeping this issue open until built-in QuickRules can be added using Java.

Saripaar v2 contains a subset of validators from the Commons Validator project. You can find them here.
Use them in combination with a QuickRule to achieve the desired result.


How to create relation between two bootstrap next tables?

I would like to have two tables that should be in connection. The first one contains orders, and the second one contains products. The orders (Order class) among other properties have an Id. The products (Product class) also have an Id among other properties, and reference to orders, which is OrderId property.
I'm able to populate orders table by using:
componentDidMount() {
  axios.get("http://localhost:1234/api/orders").then(response => {
    console.log(response.data);
    this.setState({
      orders: response.data
    });
  });
}

But, I'm not sure what to do next. I suppose I should use react-router, but in which way?
I did the following with columns:
columns: [{
  dataField: 'Id',
  text: 'Order ID',
  formatter: this.idFormatter
}, {
  dataField: 'Date',
  text: 'Date',
  formatter: this.dateFormatter
}, {
  dataField: 'Total',
  text: 'Total'
}],

idFormatter = (cell, row) => {
  return (
    <Link to={`/api/products/${row.Id}`}>
      <span>#{cell}</span>
    </Link>
  )
}

That is when I click the cell with Id in order's table, the table with products should display only products for that order's Id. Without postback of course.
I'm not sure if I should use the second api controller - products, that has one method:
public List<Product> GetProductsByOrderId(int id)
{
    return _products.Where(product => product.OrderId == id).ToList();
}

I appreciate any help.

I think there's no any best answer for this question, in your case, when user click the order, you have to populate the corresponding products via a API call. then feed the product data into product table, I think that's it.
I'm not sure why you need to use react-route, is that because you have to change to the other page when user want to see the product per order? if yes, I think you can use react-router, in this case, you still have other API to fetch the products.
Close this issue due to no it's not an issue about react-bootstrap-table but we can still discuss here :)

Thank you. I did it in first way you'd suggested.

Good job 👍


mdbook-forc-documenter needs a more sophisticated way of handling nested sections

With #2440, we needed to document with the following structure:
...
- 11.4. Plugins
  - 11.4.1 forc client
    - <IP_ADDRESS> forc deploy
    - <IP_ADDRESS> forc run
  - 11.4.2 forc explore

But mdbook-forc-documenter does not support this kind of nesting. Related to https://github.com/FuelLabs/sway/pull/2440#discussion_r936602561 and https://github.com/FuelLabs/sway/pull/2440#discussion_r936745062
I introduced some intermediate implementations to create the structure above at that PR but it only supports a single level of nesting. We may need to have a graph-like structure to hold this nesting information.

I think the intermediate implementations are close enough but could just use a little tweaking for readability - see this comment.

I don't think this should be necessary, I'm happy for us to draw the line at one level of nesting. I think all we're after is a way for a single plugin crate to support more than one command.

I think this is completed/not-planned as the linked comment is addressed and further enhancements are not needed. Feel free to re-open it otherwise.


Configuration Request - Ragnarok Online Dedicated Server

Module Request
Note: Please try setting up a configuration yourself before raising an issue to request a configuration:  https://config.getamp.sh/
Game Information:
Game Title: Ragnarok Online
How to obtain: https://github.com/rathena/rathena/
Suppoted OSs: Windows, CentOS, Debian, FreeBSD
Supports RCON/Console input? Yes, it has one main console option, or can be split into 4 specific consoles (login, char, map, web or all)
I confirm:

[x] that I have searched for an existing module request for this application.
[ ] that the server is an official server distributed by the games publishers/developers.
[x] that the server is available to the general public.
[x] that the server can be run headless (without a GUI).
[x] I have already attempted to create a configuration myself using the configuration tool at https://config.getamp.sh/

I'm putting a bounty of $25CAD on this module.
If someone gets it running in a functional state I will send to a paypal of your chioce. With how messy setting up rAthena is, it's entirely possible that it's not possible to do in AMP, but I look forward to the progress of people smarter than myself, I could not get it functional.

While I'm sure I could get this running for you, it's worth noting that AMP does not currently support SQL databases. After going through rAthena documentation on the server side setup, one would need to manually setup a SQL server as their documentation states. I will try to see if I can't get it running at some point tomorrow with a manual SQL database setup and if it works, I can do the pull request to include the Generic Template as apart of AMP.


fix pypi step in CI

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

Comparison is base (2c8787d) 88.45% compared to head (507a2c5) 88.45%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #108   +/-   ##
========================================
  Coverage    88.45%   88.45%
========================================
  Files           83       83
  Lines         7131     7131
========================================
  Hits          6308     6308
  Misses         823      823

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


provide tests for $.util and .codec

In this pull request:

added tests for $.util api
added tests for $.util.codec api

Can you please also link the PR to the issue(s) that is solving?


Update node operator documentation once the datanode is in the core repo

Blocked until the following issue is complete:

https://github.com/vegaprotocol/vega/issues/5613

[ ] Update all node operator documentation related to the change in migrating the data-node back into the core repo

[ ] Update links to get the binaries

Whoever picks this issue up may wish to consult @candida-d for further direction on what is required

closing as a duplicate of:

https://github.com/vegaprotocol/documentation/issues/162


Rob Hamilton - Created using Colaboratory

My attempt at the first assignment!

My attempt at the second assignment!


Draft: Catch the "Close normal" error

Closes #44

The exit code could not be received. Should we set another value as exit code?

I remember that we had the exit code discussion previously (as DCP is not capable of retrieving the exit code as well). In that case, we always assumed normal termination (code 0). I would expect that this exit code 0 is suitable in most occurrences of this error (or do you have other information?). So, for the first implementation, I would catch the error, assume normal termination (code 0) and be fine with that.
A possible improvement (that would require a small refactoring in CodeOcean / the API) would be to handle the exit code as an optional attribute and don't show anything to the user if it is missing.

I assume that we won't send anything to Sentry in this case, right?

I assume that we won't send anything to Sentry in this case, right?

Yes, based on a normal termination, I would not have sent anything to Sentry; although it would be exciting to see how often this occurs. Should we rather log it at "warning" level?

I see the interest but would rather not do it (at least not via Sentry). For one, Sentry is more dedicated to error tracing, and (therefore) the pricing model is based on each event. As a consequence, "counting" via Sentry might get expensive and as a project, we have an overall error limit per month (for all projects). Do you have another idea to log / count?

Do you have another idea to log / count?

We already have a "statistics" route. We could create a statistics controller and count things like "How often is which route requested" or "How often is the error code missing". With such a controller we could also provide an prometheus endpoint.
However, as this is a bit of work for currently not that much added value, I would assign it a low priority.

We already have a "statistics" route. We could create a statistics controller and count things like "How often is which route requested" or "How often is the error code missing". With such a controller we could also provide an prometheus endpoint.

I agree. Let's move this discussion into a dedicated issue and add it with a low priority. For these, we might also need to check for persistence/server restarts (depending on the concrete implementation).


RelayNode by Interface

filter node as Relay not by base class, but as IRelay interface marker. That's easiest native way to add any node to CreateNodeMenuWindow
custom command as menu item of CreateNodeMenuWindow by some interface IRelayCommand

If that's idea sound nice for you. I can create push request with that feature

Hello,
Sorry for the long delay. I'm not sure to understand the purpose of a such system, would you mind to explain me in which use case this could be useful?

Hello,
Sorry for the long delay. I'm not sure to understand the purpose of a such system, would you mind to explain me in which use case this could be useful?

So, it can easy create some Relay node but with some logic processing support under data

That's an interesting idea, but I guess it'd be simpler to inherit the Relay node class to create another type of relay that has a custom behavior?

Interface more like contract of data processing. I'm not need to has direct
refs to base class in that case and can extend my own logic.
Right now, im use graph processor only for visualization, but i wanna
change that and move base logic on new "holy land". That's not about that
this topic feature, but have some questions of base relization. For example
in my current logic of graph behaviour i have reactive observable ports
without any usages of reflection. May be your ports logic have some same
behaviour? And have you some disscord or another channel for questions and
disscussions for NodeGraphProcessor?
сб, 22 мая 2021 г. в 16:50, Antoine Lelievre @.***>:

That's an interesting idea, but I guess it'd be simpler to inherit the
Relay node class to create another type of relay that has a custom behavior?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alelievr/NodeGraphProcessor/issues/107#issuecomment-846411300,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABY4TDTZP4PVHIB57AEQ5PLTO6ZDXANCNFSM4UGQI2QA
.

--
Best Regards
Sergey Mikhaylov
Tel: +7 (931) 963-99-56
CTO
email: @.***
linkedin: https://www.linkedin.com/in/mefodei
github: https://github.com/Mefodei

I'd be interested to see the observable port designed without reflection :)
Even though I use reflection to pass the value between ports in the node, it's very fast because I'm using the linq Expression Compile feature of C#.
I just opened a discord server for NGP, you can join here: https://discord.gg/XuMd3Z5Rym


[wallet-kit-site] Rework build to use path aliases

Description
Describe the changes or additions included in this PR.
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Current dependencies on/for this PR:

main

PR #9781   👈

This comment was auto-generated by Graphite.


Usr/jasteph/debugdoc

Adding in VSCode Section to debugging doc

Docs Build status updates of commit 094309f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@Reezaali I know we already made some revisions in a previous word doc, was wondering if you still had that handy?

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

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 136ebe4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 1eded42:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

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

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

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

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 967c0c0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 46cb1b6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 0c27448:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 0c27448:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 7f7af63:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

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

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:bulb:Suggestion

Details

microsoft-edge/webview2/howto/debug.md

Line 103, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 172, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

#sign-off

Docs Build status updates of commit 286ce6b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:bulb:Suggestion

Details

microsoft-edge/webview2/howto/debug.md

Line 103, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.
Line 172, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

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

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:bulb:Suggestion

Details

microsoft-edge/webview2/howto/debug.md

Line 103, Column 1: [Suggestion-code-block-indented] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```). NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

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

File
Status
Preview URL
Details

microsoft-edge/webview2/howto/debug.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

#sign-off

#sign-off and merged.


How to run methods in Before() synchronously?

Haven't been able to find anything that covers this exactly in the issues; but I see lots of people have had trouble with getting things to execute synchronously when using helpers/page objects.
I have a PageObject for an admin area. There are two methods I'm trying to run serially: login and setClient.
When I set up my before, I do:
Before((I, adminPage) => {
  adminPage.login();
  adminPage.setClient();
});

The problem is that they run in parallel. I need to wait for login() to complete before running setClient().
I know I could move all the I.*() calls directly into the Before() but that defeats the purpose of reusability. What is the preferred method to handle this case?

Please show the code of your login, setClient methods, so I could help

@DavertMik sure; sorry about the delay:
login(username, password) {
    // Try to go to login url
    I.amOnPage('/superadmin');

    // Should be redirected to login
    I.seeInCurrentUrl('/login?return=/superadmin');

    I.fillField('username', username);
    I.fillField('password', password);

    I.click('Sign in');
},

selectClient: function* selectClient(client) {
    I.click('Clients');
    I.fillField('ng-model="search.name"', client);
    const clientShort = yield I.grabTextFrom('table td:first span');
    I.click(clientShort);
    I.seeInCurrentUrl('/reports/dashboard');
}

Seems like maybe this related to #206. If I move selectClient() to the test and call it with yield* selectClient() it appears to work.


Improve performance with concurrency/multithreading

Some of the extraction operations can be done in parallel by using async/await multithreading, which could speed up the extraction process.

Image generation is now performed concurrently on multiple threads which dramatically improves the speed of output.
The most significant improvement is in Animated GIF output.


Is there a language detect module on GoogleTranslator on Python?

Is there a language detect module on GoogleTranslator on Python?

No, but there is a general language detection module that you can use. Please check the docs


Explorer: display and/or link smart contract from token page

While on token page, we can access the issuing smart contract
We may get this by default

duplicate with https://github.com/MystenLabs/sui/issues/2126