Tensorflow r2.13 build from source: cudnn_frontend_Operation.h:413:16: error: enumeration value ‘CUDNN_POINTWISE_RECIPROCAL’ not handled in switch [-Werror=switch]

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
No
Source
source
Tensorflow Version
r2.13
Custom Code
No
OS Platform and Distribution
Linux Ubuntu 22.04
Mobile device
No response
Python version
3.10
Bazel version
6.2.1
GCC/Compiler version
11.3.0
CUDA/cuDNN version
12.1/<IP_ADDRESS>
GPU model and memory
NVIDIA GeForce 940MX
Current Behaviour?
I was trying to build Tensorflow from source (branch r2.13). During the build the following error happend which halted the build execution:
ERROR: /home/vyepishov/Documents/dev/git/tensorflow/tensorflow/compiler/xla/stream_executor/cuda/BUILD:377:11: Compiling tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc failed: (Exit 1): crosstool_wrapper_driver_is_not_gcc failed: error executing command external/local_config_cuda/crosstool/clang/bin/crosstool_wrapper_driver_is_not_gcc -MD -MF bazel-out/k8-opt/bin/tensorflow/compiler/xla/stream_executor/cuda/_objs/cudnn_plugin/cuda_dnn.pic.d ... (remaining 142 arguments skipped)
In file included from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_Operation.h:37,
from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_OperationGraph.h:36,
from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_Heuristics.h:31,
from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend.h:101,
from tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc:56:
bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_PointWiseDesc.h: In member function ‘int64_t cudnn_frontend::PointWiseDesc_v8::getPortCount() const’:
bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_PointWiseDesc.h:69:16: error: enumeration value ‘CUDNN_POINTWISE_RECIPROCAL’ not handled in switch [-Werror=switch]
69 |         switch (mode) {
|                ^
In file included from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_OperationGraph.h:36,
from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_Heuristics.h:31,
from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend.h:101,
from tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc:56:
bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_Operation.h: In member function ‘cudnn_frontend::Operation_v8&& cudnn_frontend::OperationBuilder_v8::build_pointwise_op()’:
bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_Operation.h:413:16: error: enumeration value ‘CUDNN_POINTWISE_RECIPROCAL’ not handled in switch [-Werror=switch]
413 |         switch (m_operation.pointwise_mode) {
|                ^
Standalone code to reproduce the issue
$ cd ~/Documents/dev/git/tensorflow
$ git checkout r2.13
$ git pull
$ ./configure
$ bazel build --config=mkl --config=opt //tensorflow/tools/pip_package:build_pip_package

Relevant log output
~/Documents/dev/git/tensorflow$ bazel build --config=mkl --config=opt //tensorflow/tools/pip_package:build_pip_package
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=1 --terminal_columns=211
INFO: Reading rc options for 'build' from ~/Documents/dev/git/tensorflow/.bazelrc:
  Inherited 'common' options: --experimental_repo_remote_exec
INFO: Reading rc options for 'build' from ~/Documents/dev/git/tensorflow/.bazelrc:
  'build' options: --define framework_shared_object=true --define tsl_protobuf_header_only=true --define=use_fast_cpp_protos=true --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --noincompatible_remove_legacy_whole_archive --enable_platform_specific_config --define=with_xla_support=true --config=short_logs --config=v2 --define=no_aws_support=true --define=no_hdfs_support=true --experimental_cc_shared_library --experimental_link_static_libraries_once=false --incompatible_enforce_config_setting_visibility
INFO: Reading rc options for 'build' from ~/Documents/dev/git/tensorflow/.tf_configure.bazelrc:
  'build' options: --action_env PYTHON_BIN_PATH=~/Documents/dev/programs/miniconda3/envs/tf/bin/python3 --action_env PYTHON_LIB_PATH=~/Documents/dev/programs/miniconda3/envs/tf/lib/python3.10/site-packages --python_path=~/Documents/dev/programs/miniconda3/envs/tf/bin/python3 --action_env CUDA_TOOLKIT_PATH=/usr/local/cuda-12.1 --action_env TF_CUDA_COMPUTE_CAPABILITIES=5.0 --action_env LD_LIBRARY_PATH=/usr/lib/libreoffice/program:/usr/local/cuda/targets/x86_64-linux/lib:/usr/lib/x86_64-linux-gnu --action_env GCC_HOST_COMPILER_PATH=/usr/bin/x86_64-linux-gnu-gcc-11 --config=cuda
INFO: Reading rc options for 'build' from ~/Documents/dev/git/tensorflow/.bazelrc:
  'build' options: --deleted_packages=tensorflow/compiler/mlir/tfrt,tensorflow/compiler/mlir/tfrt/benchmarks,tensorflow/compiler/mlir/tfrt/jit/python_binding,tensorflow/compiler/mlir/tfrt/jit/transforms,tensorflow/compiler/mlir/tfrt/python_tests,tensorflow/compiler/mlir/tfrt/tests,tensorflow/compiler/mlir/tfrt/tests/ir,tensorflow/compiler/mlir/tfrt/tests/analysis,tensorflow/compiler/mlir/tfrt/tests/jit,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_tfrt,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_jitrt,tensorflow/compiler/mlir/tfrt/tests/tf_to_corert,tensorflow/compiler/mlir/tfrt/tests/tf_to_tfrt_data,tensorflow/compiler/mlir/tfrt/tests/saved_model,tensorflow/compiler/mlir/tfrt/transforms/lhlo_gpu_to_tfrt_gpu,tensorflow/core/runtime_fallback,tensorflow/core/runtime_fallback/conversion,tensorflow/core/runtime_fallback/kernel,tensorflow/core/runtime_fallback/opdefs,tensorflow/core/runtime_fallback/runtime,tensorflow/core/runtime_fallback/util,tensorflow/core/tfrt/eager,tensorflow/core/tfrt/eager/backends/cpu,tensorflow/core/tfrt/eager/backends/gpu,tensorflow/core/tfrt/eager/core_runtime,tensorflow/core/tfrt/eager/cpp_tests/core_runtime,tensorflow/core/tfrt/gpu,tensorflow/core/tfrt/run_handler_thread_pool,tensorflow/core/tfrt/runtime,tensorflow/core/tfrt/saved_model,tensorflow/core/tfrt/graph_executor,tensorflow/core/tfrt/saved_model/tests,tensorflow/core/tfrt/tpu,tensorflow/core/tfrt/utils,tensorflow/core/tfrt/utils/debug
INFO: Found applicable config definition build:short_logs in file ~/Documents/dev/git/tensorflow/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file ~/Documents/dev/git/tensorflow/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:cuda in file ~/Documents/dev/git/tensorflow/.bazelrc: --repo_env TF_NEED_CUDA=1 --crosstool_top=@local_config_cuda//crosstool:toolchain --@local_config_cuda//:enable_cuda
INFO: Found applicable config definition build:mkl in file ~/Documents/dev/git/tensorflow/.bazelrc: --define=build_with_mkl=true --define=enable_mkl=true --define=tensorflow_mkldnn_contraction_kernel=0 --define=build_with_openmp=true -c opt
INFO: Found applicable config definition build:opt in file ~/Documents/dev/git/tensorflow/.tf_configure.bazelrc: --copt=-Wno-sign-compare --host_copt=-Wno-sign-compare
INFO: Found applicable config definition build:linux in file ~/Documents/dev/git/tensorflow/.bazelrc: --define=build_with_onednn_v2=true --host_copt=-w --copt=-Wno-all --copt=-Wno-extra --copt=-Wno-deprecated --copt=-Wno-deprecated-declarations --copt=-Wno-ignored-attributes --copt=-Wno-array-bounds --copt=-Wunused-result --copt=-Werror=unused-result --copt=-Wswitch --copt=-Werror=switch --copt=-Wno-error=unused-but-set-variable --define=PREFIX=/usr --define=LIBDIR=$(PREFIX)/lib --define=INCLUDEDIR=$(PREFIX)/include --define=PROTOBUF_INCLUDE_PATH=$(PREFIX)/include --cxxopt=-std=c++17 --host_cxxopt=-std=c++17 --config=dynamic_kernels --experimental_guard_against_concurrent_changes
INFO: Found applicable config definition build:dynamic_kernels in file ~/Documents/dev/git/tensorflow/.bazelrc: --define=dynamic_loaded_kernels=true --copt=-DAUTOLOAD_DYNAMIC_KERNELS
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/tensorflow/runtime/archive/7d879c8b161085a4374ea481b93a52adb19c0529.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/llvm/llvm-project/archive/dc275fd03254d67d29cc70a5a0569acf24d2280d.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/google/benchmark/archive/f7547e29ccaed7b64ef4f7495ecfff1c9f6f3d03.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://mirror.bazel.build/github.com/bazelbuild/rules_cc/archive/081771d4a0e9d7d3aa0eed2ef389fa4700dfb23e.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/abseil/abseil-cpp/archive/b971ac5250ea8de900eae9f95e06548d14cd95fe.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/google/XNNPACK/archive/b9d4073a6913891ce9cbd8965c8d506075d2a45a.zip failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/openxla/stablehlo/archive/43d81c6883ade82052920bd367c61f9e52f09954.zip failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/google/boringssl/archive/c00d7ca810e93780bd0c8ee4eea28f4f2ea4bcdc.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/nvidia/nccl/archive/v2.16.5-1.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/pybind/pybind11_abseil/archive/2c4932ed6f6204f1656e245838f4f5eae69d2e29.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/NVIDIA/cudnn-frontend/archive/refs/tags/v0.8.zip failed: class java.io.FileNotFoundException GET returned 404 Not Found
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/openxla/triton/archive/1627e0c27869b4098e5fa720717645c1baaf5972.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
INFO: Analyzed target //tensorflow/tools/pip_package:build_pip_package (631 packages loaded, 44329 targets configured).
INFO: Found 1 target...
ERROR: ~/Documents/dev/git/tensorflow/tensorflow/compiler/xla/stream_executor/cuda/BUILD:377:11: Compiling tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc failed: (Exit 1): crosstool_wrapper_driver_is_not_gcc failed: error executing command external/local_config_cuda/crosstool/clang/bin/crosstool_wrapper_driver_is_not_gcc -MD -MF bazel-out/k8-opt/bin/tensorflow/compiler/xla/stream_executor/cuda/_objs/cudnn_plugin/cuda_dnn.pic.d ... (remaining 142 arguments skipped)
In file included from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_Operation.h:37,
                 from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_OperationGraph.h:36,
                 from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_Heuristics.h:31,
                 from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend.h:101,
                 from tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc:56:
bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_PointWiseDesc.h: In member function ‘int64_t cudnn_frontend::PointWiseDesc_v8::getPortCount() const’:
bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_PointWiseDesc.h:69:16: error: enumeration value ‘CUDNN_POINTWISE_RECIPROCAL’ not handled in switch [-Werror=switch]
   69 |         switch (mode) {
      |                ^
In file included from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_OperationGraph.h:36,
                 from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_Heuristics.h:31,
                 from bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend.h:101,
                 from tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc:56:
bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_Operation.h: In member function ‘cudnn_frontend::Operation_v8&& cudnn_frontend::OperationBuilder_v8::build_pointwise_op()’:
bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_Operation.h:413:16: error: enumeration value ‘CUDNN_POINTWISE_RECIPROCAL’ not handled in switch [-Werror=switch]
  413 |         switch (m_operation.pointwise_mode) {
      |                ^
cc1plus: some warnings being treated as errors
Target //tensorflow/tools/pip_package:build_pip_package failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 1819.010s, Critical Path: 107.08s
INFO: 5429 processes: 2198 internal, 3231 local.
FAILED: Build did NOT complete successfully

Hi @spamming4 ,
The issue might be due to the cuDNN version. TF2.13 tested on cuDNN 8.6 and CUDA 11.8.
I am attaching a reference ticket #60398 of similar problem and where user able to get it resolved downgrading to cuDNN 8.6 version. Please refer to official tested build configurations.
Please also make a note that starting from Tf21.3 TF builds uses Clang 16.0 as compiler instead of GCC.


STORM-2506: Print mapping between Task ID and Kafka Partitions

@HeartSaVioR : please review at your convenience.

+1 Thanks for the work.


Switch to directory based rollover

Switches from indexed message size rollover to a size-on-disk rollover

I like this! Very simple and works would work effectively
Two small nits:

Maybe directorySize variable doesn't convey the meaning? Maybe directorySizeCalculator ?
Can we add a one assert test for the gauge that we added?


fomatting error in the contributing.md

There is a formatting error in the contributing.md. This following line is not rendered correctly.
(Example: feat//gcp-compute-engine-support)
DOD

mark the line as code to fix the error. for example, change Example: feat//gcp-compute-engine-support -> Example: feat/<issue-id>/gcp-compute-engine-support

Please assign me this issue. Really happy to get a chance to contribute.


Not possible to sync after last change and updated CLI, Error 503

Getting 503 when trying to sync (tested with 3 other ppl). Also it promted to update the CLI which only worked with npm not yarn but I guess this is irrelevant to the issue.

Hi, apologies for the delayed response over the weekend, and thanks for reporting. We indeed had a service disruption at that time - sorry for that, we're working on improving our operational infrastructure. Please let me know if you're continuing to see any further issues.

Hi, no problem, yes it is working now :)


Tomopy: add fabio dep to make the package installable into collection-2018-3.x

Per our yesterday's discussion with @tacaswell.

Thanks for merging!

ok, add collection as a build dep!

OK, will try. I removed linux-64/tomopy-1.1.2-py36_2.tar.bz2 from https://anaconda.org/lightsource2-tag/tomopy/files and alexandria.


GH-3199 issue templates for bug reports and feature requests

GitHub issue resolved: #3199
Briefly describe the changes proposed in this PR:

adds issue templates with forms for bug reports and feature requests
adds an obvious direct link to the discussion forum for questions, comments, etc.
still possible to create a "blank" new issue if none of the templates fit (e.g. for our own internal tasks).
bumped version of the new-issue github action (hopefully fixes the problems I ran into earlier, but this is as yet untested)

PR Author Checklist (see the contributor guidelines for more details):

[ ] my pull request is self-contained
[ ] I've added tests for the changes I made
[ ] I've applied code formatting (you can use mvn process-resources to format from the command line)
[ ] I've squashed my commits where necessary
[ ] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change

@hmottestad @barthanssens @aschwarte10  what do we think? I want to encourage people to report issues with sufficient information for us to act on them, so this is very intended as "guide rails". The intent is to make it easier to report a problem, rather than harder.
I personally think this is an improvement, but it's less "free format" than just a blank issue of course (although if you really want you can also still do that).


dataUri as string rather than byte32.  This allow longer data URI inc…

…luding popular cloud based data Uris

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   96.49%   96.49%
=======================================
  Files           4        4
  Lines         114      114
  Branches       24       24
=======================================
  Hits          110      110
  Misses          4        4

Impacted Files
Coverage Δ

contracts/LinniaRecords.sol
98.24% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

what is the URI schema?

How are the data URI computed? How are they resolved? For instance, what makes an IPFS URI different than an HTTP URI? What are the other possible protocols? Why string not bytes? Will this break IPFS URIs? Currently the URI are IPFS paths which is exactly 32bytes, but aren't printable ASCII. If they are stored in string, they will be 33 bytes (incl. NUL), which is 2 words.

assertRevert does not catch out of gas.

If the test runs out of gas they should fail.  assertRevert ensures this failure.  expectThrow is too broad.

Fixes #40


Ungron to ignore lines saying --

Grep can search for lines immediately preceding or following the matched line, using grep -A, grep -B or grep -C. However the output of these includes separator lines that read --. This output currently produces an error when you try to ungron the result.
Since grep is a primary use for gron, I believe it would be a useful enhancement for it to accept a common output of grep. It doesn't need to do anything with the lines, only match and exclude them.
For example:
$ gron file.json | grep -A 1 'ktoe per year'

json.resources.geothermal.production.units = "ktoe per year";
json.resources.geothermal.production.value = 2007.738525390625;
--
json.resources.hydropower.production.units = "ktoe per year";
json.resources.hydropower.production.value = 9038.80078125;
--
json.resources.wind.production.units = "ktoe per year";
json.resources.wind.production.value =<PHONE_NUMBER>175781;

Sounds like a good call to me!
It's workaround-able with a grep -v '^--' before the ungron, but it seems like it'd be a common enough occurrence to be worthwhile implementing.
I'll have a look at implementing it this evening, unless someone beats me to it of course :)
Thanks for the suggestion!

@marcusatbang sorry it took so long!
Version 0.3.2 should now not have this problem :)

That was remarkably fast. Well done.

Confirmed as working.

Fantastic! Glad to hear it :)


subscriptions UPDATE merge operational data

An external tool that implements configured subscriptions can fill other leaves by subscribing the operational data with the SR_SUBSCR_OPER_MERGE flag. For instance, the
configured-subscription-state field defined in the RFC 8639 Link:https://datatracker.ietf.org/doc/html/rfc8639.

Okay, no problem, makes sense.


Strange file formatting for JSX

Possibly related to #47446.
let x =

;

Format.
Expected
No change
Actual
let x =

;

I couldn't repro this or the other one

If you're in VS Code and have an anonymous buffer whose language starts as "TypeScript" and switches to "TypeScript React", the formatter doesn't adapt correctly. I can't tell if that's a TypeScript or VS Code bug.

The server log should indicate whether TS Server got the message that the file should be treated as TSX

The log does indicate that we got the right updateOpen request.
[Trace  - 02:19:34.109] <semantic> Sending request: updateOpen (4643). Response expected: yes. Current queue length: 2
Arguments: {
    "changedFiles": [],
    "closedFiles": [],
    "openFiles": [
        {
            "file": "^/untitled/Untitled-4",
            "fileContent": "let x =
\r\n
",
            "projectRootPath": "c:\\Users\\ME\\scratch\\example",
            "scriptKindName": "TSX"
        }
    ]
}

TypeScript also knows to interpret that code as JSX, but the formatter apparently doesn't.


Remove --name flag from app remove command

Closes: #717

What changed?
The flag --name has been removed from the command "wego app remove "

Why?
No need for this flag as the name of the application can be written as the first command argument without the need for the flag.

How did you test it?
The output after the fix:
 ./wego app remove --help
Removes an application from a wego cluster so it will no longer be managed via GitOps

Usage:
  wego app remove [--private-key <keyfile>] <app name> [flags]

Examples:

  \# Remove application from wego control via immediate commit
  wego app remove podinfo

Flags:
      --dry-run              If set, 'wego remove' will not make any changes to the system; it will just display the actions that would have been taken
  -h, --help                 help for remove
      --private-key string   Private key to access git repository over ssh

Global Flags:
      --namespace string   gitops runtime namespace (default "wego-system")
  -v, --verbose            Enable verbose output

>> ./wego app remove --name
Error: unknown flag: --name

Release notes

Documentation Changes

This has been fixed during the shift to the gitops cmd.


Add reference to sst-documentation repository for cookie information

Changed

add reference in documentation to SST-documentation repository regarding cookie usage and setup

Approved, nice job on improving the documentation.


Jellyfin tray deletes itself after a system reset.

No idea why, I closed the app, restarted windows, went to boot the tray from the shortcut, but the exe was missing.

If you downloaded the update before 3:10 AM EDT/7:10 AM UTC, then please redownload it and install over your existing version. It will add the better tray app that shouldn't have that problem.

Is this still happening for you @Darkwolf1515 or is everything okay now? 🙂

Is this still happening for you @Darkwolf1515 or is everything okay now? 🙂

Ah yeah, sorry, it works just fine, thank you, Ill close it.


Consider splitting the script loader into a separate package or file

First, let me say that, so far, I love this package. I'm just starting to wire it up, but it seems that you've given this a lot of thought.
One thing I've run into (I hesitate to call it an issue) is that the script loader that is part of the package seems to be completely separate from the cookie consent functionality. There isn't any real reason for them to share an object. While loading a script is a common thing we want to do in response to consent being granted, there are a lot of ways to accomplish this. As far as I can tell, I can include cookieconsent either entirely or not at all in my code; I can't exclude the script loader, and while it isn't a huge feature, it is some unnecessary code.
I suggest that you consider splitting the script loader into another package (or even find another existing package that serves that purpose). You could still suggest loading that package, and use it in your examples, but I think that both packages would benefit from a separation of concerns.
Of course I may be missing some way they are integrated that makes this entire suggestion moot, in which case, please disregard this suggestion and carry on. :)

Hi @karptonite,
Interesting point, as I myself have asked this question!
The main reason why the plugin's file size is bigger than necessary, is mainly because it was designed with the idea of being “very cross-browser”, in fact it uses — at most — es5 functions/syntax. We could argue if it's worth supporting older/legacy browser nowadays, but I'm going to leave that topic for another day, just to not drag this discussion too much.
Regarding only the loadScript() function:

for a developer it is something not necessary to have — just as you stated — and I totally agree with you!
for a newbie — though — it could be of some help (maybe ...)

I think the loadScript function is small enough (40/1700 lines) to not be too much of a “dead weight”. Another problem is that the removal of this feature is surely a “breaking change”, and could throw off some less expert users (as there isn't even a proper debug/error handling component)
The true upgrade to reduce the overall file size would be to just drop support for older browsers and use es6+ syntax (clearer, shorter, neater), perhaps in a future v3.0 release!
Note:
there's also an internal function called _loadScripts which — unlike the above discussed .loadScript() — is required to properly manage third party scripts

Makes sense to me!
Dropping support for older browsers (or any feature) is definitely a 3.x release. I agree that loadScript() is not a lot of code, and it isn't really a serious concern. I think I had it in my mind that _loadScripts() could also be removed, but while that is also a feature that could theoretically be done without (for users that, for example, handle the loading of scripts manually with loadScript()), splitting it out would be a lot more trouble, and would make the package(s) more difficult to use.

Yes, _loadScript() makes sure that the (configured) scripts run in the same order the user declared them; it can manage any js snippet without having to:

move them inside the onAccept/onChange methods
adapt the code

This type of “loose” dependency also allows for a smoother transition to another solution (should this plugin not fit their needs)
I'm glad I didn't merge the iframemanager into this plugin (some users had asked for it to be embedded) as that would have been quite a big chunk of dead weight for many users!


Added new eslint-rule on eslint-plugin 5.3.0-PR.42-179d

Closes #https://github.com/checkdigit/eslint-plugin/issues/29.

closing


Reimplement with smooth scroll polyfill?

https://github.com/iamdustan/smoothscroll
No easing though...

thanks @willhoag , that polyfill fixed my scrolling issues in Microsoft Edge

Great! Glad to hear it!


any app to test this broker?

do you know any app to test this @frodenas

I believe I tested the broker using the aws ruby sdk, but I never published an example app to the repo.
Happy to accept a PR with a sample app, it will be very useful.


Final report creation step is mixed with g.part1()

Hi Vincent:  I ran part 1 through part 5 sequentially on a cluster.    I encountered a couple of issues here.

When overwrite=TRUE, g.part1() is accidentally activated; therefore I can only set overwrite=FALSE.  However, in order to rerun this step to create new reports, I need to manually remove the "file summary reports" folder to proceed.
At the end of this step, there is following error:
Error in xy.coords(x, y, xlabel, ylabel, log) :
'x' and 'y' lengths differ
viewingwindow can only be set to 1, midnight to midnight.  If I set it to 0 hoping to change to noon to noon, following error appears:
Generate visual reports
[1] "File filename.cwa"
Error in g.plot5(metadatadir = metadatadir, dofirstpage = dofirstpage,  :
object 'nightsi' not found

Following is the code for creating the reports. Please note all the previous steps are completed at this point.
Thanks so much for your help.
g.shell.GGIR(
#=======================================
#-------------------------------
General parameters
#-------------------------------
datadir = datadir,
outputdir = outputdir,
f0 = 1,
f1 = c(),
file start and file end
overwrite = FALSE,
do.imp = FALSE,
#-----------------------------------
Report generation
#-------------------------------
Key functions: Generating reports based on meta-data
do.report = c(2, 4, 5),
visualreport = TRUE,
dofirstpage = TRUE,
viewingwindow = 1
) # end of g.shell.GGIR()

Do not set mode  = 1 if you only want to regenerate the report. If mode = c() then part1 does not start. Note that reports are always created, even if overwrite = FALSE.
& 3. This is too little information for me. Can you narrow down when / where this happens? Does the file have at least 1 full day of measurement with the monitor being worn between noon-noon and midnight-midnight?

Hi Jian,
The default for mode is mode=c(1,2)
If you do not want the part 1 and 2 to be processed again you will need to include mode=c() in your function call.
Best, Vincent

Vincent,
When overwrite = TRUE, part 1 is still activated even if mode = c()
if overwrite = FALSE, part 1 is not activated even if don't set mode

when overwrite = FALSE (which is the only way to avoid trigging part 1 reran, mode = c() won't make a difference), some reports are created but reports within "file summary reports" folder will not be recreated unless this folder is manually removed first.
Yes, my samples have over 10 days of good data collection.

Hi JIan,
We discussed this over e-mail, but for reference purposes for other users here a summary:
When making a repeated call to function g.shell.GGIR  make sure arguments overwrite and mode are always provided, because otherwise they use default values that may or may not align with what you want to achieve. It is better to just explicitly provide them.
Multiple visual reports are created by GGIR. Below I will focus on the visual report per measurement that has histograms on the first page. The following logic applies:
Setting mode =c( ), visualreport = TRUE, and overwrite = TRUE will create visual reports from the previously generated milestone files inside the 'meta' folder,  and overwrite the existing files inside the 'results/file summary reports' folder, if any.
Setting mode =c( ), visualreport = TRUE, and overwrite = FALSE will create visual reports from the previously generated milestone files inside the 'meta' folder, but only for files that do not have a report in the 'results/file summary reports' folder.
Setting mode =c(1, 2, 3, 4, 5) or not providing the mode argument, visualreport = TRUE, and overwrite = TRUE will re-run part 1-5, overwrite the corresponding milestone files in the 'meta' folder, and will create visual reports overwriting the existing reports inside the 'results/file summary reports' folder.
Note: Argument do.visual does not relate to the report discussed here, but relates to the horizontal bar plot produced in part 4 and stored in visualisation_sleep.pdf. If you want this file to be generated for a full dataset then run part 4 in series (non-parallel). It will go wrong if GGIR is applied in parallel, because than the parallel processes will overwrite the same file visualisation_sleep.pdf for each batch, unless you tell your parallel processing script to store each batch in a separate output folder.
Feel free to re-open this issue if you continue to experience problems with this.


Congo matching pairs loads wrong set of stimuli

Describe the bug
The 16 cards of a matching pairs board are compressed by two sets of 8 stimuli, meaning 8 matches/pairs. In the original condition of the game, the 2 sets of cards are identical, but in the degradation conditions, the first set of 8 cards are original stimuli, and the second set of 8 cards are degraded stimuli. Now, the degradation board will also load the same sets of stimuli occasionally, which means there are no degradation stimuli loaded, but only two sets of original stimuli. This behavior happens randomly when refresh the browser with a degradation game link, sometimes it loads both original and degraded stimuli, sometimes it is two sets of original stimuli, it does not seem to have something to do with Cache. When the cache is cleaned, this bug still appears.
To Reproduce
I don't know if it happens on Mac, with the tough book, if you go to any degraded link, and load the game, it can happen anytime.
Expected behavior
The board will load the correct sets of stimuli in the degradation condition, meaning loading 1 set of original stimuli and 1 set of corresponding degraded stimuli to form the game board.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
-OS: Windows
-Browser: Firefox 123.0 / Edge 122.0.2365.59
Context

see #817

@Lia-JX-Li , I see that we've not updated the logic for selecting stimuli for the matching_pairs_lite version. So there is still some logic that presupposes we have a playlist with originals, and two sets of degradations mixed. We'll need to double-check that we redesign the selection of stimuli for matching_pairs_lite, with correct assumptions about the playlists coupled to the Congo experiments.


add option for user to get multiple permits (with pre-existing permits in the household)

Extension from #284

@EUzkuraityte @sanjaypoyzer shouldn't this be titled
"User needs to get multiple permits (with pre-existing permits in the household)"
and the solution ("add option for...") put in the issue description/comments?

Hi @lizziebruce yup, made the changes


Removed await in return of User.from_code() for FastAPI / Flask support

Over the weekend, I was mentoring at a hackathon, and a team was trying to use this library. We couldn't figure out how to get the callback from Spotify working, so I poked around the source.
User.from_code() was always returning a coroutine, even when awaiting the function or using import spotify.sync as spotify. Removing await from return await cls.from_token(client, token, refresh_token) seemed to do the trick.
I hope this change doesn't break other functionality. Everything passed when I ran pytest, I suspect User.from_code() might not be covered. I tried writing a new unit test but wasn't sure how to get a valid code without spinning up a web server.
Admittedly I'm not as comfortable with async as I'd like to be, but now await spotify.User.from_code(...) seems to work. Below is an example implementation in FastAPI:
from fastapi import FastAPI, Response, Cookie
import spotify, random, string
from starlette.responses import RedirectResponse
from typing import *
from base64 import b64encode

async def from_code(
        cls, client: "spotify.Client", code: str, *, redirect_uri: str,
    ):
        """Create a :class:`User` object from an authorization code.

        Parameters
        ----------
        client : :class:`spotify.Client`
            The spotify client to associate the user with.
        code : :class:`str`
            The authorization code to use to further authenticate the user.
        redirect_uri : :class:`str`
            The rediriect URI to use in tandem with the authorization code.
        """
        route = ("POST", "https://accounts.spotify.com/api/token")
        payload = {
            "redirect_uri": redirect_uri,
            "grant_type": "authorization_code",
            "code": code,
        }

        client_id = client.http.client_id
        client_secret = client.http.client_secret

        headers = {
            "Authorization": f"Basic {b64encode(':'.join((client_id, client_secret)).encode()).decode()}",
            "Content-Type": "application/x-www-form-urlencoded",
        }

        raw = await client.http.request(route, headers=headers, params=payload)
        token = raw["access_token"]
        refresh_token = raw["refresh_token"]

        return cls.from_token(client, token, refresh_token)

app = FastAPI()
CLI_ID = 'id'
CLI_SEC = 'secret'
SPOTIFY_CLIENT = spotify.Client(CLI_ID, CLI_SEC)
REDIRECT_URI: str = 'http://localhost:5000/spotify/callback'
OAUTH2_SCOPES: Tuple[str] = ('user-top-read',)
OAUTH2: spotify.OAuth2 = spotify.OAuth2(SPOTIFY_CLIENT.id, REDIRECT_URI, scopes=OAUTH2_SCOPES)
SPOTIFY_USERS: Dict[str, spotify.User] = {}

@app.get('/spotify/callback')
async def spotify_callback(code : str):
    key = ''.join(random.choice(string.ascii_uppercase) for _ in range(16))
    SPOTIFY_USERS[key] = await from_code(
        spotify.User,
        SPOTIFY_CLIENT,
        code,
        redirect_uri=REDIRECT_URI,
    )
    response = RedirectResponse(url="/")
    response.set_cookie(key="spotify_user_id", value=key)
    return response

@app.get("/")
async def index(spotify_user_id : Optional[str] = Cookie(None)):
    print('spotify_user_id:', spotify_user_id)
    if spotify_user_id is None:
        return RedirectResponse(url=OAUTH2.url)
    return "Success!"

Thanks for PR'ing this!

Everything passed when I ran pytest

iirc I was really lazy with the tests so I wouldn't rely on pytest much (my testing strategy was to just write code that used whatever functionality I was testing and if the output looked alright then great)

Below is an example implementation in FastAPI:

Awesome! I'd actually encourage you to submit another PR that adds this to the /examples subdirectory since it may be useful for others :)

Thanks for PR'ing this!

Everything passed when I ran pytest

iirc I was really lazy with the tests so I wouldn't rely on pytest much (my testing strategy was to just write code that used whatever functionality I was testing and if the output looked alright then great)

Below is an example implementation in FastAPI:

Awesome! I'd actually encourage you to submit another PR that adds this to the /examples subdirectory since it may be useful for others :)


Is possible set versioning on route as optional?

Hey there!
I have set an api versioning on my routes and it's working smooth:
using System.Net;
using Microsoft.AspNetCore.Mvc;

namespace xpto.Controllers
{
    [Route("{version:apiVersion}/[controller]")]
    [ApiVersion("1.0")]
    [ApiController]
    public class HealthController : ControllerBase
    {
        [HttpGet]
        [ProducesResponseType((int)HttpStatusCode.OK)]
        public IActionResult Get()
        {
            return Ok();
        }
    }
}

            services.AddApiVersioning(config =>
            {
                config.DefaultApiVersion = new ApiVersion(1, 0);
                config.AssumeDefaultVersionWhenUnspecified = true;
                config.ReportApiVersions = true;
            });

Now my routes only works If I send the api version http://localhost:500/1.0/health. Is there any way to let the version as optional parameter? I would like to keep calling http://localhost:500/health and handle it as 1.0 default version.
Thank in advance.

No... And yes.
This is one of the many issues of versioning by URL segment aside from being the least RESTful. Remember that the path, the whole path, is the identifiier of a resource. It, therefore, makes sense why you can't have an optional parameter in the middle. More technically, the routing system only allows optional parameters at the end of the path. That is intrinsic to ASP.NET and transcends API Versioning.
The way you can make it work is to define 2 routes: one with the version and one without. Essentially there are 2 indentifiers. The drawback is this second, default route may need to float around if you change what the default is. There's more details in the known limitations section of the wiki.
I'd be remiss if I didn't point out that this is the only style of versioning that has this problem. All other methods have stable paths across versions. If you have the option, consider an alternate style.

Did this sufficiently answer your question? Did you have additional questions?

Hey there!
Sorry for lately answer!
yea, it's sufficient and thank for explaning :)


Create some sample transactions with dummy data

I have started making a very, very basic version of this to render out different transaction pages.

See my PR @aulneau https://github.com/blockstack/blockstack-explorer-api/pull/38


'De buurtbatterij'

During sprint planning meetings (I cannot exactly remember when) Hans Verberne asked if it would be possible to simulate a 'buurtbatterij'. For now it seems possible to install a stationary battery for storage  of surplus of solar energy. A 'buurtbatterij' however should in practice also be capable of peaksheaving and 'dalvulling' for any source of electricity.
I foresee this will be not as easy as 'turning a switch in ET-Moses' because it needs a new flex strategy.  Can we discuss this coming monday. For now I just write down an idea I have now.

Load battery when demand doesn't exceeds max capacity of network or when production exceeds max capacity of network
Unload battery when demand exceeds  max capacity of network or when production doesn't exceeds max capacity of network.

@ChaelKruip can this be closed?


Allow NettyServer to use its own ssl impl

Previously, the Netty code was forced to use the same SSL impl as the
router. This adds a config to let NettyServer use a different SSLFactory
implementation. This can be used to leverage the performance benefits of
NettySslFactory within the HTTP server, without affecting the router,
which has some compatability issues with Netty's OpenSSL-based
SSLEngine.

Codecov Report

Merging #1117 into master will decrease coverage by 0.07%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1117      +/-   ##
============================================
- Coverage     68.51%   68.43%   -0.08%
+ Complexity     5006     5004       -2
============================================
  Files           390      390
  Lines         31406    31412       +6
  Branches       4021     4023       +2
============================================
- Hits          21517    21498      -19
- Misses         8823     8841      +18
- Partials       1066     1073       +7

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com.github.ambry/config/NettyConfig.java
100% <100%> (ø)
1 <0> (ø)
:arrow_down:

...java/com.github.ambry.rest/NettyServerFactory.java
100% <100%> (ø)
12 <11> (+2)
:arrow_up:

...b.ambry.network/BlockingChannelConnectionPool.java
62.6% <0%> (-3.05%)
7% <0%> (-1%)

...github.ambry.rest/AsyncRequestResponseHandler.java
88.59% <0%> (-2.29%)
23% <0%> (ø)

...in/java/com.github.ambry.store/BlobStoreStats.java
71.9% <0%> (-1.24%)
106% <0%> (-1%)

...ain/java/com.github.ambry.router/PutOperation.java
90.68% <0%> (-0.72%)
107% <0%> (-2%)

...in/java/com.github.ambry.network/SocketServer.java
82.78% <0%> (-0.41%)
13% <0%> (ø)

Continue to review full report at Codecov.

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


[Fix-6632]fix DateTimeParseException: Text '1635474935597' could not be parsed at index 0

Purpose of the pull request
this PR close #6632

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Fixed #7205


⚠ Fortnite has degraded performance

In fff5f86, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1639 ms

Resolved: Fortnite performance has improved in 3039924.


command line completion appends unwanted forward slash

I'm using nvim-cmp.
Here is the issue I'm facing;
In the command line completion for :  if I start with / or . , path source completion gets triggered. If I tab through the entries to a directory, a trailing / is not appended, and when I type in the trailing / , then I can autocomplete the files and directories contained in the directory I just appended / to.
But if I don't start with . or / then cmdline source completion is used, and when I select a directory, a trailing / is appended and  I have to delete it and retype it to autocomplete files and directories on the next level.  Adding a second / starts autocomplete using the path source from the root directory, /.
One think I've tried doing is disabling the cmdline source for certain commands, which is not ideal;
So my question is; does anyone know how to stop the cmdline source for nvim-cmp  to not append trailing slashes to directory names? Or some more elegant way to make it autocomplete into the next directory level?
here is a minimal config to test what I'm talking about
    local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
    if not vim.loop.fs_stat(lazypath) then
      vim.fn.system({
        "git",
        "clone",
        "--filter=blob:none",
        "https://github.com/folke/lazy.nvim.git",
        "--branch=stable", -- latest stable release
        lazypath,
      })
    end
    vim.opt.rtp:prepend(lazypath)

    require("lazy").setup(
      {
        {
          "hrsh7th/nvim-cmp",
          dependencies = {
            { "hrsh7th/cmp-path",   },
            { "hrsh7th/cmp-cmdline",},
            {
              "L3MON4D3/LuaSnip", -- tag = "v<CurrentMajor>.*",
              dependencies = { "rafamadriz/friendly-snippets" },
              config = function()
                require("plugin_configs.LuaSnip")
              end,

              event = "VeryLazy"
            },
            { "saadparwaiz1/cmp_luasnip" }
          },
          config = function()
            local cmp = require 'cmp'
            cmp.setup.cmdline(':', {
              mapping = cmp.mapping.preset.cmdline(
                {
                }),
              sources = cmp.config.sources({
                { name = 'path' }
              }, {
                { name = 'cmdline' }
              })
            })
          end,
        },
      }
    )
    vim.cmd([[colorscheme elflord]])

Thanks in advance for any insight!

fixed in https://github.com/hrsh7th/cmp-cmdline/commit/d250c63aa13ead745e3a40f61fdd3470efde3923


Allow Multiple Lines in "Find" and "Search And Replace"

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.
[X] I am only requesting a single feature. Multiple changes should be split into individual requests, with links between them.
[X] I believe my requested feature will be generally applicable to other users, not just me: it is not uniquely tied to my personal workflow.

🔖 Feature description
Enhance the "Find" and "Search and Replace" to support multi-line queries.
✔ Solution
Proposed Solution

Multi-Line Input Box: Enable the "Find" and "Search and Replace" modals or panes to accept multi-line input through a dedicated multi-line input field.
Paste Multi-Line Text: Allow users to paste text with multiple lines directly into the "Find" and "Search and Replace" input fields without stripping line breaks.
Keyboard Shortcut for New Line: Introduce a keyboard shortcut (e.g., Shift + Enter) to create a new line within the input fields when they are focused.
Toggle Option for Expanded Fields: Provide a toggle option or button to expand the input fields for multi-line queries.

❓ Alternatives
No response
📝 Additional Context
No response

Hi, What has this got to do with the Tasks plugin?

Closing as nothing to do with the Tasks plugin.


Feat: Make Otp and Profile page compatible to dark mode and add responsiveness

Describe the feature
Currently, the OTP and Profile pages lacks the responsiveness and compatibility with dark mode....I would like to fix these things
Add ScreenShots

Record

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

@mdazfar2 Please assign me this issue

@Asymtode712 you are assigned


Minor rearrangement of the registration of the tracer

This is to make it easier to start using delegating tracers, i.e.
tracers that wrap other tracers

@thegreystone thanks


Jesse: Watch lifeinfocus.fl(@gmail.com)?

Jesse requests the watch of the watch_keyword lifeinfocus\.fl(@gmail\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

lifeinfocus\.fl(@gmail\.com)? has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


How to Add our magento applications login to anvil connect

Hi Team,
I just installed anvil connect in my ubuntu system and i am getting the screen like https://prnt.sc/g68hc1. Now i need to add my applications to anvil connect, please send the process for how to add applications to anvil connect.
Thanks.

Hi @avinashsamala,
You'll want to start off by reading our Getting Started documentation. Specifically you'll want to take note of our CLI documentation as well as this is the primary method of interacting with the connect service.
If you are still having further trouble then please @mention me on our gitter channel.

Hi EternalDeiwos,
I followed same steps but still it showing like https://prnt.sc/g68hc1 only. can u help me on this issue.
Thanks.

Hi @avinashsamala,
Connect does not have a web-based management interface, management tasks (such as adding clients) are done through our CLI.


Cannot use imported sound with Metasounds

Hi,
After importing sound files and trying to set wave parameter for a Metasound, I get an error: "Failed to set invalid parameter '%s' in asset '%s': Either does not exist or is unsupported type". This is probably due to UMetaSoundSource::IsParameterValid() returning false for given input object. Everything works for "traditional" sound wave objects.

Looks like UImportedSoundWave is missing REGISTER_METASOUND_DATATYPE.

Hi. MetaSounds doesn't support procedural sound waves, which are used as the basis for the plugin's imported sound wave. I plan to implement this support at some point.
Please follow the news regarding this plugin in our telegram chat: https://t.me/georgydev

There is no workaround for now, there is a plan to make a support for it soon

Will presumably be supported starting from 5.2 (depending on whether Epics approves my PRs): https://github.com/gtreshchev/RuntimeAudioImporter/wiki/How-to-use-with-MetaSounds

I contacted one of the audio team at Epic Games and he told me that they have plans in version 5.3.0 for these PRs, so it's being postponed for now

This should already be supported since the engine 5.3.0 is out here, but in order to support the MetaSounds, you should switch to the metasounds branch in this repo (https://github.com/gtreshchev/RuntimeAudioImporter/tree/metasounds) and download the plugin from there. I'll probably create a separate Marketplace-specific version of the plugin with MetaSounds support enabled by default at some point.


Parking Space Detection Based on Top-View

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title : Parking Space Detection Based on Top-View
:red_circle: Aim : The aim is to detect the parking spaces from the top view using the given dataset.
:red_circle: Dataset : https://www.kaggle.com/datasets/loveffc/parking-space-detection-based-on-top-view
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Email ID :
Participant ID (if applicable):
Approach for this Project :
What is your participant role? (Mention the Open Source program)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Hello, I am new to ssoc and i would like to contribute in this issue,
can you please assign this issue to me.
Full name: Himanshu Choudhary
GitHub Profile Link : https://github.com/Himanshu8850
Email ID<EMAIL_ADDRESS>Participant ID (if applicable):
Approach for this Project : I would be using techniques like edge detection, contour detection, deep learning-based feature extraction to extract relevant features from the images then i will train different machine learning models like SVM,CNN,Random forest etc.
What is your participant role?
I am partcipating through SSOC Season 3

Hi @Himanshu8850 this project repo is currently participating in GSSOC 2024. This repo is not a part of the SSOC event.

i have also participated in gssoc.

You have written that you are participating in SSOC, that's why I asked about the same.

Hello, I am new to ssoc and i would like to contribute in this issue, can you please assign this issue to me.
Full name: Himanshu Choudhary GitHub Profile Link : https://github.com/Himanshu8850 Email ID<EMAIL_ADDRESS>Participant ID (if applicable): Approach for this Project : I would be using techniques like edge detection, contour detection, deep learning-based feature extraction to extract relevant features from the images then i will train different machine learning models like SVM,CNN,Random forest etc. What is your participant role? I am partcipating through SSOC Season 3

Sorry for the confusion, but i have participated in both of them.

Hi @Himanshu8850 as this project repo mainly focuses on deep learning projects it'll be better if you come up with deep learning models/algorithms for this project instead of machine learning models.

Sir, I am not familiar with deep learning models but will surely try my best to understand them and implement if you are okay to cooperate.
deep learning models like CNN,prebuilt models like Mask RCNN built on Faster-RCNN etc.

Hi @Himanshu8850 it would be better if you practice these things in a small before implementing those in a competitive event. You can find out the resources in the projects-intro channel of the GSSoC Discord Server.

Hey, @abhisheks008 I would like to contribute to this issue.
Full name : Suhani Gupta
GitHub Profile Link : https://github.com/Suhani2
Email ID<EMAIL_ADDRESS>Participant ID (if applicable):
Approach for this Project : I am thinking of using various pre trained object detection models such as YOLOv8, SSD or faster R-CNN to compare the accuracy scores.
What is your participant role? gssoc'24

well! @Suhani2 issue assigned to you. Go ahead.


Create unit tests

run tests when building PR or master branch commit

@Adman I guess this is now close to being realistically possible, right?

@mrshu Right, will start working on this in the near future.


FEATURE - databricks and postgres compatibility

Are you a Fivetran customer?

nope!
Is your feature request related to a problem? Please describe.

No, just want to roll out postgres + databricks compatibility for our most popular packages.
Describe the solution you'd like

make it compatible with both!
Please indicate the level of urgency and business impact of this request

Does not seem urgent, but did come out of a customer request on dbt slack
Are you interested in contributing to this package?

[x] Yes, I can do this and open a PR for your review.
[ ] Possibly, but I'm not quite sure how to do this. I'd be happy to do a live coding session with someone to get this work implemented.
[ ] No, I'd prefer if someone else did this.  I don't have the time and/or don't know how to incorporate the changes necessary.

[X] Postgres compatibility
[ ] Databricks compatibility


Source EMIT tools for notebooks

How do we want to source other tools like the EMIT-Data-Resources if we need them for the workshop?  Should we use a submodule? That way when you pull HYR-SENSE we can also have it pull in the EMIT tools
git submodule update --init --recursive

https://stackoverflow.com/questions/3336995/git-will-not-init-sync-update-new-submodules

I've added a EMIT-Data-Resources as a submodule to HYR-SENSE so that when cloning the HYR-SENSE repo you also get those tools inside the HYR-SENSE repo and we can write notebooks that assume the path to EMIT Tools.  We may want to fix the EMIT tools to a specific version number which we can do in the submodule def in .gitsubmodules


Improve default enclosing radius for some shapes

Since navigation agents are now assumed to have Y-axis as up, the enclosing radius should be now considered a circle enclosing shape in XZ-plane - not a sphere like in the past.

I don't like this because now the functions won't do what they say they'll do. This will lead to confusion for anyone who tries to use them in the future.


feat: [CDS-99733]: Add info If you are using a custom webhook trigger in webhook notification method and if your Pipeline setting Mandate Authorization for Custom Webhook Triggers is true then you need to pass the API key in the header

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66b63107fdff580e433dd340--harness-developer.netlify.app


Implicated line is
https://github.com/NREL-SIIP/InfrastructureSystems.jl/blob/ce8f4eefadb9417fc4ed44a6808e27b7297b4409/src/utils/logging.jl#L20
Base.find_package("InfrastructureSystems") can return nothing

maybe we want pathof(InfrastructureSystems) here?

@nickrobinson251 Yes, sorry for the trouble.

@jd-lara We need to adjust the CI/cross-package testing to import packages the way users do it.

We need to adjust the CI/cross-package testing to import packages the way users do it.

Shall we open an issue for this?
I wonder what is different... maybe we need to add rather than develop? (but that's a wild guess)
https://github.com/NREL-SIIP/InfrastructureSystems.jl/blob/d15c322cd662e12cb13923d640b5cf9b40629de1/.github/workflows/cross-package-test.yml#L37

We need to adjust the CI/cross-package testing to import packages the way users do it.

Shall we open an issue for this?
I wonder what is different... maybe we need to add rather than develop? (but that's a wild guess)
https://github.com/NREL-SIIP/InfrastructureSystems.jl/blob/d15c322cd662e12cb13923d640b5cf9b40629de1/.github/workflows/cross-package-test.yml#L37

I wonder why the current set up didn't pick it up. This issue also didn't came out in the PSY stand alone tests

I'm not super familiar with this workflow, but isn't it because you're only running checks for packages that directly depend on IS? I think this error would only occur in a package that has an indirect dependency e.g. having PSY as dep but not IS

To do that I think we need to make a dummy module with an IS dependency and some test workflow. We tried that at some point to test the @forward macro but it was a huge hassle that never really worked well.


Eliminate hard-coded BN_N_LIMBS and BN_LIMB_WIDTH dependency.

In an attempt to see what would happen to the number of constraints if we changed the parameters BN_N_LIMBS and BN_LIMB_WIDTH, I found a couple places where the arithmetic inadvertently was dependent on the values being hardcoded as 4 and 64 respectively.
I believe this PR makes the necessary changes to make the codebase general enough to allow for varying these parameters. The expect tests expectedly (:drum: haha) fail when changing these two constants, but everything else passes.

The change looks good to me, though I haven't looked into this gadget enough to understand if this is enough to support arbitrary configurations. Ideally, we should replace this with bellpepper-gadgets/emulated.

@adr1anh @mpenciak
Hi folks, could we get a decision on this PR with the code base we have today? Surely it seems a 6-line change is a thing we can make a decision (positive or negative) today, even if the ultimate goal of moving to bellpepper-emulated seems optimal tomorrow?

The change does seem reasonable to me, and can be backported to Nova too, even if we move to bellpepper-emulated later.


[ADD] property last to navigation path to observe the newest incoming path

I need a possibility to observe the last incoming value to a path. My first imagination: "NBNavigationPath based on elements. Nice, then I can use the array functionality." Then I found out, that the elements property is internal. That's the reason, why I need this property getter.

Hi @AF-cgi, thanks for opening this PR. So far, I've tried to reproduce SwiftUI's navigation APIs precisely, whereas this would expose an API in NBNavigationPath that isn't available in NavigationPath. There's a couple of problems with that:

Users might come to depend on an API that prevents them from later migrating to SwiftUI's equivalent navigation.
Apple might at some point introduce a conflicting API on NavigationPath.

Have you considered using an Array rather than NBNavigationPath (like in this example)? That would allow you to inspect the array more easily, but isn't always an option.

I see your point. I found another way to do this without using this PR anymore.


Integrate Snoflakes

Ben has pulled out Snovalut into a separate branch / soon to be repo.  Integrate that into our version of encode, so we can better collaborate with the nice folks at Stanford.

WIP see http://github.com/j1z0/snovault

fine


Qualified names with .type in the middle

The parser can't handle qualified names where .type is one of the "middle" names, e.g.,
.output bar.type.foo

This can be fixed by making qualified_name into a regex token (qualified_name: $ => /((_|\?|[A-Z]|[a-z])(([A-Z]|[a-z])|[0-9]|_|\?)*)(\.((_|\?|[A-Z]|[a-z])(([A-Z]|[a-z])|[0-9]|_|\?)*))*/) but that's horrible and also causes the test "Unary functor declaration" to fail...


[AMBARI-24674] Upgrade checkstyle version to 8.9

What changes were proposed in this pull request?
Upgrade checkstyle to latest version.
https://issues.apache.org/jira/browse/AMBARI-24674
How was this patch tested?
$ mvn -am -pl ambari-utility clean test
...
[INFO] BUILD SUCCESS

$ mvn -am -pl ambari-server -Del.log=WARN -DskipTests clean test
...
[INFO] BUILD SUCCESS

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4033/
Test PASSed.


Set image in UICollectionViewCell with  af_setImageWithURL cost to much memory

Hi,
I have some memory issue using AFI. I have UICollectionView with custom cell, and each cell has a UIImageView. In cellForItemAtIndexPath, I set that image with af_setImageWithURL.
The problem is memory usage getting so high. If I set cell's image with local image, it only use 27Mb memory. But when I fetch image with af_setImageWithURL, memory usage are going about 87Mb.
So am I doing some mistake?
Thank you.

Nope, you're not making a mistake. Image loading causes your memory to increase. The ones loaded from an asset catalog are VERY optimized for rendering, whereas the ones downloaded are not and need to be inflated as well as having the pixel data rearranged for optimal performance when rendering to screen. Both of these result in your memory usage increasing for the downloaded images more than the local ones (increase is mainly due to inflation).
If you want to limit the memory usage of the caching system, you could set the NSURLCache memoryCapacity to zero. You could also set the UIImageView's AutoPurgingImageCache capacity to something really low. This is not recommended though since the defaults are most likely what you want to be using.
Best of luck! 🍻

Thank you so much, cnoon!
My problem is the resource that url provide is too big. Right now I handle it with resizing its image before placing in my UIImageView using filter.
let placeholderImage = UIImage(named: "app_default-resize")

let filter = AspectScaledToFitSizeFilter(size: self.imgProductItem.frame.size)
self.imgProductItem.af_setImageWithURL(
    url!,
    placeholderImage: placeholderImage,
    filter: filter
)

That's decrease so much memory usage. Again, thank you cnoon!

You bet! Glad you got it figured out. 🍻


Modal continuously updating

Whenever a modal is open and you check the updates being done by React, the modal is being updated constantly even when doing nothing. I don't know if this is intended or not, but I imagine it puts strain on the rendering.
Steps

Go to: http://react.semantic-ui.com/modules/modal
Click on a modal button
Enable Trace React Updates in the React tab of the Web Developer Console.
...

Expected Result
No continuous updating of the modal
Actual Result
Continuously updating the modal as shown here:

Version
Checked versions 0.61.7 and 0.61.8.

Nice catch! Just opened a PR to fix it 👍


angularfire2 version 4.0.0-rc.1 - cannot retrieve first key from list.

@davideast  I upgraded my Angular 4 app to work in conjunction with angularfire2 4.0.0-rc.1 and have run into an issue when trying to retrieve the first key from a list. I'm looking to retrieve just the key.
this._db.list('/example/', {
            query: {
                orderByKey: true,
                limitToFirst: 1
            }
        }).subscribe(x => {
            console.log(x)
        });

Version info
Angular: 4.0.0
Firebase: 4.1.2
AngularFire: 4.0.0-rc.1

@jamesamuel What is the issue you are running into? Tack on a .catch() and see if an error appears.

@davideast Instead of outputting the value as an object, I'm looking to retrieve just the key, which will then be used to trigger something else. Does firebase allow this, or the SDK?

.subscribe(x => {
        console.log(x[0].$key)
});

try that


release raiwidgets and responsibleai 0.21.0

Description
Release Notes:
v0.21.0

educational materials

[docs] Add Data Balance README by @ms-kashyap in https://github.com/microsoft/responsible-ai-toolbox/pull/1567
Replace SynapseML links with RAIToolbox links by @ms-kashyap in https://github.com/microsoft/responsible-ai-toolbox/pull/1592

new features

Responsible AI Dashboard

Data Balance UX Part 2: Add visualizations as a flight by @ms-kashyap in https://github.com/microsoft/responsible-ai-toolbox/pull/1512
add new interpret-vision library skeleton by @jamesbchao in https://github.com/microsoft/responsible-ai-toolbox/pull/1586
add support for multiclass local importance values for text by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1601
Make generic class interface for cohort based filtering by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1597
Vision explanation dashboard -- Image explorer view by @jamesbchao in https://github.com/microsoft/responsible-ai-toolbox/pull/1587

RAIInsights

bug fixes and tests

Responsible AI Dashboard

update release workflows to include interpret-text npm package by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1582
Add RAI prefix for OSS UI telemetry events by @tongyu-microsoft in https://github.com/microsoft/responsible-ai-toolbox/pull/1584
Bump moment from 2.29.2 to 2.29.4 by @dependabot in https://github.com/microsoft/responsible-ai-toolbox/pull/1538
Bump eventsource from 1.0.7 to 1.1.1 by @dependabot in https://github.com/microsoft/responsible-ai-toolbox/pull/1456
[Bugfix] Fix Dropdown bug and Update styling for feature balance measures by @ms-kashyap in https://github.com/microsoft/responsible-ai-toolbox/pull/1583
Refactor tests in test_responsibleai_dashboard_input.py to have common success and failure criteria by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1591
update raiwidgets and responsibleai to erroranalysis 0.3.6 by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1588
Fix tooltip location showing up incorrectly due to conflicting ids by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1599
Update e2e test to expand correct predictions caret symbol by @vinuthakaranth in https://github.com/microsoft/responsible-ai-toolbox/pull/1603
Updates to RAI text dashboard - update test app values to latest shap release, fix predicted label, limit selection to one row, fix random crash by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1605
[300 line limit] Refactor AxisConfigDialog: create AxisConfigBinOptions and AxisConfigChoiceGroup component by @tongyu-microsoft in https://github.com/microsoft/responsible-ai-toolbox/pull/1609
[300 line limit] Refactor CohortEditor, OverallMetricChart, set max-lines rule for core-ui package by @tongyu-microsoft in https://github.com/microsoft/responsible-ai-toolbox/pull/1612
design updates for the RAI text dashboard local importances view by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1611
[Accessibility] Fix all existing accessibility bugs by @RubyZ10 in https://github.com/microsoft/responsible-ai-toolbox/pull/1613
Move CalculateBoxPlot.test.tsx to core-ui by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1620
Add feature metadata to IDataset and JointDataset by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1614
Remove console.log() statement from CohortEditor.tsx by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1628
update raiwidgets and responsibleai to erroranalysis 0.3.8 by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1629
add new interpret-vision package to npm publish workflows in build and release gates by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1615
[Accessibility] fix several accessibility bugs by @RubyZ10 in https://github.com/microsoft/responsible-ai-toolbox/pull/1632
add UI tests for RAI text dashboard by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1636
Fix typo for indices by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1640
Add extra space between Identity feature and the actual feature name by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1641
Use JointDataset API in ProcessPreBuiltCohort.ts by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1635
add wrapping for long text documents to text highlighting component in RAI dashboard for text by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1643
add tests to RAI text dashboard importances part 2 by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1642
add radio button tests for RAI text dashboard by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1648
Localized file check-in by OneLocBuild Task: Build definition ID 21368: Build ID 69976515 by @tongyu-microsoft in https://github.com/microsoft/responsible-ai-toolbox/pull/1653
Add unit test for calculateLineData by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1654

RAIInsights

Refactor categorical statistics computation in separate module by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1600
responsibleai: Add FeatureMetadata class and related tests by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1604
Add categorical statistics computation in RAIInsights by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1598
Add cohort filtering capability in RAIInsights class by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1618
Inject FeatureMetadata into RAIInsights by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1616
responsibleai: Remove old manager namespaces by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1564
Add test for uncovered condition in responsibleai_dashboard_input.py by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1638
Fix docs in rai_base_insights.py by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1650

Counterfactual

[300 line limit] Refactor CounterfactualList by @tongyu-microsoft in https://github.com/microsoft/responsible-ai-toolbox/pull/1619
Reduce the time taken by RAIInsights counterfactual tests by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1622
[300 line limit] Refactor CounterfactualChart by @tongyu-microsoft in https://github.com/microsoft/responsible-ai-toolbox/pull/1624
Add identity feature value to counterfactual hover over by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1626
Minor refactoring in counterfactual package by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1634
[Accessibility] Add headings & change counterfactual text color by @RubyZ10 in https://github.com/microsoft/responsible-ai-toolbox/pull/1646

Causal

Add backend functions in causal_manager.py to handle dashboard requests by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1579
SDK change for computing aggregate causal effects and policy tree per cohort by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1552
Add identity feature value to causal hover over by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1633

Error Analysis

Add more classification metrics to erroranalysis by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1575
Improve error message when categorical string column is treated as numeric and error analysis is run by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1562
fix bug in treeview filtering logic in error analysis package by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1593
erroranalysis: Bug fix in classification outcome cohort filter by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1596
release erroranalysis 0.3.7 by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1606
fix error analysis predictions analyzer failing due to new filtering refactor by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1625
Make return type of string_ind_data in process_categoricals consistent by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1617
Handle 'Classification outcome' cohort filter for multiclass scenario in SDK by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1623
Handle 'Regression Error' cohort filter for regression scenario in SDK by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1644
[Accessibility] Error analysis tree node change 1. Bubble error rate colors; 2. Bubble text & fill by @RubyZ10 in https://github.com/microsoft/responsible-ai-toolbox/pull/1649

Dataset Explorer

Add identity feature value to dataset explorer scatter plot by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1645

Model Overview

If feature groups is more than 10, short it to 10 with "Other" for the rest by @RubyZ10 in https://github.com/microsoft/responsible-ai-toolbox/pull/1602
[Model Overview]Fix bug: x axis selection is not kept between tab switch by @RubyZ10 in https://github.com/microsoft/responsible-ai-toolbox/pull/1607
[ModelOverview] Fix bug- On choosing a specific feature cohort and trying to choose feature cohort again - dropdown has all cohorts selected by @RubyZ10 in https://github.com/microsoft/responsible-ai-toolbox/pull/1608
[ModelOverview]Fix bug- Model Overview: Clarification - Should state of 'use line chart' be persisted when switched from Probability -> Metrics -> Prob tab by @RubyZ10 in https://github.com/microsoft/responsible-ai-toolbox/pull/1610
enable new Model Overview experience in OSS RAI Dashboard by default by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1647

Interpret

Add backend functions in explainer_manager.py to handle dashboard requests by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1585
Add identity feature value to feature importance hover over by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1637

other

update CHANGES.md release notes to reflect delay in package release by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1581
Remove strict pinnings from rai-core-flask requirements.txt. by @janjagusch in https://github.com/microsoft/responsible-ai-toolbox/pull/1578
release rai-core-flask 0.5.0 by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1589
update deprecated pypi publish github action to new supported version by @imatiach-msft in https://github.com/microsoft/responsible-ai-toolbox/pull/1595
Update CODEOWNERS to add default reviewers for core-ui by @gaugup in https://github.com/microsoft/responsible-ai-toolbox/pull/1639

Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[x] Documentation was updated if it was needed.

Codecov Report

Merging #1656 (433536e) into main (58e2b0c) will increase coverage by 8.43%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1656      +/-   ##
==========================================
+ Coverage   80.57%   89.00%   +8.43%
==========================================
  Files          41      105      +64
  Lines        1719     5529    +3810
==========================================
+ Hits         1385     4921    +3536
- Misses        334      608     +274

Flag
Coverage Δ

unittests
89.00% <ø> (+8.43%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

erroranalysis/erroranalysis/_internal/__init__.py
100.00% <0.00%> (ø)

responsibleai/responsibleai/managers/__init__.py
100.00% <0.00%> (ø)

...onsibleai/responsibleai/serialization_utilities.py
96.55% <0.00%> (ø)

...sibleai/responsibleai/rai_insights/rai_insights.py
76.06% <0.00%> (ø)

...onsibleai/_tools/shared/attribute_serialization.py
95.94% <0.00%> (ø)

responsibleai/responsibleai/feature_metadata.py
96.00% <0.00%> (ø)

erroranalysis/erroranalysis/analyzer/__init__.py
100.00% <0.00%> (ø)

...ranalysis/erroranalysis/_internal/cohort_filter.py
87.04% <0.00%> (ø)

...is/erroranalysis/_internal/surrogate_error_tree.py
88.74% <0.00%> (ø)

...s/erroranalysis/_internal/error_report/__init__.py
100.00% <0.00%> (ø)

... and 54 more

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


style: update to clang-tidy 15

Codecov Report

Merging #211 (5936f4e) into main (cf61c84) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff            @@
##              main      #211   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         1    -1
  Lines           93        27   -66
  Branches         4         3    -1
=========================================
- Hits            93        27   -66

Flag
Coverage Δ

linux-clang-14
?

linux-gcc-12
?

macos-clang
100.00% <ø> (ø)

windows-cl
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2 files with indirect coverage changes


feat: add quote to access schema

The lack of the quote access type in the OpenAPI schema is blocking token introspection for Quote grants.

The OpenAPI errors should be resolved either here or in a separate pr.
I'm guessing some are pre-existing from when the GitHub actions weren't running.


decrease cargo test time for git workflows

Problem
The time to complete cargo test is about 15 minutes long. It is run an additional time for grcov coverage report. That's at least 30 minutes per git push and git tag.
This is a long wait for every push and tag.
Solution
Decrease the time required somehow.

3ba007d

Current runtime from Run<PHONE_NUMBER>

Longest job from first stages: 12:14 build MSRV 1.67.1 on macos-latest
Longest job from second stages: 7:57 test
Longest job from third stages: 3:20 run s4
Longest job from fourth stages: 27:22 grcov

Relates to #218


Add entire higher classification to search index

For metrics about a certain higher group, e.g. a single, whole family the index needs to know about all usages that are included in such a higher taxon. We can then provide counts and a filter to restrict a search to a certain group only.
Add a List if all higher taxon ids to the NameUsageWrapper and store it in ES.
Retrieving the entire classification ids can be expensive. Try to implement a taxonomic tree iteration that starts at the top tree roots and visits all children remembering the entire path of parent ids as an alternative implementation of the NameUsageMapper.processDatasetTaxa method

@ayco-at-naturalis We also need a way to filter by higher taxa.
Suggest to add a new NameSearchParameter.TAXON_ID that gets matched against the list of taxonIDs of all parents in the index

Ah ... I thought this request would come some time :-)

All NameSearchParameter should be searchable.
I think we should do a unit test that goes thru each parameter and makes sure its supported

All parameters are searchable as long as I include them in a simple parameter-to-field lookup; the reset happens automatically; but I will nonentheless unit test this for each and every parameter.
I was thinking, however, maybe we should include all fields that can either searched, sorted or aggregated on as a NameSearchParameter and in NameSearchParameter's constructor tell it what it can be used for (QUERY,SORT,FACET); this allows for even more automation, which will prbably pay of when implementing reference search.

Can we keep that as an enhancement in a new issue for now? I can't see immediately what we gain having the ES fields linked with the parameter - maybe to auto construct ES queries.
Right now all params should be usable as a filter and for aggregation. Ordering is entirely different

They  are currently linked and auto-constructed; it's just that I must now do separate separate lookups for query/facet fields and sort fields; it's minor and indeed I will postpone that if/when things need to become more generic when implementing ref search.


ios google map without tiles on latest packages - rn 0.60.5 / react-native-maps 0.25.0

Bug

Environment info

React native info output:
System:
    OS: macOS 10.14.6
    CPU: (12) x64 Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
    Memory: 450.21 MB / 32.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 12.3.1 - /usr/local/bin/node
    Yarn: 1.17.3 - /usr/local/bin/yarn
    npm: 6.11.3 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.0, DriverKit 19.0, macOS 10.15, tvOS 13.0, watchOS 6.0
    Android SDK:
      API Levels: 23, 28, 29
      Build Tools: 27.0.3, 28.0.3, 29.0.0, 29.0.2
      System Images: android-21 | Google APIs Intel x86 Atom_64, android-22 | Google APIs ARM EABI v7a, android-22 | Google APIs Intel x86 Atom_64, android-23 | Google APIs Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom, android-29 | Google APIs Intel x86 Atom
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>91312
    Xcode: 11.0/11A420a - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.60.5 => 0.60.5
  npmGlobalPackages:
    react-native-cli: 2.0.1

Library version: x.x.x
Steps To Reproduce

Install latest react native
Install latest react-native-maps on IOS + Google Maps (cocoapods)
Load the map
U will see an empty map where markers appear but tiles do not

Describe what you expected to happen:

See the map tiles

Reproducible sample code

basic MapView implementation
Screenshot

Be sure to add it
[GMSServices provideAPIKey:@" API Key Google"];
or Check Console API Key Google

already did that ... works on android ... doesn't on ios

"react": "16.9.0",
"react-native": "^0.61.1"

i removed ani restriction on the key from google developer console and still no tiles :(

for anyone watching this
in google developer console u have to activate 2 APIs

Maps for Android
Maps for IOS
I only activated the first one ... so ... my mistake


update check emoji to carbon icon on tile style page

Proportions for grid table on tile component's style page should be using carbon icons rather then the emoji checks check ✅

@kingtraceyj which of the checkmark icons should it use?

@jnm2377 let's use <CheckmarkOutline /> in gray 100

@francinelucca we are going with checkmark in icon-primary


Not being prompted to update vscode-go on Windows

Shows .43 which is not the latest, with no update button.

Searching for it, it thinks I don't have it installed.

And after installing it, it looks correct in the extensions UI, but I have both versions in my .vscode/extensions folder.

What are the contents of the locally installed extension's package.json?

On second thought, I think that original one was installed from a vsix, as it has __metadata: null. Sorry it was awhile ago.


Once Ads are Played, Video Stream (.m3u8) Doesn't Resume on Mobile (Chrome)!

Hello,
The code below has prevented the video from resuming after an ad ends on mobile (Chome).
Any thoughts, please?
My complete code:
<script src="~/Content/VideoJS/js/videojs.js"></script>
<script src="//imasdk.googleapis.com/js/sdkloader/ima3.js"></script>
<script src="~/Content/VideoJS/js/videojs-contrib-hls.js"></script>
<script src="~/Content/VideoJS/js/videojs.ads.js"></script>
<script src="~/Content/VideoJS/js/videojs.ima.js"></script>

<video id="content_video" class="video-js vjs-default-skin" controls width="640"
       height="360">
    <source src="https://bitdash-a.akamaihd.net/content/sintel/hls/playlist.m3u8"
            type="application/x-mpegURL">
</video>

<script>
    var player = videojs('content_video');
    var options = {
        id: 'content_video',
        adTagUrl: 'https://pubads.g.doubleclick.net/gampad/ads?' +
            'sz=640x480&iu=/37592844/5flix-vast&impl=s&gdfp_req=1' +
            '&env=vp&output=vast&unviewed_position_start=1&url=' +
            '[referrer_url]&description_url=[description_url]&correlator=[timestamp]'
    };
    player.ima(options);
    player.ima.requestAds();
    player.ima.initializeAdDisplayContainer();
    player.play();
</script>

The weird thing I tested it in Firefox (PC with mobile emulator) and it worked perfectly well, but in Chrome, it doesn't work. Is it supported on those browsers?
On Chrome (PC with mobile emulator), after the ad ends, the player stays in a loading state infinitely.
PS: Once I replaced the m3u8 with an mp4 video, it works fine.
Thanks.

The issue was solved after I added:
var startEvent = 'ready';
if (navigator.userAgent.match(/iPhone/i) ||
   navigator.userAgent.match(/iPad/i) ||
   navigator.userAgent.match(/Android/i)) {
   startEvent = 'touchend';
}
player.one(startEvent, function () {
   player.ima.initializeAdDisplayContainer();
   player.ima.requestAds();
   player.play();
});

I also added "autoplay" to the video tag.

Yep that makes sense - on mobile you need to call initializeAdDisplayContainer as the result of a user action. We did have an issue in the past as well with resuming m3u8s in the IMA SDK, so your addition of autoplay probably helped resolve that.

Hello I am also trying to play play m3u8 streaming files with IMA DAI SDK for HTML5 with its not working.
i used the above code but its not working. Please help me in this regard


Upgrade Sass to 1.80.6 (and fix as many deprecations as possible)

This upgrades Sass from 1.77.6 to 1.80.6.
The new version introduces a lot more deprecation warnings ahead of Sass 2.0.0 – so I’ve tried to fix as many as possible.
A lot of the warnings are from the vendored Sass-mq however. I've not touched that at all as it feels a bit icky to edit a vendor file (although it has had some code style tweaks over the years).
The GOV.UK Design System team are considering removing the sass-mq dependency altogether. However there was a 6.0.0 release of it last week which should resolve the deprecations.
The other remaining deprecation warnings are about use of @import - however moving away from is bigger job (changing every instance of @import to @use and namespacing all the imported functions).
Checklist

[ ] Tested against our testing policy (Resolution, Browser & Accessibility)
[ ] Follows our coding standards and style guide
[ ] CHANGELOG entry

The other remaining deprecation warnings are about use of @import - however moving away from is bigger job (changing every instance of @import to @use and namespacing all the imported functions).

From the Sass docs:

You can even load a module without a namespace by writing @use "<url>" as *.

In my limited experience trying to move over to using @use, this means you can do the following:
- @import 'core.scss'
+ @use 'core.scss' as *

This then means you don’t need to use namespaced variables. However, the Sass documentation goes on to say:

We recommend you only do this for stylesheets written by you, though; otherwise, they may introduce new members that cause name conflicts!

Which is to say this may be a short term fix, but longer term, a breaking change could introduce an nhsuk namespace for all mixins, functions and variables owned by nhsuk-frontend.

@paulrobertlloyd ooh nice, I hadn’t spotted that. It certainly could be a short-term option, and then we could gradually switch to namespaces, starting with the most-reused functions and  mixins and so on?

Trying to resolve these deprecation warnings in my prototype, one thing with the above approach (@use "<url>" as *) is that it seemingly causes conflicts if 2 SCSS files export a variable, mixin or function with the same name. As this project shares some names with GOV.UK Frontend, I suspect there’d be clashes if you were to import anything from GOV.UK Frontend.
This module system seems nice un all, but get the feeling it’s an all or not-at-all kinda thing. 😕 Definitely warrants more investigation.


[Spark 4.0] Account for CommandUtils.uncacheTableOrView signature change.

Fixes #10710.
This commit accounts for the changes in the signature of CommandUtils.uncacheTableOrView in Apache Spark 4.0. (See SPARK-47191.)

Build

Build

The CI run failed with the following error. (It looks unrelated to my current change.)
024-05-28T20:46:22.5133881Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.458Z] [ERROR] /home/ubuntu/spark-rapids/sql-plugin/src/main/spark330db/scala/com/nvidia/spark/rapids/shims/Spark321PlusDBShims.scala:44: Unused import
2024-05-28T20:46:22.5134484Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.458Z] [ERROR] import org.apache.spark.sql.rapids.execution._
2024-05-28T20:46:22.5135036Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.458Z] [ERROR]                                              ^
2024-05-28T20:46:22.5136444Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.458Z] [ERROR] /home/ubuntu/spark-rapids/sql-plugin/src/main/spark330db/scala/org/apache/spark/rapids/execution/shims/GpuSubqueryBroadcastMeta.scala:97: type mismatch;
2024-05-28T20:46:22.5136788Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.458Z]  found   : Int
2024-05-28T20:46:22.5137168Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.458Z]  required: Seq[Int]
2024-05-28T20:46:22.5137918Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.458Z] [ERROR]     GpuSubqueryBroadcastExec(s.name, s.index, s.buildKeys, broadcastBuilder())(
2024-05-28T20:46:22.5138437Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.458Z] [ERROR]                                        ^
2024-05-28T20:46:22.5140243Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.458Z] [ERROR] /home/ubuntu/spark-rapids/sql-plugin/src/main/spark330db/scala/org/apache/spark/rapids/execution/shims/GpuSubqueryBroadcastMeta.scala:40: private var broadcastBuilder in class GpuSubqueryBroadcastMeta is never used
2024-05-28T20:46:22.5140860Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.459Z] [ERROR]   private var broadcastBuilder: () => SparkPlan = _
2024-05-28T20:46:22.5141247Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.459Z] [ERROR]               ^
2024-05-28T20:46:22.5143138Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.459Z] [ERROR] /home/ubuntu/spark-rapids/sql-plugin/src/main/spark330db/scala/org/apache/spark/rapids/execution/shims/GpuSubqueryBroadcastMeta.scala:102: private method getBroadcastModeKeyExprs in class GpuSubqueryBroadcastMeta is never used
2024-05-28T20:46:22.5143839Z [2024-05-28T20:45:49.822Z] [2024-05-28T20:45:38.459Z] [ERROR]   private def getBroadcastModeKeyExprs: Option[Seq[Expression]] = {

I've merged up for #10933.  I'll kick off the CI run shortly.

Build

This PR still fails on an unrelated error:
2024-05-29T17:47:06.1091539Z [2024-05-29T17:46:28.999Z] [2024-05-29T17:46:20.351Z] [ERROR] /home/ubuntu/spark-rapids/sql-plugin/src/main/spark330db/scala/org/apache/spark/rapids/execution/GpuSubqueryBroadcastMeta.scala:33: GpuSubqueryBroadcastMeta is already defined as class GpuSubqueryBroadcastMeta
2024-05-29T17:47:06.1092047Z [2024-05-29T17:46:28.999Z] [2024-05-29T17:46:20.352Z] [ERROR] class GpuSubqueryBroadcastMeta(
2024-05-29T17:47:06.1092396Z [2024-05-29T17:46:28.999Z] [2024-05-29T17:46:20.352Z] [ERROR]       ^
2024-05-29T17:47:06.1092787Z [2024-05-29T17:46:28.999Z] [2024-05-29T17:46:20.352Z] [ERROR] one error found
2024-05-29T17:47:06.1093385Z [2024-05-29T17:46:28.999Z] [2024-05-29T17:46:20.352Z] [INFO] ------------------------------------------------------------------------

#10945 should resolve it.

build

build


[Feature request] Read video from memory buffer

It would be fantastic if it was possible to read a video from a memory buffer, in addition to reading it from a file. For example, one could have a video file being a BytesIO object and use skvideo.io.vread directly on it.
A use case would be reading videos from an archive. Right now, the videos need to be extracted into a file first, and then read into memory.

This looks entirely possible. Since we pipe bytes to and from FFmpeg already, this feature would be implemented as a lower level interface surrounding the existing pipe and subprocess-based mechanism.
Could be a fun exercise to implement it and push a pull request. I'm swamped right now so this is currently the fastest way to get the feature.

Having vreader-like functionality without the need for a file is an interesting idea. I'm not sure, but it appears the vreader uses ffmpeg/libav magic to reduce memory usage (for large videos) by loading only one frame into memory at a time. If we could do that on any video we happened to have sitting in memory, then we could use convenience functions like skvideo.measure.mse on transformations of videos (for example, converting to grayscale before calculating mse) without doubling memory usage.