Support negative Version numbers for time travle

I often find myself in situation where I want to compare the current state of a delta table with the previous state (e.g. before a MERGE/UPDATE)
right now this is always a bit cumbersome as I need to get the version/timestamp of the previous transaction using DESCRIBE HISTORY before I can use [VERSION / TIMESTAMP] AS OF xxx
It would be great to have the ability to define negative version numbers like VERSION AS OF -1 to get the previous version, VERSION AS OF -2 to get the version two commits ago etc.
basically similar to what you can do in Python with arrays to select the trailing elements

@zsxwing I can take this if not already in progress

@zsxwing should this be supported in the startingVersion for streaming scenarios? I was thinking of leaving that out because it would be idempotent unlike a true version number.
Example: If I set startingVersion=-1 on run 1 and have 123 versions, I'll start at version 122. If I run the code again on run 2 some time later, I may start at version 128.

Example: If I set startingVersion=-1 on run 1 and have 123 versions, I'll start at version 122. If I run the code again on run 2 some time later, I may start at version 128.

I think this applies to batch queries and streaming queries using startingVersion=latest as well. For streaming queries, once we start the query, it should be written to the checkpoint and after that running the query will ignore startingVersion. Hence it should be fine.

@gbrueckl @SpaceRangerWes After thinking about this a little more I'm a little worried that this can be misused and can lead to confusing behavior for users. Can you help me understand your use case better ?
If it is to just find what rows were added/updated by your command just finding the last version using history and selecting that version would work (although its two steps instead of 1).
The problem with supporting -1 is that It can be misunderstood by users in a way that they could think it's their last commit, but it could be anyone's commit on the table.  Apart from -1 I do not see much value for -n.
Unless I'm missing the use case I'm inclined to decline this PR #673, feel free to point out some other use cases if I'm missing them.

@rahulsmahadev your explanation is good enough for me. Can't really argue with it.

@rahulsmahadev thanks for the explanation - that makes a lot of sense
the initial idea was to have an easy/generic way to compare the existing version with the previous version - mainly for data validation purposes
in pure Spark SQL it is hard to get this information otherwise (PySpark, Scala, etc. would of course still work just fine as you are much more flexible there)
maybe another approach would be to allow sub-queries over DESCRIBE HISTORY and the ability to make VERSION AS OF x also able to use sub-queries?

Thanks, will close this issue for now.


Upgrade 5.2.1

Ping @cduverne

LGTM !
Is the change to "mattermost" CLI instead of "platform" a long-term choice ?

@cduverne Yes it is a long-term choice from the Mattermost developers


Feature/improve SES documentation

Brings SES documentation in line with SNS documentation:
http://docs.getmoto.org/en/latest/docs/services/sns.html
Based on Stack Overflow answer here:
https://stackoverflow.com/a/74044893/363784

I think I've fixed the lint problems now - I'm not sure why the sns check failed, it seems to be an ephemeral problem.

Resulting docs: https://moto--5585.org.readthedocs.build/en/5585/docs/services/ses.html


Build-errors due to format-security

Describe the bug
Same build error as in #1096
string-format warning in upstream code causes compilation error with default compiler flag.
To Reproduce
Using latest meta-ros on branch kirkstone:
meta-ros2-humble     = "HEAD:685215f9837d47089d2eaaa296c1dac574f44ddf"
Running:
bitbake teleop-twist-joy or  bitbake nav2-costmap-2d
Build Configuration:
BB_VERSION           = "2.0.0"
BUILD_SYS            = "x86_64-linux"
NATIVELSBSTRING      = "universal"
TARGET_SYS           = "aarch64-fslc-linux"
MACHINE              = "imx8mn-var-som"
DISTRO               = "fslc-xwayland"
DISTRO_VERSION       = "4.0"
TUNE_FEATURES        = "aarch64 armv8a crc crypto"
TARGET_FPU           = ""
DISTRO_NAME          = "FSLC Wayland with XWayland"
ROS_DISTRO           = "humble"
ROS_VERSION          = "2"
ROS_PYTHON_VERSION   = "3"

Log:
| FAILED: CMakeFiles/teleop_twist_joy.dir/src/teleop_twist_joy.cpp.o
| /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot-native/usr/bin/aarch64-fslc-linux/aarch64-fslc-linux-g++ -DDEFAULT_RMW_IMPLEMENTATION=rmw_fastrtps_cpp -Dteleop_twist_joy_EXPORTS -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/build -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/git/include -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/geometry_msgs -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/std_msgs -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/builtin_interfaces -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rosidl_runtime_c -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rcutils -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rosidl_typesupport_interface -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rosidl_runtime_cpp -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rosidl_typesupport_fastrtps_cpp -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rmw -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rosidl_typesupport_fastrtps_c -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rosidl_typesupport_introspection_c -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rosidl_typesupport_introspection_cpp -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/ament_index_cpp -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/libstatistics_collector -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rcl -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rcl_interfaces -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rcl_logging_interface -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rcl_yaml_param_parser -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/tracetools -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rcpputils -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/statistics_msgs -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rosgraph_msgs -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rosidl_typesupport_cpp -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rosidl_typesupport_c -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp_components -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/class_loader -I/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/sensor_msgs -march=armv8-a+crc+crypto -fstack-protector-strong  -O2 -D_FORTIFY_SOURCE=2 -Wformat -Wformat-security -Werror=format-security  --sysroot=/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot  -O2 -pipe -g -feliminate-unused-debug-types -fmacro-prefix-map=/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0=/usr/src/debug/teleop-twist-joy/2.4.5-1-r0                      -fdebug-prefix-map=/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0=/usr/src/debug/teleop-twist-joy/2.4.5-1-r0                      -fdebug-prefix-map=/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot=                      -fdebug-prefix-map=/workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot-native=  -fvisibility-inlines-hidden -fPIC -Wall -Wextra -Wpedantic -std=gnu++17 -MD -MT CMakeFiles/teleop_twist_joy.dir/src/teleop_twist_joy.cpp.o -MF CMakeFiles/teleop_twist_joy.dir/src/teleop_twist_joy.cpp.o.d -o CMakeFiles/teleop_twist_joy.dir/src/teleop_twist_joy.cpp.o -c /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/git/src/teleop_twist_joy.cpp
| In file included from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/logging.hpp:24,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/client.hpp:40,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/callback_group.hpp:24,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/any_executable.hpp:20,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/memory_strategy.hpp:25,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/memory_strategies.hpp:18,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/executor_options.hpp:20,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/executor.hpp:37,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/executors/multi_threaded_executor.hpp:25,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/executors.hpp:21,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/rclcpp.hpp:155,
|                  from /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/git/src/teleop_twist_joy.cpp:33:
| /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/git/src/teleop_twist_joy.cpp: In lambda function:
| /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rcutils/rcutils/logging_macros.h:72:18: error: format not a string literal and no format arguments [-Werror=format-security]
|    72 |       rcutils_log(&__rcutils_logging_location, severity, name, __VA_ARGS__); \
|       |       ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rcutils/rcutils/logging_macros.h:747:3: note: in expansion of macro 'RCUTILS_LOG_COND_NAMED'
|   747 |   RCUTILS_LOG_COND_NAMED( \
|       |   ^~~~~~~~~~~~~~~~~~~~~~
| /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rclcpp/rclcpp/logging.hpp:970:5: note: in expansion of macro 'RCUTILS_LOG_WARN_NAMED'
|   970 |     RCUTILS_LOG_WARN_NAMED( \
|       |     ^~~~~~~~~~~~~~~~~~~~~~
| /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/git/src/teleop_twist_joy.cpp:184:11: note: in expansion of macro 'RCLCPP_WARN'
|   184 |           RCLCPP_WARN(this->get_logger(), result.reason.c_str());
|       |           ^~~~~~~~~~~
| /workdir/yocto/build_fslc-xwayland/tmp/work/armv8a-fslc-linux/teleop-twist-joy/2.4.5-1-r0/recipe-sysroot/usr/include/rcutils/rcutils/logging_macros.h:72:18: error: format not a string literal and no format arguments [-Werror=format-security]
|    72 |       rcutils_log(&__rcutils_logging_location, severity, name, __VA_ARGS__); \

Expected behavior
Builds without error (but with warnings?)
Seems the problem is upstream code which relies on printf without format-strings. So the warning is valid, but the handling as error causes issues for us.

Seems to originate from:
# $SECURITY_STRINGFORMAT [3 operations]
#   set? /workdir/yocto/sources/poky/meta/conf/distro/include/security_flags.inc:18
#     "-Wformat -Wformat-security -Werror=format-security"

The security_flags.inc file even contains a few exceptions already:
# Recipes which fail to compile when elevating -Wformat-security to an error
SECURITY_STRINGFORMAT:pn-busybox = ""
SECURITY_STRINGFORMAT:pn-gcc = ""

Suggested fix:
Add exceptions for the problematic packages somewhere in meta-ros2-humble:
# Recipes which fail to compile when elevating -Wformat-security to an error
SECURITY_STRINGFORMAT:pn-teleop-twist-joy = ""
SECURITY_STRINGFORMAT:pn-nav2-costmap-2d = ""

Is meta-ros2-humble/conf/ros-distro/include/humble/ros-distro.inc the correct location for this?

Thanks for reporting this.  I would suggest keeping it localized to the recipe.
https://github.com/ros/meta-ros/blob/e45cd718ea2d319afa2c9a6b610e55397df7d0c7/meta-ros1-melodic/recipes-bbappends/microstrain-mips/microstrain-mips_0.0.3-1.bbappend#L13
There doesn't appear to be a bbappend for teleop-twist-joy but there is one for nav2-costmap-2d:
https://github.com/ros/meta-ros/blob/kirkstone/meta-ros2-humble/recipes-bbappends/navigation2/nav2-costmap-2d_1.1.12-1.bbappend
You could set SECURITY_STRINGFORMAT = "" as one possible fix.
Another route would be to demote the error back to a warning with:
CXXFLAGS += "-Wno-error=format-security"
That may be more suitable as it shows the intent and may be more obvious to remove when no longer needed.


WIP multicolor lines

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Does your submission pass tests?

This a PR related to #944 (I have built a bundled version with this there for testing)
It should not affect existing code that uses lines as colored(per vertex) lines are not used yet.
isTransparent
This also opens a discussion about isTransparent, as it should be pre-calculated by those providing the geometry, so renderer does not need to go through all colors to see if there is alpha channel somewhere. I am asking advice to how to approach this so it is consistent in jscad.
@z3dev my code as usual is scrapped so it does something, please review as you usually do with your attention to details :)
meshShader.js was not changed functionally, but only flipped position of frag and vert to more easily compare difference of single color vs colorpervertex.
#3d lines
Also line drawing shader supports lines in 3d space, that is useful for visualisations, and maybe can be added to modeling. As modeling can be pushed a bit further to also allow for shapes that are for debugging, or presenation, but not for 3d print output.

entitiesFromSolids.js converts isTransparent from the solids definition to
transparent: geometry.isTransparent,

but the field was not used in the shader. It is wise to disable blending for non transparent geometries,
and probably disable z-buffer for transparent geometries.
this is a draft PR, the changes I made initially were to be able to move fwd with my prototype, but here we can take time to also make some improvements to the code and how it is organized.

@hrgdavor there's a little issue when running the demo-cli, i.e. headless frame capture. can you take a look at this?
node ./demo-cli.js

/Users/z3dev/dev/jscad/OpenJSCAD.org.feat-multicolor-lines/packages/utils/regl-renderer/src/rendering/commands/drawMesh/index.js:57
    depth = { enable: !transparent }
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

SyntaxError: Invalid shorthand property initializer
    at wrapSafe (internal/modules/cjs/loader.js:979:16)
    at Module._compile (internal/modules/cjs/loader.js:1027:27)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
    at Module.load (internal/modules/cjs/loader.js:928:32)
    at Function.Module._load (internal/modules/cjs/loader.js:769:14)
    at Module.require (internal/modules/cjs/loader.js:952:19)
    at require (internal/modules/cjs/helpers.js:88:18)
    at Object.<anonymous> (/Users/z3dev/dev/jscad/OpenJSCAD.org.feat-multicolor-lines/packages/utils/regl-renderer/src/index.js:7:15)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)

