Unable to use parseTemplate from @angular/compiler in a custom schematic

Command
other
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Hello,
I'm trying to write a custom migration schematic that updates component templates. To accomplish this, I intended to use the parseTemplate function from @angular/compiler, as it seems ideal for accessing and manipulating the AST.
However, I'm encountering runtime errors when attempting to use this function (see Minimal Reproduction).
Is this function suitable for my needs, or would you recommend a different approach?
Thank you!
Minimal Reproduction
Attempt 1: using "module": "commonjs" in tsconfig.json

Add @angular/compiler": "^18.0.0" in the schematic project
Add import { parseTemplate } from '@angular/compiler'; in the schematic
Run the schematic

Error:
An unhandled exception occurred: require() of ES Module<EMAIL_ADDRESS>not supported.
Instead change the require of<EMAIL_ADDRESS>to a dynamic import() which is available in all CommonJS modules.

Attempt 2: using "module": "ES2022" in tsconfig.json + "type": "module" in package.json
Same reproduction steps
Error:
An unhandled exception occurred: Cannot use import statement outside a module

Exception or Error
No response
Your Environment
Angular 18
Anything else relevant?
No response

Hi Xavier !
The framework packages are shipped as ESM but the schematics only support commonjs at the moment.
We have a related issue on this:

#22786

Hey :)
Thanks for the quick reply.
Do you happen to know any workaround or is it simply not possible at all for now?

There is a workaround mentioned in the above listed issue.


Feature request: configurable temperature (or default to 0)

This is related to issue #2. Letting our agents randomize their outputs makes developing and debugging agents very difficult. Is there a way to configure the temperature parameter in requests made to LLM models?

@stephenlf Oh, very good one! I'll work on it right now, push it to master, and it will be part of the next 0.1.1 release.
It's great that you related this to #2. In #2, the issue is that there’s no code block, which isn't easy to fix since the error message is correct (I'll add plain text later to make it clearer). But with this, it should be fine.

Note: For now, the default will be unset, but the user will be able to set the "default" they want in the .devai/config.toml.

@stephenlf implemented in main. Will be part of 0.1.1 release.


Restore-SFPartition requires Connect-SFCluster while the recommended approach is to use Connect-ServiceFabricCluster instead

Restore-SFPartition requires Connect-SFCluster while the recommended approach is to use Connect-ServiceFabricCluster instead.
However, you can't use Restore-SFPartition from the powershell module that Connect-ServiceFabricCluster is part of.
Is using the API currently the only recommended option?

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

ID: 59a07a58-8603-9a0b-4338-bf64c5740d7e
Version Independent ID: fa1afa3f-7677-8af2-f1ed-d163740b71fe
Content: Restoring backup in Azure Service Fabric - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-backup-restore-service-trigger-restore.md
Service: service-fabric
GitHub Login: @aagup
Microsoft Alias: aagup

@ErikDeRoos
Thanks for your feedback! We will investigate and update as appropriate.

@ErikDeRoos Apologies in delayed response, I checked with internal team on this.
If you wish to use Restore-SFPartition cmdlet provided by Microsoft.ServiceFabric.Powershell.Http, you need to connect to the cluster using the Connect-SFCluster cmdlet provided by the module.
The other option would be to make the restAPI calls from powershell by providing the json body which would require you to connect to the cluster using Connect-ServiceFabricCluster.
Hope this information helps.

Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


[GH-645] update text to mention groups only work for legacy instance

Summary
Update message to inform users that only instances installed v2.4 or earlier apply to Jira Groups
Ticket Link
#645

Codecov Report

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

@@           Coverage Diff           @@
##           master     #655   +/-   ##
=======================================
  Coverage   34.48%   34.48%
=======================================
  Files          51       51
  Lines        5686     5686
=======================================
  Hits         1961     1961
  Misses       3532     3532
  Partials      193      193

Continue to review full report at Codecov.

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


mu: 1.10.6 -> 1.10.7

Description of changes
https://github.com/djcb/mu/releases/tag/v1.10.7

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

2 packages built:

lbdb
mu


Fix typo

@yuta0801 since you're the one who wrote that line, can you verify if it's correct, please?

This fix is not completely wrong, but it is an ambiguous sentence.
If translate it directly to English, it is  Displays the invite link of the bot..
IMO: Originally it was grammatically a bit unnatural so it may be good to fix to ボットの公式サーバーへの招待リンクを表示します。
(directly translation: Displays the invite link to an official server of the bot.)

@yuta0801 yeah invite is to invite the bot to a server so the 2nd one would be incorrect

Umm... I found join guild link in the original text
			COMMAND_INVITE_DESCRIPTION: 'Displays the join guild link of the bot.',

https://github.com/dirigeants/klasa/blob/master/src/languages/en-US.js#L118

So, is @InkoHX's description correct? Just making sure.

At least I feel like the bot's own invite (OAuth) URL, rather than the server (guild) invite URL in that sentence.

We updated the en-US file to clarify the line as of https://github.com/dirigeants/klasa/pull/693, thanks you for the suggestion.

Oh, okay. He's correct from the beginning.
I got hung up on the original "join guild link" and I did not check the command... sry
This might as well to follow original.

Just double-checking before merging, this is correct now, right?

Yes, this is correct.


Initial version of Grpc.Tools & Msbuild integration tests

Work in progress
Initial ideas for unit testing Grpc.Tools MSBuild files.
Uses a mock protoc compiler to check the compiler is being called

@jtattermusch Changes ready for review. However basic C# tests are failing in the CI environment. This is because the DLL containing the custom tasks from the Grpc.Tools project is not available (src\csharp\Grpc.Tools\bin\Debug\netstandard1.3\Protobuf.MSBuild.dll) - I was assuming the dependent project files would be available.
I'll have to think about how to get this DLL.  Not sure I want to put the binary in git. Or we can mark the tests as [Explicit] so only run by developers?

@jtattermusch I've now added the Protobuf.MSBuild.dll to git and the tests are passing in the CI on Windows, Linux and MacOS.
Ready for review.

@tonydnewell  I pushed my incremental changes from https://github.com/grpc/grpc/pull/31964 to this PR.  I think right now this PR is decent shape (close to something that can be merged). Can you please look at the changes I've made (see commits authored by me and review). If you're happy with my edits, I'll do a final review pass and then we can merge.

Note to self: this is ready to merge once the copybara import test is green.

Python distribtest failure is unrelated.


Updated authentication.md

Multiple places referenced certs being signed by ca.crt which is not
correct. Updated to reference ca.key instead.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a5f6c3285c7649f3cd9f10bc9223a4debff5c38c/

CC: @Amruta-Ranade
Thanks for the PR. I agree with you that signed by <private key> is more accurate. However, the aim of spelling out the certificate filename is to make it very clear which CA is responsible for which certificate. This becomes especially important in the split CA certificates.
We also care less about the CA key filename (the full path to the key is specified in the cockroach tooling) as opposed to the certificate filename (which is fixed).
An alternative phrasing would be: "... must be signed by the CA represented by ca.crt". While pedantic, it is accurate and keeps the emphasis on the specific CA identified by filename. The same would go for client.node.crt which was left unmodified.

Thanks for the PR! Will look into this and get back to you.

@Amruta-Ranade, any updates here?

@Amruta-Ranade or @lhirata, can one of you take this?

@Amruta-Ranade, ping.

Ah yes. Thanks for the reminder, Jesse.

Addressed in #6313


ECS version tags

Could you adjust the ECS version names a little bit? There's two things I see that could be confusing about them, in the readme.

ECS always explicitly includes all 3 numbers in the version number, including .0 patch. So instructing people to construct the URL with the "1.2" tag will give them a 404. They'll figure it out, but it's an annoyance we can avoid :-)
Also the actual tag in Git includes a leading v.

So the full 1.2 tag actually looks like v1.2.0 :-)
Loving the readme otherwise, good work!

Thanks @webmat - I have corrected the README and fixed the version number references.


Feature-request: Lookup transactions by id as contract effects.

