Error: UPGRADE FAILED: another operation (install/upgrade/rollback) is in progress

This closed issue still exists, hitting it every month as many, many others: https://github.com/helm/helm/issues/8987
this happens when: computer goes to sleep, ci/cd pipeline is interrupted, control+c etcq2
"workarounds" include:
kubectl delete secret sh.helm.release.v1.asdf.v1 -n asdf
and

We reverted to 3.2.4

Ir was closed because the person that reported the issue closed it. They determined that the problem was a flaw in their CI process.
Please provide a debug log from the fault, along with the helm history of the release.

simple control+c during install is enough to repro this

A recent fix was released in Helm 3.7 where a cancelled upgrade via a CTRL+C will force the release into a failed state, allowing you to retry the upgrade, preventing the pending-upgrade issue many users mentioned in that ticket. See https://github.com/helm/helm/pull/9180.
Issues related to connection timeouts are being tracked in https://github.com/helm/helm/issues/4558, but #9180 should solve the CTRL+C and/or CI pipeline being interrupted as both send SIGTERM.

well, just got hit by this again by simply control+c

ACTUALLY somehow an old helm install was installed - so with helm 3.8.0 this works!

https://github.com/helm/helm/issues/4558#issuecomment-958961462


Intituto Técnico Superior Japón

Intituto Técnico Superior Japón

@EMI2004-JB First stage of verification passed, official domain for Intituto Técnico Superior Japón is itsjapon.edu.ec as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@EMI2004-JB Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Bugfix - removing the code to take backup of keys.

This causes issues in case of multiple agents/multiple configurations on the machine.

@dpksinghal,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@dpksinghal If any test cases needs to be added/updated, please do them

Have updated the existing L0 tests for these changes.


Where is Disable-PnPResponsiveUI.ps1

I was looking for the PowerShell file, Disable-PnPResponsiveUI.ps1, but seems to be not available as part of this github repository. Am I missing something here?
Thank you.

Hi @astatke, no need because a method already exists here https://github.com/SharePoint/PnP-PowerShell/blob/master/Documentation/DisablePnPResponsiveUI.md.
Does it meet your needs ?

@astatke, if my previous answer fix your issue, could you close it plz ? thx by advance

Since no answer, closing this one.


Zoom setting not working

Hello,
It appears that the zoom setting is not working, my map stays the same size no matter what setting I put  in zoom.
...
'googleMapsOptions' => [
'mapTypeId' => 'roadmap',
'tilt' => 45,
'zoom' => 5,
],
Thanks for your support
Adam

Hi, @dbd5
zoom property only works when the map is empty. When you use the userLocations, our js script automatically set zoom and center properties for the map.


Instagram embed stopped working

Craft CMS 3.4.15
Embedded Assets 2.2.4

Hi @skoften,
I'm unable to replicate this, but Instagram have been locking down their usage by non-members recently, so it would make sense, depending on how many times that server has accessed Instagram recently, that trying to embed another post would give you a login page instead.  Unfortunately, if that's what's happening, I don't think Embedded Assets can do anything about it.
That said, the JavaScript error when it happens isn't ideal, so I'll fix that in the next release.

Hm, weird. Have you tries this photo/post: https://www.instagram.com/p/B_QQztfKk12/ ?
This is a public post. I tried embedding it on different server, different IP address same error.

Yeah, I had tried that post and it worked.  It's strange that you're getting the login page on a different server, though (unless both have run into the non-member view limit, I suppose, but the chances of that happening on two of your servers around the same time don't seem great to me).  I'll have a look at it on some other servers and see if it happens.

Hmmm. Don't think it's a member view limit, tested on a dev server which doesn't do any requests.
Is there a way to test it from the CLI of my server, with CURL of something? Any command? Maybe this will give me more info.

Yeah, using curl should be fine:
curl -L -o <filename to save> https://www.instagram.com/p/B_QQztfKk12/

If this saves the correct page on a server where Embedded Assets is still getting the login page, we can rule out an issue with view limits.

Tries 4 different servers, 3 different locations, all result in a not logged in page. But 3 out of the 4 servers have never made any requests to Instagram. So it seems not a single request is possible without logging in?

See oscarotero/Embed#357 for the related issue for the Embed library Embedded Assets depends on.  If a fix can be applied and released for version 3 of the Embed library, then we'll update Embedded Assets' minimum requirement and that should fix your issue.  (We are unable to use the newer version 4 at this stage, because that version has a stricter PHP version requirement than Craft has.)
If a fix is released for v4 and not for v3, we will look into alternative solutions for Instagram until we can upgrade to v4 (presumably at the time of the Craft 4 release).

Embedded Assets 2.2.6 has just been released, which upgrades the minimum requirement for the embed/embed library to 3.4.5 which fixes the Instagram embedding issue.


Update view-and-release-quarantined-messages-from-shared-mailboxes.md

@chrisda  just making some updates. please note that this feature is still in private preview

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@faoquong Please update with the feedback. Thanks.


Add the option not to open the browser automatically

I have added this option because my computer has a graphical interface, but I only have access via SSH

LGTM!

Released new version, thanks!
https://github.com/JorgeReus/aws-sso-creds/releases/tag/1.0.3


Migrate calibration matrices out of pushrim kinetics

Currently, our lab's calibration matrices are in the pushrimkinetics module. This was convenient, but it does not belong there since this is lab-only stuff. It should be migrated elsewhere out of Kinetics Toolkit.

CALIBRATION_MATRICES are now deprecated (with proper warning) until Jan 2023.


How to show a legend?

Hi, first of all, thanks for this great library.
My question is how can I show the legend? I'm trying to show pca results and I would need to show a legend for each label.
This is my code:
ipv.figure()
ipv.style.box_off()
for p in pca_df["tarjet"].unique():
    color = np.random.uniform(size=3)
    marker = np.random.choice(shapes)
    df_to_plot = pca_df[pca_df["tarjet"] == p]
    ipv.scatter(df_to_plot["pca1"].values, df_to_plot["pca2"].values, df_to_plot["pca3"].values, marker=marker, color=color)
ipv.show()

Thanks!

Hi Isaac,
thanks for you kind words. Currently there is no support for a legend. You could try building a simply one from ipywidgets, a VBox with a series of Labels
What do you think of that solution?
cheers,
Maarten
PS: would be cool to see a screenshot of what are doing! :)


[Style] Enhance the heading of Contact form

What's the style idea?

I will give it a darker effect and hover effect in styling.
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[x] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Go ahead @ayush-848


issue #81 Refactor and cleanup javabatch codes

Signed-off-by: Albert Wang<EMAIL_ADDRESS>(1) dropped process from export chunk job.
(2) used bufferstream directly for export instead of using a large string object as carrier.
(3) dropped org.json package from import job, and change to use javax.json instead.
(4) added file size limit support when exporting to multiple files in COS.
(5) Added exported COS file names to job exit status.
(6) Code cleanup and reorg.
will document the javabatch job per issue #94

made some changes per review comments from Paul

@albertwang-ibm does the COSClient work with Amazon S3 as well, or only IBM Cloud Object Storage?  Would be nice if we could work with any S3-compatible API.  That would open the door for running with things like https://min.io/ as well.

@lmsurpre I believe it works with both IBM COS and Amazon S3, actually the IBM COS JDK looks more like a Amazon S3 JDK. will open an issue to test on Amazon S3.

@lmsurpre made changes per your review comments


seqan3: add package

sdsl-lite
seqan3

ci compiler does not support full c++20

ci compiler does not support full c++20

you can disable some platform, e.g. ios, android, macos


feat: upgrade maxwell-utils to 0.7.0

upgrade maxwell-utils to 0.6.5 and support reading service config from environment

upgrade maxwell-utils to 0.7.0


Sma3

Sma base implementation. It only implements 30min prices (no confidence).
It is gated by cfg(test) flags while in development.

current_epoch_is_valid gets initialized to false now.


[Sample Onboard Request] Containerized Hello World Tab with Backend

Owner:
@hund030
ID:
hello-world-tab-docker
Title:
Containerized Hello World Tab with Backend
Description:
A containerized Hello World app of Microsoft Teams Tab app which has a backend service.
Tags:
TS
Tab
React
Graph Toolkit
Docker
Single Sign On
Time:
5mins to run
Configuration:
Ready for local debug
Thumbnail:
https://github.com/OfficeDev/TeamsFx-Samples/blob/29099060655b8cca0cb4466bd1330243522b15b0/hello-world-tab-docker/assets/thumbnail.png
(Optional) Demo GIF:
https://github.com/OfficeDev/TeamsFx-Samples/blob/29099060655b8cca0cb4466bd1330243522b15b0/hello-world-tab-docker/assets/sampleDemo.gif
(Source code in this Repo) Source Code PR:
https://github.com/OfficeDev/TeamsFx-Samples/pull/1167
(Optional) Minimum Teams Toolkit Version:
V5.0.0
(Optional) Maximum Teams Toolkit Version:
N/A

Reviewed README， please go ahead with testing.

@hund030 , did this sample pass test?

@summzhan Not yet. Testers are too busy for prerelease.

@hund030 @summzhan This sample has been onboarded.


[FEATURE]: [Airwallex] Currency Unit Conversion

:memo: Feature Description

Each currency can be described in terms of base or minor units.
For instance, when dealing with USD currency, the Cent is considered as the minor unit, while the Dollar is considered the base unit.
In Hyperswitch, the amount value is expected to be always provided in minor units.
For example: In USD, If the amount is 1 in base unit (Dollar), then it will be equivalent to 100 in minor units (Cent)
Some of the connectors integrated require the amount to be converted to a desirable unit before being passed to them.
We have functions to_currency_base_unit and to_currency_lower_unit in place to convert the minor unit amount to its decimal equivalent and vice versa. These conversions are handled explicitly based on the connector.

:hammer: Possible Implementation

ConnectorCommon trait have been implemented for the connector.
This trait contains get_currency_unit method. This method needs to be implemented.
It will define what type of conversion needs to be done during connector_router_data creation. Concurrently handle the creation of connector_router_data to be passed for the connector request body creation.
You can check this PR for further reference https://github.com/juspay/hyperswitch/pull/2196

:bookmark: Note: All the changes needed should be contained within hyperswitch/crates/router/src/connector/
:package: Have you spent some time checking if this feature request has been raised before?

[X] I checked and didn't find a similar issue

:package: Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

:sparkles: Are you willing to submit a PR?

Hey, I'd like to work on this!

Hey @SagarDevAchar, sure! I've assigned this to you.
In case you have any queries, you can ask them on this issue thread, or on our discord server, or on slack whichever you are comfortable with

