'Jud 2' goes to Revelation 1, not Judges 2.

I don't know if this quite counts as a bug. But it may not be quite the expected behaviour!
If you enter 'Jud 2' (or 3, 4, etc) into the reference search bar, it takes you to Revelation 1 (or 2, 3 respectively).
I'm guessing that Jud -> Jude, then it just counts forward 2 chapters, taking us to Revelation 1.
I discovered this while searching for Judges 6 (Jud 6), which took me to Revelation 5.
However, I suspect that it may be less confusing if the logic incorporated the maximum chapter count.
e.g. min(max_chapters_in_book, given_value)
Then 'Jud 6' would take one to Jude 1, which should hopefully be a good hint that we need to type 'Judg 6' instead!

Hi.
Yeah, I think 'Jud' is short form for 'Jude', where 'Judg' is for 'Judges'.
The logic behind the referencing and chapter counting is actually performed by the SWORD library where Eloquent is based on.
So Eloquent is not really involved in this.
I would refer you to SWORD (due to a lack of time) if you want to find out exactly what is the reason behind this behavior.

Ah! Thank you.


docs: add newer version alert messages

⚠ Must be merged once v5.3.0 is released
Closes #1575
Description
This PR is based on https://github.com/twbs/bootstrap/pull/38410 created upstream.
Depending on the Boosted version, the modifications are different:

[x] 3.3: Banner was already here so changed "The first Boosted 4 beta is here!" to "There's a newer version of Boosted!"
[x] 3.4: Banner was already here so changed "Looking for Boosted 4?" to "There's a newer version of Boosted!"
[x] 4.0 → 4.6: No banner so added a .d-block to the  plus the following code to display the banner:

        There's a newer version of Boosted!

Note: From 4.4 to 4.6, the link is slightly different to have a correct visible focus rendering:

[x] 5.0 → 5.1: No banner so added a new section after the header. Otherwise, the search bar would be hidden by an extra div on top of the header. Here's the new section:

        There's a newer version of Boosted!Newer version of Boosted!

Please note that the sentence is reduced in mobile mode and that the skip links require and extra dark background in order to be on top of our new banner:
Skip to main content

[x] 5.2: No banner so added a new section after the header. Otherwise, the search bar would be hidden by an extra div on top of the header. Here's the new section:

        There's a newer version of Boosted!Newer version of Boosted!

Local testing
php -S localhost:8008 # or any other command to launch a local server

Then check all the modified Boosted versions docs:

http://localhost:8008/docs/3.3/index.html
http://localhost:8008/docs/3.4/index.html
http://localhost:8008/docs/4.1/about/boosted-history/index.html
http://localhost:8008/docs/4.2/about/boosted-history/index.html
http://localhost:8008/docs/4.3/about/boosted-history/index.html
http://localhost:8008/docs/4.4/about/boosted-history/index.html
http://localhost:8008/docs/4.5/about/boosted-history/index.html
http://localhost:8008/docs/4.6/about/boosted-history/index.html
http://localhost:8008/docs/5.0/about/brand/index.html
http://localhost:8008/docs/5.1/about/brand/index.html
http://localhost:8008/docs/5.2/about/brand/index.html

Latest modifications:

[x] Removed docs/**/examples/** modifications (except docs/**/examples/index.html) that were issues
[x] From 4.0 to 5.1, .nav-item needed a .border-0
[x] From 4.4 to 4.6, link needed .pr-1 .pr-sm-2 for a correct visible focus rendering
[x] 5.2 stacked rendering is kept as-is because supra bar would need some extra modifications for the side navigation and it's too "complex" and far from the original design


What are the steps to add a new app to use the ui package?

For example a vite with react app.
which config should I do to make this new project us the ui package?
Thanks

+1

Steps added in 1.1.0 (b71d7828fae4b1612208f829e3be6e38a18c78dc)


Allow opting out of schema derivation per kind

The feature-flag approach was problematic since it applies to the whole dependency tree: as long as any crate depends on kube_derive without disabling default features then the derived schema becomes mandatory.
This PR adds a new #[kube(derive_schema = false)] attribute that allows you to specify this per kind.

It will force schemars into the dependency tree of people not overriding the schema, but because it's going to be required to have a schema anyway, that's probably OK.

I don't see how this PR changes anything here? kube-derive doesn't depend on schemars either way.

I don't see how this PR changes anything here? kube-derive doesn't depend on schemars either way.

oh, right that's in the generated code! that's even better. definitely no need for the feature then :+1:

@clux I redesigned the attribute a bit so that we now also support custom impl JsonSchema for MyKind.

I'd like to have this merged in because it's a great improvement. The only thing we miss here (other than solving the merge conflicts) is possibly better names, but none of my suggestions seem much better either, so lacking any new suggestions, let's merge this?

The merge conflicts should be fixed now @clux

LGTM. Although tests need the same replace. Happy to approve at green though!

There's always somethin, huh. :facepalm:


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_d4irr20sid/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_d4irr20sid/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_d4irr20sid/clone/recipe/meta.yaml:44: missing_description: The recipe is missing a description
ERROR: /tmp/abs_d4irr20sid/clone/recipe/meta.yaml:44: http_url: http://www.nvidia.com is not https
Errors were found


fix: grafana warnings

Remove grafana warnings
Merge after #1511
Description
Metrics names are not following conventions, so grafana shows a warning about that
ie: received_proofs -> received_proofs_count
How to Test

Run all the components

make run_metrics

Go to grafana dashboard localhost:3000

Grafana should show metrics correctly and should not show warnings

Type of change

[x] Bug fix

Checklist

[x] Linked to Github Issue

New clippy, new broken CI.


Crash in Android Lollipop

I am using Estimote SDK in my application.Its working fine upto 4.4.4.But crashes in Lollipop.I saw that this issue has been fixed in your update.But its still there.Please help me find the issue here.
Please see the crash log below.
Thanks.
java.lang.IllegalArgumentException: This cannot be run on UI thread, starting BLE scan can be expensive
at com.simelabs.kmb.spotbeak.internal.Preconditions.checkArgument(Preconditions.java:39)
at com.simelabs.kmb.spotbeak.service.BeaconService.checkNotOnUiThread(BeaconService.java:507)
at com.simelabs.kmb.spotbeak.service.BeaconService.access$3(BeaconService.java:505)
at com.simelabs.kmb.spotbeak.service.BeaconService$InternalLeScanCallback.onLeScan(BeaconService.java:485)
at android.bluetooth.BluetoothAdapter$2.onScanResult(BluetoothAdapter.java:1720)
at android.bluetooth.le.BluetoothLeScanner$BleScanCallbackWrapper$1.run(BluetoothLeScanner.java:330)
at android.os.Handler.handleCallback(Handler.java:739)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:135)
at android.app.ActivityThread.main(ActivityThread.java:5312)
at java.lang.reflect.Method.invoke(Native Method)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:901)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:696)

I’m pretty sure that you have not updated the library. In new one there is no such a message. Can you recheck that?
On Thu, Feb 5, 2015 at 8:40 AM, Jasmine Thomas<EMAIL_ADDRESS>wrote:

I am using Estimote SDK in my application.Its working fine upto 4.4.4.But crashes in Lollipop.I saw that this issue has been fixed in your update.But its still there.Please help me find the issue here.
Please see the crash log below.
Thanks.
java.lang.IllegalArgumentException: This cannot be run on UI thread, starting BLE scan can be expensive
at com.simelabs.kmb.spotbeak.internal.Preconditions.checkArgument(Preconditions.java:39)
at com.simelabs.kmb.spotbeak.service.BeaconService.checkNotOnUiThread(BeaconService.java:507)
at com.simelabs.kmb.spotbeak.service.BeaconService.access$3(BeaconService.java:505)
at com.simelabs.kmb.spotbeak.service.BeaconService$InternalLeScanCallback.onLeScan(BeaconService.java:485)
at android.bluetooth.BluetoothAdapter$2.onScanResult(BluetoothAdapter.java:1720)
at android.bluetooth.le.BluetoothLeScanner$BleScanCallbackWrapper$1.run(BluetoothLeScanner.java:330)
at android.os.Handler.handleCallback(Handler.java:739)
at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:135)
at android.app.ActivityThread.main(ActivityThread.java:5312)
at java.lang.reflect.Method.invoke(Native Method)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:901)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:696)
Reply to this email directly or view it on GitHub:
https://github.com/Estimote/Android-SDK/issues/100

Ok.Let me confirm once again.

