go package (and others) fail to build due to cpu arch assumption (M1 Mac OSX)

using latest, attempting to build the go package target/debug/cubicle package update go results in
Use 'docker scan' to run Snyk tests against images to find vulnerabilities and learn how to fix them
Copying/extracting seed tarball
15.5KiB 0:00:00 [4.12MiB/s] [================================>] 100%
Checking latest version of Go
./build.sh: 13: go: not found
./build.sh: 14: go: not found
Have  installed at
Downloading go1.19.1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100    75  100    75    0     0    681      0 --:--:-- --:--:-- --:--:--   681
100  141M  100  141M    0     0  80.6M      0  0:00:01  0:00:01 --:--:-- 92.1M
Unpacking go1.19.1
 141MiB 0:00:02 [54.8MiB/s] [============================================================================================================================================================================================================================================>] 100%
qemu-x86_64: Could not open '/lib64/ld-linux-x86-64.so.2': No such file or directory
qemu-x86_64: Could not open '/lib64/ld-linux-x86-64.so.2': No such file or directory
ERROR: Have  installed at , which is still not right
Error: failed to update package: "go"

Caused by:
    0: error building package "go"
    1: failed to run command in environment "package-go"
    2: Non-zero exit status (1) from docker exec

Stack backtrace:
   0: backtrace::backtrace::libunwind::trace
             at /Users/simon/.cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.66/src/backtrace/libunwind.rs:93:5
      backtrace::backtrace::trace_unsynchronized
             at /Users/simon/.cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.66/src/backtrace/mod.rs:66:5
   1: backtrace::backtrace::trace
             at /Users/simon/.cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.66/src/backtrace/mod.rs:53:14
   2: anyhow::backtrace::capture::Backtrace::create
             at /Users/simon/.cargo/registry/src/github.com-1ecc6299db9ec823/anyhow-1.0.61/src/backtrace.rs:216:13
   3: anyhow::backtrace::capture::Backtrace::capture
             at /Users/simon/.cargo/registry/src/github.com-1ecc6299db9ec823/anyhow-1.0.61/src/backtrace.rs:204:17
   4: anyhow::error::<impl core::convert::From<E> for anyhow::Error>::from
             at /Users/simon/.cargo/registry/src/github.com-1ecc6299db9ec823/anyhow-1.0.61/src/error.rs:533:25
   5: <T as core::convert::Into<U>>::into
             at /rustc/4b91a6ea7258a947e59c6522cd5898e7c0a6a88f/library/core/src/convert/mod.rs:550:9
   6: anyhow::kind::Trait::new

poking around in build.sh and in the created docker image, it looks like the build.sh assumes ~/bin is in the path, but it doesn't appear to be. At least from docker exec -i cub-package-go sh from that shell, it lists the path as PATH='/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin'
The PATH issue may be a complete red herring, and the real issue is whatever causes the qemu logging.
I'll continue to poke around.

previous ramblings about path are indeed a red herring.
The base docker VM on a M1 Mac, is an arm image.
simon@Starbuck cubicle % docker exec -i cub-package-go sh
lscpu
Architecture:                    aarch64
CPU op-mode(s):                  64-bit
Byte Order:                      Little Endian
CPU(s):                          4
On-line CPU(s) list:             0-3
Thread(s) per core:              1
Core(s) per socket:              4
Socket(s):                       1
Vendor ID:                       0x00
Model:                           0
Stepping:                        0x0
BogoMIPS:                        48.00
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Not affected
Vulnerability Mmio stale data:   Not affected
Vulnerability Spec store bypass: Vulnerable
Vulnerability Spectre v1:        Mitigation; __user pointer sanitization
Vulnerability Spectre v2:        Not affected
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Not affected

All the packages installs that install an x86/amd64 version are going to fail.
I hacked build.sh for go to install the arm64 version instead, and that all seemed to work fine.

@superfell please try out #38 to see if it fixes this

With #38 I was able to successfully package go & rust, and create new cub's with them. the firefox package still fails.

Thanks. We'll call that progress.
Did you see an arch-specific failure with Firefox? The Firefox package may need X11 to pass its tests.

Firefox is still trying x86
Downloading Firefox 105.0.1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 74.4M  100 74.4M    0     0  12.1M      0  0:00:06  0:00:06 --:--:-- 12.6M
qemu-x86_64: Could not open '/lib64/ld-linux-x86-64.so.2': No such file or directory
Error: failed to update package: "firefox"

Caused by:
    0: error building package "firefox"
    1: failed to run command in environment "package-firefox"
    2: Non-zero exit status (255) from docker exec

vscodium-go fails, not sure what's going on here
simon@Starbuck cubicle % cub package update vscodium-go
Updating "vscodium" package
[+] Building 0.1s (12/12) FINISHED
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 1.80kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/debian:11                                                                                0.0s
 => [1/8] FROM docker.io/library/debian:11                                                                                0.0s
 => CACHED [2/8] RUN echo America/Los_Angeles > /etc/timezone &&     ln -fs '/usr/share/zoneinfo/'America/Los_Angeles /etc/localtime                                                                                0.0s
 => CACHED [3/8] RUN addgroup --gid 20 simon || addgroup simon &&     adduser --disabled-password --gecos '' --uid 501 --ingroup simon simon &&     adduser simon sudo &&     mkdir /home/simon/w &&     chown simon:simon /home/simon/w                                               0.0s
 => CACHED [4/8] RUN sed -i 's/ main$/ main contrib non-free/' /etc/apt/sources.list                                                                                0.0s
 => CACHED [5/8] RUN apt-get update && apt-get upgrade --yes                                                                                0.0s
 => CACHED [6/8] RUN apt-get install --no-install-recommends --yes     apt-file     bash-completion     bind9-dnsutils     build-essential     bzip2     ca-certificates     clang     cmake     curl     dialog     eatmydata     file     gdb     git     iproute2     iputils-ping  0.0s
 => CACHED [7/8] RUN apt-file update                                                                                0.0s
 => CACHED [8/8] RUN sh -c 'echo "Defaults umask = 0027" > /etc/sudoers.d/umask' &&     sh -c 'echo "%sudo ALL=(ALL) CWD=* NOPASSWD: ALL" > /etc/sudoers.d/nopasswd'                                                                                0.0s
 => exporting to image                                                                                0.0s
 => => exporting layers                                                                                0.0s
 => => writing image sha256:73b8a53b3622edbfc33699efc6fdf070e009b3851260f749bf509c8cc031c6aa                                                                                0.0s
 => => naming to docker.io/library/cub-cubicle-base                                                                                0.0s

Use 'docker scan' to run Snyk tests against images to find vulnerabilities and learn how to fix them
Copying/extracting seed tarball
25.0KiB 0:00:00 [6.88MiB/s] [================================>] 100%
Checking latest version of VS Codium
./build.sh: 12: /home/simon/opt/vscodium/bin/codium: not found
Downloading VSCodium-linux-arm64-<IP_ADDRESS>58
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  107M  100  107M    0     0  18.6M      0  0:00:05  0:00:05 --:--:-- 21.3M
Updating "go" package
[+] Building 0.1s (12/12) FINISHED
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 1.80kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/debian:11                                                                                0.0s
 => [1/8] FROM docker.io/library/debian:11                                                                                0.0s
 => CACHED [2/8] RUN echo America/Los_Angeles > /etc/timezone &&     ln -fs '/usr/share/zoneinfo/'America/Los_Angeles /etc/localtime                                                                                0.0s
 => CACHED [3/8] RUN addgroup --gid 20 simon || addgroup simon &&     adduser --disabled-password --gecos '' --uid 501 --ingroup simon simon &&     adduser simon sudo &&     mkdir /home/simon/w &&     chown simon:simon /home/simon/w                                               0.0s
 => CACHED [4/8] RUN sed -i 's/ main$/ main contrib non-free/' /etc/apt/sources.list                                                                                0.0s
 => CACHED [5/8] RUN apt-get update && apt-get upgrade --yes                                                                                0.0s
 => CACHED [6/8] RUN apt-get install --no-install-recommends --yes     apt-file     bash-completion     bind9-dnsutils     build-essential     bzip2     ca-certificates     clang     cmake     curl     dialog     eatmydata     file     gdb     git     iproute2     iputils-ping  0.0s
 => CACHED [7/8] RUN apt-file update                                                                                0.0s
 => CACHED [8/8] RUN sh -c 'echo "Defaults umask = 0027" > /etc/sudoers.d/umask' &&     sh -c 'echo "%sudo ALL=(ALL) CWD=* NOPASSWD: ALL" > /etc/sudoers.d/nopasswd'                                                                                0.0s
 => exporting to image                                                                                0.0s
 => => exporting layers                                                                                0.0s
 => => writing image sha256:73b8a53b3622edbfc33699efc6fdf070e009b3851260f749bf509c8cc031c6aa                                                                                0.0s
 => => naming to docker.io/library/cub-cubicle-base                                                                                0.0s

