[New Port Request] libfortrt

Library name: libfortrt
Library description: Runtime library for Fort compiler. Libfortrt is a new implementation of the Fortran runtime library.
Libfortrt is the runtime library for fort (the Fortran frontend for LLVM,
https://github.com/llvm-fortran/fort). It can be used by any other Fortran
compiler.
Source repository URL: https://github.com/llvm-fortran/libfortrt
deps: Fort
https://github.com/microsoft/vcpkg/issues/10439

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Add AnonCreds Rust integration test into ACA-Py

In creating #2276 , @dbluhm and team used this gist test to verify the work they were doing. This task is to add this test to the ACA-Py BDD integration tests so that it is run automatically on merging into (for now) the AnonCreds development branch, and ultimately into the main branch when the AnonCreds branch is merged into the main branch.
The task is to adapt the test for running as an integration testing, and working with all of the other integration tests we have for ACA-Py.
As needed, please consult with @dbluhm for guidance on challenges that come up in trying to merge the work.

Had to step off to concentrate on #2304.
Leaving a trail here for myself to pick up later.  Had a lot of issues running BDD, but figure kind of fighting up hill without pulling in commits to remove 3.6. So pull in changes to get this branch closer to main, see where that leaves the original set of tests and add new ones.
Base command to use for BDD (matches github action/workflow):
LEDGER_URL=http://test.bcovrin.vonx.io PUBLIC_TAILS_URL=https://tails-test.vonx.io LOG_LEVEL=warning NO_TTY=1 ./run_bdd -t @GHA

Which currently leaves main at:
4 features passed, 0 failed, 1 skipped
26 scenarios passed, 0 failed, 38 skipped
202 steps passed, 0 failed, 538 skipped, 0 undefined

For discussion at tomorrow's (2023-08-01) Maintainer's meeting.
@dbluhm - having trouble reconciling the changes in the anoncreds-rs branch and main. Mostly having trouble getting anoncreds-rs branch to work with the test script (success) and existing BDD tests (not successful). Will need to discuss what the plan was/is...
ledger/base.py send_schema method "replaces" create_and_send_schema? But the create_and_send_schema is still used in messaging/schema/routes.py, similarly with create_and_send_credential_definition and credential_definitions.  I had to add IndyIssuer binding back in to askar profile also.
So, I am not sure if the plan is to continue retrofitting the existing schema/cred_def routes to use new functions in ledger and issuer classes? I had attempted using the issuer methods (issuer.create_schema andissuer.create_and_store_credential_definition) before calling the new ledger methods (ledger.send_schema and ledger.send_credential_definition) but they don't quite play nicely with each other (ledger method signatures do not accept the returns from the issuer methods, other logic in place that don't work with payloads/responses (like parsing seqNo from schema response and deserializing CredentialDefinition payloads...)
So all that needs to be addressed before we address the changes @andrewwhitehead has made to ledger base with updates to indy credx.
We are probably not too far away, I am just unclear on the thoughts that got us here (anoncreds-rs) and what the plan was moving forward.

The idea was that the messaging/schema/routes.py would be removed altogether, its endpoints having been replaced by those in anoncreds/routes.py.

Closing this ticket now that an initial cleanup has been done (see PR #2401).
Will add more tickets/issues related to this work.


Danroth27/restorednvm

Hi @rajthakur609, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;


Documentation Updates

Proposed Changes
When doing through documentation, I noticed there were some items which were listed, but not applicable to certain modules as well as unlisted functions in modules. I created a 'PR' markdown document which will be shown on the documentation site. The reasoning behind this is to allow users to know the feature is supported, but we just haven't gotten to the documentation yet.
Additionally, I started adding examples to the Result module. I feel the documentation is lacking in examples overall so I figured brining over some of our unit tests helps provide visuals for users who may not grasp the function signatures at first glance.
I have also alphabetized the functions under their respective modules for easier discoverability.
Checklist

[X] I have added necessary documentation (if appropriate)

This is so amazing, thank you!


Add option to not reorder loops?

As noted in #126 and in my own usage of the library, compile time is roughly factorial in the number of nested loops. For many applications, it is not too hard to guess a loop ordering that should be good enough. For example, consider this set of 6 nested loops:
    for λx in 1:Kx, λy in 1:Ky, λvx in 1:Kvx, λvy in 1:Kvy
        for αx in 1:Nx, αv in 1:Nv
            ax = as_x[λx]
            ay = as_y[λy]
            avx = as_vx[λvx]
            avy = as_vy[λvy]

            # Do some work
        end
    end

The indices λx, λy etc. are larger strides than the indices αx, αv. So we probably shouldn't even consider loop orders where the alphas are outside. In terms of ordering the outer loops, I think that a human should be able to do a pretty good job just based on the way the data is laid out in memory. It would be nice to be able to take advantage of the automatic unrolling and vectorization capabilities of LV without incurring a compile time blowup.
Is this a feasible suggestion based on how the library is structured right now?

I think that a human should be able to do a pretty good job just based on the way the data is laid out in memory.

That is a function of type. E.g., Adjointand PermutedDimsArray.
People do often get it wrong. "Fastest dimensions on the inside" is not always fastest.

I agree. LoppModels (the rewrite) of avoiding the factorial behavior.
There is #398, but I didn't actually notice it helping for some reason. It's worth looking more into.

Is this a feasible suggestion based on how the library is structured right now?

#398 is simple enough, and avoids the factorial algorithm.
Avoiding reordering at all is more difficult, because LV forgets what the original order actually was.

Maybe the logic to actually skip the factorial behavior is broken in #398.

I am doing some more investigation into my slow compile times, and I think that it may actually have more to do with LLVM being quite slow to deal with high-dimensional arrays than with LV--the compile time for my generated function is extremely slow even with no @turbo annotation.


Feedback When Adding to Encounter

Found this on r/dnd, nice job!
Just wanted to add a feature request here to have some feedback when adding a monster to an encounter since currently nothing happens and you can end up adding multiple monsters to the campaign without ever realizing it.

Thanks for the feedback! Something I'll definitely add.

Thanks again! I've added some animation to the button's hover/active states. There is also now a counter in the top right which tracks total monsters in encounter.


Get-PrintJob  -id  - use with wildcard generates an error - documentation is ambiguous.

This is information needs to be updated.
At the top it states  "You can use wildcard characters." but then below it has "Accept wildcard characters:	False".
No further information is provided on what wildcard characters are accepted.
In my testing if I use "" I get the following error:
"
Get-PrintJob : Cannot process argument transformation on parameter 'ID'. Cannot convert value "" to type "System.UInt32". Error: "Input string was not in a correct
format."
At line:1 char:18

get-printjob -id *

             ~

CategoryInfo          : InvalidData: (:) [Get-PrintJob], ParameterBindingArgumentTransformationException
FullyQualifiedErrorId : ParameterArgumentTransformationError,Get-PrintJob

"
-ID
Specifies the ID of the print job to retrieve. You can use wildcard characters.
Type:	UInt32
Position:	Named
Default value:	None
Accept pipeline input:	False
Accept wildcard characters:	False

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

ID: 6edbed07-de9b-d08f-9d50-83437779a89f
Version Independent ID: dd366084-4245-efd6-ec7b-60301e948e9f
Content: Get-PrintJob (PrintManagement)
Content Source: docset/winserver2022-ps/printmanagement/Get-PrintJob.md
Product: w10
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

@Ferretaa Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/windows-powershell-docs/pull/3589
Once the author approves, the changes will be live.
Thanks
Sri


[Doc Update] [Cards format masking property]

updated isMasked property with Style.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/task-modules-and-cards/cards/cards-format.md
82
62
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 2d3226e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/task-modules-and-cards/cards/cards-format.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix metrics uri: remove unvalid character from uri

change "jhi-metrics" to "jhimetrics"
Fix #302

Please make sure the below checklist is followed for Pull Requests.

[ ] Travis tests are green

[x] Tests are added where necessary

[x] Documentation is added/updated where necessary

[x] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

I didn't see you already made the work in #341 @pascalgrimaud

Yes this is directly linked to JHipster 6 migration. We shall wait for this other PR to be merged.


[youtube:tab] Fallback to API when webpage fails to download

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[X] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[X] Searched the bugtracker for similar pull requests
[X] Checked the code with flake8

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[X] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[X] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
This supersedes #682. (I'm making a new PR to keep things on topic - the other PR is a bit messy due to be experimental).

This PR adds support for falling back to the Innertube API when the webpage fails to download for whatever reason (429, network error, initial data extraction failure, etc.)
It is also possible to manually skip the webpage download by using --extractor-args youtubetab:skip=webpage
Also added a quick fix for the resolve MP to OLAK playlist handling, since that was broken.
At least at the time of writing, yt-dlp should now have a complete* workaround for the common** YouTube HTTP 429 error reported, for both video and tab extraction.
 *except private content accessible only through a secondary or greater account/channel
**common referring to the webpage request getting 429nd only, but the API still works.
resolves https://github.com/yt-dlp/yt-dlp/issues/926, resolves https://github.com/ytdl-org/youtube-dl/issues/23638 (and most of the other 429 reports upstream)
Known issues:

When using cookies, the webpage is required to extract the required variables. Without this, the API requests will be for the first channel of the first account.

In the case of the user passing cookies, and the webpage failing to download, the tab extractor will raise an error. This is to stop unwanted behavior in variety of situations (e.g. an automated script to download a secondary channel liked playlist, but the webpage fails and starts downloading the first channel liked playlist).

Currently, the user can skip this check by passing --extractor-args youtubetab:skip=authcheck

channel search does not work (returns 0 results)

I think this is a YouTube a bug. YouTube appears to produce params for itself that it cannot understand.
See https://github.com/yt-dlp/yt-dlp/pull/1122#issuecomment-933214758

TODO:

[ ] Improve the error message when authenticated but webpage fails.
[X] Documentation
[X] Add (and probably fixup) tests
[X] fix https://www.youtube.com/feed/recommended
[ ] Fix unavailable video messages showing - test: https://www.youtube.com/playlist?list=UUTYLiWFZy8xtPwxFwX9rV7Q for another PR

Had to fix and improve the visitorData handling since the home/recommended was broken.
Some info on visitorData for the sake of documentation:
When YT gives visitorData in the responseContext of any API request, from what I see, is the most current one (so we need to update our visitorData to it). visitorData can change in some situations, such as https://github.com/ytdl-org/youtube-dl/issues/28702.
If YouTube doesn't provide visitorData in responseContext, visitorData is the same as before (i.e., does not need updating).
We've seen it been used to keep track of what comment section version to use (ensure consistency) when YT was switching from the old to new API a while back.  Along with the recommend feed and the playlists in the above issue, maybe this suggests it is some sort of token that identifies a session state?

After this PR I do need to go through YoutubeIE and clean up it's visitorData handling. I notice the comment one no longer works. Even though it may not be required as of now, it is good to have just in case (e.g A/B testing as mentioned before).
Note: probably not a good idea to share visitorData across different client versions

Some things we'll move to a future cleanup PR(s):

generalize extract_response and extract_webpage
fixup extract_response & extract webpage INFO alerts:

show only if incomplete data
otherwise, show in debug mode

fixup tab extractor tests (needs above)
remove old python2 hacks from tab entries

Some changes:

added a few tests for API Fallback mode
raise an error when url is not resolved (no endpoint provided) instead of unexpected tab not found error. This usually happens when a, for example, invalid playlist link is provided. Normally on YouTube it redirects to the home page.

Issue: channel search (e.g. https://www.youtube.com/user/theCodyReeder/search?query=chicken):

resolve_url endpoint resolves this correctly, but the query parameter that would usually be in plaintext json is instead in the protobuf params. YouTube's backend doesn't understand itself in this situation so the API fallback does not in this case. :laughing:

Ok


Revert "Adding capabilities to change the error returned by the driver"

closes #88
Reverts shogo82148/go-sql-proxy#84
#84 introduced unexpected breaking change, so I'll revert it.

coverage: 77.893% (+0.2%) from 77.728% when pulling 918628bbdcffdaa20c5130d8f623426ab60a4b3d on revert-84-main into 300c00d683bba901e4828daa83e1ab77c3621361 on main.


Trying to pretrain RoBERTa, but having trouble making encoder.json, vocab.bpe, and dict.txt

Hi.
Thanks for releasing pretrain code for RoBERTa.
I'm trying to pretrain a roberta-base model with Japanese data.
To make a traing data set, I need to encode my data with byte level BPE, but got stuck in making encoder.json, vocab.bpe, and dict.txt.
I've tried subword_nmt with English corpus to repplicate encoder.json which was proviede with pretrained model, but result is quite different; most of tokens does not start with \u0120 which happens a lot in provided encoder.json.
Any guide about making these files in other languages?
Thanks.

I have the same question, but you can just follow the language format without using bpe

@imeanu How made encoder.json, vocab.bpe, and dict.txt for other languages?


Errors in log relating to Gas Cost

Describe the bug
Errors in the logs every 2 mins,
2021-10-14 09:40:40 ERROR (MainThread) [custom_components.hildebrandglow_dcc.sensor] Key Error - rate (3.19)
2021-10-14 09:40:40 ERROR (MainThread) [custom_components.hildebrandglow_dcc.sensor] Key Error - rate (3.19)
2021-10-14 09:40:41 ERROR (MainThread) [custom_components.hildebrandglow_dcc.sensor] Lookup Error - standing ({'rate': '3.19', 'standingCharge': '26.25'})
Although the sensors update correctly, and contain the correct cost, only the gas meter reports for me, so only gas sensors are present.
To Reproduce
Install and configure integration via HACS
Expected behaviour
No errors!
Screenshots
If applicable, add screenshots to help explain your problem.
Version
Integration v0.4.4
Home Assistant 2021.10.4 (docker)

How odd, for me the values are returned as numbers, but for you, as strings.
Easy to fix, but sorry I won't have time until the weekend.

@ColinRobbins confirmed resolved in 0.4.6, many thanks 👍


Hugo: Add popup functionality to logos

Extended the logos__item shortcode to facilitate the popup. From now on, the logos__item shortcode should always be closed.
Made the logos__item shortcodes on the homepage self closing
Added dialog typescript for popup functionality. This includes an interface, a base-dialog and a dialog ts
Added info icon
Added popup open/close translations eg for screenreaders
Improved logo styling
Improved section styling
Added dialog styling
Extended the example page and examples with the new attributes (/examples/sections/logos/)

For https://linear.app/usmedia/issue/CUE-259

Imported as https://review.gerrithub.io/c/cue-lang/cuelang.org/+/1168257

Closed via 15f284192b61eea3f9bcd2ccd25ca39fa4efa99e


Add github link on website

It could be great to add the github link on the website. ;)

Gotcha ;)


How to use Key Vault to store IoT Edge gateway certificates

Hello there. I'm deploying an IoT solution where our Edge is used a transparent gateway. Right now, we have production certificates located in our Edge, on the paths you specify in this page, at section "Install certificates on the device":
Device CA certificate: <WRKDIR>\certs\iot-edge-device-MyEdgeDeviceCA-full-chain.cert.pem
Device CA private key: <WRKDIR>\private\iot-edge-device-MyEdgeDeviceCA.key.pem
Root CA: <WRKDIR>\certs\azure-iot-test-only.root.ca.cert.pem
Well, this is pretty much not secure at all, because one with some knowledge would have access to those certificates. At the same section "Install certificates on the device", you point out using Key Vault to securely store certificates, but give no clue about how to do that. Please, could you tell more about this option?
Thank you for your help!

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

ID: 94792ff1-5976-90f4-5092-0d8d8ff01635
Version Independent ID: d65482d2-64cc-56b1-c52a-b3ddde07b205
Content: Install certificates on device - Azure IoT Edge
Content Source: articles/iot-edge/how-to-install-production-certificates.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@manueljesusrd Thank you for bringing this to our attention. I will have the content team evaluate this and update the documentation accordingly.
Note:
The Azure Key Vault link detailed in the documentation takes you to a non-specific documentation page. If the content exists, it would be better to have specific reference to how to store certificates with Key Vault, if the functionality is available.

@Mike-Ubezzi-MSFT thank you Mike. So after reading your "note", my subsequent question is: is there even the possibility of implementing a key vault to store certificates for IoT Edge Transparent Gateway? I'm not that sure now 😮

@manueljesusrd Please see this multipart tutorial (Configure an IoT Edge device) which discusses the configuration of Key Vault and using certificates. By following this tutorial, you will be able to leverage the example PowerShell scripts. The tutorial references directory paths and scripts that need to be executed, and will not be completely useful unless you understand each specific step and are able to apply the logic to your own solution.  I wanted to provide a example of how it could be implemented.

Hi @Mike-Ubezzi-MSFT .
I'm sorry, but after reading that tutorial (which I already did some days ago) I'm a little bit more confused. That tutorial says that you HAVE to download your certificates into a folder of your Edge. This doesn't make sense in terms of security, does it? I mean, even though you securely store the certificates in your KeyVault, at the end of the day you have to put them in a folder of your Edge, with no encryption and which might be exposed to anyone.
Maybe I didn't explain my point well enough, but what I wanted to do is somehow making a reference to the Secrets in the config.yaml, for example:
certificates:
device_ca_cert: "route or reference to KEYVAULT'S SECRET that stores this certificate"
device_ca_pk: "route or reference to KEYVAULT'S SECRET that stores this certificate"
trusted_ca_certs: "route or reference to KEYVAULT'S SECRET that stores this certificate"
If this is not a possibility, the I don't see the "secureness" of storing certificates in KeyVault when you however have to put them in a local folder...

@manueljesusrd What you are looking for is an application example of using Key Vault where the certificate authentication does not require placing the certificate on the device itself. In searching through available IoT reference architectures and example workloads/sample workloads, there is no Key Vault integration scenarios. Your feedback is spot on. All this logic would need to be built into your application and we do not have an example to demonstrate at this time.

@Mike-Ubezzi-MSFT is it possible to use TPM encryption when using a transparent gateway? Or when using IoT Edge as a any type of gateway, not only transparent? I've seen that TPM is supported in device provisioning, butthere's nothing about storing gateway certificates in TPM and referencing them in config.yaml

@manueljesusrd You're correct. It is currently not possible to use a TPM to secure gateway certificates.  Note, the term "gateway" certificates is a bit of a misnomer because the same certs are used to establish trust between local modules and edgeHub.

@manueljesusrd Does that answers your question? Do let us know if you have further questions.

@AshokPeddakotla-MSFT I think I have enough info. It's true I haven't got any secure storing alternative from you to implement on the IoT Edge solution, but it seems there's not a totally secure solution right now. Thank you!

We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


fatal: unable to checkout working tree

I've been checkout the repo using source tree, after some time it finished with some errors like "unable to create file Lesson08-Quiz-Example/T08.01-Solution-AddTheContentProviderPermission/droidtermsprovider/src/main/java/com/udacity/example/droidtermsprovider/DroidTermsExampleContract.java: Filename too long" and any other like that because of filename too long. After done to checkout there is fatal error said "fatal: unable to checkout working tree" and give a warning "warning: Clone succeeded, but checkout failed."

Resolved.


[Feature request] force_submission_to_leaderboard

Do you have plans to enable "force_submission_to_leaderboard" flag?
Thanks!

This can be done in the leaderboard editor.


Increment version for C SDK releases

Increment sdk version after release of C SDK

Closing. This PR was meant to demonstrate behavior.


feat: agent - eBPF Set the full path for file reading and writing

This PR is for:

Agent

Affected branches

main
v6.5

@yinjiping 这个 6.5 不用提


WIP: Campaign Pixel template tag

this isn't finished, but i'm opening this up now to get feedback @spra85 @benghaziboy @mparent61


Couldn't read cache at: .cache

Hello
I want to authenticate users with Spotify in order to access their saved tracks. I've been following example form this repo README, I got the authentication process to run on a localhost Flask server. However every time I run the app my console throws me this warning :  "Couldn't read cache at: .cache" \n "Enter the URL you were redirected to:"
If I copy and paste the URL from browser I can get this working and it does actually create .cache file in my directory. In production users won't be able to copy and paste a URL into console, this has to happen automatically.
I am running on Ubuntu 20.04, Python3.7 and latest Spotipy. My Spotify developer credentials are all set up.
Help greatly appreciated :)

