Can't receive message on foreground after turnoff notification setting for app

on iOS 13, when we turnoff notification setting of app, fcm not trigger message on foreground

Hi @letrungtrung, please provide more information on this issue by filing up the issue template,

Hi @rizafra ,
Client: latest firebase-ios sdk,
Server: latest firebase-admin sdk,
Device: iphone 11 , iOS 13.3.1
Go settings, turn off 'Allow Notifications' for my app
Open app to be ready receive notification , using firebase-admin send a notify to my device.
My app can't receive this notification
Go setting, turn on 'Allow Notifications' for my App
Testing again, received notification normally

Hi @letrungtrung, please provide more information on this issue by filing up the issue template,

Hi @letrungtrung, as far as I know, turning off "Allow notifications" in settings won't display notifications in foreground or background.

Hi @letrungtrung, as far as I know, turning off "Allow notifications" in settings won't display notifications in foreground or background.

It’s not display, but it still delegate to app to handle?

Hi @letrungtrung, as far as I know, turning off "Allow notifications" in settings won't display notifications in foreground or background.

Follow yours, on foreground fcm still use apns to send notification to app? Do you explain for me , thanks so much

We're deprecating the direct channel messages soon, but afaik this change hasn't been released yet.

can you share with us a sample payload you try to send using admin sdk?

can you share with us a sample payload you try to send using admin sdk?

Message.Builder messageBuilder = Message.builder();
messageBuilder.putData("key1", "data to test");
messageBuilder.setNotification(
Notification.builder()
.setBody("Hello notification")
.build()
);
messageBuilder.setApnsConfig(ApnsConfig.builder().setAps(Aps.builder().setBadge(1).build()).build());
messageBuilder.setToken("Token FCM");
Message msg = messageBuilder.build();
FirebaseMessaging.getInstance().sendAsync(msg);

We're deprecating the direct channel messages soon, but afaik this change hasn't been released yet.

What it means, bro?

@letrungtrung
Did you implement userNotificationCenter:willPresentNotification:withCompletionHandler: method?
If yes, does it get triggered when you try to send it?

@letrungtrung
Did you implement userNotificationCenter:willPresentNotification:withCompletionHandler: method?
If yes, does it get triggered when you try to send it?

I did it. It still not triggered.

The notification you sent has "notification " in the payload, meaning it's a display notification.And it will not be delivered once permission is off.
This is expected behavior.
If you want to send foreground silent notification that is not affected by notification permission, you should only set data payload.

Closing since this is intended (iOS platform) behavior. @letrungtrung please let us know if you have any more questions.


[Discussion] A concise script of Python for Spleeter

I'm trying to make a concise and a short python script to run the Spleeter Project.
My project requires the python code for Spleeter to be easier to read and follow. Everything seems to work well for "2stems vocals" except a small issue. The result of my script seems to add a small noise (I used Audacity to visualize it).

Does anyone know what causes these types of noises? i can't wrap my head around it. I'm worried about the effect that they may cause on the audio results.
Here is my Python code:
https://github.com/abokhalel2/Spleeter-Pytorch
Any suggestions would be greatly appreciated!

Hi @abokhalel2,
from what I understand, your issue is not linked to a bug in Spleeter but in your own code. So I think it should not be reported in Spleeter issues, and I'll close it.
Feel free to reopen and give further details if I got mistaken.


doc: Switch fuzzing docs to CMake

@pablomartin4btc

Tested on Ubuntu 22.04.
Found an issue trying to build it following the libFuzzer section, perhaps it's my setup but on the bitcoin repo following the current guide using make works fine for me.

Does https://github.com/hebasto/bitcoin/pull/115 fix this issue for you?

Shouldn't optional external signer be off as well?

Is it off on the master branch?

Shouldn't optional external signer be off as well?

Is it off on the master branch?

Yes.
That was on the fix (#115) branch (240308-cmake-CG).
On master following autotooling instructions from the current fuzzing doc:
./autogen.sh
CC=clang CXX=clang++ ./configure --enable-fuzz --with-sanitizers=address,fuzzer,undefined

Options used to compile and link:
  external signer = no
  multiprocess    = no
  with libs       = no
  with wallet     = yes
    with sqlite   = yes
    with bdb      = no
  with gui / qt   = no
  with zmq        = no
  with test       = not building test_bitcoin because fuzzing is enabled
  with fuzz binary = yes
  with bench      = no
  with upnp       = no
  with natpmp     = no
  USDT tracing    = yes
  sanitizers      = address,fuzzer,undefined
  debug enabled   = no
  gprof enabled   = no
  werror          = no

USDT tracing option is ON on both, master and cmake.
But I mentioned because during configuration user can see this legend:
CMake Warning at CMakeLists.txt:112 (message):
  FUZZ=ON will disable all other targets and force BUILD_FUZZ_BINARY=ON.

I got confused with targets/ options, perhaps we can clarify "default options x, y z, will remain on", or something like that, but maybe it's already clear and I missed it.

This needs to be rebased.

Rebased


Please add support for deerma.humidifier.jsq2q

Device model
deerma.humidifier.jsq2q
Device name
Mijia Smart Anti-bacterial Humidifier 2
Does the device require a MiCloud connection?
Do not know
Anything special about the device?
NO
https://home.miot-spec.com/spec/deerma.humidifier.jsq2g

Will add support in the next update. You can already add the device now and see if it works by auto detection.

Yes, the device is now supported!

Should work now in version 1.4.0


Installation error "module not found"

Hello, when trying to install plugin on latest netbox version (v3.4.6) I get an error "module not found".
Ubuntu 20.04, python 3.8.10
pip 23.0.1 from /opt/netbox/venv/lib/python3.8/site-packages/pip (python 3.8)
Is there a problem?
`
nbuser@netbox:/opt/netbox/netbox$ source /opt/netbox/venv/bin/activate
(venv) nbuser@netbox:/opt/netbox/netbox$ sudo pip3 install nextbox-ui-plugin
Requirement already satisfied: nextbox-ui-plugin in /usr/local/lib/python3.8/dist-packages (0.11.0)
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
(venv) nbuser@netbox:/opt/netbox/netbox$ python3 manage.py collectstatic
Traceback (most recent call last):
File "/opt/netbox/venv/lib/python3.8/site-packages/django/core/management/init.py", line 259, in fetch_command
app_name = commands[subcommand]
KeyError: 'collectstatic'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/opt/netbox/netbox/netbox/settings.py", line 688, in
plugin = importlib.import_module(plugin_name)
File "/usr/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 973, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'nextbox_ui_plugin'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/opt/netbox/venv/lib/python3.8/site-packages/django/core/management/init.py", line 446, in execute_from_command_line
utility.execute()
File "/opt/netbox/venv/lib/python3.8/site-packages/django/core/management/init.py", line 440, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/opt/netbox/venv/lib/python3.8/site-packages/django/core/management/init.py", line 266, in fetch_command
settings.INSTALLED_APPS
File "/opt/netbox/venv/lib/python3.8/site-packages/django/conf/init.py", line 92, in getattr
self._setup(name)
File "/opt/netbox/venv/lib/python3.8/site-packages/django/conf/init.py", line 79, in _setup
self._wrapped = Settings(settings_module)
File "/opt/netbox/venv/lib/python3.8/site-packages/django/conf/init.py", line 190, in init
mod = importlib.import_module(self.SETTINGS_MODULE)
File "/usr/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 671, in _load_unlocked
File "", line 848, in exec_module
File "", line 219, in _call_with_frames_removed
File "/opt/netbox/netbox/netbox/settings.py", line 691, in
raise ImproperlyConfigured(
django.core.exceptions.ImproperlyConfigured: Unable to import plugin nextbox_ui_plugin: Module not found. Check that the plugin module has been installed within the correct Python environment.
(venv) nbuser@netbox:/opt/netbox/netbox$ python3 manage.py migrate
Traceback (most recent call last):
File "/opt/netbox/netbox/netbox/settings.py", line 688, in
plugin = importlib.import_module(plugin_name)
File "/usr/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 973, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'nextbox_ui_plugin'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/opt/netbox/venv/lib/python3.8/site-packages/django/core/management/init.py", line 446, in execute_from_command_line
utility.execute()
File "/opt/netbox/venv/lib/python3.8/site-packages/django/core/management/init.py", line 440, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/opt/netbox/venv/lib/python3.8/site-packages/django/core/management/base.py", line 402, in run_from_argv
self.execute(*args, **cmd_options)
File "/opt/netbox/venv/lib/python3.8/site-packages/django/core/management/base.py", line 448, in execute
output = self.handle(*args, **options)
File "/opt/netbox/venv/lib/python3.8/site-packages/django/core/management/base.py", line 93, in wrapped
saved_locale = translation.get_language()
File "/opt/netbox/venv/lib/python3.8/site-packages/django/utils/translation/init.py", line 210, in get_language
return _trans.get_language()
File "/opt/netbox/venv/lib/python3.8/site-packages/django/utils/translation/init.py", line 65, in getattr
if settings.USE_I18N:
File "/opt/netbox/venv/lib/python3.8/site-packages/django/conf/init.py", line 92, in getattr
self._setup(name)
File "/opt/netbox/venv/lib/python3.8/site-packages/django/conf/init.py", line 79, in _setup
self._wrapped = Settings(settings_module)
File "/opt/netbox/venv/lib/python3.8/site-packages/django/conf/init.py", line 190, in init
mod = importlib.import_module(self.SETTINGS_MODULE)
File "/usr/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 975, in _find_and_load_unlocked
File "", line 671, in _load_unlocked
File "", line 848, in exec_module
File "", line 219, in _call_with_frames_removed
File "/opt/netbox/netbox/netbox/settings.py", line 691, in
raise ImproperlyConfigured(
django.core.exceptions.ImproperlyConfigured: Unable to import plugin nextbox_ui_plugin: Module not found. Check that the plugin module has been installed within the correct Python environment.
(venv) nbuser@netbox:/opt/netbox/netbox$
`

Sorry, it was a permission problem because of wrong user on ubuntu.
After
sudo chown nbuser /opt/netbox/venv/lib/python3.8/site-packages/
and
sudo chown nbuser /opt/netbox/netbox/static/
everything installed fine


Added build action

Fixed the build & add build action.

Added @disassembler to the authors.yml file
Added github-action to build on PR

@coot Ooops .. I also just did this and pushed to main. Was not sure whether we want to have PRs on this repo as well :)

I don't say we need PRs for everything, but if one creates a PR then let's give back feedback.

I sometimes push to main too :grin:


Run full specs

Summary
#1329 is big and not focused, so I'm splitting it.
This PR changes the default profile that we run in CI and locally to run more specs.
Details

I noticed that running bundle exec cucumber run way more specs than running bundle exec rake cucumber.
As far as I understand it, we're running a lot less features than we should. There's no reason to not run the "@todo-windows"-tagged scenarios by default, unless we're on Windows... right?
Motivation and Context

Prevention of regressions.
How Has This Been Tested?

Locally, via bundle exec rake cucumber.
Screenshots (if appropriate):
Types of changes

