Clarify the difference between DC_QUORUM and DC_SAFE_QUORUM

In the docs for consistency, I don't quite understand the difference between DC_QUORUM and DC_SAFE_QUORUM.

DC_QUORUM: Reads and writes are propagated synchronously to quorum number of nodes in the local data center and asynchronously to the rest. The DC_QUORUM configuration writes to the number of nodes that make up a quorum. A quorum is calculated, and then rounded down to a whole number. If all responses are different the local on is returned.
DC_SAFE_QUORUM: Similarly to DC_QUORUM, but the operation succeeds only if the read/write succeeded on a quorum number of nodes. If the quorum has not been achieved then an error response is generated by Dynamite.

A few notes on my confusion:

If all responses are different the local on is returned.

Should this be "the local one is returned"? It would be good to specify what "one" is referring to. I'm guessing it's the response to the read/write request?

DC_SAFE_QUORUM: Similarly to DC_QUORUM, but the operation succeeds only if the read/write succeeded on a quorum number of nodes.

When I read this, it implies that under DC_QUORUM, writes/reads are attempted against a quorum, but all but one could fail and that would count as a successful quorum write/read? Is that correct?
Would you be able to clarify what the differences are between these two settings, and their failure scenarios? My apologies if I've misunderstood this.

@danielcompton Your question is very valid. The difference between the two really comes in picture when the query succeeded on quorum nodes but the responses were not the same. If you are very sensitive to reads and writes getting propagated well, use SAFE_QUORUM option. DC_QUORUM gives looser guarantees on quorum in case data mismatch. SAFE_QUORUM will give you error on which you can retry or attempt to repair the data yourself. (straight forward for basic key value). But they both are sensitive to errors.
We might actually merge them both into one eventually.

Thanks for the reply. This is my current understanding:
Reads under DC_QUORUM

Reads are sent to a quorum of nodes
All of the nodes must respond with a value for a read to be successful. If less than a quorum responds, then the read fails.
If the values differ between nodes then the local read is returned as authoritative (what happens if there is no local read? Is it possible for the client request routing to pick the wrong node to query on, and therefore have no local query?)

Reads under DC_SAFE_QUORUM

Same conditions as DC_QUORUM, but if the values differ between nodes then an error is returned.

I'm not clear on how writes would differ between the two quorum consistency types, could you clarify this?

If the writes are done with DC_ONE and reads are done with any of the two quorum types, the values that are read may differ. This is typically done in situations where the application is very write heavy but sensitive to data inconsistency during reads.
Does that answer your question?

Not quite, what I'm wondering is whether there is any possible difference in behaviour for writes under DC_SAFE_QUORUM and DC_QUORUM?

It has the same difference for writes as for reads. In redis writes do return different values for example Expire. I think for most cases DC_SAFE_QUORUM is correct. Like I said before, we might combine them both into just one option. Please feel free to reach out if this still doesnt answer your questions.
We also use gitter for real time chats if you want further conversation

Has this been solved? Closing the issue. Please reopen if you still have questions.


Per object rid info for 3.2

Made a pr branch for @lawnjelly to review per rid object info. Per rid object info is not trivial to port to Godot Engine master, but I wanted to share.
Partially supports https://github.com/godotengine/godot-proposals/issues/63 by having individual object information.

Improvement notes:

The calculation are wrong. Instancing isn't being subtracted.
2d render info isn't in.
Write proposal

I don't expect this to be merged in 3.2.
Sponsored by IMVU.

I'm not actually so familiar with the 3d rasterizer, but don't we maintain stats already for the overall number of calls..., yes storage->info.render.vertices_count.
Perhaps an alternative paradigm could be to just note when we start on an object which is selected for stats, record the current storage->info.render.vertices_count, then when hitting an object that is not selected, record storage->info.render.vertices_count and find the difference?

Overall though more render stats are welcome :+1: imo, I did a proposal on this too:
https://github.com/godotengine/godot-proposals/issues/248
Myself I'm also particularly interested in being able to display non rendered stats such as the number of source verts (rather than the number of indices). I should edit that proposal to make that clear actually. The number of source verts gives an indication of the success of an import and whether vertices are being shared or too many are unique. This would not need to go via the rasterizer, it could be a query of the mesh instance itself.

Since this PR is for 3.2 and I don't have the effort to pr this for master, I'm going to close the PR. It is salvageable.


Error: No state in response (iframe)

I am trying to do the initial login in an iframe, which means opening the login screen of my identity provider (here Keycloak) in an iframe.
The iframe is embedded on my own login page.
Everything is setup for the Authorization Code Flow + PKCE.
After hitting the login button in my iframe, I get an
Error signinRedirectCallback:  Error: No state in response
error.
If I do the same login outside my iframe, everything works well.
Does anyone have an idea what I am missing here?

Assuming you see all the right protocol parameters in the response, is your starting origin and callback origin the same?

Assuming you see all the right protocol parameters in the response, is your starting origin and callback origin the same?

It's the same domain at least. The redirect points back to the previous application. For easier debugging, I'm redirecting to a /callback sub page of the same application.

@brockallen The error is thrown by catching signinRedirectCallback(). When I debug the lib, I see that the response somehow is completly undefined, not just the state field:

While debugging further, the SigningResponse constructor gets populated with some values as expected:

Weird. Maybe it gets called several times, where in one time there are no values present? But how?

Weird. Maybe it gets called several times, where in one time there are no values present? But how?

Do you have more than one instance of the UserManager? If so, don't so that :)

Weird. Maybe it gets called several times, where in one time there are no values present? But how?

Do you have more than one instance of the UserManager? If so, don't so that :)