Use 'docker scan' to run Snyk tests against images to find vulnerabilities and learn how to fix them
Copying/extracting seed tarball
3.15MiB 0:00:00 [46.5MiB/s] [================================>] 100%
No plugins installed
initializing plugin repository...Cloning into '/home/simon/.asdf/repository'...
remote: Enumerating objects: 4357, done.
remote: Counting objects: 100% (184/184), done.
remote: Compressing objects: 100% (90/90), done.
remote: Total 4357 (delta 113), reused 161 (delta 92), pack-reused 4173
Receiving objects: 100% (4357/4357), 992.30 KiB | 8.34 MiB/s, done.
Resolving deltas: 100% (2294/2294), done.
Platform 'linux' supported!
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  109M  100  109M    0     0  79.3M      0  0:00:01  0:00:01 --:--:-- 79.3M
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100    64  100    64    0     0    688      0 --:--:-- --:--:-- --:--:--   688
verifying checksum
/home/simon/.asdf/downloads/golang/1.19.1/archive.tar.gz: OK
checksum verified
Testing "go" package
Error: No such container: cub-test-package-go
[+] Building 0.1s (12/12) FINISHED
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 1.80kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/debian:11                                                                                0.0s
 => [1/8] FROM docker.io/library/debian:11                                                                                0.0s
 => CACHED [2/8] RUN echo America/Los_Angeles > /etc/timezone &&     ln -fs '/usr/share/zoneinfo/'America/Los_Angeles /etc/localtime                                                                                0.0s
 => CACHED [3/8] RUN addgroup --gid 20 simon || addgroup simon &&     adduser --disabled-password --gecos '' --uid 501 --ingroup simon simon &&     adduser simon sudo &&     mkdir /home/simon/w &&     chown simon:simon /home/simon/w                                               0.0s
 => CACHED [4/8] RUN sed -i 's/ main$/ main contrib non-free/' /etc/apt/sources.list                                                                                0.0s
 => CACHED [5/8] RUN apt-get update && apt-get upgrade --yes                                                                                0.0s
 => CACHED [6/8] RUN apt-get install --no-install-recommends --yes     apt-file     bash-completion     bind9-dnsutils     build-essential     bzip2     ca-certificates     clang     cmake     curl     dialog     eatmydata     file     gdb     git     iproute2     iputils-ping  0.0s
 => CACHED [7/8] RUN apt-file update                                                                                0.0s
 => CACHED [8/8] RUN sh -c 'echo "Defaults umask = 0027" > /etc/sudoers.d/umask' &&     sh -c 'echo "%sudo ALL=(ALL) CWD=* NOPASSWD: ALL" > /etc/sudoers.d/nopasswd'                                                                                0.0s
 => exporting to image                                                                                0.0s
 => => exporting layers                                                                                0.0s
 => => writing image sha256:73b8a53b3622edbfc33699efc6fdf070e009b3851260f749bf509c8cc031c6aa                                                                                0.0s
 => => naming to docker.io/library/cub-cubicle-base                                                                                0.0s

Use 'docker scan' to run Snyk tests against images to find vulnerabilities and learn how to fix them
Copying/extracting seed tarball
 334MiB 0:00:04 [69.2MiB/s] [================================>] 100%
go version go1.19.1 linux/arm64
Updating "go-install.github.com/go-delve/delve/cmd/dlv" package
[+] Building 0.1s (12/12) FINISHED
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 1.80kB                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for docker.io/library/debian:11                                                                                0.0s
 => [1/8] FROM docker.io/library/debian:11                                                                                0.0s
 => CACHED [2/8] RUN echo America/Los_Angeles > /etc/timezone &&     ln -fs '/usr/share/zoneinfo/'America/Los_Angeles /etc/localtime                                                                                0.0s
 => CACHED [3/8] RUN addgroup --gid 20 simon || addgroup simon &&     adduser --disabled-password --gecos '' --uid 501 --ingroup simon simon &&     adduser simon sudo &&     mkdir /home/simon/w &&     chown simon:simon /home/simon/w                                               0.0s
 => CACHED [4/8] RUN sed -i 's/ main$/ main contrib non-free/' /etc/apt/sources.list                                                                                0.0s
 => CACHED [5/8] RUN apt-get update && apt-get upgrade --yes                                                                                0.0s
 => CACHED [6/8] RUN apt-get install --no-install-recommends --yes     apt-file     bash-completion     bind9-dnsutils     build-essential     bzip2     ca-certificates     clang     cmake     curl     dialog     eatmydata     file     gdb     git     iproute2     iputils-ping  0.0s
 => CACHED [7/8] RUN apt-file update                                                                                0.0s
 => CACHED [8/8] RUN sh -c 'echo "Defaults umask = 0027" > /etc/sudoers.d/umask' &&     sh -c 'echo "%sudo ALL=(ALL) CWD=* NOPASSWD: ALL" > /etc/sudoers.d/nopasswd'                                                                                0.0s
 => exporting to image                                                                                0.0s
 => => exporting layers                                                                                0.0s
 => => writing image sha256:73b8a53b3622edbfc33699efc6fdf070e009b3851260f749bf509c8cc031c6aa                                                                                0.0s
 => => naming to docker.io/library/cub-cubicle-base                                                                                0.0s

Use 'docker scan' to run Snyk tests against images to find vulnerabilities and learn how to fix them
docker: Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: sethostname: invalid argument: unknown.
Error: failed to update package: "go-install.github.com/go-delve/delve/cmd/dlv"

Caused by:
    0: error building package "go-install.github.com/go-delve/delve/cmd/dlv"
    1: failed to create environment "package-go-install-github.com/go-delve/delve/cmd/dlv"
    2: failed to start Docker container "cub-package-go-install-github.com/go-delve/delve/cmd/dlv" (encoded for Docker as "cub-package-go-install-github.2ecom.2fgo-delve.2fdelve.2fcmd.2fdlv")
    3: Non-zero exit status (125) from docker run

Yeah, for Firefox, its not likely to work even if it gets further. Same for vscode. Perhaps need to set arch specific filters for packages?

Firefox is still trying x86

Oh I see, Firefox's "Linux 64-bit" download really means x86-64, and they don't offer official ARM64 Linux builds. https://firefox-source-docs.mozilla.org/build/buildsystem/supported-configurations.html#tier-3-targets
So, that's not great.

vscodium-go fails, not sure what's going on here
[...]
docker: Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: sethostname: invalid argument: unknown.
Error: failed to update package: "go-install.github.com/go-delve/delve/cmd/dlv"

This might be caused by the hostname being too long.
~$ docker run -it --rm --hostname package-go-install-github-com-go-delve-delve-cmd-dlv.fractal debian
root@package-go-install-github-com-go-delve-delve-cmd-dlv:/#
exit
~$ docker run -it --rm --hostname package-go-install-github-com-go-delve-delve-cmd-dlv.morecharactershere debian
docker: Error response from daemon: OCI runtime create failed: container_linux.go:367: starting container process caused: process_linux.go:495: container init caused: sethostname: invalid argument: unknown.
Command exited with status 125

My machine is named fractal (after the brand of its case), and yours must be named something longer :joy:. So I guess we need to truncate long hostnames?

#44 / #48 just now should have fixed the long hostname problem


Event publishing by the TES runner

Problem.
The TES server cannot access the status of Task processing except through the Batch APIs. This causes delays in completing tasks. The Batch APIs often mark Job/Tasks as completed much later than their actual completion time. Moreover, querying the Batch API for state leads to throttling issues and more delays when scaling up
Solution:
The TES runner controls the execution of all the steps in the Batch node. It can generate events that describe the execution and send them to a sink/destination for consumption.
AC:

Events must be published to a storage account
- This will mitigate the need for additional infrastructure to run TES on Azure and Terra.
- Must faciliate manual and programmatic discovery/consumption of the messages.

Must use the following path-pattern to publish the events:

/tes-internal/events/{event-name}/YYYY/MM/DD/{timestamp}{workflow_id}{task_id}.json

Must set tags in each blob for: {task_id}, {workflow_id}, {event_name} and {created}.

Created tag must use HTTP GET friendly format, e.g. ISO 8601

Must create events for the following actions: download start/end, execution start/end, upload start/end and completion.

Download and upload events must include number of files and total bytes transferred.
Completion event must indicate success or failure.

Events must follow the same schema,  and enable correlation to the tasks and workflows.

@giventocode can we link this issue to the TES runner https://github.com/microsoft/ga4gh-tes/issues/148? Or should this be separate from the runner implementation?


Add "Tokens" section to Explorer, similar to Etherscan and other explorers

Is your feature request related to a problem? Please describe.
It's challenging to reference the current assets available on Stacks, easily from one place. It would be helpful if it was possible to reference the currently deployed assets/tokens, similarly to how you can on other Blockchain explorers.
Describe the solution you'd like
Add a link to the top navigation bar, "Tokens" that enables to you track tokens on Stacks, similarly to how you can on Etherscan: https://etherscan.io/tokens
Another example from the Celo network: https://explorer.celo.org/tokens
Describe alternatives you've considered
A community managed Google sheet might be a stop-gap to consider in the interim, so that people know what all assets are deployed on Stacks, and where to interact with them (e.g. Arkadiko, ALEX, StackSwap, etc.)
Additional context
Moon.

Thanks for the suggestion. I'll close this one in favor of https://github.com/hirosystems/explorer/issues/1088


New Local and Remote Views

I always wanted to see the exact serialization of local and remote objects, of course nicely rendered in abapGit. I have enhance the "Diff page" to now be able to view just the local or just the remote side of files. It's a bit of a bigger change, but it works quite well... and I have added some other features as you can see below.

New items in Repo > Advanced memu "Show Local" and "Show Remote"

These will display all files included in the local or remote repositories (Note: If you do that with abapGit, it will take some time to render since the repo is so large!)

If files are shown in Repo, then the item state icons on the right side become links to view the local or remote version of this file

The new local and remote views also include statistics for number of lines, sloc, and size (same as in Github)

At the top also the totals are displayed

Also includes minor correction to map blank "last changed by" to "unknown"

Since you can display a complete repo in these new abapGit views, you can quickly search for anything using Ctrl+F, or cut&paste any code easily. And it's all there without having to launch any ABAP tools.

If you like to give it a try, use https://github.com/mbtools/abapGit/tree/local-remote-view and turn on experimental features. in your settings.
Let me know what you think!
PS: It's worth repeating since I know many will try to view the complete abapGit repo this way... It will take some time to display. The AG repo is ~85k sloc and the generated HTML page becomes huge (32MB due to syntax highlighting). My GUI went into "not responding" for about 2 min, but it did eventually show everything correct and navigating on the page was seamless.

Hi,
first of all, thanks for taking the time to make these extensive contributions!
My notes:

The item state icons are great, if a file is new or deleted I don't really need to view it all green or all red.
Totals are great to know. I have a personal gripe with sloc though_. "_Source lines of code" sounds like a case of the RAS_syndrome. I'd go with 20 lines (15 code) but you're trying to match github and they use sloc so this is just personal preference.
The last changed by improvement is nice, different values for "unknown" clutter the filters
Long freezes present very negative user experience, people are just not willing to look at a blank screen for minutes and hope it works out in the end. You're not warned and you cannot cancel.  By the way ctrl+f also gets very laggy on big pages. I know the technology severely limits what can be done about it, but I have a few suggestions:

Show a warning if repository contains >x objects. Not sure if you're able to get the diff size before rendering but that would maybe be an even better metric.
Github does not render large diffs by default for this exact reason. We now have the option to collapse a diff - maybe if large diffs were rendered collapsed this would help reduce the freeze.

