megaparsec 6.0

megaparsec-6.0.0 (Mark Karpov<EMAIL_ADDRESS>@mrkkrp) is out of bounds for:

[ ] cue-sheet-0.1.1 (>=5.3 && < 6.0). Mark Karpov<EMAIL_ADDRESS>@mrkkrp. @mrkkrp. Used by: library
[ ] dotenv-<IP_ADDRESS> (>=5.0 && < 6.0). Stack Builders<EMAIL_ADDRESS>@stackbuilders. @jsantos17 @jsl @sestrella. Used by: library, executable, test-suite
[ ] hledger-1.3 (>=5.0 && < 5.4). Simon Michael<EMAIL_ADDRESS>@simonmichael. @simonmichael. Used by: library
[ ] hledger-lib-1.3 (>=5.0 && < 5.4). Simon Michael<EMAIL_ADDRESS>@simonmichael. @simonmichael. Used by: library, test-suite
[ ] hledger-web-1.3 (>=5.0 && < 5.4). Simon Michael<EMAIL_ADDRESS>@simonmichael. @simonmichael. Used by: library
[ ] hspec-megaparsec-0.3.1 (>=5.0 && < 6.0). Mark Karpov<EMAIL_ADDRESS>@mrkkrp. @mrkkrp. Used by: library, test-suite
[ ] inflections-<IP_ADDRESS> (>=5.1 && < 6.0). Stack Builders<EMAIL_ADDRESS>@stackbuilders. @jsantos17 @jsl @sestrella. Used by: library, test-suite
[ ] isotope-<IP_ADDRESS> (==5.*). Michael Thomas<EMAIL_ADDRESS>@Michaelt293. @Michaelt293. Used by: library, test-suite
[ ] language-puppet-<IP_ADDRESS> (==5.*). Simon Marechal<EMAIL_ADDRESS>@bartavelle. @bartavelle. Used by: library, executable, test-suite
[ ] language-thrift-<IP_ADDRESS> (>=5.0 && < 6.0). Abhinav Gupta<EMAIL_ADDRESS>@abhinav. @abhinav. Used by: library, test-suite
[ ] stache-0.2.2 (>=5.0 && < 6.0). Mark Karpov<EMAIL_ADDRESS>@mrkkrp. @jsantos17 @jsl @sestrella. Used by: library, test-suite, benchmark
[ ] versions-3.1.1 (>=4 && < 6). Colin Woodbury<EMAIL_ADDRESS>@fosskers. @fosskers. Used by: library

versions updated.

Entirely redundant with the above, but do also note:
hspec-megaparsec-1.0.0 (Mark Karpov<EMAIL_ADDRESS>@mrkkrp) is out of bounds for:

[ ] cue-sheet-0.1.1 (==0.3.*). Mark Karpov<EMAIL_ADDRESS>@mrkkrp. @mrkkrp. Used by: test-suite
[ ] dotenv-<IP_ADDRESS> (>=0.2 && < 0.4). Stack Builders<EMAIL_ADDRESS>@stackbuilders. @jsantos17 @jsl @sestrella. Used by: test-suite
[ ] inflections-<IP_ADDRESS> (==0.3.*). Stack Builders<EMAIL_ADDRESS>@stackbuilders. @jsantos17 @jsl @sestrella. Used by: test-suite
[ ] stache-0.2.2 (>=0.2 && < 0.4). Mark Karpov<EMAIL_ADDRESS>@mrkkrp. @jsantos17 @jsl @sestrella. Used by: test-suite

My parser seems to break when doing a naive move to v6. It will take me a while to find out why.

Some porting notes, perhaps better here than on reddit:
added where needed:
import Text.Megaparsec.Char

replaced Dec with MPErr and added:
-- build with old or new megaparsec
#if MIN_VERSION_megaparsec(6,0,0)
type MPErr = ErrorFancy Void
#else
type MPErr = Dec
#endif

added these as Text.Megaparsec.String/Text.Megaparsec.Text seem to have gone away:
type SimpleStringParser = Parsec MPErr String
type SimpleTextParser = Parsec MPErr Text

The library itself recommends setting the error type (where MPErr is used above) to Void if its not needed.

Here's a Text.Megaparsec.Compat module, useful for supporting megaparsec <6 and >=6.

@bartavelle If you have any trouble switching please open an issue, I'll help.

@fosskers: this was unfortunate, but I removed "versions" from the build plan since it could not satisfy both the new version of base (which we are upgrading to now) and the old version of Megaparsec (which we are upgrading later).

@mrkkrp I'll try to pinpoint the problem next week. The only meaningful change I did was changing an integer into a decimal, but this gave really weird results (change in whitespace handling, 0x800 parsed as 0). I am sure I did something wrong, so will do that when I have plenty of time to do it properly.

@DanBurton but I bumped my base upper bound too... :(

Updated:

hspec-megaparsec
stache
cue-sheet

@mrkkrp Turned out my parser had bugs, but they did not surface with megaparsec 5!

released language-puppet-1.3.9

Fixed for dotenv in version <IP_ADDRESS>

inflections-hs updated! inflections-<IP_ADDRESS> in hackage

Can we perhaps update the checkboxes in the first post?

Checkboxes updated

@mihaimaruseac inflections and dotenv-hs are already updated. Could you please give check our packages in the todo? or are we missing something else?

I see they are updated already.

Yeah. I see they are already checked. Thanks! @mihaimaruseac

I would love to see megaparsec updated in stackage-nightly. It looks like many packages are currently on hold on nightly waiting for the others to make the megaparsec update. As time goes on, this seems less fair ;-)

You are right! and I am sorry for being the laggard here. I updated the hledger packages early but have been hung up on the release process. I will release them today.

@simonmichael totally cool: will get them right back as soon as the releases have been made. Thanks!

Thanks @cdornan.

Megaparsec-6 has been blocked for 4 weeks so we have decided to (temporarily we hope) evict the remaining blockers and restore them as soon as they are compatible with megaparsec-6.
I have raised issue in the three projects concerned:

hledger (though see above);
isotope
language-thrift

Now that Megaparsec 6 is in nightly builds, can we allow newest stache (1.1.1)?


docs: invalidAction is no longer a function

Tiny change.

Should probably also update the multiplayer example

Great point. You wanna do it and merge?

Well, actually I think we should reconsider this change. I don't really see the gain in this breaking API change, however I do see a downside in that stack trace will be pointing to the wrong place which would harm certain debugging cases in the future.


是否能提供查询API余额的功能？

这是一个python示例：
import requests

session_key = "Bearer sess-xxxx"
url = "https://api.openai.com/dashboard/billing/credit_grants"
headers = {
    "Content-Type": "application/json",
    f"Authorization": f"Bearer {session_key}"
}

response = requests.get(url, headers=headers)
print(response.json())

OK. I will add this recently. Thanks for your comment.

可以从另一个相关的网址查询到每个月的限额：https://api.openai.com/dashboard/billing/subscription
并且使用这个网址可以查询到每个月的使用量：https://api.openai.com/dashboard/billing/usage?end_date=2023-05-01&start_date=2023-04-01

Thanks for your comments.
I checked these APIs, but maybe they should not be added here because they are from the request in browser and not included in the OpenAI API Document. So I don't think they are stable enough. If one day these APIs fail, it may cause some accidents.


More clarity on id-token: write

The README includes this comment: https://github.com/actions/deploy-pages/blob/d6db90164ac5ed86f2b6aed7e0febac5b3c0c03e/README.md?plain=1#L32
Could we get more information on this?
It looks pretty scary. id-token: write implies that there is some deep and mysterious extra permission being granted to this workflow, potentially involving the ability to make writes and involving a token which might have permission to do other privileged things, maybe even at the account or organization level.
It's really hard to figure out the security impact of this configuration from existing GitHub documentation.

Related issue full of confused people:

https://github.com/github/docs/issues/14626#issuecomment-1472892737

Another issue about how confusing this is, closed without resolution:

https://github.com/github/docs/issues/25952

This is my fault.
As I understand it, any contributor with write access could write to a repo a script which uses that token for nefarious purposes.
It's also very hard to lock-down once you've enabled the github OIDC as an AuthN provider; with a third-party provider.
The AuthZ you configure on the trusted (acting as) user is set, in the external system; so if I were to have a repo and manage it's infrastructure as code with Amazon AWS; then there are not great ways (I think environment is one) to restrict other workflows from say deleting an RDS. Recently a co-worker on a private repo, accidentally deleted a cognito user pool because they forgot to rebase.
Not the type of change or risk I would like to invite for the discussed use-case of deploying github pages sites. The convenience seems to be at the cost of needing to make sure nobody points the barrel at anything you value.
I Believe even less teams will use this responsibly than for novelty purposes; so I'd mentioned publicly to Simon that I am deeply distrustful of public repos using this as a mechanism.
I would like to seek clarification on the claims and grants that id-token: write conveys to the running action.

Is it locked to the repo?
What guarantees does GitHub provide regarding the lifetime of a token, and it's revocation?
Is it expected that the JTI cannot be used to prevent close-timing attacks against accounts with this
Is the third party expected to as well as verifying the key, check iat, nbf, perform any kind of handshake with github to validate that the jti is not revoked?

In the same issue Simon just linked, there is a link to some docs which state that none and read are essentially the same thing. Seemingly like a boolean database column that is allowed to be null, two of the represented states are treated as not having access. But what it means to "have access" is also unclear, as if I must use this token type for pushing to github-pages via a github action; I'd like to make sure I understand what else it is capable of internally to GitHub. Can it for example write to another repo; can it be used to enumerate users of a repo, which might be leaked elsewhere to help a more skillful attack?

Another closed issue about this (apparently im collecting them now):

https://github.com/github/docs/issues/26481

I filed an issue here with my own attempt at explaining this feature:

https://github.com/github/docs/issues/32320

There is even a pages: write. So here is the weird thing. What the heck do I need a token for?
When I use contents: write I don't need an id-token alongside it to commit to a repo, so why does pages: write need id-token: write?

