[Launcher] Multiple windows with focused state

Describe the bug
Possible to trigger states where multiple applications appear focused
To Reproduce
Steps to reproduce the behavior:

Open a favorite application
Switch to another workspace
Open another favorite application
Both applications now have the focused state

Expected behavior
Only one application should have the focused state
System information:

Distro: NixOS 24.05 at https://github.com/NixOS/nixpkgs/commit/805a384895c696f802a9bf5bf4720f37385df547
Compositor: Hyprland v0.40.0
Ironbar version: Latest commit from yesterday at https://github.com/JakeStanger/ironbar/commit/6edfe940000d1a54a92fb71dd6254909724f9307

Configuration
Ironbar overlay for 24.05
final: prev: {
  ironbar = prev.ironbar.overrideAttrs (oldAttrs: rec {
    version = "0.15.1-unstable-2024-05-31";

    src = prev.fetchFromGitHub {
      owner = "JakeStanger";
      repo = "ironbar";
      rev = "6edfe940000d1a54a92fb71dd6254909724f9307";
      hash = "sha256-AD9lVgAfFUORYR/fCzO1vD7Ve8O9gFpJT/9dcZ47B2o=";
    };

    cargoDeps = oldAttrs.cargoDeps.overrideAttrs (prev.lib.const {
      name = "${oldAttrs.pname}-${version}-vendor.tar.gz";
      inherit src;
      outputHash = "sha256-eqyKX5eLiXjUFt/R5X8OIokz4s10GvFKvzoNURiWeAo=";
    });
  });
}

Additional context

Switching to the first open application removes the focus from the second one, and focus remains fixed when switching between the windows.
Switching to a new workspace and then the previous one (with the 2nd application) results in applications 1 and 2 appearing focused.

Only seems to occur when switching to a workspace that has application 2 focused
Opening a 3rd application causes the same behavior with the 1st and the 3rd
Application 1 remains focused when switching to an empty workspace

FWIW I can reproduce this with NixOS 24.05 + Hyprland v0.39.1 at https://github.com/hyprwm/Hyprland/commit/31d055f6d487bd72fca5b10822e3f3bd95e94cc9 + Ironbar at https://github.com/JakeStanger/ironbar/commit/6edfe940000d1a54a92fb71dd6254909724f9307


Outline view doesn't follow JS code in .vue files

VSCode Version: 1.25.0
OS Version: macOS 10.13.5

Steps to Reproduce:

Open a .vue file (single-file component)
Move the cursor through methods, etc

The odd thing is, if I click on a method in the outline view, it skips to the appropriate line in the code. But it can't detect where I am if I move the cursor around. The HTML/template portion of the .vue file works as expected.

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

Please file this issue against the vue extension


A missing folder on run

When I try docker-compose up, it is looking for a secrets directory. but there are no examples of the secrets or the folder is not there.
There are two folds into this conversation;

If a folder is empty and nothing to commit inside it, git doesn't add empty directories. There is a way to add empty directories in git Hin't; it's a trick with .gitignore.
The example files can be in that directory. However, the directory won't be empty at that point. In .gitignore you have two options ignore this, do not ignore that (including globbing *,? but you already know that anyways). Have a look at that! 👍

If this a question for me (or ask questions not to make mistakes), I'll whip your a***. Don't ask, wait. Try it, commit it, raise your PR. Even if you mess. This is the reason of SCMs (Source Code Manamgent - git in this case). You can return the code to any state as you would like to.
Hint;
https://git-scm.com/docs/git-status
https://git-scm.com/docs/git-check-ignore


Feature: facebook authorization and facebook post

Pull Request Test Coverage Report for Build 244

10 of 80 (12.5%) changed or added relevant lines in 6 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-1.2%) to 60.888%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/commons/events/ui/BrowserUrlChangedEvent.java
0
5
0.0%

src/main/java/seedu/address/logic/parser/FacebookPostCommandParser.java
0
5
0.0%

src/main/java/seedu/address/logic/parser/AddressBookParser.java
1
6
16.67%

src/main/java/seedu/address/ui/BrowserPanel.java
8
18
44.44%

src/main/java/seedu/address/logic/commands/FacebookLoginCommand.java
1
21
4.76%

src/main/java/seedu/address/logic/commands/FacebookPostCommand.java
0
25
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/address/logic/parser/AddressBookParser.java
1
51.85%

Totals

Change from base Build 243:
-1.2%

Covered Lines:
2030

Relevant Lines:
3334

💛  - Coveralls


Support stored procedures or SQL files?

How do I execute a stored procedure? How to execute SQL file?

@songlinshu which database are you asking about here?

Hi. I came in here looking for stored procedure support. Mariadb and postgresql. Couldn’t find it in the docs. Is it supported yet?


Package list for Go has empty package names

Package Index go-all.md - line 249 to 1005, generated by Generate-Package-Index.ps1 has empty package names, probably coming from the management libraries.

Those all have empty names in the csv https://github.com/Azure/azure-sdk/blob/main/_data/releases/latest/go-packages.csv as well. So what would we put?

@weshaggard skip them they should not go on the package list as they are track 1 and don't go on our release site on GitHub IO as well

OK that should be easy enough.


feat: dynconfig refresh and notify listeners

Signed-off-by: Gaius<EMAIL_ADDRESS>
Description

Dynconfig refresh and notify listeners.

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

Codecov Report

Merging #1579 (54df6c4) into main (22bfa87) will increase coverage by 0.20%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1579      +/-   ##
==========================================
+ Coverage   44.66%   44.86%   +0.20%
==========================================
  Files         116      116
  Lines       10568    10576       +8
==========================================
+ Hits         4720     4745      +25
+ Misses       5362     5342      -20
- Partials      486      489       +3

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
44.86% <0.00%> (+0.20%)
:arrow_up:

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

Impacted Files
Coverage Δ

client/config/dynconfig_manager.go
65.34% <0.00%> (-2.70%)
:arrow_down:

scheduler/config/dynconfig.go
26.47% <0.00%> (-1.09%)
:arrow_down:

...ent/daemon/peer/peertask_piecetask_synchronizer.go
47.67% <0.00%> (+0.84%)
:arrow_up:

client/daemon/peer/peertask_piecetask_poller.go
46.89% <0.00%> (+15.86%)
:arrow_up:


Always return transcription with highest confidence in recognize_google()

Fixes https://github.com/Uberi/speech_recognition/issues/177
Removed checking for "transcript" key since the Speech API should always be returning that.

Thanks!

Sometimes i get the error

('Unexpected error:', <type 'exceptions.TypeError'>)

using this path.

The problem is, that with standard option show_all=False, the function now returns the structure

{u'confidence': 0.98267901, u'transcript': u'hello how are you'}

Before it was only the text:

hello how are you


Bump version number in vioscsi/buildAll.bat when doing a release

Would it be possible to occasionally update the RHEL_RELEASE_VERSION, BUILD_MAJOR_VERSION, BUILD_MINOR_VERSION in vioscsi/buildAll.bat when doing a release?
The source, without any special environment variables, appears to build version "<IP_ADDRESS>000". The latest binary releases from this location:
http://alt.fedoraproject.org/pub/alt/virtio-win/latest/images/
..have version "<IP_ADDRESS>00".
I guess in your usual release process you set RHEL_RELEASE_VERSION etc. in the environment before building?
It's not necessary to update the version very often, just every release would be ok, so we can translate from binary version to approximate source version.

Hi,
Thanks for the suggestion. We might start doing this.
For now the way to figure the source for specific release is the following:
Look at the last number of the version (x.x.x.1000 in current build). The first part of it is a build number (in this case 100).
Now go to the stable branch of the source https://github.com/YanVugenfirer/kvm-guest-drivers-windows/tree/stable and look for the tags or commit message with message mmXXX <-->bYYY. YYY - will be the external build number (100): https://github.com/YanVugenfirer/kvm-guest-drivers-windows/commit/8a64980edc9f85dc0f1265827a23f2c053a4821f
Best regards,
Yan.

Sounds good, thanks.


Dev -> Test: Funding Source Features

Links to Jira Tickets
SIMSBIOHUB-171
SIMSBIOHUB-172
SIMSBIOHUB-173
SIMSBIOHUB-174
SIMSBIOHUB-181
SIMSBIOHUB-195
Description of Changes
Funding sources are a higher order item (like projects) so they are created to be used/ associated to surveys at a later time. A management page has been added for admins to create, edit and delete funding sources as required. A funding source is no longer associated to a project but now to a survey and both create/ edit forms for projects and surveys have been updated to reflect that.
Testing Notes

Will re-open when other PRs are merged in.


Update rust-installer submodule

Closes #1798

@bors: r+ p=1

:pushpin: Commit a84c5b9 has been approved by alexcrichton

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: force

:hourglass: Testing commit a84c5b9 with merge 81d09cf...

:hourglass: Testing commit a84c5b9 with merge a568977...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-64


534507E6 - Sega Rally (Sega Rally Revo)

Validation

[X] I've read the FAQ.
[X] The Xenia build used is from a canary branch. (not MLBS/AlexVS/master/pull requests, etc)
[X] This issue isn't for help (tech support) with Xenia, or a game.
[X] I've checked if this game doesn't already have a compatibility report.

Xbox 360 Marketplace link
https://marketplace.xbox.com/en-gb/Product/SEGA-Rally/66acd000-77fe-1000-9115-d802534507e6
Xenia version
https://github.com/xenia-canary/xenia-canary/commit/3876a1632ac8355fde37808d604e2050d8864f72/
Issues
No audible bugs I can hear or visual bugs I can see.
Main menu feels a little laggy, but appears to animate at near full speed.
In game everything runs smoothly and with VSYNC off some elements run above 30fps.
The problem is that the main game logic appears to be running slowly.
The last two screenshots were taken approximately 10 seconds apart.  Take note of the lap time counter in the top left corner.  Like other elements, while it is running slowly it is animating at full speed.
This could be affecting the CPU drivers as on a few occasions I have seen them driving like they've downed a bottle of whiskey each.
My system:
Windows 10
Ryzen 1700x
16GB ram
Radeon RX 6800
There are a few Sega Rally on Xenia videos on Youtube.  Some show the same slowdown as I have.  Those that list system specs are quite varied.
Log
xenia.zip
Screenshots

Labels
state-playable

Just a quick update.  Upgraded to a R7 5800x on a x570 board and a clean install of Win10.  No change.

