[BUG]: Setting Workspace LLM Provider or Workspace Agent LLM Provider fails.

How are you running AnythingLLM?
AnythingLLM desktop app
What happened?
After setting up a workspace and choosing any LLM I can get started chatting in a new thread. however, anything llm is overwriting my modelfile and preset system prompt (when using Ollama) So I go to workshop settings to add a system prompt, change my llm to the one I want and add openai agent llm.
I cannot make any changes in the Workspace LLM Provider or Workspace Agent LLM Provider without causing this error:
Error:
Invalid prisma.workspaces.update() invocation:
{
where: {
id: 1
},
data: {
chatProvider: "ollama",
chatModel: "demicube_tablereader:latest",
chatMode: "chat",
openAiHistory: 20,
openAiPrompt: "Given the following conversation, relevant context, and a follow up question, reply with an answer to the current question the user is asking. Return only your response to the question given the above information following the users instructions as needed.",
queryRefusalResponse: "There is no relevant information in this workspace to answer your query.",
openAiTemp: 0.7,
? name?: String | StringFieldUpdateOperationsInput,
? slug?: String | StringFieldUpdateOperationsInput,
? vectorTag?: String | NullableStringFieldUpdateOperationsInput | Null,
? createdAt?: DateTime | DateTimeFieldUpdateOperationsInput,
? lastUpdatedAt?: DateTime | DateTimeFieldUpdateOperationsInput,
? similarityThreshold?: Float | NullableFloatFieldUpdateOperationsInput | Null,
? topN?: Int | NullableIntFieldUpdateOperationsInput | Null,
? pfpFilename?: String | NullableStringFieldUpdateOperationsInput | Null,
? workspace_users?: workspace_usersUpdateManyWithoutWorkspacesNestedInput,
? documents?: workspace_documentsUpdateManyWithoutWorkspaceNestedInput,
? workspace_suggested_messages?: workspace_suggested_messagesUpdateManyWithoutWorkspaceNestedInput,
? embed_configs?: embed_configsUpdateManyWithoutWorkspaceNestedInput,
? threads?: workspace_threadsUpdateManyWithoutWorkspaceNestedInput
}
}

Unknown argument `chatProvider`. Available options are listed in green.