I use AG for all projects in my new business and love it. So contributing is self-serving to some degree like I guess for many here.

The red/green icons are the default for diffs (not my invention). I just added links to them. What's new is the = icon in case there's no diff. We might want to change the style of the icons to be not so bright.
Agree. "lines of code" is better
Agree, but it was v1 of the change ;-). I'm a fan of software that adapts to user demands automatically. No warning, just simplify, still deliver the important parts, inform the user what was skipped. I will try to add a toggle for the syntax highlight. "on" by default, switch automatically to "off" if number of objects becomes to high. I might as well do a perf trace to see what can be optimized.

I think your screenshots are for https://github.com/larshp/abapGit/pull/3388 ;-)

It's both really - I had a screen freeze handy after #3388 so I recorded the comparison of rendering collapsed and expanded :)

I made some major improvements: https://github.com/mbtools/abapGit/tree/local-remote-view

Offline repos now support view with files (not just objects)
Local/Remote views now works on folder, object, and file level
Syntax highlight turned off for > 20 files
Error when trying to view > 50 files
New view menu to change view settings (TODO: toggle for syntax highlight does not work yet)

Enjoy!


Current project status

Hi, what is the current status of this project? Is it still maintained?

Agree. Will probably do that.


Can I delete the 4 PR?

Do you still need my RC5 PR or can I delete them?

I should check that the PR contains updated information respect to docs develop branch.

These PR was the "up to date" docs for RC5 (and also updated later for 1.2.0).

Closed


Gateway Groups

Is your feature request related to existing pfSense functionality that is missing from the API? Please describe.
A URL or documentation about the requested functionality. Ex. /firewall_rule.php [...]
/system_gateways.php
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Would like to be able to change gateway tiers within gateway groups
Describe the solution you'd like
A clear and concise description of what you want to happen.
Be able to change / add / remove gateway groups
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
I haven't tried any
Additional context
Add any other context or screenshots about the feature request here.

Duplicate of #189.


Adding Server core OS versions under supported OS list

Updated supported OS section for Windows Server core OS versions (Windows Server 2012 R2 Server Core and Windows Server 2016 Server core. Also added cross reference hyper link for easy navigation https://docs.microsoft.com/en-us/azure/security/azure-security-disk-encryption-tsg#troubleshooting-windows-server-2016-server-core

@dikadali : Thanks for your contribution! The author, @mestew, has been notified to review your proposed change.

@dikadali Just double checking about 2012 R2 Core since I'm not 100% sure on that one.

HI @mestew, Thanks for response.  I've tested it, It works with 2012 R2 Core as well.

@dikadali I've confirmed 2012 R2 Core is supported by the ADE team as well.
Can you tweak your PR to the below for me? I want you to get collaboration credit on this doc but need it to be changed a tiny bit before I can get it merged.

Windows Server 2012 R2 Core and Windows Server 2016 Core are supported by Azure Disk Encryption once the bdehdcfg component is installed on the VM.

I used our preferred notation for the link: azure-security-disk-encryption-tsg.md#troubleshooting-windows-server-2016-server-core

@dikadali : Thanks for your contribution! The author, @mestew, has been notified to review your proposed change.

@mestew - Thanks. I've updated the pull request with your proposed change. , It looks more simple!

Close/reopen for fresh validation post-migration.

@msmbaldwin

Because of the merge conflict, I applied this update to the article in another PR.  The revised list of supported Windows Server editions is live: https://docs.microsoft.com/en-us/azure/security/azure-security-disk-encryption-prerequisites#bkmk_OSs
Thanks for the feedback, @dikadali! #please-close


enable_proof_check in a text_ctrl on Windows 11 doesn't do anything

I created a text control with the following code:
    @text_ctrl = Wx::TextCtrl.new(self, Wx::ID_ANY, '', Wx::DEFAULT_POSITION,
      Wx::DEFAULT_SIZE, Wx::TE_RICH2)
    @text_ctrl.enable_proof_check(Wx::TextProofOptions.default)

There are no complaints from Ruby, but when I enter in "tast" and a space, no spell checking is done.
By contrast, if I create the following C++ code:
    text_ctrl = new wxTextCtrl(this, wxID_ANY, wxEmptyString,
        wxDefaultPosition, wxDefaultSize, wxTE_RICH2);
    text_ctrl->EnableProofCheck(wxTextProofOptions::Default());

This works as expected -- I get the red squiqly underline under "tast" when I press space after entering in the misspelled word.
This is running on Windows 11.

I should also mention that while not documented, grammar_check appears to exist, or at least I can call it without any complaints from Ruby @textCtrl.enable_proof_check(Wx::TextProofOptions.default.grammar_check). However, like enable_proof_check it doesn't seem to have any effect (but does in equivalent C++ code).

I am somewhat surprised you report no errors as it turned out the methods and class TextProofOptions were not wrapped AT ALL yet.
But no matter, they have been added now  on the master branch through PR #152.
Things to notice though:

the only thing supported on all platforms (WXMSW, WXGTK and WXOSX) is spell checking;
the WXMSW implementation incorrectly returns false when disabling spell checking for wxWidgets < 3.3 (I submitted a PR for 3.3+);
no platform currently supports setting the language through Wx::TextProofOptions (the language setting is a dud);
ONLY WXOSX currently supports grammar checking

You say enabling grammar check had an effect so you either were working on OSX or you noticed an other effect ;-)


Cannot build the project

Hi!
I'm probably doing something wrong but no amount of Googling helped so I hope you might have an idea about what's going on.
When running "cmake --build --preset debug" the build fails with the following error:
[3/3] Running Swift build...
FAILED: tests/CMakeFiles/test_app tests/x86_64-unknown-windows-msvc/Debug/test_app.exe C:/windowsapps/swift-winrt/build/debug/tests/CMakeFiles/test_app C:/windowsapps/swift-winrt/build/debug/tests/x86_64-unknown-windows-msvc/Debug/test_app.exe
cmd.exe /C "cd /D C:\windowsapps\swift-winrt\tests && swift build -Xswiftc -debug-info-format=codeview -Xlinker -debug -j 4 -Xswiftc -disable-round-trip-debug-types -c debug --scratch-path C:/windowsapps/swift-winrt/build/debug/tests"
error: 'tests': Invalid manifest (compiled with: ["C:\\Library\\Developer\\Toolchains\\unknown-Asserts-development.xctoolchain\\usr\\bin\\swiftc.exe", "-L", "C:\\Library\\Developer\\Toolchains\\unknown-Asserts-development.xctoolchain\\usr\\lib\\swift\\pm\\ManifestAPI", "-lPackageDescription", "-sdk", "C:\\Library\\Developer\\Platforms\\Windows.platform\\Developer\\SDKs\\Windows.sdk", "-libc", "MD", "-I", "C:\\Library\\Developer\\Platforms\\Windows.platform\\Developer\\Library\\XCTest-development\\usr\\lib\\swift\\windows", "-I", "C:\\Library\\Developer\\Platforms\\Windows.platform\\Developer\\Library\\XCTest-development\\usr\\lib\\swift\\windows\\x86_64", "-L", "C:\\Library\\Developer\\Platforms\\Windows.platform\\Developer\\Library\\XCTest-development\\usr\\lib\\swift\\windows\\x86_64", "-use-ld=lld", "-swift-version", "5", "-I", "C:\\Library\\Developer\\Toolchains\\unknown-Asserts-development.xctoolchain\\usr\\lib\\swift\\pm\\ManifestAPI", "-package-description-version", "5.7.0", "C:\\Users\\STEFAN~1\\AppData\\Local\\Temp\\TemporaryDirectory.RB7uJs\\manifest.swift", "-Xfrontend", "-disable-implicit-concurrency-module-import", "-Xfrontend", "-disable-implicit-string-processing-module-import", "-o", "C:\\Users\\STEFAN~1\\AppData\\Local\\Temp\\TemporaryDirectory.JwEtcH\\tests-manifest.exe"])
error: fatalError
error: link command failed with exit code 1 (use -v to see invocation)
lld-link: error: <root>: undefined symbol: mainCRTStartup
lld-link: error: undefined symbol: memcpy
>>> referenced by C:\Users\STEFAN~1\AppData\Local\Temp\TemporaryDirectory.fqpdJu\manifest-1.o:($s18PackageDescription6TargetC10DependencyOWOb)
clang: error: linker command failed with exit code 1 (use -v to see invocation)
ninja: build stopped: subcommand failed.

