[prettier-plugin-sql] Incorrect use of MIT license when depending on a GPL-licensed library

The node-sql-parser library (used by prettier-plugin-sql) is licensed under GPLv2.
The prettier-plugin-sql plugin uses this library, but is not licensed under GPLv2 or later as the GPL license requires. Instead it uses the permissive MIT license.
I assume that this is not intentional, but simply an oversight by the author, as JavaScript libraries nowadays seem to mostly come with very permissive licenses.
Suggested fix:

Change the license of prettier-plugin-sql to GPLv2

Thanks for notification, just align the LICENSE.


[Java] Are the constraints of not-nullable MapVector necessary in Java?

There are two not-nullable constraints in the Java implementation of MapVector:

Map data should be a non-nullable struct type;
https://github.com/apache/arrow/blob/d4516c5386f84619dfdf2a9f72fed6d7df89704c/java/vector/src/main/java/org/apache/arrow/vector/complex/MapVector.java#L98
Map data key type should be a non-nullable;
https://github.com/apache/arrow/blob/d4516c5386f84619dfdf2a9f72fed6d7df89704c/java/vector/src/main/java/org/apache/arrow/vector/complex/MapVector.java#L105

Are these constraints necessary? Because I cannot find these constraints in the Arrow Specifications .
Component(s)
Java

That document is not canonical on its own, as it notes.
https://github.com/apache/arrow/blob/a153dcb7ae1f61e705472574a5890b33db8cb76a/format/Schema.fbs#L138


Traffic Router use of obsolete breaks roll back

The following line of code prevent any rollback of traffic router version previous to 3.0.0.
https://github.com/apache/trafficcontrol/blob/ebd05c703c22ae634398838a011f6715b747838b/traffic_router/build/pom.xml#L228
I recommend removing the line as you can't install any previous RPMS by specifying the version number anymore unless you use special command line for yum.

Does sudo yum --setopt=obsoletes=0 install traffic_router help?


Reactivating the plugin does not init the cron-jobs properly

Repro:

Create 4 scheduled messages:

/scheduler add @every 10s: Test1
/scheduler add @every 10s: Test2
/scheduler add @every 10s: Test3
/scheduler add @every 10s: Test4

Check that all messages get posted correctly
Deactivate the plugin
No more messages
Activate the plugin again
4 messages are posted in roughly the right time interval - but all of them say Test4
/scheduler list shows the right messages
/scheduler remove does not really work to get rid of these messages

This code seems not to work properly in this case: https://github.com/monsdar/mattermost-scheduler-plugin/blob/2be0dbba1eb9e5665d13841b89f17d0f3801acaf/server/plugin.go#L53
I guess it has something to do with the foreach overwriting the func that gets registered with Cron.


DM unread count in sidebar is messed up

also, it appears for a few seconds if you use the sidebar buttons to quickscroll to a column which is kinda weird

done


Strange WAL size, much larger than db size

I've set DB size and size increase step to be 64MB, but WAL become 600+ MB. Is it because changes are not yet applied to *.db file itself ? Or does it contain more information than just my data ? Changing bulk insert size from 1000 to 200 docs still creates 250+ MB of logs.

DbMaker.getStore().commit() resolves this issue.


python equivalent of  go code

func (wac *Conn) LeaveGroup(jid string) (<-chan string, error) {
	return wac.setGroup("leave", jid, "", nil)
}

Anyone know python equivalent of this go code

Without knowing what the wac object actually is, it's not possible to help you here.


Use Airtable API to fetch dynamic landing page student + pupil counts

Current implementation is super slow. I think that's due to a mixture of using 3rd party GraphQL API + needlessly fetching all table ids rather than getting just the table row count directly, i.e. fetching just one number for student and pupil table each rather than the current ~6.5k row ids.
@kleinicke @l8l @GG8000 Could we try this suggestion of having a derived table that counts entries in our student and pupil tables?

Our plan is anyway to write an demon that sits in the middle between Airtable and the website. This demon can also deal with counting the entries in the tables, so the website does not have to deal with that.

@kleinicke Don't think that's a good idea. Adding a middleman adds needless complexity which will likely lead to extra down time, not to mention the added maintenance burden.

Closing as stale since changes are outdated and now conflict.


spotify player - no playback found

