Sync loop, followed by silent quit

Describe the bug
Maestral has been stuck in a loop where it attempts to sync the same set of files, disconnects from Dropbox, reconnects, in a loop, until it eventually silently quits.
Reports & screenshots are linked here: https://github.com/samschott/maestral/issues/808#issuecomment-1398416394
To Reproduce
Open Maestral, leave idle for as long as possible.
Expected behaviour
Sync should complete without any additional data transferred to/from Dropbox.
System:

Maestral version: Latest dev build
Python version: 3.10.9
OS: MacOS Ventura 13.1

I suspect that the "silent quitting" is unrelated to being stuck in a disconnect - reconnect loop.
In principle, any local changes that were successfully uploaded before the connection was lost should not be retried later. From the log output at INFO level, this is however impossible to tell since it does not contain any file names.
You could check two things:

If you installed the CLI tools, type maestral activity in a Terminal. This will show a live view of all files being uploaded and their progress. Do you see any items being retried that previously appeared as completed?
If you increase the log level with maestral log level DEBUG, you will get information about which individual files are uploaded, tracebacks from the connection errors, and more. Note that those logs will personal information such as file names, timestamps for individual changes, etc, so make sure that anything you post here can indeed be shared.

Ah-ha! This is the results of maestral activity, with file names duly blurred out:

Going on Dropbox's website, it also looks like those files are not in there.
I have switched my log to DEBUG, and I get materially this (with file names stripped):
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:53 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:54 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:54 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-22 18:48:54 sync INFO: Syncing ↑ 1/22
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-22 18:48:55 sync INFO: Syncing ↑ 2/22
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-22 18:48:55 sync INFO: Syncing ↑ 3/22
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-22 18:48:55 sync INFO: Syncing ↑ 4/22
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:48:55 dropbox_client DEBUG: Request to files/get_metadata

[snipping mostly the same stuff on a loop up to file 22]

2023-01-22 18:48:59 sync INFO: Syncing ↑ 22/22
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:48:59 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-22 18:49:03 sync INFO: Syncing ↑ 1/55
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/upload_session/start
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Refreshing access token.
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/upload_session/start
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:03 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:04 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox

[more of the same]

2023-01-22 18:49:04 sync INFO: Syncing ↑ 7/55
2023-01-22 18:49:04 sync DEBUG: maestral-upload-pool_31: 238.0% CPU usage - throttling
2023-01-22 18:49:04 dropbox_client DEBUG: Request to files/upload_session/start
2023-01-22 18:49:04 dropbox_client DEBUG: Request to files/upload_session/start
2023-01-22 18:49:04 sync DEBUG: maestral-upload-pool_29: 453.6% CPU usage - throttling
2023-01-22 18:49:04 dropbox_client DEBUG: Request to files/upload_session/start
2023-01-22 18:49:04 sync DEBUG: maestral-upload-pool_25: 492.9% CPU usage - throttling
2023-01-22 18:49:04 dropbox_client DEBUG: Request to files/upload_session/start
2023-01-22 18:49:04 sync DEBUG: maestral-upload-pool_38: 617.5% CPU usage - throttling
2023-01-22 18:49:04 dropbox_client DEBUG: Request to files/get_metadata
2023-01-22 18:49:04 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-22 18:49:04 dropbox_client DEBUG: Request to files/upload_session/start
2023-01-22 18:49:04 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-22 18:49:04 dropbox_client DEBUG: Refreshing access token.

So yes, it appears to be stuck in a loop, and it continues to think that the file content is identical on Dropbox, even though Dropbox doesn't show some of these files as synced.
Once it quits out I'll post whatever it spits back, but maybe this is something?

Uff, this looks really weird. There should be by far not that many "Refreshing access token." and "get_metadata" lines in the logs.
The message "Modification of "[FILE]" detected but file content is the same as on Dropbox" does indicate that the files already exist remotely, Maestral literally queries the Dropbox API for any file at that location and then compares it's content hash with the local content before printing that message.
Could you also check the following:

While this shouldn't theoretically happen, is it possible that you have multiple instances of Maestral running and trying to sync the same folder? If you are using the app bundle, you'll see them in Activity Monitor as "Maestral", otherwise they'll show up as "Python". There should be only two processes: one with an app icon (the GUI) and one without (the sync daemon). If there are more, can you quit all of them and make sure to only start a single instance? They could be running for instance under a different user, so make sure to select "View > All Processes".

While Maestral is not running, run maestral rebuild-index in the Terminal and confirm. This will trigger a complete resync on the next start. You will get "conflicting copies" it files exist both locally and remotely but with different content. You'll need to search for those and decide which version to keep / delete manually.

This is what I see:

I did have a third, but it corresponded to the maestral activity process I had running. Quitting that shows the above.
Rebuilding index right now, will keep you posted.

Alright, finally got it to silently quit. Still got caught in a loop overnight. The final bit of the log is as follows:
2023-01-23 14:43:44 dropbox_client DEBUG: Request to files/get_metadata
2023-01-23 14:43:44 dropbox_client DEBUG: Request to files/get_metadata
2023-01-23 14:43:45 dropbox_client DEBUG: Request to files/get_metadata
2023-01-23 14:43:45 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-23 14:43:45 sync INFO: Syncing ↑ 39/47
2023-01-23 14:43:45 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-23 14:43:45 sync INFO: Syncing ↑ 40/47
2023-01-23 14:43:46 manager INFO: Connecting...
2023-01-23 14:43:47 dropbox_client DEBUG: Request to files/get_metadata
2023-01-23 14:43:48 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-23 14:43:48 sync INFO: Syncing ↑ 41/47
2023-01-23 14:43:49 sync DEBUG: Created "[FILE]" on Dropbox
2023-01-23 14:43:49 sync INFO: Syncing ↑ 42/47
2023-01-23 14:43:49 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-23 14:43:49 sync INFO: Syncing ↑ 43/47
2023-01-23 14:43:49 dropbox_client DEBUG: Request to files/get_metadata
2023-01-23 14:43:50 sync DEBUG: Modification of "[FILE]" detected but file content is the same as on Dropbox
2023-01-23 14:43:50 sync INFO: Syncing ↑ 44/47
2023-01-23 14:43:58 manager INFO: Connected
2023-01-23 14:44:32 sync DEBUG: Created "[FILE]" on Dropbox
2023-01-23 14:44:32 sync INFO: Syncing ↑ 45/47
2023-01-23 14:44:36 manager INFO: Connecting...

This is really hard to debug remotely and from the repeated "Connecting...", I suspect that what you see as a loop really might be repeated connection issues. That being said, do you see the same file name being continuously retried? Or is it getting stuck on different ones?


Update Composer dependencies (2022-06-19-00-04)

Visual regression test passed!
View the visual regression test report


page_id: make PageIdsIterator own the KeyPath

Probably less efficient than storing an iterator over chunks,
because now BitSlice needs to be constructed each time.
But useful to avoid propagating the requirements of dealing
with lifetimes in all structs that will store the iterator

#56
#55  👈
master

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

Join @gabriele-0201 and the rest of your teammates on  Graphite


Example modal dialog is bigger than the actual window height

Describe the bug
The modal component is longer the actual height of the window. The component should adapt to the height of the window and be internally scrollable

Since we are going to use react 16 we can also take advantage of react portals: https://reactjs.org/docs/portals.html

Done via #501


[macOS 12 x86_64] 4.2 beta1+ crashes trying to launch Compatibility mode project due to ANGLE lib not being compatible

Godot version
v4.2.beta1.official[b1371806a]
System information
Godot v4.2.beta1 - macOS 12.7.0 - Compatibility - integrated Intel(R) Iris(TM) Graphics 6100 - Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz (4 Threads)
Issue description
Godot 4.2 crashes when creating or opening a project using Compatibility rendering mode on Intel macOS (12.7):
Godot Engine v4.2.beta1.official.b1371806a - https://godotengine.org
ERROR: Can't create an EGL display.
   at: _get_gldisplay_id (drivers/egl/egl_manager.cpp:60)
ERROR: Condition "gldisplay_id < 0" is true. Returning: ERR_CANT_CREATE
   at: window_create (drivers/egl/egl_manager.cpp:188)
ERROR: Can't create an OpenGL context.
   at: _create_window (platform/macos/display_server_macos.mm:196)
ERROR: Condition "main_window == INVALID_WINDOW_ID" is true.
   at: DisplayServerMacOS (platform/macos/display_server_macos.mm:4470)

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.2.beta1.official (b1371806ad3907c009458ea939bd4b810f9deb21)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] 1   libsystem_platform.dylib            0x00007ff8110f9dfd _sigtramp + 29
[2] 2   ???                                 0x0000000000000001 0x0 + 1
[3] RendererCompositorRD::_create_current()
[4] RendererCompositorRD::_create_current()
[5] RendererCompositorRD::_create_current()
[6] 6   dyld                                0x000000011208a52e start + 462
-- END OF BACKTRACE --
================================================================

The same project opens successfully in Godot 4.1.2. Converting the project to Forward+ also results in a successful launch.
Steps to reproduce
Creating a new project using Compatibility rendering mode from the manager results in the crash.
Minimal reproduction project
project.godot:
; Engine configuration file.
; It's best edited using the editor UI and not directly,
; since the parameters that go here are not all obvious.
;
; Format:
;   [section] ; section goes between []
;   param=value ; assign values to parameters

config_version=5

[application]

config/name="MacOSGL"
config/features=PackedStringArray("4.1", "GL Compatibility")
config/icon="res://icon.svg"

[rendering]

renderer/rendering_method="gl_compatibility"
renderer/rendering_method.mobile="gl_compatibility"