I am using the latest updated SDK in my project.Could u tell me what is the fix you've done to support Lollipop?Icouldn't resolve this :(

The bug was inside the SDK which is closed source.
Can you remove old library? (like really remove it) and re add it again?
Can you tell me what is the jar size of the library? (just double checking)
How are you building the app? Maybe library is cached somewhere and you need to clear all caches.
On Thu, Feb 5, 2015 at 10:23 AM, Jasmine Thomas<EMAIL_ADDRESS>wrote:

I am using the latest updated SDK in my project.Could u tell me what is the fix you've done to support Lollipop?Icouldn't resolve this :(
Reply to this email directly or view it on GitHub:
https://github.com/Estimote/Android-SDK/issues/100#issuecomment-73016779

Is there any sample app which i can use to test the new sdk ?I want to confirm that my sdk is updated one.Please help.

Check out examples in SDK.
On Thu, Feb 5, 2015 at 1:30 PM, Jasmine Thomas<EMAIL_ADDRESS>wrote:

Is there any sample app which i can use to test the new sdk ?I want to confirm that my sdk is updated one.Please help.
Reply to this email directly or view it on GitHub:
https://github.com/Estimote/Android-SDK/issues/100#issuecomment-73038703

Hi Wiktor,
Now am learning your SDK. I saw previous comments on EstimoteAndroid SDK. I need to know whether it will work or not? on Android platform. If yes, means i need to know Which SDK version is minimum needed. You have any updated SDK ? Please help.

Estimote Android SDK works on Androids 4.3 or above.
On Fri, Mar 13, 2015 at 10:31 AM, siranjeevir<EMAIL_ADDRESS>wrote:

Hi Wiktor,
Now am learning your SDK. I saw previous comments on EstimoteAndroid SDK. I need to know whether it will work or not? on Android platform. If yes, means i need to know Which SDK version is minimum needed. You have any updated SDK ? Please help.

Reply to this email directly or view it on GitHub:
https://github.com/Estimote/Android-SDK/issues/100#issuecomment-78886830

I also get the ForceClose error mention above in Android 5 lollipop.
I've removed old sdk from Project and then readded the new sdk but error is still there...
/joakim


jenkins: make release.sh publish arm64 for Stable

From flatcar-3033 on, arm64 images are available for Stable channel, we need to also add arm64 to Stable boards.

[x] Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)

cherry-picked to flatcar-3033, flatcar-3066.


[AST] Preserve type sugar for generic typealiases

Introduce a new Type node, BoundNameAliasType, which describes a reference to a typealias that requires substitutions to produce the underlying type. This new type node is used both for references to generic typealiases and for references to (non-generic) typealiases that occur within generic contexts, e.g., Array<Int>.Element.
At present, the new type node is mainly useful in preserving type sugar for diagnostics purposes, as well as being reflected in other tools (indexing, code completion, etc.). The intent is to completely replace NameAliasType in the future.

@swift-ci please test

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 253bd10bd3c25c3c826d1b12b635a9cb93b7d2af

@swift-ci please test source compatibility

@swift-ci please smoke test Linux


Two question about play local file

Hello. Thank you for good library!
I try play local video file and I get exception APIError("Playback failed"):
cast = CattDevice(name=ChromecastConfig.device_name, ip_addr=ChromecastConfig.ip) cast.play_url('/home/user/Downloads/Bumblebee.2018.BDRip.720p.seleZen.mkv', resolve=True, block=True)

What a format of video file supporeted streaming?
What about path of video file? Does support abs path? or video should be
must be in directory with projet?

I'm afraid that the API does not support local file casting at all, at this moment.
On the general question of which formats the chromecast supports, please see:
https://developers.google.com/cast/docs/media

Oh, thank you. You are saved my time!


AWS BackUp

I am trying to get a template of how to create all the required AWS Backup resources (Backup plan, backup vault, etc).
It would be of great assistance if creating the above resources using the console, while having the Console Recorder running, would create the cloudformation template for these resources, so it can be automated when recreating them.
Thanks in advance.

Hi there,
I'm a little behind on the resource support for Console Recorder. https://github.com/iann0036/former2 is up to date though.
No promises on when I'll get to it in here I'm afraid.

I was not aware of former2.  That will solve exactly my problem, so thank you very much.


docs: Added documentation for useLazyQuery.

Adds documentation for useLazyQuery in package vue-apollo-composable, as mentioned in Issue #1308 .

Thank you so much for your PR! Since useLazyQuery is extending useQuery, I went with a lighter reference page. I also added a section about it in the guide.


Logging improvement

i think it would be a good solution to make a generic logging of (http) request + response in the pipeline (intercept call before processing request), so nobody would ever care to catch exceptions (they would be caught in the pipeline) unless some actions has to be taken on exception. For example do some cleanup or respond specific status/error code, but ONLY IF clients need to act on it.
Furthermore, when already have generic logging in pipeline we can upgrade it. That is to add logging of unique request correlation id of a specific request. This would allow to follow (in logs) all log path of specific request/response and see exception logged if it occurs.
Even furthermore, that correlation id could be added to headers of request (as well by interceptors in pipeline), this way the correlation id would allow to correlate all services (if there are multiple) chit-chatting.

We cant just add "middleware" to okhttp to catch all exceptions, and they are usually useful. I don't think catching all exceptions is solution for anything in any case anyway. And if we will catch all exceptions what would we return, null?
Unless I am misunderstanding what do you want I will just close this as invalid.


Export Typescript libraries as individual interfaces as well

Search Terms
export library interface
Suggestion
Right now Typescript declares it's libraries using d.ts (e.g. lib.dom.d.ts) so if you include this in your project you get the entire DOM library and it's interfaces. This is great however it makes development of isomorphic code harder.
Your only option is to add the dom library to your NodeJS project. This means that consumers could accidentally write unsafe code.
My proposal is really simple. Along with existing d.ts imports, we also export (in a separate file) the interfaces for those libraries
Use Cases
Isomorphic could be be implemented in an easier way. Right now we need to either include the whole library or use dependency injection. Which is not always trivial.
Here is an example of an issue encountered from the puppeteer repo - https://github.com/DefinitelyTyped/DefinitelyTyped/issues/24419#issuecomment-637700421
Examples
import { Window } from 'typescript/lib/lib.dom.interface'; // just an example name

window.alert('something'); // this will throw TS errors

page.evaluate(() => {
    // this will be fine and type checked according to window
    (window as Window).alert('something');
});

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Wouldn't adding the lib-reference in the source file only already solve this issue?
/// <reference lib="..." />

@MartinJohns when i used /// reference it actually includes the types for the whole project (it's similar to adding lib option in tsconfig.json)
And the Typescript documentation seems to suggest so: https://www.typescriptlang.org/docs/handbook/triple-slash-directives.html#-reference-lib-

For example, adding ///  to one of the files in a compilation is equivalent to compiling with --lib es2017.string.


Enable Running UnitTests in any version of AEDT

@maxcapodi78  I am trying to get the Unit tests to run in our internal regression tool with the newer versions, so In this PR instead of hardcoding the version to be 212, I am making it dependent on a new env var ANSYSEM_INSTALL_DIR and in your top level Unit tests *.cmd driving file I have set the path to be 21.2.

LGTM


⚠ Secret Website 3 has degraded performance

In b2413e9, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2077 ms

Resolved: Secret Website 3 performance has improved in e7e646a.


♻ refactor(gentest): Improved architecture

🗒 Description
This PR ensures data from gentest  to generated tests reaches in a type-safe manner.
The Gentest module generates a test script in three steps:

The CLI entrypoint function captures the user input required for generating the test.
This input is used to generate additional context required for creating tests.
Finally, using this context, a formatted pytest source code is generated and written to disk.

The process of generating context and rendering templates will share the same logic across different kinds of tests. These are created as modules: (cli, test_context_provider, and source_code_generator).
graph TD;
    A[CLI Entry Point] -->|Captures User Input| B[Test Context Provider]
    B -->|Generates Context| C[Source Code Generator]
    C -->|Generates Formatted Pytest Code| D[Disk]

I am using the same data type when passing data from the provider to the generated source code. This keeps the behavior predictable; Pydantic will raise errors if the RPC call returns unexpected values even before we run the test. I'm leveraging __repr__ values to print the data into the source code.
Finally, the code is formatted using the same formatter (Black) used elsewhere. The existing formatting configurations are applied, so any change in configuration should automatically reflect in the files generated as well. Formatting would become essential when reading contract bytecode, especially with the plans to convert bytecode to python objects (see #862).
This effort makes writing new kinds of gentests easier, requiring only the creation of new test providers and templates.
🔗 Related Issues
closes #859
✅ Checklist

[ ] All: Set appropriate labels for the changes.
[ ] All: Considered squashing commits to improve commit history.
[ ] All: Added an entry to CHANGELOG.md.
[ ] All: Considered updating the online docs in the ./docs/ directory.
[ ] Tests: All converted JSON/YML tests from ethereum/tests have been added to converted-ethereum-tests.txt.
[ ] Tests: A PR with removal of converted JSON/YML tests from ethereum/tests have been opened.
[ ] Tests: Included the type and version of evm t8n tool used to locally execute test cases:  e.g., ref with commit hash or geth 1.13.1-stable-3f40e65.
[ ] Tests: Ran mkdocs serve locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.

I think the output format of the generated test could be made better. Which is why the draft is still in PR.

We should ensure addresses is represented as hex and large numbers like v,r,s to be perhaps literal hex numbers. So that it is consistent with manually written tests. How about:

Field
Representation in Tests

Address
hex string

Hash
hex string

Number
hex number literal

Code
Opcode class?

I think the best way to achieve this is to override __repr__  in individual base types. Perhaps like this:
class HexNumber(Number):
    """
    Class that helps represent an hexadecimal numbers in tests.
    """

+   def __repr__(self):
+       return f"0x{123}"

    def __str__(self) -> str:
        """
        Returns the string representation of the number.
        """
        return self.hex()

I would like you to weigh in on the effect of this across the codebase.

I think the output format of the generated test could be made more human readable. Which is why the PR is still in draft.

For example, we should ensure addresses are represented as hex and large numbers like v,r,s to be perhaps literal hex numbers. So that it is consistent with manually written tests. How about:
Field 	Representation in Tests
Address 	hex string
Hash 	hex string
Number 	hex number literal
Code 	Opcode class?
I think the best way to achieve this is to override __repr__ in individual base types. Perhaps like this:
class HexNumber(Number):
    """
    Class that helps represent an hexadecimal numbers in tests.
    """

+   def __repr__(self):
+       return f"0x{123}"

    def __str__(self) -> str:
        """
        Returns the string representation of the number.
        """
        return self.hex()

I would like you to weigh in on the effect of this across the codebase.

I'll set off with a broader topic, so we have the full picture. Bear with me!
All right, so getting nice representations would also really help us with the Test Case Reference docs, see the bytes objects in the test case parameters tables, for example here: https://ethereum.github.io/execution-spec-tests/main/tests/prague/eip6110_deposits/test_deposits/test_deposit.html
For the objects defined and only used in the test cases (which aren't part of EEST's types libraries), adding a __repr__ would help and seems like the way to go.
For the objects that we do define in our test libraries, we get nice representations (of most objects, at least) when we write them as string to the fixture JSON files. And, instead of just adding __repr__ to all these classes, perhaps we can re-use the pydantic magic. I will try to get back to this later today, but if you'd like to play around in the meantime, try this:
Get some fixtures:
uv run fill tests/istanbul/

Then play around in ipython (https://ethereum.github.io/execution-spec-tests/main/dev/interactive_usage/)
uvx  --with-editable . ipython

for example:
import rich

from ethereum_test_fixtures.file import Fixtures
from ethereum_test_base_types import to_json

fixtures = Fixtures.from_file("fixtures/state_tests/istanbul/eip1344_chainid/chainid/chainid.json", fixture_format=None)
fixtures.keys()
fixtures['tests/istanbul/eip1344_chainid/test_chainid.py::test_chainid[fork_Istanbul-state_test]']
fixtures['tests/istanbul/eip1344_chainid/test_chainid.py::test_chainid[fork_Istanbul-state_test]'].env

rich.print(fixtures['tests/istanbul/eip1344_chainid/test_chainid.py::test_chainid[fork_Shanghai-state_test]'].env)

to_json(fixtures['tests/istanbul/eip1344_chainid/test_chainid.py::test_chainid[fork_Shanghai-state_test]'].env.fee_recipient)

to_json uses pydantic's model_dump():
https://github.com/ethereum/execution-spec-tests/blob/7758f37e3dca5f7b8b7ab0ea5136671ed4246623/src/ethereum_test_base_types/json.py#L10-L21
Just be aware that fill serializes from the test spec format (e.g. StateTest src/ethereum_test_specs/state.py to (state) Fixture src/ethereum_test_fixtures/state.py) and when we load the fixture from file above we're not creating a StateTest, we're creating a state test Fixture object. Quite a lot of magic happens when we serialize, for example, removing objects that are None; don't belong in state test fixture, but do in a blockchain test fixture.
I can take a deeper look myself and happy to discuss to brainstorm, but if you have time and you're interested, have play around!

I get your point. The to_json method respects Pydantic attributes (like alias) and filters out None values, while your overrides of __str__ produce nicely formatted JSON.
However, to_json outputs JSON, not valid Python code like __repr__. We need a way to combine the best of both without duplicating effort.
Let me know if my understanding is correct. Meanwhile, let me think this through.

Yup, that's right.
Yes, unfortunately pydantic doesn't deserialize to Python. Calling to_json on each individual field (the only way I can think of doing it at the mo) makes little sense. So, as we'll only need __repr__() for these base classes (I think), it'll have minimal impact to add them.
Perhaps @marioevz can chime in, if he has a better idea? :pray:
Sorry for the detour and that I couldn't give you a better answer earlier @raxhvl.

I have been marinating on this. I believe your intuition is correct: We should implement a unified serialization pipeline. This approach not only eliminates duplicate (tedious) efforts but also ensures that the framework produces consistent data outputs, whether for fixtures or gentests. Fundamentally, gentest should be as close to manual tests as possible.
Our goal is to serialize a Pydantic model to python code. After trying out few things, I have a solution.
We can override __repr__args__ in the root class, which feeds into Pydantic's __repr__. This will ensure that JSON serialization and __repr__ are consistent.
Here is a full example:
from pydantic import BaseModel, Field
from typing import List

class RecipeBaseModel(BaseModel):

    def __repr_args__(self):
       # Skip tags
        attrs_names = self.model_dump(mode="json", exclude={"tags"}).keys()
        attrs = ((s, getattr(self, s)) for s in attrs_names)
        return [(a, v) for a, v in attrs if v is not None]

# Example child model
class Ingredient(RecipeBaseModel):
    name: str
    quantity: str
    tags: List[str]

# Example nested child model
class Recipe(RecipeBaseModel):
    title: str
    ingredients: List[Ingredient]

# Example usage
ingredient1 = Ingredient(name="Tomato", quantity="2 cups", tags=["vegetable", "fresh"])
ingredient2 = Ingredient(name="Basil", quantity="1 bunch", tags=["herb", "fresh"])
recipe = Recipe(title="Tomato Basil Salad", ingredients=[ingredient1, ingredient2])

print(repr(ingredient1))  # Ingredient(name='Tomato', quantity='2 cups')
print(
    repr(recipe)
)  # Recipe(title='Tomato Basil Salad', ingredients=[Ingredient(name='Tomato', quantity='2 cups'), Ingredient(name='Basil', quantity='1 bunch')])

I know this will take some effort to implement correctly. But I think incentive to unify serialization will go down as the code diverges because of the refactoring involved.
Let me know if we should pursue this or override __repr__ everywhere for now.

I have been marinating on this. I believe your intuition is correct: We should implement a unified serialization pipeline. This approach not only eliminates duplicate (tedious) efforts but also ensures that the framework produces consistent data outputs, whether for fixtures or gentests. Fundamentally, gentest should be as close to manual tests as possible.
Our goal is to serialize a Pydantic model to python code. After trying out few things, I have a solution.
We can override __repr__args__ in the root class, which feeds into Pydantic's __repr__. This will ensure that JSON serialization and __repr__ are consistent.
Here is a full example:
from pydantic import BaseModel, Field
from typing import List

class RecipeBaseModel(BaseModel):

    def __repr_args__(self):
       # Skip tags
        attrs_names = self.model_dump(mode="json", exclude={"tags"}).keys()
        attrs = ((s, getattr(self, s)) for s in attrs_names)
        return [(a, v) for a, v in attrs if v is not None]

# Example child model
class Ingredient(RecipeBaseModel):
    name: str
    quantity: str
    tags: List[str]

# Example nested child model
class Recipe(RecipeBaseModel):
    title: str
    ingredients: List[Ingredient]

# Example usage
ingredient1 = Ingredient(name="Tomato", quantity="2 cups", tags=["vegetable", "fresh"])
ingredient2 = Ingredient(name="Basil", quantity="1 bunch", tags=["herb", "fresh"])
recipe = Recipe(title="Tomato Basil Salad", ingredients=[ingredient1, ingredient2])

print(repr(ingredient1))  # Ingredient(name='Tomato', quantity='2 cups')
print(
    repr(recipe)
)  # Recipe(title='Tomato Basil Salad', ingredients=[Ingredient(name='Tomato', quantity='2 cups'), Ingredient(name='Basil', quantity='1 bunch')])

I know this will take some effort to implement correctly. But I think incentive to unify serialization will go down as the code diverges because of the refactoring involved.
Let me know if we should pursue this or override __repr__ everywhere for now.

Thanks @raxhvl for taking the time to look so deeply into this! This looks like a great approach to unify our serialization and think we can start adding __repr__args__ to our types (or to a new class that the types inherit from).
If I understand correctly, the code paths for serializing the JSON fixture files would remain unchanged? But now we use  json.dumps() within the __repr_args__ method  to achieve compatibility? In this case, this doesn't seem like a huge change to the codebase (and certainly not a breaking change). But perhaps I've misunderstood how intrusive this is. If I'm right, you could try adding these methods upfront to achieve your goal for gentest now which would allow us to better evaluate the change.

I have added a new base class.
classDiagram
   BaseModel <|-- EthereumTestBaseModel
   EthereumTestBaseModel <|-- CopyValidateModel
   CopyValidateModel <|-- CamelModel

   class EthereumTestBaseModel{
    serialize()
    __repr_args__()
   }

The serialize method is a thin wrapper over model_dump which __repr_args__. Ideally I would like to change this:
https://github.com/ethereum/execution-spec-tests/blob/cfbfea9821e505a13227da85eb09f7d1642dc7f0/src/ethereum_test_base_types/json.py#L19
to input.serialize() so that serialization is at one place but I'm not sure if all classes does infact inherrit EthereumTestBaseModel. So the code is now duplicated at two places.
The __repr__ logic is working fine. It ignores None fields, supports custom representation if required and falls back to __str__ representation.

[ ] Migrated to_json to use serialize
[ ] Rebased on top of main

re: black to ruff migration.

The code uses cli to format code as long as ruff has a cli interface to format. The migration is trivial (<2 lines of code)
It would be nice if Mario reviews this once we are done.

All models are now using the same serialization logic. We have two types of custom models: one for BaseModel and one for RootModel. The customization for both of these models is handled through a shared mixin.
I have tested this thoroughly, but please review it with care as this change impacts EVERY Pydantic model in the codebase.
classDiagram
   BaseModel <|-- EthereumTestBaseModel
   RootModel <|-- EthereumTestRootModel
   ModelCustomizationsMixin <|-- EthereumTestBaseModel
   ModelCustomizationsMixin <|-- EthereumTestRootModel

   class ModelCustomizationsMixin{
    serialize()
    __repr_args__()
   }

I have tested this thoroughly, but please review it with care as this change impacts EVERY Pydantic model in the codebase. I'm especially unsure of EthereumTestRootModel override.

Hey @raxhvl, in regards to this, it looks good to me, I've asked @marioevz if he'd also like to take a look. I diff'd the fixtures locally and got an exact match.


Fix manual injection when webhook disabled

If webook is disabled, then Helm values for the webhook will not be
exposed. This means that in the ConfigMap that stores the values, we
will not have the rewriteAppHTTPProbe variable, causing errors. By
defaulting this to false, we keep the same behavior but succeed in the
case when the config is not present.
I also verified this is the only case of this bug, all other variables
read in the injection template are from global.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: howardjohn
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: greghanson
If they are not already assigned, you can assign the PR to them by writing /assign @greghanson in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS
pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b2f65b328615c5cd7bd951acfb0d150539f642f7
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b2f65b328615c5cd7bd951acfb0d150539f642f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b2f65b328615c5cd7bd951acfb0d150539f642f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b2f65b328615c5cd7bd951acfb0d150539f642f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-mixer-no_auth

prow/e2e-dashboard.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b2f65b328615c5cd7bd951acfb0d150539f642f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-mixer-no_auth

prow/e2e-dashboard.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b2f65b328615c5cd7bd951acfb0d150539f642f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-mixer-no_auth

prow/e2e-dashboard.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-bookInfoTests-trustdomain.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-trustdomain

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b2f65b328615c5cd7bd951acfb0d150539f642f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-mixer-no_auth

prow/e2e-dashboard.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-bookInfoTests-trustdomain.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-trustdomain

prow/e2e-simpleTests.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-mixer-no_auth

prow/e2e-dashboard.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-bookInfoTests-trustdomain.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-trustdomain

prow/e2e-simpleTests.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-mixer-no_auth

prow/e2e-dashboard.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-bookInfoTests-trustdomain.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-trustdomain

prow/e2e-simpleTests.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-mixer-no_auth

prow/e2e-dashboard.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-bookInfoTests-trustdomain.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-trustdomain

prow/e2e-simpleTests.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-simpleTests-cni

prow/e2e_pilotv2_auth_sds.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-mixer-no_auth

prow/e2e-dashboard.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-bookInfoTests-trustdomain.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-bookInfoTests-trustdomain

prow/e2e-simpleTests.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test e2e-simpleTests-cni

prow/e2e_pilotv2_auth_sds.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test istio_auth_sds_e2e

prow/release-test.sh
0a00c0742827123d995b0ddfcee5330f9b46441d
link
/test release-test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/assign @costinm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: costinm, howardjohn
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [costinm]
pilot/OWNERS [costinm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: costinm, howardjohn, john-a-joyce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [costinm]
pilot/OWNERS [costinm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn can you backport please?
Cheers
-steve

Backport to 1.1? The injector change is only in master - release 1.1 fails multicluster for other reasons

Actually you are right this isn't related to the injector change I just originally thought it was. But multicluster does still fail for other reasons, probably need to backport both

Final update -- this actually is not needed in 1.1. Helm will handle this (at install time, rather than runtime) without error. Then when we read in blank value we treat it as false and all is well. I tested this


Feature Request - AppCenter Test should support Appium .net

Currently AppCenter only supports Appium tests written in java. How about ones written in .NET

Hello, would you mind doing a feature request in the App Center Gitub repo? https://github.com/microsoft/appcenter
We like to keep this repo for feedback around the docs.
Thank you!


docs: Update the ReadMe file

Add yarn install commands instead of just yarn

Hi @abhishekpatel946! Could you please explain the problem you were running into? According to the docs, the yarn command will run yarn install. Thank you!

Hi @abhishekpatel946! Could you please explain the problem you were running into? According to the docs, the yarn command will run yarn install. Thank you!
Hi @deammer, According to docs only yarn runs perfectly but doesn't make if only yarn. If we install the package using then why don't we use yarn install instead of just yarn it much more makes sense to me. Thanks!

For the sake of brevity and ease of onboarding, I'm gonna keep it as is for now. If more people feel strongly about this, please feel free to create an issue. Thank you!


Arm Support

Is there any way to get this to work on Arm devices (specifically a raspberry pi 4)? I have managed to install everything, but when I try to run any of the commands I receive this error.
38555 illegal hardware instruction  python3 -m waifu2x.web.webgen.gen

I'm using 64 bit raspbian, and am inside a virtual environment. My python is version 3.9.2

I am not familiar with Raspberry Pi.
My first thought, the reason for the error is that PyTorch uses AVX/AVX2.
I recently wrote steps to build pytorch without AVX in the following Dockerfile. (Note that this even disables CUDA)
https://github.com/nagadomi/nunif/blob/master/Dockerfiles/Dockerfile.cpu_noavx
At that time, I read various blog posts about building PyTorch for Raspberry Pi.
However, I just searched and found that PyTorch supports aarch64, which is described in the official PyTorch article.
https://pytorch.org/tutorials/intermediate/realtime_rpi.html
I am sure that one of the pip modules uses instructions that are not supported by that CPU, but I don't know which one.
I think you should check the following.

Is pytorch working well?
which line is causing the error

For 2, I configured the faulthandler to detect C-level errors but it doesn't seem to be working.
https://github.com/nagadomi/nunif/blob/c4ef47f220d8b4e4e4af1909bd20dd67f9f2055d/waifu2x/web/server.py#L456-L460
Maybe it is crashing at module import time before main function.

Thanks! I’ll check it out, but I might not have time until this weekend. I’ll keep you posted

This error is a bug in the pytorch aarch64 binary.
Just importing torch crashes.
$ python
Python 3.9.16 (main, Mar 30 2023, 13:22:28)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import torch
Illegal instruction

pytorch developers are currently working on this error.
https://github.com/pytorch/pytorch/issues/97226


Add possibility to do reparent away from a certain tablet.

This modifies PlannedReparentShard vtctl command and adds to it -avoid_master
flag to be able to do reparent to any tablet other than the specified one. The
reparent will be done to another tablet in the same cell as the current master
on the shard if the shard has the master. If the shard doesn't have the master
then the choice of the new master will be done without cell restriction. If the
flag -avoid_master will contain an alias of a tablet that is not the current
master for the shard then the PlannedReparentShard will succeed without doing
anything.
To make the reparent command's parameters more consistent I've also changed the
way the new master is specified -- it will be done now through -new_master flag.
Again for consistency reasons that change is done both for PlannedReparentShard
and EmergencyReparentShard commands. But then with that change made I needed to
do another change to be able to have the command line of PlannedReparentShard
and EmergencyReparentShard in the same order as it was (i.e. keyspace/shard
first and new master after that) -- I've converted keyspace/shard into its own
flag -keyspace_shard as well (because flags must go before the positional
parameters). Both PlannedReparentShard and EmergencyReparentShard still support
the old command line syntax (<keyspace/shard> ), but to be able to
phase that out I've converted all usages of these commands in our tests to the
new syntax.

LGTM overall. I am not sure we need the backward compatible mode for the flags, but it's not too bad to support it, so we can keep it.

I kept the backward compatibility in case someone who already uses Vitess has some scripts for reparenting. Those scripts shouldn't get broken after an upgrade.


README: fix typo

fix ano -> and typo in README

Good catch, thanks for the pr @FrankiePo !

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Leap.Hand.SphereCenter does not work after Leap.Hand.TransformedCopy

The code inside calculateSphere() does not account for the scaling suffered in TransformedCopy when calculating the _sphereRadius (since _maxSphereradius and _minSphereRadius are always in millimeters), and then _sphereCenter is wrong too.

Thanks for reporting this. Hand.SphereCenter and Sphere radius are being removed in the next drop, so this issue is moot. Closing.


SovietSpaceProgramCareerConversion

Closes #4438
Created by shell script
Not sure if the Subassemblies should be installed to the root or not.  There is also another file in the root of the zip file, RemoteTech_Antennas.cfg

The forum thread has special install instructions:

Installation Instructions:
Unzip the contents of "gamedata" into your "gamedata" like most other mods.
If you wish, copy the craft files from the ships and subassemblies folders into your save.
If you are using remotetech - and this is VERY important - you need to delete a couple of files (sorry) in order for things to work correctly. Consider moving them off somewhere so you can replace them later if you uninstall this mod.

Delete gamedata/RemoteTech/RemoteTech_Antennas.cfg and replace it with the one in the base leve of the zip.
Delete gamedata/Contares/Patches/CONTARES_RT2.cfg
Delete gamedata/Tantares/_Extra_RemoteTech.cfg

CKAN won't do any of the that stuff for RemoteTech, so we'll have to just leave that RemoteTech config file uninstalled and mark this mod as conflicting with RemoteTech.
If there are SPH/VAB folders, we can install the craft files in Ships, otherwise, just leave them out.

@politas seems to be good, please check

The contents of Ships should go in Ships rather than GameData


Adding GetProperties policy enforcement point.

GetProperties allows getting information about what is running. This is related to debugging etc. This gates it with a simple yes/no for policy. This might be used by ACI and others for getting stats and ACI tooling should set to allow as the default.

GetProperties allows getting information about what is running. This is related to debugging etc. This gates it with a simple yes/no for policy. This might be used by ACI and others for getting stats and ACI tooling should set to allow as the default.

no need to mention ACI and I think stats is the primary feature here


Unable to fetch current robot state Moveit2

Morning,
I would like to ask you help regarding the Moveit2 setup with ignition. I'm using ROS2 Humble with ignition gazebo. After launch the ur_sim_moveit.launch.py, i tried to load the move group interface in a cpp node (the example of basic tutorials in the moveit2_tutorials pkg) and i got this error in the terminal when trying to catch the currentRobotPose.
"unable to fetch the current robot state"
Is there any steps that i forgot or someone has experiences a similar issue?
Thank you very much for your kindly support.
Stefano

I manage to solve. When you launch a node, you need to pass in the parameters the flag use_sim_time set to True. In this way, I'm able to retrieve the getCurrentPose().pose from the moveit_interface (moveit::planning_interface::MoveGroupInterface group(node, "ur_manipulator);
This is the launch file:
from launch import LaunchDescription
from launch_ros.actions import Node
from launch.substitutions import LaunchConfiguration
from launch.actions import DeclareLaunchArgument

def generate_launch_description():
    use_sim_time = LaunchConfiguration("use_sim_time")
    return LaunchDescription([
        DeclareLaunchArgument(
                "use_sim_time",
                default_value="true",
                description="Using or not time from simulation",
            ),
        Node(
            package='planning_utilities_pkg',
            executable='planning_server',
            output='screen',
            parameters=[{
                "use_sim_time": use_sim_time,
            },
        ]
        )
    ])

Yes, use_sim_time is important. Glad you solved it yourself. I'll go ahead and close this issue, if you don't mind.


Failure to send transaction to testnet

Hi,
All of a sudden (9/27 after 6 PM Berlin time) sending transactions to the testnet fails with the following failure
09-27 13:00:02.200: W/System.err(5687): javax.net.ssl.SSLHandshakeException: Connection closed by peer
09-27 13:00:02.200: W/System.err(5687): at com.android.org.conscrypt.NativeCrypto.SSL_do_handshake(Native Method)
09-27 13:00:02.200: W/System.err(5687): at com.android.org.conscrypt.OpenSSLSocketImpl.startHandshake(OpenSSLSocketImpl.java:351)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.connection.RealConnection.connectTls(RealConnection.java:299)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.connection.RealConnection.establishProtocol(RealConnection.java:268)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.connection.RealConnection.connect(RealConnection.java:160)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.connection.StreamAllocation.findConnection(StreamAllocation.java:256)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.connection.StreamAllocation.findHealthyConnection(StreamAllocation.java:134)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.connection.StreamAllocation.newStream(StreamAllocation.java:113)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.connection.ConnectInterceptor.intercept(ConnectInterceptor.java:42)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:147)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:121)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.cache.CacheInterceptor.intercept(CacheInterceptor.java:93)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:147)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:121)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.http.BridgeInterceptor.intercept(BridgeInterceptor.java:93)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:147)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.http.RetryAndFollowUpInterceptor.intercept(RetryAndFollowUpInterceptor.java:125)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:147)
09-27 13:00:02.200: W/System.err(5687): at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:121)
09-27 13:00:02.201: W/System.err(5687): at com.bigchaindb.builders.BigchainDbConfigBuilder$Builder$2.intercept(BigchainDbConfigBuilder.java:199)
09-27 13:00:02.201: W/System.err(5687): at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:147)
09-27 13:00:02.201: W/System.err(5687): at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:121)
09-27 13:00:02.201: W/System.err(5687): at okhttp3.RealCall.getResponseWithInterceptorChain(RealCall.java:200)
09-27 13:00:02.201: W/System.err(5687): at okhttp3.RealCall.execute(RealCall.java:77)
09-27 13:00:02.201: W/System.err(5687): at com.bigchaindb.util.NetworkUtils.sendGetRequest(NetworkUtils.java:80)
09-27 13:00:02.201: W/System.err(5687): at com.bigchaindb.builders.BigchainDbConfigBuilder$Builder.setup(BigchainDbConfigBuilder.java:152)