Good point. Maybe.
The login is fulfilled on the server side. Keycloak has a valid user session. The problem seems to be only in the frontend, although no tokens are getting stored in the storage. If I trigger signinRedirect() again, after ´signinRedirectCallback()`, the client shows logged in state suddenly.
My issue seems to be project specific here, although not sure if the lib has a problem with iframes. I am evaluating keycloak.js + iframe now. Lets leave it like that for now. If you have no more ideas, we can close the issue, but keeping signIn in iframes in mind as beeing potentially tricky.

To finally this issue, I probably had two instances of the UserManager, one in the app and one inside the iframe.
Anyways, I switched to keycloak.js as this seems to have better support with our Keycloak server. Thanks for your hints, @brockallen.

This seems to be most def an issue. The same thing is happening in my app. no state status gets caught in the catch promise. Even though the application has already retrieved a token on the first call. It seems the lib internals is still not aware of that and keeps running itself internally. A few seconds go by and it then stops.
Interestingly enough, it only happens when there is currently no tokens, (at the very first login auth [first callback]), the first redirect and subsequent app visits, works all well.
I'm using it in a react app/ts/redux
I have tried placing the code in a redux store and in a class, even furthering turning the manager into a static variable of that class.
My next step is to really dig into the lib itself and check why state is {} and  its getting called multiple times even though the token has already been retrieve in the first call

@codercodingthecode You are probably having two instances of the UserManager, just like me before, although I can not verify this anymore. Are you trying the login inside an iframe, like me before? If so, don't do it, been there :)

@codercodingthecode You are probably having two instances of the UserManager, just like me before, although I can not verify this anymore. Are you trying the login inside an iframe, like me before? If so, don't do it, been there :)

It is very unlikely as I set it to be static, exactly for this purpose, to make sure I wasn't having multiple instances of that class instantiated around the app with new instances of new UserManager being called.
I also used a Memo for a pure FC component, to avoid some unnecessary reloads (this actually helped a little) but didn't solve the issue completely.
I have a couple more things to try that I came across earlier today. It seems like the state and the query string might get something to do with this.
I will dig into the library this weekend to see what's happening. Perhaps we can work on a more specific error log as well.
What is cracking my brain is that, if there is no token and the user is not sso yet, it gets redirected to the login. No error here.
Then, after the user has placed the auth and redirected back, for the very first token retrieval, this is where it keeps on retrying. After this very first, all consecutive ones work well.
Somewhere down the line, the internals seems not to be aware that the token has already been retrieved and keeps on for a little longer.

I have made some changes to the file structure, yet to no avail. No matter where it lives, redux store, static class, context api, state.
It always recycles after the first load.
I have placed a counter on the context API I have placed it in now with a static counter to check on instantiations.
The last network call sometimes gets cancelled, sometimes not.
The token is available already in the first load.
Upon debugging, the callback keeps on getting called and signInRedirectCallback keeps running into the catch error, again, though the token is already in local storage.

The two instances may occur if you are trying to login inside an iframe. One instance is in the context of the iframe, the other one is in the context of the iframe's parent. I guess that was my mistake back then. Both instances won't share its state between each other. I suggest you go with the regular flow, just how it is meant to be, by allowing a regular redirect to your auth server and then redirect back to your app. So no iframe embedding as it might break the flow and make things complicated.

There is 2 IFRAMES, but the library is dispatching them.
1st we are supposed to instantiate the library, then, we need to have the routes to redirect to login and another one with a callback auth.
The callback auth function/promise is the one going crazy.
This is the flow. Everything is done according to the wacky docs. Or many other examples around.
And in react, I'm not sure about another way to go with this. The other react wrapper libs, use the same lib under the hood causing the same issues, so, it's I know it is not this specific implementation here.
I noticed that the browser issues cookies/checksessions series of errors due to LAX or lack of samesite settings to auth while locally developing.


update bpdm service and add tests

Note: Why we have those
@JsonProperty("premiseType") @JsonNames("type", "premiseType") @SerializedName("premiseType")
val type: TypeKeyNameUrlDto<PremiseType>

First:

Acapy has a problem with the word "type" in the credential subject
For the connection with BPDM we using the default serializer of Kotlin -> @JsonNames("type", ..)
For AcapyService and connection with Acapy we had to use "Jackson serializer" at the time of the implementation -> @JsonProperty("premiseType") and the issued credentials will have "premiseType" in their subjects
The Acapy java library is using Gson -> @SerializedName("premiseType")
The Credentials subject will have "premiseType" and we are converting a credential subject back a BPDM object. Therefore we need @JsonNames("type", "premiseType")

Thanks for the updates, the code changes look good.
I did some advanced testing with the local setup, including issuance of BPDM credentials for a self-managed wallet which worked successfully. Unfortunately, I was not able to update any data in the BPDM API to test update and revoke always getting unspecific "400 Bad Request" errors in the PUT operation even when using the unchanged object returned by the BPDM API itself. I still think we should test this, but didn't want to spend too much time.
While testing, I found the following issues:

Startup of the MIW service gave an error apparently related to the scheduled job run of the BPDM update, I did not really understand the details from the logs, but it looks like it was trying to trigger a revocation (not clear why) and ran into a timeout with the revocation service, maybe this was triggered before the signature callback was ready. Could you please have a look and check if there is a way to prevent or delay executing the scheduled job during startup?

2022-11-29 10:44:03.265 [OkHttp http://localhost:11000/...] DEBUG o.h.a.w.TenantAwareEventHandler - walletId: null, topic: settings, payload: Settings(authenticated=true, noReceiveInvites=false, label=CatenaXIssuer, endpoint=http://cx_acapy:8000/, helpLink=null)
2022-11-29 10:44:07.874 [db-scheduler-pool-1-thread-2] INFO  io.ktor.client.HttpClient - REQUEST http://localhost:8086/management/lists/issue-credentials failed with exception: io.ktor.network.sockets.SocketTimeoutException: Socket timeout has expired [url=http://localhost:8086/management/lists/issue-credentials, socket_timeout=unknown] ms
2022-11-29 10:44:07.951 [db-scheduler-pool-1-thread-2] TRACE c.g.k.scheduler.DueExecutionsBatch - Batch state: generationNumber:1, stale:false, triggeredExecuteDue:false, possiblyMoreExecutionsInDb:false, executionsLeftInBatch:0
Exception in thread "db-scheduler-pool-1-thread-2" java.lang.StackOverflowError
	at java.base/java.lang.reflect.AccessibleObject.checkAccess(AccessibleObject.java:673)
	at java.base/java.lang.reflect.Method.invoke(Method.java:560)
	at ch.qos.logback.classic.spi.ThrowableProxy.<init>(ThrowableProxy.java:66)
	at ch.qos.logback.classic.spi.ThrowableProxy.<init>(ThrowableProxy.java:60)
...
	at ch.qos.logback.classic.spi.ThrowableProxy.<init>(ThrowableProxy.java:60)
	at ch.qos.logback.classic.spi.ThrowableProxy.<init>(ThrowableProxy.java:72)
2022-11-29 10:44:08.471 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - RESPONSE: 200 OK
2022-11-29 10:44:08.471 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - METHOD: HttpMethod(value=GET)
2022-11-29 10:44:08.471 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - FROM: http://localhost:11000/resolver/resolve/did:sov:DU5t8DE5xeHgaftKW3Qp7L
2022-11-29 10:44:08.472 [ktor-apache-dispatcher-worker-3] DEBUG o.e.t.m.s.IWalletService$Companion - HTTP status: 200
2022-11-29 10:44:08.472 [ktor-apache-dispatcher-worker-3] DEBUG o.e.t.m.s.IWalletService$Companion - HTTP description: OK
2022-11-29 10:44:08.473 [ktor-apache-dispatcher-worker-4] INFO  io.ktor.client.HttpClient - BODY Content-Type: application/json; charset=utf-8
2022-11-29 10:44:08.473 [ktor-apache-dispatcher-worker-4] INFO  io.ktor.client.HttpClient - BODY START
2022-11-29 10:44:08.475 [ktor-apache-dispatcher-worker-3] INFO  io.ktor.client.HttpClient - {"did_document": {"@context": ["https://www.w3.org/ns/did/v1"], "id": "did:sov:DU5t8DE5xeHgaftKW3Qp7L", "verificationMethod": [{"id": "did:sov:DU5t8DE5xeHgaftKW3Qp7L#key-1", "type": "Ed25519VerificationKey2018", "controller": "did:sov:DU5t8DE5xeHgaftKW3Qp7L", "publicKeyBase58": "7o77inn8LkdZdtBbVffcj3GoceLQPTtoXdLNif5V9EGy"}], "authentication": ["did:sov:DU5t8DE5xeHgaftKW3Qp7L#key-1"], "assertionMethod": ["did:sov:DU5t8DE5xeHgaftKW3Qp7L#key-1"], "service": [{"id": "did:sov:DU5t8DE5xeHgaftKW3Qp7L#did-communication", "type": "did-communication", "serviceEndpoint": "http://cx_acapy:8000/", "routing_keys": [], "recipient_keys": ["did:sov:DU5t8DE5xeHgaftKW3Qp7L#key-1"], "priority": 1, "accept": ["didcomm/aip2;env=rfc19"]}]}, "metadata": {"resolver_type": "native", "resolver": "IndyDIDResolver", "retrieved_time": "2022-11-29T09:44:08Z", "duration": 3520}}
2022-11-29 10:44:08.475 [ktor-apache-dispatcher-worker-3] INFO  io.ktor.client.HttpClient - BODY END
2022-11-29 10:44:08.766 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - REQUEST: http://localhost:11000/jsonld/sign
2022-11-29 10:44:08.766 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - METHOD: HttpMethod(value=POST)
2022-11-29 10:44:08.766 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - BODY Content-Type: application/json
2022-11-29 10:44:08.767 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - BODY START
2022-11-29 10:44:08.767 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - {
  "doc" : {
    "credential" : {
      "id" : "http://localhost:8080/api/credentials/status/f4caf0c5-cb2c-4e22-8e1d-9049e546a3d2",
      "@context" : [ "https://www.w3.org/2018/credentials/v1", "https://w3id.org/vc/status-list/2021/v1" ],
      "type" : [ "VerifiableCredential", "StatusList2021Credential" ],
      "issuer" : "did:sov:DU5t8DE5xeHgaftKW3Qp7L",
      "issuanceDate" : "2022-11-29T09:44:00Z",
      "credentialSubject" : {
        "id" : "http://localhost:8080/api/credentials/status/f4caf0c5-cb2c-4e22-8e1d-9049e546a3d2#list",
        "type" : "StatusList2021",
        "statusPurpose" : "revocation",
        "encodedList" : "H4sIAAAAAAAAAO3BMQEAAADCoPVPbQwfoAAAAAAAAAAAAAAAAAAAAIC3AYbSVKsAQAAA"
      }
    },
    "options" : {
      "proofPurpose" : "assertionMethod",
      "type" : "Ed25519Signature2018",
      "verificationMethod" : "did:sov:DU5t8DE5xeHgaftKW3Qp7L#key-1"
    }
  },
  "verkey" : "7o77inn8LkdZdtBbVffcj3GoceLQPTtoXdLNif5V9EGy"
}
2022-11-29 10:44:08.767 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - BODY END
2022-11-29 10:44:11.181 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - RESPONSE: 200 OK
2022-11-29 10:44:11.182 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - METHOD: HttpMethod(value=POST)
2022-11-29 10:44:11.182 [eventLoopGroupProxy-4-1] INFO  io.ktor.client.HttpClient - FROM: http://localhost:11000/jsonld/sign

While playing around with the BPDM API, I found the endpoint /api/catena/legal-entities/search that seems to allow to retrieve the data of up to 5000 BPNs in one step - is there a reason why we don't use this?
In the GET /api/wallets endpoint of the MIW, the value of isSelfManaged always seems to be false even for self-managed wallets. Get wallet by identifier seems to show the proper value.


Synthesia 10.2

Thank you for the contribution. It was merged directly as b23c91b7d7a24bcff9284c719d067d12bc6bcd87 to keep commit history cleaner. Your contribution is still credited to you.

Thank you for the contribution. It was merged directly as 37687f07d47d71fab3999f0c57358493c09f54bd to keep commit history cleaner. Your contribution is still credited to you.


Can I get non localized numbers from RenderListDataAsStream?

What type of issue is this?
Question
What SharePoint development model, framework, SDK or API is this about?
SharePoint REST API
Target SharePoint environment
SharePoint Online
SharePoint Subscription Edition
SharePoint 2019
Additional environment details
No response
Issue description
Date values will be localized when returned from RenderListDataAsStream (i.e. 1 000 for Swedish, and 1,000 for English US). I know that I can opt out of localization for dates using the datesInUTC parameter.
Is the something similar for numbers?
Localized dates makes parsing them to numbers very hard since return format can vary depending on browser settings. Ideally they would be JSON numbers.

Number columns should have equivalent parsable floats versions along with the localized version. Just add a "." at the end of the field name. So if you have myNumber column in SP, row.myNumber will give you the localized number and row["myNumber."] should give you a string representation of a float.

Thanks that works greata


Issue/7694 executor cleanup

Description
Closes https://github.com/inmanta/inmanta-core/issues/7694
Add 2 config options:

executor_retention: executors idle for longer than this get cleaned up
executor_cap_per_agent: if this limit is already reached when attempting to create a new executor for this agent, the oldest executor is first stopped.

Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
- [ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
- [ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

I added the PoolManager and PoolMember interfaces suggested here as a first step, but I think further integration should go into another ticket ?

Processing this pull request

Merged into branches master in 04983c47f5d37c9ded5b386f09a74e9c943da462


Create documentation page for Python SDK unrecoverable errors

Add a site page to cover what we consider unrecoverable errors as part of the effort to reduce detection times.
Staged at https://apache-beam-website-pull-requests.storage.googleapis.com/28702/documentation/sdks/python-unrecoverable-errors/index.html

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @tvalentyn @riteshghorse

@tvalentyn ready for a second pass


Android: Google login fails because custom URI scheme is not allowed

Recently I used flutter_appauth to login my users via Google into the app. But because I am also using flutter_web_auth_2 for authenticating against many other OAuth1/OAuth2 providers, there were 2 Android activities/intents being filtered on redirects causing a popup for the user having to choose one of the activities. That's why I decided to get rid of flutter_appauth and just use this package for everything.
But... Google has deprecated custom URI schemes for new Android clients and suggests the official Google Android SDK.

Is it possible to use this package with Android clients to sign users in via Google, without enabling the deprecated custom URI schemes?

That's a tough one... I guess it is most advisable to just use google_sign_in which does the entire authentication natively (and using best practices for the specific use case of Google logins on Android).
From what I can read online, it does not seem to be easily possible to use flutter_web_auth_2 then for Google signin anymore without some compromise. I will probably add that to the README when I am 100% sure it cannot be done

I made it work with google_sign_in, but implementation is not straightforward - there are many stepstones along the way, as Google does not seem to implement a standard OIDC. Even more, Android and iOS have to be treated differently. I tried to update the documentation of the google_sign_in package (https://github.com/flutter/packages/pull/6292) but my pull request got rejected by Google.
I think you can close this issue. With a reference to my pull request people should be able to find some current documentation about the implementation.

Thank you very much for updating this issue with your findings! Indeed, for future reference, it should be enough to reference either this issue or your PR - hence I will close this issue here as well. If I find something new (which is unlikely, but one can dream...), I will reopen it and see what I can do :)


Dynamic RTP payload types break streaming in non Chromium browsers

Your environment.

Version: 3.1.34
Browser: Firefox 100.0.2 (on MacOS and Arch Linux)

Ran the pion/mediadevices webrtc example
What did you expect?
The example to work in major browsers
What happened?
The example works fine in Chromium based browsers. In Firefox everything connects correctly and the video stream shows it is playing but there's no output. The audio stream does not play. Same issue with desktop Safari, mobile Firefox, and mobile Safari.
I ran into the same issue with the VP8 codec. I wasn't able to get a stream working with Safari iOS either. I think it was the same issue but haven't confirmed yet.
Additional info
Here's the SDP log for a failed session in Firefox: https://gist.github.com/matt-allan/4bbed61755adc9cb5ef67a8e8dbe226a
This is what about:webrtc looks like when this is happening:

And this is what  I'm seeing:

This is what I see for desktop Safari:

Debugging notes
I've found a lot of previous issues suggesting this is caused by Pion using a hardcoded payload type for the RTP packets when they should be dynamic. It works in Chrome because the payload type matches what Chrome uses. It doesn't work in Firefox because Firefox uses different type IDs than Chrome.
I was able to confirm that changing the payload type from 125 to 126 (because Firefox is trying to use 126) in codec.go of the mediadevices package fixes the issue for Firefox. However I'm filing an issue here and not a PR in that repo since it appears to be an issue with the core webrtc code and not the mediadevices codec.
From what I can tell it's expected for browsers to use dynamic payload types:

payload type
values in the range 96-127 MAY be defined dynamically through a
conference control protocol
https://www.rfc-editor.org/rfc/rfc3551.html#section-6

For some (older) codecs like PCMU the payload type is assigned statically as described in RFC3551. For others, it is assigned dynamically through the SDP. Note: there are no guarantees on the stability of a payload type assignment.
For this allocation, the range from 96 to 127 is used.

https://webrtc.googlesource.com/src/+/HEAD/pc/g3doc/rtp.md
The doc comment for payloadType indicates it can be negotiated. It looks like maybe this line does that but I added some logging and it's never hit in my case.
Related issues
There are prior issues that appear to be about the same thing:
https://github.com/pion/webrtc/issues/1030
https://github.com/pion/webrtc/issues/925
https://github.com/pion/ion-sfu/issues/23
A lot of these issues were closed but they link to a page that 404s so I can't tell what code change may have addressed this. If you can point me to the right place I can try and make a PR.

Hey @matt-allan
Thanks for the detailed bug! Pion WebRTC does support dynamic PayloadTypes, so I don't believe the issue is coming from this repo (but I have been wrong before!)
TrackLocal is an interface. I believe the issue is that pion/mediadevices is missing code like this
Debugging can you check the PayloadType here and see what it is ?

Hey @matt-allan
Hopefully this worked for you! Resolving for now, if you are still having issues please re-open
thanks for using Pion :)


Pattern:- Table of Content:- In landscape mode title are not getting displayed in bold(iOS).

Issue:- On iOS mobile phone in landscape mode 'Table of Content' titles are not getting displayed in bold.
Steps to reproduce the issue

Enter the url in thebrowser
Click on 'Table of Contents'
Turn mobile in landscape mode.

Additional information
OS:- iOS 12, 13
Device :- iPhone 11 pro, 11, Xs, Xs max, X, 8
Browser:- Chrome, Safari

Moving this out to sprint 7.

This is caused by IOS and how it handles the font-sizes on landscape, as this is a low priority and none of the found solutions are suitable, we are putting this on hold.

@RobertaJHahn I don't think this is a Sprint Must Have, right? I think we should push this out.
cc: @ljcarot @RaphaelAmadeu

I agree we can push this out if sev 4.

@RobertaJHahn moving this out to the next sprint.

Hey @chetankan, I was checking it and seems to be bold, could you check again ?
This screenshot is from Iphone 11 Pro IOS 13

@chetankan checking again...

Hey @chetankan, could you check again ?? PR: #3614


Added RecordId to case search end point

Replaced Mosaic ID search with Record ID Search in ProcessDataGateway
Mosaic ID search in ProcessDataGateWay would never be triggered
Mosaic ID triggers a different path in UseCase

merged development branch, resolved conflicts and made minor tweaks to routes and validation


First class module types not exported

I have two Reason files:
testservice.re:
[@genType]
type client = {foo: string};
module TestService = {
  let request = (client: client, request: Testtype.testType) =>
    Testtype.convert(request);
};

module type TestServiceSig = {
  let request: (client, Testtype.testType) => Testtype.respType;
};

module type ITestServiceSig = {
  let request: Testtype.testType => Testtype.respType;
};

module type TestServiceClient = {let client: client;};

[@genType]
type testServiceClient = (module TestServiceClient);

[@genType]
type iTestServiceSig = (module ITestServiceSig);

[@genType]
let makeTestService =
    (client: testServiceClient, options: Js.Dict.t(string))
    : iTestServiceSig => {
  module Client = (val (client: (module TestServiceClient)));
  module IService: ITestServiceSig = {
    let request = TestService.request(Client.client);
  };
  (module IService);
};

and testservice2:
[@genType]
let initService =
    (client: Testservice.testServiceClient, options: Js.Dict.t(string))
    : Testservice.iTestServiceSig =>
  Testservice.makeTestService(client, options);

which generates to testservice.gen.js:
/**
 * @flow strict
 * @generated
 * @nolint
 */
/* eslint-disable */

// $FlowExpectedError: Reason checked type sufficiently
import * as testserviceBS from './testservice.bs';

// flowlint-next-line nonstrict-import:off
import type {Dict_t as Js_Dict_t} from './Js.gen';

// flowlint-next-line nonstrict-import:off
import type {respType as Testtype_respType} from './Testtype.gen';

// flowlint-next-line nonstrict-import:off
import type {testType as Testtype_testType} from './Testtype.gen';

export type client = {| +foo: string |};

export type testServiceClient = {| +client: client |};

export type iTestServiceSig = {| +request: (Testtype_testType) => Testtype_respType |};

export const makeTestService: (testServiceClient, Js_Dict_t<string>) => iTestServiceSig = function _(Arg1, Arg2) {
  const result = testserviceBS.makeTestService([[Arg1.client.foo]], Arg2);
  return {request:function _(Arg11) {
      const result1 = result[0]([Arg11.foo, Arg11.bar]);
      return {foo:result1[0], bar:result1[1]}
    }}
};

and testservice2.gen.js:
/**
 * @flow strict
 * @generated
 * @nolint
 */
/* eslint-disable */

// $FlowExpectedError: Reason checked type sufficiently
import * as testservice2BS from './testservice2.bs';

// flowlint-next-line nonstrict-import:off
import type { Dict_t as Js_Dict_t } from './Js.gen';

// flowlint-next-line nonstrict-import:off
import type { iTestServiceSig as Testservice_iTestServiceSig } from './Testservice.gen';

// flowlint-next-line nonstrict-import:off
import type { testServiceClient as Testservice_testServiceClient } from './Testservice.gen';

export const initService: (
  Testservice_testServiceClient,
  Js_Dict_t<string>,
) => Testservice_iTestServiceSig = testservice2BS.initService;

As you see signatures of initService and makeTestService are different and I can't use initService on js side using object as client parametr.
I think in such case we should call initService not from testservice.bs but from testservice.gen

@meafmira can you expand a bit on what is the issue you have encountered?
It looks like the signatures of initService and makeTestService are the same after considering the imported types.
For example, this type checks for me:
import * as TestService from "../TestService.gen";
import * as TestService2 from "../TestService2.gen";
import type { Dict_t as Js_Dict_t } from "../../src/shims/Js.shim";

let foo = (x: TestService.testServiceClient, y: Js_Dict_t<string>) =>
  (TestService2.initService(x, y): TestService.iTestServiceSig);

let bar = (x: TestService.testServiceClient, y: Js_Dict_t<string>) =>
  (TestService.makeTestService(x, y): TestService.iTestServiceSig);

@cristianoc sorry for misunderstanding. You’re right - signatures are identical but there different bodies for these two functions. makeTestService has conversion from js object to oxana module (array) but initService has not

@meafmira thanks, indeed the conversion is missing.
Looks like module type declarations are not picked up from other files at the moment, which needs fixing.


Create Calendar Month Markdown Table

Candidate syntax:
=cal November 2021
=endcal

Actually more like =cal 2021 11.
And I think we can highlight with a CSS class particular days:
=cal 2021 11
=caldays blue 14 15 16 17
=caldays green 18
=endcal

blue and green are classes - and the day  numbers would be wrapped with  elements with class= set.

Added syntax:
=calnote 17 A

adds annotation to calendar entry for 17th day.
This annotation can have a  in it to colour it.
Also added
=calkeys blue Day we have data for

which generates a bullet coloured using class blue and text Day we have data for to act as a kind of key to colouring.

A third parameter to =cal defines how many lines each row should occupy:
=cal 2021 11 3

Gives 3 lines in a row. The default is 2.
Here's a complete test calendar:
### November 2021

=cal 2021 12 2
=caldays blue 14 15 16 17
=caldays orange 18
=calkey blue We have data for **somehow**
=calkey orange Is focus day
=calnote 17 A
=calnote 18 B
=calnote 4 X
=endcal

* An unrelated bullet
  * An unrelated sub-bullet

Obviously - with the md2pptx dynamic contentSplit specification - this tests well with md2pptx.

Shipped in v0.6.


Add ECharts and WindiCSS

https://github.com/apache/echarts
https://github.com/windicss/windicss
Also added CSS pattern on TailwindCSS. Set confidence level at 75% because WindiCSS also uses the same CSS pattern, but doesn't have the same user base as TailwindCSS.

Hey mate, thanks for PR!! If desired, you can make several changes to bring to the ideal. It would also be great to have an example for each regex.

Hey, just fixed the issues you mentioned.
The regex's explanations:
For /echarts\\.min\\.[a-zA-Z0-9]*\\.js would be echarts.min.8asd156as.js, for example. (bundled code, noticed it on a website I use when trying to figure out other ways to detect)
For cdn\\.jsdelivr\\.net/(?:npm|gh/apache)/echarts@([\\d.]+(?:-[^/]+)?|latest)/dist/echarts.*\\.js would be<EMAIL_ADDRESS>or<EMAIL_ADDRESS>For /echarts(?:\\.simple)?(?:\\.esm)?(?:\\.common)?(?:\\.min)?\\.js would be any of echarts.simple.js, echarts.esm.js, echarts.common.js or echarts.min.js


error export and importing a database that contains lucene indices

export database was OK.
import database got this error which is very important because the database was left unusable and I needed to delete and recreate it (thank god I had the console commands saved in a txt file and scripts to recreate it). Otherwise import fails. This is dangerous for putting into production.
Post.body is a lucene fulltext index.
The exported file shows this line in it:
{"name":"Post.body","type":"FULLTEXT","algorithm":"LUCENE","clustersToIndex":["post"],"definition":{"defClass":"com.orientechnologies.orient.core.index.OPropertyIndexDefinition"}
this is the error:
Rebuilding indexes of truncated clusters ...Error on database import happened just before line 0, column 898
com.orientechnologies.orient.core.exception.OCommandExecutionException: Index 'Post.body' not found
at com.orientechnologies.orient.core.sql.OCommandExecutorSQLRebuildIndex.execute(OCommandExecutorSQLRebuildIndex.java:89)
at com.orientechnologies.orient.core.sql.OCommandExecutorSQLDelegate.execute(OCommandExecutorSQLDelegate.java:90)
at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.executeCommand(OAbstractPaginatedStorage.java:1452)
at com.orientechnologies.orient.core.storage.impl.local.OAbstractPaginatedStorage.command(OAbstractPaginatedStorage.java:1433)
at com.orientechnologies.orient.core.command.OCommandRequestTextAbstract.execute(OCommandRequestTextAbstract.java:63)
at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.command(ONetworkProtocolBinary.java:1319)
at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.executeRequest(ONetworkProtocolBinary.java:396)
at com.orientechnologies.orient.server.network.protocol.binary.OBinaryNetworkProtocolAbstract.execute(OBinaryNetworkProtocolAbstract.java:223)
at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:77)
Error: com.orientechnologies.orient.core.db.tool.ODatabaseExportException: Error on importing database 'social' from file: /home/arisalexis/test.export
Error: com.orientechnologies.orient.core.exception.OCommandExecutionException: Index 'Post.body' not found

hi @arisAlexis
which version of OrientDB?
can you send me the exported database or
some script in order to reproduce the issue?
Thanks

version is 2.1.4 I sent you in email the data.

I am sorry. It's not clear that opening the console and doing things while the server is running would cause this. it imports correctly now.

pinning down the problem, the import doesn't work (with the error given) when connected as remote with a server running.

@arisAlexis
the problem happens with remote right?
In plocal we have a test that check import/export with lucene
https://github.com/orientechnologies/orientdb/blob/2.1.x/lucene/src/test/java/com/orientechnologies/lucene/test/LuceneExportImportTest.java
I will try with remote.

yes without the server running and plocal it imports correctly.

Further testing:
I create from the studio a new database. I connect from the console.sh with remote. I import the file. it works OK. Again I import the same file, it produces the error message.
this is a good test case I think unless two consecutive imports should not happen but only in a clean new database? maybe I misread the docs.

should be fixed, closing


Double rendering because of CDNIntlProvider mismatch

When not passing the messages on client side but having them on server side a mismatch happens, and the whole app renders twice.

Warning: Expected server HTML to contain a matching <div> in <div>

In my opinion this should not happen.
Text mismatches still happen but these do not cause a re-render of the whole app, e.g.:

Warning: Text content did not match. Server: "Hang on..." Client: "Hold on..."

We simply could remove this.state.loaded to change this behaviour:
https://github.com/allthings/elements/blob/2b86f249708ded2af4e84afba0f978360e359cf9/src/behaviour/CDNIntlProvider.jsx#L91-L97

/cc @mosch @yamafaktory

Hey @dan-lee, good catch! I think we should not render everything if it's not loaded yet.
We can pass in messages to the component and it should be in the loaded state right away. So we I see two options there:

Load messages using loadLanguage before you hydrate on client side and pass them directly to the IntlProvider
Render messages into the DOM when you do SSR, so the IntlProvider can pick them up from there without any delay

Let's find the best option and add this to the docs!

In my opinion we should render something to prevent the mismatch and eventually prevent a complete remount of the app. I'd prefer flashing text messages of everything disappearing and appearing again.
Of course this shouldn't happen when the messages are rehydrated from the server. But in some cases this may not happen (for some reason) and we would prevent a remount, virtually flashing the whole page.
I like your idea mentioned in the last paragraph, but I am not sure if this can work out. We can give it a try 👍

Fixed with #178


Update player-tracker.user.js

To be able to see the nicknames of players under their labels

I clean out the dead/commented code. what's about last commit ?

Could you post a screenshot of what it looks like?

Here:

Is there an option to turn off this layer? In a busy area I suspect this might be unusable.

I'm sorry I haven't tested it yet, but with similar types of changes, I have seen problems when the map gets moved or pound or zoomed, or the person in question moves on the map. Are those cases handled correctly in this one?

@mxxcon

Is there an option to turn off this layer? In a busy area I suspect this might be unusable.

Yep, it looks like this :

@pleasantone

I'm sorry I haven't tested it yet, but with similar types of changes, I have seen problems when the map gets moved or pound or zoomed, or the person in question moves on the map. Are those cases handled correctly in this one?

I have not encountered such a problem when the map gets moved or zoomed

ES6 isn't supported by PhantomJS (let declaration)

(Unexpected token: name (iColor) (line: 433, col: 8))

It used in many screenshot bots, and other projects.


Fixing log4j-maven-shade-plugin-extensions dependency

Issue #, if available:
Description of changes:
Fixing
io.github.edwgiz
log4j-maven-shade-plugin-extensions
2.17.1
Dependency in lambda samples.
Ref:
https://mvnrepository.com/artifact/io.github.edwgiz/log4j-maven-shade-plugin-extensions/2.17.1
https://github.com/edwgiz/maven-shaded-log4j-transformer/tree/2.17.1

I made this fix also. I support merge. =)

Thanks for the PR. This sample has since been updated and the transformer dependency is no longer used. Please reopen if there is still an issue.


[Security] Vulnerability in lodash below 4.17.5 used by sass-graph@2

https://www.npmjs.com/advisories/782

Versions of lodash before 4.17.5 are vulnerable to prototype pollution.
The vulnerable functions are 'defaultsDeep', 'merge', and 'mergeWith' which allow a malicious user to modify the prototype of Object via {constructor: {prototype: {...}}} causing the addition or modification of an existing property that will exist on all objects.
Remediation
Update to version 4.17.11 or later.

Resolution: update to sass-graph@latest. The currently used version of sass-graph is the last at v2, so some testing will be needed to upgrade to v3
I'm happy to investigate a PR, though would be helpful to understand the usage of sass-graph before I get too deep. Is there anything that has prevented the upgrade in the past?

We welcome a PR.
On Fri., 15 Mar. 2019, 6:24 am Lachlan Heywood<EMAIL_ADDRESS>wrote:

https://www.npmjs.com/advisories/782
Versions of lodash before 4.17.5 are vulnerable to prototype pollution.
The vulnerable functions are 'defaultsDeep', 'merge', and 'mergeWith'
which allow a malicious user to modify the prototype of Object via
{constructor: {prototype: {...}}} causing the addition or modification of
an existing property that will exist on all objects.
Remediation
Update to version 4.17.11 or later.
Resolution: update to sass-graph@latest. The currently used version of
sass-graph is the last at v2, so some testing will be needed to upgrade to
v3
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2614, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWINK3Inyusrq3WVR7ypLKDfoDMzvks5vWqHrgaJpZM4b1CWA
.

Per lodash/lodash#4348, an upgrade is required to address a recent security vulnerability CVE-2019-10744 and now version >= 4.17.11 is recommended. This new vulnerability applies to a module used in this repo mergeWith -- I will issue a PR with another upgrade to account for this latest vulnerability.


feat(webpack): add initial configuration

:tada: This PR is included in version 0.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Make three dialects of Chinese clear.

对三种中文的写法做了调整，统一清晰。
Made the local labels of Chinese dialects clear.

@theosp Good!


Hardcoded limits changed as configurable.

Changes in this pull request:

Changed the hardcoded limits to configurable from reference.conf
This changes belongs to only Membership(groups).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Completed changes for batch change, zone , recordset for limits as configurable.


Format for co-working spaces

We need to decide on a first version of how we want to store the co-working spaces metadata.
One file vs one file per space (maybe even folders for countries, states, cities)?

one file per space could be nice as it would allow us to skip spaces with wrong properties
issue could be to many files in a folder error
Question: How do the two options behave in reviews

Which file format (yaml / json / ...)?
.yml seems pretty easy to write
Which properties should we add per space for the first version?
Version 1

name
key / id (string like de-kiel-fleet7)
links (array)

name (exp booking system, Instagram)
url

description
email
phone
address (as string)
location

lat
lon

images (array)

url
alt text

Version 2

opening_hours TBD
prices TBD

@anbraten I also added logo with url as a property.

@anbraten I think our format is fine for now so let's close this issue and see what additional properties are necessary in the future?


Move web parameters to standalone package

Hello!
In the sprit of the monorepo, I am proposing that the web UI parameters become their own package. This will allow folks to compose a lightweight UI with just the needed components. I'm thinking of the regl-renderer and these parameters controls.
There is no code change here, just moving files to a new folder in /packages.

@danmarshall hmm... so the idea is to lighten up the web UI? There’s a lot of stuff in there so I just wondering what else could be broken out.

@danmarshall sorry. I read this again.
Ok. So, you want to create a small web based UI which packages the core (for compiling), regl-renderer (for viewing), and parameters (for handling options).
I think there’s some good reasoning here but wonder if the parameters mean anything outside of the JSCAD web UI. I’m fairly fluent in VUE now and know that the current code would be kind of worthless. Not sure about REACT based UI components.
Thoughts?

Yes if someone wanted this in native Vue or React these would be additional. But if you’re in vanilla js then this package is useful.

A few more questions... sorry.
The code in designParameters() relies on a most style call back. I understand that the HTML wrapper is valuable but don’t think the callback should be based on most. Wouldn’t this be more useful split apart?
Also, I came across another critical piece of code for web UI, called wallkFileTree(). And like you, wanted the code separated from the JSCAD web UI. I moved that to core/src/web/
Wouldn’t these pieces for parameters fit into the core package? There’s core/src/parameters/ as well.

I'm planning on doing all the core file loading offline then webpacking it. So the core won't be a part of the viewer experience. I'm sot sure of the most callback, the only dependency is the nanohtml: https://github.com/jscad/OpenJSCAD.org/pull/769/files#diff-1b0c5b5ea9251678b787b60cc85e0c8d7c68e762a3368bace32ff3ca939436bcR32

Understood.
I have another tutorial planned for the Vue components that creates an input form from the design parameters. It could reuse this piece as well... hmm...
Can you wait for a week while I try to create the new tutorial?

Of course, no hurry at all :)

Hi @z3dev - I just created a new viewer tool called jscad-now which currently uses copy-pasted versions of these parameter files. I feel like there is more refactoring to do here, since I had to grab code from both web and core to get a complete working parameter feedback loop.

Hi @z3dev - I just created a new viewer tool called jscad-now which currently uses copy-pasted versions of these parameter files. I feel like there is more refactoring to do here, since I had to grab code from both web and core to get a complete working parameter feedback loop.

@danmarshall thanks. I haven’t had time to look yet at the parameters tutorial. I lost my machine on Friday. 😔 but I’m getting back on board again.
I was expecting a few more pieces would be required. If you have suggestions then let me know.
Also, I saw the post on the new app  (?). It looks exciting as JSCAD needs a replacement for the desktop app. Maybe we could collaborate on something.

@z3dev happy to collaborate! Give jscad-now a try. Also happy to pull it into this repo if you think it's up to that level.

I've been thinking about this, and think these parameters should be in vanilla javascript. I don't think we can commit to a framework like Vue or React (or angular) because not everybody uses the same framework.

Just an update, I’m working on the new tutorial for VUE now, and have reached the point where I need to create HTML for the parameters. I’ll probably start down the same road, and make a copy of those functions.
I’ll post back when I have some more progress.

@danmarshall here's an update.
i created a new tutorial for the vue components that uses the definitions and the default values provided from the callback. it took a while to get that 'vue' mind set back but i happy to say that all works well.
the only component required was parameterControls. but vue is slightly different, so the conversion of definition to HTML element was the key part; createParamRowFromDefinition(). the whole HTML from createParamControls() would be unusable inside a vue component.
also, i was kind of surprised that the returned HTML element from createParamRowFromDefinition() did not contain the parameter 'name' as a HTML attribute. so, i made a small change to the code to add the 'name' as attribute 'paramenter-name'

Thoughts on a new package...
We have some cleanup to perform on V2 planned, which includes the breakout of 'most' based functions from the core package. I really doubt that the next application will use 'most', so those will probably be moved into the web package.
This will make the core package smaller, and allow further automated testing. It looks like a few of the functions from web need to be moved to core, and facilitate the development of some new applications, like jscad-now. I also hope to rework the code slight to make the 'vtree' pluggable, not compiled in to the core.
So, I'd like to resist this new package, and pursue a more compact 'core'.
Thoughts?

I'm not quite sure what core does - it seems like a lot of file system stuff? I don't use core in jscad-now. Browserify is what pulls in files. So where will the parameters go? Into core, or a new vue component?

I'm not quite sure what core does - it seems like a lot of file system stuff? I don't use core in jscad-now. Browserify is what pulls in files. So where will the parameters go? Into core, or a new vue component?

Hmm... If jscad-now isn't using the core then how are projects possible? Basically, rebuildGeometry* functions are the core, but require a whole lot more, like IO packages, etc.

I'm not quite sure what core does - it seems like a lot of file system stuff? I don't use core in jscad-now. Browserify is what pulls in files. So where will the parameters go? Into core, or a new vue component?

Hmm... If jscad-now isn't using the core then how are projects possible? Basically, rebuildGeometry* functions are the core, but require a whole lot more, like IO packages, etc.

I see. So, if the project stays within the limits of the packages included then all should work.
The only missing piece will be the extra support for require() of an external format, e.g. require('./my.stl')

Hmm it seems even outside of jscad-now, in an ordinary node module, that require('./my.stl') would require that module to probably deserialize in order to use it? At this time I haven't imported stl files , I may have to investigate. Probably a browserify plugin would be needed.
Back to the parameters though, where will they live in the future? Probably in a ui package, which doesnt quite seem like core.

Back to the parameters though, where will they live in the future? Probably in a ui package, which doesnt quite seem like core.

I’m not sure there’s any advantage, as Browserify will only assemble the pieces required. Wouldn’t a small ‘core’ suffice?
It would be reasonable to make a new package if it’s generally useable by various applications.
For example, the vue components build a library from core, and regl-renderer. I think Browserify does all the work... discarding unused functions... including only necessary functions.

I just did a quick test. So it depends on how its imported.
const core = require("@jscad/core");
console.log(core.parameters)

This bundles the whole core - 47k lines of code
const parameters = require("@jscad/core/src/parameters");
console.log(parameters)

This bundles just the parameters, 227 lines of code.
The 2nd option is acceptable. Now the only missing part is the group of files from web. I'm unable to install @jscad/web because it references an unpublished version of<EMAIL_ADDRESS>- latest is 0.4.1

The 2nd option is acceptable. Now the only missing part is the group of files from web. I'm unable to install @jscad/web because it references an unpublished version of<EMAIL_ADDRESS>- latest is 0.4.1

Yeah. I hope @kaosat-dev will accept the changes, and merge into most-gestures

@z3dev - another reason for the standalone package, is that I've been contemplating that Maker.js use the same format/schema for parameters. For this to work it would make sense that the package be standalone. Other projects too may adopt the format, knowing they can do so in a lightweight way. Just a thought.

@danmarshall Got it!
Just some feedback...
I think those those two files from core should also go into this min-package. It seems those will be required if another website is created.
In addition, some of the code could be broken into smaller pieces, which will allow proper test suites to be added.

Final note... I’m releasing V2 JSCAD on April 1st. So, if this can wait just a little longer then these changes can be merged and released easily. There are still some steps to make the V2 JSCAD release process clean.

@danmarshall are you still interested in creating a small package for the parameter functionality?
if so then let's restart as the 'master' branch is now the working branch for changes. :)

@danmarshall lets restart on these changes. Lots of conflicts as well.


Stellar airdrop qualification confusion

Hi although its been confirmed on app that ive met all conditions to qualify when I want to join the airdrop the message says "Sorry, you dont qualify"
Can yiu please check it
Many thanks

https://github.com/keybase/client/issues/20264 known issue

Thanks weird situation

hello dear... make sure read the rules


⚠ Adacta-fintech.com has degraded performance

In a8811e5, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 200
Response time: 2414 ms

Resolved: Adacta-fintech.com performance has improved in 9af81ae.


first four revised (the previous one wasn't saved properly)

Hey Brenda! I was able to merge this myself; now that these changes are in the data branch, you might be able to commit and push your local changes (slack me if you would like to chat about this or have questions!)
FWIW, I don't think you did anything wrong here, I think you just had pushed changes early enough in the process that some of the changes I pushed later confused git about where your repo was relative to mine!


fontWeight value must be cast as string type

When defining styles to use in a component, the fontWeight attribute is requiring a string type value.  In normal CSS this is not the case.  bold and 700 are required to be cast as strings when using React Native ^0.24.1 in iOS 9.3 simulator for iPhone 6s Plus on OSX 10.11.4
Fails:
// Create our component
var DayItem = React.createClass({
  render: function() {
    return <Text style={this.style()}>
      {this.props.day}
    </Text>
  },
  style: function() {
    return {
      color: this.color(),
      fontWeight: this.fontWeight(),
      fontSize: this.fontSize(),
      lineHeight: this.lineHeight()
    }
  },
  color: function() {
    var opacity = 1 / this.props.daysUntil;
    return 'rgba(0, 0, 0, ' + opacity + ')';
  },
  fontWeight: function() {
    var weight = 7 - this.props.daysUntil;
    return weight * 100;
  },
  fontSize: function() {
    return 60 - 6 * this.props.daysUntil;
  },
  lineHeight: function() {
    return 70 - 4 * this.props.daysUntil;
  }
});

Works:
// Create our component
var DayItem = React.createClass({
  render: function() {
    return <Text style={this.style()}>
      {this.props.day}
    </Text>
  },
  style: function() {
    return {
      color: this.color(),
      fontWeight: String(this.fontWeight()),
      fontSize: this.fontSize(),
      lineHeight: this.lineHeight()
    }
  },
  color: function() {
    var opacity = 1 / this.props.daysUntil;
    return 'rgba(0, 0, 0, ' + opacity + ')';
  },
  fontWeight: function() {
    var weight = 7 - this.props.daysUntil;
    return weight * 100;
  },
  fontSize: function() {
    return 60 - 6 * this.props.daysUntil;
  },
  lineHeight: function() {
    return 70 - 4 * this.props.daysUntil;
  }
});

From the docs:
fontWeight enum('normal', 'bold', '100', '200', '300', '400', '500', '600', '700', '800', '900')
Specifies font weight. The values 'normal' and 'bold' are supported for most fonts. Not all fonts have a variant for each of the numeric values, in that case the closest one is chosen.

While the styles class is trying to mimic CSS as much as possible, there are many examples where string and integer conversion don't mix. Making sure your fontWeight is a string is not an issue that needs addressing, I don't think.

As stated by @DannyiCracked this is as stated by the documentation.


Update Strict/Pass.lhs

ghc-7.4 complains about the Note constructor for Expr.  In ghc-7.0 Expr had constructors
    ...
     | Cast (Expr b) Coercion.Coercion
     | Note Note (Expr b)
     | Type TypeRep.Type

where
 data Note = SCC CostCentre.CostCentre | CoreNote String

In ghc-7.4 which permits the -fplugin flage  CoreSyn uses a more elaborate constructor Tick which seems to carry the same information, and then some:
   ...
   | Cast (Expr b) Coercion
   | Tick (Tickish Id) (Expr b)
   | Type Type

with
data Tickish id
 = ProfNote {profNoteCC :: CostCentre.CostCentre,
             profNoteCount :: !Bool,
             profNoteScope :: !Bool}
 | HpcTick {tickModule :: Module, tickId :: !Int}
  | Breakpoint {breakpointId :: !Int, breakpointFVs :: [id]}

In any case with this revision the result ghc --make -fplugin=Strict.Plugin NonTerminating.hs yields a prompt stack overflow, as desired.

Looks good, thanks!


Incorrectly sends passwordConfirmation instead of password_confirmation on sign_up

When I examine both the post payload and the Rails log I notice that the parameter name is incorrect for the password confirmation:

Processing by DeviseTokenAuth::RegistrationsController#create as JSON
  Parameters: {"name"=>"Mickey Mouse"<EMAIL_ADDRESS>"password"=>"[FILTERED]", "confirmPassword"=>"[FILTERED]", "confirm_success_url"=>"http://localhost:4200/register", "registration"=>{"name"=>"Mickey Mouse"<EMAIL_ADDRESS>"password"=>"[FILTERED]", "confirmPassword"=>"[FILTERED]", "confirm_success_url"=>"http://localhost:4200/register"}}
Unpermitted parameters: :confirmPassword, :confirm_success_url, :registration
Unpermitted parameters: :confirmPassword, :confirm_success_url, :registration
   (0.2ms)  BEGIN
  User Exists (0.9ms)  SELECT  1 AS one FROM "users" WHERE "users"."email" = $1 LIMIT $2  [["email"<EMAIL_ADDRESS>["LIMIT", 1]]
   (0.8ms)  SELECT COUNT(*) FROM "users" WHERE "users"."provider" = $1 AND "users"."email" = $2  [["provider", "email"], ["email"<EMAIL_ADDRESS>   (0.3ms)  ROLLBACK
Completed 422 Unprocessable Entity in 247ms (Views: 0.7ms | ActiveRecord: 6.9ms)

If you write a standard validation, like so:
  validates_confirmation_of :password
  validates_presence_of :password_confirmation, if: lambda { |u| u.password.present? }

Then sign_up/registration fails because Rails thinks that password_confirmation is blank.

Never mind, I see that the confirmPassword is sent by my form, not this library


Remove font family workarounds from Shoppy

We already have external custom fonts support for all visualizations. Let's get rid of the font workarounds.

Closed by https://github.com/metabase/shoppy/pull/82


https://github.com/Andy1621/CT-Net/blob/main/scripts/train.sh

https://github.com/Andy1621/CT-Net/blob/main/scripts/train.sh
there are some mistakes on the train.sh

Sorry for late response! I just use train.sh as backup of experiments.
You can copy the command and run in your terminal.


docs: Add note about triggering builds with net-next

test-docs-please

Coverage decreased (-0.006%) to 42.962% when pulling 79b09ce3d7e91cd5991d0cec0b281807370494db on pr/brb/contrib-docs-ci-next into ba9798c3a97815f962b3ab9356d1cd741e68d233 on master.

Just a thing, I'll add ci/fail-fast also ;-)


Adopting YAML files according to TOSCA

TOSCA.meta file was added
metadata in the nxos_tosca_sample.yaml was modified according to
http://docs.oasis-open.org/tosca/TOSCA-Simple-Profile-YAML/v1.0/csprd02/TOSCA-Simple-Profile-YAML-v1.0-csprd02.html#_Toc445238250

Approved by @menib


Getting SKErrorDomain when trying to purchase

Any ideas? Running on iOS 11 device, trying to make a purchase.
I am signed into itunes account on the iPhone, and there is an internet connection.
InAppUtils.purchaseProduct("productname", (error, response) => {
       console.log("purchaseProuct callback");
        // NOTE for v3.0: User can cancel the payment which will be available as error object here.
        if (error) {
          console.log(error); //look below for the output
        }

        if(response && response.productIdentifier) {
           Alert.alert('Purchase Successful', 'Your Transaction ID is ' + response.transactionIdentifier);
           //unlock store here.
        }
     });

output of error:
{ code: 'ESKERRORDOMAIN0',
  nativeStackIOS:
   [ '0   sangbog                             0x0000000102cca790 RCTJSErrorFromCodeMessageAndNSError + 152',
     '1   sangbog                             0x0000000102cca6b8 RCTJSErrorFromNSError + 264',
     '2   sangbog                             0x0000000102f222b8 -[InAppUtils paymentQueue:updatedTransactions:] + 604',
     '3   StoreKit                            0x00000001988d0278 <redacted> + 104',
     '4   CoreFoundation                      0x0000000185491c20 CFArrayApplyFunction + 80',
     '5   StoreKit                            0x00000001988d01f4 <redacted> + 148',
     '6   StoreKit                            0x00000001988d122c <redacted> + 1220',
     '7   StoreKit                            0x00000001988d1dfc <redacted> + 144',
     '8   StoreKit                            0x00000001988d00a0 <redacted> + 168',
     '9   libdispatch.dylib                   0x000000010375149c _dispatch_call_block_and_release + 24',
     '10  libdispatch.dylib                   0x000000010375145c _dispatch_client_callout + 16',
     '11  libdispatch.dylib                   0x0000000103756050 _dispatch_main_queue_callback_4CF + 1192',
     '12  CoreFoundation                      0x0000000185573f20 <redacted> + 12',
     '13  CoreFoundation                      0x0000000185571afc <redacted> + 2012',
     '14  CoreFoundation                      0x00000001854922d8 CFRunLoopRunSpecific + 436',
     '15  GraphicsServices                    0x0000000187323f84 GSEventRunModal + 100',
     '16  UIKit                               0x000000018ea3e880 UIApplicationMain + 208',
     '17  sangbog                             0x0000000102b707c8 main + 124',
     '18  libdyld.dylib                       0x0000000184fb656c <redacted> + 4' ],
  userInfo: { NSLocalizedDescription: 'Kan ikke oprette forbindelse til iTunes Store' },
  message: 'Kan ikke oprette forbindelse til iTunes Store',
  domain: 'SKErrorDomain' }

Well that was my bad :) Had forgotten to enable In App Purchases under Capabilities in the Xcode project. The error can occur due to many things, but in this case it was no IAP capability :)


cmd/gc: does not understand old linker flags

liblink/obj6.c uses ctxt->debugstack to emit stack underflow checks. The flag was
meant to be initialized from -K 6l flag.
But now liblink is invoked from the compiler. And the compiler does not initialize
ctxt->debugstack. Moreover the compiler already uses -K flag for a different purpose.

I suspect this relates to a bunch of other flags as well.
We need to choose some other names for the old linker flags and read them in all users
of liblink.

There are lots of these it seems, my reading finds at least -Z, -W, -S, -Q, -1, -O (I think).


Options notation

Options format:
How can I use objects with copy, include, ignore properties in  fromJS/toJS calls?
This is not possible with current options notation or I'm wrong?
Options format is: {
  copy|include|ignore: <paths to properties>,
  <paths to properties>: {
    key|create|update: <callbacks>
  }
}
To fix this we need to change format to: {
  <paths to properties>: copy|include|ignore,
  <paths to properties>: {
    key|create|update: <callbacks>
  }
}
or as variant of this - replace <paths to properties> with real object structure...
Currently <paths to properties>  may be only single property name for key, create, update callback. This example will fail: {
  "a.b": {
    key|create|update: <callbacks>
  }
}
This was not fixed by #63 issue because related to indexes of arrays.
Options updating:
it's also not clear topic of options merging/updating for nested mapping.
We cannot use options for root in nested mapping in create/update because this will introduce dependencies between structures of objects.
As a solution for this I may suggest to use callback or <nested options definition> with create.
Options and arrays:
Currently all arrays are processed with compareArrays function which also sorts arrays. And how is it possible to use indexes of items in <paths to properties>?

Hi Friends,
I am facing one ignore mapping issue. Kindly help me to resolve the same.
I have a collection of employees and each employee contains below fields.
Key
Name
Amount
I would like to convert the employees array to observable with Key and Name for each employee as observable but Amount should not be observable. I am trying below mapping, But seems it not working.
var mapping = {
'ignore': ["Amount"]
}
self.BatchEmployees = ko.mapping.fromJS(listOfEmployees, mapping);
In result every fields are coming as observable.

@manishkrai Here's an example of what you need to do to have your code working. The idea is to define the copy option in the right place.
var options = {
    key: function(item) {
        return item.Key;
    },
    create: function(options) {
        return ko.mapping.fromJS(options.data, {
            copy: ['Amount']
        });
    }
};

var data = [
    {Key: '1', Name: 'John', Amount: 100},
    {Key: '2', Name: 'Doe', Amount: 200}
];

ko.mapping.fromJS(data, options);

http://jsfiddle.net/3gygwbu6/


Attributions added multiple times

We are currently experiencing unwanted behaviour when adding attributionControl and a CartoDB-layer to the same map. We want to be able to add controls after layers, but it seems the new behaviour of attributions in Leaflet maps is in conflict with this. Is this intended?
See this example:
http://jsfiddle.net/afLgdkuj/3/
This is probably related to https://github.com/CartoDB/cartodb.js/commit/fcf3356540d1164eb1d1277cf9fa38c0708e0917.

Hello @haraldkj! Thanks for your example! I can see the Statens kartverk attribution is being displayed twice, but I'd say that has to do with the way Leaflet works. If you add another L.control.attribution, it will also have that attribution... What's the behaviour you would expect? Or what exactly are you trying to accomplish?

We currently have a map stack which incorporates several different providers. We would like to add one attribution control after all other layers have been added. We would also like to remove the default leaflet prefix as in the fiddle.
An example is the map at http://www.nrk.no/norge/her-er-norges-nye-ordforere-1.12561458 which uses an older version of cartodb.js. The updated cartodb.js would result in an extra attributioncontrol on top of the map legend control.
Thanks.

Thanks again for the example. I think if you add more layers to the map, attributions should be displayed correctly. I don't see why you want/need to add another attributionControl after all layers have been added... could you please carify a bit more?

I might be wrong but, no matter when you add the L.control.attribution, it will display the attributions of all layers in the map. Take a look at this:
https://jsfiddle.net/afLgdkuj/5/

Yes, that makes sense. But it would be convenient to be able to decide when the control is to be added to the map, regardless of adding layers.
We don't want multiple controllers - we would like all of our attributions in one attribution control. We basically do this:
http://jsfiddle.net/Lt17d3f4/1/
Currently, after the commit to cartodb.js mentioned above, cartodb.js will create and add a new attribution control to the map - even if the property is set to false in Leaflet.
Maybe I'm wrong, but I think the more correct behaviour is to add attributions to the existing attribution control  - if any, and defer creating/adding of the attribution control to the encompassing application.
I can provide more examples if needed.

The thing is that we want to display the CartoDB attribution when a CartoDB layer is added to the map. That's the reason why need to add the attributionControl if the map doesn't have one.
Why wouldn't the following work?
http://jsfiddle.net/afLgdkuj/6/

I can see why you want the attribution to display regardless of Leaflet settings, but I still think it is somewhat counter-intuitive, since it overrides Leaflet behaviour.
We have developed an in-house map builder tool for journalists, built around Leaflet, CartoDB, Mapbox and several other services (modules are loaded on-demand). We try to avoid implementing custom logic when we can, and always add attributions to our maps (done as one of the last steps after all layers have been added).
If this is the intended behavior of cartodb.js going forward, I guess we would have to add some checks to handle this case. Will you consider providing some other means of controlling cartoDB attributions?
Thanks.

Hi @haraldkj
I spent some time trying to figure out how to make it work in the way you specify but I think it is not possible due the way Leaflet works.
Usually when you create a L.map an attribution control is created automatically and stored within the map.attributionControl variable. If you specify the attributionControl: false then the control is not created and the property map.attributionControl doesn't exists.
When you create a cartodb layer it tries to use the map.attributionContol if exists or creates a new one otherwise. In the last case, note the control is created but not stored withint the map.attributionControl.
Finally, if you add manually a new control like map.addControl(L.control.attribution({'prefix': false})) you have to attribution controls (without a reference to them), both showing the same attributions.
I propose you a different strategy that allows to decide you when to show the control:

Create a map with default attribution control.
Remove the control temporarily.
Add as many layers you want.
Add the control again.

See: http://jsfiddle.net/afLgdkuj/8
    var map = L.map('map').setView([65, 18], 5);

    // Hide attribute control and modify prefix
    map.attributionControl.removeFrom(map);
    map.attributionControl.setPrefix('');

    // Add layers...
    ...

    // Show control again
    map.attributionControl.addTo(map);

This way only one attribution control is used and it shows all the layers attributions.
Cheers.


Tabs completion disappear when the parameter type is uint

When i change parameter type from "int" to "uint",Tabs completion disappear in command line
Note:
This also affects vscode.
Steps to reproduce
a.ps1:

param
(
[uint]$abc
)

a.ps1 -tabkey  # Tabs completion disappear,but "[int]$abc" normal.
Important information:
1 This bug exists in 6.1 and has been fixed in 6.2 preview.1.
2 No issue, no pr. No one knows who was fixed and what was fixed. From a professional perspective, we should not condone this.
3 Both versions are using PSReadLine 2.0.0-beta3 (#7711), so I guess it has nothing to do with psreadline.
4 win10 17663.1 rs5 + powershell 5.1 has this bug,After i install PSReadLine 2.0.0-beta3, bug still.
We must have an issue and pr and test about this bug, right?
about this bug, i post issue，need test code now.
Environment data
centos75 + ps v6.1  #has this bug
centos75 + ps v6.2.0-preview.1  #not this bug
reporter
By the chinese powershell study & research group. QQ group No. is 183173532

I'm not 100% sure, but there's a good chance this was resolved due to my PRs.
Pretty sure before I edited things [uint] was not a valid type unless you manually add the accelerator.
If you type it as [uint32] it should still tab complete correctly in the old version.

yes ,you right.
i confirm byte,sbyte,int16,uint16,int32,uint32,int64,uint64,single,double,decimal. on 6.1 version. all normal.
only uint ,or with uint ,has this bug ,on 6.1 version.

Unless there is a particularly pressing need (or you can convince the PS team) I doubt it will be backported to PS 5.1. It's not a bug, it's just a new feature I added in as seemed appropriate with the new numeric literals I've been working on.
You'll note also that long and ulong accelerators are missing in order versions, but present in the current preview.
In discussion on the relevant PRs (linked in the most recent issue release / changelog) I believe the concern was raised, and the consensus was don't use the new type accelerators if you plan on running the script on older versions of PS.
It's there as a kindness and convenience measure to assist users who wish to be able to use the new type literals more seamlessly. It was never expected to be backported, at least in my mind.
/cc @SteveL-MSFT, if you have anything to add here?

@vexx32 is exactly right.  5.1 is considered done with exception of critical security fixes.  PSCore6 runs happily side-by-side with 5.1 so if one wants to leverage the new features, they should use PSCore6 on Windows.


Refs #35844 -- Used asgiref.sync.iscoroutinefunction() instead of deprecated asyncio.iscoroutinefunction().

Fixes DeprecationWarning:
'asyncio.iscoroutinefunction' is deprecated and slated for removal in Python 3.16; use inspect.iscoroutinefunction() instead.
ticket-35844

Thank you @felixxm for this branch 🌟
Question, following the deprecation notice that you included in the description, I was expecting iscoroutinefunction to be imported from inspect, but the diff shows imports from asgiref.sync. I have a preference that our code import it from its "truth source" (inspect), what's your motivation to import from asgiref?

We always import from asgiref.sync (you check the rest of the Django code) because we added this shim to asgiref  especially for us. We also use markcoroutinefunction from asgiref.sync. We can decide to switch to inspect.iscoroutinefunction when Python 3.12 will be the lowest supported version.


Needs fixed doesnt work.

Please fix. pretty sure this is the only theme on deck and its the only one I wanna use.

please elaborate or else nobody can help you. What was the problem?


Fix formatting for consistency

@kevinswiber could we get this merged? It kind of disturbs the reading flow especially because the class header in the Fields section is out of line with the others :)


Update index.md

@alexbuckgit : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@alexbuckgit: I'm sorry - only the author of this article, @timleyden, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


IGES file importer

Add a import_iges_file function to cqfiles.  More than likely this will be a wrapper around IGESControl_Reader() in the OCC API.

Done.  import_iges_file has been added to cqfiles.


[pybind11:x64-windows] debug link failure

Host Environment

OS:  Windows10
Compiler:   Microsoft Visual Studio Community 2022 (64-bit) - Current,   Version 17.5.3

To Reproduce
CMakeLists.txt file vvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv
cmake_minimum_required (VERSION 3.16)
project (firestoreTest)
set (CMAKE_CXX_STANDARD 11)
set (CMAKE_CXX_STANDARD_REQUIRED True)
set (CMAKE_TOOLCHAIN_FILE c:/projects/github/microsoft/vcpkg/scripts/buildsystems/vcpkg.cmake)
find_package (pybind11 REQUIRED)
add_executable (firestoreTest
"firestoreTest.cpp" "firestoreTest.h"
"pythonTest.cpp" "pythonTest.h"
"main.cpp"
)
target_link_libraries (firestoreTest
PRIVATE pybind11::embed
)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Steps to reproduce the behavior
vcpkg install pybind11:x64-windows
Build with above CMakeLists.txt.
Reported error:   vvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv

------ Build All started: Project: firestoreTest, Configuration: x64-Debug ------
[1/1] Linking CXX executable firestoreTest.exe
FAILED: firestoreTest.exe
cmd.exe /C "cd . && "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.exe" -E vs_link_exe --intdir=CMakeFiles\firestoreTest.dir --rc=C:\PROGRA~2\WI3CF2~1\10\bin\10.0.19041.0\x64\rc.exe --mt=C:\PROGRA~2\WI3CF2~1\10\bin\10.0.19041.0\x64\mt.exe --manifests  -- "C:\PROGRA~1\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.35.32215\bin\Hostx64\x64\link.exe" /nologo CMakeFiles\firestoreTest.dir\firestoreTest.cpp.obj CMakeFiles\firestoreTest.dir\pythonTest.cpp.obj CMakeFiles\firestoreTest.dir\main.cpp.obj  /out:firestoreTest.exe /implib:firestoreTest.lib /pdb:firestoreTest.pdb /version:0.0 /machine:x64 /debug /INCREMENTAL /subsystem:console  C:\Projects\github\microsoft\vcpkg\installed\x64-windows\debug\lib\python310_d.lib  kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib && cmd.exe /C "cd /D "C:\Projects\trials\28.2 - Firestore test\firestoreTest\out\build\x64-debug" && C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -noprofile -executionpolicy Bypass -file C:/Projects/github/microsoft/vcpkg/scripts/buildsystems/msbuild/applocal.ps1 -targetBinary "C:/Projects/trials/28.2 - Firestore test/firestoreTest/out/build/x64-debug/firestoreTest.exe" -installedDir C:/Projects/github/microsoft/vcpkg/installed/x64-windows/debug/bin -OutVariable out""
LINK Pass 1: command "C:\PROGRA~1\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.35.32215\bin\Hostx64\x64\link.exe /nologo CMakeFiles\firestoreTest.dir\firestoreTest.cpp.obj CMakeFiles\firestoreTest.dir\pythonTest.cpp.obj CMakeFiles\firestoreTest.dir\main.cpp.obj /out:firestoreTest.exe /implib:firestoreTest.lib /pdb:firestoreTest.pdb /version:0.0 /machine:x64 /debug /INCREMENTAL /subsystem:console C:\Projects\github\microsoft\vcpkg\installed\x64-windows\debug\lib\python310_d.lib kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib /MANIFEST /MANIFESTFILE:CMakeFiles\firestoreTest.dir/intermediate.manifest CMakeFiles\firestoreTest.dir/manifest.res" failed (exit code 1104) with the following output:
C:\Projects\trials\28.2 - Firestore test\firestoreTest\out\build\x64-Debug\LINK : fatal error LNK1104: cannot open file 'python310.lib'
ninja: build stopped: subcommand failed.
Build All failed.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Apparently pybind11 finds dependent libraries on it's own, insisting on using the release version of python310.lib/dll.  But cmake insists on a particular use of debug/release directories.  Thus when the project is building a debug version, only python310_d.lib/dll will be managed and found.
If I provide CMakeLists.txt the path to the python310.lib, it will link, but execution of the program will not find the python310.dll (because python310_d.dll has been copied to my executable directory).
I'm not savy enough with ports to fix this.
In my mind, pybind11 should not force the release version of python.    vcpkg install pybind11:x64-windows should work with a debug or release build.

I think you should manually define Py_DEBUG to force pybind11 to use debug build of python. See https://github.com/pybind/pybind11/pull/2025

Sounds like a good plan.   I changed my CMakeLists.txt as follows:
cmake_minimum_required (VERSION 3.16)
project (firestoreTest)

set (CMAKE_CXX_STANDARD 11)
set (CMAKE_CXX_STANDARD_REQUIRED True)
set (CMAKE_TOOLCHAIN_FILE c:/projects/github/microsoft/vcpkg/scripts/buildsystems/vcpkg.cmake)
set (Py_DEBUG ON)

find_package (Python3 COMPONENTS Interpreter Development REQUIRED)
find_package (pybind11 REQUIRED)

add_executable (firestoreTest
	firestoreTest.cpp firestoreTest.h
	pythonTest.cpp pythonTest.h
	main.cpp
)

target_link_libraries (firestoreTest
	PUBLIC Python3::Python
	PUBLIC pybind11::embed
)

Unfortunately I get the same link error:
C:\Projects\trials\28.2 - Firestore test\firestoreTest\out\build\x64-Debug\LINK : fatal error LNK1104: cannot open file 'python310.lib'
Have I not defined Py_DEBUG correctly?

Add it as define via target_compile_definitions, not as a cmake variable.

Also CMAKE_TOOLCHAIN_FILE should be set before project() call

That did it!  I can now compile with Visual Studio 2022 on Windows10 and Ubuntu 20.04 LTS.  To install I used:
Windows:   vcpkg install pybind11:x64-windows
Ubuntu:      vcpkg install pybind11:x64-linux
My final CMakeLists.txt:
'''
Top level CMakeList.txt
See:   https://pybind11.readthedocs.io/en/stable/cmake/index.html
Notes about compiling pybind11.
https://pybind11.readthedocs.io/en/stable/compiling.html
cmake_minimum_required (VERSION 3.16)
set (CMAKE_CXX_STANDARD 11)
set (CMAKE_CXX_STANDARD_REQUIRED True)
if (WIN32)
set (CMAKE_TOOLCHAIN_FILE c:/projects/github/microsoft/vcpkg/scripts/buildsystems/vcpkg.cmake)
endif()
project (firestoreTest)
find_package (Python3 COMPONENTS Interpreter Development REQUIRED)
find_package (pybind11 REQUIRED)
add_executable (firestoreTest
firestoreTest.cpp firestoreTest.h
pythonTest.cpp pythonTest.h
main.cpp
)
target_compile_definitions (firestoreTest
PRIVATE Py_DEBUG
)
target_include_directories (firestoreTest
PUBLIC ${PYTHON_INCLUDE_DIR}
PUBLIC ${pybind11_INCLUDE_DIR}
)
target_link_libraries (firestoreTest
PUBLIC Python3::Python
PUBLIC pybind11::embed
)
'''
Thank you very much for your help.  -Jim

@jimellert, Could you please provide the .cpp and .h files that reproduce your problem? I need to confirm locally if this issue is resolved.

Latest zip file posted.
Jim
From: JonLiu1993 @.>
Sent: Monday, May 8, 2023 12:58 AM
To: microsoft/vcpkg @.>
Cc: Jim Ellert @.>; Mention @.>
Subject: Re: [microsoft/vcpkg] [pybind11:x64-windows] debug link failure (Issue #30642)
@jimellert https://github.com/jimellert , Could you please provide the .cpp and .h files that reproduce your problem? I need to confirm locally if this issue is resolved.
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/vcpkg/issues/30642#issuecomment-1537918575 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOUYMAJIQZONP2DEVJG5Z3XFCRO3ANCNFSM6AAAAAAWRWXGHA .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AAOUYMEQAOQDEUW2FICP2JDXFCRO3A5CNFSM6AAAAAAWRWXGHCWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTS3VLDG6.gif Message ID: @.*** @.***> >

@jimellert, Thank you for the example you provided. I tried to reproduce your error locally yesterday, and submitted a PR to solve this bug. You can test whether your problem is solved locally. I tried it successfully locally.


toml::parse_file rises too many initializers for 'std::basic_ifstream' error

Environment
Compiler:  g++.exe (Rev2, Built by MSYS2 project) 9.2.0
C++ standard mode:  C++20
Target arch:  x64
Exceptions enabled:  Yes
Relevant toml++ configuration customizations:  No
Relevant compilation flags: Debug symbols are enabled
Using conan along cmake:
include(${CMAKE_BINARY_DIR}/conanbuildinfo.cmake)
conan_basic_setup()

And the conanfile:
[requires]
imgui/1.74
glfw/3.3.2@bincrafters/stable
glew/2.1.0@bincrafters/stable
glm/<IP_ADDRESS>
bullet3/2.89
spdlog/1.5.0
fmt/6.2.0

[generators]
cmake

Describe the bug
Using this:
void load(const char* file)
{
  auto toml::parse_file(file);
}

Causes:
/src/headers/toml.hpp: In instantiation of 'toml::parse_result toml::abi_parse_ex::parse_file(std::basic_string_view<Char>) [with Char = char; toml::parse_result = toml::table]':
/src/headers/toml.hpp:5953:20:   required from 'toml::parse_result toml::abi_parse_ex::parse_file(const Char*) [with Char = char; toml::parse_result = toml::table]'
/src/cpp/my-file.cpp, line numbers... required from here
src/headers/toml.hpp:5923:8: error: too many initializers for 'std::basic_ifstream<char>'
 5923 |   auto ifs = std::basic_ifstream<Char>{ str };
      |        ^~~

My bad! Didn't see this entry on the docs https://marzer.github.io/tomlplusplus/#mainpage-example-parsing-files
I wasn not including explicitly the fstream header, sorry.

@sigmasoldi3r No worries! You're not the first to miss that part of the docs. I should make it more obvious with a big "look at me" callout box or something.


Inline generic parameters in tooltips

It'd be much more readable if the generic parameters concrete types were just inlined into the signature itself, like C# editors do. I.e. the above signature should look like this:

Or maybe something like Flow<'t: int, 'u: int, 'mat: obj> would be even better.

I would imagine that we would keep the argument names though (in your sample screenshot they appear to have been removed)?

We can do this.

@isaacabraham yes, the latter screenshot is real :)
I do think that Flow<'t: int, 'u: int, 'mat: obj> is the best approach.
@Krzysztof-Cieslak how are you gonna do this (don't tell me by text replace :)?

By text replace

:)

@vasily-kirichenko What I meant was, I'd like to keep the argument names rather than just the types. But yes, showing concrete types is nice.

i 👎 this. a lot.
while is nice in c#, in f# ihmo is bad

type def can become really really long. just one nested generic and become really bad
lot of repetition
doesnt with the generic signature, if reusing same type for multiple generic args in that instance
layour is easier with current 't and 't: int below

This is probably an artifact of inference anyway, the flow function, if polymorphic, accepts arbitrary types, its probably clearer is they are not made rigid in the function definition.

I think I like current state of the tooltips, and this would have some disadvantages as mentioned by @enricosada


1 step forward, 2 steps back

Describe the bug
constantly making random changes and undoing all previous work. Impossible to make progress. No version control. It just burned through 8 million credits running me in circles for hours and at the end it pretty much erased half the app on a whim.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-qwfmmz
Steps to reproduce

step one: start a project
step two: try to actually make anything
step three: approach wall
step four: proceed to bang head against wall
step five: repeat

Expected behavior
a functional development experience
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 129
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 373400968,
  "usedJSHeapSize": 331200024,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = PARAD111GM
Chat ID = ab4c632122bc

Additional context
too exhausting

Apologies for the frustrating beta experience. Checkout the tips in #678 to avoid many of the issues you described!


Buffer test: Performance issue with VSBuffer#slice #76076 disabled in web

A follow up from https://github.com/microsoft/vscode/pull/114041, I had to disable one test that fails in browsers:
Performance issue with VSBuffer#slice #76076
https://github.com/microsoft/vscode/blob/aa5064d4fd47e3e0b6cd177f0a69cbf23c06d90e/src/vs/base/test/common/buffer.test.ts#L377-L377

The problem was that the test was using Buffer which was not defined in browsers. I've made that part of the test conditional.

The problem was that the test was using Buffer which was not defined in browsers. I've made that part of the test conditional.

Thanks!

Thanks!


Specify AWS Permissions if reading from S3

My sense from reading upstream issues on AWS credentials is that they're not interested in scraping AWS permissions from the various locations that exist (environment variables, config files, API endpoints) and are leaving this to downstream libraries.  (example issue: https://github.com/apache/arrow-rs/pull/4238)
We should maybe check to see if the location starts with s3:// and if so open up botocore and get whatever credentials we need from there and add them appropriately to storage_options and delta_storage_options

This works in arrow
import boto3
from pyarrow.fs import S3FileSystem

session = boto3.session.Session()
credentials = session.get_credentials()

fs = S3FileSystem(
             secret_key=credentials.secret_key,
             access_key=credentials.access_key,
             region="us-west-2",
             session_token=credentials.token,
         )

This seems to work in dask-deltalake
import dask_deltatable as ddt
import boto3

session = boto3.session.Session()
credentials = session.get_credentials()

df = ddt.read_deltalake(
    "s3://openscapes-scratch/etl-github/commit",
    delta_storage_options=dict(
             AWS_SECRET_ACCESS_KEY=credentials.secret_key,
             AWS_ACCESS_KEY_ID=credentials.access_key,
             AWS_REGION="us-west-2",
             AWS_SESSION_TOKEN=credentials.token,
    )
)


io test fixes

fixes a test warning, nothing was really broken

Also prevent accidental access to existing files by using tempfile


EOSJS - any error for a transaction returns same error msg/exception

I am using eosjs in my dAPP, where I am executing few transactions like issue and transfer. While doing so lets say I/user does something wrong while doing transfer

May be he inserts wrong/invalid symbol name,
or quantity precision is in-correct
or transfer quantity exceeds the total available balance
For any of the above action, eosjs returns me the same type of error message,
{"code":500,"message":"Internal Service Error","error":{"code":3050003,"name":"eosio_assert_message_exception","what":"eosio_assert_message assertion failure","details":[]}}

While at the same time, nodeos returns me the precise error message like,

SO do we have anything which will give me this kind more accurate error message like nodeos, so I can render it on page by which user could correct his mistake ?

I suspect it might be in here "details":[] if the API endpoint had started nodeosd with --verbose-http-errors .. I'm not sure what the trade-off is for enabling it and why it is off by default.
There is some example code parsing details in another issue, we can find it if you need it.

Ohh great, Thank you for your response, this suffice my need here, I am able to get the precise error msg now.
So thank you :)

I suspect it might be in here "details":[] if the API endpoint had started nodeosd with --verbose-http-errors .. I'm not sure what the trade-off is for enabling it and why it is off by default.
There is some example code parsing details in another issue, we can find it if you need it.

wow~ thank you again~


  As far as I can tell the script just isn't doing anything.

It was working but Youtube has updated code to somehow intentionally or unintentionally bypass the script and block player.
Script needs rework ..

You should run this script with adblocker turned on. Adblocker still able to block video player ads. And this script just blocks "Dont use Adblock" modal window, that is not an ad itself

In case you're using Youtube enhancement extensions, it might prevent the script from working properly. I was using Enhancer for YouTube and the script only worked intermittently until I turned the extension off.


Remove Linux android views

This test has deteriorated in stability: https://github.com/flutter/flutter/issues/99001
We plan on reworking the avd system, so until then, let's remove this test as it is not giving us a useful signal.

/cc @godofredoc Do we need to hold on such config changes in favor of the release?

hold on such config changes in favor of the release?
This is low risk change, let's land it.


docs: Sort all list guideline instruction

Closes #370
Summary of Changes
Sorted the exceptions in all list in src/safeds/exceptions/init.py and added an instruction to the guidelines.

Just on a side note: For updating the guidelines, this will already be addressed by another PR, see this comment, so it was not really necessary to change it.

:tada: This PR is included in version 0.15.0 :tada:
The release is available on:

v0.15.0
GitHub release

Your semantic-release bot :package::rocket:


Don't run python 2.7 virtualenv tests for newer versions of virtualenv

fixes #701
Newer virtualenv versions have dropped py2.7 support.

Looks like it's still sad about coverage :-(


This PR adds support for MAC addresses and MAC Pools management

This looks awesome!
When i search for a mac it doesn't seem like it limits the result to mac addresses entities?
For example something like:
➜  proteus_client git:(master) ✗ proteus mac search 0

Ah! - My bad. Good catch. Fixed to search only by MACADDRESS type

LGTM 👍


return null if unhandled url protocol

I don't think any additional checks are required, because the underlying methods will try to establish connection and throw a SQLException if it fails, and that is in accordance with the spec.

Proposed fix for #25

Looks good. Thanks


Wayland compile error

When I Compiled glfw with EGL & Wayland, the following issue was happened.
I find that "XK_KP_Space" is define in <X11/keysymdef.h>,and the definition in xkbcommon is "XKB_KEY_KP_Space ".
Is there something error In this process ？
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:830:5: error: ‘XK_KP_Space’ undeclared here (not in a function)
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:831:5: error: ‘XK_KP_Equal’ undeclared here (not in a function)
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:832:5: error: ‘XK_KP_Multiply’ undeclared here (not in a function)
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:833:5: error: ‘XK_KP_Add’ undeclared here (not in a function)
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:834:5: error: ‘XK_KP_Separator’ undeclared here (not in a function)
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:835:5: error: ‘XK_KP_Subtract’ undeclared here (not in a function)
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:836:5: error: ‘XK_KP_Decimal’ undeclared here (not in a function)
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:837:5: error: ‘XK_KP_Divide’ undeclared here (not in a function)
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:838:5: error: ‘XK_KP_0’ undeclared here (not in a function)
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:839:5: error: ‘XK_KP_1’ undeclared here (not in a function)
/media/work/user02/work/glfw-touch/src/xkb_unicode.c:840:5: error: ‘XK_KP_2’ undeclared here (not in a function)

What version of GLFW are you using?

The latest version：GLFW 3.1.2

By the way,glfw-touch branch also have the same error

But does master?

I mean the master branch version GLFW 3.1.2 and touch branch got the same error when I compiled.

This was fixed with 67c6a45e0e7fe09959fdbbdb0da7dfc6712d46e1 but I forgot to tag the commit.