bcdb as redis server is not working

/sandbox/code/github/threefoldtech/jumpscaleX/Jumpscale/data/bcdb/tests/7_redis.py
there seem to be issues with encoding...
also need to get it to work with
RDM = redis desktop manager
so we can see the records as hsets, so easy to work with it graphically

Had some issues with the tests as the current implementation of bcdb has changed than before.

Stopped working on it, will continue today

working on it

now the test works good using SQLITE Backend and ZDB backend

i have problem with redis desktop manager all keys and values undefined

Now Everything works Good

estimate time 3 hours:

Verification Steps:

we need to start zdb client and server
j.servers.zdb.start()
cl = j.clients.zdb.client_admin_get(port=9901)

use bcdb as a redis server
 j.data.bcdb.redis_server_start(zdbclient_port=9901)

check if it running inside tmux

and connect to it using redis desktop manager to connect to it

there is no data when I look at the schemas?

try with kosmos 'j.data.bcdb.test(name="redis")'
and then connect, should see data

@despiegk
Now should work
My PR: https://github.com/threefoldtech/jumpscaleX/pull/554
Branch : development_redis
Need to merge

if you want to see all data objects of schema after test
please comment this line https://github.com/threefoldtech/jumpscaleX/blob/development/Jumpscale/data/bcdb/BCDBFactory.py#L196
because after test we close redis and clean everything

Verification steps:

j.data.bcdb.test(name="redis")

and connect to Redis Desktop Manager


Selection to Bookmark not always saving exact position

Currently reading 채식주의자 by 한강 and finding at some points selecting to bookmark doesn't work properly. I selected the text shown in the screenshot and pressed B, but it set the bookmark at the top of the page. Up until this point, the bookmark worked correctly in every paragraph on the page. I'm not sure if this is a Firefox bug or exists in the original ttu as I usually read Japanese novels on my ereader, so I need to do some testing with ttu to see if I can reproduce it with a Japanese novel.

Appears to be the same issue as https://github.com/ttu-ttu/ebook-reader/issues/130


Simplify cookiecutter

Simplifies the cookiecutter-data-science template to something a bit more like the development environment we actually use:

Adds a makefile which creates a virtualenv with venv
Moves to a system of unpinned requirements that are pinned with make update-requirements
Adds a .envrc file for use with direnv and sets aws profile there
Splits source dependencies and environment dependencies into requirements_dev.txt and requirements_dev.txt
Other minor updates

My bad! Wrong repo!


exclude metrics URL from GA

@saeedmb8 review please
We'll need to update all our projects with this

Noted 👍


BF voice plugins

What is your question?
bf's voice is getting shorter when going normal and seems fast (fl studio)
https://user-images.githubusercontent.com/95048943/143490159-4e101a61-5425-4264-b2ef-fa3c88c7cc07.mp4

o)

Not the place to post this, but to sum it up: Pitch and speed are things that directly corelate with one another when pitch chaning the rudimentary way. Find some other way to change the pitch of the samples.

????


[Bug] [Workflow Instance/Workflow Name] The display is not standard when the workflow name is too long

Search before asking

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

What happened

The display is not standard when the workflow name is too long
What you expected to happen
Replace the part that cannot be displayed with...
How to reproduce
Create a new workflow with a long name.
Anything else
No response
Version
3.0.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

It is ok for me. Could you check it again? @Gagepmd

same issue #11787


/v2/safes returns invalid payment token error for valid address in Polygon

Describe the bug
I deployed gnosis safe relayer with Polygon and when I try to send request to /v2/safes it returns {   "exception": "InvalidPaymentToken: 0x0000000000000000000000000000000000001010" } even though the payment token address is valid in polygon network.
To Reproduce
Steps to reproduce the behavior:

Do POST on '/v2/safes' with this request.
{ "saltNonce": 0, "owners": [  "0x8198C81bdE4552344177437Fc5E4Fb0dd7f4eaCE" ], "threshold": 1, "paymentToken": "0x0000000000000000000000000000000000001010" }
Then GET on { "exception": "InvalidPaymentToken: 0x0000000000000000000000000000000000001010" }

Expected behavior
It should not say invalid payment token.

This issue belongs here, not on the tx-service 😉
To enable a payment token you need to go to the admin page /admin on the relay service and enable a Token as a payment token. You need to provide also either a fixed price or an oracle.

This issue belongs here, not on the tx-service 😉
To enable a payment token you need to go to the admin page /admin on the relay service and enable a Token as a payment token. You need to provide also either a fixed price or an oracle.

@Uxio0 I just added token in admin but still getting invalidPaymentToken issue.
any further step needed?

This issue belongs here, not on the tx-service 😉
To enable a payment token you need to go to the admin page /admin on the relay service and enable a Token as a payment token. You need to provide also either a fixed price or an oracle.

I just added payment token in admin page but still getting invalidPaymentToken issue.
Any further steps needed?

Did you set gas=True for the token?


Relax opencensus-context constraint

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

https://github.com/census-instrumentation/opencensus-python/blame/v0.10.0/setup.py#L44-L47