[ ] Refactor (code change that does not change external functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I've added tests for my code
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

And let's do this too.

I just noticed https://github.com/cucumber/cucumber/blob/master/CONTRIBUTING.md, where it says that PRs should sit for at least 36 hours to ensure that contributors in other timezones have time to review.
Sorry if I didn't follow the standard protocol, I can address any after-merge feedback on follow-up PRs. :+1:.
Also, I noticed an "Improvements" changelog section, is this kind of PRs something that should be logged in there?


Messed up text rendered by fromHTML plugin.

Code :
<a>"i"
"f you think the perfect yo clone has not been build then there is ...."

i got the output like this

I have a similar problem and have been trying to replace characters. Demo here: http://codepen.io/sol_b/pen/JbJVvQ
I can't figure out how to get the replace function working. How do you call string.replace?


[go-rancher] How to use client.Service.ActionRestart()?

Rancher Versions:
Server: 1.4.1
healthcheck: 0.2.3
ipsec: 0.8.7
network-services: metadata:v0.7.2, network-manager:v0.4.5
scheduler: 0.6.3
Docker Version: 1.13.1
I'm writing a Go app with go-rancher that needs to restart some specific containers. How is client.Service.ActionRestart supposed to be called?
func main() {

  // my starter class - go-rancher is functional in `client` from here on
  client := rancher.NewRancherServer(*configFilePtr, *keysFilePtr)

  services, err := client.Service.List(&client.ListOpts{})
  if err != nil {
    log.Fatal("Could not list services")
  }
  for _, service := range services.Data {
    if service.Name == "nginx" {
      fmt.Printf("Service: %+v\n", service.Name )
      var ret, err = client.Service.ActionRestart(&service, RancherServer.Client.ServiceRestart)
      fmt.Printf("ret: %+v\n", ret)
      fmt.Printf("err: %+v\n", err)
    }
  }

... produces error:
# command-line-arguments
./main.go:39: cannot use RancherServer.Client.ServiceRestart (type client.ServiceRestartOperations) as type *client.ServiceRestart in argument to RancherServer.Client.Service.ActionRestart

Is there a good example somewhere of how to call these action methods?

Please let us know in a comment if you can still reproduce the issue, and we'll reopen it


Improve locale sk for landing page

Coverage remained the same at 61.903% when pulling 4d4b84b429d8d334df8bfbc535e4f52b252a3f36 on Zaijo:patch-1 into dc9d5fa5e8493170428e51302eb4b6504d939ce0 on codecombat:master.

Could you sign our CLA so we could accept your PR?

Done :)
I agree.

On 5 Mar 2018, at 07:35, Valentin Bryukhanov<EMAIL_ADDRESS>wrote:
Could you sign our CLA https://codecombat.com/cla so we could accept your PR?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/codecombat/codecombat/pull/4633#issuecomment-370323220, or mute the thread https://github.com/notifications/unsubscribe-auth/AB37bB6tHsXgw-4g-6XzRLoSVzuOFnmIks5tbNytgaJpZM4STZ8x.

Thanks!


Adding

Adding power support ppc64le with Continues Integration/testing so that code remains architecture independent.
This is part of the Ubuntu distribution for ppc64le. This helps us simplify testing later when distributions are re-building and re-releasing. For more info tag @gerrith3.
The build is successful on both arch: amd64/ppc64le, please find the Travis Link below.
https://travis-ci.com/github/santosh653/errors
Please let me know if you need any further details.
Thank You !!

@santosh653 the CI is already setup for building the ppc64le architecture (This was introduced in this commit: https://github.com/masterzen/winrm/commit/e58a313440b7623a5f53a478b773ef4243c76c81 a month ago).
Wasn't that enough for your needs ?
/cc  @gerrith3.

@santosh653 looks like your patch only added a blank line and arch: ppc64le was already there??
@masterzen Not sure how that patch made it that far, but yeah, looks like you already had it covered.  Apologies for the confusion there!

@masterzen Apologies for the confusion. I am closing this PR.

Infact This PR can be closed.
Thanks !!


Bug: Edge device should be allowed on the platform

Expected Behavior

When creating an edge device on AWS, it should be allowed to connect.
Current Behavior

Screen tells that it's not allowed:

Steps to Reproduce

Create a device
Got the the device list

Context (Environment)
Portal version: v5.0.0
LoRaWAN Stack version:
Logs

Additional Information

Remarks: It seems that the sync job is removing this status.


Sample file in CMake template does not compile

Following the instructions, upon running the first cmake .. I get the following output:
> cmake ..
-- Populating raylib_cpp
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/evan/Projects/Bungeon/build/_deps/raylib_cpp-subbuild
[ 11%] Performing update step for 'raylib_cpp-populate'
[ 22%] No patch step for 'raylib_cpp-populate'
[ 33%] No configure step for 'raylib_cpp-populate'
[ 44%] No build step for 'raylib_cpp-populate'
[ 55%] No install step for 'raylib_cpp-populate'
[ 66%] No test step for 'raylib_cpp-populate'
[ 77%] Completed 'raylib_cpp-populate'
[100%] Built target raylib_cpp-populate
CMake Error at CMakeLists.txt:56 (target_link_libraries):
  The keyword signature for target_link_libraries has already been used with
  the target "raylib_cpp_example".  All uses of target_link_libraries with a
  target must be either all-keyword or all-plain.

  The uses of the keyword signature are here:

   * CMakeLists.txt:46 (target_link_libraries)

This is after copying both the CMakeLists.txt and main.cpp files from the starter project, creating, and cd'ing into the build folder.
Working from the command line on macOS Monterey.

It's also worth noting that if I clone the repo, I'm able to run cmake .. fine, but running make after that yields this result:
[ 44%] Building CXX object examples/CMakeFiles/physics_demo.dir/physics/physics_demo.cpp.o
In file included from /Users/evan/Projects/raylib-cpp/examples/physics/physics_demo.cpp:22:
In file included from /Users/evan/Projects/raylib-cpp/include/Physics.hpp:5:
/Users/evan/Projects/raylib-cpp/include/./physac.hpp:10:10: fatal error: 'extras/physac.h' file not found
#include "extras/physac.h" // NOLINT
         ^~~~~~~~~~~~~~~~~
1 error generated.
make[2]: *** [examples/CMakeFiles/physics_demo.dir/physics/physics_demo.cpp.o] Error 1
make[1]: *** [examples/CMakeFiles/physics_demo.dir/all] Error 2
make: *** [all] Error 2

#include "extras/physac.h" // NOLINT

Interesting... That's the same approach that the raylib demo takes. It should add the include directory with that extras directory there... :thinking:

CMake in the example

it's erroring out here...
# Checks if OSX and links appropriate frameworks (Only required on MacOS)
if (APPLE)
    target_link_libraries(${PROJECT_NAME} "-framework IOKit")
    target_link_libraries(${PROJECT_NAME} "-framework Cocoa")
    target_link_libraries(${PROJECT_NAME} "-framework OpenGL")
endif()

Could you try this instead...
    target_link_libraries(${PROJECT_NAME} "-framework IOKit -framework Cocoa -framework OpenGL")

Or even try removing that entirely?

Removing the if (APPLE) block in the CMakeLists seemed to have worked for me on Mac.

Thanks for the testing! Removed the block https://github.com/RobLoach/raylib-cpp/commit/e3bf04a36add426edf68b3b68a3ee59e77a62cc8

Hello @leftbones! I ran into the same issue with physac.h missing while trying to build the examples on my M1 Mac.
It looks like raysan5 forked Physac and updated it here (https://github.com/raysan5/physac), replacing the old #include "physac.h" with #include "extras/physac.h" in addition to removing Physac and its examples from raylib proper. That change was reflected here in raylib-cpp with this commit: https://github.com/RobLoach/raylib-cpp/commit/d0a04603dcfc08ed2e5a3f8b15d8c5bf1ae472e9#diff-f8b7d521f0390fa4a2c646e9e3e577d1ab82d6a78e9b4db4eb1e71ff26c9ac6d
In lieu of a proper solution, you can make a new extras folder in your local copy of https://github.com/RobLoach/raylib-cpp/tree/master/include and copy https://github.com/raysan5/physac/blob/master/src/physac.h into it. I guess it should work as long as extras/physac.h is somewhere on the include path, e.g., usr/local/include/extras/physac.h?
Once I did that, make built the examples, and the physics_demo executable ran fine.
I am sure there is a better place to put that header file or a nicer way to do this that does not involve directly modifying a local copy of raylib or raylib-cpp. I'm a little rusty on C++ and not very familiar with CMake, though.
RE the CMake error, I can confirm that @RobLoach 's fix above for the CMake file worked for me.

Thanks so much for the testing! I removed physac stuff from raylib-cpp and made a follow up issue if people still want C++ implementations of that API https://github.com/RobLoach/raylib-cpp/issues/208


Screen for showing Metadata

This isn't quite  perfect yet - but most of the pieces are in 👍

Fails

:no_entry_sign:

No CHANGELOG added.

:no_entry_sign:

Missing Test Files:

src/lib/Components/Consignments/Screens/__tests__/Metadata-tests.tsx

If these files are supposed to not exist, please update your PR body to include "Skip New Tests".

Warnings

:warning:

Please assign someone to merge this PR, and optionally include people who should review.

  Generated by :no_entry_sign: dangerJS

👍 looks good but looks like there's a merge conflict in the changelog so it won't let me merge


Convert to dotnet template?

don't you plan to release this as NuGet package compatible with dotnet/templating?

Good idea; I'll look into how we'd do that @sergey-tihon

Here you can take a look how fable-elmish do this - https://github.com/fable-compiler/fable-elmish/pull/80
Here is a list of available templates - https://github.com/dotnet/templating/wiki/Available-templates-for-dotnet-new


Wheels for Python 3.13

Py 3.13 is already a stable release so it would be nice to have wheels on PyPI as well.
Yet I have the following error:
error: Failed to prepare distributions
  Caused by: Failed to fetch wheel: sphn==0.1.4
  Caused by: Build backend failed to build wheel through `build_wheel` (exit status: 1)

[stdout]
Running `maturin pep517 build-wheel -i /home/yehor/.cache/uv/builds-v0/.tmpkMiaZV/bin/python --compatibility off`

[stderr]
📦 Including license file "/home/yehor/.cache/uv/sdists-v4/pypi/sphn/0.1.4/jeKKXRLkodekGQYr5gnKm/sphn-0.1.4.tar.gz/LICENSE"
🍹 Building a mixed python/rust project
🔗 Found pyo3 bindings
🐍 Found CPython 3.13 at /home/yehor/.cache/uv/builds-v0/.tmpkMiaZV/bin/python
📡 Using build options features, bindings from pyproject.toml
   Compiling autocfg v1.3.0
   Compiling cfg-if v1.0.0
   Compiling lazy_static v1.5.0
   Compiling arrayvec v0.7.6
   Compiling bitflags v1.3.2
   Compiling log v0.4.22
   Compiling bytemuck v1.18.0
   Compiling target-lexicon v0.12.16
   Compiling encoding_rs v0.8.34
   Compiling proc-macro2 v1.0.86
   Compiling once_cell v1.19.0
   Compiling unicode-ident v1.0.13
   Compiling symphonia-core v0.5.4
   Compiling libc v0.2.158
   Compiling crossbeam-utils v0.8.20
   Compiling shlex v1.3.0
   Compiling num-traits v0.2.19
   Compiling cc v1.1.18
   Compiling lock_api v0.4.12
   Compiling version_check v0.9.5
   Compiling pyo3-build-config v0.21.2
   Compiling quote v1.0.37
   Compiling syn v2.0.77
   Compiling parking_lot_core v0.9.10
   Compiling cmake v0.1.51
   Compiling symphonia-metadata v0.5.4
   Compiling num-integer v0.1.46
   Compiling num-complex v0.4.6
   Compiling symphonia-utils-xiph v0.5.4
   Compiling rustfft v6.2.0
   Compiling matrixmultiply v0.3.9
   Compiling memoffset v0.9.1
   Compiling pyo3-ffi v0.21.2
   Compiling strength_reduce v0.2.4
   Compiling scopeguard v1.2.0
   Compiling heck v0.4.1
error: failed to run custom build command for `pyo3-ffi v0.21.2`

Caused by:
  process didn't exit successfully: `/home/yehor/.cache/uv/sdists-v4/pypi/sphn/0.1.4/jeKKXRLkodekGQYr5gnKm/sphn-0.1.4.tar.gz/target/release/build/pyo3-ffi-f27568ae59e7c321/build-script-build` (exit status: 1)
  --- stdout
  cargo:rerun-if-env-changed=PYO3_CROSS
  cargo:rerun-if-env-changed=PYO3_CROSS_LIB_DIR
  cargo:rerun-if-env-changed=PYO3_CROSS_PYTHON_VERSION
  cargo:rerun-if-env-changed=PYO3_CROSS_PYTHON_IMPLEMENTATION
  cargo:rerun-if-env-changed=PYO3_PRINT_CONFIG
  cargo:rerun-if-env-changed=PYO3_USE_ABI3_FORWARD_COMPATIBILITY

  --- stderr
  error: the configured Python interpreter version (3.13) is newer than PyO3's maximum supported version (3.12)
  = help: please check if an updated version of PyO3 is available. Current version: 0.21.2
  = help: set PYO3_USE_ABI3_FORWARD_COMPATIBILITY=1 to suppress this check and build anyway using the stable ABI
warning: build failed, waiting for other jobs to finish...
💥 maturin failed
  Caused by: Failed to build a native library through cargo
  Caused by: Cargo build finished with "exit status: 101": `env -u CARGO PYO3_ENVIRONMENT_SIGNATURE="cpython-3.13-64bit" PYO3_PYTHON="/home/yehor/.cache/uv/builds-v0/.tmpkMiaZV/bin/python" PYTHON_SYS_EXECUTABLE="/home/yehor/.cache/uv/builds-v0/.tmpkMiaZV/bin/python" "cargo" "rustc" "--features" "pyo3/extension-module" "--message-format" "json-render-diagnostics" "--manifest-path" "/home/yehor/.cache/uv/sdists-v4/pypi/sphn/0.1.4/jeKKXRLkodekGQYr5gnKm/sphn-0.1.4.tar.gz/Cargo.toml" "--release" "--lib"`
Error: command ['maturin', 'pep517', 'build-wheel', '-i', '/home/yehor/.cache/uv/builds-v0/.tmpkMiaZV/bin/python', '--compatibility', 'off'] returned non-zero exit status 1

The cause here is this - error: the configured Python interpreter version (3.13) is newer than PyO3's maximum supported version (3.12) - as I understand.

Seems like PyO3 should be upgrade to 0.22 here - https://github.com/kyutai-labs/sphn/blob/main/Cargo.toml#L21C1-L21C16

I've just updated to PyO3 0.23 in #8 so hopefully we should be good with python 3.13. I'll have a look at building the new wheels at some point.


SEO Meta tags needs

SEO Meta tags need:
•	Meta title
•	Meta description
•	og:title Meta Tag
•	og:url Meta Tag
•	og:type Meta Data
•	og:description Meta Data
•	og:image Meta Data
•	Additional Meta Data
•	Canonical url
•	Robots Tag (index, no-index, follow, no-follow)
•	JSON-LD per page
•	In addition we need a key-value interface to add meta tags like: yandex-verification, HTTP-EQUIV )
this is how it is implemented in our AEM project

•	An example from 888poker.com site:

Adding screenshot of 888poker.com's meta

Not needed in doc metadata table

og:type can be added globally
og:url is added by default

In the metadata table for all docs:
Check if any of these should have a default value, if it is not authored on a page.

meta title = og:title
meta description = og:description
og:image
tags (keywords)
canonical -- Should the URL repeat by default? or have no value by default?
robots -- is there a default value?
yandex-verification
msvalidate.01
Google-site-verification

For json-ld, can this data be populated from the global + doc metadata table values? Or is there an additional field needed on a per-page basis?
<script type="application/ld+json">{
"@context": "https://schema.org",
"@type": "Organization",
"url": "//www.888poker.com/",
"logo": "https://images.images4us.com/888poker/en/PClogo-68x55-1487251650244_tcm1488-343920.png",
"name": "888 poker - Official Site!",
"description": "888poker is owned and operated by 888 Holdings plc as one the largest online poker rooms in the world.",
"sameAs": [
"https://en.wikipedia.org/wiki/888poker",
"https://www.facebook.com/888Poker",
"https://www.youtube.com/888poker",
"https://instagram.com/888poker",
"https://twitter.com/888poker"
] }</script>


Holiday Blueprint

Before you submit a new bug report, please check that

[X] I have read frequently asked questions and limitations

Write your question here
I have two entities (trash and recycling) on different schedules.  Is there a way to configure the Holiday blueprint to affect both?  When I select the entity in the dropdown, There doesn't seem to be a way to add an additional entity.

Just create two automatins. One for each entity.


Connection_limit in connection string has no effect

While performance testing our stack, I tried changing different aspects of test environment in order to find bottle necks.
The environment is fully hosted in kubernetes, and consists of 3 parts.

Data loading applications
Graphql-yoga server with prisma2 (2.0.0-preview024) (will refer to this as "prisma")
Postgres cluster

For the purpose of loadtesting, we made a custom app that doesnt actually do any data manipulation, it just generates data and makes http calls to an endpoint (upserts to the graphql api), it's able to push data at high volumes and thread safe, so we can scale the load indefinitely by simply adding more replicas.
For prisma I tried hosting the following:
1 - 10 prisma instances (5 CPU limit, 3Gi mem limit)
With 10 instances and full load from apps prisma cpu usage was around 230m CPU (0.23) and 800mb mem
With 1 instance around 2000m (2) CPU was used and around 800mb mem.
Tried the following connection strings set (for both the singe and the 10 instance scenario):
postgresql://postgres:pw@perftestenv/prisma?schema=public&sslmode=prefer
postgresql://postgres:pw@perftestenv/prisma?schema=public&sslmode=prefer?connection_limit=10
postgresql://postgres:pw@perftestenv/prisma?schema=public&sslmode=prefer?connection_limit=100
But this seemed to have no impact on active sessions to postgres at all. There would be a constant 4-15 number of active sessions

My question is: Is it expected the connection_limit has no effect on the number of connections prisma has to the database?

You are not using the property properly in the connection string. You will need to append it using & instead of ? if it is not the first property. It works browser query parameters.
Use the following ones:
postgresql://postgres:pw@perftestenv/prisma?schema=public&sslmode=prefer&connection_limit=10
postgresql://postgres:pw@perftestenv/prisma?schema=public&sslmode=prefer&connection_limit=100

Also, I tested them out just to make sure and the property work as expected.

Thanks @pantharshit00
I have changed it and doesnt seem to make much of a difference. What is the expected outcome - more active connections, more active+idle connections?
I tried with 10 prismas with limit=15 (left side of image), and 1 prisma with limit=100 (right side of image.

complete string env var for connection:
POSTGRES_URL=postgresql://postgres:@edh-perftest01/prisma?schema=public&sslmode=prefer&connection_limit=100

The limit is the maximum. If the pool can handle the requests with less connections, it will. You really need to blast it with huge amount of requests per prisma instance to get to the higher limits, and I don't think nodejs is able to do that...

Not quite sure what you mean with nodejs not being able to push enough requests.
We have both 20+ real java apps pushing data. Then to make sure the issue is not them being slow, added 10 replicas of a python app that sends upsert requests as fast as it can. Each can generate 1 million requests in 100 seconds (without sending the request).
When we were using prisma1 (same DB, same apps, except schema evolved) there was seemingly no bottleneck in prisma, the bottleneck then was the IOPS supported by the disk of the postgres.

@86label The connection_limit is per instance of Prisma Client. In the original comment, you are saying that you spawning 10 prisma instances so each server instance will spawn a connection pool with a max limit of 10 connections.
Can you please tell what exact behaviour you are expecting from the pool here?

@pantharshit00 Since the database is not maxed out on CPU, memory or disk IOPS, I would expect that if 1 prisma instance can make use of x connections, 10 should result in 10x active postgres connections, or up until DB exhaustion on CPU/memory/IOPS
In the image above I don't see any change in active sessions, if you look at Active Sessions the left side of that is 10 prismas running in parallel and there are approx 4 active sessions. I then shut down 9 of them and on the right hand side you can see there's still approx 4 active connections.

Fx now, we just tried making an app that does random inserts and spawned 10 of them:

And we can now see 10 active connections, and we get around 2700 transactions/sec, where with prisma it is 4 active connections regardless of having 10 or 1 prisma, and around 600 net transactions.

@86label I believe this can be closed now, right? This was addressed via meetings as far as I know.

Closing. Please re-open if you're still running into this @86label!


Feature: Pagination Component

Finished implementation of pagination component
Screenshot:
https://user-images.githubusercontent.com/104057020/178177230-41f023c0-d8d8-4a9c-b701-6f903dd50396.mov

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Avoid introducing new parentheses in annotated assignments

Summary
We decided to avoid changing this in https://github.com/astral-sh/ruff/issues/7315, but it's been reported multiple times (e.g., in https://github.com/astral-sh/ruff/issues/8226, also on Discord). I suggest we change it to improve compatibility. In general, it also seems to lend itself to better code style.
Closes #8188
Closes #8226
Test Plan
Shows improvements for CPython, home-assistant, Poetry, and typeshed.
Before:

project
similarity index
total files
changed files

cpython
0.75803
1799
1647

django
0.99983
2772
34

home-assistant
0.99953
10596
186

poetry
0.99891
317
17

transformers
0.99966
2657
330

twine
1.00000
33
0

typeshed
0.99978
3669
20

warehouse
0.99977
654
13

zulip
0.99970
1459
22

After:

project
similarity index
total files
changed files

cpython
0.75804
1799
1647

django
0.99983
2772
34

home-assistant
0.99960
10596
156

poetry
0.99897
317
17

transformers
0.99966
2657
330

twine
1.00000
33
0

typeshed
0.99980
3669
18

warehouse
0.99977
654
13

zulip
0.99970
1459
22

Went ahead and updated the docs. My current thinking here is that we should change this based on improving Black compatibility. The current formatting is better in some cases, e.g., from the docs:
# Black
StartElementHandler: Callable[[str, dict[str, str]], Any] | Callable[[str, list[str]], Any] | Callable[
    [str, dict[str, str], list[str]], Any
] | None

# Ruff
StartElementHandler: (
    Callable[[str, dict[str, str]], Any]
    | Callable[[str, list[str]], Any]
    | Callable[[str, dict[str, str], list[str]], Any]
    | None
)

But I think we can improve those by making progress on preview style -- and complex union annotations are also much rarer than simple annotations.


use array as preferred for credential types

Description
Changes PREFER_ARRAY to true for RequiredType trait implementation for PresentationType.
Received verifiable presentation submission schema validation errors complaining about type property being a string instead of an array.
Tested
Tested using VC playground oid4vp in sprucekit-mobile.

Looking into the failing test.


Move clirr.enabled = false to modules build.gradle files

Not possible. Code in the parent script is executed before the child scripts are evaluated. Due to the clirr plugin being applied in the parent script, the only place where it can be deactivated is also there.


Added changes to handle SystemPropertiesSecret credentials

What type of PR is this?

Improvement

What does this PR do / why is it needed ?
(for test purposes only) we can spin up dockerized mongodb and execute queries against it
<!--

Merged after conflict resolution


New Request copy stewardship

The 'New Request' view presents a sticky UI problem. This may not be a perfect solution but the page title 'New Request' with simple 'To' and 'From' field titles below can feel strange. e.g. The new request is 'From' the creator of the request not the receiver of the request.
Here's an attempt to provide the user with a little more guidance.

Thanks for resubmitting this.


Deem annotations in all patterns to be unrecognized.

And more clearly distinguish between:

instanceof Foo, the "type comparison operator"
instanceof Foo foo, the "pattern match operator"

(per
https://docs.oracle.com/javase/specs/jls/se22/html/jls-15.html#jls-15.20.2)
This prepares us for:

possibly someday recognizing annotations in the pattern match operator
while likely still not recognizing them in the type comparison
operator, as touched on in
https://github.com/jspecify/jspecify/issues/626

claiming to cover all of Java 22, as planned in
https://github.com/jspecify/jspecify/pull/666

To give a specific example of an annotation in a pattern other than in instanceof:
  int x(Foo<String> foo) {
    return switch (foo) {
      case Bar<@Nullable String> bar -> 0;
      case Baz<String> baz -> 1;
    };
  }


Elements Can Be Moved in Embedded Circuits

Describe the bug
Elements in the Embedded circuits are meant for display, and should not move upon dragging the elements. But Pressing any key on the Keyboard and then dragging the Element inside the embedded circuit is Possible, which should be disabled too.
To Reproduce
Steps to reproduce the behavior:

Go to 'Any Embedded Circuit'
Press any Key ( Ex: Ctrl, Shift, A, C etc. ) and drag a particular element
See error

Expected behavior
The elements should not move like they don't move while simple dragging.
Device Information [optional]:

OS: [Windows]
Browser [Chrome]

Are you working on this issue? (Yes/No)
No

@satu0king @tachyons Please have a look at this

@shubhankarsharma00  @gr455


fzy: 0.4 -> 0.7

Motivation for this change
I use the tool, and 0.7 fixes a crash bug I encountered.
Things done

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

[ ] NixOS
[x] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thanks!


assignemnt1 python done

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Added the requested python functions to check for anagrams.
What did you learn from the changes you have made?
to work with strings in python.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
The approach was pretty straight, I think. Turn words into lists of letters, sort them and check if they are the same.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
No major challenges. A couple of google searched did the trick.
How were these changes tested?
Through the check cases provided on the notebook.
A reference to a related issue in your repository (if applicable)
NA
Checklist

[X] I can confirm that my changes are working as intended

Python first assignment submitted.

You did a great job here!
Just keep in mind that the code could be written much simpler, shorter, and in a more pythonic way.
Please go ahead and close this pull request now. Thanks!

Hi @pedram-aliniaye-asli, Thank you for the comments.
I'm interested in seeing a simpler, shorter, and more pythonic version of the code.
If there is a way of sharing an example, that would be appreciated.
Thank you!


API returns wrong data

Hey,
i have a little cli script that just calls the getList() method and prints out every ID on a single row. I executed the script 7 times with 10 second delays during each execution and i get totally different results. During the time no new conversations where started on the website, nothing changed. But the API returned complete different results.
$ crisp-list-conversations | wc -l
144

$ crisp-list-conversations | wc -l
143

$ crisp-list-conversations | wc -l
122

$ crisp-list-conversations | wc -l
75

$ crisp-list-conversations | wc -l
57

$ crisp-list-conversations | wc -l
77

$ crisp-list-conversations | wc -l
47

As you can see the lines of results differ completely. I asume that you cache the requests somewhere and depending on what edge node serves the request i get a different response.
Could you please be so kind and look into this. That would help alot!
Thanks and greetings
Leo

I just executed the same script at the same time on two different laptops. One of them was in a different network, so we came from different IP addresses. I got very different results back from the API for the same call at the same time.

Closing (inactive).


Unable to disable new architecture in iOS

Description
I have recently enabled new architecture in iOS by running following command
RCT_NEW_ARCH_ENABLED=1 pod install

I would like to disable new architecture now, But unable to figure out a way. I couldn't find anything from official documentation as well.
Is there anyway to disable the new architecture or creating the new project is the only way?
Thank You.
Version
0.69.5
Output of npx react-native info
System:
     OS: macOS 12.5
     CPU: (8) x64 Apple M1
     Memory: 24.20 MB / 8.00 GB
     Shell: 5.8.1 - /bin/zsh
Binaries:
     Node: 14.0.0 - /usr/local/bin/node
     Yarn: Not Found
      npm: 6.14.4 - /usr/local/bin/npm
      Watchman: Not Found
 Managers:
       CocoaPods: 1.11.2 - /opt/homebrew/bin/pod
 SDKs:
       iOS SDK:
       Platforms: DriverKit 21.4, iOS 15.5, macOS 12.3, tvOS 15.4, watchOS 8.5
 Android SDK: Not Found
 IDEs:
        Android Studio: 4.2 AI-202.76<IP_ADDRESS>51085
        Xcode: 13.4.1/13F100 - /usr/bin/xcodebuild
 Languages:
        Java: 11.0.10 - /usr/bin/javac
 npmPackages:
        @react-native-community/cli: Not Found
        react: Not Found
        react-native: Not Found
        react-native-macos: Not Found
        npmGlobalPackages:
         *react-native*: Not Found

Steps to reproduce
Since it is not any error I cannot provide any reproducible steps. I am just looking for a way to disable the new architecture in iOS.
Snack, code example, screenshot, or link to a repository
I do not have any code snippet since I am not facing any errors.

I would like to disable new architecture now, But unable to figure out a way. I couldn't find anything from official documentation as well.
Is there anyway to disable the new architecture or creating the new project is the only way?

If you run RCT_NEW_ARCH_ENABLED=0 pod install you'll be effectively disabling the New Architecture.

I would like to disable new architecture now, But unable to figure out a way. I couldn't find anything from official documentation as well.
Is there anyway to disable the new architecture or creating the new project is the only way?

If you run RCT_NEW_ARCH_ENABLED=0 pod install you'll be effectively disabling the New Architecture.

@cortinico This solution doesn't seem like working.
Let me explain what's happening in my app,
I ran RCT_NEW_ARCH_ENABLED=1 pod install and pod installation was successful. When I ran app from Xcode faced the  following error,
 `react/renderer/components/view/ViewEventEmitter.h' file not found

This error is coming from following file,
 ios/Pods/Headers/Private/React-Codegen/react/renderer/components/rncore/EventEmitters.h

it's looking for a file which supposed to be in the following location,
 ios/Pods/Headers/Private/React-Codegen/react/renderer/components/view/ViewEventEmitter.h

But in fact, This file doesn't exist in the above path.
RCT_NEW_ARCH_ENABLED=1 pod install command resulted in  creating a folder called react(this folder contains the path of file where error is occurring) react/renderer/components/view/ViewEventEmitter.h)  which was not there before enabling  the new architecture.
Though I ran RCT_NEW_ARCH_ENABLED=0 pod install, still react folder is there,  So does error.
What should I do now to get rid of this error?

Hi @Yandamuri, thanks for opening the issue and I'm sorry you are incurring into it.
Could you try to:

navigate to the ios folder of your project
remove the build folder
run pod install
And see if it fixes the issue?

It looks like that your build folder kept some dirty files from the previous build.
Let me know if it works!

@cipolleschi This is great. Thank you so much. Issue got resolved.
I am just asking out of curiosity, Do you have any idea about why the ViewEventEmitter.h isn't there(Which explained in my above comment) when I ran RCT_NEW_ARCH_ENABLED=1 pod install?

When running RCT_NEW_ARCH_ENABLED=1 pod install, we are also running a bit of Codegen to generate some files in the React-Codegen. In the old architecture, we run a similar Codegen (actually the same code) but from a different entry points and with slightly different parameters.
I have not explored in deep why this error occur, but my guess is that the two Codegen processes generate file in slightly different way and locations. Cocoapods could have some cache heuristic which fails to pick up the differences and it could be that it doesn't update the dependencies correctly.

@cipolleschi Surprisingly The mentioned error gone now, In Fact, the new react  folder(ios/Pods/Headers/Private/React-Codegen/react) hasn't been created now. But When I ran RCT_NEW_ARCH_ENABLED=1 pod install faced following error,
 error: Could not determine react-native-codegen location. Try running 'yarn install' or 'npm install' in your project root.

Then I installed  react-native-codegen  and then ran RCT_NEW_ARCH_ENABLED=1 pod install. Then pod installation was successful.
Is react-native-codegen mandatory to enable new architecture?

react-native-codegen is indeed required by the New Architecture. It is a package within React Native, therefore, when you install React Native, it will pull also that package automatically.
From the error message, it seems like React Native did not install itself correctly. Rather than explicitly install react-native-codegen package, could you please try to just run yarn install and see if the error disappear?

@cipolleschi Yes you are right. After npm install issue got resolved.
Thank you for your support.

Hi @Yandamuri, thanks for opening the issue and I'm sorry you are incurring into it.
Could you try to:

navigate to the ios folder of your project
remove the build folder
run pod install
And see if it fixes the issue?

It looks like that your build folder kept some dirty files from the previous build. Let me know if it works!

delete ios build folder and rebuild works for me, thanks very much

For now new arch is disabled by default. If you run it without environment variable all pods will be installed with old architecture.

you can try to add a target to your podfile like below.
platform :ios, '13.0'
project './projectname.xcodeproj'


DEP: removed deprecated functionality in skbio.tree

Coverage decreased (-0.09%) to 98.78% when pulling b3a003e4ab7d65305b8c9feba2e755bedc64cc32 on ebolyen:dep_tree into d9371ebb210d6ef1b9bd1edf7ad5ee08fb58c231 on biocore:master.

Coverage decreased (-0.2%) to 98.67% when pulling b32474663dd8a738d362d0a554bcacdc6acf686e on ebolyen:dep_tree into d9371ebb210d6ef1b9bd1edf7ad5ee08fb58c231 on biocore:master.

Coverage decreased (-0.09%) to 98.78% when pulling b32474663dd8a738d362d0a554bcacdc6acf686e on ebolyen:dep_tree into d9371ebb210d6ef1b9bd1edf7ad5ee08fb58c231 on biocore:master.


Back up and restore list of installed apps

In case of new android device

@dergigi's wet dream

One of the more interesting features that damus hasnt implemented is the daniel's onboarding: https://github.com/damus-io/damus/issues/1696.
Imagine inviting your friend with pre-configured follows, relays, LN wallet, and even a prepaid subscription.
In addition to personal backup, we can have newbies onboarded to android, nostr, btc/ln/enuts the @dergigi way. I'll make a new ticket for this.

See https://github.com/zapstore/zapstore/issues/49


add private-subnet example

[Enter feedback here]
Can you add example how to specify both name of private-subnet and range of IPs.

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

ID: 5e3ccf76-e4ff-16db-01c5-25a19798a58e
Version Independent ID: d2ef04fc-c838-576e-9d40-729e30e2b3f4
Content: az databricks workspace
Content Source: latest/docs-ref-autogen/databricks/workspace.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to service team

Hi @dejan-shuki , can you describe it more clearly, what kind of example do you need ?


[ISSUE-#7261] Remove unnecessary logger which is defined  at subclass.

What is the purpose of the change
For-#7261.

Codecov Report

Merging #7262 (1c0d181) into master (15fc5b4) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

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


Document unique IDs for entity db tables

JIRA issue [TW-1330]
Documents issues around:

restoring one app's data to a different app
sharing a database between two apps
copying data from one app to another

Includes hints as to a proper microservices architecture

Approved by data storage via Slack.


update: add merkle root to tendermint bundle summary

This PR updates the @kyvejs/tendermint runtime with the Merkle root that is required for the Trustless API integration.

should we move merkle.ts into @kyvejs/sdk or @kyvejs/protocol? since other runtimes like @kyvejs/tendermint-bsync also want to use the same logic

Only generateMerkleRoot could be reused for all runtimes. dataItemToSha256 and createHashesFromBundle are runtime specific. We could create an interface that implements the core logic with dataItemToSha256 just hashing the whole data item, whereas other runtimes can overwrite this function due to more granular hashing.


Support for skipUntil and skipWhile

skipUntil
The skipUntil method skips items until the given value is found and then returns the remaining items in the collection:
collect([1, 1, 2, 2, 3, 3, 4, 4]).skipUntil(2);
// [2, 2, 3, 3, 4, 4]

Also, skipUntil method skips items until the given callback returns true and then returns the remaining items in the collection:
collect([1, 1, 2, 2, 3, 3, 4, 4]).skipUntil(item => item >= 2);
// [2, 2, 3, 3, 4, 4]

Laravel equivalent: https://laravel.com/docs/7.x/collections#method-skipuntil
skipWhile
The skipWhile method skips items while the given value is found and then returns the remaining items in the collection:
collect([1, 1, 2, 2, 3, 3, 4, 4]).skipWhile(1);
// [2, 2, 3, 3, 4, 4]

Also, skipWhile method skips items while the given callback returns true and then returns the remaining items in the collection:
collect([1, 1, 2, 2, 3, 3, 4, 4]).skipWhile(item => item < 2);
// [2, 2, 3, 3, 4, 4]

Laravel equivalent: https://laravel.com/docs/7.x/collections#method-skipwhile

These methods were added in Laravel 7.x
NOTE: If it is approved, I can open a PR.

Thanks for the heads up.
Merged https://github.com/ecrmnn/collect.js/commit/f0fdde1a90f7ffcfc9a2935116e50e361c81d65b

Thanks for the heads up.
Merged f0fdde1

Oh! you already did it!
I thought I could take this up, had already finished it, but didn't open the PR, waited for your approval!
I'm just beginning to contribute to the open-source projects I use and find really useful.
Nevermind! Maybe next time!
And yeah, the source is really really good! simple and clean.

Hehe, yeah I had it sitting in a branch but hadn't finished it yet.
Feel free to contribute next time :)

Thought so!
Sure 😄


GET  /admin/entities/{entityID} needs to return balance, maxPositiveBalanceandmaxNegativeBalance`:

@LIYINGZHEN Regarding #111, it has not been implemented correctly here. We should be returning tradingRejected and tradingPending status entities to users.

It's only the rejected and pending status entities that should not be returned, which are being filtered out correctly.

Also, the admin search should return all statuses. Right now it is leaving out rejected and pending.


Add Docker support

Add Dockerfile to enable image building.
Using the official GCC image, latest tag. More info at https://hub.docker.com/_/gcc/
Just adding files, setting working dir and running make instructions.
Build:
$ docker build -t twemproxy .

Run:
$ docker run --rm -it -p [ext_port1:port1] -p [ext_port2:port2] -p [...] -v [your_host_config_yml]:/etc/nutcracker.yml:ro twemproxy -c /etc/nutcracker.yml --verbose=6 [other options...]

FYI, there's a still quicker to test, already built image on my Docker Hub. Test it by running:
$ docker run --rm -it -p [ext_port1:port1] -p [ext_port2:port2] -p [...] -v [your_host_config_yml]:/etc/nutcracker.yml:ro pataquets/twemproxy  -c /etc/nutcracker.yml --verbose=6 [other options...]

Using --rm instead of -d makes the container not go background and it to be deleted after stop. Should stop by CTRL+C'ing it.
In order for the Docker container to connect to external memcached or Redis instances, either them should be contactable as external IPs or hosts or be linked to other previously run Docker containers via Docker's --link option.
Here it is an example Docker Compose file I'm using (I can submit it with the PR also if you find it useful):
redis1:
  image: redis:3.2
  command: --save "" --maxmemory 128mb
  ports:
    - 63791:6379

redis2:
  image: redis:3.2
  command: --save "" --maxmemory 128mb
  ports:
    - 63792:6379

twemproxy:
  image: pataquets/twemproxy
  command: -c /etc/nutcracker.yml --verbose=6
  links:
    - redis1
    - redis2
  ports:
    - 6379:6379
    - 22222:22222
  volumes:
    - ./conf/nutcracker.redis.yml:/etc/nutcracker.yml:ro

Notice that it is tuned for two Redis instances (yml file not included, mount yours)
Optional improvement to come (maybe in another issue):

Create an 'official', based on your repo, automated build at Docker Hub for the image: https://docs.docker.com/docker-hub/builds/ . Just requires a free Docker Hub account and a following a quick 'Create automated build' process. I'll be happy to help on it, if needed.

Bump.

@manjuraj : ping (since you're the main commiter)

Dynomite has Dockerfile. Maybe we should consider that too. https://github.com/Netflix/dynomite/blob/dev/docker/Dockerfile
Also, we should use Docker multi-stage build to further reduce footprint.

Any update on this?

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

@manjuraj: Anything else needed here?
I've done a quick search, and there are lots of rogue twemproxy public Docker images:
$ docker search twemproxy
NAME                          DESCRIPTION                                     STARS               OFFICIAL            AUTOMATED
zapier/twemproxy              Docker image for running twemproxy both with…   5                                       [OK]
anchorfree/twemproxy                                                          3
softonic/twemproxy            Cache proxy                                     2
malexer/twemproxy             A tiny (<6mb) Alpine-based Docker image of t…   1                                       [OK]
yfix/twemproxy                Dockerized twemproxy                            1                                       [OK]
giosg/twemproxy                                                               0
bdossantos/twemproxy          Twemproxy, aka nutcracker is a fast and ligh…   0                                       [OK]
jgoodall/twemproxy            Docker image of twemproxy proxy server in fr…   0                                       [OK]
the40san/twemproxy            twemproxy for docker                            0
ganomede/twemproxy                                                            0                                       [OK]
orimarti/twemproxy-exporter   Prometheus twemproxy exporter                   0
otus/twemproxy                                                                0
tarantool/twemproxy           This is a Docker image with a custom build o…   0                                       [OK]
vimagick/twemproxy            Twemproxy, aka nutcracker is a fast and ligh…   0
bnusunny/twemproxy                                                            0
ianblenke/twemproxy                                                           0                                       [OK]
mattrobenolt/twemproxy        Twemproxy (nutcracker)                          0
phhhoto/twemproxy                                                             0
ruo91/twemproxy               Twitter - Twemproxy                             0                                       [OK]
livelink/twemproxy                                                            0
michaeltigr/twemproxy         Twemproxy                                       0                                       [OK]
dispatchrobot/twemproxy                                                       0
gojektech/twemproxy                                                           0
perlmint/twemproxy            lightweight twemproxy docker image - based o…   0
studossoftware/twemproxy                                                      0

Having an official one would be better, IMO.

Done. Thank you @pataquets!

@manjuraj You're welcome :)
How about publishing an official image built from source on Docker Hub using Automated Builds? Alternatively, GitHub has also a public Docker image registry in their CI/CD tools. I can file an issue, if needed.
Also, do you think that submitting an example Docker Compose manifest as stated in the OP would be worthwhile? I can send another PR if it's likely to be merged.

@manjuraj Fancy to set up an Automated Build on Docker Hub? Or any other similar 'source-built' Docker public image as a snapshot and deployable artifact.


Moving to Webpack

Planning to move this build to Webpack (from Grunt/RequireJS). Build will be broken for a little while until that is completed.

Actually, the more I use Webpack the more I prefer the old ways of Grunt - so I'll move to Babel/Browserify within a Grunt setup instead.

The framework uses Babel/Browserify now, so closing this.


Networking working group blog post

What are your changes?
What is the urgency level of this change?

[ ] Blocker
[x] High
[ ] Medium

I have completed these items:

[x] I have built locally and tested for broken links and formatting.
[x] If appropriate, I have added redirects.

If you included a comment with your commit, it appears here:

@ryadav88 one more...

Scrapping this and starting over.


cannot make successfully!

I have cloned the whole project include the submodules use the "git clone --recursive https://github.com/espressif/esp32-at.git" command, and then follow the compiling guide:
rm sdkconfig
make defconfig
make
at the end of build, I met this error:
...
CC build/wpa_supplicant/src/fast_crypto/fast_sha256-internal.o
CC build/wpa_supplicant/src/fast_crypto/fast_aes-wrap.o
AR build/wpa_supplicant/libwpa_supplicant.a
CC build/xtensa-debug-module/trax.o
AR build/xtensa-debug-module/libxtensa-debug-module.a
LD build/esp-at.elf
D:/work/EADB/code/Firmware/ESP32/esp32-at/components/at/lib\libat_core.a(at_port.o):(.literal.initialise_wifi+0x14): undefined reference to `g_wifi_osi_funcs'
collect2.exe: error: ld returned 1 exit status
make: *** [D:\work\EPCID2IN1\WirelessBoard\Firmware\epcid-esp32-firmware\esp-idf/make/project.mk:406：/d/work/EADB/code/Firmware/ESP32/esp32-at/build/esp-at.elf] 错误 1
does any body build the  libat_core.a version c3c7f6fb sucessfully?

Hi,
Sometimes I get error using "make" command and using "make all" solve this maybe you can try ?

@JuPrgn Thx for the response. I have tried the "make all" command and got same error.

Hi @ldsheng ,
Please try rm -rf build, and then make.

HI @xcguang ,
Have tried that. remove the whole build directory and make again. It still reflects same error. Exactly same as I posted at the beginning.

Hi @ldsheng  ' g_wifi_osi_funcs ' is defined on  esp-idf/components/esp32/wifi_os_adapter.c L400, can you find the API under your ESP-IDF?   You can renew the esp-at project and submodule by git submodule update --init --recursive  and then export IDF_PATH=/path of esp-at/esp-idf/ to export the IDF

It's OK now.
Thanks @yake827 , thank you for your information.
It turns out that I have several version on my disk, and I forgot to re-export the IDF path to the one I recently work on:(. I have found the defination of the g_wifi_osi_funcs on wifi_os_adapter.c L400 and then I realized it's a version mismatch problem. And then I checked the IDF_PATH and BINGO! :) Thank you guys!


DMA behaviour with overlapping source and destination / Using DMA to memset

I am thinking of making memset faster using DMA. Assuming I have already implemented memcpy, then my memset is:
void memset(void *s, uint8_t c, uint32_t n) {
	(*(uint8_t *)s) = c; // initialize the first byte
	memcpy(s + 1, s, n - 1); // copy to the remaining bytes
}

However, this does not work well in the simulator.
So the question is, does the DMA has a defined behaviour when the source address is overlapping with the destination address? Is it possible to implement memset in a similar way? The docs says "Once intiated the DMA channel will transfer up to 32-bits per CPU cycle until the transfer has been completed.", so will this work if I initialize 4 bytes instead of only 1 byte?

It isn't just 4 bytes, but also 4 bytes that are 32-bit aligned. The transfer tries to align to 32-bit and then copy 32-bit at a time until it hits the end (or last 1 - 3 bytes).

What is the behavior when the source and destination addresses are not aligned? For example source = 0x70003333 and dest = 0x70000000

The alignment is focused on the destination being 32-bit aligned. So this should transfer 32-bits at a time from the beginning. If the dest were 0x70000001, then it should write 3 bytes first, then 32-bit transfers after.

Thanks for answering this (#17) and #16. I would still recommend documentation improvement at https://github.com/UCDClassNitta/riscv-console/blob/main/docs/chipset.md#dma-channel . But please feel free two close these two issues.


Byte-like object required error

I am getting the following error when runnig this.

[2018-01-04 11:12:23,372] {models.py:1342} INFO - Executing on 2018-01-04 00:00:00

[2018-01-04 11:12:23,400] {base_hook.py:67} INFO - Using connection to: <IP_ADDRESS>

[2018-01-04 11:12:24,903] {models.py:1417} ERROR - a bytes-like object is required, not 'str'

Traceback (most recent call last):

File "/home/hyperli/.local/lib/python3.5/site-packages/airflow/models.py", line 1374, in run result = task_copy.execute(context=context)

File "/home/hyperli/.local/lib/python3.5/site-packages/airflow/contrib/operators/mysql_to_gcs.py", line 91, in execute

files_to_upload = self._write_local_data_files(cursor)

File "/home/hyperli/.local/lib/python3.5/site-packages/airflow/contrib/operators/mysql_to_gcs.py", line 136, in _write_local_data_files

json.dump(row_dict, tmp_file_handle)

File "/usr/lib/python3.5/json/init.py", line 179, in dump fp.write(chunk)

File "/usr/lib/python3.5/tempfile.py", line 622, in func_wrapper return func(*args, **kwargs)

TypeError: a bytes-like object is required, not 'str'

Hi Mike,
I also face the same error, could you care to help?

@TiaMotsoane @carillpower, just saw your issues today. I am going to push some updates this week ahead of a conference I am attending, check back in a week and hopefully this will be resolved. Thank you both for commenting on the repo, I'm sure this is something related to this repo being a little out of date, stay tuned

Hi Mikeghen,
Thanks for the nice documentation. I am actually facing an error while executing MySQL_GCP.py DAG file.
Request you to kindly help me in resolving the issue.
Issue Description :
Broken DAG: [/usr/local/lib/python2.7/dist-packages/airflow/example_dags/mysql_to_gcs.py] No module named google.auth


Implemented ability to specify parameters for custom code injections

Description
This is a working version of the functionality to allow code injections into user defined methods with parameters specified. For example:
class A {
  void foo(String a) {
    [...]
  }

  void foo(int a) {
    [..]
  }
}

You can now specify the parameters in certain ways to target those methods specifically:
If you want to inject into the first method and only the first method:
before custom foo(String) {
  [...]
}

Or if you want to inject only into the second method:
before custom foo(int) {
  [...]
}

Or if you want to inject into both:
before custom foo(...) {
  [...]
}

This also injects into both (if parameters unspecified -- it is presumed that you don't care about them):
before custom foo {
  [...]
}

You can also just use multiple code injections on the same line and specify parameters for all of them:
before custom foo(int),foo(String) {
  [...]
}

Tests
Cleaned up some of the tests that needed to be changed (for minor and justifiable reasons) due to my changes to the grammar. Added in new test files to test this functionality -- covering all of the examples that I mentioned above.

It is very, very important to never commit changes to  cruise.umple/src/UmpleVersion.ump
Please get rid of this change from the PR.
I also find it odd that you have committed changes to AssociationSortedWithNameSpace_Mentor.java.txt and AssociationSortedWithNameSpace_Student.java.txt
We just changed these before the last release to make tests pass ... are you sure this is properly rebased?

Yes, I have rebased properly. The changes to the two test files entailed nothing more than removing spaces on a line. The tests wouldn't pass because of those spaces. Even reverting my changes to those files now, leads to failures in the tests.
I've removed the UmpleVersion file commit. Sorry about that.

Sorry, I nitpicked a bit. I needed a short break from my studies. :smile:
It looks good, otherwise, though.

Thanks Kevin. I fixed/committed the two issues you brought up.

Thanks for the PR.

The names in Umple are case sensitive so please make sure your solution supports it. Current version doesn't differentiate.
Furthermore, consider the following example:

class A {
	void testData(String i){
		//hello
	}
	void TestData(String i, String j){
		//hello To
		if (x==1) return;
		if (x==0) getData();
		return;
	}
	after custom testData {
		//Injected code
		This is the code.
	}
}

It injects code to TestData(String i, String j) while it must respect to names and also parameters.

The inject code doesn't have a proper format. It might be because of our new generator. Have a look at it.

We talked about events of state machines, but as I said we need to talk about it in the issue section.

If there is no matched joint point, please show a warning regarding that. for example, in the following command related to the above example, there is no method wit Integer parameter.

	after custom testData(Integer), testData(String,String) {
		//Injected code
		This is the code.
	}

It looks like you have a warning for this cases (without parameters). You need to apply it to methods with parameters as well.

Consider the following example

class A{
  data;
  setData(String i,String j){
  }
  after setData(String,String) {
    //this is injected code
  }
}

The output is not clear. If it's going to be applied to just auto-generated code why we accept parameters. If you aim at both we need to apply the inject code to both of them. I'm confused about the grammar and output. @TimLethbridge may want to comment on this for more clarification as well.

I have a question. If you use custom keyword, will you inject code for auto-generated ones?

class A{
  data;
  setData(String i,String j){
  }
  after custom setData{
  	//hey this is a injected code.
  }
}

Now, after says, any method with the name setData without paying attention to parameters. Please clarify this to me. @TimLethbridge may want to comment on this as well.

Thanks for the comments Vahdat! You raised 3 different issues in your comment: (1) injections are not case sensitive, (2) there is no warning for parameter-based injections, (3) how the injection targeting works.
(1) Regarding your example, it seems like code injections were never case sensitive. I didn't change anything with regards to that in this PR. Consider the following example:
class A {
  foo;

  after SETFoo {
    // inject
  }
}

This will inject into the setFoo method regardless of the fact that the case of the method name in the injection doesn't match. Based on a preliminary look through the code -- it seems that this is an issue with StringFormatter.toUnderscore being called on both the operation name (in the injection) and the method name. I have a rough idea of how to deal with this issue, but I think something like this should be a separate change.
So if we do want to change how Umple handles code injections to make them case sensitive (since they aren't right now), that's a separate issue and a separate PR.
(2) Warnings not being shown when a method name match exists but a parameter match doesn't.
Sorry, I forgot about the warnings. I can update the PR with this ASAP. (Probably around 10pm PST today).
(3) Confusion about grammar for injections.
Right now the parameters are completely disregarded for auto-generated methods. Specifying them won't change anything about where the injection occurs. I wasn't sure whether this is something we wanted done and I wanted to make sure that it was before I hunted down every call to getApplicableCodeInjections.

If you use custom keyword, will you inject code for auto-generated ones?

No you won't. In this situation, you'd only inject into user defined methods.

Ok. I fixed the issue with the warnings. It should now display a warning for all situations in which an injection will not take place for injections that specify parameters.

Thanks. I'll create two issues regarding this PR that need separate PR.


Improve integration with autoscoper

Rename module from AutoscoperConnect to AutoscoperM
Update icon for AutoscoperM extension & module
Improve support for starting/stopping autoscoper

Move "Launch" button to the top
Add "Close" button for closing the process
Simplify the logic expecting the executable to be in the PATH

Refactor Autoscoper communication using PyAutoscoper
Remove obsolete AutoscoperMockServer module
Remove unused sample data registration and testing code

Before
After

Context
This pull-request is based of:

https://github.com/BrownBiomechanics/SlicerAutoscoperM/pull/4

@jcfr
I'm getting this error when I try to configure the project with this command cmake .. -DSlicer_DIR="C:/D/S4R/Slicer-build" -DQt5_DIR="C:/Qt/5.15.2/msvc2019_64/lib/cmake/Qt5" -DSlicerAutoscoperM_SUPERBUILD=ON
CMake Error at C:/D/S4/CMake/SlicerExtensionCPack.cmake:174 (message):
  SlicerAutoscoperM: Variable CPACK_INSTALL_CMAKE_PROJECTS is expected to be
  set.
Call Stack (most recent call first):
  CMakeLists.txt:27 (include)

Not sure what I need to set this to, I didn't have it set to anything before.

Notes:

The option -DSlicerAutoscoperM_SUPERBUILD=ON should not be set here, this will be integrated in a follow up PR.  If you already have an extension build, set to OFF.

To ensure autoscoper is found, call the following before run the SlicerWithSlicerAutoscoperM.exe launcher
set PATH="C:\path\to\Autoscoper-build\Autoscoper-build\bin\Release;%PATH%"


Fixes the examples for the attribute_generation micro_service

In the example the ordering of target_provider and requester was the reverse from what is used in the code. Although, I believe that the ordering in the examples felt more intuitive, changing the ordering in the code would result to a breaking change for existing deployments.
All Submissions:

[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[X] Have you added an explanation of what problem you are trying to solve with this PR?
[X] Have you added information on what your changes do and why you chose this as your solution?
[ ] Have you written new tests for your changes?
[ ] Does your submission pass tests?
[ ] This project follows PEP8 style guide.


print_floyd's_triangle

📑 Description
copilot:summary
🐞 Related Issue

Closes #
📋 Explanation of changes
copilot:walkthrough
copilot:poem

Since the user in question did not respond or review, I will close this pull request.
It can be reopened upon the user's request.
Note: "user" refers to the commit author.


There was an error running this Docker on the Alpine Linux system, please resolve the issue!

The same Docker image runs successfully on Ubuntu,
but there is an error when running Docker on Alpine Linux system. Please solve the problem!
attached the log information:
✳
Failed to send flush request: Operation not permitted
RTNETLINK answers: Operation not permitted
Error: argument "qemubr1" is wrong: Device does not exist
✳
RTNETLINK answers: Operation not permitted
Cannot find device "qemubr1"
KVM not available, running in emulation mode. This will be slow.
udhcpd: started, v1.36.1
udhcpd: max_leases=235 is too big, setting to 1
udhcpd: can't open '/var/lib/udhcpd/udhcpd.leases': No such file or directory
udhcpd: ioctl 0x8933 failed: No such device
qemu-system-x86_64: -nic tap,id=qemu1,mac=54:05:AB:CD:12:31,script=/routeros/qemu-ifup,downscript=/routeros/qemu-ifdown: could not open /dev/net/tun: No such file or directory
🌏

Hi! I don't think it is possible, AplineOS is a good OS for embedded devices or for docker images, but not for dedicated servers. The warnings that are visible in your logs indicate this, you need to configure BIOS of server (enable KVM) and fix some OS settings (configure /dev/net/tun) before trying to launch the container.
But you may try to run a VDI-image of RouterOS directly via QEMU without help of Docker Engine, details in Dockerfile and entrypoint.sh.


jQuery Application Toastr Release Candidate Updates

Q
A

Bug fix?
No

New feature?
No

New sample?
No

Related issues?
None

What's in this Pull Request?
Updated the sample to take advantage of 1.2.0

@thechriskent,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

thx Chris, appreciated.


type not being recognized

@getfoodiespt please share the schema and error message

Address is a compositve type here, which is only in use for MongoDB: https://www.prisma.io/docs/concepts/components/prisma-client/composite-types


Add default setting for panel location

Feature request
Newest version (0.19) deprecated workbench.panel.location setting. Position now defaults to bottom. Please add setting to decide default panel location.
Since vscode added possibility to move panel to right i have exclusively used it that way and the newest behavior, even though i understand it's intended and it takes only one click to move the panel to desired position, is annoying because the panel pops unexpectedly on the bottom even though i almost never want it there.

This is a fair feature request. We accept PRs that fix this.
Code pointer https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/electron-browser/workbench.ts#L638

Hi @isidorn
I could take a look at this one if no one else is working on it.

@pfongkye great. Make sure the setting is only used as the default value. The value from the storage service should have priority. Due to that I would call this setting "workbench.panel.defaultLocation"

Thanks for accepting this. Much appreciated!

@pfongkye thanks for the PR, I will review in the next couple of days. No tests for this are needed and it is not so easy to test. Thanks for offering though.


UnphysicalSolution exception

A few thoughts:

I have seen suggestions that custom exceptions should be defined in their own file, to make importing them easier. Certainly from r3d2.wave import UnphysicalSolution is not the most obvious, when playing with try/except tricks. However, I don't know how "fixed" this convention is.
Would it be more consistent to have UnphysicalSolutionError to match other exceptions?

Hi both, I hope you don’t mind me giving my two cents.
On Ian’s points:
I would say that 1 is not fixed but it is something I’ve been encouraging in my team as a good practice. I think it depends on the generality of the exception. If it will be raised across many files, I would move it to an exceptions file. If it is very specific and probably only going to raised in one file, then that is probably it’s home. UnphysicalSolution probably falls into the first category as I suspect there are many places in your code where an unphysical solution may occur.
On 2, I would agree that a naming convention is useful. I tend to do something like AppropriateNameException but it seems that AppropriateNameError would be more consistent in this case.
Hope your both well.


Update join.json for Basque translation

Removed link to "joinfirefish.org".

@a-mento Thank you so much!


Status  :- How to change the status of that bot ?

Please help me

Please help me

First I suggest learning discord.js and JavaScript before trying to host a bot and 2nd its in events/client/ready.js

Please help me

Go Here ;)


Improvements

Superficial refactor to some lines to improve readability. Also remove some redundant conditionals like,
def foo(x): #
    if x>0:
        return x
   else:
        return 0

Above snippet could be rewritten as
def foo(x): #
    if x>0:
        return x
    return 0

or,

def foo(x):
    return x if x>0 else 0

to save some bytes.

Hi @marmeladze
Looks good for me, will merge today after routine work hours :)
Thank you for contribution.


Update rayon

This was causing me some... interesting issues when I implemented rayon traits in a library then tried to use them in a quicli-driven tool.

(This is a breaking change.)

cc #58

Included in #78


fix reflection

Hi,
If you add :global-vars {*warn-on-reflection* true} to your project.clj and run lein check you would see a number of reflection warnings, performance hints and other depreciation messages.
This PR fixes the performance issues and prevents reflection in a few hotspots.
Cheers

I was also tempted to upgrade the dependencies, let me know if that's something you would accept/consider.

Any feedback on this or should I release it under our org (we'll need it next by week)?

Thank you Max! Great PR.


Precompiled kernels for DigitalOcean Kubernetes cluster

What would you like to be added:
It would be great if the precompiled kernel modules included DigitalOcean's Kubernetes hosts.
Why is this needed:
To let Falco run on DO's managed Kubernetes infrastructure.
(Originally brought up on Slack, cc @fntlnz)

Hi @cnorthwood and @fntlnz
Thanks for "joining up" and responding to the messages in Slack. It would indeed be great to have pre-compiled Sysdig Probe images on the DO K8s kernel versions!
As per @fntlnz suggestion on Slack I already tried to compile my own probe module and put that as a drop-into ~/.sysdig/ in a custom Docker container on Docker Hub for our company. However, that didn't work well because I built the module on a regular (non-K8s) Droplet and it seems the K8s droplets were running one kernel version behind the ones currently offered by Debian Stretch.
So hopefully the K8s cluster image on DO is updated soon that it's in sync again with Debian Stretch kernel headers from the Debian backports repository. That way a Probe kernel module can be built and offered by Sysdig as precompiled image, or I can add it to my own Docker image of falco.

/area kernel-module

Hi there! I have a follow up on this and running into some problems.
I just spun up regular Debian Stretch image on DigitalOcean and compiled the kernel module. I did this by pulling down the falcosecurity/falco:latest container image and manually running make in the /usr/src/falco-xxx directory. Then I copied the resulting falco-probe.ko to my workstation and put that in a Docker container image on Docker Hub under /root/.sysdig/falco-probe-xxx.
When I use my own Docker container image firmhouse/falco:latest in my Kubernetes cluster, I get an error at the point of loading the module:
insmod: ERROR: could not insert module /root/.sysdig/falco-probe-0.15.3-x86_64-4.19.0-0.bpo.5-amd64-cc14bb08bc0d4bfc47c4f589c06e1935.ko: Invalid module format

I am not sure if it helps, but file falco-probe.ko on my workstation gives me this:
$ file falco-probe.ko
falco-probe.ko: ELF 64-bit LSB relocatable, x86-64, version 1 (SYSV), BuildID[sha1]=cf2164399339248ea926effb8e87ae2b0c88adb6, with debug_info, not stripped

Any ideas what I might be doing wrong?

Not exactly sure what you did wrong, but I tried to recreate and was able to build and insert the module, both on my build droplet and the k8s droplet. I built the module both with the falco container and manually on a Debian Stretch image. The steps are below.
Launch debian 9.7 image.

Install stretch-backports:
vim /etc/apt/sources.list.d/stretch-backports.list
deb http://deb.debian.org/debian stretch-backports main

wget -q https://s3.amazonaws.com/download.draios.com/DRAIOS-GPG-KEY.public
apt-key add DRAIOS-GPG-KEY.public

wget -q https://s3.amazonaws.com/download.draios.com/stable/deb/draios.list
mv draios.list /etc/apt/sources.list.d/draios.list

apt-get update

apt-get install falco

apt-get install -t stretch-backports linux-headers-4.19.0-0.bpo.5-amd64

dkms build -k 4.19.0-0.bpo.5-amd64 falco/0.15.3

md5=$(md5sum /usr/src/linux-headers-4.19.0-0.bpo.5-amd64/.config |cut -d' ' -f1)
mv /var/lib/dkms/falco/0.15.3/4.19.0-0.bpo.5-amd64/x86_64/module/falco-probe.ko ~/falco-probe-0.15.3-$(uname -m)-4.19.0-0.bpo.5-amd64-${md5}.ko

Stick the kernel module in a DO Space under a directory structure of:
/stable/sysdig-probe-binaries

Launch the Falco daemonset with an env variable like:
SYSDIG_PROBE_URL="https://myspacename.myregion.digitaloceanspaces.com"

The falco container will download the module from your Space and insert it. It you'd rather ship the module as a container, I'd suggest shipping your precompiled module as a container and having it launch as an init container that just insmod's the module.
In the meantime we are still going to research precompiling the module and making it publicly available.
 We'd love a PR for that :)

@mfdii Many tanks for this. It worked. I have published my own binary to a DO Space and could succesfully launch the daemonset!

@mfdii By the way. No Cilium messages so far yet in my Falco logs. I'm using the default yaml files from this repo as standard configuration. What messages are you referring to so that I might contribute?

I think that @mfdii wanted to tell that since the DO k8s uses cilium for networking but the cilium binary is not trusted by falco so when using falco on DO what happens is that falco will complain about cilium doing privileged networking stuff.
See: https://github.com/falcosecurity/falco/blob/dev/rules/falco_rules.yaml#L162

@fntlnz Thank you for the explanation. Would it simply be a matter of adding cilium to that list? Or would that be an insecure practice?

@fntlnz @mfdii I'm fine with creating a PR for this! So I'll continue that. For now I'll assume that simply adding Cilium to the kubernetes tools whitelist is sufficient.


[doc] Add Fault Tolerant Documentation Page

What does this PR do?

This PR adds documentation for Fault Tolerant Training.
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Extremely excited, great work @awaelchli @tchaton and others who helped in the effort to add this to Lightning :)


About DSEC results

Hi dear authors,
may I ask where are the results for DSEC-SEG dataset? I didn't see them in the paper

The experimental results of DDD17-SEG and DSEC-SEG were newly added during the paper's rebuttal process. Additionally, we have included these relevant contents in the supplementary materials of the camera-ready version.

Thank you for your previous response regarding the DSEC-SEG results. However, I couldn't find the DSEC-SEG results in the supplementary materials provided at this link.
Could you kindly point me to the correct version or location where the results for the DSEC-SEG dataset are available?
Thanks a lot for your help!

The experimental results of DDD17-SEG and DSEC-SEG were newly added during the paper's rebuttal process. Additionally, we have included these relevant contents in the supplementary materials of the camera-ready version.

We show the results of DDD17-SEG and DSEC-SEG below. Note that the model weights are the same as in the paper.

DDD17-SEG
mP
mR
$\mathrm{mIoU}(\uparrow)$
$\mathrm{aIoU}(\uparrow)$
DSEC-SEG
mP
mR
$\mathrm{mIoU}(\uparrow)$
$\mathrm{aIoU}(\uparrow)$

ETNet [1]
0.42
0.77
0.34
0.50
ETNet [1]
0.47
0.73
0.36
0.56

Ours
$\mathbf{0 . 5 9}$
0.62
$\mathbf{0 . 3 7}$
$\mathbf{0 . 5 1}$
Ours
$\mathbf{0 . 5 0}$
0.72
$\mathbf{0 . 3 8}$
$\mathbf{0 . 5 7}$

[1] Weng W, Zhang Y, Xiong Z. Event-based video reconstruction using transformer[C]//Proceedings of the IEEE/CVF International Conference on Computer Vision. 2021: 2563-2572.

Hi @zhiwen-xdu,
Thanks for presenting this result. I would like to know whether these are the results of instance segmentation or semantic segmentation. It seems that both DDD17 and DSEC datasets only provide semantic masks.


provides CancelRequest method for the PreferredHost RoundTripper to facilitate cancellation

CancelRequest exists to facilitate the cancellation. In general, there are at least three ways of canceling a request by an HTTP client:

Transport.CancelRequest (depreciated)
Request.Cancel
Request.Context (preferred)

While using client-go callers can specify a timeout value that gets passed directly to an http.Client.
The HTTP client cancels requests to the underlying Transport as if the Request's Context ended.
For compatibility, the Client will also use the deprecated CancelRequest method on Transport if found.
New RoundTripper implementations should use the Request's Context for cancellation instead of implementing CancelRequest.
Because this wrapper might be the first or might be actually wrapped with already existing wrappers that already implement CancelRequest we need to simply conform.
See for more details:
https://github.com/kubernetes/kubernetes/blob/442a69c3bdf6fe8e525b05887e57d89db1e2f3a5/staging/src/k8s.io/client-go/transport/transport.go#L257
https://github.com/kubernetes/kubernetes/blob/e29c568c4a9cd45d15665345aa015e21bcff52dd/staging/src/k8s.io/client-go/rest/config.go#L328
https://github.com/kubernetes/kubernetes/blob/3b2746c9ea9e0fa247b01dca27634e509b385eda/staging/src/k8s.io/client-go/transport/round_trippers.go#L302

/assign @sttts @mfojtik

/hold
for manual testing

/hold cancel

/lgtm
/approve


KConsumer::consume method returns, but is not consuming yet

Prerequisites
Put an X between the brackets on this line if you have done all of the following:

[X] Reproduced the problem
[X] Checked the Wiki for common solutions
[X] Checked that your issue isn`t already filed
[X] Checked that there is not already an 7Facette module that provides the described functionality

Describe the bug
final KTableTopicConfig topicConfigEssenceImportEssenceImported = new KTableTopicConfig(kafkaConfig);
topicConfigEssenceImportEssenceImported.setKafkaTopic(KafkaConfig.ESSENCEIMPORT_ESSENCEIMPORTED);
essenceImportEssenceCreatedConsumer = topicConfigEssenceImportEssenceImported.createKConsumer(true);
Thread.sleep(1_000); //the consumer needs about one second to actually consume

To Reproduce
Without the final Thread.sleep(1_000); we would miss the relevant message and thus run into a timeout.
Expected behavior
When a method named consume returns, I expect that the consumer actually consumes, instead of throwing in random periods of waiting trusting that I waited long enough (because I don't want to wait too long and make my tests slow).
Actual behavior
Although terminating, it does not consume yet.
Screenshots
-/-
Desktop (please complete the following information):

OS: Ubuntu 20.04.1 x64

Additional context
Same behavior on the pods.

It is fixed.


Incorrect permissions on /dev/kvm

Windows Build Number
Microsoft Windows [Version 10.0.22000.51]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.10.16
Distro Version
Ubuntu 20.04
Other Software
KVM
Repro Steps
Incorrect permissions on /dev/kvm is preventing startup of Android emulator although the user is a member of the required group.
$ emulator -avd api29
emulator: Android emulator version <IP_ADDRESS> (build_id 7324830) (CL:N/A)
ProbeKVM: This user doesn't have permissions to use KVM (/dev/kvm).
The KVM line in /etc/group is: [kvm:x:108:davisj]

If the current user has KVM permissions,
the KVM line in /etc/group should end with ":" followed by your username.

If we see LINE_NOT_FOUND, the kvm group may need to be created along with permissions:
    sudo groupadd -r kvm
    # Then ensure /lib/udev/rules.d/50-udev-default.rules contains something like:
    # KERNEL=="kvm", GROUP="kvm", MODE="0660"
    # and then run:
    sudo gpasswd -a $USER kvm

If we see kvm:... but no username at the end, running the following command may allow KVM access:
    sudo gpasswd -a $USER kvm

You may need to log out and back in for changes to take effect.

handleCpuAcceleration: feature check for hvf
emulator: ERROR: x86 emulation currently requires hardware acceleration!
CPU acceleration status: This user doesn't have permissions to use KVM (/dev/kvm).
The KVM line in /etc/group is: [kvm:x:108:davisj]

If the current user has KVM permissions,
the KVM line in /etc/group should end with ":" followed by your username.

If we see LINE_NOT_FOUND, the
More info on configuring VM acceleration on Linux:
https://developer.android.com/studio/run/emulator-acceleration#vm-linux
General information on acceleration: https://developer.android.com/studio/run/emulator-acceleration.

Permissions on /dev/kvm is currently (incorrect group and permissions):
$ ls -l /dev/kvm
crw------- 1 root root 10, 232 Jul  5 13:27 /dev/kvm

This should ideally be:
crw-rw---- 1 root kvm 10, 232 Jul  5 13:27 /dev/kvm

Manually changing it helps boot up the emulator. However, the settings are reverted on restart.
sudo chgrp kvm /dev/kvm
sudo chmod g+rw /dev/kvm

Looks like the correct permissions are part of udev defaults but this is not getting applied on WSL:
https://github.com/microsoft/WSL/issues/4193#issuecomment-646991317 (#4193)
KERNEL=="kvm", GROUP="kvm", MODE="0660", OPTIONS+="static_node=kvm"

This is causing issues with React Native development on WSL where the manual step is required on every boot.
#6929
The issue is reproducible on Windows 10 21H1 builds as well.
Expected Behavior
Permissions on /dev/kvm should be as follows:
crw-rw---- 1 root kvm 10, 232 Jul  5 13:27 /dev/kvm

Actual Behavior
Current permissions on /dev/kvm:
crw------- 1 root root 10, 232 Jul  5 13:27 /dev/kvm

Diagnostic Logs
No response

That's expected, WSL2 doesn't have udev.
You can use wsl.conf to change those permissions at boot time with something like this though:
# /etc/wsl.conf
[boot]
command = /bin/bash -c 'chown root:lvm /dev/kvm && chmod 660 /dev/kvm'

Release notes for boot.command: https://docs.microsoft.com/en-us/windows/wsl/release-notes#build-21286

That's expected, WSL2 doesn't have udev.
You can use wsl.conf to change those permissions at boot time with something like this though:
# /etc/wsl.conf
[boot]
command = /bin/bash -c 'chown root:lvm /dev/kvm && chmod 660 /dev/kvm'

For anyone who comes across this, I wanted to add that this works if you change lvm to kvm, which I assume was a typo.

If you still see the incorrect permissions error add your user to the kvm group and restart WSL.
sudo usermod -a -G kvm <username>

great ! works for me!

That's expected, WSL2 doesn't have udev.
You can use wsl.conf to change those permissions at boot time with something like this though:
# /etc/wsl.conf
[boot]
command = /bin/bash -c 'chown root:lvm /dev/kvm && chmod 660 /dev/kvm'

Restart your WSL after. Works like a charm.

That's expected, WSL2 doesn't have udev.
You can use wsl.conf to change those permissions at boot time with something like this though:
# /etc/wsl.conf
[boot]
command = /bin/bash -c 'chown root:lvm /dev/kvm && chmod 660 /dev/kvm'

Restart your WSL after. Works like a charm.

Doesn't work for me.
I Tried

# /etc/wsl.conf
[boot]
command = /bin/bash -c 'sudo chown USER:kvm /dev/kvm && chmod 660 /dev/kvm'

but it didn t work . Does someone has a hint for me ?

Though not a Linux command, wsl.exe --version command from Windows would show some version details without any error or help message.

Here are the corresponding informations :
wsl.exe --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.49
MSRDC version: 1.2.3770
Direct3D version: 1.608.2-61064218
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.19044.2604

This issue seem to be fixed in WSL from Windows Store if systemd is enabled.

Without systemd:

$ ls -l /dev/kvm
crw------- 1 root root 10, 232 Mar 1 00:00 /dev/kvm

With systemd:

$ ls -l /dev/kvm
crw-rw---- 1 root kvm 10, 232 Mar 1 00:00 /dev/kvm

So, you have to enabled systemd in wsl.conf file.

It's pretty strange -- I did make it work by running the android virtual device on my machine before, but after I re-install my OS. I can't seem to make it work again.

This issue seem to be fixed in WSL from Windows Store if systemd is enabled.

Without systemd:

$ ls -l /dev/kvm
crw------- 1 root root 10, 232 Mar 1 00:00 /dev/kvm

With systemd:

$ ls -l /dev/kvm
crw-rw---- 1 root kvm 10, 232 Mar 1 00:00 /dev/kvm

So, you have to enabled systemd in wsl.conf file.

Thanks it works now.


Trigger CI tests after bumping the version and before publishing

We want to run e2e tests for each commit, even for the ones from the CI.
Currently checking for updates, bumping the versions and publishing the extension happen in a single step in a job in a Github Action workflow. (for jobs publish and publish-dev)
(Note: for stable, bumping and publishing happens in memory (most changes are not committed to the repo)
Bumping the versions should be a separate step in the job, triggering the e2e tests and then publishing should happen via another step in the job.

Next steps:
Write down the full solution first, to the exact step as a spec before implementing this.

This only applies to the unstable version:
Step 1: check-update
This returns a boolean output describing if new updates are available or not.
Step 2: Bump versions
This only runs in case the output of check-update is true.
Step 3: Run headless tests
This only runs in case the output of check-update is true.
Step 4: Publish Extension
This only runs in case the output of check-update is true.


Start and end drawer support

This might be insane and out of scope for the drawer but I'm currently abusing it and nesting a left drawer and a right drawer
- drawer drawer-mobile
  ├-- drawer-content
  |    ├-- drawer drawer-mobile drawer-end
  |          ├-- drawer-content
  |          |    ├ // page content
  |          |
  |          ├-- drawer-side
  |
  ├-- drawer-side

context: a side navigation, and an activity timeline on the other side of the screen, both should be collapsible on mobile
However the right one never collapses at mobile. Working around with hidden lg:grid results in an overlay blanking everything out
Is support for this out of scope?

This started working the second I posted this, so no idea what was wrong


Address failing https tests

Sometimes, the tests are failing because the code can't access an HTTPS server (currently based on https://github.com/ArnaudBuchholz/gpf-js which may not be available).
Two solutions:

Find a stable (and always available) https server (https://www.google.com/)
Have a dedicated grunt serve (see https://stackoverflow.com/questions/17509517/how-can-i-configure-yeoman-gruntjs-to-run-as-https)

https://www.akadia.com/services/ssh_test_certificate.html

google :-)