Hey @AkshayaFoiger
I have made some modifications to the codebase with the assumption that the amount parameter is a String in the newly implemented AirwallexRouterData<T> struct. After going through the Airwallex PaymentIntent Docs, I am confused whether the amount field should be i64 or f32 or String.
Can you help me on this?
You can find my current implementation here


Dropdown should consider iOS safe area for opening upwards

Currently it uses window edge which cause tapping the item on the edge bring up the safari bottom bar
https://gfycat.com/beautifulpeskycamel

Closing as stale, and as something I think would be handled by Popper instead of our own code?


Project URL changed for Choices.js

Platform
Wappalyzer website
https://www.wappalyzer.com
Describe the bug
There is an invalid link for specific technology - Choices js library.
To Reproduce
Steps to reproduce the behavior:

Go to https://www.wappalyzer.com/technologies/miscellaneous/choices
Click on "Visit joshuajohnson.co.uk" button
There is an 404 page at folowing address: https://joshuajohnson.co.uk/Choices/

To fix
New project URL is: https://joshuajohnson.co.uk/choices-js/

Thanks!


ClassNotFoundException on JSON deserialization when the bean has an inner class

Describe the bug
ClassNotFoundException on JSON deserialization when the bean has an inner class
Actual behavior

Steps to reproduce
Run the test function
Environment

SOFARPC version: 5.9.1
JVM version (e.g. java -version): 1.8.0_291
OS version (e.g. uname -a): macOS 13.1
Maven version: 3.9.0
IDE version: IntelliJ IDEA 2020.1 (Ultimate Edition)

I find this when I'm trying to fix #1254, and I'll fix both of them


Glow - A Versatile Flow Development DSL in Golang

Glow
Grant category
Please select one or more of:

[ ] Open source maintenance
[x] Developer tools / services
[ ] Educational material

