hyprctl -j option doesn't output valid json

Hyprland Version
adding version Hyprland, built from branch main at commit ec269622fcd1fd06d2db19d98593ffab8ff041cb dirty (flake.lock: update xdph). Tag: v0.28.0-23-gec269622  flags: (if any)
Bug or Regression?
Bug
Description
I apologize if this ends up being a regression and not a bug, I never encountered this problem before updating Hyprland today.
When using hyprctl <command> -j, the first line of output breaks the json syntax.
For example, the output of hyprctl monitors -j is the following:
adding monitors
[{
    "id": 0,
    "name": "eDP-1",
    "description": "Chimei Innolux Corporation 0x1515 (eDP-1)",
    "make": "Chimei Innolux Corporation",
    "model": "0x1515",
    "serial": "",
    "width": 1920,
    "height": 1080,
    "refreshRate": 60.007999,
    "x": 1920,
    "y": 0,
    "activeWorkspace": {
        "id": 1,
        "name": "1"
    },
    "specialWorkspace": {
        "id": 0,
        "name": ""
    },
    "reserved": [0, 35, 0, 0],
    "scale": 1.00,
    "transform": 0,
    "focused": true,
    "dpmsStatus": true,
    "vrr": false
}]

Or using hyprctl version -j:
adding version
{
    "branch": "main",
    "commit": "ec269622fcd1fd06d2db19d98593ffab8ff041cb",
    "dirty": true,
    "commit_message": "flake.lock: update xdph",
    "tag": "v0.28.0-23-gec269622",
    "flags": []
}

This creates multiple issues when trying to pipe the output of hyprctl <command> -j with tools like jq, unless the output has ben sanitized.
How to reproduce

