Config to compile const enum as enum in development / Get SyntaxKind during development

Hi,
When I work on the code, I have to change SyntaxKind from const enum to enum so that I can do write(`kind: ${SyntaxKind[node.kind]}`).
Is there a way to config the compiler and jake to treat const enum as enum so this would work?
Or is there another way to get the SyntaxKind without doing SyntaxKind[node.kind]?
Thanks,

You can keep the mapping at runtime with --preserveConstEnum for debugging purposes, but this behavior is the point of const enums.

jake already passes --preserveConstEnums to tsc.js, but it still fails compilation:
error TS2476: A const enum member can only be accessed using a string literal

these exist only for debugging purposes. if you really want to do that cast to any. (<any>ts).SyntaxKind[node.kind]

great! That's exactly what I'm doing. Just to write out stuffs to understand about the node for debug purposes.
(<any>ts).SyntaxKind[node.kind] works because (<any>ts) suppress all type checks?
Thanks a lot!

yes.


Readthedocs.org lost my project

I linked my GitHub, and setup my project (as evidenced by the successful build here: http://json-query-documentation.rtfd.org/).
However, after logging into readthedocs.org a day later, it seems to have lost my project as well as the GitHub linking. Can someone please look into this?

It looks to be owned by the user satish -- is that not the username you're logging in with?

Oh wow, I must have signed up twice. You are correct, my apologies. This can be closed.


feat: add aks-engine release v0.33.1

See https://github.com/Azure/aks-engine/releases/tag/v0.33.1

Tested by pushing to my fork's master and doing brew install mboersma/aks-engine/aks-engine.


Screenshots showing Azure resources in (Peview)

The screenshots for abobe shows that PIM for Azure resources is still in preview, you may want to update it.

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

ID: a5e13581-06f7-9936-f7b6-e30b21a2d92d
Version Independent ID: f8f0d669-69fa-c154-2627-77df1a58ba0e
Content: Overview of Azure PIM Resource RBAC
Content Source: articles/active-directory/privileged-identity-management/azure-pim-resource-rbac.md
Service: active-directory
GitHub Login: @rolyon
Microsoft Alias: rolyon

Thanks for the feedback! We are looking into this and will get back to you shortly.

@aryapankaj Thanks for reporting! I have assigned the issue to the content author to update the screenshot in the document.

Thanks for the feedback. I will take a look at this article.
#in-progress


Revert "Hotfix: Stop using vscode-nls for some key localizations"

Reverts microsoft/vscode-jupyter#10773
Given that Tyler fixed the bug, here: https://github.com/microsoft/vscode/pull/154989
Fixes #10774

Does it require it to be submitted to verify the fix? Or can you build the translated version locally?


Chromium 57 embedded credentials depricated

Plugin can`t connect to kodi. Error page:
Errors
Clear all
Log
Context menu item created successfully
Log
POST {"jsonrpc": "2.0", "method": "Player.GetActivePlayers", "id": 1}
Warning
Subresource requests whose URLs contain embedded credentials (e.g. https://user:pass@host/) are deprecated, and will be blocked in M59, around June 2017. See https://www.chromestatus.com/feature/5669008342777856 for more details.
Log
POST {"jsonrpc": "2.0", "method": "JSONRPC.Version", "id" : 1}
Error
Uncaught Error: cannot call methods on slider prior to initialization; attempted to call method 'value'

In POST request password field blocked too.


Bump minio version

Bump minio version in tools_system_images
Related Issues:
rancher/rancher#27467
Related PR's:
https://github.com/rancher/types/pull/1169

@prachidamle @aiyengar2 Just updated to latest tag


Invite set of external users to register into the organization at once

Describe the solution you would prefer
The Asgardeo console currently have sending user invitations to single user to register into the organization.  But Asgardeo Console doesn't have an option to invite a set of users at once by providing their emails.
The admin can enter few emails, and assign set of roles when sending invitations.

After completing this epic:

Console has an option to invite a set of users by providing their emails.
This invitation option will be available in primary organizations and sub-organizations as well.

Additional context

Update 24/10/2023

[UI Design] Design user interfaces to invite multiple users to the organization - Done

Figma File: https://www.figma.com/file/3pAxSvbWnt7AcuJlz4neM4/Invite-Bulk-Users?type=design&node-id=3%3A16&mode=design&t=H5UIkDHg5O8sy9fA-1

[FE] Implement the invite multiple users to the organization feature - Done

PR: https://github.com/wso2/identity-apps/pull/4229

[Docs] Documentation for invite multiple users - ToDo
Issue: https://github.com/wso2/product-is/issues/17282

Steps to verify the invite multiple users feature
Prerequisites

Enable the email as username feature

https://is.docs.wso2.com/en/latest/guides/identity-lifecycles/enable-email-as-username/#:~:text=Log in to the Management,Update to save the changes.

Enable sending emails

https://is.docs.wso2.com/en/latest/deploy/configure-email-sending/

Enable ask password flow
https://is.docs.wso2.com/en/5.9.0/learn/creating-users-using-the-ask-password-option/

Steps

Login to console
Navigate to User Management > Users
Click the Add User dropdown > Multiple Users
Type the email and enter (You can type and enter multiple email addresses)
Click the Invite button
Verify the successs message is displayed
Search and verify usernames of added users are listed


Add tool to OCR scanned PDFs

We shoud add a tool for optical character recognition.
E.g., https://github.com/manisandro/gImageReader/issues/499 or another Qt based tool that uses the tesseract OCR engine.

http://www.tobias-elze.de/pdfsandwich/ might be an easier choice and is already packaged in FreeBSD; however it is a command-line tool.


binlore: 0.1.3 -> 0.1.4

Motivation for this change
Improve resholve's handling of zip and ncurses packages.
Things done

Built on platform(s)

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

[x] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg eval

@bobby285271 binlore is just an API for generating an output from other packages, so it's roughly expected that ofborg won't have much to say.
Result of nixpkgs-review pr 157234 run on x86_64-linux 1

3 packages built:

arch-install-scripts
dgoss
yadm


[proposal] add did:3 support

Background
did:3 is a DID Method maintained by Ceramic
Requirements

[x] inline resolver; we can use the resolver from ceramic: 3id-did-resolver

import { getResolver as get3IDResolver } from '3id-did-resolver'
// ...
new DIDResolverPlugin({
  // ...
  ...get3IDResolver(),
})

[ ] @veramo/did-provider-3id
This requires an implementation of AbstractIdentifierProvider that allows for the creation, update (key rotation), and deletion of 3IDv1 DIDs

It is only necessary to be able to create 3IDv1 DIDs, not 3IDv0
testing
The PR closing this issue should contain (at minimum) the following tests:

did:3 is created and used as an issuer in a call to createVerifiableCredential(), then that credential should be verified successfully using verifyCredential().
did:3 is created and a key is successfully added (e.g. didManagerAddKey)
key is successfully removed
didManagerAddService should fail, as did:3 does not currently support service endpoints

It is acceptable to provide only integration tests for this (meaning tests added to the top level __tests__ folder)
Constraints

Environments: All plugin packages published in this repository are meant to work in nodejs, web(react) and mobile(react-native) environments. This will apply to this new package too.
Minimal new dependencies: Packages in this repository are often installed together so dependencies must be reused between them when possible. No new dependencies should be introduced if this can be avoided.

Hi @nickreynolds, nice to come across your work here, I've just had a quick cursory exploration of this project. I've recently applied for the bounty, hopefully you'll consider it. I'm available to begin as soon as possible.

Hi @brymut , I've approved you to work on this issue. Please let us know if you have any questions about your implementation or Veramo in general, and I encourage you to join our Discord as well: https://discord.gg/q9uYY3nD

Link to the Bounty, for record keeping: https://gitcoin.co/issue/29622

https://github.com/uport-project/veramo/pull/627
@brymut this is a PR opened over a year ago to implement this functionality as well, but it's quite outdated and will not be merged. I think this would be a great resource for you though.

The official public gateway to the ceramic network was deprecated, so this is blocked until we can find or host another one.


support migration flow

https://github.com/kamilsk/Common/issues/28

too overly complicated mechanism


Fix and improve goexpect use in tests and add verbose flag

What this PR does / why we need it:

Use goexpect switch-case system for more flexibility on Fedora login
Fix all uses of '$' and '#' in goexpect regexes
Set verbose flag on console expecter, should make failures a lot more obvious

The original motivation for this is that the Fedora image takes quite a while to set the Fedora password:
[...]
Login incorrect

testvmiwdjvdrzphwq6kq5pkdl4hqlhd42m2624mbcnq5c8bpcmqfv7 login: [  146.207476] cloud-init[763]: Changing password for user fedora.
[  146.219633] cloud-init[763]: passwd: all authentication tokens updated successfully.

Trying to log in too early will fail, and the previous code didn't retry on failure.
The rest of the PR is what I found while debugging that.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/retest

Force-pushed minor fix (+ rebase) to ensure we only match the VM login prompt.

great job
i find some more prompts that miss \\ using this command git grep prompt | grep BExp
and in the last line of
LoggedInFedoraExpecter
utils.go
return expecter, configureIPv6OnVMI(vmi, expecter, virtClient, "#")
if you want to take it as part of this PR
thanks

/hold
Thanks, I actually pushed an old version of the code, fixing now. Will look for missing prompts too.

/hold cancel
Should be fixed now.

/approve

I thought I understood the double backslash used for the prompts, but I lost it.
How is “\$” matching the prompt? Isn’t this matching things that end with “\”?

Both golang (double-quoted) strings and golang regexp support escaped characters.
Since we want the regexp to be \$ ($ means end-of-buffer in the default mode), we escape the backslash when defining the string.
$ is not escapable anyway in strings, only in regexes, so "\$" would throw a string error.
Using backticks instead of double-quotes would disable string escapes, and therefore require only one backslash, which is ultimately the right solution. (except for strings that actually need an escaped character like \n).
I hope that makes sense...

/retest

/retest

/retest

/cherry-pick release-0.30

/retest

Taking  this PR in. Tests which should work better did not fail and we had green runs eveywhere where it matters.


Feat: Changing the logo of the website.

Describe the feature
The logo of the website can be made more bold and colorful which suits the overall design of the website.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@anuragverma108 @huamanraj please assign this issue to me, I will do my best.

@anuragverma108 @huamanraj please assign this issue to me, I will do my best.

great 👍 but first get your design approved!

@huamanraj These are some of the designs , I prefer the last one.


Have some problems lately with Azure Service Bus

Hi!
I'm using the 3.4 version for a while and this version worked well. But for some weeks now, I have a lot of troubles...
My app is working file for days, but at some point, the send method's promise never get resolved or rejected and the message is never sent to Azure.
I have to kill my app and restart it to make it works again.
I have event listeners on errorReceived and detached and they are not getting called.
My code is very simple:
return queuer.send(msg).then((state) => { 	logger.log(messageType, innerMessageId, via, retries, 'Message sent', state); }).catch((err) => { 	logger.log(messageType, innerMessageId, via, retries, 'Error while sending message', err); 	throw err; });
What should I do to get more detailed infos about what's going on? I really need to get this fixed asap.
Thanks for guiding me...
VB

These are typical issues seen in node-amqp10. It happens due to the library falling in incorrect state.
Moreover node-amqp10 is not being actively maintained. We are developing an sdk for service-bus. It is developed on top of rhea. There is an open PR.
We expect to release a version for Azure ServiceBus by the end of this month. Hopefully that can help solve some of your issues.

Hi.
Thanks, I'll keep an eye on this...
Meanwhile, is there something I could do to minimize that problem?
Thanks
VB


Can we put constraints on the search space?

Hello, I can see in the examples that the chemical space is built in a hypercubic way. However, in my case I cannot reach all the points in this hypercube, instead I will need to put some constraints (e.g. the summation of coords value on all the axis is less than 1).
Just wondering if we are allowed to do that using edbo, thanks!

Sorry for the late response. You can constrain the search space. However, you will need to do it by explicitly selecting the points. This is because edbo is designed to optimize over an arbitrary search space passed as a pandas DataFrame. edbo will optimize over whatever you pass as the domain. https://b-shields.github.io/edbo/bro.html

How does one go about updating the search space (in BO_express) after having selected the points one wants to include in it?
I have tried to update the objective like this: bo.obj = objective(domain='new domain dataframe') but am not sure whether this is the correct way.


Added Response API and books

Basically CivGuide but as an api integrated into CivMenu. @rourke750 wanted me to do it this way so, uh, here ya go!

Can one of the admins verify this patch?  Type 'ok to test' to test.

Should this be merged?

I sure think so

For some reason I dont have permission to merge stuff here, @ttk2 @rourke750  could one of you merge this?

Try now.

Thanks

WOHOOOO. Now just gotta integrate it into all the other plugins


Where to extract methods ?

I'd like to extract methods from fabricators to a helper.
How should I go about that ?

That is a great question! Could you give me an example of what you're trying to do? Pseudo-code is fine.

Let's say I have a method that generates a random string that I'd like to share across a few fabricators. Is there a natural place for that method ?

I always recommend using ffaker in conjunction with fabrication. For example,
Fabricator(:company) do
  name { FFaker::Company.name }
end

It has lots of ipsums and other random word generators. Pretty much any random data you could possibly need. Just wrap it in a block as I did above when you pass it to the attribute and it will work as expected.


SEP V008 - Functional Glyphs

https://github.com/SynBioDex/SBOLv-realizations/issues/13

Done doing closing


[WIP] Test ValidateServicePrincipalProfile

Fixes https://github.com/Azure/ARO-RP/issues/200 by adding a unit test and doing necessary refactoring

Needs to be reworked due to recent changes

Most of what I was proposing to put in the ValidateServicePrincipalProfile test (aside from looking for Application.ReadWrite.OwnedBy role) should now be in a GetToken test.
GetToken would then need a receiver so that the interval passed to PollImmediateUntil could be mocked.
Before refactoring this, we should discuss.


informant: handle signals

as discussed

I think the pcancel change is a bit dubious, more for ergonomics than for correctness, but I don't think we should block on it..

Yep, now see it's definitely dubious. Best to revert.

I think we want both PRs to be separate commits into main, right? Because https://github.com/neondatabase/autoscaling/pull/60 is a PR into this branch, I think the way to do that is to merge this first, and then https://github.com/neondatabase/autoscaling/pull/60 will become a PR into main, but yeah.

Yup, I got mixed up. Better to merge this first.
I have no more comments/issues, feel free to merge.


fixes #12

wait until the last second to make a :root {} for putting vars into

added a new test which verified locally that the bug did exist and then is fixed.
i also tested the plugin updates in a test repo where the bug was logged, and the bug is resolved 👍🏻


SPLAT-1209: Enhance datacenter not found to contain list of known datacenters

SPLAT-1209
Changes

Updated cache to added known datacenters to error text when datacenter is not found

/lgtm

/retest

/assign @gnufied

/assign @gnufied
Ready for review.  Thanks!

/label docs-approved
/label px-approved
/label qe-approved

/approve


add toshi gecko

Hey @Complexlity! Thanks for your contribution! I can see that you are missing OpenFrames metadata. Take a look at this doc.

Hey @Complexlity thanks for your contribution! Right we are looking to add open source repos that serve as example for the community. If the frame is not fully compatible then its not eligible for the bounty. Transactions are coming soon! Maybe we can review it again by then


Sg/checkpoint control

Instead of storing BTreeMap as AccountHistory, have a simple (account_id, block_number) -> Balance map

I am getting this error after merge:
error[E0277]: the trait bound `<E as Environment>::AccountId: IntoVisitor` is not satisfied
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/client.rs:327:5
    |
327 |     scale_decode::DecodeAsType,
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `IntoVisitor` is not implemented for `<E as Environment>::AccountId`
    |
    = note: required for `<E as Environment>::AccountId` to implement `DecodeAsType`
note: required by a bound in `subxt::ext::scale_decode::visitor::types::composite::CompositeField::<'scale, 'info>::decode_as_type`
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/scale-decode-0.5.0/src/visitor/types/composite.rs:167:30
    |
167 |     pub fn decode_as_type<T: DecodeAsType>(&self) -> Result<T, crate::Error> {
    |                              ^^^^^^^^^^^^ required by this bound in `CompositeField::<'scale, 'info>::decode_as_type`
    = note: this error originates in the derive macro `scale_decode::DecodeAsType` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: the trait bound `<E as Environment>::AccountId: IntoVisitor` is not satisfied
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/client.rs:327:5
    |
327 |     scale_decode::DecodeAsType,
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `IntoVisitor` is not implemented for `<E as Environment>::AccountId`
    |
    = note: required for `<E as Environment>::AccountId` to implement `DecodeAsType`
note: required by a bound in `subxt::ext::scale_decode::visitor::types::tuple::TupleField::<'scale, 'info>::decode_as_type`
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/scale-decode-0.5.0/src/visitor/types/tuple.rs:128:30
    |
128 |     pub fn decode_as_type<T: DecodeAsType>(&self) -> Result<T, crate::Error> {
    |                              ^^^^^^^^^^^^ required by this bound in `TupleField::<'scale, 'info>::decode_as_type`
    = note: this error originates in the derive macro `scale_decode::DecodeAsType` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: the trait bound `<E as Environment>::AccountId: EncodeAsType` is not satisfied
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/client.rs:334:9
    |
334 |     pub deployer: E::AccountId,
    |         ^^^^^^^^ the trait `EncodeAsType` is not implemented for `<E as Environment>::AccountId`
    |
    = note: required for the cast from `&<E as Environment>::AccountId` to `&dyn EncodeAsType`

error[E0277]: the trait bound `<E as Environment>::Hash: IntoVisitor` is not satisfied
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/client.rs:352:5
    |
352 |     scale_decode::DecodeAsType,
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `IntoVisitor` is not implemented for `<E as Environment>::Hash`
    |
    = note: required for `<E as Environment>::Hash` to implement `DecodeAsType`
note: required by a bound in `subxt::ext::scale_decode::visitor::types::composite::CompositeField::<'scale, 'info>::decode_as_type`
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/scale-decode-0.5.0/src/visitor/types/composite.rs:167:30
    |
167 |     pub fn decode_as_type<T: DecodeAsType>(&self) -> Result<T, crate::Error> {
    |                              ^^^^^^^^^^^^ required by this bound in `CompositeField::<'scale, 'info>::decode_as_type`
    = note: this error originates in the derive macro `scale_decode::DecodeAsType` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: the trait bound `<E as Environment>::Hash: IntoVisitor` is not satisfied
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/client.rs:352:5
    |
352 |     scale_decode::DecodeAsType,
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `IntoVisitor` is not implemented for `<E as Environment>::Hash`
    |
    = note: required for `<E as Environment>::Hash` to implement `DecodeAsType`
note: required by a bound in `subxt::ext::scale_decode::visitor::types::tuple::TupleField::<'scale, 'info>::decode_as_type`
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/scale-decode-0.5.0/src/visitor/types/tuple.rs:128:30
    |
128 |     pub fn decode_as_type<T: DecodeAsType>(&self) -> Result<T, crate::Error> {
    |                              ^^^^^^^^^^^^ required by this bound in `TupleField::<'scale, 'info>::decode_as_type`
    = note: this error originates in the derive macro `scale_decode::DecodeAsType` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: the trait bound `<E as Environment>::Hash: EncodeAsType` is not satisfied
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/client.rs:359:9
    |
359 |     pub code_hash: E::Hash,
    |         ^^^^^^^^^ the trait `EncodeAsType` is not implemented for `<E as Environment>::Hash`
    |
    = note: required for the cast from `&<E as Environment>::Hash` to `&dyn EncodeAsType`

error[E0277]: the trait bound `<E as Environment>::Balance: EncodeAsType` is not satisfied
  --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/xts.rs:85:5
   |
85 |     value: E::Balance,
   |     ^^^^^ the trait `EncodeAsType` is not implemented for `<E as Environment>::Balance`
   |
   = note: required for the cast from `&<E as Environment>::Balance` to `&dyn EncodeAsType`

error[E0277]: the trait bound `<E as Environment>::Balance: EncodeAsType` is not satisfied
  --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/xts.rs:99:5
   |
99 |     value: E::Balance,
   |     ^^^^^ the trait `EncodeAsType` is not implemented for `<E as Environment>::Balance`
   |
   = note: required for the cast from `&<E as Environment>::Balance` to `&dyn EncodeAsType`

error[E0277]: the trait bound `<E as Environment>::AccountId: EncodeAsType` is not satisfied
  --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/xts.rs:97:5
   |
97 |     dest: MultiAddress<E::AccountId, ()>,
   |     ^^^^ the trait `EncodeAsType` is not implemented for `<E as Environment>::AccountId`
   |
   = note: required for `subxt::utils::MultiAddress<<E as Environment>::AccountId, ()>` to implement `EncodeAsType`
   = note: required for the cast from `&subxt::utils::MultiAddress<<E as Environment>::AccountId, ()>` to `&dyn EncodeAsType`

error[E0277]: the trait bound `<E as Environment>::Balance: EncodeAsType` is not satisfied
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/xts.rs:111:5
    |
111 |     value: E::Balance,
    |     ^^^^^ the trait `EncodeAsType` is not implemented for `<E as Environment>::Balance`
    |
    = note: required for the cast from `&<E as Environment>::Balance` to `&dyn EncodeAsType`

error[E0277]: the trait bound `<E as Environment>::Balance: EncodeAsType` is not satisfied
   --> /home/sudip/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ink_e2e-4.2.1/src/xts.rs:149:5
    |
149 |     storage_deposit_limit: Option<E::Balance>,
    |     ^^^^^^^^^^^^^^^^^^^^^ the trait `EncodeAsType` is not implemented for `<E as Environment>::Balance`
    |
    = note: required for `std::option::Option<<E as Environment>::Balance>` to implement `EncodeAsType`
    = note: required for the cast from `&std::option::Option<<E as Environment>::Balance>` to `&dyn EncodeAsType`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `ink_e2e` (lib) due to 11 previous errors
warning: build failed, waiting for other jobs to finish...

cargo test

I can tell this PR is ready for merge unless you have any question


Use a REST integration test driver to create and run tests

There are many libraries and tools out there for testing REST API's. Take a look at what exists. Nice if we can use a swagger specification to drive this. That would mean creating one to describe the existing API. Then we can automate the integration testing with cache nodes etc.

I'm closing this issue. Since that we have a Perl driver, that accomplish the same task. https://github.com/marcoarthur/gorjun-perl-cli


Android <= 5 encryption support

Another update for keystore encryption. Previously encryption wasn't supported for Android 4.3, 4.4 and 5.0. This commit fixes this issue.

Is it gonna be merged?


Docs: Added Solutions to Leetcode 557

Related Issue
Closes #1272
Description
This PR adds the solution to leetcode problem 557 - (Reverse Words in a String III)
Type of PR

[ ] Bug fix
[ ] Feature enhancement
[X] Documentation update
[ ] Security enhancement
[ ] Other (specify): _______________

Screenshots / Videos (if applicable)

Checklist

[X] I have performed a self-review of my code.
[X] I have read and followed the Contribution Guidelines.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have followed the code style guidelines of this project.
[X] I have checked for any existing open issues that my pull request may address.
[X] I have ensured that my changes do not break any existing functionality.
[X] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.
[X] I have read the resources for guidance listed below.
[X] I have followed security best practices in my code changes.

Additional Context
N/A

@Ajay-Dhangar Kindly Review And Merge
Thanks


Desktop map ported to wasm

Desktop map ported to wasm - credit goes to @oblakr24

I'll keep it in my fork


[ISSUE #8177]optimising update topic from name server

https://github.com/apache/rocketmq/issues/8177

Codecov Report
Attention: Patch coverage is 71.42857% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 42.84%. Comparing base (1b42515) to head (9ae2a6b).

Files
Patch %
Lines

...rocketmq/client/impl/factory/MQClientInstance.java
63.63%
2 Missing and 2 partials :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8178      +/-   ##
=============================================
- Coverage      42.90%   42.84%   -0.06%
+ Complexity     10370    10353      -17
=============================================
  Files           1270     1270
  Lines          88721    88715       -6
  Branches       11405    11404       -1
=============================================
- Hits           38062    38008      -54
- Misses         45954    45997      +43
- Partials        4705     4710       +5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

can someone help with code review this


An ID can be present in multiple cliques (CURIE duplication)

A good example is UMLS:C0006050, which is present both as its own clique and is also present in a completely different clique, since they are chemicals and proteins respectively: https://nodenormalization-sri.renci.org/get_normalized_nodes?curie=UMLS%3AC0006050&curie=UNII%3AE211KPY694&conflate=true&drug_chemical_conflate=true
This is because db1 contains the JSON of all the identifiers, so as long as the cliques have different preferred IDs this is fine.
It's probably not possible to fix this without overhauling how the Redis databases in NodeNorm work, but it would be nice to have some sort of index-wide test (#225) to catch when this happens.

As of PR #342, we can use the DuckDB database to generate a list of the 74,886 duplicated CURIEs:
duplicate_curies-2024sep9.csv -- some of these don't look right just yet, but I figured it was worth sharing as a first-shot knowing that the results might change later.
Here's the distribution by the types of Biolink entities being combined:
   1 biolink:CellularComponent||biolink:ComplexMolecularMixture
   1 biolink:ChemicalEntity||biolink:GrossAnatomicalStructure
   1 biolink:ChemicalEntity||biolink:Polypeptide||biolink:Protein
   1 biolink:Disease||biolink:MolecularMixture
   1 biolink:Disease||biolink:OrganismTaxon
   1 biolink:GrossAnatomicalStructure||biolink:SmallMolecule
   1 biolink:MolecularMixture||biolink:OrganismTaxon
   1 biolink:OrganismTaxon||biolink:SmallMolecule
   1 biolink:PhenotypicFeature||biolink:Protein
   2 biolink:AnatomicalEntity||biolink:ChemicalEntity||biolink:Protein
   2 biolink:Cell||biolink:Disease
   2 biolink:ChemicalEntity||biolink:MolecularMixture||biolink:Protein
   3 biolink:CellularComponent||biolink:ChemicalEntity
   3 biolink:Disease||biolink:SmallMolecule
   3 biolink:GrossAnatomicalStructure||biolink:OrganismTaxon
   4 biolink:Cell||biolink:PhenotypicFeature
   4 biolink:ChemicalEntity||biolink:PhenotypicFeature
   4 biolink:GrossAnatomicalStructure||biolink:PhenotypicFeature
   6 biolink:AnatomicalEntity||biolink:ComplexMolecularMixture
   6 biolink:CellularComponent||biolink:Disease
   9 biolink:ChemicalEntity||biolink:Disease
  12 biolink:Cell||biolink:OrganismTaxon
  13 biolink:CellularComponent||biolink:PhenotypicFeature
  13 biolink:Cell||biolink:ChemicalEntity
  14 biolink:Disease||biolink:GrossAnatomicalStructure
  15 biolink:AnatomicalEntity||biolink:ChemicalEntity
  15 biolink:AnatomicalEntity||biolink:Disease
  17 biolink:ComplexMolecularMixture||biolink:Protein
  25 biolink:ChemicalEntity||biolink:Protein||biolink:SmallMolecule
  28 biolink:ChemicalEntity||biolink:OrganismTaxon
  34 biolink:Polypeptide||biolink:Protein
  37 biolink:AnatomicalEntity||biolink:OrganismTaxon
  55 biolink:Disease||biolink:Gene
  56 biolink:AnatomicalEntity||biolink:PhenotypicFeature
  57 biolink:ChemicalEntity||biolink:Drug||biolink:Protein
  81 biolink:MolecularMixture||biolink:Protein
 123 biolink:Drug
 264 biolink:Drug||biolink:Protein
 628 biolink:MacromolecularComplex
 764 biolink:Protein||biolink:SmallMolecule
6600 biolink:Gene||biolink:Protein
65977 biolink:ChemicalEntity||biolink:Protein

Some of these are being duplicated within the DuckDB database (MacromolecularComplex) and aren't actually duplicated in the compendia files, while others are duplicated in the compendia files but are returned correctly by NodeNorm (which I think is because the system is randomly choosing the correct clique to merge on). However, they all represent something wrong that needs to be fixed somewhere.

Next step:

[ ] Get some examples, talk to Biolink people about what exactly we want this kind of thing to be: proteins or chemicals.
[ ] The easiest fix would be if we can remove something from one or the other processing (i.e. exclude proteins when processing chemicals, exclude chemicals when processing proteins).

Note that there might be e.g. PubChem identifiers that aren't connected to other proteins, we would need to get that information into protein, either directly from the PubChem download or indirectly via the chemical code.

[ ] The quick-and-dirty approach would be to come up with some sort of post-processing on the compendium files (there are after all only 33k cliques that need figuring out), but that would be inelegant and possibly vulnerable to other changes down the line. More investigation up front would be useful.


Remove hash from Angular 2 routes

Thanks for all your hardwork on the theme!
Had a quick question: is it possible to remove the hash from the angular routes?  I have tried removing the HashLocationStrategy in the app module providers.

Hi @yorkstreetlabs
I'm working on upgrading the Angular version for this theme, so I'll check if it's possible to give an option to remove the hash for the URI on the next version.
Thanks!

@rluders Thanks sir.  I was able to get the hash removed by removing the HashLocationStrategy for the RouterModule so it defaults to using path.  The issue I then faced was loading from a deep-linked route.  It seemed the OctoberCMS handled it and was returning a 404 page.  I added a 404 page template to the /pages folder matching what was in the home.htm and it is loading ng.  The browser still gets a 404 response, so I'm guessing it's not a solid solution.

Just to add a few notes here:
I give it a try, and well... it seems to be easy to change the Angular's configuration to support non-hash URL, but it seems that we need to change something into the .htaccess as well... Since I'm not a .htaccess expert, I'll probably need some help here or I can try it later again.
I would prefer to not move the app access point to a subfolder, but it could be a possibility, but in this case, we should also do the same for hashed URLs to keep it consistent. Does anybody have another idea?

I've been using it without hash on my project since I migrated to Angular 6 internally, but for the life of me, I can't remember what I did to remove hash :)
And I know it hasn't got anything to do with .htaccess because I've been using Apache in local, and Nginx in production. And it worked perfectly without hash.

That is odd. It doesn't work for me at all. Maybe I do have some weird Apache configuration. I'll give another try just to make sure. In my case, every time that I try to access the Angular pages without the hash url-based, I got the OctoberCMS 404 page.

How I have my production app configured is quite different than your theme.
I have an app.htm layout for the Angular app.
This layout file contains the actual <app-root> component, just like your theme.
However, I have an angular-app.htm page in October like this:
title = "My site title"
url = "/app/:app_route/:itemId?/:custom?"
layout = "app"
description = "Angular 7 Application"
is_hidden = 0

[TestPlayer]
[InteractiveTest]
[lessonPlayer]
[MixedLessonPlayer]
[AngularHelper]
[session]
security = "user"
redirect = 404
==

This page I use to load all the necessary October components, as you see above the == line.
The trick is, I think in this page's url.
This is where the Angular router catches on.
Here's my Angular router:
import { AppComponent } from './app.component';
import { NgModule } from '@angular/core';
import { Routes, RouterModule } from '@angular/router';

// All routes prefixed with app/ load the /pages/angular-app.html from the October theme

const routes: Routes = [
  {
    path: 'app/test/:code',
    // component: TestComponent,
    loadChildren: './test/test.module#TestModule'
    // loadChildren: () => TestModule
    /* pathMatch: 'full' */
  },
  {
    path: 'app/school-test/:code',
    loadChildren: './school-test/school-test.module#SchoolTestModule'
  },
  {
    path: 'app/school-test/:code/:custom',
    loadChildren: './school-test/school-test.module#SchoolTestModule'
  },
  {
    path: 'app/interactive-test',
    loadChildren: './interactive-test/interactive-test.module#InteractiveTestModule'
  },
  {
    path: 'app/lesson/:code',
    loadChildren: './lesson/lesson.module#LessonModule'
  },
  {
    path: 'app/mixed-lesson/:code',
    loadChildren: './mixed-lesson/mixed-lesson.module#MixedLessonModule'
  },
  {
    path: 'custom-test', // loads custom-test.htm from the October theme /pages
    loadChildren: './custom-test/custom-test.module#CustomTestModule'
  },
  {
    path: 'custom-lesson', // loads custom-lesson.htm from the October theme /pages
    loadChildren: './custom-lesson/custom-lesson.module#CustomLessonModule'
  },
  {
    path: 'free-test',  // loads free-test.htm from the October theme /pages
    loadChildren: './test/test.module#TestModule'
  },
  {
    path: '',
    component: AppComponent,
    pathMatch: 'full'
  }
];

@NgModule({
  imports: [RouterModule.forRoot(routes)],
  exports: [RouterModule]
})
export class AppRoutingModule { }

You will note two routes that avoid the /app prefix, for example I have this custom-test.htm page in October theme.
title = "Custom Test Composer"
url = "/custom-test"
layout = "default"
description = "Page to compose custom tests by schools"
is_hidden = 0

[Gatekeeper]
[TestComposer]
[AngularHelper]
[session]
security = "user"
redirect = 404
==

See, here the url matches the url in Angular's router.

One more difference in my production app is that I'm not using JWT auth for communicating with the October backend.
But I have written an Angular service to communicate with the backend using October's AJAX API.
Here it is:

import { HttpClient, HttpHeaders, HttpParams } from '@angular/common/http';
import { Injectable } from '@angular/core';
import { Subject } from 'rxjs';
import { isObject } from 'util';

// this is a global variable which should be picked up from PHP server and already defined in the page
declare const themeUrl: string;
declare const initData: any;

/**
 * This is an attempt to replicate October's `request` function which executes Ajax requests to October's component's methods
 * source parts from:
 * https://github.com/octobercms/october/blob/master/modules/system/assets/js/framework.js
 *
 * The service calls October's Ajax handler. The page where this call is executed needs to have an October Component included which
 * has the handler implemented (handlers beginning with `on`, e.g. `onStartTest`)
 */

@Injectable()
export class OctoberService {

  options = {};

  themeUrl: string;

  initData: any;

  constructor(private http: HttpClient) {
    this.themeUrl = themeUrl;
    this.initData = initData;

    /* this.http.post(window.location.href, new HttpParams({ fromObject: {} }), {
      headers: new HttpHeaders({
        'Content-Type':  'application/x-www-form-urlencoded; charset=UTF-8',
        'X-OCTOBER-REQUEST-HANDLER': 'onGetThemeUrl',
        'X-OCTOBER-REQUEST-PARTIALS': '',
        'X-Requested-With': 'XMLHttpRequest'
      })
    }).subscribe(data => {
      //this.themeUrl = data.url;
      this.themeUrl.next(data.url);
      console.log('url bre ' + data.url)
    }); */
  }

  public request(handler, options) {
    const url = window.location.href;
    this.options = options || {};
    let data = {};
    /*
    * Validate handler name
    */
    if (handler === undefined) {
      throw new Error('The request handler name is not specified.');
    }

    if (!handler.match(/^(?:\w+\:{2})?on*/)) {
      throw new Error('Invalid handler name. The correct handler name format is: "onEvent".');
    }

    const loading = options.loading !== undefined ? options.loading : null,
      isRedirect = options.redirect !== undefined && options.redirect.length,
      useFlash = options.flash !== undefined,
      useFiles = options.files !== undefined;

    /*
    * Request headers
    */

    const httpOptions = {
      // 'Content-Type': 'application/x-www-form-urlencoded; charset=UTF-8',
      // ^^ removing this from headers b/c Angular automatically adds when FormData passed
      headers: new HttpHeaders({
        'X-OCTOBER-REQUEST-HANDLER': handler,
        // 'X-OCTOBER-REQUEST-PARTIALS': this.extractPartials(options.update),
        'X-Requested-With': 'XMLHttpRequest'
      })
    };

    if (useFlash) {
      httpOptions.headers.append('X-OCTOBER-REQUEST-FLASH', '1');
    }

    if (options.data !== undefined && !Object.keys(options.data).length) {
      data = { data, ...options.data };
    }
    // const httpParams = new HttpParams({ fromObject: options.data });    // https://stackoverflow.com/a/47928379
    const fd = this.createFormData(options.data);
    return this.http.post(url, fd, httpOptions);
  }

  // This function is breaking at Angular 7, so I commented it out at line 76 above
  private extractPartials(update) {
    const result = [];

    for (const partial of update) {
      result.push(partial);
    }
    return result.join('&');
  }

  // snippet from https://gist.github.com/ghinda/8442a57f22099bdb2e34#gistcomment-1997213
  // consider later posts about adding compatibility with nested Arrays
  // commented out the part of first `if` clause because it was ignoring `false` booleans.
  createFormData(object: Object, form?: FormData, namespace?: string): FormData {
    const formData = form || new FormData();
    for (const property in object) {
        if (!object.hasOwnProperty(property)/*  || !object[property] */) {
            continue;
        }
        const formKey = namespace ? `${namespace}[${property}]` : property;
        if (object[property] instanceof Date) {
            formData.append(formKey, object[property].toISOString());
        } else if (typeof object[property] === 'object' && !(object[property] instanceof File)) {
            this.createFormData(object[property], formData, formKey);
        } else {
            formData.append(formKey, object[property]);
        }
    }
    return formData;
  }

}

The idea is from the old AngularJS bridge developed by daftspunk. However I wasn't really successful in implementing the full functionality. It just does the job for me as it is in the above code.


Enable SerializeFeature.MapSortField for deterministic order

Similar to the change you already merged for JSONPParseTest2.java:
test_f in JSONPParseTest3 parses a text to a JSONPObject and compares its toJSONString representation to a hard-coded string. However, JSONPObject uses a HashMap that does not guarantee any specific order of entries. Therefore, the assertion in the test can fail if the order differs.
This PR uses SerializerFeature.MapSortField to make the order deterministic and appropriately modifies the corresponding test assertion.

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

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

Codecov Report

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

@@             Coverage Diff              @@
##             master    #3011      +/-   ##
============================================
- Coverage     80.58%   80.56%   -0.03%
+ Complexity     7234     7228       -6
============================================
  Files           145      145
  Lines         27106    27106
  Branches       6490     6490
============================================
- Hits          21844    21838       -6
- Misses         3383     3386       +3
- Partials       1879     1882       +3

Impacted Files
Coverage Δ
Complexity Δ

...com/alibaba/fastjson/parser/JSONReaderScanner.java
72.61% <0%> (-1.28%)
44% <0%> (-1%)

...main/java/com/alibaba/fastjson/util/FieldInfo.java
86.2% <0%> (-0.35%)
91% <0%> (ø)

...n/java/com/alibaba/fastjson/util/JavaBeanInfo.java
87.6% <0%> (-0.17%)
206% <0%> (-2%)

...java/com/alibaba/fastjson/parser/ParserConfig.java
83.41% <0%> (-0.17%)
206% <0%> (-1%)

...ava/com/alibaba/fastjson/parser/JSONLexerBase.java
71.35% <0%> (-0.04%)
828% <0%> (-1%)

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 97ee7b6...6d8a032. Read the comment docs.


Build in encode session limits

You may not want the script to encode all day, or to only process a number of files before stopping. Limits could include:

session encode limit (ex. 10 encodes per script execution)
session time limit (ex. Stop encodes if between "active" hours ie 8am to 11pm)

Will complete with dev branch


enable the background for any html element, not just for body

When I use vegas, I need to put the background image for a div but not for full screen.
So just change the code.
Now you can pass a jquery selector for backgroundTarget like this:
$.vegas({
src: 'http://your.image.url',
backgroundTarget: '.background'
})
then, the image will just as a background for the element which has a class call "background"

Thanks for the contribution.
I close this PR. V2 is on its way.


[Fix](SAP Hana External Table) fix that SAP Hana external table can not insert batch values

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 36.38 seconds
stream load tsv:          454 seconds loaded<PHONE_NUMBER>9 Bytes, about 157 MB/s
stream load json:         24 seconds loaded<PHONE_NUMBER> Bytes, about 93 MB/s
stream load orc:          73 seconds loaded<PHONE_NUMBER> Bytes, about 14 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230320103100_clickbench_pr_117286.html

run buildall

run buildall

run buildall

run buildall

run p0

run buildall


Setup script

Related to #7 . This creates a setup.py script so that a user can do "python setup.py install" in order to install the script as well as it's dependencies. I had to change the directory hiearchy some for this to work smoothly, although there could be a way to keep it the same with the right commands in setup.py. It also creates a link to the script so someone can just call "alchemical_analysis " from the command line. This would also be the first step in putting the package in PyPI (pymbar is there too). There are several things you should change in setup.py as far as the metadata (version number, etc.).

Thanks. I'll review this soon - I'd not created an issue yet for this, but this is something I had wanted to do, as I'd run into the same problem. Longer-term, I also want to make a conda recipe for it. We very much appreciate the help with this - above and beyond the call of duty, from my perspective. Pavel is on a time crunch towards graduating so he doesn't have much time to contribute right now.

Hey not a problem. I'm happy to contribute when possible. I saw the "help wanted" for #7 so thought I would give it a shot. This does need more thorough testing as just today I ran it with some different flags and it gave an error that a variable wasn't global (I'll add to the pull request later for this case). It has to do with creating that main() function so that variables inside it are no longer seen as global. I'm not sure if there's a better way to do that other than tracking down all the variables that need to be global and explicitly listing all of them at the beginning. I made the main() function for the "entry_point" feature of setup.py.

I tried all options which apply to my dataset and didn't run into any further global variable problems, so I think this is ready to merge if you don't have any other outstanding issues. LMK. (I did also in the process find a bug with the overlap matrix and created a separate issue for it).

The only other thing you might want to do before merging is double-check / update the metadata in setup.py (especially version number).


[Snyk] Security upgrade github-pages from 225 to 225

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `rubygems` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

docs/Gemfile

⚠ Warning
Failed to update the Gemfile.lock, please update manually before merging.

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

558/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 3.3
Out-of-bounds Read SNYK-RUBY-COMMONMARKER-3318398
No
Proof of Concept

461/1000  Why? Recently disclosed, Has a fix available, CVSS 3.5
Denial of Service (DoS) SNYK-RUBY-COMMONMARKER-3318399
No
No Known Exploit

668/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 5.5
Denial of Service (DoS) SNYK-RUBY-COMMONMARKER-3318400
No
Proof of Concept

461/1000  Why? Recently disclosed, Has a fix available, CVSS 3.5
Denial of Service (DoS) SNYK-RUBY-COMMONMARKER-3318401
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)
🦉 Denial of Service (DoS)
🦉 Denial of Service (DoS)

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


varname is availible outside if brackets not just inside the if sentence

varname is availible outside if brackets not just inside the if sentence, it works as "as" keyword in the scope

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

ID: 3126aaa9-8f5f-04f3-2d75-96477b6273cd
Version Independent ID: 3f4b3eac-11eb-2dd6-4f27-b24537367288
Content: is - C# Reference
Content Source: docs/csharp/language-reference/keywords/is.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

This was fixed in #11972


Add API to return only specified columns when fetching records

I mean like this
const columns = [
    'id',
    'title',
    'body',
];
const posts = Post.query(columns).all()

Is it possible? I tried adding the array to query(), to get(), to all(), to select(), to columns()
It doesn't look like this exists right now?

@vesper8 Thank you for the feedback! Currently you can't do this. But, the things is Vuex ORM is accessing JS object directly in Vuex Store, so you might not need this feature? I mean it doesn't change much on performance perspective.
Could you elaborate a bit more on your use case?

@vesper8 Just checking in. Is there any update? 😃

I can't really tell if performance is an issue. My Post model contains quite a lot of columns and in some cases I only want to interact with / display a small subset.. say 1 or 2 columns out of 15+
I just figured that by filling up posts with all that date and looping through it when I'm only using a small part, might be bad for performance.
Probably barely noticeable if at all, just would be a nice-to-have in the future since selecting which fields are queried is pretty common in all ORMs.
You can close this for now since there is no current implementation. Thanks for the follow-up

@vesper8 Hi! thank you for the confirmation.
You might be right! Not sure if there's going to be huge performance improvement, but I too agree that in some cases it might. Also as you mentioned, this is the pretty common feature for the ordinary ORM so I think it's nice to have.
I'll keep this issue open as an enhancement and edit the title so maybe someone can tackle this issue.

hello, will this be implemented?
I am working with vuex orm and this functionality would be very useful...

How are you managing the data from the store? I mean, do they get copied around or what we get are references to objects in the store? In that case it might be better to leave this kind of filtering at the end of the processing sequence or in user code.
However, as usual in Javascript, we end up writing boilerplate code to do obvious things that then we need to store somewhere, make more general, debug & test... so yeah, a simple function like .values() or .values_list() as used in Django's ORM would be nice to have.


Better display MultiError when errors are the same

Before:
Failed to renew TLS certificates: please check if a newer CodeReady Containers release is available: Temporary Error: No Pending CSR
Temporary Error: No Pending CSR
Temporary Error: No Pending CSR
Temporary Error: No Pending CSR
Temporary Error: No Pending CSR
Temporary Error: No Pending CSR
Temporary Error: No Pending CSR
Temporary Error: No Pending CSR
Temporary Error: No Pending CSR
...
my console is full :)

After
Failed to renew TLS certificates: please check if a newer CodeReady Containers release is available: Temporary error: No Pending CSR (x37)
Temporary error: ssh command error:
command : oc get csr --context admin --cluster crc --kubeconfig /opt/kubeconfig
err     : exit status 1
output  : The connection to the server api.crc.testing:6443 was refused - did you specify the right host or port?
 (x1)
Temporary error: No Pending CSR (x82)

Slightly better as it fits on one screen.

/retest


[SERF-1675] http server timeouts configuration

Description
inclusion of idle/write/read timeouts to /pkg/server.Config
applications using this sdk now are allowed to use mentioned timeouts through config file.
Checklist

[x] Prefixed the PR title with the JIRA ticket code
[x] Performed simple, atomic commits with good commit messages
[x] Verified that the commit history is linear and commits are squashed as necessary
[x] Thoroughly tested the changes in development and/or staging

Related links

SERF-1675

@laynax Also, we should be considering updating the README documentation by adding new configuration parameters there.


Clarifies where to put the hook to run it once

I have to admit it is a bit strange behavior that by default it runs 2 times.
I think it is better to clarify it.

Thanks! Sorry for the looooong delay before merging


Add (additional) LSP commands to Command Palette

Check for existing issues

[X] Completed

Describe the feature
While using the Zed Editor, I noticed that it does not support some additional LSP (Language Server Protocol) commands such as macro expansion and workspace refresh. These commands can improve the functionality and user experience of the editor.
I would like to request the addition of these additional LSP commands to the command input panel of the Zed Editor for easy access by users. I have searched for related issues, but have not found any similar requests. If someone finds a similar issue, please close this issue.
Proposed implementation:

Add the additional LSP commands to the Zed Editor's command input panel based on cmd + p for easy access by users.

Support common LSP commands such as macro expansion, workspace refresh, etc.

Ensure that the added functionality is compatible with the existing functionality of the Zed Editor and does not affect the performance of the editor.

Thank you for your consideration.
If applicable, add mockups / screenshots to help present your vision of the feature

Rust macro expansion is available in v0.117.0-pre, as of today.


How to generate a repo URL from android project on github

I have a working project forked from a friend of mine, but i want the gradle url to be from my own forked one

Sorry, I‘m not sure what you mean. Do you want to host an artifact repository on GitHub pages?

For instance, i have this library forked 'com.facebook.shimmer:shimmer:0.3.0'
i want it to be as 'com.nusktec.shimmer:shimmer:0.3.0'

Please refer to the user guide chapter on publishing artifacts.


Create a drill-down example where a technology is studied at two levels

Level 1: Broadly - where we study the main components of a technology
Level 2: After identification of one component, we drill down to that and create another case study focused on the details of that.
Can this procedure be generalized for ease of creation of datasets?

@tjlca I've found the point in the repo history where we did this to the template decision context. The first commit with the drilldown is https://github.com/NREL/tyche/tree/f667b3a3e0fdcf9632992aa7c07e57cc1bc540e9; previous commits had the original less detailed version. (Component B became Components B0, B1, and B2, and there were associated additional parameters added.)
The datasets before and after drilldown are attached - they can't be used directly with the current codebase but it gives you an idea of how the extra detail was added.
template-original1.xlsx
template-drilldown2.xlsx
As a side note, I doubt this process is something we can generalize programmatically (i.e. automatically) but we can probably create some guidelines on data management, when to drill down/when not to, and related topics.


Compodoc / TypeDoc / Nothing ?

@vogloblinsky Dropped a comment in another issue in regards to switching from TypeDoc to Compodoc and was kind enough to spin up a demo - https://compodoc.github.io/angular2-webpack-starter/
The options are ...

Keep TypeDoc.
Switch to Compodoc.
Drop doc generation and let users add what they will.

Personally I don't run docs on my applications currently, though Compodoc is the first offering that has the potential to drive enough value to warrant the addition.
//cc @colinskow @katallaxie @shlomiassaf

I Don't have much to contribute on this...
I saw the product that TypeDoc produces, didn't seem to be impressive.
But this is just theming...
The Compodoc looks better, I must admit.
If we consider things that does not belong to the starter, docs can easily fit in...
That said, promoting a doc system might help people document their OS code so it's good for the community.
I'm on board :)
If I need to choose, I would go with Compodoc, that's totally intuition nothing to back it up with :)

I don't generate docs at the moment, so I don't have an opinion here. But it is definitely something useful to have a demo of in the repo, and I may consider using it if there is a good example.

Currently TypeDoc is part of the project and I know at least some people use it. It has consistently had maintenance issues so we either need to remove it and let people manage their own docs or switch to Compodoc.
Personally I'm leaning towards the latter, we need a good doc generation tool and it's by far the best to come down the pipe. Can't hurt to support a new lib, particularly one with as much promise as it holds for utility.

@d3viant0ne
Yes compodoc is at the beginning, with some features which needs to be enhanced.
The target release 0.1.0 will add the last missing features for a matured tool.
I will update this discussion when it will be ready.

Wait for the following Compodoc's roadmap point to be done:

handle external markdown files as "functional" documentation

For now my modular app has some "library modules" that i get via npm, and store in "node_modules/@my-project-scope/some-isolated-functionality-module"
Unfortunately compodoc cant include documentation for them and i have only documentation for my app skeleton

@DyckGerman - I'm in the same boat there. I do much the same thing for my ui platform at work and have run into the same issue.

Hi @d3viant0ne & @DyckGerman,
Version 1.0.0 of compodoc just landed yesterday. https://github.com/compodoc/compodoc/blob/master/CHANGELOG.md#100-beta1-2017-04-03
There are great new features :

routing graph
component DOM tree graph
watch flag
etc...
and also the one @DyckGerman was waiting, additional documentation.

Have a look, and feedbacks are always welcome 😄

Any news guys about the choice ?

@gdi2290 Just saw you close this issue.
I have generate an example of outputed documentation for angular-starter if you want to test what's Compodoc can generate : https://compodoc.github.io/angular-starter/

Nice, yeah we should use it

@gdi2290 PR created  : https://github.com/AngularClass/angular-starter/pull/1877


how to track clicks,opens ?

hi
@k-pom @eddywashere @eddywashere @atarola @amatai
any of you guys pls tell how to track clicks and opens of the mail for the following nodejs code?. i use html as front end.
app.post('/', function(req, res) {
    var api_key = 'key-XXXXXxxxxxxxxXXXXXX';
    var domain = 'domain.in';
    var Mailgun = require('mailgun-js');
    var mailgun = new Mailgun({ apiKey: api_key, domain: domain });

    var data = {
        from<EMAIL_ADDRESS>        to<EMAIL_ADDRESS>
        subject: req.body.subject,
        text: req.body.plaintext,
        'o:tag': req.body.tag
    };
    console.log(req.body);
    mailgun.messages().send(data, function(error, body) {
        console.log(body);
        //Email not sent
        if (error) {
            res.render('index', { title: 'No Email', msg: 'faild.', err: true })
        }
        else {
            res.render('index', { title: 'Sent Email', msg: 'success.', err: false })
        }
    });
});

Just enable tracking in mailgun.com UI. Go to messages -> domains -> your domain  -> settings and enable tracking there.

Closing the issue due to lack of activity from the author. If you still have some issues please open a new one.


Fix MessageIndex determinism

Description
values stored in MessageIndex should be ordered by message_id, currently they are not
Motivation
required for resolution
To-do list

[ ] Ensure values added to MessageIndex are sorted by T::message_id

[ ] Replace scope.insert with equivalent binary search

Change checklist

[ ] The feature or fix is implemented in Rust and across all bindings whereas possible.
[ ] The feature or fix has sufficient testing coverage
[ ] All tests and examples build and run locally as expected
[ ] Every piece of code has been document according to the documentation guidelines.
[ ] If conceptual documentation (mdbook) and examples highlighting the feature exist, they are properly updated.
[ ] If the feature is not currently documented, a documentation task Issue has been opened to address this.

values stored in MessageIndex should be ordered by message_id, currently they are not
To-do list
  * [ ]  Replace `scope.insert` with equivalent binary search

I'm confused where you want the messages ordered. This is the current impl:
https://github.com/iotaledger/identity.rs/blob/97cfa59a334aac58ba3fb67e3061e01e1b6938f4/identity-iota/src/tangle/message_index.rs#L13-L18
The scope you are referring to is the inner Vec, ordering by message_id should hardly make any sense here (since all values in that vector should correspond to the same message_id)?
When changing the HashMap to BTreeMap

values stored in MessageIndex should be ordered by message_id

so that's what I did in #213

The scope you are referring to is the inner Vec, ordering by message_id should hardly make any sense here (since all values in that vector should correspond to the same message_id)?

The key for the map is previous_message_id and message_id is expected to be unique for all values (T has message_id and previous_message_id methods through the TangleRef trait).
Something like this:
map(previous_message_id => [(message_id, data), ...])

If there are multiple messages with the same previous_message_id they will just be prepended here:
https://github.com/iotaledger/identity.rs/blob/37d3996ba2ea4874f65b923a9244fa15c26acc56/identity-iota/src/tangle/message_index.rs#L51
and this is where ordering by message_id should happen


ci: Add nightly builds

This should allow us to get ahead of any gem incompatabilities

@kaylareopelle I deleted the canary workflows in this PR: https://github.com/open-telemetry/opentelemetry-ruby-contrib/pull/947

@kaylareopelle I deleted the canary workflows in this PR: #947

@arielvalentin Thanks, I missed that PR! What about this workflow? https://github.com/open-telemetry/opentelemetry-ruby-contrib/blob/main/.github/workflows/ci-contrib-canary.yml

UGG I missed that one huh

All good! Thanks for taking it out!


Create functional proteins from a given protein Template/Profile?

Thank you for developing Chroma. Chroma is such an excellent generative AI tool for de novo protein design.
I would like to inquire about how one can use Chroma to design new functional proteins (not just substructures), for instance:

What is the process for leveraging Chroma to create proteins inspired by the functions of existing enzymes? Could one, for example, utilize a protein template—such as a specific domain extracted from a PDB file, a MSA file, a pre-trained HMM profile etc. —for guidance in this design process?

A point of note: In given demo file, it appears that the CATH inputs are restricted to three levels (e.g., "3.30.40"), yet four levels could potentially be more beneficial for the design of a functional protein? Is there a way to incorporate this additional level of detail in Chroma's design process?

In the context of scaffold design, is there a method within Chroma to designate certain critical amino acid residues as invariant hotspots or a interface? ensuring they remain unaltered throughout the design process?

Hey @Travis13197, for question 2 you can use the selection language, here's a minimal example:
chroma = Chroma()
protein = Protein("5SV5", device="cuda")
chroma.design(protein, design_selection="not resid 1-10") # keeps residues 1-10 fixed

You can do the same thing with conditioners. https://github.com/generatebio/chroma/issues/37 has a discussion of a few methods.
For question 1, it's up to you to create your own classifier to guide Chroma, you could start from their CATH classifier and modify it. It should work as long as it follows the conditioner template/interface.


Not very helpful samples

You just forgot to mention where to write  animation code, how to apply it it to an object.
This way it is just about useless.

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

ID: 5fb76e26-b365-76c2-6a8c-c35c03e7fb2b
Version Independent ID: 9cefc858-4798-c05f-f7ae-7dedf7e5522a
Content: Simple Animations in Xamarin.Forms - Xamarin
Content Source: docs/xamarin-forms/user-interface/animation/simple.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Microsoft support for Xamarin will end on May 1, 2024 for all Xamarin SDKs including Xamarin.Forms. In preparation for this, all issues and PRs in this repository are being closed.
.NET MAUI is the replacement for Xamarin.Forms, and it's documentation can be found here.


array-type doesn't consider "this" to be a simple type

Bug Report

TSLint version: 5.2.0
TypeScript version: 2.2.1
Running TSLint via: CLI

TypeScript code being linted
public children: this[];

with tslint.json configuration:
{
    "extends": "tslint:recommended",
    "array-type": [true, "array-simple"]
}

Actual behavior
ERROR: test.ts[x, y]: Array type using 'T[]' is forbidden for non-simple types. Use 'Array' instead.
Expected behavior
No warnings reported, since this should be considered a simple type along with e.g. object, number and string. See https://github.com/palantir/tslint/issues/2351

agreed, this should be a really simple PR for anyone looking to contribute!


Accessory HomeKit

hello, there is the possibility of creating an accessory for homekit without the need for a homebridge, using the homekit libraries https://github.com/maximkulkin/esp-homekit  from @MaximKulkin for https://github.com/SuperHouse/esp-open-rtos . And it uses the OTA update system Life-Cycle-Manager (LCM) https://github.com/HomeACcessoryKid/life-cycle-manager from @HomeACessoryKid https://github.com/HomeACcessoryKid, just like ravenSystem does https://github.com/RavenSystem/esp-homekit-devices

Unfortunately this code is not written in RTOS manner. I dont have the expertise to do this, don't know about @toblum. It looks like a wontfix label. You can use a $5 RPI0 to emulate a Homebridge. You can use something like https://www.npmjs.com/package/homekit2mqtt to emulate a homebridge based on MQTT topics. I used to use it until HomeAssistant got Homekit plugin.

Hi @arturodiazad1,
this functionality would be great to have, but I'm afraid that I'm also not able to rewrite that code so that it fits into McLighting.
But have a look at homekit2mqtt or homebridge. That works quits good.
Regards
Tobias


fix: Start LLMQContext early to let VerifyDB() check ChainLock signatures in coinbase

Issue being fixed or feature implemented
Now that we have ChainLock sigs in coinbase VerifyDB() have to process them. It works most of the time because usually we simply read contributions from quorum db https://github.com/dashpay/dash/blob/develop/src/llmq/quorums.cpp#L385. However, sometimes these contributions aren't available so we try to re-build them https://github.com/dashpay/dash/blob/develop/src/llmq/quorums.cpp#L388. But by the time we call VerifyDB() bls worker threads aren't started yet, so we keep pushing jobs into worker's queue but it can't do anything and it halts everything.
backtrace:
  * frame #0: 0x00007fdd85a2873d libc.so.6`syscall at syscall.S:38
    frame #1: 0x0000555c41152921 dashd_testnet`std::__atomic_futex_unsigned_base::_M_futex_wait_until(unsigned int*, unsigned int, bool, std::chrono::duration<long, std::ratio<1l, 1l> >, std::chrono::duration<long, std::ratio<1l,<PHONE_NUMBER>l> >) + 225
    frame #2: 0x0000555c40e22bd2 dashd_testnet`CBLSWorker::BuildQuorumVerificationVector(Span<std::shared_ptr<std::vector<CBLSPublicKey, std::allocator<CBLSPublicKey> > > >, bool) at atomic_futex.h:102:36
    frame #3: 0x0000555c40d35567 dashd_testnet`llmq::CQuorumManager::BuildQuorumContributions(std::unique_ptr<llmq::CFinalCommitment, std::default_delete<llmq::CFinalCommitment> > const&, std::shared_ptr<llmq::CQuorum> const&) const at quorums.cpp:419:65
    frame #4: 0x0000555c40d3b9d1 dashd_testnet`llmq::CQuorumManager::BuildQuorumFromCommitment(Consensus::LLMQType, gsl::not_null<CBlockIndex const*>) const at quorums.cpp:388:37
    frame #5: 0x0000555c40d3c415 dashd_testnet`llmq::CQuorumManager::GetQuorum(Consensus::LLMQType, gsl::not_null<CBlockIndex const*>) const at quorums.cpp:588:37
    frame #6: 0x0000555c40d406a9 dashd_testnet`llmq::CQuorumManager::ScanQuorums(Consensus::LLMQType, CBlockIndex const*, unsigned long) const at quorums.cpp:545:64
    frame #7: 0x0000555c40937629 dashd_testnet`llmq::CSigningManager::SelectQuorumForSigning(Consensus::LLMQParams const&, llmq::CQuorumManager const&, uint256 const&, int, int) at signing.cpp:1038:90
    frame #8: 0x0000555c40937d34 dashd_testnet`llmq::CSigningManager::VerifyRecoveredSig(Consensus::LLMQType, llmq::CQuorumManager const&, int, uint256 const&, uint256 const&, CBLSSignature const&, int) at signing.cpp:1061:113
    frame #9: 0x0000555c408e2d43 dashd_testnet`llmq::CChainLocksHandler::VerifyChainLock(llmq::CChainLockSig const&) const at chainlocks.cpp:559:53
    frame #10: 0x0000555c40c8b09e dashd_testnet`CheckCbTxBestChainlock(CBlock const&, CBlockIndex const*, llmq::CChainLocksHandler const&, BlockValidationState&) at cbtx.cpp:368:47
    frame #11: 0x0000555c40cf75db dashd_testnet`ProcessSpecialTxsInBlock(CBlock const&, CBlockIndex const*, CMNHFManager&, llmq::CQuorumBlockProcessor&, llmq::CChainLocksHandler const&, Consensus::Params const&, CCoinsViewCache const&, bool, bool, BlockValidationState&, std::optional<MNListUpdates>&) at specialtxman.cpp:202:60
    frame #12: 0x0000555c40c00a47 dashd_testnet`CChainState::ConnectBlock(CBlock const&, BlockValidationState&, CBlockIndex*, CCoinsViewCache&, bool) at validation.cpp:2179:34
    frame #13: 0x0000555c40c0e593 dashd_testnet`CVerifyDB::VerifyDB(CChainState&, CChainParams const&, CCoinsView&, CEvoDB&, int, int) at validation.cpp:4789:41
    frame #14: 0x0000555c40851627 dashd_testnet`AppInitMain(std::variant<std::nullopt_t, std::reference_wrapper<NodeContext>, std::reference_wrapper<WalletContext>, std::reference_wrapper<CTxMemPool>, std::reference_wrapper<ChainstateManager>, std::reference_wrapper<CBlockPolicyEstimator>, std::reference_wrapper<LLMQContext> > const&, NodeContext&, interfaces::BlockAndHeaderTipInfo*) at init.cpp:2098:50
    frame #15: 0x0000555c4082fe11 dashd_testnet`AppInit(int, char**) at bitcoind.cpp:145:54
    frame #16: 0x0000555c40823c64 dashd_testnet`main at bitcoind.cpp:173:20
    frame #17: 0x00007fdd85934083 libc.so.6`__libc_start_main(main=(dashd_testnet`main at bitcoind.cpp:160:1), argc=3, argv=0x00007ffcb8ca5b88, init=<unavailable>, fini=<unavailable>, rtld_fini=<unavailable>, stack_end=0x00007ffcb8ca5b78) at libc-start.c:308:16
    frame #18: 0x0000555c4082f27e dashd_testnet`_start + 46

Fixes #5741
What was done?
Start LLMQContext early. Alternative solution could be moving bls worker Start/Stop into llmq context ctor/dtor.
How Has This Been Tested?
I had a node with that issue. This patch fixed it.
Breaking Changes
Not sure, hopefully none.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

CI looks good, pls re-review


Why those slices don't pay attention to time zone when they display?

superset==0.15.4
Why those slices don't pay attention to time zone when they display?

Notice: this issue has been closed because it has been inactive for 411 days. Feel free to comment and request for this issue to be reopened.


Thanks for creating this library!

Thanks for creating this library! Feel free to contact me if you have any questions about Valibot. I am happy to help.

Thank you for valibot :)
It's ok I think this library pretty much managed to reach its goal to convert the part from valibot that would make sense in JSON schema.
And I even got help for the migration to valibot 0.31 RC in #57 !
I'll probably change the versioning of the lib to follow the valibot version number (at least the major and minor part of the version) once 0.31 gets out of RC 👍


