Add content cache required changes to distribution

Rebased version of #2633

Piping the storage driver through the registry middleware enables using the driver to perform out of band storage operations outside of the normal operation of the registry. One example of this is a TTL scheduler that cleans up blobs after specific TTL times have passed, which can then be used as a downstream registry cache that pulls from upstream for blobs that don't exist locally.
Note, this represents interface breakage so I want to assess the damage done here and what can be done to reduce the scope of damage.

Signed-off-by: David Wu<EMAIL_ADDRESS>

Hello from the future @davidswu. Would you mind rebasing?

Hey @milosgajdos - done!

I'm going to close and reopen this PR because there is no other way to trigger the reworked GHA workflows.

@milosgajdos friendly ping since it seems all tests have passed since the PR has been closed and reopened.

@milosgajdos What is left before this PR can be merged? Looks like it has 2 approvals already.

I've OK-ed it. It seems we have some weird DCO issue here, but we need one more approval. I've tagged @thaJeztah and @squizzi. Can't really do more.

I've closed and reopened the PR hoping the DCO check will sort itself out but it hasnt 🫠

@davidswu can you squash the commits and sign -- wondering if the double email signage in the first commit confuses DCO

@milosgajdos done

@corhere would you mind giving this a look? Seems like other folks are busy 😇

maybe @deleteriousEffect can also give this a review.


SSL Certificate expired

The SSL certificate for backend.matic.supply expired six days ago, which is causing browsers to drop requests.

Sorry, I was out for summer, fixed


Windows tests fix from #71

Directly load schema file to use the NewStringLoader instead of the NewReferenceLoader to fix the schema loading on Windows.
I tried to figure if I could find a Windows golang image to potentially add a step to the CircleCI script but couldn't find any.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f64e409). Click here to learn what that means.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##             master      #73   +/-   ##
=========================================
  Coverage          ?   91.32%
=========================================
  Files             ?        8
  Lines             ?      242
  Branches          ?        0
=========================================
  Hits              ?      221
  Misses            ?       15
  Partials          ?        6

Impacted Files
Coverage Δ

internal/server/http/route_matchers.go
88.88% <66.66%> (ø)

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 f64e409...25c5ad8. Read the comment docs.

thanks for fix this problem on windows :D


Cannot send notifications on the boot method of a service provider

Package
filament/notifications
Package Version
2.16.36
Laravel Version
9.36.4
Livewire Version
2.10.7
PHP Version
8.1.11
Problem description
When sending a notification on a boot method in a service provider, the notification is not always displayed for the user. I have confirmed that the notification is in fact pushed to the filament.notifications session array.
One workaround I've found is if I wrap sending the notification with the Filament::serving callback, however this only works when I have the additional admin package installed.
When I go directly to the page it'll display but if it's just a refresh then the notification will not be displayed.
Expected behavior
The notification to always be displayed to the user on the request that triggered the notification.
Steps to reproduce
Add the following code to your boot method inside app/providers/AppServiceProvider.php:
$someCondition = false;
if (!$someCondition) {
    //Filament::serving(static function () {
        Notification::make()
            ->danger()
            ->title('Error title')
            ->body('Error message')
            ->send();
    //});
}

Run php artisan serve (or use a web-server) and go to the login page (such as http://localhost:8000/login). The notification will be displayed, refresh, and the notification will not be displayed. Uncomment the Filament::serving callback and the notification is always displayed.
Reproduction repository
https://github.com/realslimsutton/filament-notifications-test
Relevant log output
No response
EDIT: Updated to include details that the notification does in fact inconsistently get displayed, but wrapping sending the notification with  the Filament::serving callback, makes the notification consistently displayed to the user.

Would this not be better executed in middleware? It just sounds like a system level thing that is not dependent on filament, as I understand you're explanation. Otherwise it works in the context of filament admin. Boot in service provider is not the right place to execute such logic.

Middleware would be an ideal place but in this instance it's a modular system to separate the code, which boots various service providers. Loading the service providers in the middleware would be too late in the request lifecycle.
In the scenario that an error occurs during the boot of a service provider (or module) it'd send a notification. Which is fine as long as they're on the admin panel but if the notification is sent on any page that isn't the admin panel, then it won't be displayed.

So you need to display a notification without filament, because there is no guarantee, AFAIK, in Laravel to guarantee boot order. So, should the notification be handled at run time or pushed to something like the db notifications? I'm just not sure this is an issue with Filament. I could be wrong though.

From what I could tell from the code, sending a notification just pushes the notification into the session. Then they're loaded when a notification livewire component is loaded, so loading order shouldn't be an issue here imo. I wasn't sure if this was an actual issue or intentional or not when I made the issue but I figure worst case someone could clarify for me.
DB notifications could be a solution for me, but I was just surprised it works inconsistently rather than not at all (and looking from at the code it appears it should at least work).

When I go directly to the page it'll display but if it's just a refresh then the notification will not be displayed.

@realslimsutton could you clarify this?

DB notifications could be a solution for me, but I was just surprised it works inconsistently rather than not at all (and looking from at the code it appears it should at least work).

Yeah, like awcodes said, I don't think you can rely on the service providers to boot in a certain order. Without knowing for sure, I would guess that the order is random since you are getting unpredictable results. Some Filament provider required by the notification service doesn't always get booted before yours.
Like it's also been said, I don't think a service provider is what you want for this, commonly things like these are put in middlewares. Though from your use case regarding error handling maybe you'd want to use to use email/database notifications instead.
If you are sure using a service provider is what you really want, you could also try registering a callback like this in your register method of a service provider:
$this->app->booted(function () {
    // Do something after the application has booted
});

Thanks for commenting, @caendesilva. I wanted to comment something similar, but was confused since @realslimsutton stated the notification was always added to the session (even when not visible on the page). If that's the case, it must be related to something like a page reload / redirect resulting in the notification being removed from the session.
Again, I'm quite sure this is not an issue with Filament.
Feel free to comment if you've found any evidence that it actually is a Filament issue, @realslimsutton.


Allow multiple widgets of same type

Thank you for contributing to Homarr! So that your Pull Request can be handled effectively, please populate the following fields (delete sections that are not applicable)
Category

Feature

Overview
Added the possibility to support multiple widgets of the same type
Issue Number
Related issue: #728
New Vars
I added the new property type to the IWidget which is migrated from the previous id parameter automatically after opening a dashboard for the first time. The id property then gets a random uuid assigned.
Screenshot

Nice! Will review tomorrow

What is the edit mode password for the dev environments?

Really important changes! Nice, @Meierschlumpf 🙌

What is the edit mode password for the dev environments?

I think it’s the one in the Vercel config file, probably “edit” but we could remove that file to delete it. I don’t think I should’ve merged it

Also @manuel-rw dont forget to accept a PR after the changes are made or @Meierschlumpf dismiss the review when the changes are made so we don’t see “changes requested”

@Meierschlumpf can you rebase and merge?

copilot:all


RadioButtons labels are now clickable

Fixes #1535
Changes:
These Labels are now clickable

Choose Session Style

Single
Expandable

Choose your data source

Upload your own JSON files
API endpoint of event on OpenEvent

Screenshots for the change:

Closing this PR as this issue has been resolved please take up some other issue. :)


Enable eslint-jsdoc

Description
This PR enables eslint-plugin-jsdoc. I figured we'd go ahead and enable the defaults, and we'll turn off rules for now that will require a lot of manual fixes. We can systematically enable rules and fix them one-by-one where time allows or need arises. For example, type checking may be particularly helpful.

Uses the recommended ruleset. In the future, we may want to turn on TS mode, but for now we'll default to the recommend for JS-only.
Disable rules that are against our coding consistencies, at least for the time being and added comments why
Disabled quite a few rules which require manual fixes, like filling in types and descriptions, until we can manually fix.
Autofix with npm run eslint -- --fix

Issue number and link
Fixes https://github.com/CesiumGS/cesium/issues/10815
Testing plan

Run eslint and ensure there are no failures
Run build-docs and ensure the built documentation is not broken
Run build-ts and ensure there are no errors (there will still be warnings)

Author checklist

[x] I have submitted a Contributor License Agreement
[x] I have added my name to CONTRIBUTORS.md
[ ] ~I have updated CHANGES.md with a short summary of my change~
[ ] ~I have added or updated unit tests to ensure consistent code coverage~
[x] I have update the inline documentation, and included code examples where relevant
[x] I have performed a self-review of my code

If I could summarize the changes I'm seeing

rename properties that are synonyms to the "true" one, this is good imo :+1:

constructor to class
return to returns
exception to throws

remove empty lines as outlined in my comment above :neutral_face:

I agree with some of these that were excessive (double lines, ones between some tags)
I disagree with many of them that created valuable visual separation between chunks of tags and between the description and all tags. again, mentioned above

adding @params for those that didn't exist before, again see above :shrug:

I've reviewed about 100 of the 800+ files and these seem to be the only changes. I'm going to stop here for now in case we change how the rules above are handled so I don't waste time reviewing files that may not actually have any changes.


Added Specimens Section and Improved Performance

This PR adds the Specimens Section, including adding new components required to do so.
As part of this work, I realized we were doing way too many re-renders (@radamson also noticed this). The reason for this was because anytime a Formik value was updated, any component that used the useFormikContext hook would trigger a render, so basically every component and every child component in the form would re-render after every input. :weary:
To fix this, I refactored each component to only pass in the values that it actually needs to worry about, then wrapped it in memo so that the component would only re-render if its props changed. The performance still isn't perfect, but it's better in production mode than development. We can spend more time in the future with this, but this should work for now.

Excited to review this.  I was suspecting it was based on the broad use of the Formik Context (I noticed that even changing the first text box in the form was causing my forms way further down to rerender!), but wasn't sure how to best fix it.
There's a great post on this on the react issues page on this type of stuff: https://github.com/facebook/react/issues/15156#issuecomment-474590693

@rdingwell I interpreted the  "Additional State/local Specimen IDs" section at the very bottom of the form to give extra space in case there was more than one specimen taken of the same type, which there is not room for currently on the paper form. Does the fact that we can now add unlimited numbers of specimens, each with their own specimen ID and type, cover this?


Microsoft.Logic - Don't make ResourceReference id readonly