Install Hyprland
Try to query any json output from hyprctl with jq (e.g.: hyprctl monitors -j | jq -rc '[ .[].id ] | sort_by( .


docs: fix systemd unit example for casual hosting

The Issue
During shutdown of a Linux host, the ddev poweroff triggered in the ExecStop= directive of the provided sample systemd unit file left containers dangling around. This may result in file permission conflict, if a custom mySQL config file is provided.
How This PR Solves The Issue
Changes the After= from network.target to multi-user.target, which is fired later on startup and thus ExecStop= will be fired earlier on shutdown, leaving ddev poweroff more time to properly clean up.
Compare the traces from the systemd-analyze critical-chain ddev.service for the alternate settings:
# After=network.target

ddev.service +21.635s
└─basic.target @5.090s
  └─sockets.target @5.090s
    └─docker.socket @5.083s +5ms
      └─sysinit.target @5.056s
        └─cloud-init.service @4.063s +987ms
          └─systemd-networkd-wait-online.service @2.301s +1.757s
            └─systemd-networkd.service @2.250s +46ms
              └─network-pre.target @2.245s
                └─cloud-init-local.service @1.390s +852ms
                  └─systemd-remount-fs.service @545ms +72ms
                    └─systemd-journald.socket @464ms
                      └─-.mount @418ms
                        └─-.slice @418ms

# After=multi-user.target

ddev.service +1min 7.167s
└─multi-user.target @6.749s
  └─docker.service @5.287s +1.459s
    └─containerd.service @4.865s +420ms
      └─basic.target @4.856s
        └─sockets.target @4.856s
          └─docker.socket @4.849s +5ms
            └─sysinit.target @4.831s
              └─cloud-init.service @4.100s +728ms
                └─systemd-networkd-wait-online.service @2.385s +1.709s
                  └─systemd-networkd.service @2.330s +51ms
                    └─network-pre.target @2.323s
                      └─cloud-init-local.service @1.440s +881ms
                        └─systemd-remount-fs.service @487ms +37ms
                          └─systemd-journald.socket @415ms
                            └─-.mount @378ms
                              └─-.slice @378ms

In particular the containerd.service and docker.service are required up prior to start of ddev.service, and in turn required to stay up until stop of ddev.service.
For more details see:

Debian: https://manpages.debian.org/testing/systemd/systemd.special.7.en.html
Ubuntu: https://manpages.ubuntu.com/manpages/focal/en/man7/systemd.special.7.html

Manual Testing Instructions
Setup a Ubuntu server, setup a ddev project with a custom mySQL file, restart server with shutdown -r 0.
Automated Testing Overview
Related Issue Link(s)
Release/Deployment Notes

Thanks for this! I'll look forward to trying it on my own little VM that runs the little sites I maintain.

@vanWittlaer do you have a link to add about how you learned this? Please edit the PR to add it, thanks!

@vanWittlaer do you have a link to add about how you learned this? Please edit the PR to add it, thanks!

@rfay Thx for the hint - updated the PR description - is that what you meant?

Thanks for the update. I was also hoping for a link to Debian/Ubuntu/Systemd docs (at least in the PR description here) that explain how this works and perhaps why we should use this approach.

Links added to description.

Hi @rfay thanks for merging, and for all the discussions and thoughts on this. Amazing you could reproduce it, must be a matter of milliseconds I thought. Anyway, looks to be stable now on my server.

It takes 10 seconds or more to ddev stop and my shutdown has to do 6-7 ddev stop so it was pretty easy to get back in there and see it. What surprised me is that sudo reboot kicked my ssh session out right away, but I was still able to ssh in again to look at this process.
Thanks for all your work on this!


Ability to review multiple lines.

Feature Description
It would have been nice if gitea had the ability to review multiple lines.
Screenshots
Currently on gitea:

What it could have been (like GitHub):

no progress so far. But this could be a useful feature like GitHub.

That's definitely on the plan.
There's simply no activity yet because no one took the time to implement it.

I've already gathered some implementation thoughts on this as I think GitHub's approach has some major drawbacks:
At the moment, we only gather one number +/-$line for a review comment, where + means on new and - means on old diff, and $line is the line number.
My proposal is to limit multi-line comments to one side: Either you mark multiple lines on the old side, or multiple lines on the new side, but no mixture in between.
Reasoning: You very rarely want a multi-line message that has both old and new content (speaking from my experience as a "serial reviewer"), and it has a couple of benefits:

it is (at least for the backend) far easier to implement: Simply store another signed number meant as a relative counter:

Case 1 - |x| = 0: It's a single-line message, so use the current algorithm including the corresponding settings.
Case 2 - |x| = 1: Show only the selected line without the additional context lines. Can be useful from time to time to clarify exactly which line you're talking about, and GitHub doesn't offer this functionality. Should be the same case as case 3, but is listed here for completeness.
Case 3 - |x| > 1: Show the next/last x lines, depending on the sign of x. Only show exactly the requested lines, and nothing more or less.
it might even make sense to switch the behavior of case 1 with the one of case 2 if an edge case can be avoided that way

it is far less error-prone (see i.e. GitHubs problems with suggestions)
the (backend) implementation is straightforward, hence much easier to maintain
it is a much better user experience (I always have to hope that GitHub interprets my dragging correctly which it often just… doesn't)

Duplicate of #12640.


Bluetooth: controller: Fix Coded PHY S2 S8 coding use

Fix PHY Update control procedure to accept the preferred Tx
coding options (S2 or S8) requested even if there is no
actual PHY change happening on procedure completion.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

@carlescufi or @kruithofa need yours as second approval here


IOS11.3 Linking can't open web URL

🐛 Bug Report
<--
-canOpenURL: failed for URL: "https://www.facebook.com" - error: "This app is not allowed to query for scheme https"
-canOpenURL: failed for URL: "http://www.myweb.com" - error: "This app is not allowed to query for scheme http"
-->
To Reproduce
<--
Linking.canOpenURL(type+string).then(supported => {
if (!supported) {
console.log('Can't handle url: ' + string);
} else {
return Linking.openURL(type+string);
}
}).catch(err => console.error('An error occurred', err));
-->
Expected Behavior
<--
open web url
IOS12 is normal, but IOS11 can't work, how can I solve it?
-->
Code Example
<--
Linking.canOpenURL(type+string).then(supported => {
if (!supported) {
console.log('Can't handle url: ' + string);
} else {
return Linking.openURL(type+string);
}
}).catch(err => console.error('An error occurred', err));
-->
Environment
<--
React Native Environment Info:
System:
OS: macOS 10.14.1
CPU: (4) x64 Intel(R) Core(TM) i5-4278U CPU @ 2.60GHz
Memory: 53.71 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 8.9.0 - /usr/local/bin/node
npm: 6.5.0 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 11.3, macOS 10.14,
Android SDK:
API Levels: 14, 15, 16, 17, 18, 19, 21, 22, 23, 24, 25, 26, 27, 28
Build Tools: 23.0.1, 23.0.3, 25.0.0, 25.0.1, 25.0.2, 25.0.3, 26.0.1, 26.0.2, 26.0.3, 27.0.1, 27.0.3, 28.0.3
System Images: android-23 | Intel x86 Atom_64, android-23 | Google APIs Intel x86 Atom_64, android-27 | Google APIs Intel x86 Atom, android-28 | Intel x86 Atom, android-28 | Google APIs Intel x86 Atom
IDEs:
Android Studio: 3.0 AI-171.4408382
Xcode: 10.1/10B61 - /usr/bin/xcodebuild
npmPackages:
react: ^16.6.3 => 16.6.3
react-native: ^0.57.8 => 0.57.8
npmGlobalPackages:
react-native-cli: 2.0.1
-->

Hey @s349856186, thanks for filing!
This doesn't appear to be a bug, you'll need to enable the scheme as documented here. You may want to check out this StackOverflow question or ask for help over there as this issue tracker is not a help forum.

@rickhanlonii Thanks for your reply.
How to enable the scheme?
I have added scheme to the info.plist.


[Question] How can we share lzs across environments?

The hierarchy model and landing zone composition prescribed by CAF makes sense to me and am using this successfully. However, I haven't seen any examples of how to properly do shared services.
For example, suppose I deploy my network hub at level2 in environment X. In level3 of environment X I can reference that hub and peer to it, all good. But now a new team comes on board, and I'd like to deploy environment Y.
How can I reference the hub from environment Y? Am I misunderstanding the purpose of environments?

If i understand you correctly, you can grab state from any environment as long as you share the same storage accounts for state.
When you call in your remote states below, you can just give the name of the file that needs to be read in. I haven't tried this but you could try using folder/name.tfstate to see if it calls in subdirectories so you maintain a bit of order.
Fundamentally the point is environments are a fictious construct and its really just a case of accessing the right state file in the storage account. Perhaps we can look at adding support for multiple storage accounts as right now everything is in one.
landingzone = {
  backend_type        = "azurerm"
  global_settings_key = "foundations"
  level               = "level2"
  key                 = "shared_services"
  tfstates = {
    foundations = {
      level   = "lower"
      tfstate = "caf_foundations.tfstate"
    }
    launchpad = {
      level   = "lower"
      tfstate = "launchpad_prod.tfstate"
    }
    launchpad_dev = {
      level   = "lower"
      tfstate = "launchpad_dev.tfstate"
    }
  }
}

Thanks for the reply @rdbartram.

you can grab state from any environment as long as you share the same storage accounts for state

Yes however this is exactly my point - for the states to be under the same set of storage accounts (launchpads), your Rover -env argument must be the same.
Suppose I have an Azure tenancy with 4 subscriptions under it, central, dev, test and prod. I first want to deploy the my foundations (level1) and shared_services (level2) to the central subscription. Lets suppose I use rover [...] -env central. Now in order to access these I also have to put all my dev/test/prod resources also under the storage accounts referred to by -env central, not ideal.
Intuitively I would want to have a separate Rover environment (and hence a separate set of storage  accounts) for each of dev/test/prod. But in this case which of them contains the shared services levels? And how can the other envs look up say the central hub if it's in a different set of launchpad storage accounts it has no reference to?
One way around this is to have a completely separate tenancy for each of dev/test/prod, implying a separate Rover env for each. Though I think during your live stream you said you generally dont like this model?
So in general my question is how are the community laying these out and what is the intended best practice for Rover environment structure.

Continued in https://github.com/aztfmod/rover/discussions/231

asjuil.uliwsd.


Change total lines badge to tokei.rs in README

What changes were proposed in this pull request?
Change "total lines" badge url to tokei.rs
And use empty alt message to display nothing on failure.
See https://github.com/XAMPPRocky/tokei#badges
Why are the changes needed?
Current badge sometimes show "total lines: invalid".
Does this PR introduce any user-facing change?
No.
How was this patch tested?
https://github.com/kaijchen/incubator-uniffle/tree/tokei#apache-uniffle-incubating

Seems #222 is more stable.

Closing this due to #222.


Add RESTOCK Action as a URL

As a Inventory Manager
I need a restock Action as URL
So that I can perform the restock to an Inventory entry
Details and Assumptions

We will store the inventory details in a database
Base URL is /inventory
There is a restock_level field for each Inventory entry to indicate the restock level

Acceptance Criteria
Given the service is running
When call the "PUT http://.../Inventory/<inventory_id>/restock" with an HTTP PUT method
Then the quantity of the Inventory entry with id=inventory_id should be updated to `restock_level+1`
The action is succeed if the quantity of the Inventory entry is below `restock_level`, and `HTTP 200 OK` is returned
Return `HTTP 409_CONFLICT` if the action is not succeed
Return `HTTP 404_NOT_FOUND` if Inventory entry with id=inventory_id is not found

@zhengtr make sure to uncomment the 409 conflict in error_handlers.py and test it as well when the quantity is above the restock_level


HV-1369 Adding def types for @Positive etc.

https://hibernate.atlassian.net/browse/HV-1369

@gunnarmorling https://hibernate.atlassian.net/browse/HV-1369 this one probably can be closed then :)

@gunnarmorling yeah, I didn't do it because Marko was on it. I'll fix the issue number and merge.

Yeah, sorry, wasn't aware of that other issue. Was more like a one min fix.
Thanks!

Merged, thanks!


cargo-tarpaulin: 0.20.1 -> 0.21.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/cargo-tarpaulin/versions.
meta.description for cargo-tarpaulin is: "A code coverage tool for Rust projects"
meta.homepage for cargo-tarpaulin is: "https://github.com/xd009642/tarpaulin"
meta.changelog for cargo-tarpaulin is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.21.0 with grep in /nix/store/p23rqrf835v73sj05xijls7nw7dpsbsq-cargo-tarpaulin-0.21.0
found 0.21.0 in filename of file in /nix/store/p23rqrf835v73sj05xijls7nw7dpsbsq-cargo-tarpaulin-0.21.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cargo-tarpaulin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/p23rqrf835v73sj05xijls7nw7dpsbsq-cargo-tarpaulin-0.21.0 \
  --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 cargo-tarpaulin https://github.com/r-ryantm/nixpkgs/archive/2439b9b438b0a85037e4cf09e60809326fd1b2f0.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/p23rqrf835v73sj05xijls7nw7dpsbsq-cargo-tarpaulin-0.21.0
ls -la /nix/store/p23rqrf835v73sj05xijls7nw7dpsbsq-cargo-tarpaulin-0.21.0/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.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cargo-tarpaulin

Maintainer pings
cc @HugoReeves for testing.

Result of nixpkgs-review pr 189063 run on x86_64-linux 1

1 package built:

cargo-tarpaulin


Panic if gateway-cli tries to connect to federation it's already connected to

Run tmuxinator
Run contents of gw_connect_fed function

thread 'main' panicked at 'Error opening new rocks DB: Error { message: "IO error: lock hold by current process, acquire time<PHONE_NUMBER> acquiring thread<PHONE_NUMBER>: /tmp/nix-shell.Y0lKYu/fm-ZkPl/cfg/gateway/aada38a8da939ae320849b6add1609a2e175ccad30d10acbc4a55eb4b3da5c401afe3f76209824f8757678d6a298ec19.db/LOCK: No locks available" }', gateway/ln-gateway/src/client.rs:65:59

https://github.com/fedimint/fedimint/pull/2099 was reverted https://github.com/fedimint/fedimint/pull/2135

Is this required for the alpha release?

Would be nice to make alpha. Let me revisit the reverted change


Hotfix: Unable to determine event type HYBRID

We are getting errors that the MeetupGraphql handler is breaking due to a Unable to determine event type HYBRID error.
{
    "class": "RuntimeException",
    "message": "Unable to determine event type HYBRID",
    "code": 0,
    "file": "/home/hackg/public_html/hack/app-modules/event-importer/src/Services/MeetupGraphqlHandler.php:38",
     ...
}

Closed by #394


Fix Safari selection bug

When a position property is declared on a DOM element, Safari will show the range selection highlighting between 2 elements when the anchor node is at the end of an element and the focus node at the start of the next sibling.
Removing the styling should fix this issue without causing any regressions.
Before

After

Closes #2797

It may cause some UX confusion, so logged a WebKit bug report as well: https://bugs.webkit.org/show_bug.cgi?id=244780


[REQ] investigate macOS support

What kind of request is this?
None
What is your request or suggestion?
publish darwin binaries as part of release

@alexellis we use goreleaser for release so it's all baked in. Next release will automatically will contain both arch for linux and darwin.
I am not sure if copa had any new features to cut a v0.3.0 yet. I can update the issue once we are cutting it.

Is there a chance you can cut a release for this now, so that we can get the PR right the first time, without having to go back and update / add extra URLs? The app was originally requested by @derdanne.


[Improvement] Use diff arg when checked for empty commit

Description
This is the addition to awesome feature of diff parameter alteration.
On our org lint-staged is used on pre-push hook. As a diff we use string like current-branch..origin/master.
This leads to a situation when no staged items remains at the moment of hook running. And when check for empty commit is performed here https://github.com/okonet/lint-staged/blob/50f95b3d51e69074ab5ff5ddb7147828fcd85b7b/lib/gitWorkflow.js#L252 it fails, as can be expected.
Currently we hack over this issue with --allow-empty flag, but this isn't a solution imho.
Can same condition as here be added to applyModifications workflow, to get these check aligned?
I've tried to push 2-lines PR here, but had problems with permissions.
Steps to reproduce
Use custom diff CLI argument and run lint-stage on push. You won't be able to finish pushing even if you made changes.
Debug Logs
Not sure if this is needed.
Environment

OS: macOS Monterey
Node.js: 14.17.5
lint-staged: 13.0.1

That sounds reasonable. Please submit a PR and we will review it. WYT?

You can fork this project in GitHub and create a new branch there, and from that open a PR to this original project.

Thanks for suggestion @iiroj!
I'm not used to push to open-source :)


更新代码。

Describe what this PR does / why we need it
Does this pull request fix one issue?

Describe how you did it
Describe how to verify it
Special notes for reviews

更新代码。


Stop handled key events propagating

What: Stop the propagation of keydown events that have been handled by downshift

Why: If I am using downshift inside a component that is also handling key events then it shouldn't be receiving them if downshift has already taken action

How: using event.stopPropagation()

Checklist:

[ ] Documentation "N/A" - I think there's nothing to add to the documentation about this, the user can still disable this behaviour (as they could before) by setting event.preventDownshiftDefault, so I don't think there needs to be a new addition to the dos?
[x] Tests
[x] Ready to be merged

[ ] Added myself to contributors table - the add-contributors script failed on my windows machine

This fixes issue #630

Also, running the tests locally
npm run build-and-test failed for me... within the build-and-test script, not sure what the error was
npm run test:cypress had one failing test
but I think this might just be my machine, as they both failed locally before my changes,

I wonder if we should use stopPropagation for every event 🤔 (and not just these specific keys).

Potentially, I think if we are calling preventDefault it's a good indicator that we probably want to call stopPropagation.
I would be a little wary of overusing stopPropagation. Calling stopPropagation is effectively cancelling the event and saying that no-one else can handle it. I have also sees varying accounts for when/if you should call stop propagation, see https://css-tricks.com/dangers-stopping-event-propagation/ for instance.
I think we can safely use stopPropagation when you are convinced that nothing else should be doing something with that event (it feels like this is probably generally true). In our case, you are using the keyboard interaction to navigate the downshift control/select an element/close the downshift control. In all cases the keyboard interaction is within the context of the downshift control and doing something to that control. The same keys should not be being re-used for some other functionality whilst we're within the context of downshift. Maybe we review the other event handlers and decide if they fit the same criteria?

That makes sense. I think we'll stick with this until someone complains 👍 Thanks!

:tada: This PR is included in version 3.1.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


rockpro64 6.47-244-arm64 boot log

Newest build is not booting rockpro64:
bionic-minimal-rockpro64_244output.txt
Just attaching my output log in case it helps any.

But. This output is from 0.6.46.
On Sat, 2 Jun 2018 at 22:03, deeproot2k<EMAIL_ADDRESS>wrote:

Newest build is not booting rockpro64:
bionic-minimal-rockpro64_244output.txt
https://github.com/ayufan-rock64/linux-build/files/2065312/bionic-minimal-rockpro64_244output.txt
Just attaching my output log in case it helps any.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ayufan-rock64/linux-build/issues/205, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ACTpQW_9jYkqq5BIW_bu6F3wsZp7EZj9ks5t4u-jgaJpZM4UX0qE
.

bionic-minimal-rockpro64.txt

0.6.49: jenkins-linux-build-rock-64-247
bionic minimal fully booted. Thanks


VNC settings for Bookworm

Whilst trying to build images for some Bookworm Pi 4/5s, (and yes, my first attepmts at using sdm so beware neophyte confusion) I was unable to get the vnc connection setup to work.
I suspect this is a case of needing a little adaptation for the Wayland/X11 choice and the different config files used by Wayland.
My temporary work-around is to do the VNC setup on the target machine, which isn't horribly onerous for a few but might be a little irritating for a couple of dozen.

TBH I've not anything with enabling VNC with Wayland, so it's not surprising doesn't work.  OTOH if you're using X11, then this is quite surprising.
I was waiting for things to die down WRT realvnc/wayland/bookworm, and also for someone to ask about it. 🤔
Can you provide a simple repro for what you're seeing?
Thx

So far as I can tell the issue is that on a Pi 5 the default is to boot into Wayland and thus the X setup does nothing. The symptom is a tiny VNC window and the 'unable to show desktop' error when I open RealVNC on my iMac.
I did try the raspiconfig: plugin with "do_wayland" (sometihng or other) and it did nothing helpful.
The simple reproduction would be to build an image using the vnc plugin and any allowed vnc resolution, then run the result on a Pi 5, then run RealVNC on some other machine. Oh, I'm curently using RealVNC Viewer 7.8.0 (r51458) x64 (Nov 2 2023 15:35:05)

Here's what I found in looking at this today on 64-bit Bookworm with Wayland enabled:

RealVNC server does not work. It is unable to find a running X server (error logged in the system journal, viewable with journalctl), hence gets "unable to show desktop"
If I stop RealVNC server, and do sudo raspi-config do_vnc 0 nonint, the wayvnc server is enabled.

RealVNC viewer (at least on Windows) is able to connect to this and show the Pi desktop. TightVNC Viewer does NOT work against wayvnc, it gets "no supported security types" error
HOWEVER, wayvnc is not started until a user logs in on the console. Yes, this is a problem if the Pi reboots and you're not at home.

If you switch to X11, there is more flexibility.

RealVNC works. You do not need to be logged in on the Pi console in order to connect
Virtual VNC sessions configured using sdm's vnc plugin work correctly

Based on what I've read, RealVNC may support Wayland sometime next year, but if you don't want to wait for that, your choices are:

Use Bookworm with Wayland and wayvnc, but wayvnc doesn't start until you login on the Pi console
Use Bookworm with X11 and both RealVNC and virtual VNC sessions work

Hope this helps.

On 2023-12-29, at 5:07 PM, Benn @.***> wrote:
Here's what I found in looking at this today on 64-bit Bookworm with Wayland enabled:
• RealVNC server does not work. It is unable to find a running X server (error logged in the system journal, viewable with journalctl), hence gets "unable to show desktop"

That makes some sense.

• If I stop RealVNC server, and do sudo raspi-config do_vnc 0 nonint, the wayvnc server is enabled.

I think that corresponds to what I found

  • RealVNC viewer (at least on Windows) is able to connect to this and show the Pi desktop. TightVNC Viewer does NOT work against wayvnc, it gets "no supported security types" error
  • HOWEVER, wayvnc is not started until a user logs in on the console. Yes, this is a problem if the Pi reboots and you're not at home.

Ditto.

If you switch to X11, there is more flexibility.
• RealVNC works. You do not need to be logged in on the Pi console in order to connect

Yes

• Virtual VNC sessions configured using sdm's vnc plugin work correctly
Based on what I've read, RealVNC may support Wayland sometime next year, but if you don't want to wait for that, your choices are:
• Use Bookworm with Wayland and wayvnc, but wayvnc doesn't start until you login on the Pi console
• Use Bookworm with X11 and both RealVNC and virtual VNC sessions work
Hope this helps.

I was initially able to get RealVNC working on a Wayland Pi by editing the .config/wayvnc/config to comment out the enable_auth=true line but that is a Pi that I have not used sdm to build for as yet.
tim
tim Rowledge; @.***; http://www.rowledge.org/tim
I haven't lost my mind; it's backed up on tape somewhere.

I didn't need to comment out the enable_auth line to get RealVNC working. I did exactly what I documented above.
In any case, this is a RasPiOS issue, not an sdm issue. Both RealVNC and virtual vnc desktops created with the vnc plugin work correctly, as expected.
If you want to use Wayland, you have to accept some compromises for now, VNC being one of them. OTOH if you want VNC to be fully functional, switch to using X11.

Just-released sdm V11 has wayvnc working on Builds with Desktop. Using --plugin vnc:wayvnc, I am able to connect to the resulting system using RealVNC viewer on my Windows system with no config file editing required.
Please take it for a spin and LMK how it goes.

On 2024-01-05, at 3:16 PM, Benn @.***> wrote:
Just-released sdm V11 has wayvnc working on Builds with Desktop. Using --plugin vnc:wayvnc, I am able to connect to the resulting system using RealVNC viewer on my Windows system with no config file editing required.
Please take it for a spin and LMK how it goes.

OK - well it took a few tries; for some reason my first go resulted in an SD card that simply wouldn't boot in the Pi 5.
I can confirm that using
--plugin vnc: "wayvnc=1600x1200"
results in a system that boots with
a) wayland in use
b) wayvnc configured
c) and that RealVNC on my iMac connects to it.
However the resolution it started with was 1280720 for some reason. Setting it to 16001200 in the on-screen prefs tool and rebooting did sort that out.
Thank for fixing this issue.
tim
tim Rowledge; @.***; http://www.rowledge.org/tim
Unprecedented performance: Nothing ever ran this slow before.

Thanks for checking it out, and...whoops! I apparently was so excited that it worked at all, I didn't finish it up so that you could say wayvnc=1600x1200 and actually have it DO something with that resolution 😵
Will get that taken care of for the next update

On 2024-01-06, at 6:09 PM, Benn @.***> wrote:
Thanks for checking it out, and...whoops! I apparently was so excited that it worked at all, I didn't finish it up so that you could say wayvnc=1600x1200 and actually have it DO something with that resolution 😵
Will get that taken care of for the next update

Odd thing - I had to redo (again) trying to make adding a desktop file/association work, and this time the vnc resolution worked immediately!
tim
tim Rowledge; @.***; http://www.rowledge.org/tim
Strange OpCodes: BBL: Branch on Burned out Light

Opps. Forgot to press CR after that --customize and so I was just re-using the prior much-fudged SD card. Oh well.

On 2024-01-06, at 6:53 PM, Tim Rowledge @.***> wrote:

On 2024-01-06, at 6:09 PM, Benn @.***> wrote:
Thanks for checking it out, and...whoops! I apparently was so excited that it worked at all, I didn't finish it up so that you could say wayvnc=1600x1200 and actually have it DO something with that resolution 😵
Will get that taken care of for the next update

Odd thing - I had to redo (again) trying to make adding a desktop file/association work, and this time the vnc resolution worked immediately!
tim
tim Rowledge; @.***; http://www.rowledge.org/tim
Strange OpCodes: BBL: Branch on Burned out Light

tim
tim Rowledge; @.***; http://www.rowledge.org/tim
Sarchasm (n): The gulf between the author of sarcastic wit and the person who doesn't get it.

No worries. Just finished adding support for wayvnc=widthXheight. Just as you did with the prefs tool, sets the "headless" resolution.
I'm pretty sure that it's overriden if there's a monitor attached, but I haven't tried that yet.
Also, did you notice that wayvnc doesn't start until you actually login on the console? I guess that if you want to use wayland and wayvnc reliably setting desktop autologin becomes a necessity, esp if it's headless.
Is that what you're doing?

On 2024-01-07, at 12:30 PM, Benn @.***> wrote:
No worries. Just finished adding support for wayvnc=widthXheight. Just as you did with the prefs tool, sets the "headless" resolution.

Excellent; I'll try that soon, though right now my SD card is claiming to be full after --burn gets 700Mb written. No idea what is happening there but we'l try again.

I'm pretty sure that it's overriden if there's a monitor attached, but I haven't tried that yet.
Also, did you notice that wayvnc doesn't start until you actually login on the console? I guess that if you want to use wayland and wayvnc reliably setting desktop autologin becomes a necessity, esp if it's headless.
Is that what you're doing?

Yes, at least I think so; using --plugin raspiconfig:"boot_behaviour=B4"
tim
tim Rowledge; @.***; http://www.rowledge.org/tim
Useful Latin Phrases:- Canis meus id comedit = My dog ate it.

On 2024-01-07, at 12:30 PM, Benn @.***> wrote: No worries. Just finished adding support for wayvnc=widthXheight. Just as you did with the prefs tool, sets the "headless" resolution.
Excellent; I'll try that soon, though right now my SD card is claiming to be full after --burn gets 700Mb written.
Don't rush. It's not checked in yet. Should be there tomorrow 🤞

V11.1 is posted and enables wayvnc=widthXheight

On 2024-01-08, at 2:40 PM, Benn @.***> wrote:
V11.1 is posted and enables wayvnc=widthXheight

Excellent; worked first time (now that I completely changed the USB hardware around the dratted Pi 4 with the SSD that makes it practical to use as a builder machine!)
Now I just need to find out how to
a) create .desktop files for my loaded application that actually get hooked up
b) set the preferences for the filer windows so I get all the right options and places displayed
... so just more reams of out of date google-drivel to sift through!
tim
tim Rowledge; @.***; http://www.rowledge.org/tim
If at first you don't succeed, call it version 1.0