You want to handle any amount of user on a server, so you are looking to integrate Spotipy in an API


storage: remove (*Client).Close

Close is doing nothing in terms of freeing anything or cleaning a buffer. Therefore, it is not necessary to close the storage clients. See yourself at https://github.com/GoogleCloudPlatform/google-cloud-go/blob/master/storage/storage.go#L146
On the top of that, current Close method returns and error and documented as a necessary call which makes it harder to use the storage client and handle its life cycle.
We should remove the closer from this package.
/cc @jba @okdave @broady

It's there to support a future upgrade to gRPC without breaking any of the API surface.

Both the JSON API and gRPC API have an optimization of keeping a cached TCP connection open for reuse.
The JSON (HTTP) API does it via net/http.Transport, where it eventually times out due to lack of use.
Why can't gRPC just shut down its cached TCP connection after N seconds of idle? Maybe it already does.
I guess my question is: why does gRPC require a different API from another thing which is also just HTTP requests under the covers?

To answer your last question first, I naively assume that all HTTP is connection-per-request. Obviously that is not the case.
But even with close-on-idle, couldn't you still get into trouble if you don't have control over shutting down the connection? What if you have some sort of proxy and you need a separate backend client per frontend connection, for auth perhaps. Couldn't you run out of descriptors at peak?