Thanks for filing this bug report, @adonnini
Could you give some more details:

What operating system (and version) are you using?
What version of the Java driver?
Are you able to reproduce this error, i.e. starting from a fresh state?
Are all posted transactions failing to get though after some point? If so, do you know what the trigger was? Is it random? Is it some specific transaction?

As I write this, the BigchainDB Testnet is up and running, and when I post a new transaction to it from the page https://www.bigchaindb.com/developers/getstarted/ it works. (That page uses the JavaScript driver though.) Therefore the problem isn't with the testnet.
Did you try Googling the error message, such as javax.net.ssl.SSLHandshakeException: Connection closed by peer

I inserted answers to the best of my ability in line below.
Thanks,
Alex
Troy McConaghy wrote:

Thanks for filing this bug report, @adonnini https://github.com/adonnini
Could you give some more details:

What operating system (and version) are you using?

Android 8.0

What version of the Java driver?

1.1

Are you able to reproduce this error, i.e. starting from a fresh
state?

Yes

Are all posted transactions failing to get though after some
point? If so, do you know what the trigger was? Is it random? Is
it some specific transaction?

All transactions cannot be posted. The same code worked until earlier today
Transaction transaction = new Transaction();
                    try {
                        transaction = BigchainDbTransactionBuilder
                                .init()
                                .addAssets(assetDataMap, TreeMap.class)
                                .addAssetDataClass(TreeMap.class, null)
                                .operation(Operations.CREATE)
                                .buildAndSign((EdDSAPublicKey)
keyPair.getPublic(), (EdDSAPrivateKey) keyPair.getPrivate())
.sendTransaction(handleServerResponse());

As I write this, the BigchainDB Testnet is up and running, and when I
post a new transaction to it from the page
https://www.bigchaindb.com/developers/getstarted/ it works. (That page
uses the JavaScript driver though.) Therefore the problem isn't with
the testnet.
Did you try Googling the error message, such as
|javax.net.ssl.SSLHandshakeException: Connection closed by peer|
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bigchaindb/java-bigchaindb-driver/issues/36#issuecomment-425187063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbhsoVVLckc3wb-hc7KCXQIq3GwWUZmOks5ufRINgaJpZM4W9I5m.

