Build excludes paths to standard C++ headers when using GNUARMEMB toolchain variant

Describe the bug
Using the gnuarmemb toolchain variant to build a trivial application fails if any source file in the application includes a standard C++ header.
To Reproduce
Steps to reproduce the behavior:

Create a trivial Zephyr C++ app with CONFIG_CPLUSPLUS=y. Put the following in main.cc (or main.cpp if you prefer):

#include <cstdint>
void main() {}

Attempt to to build the app using the 'gnuarmemb` toolchain variant for an appropriate board (such as nrf52dk_nrf52832):

$ ZEPHYR_TOOLCHAIN_VARIANT=gnuarmemb west build -b nrf52dk_nrf52832

Build fails.

Expected behavior
Build should complete without error.
Impact
Unable to generate any firmware images for target hardware.
Logs and console output
The compiler invocation command that fails is shown below. Newlines have been added for readability, and some path information has been redacted.
ccache <REDACTED>/prebuilt/gcc-arm-none-eabi-10-2020-q4/linux/bin/arm-none-eabi-g++
-DBUILD_VERSION=v2.4.99-ncs2
-DKERNEL
-DNRF52832_XXAA
-DUSE_PARTITION_MANAGER=0
-D_FORTIFY_SOURCE=2
-D__PROGRAM_START
-D__ZEPHYR__=1
-I<REDACTED>/third_party/nordic/ncs_v1.5.1/zephyr/include
-Izephyr/include/generated
-I<REDACTED>/third_party/nordic/ncs_v1.5.1/zephyr/soc/arm/nordic_nrf/nrf52
-I<REDACTED>/third_party/nordic/ncs_v1.5.1/nrf/include
-I<REDACTED>/third_party/nordic/ncs_v1.5.1/modules/hal/nordic/nrfx
-I<REDACTED>/third_party/nordic/ncs_v1.5.1/modules/hal/nordic/nrfx/drivers/include
-I<REDACTED>/third_party/nordic/ncs_v1.5.1/modules/hal/nordic/nrfx/mdk
-I<REDACTED>/third_party/nordic/ncs_v1.5.1/zephyr/modules/hal_nordic/nrfx/.
-I<REDACTED>/third_party/nordic/ncs_v1.5.1/modules/hal/cmsis/CMSIS/Core/Include
-I<REDACTED>/third_party/nordic/ncs_v1.5.1/modules/debug/segger/rtt
-isystem <REDACTED>/third_party/nordic/ncs_v1.5.1/zephyr/lib/libc/minimal/include
-isystem <REDACTED>/prebuilt/gcc-arm-none-eabi-10-2020-q4/linux/bin/../lib/gcc/arm-none-eabi/10.2.1/include
-isystem <REDACTED>/prebuilt/gcc-arm-none-eabi-10-2020-q4/linux/bin/../lib/gcc/arm-none-eabi/10.2.1/include-fixed
-Os
-fcheck-new
-std=c++11
-Wno-register
-fno-exceptions
-fno-rtti
-imacros <REDACTED>/firmware/app/trivial/BUILD/zephyr/include/generated/autoconf.h
-ffreestanding
-fno-common
-g
-mcpu=cortex-m4
-mthumb
-mabi=aapcs
-imacros <REDACTED>/third_party/nordic/ncs_v1.5.1/zephyr/include/toolchain/zephyr_stdint.h
-Wall
-Wformat
-Wformat-security
-Wno-format-zero-length
-Wno-main
-Wpointer-arith
-Wno-address-of-packed-member
-Wno-unused-but-set-variable
-fno-asynchronous-unwind-tables
-fno-pie
-fno-pic
-fno-strict-overflow
-fno-reorder-functions
-fno-defer-pop
-fmacro-prefix-map=<REDACTED>/firmware/app/trivial=CMAKE_SOURCE_DIR
-fmacro-prefix-map=<REDACTED>/third_party/nordic/ncs_v1.5.1/zephyr=ZEPHYR_BASE
-fmacro-prefix-map=<REDACTED>=WEST_TOPDIR
-ffunction-sections
-fdata-sections
-nostdinc
-MD
-MT CMakeFiles/app.dir/src/main.cc.obj
-MF CMakeFiles/app.dir/src/main.cc.obj.d
-o CMakeFiles/app.dir/src/main.cc.obj
-c ../src/main.cc

../src/main.cc:2:10: fatal error: cstdint: No such file or directory
    2 | #include <cstdint>
      |          ^~~~~~~~~
compilation terminated.

Environment (please complete the following information):

OS: linux
Toolchain: GNU ARM Embedded v10 (gcc-arm-none-eabi-10-2020-q4)
Version: v2.4.99

Additional context
The reason that this failure is occurring for this toolchain variant is that the Zephyr build system adds the -nostdinc flag to the build. (zephyr/CMakeLists.txt) This is done with the (obviously wrong) assumption that the build system has discovered all of the implicit system include paths and added them as explicit system include paths. (zephyr/cmake/compiler/gcc/compiler_flags.cmake).
Zephyr's assumption that it is safe to add -nostdinc to the build is wrong. This is because the locations of the standard C++ headers are never discovered. The discovery routine only looks for the paths to the C headers include/stddef.h and include-fixed/limits.h. (zephyr/cmake/compiler/gcc/target.cmake) No attempt is made to look for any C++ headers, even if CONFIG_CPLUSPLUS is enabled. Without the paths to the standard C++ headers, the build fails.
Note that this problem doesn't occur with the host-gcc toolchain variant because in that circumstance the -nostdinc flag is not added to the build.
Proposed Fix(es)
The easiest fix would be to exclude the -nostdinc flag from the build when CONFIG_CPLUSPLUS is set.
The harder fix would be to properly discover all of the system include paths for the C++ compiler when CONFIG_CPLUSPLUS is set and explicitly add those paths to the build. This approach is challenging because the technique used for discovering the location of the C headers does not work for discovering the location of C++ headers (at least with GCC/G++). Passing the -print-file-name flag to g++ does not print full path to the specified header.
The best alternative option this developer could discover would be to use the following compiler invocation (borrowed from StackOverflow here ):
g++ -E -x c++ -v /dev/null

and to parse the resulting output.

Further investigation shows that if CONFIG_NEWLIB_LIBC is set, then the -nostdinc flag is not added to the build. However, even when not linking against newlib, headers such as <cstdint> and <cstddef> should be available.

@palchak-google C++ needs newlib to build. You cannot use C++ on Zephyr without newlib, because it relies on the lib C implementation.
@stephanosio we had this topic several times already. Maybe we should add a dependency to newlib for C++ builds.

@palchak-google Similar issue has been observed and discussed here: #36644.
In general, although not currently enforced, you should consider that, in order to utilise C++ standard headers, you must set CONFIG_LIB_CPLUSPLUS=y (i.e. use libstdc++), which depends on CONFIG_NEWLIB_LIBC, which has the effect of omitting -nostdinc flag.
This problem will eventually be addressed when #36644 is resolved.

Closing this issue since the access to the C++ headers requires CONFIG_LIB_CPLUSPLUS=y, and the header availability issues described here are already described in #36644 and will be fixed there.


Feature/34 introduce unit tests action inputs

Implement unit tests using pytest for action inputs file.

unit tests
for observation of code coverage by unit tests use pytest-cov.
goal is 80+% score.

Release Notes:

unit tests using Pytest for action_inputs.py file


cannot package

I am using Mojo version mojo 24.2.1 (2f0dcf11).
I cloned the mog repository and cd'd to the root of the repo. Then, I ran mojo package mog/ -o mog.📦, resulting in this error output:
Included from mog/__init__.mojo:1:
Included from /Users/dimitri/code/MOJO-PACKAGES/mog/mog/align.mojo:2:
/Users/dimitri/code/MOJO-PACKAGES/mog/external/weave/ansi/ansi.mojo:59:37: error: no matching function in initialization
    var p = DTypePointer[DType.int8](s.data).bitcast[DType.uint8]()
            ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~
mog/__init__.mojo:1:1: note: candidate not viable: expected at most 0 positional arguments, got 1
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'pointer<scalar<si8>>'
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'Pointer[SIMD[si8, 1], 0]'
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'SIMD[address, 1]'
from .style import Style, NO_TAB_CONVERSION
^
Included from /Users/dimitri/code/MOJO-PACKAGES/mog/external/weave/__init__.mojo:1:
/Users/dimitri/code/MOJO-PACKAGES/mog/external/weave/wordwrap.mojo:93:41: error: no matching function in initialization
        var p = DTypePointer[DType.int8](s_bytes.steal_data()).bitcast[DType.uint8]()
                ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~
mog/__init__.mojo:1:1: note: candidate not viable: expected at most 0 positional arguments, got 1
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'pointer<scalar<si8>>'
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'Pointer[SIMD[si8, 1], 0]'
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'SIMD[address, 1]'
from .style import Style, NO_TAB_CONVERSION
^
Included from /Users/dimitri/code/MOJO-PACKAGES/mog/external/weave/__init__.mojo:1:
/Users/dimitri/code/MOJO-PACKAGES/mog/external/weave/wrap.mojo:79:41: error: no matching function in initialization
        var p = DTypePointer[DType.int8](s_bytes.steal_data()).bitcast[DType.uint8]()
                ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~
mog/__init__.mojo:1:1: note: candidate not viable: expected at most 0 positional arguments, got 1
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'pointer<scalar<si8>>'
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'Pointer[SIMD[si8, 1], 0]'
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'SIMD[address, 1]'
from .style import Style, NO_TAB_CONVERSION
^
Included from /Users/dimitri/code/MOJO-PACKAGES/mog/external/weave/__init__.mojo:1:
/Users/dimitri/code/MOJO-PACKAGES/mog/external/weave/truncate.mojo:42:41: error: no matching function in initialization
        var p = DTypePointer[DType.int8](src.data).bitcast[DType.uint8]()
                ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~
mog/__init__.mojo:1:1: note: candidate not viable: expected at most 0 positional arguments, got 1
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'pointer<scalar<si8>>'
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'Pointer[SIMD[si8, 1], 0]'
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'SIMD[address, 1]'
from .style import Style, NO_TAB_CONVERSION
^
Included from /Users/dimitri/code/MOJO-PACKAGES/mog/external/weave/ansi/__init__.mojo:1:
/Users/dimitri/code/MOJO-PACKAGES/mog/external/weave/ansi/writer.mojo:36:41: error: no matching function in initialization
        var p = DTypePointer[DType.int8](src.data).bitcast[DType.uint8]()
                ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~