Don't make ResourceReference id readonly, it's causing a bug in the SDK where an Logic Apps with integration accounts can't be saved
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

Automation for azure-sdk-for-java


🐞 Remember window size and position

Description
CodeEdit does not remember the last window size and position.
To Reproduce

Open CodeEdit
I had previously opened a folder, so it opens to that folder.
On my MacBook Pro the window is neither centered nor maximized.
Maximize it.
Quit CodeEdit and open it again.
Note that the window reverts back to what appears to be a hard-coded initial X, Y location and size.

Expected behavior
Remember the user's preferred window size and position.
Version information
CodeEdit: 1.0
macOS: 13.0.1
Xcode: not installed
Additional context
No response

@austincondiff I have a fix for this. Can I work on it?

Oh for sure. Thanks for letting me know. Assigning to you


Massive refactor

change of directory structure
cleanup public api

Current coverage is 31.38% (diff: 31.89%)

Merging #24 into master will decrease coverage by 13.98%

@@             master        #24   diff @@
==========================================
  Files            43         47     +4
  Lines          2160       2619   +459
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            980        822   -158
- Misses         1064       1703   +639
+ Partials        116         94    -22

Powered by Codecov. Last update f6a6890...1ea9352


Upload bugs

Hi, thank you for this very cool project.
However, I could not make it work correctly. Multiple issue encountered:

GUI interface does not accept upload, got a undefined and GUI stuck at "Processing file"

The upload was a PDF file but seems like the behavior changes between the file type. And in the S3 buket, we endup with a bunch of this:

Using an API testing client, it behaves correctly:

There is also a credential conflict, I could not make it work with

		config.WithEndpointResolverWithOptions(r2Resolver),
		config.WithCredentialsProvider(credentials.NewStaticCredentialsProvider(
			os.Getenv("S3_ACCESS_KEY"),
			os.Getenv("S3_SECRET_KEY"),
			"")),

Changed to the default config and since we are taking the credentials from the environment variables, the SDK will automatically use AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY and added AWS_REGION
cfg, err := config.LoadDefaultConfig(context.TODO())

Here is my env file:
AWS_ACCESS_KEY_ID=
AWS_SECRET_ACCESS_KEY=
AWS_REGION=
S3_BUCKET_NAME=buk-test-stg
BIND_ADDR="<IP_ADDRESS>:4040"

I'm sorry, the frontend folder was missing the .env files, I push a quick-fix
the reason is because the import.meta.env.VITE_BASE_URL part are detected as undefined, It should be just an empty string, but I may change the code later
https://github.com/codenoid/temporar/blob/29845aa49a4bf3ea04c5e856a0297cbccb884ec8/frontend/src/routes/%2Bpage.svelte#L53

No problem, I figured out it was more a frontend issue but being not familiar with Svelt, I did not want to dive in.
As for the credential conflict, I do think, it would be easier to use the AWS default env variables, what do you think?


error with next-compose-plugins

i need to use next-translate with other plugins and I decided to use next-componse-plugins for it. (if there are others way, happy to hear that)
When I add next-translate to the list of plugins, I get error.
i also try to use it with just next-translate and I get lots of errors.
That's my next.config.js
const withPlugins = require('next-compose-plugins')
const nextTranslate = require('next-translate')

module.exports = withPlugins([nextTranslate])

Errors I get:
ready - started server on http://localhost:3000
info  - Using external babel configuration from /.../babel.config.json
Error from chokidar (/home/gio/repos/elliotforwater_v4/context): Error: EMFILE: too many open files, watch '/.../context'pos/.../.env.development'e/.../helpers'pos/.../.env.production'.../hooks'
Error from chokidar (/...): Error: EMFILE: too many open files, watch '/...'
....

p.s. I replaced my home path with ...
Any tips why I get this?
I found other issues here with next-composes-plugins where people says it works good, i replicated the same situation but I got the same error. :/

my bad, is probably coming from other issues


Add apng to FileExtensionContentTypeProvider.

Summary of the changes (Less than 80 chars)

Add apng to FileExtensionContentTypeProvider.

Addresses #bugnumber (in this specific format)
#7101
Reference
APNG Specification#MIME type

Closing based on this conversation- https://github.com/aspnet/AspNetCore/issues/7101


About the training data

<EMAIL_ADDRESS>

Hello,
The pretrained models are having good acuracy.
Can we get the training steps and training data?

Hi, everyone, the training data will available sooner or later for more data are added to the dataset.

Hi there, @wujixiu !
Really nice code and trained networks. For medium to large objects, it works perfectly!
I would like to know, if possible, more details about your training data. This would help me to build my own dataset. I would like to know things like:

How many images does your training set have?
How did you annotate your images? By hand or with some sort of pre-detection?
How much is your positive/negative samples ratio?

Really nice work, congrats!

Hi there, @wujixiu !
Really nice code and trained networks. For medium to large objects, it works perfectly!
I would like to know, if possible, more details about your training data. This would help me to build my own dataset. I would like to know things like:

How many images does your training set have?
How did you annotate your images? By hand or with some sort of pre-detection?
How much is your positive/negative samples ratio?

Really nice work, congrats!

Hi, @amenegola

Our dataset provides 3,174 images containing 18,893 hardhats instances (including non-hardhats regions).

All images are labelled mannually with labelImg.

Same pos/neg ratio in SSD.
Training Data will be released at the end of this month.

Hello @wujixiu!
Awesome, thanks for this reply, you are very helpful.
Congrats again on the model!
Cheers,
Afonso

I'm looking forward to the training data, too. Thanks for your great job.

非常感谢你所做的贡献，不知道博主是不是可以公开一下数据集，同时我也有些问题想要请教一下博主，我的QQ是1018949282，同时再次非常感谢博主！


raylib data types not properly freed

The destructors should also re-init the data after calling the unload function. For example:
~this()
{
    image.UnloadImage;
    image = image.init;
}

It seems I actually don't need to do it, so closing the issue...


Build C++ docs

Build the docs for the C++ code.
Note that this PR does not include content and/or content fixes, this is just the infrastructure.

Ok, now I have no idea what's going wrong with the manylinux builds, they say a file is missing. I have sshed in and indeed there is no adjvectorbqm.pyx, though it is there on the branch.


Adding plugin complains of older cordova version even when latest cordova version is installed

Trying to install the plugin as mentioned in docs leads to the following error:
$ cordova plugin add phonegap-plugin-push
Fetching plugin<EMAIL_ADDRESS>via npm
npm http GET https://registry.npmjs.org/phonegap-plugin-push
npm http 304 https://registry.npmjs.org/phonegap-plugin-push
Plugin "phonegap-plugin-push" already installed on android.
Installing "phonegap-plugin-push" for browser
Failed to install 'phonegap-plugin-push':CordovaError: Plugin doesn't support this project's cordova version. cordova: 3.5.2, failed version requirement: >=3.6.3
at checkEngines (/usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/install.js:116:29)
at /usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/install.js:289:16
at _fulfilled (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:787:54)
at self.promiseDispatch.done (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:816:30)
at Promise.promise.promiseDispatch (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:749:13)
at /usr/local/lib/node_modules/cordova/node_modules/q/q.js:557:44
at flush (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:108:17)
at doNTCallback0 (node.js:419:9)
at process._tickCallback (node.js:348:13)
Plugin doesn't support this project's cordova version. cordova: 3.5.2, failed version requirement: >=3
While the cordova version that I have is 5.0.0.
$cordova --version
5.0.0
I am not sure why is it complaining about older cordova version.

What version of Cordova Android do you have installed?
Run "cordova platforms"

cordova platforms
Installed platforms: android 4.0.0, browser 3.5.2, ios 3.8.0
Available platforms: amazon-fireos, blackberry10, firefoxos, ubuntu
Although browser platform is unused / not needed in my case.

@divkis01 right, and the engine tag in the plugin says the minimum version of cordova on any platform be 3.6.3 so the fact you have 3.5.2 version of browser installed it fails, which is expected behaviour.
You'll either need to upgrade or remove the browser version. Actually for iOS, you'll want to upgrade to 4.0.1 as well.


Add groupSortOn and groupSortBy

The obvious functions:
groupSortBy :: (a -> a -> Ordering) -> [a] -> [[a]]
groupSortOn :: Ord b => (a -> b) -> [a] -> [[a]]

Added.


GroovyFX Scripting Window

This is a feature request.
I'm looking for a reusable, embeddable Groovy console window for scripting in my JavaFX app. What is the best starting point in all of Groovydom? Does it make sense to include something like that in this project?

I think it would be best to start another project, as an embeddable console requires additional dependencies such as a rich text editor (https://github.com/TomasMikula/RichTextFX), undo capabilities (https://github.com/TomasMikula/UndoFX), key bindings (https://github.com/TomasMikula/WellBehavedFX), and maybe even promises (https://github.com/jdeferred/jdeferred) to handle correctly  threading.

Thanks @aalmiray for all the links and advice. I agree that it makes sense that a scripting window component be in a separate JAR because of the dependencies, it doesn't necessarily have to be in a separate repo, though.
I'm new to JavaFX (though I've been using Groovy for 10 years and Java longer than that) and it's been very frustrating searching around for what libraries to use in my project. I've been writing mostly in Java 8 because I wanted to start with the basics, but I want to start using some Groovy, too. I really want the GroovyFX property and binding support, for example. The other decidedly Groovy feature I want in my app is scripting and a DSL. I'm not that interested in the builder stuff, by the way -- I prefer to use SceneBuilder (and FXML)
As you know JavaFX is more of library than a framework and is un-opinionated about the structure of your app. So I'm looking around at various projects that I can use. I'm already using NSMenuFX because if I can't get proper Mac OS menus then JavaFX isn't a viable solution (I want to submit to Mac App Store, etc) and I'm looking at Adam Bien's Afterburner.FX because it looks like a lightweight solution to provide dependency injection and a little more structure, etc. Griffon looks really cool, but is not the correct fit for the type of App I'm building. TornadoFX from Kotlin looks interesting (another Kotlin temptation) but I want to stick with Groovy, I think.
I'm looking for one-stop shopping, I suppose. A place where I can get all the basics for a cross-platform app without choosing a heavyweight framework either. Clearly some of these pieces will be (non-Groovy) Java libraries, but since I want some Groovy in the app -- and the Groovy community is a great place that I feel comfortable in -- I'm looking to the Groovy community for best-practices.
So I guess what I'm saying is that it would be nice if GroovyFX were to provide add some optional, component modules. A scripting console is a big project (too big for me to start from scratch), but seems like a key component that highlights the advantages of Groovy. Maybe it could even replace the current (Swing-based, right?) Groovy Console someday. (Maybe I should just try to embed the current Swing console in my FX app?
I'd also like to see a great sample app that includes some lightweight pure Java libraries -- maybe Afterburner.FX (or equivalent) and NSMenuFX (or something else, if available) that can create correct Mac OS menus.
I think of the difference between Ratpack and Grails -- both of which are great projects and have their place -- and I'm looking for more of the Ratpack of FX. It seems like GroovyFX could become such a solution.
Sorry if this is a little rambling and off-topic, but I'm hoping some of the contributors here would appreciate a newbie's perspective. I'm willing to create additional issues here (or in another project) to pass this info along. Of course, I also just asked about the hard question about the Gradle FX plugin so maybe you guys are going to just ban me from the project haha.