I have Swift 5.9.1 installed and I followed the instructions in the windows-samples repository to get my environment ready. (That project doesn't build as well)

Got it working using the releases from the swift-build repository (https://github.com/thebrowsercompany/swift-build)


Enabling event bus delivery options for communication between Server …

I hereby agree to the terms of the Knot.x Contributor License Agreement.

CR fixes.


Support function panel in Query Builder

Summary
Fixes https://github.com/finos/legend-studio/issues/819

How did you test this change?

[ ] Test(s) added
[x] Manual testing (please provide screenshots/recordings)
[ ] No testing (please provide an explanation)

https://user-images.githubusercontent.com/73408381/155628216-350c26a4-87de-4ab6-9969-17134ccd5840.mov

@beekemarie Discussed with Beeke.
Consider:

Hide the parameter panel in the default page of Query Builder too
Choose a meaningful color for the second header(functional band) as blue is already used by service registration


docker build action

Added github actions for docker image builds https://github.com/mckaywrigley/chatbot-ui/issues/92

Is there anything I need to do on my end to get this to work?

@mckaywrigley  You should be able to push docker images to ghcr.io automatically after merging into the main branch without setting any tokens or other secrets.
I have confirmed this in my personal repository below.
https://github.com/tatsu-i/chatbot-ui/pkgs/container/chatbot-ui

Does secrets.GITHUB_TOKEN expose anything from my account, or how does that work?

GITHUB_TOKEN is a special access token that can be used to authenticate GitHub Actions on behalf of GitHub; GitHub automatically creates a GITHUB_TOKEN secret that can be used for authentication during workflow execution.
Github Actions will automatically generate the secrets temporarily without the need to configure Secrets, and GITHUB_TOKEN will disappear after the github actions execution is complete.
Below is my secrets settings screen, showing that the workflow is successful even though no token is set.

Wonderful work!


[Suggestion] Default font-weight 300 -> 400

Adjust font-weight to 400.

I bet the design choice was made on OS X where fonts render chunky and the default font isn't Segoe and that's fine but with every deployment I need to manually edit Cockpit's CSS!

I created a userstyle for this, but you are right, this must be dealt from upstream.


DOC:enhancement of feedback form

What's wrong with the existing documentation
enhance the ui and fields of feedback form
Add ScreenShots
mail and username should be separate and colour combination should also proper
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'23 contributor
[X] I want to work on this issue

Okay change the UI matching the taste of signin, signup and contact form.

there is no action on this issue from a while , please assign this to me


method with return type Task<IObservable> is not recogniced as Streaming method

im having a gateway that analyses the request and based on the parameters forwards the request to some backend service.
im using the signalr client in my gateway to achive that proxy behavior. what i dont like is that i have to
hubConnection.StreamAsync<TResult>("Method").Result;
because if i await the call and change my hubmethod to return a task then signalr does not recognice it as streaming method.

Just return IObservable<T> from your hub

how can i do that? if i use your signalr client to subscribe to a stream it is always async!
https://github.com/aspnet/SignalR/blob/00da533f109a0f82cf2d9d8af26d0f39666bf0a4/src/Microsoft.AspNetCore.SignalR.Client.Core/HubConnection.cs#L194

@JamesNK can you take a look?
@JanEggers You're invoking StreamAsync from the client right?

@davidfowl i have a js client that uses this:
https://github.com/aspnet/SignalR/blob/846432c9ac4599cbd24ceac6e756627d3f1859c7/client-ts/signalr/src/HubConnection.ts#L163
that connects to a gateway server with a hub. that gateway hub uses
https://github.com/aspnet/SignalR/blob/00da533f109a0f82cf2d9d8af26d0f39666bf0a4/src/Microsoft.AspNetCore.SignalR.Client.Core/HubConnection.cs#L194
to forward the request to the backend.
and that backend hub returns an IObservable<T>

I used signalr client in the server because i have to call different backends depending on parameters otherwise i would have used some websocket proxy

Great. We have a bug here that we need to fix. Does it throw when you have a Task<IObservable<T>> or Task<ChannelReader<T>> saying that the invocation isn't streaming?
As a workaround in the meantime you could do something like this (if using RX):
public IObservable<int> Proxy()
{
     var subject = new Subject<int>();
     _ =StreamAsync(hubConnection, subject);
     return subject;
}

private async Task StreamAsync(HubConnection conn, Subject<int> subject)
{
    var reader = await conn.StreamAsync("SomeBackendMethod");
    while (reader.WaitToReadAsync())
    {
        while(reader.TryRead(out var item))
        {
              subject.OnNext(item);
        }
    }
    subject.OnCompleted();
}

This basically adapts the channel to the observable.

i get this:
The client attempted to invoke the non-streaming '{0}' method in a streaming fashion.

thx @JamesNK for fixing that up

@davidfowl I am migrating my project which uses Microsoft.AspNetCore.SignalR version 1.0.0-preview2-final to 1.0.0-rc1-final.
And I run into this issue or similar issue, the exception is:
A non-streaming method was invoked with a streaming invocation : StreamInvocation.
backend (in the old version this used to work)
public class HeroHub : Hub<IHeroHub> {
        public IObservable<Hero> GetUpdates(string id) {
           //  returns an observable
         }
}

And my second questions which relates to this, in the interface how shall I create the method signature for the stream method?
I've tried the following but i had no luck:
IObservable<Hero> GetUpdates(string id);
Task<IObservable<Hero>> GetUpdates(string id);
Task GetUpdates(string id);
Task<ChannelReader<Hero>> GetUpdates(string id);

For all the above it was throwing this error:
System.InvalidOperationException: Cannot generate proxy implementation for 'Heroes.Contracts.IHeroHub.GetUpdates'. All client proxy methods must return 'System.Threading.Tasks.Task'.

I m also using System.Reactive 3.1.1. and also tried to upgrade to the prelease v4.0.0-preview.2.build.379

@claylaut The error is indicating a failure in generating the implementation for IHeroHub. IHeroHub in your example should define what methods exist on the client and Streaming is not supported from the client to the server right now. Can you show the definition of IHeroHub?

@davidfowl  removed support for observables. have a look at the samples you have to use channelreader now.
https://github.com/aspnet/SignalR/blob/9cb683a41d759e45e3227ca4b2d57d819cc97380/samples/SignalRSamples/Hubs/Streaming.cs#L14-L20

It still isn't really clear to me. You defined IHero but I don't see that type used in the example.
When you use Hub<T>, the type you use for the T parameter defines the structure of the Client methods. Client methods can't return values or stream results in 2.1, only Server methods can. So, the interface you use in Hub<T> can only contain methods that return Task. This doesn't restrict the methods you can put on the Hub itself, those can be streaming. As @JanEggers mentioned, we dropped support for IObservable due to lack of support for back-pressure, so you need to use ChannelReader<T> or Task<ChannelReader<T>>.


Add numerical identifiers for each check

I've been thinking that if we had idnetifiers for each check, it could serve a good purpose in communicating both among ourselves during development (i.e.: there's an issue with check number 37) as well as when talking to type designers (i.e.: I'm getting some error messages related to check 15, can you provide me some guidance on how to fix the issue, please?)
I know it is not strictly necessary, but I think it could be worthwhile. Also, there's the precedent that some other lint tools do just that as well. So we're not reinventing the wheel here.
@davelab6 and @m4rc1e What do you think?

oh! And it could also aid in readability on the text terminal output. I have had this idea for a long time, but I have just expressed it here now because I just felt again the need for it while scrolling a few pages of verbose log. If there were check numbers it would be easier to fing the specific check I'm interested in debugging.

I agree this would be nice someday

done

The check id numbers are saved on the check-results log and will be available on the database. We can later use that new database field for displaying the check id numbers to the users on the webapp.
The numbers are already visible on the command line if you use the -vv flag (VERBOSE level = 2)


بوت عربي

بوت عربي
Originally posted by @gsyyjvuxtsgud in https://github.com/Tocsiop/R8HEX/pull/2

بوت عربي
Originally posted by @gsyyjvuxtsgud in https://github.com/Tocsiop/R8HEX/pull/2


Add data validation to required-tasks data

Including pipeline-required-tasks.
Ref: EC-323

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

Project coverage is 100.00%. Comparing base (1c21485) to head (1619084).

Additional details and impacted files

@@            Coverage Diff            @@
##              main     #1158   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          117       117
  Lines         6053      6086   +33
=========================================
+ Hits          6053      6086   +33

Files with missing lines
Coverage Δ

policy/release/tasks.rego
100.00% <100.00%> (ø)

policy/release/tasks_test.rego
100.00% <100.00%> (ø)


[Feat]Favorites products: Scroll Animation for Your Preferred Picks

Description
n addition to the scrolling animation for the Favorite Products Page, I have considered the following alternative solutions and features to enhance user experience:
Grid Layout with Hover Effects:
Instead of a scrolling animation, implementing a grid layout for favorite products could allow users to view multiple products at once. Each product could feature hover effects that display additional information, such as pricing and quick view options, enhancing interactivity without requiring scrolling.
Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[ ] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Hey @govindbairi, this is a very good feature, but it does not look good on our website. According to our website's design, a simpler effect and feature will look more attractive. So, as of now, I don’t think this issue is necessary for our website. Please work on any other issue that is needed and raise issues accordingly. Thank you!


(Bug) ganache-cli throws "Out of gas" even with big --gasLimit

When we run ganache-cli with --gasLimit 0xfffffffffff cli parameter and then run truffle unit tests, one of the big contracts doesn't want to be deployed because of "Out of gas" exception.
The gas needed to deploy that contract is ~6700000.
The gas option which is set in truffle.js is 7000000 (we also tried 8000000, 9000000, 10000000).
We also tried to set --gasLimit to 12000000.
So, it seems that the gasLimit parameter for ganache-cli doesn't work properly.
Steps to Reproduce

Clone or download this repo tree.

Perform the next commands:

$ npm i
$ npm run test

You will see the error. It happens on this line of unit tests. Note that --gasLimit is set to a big value 0xfffffffffff and the gas in truffle.js is set to 7000000.

You can also run the next commands in separate consoles to see the output of ganache-cli:

Console 1:
$ node_modules/.bin/ganache-cli --gasLimit 0xfffffffffff --port 8544

Console 2:
$ node_modules/.bin/truffle test --network test

Context
Please help us to figure out with this issue ASAP. We can't write and use unit tests for our contracts until this issue is solved.
Environment

Version of ganache-cli used: 6.3.0
Version of Truffle used: 5.0.5
NodeJS Version: 8.11.4
Operating System and version: MacOS 10.13.6
Link to your project or repro gist: https://github.com/poanetwork/posdao-contracts
Commit hash to use with above link for reproduction: https://github.com/poanetwork/posdao-contracts/tree/7a29ace38807b6a544fa65e086de5e5180afbf47

Seems the --allowUnlimitedContractSize solves the issue. The contract size exceeds 24 Kb. Sorry for false assumption. Closing the issue.

Going to reopen this issue because I think there is an opportunity for us to be more clear as to what is actually happening here in our error messages. @varasev, thanks for taking the time to open it and follow up with the solution!

Just to say I've been struggling today with the same problem for a while until I discovered this issue and @varasev's solution. I'd definitely encourage this being made clearer in the error messaging somewhere!


Spelling/Grammar suggestions and changes

Consistently capitalized Tremor
A few minor spelling and spacing corrections

@ChrisHirsch thank you!

Thanks Chris! <3

Awesome Chris!
On Tue, 8 Sep 2020, 21:40 Heinz N. Gies<EMAIL_ADDRESS>wrote:

Thanks Chris! <3
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/wayfair-tremor/tremor-www-docs/pull/22#issuecomment-689093886,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABX5MVWEEKKW4X3X52EPCTSE2CDLANCNFSM4RAKNT2Q
.


Updates in prep for pydata-sphinx-theme update

This PR should allow for the pydata-sphinx-theme update in https://github.com/inrupt/solid-client-access-grants-js/pull/320.
👉  Let me know if I should apply directly to that patch to that PR.  This won't build because it needs that updated version.
Basically:

Explicitly mention our css file.
Include html_context for light/dark/auto mode.
Explicitly mention favicon.

Note: The docs-assets layout have been updated to address the generate_nav_html warning.

[x] Commits in this PR are minimal and have descriptive commit messages.

Closing as will apply the patch to the version update PR instead.


is this face detection ML Kit plugin able to detect only real  faces?

If I focus my Identity card also, it detects as a face. Is there any possibility to avoid that or this plugin detects any kind of faces whether they are real or not ?

This plugins detects all faces in a given image. Ultimately the photo of you id card is an image as well and it detects the faces in it.
Maybe what you need is something like FaceId that uses the TrueDepth camera and machine learning for a secure authentication solution. Which is outside of the scope of this plugin.
Try something like this:
https://github.com/SelectCode/FlutterTrueDepthCamera


CASMTRIAGE-3173 for release/1.2

Summary and Scope
Docs change for the operations/boot_orchestration/Edit_the_iPXE_Embedded_Boot_Script.md and install/pxe_boot_troubleshooting.md. same as PR #1552 but for reelase/1.2
Is this change backwards incompatible, backwards compatible, or a backwards compatible bugfix?
Issues and Related PRs
List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

Resolves [issue id](issue link)
Change will also be needed in <insert branch name here>
Future work required by [issue id](issue link)
Documentation changes required in [issue id](issue link)
Merge with/before/after <insert PR URL here>

Testing
List the environments in which these changes were tested.
Tested on:

<development system>
Local development environment
Virtual Shasta

Test description:
How were the changes tested and success verified? If schema changes were part of this change, how were those handled in your upgrade/downgrade testing?

Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)?
Were continuous integration tests run? If not, why?
Was upgrade tested? If not, why?
Was downgrade tested? If not, why?
Were new tests (or test issues/Jiras) created for this change?

Risks and Mitigations
Are there known issues with these changes? Any other special considerations?
Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[ ] License file intact
[ ] Target branch correct
[ ] CHANGELOG.md updated
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

/backport release/1.0 main

This should be reverted.


Make intro launch CLI tutorial

Fix #2186.
Here is a basic tutorial that has the user run a ROS 2 launch command. It has the user run the launch command and see what happens.
Note that I explicitly didn't include this tutorial in the ROS 2 launch tutorial's main page, because I think it doesn't read well. Specifically, the CLI tutorial section references tutorials before and after it, so it seems out of place when put with the other launch tutorials.

@Mergifyio backport foxy galactic


Retrieving to many basis elements for homology

The following program should return no 1-dim basis elements of homology. It works for cohomology.
public static void main(String[] args) {
    int prime=2;
    int d = 2;

    ExplicitSimplexStream stream = new ExplicitSimplexStream();
    stream.addVertex(0);
    stream.addVertex(1);
    stream.addVertex(2);
    int[][] edges = {{0,1},{1,2},{0,2}};
    for(int i = 0; i != edges.length; ++i){
        stream.addElement(edges[i]);
    }
    stream.addElement(new int[]{0,1,2});
    stream.finalizeStream();

    AbstractPersistenceBasisAlgorithm<Simplex, int[]> simplex_annotated_algorithm = new IntAbsoluteHomology(ModularIntField.getInstance(prime), Collections.reverseOrder(SimplexComparator.getInstance()), 0, d+1);
    AnnotatedBarcodeCollection<Double, int[]> annotated_intervals = simplex_annotated_algorithm.computeAnnotatedIntervals(stream);
    System.out.println("Homology basis for " + d + "-ball:");
    System.out.println(annotated_intervals);
}

Output:
Homology basis for 2-ball:
Dimension: 0
[0.0, infinity): [2]
Dimension: 1
[0.0, infinity): [1,2] + [0,2] + [0,1]

Why are you reversing the simplex comparator order? I'd expect this to compute some sort of relative homology the way it's setup. (see http://iopscience.iop.org/article/10.1088/0266-5611/27/12/124003/meta for why)

No reaction after feedback. Closing.


Migrate to dedicated Chinese conversion library

ICU is a decent starting point for Chinese conversion (or at least, more decent than for other scripts), especially since it’s accessible from so many languages. But ICU isn’t primarily designed for transliteration to display to the user; its constraints like reversibility make it better suited to use cases like collation and speech synthesis. Ideally, you’d use a dedicated Chinese conversion library that can handle more nuances between Chinese variants.
OpenCC is a popular Chinese conversion library that, as far as I know, has a good reputation. It has official frontends in multiple languages, though none in Java. There are several unofficial Java ports, but none seem to be quite as popular or well maintained as the original OpenCC:

android-opencc
jopencc
OpenCC-Java
opencc4j

Resolved by adding in opencc4j which seems to work.


Revert "Start waiting for processing by default."

This reverts commit b661ef1697d555e1f2ab5c42d3c60b5bb71b6cf2.
It looks like this change caused issues with analyses on pull requests from forks. This is due to the complicated nature of the permissions of workflows run on these pull requests. They are able to upload analyses for their own pull requests refs, but are not able to access other Code Scanning APIs such as the one required to get the status of a delivery.
This reverts adding this new functionality until we update the API to make it able to safely determine whether a pull request should be able to get the status of a particular delivery.
This is a hotfix on the v1 branch.
Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.
[x] Confirm the changelog has been updated if necessary.

Could we add a changelog note mentioning this revert? We mentioned the original change in the log, so I think the revert should be too.

Sure! I've put this under a 1.0.29 release with today's date on. Let me know if this is not the correct way of doing a change note for a hotfix though.

Sure! I've put this under a 1.0.29 release with today's date on. Let me know if this is not the correct way of doing a change note for a hotfix though.

Oh I've just realized this PR is targeting v1. I think it should probably target main and the note should go in the "unreleased" heading. Then our automation can handle doing a release with the fix in and do the right things with regards to bumping the version number in all the right places.

Ah, okay that makes sense. I was a bit worried that would make it more difficult to do a hotfix because it would pull in other unrelated changes, but it actually looks like there aren't any, so that should work fine.

Ah, okay that makes sense. I was a bit worried that would make it more difficult to do a hotfix because it would pull in other unrelated changes, but it actually looks like there aren't any, so that should work fine.

Yeah, if the day ever comes that we have to do a hotfix directly on v1 it will be a slight pain because we'll need to manually bump the version number in a few places then do a manual mergeback to main. No need for that here though, much easier to do the change on main and let the automation handle releasing.

:+1:


Too much lane changing in the highway.

the car bellow changes lane like crazy! (DLS =15%):
https://cdn.discordapp.com/attachments/545066674605391882/868601182044696586/Cities_Skylines_2021.07.25-00.03_Trim.mp4
this has bee a common occurrence for me for as long as I remember.

Also noticed this after starting playing CSL again recently.
It seems also exacerbated by bends in the highway - I had one van chaning lanes almost every bend in this route through the mountains:

Maybe segment speed could be factored in to lane change decisions? Faster speed = fewer lane changes? Also, cargo vehicles could be less likely to change lanes?

@aubergine10 this happens on the way when the highway is going straight out of city. They just lane change like crazy!
I think instead of reducing Dynamic lane changing percentage I want to be able to reduce its threshold.
in any case it would be nice to see you in discord.


address #391

"Address" because I don't want GitHub to close it :-)

Nice changes, definitely makes sense to have these in for 1.0.0. Thanks mate!


Learning To Simulate: unable to create animation

I was trying to recreate the 'learning to simulate example', and when I finished training the model and try to plot the trajectory using
python -m learning_to_simulate.render_rollout \     --rollout_path=/tmp/rollouts/WaterRamps/rollout_test_0.pkl
The error message was
Figure(1000x500) /usr/local/lib/python3.7/site-packages/matplotlib/animation.py:888: UserWarning: Animation was deleted without rendering anything. This is most likely not intended. To prevent deletion, assign the Animation to a variable, e.g.anim, that exists until you have outputted the Animation using plt.show()oranim.save().
What should I do to plot the animation?

Do other regular matplotlib animations work for you (for example some animations from here), or is this specific to our own code?
If the matplotlib examples have the same issue, then it is probably something specific to you matplotlib installation.

I also encountered this problem. But I solved it, I add " unused_animation.save('animation.gif') " behind
"  unused_animation = animation.FuncAnimation(
fig, update,
frames=np.arange(0, num_steps, FLAGS.step_stride), interval=10)"
so the final code :


⬆<EMAIL_ADDRESS>

Bumps sinon from 1.17.4 to 9.0.2

Closing in favour of https://github.com/atom/atom/pull/21195


Update Readme.

Readme can be updated with details of mentors,admins,slack channels and details of gssoc. Please assign me this issue.

@Sapna2001 sure. You can go ahead with this.

Can you add gssoc and difficulty label for this issue.


Example script dataset not available

The example script dataset linked from the website:
https://demianw.github.io/tract_querier/example_script.html#example-script
at the following location:
https://demianw.github.io/tract_querier/_static/wmql_example_dataset.zip
Shows a 404 "Page not found error". Can the dataset be made available again, please?

@jhlegarreta I have a copy of it, how do you suggest we make it available ?

OSF, zenodo or even better G-Node GIN.


[Tests] Added Python to Travis

Opening pull request for testing on travis!
Will update here when it actually runs the python tests correctly.

It's running the python tests now.
Just have to fix dependencies from python and should be good to go. (selenium giving errors when installing)

Selenum is installed now, and python tests are running, but failing. Have to figure out why.

Ooo, maybe we need to set the size of the output window?
https://stackoverflow.com/a/46299332

@carlschroedl Already got it setup for testing locally :)
Will look into the link you attached later today, and sorry for spamming with Commits ;)

