SPI Error

Hi i2Group,
When I run this command:
npm init @i2analyze/connector my-connector
to create a connector, and I try to run the connector, I get this error from i2Analyze:
Failed to parse configuration information for the connector with identifier 'my-connector' from the URL 'http://localhost:3002/config'. The connector with identifier 'my-connector' specified SPI version 1.1, which is not supported by this server. This server supports the following SPI versions: 1.0.
We got this error message on 2 different machines running i2Analyze4.4.0 and i2Analyze4.4.1
So I have the questions:
4.4.0
I am aware the i2Analyze4.4.0 only supports SPI 1.0 so is it possible to change parts of the generated my-connector to erase traces of SPI 1.1?
Also, I'm worried that connectors that were previously developed may become obsolete - will they continue to stay relevant or do we need to somehow upgrade them?
4.4.1
Have you had any incidents of this occurence?

Hi @RaymondVA
I had this issue when trying to do some testing for a customer that required me to create a connector that would work with i2 Analyze 4.4.0.
You need to tell npm to load an older i2connect version  so that you get the older SPI/SDK that your server needs.
If you look in package.json I expect that you will see that is says something like :
"dependencies": {
    "@i2analyze/i2connect": "^2.2.0"
  }

If you alter that to be:
  "dependencies": {
  "@i2analyze/i2connect": "2.1.4"
  }

And then run
npm install
again  it should remove the newer version and get the older compatible one you need.
As for your other questions they are probably best answered by someone who is currently developing this area of things @Jonny-Cornwell-i2  knows this stuff inside out :-)
Cheers

Hi Anthony,
Thanks so much for the help. I have confirmed that it works.
It seems like I'll just wait for @Jonny-Cornwell-i2 to help with the other questions.
Once again thanks!
:)

Hi Anthony,
I tried doing this without doing anything with the example files that get generated from the npm init @i2analyze/connector my-connector
command and it worked.
However, when I start adding extra .ts files, I get the same error message again. It seems like when I run npm start, in package.json it will change back to
"dependencies": {
"@i2analyze/i2connect": "^2.2.0"
}
even though I keep changing it to 2.1.4 and running npm start.

Hi Anthony,
Just to update you, I noticed that it is when I add new dependencies in the package.json and then run npm install, it will ask me to run npm audit fix --force.
When I run npm audit fix --force, this is what causes package.json to automatically change to "@i2analyze/i2connect": "^2.2.0".
Of course I tried to change it back to 2.1.4, run npm install, then it will prompt me with npm audit fix --force again (which changes it back to ^2.2.0)

Hi Raymond
I am glad that we were on to the right track with what I showed you to fix this initially.
Thank you for the extra information you have now provided regarding this being reset to the newer version.
I have spoken directly with @Jonny-Cornwell-i2  who is now aware of your posting here.
Cheers

Hi Raymond,
I believe the underlying cause of you being prompted to to run "npm audit fix" is because of xml2js. This is a development dependency package for the @i2anlyze/i2connect package which is used to process the xml schemas and subsequently auto generate a typescript file. I suspect if you run npm audit it will take you to the following advisory https://github.com/advisories/GHSA-776f-qx25-q3cc. Given the nature of the advisory and how it is used internally, i.e. to read schema xml files generated from schema designer it will not cause issues, assuming this is the only advisory listed when you run npm audit I would suggest it is tolerable. However, I will look to publish a patch release 2.1.5 which increments the minimum required version of xml2js.
For reference the reason the @i2anlyze/i2connect package is not auto updating to the latest minor version of xml2js to >=0.5.X is due to special behaviour regarding npm packages with a major version of 0.
I would note if you are also using xml2js <0.5.0 as a runtime dependency I would suggest it is upgraded and/or the advisory is assessed in the context of your code.
Kind Regards,
Jonny

2.1.6 has been published which should resolve the advisory, if you could update to using this and let us know if the advisory is resolved that would be great and we can close the issue.
Cheers,
Jonny

Hi Jonny,
Thanks for publishing the fix for 2.1.6.
The following is the output that I got when doing npm install and npm audit.
**npm install
npm WARN deprecated<EMAIL_ADDRESS>Deprecated in favor of https://github.com/deepsweet/start
added 141 packages, changed 1 package, and audited 402 packages in 27s
94 packages are looking for funding
run npm fund for details
4 moderate severity vulnerabilities
Some issues need review, and may require choosing
a different dependency.
Run npm audit for details.
PS C:\VSC WORKSPACES\AU_NSW_NSWPOL\test> npm audit
npm audit report
semver  <7.5.2
Severity: moderate
semver vulnerable to Regular Expression Denial of Service - https://github.com/advisories/GHSA-c2qf-rxjj-qqgw
No fix available
node_modules/nodemon/node_modules/semver
node_modules/simple-update-notifier/node_modules/semver
nodemon  1.4.10-alpha.1 - 1.4.10-alpha.3 || >=1.14.10
Depends on vulnerable versions of semver
Depends on vulnerable versions of simple-update-notifier
node_modules/nodemon
@i2analyze/i2connect-scripts  *
node_modules/@i2analyze/i2connect-scripts
simple-update-notifier  >=1.0.7
Depends on vulnerable versions of semver
node_modules/simple-update-notifier
4 moderate severity vulnerabilities
Some issues need review, and may require choosing
a different dependency.**
Despite the output of npm audit, the connector works.
Cheers,
Raymond.

Glad it is all sorted, I would note that deepsweet is not a package used by the @i2analyze/i2connect-scripts or @i2analyze/i2connect as far as I know or can see.
It looks like the semver advisory is relatively new and I can see a PR has been made to nodemon to fix this (https://github.com/remy/nodemon/pull/2118), so once that is merged it should be able to be fixed by npm audit fix soon.
Cheers

Closing as this now seems to be resolved.
Cheers


SPIKE: Chunk and/or group rewards distribution

Reporter @goral09

Moving this over to @Fraser999 for similar reasons as it's blocker: this became a group discussion and some people are writing it up, rather than an individual spike.


Implement Whirlpool

ISO/IEC 10118-3:2018 => https://www.iso.org/standard/67116.html
https://webstore.iec.ch/preview/info_isoiec10118-3{ed4.0}en.pdf

https://github.com/veracrypt/VeraCrypt

https://github.com/jzelinskie/whirlpool


Can't build with ant

I'm trying to build the SWC with ant but ti's failing on the compc task with an error:
Error: Unsupported file type: C:\development\libraries\robotlegs-framework.git\src\manifest.xml

Hey Kamcknig, I've had a got at compiling this source the other day. (I see that this message is a few months late). And it ran correctly once I had set the paths correctly. I had to make sure my JAVA_HOME was a JDK version (not JRE) and pointed to a correct flex sdk.
My output was below, what is your full output log?
C:\Users\James\Documents\Testlab\Flash\robotlegs-framework>ant compile
Buildfile: C:\Users\James\Documents\Testlab\Flash\robotlegs-framework\build.xml

clean:
     [echo] [clean] Removing Build, Report and Doc directories
     [echo] [clean] Build, Report and Doc directories removed

init:
     [echo] [init] Creating Bin, Report and Doc directories
     [echo] [init] Bin, Report and Doc directories created

compile:
     [echo] [compile] Compiling release SWC
     [echo] [compile] Using Flex SDK at: C:\Users\James\AppData\Local\FlashDevel
op\Apps\flexairsdk\4.6.0+17.0.0
     [java] Loading configuration file C:\Users\James\AppData\Local\FlashDevelop
\Apps\flexairsdk\4.6.0+17.0.0\frameworks\flex-config.xml
     [java] Warning: The swc 'C:\Users\James\AppData\Local\FlashDevelop\Apps\fle
xairsdk\4.6.0+17.0.0\frameworks\libs\framework.swc' has style defaults and is in
 the library-path, which means dependencies will be linked in without the styles
.  This can cause applications, which use the output swc, to have missing skins.
  The swc should be put in the external-library-path.
     [java]
     [java] C:\Users\James\Documents\Testlab\Flash\robotlegs-framework\bin\robot
legs-framework-v2.2.1.swc (130254 bytes)
     [echo] [compile] Release SWC robotlegs-framework-v2.2.1.swc created success
fully

BUILD SUCCESSFUL
Total time: 2 seconds


Adjust boundary data

A few related tasks:

[ ] Include Mayoral Combined Authorities (and another -- maybe LAs and not LADs? Re-reading Teams chats still)
[ ] Fill in missing LADs (see below)
[ ] Reduce coordinate precision in the GJ file to shrink it
[ ] Add optional layers to scheme browsing page to show different authority boundaries
[ ] Revisit using convex hull around boundaries to ensure the street network is connected for route-snapper

The missing LAD problem:

ATIP is using https://github.com/acteng/atip/blob/main/assets/authorities.geojson. This file comes from https://github.com/acteng/schema/blob/main/boundaries/lads_tas_combined.geojson, processed by https://github.com/acteng/abstreet-to-atip/tree/main/fix_boundaries. Checking another repo, I see https://github.com/acteng/boundaries/blob/main/lads.geojson, which looks complete to me. Should we use that file along with https://github.com/acteng/boundaries/blob/main/transport_authorities.geojson and ignore that old combined GJ file?

Revisiting https://github.com/acteng/abstreet-to-atip/issues/4#issuecomment-1419158941, checking different versions of the LPAs from OS...
BFC version (https://geoportal.statistics.gov.uk/datasets/ons::local-planning-authorities-april-2022-uk-bfc/explore?location=50.835262%2C-1.072888%2C15.49) has the same problem:

BGC also won't work: https://geoportal.statistics.gov.uk/datasets/ons::local-planning-authorities-april-2022-uk-bgc-3/explore?location=50.806798%2C-0.966544%2C12.44
Or BUC: https://geoportal.statistics.gov.uk/datasets/ons::local-planning-authorities-april-2022-uk-buc-3/explore?location=50.838586%2C-1.066440%2C12.95
But BFE (https://geoportal.statistics.gov.uk/datasets/ons::local-planning-authorities-april-2022-uk-bfe-3/explore?location=50.830575%2C-1.076023%2C13.60) looks good; it's just one polygon for Portsmouth! I think we need to switch to this geometry

Big +1 to this. My first impression is that just being able to view the boundaries would be enough.

Checking another repo, I see https://github.com/acteng/boundaries/blob/main/lads.geojson, which looks complete to me. Should we use that file along with https://github.com/acteng/boundaries/blob/main/transport_authorities.geojson and ignore that old combined GJ file?

Yes but the file needs to be updated based on this newer LAD level data. Everything can be built from LADs.

https://geoportal.statistics.gov.uk/datasets/ons::local-authority-districts-may-2023-uk-bfc-2/explore

After we change the boundaries file, I'll regenerate all route-snapper and route-info files. It's a slightly costly process, so let's batch together any changes and confirm them before proceeding. One concern is about backwards compatibility: any authority names defined by ATIP files built up to this point are based on the old boundaries. So if the newer files change anything substantial, we should understand it.
I'm still trying to detangle all the definitions of things from the chat yesterday... today we have:

LADs, which could come from https://geoportal.statistics.gov.uk/datasets/ons::local-authority-districts-may-2023-uk-bfc-2/explore
TAs, which are composed of multiple LADs using a table in https://github.com/acteng/boundaries

And Chris linked to LPAs; the BFE version of that fixes the multi-polygon problem. Today we don't have LPAs; do we want them as well?
I can't find MCAs defined on that geoportal site either.

Today we don't have LPAs; do we want them as well?

Yes I think so, a relevant boundary dataset.

Reading through https://geoportal.statistics.gov.uk/documents/boundary-dataset-guidance-2011-to-2021/explore and https://www.ons.gov.uk/methodology/geography/geographicalproducts/digitalboundaries closely... we want "extent of the realm" to avoid things like the Portsmouth issue. But the only variation available with this is BFE, which is full resolution. Just LADS with trimmed coordinate precision winds up 90MB. Do we have to make our own generalized extent of the realm variations?

If we want to revisit this and also fix #345, here's a new writeup. Ignore the rest of this issue; it was confusing other work for the browse page. This is just about the boundaries used on the start page for the sketch tool. It has:

TAs, which are composed of multiple LADs using a table in https://github.com/acteng/boundaries
LADs,  from a place like https://geoportal.statistics.gov.uk/datasets/ons::local-authority-districts-may-2023-uk-bfc-2/explore

There are a few problems:

LADs that happen to share the same name with a TA are skipped. I'm not sure that's appropriate; we should include all of them? Or if the boundaries are exactly the same for those cases, we could mark it in the geojson as "both" with some property.
In London and Portsmouth and other places, the boundary is clipped to the coastline in ways that break the route tool. We need one simple polygon covering the whole area. Previously we've tried to fix this by taking a convex hull of the input, but it makes the boundaries overlap with each other and look messy, and it doesn't work for the Thames in London. I think we want "extent of the realm" variations, but also to keep resolution low or down-scale in order to keep file sizes small.

Consolidating in #345


not run these processes with webcam

do we need uploader to run in webcam mode?
don't need ubloxd since no gps

Why no GPS, should totally work with black panda?
And sure, why not re: uploader and logger. I don't think the camera logging works on PC, but the qlog/rlog should, and there's no reason to not get them.

Why no GPS, should totally work with black panda?
And sure, why not re: uploader and logger. I don't think the camera logging works on PC, but the qlog/rlog should, and there's no reason to not get them.
Also, the less we diverge from OP on comma two, the better.

Oh yeah, I was thinking webcam even includes working with simulation; but since we run monitoring too with webcam so let's assume it is in a real car. Sorry for the confusion
Let's also keep the logs then; they will be interesting for comparing performance!


nixos/test-driver: actually use the backdoor message to wait for backdoor

New EDK2 sets up the backdoor port as a serial console, which feeds the test driver a bunch of boot logs it can safely ignore. Do so by waiting for the message the backdoor shell prints before doing anything else.
Description of changes

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)
[x] 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.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.

Both boot.biosCdrom and boot.uefiCdrom tests work now/


Gap between tiles for layers using the same tileset

Version

Phaser Version: 3.16.2

Tiled: 1.2.2

Operating system: Windows 10

Browser: Google Chrome 73.0.3683.86

Description
When two tilemap layers use the same tileset image, and they placed under each other, there appear gaps on the above layer when the camera moves.
Example Test Code
https://jsfiddle.net/Trobing/vwyebxcu/43/
In the example I have created 4 layers in Tiled, with 3x3 tile squares, the light square is under dark square: two layers that use the one common tileset (on the left), and two layers that use the separated tileset (on the right). To see the issue move to the right-bottom direction through the cursor keys.
Additional Information
I have extruded tilesets through TexturePacker on 2px.
In Firefox and Opera it seems ok - the issue appears only on Chrome with enabled hardware acceleration, with WebGL renderer and enabled Phaser antialiasing. In JSFiddle it also requires to zoom out the camera, but in my game on the local server, there are gaps (but they smaller and rare in this case) even without zoom changing.

P.S.: Only on the screenshot, I have noticed that right square also have some artifacts on corners.

Do you have a clearer visual? I'm struggling to see anything really.

@photonstorm , Well, it's strange but it's actual just if it's using Nvidia card, and it looks fine with integrated Intel card. So I suppose it's not a Phaser issue.
But recently I notice another issue with Firefox and canvas renderer - in this case, the gaps appear when it's using zoom on camera. https://jsfiddle.net/Trobing/vwyebxcu/48/

P.S: I have checked it with both GPU))

