Login UI shows Error 500

Hi I am starting to explore IdentityServer4 and I was able to execute Client Credentials Grant and Password Grant, but when I tried the quickstart for Authenticating using OpenID connect I get error Failed to load resource: the server responded with a status of 500 (Internal Server Error)
Please let me know if I am missing something:
First I copied and pasted the wwwroot, QuickStart and Views to my WebClient project. / Also tried to copied to AuthProvider but same thing happens.
Here is the config.cs of my AuthProvider:
public static IEnumerable<Client> GetClients()
    {
        return new List<Client>
        {
          //other clients were removed for clarity
             // OpenID Connect implicit flow client (MVC)
            new Client
            {
                ClientId = "mvc",
                ClientName = "MVC Client",
                AllowedGrantTypes = GrantTypes.Implicit,

                // where to redirect to after login
                RedirectUris = { "http://localhost:5002/signin-oidc" },

                // where to redirect to after logout
                PostLogoutRedirectUris = { "http://localhost:5002/signout-callback-oidc" },

                AllowedScopes = new List<string>
                {
                    IdentityServerConstants.StandardScopes.OpenId,
                    IdentityServerConstants.StandardScopes.Profile
                }
            }
        };
    }

    public static IEnumerable<IdentityResource> GetIdentityResources()
    {
        return new List<IdentityResource>
        {
            new IdentityResources.OpenId(),
            new IdentityResources.Profile(),
        };
    }

Here is my StartUp for AuthProvider:
public class Startup
{
// This method gets called by the runtime. Use this method to add services to the container.
public void ConfigureServices(IServiceCollection services)
{
services.AddIdentityServer()
.AddDeveloperSigningCredential()
.AddInMemoryIdentityResources(Config.GetIdentityResources())
.AddInMemoryApiResources(Config.GetApiResources())
.AddInMemoryClients(Config.GetClients())
.AddTestUsers(Config.GetUsers());
    }

    // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
    public void Configure(IApplicationBuilder app, IHostingEnvironment env)
    {
        if (env.IsDevelopment())
        {
            app.UseDeveloperExceptionPage();
        }

        app.UseIdentityServer();

    }
}

WebClient StartUp.cs
    public void ConfigureServices(IServiceCollection services)
    {
        services.AddMvc();

        JwtSecurityTokenHandler.DefaultInboundClaimTypeMap.Clear();

        services.AddAuthentication(_ =>
            {
                _.DefaultScheme = "Cookies";
                _.DefaultChallengeScheme = "oidc";
            })
            .AddCookie("Cookies")
            .AddOpenIdConnect("oidc", _ =>
            {

                _.SignInScheme = "Cookies";

                _.Authority = "http://localhost:5000";
                _.RequireHttpsMetadata = false;

                _.ClientId = "mvc";
                _.SaveTokens = true;
            });
    }

    // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
    public void Configure(IApplicationBuilder app, IHostingEnvironment env)
    {
        app.UseAuthentication();

        app.UseStaticFiles();
        app.UseMvcWithDefaultRoute();
    }

Logs?

info: Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager[0]
User profile is available. Using 'C:\Users\ethan\AppData\Local\ASP.NET\DataProtection-Keys' as key repository and Windows DPAPI to encrypt keys at rest.
info: IdentityServer4.Startup[0]
You are using the in-memory version of the persisted grant store. This will store consent decisions, authorization codes, refresh and reference tokens in memory only. If you are using any of those features in production, you want to switch to a different store implementation.
Hosting environment: Production
Content root path: D:\POC\IdentityServer4\3Adding Authentication with OpenID Connect\CTG.AuthProvider
Now listening on: http://localhost:5000
Application started. Press Ctrl+C to shut down.
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
Request starting HTTP/1.1 GET http://localhost:5000/.well-known/openid-configuration
info: IdentityServer4.Hosting.IdentityServerMiddleware[0]
Invoking IdentityServer endpoint: IdentityServer4.Endpoints.DiscoveryEndpoint for /.well-known/openid-configuration
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
Request finished in 213.1821ms 200 application/json; charset=UTF-8
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
Request starting HTTP/1.1 GET http://localhost:5000/.well-known/openid-configuration/jwks
info: IdentityServer4.Hosting.IdentityServerMiddleware[0]
Invoking IdentityServer endpoint: IdentityServer4.Endpoints.DiscoveryKeyEndpoint for /.well-known/openid-configuration/jwks
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
Request finished in 127.2968ms 200 application/json; charset=UTF-8
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
Request starting HTTP/1.1 GET http://localhost:5000/connect/authorize?client_id=mvc&redirect_uri=http%3A%2F%2Flocalhost%3A5002%2Fsignin-oidc&response_type=id_token&scope=openid profile&response_mode=form_post&nonce=636449516302690151.YTRlNThiNmItMDdjZi00ODRlLWIyN2UtOWUwZGVjZGVjMDRiMzRmY2Y5YTAtYmRiZi00MTJlLWE5Y2UtMmQ2ODkzMWU3MzA5&state=CfDJ8A9nVuAtMtpGsNoIVZeQrKfwq4uD7W7aSzG0_f6x1tlO49P5Qh4yVjpR45N_OhtXw9rNXlJ4pnVaqRnwELmE50vwhIAjQRWxdNNTNFJKilP4-g9KHVhIsNBkzdtmuFMeMG82huU6l5LO5zIk-o-a_3GLpa-JbV7WbnL--KtIbAlw1lcabKbCXZVkIdXH5YdcimibSWHkLETC7xRL4CMNkkNvkJT3t2Hj7tNOEXPFc2jos4_zuDKfr2gAgtsTq9tKFknfdIab9A2011EI7-E6QHy2bpFKzHBFTOoHejznZGQCQ3107JHa3CrfoJYcwuOHuw&x-client-SKU=ID_NET&x-client-ver=<IP_ADDRESS>
info: IdentityServer4.Hosting.IdentityServerMiddleware[0]
Invoking IdentityServer endpoint: IdentityServer4.Endpoints.AuthorizeEndpoint for /connect/authorize
info: IdentityServer4.Endpoints.AuthorizeEndpoint[0]
ValidatedAuthorizeRequest
{
"ClientId": "mvc",
"ClientName": "MVC Client",
"RedirectUri": "http://localhost:5002/signin-oidc",
"AllowedRedirectUris": [
"http://localhost:5002/signin-oidc"
],
"SubjectId": "anonymous",
"ResponseType": "id_token",
"ResponseMode": "form_post",
"GrantType": "implicit",
"RequestedScopes": "openid profile",
"State": "CfDJ8A9nVuAtMtpGsNoIVZeQrKfwq4uD7W7aSzG0_f6x1tlO49P5Qh4yVjpR45N_OhtXw9rNXlJ4pnVaqRnwELmE50vwhIAjQRWxdNNTNFJKilP4-g9KHVhIsNBkzdtmuFMeMG82huU6l5LO5zIk-o-a_3GLpa-JbV7WbnL--KtIbAlw1lcabKbCXZVkIdXH5YdcimibSWHkLETC7xRL4CMNkkNvkJT3t2Hj7tNOEXPFc2jos4_zuDKfr2gAgtsTq9tKFknfdIab9A2011EI7-E6QHy2bpFKzHBFTOoHejznZGQCQ3107JHa3CrfoJYcwuOHuw",
"Nonce": "636449516302690151.YTRlNThiNmItMDdjZi00ODRlLWIyN2UtOWUwZGVjZGVjMDRiMzRmY2Y5YTAtYmRiZi00MTJlLWE5Y2UtMmQ2ODkzMWU3MzA5",
"Raw": {
"client_id": "mvc",
"redirect_uri": "http://localhost:5002/signin-oidc",
"response_type": "id_token",
"scope": "openid profile",
"response_mode": "form_post",
"nonce": "636449516302690151.YTRlNThiNmItMDdjZi00ODRlLWIyN2UtOWUwZGVjZGVjMDRiMzRmY2Y5YTAtYmRiZi00MTJlLWE5Y2UtMmQ2ODkzMWU3MzA5",
"state": "CfDJ8A9nVuAtMtpGsNoIVZeQrKfwq4uD7W7aSzG0_f6x1tlO49P5Qh4yVjpR45N_OhtXw9rNXlJ4pnVaqRnwELmE50vwhIAjQRWxdNNTNFJKilP4-g9KHVhIsNBkzdtmuFMeMG82huU6l5LO5zIk-o-a_3GLpa-JbV7WbnL--KtIbAlw1lcabKbCXZVkIdXH5YdcimibSWHkLETC7xRL4CMNkkNvkJT3t2Hj7tNOEXPFc2jos4_zuDKfr2gAgtsTq9tKFknfdIab9A2011EI7-E6QHy2bpFKzHBFTOoHejznZGQCQ3107JHa3CrfoJYcwuOHuw",
"x-client-SKU": "ID_NET",
"x-client-ver": "<IP_ADDRESS>"
}
}
info: IdentityServer4.ResponseHandling.AuthorizeInteractionResponseGenerator[0]
Showing login: User is not authenticated
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
Request finished in 167.0001ms 302
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
Request starting HTTP/1.1 GET http://localhost:5000/account/login?returnUrl=%2Fconnect%2Fauthorize%2Fcallback%3Fclient_id%3Dmvc%26redirect_uri%3Dhttp%253A%252F%252Flocalhost%253A5002%252Fsignin-oidc%26response_type%3Did_token%26scope%3Dopenid%2520profile%26response_mode%3Dform_post%26nonce%3D636449516302690151.YTRlNThiNmItMDdjZi00ODRlLWIyN2UtOWUwZGVjZGVjMDRiMzRmY2Y5YTAtYmRiZi00MTJlLWE5Y2UtMmQ2ODkzMWU3MzA5%26state%3DCfDJ8A9nVuAtMtpGsNoIVZeQrKfwq4uD7W7aSzG0_f6x1tlO49P5Qh4yVjpR45N_OhtXw9rNXlJ4pnVaqRnwELmE50vwhIAjQRWxdNNTNFJKilP4-g9KHVhIsNBkzdtmuFMeMG82huU6l5LO5zIk-o-a_3GLpa-JbV7WbnL--KtIbAlw1lcabKbCXZVkIdXH5YdcimibSWHkLETC7xRL4CMNkkNvkJT3t2Hj7tNOEXPFc2jos4_zuDKfr2gAgtsTq9tKFknfdIab9A2011EI7-E6QHy2bpFKzHBFTOoHejznZGQCQ3107JHa3CrfoJYcwuOHuw%26x-client-SKU%3DID_NET%26x-client-ver%3D<IP_ADDRESS>
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
Request finished in 5.31ms 404
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
Request starting HTTP/1.1 GET http://localhost:5000/favicon.ico
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
Request finished in 6.8839ms 404