👋 I cannot talk for the questions about OIDC itself (though they are good ones and I see you have opened issues against the right repos already to get the confusions addressed).
I can talk about GitHub Pages' specific use case for an OIDC token.
First, you need to be aware of GitHub's automatic token authentication. This is the default GitHub API token (we usually refer as GITHUB_TOKEN) a workflow is given that allows it to do certain operations. It's an opaque token that only GitHub understands.
The id-token: write permission allows the GITHUB_TOKEN to make an API call to generate an OIDC token (which is an extra security token, a standard one). The OIDC token is not dangerous in itself. It is minted and signed by GitHub and includes "claims" (more on that later). What's potentially dangerous is what a third-party may be configured to do with said OIDC tokens.
To start a Pages deployment today, the actions/deploy-pages Action makes an API call. Like all authenticated GitHub APIs, it needs a GitHub API token (e.g. GITHUB_TOKEN) and it requires pages: write permission because we consider a deployment a change to a Pages site.
In the context of building and deploying a Pages site with Actions, a regular token like GITHUB_TOKEN is not sufficient. While the token is opaque it does not include specific "context" Pages needs. If you set the source branch to say gh-pages, how would you prevent a deployment from targeting another branch? There are various ways to handle this but for Pages we decided to leverage OIDC tokens. Claims they come with includes: repo name, environment, branch, etc. A lot of context. The OIDC token is used to make extra security checks (ensure a deployment is being requested via GitHub Actions, that it targets a valid environment and a valid branch when applicable).
In a perfect world, the Pages deploy API would not require an extra token. Unfortunately GitHub's API tokens predates the OIDC specification!

Thanks for the explainer @yoannchaudet I'll be interested to see what the other permissions of id-token are or if it's just "hey, it's me GitHub". There must be lots more information in there, otherwise AWS etc would be wide open from any github user

@Lewiscowles1986 The OIDC token is actually a big "hey it's me GitHub" (I like that image and I think you were spot on). It has no "permissions" associated on its own but it is minted in the context of an Action job and comes with a bunch of context. It's for third-party to decide if a token is allowed to do anything (say do something in AWS or Azure) – it's a trust relationship between GitHub and a third-party. GitHub Pages does use the token (like a third-party) to gate deployment but not alone, it has to be paired with a regular GitHub API token.
They actually document further what the claims look like here which I find useful: https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/about-security-hardening-with-openid-connect#understanding-the-oidc-token.
Anyhow, I agree that we can better elaborate about Pages' use of OIDC tokens in this repository. I'll track it on our side (Pages team).

Ah I think I see something here that I will get updated.
Looks like job_workflow_ref paired with environment and runner_environment do satisfy things from the situation I was thinking of.
actor also seems like a nice bit of auditing data when paired with actor_id as the usernames are unique, hard to spoof, and even if someone managed a rename, I think their actor_id might be a primary key that an attacker cannot change.

It would also be nice if it was clearly documented that setting id-token:write reverts all other default permission scopes to "none." This is rather inconvenient, as there is no way to say "set id-token to write, but retain all previous permissions" (which could possibly have been set at the workflow level).  As it is, using id-token:write requires one to explicitly supply permissions to scopes that were previously set to "read" or "write."

It would also be nice if it was clearly documented that setting id-token:write reverts all other default permission scopes to "none." This is rather inconvenient, as there is no way to say "set id-token to write, but retain all previous permissions" (which could possibly have been set at the workflow level). As it is, using id-token:write requires one to explicitly supply permissions to scopes that were previously set to "read" or "write."

If I'm reading these docs correctly, explicitly setting any permission will revoke access to almost everything that you don't set:

When the permissions key is used, all unspecified permissions are set to no access, with the exception of the metadata scope, which always gets read access.

@terryf82

When the permissions key is used, all unspecified permissions are set to no access, with the exception of the metadata scope, which always gets read access.

Should be in a callout box, everywhere folks are documenting just changing some flags.


Build Failed

Tried to run the project on xCode,
i got the same below error multiple times.
'default' label can only appear inside a 'switch' statement expected member name following '.'

@markantonyz  can you please tell me which version of xCode you are using? The current code is working under xCode 8 beta 5

I'm using Xcode 7 and build failed

Okay I updated to Xcode 8 and it built successful. Nice work man.

updated to Xcode 8.
Thank you guys very much for your contribution

I Love your work. Just that some of us haven't started writing Swift. Please, can you make the Objective-C version?  I have an issue with passing data from the home screen to the video player. I mean I want to pass things like videoID and videoUrl to the video player when an item is selected on the home screen. I'm getting an exception if I don't pre-set the properties on the Video Player controller. How do I go about this, please?

@tobitech  in PlayerVC you will find a property named "link" which is a URL. Every time user selects video from the list, you should pass the video link to that property.

@aslanyanhaik Thanks, but the issue I have with that is that if I set the video URL to the property link, the app throws an exception that link is nil even before the user select any video. You set the link to a global variable, but I want to set the link only when  the user selects a video (which I know how to pass) but it's nil even before the user selects a video and I'm not supposed to pre-define the value of "link" like you did. So how do I make the video player to init with URL only if a video is selected (thus a link passed)?

@tobitech make the link optional, so if it's nil it will not crash. Otherwise you can override the setter of that property to automatically set the link for the video player in order to avoid resetting again

@aslanyanhaik Please In my own app, I have the view controllers embedded in a UITabBarController. So I have TabBars at the bottom (as normal). The issue now is that, when I drag the video player down to minimise, it covers the tabs tot the right. How do I get the height of the tab bar and change the position of the minimised state of the video player?

@tobitech  The floating player coordinates calculation is done to use whole screen. You should modify "positionDuringSwipe" method to fix that bug

Hello! Thanks. I was able to move the minimised player upwards above the tab bar with proper spacing, but the tab bar items under it were still not selectable. Please how do I fix this?


Plans for a migration tool similar to go-pg's?

Do you have any plans to make a migration tool similar to what you did with go-pg? It would be nice to have an orm that also supported a tool like that easily that could be used with multiple databases. Maybe even database specific migrations?
Like SQLite requires a clause to enable foreign keys, so it wouldn't make sense to run such a script against a PostgreSQL database.

Yes, but I am considering using the ideas from https://github.com/robinjoseph08/go-pg-migrations since it looks like a more robust approach.

Like SQLite requires a clause to enable foreign keys, so it wouldn't make sense to run such a script against a PostgreSQL database.

I hope that hooks and specialized queries to create/drop columns/indexes will address this. I've tried to document the plan at https://github.com/uptrace/bun/issues/2.

Should be done - see https://github.com/uptrace/bun/tree/master/example/migrate


Information how SysSetup is actually called is missing

Please add a chapter about how this infrastructure is invoked. It took me a while to realize, that normal VS DB Sync on Dev VMs don't kick in. The actual important point is moving to Microsoft.Dynamics.AX.Deployment.Setup.exe as outlined in the last post in the community:
https://community.dynamics.com/365/financeandoperations/f/dynamics-365-for-finance-and-operations-forum/450253/how-to-move-existing-methods-subscribed-to-by-the-ondbsynchronize-event-to-a-syssetup-extended-class
The interesting part is the reply from Víðir Reyr Björgvinssonresponded on 19 Aug 2022 2:46 AM

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

ID: 55927241-9d0d-19f8-3e5a-a81184d387e6
Version Independent ID: 55927241-9d0d-19f8-3e5a-a81184d387e6
Content: SysSetupConfigAttribute attribute - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/dev-tools/syssetupconfigattribute.md
Service: dynamics-365-cross-app
GitHub Login: @josaw1
Microsoft Alias: najaidee

Bug 758102 has been logged to address this issue.

These docs were updated.


[5.4] Can't keep user logged in since 5.4

Laravel Version: 5.4.6
PHP Version: 7.0.13
Database Driver & Version:  MariaDB 10.1.21 - MySQL 15.1

Description:
I had an application made with Laravel 5.3 that was running fine, but after upgrading to Laravel 5.4, every time we authenticate, it goes fine and we have access to user informations :
<EMAIL_ADDRESS>. }
The thing is, it is not really logged in as when we want to access a protected view that this user would have access to, it redirects on the login page.
I checked a fresh install of Laravel 5.4 methods of authentication and can't find any difference between application and this fresh 5.4 application.
Moreover, it seems that the User object is getting a null value inside the middleware that protects the routes.
So if someone could tell me where this problem comes from, it would be really helpful,
Thanks.

Hi @vTripes
Did you follow upgrade guide? First thing it gets on my mind is that Sessions, had similar issue few days ago.

I did checked the sessions settings but I can't find anything related to the problem, and the application actually running with Laravel 5.3 works fine with the same config.
It seems that the redirection loses the authenticated user...

Check your clock hasn't gone skew.

No, it is still running as intended, I'm sure it has nothing to do with it.
As the 5.3 and 5.4 app versions are both on the same server, it sure can't be, and i checked it to be sure.
I keep thinking that it might be the refactoring of authentication method that broke the middleware. On routes that aren't protected by a middleware that verifies the user is correctly authenticated it works fine.
Are there undocumented changes that could be the reason of this trouble ?

Try checking the Exceptions\Handler.php file. There is an authentication check at that file and it redirects to the login page if it fails. Might be a good place to start looking.

Checked it and seems like it is not what fails.
As a reminder, once you log in, the user has access to the home page (protected by the auth middleware), but every other action returns a 401 HTTP status (Unauthorized) and it redirects to the login page.
Meaning that it can authenticate, but it is unable to keep it, without being the session or the authentication failing...

So it means that after getting redirected to your homepage, the session variable was set to null? Have you tried clearing the cache, route & views? Also try re-registering your classes in the autoloader.
php artisan view:clear
php artisan cache:clear
php artisan route:clear
php artisan optimize
If the above methods above did not help, maybe you'll have to wait for other developers that can help you with this. 🥇

Yes, already tried all these.
Guess you're right that I'll have to wait a bit.
Thank you anyway.

Your issue is strange. I just upgraded an application that uses authentication (and even with a custom UserProvider) and everything works fine with the auth middleware. My users can access all protected routes without problem.
My guess it has to do with the session if you say that somehow the User object is having a null value, maybe it cannot match the "active session" to a user.
What authentication guard driver are you using? (auth.php config file guards => web)