I had someone report something similar in tile-extruder not that long ago. When I tested using extruded tiles, there were no gaps in Chrome under CANVAS or WEBGL, but gaps were present in FF and Edge in CANVAS. At least part of the issue here seems to be how the canvas drawImage is implemented in each browser. It appears that if you draw tiles at floating point positions / float point scales with canvas in FF/Edge, the anti-aliasing filter that gets applied to each tile causes the background to bleed through the tiles.
This happens with the raw canvas API in FF/Edge:

Here's a codesandbox with a demo and a possible fix that I created (from the tile-extruder ticket). It's showing the issues with the raw canvas API. If you draw your tiles to an offscreen buffer and then draw that to the screen, it eliminates the FF/Edge issue in my simple test. That approach might work in Phaser's renderer.

@mikewesthad Thank you! I'm just considering such workaround with converting tiles layer to texture.

See this alternative for a different approach:
Render Layer as Texture


Fix race condition in LifoBlockingLimiter

If a token was acquired around the same time that an operation
timed out, then the token would get lost. That means that the
limiter's capacity would effectively be reduced by one. If
this happens enough times, or the limit is decreased enough
for other reasons, then this would cause to the limiter to
permanently not let any operations through anymore.

The tests passed locally for me, but apparently unblockWhenFullBeforeTimeout failed in CI.
Although it's a test for behaviour close to the code I changed here, I don't think the test failure indicates a bug, it looks like a flaky test to me. I suppose that when something is scheduled to run in 250ms in one thread, then another thread may perceive that as having waited a bit less than 250ms.
(Other tests in LifoBlockingLimiterTest tend to fail too when I run them repeatedly, on master.)
I pushed a commit to make that test hopefully less fragile.

For context:
We're using LifoBlockingLimiter in a new system we're building at Block, that currently acquires tokens at around 10k QPS across a handful of nodes.  I confirmed that this race condition was the cause of our problems by temporarily adding some logging in the place I patched. From that logging I see we ran into this a couple times per day, mostly in bursts, e.g. dropping 10 tokens at a time. It sometimes lead to such a node getting completely stuck (the LifoBlockingLimiter not letting anything through anymore); not immediately after hitting the race condition, but later when the limit dipped below the number of tokens that got lost.

Thanks for that context. We recently got an internal report of similar behavior so this is very well timed. @pandeyabhi1987 is investigating that, so I have added him to take a closer look.


Send email when a comment is added.

If verifier comments on a word then that comment info should be email to the respective corrector, and same should go the other way around.

Have completed on the sendMail branch on commit number :-> 849d5df
Testing is remaining...

Fetching the comment mail credentials from the database and saving the user email into the settings on commit number :-> a53ed05

Added dependencies for sending mail for windows at commit number ->0bab511a5f24e51dfb8364c83e74042cac02b421 , 7d1e62a80e3aa8d46fe012494556a3dab1e3cb38, 86b4e47c6c3730e9dc193a16481166dc1b68df35
Improved the comment mail feature at commit number :-> fc53f44c8e5478fd35a757c27ac8e664904e9dc2

Fixed a small bug in the email format on commit number :- 43a552fe70d877c94285fd5d97ead26880b961db


Hybrid Could not find route

Not correct typeof

Can you please share a bit more details on:

How do you call the bus.Send<T>(T msg)? Do you pass the IRequestMessage<PlayerReconnectRespoins> as `T?
Your hybrid bus setup around registering the request messages?

call _bus.Send(new PlayerReconnectRequest())
yeap, i registred producer at hybrid bus

.Produce<PlayerReconnectRequest>(x => x.DefaultTopic(x.Settings.MessageType.Name))

public class PlayerReconnectRequest : IRequestMessage<PlayerReconnectResponse>
{
}

@zarusz try change here https://github.com/zarusz/SlimMessageBus/blob/master/src/SlimMessageBus.Host.Hybrid/HybridMessageBus.cs#L95
to
var messageType = message.GetType();

Yes, you're right @nikes.
Can you please check if version 1.7.3 works for you?
https://www.nuget.org/packages/SlimMessageBus.Host.Hybrid/1.7.3

Test 1.7.3 Work
Thx


Degraded federations for devimint and rust tests

Closes https://github.com/fedimint/fedimint/issues/4171
Inspired by https://github.com/fedimint/fedimint/pull/4173 and https://github.com/fedimint/fedimint/pull/4198
Introduces the ability to run rust and devimint tests with a parameterized number of peers offline, defaulting to a 3/4 setup.
Running all tests with a degraded federation increases the time for CI to finish by a material amount, so we should consider if running all tests degraded is worth the additional overhead.

Build on Linux tests step: before ~10 minutes (CI), degraded ~19 minutes (CI)
Back-compat: before ~45 minutes (CI), degraded ~68 minutes (CI)

dev call: would be nice to merge flakiness offensive, then rebase this one it

Would be good if our devimint wizard @maan2003 could take a look before merging since I'm not as familiar with it :)

Thanks everyone for the reviews! I should be able to address/respond to all feedback in the next day or two. If this is approved/merged in the meantime, I'll make a followup PR.

Rebased and resolved merge conflicts, still need to address feedback 🙂

Dang, reached the 75 minute timeout in the merge queue for the 5x tests. Testing a timeout bump in a separate PR https://github.com/fedimint/fedimint/pull/4342

Dang, reached the 75 minute timeout in the merge queue for the 5x tests. Testing a timeout bump in a separate PR #4342

Wen parallel tests?

Wen parallel tests?

I don't think parallel back-compat would help for the 5x tests, but I think exploring this may help: https://github.com/fedimint/fedimint/pull/4247#discussion_r1496571057

Wen parallel tests?

I don't think parallel back-compat would help for the 5x tests, but I think exploring this may help: #4247 (comment)

Oh. OK. https://github.com/fedimint/fedimint/pull/4390
But also https://github.com/fedimint/fedimint/pull/4390
And then: Wen parallel tests? :D

Capturing an out-of-band chat with @dpc:

merge latency tests refactor https://github.com/fedimint/fedimint/pull/4390
rebase this PR so test-ci-all runs faster and we can merge with 5x tests
I'll address remaining feedback in followup PRs so we have degraded tests in CI sooner

Rebased with the latency parallel improvements

This looks great to me. But I'm a little out-of-context so I'm going to defer "approval" to @dpc @elsirion and @maan2003.

Dangit, backend_test_bitcoind is running much slower in CI than locally and timing out at 600s. Investigating...


Don't generate unit tuple in clone macro as default-return value

Fixes https://github.com/gtk-rs/gtk-rs-core/issues/1102.

looks good to me, but i still believe the actual solution is to not use @default-return at all. Still good to merge though


Target mips-unknown-linux-musl and mipsel-unknown-linux-musl are not statically linked.

Problem
From issue: https://github.com/rust-embedded/cross/issues/504
I find it's an cargo issue.
Steps

download build toolchain from: musl.cc
cargo new cargo-hello && cd cargo-hello
cargo build --target mips-unknown-linux-musl or cargo build --target mipsel-unknown-linux-musl
file target/<target>/debug/cargo-hello

Here is my ~/.cargo/config
[target.aarch64-unknown-linux-musl]
linker = "/home/fengyu/projects/cross_prefix/aarch64-linux-musl-cross/bin/aarch64-linux-musl-gcc"

[target.arm-unknown-linux-musleabi]
linker = "/home/fengyu/projects/cross_prefix/arm-linux-musleabi-cross/bin/arm-linux-musleabi-gcc"

[target.mips-unknown-linux-musl]
linker = "/home/fengyu/projects/cross_prefix/mips-linux-musl-cross/bin/mips-linux-musl-gcc"

[target.mips64-unknown-linux-muslabi64]
linker = "/home/fengyu/projects/cross_prefix/mips64-linux-musl-cross/bin/mips64-linux-musl-gcc"

[target.mips64el-unknown-linux-muslabi64]
linker = "/home/fengyu/projects/cross_prefix/mips64el-linux-musl-cross/bin/mips64el-linux-musl-gcc"

[target.mipsel-unknown-linux-musl]
linker = "/home/fengyu/projects/cross_prefix/mipsel-linux-musl-cross/bin/mipsel-linux-musl-gcc"

[target.x86_64-unknown-linux-musl]
linker = "/home/fengyu/projects/cross_prefix/x86_64-linux-musl-cross/bin/x86_64-linux-musl-gcc"

I find only mips-unknown-linux-musl and mipsel-unknown-linux-musl are not staticlly linked.
$ echo aarch64-unknown-linux-musl arm-unknown-linux-musleabi mips-unknown-linux-musl mips64-unknown-linux-muslabi64 mips64el-unknown-linux-muslabi64 mipsel-unknown-linux-musl x86_64-unknown-linux-musl | xargs -n 1 rustup target add
$ echo aarch64-unknown-linux-musl arm-unknown-linux-musleabi mips-unknown-linux-musl mips64-unknown-linux-muslabi64 mips64el-unknown-linux-muslabi64 mipsel-unknown-linux-musl x86_64-unknown-linux-musl | xargs -n 1 cargo build --target
$ file target/*/debug/cargo-hello
target/aarch64-unknown-linux-musl/debug/cargo-hello:       ELF 64-bit LSB executable, ARM aarch64, version 1 (SYSV), statically linked, with debug_info, not stripped
target/arm-unknown-linux-musleabi/debug/cargo-hello:       ELF 32-bit LSB executable, ARM, EABI5 version 1 (SYSV), statically linked, with debug_info, not stripped
target/mips64el-unknown-linux-muslabi64/debug/cargo-hello: ELF 64-bit LSB executable, MIPS, MIPS64 rel2 version 1 (SYSV), statically linked, with debug_info, not stripped
target/mips64-unknown-linux-muslabi64/debug/cargo-hello:   ELF 64-bit MSB executable, MIPS, MIPS64 rel2 version 1 (SYSV), statically linked, with debug_info, not stripped
target/mipsel-unknown-linux-musl/debug/cargo-hello:        ELF 32-bit LSB pie executable, MIPS, MIPS32 rel2 version 1 (SYSV), dynamically linked, interpreter /lib/ld-musl-mipsel.so.1, with debug_info, not stripped
target/mips-unknown-linux-musl/debug/cargo-hello:          ELF 32-bit MSB pie executable, MIPS, MIPS32 rel2 version 1 (SYSV), dynamically linked, interpreter /lib/ld-musl-mips.so.1, with debug_info, not stripped
target/x86_64-unknown-linux-musl/debug/cargo-hello:        ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), dynamically linked, with debug_info, not stripped

Notes
Output of cargo version: cargo 1.49.0 (d00d64df9 2020-12-05)

Thanks for the report! I don't think that this is a Cargo issue but rather a rustc issue perhaps?

Thanks for the report! I don't think that this is a Cargo issue but rather a rustc issue perhaps?

OK. So I will submit this issue to rust repo.

OK. So I will submit this issue to rust repo.


Getting notified when content has first appeared in the root view

On iOS, you can learn when the RCTRootView first renders content, which is useful for knowing when to fade out a loading screen for example. There currently isn't an equivalent on Android so we're simply polling, which actually works well but it'd certainly be nice to have an event.
For the API one idea is to define a RootView.Listener class that is notified of this event. We can write the implementation if that's helpful.

cc @dmmiller

This can be useful to show a splash screen until React Native does it's thing. Currently it shows only a white screen.

There doesn't seem to be a good event on Android equivalent to what exists on iOS which makes it really hard to implement.  If someone does have a good idea of how to detect it on Android, I'd be happy to look further.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/getting-notified-when-content-has-first-appeared-in-the-root-view
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug, please consider sending a PR with a fix.
We're a small team and rely on the community for bug fixes of issues that don't affect fb apps.


expression check

My colleague wrote the following wrong code
I didn't realize the mistake at the time, just wanted to use it as a ternary expression.
The code execution will not report an error, but it will never get the desired result.
local a = bit.isset(data1, 0) and false or true
local b= bit.isset(data2, 7) and false or true

After the code was sent to me, I saw the code error prompt in VSCODE.
[Warn type:15], or expression is always true
[Warn type:16], and expression is always false
Can luacheck detect such errors?

I am deeply saddened to hear about his passing. He was an exceptional developer and an admirable individual. His contributions and talent will be forever remembered. May he rest in peace.

Please feel free to re-open this issue on the new canonical fork of this repository in the @lunarmodules org.

Never mind, it appears you did, it was just buried in my notifications.
https://github.com/lunarmodules/luacheck/issues/100


Debugging API Key with Seq

We have switched to using the XML way to configure serilog for Seq and we are not sure that the API key is being used as SEQ is not showing our addtional properties we can against each API Key.
We have this in the app.config
    <add key="serilog:using:Seq" value="Serilog.Sinks.Seq" />
    <add key="serilog:write-to:Seq.serverUrl" value="http://localhost:5341" />
    <add key="serilog:write-to:Seq.apikey" value="jQDDtKXXdTFMdUCOtysf" />

It finds the server url so it's partially working at least, is there a way to see if the api key is being sent ?

Its ok, it's case sensitive and I seriously need to get some rest :-)


Missing Pipes [ | ] - OSSEM\data_dictionaries\windows\security\events\event-4647.md

Hi, i've been currently working and studying the ossem, and i saw that there were missing 1 pipe at line 23, is it possible for you to pull the request?

Thank you 👍


Add empovit to OWNERS

Add user empovit as an approver and reviewer to OWNERS.

/ok-to-test
/approve
@fabiendupont Can you please add @empovit to the org?


i am getting this issue?

var value = BarChartDataEntry(values: self.smImg[i], xIndex: i)

error:  Use of unresolved identifier 'smimg'

This is not a Swift school.


Different inferred type when importing - fp-ts

Failing module

GitHub: https://github.com/gcanti/fp-ts
npm: https://www.npmjs.com/package/fp-ts

Inferred type of result is different when using esm.sh & Deno:
import { sequenceT } from "fp-ts/Apply";
import { getApplicativeValidation, right } from "fp-ts/Either";
import { getSemigroup } from "fp-ts/NonEmptyArray";
const f = getApplicativeValidation(getSemigroup<string>());
const result = sequenceT(f)(right(""));

Error message
Inferred type (esm.sh & Deno):
const result: Kind4<F, S, R, E, { [K in keyof T]: [T[K]] extends [Kind4<F, S, R, E, infer A>] ? A : never; }>

Inferred type (tsc):
const result: Either<NonEmptyArray<string>, [string]>

Additional info

esm.sh version: V76
Deno version: deno 1.20.4 (release, aarch64-apple-darwin)

It seems d.ts files are addressing wrong path to HKT.
//<EMAIL_ADDRESS>import { HKT, ... } from './HKT/HKT.d.ts'

Currently it is pointing ./HKT/HKT.d.ts , which gives /* fake(empty) types */.
It should be ../HKT.d.ts


bool flag `--stemcell-heavy' cannot have an argument