Trying to post a long, rambling response. Is Github having issues or is it just too damn long?

I suspect that maybe you want Griffon?  http://griffon-framework.org/

Griffon looks a little too heavyweight for my application, but I'll take a second look.

We welcome all feedback and criticism on Griffon. The design of the framework is supposed to be modular, in order to avoid forcing a single path on the developer's mind. The smallest application possible (for the time being) weights ~4M due to Guice/Guava and can grow up to ~12M due to Groovy/GroovyFX. In terms of features you can override almost any portion of the framework. I'm really interested to know more about the requirements that would make Griffon be considered inadequate for this particular project. that's how we learn and make it better 😃

Hi @aalmiray I just spend some time looking at Griffon and also Basilisk. I have several applications in mind and I want reusable components that can be used in multiple applications.  The GroovyFX console window would be desktop only, but other components would be used on mobile apps as well. I'm leaning away from using JavaFX on mobile and instead using native and Android and iOS UI.
I would love to continue this conversation with you, but maybe on this particular issue thread? Where would be the best place to have that conversation?

If we were to make a JavaFX-based replacement for the current Groovy Swing Console, would you use Griffon?
I've also tried embedding the Swing Console in a JavaFX and the preliminary results aren't good. (Menus get all messed up on Mac OS) I'm wondering if there's a way to embed it using the SwingNode class from JavaFX.

I'd suggest starting a conversation at the Griffon mailing list (http://griffon-user.74797.x6.nabble.com/) would be best. Targeting both desktop and mobile (iOS & Android) using the same codebase is only possible if you choose JavaFX and the jfxmobile plugin form Gluon; this is exactly what Basilisk offers, plus all the goodies from Griffon.
Embedding Swing on a JavaFX produces mixed results. As you saw, menus are problematic, not to mention you must ensure both UI toolkits use the same UI thread, otherwise other problems will arise. In short, mixing Swing and JavaFx results in a bad development experience, let's not dwell into the user experience 😉

Hey @aalmiray I see you're working on this: https://twitter.com/aalmiray/status/1006996345243291649
That's awesome!
Is the work in a Github repo?

It's available at https://github.com/aalmiray/groovy-console-fx

Closing this issue. If you're looking for a GroovyFX Console window, please follow: https://github.com/aalmiray/groovy-console-fx
(Note: It would be nice if that project could make a re-usable component for the scripting window itself.)


Error in API for create new node

Hello, i have a error when try to create a new node using the api.
i set the request.
POST https://PANELURL/api/admin/nodes Authorization: Bearer SECRET Content-Type: application/x-www-form-urlencoded name=TEST&location_id=1&public=1&fqdn=test.go.cl&behind_proxy=0&scheme=http&memory=10&memory_overallocate=100&disk=100&disk_overallocate=110&daemonBase=/home/servers&daemonListen=9090&daemonSFTP=5550
but return
{"error":{"behind_proxy":["The behind proxy field is required."]}}
and this parameter has been set in the body and the Authorization.
PD: The docs for api not mention any for the parameter. https://pterodactyl.readme.io/v0.6.0/reference#create-node

The API uses JSON, not x-www-form-urlencoded.

Test with json and same error.

Can confirm this error, my output is exactly the same when testing the API to create a node. The worst part about this is that this field isn't even mentioned, let alone documented, in the apidocs.

This will be fixed as part of 0.7's new API.


opencl images faster transfers to java/gpu?

Question
there is quite significant lag transferring and converting, painting an opencl gpu image to java BufferedImage painted by java gpu. goal is not to use opengl interop with opencl, even if it would directly solve the issue but would also involve using opengl.
so I look for a way to get opencl gpu image to java default environment faster than 10ms at 720p, 20ms at 1080p and 100ms at 4K. the transfer phase uses pure integer buffers and conversion uses BufferedImage custom color/sample models. painting from the cpu-only image is slow, as it needs to be uploaded to the java gpu and then painted or painted by the cpu if no transfer happens automatically in java. opencl gpu to java transfer times are 1/4 of the java cpu to gpu paint times (2.5ms for 720p, 5ms for 1080p, 25ms for 4K transfer times).
yes trying to use opencl as headless render device.
any ideas or possibly optimizations/additions to the lwjgl opencl api interface functions?

Hey @goofyseeker311,
Not really, the bindings are as low-level as they can be. There are no additional copies or other overhead introduced by LWJGL that would limit transfer performance.
However, there are plenty of ways to misuse the bindings and AWT/BufferedImage are notoriously difficult/impossible to use effectively in performance-sensitive applications. You won't get much help from generic descriptions like above, so please post some code. Maybe someone will be able to spot inefficiencies that are simple to fix.
Btw, github issues are not the best place to ask such questions or seek advice. You may want to join the LWJGL discord server, there are plenty of people there with relevant experience and you'll be able to discuss specific issues in more detail.

Imho think github is the better place. as also my code is here.
below code creates usually accelerated BufferedImage images, so as long as the image is at some point converted to accelerated form then it can be painted by gpu directly fast as gpu buffered image. so only the conversion to the java gpu form take time. also  thinking that the image conversion time is only 3x of the transfer time means that its also as fast as possible right now. also gpu compute time is neglible below 0.1ms always.
BufferedImage GraphicsConfiguration.createCompatibleImage(graphicswidth, graphicsheight, Transparency.TRANSLUCENT);