Error persists in 4.2 beta 4, I hoped #83988 could have helped, but I'm getting the same result:
Godot Engine v4.2.beta4.official.93cdacbb0 - https://godotengine.org
ERROR: Can't create an EGL display.
   at: _get_gldisplay_id (drivers/egl/egl_manager.cpp:65)
ERROR: Condition "gldisplay_id < 0" is true. Returning: ERR_CANT_CREATE
   at: window_create (drivers/egl/egl_manager.cpp:193)
ERROR: Can't create an OpenGL context.
   at: _create_window (platform/macos/display_server_macos.mm:196)
ERROR: Condition "main_window == INVALID_WINDOW_ID" is true.
   at: DisplayServerMacOS (platform/macos/display_server_macos.mm:4545)

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.2.beta4.official (93cdacbb0a30f12b2f3f5e8e06b90149deeb554b)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] 1   libsystem_platform.dylib            0x00007ff8110f9dfd _sigtramp + 29
[2] 2   ???                                 0x0000000000000001 0x0 + 1
[3] RendererCompositorRD::_create_current()
[4] RendererCompositorRD::_create_current()
[5] RendererCompositorRD::_create_current()
[6] 6   dyld                                0x000000011b2d252e start + 462
-- END OF BACKTRACE --
================================================================

This is probably the same issue as #83669, the ANGLE libraries might not be compatible with macOS 12.7.
The crash in RendererCompositorRD is a bit weird, I guess we're trying to fall back to Forward+ or Mobile but somehow this logic doesn't work properly.
Does it work if you run the project from the command line with:
path/to/Godot --path path/to/project --rendering-driver opengl3

This should force using the native OpenGL drivers instead of ANGLE (which is controlled by opengl3_angle).
If you can then open the project successfully in the editor, you can change the rendering/gl_compatibility/driver.macos setting to opengl3 to enforce it.

Ah I misinterpreted the versions, I thought this was 10.12, not 12. So it may be a different issue to #83669 in the end.

@aitorciki Could you confirm if #83753 fixes the crash by testing this custom build I made after merging it?
https://downloads.tuxfamily.org/godotengine/testing/Godot_v4.2-beta_8fc215893_macos.universal.zip
I guess this wouldn't strictly fix this issue as we expect ANGLE to work fine on macOS 12.7, so this requires further investigation. But it would be good if OpenGL still works out of the box by falling back to the native driver.

Still the same error with your custom build 😞
Godot Engine v4.2.beta.official.8fc215893 - https://godotengine.org
ERROR: Can't create an EGL display.
   at: _get_gldisplay_id (drivers/egl/egl_manager.cpp:65)
ERROR: Condition "gldisplay_id < 0" is true. Returning: ERR_CANT_CREATE
   at: window_create (drivers/egl/egl_manager.cpp:193)
ERROR: Can't create an OpenGL context.
   at: _create_window (platform/macos/display_server_macos.mm:196)
ERROR: Condition "main_window == INVALID_WINDOW_ID" is true.
   at: DisplayServerMacOS (platform/macos/display_server_macos.mm:4552)

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.2.beta.official (8fc215893af3ea54a8fbd001caee0ce968d206d9)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] 1   libsystem_platform.dylib            0x00007ff8110f9dfd _sigtramp + 29
[2] 2   ???                                 0x0000000000000001 0x0 + 1
[3] RendererCompositorRD::_create_current()
[4] RendererCompositorRD::_create_current()
[5] RendererCompositorRD::_create_current()
[6] 6   dyld                                0x000000011ab1952e start + 462
-- END OF BACKTRACE --
================================================================

I've compiled current master (6afd320984) with and without angle_libs, just to compare to beta and the provided custom build when building on macOS 12 + Intel:

Compiling with angle_libs has the exact same result, Godot crashes not fallbacking to opengl3 printing ERROR: Can't create an EGL display unless opengl3 is forced.
Compiling without angle_libs launches the project successfully with opengl3 by default, and when forced to use ANGLE with --rendering-driver opengl3_angle if will fallback successfully to opengl3 after printing ERROR: Can't load EGL.

@aitorciki Could you see how the current master behaves, after https://github.com/godotengine/godot/pull/84288 was merged?

Looking good! Properly falls back to OpenGL displaying ERROR: Can't create an EGL display. in the terminal 👍

Sweet! Let's assume the crash is fixed with https://github.com/godotengine/godot/pull/84288 which closes the issue. I think there are still some concerns about how ANGLE is behaving, but that should probably be tracked separately.


Manual genre: title page shouldn't show book title twice

Ideally, the title in the header would only appear when scrolling so the title in the text's title page is no longer visible.
Or, because the ToC now includes the book title on the left, we should perhaps do away with the mdbook-inspired title pane at the top of the page entirely.

Closed by the redesign.


[Source block] Verify that navigation arrows appears when there are more than 3 cards.

Priority:
High
Preconditions
1.The 'StreetCode page' is opened.
2. Source cards have been сreated for more than 3 types of sources.
Steps to reproduce

Go to the "Для фанатів" block.
Verify the presence of navigation arrows.
Verify that the right arrow is active for scrolling, while the left arrow is inactive by default.
Scroll through the cards using the right arrow and check if the left arrow becomes active after scrolling through the cards.

Expected result

When there are more than 3 cards, navigation arrows appear.
The right arrow is active by default, allowing scrolling to the right.
After scrolling, the left arrow becomes active, allowing navigation back to the left.

User story:
#41
Test
#65

This test case was closed as it was merged with test case №1193


Adds support for larger iOS device tokens

Addresses issue #265

Changed charfield max_length from 64 to 200
Changed APNSDeviceSerializer validation to check string length for 200
Changes to README and comments

Please fixup all commits into one when you're done.

This will need a migration, by the way.

By fixing up all commits into one you mean to delete this pull request and create just one commit ?

Devs would any way have to migrate when apple makes this change.

you don't need to create a new PR, just force push to this one.

cool.

Doing it now.

@jleclanche Done

You still need to add a migration. I'll try to land this this weekend so it's ready for the .1 release.

Damn! Any tips on how to add migrations ?

@jleclanche do i need to modify the 0001_initial.py ?

No, just create a new migration as you would with makemigrations.

@jleclanche how about now?

Looks good. There's a typo devuce in one of your comments - I'll try merge it tomorrow.

Fixed.


AMQ-9074: javadoc output simplifications

use -noindex to remove various .js files, disable html source listings for javadoc
https://issues.apache.org/jira/browse/AMQ-9074

I prepared a site update with a backported version of this, see PR referenced above.


Explanation to Finding Minimum and Maximum Value in a Binary Search Tree

Description
I would like to add a conceptual question along with explanation, for finding the minimum and maximum value in a Binary Search Tree using recursive methods.
Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python

Are you contributing under any open-source program ?
Hacktoberfest 2021

@girija0707 @siddhi-244 Please consider assigning this to me under hacktoberfest 2021 tag.


Global rate limit infra components

RateLimitInfraIR - holds the config to manage the ratelimit service.
global-ratelimit runner that subscribes to XdsIR messages and translates it to RateLimitInfraIR.
enhance the infrastructure runner to subscribe to RateLimitInfraIR messages and translate it into platform specific ratelimit resources.

Relates to https://github.com/envoyproxy/gateway/issues/670
Signed-off-by: Arko Dasgupta<EMAIL_ADDRESS>

https://github.com/envoyproxy/ratelimit/pull/374 should unblock this PR and we should be able to reuse the config structure from upstream

now blocked on https://github.com/envoyproxy/gateway/pull/726 which updates the Xds IR with a rate limit definiton


Select multiple files using Chooser

How do I select multiple files, similar to the JavaScript Dropbox chooser?

Doesn't seem like it's possible?


Add new chat messages system

Add Discord integration with webhook.
Add Twilio messaging integration.

Closed because problem with --disablediscord and --disabletwilio arguments


Intermittent Unknown Collapse Issue in Repository Section

**I've noticed an issue within the repository section. When attempting to collapse it, sometimes it successfully collapses, but at other times, it unexpectedly navigates to the repository on GitHub.
demo:
https://github.com/lucavallin/first-issue/assets/91373023/feaa4256-d080-42cb-a79f-da88a3312266

@lucavallin should I fix it

@Muhammad-Tayyab-Bhutto Go ahead!

@lucavallin, could you please assign this task to me? It's been a week since the previous assignee didn't provide any code for this issue.

@lucavallin I have solve this issue and raise a new pull request . Please have a look #267


[Feature Request] Mark Frames and Jump To Marked Frames

This is a fantastic tool. Thank you!
The ability to "mark" frames and jump to marked frames could improve the editing experience.
A possible "shortcut" to this feature would be playback keys that jump to or between title frames. Users could then insert a title frame to mark and annotate the preceding or following frame without damaging the image itself (as edits are not reversible). They could then jump between title frames during editing. When they were ready to "publish" the final GIF, the would remove these "marker" title frames (and could keep the "title frame annotated" copy of their file for future edits).

Like Visual Studio bookmarks?


WIP: Fix aes-ctr based drbg's forking behavior

Fixes #12377

Merging this one would only hide a broader fork-safety issue, see https://github.com/openssl/openssl/issues/12377#issuecomment-655416222 . Closing.


Multiple Netatmo Homecoaches

Home Assistant release with the issue:

0.89.1
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Linux, manual install
Component/platform:

https://www.home-assistant.io/components/sensor.netatmo/
Description of problem:
I have two Netatmo Home Coaches, as well as a Weather Station with several modules.
Since the latest update of HomeAssistant, all Weather Station modules as well as one of the two Home Coaches are discovered. Before that, only the Weather Station modules were recognised
However, as said ... only one of the two Home Coaches show up ...
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

It’s worked for only one day then HA discovers only one Home Coach of 2.
Same issue with 0.89.2