starting with om version 7
when downloading stemcells using a download-product.yml
pivnet-api-token: ((pivnet-refresh-token))
pivnet-file-glob: "cf*.pivotal"
pivnet-product-slug: elastic-runtime
product-version-regex: ^2.10..*$
stemcell-iaas: aws
stemcell-heavy: true
om download-product --config download-product.yml --output-directory ./ downloaded-stemcell --source pivnet
2020/11/20 08:37:33 bool flag `--stemcell-heavy' cannot have an argument
this worked fine with version 6.5.0

it works when not using a file with settings like
om download-product -s pivnet -o ./ -p elastic-runtime -r ^2.10..$ --stemcell-heavy --stemcell-iaas=aws -f "cf.pivotal" -t
attempting to download the file product-files/elastic-runtime/cf-2.10.9-build.6.pivotal from source pivnet
cf-2.10.9-build.6.pivotal already exists, skip downloading
Downloading stemcell
attempting to download the file product-files/stemcells-ubuntu-xenial/bosh-stemcell-621.92-aws-xen-hvm-ubuntu-xenial-go_agent.tgz from source pivnet
503.88 MiB / 503.88 MiB [=========================================] 100.00% 11s
calculating sha sum for bosh-stemcell-621.92-aws-xen-hvm-ubuntu-xenial-go_agent.tgz.partial
Writing a list of downloaded artifact to download-file.json
Writing a assign stemcell artifact to assign-stemcell.yml

Thanks for the report, working on getting a fix out shortly.

The fixed for this will be released in 7.0.2, shortly.

@lerytke can you please confirm it works for you? We've tested against, but would like you to confirm and close the issue.

yes, it is fixed now thanks


Update Issue - Automate_undefined_1641

default description

Property
Value

Browser
Chrome 130.0

Operating System
OS X Sequoia

Resolution
1688x792

Screenshot Attached
Screenshot URL

Updated Screenshot Attached


Clean up electron-store based stores

Add some type script types for better editor experience
Remove redundent global of a singleton

Signed-off-by: Sebastian Malton<EMAIL_ADDRESS>This is a cleanup pr before one for fixing #348

deprecated since #585


[BUG]RuntimeError: Step 1 exited with non-zero status 1

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
the official doc
python train.py --actor-model facebook/opt-1.3b --reward-model facebook/opt-350m --num-gpus 1
bug
---=== Running Step 1 ===---
Traceback (most recent call last):
  File "/DeepSpeed/DeepSpeedExamples/applications/DeepSpeed-Chat/train.py", line 218, in <module>
    main(args)
  File "/DeepSpeed/DeepSpeedExamples/applications/DeepSpeed-Chat/train.py", line 203, in main
    launch_cmd(cmd, step_num)
  File "/DeepSpeed/DeepSpeedExamples/applications/DeepSpeed-Chat/train.py", line 192, in launch_cmd
    raise RuntimeError(
RuntimeError: Step 1 exited with non-zero status 1

Please see step1 output log in file: output/actor-models/1.3b/training.log

Maybe Your GPU Out-Of-Memory

hi,dear
@Grypse the bellow
huggingface/tokenizers: The current process just got forked, after parallelism has already been used. Disabling parallelism to avoid deadlocks...
To disable this warning, you can either:
- Avoid using tokenizers before the fork if possible
- Explicitly set the environment variable TOKENIZERS_PARALLELISM=(true | false)
//site-packages/torch/utils/cpp_extension.py:325: UserWarning:
                           !! WARNING !!

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
Your compiler (c++) is not compatible with the compiler Pytorch was
built with for this platform, which is g++ on linux. Please
use g++ to to compile your extension. Alternatively, you may
compile PyTorch from source using c++, and then you can also use
c++ to compile your extension.
See https://github.com/pytorch/pytorch/blob/master/CONTRIBUTING.md for help
with compiling PyTorch from source.
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
                          !! WARNING !!

warnings.warn(WRONG_COMPILER_WARNING.format(
Detected CUDA files, patching ldflags
Emitting ninja build file /data/.cache/torch_extensions/py39_cu116/fused_adam/build.ninja...
Building extension module fused_adam...
Allowing ninja to set a default number of workers... (overridable by setting the environment variable MAX_JOBS=N)
huggingface/tokenizers: The current process just got forked, after parallelism has already been used. Disabling parallelism to avoid deadlocks...
To disable this warning, you can either:
- Avoid using tokenizers before the fork if possible
- Explicitly set the environment variable TOKENIZERS_PARALLELISM=(true | false)
[1/2] c++ -MMD -MF fused_adam_frontend.o.d -DTORCH_EXTENSION_NAME=fused_adam -DTORCH_API_INCLUDE_EXTENSION_H -DPYBIND11_COMPILER_TYPE="_gcc" -DPYBIND11_STDLIB="_libstdcpp" -DPYBIND11_BUILD_ABI="_cxxabi1013" -I/data/soft/anaconda3/envs/deepspeed/lib/python3.9/site-packages/deepspeed/ops/csrc/includes -I/data/soft/anaconda3/envs/deepspeed/lib/python3.9/site-packages/deepspeed/ops/csrc/adam -isystem /data/soft/anaconda3/envs/deepspeed/lib/python3.9/site-packages/torch/include -isystem /data/soft/anaconda3/envs/deepspeed/lib/python3.9/site-packages/torch/include/torch/csrc/api/include -isystem /data/soft/anaconda3/envs/deepspeed/lib/python3.9/site-packages/torch/include/TH -isystem /data/soft/anaconda3/envs/deepspeed/lib/python3.9/site-packages/torch/include/THC -isystem /usr/local/cuda/include -isystem /data/soft/anaconda3/envs/deepspeed/include/python3.9 -D_GLIBCXX_USE_CXX11_ABI=0 -fPIC -std=c++14 -O3 -std=c++14 -g -Wno-reorder -DVERSION_GE_1_1 -DVERSION_GE_1_3 -DVERSION_GE_1_5 -c /data/soft/anaconda3/envs/deepspeed/lib/python3.9/site-packages/deepspeed/ops/csrc/adam/fused_adam_frontend.cpp -D_GLIBCXX_USE_CXX11_ABI=0 -fPIC -std=c++14 -O3 -std=c++14 -g -Wno-reorder -DVERSION_GE_1_1 -DVERSION_GE_1_3 -DVERSION_GE_1_5 -c /deepspeed/lib/python3.9/site-packages/deepspeed/ops/csrc/adam/fused_adam_frontend.cpp -o fused_adam_frontend.o
c++: 错误：unrecognized command line option ‘-std=c++14’
c++: 错误：unrecognized command line option ‘-std=c++14’
ninja: build stopped: subcommand failed.
Traceback (most recent call last):
File "//deepspeed/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 1808, in _run_ninja_build
subprocess.run(
File "//deepspeed/lib/python3.9/subprocess.py", line 528, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['ninja', '-v']' returned non-zero exit status 1.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "//DeepSpeed/DeepSpeedExamples/applications/DeepSpeed-Chat/training/step1_supervised_finetuning/main.py", line 328, in
main()
File "//DeepSpeed/DeepSpeedExamples/applications/DeepSpeed-Chat/training/step1_supervised_finetuning/main.py", line 260, in main
optimizer = AdamOptimizer(optimizer_grouped_parameters,
File "//deepspeed/lib/python3.9/site-packages/deepspeed/ops/adam/fused_adam.py", line 71, in init
fused_adam_cuda = FusedAdamBuilder().load()
File "/deepspeed/lib/python3.9/site-packages/deepspeed/ops/op_builder/builder.py", line 449, in load
return self.jit_load(verbose)
File "//deepspeed/lib/python3.9/site-packages/deepspeed/ops/op_builder/builder.py", line 480, in jit_load
op_module = load(name=self.name,
File "/deepspeed/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 1202, in load
return _jit_compile(
File "//deepspeed/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 1425, in _jit_compile
_write_ninja_file_and_build_library(
File "//deepspeed/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 1537, in _write_ninja_file_and_build_library
_run_ninja_build(
File "/deepspeed/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 1824, in _run_ninja_build
raise RuntimeError(message) from e
RuntimeError: Error building extension 'fused_adam'
[2023-04-13 14:51:47,667] [INFO] [launch.py:428:sigkill_handler] Killing subprocess 13775
[2023-04-13 14:51:47,668] [ERROR] [launch.py:434:sigkill_handler] ['/data/deepspeed/bin/python', '-u', 'main.py', '--local_rank=0', '--model_name_or_path', 'facebook/opt-1.3b', '--gradient_accumulation_steps', '2', '--lora_dim', '128', '--zero_stage', '0', '--deepspeed', '--output_dir', '/DeepSpeed/DeepSpeedExamples/applications/DeepSpeed-Chat/output/actor-models/1.3b'] exits with return code = 1

then I use another Linux server, got
RuntimeError: CUDA out of memory. Tried to allocate 256.00 MiB (GPU 0; 14.56 GiB total capacity; 13.30 GiB already allocated; 230.50 MiB free; 13.65 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
bad news
so the little 1.3b need how much memory ?

@hikerell

@ucas010, yes, 1.3B model with Adam optimizer needs at 1.3*14GB=~18GB of GPU memory. Your error message suggests that your GPU has ~14GB. Can you try multiple GPUs so that ZeRO memory optimizations could help?

@ucas010, for memory analysis see Figure 1 in https://arxiv.org/pdf/1910.02054.pdf

@ucas010, yes, 1.3B model with Adam optimizer needs at 1.3*14GB=~18GB of GPU memory. Your error message suggests that your GPU has ~14GB. Can you try multiple GPUs so that ZeRO memory optimizations could help?

with the default opt-1.3b script config and 1x A100 40G,  there is a CUDA OOM error.
so I modified opt-1.3b script to limit batch size.
deepspeed --num_gpus 1 main.py --model_name_or_path facebook/opt-1.3b \
   --gradient_accumulation_steps 2 --lora_dim 128 --zero_stage $ZERO_STAGE \
   --per_device_train_batch_size 8 \
   --per_device_eval_batch_size 8 \
   --deepspeed --output_dir $OUTPUT &> $OUTPUT/training.log

now everything works, but it need ~30GB of GPU memory.
[2023-04-13 12:15:31,180] [INFO] [timer.py:199:stop] epoch=0/micro_step=3120/global_step=1560, RunningAvgSamplesPerSec=23.920492322905034, CurrSamplesPerSec=23.957474851543964, MemAllocated=7.77GB, MaxMemAllocated=26.0GB
[2023-04-13 12:15:37,888] [INFO] [logging.py:96:log_dist] [Rank 0] step=1570, skipped=15, lr=[0.0004315618360406618, 0.0004315618360406618], mom=[(0.9, 0.95), (0.9, 0.95)]
[2023-04-13 12:15:37,891] [INFO] [timer.py:199:stop] epoch=0/micro_step=3140/global_step=1570, RunningAvgSamplesPerSec=23.920438347021395, CurrSamplesPerSec=23.945105514032797, MemAllocated=7.77GB, MaxMemAllocated=26.0GB
[2023-04-13 12:15:37,888] [INFO] [logging.py:96:log_dist] [Rank 0] step=1570, skipped=15, lr=[0.0004315618360406618, 0.0004315618360406618], mom=[(0.9, 0.95), (0.9, 0.95)]
[2023-04-13 12:15:37,891] [INFO] [timer.py:199:stop] epoch=0/micro_step=3140/global_step=1570, RunningAvgSamplesPerSec=23.920438347021395, CurrSamplesPerSec=23.945105514032797, MemAllocated=7.77GB, MaxMemAllocated=26.0GB
[2023-04-13 12:15:44,589] [INFO] [logging.py:96:log_dist] [Rank 0] step=1580, skipped=15, lr=[0.00042612547216574543, 0.00042612547216574543], mom=[(0.9, 0.95), (0.9, 0.95)]
[2023-04-13 12:15:44,592] [INFO] [timer.py:199:stop] epoch=0/micro_step=3160/global_step=1580, RunningAvgSamplesPerSec=23.920614039044654, CurrSamplesPerSec=23.953849064281354, MemAllocated=7.77GB, MaxMemAllocated=26.0GB
[2023-04-13 12:15:44,589] [INFO] [logging.py:96:log_dist] [Rank 0] step=1580, skipped=15, lr=[0.00042612547216574543, 0.00042612547216574543], mom=[(0.9, 0.95), (0.9, 0.95)]

it make me confused.

Glad to know things are working.

Why do you conclude that 30GB is used? Your log suggests ~26GB max memory usage

The extra memory usage is likely due to activation memory. You can enable HF gradient checkpointing to help.

@alibabadoufu in the multi-gpu situation, you can try increasing the --zero_stage to improve memory usage.

Sorry, but we don't currently support M1. I know torch now supports it but the biggest blocker here relates to our custom CUDA kernels that accelerate inference via our HybridEngine.

I have got this same problem:  RuntimeError: Error building extension 'fused_adam'
sloved by update gcc version and activate it!  it helps !


Mu cutoff on the online limb-darkening calculator should be on the "model" and not the "star"

Hi folks,
On today's ExoCTK team meeting, @JeffValenti suggested that the mu cutoff value that is asked to the user on the online tool shouldn't be a stellar property, but rather part of the model (i.e., be on the same spot where the limb-darkening law is defined). A picture is worth more than my previous 308 characters:

I'm assigning @hover2pi to this for efficiency, and also assigning low priority to this (set to v1.2 of ExoCTK).
N.

Fixed in #409


loadbalancer bugfix

这个 PR 实现什么功能/修复什么问题:

负载均很cache校验bug修复
去掉sku排序

是否需要 backport 到之前的 release 分支:

release/2.11
release/2.12

/area region
/cc @yousong @swordqiu @ioito

Codecov Report

Merging #3077 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #3077      +/-   ##
=========================================
- Coverage    7.16%   7.16%   -0.01%
=========================================
  Files         478     478
  Lines       76844   76889      +45
=========================================
  Hits         5509    5509
- Misses      70872   70917      +45
  Partials      463     463

Flag
Coverage Δ

#aFlag
7.16% <0%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/cloudcommon/validators/validators.go
61.18% <ø> (ø)
:arrow_up:

pkg/multicloud/aws/utils.go
0% <ø> (ø)
:arrow_up:

pkg/compute/models/skus.go
1.34% <ø> (ø)
:arrow_up:

...g/compute/models/loadbalancercachedcertificates.go
3.87% <0%> (-0.06%)
:arrow_down:

pkg/compute/models/loadbalancercertificates.go
4.73% <0%> (ø)
:arrow_up:

pkg/multicloud/aws/region.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/loadbalancerbackendgroups.go
1.59% <0%> (-0.05%)
:arrow_down:

pkg/compute/models/managedresource.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/loadbalancerlisteners.go
1.63% <0%> (-0.05%)
:arrow_down:

pkg/compute/models/loadbalancers.go
1.74% <0%> (-0.04%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


Feature Req: Notifcation when you're not chopping a tree

Currently the notifications only go off when a Birds nest drops or your inventory is full.
could you add it to go off for when a tree has been cut down and your character is idle.

Enable idle notifier and check "RuneLite" settings for some options when notifications are thrown.


Investigate downloading mechanism

Are they putting all data in RAM in one big go? that would be bad.

...yes, it's all going in to RAM and yes, it's bad :(


Update examples.md

golang.org/x/net/context package was moved into the standard library as context since Go 1.7.

Deploy preview for modest-edison-79d82e ready!
Built with commit 8f0493da61196cd90f3eb5c61f70420224a1cf8c
https://deploy-preview-7216--modest-edison-79d82e.netlify.com

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!


Stateful services should be preserved, even if they are the primary

Addresses #35
I don't think there are any repercussions, but I'm not sure if there is any nuance to this method's behavior that I'm missing.

Interesting. We haven't given this use case much thought because our stateful services have typically been vendored things (like MySQL) and the services under development aren't.
This PR doesn't go far enough to what you'd want, though, since Galley will still remove the container in the finalizeContainer step.
I think if that were in place this could work, though I'd probably also want to add in warnings if the container wouldn't be configured correctly (for example, if it had lost a link due to the container it was linking to being recreated).
You may also want to consider having your database store its data in volume-mapped data container, which could be marked as stateful and for the most part not touched. You'd have more freedom then in changing your database's image or other options without losing dev data.

Good points - I'll chase down both of those threads.

Per my comments in #36, I don't think this work is necessary. Thanks for the insight @phopkins.

SG! Thanks for the update.
On Thu, Feb 11, 2016 at 2:43 PM, David Giesberg<EMAIL_ADDRESS>wrote:

Per my comments in #36 https://github.com/twitter-fabric/galley/pull/36,
I don't think this work is necessary. Thanks for the insight @phopkins
https://github.com/phopkins.
—
Reply to this email directly or view it on GitHub
https://github.com/twitter-fabric/galley/pull/36#issuecomment-183031600.


added Dockerfile to create wallet image

Build a docker image locally by executing
docker build -t ava-wallet:latest .
and run with e.g.
docker run --name ava-wallet --publish 8080:80 -d --restart=always ava-wallet:latest

Wow! The reaction on PRs made by externals is amazing! Wonderful reaction time!


CDN assets (question)

Hey guys,
I'm looking to make a CDN with the assets generated by webpack in the build folder. The problem i'm having is more web pack specific, when I link the two entry files (main.x.js, vendor.x.js) in the HTML file that are hosted on the CDN it in turn looks at the root for the files, how can i pass a argument to webpack on build to make the path relative to the files are hosted.
Any help would be greatly appricated

I think this line is what you are looking for
https://github.com/mxstbr/react-boilerplate/blob/master/internals/webpack/webpack.prod.babel.js#L22
add     publicPath: '//cdn-path/',
in this object.


Add sirtetris.com

Added sirtetris.com.
The webring icon/link is at the bottom of all content pages (not the index page).
There's a small about section here and the hobby section is also in part kind of a self introduction.

Very nice website :) Just a comment, I love the content, but I hate how these 4 pages are totally disconnected from each other. It would be much more fun to be able to not have to go back to the empty splashpage to discover the other related interests.

It would be much more fun to be able to not have to go back to the empty splashpage to discover the other related interests.

Totally agree. Having a proper navigation on the page fell victim to me putting form over function by introducing the logo/slogan/splashpage at some point.
Looking through the webring today I learned about now pages. Maybe I'll figure something out to reintroduce a navigation and then be able to also add a now page. (:


redirectpolicy: Check lrp type before restoring lrp service

This fixes the regression caused by commit b553f49, whereby not checking the lrp type before restoring clusterIP service would lead to nil pointer access for addressMatcher lrps that don't have the serviceID field set in the lrp config. As there is no backing k8s service for such lrps, they don't need restore.
Here is the stack trace -
level=info msg="Observed a panic: "invalid memory address or nil pointer dereference" (runtime error: invalid memory address or nil pointer dereference)" subsys=klog
level=info msg="goroutine 171 [running]:" subsys=klog
:
level=info msg="\t/usr/local/go/src/runtime/panic.go:969 +0x1b9" subsys=klog
level=info msg="github.com/cilium/cilium/pkg/redirectpolicy.(*Manager).notifyPolicyBackendDelete(0xc000706480, 0xc0009790e0, 0xc000d8d2c0)" subsys=klog
level=info msg="\t/go/src/github.com/cilium/cilium/pkg/redirectpolicy/manager.go:452 +0x316" subsys=klog
level=info msg="github.com/cilium/cilium/pkg/redirectpolicy.(*Manager).deletePolicyBackends(0xc000706480, 0xc0009790e0, 0xc00038ec80, 0x10, 0xc00038ec65, 0x7)" subsys=klog
level=info msg="\t/go/src/github.com/cilium/cilium/pkg/redirectpolicy/manager.go:427 +0x1c5" subsys=klog
level=info msg="github.com/cilium/cilium/pkg/redirectpolicy.(*Manager).OnUpdatePodLocked(0xc000706480, 0xc000de5d40, 0x3ad0001)" subsys=klog
level=info msg="\t/go/src/github.com/cilium/cilium/pkg/redirectpolicy/manager.go:277 +0x3af" subsys=klog
level=info msg="github.com/cilium/cilium/pkg/redirectpolicy.(*Manager).OnUpdatePod(0xc000706480, 0xc000de5d40, 0xc000cd0000)" subsys=klog
Fixes: #13394
Fixes: #13736
[1] https://github.com/cilium/cilium/blob/master/pkg/redirectpolicy/manager.go#L452

:facepalm: Github really doesn't like multiple people making single-label changes at once, haha. The commit which this fixes was in 1.9.0-rc2 so we can mark it as release-note/bug. Agree it should be backported to 1.9. Also release-blocker from the issue that this is fixing.

test-me-please

test-me-please

test-runtime

test-gingko

test-ginkgo

restart-ginkgo

Ginkgo test had already passed. I meant to retest the gke test.

test-gke

retest-runtime

I do wonder idly in the background whether it's possible to configure a service and then configure an LRP using AddressMatcher which selects the same address, and then if you configure this combination, it will break the service and not have a way to properly restore it (without deleting the LRP & restarting Cilium), but that seems like a more rare case.

That seems plausible on surface. Based on the gsg instructions, users should not create an AddressMatcher LRP for services. But I'm not sure if it's worthwhile to enforcing this in the code (e.g., look up frontend of every addressMatcher LRP in the services map). The redirect policy manager won't track k8s service objects for AddressMatcher LRPs so such configurations will be broken anyway.

Yeah I think that case is straying fairly far off the beaten path. It seems pretty unlikely to me that a user will configure an LRP specifically with a service address but without using the servicematcher, so it's probably not necessary to put any guardrails in place (eg warning logs or logic to handle the case).


Doc actions

Fixes #581
Changes Summary

Rewrites sidebar css to a flexbox layout
Adds doc actions (edit, and translate currently)

I'll approve this code if someone else will approve the UX.

This would break IE 9, but Microsoft is dropping that soon anyway. Less than 1% of our visitors are using IE, and more than 90% of them use IE 11.


Crosslink-opt2

Optimized RAM usage in Crosslink Search.
Optimized several other things, overall the results are better.
#2039


Bitshuffle-compressed datasets cannot be read when accessed through a virtual dataset

Describe the bug
Ok, this one is a stretch. Thanks to https://github.com/silx-kit/h5web/pull/1524, it is now possible to read datasets compressed with bitshuffle. But when creating a Virtual dataset pointing such a dataset, I get the following error:
Required filter 'bitshuffle; see https://github.com/kiyo-masui/bitshuffle' is not registered

Full traceback

HDF5-DIAG: Error detected in HDF5 (1.14.2) thread 0:
  #000: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5D.c line 1061 in H5Dread(): can't synchronously read data
    major: Dataset
    minor: Read failed
  #001: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5D.c line 1008 in H5D__read_api_common(): can't read data
    major: Dataset
    minor: Read failed
  #002: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5VLcallback.c line 2092 in H5VL_dataset_read_direct(): dataset read failed
    major: Virtual Object Layer
    minor: Read failed
  #003: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5VLcallback.c line 2048 in H5VL__dataset_read(): dataset read failed
    major: Virtual Object Layer
    minor: Read failed
  #004: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5VLnative_dataset.c line 363 in H5VL__native_dataset_read(): can't read data
    major: Dataset
    minor: Read failed
  #005: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Dio.c line 383 in H5D__read(): can't read data
    major: Dataset
    minor: Read failed
  #006: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Dvirtual.c line 2768 in H5D__virtual_read(): unable to read source dataset
    major: Dataset
    minor: Read failed
  #007: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Dvirtual.c line 2689 in H5D__virtual_read_one(): can't read source dataset
    major: Dataset
    minor: Read failed
  #008: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Dio.c line 383 in H5D__read(): can't read data
    major: Dataset
    minor: Read failed
  #009: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Dchunk.c line 2856 in H5D__chunk_read(): unable to read raw data chunk
    major: Low-level I/O
    minor: Read failed
  #010: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Dchunk.c line 4468 in H5D__chunk_lock(): data pipeline read failed
    major: Dataset
    minor: Filter operation failed
  #011: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Z.c line 1356 in H5Z_pipeline(): required filter 'bitshuffle; see https://github.com/kiyo-masui/bitshuffle' is not registered
    major: Data filters
    minor: Read failed
  #012: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5PLint.c line 267 in H5PL_load(): can't find plugin. Check either HDF5_VOL_CONNECTOR, HDF5_PLUGIN_PATH, default location, or path set by H5PLxxx functions
    major: Plugin for dynamically loaded library
    minor: Object not found

To Reproduce

Select 'vds_bug.h5' in VS Code explorer
Click on data_compressed (a 1D dataset compressed with bitshuffle): it displays fine
Click on data_via_vds (a VDS pointing to non-compressed dataset data): it displays fine
Click on data_compressed_via_vds (you get it)
See error

vds_bug.zip
Expected behaviour
It should be able to display compressed datasets, even through a VDS. Interestingly, the h5wasm demo seems to display it fine ?
Context

OS: Linux x64 5.15.0-107-generic snap
VS Code version: 1.89.1
Extension version: v0.1.5

Ha, so it works if you first select data_compressed and then data_compressed_via_vds but not if you select data_compressed_via_vds right away.
It's because the virtual compressed dataset's filters metadata doesn't "mirror" the source dataset's filters metadata as it should. So vscode-h5web (or myHDF5) doesn't know that it needs to load the bitshuffle plugin.
I'll report on the h5wasm repo.

Ha, so it works if you first select data_compressed and then data_compressed_via_vds but not if you select data_compressed_via_vds right away.

Ha! That's why I thought it worked in the h5wasm demo: it did worked because I selected data_compressed first. Just retried: if I select data_compressed_via_vds first, I get the same error I report here.

It's because the virtual compressed dataset's filters metadata doesn't "mirror" the source dataset's filters metadata as it should.

I'm not sure, you can make a virtual dataset which gives access to multiple datasets stored with different compression filters... (Never seen this though)

I'm not sure, you can make a virtual dataset which gives access to multiple datasets stored with different compression filters

You can. The following snippet works without trouble:
import numpy
import h5py
import hdf5plugin

with h5py.File("double_filter_vds.h5", "w") as h5file:
    data = numpy.linspace(0, 10, 100)

    dset = h5file.create_dataset("data", data=data)
    c_dset = h5file.create_dataset(
        "bitshuffle", data=data, **hdf5plugin.Bitshuffle(cname="lz4")
    )
    c_dset_2 = h5file.create_dataset("blosc", data=data, **hdf5plugin.Blosc2())

    vlayout = h5py.VirtualLayout(shape=(200,), dtype=dset.dtype)
    vsource = h5py.VirtualSource(dset)
    vlayout[:100] = vsource[:]
    vsource2 = h5py.VirtualSource(dset)
    vlayout[100:] = vsource2[:]

    h5file.create_virtual_dataset("data_via_vds", vlayout)

I'm not sure, you can make a virtual dataset which gives access to multiple datasets stored with different compression filters... (Never seen this though)

Yep, Brian mentioned this as well: https://github.com/usnistgov/h5wasm/issues/75#issuecomment-2144936429 — he already released a new version of h5wasm that exposes virtual sources in the metadata.

Should now be fixed in v0.1.6 of the extension.


Modelica.Utilities.Files.move neglects replace parameter for current directory

Reported by Peter.Nilsson on 1 Apr 2015 09:42 UTC
For files in current directory, the replace parameter is implicitly always false, resulting in failure if file already exists. One solution would be to extend the Modelica.Utilities.Internal.FileSystem.rename, which is used for this case, with the replace parameter.

Migrated-From: https://trac.modelica.org/Modelica/ticket/1690

Modified by beutlich on 26 Oct 2015 20:21 UTC

Comment by beutlich on 30 Oct 2015 10:21 UTC
Resolved by c7f24dbab8737c732f8ff673c951fd9776eff5fd.

Changelog modified by beutlich on 30 Oct 2015 10:21 UTC
Do not neglect parameter Modelica.Utilities.Files.move.replace for current directory


Revert "Grant RHEL-6 ELS license to all member accounts"

Reverts ministryofjustice/modernisation-platform#4093
There are two problems with the original branch.

The first is that it uses the wrong data call - data.aws_licensemanager_grants - whereas it ought to use data.aws_licensemanager_received_licenses.
The second is that it will have to use providers in us-east-1 and then be replicated from there out to other regions automatically.

│ Error: creating AWS License Manager Grant (RHEL-6-ELS-111111111111): ValidationException: Grant can only be created in resource home AWS region: us-east-1
│
│   with module.grant-rhel-els.aws_licensemanager_grant.main,
│   on ../../../modules/license-manager/main.tf line 9, in resource "aws_licensemanager_grant" "main":
│    9: resource "aws_licensemanager_grant" "main" {
│

Closing this in favour of https://github.com/ministryofjustice/modernisation-platform/pull/4098


support fair share scheduling ？

alll client try to get  lock fair .

sorry, could you provide more details for your case?


Add components examples

Hello. I've noticed that doesn't exist practical examples of the components, for example a gif showing that component displays. I would like to add them to the documentation, if you don't mind.

@wps13 Pull request are always welcome. I would be more than happy to add in any improvements to the documentation.

Great, I will work on it. Thanks for the replies.


ISPN-9926 Don't run tests for disabled features

https://issues.redhat.com/browse/ISPN-9926

Lots of failures

@tristantarrant I have fixed the failures.

@tristantarrant done


Raise exception unless the caller has opted into partial repro.

Resolves #1138

Ugh. This project's CI is foundering. GitHub is dropping support for Ubuntu 18.04 and our build scripts don't work on Ubuntu 20.04 except with CMake.
See #1206.


Update CCApplyServlet.java - HCL

Fixing bug of trying to dispatcher.forward twice when using correct credentials, throwing an 500 error

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


importFixes: Only provide a fix using the best module specifier for a given module

Fixes #26649
Instead of providing this choice by means of two slightly different fixes, we will rely exclusively on UserPreferences to determine whether the user prefers relative or nonrelative imports.
Note that we still provide different fixes when the same name is exported by different modules.

Is there a test confirming that setting the UserPreference affects the code fix?

See importNameCodeFix_endingPreference.ts for an example.


Action fails if you put it before checkout

line 3: .github/brew-colima: No such file or directory

Full log:
==> Homebrew is run entirely by unpaid volunteers. Please consider donating:
  https://github.com/Homebrew/brew#donations

==> Auto-updated Homebrew!
Updated 8 taps (azure/bicep, graalvm/tap, homebrew/core, homebrew/cask, homebrew/bundle, homebrew/services, mongodb/brew and aws/tap).
==> New Formulae
ancient
cargo-make
cargo-release
clang-build-analyzer
copa
kwok
libaribcaption
liblerc
libsais
m1ddc
mdless
okta-awscli
opencl-clhpp-headers
pipdeptree
qdmr
scriptisto
tetra
xcdiff
zsh-autopair
==> New Casks
capslocknodelay
copilot-for-xcode
hidock
lotus
monofocus
rive
serene
spotify4bigsur
spybuster
ulbow

You have 18 outdated formulae and 2 outdated casks installed.
You can upgrade them with brew upgrade
or list them with brew outdated.

/Users/runner/work/_temp/0731ba17-9763-4380-a2ce-60b9b247d282.sh: line 3: .github/brew-colima: No such file or directory
Error: Process completed with exit code 1.

+1 also faicing this issue

@douglascamata Hey! I realy like setup-docker-macos-action, it so much faster then docker-practice/actions-setup-docker. But I have to switch for it, because it has docker-compose and it is easy to mount a directory, so can't test right now, sorry

Hummm, I see. I can add docker-compose too, should be easy.

Done and updated, @limpbrains. Now docker-compose also installed and properly configured to run with docker compose.

@douglascamata thank you very much. docker-compose works and this issue is resolved


ICAP issues

I have g3 running and have started testing ICAP integration but have run into weird issues.
When I start g3 it sends an initial OPTIONS to the ICAP server. But after that it does not send reqmod/respmod to the server even with the ratio set to 1. If I turn down the ratio to .5 it will occasionally send a Icap res/req. Maybe 1/100. I am wondering if it could be an issue with the ratio code. I have not seen any errors in the g3 logs.
I have started trying to track down where the issue might be but I wanted to file a ticket in case you might know or have suggestions. I will also add more general logging output to the Icap while going through this then file a PR.
I am using the open source icapeg server for testing. Below is my config
auditor:
  - name: default
    protocol_inspection: {} # Enable protocol recognition, use default parameters
    tls_cert_generator: {}  # Enable TLS hijacking, use default parameters
    tls_interception_client: {} # Can configure proxy to target address TLS connection parameters
    h1_interception: {}         # HTTP/1.0 parsing parameters
    h2_interception: {}         # HTTP/2 parsing parameters
    icap_reqmod_service: icap://<IP_ADDRESS>:1344/echo   # ICAP REQMOD service configuration
    icap_respmod_service: icap://<IP_ADDRESS>:1344/echo # ICAP RESPMOD service configuration
    application_audit_ratio: 1.0

Following up. Looks like it has to do with me having user groups. If I disable user groups it works. If an auditor is set at the base level does it also need to be set per user groups?

You should set enable_protocol_inspection to true in user's audit config. It is set to false by default to allow smoothly upgrade from old versions.

Thank you much! That worked well!


feat(panel) - 13478: Fix shadow from Analytics panel

Codecov Report

Merging #399 (d727a5b) into main (b91f067) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #399   +/-   ##
=======================================
  Coverage   11.77%   11.77%
=======================================
  Files         544      544
  Lines       38832    38832
  Branches     1167     1167
=======================================
  Hits         4571     4571
  Misses      33816    33816
  Partials      445      445


Canonical link format

What should the canonical link format be?
HTML or Markdown?
[foo]: ./relative/markdown.md
[foo]: ./relative/html.html

Relative or absolute?
[foo]: ./relative/markdown.md
[foo]: /absolute/markdown.md

Take a look at https://github.com/thejpster/book/blob/testing_links/src/intro/install/verify.md.
It contains:
The contents may not match exactly but you should get the last line about
breakpoints and watchpoints. If you got it terminate the OpenOCD process and
move to the a [test1] b [test2] c [test3] d [test4] e [test5] f [test6].

[test1]: /intro/hardware.html
[test2]: ./intro/hardware.html
[test3]: /intro/hardware.md
[test4]: ./intro/hardware.md
[test5]: intro/hardware.md
[test6]: ../hardware.md

Only test1, test3 and test6 work in mdbook serve. Only test6 works on Github. Therefore I propose all URLs are made relative, with a .md extension, and no leading ./ where the file is in the same directory.

LGTM :+1: thanks for the investigation!


Add DOI and author info to the header of the review popup

This PR adds DOI to the review popup and makes it obvious that it's clickable (fixes #80).  Along the way:

I removed the link from the title itself (since we now have the clickable DOI)
I added the year of the publication after the author (I'm guessing many researchers would identify papers based on the combination of the author and the publication year)

@coglebed I'm curious to hear what you think about this design

I think it looks fabulous. All of it - the color choice, the year, the icons. Really stellar work, Nami.


Workaround CMAKE/MSVC issue

Closes LeoTindall/libui-rs#2
This change allows error free building on MSVC in both build profiles
by forcing libui to always build with the "release" profile. This is primarily an upstream issue with libui and cmake. Reference:
https://github.com/andlabs/libui/pull/249 -- libui experiencing the same build-breaking issues.
https://github.com/andlabs/libui/issues/282 -- Cmake woes
Before merging, might be worth considering if there are any downsides to always compiling libui in release configuration.

Thank you very much for this!


This is how it writes when playing a video - The media could not be loaded, either because the server or network failed or because the format is not supported. -you have so

Bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Additional context
Add any other context about the problem here.

We're looking into some streaming failures on certain files/regions. Please check back again in a couple hours.


Make the necessary changes to realize the WIT area

### Tasks
- [ ] https://github.com/ietf-tools/datatracker/issues/6912
- [x] Obtain leadership/secretariat review, particularly of the scheduling tool
- [ ] Deploy the datatracker changes to production
- [ ] Create witarea mailing list
- [ ] Close the tsvarea group and list

This moved to manual action by the secretariat


Updating a parent definition makes "branches"

Description
I don't know how else to describe the issue, but whenever you add a new word via a child, there's a duplicate definition. It's saved under the child AND the parent.

In the example image, the word šířit was added via the child šíří. The parent/child relationship is fine, but they BOTH have the definition saved.
This means that if I later decide to update šířit via another child (for example, if I come across a brand-new meaning in another context and want to add it to the parent), the original child will STILL have the old definition as well as the new one. This means that I'd need to edit the child to remove the definition.
I believe this is unintended behavior, which is why I chose to make this a bug report rather than feature request.
To Reproduce
Steps to reproduce the behavior, e.g.:

Click on brand-new child word that doesn't have a parent in the database yet
Add the parent
Add the definition
Have two definitions for 1 parent term

Screenshots
I've got tons of these built up.

Extra software info, if not already included in the Description:

OS (e.g., iOS, windows): Pop_OS
Browser (e.g., chrome, safari): Chromium
How you've installed Lute (Docker, python, source): Python via pip
Version: 3.0.6

This is actually intended behaviour :-) but I can see how it feels odd.
The reason for this: suppose I add a term and a parent at the same time, and enter a translation.  I save the term, and then immediately click it again.  If the description were only saved in the parent, then the term translation would be blank, which looks like some data has somehow been lost -- it would feel like a bug.
As I use Lute more, I don't think of the my definitions etc as a "dictionary", but rather as a scratchpad of notes.  I don't mind about the definition being dup'd, it just there to help me out.

This means that if I later decide to update šířit via another child (for example, if I come across a brand-new meaning in another context and want to add it to the parent), the original child will STILL have the old definition as well as the new one. This means that I'd need to edit the child to remove the definition.

Yes, that's right -- I couldn't think of a way to make that better, and it felt like it could get messy to try to manage it differently.

I think it's be good to be able to view and edit parents in the same window then. I agree that it's nice to be able to add notes to children (to give specific usage notes or something), but I think that if you ever want for users to be able to share databases and entries, duplicated definitions, especially if one is outdated, is going to be weird and confusing.

Some misc notes, some copied from discord:
I’ve thought about this and I can’t see how to have the data go to the parent without it getting confusing, or messy to code/test.

For example: I have a new term XX with new parent x. If I put in a translation for XX, and that was only saved with x, then when I reopen the form for Xx again the translation would be blank. That would feel like data loss.
If I pull the translation for x into the form for XX, and I edit XX, should that change the parent?  If yes, then there’s no way to add a new XX-only translation, if no, we have the same problem.
I wouldn’t want to add a “parent translation” text box to the form, it will confuse people, and gets hard to manage if parents are added and removed.  Old issue #15 (only set translation for parent) has some notes.
If we only save the translation in the parent, then the term listing would likely need to pull in the parent translations, as noted in issue #15.  That's fine, that's just the query.  Would the CSV export also need to include that?
How would we handle CSV imports, where the translation is set for the child term?  Would it only update the child, only update the parent (and potentially create inconsistencies there, if you have multiple children in the same file updating the parent)?

I think a good first step for this issue would be to add a global (or per language) setting to disable the automatic duplication of the term definitions to the parent. Maybe even add a checkbox to the Term edit form. That way the user can configure the default behavior they prefer as well as use the checkbox to handle exception  for individual terms.

@cblanken - possibly yep, but I'm reluctant to get into it :-)  Every option means more user tweaking the settings, and another code path, though it's minor.  The questions I listed in my prior comment still apply.  Especially the first point, re something feeling like data loss.
I've never been bothered by the duplicated data, and it simplified me getting reading quickly, because I'd enter in data and it gets copied wherever it might be needed, even if not 100% accurate.  That was good enough for me ... but of course that's my own preference!
In summary, after thinking about it again, I still can't see how to make it dead simple to use.  To view and edit parents in the same window as Ben suggested above would make the form pretty busy.  I agree that the current situation is imperfect, but I feel that pushing this idea along wouldn't add much value to users, and could make things more complicated.
I wonder if there's a simpler solution that could help, e.g. something like a tool to clean up data:  "show children that have the same definition as their parents" and "clean these ones up", for example.  I'd be much more interested in this approach, again knowing it's not perfect, but at least it seems like it's clearly feasible, limited scope, easily understood by users, no interruption to existing flow, etc.

My primary problem is that I frequently need to edit a parent, usually because I improperly save something or forget to add a tag.
But if I've already closed the window, I now need to click into the child, delete it and save, then click into the child and then parent and THEN finally make the edit.
I'm not really sure why showing an editable parent box and child box would make things too cluttered.
I'm attaching a potential layout which I think isn't too cluttered, clearer and much more useful.

But a "delete identical children entries" does sound like a good first step. I would certainly use it.

Ah yeah that's a hassle, I've been fine with such cruft but can see how it would bother you.
I was thinking that the extra text boxes for parent terms would get cluttered, but that was in terms of the current layout.  Your alternative layout is nice but quite a departure from the current state.  Understood that this is just a sketch of possibilities.  Suffice it to say that it gets very fiddly on the code side!  Having dynamic forms is def stretching my abilities.
For "delete identical children descriptions", that should be relatively quick to implement.  I opened #463 to get some initial ideas down.

Yep. And I'm happy to do more sketches if it's helpful. I'm no designer, but I enjoy it.
And I'm getting a little worried about the UX of Lute being difficult, even for people who have been using it for a long time (me).


Bump whatwg-url version

Includes fix for quadratic worst case when parsing some URLs, like enormous data URLs.

Safe change, shouldn't require proper code review.


Standalone Application Deployment-Distribution Documentation

Required Info

Camera Model
D400/T26x

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
Windows/Linux

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
All

SDK Version
{ legacy / 2.. }

Language
Python, C++, Unity, OpenCV, OpenVX }

Segment
Robot, AR/VR }

Issue Description
Is there documentation on the proper way to package an application for distribution? I found an old reference on application deployment but I haven't been able to find any recent information on how to go about including driver installers or necessary libraries for distribution of an application that utilizes D400 or T265 series cameras.

I hope that the link below will be helpful.
https://github.com/IntelRealSense/librealsense/issues/4277

Same issue here.
We really want to avoid copying the SDK realsense2.dll because we assume that is for dev-only purpose.
Any ideas about how to deploy at large scale a standalone application without install anytime the SDK ?

As this case is old I cannot handle it here, so could you re-post the question as a new case of your own by using the green New Issue button on the front page of the forum please?   Thanks!
https://github.com/IntelRealSense/librealsense/issues/


Decide on public/private long-term plan

Per https://openscapes.github.io/series/github-issues.html, OHI decided to make separate repo just for issues, which can be kept permanently private. Maybe we do same?

Transitioning this repo to be our public project!


[swift/main] Use some RegexComponent in variadic inits

Cherry picks of 58c6b4e2f9911df7d67e107c147f92ccf1d242f6 and bd283f9b8e7d3aa2556a828aa4c764e38935ba45 to support the type checker fix for result builder closure type inference.
rdar://105619787

@swift-ci Please test


support for versions 1 19 2 and 1 19 4

Description:
will there be support for versions 1 19 2-1 19 4 ?

No. This mod is for Legacy Fabric. Versions 1.3.2 - 1.13.2

No. This mod is for Legacy Fabric. Versions 1.3.2 - 1.13.2
That's why "Legacy" is in the name.

aww.. okay. It's just that there is no Legacy inscription on Modrinth

1.13+ support is not planned

Didn't see this before writing my own question about this, @thecatcore, but do you know of anyone planning something similar, but for newer builds?

do you know of anyone planning something similar, but for newer builds?

Assuming you meant newer Minecraft releases:
https://github.com/KiltMC

I knew about one of their devs, but didn't realize Kilt went this far until he told me.
And yes, he knows of you. You guys were the inspiration for Kilt. But right now, BluSpring is the only dev working on Kilt.

We would love to have them join us on Discord. Any dev of a compat layer gets a channel to manage as they please.


Links to routes nested on the same page make the page jump to the content.

I am making a page with tabs. Tab header uses Links, and the tab contents have different nested routes. On tab click the page jumps to the tab container. Expected behavior: page stands still in the same position it was before clicking. I was using react router previously and the page would stay still in the same setup.  Example: https://codesandbox.io/s/y0jmjj2o8j
Or am I using it in a wrong way?

Also have the same

I think this is because Reach automatically focuses the route navigated to (for accessibility reasons). If you want to maintain the scroll position I imagine you'll have to manage that yourself

The question is, should it be the default behaviour?

Scrolling to a focused element is standard browser behaviour, so probably

Ok, I'll re-formulate the question: should it be the router's job to focus on the element? When is it good for user experience? I'd expect this to happen in case of hash links, not normal routes.

As far as I understand one of Ryan's explicit goals with the Reach ecosystem was to make everything as accessible as possible. Moving the user's focus to the newly "navigated" to element is an important part of accessibility for a client-side router.
I believe it's an attempt to re-create what happens when you visit a server-side route (e.g. click a normal link). A new page loads and focus is moved to the top of it, allowing the user to tab through the new content. Screenreaders will also announce the title of the new page.
In most other client-side routers focus generally won't move, which means when the user clicks a Link the only indication that any "navigation" happens is visual. For example a screenreader user will have no idea anything on the page changed. A keyboard user will be stuck focused wherever the Link was, or worse (if the Link un-rendered) they'll have no idea where their focus is.
I understand that this behaviour feels weird and unexpected (it was jarring for me too when I saw it), but the reason for that is all the other client-side routers not correctly replicating the behaviour/accessibility of server-side routing.

I believe you can use the primary prop on the Router component to achieve what you desire.
<Router primary={false}>
See the following docs link:
https://reach.tech/router/api/Router

@niccai that's a workaround, but not a solution to this problem. If you change the route of your application and all the content is new, you want to change the focus so keyboard users ad screen readers are placed in the correct place to consume the new content. If you set primary={false} on a page change or major content change, you're hurting the accessibility of your site.

It would be nice to have an option on a route that would make it behave in the conventional way, as if the page reloaded. I tried to add a scroll-up on certain url changes, but haven't figured out yet why it works in some cases but not others. I really did not want to go back to react router but maybe that's a better option for my use-cases

@simoroshka are you saying at the path level being able to designate if the component that is going to be rendered should be focused on or not?

Yes, something like

In what situation would you want to opt out of keyboard/screen reader users being placed in the correct place? Where would you expect focus to be after such a navigation? The body?

Accessibility is definitely a good aim but autofocus and
<Router primary={false}>

is somewhat confusing in terms of an API.

Focus automatically scrolls the page, you'll need to manage scroll yourself, or put primary=false on router and then you should be managing focus yourself.
We have plans to manage both for apps, but we need Suspense from React to mature before we can do it all reliably.
Focus and scroll management are high priority as we move both React Router and this project toward the same hook-based API. More info: https://reacttraining.com/blog/reach-react-router-future/


updates for BP GO:0019640 glucuronate catabolic process to xylulose 5-phosphate (term name, definition, hierarchy, Xref)

For GO:0019640 glucuronate catabolic process to xylulose 5-phosphate:

the pathway is specific for D-glucuronate and D-xylulose 5-phosphate
There is a caveat for D-xylulose 5-phosphate. The other BP that include this chemical all use xylulose 5-phosphate (children of term GO:0051167), normally is should have been D-xylulose 5-phosphate.
Is it better to leave xylulose 5-phosphate to avoid the mismatch?

the current definition is less specific than the GO term name suggests. The current meaning is more or less identical to  GO:0042840 D-glucuronate catabolic process

GO:0042840 D-glucuronate catabolic process
The chemical reactions and pathways resulting in the breakdown of D-glucuronate, the D-enantiomer of glucuronate.
GO:0019640 glucuronate catabolic process to xylulose 5-phosphate
The chemical reactions and pathways resulting in the breakdown of glucuronate into other compounds, including xylulose 5-phosphate.

Could MetaCyc:PWY-5525 be added as an Xref as it covers the first steps of the pathway?

Proposed changes (highlighted in bold):
id: GO:0019640
name: D-glucuronate catabolic process to D-xylulose 5-phosphate
def: "The chemical reactions and pathways resulting in the breakdown of D-glucuronate into D-xylulose 5-phosphate." [GOC:go_curators]
synonym: "glucuronate breakdown to xylulose 5-phosphate" EXACT []
synonym: "glucuronate degradation to xylulose 5-phosphate" EXACT []
xref: MetaCyc:PWY-5525 NARROW
is_a: GO:1901159 ! xylulose 5-phosphate biosynthetic process
is_a: GO:0042840! D-glucuronate catabolic process
intersection_of: GO:0009056 ! catabolic process
intersection_of: has_primary_input CHEBI:15748 ! D-glucuronate
intersection_of: has_primary_output CHEBI:57737 !D-xylulose 5-phosphate(2−)

From PMID:27189775

GO:0042840 D-glucuronate catabolic process already has MetaCyc:PWY-5525 as an XREF.


CM-279 Effect sources added to feed response

Effect sources added to feed response.

added the function get_causal_sources() to score_nodes.py
get_causal_sources is called in the function simple_scoring in
score_nodes.py
added processing of causal effect sources to the networkx object
(added code to the function makeGraph in make_graph.py

Note that the code works but that it may look wrong when looking at the
output, but this is because the function set_edge_properties() doesn’t
behave properly. But fixing set_edge_properties() and making needed
changes to remove_edge_properties_from_nodes() functions is out of the
scope of CM-264 branch/PR

I also added some code from @y-himanen zipcode branch because it corrected some errors in the main master branch that only get triggered during localhost.

are you able to make sure the strings that are returned are valid urls? there a load returning  when i tested returning Lorem ipsum ?

@NickCallaghan the links aren't valid because of an issue with backend source processing code. It's a whole ticket CM-290. I won't be able to get to it until after the zip code story is completed.
This code is correct though. The issue isn't in the scope of this PR.

@NickCallaghan I've slightly fixed that issue. If no links exist for effectSources it now returns "No sources available at present"
We'll get a better solution turned around soon.


Leave messages are not always parsed / posted

It looks like leave messages are not always handled. Didn't have time to look into this yet. I'm thinking of adding a PR that adds debug mode, just logs all received and send messages to stdout so you can see what the bot is doing. But I thought let's at least document this.

Interesting.. So it's not always handling them? That's weird..

Looks like that issue was on Discord's end yesterday due to the outage. Considering the fact it actually sent [LEAVE] to the server would mean with the API outage Discord had yesterday, the message simply failed to send.
However, when a player gets kicked/banned for reasons such as AFK, these don't log at all due to a limitation in factorio. The rewrite will have an option and a guide on adding the custom feature.

Allright I will keep an eye on this :)

Can you re-open this issue? I do continue to have this issue. I am going to add some debug logging to the bot so I can at least see what is it doing. Is it getting the LEAVE and it is posting it to Discord or not?

Do you preferer a PR for master or for rewrite? Think it would be best to know this before I add the code :)


@ApplicationPath seems to be ignored by the plugin

The swagger maven plugin does not take the javax.ws.rs.Application class and @ApplicationPath and its base URI into consideration when it generates the documentation. All paths that are generated by the plugin are missing the base URI provided by the annotation. Documentation generated by swagger JAX-RS resources handle this correctly.
To demonstrate this, I have set up a sample project that uses both the swagger resource and this plugin to create the openapi documentation. I have also attached the outputs they provide (in json format), wherer the difference can be clearly seen.
The reason for the difference is that the GenerateMojo instantiates the swagger Reader instance, but it does not provide it with the instance of javax.ws.rs.Application (possibly annotated with @ApplicationPath). As a consequence of this, the Reader produces the OpenApi instance, but ignores the Application. Luckily, it is possible to work around this limitation by creating custom ReaderListener (however, it must be annotated by @OpenAPIDefinition, otherwise it is not picked up by the plugin) and set the application class to the provided Reader by calling reader#setApplication(...)), however, I think that the plugin could somehow support this out of the box. My thoughs:

an additional parameter could be added to the plugin configuration, where the fully qualified name of the Application class could be specified
a classpath scanning could be performed to find the class automatically

Do you think this might be useful?
The json documentations for comparison:
swagger_docs.zip
The sample project:
swagger-maven-plugin-sample-project.zip

Thanks for reporting. That is a good point. Basically we try to mimic the same behaviour as the servlet for dynamic triggering generation in the swagger jaxrs module. But it makes good sense to look at the path from the @Application annotation.

Hi @langecode , thanks for your quick response. I have been playing with the plugin a bit, and I can submit a pull request with this feature if you are interested.

@tosix1988, that would be great. You are welcome to do so, thx.


Draft revised Gov theme page

When [Situation]?
Preparing to launch the revised Wordpress site
I want to [Motivation]
Direct people from gov --> WP and make sure there's no duplicated content
So I can [Expected Outcome]
connect people with service info they need.

sent to audrey for review -- over to Christina monday


RFC: secondary::VacantEntry: Provide remove_stale_entry

Closes #55
I wasn't sure about the name.  remove_stale_entry maybe makes it seem like this is a cleanup function you are supposed to call.  I considered extract_stale_entry.  I'm also not sure about entry.  Maybe just extract_stale.
If you like this, I will provide an implementation for sparse secondary maps too.

For your reading convenience
Rendered version of the docs
Returns the stale key and data, if any, which would be overwritten by inserting using this VacantEntry.
This situation arises if the stale key was removed from the primary map, and a subsequent insert into the primary map reused the slot.
remove_stale_entry can be used to handle this situation specially --- for example, if the application wants to lazily clean up tertiary data in another data structure indexed by the now-stale key, or by the value stored in the SecondaryMap.
Most applications will not need this.
Examples
# use slotmap::*;
# use slotmap::secondary::Entry;
let mut pri = SlotMap::new();
let mut sec = SecondaryMap::new();

let k1 = pri.insert(1);

let ent = sec.entry(k1);
let mut vacant = match ent { Some(Entry::Vacant(vac)) => vac, _ => panic!("1. {:?}", &ent) };
assert_eq!(vacant.remove_stale_entry(), None);

sec.insert(k1, 'a');
pri.remove(k1);
// Imagine we don't keep a note of k1, after this.
let k2 = pri.insert(2);

let ent = sec.entry(k2);
let mut vacant = match ent { Some(Entry::Vacant(vac)) => vac, _ => panic!("2. {:?}", &ent) };
// Now we have recovered k1 and the associated data:
assert_eq!(vacant.remove_stale_entry(), Some((k1, 'a')));
assert_eq!(vacant.remove_stale_entry(), None);

vacant.insert('b');
assert!(sec.entry(k1).is_none());

For your reading convenience
Rendered version of the docs
Returns the stale key and data, if any, which would be overwritten by inserting using this VacantEntry.
This situation arises if the stale key was removed from the primary map, and a subsequent insert into the primary map reused the slot.
remove_stale_entry can be used to handle this situation specially --- for example, if the application wants to lazily clean up tertiary data in another data structure indexed by the now-stale key, or by the value stored in the SecondaryMap.
Most applications will not need this.
Examples
# use slotmap::*;
# use slotmap::secondary::Entry;
let mut pri = SlotMap::new();
let mut sec = SecondaryMap::new();

let k1 = pri.insert(1);

let ent = sec.entry(k1);
let mut vacant = match ent { Some(Entry::Vacant(vac)) => vac, _ => panic!("1. {:?}", &ent) };
assert_eq!(vacant.remove_stale_entry(), None);

sec.insert(k1, 'a');
pri.remove(k1);
// Imagine we don't keep a note of k1, after this.
let k2 = pri.insert(2);

let ent = sec.entry(k2);
let mut vacant = match ent { Some(Entry::Vacant(vac)) => vac, _ => panic!("2. {:?}", &ent) };
// Now we have recovered k1 and the associated data:
assert_eq!(vacant.remove_stale_entry(), Some((k1, 'a')));
assert_eq!(vacant.remove_stale_entry(), None);

vacant.insert('b');
assert!(sec.entry(k1).is_none());


init: add pom and base classes

Add pom.xml for project config. Add base classes
for handling smart-http protocol: GitSlice is an entrypoint,
other slices will be responsible for receive and upload pack commands.

@olenagerasimova thanks, all fixed


Add notice about changed name of the old allOnQueue to new onQueue for Laravel 8

Hi @patrickbrouwers will be better if you add notice about Laravel 8 changes for allOnQueue method to onQueue
https://laravel.com/docs/8.x/upgrade#queue-allOnQueue-allOnConnection
to
https://docs.laravel-excel.com/3.1/exports/queued.html#custom-queues

Docs are open source, feel free to PR


Add info from pyodbc cursor to adapter response

Describe the feature
The pyodbc connection method (Databricks) provides a database API like other transaction database implementations with a cursor object defined here. We should the info from the cursor and other context if possible to provide relevant data in the adapter response which was added in dbt 0.19.0 (https://github.com/fishtown-analytics/dbt/pull/2961)
Describe alternatives you've considered
We could just continue to return "OK" as the adapter response, but that's lame.
https://github.com/fishtown-analytics/dbt-spark/blob/2f7c2dcb419b66d463bb83212287bd29280d6964/dbt/adapters/spark/connections.py#L275-L277
Who will this benefit?
End users and it will also align implementation across adapters.

Reopened (with more detail) as https://github.com/dbt-labs/dbt-spark/issues/497


Question about triton example in tutorial

Why using platform: "pytorch_libtorch" while model.pt as Torch TensorRT

model.pt as platform: "pytorch_libtorch"

instead of

model.pt as platform: "tensorrt_plan" in serving_torch_tensorrt_with_triton

Hi @allen-ash
Even after the torch tensorrt compilation, the model is still viewed as TorchScript, using PyTorch to execute the model. Therefore, from Triton's perspective, we still need to use a PyTorch backend.