currently on the java BufferedImage side I'm doing it like below code.
private class RenderThread extends Thread {
private static boolean running = false;
public void run() {
	if ((!running)&&(program!=NULL)) {
		running = true;
		long frametimestart = System.nanoTime();
		computetime = computelib.runProgram(device, queue, program, "renderview", gfxbuffer, 0, graphicsbuffer.length, true);
		computetimeavg = computetimeavg*0.9f+computetime*0.1f;
		computelib.readBufferi(device, queue, gfxbuffer[0], graphicsbuffer);
		DataBufferInt readdatabuffer = new DataBufferInt(graphicsbuffer, graphicsbuffer.length);
		WritableRaster readraster = WritableRaster.createWritableRaster(readsamplemodel, readdatabuffer, null);
		BufferedImage readimage = new BufferedImage(readcolormodel, readraster, false, null);
		BufferedImage newimage = gc.createCompatibleImage(graphicswidth, graphicsheight, Transparency.TRANSLUCENT);
		Graphics2D newimage2d = newimage.createGraphics();
		newimage2d.setComposite(AlphaComposite.Src);
		newimage2d.drawImage(readimage, 0, 0, null);
		graphicsimage = newimage;
		long frametimeend = System.nanoTime();
		frametime = (frametimeend-frametimestart)/1000000.0f;
		frametimeavg = frametimeavg*0.9f+frametime*0.1f;
		running = false;
	}
}

readBufferi() function is like below code.
public void readBufferi(long device, long queue, long vmem, int[] v) {
        IntBuffer resultBuff = BufferUtils.createIntBuffer(v.length);
        CL12.clEnqueueReadBuffer(queue, vmem, true, 0, resultBuff, null, null);
        Arrays.fill(v, 0);
        CL12.clFinish(queue);
        resultBuff.rewind();
        resultBuff.get(0, v);
}

Some benchmark data: at 720p (2.5ms readBufferi() function time total usually):
IntBuffer creation: 0.6ms.
CL12.clEnqueueReadBuffer with clFinish(): 0.8ms.
resultBuff.get() stuff: 0.7ms.

This issue was solved by converting and rendering the opencl int[] image buffer with lwjgl opengl glfw window and glTexImage2D.


Involved transactions weren't displayed

This is a problem rooting from planet-node / pn-chrono side: the transfer functions make a transaction with an empty updatedAddresses field. Related: planetarium/planet-node#38 planetarium/pn-chrono#11 planetarium/libplanet#2347


infracost: 0.9.6 -> 0.9.8

Motivation for this change
Bump infracost to 0.9.8
Delete some new tests that require networking and some other testing fixes
@davegallant
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[X] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 140481 run on x86_64-linux 1

1 package built:

infracost


Issue#481 Remove 'experimental' from SSR page

@sarah11918 applied quick fixes.

Thank you very much, and welcome to Team Docs! 🥳


Reference to Get-MailUser and SoftDeleted

The SoftDeleted is not a documented parameter on Get-MailUser, the parameter that is documented is SoftDeletedMailUser.

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

ID: fc998569-646c-cd20-c3cd-b5d07e6f62ee
Version Independent ID: b7d7d129-9092-8e7d-5bc1-e49e24b54603
Content: Remove-MailUser (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Remove-MailUser.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@Ritty5, fixed.
Similarly, the correct parameter name on Get-Mailbox is SoftDeletedMailbox, and I found and fixed a couple of incorrect references for that as well.
Thanks for the feedback.
Closing


Get TCP to Beta

Applicable criteria for Beta (from Istio Schedule: Phase Definition tab):
Stability

Description

"Code is well tested. Enabling the feature is considered safe. Enabled by default."

Approach:

Create (or identify existing) e2e test that:

Runs on k8s (GKE?)
Uses a client and server that both communicate with some custom protocol over TCP
Verify that the expected attributes are sent to mixer

Performance

Description

"Performance and scalability are characterized, but may have performance caveats."

Approach

TBD: Need to identify the following:

Who will do this work? (@ldemailly any thoughts?)
What will we measure? With vs without mixer? Tcp rules vs http rules vs both?

Documentation

Description

"Complete feature documentation published to istio.io. In addition to the basic alpha-level documentation, beta documentation includes samples, tutorials, glossary entries, etc."

Approach

TBD: Need to identify completion criteria for documentation.  For example, is just enumerating the list of supported protocols on the homepage sufficient?

/CC @douglas-reid @smawson

for perf: https://github.com/istio/istio/issues/454

Expectation is that once the new integration test framework is in place we'll improve coverage here

@nmittler this should probably not have been closed, right? Not sure about timelines, but eventually TCP will get to beta I suppose.

@nmittler we might be able to help pushing TCP forward. The description here is a bit vague though. Is it really more a matter of testing (functional and performance) and documentation? We are working on an e2e that involves gateways, i.e. workload -> sidecar -> egress gw -> ingress gw -> sidecar -> workload, would that be a good start?


Write REPL history to file

The REPL input history is never written to the configure history file by haskeline.
The reason is that haskeline only writes the file if the InputT monad exits properly, that is without throwing an exceptions. At the moment the radicle binaries only exit on Ctrl-C and EOF. Both cases throw an exception.

https://github.com/judah/haskeline/pull/36
https://github.com/judah/haskeline/issues/5

I don't think that's the problem. The issue is that the executable doesn't write to the file if the file/dir don't exist.

Indeed it works if I do mkdir ~/.local/share/rad. Can we change the getHistory function to ensure that the directory exists?

:+1:


does this replace old files with compressed files

does this replace old files with compressed files or it generates new files altogether.

It replaces all files with the compressed version


Fix expected error in tcp_server_spec for musl

It seems only glibc errors at listen. This spec failed on musl because it errors at bind.
We could also consider removing these strict checks for very specific error messages. It would probably suffice to check that it either contains bind or listen with any libc implementation.
But I guess being a bit more strict avoids accidentally triggering unexpected errors which would then still meet the expectation.
And in the end, it doesn't really matter if such specs happen to occasionally fail on a new platform. Having a few false positives isn't an issue.
In Crystal's aport this spec is simply muted (https://git.alpinelinux.org/aports/tree/community/crystal/fix-spec-std-socket-tcp_server_spec.cr.patch?id=29c6464be19088283c52dd74e90bbbd4f9829a6c).

Ceterum censeo continua integratio pro musl esse habendum #6943

quite related to #7422. Is that one also affected?


html2canvas:Error loading image

case 7:
 e_1 = _b.sent();
 Logger.getInstance(this.options.id).error("Error loading image " + container.src);
  return [3 /*break*/, 8];

The project uses html2canvas. js, which is very powerful, but there is a user avatar that sometimes does not exist. Check the print log error Error loading image, what is the reason?

Without more information on how to replicate the issue there is really nothing I can do to help.

in iphone 5c, if img is base64, the pages will report the error:Error loading image

below iOS 10.2.x will report the error: Error loading image data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAPoAAAD6CAYAAACI7Fo9AAAAAXNSR0IArs4c6QAAD1xJREFUeAHtnYFqHUsOBeMl///LXg+swc…

you can use the version 0.5.0-beta4.this version is ok.

it's work for me . nice :)

Use online links or img  binary format

您可以使用0.5.0-beta4版本。可以使用此版本。

这对我有用。不错:)

我生成出来是空白的

in iphone 5c, if img is base64, the pages will report the error:Error loading image

iPhone OS is IOS8 - 10 ?

in iphone 5c, if img is base64, the pages will report the error:Error loading image

iPhone OS is IOS8 - 10 ?


Updated SA1125 to also trigger for @Nullable

Fixes #2284

Codecov Report

Merging #2286 into master will increase coverage by <.01%.

@@            Coverage Diff             @@
##           master    #2286      +/-   ##
==========================================
+ Coverage   96.84%   96.84%   +<.01%
==========================================
  Files         598      598
  Lines       84061    84078      +17
  Branches     3200     3200
==========================================
+ Hits        81406    81423      +17
- Misses       1788     1790       +2
+ Partials      867      865       -2

Impacted Files
Coverage Δ

...abilityRules/SA1125UseShorthandForNullableTypes.cs
88.46% <100%> (ø)
:white_check_mark:

...Analyzers.Test/ReadabilityRules/SA1125UnitTests.cs
100% <100%> (ø)
:white_check_mark:

...p.Analyzers.Test/Helpers/CodeFixVerifier.Helper.cs
75.4% <ø> (ø)
:white_check_mark:

...yleCop.Analyzers.Test/Verifiers/CodeFixVerifier.cs
76.98% <ø> (ø)
:white_check_mark:

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 4755c66...4cdc15f. Read the comment docs.


AttributeError: 'Delegate' object has no attribute '_library'

Hello, Just installed it on MacOS 11.5 and when I run the first example "python3 bodypix.py" I get this error:
Model: models/bodypix_mobilenet_v1_075_640_480_16_quant_edgetpu_decoder.tflite Traceback (most recent call last):   File "bodypix.py", line 180, in <module>     main()   File "bodypix.py", line 161, in main     engine = PoseEngine(model)   File "/Users/fabrizio/Desktop/work/MagneticChessResearch/machine learning/Coral/edgetpu_runtime/coral/pycoral/TEST1/project-bodypix/pose_engine.py", line 142, in __init__     edgetpu_delegate = load_delegate(EDGETPU_SHARED_LIB)   File "/Users/fabrizio/.local/share/virtualenvs/Coral-u4jlFv0K/lib/python3.8/site-packages/tflite_runtime/interpreter.py", line 152, in load_delegate     delegate = Delegate(library, options)   File "/Users/fabrizio/.local/share/virtualenvs/Coral-u4jlFv0K/lib/python3.8/site-packages/tflite_runtime/interpreter.py", line 81, in __init__     self._library = ctypes.pydll.LoadLibrary(library)   File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ctypes/__init__.py", line 443, in LoadLibrary     return self._dlltype(name)   File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/ctypes/__init__.py", line 365, in __init__     self._handle = _dlopen(self._name, mode) OSError: dlopen(libedgetpu.so.1, 6): image not found Exception ignored in: <function Delegate.__del__ at 0x125e94ee0> Traceback (most recent call last):   File "/Users/fabrizio/.local/share/virtualenvs/Coral-u4jlFv0K/lib/python3.8/site-packages/tflite_runtime/interpreter.py", line 116, in __del__     if self._library is not None: AttributeError: 'Delegate' object has no attribute '_library'
Any ideas?

@fablau Were you able to fix this ?

Unfortunately not, the problem is still there. Any thoughts?

Can you try to install the edgeptu runtime folllowing these instructions: https://coral.ai/docs/accelerator/get-started/#runtime-on-mac

Yes, that's what I have done and I am able to run the tests as well as the following examples without issues:
https://github.com/google-coral/examples-camera

cool, can we close this issue?

Why? I haven't been able to make this run. I still get the error I mentioned in my first post above!

sorry, I thought you are able to run the tests without issues.

I am able to run the tests here:
https://github.com/google-coral/examples-camera
But not this specific "project-bodypix" we are discussing about here ;)

can you please confirm you have downloaded tflite runtime form this link https://github.com/google-coral/pycoral/releases/download/v1.0.1/tflite_runtime-2.5.0-cp38-cp38-macosx_10_15_x86_64.whl as you are using python3.8 and Mac
releases: https://github.com/google-coral/pycoral/releases

Yes, I can confirm that.

Hmm..please share the output of below command:
v4l2-ctl --list-devices

With that, I get a "v4l2-ctl: command not found" error.

If you see the code at https://github.com/google-coral/project-posenet/blob/ugrade-to-frogfish/pose_engine.py#L79-L83 , it requires posenet_decoder.so.
Make sure that the library is at the correct path at https://github.com/google-coral/project-posenet/blob/ugrade-to-frogfish/pose_engine.py#L32
Library is available at https://github.com/google-coral/project-bodypix/tree/master/posenet_lib/x86_64

Thank you for your suggestion, but I have that library inside the same directory of the "bodypix.py" script:
posenet_lib/x86_64/posenet_decoder.so
It should work, right?

Hi @fablau unfortunately Project-bodypix does not work for MacOS. It would work on linux machines and coral dev board.

Are there plans to support macOS? I'd love to see this running without having to drop into a VM

@aldenprudent we are planning to add support to macOS.. Meanwhile can you try
posenet_decoder.dylib by placing here

and changing this line to EDGETPU_SHARED_LIB = 'libedgetpu.1.dylib'
and changing this line to     'posenet_lib', os.uname().machine, 'posenet_decoder.dylib')

Sorry for the late reply on this. I'm unable to get the gstreamer bits working but I am able to get a basic image response now. Here's the diff.

hi @hjonnala I seem to be encountering a similar situation. I got the WSL with Ubuntu on my Windows since the repo only works on Linux based systems. So I'm using Ubuntu on Windows terminal to do most of the work now. I came across the tflite_runtime issue as well and was confused what you meant by downloading the pycoral version. I have python 3.8 and linux so I just downloaded pycoral-2.0.0-cp38-cp38-linux_x86_64.whl. I hope that's the right one, and I dragged that into the bodypix project folder. I tried the v4l2-ctl --list-devices command and got the following error: Cannot open device /dev/video0, exiting. I was confused about what exactly I should do from here.

If you see the code at https://github.com/google-coral/project-posenet/blob/ugrade-to-frogfish/pose_engine.py#L79-L83 , it requires posenet_decoder.so. Make sure that the library is at the correct path at https://github.com/google-coral/project-posenet/blob/ugrade-to-frogfish/pose_engine.py#L32
Library is available at https://github.com/google-coral/project-bodypix/tree/master/posenet_lib/x86_64

Also relating to this comment you made earlier, I was really confused what exactly I'm supposed to be doing here to get the repo working. Please let me know how I might be able to fix the above error and also what exactly I need to do to this ^^ post you made earlier. Thanks!