This is not  an IdentityServer issue.  I think it's in the OpenIDConnect middleware.  I am NOT using IdentityServer but I get the same error.

You say there's a 500 status code in the response, but I don't see any exception in the logs.

I am having the same problem been following this

This localhost page can’t be found
No webpage was found for the web address: http://localhost:5000/account/login?returnUrl=%2Fconnect%2Fauthorize%2Fcallback%3Fclient_id%3Dmvc%26redirect_uri%3Dhttp%253A%252F%252Flocalhost%253A5002%252Fsignin-oidc%26response_type%3Did_token%26scope%3Dopenid%2520profile%26response_mode%3Dform_post%26nonce%3D636456424062248347.MjA3N2U0OWUtNzM5Ny00MTgyLWJhNzYtYzcwYjIwMTExNmU0OTM3N2Q3M2MtN2MzMi00ZDViLWE4NzYtNDA0Yjk0MmIwMjA1%26state%3DCfDJ8Ir18WT9egNOoCMQs_rv1SUWaxu3IaZUQ3IeeprwoRq628hKJAWhN0-JdbTga33ubWMKL5RbE-h89p83e_k8TNVbTtY-xLGK1uTQ9HWDzULI2nYHkQVfCY30gJfJvojvQDxC_gCC2tsoOwmnFlvadK3-zesdF4KsSSNFSDnrdKcBQ60ohPWG_AuUbamgeqo2qCGlZE91nH_yLMpVqkLzjTLqL8m3N8lF6U0s7qMwgg8drhdkx5jiTg-EIjXuuu1O-amprRo5U5aIRwkEqfLKv13p1qrZVMGQs7iRS8l7-9_LY2li9t6Zwv0uw5hlJ44SBmQw3mCPxlGSOL_LLII_avA%26x-client-SKU%3DID_NET%26x-client-ver%3D<IP_ADDRESS>
Search Google for localhost 5000 account login
HTTP ERROR 404

I think the issue is that http://localhost:5000 is the auth server project yet this project doesnt have a login page you haven't told us how to create that yet.   I have been trying to piece though all of the sample projects here on GitHub trying to see how to create these login pages and have yet to get it working.

@LindaLawton  that's the exact steps I followed and I have same confusion as you I tried to locate the controlle and actions that would lead me to signin-oidc but I could not find it in the Quick Start UI.  I check some video recordings and I saw that after dropping the files in the web project all worked just fine.
@brockallen In the logs that I shared coming from IdentityServer am I not experiencing error 404?    Error 500 is showing up during redirect in the web ui.

I am trying to piece it together now if i get it working i will let you know

@LindaLawton  thanks in advance.

All set on this issue -- can we close?

@brockallen  as far as i can see the tutorial hasn't been updated to describe how to create the missing files so i would say it should remain open.    Do you have a different tutorial that shows how to do it?

Yea, the broken edit is a known issue. Sorry.
As for the problem -- no one has yet shown the exception from the server that causes the 500, so I have no idea what the problem might be and if it's something I am to fix. Find me an exception message, and we can try to debug it.

For me the issue is that the files that result in the login form don't exist.   That part of the tutorial is missing.

ok, i'll dig into it when i can.

or fix the issue with git and I will ;)

or fix the issue with git and I will ;)

I wish I knew how -- it's not my department :)

or fix the issue with git and I will ;)

BTW, that makes me think this is just a doc issue? Or is something larger broken?

I think the issue is that http://localhost:5000 is the auth server project yet this project doesnt have a login page you haven't told us how to create that yet.

But the steps here do show that: http://docs.identityserver.io/en/release/quickstarts/3_interactive_login.html#adding-the-ui
So I'm still confused what this issue is about.

First I copied and pasted the wwwroot, QuickStart and Views to my WebClient project.

Ah, is this the problem? You copied it into the WebClient and not into the IdentityServer host itself?

Is this change sufficient to clarify? https://github.com/IdentityServer/IdentityServer4/commit/c8a5c1e090f55f47c4c635647d0dfb7d40c90d0a

@brockallen I copied it into the Identity server but the login page is no where to be found. Forgive my ignorance but I presumed a quick search in the solution would reveal where that login page file is but I couldn't find any. The steps in the documentation shows how to include the login page so I downloaded it and put it into my identity server.

This is what i see in the URL when I run the web client. And developer tools shows "Failed to load resource: the server responded with a status of 404"
https://localhost:44382/home/error?errorId=CfDJ8A9nVuAtMtpGsNoIVZeQrKcD_WgOURePGNO3V8QRbouZUoRoJ9m4ds8J_neiVgEvjpIdRlSkOO_34HXFbQiIzrfiMKG0LO7z6svjAb5puySWboeSKUTstzAS2TSXBjaZ3st3p6WLwUzPn5mFY0k9ifn6IR-nYV1bh3f5cCqkOl6zeAOvrsffGVjtw4qdl566pwSKxN30SLDAxV1cPaY_EUeNlnsAQ8CTMtpAY7tMl3enCEcIqqAwiU4O1yiN0QsfpBtVbyQgs7tytBo9V1bOujBGyLZIznK092Q5-pa84b_lWlXiKZ2wX23xl7C3wmGgAQ

@brockallen  Do you have a complete working sample of this that we can take a look at?

All the QS samples should be working samples.

Ok, I think the other issue us that if you're following along, then MVC is not added as middleware. I'll add that to the QS docs as well.

So in the docs, the section that says: See the readme for the quickstart UI for more information. has a link to those instructions, but I duplicated them anyway in the QS docs. I hope that makes it clear.

@brockallen thanks for the help. I tried the Quickstart sample and found out that it is working.


[VCX] enable generic connection messages, signing and verification

I'm sorry, I don't understand. I had to change it from an enum to strings because the message type can be anything, not just one of the enum values.


Buy amount input validation

When using the Market to create a buy order, buy amount box is allowing input only the max amount in FB that it reads in the wallet, although the amount is in the token to be bought (exp FXP). For example, if I want to buy 15 FXP, when I input 15 in the Buy amount box, it always modifies that to the FB amount I have in the wallet.

Yes - working on an upgraded ux from all the things we learned after putting it out. Will respond here once its updated


add 'enable' & 'disable' commands

individual ui-*-enable and ui-*-disable commands are more useful for scripting

Thanks @JacobTravers!


Update Py3.9 to 3.10 minimum

Before submitting
Please complete the following checklist when submitting a PR:

[ ] All new features must include a unit test.
If you've fixed a bug or added code that should be tested, add a test to the
tests directory!

[ ] All new functions and code must be clearly commented and documented.
If you do make documentation changes, make sure that the docs build and
render correctly by running make docs.

[ ] Ensure that the test suite passes, by running make test.

[ ] Add a new entry to the .github/CHANGELOG.md file, summarizing the
change, and including a link back to the PR.

[ ] Ensure that code is properly formatted by running make format.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

Context: This PR removes support for Python 3.9 and migrates to 3.10 as the oldest supported version.
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues:

Just a note, https://github.com/PennyLaneAI/pennylane/pull/6223 will merge before this, and https://github.com/PennyLaneAI/catalyst/pull/1104 after this PR.


Compact select association sub context

This PR adds a new sub context to compact, called compactSelectAssociation. This is being done to be consistent with changes being made in chaise to support configuration of individual search popup displays.

@jrchudy I don't remember what we agreed on last time, but what about other variations of compact/select (for foreign key, or other popups)? And didn't we have different contexts for link and unlink?
P.S. You should update annotation.md and also do make all to make sure the new API is included in the api.md.

I added more contexts to align with the attached issue and added those to annotation.md. We didn't finalize on the names for each other the other search popups contexts, so I used what I wrote in the issue that Hongsuda saw before.
I noticed there was a compact/entry context in ermrestJS which I think is probably being used for foreign key popups, so I'm not sure what changes we should make regarding that. It seems like contextualize.compactEntry is only being called in ermrestJS code.


Add commands for Yosys Synthesis and FusesocJob

Fixes https://github.com/librecores/librecores-ci-jenkins-server/issues/67

[x] Implemented FuesocJobSpec & FuesocRunSpec library which executes a fusesoc job

[x] Implemented YosysSynthesisReport shared library for Yosys Synthesis, generalized for every Hardware  Projects. Executes Yosys synthesis and generates resource usage reports

[x] Integrate YosysSynthesis step in openriscPipeline as: YosysSynthesisReport(String core, String targetName, String yosysLogPath)

[x] Add Documentation

@oleg-nenashev Does this looks fine ? . Will add more changes to it . It works in isolated pipeline . Need to intgrate with openrisc pipeline

Hi @oleg-nenashev, This PR works completely now for openrisc pipelines. Also added documentation


Add ASAP and LEConv

Hi @rusty1s ! It's been exactly a year knowing you and your PyG.
This PR adds the ASAP pooling operator and LEConv graph neural net from the paper
"ASAP: Adaptive Structure Aware Pooling for Learning Hierarchical Graph Representations".
We really appreciate you maintaining this wonderful library and as a small token of gratitude we acknowledge your efforts in our paper above :)

@ekagra-ranjan This looks really interesting for what we want to try and do over in particle physics!
Me and a colleague have been having similar ideas (but a bit later in time!):
https://arxiv.org/abs/2002.03605
https://arxiv.org/abs/2003.08013
We are very curious about developing methods with variable number of output clusters. We'll have to segment something like 1000-2000 clusters per point cloud, each cluster consisting of ~40-1000s of points.
I'm focusing right now on deployability and scalability of the graph models, (over in the .jittable() PR). I'll give this model a try in that context once it's merged, and let you know how it goes.

Hi @rusty1s, Can you please have a look at the PR?

Hi @ekagra-ranjan, sorry for answering late! This looks super good already, thank you very much for this valuable contribution. Please give me a few more days to review and merge.

Codecov Report

Merging #1218 into master will increase coverage by 0.11%.
The diff coverage is 98.40%.

@@            Coverage Diff             @@
##           master    #1218      +/-   ##
==========================================
+ Coverage   90.80%   90.91%   +0.11%
==========================================
  Files         158      160       +2
  Lines        5818     5980     +162
==========================================
+ Hits         5283     5437     +154
- Misses        535      543       +8

Impacted Files
Coverage Δ

torch_geometric/nn/pool/asap_pool.py
97.80% <97.80%> (ø)

torch_geometric/nn/conv/__init__.py
100.00% <100.00%> (ø)

torch_geometric/nn/conv/le_conv.py
100.00% <100.00%> (ø)

torch_geometric/nn/pool/__init__.py
79.31% <100.00%> (+0.73%)
:arrow_up:

torch_geometric/utils/negative_sampling.py
91.52% <0.00%> (-8.48%)
:arrow_down:

torch_geometric/nn/conv/gin_conv.py
98.00% <0.00%> (-2.00%)
:arrow_down:

torch_geometric/nn/models/dimenet.py
19.71% <0.00%> (ø)

torch_geometric/transforms/to_superpixels.py
100.00% <0.00%> (ø)

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 6ea938a...a600a5a. Read the comment docs.

Codecov Report

Merging #1218 into master will decrease coverage by 3.60%.
The diff coverage is 97.56%.

@@            Coverage Diff             @@
##           master    #1218      +/-   ##
==========================================
- Coverage   90.80%   87.19%   -3.61%
==========================================
  Files         158      162       +4
  Lines        5818     6319     +501
==========================================
+ Hits         5283     5510     +227
- Misses        535      809     +274

Impacted Files
Coverage Δ

torch_geometric/nn/conv/le_conv.py
96.66% <96.66%> (ø)

torch_geometric/nn/pool/asap_pool.py
97.80% <97.80%> (ø)

torch_geometric/nn/conv/__init__.py
100.00% <100.00%> (ø)

torch_geometric/nn/pool/__init__.py
79.31% <100.00%> (+0.73%)
:arrow_up:

torch_geometric/data/in_memory_dataset.py
86.95% <0.00%> (-11.29%)
:arrow_down:

torch_geometric/utils/negative_sampling.py
91.52% <0.00%> (-8.48%)
:arrow_down:

torch_geometric/data/data.py
97.10% <0.00%> (-2.90%)
:arrow_down:

torch_geometric/nn/conv/tag_conv.py
97.22% <0.00%> (-2.78%)
:arrow_down:

torch_geometric/utils/convert.py
97.26% <0.00%> (-2.74%)
:arrow_down:

torch_geometric/nn/conv/gin_conv.py
98.00% <0.00%> (-2.00%)
:arrow_down:

... and 9 more

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 6ea938a...dce4e16. Read the comment docs.

Hi @ekagra-ranjan, thanks again for this PR and acknowledging my work :)
I made a careful pass to the PR and made the following changes:

I renamed it to ASAPooling
LEConv now first transforms node features with W_2 and W_3 before propagating takes place
Cleaned up the ASAP module quite a bit. It now uses SparseTensor to compute S^T A S which leads to more readible code

Please let me know what you think so that we can finally merge.
Best,
Matthias

Does ASAP really require self-loops? I thought they were already handled by LEConv.

Please feel free to merge the changes.

Perfect. These thing rocks btw, reaches really good accuracy on PROTEINS. Congratz :)

Thanks for the feedback! Really boosted my confidence :)

Yeah, I'm gonna take a look at it for our particle physics clustering problem for sure. :D Will let you know how it goes.


Example project not building

Example project not building on my XCode 8.0
Here is 52 Swift Compiler Error in RxSwift Pod

Which version of Cocoapods you're running on?

1.0.1

And which version of RxSwift is installed when you're running pod install? Could you remove Podfile.lock and perform pod install once again?

2.6.0

Are you using Swift 3.0? Or Swift 2.3?

Swift 3.0

It should download RxSwift in 3.0-beta version. Have you tried removing Podfile.lock or run pod update?. :)

After update build completed. Thank you!

Glad it worked :-)


gcloud beta run deploy - permission denied

Maybe its not supported yet in Europe ?
log:
make cloud-deploy
cd /Users/apple/workspace/go/src/github.com/barthr/Identicon && gcloud beta run deploy --image gcr.io/winwisely-cloudrun/identicon-generator --platform managed
Service name (identicon-generator):
ERROR: (gcloud.beta.run.deploy) PERMISSION_DENIED: Permission denied on 'locations/europe-west3' (or it may not exist)
make: *** [cloud-deploy] Error 1

Container is there:

make:
# see: https://bartfokker.com/posts/cloud-run/
# code: https://github.com/barthr/Identicon/tree/feature/identicon-library

LIB_NAME=Identicon
LIB=github.com/barthr/$(LIB_NAME)
LIB_BRANCH=feature/identicon-library
LIB_FSPATH=$(GOPATH)/src/$(LIB)

GO111MODULE=on

CLOUD_PROJECT_ID=winwisely-cloudrun

help:  ## Display this help
	@awk 'BEGIN {FS = ":.*##"; printf "\nUsage:\n  make \033[36m<target>\033[0m\n"} /^[a-zA-Z_-]+:.*?##/ { printf "  \033[36m%-15s\033[0m %s\n", $$1, $$2 } /^##@/ { printf "\n\033[1m%s\033[0m\n", substr($$0, 5) } ' $(MAKEFILE_LIST)

print: ## print
	@echo
	@echo LIB_NAME: $(LIB_NAME)
	@echo LIB: $(LIB)
	@echo LIB_BRANCH: $(LIB_BRANCH)
	@echo LIB_FSPATH: $(LIB_FSPATH)
	@echo

git-clone:
	mkdir -p $(LIB_FSPATH)
	cd $(LIB_FSPATH) && cd .. && rm -rf $(LIB_NAME) && git clone ssh://git@$(LIB).git
	cd $(LIB_FSPATH) && git checkout $(LIB_BRANCH)
git-pull:
	cd $(LIB_FSPATH) && git pull
git-clean:
	rm -rf $(LIB_FSPATH)

code:
	code $(LIB_FSPATH)

os-dep:
	# Assume hover make file has ben run

export PORT=8080
run-api:
	cd $(LIB_FSPATH)/cmd/api && go run .

run-cli:
	cd $(LIB_FSPATH)/cmd/cli && go run . -name joe

cloud-auth:
	gcloud auth login
	gcloud config set project $(CLOUD_PROJECT_ID)
cloud-open:
	# run
	open https://console.cloud.google.com/run?project=$(CLOUD_PROJECT_ID)
	# container
	open https://console.cloud.google.com/cloud-build/builds?project=$(CLOUD_PROJECT_ID)
	# store
	open https://console.cloud.google.com/storage/browser?project=$(CLOUD_PROJECT_ID)
cloud-build:
	echo $(CLOUD_PROJECT_ID)
	cd $(LIB_FSPATH) && gcloud builds submit --tag gcr.io/$(CLOUD_PROJECT_ID)/identicon-generator
cloud-deploy:
	cd $(LIB_FSPATH) && gcloud beta run deploy --image gcr.io/$(CLOUD_PROJECT_ID)/identicon-generator --platform managed

Hi @winwisely99 thanks for your comment!
Currently Cloud run is available in Europe but not in all zones.
The currently available zones are

Thanks for letting me know, I'll update the post accordingly to pick one of the available zones before continueing.

thanks that fixed it :)


Category annotation for functions

Is your feature request related to a problem?
The graphical view needs to know to which category a function belongs (e.g. data analysis, data visualization). This would also be useful to group functions in the documentation.
Desired solution

Add a new annotation Category.
Target should be Function.
Value should be a variant of a new enum.

Possible alternatives (optional)
No response
Screenshots (optional)
No response
Additional Context (optional)
No response

@GideonKoenig Which categories do you need for the toolbox of your graphical view?

:tada: This issue has been resolved in version 0.13.0 :tada:
The release is available on:

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.13.0
GitHub release

Your semantic-release bot :package::rocket:


Add a way to define env vars in values.yaml

I think that I may use the PR to group a few small but unrelated changes on which I am working.

I think that I may use the PR to group a few small but unrelated changes on which I am working.

## `global_node_config` defines global values which will be
## set for every node type in the `nodes` section below.
global_node_config:
    # Environment variables for every nodes' containers
    env: {}

?

Okay, I've split out all of the ancilliary work that I was putting in the PR into separate PRs. In order to write the functionality that has been requested vis a vis allowing environment variables to be set on a container by container basis---which I agree is necessary and will help us with many use cases, I thought that it would be a good idea to refactor the code even more than I did last time. I did this to minimise repetition of what will be some fun templating code for every single container that we mainain.
I'm not finished, yet, but please have a look at the existing refactoring and provide feedback.

Also, I don't like the names that I used for the config section "node_global" is a bit odd. Suggestions?

What if we just called them node_var or node_envs. Dont we know they are global just from using them?

I'm closing this to create a new PR.


Add test for parcel binary compatibility

Type of change

Enhancement / new feature

Description
Contributes to #992, still TODO equivalent tests for Wrapper
We need tests to prove that Kroxylicious can read older binary versions of Parcels. We also need tests to prove that it can write older binary versions. Comparing the output bytes is a blunt tool and will be fiddly to track down the reason for failures, but we must always be able to decode those bytes that we forwarded to Kafka to be stored forever, so starting and ending with the bytes is worth it to prove it really can.
The tests check:

an input Record, serialized with specified encryption configuration and parcel version produces expected bytes
the serialized bytes can be deserialized to produce the expected record features (Parcel deserializes in terms of value and headers only).

Note that input features may differ from deserialized features becase we can opt to not serialize headers or values despite them being present in the record.
The tests will immediately fail when a new ParcelVersion is added to the enum, requiring a new exemplar to be added for the new version to all existing json exemplars. In this case the test will display the serialized base64 in the failure message (if it can), which could be added to the exemplar file when you are happy with the output.
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Make sure all tests pass
[ ] Review performance test results. Ensure that any degradations to performance numbers are understood and justified.
[ ] Make sure all Sonarcloud warnings are addressed or are justifiably ignored.
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] For user facing changes, update CHANGELOG.md (remember to include changes affecting the API of the test artefacts too).

What do you think of the current setup where there is one JSON file per example containing:

details about the input record, value, headers etc
serialization options (serialize headers, values or some combination)
expected serialized output
expected deserialized features (when combined with a record with no headers)

It's a little tricky to make a pure test for the deserialization since the methods that do the work also know how to combine the headers from the Parcel with the headers of a record.
Maybe it would be better if the Parcel code wasn't responsible for the header combination logic, then we can more easily test that the headers are what's in the parcel.


[8.0.0-beta.4] fallbackLng is ignored

Describe the bug
dev is still used as a fallbackLng (missing string for a locale) even when setting fallbackLng: 'en'
Occurs in next-i18next version
8.0.0-beta.4
Steps to reproduce
Setting fallbackLng: 'en' in next-i18next.config.js.
Expected behaviour
Should use en as fallback language when a string is missing for a locale.

I cannot reproduce. If I checkout the next-v10 branch, and add next-i18next.config.js:
module.exports = {
  fallbackLng: 'en'
}

It works as expected. Please provide repro steps. Thanks!