I haven't put anything in sdm to specifically help with .desktop files, but once you have created one you can use the copyfile plugin to have sdm copy it from your host system into the IMG.
For the filer window preferences, I just configured up a running system with the settings the way I want them and then save libfm.conf and pcmanfm.conf so they can be fed into new IMGs using the lxde plugin.

Just connecting this to #160 for the future

I believe wayvnc issues are all corrected now. OP, please re-open if further issues warrant. Thx!


UI: 2 step ZIP import - to view diffs on import

Import ZIP is a black box now. Proposal: to make it 2 steps

Import and see diffs
Apply changes

Example after import:

yeap, this will close 816. One point: do you think that file view is needed ? I wasn't going to show files in offline initially. But indeed should not be difficult. I'll PR this week.

One more point. I added has_remote, status and reset_remote to base repo class - https://github.com/sbcgua/abapGit/commit/e0f88c71af48317ae8a2f982de38de8ea98d41e5#diff-86243fe7bed62c7e0c920cd3dc73ca08R369. Though having some doubts about proper semantics - any opinions about it? last 2 methods are then redefined in online implementation.

Closing then. Delivered

not merged yet, will merge later today/tomorrow, leaving a chance for more people to test it

ok, rushed a bit. let's leave it open then for the moment


Unable to connect Azurite using azure javascript library