@zain-altaf  seems to be you haven't installed the edgetpu runtime. Please follow these instructions and try the basic demo first before trying any other repositories. Thanks!


Question: why are exactly 2 peer connections used?

In the code and in the doc we can see, that there are two separate PeerConnection objects. This is OK for 1to1 call, but how about group calls? For SFU and 3 participants we would have per device 1 out + 2 in, however in real - 6, or did I miss something here? Asking because during implementation custom WebRTC client (based on pion, not server-sdk-go) and custom app signaling service (between the client and livekit) observed, that the client with one publisher peer and several subscriber peers probably won't work properly with livekit.

all subscriber tracks are sent down via a single peer connection.


Move runtimesRegistry to pureconfig.

This PR moves the config of the runtimes-registry to pureconfig. In addition it adds the ability to provide credentials for this registry. They are not used yet in the default implementations of the container factory. But they can be used in the SPIs.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[x] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[x] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #4343 into master will decrease coverage by 4.15%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #4343      +/-   ##
==========================================
- Coverage   85.56%   81.41%   -4.16%
==========================================
  Files         169      169
  Lines        7830     7828       -2
  Branches      535      533       -2
==========================================
- Hits         6700     6373     -327
- Misses       1130     1455     +325

Impacted Files
Coverage Δ

...penwhisk/core/containerpool/ContainerFactory.scala
85.71% <ø> (ø)
:arrow_up:

...che/openwhisk/core/loadBalancer/LeanBalancer.scala
0% <0%> (ø)
:arrow_up:

...erpool/kubernetes/KubernetesContainerFactory.scala
0% <0%> (ø)
:arrow_up:

.../containerpool/docker/DockerContainerFactory.scala
67.74% <100%> (ø)
:arrow_up:

...la/org/apache/openwhisk/core/invoker/Invoker.scala
71.66% <100%> (ø)
:arrow_up:

...e/openwhisk/core/mesos/MesosContainerFactory.scala
57.4% <100%> (ø)
:arrow_up:

.../scala/org/apache/openwhisk/core/WhiskConfig.scala
94.57% <100%> (-0.05%)
:arrow_down:

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-95.24%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-92.6%)
:arrow_down:

... and 7 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 bb4e4df...30ca02a. Read the comment docs.


goftests is not available via the conda package manager

I don't have much experience with conda, but goftests seems to be unavailable to conda install through the default repositories. I guess it needs to be uploaded to Continuum's repository in some way.

@jfinkels Do you want to own the conda package? I'm not working on this package anymore, but I could add you to the posterior org if you want.

Umm let me think about it. I don't use conda so it seems silly to put me in charge of it haha. I only ask for my colleagues who use it. Let me discuss it with them.

Thanks to @danring, version 0.2.3 seems to live in conda-forge, a community-driven conda package repository, now: https://conda-forge.github.io/feedstocks. (Not sure how to update the version there.)
Might be nice to add instructions on how to install from that repository before closing this issue.

Version 0.2.5 is also now available via conda-forge.
Assuming you have a working conda environment, you can install goftests (from the conda-forge "channel") with:
conda install -c conda-forge goftests


If inferenced on the non-rectified image (no preprocessing but the wall in the image is vertical), the corners are not correct

@sunset1995, thank you for the great work.
I have a question here:
it seems the inference corners after post-processing only work on the pre-processed image (rotate by the vp). Is it possible to visualize the corners before the rotation? Cheers.

Suppose I have an image that contains the walls vertically in the image. But after the pre-processing, the image is rotated by a certain degree.
When the image is inferenced & post-processed, the corners detected are at the wrong positions if using the non-pre-processed image.
For easier understanding, there is an example:

This is the non-pre-processed image

This is the pre-processed image
The pre-processed image is rotated along the x-axis by some degrees.
If only visualized the corners after the inference, they are all looked good. The problem is after the post-processing, i.e. rotating the scene by the avg angle of the PCA. Then the corners detected on the image are shifted.
Here is a screenshot:

Sorry for late reply.
The inference.py with --visualize can visualize the raw output (the probability map) from model.
The post-processing implemented here only support pre-processed image.

@sunset1995
Hi, thanks for the reply, the visualize argument can be used to visualize the image in 3D space without problem. May I ask how to get the corners from the original image (without pre-processing)?

Same problem here : points are moved to the left, unsure why

Same problem here : points are moved to the left, unsure why

Same here.

Same problem here. I found the vote function invalidates all XY corners, then use "best_fit = np.median(vec)".
Could you explain how it works?
Input:

Result:

pre-processing is need.
in the paper, the author says that the pre-processing algorithm fails will to correctly align the horizontal rotation of the panorama by PCA.
so check this rotation of PCA in post_proc.py and inversely use it in the final result.
In addition, I think pre-processing also need to rotate the label if annotate it in the original panorama image.


Question about current tagged alpha version

Hi!
I'm implementing this theme into a documentation build environment and noticed that you have a PR in master and not in 0.1.10-alpha, which is PR #345
It's awesome that this is in master, but as it's added to the v0.1.10 milestone, I was curious when it might be implemented into the v.0.1.10-alpha tag? Normally at home, I'd just use master, but my production environment is based solely off of tagged versions.
Thanks for the awesome work!

Please close this @ericholscher

Has it been addressed, @Blendify ?

It looks like yes, this has been addressed and is included in the tag of 0.2.0
I'll go ahead and close it.


Unable to load Kernel gemv_t... when using matmul

Not sure if this is an issue of mine, but I'm unable to use mx.matmul or use the @ operator on two mx.arrays, it throws the following error:
` libc++abi: terminating due to uncaught exception of type std::runtime_error: [metal::Device] Unable to load kernel gemv_t_complex64_bm8_bn8_tm4_tn1
zsh: abort `
This is my whole code:
`def make_nplr_hippo(N:int):
nhippo = -1 * make_hippo(N)
p = mx.array(0.5 * np.sqrt(2* np.arange(1, N+1) + 1))
q = 2*p
p = mx.transpose(mx.expand_dims(p, axis=0))
q = mx.expand_dims(q, axis=0)
S = nhippo + p_ * q_
lambda_, V = np.linalg.eig(S)
return lambda_, p, q, V
n = 3
lambda_, p, q, V = _make_nplr_hippo(n)
Vc = mx.transpose(mx.array(np.array(V, dtype=complex)))
p = mx.matmul(mx.array(Vc), p)
print(p)
`
Im using Mac M1 on VsCode
Python 3.11.3
Package         Version

mlx             0.0.4
numpy           1.26.2
pip             23.3.1
pybind11        2.11.1
pybind11-global 2.11.1
setuptools      67.6.1

You're not doing anything wrong, we don't yet have a complex64 metal gemm implemented (or complex64 cpu gemm for that matter).
@jagrit06 any thoughts on this? How hard would it be to get the complex type working?
We could easily add cblas_cgemm for the CPU backend.
For the Metal backend another option might be MPS. It has a complex type, maybe it works with their gemm kernel.


Fix small  typo

Not a typo: https://en.wikipedia.org/wiki/Frecency
Maybe the word should just to be linked to the Wikipedia article to avoid confusion.

Thanks! I didn't know about this. Wikipedia link is a good idea, or consider avoiding jargon.


macOS needs work

when running make in C-Turtle/build/ on macOS 10.14.5, I got this:
fatal error:
'omp.h' file not found
#include <omp.h>
^~~~~~~
1 error generated.
make[2]: *** [CMakeFiles/CTurtle-Sandbox.dir/sandbox.cpp.o] Error 1
make[1]: *** [CMakeFiles/CTurtle-Sandbox.dir/all] Error 2
make: *** [all] Error 2
fix!!!

That's a result of a test I ran with OpenMP support, and has since been removed. Try re-cloning the development branch and compiling it.


Need help to build if you can!

I have this error when i try to build it but dont know why i manage to build arduino code bot c++ is new to me...
`C:\ScottPlayer-main>"C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.exe" -S. -Bcmake_vs -G"Visual Studio 17 2022" -DCMAKE_PREFIX_PATH=C:\Qt\5.15\5.15.2\msvc2019_64
-- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19043.
-- The CXX compiler identification is unknown
CMake Error at CMakeLists.txt:3 (project):
No CMAKE_CXX_COMPILER could be found.
-- Configuring incomplete, errors occurred!
See also "C:/ScottPlayer-main/cmake_vs/CMakeFiles/CMakeOutput.log".
See also "C:/ScottPlayer-main/cmake_vs/CMakeFiles/CMakeError.log".
C:\ScottPlayer-main>pause;`

ok i installed windows 10 SDK and the batch file worked but now i have some new sources files do i need to compile them in visual studio now??


RTSP Audio

I am trying to get audio to work with https://github.com/AlexxIT/WebRTC.
AAC audio does not work with WebRTC (only PCM) and MSE protocol (with AAC) is so choppy it is unusable.
I see there is an PCM option in the Configurations page for RTSP audio, but it does not seem to do anything (or am I missing something)?
I would like to know if it is possible to get PCM audio in the RTSP stream, so that it might be a solution for using WebRTC with audio via yi-hack RTSP.
I am using a Yi Outdoor 1080p (h30ga) release.
Thanks!

alaw and ulaw are inside the PCM family.
They are different from linear PCM but work with WebRTC.
You can use them.

Outstanding!  They works great!  Can't really tell any difference between alaw and ulaw.  No need to reply, but if you, or anyone, have any wisdom about which to use, I would be happy to hear it.  In the mean time, I am using ulaw...
Thanks!!!

Both algorithms are implemented in telephony systems
Ulaw is currently being used by companies in North America and in Japan while alaw is being used in Europe.
Hard to say which is the best.

Thanks for your reply!
On Fri, Jun 4, 2021, 10:22 AM roleo @.***> wrote:

Both algorithms are implemented in telephony systems
Ulaw is currently being used by companies in North America and in Japan
while alaw is being used in Europe.
Hard to say which is the best.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/roleoroleo/yi-hack-Allwinner-v2/issues/181#issuecomment-854854867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADFMVIRJFEYH7UTAVMR3YB3TRD4VZANCNFSM46A77RZA
.


semgrep{,-core}: 0.112.1 -> 0.117.0

Description of changes