[#117020631] Reduce default quota to 2GB

What
We only want to allocate 2GB of "Freemium" memory in the default quota. We also limit usage to 10 basic (free) services. Because BOSH can't update values directly from manifest changes (https://github.com/cloudfoundry/bosh/issues/1201), add a step in post-deploy job to use cf-cli to update quotas. Add a small script to help extract quota settings from manifest.
How to review
Deploy. View quotas (cf quotas) and verify default has been updated to new values. Try exceeding it - you should not be able to.
Who can review
not @mtekel

Works as designed.
This removes any update made manually by cf update-quota. But I've confirmed with @Rory80Hz that this is the expected behaviour.


Overall layout of react-dates is a mess

When using <DateRangePicker>, this shows:

The Drag and Drop area (my component) as you can see was dragged to the bottom. And the sizes and layouts were disrupted!
Why is that? Were there errors in installing the package that resulted into this?

Are you using the included CSS? https://github.com/airbnb/react-dates#theming may help.

Hi @angelosembrano, this is in fact caused by not including CSS. https://github.com/airbnb/react-dates/issues/45 also has some more information and we're working on adding that information to the README.

Closing this because the solution is to load in the CSS. If you have information to add to the README on this process, please add a PR or your thoughts to #45


simplify please ack decorator

Simplify please ack decorator for inclusion in AIP 2.0 (#632). I simplified it to the most basic version where you can only indicate whether you want an ack on RECEIPT or OUTCOME.
I'm not sure if I removed too much, but I think this basic variant will be quite easy to implement for agents, while still providing a lot of value.
@dhh1128 as the original author of the RFC, what do you think?

From WG call:

Make on an array
Describe generic behaviour for OUTCOME

when interaction is done / outcome reached -> ack sent
Allow protocols to overwrite OUTCOME behaviour


Ability to produce a graphql schema file for use in other clients

Basically what the title says. juniper is very good at merging rust and graphql together so strictly, but outside of the server, there is no way to ensure the types match those defined in the server. It would be useful if there was a way to produce a .graphql file with the schema (possibly at build time) so that other clients could verify and type the requests at compile time or runtime.

You can use the juniper::introspect method

Actually, on master we have been better support:
https://graphql-rust.github.io/juniper/master/schema/schemas_and_mutations.html#converting-a-rust-schema-to-the-a-hrefhttpsgraphqlorglearnschematype-languagegraphql-schema-languagea

Is it possible to get a json schema like introspect provides in a similar fashion (i.e. without setting up an entire context)?  The schema language doesn't quite do everything that the question poses.

What is missing from the .graphql file that is generated?

introspect gives back the GraphQLObject response of from introspection in json form.


Some function may occurred “divide by zero” when specially illegal input stream (generated by AFL)

I am using AFL for SMTA fuzzing  security vulnerability checking. SMTA used MSDK libs for video decode/encode/vpp…
Now we found 2 MSDK FloatingPointException ( “divided by zero” may occurred in some functions when using specially illegal input streams) when testing SMTA.
The source file mfx_h264_enc_common_hw.cpp is at https://github.com/Intel-Media-SDK/MediaSDK/blob/master/_studio/mfx_lib/shared/src/mfx_h264_enc_common_hw.cpp. Line 532 in function GetMaxNumRefFrame and line 971 in function calculateMfeTimeout, which are highlight.
mfxU16 GetMaxNumRefFrame(mfxU16 level, mfxU16 width, mfxU16 height)
{
mfxU32 maxDpbSize = 0;
……
mfxU32 frameSize = width * height * 3 / 2;
return mfx::clamp(maxDpbSize / frameSize, 1, 16);
}
mfxU32 calculateMfeTimeout(const mfxFrameInfo& info)
{
//Just calculate based on latency expectation from framerate in microsecond now, can be changed in future
return mfxU32((mfxU64)info.FrameRateExtD * 1000000 / info.FrameRateExtN);
}

SMTA source code is at https://github.com/VCDP/SMTA. please refer 7.2 for SMTA building in https://github.com/VCDP/SMTA/blob/master/doc/SMTA_User_Guide.pdf.
bug reproduce SMTA command-line:
StreamingMediaTranscoder -params -i::h264  <crash_input>  -o::h264 -w 480 -h 720 /dev/null

please give me a link to upload the crash_inputs?

Update input files for this issue.
crash_inputs.zip


Made changes to the dockerfile

Probably worth addressing the MyPy type errors in the pre-commit checks. Once that's done this is good to merge.


Expose dataIndex during onCellClick event handler

The onCellClick handler currently exposes the colIndex and rowIndex but these are relative to what's currently on screen rather than the complete data array. This change additionally exposes the dataIndex, which does relate to the data array.
I've put together an example of the problem this will help fix. If you click a row in the list, the employee's name will appear below the table. If you then filter the table to "Bob" and click the filtered row, the name will say "Joe James" because rowIndex is "0" and that's the first item in the data array. dataIndex would return "2" allowing you to accurately access Bob's details.
The real-world problem I'm trying to solve here is making the row like a hyperlink, so clicking on a cell in it will open that employee's details.

Coverage remained the same at 78.484% when pulling eb63e79f85581c4684e2f9b1f1943d7cde421c0f on gregtyler:patch-1 into<PHONE_NUMBER>fbd011de0fead8ffb9aba91d000bd2 on gregnb:master.

Ok, let me look into this today because I was working on something similar but it's been a while. Apologies to everyone I've just been swamped with my full time job

Thanks Greg, let me know if I can help at all 😃

Can you merge this pr ?


Text document save() API closes the Untitled document which is saved

Does this issue occur when all extensions are disabled?: Yes

Version: 1.77.0-insider (user setup)
Commit: 7f329fe6c66b0f86ae1574c2911b681ad5a45d63
Date: 2023-03-29T09:56:00.839Z
Electron: 19.1.11
Chromium: 102.0.5005.196
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22518
Sandboxed: Yes
Steps to Reproduce:

Checkout the save() API here: https://code.visualstudio.com/api/references/vscode-api#TextDocument
Have an untitled file open, and attempt to save the document using the API
Textdocument.isClosed property is set to true

Also TextDocument.uri object is not updated to point to the updated saved URI. For Python extension it leads to issues like the following where wrong URI is sent to terminal, also notice that the file disappears:

Follow up question: how can one know the URI of the file after save?

Here's the workaround we're currently using:
const deferred = createDeferred<Uri>();
this.documentManager.onDidSaveTextDocument((e) => deferred.resolve(e.uri));
const commandManager = this.serviceContainer.get<ICommandManager>(ICommandManager);
await commandManager.executeCommand('workbench.action.files.save', file);
const savedFileUri = await deferred.promise;

The uri of a document doesn't change and it is expected that saving an untitled documented emits a close event for it. There is unfortunately no easy way to know what the new uri is, using the save or open events is the best you have. To double-check you can compare the contents

Related: https://github.com/microsoft/vscode/issues/124117
We could still make this a bit easier on extensions and return a URI from a command such as workbench.action.files.save or workbench.action.files.saveAs. Would that be helping?
As for the underlying issue: code editor text models cannot change their model Uri. As such, we cannot transform an existing model from untitled to file URI. I still think it would be great to have though because there are other issues that originate from this limitation such as not being able to preserve the undo/redo buffer: https://github.com/microsoft/vscode/issues/17977

That would be great.

As such, we cannot transform an existing model from untitled to file URI

Gotcha. Right from a user's perspective TextDocument is able to update other properties such as isUntitled, isClosed but not the URI itself.

it is expected that saving an untitled documented emits a close event for it.

@jrieken Intiutively TextDocument.save feels like this should be the same as workbench.action.files.save, so I believe doc should be updated to add these nuances there:

Ideally, I would have expected it to return something like Uri | undefined instead of boolean to indicate the success.

Btw, it is possible for a user to cancel the dialog to pick a save location for an untitled file, so you almost need 3 states (error, success, cancelled). But anyway, API changes like this are not in the cards and would require new methods.
I can see if the command can return the URI.

@karrtikr please give workspace.save and workspace.saveAs a try and let me know how it goes.

The APIs works perfectly for me 👍

@karrtikr the saveEditor API is has now been finalised and will be available in stable for our January release early February.

Verification:

follow along https://github.com/microsoft/vscode/issues/180666 but without enabling or configuring proposed API


CB-16208: [API E2E] testSDXResize is constantly timing outThere were two issues with the test.

The test was waiting for the flow to be complete before checking the status. This is wrong as the status of the cluster would be in running rate when the flow chain is complete. Updated the test to fix this
After the datalake is resized, the old datalake is deleted. There could be a chance for the test to poll for the flow chain status after the old datalake is deleted. This could result in a validation error. This is also addressed.

See detailed description in the commit message.

Here is the screenshot where the test ran successfully when ran locally.


beautify.onSave saving loop

When in set
"beautify.onSave": true
on save vscode starts to constantly save and change the file repedeatly times. Only way to stop it, is to switch to another file.
This is really annoying since the file is constantly marked as saved/nonsaved and also clears the undo history.

Apparently this was caused by beautify wanting to remove the final new line and my .editorconfig told vscode to insert an newline on save. So they both removed and added it constantly.

@Keale2 Apparently we are running into this. What was the setting you had to change in .editorconfig and settings.json for vscode?

awesome, adding .jsbeautifyrc with below content solved this issue:

{
    "html": {
        "end_with_newline": true
    },
    "css": {
        "end_with_newline": true
    },
    "js": {
        "end_with_newline": true
    }
}

@abhishekisnot, you can put the "end_with_newline" definition in the base JSON block if you're setting it the same for all three, no need to put it in three times.


How to pass optional data

I want to send optional data like. ["url" => "https://www.google.com"]
I don't see any option.
Additionally, how can I get a downstream response for failed tokens?

Hi @bluelupin,

On next release you can pass optional data.
For response from firebase you just need to return or dump like this:

return Larafirebase::withTitle('Test Title')
            ->withBody('Test body')
            ->sendNotification($this->deviceTokens);

And you get response like this:
{
    "multicast_id":<PHONE_NUMBER>109722248,
    "success": 0,
    "failure": 1,
    "canonical_ids": 0,
    "results": [
        {
            "error": "InvalidRegistration"
        }
    ]
}

@bluelupin I had the same problem, you can use fromArray on both methods(sendNotification and sendMessage) that will bypass the title, body ,... . Below is the part that builds the dataArray.
$fields = array(
            'registration_ids' => $tokens,
            'notification' => ($this->fromArray) ? $this->fromArray : [
                'title' => $this->title,
                'body' => $this->body,
                'image' => $this->image,
                'click_action' => $this->clickAction
            ],
            'priority' => $this->priority
        );

@alchalade yes you can use fromArray if you want to pass whatever you want for notification or data object.

Hi @bluelupin,

On next release you can pass optional data.
For response from firebase you just need to return or dump like this:

return Larafirebase::withTitle('Test Title')
            ->withBody('Test body')
            ->sendNotification($this->deviceTokens);

And you get response like this:
{
    "multicast_id":<PHONE_NUMBER>109722248,
    "success": 0,
    "failure": 1,
    "canonical_ids": 0,
    "results": [
        {
            "error": "InvalidRegistration"
        }
    ]
}

Is there any way we can know which were the failed tokens.


Mark beta failures to fix

This is a temporary marking of tests with two tags - beta, to indicate that this test is failing in beta only, and flaky, to prevent the tests from running until they pass while running against beta build and nightly builds.
Plan: work on getting these features to pass in beta and browsertest orgs.

let's only use the @beta tag. I'm already using @flaky in the existing builds and using these tags here isn't what we want, see https://github.com/SalesforceFoundation/Cumulus/blob/dev/cumulusci.yml#L81
also, we'll have to change cumulusci.yml to add '--tags ~@beta' to the build


Getting ##[error](intermediate value)(intermediate value)(intermediate value) is not iterable

Hey, I really like this action but sadly somehow it fails currently with my CHANGELOG.md with error:
  ##[error](intermediate value)(intermediate value)(intermediate value) is not iterable

See log here: https://github.com/svenstaro/site24x7_exporter/runs/842970939?check_suite_focus=true
My CHANGELOG.md: https://github.com/svenstaro/site24x7_exporter/blob/eca02aa02160f9c22d9145eb1bc5dfb08988b9b8/CHANGELOG.md
I believe my CHANGELOG.md is compliant to the expected format but obviously something throws the action off.
The vague output makes it fairly hard to see what the error is, sadly.

I just tested this on v1.2.0 of this Action and it works totally fine!

Thank for taking the time to report this issue.
It seems to be related to the SEMVER regex that has been updated in v1.3.0. I'm looking into it currently.

@svenstaro I saw you use ## [Unreleased] - ReleaseDate as a title for your unreleased section. Can you try to remove the " - ReleaseDate" to see if it works for you?
Since v1.3.0 the parser tries to parse the date too and I guess it might fail to parse "ReleaseDate".

I did it the reverse way: I added  - ReleaseDate to your tests and it works fine which is weird. I didn't try removing it yet due to time constraints. Perhaps you could add my CHANGELOG to this repo in the test flow and see if it breaks and debug it that way? Seems to me that this would be a quicker turnaround for you while debugging this.

Ok, just tried with your file. Check out the action logs:

no modification: https://github.com/mindsers/changelog-reader-action/runs/850387930
without the - ReleaseDate: https://github.com/mindsers/changelog-reader-action/runs/850464631

It is working without the fake date. I wonder if there is a use case where we need to use fake dates...

There is a use case which is that I use the ReleaseDate as a substitution marker for my automatic release tool. I imagine it's quite a common case and it doesn't really hurt to have that there. I'd like it to also work with your action as it worked before.

substitution marker for my automatic release tool

It's actually a great use case. So, I made a fix that make the action ignore the fake dates. I'll release it today I think.

Fix released!! 🎉

Tested and looking good! Thanks.


Nonlinear dyna

For now, this PR lacks unit tests. I think we'll be able to add them in the feature.

Please give me your early feedback on this PR, but don't merge it. I need to polish the code further.


Escape postgres database name

When using buffalo to generate a project using postgres that had a hyphen in it, it gave a syntax error.

Thanks!


Build dlls for 0.9

@reuvenpo I think this takes care of your issue about missing dlls in go-cosmwasm. (Which will help test wasmd for sure).
If you approve, I will merge and we will see if this works

We will want to remove this line before merging to master though. Should we open an issue on the topic, or would mentioning this in the description of the 0.9 PR be enough?

I think mentioning it on the 0.9 PR is fine. It doesn't hurt even if merged, but always good to clean up


ipv6 support in botocore without using dualstack

Is it possible to use IPV6  address directly in a url
eg
http://[999:1900:228f:20:172:19:9:35]/
Another user raised this issue and proposed a solution that was never merged.
https://github.com/boto/botocore/issues/1207

I'm not able to find a PR for this.  It looks like no one actually sent a PR.  Could you tell me more about how you're using ivp6 in URLs?  Is there a specific AWS endpoint you're using?  Not opposed to this, just would like to understand the use case a little better.

Hi Jamesis,
here is the use case:
client = boto3.client('s3', verify=False, endpoint_url='[2001:1900:228f:40:1:1:1:1]', aws_access_key_id=aws_access_key_id, aws_secret_access_key=aws_secret_access_key, config=Config(s3={'use_dualstack_endpoint': True}))
client.put_object(Bucket='vault-test', Key='Test-pd', Body='This is a test.')

@jamesls  - do you have any feedback to the above query? thanks for any feedback..

@jamesls - Hi Jamesis, we need this feedback on this rather quickly. Can you please help expedite this ? or get someone to help with this ?

Hi sorry for the delay, looks like the closing-soon-if-no-response label wasn't removed and this issue fell through the cracks.  Thanks for giving an example.  Out of curiosity, is there a reason you're specifying a hardcoded ipv6 address?  Just trying to understand this better so I can make a good case for prioritizing it.  If someone wants to send a PR for this issue, we should be able to review it quickly.


musl libc & Alpine based Docker images compatibility

DeepLearning4J is incompatible with musl libc & Alpine based Docker images
I tried running a DL4J application inside a Docker container that is based on an Alpine Linux Image (eclipse-temurin:8-alpine). When starting the application, I get a fatal error. The Java VM dies.
expected behavior: I can run a DL4J application inside a musl libc based Linux Docker container like Alpine
encountered behavior: When I run my DL4J application in a Docker container that uses Alpine Linux as its base image, I get a fatal error.
14:45:20.692 [main] INFO org.nd4j.linalg.factory.Nd4jBackend - Loaded [CpuBackend] backend
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00000000000021c6, pid=8, tid=0x00007f3c87a8cb38
#
# JRE version: OpenJDK Runtime Environment (8.0_332-b09) (build 1.8.0_332-b09)
# Java VM: OpenJDK 64-Bit Server VM (25.332-b09 mixed mode linux-amd64 compressed oops)
# Problematic frame:
# C  0x00000000000021c6
#
# Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /opt/dl4j/hs_err_pid8.log
#
# If you would like to submit a bug report, please visit:
#   https://github.com/adoptium/adoptium-support/issues
#
Aborted

I attached the file hs_err_pid8.log
One suspicious log entry is:
R12=0x00007f3c65aaa558: <offset 0x23e558> in /root/.javacpp/cache/example-1.0.0-SNAPSHOT-shaded.jar/org/bytedeco/openblas/linux-x86_64/libquadmath.so.0 at 0x00007f3c6586c000
R13=0x00007f3c8891a880: <offset 0x96880> in /lib/ld-musl-x86_64.so.1 at 0x00007f3c88884000
R14=0x00007f3c87a8cb38 is an unknown value

I guess /root/.javacpp/cache/example-1.0.0-SNAPSHOT-shaded.jar/org/bytedeco/openblas/linux-x86_64/libquadmath.so.0 is linked against glibc, but on Alpine Linux there is no glibc. There is gcompat, but that doesn't fix this issue.
I ran:
objdump -x /root/.javacpp/cache/example-1.0.0-SNAPSHOT-shaded.jar/org/bytedeco/openblas/linux-x86_64/libquadmath.so.0

and it shows:
Version References:
  required from libm.so.6:
    0x09691a75 0x00 05 GLIBC_2.2.5
  required from libc.so.6:
    0x0d696914 0x00 08 GLIBC_2.4
    0x0d696913 0x00 07 GLIBC_2.3
    0x09691a75 0x00 06 GLIBC_2.2.5
    0x06969190 0x00 04 GLIBC_2.10

So I guess these libraries were actually compiled and linked with glibc.
Version Information

Deeplearning4j version: 1.0.0-M2
Platform information (OS, etc): Alpine Linux v3.15.4, libc:musl - unknown musl - unknown

Additional Information
I attached an example project (example.zip) and corresponding Dockerfiles to replicate the problem. There is a README.md inside with instructions.
Thanks in advance.
Regards,
Andreas

@schipplock you'd have to build at least a few things that aren't based on glibc I believe. We heavily depend on a lot of javacpp projects: https://github.com/bytedeco/javacpp-presets - I don't really ever see people ask for musl.
There's also the c++ lib itself which uses glibc everywhere.
That's fixable building from source but your mileage will vary with the different depdendeices. What subset of this are you hoping to get working?
Just running a model on a cpu?

@schipplock just following up but sorry I seriously can't make this a priority without more information and clear use cases. I know a lot of folks use musl but it would basically require a separate classifier.
What I might be able to do is publish a separate classifier, but even then I need to find a way to make openblas work. That won't solve this problem for all backends either.
Our openblas that we depend on upstream by default uses glibc already. The best I can recommend if you want to help here would be to help us publish a separate musl that builds also openblas linked against musl. I'm not sure how it would work but you would also have to submit a separate issue upstream at: https://github.com/bytedeco/javacpp-presets for an openblas presets.
Chances are due to the marketshare though, you'll probably just get told to send a PR in a similar fashion to what I'd be asking you to do here. I'd be happy to guide you on our end though if you want to tackle this.

Hi @agibsonccc,
thanks for your answer.
I can see this is a lot of work, and I'm not asking nor expecting that anyone is  making it work on musl based environments :). I have to admit: I was expecting someone telling me that musl is simply not supported. I'm not the biggest musl fan either, simply because of compatibility reasons. This here is a good example. People build, link and test their stuff with glibc.
In the meantime, I could convince Payara to provide glibc based Docker images in the future :). My employer is paying for Payara Enterprise, it's not paying anything for DL4J.
Again, thanks for your feedback.
Regards,
Andreas

@schipplock depending on your team's needs commercial engagements are possible. Please ping<EMAIL_ADDRESS>if you think getting a dedicated build + some other support might be interesting. Thanks for understanding!  Helping the jarkata ee ecosystem is definitely of interest to us as a fellow eclipse project but I also understand the trade offs your team has to make.

Hi Adam,
we are more interested in "some other support". I'll forward this to my collegue (I'm just the poor IT guy :P).
Regards,
Andreas