Hi,
I am trying to connect azurite using azure javascript library.
When I make a get request to any container,  the browser tries to send preflight request which is failed with error code 501-not implemented.  Even when I make a options request using fiddler, it fails with same error.
What configuration I need to do to make it work.

Understood, thanks. I'm running azurite for development only so happy to try it out. Any chance you could check in an example Dockerfile for the new version?

I've got v3 up and running in Docker - the same issues appear to still be present.

@tomhosking V3 doesn't support OPTIONS yet, it's marked as one of not implemented features. https://github.com/Azure/Azurite/projects/1#card-18684936


Units with powers of pi lose precision

kg^(2pi) / kg^(2pi) to 1 results in approx. 1 when it should be exact.

Fixed in 8768fd2


Test

Description
[Include a brief description of the changes you've made]
Related Issues
[Include any related issues or pull requests that this PR addresses or is related to. Use GitHub's shorthand syntax to link to them, like #1234.]
Changes Proposed
[Describe the changes you've made in detail. Be specific and include any relevant code snippets.]
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Screenshots
Note to reviewers

Hey @roc102 What are doing I have merged your profile PR and its live now.


Where is the chatting

I followed your instructions step by step from the README, and the end result is that I get a webpage displayed that says "My website :-)" but there is no chat box or anything. What am I missing?

Hi, Thanks for trying the chat :smiley:. Did you edit the "index.html" and also "config.js" file with the connection to node.js? (remember that if you have it all in localhost like in the exmaple then you not need to edit them) but if you try to connect to the chat from another computer, then you must change this values.
what browser and version are you using? what webserver are you using to serve the chat?, the one described in readme (python) or a full web server like apache?.
Would be very helpful if you could provide a print screen (send me in private by email) and also if you could turn on firebug (in firefox) or console debug in chrome would help to see if there is any trouble with javascript or any lib.

I have been experienced this issue in some chrome versions, pretty sure when the next warning is displayed:

jquery-1.11.0.min.js:4 Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check http://xhr.spec.whatwg.org/.

the chat can't be loaded.
I keep searching the best solution for this issue, for now just try another browser and will work.

I tested this issue and seems to be fixed now, please update the latest changes on the master branch and tell me if it works now.
Best regards

Hi @lovelle. I just got the latest changes and it still doesn't work. I am using WIndows 8 with XAMPP.
I changed line 5 of config.js to
   server: 'localhost:3000',

I changed line 11 of index.html to
    <script src='http://localhost:3000/socket.io/socket.io.js'></script>

Now I am not getting any errors in the console, but all I see is "My website :-)"

Oh, wait a minute, I think it is working now....it is the bar at the bottom of the screen, right?

Yes :+1:, remember that you could personalize the bar with some custom params in "config.js".
With some of the latest changes pushed at github i have configured the chat to be work with a heroku app, but you could deploy your own node.js app. If you have any questions related to any functionality feel free to send me an email.


Document Eclipse URL

The comment adds information about the Eclipse download URL format to hopefully guard against links that contain a specific mirror or that don't directly link to the file.
Since this may change the future, I understand if this is not a desirable addition.

If in doubt, I use the alternate link they provide:


feat: URL to CRDs definition will ignore go-template blocks

Hi,
in helm chart templates, there are usually go-template functions or blocks, and I wish to point to the CRDs definition with go-template blocks and render yaml from this file.
for example:
https://raw.githubusercontent.com/fluxcd-community/helm-charts/main/charts/flux2/templates/helm-controller.crds.yaml
{{- if and .Values.installCRDs .Values.helmController.create }}
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
  annotations:
    controller-gen.kubebuilder.io/version: v0.15.0
  labels:
    app.kubernetes.io/component: helm-controller
...
...
...
    subresources:
      status: {}
{{- end }}

Now I getting error in UI

Hello 👋
Ah, interesting. Let me think what I can do about that. :) I can't promise that I'll be able to completely parse a CRD, but perhaps, trimming something from the beginning and the end is enough in this case. ¯_(ツ)_/¯

For example, things like this:
    app.kubernetes.io/component: helm-controller
    app.kubernetes.io/instance: '{{ .Release.Namespace }}'
    app.kubernetes.io/managed-by: '{{ .Release.Service }}'
    app.kubernetes.io/part-of: flux
    app.kubernetes.io/version: '{{ .Chart.AppVersion }}'
    helm.sh/chart: '{{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}'

I will not be able to resolve. So I'm just going to replace them with some dummy value to keep the YAML from breaking.

Hopefully, the website should also be updated.. I'll test it in a couple of minutes.

Website doesn't work because I'm a muppet.

Okay, now it's working! :) Thank you for the issue! :) 🙇

Please try to refresh the website with force refresh so the WASM binary is flushed from the cache.

Yaaaaay! Glad that it works. :) 🎉


[Bug] Fix action column dropdown

WHY
BEFORE - What was wrong? What was happening before this PR?
I updated CRUD and realized is not properly working. Maybe when I tested this the browser was using some files from v6.
AFTER - What is happening after this PR?
It fixes the way dropdown looks.
HOW
How did you achieve that, in technical terms?
Simply fixing jquery!
Is it a breaking change?
Nope.
How can we test the before & after?
Open the dropdown in action in with and without this branch and it should look good with it :)

A new inspection was created.

The inspection completed: No new issues

Hey @maurohmartinez what was the issue ?
I never used v6 branches, and the dropdown was properly working when I tested it.
Cheers

Let me have a look, I will get back to you 👍

Yes @maurohmartinez it seems is something in your installation.
AFAIK dropdown is working on master:

Cheers

@pxpm pfeew... sorry for the false alarm 🙈

Let's assume some changes in v6 may break this then.
Let's leave this open until v6 alpha is out 🙏

I don't think this is needed, everything is working without this changes.
Thanks Mauro 🙏


Prevent overriding self.headers.Cookie when explicitly already set

Prevent overriding self.headers.Cookie if the user of the request library sets it explicitly -- helpful for proxying scenarios ... can't do my job without this feature :) open to suggestions on implementation

what about redirects?
if i'm reading this right, headers.cookie will get set if you set a cookie jar, then on a redirect the header will already be set and won't pull a new header out of the jar for the new domain it's redirecting to.
we could solve this by checking for headers.Cookie and disabling cookies instead checking for the header as a condition to call in to the jar.

I'm actually setting jar to false (in my application code) for this reason and then i pass the set-cookie header back to my client in the response call back such that any additional requests from my client will include the cookies. this also insures that different clients at my server don't share session data.
i'm fine with having request do jar=false automatically but i thought that not doing that left the library more generic and enabled user to do what makes sense for them -- although maybe by not automatically setting it to false could cause hard to diagnose issues -- if request is following redirects and maintains cookies internally then by the time it hits the final page it would no longer have response.headers['set-cookie'] (because it went ahead and set the cookie) which i want to pass back to my client?
i'm also assuming, because it suits my use case, that redirects are always going to point to the same domain.

hrm...

is this still a bug or did a fix go in? it no longer merges cleanly, which is partially my fault for not seeing it sooner.

I'll test my project against the latest upstream and see what happens and report back later today or tomorrow.

big change today, might need to rebase. closing until this is updated.


Dynamic meta tags?

Hello,
has anybody managed to get Slingshot some basic functionality of dynamic meta tags, that can be read by site crawlers and scrapers (for example Facebook sharer mechanism)?
I've been playing around with React Helmet, however that one adds dynamic meta tags only after site is rendered, which is too late for scrapers.
I suppose that the possible solution would be to have at least partial server side rendering in place, but I'm not sure how we can achieve that with Slingshot boilerplate. Any ideas? :)

@paulwalczewski Sorry I am not familiar with SSR, but I will continue to push this to channels I know that have dev that are using SSR.

hi @kwelch, I've found a workaround solution for this.
Basically I serve production files as a .ejs templates rather than pure html. Then I can populate metatags directly from node - so they are both dynamic (using ejs variables) and generated on server.
Of course downside of getting them from node is that they don't come from react app itself any more, so there has to be separate handling, translations etc, so I guess it would still be awesome to get a  proper SSR solution for slingshot ;)

I don't think there was ever an intent for Slinshot to handle SSR. If something is running in both client and server I personally would extract it to a package that could be a dependency or both.

Closing since React Slingshot isn't focused on server-side rendering support. Suggest alternatives like Next.js for that.


[FEATURE] Adding PreLoader Page

As the feature increases the loading time increases so it req. to add a pre-loading page to maintain user engagement and no one want to see blank page when it loading So better to add a loading page to make website more active and alive.
If this new feature req. do assign under Gsooc 2023 .!
Changes will be done

Hello ManishaSwain8, thanks for opening a issue, your contribution is valuable to us. The maintainers will review this issue and provide feedback as soon as possible.


Make the whole command string readable in the "Select a process" dropdown window

When you want to attach with the debugger to a process you would put something like this in your launch.json
{
    "configurations": [
        {
            "name": "...",
            "type": "lldb",
            "request": "attach",
            "program": "...",
            "pid": "${command:pickProcess}"
        }
    }
}

This will open the "Select a process" dropdown

I can't see the whole string to select the correct process. Hovering over an item from the list does not display a tooltip with the whole command string. Also the window is not resizable. I think both functionalities should be available.

Note that with "cppgdb" there is a tooltip when hovering.

Fixed in 1.9.0


How to use Touch events

Question
I'am trying to use the Touch event without success. I succeeded to listen to the event but I am not able to use it. To be more precise I receive a reference and I do not know how to retrieve the X, Y inside this ref.
It works fine for event like mouse over:
... update
            Msg::MouseOver(e) => {

                let x = &e.offset_x();
                info!{"MouseOver: {:?}",x}
            },
... view
            onmouseover = self.link.callback( move |e: MouseOverEvent| Msg::MouseOver(e))

... result:
INFO:game2048_html::components::page -- MouseOver: 53.0

But I do not know how I can extract / retrieve the information from the reference in the case of the TouchStart evt.