I am having the same problem. In serverSideTranslations, only the store for initialLocale is populated.
namespacesRequired.forEach((ns) => {
    initialI18nStore[initialLocale][ns] = (
      (i18n.services.resourceStore.data[initialLocale] || {})[ns] || {}
    )
  })

I'll need someone to provide a reproducible repository.

I've been struggling with this since migrating over to the beta branch a few weeks ago (https://github.com/isaachinman/next-i18next/issues/869#issuecomment-767923052). I have tried a few times to identify why, and fix it myself, but without any success.
I put together this minimum reproduction real quick that is also failing to load fallbackLngs. Hope it helps! https://github.com/dmeents/fallback-ignored-example

Thanks for the repro, @dmeents. This is indeed a genuine bug, and I will fix it immediately.

A couple things to note:

I've re-added the functionality from current next-i18next to automatically set fallbackLng if the user doesn't pass a value, or false (to disable)
i18next technically supports fallbackLng as an array. I haven't written support for this yet, but would accept a PR from any user who desires this functionality


Usage of the value set binding

In several profiles the vocabulary binding is done at the level of coding.code
or a fixed system is used

I suggest to revise these profiles and  use - as needed - a value set binding at the level of CodeableConcept

updated the profiles in profiles.fsh
added new value sets

I merged your changes to mpd-r4b branch, so I could make my changes on top of yours and make sure they work together.
I did the manual merge together with my changes, I really hope I didn't forget anything.
A few bindings still need to be fixed, I'll take care of those very soon.

closed in R4B and merged


Sorting caret disappears when changing page by using the arrows.

You can reproduce the issue with the example found at : http://backgridjs.com/ref/extensions/paginator.html
It only happens when we change page by clicking on the pagination arrows, it works fine when changing page by clicking directly on a page number.
It might have some link with how the getPage() function is used when clicking a page number and that it is not called when clicking on the arrows. Take a look at this issue about this behavior : #17

It looks like the caret disappears if you don't pass {reset:true} in the fetch options. Looking at the code that handles the click (below), you can see that the only get<whatever>Page() that has {reset:true} is when you ask for a specific page, ie clicking on a page number. A solution would be to add {reset:true} to all of the get<>Page() arguments, but I'm not sure what the implications of that would be.
changePage: function (e) {
  e.preventDefault();
  var $el = this.$el, col = this.collection;
  if (!$el.hasClass("active") && !$el.hasClass("disabled")) {
    if (this.isRewind) col.getFirstPage();
    else if (this.isBack) col.getPreviousPage();
    else if (this.isForward) col.getNextPage();
    else if (this.isFastForward) col.getLastPage();
    else col.getPage(this.pageIndex, {reset: true});
  }
  return this;
}


[FR] Eliminate version control noise when changing platforms due to google-services-desktop being modified

Feature proposal

Firebase Component: Core

When changing platforms the google-services-desktop.json file in the streaming assets folder is modified to reflect the change in target platform.
This results in the file ping ponging around from being changed user to user with everyone having a different local configuration.
If this is for editor only emulation (suggested here https://github.com/firebase/quickstart-unity/issues/1092) then it should be stored somewhere else, perhaps the Temp folder.
If this is required for runtime, then it should be auto-generated at build time and cleaned up after the build is finished (similar to what Unity Cloud Services does - if you peek at the StreamingAssets folder during a build you can see that they inject some files there that they then clean up after the build is complete)

Thanks for the suggestion @timbo-tj
I won't disagree that how the google-services-desktop.json file is handled can be a bit annoying.  We can see if there are some improvements to be done, as you are suggesting.


Python3 support problem

I still can't make this work. On Debian, 10 I have:
vim --version | grep python
+comments          +libcall           -python            +visualextra
+conceal           +linebreak         -python3           +viminfo

and
python -V
Python 3.7.3

but Vim complains:
:python3 is not available, vdebug will not be loaded.

Any idea?

which package of vim do you have installed, I just installed vim-nox and that one has +python3

which package of vim do you have installed, I just installed vim-nox and that one has +python3

As far as I remember, it is the default Vim installed with Debian 10:
VIM - Vi IMproved 8.1 (2018 May 18, compiled Jun 15 2019 16:41:15)
Included patches: 1-875, 878, 884, 948, 1046, 1365-1368, 1382, 1401
Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>Huge version without GUI.  Features included (+) or not (-):
+acl               +extra_search      +mouse_netterm     +tag_old_static
+arabic            +farsi             +mouse_sgr         -tag_any_white
+autocmd           +file_in_path      -mouse_sysmouse    -tcl
+autochdir         +find_in_path      +mouse_urxvt       +termguicolors
-autoservername    +float             +mouse_xterm       +terminal
-balloon_eval      +folding           +multi_byte        +terminfo
+balloon_eval_term -footer            +multi_lang        +termresponse
-browse            +fork()            -mzscheme          +textobjects
++builtin_terms    +gettext           +netbeans_intg     +textprop
+byte_offset       -hangul_input      +num64             +timers
+channel           +iconv             +packages          +title
+cindent           +insert_expand     +path_extra        -toolbar
-clientserver      +job               -perl              +user_commands
-clipboard         +jumplist          +persistent_undo   +vartabs
+cmdline_compl     +keymap            +postscript        +vertsplit
+cmdline_hist      +lambda            +printer           +virtualedit
+cmdline_info      +langmap           +profile           +visual
+comments          +libcall           -python            +visualextra
+conceal           +linebreak         -python3           +viminfo
+cryptv            +lispindent        +quickfix          +vreplace
+cscope            +listcmds          +reltime           +wildignore
+cursorbind        +localmap          +rightleft         +wildmenu
+cursorshape       -lua               -ruby              +windows
+dialog_con        +menu              +scrollbind        +writebackup
+diff              +mksession         +signs             -X11
+digraphs          +modify_fname      +smartindent       -xfontset
-dnd               +mouse             +startuptime       -xim
-ebcdic            -mouseshape        +statusline        -xpm
+emacs_tags        +mouse_dec         -sun_workshop      -xsmp
+eval              +mouse_gpm         +syntax            -xterm_clipboard
+ex_extra          -mouse_jsbterm     +tag_binary        -xterm_save
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
  fall-back for $VIM: "/usr/share/vim"
Compilation: gcc -c -I. -Iproto -DHAVE_CONFIG_H   -Wdate-time  -g -O2 -fdebug-prefix-map=/build/vim-4Pursk/vim-8.1.0875=. -fstack-protector-strong -Wformat -Werror=format-security -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: gcc   -Wl,-z,relro -Wl,-z,now -Wl,--as-needed -o vim        -lm -ltinfo -lnsl  -lselinux -lacl -lattr -lgpm -ldl

could you give me the output of dpkg -l | grep vim ?

Sure:
ii  vim                              2:8.1.0875-5                  amd64        Vi IMproved - enhanced vi editor
ii  vim-addon-manager                0.5.10                        all          manager of addons for the Vim editor
ii  vim-common                       2:8.1.0875-5                  all          Vi IMproved - Common files
ii  vim-runtime                      2:8.1.0875-5                  all          Vi IMproved - Runtime files
ii  vim-tiny                         2:8.1.0875-5                  amd64        Vi IMproved - enhanced vi editor - compact version

Should I install vim-nox explicitly?

vim-tiny is definitely not the version you want to have, I think vim is a metapackage in debian? So I would go for vim-nox or if you want gui too for vim-gtk3, those should have python3 enabled

Sure:
ii  vim                              2:8.1.0875-5                  amd64        Vi IMproved - enhanced vi editor
ii  vim-addon-manager                0.5.10                        all          manager of addons for the Vim editor
ii  vim-common                       2:8.1.0875-5                  all          Vi IMproved - Common files
ii  vim-runtime                      2:8.1.0875-5                  all          Vi IMproved - Runtime files
ii  vim-tiny                         2:8.1.0875-5                  amd64        Vi IMproved - enhanced vi editor - compact version

Should I install vim-nox explicitly?

Thanks, with vim-nox it definitely works!


[Question] How to have a custom page title sent to GA ?

By default, from GA documentation, the page title which is being sent to GA (on a page view) is document.title.
The below code to provide custom title works inconsistently.
import { useLocation } from 'react-router-dom';
import { useGA4React } from 'ga-4-react';
. . .
      const location = useLocation();
      const ga4react = useGA4React('MEASURE_ID');
. . .
      ga4react.pageview(
        location.pathname + location.search,
        null,
        location.pathname, // because we do not have page title, use path instead. Can have custom logic
      );

The title being sent to GA is mostly empty, and sometimes, the page location is sent. Assigning it to a separate variable, just before pageview also doesn't helps.
Anything missed here ?

Hello, the pageview method, is nothing more than an "alias" of https://developers.google.com/gtagjs/reference/event#page_view
have you tried to use it instead?
ga4react .gtag ('event', 'page_view', {
page_position: "https://example.com/about",
page_path: "/ about",
page_title: "Information"
});
let me know

I tried with ga4react.gtag as well, but it didn't work. The only thing which worked was to set document.title. But that we cannot have in our project due to some other nomenclature for page titles.
Setting page title through GA methods dynamically did not work. Let me know, in case you can think of any other solution.

For SPA-like sites, the google documentation indicates this solution:
https://developers.google.com/analytics/devguides/collection/gtagjs/single-page-applications
gtag('config', 'UA-1234567-89', {'page_path': '/new-page.html'});

Appreciate swift responses, @unrealmanu
But this also doesn't works. If two pages are navigated in an interval less than 2-3 secs, the last page viewed is sent to GA (which is expected), but with incorrect title.
Quickly navigated pages get skipped is fine (because they can be ignored anyways), but for whatever is being sent to GA, should have correct dynamic page title.
Let me know, in case any other approach. Thanks


Making instanceTerminationAction for Compute Engine API nullable

Thanks for testing it.
I'm not sure if those specs can be changed. For now I'll keep patch on my side.


Cannot write Bengali in VSCode in Ubuntu

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.72.2 x64
OS Version: Ubuntu 22.04.1 LTS

I cannot write Bengali in VSCode. Recently I have been writing latex files in VSCode. I wanted to write something in Bangla. I used Avro Phonetic to write Bengali. But, when I try to do that it starts writing in English. I can write Bengali with that keyboard anywhere else. ex: হেলো। But, if I try to write the same in VSCode it just writes "HelO" instead of "হেলো".
I have searched for many solutions online. But, I have found none that could solve my problem. Because, there's hardly any need  to write Bengali in VSCode by Bengali developer. So, I had to open an issue here.
Steps to Reproduce:

Open VSCode and open a writable file.
Press super+space change to bn input source.
Try write something in Bengali.

@AAShakil50 Does this work when using a native <input>, like for example typing in the input of the find widget (Ctrl+F) ?

No. It does not work in the Find input either. I have tried also in other inputs like, search, command pallete, new_file. Everywhere it types in English instead of Bengali.

@AAShakil50 Are you using VS Code as a snap package? Does IME work when downloading the .tar.gz variant from https://code.visualstudio.com/#alt-downloads?

It works as expected when using .tar.gz. Thank you so much.

Let's track in https://github.com/microsoft/vscode/issues/96041


Introduce the @clerk/testing package

Description
This PR introduces the @clerk/testing package. It will provide utilities for testing Clerk applications.
It currently supports only Playwright. Will support Cypress in the future.
How to use it
Prerequisites

Node.js >=18.17.0 or later
Playwright v1+

Installation
npm install @clerk/testing --save-dev

Usage
On your global setup file for Playwright, you must use the clerkSetup function to set up Clerk for your tests.
// global-setup.ts
import { clerkSetup } from '@clerk/testing/playwright';
import { test as setup } from '@playwright/test';

setup('global setup', async ({ }) => {
  await clerkSetup();
  ...
});

Then, you can use the clerkBypassBotProtection function to bypass bot protection on your tests.
// my-test.spec.ts
import { clerkBypassBotProtection } from "@clerk/testing/playwright";
import { test } from "@playwright/test";

test("sign up", async ({ page }) => {
  await clerkBypassBotProtection({ page });

  await page.goto("/sign-up");
  ...
});

Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

!snapshot

!snapshot

Hey @anagstef - the snapshot version command generated the following package versions:

Package
Version

@clerk/chrome-extension
1.0.1-snapshot.v07d9254

@clerk/clerk-js
5.1.0-snapshot.v07d9254

@clerk/clerk-expo
1.0.1-snapshot.v07d9254

@clerk/fastify
1.0.1-snapshot.v07d9254

gatsby-plugin-clerk
5.0.0-beta.45

@clerk/localizations
2.1.0-snapshot.v07d9254

@clerk/nextjs
5.0.2-snapshot.v07d9254

@clerk/clerk-react
5.0.1-snapshot.v07d9254

@clerk/remix
4.0.1-snapshot.v07d9254

@clerk/testing
2.0.0-snapshot.v07d9254

@clerk/themes
2.0.1-snapshot.v07d9254

@clerk/types
4.1.0-snapshot.v07d9254

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/chrome-extension
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-js
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-expo
npm i<EMAIL_ADDRESS>--save-exact

@clerk/fastify
npm i<EMAIL_ADDRESS>--save-exact

gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>--save-exact

@clerk/localizations
npm i<EMAIL_ADDRESS>--save-exact

@clerk/nextjs
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-react
npm i<EMAIL_ADDRESS>--save-exact

@clerk/remix
npm i<EMAIL_ADDRESS>--save-exact

@clerk/testing
npm i<EMAIL_ADDRESS>--save-exact

@clerk/themes
npm i<EMAIL_ADDRESS>--save-exact

@clerk/types
npm i<EMAIL_ADDRESS>--save-exact

!snapshot

Hey @anagstef - the snapshot version command generated the following package versions:

Package
Version

@clerk/chrome-extension
1.0.1-snapshot.v1b21f39

@clerk/clerk-js
5.1.0-snapshot.v1b21f39

@clerk/clerk-expo
1.0.1-snapshot.v1b21f39

@clerk/fastify
1.0.1-snapshot.v1b21f39

gatsby-plugin-clerk
5.0.0-beta.45

@clerk/localizations
2.1.0-snapshot.v1b21f39

@clerk/nextjs
5.0.2-snapshot.v1b21f39

@clerk/clerk-react
5.0.1-snapshot.v1b21f39

@clerk/remix
4.0.1-snapshot.v1b21f39

@clerk/testing
0.1.0-snapshot.v1b21f39

@clerk/themes
2.0.1-snapshot.v1b21f39

@clerk/types
4.1.0-snapshot.v1b21f39

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/chrome-extension
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-js
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-expo
npm i<EMAIL_ADDRESS>--save-exact

@clerk/fastify
npm i<EMAIL_ADDRESS>--save-exact

gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>--save-exact

@clerk/localizations
npm i<EMAIL_ADDRESS>--save-exact

@clerk/nextjs
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-react
npm i<EMAIL_ADDRESS>--save-exact

@clerk/remix
npm i<EMAIL_ADDRESS>--save-exact

@clerk/testing
npm i<EMAIL_ADDRESS>--save-exact

@clerk/themes
npm i<EMAIL_ADDRESS>--save-exact

@clerk/types
npm i<EMAIL_ADDRESS>--save-exact


Feature added support for TOUCH and OBJECT IDLETIME

The changes that are included in the PR.
Added support for the TOUCH command as per Redis and OBJECT IDLETIME key as per Redis.
Added tests for both commands in tests folder.

Design document, if any.

Information on any implementation choices that were made.
The reason to implement  OBJECT IDLETIME key, standalone from other subcommands is due to no support for ref and freq for keys and object for now. However, I assumed that having object idletime is essential to verify the effect of TOUCH command. In addition to this, I have carefully structured OBJECT for extension for subcommands in the future.

Evidence of sufficient testing. You MUST indicate the tests done, either manually or automated.
Added new tests

Hi @JyotinderSingh, I have added new command support to the Dice. Please review and share your thoughts on the implementation at your convenience.

Hi @JyotinderSingh, I have added new command support to the Dice. Please review and share your thoughts on the implementation at your convenience.

Looks like the checks and linter are failing, could you take a look?

Hi @JyotinderSingh, I have checked the linter and build issue. The build issue was due to the session_test.go file at line 107, which is recently fixed by #318. For the linter, the issue comes from eval.go line 82 core/eval.go:82:6: ineffectual assignment to password (ineffassign) (at least from my local lint run). I could not access much logs from the GitHub run.
I can fix the build issue by rebasing my branch at the end and after addressing any comments, before Merge. Meanwhile I can also look into the lint issue.

Hi @JyotinderSingh , I have addressed the comments and ran the unit and integration tests locally along with the linter. Please review the changes.


Use filter_var for Email

The email validation can be done via filter_var.
Example:
if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
    // invalid emailaddress
}

