Fix executor not specifying a package runner

Problem
Fix #78
Solution
Specified package runner in run command

Hi @andreacaldera have put in a fix for #78 following the #77 PR :) Great work by the way on that, can now see all playwright output in a neat format (with this #79 fix) + it's all logged live which is much nicer!

@Danphillipz that change was intentional, we were planning to remove package runner. I tested it on yarn and pnpm locally before merging and it seems fine.
Let me have a quick look and if I can't figure out the problem, we'll merge this in temporarily

Hi @andreacaldera have put in a fix for #78 following the #77 PR :) Great work by the way on that, can now see all playwright output in a neat format (with this #79 fix) + it's all logged live which is much nicer!

@Danphillipz that change was intentional, we were planning to remove package runner. I tested it on yarn and pnpm locally before merging and it seems fine. Let me have a quick look and if I can't figure out the problem, we'll merge this in temporarily

It looks like running with a package manager (e.g. yarn nx run test-app-e2e:e2e) works correctly but running nx directly (e.g. nx run test-app-e2e:e2e) throws the error you reported. We could put the package manager back as optional so that people don't have to use it unless they run nx directly

Thanks @Danphillipz, I appreciate the quick fix!
I've made a similar change in here https://github.com/marksandspencer/nx-plugins/pull/80 to re-add package manager as an optional param

Thanks @Danphillipz, I appreciate the quick fix! I've made a similar change in here #80 to re-add package manager as an optional param

Thanks @andreacaldera - I shall revisit this on Monday!

Thanks @andreacaldera, V0.2.1 seems to be working great! Thanks for all the effort in the executor rewrite, this is a much improved approach!


Play Count

Keep record of play counts for tracks

Add a item to the Popup Menu for the track player view to display the play count for the track. How it should be displayed is via a toast message stating the track's play count


Add setenta scores calculation

Overview
Adds points for different card values and calculates scores for the setenta.

Codecov Report

Merging #14 into master will increase coverage by 0.21%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #14      +/-   ##
=========================================
+ Coverage   98.28%   98.5%   +0.21%
=========================================
  Files          14      14
  Lines         760     801      +41
=========================================
+ Hits          747     789      +42
+ Misses         13      12       -1

Impacted Files
Coverage Δ

quince/components/pila.py
100% <100%> (ø)
:arrow_up:

quince/components/card.py
94.73% <100%> (ø)
:arrow_up:

test/test_calculate_scores.py
98.75% <100%> (+1.52%)
:arrow_up:

quince/calculate_scores.py
100% <100%> (ø)
:arrow_up:

test/test_components.py
97.29% <100%> (+0.3%)
:arrow_up:

Continue to review full report at Codecov.

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


Error loading nsp file

There are issues loading the new NSP file format

I'm running Google Chrome Version 59.0.3071.115 (Official Build) (64-bit)

Hi @faviouz
Fixed!, i hope the new format helps a bit with file size and loading times!
Thanks a lot for reporting.


Port to ROS2

Hello,
I was wondering if there is any official plan or roadmap to port this package to ROS2.
I saw there is a fork here (https://github.com/ros2/ros_astra_camera) but it is not clear to me what is the current status.
Moreover, seems like this repo received significant updates since the original fork.
thanks

I've used this package on ROS1 (Kinetic) but i switched to ROS2 (Foxy) and wanted a fresh start. This package wont compile with ROS2. The Roadmap suggestet from Hi-Zed would be great or if possible to use this package in ROS2.


Update README.md

To deploy from the terminal: cdk deploy --profile [profile from credentials] devNitroWalletEth
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@woosleyn thanks for your contribution. Since that this is just a simple example of how AWS Nitro Enclaves can be managed and deployed via AWS CDK we try to keep the readme as simple as possible.
The assumption is that the cdk binary will just pull the aws default credentials. For more advanced use-cases like specifying different profiles e.g. via --profile, users should consult the official AWS CDK documentation which is linked in the readme as well.


add a C-lightning FAQ page

to be extended with more detailed info about the implementation in RaspiBlitz

can be linked from the main FAQ.md

included a manpage-like list of the short script help outputs

suggestions, additions and comments are welcome

Any chance of installing an http server plugin so light clients can communicate directly with lightning-cli?

@Fonta1n3 it would worth to have a dedicated issue for this. Is that the BlockchainCommons one used to connect Fully Noded to CLN?

@Fonta1n3 it would worth to have a dedicated issue for this. Is that the BlockchainCommons one used to connect Fully Noded to CLN?

Sure. Its just a hidden service which controls port 8080 for c-lightning. http://rpcuser:rpcpassword@xxx.onion:8080
I can certainly look at adding support for clighting rest but I really much prefer to deal directly with lighting-cli rather then a third party api.

@openoms is the FAQ ready to merge?

It can be merged, but need to be extended a bit more with at least the backups and recovery options on the end.
Will add to if there are new services or important concepts appear. Also reviews and PRs are easier if it is in a main branch.

Merging so I can add it to the README.


Add @media query to "content scrolling" docs section

Closes https://github.com/vaadin/vaadin-app-layout/issues/148

It has been a while since this PR was submitted. Apologies about keeping it open for so long without any progress.
The component has been converted to Polymer 3 so the codebase has changed.
Also, the new development for Vaadin web components will happen in the monorepo.
I'm closing this now. Will also close the original issue because we switched back to the old API docs for now.


Bugfix/notifications task fix

Previous edits to the send_push_notification function didn't work when tested with live data. The function was refactored and tested.
This pull request includes updates to the notification system to improve the handling of push notifications.
Improvements to notification handling:

notifications/tasks.py: Added a TODO comment to address the need for updating tasks to send notifications to all churches when there are multiple churches.

Code simplification and optimization:

notifications/utils.py: Refactored the send_push_notification function to use a dictionary for filtering based on user preferences, reducing repetitive code and improving maintainability.

I think we've got ourselves a winner! I tested this with all scenarios including weekly fasts.


Route pages based on NavViewItem's tag

This was the intended method that I appear to have mixed up with the InvokedItem at some point. Should allow for localization

@artehe Can you just double check this won't break anything massively before I merge it?

@artehe Can you just double check this won't break anything massively before I merge it?

You're good, that should work fine.


[openimageio] Build error with "File does not have the expected hash"

Package: openimageio[core]:x64-linux -> <IP_ADDRESS>
Host Environment

Host: x64-linux
Compiler: Clang 13.0.1
vcpkg-tool version: 2023-04-07-bedcba5172f5e4b91caac660ab7afe92c27a9895
vcpkg-scripts version: 9edb1b8e5 2023-08-14 (8 weeks ago)

To Reproduce
vcpkg install
Failure logs
-- Downloading https://github.com/OpenImageIO/oiio/archive/v<IP_ADDRESS>.tar.gz -> OpenImageIO-oiio-v2-9325beef.4.13.0.tar.gz...
[DEBUG] To include the environment variables in debug output, pass --debug-env
[DEBUG] Trying to load bundleconfig from /home/k-ye/Github/taco/third_party/vcpkg/vcpkg-bundle.json
[DEBUG] Failed to open: /home/k-ye/Github/taco/third_party/vcpkg/vcpkg-bundle.json
[DEBUG] Bundle config: readonly=false, usegitregistry=false, embeddedsha=nullopt, deployment=Git, vsversion=nullopt
[DEBUG] Metrics enabled.
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] 1000: popen( curl --fail -L https://github.com/OpenImageIO/oiio/archive/v<IP_ADDRESS>.tar.gz --create-dirs --output /home/k-ye/Github/taco/third_party/vcpkg/downloads/OpenImageIO-oiio-v2-9325beef.4.13.0.tar.gz.1925416.part 2>&1)
[DEBUG] 1000: cmd_execute_and_stream_data() returned 0 after 30108831 us
[DEBUG] Trying to hash /home/k-ye/Github/taco/third_party/vcpkg/downloads/OpenImageIO-oiio-v2-9325beef.4.13.0.tar.gz.1925416.part
[DEBUG] /home/k-ye/Github/taco/third_party/vcpkg/downloads/OpenImageIO-oiio-v2-9325beef.4.13.0.tar.gz.1925416.part has hash 9e887c7039995ce7c41556e09a7eed940863260a522ecf7d9bec300189026ed507da560620dfa4a619deeb679be7adf42fe3a7020ff3094df777c7934c771227
error: Failed to download from mirror set
error: File does not have the expected hash:
url: https://github.com/OpenImageIO/oiio/archive/v<IP_ADDRESS>.tar.gz
File: /home/k-ye/Github/taco/third_party/vcpkg/downloads/OpenImageIO-oiio-v2-9325beef.4.13.0.tar.gz.1925416.part
Expected hash: 9325beefce55b66a58fcfc2ce93e1406558ed5f6cc37cb1e8e04aee470c4f30a14483bebfb311c329f7868afb6c508a052661c6b12d819a69f707c1a30cd9549
Actual hash: 9e887c7039995ce7c41556e09a7eed940863260a522ecf7d9bec300189026ed507da560620dfa4a619deeb679be7adf42fe3a7020ff3094df777c7934c771227
[DEBUG] /mnt/vss/_work/1/s/src/vcpkg/base/downloads.cpp(961):
[DEBUG] Time in subprocesses: 30108831us
[DEBUG] Time in parsing JSON: 4us
[DEBUG] Time in JSON reader: 0us
[DEBUG] Time in filesystem: 71us
[DEBUG] Time in loading ports: 0us
[DEBUG] Exiting after 30 s (30216601us)

CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):

      Failed to download file with error: 1
      If you are using a proxy, please check your proxy setting. Possible causes are:

      1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
         to `https://address:port`. This is not correct, because `https://` prefix
         claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
         , etc..) is an HTTP proxy. Try setting `http://address:port` to both
         HTTP_PROXY and HTTPS_PROXY instead.

      2. If you are using Windows, vcpkg will automatically use your Windows IE Proxy Settings
         set by your proxy software. See https://github.com/microsoft/vcpkg-tool/pull/77
         The value set by your proxy might be wrong, or have same `https://` prefix issue.

      3. Your proxy's remote server is out of service.

      If you've tried directly download the link, and believe this is not a temporary
      download server failure, please submit an issue at https://github.com/Microsoft/vcpkg/issues
      to report this upstream download server failure.

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:270 (z_vcpkg_download_distfile_show_proxy_and_fail)
  scripts/cmake/vcpkg_from_github.cmake:106 (vcpkg_download_distfile)
  ports/openimageio/portfile.cmake:1 (vcpkg_from_github)
  scripts/ports.cmake:147 (include)

