EN Content: Signature component

Summary: Documentation of component guidance in EN for the Signature component on the Design System website.
Including use cases, how to use guidance, accessibility guidelines and more.
Acceptance Criteria:

[x] Component Kickoff
[x] Component review
[x] Sync with Dev and Design
[x] Research and consulting other design libraries
[x] Component ID (name, definition, example image
[x] Use case (when to use guidance & related components)
[x] How to use guidance
[x] Accessibility guidelines
[x] Inclusive language
[x] Descriptive text for images
[ ] Translation & Localization
[ ] Bilingual review – content design
[ ] Ensure consistency: ordering, terminology, high level voice/tone/style
[ ] Final review

First draft of Signature component content for review
https://docs.google.com/document/d/1M_wtvVaaYjE175OY18hi0hVRriHynm9p-gR5wrBY8sg/edit

Had to verify a few things with Nick, consulted the FIP design standard and asked Sarah for review. @nchoquet file's now ready for translation.

Translation is ready @nmakuch and @SmartMouthWords
https://docs.google.com/document/d/1qF2c1-17aejxDZRr6mHigXtAK3pMEsMyOaZDW7I-2fU/edit?usp=sharing

👀 Bilingual content review underway
EN: https://docs.google.com/document/d/1M_wtvVaaYjE175OY18hi0hVRriHynm9p-gR5wrBY8sg/edit
FR: https://docs.google.com/document/d/1qF2c1-17aejxDZRr6mHigXtAK3pMEsMyOaZDW7I-2fU/edit

✅ Final bilingual content review complete

Confirming final section content "Apply the wordmark type"
Missing Visuals for French Anatomy

🚀 Ready for dev

@ethanWallace @melaniebmn @ClementineHahn

@nmakuch to source the picture then to publish on the website

added french images to the docs file
https://docs.google.com/document/d/1M_wtvVaaYjE175OY18hi0hVRriHynm9p-gR5wrBY8sg/edit?pli=1

More work was needed on the images of Canada.ca , should be done by end of sprint


add a fitBounds functionality for the current Object

a rethinking of the application state is needed. Will add a new state part for mapping and do the featureCollection Calculation inside a dedicated reducer


[Bug] Pagination Error

Describe the bug
Pagination works fine as long as the pages queried have data.
As soon as the page has no data, then this error will occur.

error getting data frame. unable to merge fields due to different fields
(Trace ID: 763b0b20f3281df1fac45df608e94594)

Version Details:

Grafana version : v10.3.0-63079
Plugin version : 2.1.0

Additional context

Other related Nice to haves

5 page max is very small. The given argument for the limitation:

The limitation to the number of maximum pages per query is limited to 5. This is to avoid resource exhaustion and DDOSing of the target system.
I do not see how valid this is as you can achieve the same by duplicating the Query and changing the offset values manually. I got to 102 queries before I gave up. Then all is needed is to merge the queries into one when transforming data.
If there must be a limit, 100-500 seems fine. Especially when using List of Values Pagination type where the field value could take a Custom Variable in CSV.

Would be great if Max pages supported variables for a dynamic value. Most APIs will have an endpoint to give a total count, which can be used to calculate numbers of pages to query.

I am facing the same error:
Grafana version : 10.2.0-60982
Plugin version : 2.1.0

Having a similar issue. I'm not a golang person (yet), but I'm thinking that this could be handled in:
https://github.com/yesoreyeram/grafana-plugins/blob/main/lib/go/transformations/merge.go
Maybe somewhere after line 26:
if len(outFrame.fields) == 0 {
  continue
}

Having same issue here. It would be nice to have Max Pages as dynamic variables support field or skip pages with empty json.
My reproduce scenario:
{     "startAt": 0,     "maxResults": 1000,     "total": 3355,     "issues": [] }
If I start at >3355 issue number - I have empty  output in issues. But I cant change maxResults every time, because I have dynamic URL

I'm seeing a similar issue here. I'm interacting with an API that returns an optional nextCursor field that should be added to the following request body to fetch the next page. I'm getting the error below if the initial page does not contain the nextCursor field.
error while performing the infinity query. error while extracting the cursor value
(Trace ID: bc5ba91a4806d36935d23db072f17e27)

This appears to be a bug because the pagination logic should handle cases where nextCursor is absent or null, treating it as the end of pagination.
I attempted to bypass this limitation by using this “expression” to retrieve the nextCursor: $exists(nextCursor) ? nextCursor : null. With this expression, the error is resolved, but it highlights the fact that the pagination logic continues to paginate even if the cursor is null. I can replicate this scenario without using this expression when there are two pages and I have configured to paginate until 5 pages, and the plugin continues sending requests until the configured limit is reached.
This causes duplicate records to appear in the data table because the plugin processes the same pages multiple times.

Any updates on this?


Sublime ToggleComment/Block should be supported in TypeScript plugin

With the TypeScript plugin enabled you loose the comment block feature (default mapped to Ctrl+/ and Ctrl+Shift+/, would be great to support this!

Chris, I just added a file in a new comments branch (09e6b241d6c4c0b) which should fix this.  Can you give it a try before I merge to master and ensure this is good for you.

This is in master now.  Reopen if you have any problems with it.


Allow more forms of specifiers in the config for import-name

Continuing discussion from #387 and #429: we should allow both "graphql-tag" and "graphqlTag" in the config for import-name.
At the very least, adding _ should be done here.
/cc @massimonewsuk @levithomason

Hi @JoshuaKGoldberg , I am a novice to typescript and would like to contribute to this issue for hacktoberfest

Awesome, please do!

As per discussion #429, it should also provide ability to support entire path as replacementKey.
Proposed in #429
import _ from 'lodash'
import _ from 'lodash/fp'
import ReactDOMServer from 'react-dom/server'

"import-name": [true, {
  "lodash": "_",
  "lodash/fp": "_",
  "react-dom/server": "ReactDOMServer"
}]

So as per my understanding, this change should allow replacement-key to be entered in three ways:
Example:
import ReactDOMServerTag from 'react-dom/server-tag'

Possible ways to provide exception for import-name rule:
"import-name": [true, { "react-dom/server-tag": "ReactDOMServerTag"}]

or
"import-name": [true, { "server-tag": "ReactDOMServerTag"}]

or
"import-name": [true, { "serverTag": "ReactDOMServerTag"}]

@JoshuaKGoldberg is my understanding correct ?

Hi yes, awesome! LGTM!


[HSA_STATUS_ERROR] A generic error has occurred

Hi,
I am trying to profile some CEED benchmarks. I'm using a gfx906 card with rocm 2.10.
I'm using hipcc and the compilation/run seem fine, but I can't get any output from the profiler.
I tried different options, but I keep getting this generic error:
rcprof -C  ./bp3 -o 2 -l 8 -d hip
Options used:
   --mesh-dimension 3
   --refinement-level 8
   --order 2
   --device hip
Radeon Compute Profiler V5.6.7262 is enabled
No counter file specified. Only counters that will fit into a single pass will be enabled.
Device configuration: hip,cpu
Processor partitioning: 1 1 1
Mesh dimensions: 8 8 4
Total number of elements: 256
Number of finite element unknowns: 2601
aqlprofile API table load failed: HSA_STATUS_ERROR: A generic error has occurred.
[corona90:mpi_rank_0][error_sighandler] Caught error: Aborted (signal 6)
Failed to generate profile result /g/g91/camier1/Session1.csv.

Have you seen this kind of error?
Thank you for your help,
Jean-Sylvain

Hi Jean-Sylvain. I think what you are looking for is the rocprofiler. Easy mistake ;)
https://github.com/ROCm-Developer-Tools/rocprofiler

Hi Noah,
You're right, I was first trying to use the rocprof shipped with the rocm/2.10 software toolchain.
I switched to the rocprofiler github version, compiled it and run it with the --hip-trace option.
Here is the output:
corona_hip/mfem4_bps> ~/usr/local/bin/rocprof --hip-trace ./bp3 -o 2 -l 8 -d hip
RPL: on '191218_154552' from '/g/g91/camier1/usr/local/rocprofiler' in '~/home/benchmarks_corona/builds/corona_hip/mfem4_bps'
RPL: profiling '"./bp3" "-o" "2" "-l" "8" "-d" "hip"'
RPL: input file ''
RPL: output dir '/tmp/rpl_data_191218_154552_114455'
RPL: result dir '/tmp/rpl_data_191218_154552_114455/input_results_191218_154552'
Tool lib "~/usr/local/roctracer/tool/libtracer_tool.so" failed to load.
Options used:
   --mesh-dimension 3
   --refinement-level 8
   --order 2
   --device hip
Device configuration: hip,cpu
Processor partitioning: 1 1 1
Mesh dimensions: 8 8 4
Total number of elements: 256
Number of finite element unknowns: 2601
   Iteration :   0  (B r, r) = 0.000604889 ...
   Iteration :  56  (B r, r) = 3.78127e-28
Average reduction factor = 0.607985

Total CG time:    0.0621552 (0.0621552) sec.
Time per CG step: 0.00110991 (0.00110991) sec.

"DOFs/sec" in CG: 2.34342 (2.34342) million.

One results.db file is outputed, but the message libtracer_tool suggests me to compile the roctracer tool, but fail with the master  branch with the following error:
In file included from /g/g91/camier1/home/roctracer/src/core/roctracer.cpp:30:
/g/g91/camier1/home/roctracer/inc/roctracer_kfd.h:30:10: fatal error: inc/kfd_ostream_ops.h: No such file or directory
 #include "inc/kfd_ostream_ops.h"
          ^~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

I'd like to have the json file as I see it is possible to get one.
Thank you,
Jean-Sylvain

Ahh yes, your system is missing roctracer. Do you have sudo access on the system? If so, then the easiest method is to sudo apt install roctracer-dev.

Otherwise, if you don't have sudo access and need to build from source, perhaps try building the rocm-2.10.x branch of roctracer since you're using ROCm 2.10. The master branch has already pulled in many changes for ROCm 3.0. The software stack is moving fast.

Ok, thank you for your answer.
I don't have sudo access, I'll try to rebuild it.

Hi,
I've rebuild both roc-2.10.x branches of rocprofiler and roctracer, but I'm still hitting the same error:
Tool lib "/g/g91/camier1/usr/local/rocprofiler/roctracer/tool/libtracer_tool.so" failed to load.
ldd looks good, LD_LIBRARY_PATH too.
My command line looks like this:
~/usr/local/rocprofiler/bin/rocprof -i counterfile_HSA_Vega.txt --stats --hip-trace -t tmp -d data ./bp1 -o 2 -l 8 -d hip
Thank you for any suggestion,
Jean-Sylvain

Hi Camierjs, could you share what system and which compiler you use?

I'm on the Corona cluster, with the 2.10 ROCm stack installed on the system.
I tried on MI25 and MI60.

@camierjs:

there was a bug in the library paths of roctracer for RHEL + ROCm 2.10, this can be fixed via setting export LD_LIBRARY_PATH=$(LD_LIBRARY_PATH):/opt/rocm/roctracer/lib in your environment.  This may resolve your tool lib error.
Regardless, I believe that the hsa-amd-aqlprofile.x86_64 package must be installed via sudo yum install hsa-amd-aqlprofile.x86_64, as this library isn't open-sourced yet.  On my system this lives in /opt/rocm/hsa-amd-aqlprofile/lib/libhsa-amd-aqlprofile64.so.1.0.0.

ROCr runtime failed to ldopen '/g/g91/camier1/usr/local/rocprofiler/roctracer/tool/libtracer_tool.so' library.
If according to the message above "ldd looks good" all dependency were resolved then it might be symbols linking problem.
Could you try a simple test, for example from: '/opt/rocm/hip/samples/2_Cookbook/0_MatrixTranspose', and to check using the LD_DEBUG environment variable if some symbols were not found by dynamic linker:
$ rocprof --cmd-qts off LD_DEBUG=all ./MatrixTranspose
or just to debug symbols:
$ rocprof --cmd-qts off LD_DEBUG=symbols ./MatrixTranspose
A link for LD_DEBUG description:
http://www.bnikolic.co.uk/blog/linux-ld-debug.html

export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/opt/rocm/roctracer/lib didn't helped
The hsa-amd-aqlprofile.x86_64 package seems to be installed:

/opt/rocm/hsa-amd-aqlprofile/lib> ls
lrwxrwxrwx 1 root root     28 Mar 25  2019 libhsa-amd-aqlprofile64.so -> libhsa-amd-aqlprofile64.so.1
lrwxrwxrwx 1 root root     32 Mar 25  2019 libhsa-amd-aqlprofile64.so.1 -> libhsa-amd-aqlprofile64.so.1.0.0
-rwxr-xr-x 1 root root 220064 May  6  2018 libhsa-amd-aqlprofile64.so.1.0.0

both rocprof (system and the recompiled 2.10.x) don't have the --cmd-qts option. However, here ares the outputs of the four runs:

LD_DEBUG=all ./MatrixTranspose: MatrixTranspose.LD_DEBUG.all.gz
LD_DEBUG=symbols ./MatrixTranspose: MatrixTranspose.LD_DEBUG.symbols.gz
LD_DEBUG=symbols ~/usr/local/rocprofiler/bin/rocprof --stats --hip-trace ./MatrixTranspose:
rocprof.MatrixTranspose.LD_DEBUG.all.gz
LD_DEBUG=all ~/usr/local/rocprofiler/bin/rocprof --stats --hip-trace ./MatrixTranspose:
rocprof.MatrixTranspose.LD_DEBUG.symbols.gz

We do see the error with the rocprof:
    102080:	/lib64/libstdc++.so.6: error: version lookup error: version `GLIBCXX_3.4.20' not found (required by /g/g91/camier1/usr/local/rocprofiler/roctracer/tool/libtracer_tool.so) (fatal)
    102080:	file=/g/g91/camier1/usr/local/rocprofiler/roctracer/tool/libtracer_tool.so [0];  destroying link map

Which OS do you have and which compiler do you use?

Linux corona141 3.10.0-1<IP_ADDRESS>chaos.ch6.x86_64, and the module list is:
Currently Loaded Modules:
  1) texlive/2016   2) StdEnv   3) opt   4) gcc/8.1.0   5) rocm/2.10   6) mvapich2/2.3

Could you try to enable devtoolset-7 according to the link below?
https://www.softwarecollections.org/en/scls/rhscl/devtoolset-7/
According to ROCm GitHub https://github.com/RadeonOpenCompute/ROCm#supported-operating-systems
• CentOS v7.7 (Using devtoolset-7 runtime support)
• RHEL v7.7 (Using devtoolset-7 runtime support)

Thank you, I've forwarded your request to the admins, I'll let you know their answer.

@camierjs: Thank you!
And I would appreciate if you could send me output from the following commands in your current environment:
$ gcc --version
$ ldd --version

$ gcc --version:
Reading specs from /usr/tce/packages/gcc/gcc-8.1.0/lib64/gcc/x86_64-pc-linux-gnu/8.1.0/specs
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/tce/packages/gcc/gcc-8.1.0/libexec/gcc/x86_64-pc-linux-gnu/8.1.0/lto-wrapper
Target: x86_64-pc-linux-gnu
Configured with: /builddir/build/BUILD/gccspack/spack/var/spack/stage/gcc-8.1.0-yf4dn5leietjepntgrnkv4syhgmb2nmm/gcc-8.1.0/configure --prefix=/usr/tce/packages/gcc/gcc-8.1.0 --libdir=/usr/tce/packages/gcc/gcc-8.1.0/lib64 --disable-multilib --enable-languages=c,obj-c++,c++,fortran,objc,go,lto --with-mpfr=/ --with-gmp=/usr --enable-lto --with-quad --with-sysroot=/ --with-stage1-ldflags='-Wl,-rpath,/usr/tce/packages/gcc/gcc-8.1.0/lib -Wl,-rpath,/usr/tce/packages/gcc/gcc-8.1.0/lib64 -Wl,-rpath,/lib -Wl,-rpath,/builddir/build/BUILD/gccspack/spack/opt/spack/linux-rhel7-x86_64/gcc-4.8.5/isl-0.18-a6bgwfhlamdrd6tbb7l6oonhnxruvlfh/lib -Wl,-rpath,/usr/lib -Wl,-rpath,/usr/tce/packages/binutils/binutils-2.30/lib -Wl,-rpath,/lib -Wl,-rpath,/lib64 -Wl,-rpath,/usr/lib64 -Wl,-rpath,/usr/tce/packages/binutils/binutils-2.30/lib64 -Wl,-rpath,/lib64 -static-libstdc++ -static-libgcc' --with-boot-ldflags='-Wl,-rpath,/usr/tce/packages/gcc/gcc-8.1.0/lib -Wl,-rpath,/usr/tce/packages/gcc/gcc-8.1.0/lib64 -Wl,-rpath,/lib -Wl,-rpath,/builddir/build/BUILD/gccspack/spack/opt/spack/linux-rhel7-x86_64/gcc-4.8.5/isl-0.18-a6bgwfhlamdrd6tbb7l6oonhnxruvlfh/lib -Wl,-rpath,/usr/lib -Wl,-rpath,/usr/tce/packages/binutils/binutils-2.30/lib -Wl,-rpath,/lib -Wl,-rpath,/lib64 -Wl,-rpath,/usr/lib64 -Wl,-rpath,/usr/tce/packages/binutils/binutils-2.30/lib64 -Wl,-rpath,/lib64 -static-libstdc++ -static-libgcc' --with-gnu-ld --with-gnu-as --with-ld=/usr/tce/packages/gcc/gcc-8.1.0/bin/ld --with-as=/usr/tce/packages/gcc/gcc-8.1.0/bin/as --with-mpc=/ --with-isl=/builddir/build/BUILD/gccspack/spack/opt/spack/linux-rhel7-x86_64/gcc-4.8.5/isl-0.18-a6bgwfhlamdrd6tbb7l6oonhnxruvlfh
Thread model: posix
gcc version 8.1.0 (GCC)

$ ldd --version:
ldd (GNU libc) 2.17
Copyright (C) 2012 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Written by Roland McGrath and Ulrich Drepper.

Could you try just to enable 'devtoolset-7', it might be already installed and it seems you don't need privilege access just to enable it:
$ scl enable devtoolset-7 bash

Unable to open /etc/scl/conf/devtoolset-7!

I see, so just please wait for your admins to respond.
And thank you very much for trying!

You might also can consider to ask admins to install 'roctracer-dev' Linux package.

Ok, I'll do that: thank you for looking into this!

No problem, thank you!

Thank you for all the answers, it's now more on our installation side to get up to date.

In case anyone else is bumping against this error aqlprofile API table load failed, installing hsa-amd-aqlprofile package seemed to solve it. This issue pops up when you search for aqlprofile API table load failed. I only realised you can install that package because calling https://github.com/ROCm-Developer-Tools/rocprofiler/blob/207458f251f223803dbbce64821dde15107a1781/test/util/hsa_rsrc_factory.cpp#L131 in debug spits out the library name isn't found.
Mine was failing because ctrl was falling back onto trying to load HSA_EXTENSION_AMD_AQLPROFILE and then failing on that. This is where the HSA_STATUS_ERROR was coming from which is being invoked from https://github.com/ROCm-Developer-Tools/rocprofiler/blob/207458f251f223803dbbce64821dde15107a1781/test/util/hsa_rsrc_factory.cpp#L133


animationDuration can not be set in interface builder

I think it's because it's of type NSTimeInterval.
It will also feel like it's more responsive if you change its default value to 0.20 instead of 0.35.

Fixed in #423450073441706f362e8ff59463d5061e0976e3


bump node 20.8.0 LTS

Applied mkpr on package.json
sed s/"node": ">=20.7.0"/"node": ">=20.8.0"/

:tada: This PR is included in version 1.11.0 :tada:
The release is available on:

npm package (@latest dist-tag)
v1.11.0
GitHub release

Your semantic-release bot :package::rocket:


Auto-activation in Fish throws exception

Describe the bug
New auto-activation behaviour in Fish throws following exception.
fish: Unknown command: rtx
/opt/homebrew/share/fish/vendor_conf.d/rtx-activate.fish (line 2):
  rtx activate fish | source
  ^~^
from sourcing file /opt/homebrew/share/fish/vendor_conf.d/rtx-activate.fish
	called on line 248 of file /opt/homebrew/Cellar/fish/3.6.4/share/fish/config.fish
from sourcing file /opt/homebrew/Cellar/fish/3.6.4/share/fish/config.fish
	called during startup

Expected behavior
No exception is thrown.
rtx doctor output
rtx version:
  2023.12.25 macos-arm64 (e1f4173 2023-12-13)

build:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS, OPENSSL
  Built: Wed, 13 Dec 2023 06:56:29 +0000
  Rust Version: rustc 1.74.1 (a28077b28 2023-12-04)
  Profile: release

shell:
  /opt/homebrew/bin/fish
  fish, version 3.6.4

rtx data directory:
  /Users/roele/.local/share/rtx

rtx environment variables:
  RTX_SHELL=fish
  RTX_USE_TOML=1

settings:
  {"always_keep_download": "false", "always_keep_install": "false", "asdf_compat": "false", "disable_default_shorthands": "false", "disable_tools": "[]", "experimental": "true", "jobs": "4", "legacy_version_file": "true", "legacy_version_file_disable_tools": "[]", "plugin_autoupdate_last_check_duration": "1 week", "raw": "false", "trusted_config_paths": "[\"~/Dev\"]", "verbose": "false", "yes": "false"}

config files:
  /Users/roele/.config/rtx/config.toml

plugins:
  bun      (core)
  deno     (core)
  erlang   (core)
  go       (core)
  java     (core)
  kotlin   https://github.com/asdf-community/asdf-kotlin.git#27a36cf
  maven    https://github.com/halcyon/asdf-maven.git#83fb4db
  node     (core)
  python   (core)
  ruby     (core)
  rust     https://github.com/code-lever/asdf-rust.git#95acf4f

toolset:
  go@latest, java@latest, maven@latest, node@latest

No problems found

It seems that Fish does source the configuration files in following order according to 8553.

all conf.d files
/etc/fish/config.fish
~/.config/fish/config.fish

This makes it impossible to disable the auto-activation via RTX_FISH_AUTO_ACTIVATE=0 in a user Fish configuration as it is sourced after.

https://github.com/Homebrew/homebrew-core/pull/157141

you can disable this with set -Ux RTX_FISH_AUTO_ACTIVATE 0

@jdx Thanks for the hint, using universal scope does indeed the trick.


2131 refactor risk page component

🗣 Description

💭 Motivation and context
Closes issue #2131

🧪 Testing
All overview page components render properly and all tests pass.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[n/a] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Revert dependencies to default branches.
[ ] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

Resolving merge with React 18 changes from last week.

Hey Matt, I see that there is no check for the backend pipeline, Is that just me or do you what's going on with that?


Remove the @freecodecamp/curriculum npm dependency

The @freecodecamp/curriculum npm dependency is at the wrong level and makes this package unnecessarily tied to that curriculum. It makes it difficult to use a different curriculum. A better approach would be to remove the dependency from this repo and leave this repo with just the curriculum neutral source code.
Another repo (freeCodeCamp/learnFreeCodeCamp?) could then pull both this repo and the @freecodecamp/curriculum dependencies in to combine the two.

I don't understand the benefit in that approach?
Learn uses a custom plugin to pull challenges from the curriculum package. The learn platform was not designed to handle other types of curriculum packages?

@Bouncey I agree with @tchaffee's reasoning here. If someone hypothetically wanted to plug in their own curriculum, that should be an option. This said, I don't agree with the need for yet another repo, as @tchaffee is proposing. There must be a more graceful way.
Until we can come up with a better way, I think people forking Learn could just swap out the package with their own npm module. So barring a better approach, I propose we table this until we have finished updating the curriculum repo and have clear ways for campers to contribute to the curriculum.

Fine to close this for now. I'll take a closer look at this again when we have stabilized the contribution process and the repo structures.


Implement DataFrame's arithmetic interface

Is your feature request related to a problem? Please describe.
We should make sure that Mars DataFrame can do arithmetic e.g. add, minus etc with scalar, Mars(pandas) Series and DataFrame,  and tensor(numpy ndarray).
Describe the solution you'd like
Note that pandas will union the indices, thus this may trigger a shuffle operation when the index is not RangeIndex. We can implement the DataFrame with RangeIndex first, and consider shuffle situation when #134 is ready.

Since shuffle is supported for local mode which introduced in #236 , I am going to take over this job and add support for __add__.


[cli] Lazily install and locally version the react generator

This extends the existing Command protocol to include a GeneratorCommand, which follows our generate interface while having the lazy installation and package-level loading and versioning properties of ReferenceToCommand.
Recommend reviewing with &w=1

Fixed https://github.com/lit/lit/issues/2869


Update installation instruction with new terraform

fix #284
the new readme is here : https://github.com/MalloZup/terraform-provider-libvirt/tree/installing-readme#installing

@inercia thx :sun_with_face:


Revert "Revert "Improve task list""

Reverts alphagov/static#1187

We're going to disable the AB test that displays this in the sidebar for the duration of the test, and clear the Rails cache on Collections to get this back up and running quickly.


Migrate omnibus-chef software/project defs into omnibus/

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

/cc @manderson26 @jkeiser @schisamo

:+1:

:+1:


AttributeError: module 'torch.autograd' has no attribute 'graph'

Describe the bug
Hi dear author, when I try to use FastMoE, I find that my program aborts unexpectedly.
AttributeError: module 'torch.autograd' has no attribute 'graph'
https://github.com/laekov/fastmoe/blob/5083a7367f2defcf5e1022dcdce39022acf5ca11/fmoe/fastermoe/schedule.py#L37
This method seems to work only in Pytorch versions >=1.10. Am I understanding this correctly?
To Reproduce
Steps to reproduce the behavior:

pip setup.py install
cd tests
pytest .

Expected behavior
It seems that there is no package named 'torch.autograd.graph' in Pytorch 1.8.0.
Platform

Device: NVIDIA GeForce GTX 1080 Ti
OS: Ubuntu 16.04.6 LTS
CUDA version: 11.4
PyTorch version: 1.8.0

I think you are right. When developing these features, I was using PyTorch v1.10.0 and I did not test on older versions of PyTorch.

In PyTorch 1.8.0, you can simply remove the line of with (and correctindent of the following line). This should make everything fine.

Thank you for your very helpful reply.


Include taker version in the Hello message sent to maker

Sending the client version along will help with debugging

Resolved in #1648


C++ debugging character arrays and strings:  and "Converting character sets: Invalid argument."

When debugging a C++ file containing character arrays or strings, the contents of the array/string are not shown, but instead shows "<error reading variable>" for character arrays and "Converting character sets: Invalid argument." for strings. Integer arrays are not affected:

However, what puzzles me is that on a different machine but with the exact same VSCode version (1.52.1), C/C++ extension version (v1.1.3) and C++ compiler (mingw-w64 x86_64-8.1.0-posix-seh-rt_v6-rev0), and launch.json file, the contents of the array/string appears normally:

The contents of launch.json are given below:
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": "g++.exe - Build and debug active file",
            "type": "cppdbg",
            "request": "launch",
            "program": "${fileDirname}\\${fileBasenameNoExtension}.exe",
            "args": [],
            "stopAtEntry": false,
            "cwd": "C:\\Program Files\\mingw-w64\\x86_64-8.1.0-posix-seh-rt_v6-rev0\\mingw64\\bin",
            "environment": [],
            "externalConsole": false,
            "MIMode": "gdb",
            "miDebuggerPath": "C:\\Program Files\\mingw-w64\\x86_64-8.1.0-posix-seh-rt_v6-rev0\\mingw64\\bin\\gdb.exe",
            "setupCommands": [
                {
                    "description": "Enable pretty-printing for gdb",
                    "text": "-enable-pretty-printing",
                    "ignoreFailures": true
                }
            ],
            "preLaunchTask": "C/C++: g++.exe build active file"
        }
    ]
}

This problem has been posted elsewhere but with no solutions, as seen here and here

VSCode Version: 1.52.1
OS Version: Windows 10 20H2

Steps to Reproduce:

Create character array in C++ file
Start Debugging

Does this issue occur when all extensions are disabled?: Yes/No
Disabling extensions disables the debugger, so I cannot tell if the issue still occurs.

/extCpp

/extCpp

Since this issue is caused by the C/C++ extension, please open this issue there. Thanks!

Since this issue is caused by the C/C++ extension, please open this issue there. Thanks!


scoket-io-client  implementation

i  wanna use socket-io-client lib alongside stacked
what  is  the  good  way to do so?

The same way as you would use a normal stream. Add it to your service, consume it where you need it in the viewmodel or a different service. Stacked doesn't dictate how things should be used.
It's basically the same as a stream or will be converted into one so just use it that way.


I use the lldb-vscode debugger, which not support stdin, but program need it. When i try to input to program in [dap-repl] window, i switch to insert mode and got dap command line interface.
Is there method to put stdin to program?

I have the same problem here, although I didn't know the "stdin" was the problem. I thought that the main problem was the REPL getting the output from the program since the terminal should be the one getting the output-input from it . I used the  "console = integratedTerminal" setting in the cpp config, but it continues sending the output to the REPL. If someone knows how to make the console ( integratedTerminal ) get the input/output, please let me know.


meterId is not red again from configuration node in case of concentrator usage.

starting of https://github.com/energychain/nr-eaf-amr/commit/a5b7499b651ca7028854d1728b4b4e06c7553ccb

Fixed in https://github.com/energychain/nr-eaf-amr/commit/f83407230a8927d3b8da25de29ec55acbac6cae1


Add new code sample for GUIConsole using WinUI 3 - Windows App SDK v1.1

Description of the new feature/enhancement
I'm creating a WinUI 3 Desktop application (using VS 2022 17.3 Blank App WinUI 3 in Desktop template) where I'd like to use the Windows Terminal API to create a terminal window control to use inside my application.
I started by looking at your GUIConsole.WPF sample that is written in WPF for .NET Framework 4.6.1.
I tried to port the classes and the XAML window but I'm not able to let it work. I still have errors both on XAML and on code.
Proposed technical implementation details
Can you please add a new version of the code sample that is written using the new Windows App SDK v1.1?

Alrighty so there's kinda two separate bits here:

Create a version of the sample that uses WinUI3 & WASDK
Create a Terminal Control to use in a WASDK app

The first is probably a lot easier. The sample is mostly illustrative of the ConPTY APIs. All of the GUIConsole.ConPTY project should work the same in WASDK. The GUIConsole.WPF project is more of a placeholder app - it shows how to wire that up to some  UI layer, but I'd never recommend anyone build production code on top of that. It's a viable place to start to understand the APIs, but appending lines to a text block is a terribly un-performant way of rendering text.
The second is... much harder. We do have a TermControl written for WinUI2 today. It's not ready for someone to take a dependency on it, for sure. I don't think we're doing anything MUX 2.x with it though, but there's definitely gaps before it's ready to be used in WinUI 3. We're tracking productizing that control in #6999.
I'll use this issue to track the first point. Thanks!

Create a Terminal Control to use in a WASDK app

That would be AWESOME! ❤
It is what I was looking for at the beginning of my project. But then, when I saw that the Windows Terminal also has APIs, I thought it might be worth a try and I find out the GUIConsole WPF sample.

...but I'd never recommend anyone build production code on top of that. It's a viable place to start to understand the APIs, but appending lines to a text block is a terribly un-performant way of rendering text.

Thanks for the hint!
Given this consideration, would it be possible to improve the code sample to let it use a better approach?
Thanks a lot!

Given this consideration, would it be possible to improve the code sample to let it use a better approach

I mean, maybe? But the point of that sample isn't really an exercise in performant text rendering, it's more about "how do I use the conpty APIs in a WPF application". In that sense, the sample does quite well. We certainly don't have the bandwidth to optimize that sample farther, but I wouldn't reject PRs to it. I'd rather spend our efforts on the real solution. If you're serious about embedding a terminal control in your application, then #6999 is probably what you'd be more interested in.

Are there any news about those 2 arguments?

Create a version of the sample that uses WinUI3 & WASDK
Create a Terminal Control to use in a WASDK app

Nope, no progress on this front.


removed floor so that global pos works with scale in control

I had two issues with controls when using set_scale and set/get_global_position
Scaling a control to Vector2(3,3) and then setting the global_position to Vector2(6020,6020)
returned (60018,6018) on get_global_position
see #9464
Second problem was adding a (Vector <= scale value) to the global_position the result was no change at all in global_pos
This sloves both issues.
probably closes #6540
Some code if someone wants to try it
extends Control

func _ready():
	$ViewportContainer.set_scale(Vector2(3,3))

	print("Old Control: " + str(get_global_position()))

	print("Old Viewport: " + str($ViewportContainer.get_global_position()))

	print("Old Texture: " + str($ViewportContainer/TextureRect.get_global_position()))

	$ViewportContainer/TextureRect.set_global_position(Vector2(6020,6020))

	print("New Texture: " + str($ViewportContainer/TextureRect.get_global_position()))

	print("New Viewport: " + str($ViewportContainer.get_global_position()))

	print("New Control: " + str(get_global_position()))
#	set_process(true)

#func _fixed_process(delta):
#	$ViewportContainer/TextureRect.set_global_position($ViewportContainer/TextureRect.get_global_position()+Vector2(2,0))
#	print("New Texture: " + str($ViewportContainer/TextureRect.get_global_position()))

You commit was made with an invalid email, and thus an invalid git author: https://github.com/godotengine/godot/commit/1c33547951ae0f2a43c0f7948359d6803ec6a1aa.patch
If you want to keep your email private, see https://help.github.com/articles/keeping-your-email-address-private/

Should be fixed now thanks.

fix makes sense, that floor() was really old and should be gone, feel free to merge when conflicts are gone

Just for the reference, this commit caused a compatibility breakage between 2.1.3 and 2.1.4 (and current 2.1/HEAD) which affects the game Minilens: KOBUGE-Games/minilens#103. But now that it has been in a stable release for almost a year (2.1.4), I guess we'll have to leave with it and fix it downstream :)


[Team 9] Add key press to test the spawn of bosses

Description
Added a key press used for spawning each boss (temporary used for testing, as currently spawning the bosses require you to complete quests):

Press 1: Spawn land boss
Press 2: Spawn water boss
Press 3: Spawn air boss

Type of change

New feature (non-breaking change which adds functionality)
Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Passed the unit tests
Checklist:

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

looks good


protoc-gen-go: protoc fails to compile in Windows

What version of protobuf and what language are you using?
Version: 1.3.0
What did you do?
I tried building a proto file using protoc. I have protoc-gen-go installed and %GOPATH%/bin in my PATH environment variable.
What did you expect to see?
A built .pb.go file.
What did you see instead?
an error message: The system cannot find the path specified. --go_out: protoc-gen-go: Plugin failed with status code 1.
here's the command i ran:
protoc -I %USERPROFILE%/Documents/Projects/go/gateway/protobuf --go_out=plugins=grpc:%GOPATH%/bin/protoc-gen-go %USERPROFILE%/Documents/Projects/go/gateway/protobuf/protobuf.proto

Unfortunately, none of the core developers develop on windows, so we are unable to help directly. Marking this as "help-wanted" if someone in the community wants to help answer your question and/or fix protoc-gen-go to work on Windows.

I would expect your command-line should look something like: protoc -I %USERPROFILE%/Documents/Projects/go/gateway/protobuf --go_out=plugins=grpc:%OUTPUT_DIRECTORY% %USERPROFILE%/Documents/Projects/go/gateway/protobuf/protobuf.proto
Where %OUTPUT_DIRECTORY% is set to whatever is meaningful for your case (likely %USERPROFILE%/Documents/Projects/go/gateway/protobuf/)
I would much more preferably suggest that you run protoc from the directory directly rather than using complicated paths, that may contain spaces, or the : character, which one of the binaries might feasibly misunderstand. So: protoc -I protobuf/ --go_out=plugins=grpc:protobuf/ protobuf/protobuf.proto

In case it's useful information, I successfully ran the Go examples from the tutorials on Windows 10, Go 1.11.4, protoc 3.8.0-rc1, and the latest version of protoc-gen-go (1.3.1 as of this writing, I suppose) today.

same issue, using Windows Subsystem for Linux now.

It seems to me that this is a user-configuration error. If protoc can't find the protoc-gen-go binary, then somethings up with the PATH in Windows. I don't think there's anything actionable for us to do on our end.

Try to install additional packages for Golang other than Go:
Golang packages:
go get -u github.com/golang/protobuf/protoc-gen-go
go get -u github.com/golang/protobuf/proto
you can run this from any directory and it will install packages to somewhere:
C:\Users\Dell\go\bin
Add this to PATH from Environment variables in Windows and then it will be OK

We have the same problem now all of a sudden -- and we are using gradle (don't ask) to build it. Suddenly we have same issue on Windows on one particular build machine. I will figure it out, but would be nice to know that ".exe" is added to a plugin file name resolution on Windows anyway, even when it is invoked from cygwin

Alrighty... So found the culprit. Please don't laugh.
Gradle extracts protoc-gen-go plugin into some cache path, and it is quite long. As of today, together with file name itself it just crossed 256 path length.
I guess protoc 3.10 still cannot call a plugin sitting THAT(/s) deep in a path on Windows.
Welcome to 2020... 🤦♂

MAX_PATH we meet again. :squint:

go version go1.13 windows/amd64

➜  protobuf (master) go install -v ./...
github.com/golang/protobuf/protoc-gen-go/generator/internal/remap
github.com/golang/protobuf/protoc-gen-go/descriptor
github.com/golang/protobuf/ptypes/wrappers
github.com/golang/protobuf/proto/test_proto
github.com/golang/protobuf/ptypes/empty
github.com/golang/protobuf/jsonpb/jsonpb_test_proto
github.com/golang/protobuf/descriptor
github.com/golang/protobuf/protoc-gen-go/plugin
github.com/golang/protobuf/proto/proto3_proto
github.com/golang/protobuf/protoc-gen-go/generator
github.com/golang/protobuf/protoc-gen-go/grpc
github.com/golang/protobuf/protoc-gen-go
go install github.com/golang/protobuf/protoc-gen-go: mkdir D:\mygo\bin;C:: The filename, directory name, or volume label syntax is incorrect.

i'm on win10 and my issue was that after installing google.golang.org/grpc, which also installs github.com/golang/protobuf, protoc-gen-go.exe did not get build in GOPATH/bin. so running go get -u github.com/golang/protobuf/protoc-gen-go did the job and now protoc dir_name/whatever_name.proto --go_out=plugins=grpc:.

I am tring to install protoc-gen-go in windows10 64bit. while run go get -u github.com/golang/protobuf/protoc-gen-go , getting following error
unrecognized import path "google.golang.org/protobuf/compiler/protogen
unrecognized import path "google.golang.org/protobuf/types/descriptorpb
unrecognized import path "google.golang.org/protobuf/cmd/protoc-gen-go/internal_gengo
what is the reason for this? pleas help me on this
sree


Add in post_type filtering to the posts endpoint

We don't have any post_type filtering on the posts endpoint right now. We need to add that filtering in to our queries for the index actions.

Belongs to #26.

@JoshSmith is it filter[post_type]="string" in the url?  I can tackle this if so.

I don't think it's filter like that, I think just ?post_type=. @begedin can you confirm?

I honestly can't remember how it was in rails. Ember's URL doesn't include filter, but doesn't necessarily translate to the API. We should see what our rails API is expecting. That's what Ember is providing behind the scene and that's probably what Phoenix should expect to.


switching city and district in Taoyuan

"LI" is a city district, "TOWNNAME" is the city

Re-run this, please.


Seq2Seq inference with TensorrtExecutionProvider with use_cache=True results in Segmentation fault (core dumped)

System Info
optimum dev

Who can help?
No response
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
No issue arises with CUDAExecutionProvider.
from optimum.onnxruntime import ORTModelForCausalLM
from transformers import AutoTokenizer

provider_options = {
    "trt_engine_cache_enable": True,
    "trt_engine_cache_path": "tmp/trt_cache_gpt2_example"
}

print("Initializing ort_model")
ort_model = ORTModelForCausalLM.from_pretrained(
    "gpt2_onnx",
    use_cache=True,
    provider="TensorrtExecutionProvider",
    provider_options=provider_options
)

tokenizer = AutoTokenizer.from_pretrained("gpt2")

print("build engine for a short sequence")
text = ["short"]
encoded_input = tokenizer(text, return_tensors="pt").to("cuda")
output = ort_model(**encoded_input)

print("build engine for a long sequence")
text = [" a very long input just for demo purpose, this is very long" * 10]
encoded_input = tokenizer(text, return_tensors="pt").to("cuda")
output = ort_model(**encoded_input)

text = ["Replace me by any text you'd like."]
encoded_input = tokenizer(text, return_tensors="pt").to("cuda")

print("Generate!")
for i in range(3):
    output = ort_model.generate(**encoded_input)
    print(tokenizer.decode(output[0]))

Error:
2023-01-24 16:22:51.520935518 [E:onnxruntime:Default, tensorrt_execution_provider.h:58 log] [2023-01-24 15:22:51   ERROR] 4: [graphShapeAnalyzer.cpp::processCheck::587] Error Code 4: Internal Error (IAssertionLayer (Unnamed Layer* 105) [Assertion]: condition[0] is false: 0. For input: 'attention_mask' all named dimensions that share the same name must be equal. Note: Named dimensions were present on the following axes: 1 (name: 'past_sequence_length + sequence_length'), 2 (name: 'past_sequence_length + sequence_length'))

Expected behavior
no error.
Additional tests with TRT execution provider should be added.

I can reproduce the bug with transformers==4.25.1 + d2436cf386dd32a8d9bbad97e577502248914398 , but not with 50e87a6c90b211241a932cd00135d3bd21835a09 + transformers==4.26.0. So it was likely a bug in transformers that has been fixed in the latest release. I'll just add a test for this.


Cost function of message retrieval

What should determine the cost of retrieving a message?
PoC approaches
The server may estimate the cost as:

constant per message
constant per byte
dependent on message age (presumably, newer messages are cheaper)
dependent on other message parameters (topic, metadata, etc)

More elaborate cost calculations
The costs of a Store server are storage, bandwidth, and computation.
A Store server does two things: it stores messages, and serves messages to clients.
The cost of storing messages is composed of:

storage:

storage costs of all older messages: proportional to cumulative (message size x time stored);
the cost for I/O operations for storing new messages (roughly constant per unit time, though may fluctuate due to caching, disk fragmentation, etc.);

bandwidth (download) for receiving new messages;
computational costs.

The cost of serving messages to clients, per unit time, is composed of:

bandwidth

upload: proportional to (number of clients) x (length of time frame requested) x (message size);
download: proportional to the number of requests;

computational cost of handling requests.

Storage is likely the dominating cost.
Storage costs is proportional to the amount of information stored and the time it is stored for.
A cumulative cost of storing a single message grows linearly with time.
The number of messages in a response may be approximated by the length of the time frame requested.
Computation: the server spends computing cycles while handling requests.
This costs likely depends not only on the computation itself, but also at the database structure.
For example, retrieving old or rarely requested messages from the local database may be more expensive than fresh or popular ones due to caching.
More formal calculations should be done, under certain assumptions about message flow (i.e., that it is constant).

With the new store req-res that returns message ids, then it becomes trivial to have a cost per message.
This assumes that client does not pay for message ids, but pays for message content:
sequenceDiagram
	client-->>server: request msgs ids for period t1 to t2
	server-->>client: returns 10 msg ids
	client->>client: check seen msgs
    client->>client: 3 msgs are missing
	client->>blockchain: pays server for 3 msgs
	client->>server: request 3 messages, attaches txid
	server->>blockchain: checks payment
	server-->>client: deliver messages

PS: sorry about the label deletion, this PR will remove this kind of automtaed action: https://github.com/waku-org/pm/pull/103

client does not pay for message ids, but pays for message content

The scheme makes a ton of sense!
Do you think we should aim for it in the PoC, or in later protocol iterations? As of the current PoC outline, the client skips the request-ids stage and instead pays a fixed price per hour of history, and gets all relevant messages in response, no matter if the client already has them or not. This is kinda wasteful (client gets messages it already has) but conceptually simpler (just one round-trip).

to compare responses by different store nodes and choosing the one with the most up-to-date response

Absolutely, I think I wrote somewhere how Alice could then do a sync req to Bob and Carole.
Compare what messages they have, query/pays Bob because he's cheapest to retrieve the bulk of messages, and then query/pays Carole of a unique message she has, but she is more expensive.
It may also make the market more interesting:

Bob is cheaper because he only store 12 hours of messages.
Carole is more expensive because she stores a week of message.

Alice will always use Bob for the most recent messages, but has to use Carole if she was offline for more than 12 hours.
What is interesting here, is that once the protocol is available, this is purely an engineering problem of window-shopping and cherry-picking.

client does not pay for message ids, but pays for message content

The scheme makes a ton of sense!
Do you think we should aim for it in the PoC, or in later protocol iterations? As of the current PoC outline, the client skips the request-ids stage and instead pays a fixed price per hour of history, and gets all relevant messages in response, no matter if the client already has them or not. This is kinda wasteful (client gets messages it already has) but conceptually simpler (just one round-trip).

I will let you and @jm-clius decide here.
My current opinion is that when store sync is available, then I assume the best practice to be:
(A)

store sync
store query missing messages using id.

instead of
(B)

store query using using time frame or cursor and get full messages.

My expectations might be incorrect tho. My assumption is that (A) gives you a better bandwidth/latency ratio.
Note: when I write "store sync" I mean "store query to retrieve messages id". This may not be the correct/final name of the protocol.

My current opinion is that when store sync is available, then we can assume the best practice will become:
(A)
store sync
store query missing messages using id.

My tangential concern here is about the relationship between Store sync and incentivized Store requests. If both are available, why would a rational client use incentivized requests? It can "pretend" to be a syncing server and get the same data for free. I know there may be heuristics to distinguish such clients-pretending-to-be-servers, but I'm not sure we can do it with certainty.


Fill out XML spec document

Wrote out a spec file for the XML file format since it's long overdue. I would love feedback on this because I'm not 100% convinced on some of the descriptions, but it should hopefully be workable.
Rendered document is here.

I'd also like feedback on whether or not we should document Qdir/QFont since they're not really useful to anybody but Roblox. I can definitely see the usefulness of having that information if you were working with settings files, but that seems like an uncommon use case.
The same question applies to more esoteric elements like MetaBreakpoints and ScriptDocs. I'm leaning towards no, since those are really obscure and are only seen in internal files that aren't used for models or places, but I'd still like people's opinions.

Given nobody else has commented on this and I've addressed what feedback I got, I'm going to go ahead and merge this. If anyone has further feedback, go ahead and open an issue or PR.


update vagrant and readmes with utils/echo

Merged as https://github.com/openwhisk/openwhisk/commit/aef7885f8277977ef1a44cb81ea73db816edb007.


[QUESTION] Include javascript file generated by babel-loader/ts-loader

I have an index.ts and because I am targeting super old browsers, I need to:

Convert TS to JS via ts-loader
The newly generated JS is passed to babel-loader to make it compatible with chrome 37
Some dependencies needs to be made compatible with chrome 37 via babel-loader
Include whatwg-fetch to have a polyfill for the fetch API.
The output is a main.js file.

I would like to use HtmlBundlerPlugin but I cannot figure out how to tell him to include the resulting main.js. Doing something like this
<script src="dist/main.js"r></script>

will return the following error
...
HTML Bundler Plugin  Resolving of source files in the template file failed.
File: src/views/mytemplate.eta
Error: Cannot find module '/Users/xxx/yyy/ccc/repo/dist/main.js'

It seems everything is generated at the same time and the main.js is not yet outputted into /dist/.
Any idea how I can fix this?
My current webpack config generating the main.js.
entry: ['whatwg-fetch', './src/index.ts'],
resolve: {
  extensions: [ '.ts', '.js'],
},
module: {
    rules: [
     {
         test: /\.js$/,
         include:[
             /node_modules\/\@aaaa/,
             /node_modules\/\@bbbb/,
         ],
         use: [{
          loader: 'babel-loader',
          options: {
            presets: [
              ['@babel/preset-env', { targets: "chrome 37", useBuiltIns: "usage", modules:"cjs", corejs:3, debug:false}]
            ]
          }
        }]
     },
      {
        test: /\.ts$/,
        exclude: [
          path.resolve(__dirname, 'tests')
        ],
        use: [{
          loader: 'babel-loader',
          options: {
            presets: [
              ['@babel/preset-env', { targets: "chrome 37", debug:false}]
            ]
          }
        },
        {
        loader: 'ts-loader',
        }
     ]
    }
    ],
  },
  target: ['es5', 'web'],
  output: {
    filename: 'main.js',
    path: path.resolve(__dirname, 'dist'),
  },

@R1kk3r
you should specify the source ./src/index.ts file directly in the template using the relative path or webpack alias:
<script src="./index.ts"r></script>

The plugin resolves source files of assets (SCSS, TS/JS, etc.) in templates and replaces them with correct output URLs in the generated HTML.
Don't needed to define source TS files in webpack entry.
Read please the README. All information with examples is there.
You can create a small repo with your setup and I will help you to configure it.


decode/event: Test Applayer engine events

Ticket: #4482
Suricata verify test that insures events created with
DecodeEngineSetEvent create anomaly records.
This test uses a SWF file manually modified with an invalid version to
trigger an event. The test verifies that the anomaly record was created.
This PR is referred to by Suricata PR #6643

Do we need the full suricata.yaml? Can we either have a minimal one or a --set ... option to change the things that need to be modified?

Closing as suricata PR was closed cf https://github.com/OISF/suricata/pull/6663


Infrastructure as code static analysis plugin (checkov)

Summary
A plugin that can scan IaC manifests such as k8.yaml, serverless, terraform, cloudformation and azure arm templates before provisioning using checkov. would you see it as a good fit?
Project website (if applicable)
https://github.com/bridgecrewio/checkov/

Hey! I think this sounds like an interesting plugin indeed. Are you thinking the results of the scan would be visualised in Backstage? I think you could also consider presenting the user with the option to enable checkov scanning if this is not already enabled for the component.
I'm not sure if the plugin itself would be involved in triggering the scan though -- guessing that is done during CI?

I'm not sure about the exact implementation yet and would love to do a zoom on it.
The mission i would like to answer is making sure a service in the service catalog is vetted by checkov. is it a good match?

It sounds like it to me, even though I'm not an expert in checkov :)

can a plugin scan other plugins or services?

Not really. A plugin is primarily a UI element, see https://backstage.io/docs/plugins/


Service for scheduler

Hi
I'm just trying to figure how the tasks are run in production. Am I right in thinking that the scheduler must be running, and if i am using celery than a worker must be running as well?
I just plan on using mysql at the moment. Do you recommend creating a init script to start the scheduler?
Cheers

You are absolutely right. A more modest single worker setup could run the scheduler with the LocalExecutor (as subprocesses). No Celery or message queue setup is needed in that case, but you are limited to a single worker.

Ok great thanks. I see the LocalExecutor can run tasks in parallel using multiprocessing. Will the scheduler be able to run tasks from different DAGs in parallel?

Yes.

:+1:


Type inference for parameters in new functions

Describe the bug
Type inference for sprite variables with new functions gives an error when you drag out call function
To Reproduce
Steps to reproduce the behavior:

Go to https://makecode.com/_3jgAJRMbUK4E in beta
See error

alternatively: in a new project in beta

create a function that takes a sprite
drag a call function block into function or on start
See type of item is inferred to be number, rather than Sprite

Expected behavior
the type of item should be inferred to be Sprite
Screenshots

Additional context
orthogonal, but it would be nice if the default param when you dragged out a function with sprites was mySprite instead of item, as thats the typical default elsewhere for sprites.

I've created https://github.com/Microsoft/pxt-arcade/issues/745 for the part about default variable in function calls


Creating heaps in pre-allocated arenas

Hi!
Our team has been trying to adopt mimalloc for the project we've been working on for embedded devices and consoles. We would like to limit how much memory mimalloc would use on a given system and control that ourselves. There were issues on this subject before (e.g https://github.com/microsoft/mimalloc/issues/617). My takeaway from that thread was that we can create an arena in a specific memory region and use that arena to create thread local heaps using mi_manage_os_memory_ex and mi_heap_new_in_arena. Unfortunately, mi_heap_new_in_arena still tries to create an arena to create the heap (here), which doesn't work for us.
As it was discussed in other threads, we would like to limit all mimalloc ops to a single arena that we create and provide to mimalloc and if that's exhausted, just return NULL and we would handle OOM. Is there a way for mimalloc to use the arena that we create to allocate the heap within that arena, instead of allocating a different one?

Looking deeper into the mi_heap_new_in_arena I was able to figure out where that extra arena allocation was coming from, it was allocating the memory for the initial backing heap which is used to create a new heap, please correct me if my understanding is wrong here.
I was somewhat able to bypass this issue by setting bheap->arena_id in mi_heap_new_in_arena to the arena id I'm passing, i.e:
mi_decl_nodiscard mi_heap_t* mi_heap_new_in_arena(mi_arena_id_t arena_id) {
  mi_heap_t* bheap = mi_heap_get_backing();
  bheap->arena_id = arena_id; // <<<<<<<<<<<------------------ This line
  mi_heap_t* heap = mi_heap_malloc_tp(bheap, mi_heap_t);  // todo: OS allocate in secure mode?
  if (heap == NULL) return NULL;
  // don't reclaim abandoned pages or otherwise destroy is unsafe
  _mi_heap_init(heap, bheap->tld, arena_id, true /* no reclaim */, 0 /* default tag */);
  return heap;
}

My understanding is that arenas are shared between threads and this seem to be an acceptable change modification to scope everything to a single arena?
With the change above, the only OS call would happen when a new thread tries to call mi_heap_new_in_arena and _mi_thread_heap_init ends up calling the OS to allocate memory for the new TLS backing heap. Is backing heap only used to hold data for the heap being created? I wonder if this part could be somehow optimized as well.
There's one moment with _mi_thread_heap_init when it allocates mi_thread_data_t from the OS, it rounds its size to 16KB, asking for x4 memory.
I've tried the changes above, with disabled MI_STATS, and was able to minimize the number of OS calls from mimalloc and so far things seem to run smoothly. Does this modification breaks mimalloc architecture in some way that I'm missing?
Thank you!


Use Local & JWT Strategy on same app (on same route)

Hi, so my route looks like this:
// validating using JWT
router.post('/profile', passport.authenticate('jwt', {session: false}), function (req, res) {
        res.json({'success': true});
    }
);

// validating using LOCAL
router.post('/profile', authenticationHelpers.isAuth, function (req, res) {
        res.json({'success': true});
    }
);

// authenticationHelpers.isAuth
function isAuth(req, res, next) {
    if (req.isAuthenticated())
        return next();
    res.status(401).json({"authenticated": false});
}

So, how do i use both Local & JWT Strategy on same app (on same route) ?
Note: Local for web app, JWT for mobile app

Finally figured it out. Updated isAuth function:
function isAuth(req, res, next) {
    if (req.headers.authorization) {
        passport.authenticate('jwt', {session: false}, function (err, user, info) {
            if ((!err || !info) && user) {
                req.user = user;
                return next();
            }
            res.status(401).json({authenticated: false, message: "Login expired."});
        })(req, res, next);
    } else {
        if (req.isAuthenticated())
            return next();
        res.status(401).json({authenticated: false});
    }
}

https://gist.github.com/joshbirk/1732068
That gist has comments from the author of passport with his recommendation on how to do this.  If you've figured this out is is alright to close this issue?  It's really more of a general passport question than anything specific to this specific passport module.

Thank you for the link. You can close this issue.

Hope it helped


Change file extension from .ep to .kmp

It gives more sense to provide file with .kmp (knowledge model package) instead of .ep (events package?) as we already changed the naming in the Wizard while ago...

Definitely! As I already explained, "event" is a very bad name!


Provide an example to use --openstack_additional_flags

Hello,
can you please provide an example on how to use the flag --openstack_additional_flags ?
I'm trying to pass the following flags : --os-project-name / --os-username / --os-password / --os-auth-url
Thanks in advance.

Hey, an example of this flag would be something like:
--openstack_additional_flags=--os-project-name=admin,--os-username=admin

I'm not sure what's your use-case, but you don't need to pass --os-project-name / --os-username / --os-password / --os-auth-url or any other flags that can be read from the shell environment variables.
Usually those flags are set in your openrc file as described here:
https://docs.openstack.org/user-guide/common/cli-set-environment-variables-using-openstack-rc.html
So, all you would need to do is source the openrc file into your shell, and when you execute PKB, it will automatically pick them up.
Hope that helps!

Hello,
it seems not working with :
--openstack_additional_flags=--os-auth-url=http://<IP_ADDRESS>:5000/v2.0,--os-project-name=demo,--os-username=admin,--os-password=Pass4Admin!
Result :
2017-05-09 10:25:20,901 6adfe169 Thread-10 cluster_boot(1/1) INFO     Running: openstack image list --format json - - o s - a u t h - u r l = h t t p : / / 1 0 . 0 .  0 .  Got return code (1).
STDOUT:
STDERR: Missing value auth-url required for auth plugin password

Any idea why there is space between all characters ?
I want to script some tests and that's why I try to pass everything in the command line.
For normal test, I source my rc file and everything is working.
Thanks in advance for your help.

I think the PKB version you are running has a bug where it iterates over each character of the flag, instead of it being a single string. That's why you see that the flags are spread out with spaces in between.
- - o s - a u t h - u r l = h t t p : / / 1 0 . 0 . 0 . 1 0 : 5 0 0 0 / v 2 . 0
This was fixed in this PR #1340
Can you try pulling from master, and trying again? I've tested it locally and it seems to work for me.
Let me know your results. Thanks!


Setting the EX param not working correctly?

When using the 'EX' param to set an expiration in seconds, the TTL does not show up in the upstash console. Is this value being set incorretly?
import { Redis } from '@upstash/redis';

const redis = new Redis({
  url: process.env.UPSTASH_URL,
  token: process.env.UPSTASH_TOKEN,
});

redis.set('ratings-cache', JSON.stringify(ratings), { EX: 3600 });

Strange, using ex and EX do not set a TTL in the upstash console when looking at the Data Browser tab.

Ah, okay yeah using lowercase ex does show up for me as a TTL in the upstash console. Thanks for clarifying!

Is this a known issue? It seems it still doesnt show the TTL seconds in the console for me:

@andrewlu0 Did you make this work? I'm having the same issue.

I just tried await redis.set('foo', 'bar', { ex: 1000 }) and it shows up in the databrowser
can you share your code that doesn't work?

// Lots of obsfucated code but roughly:A

  const redis = new Redis({
    url: "https://hidden.upstash.io",
    token: 'hidden'
  });
  const { listingSlug } = req.query;
  const redisKey = listingSlug;
 const uniqueDates = ['array', 'of', 'strings'];
 await redis.set(redisKey, uniqueDates, {
      ex: 60 * 5,
    });


Create 2016-08-25-simple-sink-source-with-graphstage.md

Please final review

LGTM, after small nitpicks

Simplified Java example, OK to publish?

No

Why Integer.Max_value? You can very easily exhaust that range.

Roughly 100-200 seconds can be enough at peak perf. What was wrong with the Iterator and using a Long?

With new Iterator? That's very non nice and showing that were missing API IMHO.

The current one is something that invites patterns that might unexpectedly break. I don't see Iterator as non-nice at all. It is a very simple anonymous class.

Eh, it has extra methods I see: https://docs.oracle.com/javase/8/docs/api/java/util/Iterator.html

ah, but they are default methods. I still opt for the Iterator then.

        Source.fromIterator(() -> new Iterator<Long>() {
          private long counter = 0;
          @Override public boolean hasNext() { return true; }
          @Override public Long next() { return counter++; }
        });

This smells missing-simple-api to me but ok.

The others don't (unfold, or range which is finite)? Also, this shows a nice way to build simple sources in one go, as Iterators are a model that people already likely know.

We can update the post once we add a zipWithIndex

Found what I was looking for: https://docs.oracle.com/javase/8/docs/api/java/util/stream/IntStream.html#iterate-int-java.util.function.IntUnaryOperator-

Ok, that might be the nicest.

Use a LongStream though...

By democratic vote the version with Iterator won the ms. universe contest:
:point_up: August 25, 2016 4:24 PM


Remove ESLint rule to space brackets of multi-item arrays

Right now are forced to do [a] but [ a, b ]. This means we sometimes have to change the spacing around the brackets when we add or remove items from the array — and that is annoying. We should remove this distinction, and stick with no spaces in array brackets always. Hopefully this is automatically fixable.

Agh, I filed this in the wrong repo! Very sorry.


Fix multiple CVE

Description
This patch updates some dependencies used to fix multiple security warnings found within these libraries or its dependencies.
Motivation and Context

jsoup to 1.15.3 (fixes  CVE-2022-36033)
jackson to 2.13.4 and jackson-databind to <IP_ADDRESS> (fixes CVE-2022-42004, CVE-2022-42003)
tika-parsers to 1.28.5 (CVE-2022-33879 and for updated dependencies of jackson)

Additionally i added the new GPG key for the jackson project and remove one old key
that expired in 2016. The other key from jackson project expired just some months ago, so i
let it there. The new key is taken from the Jackson main repositories KEYS file (https://github.com/FasterXML/jackson/blob/master/KEYS)
How Has This Been Tested?

Tested ourself using jmeter with newer libraries and run gradle check
Screenshots (if appropriate):
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.

and updated xstream to 1.4.20 fixing CVE-2022-40151 and CVE-2022-41966
local tests running again without problem.


VDA 5050 Vehicle connected Failed

VDA 5050 Vehicle connected Failed

初始化工程，配置好 mqtt 服务器，Broker connected 成功，Vehicle connected Failed

@hzcfy When creating issues, please fill out the issue template and provide as much information as possible to help identify the problem.

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

Hi Stefan Walter(https://github.com/swltr), Martin Grzenia(https://github.com/martingr)
I am working on the openTCS-integration-Example and I tried to integrated it with VDA5050 adapter v2.0. After Kernel, Model Editor, Operational Desk, Kernel Control Center service turned UP. I am getting in openTCS template VDA Adapter for Operational Desk as-

While I enable the VDA5050 adapter, the Vehicle state is still "Unknown". and My Control Center is-

my topic is : uagv/v2/xyz/0001/state
This is the details I send through MQTTx-
{"headerId":11181,"timestamp":"2024-06-18T12:14:44.6046930Z","version":"2.0.0","manufacturer":"xyz","serialNumber":"0001","orderId":"","orderUpdateId":0,"lastNodeId":"","lastNodeSequenceId":0,"nodeStates":[],"edgeStates":[],"agvPosition":{"positionInitialized":true,"localizationScore":1.0,"x":-29.0,"y":-16.999998092651368,"theta":0.00019965189858339727,"mapId":"01"},"velocity":{"vx":0.0,"vy":0.0,"omega":0.0},"loads":[],"driving":false,"paused":false,"newBaseRequest":false,"distanceSinceLastNode":0.0,"actionStates":[],"batteryState":{"batteryCharge":100.0,"batteryVoltage":48.0,"batteryHealth":100.0,"charging":false},"operatingMode":"AUTOMATIC","errors":[],"information":[],"safetyState":{"eStop":"NONE","fieldViolation":false}}
### It will connected with broker, but not with the Vehicle. Guide me like how can I enable Vehicle connection.

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。


Acoustic wayfinding task

This patch adds the new acoustic wayfinding task including:

Example world
Practice worlds
Scoring plugin

How to test it?

Launch the simulation:

ros2 launch vrx_gz competition.launch.py world:=acoustic_wayfinding_task

Launch the joystick teleoperation node:

ros2 launch vrx_gz usv_joy_teleop.py

Teleoperate the robot going close to the moving pinger until the task finishes.

Note: You can uncomment the block within lines 496 and 503 of the worlds to visualize the moving pinger. Do not forget to recompile.

As per our offline discussion this scoring plugin has been updated. It now tracks the error of the vehicle with respect to a single moving pinger. There are topics to track the instant error and the mean error during the task. I also introduced two obstacles that add a penalty when touched.

@caguero This is working as advertised, but I think we should consider a few tweaks if we have time before the release:

I propose changing the name to "acoustic tracking" to avoid confusion with our previous wayfinding task and help suggest that we are practicing for working with an underwater vehicle. Open to other suggestions if you have a better name.
I think the underwater beacon should move at least twice as fast, and travel along a longer (and maybe slightly more complicated) path. Maybe something similar to path a USV would need to travel to complete the basic wayfinding task?
Can we re-use the obstacle field from the gymkhana task, or something similar to it? More obstacles would help give a better sense of what I'm hoping will be the final look of the task.

Let me know what you think. If you'd prefer to postpone any of the above until the next release I think that's fine too.

Good suggestions, thanks.
I renamed the plugin and changed the layout only in the acoustic_tracking_task.sdf world for now. Could you please test that world first before I actually modify the practice worlds? I didn't reuse the obstacle field model because I wanted to place the obstacle buoys in places that are close enough to the pinger trajectory.

Could you please test that world first before I actually modify the practice worlds? I didn't reuse the obstacle field model because I wanted to place the obstacle buoys in places that are close enough to the pinger trajectory.

Thanks! I just tested it. I like this version much better. One thing I'm trying to brainstorm is ways to make it qualitatively different from the basic acoustic task. Right now it is different some of the time, but a lot of the time the beacon is moving through regions of open water and the USV just has to stationkeep above it (with a slight drift). I think we could address this easily with two changes:

Make the beacon move even faster. Double its speed again? That way following it seems less like stationkeeping.
Make sure there are situations where the USV has to decide between minimizing distance to the buoy and avoiding obstacles. Right now if I sit on top of the buoy I also avoid all the obstacles. Two ways to fix this would be to either change the path or just add a bunch more obstacles.

Also a tiny tweak, but could we change the color of the visual marker to something different from the obstacles? Maybe white or red?

I implemented your suggestions bumping up the speed of the pinger, changing its visual to white and adding a few more obstacles in the trajectory. Let me know what you think.

Looks great! Thank you!

I implemented your suggestions bumping up the speed of the pinger, changing its visual to white and adding a few more obstacles in the trajectory. Let me know what you think.

Looks great! Thank you!

@M1chaelM , I have updated the practice worlds. #0 is very similar to the example world. In #1 the pinger follows an X pattern over 4 waypoints. In #2 the pinger follows a square pattern.

@caguero Per our offline discussion, could you adjust course 2 so it is more 2-dimensional, similar to course 1? Thank you!

@caguero Per our offline discussion, could you adjust course 2 so it is more 2-dimensional, similar to course 1? Thank you!

Done, I've reused the same waypoints as #1 but to be followed in a different order and I shuffled the obstacles a bit to be in the trajectory of the pinger. For easier testing I left some orange buoys in the four waypoints that the pinger is visiting. I'll remove them before merging the PR.


Rethink/redesign validation of prediction responses

We recently ran into an issue in the Replicate production environment where a model with an output type of cog.File returned a string rather than a file-like object. This highlighted multiple issues relating to output handling and validation of responses from the model, namely:

Cog's synchronous prediction API validates the entire prediction response before returning it to the user, but the asynchronous prediction API doesn't do this anywhere.
upload_files quietly ignores things that don't look like file handles. This results in the value returned by the model being passed back to Replicate, even though it was a 10MB base64-encoded blob and not a URL.

It's not immediately obvious how and where to fix this in the code. There are at least two things that seem a bit wrong here:

While returning an invalid type is clearly a model error, it seems to me thatPOST /predictions should probably return a prediction with a failed status and an appropriate error message rather than a 500.
The async update path (both polling and webhooks) should probably refuse to propagate invalid payloads.

Relevant Slack thread: https://replicatehq.slack.com/archives/C04V1QT8XV1/p1690925208419559

Got 500 & media type error when I managed to return numpy embedding file generated by ViT transformer. I am curious how to return a downloadable file on replicate deployment, e.g. COCO format RLE file, embedding file or pickle file.

Got 500 & media type error when I managed to return numpy embedding file generated by ViT transformer. I am curious how to return a downloadable file on replicate deployment, e.g. COCO format RLE file, embedding file or pickle file.

In general you want to write the file to disk and return a cog.Path object pointing to the file. Alternatively you can return a cog.File object to wrap the data.

Both cog.Path and cog.File did not work locally when we try to return an embedding file in .nmp format. The workaround that worked is to return an Output(embedding=Path() approach documented here and here.


Introduce Myers algorithm in linear space

This PR requires #111 to be merged first.
I will then rebase.

I'm done with this one. The refactor: clean up DiffRowGenerator commit has a bit of changes but they're mostly to remove "untyped" lambdas.
This PR is a breaking API change for DiffRowGenerator.

what's the breaking change in the DiffRowGenerator

It's just the constructor, we introduced new interfaces so I'm not sure it would work straight away.

@petertrr np. I have another PR with a small cleanup and then it would be cool to do a release imo!

Ah I see you already released ahaha, well no problem, will be for another minor release.


Adds mouse wheel input, and a 0...1 float output value

Mouse wheel input is now supported. Works when the user hovers the mouse cursor over the knob element and sends either a MOUSE_BUTTON_WHEEL_UP or MOUSE_BUTTON_WHEEL_DOWN input event by scrolling the mouse's wheel. Upon receiving the event, the _dial.rotation_degrees variable is incremented or decremented by the wheel_increment value, and also clamped within the minimum and maximum degrees permitted.
The dial_rotated signal also now outputs a float value ranging from 0 to 1. This represents the position of the dial between the minimum (at zero) and maximum (at one) degrees settings. Useful if you aren't concerned about the degrees but just want to know what percentage to maximum the dial is set to when the user is done.
The project has been bumped to version 1.1 as well, for what it's worth.

Looks good! tidied up some small things. I will also probably get rid of the plugin in a separate PR, since the only thing I needed it for was for a custom icon, but now you can do that with the @icon attribute.
Thanks!


Concurrent CT write

Scaling up the TSO by parallelizing the batched writes to the commit table. The architecture mostly changed by adding batch events and creating handlers under PersistenceProcessorImpl.
This commit increases the performance of Omid from 55K tps to 380K tps when using 6 region servers.
Please note that the addition of MonitoringContext significantly degrades performance and therefore, should becomes optional to achieve good performance.

Overall the idea is very good!

Given my comments above LGTM. We will need to rebase it though.

Code moved to a new branch.
https://github.com/yahoo/omid/pull/89


Added author filter in book recommendation

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #1634
Description
[Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.]

Added a new filter of Author for the book recommendation system.
If the user enters only genre then it will recommend books of that genre.
If the user enter genre and author then it will recommend books of that author in that specified genre.
If such book is not present in the system then it will notify that book is not found.
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
The page looks like this

Books recommended on basis of genre ONLY.

Books recommended based on genre and author BOTH.
Even if the entire author name is not added it still suggests appropriate book.

NO BOOK IS FOUND for that genre and author

Checklist:

[X] I have made this change from my own.
[X] I have taken help from some online resources.
[X] My code follows the style guidelines of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have made corresponding changes to the documentation.
[X] My changes generate no new warnings.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers and screenshots after making the changes.

@anuragverma108 can you please look into the PR and merge it. Also add suitable level labels on the PR and the related issue.

@Archiesachin Great Work!

@RitiChandak can you please add the level2 label for the related issue #1634


Fix binja-win-doc link

Hello,
it looks like the link to the binja-win-doc plugin was wrong, it pointed to riskydissonance/matteyeux/binja-win-docs instead of matteyeux/binja-win-docs

Good catch, thanks. Hmm, the generator really should have failed in that instance. We're in the middle of redoing the plugin infra though so the new system won't have that issue.


Updated enterprise + analytics link.html + created benchmark page in About

this has been fixed by @thibaultCha


Hybrid Android - Emoji picker for status page overlap with save button

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

Version Number: v9.0.51-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Navigate to Expensify hybrid android app
Go to setting - profile - status
Click drop down button to choose emoji

Expected Result:
Emoji picker pop up without any overlap
Actual Result:
Emoji picker overlap with save button when the first status was empty
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/b9478a6e-672a-478f-bb31-3477ca40de50

View all open jobs on GitHub

I marked the issue as a deploy because this issue is not repro on Hybrid prod app

Issue reproducible on both stg and prod Android App

Since this is reproducible on production as well, I'm removing the deploy blocker and taking this external.

Implement a mechanism to detect the available screen space when the emoji picker is activated. This could involve calculating the distance from the bottom of the screen to the save button and positioning the emoji picker above or below it, depending on the available space.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/nikolad54
Implement a mechanism to detect the available screen space when the emoji picker is activated. This could involve calculating the distance from the bottom of the screen to the save button and positioning the emoji picker above or below it, depending on the available space. The calculation should be based on screen size. It may use function below.
export const vhToPx = (value, isNum) => {
  var w = window,
    d = document,
    e = d.documentElement,
    g = d.getElementsByTagName('body')[0],
    y = w.innerHeight || e.clientHeight || g.clientHeight

  var result = (y * value) / 100
  if (isNum) {
    return result
  } else {
    return result + 'px'
  }
}

export const responsiveDimension = val => {
  return IsMobile
    ? window.innerWidth / window.innerHeight <= 0.68
      ? mobileScale(val)
      : vhToPx(val)
    : vhToPx(val)
}

This seems similar to https://github.com/Expensify/App/issues/48100 and should have been fixed by https://github.com/Expensify/App/pull/51307.


sidebar-off-canvas behaves strangely

I have a sidebar with a combination of sidebar-hidden & sidebar-off-canvas classes.

When the sidebar is expanded (by clicking the burger icon), and screen size reduced to mobile size, the sidebar hides fine.

But if the sidebar is NOT expanded and the page shrinks, I see about 20 pixels of sidebar shown (believe, this is due to $sidebar-width being used against $mobile-sidebar-width)

Also, sidebar-off-canvas is pushing the components to the right, instead overlaying them.

To fix my above issue, I've done these

Until it is added to the next beta ( #215 ) ,  manually add !default flag to $sidebar-width in your version's custom-variables.scss file

// _custom-variables.scss
$sidebar-width:                     200px !default;

Then override the $sidebar-width variable in your scss file

$sidebar-width: 220px;

To fix the main div being pushed to the right, when sidebar clicked on mobile

// sidebar on the mobile, pushes the main div to the right when clicked
// fix it by setting up the margins to 0
.sidebar-mobile-show .main {
  margin-right: 0 !important;
  margin-left: 0 !important;
}

@mrholek even with recent commit https://github.com/mrholek/CoreUI-Free-Bootstrap-Admin-Template/commit/d104609bafafd68ab4a4856cac73e50f98ace2e9 ... sidebar is still pushing the main div to the right (in mobile view), even though it declared off-canvas. Would you plz reopen this ticket?


python3Packages.pygeos: fix version 0.14 build

Description of changes
The earlier needed patch (./fix-for-geos-3-11.patch) is no longer needed as the test was fixed upstream in pygeos itself. The patch made the build process fail as it tried to apply obsolete changes:
 > applying patch /nix/store/mvw5phq9qpmj0dzaswm53hbanhqqp25r-fix-for-geos-3-11.patch
 > patching file pygeos/tests/test_constructive.py
 > Reversed (or previously applied) patch detected!  Assume -R? [n]
 > Apply anyway? [n]
 > Skipping patch.
 > 1 out of 1 hunk ignored -- saving rejects to file pygeos/tests/test_constructive.py.rej

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

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

2 packages built:

python310Packages.pygeos
python311Packages.pygeos


Compass and delay

Version :
Quests 3.0.0-Naomi
I got 2 issues (not sure if it's really an issue but I prefer post it here as an "information" I guess)
First is with the compass, in fact we've got few players in our server who complain that the compass don't indicate the good location of a 'Location to reach' or a NPC (Citizens2).
The second is with the Delay stage. If you have 2 Delay stages in 2 quests and you are doing both quest at the same time the delay are not independent. If you have 2 hours on first quest and 2 hours on the second one, you must wait 4 hours and both stages will be completed at the same time. And it's no really good I guess ^^
Thanks for your help :D

Compass works fine for me, so you'll need to provide a more specific scenario to test (including a quest from quests.yml would expedite that).
Your second issue was supposedly fixed in #114 - update to the latest dev build

Compass works fine for me too but not for all player which is not normal at all but we've go few permissions' bug on the server and I believe it's the main cause of this bug ^^ (Will see with the permissions' plugin update)
And ... I am a bit lost on Github and I don't find where download the dev build, so can you say me where to find it or a link? It will be really helpful :)
Thanks for your help :D

Closing until valid reason to proceed is found.
Dev builds - http://ci.ac3-servers.eu/job/Quests/


Compatibility issue with "Carry On"

Before I write my very minor bug report, I'd like to congratulate and appreciate your wonderful work on adding a port to forge with this satisfying sound mod. The best of the best, especially in terms of inventory enjoyment.
The issue is that when picking up entities or blocks with the well-renowned "Carry On" mod, it will constantly play the sound(s):
extrasounds:entity.remove.negative
extrasounds:entity.add.negative
This sound for some reason can be described as the ambient portal sounds, it doesn't exactly sound like a sound constantly cutting off and restarting, but merely like I said the portal ambience for the Nether portal. I have no clue why, an idea I thought of was to add configs to be able to toggle specific sounds and post the sound library with a "true" and "false" answer!
When I used "OmegaMute" and did the command to listen, this sound file would constantly spam only after picking up a block.

Thank you for your time!

Actually, since https://github.com/Arborsm/ExtraSoundsForge/commit/53b80c8d26e554220026594339e4adee00aec9a5, the effect sound is disable by default because mods have too many conflict behavior，btw is the entity in extrasounds:entity.remove.negative your typing wrong or?


Add kubernetes deployment/services yaml

This will provide a kubernetes yaml that can be used to deploy LfH to kubernetes and expose the necessary ports.  The one change to existing code was to update default http port from 8080 to 8081 for LfH due to a duplicate port between LfH and another service.

@ibmatc I pulled in your yaml locally and used it to deploy lfh to a free k8 cluster I provisioned on ibm cloud.
Using the node ports you defined, I'm able to reach Kafdrop (port 30000), but I wasn't able to reach the lfh app via 31000 - connection refused.
In your testing, were you able to reach the lfh app?
Thanks for contributing

@ibmatc I pulled in your yaml locally and used it to deploy lfh to a free k8 cluster I provisioned on ibm cloud.
Using the node ports you defined, I'm able to reach Kafdrop (port 30000), but I wasn't able to reach the lfh app via 31000 - connection refused.
In your testing, were you able to reach the lfh app?
Thanks for contributing

@srcarrier Yes - I was able to use the curl commands provided in the LfH documentation to post to Fhir data to the 31000 port. I also have sent Hl7 data via the mllp_send command using the 32000 port.  I'm guessing the difference is that the yaml is deploying the current docker image, and in my testing I was deploying my own image built using the application.properties changes I included in this changeset.  I updated the yaml to point the the 1.0.0 LfH image, but it will need to be updated to work.
I need to discuss with Dixon about his comment regarding using environment variables. Once that issue is resolved your issue should be taken care of too (either with a new docker image or with a reversion from 8081 to 8080.

@ibmatc @srcarrier - I think that we can accept the PR once the following changes are made:

The docker.io/linuxforhealth/connect:1.0.0 image is referenced in the K8's configurations
Environment variables are used in the K8s configuration for LFH_CONNECT_BLUEBUTTON-20_PORT and LFH_CONNECT_BLUEBUTTON-20_CALLBACK_PORT
The port changes to application.properties are rolled back.

@ibmatc @dixonwhitmire it'd be good to have a readme for k8s, like the other containers.
Nothing overly elaborate, but a place we can contribute doc to for deploying l4h to k8 clusters over time.

@ibmatc @dixonwhitmire it'd be good to have a readme for k8s, like the other containers.
Nothing overly elaborate, but a place we can contribute doc to for deploying l4h to k8 clusters over time.

I added a simple readme.md to the pull request.

can we test this on azure or aws?  when we set up travis/jenkins i want to have nightly build tests across all clouds and linuxone/z/os using k8, openshift and docker swarm.

can we test this on azure or aws? when we set up travis/jenkins i want to have nightly build tests across all clouds and linuxone/z/os using k8, openshift and docker swarm.

There is no reason (to my knowledge) that this wouldn't work on other cloud platforms, but I currently do not have access to azure/aws and have no familiarity with the nuances therein, so there could be some hurdles.


Blank (anonymous) snippet causes bot to crash.

Hello there. I believe to have found an unreported bug. Prefix is =
If you leave a snippet blank, then the bot will crash. Prefixes I am using (if this affects anything):
snippetPrefixAnon = =l
snippetPrefix = =lr
Other settings that may be conflicting:
alwaysReply = on
alwaysReplyAnon = on
dbType = mysql
More settings available upon request.
As you can see by the screenshot I am using the 'hello' snippet. I ran the snippet anonymously (=lhello) and normally (=lrhello), both working fine. As soon as I put the command blank (=lr) then the bot crashes. You can see me attempt to run the same command again with no success.

This is the exact error message:
Error: TypeError: object null is not iterable (cannot read property Symbol(Symbol.iterator))
at Client. (/home/runner/modmailp/src/modules/snippets.js:63:32)
at processTicksAndRejections (internal/process/task_queues.js:97:5)
Thank you.

Note: Type of database does not affect it. I tried it with a modmail bot with the same settings but the database was SQLite.

No config needed, just tried it with !! and !!! on their own with the same error.
Error: TypeError: object null is not iterable (cannot read property Symbol(Symbol.iterator))
at Client. (/home/runner/#######/src/modules/snippets.js:63:32)
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:97:5)


Create installation packages for Enso

This task is automatically imported from the old Task Issue Board and it was originally created by Dmitry Bushev.
Original issue is here.

Notes
In order to handle custom enso:// redirect links required for authentication, we need to install .desktop files in appropriate locations.
Packaging is required for Windows and Linux platforms. The dmg file produced for macOS should already contain the required file structure, and we should be able to handle custom URLs
sbt-native-packager should be enough to create those bundles.
Comments:
This is login redirection ticket - possibly a ticket for @murbanczyk. (James Dunkerley - Feb 3, 2023)

@indiv0 @PabloBuchu this is probably relevant for you

anyone knowing sbt can help with this one? I guess this one is blocking windows package from working @mwu-tow @4e6?

anyone knowing sbt can help with this one? I guess this one is blocking windows package from working @mwu-tow @4e6?

I don't think sbt is relevant; this should be done identically as file extension associations.

@mwu-tow is there a way to quickly test if it solves the problem with authentication on Windows and guide @PabloBuchu and @indiv0 here? The problem is that when user tries to login with Google or GitHub account he is not redirected back to application.

Is there a repro scenario that I can easily try?

This issue actually has two parts. The first part concerns the current setup's failure to properly register the custom protocol handler on Windows. However, resolving this does not seem particularly challenging. The second part, which is more interesting, is about handling custom protocol deep links.
Following discussions with @PabloBuchu, @wdanilo, and @sylwiabr, here are some key points:

We support and want to continue supporting multiple instances of the IDE running simultaneously.
We cannot rely on the system browser to send the open-url event to the correct instance of the IDE after opening our custom protocol deep link.
On Windows, this event won't be sent at all. Instead, a new IDE instance will be spawned with the URL as a command-line argument.

Since we want to support multiple IDE instances, we cannot use the single-instance lock mechanism. It is the only cross-instance IPC mechanism available in Electron, so we need to handle IPC ourselves.
A possible solution is for the IDE that directs the user to the system browser to maintain an open WebSocket (or similar) listener on a specific port. This port number will be embedded in the custom protocol deep link. When the system browser opens the deep link, whichever IDE instance receives the deep link will establish a WebSocket connection to the port and forward the URL data. The IDE that opened the system browser will then receive the URL data and handle it, including bringing itself to the foreground.
@JaroslavTulach also asked why we chose to use custom protocol handling in the first place. An alternative might be for the IDE to redirect the browser to its HTTP server. This could potentially eliminate platform-specific differences in handling deep links. However, this approach would require the IDE to run in the background constantly (i.e., it won't be possible to just spawn it from the browser).

@indiv0
What do you think about this? How should I proceed? Additionally, can you provide some pointers on how to control the URL data flow from the IDE to the browser?

another option might be to only ever have a single instance running, and let that single instance control all the windows?
i'm not 100% sure instance ≠ window, but this SO answer seems to indicate that it is
that said, i'm not sure what would be the downsides of doing things this way

@mwu-tow what is the status here? I don't see any progress update/standups.


Update 50-svtav1.sh to the latest commit, after v2.1.1

This commit has passed the official pipeline

It was updated a little over a week ago.
What changed so drastically to need another update?
I see nothing but cosmetic stuff in the commit log since. The CI failures look like just a OSX runner hickup to me.

I'm sorry that I didn't read the commit log carefully, just found it's still v2.1 but the latest is v2.1.1. I'm so sorry to have bothered you.

The version number doesn't reflect much, given it's all straight from git. Though just to make people happy it might be worth setting it.


Where is the "/data/accident_reports/test_report.txt" file?

Dear author, hello! I'd like to know where is the "/data/accident_reports/test_report.txt" file mentioned in the 17th line of the "run_transformation.py" file? Actually there is only one file named "cases.xlsx" in the "data/accident_reports" folder.

@MayDGT