The authentication guard driver is 'session'.
So after looking at the file SessionGuard file as you recommended, that means that id is null (so session doesn't have "login") and the recaller is null too.
Do you think that something has to do with cookies because of this null recaller ?

Alright... so my guess with this new info is that somewhere in between the request and session load and the execution of the auth middleware, the session is being dropped (or the user is being logged out or something).
Can you remove the auth middleware from a secured route and ``dd(Auth::user());? It should have the currently logged in user. If not can you debug the Auth::usercall? (It should go to the Session guard and through theuser()` function there). In the meantime let me debug my side and identify the line which sets the user.

Alright, we did find the problem.
The source wasn't the session as we were focusing on, but that was indeed a middleware that was causing that all.
It was the api middleware group (that had AddQueuedCookiesToResponse, StartSession and ShareErrorsFromSession middlewares, with throttle:60,1 and bindings.
First thing we did that fixed the problem is we copied every middleware from the web middleware group (except VerifyCSRFToken).
When we saw that it was working, we removed the middlewares all by one and noticed the problem.
It seems that the middleware missing was EncryptCookies, even though that the encryption in the session configuration file was set to false... So the api routes were trying to get cookies it couldn't read because of encryption, and making the user fall back to login this way.
Thank you all for taking time to help with this issue that was particularly difficult to solve.

This has been a thorn in my side. No upgrade. But I did a bit of work on a custom guard using a different machine. Hoping it may be something to do with permissions. Will run through these steps this evening!

Hi. I’ve still not been able to solve my issue.
I’m not sure what I did but my local version just started working?!
No luck with my live site tho.
I’ve scrapped my project, pulled down fresh from git and it works locally but not the live server.
To resolve the issue locally I emptied my custom model and saved. Then re added it and that somehow worked.
How do you copy everything from the web middleware group?
I removed api routes from Auth config too. Same issue.

Thanks @v-stein I had the same issue and your provided solution worked for me.

I had same issue, after login redirecting back to login. I tried above solutions but no luck. Finally I got the root cause after lot of investigation. The issue due to space on route file. Working after removed space before to <?php.


how to configure task with kotlin dsl

hello!
how I can configure task if I use Kotlin DSL to author my buildscript?
jsonSchema2Pojo {
    generateBuilders = true
}

doesn't work in kotlin.
Thank you

Oh, thank you! this was a serious issue with extension class visibility. Now you have a Kotlin DSL primer and fixed plugin!


Refactor/reorder reorgenize demo applications

Will maybe brake the ClientSide Demo Page on hosted  on GitHub.
That's why I created a MergeRequest, that @limefrogyank  can have a look at that before merge.

Looks nice.  Good work.  I guess we can leave the hosted client project in there.

So I will merge it
I would leave it in because make no difference


Store this with newfunction

Can we use bind here?

Can we use bind here?

Not really, the closure is here the object with the apply method. It would be nice to make Closure as a function (vs Object).


added POM.xml

Referring to Issue #15
This should be a working pom.xml

schau mal in der pom bei der artifactid wirklich energy-model steht.....hatte den selben fehler vielleicht klappts ja dann

The POM.XML worked on my system without warnings or errors. The failed check seems to be an issue with the builds of Travis.
The commands I used on my system in PowerShell were:
mvn clean install
After this the .jar file occured and with the next command the output of the programmed Kotlin-File "Hello World" emerged in PowerShell.
java -cp target/energy-model-1.0-SNAPSHOT-jar-with-dependencies.jar MainKt

I see you found the manifest entry that you need to get an executable JAR. Good job! I would merge this PR but already merged #18.


docs: Add a tree view to Hello Holos

A tree view of the holos-tutorial/ directory should give readers a quick, high-level understanding of the folder structure of a typical Holos platform project.

Closed via 2e3c998


remote: support for non-force, fast-forward-only fetches

This PR adds support for a missing "+" in a fetch refspec, which should indicate a fast-forward-only fetch.  (Alternatively, the caller could put Force=true into the FetchOptions to indicate that all fetches should be treated as forces.)
It relies on file-level locking provided by the storage layer, so that distributed storage layers (being accessed concurrently by multiple independent nodes each running go-git) can be supported.
Code mostly written by @taruti, based on an early version by @lupine in #490.

@mcuadros I've added more tests and addressed your comments.  Please take another look, thanks!


Issue with Group Sync from Okta in Netbird

Describe the problem
We are currently setting up group synchronization with Okta in Netbird and have followed the provided manual for Okta integration. However, the documentation lacks specifics on how to properly configure group synchronization (using JWT?). Additionally, there is uncertainty around the use of the Okta API token set up during the integration—specifically, whether it should automatically pull group information.
Any help on how to set up group syncing will be much appreciated.
To Reproduce
Steps to reproduce the behavior:
https://docs.netbird.io/selfhosted/identity-providers#okta
Expected behavior
Groups sync from Okta
Are you using NetBird Cloud?
self-hosted

Hello @calibratedcat you can ask the folks in NetBird's Slack self-hosted channel, maybe someone from the community already configured Okta with JWT group sync.

Key to solve this problem was adding groups to these vars:
NETBIRD_AUTH_SUPPORTED_SCOPES="openid profile email groups"
NETBIRD_AUTH_DEVICE_AUTH_SCOPE="openid email groups"

Awesome @calibratedcat , did you have to configure anything on the Okta side too?

In Okta, you need to configure the OpenID Connect ID Token section. You need to define Group Claim Filter. In my case, I chose any group by using .* expression. groups is a claim name that will be used in your token. Should match in env listed above.


Where could i configure max_response_headers_kb

Title: Where could i configure max_response_headers_kb
Description:
I met an issue when using envoy gateway as proxy for my backend service.
When i make a request, my backend service will respond envoy with a big header (over 100KB), and envoy throws out an error:
Error dispatching received data: http/1.1 protocol error: headers size exceeds limit
I researched and found that max_response_headers_kb is the param that i need to configure.
But i could not find any document on how to configure this param in envoy (or EnvoyPatchPolicy).
Would you please provide any help here ?

cc @alyssawilk @mattklein123

I don't think it was ever implemented.  Previously I think there were limits imposed by the codec but after switching over to balsa I think we could implement this feature

@alyssawilk i am trying to update MAX_RESPONSE_HEADERS_KB in source/common/http/http1/codec_impl.h to a larger number (e.g. 3000).
And then i compile a new envoy image by leveraging Bazel.
Do you think this image could work ?

Yes, that looks right to me. This compile-time parameter predates me, and is obeyed by BalsaParser and http-parser alike.


change admin restore table syntax to restore table syntax.

change admin restore table syntax to restore table syntax.
move restore table from adminStmt to ddl stmt. Because restore table is a ddl operation.
This is for PR: https://github.com/pingcap/tidb/pull/9090

Since you're changing the syntax anyway, could we consider whether the keyword RESTORE is appropriate? In most database systems RESTORE is typically used together with BACKUP, not to undo a DROP TABLE, e.g.

The removed RESTORE TABLE statement in MariaDB
The RESTORE statements in T-SQL
The RESTORE DATABASE command in DB2
The pg_restore utility in PostgreSQL

Perhaps RECOVER (example reference of the term: ApexSQL Recover / DB2 / TwinDB) would be better. Additionally Oracle names the command FLASHBACK but still uses the verb "recover" in the docs.

LGTM
It seems this PR should be merge with https://github.com/pingcap/tidb/pull/9090 together


Rework TypeId stuff at SubscriberImpl to be dynamic modules friendly.

Instead of relying on address of a static template variable (which is not dynamic modules friendly), switch to Nunavut generated full type name string (python code already has it), from which could generate a hash per template instantiation, and use it as type id.

Resolved by PR #396


How to use?

Hi. Sorry but I don't understand the instructions. What is the status bar? Where do I find it? What do I do then?

This indicator is designed for system status bars that allow custom user modules, like polybar and waybar.
To use dsbattery with either of those, you must install them first and then copy contents of the appropriate module  into to your status bar configuration file.


Add pagination to Costmanagement Query API

Description
Add Pagination capability to Costmanagement - Query API
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
Introduce some breaking changes by adding skipToken parameter to methods in Query API

[X] CHANGELOG is updated for new features, bug fixes or other significant changes.

[X] I have read the contribution guidelines.

General Guidelines and Best Practices

[X] Title of the pull request is clear and informative.
[X] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

Hello,
We are waiting another pull request to be completed (for API spec) before pursuing here.
Thank you.

@GuillaumeColom Thank you very much for this contribution! I highly appreciate it. The blocking PR https://github.com/Azure/azure-rest-api-specs/pull/19609 appears to have been merged now. I hope this one can then be merged as well soon. Eagerly waiting for it!

@weidongxu-microsoft Any chance of getting this merged soon? Do you have a timeline? Sorry for bugging you with this, but I'm actually blocked by this issue and I'm considering copying this patch manually into my project unless this gets merged/released within the next few days. So, if you already know it will take some weeks until it's merged, please let me know. Thank you!

@GuillaumeColom After looking into this code change in more detail, I don't think exposing skipToken in com.azure.resourcemanager.costmanagement.models.Queries#usage is not a good idea, as it's introducing a breaking change. Pagination should be handled similar how the com.azure.resourcemanager.costmanagement.implementation.OperationsClientImpl#listAsync() is doing it by chaining the requests.

@neiser
Romain is checking with cost management service to sign off on swagger https://github.com/Azure/azure-rest-api-specs/pull/19609
After it signed-off, Java will regenerate the whole SDK with 2022-08-01-preview api-version.

For the pageable, it is also due to swagger. If service supports it correctly, it should be annotated with x-ms-pageable, e.g. https://github.com/Azure/azure-rest-api-specs/blob/main/specification/cost-management/resource-manager/Microsoft.CostManagement/preview/2022-08-01-preview/costmanagement.json#L70-L72

@weidongxu-microsoft I would like to know if it's possible to reopen this PR. Since the PR in the rest api spec is merged, this sould be quite straight forward

/reopen

@Totox64
backend decides not to change (issue here https://github.com/Azure/azure-rest-api-specs/issues/23405)
java provided a workaround here https://github.com/Azure/azure-sdk-for-java/issues/34455


Can chromely be used for real time streaming like electron app

Just wondering if Chromely auto updates the UI listening to the server events


Adding a ginkgo version of monitoring e2e test

This e2e is more detailed than its shell counterpart. This PR removes the shell based monitoring e2e test.
Fixes #5212 #4186

@zmerlynn: I have enabled monitoring by default for e2e as well. Can you confirm that the changes in this PR will work for GKE based e2e clusters as well?

@vishh: This isn't enough for GKE (we don't use config-test.sh / config-default.sh, we basically only use configure-vm.sh now, once I finish my changes.)
But furthermore, I think you may just need to conditionalize the test on provider == gce anyways?

@zmerlynn: Both of your comments are valid. I have updated the test to only look for the monitoring pods. The test will also run only on GCE.

@vmarmol: Thanks for the review. Addressed your comments.

LGTM


Don't allow duplicate colors.

This PR makes player colors unique per player, making them more distinct.
Also adds color indicators to the player list in the main menu.

Also fixes an issue where only the local player time played was being shown as each player in the list's time played.
https://github.com/apetavern/sbox-grubs/assets/66768086/f950096d-41a9-45b9-a343-b644a0c51871
If a player tries to click a color that is taken, its button is darkened and does nothing. If the player doesn't select a color, they get assigned a random unused color. If all colors are taken (at a large player count) a random untaken color is assigned, if failing resulting in color overlap (again, only at really large player counts for this type of game).
https://github.com/apetavern/sbox-grubs/assets/66768086/b7683d20-fc70-4952-b10f-33959cfd3d8e

ready to go


For testing, flax==0.5.3 is incompatible with jax==0.4.14

Recent tests fail with:
__________________ ERROR collecting _src/equivalence_test.py ___________________
ImportError while importing test module '/tmp/optax-env/lib/python3.11/site-packages/optax/_src/equivalence_test.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/opt/hostedtoolcache/Python/3.11.4/x64/lib/python3.11/importlib/__init__.py:126: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
/tmp/optax-env/lib/python3.11/site-packages/optax/_src/equivalence_test.py:21: in <module>
    from flax import optim
/tmp/optax-env/lib/python3.11/site-packages/flax/__init__.py:19: in <module>
    from . import linen as linen
/tmp/optax-env/lib/python3.11/site-packages/flax/linen/__init__.py:47: in <module>
    from .attention import (
/tmp/optax-env/lib/python3.11/site-packages/flax/linen/attention.py:22: in <module>
    from flax.linen.linear import default_kernel_init
/tmp/optax-env/lib/python3.11/site-packages/flax/linen/linear.py:30: in <module>
    from jax import ShapedArray
E   ImportError: cannot import name 'ShapedArray' from 'jax' (/tmp/optax-env/lib/python3.11/site-packages/jax/__init__.py)

JAX 0.4.14 (July 27, 2023) deletes jax.ShapedArray.

jax.ShapedArray: use {class}jax.core.ShapedArray.

Optax's test suite uses flax==0.5.3, because it equivalence_test.py uses flax.optim, and flax.optim is deleted in the next release.
Unfortunately, Flax 0.5.3 uses the deleted jax.ShapedArray.
A short-term fix is to require jax<0.4.14 in test environments. In future, the flax.optim reference implementations should come from another source.

This should now be fixed


Remove powerstrip from the documentation

With the latest libnetwork release, probably makes sense to remove powerstrip demo.
Should do this once the 1.9 Docker release is made public.

Fixed by #585


Allow multiple interpreters for a pex

Hi,
I think my issue is with documentation, because from reading https://github.com/pantsbuild/pex/issues/654 I feel like my use case seems to be supported, but I couldn't quickly work out how.

So, I have an application with a script-entrypoint that I want to bundle with its dependencies to offer my clients as a single "binary" that I know will Just Work on their machine without needing to download anything that could potentially break / is untested.
I have build my application and all its dependencies as wheels, put them in a folder called wheelhouse, and use this command to build the pex file:
pex --interpreter-constraint ">=3.8" -r requirements.txt setuptools my_app -c my_app_entrypoint --no-pypi --repo wheelhouse/ -o my_app.pex

I know that it uses a python version 3.8 for the pex command, because it's the only interpreter on the build system, but the application should run just fine under python3.9 as well (which is what I wanted to express with >=3.8). It works fine with python 3.8, but when I run it with only a python3.9 on my PATH, it fails. How can I correctly tell pex during the build that it's ok to use 3.8 or higher?

@a-recknagel it appears your PEX includes at least one platform-specific wheel. You can confirm this with pex-tools <your PEX> info -i2. You should see in distributions at least one with platform specific tags (like cp38m). If you want to build a PEX that includes the distributions needed by interpreters you don't have on hand, you need to specify a --platform for each and every one of those. To see how to write a --platform, consult pex --help which will include an example for your current platform. Alternatively, use pex --platform help.

Yeah, that's probably it, I didn't really consider that. Can I create a multi- platform pex by including e.g. cp38 and cp39 wheels of such dependencies in a single one, or should I create multiple pexes instead?

You can create one PEX with all the wheels. If you specify multiple platforms (--platform X --platform Y ...), Pex does the right thing. You can again inspect this with pex-tools <your PEX> info -i2 after building the PEX.

Sweet, thanks again. I'll post the right command as soon as I worked it out, in case anyone stumbles over this post via google.


Add callback parameter in API docs

This is what the repos.getAll() documentation currently is. It doesn't specify that the last parameter is actually a callback. It would be super helpful if it was specified - I spent 30 minutes assuming it was returning a promise and wondering why it was always returning undefined.

@paolodm It does return a Promise – unless you pass in a function as last argument.
I have new docs ready as part of #680, the add examples using callbacks, would that have helped in your case?

Really? I am getting undefined for the following:
const GitHubApi = require('node-github');
const github = new GitHubApi({
    host: 'api.github.com',
    version: '3.0.0'
});

github.authenticate({
    type: 'token',
    username: 'paolodm',
    token: 'your-token-here'
});

let r = github.repos.getAll({
    affiliation: 'owner,organization_member'
});

console.log(typeof r);

I think I know what is happening. This repo's name is node-github, but it actually maps to npmjs's github.
Meanwhile, I'm using npmjs's node-github that corresponds to the now defunct https://github.com/c9/node-github3.
So confusing. UGH!

you are not the first to run into this, it happened to me in the past as well :( GitHub adopted this pacakge recently and will rename it to @octokit/rest soon, so this won’t happen in future. I’m sorry for the headaches this caused you.

We can close this and the other issue. Thanks for the help.


Automated cherry pick of #2730: Fix panic bug in Antrea Policy

Cherry pick of #2730 on release-1.0.
#2730: Fix panic bug in Antrea Policy
For details on the cherry pick process, see the cherry pick requests page.

/test-all
/test-ipv6-all
/test-ipv6-only-all

/test-all
/test-ipv6-all
/test-ipv6-only-all

Codecov Report

Merging #2760 (307c1d5) into release-1.0 (4974ca6) will decrease coverage by 18.85%.
The diff coverage is 70.00%.

@@               Coverage Diff                @@
##           release-1.0    #2760       +/-   ##
================================================
- Coverage        60.75%   41.89%   -18.86%
================================================
  Files              272      131      -141
  Lines            20429    16480     -3949
================================================
- Hits             12411     6905     -5506
- Misses            6730     8988     +2258
+ Partials          1288      587      -701

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.89% <70.00%> (+0.32%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.


vue 3 default" is not exported

vue 3
yarn build
"default" is not exported by "node_modules/quill-image-resize-module/image-resize.min.js", imported by "src/views/editor.vue".

5: import ImageResize from "quill-image-resize-module";
^

vue 3 yarn build "default" is not exported by "node_modules/quill-image-resize-module/image-resize.min.js", imported by "src/views/editor.vue".

5: import ImageResize from "quill-image-resize-module";
^

import { default as ImageResize } from "quill-image-resize-module";
I am using Vite with Vue 2. This works for me and should be good with Vue 3 too.

在main.js里加入以下代码即可
import { Quill } from 'vue-quill-editor'

window.Quill = Quill

vue 3 yarn build "default" is not exported by "node_modules/quill-image-resize-module/image-resize.min.js", imported by "src/views/editor.vue".

5: import ImageResize from "quill-image-resize-module";
^

import * as ImageResize from "quill-image-resize-module"; I am using Vite with Vue 2. This works for me and should be good with Vue 3 too.

i got the new error of " quill.js:6130 Uncaught (in promise) TypeError: moduleClass is not a constructor" using this

vue 3 yarn build "default" is not exported by "node_modules/quill-image-resize-module/image-resize.min.js", imported by "src/views/editor.vue".

5: import ImageResize from "quill-image-resize-module";
^

import * as ImageResize from "quill-image-resize-module"; I am using Vite with Vue 2. This works for me and should be good with Vue 3 too.

i got the new error of " quill.js:6130 Uncaught (in promise) TypeError: moduleClass is not a constructor" by using this

The correct way to use it is
import * as ImageResize from "quill-image-resize-module";

Quill.register("modules/imageResize", ImageResize.default);

Pay attention to the position of the default

The solution that worked for me was:
import ImageResize from 'quill-image-resize-module';

//...

// Note the module is registered as `imageResize` not `ImageResize`
Quill.register('modules/imageResize', ImageResize);
// ...

const quill = new Quill(editorEl, {
    modules: {
       // Note: 'imageResize' (the name needs to match what it was registered as).
       imageResize: {
         modules: ['Resize', 'DisplaySize', 'Toolbar'],
      },
    }
});


[HOLD for payment 2023-09-06] [$500] Dev : Splash logo is grayed out

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Launch the app
Expected Result:
Splash logo color is green
Actual Result:
Splash logo color is gray
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: dev
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @situchan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1691776351462309
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01118b202c69bb739e
Upwork Job ID:<PHONE_NUMBER>696375296
Last Price Increase: 2023-08-24
Automatic offers:

c3024 | Contributor | 26331939
situchan | Reporter | 26331942

@sakluger this is only reproducible on dev. Also confirmed by others - https://expensify.slack.com/archives/C049HHMV9SM/p1692173984147349

I'm not sure if this is actually a bug - I asked in the Slack thread if the differently styled logo would ever end up on staging or prod. If not, I don't know if it's something we need to fix.

Seems like this was the PR where the logo was changed: https://github.com/Expensify/App/pull/21551. Adding external label.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Dev logo is grayed out and not green like staging
What is the root cause of that problem?
We are using #002e22 instead of #03D47C on what we have on staging
What changes do you think we should make in order to solve the problem?
We have to edit the .st2 of to be using #03D47C instead
https://github.com/Expensify/App/blob/37ac2522440a671ec5f75e4e0685a22597db35a0/assets/images/new-expensify-dev.svg#L2C3-L4
What alternative solutions did you explore? (Optional)
None
Result
https://github.com/Expensify/App/assets/20473526/00ef0e6f-b908-45e2-bd38-3a35451524cf

Proposal
Please re-state the problem that we are trying to solve in this issue.
Dev logo is grayed out and not green like staging
What is the root cause of that problem?
We are using #002e22 instead of #03D47C on what we have on staging
What changes do you think we should make in order to solve the problem?
We have to edit the .st2 of to be using #03D47C instead
https://github.com/Expensify/App/blob/37ac2522440a671ec5f75e4e0685a22597db35a0/assets/images/new-expensify-dev.svg#L2C3-L4
What alternative solutions did you explore? (Optional)
None
Result
https://github.com/Expensify/App/assets/20473526/00ef0e6f-b908-45e2-bd38-3a35451524cf
cc: @mananjadhav @sakluger @kavimuru

Sorry for the late change here - after discussion with the team, this is a very small change and the bounty should be smaller.

@c3024's proposal is good here.
🎀 👀 🎀  C+ reviewed.

NOTE: https://github.com/Expensify/App/issues/25643 was closed in favor of this one, so I believe this should be fixed as well here.

I went ahead and paid out the reporter bounty to @situchan
@mananjadhav could you please complete the BZ checklist before tomorrow? That way we can approve your manual payment right away after the 7-day hold.

Paid @c3024 via Upwork.
We're just waiting on the BZ checklist to be completed now.

https://github.com/Expensify/App/pull/21551 is the offending PR, and I've added a comment here.
I don't think we need any changes in the process, it's a miss considering we also had 122 other file changes. Moreover considering this is Dev only and logo, I don't think we need a regression test either.
@sakluger I've raised my request on NewDot.

Thanks, I agree we don't need new regression tests since this only affected dev.
We're good to close out the issue!

$750 approved for payment based on BZ summary.


MDX Support

Hi,
First of all thanks for this awesome starter.
I am trying to get mdx to work,  tried a few times. Would you know  how to enable  support for this?

This repository is mainly intended for sourcing in content from a headless Ghost CMS. However, I do think that you can get MDX files in on separate pages (similar to the 404 page), if you install the required plugins. Could you elaborate a bit on your use case, what do you intend to do and why?

Hi
For now it does not have support for it. Will update in future if that can be added.


Accessing claim value in JWT

I'm thinking about using getClaim() method from Jose\Object\JWTInterface (just like when we're using it in ClaimCheckerInterface) to get value of that claim. But I can't figure out a way to load the JWT so I can do that.
Edit : I've just found a solution to my problem using the JWTLoader class. But when I check the time I check if the claims is valid and the time I actually want to use that claim is different. Is there a way to do this without requiring me to instantiate JWTLoader? Since to do that I'd have to instantiate CheckManager and Verifier too.

The loader has to be instantiated. There is no static method.
Please note that this library will be deprecated in less than 6 months. You should consider the use of https://github.com/web-token instead.


Superfluid integration

Make Superfluid available within composer with basic directions on usage.

I have been working on ways to make cUSD streams using ERC-1620 token standard used by Sablier

Great @lopeselio 🎉
Let us know if we can be of any help.

@lopeselio any update on this?


Unable to subclass Common.GridEditorDialog

I was trying to subclass 'Common.GridEditorDialog' to be able to have common MakeReadOnly functionality like I did with 'Serenity.EntityDialog'
but for some reason it is not possible.
any ideas why?
`namespace MyTest {
@Serenity.Decorators.registerClass()
export class MyGridEditorDialog extends Common.GridEditorDialog {
    protected MakeReadOnly(bHide: boolean) {
        if (!this.isNew()) {
            if (bHide) {
                Serenity.EditorUtils.setReadonly(this.element.find('.editor'), true);
                this.element.find('sup').hide();
                this.deleteButton.hide();
                this.saveAndCloseButton.hide();
                this.applyChangesButton.hide();
            }
        }
    }
}

}`

What problem you have actually?
Any error?

@dfaruque - Yeah, big bad red errors:
Message: Object doesn't support this action
File: http://localhost:53312/Scripts/jquery-3.3.1.js?v=agfan66TS68_dJ6Ha7_dlg2, Line: 3827, Column: 3
SCRIPT ERROR! See browser console (F12) for details.
Message: Unable to get property 'prototype' of undefined or null reference
File: http://localhost:53312/Scripts/serenity/Serenity.CoreLib.js?v=4DMOWnm4a2N3HOKpnMBgLQ2, Line: 21, Column: 5

Add /// reference to base file, this is Nth time people do same mistake

sorry. I did not find it and did not need it in other cases.
it seems to inherit fine now


bin/install.ps1 : very strange powershell execution policy check

Hello,
https://github.com/lukesampson/scoop/blob/master/bin/install.ps1#L15 says:
# show notification to change execution policy:
if((Get-ExecutionPolicy) -gt 'RemoteSigned' -or (Get-ExecutionPolicy) -eq 'ByPass') {
    Write-Output "PowerShell requires an execution policy of 'RemoteSigned' to run Scoop."
    Write-Output "To make this change please run:"
    Write-Output "'Set-ExecutionPolicy RemoteSigned -scope CurrentUser'"
    break
}

Which means that any executionpolicy greater than RemoteSigned cannot install scoop.
But Bypass is greater than RemoteSigned that cannot install scoop ?
> Get-ExecutionPolicy ; (Get-ExecutionPolicy) -gt 'RemoteSigned'
Bypass
True

I managed to install scoop by downloading the file install.ps1 locally and removing the if block.
Could you please check the if condition.

Did the installation with Bypass work? Maybe I made a mistake and it should have been
-neq 'ByPass'.
[System.Enum]::GetValues([Microsoft.PowerShell.ExecutionPolicy]) |
Sort-Object -Property @{Expression = {[int]$_}; Descending = $False} |
% { "$([int]$_) => $_" }

0 => Unrestricted
1 => RemoteSigned
2 => AllSigned
3 => Restricted
4 => Bypass
5 => Undefined

https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_execution_policies?view=powershell-6

Yes currently bypass cannot install scoop

@r15ch13
I think your suggestion on -neq 'ByPass' should fix the issue.
Or make an explict choose of the execution policy instead of int comparaison. Because it's a little bit strange to set the ByPass at 4, nobody outside of powershell team can know why.


Implement client-side GraphQL query for homepage data

Body:
User Stories

As a user, I can see every user's thoughts listed on the homepage of the application

User's thoughts listed on homepage of app


Fix text element issues with transformations

Summary
There was a regression from https://github.com/google/web-stories-wp/pull/7719 for the text element, editing the text element has various connected parts which are relying on modifying the style of the element directly. The logic for removing excess classes interferes with it and breaks various parts of it, e.g. line-height, font-size, margin for the font, etc.
Once we have SVG rendering in place, we can adjust the rendering logic for the library only not to create excess classes.
This PR reverts text-element specific changes.

Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

Add a text element
Assign line height to it
Verify everything looks normal and the frame is in sync with the displayed text.

QA

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT

[x] UAT should use the same steps as above.

This PR can be tested by following these steps:

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[ ] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #7764

@merapi FYI when reviewing: this is just a reversion of #7719.


fix: update storage_options when using DeltaTable in write_deltalake

Description
The previous code was erroneous and if a DeltaTable instance was kept for too long its credentials could expire with no way of updating them.

@NilsIrl can you add a test please

Sorry I don't have the bandwidth to do that.


file_exists(): open_basedir restriction in effect.

Hey
I'm using glide to do some image manipulations on an uploaded image.
When testing locally, everything works but when we deploy to a combell shared hosting, I get the following error:
file_exists(): open_basedir restriction in effect. File(/data/sites/web/scribblrmtantwerpeu/tmp/phpmkV0Wn/f1b906abc87cd37f44c511f8dd7acc88) is not within the allowed path(s): (/data/sites/web/scribblrmtantwerpeu:/usr/local/php-7.1/lib/php:/usr/bin:/tmp:/data/vendor-scripts:/data/sites/web/disabled:/etc/ssl/certs)
I guess this is trying to check if the file exists in the tmp folder. The problem is that it starts from the basepath, which we don't have access to. The root of our shared hosting in this case is 'scribblrmtantwerpeu'
Any way we can fix this?

This problem is probably caused by the underlying package Glide. Open an issue at their repo.
(pro-tip: don't use shared hosting 😬)


Change output-parameter--based APIs to return by value

Similarly to what we have done for sf::IpAddress, I suggest doing the same to classes such as sf::Image. In other words, sf::Image's constructor would become private, and we'd have a sf::Image::from static member function that returns a std::optional<sf::Image>.
The same would apply to sf::Music, sf::SoundBuffer, and probably others.
Thoughts?

Are you talking about replacing things like sf::Image::loadFromFile with named constructors so we can remove those member functions? I like that idea.

As a C++ oldie, it still feels unnatural to me, but I'm up for a renewed API.
But maybe we'd first have to define an error handling strategy (see #2139), as that also has heavy influence on the selected API.

I see this less as changing our error handling strategy and more than we're reimplementing the existing error handling strategy better. We're still returning truth-y values from functions but now we can do that with 1 function call instead of 2. I think that's a great improvement and very much in the style of SFML code as its been for so long.

https://github.com/SFML/SFML/issues/2139#issuecomment-1931676032
We have decided that we will in fact go forward with this so this issue can be closed.


Add a keyof(table) type

The problem
Currently, it's inconvenient to type check table content as I have to write the same thing twice, both in the type declaration and in the table.
For instance, a constant that use a bunch of strings as keys.
type PossibleOpinions = 'Cats'|'Dogs'|'Pizzas'|'Tacos'

local MY_OPINIONS = {
	['Cats'] = 'I love cats!',
	['Dogs'] = 'I like dogs! But I prefer cats!',
	['Pizzas'] = 'When I think about pizzas I think about Peppino Spaghetti',
	['Tacos'] = 'You mean that french dish where they put a bunch of stuff in a brick?'
}

function sayMyOpinion(about:PossibleOpinions)
	print(MY_OPINIONS[about])
end

sayMyOpinion('Cats') 	 -- ok
sayMyOpinion('Hexagons') -- not ok

The solution
Add a keyof type! It returns a union of all the keys of a table!
It should be backward compatible like typeof and not so chaotic to integrate.
local MY_OPINIONS = {
	['Cats'] = 'I love cats!',
	['Dogs'] = 'I like dogs! But I prefer cats!',
	['Pizzas'] = 'When I think about pizzas I think about Peppino Spaghetti',
	['Tacos'] = 'You mean that french dish where they put a bunch of stuff in a brick?'
}

function sayMyOpinion(about:keyof(MY_OPINIONS))
	print(MY_OPINIONS[about])
end

sayMyOpinion('Cats') 	 -- ok
sayMyOpinion('Hexagons') -- not ok

This seems like it would be a really nice addition.
Currently, I'm trying to make a state wrapper that auto-types its elements for the methods. The problem is there is no current method of getting the types in table, so everything gets inferred as any:
local newState = State.new {
	A = 1,
	B = 2
}

newState:SetValue("A", 2) -- "A" is not inferred, and 2 cannot be typed
newState:GetValue("B") -- Returns 'any'

This proposal would solve half of the problem, as now I could type the Key params with keyof({} :: T), such as:
type State<T> = {
	GetValue: (Key: keyof({} :: T)) -> any
}

However, this does not fix the problem with the return type or the second param for SetValue, so I think there would need to be another operator to fix this, like a valueof function or something of the likes. I'm not sure how this would all fit together, but in my mind, it would look something like:
type GetValue<T = {}, K = string> = (T: keyof({} :: T)) -> valueof({} :: T, "" :: K)

Obviously, fixing my problem is not trivial, but I don't think the addition of keyof would hurt any future attempts at solving it.

The way TypeScript solves @MagmaBurnsV's problem is using keyof, a type space indexing operator, and constrained generics, shown here. Implementors of this interface would have type-safe access and mutation of key-value pairs present in T.
interface State<T> {
  GetValue<K extends keyof T>(k: K): T[K];
  SetValue<K extends keyof T>(k: K, v: T[K]): void;
}

I've seen discussion about constrained generics somewhere in this repo, like #784, but nothing about implementing keyof (other than this post) or a type space indexing op.

I want to note this RFC outlining what kind of syntax the team might prefer for type functions and why. TL;DR use keyof<T> over keyof T or keyof(T).

#589
https://github.com/Roblox/luau/blob/master/rfcs/disallow-proposals-leading-to-ambiguity-in-grammar.md

For the record, there's a Index<T, K> type family that's going to solve the type indexing problem, so I wouldn't bother working on this design space.
As for the Keyof use case, there's a bunch of nuances to think about. Should Keyof also include all accessible keys via __index metamethod? What about if the table has indexers? The story gets blurry in a quick hurry.

I think keyof should also include all accessible keys via __index metamethod because it's very similar than directly putting keys in the table. I first thought of keyof(T) because of Luau having typeof(T) would keep the syntax similar, but I don't mind any alternative syntaxes.

For the record, there's a Index<T, K> type family that's going to solve the type indexing problem, so I wouldn't bother working on this design space.
As for the Keyof use case, there's a bunch of nuances to think about. Should Keyof also include all accessible keys via __index metamethod? What about if the table has indexers? The story gets blurry in a quick hurry.

I don't believe it should include keys from __index because that isn't what you're asking for. If you wanted to do that you should have to directly specify it with keyof(t.__index).

I think it's better to not include __index, since (maybe in the future) a user could implement this behavior themself:
local MyClass = { Foo = "Bar" }
local OtherClass = setmetatable({ Hello = "World" }, { __index = MyClass })

type MyClass = typeof(MyClass)
type OtherClass = typeof(OtherClass)
type TableIndexed = Keyof<OtherClass> & Keyof<typeof(getmetatable({} :: OtherClass).__index)>

It just occurred to me that RawKeyof could be a thing that explicitly ignores __index, then Keyof includes anything in __index and all of the types in the __index chain.

Would be awesome to have this this, allows for much stronger and automatic types.

I hope they are doing the same thing for valueof.

valueof already exists in the new solver, it's called index and rawget: https://rfcs.luau-lang.org/index-type-operator.html and https://rfcs.luau-lang.org/rawget-type-operator.html


Can't train with "python src/main.py env.train.id=BreakoutNoFrameskip-v4 common.device=cuda:0"

I can have fun with"python src/play.py --pretrained".But I can not use "python src/main.py env.train.id=BreakoutNoFrameskip-v4 common.device=cuda:"to train.
`PS C:\Users\Leon\Desktop\diamond-main> set HYDRA_FULL_ERROR=1
PS C:\Users\Leon\Desktop\diamond-main> python src/main.py env.train.id=BreakoutNoFrameskip-v4 common.device=cuda:0
Could not override 'common.device'.
To append to your config use +common.device=cuda:0
Key 'device' is not in struct
full_key: common.device
object_type=dict
Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.`
and I also can't use
PS C:\Users\Leon\Desktop\diamond-main> python src/main.py env.train.id=BreakoutNoFrameskip-v4 +common.device=cuda:0 Starting on cpu:0 A.L.E: Arcade Learning Environment (version 0.8.1+53f58b7) [Powered by Stella] A.L.E: Arcade Learning Environment (version 0.8.1+53f58b7) [Powered by Stella] A.L.E: Arcade Learning Environment (version 0.8.1+53f58b7) [Powered by Stella]
CPU is Extreme slow.
I didn't change any code or yaml.
Thank you very much for your code.

Hi, thanks for the issue!
There's indeed a discrepancy between the readme and the latest code. You need to use common.devices=0 rather than common.device=cuda:0.
We'll update the readme accordingly - thanks for letting us know!

thank you very much!


Prepare release note for 1.6.0

Diff all changes between master -> 1.5.0 tag and cats -> v1.5.1-cats  tag

Hi @julien-truffaut ,
I was trying to compare these two tags, but these are not available in git /compare drop down. Could you please help me, if I did something wrong.

Hey @julien-truffaut,
I figured out the changes between master -> 1.5.0 and cats -> v1.5.1-cats. So, the question is release notes for 1.6.0 version is the combination of these two differences or something else?
This is the first time, I am doing these types of things, I apologize for huge question lists :stuck_out_tongue:


TableView

@roberthein Can this work with a tableview and if so, can you show me how I could implement it.

No sorry, this doesn't work with UITableView. But it's pretty easy to turn your UITableView into a UICollectionView. 😉

How I Can add layout in Storyboard?


AT+BLEHIDKB behaviour is not the same as USB HID

Environment

Development Kit:     ESP32-DevKitC
Module or chip used:  ESP32-WROOM-32
ESP32-AT Release v<IP_ADDRESS>
Operating System:    Linux
Power Supply:        external 3.3V

Problem Description
Using the built in BLEHID functionality, specifically the AT+BLEHIDKB command, sending a single BLEHIDKB command results in a keypress and release on the paired device. i.e.
AT+BLEHIDKB=0,4,0,0,0,0,0 will result in 'a' being input
As per normal HID usage, sending the above should result in a single key press event on the device, and then a subsequent command of something like
AT+BLEHIDKB=0,0,0,0,0,0,0
should result in a key 'release' event. This is how the USB HID reports work.
Expected Behavior
sending AT+BLEHIDKB=0,4,0,0,0,0,0 should result in 'a' being input, then after a short delay, 'aaaaaaaaaaaaaaaaaaa' until a subsequent BLEHIDKB command is sent to clear the input.
Actual Behavior
sending AT+BLEHIDKB=0,4,0,0,0,0,0 seems to send the equivalent of a key press followed by a key release.
Steps to repropduce

send AT+BLEHIDKB=0,4,0,0,0,0,0 to a paired device, a single 'a' will be input.

Code to reproduce this issue
Any code which sends that BLEHIDKB command should reproduce the issue, it's a problem in the behaviour of the built in HID functionality.

@xiewenxiang PTAL

I think for "send AT+BLEHIDKB=0,4,0,0,0,0,0 should result in input 'a', then after a short delay, 'aaaaaaaaaaaaaaaaaaaa'" this behavior should be implemented on the MCU side, eg input How many 'a', how many times the MCU control command is input.

@Luxin812 no, that's not the behaviour of USB HID at least, and BLE HID seems to work identically. The delay is set on the client machine, say 200 ms or whatever, and the command behaviour is as I outline above i.e. the client machine receives a HID Keyboard report with a specific code in one of those slots (6 by default) which results in it outputting that character, then after the configured delay, repeating that same character. It will do this until it receives a key report that DOESN'T have that keycode in it (i.e. a 'key up') or a key report with an additional different key (i.e. another key was pressed). All that repetition behaviour is handled on the client, not on the HID device.

@ustccw i have observed the same problem and agree with @dairequinlan - the current behaviour is not how the HID report is supposed to work. Keyboard keys (as defined in USB HID section 10) are “Selectors” as per USB HID spec <IP_ADDRESS>, as such the report contains the set of keys that are currently being pressed. When you want to release a key, you provide a report (a set) which doesn’t contain the key code. Pressing and releasing a key requires two separate reports to be sent.


Add auto-generate sample for DevCenter and DeviceProvisioningServices

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Latest Version seems to be incompatible with Rider 2022.3 EAP

In the release notes for v4.3.0, support was apparently added for platform version 2022.3, however Rider claims that the plugin is incompatible:

Maybe gradle.properties needs to be updated so pluginUntilBuild = 223.*?

same problem in WebStorm 2022.3
Build #WS-223.7571.168, built on November 28, 2022


added cachedAssets config for cached headers on /_next/static/* assets.

Hello all!
Been looking at ways to allow for caching of image/fonts without having to use a custom server and decided to propose this idea. It will allow for the user to add the config option, cachedAssets, to next.config.js via an array of folder names, which will set the cache headers for assets that proceed the directory /_next/static/:cachedAssetsFolder. This proposal was done fairly quick to show the logic so im probably missing something.
The reason for proposing this was because I've been using next-optimized-images that allows for hashed statics via /_next/static/images/* directory which makes immutability feasible. They did not have cache headers that i saw (I could be blind though).
Open for discussion or other proposals, but feel as though this is fairly necessary for non custom server users.  Would this or a similar logical implementation be feasible?

I noticed they have a imagesFolder which you can explicitly use for setting the output dir to chunks/images. Thank you for your quick response. Feel free to close this.


left, I return nil, and the left will appear a blank.  (version 2.4.2)

I download 2.4.2, and only make right actions appear.
SwipeCellKit-2.4.2.zip

I download the 2.4.2 code, and only make the right actions appear
SwipeCellKit-2.4.2.zip

Duplicate of #215


Apostrophes are removed

Apostrophes are removed. For example : "I'm lucky" is expected to become "i-m-lucky", bug it produces "im-lucky".
Maybe I can solve this problem with options ? If yes, I didn't find how.

This is not a bug. The module removes all punctuation.

OK, maybe in English, it sounds OK. But in French (at least) I think apostrophes should be replaced by hyphens. Is there a way to solve this problem ? I'm ready to contribute if you suggest me where to start.
--- or ---
Is there a way to add a custom array of characters without removing/replacing the default charmap?


new matcher toBeNil

adding new matcher ###toBeNil()
Housekeeping

[x] Unit tests

Codecov Report

Merging #96 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #96   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          74     76    +2
  Lines         338    347    +9
  Branches       51     53    +2
=====================================
+ Hits          338    347    +9

Impacted Files
Coverage Δ

src/matchers/toBeNil/predicate.js
100% <100%> (ø)

src/matchers/toBeNil/index.js
100% <100%> (ø)

Continue to review full report at Codecov.

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

@subinoy7 You don't need to close the PR. Can make changes and push on the same branch.

@tejasbubane thanks for highlighting all the points, I am going to fix this right now and will update it. Thanks a lot :smiley:

@tejasbubane thanks for highlighting all the points, I am going to fix this right now and will update it. Thanks a lot 😃

Cool 👍  Also don't forget to add yourself as a contributor.
Your name will appear at the bottom of the README !! 🎉
To do that, run this command:
yarn contributor <YOUR_GITHUB_USERNAME>

See the contributing guide for more info.

@tejasbubane I have updated the files with changes.

@tejasbubane I have updated the files with changes.

Looks good 👍  I will let @mattphillips have a look and merge.

@subinoy7 great job all looks good to me! @tejasbubane thanks for taking the time to review this :smile:


Inconsistent documentation in case no possible actions

In case a player has no possible actions, the documentation is not clear.
ApplyActions states that the player should have the action kInvalidAction: https://github.com/deepmind/open_spiel/blob/53eeb6127578f91e88a6c0451983d10e3509446d/open_spiel/spiel.h#L557.
LegalActions seems to say that the player should have a empty list: https://github.com/deepmind/open_spiel/blob/53eeb6127578f91e88a6c0451983d10e3509446d/open_spiel/spiel.h#L229
However, from generate_playthrough.py, it seems the player should have kInvalidAction as action.
Also, if kInvalidAction is returned then cfr does not work https://github.com/TheoCabannes/open_spiel/issues/7.
If empty list is returned then both cfr and NFSP do not work:

Solve by https://github.com/deepmind/open_spiel/issues/588


Update documentation for schedule_task parameters in TaskSet (task.py)

The parameters "args" and "kwargs" were removed in commit 82c2d6a68f946bdd303642dff6a89e0d5c2bf298. However they still show in the latest documentation https://docs.locust.io/en/latest/api.html?highlight=schedule_task#locust.TaskSet.schedule_task

thx


Pure python implementation of Logicle Transform

I originally tried to implement the logicle transform in just python, inspired by the improved performance in python 3.11, but I had issues with precision - ultimately I couldn't replicate results in GatingML 2.0.
A pure python implementation would remove the reliance on pybind, simplify the code, and make it more maintainable.
Anyone want to give it a bash? Maybe you'll have more luck!

`import numpy as np
from scipy.optimize import brentq
class LogicleTransform:
def init(self, t, w, m, a=0.0):
self.t = t
self.w = w
self.m = m
self.a = a
self.x0 = self.w * (self.m - np.log10(self.t))
self.x1 = self.m + np.log10(self.t)
self.x2 = self.a - np.log10(self.t)
self.b = (self.x1 + self.x0) / 2
self.d = np.arcsinh((self.t * 10**(-self.x0)) / 2)
def __f(self, x):
    if x < self.x0:
        return - self.t * 10**(-self.x0)
    elif x < self.b:
        return self.t * ((10**(2*(x - self.b)) - 1) / (4 * np.e**(x - self.b)))
    else:
        return self.t * (np.e**(x - self.b) - 1)

def transform(self, data):
    transformed_data = [self.__f(x) for x in data]
    return transformed_data

def inverse_transform(self, transformed_data):
    def _inverse(x):
        try:
            return brentq(lambda y: self.__f(y) - x, -100, 100)
        except ValueError:
            return np.nan
    data = [_inverse(x) for x in transformed_data]
    return data

`


Add validation to (*Definition).Name

Why?
So that it cannot contain non-URL firendly chars.
How?
By validating and matching against the pattern below:
/^[A-Za-z0-9]+(?:-[A-Za-z0-9]+)*$/

@vgarvardt matches(pattern) doesn't work as expected I think it's due to escaping mechanism 'cause regex patterns should be back-quoted and struct tags are already back-quoted so there's no way to do that, am I wrong?
Also I think Validate() function is a good place to put various validation logic explicitly instead of adding more advanced validators to struct tags ;)

@azbshiri I created a branch from your branch and made your regexp to work with govalidator.
Here is the branch https://github.com/hellofresh/janus/tree/feature/name-slug-validation, and here is my commit https://github.com/hellofresh/janus/commit/55fb6256f8f42969aad733a7d9842eeff51ffe66 that implements the validation.
Do you mind if I close your PR and open new one from my branch that includes both your and my commits with the validation implementation?

@vgarvardt I have to ask a question, what if a new validation needed? you're gonna add one more validator to the struct tag?

@azbshiri My personal opinion - yes, we need to use as few ways of doing one thing as possible, but not less. At the moment we're using govalidator, so we're trying to avoid cusomt validation code if it is possible. If we'll need to add one more validator - the code may become too unreadable (validator+error text, too long string), so I'd created new custom validator, moved all the validation logic there (we still can call govalidator.Matches(...) etc. from it) and used something like:
type Definition struct {
    Name string `valid:"apiDefinitionName"`
    ...
}

great

Closing in favour of https://github.com/hellofresh/janus/pull/272 that is based on this PR.


Key system flows design doc

Update on the workflow document:
Port create + L2 neighbors - merged
Router + L3 neighbors - merged
Security group - in PR
NACL/EIP/SNAT - not started


Is it possible to download all the data files manually?

I am living in China and because of the Great Fire Wall I failed downloading data files in make every time. I have tried since the weekend, no matter download through VPN or not it makes no difference.
I will be very grateful if there is a way to download data files manually.

Hey @cottonty, the files are hosted in the "us-east-1" AWS region (northern Virginia in the US), so downloading hundreds of megabytes from outside of the country is probably slow. It seems like many VPN services are now banned within China
According to Amazon, hosting files in the Beijing region requires having a Chinese company with an ICP license, so that's a no-go. The closest option I could find is hosting the files on the CloudFront CDN, which has edge locations in a few nearby places: Hong Kong, Tokyo, and Seoul. So still have to cross the GFW but the data's at least closer to mainland China.
I've set up a CloudFront distribution for the model files as of #183, so if you pull latest and keep trying to run make (or can manually run the download script with ./src/libpostal_data download all $YOUR_DATA_DIR/libpostal), hopefully the nearby caches will populate and the download will eventually complete.
Also, if you happen to have any sources for address data in China that we can use for libpostal, I'd love to hear about them!

thank you.
but this time it stuck at here
address_expansions/
address_expansions/address_dictionary.dat
numex/
numex/numex.dat
transliteration/
transliteration/transliteration.dat
make[2]: *** [all-local] Error 7
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2
actually, could you give me all the download addresses of needed files like this
http://libpostal.s3.amazonaws.com/models/address_parser/2017-03-04/parser.tar.gz
I'm able to download this with download software
having these I can then extract them into the data directory.

Ok, so the basic data files use a simple url, and it looks like you already have them:
base data files: https://d1p366rbd94x8u.cloudfront.net/libpostal/data/v1/libpostal_data.tar.gz
For the models, the dates may change over time. But the urls can be obtained by checking for a file called "latest" in either of the model directories, like so:
address_parser
latest=$(curl --silent https://libpostal.s3.amazonaws.com/models/address_parser/latest)
url=https://d1p366rbd94x8u.cloudfront.net/models/address_parser/$latest/address_parser.tar.gz

language classifier:
latest=$(curl --silent https://libpostal.s3.amazonaws.com/models/language_classifier/latest)
url=https://d1p366rbd94x8u.cloudfront.net/models/language_classifier/$latest/language_classifier.tar.gz

After the tarballs have been downloaded and extracted, create the following files so libpostal's make doesn't attempt to download the data again:
echo $(date -u) > $YOUR_DATA_DIR/libpostal/last_updated
echo $(date -u) > $YOUR_DATA_DIR/libpostal/last_updated_parser
echo $(date -u) > $YOUR_DATA_DIR/libpostal/last_updated_language_classifier
echo "v1" > $YOUR_DATA_DIR/data_version

thank you so much!!
it works!

Hey @cottonty, the files are hosted in the "us-east-1" AWS region (northern Virginia in the US), so downloading hundreds of megabytes from outside of the country is probably slow. It seems like many VPN services are now banned within China as well, so that might be part of it.
According to Amazon, hosting files in the Beijing region requires having a Chinese company with an ICP license, so that's a no-go. The closest option I could find is hosting the files on the CloudFront CDN, which has edge locations in a few nearby places: Hong Kong, Tokyo, and Seoul. So still have to cross the GFW but the data's at least closer to mainland China.
I've set up a CloudFront distribution for the model files as of #183, so if you pull latest and keep trying to run make (or can manually run the download script with ./src/libpostal_data download all $YOUR_DATA_DIR/libpostal), hopefully the nearby caches will populate and the download will eventually complete.
Also, if you happen to have any sources for address data in China that we can use for libpostal, I'd love to hear about them!

hi,sorry to bother you ,and I am in China too,just tried the method referred, but it doesn't matter .I still can't  download the files ,would you pls help me?

Hi , Could anyone post the live link for downloading all the data files ? The above links are no longer functional.
Its would be great if anyone can provide all the .dat files after the model has been trained.

Sorry, I’m  also  looking for the data ,we can keep contact if anyone have progress
发件人: Abhiram<EMAIL_ADDRESS>发送时间: 2019年4月4日 13:54
收件人: openvenues/libpostal<EMAIL_ADDRESS>抄送: lichenying (A<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>主题: Re: [openvenues/libpostal] Is it possible to download all the data files manually? (#182)
Hi , Could anyone post the live link for downloading all the data files ? The above links are no longer functional.
Its would be great if anyone can provide all the .dat files after the model has been trained.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/openvenues/libpostal/issues/182#issuecomment-479760099, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ApLeFf3p3QFB_DOvUom8zNLNqoF0kVinks5vdZNfgaJpZM4M_Tea.


adroid:api-29 is missing latest build tools

Build tools 29.03 is missing from the api-29 image.

I had added it here but realized those files are autogenerated. https://github.com/CircleCI-Public/circleci-dockerfiles/pull/26

Thanks. Working on it.

I also ran into this issue:

As I was testing api-29 with a different build-tools version I noticed /opt/android/sdk/build-tools/29.0.2/llvm-rs-cc: error while loading shared libraries: libncurses.so.5: cannot open shared object file: No such file or directory.

I'm not sure yet about your latest comment. If that's an issue with the image, please create a new Issue.


Make sure that username validation matches nearcore

Right now client-side validation doesn't match nearcore and it's very confusing when one tries to create new account.
Should match nearcore validation and provide explanation to user on what exactly is wrong with username.

fixed by https://github.com/near/near-wallet/issues/123


Converter: Geocode the travelHistory with the same system as the location field

Sorry for the big diff :\ This PR is doing a few things:

Reusing the same geocoding data for the travelHistory.location as is used in the original CSV for geocoding the location field. This gets us (1) better coverage than pycountries (countries & counties only), (2) more consistency, and (3) full geospatial data (lat/longs).

Changing the script to take in a path to the location of a locally-checked out nCoV19 repo, for (1) access to the CSV, and (2) for access to the geocoding script & data.

Adding some very common abbreviations used in the travelHistory.location source, since fixing every one of these would be a lot of work.

Remove hacks and workarounds, like removing 'Travelled to' prefixes. We're logging these as errors so we can fix them at the source.

Note that the existing geocoding script from the other doesn't do exactly what we need here, hence geocode_util.py.
This reduces errors quite a bit, down to 2,441 total errors, the vast majority of which seems like problems we need to solve in the CSV, rather than fixable geocoding misses.
Some example outputs:
    "travelHistory": [
      {
        "dateRange": {
          "start": {
            "$date": "2020-03-09T00:00:00Z"
          },
          "end": {
            "$date": "2020-03-09T00:00:00Z"
          }
        },
        "location": {
          "country": "Spain",
          "geometry": {
            "latitude": 40.3396,
            "longitude": -3.6226599999999998
          }
        }
      }
    ],

    "travelHistory": [
      {
        "location": {
          "country": "United Arab Emirates",
          "geometry": {
            "latitude": 25.204849,
            "longitude": 55.270782
          },
          "administrativeAreaLevel1": "Dubai",
          "administrativeAreaLevel2": "Dubai"
        }
      }
    ],

Just curious -- is the precision of the geocoder pretty good? On one hand we're already using it in the sheet; on the other hand, we're trying to achieve a higher level of data validity/fidelity here.

yeah it seems pretty decent to me -- the main limitation is just the breadth (i.e. it doesn't have paris, tx 😝), since it will generally encompass data that's been seen in the location field of the dataset so far. but i don't think that's going to be a major issue here, and if a row is missing, this script will log an error and we can add it.
what's nice about the dataset is that it does have thinks broken down by geo precision, so it's easy to get the correct data (like centroid) for just the country

 Resolve errors in line-list data import


Support asynchronous client with Tornado

Currently there's no stable JSON RPC client for Tornado and Python3 in the Cheese Shop.

I've added an AsyncClient class, so by subclassing that instead of Client removes the need for a future or callback which was a workaround to get this library to work asynchronously. The tornado client could be updated to something like:
async def _send_message(self, request, **kwargs):
    response = await self.http_client.fetch(
        self.endpoint, method='POST', body=request, headers=headers, **kwargs)
    return self._process_response(response)

The new class uses the async/await syntax though, so switching to that would make this only work on python 3.5+.


[Ready for Review] Fix logging of title/description updates; improve Node.update

Fixes a bug in logging title and description updates
Consolidates node updating logic in Node#update.
Adds named constants Node.PUBLIC and Node.PRIVATE.
Sending a PUT or PATCH to update title or description with the same content will not generate multiple logs

Ticket
https://openscience.atlassian.net/browse/OSF-4668

Tested, LGTM

Seems to work well. No complaints about the code. Go for it. :tada: :penguin: :tada:


Reconstruction for Hint Manager.

Fixes #1289.
Changes proposed in this pull request:

Reconstruct Hint manager.
Reconstruct Hint manager holder.
Modify some cases.

Pull Request Test Coverage Report for Build 5244

32 of 33 (96.97%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.007%) to 62.869%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/src/main/java/io/shardingsphere/core/hint/HintManagerHolder.java
28
29
96.55%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/src/main/java/io/shardingsphere/core/hint/HintManagerHolder.java
1
94.44%

sharding-orchestration/sharding-orchestration-reg/src/main/java/io/shardingsphere/orchestration/reg/newzk/client/cache/PathResolve.java
1
93.75%

sharding-orchestration/sharding-orchestration-reg/src/main/java/io/shardingsphere/orchestration/reg/newzk/client/zookeeper/base/Holder.java
1
73.77%

Totals

Change from base Build 5243:
-0.007%

Covered Lines:
8063

Relevant Lines:
12825

💛  - Coveralls


Error executing any method that uses "filecoin-solidity"

Hi,
Yesterday I was testing a smart contract using "filecoin-solidity" and everything went well.
But today I'm having this error all the time:
Error: transaction failed [ See: https://links.ethers.org/v5-errors-CALL_EXCEPTION ]
Nothing changed at my side from yesterday, using pretty much the same deployed smart contract.
When I call another method on the same smart contract that do not use "filecoin-solidity", no errors.
Thanks.

:link: zboto Link

@andborges I think the issue is solved by the PR #234. Please, re open the issue if it still not working.


Remove employees from info to provide

What is the context of this PR?
Remove "employees" from information to provide in schema if the survey doesn't require employees. Fixes #450.
How to review
Open one of 102, 203 or 205 surveys and observe the statement "numbers of employees" doesn't appear in the information to provide statement on landing page. Open one of 112, 213, or 215 surveys and observe the statement does appear.
Who worked on the PR
@iwootten

Current coverage is 78.81% (diff: 100%)

Merging #451 into master will not change coverage

@@             master       #451   diff @@
==========================================
  Files           139        139
  Lines          4951       4951
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           3902       3902
  Misses         1049       1049
  Partials          0          0

Powered by Codecov. Last update d2c4fbd...058cbc7

Current coverage is 78.81% (diff: 100%)

Merging #451 into master will not change coverage

@@             master       #451   diff @@
==========================================
  Files           139        139
  Lines          4951       4951
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           3902       3902
  Misses         1049       1049
  Partials          0          0

Powered by Codecov. Last update d2c4fbd...bc1f2b2

Current coverage is 78.81% (diff: 100%)

Merging #451 into master will not change coverage

@@             master       #451   diff @@
==========================================
  Files           139        139
  Lines          4951       4951
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           3902       3902
  Misses         1049       1049
  Partials          0          0

Powered by Codecov. Last update d2c4fbd...bc1f2b2

Current coverage is 78.81% (diff: 100%)

Merging #451 into master will not change coverage

@@             master       #451   diff @@
==========================================
  Files           139        139
  Lines          4951       4951
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           3902       3902
  Misses         1049       1049
  Partials          0          0

Powered by Codecov. Last update d2c4fbd...4d5d709

Current coverage is 78.81% (diff: 100%)

No coverage report found for master at e880aae.

Powered by Codecov. Last update e880aae...4d5d709

Current coverage is 78.81% (diff: 100%)

Merging #451 into master will not change coverage

@@             master       #451   diff @@
==========================================
  Files           139        139
  Lines          4951       4951
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           3902       3902
  Misses         1049       1049
  Partials          0          0

Powered by Codecov. Last update e880aae...b8b576d


Feature request: MeasurementCollectionToFHIR function to support "POST" data to multiple FHIR servers.

@Team I would like to add a new feature to support the Transformed data to be posted to multiple FHIR servers or at least synchronization service between parent and child FHIR servers, where in the current setup. it supports the data to be posted to only one FHIR server that has been configured while deploying the connector.
pointers for the same is much appreciated.

Hello @venkateshdevireddy5353 this can be achieved with the existing architecture. In OSS you can have one device event hub but add additional consumer groups.  For each consumer group you would have a SA Job + Azure Function that targets the specific FHIR services you wish to export the data to.
In our new preview version we released to PaaS (https://docs.microsoft.com/en-us/azure/healthcare-apis/iot/) we have decoupled the normalization from the destination.  While preview is currently limited to just one FHIR Destination at this time we  could support more if there is enough demand for the feature.  Would you be able to describe your specific scenario better for us to understand?

Hi @dustinburson Thanks for your prompt response, I will just explain the scenario that I am trying to make in our development, We have two FHIR servers deployed(1. Azure API for FHIR and 2. Microsoft open source FHIR server) and we have an IoMT connector deployed with which we are using the services of "Azure API for FHIR" and the process is smooth but we also want to send the data to second server which is MS FHIR Server(which doesn't have any restrictions, I mean to say to make a POST request to it, there are no restrictions), using the connector that we have deployed to Azure App service, how can we send the data to our other server(MS open FHIR server).
if you could give us some pointers on how to achieve this that would be very much helpful.

Hi @dustinburson I was trying the above approach but couldn't find the OS FHIR Server's resource ID and it's not allowing me to add a URL as well and btw our OS FHIR server is deployed via AKS.

Nope, I have deployed the Microsoft's IoMT-FHIR connector in Azure portal and trying to send the data to a Open Source Microsoft FHIR Server deployed in AKS.(Note: MS IoMT-FHIR connector is working fine with Azure API for FHIR by MS, but wanted to add this additional functionality as well), please help me with some pointers.

Supported in PaaS, Azure Function solution in OSS has been deprecated in favor of containerized services.


Write a category-theoretic description

At its core, yaml.yaml just wants to be a functor that maps between the yaml and lambda categories.

With yaml we get a human friendly data serialization standard for all programming languages. More specifically I see:

specification
syntax
(de)serializers
performance
community
advanced editors
online editors

On the other hand, the yaml spec is pretty dense and we don't want to deal with everything from the beginning.

The mapping to lambda is not just for the math, but we actually want to run software in a computer. In #2 I took a lambda calculus interpreter written in Rust and I was able to map simple documents to lambda terms.
A different approach could map between languages (transpiling). Haskell seems to be the closest but I'm not familiar with the syntax or tooling.

I'm reading de Bruijn and I think we can use it to define a functor between the YAML number scalars and Lambda calculus in de Bruijn notation categories.

https://web.engr.oregonstate.edu/~walkiner/teaching/cs581-fa20/slides/5.LambdaCalculus.pdf

https://www.win.tue.nl/automath/archive/pdf/aut029.pdf

LAMBDA CALCULUS NOTATION WITH NAMELESS DUMMIES,
A TOOL FOR AUTOMATIC FORMULA MANIPULATION,
WITH APPLICATION TO THE CHURCH-ROSSER THEOREM
N. G. DE BRUIJN
(Comrnunicatod at tho mooting of Juno 24, 1972)
In ordinary lambda calculus the occurrences of a bound variable are made
recognizable by the use of one and the same (otherwise irrelevan+) name at all
occurrences. This convention is known to cause considerable trouble in cases of
substitution. In the present paper a different notational system is developed, where
occurrences of variables are indicated by integers giving the "distance" to the
binding L instead of a name attached to that A. The system is claimed to be efficient
for automatic formula manipulation as well as for metalingual discrission. As an
example the most essential part of a proof of the Church-Rosser theorem is presented
in this namefree calculus.

We can think of the de Bruijn notation as a lower level lambda calculus form. Closer to the machine.
This is explained in the previous paper. It could be useful to write the first bootstrapping tools as in #5.

Maybe a list is de Bruijn notation and a mapping classical notation

Different approaches for mixing named and nameless appear in the literature:

https://blog.jez.io/variables-and-binding/
https://www.cs.ru.nl/~james/RESEARCH/haskell2004.pdf