Let's resolve this by end of week.
I'd like a single answer for all clients, for consistency. Do we agree on that?
My feeling now: in my current state of knowledge, I don't know whether Close is important on this client or any other in this repo, except that the new logging client uses it to make sure async operations have finished.
None of us know if any of the clients will later need Close. They may want to shut down goroutines, or close connections using transports that don't support close-on-idle, or depend on other services that require Close for their own reasons. And I still am not convinced that close-on-idle will free resources fast enough in some cases, as I wrote above.
This is a case where we can't just add it later: it is backwards-incompatible to require calling Close when we hadn't before. Conversely, having Close now means we can remove it later, or make it a no-op, without changing behavior.
I don't think Close adds much to the complexity of using a service. Typically you'll use a client in the lifetime of one method (possibly main), in which case you just write a defer. Many other cases, like lazy creation, can be handled by defer func() { if client != nil { client.Close() }} in main, assuming the client is in a global. @rakyll, can you give examples where it's harder than that to deal with Close?
So I vote for adding Close to all clients.
Interested in your thoughts, @bradfitz @rsc @robpike.

And I still am not convinced that close-on-idle will free resources fast enough in some cases, as I wrote above.

It's not like every operation has some lingering resources associated with it. There should be exactly 1 cached & idle-for-a-bit TCP connection open for the entire program. And with HTTP/2, there should really just be one for all Google APIs, regardless of hostname (storage.googleapis.com and foo.googleapis.com sharing the same conn), since requests can be coallesced based on the hostnames listed in the TLS cert.
And I don't think any of our existing Close methods in any API library call through to see if the user's provided *http.Client's Transport supports a CloseIdleConnections method. Even if it did, I don't think you'd want to call it, since it would close all idle connections for all hosts, even non-Google API hosts.
Likewise, gRPC doesn't technically need shutdown. (Although the current implementation might a bit, but shouldn't.)

None of us know if any of the clients will later need Close. They may want to shut down goroutines,

Both flushing and closing down goroutines are a more valid reason.

So I vote for adding Close to all clients.

That's fine.
But please document the expected lifetime and concurrency-safety of the clients enough so people aren't creating & closing them a bunch.
The Close docs should de-emphasize their importance somehow.

But please document the expected lifetime and concurrency-safety of the clients enough so people aren't creating & closing them a bunch.

Not sure what you mean by "expected lifetime". By concurrency-safety I assume you mean whether it's safe to have multiple concurrent calls to Close?
I also don't understand why "creating & closing them a bunch" is a bad thing, or why anything we say about Close would make people do it. Do you mean that for simplicity someone will create all their clients in main, so they can defer Close there as well?
Probably the most common case is that you create a client, eagerly or as needed, and then just leave it open for the life of the program. I'm fine saying something like "If the program terminates without Close having been called, no big deal" on most clients, logging being the big exception.

I mean something like the docs at the bottom of Transport: https://golang.org/pkg/net/http/#Transport
If the object is heavy and will have associated TCP connections, it should be reused for a long time from multiple goroutines, and not created and closed repeatedly deep inside some call stack.
Doing a defer to close it in main is useless. Your program can just exit and tear it down. ("Crash-only software", etc.)
Just let people know what to expect and how heavy (or not heavy) each client is.

Another use case to consider is on App Engine, where currently a new client must be opened for each incoming request.
The urlfetch client doesn't keep any resources around, though, so maybe it's not much of a consideration.

I do believe that the AE folk are hoping to get to the point where we don't need a new client on each request. I wonder if we could do something intelligent today on AE to allow a long-lived client (already, each request takes a ctx – so we might be able to fake a long-lived client that did a dial-per-func underneath today?)
@bradfitz re the single connection for all APIs – do we need to do something here to make sure that happens, or is that plumbing happening under the hood already?

can you give examples where it's harder than that to deal with Close?

It is confusing. I have a crash-only program, I want to crash if any error happens and I am not sure how critical is to close the client. I jump to the definition and see what Close is doing nothing. As a developer, I learn that I shouldn't care about closing the storage client given it does nothing. Putting a Close method is not going to make people call it unless it is not doing anything it says.
Just two cents.

I want to crash if any error happens and I am not sure how critical is to close the client

The Close doc will make it clear that you don't have to close at exit.

I want to crash if any error happens and I am not sure how critical is to close the client.

I'll make it clear that you don't have to call Close on exit.


Calico (v3.0.1)  calico/node  unit test on SLES 12 SP2  frozen

Calico (v3.0.1)  calico/node  unit test on SLES 12 SP2  frozen, the problem is related to iptables.
The version is v.1.4.x,
Expected Behavior

Current Behavior

Running calico/node unit test by make st on SLES 12 SP2 is frozen.
The log shows there are ERRORs in felix log, which asks for higher version of iptables (current on SLES is v1.4.21)
  2018-02-08 20:23:32.972 [ERROR][78] table.go 778: Failed to program iptables, loading diags before panic. error=exit status 1 ipVersion=0x6 table="filter"^M
  ...

see st.log
When running unit test on RHEL 7.x or Ubuntu, no problem as the iptables version is v1.6.x.
The previous Calico versions do not have such a problem.
Possible Solution

Steps to Reproduce (for bugs)

under calico/calico_node folder, run make st

Context

Your Environment

Calico version v3.0.1
Orchestrator version (e.g. kubernetes, mesos, rkt):
Operating System and version: SLES12 sp1/sp2/sp3
Link to your project (optional):

@fasaxc have we recently changed the version of iptables we require?