Hi,
Problem solved??
I am at a different location. The problem does not occur any longer.
Thanks,
Alex Donnini
Troy McConaghy wrote:

Thanks for filing this bug report, @adonnini https://github.com/adonnini
Could you give some more details:

What operating system (and version) are you using?
What version of the Java driver?
Are you able to reproduce this error, i.e. starting from a fresh
state?
Are all posted transactions failing to get though after some
point? If so, do you know what the trigger was? Is it random? Is
it some specific transaction?

As I write this, the BigchainDB Testnet is up and running, and when I
post a new transaction to it from the page
https://www.bigchaindb.com/developers/getstarted/ it works. (That page
uses the JavaScript driver though.) Therefore the problem isn't with
the testnet.
Did you try Googling the error message, such as
|javax.net.ssl.SSLHandshakeException: Connection closed by peer|
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bigchaindb/java-bigchaindb-driver/issues/36#issuecomment-425187063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbhsoVVLckc3wb-hc7KCXQIq3GwWUZmOks5ufRINgaJpZM4W9I5m.

I'm not an expert on Android or Java, but our local expert was going to take a look at this today.
Looking at the error messages, it seems that something happened on your Android device causing SSL/TLS to stop working properly.

I think you may be right. As of now, the problem is not occurring any longer.
Thanks,
Alex
Troy McConaghy wrote:

I'm not an expert on Android or Java, but our local expert was going to take a
look at this today.
Looking at the error messages, it seems that something happened on your Android
device causing SSL/TLS to stop working properly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bigchaindb/java-bigchaindb-driver/issues/36#issuecomment-425347551,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbhsoZ2bvJBgacbD7lBvcjZ5o3FPq10Uks5ufc-XgaJpZM4W9I5m.

Thanks @adonnini for confirming. I also didn't think its anything related to the driver. So, I am gonna close this issue, if its alright.


.Net: Agent Examples README

Currently placeholder

Added: https://github.com/microsoft/semantic-kernel/pull/6070


Bigger free_num for lv_obj_t?

At present I'm using the free_num in the returned object to add an index on listview items but envisage that there will be more than 1000 items.  Would it be possible to add a free_word or is there a better way to store the index values?

I can modify the LV_OBJ_FREE_NUM define in lv_conf.h to set the the type of free_num. For example:
#define LV_OBJ_FREE_NUM   uint16_t
Using 1000  element is list could cause other issues. If an element is e.g. 50 px heigh, the last will be on 50000 px coordinate. Now the GUI should be between -16K and +16K. See misc/gfx/area.h

Interesting point.  If I limit it to 300 then that would be enough (although still outside the 256 range for 8 bits).  Either that or I'm more intelligent in how I load the list into memory and the listview.

If you tell what you would like to do hopefully I can suggest a solution. (in a new topic)

Thanks for the offer.  Let me play about with it here and see what I can come up with first as I don't want to waste your valuable time (which should be devoted to further develping littlevgl!)

Okay, se what happens :)
Anyway, the updating of LV_OBJ_FREE_NUM still might be a useful thing.

In lv_conf.h I added  LV_OBJ_FREE_NUM_TYPE to define the type of the free number (e.g. uint16_t)
It is available in lvgl/beta. It is the beta version of v5.0 so there are non-backward compatible API changes in it.