mog/__init__.mojo:1:1: note: candidate not viable: expected at most 0 positional arguments, got 1
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'pointer<scalar<si8>>'
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'Pointer[SIMD[si8, 1], 0]'
from .style import Style, NO_TAB_CONVERSION
^
mog/__init__.mojo:1:1: note: candidate not viable: argument #0 cannot be converted from 'AnyPointer[SIMD[si8, 1]]' to 'SIMD[address, 1]'
from .style import Style, NO_TAB_CONVERSION
^
Included from /Users/dimitri/code/MOJO-PACKAGES/mog/external/gojo/builtins/__init__.mojo:1:
Included from /Users/dimitri/code/MOJO-PACKAGES/mog/external/gojo/builtins/__init__.mojo:4:
/Users/dimitri/code/MOJO-PACKAGES/mog/external/gojo/builtins/errors.mojo:1:17: error: package 'sys' does not contain 'exit'
from sys import exit
                ^
mojo: error: failed to parse the provided Mojo

Hey! Sorry, I should drop a note in the README until the next Mojo release comes out. There are a few features that rely on changes currently in the nightly/mojo version, namely the Tuple unpacking update. We should probably see a new Mojo release which includes these features within a week or so I would think.
If you want to try this out now, please install the nightly mojo build with modular install nightly/mojo.

Ty!

Some day we'll have an improved packaging system with pinned versions (including mojo version) that makes all this a lot easier. The bleeding edge is bloody after all. 😄

I can't wait for a proper packaging system, it's definitely a challenge for the time being. It's like coding in hard mode just trying to build something while fighting against the compiler, the language, and the packaging haha

^this. 100% this.

Perhaps you could add a badge to the top of the readme. Something like:

I should have included easy-to-copy line:
![Mojo Nightly 24.x](https://img.shields.io/badge/Mojo%F0%9F%94%A5-Nightly_24.x-purple)

Sorry for comment spam -- I should slow down and thing before pressing enter; I'm using this issue like a rapid-fire chat channel.
Note that the sample has the version inside the alt-text ([Mojo Nightly 24.x]) and also in the badge's URL. You'll want to update both as needed.

No worries, I appreciate it! Were you able to build the package while using nightly mojo?

I haven't tried yet. Most of my keyboard time the last couple days has been on client work. And most of my Mojo time was "crazed imaginary coding while I took a few walks to break up the client work hours." 😂
I'm installing nightly/mojo right now. But not finding the right documentation on it -- do you know if you can easily toggle between nightly & stable? I'm not aware of any "nvm" like system in Mojo (yet) or version pinning or anything. After I install, am I basically doing everything in nightly when I mojo blahblah.mojo and mojo package blahblah.mojo -o blahblah.📦 etc...?
Thank you for letting me abuse you as a source of knowledge. 😊

Ah, think I got my answer. The launcher is just determined by PATH search order, so I did this as a lazy quick-n-manual toggle in my ~/.zshrc. In the future, I might add an alias that is hot-swaps my PATH settings. I've done that before in the early days of "how many versions of Python/JS/whatever do I need to keep installed??" But it gets messy. For now, this works.
# mojo versions: stable & nightly
export MOJO_STABLE_PATH="/Users/dimitri/.modular/pkg/packages.modular.com_mojo/bin"
export MOJO_NIGHTLY_PATH="/Users/dimitri/.modular/pkg/packages.modular.com_nightly_mojo/bin"
# mojo version in use
export PATH="$MOJO_NIGHTLY_PATH:$PATH"

Looks like it worked. I haven't tried importing and using the package, but it did run the command and generate the output file. I'll mark this issue closed. Ty!

There's some documentation on using nightly in the mojo GitHub repository but it's pretty sparse like you've said. I usually just update my PATH variables to swap between the two. Not very elegant, but it works


12-hour format time breaks computed time triggers

This is a bug which can prevents calculated time triggers working for people in the U.K., U.S., and other countries where using computers configured using the 12-hour time format.
I created the following sunset trigger (I have randomly scrambled the coordinates for privacy):
At sunset:
  trigger:
    time: sunset
    latitude: 37.413433
    longitude: -0.638290
  action:
    entity: lamp/front/dining
    payload: turn_on

But this gave the error

z2m: [Automations] Timout config error at 8:05:12 PM for [At sunset]

Looking at the code I could see that on line 182 it was using the Date.toLocaleTimeString() function to turn the computed date object into a string to store the trigger time.  I am in the U.K. and use Debian configured with a combination of en_GB and en_US locale settings, so it makes sense the locale would render in the 12-hour format as shown in the error message, "8:05:12 PM".
However the matchTimeString() function understandably only accepts times in the 24-hour format, so the validation check (on line 220) immediately rejects the string when it tests for exactly 8 characters.
As a fix I changed line 182 to const time = times[trigger.time].toLocaleTimeString([], { hour12: false }); to convert the time to the 24-hour format and now the trigger is accepted.  I do not know if this is the best method of forcing this format, compared to options like combing and zero padding the different elements, but it seems the simplest.
Additionally, as in the error message above,  the four debug messages in the startTimeTriggers function have a typo where it should say "Timeout".

Release 1.0.9
Fix conversion toLocaleTimeString().
We use conversion for en-GB with: { hour12: false, hour: '2-digit', minute: '2-digit', second: '2-digit' }.


feat(charts/tigera-operator): support optional affinity

Description
Adds an optional chart value field to support affinity field for the tigera operator.
Related issues/PRs
N/A
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note
Support affinity in tigera-operator chart

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

 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.

@gyuho sorry for the long delay on this one. I've merged master into it and kicked off a test run.


This function doesn't match the API

https://github.com/orbitdb/orbit-db-store/blob/931c452e5197bcd4a2cb716eecfe675da87e2838/src/Store.js#L305
loadFromSnapshot(hash, onProgressCallback)

Load the state of the database from a snapshot. hash is the IPFS Multihash of the snapshot data. Returns a Promise that resolves when the database has been loaded.

@willemneal thanks for reporting this! And sorry for the delay.
Would you care to create a PR that updates the documentation?


fix SLE12 compute nodes

I have a fix for this issue:
M?|Beta*|RC*|GMC*|GM5|GM5+up)
    cs=$cloudsource
    [[ $cs =~ GM5 ]] && cs=GM
    CLOUDDISTPATH=/install/SUSE-Cloud-5-$cs/
    CLOUDCOMPUTEPATH=$CLOUDDISTPATH
    CLOUDDISTISO="SUSE-CLOUD*1.iso"
    CLOUDCOMPUTEISO="SUSE-SLE12-CLOUD-5-COMPUTE-x86_64-GM-DVD1.iso"
    CLOUDLOCALREPOS="SUSE-Cloud-5-official"

SO CLOUDCOMPUTEISO="SUSE-SLE12-CLOUD-5-COMPUTE-x86_64-GM-DVD1.iso" instead of CLOUDCOMPUTEISO="SUSE-SLE12-CLOUD-5-COMPUTE-x86_64*Media1.iso"
Tested and it works, I can deploy sle 12 guests

I think this is right, but I'm not 100% sure because I can't find the IBS project for the released SLE12 Compute media - https://build.suse.de/project/show/SUSE:Products:12-Cloud-Compute:5:x86_64 doesn't have any builds at all.  @vuntz or @dirkmueller can you confirm?
However the commit message is not in the right style - please could you fix?  Here is the all the info you need to know:

https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure
https://help.github.com/articles/closing-issues-via-commit-messages/
http://blog.adamspiers.org/2015/03/24/why-and-how-to-correctly-amend-github-pull-requests/

Didn't double-check, but that feels correct. (needs a rebase)

should be fixed with https://github.com/SUSE-Cloud/automation/commit/c8c5568adc1f50b821aafa5c3389a140229f69ee#diff-c202af9895fd7be9224b15a93a00b0ffR872


Typo "Postgres Hyperscale"→"PostgreSQL Hyperscale"

https://docs.microsoft.com/en-us/azure/azure-arc/data/view-data-controller-in-azure-portal
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/azure-arc/data/view-data-controller-in-azure-portal.md
#PingMSFTDocs

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

#label:"aq-pr-triaged"


Allow progress ?

is there a way to get the progress events or something so we can show users its progress visually?
or even the current extracted size vs the size it will be uncompressed ?

#35  +1


binary version

Not really a bug nor a feature request, but I'd really like you to add a hyprland-bin to the AUR that is somewhat more stable that the git version. As in, compilable. I wanna try ur amazing compositor but fail to compile and it's a big bummer.

well idk why you fail to compile, but if thats because of old wlroots, a binary wont help you.
At any rate, binaries will be distributed once I get everything on the current todo list sorted.

I am using arch and have wlroots-git 0.16.0.r5480.e646d882-1. Is that not the lastest version?

yay hyprland                                                                                 ✔  5679  23:03:11
1 aur/hyprland-git r461.96cdf8f-5 (+0 0.00)
    Hyprland is a dynamic tiling Wayland compositor based on wlroots that doesn't sacrifice on its looks.
==> Packages to install (eg: 1 2 3, 1-3 or ^4)
==> 1
:: Checking for conflicts...
:: Checking for inner conflicts...
[Aur:1]  hyprland-git-r461.96cdf8f-5

  1 hyprland-git                     (Build Files Exist)
==> Packages to cleanBuild?
==> [N]one [A]ll [Ab]ort [I]nstalled [No]tInstalled or (1 2 3, 1-3, ^4)
==> n
:: PKGBUILD up to date, Skipping (1/0): hyprland-git
  1 hyprland-git                     (Build Files Exist)
==> Diffs to show?
==> [N]one [A]ll [Ab]ort [I]nstalled [No]tInstalled or (1 2 3, 1-3, ^4)
==> n
:: (1/1) Parsing SRCINFO: hyprland-git
==> Making package: hyprland-git r461.96cdf8f-5 (Thu May 19 23:03:23 2022)
==> Retrieving sources...
  -> Updating hyprland git repo...
Fetching origin
==> Validating source files with sha256sums...
    hyprland ... Skipped