No, but it's possible that we've introduced a new module requirement (maybe xt_ipvs).  @samding01 can you run the felix FV tests in the felix repo?  make fv should do that.  Those tests emit the felix logs so you should get a clearer error message (the calico ST tests only show a snippet of the logs, which in this case doesn't include the actual failing iptables command).

@fasaxc  see test_sles12_2_fv.zip,
this is make fv result of felix

We are still observing this issue in v3.0.3. We too suspect that this issue is w.r.t IPVS. Commenting line 91 in github.com/projectcalico/felix/iptables/match_builder_test.go

@@ -88,6 +88,6 @@ var _ = DescribeTable("MatchBuilder",
        Entry("Protocol and ports", Match().Protocol("tcp").SourcePorts(1234).DestPorts(8080),
                "-p tcp -m multiport --source-ports 1234 -m multiport --destination-ports 8080"),
        // IPVS.
-       Entry("IPVSConnection", Match().IPVSConnection(), "-m ipvs --ipvs"),
+       //Entry("IPVSConnection", Match().IPVSConnection(), "-m ipvs --ipvs"),
        Entry("NotIPVSConnection", Match().NotIPVSConnection(), "-m ipvs ! --ipvs"),
 )

we were able to reduce the test failure from 150 to 31 with make fv command.

@samding01 @guirish Do the tests pass if you manually load the xt_ipvs kernel module before you start?
Can you reproduce on any of our officially supported distributions (Ubuntu or RH/CentOS) or is this SLES-specific?

@fasaxc this issue is SLES specific only , we did not observe it on RHEL(7.3 and 7.4) and Ubuntu(16.04 and 17.10) distros. We will investigate further on how to manually loading xt_ipvs kernel module.

This appears down to IPVS support on the system. iptables should still work fine, as should Calico in iptables mode, and IPVS may well work too if the correct version of iptables is installed and the right kernel features are present.
In general, I think UTs are not guaranteed to pass on any given system, even in supported environments.


pacer: Introduce LinearPacer

This commit introduces a LinearPacer which paces an attack by starting
at a given request rate and increasing linearly with the given slope.
This is a pre-requisite for implementing #418

PTAL @mrnugget, @keegancsmith

cc @fluffle

@tsenart i was about to open a question issue for ramp-up and ramp-down but then i saw this.
Readme.md is not up to date with pacer
Should i expect a doc soon

@anas-qa: I haven't had the time to implementing #418, so there's no way to use this via the CLI yet.

Thanks @tsenart
This would be a huge +1 once it is
Hope soon 👍

Sorry I didn't get around to this, life sort-of got in the way. The code looks great, this is exactly the sort of thing you need to do destructive load-testing :-)


Create public profile page

The page should only be visible for users that don't have their profile marked as private
The stats leader boards should link to the achievements page for every public user


How to get up and running with Google Drive API

Hello,
Thanks for creating this package!
The Google API is a bit difficult to find your way in if you don't use it alot (at least it was for me), so I created a walkthrough on how to get the necessary ID's and secrets and how to use it in a Laravel application.
https://gist.github.com/ivanvermeyen/cc7c59c185daad9d4e7cb8c661d7b89b
Maybe this can help others.
Feel free to include any of it in your readme. 👍

@ivanvermeyen
i get this error after running through your tutorials
LogicException in Client.php line 265:
refresh token must be passed in or set as part of setAccessToken

hmmm...
I just created a new project and it still works for me... Did you check the auto refresh checkbox  and put the refresh token in your config / .env ?

it works now .. i had to set the values in the GoogleDriveServiceProvider class , looks like heroku was not loading my .env file

@ivanvermeyen
Hi, thanks for the tutorial.
It worked.
But, now, how to download files from Google Drive?
Can you give me some guidance?

@imannms
i am still having an issue with downloading from Google Drive ,i will be glad if @ivanvermeyen  gives some guidelines.

@imannms @korantengNanaYaw I will try to find some time to look into fetching files from Google Drive. I haven't done this yet, since I only needed to upload files.

@imannms @korantengNanaYaw I tried this and it worked for me:
The thing with Google Drive is that each file has a unique ID, because you can create files with duplicate names... Unfortunately this makes identifying files pretty hard...
Anyway... When I ran this, I got a list of all file ID's in the folder that is associated with the folder ID in the config.
dd(\Storage::disk('google')->allFiles());

In Google Drive you can also right click a file and generate a share link to find the file ID.
Once you have the ID, you can get the file contents:
$fileId = 'xxxx';
$rawData = \Storage::disk('google')->get($fileId);

You could then save it to a file, or download it. For example, if I fetch a PDF and want to download it, I can put this in a controller method:
return response($rawData, 200)
    ->header('ContentType','application/pdf')
    ->header('Content-Disposition', 'attachment; filename="downloaded.pdf');

This worked for me.
The hard part is getting the file ID though...

@ivanvermeyen @korantengNanaYaw
Hi, I found a way to find file id quickly.
Here the way I use it:
$filename = md5(microtime()); // to make unique filename
$content = 'Lorem ipsum dolor sit amet...';
Storage::disk('google')->put($filename.'.txt', $content);

And then I found a function on http://stackoverflow.com/questions/1019076/how-to-search-by-key-value-in-a-multidimensional-array-in-php
Check answer by stefgosselin

This brought down the the task to match the 12 000 records to a 1.5 secs.

function getId(Array $array, $key, $value)
{
  foreach ($array as $subarray)
    {
      if (isset($subarray[$key]) && $subarray[$key] == $value)
      return $subarray;
    }
}

  $listContents = Storage::disk('google')->listContents();
  $id = getId($listContents, 'filename', $filename);
  echo $id['path'];

Result:
0B0urCDfD-1otYnhwUmRZVzlKcLU

If anyone is interested, I've set up up a demo project including my walkthrough and some sample code, available on GitHub:
https://github.com/ivanvermeyen/laravel-google-drive-demo 👍

The problem for me was that I used the wrong .env names.
In the .env, I had set MAIN_GOOGLE_DRIVE_REFRESH_TOKEN
But in the config/filesystems.php I used GOOGLE_DRIVE_REFRESH_TOKEN

@imannms you guessed it right but a little correction is there
You are searching the key as filename which is not having the file extension in response.
Therefore you should search for name as key in the array
function getId(Array $array, $key, $value)
{
foreach ($array as $subarray)
{
if (isset($subarray[$key]) && $subarray[$key] == $value)
return $subarray;
}
}
$listContents = Storage::disk('google')->listContents();
$id = getId($listContents, 'name', $filename);
echo $id['path'];


WIP: Update master-ip flags and vars to plural

Fixes #595
Dependent on https://github.com/kubernetes/test-infra/pull/14613, do not merge yet

Should I also cleanup https://github.com/kubernetes/perf-tests/blob/7d8c6a23b77782c1f5d7a550c3e04f8cf03fa38b/clusterloader2/pkg/config/cluster.go#L55-L69?

/assign @mm4tt

Should I also cleanup these to be plural?
https://github.com/kubernetes/perf-tests/blob/7d8c6a23b77782c1f5d7a550c3e04f8cf03fa38b/clusterloader2/pkg/config/cluster.go#L55-L69

Let's leave it as is for now. I think @alejandrox1 is working on that.

Already have this in progress here https://github.com/kubernetes/perf-tests/pull/833

/remove-lifecycle stale


missing description in json-schema

https://github.com/stac-extensions/eo/blob/d302b29262f2769c2579ebcac9caa336e75352f5/json-schema/schema.json#L147-L184
there is no description in ☝ while it's seems to be in the spec: https://github.com/stac-extensions/eo#band-object

Issue Description
I am running into this issue too. The reason for it is the schema specifies no additionalProperties are allowed, thus blocking anything outside of name, common_name, center_wavelength, full_width_half_max , and solar_illumination .
Source of Issue
"fields": {
"type": "object",
"properties": {
"eo:cloud_cover": {
"title": "Cloud Cover",
"type": "number",
"minimum": 0,
"maximum": 100
},
"eo:bands": {
"$ref": "#/definitions/bands"
}
},
"patternProperties": {
"^(?!eo:)": {}
},
"additionalProperties": false

Proposed Change

Remove this line of code from the schema "additionalProperties": false, or...
Add "description": { "title": "Band Description", "type": "string"}

It is correct that description is missing, but what has been written about additionalProperties is not correct as it doesn't apply for the content of eo:bands. You can add whatever you want to the bands without any validation complaints...

Created PR #21 to fix this.


Add Secrets to store and manage sensitive information

Add Secrets.yaml to the template manifests to manage sensitive information such as passwords and keys.
This feature replaces some settings that are currently in place with string values so this does require some refactoring.

Some research is required on how we could circumvent exposing secrets with the custom config file that requires environment variable values since the Python code fetches these values with the os.environ.get() method.

@Abhi94N let's go with your idea and create a PoC of a dumbed-down manifest to manage Postgres ourselves to see if this resolves the issues you were having with passing in passwords and secrets. We can make a decision based on the results of your PoC.

@jgwerner Tests with using manifests instead of the Postgres helm chart dependency led to the same issue where POSTGRES_NBGRADER_PASSWORD  fails to be set as an envar in the hub pod when using a kubernetes secret.

Plan of action

Create manifests for Postgres nbgrader to configure all containers
A kubernetes secret can be created for everything except for POSTGRES_NBGRADER_PASSWORD
Pass values in as envars but we can also mount the secret as well and reference it in the code itself

@jgwerner Tests with using manifests instead of the Postgres helm chart dependency led to the same issue where POSTGRES_NBGRADER_PASSWORD  fails to be set as an envar in the hub pod when using a kubernetes secret.

That makes sense, since the JupyterHub code is managing how the secrets are fetched.


Framework Error When Replacing Column with Row Using CrossAxisAlignment.stretch

Steps to reproduce
If the code works fine with Column but causes a framework error when replaced with Row, the issue might stem from the Row widget requiring additional constraints to layout its children properly, particularly when crossAxisAlignment: CrossAxisAlignment.stretch is applied.
In Flutter, Row attempts to layout its children horizontally within its available width. When crossAxisAlignment: CrossAxisAlignment.stretch is used in a Row, it stretches its children vertically, which might conflict with their layout constraints, causing an error.
child: Row( // Replace Column with Row
  crossAxisAlignment: CrossAxisAlignment.stretch,
  children: [
    Container(
      height: 100,
      width: double.infinity,
      margin: const EdgeInsets.only(left: 30),
      color: Colors.white,
      child: const Text("Container 1"),
    ),
    const SizedBox(
      height: 20,
    ),
    Container(
      height: 100,
      width: double.infinity,
      margin: const EdgeInsets.only(left: 30),
      color: Colors.blue,
      child: const Text("Container 2"),
    ),
    const SizedBox(
      height: 20,
    ),
    Container(
      height: 100,
      width: double.infinity,
      margin: const EdgeInsets.only(left: 30),
      color: Colors.orange,
      child: const Text("Hello"),
    ),
    const SizedBox(
      height: 20,
    ),
  ],
),

Expected results
The Row should layout the Container widgets horizontally without errors, applying the specified constraints.
Actual results
The application throws a framework error when using Row with crossAxisAlignment: CrossAxisAlignment.stretch.
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Row( // Replace Column with Row to reproduce the issue
          crossAxisAlignment: CrossAxisAlignment.stretch,
          children: [
            Container(
              height: 100,
              width: double.infinity,
              margin: const EdgeInsets.only(left: 30),
              color: Colors.white,
              child: const Text("Container 1"),
            ),
            const SizedBox(
              height: 20,
            ),
            Container(
              height: 100,
              width: double.infinity,
              margin: const EdgeInsets.only(left: 30),
              color: Colors.blue,
              child: const Text("Container 2"),
            ),
            const SizedBox(
              height: 20,
            ),
            Container(
              height: 100,
              width: double.infinity,
              margin: const EdgeInsets.only(left: 30),
              color: Colors.orange,
              child: const Text("Hello"),
            ),
            const SizedBox(
              height: 20,
            ),
          ],
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
[✓] Flutter (Channel stable, 3.24.5, on macOS 15.1 24B83 darwin-arm64, locale
    en-US)
    • Flutter version 3.24.5 on channel stable at
      /opt/homebrew/Caskroom/flutter/3.24.5/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision dec2ee5c1f (6 weeks ago), 2024-11-13 11:13:06 -0800
    • Engine revision a18df97ca5
    • Dart version 3.5.4
    • DevTools version 2.37.3

[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at /Users/aoue42/Library/Android/sdk
    • Platform android-35, build-tools 35.0.0
    • Java binary at: /Users/aoue42/Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 21.0.3+-79915917-b509.11)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 16.0)
    • Xcode at /Applications/Xcode-16.0.0-Beta.app/Contents/Developer
    • Build 16A5171c
    ✗ CocoaPods not installed.
        CocoaPods is a package manager for iOS or macOS platform code.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/to/platform-plugins
      For installation instructions, see
      https://guides.cocoapods.org/using/getting-started.html#installation

[✗] Chrome - develop for the web (Cannot find Chrome executable at
    /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Android Studio (version 2024.2)
    • Android Studio at /Users/aoue42/Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 21.0.3+-79915917-b509.11)

[✓] IntelliJ IDEA Ultimate Edition (version 2024.3)
    • IntelliJ at /Users/aoue42/Applications/IntelliJ IDEA Ultimate.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin version 243.22562.3

[✓] VS Code (version 1.96.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.102.0

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile)     • emulator-5554             •
      android-arm64 • Android 14 (API 34) (emulator)
    • LOAI’s Iphone (mobile)          • 00008130-00180C2C2168001C • ios
      • iOS 18.2 22C5142a
    • macOS (desktop)                 • macos                     • darwin-arm64
      • macOS 15.1 24B83 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad     • darwin
      • macOS 15.1 24B83 darwin-arm64

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

Hi @loairebouh, This is working as intended.

The Row should layout the Container widgets horizontally without errors,

Your containers should have a finite width inorder to place them in a row. Closing as solved.


feat(eth-api): Add documentation for rewardPercentiles value range.

Description

Solution

#113  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kustrun and the rest of your teammates on  Graphite


Can't verify requests when using AndroidMockServerRule

I have verified that my tests are detecting and responding to requests with mocked responses as expected, but when I try to verify the requests on the server, I can never get any matches. The same verification code when using MockServerRule (instead of AndroidMockServerRule) can correctly verify the calls made during the request. I noticed that there aren't any tests for request verification in the androidTest source set.
I did try digging through the code around dispatchers, but I could not find anything standing out to me that was an obvious problem, since the code looks pretty much identical between the AndroidDispatcherRetainer and the DispatcherRetainer.

Yeah it seems to be a problem there. I'll take a look asap during the week. Thanks for reporting!

It looks like the web server may also not be having its whenever conditions reset after using the AndroidMockServerRule.  When my tests run in isolation, they pass just fine, but I'm having a few cases when I run the full suite, it is responding in one case with something that was set in an entirely different test for the same conditions.

I think I may have found part of the problem. ServerExtensions.kt has whenever methods that set the dispatcher to DispatcherRetainer.hiroakiDispatcher, but there's nothing to set them to AndroidDispatcherRetainer.hiroakiDispatcher. Might be that AndroidDispatcherRetainer should reference the static DispatcherRetainer.hiroakiDispatcher instead of its own instance. I suspect the same goes for the queueDispatcher.

I need to take a look at this tomorrow. Otherwise you can also open a PR today if you want to and I'll review it tomorrow. Thanks for the reports.

Release is done now. You can already fetch it. Let me know if you find any further problems on it pls!

Looks good, thanks!


refactor(app-indexing): replace RelativeLayout with ConstraintLayout

Another quickstart using ConstraintLayout: App Indexing.
See discussion in #1217
Previous layout (RelativeLayout):

New layout:

@rosariopfernandes thank you!


Lower barrier to entry for your first custom plugin

I'd propose here to Support a simple default AppTx-like handler under 0.7
It might make sense to allow users to take advantage of a simplified generalized handler which somewhat replicates AppTx - solely for the purpose of lowering the boundary of entry for writing your very first plugin - right now dealing with all these onion peels could make your code feel like grated potatoes. Ultimately a plugin should not rely on this handler - but I think the current level of complexity should be eased into - we'll likely deter a whole wack-a-mole of folks by making our counter app look like it does in https://github.com/tendermint/basecoin/pull/154 right now. This counter example is still good and we should leave it in there, but maybe we also have a dummy plugin which takes advantage of a pre-designed default AppTx Handler... Thoughts?

We have the stack.NewDefault() which builds up the middleware.  The only exposure to the onion is in the tx command to wrap and sign it.
Sure, we need to refactor the commands part there, to provide some helper method for wrapping with the default middleware.
This is only a cleanup of the commands helper functions if I understand properly.

Okay, need to reassess this issue pre-release of 0.7 after the cleanup and application of new helper functions - right now it appears to complicated

I think the code is super easy now.  I just wrote a module and server and client app and unit and cli tests in around 4 hours.  I mean, it wasn't 30 minutes, but still, the code was not in my way.
I think the hardest part is for people to understand the system well enough to work at such speeds.  Thus, I think #194 would solve the remaining issues.  Gonna mark this fixed unless you think that is not enough.


Some Safe apps cannot be loaded in the desktop app

Description
Some safe apps use HTTP headers such as Content-Security-Policy to let the website being loaded as an iframe for specific websites. This strategy works well for the interface hosted on gnosis-safe.io but doesn't for the desktop app.
We also got a request for easy detection if the app is loaded as a safe app that doesn't require loading the SDK - in the web version, one can use desktop.referrer, but not on the desktop. It could be done with the query params.
Affected apps:
Zerion, 1inch, furucombo
Environment
Latest desktop app
Steps to reproduce

Try to load Zerion safe app in the desktop app

Expected result
To be evaluated by the team
Obtained result
The app doesn't load

Some safe apps use HTTP headers such as X-Frame-Ancestors to let the website being loaded as an iframe for specific websites. This strategy works well for the interface hosted on gnosis-safe.io but doesn't for the desktop app.

To fix this I suggest letting the desktop app know about such apps so it can hide them, and introducing a new boolean flag on the backend: restrictsAccessBySource

Hola guys! Unfortunately I am having a few issues when testing this PR with this build of the app that @dasanra created for me: https://github.com/gnosis/safe-react/actions/runs/1693347662
I can still see the Zerion app in the list of apps and a few others that do not open and I think should be hidden as well after this fix (I checked on mainnet):
-DODO
-1inch Network (mainnet)
-furucombo
-StakeWise
-WrappedPunks
-yearnFinance
Also apps like Uniswap, sushi or other token swapping apps in general do not seem to fetch the safe asset balance making them pretty much useless (although this looks like something to tackle in a separate issue).


Replacing compass mixins

We are using several compass mixins in this library that we need to replace with something else so we can continue to support the files.
I have made new box-sizing helpers to replace the compass reliant ones we had before in #51. Will these suffice? We could easily make a mixin to do the same thing...
In #51 I also replaced all instances of @extend %inline-block with display: inline-block
These are the remaining compass reliant mixins that I found in the library:

[ ] border-radius
[ ] box-shadow
[ ] transition (this one had several varieties with different suffixes)
[ ] translate

so do we want to write our own mixins to replace these? Or use another library, like bourbon?
@incuna/frontend discuss

I think for border-radius, we should be okay to just use the property as it is. It's supported by pretty much everything now without vendor prefixes. IE8 and Opera Mini don't support it and for those we can fall back to just normal borders. Since border-radius is purely a decorative enhancing thing, I don't think we need a mixin for it anymore.

@pandalion I would like to have a mixin even if it's just for the purpose of being able to shorthand border-top-radius or border-left-radius because I can never remember which order they should go in and it's confusing if you have different values for each corner. But that should be something we can make ourselves

@hippogriffic Do we need a replacement for gradient backgrounds too, eg @include background

ah yes, probably, I think I just listed the ones from incuna-sass here but we should put everything we need to replace. Feel free to add to the list :)

I don't think we should be maintaining our own versions of mixins that are readily available in open source projects.
For the CSS3 vendor-prefix mixins, the ones in compass-mixins or bourbon should suffice?

https://github.com/Igosuki/compass-mixins
http://bourbon.io/docs/

After discussion yesterday it looks like using node-bourbon will be the best thing https://www.npmjs.com/package/node-bourbon

the only one of these that isn't in compass is box-shadow, do we need a mixin for that? It looks like it's pretty well supported

@hippogriffic  I''d say, only for legacy project.

I don't think we need a box-shadow mixin, closing this issue


FRONTEND WITH FRAMEWORK: VueJS-2.4 => Watchers in Vue.js

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes  #2303

Type of change 📝

[x] Doc#umentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Frontend Dev #React/Angular/Vue

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@anandhu720 sir any updates about review?

@SourabhSaraswat-191939 will review your pr bro.

@SourabhSaraswat-191939 sir, I have made the following changes:

Added HTML,CSS codes in the code examples.
Explanation of working of code and adding comments.
Added References at the end.

@anandhu720, we can merge this PR.


Support istio label for service lookup from UID

To support ingress mixer integration, this PR adds support for deriving targetService from the istio label on a Pod. This is a short-term fix for a larger issue of doing proper UID --> service lookups (based on labels) meant to unblock ingress work. A follow-on PR will add service info caching and full label-based keys. A longer term effort will be required to handle multiple target.service (and source.service) derivations from a UID.
Release note:

Support targetService derivation from "istio" label to enable Mixer-Ingress integration use case.

This change is

/lgtm


Associative comparison of schema.

Francois, thank you very much for fixing this issue REALLY appreciate it.
I just wanted to corrections to the fix you proposed....
Combine function in beam should be associative so they should work when the arguments are passed in different orders. This fix attempts to do the same processing when the API schema and the document schema are in either argument position.
Just some formatting fixes, the tests you added are still the same and pass. Thank you for adding them!

Ok.
As I saw, the function is called in a parDo, and not in a Combine function, so it don't need to be associative.
But I agree that I don't code Dataflows very often :)


License for rtc2git

Hi there,
right now we are also trying to migrate our RTC Content to GIT.
Any plans for a license? We would like to use und extend your software for our one-time migration (contributing our changes back to you)
Cheers
Michael

Hi Michael,
Until today I didnt thought about licenses. I'll think about it and decide during the next days under which license I am going to put it.
Glad that my programm is going to support you on your migration. :smile:
PS: If I would decide based on my username I would choose [WTFPL] (http://www.wtfpl.net/about/)
Regards
Manuel

I've decided for the MIT-License. Thank you in advance for any future contributions :thumbsup:


TS 3.9.3 breaks HOC around generic class-based components that worked with TS 3.8.3

TypeScript Version:  3.9.3
Search Terms:

ComponentType
React generic component
Higher-order component (HOC)
Assignment of intersections
TS2322, "is not assignable to type"

Code
import { Component, ComponentType, createElement } from 'react';

class CustomLibraryComponent
 extends Component
 {
    public render(): JSX.Element {
        return
Hello world!
;
    }
}

interface CustomProperties {
    text: string;
}

const assignment: ComponentType<CustomProperties> = CustomLibraryComponent; // ERROR

tsconfig.json

{
    "compilerOptions": {
        "jsx": "react",
        "jsxFactory": "createElement",
        "moduleResolution": "node",
        "target": "ESNext",
    },
}

package.json

{
    "private": true,
    "scripts": {
        "compile": "tsc --project tsconfig.json"
    },
    "devDependencies": {
        "@types/react": "16.9.35",
        "typescript": "3.9.3"
    }
}

Additional explanations
The error can be reproduced by executing npm run compile. The example is silly, of course, but it's the most compact way to demonstrate the problem. I want to be able to decorate a class-based component with a generic type parameter. (I want to use my custom library component in different places with different type instantiations, thus removing
 from CustomLibraryComponent solves the problem described here but my actual use case requires the generic parameter, so please ignore that P is not actually used in the example above.) And I'm not interested in the assignment per se but the following, more realistic code fails due to the same reason:
function decorator(WrappedComponent: ComponentType<CustomProperties>) {
    return class HOC extends Component {
        public render(): JSX.Element {
            return <WrappedComponent text="Example" />;
        }
    };
}
const result = decorator(CustomLibraryComponent);
In my actual use case, the decorator itself is also generic but this just adds unnecessary detail and complexity here.

Expected behavior: Either TS 3.9.3 should still not complain or give me a way to fix the issue.
Actual behavior: Compiling the above code results in
error TS2322: Type 'typeof CustomLibraryComponent' is not assignable to type 'ComponentType<CustomProperties>'.
  Type 'typeof CustomLibraryComponent' is not assignable to type 'ComponentClass<CustomProperties, any>'.
    Construct signature return types 'CustomLibraryComponent<any>' and 'Component<CustomProperties, any, any>' are incompatible.
      The types of 'props' are incompatible between these types.
        Type 'Readonly<any> & Readonly<{ children?: ReactNode; }>' is not assignable to type 'Readonly<CustomProperties> & Readonly<{ children?: ReactNode; }>'.
          Property 'text' is missing in type 'Readonly<any> & Readonly<{ children?: ReactNode; }>' but required in type 'Readonly<CustomProperties>'.

const assignment: ComponentType<CustomProperties> = CustomLibraryComponent;

It's not clear to me whether this is really a problem with TypeScript, actually. The reason why I decided to report this issue here (rather than at DefinitelyTyped or simply asking on StackOverflow) is because it isn't obvious to me why TypeScript complains about Type 'Readonly<any> & Readonly<{ children?: ReactNode; }>' is not assignable to type 'Readonly<CustomProperties> & Readonly<{ children?: ReactNode; }>'. Additionally, it's clearly TypeScript that introduced the error. Using "typescript": "3.8.3", the above code works fine.
An alternative to permitting this assignment again would be to somehow tell TypeScript the type to use instead of any but I have no idea how. The following alternatives all fail with different errors:
const assignment: ComponentType<CustomProperties> = CustomLibraryComponent<CustomProperties>;
const assignment: ComponentType<CustomProperties> = CustomLibraryComponent as CustomLibraryComponent<CustomProperties>;
const assignment: ComponentType<CustomProperties> = CustomLibraryComponent as unknown as CustomLibraryComponent<CustomProperties>;

If the error could be fixed by improving the typing of React, please let me know (ideally with a suggestion how to fix it there).
Playground Link
Related Issues: I assume the error has been introduced with the pull request #37195. The only other regression I found in this regard is #38542.
Workaround: Interestingly, generic functional components still work. The above code compiles with TypeScript 3.9.3 when the CustomLibraryComponent is declared as follows:
function CustomLibraryComponent
(props: Readonly
): JSX.Element {
    return
Hello world!
;
}

Minimal non-JSX repro:
interface CustomProperties {
    text: string;
}

declare var x: Readonly<any> & Readonly<{ children?: {} }>;
declare var y: Readonly<CustomProperties> & Readonly<{ children?: {} }>;

y = x;

likely unintended fallout of our stricter intersection member checking - namely I know that Readonly<any> actually becomes {readonly [index: string]: any}, which behaves very differently than any.

We'd need to create a special-case type for Readonly<any> that behaves more any-like than an index signature does to fix this, and, without other compelling examples, I don't see us integrating something like that in the near future.


Clarify licensing

Are custom RHEL images automatically treated as Red Hat Cloud Access (BYOS/BYOL), or is there a way to create an On Demand (PAYG) custom image?

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

ID: 48bab424-986d-9922-ec2d-942508081367
Version Independent ID: 6e9b3ebd-0571-357e-0fde-ef434a728a24
Content: Create and upload a Red Hat Enterprise Linux VHD for use in Azure
Content Source: articles/virtual-machines/linux/redhat-create-upload-vhd.md
Service: virtual-machines-linux
GitHub Login: @szarkos
Microsoft Alias: szark

@dcbrown16
Thanks for the question! We are investigating and will update you shortly.

@szarkos please add your comments

Hey @dcbrown16 - all RHEL custom images brought onto Azure will effectively be BYOS. Custom image will be billed just the base compute fee just like our BYOS images, and customers are expected to use their own source of updates. The purpose of the Azure RHUI is to provide updates to our PAYG VMs, so it is not intended for use with custom images/BYOS images.
Did that help? Happy to chat more.

Great, thanks @asinn826! That will help us support customers with RHEL images. I can close for now.


Correction for raw affiliation King's University College (Western University)

Correction needed for raw affiliation King's University College (Western University)
raw_affiliation_name: King's University College (Western University)
new_rors: 02grkyz14
previous_rors: 048f5b183;05cgtjz78;0257kt353
works_examples: W4384706336;W4360993502
contact: 6abc3d96346f757dd763856615e44d9a:d5db45ab0832fbd7 @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


Map Crashes on max zoom out.

Not sure if anyone else has noticed this but I've just started using the project (and flutter in general) and on my first initial creation of the map I noticed that a full zoom out (maximum distance using a touch gesture) will cause the entire map to lock up and stop responding. This is on the current latest version 0.6.1+2.
By not responding I mean that no further gestures function at this point but calls to the map to change tiles still respond. Clicking clustered markers does nothing at this point. Programmatic calls to the mapController to zoom or move also stop working.
I first assumed something I was doing in my app was causing it so I stripped out markers and various tile options but it appears to persist at bare minimum feature set.

This seems like it's related  to #335

This is a dupe of #59


Create ABOUT files for AboutCode Toolkit

Hello,
first of all I want to thank the developers for their work, these tools are very helpful.
I have created this issue, because I have a question which might also be of interest for other users.
Reading the descriptions of the AboutCode projects on www.aboutcode.org, I got the impression that the workflow would be like this:
source -> ScanCode Toolkit -> AboutCode Manager -> AboutCode Toolkit -> attribution notice
Then I realized, that AboutCode Manager and Toolkit don't collaborate, as the Manager cannot generate ABOUT-files, only the components list as JSON.
Is this assumption correct?
If so, would you consider adding this feature?
Best regards,
Thorsten Harter

Hey @ThorstenHarter, thanks for the ticket.
Generating ABOUT files directly from AboutCode Manager is certainly on the radar as a feature down the road. We can use this ticket to tract that progress.
In the mean time, you can generate ABOUT files for Components created in AboutCode Manager, you just have to use https://github.com/nexB/aboutcode-toolkit for now. This workflow would be:

Export Components from AboutCode Manager in either JSON or CSV formats
using AboutCode-Toolkit, run: about gen ~/path/to/exported/components.csv/json ~/path/to/aboutfile/output/

Once you have these ABOUT files created, you can then generate comprehensive attribution notices as needed using the AboutCode workflow.
Some relevant AboutCode-Toolkit docs:

https://github.com/nexB/aboutcode-toolkit/blob/develop/README.rst
https://github.com/nexB/aboutcode-toolkit/blob/develop/USAGE.rst

@ThorstenHarter:
The default location where AboutCode-Toolkit looks for the mapping.config file (assuming you cloned/downloaded AboutCode-Toolkit from github.) is in:
src/attributecode/mapping.config

You can modify this file or replace it with your own created mapping.config.
This is certainly not documented well, and I will open a ticket in AboutCode-Toolkit for this specifically.
For your specific example:

modify/replace the mapping.config file in the aboutcode-toolkit/src/attributecode/ directory, with atleast the line: about_file_path: Path
run the command ./about gen --mapping ~/path/to/components.csv ~/path/to/aboutfile-output-dir/

The ABOUT file(s) should then be successfully generated. Also note that you should export Components from AboutCode-Manager in CSV format for now.
While I stated previously that JSON files can be used by AboutCode-Toolkit, I do not think AboutCode-Manager's custom JSON component export is in the correct format to be readily accepted by AboutCode-Toolkit. I will confirm this an open up issues in the correct projects.
Hopefully this is a bit more clear now. Give it a try and let me know if all goes well! We appreciate getting the feedback; every bit helps us improve our AboutCode tools.

@majurg
Editing the mapping.config at the specified location solved the problem of the missing 'about_file_path' field, thanks for the tip!
The gen command generates now the ABOUT-files.
There was one warning though: WARNING: Field modified is present but empty
The "Modified" column in the CSV is set to "true" or "false" for each component, so I don't know how this warning could be silenced.
Creating the attribution notice from these ABOUT files worked for the simple cases with only one license and copyright.
In cases where I added multiple licenses and copyrights in AboutCode Manager however, these multiple entries are merged to one string in the CSV export, so that the AboutCode Toolkit cannot separate them.
Example: license_name: MPL-2.0MINPACKBSD-3-Clause
Having multiple licenses and a long list of copyright holders is a common use case, e.g. in the Eigen library: https://github.com/libigl/eigen
I'm not sure how this could be handled if the AboutCode Manager would export ABOUT files.
On the one hand, there is the 'license_expression' field, which let's you concatenate licenses with AND/OR, but on the other hand you can only specifiy a single 'license_file' or 'license_url'.
For the copyrights, it would be nice if each entry appears on the separate line in the attribution notice, this can be done with the following syntax in the ABOUT file:
copyright: |
  Copyright (c) 2017 Alice
  Copyright (c) 2018 Bob

I've created an ABOUT file in a recent PR - https://github.com/nexB/scancode-workbench/blob/fix/packaging-structure/workbench.ABOUT
CC @pombredanne @AyanSinhaMahapatra


Catch Throwable in yii\base\ErrorHandler::handleException()

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#12619

Fixes #12619
https://3v4l.org/DlA8p
https://3v4l.org/FL1fa

when doing a CTRL F through the framework, I see in total 47 occurrences of php catch (\Exception $e)
It would be nice to fix all in 1 commit

I don't think anything else needs to be fixed. We should not blindly catch all Throwable in framework .

could you elaborate on your decision to do this for ErrorHandler & Widget only?

Both are for easier debugging. Not really sure about Widget, I'm actually considering reverting it and leave this only for ErrorHandler.
Related discussion: https://github.com/symfony/symfony/pull/18765#issuecomment-219031688

What's the point doing that in a widget?

What's the point doing that in a widget?

I thought that without this output will be corrupted on PHP error on widget rendering, but it was a wrong assumption. I will remove it.

Merged. Thank you!


[Question] Current state is not reflected using refresh, when tree is filled with async requests

Consider the following scenario:

The tree is rendered with an initial set of nodes;
A set of service requests are launched at the same time and the tree data is being updated as requests arrive, each update followed by the use of the API refresh method;
If you open a given node in the meantime, it will collapse automatically after a refresh operation, as the state obtained (within refresh method) does not reflect the current state of the tree;

Do you have any clue on whats happening or how to overcome it?
Thanks in advance!

The sole purpose of refresh is to refresh (replace) the local state with a fresh copy from the remote. As such by design all local changes that were not persisted are lost. Let me know if I misunderstood and I will reopen.


unreproducible classification accuracy for 10 tasks

Hi, thanks for sharing the code.
I am trying to reproduce your work. For both datasets (mnist and svhn), the classification accuracy for the 5 tasks are the same as in the paper. However, for the 10 tasks, I only got ~0.94 for the mnist and ~0.63. for svhn.
Is there anything missing in the code in order to reproduce your results in the paper?

Sorry, it is reproduced by increasing iteration to 8001 for mnist.

Why the accuracy of my running results is close to zero for the previous categories？ That is, each prediction is predicted as the last category.


Adds possibility to overwrite EntityManager

In order to customize the EntityManager class it is necessary to use "static" in its create() method.

Whilst the EntityManager is not marked as final yet, it is our intention to do that in the near future (before 3.x).
Opened https://github.com/doctrine/doctrine2/issues/7105 to make this explicit.

Thanks for your quick reply. Could you elaborate on the reasons for this?

@CloudrexxTeam the EntityManager has always been "soft-final":
https://github.com/doctrine/doctrine2/blob/v2.6.1/lib/Doctrine/ORM/EntityManager.php#L63
Please consider reading https://ocramius.github.io/blog/when-to-declare-classes-final/

Ok. So if I understood correctly we are left with the following options:

Create a "new" EntityManager, using the Interface.
Propose the change in a generalized manner as a pull-request.

Assuming that the above is correct and concluding (otherwise any input is welcome):
We want to enable result cache for queries for all SELECT queries by default. Currently we have overwritten the createQuery() method to achieve this.
For such a change, should we preferably create a pull-request which solves this in a generic way (probably by introducing some kind of option whether to use result cache for all (select) queries by default) or are we to create a complete copy of EntityManager with just the change in createQuery()?
Thanks for your patience with my questions!

We want to enable result cache for queries for all SELECT queries by default.

This is already done by the L2 cache, but enabling by default for all queries is NOT endorseable.
What you can do is writing an EntityManagerInterface implementation that overrides createQuery() and createQueryBuilder() by adding a call to AbstractQuery#setCacheable(): https://github.com/doctrine/doctrine2/blob/bbc1ed93b14e079cd403cb5b2b1134da35af431e/lib/Doctrine/ORM/AbstractQuery.php#L174
This wouldn't be something to be merged in core though, as "cache-everything-by-default" is dangerous and domain-specific.

Ok, just to make sure: This equals the first option I listed (create an almost exact copy of EntityManager)?

Yes, you'd compose/decorate an existing EntityManager instance within your own implementation.


Panic with conditions that contains Go keywords or name starts from number

When matching objects against rules with conditions/fields that contains Go keywords or starts from number, library goes panic. Example of such rules:
expression: '115test == "test"'
expression: 'go == "test"'
expression: 'type == "test"'
The stack trace
runtime.fatalpanic (panic.go:1141) runtime
runtime.gopanic (panic.go:987) runtime
runtime.panicmem (panic.go:260) runtime
runtime.sigpanic (signal_unix.go:837) runtime
engine.(*RuleEngine).MatchEvent (engine_api.go:166) github.com/atlasgurus/rulestone/engine
server.(*BaseRulestoneServer).PerformMatch.func1 (base_server.go:118) github.com/atlasgurus/rulestone-server/server
actors.(*Actor).actionLoop (actor.go:38) github.com/atlasgurus/rulestone/actors
actors.NewActor.func1 (actor.go:31) github.com/atlasgurus/rulestone/actors
runtime.goexit (asm_amd64.s:1598) runtime
 - Async Stack Trace
actors.NewActor (actor.go:31) github.com/atlasgurus/rulestone/actors

I can't reproduce the issue, please create a proper test.
/usr/local/opt/go/libexec/bin/go tool test2json -t /Users/vvg/Library/Caches/JetBrains/GoLand2023.2/tmp/GoLand/___TestFilterApiExpression1_in_github_com_atlasgurus_rulestone_tests.test -test.v -test.paniconexit0 -test.run ^\QTestFilterApiExpression1\E$
=== RUN   TestFilterApiExpression1
engine_api_test.go:45: failed NewRuleEngine: 1:4: expected 'EOF', found test
--- FAIL: TestFilterApiExpression1 (0.00s)
FAIL
Process finished with the exit code 1

I was testing with rulestone-server and it goes panic because rule engine was not initialized because rules were not parsed correctly. So panic itself is a result of lack of error handling in rule-server and I will fix it there.
But as you can see in your test run, Rulestone does not support event fields that starts from number or equal to Go keywords. We need to support these field names.

Pushed issue_7_keywords_numeric_fields branch with the test case.


Codefix sa1203 take2

Fix for #1104
New branch, with the proper commits. Maybe the flag enum solution can work for other tests as well. I haven't implemented this in the analyzer, it's now just used for the code fix.

Fixed #1106 as well, using the same helper class

@hvanbakel I haven't reviewed the code, but I wanted to talk a bit about the way it behaves. In my mind, the behavior to fix a single violation on member MemberName would be to move MemberName up until it reaches a position where the violation is no longer reported. Does this describe the way you have currently implemented it?

Fix added for #1113

:thought_balloon: We might want to fix the spacing between items if we move a field to be the first member. But i think this can be done in a different PR.
When the issue with the copied trivia is resolved I think this can be merged :+1:

Latest comments by @pdelvo are now fixed

:+1:

Rebased and merged as commit 907c45fcebf6791ef436d3935280e6660f9b0639.


Added ability to flush Zipkin spans to Kafka Exporter

If running an actual Kafka backed Zipkin installation this update allows one to send the flow output to Kafka for direct injection into Zipkin using the Kafka collector.

nice this looks like a nice interim step towards flowing to kafka during the simulation #67

yes, I didn't want to implement live flowing yet as it would mean more refactoring and adjusting of the current code base. More than happy to build it but would like some comments from @adrianco on expected direction first.
I was thinking making the flow package an interface so we could have different output stream implementations.

Thanks! I'll take a look over the next few days.


Concept pages for subjects and people

Link to feature: https://wellcomecollection.productboard.com/feature-board/planning/features/4991761


openpose2 installing

hi ,
when I write $ python depthai_demo.py -cnn openpose2
in the command
I get this problem
"exit_code": 1,
"message": "Command failed with exit code 1, command: /opt/intel/openvino/deployment_tools/inference_engine/lib/intel64/myriad_compile -m /tmp/blobconverter/feea082f408c45eeae0d62e9c8360f42/openpose2/FP16/openpose2.xml -o /tmp/blobconverter/feea082f408c45eeae0d62e9c8360f42/openpose2/FP16/openpose2.blob -c /tmp/blobconverter/feea082f408c45eeae0d62e9c8360f42/myriad_compile_config.txt -ip U8",
"stderr": "The support of IR v5 has been removed from the product. Please, convert the original model using the Model Optimizer which comes with this version of the OpenVINO to generate supported IR version.\n",
"stdout": "Inference Engine: \n\tAPI version ............ 2.1\n\tBuild .................. 2021.3.0-2787-60059f2c755-releases/2021/3\n\tDescription ....... API\n"
}
Traceback (most recent call last):
File "C:\Users\HI TECH\Desktop\depthai\depthai_demo.py", line 574, in
nn_manager = NNetManager(
File "C:\Users\HI TECH\Desktop\depthai\depthai_demo.py", line 198, in init
self.blob_path = BlobManager(model_dir=self.model_dir, model_name=self.model_name).compile(conf.args.shaves)
File "C:\Users\HI TECH\Desktop\depthai\depthai_helpers\config_manager.py", line 313, in compile
self.blob_path = blobconverter.compile_blob(
File "C:\Users\HI TECH\AppData\Roaming\Python\Python39\site-packages\blobconverter_init_.py", line 252, in compile_blob
response.raise_for_status()
File "C:\Users\HI TECH\AppData\Roaming\Python\Python39\site-packages\requests\models.py", line 941, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: BAD REQUEST for url: http://luxonis.com:8080/compile?version=2021.3
what should I do ?

I think I saw this issue raised elsewhere and then closed.  Are you still experiencing the problem?

yes . I still stuck

Sorry about the trouble.  I think @VanDavv should be able to help when he wakes up tomorrow (he's in Europe, and wrote the converter that seems to not be responding here).

thank you. Great! waiting for this

@Luxonis-Brandon  Hi, could you please find out if VanDavv or anyone can help ? thank you

Asking.

Oh, sorry @R-abodyak. I've missed that. Will investigate further and circle back

@R-abodyak the model had to be compiled using the older OpenVINO version (2020.4). I added support for a custom version number in #398 and with that PR merged you should be able to start the network.
It doesn't have a handler migrated yet though, so the results will be present only as a console print at the moment. I will add it, but you can add your own handler file if you'd like to experiment with it - simply create a python script similar to this one and add a handler config like here pointing to this file

Actually, I managed to migrate the handler from Gen1 to work with our handler format - added it to the PR (#398). This means that when that PR is merged, python3 depthai_demo.py -cnn openpose2 will work and display results

@VanDavv yes it worked!  Thank you ,, Thank you from the bottom of my heart! <3

Happy to help @R-abodyak :heart:


Append "Job" to the end of webhook class name if it's not there

Coverage decreased (-0.1%) to 99.655% when pulling a2eecd8cb3c4aec7ea69bdaffdfc87abefbb2abc on seka19:master into b6dde2f935c6ede7ee555b465c9a5f902b571cc5 on ohmybrew:master.


Page offline https://github.com/harishanand95/diffusers/blob/main/examples/inference/save_onnx.py

Describe the bug
Hi Im using an tutorial to work on this with amd gpu and the project give an error because file post_quant_conv.onnx dont exist. Can you help me please?
Reproduction
No response
Logs
No response
System Info
1.4 diffuser version from this tutoria lhttps://rentry.co/ayymd-stable-diffustion-v1_4-guide

Could you try this approach?
https://gist.github.com/harishanand95/75f4515e6187a6aa3261af6ac6f61269


[Support] How to setup a contact sensor?

Describe Your Problem:

I have a switch that I'd like to convert to contact sensor. But I can't figure out how to do it reading the docs. I always get errors about trying to read properties of undefined.
Could you help me please?
Link to Logs:

Paste of Logs:

Cmd4 Config:

 {
                    "type": "Switch",
                    "displayName": "Apple TV Power",
                    "on": "FALSE",
                    "queue": "A",
                    "polling": true,
                    "state_cmd": "bash /volume1/homebridge/appletv_powerstate.sh Get 'Apple TV Power' 'On'"
                },

Screenshots:

Environment:

Node.js Version:
NPM Version:
Homebridge Version:
homebridge-cmd4 Version:
Operating System: Raspbian / Ubuntu / Debian / Windows / macOS / Docker / other
Process Supervisor: Systemd / init.d / pm2 / launchctl / Docker / hb-service / other / none

Hi,
The Get/Set DeviceName and action is sent by Cmd4.
Your config.json should read:
{
"type": "Switch",
"displayName": "Apple TV Power",
"on": false,
"polling": true,
"state_cmd": "bash /volume1/homebridge/appletv_powerstate.sh"
},
If you set "debug":true, then you would see that for an "on" event,
Cmd4 would execute your shell command with the following arguments
(Already filled in) for a "Get" event
bash /volume1/homebridge/appletv_powerstate.sh Get "Apple TV Power" On
For a "Set" event Cmd4 would execute:
bash /volume1/homebridge/appletv_powerstate.sh Set "Apple TV Power" On 1
// or
bash /volume1/homebridge/appletv_powerstate.sh Set "Apple TV Power" On 0
You do not need a queue. Queue is primarily for devices which can only
accept one command at a time, primarily write commands for devices
that take a long time to respond. I wrote it for the AdvantageAir.
Boolean characteristic values for "on" should be true/false as per
config.json rules, though Cmd4 will translate them for you.
If you want to have more specific polling you would do:
{
"type": "Switch",
"displayName": "Apple TV Power",
"on": false,
"polling": {
    "characteristic": "on", "timeout": 3000, "interval": 5

 }.
 "state_cmd": "bash /volume1/homebridge/appletv_powerstate.sh"

},
where:
timeout is in milliseconds
interval is in seconds
You should check your script by executing it from the terminal command
prompt first
bash /volume1/homebridge/appletv_powerstate.sh Set "Apple TV Power" On 1
bash /volume1/homebridge/appletv_powerstate.sh Set "Apple TV Power" On 0
rc=bash /volume1/homebridge/appletv_powerstate.sh Get "Apple TV Power" On
echo $rc
I hope this helps. Everything I wrote here is standard shell
programming. Specifically passing arguments to a shell script.
As a hint to new shell script users, I would suggest setting your
device name to 'Apple_TV_Power" as arguments can be tricky with spaces
in them.
Take care,
John
On Sat, Dec 10, 2022 at 3:57 AM Federico Zivolo @.***>
wrote:

Assigned #134 https://github.com/ztalbot2000/homebridge-cmd4/issues/134
to @ztalbot2000 https://github.com/ztalbot2000.
—
Reply to this email directly, view it on GitHub
https://github.com/ztalbot2000/homebridge-cmd4/issues/134#event-8004112541,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSBCX5P5WL4BBHTUKMVV3DWMRAYBANCNFSM6AAAAAAS2EPVMQ
.
You are receiving this because you were assigned.Message ID:
@.***>

Thanks for the answer, but I still don't understand how up set it up as Contact Sensor?

If you are still having issues, just reply to this closed ticket


refactor(coverage): remove unused run_sync_v8_test262_status

#6704  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite


Add missing dependency to //tensorflow/compiler/tests:xla_test for ROCm

The following tests needs tensorflow.python.platform.sysconfig //tensorflow/compiler/tests:matrix_solve_op_test_cpu //tensorflow/compiler/tests:matrix_solve_op_test_cpu_mlir_bridge_test

Could you point me to where //tensorflow/compiler/tests:matrix_solve_op_test_cpu and //tensorflow/compiler/tests:matrix_solve_op_test_cpu_mlir_bridge_test need/import tensorflow.python.platform.sysconfig?
From https://github.com/tensorflow/tensorflow/blob/master/tensorflow/compiler/tests/matrix_solve_op_test.py I don't see any imports of sysconfig

Could you point me to where //tensorflow/compiler/tests:matrix_solve_op_test_cpu and //tensorflow/compiler/tests:matrix_solve_op_test_cpu_mlir_bridge_test need/import tensorflow.python.platform.sysconfig?
From https://github.com/tensorflow/tensorflow/blob/master/tensorflow/compiler/tests/matrix_solve_op_test.py I don't see any imports of sysconfig

Sorry for the confusion. The context was that the recent upstream refactoring made the //tensorflow/compiler/tests:matrix_solve_op_test_cpu test in our ROCm fork fail (https://github.com/ROCmSoftwarePlatform/tensorflow-upstream) due to missing dependency. So I added the fix and would like to upstream. However, I didn't realize that the previous change in this test to disable cublasLt was never upstreamed. Now I have just added that commit in this PR as well.

Thanks for the context. Now that this is modifying XLA flags, I don't think that I am the right reviewer. @gbaned Could you find another reviewer for this PR?

Hi @wenchenvincent Can you please take look on below internal errors. Thank you!
ERROR: Strict deps violations: //tensorflow/compiler/tests:matrix_solve_op_test_cpu
In third_party/tensorflow/compiler/tests/matrix_solve_op_test.py, no direct deps found for imports:
line 26: from tensorflow.python.platform import sysconfig: Module "tensorflow.python.platform" not provided by a direct dep
Use --define=PYDEPS=<OFF|WARN> to temporarily change how strict deps are enforced.
Bug? http://go/py-strict-deps-bug
*** Please fix target dependencies. ***
build_cleaner //third_party/tensorflow/compiler/tests:matrix_solve_op_test_cpu


chore: update gitops package and go version

Signed-off-by: ankitm123<EMAIL_ADDRESS>fixes #31

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jstrachan
You can assign the PR to them by writing /assign @jstrachan in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

/cc @jstrachan

/lgtm

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

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

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


Feedback from Pun

Hi @rruiliu1005
Here is my feedback on your research proposal!
The topic is interesting as it will explore the interconnectedness among the four economies. This is also useful for policymakers who need to monitor and think beyond not only the domestic market but also the foreign market in order to conduct policy effectively. The main strength of your research proposal is that you provide justification for each variable that you will include in the model for further analysis. This strengthens your standpoint and shows how you manage variables based on your perception of the interconnectedness within these countries.
For the content in the report, it would be beneficial to introduce your readers to the characteristics of each country before delving into deeper analysis. I encourage you to include plots of data on a nominal scale. For instance, plotting the CPI on a level data will provide you and your readers with a sense of inflation among these countries, making comparisons easier. Additionally, plotting FDI will offer insight into how each country attracts investment from around the world and may provide you a sense of the size of its economy. And in the same direction for other variables as well.
For further improvement in the coding part of the index.qmd file, firstly, I notice that you use the figure plot from outside the system, and this may lead you to work redundantly across platforms. I encourage you to analyze all of the data in the same platform as well as provide codes that you manipulated with the data, as it enhances efficiency and also reduces the chance of error. Secondly, as you mentioned, you use all five variables for each country from the IMF. The IMF also provides a package called ‘imf.data’, which is similar to ‘readrba’ and ‘readabs’. You can have a look at https://cran.r-project.org/web/packages/imf.data/imf.data.pdf This package allows you to retrieve the data directly from their website without downloading the data to your hard drive first. This also facilitates your readers who are interested in your topic and want to replicate your code for their further analysis. Finally, for the exchange rate, as you used USD as the baseline, thus, in this figure, the USD exchange rate will not have any data, and therefore should not be included in the figure as it might mislead the reader.
To-do list

[x]  Include plots on the level data of all variables
[x]  Analyze all of the data in the same platform (provide the coding and plot figures)
[x]  Use package ‘imf. data’ to retrieve the data from the IMF
[x]  Exclude the USD exchange rate in the exchange rate figure

Cheers!
Pun

Hi Pun,
Thank you very much for your feedback. I had initially included my codes in the .R file in the repository instead of the .qmd file. I did use the same package that you suggested so I've now put my code into the .qmd file. I have also incorporated your suggestion to include plots of the untransformed variables and removed the USD exchange rate from the exchange rate figures.
Many thanks,
Rui


Interactive mode

Use / for diving into folders, Enter to download file or whole folder.
Possible solutions via fzf/skim augmented navigation.

Done.


Update MVC View Page

https://docs.microsoft.com/en-us/aspnet/core/tutorials/first-mvc-app/adding-view
search for MVC View Page
Per customer:
When you click on the Razor View it says on the right pane "MVC View Page". I checked online and some people are confirming this was changed to Razor View on version 15.6 of VS 2017.

Fixed in #5995


suppress Wine errors

https://godbolt.org/g/oBouwM
003c:fixme:msvcrt:_configure_wide_argv (2) stub
003c:fixme:msvcrt:_initialize_wide_environment stub
003c:fixme:ntdll:EtwEventRegister ({3124649c-36c0-5879-4b0e-fe163528322b}, 0x14002e264, 0x14005e870, 0x14005e890) stub.
003c:fixme:ntdll:EtwEventSetInformation (deadbeef, 2, 0x140056a7c, 28) stub
...

Looks like WINEDEBUG=-all could do the trick.
https://wiki.winehq.org/Wine_Developer's_Guide/Debug_Logging#Controlling_the_debugging_output

Great stuff: I was doing some (ineffective!) filtering of the output. Thanks!

Thank you for Compilet Explorer! :)

Nice it works!
Now the only reason msvc is marked yellow in the conformance view is the printed source filename example.cpp. Looks like that can't be disabled and needs to be filtered: https://stackoverflow.com/questions/1453767/prevent-cl-exe-from-printing-the-compiled-source-file
(or a missing -nologo switch, maybe that could be passed automatically)

Opened #1034 to track this (also was confused by the above note that @RabsRincon referenced the bug in a change I committed? weird!)


Log in screen never loads/pops up as blank screen

**Storage Explorer Version: 0.9.6 **
OS Version: Mac
Steps to Reproduce:

Launch Storage Explorer
1a. Disconnect from Internet (This step is just to get it to consistently reproduce)
Perform action that would cause login screen to load (add account, refresh credentials).

Expected Experience:
Page launches successfully or shows proper error message when it fails to load.
Actual Experience:
Blank screen immediately with no indication of failure.

In 1.0.0 it should propagate errors to help diagnose causes. Since it shouldn't launch any "blank" screens on error, I'm resolving this bug. We might see some related bugs in the future with the propagated error message.


Add Preparatory in codetrek module

Targets #2704
Description
Add preparatory dropdown in codetrek module
Checklist:

[x] I have performed a self-review of my own code.

Had to add dropdown by this type only