Add a timeout parameter to the Orquesta runner

Almost all action runners have a timeout parameter (see the fancy table here), but there is no timeout parameter for Orquesta.
This means that there is no way to set a time limit a subworkflow if something goes wrong.
It would be nice to allow users to specify this parameter as a fail safe in case the Orquesta engine hangs, for example. It would also be more consistent with our other action runners:
tasks:
  do_thing:
    action: mypack.myworkflow  # Orquesta workflow, may take awhile to finish, and/or hang
    input:
      timeout: 60  # seconds

This has been requested by multiple people in the StackStorm Forum.

is this solved?


Patch 1

Fixed the issue on "Change Theme" section for all the views in examples as it is currently showing "[object Object]".

@mayank091193 Thanks for the PR, but I already fixed this yesterday with the exact same updates as you have.


WIP: use chainguard's static (distroless v2) for scorecard image

Signed-off-by: Batuhan Apaydın<EMAIL_ADDRESS>What kind of change does this PR introduce?
This feature is about using the chainguard's static image instead of Google's static. I removed a few things from Dockerfile to simplify the process, as I saw that we don't need them.

[x] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
What is the new behavior (if this is a feature change)?**

[ ] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes

Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

I marked it as WIP because I can replace all other Dockerfile's base images with Chainguard's static image and do some cleanups☝ PTAL @dlorenc @naveensrinivasan

