The new rename tracking feature is slow

I have tested the current master with the --track-ranames flag set on one of my photo disks, 21 GB of fairly large RAW files, renamed 5 of them and then sync to another USB3 disk.
My implementation (https://github.com/bep/rclone/tree/renames):
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:               346
Transferred:            0
Elapsed time:       10.5s
11.32s  user 2.34s system 88% cpu 15.432 total
avg shared (code):         0 KB
avg unshared (data/stack): 0 KB
total (sum):               0 KB
max memory:                175496 MB
page faults from disk:     3630
other page faults:         365136

Current master:
Transferred:   6.004 kBytes (15 Bytes/s)
Errors:                 0
Checks:               704
Transferred:            1
Elapsed time:     6m29.5s
110.73s  user 16.39s system 31% cpu 6:39.87 total
avg shared (code):         0 KB
avg unshared (data/stack): 0 KB
total (sum):               0 KB
max memory:                173104 MB
page faults from disk:     4112
other page faults:         696919

From the log it looks like it performs hashing of every file, which sounds very sub-optimal in all the common use cases I can think of.
I have repeated the test twice with similar results.
I will continue to use my own fork for the time being, but I thought I would give a heads up.

Eeek!  Will look at it tomorrow. It should be easy to fix though.
Thanks for testing.

OK I fixed that hopefully - I've put it in a fix-track-renames branch - can you have a go with that and see what you think?
Thanks
Nick

OK, now it is just half as slow as my implementation for the test case above, which still sounds expensive, considering the added code complexity.
But, up to you.

Can you work out why it is slower?
I note that it could be optimized further by only calculating the hashes of files which have a size which might match - that is probably the difference.
The trouble with your implementation is it has an O(N^2) loop in, so if you, say, moved all your photos (10000 say) to a subdirectory and did a sync, then rclone would be doing 10000x10000 times around the loop
Will investigate some more tomorrow - past my bed time now, so probably none of the above makes any sense ;-)

Can you work out why it is slower?

Asking me to debug your code is a tall order.
As to algorithmic complexity, I assume both solutions is  O(N^2)  in the worst case scenario and also assuming that the hashes are cached on the file level, it should not make much difference. The only way I see around that would be to implement some folder renaming logic.
I did a quick test of that too, moving a folder with the same 21 GB of files (~350 files) one level down.
The fix-renames branch:
Transferred:   12.008 kBytes (16 Bytes/s)
Errors:                 0
Checks:               710
Transferred:            2
Elapsed time:     12m7.4s
191.83s  user 27.70s system 29% cpu 12:16.94 total
avg shared (code):         0 KB
avg unshared (data/stack): 0 KB
total (sum):               0 KB
max memory:                170560 MB
page faults from disk:     4153
other page faults:         694608