... update
            Msg::TouchStart(evt) => {
                info! {"Receive touch start evt: {:?}", evt};
... view
            ontouchstart = self.link.callback(move |e: TouchStart| Msg::TouchStart(e))

result:
INFO:game2048_html::components::page -- Receive touch start evt: TouchStart(Reference(1255))

Looking in the documentation it says to add the Feature to the crate ...
yew = { version = "0.14", features = ["std_web","TouchEvent"] }

but then it does not compile anymore 😢
Could you help / guide me to the good path forward?
I am using 'Cargo Web'
Thank in advance
What I've tried (optional)

@rlasjunies hey there! 👋
First, there is no need to modify cargo features to use events so you can remove the "TouchEvent" feature 👍
As to your issue, it looks like you will need to import the ITouchEvent trait from stdweb in your code. Then you can call event.touches()

@jstarry  👍
Perfect, that was the trick
Just added te line below in the code and it works as expected.
use stdweb::web::event::ITouchEvent;

I close the issue
Thanks
PS: the doc is may be confusing should not we remove the sentence?

If yes, guide me and I will do.

Ah the problem is that Yew re exports the types from web-sys. We should fix that. Can you file an issue to stop exporting those types?

Looking the issue #1134 I think you already made it.
Stay safe!
Le sam. 25 avr. 2020 à 17:30, Justin Starry<EMAIL_ADDRESS>a
écrit :

Ah the problem is that Yew re exports the types from web-sys. We should
fix that. Can you file an issue to stop exporting those types?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/yewstack/yew/issues/1126#issuecomment-619396605, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK7QL44UTHQTX4L3YAWT6DROL62ZANCNFSM4MQJSK4A
.

--
Richard Lasjunies

@rlasjunies that is actually a separate issue 😄


Uri.tryParse(null) throws a NoSuchMethodError

I don't think this is the correct type of error:
Uri.tryParse(null)

Unhandled exception:
NoSuchMethodError: The getter 'length' was called on null.
Receiver: null
Tried calling: length
#0      Object.noSuchMethod (dart:core/runtime/libobject_patch.dart:50:5)
#1      Uri.parse (dart:core/uri.dart:788:17)
#2      Uri.tryParse (dart:core/uri.dart:1033:14)
#3      main
bin/uri.dart:2
#4      _startIsolate.<anonymous closure> (dart:isolate/runtime/libisolate_patch.dart:300:19)
#5      _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

The docs aren't really clear about what should happen for null, they say:
Returns null if the string is not valid as a URI or URI reference.

However int.tryParse(null) throws Invalid argument(s): The source must not be null so it seems like this should probably be consistent.
(testing with current nightly build)

It should be consistent and always throw an ArgumentError.
I so can't wait for non-nullable types :)

I so can't wait for non-nullable types :)

You and me both (and no doubt countless others)! 😀

It's not possible to pass null to Uri.tryParse now :-)


Add authentication support

My Eco-Devices is protected with the password it provides (simple HTTP authentication).
I've never worked on a home assistant component before; but I've been able to make it work :)

Hello @trasher, thanks for the PR, I will try this week-end on my ecodevices

Hello @trasher, thanks for the PR, I will try this week-end on my ecodevices

Tested and approved, I let you see my review, I will merge after that :)

Tested and approved, I let you see my review, I will merge after that :)

Tested and approved,

Cool :)

I let you see my review, I will merge after that :)

Hum... Your review? Maybe you forget to submit it?

Tested and approved,

Cool :)

I let you see my review, I will merge after that :)

Hum... Your review? Maybe you forget to submit it?

I will close this PR since I began a full rewrite with your modification: https://github.com/Aohzan/ecodevices/pull/4
You can contribute on it if you wants

I will close this PR since I began a full rewrite with your modification: https://github.com/Aohzan/ecodevices/pull/4
You can contribute on it if you wants


Library included multiple times in bundle code

Bug report
Note
This is a similar issue to #2134, but the context is not exactly the same, so I chose to open another issue.
What is the current behavior?
Here's the configuration for the server bundle: https://gist.github.com/tstirrat15/38a1ab2b768406496284d5fdfbef63a6
When I run our production build and inspect the output, I see:
// ...
/* harmony import */ var react__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(0);
/* harmony import */ var react__WEBPACK_IMPORTED_MODULE_1___default = /*#__PURE__*/__webpack_require__.n(react__WEBPACK_IMPORTED_MODULE_1__);
// ...
/* WEBPACK VAR INJECTION */(function(module) {/* harmony import */ var react__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(0);
/* harmony import */ var react__WEBPACK_IMPORTED_MODULE_0___default = /*#__PURE__*/__webpack_require__.n(react__WEBPACK_IMPORTED_MODULE_0__);
// ...
/* harmony import */ var react__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__(0);
/* harmony import */ var react__WEBPACK_IMPORTED_MODULE_2___default = /*#__PURE__*/__webpack_require__.n(react__WEBPACK_IMPORTED_MODULE_2__);
// ...

Where __webpack_require__(0) is:
/******/ 	// Load entry module and return exports
/******/ 	return __webpack_require__(__webpack_require__.s = 868);
/******/ })
/************************************************************************/
/******/ ([
/* 0 */
/***/ (function(module, exports) {

module.exports = require("react");

/***/ }),
// ...

This indicates that there are at least 3 instances of React floating around during execution. This hasn't been a problem in the past, but with recent changes to the React context API, React now needs to be a singleton.
In #2134, I found a recommendation to use optimization: { runtimeChunk: 'single' }, and when I tried that, I only found a single instance of React in the bundle, which was good. However, running node build/server.js immediately exited 0 without waiting for requests.
Is there something obvious that I'm missing? If so, I'd be happy to add it to documentation.
If the current behavior is a bug, please provide the steps to reproduce.
I don't have a minimal repro yet. My impression is that it's the result of the confluence of a bunch of different things, rather than an easily localized bug.
What is the expected behavior?
A single instance of React, or a working server build with runtimeChunk: 'single'.
Additional context

This is an isomorphic React app, with the majority of code being shared between frontend and backend, using two different entrypoints.
We make extensive use of code-splitting

Please let me know if there's any additional context that would be helpful.
Other relevant information:
webpack version:  4.29.6
Node.js version: 8.15.1
Operating System: Ubuntu
Additional tools: (like?)

When I use runtimeChunk: 'single', I get three output files:

server.js: Contains the webpack require bootstrap code
0.server.js: seems to contain most of the app code
1.server.js: seems to contain the import + runtime code.

However, the call of the bootstrap function in server.js is empty:
// ...
/******/ 	// uncaught error handler for webpack runtime
/******/ 	__webpack_require__.oe = function(err) {
/******/ 		process.nextTick(function() {
/******/ 			throw err; // catch this error by using import().catch()
/******/ 		});
/******/ 	};
/******/ })
/************************************************************************/
/******/ ([]);
//# sourceMappingURL=server.js.map

I assume that second-to-last line is where it should be doing something with the other chunks?

I looked at the client bundle as well, and it's exhibiting the same problem. What are potential causes of this?

I'm facing the same issue!! Any update on this?

It is open source feel free investigate

I'd love to, but so far the investigations that I've done haven't borne any fruit. Do you have any recommendations on where to start looking? I've messed around with runtimeChunk, and that's the only thing that I've found that other issues/SO answers have pointed to.
I've got a few ideas of things that I can try - such as cutting down on my project until something changes, or seeing if I can repro with create-react-app - but those seem like blunt instruments, especially compared to something that someone who's already familiar with the internals would come up with.
Again, I'm happy to do the legwork, but I would love having a bit of direction beforehand.
Unless you're saying that I'm unlikely to find that sort of guidance from the maintainers, in which case I can go tool around myself.

How does require work on nodejs? Will consecutive calls to require produce a reference to the same object?
Additionally, I noticed that the webpack require references an entry in a big ol' array of import code. Is that meant to provide a caching mechanism as well? Is it possible that the webpack require array closures are messing up the node require caching somehow?

Can you provide minimum reproducible test repo with notes about that you actually have and what you expected, thanks

I'm working on a minimal reproduction repo.
One thing I did notice, though, was that when I included react in the exceptions to our externals configuration, the problem went away. Seems like it might have something to do with the interaction between node's require and the webpack require internals?

More examples of people seeing this issue here:
https://github.com/facebook/react/issues/13991
The fix described above didn't end up working - I ran into the same hooks error described in the issue above. What did fix it was removing react from externals and then adding aliases for resolution:
const config = {
  // ...
  resolve: {
    // Force webpack to resolve all references to react to the same package.
    // This should fix the problems we're seeing with hooks/context failing.
    alias: {
      react: path.resolve(path.join(__dirname, '../node_modules/react')),
      'react-dom': path.resolve(path.join(__dirname, '../node_modules/react-dom')),
    },
  },
  // ...
};

Which still seems to point to some issue with webpack internals. Is this being tracked anywhere else?

bump for future investigation

duplicate of https://github.com/webpack/webpack/issues/985


feat(MeasureTheory.MeasurableSpace.CountablyGenerated): add SeparatesPoints and related theorems

the goal of the PR is to clarify the relationship between the following closely related assumptions on a measurable space:

Being countably generated (CountablyGenerated)
Separating points (SeparatesPoints)
Having a countable separating family of measurable sets (HasCountableSeparatingOn _ MeasurableSet univ)
Having singletons be measurable (MeasurableSingletonClass)

It defines SeparatesPoints and registers all implications between combinations of these properties as instances (or rather a minimal set needed to deduce all others).
It also proves an optimal theorem regarding when a measurable space appears as an induced subspace of the Cantor Space Nat -> Bool. This will be used to generalize topological assumptions to measure theoretic ones in some theorems in MeasureTheory.Constructions.Polish.

bors r+

bors r+


Simulation fails in "run" mode.

I noticed that simulation fails in the "run" mode. I use https://github.com/akhoubani/snek5000-cbox/tree/fix/run-mode with main branch of snek and the nek directory of snek. I leave here also the OAR logs.
OAR.stderr.txt
OAR.stdout.txt
When I use "run_fg" it works.

The run rule executes in the background and it is understandable if the job scheduler would kill the job if there is no running process in the foreground. Is that is what is happening? If you want to use run, keep the while check_running(): loop.

Thanks for the comment. I didn't know. It works as you say.


Detect Serial Port disconnection #question

Hi @FengChendian, thanks for creating such a great package for flutter, just wandering is there any way to detect serial device disconnection, i was looking arround it in my application. kindly waiting for your response on this.
Thanks

I think Microsoft doesn't provide a disconnection directly. In my opinion ,there are some indirect method to detect the device connection status.

You can write bytes to COM periodically. If device is disconnected, write should fail. And GetLastError() will ERROR_HANDLE_INVALID. Maybe it's a little stupid...
GetCommState should return 0 when device disconnected. Periodically Calling GetCommState may be a good solution. I will test and add some function if it works.
I'm reading win32 doc to find better solutions


extract-from-pubmed raises TypeError