Kindly ping @spencerschrock @naveensrinivasan, folx, do you want me to continue to replace the base images with the chainguard ones?

Kindly ping @spencerschrock @naveensrinivasan, folx, do you want me to continue to replace the base images with the chainguard ones?

I don't see why not? @spencerschrock Thoughts?

regarding num_vulns - I believe we are comparing distroless/base to cgr/static. If there is no reason for us to depend on distroless/base, we could very well move to distroless/static which should have a lower num_vulns.

I'm also okay with that @azeemshaikh38 because there is not much difference between distroless/static and the cgr/static

any updates on this ?

I'm still willing to do this folx

kindly ping @azeemshaikh38 @naveensrinivasan

I'm still waiting for your comments to make it done 😇

kindly ping 🙉

@developer-guy sorry for the late reply here. My preference is a no to this change for the reason that distroless is an industry standard compared to cgr. If a majority of @ossf/scorecard-maintainers feel differently and would like to move to cgr image instead, I'd be ok with it too. @naveensrinivasan @spencerschrock @laurentsimon @raghavkaul fyi.
If there is no response from any of the Scorecard maintainers here, I will close this PR to reduce our PR queue.

@developer-guy sorry for the late reply here. My preference is a no to this change for the reason that distroless is an industry standard compared to cgr. If a majority of @ossf/scorecard-maintainers feel differently and would like to move to cgr image instead, I'd be ok with it too. @naveensrinivasan @spencerschrock @laurentsimon @raghavkaul fyi.
If there is no response from any of the Scorecard maintainers here, I will close this PR to reduce our PR queue.