Area
[] Plutus Foundation Related to the GHC plugin, Haskell-to-Plutus compiler, on-chain code
[x] Plutus Application Framework Related to the Plutus application backend (PAB), emulator, Plutus libraries
[] Marlowe Related to Marlowe
[] Other Any other topic (Playgrounds, etc.)
Describe the feature you'd like
I'd like to be able to run an extra contract effect in off-chain code to lookup any arbitrary transaction from its id. The framework already provides utxoAt for looking up funds in the UTxO, but I'd like to query historical data of some past transaction. More precisely (although it doesn't make the problem any simpler), I am really interested in the inputs of a transaction that originated a particular UTxO I have a reference to.
Describe alternatives you've considered
To cope with this so far, I've been keeping track of the relevant past transaction outside of the contract code, and passing them to the contract endpoint as inputs. Yet, this is very much an abstraction leakage from my contract's perspective; I'd prefer clients not to have to do this extra state management.
Additional context / screenshots
N/A

Asking for arbitrary transactions is a bit too powerful, because we don't want to keep the entire blockchain in our database. If you need that you have to run your db outside of the PAB.
We are going to overhaul the chain index however, to keep more information for TxOutRefs that are in the utxo set. So if your UTXO is still unspent you could get the tx that produced it from the chain index.

So if your UTXO is still unspent you could get the tx that produced it from the chain index.

That'd be sufficient for my use-case :+1: , thanks.


Change "Settings" to "More"

Summary

[ ] Change "Settings" Text in NavBar to "More"
[ ] Change icon to a plus button

Motivation
Since clicking this tab opens a popup with "Settings" as one of the options along with "About" and "Logout," this is really not a "Settings" tab but a "More" tab and should be described as such.
Describe alternatives you've considered
No

I'm on this!


Make project sd-jwt-vc compliant

This PR makes the library compliant to sd-jwt-vc since sd-jwt only defines the protocol of the selective disclosure approach.
Since the sd-jwt-vc is the only spec that seems to be stable engouth to be used with sd-jwt, I changed the core values.
First thoughts to split it up were aborted since the integration of a W3C-VC-DM is not standardized and could lead to massive interop problems.
Open questions before accepting this:
Required types
The sd-jwt-vc required the following values to be specified in a credential

iss: reference to the issuer.
iat: issuance data
vct: reference to the schema of the credential

I decided to pass them together with the claims in the issue function like:
const credential = await sdjwt.issue(
    {
      iss: 'Issuer',
      iat: new Date().getTime(),
      vct: 'https://example.com',
      ...claims,
    },
    disclosureFrame,
  );

In case the issuer is a did, the iss has to be the url referencing the key like did:web:exmaple.com#key1.
Issuance date should not be automatically calculated by the SDK since an issuer could have an intention to set it to another value (I don't see a reason for this, but doing so we are not limiting him).
I would also suggest to pass the issuer value during the issuance process and not via the init of the SDJWT Instance. The kid field will also not be updated intern, the user is able to pass it in the third parameter when issuing.
VCT should define the type, see https://www.ietf.org/archive/id/draft-ietf-oauth-sd-jwt-vc-01.html#name-verifiable-credential-type- This seems to be required, but we are not able to set this by the SDK.
Renaming
I did not rename classes like SDJwtInstance since they do not include any logic that is specific for the SD-JWT-VC usage, we can rename this, but it would be a breaking change
Signed-off-by: Mirko Mollik<EMAIL_ADDRESS>

@cre8 Looking at the code, I thought about it some more.
I think I like our first approach better which is separate sd-jwt and sd-jwt-vc.
The sd-jwt-vc standard is still at version 1, and more uses of sd-jwt may come out in the future.
I think it's too early to make a decision to merge. Approaching cautiously, expanding sd-jwt-vc class from sd-jwt would be a safe job.

@cre8 Looking at the code, I thought about it some more. I think I like our first approach better which is separate sd-jwt and sd-jwt-vc. The sd-jwt-vc standard is still at version 1, and more uses of sd-jwt may come out in the future. I think it's too early to make a decision to merge. Approaching cautiously, expanding sd-jwt-vc class from sd-jwt would be a safe job.

I first thought the same,
but since we do not know how other approaches will look like, it's difficult to build a generalised layer. So I would suggest to just go with this since it's just a small adjustment and later we can optimize it

I would also strongly recommend to make the sd-jwt-vc a separate package. sd-jwt by itself, without the vc restrictions, is still very useful and not every use case requires the
additional restrictions. Extracting it into separate should not be that much effort and it would greatly improve the usability of this library. And also, easier to extend upon in the future with different formats.

@berendsliedrecht good point, I add the other package

@lukasjhan @berendsliedrecht added a new package
New:

Implementations need to pass the head type like sd-jwt-vc
Implement the "always disclosed" function

I copied the e2e test from sd-jwt and modified the input so it's compliant.

Is there a way to just use it as a class instead of making sdjwtInstance an abstract class?

Is there a way to just use it as a class instead of making sdjwtInstance an abstract class?

Like described above, sd-jwt:
"This specification defines a mechanism for selective disclosure of individual elements of a JSON object used as the payload of a JSON Web Signature (JWS) structure. It encompasses various applications, including but not limited to the selective disclosure of JSON Web Token (JWT) claims." https://www.ietf.org/archive/id/draft-ietf-oauth-selective-disclosure-jwt-07.html
So this spec defines the mechanism
This spec for the sd-jwt-vc defines the usage of the sd-jwt inside a verifiable credential: https://www.ietf.org/archive/id/draft-ietf-oauth-sd-jwt-vc-01.html
So to be compliant to the spec you can not use only sd-jwt as the credential format, you need an implementation of the class to specify the header type

@cre8 There seems to be some debate about not being able to use sd-jwt raw. I think the rest is okay.
If I were to decide on my own, I think I would approve this PR. :)
@berendsliedrecht Can you tell me what you think?
btw you need to fix the examples/core-example. They use raw sdjwtInstance

@cre8 The big change is not being able to use sd-jwt raw. I think the rest is okay. If I were to decide on my own, I think I would approve this PR. :)
@berendsliedrecht Can you tell me what you think?
btw you need to fix the examples/core-example. They use raw sdjwtInstance

I wouldn't call it "big change" or "breaking change" since we moved back to version 0.x that allows to be unstable. Also we are getting compliant to the spec we were not in the first place.
I moved the examples from core to sd-jwt-vc and made a node in the packages about this.

@cre8 Good. Examples are all good :)

So to be compliant to the spec you can not use only sd-jwt as the credential format, you need an implementation of the class to specify the header type

It is fully valid to use an sd-jwt without any context of sd-jwt-vc maybe I am missing something?

So to be compliant to the spec you can not use only sd-jwt as the credential format, you need an implementation of the class to specify the header type

It is fully valid to use an sd-jwt without any context of sd-jwt-vc maybe I am missing something?

The spec for sd-jwt has no mentioning to set the typ field to sd-jwt since this is out of scope of the specification. That's why the spec for sd-jwt-vc was written because it is dealing with a specific implementation of the usage of the sd-jwt algorithms.
To be compliant with the jwt spec, the typ value has to be one of the IANA Media Types. It will include sd-jwt-vc once the spec is finialised, but it will not include sd-jwt since this is not a credential format.

So to be compliant to the spec you can not use only sd-jwt as the credential format, you need an implementation of the class to specify the header type

It is fully valid to use an sd-jwt without any context of sd-jwt-vc maybe I am missing something?

The spec for sd-jwt has no mentioning to set the typ field to sd-jwt since this is out of scope of the specification. That's why the spec for sd-jwt-vc was written because it is dealing with a specific implementation of the usage of the sd-jwt algorithms.
To be compliant with the jwt spec, the typ value has to be one of the IANA Media Types. It will include sd-jwt-vc once the spec is finalised, but it will not include sd-jwt since this is not a credential format.
I don't feel good to allow people to give out sd-jwt as typ because it's a non standardised approach and will lead to interoperability issues.

I understand your point, however the typ claim is optional. Meaning an sd-jwt can be fully valid without having a typ claim.
Also, the sd-jwt-vc spec was not only written to define the typ field and has more uses.

So to be compliant to the spec you can not use only sd-jwt as the credential format, you need an implementation of the class to specify the header type

It is fully valid to use an sd-jwt without any context of sd-jwt-vc maybe I am missing something?

The spec for sd-jwt has no mentioning to set the typ field to sd-jwt since this is out of scope of the specification. That's why the spec for sd-jwt-vc was written because it is dealing with a specific implementation of the usage of the sd-jwt algorithms.
To be compliant with the jwt spec, the typ value has to be one of the IANA Media Types. It will include sd-jwt-vc once the spec is finalised, but it will not include sd-jwt since this is not a credential format.
I don't feel good to allow people to give out sd-jwt as typ because it's a non standardised approach and will lead to interoperability issues.

I understand your point, however the typ claim is optional. Meaning an sd-jwt can be fully valid without having a typ claim.
Also, the sd-jwt-vc spec was not only written to define the typ field and has more uses.

It depends on the context: like I said: sd-jwt is only the algo, so it has no impact on the type. However sd-jwt-vc forces it.
There is no such sd-jwt as credential, it's only the algorithm for selective disclosure. All the credential format part is defined in and only in the sd-jwt-vc.
So again: sd-jwt should NOT be a class that can be initialized on its own.

It depends on the context: like I said: sd-jwt is only the algo, so it has no impact on the type. However sd-jwt-vc forces it.
There is no such sd-jwt as credential, it's only the algorithm for selective disclosure. All the credential format part is defined in and only in the sd-jwt-vc.

I don't think we will reach any consensus here, but sd-jwt is completely usable on its own. Not as a credential format, but as a replacement / addition to normal JWTs. sd-jwt-vc aims to bridge the specific requirements for a credential, however that does not mean that sd-jwt cannot be used in a context outside of credentials. I really feel like we massively limit the usage of this library as sd-jwt will likely, and already is, way larger than sd-jwt-vc and it will be used in contexts outside of digital idenity, ssi, VCs, etc.
If it says anywhere in the specification of sd-jwt that it should not be used directly and only extensions on top of if that define extra requirements can be used, I am perfectly fine with this approach. However I did not see any reference to this and every other sd-jwt implementation also does not take this approach.

FYI: SD JWT VC standard version 2 just released:
https://www.ietf.org/archive/id/draft-ietf-oauth-sd-jwt-vc-02.html#section-1.2
The contents of link describe SD-JWT as a Credential Format.

It depends on the context: like I said: sd-jwt is only the algo, so it has no impact on the type. However sd-jwt-vc forces it.
There is no such sd-jwt as credential, it's only the algorithm for selective disclosure. All the credential format part is defined in and only in the sd-jwt-vc.

I don't think we will reach any consensus here, but sd-jwt is completely usable on its own. Not as a credential format, but as a replacement / addition to normal JWTs. sd-jwt-vc aims to bridge the specific requirements for a credential, however that does not mean that sd-jwt cannot be used in a context outside of credentials. I really feel like we massively limit the usage of this library as sd-jwt will likely, and already is, way larger than sd-jwt-vc and it will be used in contexts outside of digital idenity, ssi, VCs, etc.
If it says anywhere in the specification of sd-jwt that it should not be used directly and only extensions on top of if that define extra requirements can be used, I am perfectly fine with this approach. However I did not see any reference to this and every other sd-jwt implementation also does not take this approach.

But isn't each JWT is a credential since it includes signed claims? It does not include the "proof of ownership" by binding a holders public key. SD-JWT-VC is not requiring this.
My biggest concern making the SdJWTInstance not abstract is that one side will use this approach, and the other SdJWTVCInstance, thinking both are doing the same thing.
I totally agree that the SD-Algorithm can be used for anything else, but then we need to split it up from the JWT part and then we can use it for non credentials.

@berendsliedrecht so we have two options:

Making SDJWTInstance not abstract and SDJWTVCInstance will just set the typ and validation function
staying abstract and mention in the readme how to use the native sdjwtInstance class

What way do you prefer, I am fine with both

If we allow people to use pure sd-jwt, then I think

Making SDJWTInstance not abstract and SDJWTVCInstance will just set the typ and validation function

is better :)

@berendsliedrecht so we have two options:

Making SDJWTInstance not abstract and SDJWTVCInstance will just set the typ and validation function

staying abstract and mention in the readme how to use the native sdjwtInstance class

What way do you prefer, I am fine with both

Id have to stay with option one. I hope my Saturday will stay free and I can give a whole lot more into the codebase and get a bit of a better sense of these exact complications.
People should be able to create an sd-jwt without creating a class for it first and as long as that is possible, I am happy with it :).


SFC syntax highlighting issue

SFC lost some syntax highlighting after I install and enable the  vue extension: https://marketplace.visualstudio.com/items?itemName=znck.vue
Notably the <template>, <script>, v-show, etc

Blocks <script>, <template> and <style> are intentionally un-highlighted.
Will fix v-show.

Blocks <script>, <template> and <style> are intentionally un-highlighted.
Will fix v-show.

Blocks <script>, <template> and <style> are intentionally un-highlighted.