This is implemented


Searcharr Keeps Restarting

running version 3.2.2 on a synology nas in a docker container. searcharr has been working great, for many many months. then last week it keeps restarting indefinitely. my sonarr version <IP_ADDRESS>9-ls224.... I have attached logs below. I also verified my api keys were correct, and revoked the token from the telegram bot and made a new one. Any advice on what to try next?
`

2024-02-13T23:33:38.627271681Z
stdout
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: http://192.168.1.XXX:8989/api/profile?apikey=XXXXXXXXXXXXX

2024-02-13T23:33:38.627162566Z
stdout
raise HTTPError(http_error_msg, response=self)

2024-02-13T23:33:38.626584242Z
stdout
File "/usr/local/lib/python3.11/site-packages/requests/models.py", line 1021, in raise_for_status

2024-02-13T23:33:38.626362562Z
stdout
r.raise_for_status()

2024-02-13T23:33:38.626095719Z
stdout
File "/app/sonarr.py", line 250, in _api_get

2024-02-13T23:33:38.626018693Z
stdout
^^^^^^^^^^^^^^^^^^^^^^^^^^^^

2024-02-13T23:33:38.625919403Z
stdout
return self._api_get("profile", {}) or None

2024-02-13T23:33:38.625825651Z
stdout
File "/app/sonarr.py", line 234, in get_all_quality_profiles

2024-02-13T23:33:38.625746062Z
stdout
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

2024-02-13T23:33:38.625658110Z
stdout
self._quality_profiles = self.get_all_quality_profiles()

2024-02-13T23:33:38.625568371Z
stdout
File "/app/sonarr.py", line 26, in init

2024-02-13T23:33:38.625449343Z
stdout
sonarr.Sonarr(settings.sonarr_url, settings.sonarr_api_key, args.verbose)

2024-02-13T23:33:38.625351529Z
stdout
File "/app/searcharr.py", line 65, in init

2024-02-13T23:33:38.625272477Z
stdout
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

2024-02-13T23:33:38.625176650Z
stdout
tgr = Searcharr(settings.tgram_token)

2024-02-13T23:33:38.624985597Z
stdout
File "/app/searcharr.py", line 1567, in

2024-02-13T23:33:38.623689845Z
stdout
Traceback (most recent call last):

`

404 Client Error: Not Found for url: http://192.168.1.xxx:8989/api/profile?apikey=XXXXXXXXXXXXX

The error message indicates Searcharr can't make an API call to Sonarr. Did you try the unredacted URL in your browser to make sure it works?

thank you for the very quick reply. Yes I did try that url in the browser but I am not sure what I am supposed to be seeing? It is just a blank page. It changes the browsers icon to a sonarr icon, and loads up, but nothing is there.
When I changed the port to radarrs port of 7878 and changed the api key to radarrs api key, the same thing happens. Blank page loads and browser icon in the tab changes to a radarr logo.

Are you sure you're running Searcharr v3.2.2? Your stack trace appears to be from an older version before support was added for the new API endpoints in Sonarr v4.

In Searcharr v3.2.2 it should be making a call to http://192.168.1.xxx:8989/api/v3/qualityprofile?apikey=XXXXXXXXXXXXX instead.

I updated the container using portainer, and it pulled with the "latest" tag. As far as I know I am running 3.2.2
Is there a way to find out?
In the mean time, I am going to create a whole new container and post back results. thank you again for the prompt responses.

The first thing in your log file when Searcharr starts up should be a line that says "Searcharr vX.XX - Logging started!"

Okay, so making a secondary searcharr2 container worked just fine - I had to re-authenticate as a user in telegram and verified by trying to add something that was already in sonarr, and was presented with "already added!" button. so thats great and that works for me and I am happy.
For some reason I cannot figure out the version of the original searcharr container. Synology's docker gui gives me a log section but it doesnt have anything about "Logging Started" at the top or bottom of the log, and in searcharr/logs folder I have nothing.

If you add the -v flag to the command in your docker run command or compose yaml, you'll get logging to the console. There's a wiki page about enabling debug logging that shows how to do it (to add the -v but the steps are the same).
You can copy the searcharr.db file to your new container's data volume to avoid having to authenticate again.


Check for classes without a state

lets break the build with the following code:
class Cat {
  public void meow() {
  }
}

@h1alexbel thank you for reporting this. I'll assign someone to take care of it soon.

@l3r8yJ this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2024-01-07T10:40:04.515498.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@l3r8yJ Don't forget to close this ticket before the deadline (2024-01-07T10:40:05). You are past the first half of the allowed period.