Trying to get the test working with travis addon for chrome instead. They also say we have to start with  --no-sandbox arg.
https://docs.travis-ci.com/user/chrome

It now runs python tests.
I have removed the dependency on selenium by making a laravel seeder for creating some default users to use in the python tests.

All working now, python tests are running. Sorry for the many commits.

Thanks Oliver! I admire the way you tackled these problems. It was very smart to seed the database. No worries on the many commits -- that's what it looks like on pretty much every project I've worked on when the project sets up Travis for the first time. Thanks again!


Help

Are there plans to add accessories to the room API？thank you

I think I implemented all features. if there is problem please open another issue.

@nazrdogan this is a great react-native library, thanks! I'm curious why the readme doesn't list all the APIs? You really have to dig through the code to find everything. Otherwise everything is working well for me over here.


fix: semantic-release peer dependency range was invalid syntax

Merging this PR will publish the following release:

3.1.2 (2018-02-11)
Bug Fixes

semantic-release peer dependency range was invalid syntax (a7d8134)


PB - 05 Mar 2019

Pull Request Test Coverage Report for Build 93

24 of 29 (82.76%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 85.313%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/chess/core/createNotation.js
7
9
77.78%

src/chess/core/internal/_applyDoubleStep.js
11
14
78.57%

Totals

Change from base Build 92:
-0.1%

Covered Lines:
276

Relevant Lines:
308

💛  - Coveralls


Improve performance and add ability to sleep

After looking more into #40 it seemed that there was some potential to improve this library more substantially than by just removing some of the I2C delays.
This adds support to put the PN532 into a low power state (which reduces the current draw by up to ~60mA from my testing). It also better handles waking up the PN532 from this low power state, which resulted in not needing many of the delays that were present in all of the communication subclasses.
The responsiveness of the PN532 is much better with these changes. I tested this with both an itsybitsy m4 express and an itsybitsy m0 express for all 3 communication methods (I2C, SPI, & UART) on the Adafruit PN532 Shield but would welcome if someone else could also verify these changes!

@nitz Do you have a PN532? I know you've been doing other NFC stuff.

Uh, you'll have to let me check when I get home. I definitely bought something for that side of the NFC equation, but it's in a box at the house and I don't remember if I've even taken it out 😂! Poke me if I don't update here in a few hours!

Okay, I'm back, with news!
 I was also able to test with a handful of different NTAG215s I have, and that went peachy too.
However, switching to UART, the initialization went fine and it was able to read the FW Version from the PN532, but as soon as I'd wave any tag near it, it'd freeze. I'd like to have more info on where or why, but I don't have that yet (and I'm starving, so I gotta go eat dinner.)
My test board is this nRF52840 kit. I'm not using the TX/RX pins on the board there, as I'm pretty sure they're default bound to something else, and I didn't feel like shaving that particular yak.
I also ran into this very bizarre thing, where I had my FTDI TTL to USB 232 attached to those TX/RX pins, and while the board was completely unpowered, the LED on the PN532 board was lit, as long as I had TX connected. But only while the PN532 was in I2C mode. Yeah, I've got no idea for that, but seems to be a hardware thing, and not a code thing.
Anyways: after I get something to eat I'll see if I can figure out -- woah, okay. Literally as I'm typing this I'm staring at the frozen output and it spits out the UID of the card I waved in front of it at least a few minutes ago, and it's running again. Waving another in front freezes it, and after a minute or two it spits the UID and keeps going. So I guess it's not a hard lock.
Stay tuned!

@nitz I might know why you saw that long delay.
What timeout value are you using when you initialize the uart object? For the old examples I think the values were never updated. So the example code sets the timeout=100, which for the newer CP that is 100 seconds, not milliseconds.
Try setting timeout=0.1 and see if that works better. I put this value in correctly for the new example I added, pn532_low_power.py, but I didn't change it in the old ones as I figured it wasn't really a part of this pull request.
Let me know how that goes!

Well that's a bingo! Yeah I just copied and pasted the old ones into my code.py, mega timeout and all!
Everything else seems solid to me, including all three methods in your pn532_low_power.py. I don't have a Pi set up with blinka, but I imagine the addition of  a reset/req won't hurt 🙂


Publish recent versions to crates.io

Why does is the cargo-ebuild published on crates.io still stuck at version 0.3.1 ? I've just discovered by luck that development hadn't stopped 2 years ago. The readme still says to install using cargo install cargo-ebuild.

@vincentdephily original cargo-ebuild author retired from gentoo, it's now maintained by me and several very good contributors.
we don't have crates.io access, we need @cardoe to somehow transfer it to me or take it down so we can replace it, idk how it works.
original issue is here: https://github.com/gentoo/cargo-ebuild/issues/14
let's use it for future discussion.


[Rules] Rule documentation

similar to ansible-lint (https://ansible-lint.readthedocs.io/rules/)

implemented.
https://github.com/ansible/ansible-risk-insight/blob/main/docs/customize_rules.md


IntelliJ/WebStorm IDE props auto-completion broken in 4.0.0 alpha

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔

In IntelliJ/Webstorm IDE, when importing component and using it in JSX, prop auto-completion/suggestions and validating should work as it did in previous version 3.9.x.
Current Behavior 😯

No props are suggested or validated.
Steps to Reproduce 🕹

import any component from core/component
use it in jsx
try to have IDE suggest props

Context 🔦

If I downgrade material-ui/core back to 3.9 version, everything works, but not on 4.0.0-alpha.0.
Your Environment 🌎

Tech
Version

Material-UI
v4.0.0-alpha.0

IDE
IntelliJ IDEA  2019.1 EAP

Solution?/Thoughts`
I noticed that if I add "typings": "./index.d.ts", line to for example core\Button\package.json, it works again just like before.
Another thing is that if I try to navigate to the source of the imported components, IDE navigates to the source in the core/esm/Component folder instead of core/Component even though I imported from core/Component. Possibly because of "module": "../esm/Chip/index.js", in the Component's package.json. But this is also fixed with the above typings addition.
(Possibly related to issue #14392 )

Another thing is that if I try to navigate to the source of the imported components, IDE navigates to the source in the core/esm/Component

Does it still do that with the added typings field?

Another thing is that if I try to navigate to the source of the imported components, IDE navigates to the source in the core/esm/Component

Does it still do that with the added typings field?

No, it will navigate to the core/component/index.d.ts where I think it is supposed to and where it was going in the previous versions.
I noticed you added the typescript and while they indeed are typescript typing files, I'm not using typescript in the project, it's just that the IDE uses them for auto-completion.

note that I'm not using typescript in the project, it's just that the IDE uses them for auto-completion.

I think this is the only actionable fix for us though. Autocompletion is working fine in codesandbox (and probably for any vscode version). It's just that webstorm is not up to date.

@ssypi This was released with<EMAIL_ADDRESS>Could you please verify if this is working for you?

Thank you, I will let you know by monday, I’m currently on a vacation trip.

For some reason, autocompletion for Button and ButtonBase components doesn't seem to work, but everything else I tried works perfectly such as Typography, Icon, IconButton, Step, StepButton, Divider etc. Button and ButtonBase seem to have slightly different d.ts files.
declare const ButtonBase: ExtendButtonBase<ButtonBaseTypeMap>;
vs
declare const IconButton: React.ComponentType<IconButtonProps>;
Other than that, this is very good.

4.11.0 still doesn't have autocomplete for  like it did in 3.9.2. I also tried 5.0.0-alpha.12 and that does not work.

I am having this issue right now. I am on the latest webstorm version. Tried downgrading to the oldest version Jetbrains Toolbox allowed me to (some 2020 version), but unfortunately had the same issue


time algo not working in Safari mode

even after 10 min the log still says just now

@kakshay21 Probably issue with timeago.js library which I am using. Thanks for reporting.

very clean UI by the way! I love it!!
I want to build something similar with vue js. My app is already pwa ready and it's configured with firebase. would you help me in UI?

@kakshay21 Hey, Thanks. You can take the components and related CSS in the repo. Try running it locally to get a local demo of it.

Yeah, I did on my local as the link isn't working now. Thanks!
Would you mind if I contact you, when I fall in any problem with codes?

Okay I take that as no.
The code is very modular and well placed. Are components standalone? Will it work with just template? Cause I want to use in my existing Vuejs project.
@gokulkrishh

@kakshay21 No code is not modular. I have written some  specific codes.  Check out the code you will know more.


Helm Dep List is returning "missing" for deps on packaged charts

Output of helm version:
$ helm version --client
Client: &version.Version{SemVer:"v2.10.0-rc.2", GitCommit:"56154102a2f25ebf679c791907fd355bb0377f05", GitTreeState:"dirty"}

Output of kubectl version:
N/A -- this is client only
Cloud Provider/Platform (AKS, GKE, Minikube etc.):  N/A
Fetch the wordpress chart: helm fetch stable/wordpress
$ helm dep list ./wordpress-2.1.5.tgz
NAME   	VERSION	REPOSITORY                                       	STATUS
mariadb	4.x.x  	https://kubernetes-charts.storage.googleapis.com/	missing

Status is missing. But the dependency is in the tarball. Unpack the chart, and this is evident:
$ tar -zxvf wordpress-2.1.5.tgz
x wordpress/Chart.yaml
x wordpress/values.yaml
x wordpress/templates/NOTES.txt
x wordpress/templates/_helpers.tpl
x wordpress/templates/deployment.yaml
x wordpress/templates/externaldb-secrets.yaml
x wordpress/templates/ingress.yaml
x wordpress/templates/pvc.yaml
x wordpress/templates/secrets.yaml
x wordpress/templates/svc.yaml
x wordpress/templates/tests/test-mariadb-connection.yaml
x wordpress/templates/tls-secrets.yaml
x wordpress/.helmignore
x wordpress/README.md
x wordpress/requirements.lock
x wordpress/requirements.yaml
x wordpress/charts/mariadb/Chart.yaml
x wordpress/charts/mariadb/values.yaml
x wordpress/charts/mariadb/templates/NOTES.txt
x wordpress/charts/mariadb/templates/_helpers.tpl
x wordpress/charts/mariadb/templates/initialization-configmap.yaml
x wordpress/charts/mariadb/templates/master-configmap.yaml
x wordpress/charts/mariadb/templates/master-statefulset.yaml
x wordpress/charts/mariadb/templates/master-svc.yaml
x wordpress/charts/mariadb/templates/secrets.yaml
x wordpress/charts/mariadb/templates/slave-configmap.yaml
x wordpress/charts/mariadb/templates/slave-statefulset.yaml
x wordpress/charts/mariadb/templates/slave-svc.yaml
x wordpress/charts/mariadb/templates/test-runner.yaml
x wordpress/charts/mariadb/templates/tests.yaml
x wordpress/charts/mariadb/.helmignore
x wordpress/charts/mariadb/OWNERS
x wordpress/charts/mariadb/README.md
x wordpress/charts/mariadb/files/docker-entrypoint-initdb.d/README.md

$ helm dep list wordpress
NAME   	VERSION	REPOSITORY                                       	STATUS
mariadb	4.x.x  	https://kubernetes-charts.storage.googleapis.com/	unpacked

For those of us who plan to consume the output of helm dep list programmatically, what is the expected behaviour here - can we still assume there will be a STATUS field, or is that likely to be omitted on packaged charts?

Still an issue in 3.1. It would be nice to update the documentation  (this part: This can take chart archives and chart directories as input. It will not alter the contents of a chart.) and mention, that in case of archives "STATUS" is always "missing".


Fix an incorrect autocorrect for Capybara/RedundantWithinFind when escape required css selector

Fix: https://github.com/rubocop/rubocop-capybara/issues/136
Previously, the following automatic corrections were made.
before:
within find_by_id("array-form-session.dates") do
  expect(page).to have_text(:visible, "YYYY/MM/DD")
end

after:
within "#array-form-session.dates" do
  expect(page).to have_text(:visible, "YYYY/MM/DD")
end

This is . in find_by_id has the same meaning as the escaped id. In other words, when replacing within, the .  must be escaped.
This PR has been modified so that escaping is correctly done in selectors that require escaping. This will prevent the behavior from changing before and after the automatic correction.

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with main (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[-] Updated documentation.
[x] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

@hatsu38 Does this change solve your issue?

@ydah
Thank you for addressing this issue.
I tested this branch by running bundle install.
It seems that RuboCop now properly escapes the ..
within find_by_id("array-form-session.dates") do ~ end

-> match

within "array-form-session\.dates" do ~ end

-> match

both within "#array-form-session.dates" and within "#array-form-session\.dates" appear to be looking for
.
So, As mentioned in the ISSUE, the behavior changes as a result of running rubocop -A.
Therefore, I believe my issue has not yet been resolved.


SAK-49463 Kernel improved SQL performance for counting role in realm for Oracle

Thanks for the review @ern


Fix comparison of binary series with a binary

This prevents the wrong data type to be compared when Polars is executing the query against the lazy frame.
Strings and binaries can be treated almost equally in Elixir, but in Rust (or in Polars) this is not the case. So we need to explicitly say the data type.
Closes #532

@josevalim I think this covers most of the problem, since we don't have the issue with numbers (polars solves that internally). WDYT?


sql: EXTRACT_DURATION built-in function for INTERVAL

PR: https://github.com/cockroachdb/cockroach/pull/9734

Closed by https://github.com/cockroachdb/docs/pull/948


Add LolAlytics ARAM Item Sets

LolAlytics has much more detailed itemsets than what LolFlavor provides. It would be really nice to be able to have more information on what items are good to build for a certain champion in ARAM.

Dup. https://github.com/dustinblackman/Championify/issues/233


Should utils.injectHTML evaluate injected scripts?

Currently it does not, because it uses .innerHTML in the browser, which doesn't evaluate injected scripts.
We could potentially change to use document.write, which does evaluate injected scripts.
Use of document.write is discouraged, but it might be OK in our use case where we are intentionally clearing the page.
More details:

https://developer.mozilla.org/en-US/docs/Web/API/Document/write
https://web.dev/no-document-write/
https://developer.chrome.com/blog/removing-document-write/

We discussed this during a call and are leaning towards using document.write by default, with an option to disable inline scripts execution.


I can send messages but I can't receive them

I am on Xchat - Ubuntu - gnome.
I could connect successfully.
I could send messages but I didn't get any messages from any the rooms (and I saw updates in the web ui)
Ideas?

Well, now it is working... the only change I made was shift to port 6667 instead of 6697... not sure if could be the cause...


Add sorting_tests macro to most sorting algorithms

There were a few algorithms I wasn't able to do this for, because the function signatures didn't match those expected by the macro. Perhaps it'd be a good idea to standardize those too?

There were a few algorithms I wasn't able to do this for, because the function signatures didn't match those expected by the macro. Perhaps it'd be a good idea to standardize those too?

Yes, that was my first thought. In the end they are all sorting algos, so it probably makes sense to just create some sort of abstraction over the algorithms and call a .sort. I'm fine with adding something that couples the algos a bit since that abstraction would be educational as well. Similar to how search problems (bfs, dfs, A*, etc) can be abstracted as well.
I'm happy to review those changes in separate PRs!

Ah, there is a small formatting issue! @aliencdh would you mind fixing those so that I can merge this PR?

@alexfertel ofc, i’ll format them now


[Style] Request to change the style

What's the style idea?
We need to improve the style of how the website shows the slides and make it more interactive. Also want to give a search symbol beside every nav bar. Also want to introduce dark themes. Please can you assign me this issue
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a KWOC'24 contributor.

Code of Conduct

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

Please provide a more clear picture of your idea with supported screenshots.

What kind of interactions do you want to implement?
Search symbol beside navbars, clarify.
The website already has a dark theme, check and lmk.


Executable Will Not Launch if SteamVR is Already Running

Trying to figure out how to actually use this program.
If I start steamVR then try to run the VRCSkeletalHands.exe, nothing happens.
Run as admin, nothing happens.
If I run the program before launching steamVR, I get the GUI and controllers not connected prompt.
Start steamVR and turn on controllers, everything tracks in the program fine.
Put on the headset and am told steam is disabled and no bindings can be found.
Essentially breaks all means of setting bindings (so it can't be used in any game since no buttons are mapped anymore)
If I try to start both at the same time it crashes steamVR.

Hi, can i just ask a few questions about how you set it up?

When extracting the zip did you move any files around?
Did you extract all files from the zip?
What OS are you using?
If you get any error messages or crash messages what are they?


Pyglet Pure Python Backend doesn't work with python 3.12+ due to distutils removal

https://github.com/pthom/imgui_bundle/blob/be8339b41ae36e3cd0de372f151ef9590ef44eee/bindings/imgui_bundle/python_backends/pyglet_backend.py#L224-L236
This function still uses the deprecated distutils module's LooseVersion. It appears the recommended fix is to use packaging.version.parse.
https://packaging.pypa.io/en/latest/version.html

So I've looked into how Distutils looseversion works, and it's quite simple. It just converts the version into a list (converting what it can to ints) and then does piecewise comparisons.
I'll make a small PR removing distutils when I can. Do you know any other places distutils might be used? I'll do a full cleanup.

Hello, and thank you for your answer!

I'll make a small PR removing distutils when I can.

Great news

Do you know any other places distutils might be used? I'll do a full cleanup.

I think this is the only place it is used.

@pthom is it possible to release a patch version with this update?


[libra framework][move-prover] Refactor Roles.move and clients

[libra framework][move-prover] Refactor Roles.move and clients
This is a refactor of Roles.move and modules that use it to make
formal verification of access control policies less difficult, by
basing permissions on role_ids of addresses, instead of Privilege and
Capability objects.
The previous approach of publishing Privilege objects and
extacting capabilities from them created several problems for the Move
Prover. The most fundamental of these is that the Move Prover cannot
prove a property unless it sees all of the code that could invalidate
that property. As currently designed, the available code is the module
containing the property and all modules in its dependencies.
The advantage of this approach is that properties that are so proved
will continue to hold even in an open blockchain with modules and
transactions that we do not trust.
More pragmatically, there were a number of other problems, ranging
from tricky verification problems that we haven't yet solved to
limitations of the verifier that we don't have time to fix.  The
refactor should eliminate the need to prove many properties and will
make others much simpler to prove.
Before the refactor, a Move function that is restricted to certain signers
has an argument of type &Capability, which can only be generated
if there is a signer of the transaction with a published Privilege.
The refactor replaces that &Capability argument with an &Signer
argument, and calls assert(has_...privilege(account), )
to abort the function if the signer does not have the required
permissions.  As it happens, has..._privilege is always a property of
the role of the address as defined by the RoleId in Roles.move, so we
don't actually need to have or publish Privilege objects.  To
achieve obvious, visible correspondence to our spreadsheet of
"governance roles and privileges", these functions have names
corresponding to privileges from the spreadsheet.  Several identical
functions have the same body, because some roles have multiple
privileges. (In the future, perhaps we can auto-generate verification
properties from something like that spreadsheet.)
This approach retains the flexibility of changing the addresses that
have a given role, because the RoleId implementation is unchanged.
The correspondence between roles and privileges can also be changed
easily by changing the appropriate has_..._privilege functions. Right
now, they all just check for a particular role ID, but there would be
no problem changing them to allow multiple alternative roles, or to
check for the presence of a published resource of some type, or lots
of other policies.
[libra framework][move-prover] Refactor Roles.move and clients
This is a refactor of Roles.move and modules that use it to make
formal verification of access control policies less difficult, by
basing permissions on role_ids of addresses, instead of Privilege and
Capability objects.
The previous approach of publishing Privilege objects and
extracting capabilities from them created several problems for the Move
Prover. The most fundamental of these is that the Move Prover cannot
prove a property unless it sees all of the code that could invalidate
that property. As currently designed, the available code is the module
containing the property and all modules in its dependencies.
The advantage of this approach is that properties that are so proved
will continue to hold even in an open blockchain with modules and
transactions that we do not trust.
More pragmatically, there were a number of other problems, ranging
from tricky verification problems that we haven't yet solved to
limitations of the verifier that we don't have time to fix.  The
refactor should eliminate the need to prove many properties and will
make others much simpler to prove.
Before the refactor, a Move function that is restricted to certain signers
has an argument of type &Capability, which can only be generated
if there is a signer of the transaction with a published Privilege.
The refactor replaces that &Capability argument with an &Signer
argument, and calls assert(has_...privilege(account), )
to abort the function if the signer does not have the required
permissions.  As it happens, has..._privilege is always a property of
the role of the address as defined by the RoleId in Roles.move, so we
don't actually need to have or publish Privilege objects.  To
achieve obvious, visible correspondence to our spreadsheet of
"governance roles and privileges", these functions have names
corresponding to privileges from the spreadsheet.  Several identical
functions have the same body, because some roles have multiple
privileges. (In the future, perhaps we can auto-generate verification
properties from something like that spreadsheet.)
This approach retains the flexibility of changing the addresses that
have a given role, because the RoleId implementation is unchanged.
The correspondence between roles and privileges can also be changed
easily by changing the appropriate has_..._privilege functions. Right
now, they all just check for a particular role ID, but there would be
no problem changing them to allow multiple alternative roles, or to
check for the presence of a published resource of some type, or lots
of other policies.

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

:umbrella: The latest upstream changes (presumably 84f74bd931fff919d2dcf80c156e798918fe473c) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably c27716be3a8c2f9b6386bfe6f9737f883b408173) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r=sblackshear

:pushpin: Commit e6e8ba9 has been approved by sblackshear

:lock: Merge conflict

@bors-libra r=sblackshear

:pushpin: Commit ef30103 has been approved by sblackshear

:hourglass: Testing commit ef30103d8725bc7ffb28483ba11dc6355a6682f0 with merge b37d494bae8e2103fe4a6463509c7566548732fb...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: sblackshear
Pushing b37d494bae8e2103fe4a6463509c7566548732fb to master...


Allow users to explicitly specify path to preferred Chrome browser.

Primary credit to the original author of #242.

Nice, thanks! Just for my understanding, what does this solve that can't be done by adding a PATH env var to the cron job invocation?

@mrooney my understanding is that the user could be already using the binary executable in the PATH; but by using this command-line option, you get the flexibility to choose another executable separate from your primary workspace.

@mrooney my understanding is that the user could be already using the binary executable in the PATH; but by using this command-line option, you get the flexibility to choose another executable separate from your primary workspace.

Yeah, I wasn't thinking of the user's PATH, but that you can specify PATH in the crontab in a couple different ways (at the top or as an env var for a specific task). Unless there's some reason that doesn't work that I'm missing, I'd prefer we just document how to use crontab better than add in extra code that duplicates a crontab feature. But please let me know if I'm missing/understanding something :)

@mrooney honestly, I have zero preference. I simply took a stale PR and tried to get it moving, which seemed to be just to add content to the README. See the referenced PR above. Whatever y'all want to do is fine with me.

@mrooney honestly, I have zero preference. I simply took a stale PR and tried to get it moving, which seemed to be just to add content to the README. See the referenced PR above. Whatever y'all want to do is fine with me.

Curious if @jaron-l has any preferences, but if this can be solved by simple prepending the command in cron or elsewhere with a PATH var, that seems better to leverage native OS features than add code here. But I don't have a strong opinion and don't mind being outvoted if this flag seems like a helpful idea. It just seems like it will need to be documented either way, so we could just as easily document using PATH. 🤷

I can definitely see both sides here as well. I understand avoiding feature bloat that already has a workaround that could be just documented. One argument for this PR would be that selenium provides a way to do it in it's API and we are hiding that ability.
I hate to be that guy that complicates the discussion... but it almost feels like we could just make it easier by documenting how to start your own webdriver and pass that into mintapi to be used. Then any supported selenium API would be usable from the python API and would also be more future proof. For example, we would refactor _sign_in from get_web_driver and document how the user can pass in a set up driver and bypass _create_web_driver_at_mint_com. I could work on this PR request if the idea is hard to imagine.

@jaron-l that's definitely a very intriguing idea and sounds like it is the best long-term solution, at least based on what I can tell from your description. I'd be fine with going that route or at least seeing how that'd look.

@jaron-l that's definitely a very intriguing idea and sounds like it is the best long-term solution, at least based on what I can tell from your description. I'd be fine with going that route or at least seeing how that'd look.

Take a look at #336. That's my initial draft of what I'm talking about. I documented how to use it in the README.md but you could use that same idea for any seleniumrequest driver.

I'm going to close this in favor of #336 as it seems we are in agreement about going that route.


Br 135 only self as owner

Implements #135
Description: Only the current logged in user can be selected as owner (only available option)
PR checklist

[x] dev-branch has been merged into local branch to resolve conflicts
[x] tests and linter have passed AFTER local merge
[x] localization is supported (Guide)
[ ] another dev reviewed and approved
[x] if feature-Branch: Teams PO has approved (show via e.g. screenshots/screencapture/live demo)

Screenshot

well done


[WIP] wasm32 backend

I've thrown together an example harness to be able to test it (based on druid's) and i've made smol compile by effectively commenting out all the code with cfg.
Will eventually close #87.
I'll start actually implementing it now, but I thought it would be good to open a PR for tracking since a few people seem interested in contributing.

The last commit sketches a possible structure out. I've created a new 'WebExecutor' and I've replaced the innards of Task with a PhantomData on wasm, and removed everything but the ability to forget it.
I'm not sure how I feel about this. I don't personally use the task handles, so I'm not sure if this is likely to be problematic for some users.

Thank you for starting work on this!
Quick question: how are we going to implement timers and task spawning? What crate does provide that? (sorry, not super familiar with web)
We can implement task cancelation by using something like https://docs.rs/futures/0.3.5/futures/future/fn.abortable.html

Hiya,
So timers are pretty easy, they can just be the equivalent of window.setTimeout, task spawning is... more complicated...

I discovered the wasm environment is not as single threaded as previously thought (though not yet as multithreaded as one would hope)
I had some difficulties attempting to just delegate the work to the browser APIs. The error messages from wasm-bindgen are often less than spectacular so i'm not entirely sure why.

I had another play a few hours ago and my gut feeling is that we should actually do something similar to the rest of smol, except probably in the first instance it should be limited to the thread local executor because we can't really get the full benefit of multithreading. I actually tried to just reuse your thread local executor, but it depends on io_event, which looks not so easily disentangled (for a start, it wants Async...).
So the current avenue I'm looking at involves async-task and essentially replicating the local executor, without the dependence on the IO stuff.
I have no idea what I'm doing :)

Oh, and it's web-sys that has all the relevant browser apis, but the moment you have to actually pass data across the boundary it gets a bit tedious.

Oh, one more thing I should mention, I have been taking inspiration (although not too much of it) from wasm-bindgen-futures.

Thank you for elaborating! I am currently working on untangling the reactor from the executor so this will soon become much easier.

Maybe we can use WebWorker for multithreading tasks?

Hey @jjl whats the status on this?

@kwek20 I haven't done anything on it in a while, nor am I likely to in the rest of this year.
i expect the status of the wasm backend makes some of this a bit easier now than it did when i started, if you were interested in picking up where i left off. Although given smol has changed so much since then, not sure how much of it you'd want to salvage.

Closing this PR as it is far out of date with the current master and would likely require a significant rework. Thanks anyways @jjl!
See #87 for WASM progress on specific subcrates


Logins not working...

When I simply type "/login," the Server say "Login done as Local Moderator, and when I type "/login [password]," it returns "Invalid command..." Any help?
Edit: the same problem occurs using "/mass" among others as well

fixed with pr


Replication of results from paper

Hi @chamorajg, thanks for the uploading pytorch implementation of the 'Unsupervised Monocular Depth Learning in Dynamic Scenes' paper. Were you able to reach the numbers mentioned in the paper for Kitti/Ciyscapes?

I am currently working on a few videos from KITTI dataset and making some minor fixes to the loss functions. Once completed, I shall also upload the pretrained models on KITTI dataset.

Thanks for the quick reply. Looking forward to the result.


Bug Table 1223 "Data Exch. Column Def" - Overflow

The length of the string is 56, but it must be less than or equal to 50 characters. Value: Nett_weight_per_unit_measure_unit_specifier_in_kilograms
Internal session id:
f79ec577-f475-4d07-92de-89b08598814a
Application Insights session id:
ff25740e-5569-46f2-84e3-e1fb7aee535f
Client activity id:
ab9c493f-eb70-934f-adea-36cb4fc41bc4
Timestamp:
2022-04-29T09:36:42.9261749Z
AL call stack:
"XSD Parser"(CodeUnit 9610).CreateDataExchColumnDefinitions line 34 - Base Application by Microsoft
"XSD Parser"(CodeUnit 9610).CreateDataExchDefForCAMT line 19 - Base Application by Microsoft
"XML Schema Viewer"(Page 9610)."GenerateDataExchSetup - OnAction"(Trigger) line 4 - Base Application by Microsoft
In procedure CreateDataExchColumnDefinitions procedure InsertRecForImport will be called.
DataExchColumnDef.Description = Text[100]. Procedure InsertRecForImport allows a value of Text[50].
 DataExchColumnDef.InsertRecForImport(DataExchDef.Code, DataExchLineDef.Code,
                      ColumnNo, CopyStr(ElementName, 1, MaxStrLen(DataExchColumnDef.Name)),
                      CopyStr("Node Name", 1, MaxStrLen(DataExchColumnDef.Description)), true,

    field(8; Description; Text[100])
    {
        Caption = 'Description';
    }

Solution:
table 1223 "Data Exch. Column Def"
NewDescription has to be Text[100] instead of NewDescription: Text[50];
//Start
//procedure InsertRecForImport(DataExchDefCode: Code[20]; DataExchLineDefCode: Code[20]; ColumnNo: Integer; NewName: Text[250]; NewDescription: Text[50]; NewShow: Boolean; DataType: Option; DataFormat: Text[100]; DataFormattingCulture: Text[10])
procedure InsertRecForImport(DataExchDefCode: Code[20]; DataExchLineDefCode: Code[20]; ColumnNo: Integer; NewName: Text[250]; NewDescription: Text[100]; NewShow: Boolean; DataType: Option; DataFormat: Text[100]; DataFormattingCulture: Text[10])
//Stop

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 42835.


Regression: exported AssignmentExpressions are treated as declarations

See the following code:
// jshint esnext: true
let x = 1;
export default -x;

This passes in version 2.8.0, but on master, JSHint issues the following warning:
{Line 3, Char 17} 'x' has already been declared.

Fix incoming.

Resolved via gh-2648


Feature Request: Add India in Amazon Cloud Player country selection

Currently there is no option to select India as a national variant in amazon cloud player. Amazon India used a .in domain  for its service https://music.amazon.in/ If you can add the support for it that would be great.

Thanks for the pull request. I added tested it and added a changelog entry.

New flatpaks were published - Amazon Cloud Player 5.9.5.