Why?

Blocks <script>, <template> and <style> are intentionally un-highlighted.

Why?


Create edit/delete categories functionality

Ready for PR https://github.com/on-site/StockAid/pull/102


impl Clone for wasmtime::component::Linker

The core Wasm Linker type is clonable, and the component linker needs to be used in similar settings where clonability is important.

D'oh, I immediately realized why there's a custom impl for the other Linker...

Thanks!


Show iOS controls without using present full screen

This allows you to show the controls without calling the present full screen method.

Can this get merged in please?

Maybe I'm missing something, but the only time I'm seeing createPlayerViewController get called is when I use presentFullscreenPlayer. Can you provide an example of how you're using this? It may also be that something has changed when they switched over to the new video player in iOS 11.
Also, I don't think we want to make this mandatory behavior, some people are going to want to provide their own controls UI, so this should be an optional prop.


Cookie not set with SameSite attribute

This issue is visible for all users on https://generator.inrupt.com. Once logged in, a console warning is generated, which reads:
A cookie associated with a cross-site resource at https://inrupt.net/ was set without the `SameSite` attribute. A future release of Chrome will only deliver cookies with cross-site requests if they are set with `SameSite=None` and `Secure`. You can review cookies in developer tools under Application>Storage>Cookies and see more details at https://www.chromestatus.com/feature/5088147346030592 and https://www.chromestatus.com/feature/5633521622188032.