Describe the bug
After application update, I get an error in the playback window stating "No playback found. Please make sure there is a running Spotify devlice and try to connect to one using the 'SwitchDevice' command. Which when I run, shows no devices. This worked perfectly fine yesterday before the update, using my own app ID like outlined in the spotify-connect setup. Using arch with the AUR build of spotify-player.
Expected behaviour
Song is played through the player like intended
Log and backtrace
2024-05-30T18:52:59.493062Z  INFO spotify_player: Configurations: Configs { app_config: AppConfig { theme: "dracula", client_id: "", client_port: 8080, player_event_hook_command: None, playback_format: "{track} • {artists}\n{album}\n{metadata}", tracks_playback_limit: 50, proxy: None, ap_port: None, app_refresh_duration_in_ms: 32, playback_refresh_duration_in_ms: 0, page_size_in_rows: 20, play_icon: "▶", pause_icon: "▌▌", liked_icon: "♥", border_type: Plain, progress_bar_type: Rectangle, playback_window_position: Top, playback_window_width: 6, enable_cover_image_cache: true, default_device: "spotify-player", device: DeviceConfig { name: "TofuBox", device_type: "speaker", volume: 85, bitrate: 320, audio_cache: false, normalization: false } }, keymap_config: KeymapConfig { keymaps: [Keymap { key_sequence: KeySequence { keys: [None(Char('n'))] }, command: NextTrack }, Keymap { key_sequence: KeySequence { keys: [None(Char('p'))] }, command: PreviousTrack }, Keymap { key_sequence: KeySequence { keys: [None(Char('.'))] }, command: PlayRandom }, Keymap { key_sequence: KeySequence { keys: [None(Char(' '))] }, command: ResumePause }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('r'))] }, command: Repeat }, Keymap { key_sequence: KeySequence { keys: [Alt(Char('r'))] }, command: ToggleFakeTrackRepeatMode }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('s'))] }, command: Shuffle }, Keymap { key_sequence: KeySequence { keys: [None(Char('+'))] }, command: VolumeUp }, Keymap { key_sequence: KeySequence { keys: [None(Char('-'))] }, command: VolumeDown }, Keymap { key_sequence: KeySequence { keys: [None(Char('_'))] }, command: Mute }, Keymap { key_sequence: KeySequence { keys: [None(Char('>'))] }, command: SeekForward }, Keymap { key_sequence: KeySequence { keys: [None(Char('<'))] }, command: SeekBackward }, Keymap { key_sequence: KeySequence { keys: [None(Enter)] }, command: ChooseSelected }, Keymap { key_sequence: KeySequence { keys: [None(Char('r'))] }, command: RefreshPlayback }, Keymap { key_sequence: KeySequence { keys: [None(Char('/'))] }, command: Search }, Keymap { key_sequence: KeySequence { keys: [None(Char('z'))] }, command: Queue }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('z'))] }, command: AddSelectedItemToQueue }, Keymap { key_sequence: KeySequence { keys: [None(Char('Z'))] }, command: AddSelectedItemToQueue }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char(' '))] }, command: ShowActionsOnSelectedItem }, Keymap { key_sequence: KeySequence { keys: [None(Char('g')), None(Char('a'))] }, command: ShowActionsOnSelectedItem }, Keymap { key_sequence: KeySequence { keys: [None(Char('a'))] }, command: ShowActionsOnCurrentTrack }, Keymap { key_sequence: KeySequence { keys: [None(Tab)] }, command: FocusNextWindow }, Keymap { key_sequence: KeySequence { keys: [None(BackTab)] }, command: FocusPreviousWindow }, Keymap { key_sequence: KeySequence { keys: [None(Char('T'))] }, command: SwitchTheme }, Keymap { key_sequence: KeySequence { keys: [None(Char('D'))] }, command: SwitchDevice }, Keymap { key_sequence: KeySequence { keys: [None(Char('u')), None(Char('p'))] }, command: BrowseUserPlaylists }, Keymap { key_sequence: KeySequence { keys: [None(Char('u')), None(Char('a'))] }, command: BrowseUserFollowedArtists }, Keymap { key_sequence: KeySequence { keys: [None(Char('u')), None(Char('A'))] }, command: BrowseUserSavedAlbums }, Keymap { key_sequence: KeySequence { keys: [None(Char('g')), None(Char(' '))] }, command: CurrentlyPlayingContextPage }, Keymap { key_sequence: KeySequence { keys: [None(Char('g')), None(Char('t'))] }, command: TopTrackPage }, Keymap { key_sequence: KeySequence { keys: [None(Char('g')), None(Char('r'))] }, command: RecentlyPlayedTrackPage }, Keymap { key_sequence: KeySequence { keys: [None(Char('g')), None(Char('y'))] }, command: LikedTrackPage }, Keymap { key_sequence: KeySequence { keys: [None(Char('g')), None(Char('l'))] }, command: LibraryPage }, Keymap { key_sequence: KeySequence { keys: [None(Char('g')), None(Char('s'))] }, command: SearchPage }, Keymap { key_sequence: KeySequence { keys: [None(Char('g')), None(Char('b'))] }, command: BrowsePage }, Keymap { key_sequence: KeySequence { keys: [None(Backspace)] }, command: PreviousPage }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('q'))] }, command: PreviousPage }, Keymap { key_sequence: KeySequence { keys: [None(Char('O'))] }, command: OpenSpotifyLinkFromClipboard }, Keymap { key_sequence: KeySequence { keys: [None(Char('?'))] }, command: OpenCommandHelp }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('h'))] }, command: OpenCommandHelp }, Keymap { key_sequence: KeySequence { keys: [None(Char('q'))] }, command: Quit }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('c'))] }, command: Quit }, Keymap { key_sequence: KeySequence { keys: [None(Esc)] }, command: ClosePopup }, Keymap { key_sequence: KeySequence { keys: [None(Char('j'))] }, command: SelectNextOrScrollDown }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('n'))] }, command: SelectNextOrScrollDown }, Keymap { key_sequence: KeySequence { keys: [None(Down)] }, command: SelectNextOrScrollDown }, Keymap { key_sequence: KeySequence { keys: [None(Char('k'))] }, command: SelectPreviousOrScrollUp }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('p'))] }, command: SelectPreviousOrScrollUp }, Keymap { key_sequence: KeySequence { keys: [None(Up)] }, command: SelectPreviousOrScrollUp }, Keymap { key_sequence: KeySequence { keys: [None(PageUp)] }, command: PageSelectPreviousOrScrollUp }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('b'))] }, command: PageSelectPreviousOrScrollUp }, Keymap { key_sequence: KeySequence { keys: [None(PageDown)] }, command: PageSelectNextOrScrollDown }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('f'))] }, command: PageSelectNextOrScrollDown }, Keymap { key_sequence: KeySequence { keys: [None(Char('g')), None(Char('g'))] }, command: SelectFirstOrScrollToTop }, Keymap { key_sequence: KeySequence { keys: [None(Home)] }, command: SelectFirstOrScrollToTop }, Keymap { key_sequence: KeySequence { keys: [None(Char('G'))] }, command: SelectLastOrScrollToBottom }, Keymap { key_sequence: KeySequence { keys: [None(End)] }, command: SelectLastOrScrollToBottom }, Keymap { key_sequence: KeySequence { keys: [None(Char('s')), None(Char('t'))] }, command: SortTrackByTitle }, Keymap { key_sequence: KeySequence { keys: [None(Char('s')), None(Char('a'))] }, command: SortTrackByArtists }, Keymap { key_sequence: KeySequence { keys: [None(Char('s')), None(Char('A'))] }, command: SortTrackByAlbum }, Keymap { key_sequence: KeySequence { keys: [None(Char('s')), None(Char('d'))] }, command: SortTrackByDuration }, Keymap { key_sequence: KeySequence { keys: [None(Char('s')), None(Char('D'))] }, command: SortTrackByAddedDate }, Keymap { key_sequence: KeySequence { keys: [None(Char('s')), None(Char('r'))] }, command: ReverseTrackOrder }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('k'))] }, command: MovePlaylistItemUp }, Keymap { key_sequence: KeySequence { keys: [Ctrl(Char('j'))] }, command: MovePlaylistItemDown }, Keymap { key_sequence: KeySequence { keys: [None(Char('N'))] }, command: CreatePlaylist }] }, theme_config: ThemeConfig { themes: [Theme { name: "default", palette: Palette { background: None, foreground: None, black: Color { color: Black }, blue: Color { color: Blue }, cyan: Color { color: Cyan }, green: Color { color: Green }, magenta: Color { color: Magenta }, red: Color { color: Red }, white: Color { color: Gray }, yellow: Color { color: Yellow }, bright_black: Color { color: DarkGray }, bright_white: Color { color: White }, bright_red: Color { color: LightRed }, bright_magenta: Color { color: LightMagenta }, bright_green: Color { color: LightGreen }, bright_cyan: Color { color: LightCyan }, bright_blue: Color { color: LightBlue }, bright_yellow: Color { color: LightYellow } }, component_style: ComponentStyle { block_title: None, border: None, playback_track: None, playback_artists: None, playback_album: None, playback_metadata: None, playback_progress_bar: None, current_playing: None, page_desc: None, playlist_desc: None, table_header: None, selection: None } }, Theme { name: "dracula", palette: Palette { background: None, foreground: None, black: Color { color: Black }, blue: Color { color: Blue }, cyan: Color { color: Cyan }, green: Color { color: Green }, magenta: Color { color: Magenta }, red: Color { color: Red }, white: Color { color: Gray }, yellow: Color { color: Yellow }, bright_black: Color { color: DarkGray }, bright_white: Color { color: White }, bright_red: Color { color: LightRed }, bright_magenta: Color { color: LightMagenta }, bright_green: Color { color: LightGreen }, bright_cyan: Color { color: LightCyan }, bright_blue: Color { color: LightBlue }, bright_yellow: Color { color: LightYellow } }, component_style: ComponentStyle { block_title: None, border: None, playback_track: None, playback_artists: None, playback_album: None, playback_metadata: None, playback_progress_bar: None, current_playing: None, page_desc: None, playlist_desc: None, table_header: None, selection: None } }] }, cache_folder: "/home/bunta/.cache/spotify-player" }
2024-05-30T18:52:59.493110Z  INFO spotify_player::state::data: Loading Playlists data from /home/bunta/.cache/spotify-player/Playlists_cache.json...
2024-05-30T18:52:59.493355Z  INFO spotify_player::state::data: Successfully loaded Playlists data!
2024-05-30T18:52:59.493357Z  INFO spotify_player::state::data: Loading FollowedArtists data from /home/bunta/.cache/spotify-player/FollowedArtists_cache.json...
2024-05-30T18:52:59.493867Z  INFO spotify_player::state::data: Successfully loaded FollowedArtists data!
2024-05-30T18:52:59.493870Z  INFO spotify_player::state::data: Loading SavedAlbums data from /home/bunta/.cache/spotify-player/SavedAlbums_cache.json...
2024-05-30T18:52:59.496945Z  INFO spotify_player::state::data: Successfully loaded SavedAlbums data!
2024-05-30T18:52:59.496948Z  INFO spotify_player::state::data: Loading SavedTracks data from /home/bunta/.cache/spotify-player/SavedTracks_cache.json...
2024-05-30T18:52:59.552559Z  INFO spotify_player::state::data: Successfully loaded SavedTracks data!
2024-05-30T18:52:59.800779Z  INFO spotify_player::auth: Successfully used the cached credentials to create a new session!
2024-05-30T18:52:59.824800Z  INFO spotify_player::token: Getting new authentication token...
2024-05-30T18:52:59.859452Z  INFO spotify_player::token: Got new token: Token { access_token: "", expires_in: TimeDelta { secs: 3600, nanos: 0 }, expires_at: Some(2024-05-30T19:52:59.859443714Z), refresh_token: None, scopes: {} }
2024-05-30T18:52:59.945338Z  INFO spotify_player: No playback found on startup, trying to connect to an available device...
2024-05-30T18:52:59.945581Z  INFO spotify_player: Starting a client socket at <IP_ADDRESS>:8080
2024-05-30T18:53:00.023272Z  INFO client_request{request=GetCurrentUser}: spotify_player::client: Successfully handled the client request, took: 77ms
2024-05-30T18:53:00.141955Z  INFO client_request{request=GetUserPlaylists}: spotify_player::client: Successfully handled the client request, took: 196ms
2024-05-30T18:53:00.227497Z  INFO client_request{request=GetUserFollowedArtists}: spotify_player::client: Successfully handled the client request, took: 281ms
2024-05-30T18:53:00.994133Z  WARN client_request{request=ConnectDevice}: spotify_player::client: No device found. Please make sure you already setup Spotify Connect support as described in https://github.com/aome510/spotify-player#spotify-connect.
2024-05-30T18:53:02.040931Z  WARN client_request{request=ConnectDevice}: spotify_player::client: No device found. Please make sure you already setup Spotify Connect support as described in https://github.com/aome510/spotify-player#spotify-connect.
2024-05-30T18:53:03.087858Z  WARN client_request{request=ConnectDevice}: spotify_player::client: No device found. Please make sure you already setup Spotify Connect support as described in https://github.com/aome510/spotify-player#spotify-connect.
2024-05-30T18:53:03.187384Z  INFO client_request{request=GetUserSavedAlbums}: spotify_player::client: Successfully handled the client request, took: 3241ms
2024-05-30T18:53:03.951409Z  INFO spotify_player::client::handlers: Current context ID (None) is different from the expected ID (Some(Tracks(TracksId { uri: "tracks:user-liked-tracks", kind: "Liked Tracks" }))), update the context state
2024-05-30T18:53:04.134058Z  WARN client_request{request=ConnectDevice}: spotify_player::client: No device found. Please make sure you already setup Spotify Connect support as described in https://github.com/aome510/spotify-player#spotify-connect.
2024-05-30T18:53:05.180055Z  WARN client_request{request=ConnectDevice}: spotify_player::client: No device found. Please make sure you already setup Spotify Connect support as described in https://github.com/aome510/spotify-player#spotify-connect.
2024-05-30T18:53:05.959074Z  INFO client_request{request=GetUserSavedTracks}: spotify_player::client: Successfully handled the client request, took: 6013ms
2024-05-30T18:53:06.231118Z  WARN client_request{request=ConnectDevice}: spotify_player::client: No device found. Please make sure you already setup Spotify Connect support as described in https://github.com/aome510/spotify-player#spotify-connect.
2024-05-30T18:53:07.291926Z  WARN client_request{request=ConnectDevice}: spotify_player::client: No device found. Please make sure you already setup Spotify Connect support as described in https://github.com/aome510/spotify-player#spotify-connect.
2024-05-30T18:53:07.629658Z ERROR
spotify_player::client::handlers: Failed to handle client request: http error: status code 404 Not Found
2024-05-30T18:53:08.348564Z  WARN client_request{request=ConnectDevice}: spotify_player::client: No device found. Please make sure you already setup Spotify Connect support as described in https://github.com/aome510/spotify-player#spotify-connect.
2024-05-30T18:53:08.500652Z  INFO client_request{request=GetUserSavedTracks}: spotify_player::client: Successfully handled the client request, took: 5316ms
2024-05-30T18:53:09.391650Z  WARN client_request{request=ConnectDevice}: spotify_player::client: No device found. Please make sure you already setup Spotify Connect support as described in https://github.com/aome510/spotify-player#spotify-connect.
2024-05-30T18:53:10.439741Z  WARN client_request{request=ConnectDevice}: spotify_player::client: No device found. Please make sure you already setup Spotify Connect support as described in https://github.com/aome510/spotify-player#spotify-connect.
2024-05-30T18:53:10.439812Z  INFO client_request{request=ConnectDevice}: spotify_player::client: Successfully handled the client request, took: 10493ms
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

OS: Arch
Application version: 0.18.2
Application features: yay -S spotify-player

Additional context
I've also noticed over the past week or so, album art image cache's haven't been downloading. I have a script I made to display them in feh seperately, but for some reason that folder is no longer being updated even though the config/cache filepaths are correct.

UPDATE: Installing the cargo package directly seemed to have fixed the playback issue, but I'm still not getting new image caches.

I don't know why you encountered the no playback found with AUR but image cache is only enabled when you have image feature enabled (disabled by default) and enable_cover_image_cache config option is true.

Strange, I've used this software for the past 6 months or so getting image cache's with the image feature off.

Is this possible to add a standard feature? I like to have the option to manage the album art myself without displaying it in the terminal. Regardless, thanks for the help and the cool software.

Is this possible to add this as a standard feature?

Yeah, should be by making enable_cover_image_cache config option independent of image feature.

I've added support for this on my own fork, but it required making the image library, the image cache's, and the logic to fetch the image cache non-optional. I can try to rewrite it as a feature named "imageCacheOnly" if you'd prefer. It otherwise works well this way and as I was expecting
https://github.com/jStimpert0430/spotify-player/commit/7069a33c9fdc855741a71d2b2d9ca9b167f396d9


Build tee-supplicant with a customizable CFG_TEE_PLUGIN_LOAD_PATH

Description of changes
Adding a NixOS option for tee-supplicant to modify CFG_TEE_PLUGIN_LOAD_PATH by build time. As an example usage of setting CFG_TEE_PLUGIN_LOAD_PATH:
hardware.nvidia-jetpack.firmware.optee.suppPluginLoadPath = pkgs.linkFarm "optee-plugin-path" [
{
    name = "myTeeSupp.plugin";
    path = "myTeeSuppPluginDerivation";
}];

Testing
Tested on orin-nx-devkit