@l3r8yJ Looks like you've missed the task deadline (2024-01-07T10:40:05). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@h1alexbel this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2024-01-17T11:00:25.132056.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@h1alexbel Don't forget to close this ticket before the deadline (2024-01-17T11:00:25). You are past the first half of the allowed period.

@h1alexbel Looks like you've missed the task deadline (2024-01-17T11:00:25). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@l3r8yJ I couldn't find any assignee for this task. This is either because there are no contributors with role DEV available or because the project does not have enough funds.
Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.


Removed conditional imports in ObjC files

Closes https://github.com/ammarahm-ed/react-native-mmkv-storage/issues/258

This PR removes old import style for react native header files and standardises imports to be # import <React/xyz.

Perfect. thanks


Add encode function and some tiny fix

Add MEncode function to reduce template code. Change all the function of wasmcloud to pointer receiver.

Just a note: this project hasn't enabled the DCO GitHub app to check pr.

@Taction this is GREAT! thanks so much!  I wasn't expecting anyone to PR that quickly, so I didnt get the branch protections set up yet. Ill get that fixed!
If possible, could you rebase on main and I will get this merged...THANKS!

@jordan-rash Rebased


Add latest Python classifiers.

Also, bump to 0.4.10pre1 to see if this will work to fix pip install for 3.7.

I've manually confirmed that this version works with 3.7.0.


Converters, quantum_info, assembler, scheduler and circuit library updates for optional registers.