I can confirm this issue.
Version 89.2 Hass.io on NUC
Host System:
Hostname | hassio
System | HassOS 2.10
Deployment | production
Hass.io Supervisor
Version | 148
Latest version | 148
Currently have:
1 x Netatmo Weather Station (Outdoor Module, Indoor Module, Rain Gauge)
2 x Netatmo Heathy Home Coach Indoor Modules
Before latest update only the Netatmo Weather Station modules would show.
After update the same modules show and also 1 x Healthy Home Coach.
My second Health Home Coach does not show up. Nothing of interest in logs, no errors.
Also - in addition to this issue, now the component allows (or should allow) multiple healthy home coaches and the indoor weather station modules, we need to get the names from Netatmo for the devices as they all currently show named as 'indoor' so it is not possible to check which is which. They all have names assigned in the Netatmo app.

Today i upgrade HA to 0.90 but only one Netatmo Heathy Home Coach Indoor Modules is found

Just upgraded to 0.90.1 but same behaviour.
Only one netatmo home coach avialable

Same condition with version 0.91.2. How can I help someone to fix the issue?

I have HA 0.91.3.
My netatmo has 2 stations. They do not seem to be auto-discovered:
# This doesnt work, nothing discovered, no errors
sensor:
  - platform: netatmo

I can specify a single platform and then the sensors for that are auto-discovered, however I cannot find a way to specify both netatmo stations (if I give multiple "sensor:" lines, only the last one counts.:
# This one works but only one station available
sensor:
  - platform: netatmo
    station: Home

Same issue.
HA Version 92.2 Hass.io on NUC

arch
x86_64

dev
false

docker
true

hassio
true

os_name
Linux

python_version
3.7.3

timezone
Europe/Madrid

version
0.92.2

virtualenv
false

System | Ubuntu 18.04.2 LTS
Hass.io Supervisor
Version | 162
Currently have:
1 x Netatmo Weather Station (Outdoor Module, 2 Indoor Modules, Rain Gauge, Wind)
3 x Netatmo Heathy Home Coach Indoor Modules
Netatmo Weather Station and all modules found.
Only 1 of the Netatmo Home Coach module shows up.
Txs.

It seems that the netatmo homecoach / weather station related modules are
not maintained.
If this module is indeed orphaned, it would be good to find a new owner.
Might need to polish up some python skills myself ;-)
For the moment, I'm using a netatmo -> fhem (www.fhem.de) -> MQTT ->
homeassistant work around ...
On Mon, 6 May 2019 at 21:03, ccorderod<EMAIL_ADDRESS>wrote:

Same issue.
HA Version 92.2 Hass.io on NUC
arch x86_64
dev false
docker true
hassio true
os_name Linux
python_version 3.7.3
timezone Europe/Madrid
version 0.92.2
virtualenv false
System | Ubuntu 18.04.2 LTS
Hass.io Supervisor
Version | 162
Currently have:
1 x Netatmo Weather Station (Outdoor Module, 2 Indoor Modules, Rain Gauge,
Wind)
3 x Netatmo Heathy Home Coach Indoor Modules
Netatmo Weather Station and all modules found.
Only 1 of the Netatmo Home Coach module shows up.
Txs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/22066#issuecomment-489736568,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBNLFOV3LU74KHDTOVOJE3PUB6HRANCNFSM4G6X3M6A
.

@erikproper #24141 might fix that issue.

@erikproper @ccorderod Please contact me on Discord if you want to provide test data as I don't have access to HomeCoaches or WeatherStations.

@cgtobi discord request sent. thank you.

@ccorderod , @cgtobi  have you guys managed to make any progress?

Was on holidays. Am back now. Would you still need test data?
On Mon, 10 Jun 2019 at 19:51, mdscarrott<EMAIL_ADDRESS>wrote:

@ccorderod https://github.com/ccorderod , @cgtobi
https://github.com/cgtobi have you guys managed to make any progress?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/22066?email_source=notifications&email_token=ABBNLFKR24XLCRHREMZL6MTPZ2ICDA5CNFSM4G6X3M6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXKTXYI#issuecomment-500513761,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBNLFO7HNTCBIH43TUUTVDPZ2ICDANCNFSM4G6X3M6A
.

I think I'm fine right now since @ccorderod provided a pretty big setup. I'm still working trough it.

@erikproper  @mdscarrott your problem maybe is solved by https://github.com/home-assistant/home-assistant/pull/24141 added in 0.94.
Do you can test please .

I installed hass.io 0.94.3 and I have again the same behavior.

Same here. Only seeing one Homecoach of the two I have.
On Fri, 14 Jun 2019 at 07:54, monsterciattolo<EMAIL_ADDRESS>wrote:

I installed hass.io 0.94.3 and I have again the same behavior.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/22066?email_source=notifications&email_token=ABBNLFPBHJBRYXJT4ECDRS3P2MW77A5CNFSM4G6X3M6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXVZGSI#issuecomment-501977929,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBNLFIXD5LKHIIKG4XTSTTP2MW77ANCNFSM4G6X3M6A
.

If the homecoache conf is same than weather station, you must have discovery=false and you must configured tour sensor @erikproper @monsterciattomo

@maheus Not sure I understand what you mean
On Fri, 14 Jun 2019 at 08:46, maheus<EMAIL_ADDRESS>wrote:

If the homecoache conf is same than weather station, you must have
discovery=false and you must configured tour sensor @erikproper
https://github.com/erikproper @monsterciattomo
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/22066?email_source=notifications&email_token=ABBNLFMDFKYC3RFTEGMBXXTP2M5EBA5CNFSM4G6X3M6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXV4DPQ#issuecomment-501989822,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBNLFLPGXYXRXVWRGXSFQDP2M5EBANCNFSM4G6X3M6A
.

sorry @erikproper  if i'm not clear. For me, if you have multiple weather stations or homecoaches .
you must have in your conf:
configuration.yaml
netatmo:
  api_key: !secret XXXX
  secret_key: !secret YYYY
  username: !secret ZZZZ
  password: !secret WWWW
  discovery: false

and
sensor:
  - platform: netatmo
    station: rdc
    modules:
      indoor:
        - temperature
  - platform: netatmo
    station: etage
    modules:
      indoor:
        - temperature

this is an example for my weather station but i suppose that it is same for homecoaches

Tried that already in the past. The HomeCoaches don't have modules.
However, I tried playing with station/module names, but to no avail.
Tried it again today, on the latest HA version. Also didn't work.
On Fri, 14 Jun 2019 at 09:46, maheus<EMAIL_ADDRESS>wrote:

sorry @erikproper https://github.com/erikproper if i'm not clear. For
me, if you have multiple weather stations or homecoaches .
you must have in your conf:
configuration.yaml
netatmo:
api_key: !secret XXXX
secret_key: !secret YYYY
username: !secret ZZZZ
password: !secret WWWW
discovery: false
and
sensor:

platform: netatmo
station: rdc
modules:
indoor:
- temperature
platform: netatmo
station: etage
modules:
indoor:
- temperature

this is an example for my weather station but i suppose that it is same
for homecoaches
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/22066?email_source=notifications&email_token=ABBNLFIPXQHOLVWDSS2NZWTP2NEEVA5CNFSM4G6X3M6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXWAAUI#issuecomment-502005841,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBNLFLPYTXOU6CGQY74SULP2NEEVANCNFSM4G6X3M6A
.

ok thanks @erikproper . I don't have a HomeCoaches so it is difficult to debug.
i think to have found the bug, in smart_home/WeatherStation.py
            for m in self.rawData[i]["modules"]:
                if "module_name" not in m:
                    if m["type"] == "NHC":
                        m["module_name"] = m["station_name"]

i suppose that NHC is for HomeCoaches , but module_name and station_name doesn't exist , Do you can test in replacinig m["module_name"] = m["station_name"] with m["module_name"] = m["name"]
according to the documentation https://dev.netatmo.com/en-US/resources/technical/reference/aircare/gethomecoachsdata, name is "Name the user gave their Healthy Home Coach"

ok thanks @erikproper . I don't have a HomeCoaches so it is difficult to debug.
i think to have found the bug, in smart_home/WeatherStation.py
            for m in self.rawData[i]["modules"]:
                if "module_name" not in m:
                    if m["type"] == "NHC":
                        m["module_name"] = m["station_name"]

i suppose that NHC is for HomeCoaches , but module_name and station_name doesn't exist , Do you can test in replacinig m["module_name"] = m["station_name"] with m["module_name"] = m["name"]
according to the documentation https://dev.netatmo.com/en-US/resources/technical/reference/aircare/gethomecoachsdata, name is "Name the user gave their Healthy Home Coach"

Good job @maheus ! do you think you will push to fix in home-assistant code?

i did other test on netatmo web site.
this is the information of missed NHC.
body:
▼devices:
▼
_id: "70:ee:50:26:12:a2"
cipher_id: "enc:16:EQq7JC2BA12+3NFwP8/XKYN/cYVzS5VEehaHYnLSZvntKMrU3dvG+Gjk4pQX5jV+"
date_setup:<PHONE_NUMBER>
last_setup:<PHONE_NUMBER>
type: "NHC"
last_status_store:<PHONE_NUMBER>
firmware: 45
last_upgrade:<PHONE_NUMBER>
wifi_status: 58
reachable: true
co2_calibrating: false
station_name: "Sala"
▼data_type:
"Temperature"
"CO2"
"Humidity"
"Noise"
"Pressure"
"health_idx"
►place:[object Object]
►dashboard_data:[object Object]
►user:[object Object]
status: "ok"
time_exec: 0.05323314666748
time_server:<PHONE_NUMBER>

Hi. I tried to fix the code a few months ago. The problem is way bigger. @cgtobi is working on it. Give him some time to fix it.