@Margen67 I've just tested with the latest build.  Gameplay is still in slow motion, roughly 3 real seconds to every one second tick of the in game timer.  Is there a more suitable tag?


proxy.golang.org: module upgrade not detecting new tag

What version of Go are you using (go version)?

$ go version
  version go1.14.4 darwin/amd64

Does this issue reproduce with the latest release?
yes, using latest stable release
What operating system and processor architecture are you using (go env)?
go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/kevingentile/Library/Caches/go-build"
GOENV="/Users/kevingentile/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/kevingentile/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/kevingentile/golinks-daemon/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/gq/3dt_qkm13qj2rpzqqhmdydfw0000gn/T/go-build868196028=/tmp/go-build -gno-record-gcc-switches -fno-common"
$ go env

What did you do?
go clean -modcache
go get<EMAIL_ADDRESS>

What did you expect to see?
Package upgraded to recently pushed tag (from v0.1.0 -> v0.1.1
What did you see instead?
go<EMAIL_ADDRESS>verifying module<EMAIL_ADDRESS>reading<EMAIL_ADDRESS>410 Gone
        server response: not found<EMAIL_ADDRESS>invalid version: unknown revision v0.1.1

It's worth noting that prior to attempting this upgrade I deleted two invalid tags from the govice/golinks repository (v0 and v.0.1.1)

Tried this upgrade again ~11 hours after posting this issue and it succeeded.

@katiehockman @hyangah @heschik

The tag was added a few minutes after this issue was posted. I haven't fully verified but this is almost certainly a case where the version was fetched before it actually existed. In that case proxy.golang.org will cache its nonexistance for up to about half an hour.

Is there a way to flush the cache from my end? @heschik

Sorry, no; generally we just suggest you make sure not to try to fetch it before you tag it. Or just wait a little while.
Setting GOPRIVATE for modules you own will avoid accidentally fetching them early, though by the same token doesn't guarantee that they have been successfully fetched.


Alert Modal Background Revisions - Design Updates

When there is no content on screen alert modals backgrounds are the same color as the app background.

[ ] Design: Change layouts and alert team.
[ ] Change all instances of alert modals with new backgrounds.

Closing this ticket out, per latest design updates this enhancement is superseded by: #493


Add missing stats in istgt

Is this a BUG REPORT or FEATURE REQUEST?
FEATURE REQUEST

What happened:
Currently we don't have all the stats from istgt such as :

r/w latency
uptime

What you expected to happen:

istgtcontrol iostats should show the above stats

How to reproduce it (as minimally and precisely as possible):

run istgtcontrol iostats and you will be able to get available stats.

Anything else we need to know?:

Link : www.github.com/openebs/istgt
Checkout at replication branch and raise your PR against it.

This feature is now available into istgt via istgtcontrol iostats command. Closing via https://github.com/openebs/istgt/pull/85 and https://github.com/openebs/istgt/pull/175


Handle conversion specifier c, s, %

I handled the conversation specifier for c, s , and %. It was tested on my machine with the main.c provided by alx


Feature request: option for SSD cached + HDD mass storage configurations for installer

The tools are there as described in #126. I acchieved it by means of LVM though. But would be great to have an option in the installer for this configuration scenario:

SSD for a 100Mb-ish /boot partition and 30Gb / parition, the rest for cache. Optionally use another partition for fast storage mounted on /opt/fast_storage
one or more HDD in a single mass storage raid/volume group to mount /home.

Once it is installed everything should work as it is transparent to the filesystems.

This is an esoteric configuration a d the distribution should not support it provided #237 is solved. Users with this configuration should go for a custom solution (I know I do ;) )


Quote Mermaid node and edge labels

This PR quotes label strings in the Mermaid produced graph, for both nodes and edges. The way pipdeptree currently produce the Mermaid graph is good and working well. This is a safety measure to make sure the output is more resilient.
See the result in Mermaid's live editor:

For the record, I found out about this more robust syntax while debugging an edge-case with Mermaid itself, in which click cannot be used for node IDs.

Thanks @gaborbernat for the merge! No need to hurry to release a v2.5.2 by the way. I'll probably have some tweaks in the same spirit to propose as PR in the near future as I uncover some other edge-cases of Mermaid.


Control operation combinations

Prevent aggregation operations besides First and Last to be paired with order by

https://github.com/trane-dev/Trane/issues/146


Use TypeScript

As part of my own work with this project, I converted the semantic search part of the project to TypeScript.
Is this something you would be interested in merging?
If so, I could make another PR :-)

Thanks for your efforts! So which files did you convert to TS - just semantic.js or also index.js?
If it's just semantic.js with only 61 lines, maybe adding TS would be overkill?
I am not an expert on TS but if it helps our contributors we could just as well go for it. We're only 3 people atm so I guess I'll just ping @VarunNSrivastava and ask for your opinion on this.

I am not an expert on TS but if it helps our contributors we could just as well go for it.

Every TypeScript dev has to know JavaScript anyway, but not every JavaScript dev requires to know TypeScript, so you pretty much shrink your potential contributer base to a subset.
I liked your initial vision of being "as simple as it gets" and now you are nearly sold by TypeScript... open your eyes, please.


pythonPackages.xmldiff: init at 2.4

Motivation for this change
Package that I need - the latest release 2.4 is from 2019, but there is still recent activity on the GitHub repo, so it seems to be still alive.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 116091 at 5cc63f33 run on x86_64-linux 1

2 packages built successfully:

python38Packages.xmldiff
python39Packages.xmldiff

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 116091 run on x86_64-linux 1

2 packages built:

python38Packages.xmldiff
python39Packages.xmldiff

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

python38Packages.xmldiff:

warning: python-include-tests
Add a checkPhase for tests, or at least pythonImportsCheck.
See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/python-include-tests.md

How does nixpkgs-hammering come to this conclusion? I checked for tests being run and something is definitely happening:
$ nix-build -K -A python38Packages.xmldiff --check
[...]
Executing setuptoolsCheckPhase
running test
[...]
Ran 114 tests in 0.609s

OK
Finished executing setuptoolsCheckPhase
$ nix-build -K -A python39Packages.xmldiff --check
Executing setuptoolsCheckPhase
running test
[...]
Ran 114 tests in 0.600s

OK
Finished executing setuptoolsCheckPhase


docs/fees limitation in connectors page

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
The changes present in this pull request are referred to the ones submitted at https://github.com/hummingbot/hummingbot/issues/4936. These changes add a new section in beaxy, blocktane, coinzoom, digifinex, kucoin, and okex markdown specific exchange documentation files present in the documentation/docs/exchanges path.
Tests performed by the developer:
The related changes have been tested to render as expected by using a MarkDown compatible editor/viewer.
Tips for QA testing:
No further testing is needed for this specific pull request.

Hello @sisco0
A Pull Request Proposal was created related to this PR: https://snapshot.org/#/hbot.eth/proposal/0x750537d889409af2b48decac6d0ae779cd7beb5d73a26af9b39ff608edd02f7f
Make sure to keep an eye on community reviews for the following days.
Thank you for contributing with Hummingbot Client!

I think this section should be more descriptive and describe what the impact is on users. If fees are not available via API, how does that impact the history calculation?

Could you kindly answer to the question @dennisocana ?

Closing this PR as we have migrated the documentation repository here: https://github.com/hummingbot/hummingbot-site


Updates to bot-builder-dotnet-dialogs.md

In bot-builder-dotnet-dialogs.md, there is a reference to Prompts.Confirm. However, that's a function from the Node.js version of the SDK - the C# function that's actually called in the associated code sample is PromptDialog.Confirm.

I pushed an additional commit to update the links within this article which pointed to 3.8-specific URLs to instead point to 3.11.0 URLs, as the 3.8 versions were dead and were now dead and redirecting to the 3.11.0 versions with an unnecessary message explaining the redirect.

Thanks for the PR, @sphanley . I will merge this in and get it into the next daily release.

Sure thing, @RobStand! Thanks!


dashboard: adds default placeholder

adds dashboard image (https://pixabay.com/vectors/network-transmission-system-concept-7174082/)
closes https://github.com/inveniosoftware/invenio-administration/issues/52

Computer screen

Mobile screen

Updated screenshot


MiniflareCoreError [ERR_RUNTIME_FAILURE]: The Workers runtime failed to start.

What version of Hono are you using?
"hono": "^4.5.5"
What runtime/platform is your app running on?
Cloudflare Workers
What steps can reproduce the bug?
Hello, I have discovered that an error occurred while executing on my machine. Could you please inform me if Hono currently does not support running on Windows machines?
Below are the steps I followed during the execution.
equipment information:

win 11
nodejs: v22.1.0
pnpm: v9.6.0

code

package.json

{
  "scripts": {
    "dev": "wrangler dev src/index.ts",
    "deploy": "wrangler deploy --minify src/index.ts"
  },
  "dependencies": {
    "hono": "^4.5.5"
  },
  "devDependencies": {
    "@cloudflare/workers-types": "^4.20240529.0",
    "wrangler": "^3.57.2"
  }
}

src/index.ts

import { Hono } from 'hono'

const app = new Hono()

app.get('/', (c) => {
  return c.text('Hello Hono!')
})

export default app

Terminal

wuche@cd MINGW64 /c/wudi/github/short-url
$ pnpm create hono my-app
.../1914eafb9ca-1394                     |   +1 +
.../1914eafb9ca-1394                     | Progress: resolved 1, reused 0, downloaded 1, added 1, done
create-hono version 0.12.0
✔ Using target directory … my-app
? Which template do you want to use? cloudflare-workers
✔ Cloning the template
? Do you want to install project dependencies? yes
? Which package manager do you want to use? pnpm
⠇ Installing project dependencies

wuche@cd MINGW64 /c/wudi/github/short-url/my-app
$ pnpm i
 WARN  2 deprecated subdependencies found<EMAIL_ADDRESS><EMAIL_ADDRESS>Packages: +83
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Progress: resolved 109, reused 83, downloaded 0, added 83, done

dependencies:
+ hono 4.5.5

devDependencies:
+ @cloudflare/workers-types 4.20240806.0
+ wrangler 3.71.0

Done in 4.6s

wuche@cd MINGW64 /c/wudi/github/short-url/my-app
$ pnpm run dev

> @ dev C:\wudi\github\short-url\my-app
> wrangler dev src/index.ts

 ⛅ wrangler 3.71.0
-------------------

⎔ Starting local server...
╭────────────────────────────────────────────────────────────────────────────────────────────────────╮
│ [b] open a browser, [d] open Devtools, [l] turn off local mode, [c] clear console, [x] to exit     │
╰────────────────────────────────────────────────────────────────────────────────────────────────────╯
C:\wudi\github\short-url\my-app\node_modules\.pnpm\wrangler@3.71.0_@cloudflare+workers-types@4.20240806.0\node_modules\wrangler\wrangler-dist\cli.js:29768
            throw a;
            ^

MiniflareCoreError [ERR_RUNTIME_FAILURE]: The Workers runtime failed to start. There is likely additional logging output above.
    at #assembleAndUpdateConfig (C:\wudi\github\short-url\my-app\node_modules\.pnpm\miniflare@3.20240806.0\node_modules\miniflare\dist\src\index.js:9178:13)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Mutex.runWith (C:\wudi\github\short-url\my-app\node_modules\.pnpm\miniflare@3.20240806.0\node_modules\miniflare\dist\src\index.js:3521:16) {
  code: 'ERR_RUNTIME_FAILURE',
  cause: undefined
}