Running the extract-from-pubmed command leads to a TypeError.
Looks like the input text ends up empty (or starts that way and doesn't get updated).
Example:
$ curategpt extract-from-pubmed -c ont_hp -o temp/ 1887852
/home/harry/curate-gpt/.venv/lib/python3.10/site-packages/torchvision/io/image.py:13: UserWarning: Failed to load image Python extension: '/home/harry/curate-gpt/.venv/lib/python3.10/site-packages/torchvision/image.so: undefined symbol: _ZN3c1017RegisterOperatorsD1Ev'If you don't plan on using image functionality from `torchvision.io`, you can ignore this warning. Otherwise, there might be something wrong with your environment. Did you have `libjpeg` or `libpng` installed before building `torchvision` from source?
  warn(
/home/harry/curate-gpt/.venv/lib/python3.10/site-packages/pydantic/_internal/_fields.py:161: UserWarning: Field "model_name" has conflict with protected namespace "model_".

You may be able to resolve this warning by setting `model_config['protected_namespaces'] = ()`.
  warnings.warn(
Traceback (most recent call last):
  File "/home/harry/curate-gpt/.venv/bin/curategpt", line 6, in <module>
    sys.exit(main())
  File "/home/harry/curate-gpt/.venv/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/home/harry/curate-gpt/.venv/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/home/harry/curate-gpt/.venv/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/harry/curate-gpt/.venv/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/harry/curate-gpt/.venv/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/home/harry/curate-gpt/src/curate_gpt/cli.py", line 778, in extract_from_pubmed
    ao = agent.extract(text, rules=rule, **filtered_kwargs)
  File "/home/harry/curate-gpt/src/curate_gpt/agents/dase_agent.py", line 106, in extract
    for obj, _, _obj_meta in self.knowledge_source.search(
  File "/home/harry/curate-gpt/src/curate_gpt/store/chromadb_adapter.py", line 336, in search
    yield from self._search(text=text, **kwargs)
  File "/home/harry/curate-gpt/src/curate_gpt/store/chromadb_adapter.py", line 351, in _search
    yield from self.diversified_search(
  File "/home/harry/curate-gpt/src/curate_gpt/store/chromadb_adapter.py", line 467, in diversified_search
    if len(text) > self.default_max_document_length:
TypeError: object of type 'NoneType' has no len()

Aha, the issue is that the fetch_full_text is run with the input PMIDs but is expecting a PMC ID (without the PMC prefix).
So it fails when a PMC doesn't exist, which it usually doesn't for an arbitrary PMID.


Code should take precedence over environment variables

The OTEL_EXPORTER_OTLP_HEADERS environment variable and its variants will clobber headers set in the code config. For example, see grpc setup or http setup.
The OTEL_EXPORTER_OTLP_ENDPOINT on the other hand, will prefer the code config, and fall back to the environment variable here.
They should probably at least behave consistently. Thoughts?

[x] This only affects the JavaScript OpenTelemetry library

OTEL_EXPORTER_OTLP_HEADERS can be merged with code defined headers where as the OTEL_EXPORTER_OTLP_ENDPOINT can be merged with the code defined one, hence this behavior. Or maybe i misunderstood what you said ?

The environment headers do get merged with the config ones, but the merge will also overwrite any duplicate header keys.
For example, if I define {foo: bar} in code config, and then set HEADERS:foo=baz, then the exporter will use baz (environment > code config). Whereas the endpoint has the opposite preference (code config > environment > default endpoint).
Does that make sense? I'm wondering if we should switch up the merge, so the environment headers get merged first, followed by code config headers. That way, if there is the same header specified in env and code, code will take precedence.

In the case that headers are duplicated in code and env, the code should take precedence. We have had this come up in other places before too, just can't remember exactly where.

This is not stale

This should probably be reopened

Looks to me like #4334 covered part of this, and we want a few more. Perhaps we can make this into a checklist to make it easier to see what still needs to get done?

[x] opentelemetry-exporter-metrics-otlp-grpc (#4334)
[x] opentelemetry-exporter-metrics-otlp-proto  (#4334)
[x] opentelemetry-exporter-metrics-otlp-http  (#4334)
[ ] opentelemetry-exporter-traces-otlp-grpc
[ ] opentelemetry-exporter-traces-otlp-proto
[ ] opentelemetry-exporter-traces-otlp-http
[ ] opentelemetry-exporter-logs-otlp-grpc
[ ] opentelemetry-exporter-logs-otlp-proto
[ ] opentelemetry-exporter-logs-otlp-http

I think this is the current list?


use enviroment POD_NAMESPACE as secretNamespace

What type of PR is this?
change hardcode of secretNamespace
/kind feature
What this PR does / why we need it:
Support customize namespace installation.
Which issue(s) this PR fixes:
Fixes #231

Not sure if we should correct hardcode everywhere not only Go code.

/easycla

❌ The commit (b258a4f). This user is missing the User's ID, preventing the EasyCLA check. Consult GitHub Help to resolve.For further assistance with EasyCLA, please submit a support request ticket.

/easycla

/assign @adrianludwin

/easycla

❌ The commit (c1b2bec). This user is missing the User's ID, preventing the EasyCLA check. Consult GitHub Help to resolve.For further assistance with EasyCLA, please submit a support request ticket.

/easycla

/easycla

/easycla

/easycla

/easycla

/wip

/WIP

https://github.com/kubernetes-sigs/hierarchical-namespaces/blob/e9d8bb80e54ecfe735235ce608905e6dfbf757ca/internal/webhooks/webhooks.go#L16-L34
https://github.com/kubernetes-sigs/hierarchical-namespaces/blob/e9d8bb80e54ecfe735235ce608905e6dfbf757ca/internal/hierarchyconfig/validator.go#L450-L468
I found that the two pieces of code provided exactly the same capabilities

@adrianludwin PTAL

I've finished the changes you suggested. Looking forward to your reply.
@adrianludwin PTAL

@adrianludwin PTAL :)

/ok-to-test

It was a pleasure to contribute to the project, and thx for your suggestions :)


undefined reference to `glfwDestroyWindow' in test/glfw/glfw.cpp

Makefile setting issue, where I explicitly set the library path


fix: LIME sometimes return NaN weights

For categorical variables, if the LIME tabular sampler created samples that do not equal the original value for all simulated samples, we end up with an all zero variable, causing NaN weights. The fix is to simply append the original row as an identity sample to the simulated samples, making sure we don't end up with an all zero variable.

/azp run

/azp run

/azp run

/azp run


find ng-container

add a reveal function which would find debug elements of ng-container.
also reveal and find should support look up in ng-container too, although it is a comment element.

[x] ngMocks.crawl(debugElement, callback(currentDebugElement, parentDebugElement?))
[x] ngMocks.reveal and ngMocks.revealAll
[x] (impossible) extend ngMocks.find and ngMocks.findAll to support ng-content nodes ( simply call itself recursively
[x] extend ngMocks.findInstance and ngMocks.findInstances to use ngMocks.crawl
[x] extend ngMocks.formatHtml to support ng-content (simply join childrens' innerHTML)
[x] share all callback functions for ngMocks.crawl

[x] docs about ngMocks.crawl
[x] (impossible) docs about ngMocks.find and ngMocks.findAll
[x] docs about ngMocks.findInstance and ngMocks.findInstances
[x] docs about ngMocks.formatHtml
[x] docs about all callback functions for ngMocks.crawl
[x] docs about ngMocks.reveal and ngMocks.revealAll

[x] check if we can assert html right on a debug element of a TemplateRef
[x] update examples with sources
[x] check e2e examples (can be simplified?)

Fix todo links

v11.8.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


BLT Config Compatibility Broken After RAID: WW2 related update

I updated the game yesterday and magically the "BLT window" on the main menu did not show up and i didn't found the mod options into the config.
When i played a heist the game did not load any of the mods, i double checked that the game directory has the "IPHLPAPI.dll" file.
The DLL was there, so to avoid issues i moved the mods folder to the desktop and copied a fresh "mods" folder. After that the game showed up the BLT options and pop ups, so i tried to move back mod by mod to see what is crashing.
The problem was the old configs of ALL mods, so i have to reconfigure everything.
What was that?
Edit: Yesterday was 1.08, today is 2.1
Note: The MD5 of my "IPHLPAPI.dll/2.1" is:
3799ED2CEDACC4B3611DFC97C547C634

Mod configuration options are saved in the mods/saves directory

Your mods save files were not transferred across form the saves directory in mods.


regreet: init at unstable-2023-02-27

Description of changes
Add ReGreet greetd greeter.
Needs this to remember past sessions per-user and write logs:
  systemd.tmpfiles.rules = [
    "d /var/log/regreet 0755 greeter greeter - -"
    "d /var/cache/regreet 0755 greeter greeter - -"
  ];

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/regreet-package/25977/2


How do you debugged the code?

Hi nickgkan,
Thanks for the good work!
I am curious how you debug the code since we ran the code by a bash script with torchrun, which is a distributed GPU training.
Do you use VScode or Pycharm? Do you run the Python script alone for debugging?
Thank you for your time!
Best,
XP

Hi,
We mainly use ipdb module for debugging, with which we can print out the numerical values of tensors on the fly.  We also use a lot of matplotlib to visualize the 3D points.

Hi twke18,
Thanks for the quick reply, I will take a look at ipdb. I also managed to use Pycharm to debug. I used torch.distributed.run --nproc_per_node=1 --master_port=1234 main_trajectory.py --....
Best,
XP


SharePoint top navigation issue

The scope of the issue is been described in the below link  with screenshots(#6672). The top navigation when your spfx component material ui library. I have attached the spfx file which has the issue. This spfx file is linked to the library package file. Am unable attach library file as the file size is more than 20 mb. Could you please provide me a link to attach my library component.
https://github.com/SharePoint/sp-dev-docs/issues/6672#issue-800570013
WaitingOnMeGrid.zip

Please don't post the project... ZIP's can contain harmful scripts which will prohibit many from downloading... plus, if it's 20MB, sounds like you're including node_modules.
Instead, create a public repo in github on your account and post a link to the original issue #6672 you referenced here to keep things consolidated in one thread. Thanks.

Hi Andrew
Below is my repo link to the library component and WaitingOnMeRequest. They do not contain any node_module folders
https://github.com/shibu-123/LibraryComponent.git
https://github.com/shibu-123/WaitingOnMeRequest.git


Refactoring of mass calculator utilities

This PR contains refactoring and various improvements:

Relocate consensus MassCalculator from kaspa_consensus to kaspa_consensus_core (kaspa_consensus is not accessible from broader framework due to dependencies, so this makes the consensus MassCalculator accessible from all crates)
Rename client MassCalculator functions with suffixes denoting transaction types (client or consensus) i.e. calculate_something_for_transaction() -> calculate_something_for_consensus_transaction()
Change client MassCalculator function names to unsigned_transaction
Introduce dedicated functions for consensus and client transaction mass estimates
Propagate mass field across all transaction data structures
Add missing mass to ITransaction TypeScript interface (so that it is visible when it comes in via RpcTransaction data type)
Fix incorrect use of XPublicOnlyKey in ECDSA address creation (https://github.com/kaspanet/rusty-kaspa/issues/536).

I couldn't find this line in the changes https://github.com/kaspanet/rusty-kaspa/blob/dev/rpc/core/src/wasm/convert.rs#L70


Run failed, Exception: Object reference not set to an instance of an object.

I am using some mock hybrid peptide data and generating the corresponding mzML files for these spectra. I am using a small database of 4 proteins with only ~200 spectra in the mzML file. However, when I try and run NeoFusion, i get the error in the title. Please let me know if there is anything I can provide you with to help out. Thanks

Thanks for reaching out!
Do you know which task it failed on? Were any parameters different than the default?
Would you be willing to share your database and mzml file so I can reproduce the crash? That would be the fastest way for me to get a fix out to you.

Thank you for sharing the files! I was able to reproduce the crash and it looks like an issue with your mzML file. There is a scan entry for spectrum #18, but no spectrum (xml object "binaryDataArrayList"). If I copy in a spectrum from another scan, I'm able to run it just fine. I'm going to close the issue for now, but let me know if you have any other issues/questions!


Theres a translate-y animation triggered when a scrollTo animation is triggered.

When the scrollTo event is triggered in AnimatedCounter.tsx, an unwanted translate-y animation occurs from the bottom. This issue appears to be related to the incorrect setting of the position value, which leads to unnecessary re-renders or state changes. I also think, improper application of containerStyles to the container might be causing layout shifts or animations during scroll.
https://github.com/TuckerMassad/react-animated-counter/assets/42804706/48088fa2-cccb-4ef8-9b55-9c8e7d79a20b

Hi @masrur-mmrs , can you give me some example code so I can reproduce this issue locally?

So the animated counter here is nested in a bunch of components here's the code for it:
 <Popover.Button className="h-full outline-none">           <LocalizedClientLink             className="hover:text-ui-fg-base inline whitespace-nowrap"             href="/cart"             data-testid="nav-cart-link"           >             <AnimatedCounter              value={totalItems}              color="#636a74"              includeDecimals={false}             incrementColor="#000000"             decrementColor="#000000"             fontSize="12px"             containerStyles={containerStyles}             />             {/* {totalItems} */}             <ShoppingCart/>           </LocalizedClientLink>         </Popover.Button>


go: Don't symlink bin-directory but binaries instead to avoid breaking pkgs.symlinkJoin without error-message

Description of changes
Creating a symlink for the whole bin-directory overrides bin in pkgs.symlinkJoin without warning but unexpected issues from user-experience.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Creating a symlink for the whole bin-directory overrides bin in pkgs.symlinkJoin without warning but unexpected issues from user-experience.

Sorry but I don't understand this?

      package = pkgs.symlinkJoin {
        inherit (pkgs.vscodium) pname version;

        name = "vscodium";
        paths = [ pkgs.go pkgs.nil pkgs.nixfmt pkgs.vscodium ];
      };

Try to use this e.g. for vscode.package or as package to install - then you will end up with a folder only containing the two go-executables as the folder is symlinked and not writable. This happens without error-message/failing build.

diff --git a/pkgs/development/compilers/go/1.18.nix b/pkgs/development/compilers/go/1.18.nix
index de74e99d9bb..5490bc1fc59 100644
--- a/pkgs/development/compilers/go/1.18.nix
+++ b/pkgs/development/compilers/go/1.18.nix
@@ -166,7 +166,8 @@ stdenv.mkDerivation rec {
     runHook preInstall
     mkdir -p $GOROOT_FINAL
     cp -a bin pkg src lib misc api doc $GOROOT_FINAL
-    ln -s $GOROOT_FINAL/bin $out/bin
+    mkdir -p $out/bin
+    ln -s $GOROOT_FINAL/bin/* $out/bin
     runHook postInstall
   '';

diff --git a/pkgs/development/compilers/go/1.19.nix b/pkgs/development/compilers/go/1.19.nix
index 0ce8fcc659a..d123c69319e 100644
--- a/pkgs/development/compilers/go/1.19.nix
+++ b/pkgs/development/compilers/go/1.19.nix
@@ -166,7 +166,8 @@ stdenv.mkDerivation rec {
     runHook preInstall
     mkdir -p $GOROOT_FINAL
     cp -a bin pkg src lib misc api doc $GOROOT_FINAL
-    ln -s $GOROOT_FINAL/bin $out/bin
+    mkdir -p $out/bin
+    ln -s $GOROOT_FINAL/bin/* $out/bin
     runHook postInstall
   '';

diff --git a/pkgs/development/compilers/go/1.20.nix b/pkgs/development/compilers/go/1.20.nix
index bd226f32157..3364ea35401 100644
--- a/pkgs/development/compilers/go/1.20.nix
+++ b/pkgs/development/compilers/go/1.20.nix
@@ -158,7 +158,8 @@ stdenv.mkDerivation rec {
     runHook preInstall
     mkdir -p $GOROOT_FINAL
     cp -a bin pkg src lib misc api doc $GOROOT_FINAL
-    ln -s $GOROOT_FINAL/bin $out/bin
+    mkdir -p $out/bin
+    ln -s $GOROOT_FINAL/bin/* $out/bin
     runHook postInstall
   '';

diff --git a/pkgs/development/compilers/go/1.21.nix b/pkgs/development/compilers/go/1.21.nix
index cdd91ab1e63..1c9404faadc 100644
--- a/pkgs/development/compilers/go/1.21.nix
+++ b/pkgs/development/compilers/go/1.21.nix
@@ -158,7 +158,8 @@ stdenv.mkDerivation rec {
     runHook preInstall
     mkdir -p $GOROOT_FINAL
     cp -a bin pkg src lib misc api doc $GOROOT_FINAL
-    ln -s $GOROOT_FINAL/bin $out/bin
+    mkdir -p $out/bin
+    ln -s $GOROOT_FINAL/bin/* $out/bin
     runHook postInstall
   '';

Thanks for reviewing, adjusted according suggestions :)


Show ecDNA context in sample page result table

Add a column to the sample page table, which appears when ecDNA context is reported in the run.json. "NA" for non-ecDNA features.

Jens to update AmpliconClassifier to place this key into the results table for the aggregator.


Update object-detection-model-builder.md

Rewrite
Summary
Describe your changes here.
Fixes #Issue_Number (if available)

files changed


When running with quick_start.yaml with own dataset getting the following error

multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
File "/opt/conda/lib/python3.7/multiprocessing/pool.py", line 121, in worker
result = (True, func(*args, **kwds))
File "/opt/conda/lib/python3.7/multiprocessing/pool.py", line 44, in mapstar
return list(map(*args))
File "/opt/conda/lib/python3.7/site-packages/graphvite-0.1.0-py3.7.egg/graphvite/application/application.py", line 411, in linear_classification
loss.backward()
File "/opt/conda/lib/python3.7/site-packages/torch/tensor.py", line 118, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph)
File "/opt/conda/lib/python3.7/site-packages/torch/autograd/init.py", line 93, in backward
allow_unreachable=True)  # allow_unreachable flag
RuntimeError: CUDA error: CUBLAS_STATUS_EXECUTION_FAILED when calling cublasSgemm( handle, opa, opb, m, n, k, &alpha, a, lda, b, ldb, &beta, c, ldc)
"""
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/opt/conda/bin/graphvite", line 11, in
load_entry_point('graphvite==0.1.0', 'console_scripts', 'graphvite')()
File "/opt/conda/lib/python3.7/site-packages/graphvite-0.1.0-py3.7.egg/graphvite/cmd.py", line 212, in main
commandargs.command
File "/opt/conda/lib/python3.7/site-packages/graphvite-0.1.0-py3.7.egg/graphvite/cmd.py", line 110, in run_main
app.evaluate(**cfg.evaluate)
File "/opt/conda/lib/python3.7/site-packages/graphvite-0.1.0-py3.7.egg/graphvite/util.py", line 95, in wrapper
result = function(*args, **kwargs)
File "/opt/conda/lib/python3.7/site-packages/graphvite-0.1.0-py3.7.egg/graphvite/application/application.py", line 101, in evaluate
result = getattr(self, func_name)(**kwargs)
File "/opt/conda/lib/python3.7/site-packages/graphvite-0.1.0-py3.7.egg/graphvite/application/application.py", line 261, in node_classification
results = self.gpu_map(linear_classification, settings)
File "/opt/conda/lib/python3.7/site-packages/graphvite-0.1.0-py3.7.egg/graphvite/application/application.py", line 159, in gpu_map
results = pool.map(func, settings, chunksize=1)
File "/opt/conda/lib/python3.7/multiprocessing/pool.py", line 268, in map
return self._map_async(func, iterable, mapstar, chunksize).get()
File "/opt/conda/lib/python3.7/multiprocessing/pool.py", line 657, in get
raise self._value
RuntimeError: CUDA error: CUBLAS_STATUS_EXECUTION_FAILED when calling cublasSgemm( handle, opa, opb, m, n, k, &alpha, a, lda, b, ldb, &beta, c, ldc)

application:
graph
resource:
gpus: []
cpu_per_gpu: 2
dim: 128
format:
delimiters: " "
comment: "#"
graph:
#file_name: '/home/shraban/Paper2/LINE/cora_edge_list.txt'
file_name: '/home/shraban/Paper2/LINE/demo_list.txt'
as_undirected: true
build:
optimizer:
type: SGD
lr: 0.025
weight_decay: 0.005
num_partition: auto
num_negative: 1
batch_size: 7 # must be greater than numbe of nodes
episode_size: 2
train:
model: LINE
num_epoch: 10
negative_weight: 2
augmentation_step: 2
random_walk_length: 2
random_walk_batch_size: 2
log_frequency: 10
evaluate:
task: node classification
#file_name: '/home/shraban/Paper2/LINE/cora_labels.txt'
file_name: '/home/shraban/Paper2/LINE/demo_labels.txt'
portions: [1]
times: 1
save:
file_name: line_cora.pkl


Border highlighting starts in wrong cell

Should start in leftmost or topmost cell of a block, but starts in the currently highlighted cell instead.  Also, highlighting should be solid, not 50% transparent, so it shows up better.

fixed it!


consider rushstack/eslint-patch

https://github.com/microsoft/rushstack/blob/master/stack/eslint-patch/README.md

clearly not needed anymore


Refactor autoskip functionality into a separate method

This is the first step towards improving the auto-skip functionality. I haven't attempted to make any functionality changes in this PR. My only goal here is to refactor the functionality into a self-contained method as a starting point towards further improvements.
I also haven't moved the calculation to happen outside of draw. I'm not confident I know this section of the code well enough yet to do that safely and would prefer to save that for later when I have a better understanding and am sure the change is safe.

For context, the branch where I played around with tick alignment is https://github.com/chartjs/Chart.js/blob/vertical-tick-alignment/src/core/core.scale.js. There are some changes in calculateTickRotation. There are also some changes to try and work on the auto-skipper. It's definitely not production ready, but could be a useful data point


Update to use v1.27.0 Compliance::API of "inspec" gem.

Description
v1.27.0 of Compliance::API#version only takes one parameter, this cookbook is using the API of v1.24.0 of Compliance::API#version which used to take two parameters.
Update to use v1.27.0 Compliance::API of inspec gem.
Issues Resolved
#232
Check List

[X] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[N/A] New functionality includes testing. (bug fix)
[N/A] New functionality has been documented in the README if applicable (bug fix)
[X] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Rebase is covered in #251


Slight adjustments to the wording

Slight adjustments to the wording of some tasks.
Additionally, added Cleanup manual to get rid from all of the resources after course completion.

@kustikov112 Great corrections with lots of details! thanks a lot!


user/v-danwesley/writer pass on: "Update a Microsoft Edge extension"

run Acrolinx and do additional edits, made minor format changes

Learn Build status updates of commit da7c592:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/extensions-chromium/publish/update-extension.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@Reezaali meant to request a merge, not a review

Learn Build status updates of commit bb742a2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/extensions-chromium/publish/update-extension.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Any plan to update to Xcode 7 with latest Swift syntax?

When I switch to Xcode 7-beta, it shows some suggestion to convert my current project to Latest Swift Syntax.
Also, I see there are a lot Syntax change in the new version like this below:
Before conversion
private subscript(#index: Int) -> JSON {
After conversion
private subscript(index: Int) -> JSON {

+1

+1

+1

+1

try xcode7 branch

Worked for me with one warning: stringByAddingPercentEscapesUsingEncoding' was deprecated in Apple watchOS 2.0: Use -stringByAddingPercentEncodingWithAllowedCharacters: instead, which always uses the recommended UTF-8 encoding, and which encodes for a specific URL component or subcomponent since each URL component or subcomponent has different rules for what characters are valid.

@tangplin Any plan to merge back to mater? Xcode7 branch works fine.

with latest Xcode beta3, there are many errors.

@coolgeng I guess when Xcode7 is no longer in beta

@halgatewood did fix the warning or ignore it?

branch Xcode7，try it！
发自我的 iPhone

在 2015年7月9日，20:44，G Alexander<EMAIL_ADDRESS>写道：
@halgatewood did fix the warning or ignore it?
—
Reply to this email directly or view it on GitHub.

anyway to use cocoapods with Xcode 7 branch? Or we have to manually put the files in the project?

I can't download beat3 yet in China!
发自我的 iPhone

在 2015年7月9日，07:33，G Alexander<EMAIL_ADDRESS>写道：
with latest Xcode beta3, there are many errors.
—
Reply to this email directly or view it on GitHub.

This is just in beta. Don't use beta xcode Formal Environment！

在 2015年7月9日，07:33，G Alexander<EMAIL_ADDRESS>写道：
with latest Xcode beta3, there are many errors.
—
Reply to this email directly or view it on GitHub.

If Xcode 7 do release ！
发自我的 iPhone

在 2015年6月16日，09:52，cooper<EMAIL_ADDRESS>写道：
@tangplin Any plan to merge back to mater? Xcode7 branch works fine.
—
Reply to this email directly or view it on GitHub.

@gaming-hacker that cleared up the warnings for me. Thanks!

has anyone gotten this to run with Xcode 7beta3?
i have a simple file for testing
{
"scaleFactorDB": "3.0",
"offsetDB": "0.0",
"diameter": "100.0",
"font-size": "7",
"font-color": "#ffcc33",
"sun-color": "#ff9900",
"edge-color": "#ff0000",
"line-color": "#ffdb01",
"filename": "Huck_TV_sunview_data.json"
}
then use
let jsonData = NSData(contentsOfFile: filePath)!
print(jsonData)

<7b0a2020 22736361 6c654661 63746f72 4442223a 2022332e 30222c0a 2020226f 66667365 74444222 3a202230 2e30222c 0a202022 6469616d 65746572 223a2022 3130302e 30222c0a 20202266 6f6e742d 73697a65 223a2022 37222c0a 20202266 6f6e742d 636f6c6f 72223a20 22236666 63633333 222c0a20 20227375 6e2d636f 6c6f7222 3a202223 66663939 3030222c 0a202022 65646765 2d636f6c 6f72223a 20222366 66303030 30222c0a 2020226c 696e652d 636f6c6f 72223a20 22236666 64623031 222c0a20 20226669 6c656e61 6d65223a 20224875 636b5f54 565f7375 6e766965 775f6461 74612e6a 736f6e22 0a7d>
let json = JSON(data: jsonData)
print(json)

unknown

i tested my fix and it works well for my data

+1

+1

+1

It's done
:beers:

Cool !!!

Thank you!


Default queue for tasks does not match celerys default queue

I'm not sure if this is intentional, but when I first added the application to my project no tasks were dequeued. This is due to task decorators defaulting the queue to 'default'. Celerys default queue is in fact 'celery'.

That is true. You can either change the default queue name from the settings by adding:
PUSHY_QUEUE_DEFAULT_NAME = 'celery'

or start your workers on the default queue.

That is true, however unless you had a specific reason behind the decision to default to 'default' I would suggest rather letting celery choose the default queue name by simply using None in the getattr calls instead of 'default'.
I think most people would expect the application to work without having to think about choosing the right queue for the tasks, a casual user of celery wouldn't even begin to bother with setting up multiple queues. Some might even be unaware that the queueing feature of Celery even exists.

hmm... good point. I'll re-open the issue, fix this and push a pip update today.


Bug: Top Railing geometry not sent from Revit

What package are you referring to?
Revit Connector
Describe the bug
Revit connector doesn't send top railing from Revit. See GIF below👇
To Reproduce

Create a railing in Revit
Select it and send it to Speckleverse
Investigate it in the viewer or receive it in another software

Expected behavior
I was expecting it to send it.
Screenshots

I think this is due to 'TopRail' elements aren't supported at the moment when converting to Speckle objects.

Hey @Ibrahim5aad ,
Thanks for looking into this :) If the TopRail and HandRail are hosted elements, then adding this might be enough to get them through:
https://github.com/specklesystems/speckle-sharp/blob/main/Objects/Converters/ConverterRevit/ConverterRevitShared/Partial Classes/ConvertWall.cs#L231

Hey @teocomi,
I'm glad I could help, I just started looking into Speckle and I'm looking forward to help on the open issues.
But I'm afraid this won't do it GetHostedElements expects an element that is is/inherits from HostObject which is not the case with Railing elements. The relationship between Railing element associated TopRail can be checked by the TopRail property of the Railing element. And for the HandRail elements can be found with GetHandRails method.
This can help with the detection of those hosted elements. But for the conversion to Speckle object, a support for those elements has to be added to the ConvertToSpeckle method.


AssertionError

Getting an AssertionError when attempting to run unityextract on an .assetbundle
py unityextract No_1.assetbundle

Traceback (most recent call last):
  File "unityextract", line 146, in <module>
    main()
  File "unityextract", line 143, in main
    exit(app.run())
  File "unityextract", line 53, in run
    bundle = unitypack.load(f)
  File "C:\Users\XXX\AppData\Local\Programs\Python\Python36-32\lib\site-packages\unitypack-0.6.1-py3.6.egg\unitypack\__init__.py", line 11, in load
    return env.load(file)
  File "C:\Users\XXX\AppData\Local\Programs\Python\Python36-32\lib\site-packages\unitypack-0.6.1-py3.6.egg\unitypack\environment.py", line 21, in load
    ret.load(file)
  File "C:\Users\XXX\AppData\Local\Programs\Python\Python36-32\lib\site-packages\unitypack-0.6.1-py3.6.egg\unitypack\assetbundle.py", line 44, in load
    assert self.signature in (SIGNATURE_RAW, SIGNATURE_WEB), self.signature
AssertionError

Looking at the docs, it seems this utility only supports .Unity3D bundles. Are .assetbundle files not supported?

Can you upload the file somewhere?

I am running into the same thing. Here is the file I am trying to extract:
sharedassets11.zip

I ran into the same thing was able to resolve it with Asset.from_file.  For example:
import unitypack.asset

with open("example.assets", "rb") as fp:
    asset = unitypack.asset.Asset.from_file(fp)
    for id, object in asset.objects.items():
        print(object.type)

Note: It doesn't know how to handle MonoBehaviour objects, so I had to wrap the ObjectInfo.type getter call in a try block.


Adaptive concurrency rejects should put reason into grpc-message header

https://github.com/envoyproxy/envoy/blob/a8b946e1c30618dfcab6c7e90333301c16b56a65/source/extensions/filters/http/adaptive_concurrency/adaptive_concurrency_filter.cc#L40-L42

I would have thought our gRPC local reply code would already do this, but maybe not. cc @alyssawilk who has been in here recently.

the gRPC code sends the body text as the error message, but for adaptive concurrency that's "" so not very informative.
Given I've been playing in this space it should only be a 10 minute fix for me - happy to throw something together.

Ah OK I missed that. I thought it would potentially be the reason details string. I wonder if it should be the reason details string?

Move gRPC to sending response code details as the body?  I don't have strong feelings either way, but it's a larger change and it strikes me as weird to send different information in the non-gRPC body than we do in the gRPC header.  Then again the two are often similar, it's just that details are more optimized for logging (terse, avoid spaces) and bodies are optimized for human readability.
cc @lizan because gRPC

My thinking was mostly that the grpc-message is probably expected to not be very long, but not a big deal.
I guess my higher level comment is that it seems strange that in the common case of the body/details being the same (I'm guessing this happens a lot) the programmer has to duplicate it. I wonder if there is a cleaner way of handling this common case.

Great! Thanks @alyssawilk!


Change name of "Open Visualizer" -> "Open Visualizer in New Tab"

Open to a discussion. Would it be more clear for the dropdown to say Open Visualizer in new tab to be more consistent with the other command saying open visualizer to the side?

Closing, waiting for user feedback on this