Embedded try-catch clause erases exception class

The following test passes on OTP but fails on AtomVM:
test_exit() ->
    try
        try_exit(),
        fail
    catch
        exit:foo ->
            ok
    end.

try_exit() ->
    try
        exit(foo)
    after
        ok
    end.

with the error:
CRASH xception:
======
pid: <0.1.0>

Stacktrace:
[{test_exception,test_exit,0,[{file,"/.../AtomVM/tests/erlang_tests/test_exception.erl"},{line,32}]},{test_exception,start,0,[{file,"/.../AtomVM/tests/erlang_tests/test_exception.erl"},{line,26}]}]

cp: #CP<module: 0, label: 4, offset: 13>

x[0]: error
x[1]: foo
x[2]: {2,2,85,1,[{0,18},{0,65}]}

Stack
------

[]
#CP<module: 0, label: 2, offset: 10>
#CP<module: 0, label: 14, offset: 0>

Registers
----------
x[0]: error
x[1]: foo
x[2]: {2,2,85,1,[{0,18},{0,65}]}
x[3]: []
x[4]: []
x[5]: []
x[6]: []
x[7]: []
x[8]: []
x[9]: []
x[10]: []
x[11]: []
x[12]: []
x[13]: []
x[14]: []
x[15]: []

The OTP-24 generated code is as follows:
{function, try_exit, 0, 8}.
  {label,7}.
    {line,[{location,"../tests/erlang_tests/test_exception.erl",39}]}.
    {func_info,{atom,test_exception},{atom,try_exit},0}.
  {label,8}.
    {allocate,1,0}.
    {'try',{y,0},{f,9}}.
    {move,{atom,foo},{x,0}}.
    {line,[{location,"../tests/erlang_tests/test_exception.erl",41}]}.
    {call_ext,1,{extfunc,erlang,exit,1}}.
  {label,9}.
    {try_case,{y,0}}.
    {move,{x,1},{x,0}}.
    {move,{x,2},{x,1}}.
    {bif,raise,{f,0},[{x,1},{x,0}],{x,0}}.

The issue seems to boil down to our implementation of OP_TRY_CASE, which clears y[0], and the RIASE bif, which always raises an error.
    case OP_TRY_CASE: {
                int next_off = 1;
                dreg_t dreg;
                dreg_type_t dreg_type;
                DECODE_DEST_REGISTER(dreg, dreg_type, code, i, next_off);

                TRACE("try_case/1, reg=%c%i\n", T_DEST_REG(dreg_type, dreg));

                #ifdef IMPL_EXECUTE_LOOP
                    // clears the catch value on stack
                    WRITE_REGISTER(dreg_type, dreg, term_nil());
                #endif

                NEXT_INSTRUCTION(next_off);
                break;
            }

    case OP_RAISE: {
                int next_off = 1;
                term stacktrace;
                DECODE_COMPACT_TERM(stacktrace, code, i, next_off);
                UNUSED(stacktrace);
                term exc_value;
                DECODE_COMPACT_TERM(exc_value, code, i, next_off);

                #ifdef IMPL_CODE_LOADER
                    TRACE("raise/2\n");
                    UNUSED(exc_value);
                #endif

                #ifdef IMPL_EXECUTE_LOOP
                    TRACE("raise/2 stacktrace=0x%lx exc_value=0x%lx\n", stacktrace, exc_value);

                    RAISE_ERROR(exc_value);
                #endif

                NEXT_INSTRUCTION(next_off);
                break;
            }

Comment:
// clears the catch value on stack

is misleading. y[0] is not the exception class here but the catch label written by OP_TRY. Likewise we should remove the TODO in OP_TRY.
The bug indeed is that OP_RAISE does always set the class to error. We need to remember what the class is when exit/1, throw/1 or raise/3 are called.
Because of after we probably cannot use the latest class as the following pseudo-code test should work:
ok = try
    try
        raise(error, reason, [])
    after
        try
            raise(throw, reason, [])
        catch
            throw:reason -> ok
        end
    end
catch
    error:reason -> ok;
    throw:reason -> fail
end


Change submodule paths.

The<EMAIL_ADDRESS>URLs don't work for external contributors who do not have access to non-https URLs.  This change fixes both git clone --recurse-submodules of the initial repo and the install.sh script, which attempts to fully check out sparse clones.

what about also changing the path of the main repo to git@github.com:princeton-vl/infinigen.git because procgen seem to be private or something
https://github.com/princeton-vl/infinigen/blob/main/docker/setup.sh#L1


feat: Add full ESM support via an import export condition

We currently support ESM partially. This PR investigates switching to bunchee to enable full ESM while supporting:

'use client' directives split bundles
Separate development/production build for ESM to avoid process.env.NODE_ENV checks
CJS/ESM dual build

TODO

[ ] Wait for bunchee bugs

[ ] https://github.com/huozhi/bunchee/issues/575
[ ] https://github.com/huozhi/bunchee/issues/540