My code ((https://github.com/bep/rclone/tree/renames):
Transferred:   6.004 kBytes (10 Bytes/s)
Errors:                 0
Checks:                 2
Transferred:            1
Elapsed time:     9m41.7s
187.39s  user 26.75s system 36% cpu 9:51.66 total
avg shared (code):         0 KB
avg unshared (data/stack): 0 KB
total (sum):               0 KB
max memory:                173552 MB
page faults from disk:     3374
other page faults:         686099

I'm checking out of this discussion. It is your project, you do as you please.
I can understand and sympathise with the motivation to guard your own code, so I will say this in the nicest possible way:
Taking pull requests and repeatedly asking for improvements, then saying "the only thing missing now is a test or two", and "oh, and could you add some documentation?". And when you finally "merge" it, you wipe out everything but the testing and the documentation without any discussion.
You should not get surprised when you don't get any "repeat Pull Requests" from people.
Ref. issue #888

Firstly I'd like to say thank you for your contribution.
I'm sorry you feel that I wiped out your code. I overstepped the mark in changing the code without discussing my concerns about it with you first.  Please accept my apologies.
Yes I am probably too picky about the code, and I'll try to take your comments on board for future pull requests.  I definitely haven't mastered the art of being an open source maintainer so I'm always willing to learn.
Thanks for your feedback
Nick

I'm sorry you feel that I wiped out your code.

You did wipe out my code and replaced it with some hard to understand and buggy code. No feelings involved.

@bep mate he's the project maintainer, the second he merges in "your code", it becomes his responsibility to maintain it. When it breaks, he has to fix it. During refactoring, he has to include that code. Really, the second it merges into his repo, it's not "your code" anymore. It's completely normal to ask for tests, documentation, refactoring, or to even simply reject a patch if it doesn't meet the maintainer's (subjective) standards. That's the point of being a maintainer.
I can't tell from a glance whose patch makes "more sense" or "is cleaner", so you may very well be objectively right, here. Nonetheless; your comment is uncalled for. Submit a PR, or don't, and move on. Don't make it personal.
Maintaining an open source project is more than just green lighting patches from strangers. It's unrewarding, ungrateful work, and comments like this can be demoralising.

@hraban don't comment on something you have no clue about.

@bep If I lived by that rule, I would be mute.

@hraban @leonwilliams I'm going to give you the benefit of doubts and assume you really don't understand this situation.
This isn't about someone rejecting or deleting my code. I do that all the time to others. That is fine.
This is all about wasting peoples time. This was an ongoing PR for weeks with "this looks very good, but could you just adjust this part over there ... " and then ... boom PR removed without any explanation.*
That is bad maintainer manners from @ncw -- and I would assume he have learned a lesson from it.

Thank you for the context.


Apply "Navigation Drawer" screen design

Overview (Required)
The title says everything.
Make it previewable(Prease add @Preview methods) and try to implement it paying attention to state hoisting.
Links
https://www.figma.com/file/IFlrbfmBSdYvUz7VmSzfLV/DroidKaigi_2021_official_app
There are some parts where the design size is not accurate. Use 4dp * x size and similar Material components.
Please log in to see the size on the design.

@takahirom
I will try this issue!

Thanks!  The navigation drawer is also an important feature you often see! Assigned!🙌

@takahirom
Could I download icons at 24dp from Figma?
Some of the icons can be used from Vector Asset in Android Studio, but for example the podcast icons are original and should be downloaded from Figma.
I'm not used to using Figma, so please let me know.

👀

Probably you can export SVG and it can import from Android Studio 👀
Maybe you need to log in?

I logged in to Figma and I can find icon.
But when I press the Export button, show Export failed.

I was able to export the data named Vector, but the sizes are different.
I would like to have the icon data sliced at 24dp×24dp if possible.

👀

If you use Android Vector Drawable, it may be faster to download it from the following. You need to make sure it's the same design, but it seems faster than calling a designer. 🙇
https://material.io/resources/icons/?style=baseline

The following icons have already been downloaded from Android Vector Drawable and they work fine.

android icon
contributor icon
staff icon
settings icon
home icon (already exist in project)

However, these icons were not available in the Android Vector Drawable

blog icon
video icon
podcast icon

Will icon datas be updated?

How about doing the following?
1 Implement that part empty and merge.
2 I contact the designer and apply the image as another issue.

OK!  I'll continue with the implementation.

@nontan Hello! I want to export the following 24dp x 24dp svg image. Do you know how to do it with Figma?
blog icon
video icon
podcast icon

It seems that 16x20 svg images can be downloaded now.


Customize artifact names

Consider allowing configuration around artifact file names. For example:

Name with placeholders for pid, process name, hostname, timestamp
Ambient information such as environment variables that indicate pods/container info

Related to https://github.com/dotnet/dotnet-monitor/pull/2290

Tentative design for this will be to create additional configuration with a default pattern that can be applied to each artifact type. This would include our typical substitution tokens, such as $(Process.Pid) ${Process.Name). We would likely expand this to include information such as $(Timestamp) and ideally $(Environment.EnvValue).
Note that this would apply to all artifacts of that type, not a specific egress configuration. The pattern would also not apply to direct Api based downloads.

I think this would be a great change! Especially so it could align with the createdump tool that's included in the runtime. We could use the same identifiers in the pattern: https://learn.microsoft.com/en-us/dotnet/core/diagnostics/collect-dumps-crash#file-path-templates

Specifier
Value

%%
A single % character

%p
PID of dumped process

%e
The process executable filename

%h
Host name return by gethostname()

%t
Time of dump, expressed as seconds since the Epoch, 1970-01-01 00:00:00 +0000 (UTC)


utoronto: Bump storage size

I just checked, and we were almost full

I've applied this manually.


ALIGN to work with open pdk

Greetings,
So this is rather a very general question. Does anyone here ever try to use ALIGN to work with some open pdks such as skywater130? The reason I am asking is I intend to actually do so and I assume the biggest hump here is to (probably very labor intensive but should be straightforward) have all skywater130 design rules written in the JASON file (similar to that mock 14nm FInFET). Is this a good start? Or any advice here? Or anyone here is already doing so and probably we can co-operate (just do not want to do something and then it turns our somebody already done it or finish most of it)? Thanks.

Its already done: You can follow the following repo: https://github.com/ALIGN-analoglayout/ALIGN-pdk-sky130

Affirmative. I will follow that repo and see how it goes. Thanks.
From: arvuce22 @.>
Date: Thursday, November 10, 2022 at 10:13 AM
To: ALIGN-analoglayout/ALIGN-public @.>
Cc: Chris Li @.>, Author @.>
Subject: Re: [ALIGN-analoglayout/ALIGN-public] ALIGN to work with open pdk (Issue #1204)
Its already done: You can follow the following repo: https://github.com/ALIGN-analoglayout/ALIGN-pdk-sky130
—
Reply to this email directly, view it on GitHubhttps://github.com/ALIGN-analoglayout/ALIGN-public/issues/1204#issuecomment-1310440502, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIKIPVYXGI4W67BIINN7SVDWHUGKDANCNFSM6AAAAAARYA2BTA.
You are receiving this because you authored the thread.Message ID: @.***>


Parameters<...> fails to resolve parameter type with dynamic type narrowed with .. exteds keyof ..

Bug Report

🔎 Search Terms

parameters type
parameters keyof
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about everything that mentions parameters or keyof.

⏯ Playground Link

Playground link with relevant code
💻 Code

type TypeWithFunctions = {
  fn: (arg: string) => void;
};

const higherOrderFn =
  <FnName extends keyof TypeWithFunctions>(
    fnName: FnName
  ): TypeWithFunctions[FnName] =>
  // Expected: should not produce an error but does
  (...args: Parameters<TypeWithFunctions[FnName]>) => {};

🙁 Actual behavior

Parameters<...> line errors with
Type '(...args: Parameters<TypeWithFunctions[FnName]>) => void' is not assignable to type 'TypeWithFunctions[FnName]'.
  Type '(...args: Parameters<TypeWithFunctions[FnName]>) => void' is not assignable to type '(arg: string) => void'.
    Types of parameters 'args' and 'arg' are incompatible.
      Type '[arg: string]' is not assignable to type 'Parameters<TypeWithFunctions[FnName]>'.(2322)

Parameters<...> fails to resolve the parameter type tuple.
🙂 Expected behavior

Parameters<...> should correctly and completely infer the relevant parameter type.

There are a few cases here to think about. TypeWithFunctions is obviously oversimple; if every property in it had the same signature, you could just write a less-generic version:
const higherOrderFn = <FnName extends keyof TypeWithFunctions>(fnName: FnName): TypeWithFunctions[keyof TypeWithFunctions] => (...args: Parameters<TypeWithFunctions[keyof TypeWithFunctions]>) => {

};

and indeed that works. You can also keep the return type generic without loss of functionality:
type TypeWithFunctions = {
  fn: (arg: string) => void;
  fn2: (arg: number) => void;
};

const higherOrderFn = <FnName extends keyof TypeWithFunctions>(fnName: FnName): TypeWithFunctions[FnName] => (...args: Parameters<TypeWithFunctions[keyof TypeWithFunctions]>) => {

};

higherOrderFn("fn")("ok");
higherOrderFn("fn2")(32);
higherOrderFn("fn2")("correct error here");

However (considering the non-degenerate case of when fn2 is in the mix), Parameters<TypeWithFunctions[FnName]> as an assignability source gets treated as its possible inhabitants, [arg: string] | [arg: number], and obviously that's not directly correlated to the lookup type.
It's also the case that we can't say for certain that this expression form necessarily assigns to an arbitrary type in the target. For example, TypeWithFunctions might well be
type TypeWithFunctions = {
  fn: (arg: string) => void;
  fn2: (arg: number) => void;
  fn3: {
    (arg: number): void;
    x: string;
  }
};

but if you're disavowing that possibility, then this example can be rewritten WLOG as
type TypeWithFunctions = {
  fn: [arg: string];
  fn2: [arg: number];
};

const higherOrderFn = <FnName extends keyof TypeWithFunctions>(fnName: FnName): (...args: TypeWithFunctions[FnName]) => void => (...args: TypeWithFunctions[FnName]) => {

};

So anyway there's not a steadfast argument for allowing this assignment, in a place that matters, unless TS was going to do wholesale enumeration of all the possible values of FnName, which doesn't really scale.
So on net the logic here to make this "work" doesn't scale up, and you don't really need it in the first place since the slightly-less-generic form works just as well (including in the function body).


Implements a solver of a linear matrix equation

This PR implements cupy.linalg.solve to cupy.

Thanks. I've just fixed.

@okuta Hi, is there any progress?

jenkins, test this please.

@okuta It seems that this PR already passed the tests.

Please add a test on whether solve can be called in the CUDA 8 and later environment.

How? I have no idea to deal with it.

Sorry, it is my mistake. We should add test for cusolver in another PR.

LGTM!


Unable to filter by dates in Dashboard.Throws ORA-01843: not a valid month

I have tried to create a dashboard which includes a date column.
When I try to open the dashboard and select a filter which is a date column , it gives me ORA-01843: not a valid month error.
But when I try to take the query object and run in Oracle DB directly , it works well.
But if I run the same query in superset , it throws me error.
When I do the below modifications to the query it works :
1.Either add TO_DATE to the values selected .Example
where sample_date in TO_DATE('10-10-2019','MM-DD-YYYY')
or
2.Add TO_CHAR to the column
where TO_CHAR(sample_date,'MM-DD-YYYY) in ('10-10-2019')
Why is it like that.I am unable to perform any date comparisons

Please use the standard issue template, it helps the maintainers understand context around your issue. Details such as version number are very helpful information as resolution of certain bugs can be attributed to specific version numbers.
In your dashboard, please click Explore chart on the chart that is emitting the error message, and once in the Explore view, click on the upper right-hand menu and select View query and paste the contents here. Furthermore, please check the datatype of your temporal column and share that here.

here are the screen shots for the issue

when im selecting any dates from filter box i am getting the error not a valid month

I have a hunch where the problem might be, but I seem to have misplaced my Oracle container so can't reproduce+fix+test right away. Will try to look into this in the coming days.


Authorization, additional users don't see host

We have a fresh Vagrant-based setup. A single host inside.
Administrator enabled access-control and authorized one of github-organizations for accessing. Administrator sees the host after he is logged in.
Everyone else in this organization can log-in, but can not see the host.
We didn't create any projects. Host is in "default" one

Direct link to host results in additional authorization request and (after second round of oauth) 404 page

We need to make this friendly.  What's happening is that "default" is not a single project...  The administrator and each individual user have their own independent "default" projects.  The host you've added belongs to the administrator's, so only they have access to it.  And there's currently no way to grant access to others to a "default" project.
If you want a host shared with several people (today), you need to first create a project owned by a team/org, then add a new host to that project.
Will update later with proposed changes..

FYI @feliksik We will track the "make github and projects" more user friendly here.

@feliksik and @indeyets - We are re-designing the authorization/project access experience in this issue: https://github.com/rancherio/rancher/issues/189 This doesn't change the fact that you will need to create a project before being able to share hosts, but will address the authorization/project access needs.

You won't have to explicitly create a new project to do that (once #189 is done).  Hosts and Containers will only be allowed to exist within a project, and you'll be able to add additional users/teams/orgs to any project. (currently the "default" project is special, it's not really a project and the resources are owned directly by the user account)

@feliksik @indeyets This should be in the 0.17 release tomorrow (and :beta/:latest shortly thereafter):
When you first install Rancher and use it with Access Control disabled:

A "default project" is automatically created
You can create and use multiple projects
All hosts/containers/etc go now belong to a project (they are not owned directly by a user anymore)

When you turn Access Controls on:

The user who turns it on becomes the Admin.
Existing projects created wile Access Control was off will belong to the Admin user.
Any project (including the default one) can have multiple member and owner GitHub user/team/orgs.
Members can see and use all the resources of a project.
Owners can change the project, add/remove members/owners, and delete the project.
The Admin user (who turned Access Controls on) can always see/manage/delete all projects.

v0.18.1
Verified fixed


Issue loading model containing Dense layer with Identity initializer

Hi, I've been migrating some model code from Keras 2 to Keras 3, and I think I stumbled upon a bug.
I've noticed that if I save a model containing a dense layer with an identity kernel initializer in Keras 3+, I cannot reload the model. I get an exception that says Keras cannot interpret the initializer identifier in config.json in the Keras model file.
A snippet from the exception (rest at the bottom of this post):
Exception encountered: Could not interpret initializer identifier: {'module': 'keras.initializers', 'class_name': 'IdentityInitializer', 'config': {}, 'registered_name': None}
I have tried this out on Windows using Keras 3.6 + Jax 0.4.35 and Ubuntu using Keras 3.6 + Tensorflow 2.18.0, and I get the same exception.
Currently, I am using a hacky workaround to replace the faulty 'IdentityInitializer' string in the config.json in the Keras model file with 'Identity'
Here are pytest tests for replicating this issue (see test_save_read_dense_layer_model_with_identity_initializer for replicating this exception, see attached
testoutput.txt
file for test output which includes the full exception traceback):
from pathlib import Path
from tempfile import TemporaryDirectory
from zipfile import ZipFile

import keras

def load_model_with_workaround(model_path: Path) -> keras.Model:
    with (
        TemporaryDirectory() as tmp_dir,
        ZipFile(model_path, "r") as original_model_file,
    ):
        new_model_path = Path(tmp_dir) / "new.keras"
        with ZipFile(new_model_path, "w") as new_model_file:
            for file_name in original_model_file.namelist():
                original_data = original_model_file.read(file_name)

                if file_name == "config.json":
                    original_data = (
                        original_data.decode("utf-8")
                        .replace(
                            'class_name": "IdentityInitializer"',
                            'class_name": "Identity"',
                        )
                        .encode("utf-8")
                    )

                with new_model_file.open(file_name, "w") as f:
                    f.write(original_data)

        return keras.models.load_model(new_model_path)

# This test will fail
def test_save_read_dense_layer_model_with_identity_initializer() -> None:
    model = keras.Sequential(
        [
            keras.layers.Input((5,)),
            keras.layers.Dense(5, kernel_initializer=keras.initializers.Identity()),
        ]
    )
    with TemporaryDirectory() as tmp_dir:
        save_path = Path(tmp_dir) / "mymodel.keras"
        model.save(save_path)
        model_from_file = keras.models.load_model(save_path)

# This test will pass
def test_save_read_dense_layer_model_with_identity_initializer_using_workaround() -> None:
    model = keras.Sequential(
        [
            keras.layers.Input((5,)),
            keras.layers.Dense(5, kernel_initializer=keras.initializers.Identity()),
        ]
    )
    with TemporaryDirectory() as tmp_dir:
        save_path = Path(tmp_dir) / "mymodel.keras"
        model.save(save_path)
        model_from_file = load_model_with_workaround(save_path)

I have submitted a PR for this issue here: #20484
You can also patch keras.src.initializers.ALL_OBJECTS_DICT in your code to address it.
import pathlib
import tempfile

import keras
from keras.src import initializers

model = keras.Sequential(
    [
        keras.layers.Input((5,)),
        keras.layers.Dense(
            5, kernel_initializer=keras.initializers.IdentityInitializer()
        ),
    ]
)
initializers.ALL_OBJECTS_DICT["IdentityInitializer"] = initializers.Identity  # <--

with tempfile.TemporaryDirectory() as tmp_dir:
    save_path = pathlib.Path(tmp_dir) / "mymodel.keras"
    model.save(save_path)
    model_from_file = keras.models.load_model(save_path)


JsonToProtoMessage refactoring

I read through JsonToProtoMessage and would like refactor it as my first contribution. I'd like to know your opinion about the proposed changes and ask for some explanations.

All the fields should be final and CAPITALIZED (they are all deeply constant)
We should migrate from import org.threeten.bp to java.time. This would require migration of com.google.cloud.bigquery.storage.v1.CivilTimeEncoder. As we support Java 8, we should not need it anymore.
Code like /*ignoreUnknownFields*/ false should be replaced with DONT_IGNORE_UNKNOWN_FIELDS field
The topLevel parameter should be removed
fillField and filRepeatedFIeld switch options should be extracted into methods. And even divided more to help JIT optimise the code.
The failing mechanism in fillRepeatedField should be replaced with throwing exception immediately. Otherwise one may think something can happen along the way to fix the situation. The call should be extracted into method
Two variants of convertJsonToProtoMessage method should be removed as they are used only in tests. The conversion should not support null in TableFieldSchema and FieldDescriptor. I think there is no case when it happens except tests. Is there? If yes, then it should be marked @Nullable
Comparing "true" with toLowerCase() should be replaced by equalsIgnoreCase()
The test should not use asterisks for imports
We should prefer for(X x: xs) over for (int i = 0; i < xs.length; i++) {X x = xs[i];}
The JsonDataHasUnknownFieldException should not be final and have protected constructor at the same time. It should be renamed to DataHasUnknownFieldException to be used by developers for other transformers.
Should we support conversion from JSON strings to booleans, ints, etc.?

Please tell me what you think.

@yirutang Maybe you are able to answer to my questions?


Babel 7 Support

Type:

[x] bug

Environment:

OS: Mac
Library Version: 0.8.3

I'm going to open a PR:

[x] maybe

Description:
Just a report that the extension reloader appears to have issues with babel 7, I've used it successfully before on a previous version but after upgrading it seems very inconsistent. Noticed this while running the example provided in the sample folder.

Hi @zsherman , thanks for the info.
Sorry about the late response (changing from company, not enough time to work on my projects)
Whenever I get some time I'll work on that :)


Cannot create Intel machine without Rosetta (ip missing)

Describe the bug
When creating a Linux machine without Rosetta enabled in settings, an error occurs.
The following error message is shown 30 seconds after creating the machine:

Can't create machine
...machine didn't start in 30s (missing ip address)

To Reproduce

Turn of Rosetta in Settings > System > Use rosetta to run intel code
Create intel machine
Observe error message after 30s

Expected behavior
Expected to be able to create machine without Rosetta.
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.9.2
Commit: f56c5adaa796a0902c648f038307ed8d434b0522 (v1.9.2)
System info:
macOS: 15.2 (24C101)
CPU: arm64, 14 cores
CPU model: Apple M4 Pro
Model: Mac16,8
Memory: 24 GiB
Screenshots and additional context (optional)
No response

What distro and version? Please include the full diagnostic report so that there's enough info.

Also, why are you disabling Rosetta? Keeping it on is highly encouraged and the option to disable it may be removed in the future.


Format numeric constants

Summary
Format int, float and complex constants.
Test Plan
Existing snapshots.

I'll implement optimization like normalize_string. Thanks!


Revert PIXI.DisplayObjectContainer.getBounds call to updateTransform

This PR changes

Nothing, it's a bug fix

Describe the changes below:
This reverts commit 410237881eb8c7bdc21753dd56e1b97885306737.
Calling updateTransform inside of DisplayObjectContainer.getBounds() has the following two
detrimental side effects:

getLocalBounds() actually returns world bounds.
PIXI.identityMatrix becomes corrupted because worldTransform is updated destructively in updateTransform().

There is probably a way to rewrite getLocalBounds() to work without reverting this, but I don't see an obvious route.

Closing, as this is covered by #2639


Add support for filtering

This commit adds support for filtering via the "--min-age", "--max-age",
"--min-size" and "--max-size" flags. These values can only be set via
flags and they are applied for all remotes. The default value for all
flags is "off" and they are using the same format as the corresponding
rclone flags.
NOTE: Maybe we can add a modal to set these flags during the runtime of
the rcloneui.
Closes #4.

Thanks for adding this - a modal would be fantastic!


No Error With Expected Return Not Present

Description
If adding a return type, you can omit the return and it will compile without error. This causes a warning in the C++ compiler, which then crashes when running the executable.
Expected Behaviour
This can go two ways:

A default value returned when omitted
Error thrown at compile-time, that expects a return to be present when a type is specified.

Examples
fn get_value() int {}

fn main() int {
    bad_value :: get_value();
}

Another version, that conditionally returns also results in the same issue:
fn get_value(maybe bool) int {
    if maybe {
        return 1;
    }
}

fn main() int {
    bad_value :: get_value(false);
}

Fixed bug in: Commit 1 and Commit 2.


MudDataGrid NullReferenceException when Property's value is null

Bug type
Component
Component name
MudDataGrid
What happened?
When you define a PropertyColumn with a nested Property that is null - the DataGrid will throw a NullReferenceException.
If you do not set the Property you´ll lose sorting and filtering capabilities.
Is there a workaround?
Expected behavior
The cell should be empty if no values does not exist.
Reproduction link
https://try.mudblazor.com/snippet/wOQxEfEHnUdddfrT
Reproduction steps
Run the snippet.
If you want a working example, assign the teacher in the class object initializer.
Relevant log output
No response
Version (bug)
6.2.2
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Mac OSX
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This is not a bug, C# not support expression with the ?.
I think PropertyColumn<T, TProperty> of MudDataGrid can use public Func<T, TProperty> Property { get; set; } instead of public Expression<Func<T, TProperty>> Property { get; set; }

Any workaroIund for this problem?

Would it be sufficient to just initialize the nested object? I have had success with this approach.
Using the example provided by @mkalinski93, I modified the Class.Teacher to always be initialized. Removing the setter is optional, but it ensures that the value can never be set to null.
  public record Class {
      public string Name { get; set; }
      public Teacher Teacher { get; } = new();
  }

  public record Teacher {
      public string Name { get; set; }
  }

In my case this will not work. The object will be serialized and send to another service, so its important that null values will survive.
My workaround so far: adding a method returning a default value instead of null only for the grid.
public DateTime? NextMaintenance { get; set; }  public DateTime GetNextMaintenance() => NextMaintenance ?? DateTime.MinValue;
 <PropertyColumn Property="x => x.GetNextMaintenance()" Title="Next maintenance "/>
Not great, but works for the moment.

My workaround is this:
<PropertyColumn Property="b => b.Category == null ? string.Empty : b.Category.Name" Title="Category" />

Here, I am listing Brands in a MudDataGrid. A Brand has a nullable Category property, for which I want to display the Category's Name property for the Brand. Here, I simply check if the Category property is null for the Brand, and, if so, I output an empty string. Otherwise, the Category exists, in which case I output the Category's Name property.

My workaround is this:
<PropertyColumn Property="b => b.Category == null ? string.Empty : b.Category.Name" Title="Category" />

Here, I am listing Brands in a MudDataGrid. A Brand has a nullable Category property, for which I want to display the Category's Name property for the Brand. Here, I simply check if the Category property is null for the Brand, and, if so, I output an empty string. Otherwise, the Category exists, in which case I output the Category's Name property.

This is also a great solution for an editable MudDataGrid bound to non-nullable numeric values. I have a column of doubles that will throw a null reference exception if the user erases the default value and then leaves the cell totally blank. Your approach works great here:
<PropertyColumn Property="x => x.MyDouble == null ? 0 : x.MyDouble " Title="My Double Property" />

I had this same issue with a DateTime property, I had to set to the default value using new DateTime() so it was not null, again not a great workaround but at least it doesn't throw an exception.

I had this same issue.
https://try.mudblazor.com/snippet/GamyktmNyCKVznLa

Fixed in https://github.com/MudBlazor/MudBlazor/pull/9916, the next release 7.9.0 should contain the fix.


Website down

Has this project been abandoned? The website is currently down.
https://vulkango.com/

Was not ready. Never finished that site :)


Prevent retries for genuinely failed uploads

What is the user problem or growth opportunity you want to see solved?
No response
How do you know that this problem exists today? Why is this important?
The app retries all failed uploads, even if they fail for a genuine reason like invalid filename. This impacts the UX badly [details].
Who will benefit from it?
No response
Anything else you would like to add?
No response

Hi, me and my group are taking a crack at this as part of a uni course for the next week. Could we get assigned this?

@Glassar It is yours! :-) If too hard, easier issues like https://github.com/commons-app/apps-android-commons/issues/5565 or https://github.com/commons-app/apps-android-commons/issues/4895 are available.
(note for GSoC: If this bug gets solved, we will replace the GSoC's tasks with another of similar size)

Thank you, and great to have those suggestions. We have done a first code inspection and think this should be doable though so we'll start here

We have done quite a bit of digging and found where an upload attempt fails (in uploadWorker, or this is at least where the failure is determined).
So our plan right now is to add code to the "failure" points in UploadWorker.uploadContribution which passes the failure reason onwards. This can then be used in ContributionsFragment.retryUpload to determine if this is a genuine failure and the attempt abandoned, or if it should try again. Finally this failure reason could also be used give the user better information about why their upload has failed repeatedly link.
But to determine what is a genuine failure, we need to know/decide what is a genuine failure which can the be used as a whitelist to stop retrying (one obvious example which won't change is if we get a FileNotFoundException). And this might be identified through for example exception messages, but we are having some trouble with determining what those exact conditions might be. Any ideas of the kind of errors might belong here, or where we should look?
Another thing we are wondering about is passing the information. The easiest and most intuitive way would be to add another field to the Contribution class for an error message as this how if it failed is already passed around. But the contributions are stored on the DB and as such we  are not sure it storing the failure information there is optimal, any tips here?
Finally it seems reasonable to notify the user about what is going wrong with their upload, both when it fails to many retries and when we determine that it is a genuine failure. In what way should these notifications be sent (as far as I can see there is already sent something to the user in UploadWorker when it fails)?

I shared one reason for the case Nicolas mentioned above in this issue: #5280, might be helpful.

Any ideas of the kind of errors might belong here, or where we should look?

Invalid filenames also cause this issue. I tried adding a § sign in the caption, it keeps failing.

Invalid filenames also cause this issue. I tried adding a § sign in the caption, it keeps failing.

Hey, I was just trying to reproduce this issue, but surprisingly for me, the image successfully uploaded with a § in the caption:

Unable to continue due to time constraints and implemented testcases being unable to run due to the mockup nature of the testcases.

Unable to continue due to time constraints and implemented testcases being unable to run due to the mockup nature of the testcases. A better whitelist is needed with better mockup data as they never break. The tests also don't notify the user in the case of a failure. We were also unable to make our own test execute, particulary due to the difficult class structure.

The caption has to be only § for the upload to fail. A caption with § at the end does not fail.


[cuda:x64-windows] build failure

I installed CUDA in C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.2. Then, it seems that vcpkg failed to find where cuda is and the installation failed. How can I solve this?
Host Environment

OS: Windows 10
Compiler: Visual Studio 2019
CUDA: 10.2

To Reproduce
Steps to reproduce the behavior:

nvcc --version

nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2019 NVIDIA Corporation
Built on Wed_Oct_23_19:32:27_Pacific_Daylight_Time_2019
Cuda compilation tools, release 10.2, V10.2.89

.\vcpkg.exe install cuda:x64-windows

Computing installation plan...
The following packages will be built and installed:
cuda[core]:x64-windows
Detecting compiler hash for triplet x64-windows...
Starting package 1/1: cuda:x64-windows
Building package cuda[core]:x64-windows...
Could not locate cached archive: C:\Users\admin\AppData\Local\vcpkg/archives\dc\dc44fdcc6d6e0a90de6ff1e1cbd0b03b16fbade6.zip
-- Executing NVCC-NOTFOUND --version resulted in error: 1
CMake Error at ports/cuda/portfile.cmake:66 (message):
Could not find CUDA.  Before continuing, please download and install CUDA
(v10.1.0 or higher) from:
  https://developer.nvidia.com/cuda-downloads

Call Stack (most recent call first):
scripts/ports.cmake:79 (include)
Error: Building package cuda:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: cuda:x64-windows
Vcpkg version: 2020.06.15-nohash
Additionally, attach any relevant sections from the log files above.

I find that nvcc is installed in another location by the command
`Get-Command nvcc
D:\Program Files (x86)\cuda\bin\nvcc.exe`
So is there any solution to this problem?

Please set this path to environment var CUDA_PATH and try again.

Thank you, this fixes the bug.


[FLINK-29191][hive] fix Hive dialect can't get value for the variables via SET command

What is the purpose of the change
To fix Hive dialect can't get value for vaiables via SET command
Brief change log
Also try to look up the value in flink's table config.
Also in this pr, it fix two thing:
1: for the SET command, the previous behavior will print all options including Hiveconf, hivevar,.. It contains too many things, and will fill the console out. So, the SET command in Hive dialect will delegate to Flink's own implementation which will only print the flink's table config. If users want the all options, they can use SET -v.
2: when show options, for flink's table configuration, not to add prefix flink:.  for Hive's conf, to add the prefix Hiveconf:
Verifying this change
Added test in flink-table/flink-sql-client/src/test/resources/sql/set.q
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure


Scope information for additional properties

Hi Mark, thanks for creating this library. Recently I've made a little utility called jschon-sort which leverages jschon's very useful feature of having the evaluation's result map evaluated values back to the schema that validated them.
One edge case I've noticed is that additionalProperties, when present as "additionalProperties": true (rather than being true by default), don't map to the properties that they validated. (TBH I haven't tested whether the same holds for pattern properties, etc.)

Hi Ilya, glad to hear you are finding jschon useful. Could you perhaps provide an example to clarify the issue you are describing, and what your expectation would be?
The Scope.schema attribute is there primarily to be able to generate absolute URIs for annotation and error output - though it's also used in $dynamicRef resolution. So yours is an interesting use case because it's an 'unexpected' way of using Scope.schema! It has been in the back of my mind to look more carefully at how boolean schemas are processed - there's an issue with poor error messaging for e.g. "additionalProperties": false - and I have a very speculative hunch that these issues might be related.

My tool reorders properties to be in the same order as they're defined in the schema. It relies on mapping from the document to the schema that's in the evaluation results. (Welcome to see how it's implemented, there's not much to it.)
For additional properties, I would expect a mapping from each such otherwise unknown property to the additionalProperties that allows it, though I'm not sure if that's all that obvious for it to happen.

Okay, I see that you're using the instpath and path attributes of scope nodes to create a mapping from document nodes to schema 'sort keys', and I see how this enables you to sort document nodes that have been validated by properties. (This is very cool, btw!)
So I've worked through your test case, adding an additional "foo" property to your test instance, and setting the inner additionalProperties value to True in the test schema. I understand that in your _traverse_scope function, you might expect to visit a scope child node with instpath pointing to /ranges/0/foo, with a corresponding path of /properties/ranges/items/additionalProperties.
The difficulty here is that document evaluation is done from the perspective of the schema, and the resultant structure of the scope tree reflects the evaluation path taken through the schema and any referenced schemas. (The term 'scope' comes from 'dynamic scope' described here). So in the scope tree, under a properties node, you'll have child nodes for each of the properties that appear in the schema (each of those child nodes being associated with a property subschema), but you won't have any such child nodes under an additionalProperties node, because the value for additionalProperties is just a single subschema, against which any additional properties are validated. The names of any properties validated by the additionalProperties keyword are stored in the annotation of the corresponding scope node.
Having said all that, unless I'm missing something, I don't think the existence or otherwise of child property nodes for additionalProperties should impact your sort algorithm, as the schema wouldn't know about those properties anyway?

The term 'scope' comes from 'dynamic scope' described...

Thanks for linking. TIL.

the existence or otherwise of child property nodes for additionalProperties should impact your sort algorithm, as the schema wouldn't know about those properties anyway

The idea was a somewhat obscure ability for a schema to hint whether additional properties should be ordered before or after all other properties. Right now it's hardcoded to 'after'. Considering patternProperties do work, I'll close this one.


Added an about page

Added message for "about"
changed function call in bakwan.singkong

Thanks for the pull request, @CelosiaX. I will merge the request, but will make some edits later.


[Collector Interaction] Interaction has already been acknowledged.

So I've tried to use the collector interaction like:
const msg = await interaction.followUp({ components: [row5], content: `Weird button just showed up!`, ephemeral: false })
const collector = msg.createMessageComponentCollector({ time: 90000 })

collector.on('collect', async (i) => {
    if(i.customId === 'trywordbtn') {
        showModal(modal, {
            client: client,
            interaction: i
        })
     }
})

The first time the command works fine, but the second time I use the command I get Discord API error with "interaction has been already acknowledged. Here's the modalSubmit event:
 client.on('modalSubmit', async (modal) => {
            if(modal.customId === 'wordleinput') {
                await modal.deferReply({ ephemeral: true }) // Error from here belongs...
                const firstResponse = modal.getTextInputValue('wordinput')

                if(firstResponse === randomWordg) {
                    await modal.followUp({ content: `Congrats 🥳 You guessed the word right, and it was \`${randomWordg}\``, ephemeral: true })

                    await interaction.editReply({ content: `Good job! \`⍟Soon\` tokens has been added to your wallet.`, components: [deadraw] })

                } else if(trys === 4) {
                    await modal.followUp({ content: `You've 4 attempts to guess the word.`, ephemeral: true })

                    await interaction.editReply({ content: `Hmm... You should think more about it!`, components: [row5] })
                    trys = trys - 1

                } else if(trys === 3) {
                    await modal.followUp({ content: `\`3 of 4\` Attempts left.`, ephemeral: true })

                    await interaction.editReply({ content: `2 more attempts left, it's really easy!`, components: [row5] })

                } else if(trys === 2) {
                    await modal.followUp({ content: `\`2 of 4\` Attempts left.`, ephemeral: true })

                    await interaction.editReply({ content: `Really it's that much hard?!`, components: [row5] })
                } else if(trys === 1) {
                    await modal.followUp({ content: `\`1 of 4\` Attempts left.`, ephemeral: true })

                    await interaction.editReply({ content: `Common, think think!`, components: [row5] })
                }

                if(trys === 0) {
                    await modal.followUp({ content: `Game end! You lost...`, ephemeral: true })

                    await interaction.editReply({ content: `You've ran out of attempts.. the word was \`${randomWordg}\``, components: [deadraw] })
                }
            }
});

I am suffering this issue ages ago, idk what to do to solve it.. any solution??

Do you have the client.on('modalSubmit) method in the same file as the command? If so, what worked for me was moving that method to another file like main.js or with an event handler.

Do you have the client.on('modalSubmit) method in the same file as the command? If so, what worked for me was moving that method to another file like main.js or with an event handler.

Well, I've tried to do that so I wanted to use the interaction from the command, so I've tried to use parameters, but the same issue still appears, here's example, in the component collector:
const modalHandler = require('../../events/modalSubmit')

collector.on('collect', async (i) => {
    if(i.customId === 'button') {
        showModal({
             client: client,
             interaction: i
         })
        modalHandler(interaction, collector)
    }
})

And the event file
const modalHandler = async (interaction, collector) => {
    client.on('modalSubmit', async (modal) => {
          // Th same event code goes here
    }
}

module.exports = modalHandler

Do you have on the interactionCreate event a deferReply method?

Do you have on the interactionCreate event a deferReply method?

Well, I use command handler, I've used the interaction.deferReply() first and then I've used followUp to reply with the rows..
Like:
// On the top of the code
await interaction.deferReply()

// Some rows defining goes here...

// And now followUp with the components
const msg = await interaction.followUp({ ... })

Delete the deferReply method and if u have followUp or editReply methods on your commands change It by reply or limit this deferReply method to all your command except this one

But it's not caused from the command interaction.. I've marked from where does the error belongs to, when I use modal.deferReply() for the second time, it errors

The problem is the thing that i said to you

Because the collector collect those interactions and you on your interactionCreate you deferReply that interaction and the error is because you cannot deferReply this interaction and use on the show Modal method

Try It and you Will se

You can do this instead of replace the followUp and editReply methods

if(!<cmd> === "your_modal_command"){
interaction.deferReply()
}

Alrighty, should I disable the interaction.deferReply() from the command? And use interaction.reply() instead?

If you do not understand something let me now

Perhaps meeting me on Discord? If so click this icon  to get directed to my profile!

Yeah true, anyways, I've tried to not to use Interaction#deferReply and use Interaction#reply like:
const msg = await interaction.reply({ components: [row5], content: `Weird buttons just showed up!`, ephemeral: false, fetchReply: true })
const collector = msg.createMessageComponentCollector({ time: 120000 })

But like uh, am bored like literally I don't know what to do, would be my pleasure if you explain me it step by step...

ps: I use Interaction#deferReply in all of my commands not in the event itself

Whats the name of your command?

wordle

ok

if(!<cmd> === "wordle"){
interaction.deferReply()
}```

Like that and only you try the command and luckily it will works propertly

in cmd put how you define your command

okay I'll try

so it has different if deferReply in the event or in the command file?

Well nope, still facing the issue...

@PabloRNC I'm having this same error, with nothing but the modalSubmit event giving a deferReply
you can contact me at 10KAIDAN#0001 on discord

Show me your command and your interactionCreate

@PabloRNC is it possible to edit the current message without following it up because the bot im making is using it like a ban reason and then edit the message saying banned successfully?

Mmm i do not know what do you mean if can provide some screenshots

@PabloRNC is it possible to edit the current message without following it up because the bot im making is using it like a ban reason and then edit the message saying banned successfully?

You can like modal.reply with fetchReply in true and then edit It like a interaction.reply method

@PabloRNC is it possible to edit the current message without following it up because the bot im making is using it like a ban reason and then edit the message saying banned successfully?

You can like modal.reply with fetchReply in true and then edit It like a interaction.reply method

i already changed to discord.js v14 and it had the necessary things with the function awaitModalSubmit made it that i can edit current message without making it reply again
https://user-images.githubusercontent.com/63213308/166118685-80769e88-066d-41c9-9e70-9496f1682c16.mp4

You can edit It with modal.interaction.message.edit on the discord -modals npm i think

it tried I just got undefined

On discord-modals is modal.update(...).


Add configuration file for Graham

Wait for #166 before merging

Coverage increased (+1.2%) to 95.725% when pulling 090309b769f422dca5e40430343b4985b9c4d960 on bouthilx:graham_config into 9133e15ae91e1e5b0a2b7e90f442c1ebfb623e86 on SMART-Lab:master.

@nouiz It's a branch forked from adrien_slurm, the one PR #166 is based upon. Yes it should. The question is more: are those configurations the one we should define or not. We don't have strict queues like torque clusters, so it's up to us to decide how we define/devide queues.

Coverage increased (+1.2%) to 95.738% when pulling cac2f0891ab904a7f979ac6306100ac5a9e4f3f6 on bouthilx:graham_config into 9133e15ae91e1e5b0a2b7e90f442c1ebfb623e86 on SMART-Lab:master.


Does this support pytorch-based agents?

Hi, guys.
Looks interesting.
Can I write policies in Pytorch? Or is this tf only?
Thanks.

Hi drozzy -- thanks for your interest. Yes! We've tried as much as possible to decouple things from DL frameworks. See e.g. the JAX agents for ideas about how you can do things in PyTorch. Let me know how you get on!

Thanks @aslanides
Just curious if you looked at erlang for your distribution strategy?  A lot of patterns you talk about (e.g. client for polling from learner) come for free out of the box there.
(Or Elixir which is the same bytecode compatible language as erlang)
Good luck.

Internally at DeepMind we use our own distributed computing stack built on top of gRPC and integrated with Borg and other DeepMind/Google infrastructure. We don't plan to move off this any time soon ;). But thanks for the recommendation, I've heard good things about Elixir.

I was thinking maybe for public version of acme. :) If and when you decide to add distributed component.

Yes, understood. What I meant to say is that if/when we do open-source distributed components they'll most likely be simplified versions of what we use internally. You've seen first-hand from the issues with the tutorial colabs that things that we don't use internally are not quite as well maintained as the rest of the codebase ;). So I think our building on a separate  stack just for open-source would probably be undesirable for similar reasons.

That said, Erlang is certainly an interesting idea for distribution. While it is not what we use internally I think it could be compatible with Acme and would be curious to see how it works out.

If I was changing some code which used ACME for jax, how would I convert it to something for pytorch?


Add Web Application Scanner Script

Aim
The aim of a Web Application Scanner script is to automate the process of scanning and testing web applications for common vulnerabilities and weaknesses. It helps in identifying security flaws in web applications, allowing developers and security professionals to address these issues and improve the overall security posture. Here are the key aims of a Web
The script aims to identify common vulnerabilities in web applications, such as Cross-Site Scripting (XSS), SQL Injection, Command Injection, Cross-Site Request Forgery (CSRF), Server-Side Request Forgery (SSRF), Path Traversal, and more. It scans the web application for known patterns or indicators of these vulnerabilities.
The script aims to detect misconfigurations in the web application and underlying server components. It looks for insecure configurations, default credentials, open directories, exposed sensitive information, or any other misconfiguration that could be exploited by an attacker.
The script tests the authentication and authorization mechanisms of the web application to identify weaknesses. It may attempt common password guessing, brute-force attacks, session management issues, or bypass techniques to evaluate the effectiveness of these security controls.
Details
URL Crawling: The script should be able to crawl and discover all accessible pages and endpoints of the web application. This involves parsing HTML responses, following links, and building a comprehensive map of the application.
Do I want to work on this:

[x]  Yes
[ ] No

@avinashkranjan please assign this issue to me
thanks

Go Ahead @Abhinavcode13


BUG : New video Fixed (#151)

Title and Issue number
Title :  Bug : New Video #151
Issue No. : 151
Code Stack : HTML ,CSS
Close #151
Video/Screenshots (mandatory)
https://github.com/apu52/Travel_Website/assets/146833908/7447ca73-6f3a-4dbc-aea6-09e5be3b63e9
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )

Change the name of new video for proper working .
Updated the story.html.
Updated CSS in styles.css to fix the video button on index.html and also correctly display the video.

Are you contributing under any Open-source programme?
Contributing under JWOC

Name : Vishal Malyan

@vishal2005025 Attach a more descriptive video . the video is more basic . make any type of video that describes the features of the TOURGUIDE website otherwise the changes are fine


Add  fields in BO datamodel

Is your feature request the result of a bug?
NO
Problem
We would need a "note" field in BO, and a dotted progress bar or step dot progress bar which could be a datamodel itself
Explanation :
When buyers exchange information with prodution teams regarding a SO ans its BO, they use today an excel file which one column is "notes" for example "Will lack 200 x 2N2222 awaited for March 11th" => with that king of informations production teams can state wether the start one batch now and 2nd batch when received missing 2N2222 or the whole BO in one batch on complete 200 last PCB later.
dotted progress bar ( I assume this is more hard to implement )
We use 5 or 6 key indicators to produce a PCB :
PCB are availables in inventory / Stencil does already exist or must be prepared ; if Stencils exists, include its Id
SMD components are availables in inventory ( means BO can start for SMD assembly step )
Trough-hole components are availables in inventory ( means BO can start for trough-hole assembly step )
Production datas are available ( P&P datas ) and ready to use or need to be prepared
Suggested solution
At first maybe 5 or 6 more fields and when possible these files will be concatened into a dotted progress bar ?
Describe alternatives you've considered
Just 5 or 6 more fields which name is "user-defined" with default set to "step1, step2, etc ..."
Examples of other systems
our excel file :

Do you want to develop this?
As previous, why not with help

@tse-tarn-france I'm going to close out this issue for now, due to lack of response. If you have anything further to add to this, please get back to us!


Support for .NET core 2.1 preview-2

Are you planning to support .NET core 2.1 preview-2 or we have to wait for the RTM?

@raunakpandya @vturecek Do either of you know the answer to this?


feat: generate requirement test for other mappings

Added support of generating requirement tests for other_mappings
Added/Updated e2e tests for requirement tests generation
Added/Updated unit tests

:tada: This PR is included in version 5.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


New scroll widget monitoring approach

Fixes #10

Codecov Report

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

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage          ?   15.78%
=========================================
  Files             ?       20
  Lines             ?      323
  Branches          ?        0
=========================================
  Hits              ?       51
  Misses            ?      272
  Partials          ?        0

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 4d406ea...a764888. Read the comment docs.


Support for multi-response modelling

Greetings,
I have been trying to fit a multi response model using R-INLA package, with the following data:
dd = data.frame(
  x1 = runif(100),
  x2 = runif(100)
) |>
  mutate(
    y1 = 2 - x1 + rnorm(100),
    y2 = 2 - .4*x1 + 1*x2 + rnorm(100)
  )

formula <- cbind(y1, y2) ~ x1 + x2
result <- inla(formula, family="gaussian", data=dd)

Is this currently supported by INLA, as I seem to have gotten the formula wrongly.
Best.

This is not how it can be done.
can you post to @.*** instead ?
On Wed, 2024-03-13 at 10:06 -0700, Stanley sayianka wrote:

Greetings,
I have been trying to fit a multi response model using R-INLA package,
with the following data:
dd = data.frame(
  x1 = runif(100),
  x2 = runif(100)
) |>
  mutate(
    y1 = 2 - x1 + rnorm(100),
    y2 = 2 - .4x1 + 1x2 + rnorm(100)
  )
formula <- cbind(y1, y2) ~ x1 + x2
result <- inla(formula, family="gaussian", data=dd)
Is this currently supported by INLA, as I seem to have gotten the
formula wrongly.
Best.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this
thread.Message ID: @.***>

--
Håvard Rue
Professor of Statistics
Chair of the Statistics Program
CEMSE Division
King Abdullah University of Science and Technology
Thuwal 23955-6900
Kingdom of Saudi Arabia
@.***
Office: +966 (0)12 808 0640
Mobile: +966 (0)54 470 0421
Research group: bayescomp.kaust.edu.sa
R-INLA project: www.r-inla.org
Zoom: kaust.zoom.us/my/haavard.rue
--
This message and its contents, including attachments are intended solely
for the original recipient. If you are not the intended recipient or have
received this message in error, please notify me immediately and delete
this message from your computer system. Any unauthorized use or
distribution is prohibited. Please consider the environment before printing
this email.

n <- 100
dd = data.frame(
x1 = runif(n),
x2 = runif(n)
) |>
mutate(
y1 = 2 - x1 + rnorm(n),
y2 = 2 - .4x1 + 1x2 + rnorm(n)
)
N <- 2*n
Y <- matrix(NA, N, 2)
Y[1:n, 1] <- dd$y1
Y[n + 1:n, 2] <- dd$y2
na <- rep(NA, n)
x1.1 <- c(dd$x1, na)
x2.1 <- c(dd$x2, na)
x1.2 <- c(na, dd$x1)
x2.2 <- c(na, dd$x2)
this will use the same intercept and two precisions for the
obsersations, one for each.
formula <- Y ~ x1.1 + x2.1 + x1.2 + x2.2
result <- inla(formula, family=rep("gaussian",2),
data=list(Y = Y,
x1.1 = x1.1,
x1.2 = x1.2,
x2.1 = x2.1,
x2.2 = x2.2))
On Wed, 2024-03-13 at 20:28 +0300, Haavard Rue wrote:

This is not how it can be done.
can you post to @.*** instead ?
On Wed, 2024-03-13 at 10:06 -0700, Stanley sayianka wrote:

Greetings,
I have been trying to fit a multi response model using R-INLA
package,
with the following data:
dd = data.frame(
  x1 = runif(100),
  x2 = runif(100)
) |>
  mutate(
    y1 = 2 - x1 + rnorm(100),
    y2 = 2 - .4x1 + 1x2 + rnorm(100)
  )
formula <- cbind(y1, y2) ~ x1 + x2
result <- inla(formula, family="gaussian", data=dd)
Is this currently supported by INLA, as I seem to have gotten the
formula wrongly.
Best.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this
thread.Message ID: @.***>

--
Haavard Rue
@.***

This is not how it can be done. can you post to @.*** instead ?
…

Sorry, I have not understood what you meant here.

n <- 100 dd = data.frame( x1 = runif(n), x2 = runif(n) ) |> mutate( y1 = 2 - x1 + rnorm(n), y2 = 2 - .4x1 + 1x2 + rnorm(n) ) N <- 2*n Y <- matrix(NA, N, 2) Y[1:n, 1] <- dd$y1 Y[n + 1:n, 2] <- dd$y2 na <- rep(NA, n) x1.1 <- c(dd$x1, na) x2.1 <- c(dd$x2, na) x1.2 <- c(na, dd$x1) x2.2 <- c(na, dd$x2) ## this will use the same intercept and two precisions for the ## obsersations, one for each. formula <- Y ~ x1.1 + x2.1 + x1.2 + x2.2 result <- inla(formula, family=rep("gaussian",2), data=list(Y = Y, x1.1 = x1.1, x1.2 = x1.2, x2.1 = x2.1, x2.2 = x2.2))
…

Thank you for the quick response.
This works.
Best,


Align name mangling in clang 13 host pass with upstream clang and restrict uses of createDeviceMangleContext()

In the hipSYCL 0.9.2+ name mangling logic for clang 13+, it turns out that we have used createDeviceMangleContext() too liberally in the host pass. It should only be used when the ABIs are incompatible, and the regular mangler otherwise.
The previous code could result in mangling mismatches between host and device in integrated multipass scenarios, which can lead to kernel launch failures.
This PR resolves those issues by replicating the logic that clang itself uses (e.g. in CGCUDANV.cpp) more closely.
Hopefully fixes #718

@hipSYCL-ci-bot test


Adding Memory Region Analysis results to IR

We want to add the results from the Memory Region Analysis to the intermediate representation. This means replacing all references to 'mem' in the IR with new variables representing discrete memory regions, derived from the memory region analysis. Each memory load and store should only have a single memory region associated with it - if the analysis says there are multiple regions, those regions need to be universally merged (in terms of the IR, anyway) to only have a single memory variable represent them. However, we want to keep stack regions distinct from other regions, so if there is a case where a memory access could potentially be either a stack access or a non-stack access, we should not merge the regions and instead create new blocks to represent both possibilities. I'm realising now we probably need to add non-deterministic branching to the IR for this case?
We want to keep track of which memory regions are stacks, so we probably need a flag for that in the Memory class.

Add a switch to choose between #71's implementation and this

Ideally the MRA will be able to replace that entirely, but an option does make sense for the time being

Hey I started looking at this and I was able to replace memLoads and memStores with stack identifier using MRA.
I was abe to replace the memLoads with values if only a single one is given from VSA.
However, I have also used OR to combine values in a case where a memLoad could lead to multiple values (is this correct?). I am not sure what you mean by "universally merged to only have a single memory variable represent them" as I am not sure how you would merge the values without using BVOR for all of them.
Here is the PR for this: #74

What I mean is merging regions - if an access is ambiguous and could possibly be multiple regions according to the MRA & VSA, it's necessary to merge those regions into a single region for the purposes of the IR.

To be clearer, the idea is to replace the mem variable for all accesses (MemoryLoads and MemoryStores) with a new Memory variable that represents the memory region. If an access could point to multiple regions, then you need to merge all those possible regions together for the purposes of the IR and represent them with a single Memory variable wherever any of those regions would be mentioned.
For example, if there is a MemoryStore that could be to the regions region1 or region2, then you need to merge region1 and region2 and represent them both with a new Memory variable region1_region2 (the names I've chosen are arbitrary) that is used in that MemoryStore. If there is a separate MemoryLoad that is only to region1, then you need to use the Memory region1_region2 for it as well.
As discussed yesterday, we don't want to replace MemoryLoads with static values from the VSA at the moment.
I hope that all makes sense.


ME conduit does not correctly connect to AE2 terminals

From talking with the AE2 people, they said EnderIO needs to support adding AE 'parts' in order to work with the new terminals.

yeah it has no support for it at the moment and they are supper buggy also you can connect a normal me cable with terminals then hook it up to the conduit

Having the same issue.
Atm I prefer the AE2 cables because of that issue.

It's not easy (and perhaps not possible) to support AE parts on ME conduits. I may pursue it in the future but for now it will remain as it is.


Update README.md

FAQ and credits updated

The logos are of very low quality. Can you take the .svg files? You can re-use these: https://github.com/LCSB-BioCore/COBREXA.jl/tree/master/docs/src/assets

Done, but I could not find an svg for research luxembourg logo. I think the current one is of enough good quality


Add peerDependencies for eslint-config-airbnb-typescript

Hi,
Could you please add peerDependencies for eslint-config-airbnb-typescript?  For that I can get the peer dependencies through npm info *** peerDependencies
Thank you.

I agree with @abcrun, but for different reasons. When using yarn 2, yarn throws tons of warnings because eslint-config-airbnb-typescript doesn't provide the peer dependencies (which it shouldn't), but it should list the peer dependencies. I understand not wanting to do it because the peer dependencies vary between the main and base configs, not to mention npm and yarn (for now, that's coming in yarn 3) don't support conditional peer dependencies. However, the yarn developers have said that packages that do things like this will eventually be blocked from being installed entirely. I'm open to doing a PR if you want. It's literally a minor change to the package.json.

:tada: This issue has been resolved in version 13.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

13.0.0 fixed most of the peer dependency warnings, but eslint-config-airbnb-base still wants eslint and eslint-plugin-import ... I think to properly address these last two we'd need to add them as peerDependencies of this package, right? Am I missing something here?


[Docs] Updated docs page that lists all third-party tools

Background: https://airtable.com/tblD3oZPLJgGhCmch/viw1DKmbKhg2MIECH/recseiOcZshSc0jZk
Description: CockroachDB's third-party tooling page now contains an up-to-date via of support for drivers, ORMs, database migration tools, GUIs, IDEs, etc.
People will look for driver and ORM content outside of the specific language landing pages (e.g., Python page).
Team: Andrew Woods, Rafi Shamim, Eric Harmeling, Richard Loveland
Github Tracking Issue: https://github.com/cockroachdb/docs/issues/5626

Closing this as a dupe of https://github.com/cockroachdb/docs/issues/6326


Removing JSON.stringify from errors

JSON.stringify() of an Error object just returns "{}", which leads to logging empty messages.
Also, I've joined consequent logging to improve search in logs. Plus, I've removed some of the double-logging: if the error is logged, it shouldn't be thrown and vise-versa

backend doesn't work and logs this error

reverted with https://github.com/w3f/1k-validators-be/pull/3051


initial controller layout for uframe assets

@DanielJMaher

@FBRTMaka We are currently rebasing master to the release 3, so I am going to hold off on this.  I do have a question though that I put in the Changed files.

Might want to rebase with master and reissue PR??

@DanielJMaher Please merge this for this morning, work with B.Smith will be blocked DUN DUN DUN

@FBRTMaka this has been rebased?


Master - Instructions

Addition of first-run instructions

@JakeWharton @andysylvester Maybe create an INSTRUCTIONS.md so we do not clutter the README.md? Also the pictures would need to be updated when any major UI changes were implemented.  👍


Incorrect variant display when using a product slider (shopping experience)

Description: The configured storefront display for variant articles is not adopted 100% correctly in a product slider.
Environment: <IP_ADDRESS>
Steps to reproduce:

create a dynamic product group in which a variant article has been added via TAG (all child + father).
set the storefront display for this variant article to "Display single product > Main product"
link the dynamic group in an article slider on a store page.
Result: A variant is displayed in the article slider instead of the main article.

If you now link the same dynamic group to an article as cross-selling, the correct main article is displayed under the actual article as cross-selling.
The storefront display is therefore not processed correctly in the product slider.
Expected result: The main article is displayed.
Actual result: A variant is displayed.
[5_Frontend_Crossselling.png|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1710228705_65f004e186894_1710228316_65f0035c3d177_5_Frontend_Crossselling.png]  [4_Frontend_Slider.png|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1710228705_65f004e1c7bfb_1710228316_65f0035cb5045_4_Frontend_Slider.png]  [3_Shopping_Experience.png|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1710228706_65f004e214f23_1710228317_65f0035d04c24_3_Shopping_Experience.png]  [2_Dynamic_Group.png|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1710228706_65f004e2417de_1710228317_65f0035d318c2_2_Dynamic_Group.png]  [1_Storefront.png|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1710228706_65f004e251a26_1710228317_65f0035d51f85_1_Storefront.png]

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-34338.
[created from NEXT-34338, comment 484669]


Actionable button parameter "value" can not be set leading to error "Resource not found"

When i configure the Actionable Button, i dont get the option to specify the pzInsKey. When i only specify the flowaction I get the error "Resource not found"
Documentation:

Design time configuration:

Error at runtime:
{
"errorClassification": "Resource not found",
"localizedValue": "The resource cannot be found.",
"errorDetails": [
{
"message": "Error_Case_Not_Found",
"erroneousInputOutputFieldInPage": "",
"erroneousInputOutputIdentifier": "",
"errorClassification": "",
"localizedValue": "Case not found for the given parameter ID",
"messageParameters": []
}
]
}

Note that you should use promoted action if you want to show a button in the case view https://academy.pega.com/topic/optional-actions/v5#:~:text=The promoted Action is the,the first Case-wide Action.

Oh ok, so this only works when you use this control on the pzInsKey field?
I guess I then misunderstood the instructions in the documentation.
Op wo 6 mrt 2024 om 18:24 schreef Richard Marsot @.***>

Screenshot.2024-03-06.at.12.23.14.PM.png (view on web)
https://github.com/pegasystems/constellation-ui-gallery/assets/29442140/9bee6946-eed6-444b-8fde-db922c9a64ec
You should see the 'Instance Handle Key' in any case type - if not, check
if the pzInskey is a relevant record for Work-
—
Reply to this email directly, view it on GitHub
https://github.com/pegasystems/constellation-ui-gallery/issues/32#issuecomment-1981400673,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGJLJ5PW452KUI7YE2KCJGDYW5GMPAVCNFSM6AAAAABEJIWVOOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBRGQYDANRXGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Note that you should use promoted action if you want to show a button in the case view https://academy.pega.com/topic/optional-actions/v5#:~:text=The promoted Action is the,the first Case-wide Action.

Yes I am well aware of (and really appreciate) this option and will use it whenever I am able to.

You should see the 'Instance Handle Key' in any case type - if not, check if the pzInskey is a relevant record for Work-

Yes, I tried it and it works. Note that you can only use this control/component on a field that contains the pzInsKey. Perhaps something to add to the documentation to make it more clear? thanks for your support!

you can also use it on data object with pyGUID - I have updated the doc and added screenshot - hope it help - the action button is also showcases in the application provided here https://github.com/pegasystems/constellation-ui-gallery/releases

you can also use it on data object with pyGUID - I have updated the doc and added screenshot - hope it help - the action button is also showcases in the application provided here https://github.com/pegasystems/constellation-ui-gallery/releases

Thanks! Are there plans to enable the use of this actionable button inside a Gallery tile? I have a use case to make something similar to this:

For this use case, the promoted action is not available nor is the regular actions menu. The only option i have now is to use the preview and show the custom action button in the preview (because also the promoted action is not available in the preview).

there is no plan right now but you can build it as a constellation DX component - look at the kanban example - each card is a view - the view can contain the actionable button.

I released last week a new component called Card Gallery that would implement this use case - you use a view and the view can have an actionable button - it also have the built-in edit capability https://pegasystems.github.io/constellation-ui-gallery/?path=/docs/widgets-card-gallery--docs

thanks Richard, is this already included in the ConstellationUIGallery
ruleset in the package that is downloadable?
Op di 12 mrt 2024 om 12:23 schreef Richard Marsot @.***

:

I released last week a new component called Card Gallery that would
implement this use case - you use a view and the view can have an
actionable button - it also have the built-in edit capability
https://pegasystems.github.io/constellation-ui-gallery/?path=/docs/widgets-card-gallery--docs
—
Reply to this email directly, view it on GitHub
https://github.com/pegasystems/constellation-ui-gallery/issues/32#issuecomment-1991423214,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGJLJ5PJSYYFU3YARK3XZTTYX3QSLAVCNFSM6AAAAABEJIWVOOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJRGQZDGMRRGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Vriendelijke groeten,
Sander Schouten
T. +31683921155

Is the Pega jar also updated Rich with this component?

@vbhardwaj84  and @sschoute  - new release publish https://github.com/pegasystems/constellation-ui-gallery/releases/tag/1.0.8 with the Card gallery - Hope it helps!


Add README for addons that don't have them

Those add ons that I have recently contributed lack addons. This issue is to track the addition of them.

Merged via pull request #23


Allow multiple reactions to posts

There should be an option in the configuration to enable users to make multiple reactions to posts (e.g., like + share). There should be no limits to this (users should be able to like and dislike a post, if they wish).
The reactions column in the results should be split into a column per reaction. These columns can then contain true/false depending on whether the reaction was chosen.
If more than one reaction is made, then the dynamic feedback for both should be applied additively.

This is now available since v2.0.0


fix($animateCss): respect transition styles already on the element

Previously, $animateCss wouldn't use transition styles that were on the element
before the animation process started. Precisely, transition property, timing-function
and delay were overwritten in the process.
This is a fix for #12656
It's a bit of a special case, because all the transition properties are sitting on the element already before $animateCss is called.
This could also affect the animation property, I haven't checked yet.
Strangely enough, this fix doesn't work when the transition property is all, because for some reason $animateCss blocks transition duration etc. when it's all - see https://github.com/angular/angular.js/blob/master/src/ngAnimate/animateCss.js#L596 Removing this doesn't break any tests, though.

This looks good, we just need to check the ease default value and to update the timings value.

Please also squash both commits together since the first one fails without the second one.

Excellent work!

Thanks for the review!
I've added two more test:

Ensure that timing-function is set to ease if not defined (that should fail if a browser doesn't follow spec)
Ensure that transition-property is 'all' if not defined. I had to remove a condition from the code for that to pass.

Reopening to check what travis says.


New Ionicons support

Hello the ionic framework team just added over 1K icons on the release of Ionic 5.
Can you add the new ionic icons to the project ?
https://ionicons.com/
Thanks

Due to the large changes in version 5.0, I will update it in the larger version. Please stay tuned

Please update to Ionicons 5.0, it has really great icons unified between iOS and Android. Thanks.


Uptime, response time graph links misformatted

Describe the bug
Links get created as https://https//
To Reproduce
Steps to reproduce the behavior:
See here: https://github.com/chain-courses/exchanges/commit/f082911da37dc1fda171ba3db465bd4adbcdde35?short_path=b335630#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5
Expected behavior
A non-broken link
Additional Info
I followed the instructions, ended up with this. It seems a bit strange, not sure what went wrong but figure that it's an easy fix from the dev side where you might be more familiar with the code that creates these links :) Might be user error - in which case I might suggest revising the instructions for how to set up the .upptimerc.yml file!
EDIT: I think this might be related to using a full URL (incl https) in the cname line of the yml file. Maybe parsing that out, or providing docs to clarify would help?
This issue seems to have broken the repo - it caused the graph directory to be missing, which causes the setup CI breaks. Whoops.

Yes, it's most likely due to the CNAME, which should only have the hostname, not the protocol. I'll have a closer look soon!

The follow-up issue I faced:
Setup CI Update Template step
Run<EMAIL_ADDRESS>Removed legacy .github/workflows
Added new .github/workflows
Removed template files
Removed old data from api
ERROR [Error: ENOENT: no such file or directory, scandir 'graphs'] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'scandir',
  path: 'graphs'
}
Error: ENOENT: no such file or directory, scandir 'graphs'```

Great project BTW! As soon as I saw it I wanted to set it up and play with it 👍

Mohammed alrashdi


Renovate automerge occasionally skips Jenkins PR checks

I have logged proof of Renovate merging a PR when Jenkins tests had not registered a status yet. Flowzone did not update branch protections in any way, so the required checks were never removed.
But when Renovate queries the GH API to get a list or status checks it can sometimes fall into a window where the Jenkins build is done but the tests are not appearing. To me this is a bug in the GH API that you when the status checks are queried, required jobs must have started or registered in order to be returned. I don't really think Renovate could fix this even if we asked. I don't see how Renovate-approve bot would be any different, and we had similar issues in Flowzone and stopped relying on that endpoint entirely.
PR: https://github.com/balena-os/balena-radxa/pull/339
Renovate run: https://github.com/balena-os/renovate-config/actions/runs/4534103604/jobs/7987835601
2023-03-27T16:26:10.0862266Z DEBUG: branch status check result (repository=balena-os/balena-radxa, branch=renovate/layers-meta-balena-digest)
2023-03-27T16:26:10.0863199Z        "state": "success",
2023-03-27T16:26:10.0863684Z        "statuses": [
2023-03-27T16:26:10.0864114Z          {
2023-03-27T16:26:10.0865731Z            "url": "https://api.github.com/repos/balena-os/balena-radxa/statuses/bb9fd50bd637f8d63ba2b340ddf72d0a5cfda1bf",
2023-03-27T16:26:10.0866756Z            "avatar_url": "https://avatars.githubusercontent.com/in/278558?v=4",
2023-03-27T16:26:10.0867438Z            "id":<PHONE_NUMBER>8,
2023-03-27T16:26:10.0867990Z            "node_id": "SC_kwDOFyVqnc8AAAAFLwjNHA",
2023-03-27T16:26:10.0868557Z            "state": "success",
2023-03-27T16:26:10.0869133Z            "description": "All rules are approved",
2023-03-27T16:26:10.0870207Z            "target_url": "https://policy-bot.product-os.io/details/balena-os/balena-radxa/339",
2023-03-27T16:26:10.0871133Z            "context": "policy-bot: master",
2023-03-27T16:26:10.0871862Z            "created_at": "2023-03-27T14:34:05Z",
2023-03-27T16:26:10.0872949Z            "updated_at": "2023-03-27T14:34:05Z"
2023-03-27T16:26:10.0873633Z          },
2023-03-27T16:26:10.0874020Z          {
2023-03-27T16:26:10.0875128Z            "url": "https://api.github.com/repos/balena-os/balena-radxa/statuses/bb9fd50bd637f8d63ba2b340ddf72d0a5cfda1bf",
2023-03-27T16:26:10.0876170Z            "avatar_url": "https://avatars.githubusercontent.com/u/15143461?v=4",
2023-03-27T16:26:10.0876978Z            "id":<PHONE_NUMBER>2,
2023-03-27T16:26:10.0877708Z            "node_id": "SC_kwDOFyVqnc8AAAAFLycB-g",
2023-03-27T16:26:10.0878274Z            "state": "success",
2023-03-27T16:26:10.0878809Z            "description": "Build finished. ",
2023-03-27T16:26:10.0879806Z            "target_url": "https://jenkins.product-os.io/job/yocto-balena-radxa/768/",
2023-03-27T16:26:10.0880527Z            "context": "Jenkins build",
2023-03-27T16:26:10.0881218Z            "created_at": "2023-03-27T15:42:31Z",
2023-03-27T16:26:10.0881921Z            "updated_at": "2023-03-27T15:42:31Z"
2023-03-27T16:26:10.0882389Z          }
2023-03-27T16:26:10.0882785Z        ]

From a previous run:
2023-03-27T13:53:20.6961063Z DEBUG: branch status check result (repository=balena-os/balena-radxa, branch=renovate/layers-meta-balena-digest)
2023-03-27T13:53:20.6961541Z        "state": "pending",
2023-03-27T13:53:20.6961808Z        "statuses": [
2023-03-27T13:53:20.6962043Z          {
2023-03-27T13:53:20.6962688Z            "url": "https://api.github.com/repos/balena-os/balena-radxa/statuses/c244ed4979bb37338e3fdbbb63b7bc31cdba7108",
2023-03-27T13:53:20.6963247Z            "avatar_url": "https://avatars.githubusercontent.com/in/278558?v=4",
2023-03-27T13:53:20.6963612Z            "id":<PHONE_NUMBER>1,
2023-03-27T13:53:20.6963925Z            "node_id": "SC_kwDOFyVqnc8AAAAFLsRZNw",
2023-03-27T13:53:20.6964237Z            "state": "success",
2023-03-27T13:53:20.6964547Z            "description": "All rules are approved",
2023-03-27T13:53:20.6965125Z            "target_url": "https://policy-bot.product-os.io/details/balena-os/balena-radxa/339",
2023-03-27T13:53:20.6965632Z            "context": "policy-bot: master",
2023-03-27T13:53:20.6966037Z            "created_at": "2023-03-27T10:21:17Z",
2023-03-27T13:53:20.6966424Z            "updated_at": "2023-03-27T10:21:17Z"
2023-03-27T13:53:20.6966685Z          },
2023-03-27T13:53:20.6966900Z          {
2023-03-27T13:53:20.6967587Z            "url": "https://api.github.com/repos/balena-os/balena-radxa/statuses/c244ed4979bb37338e3fdbbb63b7bc31cdba7108",
2023-03-27T13:53:20.6968148Z            "avatar_url": "https://avatars.githubusercontent.com/u/15143461?v=4",
2023-03-27T13:53:20.6968499Z            "id":<PHONE_NUMBER>5,
2023-03-27T13:53:20.6968810Z            "node_id": "SC_kwDOFyVqnc8AAAAFLtqVsQ",
2023-03-27T13:53:20.6969117Z            "state": "success",
2023-03-27T13:53:20.6969413Z            "description": "Build finished. ",
2023-03-27T13:53:20.6970000Z            "target_url": "https://jenkins.product-os.io/job/yocto-balena-radxa/767/",
2023-03-27T13:53:20.6970355Z            "context": "Jenkins build",
2023-03-27T13:53:20.6970695Z            "created_at": "2023-03-27T11:41:55Z",
2023-03-27T13:53:20.6971031Z            "updated_at": "2023-03-27T11:41:55Z"
2023-03-27T13:53:20.6971249Z          },
2023-03-27T13:53:20.6971438Z          {
2023-03-27T13:53:20.6971954Z            "url": "https://api.github.com/repos/balena-os/balena-radxa/statuses/c244ed4979bb37338e3fdbbb63b7bc31cdba7108",
2023-03-27T13:53:20.6972440Z            "avatar_url": "https://avatars.githubusercontent.com/u/15143461?v=4",
2023-03-27T13:53:20.6972758Z            "id":<PHONE_NUMBER>9,
2023-03-27T13:53:20.6973357Z            "node_id": "SC_kwDOFyVqnc8AAAAFLtwS4Q",
2023-03-27T13:53:20.6973630Z            "state": "pending",
2023-03-27T13:53:20.6973887Z            "description": "Build started",
2023-03-27T13:53:20.6974371Z            "target_url": "https://jenkins.product-os.io/job/leviathan-v2-template/15764/",
2023-03-27T13:53:20.6974916Z            "context": "test-rockpi-4b-rk3399",
2023-03-27T13:53:20.6975281Z            "created_at": "2023-03-27T11:47:33Z",
2023-03-27T13:53:20.6975623Z            "updated_at": "2023-03-27T11:47:33Z"
2023-03-27T13:53:20.6975853Z          }
2023-03-27T13:53:20.6976052Z        ]

I disabled automerge in balena-os renovate again, and will investigate workarounds in Flowzone that could query the repo settings for required checks and block the rest of the job until they register.

This seems to have been resolved by ensuring the actor performing the automerge does not have admin:write permissions on the repository. This includes Flowzone toggling GitHub automerge functionality, and Renovate's built-in automerge features.


Update package.json

npm run build failing as ajv-keywords npm package is not present in the package.json. After adding this package the npm run build ran successfully.

While trying to Package the client app using npm run build, and error was encountered as there was a missing package ajv-keywords. Adding this package to the package.json solved the issue.

havent heard of this issue from other folks, and im not personally repo-ing so abandoning this for now, will reopen if other folks  see it


Crash when mass downloading

What steps will reproduce the problem?
Download every image on every booru
What is the expected behavior? What do you get instead?
not crash
What version of the program are you using? On what operating system?
v5.2.2 windows
Please provide any additional information below
Had to upload my log to this site since half the internet is down from the amazon outage,
https://u.nya.is/ebpgnm.zip

Download every image on every booru
expected not crash

Heh, what could go wrong? Right?
I don't really think that there are exist some downloader, who can do it.
P.S.
haw many images of 14 000 000 are you downloaded?

I got 22,415 images downloaded.
On Tue, Feb 28, 2017 at 4:06 PM, MasterPetrik<EMAIL_ADDRESS>wrote:

Download every image on every booru
expected not crash
Heh, what could go wrong? Right?
I don't really think that there are exist some downloader, who can do it.
P.S.
haw many images of 14 000 000 are you downloaded?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Bionus/imgbrd-grabber/issues/820#issuecomment-283160332,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AS2t8LEQN-btAkE2E8kwdQJY-0Dgd6yPks5rhIxNgaJpZM4MOzOE
.

well you could say. This was sorta a test of the new system.

Not bad! I wonder, how your IP still not bannd on boorus...

lol

@Bionus Wanna check this out for me?

I think that this issue related to https://github.com/Bionus/imgbrd-grabber/issues/618

Here is crash number two,
Crash two.zip

I think that this issue may be connected with https://github.com/Bionus/imgbrd-grabber/issues/547

The crash occured when downloading a 145MB file (NFSW: file & page), so the crash is more than likely to be caused by the size of said file.
See issue #954 and #963 for follow-up, I'll be closing this issue.


Fix double free of VirDomain object

What this PR does / why we need it:
The PR fixes a double-free error.
Do not call Free() in the main function body. GetDomainStats already registers a defered routine to cleanup the domains.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
I already have a PR addressing Free() calls in virtwrap/manager tests: https://github.com/kubevirt/kubevirt/pull/7700. This definitely needs to be covered by a unit-test so will try to add one there. Meanwhile submitting the fix.
Release note:

NONE

I am a bit concerned that it is already the second time I see similar error. There are no failures here in the upstream CI but when I run e2e tests with the downstream images I observe a crash.
Previously it was here: https://github.com/kubevirt/kubevirt/pull/5519#issuecomment-826055148
@davidvossel , @jean-edouard , @xpivarc , @vladikr FYI. Any ideas on that?

maybe a different go version?

maybe a different go version?

For the downstream build we use go 1.17.9. Here it is 1.17.8. But I dont think it is the reason since the same was observed before near 0.40.0 release. Another thing is that we base our images on TW. I wonder if it might be related to different build options in e.g. libvirt (compared to centos-strema).

maybe a different go version?

For the downstream build we use go 1.17.9. Here it is 1.17.8. But I dont think it is the reason since the same was observed before near 0.40.0 release. Another thing is that we base our images on TW. I wonder if it might be related to different build options in e.g. libvirt (compared to centos-stream).

Maybe @andreabolognani has an idea

I am a bit concerned that it is already the second time I see similar error. There are no failures here in the upstream CI but when I run e2e tests with the downstream images I observe a crash.

@vasiliy-ul can you get the crash to reproduce reliably downstream? Is there a specific e2e test where you hit it?
I can't think of any particular reason why you'd see the crash in one context but not the other. It would be really interesting if we could get the problem to show up upstream as well.

@vasiliy-ul can you get the crash to reproduce reliably downstream? Is there a specific e2e test where you hit it?

Yes, with downstream images it crashes all the times. It is test_id:4142: Prometheus Endpoints should include .... There are 8 tests with that id and all reliably fail.
Actually I have more failures during that run but checked only the Prometheus ... tests so far.

It would be really interesting if we could get the problem to show up upstream as well.

Yeah, that would be nice if we could spot such issues earlier.

Is the libvirt version very different between upstream and downstream? Honestly this doesn't look like the kind of issue that is introduced by changes in libvirt, but the Go versions are similar enough that it doesn't seem likely for that to be the culprit either.
A nice test might be to build images where both libvirt and Go come from entirely vanilla sources and see whether the behavior matches when using different base images. Not a trivial amount of work, I'm afraid.

Some new findings: the upstream libvirt somehow catches this issue:
{"component":"virt-launcher","level":"warning","msg":"Error freeing a domain.","pos":"libvirt.go:277","reason":"virError(Code=7, Domain=6, Message='invalid domain pointer in virDomainFree')","timestamp":"2022-05-13T09:22:19.347662Z"}

and just returns an error on the seconf Free() call.

/retest

Feel free to unhold. I am just not sure if you want to continue any discussion first.

I think the fix is good to go in. The discussion can still continue if someone has more thoughts about it. From my side I looked at libvirt code and virtDomainFree eventually just unrefs the objects. That leads to releasing the memory. Under the hood it uses Glib so my assumption would be just different flags related to memory mgmt.
/unhold

/cherrypick release-0.53


[Slider] onDragStop and onDragStart events have no value

Problem description
One cannot access the Slider's value on events onDragStop and onDragStart.
Edit: neither does onMouseUp work. Only onChange gets the event value attached.
I did injectTapEventPlugin();.
Steps to reproduce
<Slider onDragStop={this.onTest} ... />

onTest(e, value) {
    console.log(e, value, e.target.value)
}

Returns: Event, undefined, undefined.
Versions

Material-UI: 0.15.2
React: 15.2.1
Browser: Chrome

I think is you event bind wrong.
I wrote a demo for you, and the value is right.

import React from 'react';
import Slider from '../src/Slider';

import injectTapEventPlugin from 'react-tap-event-plugin';

// Needed for onTouchTap
// http://stackoverflow.com/a/34015469/988941
injectTapEventPlugin();

/**
 * `SelectField` supports a floating label with the `floatingLabelText` property. This can be fixed in place with the
 * `floatingLabelFixed property, and can be customised with the `floatingLabelText` property.
 */
export default class App extends React.Component {
  constructor(props) {
    super(props);
  }

  changeSlider(event, value) {
    console.log(value);
  }

  render() {
    return (
      <Slider
        defaultValue={1}
        onChange={(event, value) => this.changeSlider(event, value)}
      />
    );
  }
}

This is onChange in your example. onChange works fine. onDragStop does not.

Related PR: https://github.com/callemall/material-ui/pull/4712

I have this use-case where  the slider's value is changed with a timer (setInterval)  but it can be changed by the user if he slides the slider, the problem is that i want to catch the change of the value only if the user slides the slider and this is impossible right now :(  because the event we get from onDragStop doesn't have a value attribute.

We plan on migrating the component to the v1-beta branch. I'm closing the issue for #4793.


Large sitemaps time out on Vercel free tier

Summary: Because my Notion document is growing somewhat large, the sitemap request times out under Vercel's free tier (10 seconds).
Notion page: https://www.notion.so/dbredvick/Drew-tech-1e0b275c5ca44b4bbf5dc0e24017f593

I'll be looking into fixing this on my blog, will contribute upstream when I have a reasonable fix.

As an initial fix, bumping the concurrency from concurrency: 4 to  concurrency: 5 was sufficient for my blog in get-site-maps.ts.

Thanks @dbredvick; WIP solution tracking here https://github.com/transitive-bullshit/nextjs-notion-starter-kit/pull/83

I have the same issue, but "bumping the concurrency from concurrency: 4 to concurrency: 5" don't work for me.
Then I tried to access sitemap.xml locally,but I got 404.

I believe this should be fixed with the recent addition of optional redis caching for pathname to notion page ID lookups.

Unfortunately sitemap.xml generation outputs a 504 error.
I read that this is due to a 10 second limit on the free plan from vercel, but has been resolved with the redis labs update.
I have caching connected via redis labs (free plan), I checked the operation via monitor in redis-cli as I thought I might have a connection problem, but I can see the image caching working.
I've been reading the @keyvhq/redis documentation and googling solutions and similar errors, but unfortunately haven't been able to find any solutions in the last 5 days.
Perhaps the problem is related to the fact that many pages have manually (via table in notion) changed Slug.
Error page: https://www.boklanov.ru/sitemap.xml


[AutoPR cdn/resource-manager] [CDN] Add missing customHttpsParameters field of CustomDomainProperties.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7441

(message created by the CI based on PR content)