Added in v5.0.0


Deserialise not respecting type passed in for JsonPatchDocument

Source/destination types
    public class MyPatchModel
    {
        public string Name { get; set; }
        public string Notes { get; set; }
    }

Source/destination JSON
"[\r\n    {\r\n      \"op\": \"replace\",\r\n      \"path\": \"/NoneExistentProperty\",\r\n      \"value\": \"Some Value\"\r\n    }\r\n]"

Expected behavior
I Expected either a deserialisation error telling me it couldnt deserialise, or a model with no operations
Actual behavior
Instead i got a model with an operation that is not valid for the type passed in

Steps to reproduce
JsonConvert.DeserializeObject<JsonPatchDocument<MyPatchModel>>(myJson)

I don't understand your expectations. There is no issue/bug here.
You are deserializing a JsonPatchDocument<T>. You are not deserializing a MyPatchModel.
As such, the JsonPatchDocument<PatchModel> instance got all the data (i.e., operation) from the json data, as should be expected.

I don't understand your expectations. There is no issue/bug here.
You are deserializing a JsonPatchDocument<TModel>. You are not deserializing a MyPatchModel.
As such, the JsonPatchDocument<MyPatchModel> instance got all the data (i.e., operation) from the json data, as should be expected.
Note that the value property of Operation is of type object (it is inherited from the non-generic Operation type). Thus, from the point of view of Newtonsoft.Json, having a "Some Value" string (or anything else unrelated to MyPatchModel for that matter) for the operation value is totally a-okay.
If that is a problem for you that you cannot work around, you will have to take this up with the ASP.NET Core team...

I guess I assumed that the deserialising would honour the object being used, either through reflection of properties compared against the paths of some other logic.
I feel like how it is now is misleading as I could pass any object into the deserialisation and get a list of operations that have no bearing on that type

I feel like how it is now is misleading as I could pass any object into the deserialisation and get a list of operations that have no bearing on that type

Yeah, it kinda is. But it is not related to Newtonsoft.Json functionality, bugs or limiations. Your problem is chiefly with ASP.NET's Operation<TModel> and ASP.NET's TypedJsonPatchDocumentConverter.
I unfortunately don't know what would be considered best practices with regard to handling of inapplicable operators in ASP.NET's JsonPatchDocuments. It would probably be a good question for stackoverflow.com...

I feel like how it is now is misleading as I could pass any object into the deserialisation and get a list of operations that have no bearing on that type

Yeah, it kinda is. But it is not related to Newtonsoft.Json functionality, bugs or limiations. Your problem is chiefly with ASP.NET's Operation and ASP.NET's TypedJsonPatchDocumentConverter.
I unfortunately don't know what would be considered best practices with regard to handling of inapplicable operators in ASP.NET's JsonPatchDocuments. It would probably be a good question for stackoverflow.com...

No problem. I appreciate your time and patience! I'll try to find an alternative solution.


Dependency update in go-ns to log partial tokens in middleware

What
Tokens are partially logged in middleware as well as hasPrefix and the number of token parts (represents the number of times the string was split on whitespace)
How to review
Check we are happy with logging out the last 6 characters of the token
Who can review
Team B

Closing as it is out of date


Fix _RowLinearAsyncCommunication

As discussed in #46, the implementation of row parallel linear backward pass seems wrong, when TP mode is reduce-scatter and async communication is enabled. To be more specific, the gradient of input tensor is produced by

compute local slice of input gradient
allgather slice of input gradient

This will produce the same input gradient for all TP shards, which is not the correct behavior of Row Parallel Linear layer.
Also, the check of input gradient for row parallel is also missing in test cases. After adding the test, the result for test_tensor_parallel.py::test_row_linear[True-TensorParallelLinearMode.REDUCE_SCATTER-4-1-1] is as follows:

Only 1/4 of the input gradient values are correct, because that's the locally computed part.
This PR does the following:

Fixed the backward pass of _RowLinearAsyncCommunication, in a similar way to forward pass of _ColumnLinearAsyncCommunication that overlaps communication with part of computation.
Add the missing test that checks the correctness of input gradient in _RowLinearAsyncCommunication

This bug is related to convergence and can be triggered when TP mode is reduce-scatter, and async communication is enabled (which is a common setup for users).
Please fix this, thanks!

merged upstream and fetched


Voice support

It'd be great to build an interface that enables communication from Abot via voice.
Whoever wants to tackle this would have to design an interface to communicate with arbitrary voice recognition APIs (Google, Nuance, etc.), then implement that interface in a driver for at least one of those.
To get started, take a look at one of the existing interfaces, such as the SMS interface. Then see how a driver integrates with it by implementing that SMS interface: Twilio driver.

you mean speech to text or text to speech?

Great point. We'll actually need both! It does no good to understand speech but not be able to respond in kind.
Any interface would have to ensure that drivers supported both speech to text and text to speech.

Ignore the removal of the help wanted label. Help is still wanted :)

Google's speech API isn't public yet, so we'll have to wait on that.
Side note: is this feature specific to desktop clients? Because most mobile interaction would be through text, which would use iOS/Android voice support.

Desktop and mobile in the case of an app. Think Siri more than Facebook M

IBM Watson also have an API for speech-to-text and text-to-speech http://www.ibm.com/smarterplanet/us/en/ibmwatson/developercloud/speech-to-text.html

IBM Watson speech-to-text accuracy is so so.
Bing Speech API is surprisingly a lot more accurate and fast -
https://www.microsoft.com/cognitive-services/en-us/speech-api

The Google Speech API is now also available and quite good:
https://cloud.google.com/speech/
Besides that, there are some other very interesting APIs, like the Natural Speech API for analyzing the text:
https://cloud.google.com/natural-language/
I would use those


rootCertificatePath is nil

I just tried the latest build and rootCertificatePath seems nil and fails verification.

@chrisvanbuskirk checking in a few...

@chrisvanbuskirk fixed! Updating cocoapods now.
Thank you!

It looks like the issue was only fixed for CocoaPods users. When integrating TPInAppReceipt with SwiftPM, the certificate file won't be found because the current SwiftPM release misses the feature of adding resources to package. Until SE-0271 makes it into Swift 5.3, it should probably be documented to add the cer file to the main bundle.

@IngmarStein You

@IngmarStein Just updated Readme. Thanks again for pointing this out.


Envoy %GRPC_STATUS_NUMBER% got String instead of Number

Hi, I have use Envoy with custom logging with %GRPC_STATUS_NUMBER% but I got String instead of Number
Config
"status_code": "%GRPC_STATUS_NUMBER%",

Result
"status_code": "OK",

Could you share which version of Envoy you use?

@daixiang0 I use this version
envoy  version: 4ce93dc3ace00ae9108b179d0afaceac13f4602a/1.17.4/Modified/RELEASE/BoringSSL
I think this my misunderstand; I think if this command operator not set will be error instead of return the main string. I will try to upgrade envoy to >= 1.22.0 since command operator %GRPC_STATUS_NUMBER% added.
Thank you

Sure, you are welcome, hope it helps you.


Filter Function

Hi,
I would like to use your API for a simple Widget and wondered whether there is a filter function provided to filter the route results by product / type to make the JSON processing easier. Maybe this would be a nice way to improve the API.
Keep on the good work!

Hi,
thanks for your input, I can definitely see how this can be useful but I'm a bit unsure whether I would like it to be part of the library. Right now, the library doesn't make a lot of assertions about the contents of the api responses as it proved to be quite unstable in the past.
However, I'd definitely like having a few recepies for things like this in the documentation.
For example, you can filter by product like this:
filter(lambda d: d['product'] == 'UBAHN', mvg_api.get_departures(1))


Clean up play tests

Codecov Report

Merging #151 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #151   +/-   ##
=======================================
  Coverage   78.64%   78.64%
=======================================
  Files          32       32
  Lines         281      281
  Branches        2        2
=======================================
  Hits          221      221
  Misses         60       60

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 4d8acbf...6870b10. Read the comment docs.

I am not sure why these two lines aren't covered.

@peterneyens maybe the test not using that constructor?


Use std::ptr::write instead of dereferencing a raw pointer and assigning

https://doc.rust-lang.org/std/ptr/fn.write.html

ptr:write does not call Drop. This is not the right thing to do. WONTFIX.


Broken link on CC0 page

"Use this license" returns a 404 https://creativecommons.org/publicdomain/zero/1.0/
https://creativecommons.org/choose/results-one?license_code=CC0&jurisdiction=&version=1.0&lang=en

Try now?

This is still broken, got an email from someone from info@:
On this page https://creativecommons.org/publicdomain/zero/1.0/
at the bottom it says "use the license" and gives a link. This link is dead:
400 Bad Request
The server could not comply with the request since it is either malformed or otherwise incorrect.
One or more query values were bad.

this is still broken. Any chance @TimidRobot can help?

I just ran into this issue as I was showing someone how to use the license. Would be great to have it fixed.


Building fails because building_map_model_downloader downloads to home directory