Language Label Error

Your entry Lule (SAPHON 1930) is incorrectly associated with Glottolog language Olu'bo (olub1238).  It should be associated with Glottolog language Lule (lule1238).

@Taivo55 -- thanks for pointing this out. We have fixed this in the development version of phoible:
https://github.com/phoible/dev/issues/245
and when we release a new version of the CLDF data and upload it to the website, this will be corrected.
FYI @drammock, @xrotwang


[joy-ui][Autocomplete] Typescript & React runtime error using Autocomplete

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Trying to use Joy UI's Autocomplete in a controlled state as per the docs, I get a TypeScript error:

I also get a runtime error on the browser from React.

I've tried passing objects with value and label as options, passing a value prop, but I also get the same typescript error. It expects it to be an array of string or the inferred type from option. I also get errors if I pass it an array.
Am I using this component wrong, or is there an issue with these types?
Reproduction:
(MUI Joy's own example has type errors)
https://codesandbox.io/s/cp2zmj?file=/Demo.tsx
Current behavior 😯
No response
Expected behavior 🤔
No response
Context 🔦
No response
Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

@Onurfesci Thanks for reporting this ~ I can reproduce it in our docs example

I have the same issue as above

I will take a look tomorrow.

Based on the CodeSandbox you provided, I guess there is something wrong with CodeSandbox because I copied the code and put it in my local react-vite (from scratch) + Joy UI without any type errors.
Could you provide the code that you use in your local project?

@siriwatknp you can check on this codesandbox, press f12, click on the dropdown icon and the error will occur
https://codesandbox.io/p/sandbox/spring-browser-mypv5f

@siriwatknp you can check on this codesandbox, press f12, click on the dropdown icon and the error will occur
https://codesandbox.io/p/sandbox/spring-browser-mypv5f

I tried but what's the error that I should see? Is it related to this issue?

You can see the issue like bellow, for the issue on the issue "Property id ...", I cannot duplicate it on my codesandbox

Looks like a similar issue to https://github.com/mui/material-ui/issues/39474
Though I can't reproduce the issue with Joy anymore (even with Next.js like in @sang-au's sandbox) https://codesandbox.io/p/sandbox/sleepy-liskov-fvjhmf?file=%2Fsrc%2Fapp%2Fpage.tsx%3A1%2C1-21%2C1

I copied my implementation as-is to codesandbox, but I'm not seeing the same issues that I'm seeing on my VSCode. Really strange...
Sandbox repro (No TS Errors): https://codesandbox.io/s/billowing-smoke-vpcn32?file=/src/Demo.tsx
Local VSCode (Getting type errors on 3 props: value, getOptionLabel, onChange). The inference seems to be completely different. For some reason, on my VSCode, it is inferring value as an array of string or the inferred option type, rather than the option type itself. If I @ts-ignore, it works perfectly so I'm not implementing it wrong. See the images below.
Type error on value:

Type error on getOptionLabel (Thinks option is the option or a string?):

Type error on onChange:

Notes:

@mui/joy and typescript are both the latest versions on my repo, same as the codesandbox
Have tried restarting VSCode, TypeScript server multiple times

Does anybody have an idea why this is happening? Could it be a tsconfig issue with Next.js projects, clashing with Joy's type inferences?

I got the codesandbox above to show some TS errors now. Basically, if the value a string and the options are an array of objects, the Autocomplete isn't happy. So what I want to ask to the Joy team is...

Is this intentional? Does value HAVE to be the same type as one of the options?
If this is the case, the isOptionEqualToValue field doesn't have much use.
I think value should be able to be a string, for example an id of one of the options, while the options can be objects of any type.
Perhaps the generic type of Autocompleteprops should allow for another parameter, TValue? This way the default type casting could be overriden if needed?

Perhaps the generic type of AutocompleteProps should allow for another parameter, TValue? This way the default type casting could be overridden if needed? Better yet, I think the default casting of value type should be from the value prop itself

I think so, this would let developers gain more control over the types they want. Since this is related to Base UI useAutocomplete, I am tagging @michaldudak and @mj12albert for further discussion.

This is already being tracked in #23708.


[Feature Request]: Avatar + User Card component

Description
As a user, it would be greatly beneficial to have an avatar + user card component available, where an icon or custom image could be added that included the user's name and, if desired, any additional information pulled from an API or included JSON or YAML file.
Current areas where this component could be utilized:

Cards
Table
Tiles
Top navigation (replacing the existing Customer Name drop-down with generic icon)

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate requests

Thank you for the update @gethinwebster. If desired, I would be happy to submit a design for consideration, following the contribution guidelines.

We recently released an avatar component. It is more focused on the first part of the request here, but could be combined with other components to create richer user information experiences.


Play with the Smooth Terrain demo

The scene is located at:
https://github.com/Zylann/voxelgame#runnable-scenes
https://github.com/Zylann/voxelgame/tree/master/project/smooth_terrain

This is exactly the demo I was searching for. Probably the one for https://github.com/tinmanjuggernaut/voxelgame is more impressive but let's start with this one, in which you can add spheres and you can also remove sphere. But not sure if there are collisions samples. Because you cross the terrain and ... it is empty :(

I would like to play with the demo from Tokisan: https://www.youtube.com/watch?v=i-AskdAeOnI which is https://github.com/tinmanjuggernaut which has a fork of https://github.com/tinmanjuggernaut/voxelgame
Let's try to find the code for the demo.

The problem with the above demo is that in a small window it has 30-35 FPS. So the performance is not good.

Ok, let's go with the demo.The goals:

Understand how the player walks, jumps and fall down
Understand how the caves are done: But this is pretty simple. just create empty spheres probably.
Analyze the performance

All the physics is not working at voxel level, but at block level probably according to how the collision shape is created (covering the surface of a block with a concave polygon).

The FPS demo is impressive. Let's continue working in this line to understand all the details.


feat: adding secrets-manager access poilcy with SecretsReader role

For: https://github.ibm.com/project-fortress/pm/issues/15880
To grant SCC access to the customer's secret manager, the trusted profile must be assigned a policy for the secret manager service with the SecretsReader role.

Not needed.


Fix SonarQube Issue : Remove this use of "defaultUnitSystem"; it is deprecated

Describe the technical debt
This issue addresses the elimination of the usage of 'defaultUnitSystem' as it is outdated and deprecated, which may lead to technical debt and potential future compatibility issues.
Location: src/.../dennisguse/opentracks/settings/PreferencesUtils.java

Consequences of the Issue:

Redundancy and Code Maintenance.
Maintainability and Debugging.

Why It Was Necessary to Be Fixed:

Code Redundancy.
Consistency and Clarity.

Proposed solution
Define the default UnitSystem and initialize it with UnitSystem.METRIC. This should be used when the records are filtered using firstFirst() and orElse() methods.

Technical information

Device: [e.g. Nexus 7 (2013)]
OS: [e.g. LineageOS 15.1]
OpenTracks version: [e.g. v3.2.4]
OpenTracks commit id (only for nightly builds): [e.g. 4b333bb]

Link to the commit

A code change to resolve the technical debt is now available as part of PR https://github.com/rilling/OpenTracksConcordia/pull/139.
Hence closing the issue.


Is IKCP is good for live file streaming

I am currently developing  an  app for file streaming like video file to 1 or many clients .I have done this using udp but due to packet lost I am facing many issues .So I was looking into your library it is looking good will it work for my needs ?. I want to know before I give it a try

kcp has already been used successfully in video / audio streaming:
https://github.com/skywind3000/kcp/wiki/Success-Stories
You can try kcptun as a beginning


Changeling Sting Update

updated sprite and added to vending machine when hacked by emag

About the PR

I noticed that this drink has an outdated texture and I wanted to update it, I also added to the cola machine, it appears after applying the emag, you get two cans.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: when you emag a cola machine you will get two cans of Changeling Sting
tweak: Changeling Sting sprite changed

type: vendingMachineInventory
id: RobustSoftdrinksInventory
startingInventory:
DrinkColaCan: 4
DrinkGrapeCan: 2
DrinkRootBeerCan: 2
DrinkIcedTeaCan: 2
DrinkLemonLimeCan: 2
DrinkFourteenLokoCan: 2
emaggedInventory:
DrinkNukieCan: 2
DrinkChangelingStingCan: 2

everything seems to be correct, just like on other machines

cl should explicitly say emagging since you could say cutting mgr wire is hacking it too

maybe a problem with the sprite?

no just change the changelog to say emag not hack

no just change the changelog to say emag not hack

I changed the descriptions, but how do I pass the final check?

ignore it ci is dead rn

I light a signal fire to attract attention


Gem bumps for CVE-2015-3226, CVE-2015-3227, CVE-2015-1840 and CVE-2015-3225

@dsander @knu I'm away this weekend.  If these look good to you, could one of you merge?

Deployed to production add found no issues so far.

Thanks!
On Friday, June 19, 2015, Dominik Sander<EMAIL_ADDRESS>wrote:

Merged #868 https://github.com/cantino/huginn/pull/868.
—
Reply to this email directly or view it on GitHub
https://github.com/cantino/huginn/pull/868#event-335658064.

No issues noted here, either.


Scala 2.12 compilation fixed

Fixed the compilation issues for Scala 2.12 by changing the import statement.

Nice import fix!


Enhancement: Wedding Countdown

Countdown Clock: http://flipclockjs.com/ --> https://github.com/stevermeister/angular-flipclock
Countdown Ribbon: https://github.com/paveldemyanenko/ng-countdown-ribbon


Fix missing rotations display and sorting by Stock in the main list

Rotation column is empty regardless of the regex. This seems to be a display bug. This pull request brings it back.

Additional explanations at the request of chmorgan:
It seems like Qt UI doesn't recognise the non-string type for a column, so it needs to be explicitly converted to string (the  3b8595e commit where str() is added around correction variable). Here's the result:

Sort order commit (191cd95) was different from the database column order list, so clicking the header would use a different column for sorting than the one displayed in the header, thus sorting wouldn't work as expected.


Improve stubs in av.video

I wanted to also add codec_context typing in av.audio.stream.AudioStream, but that caused conflict...

You should just overwrite your previous commit then force push. It's okay when it's on your branch.

Should I open another PR for changing codec_context in av.audio.stream.AudioStream? Or should I just leave that to you?

Once a PR is merged in main. You can't revert it.
Just make a new PR.

Sure! Opened #1321


Fix for Get-MasterZoneFile HTTP 406 error

This change addresses issue #2 by adding an optional -AcceptHeader parameter to Invoke-AkamaiRestMethod which defaults to application/json. Get-MasterZoneFile was then modified to use the new parameter to set the accept header for its request to text/dns.

Hi @rmbolger, I have implemented this on a variety of other cmdlets but using the AdditionalHeaders object (e.g. in Get-CPSCertificateHistory), which will overwrite any headers, including Accept, that you might add to the hashtable. I am happy to discuss adding the accept option as you have suggested though, but wouldnt want to add more than maybe accept and content-type. What do you think?

I totally missed that the AdditionalHeaders param would overwrite the hard coded Accept header. No need to complicate things. I'll just tweak things to use that instead.

I added commits that revert the change to Invoke-AkamaRestMethod and change the fix to use the AdditionalHeaders param. Did you want me to submit a new PR from scratch? Or just re-open this one?


Add upper bound to repeat operator (fix #27)

I don't understand what the error log means at https://travis-ci.org/lihaoyi/fastparse/jobs/88978992#L3108

Looks like just a flake
On Tue, Nov 3, 2015 at 4:36 AM, Alain Béarez<EMAIL_ADDRESS>wrote:

I don't understand what the error log means at
https://travis-ci.org/lihaoyi/fastparse/jobs/88978992#L3108
—
Reply to this email directly or view it on GitHub
https://github.com/lihaoyi/fastparse/pull/49#issuecomment-153339618.

That is all. Thanks for the PR =)

Oh I didn't mean to get rid of the other cases! Just to add that one on top of them.

Sorry! I misunderstood. I will push the ab4c example back.

Can you rebase this on top of master?

Looks like a real breakage

Skipping it works for me :+1:

That should really be all. Sorry for so much back and forth!

Merged, Thanks!!!


unsigned integer:  strutils.nim(1091)

I am getting error while opening ttop:
@ubuntu:~$ sudo ttop
invalid unsigned integer:
strutils.nim(1091)       parseUInt

@adrianmihalko
Can you please run ttop-debug version? It will output stacktrace for the error. Thank you

@adrianmihalko
Please find that the last version provides more detailed error info even without stacktrace (non -debug)

I can’t reproduce the issue for now, after a restart. I’ll add more info when it appears again. Thank you


"mute" and "ban" options for self

In the list of online users, the "mute" and "ban" appear for self. Those should not appear 😛
One step further, put in a "(you)" at the end of naming self, for ultimate clarity on why those links are not present.

👍


[tool_crash] ArgumentError: Invalid argument(s): Uri c:/Users/tinoc/pos_devices_app/fonts/Roboto-Regular.ttf must have scheme 'file:'.

Command
flutter run

Steps to Reproduce

...
...
...

Logs
ArgumentError: Invalid argument(s): Uri c:/Users/tinoc/pos_devices_app/fonts/Roboto-Regular.ttf must have scheme 'file:'.
#0      WindowsStyle.pathFromUri (package:path/src/style/windows.dart:86:7)
#1      Context.fromUri (package:path/src/context.dart:1040:40)
#2      ManifestAssetBundle._resolveAsset (package:flutter_tools/src/asset.dart:1164:47)
#3      ManifestAssetBundle._parseAssets (package:flutter_tools/src/asset.dart:957:34)
#4      ManifestAssetBundle.build (package:flutter_tools/src/asset.dart:308:54)
<asynchronous suspension>
#5      ResidentWebRunner._updateDevFS (package:flutter_tools/src/isolated/resident_web_runner.dart:544:26)
<asynchronous suspension>
#6      ResidentWebRunner.run.<anonymous closure> (package:flutter_tools/src/isolated/resident_web_runner.dart:322:41)
<asynchronous suspension>
#7      asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:24)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.22.0, on Microsoft Windows [Version 10.0.22621.3593], locale en-ZW)
    • Flutter version 3.22.0 on channel stable at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5dcb86f68f (5 weeks ago), 2024-05-09 07:39:20 -0500
    • Engine revision f6344b75dc
    • Dart version 3.4.0
    • DevTools version 2.34.3

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.9.7)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.9.34902.65
    • Windows 10 SDK version 10.0.22621.0

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[✓] VS Code (version 1.90.0)
    • VS Code at C:\Users\tinoc\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.90.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22621.3593]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 125.0.6422.142
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 125.0.2535.92

[!] Network resources
    ✗ A network error occurred while checking "https://pub.dev/": The semaphore timeout period has expired.

    ✗ A network error occurred while checking "https://storage.googleapis.com/": The semaphore timeout period has expired.

    ✗ A network error occurred while checking "https://maven.google.com/": The semaphore timeout period has expired.

    ✗ A network error occurred while checking "https://cocoapods.org/": The semaphore timeout period has expired.

    ✗ A network error occurred while checking "https://github.com/": The semaphore timeout period has expired.

! Doctor found issues in 3 categories.

Flutter Application Metadata
Type: app
Version: null
Material: false
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 5dcb86f68f239346676ceb1ed1ea385bd215fba1
Plugins
app_links-6.1.1
connectivity_plus-5.0.2
gtk-2.1.0
path_provider-2.1.3
path_provider_android-2.2.4
path_provider_foundation-2.4.0
path_provider_linux-2.2.1
path_provider_windows-2.2.1
shared_preferences-2.2.3
shared_preferences_android-2.2.2
shared_preferences_foundation-2.4.0
shared_preferences_linux-2.3.2
shared_preferences_web-2.3.0
shared_preferences_windows-2.3.2
sqflite-2.3.3+1
url_launcher-6.2.6
url_launcher_android-6.3.2
url_launcher_ios-6.3.0
url_launcher_linux-3.1.1
url_launcher_macos-3.2.0
url_launcher_web-2.3.1
url_launcher_windows-3.1.1