Bumped semgrep and added an update script
Still outstanding: resolve #192149
I have a different issue that is blocking me from checking if the issue above is fixed:
I've added tomli but it's not picking it up when running semgrep scan
It's also supprising that the pip hook doesn't complain if I don't include tomli even though setup.py asks for it...
To reproduce follow the same steps as the issue above & see issues with finding tomli
cc: @SuperSandro2000
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Is the python script correct wrapped?

It was working fine until a couple versions ago when they added tomli as a dependency. tomli seems to be the only dependency with an issue 🤷♂

@ofborg build semgrep


(valid 'to' email address required') error even i entered correct email

let email =<EMAIL_ADDRESS>var input =	{ 'to': { email: 'to whom!' },
'from'<EMAIL_ADDRESS>'from email!'],
'subject': 'Test mail form sendinblue',
'html': 'This is the
HTML
'

};
sendinObj.send_email(input, function(err, response){
if(err){
console.log(err);
} else {
console.log(response);
}
});
if i put email in a variable give me this error
{ code: 'failure',
message: 'valid 'to' email address required',
data: [] }

Hi @abdelrahmanbadr
Your sample { 'to': { email: 'to whom!' } seems incorrect, please refer below:
require("../mailin.js");
	var client = new Mailin("https://api.sendinblue.com/v2.0","your access key");
	data = { "to" : {"to@example.net":"to whom!"},
		"from"<EMAIL_ADDRESS>"from email!"],
		"subject" : "My subject",
		"html" : "This is the HTML",
		"attachment" : ["https://domain.com/path-to-file/filename1.pdf", "https://domain.com/path-to-file/filename2.jpg"]
	}

	client.send_email(data).on('complete', function(data) {
		console.log(data);
	});

Doc reference: https://apidocs.sendinblue.com/tutorial-sending-transactional-email/
Thanks

no i just didn't put the whole sample.
the is the whole code sample :
var sendinblue = require('sendinblue-api');
var env = require('./environments/env');
var parameters = { "apiKey": env.sendInBlueKey, "timeout": 5000 };	//Optional parameter: Timeout in MS
var sendinObj = new sendinblue(parameters);

let email =<EMAIL_ADDRESS>var input =	{ 'to': { email: 'to whom!' },
'from'<EMAIL_ADDRESS>'from email!'],
'subject': 'Test mail form sendinblue',
'html': 'This is the
HTML
'

};
sendinObj.send_email(input, function(err, response){
if(err){
console.log(err);
} else {
console.log(response);
}
});

but it will work if don't put email in a variable like that :
do u have any idea why ?
var input =	{ 'to': {<EMAIL_ADDRESS>'to whom!' },
'from'<EMAIL_ADDRESS>'from email!'],
'subject': 'Test mail form sendinblue',
'html': 'This is the HTML'