Is there a use case for dynamically changing plugins that exist under the tee supplicant plugin path? If not, I propose this option's type be types.listOf types.package. If empty, we can build tee supplicant with CFG_TEE_SUPP_PLUGINS=n, and if not empty, we can use linkFarm or something similar and build tee supplicant with CFG_TEE_PLUGIN_LOAD_PATH=${accumulatedPlugins} or whatever name we end up choosing. In that case, a name like supplicantPlugins probably works.
I recognize that the existing option clientLoadPath defaults to a path where it makes it easy for the user to dynamically load TAs at runtime, but I think that it should probably move to this style as well with just a list of derivations.

Maybe an explicit option for CFG_TEE_SUPP_PLUGINS would make more sense than deducting it from CFG_TEE_PLUGIN_LOAD_PATH? Maybe a potential user would like to keep plugin framework even if they would not create plugins during "initial deployment". Equally same would be if user would know that there are no use for plugins and it can be turned off.
Thinking further then adding an option for CFG_TEE_PLUGIN_LOAD_PATH might be excessive due same thing could be done with tee-supplicant command line parameters. Daemon could start with plugin-path-option.

As a rationale to this pull request then this is related to plugin framework and OPTEE xtests (with pkcs11 support).
I have a fork with xtests and pkcs11: https://github.com/TanelDettenborn/jetpack-nixos

@jmbaur I have rebased and updated pull request. Updated pull request addresses the same problem, but its approach is more common than a specific option. It introduces nixos option for providing extra make flags to OP-TEE client build. Even if the extra make flags makes clientLoadPath redundant then I have kept it for legacy reasons.
For documentation purposes I have kept the original topic and body, but if preferred then I can update topic and body accordingly. Or if this feature is not desired then I can close this pull request.

I might have been thinking about this incorrectly. tee-supplicant has a --plugin-paths cli argument that it will accept to override any default value specified at build time. We should probably make the default more nixos-friendly, but we do already have a nixos option for passing extra args to tee-supplicant.

Apologies for my vague expressions. It might have been clearer, if I would have edited pull request topic/body or closed and opened another pull request.
This pull request introduces nixos-option for providing a list of extra make flags to OP-TEE's client build. It does not add a specific option for a specific flag, but a general option. Option type is a list of strings. It is a similar option as there is already for tos-build. OP-TEE client can be configured with following flags: config.mk.
For example if there is a need for minimizing(/removing unused features) from OP-TEE client then currently it is not possible. For example GP socket support (CFG_GP_SOCKETS) can be only removed at the build time.
I do not have a strong opinions about user/nixos-friendliness, but kind of related then I have another pull request open (pkcs11 and xtest) in which tee-supplicant CLI arguments are break into smaller parts for sake of code readability rather than user/nixos-friendliness.

My opinion is that we should not continue adding NixOS module options for modifying how packages are built, and rather allow the user use nixpkgs.overlays to add their own build customizations. Many packages built by this repo are not currently in a condition to do this however (any package that ends up under config.system.build.jetsonDevicePkgs). In #156 I've placed opteeClient under the main package set so that it will end up under pkgs.nvidia-jetpack.opteeClient, so this should be extendable by just using regular overlays. I'm also working further to move more packages into a condition where they can be overridden more easily without adding further nixos module options.
So since #156 is not landed, I add my approval here, but I think what I've mentioned above is how we should move forward with similar changes.

Overlays would do it and using them might be a better approach for the project! I have skimmed through #156 and it is a solid work. As soon as OP-TEE's client patch is final and merged into OP-TEE main then this pull request will be outdated/useless. So rather than merging it into main, I would close this pull request to save from conflicts and keeping commit history clean.

@danielfullmer I think this PR can be closed since it is resolved by #156


Markdown code block with type and extra space breaks rest of document

VSCode Version: 1.4.0-insiders
OS Version: OSX