Bug report
Required information:
This is the exact error on the buildfarm:
https://build.ros2.org/view/Rbin_uF64/job/Rbin_uF64__rmf_demos_maps__ubuntu_focal_amd64__binary/1/console
Description of the bug
building_map_model_downloader tries to download files to the user's home directory:
https://github.com/open-rmf/rmf_demos/blob/6c76a8ed135752e4d4c02287449a83d8d9876ec6/rmf_demos_maps/CMakeLists.txt#L44
While installing to the user's home directory works when building from source, it does not when we try to create a package out of that source.
Steps to reproduce the bug
Build a package.
Expected behavior
Successfully builds.
Additional information
A refactoring of this behavior is needed to install those files in a package relative location so that all required files can be packaged and installed adequately (suggestions to this approach are more than welcomed).

This has been solved by https://github.com/open-rmf/rmf_demos/pull/68. The solution it's probably not perfect but it solves our needs for now. Whenever packages grow too big or building sources becomes an issue we shall revisit this issue. Will close for now.


Fix ControlMessage::encode_into when encoding multiple messages

copy_bytes updates dst so that it points after the bytes that were just
copied into it. encode_into did not advance the buffer in the same way
when encoding the data.
See #473

r? @kamalmarhubi @posborne

@homu r=fiveop

:pushpin: Commit 3faaca1 has been approved by fiveop

:hourglass: Testing commit 3faaca1 with merge 5d81571...

:sunny: Test successful - status


What should I do if windows runs a compiled version and reports the following error

which version is it? I've deleted some buggy versions from releases page. Please try with the latest version. If you're still get bugs, please let me know.


[Crunchyroll] JSON Metadata - 403 Forbidden

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
United States
Provide a description that is worded well enough to be understood
Receiving 403 Forbidden errors trying to download from Crunchyroll. I've attempted two different videos so far, both work in browser from same IP, neither work with yt-dlp, and both produce the same 403 Forbidden error while attempting to download the JSON metadata.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '--username', 'PRIVATE', '--password', 'PRIVATE', '--sub-langs', 'en-US', '--remux-video', 'mkv', '--embed-subs', 'https://www.crunchyroll.com/watch/G6P8170J6/whisper-chant-prayer-awaken']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [4d9231208] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.22631-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg N-116599-g43cde54fc1-20240811 (setts)
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.07.04, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.35.5, urllib3-2.2.2, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1830 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[debug] Loading crunchyroll.<PRIVATE> from cache
[crunchyroll] Extracting URL: https://www.crunchyroll.com/watch/G6P8170J6/whisper-chant-prayer-awaken
[crunchyroll] Requesting token
[crunchyroll] G6P8170J6: Downloading object info JSON (cms)
ERROR: [crunchyroll] G6P8170J6: Unable to download JSON metadata: HTTP Error 403: Forbidden (caused by <HTTPError 403: Forbidden>)
  File "yt_dlp\extractor\common.py", line 740, in extract
  File "yt_dlp\extractor\crunchyroll.py", line 389, in _real_extract
  File "yt_dlp\extractor\crunchyroll.py", line 149, in _call_api
  File "yt_dlp\extractor\crunchyroll.py", line 140, in _call_base_api
  File "yt_dlp\extractor\common.py", line 1139, in download_content
  File "yt_dlp\extractor\common.py", line 1099, in download_handle
  File "yt_dlp\extractor\common.py", line 960, in _download_webpage_handle
  File "yt_dlp\extractor\common.py", line 909, in _request_webpage

  File "yt_dlp\extractor\common.py", line 896, in _request_webpage
  File "yt_dlp\YoutubeDL.py", line 4165, in urlopen
  File "yt_dlp\networking\common.py", line 117, in send
  File "yt_dlp\networking\_helper.py", line 208, in wrapper
  File "yt_dlp\networking\common.py", line 340, in send
  File "yt_dlp\networking\_requests.py", line 365, in _send
yt_dlp.networking.exceptions.HTTPError: HTTP Error 403: Forbidden

See #10207, Crunchyroll content is completely protected by DRM now, nothing yt-dlp can do anymore. Your error comes from bot protection if I'm not mistaken, and should be circumvented by specifying a valid user agent.

The solution would be to pass --impersonate="". This extractor will not be worked on any further (and is pending removal) unless there is a non-DRM solution to #10207

(and is pending removal)

Is it being removed for sure? I would prefer the extractor not being removed, at least not until CR changes things to the point where downloading metadata also no longer works. But if it comes to it, could you notify me before it is being removed?

@Tama47 it will most likely be removed eventually. Seems like a perfect candidate for an extractor plugin


Event Request codeunit 99000773 "Calculate Prod. Order": OnBeforeCalcStartEndDate

In codeunit 99000773 "Calculate Prod. Order" please insert a new event in procedure CalculateRoutingFromActual:
    repeat
        // >>>
        OnCalculateRoutingFromActualOnBeforeCalcStartEndDate(ProdOrderRoutingLine, CalcStartEnd);
        // <<<
        if CalcStartEndDate and not ProdOrderRoutingLine."Schedule Manually" then
            if ((Direction = Direction::Forward) and (ProdOrderRoutingLine."Previous Operation No." <> '')) or
               ((Direction = Direction::Backward) and (ProdOrderRoutingLine."Next Operation No." <> ''))
            then begin
                ProdOrderRoutingLine."Starting Time" := 0T;
                ProdOrderRoutingLine."Starting Date" := 0D;
                ProdOrderRoutingLine."Ending Time" := 235959T;
                ProdOrderRoutingLine."Ending Date" := CalendarMgt.GetMaxDate;
            end;
        CalculateRoutingLine.CalculateRoutingLine(ProdOrderRoutingLine, Direction, CalcStartEndDate);
        CalcStartEndDate := true;
    until ProdOrderRoutingLine.Next = 0;

The definition of the new event should looks like:
[IntegrationEvent(false, false)]
local procedure OnCalculateRoutingFromActualOnBeforeCalcStartEndDate(var ProdOrderRoutingLine: Record "Prod. Order Routing Line"; var CalcStartEndDate: Boolean);
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Use loop-react-auth from NPM registry

By using this lib from NPM we no longer need to add
github personal access token to install the dependencies.

:page_facing_up: Description:
Provide a good description of the problem this PR is trying to address.

:pushpin: Notes:

Include TODOS, warnings, things other developers need to be aware of when merging, etc.

:heavy_check_mark: Tasks:

Write the list of things you performed to help the reviewer.

:play_or_pause_button: Demo:

Include a video or picture.

@loopstudio/react-devs

Good stuff,
Could we delete the Personal Token from our CI too?

@nicocadq Almost sure than we can remove it from the CI too, but let's wait to get this merged and give it a try locally before removing the token from the CI. 🤞


remove novalidate attr

https://github.com/sanoma/django-arctic/issues/254

Coverage remained the same at 72.056% when pulling d48a68f417fc5945641ba558443f4a972d9f77a9 on bugfix/254-remove-novalidate-attr into ea98cad9ab44f456b24b11d1a0358b44dd097c8e on develop.

@mmarcos Can you see what the impact will be in this for Arctic?

It should be fine, but that syntax is outdated, the correct one should be just “novalidate”:
https://www.wufoo.com/html5/attributes/11-novalidate.html

@mmarcos also 'novalidate' skips browser default validation: text, number, email, url type of fields,  etc... and also for required  fields. So, I suggest enable that validation. I think we need some validation before
sending form to back end.


Use dignified.js

https://github.com/dignifiedquire/dignified.js/issues/1

thank you @dignifiedquire :D


Add support for MobileLLM

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Please, reduce the numer of provided options required

Haskell has hierarchical module names hardcoded to the directory structure. At the very least so far - I did not found any counter-statement, as also not found a way to circumvent the system. So module names and directory tree are hardcoded. And are fully isomorphic.
The 2-3 times belaboring the same information defeats the half of the purpose of the program. Is there is dozens of modules to move - one needs to write a wrapper around the program to automate the module movement, and it is not easy because the program requests the information input in several forms at the same time.

It is enough information to provide one of the {MODULEPATH, --from, --from-path} and one of {--to, --to-path} not all at the same time.

There's a fork button at the top right. Use it.


Few problems with rowGrid.js

Hi!
Sorry, if this is not very right place for such report. In fact, rowGrid is embeded in CMS, which I use as the base for my website, but maintainers of CMS said that they can't help me as the problems I facing are likely to be on rowGrid side.

First of all rowGrid works great in 99.999% cases when the size of screen is large, lets say 1920px, in most browsers when there is no additional styles applied.

For example: http://artlab.club/graphic-kitchen-akaz/privetstvie-2.html
However I have an option when users apply their own branding to their blogs, meaning there is additional CSS in the page code. One of the significant changes (to my mind) is that there is
padding: 20px;
style added to the div where photoset resides.
Examples of wrong alingment:
http://artlab.club/radha/oranzhevoe-nastroenie.html

http://artlab.club/daria_miazhevich/pro-otnoshenie-k-printshopam.html

Alingment also depends on the screen scale and size, f.i. the picture above I got on all (17", 24") 1920x1080 monitors both in Chrome and Mozilla but with 100% scale. If I set screen zoom to 125% which is default in many Windows systems, it looks a little bit different (but still having this issue with alignment).

Secondly I'm still getting this issue, with tiny images but only in Chrome. I have changed this code:

function setRowGrid() {
           $('.js-topic-photoset-list').rowGrid({
               itemSelector: '.topic-photoset-item',
               minMargin: 10,
                maxMargin: 15,
               resize: false,
                lastRowClass: 'topic-photoset-last_row',
               firstItemClass: "first-item"
             });
         }