Description
Flow development in Golang! A versatile DSL to test your contracts/txns/scripts on Mainnet, Testnet or the embedded emulator.
Problem statement
Bjartek's Overflow (https://github.com/bjartek/overflow) makes building, testing, and deploying smart contracts less intimidating and relatively straightforward; however, its opinionated structure and functional restrictiveness are constraining for developers who are working with multi-project repositories or need high levels of configurability. The Flow Go SDK (https://github.com/onflow/flow-go-sdk) provides more versatility but is quite verbose for developers who are trying to simply develop and test smart contracts. I believe that a third golang alternative, one that supports multiple project scopes and ease of use functions like creating disposable user accounts for testing on Testnet/Mainnet would not only be a welcomed addition to the ecosystem but a necessary one.
Proposed solution
An open source DSL for developing, testing, and deploying smart contracts called: Glow (https://github.com/rrossilli/glow). Please give the repo a quick run-through!
For almost 2 years now I've helped NFT Genius develop, test, and deploy NFT projects to their curated marketplace, Gaia (https://ongaia.com/). During this time, I was also building a passion project in my free time, Glow. This library has helped us build and deploy numerous projects to the Mainnet. With each project, I was able to gain new perspective and hone the library to fit a Cadence developer's needs.
Milestones and funding

Milestone
Deliverables
Timeline
USD Proposal

1 - Update
It has been about 9 months since Glow received much love - and we know a lot has changed. We'll start by getting antiquated with the most recent changes/standards and updating all of the libraries. Since we don't know what we don't know, it is difficult to estimate how long this will take. We'll make sure to update the flow team provided that this takes less time than anticipated. In such a case we can either decrease the budgeted amount or allocate the extra time elsewhere.
~2 weeks
~$4,000

2 - Code Refactoring
We've proven that Glow is lightweight and versatile but it can be better. We'll spend some quality time in the bowels of the codebase to make sure everything is neat and tidy so that the code is readable and inspires contribution. We'll also add some commonly used helper functions that provide useful shorthand for developers, implement a reliable way of specifying a configuration (similar to a hardhat.config.json), and build out a "wrapped client" that allows for method chaining - something highly requested by other flow developers. Furthermore, we'll build build out 100% test coverage of Glow features and implementation examples so that we can maintain confidence in the codebase after each update.
3 Weeks
$6,000

3 - Documentation/Tutorials
A good library is nothing without clear and thorough documentation. We will spend a good chunk of time writing thorough docs with plenty of examples.
1 week
$2,000

4 - Social Media Promotion
Sharing more information via social media posts: tutorials, medium articles, flow forum, etc... to bring more attention to Glow
1 week
$1,000

Team

Name
Role
Relevant Background
Contact

Robert Rossilli
Lead Developer
Cadence developer for NFT Genius, wrote contracts for NFL All Day and numerous NFT projects for Gaia. (https://ongaia.com/, https://nflallday.com)
<EMAIL_ADDRESS>

Final Thoughts
Glow was my passion project for months but I had to stop working on it when I was pulled in a different direction. I want to maintain it but am left with little time to do so these days. Any financial support would allow me to continue development and maintenance of the codebase. I want nothing more than to get Glow in the hands of some salacious Flowsters like myself and inspire further development through open source collaboration!

HI @rrossilli - we took a look at the proposal and we think it looks good. We would like to suggest that you add another milestone - #4 - where you spend time promoting and sharing this on social media, through blog articles etc so that we can make sure there's a focus and attention being brought to this. Could you update this with the suggested change?

That's a great idea - updated! Let me know if there are any other changes you'd like me to make

@franklywatson any update on this?

Hi @rrossilli we'll be sending out a grant letter for this later this week. Thanks

@franklywatson perfect, thanks!

From discussing this a bit we would more specifically like understand the rough task breakdown for milestones 1 & 2. Would be most helpful to see them articulated as issues in the Glow repo. Is that something you can provide?

@franklywatson I also updated the first 2 milestones to be a bit more descriptive

@franklywatson milestone 1 is complete.
merged PR:
https://github.com/rrossilli/glow/commit/fb1e5a4461560c34de24c0162b95e7c624e2ebcc
issue #15, #20:
https://github.com/users/rrossilli/projects/1

Thanks, have reviewed and marked as accepted

@franklywatson just checking in, any idea when I'll be contacted about this?

@rrossilli are you asking when the milestone will be paid? It will be this Friday

@franklywatson great! appreciate the update

Can you elaborate on this please?

its opinionated structure and functional restrictiveness are constraining for developers who are working with multi-project repositories or need high levels of configurability.

Can you elaborate on this please?

its opinionated structure and functional restrictiveness are constraining for developers who are working with multi-project repositories or need high levels of configurability.

Howdy sir! Nice to meet ya - big fan. So it looks like I may have been mistaken here, that assertion was most likely based upon my experience with your library when I used it last (quite a while ago). I'm assuming that the ScriptBasePath/TransactionBasePath constructor args - among other notable changes -  would allow overflow developers to organize multiple project scopes into a single monorepo. Is that assumption correct?

Hey, thank you for your kind words.
I think you should be able to do what you are asking but I am not sure. You could always make multiple overflow clients with different base paths for the different parts of your mono repo. It is also totally possible to override the base path for a single interaction (script/transaction)i belive, if not that builder option is really easy to add.

You could also use go:emed and just embed all your scripts/transactions/flow.json from the library and use it in the other places. See here for a naive example
client: https://github.com/bjartek/embed-test-client/blob/main/main.go
server: https://github.com/bjartek/embed-test/blob/main/foo/main.go

@robbie-wasabi just following up on the status of the remaining deliverables, noticed it's been over a year but haven't seen any updates on this thread.

@robbie-wasabi just following up on the status of the remaining deliverables, noticed it's been over a year but haven't seen any updates on this thread.

Hey there, thanks for the reply. Being honest, a lot of what I spoke about in my initial milestones was accomplishable a lot easier than I expected and I simply didn't feel comfortable collecting the remaining $9,000 to write some documentation and rewrite/bloat code that I believe is already quite simplified and elegant. I spent quite a bit of time adding more extensibility to the DSL but decided against committing it as I want to keep it lightweight and simple not only out of principle but to diversify my framework further from Bjartek's. Anyways, I should have been more communicative for certain and will do my best going forward.

As some sort of quantifiable update, I did push my wip changes that include some general cleanup and quite a bit of comments and documentation.

Amazing and happy to hear @robbie-wasabi, I'll close this one down :)


Having issue fetching AWS data and connecting to data

I have followed the documentation written here https://www.cloudcarbonfootprint.org/docs/aws/
I am trying to run yarn start-api to test out to see if my application will retrieve any details from Athena where following .env files are setup:
AWS_ATHENA_DB_NAME=CURDATA
AWS_ATHENA_DB_TABLE=CURTABLE
AWS_ATHENA_REGION=us-east-1
I am running command "yarn start-api" once API is up an running I am using http://localhost:4000/docs and then api/footprint
I am getting following error:
$ yarn start-api
[INFO] 13:44:23 ts-node-dev ver. 2.0.0 (using ts-node ver. 10.8.2, typescript ver. 4.7.4)
2024-06-11T12:44:35.680Z [Server] info: Cloud Carbon Footprint Server listening at http://localhost:4000
2024-06-11T12:44:35.683Z [server] info: Documentation available at http://localhost:4000/docs
2024-06-11T12:46:20.858Z [api] info: Footprint API request started.
2024-06-11T12:46:20.863Z [Cache] info: Using local cache file...
2024-06-11T12:46:20.870Z [App] info: Starting AWS Estimations
2024-06-11T12:47:53.226Z [CostAndUsageReports] error: Error verifying schema for Athena table: "CURTABLE"AggregateError:
at internalConnectMultiple (node:net:1114:18)
at afterConnectMultiple (node:net:1667:5)
2024-06-11T12:47:53.226Z [CostAndUsageReports] warn: 'product_vcpu' column could not be verified in Athena table schema. This may occur if there was an error fetching the schema or when there is no historical CPU usage (i.e. EC2) for the configured account. The CPU column will be excluded from Athena Query
2024-06-11T12:49:25.326Z [api] error: Unable to process footprint request.Error: Athena start query failed. Reason null.
Can you let me know what is wrong? I can see in athena this database and table. What DataCatalog are you using for query? Where can I see what query is being executed as I do not see it in athena coming through

@Denjaa Thanks for opening this issue! I can see there being an opportunity to make our documentation more specific or ti improve our error logging here to make what's happening more clear.
By the look of the logs reporting an issue with validating the schema and executing the query, it looks like the app is having trouble connecting to your Athena table. This is probably why you aren't seeing the query come in as it's never making it to the step to execute due to not being able to connect to the account or service.
I would recommend double-checking your .env config and ensuring that the DB name and table variables match the actual name of your Athena DB and the CUR table. In addition, make sure you have the other variables such as AWS_BILLING_ACCOUNT_ID and AWS_BILLING_ACCOUNT_NAME set to the right values so that CCF can connect to your account. Let me know if that helps

@Denjaa Thanks for opening this issue! I can see there being an opportunity to make our documentation more specific or ti improve our error logging here to make what's happening more clear.
By the look of the logs reporting an issue with validating the schema and executing the query, it looks like the app is having trouble connecting to your Athena table. This is probably why you aren't seeing the query come in as it's never making it to the step to execute due to not being able to connect to the account or service.
I would recommend double-checking your .env config and ensuring that the DB name and table variables match the actual name of your Athena DB and the CUR table. In addition, make sure you have the other variables such as AWS_BILLING_ACCOUNT_ID and AWS_BILLING_ACCOUNT_NAME set to the right values so that CCF can connect to your account. Let me know if that helps

I have provided correct details to .env file.
I have tried to debug a code in api/src/middleware.ts for following part:
`try {
console.log('estimationRequest')
const estimationRequest = createValidFootprintRequest(rawRequest)
console.log(estimationRequest)
console.log('estimationResults')
const estimationResults = await footprintApp.getCostAndEstimates(
estimationRequest,
)
console.log(estimationResults)
res.json(estimationResults)
} catch (e) {
apiLogger.error(Unable to process footprint request., e)`
I can see following
estimationRequest {   startDate: 2024-05-10T00:00:00.000Z,   endDate: 2024-05-10T00:00:00.000Z,   ignoreCache: false,   groupBy: 'day',   limit: 50000,   skip: 0,   cloudProviders: undefined,   accounts: undefined,   services: undefined,   regions: undefined,   tags: undefined } estimationResults 2024-06-20T07:33:50.774Z [Cache] info: Using local cache file... 2024-06-20T07:33:50.809Z [App] info: Starting AWS Estimations 2024-06-20T07:35:23.177Z [api] error: Unable to process footprint request.Error: Athena start query failed. Reason TimeoutError: null.
It seems that query is timing out somehow. I am also wondering why const rawRequest: FootprintEstimatesRawRequest is setting up a lot of values as undefined. What could be the cause of that?

I can also see results are correct when executing
new AWSAccount(
          AWS.BILLING_ACCOUNT_ID,
          AWS.BILLING_ACCOUNT_NAME,
          [AWS.ATHENA_REGION],
)

Sorry for the late reply. Any luck with additional debugging? @Denjaa

It seems that query is timing out somehow. I am also wondering why const rawRequest: FootprintEstimatesRawRequest is setting up a lot of values as undefined. What could be the cause of that?

Those undefined values are optional filters. If they are not passed as API params then they will be undefined which is expected behavior and should be unrelated to your issue.
If you are still experiencing errors, this log seems to stand out to me:
2024-06-11T12:47:53.226Z [CostAndUsageReports] error: Error verifying schema for Athena table: "CURTABLE"AggregateError: at internalConnectMultiple (node:net:1114:18)
This tells me that there are issues connecting to your Athena table (most likely config), there are issues with the table itself (i.e. no usage data, invalid columns). Fortunately, the schema validation seems like a warning and that there is no cpu usage but nothing breaking. However, I would recommend starting to debug there to see what response/data you're getting following that line. Alternatively, this could also be related to #1359


updated production.yml

Included minio in production workflow, initially missing causing tests to fail in production workflow.
Removed command value causing error.
Fixes #3

Error still exists


[Event Request] Report 10710 Make 349 Declaration [ES Localization]

Hello everyone,
in the past i´ve asked for a event on this function, just to skip functionality. Now i want to skip functionality but to return a value according to my custom comprobations.
Event Location
OnBeforeIncludeIn349(VATEntry,SkipEntry);
IF SkipEntry THEN
  EXIT(FALSE);

//-- EVENT LOCATION--
OnBeforeIncludeIn349Handled(VATEntry, AmountIn349, LocationDifferentCountryCode, IncludeIn349Result, IsHandled);
IF IsHandled THEN
  EXIT(IncludeIn349Result);
//++ EVENT LOCATION ++

AmountIn349 := 0;

Event Declaration
 LOCAL [IntegrationEvent] OnBeforeIncludeIn349Handled(VATEntry : Record "VAT Entry";VAR AmountIn349 : Decimal;VAR LocationDifferentCountryCode : Boolean;VAR IncludeIn349Result : Boolean;VAR IsHandled : Boolean)
This event doesnt affect the other one so you can mantain the two events, but this one would be enough.
Kind regards.

This event is related with #1018

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


etcdserver: fix recovering snapshot from disk

Reject applying entries that have been executed to v3 storage when restoring
snapshot from disk. Also, reject snapshot request when v2 store and v3
storage have not been applied to the same consistent index.
I have manually tested it and it fixes previous wrong recover problem.

Updated based on our discussion. It moves the logic of check applying twice into storage package to reduce the logic at apply loop.
I find treating all operations as nop is a little weird. If we apply a transaction with compare, it may execute in different path (success/failure) with different operations. But it doesn't affect the correctness because apply result is not used at all.

change the var name and reset skp var when TxnEnd. Moreover, adding one related test. PTAL

I find treating all operations as nop is a little weird.

I actually have an idea to solve this weirdness. But I think this is fine for now at least...

I am fine to let it work in this way now, and I think we should improve it. I will file an issue considering we agree on this.
Do you have any other comments for this PR?

LGTM


Featured Image Excluded From AMPforWP

Images inserted into blog posts as the Featured Image are excluded from AMP.
Was this by design? Other images that inserted as media into the post appear just fine.
Thanks.

Yes, this was intentional. But we will create an option in future to Show/Hide featured image in single post.
So users will have more control.

This was fixed in d1be130cec3de0122dc13a59ff89660c62045b0c


[embedded] Port and start building Darwin.swiftmodule as embedded

Building in embedded Swift mode is useful even for non-standalone environments/targets, like Darwin, at least for testing but also possibly for writing userspace library code that wants to get the no-runtime no-metadata guarantees on embedded Swift. Let's make a step towards that by porting and providing an embedded build of the Darwin overlay (Darwin.swiftmodule).

Is there a way I can download this toolchain built in CI and try it locally?

@rauhul  I think the swift-ci command I just made should produce a downloadable toolchain.

@swift-ci please test

@swift-ci please build toolchain macOS platform


[Bug] Seashelf of Aquatic Filtration has missing sprite

Mods List: latest.log
This bug has existed since last year.

This issue does not reproduce in isolation, which means you have another mod that is messing with the model. You will need to source the offending mod and report it to them.


Add xAppVersion and xAssosciatedBrand config variables

The American Standard Home service which is now live requires a couple of extra headers. It requires the X-AppVersion and X-AssociatedBrand header.
If a request is sent without those two headers, from a fully setup mobileId <-> apikey pair, American Standard will respond with an error:
{
    "success": false,
    "error": {
        "code": 5,
        "message": "This mobile device is not valid. We see you are trying to log in to Nexia™. We’ve updated your experience, and you’ll need to download American Standard® Home to log in. Need help? Call our customer support team at:<PHONE_NUMBER>",
        "additional_error_fields": {}
    },
    "result": null
}

After the above response the mobile device is removed from the account and API access is disabled.

If you're changing the name, you might want to get it in the readme as well.

Oops, going to close this for now while I test out my update and get it working.

For what it's worth, I installed it and got it working, and it seems to be good. :D

@kemayo Oh wow this is working good now for me... I must have messed something up in my config originally. Awesome! Glad it is working for you, will make a new PR with these changes. Going to leave my package up in the mean time.

Here is updated PR #11


cannot  load llama_cpp\llama.dll'  of the rocm wheels. but the installation is suceed.

2023-11-24 03:23:15 SORANET main[11664] WARNING Auth is disabled!
2023-11-24 03:23:15 SORANET main[11664] INFO Current server config: Server(listen: <IP_ADDRESS>:5000, auth: None:None)
2023-11-24 03:23:15 SORANET main[11664] INFO Current model config: SakuraModelConfig(model_name_or_path='ggml-model-4bit.gguf', use_gptq_model=False, trust_remote_code=True, text_length=512, llama=False, llama_cpp=True, use_gpu=True, n_gpu_layers=0, model_name=None, model_quant=None, model_version='0.8')
Traceback (most recent call last):
File "C:\Users\shing\AppData\Local\Programs\Python\Python311\Lib\site-packages\llama_cpp\llama_cpp.py", line 69, in _load_shared_library
return ctypes.CDLL(str(lib_path), **cdll_args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\shing\AppData\Local\Programs\Python\Python311\Lib\ctypes_init.py", line 376, in init
self._handle = _dlopen(self._name, mode)
^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: Could not find module 'C:\Users\shing\AppData\Local\Programs\Python\Python311\Lib\site-packages\llama_cpp\llama.dll' (or one of its dependencies). Try using the full path with constructor syntax.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "W:\llama\Sakura-13B-Galgame\server.py", line 122, in
state.init_model(cfg)
File "W:\llama\Sakura-13B-Galgame\utils\state.py", line 21, in init_model
sakura_model = SakuraModel(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "W:\llama\Sakura-13B-Galgame\utils\model.py", line 127, in init
(tokenizer, model) = load_model(cfg)
^^^^^^^^^^^^^^^
File "W:\llama\Sakura-13B-Galgame\utils\model.py", line 71, in load_model
from llama_cpp import Llama
File "C:\Users\shing\AppData\Local\Programs\Python\Python311\Lib\site-packages\llama_cpp_init_.py", line 1, in
from .llama_cpp import *
File "C:\Users\shing\AppData\Local\Programs\Python\Python311\Lib\site-packages\llama_cpp\llama_cpp.py", line 82, in
_lib = _load_shared_library(_lib_base_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\shing\AppData\Local\Programs\Python\Python311\Lib\site-packages\llama_cpp\llama_cpp.py", line 71, in _load_shared_library
raise RuntimeError(f"Failed to load shared library '{_lib_path}': {e}")
RuntimeError: Failed to load shared library 'C:\Users\shing\AppData\Local\Programs\Python\Python311\Lib\site-packages\llama_cpp\llama.dll': Could not find module 'C:\Users\shing\AppData\Local\Programs\Python\Python311\Lib\site-packages\llama_cpp\llama.dll' (or one of its dependencies). Try using the full path with constructor syntax.
What dependency do i missed? thanks.

Hard to say for certain. The main external requirement for for the ROCm version is ROCm itself.
Windows ROCm is here: https://www.amd.com/en/developer/rocm-hub/hip-sdk.html
GPU compatibility chart: https://rocm.docs.amd.com/en/docs-5.5.1/release/windows_support.html

Hard to say for certain. The main external requirement for the ROCm version is ROCm itself.
Windows ROCm is here: https://www.amd.com/en/developer/rocm-hub/hip-sdk.html GPU compatibility chart: https://rocm.docs.amd.com/en/docs-5.5.1/release/windows_support.html

Do Rocm require specific GPU drivers. I am using the gaming driver. that might be the issue. The SDK come with a bundle optional driver through

reinstall hip sdk with its driver as well, it does not work. I got a 7900XTX so i think hardware is compatible

I honestly don't know what the issue is. The error is simply too generic to know how to fix it. It is missing something that it requires, but that could be many things or it could simply be unable to find what it requires, which can be caused by many things.

I honestly don't know what the issue is. The error is simply too generic to know how to fix it. It is missing something that it requires, but that could be many things or it could simply be unable to find what it requires, which can be caused by many things.

I was able to get it working by adding Rocm/bin to the ENV, but the speed is extremely slow.
By replacing DLL that I compiled from source, I was able to run a 4bit 13B model at 55token/s on my 7900XTX. That's great.


meta: build flags for plugins?

I was thinking that as the number of plugins for balenaSound grows, it may be that rather than just turning off a plugin using SOUND_DISABLE_PLUGIN options in balenaCloud, it might be better to have it so that you can add some kind of build flags / options that stop the builder from building certain plugins that you don't need.
For example, by default it could add all plugins, but if you only needed the spotify and bluetooth plugins you could add --plugin spotify bluetooth or something like that. Or if you want to disable some plugins you could use --plugin-disable upnp roon for example.
This would have the added benefit of smaller image sizes being built and pulled down to the devices as well - meaning quicker builds and downloads/updates.
I am not sure if this is possible to do with flags, or perhaps via some environment variables on the application?

@shawaj i assume you can partially already exclude containers through the dockerignore file? I also think this should be forwarded to the cli github repository and maybe renamed to --exclude-containers? Here is a link to the cli: https://github.com/balena-io/balena-cli

@shawaj i assume you can partially already exclude containers through the dockerignore file? I also think this should be forwarded to the cli github repository and maybe renamed to --exclude-containers? Here is a link to the cli: https://github.com/balena-io/balena-cli

@AlexProgrammerDE yeah I agree I think this has wider use cases than just balenaSound so probably makes more sense in the CLI
Would be nice to somehow deal with this through the balenaCloud dashboard when using the deploy button as well. So you can do it all without touching the terminal at all.

@AlexProgrammerDE yeah I agree I think this has wider use cases than just balenaSound so probably makes more sense in the CLI
Would be nice to somehow deal with this through the balenaCloud dashboard when using the deploy button as well. So you can do it all without touching the terminal at all.

Yes! A growing concern of mine, as we get more plugins the project gets bigger and bigger, it takes more time to build, to deploy and for low end devices it could mean a performance impact. If I had to guess most people use one or maybe two plugins so we should probably do something about it. @shawaj I'll tag this for a labs brainstorm!

Yes! A growing concern of mine, as we get more plugins the project gets bigger and bigger, it takes more time to build, to deploy and for low end devices it could mean a performance impact. If I had to guess most people use one or maybe two plugins so we should probably do something about it. @shawaj I'll tag this for a labs brainstorm!

@tmigone I have a feeling I posted this somewhere in a FD thread about the CLI, but I can't seem to find it now. Will see if I can dig it out!

@tmigone I have a feeling I posted this somewhere in a FD thread about the CLI, but I can't seem to find it now. Will see if I can dig it out!

Superseded by https://github.com/balenalabs/balena-sound/issues/412


⚠ Bofrost.es has degraded performance

In 73df12a, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1527 ms

Resolved: Bofrost.es performance has improved in eeab05d after 14 minutes.


Tutorial on using multiple LLM calls using embedchain

🚀 The feature
tutorial on multiple LLM chain using embedchain
Motivation, pitch
this will enable developers to learn more

hey I would Like to be assigned this documentation work


[chore] Release 13.0.0

Includes:

https://github.com/firebase/firebase-admin-node/pull/2756
https://github.com/firebase/firebase-admin-node/pull/2466
https://github.com/firebase/firebase-admin-node/pull/2759
https://github.com/firebase/firebase-admin-node/pull/2760
https://github.com/firebase/firebase-admin-node/pull/2757

Does this release require any changes by end users if coming from an earlier version?

Does this release require any changes by end users if coming from an earlier version?

The changes to credentials handling shouldn't require any changes by end users. You can find the release notes with all the changes here.


Added KeyRepeat event.

KeyPress no longer repeats.

Closes https://github.com/emilk/egui/issues/2387.

Ah nice, that's a pretty elegant solution!

…but it needs cargo fmt

Did run cargo fmt & force pushed.
btw. nice didn't know about cargo fmt. Will use that for my projects as well.

you seem to have forgotten to push

Thanks


Facing Issue with Using in TypeScript Project,

Respected  @haversnail,
I am Using this library with Inquire in my Type Script Project But, Getting Some Error's.  Un-able to find its  @types/inquirer-date-prompt  & its any resource for Type Script Project.
I'mFacing Module has no default export Error in my tsc file .

My Dependecies in Pacage.json :
  "dependencies": {
    "@types/inquirer": "^9.0.7",
    "chalk": "^5.3.0",
    "inquirer": "^9.2.12",
    "inquirer-date-prompt": "^3.0.0"
 }

+1

+1 same issue.
Were you able to work around it somehow?


WRO-11485: Update package dependencies to fix security vulnerabilities - Agate

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[x] Documentation was verified or is not changed
[ ] UI test was passed or is not needed
[ ] Screenshot test was verified or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Updated dependencies for Agate.
Resolution
Dependencies now use the latest version.
Additional Considerations
Links
WRO-11485
Comments
Enact-DCO-1.0-Signed-off-by: Andrei Tirla<EMAIL_ADDRESS>

Please update package.json and npm-shrinkwrap.json in agate/samples/sampler, too.

Please update package.json and npm-shrinkwrap.json in agate/samples/sampler, too.

I have managed to update sampler dependencies but regarding all @storybook dependencies, they cannot be updated at the moment because some several webpack configurations must be made. They were handled in this PR https://github.com/enactjs/agate/pull/681

I checked again the PR and did the isomorphic build. looks ok


Terraform deployment issue: Terraform Output failed with Exit Code: null       Error Message: spawnSync /usr/bin/terraform ENOBUFS

Hi
I am using Terraform to deploy the infrastructure for the ADB2C service in Azure through an Azure DevOps CI/CD pipeline. Everything was functioning smoothly until recently, but now I'm encountering an issue. While the build pipelines run without any problems, the Terraform deployment in the release pipeline is failing during the Terraform Load Output task. I've even tried removing the .git folder from the published artifact in the build pipeline, but the issue persists.
Any assistance would be greatly appreciated.
##[error]Terraform Output failed with Exit Code: null
Error Message: spawnSync /usr/bin/terraform ENOBUFS
Regards
Syed Obaid.

Forgot to mention the size of terraform plan
printing size of plan
svc_d_hf_bld domain users 3607795 Nov  7 20:31 tfplan

Any update? Your kind help will be highly appreciated

This issue is due to the size of your plan output and the limitation of the buffer set by the azure-pipelines-task-lib
Mind you, this is set to 1024x1024x1024 by default
For me to fix this, it would require a PR here https://github.com/microsoft/azure-pipelines-task-lib/blob/7898b15726774c98a607a7e2d69898481caed201/node/toolrunner.ts#L594
Which Microsoft don't accept PR's for :(
Maybe you could raise an issue there and link it to this issue?
Ideally the IExecOptions would include the ability to set the buffer size (within limits ofc)


Add a CI check that tests that nix develop works as expected

A recent commit of mine (5dad24283f4d637dbc276ff833a954fe6c699f4c) broke this. See https://github.com/srid/haskell-template/pull/10

We are doing this in garnix, https://github.com/srid/haskell-template/blob/1aa9e81142161bc6eb313ee0bbc622fde85b3929/garnix.yaml#L5-L6


Tried to register two view with the same name RNCSafeAreaView

it shows in ReactNative version :0.65.0

It means you have installed same extension multiple times.

Running "npm dedupe" command solved my problem.

https://stackoverflow.com/questions/64931250/invariant-violation-tried-to-register-two-views-with-the-same-name-rncsafeareap


Allow custom CheckString file buffers

In doing so, close a loophole around the parsing of check string buffers; if they were files, the implementation assumes that there's a trailing newline that we can always walk to.
This implementation has to work around SR-3258 by exposing a less-than-desirable ExpressibleByStringLiteral conformance that exists solely for the #file default initializer.

⛵


Partition module implementing the documentation available in SPEC.md

Remaining:

[ ] Test that the hashes and encoding are the proper ones
[ ] Test that the algorithm is correct

Could you rebase it to master? There is a lot of unecessary noise in the diff

Done ;-)


Disable full paths in production bundle

What's the reason for enabling fullPaths?
b = browserify({
    entries: ['./client-js/index.js'],
    fullPaths: true
})

Disabling it would bring the bundle size down from 1.5 to 1.3 MB.

Ironically that was in to be able to see the module sizes within the
bundle. It can safely be removed.
On Thu, Dec 15, 2016 at 4:41 AM, Vincent Weevers<EMAIL_ADDRESS>wrote:

What's the reason for enabling fullPaths?
b = browserify({
entries: ['./client-js/index.js'],
fullPaths: true
})
Disabling it would bring the bundle size down from 1.5 to 1.3 MB.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rickbergfalk/sqlpad/issues/147, or mute the thread
https://github.com/notifications/unsubscribe-auth/AASjXg5CC05M3BpFBXSgMF5dv-iOc9DAks5rIRlIgaJpZM4LN84p
.


Can't get it working

Have tried to implement in a new project, as well as cloned this and ran the dummy. No matter what I do, the page still wants to refresh. I figure it has to be something environmental about my machine.  Where should I start troubleshooting?

I'm on macOS Sierra, running Ember 2.9.
I was able to work through this a little bit, but haven't completed resolved it. I have completely resolved it for the cloned app and dummy. Watchman seems to somehow be involved. I was able to uninstall and reinstall watchman and hot-reload job followed. I was also able to get this working on a new project from scratch. On two existing projects that I'm working on, no such success. My next step will be to clone the problematic projects to a fresh VM or another laptop that I have which is a clean install.

@victornamuso thanks for taking a run at this to provide more detail. I don't use watchman myself these days but maybe I should :)
When you provide some additional info it may help us identify an issue/or at the very least add something to the README so others are aware of the incompatibility


Fix close button for rules editing

The close button is disabled for the rules editing element when looking at something from compendium, creating a scenario in which the user cannot navigate backwards.
This is done on foundry's side, so this PR attempts to remedy that by explicitly removing the disabled attribute from the close button.

Amended code to adhere to linter config

Amended code to adhere to linter config

Not the way I was imagining, but not a wrong way either. Sure! Thanks for the PR.


pluto: configure backups

sets the mountpoints for prometheus and victoriametrics dataset
copies the backup module from non-critical-infra
prepare the backup to the storagebox
configure backups to include prometheus, grafana, victoriametrics

Untested.

Deployed and tested. I think the zfs integration in the backup module provides sufficient decoupling for now.


ENH: estimate condition number for sparse matrices

Reference issue
Closes gh-18969
What does this implement/fix?
Add

scipy.sparse.linalg.SuperLU.rinvnormest to compute reciprocal norm of inverse matrix given an LU decomposition,
scipy.sparse.linalg.rinvnormest to compute LU decomposition and call the function mentioned previously,
scipy.sparse.linalg.cond1est to compute the 1-norm condition number, calling the function mentioned previously  and scipy.sparse.linalg.onenormest

See https://github.com/scipy/scipy/issues/18969 for previous (and not yet completed) discussions on the API.
Also, otvam proposed another implementation on the Scientific Python Discourse forum that does not use SuperLU

I think the terminology is going a bit different here. The condition number is defined as $\kappa(A) = \left|A\right|\cdot  \left|A^{-1}\right|$ with respect to inversion and not the ratio of these quantities. And gscon returns $1/\kappa$ hence the name rcondest with "reciprocal" of the estimate. Thus it is the inverse of cond1est.
So they are inverse of one another. I'm not sure the docstrings are also correct in that regard. onenormest proposed in Discourse is already in sparse.linalg and also another implementation was in dense linalg until recently https://github.com/scipy/scipy/blob/867638ec7ba23d8b2f27f5246a0d512ed54ebc41/scipy/linalg/_matfuncs_expm.pyx.in#L495-L628 which can be rearranged to use matvecs and rmatvecs only.

I've added non-zero imaginary part to the complex64 and complex128 testcases in 7e33770 and since then the complex testcases fail. I need to investigate this further.

I think the terminology is going a bit different here.

I'm not sure which changes would be required. The new cond1est computes onenormest(A)/rinvnormest(A), which is |A| divided by 1/|A^-1|

Here's a performance plot for 500x500 np.float64 matrices, produced with  performance.py.txt on a 4-core system:

For more dense matrices (200x200 np.float64), the performance advantage is not as big:

dgscon takes an input argument anorm, which is the norm of the matrix A (not inverted). In our implementation of rinvnormest, we set anorm to 1.0. I think it would be a weird API to have the scipy sparse cond1est function ask for the norm of A as an input argument; it appears cleaner to have cond1est compute the norm of A and multiply by it.

Yes, you are right. And that's why typically, you slap a call to lacon2  before the gscon call to get an estimate (the un-SAVEd version of lacon). Basically SuperLU is following the LAPACK model of getting the condition number with the difference that lacon2 only needs rmatvev/matvec operation for the estimate.

Interesting, I wasn't aware of lacon2 and currently use SciPy's own onenormest. Are you suggesting one of these:

wrap SuperLU's lacon2 to Python and use it in cond1est (and potentially make it part of the public API)
call lacon2 in the C++ code and offer cond1est directly instead of rinvnormest
or something else

or are you ok with the current version of the API?

Yes the story is a bit convoluted. The one norm estimation originally started from the need of matrix exponential. When the original expm was being written it is written for both sparse and the dense versions. In that algorithm, you need to investigate the one-norm of an array repeatedly hence for very large arrays 1-norm estimation (instead of calculating exactly) becomes very important. Then we rewrote the dense version independently and broke that relationship between sparse and dense expm.
Since you are wrapping SuperLU within C, I think lacon2 way is more ergonomic for your current way of doing it, that is to say just call lacon2 inside the function. A similar thing is also happening in the dense expm, but obviously here I use gemv since the dense array is available;
https://github.com/scipy/scipy/blob/e7c89a7f2764d34a465b18d3121f4733b0ff0c8f/scipy/linalg/_matfuncs_expm.c#L258-L287
But I don't know if you have access to matvec/rmatvec methods at C level. If that is not a problem indeed number 2 is much simpler in my opinion.
For number one, I think onenormest is good enough for folks historically speaking. But as you rightly pointed out passing the norm value back to C-level seems a bit awkward in terms of public API. So I agree that's a bit limping.


Support for Affects Version/s field

Hi,
I'm trying to implement support for the affect version/s field, but I'm hitting a wall.
I tried to do it the same way that fix versions is implementend, in Issue.cs:
`
public List AffectsVersions { get; set; }
        AffectsVersions = new List<ProjectVersion>();
        if (fields.ContainsKey("versions") && fields["versions"] != null)
        {
            JArray versionArray = (JArray)fields["versions"];
            if (versionArray.Count > 0)
            {
                AffectsVersions = ((JObject)fields["versions"]).ToObject<List<ProjectVersion>>();
            }
        }

`
But I get an error when I run this version.
I don't know much about this library or the JSON format used for the versions field, so I'm pretty much stuck here.
Would be grateful for any ideas on how to do it.

Figured it out. Its supposed to be
          public List<ProjectVersion> AffectsVersions { get; set; }

          AffectsVersions = new List<ProjectVersion>();
          if (fields.ContainsKey("versions") && fields["versions"] != null)
          {
              JArray versionArray = (JArray)fields["versions"];
              if (versionArray.Count > 0)
              {
                  AffectsVersions = ((JArray)fields["versions"]).ToObject<List<ProjectVersion>>();
              }
          }


Could not find a version that satisfies the requirement vtk<9.1.0

I had been getting an error message when trying to add a scale bar to one of my plots:

Traceback (most recent call last):
File "/Users/plelievre/Work/Research/Current/FloodRiskManagement/Code/Python/show3dinvslice_x.py", line 25, in
box_msh.addScalarBar3D(title = 'Resistivity (\Omegam)', titleSize = 3, titleRotation = 180, titleXOffset = 2, labelSize = 3, pos = (17, 4.3, 9), s = (1, 8.), drawBox = True)
TypeError: BaseActor.addScalarBar3D() got an unexpected keyword argument 's'

To me that looked like perhaps the 's' argument was a newer option, and I realized that I was running an older version of vedo, so I tried updating to the most recent version of vedo using:

pip install -U git+https://github.com/marcomusy/vedo.git

I'm on macOS 12.5.1 (Monterey) and running a couple versions of python. I have no problem on python 3.7.11 + latest version of vedo. However, python 3.10.6 complains when I try to install the latest version of vedo as above. The error message is:

ERROR: Could not find a version that satisfies the requirement vtk<9.1.0 (from vedo) (from versions: 9.2.0rc1, 9.2.0rc2)
ERROR: No matching distribution found for vtk<9.1.0

hi, vedo works best with vtk9.1 but it also works ok with vtk9.2, you can override it with
pip install vedo -U --no-deps


add DisplayLinkUpdateActiveMonitor to MTLGraphicsDevice for updating CVDisplayLink current display

This allows a user of the renderer to update the CVDisplayLink's current display by passing the active display's origin coordinates and size.
This is needed to correctly set CVDisplayLink's target refresh rate when:

the window is moved across monitors
CVDisplayLinkCreateWithActiveCGDisplays (at Veldrid.MetalBindings/CVDisplayLink.cs#L21) doesn't return a CVDisplayLink set to the correct refresh rate; I've noticed this is especially prevalent when the high refresh rate monitor is mirrored to another lower refresh rate monitor, or in the case where the main monitor is 60Hz but a game using Veldrid is being run on a different high refresh rate monitor.

I'm using origin and size because it's the only relatively straightforward way of finding the correct CGDirectDisplayID for calling CVDisplayLinkSetCurrentCGDisplay with (that I could find).
I feel that the way things are laid out (having such a deep call stack until actually calling CVDisplayLinkSetCurrentCGDisplay) is not great, but I'm not sure how else this should be laid out. I wouldn't mind if this code is completely thrown out in favour of something else, but at least this code should give an idea of what needs to be done.

Looks good to me, is this ready for review?

I've tested this code and it seems to hang for me:

@ILW8 did you manage to make this work with the framework diff you provided?

Adding this line also works, but I'm not sure that's ideal (I don't really know what this does).
index 478a502..56de90b 100644
--- a/src/Veldrid/MTL/MTLGraphicsDevice.cs
+++ b/src/Veldrid/MTL/MTLGraphicsDevice.cs
@@ -62,6 +62,7 @@ namespace Veldrid.MTL
         {
             if (_displayLink != null)
             {
+                _frameEndedEvent.Set();
                 _displayLink.UpdateActiveMonitor(x, y, w, h);
             }
         }


Update .npmignore file

This changes the "unpacked size" of the package from 2.2 MB to 194.9 kB

@locks I totally agree, except: https://github.com/npm/npm/issues/5673

@marcoow I don't understand, can you elaborate how that is relevant for the .npmignore file? are you confusing this with .gitignore?

There's https://github.com/simplabs/ember-simple-auth/issues/1339#issuecomment-306852493 for example - I really don't remember all of the details and would have to dig through the issue history…

also see #1482

😩


bug: WalletConnectLegacyConnector used wrong RPC url

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
@wagmi/core - 0.10.2
Current Behavior
When connecting a wallet (in my case Metamask) on mobile using the WalletConnect legacy connector and performing any sort of transaction - a request will be made to https://mainnet.infura.io/v3/undefined and fail. You can see in the code below, the only chain that was defined was polygonMumbai.
I've found a workaround using the following:
const walletConnectProvider = await client.connector?.getProvider()
const chainId = await client.connector.getChainId()
walletConnectorProvider.http = walletConnectProvider.setHttpProvider(chainId)

This seems to fix it and requests now go to https://matic-mumbai.chainstacklabs.com.
All of this was done using the publicProvider(). Unfortunately, there's a related second issue where if instead of the publicProvider, I use the alchemyProvider() with a proper api key, the request still goes to chainstacklabs.
Expected Behavior
I expect the request to go to the correct RPC url depending on the chain & providers defined in configureChains without the need of a workaround.
Steps To Reproduce
const chains = [polygonMumbai]

const { provider } = configureChains(chains, [
  publicProvider()
])

export const client = createClient({
  autoConnect: true,
  provider,
  connectors: [
    new WalletConnectLegacyConnector({
      chains,
      options: {},
    }),
  ],
})

await connect({ connector: client.connectors[0] })

// Do some action like sendTransaction or writeContract here

Anything else?
Related issues:
https://github.com/wagmi-dev/wagmi/issues/116
https://github.com/wagmi-dev/wagmi/issues/365

@KevinLu Thanks for the suggestion - I'll try it out!
Although, doesn't that kind of defeat the purpose of the alchemy / infura providers?

I think the issue here is that the wallet you are connecting to does not support the Polygon Mumbai network, so WalletConnect falls back to Mainnet (which it cannot find an RPC URL for as it is not in your configured chains). However, I believe it should fall back to the unsupported chain's RPC URL as public RPC methods should still work (I'd expect it to use that over mainnet anyway.
Fix at #196.

@jxom - I'm not sure if you might have missed it, but I mention that I'm using MetaMask which does support Polygon Mumbai

Looks like WC is doing something else strange then :D

@jxom - Just saw your PR -  the workaround does fix the mainnet issue, but using the alchemy or infura provider still doesn't work as expected and will still go to the public RPC url rather than alchemy or infura URLs.
This issue should probably remain opened.

Seeing this issue as well. WalletConnectLegacyConnector always uses the default/public RPC url regardless of the provider configuration.

Same problem here:
"Using the alchemy or infura provider still doesn't work as expected and will hit the public RPC url rather than alchemy or infura's URL."


args_sizes_get has different interface

Hello. First of all, thanks for the great project.
I found that in this repo (and in deno/std/wasi which is deprecated and removed), the interface of args_size_get is args_sizes_get(argc: number, argv_buf_size: number): number. However, the standard of wasi has the interface as args_sizes_get() -> Result<(size, size), errno>
May I ask why there's the difference?

Result and tuples are not actually representable in webassembly. What ends up happening is that behind the scenes a signature like () -> Result<(size, size), errno> is lowered into (i32, i32) -> i32. In case of a success the size values are written to the memory pointed to by the pointers passed to args_sizes_get as i32 and 0 is returned from the function. In case of an error, the actual error number is returned instead.
See also https://docs.rs/wasi/latest/src/wasi/lib_generated.rs.html#1219-1237 where the user facing args_sizes_get() -> Result<(size, size), errno> is lowered into a call to args_sizes_get(i32, i32) -> i32.

Thank you very much for your kind explanation.
在 2023年12月13日，01:05，bjorn3 @.***> 写道：

I'm going to close this as I assume my preview comment cleared up the confusion you had. If not, please let me know.
—
Reply to this email directly, view it on GitHubhttps://github.com/bjorn3/browser_wasi_shim/issues/59#issuecomment-1852450343, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD7U6XLAP7SWU3YCPOBRLB3YJCFFRAVCNFSM6AAAAABARP7VCOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJSGQ2TAMZUGM.
You are receiving this because you authored the thread.Message ID: @.***>


Parser: don't save returnStmt result

Fixes #272

It would be nice to have that saveValue there for return statements with more complex constant expressions. One way to archive that would be to copy the result as is done in constExpr or add a new saveValueExtra which doesn't invalidate the result value and implement saveValue with that.

Did you see my follow-up comment here https://github.com/Vexu/arocc/issues/272#issuecomment-1093076303
The signExtend function in Value.zig assumes that integer values will be promoted to int, which doesn't happen for char return values. So if signExtend is allowed to operate on 1 and 2 byte types that would also solve the problem without having to change the parser.

I did see that but didn't realize signExtend was the problem. Thinking about it again, copying the value doesn't make any sense since it isn't used after the return statement.

So if signExtend is allowed to operate on 1 and 2 byte types that would also solve the problem without having to change the parser.

That seems reasonable, the function was written with mostly expressions in mind in which the assumption holds true.

Just to clarify - should I add 1 and 2-byte integer support to Value.signExtend? And keep the change that removes the saveValue for return statements? Or should I revert that part.

Just the signExtend part, saving the return value is desired.


chore: Add run to makefile

Add run command to makefile

Current dependencies on/for this PR:

main

PR #67   👈

PR #68

This comment was auto-generated by Graphite.


Added checking before assignment to window

@megawac The changes are in the first line and the last line, rest are indentation changes.
Please check this. #34

Ah that makes sense, good catch @siawo


Magic Behavior Bug. (IndexError: pop from empty list)

Description of the Bug
Magic Behavior example crashes with Exception IndexError
Code and Logs
from kivy.lang import Builder

from kivymd.app import MDApp

KV = '''
<MagicButton@MagicBehavior+MDRectangleFlatButton>

MDFloatLayout:

    MagicButton:
        text: "WOBBLE EFFECT"
        on_release: self.wobble()
        pos_hint: {"center_x": .5, "center_y": .3}

    MagicButton:
        text: "GROW EFFECT"
        on_release: self.grow()
        pos_hint: {"center_x": .5, "center_y": .4}

    MagicButton:
        text: "SHAKE EFFECT"
        on_release: self.shake()
        pos_hint: {"center_x": .5, "center_y": .5}

    MagicButton:
        text: "TWIST EFFECT"
        on_release: self.twist()
        pos_hint: {"center_x": .5, "center_y": .6}

    MagicButton:
        text: "SHRINK EFFECT"
        on_release: self.shrink()
        pos_hint: {"center_x": .5, "center_y": .7}
'''

class Example(MDApp):
    def build(self):
        return Builder.load_string(KV)

Example().run()

Log
 Exception ignored in: 'kivy.graphics.instructions.InstructionGroup.build'
 Traceback (most recent call last):
   File "kivy\graphics\compiler.pyx", line 100, in kivy.graphics.compiler.GraphicsCompiler.compile
   File "kivy\graphics\instructions.pyx", line 271, in kivy.graphics.instructions.ContextInstruction.apply
   File "kivy\graphics\instructions.pyx", line 863, in kivy.graphics.instructions.RenderContext.pop_states
   File "kivy\graphics\instructions.pyx", line 855, in kivy.graphics.instructions.RenderContext.pop_state
 IndexError: pop from empty list
[INFO   ] [Base        ] Leaving application in progress...
 Traceback (most recent call last):
   File "test.py", line 43, in <module>
     Example().run()
   File "C:\Users\gogopro\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\kivy\app.py", line 955, in run
     runTouchApp()
   File "C:\Users\gogopro\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\kivy\base.py", line 574, in runTouchApp
     EventLoop.mainloop()
   File "C:\Users\gogopro\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\kivy\base.py", line 339, in mainloop
     self.idle()
   File "C:\Users\gogopro\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\kivy\base.py", line 400, in idle
     window.dispatch('on_draw')
   File "kivy\_event.pyx", line 731, in kivy._event.EventDispatcher.dispatch
   File "C:\Users\gogopro\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\kivy\core\window\__init__.py", line 1627, in on_draw
     self.render_context.draw()
   File "kivy\graphics\instructions.pyx", line 637, in kivy.graphics.instructions.Canvas.draw
   File "kivy\graphics\instructions.pyx", line 640, in kivy.graphics.instructions.Canvas.draw
   File "kivy\graphics\instructions.pyx", line 910, in kivy.graphics.instructions.RenderContext.apply
   File "kivy\graphics\instructions.pyx", line 863, in kivy.graphics.instructions.RenderContext.pop_states
   File "kivy\graphics\instructions.pyx", line 855, in kivy.graphics.instructions.RenderContext.pop_state
 IndexError: pop from empty list

Versions

OS: Windows 10 64 bit
Python: 3.8.10
Kivy: 2.1.0
KivyMD: 0.104.2

@gogopro-dev This is fixed in the master branch of the KivyMD library


Update build process with doc publishing

LGTM after tests pass

Codecov Report

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

@@            Coverage Diff            @@
##             master      #13   +/-   ##
=========================================
  Coverage     56.61%   56.61%
  Complexity      126      126
=========================================
  Files            15       15
  Lines           590      590
  Branches         91       91
=========================================
  Hits            334      334
  Misses          233      233
  Partials         23       23

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 66eab39...219bfde. Read the comment docs.


saving recordings in mp4

could you help me on how to save my recordings as mp4?
thanks heaps this project is ACE

Mp4 is the default extension, unless your camera has a certain audio codec which is not supported in mp4.
There should be a warning in the logs if this is the case, about switching to mkv instead.
Is this what's happening here?

Apologies I’m quite ignorant when it comes to codecs etc yes you sound
right I believe the camera stream is h264 and all recordings are mkv.
Am I able to add a config line to convert to mp4. I’m happy to drop audio
as I don’t use it.
Thank you again for response
On Tue, 14 Feb 2023 at 3:02 am, Jesper @.***> wrote:

Mp4 is the default extension, unless your camera has a certain audio codec
which is not supported in mp4.
There should be a warning in the logs if this is the case, about switching
to mkv instead.
Is this what's happening here?
—
Reply to this email directly, view it on GitHub
https://github.com/roflcoopter/viseron/issues/485#issuecomment-1428199297,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDRZMQD62HD2RM5A7SDKNTWXJLJ5ANCNFSM6AAAAAAU2D7JJE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Cheers,
Dom

Then it should be enough to just set audio codec: null for that camera

Awesome, thanks!
On Tue, 14 Feb 2023 at 5:35 pm, Jesper @.***> wrote:

Then it should be enough to just set audio codec: null for that camera
—
Reply to this email directly, view it on GitHub
https://github.com/roflcoopter/viseron/issues/485#issuecomment-1429196105,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDRZMWJLIJBFYXY7QRPAXDWXMRRPANCNFSM6AAAAAAU2D7JJE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Cheers,
Dom

after adding audio_codec: null it still looks like I am getting this in my log:
'Container mp4 does not support pcm_alaw audio codec, using mkv instead. Consider changing extension in your config'
any other suggestions?

Seems to be a bug here actually, until i fix it you should be able to circumvent it by specifying a few more config options.
You need to add fps, width, height, codec and audio_codec .
By doing this Viseron will not probe the stream for information and will instead use the supplied values from the config.
The problem now is that you set audio_codec, but Viseron still probes the stream and receives the pcm_alaw codec.

Awesome, thanks!
On Wed, 15 Feb 2023 at 2:17 am, Jesper @.***> wrote:

Seems to be a bug here actually, until i fix it you should be able to
circumvent it by specifying a few more config options.
You need to add fps, width, height, codec and audio_codec .
By doing this Viseron will not probe the stream for information and will
instead use the supplied values from the config.
The problem now is that you set audio_codec, but Viseron still probes the
stream and receives the pcm_alaw codec.
—
Reply to this email directly, view it on GitHub
https://github.com/roflcoopter/viseron/issues/485#issuecomment-1429913286,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDRZMX2YNTPTNU4TFCXHX3WXOOXRANCNFSM6AAAAAAU2D7JJE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Cheers,
Dom

all good, worked


New CropLike Layer

A CropLike layer should take two tensors as inputs, [origin, target], and crop the origin tensor so that it matches the shape of target.

cropped = CropLike(offset=(5, 5))([origin, target])

#13


Does work correctly with django permissions.

Hey, thanks for building this.
So, I noticed that if you have a order defined and a certain model is not in that order but the user has explicit django permission to see/change it, the model will show up regardless.
That's a bug, right?

Nevermind, looks like the user must have permission to access the models defined in that list of segregated models.


SQL/JPA 2.3.1 with PostgreSQL wrong Escape

2.3.1 introduced some new functionality regarding escaping.
This breaks queries for Postgresql 8.4.x/9.0.x.
So
EntityPathBase.someStringField.containsIgnoreCase(searchString)
produces SQL like:
select * from FOO foo where lower(foo.bar like ? escape '')
This does not execute within PostgreSQL, it needs double backslashes.

Which JPA provider are you using?

Hibernate 4.1

I will test it on monday morning, thanks for your quick reaction.

When using 2.3.1.BUILD-SNAPSHOT its working.
Thanks have a nice week.

I'll close this when it's released

released in 2.3.2


Error: RequestError [HttpError]: Unprocessable Entity: "Pull request review thread path is invalid and Pull request review thread diff hunk can't be blank"

There seems to be an error when submitting the review to github.
Error log:
Error: RequestError [HttpError]: Unprocessable Entity: "Pull request review thread path is invalid and Pull request review thread diff hunk can't be blank"
    at /home/runner/work/_actions/freeedcom/ai-codereviewer/main/webpack:/open-ai-reviewer/node_modules/@octokit/request/dist-node/index.js:7[8](https://github.com/LightConsultingInc/alphaedu-FE/actions/runs/7153530492/job/19480089954#step:3:9):1
    at processTicksAndRejections (node:internal/process/task_queues:[9](https://github.com/LightConsultingInc/alphaedu-FE/actions/runs/7153530492/job/19480089954#step:3:10)6:5) {
  status: 422,
  response: {
    url: 'https://api.github.com/repos/LightConsultingInc/alphaedu-FE/pulls/178/reviews',
    status: 422,
    headers: {
      'access-control-allow-origin': '*',
      'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset',
      connection: 'close',
      'content-length': '246',
      'content-security-policy': "default-src 'none'",
      'content-type': 'application/json; charset=utf-8',
      date: 'Sat, 09 Dec 2023 21:07:06 GMT',
      'referrer-policy': 'origin-when-cross-origin, strict-origin-when-cross-origin',
      server: 'GitHub.com',
      'strict-transport-security': 'max-age=31536000; includeSubdomains; preload',
      vary: 'Accept-Encoding, Accept, X-Requested-With',
      'x-accepted-github-permissions': 'pull_requests=write',
      'x-content-type-options': 'nosniff',
      'x-frame-options': 'deny',
      'x-github-api-version-selected': '2022-11-28',
      'x-github-media-type': 'github.v3; format=json',
      'x-github-request-id': '6CE0:9641:32C0EE5:3476A4E:6574D6FA',
      'x-ratelimit-limit': '[10](https://github.com/LightConsultingInc/alphaedu-FE/actions/runs/7153530492/job/19480089954#step:3:11)00',
      'x-ratelimit-remaining': '979',
      'x-ratelimit-reset': '1702158488',
      'x-ratelimit-resource': 'core',
      'x-ratelimit-used': '21',
      'x-xss-protection': '0'
    },
    data: {
      message: 'Unprocessable Entity',
      errors: [Array],
      documentation_url: 'https://docs.github.com/rest/pulls/reviews#create-a-review-for-a-pull-request'
    }
  },
  request: {
    method: 'POST',
    url: 'https://api.github.com/repos/LightConsultingInc/alphaedu-FE/pulls/178/reviews',
    headers: {
      accept: 'application/vnd.github.v3+json',
      'user-agent': 'octokit-rest.js/19.0.7 octokit-core.js/4.2.0 Node.js/[16](https://github.com/LightConsultingInc/alphaedu-FE/actions/runs/7153530492/job/19480089954#step:3:17).[20](https://github.com/LightConsultingInc/alphaedu-FE/actions/runs/7153530492/job/19480089954#step:3:21).2 (linux; x64)',
      authorization: 'token [REDACTED]',
      'content-type': 'application/json; charset=utf-8'
    },
    body: '{"comments":[{"body":"The `permissions: write-all` setting gives all actions run in this workflow write access to all the repository\'s contents. This is a high level of access and could be a security risk. Consider specifying more granular permissions to limit the access of the actions.","path":".github/workflows/pr.yml","line":8},{"body":"The `exclude` parameter is set to exclude all `.json` and `.md` files. If there is a need to review these types of files, consider removing them from the exclude list.","path":".github/workflows/pr.yml","line":[22](https://github.com/LightConsultingInc/alphaedu-FE/actions/runs/7153530492/job/19480089954#step:3:23)}],"event":"COMMENT"}',
    request: { hook: [Function: bound bound register] }
  }
}

Looks like this applys to any changes that happen inside the .github/* folder

Hi @zainlks, I had a similar issue that was resolved by forking the repository and then updating my workflow file to use the name of the forked repository instead of freeedcom/ai-codereviewer@main

Some error

I caught the same issue:
Run vvandriichuk/ai-codereviewer@main
Error: RequestError [HttpError]: Unprocessable Entity: "Pull request review thread path is invalid and Pull request review thread diff hunk can't be blank"
at /home/runner/work/_actions/vvandriichuk/ai-codereviewer/main/webpack:/open-ai-reviewer/node_modules/@octokit/request/dist-node/index.js:78:1
at processTicksAndRejections (node:internal/process/task_queues:96:5) {
status: 422,
response: {
url: 'https://api.github.com/repos/vvandriichuk/fastapi_db_template_02_2024/pulls/6/reviews',
status: 422,
headers: {
'access-control-allow-origin': '*',
'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset',
connection: 'close',
'content-length': '246',
'content-security-policy': "default-src 'none'",
'content-type': 'application/json; charset=utf-8',
date: 'Sat, 16 Mar 2024 13:17:17 GMT',
'referrer-policy': 'origin-when-cross-origin, strict-origin-when-cross-origin',
server: 'GitHub.com',
'strict-transport-security': 'max-age=31536000; includeSubdomains; preload',
vary: 'Accept-Encoding, Accept, X-Requested-With',
'x-accepted-github-permissions': 'pull_requests=write',
'x-content-type-options': 'nosniff',
'x-frame-options': 'deny',
'x-github-api-version-selected': '2022-11-28',
'x-github-media-type': 'github.v3; format=json',
'x-github-request-id': '1400:78F9:1843BAE:2DA7574:65F59BDB',
'x-ratelimit-limit': '5000',
'x-ratelimit-remaining': '4994',
'x-ratelimit-reset': '1710598299',
'x-ratelimit-resource': 'core',
'x-ratelimit-used': '6',
'x-xss-protection': '0'
},
data: {
message: 'Unprocessable Entity',
errors: [Array],
documentation_url: 'https://docs.github.com/rest/pulls/reviews#create-a-review-for-a-pull-request'
}
},
request: {
method: 'POST',
url: 'https://api.github.com/repos/vvandriichuk/fastapi_db_template_02_2024/pulls/6/reviews',
headers: {
accept: 'application/vnd.github.v3+json',
'user-agent': 'octokit-rest.js/19.0.7 octokit-core.js/4.2.0 Node.js/16.20.2 (linux; x64)',
authorization: 'token [REDACTED]',
'content-type': 'application/json; charset=utf-8'
},
body: '{"comments":[{"body":"The @staticmethod decorator is not necessary when using the @field_validator decorator from Pydantic. The @field_validator should be applied to instance methods, and the cls parameter should be ... {"body":"Ensure that the input s is of a type that has the .lower() method, such as str, to avoid AttributeError when a non-string type is passed.","path":"app/utils/str_to_bool.py","line":2}],"event":"COMMENT"}',
request: { hook: [Function: bound bound register] }
}
}
It happened when I fixed some issues in PR that AI Assistant commented and re-updated PR with fixed issues.

We got this when rebasing, maybe the action doesn't like that type of branch operation?


(chore): rename image means and stds with plural

Description
Small PR, we have image_means and image_stds throughout all the code except in the Pydantic model, so I added the plural back into the name.
Since it is a list, it makes sense to me.

Please ensure your PR meets the following requirements:

[x] Code builds and passes tests locally, including doctests
[x] New tests have been added (for bug fixes/features)
[x] Pre-commit passes
[ ] PR to the documentation exists (for bug fixes / features)

Requires https://github.com/CAREamics/careamics/pull/149 to be merged for tests to pass.


Clean up the manpages

The html manpages do not always render well, see e.g.
https://microbiome.github.io/mia/reference/getExperimentCrossAssociation.html

Do we have pending tasks to clean up rendering in manpages?

Could you for instance browse through all reference pages in mia, miaViz, miaTime and see if you notice anything that could be easily improved / clarified?
If examples throw warnings it might be good to either modify the example to get rid of warnings, or comment on why those warnings appear and show how to deal with them.
This task is secondary priority if more urgent tasks pop up.

mia and  miaViz  are addressed in #603 and https://github.com/microbiome/miaViz/pull/140 respectively.
Will add a PR for miaTime.

Ok can we close this one or better to keep open still?

I think we can close.

Great. Please actively close all issues as they become ready.


Clue steps

Displays the amount of steps completed on each clue.
Also, why does it show my full commit log when adding a new plugin?

it shows the whole log because you branched off a branch with other commits instead of runelite:master


chore: use<EMAIL_ADDRESS>

Updating to the latest @apollo/client beta - I can ship a new patch version once this lands 🎉

This is superseded by other stuff merged in-between, so I'll close it.


NumberFormat #72

This pull request refer to "TypeError: _this4.applyNumberFormat is not a function" #72
It s also implement a callback for adding eg setProperty in datatable.
Thanks for watching, it s my first pull request , i hope i have not make mistakes.

Thanks for the PR !
Should be fixed in latest.


Kubernetes storage class support

It would be nice to support Kubernetes storage class for dynamic provision.

Hi @hiscal2015
We've added the support for Kubernetes storage class. Please give it a try.


Failed to load "The kubernetes.io analytics dashboard"

It shows:
Data Set Configuration Error
Looker Studio cannot connect to your data set.

Looker Studio cannot connect to your data set.
The data source no longer has access to the underlying data set. Please contact the data source owner to fix it.
Error ID: 25c1890c

URL: https://lookerstudio.google.com/reporting/fede2672-b2fd-402a-91d2-7473bdb10f04/page/567IC
From: https://kubernetes.io/docs/contribute/analytics/

/kind bug
/sig docs

We've already got this reported as part of issue https://github.com/kubernetes/website/issues/43281, so it seems like a duplicate.
/triage duplicate

Oh, thanks.
closing
/close


docker-init: init at v1.30.0

I added the docker-init CLI plugin found in Docker Desktop as a togglable attribute in the docker package like other docker CLI plugins.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] 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

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.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.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge

@BastianAsmussen The merge bot will only merge changes autogenerated from an update script by the @r-ryantm bot, and only an accepted maintainer can request a merge like that

I see, thanks for letting me know. :+1:

I'm very sorry to bother you @drupol, but do you think you could possibly review my pull request? :)
You seem to be the latest to work on something related.

Thank you for your first contribution, :) Keep PRs coming!


SPUtils.java 's name is confusing

this issue is dummy but the shared prefs Util class name is confusing I think you should rename it I was about to use another library or use my own class.

SharedPreferences getPreferences (int mode)
Retrieve a SharedPreferences object for accessing preferences that are private to this activity. This simply calls the underlying getSharedPreferences(String, int) method by passing in this activity's class name as the preferences name.
U can read the document


App autoscaler 2.0.0

all changes should be in place, updated from 1.2.2 to 2.0

Thanks for this!
@dennisjbell and I are trying to integrate it into the latest 2.x release candidate (a major update to bring the CF kit up to modern-ish release versions), and it doesn't work for us.
Specifically, we're getting auth failures when talking to UAA to get our client token.  Looking back through the PR, I don't see where you're adding the new app_autoscaler client to the UAA itself (although I do see the credentials being generated in the Vault).
Is this working for you, locally?

I haven't tried it against 1.5.0 yet, hopefully I'll do it tomorrow and add in the fixes


Mention WolfRam Alpha in README + Admin's Manual

why did you close this!??!!?!?1? I just havent had a look at it yet. Please don't close without actual reasons.


Remove unnecessary sysctl configuration and reduce privilege

Describe the bug
Currently antrea-agent container needs to run in privileged mode because it needs to manipulate sysctl to avoid redirect packets being sent to source Pods when they are routed by gateway interfaces. However, with L3 flows, Pod-to-Pod traffic will no longer go through gateway interfaces. And for Pod-to-Service traffic, perhaps because there's a DNAT action, there is no redirect packets generated either when the packets are routed back from the gateway interface in practice.
Since sysctl configuration is the only reason why antrea-agent needs privileged mode, its securityContext can be changed to require specific capacities only, reducing the security risk.
To Reproduce
Revert the sysctl configuration manually on every Node, no traffic will be affected and no redirect packets will be received on interfaces of source Pods.
sysctl -w net.ipv4.conf.all.send_redirects=1
sysctl -w net.ipv4.conf.antrea-gw0.send_redirects=1

Expected
Unnecessary sysctl configuration should be removed, privileged mode should be avoided if not needed.
Versions:
Please provide the following information:

Antrea version (Docker image tag). v0.10.1

cc @antoninbas I have verified pod-to-service packets are not impacted manually, will create a PR to verify all e2e tests are not impacted.

The Agent no longer writes the send_redirects kernel parameters (v0.11). However, the FlowExporter feature uses sysctl to set nf_conntrack_acct and nf_conntrack_timestamp. Because the Agent does not fail (but instead logs a warning) when these parameters cannot be written to, "privileged" mode is technically no longer required. However, we still run the Agent in privileged mode by default, as it enables us to set these kernel parameters for FlowExporter and include more information in the flow records we export.
We are keeping this issue open for now because there are some options we can pursue to slightly improve things (see discussion in https://github.com/vmware-tanzu/antrea/pull/1364#discussion_r506299104):

support env variable to disable the warnings altogether
use privileged initContainer to set the sysctl kernel parameters


How do I contact you?

Hey Herman,
First off, thank you so much for building judge0, it's an incredible tool and I can't thank you more for open-sourcing it. I scheduled a call with you through the website (there was a link and then I scheduled for basically now, however it doesn't seem like you are available at the moment). Do let me know where and how we can chat!
Appreciate it, and apologies for creating an issue - for I could not find any other active platform to contact you through!

Hey Herman,
First off, thank you so much for building judge0, it's an incredible tool and I can't thank you more for open-sourcing it. I scheduled a call with you through the website (there was a link and then I scheduled for basically now, however it doesn't seem like you are available at the moment). Do let me know where and how we can chat!
Appreciate it, and apologies for creating an issue - for I could not find any other active platform to contact you through!

You can have a try.
https://judge0.com/#contact-us
Good luck.


Attach artifacts to pull requests

Fixes #41
Adds artifacts to the pull request and uses the pull request number for the firmware version number.

Apparently this has to be on the main branch to work so... merging to see what happens.


Draft notebooks for pangeo

Related to #44
@tinaok  I just started the draft; still need some work to finalise it.

Added  Pangeo notebooks for 2.3_data_access/exercises/_alternatives
Added Pangeo notebook for 3.1_data_processing/3.1_exercises/_alternatives/31_data_processing_pangeo.ipynb
added 3.2_validation/3.2_exercises/32_validation_pangeo.ipynb
drafted 3.3_data_sharing/3.3_exercises/_alternatives/33_data_sharing_pangeo.ipynb (create COG file but stac item, etc. need to be added)


devtool: use public influxdb endpoint

这个 PR 实现什么功能/修复什么问题:
是否需要 backport 到之前的 release 分支:

3.0
/area devtool
/cc @zhasm

Codecov Report

Merging #4533 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #4533      +/-   ##
=========================================
+ Coverage    6.83%   6.85%   +0.02%
=========================================
  Files         522     521       -1
  Lines       88600   88417     -183
=========================================
+ Hits         6052    6059       +7
+ Misses      82043   81853     -190
  Partials      505     505

Flag
Coverage Δ

#aFlag
6.85% <ø> (+0.02%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/cloudcommon/db/quotas/register.go
10.34% <0%> (-2.82%)
:arrow_down:

pkg/compute/models/zonequota.go
20.35% <0%> (-0.56%)
:arrow_down:

pkg/compute/models/guests.go
0.55% <0%> (-0.01%)
:arrow_down:

pkg/compute/models/cloudproviders.go
1.01% <0%> (-0.01%)
:arrow_down:

pkg/compute/models/disks.go
0.58% <0%> (-0.01%)
:arrow_down:

pkg/cloudcommon/db/db_dispatcher.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/instance_snapshots.go
4.61% <0%> (ø)
:arrow_up:

pkg/cloudcommon/db/quotas/quotas.go
0% <0%> (ø)
:arrow_up:

pkg/compute/guestdrivers/managedvirtual.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/guest_actions.go
0% <0%> (ø)
:arrow_up:

... and 14 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 cd52997...fa2e0e3. Read the comment docs.


174654774/add api tests to ci

What's the status on this? I'd like to not have PRs that are still in active development. It gets really confusing for me.


Bump golang to 1.21.7

Kind
/kind cleanup
Release Note
NONE

/cherrypick release-1.14

/approve

https://github.com/cert-manager/cert-manager/pull/6732 failed to apply on top of branch "release-1.13":

Whoops, 1.13.x uses go 1.20.x!


add rabbitMQ tracing.

Add RabbitMQ tracing. See https://github.com/opentracing-contrib/java-spring-cloud/issues/5

@gytis @geoand could you please also have a look?

@pavolloffay nope

Hopefully I'll be able to take a look tomorrow.
On Tue, Apr 17, 2018, 08:40 Gilles Robert<EMAIL_ADDRESS>wrote:

@pavolloffay https://github.com/pavolloffay nope
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opentracing-contrib/java-spring-cloud/pull/126#issuecomment-381850673,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AELBv_OxMTV2d-cFY9bJLj0intu6IYpPks5tpYBUgaJpZM4TVRoY
.

@pavolloffay Thanks for your comments. I'm waiting for other remarks from @geoand and @gytis to tackle everything in one go :) .

@pavolloffay @ask4gilles I don't have enough tracing - messaging experience to be able to provide much insightful information on this PR.
So I will hand it overt to @gytis to provide more insights
I would be more than happy however to comment on any Spring or coding issues however so I will leave the occasional comment if I come across anything that seems odd.

Thank you for your time @geoand! I'll wait for the last round from @gytis and then I'll implement your remarks.

@gytis does it make sense to move it to https://github.com/opentracing-contrib/java-spring-messaging or create java-spring-rabbitmq?

@pavolloffay I think java-spring-rabbitmq would make more sense.

@ask4gilles I have created https://github.com/opentracing-contrib/java-spring-rabbitmq. I have also added you to the committers with admin permissions.

@pavolloffay Thanks, I'll have a look ASAP.

Can we commit this pull request asap, I want to use opentracing with RabbitMQ

@abhsinh2 this feature was moved to a different project: https://github.com/opentracing-contrib/java-spring-rabbitmq/pull/1

I just merged https://github.com/opentracing-contrib/java-spring-rabbitmq/pull/1
I guess this PR can be closed.
We'll need to integrate https://github.com/opentracing-contrib/java-spring-rabbitmq in a separate PR.


Traversable: derive

Continuation on traversable type class. Follows from #217
TO CONTRIBUTORS:
Make sure you have:

[x] reviewed and applied the coding style: coding, naming
[x] make sure definitions and lemmas are put in the right files
[x] make sure definitions and lemmas are not redundant

For reviewers: code review check list

Can we restart this build please?

I ended up implementing the derivation of lawful instances and the definition of standalone list.map and list.traverse but they are still no more universe polymorphic than traversable. There are two interesting features that I'm unsure how to implement:

standalone lemmas for the traversable laws
making the standalone definitions and lemmas maximally universe polymorphic.

I'll leave them to a future PR and focus on getting this PR merged.


Update api-management-howto-create-or-invite-developers.md

we need to be very clear as to which specific persona we are talking about in each of the articles.  "developers" is an overloaded term, especially in API eco-system.  you have developers who produce APIs, and producers who consume other people's APIs, and then there are consumers who do not publish anything but are interested in consuming someone else's APIs. Finally, there is the APIM back-office or admins.
In my commit, i have tried to articulate how a producer can manage its own consumers, since they are the best people to know who they are writing the APIs for.  It could be part of their own org, or outside the org, or outside the company.

@dlepow
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Added Filter error when file loaded does not match schema

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/2486/
Test PASSed.

Merged! Thanks @akmorrow13!