Steps to Reproduce:
Create markdown file with content:
(Notice extra space after closing ``` in first and last case)

Github handles it fine:
let test = 0

Coloring ok here (closing code block has space)
let test = 0

Coloring ok here (closing code block has NO space)
let test = 0

Coloring ok here (closing code block has space)

I believe this was fixed by #12342.
We now correctly support any number of spaces after the end of a fenced code block. The fix is in the latest insiders builds.


Dependencies outdated?

Since Blessed isn't been maintain can this be used with neo-blessed?

are there specific deps that need to be updated?

Just looking into this and trying to decide where to start and which one to use. This depends on blessed right?

are there specific deps that need to be updated?

marked-terminal seems out of date, I've been running in to the following using the markdown module:
[ERROR] 09:12:10 TypeError: this.renderer.checkbox is not a function                                                                                │
Please report this to https://github.com/markedjs/marked

Searching the marked issues, it seems this is actually a marked-terminal bug (which is apparently fixed): https://github.com/markedjs/marked/issues/1323#issuecomment-415813602

@daniel-bytes thanks!
looks like latest version is 4, can you confirm it is indeed working well with blessed-contrib and not breaking anything? if so I'll happily release with that

@lirantal thanks!  I cloned the repo and created a file test.js:
const blessed = require("blessed")
const contrib = require("./index")

const screen = blessed.screen()
const markdown = contrib.markdown()
screen.append(markdown)
markdown.setMarkdown("- [x] Checkbox")
screen.render()

running this via node test.js gives the following error:
TypeError: this.renderer.checkbox is not a function
Please report this to https://github.com/markedjs/marked.
    at Parser.tok (/Users/dbattaglia/code/github/blessed-contrib/node_modules/marked/lib/marked.js:1270:31)
    at Parser.tok (/Users/dbattaglia/code/github/blessed-contrib/node_modules/marked/lib/marked.js:1258:22)
    at Parser.parse (/Users/dbattaglia/code/github/blessed-contrib/node_modules/marked/lib/marked.js:1149:17)
    at Function.Parser.parse (/Users/dbattaglia/code/github/blessed-contrib/node_modules/marked/lib/marked.js:1131:17)
    at marked (/Users/dbattaglia/code/github/blessed-contrib/node_modules/marked/lib/marked.js:1632:19)
    at Markdown.setMarkdown (/Users/dbattaglia/code/github/blessed-contrib/lib/widget/markdown.js:35:19)
    at Object.<anonymous> (/Users/dbattaglia/code/github/blessed-contrib/test.js:8:10)
    at Module._compile (internal/modules/cjs/loader.js:959:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:995:10)
    at Module.load (internal/modules/cjs/loader.js:815:32)

After updating marked and marked-terminal to latest versions
"marked": "^0.8.0",
"marked-terminal": "^4.0.0",

I ran the test again and this time I get the appropriate output:
    * [X] Checkbox

running node ./examples/dashboard.js also seemed to work well.
One thing that is odd though is, when I use npm link to run my locally cloned copy with updated dependencies against my project I'm working on, I'm getting the following error :
No active screen.
    at /Users/dbattaglia/code/github/blessed-contrib/node_modules/blessed/lib/widget.js:41:26
    at Tree.Node (/Users/dbattaglia/code/github/blessed-contrib/node_modules/blessed/lib/widget.js:41:58)
    at Tree.Element (/Users/dbattaglia/code/github/blessed-contrib/node_modules/blessed/lib/widget.js:2128:8)
    at Tree.Box (/Users/dbattaglia/code/github/blessed-contrib/node_modules/blessed/lib/widget.js:4658:11)
    at new Tree (/Users/dbattaglia/code/github/blessed-contrib/lib/widget/tree.js:18:7)
    at Object.Tree (/Users/dbattaglia/code/github/blessed-contrib/lib/widget/tree.js:8:12)
    at Object.<anonymous> (/Users/dbattaglia/code/projects/my-project/app.ts:76:24)
    at Generator.next (<anonymous>)
    at fulfilled (/Users/dbattaglia/code/projects/my-project/ui/app.ts:5:58)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

where my code at my-project/app.ts:76:24 looks like
  const screen = blessed.screen({ title: "Working On Now" })
  const grid = new contrib.grid({
    rows: 1,
    cols: 4,
    screen: screen
  })

  const tree = contrib.tree({  // <-- Line 76 where error is being thrown
    parent: grid.set(0, 0, 1, 1, blessed.box, { label: 'Working On Now' }),
    fg: 'green',
    keys: ['space']
  })

If I run unlinked using "blessed-contrib": "^4.8.18" in my package.json file it works fine.  Very likely I'm just not using npm link correctly though.

Haven't had a chance to look at why that would fail for you. I'll shoot out a PR to upgrade marked-terminal to 4.

BTW 4.8.19 https://www.npmjs.com/package/blessed-contrib published


Update Diagram for Central API Blog Post

Since merging #1003, it is now again possible to test patches locally.
For this I pulled the branch from this PR, checked it out locally and ran bash run-podman.sh. The local container builds the Jekyll webpage to be looked at http://localhost:4000.


read big table

If I read the whole records of the big table, does moor retrieve all data at once? If the size of the table is too big to fill the entire memory, what happens?

I found the limit operation can handle this. But I still wonder what happens. App crashes?

Yeah, the app will crash when reading data larger than its available memory, there's not much moor can do about that :smile: But as you've said, with a limit operator in place this shouldn't happen. Let me know if you have any more questions.

@simolus3  Thanks!


[SPARK-41256][CONNECT][FOLLOWUP] Fix compile error

What changes were proposed in this pull request?
https://github.com/apache/spark/commit/4c35c5bb5e545acb2f46a80218f68e69c868b388 made a breaking change in proto message, and cause https://github.com/apache/spark/commit/dcd9cae4a3cb715aa91e9679b0adc9aa43035b2f failed after merged
Why are the changes needed?
to fix compile error:
[error] /__w/spark/spark/connector/connect/src/test/scala/org/apache/spark/sql/connect/planner/SparkConnectPlannerSuite.scala:441:67: value setI32 is not a member of org.apache.spark.connect.proto.Expression.Literal.Builder
[error]                   .setLiteral(proto.Expression.Literal.newBuilder.setI32(32))))
[error]                                                                   ^
[error] /__w/spark/spark/connector/connect/src/test/scala/org/apache/spark/sql/connect/planner/SparkConnectPlannerSuite.scala:446:67: value setI32 is not a member of org.apache.spark.connect.proto.Expression.Literal.Builder
[error]                   .setLiteral(proto.Expression.Literal.newBuilder.setI32(32)))))
[error]                                                                   ^
[error] /__w/spark/spark/connector/connect/src/test/scala/org/apache/spark/sql/connect/planner/SparkConnectPlannerSuite.scala:466:69: value setI32 is not a member of org.apache.spark.connect.proto.Expression.Literal.Builder
[error]                     .setLiteral(proto.Expression.Literal.newBuilder.setI32(32)))))
[error]                                                                     ^
[error] three errors found

Does this PR introduce any user-facing change?
No
How was this patch tested?
local test

cc @HyukjinKwon @amaliujia

Merged to master.

Thanks for fixing this quickly!
LGTM!


Add EntityFilter helper

Description:
This PR adds a new EntityFilter helper class that implements a standardized include/exclude filter for use in components.  I plan to add capability to mqtt_statestream to select the items that are published, and I thought it made sense to break the code out so that it could be used in other modules.
It's inspired by the include/exclude options in the recorder component (if this is accepted, I'll submit a PR to update the component to use this class), but with the cases a little more formalized:

If there are no includes or excludes specified, return true.
If there are only includes (domain and/or entity), then only return true for the specified domains/entities, otherwise return false.
If there are only excludes (domain and/or entity), then only return false for the specified domains/entities, otherwise return true.
If both includes and excludes are specified:
a. If included domains are specified:

If the entity's domain is included, and the entity is not excluded, return true.
If the entity's domain is not included, and then entity is not specifically included, return false.
b. If excluded domains are specified:
If the entity's domain is excluded, and the entity is not specifically included, return false.
If the entity's domain is not excluded, and the entity is not specifically excluded, return true.
c. Neither included or excluded domains are specified:
If the entity is specifically included, return true.

I've added a FILTER_SCHEMA to config_validation that can be used to set up the config parsing.
To get started:
from homeassistant.helpers.entityfilter import EntityFilter
import homeassistant.helpers.config_validation as cv

CONFIG_SCHEMA = vol.Schema({
    DOMAIN: cv.FILTER_SCHEMA.extend({
      ...
    })
})

Then, in your component/platform setup:
...
    conf = config.get(DOMAIN, {})
    my_filter = EntityFilter(conf.get(CONF_INCLUDE,{}), conf.get(CONF_EXCLUDE, {}))
...

When you want to check an entity:
  if my_filter.check_entity(entity_id):
    """Do something here for valid entities."""
  else:
    """It returns False for ignored entities."""

I couldn't figure out where would be appropriate for documentation - if you let me know, I'll add a PR for home-assistant.github.io to add them.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
your_component:
  include:
    domains:
      - light
      - sensor
    entities:
      - binary_sensor.sensor1
  exclude:
    entities:
      - light.badlight

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Hi, @balloob - thanks for the notes.  I've made those changes, except for this one - see my comment there.
I've also converted the recorder component over to use the EntityHelper class, and the existing tests pass.  It looks like there are quite a few other components using the include/exclude pattern, particularly the logging ones (Prometheus, InfluxDB, etc).  Once everything is nailed down, I'll start on converting the rest.

@balloob - I've updated the entity filter helper to return a function, and updated the recorder component.

Thanks @balloob !  For the updates to the other components using filters, would you like individual PRs, or just all in one?

Prefer individual ones.


Release 1.2.1

This release fixes bug #583, which was reported in #582 and #579 and #598.

coverage: 99.726%. remained the same when pulling 4f564f037da931f250b87f9e102cd571db92f68d on release-1.2.1 into 1b34599e9812b3aa55a2f5010e086e56cd62ad0a on master.


#83: Fix native image error with dependencies

This PR fixes the not reachable additional dependencies like commons-compress.
Fixes #83

Could you test the exe built with this commit before merging it? Because although the unzip methods worked and oc is installed correctly, I am not sure if this single upgrade of the native-maven-plugin version fixes all the errors
You can find the build executable for windows here: https://github.com/alfeilex/IDEasy/releases/download/test/ide-cli-windows-x64.tar.gz


Unknow issue when running prompt inside the webui with WSL Ubuntu

Make sure that you have set your OPENAI API Key

Make sure that you have set your OPENAI API Key

I did but it's not working with Claude

it should be LLM_API_KEY: https://github.com/opendevin/opendevin?tab=readme-ov-file#picking-a-model

Make sure that you have set your OPENAI API Key

I did but it's not working with Claude

ensure this command prompt claude POST https://api.openai.com/v1/completions -H "Content-Type: application/json" -H "Authorization: Bearer YOUR_API_KEY" -d "{\"prompt\":\"Hello, world!\"}"
and replace YOUR_API_KEY with your actual OpenAI API key.

This is looking like user error--we do need to rework the key/model interaction a bit though. E.g. only make models available if the user has plugged in a key, or prompt for a key when choosing a model.
I'll open a PR


[WIP] Modified volume configuration page

appended possibility of creating new volume.
because i am new in ecmascript 6 and reactjs, this code not final.
I wait some comments from project developers

Can you add [WIP] to the title, so we know it's not final - Also would you mind signing your commit to comply with our Contribution Guidelines?

Hi @bessdsv, are you still working on this? Please lmk if you need some help, I would be happy to help.


Adding 4, 5, 6

Should be updated to use node_js key, but I really just want to see if this is building on these versions and I'm too lazy to do it manually

I have a feeling I know how this ends... https://travis-ci.org/tessel/t2-cli/jobs/125957491

Whooooooops, wrong repo.


The "quick start" redirect in readme is broken

the redirect doesnt follow the latest git wiki, either update it or remove it

thanks, fixed in f37866eb7ecf0ddb9b1924cbbc15af90143a24d3


Cannot download dataset programmatically

Hello,
I have registered for the dataset using my gmail account, and I can now see the cloud storage console and download files manually, but whenever I try to download files programmatically (e.g., using the requests module or the cloud storage python sdk), I get authentication errors. I already tried to make a service account corresponding to the email I signed up with, but it does not seem to be connected to the dataset's cloud storage. I also tried registering under a different email at https://waymo.com/open/licensing/, but the form appears to be broken, as I cannot check the box to accept the licensing agreement.
Any help would be greatly appreciated! I am mostly looking for help on how to download select .tfrecord files programmatically using the cloud storage python sdk if possible.
Thank you!

Hi @nicomon24,
Thank you for the reply, that stackoverflow link is definitely useful!
I tried getting the blob directly, but I am still getting permission denied errors. I think I may be using the wrong credentials however, as I made a service account to be able to use the Python SDK, but this presumably does not have access to the dataset. Do you have any pointers on how to authenticate with Google cloud using my email, as I assume this is how I am able to access the web console?
If it helps, this is the code snippet I am using:
from google.cloud import storage
from google.oauth2 import service_account

credentials = service_account.Credentials.from_service_account_file("<path-to-service-account-credentials.json>")

storage_client = storage.Client(project="<project-name-I-made>", credentials=credentials)

bucket = storage_client.bucket("waymo_open_dataset_v_1_4_3")

bucket.get_blob("individual_files/training/segment-10017090168044687777_6380_000_6400_000_with_camera_labels.tfrecord")

The final last throws the following exception:
Forbidden: 403 GET https://storage.googleapis.com/storage/v1/b/waymo_open_dataset_v_1_4_3/o/individual_files%2Ftraining%2Fsegment-10017090168044687777_6380_000_6400_000_with_camera_labels.tfrecord?projection=noAcl&prettyPrint=false<EMAIL_ADDRESS>does not have storage.objects.get access to the Google Cloud Storage object. Permission 'storage.objects.get' denied on resource (or it may not exist).

Thanks!

Related stackoverflow hhttps://stackoverflow.com/questions/44328277/how-to-auth-to-google-cloud-using-service-account-in-python#comment128852228_44329070
The service account being different from your google account should probably be the issue I guess

I think you'd have to use OAuth to obtain the access token (with your Google account that signed into the Waymo dataset) and use that access token to issue a RESTful request like documented here: https://cloud.google.com/storage/docs/downloading-objects#rest-download-object
I used https://developers.google.com/oauthplayground/ to get myself started without writing a ton of code handling OAuth. This approach requires you to authenticate (like you logged in with your Google account for other websites) every time you want to initialize the batch download, which means you can't use them on servers and run as parallel programs.

Thanks @nicomon24 and @haoming29 for your input!
What ended up working was installing google-cloud-sdk and running gcloud auth application-default login to log in with the Google account I used to register for the dataset. This generated the required environment variables to be able to run the following Python code to download and process a .tfrecord file:
import tempfile
from google.cloud import storage

client = storage.Client(project=None)
bucket = client.bucket(bucket_name="waymo_open_dataset_v_1_4_3")

with tempfile.NamedTemporaryFile() as temp_file:
    blob = bucket.get_blob(blob_name="<path to .tfrecord file in bucket>")
    blob.download_to_filename(filename=temp_file.name)

    # Code to process .tfrecord file now downloaded to temp_file

Note: I had to use the download_to_filename() method instead of the .download_to_file() method, as the latter frequently resulted in incomplete files leading to a TensorFlow DataLossError. I didn't investigate further into this issue, as using .download_to_filename() fixed the issue.


Docker

alternative to https://github.com/CCPPETMR/SIRF/issues/29

not yet finished with this

OK this works on linux out-of-the-box. docker pulls and builds an image which takes up about
On windows (as noted in the wiki) you'd have to install a desktop / vnc server within the container and connect for a GUI. Will add docker files for that later...

@bathomas

removed 8 from make -j
wiki updated with --build-arg notes
--no-install-recommends added, possibly reduces image size by <1% but I'll leave it there anyway

Note that an image based on alpine might be relatively painless to create and would be roughly half the size but also half the speed when running, so I don't think it's worth it.

There is obviously overlap with CCPPETMR_VM. Most obvious is https://github.com/CCPPETMR/CCPPETMR_VM/blob/master/scripts/INSTALL_prerequisites_with_apt-get.sh, but also some other content of its Vagrantfile.
It makes me wonder if the Dockerfile belongs in https://github.com/CCPPETMR/CCPPETMR_VM, or at least should use some of it.
For discussion tomorrow (if there's time)

I feel we should really have the VM repo deprecated and vagrantfiles/dockerfiles etc put here. I think we raised this point before but not sure what the conclusion was.

it's a possibility of course. I don't have a strong opinion either way. Yet another discussion point!

I agree with @casperdcl that alpine images are not really worth it yet. Perhaps something to keep in mind for the future. I recently built ITK in an alpine container and it is considerably smaller than with ubuntu. In theory I suppose we could use Docker hub's automated building, but in reality I suspect the build would not complete within the current 2 hour/1 core limit.

Yes...
To get around issues with installing gadgetron in windows (that's still an issue, right?) I was also thinking of splitting gadgetron into its own container service and connecting to it. That would mean Windows users could install everything locally and just have gadgetron in a container.

windows: yes, that's what we currently do, but via a VM, not Docker. Gadgetron does supply a docker image, so for people having docker on Windows (not trivial as dicussed before) that'd work as well. Probably worth documenting, if someone ever tries this out.

@KrisThielemans yes we could move then to a scripts subdir but 1) if that's in a different repo then yes, would have to install git and clone first. 2) would have to move scripts to a subdir of the docker source in order to be visible to the Dockerfile.
Do you think we should rely less on the SuperBuild and actually USE_SYSTEM_* more (like in .travis.yml)? This would reduce build time and image size, but might mean more pain for us maintainers to update.

I think USE_SYSTEM_* is fine for inside Docker or Virtualbox, as we have control over the OS and packages (ok, not really if there's an apt-get update in there). We had this discussion for Vagrant as well. @bathomas, opinions?

So I suppose all issues here are now sorted apart from USE_SYSTEM_*?

yes. looks fine to me. We can do the USE_SYSTEM_* later. Squash-merge?

Yes, I've been at the summer school thing so been contributing via terminal on phone. Hard to test things... Will --no-ff merge, as it's better to track the feature branch history without making it hard to see master history. Already rebased to remove superfluous commits.


Added traits for Repeated

I implemented a handful of From traits for the Repeated type. Also I added in the to_vec function which a todo comment called out.

There were ~6 other files touched when I ran cargo fmt it was still passing tests tho.

Could you revert formatting please? Rustfmt is not used in rust-protobuf because of issues in rustfmt.

Sorry got involved with RL stuff.

OK, I'll do that myself.

I cherry-picked first of your commits. Thanks!


Introdução ao git

Git is a free and open source distributed version control system designed to handle everything from small to very large projects with speed and efficiency.
About git

https://git-scm.com/video/what-is-version-control


Building probe-run against specific defmt

A couple times, I've found it necessary to build my own probe-run (most recently, to use a newer probe-rs), but doing that leads to a defmt version mismatch problem that can be difficult to resolve.  Is there a way to build probe-run against specific version of defmt?
Specifically, if I do cargo install probe-run, I currently get probe-run 0.2.1, with supported defmt version 0.2, however doing cargo install --path . in probe-run source checked out at v0.2.1 yields a probe-run with supported defmt version 60c6447f8ecbc4ff023378ba6905bcd0de1e679f .
I've tried using Cargo's patch section to update my firmware's defmt version from only one place, but haven't managed to get that working.

Hi @ianrrees, Can you please check if #174 is a viable workaround for you?

Yep, works great, thanks!


Refact: expected

Changing implementation from std::variant to a custom union type.
Marking as experimental (due to unfinished features: rvalues)
Added monadic operations

Missing version bump.


doc: Add binaries dashboard (install instructions)

Context
Following up on discussion in https://github.com/atom-community/atom/pull/456.
Description of the Change
Add link to a nice binaries dashboard to the Installation.md instructions.
(Dashboard is originally by @autumnblazey, but not maintained by them anymore. atom-community org is welcome to fork the dashboard repo if there is a need to make updates.)
Release Notes
N/A

(Dashboard is originally by @autumnblazey, but not maintained by them anymore. atom-community org is welcome to fork the dashboard repo if there is a need to make updates.)

"but not maintained by them anymore": it is still up, no transfers done.


Bump macOs version to macos-11

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
Resolve https://github.com/Azure/azure-sdk-tools/issues/3715
Tracking issue: https://github.com/actions/virtual-environments/issues/5583
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

@praveenkuttappan any reason why this PR reported back API changes from APIVIew?


Hackathon project

Yo @brandonshin, so I made a buttload of changes to get this thing to work in a slightly more modular way. I'm thinking that I'd be willing to take over the slackbot workout project if you feel like you don't want to maintain it anymore, since I'll probably be tinkering with it bit by bit and would be interested in keeping it up to date.
The big changes are that the different functional elements have been separated into different modules. Some new functionality has been added as well:

There's no need for the URL token, you can just use the user token to communicate with the slack channel. This token can be taken from the environment or hard-coded into the file.
 This is useful for me since at my company we wanted to have a small dashboard showing people's participation, and it is easy to collect that from a database.
There's now a UserManager which is the main interface to the bot for looking up the users on a channel and seeing if they are active.
I got rid of the user cache since it doesn't really save all that much time, as you're still getting all of the information about users each iteration anyways, and past exercises are being logged.
The algorithm for selecting a user and the next exercise time is quite a bit different. The time is adaptive based on the number of users and the number of exercises they've done. When I used your version at work, we ran into an issue where if there were only one or two people on the channel, they would get blasted with exercises. This imposes a max cap on exercises so that doesn't happen. In the future per-user limits would be great!
This PR also adds a webserver which allows the bot to use outgoing webhooks to respond to messages from the channel. this lets people query the bot during the day and ask how people have been performing over the course of the day.
The setup.py file adds the ability to make wheels and upload them to PyPI.

Hey @mgyucht I'm giving you contributor access 👌

@mgyucht Any chance some of those fixes can get merged in?

Yeah, so I guess I'd defer to @brandonshin on this. I've kind of taken this
project in my own direction, but if you guys like what I've done with it,
I'd be willing to merge my work into this repository as well.
On Thu, Jan 21, 2016, 3:42 PM Steven Nunez<EMAIL_ADDRESS>wrote:

@mgyucht https://github.com/mgyucht Any chance some of those fixes can
get merged in?
—
Reply to this email directly or view it on GitHub
https://github.com/brandonshin/slackbot-workout/pull/37#issuecomment-173751340
.


[Improvement]: Configure iceberg.worker.num-threads in the configuration file

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
When Runtime.getRuntime().availableProcessors() is not obtained correctly or the obtained value is less than or equal to 2, the default value of iceberg is 2, AMS is congested in thread iceberg-work-pool-xx.
How should we improve?
Add parameters in config.yaml to overwrite the value of iceberg.worker.num-threads.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Subtasks
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

This is a java property used by the Iceberg and it can be set in ams.sh like '-Diceberg.worker.num-threads=xxx'.


Circular dependencies in UI

In the UI there is a circular dependency. The following is the warning message:

WARNING in Circular dependency detected:
src/app/myworkflows/myworkflows.service.ts -> src/app/workflow/register-workflow-modal/register-workflow-modal.component.ts -> src/app/myworkflows/myworkflows.service.ts
WARNING in Circular dependency detected:
src/app/workflow/register-workflow-modal/register-workflow-modal.component.ts -> src/app/myworkflows/myworkflows.service.ts -> src/app/workflow/register-workflow-modal/register-workflow-modal.component.ts

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.8
┆Sprint: Seabright Sprint 26 Abalone
┆Issue Number: DOCK-1100

Doesn't really need verification on staging


Add Navigating the MLOps Landscape Part Three

Talk about technology radar from thoughtworks, what each represents
Quadrant 4  (Adopt), diagonal L->R down (Trial), Quadrant 2 (Assess), upper-rightmost HOLD
Buy or build


Add a minimal default style for tables

Expected behaviour
Markdown tables must have a minimal default style
Current behaviour
If I use this table markdown
| Tables        | Are           | Cool  |
| ------------- |:-------------:| -----:|
| **col 3 is**  | right-aligned | $1600 |
| col 2 is      | *centered*    |   $12 |
| zebra stripes | ~~are neat~~  |    $1 |

The result is:

Inspecting the web, I can see that table does not have any class or style

But in demo https://markdown-it.github.io/ , the table looks pretty due to this class:
class="table table-striped"

And the bootstrap.css

Workaround 1
Add the bootstrap.css, not everyone can use it
Workaround 2
Add class to all  using regex:
  var html = markdownConverter.render(markdownString);
  html = html.replace(/<table>/g, '<table class="styled-table">');

and load this css file
.styled-table {
  font-size: 100%;
  color: #222;
  ...etc
}

Propose
Maybe a possibility to add the table class or has a default inlined style for tables
Thanks for your awesome library!

That's intended. We thinked a lot about that before implementation. And found impossible to provide universal solution out of box. So, we did minimal possible intrusion, to let users play with renderer rules as they wish.
https://github.com/markdown-it/markdown-it/blob/7b8969ce5cb2edc54f2c1aa39a85a3a08076337d/support/demo_template/index.js#L157-L159


MM-49477_Able to open team creation window after trial ends

Summary
Should not be able to open team creation window after page is notified about the trial end.
Ticket Link
https://mattermost.atlassian.net/browse/MM-49477
Screenshots

Release Note
NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

/e2e-test

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

/e2e-test

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

Closing this as the root cause of the issue should be solved in https://mattermost.atlassian.net/browse/MM-49936


Create versioned Alertmanager documentation

Related PR: Prometheus #1084

@lucperkins Thanks! The previous attempt of this PR (https://github.com/prometheus/alertmanager/pull/1062) was blocked because @grobie mentioned that this still depends on a rewrite of the docs infrastructure (including nanoc4 upgrade). See this comment: https://github.com/prometheus/alertmanager/pull/1062#issuecomment-360799938
Are you aware of any change regarding that?
@grobie I know you're super busy, but could you give any indication of what should happen here, or if someone else should take over?

As far as I can tell a nanoc upgrade shouldn’t be necessary, and the infrastructure in the docs repo will be taken care of in the PR mentioned in this PR’s description.

@lucperkins Yeah, I'm not sure, but probably you are right. Maybe @grobie's concern was just about the site build process taking long with the external docs that are cloned every time, and that it would get even slower when extracting the AM docs. Maybe he had ideas of optimizing all that, but I'm just guessing now :)


[Chery-pick][branch-2.0] [Feature] Add InsertTxnCommitAttachment to support version rollback

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Feature is forbidden to cherry-pick into the lists branch.


Update CodeQL GitHub Actions

Set top-level permissions. Remove unneeded comments.

Thank you, @goneall!


Cannot choose which device to run on when multiple devices are running

Is there an existing issue for this?

[X] I have searched the existing issues and didn't find mine.

Steps to reproduce
Steps to reproduce:

Open multiple devices e.g. x1 iOS simulator & x1 Android Emulator.
Run the maestro test yourFlow.yaml command.
Observe no Multiple running devices detected. Choose a device to run on. prompt.

Actual results
When I have multiple devices running, I cannot select which device to run the flow on with the maestro test command anymore.
If I have both an iOS simulator and android emulator running and I want to run X flow on the android emulator, I need to close the iOS simulator first, vice versa.
Expected results
When I have multiple devices running, I should be prompted with:
Multiple running devices detected. Choose a device to run on.
Enter a number from the list above:

The maestro studio command still allows this functionality.
I do not require any devices to be closed to run X flow on the correct device.
About app

Native iOS & Android app

About environment

MacBook Pro M2 Max - 32 GB
macOS Sonoma 14.6
arm64

Maestro version
1.37.4
How did you install Maestro?
install script (https://get.maestro.mobile.dev)
Anything else?
1.36.0 did not have this issue.
Occurred since 1.37.0 release.

+1

I confirm this issue occurs on (tested on v1.37.4).
I verified this is a regression introduced in #1723 (commit 5f994dacc573d33c079daf3069193da7a03b9f09).
The issue does not occur on the commit right before it: 5f994dacc573d33c079daf3069193da7a03b9f09

I've noticed whilst attempting to run a flow on an iOS simulator the error (Unable to launch app xxxxxxxxxxxx: Package xxxxxxxxxxxx is not installed) occurring. This only occurs when I have both an Android emulator and iOS simulator device running and then attempt to run an iOS flow (with iOS specific app id).
I believe this is because it's attempting to launch the iOS specific flow on the Android emulator rather than the iOS simulator.
Fixing this issue should resolve this as when we have
When I have multiple devices running, we should be prompted with:
Multiple running devices detected. Choose a device to run on.
Enter a number from the list above:


Added installation guide for react-native >=0.60

Overview

Added installation guide for react-native >= 0.60
Test Plan

No plans.

Thank you for this PR. I'm working on the revamped version of the library in the next branch. As it will include complete docs revamp, and current docs have been working well so far, I'm closing this PR. In case you'd like to participate in further development, feel free to reach out to me 😃


postCreateCommand hangs when user interaction is needed

Version: 1.64.0 (Universal)
Commit: 5554b12acf27056905806867f251c859323ff7e9
Date: 2022-02-03T04:20:17.224Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 19.6.0
Reproduces in: Codespaces
Name of Dev Container Definition with Issue: mcr.microsoft.com/vscode/devcontainers/python:3.10

I have a script I added in Dockerfile in location /usr/local/bin/workspace-one-time-setup.sh. The script has a header of #!/bin/bash -i
In my devcontainer.json file I have "postCreateCommand": "bash -i /usr/local/bin/workspace-one-time-setup.sh",
In the /usr/local/bin/workspace-one-time-setup.sh, it auto starts sso sign-in that requires user interaction. Because of that, it keeps the script running indefinitely

I have to view creation logs to see it's waiting for my interaction to sign me in. Is there a way I can bring it to the foreground when my interaction is needed? or better yet execute the script in the foreground

There's a way to add finer-grained progress/output to that terminal output by having your postCreateCommand script output some specially-formatted lines to stdout with a ::step:: prefix. Would this help you out? The View Creation Logs is the proper way to see the full script output while it is running, but the shorter status update could be used to let the user know that they need to take some action to proceed.
Example:
echo "::step::Waiting for SSO at <url>"

Right now user interaction during any of the automated commands is not available, so this is not related to the images in this repository.
With that in mind, for Remote - Containers I'd move it to https://github.com/microsoft/vscode-remote-release as a feature request. However, given this is related to Codespaces, I'd suggest raising a feature request at https://github.com/github/feedback/discussions/categories/codespaces-feedback

@jkeech ah, I will try that..Is there any documentation for that?
@Chuxel  good point...I guess it will be better to move it to vscode remote release as a feature request

@wobeng Codespaces and Remote - Containers generally have similar capabilities, but the UX here is one area they differ. Remote - Containers does not cover up the details of the execution like Codespaces does, like Codespaces it does not allow user interaction either.
That said, in checking, there's an existing feature request for this in Remote - Containers you can up-vote here: https://github.com/microsoft/vscode-remote-release/issues/5225
It would be worth opening one in the codespaces feedback channel to track the request there. Unfortunately, I can't transfer this there myself.
Closing as a dupe - but understand the request.


Platform usage compatibility

With this package, would we be able to develop a hook library for the kea dhcp server for linux? If so, we would be willing to create one and share it on gtihub as a good real-world working example for others to learn.
Hooks Developer's Guide

would we be able to develop a hook library for the kea dhcp server for linux?

@johnwc Yes! That is an example of where DNNE could be a good solution I think. In fact, I would really appreciate someone helping to validate more complex scenarios on the linux platform. Let me know if there is anything I can help with.
/cc @adityamandaleeka @richlander

Awesome! I will try to put something together to test within the next week or two, will get back with you.

Is it possible to compile a linux library using VS in windows? Or do I need to be copying the files to a linux machine to compile it?

Is it possible to compile a linux library using VS in windows?

@johnwc Nope. Unfortunately that is a really complicated issue. However, this has been asked before at https://github.com/AaronRobinsonMSFT/DNNE/issues/84#issuecomment-924191881 and I provided some details.

Or do I need to be copying the files to a linux machine to compile it?

The DNNE tooling should "just work" on Linux as long as clang is on the path—typically the default once installed. Using gcc is possible using the following MSBuild property.
https://github.com/AaronRobinsonMSFT/DNNE/blob/d48209b02e956b3a56b19dc6672e56ccfeefaf55/src/msbuild/DNNE.props#L51-L54
The CI leg compiles and runs on Linux.
https://github.com/AaronRobinsonMSFT/DNNE/blob/d48209b02e956b3a56b19dc6672e56ccfeefaf55/.github/workflows/main.yml#L10-L25

I created a project and pushed it to GH. I was able to get just a plane sample to compile in linux. Then added the hooks.h include, received a lot of errors. Changed to to use g++ from clang, most of the errors disappeared. But now get this single error. Any ideas?
My background: I am not at all new to programming. Just not very experienced in C++, but have done quite a bit with managed -> unmanaged calls. Have been developing in c# mostly for past 15 years, so feel free to get as technical as needed in conversation.
$ dotnet build -c Release
Microsoft (R) Build Engine version 16.9.0+5e4b48a27 for .NET
Copyright (C) Microsoft Corporation. All rights reserved.

  Determining projects to restore...
  All projects are up-to-date for restore.
  Building native export: "g++" -O2 -shared -fpic -D DNNE_ASSEMBLY_NAME=Test -D DNNE_COMPILE_AS_SOURCE -I "/home/jcarew/.nuget/packages/dnne/1.0.27/tools/platform" -I "/usr/lib64/dotnet/packs/Microsoft.NETCore.App.Host.centos.8-x64/5.0.12/runtimes/centos.8-x64/native" -I "/usr/include/kea" -o "/home/jcarew/hooks/obj/x64/Release/net5.0/dnne/bin/TestNE.so" "/home/jcarew/hooks/obj/x64/Release/net5.0/dnne/Test.g.c" "/home/jcarew/.nuget/packages/dnne/1.0.27/tools/platform/platform.c" -lstdc++ "/usr/lib64/dotnet/packs/Microsoft.NETCore.App.Host.centos.8-x64/5.0.12/runtimes/centos.8-x64/native/libnethost.a"
  /home/jcarew/hooks/obj/x64/Release/net5.0/dnne/Test.g.c: In function ‘int32_t FancyName(int32_t)’:
/home/jcarew/hooks/obj/x64/Release/net5.0/dnne/Test.g.c(72,59): error G7E4560F8: invalid conversion from ‘void*’ to ‘int32_t (*)(int32_t)’ {aka ‘int (*)(int)’} [-fpermissive] [/home/jcarew/hooks/Test.csproj]
           FancyName_ptr = get_fast_callable_managed_function(t1_name, methodName);
                           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
  /home/jcarew/.nuget/packages/dnne/1.0.27/tools/platform/platform.c:34: warning: "_GNU_SOURCE" redefined
       #define _GNU_SOURCE

  <command-line>: note: this is the location of the previous definition
/home/jcarew/.nuget/packages/dnne/1.0.27/build/DNNE.targets(145,5): error MSB3073: The command ""g++" -O2 -shared -fpic -D DNNE_ASSEMBLY_NAME=Test -D DNNE_COMPILE_AS_SOURCE -I "/home/jcarew/.nuget/packages/dnne/1.0.27/tools/platform" -I "/usr/lib64/dotnet/packs/Microsoft.NETCore.App.Host.centos.8-x64/5.0.12/runtimes/centos.8-x64/native" -I "/usr/include/kea" -o "/home/jcarew/hooks/obj/x64/Release/net5.0/dnne/bin/TestNE.so" "/home/jcarew/hooks/obj/x64/Release/net5.0/dnne/Test.g.c" "/home/jcarew/.nuget/packages/dnne/1.0.27/tools/platform/platform.c" -lstdc++ "/usr/lib64/dotnet/packs/Microsoft.NETCore.App.Host.centos.8-x64/5.0.12/runtimes/centos.8-x64/native/libnethost.a" " exited with code 1. [/home/jcarew/hooks/Test.csproj]

Build FAILED.

/home/jcarew/hooks/obj/x64/Release/net5.0/dnne/Test.g.c(72,59): error G7E4560F8: invalid conversion from ‘void*’ to ‘int32_t (*)(int32_t)’ {aka ‘int (*)(int)’} [-fpermissive] [/home/jcarew/hooks/Test.csproj]
/home/jcarew/.nuget/packages/dnne/1.0.27/build/DNNE.targets(145,5): error MSB3073: The command ""g++" -O2 -shared -fpic -D DNNE_ASSEMBLY_NAME=Test -D DNNE_COMPILE_AS_SOURCE -I "/home/jcarew/.nuget/packages/dnne/1.0.27/tools/platform" -I "/usr/lib64/dotnet/packs/Microsoft.NETCore.App.Host.centos.8-x64/5.0.12/runtimes/centos.8-x64/native" -I "/usr/include/kea" -o "/home/jcarew/hooks/obj/x64/Release/net5.0/dnne/bin/TestNE.so" "/home/jcarew/hooks/obj/x64/Release/net5.0/dnne/Test.g.c" "/home/jcarew/.nuget/packages/dnne/1.0.27/tools/platform/platform.c" -lstdc++ "/usr/lib64/dotnet/packs/Microsoft.NETCore.App.Host.centos.8-x64/5.0.12/runtimes/centos.8-x64/native/libnethost.a" " exited with code 1. [/home/jcarew/hooks/Test.csproj]
    0 Warning(s)
    2 Error(s)

Time Elapsed 00:00:02.86

@johnwc The issue above is that you are using g++ and not gcc. The generated source for DNNE is pure C99, not C++.

@johnwc The precise issue at hand here is that C++ has stricter casting rules than C99. The reasoning behind using C99 instead of C++ is it is the lingua franca for most of this embedding work. C++ has some subtle issues that can surprise and making a "safe" C++ API is non-trivial and generally winds up being mostly C with some C++ niceties. Unfortunately these niceties typically introduce C++ features that aren't expected (e.g., exceptions).

I'm currently looking at the hooks.h and see that the entire API being used here is C++. Let me see what I can do about making the generated code more C++ friendly. It could be simply adding the appropriate casts.

@johnwc I added a build/test for both gcc and g++ on Linux. This required supporting compiling and linking as C++. I've only added testing for g++ but will likely add more if I hear more. A new package with these changes has been published, https://www.nuget.org/packages/DNNE/1.0.28.

I've only added testing for g++ but will likely add more.

Way easier than I thought. There are now tests for compiling as C++ on all platforms.

This is great! With that update, I was able to create the version() method and return the same value that is set for KEA_HOOKS_VERSION in the hooks.h. I also added a simple File.AppendAllLines to the method to have a log to prove it was being called. I compiled just fine, and worked as a a hook when configured and the service restarted. I will attempt to write a simple command hook and see how it goes.

@AaronRobinsonMSFT how do I handle the LibraryHandle class as a parameter for load(LibraryHandle& libhandle)? I don't seem to be able to use classes for parameters with UnmanagedCallersOnly

@johnwc This will get very complex and depend on how the Kea types want to be projected into managed code. In this case, the LibraryHandle seems to have some complexity that is going to make thing complicated–uses many C++ types and some are passed around by value (that is, std::vector<std::string> getParameterNames();). The int field is easy, but the CalloutManager& field makes natural projections into .NET difficult. There are many options but let me sketch one out for you below. Note that if Kea types are being projected into .NET that involve inheritance the complexity increases substantially.
unsafe struct LibraryHandle
{
    // This function pointer can be acquired via a DllImport to dlsym, GetProcAddress, or the NativeLibrary API.
    // Another approach would be to create a DllImport into the generated native binary being compiled by DNNE.
    // Then pass all the parts back into native code and make the call as expected.
    // See the readme about overriding dnne_abort. The point here is you can pass a .c or .cpp file
    // to compile into the generated native component and then DllImport into that function.
    private static ??? registerCalloutFptr;

    public static void registerCallout(LibraryHandle* h, string name, delegate*<void*, int> callout)
    {
        // Marshal the string to an byte* using System.Text.UTF8Encoding and a fixed statement.
        UTF8Encoding utf8 = new();
        byte[] encodedBytes = utf8.GetBytes(name);
        fixed (byte* b = encodedBytes)
        registerCalloutFptr(h, b, callout);
    }
}

[UnmanagedCallersOnly]
private static int Callback(void* ptr)
{
    return 0;
}

[UnmanagedCallersOnly]
public static unsafe ??? load(void* libhandle)
{
    LibraryHandle* h = (LibraryHandle*)libhandle;
    LibraryHandle.registerCallout(h, "name", &Callback);
    ...
}

Sounds like I will have to create proxy methods like described here.
class Foo {
public:
  int Bar(int a, int b);
};

// Proxy methods
extern "C" int Foo_Bar(Foo* pFoo, int a, int b) { return pFoo->Bar(a, b); }

static class NativeFoo {
    [DllImport("Foo", EntryPoint = "Foo_Bar")]
    public static extern int Bar(IntPtr obj, int a, int b);
}


Problem with Gogs avatars

Hi,
I have a problem with Gogs 0.8.25 <-> Drone 0.4.0 integration.
I think it's related to #1263.
The problem is that Drone is rendering wrong URL's for Gravatar avatars:
http://gogs.domain.com/https://secure.gravatar.com/avatar/1a397956970bf0945881510b7f5280ae

Custom avatar images are working ok.

yes, I've noticed something changed in the latest Gogs release that has broken avatars. It likely has something to do with this code:
https://github.com/drone/drone/blob/master/remote/gogs/helper.go#L122
and related unit test:
https://github.com/drone/drone/blob/master/remote/gogs/helper_test.go#L146

Just reviewed a webhook sent by Gogs. This URL comes from the webhook, so it's definitely Gogs fault. And it seems it's fixed already: https://github.com/gogits/gogs/issues/2630
So issue can be closed, thanks.


Roll Flutter Engine from acdb364a42d5 to 854b3747a07b (2 revisions)

https://github.com/flutter/engine/compare/acdb364a42d5...854b3747a07b
2023-09-26<EMAIL_ADDRESS>Clean up the docstring for ColorFilter.matrix (flutter/engine#46298)
2023-09-26<EMAIL_ADDRESS>Bump uuid from 4.0.0 to 4.1.0 in /lib/web_ui (flutter/engine#46293)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Unnecessary vendor prefixing in Safari

I have an element with the property: textDecoration: 'none', but when running it in Safari I get the following console warning: Unsupported CSS value "none" for property "WebkitTextDecoration"
This version of Safari supports textDecoration unprefixed, so I am not sure why this error occurs in the first place. Interestingly, though, the desired style is still applied.

Fun! Another weird one. Should be solved by #374.

Fixed with use of inline-style-prefixer.


feat: add rag-pgvector example

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


The 4.2.x version jar in not available on jitpack.com

What am I experiencing
following the instructions according to the documentation to set up a java maven spring boot project
What am I using

jitpack.io
https://jitpack.io

com.github.h-omer
neo4j-versioner-core
4.2.4
provided

How to reproduce the problem
The project build fails with error :
Failed to execute goal on project Could not resolve dependencies for project XX:jar:0.0.1-SNAPSHOT: Could not find artifact com.github.h-omer:neo4j-versioner-core:jar:4.2.4 in jitpack.io (https://jitpack.io)
error on jitpack
Build starting...
Start: Tue Mar 30 07:55:04 UTC 2021 1023b9c3c1f8
Git:
4.2-0-g443bd9e
commit 443bd9e0cc9fd3d405539e9397a44505245d66ca
Author: mfalcier
Date:   Wed Mar 24 09:44:20 2021 +0100
:books: Added version compatibility matrix

Found gradle
Gradle build script
Found gradle version: 6.7.
Using gradle wrapper
Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8 -Dhttps.protocols=TLSv1.2
Downloading https://services.gradle.org/distributions/gradle-6.7-bin.zip
.10%.20%.30%.40%.50%.60%.70%.80%.90%.100%

Gradle 6.7
Build time:   2020-10-14 16:13:12 UTC
Revision:     312ba9e0f4f8a02d01854d1ed743b79ed996dfd3
Kotlin:       1.3.72
Groovy:       2.5.12
Ant:          Apache Ant(TM) version 1.10.8 compiled on May 10 2020
JVM:          1.8.0_252 (Private Build 25.252-b09)
OS:           Linux 4.18.0-25-generic amd64
0m3.290s
Getting tasks: ./gradlew tasks --all
Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8 -Dhttps.protocols=TLSv1.2
Tasks:
Found javadoc task
WARNING:
Gradle 'install' task not found. Please add the 'maven' or 'android-maven' plugin.
See the documentation and examples: https://jitpack.io/docs/
Looking for android-library
Looking for com.android.application
Adding maven plugin
Running: ./gradlew clean -Pgroup=com.github.h-omer -Pversion=4.2.4 -xtest install
Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8 -Dhttps.protocols=TLSv1.2

Task :clean UP-TO-DATE
Task :compileJava FAILED

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':compileJava'.

Could not target platform: 'Java SE 11' using tool chain: 'JDK 8 (1.8)'.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.7/userguide/command_line_interface.html#sec:command_line_warnings
BUILD FAILED in 12s
2 actionable tasks: 1 executed, 1 up-to-date
Build tool exit code: 0
Looking for artifacts...
Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8 -Dhttps.protocols=TLSv1.2
Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8 -Dhttps.protocols=TLSv1.2
Looking for pom.xml in build directory and ~/.m2
2021-03-30T07:55:26.569761427Z
Exit code: 0
ERROR: No build artifacts found

Hi, I just added neo4j versioner core 4.2.4 as a dependecy in my project, using gradle, without having any issue.
Can you please share your build.gradle file or your pom.xml?

@mfalcier Hi, Is this newest version of neo4j versioner core will work with neo4j version 5.x?
if not, how about the compatibility with other neo4j 4.x versions (4.4.x)?
Thanks


[WIP] ci: remove add-pr-link related actions

Problem
We added add-pr-link for tracking commits. Actually we can find the PR number in the web page

or if we have setup gh, we can use this command to find it as well
gh pr list --search <SHA> --state merged --repo solana-labs/move

I guess this action is a little bit trivial atm. (also a little bit aggressive cuz we just modify the original PR with a third party)
Summary of Changes
Remove all add-pr-link related code.

@ksolana this one just a draft. we can discuss more if you have any concerns!

how do you feel about this workflow:

we git log and search something
we find a commit which we are interested
use gh pr list --search <SHA> --state merged --repo solana-labs/move to get the PR number

I still feel it's not a very good idea to modify all commits to a PR.

the PR number is incremental. if you want to keep a number, I guess you need to create it first or someone will take the number.
or maybe we can have a script which looks like
echo "https://github.com/solana-labs/move/pull/$(gh pr list --search <SHA> --state merged --repo solana-labs/move --json number | jq '.[0].number')"

it can print the pr links for us as well. (the only requirement is to setup gh cli)

What if two people run this command at the same time

This command is used to query a PR link for an existing commit. I think it is totally fine to query the same commit at the same time


track Spring Loaded Front Bumper via preference

Attempting to fix the bug with Asdon Martin Spring Loaded Front Bumper as listed here:
https://kolmafia.us/threads/banished-monster-list-spring-loaded-front-bumper-tracking-issues.27457/#post-168011

I have now understood this and realised that this is not the right fix. If banishes are disappearing incorrectly we need to fix that in the code, not patch around it


Write a PHP program to calculate the permutation of n objects taken r at a time

Description
Write a PHP program to calculate the permutation of n objects taken r at a time
Permutation is an arrangement of objects in a particular order. The number of permutations on n objects taken r at a time is given by the formula n!/(n-r)!
Input  : 5, 2
Output : 20

How to contribute

Save the solution in program/calculate_the_permutation_of_n_objects_taken_r_at_a_time.php file
Add calculate_the_permutation_of_n_objects_taken_r_at_a_time.php file in calculate-the-permutation-of-n-objects-taken-r-at-a-time folder

!assign


SVGs in corners with border radius are not aligned properly

We have a modal with a border radius of 15px that renders on top of other content. The corners allow a small amount of bleed through of the content below inbetween the edge of the view and the start of the shadow. After digging in it looks like if I adjust the offset value just slightly down for the <Stop/> components in the <RadialGradient/> it appears to resolve the issue. There seems to be a bit of math involved in calculating those at this time, and I'm not really sure how to accomplish this programatically as I could only get it to work by tweaking things in the inspector.
See this image below of an example of the bleed through between the edge of the view and the start of the shadow, the white card in the background comes in strongly.

Is it iOS?

Also, please provide me a reproducible example in Expo Snack

Web, iOS, and Android. I'll hopefully be able to work on a snack for you soon.

Hi @SrBrahma I found the time to create a snack that exhibits the issue.
The modal is opened with transparent={true} on top of a view that has a red background. You can clearly see the red leaking through on the edges of the modal before the shadow begins. See accompanying screenshots.

Let me know what you think or if there is another way I can help you out with this.

The thing is that the borderRadius is handled differently by React Native styles and the SVG engine, and that engine depends on the platform (and maybe even on the browser).
In that example, you can put the View with the white background as a child of the Shadow component, and change the paintInside to true. This way, any gap caused by the different borderRadiuses will just have the Shadow color filling it.

Also, in your example, you can just simply put that white background color inside the style property of the Shadow instead of having it on the wrapping View, together with the paintInside=true.

Interesting, when I tried paint inside before I did not expect it to be underneath the background-color. But it does appear to give the result I'm looking for.
It's worth noting that I imagine you should be able to resolve this because the behavior is identical across android,iOS, and web, so I don't think it's just caused by a difference in the internal react-native or web-specific engines. Strangely in our implementation it is identical across iOS/Android/web but in the snack it is not. Possibly caused by some of our nested views in the local implementation. Or potentially something with expo 49, but for some reason there are errors running it on expo 49 in the snack...
I will give your solution a try and see how it works in our environment. Cheers

The solution you mentioned has worked for us and we've gone ahead with that for now.
I wish there was some way I could have guessed this behavior and come up with the solution on my own. I did try paintInside={true} but saw that it covered the whole of the component and did not expect that setting a background would paint the background on  top of the shadow :/
Thanks again for your help. Maybe this would be worth adding to the FAQ section?

I believe the reason it covered the whole component is because the white background you had was caused by the parent component, so the child component is rendered after the parent, that's why the paintInside was on top of the parent's background.
Any idea how to write about that on the README? Either on FAQ or on the paintInside/style props description.


Item do not save

What is your Quests version / build number?
2.9.5-b152
Please describe the problem and how to replicate it. Any console errors?
I'm having two issues one being an item issue.

I created a quest and I named it 'Help!'. The quest loads fine and I set everything up as I liked. The next thing is that when I try adding the rewards for the quest it does not save. This is only for an item. I tried to manually all the item, and also using '0' to automatically add the item. The item will not save when I save the quest. On the other hand, if I go do it manually it will actually save and work when I go into the config and do /questsadmin reload. I do not know how to add lores and enchantments and all the other configurations you can do in-game, in the config file. It would save me a lot of time being able to do it in-game rather then having to go in the quests.yml file and do it manually each time. Everything else works besides that.

Running: Spigot 1.8.8
Can you provide an example quest from quests.yml using Pastebin.com?

https://pastebin.com/KVnFESq9
This is the config that I did manually, since it was not working in-game.

I was able to add a Thorns I Diamond Sword named "Coolest Sword" to your quest using the in-game editor. Here's what that looks like - https://pastebin.com/3Ask9SwH
For more information on this format, please see - https://github.com/FlyingPikachu/Quests/wiki/Item-Formatting
Please note that, if entering it manually, the enchantments menu is picky when it come to names. You likely must enter it as the menu presents ("Thorns" rather than "thorns" or "Thorn"). Afterwards you must enter the level (1, 2, 3, et al.).
For any additional help with the quests.yml, check out - https://github.com/FlyingPikachu/Quests/wiki/Configuration
Thanks for your interest in Quests!


How delete message?

How delete message/
I write code
bot.on('message', (msg) => {
bot.deleteMessage(msg.chat.id, msg.message_id);

console.log(msg);

});
but in console return ETELEGRAM: "400 Bad Request: message can't be deleted"

Make sure your bot has privilegies to delete messages

@voed
In my group, the bot has administrator rights


AccountSAS and Blob_StartCopyFromURL

Which service(blob, file, queue, table) does this issue concern?
blob
Which version of the Azurite was used?
6e64127f22f0208701f0ba7cca82149fa9dbd648 (v3.0.0-preview)
What problem was encountered?
When using account SAS and copying an object to a destination that already exists, that would fail with this check:
https://github.com/Azure/Azurite/blob/6e64127f22f0208701f0ba7cca82149fa9dbd648/src/blob/authentication/AccountSASAuthenticator.ts#L265
accountSASPermission came from OPERATION_ACCOUNT_SAS_PERMISSIONS and for Blob_StartCopyFromURL that's set to Write + Create.
Steps to reproduce the issue?
create blob1
create blob2
copy blob1 to blob2
Have you found a mitigation/solution?
no

@kahing Thanks for report! We should check write permission in included instead of only write.


Customize docker-compose.yml

@gsiv As discussed:
Please fill this issue with your favorites that should be customizable in the docker-compose.yml and/or the docker-stack.yml.
The current proposal is that we add a flag like --profile=swarm (default would be --profile=compose) to produce a docker-stack.yml instead of a docker-compose.yml. What are the differences between these two files?
What customizations do you need else (which should not be solved by a fork of this repo). Such customitzations could be provided by a config file with al --config flag as discussed.

To add some context to this issue, the purpose of openslides-manage-service is to simplify the setup of OpenSlides instances for individuals intending to self-host OpenSlides.  This discussion is about the management tool's suitability for managing a very large number of instances.
openslides-manage-service manages one instance at a time.  For managing multiple instances, a wrapper tool is going to iterate over the individual instances and call this new management tool for each one.  osinstancectl basically already does just this; it just needs to be updated to additionally call the management tool where necessary.  This is all fine by me.

The current proposal is that we add a flag like --profile=swarm (default would be --profile=compose) to produce a docker-stack.yml instead of a docker-compose.yml. What are the differences between these two files?

The differences between Compose and Stack YAML files are rather small.  You can see the actual differences in regards to OpenSlides by comparing https://github.com/OpenSlides/OpenSlides/blob/master/docker/docker-compose.yml.m4 and https://github.com/OpenSlides/OpenSlides/blob/master/docker/docker-stack.yml.m4.  You will notice that it mostly comes down to incompatibilities with depends_on and deploy.
The differences are, in fact, small enough that they could be managed within a single template.  When I introduced the m4 templates, I avoided doing so right away for the sake of simplicity but it would be rather straightforward to merge them.
See also https://github.com/compose-spec/compose-spec/blob/master/deploy.md.

What customizations do you need else (which should not be solved by a fork of this repo). Such customizations could be provided by a config file with al --config flag as discussed.

Having thought about this a little more, I believe asking which docker-compose.yml customizations we want the tool to support is the wrong approach.  Also, accepting that users need to recompile the tool in order to make minor changes to a template is, in my opinion, indicative of something being wrong.  (Yes, I know I was more open to it during our talk. :)
As you are aware, docker-compose.yml allows for almost infinite customizations that cannot possibly be covered by the tool or a configuration file format.  As long as you are dealing with one or two instances, you may be content with having to make certain customizations after the configuration files have been generated; however, that obviously stops being a viable method very quickly.
Compiling custom versions of the tool is an option; however, it's a silly thing to do just because the template must be baked in.  You may also underestimate just how often customizations to the templates are required.  If we were to compile custom versions of the tool for each – possibly ad hoc – template modification, we would end up with a significant number of management tools in use at the same time.  (Ultimately, we would probably end up with a wrapper around some kind of recompile-on-each-invocation workflow instead. :)
These are examples of docker-stack.yml.m4 customizations currently in production here:

image for various services (additionally via .env)
deploy settings, including replicas and placement constraints (additionally via .env)
command overrides
networks settings, e.g., regarding encryption
sysctls settings

Additional likely candidates:

volumes settings
entirely new services

Before making a list of which of these configuration options the tool should support, I would like to propose a --template option.  It should be possible to dump the included template, customize it, and then use that copy instead.  For example:
./manage dump-template --profile=swarm > custom.yml.tpl
./manage setup --template=custom.yml.tpl

Given the --template option, the requirements for the tool's configuration files (--config) become much more manageable.  They would remain unchanged from their role in OpenSlides 3 which is to persist all instance-specific configuration.  This includes non-global settings such as image versions, the domain and port, and scaling settings.  See https://github.com/OpenSlides/OpenSlides/blob/master/docker/.env for a complete list of options.
I don't know where the file for --config would come from yet.  Would the management tool write it when creating the instance?  Would it be based on another template like .env?  In that case the same considerations would probably apply to this template as well, i.e., it should be customizable with an option like --config-template.
Obviously, you could and should still hard-code decent defaults and continue to hide the templates from casual users.

@gsiv This sounds like a good idea and should be easy to do.

Should be done with #74


Upgrade to Terracotta 3.2.0.

Upgrade to the latest Terracotta 3.2.0.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 14 Jan 2010 at 9:08

Original comment by<EMAIL_ADDRESS>on 14 Jan 2010 at 9:27

Changed state: Started

Upgraded to terracotta 3.2.0, and used patched tim-concurrent-collections-1.3.0
version (at least for now).

Original comment by<EMAIL_ADDRESS>on 15 Jan 2010 at 6:03

Changed state: Fixed


Wrong example for stream deck plus dial

missing () from brightness: '{{ dial_value() | int}}'

entity_id: light.testing
service: light.turn_on
service_data:
brightness: '{{ dial_value | int}}'
icon: >
{%- set state = dial_value() -%}
{%- set min = dial_attr("min") -%}
{%- set max = dial_attr("max") -%}
{%- set pct = ((state - min) / (max - min)) * 100 -%}
ring:{{ pct | round }}
text: >
{%- set state = dial_value()  -%}
{%- set ha_state = states('light.testing') -%}
{%- if ha_state == "off" and dial_value() == 0 -%}
{{"off"}}
{%- else -%}
{{state | int}}
{%- endif -%}
state_attribute: brightness
allow_touchscreen_events: true
delay: 0.5
dial_event_type: TURN
attributes:
min: 0
max: 100
step: 1

Thanks for reporting! Could you make a PR to fix the README?


Add FreeBSD installation instructions

Can you rebase this against pgaskin/gh-pages-new? It contains the new site which I'm publishing with kepubify v4.


Checkbox state doesn't change automatically with user interaction when checked is set from props

What RMWC Version are you using [major.minor.patch]: 6.0.5

Name your build system [Webpack, Rollup...]: Webpack

What happened, and what was supposed to happen:

When using a Checkbox and setting the checked property, the checked state doesn't automatically change when the user clicks the checkbox, instead, we have to do it manually as explained in the docs:
function Example() {
  const [checked, setChecked] = React.useState(false);
  return (
    <Checkbox
      label="Cookies"
      checked={checked}
      onChange={evt => setChecked(!!evt.currentTarget.checked)}
    />
  );
}

While this is not a huge problem, it does make it a bit annoying, as I assume in most cases devs just want to set this checked state initially but allow the user to change it later on.
I'd suggest adding 2 different properties for these scenarios:

checkedAlways: boolean which behaves as the current checked property does and prevents the user from changing this state automatically.

checkedInitial: boolean which sets the initial checked state, but allows the user to change it automatically.

This is just standard React and works the same as a native checkbox. I believe what you’re looking for is the “defaultChecked” prop.

Thanks for the quick answer @jamesmfriedman, is there a defaultChecked property in the RMWC Checkbox component? I didn't see that in the docs or the source code.

All of the components inherit from the full dom API. In the case of checkbox, it’s HTMLInputElement. When dealing with form controls, they should all have nearly identical behavior to plain old html form elements.

Perfect, thanks for the info @jamesmfriedman 👍

No problem, I’ll see about adding controlled vs uncontrolled examples to all form components.


search bar refresh

What's the Problem?
No response
Solution/Idea
the search bar refeshing every time you type a character is kind off jarring
Alternatives
I think something smoother would be better
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
No response
Windows Version
No response
Comments
No response

Could you post a screen recording showing this?


Refactor MOABB train.py to provide easier access to prepared dataset iterators

It is sometimes useful to inspect the prepared datasets or the instantiated model in a Jupyter notebook.
In order to facilitate this use case, I have refactored the MOABB/train.py file in order to make it easy to import the code which loads the hparams file and prepares the datasets.
Breaking Changes:

None

Added Dependencies:

None

Thank you @Drew-Wagner for this contribution. I tested it and everything works as expected.


Search stopped working

About
After releasing this canonical repository for Cloud documentation today, the search functionality might have stopped working / is not configured correctly yet. You can verify it by using this link: https://crate.io/docs/cloud/en/latest/search.html?q=azure.

I think we may need to add the maintainer account crate-ro to the RTD project, in order to make it work. Otherwise, a program I was using to enumerate the projects currently may not see this project.
-- https://readthedocs.org/profiles/crate/

I just added the needed environment variables to the crate-cloud RTD project manually.

Works again.
-- https://crate.io/docs/cloud/en/latest/search.html?q=azure