@monsterciattolo do you can test with this config: (replace monitoring conditions if necessary)
sensor:
  - platform: netatmo
    station: Sala
    modules:
      Sala:
        - temperature
  - platform: netatmo
    station: Camera da letto
    modules:
      Camera da letto:
        - temperature
```

and check log response.
I dumped the result of your api export and i transformed it in json for my tests and with this config example i have  good results

Could someone please test if this fixes the homecoach situation as well?
https://github.com/home-assistant/home-assistant/pull/24788

@monsterciattolo do you can test with this config: (replace monitoring conditions if necessary)
sensor:
  - platform: netatmo
    station: Sala
    modules:
      Sala:
        - temperature
  - platform: netatmo
    station: Camera da letto
    modules:
      Camera da letto:
        - temperature

and check log response.
I dumped the result of your api export and i transformed it in json for my tests and with this config example i have good results

I upgraded hass.io to 0.95.3 and i followed your guide.
Both station "Camera da letto" and "Sala" are now on home-assistant!!!!

I followed the @maheus's configuration and i upgraded hass.io to 0.95.3.

platform: netatmo
station: Sala
modules:
Sala:
- temperature
- co2
- noise
- humidity
- wifi_status
- health_idx
- pressure
platform: netatmo
station: Camera da letto
modules:
Camera da letto:

temperature
co2
noise
humidity
wifi_status
health_idx
pressure

I'm doing some test and seems both home coach stations are recognized!!!

Cool , i think that i will does update documentation

Hi all. I  updated to  0.94 and tried the code listed above using the name of my NHCs as per the HC app and  i still only see the 1 indoor module which is part of my Netatmo Weather Station and 1 Netatmo Home Coach. The second home coach still nowhere to be seen.
Code as i used it was:
`sensor:

platform: netatmo
station: Bedroom
modules:
Bedroom:
- temperature
- co2
- noise
- humidity
- wifi_status
- health_idx
- pressure
platform: netatmo
station: Storeroom
modules:
Storeroom:
- temperature
- co2
- noise
- humidity
- wifi_status
- health_idx
- pressure`

Yeah, it’s not fix. I’m working on it.

@mdscarrott, do you can test in adding the weatherstation conf with the nhc conf .

Okay - I have added in the weather station as requested by @maheus  as follows with the same result. Still only one Home Coach showing up with the weather station modules.
Thanks for your reply @cgtobi and your hard work
sensor:
  - platform: netatmo
    station: Bedroom
    modules:
      Bedroom:
        - temperature
        - co2
        - noise
        - humidity
        - wifi_status
        - health_idx
        - pressure
  - platform: netatmo
    station: Storeroom
    modules:
      Storeroom:
        - temperature
        - co2
        - noise
        - humidity
        - wifi_status
        - health_idx
        - pressure
  - platform: netatmo
    station: Hill Farm
    modules:
      Indoor:
        - temperature
        - co2
        - noise
        - humidity
        - wifi_status
        - health_idx
        - pressure
      Outdoor module:
        - temperature
        - humidity
        - wifi_status
        - health_idx
        - pressure
        - battery_vp
        - battery_percent
      Rain Gauge:
        - sum_rain_1
        - sum_rain_24
        - rain
        - wifi_status
        - battery_vp
        - battery_percent

The issue is caused by the upstream python module and has been fixed. I'll let you know once its new release is out.

For those willing to test, replace  smart_home/WeatherStation.py with https://github.com/cgtobi/netatmo-api-python/blob/homecoach_fix/smart_home/WeatherStation.py and let me know if you get any errors.

I would be happy to test but there isn't a smarthome folder in /usr/local/lib/python3.7/site-packages/ on my hass.io.
Home Assistant 95.4
Hass IO Supervisor 167
HassOS 2.12

@mdscarrott if you already have Netatmo configured there should be pyatmo.py and smart_home/ in your python site-packages folder. But you probably can just wait for 0.96 beta which should ship tomorrow.

@mdscarrott the beta is out, feel free to test.

I installed on bass.io the beta 0.96.0b0.
I had to change the sensor configuration because it was incorrect.
I simply leave platform:netatmo and seem work fine but my setup worked also with 0.95.4.
Both HCM were recognized fine

Yeah, there was a breaking change regarding the config.

Hi @cgtobi
I cant find a changelog for 0.96.0b1 to understand what the breaking change is but if i remove the config and only include platform:netatmo i only get 1 Home Coach, 1 Weather Station Rain, 1 Weather Station Outdoor and 1 Weather Station Indoor. I am missing 1 Home Coach still.

Hi @mdscarrott

Please log into the Netatmo developer page.
Then go to Netatmo Energy - Homesdata, click that yellow arrow and hit try it.

Check the results and maybe send me the results via Discord/email/HA-forum

Not sure if I should be posting here but I'm running 0.100.2 and I have 2x Netatmo Home Coach stations but I can see only one in Home Assistant (sensor.netatmo_indoor_*_2). Pyatmo is giving both though:
`python3 gen_test_data.py --CLIENT_ID xxx --CLIENT_SECRET xxx --USERNAME xxx --PASSWORD xxx --mode=long --modules HomeCoach
Checking for available home coach stations...
Parents Bedroom (70:ee:50:3e:c5:46)
Modules ['Indoor']
Baby Bedroom (70:ee:50:26:68:92)
Modules ['Indoor']
HomeCoach [OK]`
I also noticed that the ID of entities is left to some defaults "sensor.netatmo_indoor_*" for both Weather Station and Home Coach Station despite them having names in my setup (Living Room, Parents Bedroom, Baby Bedroom). It seems to work fine for interior and exterior modules though. The friendly names are set to the right values but I override them to save space in the UI.
`python3 gen_test_data.py --CLIENT_ID xxx --CLIENT_SECRET xxx --USERNAME xxx --PASSWORD xxx --mode=long --modules WeatherStation
Checking for available weather stations...
Villa Astoria (70:ee:50:2c:9d:a4)
Modules ['Outside', 'Living Room', 'Bathroom']
Villa Astoria - Outside
Temperature - 11.6
Humidity - 79
min_temp - 11.5
max_temp - 15
temp_trend - stable
rf_status - 69
battery_vp - 4978
battery_percent - 57
reachable - True
Villa Astoria - Living Room
Temperature - 22.4
CO2 - 716
Humidity - 59
Noise - 38
Pressure - 1002.3
AbsolutePressure - 992.1
min_temp - 22.4
max_temp - 22.9
temp_trend - stable
pressure_trend - down
wifi_status - 57
reachable - True
Villa Astoria - Bathroom
Temperature - 21.8
CO2 - 693
Humidity - 65
min_temp - 21.8
max_temp - 22.5
temp_trend - stable
rf_status - 66
battery_vp - 6170
battery_percent - 100
reachable - True
WeatherStation [OK]
`

I don't have any specific in the "configuration.yaml" besides the credentials and sensor platform.
I can open a separate issue/feature request for this if needed ;-)
Cheers,
G.

A separate issue would be appreciated. Thanks for reporting. I'll be looking into it.

No worries at all, here you go https://github.com/home-assistant/home-assistant/issues/27845  ;-)


Update sbt-kanela-runner to 2.0.11

Updates io.kamon:sbt-kanela-runner from 2.0.10 to 2.0.11.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.kamon", artifactId = "sbt-kanela-runner" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #270.


test: fixing broken sample-dapp tests

Fixing broke sample dapp tests

#9597  👈
master

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

Join @benesjan and the rest of your teammates on  Graphite


Test and migrate GitLab example to 1.9.

I'm working on this.

Thanks for taking this example on!
I've noticed there hasn't been a pull request submitted yet, do you need any help, or just need some more time? If you ran into a problem, updating this issue to explain would be a huge help so we can try to assist you.
If something came up and you aren't able to complete updating this example, no problem! Just let us know that so we can assign it to another community member. We'll look forward to working with you next time :)

Hi!
I ran dcos 1.9 with vagrant and I followed the example of the 1.8 version (configured Jenkins, configured marathon-lb, configured gitlab).
Everything seems ok... but RAM is over :(
It looks like 16gb of ram is not enough (and I can't upgrade anymore).
I think I'm about 60% of the sample (I managed to create the project on gitlab with difficulty). :(

Thanks for the update, I'll have a look at the resources required here.
If you are able to complete this, for the PR you can copy (not rename or move) some of the materials from the 1.8 example, but screenshots and some other things may need to be updated. We want to keep the 1.8 example in the repository for folks who are still using the older version.

Update to 1.9 proposed in #183


Time to stop describing youtube-dl as "small"?

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.06.16. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.06.16

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[x] Other

Description of your issue, suggested solution and other information
Is it time to stop describing youtube-dl as "small"? I counted the current version with cloc, as installed from the webupd8 PPA, and it comes to a shade under 80kLOC in just under 650 files. I don't think that qualifies as small, either in lines of code or in complexity of installation (e.g. cloc itself is about 10kLOC, but ships as a single Perl file, so it's sort of small).
Suggested change: remove the word "small" from the project description on GitHub, from https://rg3.github.io/youtube-dl/ and from any other corresponding places. I note that the README in particular does not seem (any more?) to describe youtube-dl as "small".
youtube-dl is great; thanks for working on it!

Good idea!

Good point. On http://rg3.github.io/youtube-dl/ it says that

youtube-dl is a small command-line program to download videos from YouTube.com and a few more sites.

Maybe change 'few' to 'many' or something.

change small to awesome  ;-)


add one data set group per doi

Let's add a group.py with the corresponding full group of data sets into each directory and let s reference this in__init__.py of that directory so that they can be accessed in a streamlined fashion.

Would be nice if you could check this solution @le-ander, just so that we do not miss a less repititive solution that does not require a copy of group.py in every folder.

Would be nice if you could check this solution @le-ander, just so that we do not miss a less repititive solution that does not require a copy of group.py in every folder.

We could probably achieve something similar to what is in the PR if we just set a constant in __init__.py as __file__and then use these constants as directory identifiers in a single file, that would much leaner and probably close to minimally reptitive as we would only have one line FILE_PATH = __file__ in each __init__.py.

We could probably achieve something similar to what is in the PR if we just set a constant in __init__.py as __file__and then use these constants as directory identifiers in a single file, that would much leaner and probably close to minimally reptitive as we would only have one line FILE_PATH = __file__ in each __init__.py.


Please don't download binary into user's home directory in build.rs

Here: https://github.com/plotly/plotly.rs/blob/main/plotly_kaleido/build.rs
I think this is unhygienic. It is surprise to me that building something would add files to be added to my home directory. This also breaks when building in a sandboxed environment (which is how I found out).
Plus, it is downloading something that is not strictly needed for the build process. And it is also not going to achieve what I assume is your goal. What if I build on one machine then ship the binary to run somewhere else?

Valid point. This has come up before also in mistral-rs
Will look into it when I have more time available.
Just as a side note, this specific issues is in context of the kaleido feature. If you do not use it, my advice is to build without it. Building without it avoids all the issues you have mentioned.
I am also not a big fan of kaleido to be honest since as you highlight, it introduces an external dependency. I don't use it in my own projects for exactly that reason. Its features are something I miss in plotly itself and still hope for the ppl at plotly to come up with a better solution for that.

@yshui
I have changes the way Kaleido is downloaded/installed at compile time.
I introduced a new feature flag kaleido_download that allows for opt-in automatic download and install of Kaleido at build time. If only the kaleido feature is enabled, the user is required to manually install Kaleido and provide the path to the installation directory using the environment variable KALEIDO_PATH.
I have not changed the location where Kaleido is installed when kaleido_feature is enabled for a couple of reasons.

on Linux the preferred locations for installing external stuff would have been /opt or /use/local , both requiring sudo privileges. That would have been ugly in an build.rs script IMO.
also on most Linux-es, an option would have been ~/bin, however that is no different in my perspective than ~/.config
Hence I decided to leave it as is. Whomever dislikes the path, can decide to install Kaleido manually and use the environment variable approach.

Currently this new setup is available on the main branch. I plan to make a new release in the upcoming weeks.


QCoDeS: Keithley System Switch Instrument Driver

This PR introduces the instrument driver for Keithley System Switch. While the main driver is ready and in working condition (parties who are in urgent need for a driver can in fact make use of the current version), followings should be done before general review process and hopefully merging:

[x] Add general validator functions and exception handling
[x] Add tests
[x] Add example notebook

@jenshnielsen @Dominik-Vogel @astafan8

There are some necessary and unnecessary repetitions in the code concerning exception handling. I will reduce them while finalizing.

@astafan8 @GateBuilder this has been in PR review since Jan 3rd. Anything we can do to support turnaround?

Pipelines passed but the update did not propagate back due to a server side issue at github https://dev.azure.com/ms-quantum-public/Microsoft Quantum (public)/_build/results?buildId=8376&view=results


Independent api server

Made the API server independent from the energy system interface. closes #72
TODO:

where to start the new API server? in the create() of energy system interface? -> init of silinterface
adjust create() of energy system interface to PUT initial values of the sim; how? -> resolved
how to realise logging behaviour in API server, see #62 ? currently only temp solution implemented -> with iso timestamps

closes #62

closes #63


Cover AWS region finder with unit tests

Coverage increased (+0.6%) to 59.985% when pulling 9e0db5b4701332c562bbf79a73a5e88f33d7a7e6 on stgleb:region_finder_aws into aab70feb6195704f0b3e68737f00ef3034625537 on supergiant:master.


Automated cherry pick of #9266: fix(region): openstack available zones

Cherry pick of #9266 on release/3.6.
#9266: fix(region): openstack available zones

Codecov Report

Merging #9276 (7abd40c) into release/3.6 (b9d30c0) will increase coverage by 10.98%.
The diff coverage is 8.77%.

@@               Coverage Diff                @@
##           release/3.6    #9276       +/-   ##
================================================
+ Coverage         5.86%   16.85%   +10.98%
================================================
  Files              828       85      -743
  Lines           129133     8900   -120233
================================================
- Hits              7577     1500     -6077
+ Misses          120876     7268   -113608
+ Partials           680      132      -548

Flag
Coverage Δ

aFlag
16.85% <8.77%> (+10.98%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/monitor/alerting/conditions/nodataquery.go
0.99% <0.00%> (ø)

pkg/monitor/alerting/conditions/suggestquery.go
2.00% <2.00%> (ø)

pkg/mcclient/modules/monitor/suggestsysrule.go
100.00% <100.00%> (ø)

pkg/mcclient/options/dns_trafficpolicy.go

pkg/multicloud/aws/loadbalancerbackend.go

pkg/hostman/monitor/monitor.go

pkg/util/imagetools/imagetools.go

pkg/lbagent/models/agentparams.go

pkg/multicloud/ucloud/snapshot.go

... and 666 more

Continue to review full report at Codecov.

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


Category mapper lower to llvm part2 (test)

This PR implements the support to lower the CategoryMapper operator from the Krnl dialect to the LLVM dialect.
It also add LIT tests to verify that the operator is lowered to LLVM as expected.

@jenkins-droid test this please

@jenkins-droid test this please

Jenkins Linux amd64 Build #2899 [push] Lower CategoryMapper to ... started at 08:26

Jenkins Linux ppc64le Build #2006 [push] Lower CategoryMapper to ... started at 09:30

Jenkins Linux ppc64le Build #2006 [push] Lower CategoryMapper to ... failed after 17 min

Jenkins Linux amd64 Build #2899 [push] Lower CategoryMapper to ... passed after 51 min


issue39_path_separation

DPATH is now two separate variables:
FLOWPATH - the location of the ingested netflow data
DNSPATH - the location of the ingested DNS data

Needed to merge DNS analysis into 1.0.1

This is not scalable.  The $DSOURCE variable is passed in via ml_ops.sh or run_oa.sh, and it builds based on the data source.
I believe people are not observing that $DSOURCES and $DSOURCE are two separate variables that perform different functions, and are documented as such

"not scalable" - in what way? as more input sources are added?
But the important thing is to solve Ricardo's problem,  where the flow and DNS are kept in two completely different stores... https://github.com/Open-Network-Insight/open-network-insight/issues/39
anyway, the current documentation at https://github.com/Open-Network-Insight/open-network-insight/wiki/Edit Solution Configuration identifies DPATH as the path to the flow records, not a common store containing both flow and DNS data
anyway, important thing is to get system working now and DNS and flow are not in a common store

I thought this was a separate issue.  I thought this had already been merged when we discussed some time ago.
OK to merge.

The better approach would be to leave one variable as DPATH, then add the temporary path as DNSPATH.  That way it can be removed in 1.0.1 with minimal disruption.

Ricardo has tested the code and it works.

Regarding DPATH versus FLOW_PATH:  FLOW_PATH is a far more meaningful name.


Add Support for  Rocky8

Run tests on Rocky 8
Limit RedHat 7 version to last version available for release
Set RedHat 8 version for tests
This is not attempting to update the tests for newer versions of GitLab

Rebased against master.
In it's current state the role does not test properly on EL8 distributions as it tries to install a GitLab version only available on EL7


Add imagesetinterpolation support on GD implementation

refs https://github.com/Intervention/image/issues/504#issuecomment-724975605

I'm curious how this plays out with the FitCommand from Imagick.

I'll happily look into this. I first wanted to touch base with you if you agree to this contribution ;-)

touch base with you

Achtung: I'm not even a contributor, just a user of this lib ❗

Spare me your sarcasm. My spirit is that you sir can use everything I have developed as is free of charge, but you cannot expect anything more.

Your are right. I don't expect anything anymore.


Real-time HRV analysis and LSL

Hi Marcus - Thanks for making this tool available on Matlab. I'm hoping you might be able to answer two questions or offer some advice.
(1) Is it possible to do real-time streaming hrv analysis (LF vs HF power) using your toolbox?
(2) I'm currently working with a Polar H10 that's feeding data into an LSL stream. Any thoughts on integrating LSL formats into the toolbox?
Thanks!

Hi,
Thanks for the questions.
(1) Yes and no. No, because the application does not support real-time streamings. The resulting waveform need therefore be loaded manually. Yes, because the Matlab class HRV.m does contain functions that are fast enough to support real-time analysis. Every time a heart beat is detected you can call the functions HRV.fft_val_fun(RR,1000) or HRV.fft_val(RR,60,1000) to compute the spectral measures from the interbeat-intervals, as documented here:
HRV.m line 775
(2) I don't have experience with this streaming layer. But the use in Matlab seems possible, see liblsl-Matlab
Unfortunately, I have currently no time to adapt the code for necessary modifications on HRVTool to allow the import of real-time streaming data. Any supporting help is welcome.
Best,
Marcus

Thanks Marcus, very helpful!  I'm trying to do this in real-time using python, but I'll look at the references you mentioned. As for LSL, I can definitely capture the ecg data in Matlab using LSL -> Labrecorder, but it saves as an XDF file, not any of the files that HRV.m seems to recognize.

Hi,
If you are able to save XDF files, I could write a function that does the import of such files in HRVTool. To do so, it would be nice if you can check whether the specifications of the format are matching with the information given here: the OpenXDF_Specification
A sample file would be helpful for me set up the import function.


docs: enhance styles section (fixes: #1907)

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests

Description
fixes #1907
Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

This is looking great! we will be making more improvements here once we get more integrations
We can add notes about tailwinds and vanilla-extract!
We can merge this!


Update go clients

Description
Update gocsi and gopowerstore
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/243

Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have not allowed coverage numbers to degenerate
[x] I have maintained at least 90% code coverage
[ ] I have inspected the Grafana dashboards to verify the data is displayed properly
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have maintained backward compatibility

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[x] Unit test

Manual inspection of the GUI
I have verified that the dashboards show the data properly while generating I/O and storage resources

[ ] Yes
[x] No

e2e pass

run e2e test

run e2e test

run e2e test

run e2e test

e2e pass


ref site doesnt work for almost a month

Challenge No repeats please has an issue.
The ref site doesnt work for almost a month
https://www.mathsisfun.com/combinatorics/combinations-permutations.html
I advise changing it to khanacademy video tut and wikipedia

Hi @tsalexey544 , are you unable to load the site https://www.mathsisfun.com/combinatorics/combinations-permutations.html ? Its loading fine for me.

hi @ajain17, yes. I am located in russia. When I open it I get

Error 522. The initial connection between Cloudflare's network and the origin web server timed out. As a result, the web page can not be displayed.

I am able to access it. Try deleting your cookies and clearing your cache and see if that helps.

@tsalexey544 I'd recomend you to search for the terms "Permutation" and "Combination" - there might be other resources that could help you. If not, feel free to ask in any of our chat rooms or in our forum.
Good luck, and happy coding! :blush:
Closing this as not related to FCC. Thanks @ajain17 for the triage :ok_hand:


Fixing specific HTML syntax errors

It is now valid HTML5 according to http://validator.w3.org/check

Great, thanks :+1:


When to use the long-term memory

Hello, thanks for sharing your great work! I checked the code in the part of trainer.py and did not include " load long-term memory'. Besides, I found long-term memory in the memory_manager.py. Does it mean the training stage only uses working memory? and long-term memory only used in the test/eval stage?

Yes; there are no trainable parameters in the long-term memory

Yes; there are no trainable parameters in the long-term memory

Thanks for your kind reply. When to engage the long-term memory during eval stage ? In the default config to evaluate DAVIS17, I  noticed the long-term would been engaged  when the number of frames in working memory is 9. Is it an initialization setting ?


Turn on dask-expr switch

[ ] Closes #xxxx
[ ] Tests added / passed
[ ] Passes pre-commit run --all-files

cc @rjzamora for gpu ci

Yes, correct, hopefully we can remove pandas 1.x support soon.
Joris Van den Bossche @.***> schrieb am Mo. 11. März
2024 um 08:57:

@.**** commented on this pull request.
In dask/dask.yaml
https://github.com/dask/dask/pull/10967#discussion_r1519277945:

query-planning: false  # Whether to use dask-expr

query-planning: null  # Whether to use dask-expr

Ah, yes, so the None is translated into False if pandas < 2 is installed.
That's something I need to check downstream as well then to determine if
query planning is on or not.
Maybe I can just copy the definition of _dask_expr_enabled for now
—
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/pull/10967#discussion_r1519277945, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOYQZGFSYJLLQ3VXHOQVIX3YXVPYBAVCNFSM6AAAAABD64NIFGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSMRXGEZDONBVGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


The size queried by show data is not  correctly

Steps to reproduce the behavior (Required)

CREATE TABLE t1
load data
show data from t1

Expected behavior (Required)
exact size
Real behavior (Required)
wrong size
StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

mysql> select current_version();
+------------------------------------------+
| current_version()                        |
+------------------------------------------+
| PIPELINE_SQLANCER_MASTER_RELEASE a87e76c |
+------------------------------------------+

This may not a problem.It may be that your data is particularly easy to compress.
You can use some of the following commands to help you identify the problem.
show partitions from $table
show tablet from $table


Consider not filtering .byte, .zero etc inside "used" code blocks

E.g. in https://godbolt.org/g/eVf1KE (with reference too #192 and @pcordes) it would probably be helpful to see the variable definitions (the .zero 1).

Usually the variable defs aren't filtered.  e.g. int foo = sizeof(something) works, and is quicker to type and has more compact output than int foo(){return sizeof(something); }.

I think it's just the .zero that gets filtered. Will take a look.
On Fri, Nov 25, 2016 at 9:35 PM, Peter Cordes<EMAIL_ADDRESS>wrote:

Usually the variable defs aren't filtered. e.g. int foo =
sizeof(something) works, and is quicker to type and has more compact
output than int foo(){return sizeof(something); }.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mattgodbolt/gcc-explorer/issues/193#issuecomment-263041919,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmsdcCVzPNMdm_lMkUggiOAsmPqto4Jks5rB6j5gaJpZM4K75-m
.

I was wrong; my simplistic way of determining unused labels and the data defined at those labels is at fault. I currently filter any data defined at labels that aren't directly used. I recently updated things to also follow dependency chains of labels; but there's a hole in the logic for resurrecting data definitions that become used as a result of other definitions being reachable.
There might be a quick fix; or else I might have to bite the bullet somewhat and do proper CFG-style stuff with proper "unreachable" code and definition detection.

Actually in this case, the issue is the unmangled names of guard variables that are the problem. They have spaces in them (guard variable for asdf<float>) -- which my primitive code can't understand. If I consider it to be a label name, then the "usage" detector (also a very simplistic piece of code) doesn't pick up the label as being used.
In general it seems like the code analysis/demangling needs to be thought about some more: should demangling be done post label finding/dead code detection, perhaps?

my simplistic way of determining unused labels and the data defined at those labels is at fault.

That fits with what I was seeing.  .zero does show up normally.  It was only with this guard-variable stuff that it and other stuff was being hidden.
It hasn't usually been a problem, at least for me.  But it looks like it could catch people by surprise, esp. if they don't realize the the filtering isn't guaranteed to be robust, and don't consider that as an explanation for what they're seeing.
I haven't looked at the code, but demangling as late as possible seems sensible.  Then you get to make lots of assumptions about symbol names, since they conform to a bunch of rules that demangled names don't.

Hopefully fixed (in master) as part of a whole bunch of refactorings. Will be a little while before it makes it to beta, as I need to do some tests of the refactoring to ensure I haven't broken other things. Will keep this open until we can test it on the live site with the links you have.

Fix deployed now; hopefully the output of the filterer is much better. Don't hesitate to reopen or add new issues if you find them!

@mattgodbolt: This is issue (or a related one) is back.
https://godbolt.org/g/ovL706
struct foo {
    void *p;
    int i;
};

int a = _Alignof(struct foo);
int s = sizeof(struct foo);

produces asm output of
a:

s:

with gcc, clang, and icc, default filtering settings.  (MSVC doesn't grok C11 _Alignof).
To find what I was looking for, I had to wade through
        .data
        .globl  a
        .p2align        2
a:
        .long   8                       # 0x8
        .size   a, 4

        .type   s,@object               # @s
        .globl  s
        .p2align        2
s:
        .long   16                      # 0x10
        .size   s, 4

Thanks for the heads up @pcordes. I'll see what I can do; it's hard to distinguish useful global variables defined from all the cruft that often comes from including header files.

I think I'd rather see the cruft from <stdio.h> or <iostream> than miss any of my own globals, as long as the cruft is at the end of listing.  I guess it's a little distracting, so ideally it would be nice to filter it.
Maybe others have a different preference, and don't tend to use globals this way, and really don't want to see the extra stuff that merely including a header adds to your executable.  (I think it's at least somewhat interesting that headers potentially make your object files bigger even if you don't use anything from them directly.  Hmm, does stdio.h actually increase the size of an executable, or does dynamic linking end up matching those symbols in libc and not putting them in the executable's data or bss?)

Very valid points there. At the very least we should try and smart-include your own data; and make sure it's covered by options to see everything. Thanks for your remarks.


Add a --json flag, displaying results a pretty-printed JSON

Sorry @brouberol , I think my changes broke the fastforward merge of this PR.

Thanks for this PR !
The main.py file has been renamed to __main__.py with #1, can you adapt it before merging please ?

Done:

Thanks @brouberol !


race handling

A basic attempt to avoid race condition https://github.com/dgomes/hasp-lvgl-custom-component/pull/14#issuecomment-804202017

It works 90% of the cases. If I drag too fast, 10% cases still jumps. I guess it's a timing issue.
I think it can safely be hardcoded to val as that's the only existing case which could ever cause this. You're setting back the same val that triggered the change.

Not needed anymore because https://github.com/fvanroie/hasp-lvgl/pull/111 lets us use up events.

Could you save timestamp for arc/slider up events in cc ? Then you could do time compare against this to decide if HA value should be pushed to plate or not.

that's the throttle I'm mentioning... but it will also make things less "snappy"

If you only throttle arc/slider values pushed TO plates, I guess no one will be complaining 🙂
All such direct HA sensor <--> arc/slider implementations would all start a unintentional race condition.
Can't think of a use case where you actually would wan't instant sync of value from HA to plate, as plate already has the value.
Or am I missing the obvious  exception ??? 😉

I measure power consumption in the plate, I like to have my values in RT :)

BTW I was thinking of throttling the other way around PLATE -> HA, from what I'm seeing that's where to many events are being generated

@fvanroie already implemented mqtt message throttling some days ago 🙂
5 mqtt messages max. pr. second now
So no need to cut down on the flow from plate to HA.

@fvanroie already implemented mqtt message throttling some days ago

Yes that's very noticeable and it fixes the "elasticness" of the object during motion.
The problem is that:

while you drag the handle, a change is sent with a value at an unknown moment in time
you still keep dragging, but meanwhile a message is coming back from HA for the same object with the earlier val
this causes a jump back of the handle from under your finger. which causes a change message again

We should simply block sending back "val" back to the same object and plate which originated it.

Moreover: I see that change is useless to update other plates on the network. When I drag the volume slider of a media player in Home Assistant (that doesn't generate change events, the player just sends the val through the component when slider released) causes the slider in hasp-lvgl to move smoothly with animation.
So there's no need to stress plates with messages originating from "change" at all. Actually "change" messages should only be processed by HA for is own use, feedback changes to the plates in these cases could be blocked.

Now with @fvanroie's throttling it's already very effective.


Scrolling does not work with a small screen

Scrolling does not work with a small screen - for example, with a max-width 550px and below work on small screen
the code https://github.com/igor3310/jcarousel2

I found that on screens narrower than around 525px, the scrolling/navigation was not working. The problem was that "jCarousel tries to auto-detect the orientation by simply checking if the list elements’s height is greater than the list element’s width." On small screens, it wanted to move horizontally, not vertically. When I added the option  vertical: false, as mentioned on https://sorgalla.com/jcarousel/docs/reference/usage.html, it began to work on small sizes too.


Develop save particle files

Export and Import particles in .particles files. They are stored in TheOneEditor/Assets/Particles/.
You can export them in Export in the Inspector.
You can import them in Import in the Inspector.
TO-DO: Be able to change the name of the Component, now it exports it with the name ParticleSystem. For now we can change the name of the file manually.

Need to add change Particle Component name. Wait for pull.


curses example: use Lua boolean false instead of 0

curses.echo(0) is actually curses.echo(false), because value 0 is truthy in Lua.

Agreed.  Thanks for the pull-request :-)  Merged!


[ask] How to reuse repo code in subquery

const data = db.xxx.select({
  relationCount: q => q.relation.where(xxx).count() // the q in subquery can't re-use repo methods
})

How about myRepo(q.relation).whereSomething(xxx)?
export const myRepo = createRepo(db.relationTable, {
  queryMethods: {
    whereSomething: (xxx: SomeType) => {
      return q.where(xxx);
    },
  },
});

const data = db.xxx.select({
  relationCount: q => myRepo(q.relation).whereSomething(xxx).count() // the q in subquery can't re-use repo methods
})

Thank you, romeerez, it works well, I didn't think it would work.
This technique should be written into docs.

It's present here in repo docs in examples.


(Cocoa pods 0.39.0) "file not found with  include use quotes instead" for "NSString+Color.h"

Changing manually #import <NSString+Color.h> to #import "NSString+Color.h" solves the issue.

hum, very strange. I am using 0.39 too, and do not have any problem. Are you sure you are not overriding Library / headers search path elsewhere ?

Thanks for the response Nicolas, what should I look for exactly in my header/library search settings? I saw this response from 'rota' at CocoPods:

In simplified: angled bracket imports are for externals, and should be scoped according to where they come from, so this for example should be:

#import <RXCollection/RXCollection.h>`