==> Making package: hyprland-git r461.96cdf8f-5 (Thu May 19 23:03:24 2022)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> Retrieving sources...
  -> Updating hyprland git repo...
Fetching origin
==> Validating source files with sha256sums...
    hyprland ... Skipped
==> Removing existing $srcdir/ directory...
==> Extracting sources...
  -> Creating working copy of hyprland git repo...

-- The C compiler identification is GNU 12.1.0
-- The CXX compiler identification is GNU 12.1.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring Hyprland!
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE
-- Found PkgConfig: /usr/bin/pkg-config (found version "1.8.0")
-- Checking for modules 'wayland-server;wayland-client;wayland-cursor;wayland-protocols;cairo;pango;pangocairo;libdrm;egl;xkbcommon;wlroots;libinput;xcb'
--   Found wayland-server, version 1.20.0
--   Found wayland-client, version 1.20.0
--   Found wayland-cursor, version 1.20.0
--   Found wayland-protocols, version 1.25
--   Found cairo, version 1.17.6
--   Found pango, version 1.50.7
--   Found pangocairo, version 1.50.7
--   Found libdrm, version 2.4.110
--   Found egl, version 1.5
--   Found xkbcommon, version 1.4.0
--   Found wlroots, version 0.16.0
--   Found libinput, version 1.20.1
--   Found xcb, version 1.15
-- Configuring Hyprland in Release with CMake!
-- Configuring done
-- Generating done
-- Build files have been written to: /home/rat/.cache/yay/hyprland-git/src/hyprland/build
cmake --build ./build --config Release --target all -j 10
[2/37] Building CXX object CMakeFiles/Hyprland.dir/src/Compositor.cpp.o
FAILED: CMakeFiles/Hyprland.dir/src/Compositor.cpp.o
/usr/bin/c++ -DGIT_BRANCH=\"main\" -DGIT_COMMIT_HASH=\"2511d8981a0de6e1256662fd5cd4eec0adc2b64e\" -DGIT_COMMIT_MESSAGE="\"    fix a lil oopsie\"" -DGIT_DIRTY=\"\" -I/home/rat/.cache/yay/hyprland-git/src/hyprland/. -isystem /usr/include/cairo -isystem /usr/include/lzo -isystem /usr/include/libpng16 -isystem /usr/include/freetype2 -isystem /usr/include/harfbuzz -isystem /usr/include/glib-2.0 -isystem /usr/lib/glib-2.0/include -isystem /usr/include/sysprof-4 -isystem /usr/include/pixman-1 -isystem /usr/include/pango-1.0 -isystem /usr/include/libmount -isystem /usr/include/blkid -isystem /usr/include/fribidi -isystem /usr/include/libdrm -O3 -DNDEBUG -std=c++20 -DWLR_USE_UNSTABLE -Wall -Wextra -Wno-unused-parameter -Wno-unused-value -Wno-missing-field-initializers -Wno-narrowing -pthread -MD -MT CMakeFiles/Hyprland.dir/src/Compositor.cpp.o -MF CMakeFiles/Hyprland.dir/src/Compositor.cpp.o.d -o CMakeFiles/Hyprland.dir/src/Compositor.cpp.o -c /home/rat/.cache/yay/hyprland-git/src/hyprland/src/Compositor.cpp
/home/rat/.cache/yay/hyprland-git/src/hyprland/src/Compositor.cpp: In constructor 'CCompositor::CCompositor()':
/home/rat/.cache/yay/hyprland-git/src/hyprland/src/Compositor.cpp:70:42: error: too many arguments to function 'wlr_xdg_shell* wlr_xdg_shell_create(wl_display*)'
   70 |     m_sWLRXDGShell = wlr_xdg_shell_create(m_sWLDisplay, 2);
      |                      ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
In file included from /usr/include/wlr/types/wlr_xdg_decoration_v1.h:5,
                 from /home/rat/.cache/yay/hyprland-git/src/hyprland/src/includes.hpp:67,
                 from /home/rat/.cache/yay/hyprland-git/src/hyprland/src/defines.hpp:1,
                 from /home/rat/.cache/yay/hyprland-git/src/hyprland/src/Compositor.hpp:7,
                 from /home/rat/.cache/yay/hyprland-git/src/hyprland/src/Compositor.cpp:1:
/usr/include/wlr/types/wlr_xdg_shell.h:266:23: note: declared here
  266 | struct wlr_xdg_shell *wlr_xdg_shell_create(struct wl_display *display);
      |                       ^~~~~~~~~~~~~~~~~~~~
[11/37] Building CXX object CMakeFiles/Hyprland.dir/src/config/ConfigManager.cpp.o
ninja: build stopped: subcommand failed.
make[1]: *** [Makefile:82: release] Error 1
make[1]: Leaving directory '/home/rat/.cache/yay/hyprland-git/src/hyprland'
make: *** [Makefile:95: all] Error 2
==> ERROR: A failure occurred in build().
    Aborting...
 -> error making: hyprland-git

That's my output.

no its not

Since when is a git version that's up-to-date not the latest? How do I get the latest?

-git is the latest. You just don't have the latest installed. Head is at 5482, yours is at 5480.

Okay I managed to install the right wlroot package and compile it succesfully but it breaks sway. I would like to have both sway and hyprland installed at the same time. Is that even possible? Sway depends on the mainstream package and you are unable to select the git version while installing.

if sway-git doesn't use wlroots-git, no.

well shit
can I not force sway to use wlroots-git or smth? Or should I ask over on sway github?

u cant force sway to use wlroots-git, and I doubt they will.


Describe Primary data origin

It is data genesis!
Which problem is this feature request solving?

Where and from whom data came from.
Currently ODPS does not offers any attributes or mechanism to define it.
Expands the checksums etc

Describe the solution you'd like

A way to surely identify define the source, provide the mechanism to describe it
What kind of steps in between. From where it started and what was the path. Verify the stakeholders?
What kind of entities: machine, human, or something else is related
Where and by what exactly the data was created, prove it. Authenticity
This is something similar to observability, but this is about describing the context among other things from the actual sources behind the value chain (backward)

Any known practical use cases to apply?
Yes we do. It will provide business value. This is coming from practitioners. Details not revealed yet to protect frontrunner business
Can you submit a pull request?
No.
---- Leave intact! Approval of Contributor Agreement -----
By submitting issue you approve the Contributor Agreement, https://governance.opendataproducts.org/v1/contributions/contributor-agreement

Implemented first version of this in the DataOps component. The origin attempts to describe the sources of data

dataOps:
  data:
    schemaLocationURL: http://http://<IP_ADDRESS>/schemas/2016/petshopML-2.3/schema/petstore.xsd
    origin:
      - source: human # sensor, human, analytics
        sourceId:
        type: raw # raw, cleansed
        description:
        checksum: # ?
      - source: sensor # sensor, human, analytics
        sourceId:
        type: cleansed # raw, cleansed
        description:
        checksum: # ?
    lineage:
      dataLineageTool: Collibra
      dataLineageOutput: http://<IP_ADDRESS>/lineage.json

  infrastructure:
    platform: Azure
    region: West US 2 (Washington)
    storageTechnology: Azure SQL
    storageType: sql
    containerTool: helm

  build:
    format: yaml
    hashType: SHA-2
    checksum: 7b7444ab8f5832e9ae8f54834782af995d0a83b4a1d77a75833eda7e19b4c921
    signatureType: JWK
    scriptURL: http://<IP_ADDRESS>/rundatapipeline.yml
    deploymentDocumentationURL: http://<IP_ADDRESS>/datapipeline

Can we somehow add here "as code" part as well? A method to verify the source system and authenticity of theta directly?
Something similar what is in data quality
dataQuality:
  - dimension: accuracy
    objective: 98
    unit: percentage
    monitoring:
      type: SodaCL
      spec:
        - require_unique(member_id)
        - require_range(age_band, 18, 100)

To be moved to the next version.


chore: Rename app id to correspond to the ID planned for notarization use

Renames the app ID specified in the package.json to correspond to the ID we plan to use when notarizing the application for macOS.

:grey_exclamation: Build Aborted

Either there was a build timeout or someone aborted the build.

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-01-04T20:59:28.151+0000

Duration: 63 min 6 sec

Commit: 48448e2784205a704d318e4f6cacde97d1210b1c

Test stats :test_tube:

Test
Results

Failed
0

Passed
53

Skipped
0

Total
53

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

/test


#1689 - Workspace Description Optional

Checklist

[x] manage.py test passes
[x] migrations are included

Affected app(s)

storage
Description of change

Documentation lists the workspace description field as optional but an IntegrityError is thrown if a Null value is provided to the database. Updated the workspace description field to truly be optional and allow null values in the database.

Closes #1689


hpx::naming::locality leaks parcelport specific information into the public interface

Currently, hpx::naming::locality is implemented to hold all locality relevant information for all configured parcelports. The effect of this is that 1) Implementation details of the used parcelport is leaked into the public interface of hpx 2) The class implementation grows in complexity which doesn't scale with the number of different parcelports with different needs.
The solution will be to create parcelport specific locality classes which only contains the parcelport specific information. This changes requires locality factories inside of the parcelports instead of the runtime_configuration as it is right now.

This has been fixed with f891c98


example run error

npm isntall
npx ts-node ./example/example-ws.ts
npx ts-node ./example/example-tcp.ts
example run error, please check sample code

Hi @eaglebetter, thanks for pointing this out.
We have been updating with some breaking changes and the sample code has unfortunately been lagging behind.
Will fix this.


Interpretability (May 18 2022)