Closing this as a duplicate of https://github.com/flutter/flutter/issues/116737


Prompt for Incremental Naming

I would like Incremental Naming (triggered when a moved/copied file repeats the name of an existing file) to prompt if I’d like to increment, overwrite or cancel.
This is similar to the dialog from the "explorer.incrementalNaming": "disabled" config, but with an extra button for incrementing.
Windows Explorer can also do incremental naming by selecting both files in “let me choose”.

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Assertion `(c->flags & REDIS_SUBSCRIBED || c->flags & REDIS_MONITORING)' fails when calling "SUBSCRIBE aaa" and then "UNSUBSCRIBE aaa bbb"

Hiredis crashes after calling redisAsyncCommand(c, NULL, NULL, "subscribe aaa") and then redisAsyncCommand(c, NULL, NULL, "unsubscribe aaa bbb").
The code is simply modified from the example "examples/example-ae.c". I compiled the code like this: AE_DIR=~/redis/src make hiredis-example-ae -j16:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <signal.h>

#include <hiredis.h>
#include <async.h>
#include <adapters/ae.h>

/* Put event loop in the global scope, so it can be explicitly stopped */
static aeEventLoop *loop;

void connectCallback(const redisAsyncContext *c, int status) {
    if (status != REDIS_OK) {
        printf("Error: %s\n", c->errstr);
        aeStop(loop);
        return;
    }

    printf("Connected...\n");
}

int main (int argc, char **argv) {
    signal(SIGPIPE, SIG_IGN);

    redisAsyncContext *c = redisAsyncConnect("<IP_ADDRESS>", 6379);
    if (c->err) {
        /* Let *c leak for now... */
        printf("Error: %s\n", c->errstr);
        return 1;
    }

    loop = aeCreateEventLoop(64);
    redisAeAttach(loop, c);
    redisAsyncSetConnectCallback(c,connectCallback);
    redisAsyncCommand(c, NULL, NULL, "subscribe aaa");
    redisAsyncCommand(c, NULL, NULL, "unsubscribe aaa bbb");
    aeMain(loop);
    return 0;
}

After starting a redis-server on localhost:6379 and run the code, it gave the error below: hiredis-example-ae: async.c:567: redisProcessCallbacks: Assertion `(c->flags & REDIS_SUBSCRIBED || c->flags & REDIS_MONITORING)' failed.

I looked into the codebase of Hiredis and found some possible problem in async.c: in redisProcessCallbacks, the function uses a loop while((status = redisGetReply(c,&reply)) == REDIS_OK) to get the reply of commands. For example, after processing SUBSCRIBE aaa, the redisProcessCallbacks function is now getting the reply of UNSUBSCRIBE aaa bbb. When the reply of UNSUBSCRIBE aaa arrives, all channels has already been deleted in __redisGetSubscribeCallback and then c->flags &= ~REDIS_SUBSCRIBED. Then when redisProcessCallbacks process UNSUBSCRIBE bbb's reply, the assertion assert((c->flags & REDIS_SUBSCRIBED || c->flags & REDIS_MONITORING)) fails.
void redisProcessCallbacks(redisAsyncContext *ac) {
    redisContext *c = &(ac->c);
    redisCallback cb = {NULL, NULL, 0, NULL};
    void *reply = NULL;
    int status;

    while((status = redisGetReply(c,&reply)) == REDIS_OK) {
        if (reply == NULL) {
            /* When the connection is being disconnected and there are
             * no more replies, this is the cue to really disconnect. */
            if (c->flags & REDIS_DISCONNECTING && sdslen(c->obuf) == 0
                && ac->replies.head == NULL) {
                __redisAsyncDisconnect(ac);
                return;
            }

            /* If monitor mode, repush callback */
            if(c->flags & REDIS_MONITORING) {
                __redisPushCallback(&ac->replies,&cb);
            }

            /* When the connection is not being disconnected, simply stop
             * trying to get replies and wait for the next loop tick. */
            break;
        }

        /* Keep track of push message support for subscribe handling */
        if (redisIsPushReply(reply)) c->flags |= REDIS_SUPPORTS_PUSH;

        /* Send any non-subscribe related PUSH messages to our PUSH handler
         * while allowing subscribe related PUSH messages to pass through.
         * This allows existing code to be backward compatible and work in
         * either RESP2 or RESP3 mode. */
        if (redisIsSpontaneousPushReply(reply)) {
            __redisRunPushCallback(ac, reply);
            c->reader->fn->freeObject(reply);
            continue;
        }

        /* Even if the context is subscribed, pending regular
         * callbacks will get a reply before pub/sub messages arrive. */
        if (__redisShiftCallback(&ac->replies,&cb) != REDIS_OK) {
            /*
             * A spontaneous reply in a not-subscribed context can be the error
             * reply that is sent when a new connection exceeds the maximum
             * number of allowed connections on the server side.
             *
             * This is seen as an error instead of a regular reply because the
             * server closes the connection after sending it.
             *
             * To prevent the error from being overwritten by an EOF error the
             * connection is closed here. See issue #43.
             *
             * Another possibility is that the server is loading its dataset.
             * In this case we also want to close the connection, and have the
             * user wait until the server is ready to take our request.
             */
            if (((redisReply*)reply)->type == REDIS_REPLY_ERROR) {
                c->err = REDIS_ERR_OTHER;
                snprintf(c->errstr,sizeof(c->errstr),"%s",((redisReply*)reply)->str);
                c->reader->fn->freeObject(reply);
                __redisAsyncDisconnect(ac);
                return;
            }
            /* No more regular callbacks and no errors, the context *must* be subscribed or monitoring. */
            assert((c->flags & REDIS_SUBSCRIBED || c->flags & REDIS_MONITORING));
            if(c->flags & REDIS_SUBSCRIBED)
                __redisGetSubscribeCallback(ac,reply,&cb);
        }

        if (cb.fn != NULL) {
            __redisRunCallback(ac,&cb,reply);
            if (!(c->flags & REDIS_NO_AUTO_FREE_REPLIES)){
                c->reader->fn->freeObject(reply);
            }

            /* Proceed with free'ing when redisAsyncFree() was called. */
            if (c->flags & REDIS_FREEING) {
                __redisAsyncFree(ac);
                return;
            }
        } else {
            /* No callback for this reply. This can either be a NULL callback,
             * or there were no callbacks to begin with. Either way, don't
             * abort with an error, but simply ignore it because the client
             * doesn't know what the server will spit out over the wire. */
            c->reader->fn->freeObject(reply);
        }
    }

    /* Disconnect when there was an error reading the reply */
    if (status != REDIS_OK)
        __redisAsyncDisconnect(ac);
}

I believe the PR #1036 would help in this case too.

In this case Redis will reply for each channel given in the unsubscribe command, even if a channel has not been subscribed to.
There is probably a need to collect how many channels, i.e responses that is expected when sending the unsubscribe command to avoid getting out of sync..

Today there is a check in __redisAsyncCommand() that an unsubscribe command can't be sent unless a subscribe command already has been sent, so one option could be that it also checks the channels (if given) to make sure they all exists internally..
The redisAsyncCommand(unsubscribe) would in this testcase return an REDIS_ERR.
But than might seem too obstructive and maybe not what you want.. filtering out unknown channels might also be an option..
Any thoughts @michael-grunder ?

Today there is a check in __redisAsyncCommand() that an unsubscribe command can't be sent unless a subscribe command already has been sent, so one option could be that it also checks the channels (if given) to make sure they all exists internally.. The redisAsyncCommand(unsubscribe) would in this testcase return an REDIS_ERR. But than might seem too obstructive and maybe not what you want.. filtering out unknown channels might also be an option..
Any thoughts @michael-grunder ?

Yeah I think it's better to filtering out unknown channels, because sending the test case of SUBSCRIBE aaa and UNSUBSCRIBE aaa bbb to redis-server will not return an error.


polymc: init at 1.0.4

Motivation for this change
Upstream maintainers of MultiMC have become increasingly hostile to 3rd party package maintainers (such as myself) ever since the addition of Microsoft account login was added to MultiMC. We've been asked to remove official MultiMC branding from our package in the past so this is a good opportunity to switch the MultiMC package over to the PolyMC fork who's mission is to swich away from official MultiMC branding and provide good 3rd party packages.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[x] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[x] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Can we insert the msa client-id in the buildprocess without referencing to the upstream client-id? Maybe with some kind of regex replacement actually referencing the field name and not its upstream value?
I am unsure if microsofts terms and services would allow for that: they treat the msa client-ids as secrets. If polymc decides to bake theirs into their repo it is their decision but I feel we should not needlessly replicate that attitude.
A more practical reasoning might be, that replacing a particular client-id will break (probably without any errors and thus no one noticing) when the upstream value changes.

Can we insert the msa client-id in the buildprocess without referencing to the upstream client-id? Maybe with some kind of regex replacement actually referencing the field name and not its upstream value?

From what I understand, Launcher_MSA_CLIENT_ID can be passed as a CMake option directory because https://github.com/PolyMC/PolyMC/blob/develop/CMakeLists.txt#L78 does not change the value if it's already in the cache.

@erdnaxe I removed the client ID and inserted it as a cmake build flag