[ ] Ensure Intl constructors are still shared (and bundle size doesn't go up)
[ ] Make sure entries share code as expected
[ ] Update use-intl example to remove vite override
[ ] Ensure prod condition is applied (webpack)
[ ] Ensure dev condition is applied (webpack + turbo)
[ ] Try plugin cjs/esm
[ ] Add examples (or matrix config) to verify next.js 13/12
[ ] Test other tools: jest, vitest, … smth else?

Release docs

Might have to update vitest to latest in case dev condition is not applied

Fixes https://github.com/amannn/next-intl/issues/1153

Closed in favor of https://github.com/amannn/next-intl/pull/1470


Exception in Datatables.php line 52: No available engine for Illuminate\Database\Eloquent\Collection

Problem when getting data
I have updated laravel-datatables to 7.0 and I'm using laravel 5.4. The problem is i cant use my Models Directly and i have to use DB to return the query, If i use Model::ModelName, I get this error:
Exception in Datatables.php line 52: No available engine for Illuminate\Database\Eloquent\Collection
My Controller:
$var = ModelName::all();
return Datatables::of($var)
  ->make(true);

My View
var dt = '';
     $(document).ready(function () {

         dt = $('#table').DataTable({
             processing: true,
             serverSide: true,
             ajax: {
                 url: '{!! route('route') !!}',
                 type: 'POST'
             },
             columns: [
                 {data: 'data'},
             ]
         });
     });

I'm sure the Model works. I can dd the response but Whenever i use Model I get error and If i use DB like:
$var = DB::table('table')->select('*')->whereNull('deleted_at');
Everything works!

Operating System: Windows 8.1
PHP Version: 5.6.30
Laravel Version: 5.4
Laravel-Datatables 7.0

Please update to the latest version or at least ^7.1. Thanks!

And you may need to force publish the config.
php artisan vendor:publish --tag=datatables --force


Fixes a number of tests.

Fixes multiple overlapping outlining spans.
Fixes cancelled custom commands not throwing.
Fixes Project.AddExistingFolder missing.

:+1:


Authenticate recurrent users with memo.

Closes: #237.
Context and Preliminaries.
This PR adds a way to authenticate users by means of a signature using their wallet (commonly called: Sign in with web3, SIWE). The signer-service will use this signature in a JWT fashion to authorize the signing of the sep-10 operations from the signer-service.
The motivation for these changes is to add a scheme where the application is able to authenticate the users by it's initial Polygon/EVM address and avoid re-do of KYC and storing extra credentials.
Detailed overview
To identify the users based on their authenticated address, the sep-10 operations now correspond to the specification of a memo that is derived uniquely and in a deterministic way only from the EVM address.
The user identifier is then the combination of a master account concatenated with the memo (master:memo) and must be seen by the Anchor as a completely different user than one authenticated with only the master account. (relevant section of the protocol)
Only with a valid signature corresponding to the EVM address in question will the signer-service respond with the corresponding signatures, which verifies: memo, client-domain and master account.
Security guarantees
Without these signatures from the service, the sep-10 will fail when requested to the Anchor. Although it is not stated on the protocol (or I missed it) it is expected that the Anchor would ONLY allow a memo specification when the client-domain is defined, and valid. Apparently, anchors could also whitelist the master account for this purposes.
Flow of operations

The user opens Vortex.
(optional) Upon connect/change of address it is presented with a modal asking to sign a message.
The user attempts an offramp. If needed by Anchor, we search for the user's signature in the storage, check it's expiration time, and potentially ask the user to sign the message again if it doesn't exist or if it's expired. (we don't ask for validation to the service here, to save requests).
The user now sends a request to the signer-service with this signature, and the corresponding sep-10 parameters. The service returns the corresponding stellar signatures.
If the user's signature turns out to be invalid (eg.: the service has lost it's state), then a new sign-in operation will be possible without interrupting the flow.

Session
So far, the concept of session is simply kept by means of the signature stored in the UI (local storage...). Again, drawing parallels to a JWT. The message signed has a date parameter checked by the signer-service.
Alternatively, if the user signs from another browser/machine etc, then the old signature will be invalidated on the signer-service.
Requirements for log-out have not been defined yet.
Security risks, and potential improvements.
This is as far as I would go with this simple/prototypical implementation of using the signature as means of authentication.
Update: We switched to use secure cookies now, instead of storing the signature in the local storage. Should be much more secure.
There is a potential risk since this will be stored in the localStorage, unlike secure cookies. Therefore, if we are going to aim for a comprehensive session management, I believe we should implement something like iron-session to keep the session, while the initiation would still be by means of this signature verification. This seems to be the common standard practice.
Extra libraries concerns.
This pr adds the library siwe which is useful for creating and parsing the message, and potentially verifying. This is useful yet not critical, and the issue is that it requires ethers dependency. So to be more compatible with our current libraries and have the freedom to remove this one, we use viem to verify on the backend side.

@pendulum-chain/devs You can try this one locally, including Anclap until the offramp UI (or use the "testing" credentials to interact)
The signer-service must be ran locally.

@gianfra-t as I don't have ARS, I tried to offramp EUR. When I click on the Confirm button, I am presented the sign in popup.
When I click on "Sign Message", nothing happens, the popup just disappears and the form is shown again. If I click on the Confirm button again, the same happens – indefinitely.
Is it even intended that the popup is shown in the first place? The identity verification via signature is not required for Mykobo.

If I click on the Confirm button again, the same happens – indefinitely.

This behavior is not intended definitely. What the console shows? We need some more error handling from the UI side, perhaps a toast, I left that open on purpose since there was also no agreement on that.
Nevertheless, for me there is no issue when selecting EURC and it works the same as before. So I imagine there must be a variable missing when you run signer-service.
Also we need to add a condition to NOT ask for signature with EURC probably (this, I forgot), although I will not add it now until I understand what happens in your case.

Yes, sorry, should have provided that:

I guess the signer service of the preview deployment does not have the new changes yet and I would need to run locally instead.

Yes you need to run it locally. Sadly we don't deploy a service preview so far.

I didn't review the entire code yet, but when I tested locally I found the following.

Why create the siwe message in the backend? I would not expect that we need this new endpoint and we can simply do it as in this example.

Metamask shows the following warning when I want to sign the signing request. Is this only happening because I run on localhost and it would disappear once deployed in the cloud (I tried to change the parameters for siwe.SiweMessage but could not make the warning disappear)? It would be good if this works for any domain, particularly also for the netlify preview domains.

I need to sign in even when I just select EUR, can we avoid signing in if we will not use the memo for the respective anchor (i.e., if usesMemo: false). Let's avoid any conversion rate hurdles.

When I select EUR, then click on "Confirm" so that the button shows "Continue with Partner", then select ARS, then the button still shows "Continue with Partner" and I will start the Mykobo anchor flow. We should reset when switching the output currency.

Nice observations @TorstenStueber:

I would not expect that we need this new endpoint

It is true that the message can be created either in the backend, or in the frontend. But even when created on the fronted, we need an endpoint that sends a random nonce or a way to derive the nonce such that we are able to verify it on the service, which seems more complex. The nonce is to prevent replay attacks, more info here.
Now it is true that the message could be generated on the frontend and then validate the values on the backend. See this example from wagmi. Still, the nonce is fetched. So I thought it was better to abstract the message creation to the backend already. I don't see much benefits/cons on either way.

Metamask shows the following warning

Yes, I thought of using vortex main domain, but I will try to find a way to remove the warning for the staging and previews.

I need to sign in even when I just select EUR

We were discussing it here, if we add a signature when the app loads/address changes or just when clicking confirm and the asset requires it. So far it's enabled to test how it "feels".

When I select EUR, then click on "Confirm"

I didn't think about this case! Will add a reset.

As long as the user is not malicious, they will protected from replay attacks if we create the nonce in the frontend. For malicious users – as you point out – they can just create the message anyway and not use the backend.
If there are equal solutions, I would always pick the simpler one – in this case this means to handle this completely in the frontend. Also: this would make it easier to just use the current domain for the message so that Metamask does not show the warning.

We were discussing it https://github.com/pendulum-chain/vortex/pull/257#discussion_r1838700337,

This is a user facing decision and we should escalate to @pendulum-chain/product. I will move the discussion elsewhere.

@pendulum-chain/devs on the last additions we switched to secure cookies from the local storage, to avoid leaking the signature.

I think the main comments are all addressed except for the part about having the Siwe creation on frontend vs backend. @gianfra-t are you still testing this or do you have hard feelings and would insist on keeping it the way it is right now?

@ebma I'm on it since this morning! Took a few more changes, but is almost there.

@ebma I tested another offramp and it works. I will merge this and then someone could test an EURC offramp, I can only tell that the anchor's window is opened.

@ebma I can do the EUR off ramp today using staging.


[Question] Using waifu2x_win_koroshell with the latest version of waifu2x-converter-cpp

So this maybe a dump question.

So I was using waifu2x_win_koroshell download fron here like the instruction. That version is supposed to be very old if I am not mistaken? So I downloaded the latest release of you, and overwrite every files. Is it suppose to be the correct way to no this? I hope not.
I don't know how to implement the latest release to that waifu2x_win_koroshell.
Also, I am fine with using command line, but I don't know how to convert multiple images, maybe a folder of images with the command. Like what is the syntax for converting an entire folder?
Finally, the last release was from Dec last year. Any release coming soon with OpenCV 3.3 (not sure if it's important anyway)?

@ffleader1

Nothing to be overwritten, download the release of my fork and use no other binary, my fork only has command line, there is a GUI inside but I never touched it, not sure if it works, I wouldn't count on it.

Scaling multiple images would work like waifu2x-converter-cpp.exe -i test_folder_of_images --scale_ratio 4 -o test_folder_scaled but that does not yet work, I will compile once @iame6162013 checks 1 more thing

see above

@ffleader1 try: https://github.com/DeadSix27/waifu2x-converter-cpp/releases/tag/5.0


Re-arrange message modeling, add Stream

Re-arrange message modeling, add Stream
docs

Current dependencies on/for this PR:

main

PR #20   👈

PR #21

This stack of pull requests is managed by Graphite.


polars pyo3_runtime.PanicException makes notebook unresponsive

Describe the bug
I create a new notebook, and paste the following into the first cell, and execute the code. The notebook is now unresponsive. As in when I enter 4 in the next cell, and hit shift-enter no output appears showing the result 4.
In all other cases of exceptions, the notebook remains responsive, so I expect it to do so for this exception.
Also I ran the same code in an ipython REPL. I get the same exception, but the REPL remains responsive. This leads me to believe the bug lies within Marimo rather than polars.
Environment
{
"marimo": "0.6.25",
"OS": "Windows",
"OS Version": "10",
"Processor": "Intel64 Family 6 Model 85 Stepping 4, GenuineIntel",
"Python Version": "3.10.11",
"Binaries": {
"Browser": "126.0.6478.127",
"Node": "--"
},
"Requirements": {
"click": "8.1.7",
"importlib-resources": "missing",
"jedi": "0.19.0",
"markdown": "3.6",
"pymdown-extensions": "10.8.1",
"pygments": "2.16.1",
"tomlkit": "0.12.5",
"uvicorn": "0.30.1",
"starlette": "0.37.2",
"websockets": "12.0",
"typing-extensions": "4.7.1",
"black": "23.7.0"
}
}
Code to reproduce
import polars as pl
import numpy as np
d = pl.DataFrame({"a": [np.arange(5) + i for i in range(5)]})
res = d.select(
    pl.col("a").map_batches(
        lambda x: {"filt_value": np.dot(x, x), "filt_phase": 5.0},
        is_elementwise=True),
    )

thanks for sharing this issue. this looks like a bug in marimo trying to stringify the variable for the variable explorerer

@ggggggggg just cut version 0.6.26, which contains a fix. Should be on PyPI momentarily.


fix(settings): allow showBots only when detailedNotifications is selected

Also make it null safe for notifications that don't have users (ie: workflow runs)

Seems logical I suppose!

now with added tests :)


Different number of Messages after sorting Latest first/Newest first

Describe the bug
(A clear and concise description of what the bug is.)
After clicking the button Latest first/Newest first on the tab 'Messages' the different number of messages is displayed.
Set up
(How do you run the app?
Which version of the app are you running? Provide either docker image version or check commit hash at the top left corner. We won't be able to help you without this information.)
http://redesign.internal.kafka-ui.provectus.io/
Steps to Reproduce
Steps to reproduce the behavior:

Navigate to the menu item Topics
Choose some topic with a number of messages of more than 2
Click the button 'Latest first/Newest first' choosing the order of sorting
Compare the written number under the button for both options of sorting.
Compare the written number to the number of messages on the page after sorting.

Expected behavior
(A clear and concise description of what you expected to happen)

The written number under the button is the same before and after sortings.
The written number equals the number of messages on the page after sorting.

Screenshots
(If applicable, add screenshots to help explain your problem)

Additional context
(Add any other context about the problem here)
This bug is floating, sometimes the number is really different like for this topic http://redesign.internal.kafka-ui.provectus.io/ui/clusters/local/topics/_schemas/messages?attempt=1&limit=100&seekDirection=BACKWARD&seekType=OFFSET&seekTo=0::57
Sometimes only the written number is different but the amount of messages is right and equal like here http://redesign.internal.kafka-ui.provectus.io/ui/clusters/local/topics/dev-cp-kafka-connect-status/messages?attempt=2&limit=100&seekDirection=FORWARD&seekType=OFFSET&seekTo=0::0,1::0,2::0,3::0,4::0
Sometimes it works correctly like here http://redesign.internal.kafka-ui.provectus.io/ui/clusters/local/topics/Test_Part6/messages?attempt=1&limit=100&seekDirection=BACKWARD&seekType=OFFSET&seekTo=0::1,1::2,2::2,3::3,4::2,5::2
The same behaviour was before the redesign.

@agolosen it is a expected behaviour since 'messages' it not a number of all messages in topic, but a number of scanned messages needed to form result page, which can different for different sorting options. It can be be more visible if you fill Seach field. The same is for size & time near messages count.
Maybe we should explicitly rename 'messages' to 'scanned messages' to get rid of such confusion.


Change PropTypes import to use prop-types instead of react. Fixes #538

Thanks for submitting a pull request to RGL!
Please reference an open issue. If one has not been created, please create one along with a failing
example or test case.
Please do not commit built files (/dist) to pull requests. They are built only at release.
HEY - Full disclaimer, I did not test this! :)
I just found all the places I could find where PropTypes was being imported from react and changed it to use prop-types, and added the dependency of course.

Thanks. There are two other missing references: WidthProvider and ResponsiveReactGridLayout are using React.PropTypes.

Thanks!

0.14.5.


config with environment variables

I would like to suggest doing configuration with environment variables instead of in source code. It makes it easier to build containers and during deployment set configuration with e.g. config maps and sealed secrets on k8s. Furthermore, in that way it becomes easier to build a container that does not contain anything secret.
For local development you can also use dotenv so you can have a file locally for the configuration. Preferably a template for it in the code repo
When the content is large, you can enter a path in the config environment variable and then you can mount a file with the content on that path.
All so as not to accidentally insert secrets or customizations into the container image.

+1


⚠ Bofrost.be has degraded performance

In f3adf1c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1235 ms

Resolved: Bofrost.be performance has improved in 4965c9d after 15 minutes.


No mention of VPC or subnet config needed

On https://eksworkshop.com/prerequisites/workspace/ (source: master/content/prerequisites/workspace.md)  the instructions say

Select Create environment
Name it eksworkshop, and take all other defaults

However, brand new users will find there is no VPC to select, so one needs to be created. A nudge to help users here would be welcome, because VPC creation has lots of options - can they all just be left as defaults, too?

When I got to the page, the VPC and subnet drop-downs were in a red error state and no option was available for me to select, so I clicked the button to create a VPC and did so in a new tab, then refreshed the drop-down and it auto-populated with the just-created VPC.
Hope that helps.
(I wish I'd screenshotted)

This may be related: note how the demo looks like it's using a public IP

but when using the just-created-with-default-values VPC, I'm on a private IP

172.31 is private also, and tends to be the range used in default vpcs. Can
you confirm whether or not you have a default vpc in your region?
On Tue, Sep 10, 2019, 9:12 AM Steve Jalim<EMAIL_ADDRESS>wrote:

This may be related: note how the demo looks like it's using a public IP
[image: Screenshot 2019-09-10 at 15 10 20]
https://user-images.githubusercontent.com/101457/64621301-5b081400-d3dd-11e9-8eb8-b07b90086719.png
but when using the just-created-with-default-values VPC, I'm on a private
IP
[image: Screenshot 2019-09-10 at 15 10 32]
https://user-images.githubusercontent.com/101457/64621287-50e61580-d3dd-11e9-8ee0-67524250257d.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/eks-workshop/issues/442?email_source=notifications&email_token=AAB3SEEEQVHXAFRMB3GKYB3QI6TMRA5CNFSM4IVH6ITKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6LHNBY#issuecomment-529954439,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB3SEFDJ363L3U5B5FG3Z3QI6TMRANCNFSM4IVH6ITA
.

Apols re missing the private IP.
The only VPC I have is the eksworkshop one I just created and this is now classed as default.

It sounds like you're working in a non standard account... Perhaps it's
corporate issued, and they strip out the default vpc?
On Tue, Sep 10, 2019, 9:52 AM Steve Jalim<EMAIL_ADDRESS>wrote:

Apols re missing the private IP.
The only VPC I have is the eksworkshop one I just created and this is now
classed as default.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/eks-workshop/issues/442?email_source=notifications&email_token=AAB3SEFI2XMWUDRVYYUYDALQI6YBDA5CNFSM4IVH6ITKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6LL7HY#issuecomment-529973151,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB3SEGSIDUMMDBJCOZ2A4TQI6YBDANCNFSM4IVH6ITA
.

It's certainly an IAM user under an organisation, rather than a 'personal' AWS account, so that might well be it.

Yeah. That makes sense.
On Tue, Sep 10, 2019, 10:10 AM Steve Jalim<EMAIL_ADDRESS>wrote:

It's certainly an IAM user under an organisation, rather than a 'personal'
AWS account, so that might well be it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/eks-workshop/issues/442?email_source=notifications&email_token=AAB3SEDGT7SBJPGAOLY5U73QI62FBA5CNFSM4IVH6ITKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6LOF3Q#issuecomment-529982190,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB3SEHELRZLZXSUJE6S4FTQI62FBANCNFSM4IVH6ITA
.

Thanks, by the way, for the workshop - it's great

Awesome! Let me know if you run into any other issues.
On Tue, Sep 10, 2019, 10:46 AM Steve Jalim<EMAIL_ADDRESS>wrote:

Thanks, by the way, for the workshop - it's great
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/eks-workshop/issues/442?email_source=notifications&email_token=AAB3SECJ6XIZOWBLUUOWJQLQI66MBA5CNFSM4IVH6ITKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6LSJUY#issuecomment-529999059,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB3SEFKDRRA5VUBO5C7IFLQI66MBANCNFSM4IVH6ITA
.


ci: Remove merge queue workflow

Reason for Change:

This workflow was used as a workaround to trigger multiple merge queue runs. Is no longer required due to #2681
Issue Fixed:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[x] relevant PR labels added

Notes:

Merge queue GH workflow is no longer needed. This can be merged

/azp run Azure Container Networking PR


Another proposal for single-thread environments

class NullableStringHolder {
    public var s : Null<String>;

    public function new(s : Null<String>) {
        this.s = s;
    }
}

class NotABugButProposal10 {
    public function proposal() : Void {
        var h = new NullableStringHolder((Math.random() > 0.5) ? "A" : null);

        if (h.s != null) {
            // Safety: Cannot access "length" of a nullable value.
            // While it makes sense in case of multithreading, is it possible to add something like `@:safety(threadsafe)`?
            h.s.length;
        }
    }
}

See https://github.com/restorer/haxe-safety-bugs/blob/master/safetybugs/Main.hx#L140 for working example.

This is for Haxe 4.0.0-preview.5 and Safety from master.

It also doesn't make sense for a single-threaded environment if there are some calls before the access to h.s.length.
I'm not sure if it worth to change.

Hmm. Something like
if (h.s != null) {
    h.s = null;
    h.s.length;
}

Make sense. Kotlin do the same thing: Smart cast to 'String' is impossible, because 'h.s' is a mutable property that could have been changed by this time.
New proposal: maybe more descriptive error message (like in kotlin)?


Missing APR library

tomcat:8.0 starts with
26-Aug-2015 15:00:37.973 INFO [main] org.apache.catalina.core.AprLifecycleListener.lifecycleEvent The APR based Apache Tomcat Native library which allows optimal performance in production environments was not found on the java.library.path: /usr/java/packages/lib/amd64:/usr/lib/x86_64-linux-gnu/jni:/lib/x86_64-linux-gnu:/usr/lib/x86_64-linux-gnu:/usr/lib/jni:/lib:/usr/lib

There is another issue that already deals with installing native libraries to support optimal Tomcat performance and other features, though it deals more with libtcnative and not libapr1: https://github.com/docker-library/tomcat/issues/7
I think it would be best to consolidate the discussion there.

Closing in favor of #7


Update Microsoft.Azure.Management.LabServices.Tests to SDK Type

Update LabService.Test project to SDK type since the whole solution not working now.
Replace
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Microsoft.Azure.Management.LabServices.Tests", "tests\Microsoft.Azure.Management.LabServices.Tests.csproj", "{DC1024AF-4E20-4E65-9ED8-09CC26385B8B}"
To
Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "Microsoft.Azure.Management.LabServices.Tests", "tests\Microsoft.Azure.Management.LabServices.Tests.csproj", "{DC1024AF-4E20-4E65-9ED8-09CC26385B8B}"
At line 6 in Microsoft.Azure.Management.LabServices.sln file.

https://github.com/Azure/azure-sdk-for-net/issues/9873


medmc_qa dataset does not contain generated CoTs

Looking at both the dataset viewer and a local dump I generated, it seems like the medmc_qa dataset does not contain generated CoTs?

Yes, I noticed that too recently, unfortunately this slipped through somehow. This is fixed in the PR for the new CoTs.


Populate conformance suite with "interesting" test cases

Our current methodology for testing the reference implementation involves writing a minimal amount of tests (https://github.com/openxla/stablehlo/blob/main/docs/reference_checklist.md), but for a conformance suite we'll likely want to write "interesting" tests that cover potentially surprising behavior, even if strictly speaking these tests don't increase coverage of the codebase. Let's think about how this could look like.

Covered by testdata which will evolve on an as-needed basis (i.e. add dynamism tests and quantized tests).


 Theres nothing we can do to fix it, though > you'll get much stronger welds if you freeze your props before duping them"
ya but advdupe2 can do it
"grumbles and leaves"

-- Me, versus the unwashed masses
Okay, so advdupe2's solution was to hook into OnEntityCreated, and save the local positions of all physics objects involved in the constraint into the constraint itself. When pasting, it simply moves the physics objects to their original locations when the constraint was first made, pastes the constraint, then moves them back. It may seem a bit hacky, and its not 100% effective in all cases, but its a whole lot better than what we do now with regards to constraints.
I don't know if TB can copyright the basic idea of the above paragraph, as obviously copypasting his exact code isn't applicable, but we need to do something like it.
Advdupe2:
Save initial constraint positions line 18
Hook into OnEntityCreated to call the above function line 72
Inside PasteConstraint, before running the factory function, move ents into place line 519
Inside PasteConstraint, after running factory function line 569

Just for further research in bettering this issue. I didn't have any benefit with the ropes, but with out replicating positions when it was constrained, it will still degrade. If you constrained the 10 props, froze 1 end and curved the other end around and froze it, when you duplicate it, it will be replicated very distorted. Since AD2 saves positions when constrained, it will not be distorted after replication. So, without saving positions if you try and lock and freeze your 100 prop contraption it may move slightly over time of copying and wouldn't perform the same; and also have to do the hassle of locking and freezing. It could probably go a step further and try and spawn the entities and constraints in in the order they were spawned into the game to try and get closer to matching their stepping calculations in the physics engine from the original creation.

Yea, if constraints are ordered randomly then the contraption will spawn with random constraint systems and won't be a single solid constraint system. Advdupe2 orders them.


Workflow "Create a  new app" fails

https://github.com/achim-t/shiny-octo-computing-machine/actions/runs/2263871095
I was following scenario 1.
git push ***github.com/achim-t/shiny-octo-computing-machine
remote: Permission to achim-t/shiny-octo-computing-machine.git denied to github-actions[bot].
fatal: unable to access 'https://github.com/achim-t/shiny-octo-computing-machine/': The requested URL returned error: 403
Error: Adding a new app failed.

This issue is caused by a change in GitHub defaults.
It is fixed in the preview of AL-Go - but for now, you can go to:
https://github.com/achim-t/shiny-octo-computing-machine/settings/actions
and set default workflow permissions to read and write:

In the next version, permissions are specified in the workflows and this setting is no longer needed.

Yes, that worked.

A fix for this is shipped in v1.5


Incomplete profile property list

The list of the profile properties isn't complete.
There are missing

Description
Caption
RoleCenter
Customizations

The ML Properties are missing too, but since xliff is the future, that's okay for me.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 20446c43-f374-740f-2539-08af50441519
Version Independent ID: 642859b6-87ca-8b02-ee31-edc84a5d3f49
Content: Profile Properties - Business Central
Content Source: dev-itpro/developer/properties/devenv-profile-properties.md
Service: dynamics365-business-central
GitHub Login: @solsen
Microsoft Alias: solsen

Thanks for your feedback - this topic will be updated with the next doc refresh.


DBeaver crashes due to  problem with script "not found" in the tree

Description
Hello, I am not sure this is a bug, but we have an issue that prevents DBeaver to open and I did not find aby information about the error encountered.
DBeaver crashes on Opening with the message
org.eclipse.core.internal.dtree.ObjectNotFoundException: Tree element '/General/Scripts/Script-25.sql' not found.
The header of log file is
eclipse.buildId=unknown
java.version=17.0.6
java.vendor=Eclipse Adoptium
BootLoader constants: OS=win32, ARCH=x86_64, WS=win32, NL=en
Command-line arguments:  -os win32 -ws win32 -arch x86_64
!ENTRY org.eclipse.core.resources 2 10035 2023-12-04 16:37:38.665
!MESSAGE The workspace exited with unsaved changes in the previous session; refreshing workspace to recover changes.
The file mentioned here (/General/Scripts/Script-25.sql' ) exists in the user directory, it was empty. We added a line inside, and tried again to launch DBeaver, but the problem remains.
Pls find log attached.  Thank you for any help, and sorry if this is not the right place to report
Isa
Log file<
DBeaver-ObjectNotFound.log
DBeaver Version
Community edition Version <IP_ADDRESS>306251800
Operating System
Windows Server 2019 datacenter
Database and driver
Vertica DB 12.0.4 , vertica jdbc driver
Steps to reproduce
No response
Additional context
No response

Hi @ivela06, could you please try to remove script manually from workspace folder located by path:
"...USER_HOME_FOLDER\AppData\Roaming\DBeaverData\workspace6\General\Scripts"
and launch the product.
The case is hardly reproduceble on my side, it is seems a result of "The workspace exited with unsaved changes in the previous session; refreshing workspace to recover changes."

there have been no updates for a long time. If the issue is still actual for you - feel free to reopen it

Removing
workspace6/.metadata/.plugins/org.eclipse.core.resources/*.snap

e
workspace6/.metadata/.plugins/org.eclipse.core.resources/.root/*tree

seems solve the problem. They are the only remaining files with reference to the "missing" script.

Removing
workspace6/.metadata/.plugins/org.eclipse.core.resources/*.snap

e
workspace6/.metadata/.plugins/org.eclipse.core.resources/.root/*tree

seems solve the problem. They are the only remaining files with reference to the "missing" script.

Thank you so much, this solved the problem indeed !
The ticket can be closed now.
Thanks again for your help


nimble/phy: Introduce common phy for nRF52 and nRF53

This is first cut of common nrf5x phy which works for both nRF52 and nRF53. The goal is to have the same code used on both series in order to make maintenance easier and have the same fixes and features supported.

rebased on https://github.com/apache/mynewt-nimble/pull/1345

@haukepetersen fyi, this needs to be updated in RIOT


Adds include support to match tests before calling run().

Updated PR because $$ means I need to name the variable $include not $include_regex. Tested that it's working.

Looks great IMO, I just need to give a try before merging.

As a side note, it would be cool if there were unit tests for the project, just because manual testing is kind of a pain.

Fixes #5

cool, thanks! I put up the inclusion of the GoTestEngine as well.
On Wed, Apr 29, 2015 at 6:45 PM, Rodrigo Navarro<EMAIL_ADDRESS>wrote:

Looks great IMO, I just need to give a try before merging.
—
Reply to this email directly or view it on GitHub
https://github.com/tagview/arcanist-extensions/pull/8#issuecomment-97630281
.

any chance you want to merge this still? I can keep using my own repo, but I like using a single one ;)


Flutter application is not installed on vivo mobile android version 13 funTouch operating system

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
The app is stuck on The Dart VM service is listening on *****
Expected results
The app stuck not installed on the The Dart VM service is listening on *****
Actual results
App is stuck on vivo funTouch operating system on vivo mobiles
Code sample
Code sample
[The Dart VM service is listening on ***** ]

Screenshots or Video

Screenshots / Video demonstration
[]

Logs
Logs
[The Dart VM service is listening on ***** ]

Flutter Doctor output
Doctor output
[The Dart VM service is listening on ***** ]

@Talha137m
The issue isn't specific to Flutter, but related to Vivo device and Android 13 version. You can read more about it here: https://github.com/flutter/flutter/issues/82520#issuecomment-1001462546
Closing as such, as it is not actionable from Flutter side.


npm run build throws -angular2-jwt/angular2-jwt"' has no exported member 'AUTH_PROVIDERS'.

Hi There,
Great work on pimping up the original ionic2-conference-app, very impressed with your RxJs mods !!
"npm run dev" worked fine, but "npm run build" threw a jwt exception:
10:22:00]  ngc: Error: Error at /Users/XXX/javasource/ionic/ionicV2/ddellamico-ionic-conference-app/.tmp/app/app.module.ngfactory.ts:248:33: Module '"/Users/XXX/javasource/ionic/ionicV2/ddellamico-ionic-conference-app/node_modules/angular2-jwt/angular2-jwt"' has no exported member 'AUTH_PROVIDERS'.
Error at /Users/XXX/javasource/ionic/ionicV2/ddellamico-ionic-conference-app/.tmp/app/app.module.ngfactory.ts:509:38: Module '"/Users/XXX/javasource/ionic/ionicV2/ddellamico-ionic-conference-app/node_modules/angular2-jwt/angular2-jwt"' has no exported member 'AUTH_PROVIDERS'.
It was reported here also:https://forum.ionicframework.com/t/angular2-jwt-after-rc0-broken/65043
I removed AUTH_PROVIDERS from app.module.ts and it worked fine.
Cheers,
Steve

Hello Steve !
Many thanks for your suggestion, I removed 'AUTH_PROVIDERS' and all works like charm now !
When I think how many hours I spent to find out a fix to this problem ! :)
Cheers,
Damien

Hi Damien,
No worries, glad I could help - it took me a while reading through the forums threads but eventually found what worked. As I said you really have done some great work here as it's given me a crash course in both RxJs and RX Store.  I am still not quite sure about ngrx/effects...and how it helps as opposed to not having it (probably my current lack of understanding).
Cheers,
Steve


README : configuration part

How to configure this plugin on cypress 12/13 ?
Configuring plugins via cypress/plugins/index.js is no longer supported as of Cypress version 10.0.0.
We recommend that you update your configuration. Please see the plugins guide and the migration guide for more information on how to update your configuration.

Alright. I can take care of it.
Who's we?

Thanks for your help. It's work fine.
The part below was a copy-paste from the Cypress documentation, so "we" refers to the Cypress developers.


fix loop scoping bug

fixes #1427
changes in f524bcce0cf295941fe665cbcb6846e7e9f39df5 caused assignments in nested loops to still be referenced after the end of the iteration. now, all variables assigned in loops will be cleared from the top level so that jinja won't attempt to resolve them outside of the loop.
Checklist:

[x] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[x] Add or update relevant docs, in the docs folder and in code.
[x] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[x] Run pre-commit hooks and fix any issues.
[x] Run pytest and tox, no tests failed.

Remember to rebase to 3.0.x


Net perf minimal

Changes introduced with this PR
Initial draft for net-perf workflow yaml.

By contributing to this repository, I agree to the contribution guidelines.

Initially the pseudo-code was in the README because we didn't yet know what a real workflow should look like. I really don't think it's necessary here now, nor is the detailed documentation. The docs should move somewhere else with examples, and this should be whittled down to just the workflow fundamental files and a descriptive README. I'll work on that.


chore: Replace jQuery with native API calls

Note: these changes intentionally disregards support for Internet Explorer. Edge is still supported, as is newest versions of Chrome, Firefox and Safari.

Yup, everything seems to be OK here now.


Can't get Biome Volume Factory with factory provider

Running this code:
Sponge.game().factoryProvider().provide(BiomeVolumeFactory.class);

Results in following exception:
[19:24:44] [Sponge-AsyncScheduler-0/ERROR] [sponge]: The Scheduler tried to run the task 'GenerateProceduralTerrain-A-#0' owned by 'spongetest' but an error occurred.
org.spongepowered.api.registry.TypeNotFoundException: Type 'interface org.spongepowered.api.world.volume.biome.BiomeVolumeFactory' has no factory registered!
	at org.spongepowered.common.registry.SpongeFactoryProvider.provide(SpongeFactoryProvider.java:189) ~[SpongeFactoryProvider.class:1.16.5-8.0.0-RC1060]
	at ru.meproject.rustmesponge.procedural.BiomeConstructionTask.run(BiomeConstructionTask.java:58) ~[BiomeConstructionTask.class:?]
	at org.spongepowered.api.scheduler.Task$Builder.lambda$execute$0(Task.java:85) ~[Task$Builder.class:1.16.5-8.0.0-RC1060]
	at org.spongepowered.common.scheduler.SpongeScheduler.lambda$startTask$1(SpongeScheduler.java:278) ~[SpongeScheduler.class:1.16.5-8.0.0-RC1060]
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515) ~[?:?]
	at java.util.concurrent.FutureTask.run(FutureTask.java:264) ~[?:?]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130) ~[?:?]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630) ~[?:?]
	at java.lang.Thread.run(Thread.java:831) [?:?]

Using API version 8.0.0

This isn't an API issue, but an implementation one. Please re-create this on https://github.com/SpongePowered/Sponge/issues/new/choose with all the required information (including version of server that you're running).
Thanks.


Cannot read field "pins" because "helper" is null

Environment info

BlockMap version: 2.2.1
Java version: "16.0.1" 2021-04-20
Operating system: CENTOS Linux aimba 3.10.0-1127.19.1.el7.x86_64 #1 SMP Tue Aug 25 17:23:54 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Description
On one among several worlds, I am getting the following error:
ava.lang.NullPointerException: Cannot read field "pins" because "helper" is null
	at de.piegames.blockmap.world.RegionFolder$SavedRegionFolder.<init>(RegionFolder.java:293)
	at de.piegames.blockmap.world.RegionFolder$LocalRegionFolder.<init>(RegionFolder.java:355)
	at de.piegames.blockmap.world.RegionFolder$CachedRegionFolder.<init>(RegionFolder.java:442)
	at de.piegames.blockmap.world.RegionFolder$CachedRegionFolder.create(RegionFolder.java:551)
	at de.piegames.blockmap.standalone.CommandLineMain$CommandRender.call(CommandLineMain.java:236)
	at de.piegames.blockmap.standalone.CommandLineMain$CommandRender.call(CommandLineMain.java:113)
	at picocli.CommandLine.executeUserObject(CommandLine.java:1933)
	at picocli.CommandLine.access$1100(CommandLine.java:145)
	at picocli.CommandLine$RunLast.executeUserObjectOfLastSubcommandWithSameParent(CommandLine.java:2332)
	at picocli.CommandLine$RunLast.handle(CommandLine.java:2326)
	at picocli.CommandLine$RunLast.handle(CommandLine.java:2291)
	at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:2159)
	at picocli.CommandLine.execute(CommandLine.java:2058)
	at de.piegames.blockmap.standalone.CommandLineMain.mainWithoutQuit(CommandLineMain.java:399)
	at de.piegames.blockmap.standalone.CommandLineMain.main(CommandLineMain.java:403)

Steps to reproduce
I cannot instruct how to reproduce without uploading the whole world somewhere. Is there a chance to make a test for "helper" being null in the code and outputting which file is making trouble?
I tried to compile it myself but my Java skills are close to zero so I could not get it done. Compiling works but running throws this here:
java.lang.IllegalArgumentException: Did not find internal color map default
        at de.piegames.blockmap.color.BlockColorMap.loadInternal(BlockColorMap.java:348)
.....

Did you compile it with our gradle tasks?
Because besides compiling the code, there is also a task that downloads and regenerates the colormaps from the minecraft files.

Yes, if you want to run it yourself see https://github.com/Minecraft-Technik-Wiki/BlockMap#run-it . Essentially, you should run ./gradlew regenerate and then it should work.
For reproduction, it'd be cool if you found a small example world that fails. But more importantly, please post the commands you are running that give you this result. Together with the stack trace, this should hopefully be enough for me figuring out what's going wrong.

The command I am using is this:
/etc/alternatives/jre_16/bin/java -jar /folder/BlockMap-cli-2.2.1.jar render -f /folder/kingdom --create-big-image --min-X=-3264 --max-X=3264 --min-Z=-3264 --max-Z=3264 -o /folder/png --create-tile-html

regenerate works. but then when I do the next one it fails.
gradlew :BlockMap-cli:run

> Configure project :BlockMap-gui
Project :BlockMap-gui => no module-info.java found

> Configure project :BlockMap-internal
Project :BlockMap-internal => no module-info.java found

> Task :BlockMap-cli:run FAILED
Missing required subcommand
Usage: blockmap [-hvV] COMMAND
  -h, --help      Print this help message and exit
  -v, --verbose   Be chatty
  -V, --version   Print version information and exit.
Commands:
  render       Render a folder containing region files to another folder
                 through the command line interface
  render-many  Render multiple worlds using a configuration file for usage in
                 servers

This is the command line interface of blockmap. To access the GUI (if
installed), run `blockmap-gui`.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':BlockMap-cli:run'.
> Process 'command 'C:\Program Files\Java\jdk-11\bin\java.exe'' finished with non-zero exit value 2

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.3/userguide/command_line_interface.html#sec:command_line_warnings

BUILD FAILED in 1s
6 actionable tasks: 4 executed, 2 up-to-date

That already looks promising. You can now pass command line arguments to the application. Doing so through Gradle is a bit unweildy because of potential escaping issues: ./gradlew :BlockMap-cli:run --args="render -f /folder/kingdom --create-big-image --min-X=-3264 --max-X=3264 --min-Z=-3264 --max-Z=3264 -o /folder/png --create-tile-html"

Regarding the issue, there should be a rendered.json.gz file in your output folder. It seems to be broken, please send me its content. You may also try deleting it and see if it helps.

Things are starting to get really weird. The application should create a /folder/png output folder and a rendered.json.gz in it. If you already run the command previously, the one already existing should be used instead.
Please try running in a clean output directory, just to make sure no corrupt files are causing this. After that /folder/png/rendered.json.gz should definitely exist as the error happens later, and the unzipped content should be {regions:[]}.

Ah ok, I thought that .gz is a part of the compile output, not the run. I'll check that and get back to you.

ok so....

If I run the latest version under java 16, the rendered.json.gz from the initial error report is empty.
Making the map on Java 11 on windows works, the rendered.json.gz is created, but I assume you don't need that in this case, anyhow the text file in there is huge (18MB).
I am trying to compile under Java 11 now a ever so slightly changed version (where I add some more debug output to test running later under Java 16) with the instructions from the wiki (gradlew release) but get the following (after having run a "regenerate" sucessfully)

gradlew release

> Configure project :BlockMap-gui
Project :BlockMap-gui => no module-info.java found

> Configure project :BlockMap-internal
Project :BlockMap-internal => no module-info.java found

> Configure project :BlockMap:BlockMap-gui
Project :BlockMap-gui => no module-info.java found

> Configure project :BlockMap:BlockMap-internal
Project :BlockMap-internal => no module-info.java found

> Task :BlockMap:BlockMap-internal:test FAILED

de.piegames.blockmap.ColorMapTest > initializationError FAILED
    java.lang.NoClassDefFoundError: org/hamcrest/SelfDescribing
        at java.base/java.lang.ClassLoader.defineClass1(Native Method)

and repeating...
Not sure where I should get/include/place this org/hamcrest/SelfDescribing

Yes, thank you. To be honest, I'm astonished that an empty file serializes to null instead of throwing a DeserializationException for invalid JSON.


Temperature/Humidity values get randomly mixed between runs

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.32.2
Python release (python3 --version):
Python 3.4.2
Component/platform:
sensor/rfxtrx
Description of problem:
Values of combined temperature/humidity sensor get changed between runs.
A combined temperature/humidity sensor reports the state of the temperature and the humidity. I have such a sensor in the piano (mainly to measure the humidity to see if it doesn't get too dry in winter). The temperature sensor is usually called sensor.piano and the humidity sensor sensor.piano_2. I want to write an automation rule that notifies me when the humidity gets lower than 50%, so I have to watch sensor.piano_2. However, when those two values get mixed, the automation will be triggerd because the temperature (in Celcius) will always be below that value.
Expected:
Values of temperature and humidity don't get mixed up between runs.
Problem-relevant configuration.yaml entries and steps to reproduce:
sensor:
  platform: rfxtrx
  devices:
    0a520d00390100c4370169:
      name: Piano
      data_type:
        - Temperature
        - Humidity

Traceback (if applicable):
None

Additional info:
I have already debugged pyRFXTRX, but that passes the values directly from the sensor and the sensor always reports the values in the same order (which was to be expected, or otherwise the values would get mixed in the same hass run as well). It really appears that home-assistant randomly assigns the values to either piano or piano_2 at startup, and as far as I can see there is no way I can dictate the order.
See attached picture how the temperature changes between runs.

Just a comment to say I've the same problem (see also https://community.home-assistant.io/t/rfxtrx-sensor-data-types-get-swapped/5797).

Could you try this fix: https://github.com/home-assistant/home-assistant/compare/Bug_%234284?expand=1
It is untested code, so there might be some bugs there.
Please check that the entity_id is the same for all other rfxtrx sensors, so that this fix do not break any automation scripts for other users

I've just tried it after a busy weekend, but unfortunately that makes the combined temperature/humidity sensor a single binary sensor:
 entity_id: binary_sensor.sensor_3901
 state: 54

{
  "Humidity": 54,
  "friendly_name": "Piano",
  "Humidity status": "comfort",
  "Humidity status numeric": 1,
  "Temperature": 17.8,
  "Battery numeric": 9,
  "Rssi numeric": 6,
  "unit_of_measurement": "%"
}

Why is it at binary_sensor ?

I don't know, I'm not enough at home in the hass code to be able to tell what's going wrong.
FWIW, I applied your fix to v0.32.3.

Forgot to tell: there were no changes to the configuration.

FYI: you commit 397ae8129ef26e1f0cee82513bc80ee8a5649c61 breaks with an error:
16-11-14 17:13:31 homeassistant.components.sensor: Error while setting up platform rfxtrx
Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/homeassistant/helpers/entity_component.py", line 148, in _async_setup_platform
    entity_platform.add_entities, discovery_info
  File "/usr/lib/python3.4/asyncio/futures.py", line 388, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
    value = future.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/sensor/rfxtrx.py", line 51, in setup_platform
    entity_id = generate_entity_id(ENTITY_ID_FORMAT, hass=hass)
TypeError: generate_entity_id() missing 1 required positional argument: 'name'

Looking at other uses in hass the device_id is a mandatory argument.

@erikmouw can you use the latest Home Assistant version and confirm that the problem still persists ?

I can confirm that bug still persists in the latest version

Can confirm as well the issue is still present in the last version I'm using a nest thermostat and humidity for outside and inside is mixed up after every restart sometimes you're lucky and you get the right mix out of temp and humidity but it's still messing up the influxdb

@balloob: I just upgraded to 0.33.1, and yes, the bug is still there:

Notice the temperature peak. After a restart it's back at how it should, so the figures below the graph reflect how I expect it to be.

@balloob has made a fix, that hopefully will solve our problems.
@erikmouw , @hjbout , @nbittmann :
I will not be able to test it the next days, so would be great if you could try.
You will find the changes here: https://github.com/home-assistant/home-assistant/pull/4530/files

This issue is fixed in dev and will be available in 0.34.

@balloob: Applied this fix on top of 0.33.3, it works perfectly.
I had to rename the devices (sensor.piano and sensor.piano_2 became sensor.piano_temperature and sensor.piano_humidity), but the sensor naming is consistent between restarts. Thanks!


package.json: Do dependencies need to be "master"?

I'd like to use xml3d as an npm dependency, and was wondering whether you absolutely need to depend on the master branches of gl-matrix and texture-manager, rather than a specific version. Especially since gl-matrix is on npm proper.
For texture-manager, wouldn't the <URL>#<commitish> syntax for dependencies be more appropriate?

6584825834252dff16373c1d801c9df4cdeb1bdf fixes this for gl-matrix.

Fixed for the texture-manager in the 5.2 release


core-components: extract react components and hooks from core

See changeset for details.

Nah, not worth keeping around. Closing.


[Bug] Webpage Titles not showing on Tabs

Currently using the following to complete the task:
    Tabcontrol1.SelectedTab.Text = CType(Tabcontrol1.SelectedTab.Controls.Item(0), GeckoWebBrowser).DocumentTitle

Appears to work on some webpages, but there is a delay, a fix is adding it to the loading event rather than the done event, or adding it to both incase one don't work it can rely on the other.

Will be fixed in the update

Was resolved by adding a handler for those pages without a document title.


Improve our approach for testing auth

Our auth tests tend to fall into one of two categories
Either we implement auth on a base class, then test the auth is handed correctly in the base class using a dummy
https://github.com/badges/shields/blob/master/services/stackexchange/stackexchange-base.spec.js
Or we pick one concrete implementation and test that
https://github.com/badges/shields/blob/master/services/gitlab/gitlab-tag.spec.js
This is not water-tight.
As @calebcartwright suggests in https://github.com/badges/shields/pull/9387#discussion_r1299106960
it would be good if we could be a bit more robust about this and make sure we cover all the concrete service classes that are supposed to use auth.
I've not tried it yet, but one possible solution might be to encapsulate the test boilerplate once in a function, then loop over all the relevant classes and call it
something like
function runTheTests(ServiceClass) {
  const service = new ServiceClass();
  // setup
  // make assertions
}

for (const serviceClass of [ServiceClass1, ServiceClass2]) {
  runTheTests(serviceClass)
}

@jNullj wanted to flag your attention in case this was something you'd be interested in working on (definitely no pressure though!)

I gave this a quick look and i think i am up to the task.
Just to make sure i got the gist of it, the idea is to test every service (and not just base service) while using the same base function.
Would that mean i should probebly loop over services for each of the base classes the require auth?
Should I add those tests to each base class spec file?

Just to make sure i got the gist of it, the idea is to test every service (and not just base service) while using the same base function.

Bingo

Would that mean i should probebly loop over services for each of the base classes the require auth?
Should I add those tests to each base class spec file?

First off: I think probably the best way to tackle this will be to pick one service. Lets say stackexchange for the sake of argument - just so we can use concrete examples in the next bit - but we could pick another one first. Work on a PR for that one first to establish the pattern. That will mean we don't have to modify too much code if there is a bit of back-and-forth on the PR. Then once we've reviewed that and decided on an approach we like we can do another PR (or a series of them) applying that approach to the other services.
Beyond that, I think that's a couple of ways to approach it:
One option would be the example I gave in the top post. So that one would probably look like a single test file (in this case stackexchange.spec.js) which imports StackExchangeMonthlyQuestions, StackExchangeReputation, StackExchangeQuestions and tests all of them.
Another way would be to define a re-usable helper in a stackexchange-test-helpers.js that can then be imported into individual test modules for stackexchange-monthlyquestions.spec.js, stackexchange-reputation.spec.js and stackexchange-taginfo.spec.js.
I think either of those could work. I think we probably want to optimise for:

reducing boilerplate and also
making it clear what failed if something failed

It seems like https://github.com/badges/shields/pull/9681 is pretty close to merging which gives us a good starting point for a shared helper and pattern we can apply.
Once we merge that, the next steps of this will be to make use of those and make a series of follow up PRs gradually bringing other services into line with the convention.
The following services have got some kind of tests in place. Some of them are testing just the base class, some test one concrete implementation or all concrete implementations. They're a bit of a mixed bag:

bitbucket: Basic Auth
discord: Authorization header
drone: Authorization header
gitea: Authorization header
gitlab: Authorization header
jenkins: Basic Auth
jira: Basic Auth
librariesio: Query String
nexus: Basic Auth
npm: Authorization header
obs: Basic Auth
opencollective: Query String
sonar: Basic Auth
symfony: Basic Auth
teamcity: Basic Auth
wheelmap: Query String

These ones all have no tests at all covering the auth:

azure-devops: Basic Auth
bower: Query String, shared with librariesio
curseforge: API Key header
pepy: API Key header
youtube: Query String
weblate: Authorization header

..and then these three are complicated and messy. I'm going to suggest we ignore these as they are unlikely to fit into any generic pattern.

github
twitch
docker

One thing we could do next is pick off all the services that use query string auth first.
Another one could be to concentrate on making the test helper more generic. If we've got a test helper (or 3 helpers) that work for services that implement auth using query string, basic auth, and authorization header, that would cover most cases.

https://github.com/badges/shields/pull/9681 implements the necessary test helpers an applies the pattern to:

services/discord/discord.spec.js
services/docker/docker-automated.spec.js
services/obs/obs.spec.js
services/pepy/pepy-downloads.spec.js
services/stackexchange/stackexchange-monthlyquestions.spec.js
services/stackexchange/stackexchange-reputation.spec.js
services/stackexchange/stackexchange-taginfo.spec.js

Will add other services today or tomorrow

Awesome :+1:
When you do this, can you split them down into chunks somehow. Doesn't necessarily have to be as fine grained as one PR per service, but maybe one per auth method: All the Basic Auth services in one PR, all the Query String services in another. Something like that.
If I've got a bunch of small PRs to review, its easier to make time for them. One huge PR will ens up sitting around for ages because it is harder to find the time to review it. Ta


Removed the repetition in the RNS user guide

What

Removed a duplicate sentence found in the RNS user guide. The sentence 'One of the ways to operate with the service is through the application known as the manager' was repeated twice.

Why

The removal of the repeated sentence enhances the readability and clarity of the documentation. Duplicate information can be confusing or misleading for users, and maintaining concise and accurate documentation is essential for user understanding and experience.

Refs

(optional: include links to other issues, PRs, tickets, etc)

Congratulations @zdaodu-iov on your first contibution 🚀.


Invalid pointer Error

After starting ./koadic, when i run 'info' command it is existing with the below error. Any idea why ?
     ~[ Version:  0xB ]~
        ~[ Stagers:    6 ]~
        ~[ Implants:  44 ]~

(koadic: sta/js/mshta)# info
free(): invalid pointer
Aborted**
root@kali:~/Packages/Post-Exploitation/koadic#

Looks like internal CPython error, perhaps out of memory.


circle ci detection

circle ci detection

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
Circle CI Disable Security Step :x:

diskUsage: 483328
runDuration: 3.1s
scanCount: 4

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
Circle CI Disable Security Job :white_check_mark:

diskUsage: 163840
runDuration: 1.469s
scanCount: 19


Data Visualization Pipeline

As a personal open-source project owner, even if our open source has statistical data visualized in the user interfaces of our application. I think if there is any possible way to visualize those data outside.

Hi, I am happy to work on this issues, this is the roughly sketch for this issues. I planned to came up with 2 approaches. 1 is a local infrastructure for data visualization (Docker based) and native cloud infrastructure.


Rename createPasswordResetChallenge to sendPasswordResetEmail

Description
Documentation
Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.
[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

Current dependencies on/for this PR:

main

PR #853

PR #854

PR #855   👈

This comment was auto-generated by Graphite.


update the voice style for pt-br-franciscaneural

update the SSML section 'Adjust speaking styles" to include the new style for franciscaneural

@qiliao123 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


[Recorder] Rebrand recorder-new 1.0.0 as recorder 2.0.0

Fixes #19743
What's going on?

[x] Publish<EMAIL_ADDRESS>with the unreleased changes (thanks to @praveenkuttappan 🙂)
[x] Delete the existing recorder sdk/test-utils/recorder
[x] Rename recorder-new -> recorder
[x]<EMAIL_ADDRESS>-><EMAIL_ADDRESS>

Now that #18227 is in, we'll need to update the template to point to the rebranded package when this PR goes in.


Request to update the website

Hi @MartinThoma 🙂
Could the static website be updated please, following the merge of this PR in August?
https://github.com/py-pdf/py-pdf.github.io/pull/4
It seems that I do not have the necessary rights to manually run the pages-build-deployment workflow:
https://github.com/py-pdf/py-pdf.github.io/actions/workflows/pages/pages-build-deployment

Yes, I see fpdf2 now on https://py-pdf.github.io/ . I'm sorry for that failure :see_no_evil:

Yes, I see fpdf2 now on py-pdf.github.io . I'm sorry for that failure 🙈

No worries, and thank you!

Permission check: I want you to have those permissions. I don't know what is missing for that.

I wonder if it's not a matter of Base permissions: https://github.com/organizations/py-pdf/settings/member_privileges
We could either raise this to Admin or add me explicitely as a member to this repository, with higher permissions.

I've added you to this repository as an admin as this seems the obvious no-regret choice (I'm uncertain about the other option and what else this might change).


Initial state parser modifies input states inplace

The root cause of https://github.com/dwavesystems/dwave-tabu/issues/86 is the behavior of Initialized.parse_initial_states, where as_samples is used with copy=False default, and then vartype converted in place, modifying the input initial_states -- but only partially. Raw sample data is converted, but SampleSet's vartype is not.
https://github.com/dwavesystems/dimod/blob/30272a7e47ec71988413ce8f8339d397ceff8d17/dimod/core/initialized.py#L126-L131

There seems to be a bug in as_samples as well, where the copy argument is ignored for certain cases (ie. the one mentioned in dwavesystems/dwave-tabu#86)
https://github.com/dwavesystems/dimod/blob/43fa43063067ae81d160d7f0ab2dd5e5f0b7e931/dimod/sampleset.py#L266-L269
It seems like the fix for this issue is to address the as_samples bug and then call as_samples with copy=True.


Visual F# Tools leak memory while reloading solutions

Scenario:
◾Disable VFPT in VS if it is installed
◾Open VFPT solution (https://github.com/fsprojects/VisualFSharpPowerTools), VS occupies ~ 300MB RAM
◾Repeat the following steps 15 times: (1) Switch to a new git branch (2) Reload the solution (3) Rebuild the solution
◾VS now occupies ~ 1.7 GB RAM
Context:
We are hunting down memory leaks in Visual F# Power Tools. It turned out FSharp.Compiler.Service (FCS) leaked memories when Xaml type providers (FsXaml) didn't dispose resources correctly. These bugs have been fixed in FCS and FsXaml. See the discussion at  https://github.com/fsharp/FSharp.Compiler.Service/issues/158.
After bug fixing, there are still memory leaks. I think Visual F# Tools has the same issue where type providers are in use. We did profile using PerfView without enabling VFPT. Here is a diff of two snapshots where we reload and rebuild solutions twice.
https://cloud.githubusercontent.com/assets/941060/3277997/c56876ec-f39f-11e3-8042-4840c6cffffd.png
https://cloud.githubusercontent.com/assets/941060/3278000/13a16d6e-f3a0-11e3-97cd-b994b8086d8e.png
Configuration:

Visual Studio 2013
Windows 7
A recent build of Visual F# Tools 3.1.1

I believe the fix should be similar to that of FCS. Here are relevant commits:
https://github.com/fsharp/FSharp.Compiler.Service/commit/a325df24d3a1a96524742fe0946407c69b02036e
https://github.com/fsharp/FSharp.Compiler.Service/commit/4e7c8bca24518fb9c77469fa6715d26b0eca3a63
Let me know if you need more information.

We should apply the linked fix to VF# Tools.

/cc @quasilord

We should apply the linked fix to VF# Tools.

I tried to do this, but there are more commits involved.

This was partially addressed by recent fix from @dsyme, but it looks like we still leak a bunch on reload.  I've left it open to keep tracking remaining leaks.
Moving out of VS 2015 milestone, though.

@latkin AFAIK a well written type provider shouldn't leak on reload.  What repro steps should I follow to check this out (I'm ok with this being vNext)

Sorry, I was not specific enough.  I trust that the type provider leak in particular is now fixed, but the general problem indicated by the title of the bug ("Visual F# Tools leak memory while reloading solutions") is not.

OK, got it, thanks

Re memory leaks - I noticed another case where the Fsharp.Compiler.Service code is correctly disconnecting a callback when a prjoect is unloaded, see https://github.com/fsharp/FSharp.Compiler.Service/blob/0c37365691e5568adfc357fc915eed645b45481d/src/fsharp/CompileOps.fs#L4093.  Specifically we are missing this commit: https://github.com/fsharp/FSharp.Compiler.Service/commit/4e7c8bca24518fb9c77469fa6715d26b0eca3a63


Make yield is not complex in Style/TernaryParentheses cop

yield is simple enough.
# EnforcedStyle: require_parentheses_when_complex
#
# # bad
# foo = (yield) ? a : b
#
# # good
# foo = yield ? a : b

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[ ] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

👍


internal redirects: support passing headers from response to request

Commit Message:
This adds a new (repeated) field in the internal redirect policy, "response_headers_to_preserve".  When set, the headers named there will be copied from the response that triggers an internal redirect into the request that follows.
This allows some limited information passing through the internal redirect system.
The current system is faithful to the idea that internal redirects are purely a latency optimization, and should behave similarly to if the redirect had been passed to the downstream user-agent. This does violate that idea.
Other proxies, such as Nginx, have a much more flexible way of handling internal redirects that allows a fair bit of information passing like this. This should allow implementations to adopt Envoy that are using this kind of information passing, with reduced needs to rearchitect.
Risk Level: Low
Testing: Various tests added. Manual test done as well.
Docs Changes: Docs updated to reflect the additional features.
Release Notes: Release notes updated.
Fixes: #30441
Fixes: #16777
API Considerations: Requires configuration, so default is no behavior change. Otherwise, I believe everything else is covered or not relevant.

I'm not particularly excited about the field name, so I'm happy to take suggestions for a better name if desired.
Also, I feel like I don't understand the lifetimes for the header maps very well, so I defaulted to using SetCopy(). I'm happy to change those if that helps at all.

At least one of the failed checks looks to be a flake in the test infrastructure, btw.

/lgtm api

/wait on CI and comments

I've been fighting a bit with the pre-push checks because they don't work on M2 Macs, which is my primary development environment for this work.  I think things will pass now, at least.

I have rebased to try to deal with the repeated conflict resolution I was doing.
In effect, I've added another 2 commits to the stack, so you can see the changes easily, still, I hope.
  If that's not the case, well, I can clean things up a bit, heh.)

LGTM, though it looks like this needs a main merge.

/wait

Fixed the merge conflicts in the changelog since I think this is hopefully done now.

I don't think the latest changes caused any of these test failures, so if someone can retrigger the tests, that would be great.
(The tests failed after I did the conflict-merge for the changelog, which ... seems unlikely to cause the failures seen, lol.)

/retry-azp
(Not sure if that will work, but worth a shot)

/retest :-)


Small design changes

See commits for changes

Thanks for the feedback! - Understandable, changed them to calendar icons and also added the memberlist to groupOptions.
Will leave the PR for now. Awaiting review.


Erroneous import in vmware/vapi/vmc/client.py from vapi-vmc-client==2.9.0 package

[x] I am using the latest SDK version
[x] This API is compatible with my vCenter version (You can get the info from 'vcenter_version' in each sample)
[x] I have searched existing issues

Environment

sdk package version:

$ pip freeze | grep vapi
vapi-common-client==2.9.0
vapi-runtime==2.9.0
vapi-vmc-client==2.9.0

python version:

$ python --version
Python 2.7.13

wheel package used: https://github.com/vmware/vsphere-automation-sdk-python/tree/346f5b1/lib/vapi-common-client

Steps or code snippet to reproduce + Actual behavior
$ ipython -c 'from vmware.vapi.vmc.client import create_vmc_client'
---------------------------------------------------------------------------                                                                                ImportError                               Traceback (most recent call last)
<ipython-input-1-fad65e76a271> in <module>()                                                                                ----> 1 from vmware.vapi.vmc.client import create_vmc_client
                                                                                /home/lucas_cimon/.local/share/virtualenvs/infralib-py2/lib/python2.7/site-packages/vmware/vapi/vmc/client.py in <module>()
      8 import requests                                                                                9
---> 10 from com.vmware.vmc_client import StubFactory                                                                                11 from vmware.vapi.bindings.stub import ApiClient
     12 from vmware.vapi.lib.connect import get_requests_connector
ImportError: No module named com.vmware.vmc_client

Expected behavior
No ImportError raised.
Resolution
Simply replace the 2 following lines in vmware/vapi/vmc/client.py:
from com.vmware.vmc.client import StubFactory
from vmware.vapi.bindings.stub import ApiClient

By:
from vmware.vapi.bindings.stub import ApiClient, StubFactory

I have the same error!

... ImportError: No module named vsphere.client

@dungla2011 The binding version you installed(2.5.0) is not the latest and thus you don't see vmware.vapi.vsphere module. Once you installed the correct version 1.5.0, you should see the module.
@Lucas-C The StubFactory should come from com.vmware.vmc.client. The pip dependencies issue is tracked by https://github.com/vmware/vsphere-automation-sdk-python/issues/70. Closing this as not a bug.


batch size will affect llm inference results?

System Info
GPU：A30*1
system: Ubuntu 22.04
trtllm: 0.7.1
tritonserver: 23.12
model: qwen 7b/14b
Who can help?
@byshiue
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
1
python build.py --hf_model_dir ./model_zoon/Qwen-7B-Chat/
--dtype float16
--remove_input_padding
--use_gpt_attention_plugin float16
--enable_context_fmha
--use_inflight_batching
--paged_kv_cache
--use_gemm_plugin float16
--output_dir ./model_zoon/Qwen-7B-Chat-fp16-trt/
2 build model repository
trtllm_bls+trtllm+preprocess+postprocess
inflight batching enable
3. launch server
python3 scripts/launch_triton_server.py --model_repo=tensorrtllm_backend-0.7.1/trt-models-repository-2/ --metrics_port=8005 --world_size=1
4 test
q1 = {
"stream":false,
"max_tokens": 1000,
"bad_words": "",
"stop_words": "<|im_end|>",
"pad_id":151645,
"end_id":151645,
"top_k":1,
"presence_penalty": 1.05,
"text_input": "<|im_start|>\nYou are a helpful assistant.<|im_end|>\n<|im_start|>user\n北京的夏天是什么样子<|im_end|>\n<|im_start|>assistant\n"
}
q2={
"stream":false,
"max_tokens": 1000,
"bad_words": "",
"stop_words": "<|im_end|>",
"pad_id":151645,
"end_id":151645,
"top_k":1,
"text_input": "<|im_start|>\nYou are a helpful assistant.<|im_end|>\n<|im_start|>user\n北京的春天是什么样子<|im_end|>\n<|im_start|>assistant\n"
}
First, initiate program A which comprises four threads concurrently sending the following request q1 in a continuous loop for 1000 iterations. Simultaneously, initiate program B to send the following request q2. In theory, the response from program B should be consistent every time, but in practice, program B occasionally produces varying results.
Expected behavior
Theoretically, when topk=1, the response to each request should be the same. However, in reality, there are occasionally a few characters that are different.
"text_output":"北京的春天一般在3月中旬到5月中旬，这段时间的天气温暖宜人，白天和晚上的温差较大，白天最高气温一般在20-25摄氏度，晚上最低气温一般在10-15摄氏度。春天的北京，万物复苏，花儿盛开，树木发芽，鸟儿鸣叫，是一年中最美的季节之一。"
actual behavior
In practice, the following two types of responses will be received.
First:
"text_output":"北京的春天一般在3月中旬到5月中旬，这段时间的天气温暖宜人，白天和晚上的温差较大，白天最高气温一般在20-25摄氏度，晚上最低气温一般在10-15摄氏度。春天的北京，万物复苏，花儿盛开，树木发芽，鸟儿鸣叫，是一年中最美的季节之一。"
second:
"text_output":"北京的春天一般在3月中旬到5月中旬，这段时间的天气温暖宜人，白天和晚上的温差较大，白天最高气温一般在20-25摄氏度，晚上最低气温一般在10-15摄氏度。春天是北京最美的季节之一，此时的北京公园、街道、胡同等地方都充满了生机和活力，各种花卉盛开，景色宜人。"}
additional notes
I suspect it's because floating-point calculations do not obey the commutative law. Requests from different batches may have varying block sizes in GEMM, potentially leading to slight differences in GEMM results and ultimately causing variations in inference outcomes. However, I don't have any evidence to support this, haha.

I agree with your guess. When the problem size changes, TensorRT might select different kernel and lead to little different results. So, I think it is not a bug.

hi @tdeng521 do u still have further issue or question now? If not, we'll close it soon.


gluster-block: replace ReadWriteMany with ReadWriteOnce in pvc

This patch replaces ReadWriteMany with ReadWriteOnce in pvc example. Currently gluster-block does not support ReadWriteMany as GetAccessModes() in glusterblock-provisioner.go.

/lgtm

/area gluster-block

@wongma7 please merge this.


How to cancel the effect of the bottom margin or padding

How to cancel the effect of the bottom margin or padding,,,I want full screen effect

Hello,
Can you please show the code you used to create the indicator? If you don't add use any of the android:padding*=... attributes on the indicator, it wraps to the dots on the top and bottom by default.
In the sample app, the bottom padding is coming from the outer activity layout (the RelativeLayout), not from the ViewPager or ViewPagerIndicator.

I want full screen effect

Hi @zhouxin1233,
I'm so sorry about falling off the conversation here. I started a new job recently and let this fall through the cracks.
Were you able to resolve your problem? Removing the android:paddingBottom attribute from the indicator should have been all you needed to do to position the indicator back to where it was in the first image. I'm not quite sure I understand exactly what you want for your "full screen effect," so feel free to reply with more details if you're still looking for help.

Closing due to inactivity.


Check IPv6 display in Sunstone

Description
It's been reported that VMs with IPv6 do not show its IP information in Sunstone
To Reproduce
Create a VM with a NIC with IPv6. Try to get the IP from Sunstone
Expected behavior
IPv6 info displayed
Details

Affected Component: Sunstone
Version: 5.8.1

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Addressed in #3241


neXtSIM-NEMO coupling

This issue should be the main issue describing what is needed to couple neXtSIM to NEMO via OASIS. It may refer to other issues as they arise to allow us to focus on those.
Exchange fields required for the coupling
ice -> ocean

ice-ocean stress in x-direction [Pa]
ice-ocean stress in y-direction [Pa]
ice-ocean stress magnitude [Pa]
evaporation minus precipitation [kg/m^2/s]
non-solar heat fluxes (longwave + latent + sensible) [W/m^2]
short-wave flux [W/m^2]
surface salt flux [kg/m^2/s]
surface fresh-water flux [kg/m^/s]
sea-ice concentration [0-1]

ocean -> ice

ocean velocity in x-direction [m/s]
ocean velocity in y-direction [m/s]
sea-surface height [m]
sea-surface temperature [°C]
sea-surface salinity [PSU]
height of first ocean layer [m]
the fraction of short-wave radiation penetrating the first ocean layer [0-1]

Location of values on the (Arakawa) grid and vector rotation.
All values are passed at the centre of the cell (T-point), with the following exceptions:

NEMO sends ocean velocities located at the U-point for x-direction and V-point for y-direction values.
NEMO expects ice-ocean stresses located at the U-point for x-direction and V-point for y-direction values.

The two models have different definitions of x and y directions, so these vectors must be rotated before or after coupling. The most practical way to handle this is to interpolate both components to the centre (T) point, do the rotation there, and interpolate back to U/V points if needed. We may think of a more sophisticated way to handle this.
Tasks to complete the coupling

[ ] Make sure neXtSIM calculates all the fields needed by NEMO
[ ] Make sure neXtSIM can use all the fields provided by NEMO
[ ] For fields calculated by neXtSIM, make sure we can pass the fields from the physics routine that generates them to the coupler interface
[ ] For fields provided by NEMO, make sure we can pass the fields from the coupler interface to the physics routine that needs them
[ ] Initiate the state of OASIS at the start of the run and close everything at the end
[ ] Implement an OASIS "send" interface in neXtSIM
[ ] Implement an OASIS "receive" interface in neXtSIM

See also issues #69 and #80

Hi @timspainNERSC & @ericmaisonnave! Can you please read through this issue and see if it describes the problem in sufficient detail?

Are the velocities on U/V points for nextSIM-DG? As far as the Magdeburg dynamics are concerned, they are on a CG grid. Currently that is fitted to a set of DG components and then the DG0 value is passed back, which is defined at the T point, at the same point as everything else.
Presumably NEMO uses the U/V points.
This might be an argument for adopting and adapting the scheme interpolating to the T points, or it might be an argument that I need to look at the way the velocities are passed around again.

You're right that in neXtSIM, the velocities and stresses are not at the U/V points but on a CG grid. I've updated the issue to make this clear.

Velocities: it's possible to calculate these values at T point in NEMO, if needed (but I am not sure of the accuracy near the shore)
Cpl fields from ice to ocean, water: if "surface fresh-water flux" means, total E-P-R-C, with R=runoff and C=calving, it's fine for me.  One would also send a runoff only variable, in a first step (again, it depends if we are building a demonstrator or a final operational model)
Cpl fields from ice to ocean, ice-ocean stress: NEMO also needs the atmosphere to ocean (wind) stress. On the ice covered region, it would better need the combination between wind and ice stresses (in one variable)
Cpl fields from ocean to ice: is neXtSIM really waiting for the fraction of short-wave radiation penetrating the first ocean layer ? And SSH plus first level height ?

Masks are different in T, U and V grids. It means that some U or V values can be missing on the T point of the neXtSIM grid. That's why I propose to let NEMO interpolate the values on its T grid before coupling
ARPEGE will deliver separate values for R and C (different atmosphere/ocean interpolations) but neXtSIM can add all the water fluxes in one E-P-R-C value before sending to NEMO
3a. OK for receiving the total surface stress in NEMO
3b. As mentioned last Autumn in Grenoble, it should be possible to use OASIS for coupling Laurent's Aerobulk library. Tell me if you are interested (this is also something I would like to do)
OK to couple SSH + e3t(first_layer). If e3t stays constant, we will remove this coupling field.

The latitude, longitude (center + 4 corners), mask and edge/NS angle variables for both Northern and Southern ORCA2 grids are ready (I send the netCDF files in a separate email to Tim). I adapted my ORCA/hemisphere input file converter to produce SI3 or neXtSIM input files.
The netCDF format can be changed to fit neXtSIM requirements (please provide the specifications if any)
All angle, latitude and longitude units are degrees. The 4 corner coordinates are provided in this order (for both N and S grids):
2-----1

  -

  -

3-----4

I've pushed a branch called issue240_oasis_ocean_coupling to work on this issue.
I've created a new IOceanBoundary-derived module to handle the coupling in this branch. Most of the work Eric did in issue240_oasis_interface should be ported to here. In particular, what Eric put in the slab-ocean module should be in the new OASISCoupledOcean module.


Add String description in log arguments

Summary
Trying to add an integer directly to the log was throwing EXEC_BAD_ACCESS
NONE

/cherry-pick release-2.0

Cherry pick is scheduled.


modify DistinctSiteProperty

Modified DistinctSite to work with regard to coordination number instead of wycoff symbols
Now the describer can work with both single-specie and mixed-species for each coordination number
Modified relative tests. Passed individual tests, but does not work with MultiDescriberTest yet. working on it.

Fix the conflict. You need to always pull and merge latest origin:master before submitting pull reqs.

Also, this is a public code base. Do not put work in progress here.


Docs: Show how to attach debugger for both server + client code

See title

Could it be helpful to to also generate a launch.json config if the VSCode option is set to true during init?

Sure, that's a good idea.


speech cann't be chinese

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

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Please include more information. Does this mean that it's responding in chinese?

Hi, Please create an issue with more context, Thanks. Closed for insufficient context.


Support multiple typeNames on @SchemaMapping and @BatchMapping

I'm not sure how universal this issue is but I've run into a use-case where it would be useful to be able to define multiple typeNames on a @BatchMapping (and @SchemaMapping) annotation.
My use-case is I have a  filed populated by an "expensive" operation that is modeled in GraphQL as a part of an interface and a union type.
interface Model {
   id: String!
   expensiveField:  String!
}

type A implements Model {
   id: String!
   expensiveField:  String!
}

type B implements Model {
   id: String!
   expensiveField:  String!
}

union AorB = A | B

type Query {
 findAorBbyId(id: String!): AorB
}

In Java (Kotlin) I've modeled this schema as
interface Model {
   val id: String
}

data class A(
   override val id: String
)

data class B(
   override val id: String
)

@QueryMapping(name="findAorBbyId")
fun findAorBbyId(@Argument(name = "id"): Model {
   return if(xxx) {
      A("id")
   } else {
      B("id")
   }
}

This works well and allows me to skip defining expensiveField on my Java model and avoid any unnecessary work unless this field is explicitly requested.
In order to resolve the expensiveField field I'm using @BatchMapping, although @QueryMapping would work exactly the same.
    @BatchMapping(typeName = "A", field = "expensiveField")
    fun batchLoadExpensiveFieldA(models: List<Model>) = resolve(models)

    @BatchMapping(typeName = "B", field = "expensiveField")
    fun batchLoadExpensiveFieldB(models: List<Model>) = resolve(models)

This works fine but since I have this field defined on both A and B in my GraphQL schema but not on the shared Java model I have no way to specify a common typeName requiring duplicated @BatchMapping functions.
Checking through the AnnotatedControllerConfigurer it seems like it would be easy enough to update getMappingInfo to return a list of MappingInfos and support defining multiple typeNames.
https://github.com/spring-projects/spring-graphql/blob/3e29e71a6e437cef93753b6c4615dfe42f0443c7/spring-graphql/src/main/java/org/springframework/graphql/data/method/annotation/support/AnnotatedControllerConfigurer.java#L260-L271
Would a change like this be useful? Am I missing another way to support this use-use instead?
Thanks!

Superseded by #236.


Add instructions for using this library

I am not clear on how best to use this library. Do I need to make tsconfig.json changes? Or ///reference..
Do I use  "typeRoots": ["./node_modules/@types", "./node_modules/chrome-types"]?

@vjpr what I did was creating a chrome.d.ts file inside my types folder and added this as its content:
/// <reference types="chrome-types" />

declare global {
  interface Window {
    chrome: chrome;
  }
}

not really sure if that's the correct way to do it, but it's working. Hope this helps!

For now I'm using it like this, I'm not a fan of referencing from global scope
I had to use patch-package