So that the compiler knows where to link correctly. I'm not sure what's changed in CP, but we shouldn't have been supporting #import <RXCollection.h> as it breaks the compiler contract.

So from this I understand it should be something like
#import <NSString+Color/NSString+Color.h>
but I could be wrong.


Calling stop on a nonplaying sound gives an error

Hello,
I am trying to stop all sounds at once. I don't know exactly what sounds are playing at any one moment in time, but I have a list of all sounds. I did the following code to stop all sounds:
sound_list.map(sound=>sound.stop())

I get the following error:
TypeError: this.gain[0] is undefined
["./node_modules/web-audio-daw/build/wad.js"]/</<["./src/wad.js"]/</Wad</Wad.prototype.stop
C:/path_to_project/node_modules/web-audio-daw/build/wad.js:3272
  3269 |     }
  3270 | }
  3271 | if ( !label ) {
> 3272 |     this.gain[0].gain.cancelScheduledValues(context.currentTime);
  3273 |     this.gain[0].gain.setValueAtTime(this.gain[0].gain.value, context.currentTime);
  3274 |     this.gain[0].gain.linearRampToValueAtTime(.0001, context.currentTime + this.env.release);
  3275 | }

Note that a playing attribute may do the trick as well, then I don't need to stop a stopped sound. Something like:
sound.play