it was a silly copy pasta typo depth = { enable: !transparent } -> depth: { enable: !transparent }

/Users/z3dev/dev/jscad/OpenJSCAD.org.feat-multicolor-lines/packages/utils/regl-renderer/src/rendering/commands/drawMesh/index.js:65
func: { src: 'src alpha', dst: 'one minus src alpha' }
^^^^
SyntaxError: Unexpected identifier
AND
/Users/z3dev/dev/jscad/OpenJSCAD.org.feat-multicolor-lines/packages/utils/regl-renderer/src/rendering/commands/drawLines/index.js:41
func: { src: 'src alpha', dst: 'one minus src alpha' }
^^^^
SyntaxError: Unexpected identifier
Please try executing the demos to ensure that the rendering works as expected.

sorry bout that. I forgot to run standardx (it would have caught such syntax errs)...
now it is fixed, and I was able to run demo-cli

I have updated the code to add support for colors on geom2 and updated demos to use colors

I made the square and the star bigger so they are not obscured by the grid
few things are tested

usig only color and simpler shader when no color per side
backfilling default color if not all sides have color defined
adding alpha channel if not defined in a color for a side

@z3dev take another look, and let me know what you think about the direction
@platypii finally the colored lines can be actually used.

@z3dev have you had issue with this ?
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `prebuild-install || node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

tests are ok for me locally, but fail in travis

@z3dev have you had issue with this ?
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `prebuild-install || node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

tests are ok for me locally, but fail in travis

Can you check the requirements for gl 5.0.0? Maybe we are trying to test using an old version of nodejs / NPM

I see. GL was added to the dependencies.
there’s some issue with that for Travis. See the readme file. You may have to remove it.

I don’t think the test suite needs GL, so please remove from all dependencies.
however, you’ll have to test the web UI, which uses the rendering.

I removed gl as suggested.
gl is used by demo-cli , those using demo-cli would need to install it maybe globally,
// create webgl context
const gl = require('gl')(width, height)

I have actually made another change to line colors.
By nature of the how code is written for shaders lines have start and end color for each segment in colorPerVertex mode.
so I added the support for optional endColor
This is trivial to support on the rendering side, and we can see if it will have use in the modeling later,.


Style settings modal according to graphic design

What's Changed

BC: Increase hierarchy of link button in consent modal to secondary button
BC: Drop default dark theme
Style settings modal
Add two dozens of new CSS custom properties
Support all UI options of the vanilla library
Update Spirit Design Tokens
Organise styles into multiple directories and SCSS files to improve maintainability
Organise CSS custom properties table
Watch and build SCSS during development moved to #199

Upgrade Guide

Secondary button (Accept necessary) in the consent bar is now rendered as button (previously rendered as link). Should you have made any design changes through the theming API, you may need to provide additional theming for secondary buttons using new --lmcccm-btn-secondary-* custom properties.
While the new Settings Modal inherits many design choices from the existing theme, you may still need to adjust it to match your design through some of the two dozens of new CSS custom properties. Please refer to README to get the full list of theming options.
Default dark theme has been dropped. Please refer to README to learn how to configure a theme or even support the real dark mode.

TODO

[x] Add upgrade howto (description of the BC breaks and how to upgrade) not a BC so far! 🎉
[x] Once squashed, refer to issue #CCM-56 instead of #CCM-15
[x] Figure out what to do with dark theme now when we have secondary buttons that are hardly visible on dark mat:

Or even invisible on light background when styled for dark background:

Consent Bar

Settings Modal

Don't forget on this please :)

[ ] Add upgrade howto (description of the BC breaks and how to upgrade)

BTW, as a result of the refactoring, all those files are now built to dist/scss/, so they should all be part of dist folder, is this expected? (Compare with current state.)
dist/scss/
├── index.scss
├── settings
│   └── _settings.scss
├── style
│   ├── _buttons.scss
│   ├── _consent-modal.scss
│   ├── _links.scss
│   ├── _main.scss
│   ├── _settings-modal.scss
│   ├── _toggles.scss
│   └── _typography.scss
├── theme
│   ├── _default.scss
│   └── _vanilla.scss
└── tools
    └── _units.scss

BTW, as a result of the refactoring, all those files are now built to dist/scss/, so they should all be part of dist folder, is this expected? (Compare with current state.)

Yes, it's correct. Sass source is intended for use with Spirit so it can be compiled with custom colors etc.

Nice 👍. Could you have a look at those two issues please?

Fixed and ⚠ REBASED ⚠.


fix: ensure 5.3 cannot be installed for PHP 8.0

See https://github.com/guzzle/guzzle/pull/2833
guzzle 5.3 is not compatible with PHP 8.0. Update composer.json to ensure this is taken into account by composer.

This PR won't work, because composer can just resolve the previous version of the 5.3.x release which says it works on PHP 8.

Moreover, Guzzle 5 is EOL anyway.

It will work, as libraries can use ^5.3.5, and --prefer-lowest would then not install 5.3.5 on PHP 8.0 as it's been properly marked as incompatible.
I understand Guzzle 5 is EOL, but I imagine it still receives lots of downloads, and this change would be useful for some people hoping to ensure the proper version is installed and it'd be nice to fix the bug.
But no hard feelings if you'd prefer not to make this change.  I added the following workaround to my github actions:
- if: ${{ matrix.php == '8.0' || matrix.composer-flags == '--prefer-lowest' }}
   name: Update guzzlehttp/ringphp dependency

I'd prefer not to need this, however. Thanks for your time!

I don't think many libraries will do that. Indeed, nobody should still be using Guzzle 5 anyway. It will not be getting any security fixes.

And even if a library does that, older versions of the library exist... This is why upper bounds on dependencies are so important, since they cannot be retroactively fixed.

While, Guzzle 7 has taken care to use upper bounds, other major projects are actually deleting them, so this problem with prefer-lowest is going to become a real issue for people using phpunit and symfony in the future!

That's unfortunate, I don't know why anyone would see that as a good idea. Define the versions your library works with accurately!
Using --prefer-lowest is a useful way to test multiple major versions of dependencies, but there are other ways as well. If y'all are refusing security fixes for this branch then it makes sense to not merge. However, I might suggest removing the 5.3 branch, as to me that implies it's still accepting PRs.

Thank you for this PR.
I understand your situation but I don't think we should make more effort into 5.x. It only gets a few thousands downloads per month since 2017.
I dont think we can remove the 5.3 branch because we have tags to it. I think it will make those tags uninstallable... I might be wrong but Im not interested to take a chance =)

Thank you for this PR.
I understand your situation but I don't think we should make more effort into 5.x. It only gets a few thousands downloads per month since 2017.
I dont think we can remove the 5.3 branch because we have tags to it. I think it will make those tags uninstallable... I might be wrong but Im not interested to take a chance =)


April Tags Runs Constantly, Using Much CPU

This is may not technically be an issue, but it is negatively impacting my project and I'm fixing it.  In case anyone else has a similar problem I want to share my solution.
I'm working on a robotics application with high CPU needs.  I occasionally need to scan for fiducials, but having apriltags scan constantly for them consumes too much cpu and negatively impacts other aspects of robot functionality.
To address this I am adding a flag which will allow you to disable and enable apriltags by publishing a bool. I should have it running shortly and will make a pull request once done. It should not add any computational demand, nor should it have any impact on users not using the flag.

Thanks @davelkan.  The Apriltags node will actually stop running detections if there are no subscribers, so another option is to simply unsubscribe from the topic when you are not scanning for fiducials.  It is a little bit slower, if that matters for your use case.
https://github.com/personalrobotics/apriltags/blob/master/src/apriltags.cpp#L244

That's a good point.  I was actually considering looking into setting up apriltags with shared pointers to eliminate the need for subscription, but that's a little ways off.

I do have the changes I made working locally, though i have decided not to use them as your suggestion is better. If you would like me to push upstream anyways let me know.

Glad to hear it's working for you!


External/Remote Resources

Hello,
I'm musing at refactoring some of my servers to smaller services -- one of which is an email server. I would like to take the email components out of my other servers and combine them.
On many of my servers, the User resources has an emails: [Array('email'), 'user'] property. If I extract the Email resources into its own server, how do I maintain this relationship?
I figure that the User will know about the Emails, but the Emails don't need to know about the User
Thank you for any help you might provide 😄

This is a hard problem because you're asking to maintain a bidirectional relationship on different servers and assuming you have the permission to do it, and that the remote server is going to sync data back.
I would suggest just having a plain string to identify the resource on the remote server, probably a URL. That's how the web at large works.

Thank you


Add Microsoft.Extensions.Http as a prerequisite

Its in the Microsoft.Extensions.DependencyInjection namespace.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5786402e-9d88-5de0-607a-1c2a143af006
Version Independent ID: 15368d1c-b22e-7eb2-413f-cc0d41d2b909
Content: Use dependency injection in .NET Azure Functions
Content Source: articles/azure-functions/functions-dotnet-dependency-injection.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@zpittmansf Thank you for your feedback! We will review and update as appropriate.