To this one:
function setRowGrid() {
           $('.js-topic-photoset-list img').load(function () {
               $('.js-topic-photoset-list').rowGrid({
                    itemSelector: '.topic-photoset-item',
                    minMargin: 10,
                    maxMargin: 15,
                    resize: false,
                    lastRowClass: 'topic-photoset-last_row',
                    firstItemClass: "first-item"
               });
             });
         }

And still getting this issue
http://artlab.club/letta-art/pervyj-mesyacz-risovaniya-v-365-itogi-i-razmyshleniya.html

http://artlab.club/radha/oranzhevoe-nastroenie.html

You need to press referesh several times to reproduce.
However it seems that this problem is now only in Chrome. In FF I wasn't able to get to same results.
You may find that rowGrid.js is not the latest release, but I already tried latest 1.0.3 version and also tried to play with different options like min/max margin, resize, etc. I have test server and I could give you the link to see how it works there. (It's almost the same)
Please help, have now idea what to try else.
Any advice would be appreciated......

Hi,
you should add this to your CSS:
.topic-photoset-item a {
  display: block;
}

I think this should fix the errors. If you still have problems don't hesitate to write again.
You should also set the resize option to true instead of false because you have a responsive design.

Hi Bruno!
Thank you very much for your reply!
I have tried to apply this style and now I'm getting these pictures:
(All on 17" 1920x1080 Mozilla FireFox with 125% zoom - it's system wide scale in Windows, so FF sets this by default when viewing web pages - many people with FullHD screens have this setting. In Chrome it's still 100% by default and it looks better).
http://<IP_ADDRESS>/testtest/post-s-kartinkami.html

http://<IP_ADDRESS>/radha/oranzhevoe-nastroenie.html

This is my test machine - I have also updated rowGrid there to latest version and set 'resize' to 'true', as you recommended.

Hi Bruno,
due to search bots activity I had to close the site for non-authorized. Please use this login pass
user1/tttest

Due to errors (and I already spent a lot of time on it) I decided to move away from rowGrid.js to CSS flex box. Thanks for assistance.


[BUG] Inconsistent keygen threshold reported data in stats dApp

After reviewing the keygen threshold changes it seems the stats dApp is either not picking up on the in-session change or reporting the next keygen threshold.
Expected Behavior
The keygen table should accurately report the current keygen threshold and update in real-time to accurately reflect any changes that occur in that session. For example, if an authority gets jailed the keygen threshold will decrease by 1, we need to pick up on both the change in authority set and keygen threshold change.
Current Behavior
Currently it seems we are writing the keygen threshold and authority count once and presuming it as accurate throughout the session.
Possible Solution

We will need to subscribe to event changes to pick up on in-session changes and update the db accordingly. This work will need to be done in webb-subql repo.
Steps to Reproduce

Go to Polkadot explorer archive node
Query for a block where a session change took place (e.g. 6603)
Copy block hash
Navigate to Developer > Chain state and select dkg and keygenThreshold() , and the copied block hash from above
See the accurate keygen threshold reported and compare to https://www.stats-dev.webb.tools/#/keys
Repeat same flow for current / best authorities

Closed in #1154


OTel global tracer ID needs to be updated

Originally posted by @MrAlias in https://github.com/open-telemetry/opentelemetry-go-instrumentation/issues/1275#issuecomment-2481389387
The tracer is identified with the scope attributes now:
https://github.com/open-telemetry/opentelemetry-go/pull/5924/files#diff-d14dbf935696bfe625499cf63b0a958a997c8f2cd8392e859e865ddf8de82675R106-R111
This means we are no longer able to correctly parse/process the tracer ID in our probes: https://github.com/open-telemetry/opentelemetry-go-instrumentation/actions/runs/11880399901/job/33103525079?pr=1275
Proposals

move forward with deprecating this in favor of the auto SDK support.
update our map parsing to support this, including adding backwards compatibility.

SIG meeting notes:
We can update the existing probe with the additional size requirement of the added attribute.Set. This will correctly parse what we already parse (name, version, schema_url), but not add attribute support. This is preferred until we can move entirely to the auto package.


Adjust y-axis on mask data to better fit data spread

Actual Behavior:
Mask data in charts is displayed on a 0% to 100% scale, squishing the data down to a fraction of the available area and thus making it hard to read
When I...
View mask data in the COVIDcast visualization, top 10, possibly more

go to https://covidcast.cmu.edu/?sensor=fb-survey-smoothed_wearing_mask&level=county&date=20201102&signalType=value&encoding=color&mode=top10&region=55087
use eyes to view masks signal

Expected behavior
Mask data would be displayed on a more appropriate scale. I would suggest going from 50 or 60% to 100% at least, or if we want to get fancy consider doing, say, 2-3x the difference from highest to lowest value in the current dataset (so if the spread is 3%, show 6% or 9%). This could be tricky since we could make small differences seem large if we make the scale too small.
Screenshots

Rating Scale (1 is minor, 7 is severe):
2

Context

Browser: Chrome

Additional context
n/a

Interesting suggestion.
We do already adjust the max of the y-axis to avoid leaving empty space above the data.  We are explicitly using 0 for the min of the y-axis, though it does have this effect of compressing the data to the top, if it is all close to the max.
The top-10 charts use to clip the data on the top, but now, we avoid any clipping on the high side by iterating over the data for every location to find the maximum across the entire timespan.  But I am making a change to limit the timespan to a smaller subset, so that the time around the currently selected date is less compressed.   Showing the clipped region clearly is part of the puzzle.  See #507 and #589 for discussion on this.
So this suggestion means we are doing something similar on the low side, but doing this raises the question whether we should similarly clip lines on the low end, if they are outside of the neighborhood of the current date.

from a visualization point of view it is highly encouraged to start a value axis with 0.  If we wanna go in this direction we might not show the y-axis (the axis and its ticks) at all and just show it as a sparkline.
re value range: for coloring the map there is already a similar thing is place. the start/end values are mean +- 3 * stdev:

There's also the option of some kind of rescaling. The New York Times map over the summer plotted the probability that, in five different encounters, everyone would be wearing a mask; that would exaggerate differences and pull a lot of the numbers down closer to 0. Maybe we could do a similar rescaling, like "the probability that 5 people would wear masks most or all of the time" (signal^5).

Regarding skewed perceptions, a similar skewing happens if percentage values are all small, say less than 5 percent, but we don't show the full axis up to 100%.
But I think that if expectations are that a set of values ought to be all low (or all high) then it is reasonable to be able to zoom in to the area of interest and not show the full axis.  But I think it is important to somehow emphasize that the axis is truncated.

Let's close this for now.  We'll continue to address the y-axis scale issue in the redesign.  There are a few proposals that give us quality improvements.


[BUG] Diagnostic ResourceAlreadyImported Not Reported Since v0.94

Describe the bug
Since v0.94, aka this works with v0.93.1, robotcode stopped reporting ResourceAlreadyImported.  I have tried a few of the diagnostics errors here
https://github.com/robotcodedev/robotcode/blob/6022e917124d66c5522f72930ec97d14459baea9/packages/robot/src/robotcode/robot/diagnostics/errors.py#L7-L42
and they appeared to be ok, but I have not done an exhaustive test
Steps To Reproduce
Steps to reproduce the behavior:

Import the same resource file twice in a row
Observe no ResourceAlreadyImported
Also, when a duplicate resource exists, the control/command + click to go to the definition does not work.  For non duplicate resource files, this works.

I'll continue to try to debug, but this appears to be a regression
Desktop (please complete the following information):

VS Code Version - 1.95.3
RobotCode Version - v0.94 and up
OS: both Win 11 and macOS Sequoia 15.1
Python Version - 3.12.7
RobotFramework Version - 7.1.1
robocop - 5.4.0
robotidy - 4.14.0

Thank you @d-biehl!


Added a file to Project 2

Followed the tutorial and added the speech file.

Okay I'll give it a go. @Sammybams


AvailabilitySet failed if resource group has a long name

Error:
Resource group name: haiche-long-long-resource-group-cluster-validate-fix-b78dd658-8ca9-4b2c-b710-12d13c9b3fbe
{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"InvalidParameter","message":"The entity name 'resourceGroupName' is invalid according to its validation rule: ^[^\W][\w-.]{0,79}(?<![-.])$."}]}

Also failed deploying from Azure Portal.

@edburns  I think we can mark this as known issue.


create jettopythontools backend for duqtools

Codecov Report

Merging #276 (03efb84) into main (124ba29) will decrease coverage by 0.82%.
The diff coverage is 7.89%.

@@            Coverage Diff             @@
##             main     #276      +/-   ##
==========================================
- Coverage   49.55%   48.72%   -0.83%
==========================================
  Files          54       56       +2
  Lines        1891     1925      +34
  Branches      261      263       +2
==========================================
+ Hits          937      938       +1
- Misses        920      953      +33
  Partials       34       34

Impacted Files
Coverage Δ

src/duqtools/init.py
0.00% <ø> (ø)

src/duqtools/jettopythontools/__init__.py
0.00% <0.00%> (ø)

src/duqtools/jettopythontools/_system.py
0.00% <0.00%> (ø)

src/duqtools/system.py
0.00% <0.00%> (ø)

src/duqtools/cli.py
46.66% <20.00%> (-1.59%)
:arrow_down:

src/duqtools/schema/cli.py
93.15% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Fix markdown syntax-highlighting at javascript-utilities doc page.

Today we have a markdown syntax-hightlighting problem at javascript-utilities.

I've just closed the tag.

Thanks!