Hey -- apologies, got out of the habit of proofing this, but finally got around to it.  I printed this out a while ago, so it may be that some or all of these are fixed by now.  S.
P1013 L4  "don't cause harms"-->"don't cause harm"
P1013 L5 "some harms" --> "some types of harm"
P1013 L9 "Chapter Chapter"-->"Chapter" (a few instances of this, and "Section Section)
P1013 L16 "irreversible harms"--> "irreversible harm"
P1014 L2 "methods" of "interpretability methods"?
P1014 L44 "More broadly..." this is a very long sentence
P1016 L31 "do the explanation" --> "do the explanations"
P1016 L32 "describes of the impact" --> "describes the impact"
P1018 L32 "blackboxes" --> "black boxes"
P1021 L23 "When the model is its explanation" --> "When the model is its own explanation"
P1024 L22 "all inputs predicted to one class" --> "all inputs assigned to one class"
P1034 L26 "provided an interpretability method" --> "given an interpretability method"
P1034 L30 "Figure32.5" --> "Figure 32.5"
P1043 L29 "may enable a person realize" --> "may enable a person to realize"
P1044 L21 "like explanations are simple" --> "like explanations that are simple"
P1044 L29 "Shapely values" --> "Shapley values"

Been and Finale are fixing some of these issues,AFAIK


Add additional Dockerfile packaging logging during debug

Overview
When running taskcat -d package where a taskcat project uses Dockerfile packaging the loggin output is only the last  n lines of the docker run output.
Testing/Steps taken to ensure quality
ran task package against a quickstart with Dockerfile for it's function packaging.
Notes
Optional. Caveats, Alternatives, Other relevant information.
Testing Instructions
How to test this PR Start after checking out this branch (bulleted)
run taskcat -d package in a taskcat project with Dockerfile packaging

Closing in favor of #753


Install iTerm2 and set some necessary options

In particular, Applications in terminal may access clipboard needs to be enabled, as well as Left/Right option key acts as +Esc. We'll need a reasonable way of editing XML files in the setup script, since I don't consider wiping the user's entire config a viable option.
Unfortunately, the version of this in rally-emacs cannot be used as it does overwrite the entire plist file.

This will probably be much easier from Python. See #193.

Blocked by #268.

We no longer have a setup script.


Add "dynamic_state.ml" example

Adds "dynamic_state.ml" to demonstrate the use of dynamically generated
effect names for simulating ML-style references without the use of
universal types.

Very nice!

Excellent 👍


Not opening all plot states in a database

Describe the bug
Visit not loading all plot states of a time-varying database
Helpful additional information

Did VisIt crash: no
Did you get wrong results: no

To Reproduce
Steps to reproduce the behavior. For example:

Launch default version as usual, i.e., /usr/gapps/visit/bin/visit
"Open", as usual, both "serial" and "parallel interactive" exhibit this behavior
Add a mesh plot
Go to the last time state loaded, and it won't be the last one in the database
N.B.: No error is thrown

Expected behavior
All plot states should be loaded up
Desktop

OS and version: running directly from rzgenie as an xwindow
VisIt Version: 3.1.4
Server info (if applicable): rzgenie

Additional context
I can give files on the rz

You can give the files to kbonnell.

You can give the files to kbonnell.

Heh, files are way too large for that . . .  Will email directly to discuss.

Ha!!  As follows:

When I open the database via our *.visit file (text file that points to all of the individual *.silo files, one per line), the pathology presents itself.

When I open the database “directly” (open the *.silo files with Smart file grouping), all plot states are opened as expected.

So, it seems like the way visit is reading the text file is somehow goofy.

You pointed directly to the problem: the .visit file with comments for every timestep/file listed. For some reason we have logic to stop processing the .visit file after encountering 10000 comments.  I can get that fixed for the next release. Can you live with the work-around to open the database directly with Smart file grouping in the meantime?

to open the database directly with Smart file grouping in the meantime?

Are these !TIME type comments or something we don't process? If latter, how about just creating a file without the comments?

Just to be clear for posterity, this is what the *.visit file looks like:
# comment 0
dblplt/dblplt_step000000000.silo
# comment 1
dblplt/dblplt_step000000001.silo
...
# comment <n>
dblplt/dblplt_step<n>.silo
...

Anyway, yes, I can live with the workaround, so not emergent, or even urgent.
Thanks @biagas! :)

What's wrong with having comments in the file that VisIt doesn't necessarily process?

What's wrong with having comments in the file that VisIt doesn't necessarily process?

Nothing but to work around current limitations, I would just create a .visit file without them instead of relying upon smart file grouping which isn't IMHO as robust.

# comment 0
dblplt/dblplt_step000000000.silo
# comment 1
dblplt/dblplt_step000000001.silo
...
# comment <n>
dblplt/dblplt_step<n>.silo
...

@nDimensionalSpace are these comments of the ilk you shared in another conversation aimed at capturing the iteration space of each save (main PDE solve:refinement step:relaxation step:...:)?

@markcmiller86: Yes, the comments are the time for each plot state . . .  Having that data there allows us to easily parse that file, and overwrite states that are going to be recalculated on restarting the code (of course, we could read the Silo files, but that would be a fair bit of new code for us, and this was easier).  Without that, we can get a fair bit of duplication on restart.

It occurs to me...there is no reason (other than the coding needed to enable it) that these comment lines should NOT appear as the dbcomment member of avtDatabaseMetaData. Of course, a comment read from the actual plugin would override. But, I'll add a ticket to allow the comment just ahead of a db line in a .visit file to get routed to avtDatabaseMetadata.


Unable to create more characters

If I click on the "Add" button in the character section, nothing happens.
I can only edit the "Maid" character, but every time the app is repainted this resets
Android 14 Graphene OS
Pixel 6a

Fixed this, will be in next update

Fixed this, will be in next update

Is it fixed in version 1.3.1?
i tried but still have this problem

Oh i see what you mean now


engine: docker-compose log actions should not contain timestamps or service prefixes

Hello @maiamcc,
Please review the following commits I made in branch nicks/dclogs:
47aca0da54791dbfa8a14436b4ca8cce5ca7dcbb (2019-12-03 18:26:04 -0500)
engine: docker-compose log actions should not contain timestamps or service prefixes
Per https://github.com/windmilleng/tilt.specs/pull/19, we're moving towards a system
where the engine stores structured logs, and adds the prefixes later.

yup! exactly, you got it


feat: add sass and less plugin in web builder

PR Details
add sass and less plugin in web builder.
Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[ ] I have updated changeset
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

LGTM 👍🏻


[BUG] ConvertTo-PnPClientSidePage is still available in site documentation

Reporting an Issue or Missing Feature
Removed command is still documented.
Expected behavior
The module documentation still indicate that the command is available although it has been replaced by ConvertTo-PnPPage:
https://pnp.github.io/powershell/cmdlets/ConvertTo-PnPClientSidePage.html
Actual behavior
Command should not be available in the module documentation anymore, as it has been removed.
Steps to reproduce behavior
N/A
What is the version of the Cmdlet module you are running?
N/A
Which operating system/environment are you running PnP PowerShell on?

[ ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[x] Other : N/A

This is an update to #1676 since it has been closed.

Thanks for noticing and making us aware @michaelmaillot. I have removed the file.


allow dynamic properties (fixes #18)

What's the status on this? Been a long time now, and the project I'm working on could very much use this

@RedactedProfile the status is that I'm still waiting for tests to support this.


New environments for CWA

Environment details
There are 5 environments currently for this application and we are in the process of understanding whether one of them can be removed. If this is not possible, are we allowed to have 5 environments in MP?
Application Name
contract-work-administration
Description of application
CWA is the application used by:

Providers to submit their bills and view their contracts and schedules
Contract management to upload contracts and schedules
To hold and update all provider contact details and user management for LAA

GitHub team slug
laa-aws-infrastructure
GitHub code owner team slug
laa-aws-infrastructure
GitHub actions reviewer team slug
laa-aws-infrastructure
Environments

[X] Development
[X] test
[X] Preproduction
[X] Production

Environment access level Development
sandbox
Environment access level Test
developer
Environment access level Preproduction
developer
Environment access level Production
developer
application
Contract & Work Administration
business-unit
LAA
infrastructure-support
<EMAIL_ADDRESS>owner
<EMAIL_ADDRESS>Subnet sets

[X] General
[ ] Isolated

How do users connect to the application
Over the public internet
Additional features
Please check any additional features required

[ ] Additional VPC Endpoints
[ ] Extended DNS Zones
[x] Other - please specify - Bastion access is required due to the age of the operating system.

Other information
No response
Definition of Done
Definition of Done

[ ] account created
[ ] customer informed

Hi @vc13837 we cannot provide 5 separate environments for an application, however if additional environments are needed they can be implemented within a current environment, for example if you need an additional environment for training, servers could be stood up inside the testing or preproduction accounts depending on the data, by creating additional conditional EC2s in that account or using a terraform count on the existing resources which conditionally increases in certain environments. Some teams have put their server code into modules so that they can easily stand up multiple servers in one account.

Named the account contract-and-work-administration to avoid using "&".

I will email this information and access to our user guides to the address above.


[sample app] Bug in Viewpager example

There is a bug in your viewpager example which is also existing in my current project, i used loadvideo in your viewpager example and every time i swipe fast to the next page the previous video plays its audio in the background.

I am able to reproduce, i think it's just a matter of calling the method at the right time. Feel free to send a pull request if you have a fix :)

i am working on a fix but it seems i cant get it right, i am hoping for your help on this bug :)

Hello, is there someone who can help me on this issue (Java)

I also face the same issue, hope someone can help fix it

I looked at the sample code and the reason this happens is probably because sometimes initializedYouTubePlayer variable is null and it simply cant call pause().
initializedYouTubePlayer is sometimes null because the variable is assigned inside onReady callback and the callback doesnt get fired immediately but setMenuVisibility callback probably does, So it tries to pause it before it was assigned.
And everytime you swipe the the fragment fires onCreateView which doesnt help.
To fix it in the simplest way:

Create a global boolean variable named isMenuVisible in the fragment and assign it to boolean comes from setMenuVisibility. Use the boolean inside onReady, if it is true call loadVideo() if not call cueVideo. Then inside setMenuVisibility do the same thing but call pause or play according to the boolean.
(Btw i didnt test this code there might be some syntax issues)

public class ViewPagerFragment extends Fragment {

    private final String videoId = VideoIdsProvider.getNextVideoId();
    private boolean isMenuVisible;
    private YouTubePlayer initializedYouTubePlayer;

    @Override
    public View onCreateView(@NonNull LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        ViewGroup view = (ViewGroup) inflater.inflate(R.layout.fragment_view_pager, container, false);

        YouTubePlayerView youTubePlayerView = view.findViewById(R.id.youtube_player_view);

        getLifecycle().addObserver(youTubePlayerView);

        youTubePlayerView.addYouTubePlayerListener(new AbstractYouTubePlayerListener() {
            @Override
            public void onReady(@NonNull YouTubePlayer youTubePlayer) {
                if (!isMenuVisible) {
                    youTubePlayer.cueVideo(videoId, 0);
                } else {
                    youTubePlayer.loadVideo(videoId, 0);
                }
                initializedYouTubePlayer = youTubePlayer;
            }
        });

        return view;
    }

    // pause when fragment goes offscreen
    @Override
    public void setMenuVisibility(final boolean visible) {
        super.setMenuVisibility(visible);
        isMenuVisible = visible;
        if (initializedYouTubePlayer != null) {
            if (!isMenuVisible) {
                initializedYouTubePlayer.pause();
            } else {
                initializedYouTubePlayer.play();
            }
        }
    }
}

You can also use viewpager2 instead of viewpager. It supports recyclerview adapter and is better than regular old viewpager.

This is perfect! Is there an implementation using viewpager, i am making similar to tiktok but while the video is playing and going to another fragment is audio still plays in the background, does the fix above makes my current onpause, onresume redundant?

@fdcgica
The code above is edited version of viewpager sample code from this library. So if you are using the sample code, replacing or implementing my changes to original code should be enough.
If you are using  getLifecycle().addObserver(youTubePlayerView); you dont need to call pause in onPause or other way around since this line gives your player ability to observe lifecycle and the player will automacially call it for you. But if you are not using that line you will have to call them manually.
And if you are trying to make an app similar to tiktok that has vertical scrolling. I recommend you to use ViewPager2 instead of regular Viewpager. Because ViewPager2 natively supports vertical scrolling and you can even use recyclerview adapter as viewapger 2 adapter.


Support for OpenAPI Specification Version 3.0.1

Description:
OpenAPI Specification 3.0.1 is released at Dec 7, 2017.
See https://github.com/OAI/OpenAPI-Specification/releases
There are API-Definitions out there which uses this version of OpenAPI Specification.
It is not possible to import them in WSO2 API Manager Version 2.5.0. If I try I get the
following error:

Unsupported OpenAPI version provided. Please re-import with OpenAPI 3.0.0

Affected Product Version:
Problem exists in the latest release: 2.5.0.
Steps to reproduce:
As an example you can save this file locally and try to import in WSO2 Version 2.5.0:
https://github.com/opengeospatial/WFS_FES/blob/master/openapi.yaml
Therefore you go to the publisher -> Add New API -> I have an existing API -> Upload the locally saved file

I am facing same issue

In favour of this change as we experience the same issue.
We have found a (rather ugly) workaround. And that is to manually change the API version to 3.0.0 from 3.0.1 which will succesfully be imported. We have not encountered any issues, as of yet. However, we are busy testing.

This is fixed in API-M 3.0.0 onwards. Please find the Git issue [https://github.com/wso2/product-apim/issues/4711]
The above is fixed in latest wum updated packs of API-M 2.5 and 2.6 as well.


update getItemCount to have only return statement

update getItemCount to have only return statement
A function must have one return statement as per Code Complete standard.
What: Updated getItemCount() to have one return statement
Why: It is easier to understand the flow of the function and improved readability
How:
Added only one return statement and reduced lines of code
Checklist:
Documentation
Tests
Ready to be merged
Added myself to contributors table

Thanks for approving my PR. 👍


Feature/RP/remove unused colors

Closes #59 .
By setting a parameter n_colors to the number of class IDs, I limited the number of colors that appear on the colormap. I also had to remove the clim key in the TEN_CLASS_VIS_KWARGS and TWENTY_CLASS_VIS_KWARGS since those were making the labels unnaturally group together. As seen below, the labels are equally spaced from each other.
This current method takes colors that are sampled from even intervals of size of N from the color map (for example tab20), not the first N colors from the color map.
This is what a mesh would look like with the current changes (taken from render_labels.ipynb)

I think you may be able to get the consecutive behavior if you set clim to (0, num_classes+1), but I'm not sure.


Wrong conversion for "get Plane" for UR-Robots

Is there an existing issue for this?

[X] I have searched the existing issues

What Robots version are you using?
1.6.5
What Rhino version are you using?
Rhino 7 on Windows
Current Behavior
Universal robots uses meters as its unit of measurement.
When converting from a plane to Numbers with the "from plane" component the numbers are accurately converted from millimeters to meters (300 becomes 0.3).
When converting in the other direction with the "get plane" component this conversion does not take place.
if the input numbers for x,y,z are 0.3 the resulting plane will be at (0.3,0.3,0.3)  not at (300,300,300) as it should be.
The plugin should be consistent with the way it converts to and from a manufacturers units.
Having a conversion in some components but not in others is error-prone in my experience.
Is this intentional or is it a bug? I can see how it might be the intuitive way for certain workflows.

Expected Behavior
No response
Steps To Reproduce
No response
Anything else?

It looks like a bug, I'll fix it in the next release. Thanks for reporting.

Version 1.6.7 is now available with a fix for this.


fix(backend): use deep hash of backend for cache purging

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#388  👈
#387
#386
#385
#384
#383
#382
#381
#379
#378
#377
#376
#375
#349
#374
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Check if folder name exists - create a new folder with a different name

Let's say I have a URL where every link have the same pageTitle. For example: "TEST".
And when I am downloading images (from these let's say 10 sub-urls), all files are in ONE folder.
I would like to have an index number, to each folder.
But is there a way to check if a folder exists, and if YES, just create a new one with the same name + index (or any other text, number, whatever) ?

But is there a way to check if a folder exists, and if YES, just create a new one with the same name + index (or any other text, number, whatever) ?

Nope. Extensions have no access to the file system.

However, if you batch download these images, you can change the filename pattern to something like
${pageTitle} ${index}/${name}${ext}

so those images will be put in different folders.


Added support to configure custom auth header

Related Issues

closes #5078
Short Description and Why It's Useful

Added support to configure custom auth header. User can define the auth header name in config file, this will work with tokenInHeader configuration. If tokenInHeader set to true, getPayload will use the tokenHeaderName and send token along with it.
Screenshots of Visual Changes before/after (if There Are Any)

Which Environment This Relates To
Check your case. In case of any doubts please read about Release Cycle

[ ] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[ ] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[ ] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[x] I've updated the Upgrade notes and Changelog on how to port existing Vue Storefront sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and Currently Important Rules Acceptance

[x] I read and followed contribution rules

Thanks for review @Fifciu, Let me update code

I'd like to use tokenAuthenticationScheme instead of prefix
https://developer.mozilla.org/en-US/docs/Web/HTTP/Authentication#Authentication_schemes

Sound reasonable. But there might be some custom shapes like AWS4-HMAC-SHA256 which is not a part of the table here (https://www.iana.org/assignments/http-authschemes/http-authschemes.xhtml). I would like to support more than less shapes.
How do you see the implementation?

Updated code, used tokenAuthScheme in default.json config file, default value will be Bearer

Sound reasonable. But there might be some custom shapes like AWS4-HMAC-SHA256 which is not a part of the table here (https://www.iana.org/assignments/http-authschemes/http-authschemes.xhtml). I would like to support more than less shapes.
How do you see the implementation?

Frankly speaking, I haven't got a chance to work with custom shapes.
General pattern for Authorization header is like
Authorization: <type> <credentials>
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Authorization
So I think we can use any type with the current implementation, but if we want to prepare customize token then we need to do customization in token preparation work .


Specto & Library Updates

I use a TV Shows Library that was previously set up with Genesis and has now been set up with Specto for the last 6 months. After the switch to Specto, everything worked perfectly but over the last couple of months, I've noticed a problem that seems to be getting worse.
The issue is with my TV Shows Library not being consistent when it updates new episodes from Specto. Some shows receive the library update for a new episode very quickly whereas others may not update in the library for days/weeks after they appear on Specto and sometimes not at all.
An example is Cold Feet (UK TV Show - S06E01) which I can find in Specto and was aired on 5th Sept. My library updates every 4 hours but 4 days later and after maybe 24 automated updates, it is still not showing in my Library. If I run a manual update, it will also not add it to my library.
The only way I can fix this is to manually delete the whole show out of my library, delete the files off my hard drive and then add the show back to the library again. Doing this will import every episode that is available on Specto including the previously missing episode. But I don't understand why I have to do this when other shows are importing automatically and correctly.
One clue that may help is that all the affected shows are UK shows.....but not all of the UK shows are affected. US shows tend to update within 24 hours on a consistent basis???
I've done a reinstall and clean library but it hasn't solved anything. Does anyone have any ideas of what else I can try or how to fix this as I don't know if it's Specto, TVDB or my Library that's at fault.
Thanks very much for your time.

....and to add something extra, I've just manually deleted 2 shows to add them both back to the library to capture the new episodes. One of them loaded back in without a problem and the new episode was included in the update. The other show now won't scan back in at all despite me deleting the show/files........it says 'adding to library' but when the scan starts, it pauses for a few seconds and the stops scanning? I've tried to rescan it a few times and the same result each time?
I'm wondering if I'm going to need to do a fresh install again but would prefer to try any other options first to save me some time......
Thanks again.

Any ideas anyone please?

old issue, closing


No lambda in JSX and lamda ref conflict

If ref defined as lamda, tslint warn about no lambda in JSX. But ref is string attr, tslint warn about callback to ref prop.
[tslint] Lambdas are forbidden in JSX attributes due to their rendering performance impact
<input type="input" ref={(c) => this._input = c} />

[tslint] Pass a callback to ref prop instead of a string literal
<input type="input" ref="input" />

This is intentional. The recommended pattern for satisfying the jsx-no-lambda rule is to bind your callback elsewhere, not inline. This is orthogonal to the jsx-no-string-ref rule.

Also see #26

Closing as there's no action to take here


Override problem

I tried to override the DummyAuthenticatorService but it give a weird error.
class OverrideAuthService extends DummyAuthenticatorService {
  override def retrieve(implicit request: RequestHeader) = {
    Future.successful(Some(DummyAuthenticator(LoginInfo("abc", "123"))))
  }
}

[error]  found   : Some[com.mohiva.play.silhouette.impl.authenticators.DummyAuthenticator]
[error]  required: None.type
[error]     Future.successful(Some(DummyAuthenticator(LoginInfo("abc", "123"))))

It's the current implemantation
class DummyAuthenticatorService extends AuthenticatorService[DummyAuthenticator] {
  def retrieve(implicit request: RequestHeader) = Future.successful(None)
}

It only works when add the return type
class DummyAuthenticatorService extends AuthenticatorService[DummyAuthenticator] {
  override def retrieve(implicit request: RequestHeader):Future[Option[DummyAuthenticator]] = Future.successful(None)
}

@akkie Should we add the return type for all methods?

We do not need define the return type for every method. Only for methods which have defined a container type like Option or Either. And only if the compiler cannot infer the correct type. If I understand it correctly then Scala allows an overridden method to have a covariant return type. So in the base class the compiler infers None as return type because it's a subclass of Option. But now you try to override the return type None with Some, which ends in a compiler error, because there is no covariance between the both types.

Ok, i'm going review and make the changes then i submit a pull request.

IMHO,
I think that public and protected methods and fields should be explicitly type as a form of contract and documentation. Private ones don't have to be.

Fixed with #354


Fix hover state of inner nav

Goes with https://github.com/UW-Madison-DoIT/angularjs-portal/pull/389.
Does anyone know when or why the border-bottom was changed from 3px to 4px? I think it should be 3px but I'm not sure if this breaks something someone else did.
Before (when hovering on 'MyUW'):

After:

:+1:

Turns out this was my doing, last March. I'm fairly confident it should be 3px, but let's keep an eye on that inner nav.

that'll happen. :+1:


Bug Report: Ghost Tapping Crashes, Chart Zooming acts weirdly, Memory doesn't get cleared

Bugs

[x] Non Ghost Tapping Press Crashes
[x] Chart Zooming is not capped
[x] Chart Zooming behaves weirdly under too low/high zooms
[x] Unused Cached Memory isn't cleared on state switch

Explanation

Disabling Ghost Tapping and pressing any key crashes (it's likely referencing a nulled note instead of passing a direction to the miss function but i don't know where codename stores input stuff to confirm this)

Zooming in on the charter makes bitmaps flicker a LOT but doesn't really increase memory(?)

Zooming out on the charter makes the memory increase (likely backdrops)

Leaving a State doesn't clear unused cached memory for anything, MEANING THAT, you can stack the Chart Zoom-out memory leak until the game becomes unstable

Proposal of fixes for this

for the Non Ghost Tapping issue, instead of passing a note to the noteMiss function (if it is like that, that is), instead, simply pass a integer value representing a direction
for the Chart Zooming issue, it should have been capped from the start, 0.5 minimum, 3 maximum
for the State leaving issue, Just straight up adding a proper way of catching unused data from the cache and clearing it when leaving a state should be enough to address stackable memory leaks

My Target platforms
Windows, Linux

Ghost Tapping issue error report
https://github.com/FNF-CNE-Devs/CodenameEngine/assets/93128334/839a07ba-e70d-4c29-92f5-0a38d240f308
Zooming-in issue, EPILEPSY WARNING!!!

can't really show the zooming out incident here since it crashes the recording software before I'm able to

@NeeEoo close this i deleted that account LOL

@BeastlyGhost no

Idk how to close on mobile


refactor(proto): clean up unused methods in risingwave_pb

Signed-off-by: TennyZhuang<EMAIL_ADDRESS>What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Remove unused methods
Move some methods to common.

Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

LGTM but I guess people would still use those function wrongly. An autocompletion will bring those functions into namespace. If they are only used in tests, name those functions like test_new_struct.

ntfs, there are many other methods.

LGTM but I guess people would still use those function wrongly. An autocompletion will bring those functions into namespace. If they are only used in tests, name those functions like test_new_struct.

ntfs, there are many other methods.

Then at least add docstring for those functions, /// only used for tests.


typo(nativescript-couchbase): Couchbase causes confusion in documentation

It seems that Couchbase class mentioned in https://triniwiz.github.io/nativescript-plugins/api-reference/couchbase.html has actually been renamed to CouchBase in source code.
This causes confusion to people using plugin for first time.

Ah nice catch


DB connect kills the process on error

Link to reproduction
No response
Describe the Bug
It is not possible to use the db.connect method with retrying on error, as it catches the connection errors and kills the process:

https://github.com/payloadcms/payload/blob/main/packages/db-postgres/src/connect.ts#L31
https://github.com/payloadcms/payload/blob/main/packages/db-mongodb/src/connect.ts#L59

IMHO, instead of killing there, the error should be allowed to bubble up and caught and possibly process-killed in Payload's init default connection attempt: https://github.com/payloadcms/payload/blob/main/packages/payload/src/payload.ts#L360
This would allow to set disableDBConnect to false and for example try and retry the DB connection in the onInit handler.
My motivation is to start a dockerized DB in parallel to Payload and just retry until it is available. Using something like wait-for-it.sh always feels like code not handling unready resources very well to me. I might even argue for the default connect attempt in the init method to retry, maybe with an max timeout to allow waiting for starting databases.
To Reproduce

Start Payload without a running DB, even with retrying payload.db.connect() on error in the onInit handler with disableDBConnect set to false.

Payload Version
2
Adapters and Plugins
db-postgres

Hey @c-wygoda I think this is a reasonable request. We could attempt to handle connections as you've noted, and then only throw after a certain amount of failed attempts.
Have you done this successfully with other packages in the past? Say, a raw ORM or similar?
I will add this as a roadmap candidate. For now, you could ensure database availability before you call payload.init(), but I could see how that would be cumbersome and it would be more graceful to have Payload handle it for you.
I'll keep you updated regarding the status of this feature request on our roadmap!


Error "Cannot read properties of null (reading 'replace') when importing a Visio VSDX file

Preflight Checklist

[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
When trying to import the attached Visio 2013 VSDX file (either using using drag-and-drop or the menu) 2080261 Blatt 1.zip, after a short while I get the error "Cannot read properties of null (reading 'replace')".
To Reproduce
Steps to reproduce the behavior:

Open draw.io in the browser
Drag-and-drop the given file onto the drawing area or use the import method from the menu
See error

Expected behavior
The file is loaded and shown.
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 21.1.7

Desktop

OS: Windows 10 Pro 22H2
Browser: Chrome
Version: 111.0.5563.147 (Offizieller Build) (32-Bit)

Additional information: Just tried the same with Firefox 111.0.1 (64-Bit) and got the error message "p.Name is null".

Thanks for the report. This will be fixed in the next release.

Released in 21.1.8


FR - add version in transitive dependencies

Right now transitive dependencies don't include versions, which potentially prevents pulling them up into the consuming app
{
  "identifier": ":jobqueue",
  "usedTransitiveDependencies": [
    "com.birbit:android-priority-jobqueue",
    "org.jetbrains:annotations",
    "androidx.core:core",
    "androidx.annotation:annotation",
    "androidx.lifecycle:lifecycle-common",
    "androidx.arch.core:core-common",
    "androidx.collection:collection",
    "androidx.loader:loader",
    "androidx.viewpager:viewpager",
    "androidx.coordinatorlayout:coordinatorlayout",
    "androidx.interpolator:interpolator",
    "androidx.swiperefreshlayout:swiperefreshlayout",
    "androidx.fragment:fragment"
  ]
}

Fixed by https://github.com/autonomousapps/dependency-analysis-android-gradle-plugin/releases/tag/v0.12.0


Tabulous and Devise Controllers

Hi there,
I'm trying to get tabulous working with devise but am having trouble with the devise controllers.
In config.tabs I have the following line which correctly generates a "Sign Up" tab when the user is not signed in:
[:signup_tab, "Sign Up", new_user_registration_path, !user_signed_in? , true ]

Problems occur when trying to get the tabs to render on the signup page and show "Sign Up" as the active tab. For example, if I have the following line in config.actions I get an exception error:
[:devise/registrations, :all_actions, :signup_tab]

undefined local variable or method `registrations' for #<#<Class:0x007fa8902f40f0>:0x007fa88d881160>

"rake routes" shows that I should be using "devise/registrations" as the controller:
new_user_registration GET    /users/sign_up(.:format)                                      {:action=>"new", :controller=>"devise/registrations"}

As far as I can see, registrations is a controller itself, not a method within the devise controller.
Is there any way around this?

Hi jmaskell,
Thanks for bringing this to my attention.  The nature of your error is Ruby syntax.  :devise/registrations is not a valid Ruby symbol.  Ruby treats this as :devise / registrations, so it expects registrations to be a local variable and hence throws the error.  Tabulous does not expect you to provide the controller namespace, so you can simply provide :registrations as the controller name and it should work.  The following worked for me:
[ :sessions, :all_actions, :signup_tab ],
[ :registrations,  :all_actions, :signup_tab],

which associated the sign up and sign in views with the "Sign Up" tab.
I hope this gets you unstuck!
--Wyatt


The plugin is incompatible without type: "module" in package.json

I tried to use this plugin and got an error
Error [ERR_REQUIRE_ESM]: require() of ES Module node_modules/vite-plugin-bundlesize/dist/plugin/index.js from vite.config.js not supported.
Instead change the require of index.js in vite.config.js to a dynamic import() which is available in all CommonJS modules.
    at Object._require.extensions.<computed> [as .js] (node_modules/vite/dist/node/chunks/dep-6305614c.js:63133:17)

there were no require() statements in vite.config.js nor in node_modules/vite-plugin-bundlesize/dist/plugin/index.js, so I was very confused about this error, the solution was adding "type": "module" in package.json, which caused a lot of errors related to postcss and tailwind module resolution, which I fixed via renaming them from js to cjs.
After fixing the issues, I was able to run the build command, but the file bundlemeta.json was nowhere to be found, because the plugin checks for hardcoded mode: "production" in vite options, instead of process.env.NODE_ENV.
Hopefully this info can help to save some time for people who run into similar issues.

Thanks for filing! And thanks for trying a beta plugin to help make it better.
There are 2 issues here, I think. The first is fixing a bug with CommonJS (legacy) projects—where there isn’t "type": "module" in package.json. In general, all modern Node projects should have this these days—it just runs Node in modern ESM JS. But I never want to list that as a requirement, because Vite wasn’t always running in modern ESM mode, and there are projects that can’t make this upgrade because of old dependencies. But I think this could be fixed silently.
The second issue about the production mode probably requires some documentation, and possibly a warning. I’m not sure why your build isn’t running in production mode, because none of the reported sizes will be accurate if it’s not. But that should fail loudly, or have some warning, or something. And the documentation should make this clearer when it runs.

I pushed a fix in v0.0.5 to fix the mode: "production" bug. Now it uses the Vite-recommended apply: 'build' (docs) for better results.
For the "type": "module" bug, I didn’t want to have to enforce this, but after digging into what it would take to support CommonJS as well, it would be too much. It would basically require maintaining 2 versions of this library, which is not in the cards for now. Since I’d recommend all users operate in "type": "module" just as a generally-better experience, I’ve added it to the docs as a requirement to use this plugin for now.
This resolves the two issues posted. Thanks again for giving this a try in the alpha stages, and please let me know if you encounter any more issues

Thank you


Are CPAPs included?  Mercedes F1 HPP, University College London (UCL)

Free Use Licence
A breathing aid developed by engineers at Mercedes HPP, engineers at University College London (UCL) and clinicians at UCL Hospital has now been approved for use by the UK's National Health Service in the battle against coronavirus.
The designs of a new breathing aid developed by engineers at UCL and Formula One working with clinicians at UCLH have been made freely available to support the global response to Covid-19
https://www.ucl.ac.uk/news/2020/apr/designs-life-saving-breathing-aid-are-made-freely-available
UCL-Ventura breathing aids for Covid-19 patients
https://www.ucl.ac.uk/healthcare-engineering/covid-19/ucl-ventura-breathing-aids-covid-19-patients
** UCL-Ventura breathing aid (CPAP) – Design and manufacturing package
https://covid19research.uclb.com/product/ucl-cpap

Thank you very much for this suggestion, I've added the device to the spreadsheet


using :var: in docstrings to document class attributes

Current coverage is 53.44% (diff: 100%)

Merging #43 into master will not change coverage

@@             master        #43   diff @@
==========================================
  Files             9          9
  Lines           348        348
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            186        186
  Misses          162        162
  Partials          0          0

Powered by Codecov. Last update 2ddb8fa...d679dfb


Updated to reflect Power Automate changes

Category

[x] Content fix
[ ] New article

What's in this Pull Request?
With the rename of Microsoft Flow to Power Automate, there were a number of issues with this article. There have also been many UI changes in flow.microsoft.com since the article was last updated in October, 2018. There have also been licensing changes for triggers like Request.
I've made changes to reflect all of this, to the best of my ability.
Guidance
These corrections will make the article useful again.

thx @sympmarc, appreciated.


Strange enqueue behaviour, with nothing being enqueued at all. Needs fix in Enqueue.php:297 source

I am not sure what I am doing wrong, but I had numerous plugins and themes already developed with wpack.io (version 5.x) with no issues. This time however whenever I try to enqueue my ts/scss into the backend nothing shows up. I debugged the issue and possibly solved it.
in my wpproject:
name:'referral',
entry: {
	reftable:'./src/js/reftable.tsx',
	reftablestyles:'./src/scss/main.scss',
},

in the plugins main file:
include_once 'vendor/autoload.php';
	global $enq;
	$enq = new \WPackio\Enqueue('dbireferral', 'dist', $ver, 'plugin', __FILE__);

and in the hook to enqueue scripts
add_action('admin_enqueue_scripts', function(){
    global $enq;
    $res = $enq->enqueue('referral', 'reftable', []);
    $enq->enqueue('referral', 'reftablestyles', []);
});

So far pretty standard. However with the current setup it did not work for me. I debugged it and compared it with other plugins I had developed with wpack.io. The reason seems to be in wpacks main php file: Enqueue.php in the getAssets function
It loads the part of the manifest where the js files are all linked.
$enqueue = $manifest['wpackioEp'][ $entryPoint ];
For some reason the manifest looks different for my older plugins vs my current one. And it differs by adding an assets node around the js/css nodes. That way the above code in $enqueue doesn't give any results and nothing is enqueued:
 wpackioEp:{     reftable: {         assets: {             js: [....]            ....         }     } }
If in line 297 you change the enqueue variable it seems to work:
$enqueue = $manifest['wpackioEp'][ $entryPoint ]['assets'];
Maybe it has something to do with versions and how webpack delivers data. or maybe it is specific to backend. But someone should look at it, why this happens.

Confirmed that behaviour.
With "@wpackio/scripts": "6.4.0" (npm) and "wpackio/enqueue": "2.2" (composer) I get the behaviour described above. So I believe something changed in the web pack config and it doesnt work with the php side anymore, which needs a change as described above. Maybe there is a setting to prevent this "assets" node to be created.
With "@wpackio/scripts": "5.0.0" (npm) and "wpackio/enqueue": "2.2" (composer) everything works fine and as expected.
It's a shame that wpack is not really being supported developed anymore. Was such a great library! : /

Hello,
With @wpackio/script v6.4.0, you have to use wpackio/enqueue v3.4.0. There has been a change in how manifest json files are generated from the upstream package and the PHP package has been updated accordingly. You can safely upgrade both to the latest version and it should work.
I would appreciate if you'd like to add this to the website documentation.


Cassandra Latency

Hello everyone,
I am benchmarking a Cassandra installation on Azure composed of 4 nodes (Standard_D2S_V3 - 2vCPU and 8GB ram) with a replication factor of 2.
To benchmark this testbed, I am using a single YCSB instance with the workload C (100% read request), a Consistency level ONE and only 10 clients (so very low load).
However, I founded that the average latency gathered by YCSB is much different from the one obtained from the JMX (org.apache.cassandra.metrics:type=ClientRequest,scope=Read,name=Latency)
I ran the workload for over an hour and:

YCSB returns around 4000 us as average latency
JMX returns around 780 us as average latency

It is a quite an important difference in performance that I don't know how to justify.
Do you have any idea from where this difference comes from?
Regarding the throughput, the value is the same across both the reading (~2400ops/sec)
Thank in advance

where is the YCSB client running?

@dipietro-salvatore, ycsb client reported latency is addition of network latency + cassandra latency (which is available through JMX metrics). Please validate your network latency.

closing as stale. if you're still having issues please reopen and address the questions above.


Same component ID for two structs in a specific case

I found a weird bug.

This occurs because the two components used ComponentA and ComponentB share the same component ID but only in this specific case.
Here is my code
main.cpp
#include "ComponentA.h"
#include "ComponentB.h"
#include "entt/entt.hpp"

void test(entt::registry& registry) {
	auto view = registry.view<ComponentA>();

	for (auto e : view) {
		view.get<ComponentA>(e);
		break;
	}
}

int main() {
	entt::registry registry;

	auto entity = registry.create();
	registry.assign<ComponentB>(entity);

	test(registry);
	return 0;
}

ComponentA.h
#pragma once

struct ComponentA {
	float val1, val2;
};

ComponentB.h
#pragma once

struct ComponentB {
	float val;
};

System.h
#pragma once
#include "entt/entity/registry.hpp"
#include "ComponentA.h"

void Update(entt::registry& manager) {
	auto view = manager.view<ComponentA>();
}

System.cpp
#include "System.h"

If anything changes, like for example System.cpp does not include System.h or ComponentB has two variables instead of one the bug does not show.
I got this Bug on Visual Studio 2017

Are you working across boundaries?

Not that i know of. All the files used are put in the project folder and as you can the code used does not access anything other than the files in the project folder. In theory if create a empty project, create the same files and compile is should also crash. If not then the problem is most likely with my system.

Not that i know of.

Ok, sorry, I was in a rush and I couldn't drop down more than that comment.
What I meant is if you're using shared or static libraries in your application and therefore you pass the registry from the main executable to those libraries.

This occurs because the two components used ComponentA and ComponentB share the same component ID

In general, components don't share the same id. This is sequentially generated on the fly and we can rely on the fact that it works, otherwise the whole library would be unusable (and it is not, ofc).
In case of shared/static libraries on Windows though, you've to use named components to work around an idiosyncrasy of the system. That's why I'm asking.

To make sure that i accidentally didn't have some includes or libraries linked i created a new empty C++ project. Download the repository as zip and copied entt-master/src/entt in the project. After that i created each file in the same order as listed above and copied the given code. Went into project options and set language standard to c++17. The same problem occurred after compiling and running the program.

I cannot reproduce it in any way. I've created the files in the order you mentioned and with the code you mentioned but that's perfectly normal code and it works as expected. There isn't much I can do here. It definitely doesn't look like a bug of this library, I'm sorry.

@Steki95 Could you provide a call stack for the assertion?

I have installed the newest version of visual studio and now the bug has disappeared so it was a problem with my version of visual studio and not the library. Thanks for the help.

You're welcome.


Lint with --profile showing suspiciously low num_eval count

Linting Regal's own bundle directory, which contains 118 Rego files,
{
  "location": "/regal/rules/style/line_length.rego:18",
  "total_time_ns": 748651746,
  "num_eval": 364,
  "num_redo": 182,
  "num_gen_expr": 4
},

Line 18 is the last line in this snippet:
some i, line in input.regal.file.lines

line_length := count(line)
line_length > cfg["max-line-length"]

OK, so number crunching isn't our strong side, but...
Cloc tells us we have 5638 lines of code, and of course even more counting whitespace
$ cloc bundle
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Rego                           118           1461            516           5637

As each line of each file is evaluated, I'd expect the num_eval number to be somewhere close to that, but instead we have 364. Perhaps I need to be educated in how that works, or perhaps there's an issue in how we collect and report that.

If I had to guess, the profile is perhaps overwritten for each lint run...?  Will need to dig in 🤓

There's some profile aggregation logic over here, perhaps it could come in handy.


Question: Available on a CDN?

I need to load this module in a browser from a CDN. I can't find it anywhere. Any ideas?
e.g<EMAIL_ADDRESS>seems unhappy.

Was able to create a custom build and host it so it works now.


Multiple buttons

It would be great to be able to use multiple buttons in the form. Maybe we could pass them via children ?

Sure! Do you want to work on a patch?

Yes, of course. It should be just as simple as to replace :
<button type="submit">Submit</button>
 by { children ? children :
<button type="submit">Submit</button>
 } (I just tested it). I may also add a test and some doc in README.md if this solution is OK for you.

It should be just as simple as to replace

We'll need to expose the submit, change and error handlers so children can leverage them as well.

I use onClick handlers on children buttons (and I need to save formData on my parent component's state to be able to pass it to these handlers). It's mostly because I don't know if it is possible to know which submit button was clicked in the handleSubmit handler...

It's mostly because I don't know if it is possible to know which submit button was clicked in the handleSubmit handler

I don't get it, so how do you get form validation?

In my use case, the form validation is made on the server side and for some buttons (e.g. a Cancel button), we don't want form validation). I agree it is an exception and we need to be able to validate the form in each button.
What do you mean by

expose the submit, change and error handlers so children can leverage them as well

?

I'm just realizing that adding a button type=submit is enough to trigger the form submission, which triggers the validation anyway. So yeah, just being able to pass a submit button as a children is enough. Nice.
Could you take care of working on a patch?

The problem is that form validation happens after the click event on the button. Then I tried to set some flag in the state of my parent component and to use this flag in the onSubmit handler but when I do this, the values in my fields "dissapears" and so the validation fails. It's strange that setting the state on my parent component can have an effect on the fields... So maybe it's not a good idea to have multiple submit buttons.
I can work on the patch though. It will be useful to allow users to use their own markup for buttons instead of the default submit button.

Fixed by #34

I was not able to make the above solution work,
My use case is,
I want 2 submit buttons on my form. onclick of them, separate flows will be kicked in but both of them need the formdata, I tried the above approach but my non default button's onclick/onsubmit handler is not getting data. Is this feature currently supported?
Please help

@bhunwal please see the "Validate and submit form from external control" example in tips and tricks


Add bump version command that creates a git tag and a bower.json file (issue #203)

Description of the change
Implements #203. Not yet feature complete.
Checklist:

[ ] Added the change to the "Unreleased" section of the changelog
[x] Added some example of the new feature to the README
[x] Added a test for the contribution (if applicable)

It looks like there are quite a few conflicts with #272 -- once that is merged I will resolve them.

It looks like there are quite a few conflicts with #272 -- once that is merged I will resolve them.

@Dretch this looks quite good already, while I think #272 will need some more work, so I'd say we could try to merge this first and then rebase that one? (cc @jonathanlking)
What's missing from here to be feature complete?

@Dretch @f-f please go ahead and merge this first - I’m very happy to rebase and I still need to do some work.

@f-f

What's missing from here to be feature complete?

Right now I thinking of:

Taking the license and repository field from spago.dhall and putting them in the generated bower.json.
Readme and --help message tweaks
Moving all messages into Messages.hs
Maybe handling hash and branch versions from the package set (right now I think this will just create a broken bower.json)
Maybe running bower install after generating the bower.json, as a sanity check.
Maybe more tests.

@Dretch thanks for the summary. Some suggestions:

Taking the license and repository field from spago.dhall and putting them in the generated bower.json

For doing this let's add an ensurePublishConfig nearby here that tries to conform to a new PublishConfig type

Moving all messages into Messages.hs

As time goes I'm less sure the Messages module is useful, as you lose a bit of context if the relevant error message is one indirection away. So far the general attitude in my PRs has been to only put there multiline messages, and keep short ones in the relevant places. But I'd say to not worry much about all of this

Maybe handling hash and branch versions from the package set (right now I think this will just create a broken bower.json)

Right, great point. Fortunately we have some facilities for this already 🎉
You could use getMetadata (we already use this for the global cache) to get a map containing (among other things) a list of tags for every package in the set, so you could just check if all the refs are in there.

Thanks for the feedback @f-f

For doing this let's add an ensurePublishConfig nearby here that tries to conform to a new PublishConfig type

I will look at doing this.

As time goes I'm less sure the Messages module is useful, as you lose a bit of context if the relevant error message is one indirection away. So far the general attitude in my PRs has been to only put there multiline messages, and keep short ones in the relevant places. But I'd say to not worry much about all of this

Okay, great.

Right, great point. Fortunately we have some facilities for this already. You could use getMetadata (we already use this for the global cache) to get a map containing (among other things) a list of tags for every package in the set, so you could just check if all the refs are in there.

Okay, I'll look at getMetadata. There are two options I can think of for handling non-tagged versions, though:

Fail when generating the bower.json if the package set refers to any such versions.
Generate a bower.json with Git URL versions, like:

dependencies: {
  "mkdirp": "https://github.com/joshuahhh/purescript-mkdirp.git#master"
}

Do you have any preference?

@Dretch yeah, I also have no idea what's going on. The closest thing I found is this issue, that mentions that inproc could be a problem here (we use inproc for Git and procs for Bower)

@Dretch yeah it looks like https://github.com/spacchetti/spago/pull/289/commits/3c14203486314c42b1b25705ea65c1814d7756b7 fixed it 🎉

This looks reasonable to me, but if I am reading correctly I think this is currently assuming that a package called foo in your package set is always the same as the package purescript-foo on Bower? That's only a safe assumption if you're using an official package set with no additions or overrides, right?

(Not that this should prevent this from being merged, just something that's probably worth being aware of.)

@hdgarrood yeah that's right. I think this assumption is reasonable as we use it around already (e.g. when suggesting deps names), though your comment made me realise that with this implementation you'd be allowed to publish a package that only depends on packages in the official set because we search for every dependency's tag in the "packages metadata", which is a big map containing all tags and commits for all packages in the official package set
So maybe we should just generate the bower file and try to install and see if bower complains? (cc @Dretch)

So maybe we should just generate the bower file and try to install and see if bower complains? (cc @Dretch)

Oh yes of course, this PR does this already -- it's necessary for pulp publish to succeed -- so that should be fine.

@hdgarrood yeah, what I meant there is that we should remove the check here otherwise one wouldn't be able to publish anything that doesn't depend from the packages in the package set.


[r19.03] cloudfoundry-cli: 6.41.0 -> 6.43.0, fixing CVE-2019-3781

Motivation for this change
https://nvd.nist.gov/vuln/detail/CVE-2019-3781
Things done

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

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

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

@GrahamcOfBorg build cloudfoundry-cli


Incorrect selection in reactions

Affected Version: Remote
Steps to Reproduce:

Open
15Ritalin.zip in Ketcher.
Create two more structures at canvas.
Add plus sign and reaction arrow between them.
Select part of Ritalin structure that contains nitrogen atom.
Click 'Calculated Values' button.

Expected Result:
In 'Chemical Formula' field correct formula of selected part of structure displays.
Actual Result:
'Select' send incorrect values when there is reaction on canvas.

NB: Issue will be fixed after after implementing 'Internal Format' milestone.

It will be fixed in scope of 'Internal Format' milestone

Reopened as result in Standalone and remote is different

Case about incorrect selection was fixed by our side but after calculate we get different results in Remote and Standalone modes - this is bug of BE.
Related bug Indigo: https://github.com/epam/Indigo/issues/452

Th rest  of issue should be fixed on BE side so it has been moved to release 2.5


#2585 Show evaluation timeframe of selection evaluation

Fixes #2585

Codecov Report

Merging #2664 (242e1b0) into release-0.7.3 (a8301e1) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           release-0.7.3    #2664   +/-   ##
==============================================
  Coverage          35.08%   35.08%
==============================================
  Files                158      158
  Lines               9205     9205
  Branches             195      195
==============================================
  Hits                3230     3230
  Misses              5480     5480
  Partials             495      495

Flag
Coverage Δ

moduleA
22.54% <ø> (ø)

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

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 a8301e1...242e1b0. Read the comment docs.


"The connection to the terminal's pty host process is unresponsive, the terminals may stop working"

Issue Type: Bug
I regularly receive the following error in VSCode, whether I have a terminal open or not:

The connection to the terminal's pty host process is unresponsive, the terminals may stop working

It is to the point where the terminal is unusable. I have done an extension bisect, and it occurs with all extensions disabled. I've also done a fresh install of the latest version of VSCode and it occurs even with no additional extensions installed.
Note that in the information below I have installed a few of my frequently-used extensions back into VSCode.
VS Code version: Code 1.69.2 (Universal) (3b889b090b5ad5793f524b5d1d39fda662b96a2a, 2022-07-18T16:12:57.074Z)
OS version: Darwin x64 21.6.0
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz (8 x 2900)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
7, 6, 5

Memory (System)
16.00GB (0.26GB free)

Process Argv
--crash-reporter-id 4833826f-69e8-4602-b6d3-7d5f7424e89c

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

markdown-mermaid
bie
1.14.2

EditorConfig
Edi
0.16.4

markdown-table-formatter
fcr
2.2.4

vscode-yaml
red
1.9.1

markdown-all-in-one
yzh
3.4.3

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vscscmwlcmt:30465135
cppdebug:30492333
pylanb8912:30529769
vsclangdf:30486550
c4g48928:30535728

/duplicate https://github.com/microsoft/vscode/issues/130320

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

@meganrogge - Please reopen. #130320 references a Windows solution, but this setting does not exist for macOS.

@drenze-athene what will work for you is to set terminal.integrated.enablePersistentSessions to false

@drenze-athene what will work for you is to set terminal.integrated.enablePersistentSessions to false

That did not work, continuing to get these errors.


we must  conda install ffmpeg

When we choose the inference mode as "3s high-speed replication", if we do not install ffmpeg, the program will report an error and lack a component called ffmpeg. However, if we install ffmpeg in a traditional way (such as pip install ffmpeg), the program will still report an error and we need to install it using conda install ffmpeg. Afterwards, we can use the '3s high-speed replication' normally.

The ffmpeg installed by pip is just a highly abstracted library for using ffmpeg. It is not a download of ffmpeg as a program. But conda downloads ffmpeg as software. But conda's version of ffmpeg is too low, still 4.0. Usually ubuntu or other systems are using version 6.0.


Implement HiDPI support on Wayland

Windows now keep track of the monitors they are on, so we can calculate
the best scaling factor for them, by using the maximum of each of the
monitors.
The compositor scales down the buffer automatically when it is on a
lower density monitor, instead of the previous way where it was scaling
up the buffer on higher density monitors, which makes the application
look much better on those ones.

Pushed a newer version, addressing all of @jadahl’s comments.

I just pushed the commit with two changes:

you used min(x1, y1) which doesn't exist. I defined in wl_init.c for now.
I added monitor->wl.scale = 1; to monitor creation as so is specified by the spec (no .scale event == scale is 1). Not doing this would not cause a bug since you always set the buffer scale to at least 1, but so changed just to do what the spec says.

I also added a related extra commit doing version checks on wl_compositor so we don't try to set the buffer scale if it's not supported. This is very unlikely to ever to happen since compositors will probably implement both these features at the same time, so it was more or less only for correctness, not as much for practical reasons.