Encoder admits invalid color configuration

For example, at src/header.rs line 418. If seq.color_description is: ColorPrimaries::BT709, TransferCharacteristics::SRGB, MatrixCoefficients::Identity
assert!(seq.chroma_sampling == ChromaSampling::Cs444);

This assertion can fail with an encoder configuration that currently passes validation.

Test unit written to fuzz/artifacts/encode/crash-8bc4c7379c400a52c93043fb5617767cfba41228
Base64: Dv///xAKAAAAbJOTbAoKCpCQkJCQkMMJAAAAAAAAAAkAAAAAAAAA/////////wD/////////////AAqTbF3hZgoKk8/Pz8/Pz8//AGxdcgAKCpCUkJCQkMMJAJCQwwkAAAAAAAAACsMKw89sHgq6CpNsXeFsCgqTfwAAAJSQkJCQwwkAAAABAAAAAAAAAAAAAM8ACsMKw2weCroKk2xd4WwKCpN/AAAAAAA=

[...]

Failing input:

        fuzz/artifacts/encode/crash-8bc4c7379c400a52c93043fb5617767cfba41228

Output of `std::fmt::Debug`:

        ArbitraryEncoder {
            config: Config {
                enc: EncoderConfig {
                    width: 148,
                    height: 145,
                    sample_aspect_ratio: Rational {
                        num:<PHONE_NUMBER>5854093824,
                        den:<PHONE_NUMBER>50259600,
                    },
                    time_base: Rational {
                        num: 639888,
                        den: 589824,
                    },
                    bit_depth: 8,
                    chroma_sampling: Cs400,
                    chroma_sample_position: Unknown,
                    pixel_range: Limited,
                    color_description: Some(
                        ColorDescription {
                            color_primaries: BT709,
                            transfer_characteristics: SRGB,
                            matrix_coefficients: Identity,
                        },
                    ),
                    mastering_display: Some(
                        MasteringDisplay {
                            primaries: [
                                ChromaticityPoint {
                                    x: 2746,
                                    y: 27795,
                                },
                                ChromaticityPoint {
                                    x: 57693,
                                    y: 2668,
                                },
                                ChromaticityPoint {
                                    x: 37642,
                                    y: 127,
                                },
                            ],
                            white_point: ChromaticityPoint {
                                x: 0,
                                y: 37012,
                            },
                            max_luminance:<PHONE_NUMBER>,
                            min_luminance: 9,
                        },
                    ),
                    content_light: None,
                    enable_timing_info: false,
                    still_picture: false,
                    error_resilient: true,
                    switch_frame_interval: 0,
                    min_key_frame_interval: 0,
                    max_key_frame_interval: 4,
                    reservoir_frame_delay: None,
                    low_latency: true,
                    quantizer:<PHONE_NUMBER>8492891135,
                    min_quantizer: 255,
                    bitrate: -1,
                    tune: Psnr,
                    tile_cols: 0,
                    tile_rows: 1,
                    tiles: 12,
                    rdo_lookahead_frames:<PHONE_NUMBER>7477888522,
                    speed_settings: SpeedSettings {
                        multiref: false,
                        fast_deblock: true,
                        reduced_tx_set: true,
                        tx_domain_distortion: true,
                        tx_domain_rate: false,
                        encode_bottomup: false,
                        rdo_tx_decision: false,
                        prediction_modes: Simple,
                        include_near_mvs: false,
                        no_scene_detection: false,
                        fast_scene_detection: false,
                        cdef: true,
                        lrf: true,
                        sgr_complexity: Reduced,
                        use_satd_subpel: true,
                        non_square_partition: false,
                        enable_segmentation: false,
                        enable_inter_tx_split: false,
                        fine_directional_intra: true,
                        partition_range: PartitionRange {
                            min: BLOCK_8X8,
                            max: BLOCK_64X64,
                        },
                    },
                },
                rate_control: RateControlConfig {
                    emit_pass_data: false,
                    summary: None,
                },
                threads: 1,
                pool: None,
            },
            frame_count: 1,
            pixels: [
                0,
                0,
                0,
                0,
                0,
                207,
                0,
                10,
                195,
                10,
                195,
                108,
                30,
                10,
                186,
                10,
                147,
                108,
                93,
                225,
                108,
                10,
                10,
                147,
                127,
                0,
                0,
                0,
                0,
                0,
            ],
        }

Reproduce with:

        cargo fuzz run encode fuzz/artifacts/encode/crash-8bc4c7379c400a52c93043fb5617767cfba41228

Minimize test case with:

        cargo fuzz tmin encode fuzz/artifacts/encode/crash-8bc4c7379c400a52c93043fb5617767cfba41228

I'm encountering this with the following image:

avifenc --lossless --tilecolslog2 1 -c rav1e -s d -o RAV1E_out/pixelart/0ljbcwr1yeh41/0ljbcwr1yeh41-lossless.avif pixelart/0ljbcwr1yeh41.png
Successfully loaded: pixelart/0ljbcwr1yeh41.png
WARNING: [--lossless] Only aom (-c) supports lossless transforms. Output might not be lossless.
AVIF to be written: (Lossy)
 * Resolution     : 256x256
 * Bit Depth      : 8
 * Format         : YUV444
 * Alpha          : Present
 * Range          : Full
 * Color Primaries: 1
 * Transfer Char. : 13
 * Matrix Coeffs. : 0
 * ICC Profile    : Absent (0 bytes)
 * XMP Metadata   : Absent (0 bytes)
 * EXIF Metadata  : Absent (0 bytes)
 * Transformations: None
Encoding with AV1 codec 'rav1e' speed [-1], color QP [0 (Lossless) <-> 0 (Lossless)], alpha QP [0 (Lossless) <-> 0 (Lossless)], tileRowsLog2 [0], tileColsLog2 [1], 1 worker thread(s), please wait...
thread '<unnamed>' panicked at 'assertion failed: seq.chroma_sampling == ChromaSampling::Cs444', src/header.rs:418:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
fatal runtime error: failed to initiate panic, error 5
fish: “avifenc --lossless --tilecolslo…” terminated by signal SIGABRT (Abort)

I'm encountering this with the following image:

avifenc --lossless --tilecolslog2 1 -c rav1e -s d -o RAV1E_out/pixelart/0ljbcwr1yeh41/0ljbcwr1yeh41-lossless.avif pixelart/0ljbcwr1yeh41.png
Successfully loaded: pixelart/0ljbcwr1yeh41.png
WARNING: [--lossless] Only aom (-c) supports lossless transforms. Output might not be lossless.
AVIF to be written: (Lossy)
 * Resolution     : 256x256
 * Bit Depth      : 8
 * Format         : YUV444
 * Alpha          : Present
 * Range          : Full
 * Color Primaries: 1
 * Transfer Char. : 13
 * Matrix Coeffs. : 0
 * ICC Profile    : Absent (0 bytes)
 * XMP Metadata   : Absent (0 bytes)
 * EXIF Metadata  : Absent (0 bytes)
 * Transformations: None
Encoding with AV1 codec 'rav1e' speed [-1], color QP [0 (Lossless) <-> 0 (Lossless)], alpha QP [0 (Lossless) <-> 0 (Lossless)], tileRowsLog2 [0], tileColsLog2 [1], 1 worker thread(s), please wait...
thread '<unnamed>' panicked at 'assertion failed: seq.chroma_sampling == ChromaSampling::Cs444', src/header.rs:418:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
fatal runtime error: failed to initiate panic, error 5
fish: “avifenc --lossless --tilecolslo…” terminated by signal SIGABRT (Abort)

Format         : YUV444
Alpha          : Present
Range          : Full
Color Primaries: 1
Transfer Char. : 13
Matrix Coeffs. : 0

I wonder if this error occurs while encoding the alpha image. The RGB image shouldn't trigger this assertion since it is YUV444.

Format         : YUV444
Alpha          : Present
Range          : Full
Color Primaries: 1
Transfer Char. : 13
Matrix Coeffs. : 0

I wonder if this error occurs while encoding the alpha image. The RGB image shouldn't trigger this assertion since it is YUV444.

I'm encountering this with the following image:

I've confirmed with a local build of #2645 that this example is resolved.
RUST_BACKTRACE=1 avifenc --lossless --tilecolslog2 1 -c rav1e -s d -o 0ljbcwr1yeh41-lossless.avif ~/Downloads/0ljbcwr1yeh41.png
Successfully loaded: ~/Downloads/0ljbcwr1yeh41.png
WARNING: [--lossless] Only aom (-c) supports lossless transforms. Output might not be lossless.
AVIF to be written: (Lossy)
 * Resolution     : 256x256
 * Bit Depth      : 8
 * Format         : YUV444
 * Alpha          : Present
 * Range          : Full
 * Color Primaries: 1
 * Transfer Char. : 13
 * Matrix Coeffs. : 0
 * ICC Profile    : Absent (0 bytes)
 * XMP Metadata   : Absent (0 bytes)
 * EXIF Metadata  : Absent (0 bytes)
 * Transformations: None
Encoding with AV1 codec 'rav1e' speed [-1], color QP [0 (Lossless) <-> 0 (Lossless)], alpha QP [0 (Lossless) <-> 0 (Lossless)], tileRowsLog2 [0], tileColsLog2 [1], 1 worker thread(s), please wait...
Encoded successfully.
 * Color AV1 total size: 1128 bytes
 * Alpha AV1 total size: 210 bytes
Wrote AVIF: 0ljbcwr1yeh41-lossless.avif

I'm encountering this with the following image:

I've confirmed with a local build of #2645 that this example is resolved.
RUST_BACKTRACE=1 avifenc --lossless --tilecolslog2 1 -c rav1e -s d -o 0ljbcwr1yeh41-lossless.avif ~/Downloads/0ljbcwr1yeh41.png
Successfully loaded: ~/Downloads/0ljbcwr1yeh41.png
WARNING: [--lossless] Only aom (-c) supports lossless transforms. Output might not be lossless.
AVIF to be written: (Lossy)
 * Resolution     : 256x256
 * Bit Depth      : 8
 * Format         : YUV444
 * Alpha          : Present
 * Range          : Full
 * Color Primaries: 1
 * Transfer Char. : 13
 * Matrix Coeffs. : 0
 * ICC Profile    : Absent (0 bytes)
 * XMP Metadata   : Absent (0 bytes)
 * EXIF Metadata  : Absent (0 bytes)
 * Transformations: None