Additional context
vcpkg.json
{
  "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg/master/scripts/vcpkg.schema.json",
  "name": "taco",
  "version-string": "0.0.1",
  "description": "Taitopia Core",
  "dependencies": [
    "abseil",
    "benchmark",
    "freetype",
    "glfw3",
    "glm",
    "grpc",
    "gtest",
    "imgui",
    "spdlog",
    "stb",
    "tinyobjloader",
    "openexr",
    "openimageio",
    {
      "name": "ffmpeg",
      "default-features": true,
      "features": [
        "x264",
        "nvcodec"
      ]
    }
  ]
}

https://github.com/microsoft/vcpkg/pull/32920 is the one that updates the SHA512. However, if I just download the corresponding tar from https://github.com/OpenImageIO/oiio/archive/v<IP_ADDRESS>.tar.gz, the computed hash is indeed 9e887c7039995ce7c41556e09a7eed940863260a522ecf7d9bec300189026ed507da560620dfa4a619deeb679be7adf42fe3a7020ff3094df777c7934c771227. Not sure how the diff was introduced.

Duplicates #34173. Explained in https://github.com/microsoft/vcpkg/issues/34173#issuecomment-1743142999. Fixed in https://github.com/microsoft/vcpkg/pull/34231.


UI update

If you have any suggestions or ideas to update the UI of the app, you're welcome.

the dimensions of different screens can affect our application UI. Use of Mediaquery or Screenutil will make our application more compatible and increase the performance of the app. your app is will not work properly on different screen sizes so i can suggest that.

 This app is using responsive widgets only like ListView and GridView and I don't think it will be affected much if the screen size differs.
You can work on the Home screen of the app as it has just gifs in all grids.
Or you can add a new feature to the UI to make it more interactive for students.
Or it'll be good if you add a Popular Courses and Opted Courses section

okay assign me this issue

i provide one switch to your app by which your app is now get the light mode and dark mode

i will share you some screenshots

see i plugin one switch on the app bar that is doing all this

it is one the homescreen

my commits did not increased in kwoc .

my commits did not increased in kwoc .

Your PR has increased or not?

Your PR has increased or not?

pr increased

pr increased

That's fine then, your PR is merged. Commits actually doesn't matter. But still you can ask this in: https://github.com/kossiitkgp/kwoc-bugs/issues

That's fine then, your PR is merged. Commits actually doesn't matter. But still you can ask this in: https://github.com/kossiitkgp/kwoc-bugs/issues


RoomlifecycleManager: Unsubscribe room monitoring as a part of room release

Currently, Room monitoring is enabled for each feature using channel.on('update') and all other channel events.
After setting status to RoomStatus.Released
https://github.com/ably/ably-chat-js/blob/0e1b52336a9e15c711d62848a0c3ba841dc6ae77/src/core/room-lifecycle-manager.ts#L891.
we should deregister all of those listeners using

    for (const feature of this._contributors) {
      feature.channel.offAll();
    }

All of the code blocks you point to have a check that causes them to effectively no-op if a release (or any other) operation is in progress. The only exception to this is checking for discontinuity.

Yeah, I agree they cause no-op after release ( I think we listen to detach when release is happening ), but callbacks will still be a part of internal listeners after release and won't be garbage collected.
I think when Room goes into Released state, can remove those listeners.

I believe we might like to unsubscribe to room monitoring listeners right before we set
https://github.com/ably/ably-chat-js/blob/533c4d0314bcce942a30b4d7d6a012c3b081420d/src/core/room-lifecycle-manager.ts#L824

Since we are commencing the Release operation, there's no point in listening to discontinuity events. Room will emit Releasing and Released status which denotes end of current room lifeycle.

This is because the release operation mainly detach each channel. This will emit DETACHED or FAILED channel events and will be captured by room monitoring listeners.

Although, we do nothing for Detached, for Failed channel event, we set duplicate failed room status and start channel-WindDown again
https://github.com/ably/ably-chat-js/blob/533c4d0314bcce942a30b4d7d6a012c3b081420d/src/core/room-lifecycle-manager.ts#L233-L244


Ask user to create file when non-existing path provided via commandline.

This implements "broadFileSystemAccess" too.


#252 Add unit test and fix for complex interface renaming

fixes #252