@starcraft66 please fix the conflicts.
Further, I suggest putting the documentation updates in a separate commit.
(I don't remember correctly, but is the XML file machine-generated?)

I rebased and split the commit as requested.

LGTM. Waiting for OfBorg.

poke @AndersonTorres

Ping @starcraft66 to fix merge conflicts.

polymc still need overide myself msaClientID ?

@AndersonTorres done
@NIXFOREVERNIX You won't need to override it as PolyMC provides one by default but you can if you prefer to.

Muchas gracias!

@starcraft66 Thank you for your work!

@starcraft66 .desktop file exec  parameter empty
[Desktop Entry]
Version=1.0
Name=PolyMC
Comment=A custom launcher for Minecraft that allows you to easily manage multiple installations of Minecraft at once.
Type=Application
Terminal=false

Exec=

StartupNotify=true
Icon=org.polymc.PolyMC
Categories=Game;
Keywords=game;minecraft;launcher;

Okay, I see what's going on here. I think the desktop file you're looking at comes from the PolyMC source repo and isn't the one from this package. Admittedly, I don't use desktop launchers and didn't test this myself which I should do in the future.
I'll take a look at this and open a PR this week to fix it.

Okay, I see what's going on here. I think the desktop file you're looking at comes from the PolyMC source repo and isn't the one from this package. Admittedly, I don't use desktop launchers and didn't test this myself which I should do in the future.
I'll take a look at this and open a PR this week to fix it.

Thank you,
The desktop file in /nix/store/0xn1rrlz6fj2cwjzfb3pqh4s3mig4q50-polymc-1.0.4/share/applications/org.polymc.PolyMC.desktop
This is github:PolyMC/program_info/org.polymc.PolyMC.desktop.in file:
...
Exec=@Launcher_APP_BINARY_NAME@
...

Maybe can't read "Launcher_APP_BINARY_NAME" , But i don't konw how to fix.


SyntaxError: Unexpected token < in JSON at position 0

Please fix this issue. It is not allowing us to do anything except trivia help

Hello. This is a result of the database being offline. See the notice on https://opentdb.com/ for more info.
As it stands, the only thing that can be done about this is to wait for the database to return to full function, or look at self-hosting (https://lakeys.net/triviabot/install.html) and using custom questions. Sorry for the inconvenience.


Create new page (titled "Our team") to the right of the "Help" page

Who are the Weave team members?
Weave was co-founded by Jie Jane Chen and Jonathan Kusner in 2017 during their first year of medical school at Harvard Medical School and built by Razzi Abuissa. Jane, Jon, and Razzi are actively developing Weave to expand its offerings to students, residents, and faculty. Dr. Jennifer Potter (Advisory Dean for the William Bosworth Castle Society) and Dr. Fidencio Saldaña (Dean for Students) are advisors for this mentorship initiative.
How was Weave developed?
Weave was created with input from both students and faculty, all seeking more invested mentoring relationships at Harvard Medical School. In working towards this goal, Weave allows students to search for mentors in dimensions that are not strictly academic or scholastic. Does a faculty member like to cook, play guitar, were they also a first generation student, or do they share the same faith as you? In addition to allowing students and faculty to connect in this way, Weave allows faculty to clearly communicate their capacity for mentorship at a given point in time; faculty may indicate their preferred meeting cadence, method of contact, and any additional opportunities offered so that students can enter with realistic expectations for each relationship. To read more about the motivation and process for creating Weave, please visit: https://hwpi.harvard.edu/jhsma/learners-review, https://hms.harvard.edu/news/value-mentorship.
Does Weave stand for something?
We named our mentorship platform "Weave" to symbolize the woven tightness of a tapestry, where each thread is important to contributing to the beautiful image when they all come together. Similarly, community is enriched through the diversity of identities and experiences of individuals. We aspire to illuminate, celebrate, and nourish both personal and professional identities and interests through this platform.
How do I contact the Weave team?
Please contact us at<EMAIL_ADDRESS>

One edit to above: Change "Who are the Weave team members?" to "About our team"

Just a couple wording changes to the first section:
In 2017, Weave was co-founded by Jie Jane Chen (MD Class of 2020) and Jonathan Kusner (MD Class of 2021), Harvard Medical School students, and built by Razzi Abuissa. Jane, Jon, and Razzi are actively developing Weave to expand its offerings to students, residents, and faculty. Dr. Jennifer Potter (Advisory Dean for the William Bosworth Castle Society) and Dr. Fidencio Saldaña (Dean for Students) are advisors for this mentorship initiative, which is endorsed by the Office of Student Affairs at Harvard Medical School.


Can't test ReactFlow in vitest

What platform were you using when you found the bug?

React Flow / Svelte Flow version: "@xyflow/react": "^12.0.1", "react": "^18.3.1",
Browser and version: Safari/605.1.15
OS and version: MacOS 14.5 (23F79)

Live code example
No response
Describe the Bug
I couldn't get the TS codesandbox to work, so I'll just post the code here. I'm trying to run this simple test vith "vitest": "^2.0.4":
import "@testing-library/jest-dom";
import { render } from "@testing-library/react";
import { ReactFlow } from "@xyflow/react";
import "react";
import { describe, test } from "vitest";
import { mockReactFlow } from "../../designer/testing/mockReactFlow"; // this is the thing you recommended on your testing doc page

describe("mini test", () => {
    beforeEach(() => {
        mockReactFlow();
    });

    test("should work", () => {
        const {container} = render(<ReactFlow />); // This is the simplest example I could come up with
    });
});

When I run it, I get the following error (repeated a few times, I'm pasting only once):
Warning: Invalid hook call. Hooks can only be called inside of the body of a function component. This could happen for one of the following reasons:
1. You might have mismatching versions of React and the renderer (such as React DOM)
2. You might be breaking the Rules of Hooks
3. You might have more than one copy of React in the same app
See https://reactjs.org/link/invalid-hook-call for tips about how to debug and fix this problem.
Error: Uncaught [TypeError: Cannot read properties of null (reading 'useState')]
    at reportException (......../node_modules/jsdom/lib/jsdom/living/helpers/runtime-script-errors.js:66:24)
    at innerInvokeEventListeners (......../node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:353:9)
    at invokeEventListeners (......../node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:286:3)
    at HTMLUnknownElementImpl._dispatch (......../node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:233:9)
    at HTMLUnknownElementImpl.dispatchEvent (......../node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:104:17)
    at HTMLUnknownElement.dispatchEvent (......../node_modules/jsdom/lib/jsdom/living/generated/EventTarget.js:241:34)
    at Object.invokeGuardedCallbackDev (......../node_modules/react-dom/cjs/react-dom.development.js:4213:16)
    at invokeGuardedCallback (......../node_modules/react-dom/cjs/react-dom.development.js:4277:31)
    at beginWork$1 (......../node_modules/react-dom/cjs/react-dom.development.js:27451:7)
    at performUnitOfWork (......../node_modules/react-dom/cjs/react-dom.development.js:26560:12) TypeError: Cannot read properties of null (reading 'useState')
    at useState (......../node_modules/react/cjs/react.development.js:1622:21)
    at useColorModeClass (......../node_modules/@xyflow/react/dist/esm/index.mjs:279:49)
    at ReactFlow (......../node_modules/@xyflow/react/dist/esm/index.mjs:2935:32)
    at renderWithHooks (......../node_modules/react-dom/cjs/react-dom.development.js:16305:18)
    at updateForwardRef (......../node_modules/react-dom/cjs/react-dom.development.js:19226:20)
    at beginWork (......../node_modules/react-dom/cjs/react-dom.development.js:21636:16)
    at HTMLUnknownElement.callCallback (......../node_modules/react-dom/cjs/react-dom.development.js:4164:14)
    at HTMLUnknownElement.callTheUserObjectsOperation (......../node_modules/jsdom/lib/jsdom/living/generated/EventListener.js:26:30)
    at innerInvokeEventListeners (......../node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:350:25)
    at invokeEventListeners (......../node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:286:3)
The above error occurred in the <ForwardRef(ReactFlow)> component:

    at ReactFlow (......../node_modules/@xyflow/react/dist/esm/index.mjs:2933:22)

Consider adding an error boundary to your tree to customize error handling behavior.
Visit https://reactjs.org/link/error-boundaries to learn more about error boundaries.

Steps to reproduce the bug or issue

create a new React project using vite
install @xyflow/react, vitest, possibly some other dependencies
create the test file
run vitest

Expected behavior
I expected this test to pass.
Screenshots or Videos
No response
Additional context
I'm a pro customer.

Thanks for the report. We will check it!

Hey, it took me some time but I eventually figured out that's a problem with my project configuration: I suspect I had 2 versions of React included at some point. This was my smallest possible example of a failing test:
       const Z: React.FC<{}> = ({}) => {
           useEffect(() => { console.log("Test"); }, []); // it worked without this line
           return
Test
;
       };
       render(<Z />);

In any case, I could get RF working in vitest, so this can probably be closed.


Local and available local

Hi,
I don't know if MissionControl is the guilty, but since I have installed, I get random error on my main app, and crash with :
:fr is not a valid locale
On my app I've this :
   I18n.load_path += Dir[Rails.root.join("config", "locale", "*.{rb,yml}")]     config.i18n.default_locale = :fr
Can be related to MissionControl code in https://github.com/rails/mission_control-jobs/blob/66f829f795c1f96a33484fbe410a2bd4e9f69640/lib/mission_control/jobs/i18n_config.rb.

Hey @kaluznyo, what version of MIssion Control are you using?

Sorry ! 1.0.1, and rails 8.0.1

Huh, 1.0.1 should have a fix for that problem... could you copy the full trace from the error?

The error and the trace is on my 'main' app, when it crash, mission control is still alive
https://gist.github.com/kaluznyo/e99d274c971c4a37e534e43aa2573535

Oh, interesting... it's different from the error I fixed in version 1.0.1. 🤔 The code here is only used in Mission Control's controllers, so it shouldn't affect the code path in your error trace 🤔
Could you check what locale is being passed to the translate call in /PROJECT_ROOT/app/views/tasks/index.html.erb:17?

I just use the translate (t()) rails method on line 17. I suppose it's related to this, because when I put in production with Mission Control tab open, my main app crash. If I put in production without Mission Control open, it's work. My intuition, says the available locals is redefined globally by Mission Control. But if you say it's not possible, I'm probably wrong !

I put in production with Mission Control tab open, my main app crash. If I put in production without Mission Control open, it's work.

Huh, so do you mean that in both cases, Mission Control is installed in your app, just that in the second case you're not using it? So strange!

@kaluznyo, could I get more details on how you have configured Mission Control? The routes where it's mounted and all that?

Only
mount MissionControl::Jobs::Engine, at: "/jobs"

This is so strange. The custom I18n configuration is only used here, to set I18n.config, which is thread-safe, so it shouldn't affect a subsequent request at all.
Could you debug your app so that you check the value of I18n.available_locales first on the beginning of a request to / after restarting your server, then on a request to /jobs and then on another request to /?

I try to make a new rails 8 project on https://github.com/kaluznyo/test_mc_i18n, the only modification is :

Add a dummy controller with a show method. It use <%= t('.test') %>
Add a fr.yml and config.i18n.default_locale = :fr on application.rb
Add Mission Control

I reproduce with the same way. Launche server, open directly /jobs, then /dummy -> Crash. Inverse work well.

Yes I agree with you... I don't know where is wrong, the I18n.available_locales seems good on console on crashed page...

Oh, thank you so much for that reproduction app! I can reproduce with that, so I should be able to debug it. Looking into it...

To add more incomprehension, if I add p I18n.locale_available?(:fr) in the gem on app/controllers/mission_control/application_controller.rb, line 24 at the beginning on set_current_locale(&block)... I think it's work .....

@kaluznyo I think I found the problem thanks to your app and reproduction steps. The key is that the request to /jobs needs to happen before any other request 😅 The problem is that available_locales defined in Mission Control's I18n configuration ends up polluting the non-thread safe class variable @@available_locales_set 😅  To make things a bit more confusing, there's yet a 3rd config class being used here, Action View's own I18n config proxy class, that I didn't know.
I have a possible fix ready, will push tomorrow!

@kaluznyo, I've prepared a fix in https://github.com/rails/mission_control-jobs/pull/231, could you confirm whether it fixes the bug for you?

Works fine ! Thanks !


Cannot build large project under Windows

Due to limitations of command length in Windows I getting next error:
org.gradle.api.internal.project.ant.AntLoggingAdapter] [ant:java] : java.io.IOException: Cannot run program "C:\Program Files\Java\jdk1.8.0_05\jre\bin\java.exe" (in directory "C:\Users\Xelat.gradle\gradleFx\sdks\c1cf6cd41649e6a629464da8132b10622a37e3f2\frameworks"): CreateProcess error=206
If I understand situation correctly - it could be fixed with replacement of passing sources names via command line args to passing via xml file?

I have the same problem. Do you have any fix or workaround for this?


Feature/soil absorption

This PR introduces the soil energy balance to the new abiotic model. It's basically a series of functions that divide incoming shortwave radiation into longwave radiation, sensible heat flux, latent heat flux, and ground heat flux. Some variables are shared with the hydrology model, so I had a few adjustments to the selected heights and calculations there. I also finally added wind as an input to the example data and updated all tests accordingly.
Type of change

[x] New feature (non-breaking change which adds functionality)
[x] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (0cfab0a) 93.90% compared to head (1960c31) 93.98%.

Files
Patch %
Lines

...le_data/generation_scripts/climate_example_data.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@                      Coverage Diff                      @@
##           feature/new_abiotic_model     #377      +/-   ##
=============================================================
+ Coverage                      93.90%   93.98%   +0.07%
=============================================================
  Files                             65       65
  Lines                           3151     3207      +56
=============================================================
+ Hits                            2959     3014      +55
- Misses                           192      193       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


add Plusx.toml for v0.20.1

If this is an UPDATE for  https://github.com/anoma/namada-testnets/pull/962

If this is an UPDATE for https://github.com/anoma/namada-testnets/pull/962


add a data download timeout option for NWIS data

Added a timeout parameter for NWIS daily and peak flow downloads. You can test that these will time out by changing the timeout parameter to 1 second.
Also added function descriptions and parameter descriptions for download functions.
Closes #88

It was stuck at starting site 03145000. Didn't receive any other messages, warnings or anything.

Wait, 03145000 is not in the list of gages...why did it try to download that site?

Well, I deleted all the .csv files and re-ran. This time... it did not try to download 03145000. So idk what the heck happened.

OR I cloned your repo, ran the _targets.R script from the main branch, then realized i wasn't on your trycatch branch. So I wiped and re-cloned, made sure i was on the correct branch the entire time and now I'm re-running.

Okay, thanks for the update. There is a site with a similar number: 02314500. Let me know if there is any trouble this time

I've run it twice with the additional messages. The second time I cloned from scratch and re-ran the whole pipeline (with the new messages). Both times it went through without a snag. So I'm not sure how to proceed.


Update phone number metadata MX 56

Country: Mexico
Example number(s) and/or range(s): 56*,<PHONE_NUMBER>
Number type ("fixed-line", "mobile", "short code", etc.): Mobile
For short codes, cost and dialing restrictions:
Where or whom did you get the number(s) from: Production environment validation of phone numbers failing. AT&T latest mobile lines on Mexico City are using this prefix. Our merchants are failing to receive validation SMS as we first use this lib to check numbers.
Authoritative evidence (e.g. national numbering plan, operator announcement):
Link from demo (http://libphonenumber.appspot.com) showing error: http://libphonenumber.appspot.com/phonenumberparser?number=5610132441&country=MX

New value used on Mexico.
Saw this : https://github.com/googlei18n/libphonenumber/pull/2285 but i'm not sure if this particular file is generated from other sources.
Nevertheless we have found out multiple occurrences of this new prefix on past weeks.

+669525211115

+66952511115

Hi,
Thank you for taking the time to improve libphonenumber!
As discussed in https://issuetracker.google.com/issues/120241612, this issue is a duplicate of https://issuetracker.google.com/issues/119874122.
Closing this issue.


Can't get dask dashboard nbserverproxy working on Pleiades

Viewing dashboard over ssh tunnel on Pleiades works fine.
Using nbserverproxy hits this issue: https://github.com/dask/dask-labextension/issues/51

Fixed by switching to jupyter-server-proxy.


How would write validation for the following?

{
    "p_code":"ABA", // ABA or ACD
    "p_options":{
        "a": "5",
        "b": "5"
    }
}

How can I validate that when p_code is ABA, p_options.a is required? and when p_code is ACD, p_options.b is required?

There is no way to do this right now.

Are you not validating the wrapping object? If the outer brackets were of type 'p_container' then whatever wanted to use that type could use a custom validator for that type? Could be wrong here, I'm new to this lib.


Use aliases instead of common table expressions in the SQL backend

Fixes #1278

@llllllllll do you depend on CTEs anywhere?

Nope, we are not using the intermediate results of the sql backend currently, just pure blaze.

sweet merging


Fix date-only properties

Pretty simple fix really. Dates in Notion that didn't have a time set failed, since the datetime.date has no strptime method.

Thanks so much!

Released as part of 0.0.19!


Redis Adapter Integration With Laravel Echo

To make Laravel Echo Server Horizontally Scalable Redis Adapter is integrated with Echo Server.
The following changes are made:

Npm Version Update for Socket and Ioredis.
Added two new modules Redis and Redis Adapter
Socket Server Creation updated as per Socket 4.x module documentation.
Attaching Redis Adapter in init method

It will not effect the existing behaviour of Laravel Echo Npm Module, It will make sure that if multiple echo servers are running with same Redis Connection then the Event is emitted to other server as well if they are subscribed to that event.


Fixing Microsoft Learn link in CDN related commands documentation

update spo cdn commands documentation pages to fix link mentioned in More information section

Closes #4352

Nicely done @michaelmaillot! We'll try to review it ASAP!


Split HindleyMilner into library and test projects

Codecov Report

Merging #14 (dd835ca) into main (03a930d) will decrease coverage by 1.13%.
The diff coverage is 89.10%.

@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
- Coverage   73.57%   72.44%   -1.13%
==========================================
  Files          16       17       +1
  Lines        1525     1626     +101
  Branches      237      253      +16
==========================================
+ Hits         1122     1178      +56
- Misses        403      448      +45

Files
Coverage Δ

src/Escalier.HindleyMilner/Library.fs
89.10% <89.10%> (ø)

... and 2 files with indirect coverage changes


sst cli not working on Windows

Hi,
I am getting this when cloned the repo, run npm install and try to dev:
>npx sst dev
It seems that your package manager failed to install the right version of the SST CLI for your platform. You can try manually installing the "sst-win32-x64" package.

I am new to SST, sorry if its something obvious. I am trying to follow the sst guide. I am on Win 10 pc. Is the ion version of sst not supposed to work on windows?
Thanks

Ah yeah we don't have Windows support yet. It's WSL only at the moment.
You can track this here https://github.com/sst/ion/issues/347


/usr/lib/systemd error on OS X.

$ docker run -it --net host --pid host --cap-add audit_control \
    -v /var/lib:/var/lib \
    -v /var/run/docker.sock:/var/run/docker.sock \
    -v /usr/lib/systemd:/usr/lib/systemd \
    -v /etc:/etc --label docker_bench_security \
    germanramos/docker-bench-security
docker: Error response from daemon: Mounts denied:
The path /usr/lib/systemd
is not shared from OS X and does not belong to the system.
You can configure shared paths from Docker -> Preferences...

There is no /usr/lib/systemd on Mac. Is this volume mapping required? The test seem to run fine without it? Maybe the docs need to be updated to reflect this?

Hi @willfarrell, how does PR #160 look?

That should cover it. If not, there is this issue to catch any that didn't read the docs.

Would there be scope for instructing macOS users better on what to do to solve this?
I'm aware that the readme states:

Don't forget to adjust the shared volumes according to your operating system, it may not for example use systemd.

Nevertheless, it would be helpful - would those directories actually exist on macOS - if it was just a matter of adding those paths to the File Sharing Preferences in Docker, as is suggested in the hint that Docker mentions in the error thrown:

docker: Error response from daemon: Mounts denied:
The paths /usr/bin/docker-runc and /usr/bin/docker-containerd
are not shared from OS X and are not known to Docker.
You can configure shared paths from Docker -> Preferences... -> File Sharing.
See https://docs.docker.com/docker-for-mac/osxfs/#namespaces for more info.

Removing them from the docker run command indeed works (or seems to), but adding that fact to the readme.md in a section "Note for macOS users" might be helpful for some.
You can configure shared paths from Docker -> Preferences... -> File Sharing.
See https://docs.docker.com/docker-for-mac/osxfs/#namespaces for more info.

@mrdulin, I've updated the instructions.


I am interested in contributing in the Spanish language

Hello, I am interested in contributing in the Spanish language of ppl2. In case you have any doubts, I am a native speaker and I would like to contribute, thank you.

Hello!
So far, the Spanish translation is missing some strings, it would be greatly appreciated if you translated those.
If you find other translations that can be improved, feel free to do so!
If you have questions, please join the Discord server, where I'll answer them.
Thank you!
Go ahead and get started by forking and editing this file: https://github.com/pewpewlive/ppl-i18n/blob/master/translations/spa.po


Allow App names with length > 39

Hi,
I noted that there's an arbitrary length restriction on the length of the application name to 39 characters.
In our application we have microservices which have a name with length of 40-45 characters.
And who knows whether they get longer in the future.
I don't think it makes sense to trim a character of every service for just this application, as this could also impose uniqueness issues.
The change was introduced with #814.
From Argo-CD side it looks like this is possible to have quite long application names, so this should not conflict:
https://github.com/argoproj/argo-cd/pull/7468
Proposal:
So maybe just add 31 more characters as a buffer.

Thanks for the fast feedback.
I'll have a deeper look next week!

Added a comment here
https://github.com/freiheit-com/kuberpult/pull/1183#issuecomment-1840790228
Should be possible with configuring ArgoCD correctly.

Was this closed with https://github.com/freiheit-com/kuberpult/pull/1183 ?

@jdvgh no worries. Thanks for the feedback! I included it in my PR - feel free to check it out.


Generalize BasePathFs usage

Related to spf13/afero#76

See my comments on the other thread.

I'm not sure what the specific security concern is here.  How is hugofs.Fs.WorkingDir vulnerable to abuse?  If it's open to abuse/misuse somehow, we should change the hugofs.Fs data structure to protect the underlying afero.Fs implementations with Setters and type assertions.

'm not sure what the specific security concern is here. How is hugofs.Fs.WorkingDir vulnerable to abuse?

I want to know that the file access is confined to a restricted directory (i.e. WorkingDir) to prevent obvious stuff like readFile "/etc/passwd".
But I'm now sure why I should make any arguments here, you are the one who proposes a change.


Add ci_script action plugin

As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date

/lgtm


Altered Welcome page

•This All Are Changes On Main Pages

• These All Are codes that have been changed .

Please note that this is not in line with our code of conduct. To ensure clarity and proper assignment tracking, please attach the issue number to your PR description. This helps us confirm that the issue is assigned to you. Kindly update your PR with the issue number and resubmit.
Example:
Fixes #123 - Brief description of the fix.


Opset 18 Reduce ops

All reduce ops in opset 18 have been modified to match the specification of ONNXReduceSum. Namely they add a noop attribute, and axes is changed from an attribute to an input.
The simplest approach is to simply rewrite all older versioned ReduceOps into their opset 18 counterparts. I have elected to only take this approach with

ReduceL1
ReduceL2
ReduceLog
ReduceLogSum
ReduceLogSumSquare

However I've elected to keep the implementations of the following ops. And rename them to V13.

ReduceMax
ReduceMin
ReduceMean
ReduceProd

Namely they add a noop attribute, and axes is changed from an attribute to an input.

It looks to me V13 can be totally converted to V18, e.g. by creating a value from the axes attribute.

However I've elected to keep the implementations of the following ops. And rename them to V13.

Could you explain more about the reason to keep the implementation? I don't see any good reason to keep the shape inference and lowering of V13. Thanks!

I concur, the preferred option is to migrate the old ops to the new one and ensuring that the same quality of code is generated for the old ops (as typically older version means more things known at compile time). This has the added benefit that the newer op that still use constant (which is just the new way to express the semantic of the older op) will also be optimized to be as good as the older versions of the op...

@AlexandreEichenberger, @tungld, @chentong319
Thanks to all three of you for the feedback.
One thing I would like to mention when considering whether to remove the old lowering for (ReduceMax, ReduceMin, ReduceProd, ReduceMean) is the testing situation. For end to end testing we rely on the ONNX backend tests. Unfortunately these are only for opset 18. It therefor becomes quite difficult to validate that opsets before 18 work correctly (which are the implementation/opsets used in production). There are tests for full models, but its not obvious which ops are called on which models.
I would like to point out I didn't really write any code. Unfortunately the git diff doesn't really reflect whats happening. ReduceSum already existed with two lowering implementations, and two shape inference implementations. One for ReduceSum11 and another for ReduceSum (introduced in opset 13 with axes inputs). As a result, My code simply templated this implementation to work with all the modified reduction ops, which have lowering implementations.

Could you explain more about the reason to keep the implementation? I don't see any good reason to keep the shape inference and lowering of V13. Thanks!

The reason for keeping the implementation is that it relax's the pass ordering dependency. Without shape inference implementations for the older versions of the ops, decomposition must be called before shape inference. In general the shape inference already exists for the older version of the op.
I do agree that Rewriting to the latest op is always the cleanest solution. However with the current testing situation, I feel that adding the code as is, reduces the chance of introducing bugs to people who currently use onnx-mlir in production.

@philass thanks for your careful explanation, it makes sense in the light of our discussion at last meeting and your text above.

Unfortunately these are only for opset 18. It therefor becomes quite difficult to validate that opsets before 18 work correctly (which are the implementation/opsets used in production). There are tests for full models, but it's not obvious which ops are called on which models.

Is it true that we can preserve some testing if we migrate some of the lit tests before the upgrade to the vXXX op names, and then add the same tests for the new opset? That way we can keep some testing.
We can ask @gongsu832 if we should have tests (all the times or sanity from time to time) of backend tests for older versions (not sure how easy/hard it would be from a logistics perspective).

Any blocker on this?

@AlexandreEichenberger no blockers,
I need to chug a coffee and resolve the merge conflicts. Thanks for the reminder. I'll try and have this done by the meeting tomorrow

Hey @AlexandreEichenberger,
Thanks for the review! I think the  feedback you gave makes sense. I'll make those changes in a following PR, where I handle the noop case.
I want to make sure I land this tonight before it gets to out of sync with head

Jenkins Linux amd64 Build #9907 [push] Opset 18 Reduce ops (#19... started at 02:09

Jenkins Linux ppc64le Build #8972 [push] Opset 18 Reduce ops (#19... started at 03:14

Jenkins Linux s390x Build #9922 [push] Opset 18 Reduce ops (#19... started at 03:09

Jenkins Linux amd64 Build #9907 [push] Opset 18 Reduce ops (#19... passed after 1 hr 47 min

Jenkins Linux ppc64le Build #8972 [push] Opset 18 Reduce ops (#19... passed after 3 hr 6 min

@philass thanks for the PR. Your help is much appreciated. Big thanks


stBTC liquid staking token and Babylon integration

This PR adds a staking pool which locks nBTC and moves the underlying BTC into staking UTXOs via the Babylon BTC staking protocol.

Codecov Report
Attention: Patch coverage is 25.73029% with 1856 lines in your changes are missing coverage. Please review.

Project coverage is 34.57%. Comparing base (2d44997) to head (4a2a56a).

Files
Patch %
Lines

src/babylon/relayer.rs
0.00%
333 Missing :warning:

src/frost/signer.rs
0.00%
320 Missing :warning:

src/frost/mod.rs
0.00%
268 Missing :warning:

src/babylon/mod.rs
76.98%
183 Missing :warning:

src/frost/signing.rs
0.00%
153 Missing :warning:

src/app.rs
0.00%
145 Missing :warning:

src/frost/dkg.rs
0.00%
115 Missing :warning:

src/babylon/proto/mod.rs
0.00%
95 Missing :warning:

src/bin/nomic.rs
0.00%
65 Missing :warning:

src/babylon/proto/btccheckpoint.rs
0.00%
40 Missing :warning:

... and 5 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #295      +/-   ##
===========================================
- Coverage    36.84%   34.57%   -2.27%
===========================================
  Files           25       35      +10
  Lines         9459    11933    +2474
===========================================
+ Hits          3485     4126     +641
- Misses        5974     7807    +1833

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Converted to work as a .Net 4 class library. Also made to work best w…

…ith Frazer software.
What changed?
Gutted this repo and made it as minimal as possible and to work with .net framework 4.0
Describe the changes you have made to improve this project.
Not sure if it is an improvement. Just made to work with .net framework 4.0
Closed issues.

Woops...wrong place for this. Closing.


notebook and data from July 2024 webinar

There are 2 data files, and the notebook prompts you to pick one of the files.
For @simonbowly

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@Dr-Irv I've moved these as-is into the existing webinars directory so we can point people to it: https://github.com/Gurobi/gurobipy-pandas/tree/main/webinar/2407-data-first
Opened #86 separately for integrating this into the docs (that might not happen until after the webinar).


wait* steps not works with arguments {name | id }

I try to call:
I.waitForVisible({ name: 'field_name' })

but locator contains:
Locator {
  type: 'name',
  output: null,
  strict: true,
  locator: { name: 'field_name' },
  value: 'field_name'
}

And the waitForVisible step checks if this locator isCss will make locator.simplify, else waitByXpath, but name is not css, and CodeceptJS tries to search element by following xpath:  'field_name', but it is invalid xpath

A workaround is explicitly declaring your locatorObj type to 'css'. Working example:
module.exports = () => {
    codeceptjs.locator.addFilter(( providedLocator, locatorObj) => {
    locatorObj.type = 'css';
    ...


Serverless setup

Hi,
I like the project ideas and goal but I'm wondering how things could be refactored to allow for a serverless setup.
My understanding is that the API/server logic is being wireguard when it could be hosted on a serverless platform (I'm thinking of Cloudflare Workers).
The initial invite logic could be substituted by oneshot API tokens and it would be relatively easy to leverage SSO/IdP and other features without having to do a lot of work.
Is this something that was explored during the design of innernet?
Cheers

@calderonth we didn't explore any of these lambda function options too deeply because it typically requires a lot more vendor-specific code compared to just expecting a plain old server. That said, I'm not aware of any "serverless" platform providers that can run WireGuard interfaces...
The long story short is that, at a minimum, innernet needs a centralized database to manage IP address allocation, since it doesn't do distributed negotiation of addresses.

Thanks for your reply.
This is where I was missing some context even with the blog post details.
I understand the innernet server is used as the endpoint for the API frontend/DB backend and is sitting behind Wireguard.
Innernet clients then connect via Wireguard and can fetch their respective configuration from the server.
However, what is the underlying wireguard topology? Does the innernet server behaves as a VPN concentrator or is the network more of a mesh?
If it's a mesh, then it would be feasable to split the architecture:

control plane: expose (securely) the API/DB backend without Wireguard hosted on a serverless platform, have clients authenticate/pull their config
data plane: the config is used by client to setup tunnels and access the various networks

If I go back to my serverless example, Cloudflare Workers supports a K/V storage backend (not SQL-type) and that would be enough to store innernet configuration.

Innernet is mesh-like - the server doesn't do any proxying of actual traffic, only coordinating connections between peers.
You could certainly make a control server that didn't use WireGuard at all, but I find it very convenient personally to do everything within a WireGuard network to limit the externally exposed surface.
I don't think Cloudflare Workers over TLS instead of WireGuard is something I'm particularly interested in supporting in the innernet project, but you are of course welcome to fork it and do it yourself!

Alright thanks for the clarification :-).
I might investigate a fork but already have loads of side projects ;-)

No problem! Best of luck on your projects, whichever you choose :P.
I'm gonna go ahead and close this, but definitely let me know if you decide to take on that fork project.


Allow standard clipboard format in Windows

This PR try to allow druid_shell::Clipboard to use Standard Clipboard Formats in Clipboard::preferred_format and Clipboard::get_format, such that the following code works in Windows:
  let format_id = match clipboard.preferred_format(&["CF_DIBV5"])?;
  let data = clipboard.get_format(format_id);

Fixed :)