Encoding with AV1 codec 'rav1e' speed [-1], color QP [0 (Lossless) <-> 0 (Lossless)], alpha QP [0 (Lossless) <-> 0 (Lossless)], tileRowsLog2 [0], tileColsLog2 [1], 1 worker thread(s), please wait...
Encoded successfully.
 * Color AV1 total size: 1128 bytes
 * Alpha AV1 total size: 210 bytes
Wrote AVIF: 0ljbcwr1yeh41-lossless.avif


Add support for OpenAPI schema generation for POCOs

This is related to https://github.com/microsoft/OpenAPI.NET/issues/836.
For certain native .NET types, it would be great if OpenAPI.NET could provide the authorative OpenApiSchema definitions for these types. Current prototypes maintain a dictionary like this:
private static readonly Dictionary<Type, Func<OpenApiSchema>> _primitiveTypeToOpenApiSchema =
        new()
        {
            [typeof(bool)] = () => new OpenApiSchema { Type = "boolean" },
            [typeof(byte)] = () => new OpenApiSchema { Type = "string", Format = "byte" },
            [typeof(int)] = () => new OpenApiSchema { Type = "integer", Format = "int32" },
            ...
            [typeof(DateTime)] = () => new OpenApiSchema { Type = "string", Format = "date-time" },
            [typeof(DateTimeOffset)] = () => new OpenApiSchema { Type = "string", Format = "date-time" },
            [typeof(Guid)] = () => new OpenApiSchema { Type = "string", Format = "uuid" },
            [typeof(char)] = () => new OpenApiSchema { Type = "string" }
        };

And there a variety of open source packages/projects that maintain their own associations (see this, this, and this).
Since this is a finite set of mappings, it would be great if they would be centralized in the OpenAPI.NET package to avoid duplication and ambiguity about the right schemas to generate for different native types in .NET.
cc: @darrelmiller  @baywet

Additionally, Kiota is doing something similar to get from OpenAPI type/format to dotnet (or other languages) types. Having the opposite dictionary would be helpful as well.
CC @CarolKigoonya to queue this in the pipeline


disable swipe on youtube chanel page and disable PIP mode with swipe down

What website(s) would this filter apply to?
youtube
What content should be blocked?
it is anoying if you want to start video on channel, but because of light swipe you change the tab to shorts. The same iif you want to scroll down with swipe and youtube switch to picture in picture mode.
I got an answer with solution on ublock reddit so you can add it if you want.
https://www.reddit.com/r/uBlockOrigin/comments/115f6rz/disable_some_touch_functionality_on_a_full/
What options should be offered?
No response
Did you write a custom uBlock filter already?
www.youtube.com##+js(aeld, /^pan(start|move|end)$/)

Additional notes
No response

