Deno support

Is your feature request related to a problem? Please describe.
Hi.
First of all, thank for this great library. I have been maintaining a version of this code to work on Deno. You can see the repo here: https://github.com/lumeland/imagemagick-deno
Basically, it's a script to convert the CJS code to ESM and fix some minor issues.
In the latest two versions, the wasm file was moved to a different file. This is a good idea but makes it more difficult to maintain the Deno compatible version. As I can see, the code detects Deno as a Browser environment, so try to load the wasm file using XMLHttpRequest, which is an old API that Deno doesn't support.
error: Uncaught (in promise) RuntimeError: Aborted(ReferenceError: XMLHttpRequest is not defined). Build with -sASSERTIONS for more info.

Describe the solution you'd like
If the file was loaded using fetch() probably it would work on Deno.
Describe alternatives you've considered
As an alternative, maybe providing a way to pass directly the wasm content (instead of the file path) in Uint8Array (or any other format you consider) would be great. For example:
const response = await fetch(wasmFile);
const content = await response.blob();
await initializeImageMagick(content);

In this way the code would be more portable to other environments (like Deno).
Additional context
No response

It looks like the generated emscripten code contains an option to specify the content of the wasm binary and then that will be used instead of the location of the file. And I think the signature of the initializeImageMagick method can be changed to this:
initializeImageMagick(wasmLocationOrData?: string | Uint8Array | Buffer):

But I will need to experiment with that and see if that works.
And I am wondering if we could combine our efforts and automatically generate the deno library in this project? Not yet sure how we can do then you would not longer need to maintain that library and we can hopefully detect issues earlier. Maybe we could even run the unit tests in deno?

And I am wondering if we could combine our efforts and automatically generate the deno library in this project?

That would be great and I can help you, if you want. Due the code is written in TypeScript, I propose to make some tweaks in the code so it would work primary on Deno (and browser) and use https://github.com/denoland/dnt to build the Node version. I'm using this strategy in one of my projects and works great (https://github.com/oscarotero/keep-a-changelog/blob/master/build_npm.ts).

I could use some help with this but I would prefer to go from node to deno instead of the other way around. This project use vite and a lot of its tools for the build process so I would like to create the deno package from the existing code in this project. And I also wonder if the the dist/index.mjs file is not already compatible with deno.

Okay, understood.
Deno works very similar to browsers (it uses Web standards as much as possible) but it also has native support for TypeScript. In Node the imported files are resolved magically. For example:
import magick from './magick'

This can import the file ./magick.js, ./magick.mjs, ./magick/index.js, ./magick.ts, ./magick/index.ts ...etc. It depends on the real filename so it has to check different combinations before load the file. Deno works like a browser so you have to pass the real filename including the extension: import magick from './magick.ts'. My script to convert the code from ./src folder to Deno only adds these extensions to the imported files.
The other thing was to copy the module that had the wasm code, including a ESM export to be imported correctly in Deno (because it was only UMD). In the new versions, this step is broken, because the wasm file is loaded dynamically in the browser environment, which uses XMLHttpRequest (not implemented by Deno because they recommends to use Fetch).
If want to know which web standards are supported by Deno, take a look to this cheatsheet (under the Web APIs section). As you can see, only Fetch is supported but there are different ways to instantiate WebAssembly modules.

Are you aware that the content of the dist folder has changed since version 0.0.16 and that there now is a single file with all the exports and both an index.mjs and index.umd.js file. Would it be possible to use these files in a deno package?

I just tried index.mjs but got this error:
error: Uncaught ReferenceError: __DENO_NODE_GLOBAL_THIS_0_38_0__ is not defined
    at file:///Users/oscarotero/imagemagick-deno/node_modules/@imagemagick/magick-wasm/dist/index.mjs:1:18

I would also prefer to use the TypeScript code in Deno, in order to have proper types.
I also tried your latest commit on Deno (after transforming the code using my script):
console.log("Initialize");

const wasmFile = import.meta.resolve("../deno/src/wasm/magick_native.wasm");
const file = await fetch(wasmFile);
const wasm = await file.arrayBuffer();

await initializeImageMagick(wasm);

console.log("Read");
const data: Uint8Array = await Deno.readFile("test/unsplash.jpg");

await ImageMagick.read(data, async (img: IMagickImage) => {
  console.log("crop");
  img.crop(1000, 1000);

  console.log("resize");
  img.resize(200, 100);

  console.log("write");
  await img.write(
    (data: Uint8Array) => Deno.writeFile("test/unsplash-blur.png", data),
    MagickFormat.Png,
  );
});