@abdelrahmanbadr If you use
let email =<EMAIL_ADDRESS>var input = { 'to': { email: 'to whom!' },

Node will not replace email value instead its treated as an object key.

Hi ekta-slit,
Can you guide us in how to achieve this requirement, because this is a valid scenario where you have to call the Sendinblue function to send an email and pass the receiver email id as a parameter from some other function?
I'm also stuck in the same issue. In our case, the above code is a lambda function so we need to pass receiver's email as a parameter to this lambda, please suggest if you have any idea to avoid this issue.
Thanking you in advance.

Hi @hitenkaram
You can do it like this: first initialize the object and use brackets [] to set key value.
A simple example:
let obj = {};
let key =<EMAIL_ADDRESS>obj[key] = "to whom!";

let field = "to";
let input = {};
input[field] = obj;

console.log(input);

Response:
{ to: {<EMAIL_ADDRESS>'to whom!' } }

Thanks

Thank you ekta-slit for the help.


Patient ID sequence should not go backwards (greater to lesser)

It was observed and confirmed that at the Tebow hospital, the patient ID sequence was set at a number less than the greatest patient ID in the system.   There were 2000+ patients in the database currently, but the sequence was set at something around 1400.   It was not causing an issue, since the system prevented creation of patients with duplicate IDs.   However, in consult with @jkleinsc it was agreed that this behavior should be corrected.

The suggested next patient ID is saved in the sequence model. However, if we use a custom ID in the form, the model won't update accordingly. For example, entering P00010 for a suggested P00005, it would next time suggest P00006 rather than P00011. Guess this is part of the reason why the sequence comes slower than the greatest patient ID.
A related problem happens when we cancel a new patient form. Because it's cancelled, the suggested ID is not used, but the number keeps getting incremented regardless. The sequence just goes too fast this time.
Not sure if these are root cause of what @mnorbeck encountered, but the situation shouldn't appear again once the sequence cached up with the greatest patient ID.
These two problems can be tackled separately, and I'd like to try sending a pull request for the second one first.

Is this assigned? I don't see anyone assigned yet.

@mdarmani @gnowoel is working on this.

I am going to close this as #667 addressed this issue.  #572 is closing the loop on making sure dups can't be entered.


questions: key shortcut and start

1.why key shortcut use webstorm double cilck change…
2.default disable all plugins keyshortcut ，show key exec command at readme，users  control on/off（like checkbox）
3.atom start time very slow （compare for sublime）……next version can reduce start time？

Hey there @crperlin!
Your first issue is already being tracked at atom/settings-view#163.
For your second issue, you can now enable/disable all keybindings for a particular package:

And for your third, that's being tracked at #2654 and #3673.
Hopefully that answers your questions!


v0.14 is throwing build errors on Cloudflare Pages

I believe that with the introduction of glob in d41c0cd, super-sitemap stops working with Cloudflare Pages, which is not a Node runtime.
I may be wrong, but I think the issue is related to that, since the modules it's trying to import are not available in Pages and glob depends heavily on them.

These are the changes I made to use indexed sitemaps in my project:

Thanks for the great library! I'm glad it is still being maintained :)

Are you using sampledPaths() or sampledUrls()? Those are the only methods that should pull in glob, and those are for use in Playwright only.
Asking because I also use Cloudflare Pages and have the latest super sitemap building and deploying without errors. My build command is the standard "build": "vite build",.

Thanks for the quick reply! I am not importing sampledPaths() or sampledUrls() and this is how the Playwright tests look like:
import { expect, test } from '@playwright/test';

test('/sitemap.xml is valid', async ({ page }) => {
	const response = await page.goto('/sitemap.xml');
	expect(response?.status()).toBe(200);

	// Ensure XML is valid. Playwright parses the XML here and will error if it
	// cannot be parsed.
	const urls = await page.$$eval('url', (urls) =>
		urls.map((url) => ({
			loc: url.querySelector('loc')?.textContent,
			changefreq: url.querySelector('changefreq')?.textContent, // if you enabled in your sitemap
			priority: url.querySelector('priority')?.textContent
		}))
	);

	// Sanity check
	expect(urls.length).toBeGreaterThan(5);

	// Ensure entries are in a valid format.
	for (const url of urls) {
		expect(url.loc).toBeTruthy();
		expect(() => new URL(url.loc!)).not.toThrow();
		expect(url.changefreq).toBe('weekly');
		expect(url.priority).toBe('0.7');
	}
});

I tried manually removing the sampledPaths and sampledUrls exports from super-sitemap's index.js and the build worked. Here's a demo:
https://github.com/jasongitmail/super-sitemap/assets/47724710/03af41d5-f6f5-4c9b-81a6-f07a0742000d
I'm using "@sveltejs/adapter-cloudflare": "2.3.3".

What is doppler? Have you tried pushing to Cloudflare? There might be a separate issue here, but I'm curious if yours works on actual Cloudflare, as mine does.
Can you share you sitemap code? If you're using import * as something, then it may be pulling in sampledPaths or sampledUrls

I can always use a 2nd export in the package for sampledUrls and sampledPaths, if needed, but I'd prefer not to if we can identify what is causing it here.

I'm going to replace glob for directory-tree because it's much smaller and has no deps of its own other than fs and path. We can continue troubleshooting after that, since it'll simplify things to some extent.

@fell-lucas I pushed 0.14.5 which uses directory-tree now and removes glob.
All super sitemap tests pass, and it builds successfully in my Cloudflare Pages SK project.
Can you try building again with 0.14.5? I'd like to see what your latest error message is.

I just tested with 0.14.5 and it still fails, but with less errors:

Also, the generated js code in .svelte-kit/output/server/entries/endpoints/sitemap__page__.xml/_server.ts.js is much more simplified than before.
If there is any additional information I can provide, let me know.

@fell-lucas Give 0.14.6 a try. I tried refactoring the exports. It should work the same either way, but worth a try.
Otherwise, I'm running out of ideas for the moment and would benefit from a minimal repro showing the issue on stackblitz or steps to reproduce, because I'm unable to reproduce a similar error myself currently.

Thanks. I tried, but still the same errors occur. I was able to solve it by adding a ./sitemap attribute to the exports in package.json:
{
  ...
  "exports": {
    "./sitemap": {
      "types": "./dist/sitemap.d.ts",
      "svelte": "./dist/sitemap.js",
      "default": "./dist/sitemap.js"
    },
    ".": {
      "types": "./dist/index.d.ts",
      "svelte": "./dist/index.js",
      "default": "./dist/index.js"
    }
  },
}

And then importing like
import { response } from 'super-sitemap/sitemap';

This way, only the code from sitemap.js is imported, ignoring sampled.js. However, I'm not sure how to fix the TS error that started happening:

I'll see if I can publish a reproducible stackblitz tomorrow

Thank you. That stackblitz was helpful!
I was able to reproduce it now too. I see fs and path imported into the sitemap endpoint during the build at .sveltekit-output/server/entries/endpoints/sitemap__page__.xml, for some reason, even though neither fs nor path are called within that file or by sitemap.response()
import "fs";
import "path";

The reason I couldn't see the error is because I use export const prerender = true in my project. Adding this to your project, if it is suitable for your data, can be a solution or temporary workaround.

I was able to solve it by adding a ./sitemap attribute to the exports in package.json:

Yeah, that's the multiple exports solution I mentioned earlier. I could move sampled into it's own export, but it's not as clean of a DX for devs imo. Might have to though. (Your type error would go away if I do this within super sitemap.)
But I'd like to figure out a solution to the root cause though, which is why is the build importing fs and path in the sitemap endpoint, even though it's not needed.

@fell-lucas I pushed 0.14.7 and have the Stackblitz repo building successfully with it: https://stackblitz.com/edit/node-mzeci4?file=package.json
Can you try in your actual project too, to confirm?

That's great! I didn't know about sideEffects. Is that a configuration specific to Vite?
I just updated to v0.14.7 here and it is building correctly. Sitemap is also being generated successfully. Thanks a lot for the quick replies and code changes!

Great to hear!
Seems to be a feature supported by Vite, Webpack, and maybe others. Just learned about it too!


query frontend: sorted queries do not produce sorted results for shardable queries

Thanos, Prometheus and Golang version used:
thanos, version  (branch: , revision: 2d6b0d43f4e6cbd6089a70b4546a324052bacc8e)
build user:
build date:
go version:       go1.19
platform:         darwin/arm64
(Latest main as of 2023-01-19)
Object Storage Provider: N/A
What happened:
When running a query-frontend with vertical sharding enabled, wrapping shardable instant queries with sort() or sort_desc() does not
produce sorted results.
What you expected to happen:
Instant queries wrapped in a sort() should return results ordered by value
How to reproduce it (as minimally and precisely as possible):
Run a query-frontend with --query-frontend.vertical-shards=2
Issue a shardable query wrapped with sort, e,g,:
sort(sum(rate(container_cpu_usage_seconds_total{}[2m])) by (pod))

Full logs to relevant components:
N/A

This seems a bug indeed. I can see if sort or sort_desc is the root expression, then we will hit the bug because we always sort by series labels when merging, not by sample values.
We can:

disable sharding if sort or sort_desc is the root expression
Sort based on values if we see sort or sort_desc

We should be able to implement 2. fairly easily. We did something similar in the engine for topk/bottomk: https://github.com/thanos-community/promql-engine/pull/156/files


⚠ PrivateHD has degraded performance

In c923056, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2255 ms

Resolved: PrivateHD performance has improved in be98f5d.


Double Licenses

Hey Team
Just a quick note to let you know that we are confirming which license is the correct one for this dataset. The original one has the MIT but the tidytuesday uses Creative Commons Zero v1.0 Universal.
We will keep you posted, we are confirming with Tiffany.
Thanks!

Hi Kristin,
It's fine to use the MIT one! Other groups are using it as well. Here's from her instructions:
"You may have noticed when you create a new repo on GitHub, you’re asked to set a license for your project. If you do not currently have a preference, we recommend you use the MIT license for your project."

Thank you! Sounds good, fine for us to use for our dataset.
I still want to confirm which one we should be quoting about the original data set, there is reference to two and I just want to be sure we are referencing the correct one.

Hi everyone, just heard back from Florencia on this issue. She confirms that we should be using the MIT license on our repo as well. I have added this to the readme file as well.


If i have both a nvidia and intel arc card will Ollama use both?

I am considering a intel arc card for AV1 encoding and i would like to get more use out of it than just encoding, so if i have both a nvidia card and intel arc in the same system, will Ollama use both cards simultaneously just like any other multi gpu setup or do they have to be the same type because of cuda?

@AncientMystic Intel has been a great partner of Ollama, and we are of course working hard to make it run smoothly. That being said, Ollama doesn't support the Arc cards well enough yet.
As of now, it's really hard to recommend using cards from multiple vendors as driver support is just getting there.
Thanks!


Tiny spelling mistake in project submission confirmation

Description
It says 'Out' instead of 'Our mentors' on in the confirmation displayed after submitting a project. (https://courses.openmined.org/courses/our-privacy-opportunity/project)
How to Reproduce

Go to 'https://courses.openmined.org/courses/our-privacy-opportunity/project'
Submit a project
Scroll down to the end of the page
Click the blue box where it says to write your proposal.
Text pops up Starting with "Thank you for your submission! Out mentors will..."

Expected Behavior
"Our" instead of "Out"
Screenshots
If applicable, add screenshots to help explain your problem.
System Information

OS: OSX

Thank you @nvw1, I'll ship this in the next batch.


Secuyou Smart Lock app

Hi!
Cool project, I'm trying to run this with my lock that uses the Secuyou Smart Lock app.
I'm curious how you managed to connect to your lock, is it also using a pin code?
Could you point me to the code that handles the pairing?

Hi
Sorry for not noticing this before. I had an old lock, but I recently bought the new one.
It works a bit differently, so I ended up creating a new project for it.
It's still a bit buggy, but it works. Check it out if you're interested: https://github.com/joymyr/secuyou_remote

Thanks for the reply!
I actually managed to do something similar by mimicking the original app and adding mqtt to it.
Will check out your project


Add symbolize::Input::into_inner() helper

Introduce a helper method, into_inner(), that allows for convenient retrieval of the payload of a symbolize::Input object.

Great, and we are back at Codecov erring out due to miniscule coverage decreases...


feat: Implement attribute input token mapping, fix attribute item token mapping

The token mapping for items with attributes got overwritten partially by the attributes non-item input, since attributes have two different inputs, the item and the direct input both.
This PR gives attributes a second TokenMap for its direct input. We now shift all normal input IDs by the item input maximum(we maybe wanna swap this see below) similar to what we do for macro-rules/def. For mapping down we then have to figure out whether we are inside the direct attribute input or its item input to pick the appropriate mapping which can be done with some token range comparisons.
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/9867

Haven't read this through yet, but my gut feeling is that we should be able to use existing infra without modifications. I don't immediately see the place where we produce two inputs for proc macro, but can we modify that place to use disjoint ids? In some sense, can we say that the input to proc macro is always a single subtree with none parens and two children (the attribute and the item)?
Again, don't yet understand the code, might be talking nonsense here!

Ah, I see now! Here
https://github.com/matklad/rust-analyzer/blob/d1cd81f3874cb71a514427060e037b4dcae9b814/crates/hir_expand/src/db.rs#L378-L392
we get two arguments from separate code paths. The item is macro_arg, and attr_args we get from macro id. I wonder why is that? Naively, I'd expect both to be a part of macro_arg

I wonder why is that?

Ah, so the thing is, becaues we don't know if attr's arg is actually the macro input. Things like #[allow(missing_docs)] are not macros, and we want to preserve then in hir. That's why we store argument as a tt in AttrInput, and that's what we re-use in expansion. And, indeed, tokens from AttrInput are missing any kind of token map.

Can we add a test here?

We don't really have a testing infra for mapping yet do we? Guess I'll look into that first then.


Errors trying to use logomaker

I am trying to use the logomaker python package, https://logomaker.readthedocs.io/en/latest/examples.html.
I try this:
logomaker = PythonCall.pyimport("logomaker")
crp_df = logomaker.get_example_matrix("crp_energy_matrix"; print_description=false)
crp_logo = logomaker.Logo(crp_df, shade_below=.5, fade_below=.5, font_name="Arial Rounded MT Bold")
crp_logo.style_spines(visible=false)
crp_logo.style_spines(spines=["left", "bottom"], visible=true)
crp_logo.style_xticks(rotation=90, fmt="%d", anchor=0)
crp_logo

following the first example in the link above.
However this gives a series of errors.
First, the following line is printed multiple times: Intel MKL ERROR: Parameter 11 was incorrect on entry to DGEEV.. Not sure what that means.

For the first, I'm not sure. I'd guess Python's MKL is conflicting with Julia's somehow.

Thanks. Using pylist solves the second issue. However I still do not see a plot. Instead a python object is returned.

Return crp_logo.fig to return the underlying Matplotlib figure instead.

That works, thanks.
The only remaining issue is MKL. Do you have any ideas about that?
Or it means I should not use MKL in a Julia session if I want to call PythonCall?

There's a warning on the MKL readme that it needs to be the first package loaded. Maybe it's that?
Or maybe you need to make sure that Python is using the same version of MKL?
Or just don't use MKL.jl like you suggested. I'm not sure.

Ok I'll close this issue.
Thanks for your help.


#14001: Run pre-commit on PRs so CMake formatting (and others) are checked

Follow-up to #14117 to enforce formatting in future edits.

Sample output: https://github.com/tenstorrent/tt-metal/actions/runs/11485537027/job/31965852147


Slowest unit-tests

        PASS [  34.959s] fedimint-ln-tests::fedimint_ln_tests gateway_protects_preimage_for_payment
        PASS [  40.010s] fedimint-server net::peers::tests::test_connect
        SLOW [> 60.000s] fedimint-ln-gateway::gatewayd-integration-tests test_gateway_cannot_pay_expired_invoice
        SLOW [> 60.000s] fedimint-ln-gateway::gatewayd-integration-tests test_gateway_client_pay_unpayable_invoice
        PASS [  66.788s] fedimint-ln-gateway::gatewayd-integration-tests test_gateway_client_pay_unpayable_invoice
        PASS [  68.614s] fedimint-ln-gateway::gatewayd-integration-tests test_gateway_cannot_pay_expired_invoice

If you look at env RUST_BACKTRACE=1 c nextest run  -E 'test(test_gateway_cannot_pay_expired_invoice)' --nocapture
It gets stuck for a long time on:
2024-04-11T04:11:43.630878Z  WARN task{name="state machine executor"}: fedimint_ln_client::pay: Error while trying to reach gateway: Lightning Gateway failed to
 pay invoice. ErrorCode: Some(500) ErrorMessage: An internal gateway error occurred
2024-04-11T04:11:44.633183Z ERROR pay_invoice{payload=PayInvoicePayload { federation_id: FederationId(3559d055a1ca14eacb19b49af071beb92cbbaf607cb3a3bf6fe5a9eb03
19dbdb), contract_id: c7061d6728bf61b3948340818a9d8b190c637a71a57090bf5c484f021cca0a74, payment_data: Invoice(Bolt11Invoice { signed_invoice: SignedRawBolt11Inv
oice { raw_invoice: RawBolt11Invoice { hrp: RawHrp { currency: Regtest, raw_amount: Some(10), si_prefix: Some(Micro) }, data: RawDataPart { timestamp: PositiveT
imestamp(1712808688s), tagged_fields: [KnownSemantics(Description(Description(""))), KnownSemantics(PaymentHash(Sha256(66687aadf862bd776c8fc18b8e9f8e20089714856
ee233b3902a591d0d5f2925))), KnownSemantics(MinFinalCltvExpiryDelta(MinFinalCltvExpiryDelta(0))), KnownSemantics(PaymentSecret(PaymentSecret([0, 0, 0, 0, 0, 0, 0
, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]))), KnownSemantics(Features([0, 65])), KnownSemantics(ExpiryTime(ExpiryTime(1s)))]
} }, hash: [228, 167, 177, 64, 117, 72, 64, 44, 167, 53, 50, 237, 209, 249, 228, 95, 103, 100, 252, 247, 90, 99, 231, 224, 238, 219, 132, 96, 50, 6, 156, 215],
signature: Bolt11InvoiceSignature(RecoverableSignature(dff3ff5aa87d74a54336349e5a6e30f06c3722815bb99aafdb3bb184bfe943a12fd48c6ea5370208eda6fb385910c392874e702d2
cd6b39ab8027d7eff13162d00)) } }), preimage_auth: 8c46128c9dfe8d2a554170d0a7704dc0555b5e3732b6e51b55b1fc082cf3842a }}: ln_gateway::rpc::rpc_server: error=Invalid
 Metadata: No federation with id 3559d055a1ca14eacb19b49af071beb92cbbaf607cb3a3bf6fe5a9eb0319dbdb
2024-04-11T04:11:44.633364Z  WARN task{name="state machine executor"}: fedimint_ln_client::pay: Error while trying to reach gateway: Lightning Gateway failed to
 pay invoice. ErrorCode: Some(500) ErrorMessage: An internal gateway error occurred

while from the reading of the test, it should be done in like 5 seconds.
@m1sterc001guy


[fix] Fixed date formatting in test_password_expiration_mail #367

Related to #367

coverage: 97.858%. remained the same
when pulling 2289aee6ddf5e27674a5534e0748705de1e1cec3 on fix-expired-email-test
into 13e6a87f96b291b16c431b2d1828808c2b712c07 on master.


Benchmark modal script revamped - profiling and external libraries (Issue #504)

Basically added support for libraries like cuDNN on modal so that people can run using libraries on cloud gpus. Basically added my own custom docker image for cuda environment setup and also added support for profiling using nsight systems which comes pre-installed. Based on issue #504
Features:

cuDNN supported
openMPI supported
Nsight Systems supported and now by attaching a volume, you can download the report to view it on your local machine in GUI form
instructions in the benchmark_on_modal.py script on how to use the volumes/download reports and existing errors.
-(Please not that profile_gpt2cu.py does not work with this and you have to manually change the command to run nsys profile)

Examples:
training the gpt2 model with cuDNN use:
GPU_MEM=80 modal run dev/cuda/benchmark_on_modal.py
--compile-command "make train_gpt2cu USE_CUDNN=1"
--run-command "./train_gpt2cu -i dev/data/tinyshakespeare/tiny_shakespeare_train.bin -j dev/data/tinyshakespeare/tiny_shakespeare_val.bin -v 250 -s 250 -g 144 -f shakespeare.log -b 4"
For profiling using nsight system:
GPU_MEM=80 modal run dev/cuda/benchmark_on_modal.py
--compile-command "make train_gpt2cu USE_CUDNN=1"
--run-command "nsys profile ./train_gpt2cu -i dev/data/tinyshakespeare/tiny_shakespeare_train.bin -j dev/data/tinyshakespeare/tiny_shakespeare_val.bin -v 250 -s 250 -g 144 -f shakespeare.log -b 4"
NOTE: Currently there is a bug in the profiling using nsight system which produces a lot of errors on the command line but it
does not actually interfere with the model training and validation. The report (that you download) is still generated and can be viewed from Nsight Systems. Additionally, 'profile_gpt2cu.py' does not work with this too.
Basically enables more features to be used on the cloud gpus. Please feel free to add-on/optimize this script so that more folks can explore the codebase and test out all kinds of features and profiling the codebase has to offer currently.

I am still working on a newer base image for this so that the nsight errors disappear but meanwhile it should be working for most of it

I just fixed the gcc versioning mistake, and cuDNN should also work with the modal script now.

Since the -std=c++17 flag was added into Makefile in a previous commit, this should work perfectly now.

cudNN works ✅
Nsys profile using "nsys profile --cuda-graph-trace=graph --python-backtrace=cuda --cuda-memory-usage=true  {executable} also works (although a GPU uuid error is generated, almost everything is profiled) ✅
OpenMPI also installed ✅


Add ORCID information to descriptive metadata

ORCID IDs added to H2 should be stored in descMD.
@arcadiafalcone can you confirm the location where these should be stored? Thanks.

https://github.com/sul-dlss/dor-services-app/blob/9ac97a4830684223e27c34d1dcfd84b26fcec27a/spec/services/cocina/mapping/descriptive/h2_datacite/contributor_h2_datacite_spec.rb#L1601 and https://github.com/sul-dlss/dor-services-app/blob/9ac97a4830684223e27c34d1dcfd84b26fcec27a/spec/services/cocina/mapping/descriptive/h2_datacite/contributor_h2_datacite_spec.rb#L1700 show the H2-cocina-DataCite mappings. I'll add ORCID to the H2-cocina-MODS mappings as well.

PR for the MODS mapping: https://github.com/sul-dlss/dor-services-app/pull/2946

this may be partially done?  Also needs another pass from @arcadiafalcone re: roundtripping concerns (?)

@ndushay no, this ticket primarily about exporting this data from h2.


Feature/pkp sui

This module defines the PKPSuiWallet class, which is a modified version of SignerWithProvider class from @mysten/sui.js
This feature is from yhl125/pkp-sui
Demo wallet extension: yhl125/lit-suiet
Demo video: https://youtu.be/IhUsXxZzumE
For detailed explanation, please refer to https://github.com/LIT-Protocol/LitGrants/issues/52

Hello @Ansonhkg! Thank you for the review!
I've signed Contributor License Agreement and fixed all things you commented.
We don't know when devnet and testnet will reset. In sui discord they said "Devnet is reset weekly. Testnet was reset on May 11".
If unit test fails, request sui testnet faucet and try again and it should work.

@yhl125 Published version 2.2.45!


"Tried to register widget" error when performing a search in shared version of app

Steps to repro:

Launch either http://<IP_ADDRESS>/EventCalendarDec15 or http://<IP_ADDRESS>/ShelterLocatorDec15 and click OK on the splash screen.
On the search panel, search using the defaults
Once the search completes, share the application via email
Open the shared version of the application. On the search panel, perform a new search

for Event Calendar, search "Naperville" and select "Naperville Ale Fest" from the list
for Shelter Locator, search "Larkspur" and select "Larkspur Middle School" from the list

Note you get an error

Could this be related to the blue and purple themes?  We are able to reproduce this behavior with apps configured with the blue and purple themes, but not green or orange.
Note: This is a new behavior.  It is not reproducible with the currently-released version.
FYI @jessnoo

Can no longer repro with https://github.com/Esri/configurable-locator/commit/6df4bd1ae2006002827add9aaae58d9a380d55dc


Deploy to Pypi

The guide is here: https://packaging.python.org/guides/distributing-packages-using-setuptools/

Another<EMAIL_ADDRESS>

Work has been initiated. The project is currently registered by someone else with no release. Issue was opened with pypi support.
https://github.com/pypa/pypi-support/issues/612


scrolling is not working well with your large dataset (and even worse with larger ones)

is it just me or does the scroll handle on the large dataset not follow the mouse?
https://diogofcunha.github.io/react-virtualized-tree/#/examples/large-collection
note the the behavior gets better as you scroll from the very top to bottom and back a few times, like its learning something along the way?
on a larger dataset, I have problems where the scroll handle will run up to the top of the tree and flicker whenever I scroll around?

Hi @wysisoft. Thanks for reporting.
I can't reproduce that behaviour.
Can you give a few more hints about the platform, browser and maybe cpu power?

I adapted react-virtualized example of a treeview and its scroll behavior
has been solid.
https://rawgit.com/bvaughn/react-virtualized/master/playground/tree.html
On Sat, Nov 3, 2018 at 3:24 AM Diogo<EMAIL_ADDRESS>wrote:

Hi @wysisoft https://github.com/wysisoft.
Interesting. I do not scroll like that, that's why I couldn't reproduce
it. Thank you for your video, it's awesome, and sorry about the delay, I
was on holiday 😸.
This will require some deep investigation on my side, I'm not sure the
issue is inside this library or on react-virtualized.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/diogofcunha/react-virtualized-tree/issues/62#issuecomment-435573576,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOeY7DOuoEBNNxjbyNn9MU6EmCYNnhadks5urWDfgaJpZM4Xa7Va
.


[Refactor] From legacy build-tools to opensearch build-tools

Refactors the build-tools dependency from legacy to opensearch build-tools.
Note: still in work...

Merging... License headers task will be addressed in a follow up PR.


Code refactor

Fixes Issue  #90

Changes proposed

Improved avatar component
improved ESlint config
remove the unwanted package
types improved

Note to reviewers

Screenshots

@leewan-09 could you please check the build issue

why i cant view

@leewan-09 i believe it's because you don't have access to the vercel. but you can try to build at your local machine and debug. that would be same.

@leewan-09 i think there's an issue occurred in modal component. please could you check it

Checking

Why did the building fail?  I have tested it locally; it's fine.

this is the error which i can see in vercel terminal @leewan-09

you need to make the filename change. please see.

you need to make the filename change. please see.

you need to make it "Avatar.interface.ts".

Look I have changed like that..

it's still showing the lower case in github. strange

Yes, very strange.

Look in here also

@leewan-09 could you delete that interface file and push it. after that again create that file and push. let's see if that updates in github

Ahhh. now it's okay, That was strange


Standalone application

Hello team, thanks for this repo!
Would you be interested to release a standalone app for this package?

Hi,
Thanks for the interest. Not yet. I think there is more work for this to be really valuable.


credit card options

Add ability for people to pass in options.

[x] Credit card type: Visa, Mastercard, etc...
[x] Luhn boolean
[x] Length
[x] Format

v5.6.0