The only cookie on the generator.inrupt.com site is for inrupt.net, which is triggering the warning.
The links in the warning (https://www.chromestatus.com/feature/5088147346030592 etc) both go into more details on the issue as well, including linking to a timeline.

I think this might be an NSS issue because cookies are set by the server.

This issue also appears using solid.community and this Chrome feature is now enabled on Android (breaking all Solid apps using the nssidp.sid cookie), but can be disabled through chrome://flags/#same-site-by-default-cookies.
It appears the change required in NSS is not too difficult but this could also be considered a solid-auth-client issue - if cookie-based authentication fails (e.g. because the client rejects third party cookies for any other reason), ideally it should be possible to fall back to another mechanism?
The NSS issue about rethinking authentication configuration also seems relevant https://github.com/solid/node-solid-server/issues/672

If I understand correctly, this authentication method no longer works on new servers anyway (https://github.com/solid/solid-auth-client/pull/173), so issues with SameSite in this library are no longer relevant and this issue can be closed as such?

If I understand correctly, this authentication method no longer works on new servers anyway (https://github.com/solid/solid-auth-client/pull/173), so issues with SameSite in this library are no longer relevant and this issue can be closed as such?


Sort the issues by severity in descending order

Sort the issues by severity in descending order before creating the report. This will print the most critical issues at the top of the report.
This is useful when doing security audit in large code bases with hundred of warnings.
cc @gcmurphy

sort.Slice was introduced in Go 1.8, the travis job will try to build against 1.5 and latest. I'm not sure if we need to support versions as far back as 1.5..but I think it's reasonable to support N-2 which would include 1.7.

I switched to old style sort interface. In addition, I update the build file to validate the tool from go version 1.7 onward.


Improvement in Rules Page

as you can see that here the interference of the rules page is looking so normal can you allow me to make rules pages more attractive
it would be really great if you allow me, it make changes in rules page

I want to work this project. please assign


Error while setting up CLI with GPG smartcard (yubikey)

I just tried to install camput with my gpg key which is on a yubikey. But camput --init throws an error.
> camput -secret-keyring="/home/niklas/.gnupg/secring.gpg" init --gpgkey XXX
Error: Could not find keyId XXX in ring /home/niklas/.gnupg/secring.gpg: openpgp.ReadKeyRing of "/home/niklas/.gnupg/secring.gpg": openpgp: unsupported feature: hash for S2K function: 0

Looks like the Go opengpg library does not support these stubs from GPG smartcards.
Is there any method to get that working anyway?
Thanks

Ok I dug through the Golang issue tracker. If this CL:https://go-review.googlesource.com/c/crypto/+/32797 will be merged this should work.


Feature: Epub file support added

changelog

Support for Epub files added
Code refactored

Thanks for pointing out the issues. It's commit mistake


Add a missing parenthesis

This is a minor markdown issue with a URL. In the Supported Dynamics 365 payment features, notice that the URL is displayed correctly.
Actual:
[External gift cards for call center and e-commerce](https://docs.microsoft.com/dynamics365/commerce/dev-itpro/gift-card
Notice the closing parentheses is missing, which causes the URL to display poorly.
Proposed:
External gift cards for call center and e-commerce

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.

Follow the new one at https://github.com/MicrosoftDocs/Dynamics-365-Operations/pull/9896


Added ability to delete ephemeral messages

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[x] Internal code
[ ] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
Added ability to delete ephemeral messages

Alright, tested this and it seems to be working.

did you update the documentation for deferReply(true)

Just pushed a new commit that should fix that

did you update the documentation for deferReply(true)

Just pushed a new commit that should fix that

might wanna put the code for the method to delete it


Feature: Allow bypass of AppPool identity configuration

In the ConfigureAppPool MSBuild task, add a condition for empty $(VDIR_UserPass) to skip execution of the task.  This will allow someone with an existing AppPool to skip the identity configuration.
This work item was migrated from CodePlex
CodePlex work item ID: '10067'
Assigned to: 'tfabraham'
Vote count: '1'

[UnknownUser@5/4/2013]
Resolved with changeset 72724: Add support for IIS 8 and pre-configured AppPools; drop legacy VDirList.txt; refactor DeployVDirs target (#10067, 10082)

[UnknownUser@5/16/2013]


Error on docker cache

Hi.
Just try to compile example of docker cache, and pay attention to that line:
tags: |         '$(tag)'
I am tired with errors in every article and labs for azure devops.

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

ID: 8dd39237-2778-c4f1-3ce4-bde343ecb59d
Version Independent ID: e3daf460-128d-cb57-e899-ca0c841dff07
Content: Pipeline caching - Azure Pipelines
Content Source: docs/pipelines/release/caching.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-release
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@titans2016 -- Maxim, thank you for your feedback.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@ramiMSFT -- Rami, please look into this issue.

@WilliamAntonRohm thank for links, but I am top expert in azure devops in europe. I am also a teacher and can resolve easy this issue by my self. just point that there are a lot of mistakes in documentations.

@titans2016 -- Maxim, thank you for your follow-up!


[Python] Reading parquet file with map without annotated map keys

Describe the bug, including details regarding any error messages, version, and platform.
Hello,
While reading a generated parquet file from AWS Athena (Trino engine behind) we have the following error:
  File "pypoetry/virtualenvs/lib/python3.10/site-packages/pyarrow/parquet/core.py", line 308, in __init__
    self.reader.open(
  File "pyarrow/_parquet.pyx", line 1239, in pyarrow._parquet.ParquetReader.open
  File "pyarrow/error.pxi", line 100, in pyarrow.lib.check_status
pyarrow.lib.ArrowInvalid: Map keys must be annotated as required.

(parquet-tools give the same error if you want to reproduce)
We don't have this error will reading it with spark. Is it Spark that's being lax with parquet spec? Or an issue with Arrow?
Athena Query if you want to reproduce
UNLOAD (
SELECT ARRAY [
          MAP(ARRAY['key_1', 'key_2'], ARRAY['value_1', 'value_2']),
          MAP(ARRAY['key_1', 'key_2'], ARRAY['value_2', 'value_2']),
          MAP(ARRAY['key_1', 'key_2'], ARRAY['value_3', 'value_2'])
      ]  AS array_map
)
TO 'path'
WITH (format = 'PARQUET')

If you want directly the file:
https://file.io/Bs6BtISt3GFb
Component(s)
Parquet, Python

Would you mind print the schema using Python SDK? Personally I think it's a schema problem. Spark cannot produce "nonnull" field, but maybe we expect the map-key with nonnull.

Sure here the schema produce by pyspark:
>>> df.printSchema()
root
 |-- array_map: array (nullable = true)
 |    |-- element: map (containsNull = true)
 |    |    |-- key: string
 |    |    |-- value: string (valueContainsNull = true)

I assume you meant pyspark by Python SDK, but let me know if I'm wrong, and I'll be more than pleased to help you!

Emmm I found that:

In standard, the map key should be a "required" field: https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#maps
The schema in your file is

required group field_id=-1 hive_schema {
  optional group field_id=-1 array_map (List) {
    repeated group field_id=-1 bag {
      optional group field_id=-1 array_element (Map) {
        repeated group field_id=-1 key_value (Map) {
          optional binary field_id=-1 key (String);
          optional binary field_id=-1 value (String);
        }
      }
    }
  }
}

SchemaNode: required group field_id=-1 hive_schema {
  optional group field_id=-1 array_map (List) {
    repeated group field_id=-1 bag {
      optional group field_id=-1 array_element (Map) {
        repeated group field_id=-1 key_value (Map) {
          optional binary field_id=-1 key (String);
          optional binary field_id=-1 value (String);
        }
      }
    }
  }
}

Which the key is optional. cc @wgtmac do we need to support file like this?

IMO, this is something that the writer should fix. optional map key is ill-formed as per the spec.

@wgtmac is there a downside of supporting this as we are encountering such files in the wild and can't influence the writer in any way?

Maybe we can find the version of PySpark and found how other user solve this problem, since PySpark is so widely used?

@wgtmac we are having this problem with this dataset:
https://github.com/OvertureMaps/data/tree/main
I'll open an issue there.

cc @theosib-amazon Not sure if you are still working at AWS Athena.

@wgtmac my argument for the change is -
the data is precious and it's source might be never available again. The code is secondary. Therefore the code must make every effort to read the data that is imperfect. Otherwise it is like refusing to read a good book because the table of contents has a typo :)

Depending on where the error is thrown it might be possible to provide an override arrow schema that uses a list of key-value pairs, it seems like this approach or delaying error until null might be best
While I agree data is precious maintaining loop holes for technically invalid files has a cost as well.

If already have some solving methods we can use that, otherwise I think whether we can regard that as a List<Struct<Key, Value>> and Cast to a Map?
Since maintaining read this from parquet is really dirty.

Technically this can be fixed by postponing the exception until a real null map key data has been caught.

That sounds like quite a bit of additional maintenance. Also, I'm not sure delaying the error until actual nulls are read is really user-friendly...

I think whether we can regard that as a List<Struct<Key, Value>> and Cast to a Map

This has the same problem of additional maintenance and delaying the error until much later.
This might not be an immediate solution, but perhaps a third-party utility could be created to "fix" non-conforming Parquet files by rewriting them? It could address a growing number of non-conformity issues while not forcing mainline implementations to reimplement compatibility hacks.

Yeah, I know. I was suggesting that perhaps someone would want to work on such a project :-)

@wgtmac did not find a way to manipulate the column metadata with Spark so far, do you know one?

@wgtmac
aws s3 cp --no-sign-request s3://overturemaps-us-west-2/release/2023-07-26-alpha.0/theme=transportation/type=connector/20230726_134827_00007_dg6b6_01b086fc-f35b-487c-8d4e-5cdbbdc1785d test.parquet

@negrusti Sorry for the late reply. I have tried to use Apache Spark to rewrite the file and it seems worked.
package a.b.c

import org.apache.spark.sql.SparkSession

object ParquetTest {

  def main(args: Array[String]) {
    val spark = SparkSession.builder.master("local[1]").appName("Parquet Test").getOrCreate()
    spark.read.parquet("/tmp/test.parquet").repartition(1).write.parquet("/tmp/rewrite")
  }

}

The original schema using parquet-cli from parquet-mr
Created by: parquet-mr version 1.12.2-amzn-athena-1 (build 6c6353027ef5d7782e8657ea59b290452cdfdaee)
Properties:
  writer.time.zone: UTC
Schema:
message hive_schema {
  optional binary id (STRING);
  optional int96 updatetime;
  optional int32 version;
  optional int32 level;
  optional binary subtype (STRING);
  optional group connectors (LIST) {
    repeated group bag {
      optional binary array_element (STRING);
    }
  }
  optional binary road (STRING);
  optional group sources (LIST) {
    repeated group bag {
      optional group array_element (MAP) {
        repeated group key_value (MAP_KEY_VALUE) {
          optional binary key (STRING);
          optional binary value (STRING);
        }
      }
    }
  }
  optional group bbox {
    optional double minx;
    optional double maxx;
    optional double miny;
    optional double maxy;
  }
  optional binary geometry;
}

The schema in the rewritten file is
Created by: parquet-mr version 1.12.2 (build 77e30c8093386ec52c3cfa6c34b7ef3321322c94)
Properties:
                   org.apache.spark.version: 3.3.2
  org.apache.spark.sql.parquet.row.metadata: {"type":"struct","fields":[{"name":"id","type":"string","nullable":true,"metadata":{}},{"name":"updatetime","type":"timestamp","nullable":true,"metadata":{}},{"name":"version","type":"integer","nullable":true,"metadata":{}},{"name":"level","type":"integer","nullable":true,"metadata":{}},{"name":"subtype","type":"string","nullable":true,"metadata":{}},{"name":"connectors","type":{"type":"array","elementType":"string","containsNull":true},"nullable":true,"metadata":{}},{"name":"road","type":"string","nullable":true,"metadata":{}},{"name":"sources","type":{"type":"array","elementType":{"type":"map","keyType":"string","valueType":"string","valueContainsNull":true},"containsNull":true},"nullable":true,"metadata":{}},{"name":"bbox","type":{"type":"struct","fields":[{"name":"minx","type":"double","nullable":true,"metadata":{}},{"name":"maxx","type":"double","nullable":true,"metadata":{}},{"name":"miny","type":"double","nullable":true,"metadata":{}},{"name":"maxy","type":"double","nullable":true,"metadata":{}}]},"nullable":true,"metadata":{}},{"name":"geometry","type":"binary","nullable":true,"metadata":{}}]}
Schema:
message spark_schema {
  optional binary id (STRING);
  optional int96 updatetime;
  optional int32 version;
  optional int32 level;
  optional binary subtype (STRING);
  optional group connectors (LIST) {
    repeated group list {
      optional binary element (STRING);
    }
  }
  optional binary road (STRING);
  optional group sources (LIST) {
    repeated group list {
      optional group element (MAP) {
        repeated group key_value {
          required binary key (STRING);
          optional binary value (STRING);
        }
      }
    }
  }
  optional group bbox {
    optional double minx;
    optional double maxx;
    optional double miny;
    optional double maxy;
  }
  optional binary geometry;
}

If this works for you, feel free to close this issue or let me know if anything wrong happens. @negrusti @Max0u


Side-by-side comparison disabled

[Enter feedback here]
Fix your CSS! A "side-by-side" comparison is impossible without two 24" monitors, because whenever you increase the width of the browser window, the column that gets the most expansion is the RHS sidebar.
Either add column minimiser buttons, or restrict expansion of the sidebar to a max pixel count (not %age)

See attached:  Why is there a horizontal scroll bar for the main (middle) content, and three miles of whitespace to the right of the column with: 'Branches', 'Conflict Resolution', 'Next Steps', 'See Also'

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

ID: aa1e2fe5-6a2b-f507-5b04-ba9dbd6783d5
Version Independent ID: dfce8efa-e1b7-89b3-6d91-5d2865c1a7da
Content: Side-by-side comparison of Git and Team Explorer in Visual Studio 2019
Content Source: docs/version-control/git-team-explorer-feature-comparison.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @Taysser-Gherfal
Microsoft Alias: tglee

@mcalexster -- thank for your feedback; we appreciate it. You can submit issues related to overall site experience, or suggestions for improved site functionality or design, at https://aka.ms/sitefeedback/.


[sitemap plugin] lastmod is ctime

Version
No response
Platform
No response
What steps will reproduce the bug?
import sitemap from "lume/plugins/sitemap.ts";

site.use(sitemap());

How often does it reproduce? Is there a required condition?
100%
What is the expected behavior?
it should be mtime
What do you see instead?
inside sitemap.xml:
<lastmod>(ctime here)</lastmod>
Additional information
data.date is ctime i think

It makes sense.
date value is used to assign a date to a page, but and this date can mean anything. By default Lume uses the creation date, but technically it could be what do you want (you can assign the date you want in the frontmatter).
I this case, I agree that mtime makes more sense, but this value is not available automatically in the page date. Maybe Lume should have both dates, but I'm not completely sure about that.
As a workaround, you can create a preprocessor to obtain this data:
site.preprocess([".html"], (pages) => {
  for (const page of pages) {
    const info = page.src.entry?.getInfo();
    page.data.lastmod = info?.mtime;
  }
});

wouldn't what your proposed be the default for a sitemap? i assume only search engines use read a sitemap.

lastmod is not a reliable value. Sometimes in some CI environments it's the present time (the moment where the file is created after cloning the repo, instead of when this file content was modified for the last time). Another option is using Git Last Updated value, but this also doesn't work for CI like GitHub Actions, because the repository is cloned without the history (only the last commit).

well that is cursed. you should probably paste this paragraph to https://lume.land/plugins/sitemap/

Fair enough. I've updated the documentation including that.


Crypt module not supported on windows

Steps to obtain:
Follow the steps on the readme file

Can I start working on it ?

Can I start working on it ?

Sure.

@harith-hacky03  I removed the crypt module as it was unnecessary. It should work properly on Windows now.

Tested in Windows 10. The server is running without a problem.


Split localization samples per AspNetCore version

Fixes #18545

@Rick-Anderson there's few things we need before merge:

Update the references in the docs, I may need your help here

Check the .csproj

Shall we add a PoLocalization sample?

Update the references in the docs, I may need your help here

I'll take care of that.

Check the .csproj

OK

Shall we add a PoLocalization sample?

Let's do that in another issue/PR when you have time.

Regarding the docs it will detect the proper sample version based on the selected version or shall I add sort of monikers?

Let's do that in another issue/PR when you have time.

Ok
For me I think I done, but let me do a quick review before final approval
Thanks

Regarding the docs it will detect the proper sample version based on the selected version or shall I add sort of monikers?

I'll take care of monikers


Wrapper Color

Hi,
is there a chance to add wrapper color to props?

Short answer is: no.
To support more custom configurations, the parameters should be redesigned. Modify the code in your project maybe the fastest way for you.  And feel free to PR :)


Test the Stargate APIs in e2e tests

What this PR does:
This PR adds checks for the Stargate APIs in e2e tests.
Which issue(s) this PR fixes:
Fixes #65
Checklist

[x] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[ ] CHANGELOG.md updated (not required for documentation PRs)
[x] CLA Signed:  DataStax CLA

Rebased to benefit from #90 and added some tests for Stargate deletion.


VRCFaceTracker MeowFace module forces process close on start when the connection is opened

I.E. i have a MeowFace on Android that is already opened a connection to the module's socket, lemme provide the logs
latest.log

same here


Verify contracts in nested folders

Codecov Report

Merging #258 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #258      +/-   ##
==========================================
+ Coverage   97.41%   97.41%   +<.01%
==========================================
  Files          81       81
  Lines        3128     3129       +1
  Branches       20       20
==========================================
+ Hits         3047     3048       +1
  Misses         78       78
  Partials        3        3

Impacted Files
Coverage Δ

...ckages/etherlime/cli-commands/verifier/verifier.js
100% <100%> (ø)
:arrow_up:

...ckages/etherlime/cli-commands/flattener/flatten.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Help in format (-#,##0)

Hello,
I would like to format my numbers when negative for the following format:
Format:
(-#,##0)
Example 1:
Input: 1000 | Output: 1.000
Example 2:
Input: -1000 | Output: (-1.000)
It is possible?

numeral(input).format('0.000')
negative numbers should work as intended
fiddle: https://jsfiddle.net/38fvk4ra/

Hello,
Expected to present: (-2.234)
Including the brackets ().
not work in fiddle: https://jsfiddle.net/38fvk4ra/1/

so the brackets should only be applied to negative numbers? id suggest adding them after format then because its such a random usecase

Numeral supports a - sign or ()'s for negative numbers, not both.  do as bertuzb suggested and add the brackets afterwards.  Thanks


rest_command - unable to use variables in service calls

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.40.2
Python release (python3 --version):
Python 3.4.2
Component/platform:
rest_command
Description of problem:
It's not possible to pass parameters to rest_command via service calls.
longer description https://community.home-assistant.io/t/rest-command-vs-notify-problems/9949
Expected:
It's possible to use rest_command in automations, controll it via service calls
Problem-relevant configuration.yaml entries and steps to reproduce:
rest_command:
  test:
    url: 'https://example.com/'
    payload: 'asd=dsa'

Open service tab
Select Domain rest_command and Service test
Press CALL SERVICE button. HA should call https://example.com/?asd=dsa but actually only https://example.com is requested

Open service tab
Select Domain rest_command and Service test
Add service data {"username":"username","password":"123456","payload": "11111111=asdasdasd","method":"POST"}
HA should make POST call with payload and username/password but actually it makes GET (default) call to https://example.com

Traceback (if applicable):
<IP_ADDRESS> - - [25/Mar/2017:20:47:57 +0200] "GET / HTTP/1.1" 200 379 "-" "HomeAssistant/0.40.2 aiohttp/1.3.3 Python/3.4"
<IP_ADDRESS> - - [25/Mar/2017:20:49:25 +0200] "GET / HTTP/1.1" 200 379 "-" "HomeAssistant/0.40.2 aiohttp/1.3.3 Python/3.4"

Additional info:

Payload is the body of HTTP request. Use url to get parameter like http://xyget.com?bla={{id}} and use post it is not case sensitive. I think I will change that in future.

It is not possible to use method dynamisch. You need make a test_post and test_get or some things.


loadLanguageAsync seems to not be working for me

I'm not sure if this is a bug or I'm using it incorrectly.
I have a business case where the app language needs to change based on a user action. I'm using the loadLanguageAsync method for this.
After calling the method, I can console.log and see the active language has been changed, but none of the text on the screen is updating.
Is this the correct usage?
methods: {
    async changeLanguage(newLang) {
      await loadLanguageAsync(newLang);
    },
  }

This was my fault. I was using trans instead of wTrans and also in my resolve function, lang was not being passed.


FontAwesome5 pack not rendering icons in version 11.3.0

I'm currently working on Windows 10 Pro with Gradle. I updated from v11.1.0 which was working perfectly with JDK and JavaFX 11/12 to v11.3.0 and now all I get is an empty rectangle instead of icons.
According to https://github.com/aalmiray/ikonli/issues/48, it was already fixed, but I'm unable to make it work.

Which icon pack are you using? Do you have both ikonli-core and ikonli-javafx dependencies set in your project?

Version 11.3.1 fixed this; didn't have to include ikonli-core, only ikonli-javafx and fontawesome5-pack.

With 11.3.0 and 11.3.1 I have to add explicit ikonli-core or i get
error: cannot access Ikon  new FontIcon("fa-question"));
With 11.2.0 I didn't have to add ikonli-core

A side effect of using the implementation configuration instead of api. This sets in the pom.xml the ikonli-core dependency in runtime scope instead of compile. Will fix soon.


POST succeeds, but GET failed

I encountered a weird issue when calling curl impersonate against a website.
The POST request works just fine and return result successfully.
However, GET request returns Unauthorized error from the server, which seems like cookie was not passed over. Does anyone encounter this before? All cookie values are correct as they all succeed with POST.

One random thing you could check is how you're positioning the cookie in your header order. I've found that if you use curl's default cookie handling it tends to insert the cookie earlier in the header order than the real Chrome does. I believe the real Chrome puts the cookie at the end of all the other headers.
It's possible but probably unlikely that the site you're using might be checking for that. In my case I had to use my own cookie handling code and manually put the cookie header into the right spot.
Also a suggestion I guess to the curl-impersonate project would be to add cookie header positioning as a further feature to think about incorporating?


Run ARM64 builds on ARM64 machines

When building, the ARM64 images fail.
https://travis-ci.com/github/EugenMayer/docker-image-unison/jobs/503519313
standard_init_linux.go:190: exec user process caused "exec format error"

It looks like people have hit this error when building on an
architecture different than the target.
https://forums.docker.com/t/standard-init-linux-go-190-exec-user-process-caused-exec-format-error/49368/3

I was hoping to kick off Travis CI to test this. 😅


CDK execution role (custom template) throws S3 access denied error for pivotRole auto-created nested stack

Describe the bug
When using the cdk bootstrap with the new IAM policy provided in the UI some policies seem to be missing to access the template for the nested template in S3.
We created the IAM policy and bootstraped, then when linking the environment, the CloudFormation stack failed with a CREATE FAILED - S3 access denied from the nested pivotRole stack.
The nested stack does not even get created +  there are no S3 calls in the pivotRole stack. Both suggest that the issue is on reading the CloudFormation template from the S3 bucket that CDK uses to store the synthesized template.
We tested using the Administrator policy in the bootstrap command and trying to link again and it works without issue. We can confirm that the issue is in the CDK custom exec role and that is missing some S3 policies
How to Reproduce
*P.S. Please do not attach files as it's considered a security risk. Add code snippets directly in the message body as much as possible.*

Expected behavior
No response
Your project
No response
Screenshots
No response
OS
na
Python version
na
AWS data.all version
V1.6.1
Additional context
No response

I'd also polish the frontend a bit under this issue to make clear the differentiation between the 2 kind of bootstrapping method.


add await to thenable method

This PR addresses  #13628

Pull Request Test Coverage Report for Build 397157197

5 of 7 (71.43%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 41.241%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/notebook/src/book/bookTreeView.ts
0
2
0.0%

Totals

Change from base Build 396968979:
-0.001%

Covered Lines:
21862

Relevant Lines:
48635

💛  - Coveralls


Refactored ML-KEM SIMD scaffolding

Broke out SIMD-specific code into a separate simd directory
Created a trait called Operations to provide a unified API over the different SIMD bit-widths and architectures
Removed the *_int_vec_* prefixes, suffixes, and affixes and renamed constants

This will need more hax changes. We will work on dev for now and do this refactoring later.

ugh, was a local experiment. should not have been pushed.
On Sun, Apr 28, 2024, 20:35 Franziskus Kiefer @.***>
wrote:

@.**** approved this pull request.
There's a lot here that needs cleanup. But we'll do that in follow-ups.
I only reverted the sha3 changes. If that's something you want
@karthikbhargavan https://github.com/karthikbhargavan, do them
separately.
In libcrux-ml-kem/src/simd/mod.rs
https://github.com/cryspen/libcrux/pull/233#discussion_r1582267094:

@@ -0,0 +1,20 @@
+pub(crate) mod simd_trait;

+#[cfg(all(not(feature = "simd128"), not(feature = "simd256")))]
+mod portable;
Portable will always have to be built.
It's ok for now. I'll change that later.
In sys/hacl/c/src/Hacl_Hash_SHA3_Scalar.c
https://github.com/cryspen/libcrux/pull/233#discussion_r1582275433:

@@ -1158,14 +1158,12 @@ void Hacl_Hash_SHA3_Scalar_sha3_224(uint8_t *output, uint8_t *input, uint32_t in
uint64_t c = Hacl_Impl_SHA3_Vec_keccak_rndc[i0];
s[0U] = s[0U] ^ c;
}

uint8_t hbuf[200U] = { 0U };

What are these changes? They shouldn't be here.
—
Reply to this email directly, view it on GitHub
https://github.com/cryspen/libcrux/pull/233#pullrequestreview-2027211877,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFUVS3TJKY6UGH63RKUSQDY7U6OLAVCNFSM6AAAAABGVGB566VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMRXGIYTCOBXG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Update index.html

updates

@pankhuri22 thanks for contributing, you can now have a look at some existing issues.


Trouble shooting for luarocks images lib

I didn't see this particular error in any open/closed issue.
If the follow error message is received, then image must be installed via luarocks
/torch/install/share/lua/5.1/trepl/init.lua:384: module 'libjpeg' not found:No LuaRocks module found for libjpeg
    no field package.preload['libjpeg']

Same problem

exact same.

Did the solution work for y'all? That is the important thing.
On Tuesday, March 8, 2016, thunderdork<EMAIL_ADDRESS>wrote:

exact same.
—
Reply to this email directly or view it on GitHub
https://github.com/jcjohnson/neural-style/issues/140#issuecomment-194084373
.

--
Steve Bussey • SalesLoft
• Software Engineer
| e<EMAIL_ADDRESS>| w. http://salesloft.com
http://salesloft.com/?v=1&utm_expid=76406993-1.E5ncJhwyTAW9uGaR1WdBTA.1?utm_source=email-signature
| s. 3405 Piedmont Rd NE, Atlanta, GA 30305
https://www.facebook.com/SalesLoft https://twitter.com/SalesLoft
https://www.linkedin.com/company/salesloft
http://salesloft.com/prospector/#video http://blog.salesloft.com/blog/

Worked for me

on ubuntu installing this packages (png is probably not needed, but i've installed it anyway, that's why it's listed below)
$ sudo apt-get install libjpeg-dev libpng12-dev
and reinstalling image package
$ luarocks install image
worked for me

@vrockai
thanks any way ,it's work for me either,it was such a problem

try
bew install jpeg
then
'luarocks install image'
worked for me


SyntaxError: Missing parentheses in call to 'print'

File "./apt2.py", line 20
print "*** print_tb:"
^
SyntaxError: Missing parentheses in call to 'print'. Did you mean print("*** print_tb:")?

What is about?

It's python2 but looks like you're using python3 to run it... Try python2 ./apt.py (if you still have python2 on your computer ;) )


Unable to decode images when using Impeller

With Impeller, basic widgets render fine but images fail to decode. Removing the impeller flag, images successfully decode with the same codebase. Impeller would be very useful as smart TVs appear to quite easily suffer from shader compilation jank.
To test:
const Image(image: AssetImage("assets/image.png"));

flutter-tizen run --enable-impeller

The error:
══╡ EXCEPTION CAUGHT BY IMAGE RESOURCE SERVICE ╞════════════════════════════════════════════════════
The following _Exception was thrown resolving an image frame:
Exception: Codec failed to produce an image, possibly due to invalid image data.

When the exception was thrown, this was the stack

Image provider: AssetImage(bundle: null, name: "assets/image.png")
Image key: AssetBundleImageKey(bundle: PlatformAssetBundle#04dac(), name: "assets/image.png", scale:
  1.0)
════════════════════════════════════════════════════════════════════════════════════════════════════

Hmm. Impeller has not been implemented for Tizen yet, so basically the --enable-impeller flag should have no effect. We need to investigate where the bug comes from.
We're planning to add Impeller support to Tizen using the Vulkan backend. You can track the progress at https://github.com/flutter-tizen/embedder/issues/15.


Trying to install express-generator globally but getting EACCES

Hi, I'm following a  MDN guide on how to setup a Node/Express environment, but when I try to run express through the command line, I get bash: express: command not found, even though I have express installed globally and locally in my project. Express-generator won't install globally (it has installed in my project, but doesn't work), and when I try to install I get the following:
`npm WARN checkPermissions Missing write access to /usr/local/Cellar/node/9.3.0_1/lib/node_modules/express-generator
npm ERR! path /usr/local/Cellar/node/9.3.0_1/lib/node_modules/express-generator
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall access
npm ERR! Error: EACCES: permission denied, access '/usr/local/Cellar/node/9.3.0_1/lib/node_modules/express-generator'
npm ERR!  { Error: EACCES: permission denied, access '/usr/local/Cellar/node/9.3.0_1/lib/node_modules/express-generator'
npm ERR!   stack: 'Error: EACCES: permission denied, access '/usr/local/Cellar/node/9.3.0_1/lib/node_modules/express-generator'',
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'access',
npm ERR!   path: '/usr/local/Cellar/node/9.3.0_1/lib/node_modules/express-generator' }
npm ERR!
npm ERR! Please try running this command again as root/Administrator.
npm ERR!      /Users/stephe/.npm/_logs/2017-12-31T00_36_51_307Z-debug.log
Stephe-Mac:mozillaNode stephe$ npm install express-generator
npm WARN<EMAIL_ADDRESS>No repository field.`
I'm using OSX sierra,  recently setup a separate account to make projects, and I'm using Homebrew as a package manager...

I don't have an OS X machine on hand to take a look, but I believe I think I remember seeing this is a common issue if you ever accidentally used "sudo" with an npm command. Have you opened a support issue with npm yet?

No, I haven't. I'll post my issue there as well...

No problem. The error above is happening in the npm tool, before our code has a chance to run (since you were not able to successfully install). We have some simple npm knowledge, but I don't happen to know this issue off hand, unfortunately.

This is a permissions error with installing via brew. Not worth posting to
npm I think. You just need to fix the permissions. Not at my computer or I
would help more sorry.
On Sat, Dec 30, 2017 at 7:05 PM Douglas Wilson<EMAIL_ADDRESS>wrote:

I don't have an OS X machine on hand to take a look, but I believe I think
I remember seeing this is a common issue if you ever accidentally used
"sudo" with an npm command. Have you opened a support issue with npm yet?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/expressjs/express/issues/3520#issuecomment-354578681,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-uwGFUG-2JnfkxCwjdD9XxcLXD7Vq8ks5tFt27gaJpZM4RPvQA
.

No problem! I figured it all out! Thanks for pointing me in the right directions!


The standard library processes should be registered by default instead of forcing the user to register them.

Nope, I think I'll leave them as open options.


Blockly.JavaScript.workspaceToCode is missing

How to generate JavaScript code from Scratch? I see that toCode() function in tests doesn't work, and there is no Blockly.JavaScript.workspaceToCode object.

This isn't clear blockly. This is Scratch 3.0 block runtime. It can't generate JS code.

I see. What it can generate?

Look at Scratch VM playground: https://github.com/LLK/scratch-vm and http://llk.github.io/scratch-vm

We aren't supporting Blockly code generators right now, because they would likely be a large amount of work to maintain and we only have so much time! Our focus for scratch-blocks is generating events and data structures that can be consumed by a Scratch interpreter. But I have heard rumors that if you're brave and copy the Blockly generator code into scratch-blocks, it might not be too hard to get working...

Actually scratch-vm may be the exact thing I need. I'm doing kinda rules engine, so I just need to define some logic and execute it. I don't need explicitly JS code.

Is there any example how to run code in scratch-vm separately from the visual workspace? Example in https://github.com/LLK/scratch-vm presumes workspace there, right?

No, you must load /src/blocks/* for working

@mishoboss You shouldn't need the visual workspace to run code. But, you need to find some way to load blocks into the VM if you don't have the visual workspace. One way that is currently possible is to import an SB2.

But you can open issue on scratch VM repo

Please excuse me, but I'm not sure I fully understand what you mean. SB2 = Scratch Blocks 2? What to do with it?
Here's in short the whole stuff - I need to store the blocks as XML and then load it somehow into a node.js server that executes the logic. I already made it saving as string into a DB, then loading this XML string into the visual workspace and blocks are rendered nicely. So far it's awesome. Now, how can I load this same XML string into the scratch-vm that is on node.js and execute the logic server-side?

SB2 is format for Scratch 2.0

And node program doesn't execute the logic. it's only server that "translates" code to browser-specific format, just like this:
var a = require('./a.js') // module.exports = 'Hello, '
var b = require('./b.js') // module.exports = 'world.'
console.log(a + b);

to:
(function(...){
...
})([
function(...) {
var a = _webpack_require_(1);
...
], function(/* module */...){
module.exports = 'Hello, '
}/*, ...*/)

@mishoboss We currently don't have infrastructure for loading XML into the VM. The only things the VM can load right now are SB2-format JSON projects (created by Scratch 2.0) and events generated by scratch-blocks.

You could try to do this by emulating an event. Check out the CREATE events generated by scratch-blocks, which contain an XML block string that the VM knows how to process. But overall I can't recommend this approach.

OK, seems I'm totally lost now :( Here is what I understood so far:

SB2 is a JSON format used by Scratch 2.0
scratch-vm consumes only SB2
scratch-blocks can't export SB2, but only XML
I have no idea what are these "CREATE events generated by scratch-blocks" :(

Where can I read more about the whole picture of import/export/execute?

In scratch-vm/src/engine/execute.js

No docs?

You can read Wiki and some JSDocs in some scripts.

To add to the confusion, here is some documentation about the events generated by scratch-blocks: https://developers.google.com/blockly/guides/configure/web/events
The current flow for import/export/execute looks like this:

scratch-vm loads SB2 into a block representation.
scratch-vm pushes data to scratch-blocks.
When blocks change in scratch-blocks from UI actions, scratch-blocks tells VM to update its block representation.
scratch-vm executes the blocks.

There is no export. The XML export that scratch-blocks can do can't be readily consumed by the VM, except the same format is used to process the UI actions (events) from step 3 above.
Here is the handler in scratch-vm for scratch-blocks events:
https://github.com/LLK/scratch-vm/blob/develop/src/engine/blocks.js
I'm sorry but this project is under heavy development (it's existed for only a few months now), so we don't support your use case and probably won't for several months.

Thanks!
So it is something like this:

scratch-vm loads SB2
it does some conversion magic and pushes the produced stuff to scratch-blocks internally (not using XML)
on scratch-blocks UI event (like a new block attached) it pushes this data internally again to scratch-vm

So, that said, could I use this approach and store SB2 data into DB from scratch-vm? Then load it and its rendered OK in scratch-blocks?

That is more or less correct. The "internal" pushing back and forth does contain some XML to represent the blocks, but not precisely.
From scratch-vm to scratch-blocks, XML is emitted when first loaded and when switching sprites: https://github.com/LLK/scratch-vm/blob/develop/src/engine/blocks.js#L383
When scratch-blocks emits certain XML-containing events like CREATE, scratch-vm parses and loads it into its own structure: https://github.com/LLK/scratch-vm/blob/develop/src/engine/blocks.js#L203
That approach could work, but we don't currently have a way to export SB2 data from scratch-vm or anywhere else.

@tmickel Look here: LLK/scratch-vm#266 and LLK/scratch-vm#259

Last question that is too far from the initial question, but I would be thankful of an answer. :)
What would be the easiest way to transcode the XML produced from scratch-blocks to json-logic-js JSON code. Can I grab some part of scratch-blocks or maybe scratch-vm and use it as a base to build the conversion logic above it? Can you point me the most right way doing this? Or maybe this idea is not worth investing in, because it will take a lot of time? I know I'm going out the scope of this project, but will appreciate a little help. Thanks in advance!


missing support for text on the left side

https://zh.wikipedia.org/wiki/Module:Routemap#雙文字欄
左備註~~左文字~~左距離! !STR\STR\STR~~右距離~~右文字~~右備註

Rather than making the left texts actually supported, IMO it would be much easier and definitely helpful to apply a quick fix that just ignore anything before ! !.


Fix 001

Turns out that envVars can’t be set from the pom for jetty:run, values
in app.yam can’t have filters on them, and since we are in war, it’s
harder to substitute in properties files. (since they aren’t auto
loaded). So, we switch to web.xml which allows filters (if you enable
it).

Switch to context-param, and change code to use it.
add vars to pom, so they can be changed w/ -D on the mvn line.
Capitalize comments where appropriate.
Add a logging directive to app.yam
Tell .gitignore to ignore intellij.

@tswast FYI
@shun-fan PTAL
I'm going to merge this (after I review) and then merge Shun's changes.

LGTM


Cannot login if 2FA is enabled

The application currently doesn't allow the connection of accounts with Two-factor authentication enabled (https://meta.wikimedia.org/wiki/Help:Two-factor_authentication). The authentication is rejected because there is no way to enter the Google Authenticator Token. Is this possible to enable this feature in the further releases ?

Hi @Linedwell ,
Yes, we are definitely planning to enable two-factor authentication as soon as we can (see #328 ) but unfortunately we're encountering a few problems with the implementation. Hopefully we'll manage to do it soon!

Hi @misaochan,
Thank you for the answer, you can close/merge this issue as duplicate of #328 if you want to (I subscribed to it).
Regards,

@Linedwell Will close and give the other issue the high priority label.


Problem with gathering Token at simplenote.py that outputs Error: Unable to connect to server.

I have attached 2 files : Dockerfile.txt and Makefile.txt.
Please remove the extension for files and then:
make build && make test
What it does - creates a ubuntu:trusty docker container and adds simplenote.vim and simplenote.py (api) . Then it runs vim +SimplenoteList within bash of container.
I've created a temporary user for simplenote and added to .vimrc user&password.
The command outputs error instead of making call with token to list files:
"Simplenote" [New File]
Error: Unable to connect to server.
Press ENTER or type command to continue

I have traced problem to a authenticate method inside of
~/.vim/bundle/simplenote.vim/autoload/simplenote.py/simplenote/simplenote.py

Does not return token, instead it returns None, which then makes  handling request on method:
__get_notes

to have a token=None inside of arguments.
Can you please support with this matter ?
Dockerfile.txt
Makefile.txt

Can you try using simplenote.py and seeing how far you get with that? And then report which version of python you are using with it?
I have a suspicion that since this uses the old version of the API and they decided against releasing the new version of the API it maybe a case of luck whether this now works for folk. I.e. perhaps for newer accounts they just disabled access?
Unfortunately there is probably very little I can do about it.

Ok I will check that, But in the meantime can you guide me to the new API that simplenote has released? So that I could make a PR on simplenote.py
with changes that should be done for the python-api to work with this vim-plugin?

It looked like (from Twitter) there were server updates happening for the web client last week. Everything should be working.
However, it does look like they made a change that breaks some aspects of using this plugin, following this server "maintenance": #99

But in the meantime can you guide me to the new API that simplenote has released? So that I could make a PR on simplenote.py

I actually made these branches some time ago (at the time thinking they'd switch to it soon):

https://github.com/mrtazz/simplenote.py/tree/simperium-using-http
https://github.com/mrtazz/simplenote.py/tree/simperium-using-lib

Discussed here (with link to the Simperium API): https://github.com/mrtazz/simplenote.py/issues/11

That's the thing. They didn't announce anything except for server maintenance. However, when things were back up, the issue seen in #99 appeared.
tweet referenced

We've now switched to the Simperium API so would be interested to now if these auth issues, etc have gone away.

Whoops ^ that commit was not meant to reference this issue. Picked the wrong one.
Previous comment still applies though.


ImageFace aspect ratio bug

Casting width/height*scale to int in the ImageFace constructor can break the aspect ratio. This is not very noticeable when using mm but very noticeable when using inches. For example, if I have a 1999x1000 pixel image and I want the height of the image to be 1 inch, I would use a scale of 0.001. Because of the int cast, this would currently result in a skewed image being rendered in a 1x1 inch rectangle, rather than the expected 1.999x1 inch.

fixed in 2.4.0


Theme not work

I got warning about styles
Invalid prop: type check failed for prop "styles". Expected Array, got String with value "aubergine"
                        <GoogleMap
                            ref="googleMapDetail"
                            style="width: 100%; height: 50vh"
                            :api-key="googleMapData.apiKey"
                            :center="googleMapData.center"
                            :zoom="googleMapData.zoom"
                            :styles="'aubergine'"/>

whats wrong about this code?
Sample code
https://codesandbox.io/s/vue3-google-map-demo-forked-6wrdh5?file=/src/App.vue
I using developer mode API google map

Hi @teguhsatria892
Please make sure you're using the latest version. Themes should now be imported from vue3-google-map/themes and passed through the styles prop. Example:
https://codesandbox.io/s/vue3-google-map-demo-forked-7hriki?file=/src/App.vue
Note that you'll need an API key for themes to work correctly

thanks a lot, i will try


adds a space between 'go' and 'to' on line 15 and and delete the void…

… modifier on lines 64, 65, and 66

Looks so good, Why closed it.

Looks so good, Why closed it.

I'm new to contributing on GitHub. My mistake.

Thank you to both of you!


Create new dictionary

Combine keywords list together with "Title DWPI" and "Crop Ontology Terms"

Created


HiltTestActivity missing setTheme()

I was looking at HiltTestActivity comparing to EmptyTestActivity it is missing setTheme() during onCreate() method.
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setTheme(
                intent.getIntExtra(
                        "androidx.fragment.app.testing.FragmentScenario.EmptyFragmentActivity.THEME_EXTRAS_BUNDLE_KEY",
                        R.style.FragmentScenarioEmptyFragmentActivityTheme
                )
        )
    }

This ignores the theme passed to launchFragmentInHiltContainer method

I second this. Without it, my test fails with
You need to use a Theme.AppCompat theme (or descendant) with this activity.
Thanks for the hint @Sserra90


Icons not displayed correctly in version 6.11.0

Hi,
simple_icons library worked very well for me. I am using version 6.1.0, and the next version to upgrade would be 6.11.0 as describe in the pub.dev available versions
However, when I update to version 6.11.0, all icons are mixed up.
Example of how it looks on version 6.1.0:

And this is how the icons look on version 6.11.0:

I tried cleaning the cache and everything but it doesnt work.
Does anyone else have the same problem with latest version?

Thanks a lot for notifying me about this problem :smiley:
I uploaded the version 6.13.0. Please confirm, that the issue is fixed for you.

Hi,
Thank you for the quick reply. The issue seems to be fixed in 6.13.0  Thank you!


⚠ WSINGRESOS - OXXO has degraded performance

In 5205a07, WSINGRESOS - OXXO (https://srvshyweb.yucatan.gob.mx/cgi-bin/wspd_cgi.sh/WService=wsingresos/internet/compartido/wspagosoxxo.r) experienced degraded performance:

HTTP code: 200
Response time: 22149 ms

Resolved: WSINGRESOS - OXXO performance has improved in 89eef20 after 9 minutes.


newrelic-client-go v0.86.1

Signed-off-by: smcavallo<EMAIL_ADDRESS>
Description of your changes

Updates to the latest newrelic-client-go -> https://github.com/newrelic/newrelic-client-go/releases
go 1.16 -> 1.18
move clients and controller directory from internal (default from the template) to pkg  (matches other providers)
update libraries and code to use the latest crossplane-runtime and crossplane-tools

Note there are zero functional changes for end users.  Internal upgrades and improvements only
I have:

[X ] Read and followed Crossplane's contribution process.
[X ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

local environment testing

@smcavallo I’m interested in likely picking things up from here. Can you please tell me what’s missing from your side on this PR?

closing in favor of https://github.com/crossplane-contrib/provider-newrelic/pull/6


TextInput's onLayout not called

Not sure why the onLayout is not called in TextInput component but called in View component.
  <TextInput
              ref="textInput"
              onLayout={(e) => {
                console.log('--- this is not called?')
              }}
              style={{height: 44, color: 'black', borderColor: 'black', borderWidth: 1, paddingHorizontal: 10, marginHorizontal: 10}}
              placeholder='text here'
              text={'cool'}
              placeholderTextColor={'rgba(0,0,0,.4)'}
              underlineColorAndroid='rgba(0,0,0,0)'
            />

Sample project here
React Native version: 0.28.0
Target: iOS 9.3
Development OS: OS X

https://github.com/facebook/react-native/releases/tag/v0.29.0-rc.0
Fixed in RN 29

Fixed via https://github.com/facebook/react-native/commit/92926f9858c0b60cf9f2e51cb8f25f2e1501c9c1

not fix ，I found this bug on  a little android‘s device ，example ：
Nexus and MEIZU

@facebook-github-bot open
this is an outstanding bug and it is not fixed on version 0.38.0-rc1
I confirm

Not fixed in
react-native-cli: 1.0.0
react-native: 0.37.0
Not working on View
<View onLayout={(event) => this.measureView(event)} ref="imageProfileHolder" style={{
flex: 1,
height: 120
}}>
measureView(event) {
console.log('event peroperties: ', event);
this.setState({svgHeight: event.nativeEvent.layout.height, svgWidth: event.nativeEvent.layout.width});

}
<3 UnicornRainbows

+1
Confirmed that it's broken in RN 0.37.0.
What's the timeline for this fix?

This issue has been fixed in this commit -> https://github.com/facebook/react-native/commit/8a073c1d8b89305a9a2561a7c33740919730f408#diff-b48972356bc8dca4a00747d002fc3dd5


Multiple errors in alta-additions.css file

Could you please have a look at alta-additions.css file of UIKitCommon project, as it has multiple errors.
Such as:
right; instead of right:
Unknown pseudo element :criterion-label-top;
Unknown property fill: etc.

Hi Lancy, when i try to run RDK, application didn't show pages. I get that error:
Root cause of ServletException.
javax.el.PropertyNotFoundException: The class 'oracle.adf.view.rich.component.rich.layout.RichPanelHeader' does not have the property 'pageStickyTitle'.
How can i get over it?


Problems with prepared query parameters in nested query [moved]

This is Issue 1069 moved from a Google Code project.
Added by 2012-09-24T07:43:38.000Z by<EMAIL_ADDRESS>Please review that bug for more context and additional comments, but update this bug.
Original labels: Type-Defect, Priority-Medium, Release
Original description
It seams query parameters (both positional and named) don't work in nested queries. There is no error - the query simply does not return any results.
A simple test case reproducing the problem is available here: https://gist.github.com/3246359#file_query_param_test.java

There are 3 tests :

- one using positional parameter
- one using named parameter
- one manually replacing the '?' withe the appropriate value

and only the last one returns records.

When this is solved, will it be possible to use named and/or positional parameters or  when you assign a context variable to the result of a sub-query?

the new parser fixes all these cases.
It was committed in 2.1 snapshot with some test cases.
Closing this, please re-open if you find uncovered corner cases

Glad to hear that, thanks @luigidellaquila.


Pagination

This adds cursor based pagination support to articles, topics, archives, and users.
Tracking

[x] Articles

[x] Backend
[x]   Frontend

[x] Topics

[x]   Backend
[x]   Frontend

[x] Archives

[x]   Backend
[x]   Frontend

[x] Users

[x]   Backend
[x]   Frontend

Client side updates will follow.

@krishnagopinath I think I'm done here.
The api now returns,
{
  articles: [],
  meta: {
    totalRecords,
    totalPages,
    reminaingPages,
    pageNumber
  }
}

For displaying, as of now I've added a simple Load more button, which we can also switch to pagination controls later (our api allows that).


Lucene Spatial ST_Buffer function

OrientDB Version: 2.2.26
Java Version: 1.8
OS: Windows 7
Expected behavior
ST_Buffer returns a geometry that represents all points whose distance from this Geometry is less than or equal to distance.
Syntax: ST_Buffer(geometry,distance [,config])
In which units is the distance measured? And what limits does this distance have.
Actual behavior
The distance is measured not in meters and kilometers.

Hi @kornelik
it should be degrees, since the only Spatial Reference System supported is 4326
https://orientdb.com/docs/2.2/Spatial-Index.html#spatial-reference-system


Set bitbucket labeler to nil

As bitbucket does not support the labeler, @iinuwa tried to fix it in https://github.com/dependabot/dependabot-core/commit/52db87ebd750452d500e713bc85b745bc84f4352 by setting the default labeler in the constructor to nil. However from my testing this does not fix it, as we're still injecting the default labeler here.

Hitting this as well attempting to use the generic-update-script

Closing as this was handled in https://github.com/dependabot/dependabot-core/pull/4608 (many thanks @stefangr!)


Automate performance testing for CCR plugin

Is your feature request related to a problem? Please describe
We want to to be able to run performance tests for cross cluster repliacation plugin using opensearch-benchmark.
Describe the solution you'd like
We'd like to add the support for replication performance testing as part of OS performance testing framework. Eventually, We'd want to have periodic runs for the replication plugin as part of Jenkins CI.
Describe alternatives you've considered
We explored whether we should have a separate dedicated Jenkins setup for CCR plugin but it seems like a duplication of effort and thus decided to just focus on making it work with OS performance framework instead.
Tasks:

[ ] https://github.com/opensearch-project/opensearch-infra/issues/146
[ ] https://github.com/opensearch-project/opensearch-build/issues/1496
[ ] https://github.com/opensearch-project/opensearch-build/issues/1498
[ ] https://github.com/opensearch-project/opensearch-build/issues/1499
[ ] https://github.com/opensearch-project/opensearch-build/issues/1500

After https://github.com/opensearch-project/opensearch-build/issues/126 we should be able to onboard plugins into whatever we come up with, including CCR.

Yep, I've been following up that issue closely. Let me come up with the list of tasks required to support CCR perf suite and update here.

We'll add a script in replication repo that can be used by everyone contributing to CCR as well as opensearch build.
This script would be invoked as part of distribution build regularly within Jenkins.
Script would use PerfTestSuite & PerfTestCluster to setup multiple clusters and run the replication perf tests. If there are any changes required to support these on OS infra side, I'll take those up and raise the PRs.

What will a script do that existing scripts don't?
I would say this is already test.sh and it takes a manifest as input.

These are the steps we have for running a replication perf test:

Create 2 clusters(Leader and follower).
Connect the two clusters and start replication.
Start data ingestion on leader using OSB and monitor replication metrics.
Stop the replication and remove the connection(Can be skipped in this case since these clusters  won't be reused).

We need to execute the additional logic to setup the replication prior to running the tests. I'm assuming you'd prefer that we keep this logic(same for other plugin owners) in our own repo. If this is not the case, let me know, then we don't really need any separate script we can just call test.sh with an input manifest.

[Triage] Hi @ankitkala , In which repository is the work related to this issue is gonna happen?

Sorry for late reply. The code is expected to be in build repo itself.

Hi @ankitkala , thanks for the update. I dont see any updates on the tasks listed on this META issue. Can you provide an update?

I'll be resuming on this task by end  of this week. Expect the PRs by early next week or so.

https://github.com/opensearch-project/opensearch-infra/pull/151


[NEW]:

Date
May 16, 2022
Description
Miamicoin down 95%
Links
https://qz.com/2165639/miamis-mayor-backed-miamicoin-then-its-price-dropped-95-percent/
Agreement

[X] I irrevocably agree to release my contribution under the CC-BY 3.0 License

Attribution name
joeldg
Personal link
No response

Already somewhat covered at https://web3isgoinggreat.com/?id=miamicoin-loses-money-for-investors


heavy_task_schedule example will cause ServiceProtocolStream dropped

No recurrence, and review code can't find the problem, hold live

Close because can not reproduce for now, reopen when reproduced.


Make the script a module for PyPi

Description
Make the script a "module" for PyPi, while not having to change anything. main.py will still run the script normally, so running from source doesn't change at all.
Docs were updated as well for the new features. It outputs debs into ~/.permasigner/output.
Current issues

[ ] Will probably fail to get items from the data folder, should we put these in ~/.permasigner? Might be better to clone the repo into ~/.permasigner...
[ ] Args won't be passed and it's failing to run because of it.

My suggestion is to add support for XDG_DATA_HOME variable so that we don't pollute the home directory for those who have set it.

Is this set by default? Or should we check if it exists.

Hmm, I think better idea would be to require the output flag.


fit_transform hangs after computing embeddings with bigger data sets

I'm not sure where this issue is stemming from, my data set is of shape (253695, 6716). I set the verbose flag to True and UMAP hangs after it succesfully computes the embeddings.
Training with a subset (50k) finishes (and returns directly) within a minute. 100k finishes computing the embedding after a minute or two but only returns after 5 minutes. 150k returns after 15 minutes and so on, increasing the data size seems to increase this overhead after computing the embeddings.
However, if I use fit().transform(), everything works smoothly

@jyrr Can you provide more details? Code snippet, version info, or any suggestions on how to reproduce this issue?


Out of space so CS50 codespace fails to start

I can't figure out what to do next.  I get the following message.
An unexpected error occurred that requires a reload of this page.
The workbench failed to connect to the server (Error: connection record disposed by ConnectionHub: {
"errorMessage": "Failed to start VS Code Remote server due to low disk space. Export your changes and create a new codespace to continue working.",
"originalErrorType": "",
"errorCode": -1,
"errorType": "GenericError",
"stack": "Error: Failed to start VS Code Remote server due to low disk space. Export your changes and create a new codespace to continue working.\n at [CODESPACE_URL]\n at [CODESPACE_URL]\n at [CODESPACE_URL]\n at [CODESPACE_URL]\n at d ([CODESPACE_URL]\n at a ([CODESPACE_URL]"
} $1 $1)
Version: 1.85.2
Commit: 8b3775030ed1a69b13e4f4c628c612102e30a681
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

/causedByExtension please use the course support channel.

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


Running it only once

Hi
First of all thanks for this awesome plugin, very helpful.
I had a little bit of hassle figuring out how to only run the watch function callback only once. Could not find a clue how to.
I resolved my problem with a work around, my callback function is now wrapped around an if sentence so that it only watches the class name I want like so:
if (newValue === 'the-class-I-want') {
But again if this specific class is added again the callback function will run which is not something I would want.
Is there some way to accomplish what I'm after?

Sorry for the delay in responding. I missed this issue.
I'm not sure I understand. The watcher is designed to indefinitely monitor the watched properties, but it should be pretty straight forward to add some logic into the event handler to shut the watcher down after the first hit.
// some element to monitor
var el = $("#notebox");

// hook up the watcher
el.watch({
    // specify CSS styles or attribute names to monitor
    properties: "top,left,opacity,attr_class,prop_innerHTML",

    // callback function when a change is detected
    callback: function(data, i) {
        var propChanged = data.props[i];
        var newValue = data.vals[i];

        var el = this;
        var el$ = $(this);

        // do what you need based on changes
        // or do your own checks

        // now unwatch - delay to ensure ops have run aren't reset
       setTimeout(function() { el$.unwatch(data.id); }, 10);
    }
});


feat: add detection for multiple js frameworks (#105)

it resolves #105

I just need to update the documentation but the code is ready for review. As usual 95% of the file changed are file used for testing it.

Still questioning if we handle framework versions: ex Angular 1.x vs Angular 2.x as I think the devfile is Angular 2.x, worth checking. Another topic, do we have a documentation of the port lookup limitations somewhere ?

Now Angular reached version 15 so I don't think 1vs2 is still a problem. It was something related to 5 years ago. If someone is still using v.1... well, they should expect to find tools that do not support it fully.
About the devfile, i don't think it is bundle to a specific version. It just execute some npm command so it really depends on what the user add in the scripts, no?
What do you mean about port lookup limitations? To write a doc like the existing one that explains how ports are retrieved with these frameworks?

Still questioning if we handle framework versions: ex Angular 1.x vs Angular 2.x as I think the devfile is Angular 2.x, worth checking. Another topic, do we have a documentation of the port lookup limitations somewhere ?

Now Angular reached version 15 so I don't think 1vs2 is still a problem. It was something related to 5 years ago. If someone is still using v.1... well, they should expect to find tools that do not support it fully. About the devfile, i don't think it is bundle to a specific version. It just execute some npm command so it really depends on what the user add in the scripts, no?

Trying to prevent false positive. If the Angular devfile does not work for Angular 1.x then we should map Angular 1.x applications to this devfile.

What do you mean about port lookup limitations? To write a doc like the existing one that explains how ports are retrieved with these frameworks?

Depending on the framework, there are may ways to configure the port. So we should make sure the way we support is well documented.


Release memory for cancelled tasks earlier in TaskExecutor

We can save some memory in failure scenarios here (and a tiny bit in every case) by moving our started flag to the FutureTask and using the callable outright. First of all, we save the wrapper callable, but that also allows us to just set(null) on cancellation instead of waiting for the task to run to set the null.
In case we have longer running tasks executing and it would take a while to get to the cancelled tasks, this saves some memory and allows us to return from the method earlier.

I believe this change is ok, but I am no longer very confident about TestTaskExecutor#testCancelTasksOnException testing a real-life scenario, because it was recently changed to no longer rely on executorService. If I restore that, I get some failures here and there (before as well as after this change). It seems to me that the test was simplified by removing concurrency, and that changes entirely what the test ends up testing.
Additionally, could we have a test around memory being released? (does not have to be perfect, but just something that verifies that the result is nullified?

I believe this change is ok, but I am no longer very confident about TestTaskExecutor#testCancelTasksOnException testing a real-life scenario, because it was recently changed to no longer rely on executorService. If I restore that, I get some failures here and there

I don't think it was ever materially changed. Before it used to run on a single threaded executor and we'd block the main thread so it was effectively single threaded anyway?
If you want to test that a task failure makes all tasks past it in the queue fail, you kinda need a single thread only for things to be deterministic?

(does not have to be perfect, but just something that verifies that the result is nullified?

Hmm, it's quite hard to create a useful test here with concurrency enabled. But maybe we could assert that all tasks/runnables are either completed or have been started at the end of setException, i.e. after the cancelAll?

Ok , my hope was that we can somehow test cancellations with a scenario that's closer to reality, but I do see challenges. I also spent quite a bit of time on this and hit different issues that made testing very difficult.
Perhaps we could at least write a test that fails without the change you made, around nulling memory?

@javanna

Perhaps we could at least write a test that fails without the change you made, around nulling memory?

I'm really sorry. I thought about this quite a bit, but outside of actually testing that the callables go out of scope via some Cleaner magic I think there's no way to do this, all of the functionality is internal to the task executor class. There's just not really any functional change here, it's just that I made the Callables get GCed quicker and saved a little bit of indirection in general.
I think this might be one of these things where we have to trust the code?

Presumably, this micro optimization wouldn't be visible in practice because the tasks that we run allocate even bigger objects such as IndexInput clones? That said, it doesn't look more complicated to me, so I'm good with it.

Presumably, this micro optimization wouldn't be visible in practice because the tasks that we run allocate even bigger objects such as IndexInput clones?

I wouldn't even see this as an optimization. The point is more to avoid potential memory leaks after cancellation (I guess not quite leaks they'd resolve eventually) in case of a long queue on the executor because we'd still hold a reference to the futures list unless cancel resolves the future.

Thanks Luca!


SR-8407: NSNumber as? Bool should only work for 0 or 1

NSNumber as? Bool should only work for NSNumber values 0 and 1 to
match Darwin.

@swift-ci please test

I recall @phausler doing a lot of work in this area last year as far as bridging goes; do we need to update it elsewhere here now that we have more consistent bridging with linux?

@parkera The behaviour for as? Bool now matches Darwin, although I think when the bridging was merged to master and swift-4.2-branch it actually regressed the previous behaviour which as also correct, so this probably warrants a cherry-pick to 4.2 as well.