Summary
Following #5498 , updates converters, quantum_info, assembler, circuit library and scheduler to support optional registers, and to prefer inspecting bits over registers where possible.
Details and comments
~on-hold until #5498 merges.~
Commits in this PR (without #5498 and #5486 ): https://github.com/Qiskit/qiskit-terra/pull/5504/files/18c95db0f2daef7fe52674ca1ec7fb5e7f1eae3e..5c5ac703ff05a25bdea6ae57346440d15d45033f

I know that means we have 3 views of a the bits in a circuit which seems like a headache. But maybe we can combine _qubit_set and the map (like add a getter for qubit set that returns set(map)).

I agree, and the cost of keeping these in sync was why I held off initially (this was before the addition of _qubit_set), assuming that if generating these dicts were impacting performance, they could be consolidated as a private attribute later. I can add a PR to the end of the set to clean these up (and consolidate with _qubit_set) unless you think its better to fix them now.

I would probably just do it in this PR I feel like that'll be simpler unless it has implications for the next patches in the series.

I would probably just do it in this PR I feel like that'll be simpler unless it has implications for the next patches in the series.

I'm a little worried about introducing more conflicts with the remaining PRs, maybe as part of #5868 ?

I would probably just do it in this PR I feel like that'll be simpler unless it has implications for the next patches in the series.

I'm a little worried about introducing more conflicts with the remaining PRs, maybe as part of #5868 ?

Ok sure sounds good, then this LGTM


Spell Checker

In taking a look at the project, there are a few typos in type names that potentially cause a lot of churn to fix. It's better to catch them now, than have to touch a thousand files later when entitiy is used in a hundred places.
Not trying to be a grammar nazi or anything in comments, but type names matter!
I've separated the spell corrections to different commits to minimize the number of files touched by each commit.
If you don't want to have a bunch of commits for spelling mistakes, make sure to commit and squash.

Hey. Looks good. Thanks. I just writen coden too fast too often haha

But you have to resolve the conflicts and fix the CI before i can merge

I will rebase when I'm home.

Great. CI is passing now, Just fix the conflicts then again

If you're going to merge other PRs at 6AM my time, and I rebase at 6PM my time, this is going to go on for a while. Feel free to rebase yourself. It's not like these are complex changes.

Is this ready for merge ?

I don't see why it wouldn't be. Go ahead when you're ready.


@pa.dataframe_check fails to identify issues in rows with NAs

Describe the bug
When using the @pa.dataframe_check for a custom SchemaModel, validation is not correctly performed on rows if their any of their columns contains NAs/None. These columns need not be defined in the SchemaModel and need not be referenced with the decorated validation function.

[x] I have checked that this issue has not already been reported.
[x] I have confirmed this bug exists on the latest version of pandera.
[ ] (optional) I have confirmed this bug exists on the master branch of pandera.

Note: Please read this guide detailing how to provide the necessary information for us to reproduce your bug.
Code Sample, a copy-pastable example
import pandera as pa
import pandas as pd

class TestSchema(pa.SchemaModel):
    x: pa.typing.Series[int] = pa.Field(nullable=False)

    @pa.dataframe_check
    def fail_all(cls, df: pd.DataFrame) -> pa.typing.Series[bool]:
        # Use a dataframe validator that should fail every row to demonstrate the issue
        return df["x"].map(lambda x: False)

    class Config:
        strict = False # Allow extra columns

df1 = pd.DataFrame({"x": [1, 2, 3], "y": [None, None, None]})
TestSchema.validate(df1)
# Passes validation even though each row should fail. Likely occurs b/c y=None for each row

df2 = pd.DataFrame({"x": [1, 2, 3], "y": [None, 0, None]})
TestSchema.validate(df2)
# Only flags row at index 1 as failing. Other rows with y = None are not flagged
# failure cases:
# column  index  failure_case
# 0      x      1           2.0
# 1      y      1           0.0

df3 = pd.DataFrame({"x": [1, 2, 3], "y": [0, 0, 0]})
TestSchema.validate(df3)
# Correctly flags all 3 rows as failing b/c y is no longer None/NA
# failure cases:
  column  index  failure_case
0      x      0             1
1      x      1             2
2      x      2             3
3      y      0             0
4      y      1             0
5      y      2             0

Expected behavior
In the example above, I expected df1, df2 and df3 to result in the same validation failure cases, with all 3 rows resulting in errors. Column y should have no bearing on the validation results as it's not referenced within fail_all or the schema.
With the current behavior, @dataframe_check validators are likely ignoring every row with an NA in any column
Desktop (please complete the following information):

MacOS Mojave
chrome

Screenshots
NA
Additional context
I suspect this is somehow related to a pandas.groupby, in which dropna defaults to True and thus any rows with an NA will be ignored.

Looks like I missed the mark here ... adding ignore_na=True to @pa.dataframe_check solved all of my issues.


Request: Opus Codec

Opus is a modern audio codec with better compression at the same apparent audio quality. 96kbps fared well in HydrogenAudio (audio enthusiast community) tests.
https://www.opus-codec.org/
It's built into ffmpeg as well.

Hm, I hadn't researched Opus that much and it seems promising. I might as well replace the default libvorbis method with opus if it does indeed offer better results!

V1.2.2 now uses the OPUS codec with some predefined quality bitrates


GET CloudStampy Secrets from KeyVault instead of SecretStore

On of the instances, the bootstrapper.exe started at 21:31:39 and OnStart code started at 21:40:35. Almost a 10 mins delay.
Delay might be coming from secretstore. Get secrets from KeyVault as part of the deployment and put secrets into environment variables

https://docs.microsoft.com/en-us/azure/azure-resource-manager/resource-manager-keyvault-parameter


Add guideline regarding "optional" and "required" in form components

As per discussion, add guideline to documentation how to mark optional and required fields in a form.
Add to documentation of:

[x] Text input
[x] Text area
[x] Select
[x] Checkbox
[x] Radio
[x] Switch

I added a comment regarding the Notification für the Checkbox... https://www.figma.com/design/JIT5AdGYqv6bDRpfBPV8XR/Foundations-%26-Components-Next-Level?node-id=5485-60641&t=bktSFWkp6UeN7Zry-1


iPad Magic Keyboard and Extended Keys

It's not the VM, it's very simple. With the magic keyboard, the bottom keyboard keys bar does not disappear and comes back with the keyboard interaction and it overlays on top of the VM.

Configuration (required)

UTM Version:  v3.1.5
OS Version: 15.5 (19E241)
Device Model: iPad Pro 11 inch 2nd generation my232vc/a
Is it jailbroken (name jailbreak used)?  No
How did you install UTM? iPA file then Altserver and altstore.

Does the bar go away when you tap the downwards arrow on the right? (bottom right corner in your screenshot)

I tested this myself (with a bluetooth keyboard), the bar does go away when I tap the downwards arrow.


Surface FSharpCheckFileResults in Context

The Context type surfaces several fields that are based on FSharpCheckFileResults, but not FSharpCheckFileResults itself. This type has several useful methods, such as GetSymbolUseAtLocation, that should be available to analyzers. I therefore propose surfacing this type in Context.

Done in https://github.com/ionide/FSharp.Analyzers.SDK/pull/73


Adding edge case handling for string inputs into norm

Description
Raising an error for non-supported string arguments into norm atom, and handling the fro norm.
Addressing issue: https://github.com/cvxpy/cvxpy/issues/1865
Type of change

[ ] New feature (backwards compatible)
[ ] New feature (breaking API changes)
[x] Bug fix
[ ] Other (Documentation, CI, ...)

Contribution checklist

[ ] Add our license to new files.
[ ] Check that your code adheres to our coding style.
[ ] Write unittests.
[ ] Run the unittests and check that they’re passing.
[ ] Run the benchmarks to make sure your change doesn’t introduce a regression.

 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.EmilKerimov seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Todo: Prevent view from clipping into the ground with CalcView

Player view clips into the ground when they are very small, but this can be fixed with the CalcView hook

Done


DB: Implement Drizzle Schema

most of the initial work is done for bootstrapping the schema, move to review with some minor future changes


WIP: Cryptofuzz tests integration

Fixes #9744
Checklist

[ ] documentation is added or updated
[x] tests are added or updated

This is looking pretty good.  You're missing something, though, according to Travis:
95-test_external_cryptofuzz.t .. 1/1
1..1
make[2]: Entering directory '/home/travis/build/openssl/openssl/cryptofuzz/modules/openssl'
In file included from ../../fuzzing-headers/include/fuzzing/types.hpp:7:0,
                 from ../../fuzzing-headers/include/fuzzing/datasource/datasource.hpp:5,
                 from ../../include/cryptofuzz/generic.h:5,
                 from ../../include/cryptofuzz/components.h:3,
                 from module.h:3,
                 from module.cpp:1:
../../fuzzing-headers/include/fuzzing/memory.hpp:5:20: fatal error: optional: No such file or directory
compilation terminated.
Makefile:10: recipe for target 'module.o' failed
make[2]: *** [module.o] Error 1
make[2]: Leaving directory '/home/travis/build/openssl/openssl/cryptofuzz/modules/openssl'
make[2]: Entering directory '/home/travis/build/openssl/openssl/cryptofuzz'
In file included from fuzzing-headers/include/fuzzing/types.hpp:7:0,
                 from fuzzing-headers/include/fuzzing/datasource/datasource.hpp:5,
                 from driver.h:5,
                 from driver.cpp:1:
fuzzing-headers/include/fuzzing/memory.hpp:5:20: fatal error: optional: No such file or directory
compilation terminated.
Makefile:6: recipe for target 'driver.o' failed
make[2]: *** [driver.o] Error 1
make[2]: Leaving directory '/home/travis/build/openssl/openssl/cryptofuzz'
/home/travis/build/openssl/openssl/util/shlib_wrap.sh sh /home/travis/build/openssl/openssl
test/recipes/95-test_external_cryptofuzz_data/cryptofuzz.sh => 2
not ok 1 - running Cryptofuzz tests

This is looking pretty good. You're missing something, though, according to Travis:

Yeah, the problem is that Cryptofuzz uses the C++ 17 optional header, and older versions of GCC support it by providing <experimental/optional> instead of <optional>.
I think I can resolve this on the fly with sed.
However, there is another issue ahead: GCC does not have libFuzzer built-in, while (recent) clang does (with -fsanitize=fuzzer). Of course we are not going to fuzz in Travis, but I need libFuzzer as a driver to load and run inputs in the corpus. Would it be possible to compile the external tests, or at least Cryptofuzz, with clang? This might also spontaneously resolve the issue with <optional>.
Furthermore, for this effort to be as effective as possible, I'd prefer to compile OpenSSL with -fsanitize=address,undefined. Are you OK with this, and should I either:

Alter the existing rules in .travis.yml to use clang + sanitizers
Create a separate target for Cryptofuzz

I have zero problems switching that job to use clang.

We have a fuzz/test-corpus.c file that we use, is that useful for you?

@kroeckx I had to use clang to work around the  issue, and it turns out the version of clang that Travis uses is recent enough to support -fsanitize=fuzzer, so that problem is solved. Thanks though.
The build seems to be succeeding (Travis signals failure because the Pyca test segfaults, I think). The next steps are adding a corpus (I'll add this as a submodule to Cryptofuzz, so it gets downloaded automatically), and running Cryptofuzz against this corpus.
What do you think about the sanitizers? I can add a new target specifically for Cryptofuzz, or add sanitizers to the existing build rule for external tests. The latter alternative will unnecessarily slow down the building/running of the existing tests though. Let me know, thanks.

I don't think we want to slow down the existing tests.

We already have a target with ubsan, asan and msan, I don't see a
problem with an other target enabling them. I'm also fine with the
other external tests running with them enabled.
Maybe it's useful that not all external tests get run in the same
travis job.

OK, I'll modify the script now to build with the sanitizers. We can always revert and create a separate rule in the final stage.

Maybe it's useful that not all external tests get run in the same travis job.

Agreed, especially if there are one or more broken tests (as seems to be the case with Pyca). The Travis job will fail regardless of whether a PR contains bugs.

Could you please rebase the PR to resolve conflicts?
Marking as inactive to be closed at the end of 3.4 dev barring further input.

@guidovranken your toolkit has found bugs before, right?  interested in bring it to quictls?


I don't try to be disrespectful , i use sqli spiders to many years , before google try to stop them.
I show an example and newbies like you can build a sqli spider.
Give respect and not trying to scamming the mind of people.


Unpacking tuple with asterisk causes "Tuple size mismatch error"

Environment data

Pylance: 1.054.404
Language Server version: Pylance language server 2021.2.0 (pyright e278f28d)
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): 3.7

Expected behaviour
l1: List[int] = [1, 2, 3]
l2: List[str] = ['test', 'test2', 'test3']
res: List[Tuple[int, str]] = list(zip(*[l1, l2]))

The return type of list(zip(*[l1, l2])) should be List[Tuple[int, str]].
Actual behaviour
The return type of list(zip(*[l1, l2])) is detected as list[Tuple[Unknown]]
This results in the following type warning:

Expression of type "list[Tuple[Unknown]]" cannot be assigned to declared type "List[Tuple[int, str]]"
TypeVar "_T@list" is invariant
Tuple size mismatch; expected 2 but received 1
Pylance(reportGeneralTypeIssues)

For pyright a similar (same title) issue already existed in the past: https://github.com/microsoft/pyright/issues/627
Additional failing MWE:
l1: List[int] = [1, 2, 3]
l2: List[str] = ['test', 'test2', 'test3']

def test() -> List[List[Any]]:
    return [l1, l2]

res: List[Tuple[int, str]] = list(zip(*test()))

Expression of type "list[Tuple[Any]]" cannot be assigned to declared type "List[Tuple[int, str]]"
TypeVar "_T@list" is invariant
Tuple size mismatch; expected 2 but received 1

The code above is not unpacking a tuple; it's unpacking a list. If you change the operand of the upack operator to a tuple, it will type check as expected.
l1: List[int] = [1, 2, 3]
l2: List[str] = ['test', 'test2', 'test3']
res: List[Tuple[int, str]] = list(zip(*(l1, l2)))

Likewise, you can make the second example work as follows:
from typing import List, Tuple

l1: List[int] = [1, 2, 3]
l2: List[str] = ["test", "test2", "test3"]

def test() -> Tuple[List[int], List[str]]:
    return (l1, l2)

res: List[Tuple[int, str]] = list(zip(*test()))

Neither of your samples type check within mypy either. If you look at the type signature for zip in builtins.pyi, you can figure out why.
I think Pyright is doing the right thing here.

Thanks for the fast reply. I had a more complicated case. I will check tomorrow if your suggestion also would fix that case.
I will come back to this as soon as possible.

@erictraut This does not seem to be sufficient. My MWE was not an appropriate representation of my use-case.
The method implemented is generic and can return any type of list and also any number of lists.
Still trying to give a simplified MWE here:
def test(*args: List[Any]) -> Tuple[List[Any], ...]:
    # Complicated logic here
    return tuple(args)

l1: List[int] = [1, 2, 3]
l2: List[str] = ['test', 'sfd', 'sfds']

res: List[Tuple[int, str]] = list(zip(*test(l1, l2)))

The MWE is now returning a Tuple but it is still not working.
The length (number of elements) is dynamic but the parameters supplied are static. So it can be possible to infer that exactly two object are applied.
I would have expected this to work. At least it would be nice. Don't you think?

Expression of type "list[Tuple[Any]]" cannot be assigned to declared type "List[Tuple[int, str]]"
TypeVar "_T@list" is invariant
Tuple size mismatch; expected 2 but received 1

The problem here is that your test method is returning an unknown number of List[Any] values (as indicated by the ...). The function zip is defined in builtins.pyi using a series of overloads. Each overload uses a different number of input arguments, corresponding to the number of iterables being zipped together. In your sample, the type checker doesn't know how many iterables are being passed to zip because of the ....
You can see the problem by deconstructing the expression in the last line of your sample:
v1 = test(l1, l2)
reveal_type(v1) # Type of "v1" is "Tuple[List[Any], ...]"
v2 = zip(*v1)
reveal_type(v2) # Type of "v2" is "Iterator[Tuple[Any]]"
v3 = list(v2)
reveal_type(v3) # Type of "v3" is "list[Tuple[Any]]"
res: List[Tuple[int, str]] = v3 # Error: Type "list[Tuple[Any]]" cannot be assigned to type "List[Tuple[int, str]]"

If you modify test to return a specified number of iterables, the sample will type check:
def test(*args: List[Any]) -> Tuple[List[Any], List[Any]]: ...

One potential solution is for you to use overloads in your implementation:
@overload
def test(v1: List[Any]) -> Tuple[List[Any]]:
    ...
@overload
def test(v1: List[Any], v2: List[Any]) -> Tuple[List[Any], List[Any]]:
    ...
@overload
def test(v1: List[Any], v2: List[Any], v3: List[Any]) -> Tuple[List[Any], List[Any], List[Any]]:
    ...
def test(*args: List[Any]) -> Tuple[List[Any], ...]:
    # Complicated logic here
    return tuple(args)

Thanks for the suggestion. You always learn something new. I actually wasn't aware that method overloading is possible in python. Thanks for that hint.
Nonetheless, I don't think that is reasonable here. For a generic method that can take an unspecified amount of input variables this is just not feasible.
Since the problem here seems to be the implementation of the build-in zip, I switched to itertools.zip_longest which outputs Iterable[Any]. This does resolve the type warning.
from itertools import zip_longest

def test(*args: List[Any]) -> List[List[Any]]:
    # Complicated logic here
    return list(args)

l1: List[int] = [1, 2, 3]
l2: List[str] = ['test1', 'test2', 'test3']

res: List[Tuple[int, str]] = list(zip_longest(*test(l1, l2)))

I guess that is a workaround I can live with. I'm going to close this issue. Again, thank you for your time and suggestions!

@erictraut Is this something that will be improved by variadics? E.g., make *args be variadic in the number of lists it takes, then the return type unpacks it into a tuple.

@jakebailey, yeah, kind of. With Python 3.10 and PEP 646, we could define test as follows:
_Ts = TypeVarTuple("_Ts")

def test(*args: *_Ts) -> Tuple[*Ts]:
    ...

The problem is that TypeVarTuple  (at least in the current draft of the PEP) doesn't support bound, so there's no way to specify that each parameter passed to *args must be a list or an Iterable. That means the implementation of test would need to include some ugly cast calls. So it's not great, but it's probably better than what can be done in Python 3.9.

That's what I feared when I tried to write out an example... I couldn't figure out where the List was supposed to go. Darn.

@gvanrossum, @mrahtz, you might find this thread interesting as it pertains to PEP 646.

So the crux is that we need TypeVarTuple to take a bound=... argument. Or we could go back to using my proposal, TypeVar(bound=Tuple), because then we could just use bound=Tuple[Iterable, ...] in this example.

Thanks for the heads up, Eric.
I'd prefer not to add bound= to PEP 646 itself because I suspect we might want finer-grained control when using TypeVarTuple for array typing (e.g. bounds for individual types in the TypeVarTuple), and a) I want to get PEP 646 wrapped up, and b) I'd prefer to wait until we've used TypeVarTuple in the wild a bit more before making a decision on this.
What I have done is start a doc here on notes for a potential future PEP detailing how bounds and variance should work for TypeVarTuple referencing this issue so we have an example use-case.


putting nav links in active mode when page is loaded

@jdorfman per your email

:+1: :ship:

What is the reasoning for this?

The nav button for the current page wasn't shown as active when the page was loaded. This makes it so.

We should make use of the active class then via pug.
On Aug 8, 2016 00:47, "Joshua Mervine"<EMAIL_ADDRESS>wrote:

The nav button for the current page wasn't shown as active when the page
was loaded. This makes it so.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MaxCDN/bootstrap-cdn/pull/635#issuecomment-238110648,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVVtX1t_7EmZwy9UiqZKINsDj941Y3Pks5qdlJlgaJpZM4HXtSx
.

I considered that when breaking things out in to their own pages, but it was actually more work/code to pass active through as a local in the route. That said, I have no strong preference one way or another.


HikariCP  slow Sometimes on a low-load service

Environment
springboot:2.7.1
HikariCP version:4.0.3
JDK version     : 1.8.0_111
Database        : MySQL

config
 hikari:
      pool-name: hikariCP
      minimum-idle: 5
      idle-timeout: 180000
      maximum-pool-size: 10
      auto-commit: true
      max-lifetime: 1800000
      connection-timeout: 30000

there are some requests that can be very slow in a service that is only a few dozen times a minute.
Try to monitor through Skywalking,there is a step: select 1 (Guess this is verifying whether the connection is OK) took 30ms  , found that in the time-consuming long requests.
There is no slow sql in mysql's monitoring either.
I can't find the reason, can anyone help me?

But i don't set "connectionTestQuery" .
The slowly request cost 32ms ,and select 1 cost 30. So i think it is reason.
What is 'a profiler'？How do it？ Can you give me some help？


Podboat: Support file deletion

Implements feature request from issue #669.

I'm doubtful about the new setting's name, I need to sleep on it.

I don't like the setting's name either so I'm happy if you come up with something better. (I didn't think of a better name yet)