Thanks for the idea.  @xvello has to decide if we could implement this rule in one of the existing templates (https://letsblock.it/filters/tag/youtube).

Thanks @mailyspamy!
I feel like it fits within the scope of youtube-cleanup, alongside the no-fullscreen-on-dblclick option. It is now available at https://letsblock.it/filters/youtube-cleanup

Thanks xvello. Hope it will be usable also for another people.


[FEATURE] Changing placeholder attribute

Description:
Basically, "Share  your thoughts" may be not a good placeholder for input the daily important tasks specially in To-Do-List website.
So, Changing it to something better will  help users.
If you find it right, then pls assign this issue to me.....

Hello! I'd like to work on this issue. Could you please assign it to me?


Run pre-commit updates

... the docker build has started failing in Github actions (but not locally): https://github.com/opendatacube/eo-datasets/runs/6124761702?check_suite_focus=true
Not yet sure why, but I've manually run the formatting here.

... the docker build has started failing in Github actions (but not locally): https://github.com/opendatacube/eo-datasets/runs/6124761702?check_suite_focus=true
Not yet sure why, but I've manually run the formatting here.

The pre-commit has been failing for a while.  The failure is caused by this line:
lint-root:
	docker run --volume "${PWD}":/tests --user root -w /tests eodatasets:test pre-commit run -a

The error:
Run make lint-root
[4](https://github.com/opendatacube/eo-datasets/runs/6167601805?check_suite_focus=true#step:4:4)
docker run --volume "/home/runner/work/eo-datasets/eo-datasets":/tests --user root -w /tests eodatasets:test pre-commit run -a
[5](https://github.com/opendatacube/eo-datasets/runs/6167601805?check_suite_focus=true#step:4:5)
An error has occurred: FatalError: git failed. Is it installed, and are you in a Git repository directory?
[6](https://github.com/opendatacube/eo-datasets/runs/6167601805?check_suite_focus=true#step:4:6)
Check the log at /root/.cache/pre-commit/pre-commit.log
[7](https://github.com/opendatacube/eo-datasets/runs/6167601805?check_suite_focus=true#step:4:7)
make: *** [Makefile:37: lint-root] Error 1
[8](https://github.com/opendatacube/eo-datasets/runs/6167601805?check_suite_focus=true#step:4:8)
Error: Process completed with exit code 2.


Alias for Mastela Fruit

Could we have an alias for Mastela Fruit and Seed of Mastela?

it probably works. idk.
"Mastela" = "Mastela Fruit" = "Seed of Mastela"


Establish CI/CD pipeline with GitHub Actions

The project should setup a CI/CD pipeline based on GitHub Actions.

There's this one already: https://github.com/kroxylicious/kproxy/blob/main/.github/workflows/maven.yml. Is there any specific flows you'd like to add?

somehow I failed to see it, thank you.


farmware button actions

Coverage decreased (-0.002%) to 20.944% when pulling a439ae3b8ecd88148b5cbfc5c8aed1e0fc6d5b34 on gabrielburnworth:master into 29c494489d174314ab4efd2df212a8f7eabc8d6c on FarmBot:master.

Coverage increased (+0.007%) to 20.953% when pulling b409cac8f668114e78a8938c49a14d8de582d931 on gabrielburnworth:master into 29c494489d174314ab4efd2df212a8f7eabc8d6c on FarmBot:master.


Gets stuck on UWPHook loading screen

Hello, I ran UWPHook on Minecraft for WIndows to get it into Steam so I can remote play it to my Linux machine.
I enabled the Streaming toggle in UWPHook's settings as it states it's required for in-home Steam streaming to function.
When I go to launch the game, the screen never advances past the "Welcome to UWPHook, the easier way to add UWP games and apps to Steam!" screen. Audio plays just fine, and the game can even be controlled (albeit completely blindly), but the display doesn't update.
What's the problem here?

I am also having a similar issue with Minecraft BE, although a different one. (#150) I believe this might be a problem with the fact that UWPHook doesn't detect when the Minecraft window is open, or where it is. I don't know how to fix this myself, and am looking for a solution.


[SPARK-21779][PYTHON] Simpler DataFrame.sample API in Python

What changes were proposed in this pull request?
This PR make DataFrame.sample(...) can omit withReplacement defaulting False, consistently with equivalent Scala / Java API.
In short, the following examples are allowed:
>>> df = spark.range(10)
>>> df.sample(0.5).count()
7
>>> df.sample(fraction=0.5).count()
3
>>> df.sample(0.5, seed=42).count()
5
>>> df.sample(fraction=0.5, seed=42).count()
5

In addition, this PR also adds some type checking logics as below:
>>> df = spark.range(10)
>>> df.sample().count()
...
TypeError: withReplacement (optional), fraction (required) and seed (optional) should be a bool, float and number; however, got [].
>>> df.sample(True).count()
...
TypeError: withReplacement (optional), fraction (required) and seed (optional) should be a bool, float and number; however, got [<type 'bool'>].
>>> df.sample(42).count()
...
TypeError: withReplacement (optional), fraction (required) and seed (optional) should be a bool, float and number; however, got [<type 'int'>].
>>> df.sample(fraction=False, seed="a").count()
...
TypeError: withReplacement (optional), fraction (required) and seed (optional) should be a bool, float and number; however, got [<type 'bool'>, <type 'str'>].
>>> df.sample(seed=[1]).count()
...
TypeError: withReplacement (optional), fraction (required) and seed (optional) should be a bool, float and number; however, got [<type 'list'>].
>>> df.sample(withReplacement="a", fraction=0.5, seed=1)
...
TypeError: withReplacement (optional), fraction (required) and seed (optional) should be a bool, float and number; however, got [<type 'str'>, <type 'float'>, <type 'int'>].

How was this patch tested?
Manually tested, unit tests added in doc tests and manually checked the built documentation for Python.

Test build #80870 has started for PR 18999 at commit 5de97d1.

Test build #80870 has finished for PR 18999 at commit 5de97d1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80870/
Test PASSed.

cc @rxin. Dose this make sense to you?

Test build #80898 has started for PR 18999 at commit 0328446.

Test build #80898 has finished for PR 18999 at commit 0328446.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80898/
Test PASSed.

https://github.com/apache/spark/pull/18999#discussion_r134131441 looks hidden. I addressed the other comment.

Test build #80911 has started for PR 18999 at commit 24525bc.

Test build #80912 has started for PR 18999 at commit f2608ab.

Test build #80912 has finished for PR 18999 at commit f2608ab.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #80911 has finished for PR 18999 at commit 24525bc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80912/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80911/
Test PASSed.

@rxin, would you maybe have some opinion on https://github.com/apache/spark/pull/18999#discussion_r134131441 (avoiding fraction checking within Python side)?

cc @holdenk and @ueshin, could you maybe take a look when you have some time?

LGTM.

retest this please

Test build #81303 has started for PR 18999 at commit f2608ab.

Test build #81303 has finished for PR 18999 at commit f2608ab.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81303/
Test PASSed.

Merged to master.

Thank you @viirya, @felixcheung, @rxin and @ueshin.


builder: don't drop async sqlx connection in sync context

We were seeing a new error in sentry, starting yesterday evening:

panic
this functionality requires a Tokio context

interesting parts of the stack trace:

I imagine every place in the build_package method where we use ? to return early could lead to this error. But strange is:

it didn't happen until now
every early return should actually lead to another sentry error, which we don't see.

So for now this PR is a guess at what the temporary solution could be.
Long-term, when we have separate runtimes ( aka: separate webserver & builder processes) I imagine that the builder also would have a global runtime through tokio::main.

One alternative would be to let guard = context.runtime().enter(); early in the thread so that the runtime is accessible when the connection is dropped in sync context. It only wants to spawn a task, and should never be doing anything other than that from a Drop, so that should work.

Inspecting the server logs it seems to be https://crates.io/crates/aymr somehow causing the issue when uploading its source, we don't increment the attempt counter or last_attempt when a panic occurs so I have manually set it to 5 for this crate to stop it from rebuilding continuously.

At a guess, there's a 0 byte file in the crate which probably gets stored without compression and hits https://github.com/rust-lang/docs.rs/blob/fe3197616f3fa0497520a2d5eb89d09937a510c4/src/storage/archive_index.rs#L66

One alternative would be to let guard = context.runtime().enter(); early in the thread so that the runtime is accessible when the connection is dropped in sync context. It only wants to spawn a task, and should never be doing anything other than that from a Drop, so that should work.

You're right, this is the better solution.

Inspecting the server logs it seems to be https://crates.io/crates/aymr somehow causing the issue when uploading its source, we don't increment the attempt counter or last_attempt when a panic occurs so I have manually set it to 5 for this crate to stop it from rebuilding continuously.

~At a guess, looking in the crate there's a 0 byte file, which probably gets stored without compression and hits ~
https://github.com/rust-lang/docs.rs/blob/fe3197616f3fa0497520a2d5eb89d09937a510c4/src/storage/archive_index.rs#L66
EDIT: Wrong deduction, creating the source archive works, the last log line before the panic is docs_rs::db::add_package: Adding package into database

Ah, so there is some error happening in add_package_into_database, and we don't see it because the panic happens before we return or report the error.
Also we could ask ourselves if we should increase attempts on panics, or stop building?

I just had another idea for an generic solution for handling entering the runtime.
#2476

closing in favor of #2479


Started Firefox Themes (help would be appreciated)

I made a light, a dark and a dynamic theme (or addon) for the Firefox browser.
Links: Light, Dark, Dynamic
The source code for the dynamic theme is public.
The light and dark theme use the colors provided there.
However, this is my first try at firefox addons and if anyone knows how to improve things, just send me some hints or pull requests. For example, currently the dynamic version is hard coded to use the light theme between 7am and 10pm, I would rather like to use the preferred appearance provided by the user's OS if possible.
Also note that I tried to stick to the Flexoki Description as strictly as possible, but I had to shift the text colors in the dark theme 2 tones lighter for better contrast and readability. Also there might be alternative ways to use the colors that might look even better. Feel free to discuss changes here or on the repo linked above.

I am already using it since some days ago I am really happy with it, thank you for your work! If I find something strange or that could be a bug, I will report you in your repository ❤

Next time I find some spare time, I will probably look more into making the theme use the preferred appearance given by the OS. Also I would like to make Thunderbird themes as well, probably not too different to firefox themes.
@kepano Should I keep my repo for now or do you want me to make a pull request and add it to this flexoki repo here? Personally I feel like it would be cleaner to collect everything in the flexoki repo.

I have linked to your repo from the readme. If you prefer adding it here, that's okay too. Thanks!


Kargo should re-apply "promotions" if deviations detected

Once state is "promoted" the Kargo should re-apply promotions change then the deviation is detected:
Lets use kargo-demo-test from quick start:

Create stage

apiVersion: kargo.akuity.io/v1alpha1
kind: Stage
metadata:
  name: test
spec:
  promotionMechanisms:
    argoCDAppUpdates:
    - appName: kargo-demo-test
      appNamespace: argocd
    gitRepoUpdates:
    - kustomize:
        images:
        - image: nginx
          path: stages/test
      repoURL: https://github.com/alexmt/kargo-demo
      writeBranch: main
  subscriptions:
    repos:
      images:
      - repoURL: nginx
        semverConstraint: ^1.24.0
        updateStrategy: SemVer

Make sure the is a change and promote it
Wait for kargo to push git commit and sync kargo-demo-test application
Manually sync kargo-demo-test application to previous version

Expected functionality:
User should be able to "apply" promotion one more time to fix deviations. Bonus feature: addition setting in Auto-sync policy to auto-reapply  promotions ( aka self heal  )
Current behavior:
Promotion is atomic and Kargo does not do anything to get test to desired state.

I spent a lot of time thinking about this and I am not convinced yet that we should do this...
If someone cares about their Apps remaining synced to the correct state, Argo CD already implements that well. I'm not sure what the benefit would be of making Kargo do part of Argo CD's job.

For me it makes more sense if you assume Kargo is responsible for promoting changes to git. If Kargo finds a diff between its current state and git, Kargo should handle the drift. Without this, Kargo's state of the world is incorrect.

This hasn't been ruled out, but (in case it makes you feel any better) for the common case Kargo's "state of the world" won't be incorrect. Kargo knows what the Apps should be synced to and if they're synced to something else, that's a health problem for the Stage.
The less common cases, where "state of the world" could be incorrect would be cases that either don't integrate with Argo CD or cases that aren't real GitOps -- e.g. App(s) get patched instead of commits being pushed to a repo. We're good about checking that Apps are synced to what they're supposed to be synced to, but don't really check (yet) that app.spec.kustomize.images or app.spec.helm.parameters, for instance, match what Kargo thinks they should be. This has been on my radar for a while (but there isn't an issue for it yet) and needs to get done regardless, but it would also be a pre-req for any possible movement on this issue.
I will write up a separate issue later today to address the need for checking additional dimensions of an App's status, beyond just what it is synced to.

Opened #2340 to track some known pre-reqs for this.


How to represent a parallel state with indipendent state for each contained branch?

Like in figure

I would like to start F1 and F2 in parallel
then:

start F3 when F1 and F2 have both terminated
start F4 when F2 terminated
Start F5 when F1 terminated

If I collect F1 and F2 as branches of a single parallel state I am not to react when only of the function terminates. Is there a way?

@christiansicari, this is just a rough idea from the top of my head. Please take it with a grain of salt.
To make it simple, I'd go with subflows and reuse the output of the parallel processing later in F3.
So F1->F5, subflow 1 (SB1); and F2->F4, subflow 2 (SB2).
The trick here, is to preserve the data from F1 and F2 output in the end, so you could use it in F3. The main flow might be something like this:
{
  "id": "parallelsubflows",
  "version": "1.0",
  "specVersion": "0.8",
  "name": "Parallel Subflow",
  "start": "StartState",
  "states": [
    {
      "name": "StartState",
      "type": "parallel",
      "completionType": "allOf",
      "transition": "StateF3",
      "branches": [
        {
           "name": "CallSubFlow1",
            "actions": [{
               "subFlowRef": {
                  "workflowId": "subflow1"
               }
            }]
        },{
          "name": "CallSubFlow2",
          "actions": [{
             "subFlowRef": {
               "workflowId": "subflow2"
             }
          }]
        }
      ]
    },{
      "name": "StateF3",
      "type": "inject",
       "data": {
         "foo": "bar"
       },
        "end": true
    }
  ]
}

Then you write the subflows the way you need:
{
  "id": "subflow1",
  "version": "1.0",
  "specVersion": "0.8",
  "name": "SubFlow1",
  "start": "StateF1",
  "states": [
    {
      "name": "StateF1",
      "type": "inject",
      "transition": "StateF5",
      "data": {
        "stateF1": "called"
      }
    },{
      "name": "StateF5",
      "type": "inject",
       "data": {
         "stateF5": "called"
       },
        "end": true
    }
  ]
}

{
  "id": "subflow2",
  "version": "1.0",
  "specVersion": "0.8",
  "name": "SubFlow2",
  "start": "StateF2",
  "states": [
    {
      "name": "StateF2",
      "type": "inject",
      "transition": "StateF4",
      "data": {
        "stateF2": "called"
      }
    },{
      "name": "StateF4",
      "type": "inject",
       "data": {
         "stateF4": "called"
       },
        "end": true
    }
  ]
}

wdyt?

Thank you guys for the replies. I am sorry if I make a mistake with the sections.
I am not an expert on this DSL, so I have a question that could be stupid: using this approach I think F3 needs to wait for the SB1 and SB2, creating a delay that should not exist.
The second question is a kind of "Discussion at the Bar", Don't you think this kind of workflow should be easier to define with the DSL? In some way, we used a workaround for describing a situation that could be very common.

Just to illustrate what @tsurdilo just said:
{
   "id":"parallelsubflows",
   "version":"1.0",
   "specVersion":"0.8",
   "name":"Parallel Subflow",
   "start":"StartState",
   "functions":[
      {
         "name":"funcF1",
         "operation":"spec.yaml#op1"
      },
      {
         "name":"funcF2",
         "operation":"spec.yaml#op2"
      }
   ],
   "states":[
      {
         "name":"StartState",
         "type":"parallel",
         "completionType":"allOf",
         "transition":"StateF3",
         "branches":[
            {
               "name":"BranchF1",
               "actions":[
                  {
                     "name":"actionF1",
                     "functionRef":{
                        "refName":"funcF1"
                     }
                  },
                  {
                     "name":"actionF5",
                     "functionRef":{
                        "refName":"funcF2"
                     }
                  }
               ]
            },
            {
               "name":"BranchF2",
               "actions":[
                  {
                     "name":"actionF2",
                     "functionRef":{
                        "refName":"funcF1"
                     }
                  },
                  {
                     "name":"actionF4",
                     "functionRef":{
                        "refName":"funcF2"
                     }
                  }
               ]
            }
         ]
      },
      {
         "name":"StateF3",
         "type":"inject",
         "data":{
            "foo":"bar"
         },
         "end":true
      }
   ]
}

I recommended using subflows to give you more flexibility. So you could use callbacks or any other constructs. Within the branches is also possible to achieve that, but you will be limited to function or event calls.

will close this issue for now, please feel free to open a new one or start a discussion if you have any further questions


Can't make "i" default letter for "4" key

On cat s22 flip, I am unable to set the default single letter to "i" when pressing the 4 key. Every other letter on the keyboard is able to be assigned as the default by holding the center/select button on the suggestion. When doing this with 4, G and H can be made the default letter, just not i.

I can confirm I have this issue on the same phone as well.

TT9 ignores holding the central button, it just passes it through to the current app. You can not use it for making any word the default one.
To do so, just type normally. Both the central key and 0-key (space key), will change the word priority, but don't hold them, use single press.
And if there is a problem with the dictionary, I believe it will be fixed in #366, meaning - in the next release. If you still have problems even in version 27 (when it comes out), please report again. Otherwise, I'll consider the bug fixed after the release.

I had this problem, where pressing 4 would return "g" when in predictive mode. What fixed it for me is adding "I" to dictionary with Asterisk key, pop up says "this is already in the dictionary" but now when I press 4 once it's returning I in predictive

Everyone, please try out the new v27.0. I believe it fixes the bug, but please let me know if it still persists.


Want to enhance the sorting function

The content is translated by Google, please forgive me if the description is wrong
When you open the database to view all tables, or view table properties, you can sort by display or column name by clicking the table header, but once sorted, you can only sort in ascending or descending order, and the original order cannot be restored. I hope that the sorting function can be enhanced. Revert sorting in addition to ascending and descending

Thank you for feature request.

We decided to fix this issue in the following way:

When clicking on a column header, sort the column ASC (just like now)
On a second click, sort it DESC (just like now)
On the third click return the original order


ability to add logging info when binding to non-existent properties

the idea is that sometimes we bind to a none existent property, and the logger should tell us that  this property does not exist.
but one of the great things about Aurelia is that sometimes you want to bind to dynamic properties, so making this an optional flag in logging verbosity level or something would make it better.

related to https://github.com/aurelia/binding/issues/23

maybe it would also be useful to log when dirty-checking is used in a browser with O.o support.

this came up in the gitter again today:  https://gitter.im/Aurelia/Discuss?at=55140f63c25916c13d35183d

I tried to look into this but got stuck at adding logging to property-observation.js
I changed the code there to inject the logger (similar to how it's done in other places)
System.register(["aurelia-logging"], function (_export) {
   var LogManager, _logger, ...
   ...
  return {
    setters: [
      function(_aureliaLogging) {
        LogManager = _aureliaLogging;
      }
    ],
    ...

and updated<EMAIL_ADDRESS>to include the new dependency, but this didn't work - until I also mentioned the new dependency in my app's config.js
System.config({
  "map": {
     ...
<EMAIL_ADDRESS>{
      "aurelia-logging"<EMAIL_ADDRESS>       ...
    },
   ...

why is that? Am I missing anything, or it's really necessary to update the main config.js?

Hey- thanks for looking a this, you're on the right track.  What you're running into is aurelia/binding doesn't have a dependency on aurelia/logging yet.  You'd need to `jspm install aurelia-logging' (which would add the entries in config.js).  config.js is typically maintained by jspm, usually you don't need to hand edit it.
The logging code might go higher up, in the ObserverLocater class, and should be configurable.  Reason is certain observer classes are used in certain scenarios, depending on the browser's support for O.o, etc.

Thanks for the explanations! Didn't realize jspm install is the way to change config.js
I'm quite new to all this so will need some more guidance. Can you look at my commit https://github.com/andreister/binding/commit/f616ec88494f22bf82e9a6b42b1e7fffc6a26f28 and see if it's close?
I'm not sure about your "...should be configurable...", what kind of configuration do you mean? I see configurable: true all over the code, but cannot yet understand how it's used.

Adding property existence checks in AccessScope, AccessMember, CallScope, CallMember could be bad news for performance, especially with AccessScope's ability to find props on the ancestor scopes.

Is there some way that we could have a debug mode...where when it is turned on the AST parsing substitutes different debug mode AST instances perhaps? Just thinking out loud here...

Sounds like a good idea to me. Will need to make sure the test coverage is beefed up in those areas. I think we could modify some of the existing AST tests to enable reuse between AccessScope and "DebugAccessScope" if that's the way we go about implementing this feature.

Sounds good. It would be nice to have something like this. We'll have to consider how it should be configured for debug mode. We may need to put a setting on the BindingEngine and then somehow surface that through the Aurelia FrameworkConfiguration object.

The configuration piece will be important to nail down for #205 as well

Good point.

@jdanyow @EisenbergEffect  Can we simplify it down to this line
When we try to do everything in getObserver but couldn't figure out the right observer, and the property doesn't exist in prototype chain, we log a warning ?
// Instead of
// return new SetterObserver(this.taskQueue, obj, propertyName);

// We do
this.logger
      .warn(`Attempted to observe non existing property [${propertyName}] on instance of [${obj.constructor.name}]. Fallback to SetterObserver`);
    return new SetterObserver(this.taskQueue, obj, propertyName);

My only concern would be whether or not this causes a perf regression.

we can add PLATFORM.isProduction property to use for this purpose, what do you think ? @EisenbergEffect


update gene prefix

three letters from genus
four letters from species
example : For Phytophthora sojae, gene prefix will be Physoja instead of Physo

after annotation, gff files look like this :
JAIKVF010000001.1       funannotate     gene    534     1529    .       -       .       ID=g1;name=g1;product=
JAIKVF010000001.1       funannotate     mRNA    534     1529    .       -       .       ID=g1.t1;Parent=g1;product=-;note=EggNog:ENOG503QAM1,SECRETED:SignalP(1-23,SECRETED:cutsite-VDA-HG,SECRETED:prob-0.7168)
JAIKVF010000001.1       funannotate     exon    534     1529    .       -       .       ID=g1.t1.exon1;Parent=g1.t1
if the species is Phytophthora melonis then the genes are named as Phyme_g1 in the database after uploading through galEupy.
Species name if available must be incorporated in this naming scheme. Suppose species is CJ26. According to new convention the gene name will be Phymelo_CJ26_g1 for gene g1.

Updated the organism prefix.


fix: remove pod config that break simulators on m1

@rathishubham7

@rathishubham7 btw please merge this as well or m1 simulators are broken


ble_server.cc include for freertos/queue.h

Error:
esp-idf/examples/git/Esp-Mobile-Apps-Esp32/EspApp/main/util/ble_server.cc:24:28: fatal error: freertos/Queue.h: No such file or directory
Fix:
change ble_server.cc include to case correct file name. current freertos queue is lowercase. 'freertos/queue.h'

Hi, @paulhaggard
Yes, I not detected it, due MacOS is by default case insensitive FS, I will change this.
Thanks for your feedback


feat: add steps to melos.yaml schema

Supported https://melos.invertase.dev/configuration/scripts#steps

This feature is very important for developing on Windows systems.
See this issue.


Missing output

I wanted to try out your tool as you recommended in my issue on kraken. I started it with:
./conifer --both_scores -s -i kraken.out.txt -d /scratch/databases/Standard_v2/taxo.k2d

then saw output
1000000 lines processed...
2000000 lines processed...
3000000 lines processed...
4000000 lines processed...
5000000 lines processed...
6000000 lines processed...
7000000 lines processed...
8000000 lines processed...
9000000 lines processed...
10000000 lines processed...
11000000 lines processed...
12000000 lines processed...
13000000 lines processed...
14000000 lines processed...
15000000 lines processed...
16000000 lines processed...
17000000 lines processed...
18000000 lines processed...
19000000 lines processed...
20000000 lines processed...
21000000 lines processed...
22000000 lines processed...
23000000 lines processed...
24000000 lines processed...
25000000 lines processed...
26000000 lines processed...
27000000 lines processed...
28000000 lines processed...
29000000 lines processed...
30000000 lines processed...
31000000 lines processed...
32000000 lines processed...
33000000 lines processed...
34000000 lines processed...
35000000 lines processed...
36000000 lines processed...
37000000 lines processed...
38000000 lines processed...
39000000 lines processed...
40000000 lines processed...
41000000 lines processed...
42000000 lines processed...
taxon_name      taxid   reads   P25_conf        P50_conf        P75_conf        P25_rtl P50_rtl P75_rtl

I expected to see more in the table. Any ideas what could cause this?

Hi, can you provide me with a sample of your input (lets say first 100 lines) and the taxo.k2d file?

I must admit that I'm not sure any longer on which file I tested this. I've had some success with other reports in the meantime. One thing I noticed is that I include taxa with zero assigned reads in the report and conifer includes those, too. That's not very useful and those lines could be excluded I think.
I also had one report with ~30,000 lines which lead to a segmentation fault.
./conifer: unrecognized option '--both-scores'
Segmentation fault (core dumped)

I think the message ./conifer: unrecognized option '--both-scores' is a false hint since the flag works fine with other reports.

Okay, I have been able to retrace the error above, it happens when running conifer on the kraken standard output rather than the report. So that's my fault. However, the points above remain. I could send you the report that leads to a segmentation fault by email if you like.

If conifer fails with kraken standard output, I would definitely be interested in checking the input files to make the software more robust.
As for kraken reports - Conifer, at least currently, is not designed to accept them as input. Conifer generates reports from kraken standard output.

Okay, I'll send you the files and maybe you can figure out what is going on.

It seems that the issue was with the taxid field in the input file.
Conifer assumed that the field contains only the taxid, without names.
I updated the code, to extract taxid from input, when kraken is executed with --use-names option.
See if it works now.

That works for me, thank you! Any idea why I get the following message, though?
./conifer: unrecognized option '--both-scores'

I believe it's a typo. Option should be provided with an underscore (--both_scores) or you can use the short version (-b)

Doh :facepalm: thanks :slightly_smiling_face:


Release 0.3.1

This pull request includes (pick all that apply):

[ ] Bugfixes
[ ] New features
[ ] Breaking changes
[ ] Documentation updates
[ ] Unit tests
[x] Other

Summary
Release 0.3.1

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@be90e2e). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #16   +/-   ##
=========================================
  Coverage          ?   93.48%
=========================================
  Files             ?       28
  Lines             ?      737
  Branches          ?        0
=========================================
  Hits              ?      689
  Misses            ?       48
  Partials          ?        0

Flag
Coverage Δ

#main
93.48% <ø> (?)

Continue to review full report at Codecov.

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


consider integrating top-k planners for task planning

example: https://github.com/speckdavid/symk

this one works better for our purposes: https://github.com/ctpelok77/kstar

Code runs on the topk branch. To run it, first clone our fork of the kstar repo (https://github.com/ronuchit/kstar) at the same directory level as your predicators repo, then run ./build.py.
We tried cover and blocks with debug grammar:
python src/main.py --env cover --approach grammar_search_invention --seed 0 --grammar_search_use_handcoded_debug_grammar True --excluded_predicates all
python src/main.py --env blocks --approach grammar_search_invention --seed 0 --grammar_search_use_handcoded_debug_grammar True --excluded_predicates all
Both work correctly, but are much slower than master:
master:
cover: Main script terminated in 46.27782 seconds
blocks: Main script terminated in 104.45043 seconds
topk:
cover: Main script terminated in 90.52044 seconds
blocks: Main script terminated in 145.83503 seconds

The evaluations for cover take around 7 seconds each for topk, but 0.1 seconds each for master. Upon investigation, we found that the issue is something related to time overhead from launching a shell process. You can see this discrepancy between "Total time" printed out by FD versus actual shell call time even in the example bundled with the kstar repo (run this command while in kstar/):
time ./fast-downward.py examples/blocks/domain.pddl examples/blocks/probBLOCKS-9-2.pddl --search "kstar(lmcut(),k=5)"

closing in light of comment above


“file does not exist or is not readable” in laravel while the image has been uploaded successfully

Laravel Version:6.4.0
PHP Version: 7.3.11

Here is the code in my project, in BlogController.php file and in store method :
public function store(Request $request)
    {
        $data = self::validate_and_upload($request);
        Blog::create($data);
        return redirect()->route('blog.index')->withMessage(__('CHANGES_MADE_SUCCESSFULLY'));
    }

validate_and_upload() method, validates incoming data in request and, uploads files from request into storage folder and finally returns an array. When Blog::create($data); is executed I face following error:

The "C:\xampp\tmp\php534B.tmp" file does not exist or is not readable.
This error is so confusing and even though file is successfully uploaded, it made me think that error is related to file upload process.
After I wrote following code in store method:
try {
  Blog::create($data);
} catch (\Exception $e) {
  dd($e);
}

and dumped $e I realized the actual error is something else.

The actual error was "Field 'author_type' doesn't have a default value", while Laravel showed me an unrelated error:  "The "C:\xampp\tmp\php534B.tmp" file does not exist or is not readable."
after I fixed the actual error problem was solved.
The problem is, this error page is so confusing and took me hours to find out the actual error. I thought this is happened only for me, but I saw a question on stackoverflow which proves it has happened for many Users. This is the link to stackoverflow question

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Github actions for more connectivity!

We can implement github workflows for the following things:

Auto comment on issue Closed and Opened
Auto comment on PR mergs
Auto comment on PR raises

Using this workflows makes you'r repo more interactive ! @rohitinu6
i'm expecting a level3 with gssoc-ext and hacktoberfest-accepted label

Added level3 label!! @rohitinu6


Add script to report image vulnerabilities

Example output:
$ scripts/vulnerabilities.sh
Scanning..........
IMAGE                                                                          VULNERABILITIES
gcr.io/config-management-release/admission-webhook:v1.15.0-rc.1                12
gcr.io/config-management-release/git-sync:v3.6.4-gke.1__linux_amd64            0
gcr.io/config-management-release/helm-sync:v1.15.0-rc.1                        46
gcr.io/config-management-release/hydration-controller:v1.15.0-rc.1             86
gcr.io/config-management-release/hydration-controller-with-shell:v1.15.0-rc.1  86
gcr.io/config-management-release/nomos:v1.15.0-rc.1                            12
gcr.io/config-management-release/oci-sync:v1.15.0-rc.1                         12
gcr.io/config-management-release/reconciler-manager:v1.15.0-rc.1               12
gcr.io/config-management-release/reconciler:v1.15.0-rc.1                       12
gcr.io/config-management-release/resource-group-controller:v1.0.9              0
ERROR: 278 critical, high, or medium vulnerabilities are fixable

/retest
Build log is weirdly empty, even though it ran 10m...

This script only works for the OSS version of Config Sync. Given that the mono-repo hasn't been deprecated yet, should we move the script to the operator repo and expand it to support mono-repo images as well?

That's the point. We want to make sure these are discovered and fixed before cutting a release candidate AND updating ACM AND cutting an ACM release candidate.
Shift Left!

+1 to @karlkfi
We should aim to shift this left, meaning we should probably have a similar script for both ACM and monorepo

/lgtm
/approve


If the array property addable is false, only the first element of the array is showed

If we define in our json the property addable === false, not all the array elements are showed, only the first one.
There is a wrong placed condition in layout.function.ts (line 310)

Moving the line 310 of layout.functions.ts to the line 340 (before
let buttonText:string = 'Add';
the problem is solved.
All the array is showed and the Add button isn't displayed

This bug has been fixed in the last update

I'm sorry, the bug still exits

I'm having trouble recreating this. Can you post an example schema and layout?

With the last update, you've solved it.
You've added this code to layout.functions.ts and now it works in all the cases. Thanks!
// If needed, add button to add items to array
if (newNode.options.addable !== false &&
newNode.options.minItems < newNode.options.maxItems &&
(newNode.items[newNode.items.length - 1] || {}).type !== '$ref'
)

@efrengp Got it. Thanks for the update.


URL signing & URL encoded characters

We have a problem with signing requests that have URL encoded characters e.g. comma.
For example, the original URL string on which I have calculated the signature is:
https://3.bp.blogspot.com/-U5eFGLc-DF8/WdJ6XXrTj-I/AAAAAAAAqKE/Kk5LN3ev5soLDlQ0Gpts_AllY5Q-Pu7kwCLcBGAs/s1600/Plitvice+National+Park%2C+Croatia+-+Posh%2C+Broke%2C+%26+Bored_1.JPG

But when image proxy is checking that signature, it will calculate it upon normalised URL:
https://3.bp.blogspot.com/-U5eFGLc-DF8/WdJ6XXrTj-I/AAAAAAAAqKE/Kk5LN3ev5soLDlQ0Gpts_AllY5Q-Pu7kwCLcBGAs/s1600/Plitvice+National+Park,+Croatia+-+Posh,+Broke,+&+Bored_1.JPG

I think that the right way for solving this would be that imageproxy is calculating the signature upon original request, req.URL.RawPath
The only thing that needs to be changed is:
data.go#L336
	path := r.URL.Path[1:] // strip leading slash

	path := r.URL.RawPath[1:] // strip leading slash

RawPath isn't always populated, so EscapedPath() is actually what is needed here.  Fixed, and thanks for the bug report!


[loc][query][VSCode] Please clarify "mocha"

Hi, we have a question from our translator. Can you help please? Thanks!
Source:
Debug mocha tests
File Name:
package.nls.resx
Resource ID:
".resx", 0; "node.snippet.mocha.description"
Query:
Could you please confirm "mocha" should remain untranslated?

Yes, mocha is a testing framework https://mochajs.org/


No sensor data available

I have installed the custom_component and can also start it within HA. However, I do not get any sensor data in the GUI. If I look into the home-assistant.log file, I get KeyErrors similar to:
KeyError: 'Sfrequency'
2024-01-30 23:00:59.526 ERROR (MainThread) [homeassistant.helpers.entity] Update for sensor.renac_t_frequency fails
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 898, in async_update_ha_state
    await self.async_device_update()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1218, in async_device_update
    await hass.async_add_executor_job(self.update)
  File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/renac/sensor.py", line 174, in update
    self._attr_native_value = self.updater.fetch(self.field)
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/renac/sensor.py", line 46, in fetch
    return self.data[field]
           ~~~~~~~~~^^^^^^^
KeyError: 'Tfrequency'

I hope you can help me with it, as I really would like to make the data of my PV system available within my HA. I do have a three phase hybrid inverter (model N3-HV-10.0), just in case this makes a different to your setup.

hops-1 -
I had this working very nicely until a couple of days ago. I now get the KeyErrors you describe. But I have also noticed (again for the last couple of days) that I cannot get my data properly from the web login page anymore unless I use the Asia or S.America data centre (mobile app still works fine). I suspect that the 2 issues are related so I was waiting to see if the data centre issue gets resolved before flagging it up with the dev. As an aside, I'd previously contacted Renac in China<EMAIL_ADDRESS>and they've told me that their R&D may look at implementing something for HA although I'm not holding my breath for that! I hope this gets sorted because it's clearly a good integration.

Hi datshi001! Thanks for sharing that you run into the same issue. Is there a way to change to the other data centre you refer to?

hops-1
log-in page (https://sec.renacpower.cn/#/access/signin) there's an option on the top right of the screen. Curiously the dashboard page works for the europe data centre - clearly something not right with the Renac set-up just now.

Indeed, the underlying API does not return any data about the Inverter anymore. The iPhone APP still works fine.
I'll check if I can do a quick change to the code to switch to a different data center (thanks for the hint @datshi001 ). I'll keep you posted

Give it a try to the version 0.0.6

version 0.0.6 working great for me :) thank you @gastush

Thanks @gastush for the really quick reply and updated version 0.0.6.
However, I still get the same kind of errors after the update and a reboot of the system:
2024-01-31 22:31:08.694 ERROR (MainThread) [homeassistant.helpers.entity] Update for sensor.renac_generated_power fails
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 898, in async_update_ha_state
    await self.async_device_update()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1218, in async_device_update
    await hass.async_add_executor_job(self.update)
  File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/renac/sensor.py", line 111, in update
    self._attr_native_value = self.updater.fetch(self.field)
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/renac/sensor.py", line 48, in fetch
    return self.data[field]
           ~~~~~~~~~^^^^^^^
KeyError: 'acPower'

What is your inverter model ? are you sure that  you used the right serial number (you can see it in the Renac SEC app) ?

With regards to the Serial number - yes, I double checked and it is correctly.
I am using a three phase hybrid inverter (model N3-HV-10.0) from Renac. The web interface is the same, but possibly the values provided by the API are different than for your model. I am not sure how to check this.

Indeed, the data output looks quite different. Note that the output for the On-Grid inverter is a bit different from what I get directly from the API... I'll see what I can do to make it work for the Energy Storage Inverter.
I guess that having the power of the battery makes sense in that case. I'll if I can make the creation of the sensors a bit more dynamic; so only the relevant ones are created.
It may take a bit of time as I don't have a system that I can use to test, but I'll keep you posted

Thanks @gastush - I wasn't expecting this, but I could offer to perform some tests on my system of course.
Apparently, there is no way to communicate directly via email anymore. However, you can reach out to me via the linkedin profile that I added to my profile just a minute ago.


PU-8 playstation , using arduino mini pro 8 mhz, no disk detected with psnee

PU-8 playstation , using arduino mini pro 8 mhz, no disk detected with psnee.
I have ps1 PU-8.
I used arduino mini pro 8mhz model.
PSNEE uploaded and the line for the mini uncommented.
All wires soldered as per diagram.
After the modchip is installed, the disks are spinning but i see the menu for the audio disk or memory card.
if i disconnect the chip the ps1 is working again with original disk.
What could be the issue?

hello a user pointed out to me that I had made an error in the wiring
diagram for the Pu-8 which I have just updated. check that you are using
the latest version of the PDF. If it persists it could be due to timing
issues.
Le sam. 16 déc. 2023 à 00:46, interfan @.***> a écrit :

PU-8 playstation , using arduino mini pro 8 mhz, no disk detected with
psnee.
I have ps1 PU-8.
I used arduino mini pro 8mhz model.
PSNEE uploaded and the line for the mini uncommented.
All wires soldered as per diagram.
After the modchip is installed, the disks are spinning but i see the menu
for the audio disk or memory card.
if i disconnect the chip the ps1 is working again with original disk.
What could be the issue?
—
Reply to this email directly, view it on GitHub
https://github.com/kalymos/PsNee/issues/47, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDOKBFJ22PVXHGFCXYIC63YJTOO5AVCNFSM6AAAAABAXECU3SVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA2DINBZHE2DSNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

My wiring is correct as per latest diagram.
I tried increasing the attempts to (inject symbols now. 2 x 3 runs seems optimal to cover all boards) from 2 to 3.
#define ARDUINO_328_BOARD
I have also tried #define APPLY_PSONE_PAL_BIOS_PATCH
I have also tried to set delay_between_injections o 72
Im tottaly blocked.....with this one.
Disk is spinning (original disk) , but i am sent to the screen for audio disk and memmory card.
How i can debug what is going on?
Any one with the same board that can help on this?


Have bot only report failures on maintained branches

Summary
Currently the QacoBot for reporting travis failures on commits is
reporting failures on temporary PR branches created on terra. There are
use cases for creating a quick branch to either test CI with secrets
(like to manually retrigger a failed wheel build) or a quick web ui
edit. However, failures on these branches should not have issues opened
against them because they are not maintained branches. This commit
updates the report_ci_failure.py script to only report a failure when
either master or a stable branch encounters a CI failure.
Details and comments

on hold for after the 0.14 release


feat: Code samples and tests for Retail Search Service are added

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@bcoe , @chenlei1216  Hi Lei, Benjamin,
could you please check the review comments and resolve the conversations if you are satisfied with the ansvers

I am closing this Pull Request in order to split the code into smaller pieces, please the first of which is: https://github.com/googleapis/nodejs-retail/pull/131

I am closing this Pull Request in order to split the code into smaller pieces, please the first of which is: #131


Added user input checks

Added the getInput() method that asks for input until it's valid. Also, a while loop in main that ensures the number of reveals isn't greater than the number of coins.

Approved!

LGTM :)

Approved!


RuntimeError: The size of tensor a (33) must match the size of tensor b (34) at non-singleton dimension 2

i try to run icnet with pascalvoc VOC2012, BUT i met this error:
train.py:230: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
  cfg = yaml.load(fp)
RUNDIR: runs/icnet_pascal/84704
Exception ignored in: <bound method _DataLoaderIter.__del__ of <torch.utils.data.dataloader._DataLoaderIter object at 0x7fdc35894278>>
Traceback (most recent call last):
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 399, in __del__
    self._shutdown_workers()
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 378, in _shutdown_workers
    self.worker_result_queue.get()
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/multiprocessing/queues.py", line 337, in get
    return _ForkingPickler.loads(res)
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/site-packages/torch/multiprocessing/reductions.py", line 151, in rebuild_storage_fd
    fd = df.detach()
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/multiprocessing/resource_sharer.py", line 57, in detach
    with _resource_sharer.get_connection(self._id) as conn:
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/multiprocessing/resource_sharer.py", line 87, in get_connection
    c = Client(address, authkey=process.current_process().authkey)
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/multiprocessing/connection.py", line 493, in Client
    answer_challenge(c, authkey)
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/multiprocessing/connection.py", line 732, in answer_challenge
    message = connection.recv_bytes(256)         # reject large message
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/multiprocessing/connection.py", line 216, in recv_bytes
    buf = self._recv_bytes(maxlength)
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/multiprocessing/connection.py", line 407, in _recv_bytes
    buf = self._recv(4)
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/multiprocessing/connection.py", line 383, in _recv
    raise EOFError
EOFError:
Traceback (most recent call last):
  File "train.py", line 242, in <module>
    train(cfg, writer, logger)
  File "train.py", line 140, in train
    outputs = model(images)
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/site-packages/torch/nn/modules/module.py", line 477, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/site-packages/torch/nn/parallel/data_parallel.py", line 121, in forward
    return self.module(*inputs[0], **kwargs[0])
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/site-packages/torch/nn/modules/module.py", line 477, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/xy/disk2/zhoubo/pytorch_semsge/ptsemseg/models/icnet.py", line 210, in forward
    x_sub12, sub24_cls = self.cff_sub12(x_sub24, x_sub1)
  File "/home/xy/anaconda3/envs/pytorch_semseg/lib/python3.6/site-packages/torch/nn/modules/module.py", line 477, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/xy/disk2/zhoubo/pytorch_semsge/ptsemseg/models/utils.py", line 764, in forward
    high_fused_fm = F.relu(low_fm + high_fm, inplace=True)
RuntimeError: The size of tensor a (33) must match the size of tensor b (34) at non-singleton dimension 2

i try it with pytorch0.4.1 and 1.0,  AND my config is like this:
model:
    arch: icnet
data:
    dataset: pascal
    train_split: train_aug
    val_split: val
    img_rows: 'same'
    img_cols: 'same'
    path: /home/xy/disk1/pascalvoc/VOCdevkit/VOC2012/
    sbd_path: /home/xy/disk1/pascalvoc/benchmark_RELEASE/
training:
    train_iters: 300000
    batch_size: 1
    val_interval: 1000
    n_workers: 16
    print_interval: 50
    optimizer:
        name: 'sgd'
        lr: 1.0e-10
        weight_decay: 0.0005
        momentum: 0.99
    loss:
        name: 'cross_entropy'
        size_average: False
    lr_schedule:
    resume: icnet_pascal_best_model.pkl

How can I solve this problem? Thank you

Hi，have you solved the problem? I had a similar problem.can you help me?

I have the same problem.

我已收到，谢谢


Spike - Figure out how users should extend code-lists

Should the user have do do some editing in a CSV to build up the new code-list themselves?
Should the user be able to extend a code-list using a python API where they say 'add a new concept to this code-list'
Do we need an info.json approach to allow users to extend code-lists there?
Can we restrict this to just one code-list at a time or should the user be able to splice multiple code-lists together?

AND

What is our approach:

Do we use a local codelist using owl:sameAs to refer to the existing URIs from other codelists -- this should presumably then be 'normalised' so we don't need the owl:sameAs to be pushed to PMD (to avoid performance problems). i.e. the local code-list URI gets replaced with the existing URI in Jenkins. YES

The csvcubed components to use when creating these mixed/extended code-lists are:

CompositeQbCodeList
DuplicatedQbConcept

This has already been implemented. We use the owl:sameAs relationship to link concepts back to their original meaning.


data-diff --version

Is your feature request related to a problem? Please describe.
When reporting bugs or posting feature requests, it would be nice to be able to quickly obtain currently installed data-diff version. For instance, by executing
data-diff --version

Absolutely agree, we'll add it!

Merged to master


use sphinx autosummary to make one doc page per function

I'd like to change to API documentation format to have:

a separate summary page for each module
a separate detailed page for each function

see the new documentation here:
http://wholmgren-pvlib-python-new.readthedocs.org/en/doc-reorg2/modules.html
I think it's a lot easier to navigate this. Separate pages for each function also makes it reasonable to add example documentation for each function (like the matlab library already has).

Looks good except for the scrollbars on the tables. You might try some custom CSS so that the descriptions wrap like in https://github.com/qtile/qtile/commit/3eadbbf20adf975ed2ac502f59a6a7d854b7de91.

Also, is there a way to have the +- boxes in the menu on the left for the autosummary items (like in Time and time zones)?

I applied that scroll bar patch. Nice find. The some of the tables look a little messy, but I prefer this to scrolling.
I noticed earlier that the +/- symbols were not being created, but I couldn't figure out how to fix it.
autosummary is creating new pages for imported functions and classes. Not sure if this due to a sphinx bug or if I'm doing something wrong.

I forgot to add that I am fairly sure that the duplicated pages are due to a sphinx bug, and I'm waiting to merge this until they fix it.

closed in favor of #258.


ESP8266 Community board upgrade to >2.5.0 breaks the parser

Just heads up. At least for me, this cool parser stops work well once ESP8266 board is upgraded from 2.3.0 to 2.5 or higher. I am using this to parse Home Assistant data. Once board is upgraded, parser does return mismatch values.
On Esp8266 2.3.0 still works well :) 👍

Try with:
https://github.com/mrfaptastic/json-streaming-parser2
Because this library isn't maintained anymore it seems .

how do i parse from this
https://data-live.flightradar24.com/zones/fcgi/feed.js?faa=1&bounds=3.462%2C2.438%2C101.156%2C102.504&satellite=1&mlat=1&flarm=1&adsb=1&gnd=1&air=1&vehicles=1&estimated=1&maxage=14400&gliders=1&stats=1&enc=aUZxeUj-cudNzTMJ_LNTMhchOm2J_TlYbC5HPIDKD_0
if any vidoe toturial please give link.

I suggest using the ArduinoJson library. Much easier and lots of documentation.
Try using ChatGPT to get example code.

Try with:
https://github.com/mrfaptastic/json-streaming-parser2
Because this library isn't maintained anymore it seems .

So, cool! Many thanks! I will start my testing soon. Looks like your parser is based on squix78 which means easier jump to yours parser.

So, cool! Many thanks! I will start my testing soon. Looks like your parser is based on squix78 which means easier jump to yours parser.

Yep. If you're already using squix78's then you should be able to migrate across to parser2 with minimal effort.

Yep. If you're already using squix78's then you should be able to migrate across to parser2 with minimal effort.

Indeed, it was quite simple.
Many thanks for your work on parser2 (anto to squix78 for parser1)!
I am parsing data from Home Assistant over parser2. Now testing and all good.
Actually, thanks to opportunity on working on implementing parser2, I noticed that issue I initially hit with parser1 /squix78/ (subject of this thread) also occurs on parser2, so may be caused not by parsers but something different than parsers or ESP board updates.
I think issue was around the way how data is pulled from server, means networking issues.
For parser1, for each entity I pulled data from server by following way. Parser2, time to time, returned wrong data. Same as parser1.
    while (client.connected()) {
      while ((size = client.available()) > 0) {
        c = client.read();
        //Serial.println(c);
        if (c == '{' || c == '[') {
          isBody = true;
        }
        if (isBody) {
          parser.parse(c);
        }

Probably in a way above, in case of weak WiFi signal, reading source XML byte by byte could simply miss some characters. Then, if missed was "{", parser probably could take values from other entities or it took something from "cache". As a results, let's say state of switch #1 had state of switch #2. Or state has parsed value from temp. sensor which couldn't happen in expectations. It happen not so often so my systems worked well enough to live with it (always trying to have good wifi coverage).
The way you proposed in parser2 is to pull data is based on http library, so probably over TCP:
(http.writeToStream(&parser));
Looks like this way, ensures pulling "confirmed" xml from server over http or nothing will be parsed. Thanks to that, parsed data is 100% accurate or there is no data at all, which is great approach.
I confirmed this in initial testing - 23k queries and 100% accurate.
Many thanks for your help here! :)


mLab upgrade of mongo version to 3.0 and 3.2 in September

This is more a question / discussion point so if it belong elsewhere just let me know. :-)
Received this email from mLab this morning and would like to get a steer on whether it poses a problem to the parse-server-example deployments that are using Heroku and mLab.
You are receiving this email because you have one or more mLab deployments in the "bmacdonaldyyz" mLab account that will be affected by upcoming maintenance.
"MongoDB 3.2 is now generally available on mLab, so we are in the process of making it the default release version for all new mLab deployments. We are also taking steps to discontinue our support for MongoDB 2.6.
To this end, on Tuesday, September 27, we will start a four-day maintenance window to upgrade all free Sandbox databases running MongoDB 3.0 to MongoDB 3.2. In addition, we will upgrade all for-pay Shared databases running MongoDB 2.6 to MongoDB 3.0. Dedicated plan deployments will not be impacted."
Thanks...Brian

parse-server is compatible with most versions of mongoDB and we expressively test on 2.6.11, 3.0.8 and 3.2.6, and we're planing to keep it on moving forward.

can we update the documentation so that this is noted? https://github.com/ParsePlatform/parse-server/wiki/Parse-Server-Guide#database

Yup.  Will do.

On Sep 23, 2016, at 11:06 AM, Dan Huang<EMAIL_ADDRESS>wrote:
can we update the documentation so that this is noted? https://github.com/ParsePlatform/parse-server/wiki/Parse-Server-Guide#database
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@acinader, i updated already :)

:)


⚠ GitLab Server has degraded performance

In ca39f8c, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6232 ms

Resolved: GitLab Server performance has improved in faa6e4e.


bug(CdkPortal): Change detection loop when component portal uses NgModel

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Adding a ngModel to a ComponentPortal makes change detection loop!
Reproduction
Just clone repo:
https://github.com/meriturva/cdkPortal-NgModel-CD-Loop
Please comment on that line to make it work again:
https://github.com/meriturva/cdkPortal-NgModel-CD-Loop/blob/main/src/app/text-editor/text-editor.component.html#L4
Expected Behavior
Works correctly!
Actual Behavior
Loop during change detection.
Environment
Angular CLI: 17.3.7
Node: 20.12.2
Package Manager: npm 10.5.0
OS: win32 x64
Angular: 17.3.7
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, platform-browser, platform-browser-dynamic, router
Package                         Version
@angular-devkit/architect       0.1703.7
@angular-devkit/build-angular   17.3.7
@angular-devkit/core            17.3.7
@angular-devkit/schematics      17.3.7
@schematics/angular             17.3.7
rxjs                            7.8.1
typescript                      5.4.5
zone.js                         0.14.5

This line in your code is the problem: https://github.com/meriturva/cdkPortal-NgModel-CD-Loop/blob/main/src/app/viewer/viewer.component.html#L3
It means that every time change detection runs, it'll return a brand-new portal which will then re-render since Angular will compare them by reference. You should consider caching the portals so that they're only created once.

Thanks @crisbeto for loop with track options it doesn't prevent re-run cd. Thats clear. But the introduction of Signal Components and local change detection could prevent it in the future?

Signals might prevent it from running continuously, but having getComponentPortal return a new portal every time is still inefficient since the portal itself isn't changing.


chore: added async to currencies.json

was missing
no tests

https://www.npmjs.com/package/async
https://www.ibm.com/docs/en/instana-observability/current?topic=technologies-monitoring-nodejs#async

Created https://jsw.ibm.com/browse/INSTA-14068

@kirrg001 I'm reviewing our currency report and noticing that the support policy for Timers has been set to 45 days. Since Timers is a native Node.js module, could this be  0 day instead of 45?


Specify python.tensorBoard.logDirectory for each workspace

Currently, vscode provides a setting option 'python.tensorBoard.logDirectory' for its integrated tensorboard service to launch from. However, this setting item seems only effective in the user setting file but not the workspace settings(it doesn't work when adding 'python.tensorBoard.logDirectory' item to the workspace setting.json, a popup asking launch place for tensorboard still shows up, or am I getting something wrong?). I think it's a common demand to specify a log directory for each workspace, please consider this feature to make vscode's tensorboard experience more fluent.

Thanks for the feedback--we'll discuss at our team's weekly triage meeting.


Hightlights 1. element since Angular 1.3.1

Since Angular version 1.3.1 the first element will be highlighted and not the element which is defined in the js. After clicking on an element, it highlights correctly.
See: http://plnkr.co/edit/Sza2ZshnNlU4jdjSOFaP?p=preview
It was not a problem in version 1.3.0 and before

I updated your plunker with last library version and looks like its working correctly now.
Updated plunker

Great thanks for a fast fix!


Remove \lang in prefer of dictionary grammers.

Specifying \lang is complicated to translaters. The type of paras could be HashMap<Locale, Vec<Paragraph>>.

47f2f2d


Use of private functions from OreDictionary.java?

I don't quite understand why this is only an issue in IntellJ and not Netbeans, but I can't compile the API classes without editing the following lines:
https://github.com/Azanor/thaumcraft-api/blob/master/crafting/InfusionRecipe.java#L83
https://github.com/Azanor/thaumcraft-api/blob/master/crafting/InfusionRecipe.java#L83
https://github.com/Azanor/thaumcraft-api/blob/master/crafting/InfusionEnchantmentRecipe.java#L99
https://github.com/Azanor/thaumcraft-api/blob/master/crafting/InfusionEnchantmentRecipe.java#L101
The called OreDictionary functions are all depreciated and scheduled for removal in 1.8 as well.
The error I get is below:
Error:(85, 65) java: C:\Users\CovertJaguar\Desktop\Minecraft\Developement\Projects\Railcraft\RailcraftDev\api\thaumcraft\thaumcraft\api\crafting\InfusionRecipe.java:85: getOres(int) has private access in net.minecraftforge.oredict.OreDictionary
Error:(101, 65) java: C:\Users\CovertJaguar\Desktop\Minecraft\Developement\Projects\Railcraft\RailcraftDev\api\thaumcraft\thaumcraft\api\crafting\InfusionEnchantmentRecipe.java:101: getOres(int) has private access in net.minecraftforge.oredict.OreDictionary

It works fine in eclipse as well. Very strange.


Add test coverage check for pull request and set threshold as 0.1%

What is the purpose of the change
Add test coverage check for pull request
Brief changelog

modify codecov configuration for pull request only
set threshold as 0.1%
reviewed https://codecov.io/gh/apache/incubator-dubbo/pulls. for small change, the coverage change is round 0.01%--0.06%. As discussed in mail list, considering small change, I put threshold as 0.1%

Verifying this change
Codecov config is valid.

Codecov Report

Merging #1678 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1678      +/-   ##
============================================
- Coverage     35.49%   35.45%   -0.04%
+ Complexity     3780     3776       -4
============================================
  Files           628      628
  Lines         30976    30976
  Branches       5448     5448
============================================
- Hits          10994    10983      -11
- Misses        18089    18099      +10
- Partials       1893     1894       +1

Impacted Files
Coverage Δ
Complexity Δ

...n/java/com/alibaba/dubbo/common/utils/JVMUtil.java
72.22% <0%> (-7.41%)
13% <0%> (-2%)

...libaba/com/caucho/hessian/util/IdentityIntMap.java
25% <0%> (-5.96%)
5% <0%> (-2%)

...om/alibaba/dubbo/rpc/filter/ActiveLimitFilter.java
83.33% <0%> (-5.56%)
6% <0%> (ø)

...ubbo/rpc/protocol/dubbo/ChannelWrappedInvoker.java
41.66% <0%> (-4.17%)
3% <0%> (ø)

...java/com/alibaba/dubbo/config/ReferenceConfig.java
47.97% <0%> (+0.73%)
36% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

Looks good, it will merge into 2.6.3.

close #1638


Supabase auth admin missing proper getUser function

Is your feature request related to a problem? Please describe.
The NodeJS version has an admin function called getUser, this function accepts a custom token and practically allows verifying the user identity when handling a request in a custom server environment. This function seems to be missing in the current Dart package.
Describe the solution you'd like
A function with the same functionality being added to the package
Describe alternatives you've considered
At least describe the API request needed to get the same information (I tried reading the source code and trying to replicate the desired API request myself, but I couldn't find all the needed info)
Additional context
Example use case: There's a custom-made Dart TCP server that needs to be integrated into the project. The server first needs to verify the identity of the connecting socket and drop the connection if it's not authentic. It gets a special auth request package, parses it, gets the user access token sent by them, passes this token to a supabase.auth.admin.getUser(token) function and in return either gets a valid user object or null, indicating that the verification has failed.
Edit: added example use case

Yeah I know and I'm doing exactly that right now, but the downside is that I have to store it as env var on the server side which means a) it has to be manually updated if I change it in Supabase settings b) it's just a sensitive info that I want to avoid storing:) Huge thanks for the pull request!

Also, just to make sure that I understand everything correctly, you said

You can use the jwt secret to encrypt the header and payload
but I guess you meant "sign" instead of "encrypt", right?

Yeah I meant to sign the header and payload.


NR-52118 chore: create spec file

This PR adds the spec file for the e2e.
The only consideration here it's that there're a ton of metrics that are reported as untyped type. By default they are set to gauge by the tool. I tried to check them but there are >1k metrics fitting this case. So, it's hard for me to say that all of them are properly set 😅
All the exporter metrics are listed in this internal sheet: https://docs.google.com/spreadsheets/d/1aYdE58VQryLJ4TGFGvOW9ZlPvB2oWwrFNY5ECANeILU/edit?pli=1#gid=1357724511

No idea why you requested reviewers by they are not in the column 🤔 I'll try to add them again

I saw that there are multiple metrics with many dimensions:
     dimensions:
          - name: cl_id
            type: string
          - name: cl_role
            type: string
          - name: member_idx
            type: string
          - name: member_state
            type: string
          - name: rs_nm
            type: string
          - name: rs_state
            type: string

Is this expected? Should we ignore some of them? I guess not, but asking just in case
Wondering because of this:

CAUTION: Remember that every unique combination of key-value label pairs represents a new time series, which can dramatically increase the amount of data stored. Do not use labels to store dimensions with high cardinality (many different label values), such as user IDs, email addresses, or other unbounded sets of values.

Some metrics look weird, they are generated, so nothing should be wrong, but can we doublecheck?

mongodb_members_optimeDurable_t
mongodb_oplog_stats_wt_LSM_queries_that_could_have_benefited_from_a_Bloom_filter_that_did_not_exist

Sorry for the noise but I just saw that I didn't update the PR with the latest updates I did changing last missing metrics from gauge to counter.


Implement TryFrom for Option<i32>

This closes #17.
I tried, although it may not be adequate. Please feel free to request any changes if there's something you don't like!

I mean something like the following:
impl<T> TryFrom<EString> for Option<T>
where T: TryFrom<EString>

but seems it doesn't possible. We need to create our own trait instead TryFrom


#pragma GCC issue warning with MSVC

#pragma GCC issue warning with MSVC
Hello ! with the latest change in https://github.com/SFML/SFML/blob/master/include/SFML/System/Vector2.inl from commit https://github.com/SFML/SFML/commit/cd985e37d2239437b33449b95466579c4239d82b
I get this warning at compilation :
SFML/System/Vector2.inl(40,9): warning C4068: unknown pragma 'GCC'
Your environment

windows 10
latest SFML master
MSVC

I checked the code and I think it could be easily fixed by either adding
#if defined(__GNUC__)
around the pragma or removing it totally and using other names for the parameter. Would it be okay to do this ? Would you like me to do the change and do a Pull Request ?
thanks for your awesome library !

That's my bad. I know how to fix it and want to get that patched soon. In the meantime you can add /wd4068 to work around this.

No problem ! Thanks the quick answer

https://github.com/ChrisThrasher/SFML/commit/77dacb2c20bbe8faa631ad72efcbc33faa4163ce
Does this fix your problem?

Check out #2267 and let me know if that works.

Thanks for the fix, I can confirm it works

Thanks for reporting this! I merged the fix.


Missing Permissions during GitHub Actions

When using GitHub Actions I had to add the following permission to the IAM policy:

codebuild:StartBuild
codebuild:BatchGetProjects

      {
        "Action": [
          "codebuild:BatchGetProjects",
          "codebuild:StartBuild"

        ],
        "Effect": "Allow",
        "Resource": "****",
        "Sid": "Projects"
      },

Fixed in https://github.com/apppackio/apppack-backend/commit/6176e30963259da628c52a0b9b8f512ee7c77580


fix: correcting node-exporter flags that blocked seeing sysd metrics

I noticed that our node-exporter was having a bad time trying to ship a few metrics. This will ensure that it knows where to look for proc, sys, and the rootfs while also being able to pull metrics from systemd. Notably, we can get unit stats so we can tell if a service is thrashing.
[root@ip-12-0-10-253 ec2-user]# curl localhost:9100/metrics | grep rebaser
...
node_systemd_unit_state{name="rebaser.service",state="activating",type="exec"} 0
node_systemd_unit_state{name="rebaser.service",state="active",type="exec"} 1
node_systemd_unit_state{name="rebaser.service",state="deactivating",type="exec"} 0
node_systemd_unit_state{name="rebaser.service",state="failed",type="exec"} 0
node_systemd_unit_state{name="rebaser.service",state="inactive",type="exec"} 0

bors merge


Add missing Zowe components to the description section

Signed-off-by: nannanli<EMAIL_ADDRESS>fixes #https://github.com/zowe/docs-site/issues/1471

@jmertic Hi John, this PR has been reviewed by Joe. Could you take a look and help merge it? Thanks!


Add support for ArrayIterator

ArrayIterator's "getArrayCopy" can return something better to serialize than the object itself.

Obviously this need tests

Does anyone understand the reasoning behind Encoder:133-139? There are checks and tests against Iterator and IteratorAggregate yet no conversions to an array. I would assume this pull request could be changed to just change those lines to _encodeArray(iterator_to_array($value)), correct? I am concerned about BC breaks in that case.

That part of code is covered with these tests https://github.com/zendframework/zend-json/blob/ac34203e60c19e0b89f16673332a7ddbeb5e1ea7/test/JsonTest.php#L748-L770
Feel free to change the code. If tests fail then there is a BC Break.


Question of install

error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> [15 lines of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

This doesn't seem to be a problem with HAPPE or pip. Can you provide more information?


perf: Avoid unnecessary intermediate JsonNode

This is the more aggressive version of https://github.com/wiremock/wiremock/pull/1772 that eliminates all four major stacks in the flame graph below--not just the highlighted ones.
Dealing in terms of Body as a first-class entity separates out concerns nicely. PR diff is LOC negative already, but we could also delete ResponseDefinitionBuilder.with...Body(...), ResponseDefinition.get...Body, and ResponseDefinition(..., bytes[] body, ...) if you don't mind the breaking API changes. We could also deprecate them for a while, or just leave them as is. Let me know your preference.
I'm already enjoying the performance benefits of this change via ResponseDefinition.class.getDeclaredField("body").setAccessible(true) etc., but it'd be nice to switch back to public APIs.

This looks good.
I think deprecating the with...Body() methods in 2.x then removing in 3 is the best approach. If you're happy to do add the deprecations I'll merge this one.

There are 100+ uses of the .withBody() variants. Shall I migrate those at the same time?

I'd suggest merging as-is to start getting the benefits of the perf improvement. Minimizing the public API footprint is an opportunity, but not critical. All the existing methods still work properly.

Agree it's worth merging now and building on. Thanks for submitting this!


Revert ae3555f0dfe8eda554543f80c0d0c383836a9a64

Reverts crowbar/crowbar#2159, which was merged prematurely. It causes 100% installation failures and turned all gating red.

+1

+1

I've added the revert to the package for now. Until we decide about this one.

We can close this in favor of #2183


Dungeon Diving

Implemented switch for Dungeon Diving mode

Some minor problems.
When the settings are booleans the value has to be a string, aka "true" (for triforce_hunt)
The trials setting is named trials_random.
The locations part of the plandofile is not stored inside the settings list.
It's a separate list on the same level as settings.
Can you also add at the top of main() so i can pass in an argument
# Load in command line args
    if len(sys.argv) >= 2:
        print(sys.argv)
        if sys.argv[1] == "Dungeon_dive":
            global DUNGEON_DIVING
            DUNGEON_DIVING  = True


feat: leverage + historacle

Description
In progress - ensure that every MsgBorrow, MsgDecollateralize, and MsgWithdraw is valid under both current and recent historacle prices.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] added appropriate labels to the PR
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Codecov Report

Merging #1654 (84065b5) into main (0903f03) will increase coverage by 0.88%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1654      +/-   ##
==========================================
+ Coverage   54.02%   54.91%   +0.88%
==========================================
  Files          73       73
  Lines        7536     7556      +20
==========================================
+ Hits         4071     4149      +78
+ Misses       3148     3087      -61
- Partials      317      320       +3

Impacted Files
Coverage Δ

x/leverage/keeper/oracle.go
46.82% <0.00%> (-17.31%)
:arrow_down:

x/oracle/keeper/historic_price.go
72.02% <0.00%> (+6.42%)
:arrow_up:

x/oracle/abci.go
82.52% <0.00%> (+75.72%)
:arrow_up:

@toteki would you like a review here? Is it just in draft because we haven't cut v3.3 yet?

@adamewozniak I've got a massive diff locally on the unit tests that I need to finish first - then a few other things to clean up (modify QueryMaxWithdraw to do both prices, add a parameter) and I'll mark R4R.
Also some new queries (as opposed to modified above) will be a separate PR

Extracted style comments to new issue
https://github.com/umee-network/umee/issues/1681
to accelerate review


Fix:205 bug in acquiring stopsfeather

Fixes #205
Thanks to great help from @nkshaw23 we have managed to get the bug sorted.
A better and more robust solution might involve integrating the API calls and 28 day check into the any_to_pd function but what we have created here is sufficiently robust and runs fine.

@jwestw We ran this first time and had "Angus" as our LA, where we received an error on line 288 (when changing the number of disabled into an integer). This error was: Cannot convert non-finite values (NA or inf) to integer
We believe this to be because the LA_df does not have all the local authorities in the dataset! We have ran this with 2019 and 2020 as our calculation years and it seems to have the same issue. We will create a ticket explaining this more.

I think problems like this could be avoided more easily if this module was re-written in OOP. Then we could share a parameter like self.stops_path amongst the different methods in this context.
I made a ticket about this ages ago (https://github.com/ONSdigital/SDG_11.2.1/issues/85), and also attempted the re-write a while back, but found that it was a bit too much of a lengthy task to take on myself while there was so much else to get on with.

Merging without Paige re-approving as the requested changes have been made but Paige is away.


Version 4.0 Error: Use of const in strict mode

Freshly installed, and this is after npm install.
Oddly enough, for some commands, running node --harmony ./zenbot.sh <command> works, but for most it doesn't.
zenbot trade --paper
fetching pre-roll data:
/home/steve/zenbot/node_modules/gdax/lib/request_signer.js:2
const crypto = require('crypto');
^^^^^
SyntaxError: Use of const in strict mode.
    at Module._compile (module.js:439:25)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/home/steve/zenbot/node_modules/gdax/lib/clients/websocket.js:5:19)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)

I originally had this error when I was using nodejs legacy, after I updated it to proper nodejs the error went away. Idk if this is related to your issue.
You might want to uninstall all npms and do a reinstall(though u said it was a fresh install).

I installed node via sudo apt-get install nodejs npm

You might want to check your node version.
I installed with the below commands. (No quotes)
"curl -sL https://deb.nodesource.com/setup_7.x | sudo -E bash -"
"sudo apt-get install -y nodejs"
Just remember that i'm not 100% sure this is your issue.

Yes, that was the issue, thanks!

Awesome happy it worked out


Conversational learning page hero section #79

Description
:lipstick: :art: created the conversational learning page hero section #79
Fixes  #79:
Please delete options that are not relevant.
Screenshot (optional)

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes

Good work. The work looks nice, though a bit off from the design. Good pr name title. Use an emoji next time. You can use github desktop to solve the conflicts. Always just touch the required files for the pr.

Thankyou very much for your positive remarks, it was tough honestly and i will work on its perfection.

Good work. The work looks nice, though a bit off from the design. Good pr name title. Use an emoji next time. You can use github desktop to solve the conflicts. Always just touch the required files for the pr.