And the code seems to work (it read, crop and resize the image) but fails in the img.write:
Initialize
Read
crop
resize
write
error: Uncaught (in promise) TypeError: str.charCodeAt is not a function
    at WasmLocator.lengthBytesUTF8 (file:///Users/oscarotero/imagemagick-deno/deno/src/wasm/magick_native.js:8:15138)
    at _withNativeString (file:///Users/oscarotero/imagemagick-deno/deno/src/internal/native/string.ts:23:22)
    at _withString (file:///Users/oscarotero/imagemagick-deno/deno/src/internal/native/string.ts:38:10)
    at new NativeMagickSettings (file:///Users/oscarotero/imagemagick-deno/deno/src/settings/native-magick-settings.ts:69:7)
    at MagickSettings._use (file:///Users/oscarotero/imagemagick-deno/deno/src/settings/magick-settings.ts:95:22)
    at file:///Users/oscarotero/imagemagick-deno/deno/src/magick-image.ts:2356:24
    at Function.use (file:///Users/oscarotero/imagemagick-deno/deno/src/internal/pointer/pointer.ts:27:14)
    at file:///Users/oscarotero/imagemagick-deno/deno/src/magick-image.ts:2355:15
    at file:///Users/oscarotero/imagemagick-deno/deno/src/internal/exception/exception.ts:46:22
    at Function.use (file:///Users/oscarotero/imagemagick-deno/deno/src/internal/pointer/pointer.ts:27:14)

I was able to run this project with deno. I have added an example to this project to show how I did that. And I have also added a build step that will run a simple example that reads and writes an image using deno.
Your code is failing because I made another breaking change in the main branch that swaps the arguments of the img.write method.

Okay. I'd rather to use directly the TypeScript code in Deno (for better debugging) but if the mjs file works, that's fine.
My version is published on lume.land/x<EMAIL_ADDRESS>If you are going to maintain the Deno version, probably I should deprecate this package and create a new one pointing to this repo?
Or perhaps it's possible to edit the current package to use this repo.

Not related with this, but when I try to convert images to AVIF format, the browser cannot display them (tested on Firefox and Chrome). It says the image cannot be shown because it contains errors.

Not sure how I want to move forward with Deno support. At least I have a proof of concept now on how someone could get it working but a proper module would probably be the right idea. Maybe a separate build step that makes this project available as a module?
I cannot reproduce your avif issue. Maybe create a separate discussion for that? And make sure that you are using the latest code on main. I recently fixed a bug in the aom encoder build so that might be causing what you see.

In deno.land you can register a new module choosing a directory inside a GitHub repo. For example, my version (https://deno.land/x/imagemagick_deno) contains the code inside the /demo/ folder of my repo (https://github.com/lumeland/imagemagick-deno/tree/main/deno). Your build code could export a Deno version in a subdirectory and register the module from that. Then, it uses git tags for new versions. In Deno is common to use the mod.ts instead of indext.ts as the filename for the main module, so you can create a different entry points for Deno.
Another think is about cache. Deno doesn't have a node_modules folder or a npm install. It download and cache the modules the first time they are used. If the wasm code is loaded from fetch, it won't be cached so it must be downloaded everytime the file is runned. Related: https://github.com/denoland/deno/issues/5987
And avout avif, I'll do more tests and wil open a new issue if I cannot fix it. thanks!

Can I just say thank you to you two for working on this? I'm dying to using imagemagick via wasm + Deno. There is no way I want to call out to a local bin when accepting user file input.


missing code to generate afd/flow

In reference to create-it-yourself
it seems that generate_afd.py and generate_flow.py codes are missing...
Could you please check?
thanks a lot.

Hi,
At the top of your linked paragraph it says:
Note: To run the following scripts cd into afd/
Since the afd/ directory is a git submodule be sure to clone the main repo properly with the --recurse-submodules flag, i.e.:
git clone --recurse-submodules git@github.com:f-ilic/AppearanceFreeActionRecognition.git
The files you mentioned are present here:

afd/generate_afd.py
afd/generate_flow.py

Does that help?

this is super helpful. thank you so much for such prompt response and wonderful work!


the path for image generation is not correct

In the definition of the sdk you have
@Post("/images/generations:submit")
@ExpectedResponses({202})
@UnexpectedResponseExceptionTypes({@UnexpectedResponseExceptionType(
value = ClientAuthenticationException.class,
code = {401}
), @UnexpectedResponseExceptionType(
value = ResourceNotFoundException.class,
code = {404}
), @UnexpectedResponseExceptionType(
value = ResourceModifiedException.class,
code = {409}
), @UnexpectedResponseExceptionType(HttpResponseException.class)})
Response beginAzureBatchImageGenerationSync(@HostParam("endpoint") String var1, @QueryParam("api-version") String var2, @HeaderParam("accept") String var3, @BodyParam("application/json") BinaryData var4, RequestOptions var5, Context var6);
there are 2 error
first is, when you call image/generation api, there is no submit suffix in the request path
second is, you need deployment prefix to correctly route to the endpoint, like deployments/{deploymentId}/xxx

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

ID: a47134e6-8e58-e26f-a452-b08e57c5cdf0
Version Independent ID: 2fe238ea-1d13-f8d9-c549-09bdb6858c03
Content: Quickstart: Generate images with Azure OpenAI Service - Azure OpenAI
Content Source: articles/ai-services/openai/dall-e-quickstart.md
Service: azure-ai-openai
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@wangjl1110
Thanks for your feedback! We will investigate and update as appropriate.

@wangjl1110
I've delegated this to @PatrickFarley, a content author, to review and share their valuable insights.

@wangjl1110 Are you calling the REST API directly or using the Java SDK, or something else?

@wangjl1110 Are you calling the REST API directly or using the Java SDK, or something else?

Feel free to reopen this issue to continue debugging
#please-close


Basic UGC Limits + Helpers

Fixes #71

Discussed offline - OpenAPI spec does support a limitation on string length, but the codegen tools we use do not support it :(
https://swagger.io/docs/specification/data-models/data-types/#:~:text=multipleOf%3A -5.-,Strings,-A string of


[Bug] User site: the second date in a project overlaps the 'delete' button

http://screencast.com/t/B9BA9FzWWaF

Fixed already.


Remove version: Jackett.Jackett version 0.21.2536

All InstallerUrls returned 404 Not Found

https://github.com/Jackett/Jackett/releases/download/v0.21.2536/Jackett.Installer.Windows.exe

Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.21.2536/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


Refactor sql db delete all

Added the delete option for internal tests. Everything works fine.

Yea, looks mostly good, but I'm going to move this cleanup code to the Arquillian extension in the common module, so that it applies to all test suite modules.

I did this here: https://github.com/rhoar-qe/thorntail-openshift-test-suite/pull/12/commits/1b1e62695d4bf7b6f043bf5694096d78946543ed


Documentation improvements

Meta: would this be better as one amorphous commit "Documentation improvements", or three separate PRs (*shudder*)?

:+1:


Add site for v1.0.0-alpha.2

Make the site for v1.0.0-alpha.2
Update version warning header to include logic for alphas

Signed-off-by: Nolan Brubaker<EMAIL_ADDRESS>

I've also generated updated master docs. Want them in this PR, or separated?

no preference


Port page_allocator API to OS/2

There is AllocPages & friends API (page_allocator.h) in base used for memory allocation (it's not the only one but anyway). Currently, we follow the POSIX code path that uses mmap. However, our mmap implementation is limited ATM (see https://github.com/bitwiseworks/libcx/issues/75, https://github.com/bitwiseworks/libcx/issues/76) and doesn't pass Chromium checks (mprotect expectedly fails, see #4).
In fact, this is a normal memory allocation which uses VirtualAlloc on Windows and mmap(MAP_ANONYMOUS) on Posix. It makes no sense to use mmap on OS/2 in this simple case due to inevitable overhead and current incompleteness of the implementation. We will use DosAllocMem directly as we do in several other places in Chromium already.

I've done this (not a big deal) and it runs much further now but then the Chromium code tries to free the block of memory not at its base address but with some offset. This expectedly fails as it's not possible on OS/2. The problem is that it's not possible on Windows either so it should fail there as well but apparently it doesn't... I need to dig deeper to see if the code calling base::FreePages contains some platform dependent code.

The last problem is fixed in the above commit.


new(pkg/driverbuilder): support photonOS 5.0.

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area pkg
What this PR does / why we need it:
Moreover, support aarch64 kernelurls for it.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

new(pkg/driverbuilder): support photonOS 5.0.

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

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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

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

Needs approval from an approver in each of these files:

OWNERS [FedeDP,dwindsor]

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

/hold

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

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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

I had to revert fb7fb6445afac8cb8e863d1783430ed1a02b8f01 since debian friends fixed it after all :D

/unhold

/cc @dwindsor another approve? :D

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

Needs approval from an approver in each of these files:

OWNERS [FedeDP,dwindsor]

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


CCUserDefault crashing on Windows Phone 8.1 (WP8.1)

See forum post
System.NotSupportedException: Specified method is not supported.
at Microsoft.Xna.Framework.Storage.ShowSelectorAsynchronousShowNoPlayer.BeginInvoke(Int32 sizeInBytes, Int32 directoryCount, AsyncCallback callback, Object object)
at Microsoft.Xna.Framework.Storage.StorageDevice.BeginShowSelector(Int32 sizeInBytes, Int32 directoryCount, AsyncCallback callback, Object state)
at CocosSharp.CCUserDefault.CheckStorageDevice()
at CocosSharp.CCUserDefault..ctor()
at CocosSharp.CCUserDefault.get_

MonoGame's StorageDevice implementation does not work on WindowsPhone 8.1.  BeginInvoke is not supported.

PR submitted to MonoGame

This has been merged in MonoGame code base.  We will get the change when we update our xamarin branch from develop branch.


Swapping steps for health probe creation

This was pointed out by Aidan Finn via email to a distribution list. The Health probe was being used before it was created.

Hi @mikewo, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, AZPRBOT;

@mikewo : Thanks for your contribution to the Azure documentation! The author, @sdwheeler, has been notified to review your proposed change.

@iainfoulds would you make sure to ingest this as well? :-)

@tysonn - approved

#sign-off

@sdwheeler - Hi Sean, please fix the merge conflict. If you need guidance, please see the wiki for instructions on fixing it.
#hold-off

@mikewo - There have been a few changes to this content since you submitted your PR. I cannot merge it as is. We can either close this PR and I make the changes you want or we can close this PR and then you can submit the change again from the current version. What would you like to do?

@sdwheeler I'll close this one and make the changes against the most current version. I'll then resubmit.

Created #7604


Add copyMessage and copyMessages methods

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (8f6ae76) to head (0c5483e).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##              master       #26   +/-   ##
===========================================
  Coverage     100.00%   100.00%
- Complexity       901       914   +13
===========================================
  Files            203       205    +2
  Lines           3222      3296   +74
===========================================
+ Hits            3222      3296   +74

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


[FC UI] Test plan for full flow

We learned from PFP M1 that some points should be set aside for testing the full flow (can be manual). This issue is specifically for writing the test plan
AC:
test plan for full flow is written (can be manual)

as part of this we should define any new capabilities that are needed for our mock provider and make sure those are captured in issues

https://docs.google.com/document/d/1tVSk6KfJu0_I8fTTFMn1WRb-PCuZr4bKNeIyKpa8DAo/edit?usp=sharing

I collected the requirements for mock-provider for the test plan, and compared them to what mock-provider currently does, and it looks like we're set!

(after my small tweak here https://github.com/fiatconnect/mock-provider/pull/12 )


New Contribution

Contributor: [contributor information - Optional]
SSH Public key: [contributor SSH public key (if you don't have one, submit the issue to receive instructions)"- Required]
Attestation: [any form of attestation for contribution (e.g., signature) - Optional]

AAAAC3NzaC1lZDI1NTE5AAAAICZ3JDBeDeBTS1wC9jlhlYDeJGDIpaY+0+tBntjG8DUK


Qualtrics Quota

Hey Chris,
I did what you suggested and checked the quota when actually activating the survey. Unfortunately, while the partial responses are recoded, the quota does not increase accordingly. I tried to find out how to fix it but was not able to so far. If you have time or a hunch, maybe you can take a look into it or I will try to fix it on Friday after exams and presentations.

I just tried it out after activating the survey for a short moment (now not active anymore) and it seems to work nonetheless?
(should not forget to clear the quotum of course)

I think I solved the problem.
I just made the page with the debriefing that was our last page before the end of survey message so far the end of survey message. So now, if you answer the scheduling question you automatically complete the survey. I don't thinkt this is a pretty solution but I think it should work


Multiple boradcasts to different clients

I'm trying to use SignalR. However, I am not sure if it can do what I need it to do.
When setting up SignalR in Startup we have to call the following:
app.UseSignalR(routes => {     routes.MapHub<Chat>("chat"); });
Now please think of this scenario, I tried to simplify the scenario as much as I could.
Let's say I go to my website and I open my profile page. That page lists all sort of chat messages that have my name in it. Then I open another tab and go to home page. There I type in the chat my name and send a message. This message will show on my profile page, but then I enter another user and send a message. This second message should not show up on my homepage, it should be on the homepage of that user.
To achieve this, from what I can tell, I have to map 2 hubs. However, I need this for every user and they will register to the site after startup runs. The other part to consider is if let us say this was Twitter. TONS of users. I thought about just filtering the message through javascript and only displaying if it has my name in it on the profile page, but if this is a very heavy hitting website, that is far too much data being sent to every client.
Is limiting what is sent to which client possible? The only way I thought of it was having multiple chat hubs and registering them on startup. More than likely the answer is yes and I did not fully understand the proper use of SignalR. If you could, please point me in the right direction.

Sounds like you want something like this.
https://github.com/https://github.com/PeteGoo/PushqaPeteGoohttps://github.com/PeteGoo/Pushqa/https://github.com/PeteGoo/PushqaPushqahttps://github.com/PeteGoo/Pushqa
Unfortunately this was not built for Signalr core.

From: bagzli<EMAIL_ADDRESS>Sent: Monday, October 30, 2017 9:15:08 PM
To: aspnet/SignalR
Cc: Subscribed
Subject: [aspnet/SignalR] Multiple boradcasts to different clients (#1067)
I'm trying to use SignalR. However, I am not sure if it can do what I need it to do.
When setting up SignalR in Startup we have to call the following:
app.UseSignalR(routes => { routes.MapHub("chat"); });
Now please think of this scenario, I tried to simplify the scenario as much as I could.
Let's say I go to my website and I open my profile page. That page lists all sort of chat messages that have my name in it. Then I open another tab and go to home page. There I type in the chat my name and send a message. This message will show on my profile page, but then I enter another user and send a message. This second message should not show up on my homepage, it should be on the homepage of that user.
To achieve this, from what I can tell, I have to map 2 hubs. However, I need this for every user and they will register to the site after startup runs. The other part to consider is if let us say this was Twitter. TONS of users. I thought about just filtering the message through javascript and only displaying if it has my name in it on the profile page, but if this is a very heavy hitting website, that is far too much data being sent to every client.
Is limiting what is sent to which client possible? The only way I thought of it was having multiple chat hubs and registering them on startup. More than likely the answer is yes and I did not fully understand the proper use of SignalR. If you could, please point me in the right direction.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/aspnet/SignalR/issues/1067, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADQA1y6B6WXorjO5-XkZDeFyDAviUwehks5sxjxcgaJpZM4QL3W_.

So you just want the ability to send messages to individual users?
In your hub you can do
// for connectionIds
Clients.Client(connectionId).InvokeAsync("clientmethod", args);
// or for user names
Clients.User("userName").InvokeAsync("clientMethod", args);

or use groups...

@beatsm your link doesn't work for me.
@BrennanConroy Where would I run that code, not on startup I'm sure?
@moozzyk could you please elaborate?

@BrennanConroy To answer your question. Not so much to an individual user, but more to an individual URL. The user example I came up with above is not exactly what I am trying to do, but it is very close to it in design.
So, for example, I will have 2 pages. Page one has controls. Page 2 has actions that happen when controls are sent. However, Every user has their own page one and page 2. Furthermore, Each user may have numerous sets of page 1 and page 2. So I'm looking for a way to target a specific URL. All clients that have a specific page opened.

https://github.com/PeteGoo/Pushqa
Sorry sent it via email and somehow got garbled.
The idea is that the client is able to filter the messages it receives on the server. I was thinking that each page would advise the server what messages it was interested in. But as I said this is quite old code now that does not work on Signalr core.

The simplest way to make it work is as @BrennanConroy said to use Client.User. Whenever you have an update for the user you invoke a client side method with the data. Now, you either pass the name of the page you want to show the notification on as a parameter or you use a different client side method for each page. In the first page you would ignore messages that are not targeting the page that is open. In the second case the client will receive a client side invocation message but since there will be no handler registered for the method the message will be ignored. The disadvantage of this approach is that you will be sending messages to a user even if the page the message is meant for is not open.
To fix this disadvantage you can use groups. Whenever a user opens a page they want to receive notifications for they add themselves to a group called for instance "UserName_PageName". Now, whenever you have a message for a given user for a given page you send this message to the "UserName_PageName" group. If the user has multiple instances of a page open they all should receive the message.


Get all country data

Hi, I want to get the graph for whole country to calculte the distance between some points, my code is like this:
ox.config(use_cache=True, log_console=True)
G = ox.graph_from_place('France', network_type='drive', which_rresult=2)

But I think that is too large, so I got python request's time out errors each time, I try to change these two parameters: max_query_area_size, timeout, but I seem also take so much time.
Is this the right way to get all the graph for the whole country ? or some other way?
Thx a lot!

Your which_result argument is misspelled. But the main issue here will be RAM on your computer. OSMnx automatically handles large queries, subdividing them so the API can handle it. But your computer needs enough memory to construct a massive country-wide graph. You might want to filter to only retain freeways or something like that if memory is an issue.
In general, the issue tracker is for documenting bugs or proposing new features. Best to ask how-to questions at the user community on StackOverflow, per the contributing guidelines.


Add maintenance page for Shelly deployments

fixes #483

@moonglum We could still reuse this for Uberspace. Merge pls? :smiley_cat:

Beautiful! :smile:


How do you get these?

In the creative inventory, you can only get 4x4 and 8x8 SUs in 1.16. Also, I can't get the hammers to work. Is there any way to get the 2x2 SU or any smaller than 8x8?

Try the one from the discord, also the hammers are crafting items.
https://cdn.discordapp.com/attachments/723243821839941753/838224271004008489/smallerunits-1.16.jar

Thank you!

What about the 128x128?

configs

This can be closed.


Merge Ivraj's work into Jeffrey's

@Ivraj has done some great work reorganizing our code, as has @jdhe1120. Let's combine all this into one repo so we can make sure everyone's in sync.
Since Jeffrey and Ivraj have made different changes, we'll need to manually reconcile their changes. Then we can have everyone work off of that.

Looks great, guys!


github_actions

Please check code in workflow-> CI.yml

Here
409 != 201 : Could not create test Wishlist this error is persists if flask db-create was not included in the actions before green
hence included flask db-create
if you dont want to include flask db-create in actions then creation and deletion of table should be taken care in models.py when we create and delete


activeEditor.document.save() causes the editor to be closed

Does this issue occur when all extensions are disabled?: N/A

VS Code Version: 1.57.1
OS Version: windows 10 - 21H1

Steps to Reproduce:
I'm working on an extension that needs to save an untitled file before doing some operation on it. but after calling save() method the editor closes.
let isSaved;
if (isUntitled) {
	isSaved = await vscode.window.activeTextEditor!.document.save();
}
// in this line of code the editor is closed. why?
// do the rest

This is as designed, because the untitled document changes to a different document.
//cc @jrieken

Well..., but when user saves it e.g. by pressing CTRL+S it doesn't get closed.


Add API tools & schema generation

adds docker-compose config for Swagger Editor & Swagger UI
initial OpenAPI Spec (v3.0)
adds script to generate JSON schema's for validation
adds some examples for generated schema's

Next step is probably replacing Express by Fastify and using the schema's

updating the API spec so the schema's are correct

Also did some research about the current status of the OpenAPI and JSON schema standards but the tooling is currently not very up to date with the latest specs.
So this is Open API v 3.0 and we have to use JSON format or we can maybe use YAML and convert it to JSON with an extra step in the script, maybe good to ask people's preferences... might be easier to edit the API spec. We also need to check compatiblity with JSON schema and validation in Fastify if we add that.

Looks good for now, maybe the next step is migrate to fastify and search how we can use json schema at the routes level and see if it's possible to convert to openapi yaml

@neemiasjnr yeah there is currently a conversion from OpenAPI JSON to JSON schema but that doesn't do very much at the moment, maybe we can try if it works YAML => OpenAPI JSON => JSON Schema and then use it like that


Improve REPL startup message

Changes the startup message to be clearer.

Functionality is probably better than parity.


Swipe both ways

See https://github.com/superhuman/superhuman-ios/pull/4601

Whoops used the wrong branch


Command not found: __git_aliased_command

This is probably a dupe of https://github.com/robbyrussell/oh-my-zsh/issues/4553 but it was closed... Here is a gist of my omz_diagdump: https://gist.github.com/tdyer-aio/b3e8d9f794b521f2fd81
That said... running omz_diagnostic_dump seems to fix the issue I was having. (specifically git checkout top<tab> was causing the tab completion to have issues in the above manner, than it would start a function i defined in my zshrc that had been named test

Sounds similar to #4553 but this is new info since you're not running gitfast. Want to post this as a comment over there and we'll reopen it? No need to split in to separate issues.

will do

then it would start a function i defined in my zshrc that had been named test

Don't define a function called test. That's a standard shell command and redefining it will break things. I don't see it in your .zshrc, though.

running omz_diagnostic_dump seems to fix the issue I was having

Beats me why that would happen. omz_diagnostic_dump is supposed to be minimally invasive and not change anything about your shell state.
Continuing the rest of the conversation over on #4553.


Application data should be stored in the form_data table

Description
Because an application may have multiple revisions in the future, i.e. multiple form data, we need to store the json data in a separate table.
From a user's perspective, the current behaviour of the app should remain the same.

[ ] create a form_data table, containing a json_data jsonb column
[ ] create an form_data_status_type table, with the draft and submitted records hardcoded in it
[ ] create an application_form_data table, with an id and the following foreign keys: application_id, form_data_id
[ ] add a form_data computed column on application, to retrieve the current form data (ordering form_data by id)
[ ] the application_form_data table change should add RLS policies on the form_data table, ensuring that form_data can only be created by users if they have access to the corresponding application.
[ ] when creating an application, a form_data and application_form_data records should be created

This can be done after prod release, but will be more costly then as we will need to migrate existing forms, which will result in a short (<2min) outage

Hey team! Please add your planning poker estimate with Zenhub @wenzowski @VB-at-Bis

Is the application_form_data_status_type any different than application_status_type?

Yes, an application can have more statuses than the form itself. The form itself is either draft or submitted, that's it. I'm wondering it should be form_data_status_type instead of application_form_data_status_type, so that this type table is reused for the other forms we will create.

For whoever takes this on, please communicate whether it will or won't be shipped to prod before Wednesday, September 28th.

yes go ahead and delete the test env data


Uncaught ImportError in help.py line 114

Aider version: 0.59.2.dev45+ge76704e2
Python version: 3.12.4
Platform: macOS-10.16-x86_64-i386-64bit
Python implementation: CPython
Virtual environment: No
OS: Darwin 23.6.0 (64bit)
Git version: git version 2.46.1
An uncaught exception occurred:
Traceback (most recent call last):
  File "aider", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "main.py", line 771, in main
    coder.run()
  File "base_coder.py", line 730, in run
    self.run_one(user_message, preproc)
  File "base_coder.py", line 767, in run_one
    message = self.preproc_user_input(user_message)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "base_coder.py", line 756, in preproc_user_input
    return self.commands.run(inp)
           ^^^^^^^^^^^^^^^^^^^^^^
  File "commands.py", line 233, in run
    return self.do_run(matching_commands[0][1:], rest_inp)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "commands.py", line 208, in do_run
    return cmd_method(args)
           ^^^^^^^^^^^^^^^^
  File "commands.py", line 972, in cmd_help
    self.help = Help()
                ^^^^^^
  File "help.py", line 114, in __init__
    from llama_index.core import Settings
ImportError: cannot import name 'Settings' from 'llama_index.core' (/usr/local/Caskroom/miniconda/base/lib/python3.12/site-packages/llama_index/core/__init__.py)

Thanks for trying aider and filing this issue.
How did you install aider? What environment are you using? It looks like you don't have the correct set of dependencies installed. This doc may be helpful:
https://aider.chat/docs/troubleshooting/imports.html
I'm going to close this issue for now, but feel free to add a comment here and I will re-open. Or feel free to file a new issue any time.


Automations and Polling Interval Variability

Hi there
After a bit of messing about with my AX Pro, I have discovered a way to get motion detection events even when the alarm is not armed. This has spectacular automation and energy saving potential. I have described it here: https://community.home-assistant.io/t/hikvision-ax-pro-alarm/541882/5?u=5ur3n
However, to do something like turn the lights on when motion is detected, I've had to set the polling interval down to 1-2 seconds, but this causes the alarm to behave erratically, for instance,

I sometimes have to try multiple times to open gates or disarm, even with the keyfob.
The latency of loading the web ui or refreshing the app is a lot higher
Sometimes the alarm refuses to respond to a silence alarm for a minute or more.
PIRCam picture on demand times out in the HikConnect app.

My assumption here is that with 45-odd devices on the AX Pro, refreshing frequently might be maxing out the CPU or other resources and worsening latency.
I'm not very familiar with the ISAPI interfaces, but is it possible to create a workaround for this problem? Some ideas that could perhaps work:

have an asynchronous update back to Hass for a specific sensor
perhaps a group of sensors that are polled with higher frequency than the others
I'm not sure if the implementation is refreshing all configs when it refreshes or merely updating statuses for sensors it is already aware of, but perhaps minimizing the volume of calls will help.
I'm not sure if there is something like a CPU usage call but there isn't anything obvious in HikConnect to help with this.

My hypothesis here is that load reduction would mitigate the erratic behavior by reducing load on the hub with frequent calls. Of course, I have very little data to support this since there are no errors in the logs that I have seen.
What are your thoughts? Is this worth thinking about given the potential for smart automation?

This issue is being tracked basically from start. #2
As far as I know there is no API how to monitor the CPU Usage.
There is a SSH but I had not time to discover what we can do.
Workaround is basically impossible.

no API for "specific"  sensors
on update we are updating "statuses" not configs. Configs are updated only on start currently
If you use even "camera" monitoring on the device (linked cameras) it will put load on the CPU

All of that is a reason why default pull interval is 30 seconds. Because it can cause overload.
You can get a lot of "why something does not work as expected" from this picture

Thank you @petrleocompel that makes sense to me. I have a spare hub so I'll explore a bit what is possible with SSH, security logs and perhaps the hardware interfaces on the device like RS232.

I just wanted to check in...  I've not found a way to do this with ARC protocols. The AX Pro does not seem to forward these sorts of detection alerts.
But, do you know if the AX Pro supports the following? This seems to be implemented on cameras and NVRs, but did you ever try this on the AX Pro previously?
/ISAPI/Event/notification/alertStream

I just wanted to check in...  I've not found a way to do this with ARC protocols. The AX Pro does not seem to forward these sorts of detection alerts.
But, do you know if the AX Pro supports the following? This seems to be implemented on cameras and NVRs, but did you ever try this on the AX Pro previously?
/ISAPI/Event/notification/alertStream

I just wanted to check in...  I've not found a way to do this with ARC protocols. The AX Pro does not seem to forward these sorts of detection alerts.
But, do you know if the AX Pro supports the following? This seems to be implemented on cameras and NVRs, but did you ever try this on the AX Pro previously? If not I'll happily investigate.
/ISAPI/Event/notification/alertStream

I just wanted to check in...  I've not found a way to do this with ARC protocols. The AX Pro does not seem to forward these sorts of detection alerts.
But, do you know if the AX Pro supports the following? This seems to be implemented on cameras and NVRs, but did you ever try this on the AX Pro previously? If not I'll happily investigate.
/ISAPI/Event/notification/alertStream

I just wanted to check in...  I've not found a way to do this with ARC protocols. The AX Pro does not seem to forward these sorts of detection alerts.
But, do you know if the AX Pro supports the following? This seems to be implemented on cameras and NVRs, but did you ever try this on the AX Pro previously? If not I'll happily investigate.
/ISAPI/Event/notification/alertStream

@5ur3n  it is very rude to send the message 5 times...


weights of trained model

Can you share it via Baidu disk? It would be easier to download the final trained model #32

We have updated the logs and weights we have.


Improve query entities and usability

Codecov Report

Merging #12 (c19a36c) into main (70e5fa9) will increase coverage by 0.44%.
The diff coverage is 87.50%.

@@             Coverage Diff              @@
##               main      #12      +/-   ##
============================================
+ Coverage     84.52%   84.97%   +0.44%
- Complexity      330      364      +34
============================================
  Files            61       68       +7
  Lines          1144     1278     +134
  Branches         88       95       +7
============================================
+ Hits            967     1086     +119
- Misses          129      141      +12
- Partials         48       51       +3

Impacted Files
Coverage Δ

...gnite/migration/dao/IgniteCacheMetaDataWriter.java
94.44% <ø> (ø)

...liedium/ignite/migration/dao/IgniteDataWriter.java
85.71% <ø> (ø)

...ignite/migration/dao/IgniteWritersFactoryImpl.java
100.00% <ø> (ø)

...igration/dao/converters/BinaryObjectConverter.java
100.00% <ø> (ø)

...ao/datamanager/BinaryObjectFieldsInfoResolver.java
85.00% <ø> (-1.37%)
:arrow_down:

...lliedium/ignite/migration/dto/CacheEntryValue.java
87.50% <ø> (ø)

...um/ignite/migration/serializer/AvroFileWriter.java
72.22% <ø> (+3.17%)
:arrow_up:

...ain/java/org/alliedium/ignite/migration/Utils.java
63.82% <50.00%> (-4.75%)
:arrow_down:

...ite/migration/dao/dataaccessor/IgniteCacheDAO.java
66.66% <66.66%> (+3.03%)
:arrow_up:

...ion/properties/AtomicLongNamesProviderFactory.java
72.97% <72.97%> (ø)

... and 19 more

Continue to review full report at Codecov.

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


Sensu Redis - Support boolean redis_tls parameter (cont #967)

Pull Request Checklist

Description

Add on to work in #967
Related Issue

Fixes #900 and extends #967 .
Motivation and Context

Ensure redis_tls is a proper boolean property and ensure the provider is checking the symbol value.
How Has This Been Tested?

I modified tests/sensu-server.pp to add redis_tls => true:
[root@sensu-server ~]# puppet apply sensu-server.pp
Notice: Compiled catalog for sensu-server.example.com in environment production in 1.87 seconds
Notice: /Stage[main]/Sensu::Redis::Config/Sensu_redis_config[sensu-server.example.com]/tls: tls changed 'false' to 'true'
Notice: /Stage[main]/Sensu::Api/Service[sensu-api]: Triggered 'refresh' from 2 events
Notice: /Stage[main]/Sensu::Server::Service/Service[sensu-server]: Triggered 'refresh' from 1 events
Notice: Applied catalog in 3.21 seconds

Resulting config:
[root@sensu-server ~]# cat /etc/sensu/conf.d/redis.json
{
  "redis": {
    "port": 6379,
    "host": "<IP_ADDRESS>",
    "reconnect_on_error": true,
    "db": 0,
    "auto_reconnect": true,
    "tls": {
    }
  }
}

General

[ ] New parameters are documented

[ ] New parameters have tests

[ ] Tests pass - bundle exec rake validate lint spec

Worth noting testing with tests/sensu-server.pp is not great because once tls support is added to redis.json it breaks sensu-server service since the Vagrant environment doesn't configure Redis for TLS, is my guess.

@jaredledvina Could you please confirm this works for you.

@ghoneycutt - Yep, I'm AFK but will check once I'm back at a computer (probably early AM Tuesday). Thanks!

Alright, sorry for the delay, I applied this branch to our internal version of the module and undid my forked changes. Gave it a run on our Sensu instances and it works like a charm.
And our redis.json file looks solid!
> cat /etc/sensu/conf.d/redis.json
{
  "redis": {
    "port": 6379,
    "host": "SNIP",
    "password": "SNIP",
    "reconnect_on_error": true,
    "db": 0,
    "auto_reconnect": true,
    "tls": {
    }
  }
}

Amazing, thanks so much @treydock and @ghoneycutt !

Released in v2.54.0


User Feedback: flag the violations which are New violations since the last time Forseti ran

Issue Review: Team thought this was important and should be thought about and prioritized in Auditor 2.0 work.

Design pending.

Design doc created and shared with team internally.


.NET Standard / Core Support

This is a very interesting library -- I'd like to try it out in my engine. Unfortunately, it doesn't support .NET Core as-is. Any desire to support an alternate configuration, or switch to .NET Standard as the build target? From my experimentation, it seems that the library will compile without any code changes.

Unity3D runs .NET Standard 2.0 now. Please target that and release on nuget.

Hi, I tested right now in a Console App with .NET Core 3.1 and it is working perfectly


Warnings when compiling Python bindings on macOS + clang 12.0.0

I'm getting the following warnings while compiling the Python bindings on macOS:
ld: warning: direct access in function 'pybind11_init_cflinkcpp(pybind11::module_&)' from file '/tmp/lto.o' to global weak symbol 'bitcraze::crazyflieLinkCpp::Packet::valid() const' from file 'libcrazyflieLinkCpp.a(Connection.cpp.o)' means the weak symbol cannot be overridden at runtime. This was likely caused by different translation units being compiled with different visibility settings.
ld: warning: direct access in function 'pybind11_init_cflinkcpp(pybind11::module_&)' from file '/tmp/lto.o' to global weak symbol 'bitcraze::crazyflieLinkCpp::Packet::port() const' from file 'libcrazyflieLinkCpp.a(Connection.cpp.o)' means the weak symbol cannot be overridden at runtime. This was likely caused by different translation units being compiled with different visibility settings.
ld: warning: direct access in function 'char const* std::__1::__find_end<bool (*)(char, char), char const*, char const*>(char const*, char const*, char const*, char const*, bool (*)(char, char), std::__1::random_access_iterator_tag, std::__1::random_access_iterator_tag)' from file '/tmp/lto.o' to global weak symbol 'std::__1::char_traits<char>::eq(char, char)' from file 'libcrazyflieLinkCpp.a(Connection.cpp.o)' means the weak symbol cannot be overridden at runtime. This was likely caused by different translation units being compiled with different visibility settings.

The Python bindings seems to work fine, though, at least I could definitely run the console example without problems so I'm not sure this is something to worry about.

sorry for the late reply... I guess this is still an issue for you currently right?
Probably not immediately something to worry about I would say. @whoenig is more acquinted with python binding though.

CMakeLists.txt patch as outlined in the previous comment is not necessary any more now that<PHONE_NUMBER>0a6b5538afc91aed5059e77f5a06eb was committed.
The original warnings are still there, but indeed they don't seem to cause any problems.


Does CollisionShape::resource_changed need its parameter?

While documenting CollisionShape I noticed this method has a parameter of "RES res" but then it doesn't use it for anything. Should the parameter be removed?
Here is the entire method definition:
`void CollisionShape::resource_changed(RES res) {
update_gizmo();

}`

I don't think so. It might be a GDScript binding and if number of parameters doesn't match, it'll print an error message and the method won't be called.

Okay, so this particular case isn't a call from C++ into GDScript, but from GDScript into C++. It doesn't make sense for the user to pass in a resource that's then completely ignored. In this case it's seems to make much more sense to remove the binding altogether and just have them call 'update_gizmo' which is also a binding for GDScript.
If this was a call into the user's script it would make sense, but it doesn't look like it is. I'm quite sure
API breakage is acceptable for Godot 3.0, especially on a method that is likely not used in most cases, right?
GridMap does the exact same thing, but it looks like no other classes in all of Godot do this.

In core/resource.cpp:
obj->call("resource_changed", RES(this));
I'm not sure. AFAIK, currently nothing uses the given parameter, but in the future things might change. The implementation is very odd however. It'd make more sense if it were a virtual method.

After doing some testing, it calls "resource_changed" internally whenever the 'Shape' resource is modified, as expected. If two nodes share the same resource they are both called internally, as expected.
I'm now puzzled as to why this is user API facing?
It seems:
A) Redundant, since it's already called internally when a resource changes
B) Confusing, since the argument passed does nothing in this nor the GridMap node, and effectively does nothing because of point A
C) Something the user should never have to worry about anyway since any change notifications should be recognized internally... which they currently are.

Could be useful for making an EditorPlugin? It should be a notification, a signal, or a virtual method though.

Thanks for helping me to work through this issue. I'm still getting used to some things about Godot.
I made a pr making these methods virtual in both C++ and the documentation. Even before this pr, if the 'resource_changed' function is added to GDScript it overrides the C++ version, thus acting as if it were virtual. I left the binding so the user can still call back into the C++ version.


Redis command listener

This builds on #1791 with the following:

adds JedisCommandListener.commandConnected() to allow tracking of connection time metrics separately from command execution time
adds coverage with unit tests
rebases from the current master

More details from the PR #1791 description:
As per https://groups.google.com/forum/?fromgroups#!topic/jedis_redis/UaJubt42f_0,
This PR adds a command listener that allows for
sane integration with tracing libraries like http://opentracing.io/
capturing metrics and integration with metric libraries like http://metrics.dropwizard.io/4.0.0/
integration with logging libraries
None of the public signatures was changed, so the change should be fully backwards compatible.

@gkorland , any update here?  I believe I've addressed all your concerns.

Did you check my latest comments?

@gkorland which comments?  AFAICT they've all been addressed.

Any update here @gkorland ?

Hey, @gkorland! Would you be able to take a look at the PR? I believe it's almost there, just need to agree on some minor points :)

Hey, @gkorland, @sazzad16! Would you be able to take a look? There is clearly a need for this, https://github.com/xetorthio/jedis/pull/1791#issuecomment-515966697.

@SerCeMan can you please rebase the PR

OK, I've updated to the latest from master @gkorland


Use indy-client 3.4.0-SNAPSHOT

Because indy-client 3.4.0-SNAPSHOT has removed the o11yphant deps,
this needs some code change in Indy side.

LGTM.


[3.2] [iOS] Native loading screen implementation

Same as #40766 for 3.2

If storyboard named Launch Screen is present it will be used as overlay while Main performs setup and setup2.

Removes the black screen that appears after initial launch screen disappears and before game starts.

Thanks!


gateway & p2p improvements

Currently many requests go and invoke fetch_bitswap with an empty provider list (ref trace ID 2b90b66cb34b936f005cde120d35de0e). We should fail fast here and return 404 or something similar.
The response from the resolver on gateway should be a stream, both to avoid loading everything into memory and significantly reduce the time to first byte which is currently t(fetch_bitswap) + some_overhead while it could be t(fetch_bitswap_first_byte) + some_overhead. This also allows us to return metrics like time to first byte/block, latency vs time etc.

The response from the resolver on gateway should be a stream

Given that our blocks are limited at 2MiB, and that neither our DB nor the rpc layer are currently streaming them, I am not sure this is actually worth the work currently.

The response from the resolver on gateway should be a stream

Given that our blocks are limited at 2MiB, and that neither our DB nor the rpc layer are currently streaming them, I am not sure this is actually worth the work currently.

Agree, this was more a "we should do it eventually thing"


Phoebemirman/ch12748/dispute round is empty

This pull request introduces 1 alert when merging 78e5c0aded742b6c929207036c74b2b4693807e3 into dfba8658ea96809e54e7d6b5c9fb446a9c105ed3 - view on LGTM.com
new alerts:

1 for Syntax error

Comment posted by LGTM.com


Fetching large chunks

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
This PR fixes issue with fetching chunks that are bigger than 16000 bytes. Under some circumstances it is possible to come across chunks like that, and under some conditions Thanos may currently fail to fetch such chunks. This PR adds fetching of remaining chunk data from the bucket.
Verification
Added unit test. Also tested manually against real block found in our test cluster.
Fixes issue https://github.com/thanos-io/thanos/issues/2945.

Thanks!

I'm joining the party very late, but good job @pstibrany ! 👏


fix double trap

Summary and Scope

fix double trap

Issues and Related PRs
List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

Resolves [issue id](issue link)

Testing
List the environments in which these changes were tested.
Tested on:

<development system>
Local development environment
Virtual Shasta

Test description:
How were the changes tested and success verified? If schema changes were part of this change, how were those handled in your upgrade/downgrade testing?

Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)?
Were continuous integration tests run? If not, why?
Was upgrade tested? If not, why?
Was downgrade tested? If not, why?
Were new tests (or test issues/Jiras) created for this change?

Risks and Mitigations
Are there known issues with these changes? Any other special considerations?
Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[ ] License file intact
[ ] Target branch correct
[ ] CHANGELOG.md updated
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

/backport main


Cloudflare workers give error

Hi
I am trying to use cohere embedding in Cloudflare Workers, when I initialize the client, I get the error
The version I am using is 7.6.2
import { CohereClient } from "cohere-ai";
const cohere = new CohereClient({
    token: "_______"
});
const handler = {
    async fetch(request: Request, env: Env, ctx: ExecutionContext): Promise<Response> {

        let result = {}
        try {
            result = await cohere.embed( { texts:["test 123", "New teste 123"], model:"embed-multilingual-light-v3.0", inputType:"search_document"})

            return Response.json(result);
        } catch (e) {
            console.log(e)
        }
        return Response.json(result);
    },
  };

export default handler;

✘ [ERROR] service core:user:backend: Uncaught ReferenceError: global is not defined

    at db_addchunk.js:3605:7 in node_modules/cohere-ai/core/fetcher/Fetcher.js
    at db_addchunk.js:11:50 in __require
    at db_addchunk.js:3801:21 in node_modules/cohere-ai/core/fetcher/index.js
    at db_addchunk.js:11:50 in __require
    at db_addchunk.js:6466:18 in node_modules/cohere-ai/core/index.js
    at db_addchunk.js:11:50 in __require
    at db_addchunk.js:12524:29 in node_modules/cohere-ai/Client.js
    at db_addchunk.js:11:50 in __require
    at db_addchunk.js:13146:20 in node_modules/cohere-ai/index.js
    at db_addchunk.js:11:50 in __require

In Fetcher.js if I comment this line it works
const form_data_1 = __importDefault(require("form-data"));
const qs_1 = __importDefault(require("qs"));
if (typeof window === "undefined") {
    //global.fetch = require("node-fetch");    <-- comment this line and works
}

@bonadio thanks for reporting this issue. To get around this issue in cloudflare workers, you can try enabling the node-compat flag. See the docs here: https://developers.cloudflare.com/workers/runtime-apis/nodejs/#enable-nodejs-with-workers.
For context, Cohere's SDKs are automatically generated using Fern. For the longer term, we'll see if we can make some updates so that the SDK can run without having to enable node-compat.

Closing in favour of https://github.com/fern-api/fern/issues/2555

Hey @bonadio this should be working as of 7.7.5. Let me know if you have any further issues! Thanks


Certain blocks saying 5%

Just a note this is from a dev environment but i havent even implemented top yet

That's the block break indicator. Does your mod do anything unusual to block breaking?

Not a thing is changed. source can be found here

Ok so i restarted my instance and now its not showing?!? close the issue?

I suppose so. Let us know if you can figure out how to reproduce it.


HDDS-9725. Recon - OpenKeys Pagination Issue and Search Functionality Issue

What changes were proposed in this pull request?

Search Functionality for Open Keys was not working because of Non Unique key passed as unique Columns in Table.  so changed rowKey='path' from rowKey="key" keys which are unique values.
Pagination  Count was not working properly so addressed this issue.

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9725
How was this patch tested?
Manually and in Cluster Also
After Doing Testing

After Clicking on Non FSO

cc: @devmadhuu

@smitajoshi12 , thanks for the patch, can you show how the open key table data looks like with non-fso data ?

@smitajoshi12 , thanks for the patch, can you show how the open key table data looks like with non-fso data ?

Yes  devesh attached new screenshot.

Hello @smitajoshi12, I was testing your PR locally, and ran it with local mock data.
It would be great if you could clarify the below points:

First page is showing No Records. But when we click next page, we are getting the page as 1. Could you check if the state is being set properly here?

Also the number of entries do not change when we change the entries per page from dropdown. I checked the network request and it seems we are not passing a limit param in the request.

Hello @smitajoshi12, I was testing your PR locally, and ran it with local mock data. It would be great if you could clarify the below points:

First page is showing No Records. But when we click next page, we are getting the page as 1. Could you check if the state is being set properly here?

* Also the number of entries do not change when we change the entries per page from dropdown. I checked the network request and it seems we are not passing a limit param in the request.

@devabhishekpal  Abhishek
Checked with cluster data and can see limit got changed.

Hello @smitajoshi12, thanks for the info, I checked locally and the limit param is being sent properly.
For the pagination as I understood from the call, No Records is the placeholder equivalent for << or the prev-page button.
But I was suggesting that, we can instead start page 1 from there and completely remove previous page button from first page.
Then onwards on page 2 it will be << 2 >> and so on.
So:
1 >> then << 2 >>, << 3 >>, << 4
This is assuming 4th page is the last page.

Hello @smitajoshi12, thanks for the info, I checked locally and the limit param is being sent properly. For the pagination as I understood from the call, No Records is the placeholder equivalent for << or the prev-page button.
But I was suggesting that, we can instead start page 1 from there and completely remove previous page button from first page. Then onwards on page 2 it will be << 2 >> and so on.
So: 1 >> then << 2 >>, << 3 >>, << 4 This is assuming 4th page is the last page.

@devabhishekpal   Addressed above Issue in Current Comit. Please check

Hello @smitajoshi12, thanks for the info, I checked locally and the limit param is being sent properly. For the pagination as I understood from the call, No Records is the placeholder equivalent for << or the prev-page button.
But I was suggesting that, we can instead start page 1 from there and completely remove previous page button from first page. Then onwards on page 2 it will be << 2 >> and so on.
So: 1 >> then << 2 >>, << 3 >>, << 4 This is assuming 4th page is the last page.

I agree with @devabhishekpal, I wouldn't show the No Records, rather just nothing. Just like in the example he wrote in the last sentence.

Thanks for working on this @smitajoshi12! I tried it locally and I faced these:

If I click on the number, which page I am on, it will forward me to the next page. I don't think it should work like this.
Corrected in the current commit.

If I'm on the last page, I still see the arrows next to it and I'm able to click on it, which forwards me to the next, non-existing page, where the table is empty. On that page I can't navigate back, so I'm stuck there. This should be fixed.
Yes This is known issue we need to work with backend and frontend changes so will address this issue in seprate JIRA.

When I selected to show 20 (or more) items per page nothing changed, I still saw 10 per page.
Current Mock Data we do not have any provision to with large data in Db.json but it is working correctly with Cluster data.

Can you check these out? Please next time test manually the basic functionalities more thoroughly.
Yes ZIta

@dombizita
I have tried to override Default Pagination Looks but it was having CSS Issues as here we are dynamically updating API call on  >> Button. Removed No Further Records and No Records. On Clicking Number it will not do navigation.

Thanks for the modifications, @smitajoshi12. Please include a button that allows users to return to the previous page if there are no remaining records. Alternatively, we can prevent users from viewing the last empty page by notifying them that there are no records left.

Also based on our offline discussion, we encountered a minor intermittent glitch where the page number was displayed as zero. Please investigate and address this issue to ensure accurate page number representation.

Thanks for the modifications, @smitajoshi12. Please include a button that allows users to return to the previous page if there are no remaining records. Alternatively, we can prevent users from viewing the last empty page by notifying them that there are no records left.
Also based on our offline discussion, we encountered a minor intermittent glitch where the page number was displayed as zero. Please investigate and address this issue to ensure accurate page number representation.

@ArafatKhan2198
Yes Arafat completed all changes and attached screenshot with latest changes.

While working and reviewing this patch we found that the pagination on the OM DB Insights page is not working correctly at all. @smitajoshi12 tried to fix it this PR, but it seems that part needs a bigger change, so we agreed to do that in another jira. I filed HDDS-10282 to do that.
The problem is that currently all the requests (for all 4 endpoints) have both the limit and the prevKey parameters set. The limit is set based on the pages size and the prevKey based on the previous page's last element. Another thing we should notice is that we can set the page size (10, 20, 30, 50). To be able to navigate back we would need to know the last element of the previous page's previous page. To solve this @smitajoshi12 created a map in this PR, where we are storing the page numbers as keys and the last elements of the previous pages as values paired to it. After you put a key-value pair to the map, they won't change the value later.
The problems we faced because of this:

If we are changing the size of the page, we will read the prevKey from the map, which will be the same that was already calculated for a different page size; only the limit will be changed in the request. We will face a problem, where we are showing the same elements on consecutive pages. I described one scenario here: https://github.com/apache/ozone/pull/5658#discussion_r1469353863
On each page we are only loading the page size number of elements, so all the sorting and searching (based on the columns) will happen only with the elements on the page.

I looked into what kind of solutions are there at other UI platforms. First of all a problem that we are facing, is that we have key-value store table interface, so we can't get an element based on offset.
Another solution could be if we won't let users to change the page size and use the map solution, but I don't like the idea that the search/sorting is not working on the whole dataset that we have requested.
I'd suggest to follow the same solution that @ivandika3 did in #4585, so we would retrieve a limited amount of the data, that can be set via the user and the whole pagination is solved by the ant library. Later we can add an interface in Recon, where you can ask for a specific key/container with the APIs present on the OM DB Insights page.
What do you think of this approach? @smitajoshi12 @ArafatKhan2198 @devmadhuu

Just FYI, I created the ticket for index-based pagination support in https://issues.apache.org/jira/browse/HDDS-9338. However, as @dombizita said, this require a substantial change of the Recon backend from KV store (RocksDB) to embedded SQL DB (Derby, SQLite, DuckDB).

Hi @smitajoshi12, I just merged #6190, I think everything that was planned here is fixed in that PR, do you think we can close this? Can you check it if anything is missed there? Thanks!

Closing this PR Same PR has been raised and merged by Zita.
https://github.com/apache/ozone/pull/6190

Hi @smitajoshi12, I just merged #6190, I think everything that was planned here is fixed in that PR, do you think we can close this? Can you check it if anything is missed there? Thanks!

Will Close this PR.


Using accounts displays confirmation dialog

When on the account transfer tab, if you click the "wheel icon", a dialog pops up asking to set the confirmations and outputs. See screenshot.

Confirmed not happening anymore in latest build.

Yes, currently looks good


Scope launch returns no helpful message when image pulling fails

vagrant@jessie-amd64:~$ ./scope launch
Unable to find image 'weaveworks/scope:latest' locally
Pulling repository weaveworks/scope
time="2015-06-11T09:37:03Z" level=fatal msg="Get https://index.docker.io/v1/repositories/weaveworks/scope/images: dial tcp: lookup index.docker.io on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:53: i/o timeout"
Unable to find image 'gliderlabs/alpine:latest' locally
Pulling repository gliderlabs/alpine
time="2015-06-11T09:37:18Z" level=fatal msg="Get https://index.docker.io/v1/repositories/gliderlabs/alpine/images: dial tcp: lookup index.docker.io on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:53: i/o timeout"

Weave Scope is reachable at the following URL(s):
vagrant@jessie-amd64:~$

+1 Same here.

@c1freitas Sorry about that!  I'll work on a nicer message tomorrow.

Thats better:
vagrant@ubuntu-14:~/src/github.com/weaveworks/scope$ ./scope launch
Unable to find image 'weaveworks/scope:latest' locally
Pulling repository weaveworks/scope
time="2015-06-23T14:26:19Z" level=fatal msg="Get https://index.docker.io/v1/repositories/weaveworks/scope/images: dial tcp <IP_ADDRESS>:443: connection timed out"
vagrant@ubuntu-14:~/src/github.com/weaveworks/scope$


Update react flow example with the react flow renderer version 10

Hello,
The example on this repository is using react flow renderer version 9. Could you update the example to use the react flow renderer version 10?

Great idea! Thanks for telling me.
v10 has a fundamentally different state implementation, so it may take a little while to get everything working properly.
Essentially, the only thing you really need to do is split the useUndoable state into an object with separate arrays for nodes and edges.
useUndoable({
    nodes: initialNodes,
    edges: initialEdges,
});

The problem I am running into is the onNodesChange function. It will change the state after literally every pixel move. This means that when you undo after moving a node, it will start by undoing each individual pixel you move. The only solution to this problem is the onNodeDragStop function, but that doesn't seem to play nicely with a custom state object.
I'll update it to v10 when I can. Thanks again for letting me know!

Yes, I am facing the same situation. To avoid every pixel move, I am filtering the changes on onNodeChangeCallback
const change = changes[0]
if (change.type === "position" && change.dragging) {
  return;
}
else {
  // update state
}

I am also having trouble when I try to delete a node. When I try to delete a node two events are fired one after the other. First, a remove event is fired followed by a select event. The remove event removes the node, but the select event recreates it again. For now, I am filtering the select event, like this:
const change = changes[0];
if (change.type === 'select'){
  return
}

I am having the same issue, actually.
Could you share some more of your code?

I forgot to mention the principal thing. I am managing the state and the historic separately. I am still using useState for nodes and edges, and for the history I am using useUndoable. I still do not have a working example. But if helps this is what I get so far.
const handleHistory = (changes: NodeChange[]) => {
  const change = changes[0];
  if (
    change.type === "select" ||
    (change.type === "position" && change.dragging)
  ) {
    return;
  }

  setHistory((oldElements: Elements) => {
    return {
      // we need to get current updated nodes
      // maybe we can pass the new nodes
      nodes: []
      edges: oldElements.edges,
    };
  });
};

const onNodesChange = (changes: NodeChange[]) => {
  // current code
  handleHistory(changes);
  setNodes((oldNodes) => applyNodeChanges(changes, oldNodes));

  // future code
  // Need to test passing the new nodes
  // setNodes(oldNodes => {
  //   const newNodes = applyNodeChanges(changes, oldNodes);
  //   handleHistory(newNodes);
  //   return newNodes}
  //   );
};

I forgot to mention the principal thing. I am managing the state and the historic separately. I am still using useState for nodes and edges, and for the history I am using useUndoable. I still do not have a working example. But if helps this is what I get so far.
const handleHistory = (changes: NodeChange[]) => {
  const change = changes[0];
  if (
    change.type === "select" ||
    (change.type === "position" && change.dragging)
  ) {
    return;
  }

  setHistory((oldElements: Elements) => {
    return {
      // we need to get current updated nodes
      // maybe we can pass the new nodes
      nodes: []
      edges: oldElements.edges,
    };
  });
};

const onNodesChange = (changes: NodeChange[]) => {
  // current code
  handleHistory(changes);
  setNodes((oldNodes) => applyNodeChanges(changes, oldNodes));

  // future code
  // Need to test passing the new nodes
  // setNodes(oldNodes => {
  //   const newNodes = applyNodeChanges(changes, oldNodes);
  //   handleHistory(newNodes);
  //   return newNodes}
  //   );
};

In this particular case, using multiple state values can easily get tricky; keeping them in sync can lead to some weird bugs.
There's really no need to manage them separately. If you only want to use the history internally, just don't give users the option to undo. You can access history, as you might know, by using the future or past variables that the hook gives you.
The main thing to note, and why you're having a problem, is that useUndoable is not primarily a history tracker: its primary purpose is to manage state, keeping a record of which is only a feature.
The reason we are both getting issues with v10 is because we are trying to manage multiple state instances separately.
The only solution is to make useUndoable the exclusive manager of state.
For more information, see the updated  react-flow-example and the README for more information about v10.
Thanks again for opening an issue!

I am also having trouble when I try to delete a node. When I try to delete a node two events are fired one after the other. First, a remove event is fired followed by a select event. The remove event removes the node, but the select event recreates it again. For now, I am filtering the select event, like this:
const change = changes[0];
if (change.type === 'select'){
  return
}

@xplato, what about this issue? Have you found a solution? I am not able to delete any nodes.

Are you filtering the change.type from the node data directly? Is this an accurate way to know what changes were made?
From React Flow's documentation, it doesn't look like the onNodesChange function exports some identifier related to the actual action.
The best solution would probably be to create a new reducer and find a way to dispatch actions from onNodesChange instead of piping the result directly into state.
For instance, instead of
const onNodesChange = useCallback(
    (changes) => {
        triggerUpdate('nodes', applyNodeChanges(changes, elements.nodes));
    },
    [triggerUpdate, elements.nodes]
);

you'd need to find a way to specify specific actions and dispatch them explicitly:
const onNodesChange = useCallback(
    (changes, actionType) => {
        dispatch({ type: actionType, payload: applyNodeChanges(changes, elements.nodes) });
    },
    [triggerUpdate, elements.nodes]
);

You would then, of course, manually modify the state for each specific action.

As for the issue where moving a node will add tons of new re-renders for each pixel, you could reduce this by changing the state representation onNodeDragStart and then reset it onNodeDragStop.
<ReactFlow
    nodes={hasUserStartedDragging ? stateOfNodesBeforeDragStart : elements.nodes}
/>

The react-flow-example is intended to be a simple example demonstrating useUndoable integration with React Flow. Since useUndoable itself is working as expected, all of these little quirks need to be solved on a project-by-project basis.
This means that the way you determine which actions to dispatch is up to you. To be honest, I'm not sure exactly how to differentiate between different actions. You could try opening an issue in React Flow and asking if adding the change type to the onNodesChange, onEdgesChange functions. That is, instead of just the changes, you also pass a string ID of the action like so:
type Change = 'move' | 'delete' | 'select'; // ... and so on
const onNodesChange = (changes: Node[], changeType: Change) => { ... }

If you have any issues implementing what I've suggested here, feel free to reach out.

@carlosporta @xplato Hello guys.
First of all, thank you very much @xplato for your work on useUndoable, it's awesome!
I saw you both have an issue that I'm facing now : the "pixel by pixel" undo when moving a node.
Have you found any solution for this problem ?
Thanks in advance!

Hello, @jimymltta!
I haven't tested yet, have you tried the solution proposed by @xplato?

As for the issue where moving a node will add tons of new re-renders for each pixel, you could reduce this by changing the state representation onNodeDragStart and then reset it onNodeDragStop

<ReactFlow
    nodes={hasUserStartedDragging ? stateOfNodesBeforeDragStart : elements.nodes}
/>

@jimymltta Thank you for your kind words!
As @carlosporta pointed out, I made a suggestion on how to avoid this, but I'm not sure if it will work consistently. Really, the ideal solution would be if the onNodesChange function could ignore move events, so that you could hook those up manually.
I really hope React Flow later finds a way to identify specific events within these catch-all change functions.
I wish you guys the best! Let me know if I can help.

Late here, but - for posterity - one option is to do something like this:
const [undoableLoggingDisabled, setUndoableLoggingDisabled] = useState(false)
...
// adopted from https://github.com/Infinium8/useUndoable/blob/main/react-flow-example/src/App.jsx#L32
const updateElements = useCallback(
  (t: 'nodes' | 'edges', v: Array<any>) => {
    setElements(
      e => ({
        nodes: t === 'nodes' ? v : e.nodes,
        edges: t === 'edges' ? v : e.edges,
      }),
      null,
      // @ts-ignore https://github.com/Infinium8/useUndoable/issues/9
      undoableLoggingDisabled
    )
  },
  [setElements, undoableLoggingDisabled]
)
...
const onNodeDragStart = useCallback(
  () => {
    setUndoableLoggingDisabled(true)
  },
  [setUndoableLoggingDisabled]
)

const onNodeDragStop = useCallback(
  () => {
    setUndoableLoggingDisabled(false)
  },
  [setUndoableLoggingDisabled]
)

does anyone have a working solution for this problem? I have tried the suggestions above but have had no success.

the issue with the @rfdearborn solution is that you receive a onNodeDragStart even when a node is selected, so you end up disabling logging forever until a node is dragged (and the onNodeDragStop event is triggered)


Better way to add single entries to cache config

If I want to add a single new configuration cache without removing existing ones, I have to do some work in the set_config() call:
caches.set_config({
    **caches.get_config(),
    'new_cache': {
        'cache': 'aiocache.SimpleMemoryCache',
        'namespace': 'new_namespace',
    },
})

What do you think about adding a new method, add_config(name, <params>) that just adds a new configuration cache?
caches.add_config('new_cache', cache='aiocache.SimpleMemoryCache', namespace='new_namespace')

The way the factory/cache manager was thought originally was a way to set up a set of caches that would then later be used in the different services.
In general the pattern is:

Have a config file defining the caches you want
In app startup time call caches.set_config() with the whole configuration
Use the aliases later in the code

What's your use case? Are you initializing each alias in the service constructors? Is there a specific reason why you doing it this way?

I have a project with several client libraries which are completely decoupled from the main app. It made sense to have their configuration separated, inside their own file, instead of the main app.

Then those caches are local to the services? Why don't you create the cache explicitly or using caches.create? No need to call caches.set_config and retrieve them with an alias.
Naming caches with an alias and storing their config with caches.set_config is in case you want to reuse them in any part of your code.


Feature/update readme

Ⅰ. Describe what this PR did

add en readme
add zh readme

Ⅱ. Does this pull request fix one issue?
no
Ⅲ. Why don't you add test cases (unit test/integration test)?
no need
Ⅳ. Describe how to verify it
check readme markdown UI
Ⅴ. Special notes for reviews
none

@dk-lockdown 有空时候来review一下哇~


initialDirectory not working in OpenFilePicker

Hi,
I run the walpaper example with flutter and initialDirectory is not taken into account by the app even at fresh start after rebooting computer.
My system : windows 10 pro 22H2
How to reproduce :
A)
run the walpaper example with "Flutter run lib/main.dart" and choosing Windows option.
What I observed : at start the app is not opening in "My Pictures" folder but in the last directory I opened with a flutter app.
What  I did :
1) I checked that the 'picturesDirectory' variable was correct with a simple print(picturesDirectory)
2) I tried to change the value of "initialDirectory" by a correct folder path on my computer but without success.
B)
run the following code with dart :
import 'package:filepicker_windows/filepicker_windows.dart';

void main() {
  final file = OpenFilePicker()
    ..filterSpecification = {
      'Word Document (*.doc)': '*.doc',
      'Web Page (*.htm; *.html)': '*.htm;*.html',
      'Text Document (*.txt)': '*.txt',
      'All Files': '*.*'
    }
    ..defaultFilterIndex = 0
    ..defaultExtension = 'doc'
    ..title = 'Select a document'
    ..initialDirectory = "C:\\some_path";

  final result = file.getFile();
  if (result != null) {
    print(result.path);
  }
}

=> initialDirectory is not taken into account and FilePicker propose the last directory opened.

Thanks for reporting this issue.
I just released a new version (v2.1.2), in which I added a new field alwaysShowInitialDirectory to control whether the dialog box should always show the directory defined in initialDirectory to the user, regardless of previous user interaction.
Try this:
import 'package:filepicker_windows/filepicker_windows.dart';

void main() {
  final file = OpenFilePicker()
    ..filterSpecification = {
      'Word Document (*.doc)': '*.doc',
      'Web Page (*.htm; *.html)': '*.htm;*.html',
      'Text Document (*.txt)': '*.txt',
      'All Files': '*.*'
    }
    ..defaultFilterIndex = 0
    ..defaultExtension = 'doc'
    ..title = 'Select a document'
    ..initialDirectory = "C:\\some_path"
    ..alwaysShowInitialDirectory = true;

  final result = file.getFile();
  if (result != null) {
    print(result.path);
  }
}


python3Packages.dask-glm: use pytestCheckHook

Also use pytestCheckHook.

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 120318 at 0258f11f run on aarch64-linux 1

1 package marked as broken and skipped:

python38Packages.dask-ml

2 packages built successfully:

python38Packages.dask-glm
python39Packages.dask-glm

Can you rebase? setuptools-scm got already fixed with #120233

python3Packages.dask fails to build, so I cannot test this.

Either wait with this until staging-next is merged or push it to staging-next.

/rebase staging-next

/rebase master

rebased to master again because staging-next got merged.

rebased to master again because staging-next got merged.

You could just have changed the target.

@ofborg build python38Packages.dask

Is this still needed after fc98e85759e93d3041026c91150f4303d706f272 ?

@nbren12 Can you test this on macOS?

@nbren12 Can you test again on macOS?


Segmenting regions

How best to segment regions correctly:

only on TH is too sensitive
NET
and/or SLC6A2
and/or DBH
SNAP25 (may be too many spots)

Alternatively:

extending region by a few pixels (but is also not very precise)

Alternatively: manual annotation of regions

either cut-out of whole region
or individual spots/cells

Two types of analyses, both useful

As discussed on 4 Nov: try manual annotation of regions of interest, using Shiny app with high-resolution image
2 types of manual annotation, which will be used for 2 sets of analyses:

cut-out of whole region including a small amount of nearby white matter
NE neurons only

Start with one sample, e.g. Br6522_LC_1_round1
Round 2 samples may be difficult due to issues in histology images

White matter: only interested in white matter in regions very close to LC, not main background region


https://github.com/hakimel/reveal.js/blob/0dbe29b53eb9ecf38e0b5247ca81748481f48bba/.github/FUNDING.yml#L1

Perceived Problem
Ideas / Proposed Solution(s)

Perceived Problem
Ideas / Proposed Solution(s)

https://wmlink.wal-mart.com/o365login


Workload identity federation support

I was wondering if there is a plan to add Workload identity federation support to the .net client libraries?
According to the documentation it is not supported yet and I also tried to point the GOOGLE_APPLICATION_CREDENTIALS to a credential configuration file but then it throws an error
 Error creating credential from JSON or JSON parameters. Unrecognized credential type external_account.

It's supported in the recently-released beta of Google.Apis.Auth. See this issue for more details and links to the PRs involved. (Note that for auth queries, that's the right repo to raise issues on, as that's where the code for Google.Apis.Auth lives.)


High Memory  Consumption

Hi @Atem18,
opened ne issue because not definitely sure if the same root cause.
Could you provide how you are profiling and the minimal code for reproduction?
My current setup is guppy3 and the following code (readme, public api only):
https://pastebin.com/pdu5Kxzm
Results after about a 20 minutes run:
first:
Partition of a set of 761710 objects. Total size = 76471286 bytes.
 Index  Count   %     Size   % Cumulative  % Kind (class / dict of class)
     0 167518  22 17767591  23  17767591  23 str
     1 146885  19  9673200  13  27440791  36 tuple
     2  31820   4  4852848   6  32293639  42 list
     3  26729   4  4765545   6  37059184  48 types.CodeType
     4  13259   2  4272904   6  41332088  54 dict (no owner)
     5 141435  19  3985788   5  45317876  59 int
     6  52951   7  3940826   5  49258702  64 bytes
     7   3597   0  3588208   5  52846910  69 type
     8  24096   3  3277056   4  56123966  73 function
     9  27418   4  3070816   4  59194782  77 asttokens.util.Token
<1349 more rows. Type e.g. '_.more' to view.>

after 20 minutes:
Partition of a set of 761735 objects. Total size = 76488074 bytes.
 Index  Count   %     Size   % Cumulative  % Kind (class / dict of class)
     0 167527  22 17768268  23  17768268  23 str
     1 146885  19  9673200  13  27441468  36 tuple
     2  31822   4  4853136   6  32294604  42 list
     3  26729   4  4780648   6  37075252  48 types.CodeType
     4  13257   2  4272608   6  41347860  54 dict (no owner)
     5 141438  19  3985876   5  45333736  59 int
     6  52951   7  3940826   5  49274562  64 bytes
     7   3597   0  3588208   5  52862770  69 type
     8  24096   3  3277056   4  56139826  73 function
     9  27418   4  3070816   4  59210642  77 asttokens.util.Token
<1349 more rows. Type e.g. '_.more' to view.>

So I wasn't able to reproduce the memory issue.

Dear @Kiessar, the minimal code is this one : https://github.com/Atem18/kraky#websocket
The memory seems to jump really fast over 5 minutes without any reasons.
The best would be if I could put an APM to trace the whole process.

Dear @Atem18,
do you use the private API in your test?
I don't use it  (public only) and can't reproduce the error.

@Atem18 fresh environment + private api use, no rising memory consumption after 1 hour runtime.
Which OS do you run on?
Would you mind sharing your dependencies including versions?
The only things I changed was removing, the prints for the "results" and adding memory consumption print outs.

@Kiessar, Ubuntu with only kraky as deps.
And the memory was increasing suddently after multiples hours.

@Atem18 could you try to add like 10 other pairs to the example and check if the memory consumption happens faster then?

@Kiessar Ok, and I will try to use memray from Bloomberg

@Kiessar Do you have any issue with memory ? If not, I propose to close this.


Typo

You can also create a pull request for changing the Batchnorm layer's axis parameter to -1. As the code is using tensorflow the axis parameter should be set to -1, which I haven't corrected yet.


Remove Iter8 extension

Related to https://github.com/kiali/kiali/issues/4643
https://github.com/kiali/kiali-ui/pull/2284
https://github.com/kiali/kiali/pull/4653

Built and sanity checked with operator, server and ui prs.

Thanks. I have also some concerns about backward compatibility, too. Well, not fully worried as this extension is experimental but I haven't removed the data from the previous versions to "make it compatible". But some look to check if that's the right way would be good.
I think this PR can wait until we have a proper test, if not shipped in 1.45, it can get in for the next one, as the last thing I'd like to do is to introduce a regression in the operator.

I have tested with Operator PR, it looks good to me. Nothing broken

I think the changes won't change the feedback from @pbajjuri20.
So, I think we can consolidate and fix it later if there is any new issue.


Size tap targets appropriately

Issue

Reproduction
Run lighthouse on docus.dev.
Just need to make their paddings a bit bigger and maybe the icon size up a tiny bit

For mobile I guess it's fine, I don't see that very bad honestly

Ye only needed for mobile, it's something Google penalises for so probably worth fixing


Dug interface

This PR adds a Dug class to serve as the main Python entrypoint to Dug. It also significantly expands the CLI both for readability and to use the Dug class.

Would echo Alex's comments that this needs to be tested with the UI integrated.

With regards to the API, the API only consumes the Search class directly. This PR doesn't make any changes to that class so

@alexwaldrop @jcheadle-rti I added the wrapper for the search-kg and search-nboost.


Remove [Cast] Attribute

The Cast Attribute is no longer required. The parser should properly handle all C# casting scenarios.

The Cast Attribute has been removed the Attribute Reference knowledge base article.
http://bridge.net/kb/attribute-reference/


Edit and Remove Assessments (Frontend)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 61.25%. Comparing base (004c016) to head (73c5513).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##             master      #93   +/-   ##
=========================================
  Coverage     61.25%   61.25%
  Complexity      534      534
=========================================
  Files            65       65
  Lines          1879     1879
  Branches         73       73
=========================================
  Hits           1151     1151
  Misses          696      696
  Partials         32       32

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

Most of the issues were not addressed


__Nonexhaustive in State

While browsing the source code I found this:
https://docs.rs/battery/0.7.5/src/battery/types/state.rs.html#10-20
And more precisely this:
    // Awaiting for https://github.com/rust-lang/rust/issues/44109
    #[doc(hidden)]
    __Nonexhaustive,

It seems that the referenced issue has been closed. Is the __Nonexhaustive field still needed?

#[non_exhaustive] attribute appeared in Rust 1.40.0 first, while this crate support Rust 1.36.0+ right now, therefore it is impossible to use it yet.


Bridge from Ketla to Holesky

Describe the bug
Description of the bug here.
Bridging from Hekla to Holesky keep failing but from Holesky to Hekla is very successful
Steps to reproduce
Steps to reproduce here.
Just connect your wallet to bridge, switch network from Holesky to Hekla, them try to bridge

Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

Has been fixed,try again


Ignore gain maps with unsupported version.

Allow extra bytes after the gain map metadata of writer_version is larger than the supported version.
#260

Also, note that we generally don't merge CrabbyAvif PRs from github, while we can do the review process here if you prefer that, it is simpler if the final merge happens internally in google3 and it will be sync'ed by copybara automatically to the github repository. :)

Thanks for the review, and sorry I didn't realize the source of truth was in google3 x)
Since we discussed some changes to the API in https://github.com/AOMediaCodec/libavif/issues/2369#issuecomment-2284601520 I will first update libavif then will resend this internally.

Thanks for the review, and sorry I didn't realize the source of truth was in google3 x) Since we discussed some changes to the API in AOMediaCodec/libavif#2369 (comment) I will first update libavif then will resend this internally.

sure, sounds good.


[WIP] copied work

Description of change
This is some thoughst I had on the structure that did make it into the previous PR
Link to relevant ticket
NA

was a POC only so closing - will; incorporate parts into future PR's


Investigate options for running theme code in a sandbox

A prerequisite for moving to stage three is we need some way to accept untrusted theme code. Cloudflare's recently announced workers for platforms looks perfect, however it's still in closed beta and I doubt they would give us access. We're not in a huge rush to get to stage three yet, so maybe we just wait until that's available for everyone. (How long does Cloudflare usually take to move features from closed to open beta?) If anyone is interested, it wouldn't hurt to explore other options though. I spent a day looking into this and my conclusion was that besides the Cloudflare thing, AWS Lambda would probably be the way to go (in which case it would be especially great for someone else to take this issue because I have little experience with AWS and prefer to keep it that way). I'm open to other things too.
Whatever we use for the sandbox, we need to be able to zip up the theme folder (e.g. themes/default/), send it to the sandbox, and have it unzip, generate site files, and return them in another zip file. or something like that. And ideally have it be fairly snappy (under 10 seconds is probably acceptable, though the faster the better). If we can let the theme code specify dependencies (such as npm dependencies) and have the sandbox install + cache the dependencies that would be swell. If not, it's probably good enough to start out with if we just have a fixed list of dependencies (like tailwind etc).

It occurred to me that since we're already using netlify, and netlify's backend fns are just wrappers for AWS lambda fns, we could try to use those. For example, platypub could create a backend fn for each (user, theme) combination. e.g. if you deploy a site that's using the default theme, platypub checks to see if you already have a backend fn for the default theme and creates it if not. The backend fn accepts the input.edn file via a POST request and returns a tarball/zip file of the outputed static files, which can then be deployed to netlify by platypub.
It would be great if someone wanted to play around with this. e.g. make a backend fn as described above, rewrite the default theme with nbb (this might be a lot of work?), and see how fast it goes.
Need to make sure that backend fns are sufficiently isolated. i.e. are two backend fns made with the same Netlify account guaranteed to be unable to access each other's data?

I think this is the answer: https://fly.io/blog/fly-machines/

Fly Machines are VMs with a fast REST API that can boot instances in about 300ms.

We'll need to figure out how to cache npm/deps.edn deps. doing it in a dockerfile might be ok--are there security risks with putting untrusted code in dockerfiles for fly machines?

Some reading material:

https://disaev.me/p/sandboxing-untrusted-code/
https://github.com/glotcode/docker-run
https://github.com/rapidlua/luajit.me/wiki/On-running-untrusted-code-in-AWS-Lambda


Missing my monyx coin

Dear sir
I have 1,779.37325220 MYX coin but its not showing in my wallet what happened with me ??
User name - rathorepritish
Email id -<EMAIL_ADDRESS>

@rathorepritish you was one of the users whose balance was affected by the bug, so you got 0.08168995 BTC than needed.
Since you didn't have any BTC on your account, we deducted the equivalent amount of MYX.

No sir i have not got any btc ,, some hacked my account 7feb after i was complained reports here after i have returned btc again and purchase monyx  again

7 feb i have also deposit 0.00719870 BTC in my wallet and purchase some monyx coin its is also dedication from my account

This is unfair with me i have not received any btc 0.08168995  and l also complained regarding to this on 7 feb you can checked it here

@rathorepritish ok, give me some time to investigate it closely.

This unfair with me
On Tue, 13 Feb 2018 at 12:10 AM, monyx.io<EMAIL_ADDRESS>wrote:

@rathorepritish https://github.com/rathorepritish ok, give me some time
to investigate it closely.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/monyxcoin/monyx-coin/issues/139#issuecomment-365020305,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AijkT-6RDfYth8MtUDTGWxbTH731qh4Zks5tUIWhgaJpZM4SCmBh
.

@rathorepritish should be restored to the point where bug affected the balance.

Thank you sir
Give me life again i was died for some time
On Tue, 13 Feb 2018 at 12:17 AM, monyx.io<EMAIL_ADDRESS>wrote:

@rathorepritish https://github.com/rathorepritish should be restored to
the point where bug affected the balance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/monyxcoin/monyx-coin/issues/139#issuecomment-365022637,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AijkT45WaxABTwSet9pdd8r4JmrmCnSTks5tUIdTgaJpZM4SCmBh
.


update forgotPasswordPage

Logic for sending reset password link (to be implemented)
For now, I assumed a simple response

sending reset password link to the email is done @BinaryQuBit