![image](https://github.com/Mintplex-Labs/anything-llm/assets/80002617/7680d23c-4992-4905-8283-6581a520aef8)

### Are there known steps to reproduce?

1. set up new workspace
2. try to change any settings in Workspace LLM Provider or Workspace Agent LLM Provider

This is most likely caused by your database schema not being migrated when you updated your version of AnythingLLM. I see you're on Windows and if you don't have an issue resetting your AnythingLLM instance, you can try these steps to resetting your desktop app.

Close the AnythingLLM desktop app.
Open File Explorer/Windows Explorer. Type %AppData% into the address bar and hit enter.
Delete the anythingllm-desktop folder.
Restart the app and this should reset your desktop app and migrate the database properly.

Also which OS are you one?

I meet same issue, and I am using AnythingLLM desktop app Windows OS. I can't do any change in "Chat Settings".
Error:
Invalid prisma.workspaces.update() invocation:
{
where: {
id: 1
},
data: {
chatProvider: "openai",
chatModel: "gpt-4o",
chatMode: "query",
openAiHistory: 20,
openAiPrompt: "Reply with an answer to the current question the user is asking. Return only your response to the question given the above information following the users instructions as needed.",
queryRefusalResponse: "There is no relevant information in this workspace to answer your query.",
openAiTemp: 0.7,
? name?: String | StringFieldUpdateOperationsInput,
? slug?: String | StringFieldUpdateOperationsInput,
? vectorTag?: String | NullableStringFieldUpdateOperationsInput | Null,
? createdAt?: DateTime | DateTimeFieldUpdateOperationsInput,
? lastUpdatedAt?: DateTime | DateTimeFieldUpdateOperationsInput,
? similarityThreshold?: Float | NullableFloatFieldUpdateOperationsInput | Null,
? topN?: Int | NullableIntFieldUpdateOperationsInput | Null,
? pfpFilename?: String | NullableStringFieldUpdateOperationsInput | Null,
? workspace_users?: workspace_usersUpdateManyWithoutWorkspacesNestedInput,
? documents?: workspace_documentsUpdateManyWithoutWorkspaceNestedInput,
? workspace_suggested_messages?: workspace_suggested_messagesUpdateManyWithoutWorkspaceNestedInput,
? embed_configs?: embed_configsUpdateManyWithoutWorkspaceNestedInput,
? threads?: workspace_threadsUpdateManyWithoutWorkspaceNestedInput
}
}

Unknown argument `chatProvider`. Available options are listed in green.

This is related to permissions on the machine most likely. Additionally, if there is no internet connection on the first boot and a migration is required the prisma engine will fail to download and migrate. After which, no migrations will be needed. We pre-migrate the client before packaging anyway so this should be extraneous.
@mooreezen what version are you on?

Hi. I'm on version 1.5.3.

Hello, it seems ok now after upgrading to version 1.5.5.

This is most likely caused by your database schema not being migrated when you updated your version of AnythingLLM. I see you're on Windows and if you don't have an issue resetting your AnythingLLM instance, you can try these steps to resetting your desktop app.

Close the AnythingLLM desktop app.
Open File Explorer/Windows Explorer. Type %AppData% into the address bar and hit enter.
Delete the anythingllm-desktop folder.
Restart the app and this should reset your desktop app and migrate the database properly.

this deletes the storage folder and all your documents


HackerRank upgrade

Hi Jainam,
These are some suggested enhancements to the HackerRank scrapper.
I needed to make these changes to download my solutions.
Please feel free to pick only the ones that make sense and ignore the ones that don't.

Wait and retry to avoid rate limit
Skip already downloaded solutions
More file extensions
Added pagination support for tracks (current API limit is 50)

Best regards,
Felipo Soranz

Thank you @feliposz


is it possible to ?

hi,
I'd like to use frappe/gantt as a graphical planning for rent items ,
is it possible to put several tasks on the same line ?
is it possible to have a day view ?
thanks for your support,
Jean-Louis

@jlcurty Can you provide a mockup?
We already have a Day view mode.

I also need this. Mockup - http://prntscr.com/ihdhx7

Duplicate of #95.


duplicate "productName" in package.json

Search before asking

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

Operating System

[ ] macOS
[ ] Windows
[X] Linux

Operating System Version
Alpine Linux edge
It happens on the web browser too?
No, it just happens on the Desktop app
Rocket.Chat Desktop App Version
3.8.17
Rocket.Chat Server Version
n/a
Describe the bug
package.json contains a value for the key productName twice. this leads to an error in some JSON parsers (namely, serde_json)
How to Reproduce
tasje pack --config electron-builder.json
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error("duplicate field `productName`", line: 136, column: 15)', src/main.rs:88:84

Describe your Expected behavior
productName defined once, package.json parsing properly anywhere
Anything else
No response
Are you willing to submit a code contribution?

[ ] Yes, I am willing to submit a Pull Request!

Fixed, thank you!


ExtEventLoop. Warning: Event::add(): Epoll ADD(1) on fd 16 failed

I have problem with send file from docker container on ExtEventLoop.
I have exception for static-server.
Warning: Event::add(): Epoll ADD(1) on fd 16 failed.  Old events were 0; read change was 1 (add); write change was 0 (none): Operation not permitted
File is avaible
step1
step2
step3
step4
Problem reproduce only docker container for me.
Any idea?

PHP 7.2, event (ext for loop)
issue in phppm repo:
https://github.com/php-pm/php-pm/issues/396
I prepared simple example for reproduce problem - https://github.com/alexpts/phppm-demo-bug
Problem reproduces only container FS, if you use docker volume FS it works.

@alexpts Thanks for reporting!
This sounds like it may be related to #156 and it looks like PHP-PM updated the dependency for this package only a few hours ago: https://github.com/php-pm/php-pm/pull/401
Can you try seeing if you're running this against the latest version of this package (composer info) and/or check if updating this fixes your issues?

@clue Ok, I will check latest version and master php-pm.

@clue Problem was resolve on master branch php-pm/php-pm


upgrade grpc version

What would you like to be added:

Now grpc FromIncomingContext method cost too many cpu, we can upgrade grpc version, and replace with  ValueFromIncomingContext.
goos: darwin
goarch: amd64
pkg: google.golang.org/grpc/metadata
cpu: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
BenchmarkFromIncomingContext-12                 4657538    260.4 ns/op     416 B/op     3 allocs/op
BenchmarkValueFromIncomingContext-12      28756804   38.28 ns/op     16 B/op        1 allocs/op
PASS
ok google.golang.org/grpc/metadata 3.475s
Why is this needed:

/area performance
/area optimization

/reopen

we can only get header one time..


createRule should not add a reset sheet

This branch fixes an issue that occurs when trying to use jss.createRule after jss.createStyleSheet. jss-isolate throws an error because by then a sheet already exists and there is no safe-guard check for this - the expected behaviour for createRule is to not add a reset sheet.

https://github.com/cssinjs/jss-isolate/commit/6c061755b9ab63df4d559f6e31755f4fa51927d1


Build failure: python3Packages.opentelemetry-instrumentation

Steps To Reproduce
Steps to reproduce the behavior:

no specific steps because python3 overridden with a local variation but I am getting the exact same error as hydra (linked below)

Build log
https://hydra.nixos.org/build/281240821/nixlog/1
Additional context
NIL
Metadata

 - system: `"x86_64-linux"`
 - host os: `Linux 6.8.0-51-generic`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.24.11`
 - channels(root): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Notify maintainers

@natsukium

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Seems like this is caused by python 3.12.8, specifically as part of this change

gh-104745: Limit starting a patcher (from unittest.mock.patch() or unittest.mock.patch.object()) more than once without stopping it

Closing this as it is unrelated to opentelemetry-instrumentation specifically.


Using async, await makes workbox without further transpilation break on Samsung Internet

See twitter discussion here: https://twitter.com/addyosmani/status/910631449753141253
@poshaughnessy noticed the starbucks PWA's SW isn't installing properly on Samsung Internet.
Turns out this is because of usage of async and await.
Library Affected:
all
Browser & Platform:
Samsung Internet
Issue or Feature Request Description:
Probably best to not use it in distributed versions if you expect people to be able to use it as is in Samsung Internet.
To repro, open https://preview.starbucks.com in inspect devices on samsung internet. It fails on line 80 of our SW which is the included Workbox code: https://preview.starbucks.com/serviceworker.js
WHich is the first usage of async/await.

I guess UCBrowser 5 which supports SW doesn't support async/await yet too.

I'm in favor of dropping async and await usage in favor of  promise chains. Seems like the sanest approach for developers using Workbox.

I'm strongly in favor of us switching to just using promises. I would like to see if this is something we could do for the next short-term release. @gauntface @jeffposnick wdyt?
It would unblock a few folks that are trying to use Workbox for production and as a forcing function would also help us figure out the browser support matrix for older versions of Chrome on mobile platforms.

While it's still open for discussion, I wanted to take a look at the impact of changing our current Babel usage to include babel-preset-env  with the following configuration:
const babelPlugin = babel({
  presets: [['env', {
    targets: {
      browsers: ['chrome >= 51'],
    },
    modules: false,
  }], ['babili', {
    comments: false,
  }]],
  plugins: ['external-helpers'],
});

It brings the final size of workbox-sw.prod.v2.0.1.js up by 1.8kb, to a total of 48879 bytes.

<EMAIL_ADDRESS>is out (along with versions for our other modules), and it includes the transpilation.
It looks okay in my testing, but our naming convention for the built modules doesn't play nicely with the -rc1, so the one issue is that the output name is workbox-sw.prod.v2.0.2-rc1-2.0.2-rc1.0.js.
@HenrikJoreteg, if you could confirm that this works in your testing, we can tag it the official 2.0.2 release.

Just a heads-up: we're going to tag this code as the official 2.0.2 release tomorrow. If there are any last-minute incompatibilities, let us know.


docker: 26.0.0 -> 26.0.1

Description of changes
Changes: https://docs.docker.com/engine/release-notes/26.0/#2601

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

closed in favour of: #305229


2d frustum culling

Objective
Enable frustum culling for 2D, to be more precise, for Mesh2d, Sprite and TextureAtlasSprite.
Solution

Mesh2d is essentially only a regular Mesh with its handle wrapped by Mesh2dHandle, which enables it to be drawn by the 2d pipeline. As such it already has the compute_aabb() function, which i used for frustum culling the same way it is done for the regular Mesh.
Sprite has an image/texture, which provides a size, or a user specified custom_size. I've added ComputedVisibility to the SpriteBundle and use the size to create an Aabb and insert it as a component. The visibility is computed by the already existing check_visibility system.
For TextureAtlasSprite i use the user specified custom_size, or a size computed from the sprite's Rect as specified in the TextureAtlas. I've added ComputedVisibility to the SpriteSheetBundle too and the rest is the same as with what i have done for the Sprite.

Benchmarks
System: Windows 10, AMD Ryzen 5 5600XT, AMD Radeon RX 570

Bevymark
many_sprites

without culling
~32-33 fps / CPU  ~9%
~52 fps / CPU ~19%

with culling
~32-33 fps / CPU ~10%
~60 fps / CPU  ~6%

Commands
cargo run --release --example bevymark -- 13 10000
cargo run --release --example many_sprites
Additional stuff
I have run tracy-trace too, here are two captures of many_sprites, one with and one without culling.
tracy_traces.zip
The traces show, that a lot of the additional fps come from the time drop in queue_sprites, from 15.5ms to 43us. That should come from the reduced number of extracted sprites that need to be queued.

However, this needs a lot of tests: this functionality is notoriously fragile. We should test with:

scaled sprites
scaled cameras
orthographic zoomed cameras
rotated cameras and sprites
sprites that are on the border of the screen
sprites that are larger than the camera's view
sprites that are moving onto the screen
sprites that are just spawned

Uff...well...do such tests exist for 3d frustum culling? I am mostly using what is already used for 3d, so it should work equally well for 2d as it does for 3d.
But i have tested all your points

[x] scaled sprites: manually tested with sprite and a scale of Vec3::new(2.0, 1.0, 1.0) and Vec3::new(2.0, 2.0, 1.0)
[x] scaled cameras: manually tested with sprite and the camera transform set to a scale of Vec3::new(0.5, 0.5, 1.0) and Vec3::new(2.0, 2.0, 1.0)
[x] orthographic zoomed cameras: that is essentially just a scaled camera.
[x] rotated cameras and sprites: covered by bevymark and many_sprites
[x] sprites that are on the border of the screen: manually tested with mesh2d and sprite_sheet, by moving the camera to values around x = 700.
[x] sprites that are larger than the camera's view: manually tested with sprite and a scale of Vec3::new(10.0, 10.0, 1.0)
[x] sprites that are moving onto the screen: covered by bevymark
[x] sprites that are just spawned: covered by bevymark

Uff...well...do such tests exist for 3d frustum culling

There aren't, but there should be :p I didn't review that PR though. It should be fine to create these tests for either 2D or 3D.

But i have tested all your points

It's great to hear that these all work, and it makes me much more confident in this PR. But tests aren't just for verifying that the code works at time of creation; they're essential for ensuring that code isn't accidentally broken in the future <3 I want contributors to feel confident when trying to further optimize or refactor this code, so we should add automated tests for this to the code base.

Sure, i can understand that and think it is useful. But such tests also don't exist for 3d, so i think that is out of scope for this PR and should probably be its own PR which adds such tests for both cases, 2d and 3d.

Thanks for this, I tested it in its current state. Unfortunately it broke the rendering of my project. I have made a small reproducing example:
use bevy::prelude::*;

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .add_startup_system(setup)
        .run();
}

const SPRITE_WIDTH: f32 = 32.0;
const SPRITE_HEIGHT: f32 = 32.0;

fn setup(
    mut commands: Commands,
    asset_server: Res<AssetServer>,
    mut texture_atlases: ResMut<Assets<TextureAtlas>>,
) {
    let texture = asset_server.load("texture.png");
    let mut camera = OrthographicCameraBundle::new_2d();

    camera.transform.scale.z = 1000.;
    camera.transform.scale *= 0.5;
    camera.camera.near *= 3.;
    commands.spawn_bundle(camera);

    for i in 0..10 {
        for j in 0..10 {
            let (x, y) = (i as f32 * SPRITE_WIDTH, j as f32 * SPRITE_HEIGHT);

            commands.spawn_bundle(SpriteBundle {
                texture: texture.clone(),
                transform: Transform::from_translation(Vec3::new(x, y, -y)),
                ..Default::default()
            });
        }
    }
}

Expected result (from current master) [sorry for the hexagonal tile, that's what I had at hand]:

Result with this PR:

Ok...it boils down to how the two orthographic cameras are set up.
2D has the camera at Transform::from_xyz(0.0, 0.0, 1000.0 - 0.1), while the 3D orthographic camera is at Transform::from_xyz(0.0, 0.0, 0.0). That means that 0 on the z-axis is closest to the 2D orthographic camera, so if z is negative in a sprite transform, then that "moves" the sprite essentially behind the camera, and it gets culled. How do other engines behave in such a case? Is this behavior expected/normal for an orthographic camera?

if z is negative in a sprite transform, then that "moves" the sprite essentially behind the camera
In the context of bevy we use a reverse-z, so I would expect negative z to show since it should be in front of the camera.

Other than that, I would indeed expect sprites behind the camera to be culled, it doesn't really make sense to have sprites behind the camera.

Closing in favor of #7885


bug: vm.stopAndReturnDebugTraceRecording consumes excessive amount of gas

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (59f354c 2024-12-11T00:29:25.224313000Z)
What command(s) is the bug in?
forge test
Operating System
None
Describe the bug
The erc4337-validation library leverages startDebugTraceRecording but the stopAndReturnDebugTraceRecording consumes tens or even hundreds of millions of gas thus when invoking the function many times, it will become increasingly more likely to result in OOG.
I forked the erc4337-validation repo and log the gas used (including stopAndReturnDebugTraceRecording) here and this is the sample output:
> forge test -vvvv --mt testSimulate
Ran 1 test for test/SimulatorV060.t.sol:SimulatorTest
[PASS] testSimulate() (gas: 122687982)
Logs:
  Gas used by stopAndReturnDebugTraceRecording 48967003
  ...

On my dev environment which uses erc4337-validation, the stopAndReturnDebugTraceRecording can cost up to 113205509 gas

@grandizzy Thanks! pause/resume Gas Metering solve the OOG problem for me. But now I ran into other problems with my tests (they are updated with pause/resume Gas Metering):

Still some tests fail with MemoryOOG, they all fail right after calling vm.stopAndReturnDebugTraceRecording. If I bump the gas_limit param 10x, still MemoryOOG, if I bump it 100x, then I get MemoryLimitOOG. After I also bump memory_limit 10x, the test can finally pass.
Next, I try to run the whole test suites (including about 120 tests, each would invoke debug trace recording multiple times) but the process always gets killed.

Any idea how can I debug or solve this?

@NIC619 for point 1 try setting gas_limit = "18446744073709551615" in config
For point 2 - can you provide more details, how is getting killed, is that an OOM?

for point 1 try setting gas_limit = "18446744073709551615" in config

@grandizzy Setting the gas limit high enough would get rid of the MemoryOOG error but still gets MemoryLimitOOG error. Any recommendations about how to set the value of memory_limit in config?

For point 2 - can you provide more details, how is getting killed, is that an OOM?

This is the log:
> npm run test:e2e:sepolia
> forge test -vvv --show-progress --fork-url sepolia --fork-block-number 7100540 --mp 'test/e2e/*.t.sol' --nmc 'SwitchBetweenOwnerValidatorsTest'

[⠊] Compiling...
No files changed, compilation skipped
test/e2e/ValidateUserOpCalls.t.sol:ValidateUserOpCallsTest
  ↪ Suite result: ok. 30 passed; 0 failed; 0 skipped; finished in 202.38ms (13.74ms CPU time)
test/e2e/gasPayment/GasPaymentWithoutPaymaster.t.sol:GasPaymentWithoutPaymasterTest
  ↪ Suite result: ok. 3 passed; 0 failed; 0 skipped; finished in 591.10ms (409.45ms CPU time)
test/e2e/initialization/InitializeEntryPoint.t.sol:InitializeEntryPointTest
  ↪ Suite result: ok. 1 passed; 0 failed; 0 skipped; finished in 377.92ms (186.00µs CPU time)
test/e2e/initialization/InitializeOwnerValidators.t.sol:InitializeOwnerValidatorsTest
  ↪ Suite result: FAILED. 2 passed; 3 failed; 0 skipped; finished in 216.22ms (3.43ms CPU time)
test/e2e/AccountsManagement.t.sol:AccountsManagementTest
  ↪ Suite result: FAILED. 2 passed; 3 failed; 0 skipped; finished in 3.19s (3.00s CPU time)
test/e2e/gasPayment/e2eOffChainPaymaster.t.sol:e2eOffChainPaymasterTest
  ↪ Suite result: FAILED. 13 passed; 3 failed; 0 skipped; finished in 3.94s (3.76s CPU time)
test/e2e/updateValidator/UpdateECDSAValidatorSettings.t.sol:UpdateECDSAValidatorSettingsTest
  ↪ Suite result: ok. 2 passed; 0 failed; 0 skipped; finished in 4.10s (2.29s CPU time)
test/e2e/DeployAccountEntryAndimAccount.t.sol:DeployAccountEntryAndimAccountTest
  ↪ Suite result: FAILED. 1 passed; 2 failed; 0 skipped; finished in 2.90s (2.69s CPU time)
test/e2e/AccountExecutor.t.sol:AccountExecutorTest
  ↪ Suite result: ok. 3 passed; 0 failed; 0 skipped; finished in 6.24s (6.06s CPU time)
test/e2e/imAccountFallback.t.sol:imAccountFallbackTest
  ↪ Suite result: ok. 16 passed; 0 failed; 0 skipped; finished in 258.94ms (14.33ms CPU time)
test/e2e/updateValidator/SetInvalidOwnerValidator.t.sol:SetInvalidOwnerValidatorTest
  ↪ Suite result: ok. 2 passed; 0 failed; 0 skipped; finished in 745.94ms (519.58ms CPU time)
test/e2e/AccountEntriesManagement.t.sol:AccountEntriesManagementTest
  ↪ Suite result: FAILED. 0 passed; 3 failed; 0 skipped; finished in 12.08s (10.32s CPU time)
test/e2e/updateValidator/UpdateSimpleMultiSigValidatorSettings.t.sol:UpdateSimpleMultiSigValidatorSettingsTest
  ↪ Suite result: ok. 4 passed; 0 failed; 0 skipped; finished in 8.77s (7.59s CPU time)
test/e2e/gasPayment/GasPaymentWithPaymaster.t.sol:GasPaymentWithPaymasterTest
  ↪ Suite result: ok. 7 passed; 0 failed; 0 skipped; finished in 10.92s (6.50s CPU time)
test/e2e/module/InstallAndUninstallModule.t.sol:InstallAndUninstallModuleTest
  ↪ Suite result: ok. 7 passed; 0 failed; 0 skipped; finished in 35.56s (35.19s CPU time)
test/e2e/updateValidator/UpdateUntrustedECDSAValidatorWithTrustedECDSAGuardianSettings.t.sol:UpdateUntrustedECDSAValidatorWithTrustedECDSAGuardianSettingsTest
  ↪ Suite result: ok. 8 passed; 0 failed; 0 skipped; finished in 37.57s (35.73s CPU time)
###############################---------      16/21      completed (with 10 threads)
⠁ test/e2e/updateValidator/UpdateTrustedWebAuthnValidatorWithUntrustedECDSAGuardianSettings.
⠁ test/e2e/guardian/e2eUntrustedECDSAValidatorWithTrustedECDSAGuardian.t.sol:e2eUntrustedECD
⠐ test/e2e/guardian/e2eUntrustedWebAuthnValidatorWithTrustedECDSAGuardian.t.sol:e2eUntrusted
⡀ test/e2e/guardian/e2eTrustedWebAuthnValidatorWithUntrustedECDSAGuardian.t.sol:e2eTrustedWe
⡀ test/e2e/updateValidator/UpdateWebAuthnValidatorSettings.t.sol:UpdateWebAuthnValidatorSett[1]    5062 killed     npm run test:e2e:sepolia

Are there other configs or options I can add to the command to provide more details?

 My machine specs are MacBook Pro w/ M1 Pro chip, 10 cores and 16 GB memory.
This is the top command snapshot when I ran the test command:

Then I switch back to my main branch which does not contain changes related to vm.startDebugTraceRecording, the memory usage seems normal, compared to the one above:

is that an OOM happening?

Is there a way I can tell if a process is terminated due to OOM?

The issue here is that debug steps recording needs a lot of memory to store those steps both on foundry (Rust) and the test contract (Solidity) side. This is getting even worse when a lot of tests are run in parallel, especially if those are fuzz tests
@NIC619 is this happening if you set --threads flag to a lower value?

@klkvr Thanks. After some trial and error, reducing the thread from the default (10) to 5 works. The tests are executed successfully on my machine.

@NIC619 that's great, mind to close the issue as don't think there's any action from our side. Thank you

Closing per above. Please reopen if still issues. Thank you.


Support next/prev function with the behavior that step is skipped with TargetNotFound

This is not a bug.
This function will improve vue-tour behavior when the tag element is not founded.
I use next/prev override function like below.
How do you think it supported in vue-tour with selectable option?
    nextStepTour(currentStep) {
      for (var i = currentStep + 1; i < this.steps.length; i++) {
        const target_element = document.querySelector(this.steps[i].target);
        if (target_element) {
          if (i == currentStep + 1) {
            break;
          } else {
            const tour = this.$tours["home"];
            this.$nextTick(() => {
              tour.currentStep = i;
            });
            break;
          }
        }
      }
    },
    previousStepTour(currentStep) {
      for (var i = currentStep - 1; i > -1; i--) {
        const target_element = document.querySelector(this.steps[i].target);
        if (target_element) {
          if (i == currentStep - 1) {
            break;
          } else {
            const tour = this.$tours["home"];
            this.$nextTick(() => {
              tour.currentStep = i;
            });
            break;
          }
        }
      }
    }

Maybe we need to add additional codes to check last step to change buttons showing.

@quroom how do you account for the events that are meant to be triggered before the step.
For example: if I assign a click event to a dropdown before highlighting the dropdown so that i could highlight it's active state but if I use these custom navigation functions then it would fail to find the element in the dom and skips that step. Dom would only see that element when it is clicked.


training step generative models

train_step() currently only updates self.generator weights and does not include output layer

taken care of by latest merge for cgan model


[A11y] Screen reader announces User message twice when using Directline

When using directline, Announcer is reading user message twice before proceeding to bot message. This behavior is noticed only when using directline. On emulator, screen reader reads user message only once then continues with bot message.
I have a feeling that this could be related to the sequence of how activityMiddleware processes activities. Below are some list of activityId's taken from each iteration of activityMiddleware both with directline and using emulator.
emulator:
empty
22527030-dae7-11ea-a51d-a5f7df81d13c -- previous bot activity
24e5c540-dae7-11ea-a51d-a5f7df81d13c -- user activity
22527030-dae7-11ea-a51d-a5f7df81d13c
24e5c540-dae7-11ea-a51d-a5f7df81d13c
22527030-dae7-11ea-a51d-a5f7df81d13c
2575f750-dae7-11ea-a51d-a5f7df81d13c -- bot response from user activity
259406a0-dae7-11ea-a51d-a5f7df81d13c
2575f750-dae7-11ea-a51d-a5f7df81d13c
24e5c540-dae7-11ea-a51d-a5f7df81d13c
directline:
empty
IvBPoaBsTYE6vz49XhrjY1-h|0000011 -- previous bot activity
IvBPoaBsTYE6vz49XhrjY1-h|0000012  -- user activity
IvBPoaBsTYE6vz49XhrjY1-h|0000011
IvBPoaBsTYE6vz49XhrjY1-h|0000013 -- bot response from user activity
IvBPoaBsTYE6vz49XhrjY1-h|0000014
IvBPoaBsTYE6vz49XhrjY1-h|0000013
IvBPoaBsTYE6vz49XhrjY1-h|0000012
IvBPoaBsTYE6vz49XhrjY1-h|0000012
IvBPoaBsTYE6vz49XhrjY1-h|0000011
I am trying to understand why there is a different sequence of which activities are processed in middleware when using directline and emulator and is there any possibility to influence this sequence? And If the fact that previous bot activity is on the last iteration could make announcer read user message twice?

Azure Bot Services is a distributed system and subject to network latencies and various factors. That means, activity going in and out may not be in exact order. We do sort activities using timestamp. All activities are timestamped when they arrive on the Azure Bot Services.
Emulator, in contrast, is emulating Azure Bot Services. The emulated service is locally-hosted and not distributed. Thus, activity you see here should be in an exact order as the bot send. This is because everything is local and not subject to any network latencies.
Given that, if activities are sending fast enough through Azure Bot Services. You might see activities going out-of-order.
SDK might have ways to help you queue the outgoing activities. I am not sure as I don't work on the SDK. But you can post on their GitHub repo and ask.
Are there any accessibility specific issues here?

Thanks @compulim , I think what you said about "Given that, if activities are sending fast enough through Azure Bot Services. You might see activities going out-of-order." should be what is causing our accessibility issues.
The issue with accessibility is that this causes user message to be read twice by assistive technologies if activities are received too quickly. I was setting delays for bot messages from service side and announcer was reading messages properly.
However, purposefully slowing chat would not be desirable. Is there any way to prevent announcer from reading messages twice in this case?

@agriniex we cannot reproduce this, and neither have some large customers of web-chat using the control in a very distributed, high throughput scenario.
Can you update your version of the control to the latest and see if the issue continues to persist?

We cannot reproduce this issue as well anymore, though there has been quite a lot of changes from our side, might be that some of those changes fixed this issue. Closing this


[SPARK-45042][BUILD][3.5] Upgrade jetty to 9.4.52.v20230823

What changes were proposed in this pull request?
The pr aims to Upgrade jetty from 9.4.51.v20230217 to 9.4.52.v20230823. (Backport to Spark 3.5.0)
Why are the changes needed?

This is a release of the https://github.com/eclipse/jetty.project/issues/7958 that was sponsored by a support contract from Webtide.com

The newest version fix a possible security issue:
This release provides a workaround for Security Advisory https://github.com/advisories/GHSA-58qw-p7qm-5rvh

The release note as follows:
https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.52.v20230823

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @LuciferYang

cc @srowen

Merged to 3.5


Remove IOptionsFactory new constraint

Why was IOptionsFactory ever released with a new() constraint on the interface?!
It kills a lot of useful other construction patterns where you might not have an empty constructor for a good reason, (which is why you'd want a factory interface in the first place)
And it also makes it painful to use with F# records (which don't have an empty constructor).
I'm not sure why this constraint wasn't put onto the IServiceCollection.Configure method type argument?
Could anybody explain?

Well the original IOptions had a new() constraint, and the default implementation of the OptionsFactory is using new().  The factory wasn't really meant changing how TOptions instances are created (which is always done via new), but more for applying all of the configurations to the instance.

Its main job is to apply these:
OptionsFactory(IEnumerable<IConfigureOptions<TOptions>> setups, IEnumerable<IPostConfigureOptions<TOptions>> postConfigures, IEnumerable<IValidateOptions<TOptions>> validations)

Would there still be some room to re-evaluate this?
Like either removing the constraint from the interface and moving it up to the Configure method.
Or possibly detect another, unconstrained, factory interface during options creation?
You would make a lot of people with uncommon use cases happy with such a change, and the F# community would love it too.

@haok ping.
I'm willing to drive this break too, investigate breaking places, fix the code, write up a breaking change issue etcetera. But 3.0 is the best time to remove this constraint if possible.

Feel free to give it a shot so we can see what it looks like, but no guarantees this will be something we take.

Addressed by https://github.com/aspnet/Extensions/pull/2169

@anurse this is considered a 5.0 change since it was merged to master right?

Correct. I put it in the placeholder 5.0.0-alpha1 milestone (just to track that it was in before any public preview releases), which isn't a committment to any specific release, just a placeholder until the schedule starts to take shape :).

@anurse did you get more info on backporting to 3.0?

I'll run it up the flagpole but a change like this to a public API will be challenging. Do you have specific scenarios in production applications that don't work and for which there is no workaround? That's probably what we're going to need to make an API change (especially since I believe this is a breaking change to an interface) in 3.0.

Do you have specific scenarios in production applications that don't work and for which there is no workaround?
We don't, possibly people want to use types that they don't control and which don't have an empty constructor.

(especially since I believe this is a breaking change to an interface)
I always thought removing constraints was non breaking?

but a change like this to a public API will be challenging
That saddens me a bit, I would have loved to have it in for 3.0. But, and I think this is a fair point of critique, you all went rather radio silent on this repo for months when 3.0 was still open for changes.

I always thought removing constraints was non breaking?

Yep, I suspected that was the case too but wasn't sure. I've confirmed it is indeed non-breaking. Removing the constraint is generally fine, my concern was how it would affect implementations. It looks like it's safe, after exploring the scenarios in SharpLab

That saddens me a bit, I would have loved to have it in for 3.0. But, and I think this is a fair point of critique, you all went rather radio silent on this repo for months when 3.0 was still open for changes.

Yep, definitely a fair observation. We're a small team with a lot of code to manage and things get prioritized (and deprioritized) as part of that. I don't disagree that this repo has been quiet for a while though, and that's something we're looking to improve.
Regardless of how we got here, we still have to balance the risk of any change in 3.0 (there's always non-zero risk) against the customer value it delivers. A lot of that is driven by having a clear understanding of customer scenarios that are blocked by the issue and the value of taking the backport.


[Python][Style] Autopep8 + Flake8

Changes

Add excluded files to .flake8: dist and build
Change  .pre-commit-config.yaml to take the flake8 configuration from .flake8

Example

Note: autopep8 will not fix issues that have to do with naming conventions as that may end up breaking the code. Those changes will have to be done manually.

@Aliandi my understanding is that re-enabling autopep8 won't cause any problems before we fix the remaining style issues, right?
Also, can we use a config file (pycodestyle) in the pre-hook trigger like the flake8 case? Feel free to rename both config files.
Thanks!

@tellarin it wouldn't cause any problems as it only changes the staged files!
Also, yes, we can. We are adding a setup.cfg file to the root of the Python folder, following this documentation where we unify all the configurations related to style

After this PR, would it be possible to create a PR to only restyle all the python code (using the autopep8)? I did see some mixed styles in the code, in terms of line length limits and indentations.

@Aliandi, we should also delete the pycodestyle file now, right? Does a PR with he current flake8/autopep8 passes (i.e., it ignores those warning marked in config and don't raise others)?
@STayinloves, PR #1668 already applied the automatic autopep8 changes. But as @Aliandi says, some changes still need to be manually made.

@tellarin yes, we've just deleted it. Also, the PR passes as the build is not configured to fail on flake8 or autopep8 issues. If you look at the AppVeyor build, you'll se that flake8 is being run and detecting the current naming issues. We'll create another PR fixing them manually to merge before this one

@Aliandi Great!  Merging it.


Not able to serve gzip version of static resources over OC

I am trying to serve gzip versions of static resources (javascript and css files) in my index.html over OC and it does not seem to quite pick up/serve the gzip versions of the file.
Considering the advantages of serving gzip versions of files (Javascript/css etc), i think it is one of the must have features that every front end developer would like to have in his project.
Kindly share your thoughts.

Not sure if i have done a good job on articulating the problem properly in above comments,
For Example:
If i have a link tag in my template.html as below,

and if my s3 Bucket has below files available,
css/sample.css
css/sample.cssgz (gzipped file)
Does OC retrieve the gzipped version of the file. ?

Uhm, I think OC automatically handles css.gz and js.gz. Maybe the problem is that you are using the cssgz extension.
Tests: https://github.com/opentable/oc/blob/9f0d07b6c22f4331b2e6a7c02dfab3ff66c39c04/test/unit/utils-get-file-info.js#L25-L55

I did try out one workaround on this,
i uploaded the gzipped version of the file on s3 Bucket with below configuration and it retrieved the gzipped version of the file for me,
Content Type: text/javascript
Content Encoding: gzip
But ideally in a environment where a web server is used to serve static resources, the server configuration would take care of compression and retrieves the gzipped version of the resource if already compressed.
Am i missing anything over here ??

Ok ok, let me clarify.
The test I linked is to show that if you have a build pipeline for making a css.gz file yourself and you put it in the static asset folder, that will be uploaded and when referenced from markup, it should work.
On the other hand, that is something that we have in our codebase from the old ages (when the cli allowed you to automatically do the gzipping) and we stopped supporting it. The reason, is that we found out it's much simpler to delegate such things to a CDN that sits on top of S3.
In our case and most of the people using OC in prod, a CDN is necessary anyways to distribute the content in multiple regions, and modern CDNs like akamai or cloudfrount (which you can setup very easily on top of your s3 account) support http2, which gzips all by default.
So, the way we do this in production, we setup the cdn url on top of oc configuration for OC (which almost serves as a proxy on top of S3) and then we just reference the resources with the real extensions. The cdn will take care of serving gzip if the browser supports, and do all the magic for us.
I hope it makes sense. Are you planning to use a CDN on top of your S3 account?

I hope it makes sense. Are you planning to use a CDN on top of your S3 account?

Not anytime soon :(
Are there any better workarounds ??

So, let's go back a couple of steps.
You have a .css file, and you are referencing it in your component, correct?
My question is. If you manually gzip it, and then reference it as my-file.css.gz in your component's markup,

does it work on dev?
does it work when published?

Because if we establish that it works, than the best workaround is to setup a task that does the gzipping automatically with a watcher or something like that. If it doesn't work, we may have some issues to fix here.

One pre-requisite functionality that needs to be there in oc for any of the approaches mentioned above to work, is the ability to set the metadata parameters (content-encoding and content-type) of the files/folders that get uploaded into s3 bucket with oc publish.
Of course there is a workaround for this - a user can manually do it in s3 bucket.
I am thinking about the production environment here.
Any thoughts ??

@NapoleanReigns the current strategy is to delegate this to automation by following some conventions. Any filename ending in .gz has content-encoding set to gzip and the content-type is defined by its extension (for instance .css.gz gets 'text/css').
This is why I asked in my previous comment if you could do that attempt and tell me if that works. I think conventions make life easier as they bring simplicity to the table, with inherited consistency.

Hey @matteofigus sry for getting back late on this one, was pretty held up with other tasks.
I tried the above,

Compressed (pretty huge) javascript files with extension .js.gz
Referred the same in my import statements with same extension .js.gz

and it worked. :)
Awesome. Thanks

Cool stuff. Is it anything else we can do, or should we close this issue?

I think we can close this issue, unless we want to keep this open for discussion to address other possibilities/issues w.r.t serving files in gzip format over OC.
Your call.
Thank you.


Ensure that sample web apps have endpoints and corresponding tests

In order to ensure that web applications in spring-graalvm-native-samples are testable and properly tested (at least with a simple in-container smoke test), we should do the following.

Ensure that each web app has at least one web endpoint.
Ensure that an in-container integration test is in place.

An "in-container integration test" is a test class annotated with @SpringBootTest(webEnvironment = WebEnvironment.RANDOM_PORT) -- instead of simply @SpringBootTest -- with at least one @Test method that connects to and validates the response from an endpoint via  TestRestTemplate or WebTestClient.
Launching the embedded container (e.g., Tomcat) also ensures that the type of ApplicationContext used in the integration test is the same as the one used when the application is launched within a native image, which helps the GraalVM native image JVM agent to better analyze what will be needed in the compiled native image.
For example, springmvc-tomcat already fulfills requirement # 1 due to the existence of the Controller; however, AppContextStartupTest does not fulfill requirement # 2 since it does not start the embedded Tomcat container or test the Controller endpoint over HTTP.

@sbrannen Any chance you could take care of it?

I'll see if I can fit this in next week.

Let's move that on to 0.9.1 which will have a focus on tests.

Spring Native is now superseded by Spring Boot 3 official native support, see the related reference documentation for more details.
As a consequence, I am closing this issue, and recommend trying your use case with latest Spring Boot 3 version. If you still experience the issue reported here, please open an issue directly on the related Spring project (Spring Framework, Data, Security, Boot, Cloud, etc.) with a reproducer.
Thanks for your contribution on the experimental Spring Native project, we hope you will enjoy the official native support introduced by Spring Boot 3.


GROOVY-7241

Hi Keegan,
thanks for the pull request. :+1:
I would have merged it already, but I did hold of awaiting the imminent creation of your apache account, which will allow you to merge it yourself.
-Pascal

I kinda figured that was the case.  It's cool.

This was actually merged with c3a802fe13a6fc7267130db805940bd2be41b4f8.  I messed up the commit message.


Fix link on main page readme to get to tutorial

Previous tutorial name was still present.

Apologies, just looked at a number of the commits. It would likely be more appropriate to rename the clus tutorial to castor from what I can see.
Either way, very interesting program you have here, I got here looking for VRI to FBP functionality!

Thanks!


⚠ Bofrost.de has degraded performance

In 1308a5e, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1174 ms

Resolved: Bofrost.de performance has improved in c9641d1.


Unable to install rmagick on Mac OS X Yosemite (10.10.2)

From a fresh install, I used homebrew to install rbenv, ruby-build and ImageMagick.
Used rbenv to install ruby, tried with 2.0.0-p598 and 2.2.0, without any luck.
Willing to give more information if necessary, I'm stuck at the moment.
administrators-Mac-mini:~ administrator$ rbenv version
2.0.0-p598 (set by /Users/administrator/.rbenv/version)

administrators-Mac-mini:~ administrator$ Magick-config --version
6.9.0 Q16

administrators-Mac-mini:~ administrator$ sudo gem install rmagick
Building native extensions.  This could take a while...
ERROR:  Error installing rmagick:
	ERROR: Failed to build gem native extension.

    /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby extconf.rb
======================================================================
Tue 10Feb15 10:31:52
This installation of RMagick 2.13.4 is configured for
Ruby 2.0.0 (universal.x86_64-darwin14) and ImageMagick
======================================================================

make "DESTDIR="
compiling rmagick.c
compiling rmdraw.c
rmdraw.c:315:15: warning: comparison of constant 100 with expression of type 'WeightType' is always true [-Wtautological-constant-out-of-range-compare]
        if (w < 100 || w > 900)
            ~ ^ ~~~
rmdraw.c:315:26: warning: comparison of constant 900 with expression of type 'WeightType' is always false [-Wtautological-constant-out-of-range-compare]
        if (w < 100 || w > 900)
                       ~ ^ ~~~
rmdraw.c:1878:17: warning: 'AllocateImage' is deprecated [-Wdeprecated-declarations]
        image = AcquireImage(info);
                ^
./rmagick.h:176:28: note: expanded from macro 'AcquireImage'
#define AcquireImage(info) AllocateImage(info)
                           ^
/usr/local/Cellar/imagemagick/6.9.0-3/include/ImageMagick-6/magick/deprecate.h:127:4: note: 'AllocateImage' has been explicitly marked deprecated here
  *AllocateImage(const ImageInfo *) magick_attribute((deprecated)),
   ^
3 warnings generated.
rmdraw.c:315:15: warning: comparison of constant 100 with expression of type 'WeightType' is always true [-Wtautological-constant-out-of-range-compare]
        if (w < 100 || w > 900)
            ~ ^ ~~~
rmdraw.c:315:26: warning: comparison of constant 900 with expression of type 'WeightType' is always false [-Wtautological-constant-out-of-range-compare]
        if (w < 100 || w > 900)
                       ~ ^ ~~~
rmdraw.c:1878:17: warning: 'AllocateImage' is deprecated [-Wdeprecated-declarations]
        image = AcquireImage(info);
                ^
./rmagick.h:176:28: note: expanded from macro 'AcquireImage'
#define AcquireImage(info) AllocateImage(info)
                           ^
/usr/local/Cellar/imagemagick/6.9.0-3/include/ImageMagick-6/magick/deprecate.h:127:4: note: 'AllocateImage' has been explicitly marked deprecated here
  *AllocateImage(const ImageInfo *) magick_attribute((deprecated)),
   ^
3 warnings generated.
compiling rmenum.c
rmenum.c:450:13: warning: 5 enumeration values not handled in switch: 'DivideSrcCompositeOp', 'MinusSrcCompositeOp', 'DarkenIntensityCompositeOp'... [-Wswitch]
    switch (op)
            ^
rmenum.c:565:13: warning: enumeration values 'LZMACompression', 'JBIG1Compression', and 'JBIG2Compression' not handled in switch [-Wswitch]
    switch (ct)
            ^
rmenum.c:674:12: warning: 9 enumeration values not handled in switch: 'SincFastFilter', 'LanczosSharpFilter', 'Lanczos2Filter'... [-Wswitch]
    switch(type)
           ^
rmenum.c:922:12: warning: 5 enumeration values not handled in switch: 'Average9InterpolatePixel', 'Average16InterpolatePixel', 'BlendInterpolatePixel'... [-Wswitch]
    switch(interpolate)
           ^
4 warnings generated.
rmenum.c:450:13: warning: 5 enumeration values not handled in switch: 'DivideSrcCompositeOp', 'MinusSrcCompositeOp', 'DarkenIntensityCompositeOp'... [-Wswitch]
    switch (op)
            ^
rmenum.c:565:13: warning: enumeration values 'LZMACompression', 'JBIG1Compression', and 'JBIG2Compression' not handled in switch [-Wswitch]
    switch (ct)
            ^
rmenum.c:674:12: warning: 9 enumeration values not handled in switch: 'SincFastFilter', 'LanczosSharpFilter', 'Lanczos2Filter'... [-Wswitch]
    switch(type)
           ^
rmenum.c:922:12: warning: 5 enumeration values not handled in switch: 'Average9InterpolatePixel', 'Average16InterpolatePixel', 'BlendInterpolatePixel'...
Results logged to /Library/Ruby/Gems/2.0.0/gems/rmagick-2.13.4/ext/RMagick/gem_make.out

I guess it might be architecture mismatch.
Try this command:
ARCHFLAGS="-arch x86_64" gem install rmagick

If not solved, what I need from you:

mkmf.log
What version of rubygems (gem -v) you're using
What version of the developer tools (clang -v)

It worked.
Thanks a lot.


Fails to build after install

Hi,
Just tried installing this in the root of my ember-cli project,  with npm install --save-dev ember-masonry-grid and getting the following when running ember server:
Path or pattern "bower_components/jquery-masonry/dist/masonry.pkgd.min.js" did not match any files

Is there something else I need to do?
Update:
Surprise, surprise; bower install jquery-masonry --save fixed that.  I've added a blueprint to install the bower dependencies by default, so running ember install:addon ember-masonry-grid will setup the addon entirely. Thanks for pointing out the issue!

No worries, happy to have helped!


Easy way to access exported variables from script editor

If I'm working on the script editor sometimes I need to jump back and forth between a section of code and the top where all exported variables are declared.
I can click on the node from the hierarchy of nodes (Scene panel) which will also bring the 3D scene editor, then make the changes to the exported variables and open the script editor again, although that's a bit complicated.
Also from the script editor if one clicks on the script button to the right of the current node on the Scene panel, it'll show the script's attributes with the resource and reference attributes, so that won't show the exported variables fro the attributes inspector.
What do you think guys?
By the way, I decided to post these kind of suggestions here, before adding features is closed, but I can post them at the forums instead, the thing is that I'm still not familiar with how adding features is handled.
Any suggestion or comment is appreciated,
Cheers!

There is the "outline" list of functions on the left panel to jump back and forth, maybe a "top" item could be added, like docs?

@Zylann, there are couple of issues with the way the exported vars work, maybe it's just me, but I stopped using those to quickly update changes in order to test some piece of code. A "top" item would help too, although I'm using the keyboard shortcut CMD+UP for that, thanks.


Fetching current expiry value from redis

Is it somehow possible to fetch the current expiry time in seconds from the key/value stores such as Redis?
Redis provides a method called TTL to get such information.

Hi,
I think at present the only way to do this at present is to access the backend directly and do the ttl query on the underlying Redis object.  You can do this with store.backend or store.adapter.backend (the latter most likely).  Unfortunately this means that you will have to do any key transformation manually, so it's not a great solution.
We could also look at adding an expiry query to the Moneta interface.  This would depend on our being able to implement the feature on other backends.

A roadblock for this is that memcached (one of the main supported stores with native expires support) has no equivalent of Redis' ttl command: https://github.com/memcached/memcached/issues/239


Crashed under heavy load.

System.InvalidOperationException: No writing operation. Make sure GetMemory() was called.
at System.IO.Pipelines.ThrowHelper.ThrowInvalidOperationException_NotWritingNoAlloc()
at System.IO.Pipelines.Pipe.DefaultPipeWriter.Advance(Int32 bytes)
at System.IO.Pipelines.Networking.Libuv.UvTcpConnection.OnRead(UvStreamHandle handle, Int32 status)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.ThreadPoolWorkQueue.Dispatch()
looks like either uv_alloc is not called or uv_read_cb returned incorrect status.
git commit hash:	0b605e257632eb260b5e377574fd902c61854013

The libuv and sockets prototypes in here are just prototypes, there are no plans to take them any further. Future work in this area will revolve around this https://github.com/aspnet/KestrelHttpServer/issues/1980

Closing per @davidfowl's answer.


author order

Did we talk about this earlier? I don't recall and can't find it in the issues, but if I missed something feel free to ignore this.
As far as I know, conventions for author order are field-dependent. In my field (astronomy), author order usually reflects descending order of contribution; names are alphabetized if there's no distinction between the amount of contribution, and last author generally doesn't mean anything special.
If we want to order by contributions then should we just count commits? Lines of manuscript? I think @emhart should be first since he's been coordinating. Looking at the commit log, @gvwilson made a commit that did end up in the final text but is not currently listed as an author.
If we don't want contribution order, I'm perfectly happy with alphabetical, reverse alphabetical, alphabetical-by-middle-initial, or whatever other scheme anyone can dream up.

I am fine with whatever order works for others, as long as @emhart is first. In ecology, last author is usually senior author (PI), with the rest of the order done by descending degree of contribution.

So going by number of lines added, I get

Hart
Hollister
Michonneau
Zimmerman
Poisot
LeBauer
Mount
Barmby
Woo
Cornwell
Falster
Turner
Wilson

Probably don't want me top three.  I'd have to pass the paper through
internal review prior to submission or pushing up to a pre-print server.
That could add 6 weeks (at best).
On Oct 17, 2015 7:21 PM, "Timothée Poisot"<EMAIL_ADDRESS>wrote:

So going by number of lines added, I get

Hart
Hollister
Michonneau
Zimmerman
Poisot
LeBauer
Mount
Barmby
Woo
Cornwell
Falster
Turner
Wilson

—
Reply to this email directly or view it on GitHub
https://github.com/emhart/10-simple-rules-data-storage/issues/92#issuecomment-148960220
.

Hi all, I reached out to Greg about authorship yesterday and asked if he wanted a spot, and he declined.
Falster, Turner and Corwell all just add their contact info to the readme, but didn't actually commit anything to the text. (e.g. https://github.com/emhart/10-simple-rules-data-storage/commits?author=stephenturner)  So they were never added to the list of authors.
So going by @tpoisot list (and moving @jhollist around) it would look like this.  I'll make this change unless anyone objects.

Hart
Michonneau
Zimmerman
Hollister
Poisot
LeBauer
Mount
Barmby
Woo

While the idea of using an 'objective' quantification of contribution is admirable, I respectfully propose that 'lines of code' is not a meaningful metric. Based on 'lines of code', this is how different contributions are ranked:

contribution
lines of code value

initiate effort
0

propose rule
0

comment on issue
0

add paragraph
1

fix typo
1

add name as author
1

add paragraph with line breaks
5

add citation to .bib file
10

add figure
1000

add template files
3000

I don't know if there is any existing protocol for using such a metric, but I do not believe that this is a valid, and would prefer almost any existing rubric for ranking authorship. If we do do this, we should have a clear statement of contributions and the analysis used to rank the contributors.
My suggestion would be to follow the convention of lead author followed by an alphabetical list (the least contentious). And we should add Greg to the acknowledgements (the effort traces back to his swcarpentry/site#797 so I would otherwise suggest listing him last as senior author).
It is not even clear how the 'lines of code' metric is computed. Does it include pull requests that were not merged? Is it based on the Github Graphs?

Or perhaps git-blame?:
git fame --bytype

+----------------------+-------+---------+-------+--------------------+---------+-----+-----+-----+------+-----+-----+
| name                 | loc   | commits | files | distribution       | unknown | md  | bib | csl | bst  | tex | eps |
+----------------------+-------+---------+-------+--------------------+---------+-----+-----+-----+------+-----+-----+
| Jeff Hollister       | 2,843 | 5       | 8     | 59.6 / 3.1 / 57.1  | 8       | 53  | 19  | 316 | 1882 | 565 | 0   |
| Francois Michonneau  | 1,014 | 35      | 4     | 21.3 / 21.7 / 28.6 | 0       | 173 | 45  | 0   | 0    | 0   | 796 |
| Ted Hart             | 350   | 11      | 2     | 7.3 / 6.8 / 14.3   | 0       | 13  | 337 | 0   | 0    | 0   | 0   |
| Edmund Hart          | 212   | 43      | 3     | 4.4 / 26.7 / 21.4  | 116     | 96  | 0   | 0   | 0    | 0   | 0   |
| Timothée Poisot      | 146   | 8       | 2     | 3.1 / 5.0 / 14.3   | 0       | 146 | 0   | 0   | 0    | 0   | 0   |
| snim2                | 60    | 10      | 5     | 1.3 / 6.2 / 35.7   | 31      | 7   | 22  | 0   | 0    | 0   | 0   |
| Jeffrey W. Hollister | 52    | 1       | 2     | 1.1 / 0.6 / 14.3   | 0       | 34  | 18  | 0   | 0    | 0   | 0   |
| Naupaka Zimmerman    | 47    | 10      | 2     | 1.0 / 6.2 / 14.3   | 1       | 46  | 0   | 0   | 0    | 0   | 0   |
| Kara Woo             | 13    | 7       | 4     | 0.3 / 4.3 / 28.6   | 0       | 8   | 1   | 0   | 0    | 4   | 0   |
| David LeBauer        | 13    | 9       | 1     | 0.3 / 5.6 / 7.1    | 0       | 13  | 0   | 0   | 0    | 0   | 0   |
| Pauline              | 10    | 4       | 2     | 0.2 / 2.5 / 14.3   | 0       | 1   | 9   | 0   | 0    | 0   | 0   |
| Sarah Mount          | 6     | 5       | 2     | 0.1 / 3.1 / 14.3   | 0       | 1   | 5   | 0   | 0    | 0   | 0   |
| wcornwell            | 1     | 2       | 1     | 0.0 / 1.2 / 7.1    | 0       | 1   | 0   | 0   | 0    | 0   | 0   |
| Stephen Turner       | 1     | 1       | 1     | 0.0 / 0.6 / 7.1    | 0       | 1   | 0   | 0   | 0    | 0   | 0   |
| Pauline Barmby       | 1     | 3       | 1     | 0.0 / 1.9 / 7.1    | 0       | 1   | 0   | 0   | 0    | 0   | 0   |
| Daniel Falster       | 1     | 1       | 1     | 0.0 / 0.6 / 7.1    | 0       | 1   | 0   | 0   | 0    | 0   | 0   |
| Greg Wilson          | 0     | 1       | 0     | 0.0 / 0.6 / 0.0    | 0       | 0   | 0   | 0   | 0    | 0   | 0   |
| Jeffrey W Hollister  | 0     | 4       | 0     | 0.0 / 2.5 / 0.0    | 0       | 0   | 0   | 0   | 0    | 0   | 0   |
| karawoo              | 0     | 1       | 0     | 0.0 / 0.6 / 0.0    | 0       | 0   | 0   | 0   | 0    | 0   | 0   |
+----------------------+-------+---------+-------+--------------------+---------+-----+-----+-----+------+-----+-----+

Also note that using 'lines of code' to rank authors would potentially require reordering after each revision. As a plus, this could be built into the makefile :8ball:

Seems there are probably many ways to quantify contributions, but my sense is that any is probably ok, as long as Ted is first, and Jeff is not top 3. There's also good 'ol sample() :game_die:

Since there's some dispute about how best to quantify effort let's go with alphabetical order.  @dlebauer as far as @gvwilson this is the text from the beginning of the acknowledgements:

We would like to thank G. Wilson and the Software Carpentry instructor community for providing the initial inspiration for this paper and the discussion held at https://github.com/swcarpentry/site/issues/797.

Feel free to amend that if you'd like to change the wording, but I wanted to thank Greg, all SWC instructors and reference the initial discussion thread in our manuscript.
@gvwilson It's still not too late if you want to be senior author :wink:


May it have affected the test results？

No this technology does not affect the testing process. It only creates an HTML reports based on the already existing test results.


Create SurveyResponse when starting a session

Resolves https://github.com/rubyforgood/inkind-volunteer/issues/50
WIP
Description
It creates a new SurveyResponse when a user clicks on the "Begin Survey" button. This SurveyResponse will be used to save the survey answers.
Type of change

New feature (non-breaking change which adds functionality)

Screenshots

Awesome work! Thank you for this contribution!


Create habitrpg task when doesnt exist

When the github task does not exist - create it.
Part of the actual score task process takes in a task object so should be relatively trivial.

This is now done during the setup process.


Some quick observations on site

Shouldnt the top right Icon by CL icon (instead of OBO) as per website convention to go back to front page?
Its not ideal that the CL download link points to a raw GitHub URL instead of a purl, but I get way (https problems). Imagine we switch to GitHub release due to file sizes.
The query should redirect to an actual query on ubergraph, e.g.: https://api.triplydb.com/s/eiTP_j7_M
The favicon should be the CL logo?
"Ontology Tools Integrated" - could be better: Integrated in standard ontology tools (you can also list OAK)
"Widely Compatible" - (cell ontology has wrong capitalization (consistent Cell Ontology). Also later on the site in "applications"
Rename "Widely compatible" to "Distributed in standard formats". Also, I would mention that owl (RDF/XML) as this is very important for people using the RDF stack
I would prefer "Applications" to be "Users" or "Uses" because the word is so ambiguous
The logo in "direct contact" is skewed
Cell Ontology Documents should be just "Documentation" or "Developer Documentation"
Dont know if the link to OBOOK is useful here if it is so general; less is better. The CL documentation should link to OBOOK in the appropriate places
Same with ODK - all the relevant things for CL are already in the CL documentation
I would prefer if we do not add a blog. I can already promise to you it will not be maintained and just make us look out of date.
In footer add link to provide feedback on the website (issue tracker)
Foundry page in header menu should be "CL in OBO Foundry" or "OBO Foundry Page" or some such. Foundry is not really widely understood

@shawntanzk As always, I don't really insist on anything. Feel free to pick and chose from the feedback and then close the issue. Very well done all!

Thanks @matentzn will look into them sometime

Shouldnt the top right Icon by CL icon (instead of OBO) as per website convention to go back to front page?

This was me thinking ahead for template splash page for obo foundry ontologies. We already have a back to top button and we don't have multiple page so I figured that icon would be best used to bring back to obo foundry - that way when distributed widely, it all links to obo foundry.

Its not ideal that the CL download link points to a raw GitHub URL instead of a purl, but I get way (https https://github.com/OBOFoundry/purl.obolibrary.org/issues/705). Imagine we switch to GitHub release due to file sizes.

was trying to get it to just download as an owl file, will leave it for now given the issue you mentioned (don't understand it fully but I'm guessing its alright for now)

The query should redirect to an actual query on ubergraph, e.g.: https://api.triplydb.com/s/eiTP_j7_M

[ ] To fix

The favicon should be the CL logo?

[ ] To fix (thought about this but slipped my mind when doing stuff)

"Ontology Tools Integrated" - could be better: Integrated in standard ontology tools (you can also list OAK)

[ ] To fix

"Widely Compatible" - (cell ontology has wrong capitalization (consistent Cell Ontology). Also later on the site in "applications"

[ ] To fix - will scan through all and make sure all are done as Cell Ontology

Rename "Widely compatible" to "Distributed in standard formats". Also, I would mention that owl (RDF/XML) as this is very important for people using the RDF stack

[ ] To fix

I would prefer "Applications" to be "Users" or "Uses" because the word is so ambiguous

[ ] To fix

The logo in "direct contact" is skewed

[ ] Will try to fix (not 100% sure I can figure this out cause I'm using their old code and just swapping symbol)

Cell Ontology Documents should be just "Documentation" or "Developer Documentation"

[ ] To fix

Dont know if the link to OBOOK is useful here if it is so general; less is better. The CL documentation should link to OBOOK in the appropriate places
Same with ODK - all the relevant things for CL are already in the CL documentation

[ ] Will fix, ill be honest, I was just populating things cause i like the footer and it looked empty without stuff LOL

I would prefer if we do not add a blog. I can already promise to you it will not be maintained and just make us look out of date.

[ ] Will remove

In footer add link to provide feedback on the website (issue tracker)

[ ] To fix

Foundry page in header menu should be "CL in OBO Foundry" or "OBO Foundry Page" or some such. Foundry is not really widely understood

[ ] To fix


MSE and MP4 based HLS streams render scaled down with a green backdrop on Safari and iOS apps

Using go2rtc Home Assistant add-on 1.9.2. Input streams are RTSP via Home Assistant camera entities. WebRTC renders well on all browsers and MSE/HLS render normally on Firefox and Chrome. On Safari and the Home Assistant iOS app, however, when selecting MSE or HLS, the video renders at about a quarter-scale size with a green background. This is reproducible with all MP4 formats, including MP4 snapshot, but not Legacy HLS/TS.
Home Assistant's built in HLS streaming from the Generic Camera RTSP source does not exhibit the issue.
See Safari screenshots below for examples. For these, I am manually selecting the stream type via the go2rtc Web UI:
WebRTC

MSE:

Modern HLS (MP4):

Legacy HLS TS

Home Assistant native HLS from the same camera entity:

I think it's camera related problem. I need access to stream or maybe mp4 record will be useful
https://alexxit.github.io/go2rtc/api/#tag/Consume-stream/paths/~1api~1stream.mp4?src={src}/get

Here is a five second clip pulled via /api/steam.mp4.
Interestingly, when this file is opened in Firefox or Chrome, it displays at the expected resolution from the DVR (944x480) If I open it in QuickTime Player or VLC locally, it seems to be a full HD 1920x1080 file with the extra space filled in with the green background.
If I open the RTSP stream directly from the DVR in VLC (same RTSP URL that go2rtc is pulling from), the resolution is recognized correctly as 944x480.
https://github.com/AlexxIT/go2rtc/assets/6099938/96d5ed07-ddfc-4833-bb09-c986b7127ed5

I have the same issue with my iPad when viewing the camera using the webrtc-camera custom card. It only started when I changed my encoding from H264 to H265. This was on UniFi Protect’s latest update that allowed you to run a new enhanced mode. It was working fine before that and the 2 cameras that were incompatible with the enhanced mode are still working fine. I use the same card on my desktop pc  and the image is fine. I think it is some weird apple rendering thing, as the picture glance card from the same camera also displays fine.
Hope this info helps


Training code

Do you have plans to release your training code?

Thanks for your intest. Currently, we have no plan to release trainning code, but focus more on more stable model inference.


Fix for CompilerWarningMessageParsingUniTest

@EBatTiVo
this fixes how warning messages are being parsed & corresponding unit test

Approved for merge.  However, line 89 is the reason that we aren't parsing errors properly.  It needs to be the last "else if" clause, placed at line 111.  I think that will fix a couple of outstanding bugs, too.  If you've got a minute to figure that out and test it, it would be great.  If not, well, we'll get to it when we get to it.

That is, if the comments are correct about what the matchers' regular expressions are.

Is there any sample code that can be used to reproduce the failures in parsing error messages?
It will be better to address those cases as well - within this fix.


Loosen ui.TextStyle toString tests

Fixing tests for https://github.com/flutter/engine/pull/24025. Some framework tests currently depend on the toString implementations of some dart:ui classes: https://api.cirrus-ci.com/v1/task/6162473835298816/logs/test_framework.log
Related issue: https://github.com/flutter/flutter/issues/72521
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I signed the CLA.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

One of the things we should check which this test doesn't check as far as I can tell (but which we broke in the past, which is why we should test it) is whether we have commas between each value in the output (and not at the end). It's very easy to accidentally have output like TextStyle(foo:barbaz: quux) or TextStyle(foo: bar, ).

@Hixie I'll add that to https://github.com/flutter/engine/blob/master/testing/dart/text_test.dart


submodule hack and simple api

This is a kinda hacky bit to get submodules to work--I will update in separate commit with fix for all effects (this is only for a couple to sample on).
I was thinking it would be good to separate this fix into a simple util function (that way if we ever get a better way to do this than using the os path, it's only a fix in one place). But I'm mostly using this PR as a way to direct your attention to the following:
https://github.com/sgoldst3/inyourface-api
Right now it's extremely simple. I just want to know what you think about how it's starting and if it's easy to get it up and running. Let me know if there are things I missed and should add to the readme, etc!
Also I don't necessarily think this PR needs to be merged yet (maybe when I've fixed it up for all the effects?)

I got this working in a similar hacky way using __file__ instead of inspect, over here: https://github.com/yacomink/inyourface/blob/run-it-on-gcp/inyourface/effect/Juggalo.py#L18-L20 and similarly only did it for a couple of effects!
Your start on the API looks great! I would consider doing the google credential and image directory configuration a little bit differently, and passing those values to your process as environment variables. There's some good justification for that pattern in the 12 factor app. (That's also generally an interesting thing to skim when you're designing a new app/api/service). More practically ... it would make it easier to spin it up as a service on most cloud providers.

Ah great! I'm going to take a look at it now and get a PR ready for ya! Do you think it's a good idea to separate this stuff out into two different repos like I have it here?
It seems like the direction we're going is: this repo for the inyourface library proper (to be come pip library <3), and maybe another one for the api/slack app--does that seem reasonable?
Thinking (kinda just malformed musings tbh) about how we could deal with the slack app needing a return object in 3s:
The file names are deterministic right? Are they seeded based on the input params? In which case could we maybe return back the name of the file, before it was properly generated and returned from the google vision api? And furthermore, would having the file name but no file yet make any difference, or would it still cause the same issues?

Yeah, I think splitting the api out into a different repo is a great idea, and seeing that totally sparked the idea of just making this bit into a pip package. Also I was checking in like docker files for getting it working on GCP, and this felt like a bad place for them.
The file names are deterministic, but Slack will try to unfurl them immediately, so I think you'd wind up with a broken looking image if you returned it immediately. I've been thinking about a scheme where the request from slack always just queues a message and returns null, and then there's a worker running that pulls the queued message, makes the image, and does a callback to slack.

Boop boop! there's one thing in here i feel a bit squiggly about and that's the way I'm not using the Animator object function in the masked files since those are global vars and don't have the class's state. Anyway--i can look into that later or if you have a quick fix that's coo. Let me know what you think. As always, happy to change whatever

Looking good! I think you could make get_os_dir a static method (
https://docs.python.org/2/library/functions.html#staticmethod) in Animator
and then you'd be able to call Animator.get_os_dir without having an
instance of the class
On Mon, Dec 11, 2017, 11:49 PM Samantha<EMAIL_ADDRESS>wrote:

Boop boop! there's one thing in here i feel a bit squiggly about and
that's the way I'm not using the Animator object function in the masked
files since those are global vars and don't have the class's state.
Anyway--i can look into that later or if you have a quick fix that's coo.
Let me know what you think. As always, happy to change whatever
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/yacomink/inyourface/pull/6#issuecomment-350944484,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAaxLg-P6JkS-TbtvXr1QIaaI1rCcbzuks5s_gW7gaJpZM4QqS2W
.

yeee perf. Static methods appeared to me in my dream last night, like, why didn't you use this. I'll update in a bit!


dependencyExtractor should consider import 'xxx' statement

🐛 Bug Report
jest's --onlyChanged option does not consider the module in the import 'xxx'; statement as dependencies.
So I can not get the advantage of --onlyChanged or --changedSince
To Reproduce
Consider the following code snippet:
import './dep';

describe('this test depends on "dep"', () => {
  it('but jest does not consider "dep" as dependencies', () => {
    // do something
  })
})

When I modify dep.js, jest --onlyChanged do not detact the change and re-run the test.
Expected behavior
jest --onlyChanged should re-run the test

Fixed in #8670


CSS page property (for printing)

hi
please add the page property (i.e.  not the same as @page)
e.g.
@page wide {size: A4 landscape}
table { page: wide } /* this property  */

cheers
andrew

Firefox/Gecko has issued Intent to prototype

https://twitter.com/intenttoship/status/1602468085210005510
https://groups.google.com/a/mozilla.org/g/dev-platform/c/w-ddRvzGiTI/m/aoNSGNxDBQAJ
https://bugzilla.mozilla.org/show_bug.cgi?id=1723234

"Firefox now supports CSS named pages, allowing web pages to perform per-page layout and add page-breaks in a declarative manner when printing." https://www.mozilla.org/en-US/firefox/110.0/releasenotes/
It seems MDN has not documented this yet, but it appears they are working on it: https://github.com/mdn/content/issues/23682

I have now finished the docs for this, associated PRs:
content
BCD

Available at https://caniuse.com/mdn-css_properties_page (thanks @dletorey!)


htmlunit-cssparser 1.8.0 (was 1.6.0)

Updates net.sourceforge.htmlunit:htmlunit-cssparser from 1.6.0 to 1.8.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "net.sourceforge.htmlunit", artifactId = "htmlunit-cssparser" } ]

labels: library-update, semver-minor

Superseded by #321.


Receive Max privacy. Comment "can take up to XX days"

Please add comment to Max privacy tx: "Max Privacy transaction can last at most 72 hours." - https://zpl.io/VOeRM31

Please, make the text exactly as in the design

Please, divide both texts for Max privacy transaction in two lines as in the design putting enter after words 'can last' and 'transaction fee'.

Change the letter P in the comment "Max privacy transaction can last at most 72 hours." to lower case.

Checked.


Release CDI-Unit 4.0.2

Hi @BrynCooke master branch has two new fixes, including basic Java 9 compatibility.
https://github.com/BrynCooke/cdi-unit/milestone/4
Could you make a new release, please?

Done. Sean, is there a way that I can give you access to create releases?

Thanks @BrynCooke.
The OSSRH team should be able to help with the release problem. I've just asked on https://issues.sonatype.org/browse/OSSRH-2949.

@BrynCooke I'm not sure if you saw https://issues.sonatype.org/browse/OSSRH-2949, but the OSSRH team has added me to the org.jglue.cdiunit group, so I should be able to cut the next release. Thanks!

That's great. Thanks Sean.


Update the metadata on an issue with GraphQL

### Tasks
- [ ] Given an issue number, fetch the issue
- [ ] Update the assignee
- [ ] Update the milestone
- [ ] Update labels
- [ ] Make a list of metadata that can be updated and add it on this card

Here is the code:
https://github.com/NASA-IMPACT/github-projects-automation/tree/11-add-metrics-from-the-sde-board-on-the-sample-project-board


Automatic F1 vote for marked players

It would be nice to have the option to automatically cast F1 votes for marked players in your list when a team mate calls the vote.
Because sometimes there is just too much going on in the game itself. So checking the player in the app may be too distracting.

related: #20 PazerOP/tf2_bot_detector#84 PazerOP/tf2_bot_detector#380
you have to hook DispatchUserMessage for this so uhh


Player Type and Auto Buffer (Dash turned off) not working

I have been using the YouTube Centers Developers Build userscript 548 for a long time both in Google Chrome and in Mozilla Firefox and it has always worked extremely well, but now I have noticed a few things, probably with Mozilla and Chrome updates. I have noticed that the YouTube Center 548 has not been updated for awhile.
I have dash playback disabled as always so the video would buffer really fast to the end, even while watching it and would even continue to buffer when paused, but now the YouTube videos are acting like dash playback is enabled. They are buffering slightly ahead of the video in dashes. Am I doing something wrong? I have also noticed that the video looks and acts the same whether Player Type: HTML5 or Flash is selected. Does this option not longer work?

same, i've also noticed this and its been acting strangely for some weeks now. funny how the developer picks the absolute worst time to abandon the software. not a peep from YePpHa

can only recommend to do what i did which is switch completely to youtube plus https://github.com/ParticleCore the only CON is there is no download feature so youll need a seperate script for that.
RIP youtubecenter and developer edition

I do have that YouTube Plus Add-On installed but currently disabled. The other thing is that I became so frustrated with the new Firefox because I lost alot of my add-ons that I have used for years, making Mozilla the same as other Isps so I took other peoples advice and installed the previous Firefox before the Quantum update, both on my computer and cellphone and turned off the automatic updates so it won't change.


ultracraft mod crash with zeta mod

Mod Name
zeta ultracraft
Mod Homepage
https://modrinth.com/mod/zeta/version/1.20.1-1.0-14 https://modrinth.com/mod/absolute-ultracraft/version/1.20.1-2.1.0
Minecraft version
1.20.1
Describe the bug
game crashed when loading world
Steps to reproduce
beta41 dev
Logs
https://pastebin.com/SJTVTQHF
Additional context
No response

Duplicate of #957

#957的重复

This is not the same crash issue, please bother to take a look.
https://pastebin.com/FPndCL9U

Yes I have, and I've tested ultracraft with zeta on the latest version of connector. It works fine. You log also mentions unrelated mods such as oculus, which might be the cause.

是的，我有，并且我已经在最新版本的连接器上使用 zeta 测试了 ultracraft。效果很好。您的日志还提到了不相关的模组，例如 oculus，这可能是原因。
thanks. i will test it.

Yes I have, and I've tested ultracraft with zeta on the latest version of connector. It works fine. You log also mentions unrelated mods such as oculus, which might be the cause.

it works. thanks 4 the help.


Resolve #74, related to #135 example files

Add user files for fitting and control
Fix bug in hanging power sweeps with JACOB
Add multiple resonator sweeps
Add broadband stitched plots
Add fridge status for T, P, flow rate
Add basic VNA communication template

This is a serious issue, the code will not fit data in #141


WinSCP.WinSCP version 5.19

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

6/17/2021 9:25:57 PM
Status: Completed
##[error] Installer https://sourceforge.net/projects/winscp/files/WinSCP/5.19/WinSCP-5.19-Setup.exe/download
##[error] Result: Failed
##vso[task.logissue type=error] Error occurred while downloading installer from https://sourceforge.net/projects/winscp/files/WinSCP/5.19/WinSCP-5.19-Setup.exe/download. 0x80072EFD.
Error information: Error occurred while querying URL for headers: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.

dumb bot 🤦♂

@karankad Try now since another PR got trough


[Bug] 2.1.1 Cant catch pokemon, biome restriction

Hello, I am currently running on the latest mod version and having the config set to first-encounter-restriction to true, I have noticed in my new SP world, I caught 1 pixelmon but up to 3 biomes were marked as being unable to catch a pixelmon. I caught it in taiga, but forest and plains were marked as well for some reason.
As a temporary workaround, I have to disable first-encounter-restriction and write down what each caught pixelmon biome was, and if I enter it again to make a mental note to not catch anything.

No. I was trying to catch them, that's why I was confused.


Add Dark Theme toggle

I want to add a toggle to swap the website to a dark theme. There could also be an option to match the system theme too

I and @n00bmaster2810 would like to work on this issue


Documentation updates from Docbook to CommonMark

Motivation for this change
OCaml
https://github.com/NixOS/nixpkgs/issues/104968
Shell-helpers
https://github.com/NixOS/nixpkgs/issues/105261
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Yes, I am pulling various at once.

Thanks!


Package and image bumps

Ticket Number
DL-5668
Description
In order to allow access to the vendor API, the vendor-login-service must be integrated in the relevant application. However, the service was relying on old mu-javascript-template that was based off an unsupported node version (v14.x.x); both feature and security updates are no longer provided.
This PR bumps the template version in addition to a couple of packages inside package.json; it also updates the lock file.
How to Test

Insert a mock vendor in the database:

PREFIX muAccount: <http://mu.semte.ch/vocabularies/account/>
PREFIX mu:        <http://mu.semte.ch/vocabularies/core/>
PREFIX foaf:      <http://xmlns.com/foaf/0.1/>
PREFIX ext:       <http://mu.semte.ch/vocabularies/ext/>

INSERT DATA {
  GRAPH <http://mu.semte.ch/graphs/automatic-submission> {
    <http://example.com/vendor/mechelen> a foaf:Agent, ext:Vendor ;
      muAccount:key "mechelen-secret-key" ;
      muAccount:canActOnBehalfOf <http://data.lblod.info/id/bestuurseenheden/be278471a2a318edba32e7ac4294c0eafbe4c8077a34dcbb9c2e43211d4a78a6> ;
      foaf:name "Mechelen vendor" ;
      mu:uuid "aaaa-bbbb-cccc-1111-2222-3333" .
  }
}

Send the following cURL request:

curl -v -X POST \
      -H "Content-Type: application/json" \
      -b CookieJar.tsv -c CookieJar.tsv \
      -d '{
  "organization": "http://data.lblod.info/id/bestuurseenheden/be278471a2a318edba32e7ac4294c0eafbe4c8077a34dcbb9c2e43211d4a78a6",
  "publisher": {
    "uri": "http://example.com/vendor/mechelen",
    "key": "mechelen-secret-key"
  }
}' http://localhost:90/vendor/login

The request will save the cookie in a cookie jar (CookieJar.tsv), which will be used down the line to send SPARQL queries from a vendor. Check the Response subsection under this section to compare your output and understand what it means; the expected code in this case should be 201 Created.

This commit that fixes the prettier complaints on woodpecker can be confusing as it now causes my editor to complain about them (VS Code with node v21.x.x installed); I tried to disable trailing comma errors by editing the configs, but that then complained about other code snippets that already have trailing commas.
As such, I opted to add trailing commas where applicable. If there are any suggestions or preferences for this, let me know.

Same comments as on https://github.com/lblod/sparql-authorization-wrapper-service/pull/5 about package updates.


Record logged network requests as completed spans

Record network requests as spans.

Consolidate logNetworkCall and logNetworkError methods

EmbraceNetworkLoggingService had two methods: logNetworkCall and logNetworkError. They were both doing essentially the same, but with a different NetworkCallV2 object: strip the URL, sanitize the TraceId, logging the networkCapturedData, and processing the network call.
We don't need a NetworkCallV2 object anymore, and the EmbraceNetworkRequest is the same for both network call and network error, so we can just use one method.

Remove unused methods.

Questions:

Why do we need to strip the URL?
Why are we stripping the URL for the span, but not for the networkCapturedData?

Merge activity

Apr 18, 7:50 PM EDT: @bidetofevil started a stack merge that includes this pull request via Graphite.


chore(package): update tap to version 10.2.0

Closes #46
https://greenkeeper.io/

Coverage remained the same at 28.571% when pulling 99e29e8c0e310f5580eb1bafa633c5e114ec039f on greenkeeper/tap-10.2.0 into 93c323a1a6ab38c686686af4b6e316cb26b71c39 on master.


Remove the uuid dependency

It should be possible to remove the UUID package and parse the uuid string into a buffer with native functions

Decided to stay with the UUID lib as the required functions, parse and stringify, are well optimized for performance.
A good custom implementation would be more or less copy & paste.


Add University of Colorado Denver

@PooyaRah Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Update docs.microsoft.com references in READMEs

Update README links to use learn.microsoft.com instead

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

/check-enforcer override


Idea - Optimal skill tree?

Wouldn't it be possible to do a DFS throught the skill tree looking for the maximum damage for a given skill and equips? ( just disconsider jewels, this part could be adjusted by hand). You could also set another parameters like minimum % multiplyer etc. This seems fair reasonable given the small tree (for computation size)

I thought about this not but a couple of minutes ago too, as I was looking at a tree and attempting to prune it to get to better nodes. FOR THAT MATTER, given that it can look into your inventory, why not have something that could determine what the ideal setup given your gear in the stash would work?
There ARE reasons why sheer DPS doesn't work out as something to optimize for. Survival is important too. And it doesn't account for complex situations like... storm brand not particularly benefiting from cast speed.


chore: Add @oleg-nenashev to maintainers

Closes https://github.com/keptn/community/issues/61

@oleg-nenashev please sign off your commits. Thx.

Wow. Something went wrong because I make the change through the GitHub web UI, it should have been signed off by default. Will get it fixed

Closed in favor of: #6256


Postprocessing / denormalization

Hi all,
maybe I'm missing something obvious, but where can I specify how the result is denormalized? Since one can now add preprocessing to any input node, should each output node not also have an optional postprocessing parameter? Or is this also called preprocessing for output nodes?
Best, Debo

We were discussing in all brevity about such options but then concluded its not needed. (It was also after quite some hours of intense discussion...)
Now that you point, you actually have a point. For virtually all image restoration applications we indeed want to bring the outputs back in the original image range...
I think we should discuss this in tomorrows meeting when we are all present.

I think it would be great to have an analog postprocessing for the outputs. 👍

Hi, can't you just add the denormalization op into the network itself when you export it?

I'm just wrapping existing networks, I'm not designing them. You could also add normalization to the network, no? If there's a way to specify normalization here, it just seems weird that there is no way to specify denormalization.

Decided in today's bioimage.io meeting:

design postprocessing analog to preproessing

for valid postprocessings we start with:

sigmoid, no kwargs

scale_mean_variance with kwargs:

mode: 'per_dataset' / 'per_sample'
reference_input: name of input tensor to derive mean and variance from
(note: if you want to use fixed mean, std, use scale_linear)

scale_min_max with kwargs:

reference_input: name of input tensor to use for statistics
mode: (fixed/per_dataset/per_sample)
min_percentile: (if mode is not 'fixed') default 0
max_percentile: (if mode is not 'fixed') default 100 (or 1.0?)
min: (if mode is 'fixed')
max: (if mode is 'fixed')

scale_linear with kwargs:

'gain': default 1.0
'offset': default 0.0

outputs
 - name: output_logits
   drange: ...
   ...
   postprocessing: {name: sigmoid, kwargs: {}}

todos:

[ ] PR

We have added the postprocessing description in #43 now.


Fix all issues in this library

There are a lot of issues with this library. This pull request should fix most if not all.


feat(0.24.1) added generator and bumped 0.24.0 to 0.24.1

Signed-off-by: haarchri<EMAIL_ADDRESS>
What this PR does / why we need it:

bump upstream flux 0.24.0 to 0.24.1
added generator to get all crds, add our lables, add if block
added make generate
added make helmdocs
added make check-diff

Special notes for your reviewer:

as followup we should add in ci generate,helmdocs,check-diff then we can check that all needed steps are executed in the commit
as followup we should generate version in Chart.yaml
as followup we should generate imagetags from upstream
as followup we should generate version, chartVersion, imagetag in all testfiles
generator.sh uses FLUX2_VERSION out of Makefile

make generate
09:06:34 [ .. ] Fetch Flux2 GitRepo
HEAD ist jetzt bei ff7df54 Merge pull request #2178 from fluxcd/update-components
09:06:35 [ OK ] Fetch Flux2 GitRepo
09:06:35 [ .. ] Generating CRDs
09:06:45 [ OK ] Generating CRDs

make helmdocs
helm-docs
INFO[2021-12-12T09:10:14+01:00] Found Chart directories [charts/flux2, charts/flux2-sync]
INFO[2021-12-12T09:10:14+01:00] Generating README Documentation for chart /Users/haarchri/Documents/fluxcd-community/helm-charts/charts/flux2-sync
INFO[2021-12-12T09:10:14+01:00] Generating README Documentation for chart /Users/haarchri/Documents/fluxcd-community/helm-charts/charts/flux2

make check-diff
09:06:50 [ .. ] checking that branch is clean
09:06:50 [ OK ] branch is clean

Checklist

[x] DCO signed
[x] Chart Version bumped
[x] helm-docs are updated
[x] Helm chart is tested

LGTM


queue navigation calls on ios

if we chain push and pop calls too fast on ios and if the page transitions animations are not finished, the navigation controller can fall into an unstable state complaining like this
nested push animation can result in corrupted navigation bar
Finishing up a navigation transition in an unexpected state. Navigation Bar subview tree might get corrupted.
Unbalanced calls to begin/end appearance transitions for <FCMyViewController: 0x17dac270>.

planned for 0.5.1


repeat prefix and register prefix don't play nicely

I want to copy 5 lines into the a register.
"a5yy doesn't work - it saves it into the default register and leaves a untouched
5"ayy works fine
I believe that because the function repeatPrefix (in vim-state.coffee) saves the prefix into the operations stack and registerPrefix (in vim-state.coffee) does not that makes the repeat something that is remembered while the register gets forgotten if anything else (like a number) happens between setting the register and executing the command.
However, I will confess that this whole business of prefixes and opStack and etc (especially registerPrefix!) is a bit like "magic" to me right now. Is there any documentation of how this works anywhere?
I tried simply changing the definition of registerPrefix from this:
registerPrefix: (e) ->     new Prefixes.Register(@registerName(e))
to this:
registerPrefix: (e) ->     @pushOperations(new Prefixes.Register(@registerName(e)))
but that caused other problems immediately and I don't have even close to a deep enough understanding to be able to debug it from there.
Sorry - I tried.

As stated in the README, this package is no longer maintained and is deprecated. We recommend that people use the vim-mode-plus package instead. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!