@zpittmansf Thanks again for sharing your feedback. The use of Microsoft.Extensions.Http package appears to be taken as an example and doesn't seem to be a required detail of an overall Prerequisite for the setup of a DI pattern for Functions which I think this documentation is primarily about. Moreover, the documentation does appear to state the below:
This example uses the Microsoft.Extensions.Http package required to register an HttpClient at startup.
Based on the above, we will hold off from making this update and proceed to close this thread for now. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Update 1.18 wiki tutorial to reflect change to biome source bclib:end_biome_source

Tutorial references betterend:better_end_biome_source, should reference bclib:end_biome_source

Apologies, duplicate of #386


Private key passphrase to connect to compute instance

Hi team,
I am able to successfully use the terraform code to provision redis, after creation I am trying to connect to the instance using the generated private key - but getting an error message as shown. Have provided 400 permissions for the file, can u guide me on what I may be missing?

@sylvinameliajohn I am assuming you have used Deploy Button and OCI Resource Manager has been used to automate the deployment. In that case, generated private key is wrongly formatted. Please use https://www.samltool.com/format_privatekey.php to format the key properly.

Amazing that worked! Thank you for the super quick response..


delete unnecessary parameter

- What I did
Delete unnecessary parameter of getDaemonConfDir in cmd/dockerd/daemon.go.
- How I did it
I just delete the unnecessary parameter of getDaemonConfDir, and modify the function call.
- How to verify it
The docker engine works well.
- Description for the changelog
Delete unnecessary parameter.
- A picture of a cute animal (not mandatory but encouraged)
Signed-off-by: zhukj<EMAIL_ADDRESS>

This won't work because other platforms actually use this parameter; see for example windows https://github.com/docker/docker/blob/master/cmd/dockerd/daemon_windows.go
I'll close this PR, but thanks fir contributing!


Handle newtypes

We don't represent newtypes in env.ml or in the typechecker yet. There are more pressing things I want to fix in the typechecker but I wanted to open an issue about type before I forget...

Newtypes are here in the spec: https://p4.org/p4-spec/docs/P4-16-v1.1.0-spec.html#sec-newtype

Fixed in f61eacbea7b0aa8537b8109cab08fee07d64cf76


Update latest language codes

Inspired by https://inindca.atlassian.net/browse/DEVTOOLING-632 and in addition to https://github.com/MyPureCloud/terraform-provider-genesyscloud/pull/1127 , this pulls in the latest language codes the API schema supports for Knowledge / Knowledge Base / Categories / Docs.
See values available in dropdown for language field in API Explorer:

https://apicentral.genesys.cloud/api-explorer\#post-api-v2-knowledge-knowledgebases
https://apicentral.genesys.cloud/api-explorer\#post-api-v2-knowledge-knowledgebases--knowledgeBaseId--languages--languageCode--categories

Nice idea @dginty4 . I did just that and pulled the logic into a shared Validator function.


Downgrade CI support for MIPS

MIPS targets are now tier 3, this downgrades CI support for MIPS just like other tier 3 targets.

@bors r+

:pushpin: Commit f5ba3fdf6c041ab7bfb9f8cdc4b6b0d8970bba57 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit f5ba3fdf6c041ab7bfb9f8cdc4b6b0d8970bba57 with merge 4e2b133be7879f79f22e403abd0aa768f0cb01a9...

:broken_heart: Test failed - checks-cirrus-freebsd-14

@bors r+ p=10

:pushpin: Commit ef3a782288d017e89fb7a014d086280ae25fe76f has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit ef3a782288d017e89fb7a014d086280ae25fe76f with merge 614d5e244187f8461d17d402e3294dc7fd53c0d3...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 614d5e244187f8461d17d402e3294dc7fd53c0d3 to main...


[Wiki] Code sample dead Link