true

In the latest version, calling stop on a Wad that has never played will log a harmless warning instead of throwing an error.


Marxist propaganda on the website

Hi,
I am seeing a marxist propaganda on the old and new website(https://godoc.org/, https://pkg.go.dev/), called Black Lives Matter.
Can you please remove it and stop forcing your own political views on the entire dev community that is mostly not even based in USA?
 Here in Europe, we have experienced out own fair share of communism and we are definitely not looking to bring it back into fashion.
Cheers.

Please see https://github.com/golang/go/issues/41094#issuecomment-682574230.

Please see https://github.com/golang/go/issues/41094#issuecomment-682574230.


[Feature][Ready] Adding option to check/uncheck all fields in checklist

Title says it all, I've added a config option for checklist field to include "Select All" functionality.
As always, feel free to comment, suggest changes etc.

Ugh... this one sure slipped through the cracks... I'm sorry for the bad experience @pavoltanuska 🙏
Let's close this in favor of https://github.com/Laravel-Backpack/CRUD/pull/3989 which is a PR towards 4.x to do the same thing. That one we can merge, this one... would be very difficult, we'd have a lot of conflicts to fix.
Thanks again @pavoltanuska and sorry I didn't seen or react to this sooner.
Cheers!

Not a problem, it was easy enough to use custom field for the time being. One of the many reasons I enjoy using Backpack for all these years  :)


Support JSON for Postgres

Support taking and returning serde_json::Value (maybe others as well?) in queries.
I think this would be a Postgres-only feature, right?

Postgres has first class JSON support, but technically you can store JSON in text blobs in mysql as well - Indeed we actually do exactly that at my work.

We could serialize to JSON but the query!() macro couldn't possibly know to read out a varchar field as JSON.

I'm not an expert in MySQL, but why this isn't suiting? https://dev.mysql.com/doc/refman/8.0/en/json.html

It depends on the MySQL version. We're still using 5.6. because of some other tooling requirements, so it's not available there.

Is there anything for jsonb ?

JSON for Postgres is in master. Thanks @oeb25! I'll spin out another issue for MySQL.


Issue when format using short key without selecting all text

When a user didn't select all the text and use the short key for formating while placing the cursor at a random space in code, the text "CREATE OR REPLACE" printed into the code.

Create or replace will no longer be added when formatting using the shortcut key. For now, we need to select all before performing unwrap or formatting. This will be fixed in the next patch


Change F# if formatting guidelines

Clarified the meaning of small / large lines
Changed one of the recommended formats
Fixes #5744

 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.:x: giedriusbruzas sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

//cc @cartermp

Is there anything I'm supposed to do to get this merged, or?

I can merge. Thanks for the ping @giedriusbruzas


Initial model has ending line break ignored

I have a v-model ending with a line break

Reproduction Link
https://jsfiddle.net/h548vrgn/3/
Steps to reproduce
Use a v-model with a value ending by

.
What is Expected?
The content of the editor should end with a new empty line
What is actually happening?
No empty line at the end of the content.

So I am ...

@jpfong
Do you find a trick to solve the problem ?
I 'm still on it... without success

No, I'm working on other things now...


ConfigurationPathNotFoundException: A configuration file could not be found for: template-analyzer

Hello,
I'm configuring the task to run template-analyzer tool
- task: MicrosoftSecurityDevOps@1
      displayName: 'Microsoft Security DevOps'
      inputs:
        config: '$(Build.SourcesDirectory)/.gdn/.gdnsettings'
        tools: 'template-analyzer'

I created the gdnsettings using this example
https://github.com/microsoft/security-devops-azdevops/wiki#templateanalyzer-options
But when I execute the pipeline result this error
##[error]ConfigurationPathNotFoundException: A configuration file could not be found for: template-analyzer
##[error]MSDO CLI exited with an error exit code: 1
Can you please help me with this error?
Thank You

Hi @leandromsft , what is the content of your .gdnsettings file and is it checked into your repository under the given path ([repo_root]/.gdn/.gdnsettings)?

Hi @chrisnielsen-MS
The content of .gdnsettings is based in this doc
https://github.com/microsoft/security-devops-azdevops/wiki#templateanalyzer-options
The file .gdnsettings is stored in my git repo in the correct path

Thank you @leandromsft
I notice your error message has "template-analyzer" with a hyphen, but I don't see it referred to that way in the samples or in our codebase. Are you using "template-analyzer" for the tool name inside your .gdnsettings file? If so, try removing the hyphen and using "templateanalyzer" instead.

Thank You @chrisnielsen-MS for help! You're right I change the tool name to templateanalyzer and everthing works fine!
- task: MicrosoftSecurityDevOps@1
      displayName: 'Microsoft Security DevOps'
      inputs:
        tools: 'templateanalyzer'


Air-gapped provisioning of RKE2 cluster fails when vSphere Cloud Provider (CPI/CSI) enabled in cluster configuration

Issue description:
When provisioning an RKE2 Node Driver cluster with the vSphere Cloud Provider (i.e. CPI/CSI) configured in the cluster configuration (versus installed via Apps & Marketplace after cluster provisioning) the CPI and CSI charts do not use the configured system-default-registry, but attempt to pull from DockerHub. In an air-gapped environment without access to DockerHub the CPI and CSI workloads therefore remain in a CrashLoopBackoff attempting to pull the image, and node provisioning does not complete.
Based on backend investigation, it doesn't really have anything to do with the charts because of being able to edit as YAML, the user is able to set the global.cattle.systemDefaultRegistry, which means that it's able to use a private registry. It looks like the UI (or wherever this is configurable) should use it when provisioning.
There is a workaround (noted below).
Business impact:
Unable to provision RKE2 cluster with vSphere cloud provider set in cluster configuration if using a private registry in an air-gapped environment.
Troubleshooting steps:
N/A
Repro steps:

Populate a private registry with the Rancher v2.6.0 images (https://rancher.com/docs/rancher/v2.6/en/installation/other-installation-methods/air-gap/populate-private-registry/)
Install Rancher v2.6.0 on a single node RKE cluster with --set systemDefaultRegistry per air-gap installation docs https://rancher.com/docs/rancher/v2.6/en/installation/other-installation-methods/air-gap/install-rancher/
Provision an RKE2 cluster using the vSphere Node Driver, and the private registry config, without setting the vSphere Cloud Provider enabled in the cluster configuration, and observe successful cluster provisioning. After cluster provisioning deploy the vSphere CSI and CPI charts from the Apps & Marketplace and observe chart workloads successfully deployed using private registry images.
Provision an RKE cluster using the vSphere Node Driver, and the private registry config, and set the vSphere Cloud Provider to enabled in the cluster configuration, passing in configuration under Addons, and observe successful failed provisioning. Download SSH key for node and using kubectl on the node, observe CPI and CSI workloads failing attempting to pull image from DockerHub not private registry

Workaround:
Is workaround available and implemented? Yes
What is the workaround:
Install cluster without configuring vSphere Cloud Provider and install vSphere CPI/CSI charts after cluster provisioning
Actual behavior:
vSphere CSI/CPI workloads do not use configured system-default-registry if configured at cluster provisioning time, preventing successful cluster provisioning in air-gapped environment
Expected behavior:
vSphere CSI/CPI workloads use configured system-default-registry if configured at cluster provisioning time
Files, logs, traces:
N/A
Additional notes:
By clicking 'Edit as YAML' on the cluster configuration user could set global.cattle.systemDefaultRegistry manually for the rancher-vsphere-cpi and rancher-vsphere-csi but expectation would be this uses the system-default-registry automatically

@MKlimuszka  & @SheilaghM  - Can you guys sync up with @PennyScissors and see if this is something that would fall to Team 3 to handle based on how the vSphere Cloud Provider is enabled? If not, we might need to pass it to Team 2, but I want to confirm first.

We made the workaround known to Ansil, but we will need to do something about this.  They appear to be using RKE2, but they're not manually configuring the vSphere charts to point to the airgap resources. In the UI it allows some configuration, but probably not everything an airgap setup needs and RKE2 doesn't collect the variables necessary to pass that info to the UI.
We've asked for a copy of the chart (or the snippet where the systemdefaultregistry would be set) so we can verify how it is set and whether it is left to default as a sanity check.

An other workaround might be to use the registries.yaml redirect for images instead of the systemdefaultregistry - see https://docs.rke2.io/install/containerd_registry_configuration/

@codyrancher , The chart is here -> https://github.com/rancher/charts/blob/dev-v2.6/charts/rancher-vsphere-cpi/100.1.0%2Bup1.0.100/templates/_helpers.tpl

@codyrancher , The helper template I pasted is coming from this chart -> https://github.com/rancher/charts/blob/dev-v2.6/charts/rancher-vsphere-cpi/100.1.0%2Bup1.0.100/templates/_helpers.tpl

Ass per my understanding , who ever is creating helm-install-xxxx Pod (in this case  helm-install-rancher-vsphere-cpi-xxx &  helm-install-rancher-vsphere-csi-xxxx  Pods) that installs the chart should pass global.cattle.systemDefaultRegistry

Found a similar commit for rke2 upgarde, and I hope this will give some more idea on how to locate and fix the issue on cloud provider charts.
https://github.com/rancher/rancher/commit/3532d5253c1b1ffd9c7e0080d29a3216c6a05888

Found one more -> https://github.com/rancher/rancher/commit/e4377ca38d3094ea647244d7e7e6f627965c110f

Yeah, this needs to be fixed in the RKE2 vSphere provider Helm chart as described in https://github.com/rancher/rancher/issues/36467#issuecomment-1034809994

This issue is going to be addressed in RKE2. RKE2 uses a different value for the systemDefaultRegistry than Rancher and Rancher based charts expect global.cattle.systemDefaultRegistry. The RKE2 team is going to add support for passing global.cattle.systemDefaultRegistry, so RKE2 shipped Rancher charts work correctly.
Installing the vSphere charts from Apps and Marketplace isn't a tested scenario and isn't recommended. One way to work around this issue would be to set the global.cattle.systemDefaultRegistry in the addon-config for both charts. That would need to be done in the YAML.

Jamie will dev validate if assigned by QA because he did not work on these issues, based on KDM RC for RKE2.

A cluster has been created using RKE2 1.22.9-rc4+rke2r1 and Rancher 2.6.5-rc5. The vsphere charts are now picking up the private registry correctly.


Kube-VIP ARPs conflicting VIP information

Describe the bug
We are using kube-vip to advertise LoadBalancer VIPs using ARP. Normally, this works fine. In at least two cases, we have observed duplicate conflicting ARP advertisements happening simultaneously.
To Reproduce
The issue has been difficult to reproduce in isolation.
The cluster will run happily for a while and then seemingly at random we'll see issues resulting from the conflicting ARPs.
20:28:45.815587 ARP, Reply <IP_ADDRESS> is-at fa:c3:09:08:be:14, length 28
20:28:47.130643 ARP, Reply <IP_ADDRESS> is-at 42:92:ad:23:12:3z, length 46
When this occurs, the instance sending out ARP appears not to know that it a leader for the other IPs, because when shut down (by killing the pod), it does not report cleaning up the IP in question.
Shutting down the bad node will cause ARPs to stop but will not clear out the local IP address on the interface.
It appears that the ARP broadcasting goroutine is somehow running in the background while kube-vip is in this state. An investigation of the code made me suspicious of this line: https://github.com/kube-vip/kube-vip/blob/main/pkg/cluster/clusterLeaderElection.go#L116 not calling arpCancel(). But, of course, that is just a wild guess.
Expected behavior
All VIPs should only ever be advertised from one leader at a time. If an instance is not leader for a service, it should not send ARPs for that service.
Environment (please complete the following information):

OS/Distro: SLE Micro (based on SLES 15.5)
Kubernetes Version: v1.26 (rke2, rancher)
Kube-vip Version: 0.6.2

Kube-vip.yaml:
I think this is the relevant config. The only non-obvious variable here is the vip_interface, which is set to eth1 in this instance.
---
apiVersion: apps/v1
kind: DaemonSet
metadata:
  labels:
    app.kubernetes.io/name: kube-vip-ds
    app.kubernetes.io/version: ${daemonset_version}
  name: kube-vip-ds
  namespace: ${app_namespace}
spec:
  selector:
    matchLabels:
      app.kubernetes.io/name: kube-vip-ds
  template:
    metadata:
      labels:
        app.kubernetes.io/name: kube-vip-ds
        app.kubernetes.io/version: ${daemonset_version}
    spec:
      affinity:
        nodeAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            nodeSelectorTerms:
              - matchExpressions:
                  - key: node-role.kubernetes.io/master
                    operator: Exists
              - matchExpressions:
                  - key: node-role.kubernetes.io/control-plane
                    operator: Exists
      containers:
        - args:
            - manager
          env:
            - name: vip_arp
              value: 'true'
            - name: port
              value: '6443'
            - name: vip_interface
              value: '${vip_interface}'
            - name: vip_servicesinterface
              value: '${vip_interface}'
            - name: vip_cidr
              value: '32'
            - name: svc_enable
              value: 'true'
            - name: svc_election
              value: 'true'
            - name: enable_service_security
              value: 'false'
            - name: prometheus_server
              value: ':2112'
          image: ghcr.io/kube-vip/kube-vip:${daemonset_version}
          imagePullPolicy: IfNotPresent
          name: kube-vip
          resources: {}
          securityContext:
            capabilities:
              add:
                - NET_ADMIN
                - NET_RAW
      hostNetwork: true
      serviceAccountName: kube-vip
      tolerations:
        - effect: NoSchedule
          operator: Exists
        - effect: NoExecute
          operator: Exists
  updateStrategy: {}
status:
  currentNumberScheduled: 0
  desiredNumberScheduled: 0
  numberMisscheduled: 0
  numberReady: 0

Additional context
Add any other context about the problem here.

You enabled svc_election  and this is only for advertising service type lb, so it's expected that each service will have its own leader, and they acquires leader on each kube-vip independently, so there could be 2 services acquiring leader on same node or on different node.
Relevant code is here https://github.com/kube-vip/kube-vip/blob/main/pkg/manager/servicesLeader.go#L18
It's possible that the previous leader didn't exist correctly, so two leader are ARPing for same service, but it's possible that two service type lbs got assigned with same IP, so two node are reporting the same IP, this is also worth checking.
I would recommend using newer kube-vip since many refactoring have been made, and please upload kube-vip log for each of the leader when this issue happens, at least we could check if it's ARPing for same service or not

@lubronzhan Thanks for the help!

there could be 2 services acquiring leader on same node or on different node.

How do I check for this condition and prevent it's occurrence?
Will do on the update.

there could be 2 services acquiring leader on same node or on different node.
How do I check for this condition and prevent it's occurrence?

This is expected, since you enabled svc_election, each service will has its own leader. and it should work as long as two services have different loadbalancerIP. So I don't think you want to prevent its occurrence. Unless you have two services share the same loadbalancerIP.
I would need more log to help understand the issue. Please update once you run into that again.
Thanks

Ack on the per-service load balancer. I was getting confused on nodes vs service leader but I think I get it now. My understanding is that each service (differentiated by loadbalancerIP) should have a unique leader. Which is what appears not to have happened as two instances were ARPing the same loadbalancerIP to different interfaces simultaneously.

Unless you have two services share the same loadbalancerIP.
That is not the case now but might have occurred transiently and been subsequently corrected.

Working on an update and will provide additional logs.

Will close this as no update provided since 6 months.