:white_check_mark: Build ConfuserEx 553 completed (commit https://github.com/mkaring/ConfuserEx/commit/f01a010ef8 by @mkaring)

:white_check_mark: Build ConfuserEx 555 completed (commit https://github.com/mkaring/ConfuserEx/commit/8b36b43ffc by @mkaring)

:white_check_mark: Build ConfuserEx 556 completed (commit https://github.com/mkaring/ConfuserEx/commit/8b36b43ffc by @mkaring)


added competitive programming book and problems

What does this PR do?
Add Resource(s)
For resources
Description
added Competitive Programmer's Handbook for beginner( PDF) by Antti Laaksonen (https://codeforces.com/profile/pllk) and  beginner to advance level problems (sheet)
Why is this valuable (or not)
they are beginner friendly and easy to understand
How do we know it's really free?
for book it is a pdf link and sheets are available on authors youtube channel
For book lists, is it a book?
yes
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

please fix linter errors

i had fixed the linter errors and  sorry for the above mess, thanks

please don't create a category for a single book.

python
在 2020-10-12 10:12:59，"Eric Hellman"<EMAIL_ADDRESS>写道：
please don't create a category for a single book.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Update README.md

fix spelling

thanks for fixes @aaronjrodrigues . Can you please run linters to make sure the failing CI passes before we merge?

seems like the linting check doesn't like that I changed 'timezone' to 'time zone', I'll try reverting it back and see how it goes.


Setting sync overrides current settings with very old version

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.87.0
OS Version: Linux x64 6.5.0-21-generic/Windows_NT x64 10.0.22631

Steps to Reproduce:

Open my usual vscode install on Linux, my primary machine
Settings are all synced
Open my rarely-used vscode install on Windows
Vscode overrides the remote settings, extensions, ui, etc with whatever the windows install had. In this case it changed all of my copilot extensions to pre-release, reinstalled the deprecated labs extension, as well as a handful of extensions I've uninstalled on my linux machine

Can you please share the settings sync log from the windows machine?

F1 > Open View... > Settings Sync

2024-03-05 11:59:49.202 [info] Using settings sync service https://vscode-sync.trafficmanager.net/
2024-03-05 11:59:49.202 [info] Auto Sync is enabled.
2024-03-05 11:59:49.202 [info] Auto Sync: Suspended until auth token is available.
2024-03-05 11:59:49.832 [info] Auto Sync: Triggered by Interval
2024-03-05 11:59:49.832 [info] Sync started.
2024-03-05 11:59:51.071 [info] Deleting non existing profile from backup /C:/Users/user/AppData/Roaming/Code/User/sync/extensions
2024-03-05 11:59:51.118 [info] Deleting non existing profile from backup /C:/Users/user/AppData/Roaming/Code/User/sync/globalState
2024-03-05 11:59:51.243 [info] Deleting non existing profile from backup /C:/Users/user/AppData/Roaming/Code/User/sync/keybindings
2024-03-05 11:59:51.275 [info] GlobalState: Last sync resource does not exist locally.
2024-03-05 11:59:51.291 [info] Deleting non existing profile from backup /C:/Users/user/AppData/Roaming/Code/User/sync/profiles
2024-03-05 11:59:51.440 [info] Request failed {"url":"https://vscode-sync.trafficmanager.net/v1/resource/globalState/bd485844-fbbc-46a2-9595-71ba6c28584c","status":404,"operation-id":"e485b749a91e469498b493418d69faee"} Not Found
2024-03-05 11:59:51.440 [info] GlobalState: Last sync resource does not exist remotely.
2024-03-05 11:59:51.458 [info] Deleting non existing profile from backup /C:/Users/user/AppData/Roaming/Code/User/sync/settings
2024-03-05 11:59:51.583 [info] Deleting non existing profile from backup /C:/Users/user/AppData/Roaming/Code/User/sync/snippets
2024-03-05 11:59:51.662 [info] Deleting non existing profile from backup /C:/Users/user/AppData/Roaming/Code/User/sync/tasks
2024-03-05 11:59:53.413 [info] Settings: Updated local settings
2024-03-05 11:59:53.424 [info] Settings: Updated last synchronized settings
2024-03-05 11:59:53.426 [info] Keybindings: Last sync resource does not exist locally.
2024-03-05 11:59:53.566 [info] Keybindings: No changes found during synchronizing keybindings.
2024-03-05 11:59:53.851 [info] Snippets: Last sync resource does not exist locally.
2024-03-05 11:59:54.787 [info] Snippets: No changes found during synchronizing snippets.
2024-03-05 11:59:55.882 [info] Tasks: Last sync resource does not exist locally.
2024-03-05 11:59:56.077 [info] Tasks: No changes found during synchronizing tasks.
2024-03-05 11:59:56.081 [info] GlobalState: Last sync resource does not exist locally.
2024-03-05 11:59:56.241 [info] Request failed {"url":"https://vscode-sync.trafficmanager.net/v1/resource/globalState/bd485844-fbbc-46a2-9595-71ba6c28584c","status":404,"execution-id":"17cd305a-d0f9-48de-9ef2-d1fa736c4a0e","operation-id":"af5395ca54344f6ba4e222c86e4f6359"} Not Found
2024-03-05 11:59:56.241 [info] GlobalState: Last sync resource does not exist remotely.
2024-03-05 11:59:56.363 [info] GlobalState: Accepting remote because it was synced before and the last sync data is not available.
2024-03-05 11:59:56.373 [info] GlobalState: Updated global state ["encryption.migratedToGnomeLibsecret","remote.tunnels.toRestore.ssh-remote","commandPalette.mru.cache","commandPalette.mru.counter","cpp.1.sessionCount","csharp.1.sessionCount","inline-chat-history","java.2.sessionCount","snippets.usageTimestamps","typescript.1.sessionCount","workbench.activity.pinnedViewlets2","workbench.statusbar.hidden"]
2024-03-05 11:59:56.373 [info] GlobalState: Updated local ui state
2024-03-05 11:59:56.505 [info] GlobalState: Updated remote ui state.
2024-03-05 11:59:56.511 [info] GlobalState: Updated last synchronized ui state
2024-03-05 11:59:56.513 [info] Extensions: Migrating last sync user data. Resource does not exist.
2024-03-05 11:59:56.513 [info] Extensions: Last sync data state does not exist.
2024-03-05 11:59:57.184 [info] Extensions: Skipped synchronizing extension because the extension is not found. thiagoabreu.vala
2024-03-05 11:59:57.496 [info] Extensions: Migrating last sync user data. Resource does not exist.
2024-03-05 11:59:57.496 [info] Extensions: Last sync data state does not exist.
2024-03-05 11:59:57.954 [info] Extensions: Migrating last sync user data. Resource does not exist.
2024-03-05 11:59:57.955 [info] Extensions: Last sync data state does not exist.
2024-03-05 11:59:58.586 [info] Extensions: Migrating last sync user data. Resource does not exist.
2024-03-05 11:59:58.586 [info] Extensions: Last sync data state does not exist.
2024-03-05 12:00:02.682 [info] Extensions: Migrating last sync user data. Resource does not exist.
2024-03-05 12:00:02.682 [info] Extensions: Last sync data state does not exist.
2024-03-05 12:00:07.071 [info] Extensions: Installed extension. ms-vscode.remote-explorer 0.4.1
2024-03-05 12:00:07.071 [info] Extensions: Installed extension. uctakeoff.vscode-counter 3.4.0
2024-03-05 12:00:07.072 [info] Extensions: Installed extension. ms-python.flake8 2023.10.0
2024-03-05 12:00:07.072 [info] Extensions: Installed extension. github.copilot 1.171.0
2024-03-05 12:00:07.072 [info] Extensions: Installed extension. github.copilot-chat 0.12.2
2024-03-05 12:00:07.072 [info] Extensions: Installed extension. github.codespaces 1.16.13
2024-03-05 12:00:07.072 [info] Extensions: Installed extension. ms-toolsai.jupyter 2024.1.1
2024-03-05 12:00:07.072 [info] Extensions: Installed extension. ms-python.pylint 2023.10.1
2024-03-05 12:00:07.072 [info] Extensions: Installed extension. qwtel.sqlite-viewer 0.3.13
2024-03-05 12:00:07.120 [info] Extensions: Migrating last sync user data. Resource does not exist.
2024-03-05 12:00:07.120 [info] Extensions: Last sync data state does not exist.
2024-03-05 12:00:07.226 [info] Extensions: Updated remote extensions. Added: ["github.copilot-labs","ms-vscode-remote.remote-ssh-edit","ms-vscode-remote.remote-wsl","ms-vscode.remote-repositories","piotrpalarz.vscode-gitignore-generator","sneezry.vscode-instant-weather"]. Updated: ["github.copilot-chat","github.copilot"].
2024-03-05 12:00:07.244 [info] Extensions: Updated last synchronized extensions. Skipped: ["thiagoabreu.vala"].
2024-03-05 12:00:07.250 [info] Profiles: Last sync resource does not exist locally.
2024-03-05 12:00:07.361 [info] Profiles: No changes found during synchronizing profiles.
2024-03-05 12:00:07.410 [info] Sync done. Took 17579ms
2024-03-05 12:00:07.411 [info] Auto Sync: Triggered by Activity
2024-03-05 12:00:07.411 [info] Sync started.
2024-03-05 12:00:07.564 [info] Settings: No changes found during synchronizing settings.
2024-03-05 12:00:07.575 [info] Keybindings: No changes found during synchronizing keybindings.
2024-03-05 12:00:07.608 [info] Snippets: No changes found during synchronizing snippets.
2024-03-05 12:00:07.619 [info] Tasks: No changes found during synchronizing tasks.
2024-03-05 12:00:07.853 [info] GlobalState: Updated remote ui state. Added: workbench.panel.markers.hidden,workbench.panel.output.hidden,terminal.hidden,workbench.explorer.views.state.hidden,workbench.scm.views.state.hidden,workbench.view.search.state.hidden,workbench.activity.pinnedViewlets2,nps/lastSessionDate,nps/sessionCount,workbench.telemetryOptOutShown,workbench.view.debug.state.hidden,workbench.statusbar.hidden,workbench.panel.pinnedPanels,cpp.1.lastSessionDate,cpp.1.sessionCount,java.2.lastSessionDate,java.2.sessionCount,javascript.1.lastSessionDate,javascript.1.sessionCount,typescript.1.lastSessionDate,typescript.1.sessionCount,workbench.welcomePage.walkthroughMetadata,colorThemeData,workbench.panel.repl.hidden,ces/skipSurvey,memento/gettingStartedService,commandPalette.mru.cache,commandPalette.mru.counter,extensionsAssistant/fileExtensionsSuggestionIgnore,fileBasedRecommendations/promptedRecommendations,memento/workbench.editor.keybindings,terminal.integrated.showTerminalConfigPrompt,workbench.view.extension.comment-anchors.state.hidden,workbench.view.extension.sshfs.state.hidden,workbench.view.extension.test.state.hidden,workbench.view.extensions.state.hidden,workbench.view.remote.state.hidden,expandSuggestionDocs,memento/notebookGettingStarted2,workbench.view.extension.jupyter-variables.state.hidden,recently.opened,workbench.view.extension.github-pull-requests.state.hidden,workbench.view.extension.github-pull-request.state.hidden,~remote.forwardedPortsContainer.hidden,workbench.view.extension.gitlab-workflow.state.hidden,extensionTips/promptedExecutableTips,workbench.sideBar.position,workbench.panel.alignment,workbench.activityBar.hidden,workbench.statusBar.hidden,workbench.auxiliarybar.pinnedPanels,remote.explorerType,workbench.view.extension.serialport-view.state.hidden,snippets.usageTimestamps,views.customizations,workbench.view.extension.jupyter.state.hidden,workbench.view.sync.state.hidden,workbench.view.extension.project-dashboard.state.hidden,workbench.view.extension.bookmarks.state.hidden,javascript.1.editedCount,javascript.1.editedDate,workbench.view.extension.dockerView.state.hidden,http.linkProtectionTrustedDomains,nps/isCandidate,nps/skipVersion,javascript.1.isCandidate,javascript.1.skipVersion,github-GammaGames,workbench.views.service.sidebar.0ac4a0f2-96a1-4201-af31-18a1a5950495.state.hidden,csharp.1.lastSessionDate,csharp.1.sessionCount,themeUpdatedNotificationShown,interactiveEditor.storage.inlineDiff,workbench.panel.interactiveSessionSidebar.copilot.hidden,workbench.view.extension.copilot-labs.state.hidden,workbench.welcomePage.hiddenCategories,extensionsAssistant/ignored_recommendations,Comments.hidden,workbench.panel.chatSidebar.copilot.hidden,workbench.views.service.sidebar.d5bd57c3-4d2f-4c8c-b55b-5dfd89ca002f.state.hidden,workbench.panel.chatSidebar.hidden,workbench.activityBar.location,inline-chat-history,userDataProfiles.state.hidden,encryption.migratedToGnomeLibsecret.
2024-03-05 12:00:07.857 [info] GlobalState: Updated last synchronized ui state
2024-03-05 12:00:08.157 [info] Extensions: Updated remote extensions. Updated: ["github.copilot-chat","github.copilot"].
2024-03-05 12:00:08.161 [info] Extensions: Updated last synchronized extensions. Skipped: ["thiagoabreu.vala"].
2024-03-05 12:00:08.164 [info] Profiles: No changes found during synchronizing profiles.
2024-03-05 12:00:08.167 [info] Sync done. Took 756ms
2024-03-05 12:00:08.167 [info] Auto Sync: Triggered by Activity
2024-03-05 12:00:08.167 [info] Sync started.
2024-03-05 12:00:08.307 [info] Settings: No changes found during synchronizing settings.
2024-03-05 12:00:08.314 [info] Keybindings: No changes found during synchronizing keybindings.
2024-03-05 12:00:08.340 [info] Snippets: No changes found during synchronizing snippets.
2024-03-05 12:00:08.347 [info] Tasks: No changes found during synchronizing tasks.
2024-03-05 12:00:08.357 [info] GlobalState: No changes found during synchronizing ui state.
2024-03-05 12:00:08.508 [info] Extensions: No changes found during synchronizing extensions.
2024-03-05 12:00:08.510 [info] Profiles: No changes found during synchronizing profiles.
2024-03-05 12:00:08.512 [info] Sync done. Took 344ms
2024-03-05 12:00:09.502 [info] Auto Sync: Triggered by Activity
2024-03-05 12:00:09.502 [info] Sync started.
2024-03-05 12:00:09.652 [info] Settings: No changes found during synchronizing settings.
2024-03-05 12:00:09.658 [info] Keybindings: No changes found during synchronizing keybindings.
2024-03-05 12:00:09.681 [info] Snippets: No changes found during synchronizing snippets.
2024-03-05 12:00:09.688 [info] Tasks: No changes found during synchronizing tasks.
2024-03-05 12:00:09.695 [info] GlobalState: No changes found during synchronizing ui state.
2024-03-05 12:00:09.835 [info] Extensions: No changes found during synchronizing extensions.
2024-03-05 12:00:09.837 [info] Profiles: No changes found during synchronizing profiles.
2024-03-05 12:00:09.838 [info] Sync done. Took 336ms
2024-03-05 12:00:45.320 [info] Auto Sync: Triggered by Activity
2024-03-05 12:00:45.321 [info] Sync started.
2024-03-05 12:00:45.471 [info] Settings: No changes found during synchronizing settings.
2024-03-05 12:00:45.476 [info] Keybindings: No changes found during synchronizing keybindings.
2024-03-05 12:00:45.504 [info] Snippets: No changes found during synchronizing snippets.
2024-03-05 12:00:45.510 [info] Tasks: No changes found during synchronizing tasks.
2024-03-05 12:00:45.517 [info] GlobalState: No changes found during synchronizing ui state.
2024-03-05 12:00:45.677 [info] Extensions: No changes found during synchronizing extensions.
2024-03-05 12:00:45.679 [info] Profiles: No changes found during synchronizing profiles.
2024-03-05 12:00:45.681 [info] Sync done. Took 360ms

These are the extensions that were incorrectly synced

Can you please be more specific about what is incorrect here?

Sure! On December 20th I cleaned up my installed extensions on on my primary Linux machine, shortly after Copilot Labs was discontinued (December 15th) because Labs kept popping up a warning. I changed copilot and copilot chat to the release channels as well, since I'd only had them on prerelease so they would work with Labs.

Added: ["github.copilot-labs","ms-vscode-remote.remote-ssh-edit","ms-vscode-remote.remote-wsl","ms-vscode.remote-repositories","piotrpalarz.vscode-gitignore-generator","sneezry.vscode-instant-weather"].

All of the extensions that were added in the above quote were added when my Windows install updated and synced to the settings sync service. They should not have been because they were uninstalled months prior.

Updated: ["github.copilot-chat","github.copilot"].

These extensions were also updated to the prerelease versions after the setting sync.

Added: ["github.copilot-labs","ms-vscode-remote.remote-ssh-edit","ms-vscode-remote.remote-wsl","ms-vscode.remote-repositories","piotrpalarz.vscode-gitignore-generator","sneezry.vscode-instant-weather"].

Where did you see this?
Probably there is another machine syncing these extensions? You can check this by opening synced data (remote) view : F1 > Settings Sync: Show Synced Data

Line 54 from the original log - https://github.com/microsoft/vscode/issues/206911#issuecomment-1983798437. My linux machines all had the extensions removed, the older Windows install decided to install its current outdated extensions over the top of the remote extensions

May I know if the older Windows machine has version of VS Code which is very old?

It would've been 2-3 months behind, yes. The main reason I started it was to get it upgraded.

Hey I have a similar issue! In my case I constantly keep changing an extension from pre-release to release but settings sync brings it back to pre-release.
I went to Show Sync Data and I see that there's two machines that override the parameter instead of applying the most recent sync settings. What should I provide you?

So I just noticed that I have a Linux machine currently forcing its settings:

In the image "Linux (Code) #1" overrode the preRelease parameter to true

Can you please share following from the Linux machine

F1 > Download Settings Sync Data

Note that this will download all the data you synced to cloud. If you are OK to share, please share it via email -<EMAIL_ADDRESS>

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


[MIST-997] Implement application-aware query allocation

This PR resolves #997 via

Implement application-aware query allocation
Implement necessary test for it

Closes #997.

LGTM. I'm merging it


RC-2023-03-02 - Merge stage into main

@MarkusHorstmann, @PaulLinYao - In preparation for deployment, can you review this PR?


Generated project does not work when updating cozy-ui dependency

Summary
When creating a brand new project with create-react-app, upgrading cozy-ui to the latest version results in the project not being buildable.
How to reproduce

Create a new project: yarn create cozy-app demo-cozy-ui-issue && cd demo-cozy-ui-issue
Upgrade the cozy-ui dependency to a very recent version: yarn add<EMAIL_ADDRESS>Either build the project yarn build or start the development server yarn start; an error is shown and the process fails

Error message
    ERROR in ./src/styles/index.styl
    Module build failed (from ./node_modules/mini-css-extract-plugin/dist/loader.js):
    ModuleBuildError: Module build failed (from ./node_modules/stylus-loader/index.js):
    Error: Workspace/cozy/demo-cozy-ui-issue/node_modules/cozy-ui/stylus/cozy-ui/build.styl:13:10
        9| \*------------------------------------*/
       10| @require '../settings/*'
       11| @require '../tools/*'
       12| @require '../generic/animations'
       13| @require '../../node_modules/normalize.css/normalize'
    ----------------^
       14| @require '../elements/*'
       15| @require '../objects/*'
       16| @require '../components/*'

    failed to locate @require file ../../node_modules/normalize.css/normalize.styl

        at CachedPathEvaluator.visitImport (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus-loader/lib/evaluator.js:157:21)
        at CachedPathEvaluator.Visitor.visit (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/index.js:28:40)
        at CachedPathEvaluator.Evaluator.visit (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/evaluator.js:160:18)
        at CachedPathEvaluator.Evaluator.visitBlock (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/evaluator.js:720:39)
        at CachedPathEvaluator.Visitor.visit (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/index.js:28:40)
        at CachedPathEvaluator.Evaluator.visit (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/evaluator.js:160:18)
        at CachedPathEvaluator.importFile (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus-loader/lib/evaluator.js:84:18)
        at CachedPathEvaluator.visitImport (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus-loader/lib/evaluator.js:162:27)
        at CachedPathEvaluator.Visitor.visit (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/index.js:28:40)
        at CachedPathEvaluator.Evaluator.visit (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/evaluator.js:160:18)
        at CachedPathEvaluator.Evaluator.visitRoot (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/evaluator.js:707:27)
        at CachedPathEvaluator.Visitor.visit (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/index.js:28:40)
        at CachedPathEvaluator.Evaluator.visit (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/evaluator.js:160:18)
        at CachedPathEvaluator.Evaluator.evaluate (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/visitor/evaluator.js:247:15)
        at Renderer.render (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/renderer.js:86:26)
        at Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus-loader/index.js:167:12
        at runLoaders (Workspace/cozy/demo-cozy-ui-issue/node_modules/webpack/lib/NormalModule.js:301:20)
        at Workspace/cozy/demo-cozy-ui-issue/node_modules/loader-runner/lib/LoaderRunner.js:367:11
        at Workspace/cozy/demo-cozy-ui-issue/node_modules/loader-runner/lib/LoaderRunner.js:233:18
        at context.callback (Workspace/cozy/demo-cozy-ui-issue/node_modules/loader-runner/lib/LoaderRunner.js:111:13)
        at Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus-loader/index.js:169:11
        at Renderer.render (Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus/lib/renderer.js:107:12)
        at Workspace/cozy/demo-cozy-ui-issue/node_modules/stylus-loader/index.js:167:12
        at tryCatchReject (Workspace/cozy/demo-cozy-ui-issue/node_modules/when/lib/makePromise.js:840:30)
        at runContinuation1 (Workspace/cozy/demo-cozy-ui-issue/node_modules/when/lib/makePromise.js:799:4)
        at Fulfilled.when (Workspace/cozy/demo-cozy-ui-issue/node_modules/when/lib/makePromise.js:590:4)
        at Pending.run (Workspace/cozy/demo-cozy-ui-issue/node_modules/when/lib/makePromise.js:481:13)
        at Scheduler._drain (Workspace/cozy/demo-cozy-ui-issue/node_modules/when/lib/Scheduler.js:62:19)
        at Scheduler.drain (Workspace/cozy/demo-cozy-ui-issue/node_modules/when/lib/Scheduler.js:27:9)
        at process._tickCallback (internal/process/next_tick.js:61:11)
     @ ./src/targets/browser/index.jsx 2:0-16
     @ multi ./node_modules/babel-polyfill/lib/index.js ./node_modules/cozy-scripts/utils/reactExposer.js ./src/targets/browser/index.jsx

Additional information
I came across the issue after noticing that the default cozy-ui dependency of the generated project was quite old, and I wanted to use the Card component in my project (I just noticed that that component is actually a lot more recent that I thought). So I thought "Let's just bump the version of the dependency to the latest available, nothing too bad should happen..." Famous last words. :laughing:
More seriously, the issue seems to have appeared following PR 833 in cozy/cozy-ui as using the following release (18.15.0) in the generated project shows the error above, but the previous version (18.14.0) does not.
I assume cozy-ui works as intended, hence why I am filing an issue here. I do not have enough experience with Webpack to know with certainty whether it is the case though.

Thanks for reporting this issue. We will look at it pretty soon.
I think this is because we didn't publish the current master of this repo...
Can you try to add an alias :
alias : {
    'cozy-ui/react': 'cozy-ui/transpiled/react'
}

In the module  object here: https://github.com/CPatchane/create-cozy-app/blob/v1.12.x/packages/cozy-scripts/config/webpack.config.react.js#L14

Sorry to leave you waiting, there's been a big shift in priorities here for the last couple of weeks. I should hopefully be able to have some time dedicated to Cozy app development by the end of the week.

Hi @klehelley, thanks for reporting this issue and sorry for my late reply, but I wanted to fix some issues on the current unreleased version of CCA before and we were working on a v2 of this tool. Good news, this new version just released with all dependencies up to date.
Could you try again with the v2? Just doing yarn create cozy-app <myapp> should use directly the most recent version of CCA.

Hi @CPatchane. I have tried to reproduce the issue with the latest version of create-cozy-app and it works well, with cozy-ui set at a more recent version than the one I was trying to use when I stumbled upon the issue. Thanks!


[FLINK-2235] fix calculation of free memory for local execution

The Java runtime may return Long.MAX_VALUE for a call to the maxMemory()
method of the Runtime class. In these cases, we can get hold of the free
physical memory size of the operating system by using a proprietary
Oracle JDK method. Otherwise, we fail with an explanatory exception.

I found a fix for the Oracle JVM (the only known JVM with this problem). We call a proprietary method and fail on other JDKs with a nice exception message. @uce What do you think?

Is this value actually accurate? It assumes that the JVM can grow to occupy the entire available main memory.
Is that the case?

Typically, programs can allocate as much memory as they like. We only take a fraction of the free physical memory for the manged memory. We could also take only half of the physical memory. Or, alternatively, fail with an exception that the maximum memory for the JVM is not set (-Xmx is missing). In my opinion, it is ok to take a fraction of the physical memory for local execution.

Okay, from skimming over some Oracle docs, it seems the default max heap is 1/4 of the physical memory. Let's use that.

1/4 of the physical memory seems sensible and indeed returns almost the same memory size as the maximum memory setting on my machine. I've adapted the pull request.

Looks good. Merging this with a modification of the tests that they permit the runtime exception in some cases.


Added trace level configuration option for Mbed OS usage.

Also fixed an issue where MBED_TRACE_MAX_LEVEL would be ignored when mbed_vtracef() was called directly.

Migrated from here https://github.com/ARMmbed/mbed-os/pull/8480

Mbed Edge relies that mbed_vtracef does not obey trace levels:
https://github.com/ARMmbed/mbed-edge-internal/blob/0.6.x/edge-client/edge-client/edge_client_impl.h#L156-L163
There is a need to have some info logging regardless of generic log trace level.

Oh boy.. :P

I guess it kind of makes sense. I removed the check from mbed_vtracef() but left the MBED_TRACE_MAX_LEVEL flag to mbed_lib.json.

Failing on
12:18:29 [Cobertura] Code coverage enforcement failed for the following metrics:
12:18:29
12:18:29 [Cobertura]     Lines's stability is 90.74 and set mininum stability is 90.77.
12:18:29
12:18:29 [Cobertura]     Conditionals's stability is 73.26 and set mininum stability is 73.56.
12:18:29
12:18:29 [Cobertura] Setting Build to unstable.

@jupe, flag name changed and added more documentation.

@jupe, flag name changed and added more documentation.

ups, I meant that should add documentation to readme instead.. 🙄😁 maybe you could add some mentions about this flag there as well...

Makes sense

Added section to README.md about tracing level.


Fix UTF-8 Character

Error occured before the fix

Added this to gradle.build file compileJava.options.encoding = 'UTF-8'
Fixed version
.
.
skript-packet-main-2.1.1.zip

Tested on 1.19.3

thanks works on 1.19.2

worked, thanks <3

works on 1.19.4

Official fix with 2.1.2


Updated code examples

Updated code examples with column names from AdventureWorks2012 database.

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

@hristochr : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

@hristochr Thank you for your efforts to bring this article up to date. The real problem, however, is that the article says "AdventureWorks" when in fact the DimEmployee table is in the AdventureWorksDW (or in this case, the AdventureWorksDW2012) database. Since the DimEmployee table still exists, we prefer to fix the original error, instead of changing the tables and fields. I will create a separate PR for this and let you know. Then I will ask you to close this PR.

Chris, Today I fixed this issue by changing all occurrences of "Uses AdventureWorks" in this article to "Uses AdventureWorksDW," since that's the database in which the DimEmployee table lives. The updated page will be published later today. After you review the changes, please close this PR. Thank you for bringing this to our attention!

@hristochr Please close this PR after you review the changes that I made and published in a separate PR.
WHERE (Transact-SQL)

closing.  thank you


https://dream2nix.dev/ is down

nix-repl> builtins.fetchTarball "https://dream2nix.dev/tarball/1.0"
warning: error: unable to download 'https://dream2nix.dev/tarball/1.0': SSL connect error (35); retrying in 349 ms
warning: error: unable to download 'https://dream2nix.dev/tarball/1.0': SSL connect error (35); retrying in 638 ms
warning: error: unable to download 'https://dream2nix.dev/tarball/1.0': SSL connect error (35); retrying in 1046 ms
warning: error: unable to download 'https://dream2nix.dev/tarball/1.0': SSL connect error (35); retrying in 2500 ms
error: unable to download 'https://dream2nix.dev/tarball/1.0': SSL connect error (35)

Hello,
If you check with e.g. curl -k you'll see that dream2nix.dev is just a "domain parked" site atm - and the registrar seems to use quite old SSL ciphers.
As the only location referring to that URL are the v1 docs: the v1 api isn't actually a supported thing yet.


Relative path does not work in default subroute

http://jsfiddle.net/nkovacs/9sx6vy04/
If you go to /foo/bar, then click the "go to baz" link inside foo, it works. But if you go to /foo, it doesn't work.

It works if the "Go to foo" link has a trailing slash: http://jsfiddle.net/nkovacs/9sx6vy04/1/, but I think it should work without the trailing slash too.
Also, the relative link doesn't get an active or active-exact class.

That's working as intended as far as I can tell: if you are at /foo and navigate to a relative link baz, you end up at /baz. If it assumes a trailing slash, when you go from /foo/bar, you'd end up at /foo/bar/baz instead of /foo/baz.
That's why you probably should avoid relative links when you can.

I wanted to use relative links so that I don't have to hard-code the base url within the subcomponents' templates. That way I can easily reuse the component or change its base url.
Basically what I'd need is:

child link: a way to link to /foo/bar inside Foo's template
sibling link: a way to link to /foo/baz inside Bar's template

in both cases without knowing that Foo is mapped to '/foo'. It could even be '/user/:id', so I'd need the dynamic segments inserted into the full url.

Take a look at named routes in 0.6.0: http://router.vuejs.org/en/named.html


Request : change the CW icon to be less confusing for some people

Hello,
In discussions with users, some have found that the button for flagging media as adult content was a little counter-intuitive, both in its positioning and in the proposed icon (they confused it with a post flagging button).
Would it be possible to make some changes in this area?
-> I think a modification to add the button at the same level as the "alt" button would be an interesting possibility.
-> As for the icon, I'm not sure what could improve things, maybe just putting a "CW" button in place of the icon would be less disruptive?
Please find attached the button as it is right now :

Thanks in advance and thanks for your work!

Honestly, even just making it more apparent that it is a button, would probably go a long way. Having two different styles of button directly next to each other isn't all that intuitive.
I'd go so far as to suggest giving it a plainly-stated label, "Add Warnings" or something to that effect.

I think this can be closed now! Was fixed in https://github.com/bluesky-social/social-app/pull/5780


getItemId() support

Library version: 0.2.2
Is it possible to add getItemId() support? This introduces a unique long type id for each item, and is used in conjunction with adapter.setHasStableIds(true). At the moment, I only have the absolute position, whereas I need the relative position.
Thanks! Your library has really helped! 😄

Isn't getItemId() just part of the base class? You should be able to just override that and make it return whatever ID you need to. My biggest beef with getItemId() is that it's restricted to just "long" types. My IDs are GUIDs, which don't map well to longs.

I'm afraid I did not explain this well enough above, but the support I needed in getItemId() was for differentiating between header and non-header items, as the item id I am generating is dependent on whether the item is a header.
As for the bit about it being restricted to the long type - is there any reason for that?
Thanks!

Well, there's a private member in the Sectioned adapter called: getSectionIndexAndRelativePosition(). Perhaps make that private and call that from your getItemId() override (not sure why it's private, it seems useful for derived classes).
As for the long type, that's Google's design decision, not sure how we could go about using different types for IDs. Perhaps a mapping of something like GUIDs to longs, but that seems yucky. Anyway, that's off topic, not sure why I brought it up.

Thought I should mention that newer versions expose ID getters for headers and items.

@afollestad awesome, thank you for the heads-up and the new feature!


MB-15217 Update page title on navigation

Jira ticket for this change
Summary
This PR adds a useTitle hook and PageTitle component that sets the page title and announces it using an aria-live element. This strategy is suggested here.
The title is generated from a base title dependent on which application the component is in, and a subtitle based on the path. This subtitle can also be manually overridden by calling useTitle(override) at the component level, which this PR does in cases where manual titles were already being set.
The PageTitle component is included in all three applications at the App level, which gives us application-wide coverage.
Note that in some instances, generated page titles may include IDs or alphanumeric params from the path (e.g. the Move review page). This PR does not address these instances specifically.
Setup to Run Your Code

💻 You will need to use two separate terminals to test this locally.
Terminal 1
Start the UI locally.
make client_run

Terminal 2
Start the UI locally.
make server_run

Additional steps

Access any page on the site
Note the title
Note that this title is different from others on the site
Note that a screenreader announces page changes

Great work on this @scottmries! Quick question: should I be testing this on the ephemeral deploy? I just poked around different pages and it might be something I'm doing wrong while navigating with VoiceOver but I'm not hearing the title announcement when I navigate between pages unless I tab to it. Additionally, I noticed that when navigating to edit a shipment the page's title contains many additional characters.
I took a screen recording for reference:
https://user-images.githubusercontent.com/46329147/231217073-5b9201a6-55ff-46f7-8813-906d5402c797.mov
Let me know if you have any questions! I'm also happy to jump on a call to pair.

@rileyorr I refactored this after an update to route handling, and the title should be announcing across the entire app now.

Thanks, @rileyorr! Good point about the DOD Info page; we also have issues wherever there are IDs in the path, since those get read out literally.
I wonder if a good strategy would be to do a pass over the site and make a list of manual title overrides, and knock them all out in one go. That could be part of this work, but like you said, I'd just as soon merge this and follow up on those later on.


"failure obtaining db row lock :lock wait timeout exceeded try restarting transaction" problem

I meet " failure obtaining db row lock :lock wait timeout exceeded try restarting transaction problem " error message when some jobs will be triggered. It seems a thread doesn't release the row lock and block the following job triggers.
How to avoid this problem ? and when it occurs, how to solve it?
google group ref:https://groups.google.com/forum/#!category-topic/quartz-scheduler/triggers--scheduling-questions/9A32WVWymq0

Closing outdated, unresolved issues. Please, reopen if still relevant.

See related #268


rcongui: Fix stats page horizontal scroll

The stats page had a horizontal scrollbar. Reorganizing the grids for a bit solves this.

Needs more work

Actually it was fine all along. I had some non-committed local changes that ironically introduced the same issue again

Has some merge conflicts now, could you resolve those please.

It's ready now

It's really ready now

I built it and connected it to our live server, looks fine to me.


What version should I install?

Hello, I want to try this plugin, I have installed the same version as in your readme file, but it ended up getting an error Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'notify').
Can someone tell me what commit should I use to install this package?
Project detail :

"vue": "^2.6.14"
"vue-cli-plugin-tauri": "~1.0.0-rc.1"
"tauri-plugin-sql-api": "github:tauri-apps/tauri-plugin-sql#2be17fae3832e55a18c0beaf7b76f341e7a755da",

And also where I can put my sqlite file?
The readme is only say that the path is relative to tauri::api::path::BaseDirectory::App.
But I don't know where is tauri::api::path::BaseDirectory::App or how to set up it

Currently using without any issues.
package.json
"tauri-plugin-sql-api": "github:tauri-apps/tauri-plugin-sql#b13efd1e9d8f6c862b67917f3c62536ed1c64e9c",

Cargo.toml
[dependencies]
tauri-plugin-sql = { git = "https://github.com/tauri-apps/tauri-plugin-sql", rev = "b13efd1e9d8f6c862b67917f3c62536ed1c64e9c", features = ["sqlite"] }

Recently updated my packages and noticed things weren't playing so nicely so sticking with this rev for now.

Thanks


HOSTEDCP-1374: external OIDC: copy Authentication OIDC client secrets through to guest

https://issues.redhat.com/browse/HOSTEDCP-1374
Worth noting there is a bug enabling TechPreviewNoUpgrade feature gate in the HC to allow Authentication type OIDC.  The HCP fails to rollout with crash loop in the KAS.  This will need to be solved before QE can verify this.
Update:
Found the issue with HCP rollout https://github.com/openshift/hypershift/pull/3377

/lgtm

/retest

/hold cancel
/test e2e-kubevirt-aws-ovn

/cherry-pick release-4.15


add RarArchiveReader

This works as expected, but will be harder to test than the other archive readers. The problem is that since the rarfile package can only read rar archives but not write them, we cannot generate the test data at runtime. One option would be to put fixed data in say test/assets/test_rar.rar and use this.

Could you add local test for this DataPipe by creating fake rar archive?

You mean I should create a rar archive and put it in the test folder so we can use that instead generating one at runtime?

And, do you also want to change the name to Opener to save another round of revamping?

Why Opener? Sure, the datapipe opens the file, but the interesting part is listing its contents. So I would say it should be RarArchiveLister.

You mean I should create a rar archive and put it in the test folder so we can use that instead generating one at runtime?

Either way is fine to me. Fakedata held in GH is currently used to test remote http access.

Why Opener? Sure, the datapipe opens the file, but the interesting part is listing its contents. So I would say it should be RarArchiveLister.

Yes, it does list its contents. But, it also opens file handles for each content. Then, extractor like https://github.com/pytorch/data/blob/673d3b210d16367fc944afed30971bf76c26e9e5/torchdata/datapipes/iter/util/extractor.py#L25

Either way is fine to me.

Just to be clear, we can't generate at runtime from Python, because rarfile can only read but not write archives. We could probably reach through the system layer and do it that way, but I prefer to provide static data in that case.

Yes, it does list its contents. But, it also opens file handles for each content.

True, but that means neither Lister nor Opener is a good git here, since both actions are performed. Maybe RarArchiveLoader?

Then, extractor like

Not sure why it was named Extractor, because that might be confusing. Executing "extract" on an archive will usually put the individual files on disk, whereas in this case it will simply remove the decompression.

Just to be clear, we can't generate at runtime from Python, because rarfile can only read but not write archives. We could probably reach through the system layer and do it that way, but I prefer to provide static data in that case.

Sounds great to me.

Not sure why it was named Extractor, because that might be confusing. Executing "extract" on an archive will usually put the individual files on disk, whereas in this case it will simply remove the decompression.

Yeah, you are right. I misunderstood the functionality of Extractor. Then, let's use loader.

I've added a test basically copy-pasting from the other archive readers. There are two sets of problems:

To dynamically create .rar archives we need the rar utility. We can install them on Ubuntu in CI, but I don't know if that is possible for MacOs and Windows. On the same note, rarfile can be installed by pip, but also needs a system utility to work. So this will be hard to test on anything but Ubuntu.
When 1. is resolved, the test is also failing and I don't know why. It would be good if one of the maintainers can have a look.

For macos, the Error from subprocess seems to be FileNotFoundError. By adding it to except condition for HAS_RAR_TOOLS, we can eliminate the Error.
I thought we were going to use static example. Then, we should be able to test it on these machines.

I thought we were going to use static example. Then, we should be able to test it on these machines.

Problem is that for the current test strategies we need to not only store the archive, but also somehow the values of the content. We could use the values of the dynamically generated files. This is begging for broken tests though, because we need to remember to always regenerate the archive if we change the create_temp_files function. Note that would only solve the generating part, but we would still need to install the system utilities for loading the archive.

For macos, the Error from subprocess seems to be FileNotFoundError.

Two issues here:

The error makes little sense, since there is no file to be found in the first place. I'll look into it.
That is not the error I was referring to. Currently no CI is showing it, because we are missing the system utilities. I'll fix CI and get back to you when the error is visible.

It's interesting. The error points to the second reading test, and the first test is passed.. I will grab this PR and test locally.

The problem came from rarfile module. When rarfile open a file from archive file handler, it would seek to the right offset (this happens when open is called). When we starts reading, if original file handle moves (seek is invoked afterward), the data (size) would be incorrect.
When we deplete the DataPipe first, it means we opened multiple files from archive file handle. And, these new file handles share a same archive handle and the archive handle has been seeked to the last file offset. When we start to read from the first file handle, it will raise Error as the offset and byte number are incorrect.
I added a patch to rarfile module locally and it works in all tests. But, it's a hack as this would only work with read, other methods can be broken.
--- a/rarfile.py
+++ b/rarfile.py
@@ -2433,9 +2433,6 @@ class DirectReader(RarExtFile):

         self._volfile = self._inf.volume_file
         self._fd = XFile(self._volfile, 0)
-        self._fd.seek(self._inf.header_offset, 0)
-        self._cur = self._parser._parse_header(self._fd)
-        self._cur_avail = self._cur.add_size

     def _skip(self, cnt):
         """RAR Seek, skipping through rar files to get to correct position
@@ -2460,6 +2457,9 @@ class DirectReader(RarExtFile):

     def _read(self, cnt):
         """Read from potentially multi-volume archive."""
+        self._fd.seek(self._inf.header_offset, 0)
+        self._cur = self._parser._parse_header(self._fd)
+        self._cur_avail = self._cur.add_size

         buf = []
         while cnt > 0:

I can open a PR to rarfile. But, I find the maintainer seems not interested to do another release, which can cause more headache.

I can open a PR to rarfile. But, I find the maintainer seems not interested to do another release, which can cause more headache.

Indeed, sending a PR to rarfile, getting it merged, and waiting for a new release is the usual way to go. But this is Python after all. If the former approach doesn't work for whatever reason, we can always opt to patch at runtime. That is bad practice, but at least it lets us move forward. If our patch is accepted and released in future, we can always remove our hack. Given that the issue is that no new releases are happening, I guess the risk is fairly low to break stuff.
I've added a commit that patches rarfile at runtime with the patch you suggested. We simply start it before opening a rarfile. I would stop it whenever the file is closed. Maybe we can piggyback on the behavior of StreamWrapper?

If the former approach doesn't work for whatever reason, we can always opt to patch at runtime. That is bad practice, but at least it lets us move forward. If our patch is accepted and released in future, we can always remove our hack. Given that the issue is that no new releases are happening, I guess the risk is fairly low to break stuff.

Yeah. Sounds great to me. I will send a patch to the repo later today.

Maybe we can piggyback on the behavior of StreamWrapper?

I don't think it would be a good idea as StreamWrapper is for more general purpose. I would like to keep rarfile related at a single source.


#456 BkReuse removed. Tests from BkReuseTest moved to BkBasicTest

#456. I removed BkReuse and BkReuseTest. All tests from BkReuseTest moved to BkBasicTest. Tests are still marked as ignored and must be refactored(#454).

@xupyprmv Many thanks for the PR, let me find a reviewer for it

@dmzaytsev please review,thanks

@xupyprmv according the original ticket

Let's get rid of BkReuse, move all tests from BkReuseTest to BkBasicTest and implement persistent connections inside BkBasic.

Let's add a puzzle to and implement persistent connections inside BkBasic.

@dmzaytsev

Let's add a puzzle to and implement persistent connections inside BkBasic.

according the original ticket (bottom of discussion)

@HDouss I see next line inside issue description:
implement persistent connections inside BkBasic
Is that comment related to current issue?

@xupyprmv No. It is a suggestion to let #418 as it is and not to invalidate it (but may be update it).

So, I'm not sure that we create something new. We need just update #418. Add something like:

implement persistent connections inside BkBasic instead of BkReuse. (#456)

Do you still think that we need to add new puzzle?

@xupyprmv anyway let's ask the  author
@HDouss what do you think about the puzzle?

@dmzaytsev On one hand, the puzzle have to be updated (or a new one created). On the other hand, reference to initial issue #306 is important, and the puzzle and associated issue (#418) are still relevant. I suggest to let the initial puzzle and only remove the BkReuse.accept reference.

@dmzaytsev I added puzzle with updated description (to avoid confusion)

@xupyprmv thank you!

@rultor merge

@rultor merge

@dmzaytsev I'm sorry, I don't understand you :( Check this page and try again please

@rultor merge

@dmzaytsev Thanks for your request. @yegor256 Please confirm this.

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@xupyprmv @yegor256 Oops, I failed. You can see the full log here (spent 13min)
INFO: "reading src/test/java/org/takes/rq/RqWithHeadersTest.java..."
INFO: "reading src/test/java/org/takes/rq/RqWithoutHeaderTest.java..."
INFO: "reading src/test/java/org/takes/rq/package-info.java..."
INFO: "reading src/test/java/org/takes/rs/RsGzipTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsJSONTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsPrettyJSONTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsPrettyXMLTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsPrintTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsTextTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsVelocityTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithCookieTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithHeaderTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithHeadersTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithStatusTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithTypeTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithoutHeaderTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsXSLTTest.java..."
INFO: "reading src/test/java/org/takes/rs/package-info.java..."
INFO: "reading src/test/java/org/takes/rs/xe/RsXemblyTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeAppendTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeDateTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeLinkHomeTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeLinkSelfTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeLinkTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeLocalhostTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeSLATest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeTransformTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeWhenTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/package-info.java..."
INFO: "reading src/test/java/org/takes/tk/TkCORSTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkClasspathTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkFilesTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkGzipTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkHTMLTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkMeasuredTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkProxyTest.java..."
INFO: "puzzle 377-e6ce369a 30/DEV at src/test/java/org/takes/tk/TkProxyTest.java"
INFO: "reading src/test/java/org/takes/tk/TkReadAlwaysTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkRedirectTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkRetryTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkSlf4jTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkTextTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkVerboseTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkVersionedTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkWithHeadersTest.java..."
INFO: "reading src/test/java/org/takes/tk/package-info.java..."
INFO: "reading src/test/resources/log4j.properties..."
INFO: "reading src/test/resources/org/takes/rs/inclass/imported.xsl..."
INFO: "reading src/test/resources/org/takes/rs/included.xsl..."
INFO: "reading src/test/resources/org/takes/rs/includes.xsl..."
INFO: "reading src/test/resources/org/takes/rs/simple.xsl..."
INFO: "reading src/www.takes.org/index.html..."
ERROR: "puzzle src/test/java/org/takes/http/BkBasicTest.java:57-58 has a very short description of just 11 words while a minimum of 20 is required"
/var/lib/gems/2.2.0/gems/pdd-0.15.4/lib/pdd.rb:150:in `rules': 1 errors, see log above (PDD::Error)
	from /var/lib/gems/2.2.0/gems/pdd-0.15.4/lib/pdd.rb:92:in `xml'
	from /var/lib/gems/2.2.0/gems/pdd-0.15.4/bin/pdd:90:in `<top (required)>'
	from /usr/local/bin/pdd:23:in `load'
	from /usr/local/bin/pdd:23:in `<main>'
container 4b1d60cfbda0f9be81e91c994b949913808f331c3a53d479d5ff279f79474003 is dead

@yegor256 I have to update puzzle description. Am I right?

@rultor try to merge

@rultor try to merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor try to merge

@xupyprmv @yegor256 Oops, I failed. You can see the full log here (spent 14min)
INFO: "reading src/test/java/org/takes/rq/RqWithHeadersTest.java..."
INFO: "reading src/test/java/org/takes/rq/RqWithoutHeaderTest.java..."
INFO: "reading src/test/java/org/takes/rq/package-info.java..."
INFO: "reading src/test/java/org/takes/rs/RsGzipTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsJSONTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsPrettyJSONTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsPrettyXMLTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsPrintTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsTextTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsVelocityTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithCookieTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithHeaderTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithHeadersTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithStatusTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithTypeTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsWithoutHeaderTest.java..."
INFO: "reading src/test/java/org/takes/rs/RsXSLTTest.java..."
INFO: "reading src/test/java/org/takes/rs/package-info.java..."
INFO: "reading src/test/java/org/takes/rs/xe/RsXemblyTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeAppendTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeDateTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeLinkHomeTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeLinkSelfTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeLinkTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeLocalhostTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeSLATest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeTransformTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/XeWhenTest.java..."
INFO: "reading src/test/java/org/takes/rs/xe/package-info.java..."
INFO: "reading src/test/java/org/takes/tk/TkCORSTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkClasspathTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkFilesTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkGzipTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkHTMLTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkMeasuredTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkProxyTest.java..."
INFO: "puzzle 377-e6ce369a 30/DEV at src/test/java/org/takes/tk/TkProxyTest.java"
INFO: "reading src/test/java/org/takes/tk/TkReadAlwaysTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkRedirectTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkRetryTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkSlf4jTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkTextTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkVerboseTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkVersionedTest.java..."
INFO: "reading src/test/java/org/takes/tk/TkWithHeadersTest.java..."
INFO: "reading src/test/java/org/takes/tk/package-info.java..."
INFO: "reading src/test/resources/log4j.properties..."
INFO: "reading src/test/resources/org/takes/rs/inclass/imported.xsl..."
INFO: "reading src/test/resources/org/takes/rs/included.xsl..."
INFO: "reading src/test/resources/org/takes/rs/includes.xsl..."
INFO: "reading src/test/resources/org/takes/rs/simple.xsl..."
INFO: "reading src/www.takes.org/index.html..."
/var/lib/gems/2.2.0/gems/pdd-0.15.4/lib/pdd.rb:150:in `rules': 1 errors, see log above (PDD::Error)
	from /var/lib/gems/2.2.0/gems/pdd-0.15.4/lib/pdd.rb:92:in `xml'
	from /var/lib/gems/2.2.0/gems/pdd-0.15.4/bin/pdd:90:in `<top (required)>'
	from /usr/local/bin/pdd:23:in `load'
	from /usr/local/bin/pdd:23:in `<main>'
ERROR: "puzzle src/test/java/org/takes/http/BkBasicTest.java:57-58 has a very short description of just 11 words while a minimum of 20 is required"
container fd769c4ddcd80092c2b6e32e02fce482d7748c5f8ba6a86d0474cfd572a4f5d5 is dead

@xupyprmv You have to make a longer description for the puzzle.

@yegor256 I updated puzzle description. Please continue with merge.

@rultor try to merge

@rultor try to merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor try to merge

@yegor256 Done! FYI, the full log is here (took me 16min)

@ypshenychka please, review this issue for QA compliance, as per par.24

@rultor deploy pls

@rultor deploy pls

@davvd OK, I'll try to deploy now. You can check the progress here

@rultor deploy pls

@davvd Done! FYI, the full log is here (took me 13min)

@dmzaytsev
According to our QA Rules:

The code reviewer found at least three problems in the code.

Only one issue was found during code review.
Please confirm that you'll try to find at least three problems while future reviews.

@ypshenychka I confirm

@dmzaytsev Thank you

@davvd Quality is acceptable.

@davvd Quality is acceptable.

@ypshenychka thanks, got it. everybody, please try to make it good next time

@dmzaytsev I added 10 mins to @ypshenychka (for QA review) in transaction 74692502... I just added 21 mins to your account, many thanks for your contribution (74692519).. 192 hours and 14 mins spent here... review comments (c=6) added as a bonus... +21 added to your rating, at the moment it is: +1913


[Question] What is the usage of node.Spec.Name in NodeSpec?

I found that only in docker remote API, we can update a swarm node name.
However I CANNOT update this in the cli.
And we cannot set node name constraint for a service when creating service.
Here are some tests:

update a node's name with pure remote API

POST <IP_ADDRESS>:2375/nodes/b0gnnf24np7vddvk46uglsfgr/update?version=10
BODY:
{
	"Availability": "active",
	"Name": "node-name", // this is to update a node name, which I think is useless.
	"Role": "manager",
	"Labels": {
		"foo": "bar"
	}
}

then we inspect this node with pure api:
GET <IP_ADDRESS>:2375/nodes/b0gnnf24np7vddvk46uglsfgr
RESPONSE:
{
  "ID": "b0gnnf24np7vddvk46uglsfgr",
  "Version": {
    "Index": 18
  },
  "CreatedAt": "2016-11-11T09:25:01.879387543Z",
  "UpdatedAt": "2016-11-11T10:10:56.320127766Z",
  "Spec": {
    "Name": "node-name", // this is something changed, which I think still useless. :)
    "Labels": {
      "foo": "bar"
    },
    "Role": "manager",
    "Availability": "active"
  },
  "Description": {
    "Hostname": "ubuntu",
    "Platform": {
      "Architecture": "x86_64",
      "OS": "linux"
    },
    "Resources": {
      "NanoCPUs":<PHONE_NUMBER>,
      "MemoryBytes":<PHONE_NUMBER>
    },
    ......
}

while in docker cli, I have not seen any node name update description:

root@ubuntu:~# docker node update --help

Usage:	docker node update [OPTIONS] NODE

Update a node

Options:
      --availability string   Availability of the node (active/pause/drain)
      --help                  Print usage
      --label-add value       Add or update a node label (key=value) (default [])
      --label-rm value        Remove a node label if exists (default [])
      --role string           Role of the node (worker/manager)

So,  I have not seen any usage of this field node.Spec.Name.
Description

Steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
root@ubuntu:~# docker version
Client:
 Version:      1.12.3
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   6b644ec
 Built:        Wed Oct 26 21:44:32 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.3
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   6b644ec
 Built:        Wed Oct 26 21:44:32 2016
 OS/Arch:      linux/amd64

Output of docker info:
root@ubuntu:~# docker info
Containers: 4
 Running: 1
 Paused: 0
 Stopped: 3
Images: 86
Server Version: 1.12.3
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 526
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: null host bridge overlay
Swarm: active
 NodeID: b0gnnf24np7vddvk46uglsfgr
 Is Manager: true
 ClusterID: 8gy19nuutimif702pwksgaiyg
 Managers: 1
 Nodes: 1
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
 Node Address: <IP_ADDRESS>
Runtimes: runc
Default Runtime: runc
Security Options: apparmor
Kernel Version: 3.19.0-25-generic
Operating System: Ubuntu 14.04.3 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.954 GiB
Name: ubuntu
ID: HRBI:EV6M:3NAC:CXZ3:BDS7:UOKL:PD6Y:ZCK4:V3PF:V7MR:CU2Y:3LGG
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 38
 Goroutines: 139
 System Time: 2016-11-11T19:15:24.90631625+08:00
 EventsListeners: 1
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Labels:
 a=b
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
VirtualBox

I think we have something discussed if the refereneced PR. Closing this.


Switching branches in Git makes editor freeze

I try to use git more. I noticed when I switch to a branch with quite some changes, the editor freezes (I see a white overlay) and never recovers.
This mostly happens when I have several tabs with files open. This isn't new, it was in some previous versions also the case.
The temporary solution is to close the editor before switching.

Hm, never had that issue. Do you have "auto reload changed files" enabled?

No I have not enabled this feature.

I've seen that a few times but never got to find a reproducible scenario...


Transformer model

Excuse me.Is your transformer model based on the <> model framework, changed to the structure of the picture and the output of the sentence? Are there any other changes?
Thanks!!! @raoyongming

Excuse me.Is your transformer model based on the <> model framework, changed to the structure of the picture and the output of the sentence? Are there any other changes?
Thanks!!!


Error in DB connection

When I execute the following command to deploy governance CDK app, replacing <PROFILE_NAME> with the AWS CLI profile name mapping to your governance account:
cdk deploy --app "python app_governance.py" --profile <PROFILE_NAME> --all
Getting error and see the attached screenshot.

Not able to proceed further because of the DB connection issue


activemq: 5.17.1 -> 5.17.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/apache-activemq/versions.
meta.description for activemq is: "Messaging and Integration Patterns server written in Java"
meta.homepage for activemq is: "https://activemq.apache.org/"
meta.changelog for activemq is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.17.2 with grep in /nix/store/2l15ljv4wjbnpf9h9f87yp5510wips1z-apache-activemq-5.17.2
found 5.17.2 in filename of file in /nix/store/2l15ljv4wjbnpf9h9f87yp5510wips1z-apache-activemq-5.17.2

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
activemq

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2l15ljv4wjbnpf9h9f87yp5510wips1z-apache-activemq-5.17.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A activemq https://github.com/r-ryantm/nixpkgs/archive/150e5d9f4591191484ca8416b6ffb8c9594eeb73.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2l15ljv4wjbnpf9h9f87yp5510wips1z-apache-activemq-5.17.2
ls -la /nix/store/2l15ljv4wjbnpf9h9f87yp5510wips1z-apache-activemq-5.17.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.


Instagram api uses stringified json for the usertags...

After inspecting the api through my phone, I discovered that the userTags were not working because you need to send a string with the JSON inside. Also added the possibility to tag users when configuring the photo, in order to avoid editing if not needed.

:) thanks


[🐛 BUG]: Build with Velox failing [problem with incorrect host go.buf.build]

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

What happened?
Hello! I can't build roadrunner with velox because I have a problem with it.
A log log line which says me what's happened:
195.9 {"level":"debug","ts":1694898397.604284,"msg":"[STDERR OUTPUT]","log":"go: github.com/roadrunner-server/roadrunner/v2023/internal/cli/jobs imports\n\tgo.buf.build/protocolbuffers/go/roadrunner-server/api/jobs/v1: cannot find module providing package go.buf.build/protocolbuffers/go/roadrunner-server/api/jobs/v1: unrecognized import path \"go.buf.build/protocolbuffers/go/roadrunner-server/api/jobs\": https fetch: Get \"https://go.buf.build/protocolbuffers/go/roadrunner-server/api/jobs?go-get=1\": dial tcp: lookup go.buf.build on <FILTERED>:53: no such host\n"}
This is because go.mod contains the wrong host go.buf.build, with the wrong url now
`module github.com/roadrunner-server/roadrunner/v2023
go 1.20
require (
....
go.buf.build/protocolbuffers/go/roadrunner-server/api v1.3.40
....
)
exclude github.com/uber-go/tally/v4 v4.1.7
require (
....
go.buf.build/grpc/go/roadrunner-server/api v1.4.40 // indirect
....
)
`
Version (rr --version)
2023.2.0
How to reproduce the issue?
Build roadrunner v2023 with velox
Relevant log output
https://pastebin.com/nrLVKRFK

Hey @loperd 👋🏻
Unfortunately, Buf removed their API for downloading Go dependencies. This solution has broken many libraries, including RR. I've moved the RR API proto to GitHub; however, you'll need to use the master (this is safe) branch for RR until I release 2023.3: https://github.com/roadrunner-server/velox/blob/master/velox_rr_v2023.toml#L5

Thank you 🙏🏻

I'll leave this ticket open, should be auto-resolved when 2023.3 is released.

@loperd Resolved by v2023.3.0. Velox configuration file updated: link

Hey @touqeershafi 👋
Please update plugins to their latest versions.


It seems not to support IE8

Clickoutside seems don't work in browser of IE8.I have to suuport IE8 because of users.Can you give me some advice.Thank you very much.

I think it's something really hard to achieve. react-datetime is using react-onclickoutside to handle those clicks, and if IE8 doesn't follow the modern event standards it might fail on checking those clicks. Maybe you can raise the issue in https://github.com/Pomax/react-onclickoutside


Visual Selector Screenshot and selector window aspect ratio is not consistent

Describe the bug
The visual selector shows a “cropped” screenshot that does not match the selectors (red selection boxes)
Version
Exact version: v0.46.04
To Reproduce

Set Browserless cromium with ws://2937404c-browserless-chrome:3000/chromium?launch={"defaultViewport":{"height":720,"width":1280},"headless":false,"stealth":true}&blockAds=true
Watch Google.com (Share-Link)
Try to use Visual Selector Tool

Expected behavior
That the two aspect ratios are consistent
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser: Chrome
Version: 128.0.6613.138

questions

does it happen with all sites?
does it happen using browser-steps?
does it happen when NOT using browser-steps?

Any site: yes
happen using browser-steps: no, in browser step shows correctly
visual selector shows wrongly before and after loading broswer step, nothing change

We dont support browersless, can you be sure you are using sockpuppetbrowser?
https://github.com/dgtlmoon/changedetection.io/blob/1b625dc18a6189f0b08e7efa7b94de7399d2d510/docker-compose.yml#L77-L90

currently my setup does not allow me to test, I plan to change setup so I can use docker freely, so if you want to close the issue and when I will test I will put a comment to let you know, thanks for the support 🙏

We dont support browersless, can you be sure you are using sockpuppetbrowser?
https://github.com/dgtlmoon/changedetection.io/blob/1b625dc18a6189f0b08e7efa7b94de7399d2d510/docker-compose.yml#L77-L90

Yep confirmed, sockpuppetbrowser works perfectly thx 3000 🙇♂


fix: take_bytes should not reuse indices null buffer

Which issue does this PR close?

Closes #6617.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Comet shouldn't be modifying a buffer that is not exclusively owned by it, this is UB and breaks all sorts of invariants

It is not about should or shouldn't. Comet reuses underlying buffers in the native reader. This is not exclusive to Comet.

It is not about should or shouldn't. Comet reuses underlying buffers in the native reader. This is not exclusive to Comet.

Rust's memory model disallows aliasing buffers in this way - see here. What you describe contravenes this model and is therefore UB. If you cannot guarantee that comet's native reader won't reuse the buffer behind Rust's back, you have to copy the memory into a fresh allocation.

Good. Let's not talk about Comet.
Let's simply focus on the kernel behavior. I think for take semantics (it is not "clone"), it should not just slice input arrays (data or null).

Why? Currently you have a kernel with inconsistent behavior. Isn't it enough? It makes confused for users and users might need to look into the implementation details to know what the inconsistency comes from.
For example, you tries to modify the indices array after calling take. But some cases you can do it because the array is not cloned, but some cases you cannot because the array is cloned.
I don't think this is good design for a kernel behavior.

For example, you tries to modify the indices array after calling take. But some cases you can do it because the array is not cloned, but some cases you cannot because the array is cloned.
I don't think this is good design for a kernel behavior.

The contract of a kernel should be on the semantic value of the output, this leaves kernels free to implement physical layout optimisations such as this. I don't believe we ever document or articulate any contract on how various kernels should behave w.r.t their inputs, this would not only be very restrictive but extremely fragile. Take is far from the only kernel that will simply clone input buffers, especially if one considers types like DictionaryArray where the underlying dictionary may or may not be recomputed depending on heuristics.
From the take kernel's perspective it has no way to know that you want to reuse the null buffer that it was given, so the correct thing is for it to not create a fresh allocation unnecessarily. If code then wants to try to reuse the buffer, it can try, falling back to performing the allocation if necessary. This is safe, sound and optimal.

I don't agree but I respect.

Perhaps @alamb might provide a second opinion.
FWIW you need to fix the aliasing bug, regardless of thoughts on what the take kernel is doing, what you describe is UB

The consistency of kernel behavior and design is higher than such minor optimization

Perhaps you could articulate what the policy is you expect to be? Should all kernels deep clone all inputs? Would this not have severe performance ramifications?

For now, users need to dig into the details inside the kernel when calling

Yes relying on behaviour beyond the documented contract of a kernel is a recipe for needing to dig into the details of said function. https://xkcd.com/1172/ comes to mind here.

I don't think this is a good design from a higher level point of view.

We may have to agree to disagree, but let's see what Andrew has to say

Perhaps you could articulate what the policy is you expect to be? Should all kernels deep clone all inputs? Would this not have severe performance ramifications?

For take kernel, the current behavior surprises me. Although we don't explicitly say it won't clone input array. But I image that for an user without any knowledge about its internals, it "takes" some elements from input array, so it means it "copies" values from the input and produce a "new" output.
This is the take kernel document:

Take elements by index from [Array], creating a new [Array] from those indexes.

I'm not sure for others, but for me "take" semantics is to have something separately, not share it. The current behavior is counterintuitive (at least to me).

FWIW my view: I agree with @tustvold many kernels try to reuse buffers / input arrays if possible to minimize allocations / optimize performance and I would expect arrow-rs to do so if possible (for whatever kernel) whenever dealing with immutable buffers (behind Arc).

Consider the semantics of take kernel, its output array should not reuse input array. The current behavior looks incorrect.

I don't fully understand this assertion. Is the idea that since the docs say

Take elements by index from Array, creating a new Array from those indexes.

That implies the output array should be entirely new (aka not share anything with the source arrays)?

Instead of modifying the take kernel, I wonder if we could add some explicit API for comet to call that would force a null buffer copy (or maybe we help write a workaround)
Like could we have comet call some function like
/// Ensure that all underlying buffers are purely owned (not shared)
fn ensure_owned(array: ArrayRef) -> ArrayRef {
  // downcast DictionaryArray and look for slice / shared references
  ...
}

🤔

The consistency of kernel behavior and design is higher than such minor optimization. For now, users need to dig into the details inside the kernel when calling, if they don't have any knowledge about this crate. I don't think this is a good design from a higher level point of view.

I agree that having to dig into the internals of the implementation is not a good design. Maybe we should better document that the arrow-rs kernels / implementations all assume immutable buffers and have various optimizations that may reuse portions of the input

My biggest concern is not really about this particular PR, but more the general question of "what can be assumed about the output of a function given it got an ArrayRef in ?
Do we need to document / test the various kernels and operations that guarantee no part of the input array would be reused 🤔

Do we need to document / test the various kernels and operations that guarantee no part of the input array would be reused 🤔
Maybe we should better document that the arrow-rs kernels / implementations all assume immutable buffers

We do have the beginnings of the docs here - https://docs.rs/arrow-array/latest/arrow_array/#zero-copy-slicing
But we could definitely document better that the array abstractions, and by extension the kernels that operate on them, are built around reference counted immutable buffers, and implementations should not make any assumptions about sole ownership of buffers. If codepaths require sole ownership they must use the safe APIs for doing so, such as into_mutable.

What can be assumed about the output of a function given it got an ArrayRef in

Tbh I am a little confused why one would make any assumptions here, but it certainly couldn't hurt to document that we avoid allocating whenever possible.

Take elements by index from Array, creating a new Array from those indexes.

That implies the output array should be entirely new (aka not share anything with the source arrays)?

Once you took something from someone, do you expect that you and someone both have it? 🙂

For a DictionaryArray would you expect it to create a deep clone of the dictionary values? What about a ViewArray where the whole design is to avoid copying the values buffers?
I think this discussion is getting a bit philosophical and not particularly productive, we fundamentally cannot change take to always perform a deep copy of the input without significantly regressing performance. It is a fundamental design of arrow-rs, and other arrow implementations, that buffers are immutable reference counted memory regions and kernels are free to take advantage of that.
Perhaps you might be willing to suggest some documentation improvements that would clarify this to avoid future confusion?

I will add some documentation to take kernel to clarify its behavior.
I am adding workaround in Comet codebase and will also have some custom unpack dictionary and take to prevent the issue. So it won't rely on arrow-rs kernel.

For a DictionaryArray would you expect it to create a deep clone of the dictionary values? What about a ViewArray where the whole design is to avoid copying the values buffers?

What are you trying to say? When unpacking dictionary or take it, how do you not copy dictionary values?

Let me close this. I will open another PR for documenting the kernel. Thanks for your review.

I think this discussion is getting a bit philosophical and not particularly productive, we fundamentally cannot change take to always perform a deep copy of the input without significantly regressing performance.

Btw, note that take already does deep copy for other cases. It only clones the null buffer for a particular case. I don't think that such change brings significant regressing performance. If any, it already does. Not to mention it is null buffer, not the data buffer.

Btw, note that take already does deep copy for other cases.

Nobody is disputing this, however, there are many cases where it reuses buffers and entire arrays. Changing this would significantly regress performance for those cases

Thank you @viirya  and @tustvold  -- I hope we end up in a place that is good for Comet and good for other users of arrow as well

Nobody is disputing this, however, there are many cases where it reuses buffers and entire arrays. Changing this would significantly regress performance for those cases. Yes copying a null buffer as in this PR isn't a terrible regression, but the broader principle you're advocating for that take should never reuse its inputs would be terrible for performance

I don't really find "many" cases take reuses buffers and entire arrays. Would you mind point it out?

From a quick scan

https://github.com/apache/arrow-rs/blob/master/arrow-select/src/take.rs#L653
https://github.com/apache/arrow-rs/blob/master/arrow-select/src/take.rs#L669
https://github.com/apache/arrow-rs/blob/master/arrow-select/src/take.rs#L540

Not sure if that counts as "many", but it is very important for those cases that we don't clone.
Taking a step back though, it is kind of irrelevant whether the kernel reuses or doesn't reuse buffers, arrow is explicitly designed to encourage such reuse and we shouldn't be seeking to document or make commitments either way, as doing so would be extremely fragile and hard to test.

Yea, these are the only cases and I believe that are not "many". Not to mention that they are reasonable cases as these arrays are designed to be shared/reused. I already mentioned this in previous comment.
That's why I don't understand that you claim that performance regression will be a problem for such a change. Except for above cases, the only cases reusing is happening on null buffer in take kernel. I doubt it will bring much extra cost on copying them.
With a negligible cost, making take behaves more close to the common semantics of "take" (philosophical? 🙂 ). I don't get why this patch gets string objection.
Anyway, as the workaround was already added to bypass the issue. I respect your decision here.

I don't get why this patch gets string objection.

You are regressing performance, albeit marginally, to achieve something that isn't part of any kernel's public contract, nor that would be maintained or tested going forwards. So it is hard for me to see this as anything but a net loss, it doesn't add anything and makes performance worse.

to achieve something that isn't part of any kernel's public contract

function name is not public contract? You design a function called "not_copy" but it copies?

You design a function called "not_copy" but it copies
Not to mention that they are reasonable cases as these arrays are designed to be shared/reused.

These statements are contradictory.
I'm not going to engage further on this ticket