@developer-guy  Thanks!
For now, we are going to close this. We can relook at this later.


Add commit body & footer after the bullets list

currently we only show the subject on the bullets list of changes.
It is a template thing and is configurable, but we should have good defaults.
template is here
https://github.com/tunnckoCore/semantic-release-app/blob/99fe28d3a4aeb04b20489fb42b47228dbd7c8705/src/lib/config.js#L13-L22

i know that bug... it  is because commit.body is always (?) empty/null, because of the parse-commit-message bug.


Improve logging and timeouts for geocodings

[x] Added logs for the geocoder cache
[x] Fixed timeouts problems, now timeouts for cache are working ok
[x] Added pid field to store the PID of the background process to facilitate the search of problematic/stucked processes
[x] Append and store method syncs data before store it to avoid shared state problems
[x] Cancel is now working for here batch jobs
[x] Stored status and remote id always as well as use model as shared state object to communicate sync and async part of geocoding

@rafatower check it out please

:+1:

@rafatower check it again please :)

@ethervoid there are many relevant tests broken. Can you please take a look?

On it!

@rafatower tests passing!

:+1:

:+1:

Please @rafatower check the last commit


React.forwardRef accepts a render function, not Functional Component

React.forwardRef accepts a function like render(props: Props, ref: Refs), which is similar with Functional Component but Functional Component accepts a legacy context as the 2nd argument.
I know React.forwardRef works fine with Functional Component in most case but  I think it should distinguish between them.