Have a look to your code sample in the wiki:
https://github.com/clun/ff4j/wiki/Getting-Started#codes-samples
The project "ff4j-samples" doesn't exist any more. Dead link :o/
| Link  | Description |
| ------------- | ------------- |
| X  | [5 min tutorial](https://github.com/clun/ff4j-extra/tree/master/ff4j-samples/hello-ff4j)    |
| X  | [Spring Boot](https://github.com/clun/ff4j-extra/tree/master/ff4j-samples/ff4j-sample-springboot) |
| X  | [Web API](https://github.com/clun/ff4j-extra/tree/master/ff4j-samples/ff4j-sample-web-api)|
| X  | [Web](https://github.com/clun/ff4j-extra/tree/master/ff4j-samples/ff4j-sample-web) |

I found the new repo. I correct the wiki.

Thank you, if you need informations do not hesitate to ask.


JSX quotes config

I don't care anymore. I just want something standardised.

:sparkles:

:sparkles:

:shipit:

:sunflower: :sunflower:


Improve naming

Hamiltonian_flow may be too long of a name. TensorFlow has two syllables and six letters in the "tensor" part, whereas "hamiltonian" has four and eleven, respectively.
We probably need to make it shorter to make it easier to advertise...
Candidates from my side:

HamiltonFlow
HamilFlow
HamiFlow
cc @ErbB4 @emptymalei

Instead of xxflow, we can use acronyms.
https://acronymify.com/search?q=physical+and+dynamical+systems+dataset
Some candidates:

PLAID
PHYSICIST
IDEA
INDICATE
PALLET

I still believe Hamiltonian is a good idea, and would like to keep a part of the word Hamiltonian in the name.

No strong opinion here. HamiltonFlow sounds good. The important thing is to publish a first version ASAP to take the name on pip.

If you are fine with the name HamiltonFlow, we can refactor the code to this name.

we use HamilFlow


CI build should test example projects with camel-quarkus SNAPSHOT

Relates to https://github.com/apache/camel-quarkus/issues/3157#issuecomment-935613615.
There are (maybe) 2 parts to this.

We should be able to build & test the examples in the CQ CI build with the CQ SNAPSHOT version & whatever Quarkus version it is currently aligned to.

The camel-quarkus-examples CI build for the camel-quarkus-main branch needs modifying. From what I can see, it checks out and builds CQ main, but never does anything with it. The core component versions remain aligned to the release versions.

Maybe it's just a simple case of using CQ SNAPSHOT version on the examples camel-quarkus-main branch. I think we did that in the past but we seem to have abandonded doing that at some point.

@ppalaga can you please explain what was done in your many PRs and what's missing? Is this issue still valid ?

I think this was sufficiently addressed in https://github.com/apache/camel-quarkus/pull/3165
Testing examples still fail on the CI with in 2.7.x branch but that's an independent problem for sure.

Thanks for doing the cleanup, @zbendhiba !


Protobuf aggregate definitions

PTAL.

Please adjust file layout to match style in spine/base.

Other than that, LGTM.

Should I remove empty lines after message headers?

And use 2-spaces indent?

Please remove empty lines after message headers.
Please keep 4 spaces, which is standard to all our code. (And most of the code around too).


nixos/db-rest: init

Description of changes

added nixos module for db-rest
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3361

Anything else missing to get this merged?

Hi, is there anything that is still blocking this PR?

@ofborg test db-rest


Windows not auto tiling anymore (maybe related to Sanoma 14.6.1 upgrade?)

I'm seeing very weird behavior this morning after my computer updated to macOS 14.6.1 last night. Most of my windows are no longer being positioned correctly.
Let's start with what still seems to be working.

I can assign windows to workspaces just fine
I can move to workspaces just fine
It appears that apps are appropriately being auto-assigned to the right workspaces when they startup
Aerospace seems to be starting up when my computer starts
Any chrome window I have seems to behave perfectly

But weird things are happening within my workspaces and apps that aren't chrome.
I have a workspace for my Messages and What's App client, and I can't get them to automatically tile in any way (vertically, horizontally, or with the accordion). I also can't switch between them at all. I can still move them to another screen, but besides that, it appears that they are not under the control of aerospace at all.
I have a different workspace with a chrome window and an iterm2 terminal. In that workspace, I can switch between tiles appropriately, but the tiles don't align themselves as they should (they don't move at all unless I manually adjust where they sit on the screen). Not sure whether this partial working behavior is due to the fact that there is a chrome window involved or not, but it's definitely odd.
I have yet another workspace where I keep my Slack app, and for whatever reason, it keeps opening only halfway vertically. I can double click the title bar to open it full screen, but that doesn't feel like it should be needed.
Everything was working fine yesterday. Since yesterday, the things that I know of that are different are

My computer upgraded to macOS 14.6.1
I unplugged my external monitor, and then replugged it back in (the behavior above seems to be consistent whether I'm using an external monitor or just my laptop screen).

I love this tool, and I'm super thankful for you working on it 🙇

Here's my config
# Place a copy of this config to ~/.aerospace.toml
# After that, you can edit ~/.aerospace.toml to your liking

# It's not necessary to copy all keys to your config.
# If the key is missing in your config, "default-config.toml" will serve as a fallback

# You can use it to add commands that run after login to macOS user session.
# 'start-at-login' needs to be 'true' for 'after-login-command' to work
# Available commands: https://nikitabobko.github.io/AeroSpace/commands
after-login-command = []

# You can use it to add commands that run after AeroSpace startup.
# 'after-startup-command' is run after 'after-login-command'
# Available commands : https://nikitabobko.github.io/AeroSpace/commands
after-startup-command = []

# Start AeroSpace at login
start-at-login = true

# Normalizations. See: https://nikitabobko.github.io/AeroSpace/guide#normalization
enable-normalization-flatten-containers = true
enable-normalization-opposite-orientation-for-nested-containers = true

# See: https://nikitabobko.github.io/AeroSpace/guide#layouts
# The 'accordion-padding' specifies the size of accordion padding
# You can set 0 to disable the padding feature
accordion-padding = 30

# Possible values: tiles|accordion
default-root-container-layout = 'tiles'

# Possible values: horizontal|vertical|auto
# 'auto' means: wide monitor (anything wider than high) gets horizontal orientation,
#               tall monitor (anything higher than wide) gets vertical orientation
default-root-container-orientation = 'auto'

# Possible values: (qwerty|dvorak)
# See https://nikitabobko.github.io/AeroSpace/guide#key-mapping
key-mapping.preset = 'qwerty'

# Mouse follows focus when focused monitor changes
# Drop it from your config, if you don't like this behavior
# See https://nikitabobko.github.io/AeroSpace/guide#on-focus-changed-callbacks
# See https://nikitabobko.github.io/AeroSpace/commands#move-mouse
on-focused-monitor-changed = ['move-mouse monitor-lazy-center']
on-focus-changed = ['move-mouse window-lazy-center'] # Mouse lazily follows any focus (window or workspace)
# Gaps between windows (inner-*) and between monitor edges (outer-*).
# Possible values:
# - Constant:     gaps.outer.top = 8
# - Per monitor:  gaps.outer.top = [{ monitor.main = 16 }, { monitor."some-pattern" = 32 }, 24]
#                 In this example, 24 is a default value when there is no match.
#                 Monitor pattern is the same as for 'workspace-to-monitor-force-assignment'.
#                 See: https://nikitabobko.github.io/AeroSpace/guide#assign-workspaces-to-monitors
[gaps]
inner.horizontal = 0
inner.vertical =   0
outer.left =       0
outer.bottom =     0
outer.top =        0
outer.right =      0

# 'main' binding mode declaration
# See: https://nikitabobko.github.io/AeroSpace/guide#binding-modes
# 'main' binding mode must be always presented
[mode.main.binding]

# All possible keys:
# - Letters.        a, b, c, ..., z
# - Numbers.        0, 1, 2, ..., 9
# - Keypad numbers. keypad0, keypad1, keypad2, ..., keypad9
# - F-keys.         f1, f2, ..., f20
# - Special keys.   minus, equal, period, comma, slash, backslash, quote, semicolon, backtick,
#                   leftSquareBracket, rightSquareBracket, space, enter, esc, backspace, tab
# - Keypad special. keypadClear, keypadDecimalMark, keypadDivide, keypadEnter, keypadEqual,
#                   keypadMinus, keypadMultiply, keypadPlus
# - Arrows.         left, down, up, right

# All possible modifiers: cmd, alt, ctrl, shift

# All possible commands: https://nikitabobko.github.io/AeroSpace/commands

# See: https://nikitabobko.github.io/AeroSpace/commands#exec-and-forget
# You can uncomment the following lines to open up terminal with alt + enter shortcut (like in i3)
# alt-enter = '''exec-and-forget osascript -e '
# tell application "Terminal"
#     do script
#     activate
# end tell'
# '''

# See: https://nikitabobko.github.io/AeroSpace/commands#layout
alt-slash = 'layout tiles horizontal vertical'
alt-comma = 'layout accordion horizontal vertical'

# See: https://nikitabobko.github.io/AeroSpace/commands#focus
alt-h = 'focus left'
alt-j = 'focus down'
alt-k = 'focus up'
alt-l = 'focus right'

# See: https://nikitabobko.github.io/AeroSpace/commands#move
alt-shift-h = 'move left'
alt-shift-j = 'move down'
alt-shift-k = 'move up'
alt-shift-l = 'move right'

# See: https://nikitabobko.github.io/AeroSpace/commands#resize
alt-shift-minus = 'resize smart -50'
alt-shift-equal = 'resize smart +50'

# See: https://nikitabobko.github.io/AeroSpace/commands#workspace
alt-1 = 'workspace 1'
alt-2 = 'workspace 2'
alt-3 = 'workspace 3'
alt-4 = 'workspace 4'
alt-5 = 'workspace 5'
alt-6 = 'workspace 6'
alt-7 = 'workspace 7'
alt-8 = 'workspace 8'
alt-9 = 'workspace 9'
alt-a = 'workspace A' # Althea chrome profile
alt-b = 'workspace B'
alt-c = 'workspace C' # Code / Terminal
alt-d = 'workspace D' # Discord
alt-e = 'workspace E'
alt-g = 'workspace G'
alt-i = 'workspace I'
alt-m = 'workspace M' # Music / spotify
alt-n = 'workspace N' # Notes
alt-o = 'workspace O' # Other
alt-p = 'workspace P' # Personal chrome profile
alt-q = 'workspace Q'
alt-r = 'workspace R' # Reminders
alt-s = 'workspace S' # Slack
alt-t = 'workspace T'
alt-u = 'workspace U'
alt-v = 'workspace V'
alt-w = 'workspace W'
alt-x = 'workspace X' # Text apps (texts and whatsapp)
alt-y = 'workspace Y'
alt-z = 'workspace Z'

# See: https://nikitabobko.github.io/AeroSpace/commands#move-node-to-workspace
alt-shift-1 = 'move-node-to-workspace 1'
alt-shift-2 = 'move-node-to-workspace 2'
alt-shift-3 = 'move-node-to-workspace 3'
alt-shift-4 = 'move-node-to-workspace 4'
alt-shift-5 = 'move-node-to-workspace 5'
alt-shift-6 = 'move-node-to-workspace 6'
alt-shift-7 = 'move-node-to-workspace 7'
alt-shift-8 = 'move-node-to-workspace 8'
alt-shift-9 = 'move-node-to-workspace 9'
alt-shift-a = 'move-node-to-workspace A'
alt-shift-b = 'move-node-to-workspace B'
alt-shift-c = 'move-node-to-workspace C'
alt-shift-d = 'move-node-to-workspace D'
alt-shift-e = 'move-node-to-workspace E'
alt-shift-g = 'move-node-to-workspace G'
alt-shift-i = 'move-node-to-workspace I'
alt-shift-m = 'move-node-to-workspace M'
alt-shift-n = 'move-node-to-workspace N'
alt-shift-o = 'move-node-to-workspace O'
alt-shift-p = 'move-node-to-workspace P'
alt-shift-q = 'move-node-to-workspace Q'
alt-shift-r = 'move-node-to-workspace R'
alt-shift-s = 'move-node-to-workspace S'
alt-shift-t = 'move-node-to-workspace T'
alt-shift-u = 'move-node-to-workspace U'
alt-shift-v = 'move-node-to-workspace V'
alt-shift-w = 'move-node-to-workspace W'
alt-shift-x = 'move-node-to-workspace X'
alt-shift-y = 'move-node-to-workspace Y'
alt-shift-z = 'move-node-to-workspace Z'

alt-shift-f = 'fullscreen'

# See: https://nikitabobko.github.io/AeroSpace/commands#workspace-back-and-forth
alt-tab = 'workspace-back-and-forth'
# See: https://nikitabobko.github.io/AeroSpace/commands#move-workspace-to-monitor
alt-shift-tab = 'move-workspace-to-monitor --wrap-around next'

# See: https://nikitabobko.github.io/AeroSpace/commands#mode
alt-shift-semicolon = 'mode service'

# 'service' binding mode declaration.
 My search queries were (please don't forget to search in closed issues too):

Windows not tiling
Sanoma
14.2.1
(I wasn't totally sure what all to search here because what it feels like is that some of the basics have just stopped working, which is super odd)

[x] I've read the documentation https://nikitabobko.github.io/AeroSpace/guide & https://nikitabobko.github.io/AeroSpace/commands

[ ] I've searched in the documentation. My search queries were:

<!-- Replace with `aerospace --version` output -->
aerospace CLI client version: 0.14.2-Beta 0cb8dbdfc5ee73b8cbc200f175f467ebead55201
AeroSpace.app server version: 0.14.2-Beta 0cb8dbdfc5ee73b8cbc200f175f467ebead55201

Same here. Just installed, played around with config, but quickly realised that windows no longer change focus using keybinds, can't be moved to other workspaces and don't switch places. I  reset to default config, and installed aerospace after the new macos update (Sonoma 14.6.1)

I just fixed an issue where windows from one app would tile nicely, but windows from other apps wouldn't join the tiling. (Only windows from one app could be seen at once in a tiling layout.)
Fixed it by disabling Stage Manager in System Preferences → Desktop & Dock. Maybe this is your issue?
Couldn't find any documentation on this issue anywhere, unfortunately.

I appreciate the suggestion! I already have Stage Manager disabled though :( I've tried uninstalling and re-installing aerospace but still seem to have the same issue where certain apps just don't seem to be under the control of aerospace.

Aha! I found the fix. The problem for me seemed to stem from the way I had my on-window-detected setup, specifically the run command.
I was doing this previously, which I'm sure I copied from some youtube video or something or other. That did appear to work at one point fwiw.
[[on-window-detected]]
if.app-id = 'com.tinyspeck.slackmacgap'
run = ['layout floating', 'move-node-to-workspace S']

But I don't really want these windows to be tiling, so I changed it to just this
[[on-window-detected]]
if.app-id = 'com.tinyspeck.slackmacgap'
run = 'move-node-to-workspace S' # See the change here!

And now everything seems to be working great again! I'll close this as resolved 🙇


i have some pictures in the excel, but  when I use the GetPictures() method to get them,some of resulet is  null,why?

Description
Steps to reproduce the issue:
i have some pictures in the excel, but  when I use the GetPictures() method to get them,some of resulet is  null
Describe the results you received:
some of the pics is null
Describe the results you expected:
all of them should not be null
Output of go version:
1.20.6

Excelize version or commit ID:
v2.7.1

Environment details (OS, Microsoft Excel™ version, physical, etc.):
WINDOWS

Thanks for your feedback. Could you show us a complete, standalone example program or reproducible demo?

It seems still not have enough info of code and attachments to reproduce this issue, so I'll close this issue, if you have any questions, please let me know to reopen this anytime.

I also have a similar problem, I don't know where is the problem：
I don't know if it's a problem with the excel format or a problem with the code
go version：
1.17.13
Excelize version ID：
v2.7.1
demo:
	reader, err := excelize.OpenReader(file)
	rows, err := reader.GetRows("sheet1")
	picCellCos := []string{"F", "G", "H"}

	for index, row := range rows {
		if index == 0 {
			continue
		}

		// the text in the cell
		id := row[0]

		for _, cos := range picCellCos {
			// but some pics is nil
			pics, picerr := reader.GetPictures("sheet1", cos+conv.ToString(index+1))
			if picerr == nil {
				for _, pic := range pics {
					picByte := bytes.NewReader(pic.File)

					// do something

				}
			}
		}
	}

Please tell me where I did wrong, thank you ～～
Here is my test data
testupload.xlsx

Hi @ihater, thanks for your feedback. I have fixed this issue, please try to upgrade the master branch code, and this patch will be released in the next version.


The nil in Lua does not calculate the length

let x:number[] = [null,null,null,null];
print(x.length) //4

local x = {nil,nil,nil,nil}
print(#x)  -- 0

The nil in Lua does not calculate the length

I don't think this is even something we can fix, this is just how lua # works.

Maybe we could define our own null/undefined somewhere, but that might cause other issues or doesnt really work

The question is if something like that is worth the effort. In my opinion this is a case of KISS.

null in Lua is much closer to undefined than null.

Will close this for now, maybe we can revisit later.


User difficulty to place outfitted tracks

Currently, there is no way that an API user can place an outfitted track or get an item for an outfitted track. This disturbs my attempt to add compatibility to Steve's carts.

Fixed by dabb1e138a63ba54b839085d0ed9f285741a0fa6. Thanks CovertJaguar!


HuberLoss() TypeError

Description
Using the HuberLoss()  (with or without parameters) from the module loss raise a TypeError: exception with the message using it in a simple regression computation where for example L2Loss or L1Loss raise no exception or problem
Error Message

TypeError: Operator abs registered in backend is known as abs in Python. This is a legacy operator which can only accept legacy ndarrays, while received an MXNet numpy ndarray. Please call as_nd_ndarray() upon the numpy ndarray to convert it to a legacy ndarray, and then feed the converted array to this operator.

Stack Trace:
Traceback (most recent call last)
<ipython-input-13-46119558e5f5> in <module>
      3     for X, y in data_iter:
      4         with autograd.record():
----> 5             l = loss(net(X), y)
      6         l.backward()
      7         trainer.step(batch_size)

~/.conda/envs/d2l/lib/python3.7/site-packages/mxnet/gluon/block.py in __call__(self, *args)
    691             hook(self, args)
    692
--> 693         out = self.forward(*args)
    694
    695         for hook in self._forward_hooks.values():

~/.conda/envs/d2l/lib/python3.7/site-packages/mxnet/gluon/block.py in forward(self, x, *args)
   1156                     params = {k: v.data(ctx) for k, v in self._reg_params.items()}
   1157
-> 1158                 return self.hybrid_forward(ndarray, x, *args, **params)
   1159
   1160         params = {i: j.var() for i, j in self._reg_params.items()}

~/.conda/envs/d2l/lib/python3.7/site-packages/mxnet/gluon/loss.py in hybrid_forward(self, F, pred, label, sample_weight)
    605     def hybrid_forward(self, F, pred, label, sample_weight=None):
    606         label = _reshape_like(F, label, pred)
--> 607         loss = F.abs(label - pred)
    608         loss = F.where(loss > self._rho, loss - 0.5 * self._rho,
    609                        (0.5 / self._rho) * F.square(loss))

~/.conda/envs/d2l/lib/python3.7/site-packages/mxnet/ndarray/register.py in abs(data, out, name, **kwargs)

~/.conda/envs/d2l/lib/python3.7/site-packages/mxnet/ndarray/register.py in _verify_all_legacy_ndarrays(op_name, func_name, args, out)
     97                             'convert it to a legacy ndarray, and then feed the converted '
     98                             'array to this operator.'
---> 99                             .format(op_name, func_name))
    100     if out is None:
    101         return

TypeError: Operator `abs` registered in backend is known as `abs` in Python. This is a legacy operator which can only accept legacy ndarrays, while received an MXNet numpy ndarray. Please call `as_nd_ndarray()` upon the numpy ndarray to convert it to a legacy ndarray, and then feed the converted array to this operator.

To Reproduce
In d2l-course chapter 3.3 is a question of substituting the L2Loss by the HuberLoss. Doing this cause the problem
but it also fails with a simple code like this:
from mxnet.gluon import loss as gloss
loss = gloss.HuberLoss()
input_scalar = np.array([5])
output_scalar = np.array([6])
loss(input_scalar,output_scalar)

Steps to reproduce
from mxnet.gluon import loss as gloss
loss = gloss.HuberLoss()  # The squared loss is also known as the L2 norm loss
input_scalar = np.array([5])
output_scalar = np.array([6])
loss(input_scalar,output_scalar)

What have you tried to solve it?

I have tried to reinstall last version of mxnet again and the problem keeps.
I have tried other losses to see if there was a more general problem, with L1Loss and L2Loss the problem disappears.

Environment
We recommend using our script for collecting the diagnositc information. Run the following command and paste the outputs below:
curl --retry 10 -s https://raw.githubusercontent.com/dmlc/gluon-nlp/master/tools/diagnose.py | python

# ----------Python Info----------
Version      : 3.7.5
Compiler     : Clang 4.0.1 (tags/RELEASE_401/final)
Build        : ('default', 'Oct 25 2019 10:52:18')
Arch         : ('64bit', '')
------------Pip Info-----------
Version      : 19.3.1
Directory    : /Users/gonzalopolo/.conda/envs/d2l/lib/python3.7/site-packages/pip
----------MXNet Info-----------
Version      : 1.6.0
Directory    : /Users/gonzalopolo/.conda/envs/d2l/lib/python3.7/site-packages/mxnet
Num GPUs     : 0
Commit Hash   : 4da14a22385622c35e9a5c9c3e8a17c07f718cad
----------System Info----------
Platform     : Darwin-19.2.0-x86_64-i386-64bit
system       : Darwin
node         : Gonzalos-MacBook-Pro.local
release      : 19.2.0
version      : Darwin Kernel Version 19.2.0: Sat Nov  9 03:47:04 PST 2019; root:xnu-6153.61.1~20/RELEASE_X86_64
----------Hardware Info----------
machine      : x86_64
processor    : i386
b'machdep.cpu.brand_string: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz'
b'machdep.cpu.features: FPU VME DE PSE TSC MSR PAE MCE CX8 APIC SEP MTRR PGE MCA CMOV PAT PSE36 CLFSH DS ACPI MMX FXSR SSE SSE2 SS HTT TM PBE SSE3 PCLMULQDQ DTES64 MON DSCPL VMX EST TM2 SSSE3 FMA CX16 TPR PDCM SSE4.1 SSE4.2 x2APIC MOVBE POPCNT AES PCID XSAVE OSXSAVE SEGLIM64 TSCTMR AVX1.0 RDRAND F16C'
b'machdep.cpu.leaf7_features: RDWRFSGS TSC_THREAD_OFFSET SGX BMI1 AVX2 SMEP BMI2 ERMS INVPCID FPU_CSDS MPX RDSEED ADX SMAP CLFSOPT IPT SGXLC MDCLEAR TSXFA IBRS STIBP L1DF SSBD'
b'machdep.cpu.extfeatures: SYSCALL XD 1GBPAGE EM64T LAHF LZCNT PREFETCHW RDTSCP TSCI'
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0147 sec, LOAD: 1.0370 sec.
Timing for GluonNLP GitHub: https://github.com/dmlc/gluon-nlp, DNS: 0.0005 sec, LOAD: 0.9149 sec.
Timing for GluonNLP: http://gluon-nlp.mxnet.io, DNS: 0.0511 sec, LOAD: 0.5621 sec.
Timing for D2L: http://d2l.ai, DNS: 0.0405 sec, LOAD: 0.0580 sec.
Timing for D2L (zh-cn): http://zh.d2l.ai, DNS: 0.0492 sec, LOAD: 0.4651 sec.
Timing for FashionMNIST: https://repo.mxnet.io/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0528 sec, LOAD: 0.8639 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0340 sec, LOAD: 0.5830 sec.
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.0264 sec, LOAD: 0.0592 sec.

Unfortunately, only L1 and L2 loss are supported by DeepNumpy(MXNet numpy ndarray) currently. It's actually pretty easy to fix, but I am not sure if the patch will become part of MXNet 1.6.
Maybe @haojin2 could provide you more information.

Hi @xidulu ,
Thank you very much, it seems that the course d2l.ai is based on the previous interface and they have not change this part. So, not even  the 'SoftmaxCrossEntropyLoss or 'SigmoidBCELoss' would work?
I would add this ones first, I think they much more used than the Huber loss.

Hi @GPoloVera
It would be great if you could contribute DeepNumpy-compatible version of these Loss function to MXNet, but I just wanna inform you that it's vert likely you contributions will only be accessible through either build from source or the nightly build version.

Hi @xidulu,
I would love to do that but I would need some help because I am not familiarise enough with the code.
You said it is already done with the L1 and L2 losses but not with the rest isn't it?
Maybe trying to replicate the things done on this two and port to the rest is enough or there would be more challenging things?

@GPoloVera
Yep, you simply need to replicate the modification done to the L1 and L2 to make other loss functions compatible with DeepNumpy API.
You could refer to this PR: https://github.com/apache/incubator-mxnet/commit/65928b1936d5d43f237b0f9f9e43115861faf4c0#diff-8b9ba6d32a6b2369b6ffde76f0986d1f
p.s. You could just a leave a comment under this thread if you meet any problems during your development.


Patch 2

This now allows aligning between other elements including other buttons. This will fix any alignment issue from Left - Right if there are any others.

The first 2 will not fix an existing bug however, will overwrite it with a fix. The last will fix the check bug I believe.

Dont use this source

Can this branch be deleted?


Which docker account shell we use for the docker image?

vernemq https://hub.docker.com/u/vernemq
and
erlio https://hub.docker.com/u/erlio
Which one to use?
By the way erlio is used in  documentation.

vernemq/vernemq is the newer one.

Thanks


Sdk-follow-up

Adresses point 1 from #233
Currently blocked by the fact that the lakefs API does not provide a way to discard an opened file.

Re: our discussion yesterday, I opened https://github.com/treeverse/lakeFS/issues/7195 as a request for discard APIs on both reader and writer.
For now, the read/write attempt is blocked by some sharp edges in the lakefs file-like interfaces, so I think it's best to leave that point as last and focus on the remainder instead.

Superseded by #240, which contains two commits cherrypicked from here.


[UIFlow 2.0] [JP] Hat-PIR

Translate the "PIR" hat in UIFlow 2.0 under the Hat category into Japanese.

Translation is finished.


feat(webhooks) : Masking outgoing webhook header values in business profile

Currently we are not masking outgoing custom headers in business profile section.

Solution :
Backend will now send masked values to dashboard. So dashboard should allow user to view masked values but should not allow direct update functionality, instead they need to update values just like how connector api keys work today. There should be update which will update all key value pairs for custom outgoing webhooks.

Note: backend changes will deployed after dashboard changes.

sharing backend PR for the above changes, can find how profile response looks like in testcases
https://github.com/juspay/hyperswitch/pull/6798
cc : @JeevaRamu0104


⚠ Bofrost.at has degraded performance

In 9270669, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1765 ms

Resolved: Bofrost.at performance has improved in e7c345c.


Add banner image to PPT repo

May be visible via "social preview".
We need an image, tho...

https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/customizing-your-repositorys-social-media-preview
We'll need to track down the snake image

Does the snake have a name?  "sLINCCy", perhaps?


feat: change transaction/address api to /api/transactions

Description: change transaction/address api to api/transactions
Test Plan: make dev then open up http://localhost:3000/api/transactions/bitcoincash:qrmm7edwuj4jf7tnvygjyztyy0a0qxvl7q9ayphulp for instance, remember to add a valid bchd instance on your .env or .env.local file: GRPC_NODE_URL=bchd.fountainhead.cash
trying to open up http://localhost:3000/api/transactions/ should throw an error.

Can't get it working no matter what I try.

GRPC_NODE_URL=bchd.fountainhead.cash gives 14 DNS RESOLUTION FAILED

this should be a connection error, make sure it is GRPC_NODE_URL=bchd.fountainhead.cash
the node could have gone offline for a bit. (accidentally edited your comment, using github app on mobile)

Indeed, I don't know how I didn't see that, it works now

Not providing an address takes us to the admin page 404 which should be changed at some point. We can make a new task for that.

The API should respond with a 400 error to bad requests

API should never respond with 404


chore: go 1.19

Codecov Report
Base: 82.73% // Head: 82.73% // No change to project coverage :thumbsup:

Coverage data is based on head (10901ed) compared to base (014acdb).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #27   +/-   ##
=======================================
  Coverage   82.73%   82.73%
=======================================
  Files           7        7
  Lines         278      278
=======================================
  Hits          230      230
  Misses         32       32
  Partials       16       16

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


transfer projection to implied subplots when faceting

this catches transform kw passed to plot() when faceting, and pass the associated projection to the subplots. This does what was suggested in issue #1201.
(sorry for the irrelevant change to reshaping.rst, it seems I've stuck myself in a git hole and can't get out)

Thanks! I think we need to document this somewhere, and also some tests would be nice.
Do you feel like trying to add the same functionality to normal 2d plots also? (this will represent a little more work so if you don't have time no big deal).
Also, you should credit yourself in the "what's new" section of whats-new.rst

One concern I have is that it is conceivable that one might pass a transform keyword to plot() intended to interact with another library that hijacks matplotlib (i.e. not cartopy). In that case, matplotlib will get an unintended projection argument. That is a corner case though.

In that case, matplotlib will get an unintended projection argument. That is a corner case though.

Yes, I was about to ask @jhamman and @shoyer about their opinion about this. I'm actually not sure if this really is a corner case...

As it is, the xarray code becomes tied up with cartopy. If cartopy somehow changes its syntax (e.g. by renaming the transform kw) this code won't work.
Maybe there's another, more abstract way to do this?

As it is, the xarray code becomes tied up with cartopy. If cartopy somehow changes its syntax (e.g. by renaming the transform kw) this code won't work.

cartopy uses transform because this is an interface provided by matlotlib: http://matplotlib.org/users/transforms_tutorial.html . This is not likely to change anytime soon.

I know at least one usage where the projection keyword is needed for something else than cartopy (for polar plots, see example here by @hoonhout), but I don't know how often do people need  a transform though.
In the end, I'm still in favor of this change:

I suspect the large majority of the use cases are for cartopy
it's still possible to override the this default behavior by setting the projection kwarg yourself

@vnoel and @fmaussion - my apologies for being quite tardy in looking at this PR. I think this is a valuable change but I do have one comment on the implementation.


Add MRA token

Official announcement:  http://mtra.io/
Project team background:
The MRA team is composed of blockchain technical talents, financial risk control experts, and a senior operation team. It took two years to carefully build. The team has rich R&D experience in the Internet and blockchain fields.CEO Martin graduated from the University of Toronto with a PhD in Economics. CFA license holder, digital encryption currency enthusiast, one of the early builders of the European Bitcoin community, and angel investor in blockchain projects. He has participated in early investments in famous projects such as Ethereum, Ripple, and Polkadot. Martin has extensive experience and global vision in the fields of financial analysis, asset allocation, equity strategy, asset and bond investment, and digital currency investment.
Project basic information:
MRA is a fair decentralized financial system, a one-stop digital asset investment service platform. Provide users with lending, payment and multi-currency wallet exchange services.MRA is a digital token issued by the Meteora Foundation. The Meteora Foundation has reached a strategic cooperation with Renyou Stop to open up the ecology and let MRA truly land, and trade markets for derivatives such as trendy gaming circles, supercar clubs, artworks, and collectibles. Realize online and offline connections and realize value support.
Recommended Gas Limit (For ERC-20 project teams) for transaction: 60,000

欢迎在社区中推荐用户使用 imToken 2.0 ，imToken 支持比特币, 以太坊, Comos, EOS, 波场, Nervos, 比特币现金, 莱特币，波卡等资产管理；同时支持去中心化币币兑换功能 Tokenlon；转账页面支持 ETH, USDT, EOS, BTC OTC 功能；另外对于没有 ETH 矿工费的用户，也提供 ETH 手续费充值功能。


More Info :SWT(stroke width transform)

I think we should add a paper which is named "Detecting Text in Natural Scenes with
Stroke Width Transform",although,it is very early,lots of classical papers mention it.
address:https://www.microsoft.com/en-us/research/wp-content/uploads/2016/02/201020CVPR20TextDetection.pdf

Thank you for sharing :)
I'll add it on next update.
Regards,
Anthony


Add Slint Enums for determining Active Tab on Config Page

Slint 1.1 added support for enums within the Slint language. Update the config page to use an enum for each active tab instead of the integer.

Actually....I'm not sure this really provides me a lot of advantage. While I have a fixed set of 4 types of tabs for my slid over window, and I could convert that into an enum, right now I have the 4 tabs set as a model, of icon images, and use the index to determine the active page. If I change that to an enum, I can't iterate over that using a for loop, which means more code to accomplish the same thing. If I try to iterate over a model with a for loop but still use the enums, I need to add in code to map between the enums and the index anyway....so what advantage do I have to using the enum over what's already there?
I'll close this for now, but if I can reconsider later if I think it might make sense again.


Debug Tool: Show view filepath

Hello, @lonnieezell !
Today I remembered that Magento makes it possible to see the paths of the blocks in the frontend and considered this useful because it facilitated the development.
Then I implemented something in the CI, if it is in the development environment will have a button in the debugbar allowing to see the path of the views.
I want to open a PR for you to analyze, just tell me what you think I'll end later and send the PR.
Thanks!

Now that you bring this up, this is actually something that I remember wondering how tough it would be to do, but never looked into. I'd be very interested in seeing what you came up with for this.


OpenLayers 4.6.5 - Map Remove Tiles Transition

We were using the openlayers v4.2.0, and the tiles transition was not performed by default, but in the latest v4.6.5, in which the tiles transition is activated by default, which we can't control.
The problem is when there are small layers (like 1 or 2), v4.6.5 works fine, but when there are many layers (like 10), the transition takes a lot of time to be applied on the all the activated layers on the map, which is very slow for our application.
Is it possible to disable the transition of layers in v4.6.5 like we had in v4.2.0
Thanks in advance!

To remove the tile transition, simply configure the source with transition: 0.


Allow class level and method level query generics

This PR provides the ability for users to propagate the generics defined at the collection.get and collection.create levels into their queries automatically if their return_properties in the query is None. In this way, the following use-case is supported:
class That(TypedDict):
    this: str
    that: str

collection: _CollectionObject[That] = client.collection.get("That", That)
objs: _QueryReturn[That] = collection.query.fetch_objects()

in addition to specifying a query-specific generic like so:
class This(TypedDict):
    this: str

class That(TypedDict):
    this: str
    that: str

collection: _CollectionObject[That] = client.collection.get("That", That)
objs: _QueryReturn[This] = collection.query.fetch_objects(return_properties=This)

In the absence of any generics, the default is still Dict[str, Any] so that the base use-case is:
collection: _CollectionObject[Dict[str, Any]] = client.collection.get("That")
objs: _QueryReturn[Dict[str, Any]] = collection.query.fetch_objects()

Codecov Report
Attention: 134 lines in your changes are missing coverage. Please review.

Comparison is base (8c922b0) 92.61% compared to head (6667fdc) 92.45%.
Report is 63 commits behind head on pydantic_experiment.

Additional details and impacted files
@@                   Coverage Diff                   @@
##           pydantic_experiment     #569      +/-   ##
=======================================================
- Coverage                92.61%   92.45%   -0.17%
=======================================================
  Files                      115      116       +1
  Lines                    12829    13281     +452
=======================================================
+ Hits                     11882    12279     +397
- Misses                     947     1002      +55

Files
Coverage Δ

integration/test_authentication.py
90.00% <100.00%> (+0.65%)
:arrow_up:

integration/test_backup.py
91.89% <100.00%> (+0.05%)
:arrow_up:

integration/test_batch.py
98.47% <100.00%> (+0.03%)
:arrow_up:

integration/test_classification.py
100.00% <100.00%> (ø)

integration/test_cluster.py
100.00% <100.00%> (ø)

integration/test_collection.py
99.73% <100.00%> (+0.02%)
:arrow_up:

integration/test_collection_config.py
100.00% <100.00%> (ø)

integration/test_collection_filter.py
100.00% <100.00%> (ø)

integration/test_collection_multi_node.py
100.00% <100.00%> (ø)

integration/test_collection_references.py
99.37% <100.00%> (+0.07%)
:arrow_up:

... and 59 more

... and 1 file with indirect coverage changes

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


ci: improve renovate config

Issue
na
Description
ignore typescript minor and major
ignore rxjs major
Additional context

Gravitee.io Automatic Deployment
A prerelease version of this package has been published! 🚀
To use it in your project run the following lines:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here


Add a visual effect on hover

Description
On the events page, there are certain events mentioned. But the whole event box is not clickable and only the title of the box is such as API Days Patna in the screenshot. So, can we add a simple underline below the title. So, it will enhance the UX.

Screenshots
No response
Additional information
No response

I am writing to express my interest in contributing to your project on [LinkFree]. After reviewing the project requirements and objectives, I believe that I can add value to your team and help drive the project to success.
I have experience working with relevant technology or tools], and I am excited to apply my skills to this project. Additionally, I am highly motivated to collaborate with your team and contribute to the open-source community.

@Palakkgoyal can i work on this issue it will be an easy fix

Hey @RutamBhagat and @dheerajmw, I am also a learner like you and I have just created this issue. Cause I felt it is missing. But it need to get the request from the maintainers of this project that whether anyone can work on this or not. I created this issue because I was not able to find issue that is free(I mean on which I can work and no other person is working). Like this you can also go to LinkFree and see if you think if there is any change required. If there, then create a new issue and work on that. This thing seems pretty easy to me then to search for an issue that is available.
What do you think?

We have a component for links that would help with this 👍

Hey! I can work on this issue. Can you please assign me?

@Vishrut19 other people have requested the issue before you, also you have requested on another issue, please do not request more than 1 issue at a time as it creates more work/confusion for the maintainers

@RutamBhagat I have assigned this issue to you

@eddiejaoude just needed to confirm one thing. The underline should only appear on mouse hover right?

Like this?
And shouldn't be static like this

yes on mouse hover, the Link component should do this already for you

@eddiejaoude #5836 fixes this issue, please review and merge the pull request

Closing in favour of another solution https://github.com/EddieHubCommunity/LinkFree/issues/5870


Demonstration video not working

Describe the bug
See above. I am using an iPad.
Reproduction
https://hybridly.dev/guide/demonstration.html
Steps to reproduce
No response
System information
iOS 16.1.1

Used package manager
npm
Logs
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Hybridly issue and not an issue related to something else (Vite, Vue...). For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

Thanks, indeed Safari on iOS is the only browser not supporting webm. I'll see if I can add a fallback

I think I won't add a fallback video so it doesn't add a few MB on the repository just for iOS Safari support. Thanks for the report and here's the video in another format:
https://user-images.githubusercontent.com/16060559/201891747-1d8110ee-ea20-4ab8-a354-0c6642c5e102.mp4

Would be nice but had no idea which one to use. Wanted to use Vimeo but didn't manage to integrate them cleanly. Totally open to suggestions


debug: openocd: Fix building for SMP platforms

struct k_thread *current is only defined for !SMP platforms
leading to the following build-time failure:
| openocd.c:39:35: error: 'struct z_kernel' has no member named 'current'
|  [OPENOCD_OFFSET_K_CURR_THREAD] = offsetof(struct z_kernel, current),
^~~~~~~~
Disable calculation of meaningless offset then.
Fixes https://github.com/zephyrproject-rtos/zephyr/issues/18124.

@galak @ruuddw so maybe something like this until we really make sure OpenOCD works with Zephyr-aware on SMP systems?
diff --git a/subsys/debug/Kconfig b/subsys/debug/Kconfig
index a55d0f3350..a5dff0ad4f 100644
--- a/subsys/debug/Kconfig
+++ b/subsys/debug/Kconfig
@@ -223,6 +223,7 @@ config EXCEPTION_STACK_TRACE

 config OPENOCD_SUPPORT
        bool "OpenOCD support [EXPERIMENTAL]"
+       depends on !SMP
        select THREAD_MONITOR
        select THREAD_NAME
        help

@nashif @galak @ruuddw could you finalize reviewing this bug-fix of a a high priority bug?

Moving to next release since the review was not finalized for v.2.0.0

Any updates on this?

@EvgeniiDidin could you please take a look at this one since you anyways deal with OpenOCD/GDB thingy these days?

@abrodkin, @EvgeniiDidin  any update on this?

Hi,
I tested SMP Zephyr smp/pi example with these changes on HSDK(4 core) board using OpenOCD/ gdb. It is possible to load and start the application on all cores, also I was able to get callee registers from stack. But after stopping execution "continue" command failed with next message:
(gdb) c
Cannot execute this command without a live selected thread.

I guess it is related to the lack of  current_thread{id} values in OpenOCD, which come from "current" struct.
If  incomplete smp aware support is permissible for Zephyr/OpenOCD,  this commit is fine. Otherwise we should disable  simultaneous usage of CONFIG_SMP and CONFIG_OPENOCD options.

@abrodkin, @EvgeniiDidin any update on this?

@galak I think the most recent comment of Eugeniy (https://github.com/zephyrproject-rtos/zephyr/pull/18125#issuecomment-559171331) still valid. We either declare SMP mode doesn't support additional OpenOCD-related visibility options or ignore pointer to the "current thread" implemented in this commit. I now lean towards making CONFIG_OPENOCD incompatible with CONFIG_SMP as at least it won't introduce any funky behavior.

Let's prevent openocd and SMP / multiple CPUs in kconfig.
SMP/multi-core is simply not yet supported. See also PR #23503 : adds a #error in openocd.c.


Add Conv1D impementation

Resolves #59
I wasn't able to hide the Conv2DImpl class in any way that come to my mind, maybe you have some idea.
My first approach was to create delegate field in Conv1D but then there is problem with Layer functions that are not visible and imo should also be then overriden by with delegate calls
I create a few helper internal functions in ShapeFunctions they maybe useful in more places than the conv tests but I wasn't sure if this is the best place for them.
If implementation and helper function will be ok I can think of some extra tests (I'm open to suggestions)

@avan1235 Please merge the master branch, it helps me to run examples on your branch

@zaleslaw  I did the rebase and now it should have the newest source code from master

@avan1235 I recognized that the layer export/import is missing.
Could you please add it close to the Conv2d export/import?

@zaleslaw thanks for pointing this out, I've now added the proper functions in ModelLoader and ModelSaver

LGTM for me, first in the queue to merge at this moment, I'll run the TC build and merge if there are no new issues there.

Great, just rebased the branch to enable testing


product handle not coming through webhooks starting this morning?

We started getting alerts this morning about update and create webhooks not including a product handle (.handle is throwing undefined method exceptions).  Is anyone else experiencing the same?

See https://ecommerce.shopify.com/c/shopify-apis-and-technology/t/product-webhook-sending-variants-as-a-product-539036


Update the Android NDK to r25b LTS

Currently the Android build of TensorFlow and TensorFlow Lite use an older NDK, either 19 or 21. The latest LTS release of the Android NDK is r25b (the exact version is currently 25.1.8937393), and supports the latest features and compilers (LLVM 14).
See earlier NDK updates:

https://github.com/tensorflow/tensorflow/commit/63e7c149d7817edb63adc085c9650e1659f3b699 Update TFLite to use Android NDK r18b
https://github.com/tensorflow/tensorflow/commit/60bbb7c240580e79ad157e41c7c8058147e5085d Update the TFLite Android Dockerfile to use NDK r19c
https://github.com/tensorflow/tensorflow/commit/1a0b21e16b08ab0bb1c15f9b5ec040bfbdee2685 Update NDK version to r19c
https://github.com/tensorflow/tensorflow/commit/f4a65d74eb49eace09483df48b4ce2d94b6b354d [tf.lite] Update stale doc reference to recommended NDK version
https://github.com/tensorflow/tensorflow/pull/34419

@EwoutH !
Could you share your pointers on the concerned PR #59422 .
Thank you!

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The TFLite team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the TFLite space.
Thanks.


[Enhancement] Implement IconTintColorEffect for Image and ImageButton

Summary
To be able to better support theming, or even for whitelabeling your apps, I would like to have an IconTintColorEffect. With this effect you could tint an Image or ImageButton without needing to add your icon in multiple colors to your project.
API Changes
Added IconTintColor Effect which holds an attached property Xamarin.Forms.Color TintColor.
When this attached property is set, the effect will automatically get added to the view.
Implemented a shared IconTintColorEffectRouter and implemented an Android and iOS specific IconTintColorEffectRouter to implement this effect for these platforms.
Intended Use Case
If your app supports theming, you already easily can change colors on almost anything, except for your icons that you might use in an Image control or ImageButton. With this effect you can now easily change the color of your icons as well.
Who Will Do The Work?

[x] I am willing to take this on myself
[ ] Just putting this out there for someone to pick up

Look useful to me!

Look useful to me! Should we indicate in any way that this is only for iOS? How will we do that?

Why did you decide it's for iOS only?)

Look useful to me! Should we indicate in any way that this is only for iOS? How will we do that?

I will implement this for Android and for iOS! Not just for iOS.


Feat/popup tooltip

Before submitting the PR:

[x] Does your PR reference an issue? If not, please chat to the team on Discord or GitHub before submission.
[ ] Did you update and run tests before submission using npm run test?
[x] Does your branch follow our naming convention? If not, please amend the branch name using branch -m new-branch-name
[ ] Did you update documentation related to your new feature or changes?

What does your PR address?
Adds basic tooltip functionality so someone can easily add inline tooltips to elements without having to create separate popup elements on the page.
This is still a work in progress. I still need to:

[ ] Create element dynamically instead of always existing on page
[ ] Destroy dynamically created element when tooltip is hidden
[ ] Make sure all types are correct

If anyone can help with destroying/creating the element on the fly, that would be great.
Docs

[ ] Provide more examples
[ ] Make sure description is correct
[ ] Make sure params, etc all work
[ ] References to Popup where appropriate

Additional features
I would love to have some sort of top level wrapper that allows any elements with a "title" attribute to display a tooltip on hover. This seems sort of like a separate PR though.

@saturnonearth I see the benefit of this change, however I still believe this is something that can be worked into the current popup iteration. Give this, I'm going to close out this PR. However, I have noted it as a potential update for the upcoming popup refactor detailed here:
https://github.com/skeletonlabs/skeleton/issues/1350


Kill 9 error and running in parallel

Hi @ekg @AlistairNWard
I am now able to run freebayes on El Cpitan OS X on Mac. However, in between the process stops and gives me Kill 9 error on terminal and I get the pompt again. I ran this twice. Every time its stops at a different location and the VCF is created but partial.
Also, can you help me with the command to run freebayes in parallel using freebayes-parallel
freebayes - f hg38.fa S1.bam S2.bam S3.bam S4.bam
I am using the entire hg38.fa as the reference.
On some other biostars forum I read:
parallel --keep-order --max-procs 11 "freebayes --fasta-reference $REF \
--genotype-qualities --experimental-gls \
--region {} $BAM  " ::: $seqnames \
| vcffirstheader \
| vt normalize -r $REF - > $VCF

I am a bit confused I have two BAM files that need to be run with freebayes in one case I dont want to use vcffirstheader and vt normalise and in the second run I want to
Lets say the files are S1.bam and S2.bam and the reference is hg38.fa. Also do I need
--region {} $BAM ?
parallel --keep-order --max-procs 0  "freebayes --fasta-reference hg38.fa  " ::: S1.bam S2.bam > output_1.vcf
with the vcffirst header and vt normalise would it look like:
parallel --keep-order --max-procs 11 "freebayes --fasta-reference  hg38.fa " ::: S1.bam S2.bam | vcffirstheader | vt normalize -r hg38.fa - > output_2.vcf
Thanks,
Yaseen

Did you run without this parameter: --experimental-gls ? It no longer
exists.
On Nov 16, 2015 10:10 PM, "Yaseen Ladak"<EMAIL_ADDRESS>wrote:

Hi @ekg https://github.com/ekg @AlistairNWard
https://github.com/AlistairNWard
I am now able to run freebayes on El Cpitan OS X on Mac. However, in
between the process stops and gives me Kill 9 error on terminal and I get
the pompt again. I ran this twice. Every time its stops at a different
location and the VCF is created but partial.
Also, can you help me with the command to run freebayes in parallel using
freebayes-parallel
freebayes - f hg38.fa S1.bam S2.bam S3.bam S4.bam
I am using the entire hg38.fa as the reference.
On some other biostars forum I read:
parallel --keep-order --max-procs 11 "freebayes --fasta-reference $REF \
--genotype-qualities --experimental-gls
--region {} $BAM  " ::: $seqnames
| vcffirstheader
| vt normalize -r $REF - > $VCF
I am a bit confused I have two BAM files that need to be run with
freebayes in one case I dont want to use vcffirstheader and vt normalise
and in the second run I want to
Lets say the files are S1.bam and S2.bam and the reference is hg38.fa.
Also do I need
--region {} $BAM ?
parallel --keep-order --max-procs 0 "freebayes --fasta-reference hg38.fa "
::: S1.bam S2.bam > output_1.vcf
with the vcffirst header and vt normalise would it look like:
parallel --keep-order --max-procs 11 "freebayes --fasta-reference hg38.fa
" ::: S1.bam S2.bam | vcffirstheader | vt normalize -r hg38.fa - >
output_2.vcf
Thanks,
Yaseen
—
Reply to this email directly or view it on GitHub
https://github.com/ekg/freebayes/issues/221.

 I have two questions can I run using parallel this command of freebayes  and specify the number of cores.

I have just updated in the previous thread about make and make test.

@ekg  @AlistairNWard
any suggestions on how to run in parallel ?


Missing unversioned I-D containers

For most I-Ds we offer an unversioned “series” bibitem as well as a bibitem for each concrete I-D version.
For example: https://github.com/ietf-tools/relaton-data-ids/blob/main/data/draft-aa-ldp-link-shut.yaml and https://github.com/ietf-tools/relaton-data-ids/blob/main/data/draft-aa-ldp-link-shut-01.yaml.
However, for some items we only have the versioned bibitem. For example: https://github.com/ietf-tools/relaton-data-ids/blob/main/data/draft-avasarala-sipping-comm-div-notification-01.yaml

However, for some items we only have the versioned bibitem.

This should not happen? @andrew2net can you investigate?


Don't calculate expire timestamp manually

The token expire timestamp can be retrieved from the auth response and doesn't have to be calculated by us.

Fixes #127

Fixes #127


external links are broken in Simulator view

Please describe the bug
Some external links are broken on the simulator view page
From the https://biosimulators.org/simulators/vcell/latest page:
Working links

[x] Kisao Link in parameter table -- mostly addressed; see below and #1313
[x] Kisao link in description
[x] Framework link
[x] Format link
[ ] Citation Link -- partly addressed; see below and #1311
[x] Documentation link
[x] More info link

The issue with the citation links is addressed in the UI. The problem is that the database doesn't conform to the expected data model outlined by the API. See #1311. Once the database is fixed, the UI will be fixed.

The remaining issues are covered by #1311.


WIP: Update to Spring Boot 2.x

includes

Spring Security Java Config #456
Mockito 2.x #492
#494

Coverage decreased (-0.3%) to 66.625% when pulling e367747eaddd820167e5f707fc28a43e056a34a6 on grafjo:feature/spring-boot-2-update into 8ed01d6c2babc64bcc0117da7e33ee6242dbcb79 on synyx:master.

will be done in #501


Fixing pod bundled image assets issue when using swift+frameworks

So this addresses #213 . The commit 38d3d83 fixed this for the json file but this pull request fixes the issue for the images that are bundled with the pod

:+1:

Thank you @hamin :+1:

@KrauseFx not a problem :). Thanks for the great library and also thanks to @ribeto for helping me out with some bundled images stuff for this pull request :)


Change factory class to use conditional compilation to switch class types

Modify FClassOfCategory.cpp so that it has conditional compilation statements, and Makefile so that the switch is there. The idea is, as a stopgap, to not have the hard coding for different models buried inside this class's methods.

Each createXXXXX() method in the FClassOfCategory class get the parameter (TiXmlElement* parts), which is the pointer to the xml parameter description. The idea is that once we define the format how to specify class name in the description file, we can get the name of the class of each category and create the instance of the specified class.

Right, all I've done is modify the code for now so that this can be done in the Makefile, rather than having it hardcoded in that class. This will make it a little more obvious in the interim; my guess is it will be a little while before we revise our parameter file format and incorporate that into the program (and the workbench). I've committed the change to a separate branch, and will hold off on merging it into the main branch until our 3/3 meeting.


Adds node v4.0 to Travis build

Adds node v4.0 to Travis build
Description

Adds node v4.0 to Travis build
Please note: the v4.0 build passes and is not allowed to fail.

Replaced by #18.


[Bug] Error deploying to Heroku

Describe the Bug
Hi, I tried to deploy to Heroku.
Build is successful but when trying to access the app it crashes.
Log:
2020-03-30T14:44:39.134635+00:00 heroku[web.1]: State changed from crashed to starting
2020-03-30T14:44:50.705427+00:00 heroku[web.1]: State changed from starting to crashed
2020-03-30T14:44:50.214121+00:00 app[web.1]:
2020-03-30T14:44:50.214136+00:00 app[web.1]: ><EMAIL_ADDRESS>prestart /app
2020-03-30T14:44:50.214141+00:00 app[web.1]: > npm run css
2020-03-30T14:44:50.214142+00:00 app[web.1]:
2020-03-30T14:44:50.581385+00:00 app[web.1]:
2020-03-30T14:44:50.581399+00:00 app[web.1]: ><EMAIL_ADDRESS>css /app
2020-03-30T14:44:50.581400+00:00 app[web.1]: > postcss src/assets/tailwind/tailwind.src.css -o src/assets/tailwind/tailwind.css
2020-03-30T14:44:50.581401+00:00 app[web.1]:
2020-03-30T14:44:50.591869+00:00 app[web.1]: sh: 1: postcss: not found
2020-03-30T14:44:50.595343+00:00 app[web.1]: npm ERR! code ELIFECYCLE
2020-03-30T14:44:50.595536+00:00 app[web.1]: npm ERR! syscall spawn
2020-03-30T14:44:50.595664+00:00 app[web.1]: npm ERR! file sh
2020-03-30T14:44:50.595810+00:00 app[web.1]: npm ERR! errno ENOENT
2020-03-30T14:44:50.596766+00:00 app[web.1]: npm ERR<EMAIL_ADDRESS>css: `postcss src/assets/tailwind/tailwind.src.css -o src/assets/tailwind/tailwind.css`
2020-03-30T14:44:50.596823+00:00 app[web.1]: npm ERR! spawn ENOENT
2020-03-30T14:44:50.596926+00:00 app[web.1]: npm ERR!
2020-03-30T14:44:50.597062+00:00 app[web.1]: npm ERR! Failed at the<EMAIL_ADDRESS>css script.
2020-03-30T14:44:50.597131+00:00 app[web.1]: npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

Sorry if this is a really basic issue. :)

Thank you for bringing this to my attention :) Wouldn't have caught it otherwise. I will do run some rounds of testing on the app.json and fix this ASAP.

@jstrnbrg This has been fixed, and you should now be able to deploy with one-click :)
There's a handy deploy button that has been fixed now too, you can find it here.

@AmruthPillai that was quick 😮 and works like a charm. Thanks a lot 👍


Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined.

@MLPXBrachmann I'm still cutting my teeth on the React ecosystem, and am getting the above mentioned error when doing the following:
Rehyrdated.js
import React from 'react';
import { connect } from 'react-redux';

export const Rehydrated = connect(({ rehydrated }) => ({ rehydrated }))
((props) => props.rehydrate ? props.children : props.loading);

app.js
import Rehydrated from './components/Rehydrated';

      <ApolloProvider store={store} client={client}>
        <Rehydrated>
            <App>
        </Rehydrated>
      </ApolloProvider>

Can you tell me what exactly I'm doing incorrectly in Rehyrdated.js and what specifically { rehydrated } is supposed to reference if any?
Many thanks in advance

@TheoMer Don't worry, React can indeed be quite confusing at first.
The error you are getting basically tells you React is trying to render something that does not exist. In this case it may be the loading indicator (props.loading in Rehydrated.js). You need to either pass a React Element for app.js or add it to the Rehydrated component's defaultProps.
The updated code would look something like this:
components/Rehydrated.js
import React from 'react';
import { connect } from 'react-redux';

+ const Rehydrated = (props) => props.rehydrated ? props.children : props.loading;
+
+ // If you decide to not create a component to display while loading, use 'defaultProps'
+ Rehydrated.defaultProps = {
+   loading: false,
+ };
+
+ // Use 'export default' to declare a module's main export
- export const Rehydrated = connect(({ rehydrated }) => ({ rehydrated }))
- ((props) => props.rehydrate ? props.children : props.loading);
+ export default connect(({ rehydrated }) => ({ rehydrated }))(Rehydrated);

app.js
+ import offline from 'apollo-offline';
+ import React from 'react';
+ import { ApolloClient, ApolloProvider, createNetworkInterface } from 'react-apollo';
+ import ReactDOM from 'react-dom';
+ import { applyMiddleware, combineReducers, compose, createStore } from 'redux';
+ import config from 'redux-offline/lib/defaults';
+
+ import App from './App';
+ // Now you can use the default export
import Rehydrated from './components/Rehydrated';

+ /* Your setup goes here (see apollo-offline's README.md) */
+
+ /** Component to display while loading (if you choose to use one) */
+ const Loading = () => <div> Loading... </div>;
+
+ ReactDOM.render(
  <ApolloProvider client={client} store={store}>
-     <Rehydrated>
+     <Rehydrated loading={<Loading />}>
-       <App>
+       <App />
    </Rehydrated>
  </ApolloProvider>
+   document.getElementById('root'),
+ );

You do not have to specify Rehydrated.defaultProps and pass a loading element (although you may do both) - doing one should be enough to fix the error.
The { rehydrated } construct is a destructuring assignment that is used to map the Redux store's rehydrated field to the Rehydrated component's props (props.rehydrated).
The following code would be equivalent:
components/Rehydrated.js
/* Code from above... */

export default connect((state) => {
  return { rehydrated: state.rehydrated };
})(Rehydrated);

I hope this helps you solve your problem. If the error persists or you have any further questions, feel free to just comment again!