Node.js v22.1.0
 ELIFECYCLE  Command failed with exit code 7.

What is the expected behavior?
No response
What do you see instead?
No response
Additional information
No response

Hi @WuChenDi
This may be related to this Workers SDK issue: https://github.com/cloudflare/workers-sdk/issues/4709#issuecomment-2289638966

@yusukebe
Thank you for your response. I will look into the best way to handle this.


Questions about the keyboard overlay

[ In retrospect it was probably a mistake to post these in a closed issue (#138), although its topic was appropriate, so I'm adding them here in full view :-) ]
Since I upgraded to the "keyboard" target, my ARM Chromebook has become a lot more comfortable to use (I had tried to hack some Search key shortcuts with xmodmap myself, with little success).
I do have some follow-up questions about how this is done :

was it finally implemented through xbmap ?
in the initial xmodmap approach, why and how was the Search key remapped to Mode_switch ?
for non US keyboards, the wiki says that "setxkbmap -layout xx" should be ran each time the chroot is restarted : on mine (xfce + raring), the layout seems to stick around, even across reboots.
At first I thought this was because I left "save session" checked by default in the logout panel, but it works just the same with that unchecked.
About once a day, after using some Search+ combination, I end up stuck in Caps Lock. Is there a way to get out of that mode, other than restarting xfce ?
now I would like to define my own Search+key combos (with plain letters, to launch apps).
What would the simplest way to go about this without breaking anything else ? xmodmap ? .xbindkeys.scm ?
Most user-level utilities (like xfce's shortcut manager) don't recognize the Search key (Overlay1_Enable in xev) as a modifier, and by tinkering with .xbindkeys.scm I even managed to disable the functionality on F6/F7 :-)

Thanks for any tips,
fp

I found this very useful. It is also possible to change Mod4 to Control (https://pastebin.com/e68xg2mG). Very useful for Emacs users ^_^


Add sound effects

Add sound effects when a player shoots. Should we add a sound effect when a player is hit?

https://freesound.org/ seems like a good place for sound effects. Find a "laser" sound effect for the shots, kinda like in Star Wars. A hit sound seems like a good idea too.

Planning on using this sound effect. https://freesound.org/people/bubaproducer/sounds/151022/

:+1:

Added a retro explosion sound when hit.  https://freesound.org/people/jalastram/sounds/317750/

Why did you close this? You haven't put up a pull request.

:sound: #47


Remove attachments from 620632

Detailed guide 620632 has 454 attachments, therefore the draft cannot be discarded as it results in a gateway timeout error. This migration removes all the attachments from the draft to allow it to be discarded.
Zendesk: https://govuk.zendesk.com/agent/tickets/1417917

:+1: quando i test sono verdi


chore(pie-monorepo): DSW-1540 extract create and publish snapshot step into its own JS file

Describe your changes (can list changeset entries if preferable)

Extract the script into its own file
Write tests for the script (!)
Update the comment to include guidance to update all packages

There's a reasonably high chance of this breaking when we extract it, but I followed these docs for the github-script action: https://github.com/actions/github-script/tree/v6/?tab=readme-ov-file#run-a-separate-file
Author Checklist (complete before requesting a review)

[ ] I have performed a self-review of my code
[ ] If it is a core feature, I have added thorough tests
[ ] If it is a PIE Docs change, I have reviewed the Docs site preview
[ ] If it is a component change, I have reviewed the Storybook preview
[ ] If there are visual test updates, I have reviewed them properly before approving

Reviewer checklists (complete before approving)
Reviewer 1

[ ] If it is a PIE Docs change, I have reviewed the PR preview
[ ] If there are visual test updates, I have reviewed them

Reviewer 2

[ ] If it is a PIE Docs change, I have reviewed the PR preview
[ ] If there are visual test updates, I have reviewed them

/snapit

/snapit test with sample button changes

@xander-marjoram this work is fantastic, I am temporarily requesting a revert just because there is some issue with snapit not being able to locate the new JS file when people attempt to run it.
I want us to keep this change, we might just need to do a bit more testing in the new year (happy to help with this).
@ashleynolan please revert when ready 👍

For reference – this is the revert. @xander-marjoram feel free to create a new PR with you branch when you're back in the new year so we can get it back in.


Method for setting CSRF token on client

Laravel Precognition Plugin Version
0.5.2
Laravel Version
10.35.0
Plugin
Alpine
Description
We have a setup where the CSRF token or cookie may not be available. (We're serving statically cached html).
We need a way to set the X-CSRF-Token header somehow.
I see that there's a way to set the url argument using a function. I think if we could set the config argument with a function, that'll probably work too.
We can grab the token via an ajax request to a route that just returns it.
Route::get('token', fn () => csrf_token());

const getConfig = function () {
  return { headers: { 'X-CSRF-TOKEN': getTokenViaAjax() } };
}

// ...

$form('post', '/the-url', {}, getConfig);

Or maybe a way of modifying the client after the fact.
document.addEventListener('got-csrf-token', () => {
  $form.setHeaders(...);
});

Or maybe there's already a way to do this. Thanks!
Steps To Reproduce

Create a route outside of your web routes, so that there is no session.
Output {{ csrf_token() }} to the page, see that its blank.
Create a form with precognition. The requests will give 419 responses.

Someone found a solution for us that seems to work.
In the custom event I mentioned, we would update the token on an html element.
Then we can provide an object with a toString method to the header, which will grab the token from the element.
$form('post', '/the-url', {}, {
  headers: {
    'X-CSRF-Token': {
      toString: () => document.getElementById('the-element').value
    }
  }
});

@jasonvarga sorry I didn't see your issue here.
Sounds like you have it solved, but for anyone seeing this I would also recommend doing this globally somewhere via the Axios client configuration...
import { client } from 'laravel-precognition-{package_flavour}';

getCsrfToken().then(csrfToken => {
    client.axios().defaults.headers.common['X-CSRF-TOKEN'] = csrfToken;
})

Thanks Tim.
In our case, we are injecting some JS into the html response via a package. We don't have access to their JS build process so we can't import { client } but this is good to know anyway!

Gotcha! Another option which might feel a little cleaner is to set the _token input on the form, either by injecting a hidden HTML input into the form like @csrf does or adding it in the form data. This would require the the-element to be available on load.
$form('post', '/the-url', {
    _token: document.getElementById('the-element').value,
});

Either way, a few options depending on what you need.


MAINT: Final fixes for upcoming v0.97.0

Hi @AntoineSIMTEK - sorry, I had to fix two more bugs, can you look at this again?


How can I get the path considering the agent size?

Hi.  Navmeshes built such that they take into account the agent radius and agents navigate as a point mass.  When you build a navmesh you should set walkableRadius in your rcConfig instance to the radius of the agent.


Homebrew Bundle always starts services

Description
My global Brewfile includes some formulae with services. Initially I had the services set up to restart when changed, like this:
tap "homebrew/bundle"
tap "homebrew/services"
brew "black", restart_service: :changed
brew "postgresql@16", restart_service: :changed

I rarely use the services, so I don't need them running continuously. I updated the Brewfile to include restart_service: false, start_service: false like this:
tap "homebrew/bundle"
tap "homebrew/services"
brew "black", restart_service: false, start_service: false
brew "postgresql@16", restart_service: false, start_service: false

When I run brew services stop --all, I can see that the services stop, but then they get restarted on the next brew bundle install --global.
When I uninstall the formulae with brew uninstall, then run brew bundle install --global, the services are installed and started again.
I tried deleting .Brewfile.lock.json but this had no effect.
Example terminal session (expand)
~ on ☁  (us-east-2)
❯ cat ~/.Brewfile
cask_args appdir: "/Applications", require_sha: true
tap "homebrew/bundle"
tap "homebrew/services"

brew "awscli"
brew "bash"
brew "black", restart_service: false, start_service: false
brew "ffmpeg"
brew "findutils"
brew "flake8"
brew "gh"
brew "git"
brew "git-lfs"
brew "gnupg"
brew "gnu-sed"
brew "gnu-tar"
brew "go"
brew "grep"
# brew "helm"
brew "httpie"
brew "jq"
brew "jupyterlab"
brew "libfido2"
brew "macos-trash" if OS.mac?
brew "mas" if OS.mac?
brew "media-info"
brew "mkvtoolnix"
brew "mp4v2"
brew "mypy"
# brew "mysql-client"
brew "node"
brew "openssh"
brew "opentofu"
brew "pinentry"
# brew "pinentry-mac"
brew "pipx"
brew "pnpm"
brew "postgresql@16", restart_service: false, start_service: false
brew "pre-commit"
brew "prettier"
brew "python"
brew "r"
brew "rclone"
brew "rust"
brew "shellcheck"
brew "shfmt"
brew "starship"
brew "taplo"
brew<EMAIL_ADDRESS>brew "terragrunt"
# brew "terraformer"
brew "typescript"
brew "wrangler"
brew "ykman"
brew "zsh"
brew "zsh-completions"
brew "zsh-syntax-highlighting"

cask "1password"
cask "1password-cli"
cask "brave-browser"
# cask "datagrip"
# cask "docker"
cask "firefox"
# cask "inkscape"
cask "karabiner-elements"
cask "kitty"
# cask "notion"
cask "orbstack"
# cask "postico"
# cask "postman"
cask "protonvpn"
# cask "rstudio"
# cask "session-manager-plugin"
# cask "slack"
cask "syncthing"
# cask "visual-studio-code"
cask "vlc"
cask "vscodium"
# cask "zoom"
cask "zotero" # no arm64_big_sur yet

# mas "1password", id:<PHONE_NUMBER>
mas "1password for safari", id:<PHONE_NUMBER>
mas "bear", id:<PHONE_NUMBER>
mas "daisydisk", id: 411643860
mas "duckduckgo privacy essentials", id:<PHONE_NUMBER>
mas "keynote", id: 409183694
mas "numbers", id: 409203825
mas "pages", id: 409201541
mas "pdf expert", id:<PHONE_NUMBER>

~ on ☁  (us-east-2)
❯ brew services list
Name          Status  User    File
black         started brendon ~/Library/LaunchAgents/homebrew.mxcl.black.plist
dbus          none
postgresql@16 started brendon<EMAIL_ADDRESS>unbound       none

~ on ☁  (us-east-2)
❯ brew services stop --all
Stopping `black`... (might take a while)
==> Successfully stopped `black` (label: homebrew.mxcl.black)
Warning: Service `dbus` is not started.
Stopping<EMAIL_ADDRESS>(might take a while)
==> Successfully stopped `postgresql@16` (label: homebrew.mxcl.postgresql@16)
Warning: Service `unbound` is not started.

~ on ☁  (us-east-2)
❯ brew services list
Name          Status User File
black         none
dbus          none
postgresql@16 none
unbound       none

~ on ☁  (us-east-2)
❯ brew bundle check --global
==> Downloading https://formulae.brew.sh/api/cask.jws.json
############################################################################################ 100.0%
==> Downloading https://formulae.brew.sh/api/formula.jws.json
############################################################################################ 100.0%
brew bundle can't satisfy your Brewfile's dependencies.
Satisfy missing dependencies with `brew bundle install`.

~ on ☁  (us-east-2) took 2s
❯ brew bundle install --global
Using homebrew/bundle
Using homebrew/services
==> Downloading https://formulae.brew.sh/api/formula.jws.json

Using awscli
Using bash
Using black
Using ffmpeg
Using findutils
Using flake8
Using gh
Using git
Using git-lfs
Using gnupg
Using gnu-sed
Using gnu-tar
Using go
Using grep
Using httpie
Using jq
Using jupyterlab
Using libfido2
Using macos-trash
Using mas
Using media-info
Using mkvtoolnix
Using mp4v2
Using mypy
Using node
Using openssh
Using opentofu
Using pinentry
Using pipx
Using pnpm
Using postgresql@16
Using pre-commit
Using prettier
Using python
Using r
Using rclone
Using rust
Using shellcheck
Using shfmt
Using starship
Using taplo
Using<EMAIL_ADDRESS>Using terragrunt
Using typescript
Using wrangler
Using ykman
Using zsh
Using zsh-completions
Using zsh-syntax-highlighting
==> Downloading https://formulae.brew.sh/api/cask.jws.json

Using 1password
Using 1password-cli
Using brave-browser
Using firefox
Using karabiner-elements
Using kitty
Using orbstack
Using protonvpn
Using syncthing
Using vlc
Using vscodium
Using zotero
Using 1password for safari
Using bear
Using daisydisk
Using duckduckgo privacy essentials
Using keynote
Using numbers
Using pages
Using pdf expert
Homebrew Bundle complete! 71 Brewfile dependencies now installed.

~ on ☁  (us-east-2) took 4s
❯ brew services list
Name          Status  User    File
black         started brendon ~/Library/LaunchAgents/homebrew.mxcl.black.plist
dbus          none
postgresql@16 started brendon<EMAIL_ADDRESS>unbound       none

~ on ☁  (us-east-2) took 4s
❯ brew uninstall jupyterlab python-lsp-server black postgresql@16
Warning: Treating jupyterlab as a formula. For the cask, use homebrew/cask/jupyterlab
Uninstalling /opt/homebrew/Cellar/jupyterlab/4.0.10... (4,650 files, 91.9MB)
Uninstalling /opt/homebrew/Cellar/python-lsp-server/1.9.0_1... (2,363 files, 7.5MB)
Uninstalling /opt/homebrew/Cellar/black/23.12.1... (301 files, 4.0MB)
Uninstalling<EMAIL_ADDRESS>(3,796 files, 68.3MB)

~ on ☁  (us-east-2) took 2s
❯ brew services list
Name    Status User File
dbus    none
unbound none

~ on ☁  (us-east-2)
❯ brew bundle install --global
Using homebrew/bundle
Using homebrew/services
Using awscli
Using bash
Installing black
Using ffmpeg
Using findutils
Using flake8
Using gh
Using git
Using git-lfs
Using gnupg
Using gnu-sed
Using gnu-tar
Using go
Using grep
Using httpie
Using jq
Installing jupyterlab
Using libfido2
Using macos-trash
Using mas
Using media-info
Using mkvtoolnix
Using mp4v2
Using mypy
Using node
Using openssh
Using opentofu
Using pinentry
Using pipx
Using pnpm
Installing postgresql@16
Using pre-commit
Using prettier
Using python
Using r
Using rclone
Using rust
Using shellcheck
Using shfmt
Using starship
Using taplo
Using<EMAIL_ADDRESS>Using terragrunt
Using typescript
Using wrangler
Using ykman
Using zsh
Using zsh-completions
Using zsh-syntax-highlighting
Using 1password
Using 1password-cli
Using brave-browser
Using firefox
Using karabiner-elements
Using kitty
Using orbstack
Using protonvpn
Using syncthing
Using vlc
Using vscodium
Using zotero
Using 1password for safari
Using bear
Using daisydisk
Using duckduckgo privacy essentials
Using keynote
Using numbers
Using pages
Using pdf expert
Homebrew Bundle complete! 71 Brewfile dependencies now installed.

~ on ☁  (us-east-2) took 19s
❯ brew services list
Name          Status  User    File
black         started brendon ~/Library/LaunchAgents/homebrew.mxcl.black.plist
dbus          none
postgresql@16 started brendon<EMAIL_ADDRESS>unbound       none

Output from recommended Homebrew troubleshooting steps (expand)

~ on ☁  (us-east-2)
❯ brew update
Already up-to-date.

~ on ☁  (us-east-2)
❯ brew update
Already up-to-date.

~ on ☁  (us-east-2)
❯ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed formulae are deprecated or disabled.
You should find replacements for the following formulae:
<EMAIL_ADDRESS>
Warning: Putting non-prefixed findutils in your path can cause python builds to fail.

~ on ☁  (us-east-2) took 4s
❯ brew config
HOMEBREW_VERSION: 4.2.2
ORIGIN: https://github.com/Homebrew/brew
HEAD: 705d2564ddfb9c915ee9425dd9ed408bfac61005
Last commit: 28 hours ago
Core tap JSON: 02 Jan 21:23 UTC
Core cask tap JSON: 02 Jan 21:23 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_EDITOR: codium --wait
HOMEBREW_MAKE_JOBS: 8
HOMEBREW_NO_ANALYTICS: set
Homebrew Ruby: 3.1.4 => /opt/homebrew/Library/Homebrew/vendor/portable-ruby/3.1.4/bin/ruby
CPU: octa-core 64-bit arm_firestorm_icestorm
Clang: 15.0.0 build 1500
Git: 2.43.0 => /opt/homebrew/bin/git
Curl: 8.4.0 => /usr/bin/curl
macOS: 14.2.1-arm64
CLT: <IP_ADDRESS>.1.1700200546
Xcode: N/A
Rosetta 2: false

Suggestions
Expected behavior:

brew bundle install would not start services if restart_service: false, start_service: false
brew bundle check would take restart_service: false, start_service: false into account

Related

https://formulae.brew.sh/formula/black
https://formulae.brew.sh/formula/postgresql@16
https://github.com/Homebrew/homebrew-services
https://github.com/Homebrew/homebrew-bundle/issues/188

Can you try omitting restart_service and start_service entirely? I suspect we don't handle false correctly here.

Can you try omitting restart_service and start_service entirely? I suspect we don't handle false correctly here.

Thank you for the suggestion. Indeed, when I omit restart_service and start_service entirely, like this:
tap "homebrew/bundle"
tap "homebrew/services"
brew "black"
brew "postgresql@16"

I then run brew bundle install --global, and see that the services do not start and that brew bundle check --global passes.
~
❯ brew services list
Name          Status User File
black         none
dbus          none
postgresql@16 none
unbound       none

~
❯ brew bundle check --global
The Brewfile's dependencies are satisfied.

I'm happy to help resolve the issue if you can point me in the general direction. I'm not much of a Rubyist but can probably figure it out.

@br3ndonland Thanks for reporting back, happy to help!


Unable get position body and imu information through Raspberry Pi

Hi @julianoes ,
I am working on a algorithm that will need the body position from odometry under telemetry. When I test my code with Gazebo, function odometry works just fine. But when I try to test my code on board with Raspberry Pi connected with the PixHawk, I couldn't get any information of the body position and my code just stuck at the function waiting to get the body position. The drone system is properly connected because I can run takeoff mission through my code.
I'm wondering that does odometry only works under virtual machine like Gazebo, or do I miss checking any needed information to get the body position.
Since I can't get the body position from odometry, I tried to use imu's acceleration to calculate displacement instead of body position and the same problem occurs.
This is the testing code I rewrite from the example code telemetry.py, and I put it on Raspberry Pi to test but I can't get the information I wanted.

Odometry is only sent out by a vehicle that actually estimates odometry. I think that's only the case if you have some sort of external vision or similar hooked up to PX4, so that the EKF2 can estimate odometry.
What information do you actually require? Global position (latitude/longitude from GPS) or local position (meters from estimator initialization)?

First of all, thank you so much for your immediate reply.
Do you mean if I need the information (local position) from odometry, I'll have to connect something that estimates odometry with my Pixhawk? I thought at the position where I takeoff, odometry will initial the position coordinate (x, y) as (0, 0), and calculate displacement with imu or something else in PixHawk.
At my current situation that I simulated on Gazebo, I need odometry's position_body (x_m, y_m, z_m) to initialize the local position at where I takeoff. Furthermore, every time I move my drone to a new position, I use position_body to get the new local position and calculate displacement with the previous local position. I use local position to calculate the displacement instaed of velocity because there will be interference from external factors no matter on Gazebo or in reality.
In addition to the previously mentioned issue that I couldn't get the local position, I found out if I takeoff at coordinate point (8, 0) in Gazebo, the initial local position I get will also be (8, 0). Does it mean if I use it on board I'll have to give a initial coordinate point since there's no virtual coordinate in reality? Or does it mean if I use class odometry properly, the initial local position will be (0, 0)?

Please take a step back and describe what you're actually want to do. I think the whole odometry topic is just a bit of a confusion/distraction here. It's something that is available in SITL but not something that you can use easily overall.
Are you flying indoor or outdoors (with GPS)?

Currently I'm flying outdoors with GPS,  and I'm trying to record the drone's local position every sampling time including the local position where it takeoff.

Aha, you should use: position_velocity_ned instead.

I tried it and it works, thanks a lot.
By the way, I found out that it seems to be calculating the local position with the GPS signal. Is there any function that does the same thing under GPS-denied situation?


Reporting errors in-app does not work if the SSL certificate is bad

Keybase GUI Version: 6.2.4-20231019211625+5cfcf6b41e
I'm getting this:
Error: ERROR CODE 1601 - API network error: Post "https://api-0.core.keybaseapi.com/_/api/1.0/logdump/send.json": x509: certificate signed by unknown authority in method keybase.1.config.logSend at new RPCError (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/main.bundle.js:1:4125048) at convertToRPCError (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/main.bundle.js:1:4102243) at convertToError (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/main.bundle.js:1:4102334) at file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/main.bundle.js:1:2473166 at file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/105.bundle.js:2:215977 at Deferrals._call (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/105.bundle.js:2:894539) at file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/105.bundle.js:2:894731 at trampoline (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/105.bundle.js:2:894267) at Deferrals._fulfill (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/105.bundle.js:2:894684) at ret (file:///Applications/Keybase.app/Contents/Resources/app/desktop/dist/105.bundle.js:2:893677)
when it tries to send feedback in-app.  The error looks transient - that is, when you fix the certificate, it will be a non-issue.  But a feedback mechanism should not "fail with a stack trace"

Hi,
it was an issue with the SSL certificat being expire or unvalidate.
That issue has been corrected on the last update.

Yes, the certificate may be repaired but the exception path through the app still exists in a "sub-optimal" form. When someone wants to report a problem, the app needs to work - falling back on a mailto: link would be an improvement.

certificate is valid , but it doesn't not have the full chain , so it throws error for the unknown Certificate Authority . Uploading a file ,opening the teams or reporting an issue gives and error.

Again, I feel like the point is being missed.  The issue is that the "report a problem" feature threw a problem of its own, rather than falling back on catching the error cleanly and saying something like "I really can't cope with this - please copy the following and send an email to<EMAIL_ADDRESS>Yes, the certificate is bad and will be better at some point.  That's not the issue.
There was no catch statement to intercept that stack trace display, and that looks catastrophically bad to someone who wants to trust the software not to "lose their stuff" even when things go wrong.
Exception paths in code are HARD to make user friendly, and I accept that there will be lots of paths through the regular app that can fail for all sorts of unimaginable reasons.  But the REPORT A PROBLEM function needs to be, or at least appear to be, a lot more robust.

None of you mentioned the platform (OS). It's an important detail. After contributing to the Windows package update that was recently merged, I just installed the new version on Win11 and everything seems to work fine. I could reproduce only the Feedback issue, which indeed throws:
API_NETWORK_ERROR: ERROR CODE 1601 - API network error: Post "https://api-0.core.keybaseapi.com/_/api/1.0/logdump/send.json": EOF in method keybase.1.config.logSend

Sorry, I figured that seeing the traceback is out of /Applications/Keybase.app told everyone this is MacOS.
And, correct me if I'm wrong, but you seem to be saying you see the same issue on Windows, which will be no surprise.
To be honest, I think this is one of those "why are you bothering to use HTTPS on a feedback form?" issues. If the modern internet fetish for "encrypt everything, just in case" wasn't there, this wouldn't be an issue.  So, as it stands, I think a mechanism that tried https, then fell back on plain http would have completely avoided this problem.
AGAIN, this is for the Bug Feedback data path, and it could even warn the user that it was doing so.

Sorry, I figured that seeing the traceback is out of /Applications/Keybase.app told everyone this is MacOS. And, correct me if I'm wrong, but you seem to be saying you see the same issue on Windows, which will be no surprise.
To be honest, I think this is one of those "why are you bothering to use HTTPS on a feedback form?" issues. If the modern internet fetish for "encrypt everything, just in case" wasn't there, this wouldn't be an issue. So, as it stands, I think a mechanism that tried https, then fell back on plain http would have completely avoided this problem.
AGAIN, this is for the Bug Feedback data path, and it could even warn the user that it was doing so.

You were right about that. I was weighting on all inputs in this thread at the time, so I've lost sight of the obvious macOS path. I still think it's a good idea to be thorough when reporting an[y] issue.


sometimes doesnt pick up aliases

OS version and name: macOS 10.12.4 (16E144f) Sierra
Hyper.app version: 1.1
Link of a Gist with the contents of your .hyper.js: https://github.com/mcansh/dotfiles/blob/master/home/.hyper.js

Issue

Sometimes when using hyper, either in a new tab or next command after using an alias, it will say its not found, not sure if its oh-my-zsh or hyper

hmm, appears my .zshrc was misconfigured,


Call mypy with newly added parameter flush_errors

Unfortunately the target function does not have any docstring in mypy,
but it is typed as Optional[] which means the function should accept
None as a value. Doing so seems to suppress issues reported in #6.
Quick skim of the culprit PR in mypy
(https://github.com/python/mypy/pull/4396/files) shows the code really
counts with None being passed and adjusting appropriately, citing:

If we were not given a flush_errors, we use one that will populate
those fields for callers that want the traditional API.

It's only the Python 3.3 that failed. Given that mypy itself now needs at least Python 3.4 to run, it seems that can be safely removed actually from Travis setup and other places. I will submit a PR for that.

Please please please can we get this merged and released ASAP?

:+1:

LGTM, thanks guys. I'm so swamped right now with other stuff—anybody interested in becoming a collaborator on this repo and helping out with maintaining this thing?


lazyloading both Bgset with data-srcset

Both will work,
but the lazyload class will change to lazyloaded
whenever bgset or data-srcset first finish loading.
Ideal behavior is class change will wait unitl BOTH bgset  & datasrcset finish lazyloading.
Any other alternative to do this?

I don't understand. Please provide a test case.


Interoperability Issues: How does the Thread SRP Client ensure that the expected AAAA Record is obtained

According to commit fa3563a , I want to bring up an interoperability issue. How about Thread SRP server do include AAAA address in additional records but only link local address from query response. Not sure this condition ( https://github.com/project-chip/connectedhomeip/blob/master/src/platform/OpenThread/GenericThreadStackManagerImpl_OpenThread.cpp#L2852 )is enough.
More Details: https://github.com/openthread/openthread/discussions/8967
This commit fa3563a work fine with apple ecosystem Homepod mini (for additional records does not carry AAAA address).
Update my test result with Nest Hub Gen2.
1.Thread Accessory(SRP Client) Query Request
Queries
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN

2.Nest Hub(SRP Server)  Query Response with LLA
Queries
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN
Answers
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN, priority 0, weight 0, port 5540, target DCA632F3954A0000.default.service.arpa
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN
Additional records
    DCA632F3954A0000.default.service.arpa: type AAAA, class IN, addr fe80::dea6:32ff:fef3:954a
        Name: DCA632F3954A0000.default.service.arpa
        Type: AAAA (IPv6 Address) (28)
        Class: IN (0x0001)
        Time to live: 10 (10 seconds)
        Data length: 16
        AAAA Address: fe80::dea6:32ff:fef3:954a

3.Let Thread Accessory Query Request for AAAA
Queries
    DCA632F3954A0000.default.service.arpa: type AAAA, class IN
        Name: DCA632F3954A0000.default.service.arpa
        [Name Length: 37]
        [Label Count: 4]
        Type: AAAA (IPv6 Address) (28)
        Class: IN (0x0001)

Nest Hub(SRP Server)  Query Response with ULA

Queries
    DCA632F3954A0000.default.service.arpa: type AAAA, class IN
Answers
    DCA632F3954A0000.default.service.arpa: type AAAA, class IN, addr fddc:3740:f4b5:4ff4:dea6:32ff:fef3:954a
        Name: DCA632F3954A0000.default.service.arpa
        Type: AAAA (IPv6 Address) (28)
        Class: IN (0x0001)
        Time to live: 10 (10 seconds)
        Data length: 16
        AAAA Address: fddc:3740:f4b5:4ff4:dea6:32ff:fef3:954a

5.Network is Reachable now
So currently there are maybe some options:

Add some requirements and mention in Thread Spec for SRP Server, I believe this needs to be mandatory, otherwise there will be interoperability issues. At least for Matter over Thread devices using SRP to  register their services it should not be a case, because they're assigning only "external" IPv6 addresses to the SRP host: https://github.com/project-chip/connectedhomeip/blob/master/src/platform/OpenThread/GenericThreadStackManagerImpl_OpenThread.cpp#L2447 External then propagates to the GUA addresses only: https://github.com/project-chip/connectedhomeip/blob/master/src/platform/OpenThread/GenericThreadStackManagerImpl_OpenThread.cpp#L1006
But even if for some reason the device would receive only link local IP address what should we do? Using this IP address directly will result in not being able to reach the destination and querying for AAAA record again may end with returning exactly the same address.
Query again? Or whether it should be informed to the application layer to allow the device maker to make a decision.

Bug prevalence
100%
GitHub hash of the SDK that was being used
e62d3ddaeeaed47535060f9d542ef87c655fef97
Platform
core
Platform Version(s)
1.0
Anything else?
No response

It's not clear what you're asking for here.

@Abhayakara According the logic from commit fa3563a，srp client will not further query about AAAA record AAAA if the previous query response contains AAAA record.
So the question is about if the AAAA address obtained in the previous query response was only the link local address. But only get the link local address(outside of thread network) will cause the network to be unreachable, right?
Network Topology:
raspberry pi (11DA090384C9CFB4-0000000000000001._matter._tcp.local)<------> WiFi Router <------> nest hub gen 2 <------> matter over thread accessory

If we follow the logic of this commit fa3563a，the result would be as following.
a.Thread Accessory(SRP Client) Query Request
Queries
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN

b.Nest Hub(SRP Server) Query Response with LLA
Queries
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN
Answers
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN, priority 0, weight 0, port 5540, target DCA632F3954A0000.default.service.arpa
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN
Additional records
    DCA632F3954A0000.default.service.arpa: type AAAA, class IN, addr fe80::dea6:32ff:fef3:954a
        Name: DCA632F3954A0000.default.service.arpa
        Type: AAAA (IPv6 Address) (28)
        Class: IN (0x0001)
        Time to live: 10 (10 seconds)
        Data length: 16
        AAAA Address: fe80::dea6:32ff:fef3:954a

c. Network is unreachable
So I added a logic that if the received address is a LLA, then send additional DNS query to obtain IPv6 address for ULA. Here is my test result.
a.Thread Accessory(SRP Client) Query Request
Queries
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN

b.Nest Hub(SRP Server) Query Response with LLA
Queries
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN
Answers
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN, priority 0, weight 0, port 5540, target DCA632F3954A0000.default.service.arpa
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN
Additional records
    DCA632F3954A0000.default.service.arpa: type AAAA, class IN, addr fe80::dea6:32ff:fef3:954a
        Name: DCA632F3954A0000.default.service.arpa
        Type: AAAA (IPv6 Address) (28)
        Class: IN (0x0001)
        Time to live: 10 (10 seconds)
        Data length: 16
        AAAA Address: fe80::dea6:32ff:fef3:954a

c.Let Thread Accessory Query Request for AAAA
Queries
    DCA632F3954A0000.default.service.arpa: type AAAA, class IN
        Name: DCA632F3954A0000.default.service.arpa
        [Name Length: 37]
        [Label Count: 4]
        Type: AAAA (IPv6 Address) (28)
        Class: IN (0x0001)

d.Nest Hub(SRP Server) Query Response with ULA
Queries
    DCA632F3954A0000.default.service.arpa: type AAAA, class IN
Answers
    DCA632F3954A0000.default.service.arpa: type AAAA, class IN, addr fddc:3740:f4b5:4ff4:dea6:32ff:fef3:954a
        Name: DCA632F3954A0000.default.service.arpa
        Type: AAAA (IPv6 Address) (28)
        Class: IN (0x0001)
        Time to live: 10 (10 seconds)
        Data length: 16
        AAAA Address: fddc:3740:f4b5:4ff4:dea6:32ff:fef3:954a

e.Network is Reachable now
So there are some issues left over here:

If for some reason the device would receive only link local address what should we do? Query again？But maybe still return the same address only LLA.
What is the best practice? Anyway, we need to consider this situation and solve this issue. Otherwise, I believe this will affect the related function such as ICD, Binding, etc

It’s an error for you to get a link local address that doesn’t work. How did this happen?

raspberry pi (11DA090384C9CFB4-0000000000000001._matter._tcp.local)<------> WiFi Router <------> nest hub gen 2 <------> matter over thread accessory

raspberry pi info:
[11DA090384C9CFB4-0000000000000001._matter._tcp.local]
[fe80::dea6:32ff:fef3:954a]
[fddc:3740:f4b5:4ff4:dea6:32ff:fef3:954a]

As shown in the network topology diagram, A Matter Thread Device attempts to resolve 11DA090384C9CFB4-0000000000000001._matter._tcp.local and send out dns query for SRV & TXT.
Matter Thread Device --------> Nest Hub Gen2 (Sniffer Log No.32117)
Queries
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN

Nest hub as the SRP server to answer this query include the Additional records but only have LLA. So that's why the device received the LLA.
Matter Thread Device <-------- Nest Hub Gen2  (Sniffer Log No.32127)
Queries
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN
Answers
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN, priority 0, weight 0, port 5540, target DCA632F3954A0000.default.service.arpa
    11DA090384C9CFB4-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN
Additional records
    DCA632F3954A0000.default.service.arpa: type AAAA, class IN, addr fe80::dea6:32ff:fef3:954a
        Name: DCA632F3954A0000.default.service.arpa
        Type: AAAA (IPv6 Address) (28)
        Class: IN (0x0001)
        Time to live: 10 (10 seconds)
        Data length: 16
        AAAA Address: fe80::dea6:32ff:fef3:954a

Thread Info:
PANID: 0x703D
Thread Network Key: A4D6078BDD1929443E9246CE7BC16C40
Thread Sniffer log

Ah. So the bug is in the Google discovery proxy. It should not be returning link-local addresses from other links. It should be returning all AAAA records in the additional section though. Is it not?

Yes. This is what I observed it.  It does not return all address information on the additional records. I am checking the thread spec and certification test plan, but I have not found the relevant requirements and test cases to ensure that the problem I encountered does not occur.
In addition, I also found another issue on smartthings hub v3. It does not contain answer information in the query response.
Frame 1262: 91 bytes on wire (728 bits), 91 bytes captured (728 bits) on interface \Device\NPF_Loopback, id 0
Null/Loopback
Internet Protocol Version 4, Src: <IP_ADDRESS>, Dst: <IP_ADDRESS>
User Datagram Protocol, Src Port: 57277, Dst Port: 49999
NXP 802.15.4 Sniffer Protocol
IEEE 802.15.4 Data, Dst: 0x0000, Src: 0xc000
6LoWPAN
Internet Protocol Version 6, Src: fd0b:7831:4c78:0:4d56:b953:cc06:92bd, Dst: fd0b:7831:4c78:0:a326:a042:e2a4:1eb1
User Datagram Protocol, Src Port: 53, Dst Port: 49154
Domain Name System (response)
    Transaction ID: 0x6236
    Flags: 0x8000 Standard query response, No error
    Questions: 2
    Answer RRs: 0
    Authority RRs: 0
    Additional RRs: 0
    Queries
        21C4E71E8CA1BE12-0000000000000001._matter._tcp.default.service.arpa: type SRV, class IN
        21C4E71E8CA1BE12-0000000000000001._matter._tcp.default.service.arpa: type TXT, class IN
    [Retransmitted response. Original response in: 1143]
    [Retransmission: True]

Maybe I missed some important information, but how can we ensure the Interoperability at Thread.
@jwhui @Abhayakara

@Abhayakara Let me summarize all of the above. Here is my experience in different ecosystems.
Apple(with homepod mini):
When a Thread device queries for an SRV+TXT Record, AAAA record is not included in the query response. So it need to do the additional AAAA query. Commit fa3563a is used to implement this mechanism.
Google(with Nest Hub Gen2):
When a Thread device queries for an SRV+TXT Record, AAAA record is included in the query response. But this AAAA record from query response is link local address. Please check the sniffer Log No.32127. However, if do the additional AAAA query, it will return the different AAAA record which is Unique Local Address. Please check the sniffer Log No.32161.
Thread Info:
PANID: 0x703D
Thread Network Key: A4D6078BDD1929443E9246CE7BC16C40
Thread Sniffer log
SmartThings(with SmartThings Hub v3):
When a Thread device queries for an SRV+TXT Record, there is no SRV or TXT record in the query response. That is, there is no answer information in the the query response. Please check the sniffer log No.1262.
SmartThings:
Channel:14
panid: 0x65aa
networkkey:7f200b0a0f236afb5818b2e302172909
smartthing_srp_issue.zip
Alexa(with Echo Gen4):
When a Thread device queries for an SRV+TXT Record, there is no SRV or TXT record in the query response. That is, there is no answer information in the the query response. Please check the sniffer log No.1714.
SmartThings:
Channel:11
PANID: 1713
NetworkKey:5392c9b34603a739554cda30592a4665
alexa-srp-issue-1714.zip

Okay, in the last case this is because it's invalid to have two questions in a DNS lookup. The Google implementation and the Apple implementation both respond to both questions, but they aren't required to—the behavior in this case is undefined. So that's why the Echo and the SmartThings hubs are responding as they are. Although they should be returning an rcode of FormErr if they aren't going to handle the two-question query. As it is, this response seems to be saying that there is no TXT or SRV record under that name, but that the name exists (which I suppose is also possible).
The behavior of the Nest hub just looks like a bug. It would be wrong to add code to work around it. Similarly, it would be wrong to add code to work around the Echo and SmartThings failures. In all three cases, you have an answer, but it's the wrong answer, and that's why it doesn't work. The right thing to do is for Google, Nest and SmartThings to fix their implementations.
Whether that's the expedient thing I can't say. I am somewhat regretting implementing the two-question query response at this point—my goal was to avoid a train wreck on release, but it looks like we haven't done that.

Thank you. Yes, I agree that we shouldn't add code to workaround that. I just wanted to see what would happen if I did additional AAAA records for this case.
Matter1.0 supports Binding, Multi-fabric, OTA.

Commission Matter Thread Accessory to Google/Apple/SmartThings/Alexa
Share this Accessory to our Vendor fabric
Want to trigger OTA or Binding in our Vendor fabric
At least for now, I did run into some issues and it is difficult to solve on the device side. However, commit fa3563a do help my use case with Apple Ecosystem.

I believe these issues will also block the Intermittently Connected Devices (ICDs) feautre.

I think we also need relevant test cases and certification requirements, especially for thread border routers, as there will be more and more thread border router products on the market.

Okay, in the last case this is because it's invalid to have two questions in a DNS lookup. The Google implementation and the Apple implementation both respond to both questions, but they aren't required to—the behavior in this case is undefined. So that's why the Echo and the SmartThings hubs are responding as they are. Although they should be returning an rcode of FormErr if they aren't going to handle the two-question query. As it is, this response seems to be saying that there is no TXT or SRV record under that name, but that the name exists (which I suppose is also possible).

OpenThread's current DNS Client implementation defaults to sending separate queries for SRV and TXT if the combined query fails. But it does currently depend on receiving an error from the server.


Update run.md to include OneNote plugin description

@palenshus : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mattwojo
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Test adapter loading strategy

Description
The code here is cherry-picked from our 17.1 release.

Fixes #2580

Work left

[x] Fix the bug where we fail to retrieve TestAdapterLoadingStrategy from .runsettings file.
[x] Fix typos in console help text.
[ ] Add documentation
[ ] Add tests

@Evangelink thanks for the review, I'll add the tests and ask for another review.


Move setting listenerType outside while loop

No need to be inside while loop, so this PR moves it outside.

Thanks; good catch.


styled-jsx-postcss example broken after in next 4

example of styled-jsx-postcss broken after commit 559c25253ea4a50fbc04ef37f466d9e9b616512a

[x ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
The custom babel-present.js in example can modify the next/babel
Current Behavior
after commit, next/babel became a function and cannot be modified
Steps to Reproduce (for bugs)

Use example in here next.js/examples/with-styled-jsx-postcss/
run dev

Context
We use styled-jsx-postcss, but since we want to update to the latest of next, it got broken
Your Environment

Tech
Version

next
4.0.1

node
v6.11.3

FYI. There is an styled-jsx update coming this week to address postcss issue for styled-jsx.
https://github.com/giuseppeg/styled-jsx-postcss/issues/24

Hi, I left a PR with my solution to this issue.Although I'm waiting the styled-jsx update, this code can help you meanwhile.
const nextBabelPreset = require('next/babel');

/*
  This code will load the 'styled-jsx-postcss/babel' package instead
  of built-in 'styled-jsx/babel'
*/

let presetArray = nextBabelPreset();

presetArray.plugins = presetArray.plugins.map(plugin => {
  if (!Array.isArray(plugin) && plugin.indexOf('styled-jsx/babel') !== -1) {
    return require.resolve('styled-jsx-postcss/babel');
  }
  return plugin;
});

module.exports = presetArray;

When you switch to NextJS v4, './dist/server/build/babel/preset' now returns a function which result is the plugins + presets array. So you can use an auxiliar to call that function and then do the plugin replacement :)
Hope it can help you.
https://github.com/zeit/next.js/pull/3083

Also the example using postcss as plugin is not working :https://github.com/zeit/next.js/tree/master/examples/with-styled-jsx-plugins
@MFCo according to the example, it should work without replace 'styled-jsx/babel' I suppose.

@younes200 yeah, that code that I left was working before 4.1.0, now you don't have to replace 'styled-jsx/babel'. I haven't tested 4.1.0 myself yet

Exactly, styled-jsx-postcss as npm module is deprecated now, since you can just use styled-jsx-plugin-postcss 👍


Bump service commons

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.297%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
609

Relevant Lines:
768

💛  - Coveralls


Target Directory

How come the target directory is not included for IntelliJ ?

The original template was copied from https://github.com/github/gitignore/blob/master/Global/JetBrains.gitignore.  The templates on https://github.com/dvcs/gitignore are all based on GitHub's original list, but GitHub's list lacks flexibility that gitignore.io users are asking for.  I'm working update to address issues like this.
The main side effect of my proposed updates means that gitignore.io will fork from GitHub's.  Until recently I wasn't comfortable with doing that as GitHub has been pretty responsive with updating their templates.  The fork would me that when GitHub does it's template updates, gitignore.io won't automatically benefit from them.

I'm going to close this task.  If you believe the IntelliJ template needs modification, you can submit a pull request on the @dvcs/gitignore repository.


feat: update coverage badge only on merge to main

closes #488
PR Type

CI related changes

Description
Use the approach of #480 to create a new PR to update the code coverage badge.
This ensures the badge on main is up-to-date. Also prevents excess bot commits.
How Has This Been Tested?
Not directly tested. Relying on minimal edits to code copied from elsewhere that has been tested as part of #435 and #480. Cannot test properly until merge into main.
Does this PR introduce a breaking change?
No
Checklist before requesting a review

[x] I have made sure that my PR is not a duplicate.
[x] My code follows the style guidelines of this project.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have performed a self-review of my code.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

@tc85324 I have reworked this using a Gist. The PAT should be added to the repo secrets tomorrow. Ready for rereview.

PAT added. This should now be working on every merge to main.


Add a license

Could you please add a license to the project so others can contribute and/or use the project. I would like to make some suggestions/changes, but without proper licensing I'm not able to contribute.

done. hope MIT works for you - should be pretty unencumbering.


Arabic body copy isn't right aligned

For whatever reason, I cannot get Arabic copy in the body section to right align. Open to anyone who speaks Arabic taking this issue!

@cereallarceny I've fixed this in my upcoming PR.

Great that you fixed it @s0l0ist , but until your PR has been approved, we're gonna leave this ticket open. @SETIADEEPANSHU will be responsible for reviewing that PR.  :)


Possible to change class name "demo-icon" before download?

Well, I am new to fontello, just like to know if this is an option.

prefix is tuneable via settings popup


[FEATURE] Personal details step for logged in user

Description

Blocked by: https://github.com/DivanteLtd/storefront-ui/issues/1025

Acceptance criteria

[ ] Implement solution after https://github.com/DivanteLtd/storefront-ui/issues/1025 is done

Implement this form as a new component

and use it in MyAccount as well for consistency


Add JobArtifact model and storeNewArtifact function in StorageProvider

Add JobArtifact model and storeNewArtifact function in StorageProvider interface
Context
This PR aims to define the model for Job artifacts we plan to store going forward. A new functionality has been added to MantisStorageProvider to be able to store new job artifacts (right now it only saves 1 artifact at the time, maybe in the future we can store more than one at the time).
Note: before merging I plan to change JobArtifact model a bit to have a list of dependencies where the runtime is only 1 of the dependencies; can be empty if the artifact is in the form of a uber jar.
Checklist

[ ] ./gradlew build compiles code correctly
[ ] Added new tests where applicable
[ ] ./gradlew test passes all tests
[ ] Extended README or added javadocs where applicable

LGTM. Can you track follow-up/TODOs in the PR description or issue also?


[Docs][AlertIOS] Prompt button examples failing in appetize.io sim

Just a heads up I'm seeing a red screen of death (n.forEach is not a function, or invariant violations) for just the prompt button section of the AlertIOS examples:
http://screencast.com/t/Zlvhxp3uBG
P.S. Awesome use of appetize.io!

/cc @jsierles

Sounds like it just works as of now -> https://facebook.github.io/react-native/docs/alertios.html#content.


Make DistributedSampler stateful

🚀 The feature
Currently RandomSampler, BatchSampler are patched here https://github.com/pytorch/data/blob/main/torchdata/stateful_dataloader/sampler.py#L134-L135 to make them stateful and work out of the box with StatefulDataLoader.
It would be useful to consider making DistributedSampler (https://github.com/pytorch/pytorch/blob/2176ef7dfaf02dd6dbb8484a50c99d5fadf3ea0b/torch/utils/data/distributed.py#L13) also implement stateful methods and patch it in torchdata.
Motivation, pitch
So that users can use DistributedSampler also out of the box with checkpointing capability
Alternatives
Users would have implement the stateful interface for DistributedSampler but extending it
Additional context
No response

This currently isn't broken right? ie fast-forwarding the sampler will work, but may be inefficient. I'm OK either way for before/after release branch cut

Hi @gokulavasan @andrewkho ,
I found that current StatefulDataloader works well with DistributedSampler without any modifications.
Would you mind please explaining why it might be inefficient?
Thanks in advance.

This currently isn't broken right? ie fast-forwarding the sampler will work, but may be inefficient. I'm OK either way for before/after release branch cut

Does fast-forwarding here mean that the sampler would iterate from the head to the checkpointing point? If so, is it inefficient?
An efficient way would be to jump directly to the checkpointing point, right?
Please correct me if my understanding is wrong. Thank you.

HI @ShoufaChen you're correct, it should work without modifications but may be slow for large tables. https://github.com/pytorch/data/blob/main/torchdata/stateful_dataloader/sampler.py#L47 Here is where we've done the conversion for RandomSampler and BatchSampler as examples.
You can see for example the default batch sampler calling next() to naively fast-forward the sampler.
Here's an example where you can see that increasing the samples to iterate through increases the time required to fast-forward, and when you get to very large scales (eg billion scale) this starts to slow down to order of minutes: https://colab.research.google.com/drive/1UlJAMqzaCjtbW4RPaaoHxGd9sjiKFk7O?usp=sharing


21/06/23 16:06:52 INFO scheduler.TaskSetManager: Finished task 20.0 in stage 0.0 (TID 29) in 582695 ms on gz-dt-172-16-11-56.ux168.cn (executor 18) (80/91)
21/06/23 16:06:52 INFO cluster.YarnSchedulerBackend$YarnDriverEndpoint: Disabling executor 8.
21/06/23 16:06:52 INFO scheduler.DAGScheduler: Executor lost: 8 (epoch 0)
21/06/23 16:06:52 INFO storage.BlockManagerMasterEndpoint: Trying to remove executor 8 from BlockManagerMaster.
21/06/23 16:06:52 INFO storage.BlockManagerMasterEndpoint: Removing block manager BlockManagerId(8, gz-dt-172-16-11-51.ux168.cn, 37247, None)
21/06/23 16:06:52 INFO storage.BlockManagerMaster: Removed 8 successfully in removeExecutor
21/06/23 16:06:52 INFO cluster.YarnScheduler: Executor 8 on gz-dt-172-16-11-51.ux168.cn killed by driver.
21/06/23 16:06:52 INFO spark.ExecutorAllocationManager: Existing executor 8 has been removed (new total is 15)
21/06/23 16:06:53 INFO scheduler.TaskSetManager: Finished task 32.0 in stage 0.0 (TID 20) in 584913 ms on gz-dt-172-16-11-50.ux168.cn (executor 12) (81/91)
21/06/23 16:06:56 INFO scheduler.TaskSetManager: Finished task 15.0 in stage 0.0 (TID 11) in 591485 ms on gz-dt-172-16-11-51.ux168.cn (executor 7) (82/91)
21/06/23 16:07:05 INFO scheduler.TaskSetManager: Finished task 2.0 in stage 0.0 (TID 18) in 597222 ms on gz-dt-172-16-11-58.ux168.cn (executor 21) (83/91)
21/06/23 16:07:07 INFO scheduler.TaskSetManager: Finished task 12.0 in stage 0.0 (TID 28) in 597928 ms on gz-dt-172-16-11-56.ux168.cn (executor 18) (84/91)
21/06/23 16:07:10 INFO scheduler.TaskSetManager: Finished task 35.0 in stage 0.0 (TID 73) in 577928 ms on gz-dt-172-16-11-56.ux168.cn (executor 19) (85/91)
21/06/23 16:07:12 INFO spark.ExecutorAllocationManager: Request to remove executorIds: 24
21/06/23 16:07:12 INFO cluster.YarnClientSchedulerBackend: Requesting to kill executor(s) 24
21/06/23 16:07:12 INFO cluster.YarnClientSchedulerBackend: Actual list of executor(s) to be killed is 24
21/06/23 16:07:12 INFO spark.ExecutorAllocationManager: Removing executor 24 because it has been idle for 60 seconds (new desired total will be 14)
21/06/23 16:07:14 INFO cluster.YarnSchedulerBackend$YarnDriverEndpoint: Disabling executor 24.
21/06/23 16:07:14 INFO scheduler.DAGScheduler: Executor lost: 24 (epoch 0)
21/06/23 16:07:14 INFO storage.BlockManagerMasterEndpoint: Trying to remove executor 24 from BlockManagerMaster.
21/06/23 16:07:14 INFO storage.BlockManagerMasterEndpoint: Removing block manager BlockManagerId(24, gz-dt-172-16-11-58.ux168.cn, 46209, None)
21/06/23 16:07:14 INFO storage.BlockManagerMaster: Removed 24 successfully in removeExecutor
21/06/23 16:07:14 INFO cluster.YarnScheduler: Executor 24 on gz-dt-172-16-11-58.ux168.cn killed by driver.
21/06/23 16:07:14 INFO spark.ExecutorAllocationManager: Existing executor 24 has been removed (new total is 14)
21/06/23 16:07:14 INFO scheduler.TaskSetManager: Finished task 28.0 in stage 0.0 (TID 14) in 606705 ms on gz-dt-172-16-11-51.ux168.cn (executor 9) (86/91)
21/06/23 16:07:17 INFO scheduler.TaskSetManager: Finished task 24.0 in stage 0.0 (TID 9) in 617005 ms on gz-dt-172-16-11-59.ux168.cn (executor 4) (87/91)
21/06/23 16:07:18 INFO scheduler.TaskSetManager: Finished task 25.0 in stage 0.0 (TID 16) in 609664 ms on gz-dt-172-16-11-50.ux168.cn (executor 10) (88/91)
21/06/23 16:07:21 INFO scheduler.TaskSetManager: Finished task 30.0 in stage 0.0 (TID 36) in 611281 ms on gz-dt-172-16-11-51.ux168.cn (executor 13) (89/91)
21/06/23 16:07:21 INFO spark.ExecutorAllocationManager: Request to remove executorIds: 5
21/06/23 16:07:21 INFO cluster.YarnClientSchedulerBackend: Requesting to kill executor(s) 5
21/06/23 16:07:21 INFO cluster.YarnClientSchedulerBackend: Actual list of executor(s) to be killed is 5
21/06/23 16:07:21 INFO spark.ExecutorAllocationManager: Removing executor 5 because it has been idle for 60 seconds (new desired total will be 13)
21/06/23 16:07:24 INFO spark.ExecutorAllocationManager: Request to remove executorIds: 1
21/06/23 16:07:24 INFO cluster.YarnClientSchedulerBackend: Requesting to kill executor(s) 1
21/06/23 16:07:24 INFO cluster.YarnClientSchedulerBackend: Actual list of executor(s) to be killed is 1
21/06/23 16:07:24 INFO spark.ExecutorAllocationManager: Removing executor 1 because it has been idle for 60 seconds (new desired total will be 12)
21/06/23 16:07:24 INFO scheduler.TaskSetManager: Finished task 40.0 in stage 0.0 (TID 75) in 592287 ms on gz-dt-172-16-11-50.ux168.cn (executor 11) (90/91)
21/06/23 16:07:25 INFO cluster.YarnSchedulerBackend$YarnDriverEndpoint: Disabling executor 5.
21/06/23 16:07:25 INFO scheduler.DAGScheduler: Executor lost: 5 (epoch 0)
21/06/23 16:07:25 INFO storage.BlockManagerMasterEndpoint: Trying to remove executor 5 from BlockManagerMaster.
21/06/23 16:07:25 INFO storage.BlockManagerMasterEndpoint: Removing block manager BlockManagerId(5, gz-dt-172-16-11-59.ux168.cn, 46772, None)
21/06/23 16:07:25 INFO storage.BlockManagerMaster: Removed 5 successfully in removeExecutor
21/06/23 16:07:25 INFO cluster.YarnScheduler: Executor 5 on gz-dt-172-16-11-59.ux168.cn killed by driver.
21/06/23 16:07:25 INFO spark.ExecutorAllocationManager: Existing executor 5 has been removed (new total is 13)
21/06/23 16:07:27 INFO cluster.YarnSchedulerBackend$YarnDriverEndpoint: Disabling executor 1.
21/06/23 16:07:27 INFO scheduler.DAGScheduler: Executor lost: 1 (epoch 0)
21/06/23 16:07:27 INFO storage.BlockManagerMasterEndpoint: Trying to remove executor 1 from BlockManagerMaster.
21/06/23 16:07:27 INFO storage.BlockManagerMasterEndpoint: Removing block manager BlockManagerId(1, gz-dt-172-16-11-61.ux168.cn, 41555, None)
21/06/23 16:07:27 INFO storage.BlockManagerMaster: Removed 1 successfully in removeExecutor
21/06/23 16:07:27 INFO cluster.YarnScheduler: Executor 1 on gz-dt-172-16-11-61.ux168.cn killed by driver.
21/06/23 16:07:27 INFO spark.ExecutorAllocationManager: Existing executor 1 has been removed (new total is 12)
21/06/23 16:07:27 INFO scheduler.TaskSetManager: Finished task 34.0 in stage 0.0 (TID 55) in 609751 ms on gz-dt-172-16-11-54.ux168.cn (executor 6) (91/91)
21/06/23 16:07:27 INFO cluster.YarnScheduler: Removed TaskSet 0.0, whose tasks have all completed, from pool
21/06/23 16:07:27 INFO scheduler.DAGScheduler: ResultStage 0 (foreachPartition at Clickhouse.scala:162) finished in 626.701 s
21/06/23 16:07:27 INFO scheduler.DAGScheduler: Job 0 finished: foreachPartition at Clickhouse.scala:162, took 626.747644 s
21/06/23 16:07:27 INFO server.AbstractConnector: Stopped Spark@4638ade9{HTTP/1.1,[http/1.1]}{<IP_ADDRESS>:4040}
21/06/23 16:07:27 INFO ui.SparkUI: Stopped Spark web UI at http://gz-dt-172-16-11-50.ux168.cn:4040
21/06/23 16:07:32 INFO cluster.YarnClientSchedulerBackend: Interrupting monitor thread
21/06/23 16:07:32 INFO cluster.YarnClientSchedulerBackend: Shutting down all executors
21/06/23 16:07:32 INFO cluster.YarnSchedulerBackend$YarnDriverEndpoint: Asking each executor to shut down
21/06/23 16:07:32 INFO cluster.SchedulerExtensionServices: Stopping SchedulerExtensionServices
(serviceOption=None,
 services=List(),
 started=false)
21/06/23 16:07:32 INFO cluster.YarnClientSchedulerBackend: Stopped
21/06/23 16:07:32 INFO spark.MapOutputTrackerMasterEndpoint: MapOutputTrackerMasterEndpoint stopped!
21/06/23 16:07:32 INFO memory.MemoryStore: MemoryStore cleared
21/06/23 16:07:32 INFO storage.BlockManager: BlockManager stopped
21/06/23 16:07:32 INFO storage.BlockManagerMaster: BlockManagerMaster stopped
21/06/23 16:07:32 INFO scheduler.OutputCommitCoordinator$OutputCommitCoordinatorEndpoint: OutputCommitCoordinator stopped!
21/06/23 16:07:32 INFO spark.SparkContext: Successfully stopped SparkContext
21/06/23 16:07:33 INFO util.ShutdownHookManager: Shutdown hook called
21/06/23 16:07:33 INFO util.ShutdownHookManager: Deleting directory /tmp/spark-dfc27973-6932-4c71-9f64-52b3b4674607
21/06/23 16:07:33 INFO util.ShutdownHookManager: Deleting directory /tmp/spark-3a9ecc6f-8555-4af5-b3e6-60624214c307

clickhouse 需要创建分布表


[Xbox Metadata] Merge plugins

Extension name
Xbox Metadata
Feature Description
Hey, I see you've released an Xbox Metadata plugin - I've released one previously, and there's no hard feelings here, just that here's now two extensions named Xbox Metadata, which is confusing for users. I suggest we remove one, and it's fine if it's yours that remains.
I haven't had a chance to compare the two, in practice, but looking at the code, it looks like we could add the Xbox 360 system search/results from my repo. Something I hadn't been able to fix about mine was that the game search always lasted about 10 seconds, which you might have solved. Either way a merger is probably best, whether as a PR to my repo or yours.
Screenshots
No response

Hi, I'm really sorry, I didn't know there was a plugin for Xbox. I totally missed it 😣😣😣
I apologize, I'm unable right now but I'll be removing it asap, likely in the afternoon.

PR to delete it opened, I apologize if this caused any discomfort https://github.com/JosefNemec/PlayniteAddonDatabase/pull/345

No worries! Please don't delete it from this repo yet, you've got at least one search method I hadn't seen yet.

No worries, I plan to keep it here for personal use only.


AzureWebAppV1: Resolve Heap out of memory issue.

Task name: AzureWebAppV1
Description:  clone of https://github.com/microsoft/azure-pipelines-tasks/pull/19139
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue: (Y/N) https://github.com/microsoft/azure-pipelines-tasks/pull/19139
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

/azp run

/azp run


XmlDriver incorrectly parses boolean option values

I have a mapping for a column like that:
<field name="alpha2Code" column="country_code" type="string" length="2" column-definition="CHAR(2) NOT NULL">
    <options>
        <option name="fixed">false</option>
    </options>
</field>

Despite that I specified false as a value of fixed option, it will be true, because in XmlDriver::_parseOptions method the string false will be casted to true with (bool):
if ($option->count()) {
    $value = $this->_parseOptions($option->children());
} else {
    // here
    $value = (string) $option;
}

It probably should be:
$value = $this->evaluateBoolean((string) $option);

@kalimatas can this be converted into a test case?

I'm on it right now.

Tests in #6130

I would like also to fix it, though I'm not sure, if maintaining an array of option names, that are supposed to be boolean in the _parseOptions() method , is a nice solution.

@kalimatas give it a shot - it can be a quick fix with a later refactoring, but if there's a possible BC break, we need to expose that as well.

I created a generic test for all drivers.

Closing as per work done in #6130

Note: this is only merged into 2.6.0 due to risk of regressions in the mapping driver change.