I agree this is clearer and would also prefer this. @bvaughn?

I still feel like explaining intricacies of displayName assignment gets in the way of communicating what this feature is for, and doesn't make the blog post very helpful. Instead, the indirection makes me skip over the whole example because I find it too difficult to comprehend. I'd still prefer the original version suggested by @koba04 (and I'd remove the displayName logic here altogether). We could keep this logic as a "tip" in ForwardRef doc instead.
I don't have a strong opinion though.

Hm 🙁
People will likely copy-paste our example though. If we don't show the naming bit, everyone's DevTools will likely end up with the anonymous forward ref label.
I don't feel strongly about this. If you both think the previous, anonymous function version was more readable- I'll defer to your judgement. 😄
(Sorry for the back and forth on this, @koba04)

Blog post is introductory. With time people will start copy-pasting from the main doc. The main doc can have a section that shows an example with a screenshot. People who write HOCs probably will read this doc in depth because it's not a beginner-friendly feature anyway, and by the point they need it they're probably comfortable reading docs.

I think I'm probably too close to this to trust my opinion. I trust yours, Dan.

Let's add a link to main doc ("Forwarding Refs") after the code example. In the main doc, let's add a separate section about "Customizing Name in DevTools".

I like that suggestion.

This looks nice. Thanks for the change, @koba04. Dan or I can make the follow up change he proposed. (Or you could in another diff, if you're interested.)

Related: I found it a bit overwhelming that we have three very similar examples but with slightly different naming in the blog post, in Forwarding Refs doc, and in forwardRef() reference.

Yeah, maybe.
I thought sharing a use case (e.g. theming) might make things easier for people, since they wouldn't need to stop and think about a new use case and why you'd want to use it, etc.

Thanks! I'll try to do the follow-up changes!


Fix an assertion during training issue #65

Are we good to go here? Maybe we could provide a more extensive comment if the current solution is not straightforward.

I'd like to implement what I had in mind when I made the suggestion

My previous solution based on your suggestion seems better. The question is does the current one work as well?

Current solution works as well functionally: no assertion during training, assertion during inference.

ok i understand now, thank you @carmocca


redux-form prop is null until onBlur event is triggered

I have a Field input, lets call it dummyCalculated. I also have a state call distance.
When I first click "Submit", it returns an empty object. Only when I select the input dummyCalculated Field and click on Submit again, I will get the props.dummyCalculated value.
I tried with initialValue and onChange event to forcefully update the value, but it doesn't work as well.
const renderField = ({input, calculatedDistance, onChange, label}) => (

    {label}
    <input type="text" {...input} value={calculatedDistance}/>

  constructor(props)
  {
    super(props);
    this.state = {
      distance: 30
    };
  }

  onSubmit(props)
  {
    console.log(props);
  }

    return (
    const {handleSubmit, dummyCalculated} = this.props;

        <form onSubmit={handleSubmit(this.onSubmit.bind(this))}>

            <Field
              label="Dummy label "
              name="dummyCalculated"
              component={renderField}
              calculatedDistance={this.state.distance}
              />

          <button type="submit" className="btn btn-primary">Submit</button>
        </form>

Why are you trying to override the value with one you are keeping in state? That defeats the whole purpose of redux-form. Your code should look more like this:
const renderField = ({input, label}) => (

    {label}
    <input type="text" {...input}/>

  constructor(props)
  {
    super(props);
  }

  componentDidMount() {
    this.props.initialize({ dummyCalculated: 30 })
  }

  onSubmit(props)
  {
    console.log(props);
  }

  render() {
    const {handleSubmit, dummyCalculated} = this.props;
    return (

          <form onSubmit={handleSubmit(this.onSubmit.bind(this))}>

              <Field
                label="Dummy label"
                name="dummyCalculated"
                component={renderField}/>

            <button type="submit" className="btn btn-primary">Submit</button>
          </form>

      )
  }

Because this issue saw almost no activity for almost a year, I'm closing it.


Add tutorial

This adds a small tutorial with fake data created ad hoc.
Also fixes a bug in the binning function.
It also refactors the code following the definitions in the pre-commit hooks.

I would not say this closes #41 but rather adds one step towards it.


Consider refactoring DBA to Majorize-Minimize Mean Algorithm

Referencing issue #44, DBA can be expressed in matrix and vectorized terms that might be more suited for numpy's vectorized operations. I think it would be worthwhile to explore this to see if we can get a considerable performance boost by expressing the function in this form.
Instead of removing the internals of the original algorithm, it might be worth considering adding a parameter to the function which takes values such as 'mm' or 'original' to differentiate what kind of internal operation it uses: either the form in the original paper, or the MM expression of the algorithm.
Relevant section in the Schultz & Jain (2017) paper is 4.2. The Majorize-Minimize Mean Algorithm and it's respective algorithm: 2.

It is now the default for DBA on dev branch, and will be included in tslearn from version 0.3.
Thanks @unnamedplay-r for the ref, computations are now twice faster in my tests!

Yay! This is so wonderful @rtavenar, I’m grateful you took the recommendation into consideration.


Adding ubi buildpackless builder integration

Merge After

https://github.com/paketo-community/ubi-nodejs-extension/pull/56

Summary

This PR, tests node-run-script against ubi buildpackless base builder in conjuction with ubi nodejs extention .
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@TisVictress For some reason the if statement had been committed wrongly, fixed it on this commit https://github.com/paketo-buildpacks/node-run-script/pull/274/commits/399f31874946c2d5d9bf8643acccd29538d1be08
Also the ubi-nodejs-extension is failing, as the latest changes are not included on the latest release which is requirement for the tests to pass. I opened an issue for cutting a new release https://github.com/paketo-community/ubi-nodejs-extension/issues/71


Integrate the newly-created GeneralClient into the SMA.

Integrate the newly-created GeneralClient into the SMA. The PR (for that GeneralClient) was recently merged into master: Add management client to fetch microservices configuration and metrics.
Updates being made here: Construction Site

This issue is directly related to: Issue #690

@jpwhitemn @tsconn23 Based on some preliminary testing I did a few minutes ago, my newly-pushed change-set for the related branch is working to get us api/v1/config data for all EdgeX services. Now moving on to get us api/v1/metrics data as well. To be followed by lots more testing, and, finally, making sure that the (original) CoreCommand-related updates to master branch are reverted out.


Fails to display Japanese characters

Contour Terminal version
0.3.12
Installer source
something else (please specify below)
Operating System
Artix Linux Rolling
Architecture
x86-64
Other Software
noto-fonts 1:23.11.1-1
Steps to reproduce
Attempt to read any Japanese characters
Expected Behavior
Japanese characters mostly display correctly:

(from kitty)
Actual Behavior
no Japanese text displays:

Additional notes
No response

Hi @binex-dsk, thanks for reporting this issue, can you please provide also output of contour debug font.locator,font.textshaping ? while shaping Japanese characters

Also, try setting monospsce to false in there configuration.
I think this is an issue with the hot config reload, as I had it set before and the hot config reload didn't reset the font. Only when I closed and reopened contour did this change the font.

It might be that live config is not working at the moment :(

@binex-dsk  if issue is not resolved after changing config file you need to show full output of contour debug font.locator,font.textshaping

I sent it here https://github.com/contour-terminal/contour/issues/1340#issuecomment-1819461137
The issue is resolved, Japanese text is displayed properly now. The issue is related to live reload.

@binex-dsk many thanks for the patience.

The issue is related to live reload.

We'll be working on that (and configuration in general) in the near future. so I'm quite confident it won't take too long to get that in.
Mind, live reload should be working, but while I was implementing it, I didn't think it might make sense for a few settings. Turns out it does most of all. Give us some time. Enjoy. ;-)

Thanks! I'll close this issue.


Fix memory leak in bm_cq_multiple_threads (due to not calling grpc_shutdown)

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


Background video not shown in Page Preview

Bug Description

It looks like background videos don't show up in Page Preview nor Grid View
Expected Behaviour
Background video should be visible in Page preview and Grid View.

Steps to Reproduce

Add a Video
Set it as background
See the Page carousel previews / grid.

Screenshots

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

Duplicate of #999


main.go: split main into Anew function and main so it can be exported and used…

… by other golang scripts. Tested and this script appears to work fine as well.

Apologies but this now conflicts due to me finally catching up with other PRs.
If this could be fixed up (including not using github.com/manasmbellani/anew/anewlib as an import path) I could merge it. I'd also prefer a name other than anewlib to be honest, but I don't know exactly what that should be.
Thank you for your contribution even though it won't be merged at this time!


Major clippy overhaul.

There were severe problems with our clippy linting:

clippy suggested newer features than our MSRV
clippy did not check all targets, e.g. "tests" were completely unchecked

This PR:

adds clippy.toml to be able to set MSRV for clippy. However, this only works if clippy is invoked like this: cargo +nightly clippy.
adds --target=${{ matrix.job.target }} to github actions (I hope this is enough, locally I run cargo +nightly clippy --all-targets --all-features
fixes all clippy warnings that were exposed with the previous steps

There is:
https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_wraps
Jan Scheer @.***> schrieb am Sa., 29. Mai 2021, 19:08:

@.**** commented on this pull request.
In src/uu/cp/src/cp.rs
https://github.com/uutils/coreutils/pull/2298#discussion_r641961466:

@@ -1089,7 +1089,7 @@ fn copy_attribute(source: &Path, dest: &Path, attribute: &Attribute) -> CopyResu
}

#[cfg(not(windows))]
-#[allow(clippy::unnecessary_wraps)] // needed for windows version
+#[allow(clippy::unnecessary_unwrap)] // needed for windows version
I'm not sure I understand. Which unwrap are you referring to? Any error
from the call to std::os::unix::fs::symlink(source, dest).context(context)
is not handled, or am I missing something?
Oh, I was referring to L539:
https://github.com/uutils/coreutils/blob/3d42d41a72e7f43d6d5a88d73eed4dae181f7dc9/tests/common/util.rs#L533-L540
Also, you changed this from unnecessary_wraps to unnecessary_unwrap,
but... there's no unwrap in here. That's what confused me.
There is no unnecessary_wraps, probably a typo and clippy's suggestion
was to use unnecessary_unwrap.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/uutils/coreutils/pull/2298#discussion_r641961466, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQ77NSAOPTRA6A6MRBEY6TTQENRNANCNFSM45XTOCSA
.