Also, this feature is a good candidate to be unit-tested. We don't have any coverage for QueueLoader yet, but FileUrlReader is quite similar, and you can take a look at this test as an example. It's fine if you don't—I only require tests for modules that already have some tests, so this PR is fine without.

I'm planning to write some tests but before that I think some small refactoring is useful to make writing tests easier.
I think it is better if I open a separate pull request for writing test+refactor.

I think the setting's name should be delete-played-files. It's a bit misleading, since finished files are deleted as well, but that can be amended with docs.

I'm planning to write some tests but before that I think some small refactoring is useful to make writing tests easier.

That's risky: you might inadvertently break something during refactoring, and won't know about it because there are no tests. Writing tests first doesn't eliminate the risk completely, but lowers it (significantly IMHO).
You can try your approach, but before submitting the PR, please run a simple check: remove all refactorings (with interactive rebase, or by cherry-picking tests onto "master"), and make sure the tests pass.
It might be useful to look at code coverage while writing tests. You can either submit a PR marked as draft, and view Coveralls reports online (the bot will post a link after CI build); or run test/generate_coverage_report.sh locally.

I think it is better if I open a separate pull request for writing test+refactor.

Yeah, definitely. Thanks!


http://www.franceculture.fr/emission-l-atelier-fiction-true-love-will-find-you-in-the-end-2014-09-09

[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['http://www.franceculture.fr/emission-l-atelier-fiction-true-love-will-find-you-in-the-end-2014-09-09', '-v']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2014.09.16
[debug] Python version 2.7.3 - Linux-3.2.0-4-amd64-x86_64-with-debian-7.6
[debug] Proxy map: {}
[generic] emission-l-atelier-fiction-true-love-will-find-you-in-the-end-2014-09-09: Requesting header
WARNING: Falling back on generic information extractor.
[generic] emission-l-atelier-fiction-true-love-will-find-you-in-the-end-2014-09-09: Downloading webpage
[generic] emission-l-atelier-fiction-true-love-will-find-you-in-the-end-2014-09-09: Extracting information
ERROR: Unsupported URL: http://www.franceculture.fr/emission-l-atelier-fiction-true-love-will-find-you-in-the-end-2014-09-09; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
  File "./youtube-dl/youtube_dl/extractor/generic.py", line 545, in _real_extract
    doc = parse_xml(webpage)
  File "./youtube-dl/youtube_dl/utils.py", line 1509, in parse_xml
    tree = xml.etree.ElementTree.XML(s.encode('utf-8'), **kwargs)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1301, in XML
    parser.feed(text)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1643, in feed
    self._raiseerror(v)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1507, in _raiseerror
    raise err
ParseError: mismatched tag: line 61, column 2
Traceback (most recent call last):
  File "./youtube-dl/youtube_dl/YoutubeDL.py", line 524, in extract_info
    ie_result = ie.extract(url)
  File "./youtube-dl/youtube_dl/extractor/common.py", line 178, in extract
    return self._real_extract(url)
  File "./youtube-dl/youtube_dl/extractor/generic.py", line 890, in _real_extract
    raise ExtractorError('Unsupported URL: %s' % url)
ExtractorError: Unsupported URL: http://www.franceculture.fr/emission-l-atelier-fiction-true-love-will-find-you-in-the-end-2014-09-09; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.

Note that using the player URL (http://www.franceculture.fr/player/reecouter?play=4902544) works.

Thanks to @flatgreen it's supported in the last version.


Set None as default for Optional fields

Optional and Required have different semantics, and the Pydantic v2 cleanup makes the fields required unless None is set as default value.
Without adding = None in those optional fields, the library fails, e.g.:
pydantic_core._pydantic_core.ValidationError: 2 validation errors for KeycloakUser
realmRoles
  Field required [type=missing, input_value={'id': '898e45fa-d868-4b3...: True, 'manage': True}}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.1.2/v/missing
attributes
  Field required [type=missing, input_value={'id': '898e45fa-d868-4b3...: True, 'manage': True}}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.1.2/v/missing

Can we please merge this?

Damm, I needed to test some things, throw a dirty commit in my branch and did not realize that the branch was being monitored from here and now everything is in shambles. (I don't know why I believed that the PR was "frozen" at the previous commit)

@alexbarcelo Should I review again?

@martincpt I would have thought that my last force-pushed (removing the invalid commit, that I will be using from another branch ofc) invalidated your code review. But GitHub shows me a tick as if you have already validated it. And @yannicschroeer approved this in its current state, but the commit history seem to not sit well with GitHub and does not show a tick :shrug:
Maybe I am not realizing where @yannicschroeer left a comment?

any updates?

Sorry. I did not see this. I'll set alexbarcelo as an additional owner of the repository

@yannicschroeer @alexbarcelo I understand you have lack of time for this projects, but do you have some time estimate of merging this PR and releasing it? We either need this fix or we will unfortunately need to fork the repo and do the fix on our side.

I am not an owner but a collaborator, so I cannot expedite the merge. Unless I have overlooked something obvious, I have to wait until the owners relax the pipelines and restrictions.

@yannicschroeer @alexbarcelo I understand you have lack of time for this projects, but do you have some time estimate of merging this PR and releasing it? We either need this fix or we will unfortunately need to fork the repo and do the fix on our side.

I do not only have a lack of time, we straight up announced that we can not offer any support in this project anymore. So far, nobody every contacted us to maintain the library.

I am not an owner but a collaborator, so I cannot expedite the merge. Unless I have overlooked something obvious, I have to wait until the owners relax the pipelines and restrictions.

I have escalated your rights further.

Thank you @alexbarcelo 🙏

Finally! Thanks!


Built application's javascript doesn't run?

Hello, my application runs perfectly fine when run with electron index.js, but when I try to build it with electron-packager, the resulting binary's javascript isn't willing to run.
The window and index.html/css load fine, but the javascript clearly isn't running based on the behavior of the templates, button clicks, and icon changes. You can observe that the default "cat" icon is still there when I configured it with a custom icon. Every other configuration property when the menubar constructor is initialized runs fine.
The build's resources contain all necessary files, I even ran electron index.js inside the resources and it worked great. I think there is something fishy that happens when the application is run as a binary...
Could anyone help me out? This is clearly abnormal and I can't seem to figure out the solution.

add this to your app:
mb.once('show', function () {
  mb.window.openDevTools();
})

which lets you see the error, which is that you get an EACCESS error when the node-persist module tries to mkdir /persist. I think this is because process.cwd() is set to / in electron (mentioned here).
You should be able to fix it by passing process.resourcesPath + '/persist' as the dir

Thanks. Not that I have anything against cats, but it seems I can't change the icon in the config object, neither can I change it using the tray's  setImage() function, even when concatenating: process.resourcesPath + "/icons/inbox.png". For the latter, there are some nasty-looking eventEmitter argument errors that are thrown.

Nevermind, had to do __dirname and it worked fine (forgot preceding /). Thanks!


Citing HTTP/3 explained

I've run into the "issue" of wanting to cite this book. I have a hard time finding example s of properly cited gitbooks.
@book{StenbergHTTP2018,
  title = {{{HTTP}}/3 Explained},
  author = {Stenberg, Daniel},
  year = {2018},
  edition = {2020-10-11},
  publisher = {{gitbooks}},
  url = {https://http3-explained.haxx.se/},
  abstract = {This book effort was started in March 2018. The plan is to document HTTP/3 and its underlying protocol: QUIC. Why, how they work, protocol details, the implementations and more. The book is entirely free and is meant to be a collaborative effort involving anyone and everyone who wants to help out.},
  langid = {english}
}

Do you think the above is a good enough citation in bib(la)tex?
Example:
Stenberg, D. (2018). HTTP/3 explained (2020-10-11). gitbooks.
If its good enough I'll add it as a pull request to the readme as well. Perhaps its possible to have a rolling edition date as well?

It seems fair and detailed enough to me. If you wanted to be even more specific than date you could refer to a specific commit hash that was at the top when the book was made that you read, but for most purposes the date should be enough I think.


Fingerprinting js is not documented but a hard dependency

Fingerprinting bit that uses server side (_r) and its invocation is not included in the SDK (unless you paste that initialization blob).
It seems that it's required for SDK to work, understandably. Even if it requires the server side call, it should be performed on init() or in otherwise controllable fashion.

this is fixed in the v2 branch as well.


Changed Source Url

The code is now on github, I changed the link from the codepen to the github url.

Looks like you changed the demo link rather than the source link!


Remove default Github Auth or edit it in place using groovy

When configuring the plugin through groovy, it looks like a default blank auth method is configured.   We tried using the example groovy configuration, but that just added an additional auth method.   We'd like to have some way to remove the default/blank auth method or an example of how to edit this in place in our groovy config.

👍

Update here... we worked around this issue by manually removing the first auth method prior to creating our own in the groovy configuration:
List<GhprbGitHubAuth> githubAuths = descriptor.getGithubAuth()

githubAuths.removeAt(0)


Fixing unintentional clicking on hierarchy window when another window is on top of it

Removing flag that activates all other flags in IsItemHovered that made the items in the hierarchy window be selected while using other windows/popups that were physically on top of them, creating sometimes engine braking bugs due to loading of scene and picking of unexisting GO at the same time.

Task linked: CU-23dy3md Fix hierarchy items being clicked even when dialog open on top

Its ok on my part, sonarcloud crying tho