Cake.CoreCLR can't handle white space in path

Steps to reproduce:
C:\temp> mkdir "some path"
C:\temp> cd '.\some path\'
C:\temp\some path> nuget install Cake.CoreCLR -o tools
C:\temp\some path> echo 'Information("Hello World");' > build.cake
C:\temp\some path> dotnet .\tools\Cake.CoreCLR.0.29.0\Cake.dll
Error: Could not find a part of the path 'C:\temp\some path\path\tools\Cake.CoreCLR.0.29.0'.

Notice how the path is duplicated in the above path.
//cc @gep13

Fun fact, if you have more than one whitespace in your path, you'll get another error message.
C:\temp\some other path> dotnet .\tools\Cake.CoreCLR.0.29.0\Cake.dll
More than one build script specified.
For usage, use parameter --help

I believe issue is this https://github.com/cake-build/cake/blob/a49462f9cd812a13255122220f9a4b954ea829b4/src/Cake/Polyfill/EnvironmentHelper.cs#L15

Related to https://github.com/cake-build/cake/issues/1272.

@mholo65 did you have a plan for how this one can get fixed?


backtabs don't work well in PTN

Specifically, it flickers the autocomplete menu, but doesn't keep it up. It should either not show it at all, or show it and keep it. This was introduced in PR #22.

Not sure when this got fixed, but it's fixed as of c740f81 at least.


Need to pass my class to slick-track

hi, thanks for your component, but according to my requirement so I need to pass my class to slick-track component (parent component to slick-slide), is there any way to do so?

I'm afraid there's not a direct way to do that. However, you can use the className prop to pass a custom class to the slider component and then can apply styling with CSS children selectors.

I'm afraid there's not a direct way to do that. However, you can use the className prop to pass a custom class to the slider component and then can apply styling with CSS children selectors.

using style in jsx but when passing a className prop I see the class in html but corresponding styles are not visible


Compile error when using c++17

I get this compiler error when enabling c++17:
/home/scxiao/Workplace/projects/AMDMIGraphX/deps_rocblas2.7/include/hip/hcc_detail/helpers.hpp:101:48: error: no member named 'invoke_result' in namespace 'std'
struct is_callable_impl<F(Ts...), void_t_<std::invoke_result<F(Ts...)> > > : std::true_type {};
                                          ~~~~~^
/home/scxiao/Workplace/projects/AMDMIGraphX/deps_rocblas2.7/include/hip/hcc_detail/helpers.hpp:101:64: error: 'Ts' does not refer to a value
struct is_callable_impl<F(Ts...), void_t_<std::invoke_result<F(Ts...)> > > : std::true_type {};

This is because the header assumes that std::invoke_result is available when __cplusplus is greater than 201402L which is not the case. This only means the compiler has enabled c++17 mode, it doesn't mean the c++ runtime has c++17. However, this header has other problems as well.
Doing std::invoke_result<F(Ts...)> is completely wrong as well(as shown here). It is supposed to called as std::invoke_result<F, Ts...>. And furthermore in this context it would be better to use std::is_invocable instead of doing the void_t dance.
Also, the is_callable trait should not de defined as is_callable<F(Ts...)> as this will lead to bugs when passing arrays or abstract type or distinguishing between top-level cv-qualifiers.

I was having the same issue with a hoomd-blue build and resolved it as described here: https://gist.github.com/jlblancoc/99521194aba975286c80f93e47966dc5
sudo apt-get install -y software-properties-common
sudo add-apt-repository ppa:ubuntu-toolchain-r/test
sudo apt update
sudo apt install g++-7 -y
sudo update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-7 60 \
                         --slave /usr/bin/g++ g++ /usr/bin/g++-7
sudo update-alternatives --config gcc

Please try with https://github.com/ROCm-Developer-Tools/HIP/pull/1631

Hi @pfultz2, just checking if your issue has been fixed.  I want to close the ticket if the reported issue has been fixed.  Thanks.

Closing the ticket as it is stale.  @pfultz2, please re-open if this issue still exists with latest ROCm 6.0.2 (HIP 6.0.32831).


Value of type 'DatabaseQuery' has no member 'idKey'  - FluentMySQLDriver 4.0.0-beta.3

Value of type 'DatabaseQuery' has no member 'idKey'  - FluentMySQLDriver 4.0.0-beta.3

Running swift package update or Xcode > Swift Packages > Update to latest should fix this.


Update docs with mention of default keys being used

@kacf @eysteins

@kacf Updated with links to the files


Create geonodes.toml

Last time when submitting PR  for testnet -11 , I made mistake submitting my toml file to main directory
https://github.com/anoma/namada-testnets/pull/1059
Description
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging to draft

[ ] Only one toml is added in this PR
[ ] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present

Your toml was not generated using the correct version of namada.


CertificateRequestPolicy based on which namespace the certificate request belongs to

My use case is to only allow certificate requests with dnsName which are service-names from the namespace to which the service-name belongs to, for a certain cluster-issuer. For example the dnsName my-service.my-space.svc should only be allowed when requested from the "my-space" namespace.
Preferably, I'd also like to restrict namespaces for a certain cluster issuer.
I don't seem to find how to do that in the current implementation. Am I missing something or is this an uncommon usecase?

Is this issue still valid?  Looks like it was resolved in #96
To accomplish this, there is a fairly complex rbac setup, but in order for a serviceaccount to make use of a request policy it must have the "use" action for the exact policy name.

Ops, I hadn't seen the PR. Let me check it out.

I would like to see an option where I can specify a fixed policy for a number of namespaces, without having to create a policy for all those namespaces.
An example would be that the allowed list could have placeholders.
Example policy:
---
apiVersion: policy.cert-manager.io/v1alpha1
kind: CertificateRequestPolicy
metadata:
  name: example-com
spec:
  allowed:
    # Name TBD - trigger that a <namespace> placeholder have to match the requesting namespace
    namespace_restricted: True
    commonName:
      value: "*.<namespace>.example.com"
    dnsNames:
      values:
      - "*.<namespace>.example.com"
    usages:
    - "server auth"
  selector:
    issuerRef:
      name: le-issuer
      kind: ClusterIssuer
      group: cert-manager.io

Should be allowed:
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
  name: my-certificate
  namespace: my-test
spec:
  dnsNames:
  - my-certificate.my-test.example.com
  issuerRef:
    kind: ClusterIssuer
    name: le-issuer
  secretName: my-certificate

Should be rejected
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
  name: my-certificate
  namespace: my-othernamespace  # Namespace changed
spec:
  dnsNames:
  - my-certificate.my-test.example.com
  issuerRef:
    kind: ClusterIssuer
    name: le-issuer
  secretName: my-certificate

This would prevent:

copy/paste errors
One namespace grabbing or overwriting a hostname for one namespace into the other

Would achieve:

Lock certificates to hostnames with their namespace in it only without managing complex and ever changing RBAC rules.

This kind of "complex" logic can best be handled through a plugin I think.

With helm and a list of namespaces I can create one CertificateRequestPolicy per namespace to create the appropriate policies to allow dnsName like *.{namespace}.svc.
@inteon Please correct me if I'm wrong. My understanding is that the approver-policy is, generally speaking, a rule based engine for approving certificate requests. As such, having templating within the rules to handle for example the origin of the request isn't that far fetched, is it?
I think I'm agreeing with @sastorsl  that the CertificateRequestPolicy needs a templating model.
@inteon What do you think, should I rework the title and description of this issue or should I create a new?

As I think this issue is still valid and the structure as all been fixed, I created a PR to enable templating in the approver-policy. Please tell me what you think!

@sastorsl I believe my PR together with the namespaceSelector on CertificateRequestPolicy would satisfy your use case, correct?

@inteon I'm looking at CEL and that's a very different thing than just templating the allowed values. 😀 How did you imagine this from a configuration point-of-view, a new array of rules? Perhaps spec.rules[] or spec.constraints.rules[]?

@inteon I'm looking at CEL and that's a very different thing than just templating the allowed values. 😀 How did you imagine this from a configuration point-of-view, a new array of rules? Perhaps spec.rules[] or spec.constraints.rules[]?

Indeed, that might be even more powerful. Would it be possible to join one of our standups to discuss this in more detail? https://cert-manager.io/docs/contributing/#meetings

Sure, I'll try to join today!

Sorry about being late with a reply, the famous "everything else happened".
I added a comment to your new PR.

Combining selector.namespace with the namespace-configuration-operator should make the configuration across namespaces a bit easier to manage.

Over the weekend I've put together a plugin to see if CEL can be used for this: https://github.com/erikgb/cel-approver-policy-plugin
Feel free to try it out, but I am not planning to support this feature as a plugin. IMO the core approver in cert-manager approver-policy should support this. I am planning to discuss how we can get there with the cert-manager maintainers....


Unable to run  https://. Browser not shows anything. Using docker swarm

able to up the dockers using make command
started docker swarm with init command
Launched the browser..

Do you replace  with your docker swarm master hostname?
Do you see a security warning from the browser?

Yes. I did that.

I have set up this on another machine
Trying to access https://<< hostname >> from my local machine
Following services are installed

As it was not working, I did the following,
Here I ran docker instance of  ssai_content_provider_frontend , but exposed internal port 8080 to outer port 80

CONTAINER ID        IMAGE                                                     COMMAND                  CREATED             STATUS              PORTS                                    NAMES 1b3f067fa5ed        ssai_content_provider_frontend:latest                     "/bin/bash -c /home/…"   10 hours ago        Up 10 hours         <IP_ADDRESS>:80->8080/tcp                     laughing_williamson
which resulted in n showing the UI as mentioned in README.md. But, Video URL is empty. Also I can not edit that. I could see some errors in the debug window.

Your service list showed that CDN was not up and running. Could you check what was the log message?
The CDN service is internally configured as https. Thus mapping it to port 80 will not work as is. Is it possible that your machine port 443 is occupied? In that case, please map the port to 8443 in the deployment script. Try to access it via https://<< hostname >>:8443.

I have taken down custom installed docker instance as mentined in step#4. This was an extra step i did because, the thing as mentioned as per documentation is not showing anything in the browser.
Now there is

docker swarm instance is running
docker stack contains all adi services

I am new to the docker swarm. So not familiar with its architecture. sorry for that, but can you help me here. Following can be silly questions
Question

Why I still can't access it?
how https://<< hostname >>, docker swarm master node and adiservices are related. who tells to render ad insertion landing page and how?
Do I need to start httpd instance?

Here are the steps for browsers to see the URL:
(1) The CDN deployment script maps the external 443 port to an internal 8443 port. https://github.com/OpenVisualCloud/Ad-Insertion-Sample/blob/master/deployment/docker-swarm/cdn.m4. This port talks https.
(2) The CDN nginx instance forwards the web request to the AD insertion service, which is at internal port 8080: https://github.com/OpenVisualCloud/Ad-Insertion-Sample/blob/master/cdn/nginx.conf
(3) The AD insertion service forwards the web request to the content provider service, which is at internal port 8080: https://github.com/OpenVisualCloud/Ad-Insertion-Sample/blob/master/ad-insertion/frontend/nginx.conf
(4) The content provider nginx instance handles the web request and returns the web pages: https://github.com/OpenVisualCloud/Ad-Insertion-Sample/blob/master/content-provider/frontend/nginx.conf.
For now, it's not clear why you don't see the web pages. Please do two experiments as follows. Please report the results:
(1) Change 443:8443 in this script to be "8443:8443": https://github.com/OpenVisualCloud/Ad-Insertion-Sample/blob/master/deployment/docker-swarm/cdn.m4. Restart the app. See if you can see the web page at https://hostname:8443
(2) If you still cannot see it, please in addition, change: "listen 8443 ssl" to "listen 8443". Rebuild and restart the app. See if you can access the content via http://_hostname:8443.

Any update?

`diff --git a/cdn/nginx.conf b/cdn/nginx.conf
index de0806c..044a5d5 100644
--- a/cdn/nginx.conf
+++ b/cdn/nginx.conf
@@ -17,7 +17,7 @@ http {
limit_conn_zone $binary_remote_addr zone=addr2:10m;
 server {

   listen 8443 ssl;

   listen 8443;
   server_name _;
   client_body_timeout 5s;
   client_header_timeout 5s;

diff --git a/deployment/docker-swarm/cdn.m4 b/deployment/docker-swarm/cdn.m4
index 4fd3250..c4dbc57 100644
--- a/deployment/docker-swarm/cdn.m4
+++ b/deployment/docker-swarm/cdn.m4
@@ -2,7 +2,7 @@
cdn-service:
image: defn(`REGISTRY_PREFIX')ssai_cdn_service:latest
ports:

       - "443:8443"

       - "8443:8443"
   environment:
       NO_PROXY: "*"
       no_proxy: "*"

`

Did the following changes
rebuild application using https://github.com/OpenVisualCloud/Ad-Insertion-Sample#build
started docker swarm using https://github.com/OpenVisualCloud/Ad-Insertion-Sample#startstop-services
Accessed http://:8443 . still getting an error, can not connect to the server
 Ad-Insertion-Sample]$ curl "http://localhost:8443" curl: (7) Failed connect to localhost:8443; Connection refused when tried from command line on same machine. getting connection refused error

diff --git a/cdn/nginx.conf b/cdn/nginx.conf
index de0806c..044a5d5 100644
--- a/cdn/nginx.conf
+++ b/cdn/nginx.conf
@@ -17,7 +17,7 @@ http {
limit_conn_zone $binary_remote_addr zone=addr2:10m;
server {

   listen 8443 ssl;

   listen 8443;
   server_name _;
   client_body_timeout 5s;
   client_header_timeout 5s;

diff --git a/deployment/docker-swarm/cdn.m4 b/deployment/docker-swarm/cdn.m4
index 4fd3250..c4dbc57 100644
--- a/deployment/docker-swarm/cdn.m4
+++ b/deployment/docker-swarm/cdn.m4
@@ -2,7 +2,7 @@
cdn-service:
image: defn(`REGISTRY_PREFIX')ssai_cdn_service:latest
ports:

       - "443:8443"

       - "8443:8443"
   environment:
       NO_PROXY: "*"
       no_proxy: "*"`

Hello @xwu2git ,
I am now unable to run the setup. But now facing the same issue (see above browser screen with firebug) .. getting 404 for js and map files
0/1 replica  issue was occurring because was getting this error "Err": "starting container failed: linux spec user: unable to find user docker: no matching entries in passwd file", in case of  adi_cdn-service. The reason mostly is this file https://github.com/OpenVisualCloud/Ad-Insertion-Sample/blob/master/cdn/Dockerfile . it searches for the user docker , which is missing i gues in base image ssai_common.
Please, guide me around that path mapping issue

I think I found the issue. Did you build as root? Could you try to build it as a regular user instead?
The root cause is "USER ${USER}" in the Dockerfile. If build as root, the user "docker" is not added during the build stage thus cannot be found during runtime. It will be fine if you build as a regular user.
I will fix this issue. Stay tuned.

Fixed in #240.

@xwu2git , yes I am building this using sudo access for enabling setting up this application docker images.
As discussed above, I am facing now another issue  of path mapping in case of js and map files (see screenshot as pasted earlier https://github.com/OpenVisualCloud/Ad-Insertion-Sample/issues/236#issuecomment-646411052)

Sorry, can we start from the beginning? The issues are compounded together.
After applying #240, do you see the homepage? The CDN instance is now running? Not 0/1 in docker stack services adi. Is the video list showing up?

Also, how many nodes are there in your docker swarm? If it's more than 1, you need to share the volume folder on all worker nodes as described in deployment/docker-swarm/README.md. If it's a single node, then there is no specific setup needed other than docker swarm init.

I haven't taken the latest pull including #240 yet as I an see the homepage now.
Yes CDN instance is now running, showing 1/1 in docker stack services ad
On the homepage, I can see the playlist. But still now getting video URL and video played. Here is screenshot

As you can see above screenshot, I am now facing the path mapping issues. Right now it is for some of the js and .map files. It is returning 404 errors. Looking for help around it! Thanks!

There is no map file in the repo. The error does not impact playing back.
Did you double click on the videos?

Checking

Oh...  it is playing the video after selecting from the playlist. Better if we can mention it in documentation

Good.

One question
Is this code Open source ?
Is it production-ready? we are working on some POC using it ?

Yes, it's fully open sourced.
The ad-insertion feature is validated. However, this is a sample. Certain features are intentionally simplified, such as user account management, AD content management, AD decision making, as well as security designs. You might want to enhance those in a product.

Thanks @xwu2git  for update !

Hello @xwu2git , can we stitch multiple ads within single cue point here

Yes, it is possible. In practice, you need to pay attention to resolution change during serving multiple ads. All ads of all resolutions of a cue point must be prepared as an atomic operation, or ad-falling back may kick in.

Is this supports VAST capabilities

We followed similar concept as VAST but did not follow the exact syntax, which is a bit overkill for this small sample. You can easily modify the code to use VAST.

thanks  !

closed.


[vm/ffi] Performance cliff for nested structs and arrays if used both with Pointer and TypedData backing store

https://github.com/dart-lang/sdk/blob/ab09f99638af8b50c2b8700cd6bd07b9e89f6337/pkg/vm/lib/transformations/ffi/common.dart#L913-L928
This pattern of checking for whether something is backed by a Pointer, and doing pointer arithmetic if it is a pointer and a typed data view if it is not a pointer causes a performance cliff when TFA cannot prove that one of the branches is definitely not happening.
https://github.com/dart-lang/sdk/commit/ddfc00e7739108a7e29092ec4559fbd74733f2bb caused benchmarks/FfiStructCopy/dart/FfiStructCopy.dart to slow down 15x. This is because the TypedData path in the if is no longer removed by TFA (due to the TypedData constructors for Array now being shared). This causes the inliner to bail out on inlining all Pointer and TypedData view calculations. Which in turn causes a bunch of calls and allocations.
I presume that any reasonably large FFI program would have both Pointer and TypedDatas backing structs, so this micro-benchmark was probably giving us a too rosy view. (Because if this, I don't believe a revert is in order.)
We should try to remove any branching on something being a Pointer or TypedData.
Some ideas:

a. Making a recognized method that does a typedDataBase.offsetBy and calling that in the struct/array implementation instead of branching on an instanceOf<Pointer>. The question is what IL/machine code to generate in this case. We'd still need to branch on whether to allocate a Pointer object or an TypedDataView object.
b. Make Structs/Arrays always work on external typed data and never use Pointers.
c. Have an offset field in _Compound and never create new typed data views or pointers at all. Nested struct and array accessors would return the same typed data base object but with a different int offset.

cc @mkustermann I believe you have argued for option (c) before.

Performance regression has been addressed: https://golem.corp.goog/Revision?repository=dart&revision=109399


docker volumes ERROR

Error sharing volumes
postgis_1   | FATAL:  data directory "/var/lib/postgresql/data" has wrong ownership
postgis_1   | HINT:  The server must be started by the user that owns the data directory.

Exist solutions for error? | postgres 9.6
docker-compose
version: "3"
services:
    postgis:
        image: mdillon/postgis:9.6
        environment:
            POSTGRES_USER: usertest
            POSTGRES_PASSWORD: usertest
            POSTGRES_DB: db_engenhagro
            PGDATA: /var/lib/postgresql/data
        ports:
            - 5432:5432
        volumes:
            - .data_postgres:/var/lib/postgresql/data

this repo only a simple postgres docker image extension

https://github.com/docker-library/docs/tree/master/postgres

so you can search this type of errors in the original repo:

https://github.com/docker-library/postgres/search?utf8=✓&q=ownership&type=Issues


Unable to get started

The README specifies Sonatype Snapshots repo, but doesn't specify the group id or artifact id to use. I tried to find them myself, but the repo doesn't contain any relevant artifacts.
The Getting Started wiki document provides a different set of snapshots, e.g.: snapshots, however, the recommended URL is over HTTP and there doesn't appear to be a different trust rooting/code signing mechanism in place. These appear to be snapshots of 1.0.0 around March 2016.
The Overview wiki document provided a great overview of the difference between the different Azure artifacts. Fortunately, these artifacts appear to be available on Maven Central; like the above snapshot server, it appears to have 1.0.0-x versions, specifically 1.0.0-beta, released 02-Mar-2016. However, these ostensibly similar versions appear to provide drastically different APIs: for example, the ARM usage section in the README suggests that you can do:
ResourceManagementClient client = new ResourceManagementClientImpl(
    new ApplicationTokenCredentials("client-id", "tenant-id", "secret", null) // see Authentication
);

... a ctor whose existence is confirmed by the javadoc. The Getting Started doc claims you can do:
ComputeManagementClient client;

// Default environment, no credentials
client = new ComputeManagementClientImpl();

... again confirmed by the javadoc. Except neither of these ctors is available in 1.0.0-beta that got shipped to Maven Central! From that Javadoc (no direct link since I couldn't find a place that it was hosted):

ComputeManagementClientImpl(org.apache.http.impl.client.HttpClientBuilder httpBuilder, ExecutorService executorService)
Initializes a new instance of the ComputeManagementClientImpl class.
ComputeManagementClientImpl(org.apache.http.impl.client.HttpClientBuilder httpBuilder, ExecutorService executorService, SubscriptionCloudCredentials credentials)
Initializes a new instance of the ComputeManagementClientImpl class.
ComputeManagementClientImpl(org.apache.http.impl.client.HttpClientBuilder httpBuilder, ExecutorService executorService, SubscriptionCloudCredentials credentials, URI baseUri)
Initializes a new instance of the ComputeManagementClientImpl class.
ComputeManagementClientImpl(org.apache.http.impl.client.HttpClientBuilder httpBuilder, ExecutorService executorService, SubscriptionCloudCredentials credentials, URI baseUri, String apiVersion, int longRunningOperationInitialTimeout, int longRunningOperationRetryTimeout)
Initializes a new instance of the ComputeManagementClientImpl class.

Similarly, the ResourceManagementClientImpl has totally different ctors in that version.
Could you please clarify the distinction between these repos? Is the current Microsoft-recommended way to interact with Azure to download snapshots off of ADX? Are the versions on Maven Central not supposed to work? Is this wiki the official documentation for using these SDKs?

Hi @lvh,
Sorry for the confusions around the README and javadocs. We were busy developing for beta2 and there could be some mismatches from one place to another.
The good news is that we just shipped beta2. Please refer to the latest README for samples, how to download, etc. Thanks!

Closing as no action to take. Beta2 should not require anything in snapshots.


Improve MEDIATYPE support and convert legacy TYPE

Currently MEDIATYPE is only supported for DataProperty instances. Although DataProperty handles its URL/URI differently, RFC 6350 permits any property with a URI value to use MEDIATYPE. It makes sense, therefore, to break the MEDIATYPE support into its own class.
A related issue is that VCard 3.0 records often held what is now MEDIATYPE in TYPE as in the JohnDoe example:
PHOTO;VALUE=URL;TYPE=PNG:http://upload...

This information needs to be captured and converted from legacy VCards. Generally, the VCard 3.0 media-types were restricted to a particular subset (e.g. images for PHOTO), so that information can help us capture and translate the correct TYPE parameters.
#17 will require resolution of this issue because of the legacy media-type embedded in the sample VCard.
This issue is leading toward a need to organize the "tweaks" which VCardParser makes after the initial parsing and before the VCard is returned, such as capturing bare TYPE parameters, PREF, AGENT, and so forth. It may be useful to have a "tweak" record which incorporates a test to see if it is applicable, an action to apply, and perhaps any ordering constraints (e.g. MEDIATYPE and PREF must be translated after bare TYPEs are handled). There are also a number of extended properties which are obsoleted by VCard 4.0 properties which could then be handled cleanly. Need to explore this and possibly open a new issue for dealing with it.

VCard 4.0 MEDIATYPE use RFC 4288 media-types, e.g. image/jpeg;width=400. The current implementation ensures that MEDIATYPE matches this format.
VCard 3.0 used just the image types in the TYPE parameter (e.g. TYPE=JPEG). So, when translating the media-types from 2.1 or 3.0, also need to convert them to RFC4288-compliant.
Then we verify that the RFC4288 media-types match AllowedMediaTypes, if supplied, or if not supplied, the regex currently used.

VCard 4.0 MEDIATYPE use RFC 4288 media-types, e.g. image/jpeg;width=400. The current implementation ensures that MEDIATYPE matches this format.
VCard 3.0 used just the image types in the TYPE parameter (e.g. TYPE=JPEG). So, when translating the media-types from 2.1 or 3.0, also need to convert them to RFC4288-compliant.


refactor: extract external routes from eip func, make it the same as …

What type of this PR
refactor: extract external routes from eip func, make it the same as vpc subnet route

cherry pick this commit into 1.10


Making typia ZodEsque

Feature Request
It would be great if typia could be considered ZodEsque
By trpc and elysia
i would like to experiment with using typia instead of zod and typebox and curious if you plan to offer parse functions similar to zod.

Possible to use typia in tRPC.
Have not tested elysia, but it seems possible. Will you try it?
import { initTRPC } from "@trpc/server";
import { v4 } from "uuid";
import typia from "typia";

interface IBbsArticle {
    id: string;
    writer: IBbsArticle.IWriter;
    title: string;
    body: string;
    created_at: string;
}
namespace IBbsArticle {
    export interface IStore {
        writer: IWriter;
        title: string;
        body: string;
        /**
         * @minLength 4
         * @maxLength 16
         * @pattern ^[a-zA-Z0-9]+$
         */
        password: string;
    }
    export interface IWriter {
        /**
         * @format uuid
         */
        external_service_id?: string;
        /**
         * @format email
         */
        email: string;
    }
}

const t = initTRPC.create();

export const appRouter = t.router({
    store: t.procedure
        .input(typia.createAssert<IBbsArticle.IStore>())
        .output(typia.createAssert<IBbsArticle>())
        .query(({ input }) => {
            return {
                id: v4(),
                writer: input.writer,
                title: input.title,
                body: input.body,
                created_at: new Date().toString(),
            };
        })
});
export type AppRouter = typeof appRouter;

input(typia.createAssert<IBbsArticle.IStore>())

Ahhhh using create assert... Nice, yeah I'll try that over the weekend!
Thanks 🙏

I just followed guide of tRPC. They delivers parsed object into the input function.

So i didn't try with our trpc, because my goal is to use the trpc plugin for elysia... Which is optimized for bun, and i didn't manage to get the server running with typia involved. I could run the ttsc command but the run commands didn't work.
I expect this is because bun does not employee the transformer properly.
More investigation needed.

Will support through https://github.com/samchon/typia/issues/509


Inconsistent result with line sort and block sort

I noticed a few times block sort and line sort generate different result on special characters.
Line sort puts _ before : while for block sort it's the other way around.

I can look into that, currently it simply sorts based on the byte value of the characters

Sorry for the delay. Should be fixed in the next update. It was due to Sort Lines using localeCompare instead of the < operator.

Update: I have now updated the release token, it was invalid... It has now been actually release


Add MiniscopeConverter

This PR adds the MiniscopeConverter, which currently has an error from the NWB Inspector but can complete a full conversion despite this.

Indeed files look fine, but Inspector message is actually a PyNWB validation error. Also the CLI report shows more info about this issue

The validation error is a known and has been raised upstream for a while. It might actually prevent DANDI upload from within the app, not sure since DANDI uses a slightly different strategy for assessing that aspect

Realized the base of this too late


closed caption disappears from video controls

Hey, I love the possibilities of using player.style! One thing I can't figure out is why the CC icon disappears from my implementation. I have a custom old-school VHS video with a 4/3 aspect ratio and a smaller 320 x 240 size, so it could be this. But I don't even find CC under the gear/settings icon, so I don't know what to do about this. Am I missing something?
I've used player.style once, here: https://monkitale.com/danimations/railroad/
Thank you,
Dan

Hi @danodna, thanks for the feedback. Unfortunately the YouTube iframe API lacks API's to list and control caption selection I believe.
So the themes don't have anything to work with.
Some media types have some restrictions like this because the API's are missing some features. I believe YT is one of the most restrictive ones.


Upgrade ruby/setup-ruby v1.165.1 -> v1.170.0

This PR contains the following updates:

Package
Type
Update
Change

ruby/setup-ruby
action
minor
v1.165.1 -> v1.170.0

Release Notes

ruby/setup-ruby (ruby/setup-ruby)
v1.170.0: Add CRuby 3.2.3 on Windows
Compare Source
What's Changed

Update CRuby releases on Windows by @ruby-builder-bot in https://github.com/ruby/setup-ruby/pull/566

Full Changelog: https://github.com/ruby/setup-ruby/compare/v1.169.0...v1.170.0
v1.169.0: Add windows-toolchain input
Compare Source
What's Changed

Add windows-toolchain input by @MSP-Greg in https://github.com/ruby/setup-ruby/pull/563

Full Changelog: https://github.com/ruby/setup-ruby/compare/v1.168.0...v1.169.0
v1.168.0: Add ruby-3.2.3
Compare Source
What's Changed

Add ruby-3.2.3 by @ruby-builder-bot in https://github.com/ruby/setup-ruby/pull/565

Full Changelog: https://github.com/ruby/setup-ruby/compare/v1.167.0...v1.168.0
v1.167.0: Workaround bug in XCode 14.2 linker not respecting RTLD_LAZY
Compare Source
See https://github.com/oracle/truffleruby/issues/3390
v1.166.0: Add truffleruby-23.1.2, truffleruby+graalvm-23.1.2
Compare Source

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade ruby/setup-ruby v1.165.1 -> v1.170.0


Automatically recalculate promotions when making changes to an order

When manually editing an order, any promotions in the order are not recalculated. You need to specifically disable and re-enable automatic promotions for this to happen, which is something that gets overlooked.
In turn, this gives issues with e.g. reporting plugins that show discount per item because the referenced line items in the promotion composition do not exist anymore.
Ideally the promotions are recalculated automatically when making changes to an order.

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-23440
[created from NEXT-23440, comment 491495]


componentWillReceiveProps: how to know when dynamic segment changes?

tl;dr: what is the recommended way in componentWillReceiveProps to find the previous dynamic segment?
Longer:
I understand the note about dynamic segments: the lifecycle hook to fetch data when a dynamic segment changes is componentWillReceiveProps. Cool.
In React, componentWillReceiveProps receives as an argument the nextProps. These can be compared to this.props and used to determine whether a relevant property changed and data should be fetched.
What I want to do is only fetch new data in componentWillReceiveProps if the dynamic segment changes. Significantly, in the presence of other props, I cannot assume that componentWillReceiveProps is called only when the dynamic segment changes. However I haven't found a way to recover the previous routing params.
I can easily work around by setting an instance variable in componentWillReceiveProps to record the current routing params for comparison on the next invocation of componentWillReceiveProps. That seems sort of nasty and stateful.
I hope I haven't misunderstood. But if I have understood correctly, then the routing params context is mutable and the previous route is lost, and cannot be compared against. That seems contrary to the concept of componentWillRecieveProps.

see https://github.com/rackt/react-router/issues/579#issuecomment-72350478


MM-20621 - Updating z-index for fullscreen modals

Summary
MM-20621 - Updating z-index for fullscreen modals
Incorrect z-index is being applied to the fullscreen modal css that is added to the plugins. Instead of fixing that in different plugins, we should remove all fullscreen css from plugins as that is already available in the webapp.
Here's a ticket for that:
https://mattermost.atlassian.net/browse/MM-20744
Ticket Link
https://mattermost.atlassian.net/browse/MM-20621

Removing the QA label since it's verified. If anything changes, please let me know.

/check-cla


Making GetModuleReady/GetDevicePluginReadyLabels functions public

since "ready" labels are public APIs of KMM, operators using KMM may want to schedule they workloads based on the presence of those labels.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f2550bb) 79.07% compared to head (c86574a) 79.09%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #672      +/-   ##
==========================================
+ Coverage   79.07%   79.09%   +0.01%
==========================================
  Files          49       50       +1
  Lines        5080     5084       +4
==========================================
+ Hits         4017     4021       +4
  Misses        879      879
  Partials      184      184

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

/lgtm


Importing app.py to userauth .py files

I am getting an error in which when I try import app from app.py I get an error saying module not found. Code for this is
from app import app.
Code:

Error:

File structure:

Anyone know if the issue is the name of the directory being called app and the app.py file is a factor?
If not why would the modulenotfounderror be occuring?
Thanks

I believe its because the directory is now named userauth and not app. When you try and import the flask instance created with: app = Flask(__name__), you are using from app import app, here the first app being the module/directory name. A simple fix I think would be from userauth import app.

ISSUE SOLVED
@normit581 redid the file structure and made sure the imports were valid and the code was working.


SearchPerformanceTest times out with only moderately-sized data

Describe the bug
While upgrading to FHIR 4.3.0, one of our existing e2e tests kept timing out.
From fhir-server-test SearchPerformanceTest:
    @Test(groups = { "server-search-performance" })
    public void testSearchObservationWithPatientName() {
        WebTarget target = getWebTarget();
        Response response =
                target.path("Observation")
                        .queryParam("subject:Patient.name", "john")
                        .request(FHIRMediaType.APPLICATION_FHIR_JSON)
                        .header("X-FHIR-TENANT-ID", tenantName)
                        .get();
        assertResponse(response, Response.Status.OK.getStatusCode());
        Bundle bundle = response.readEntity(Bundle.class);
        System.out.println(bundle);
        assertNotNull(bundle);
    }

string queries like this are known to be slow and this is covered in our performance guide, but i was surprised this was timing out with such little data (~750 Observations, ~250 Patients).
changing the query to use :exact got me past the issue, but we should consider investigating this to make sure its just a dumb derby thing (and doesn't affect real dbs like postgresql or db2)
Environment
r4b branch
To Reproduce
Steps to reproduce the behavior:

checkout r4b branch
revert my
run the full e2e test suite
Scroll down to '....'
See error

Expected behavior
It runs with success and passes the test
Additional context
stacktrace with the timeout.txt
trace output after changing to use :exact:
[INFO] [12/16/21, 19:39:19:090 UTC] 0000002c FHIRRestServl I   Received request: tenantId:[default] dsId:[default] user:[fhiruser] method:[GET] uri:[https://localhost:9443/fhir-server/api/v4/Observation?subject:Patient.name:exact=John]
[INFO] [12/16/21, 19:39:19:093 UTC] 0000002c FHIRDbDAOImpl 1   Getting connection for tenantId/dsId: [default/default]...
[INFO] [12/16/21, 19:39:19:107 UTC] 0000002c FHIRDbDAOImpl 1   Got the connection for [default/default]. Took 13 ms
[INFO] [12/16/21, 19:39:19:108 UTC] 0000002c QueryUtil     1   bind marker count: 1
[INFO] [12/16/21, 19:39:19:108 UTC] 0000002c QueryUtil     1        query string:
[INFO]       SELECT COUNT(*) AS CNT
[INFO]         FROM Observation_LOGICAL_RESOURCES AS LR0
[INFO]        WHERE LR0.IS_DELETED = 'N'
[INFO]          AND EXISTS (
[INFO]       SELECT 1
[INFO]         FROM Observation_LOGICAL_RESOURCES AS LR1
[INFO]   INNER JOIN Observation_TOKEN_VALUES_V AS P2 ON P2.LOGICAL_RESOURCE_ID = LR1.LOGICAL_RESOURCE_ID
[INFO]   INNER JOIN Patient_LOGICAL_RESOURCES AS LR2 ON LR2.LOGICAL_ID = P2.TOKEN_VALUE
[INFO]          AND P2.PARAMETER_NAME_ID = 20718
[INFO]          AND P2.CODE_SYSTEM_ID = 20832
[INFO]          AND LR2.IS_DELETED = 'N'
[INFO]   INNER JOIN Patient_STR_VALUES AS P3 ON P3.LOGICAL_RESOURCE_ID = LR2.LOGICAL_RESOURCE_ID
[INFO]          AND P3.PARAMETER_NAME_ID = 20467
[INFO]          AND (P3.STR_VALUE = ?)
[INFO]        WHERE LR1.LOGICAL_RESOURCE_ID = LR0.LOGICAL_RESOURCE_ID)
[INFO] [12/16/21, 19:39:22:286 UTC] 0000002c FHIRDbDAOImpl 1   Successfully retrieved count; count=48 [took 0.147745 ms]
[INFO] [12/16/21, 19:39:22:287 UTC] 0000002c QueryUtil     1   bind marker count: 1
[INFO] [12/16/21, 19:39:22:287 UTC] 0000002c QueryUtil     1        query string:
[INFO]       SELECT R.RESOURCE_ID, R.LOGICAL_RESOURCE_ID, R.VERSION_ID, R.LAST_UPDATED, R.IS_DELETED, R.DATA, LR.LOGICAL_ID
[INFO]         FROM (
[INFO]       SELECT LR0.LOGICAL_RESOURCE_ID, LR0.LOGICAL_ID, LR0.CURRENT_RESOURCE_ID
[INFO]         FROM Observation_LOGICAL_RESOURCES AS LR0
[INFO]        WHERE LR0.IS_DELETED = 'N'
[INFO]          AND EXISTS (
[INFO]       SELECT 1
[INFO]         FROM Observation_LOGICAL_RESOURCES AS LR1
[INFO]   INNER JOIN Observation_TOKEN_VALUES_V AS P2 ON P2.LOGICAL_RESOURCE_ID = LR1.LOGICAL_RESOURCE_ID
[INFO]   INNER JOIN Patient_LOGICAL_RESOURCES AS LR2 ON LR2.LOGICAL_ID = P2.TOKEN_VALUE
[INFO]          AND P2.PARAMETER_NAME_ID = 20718
[INFO]          AND P2.CODE_SYSTEM_ID = 20832
[INFO]          AND LR2.IS_DELETED = 'N'
[INFO]   INNER JOIN Patient_STR_VALUES AS P3 ON P3.LOGICAL_RESOURCE_ID = LR2.LOGICAL_RESOURCE_ID
[INFO]          AND P3.PARAMETER_NAME_ID = 20467
[INFO]          AND (P3.STR_VALUE = ?)
[INFO]        WHERE LR1.LOGICAL_RESOURCE_ID = LR0.LOGICAL_RESOURCE_ID)) AS LR
[INFO]   INNER JOIN Observation_RESOURCES AS R ON LR.CURRENT_RESOURCE_ID = R.RESOURCE_ID
[INFO]     ORDER BY LR.LOGICAL_RESOURCE_ID
[INFO]  FETCH FIRST 10 ROWS ONLY
[INFO] [12/16/21, 19:39:25:244 UTC] 0000002c FHIRDbDAOImpl 1   Successfully retrieved FHIR objects [took 2956.175614 ms]

seeing something similar on SearchReverseChainTest.testSearchMultipleReverseChainWithMultipleChainedSearchParm as well

resolution:  before shipping 5.0, we should run our query performance test suite.
we should make sure at least one search there is on a string search parameter and doesn't use the :exact modifier.

I populated a DB with ~35,000 resources, ~15,000 were Observations, and 100 Patients.  whole test ran in 7 seconds.  Closing issue.


Typeahead not working with JoinTable

Bug Report
Steps to Reproduce:

Create a join table in our demo app (e.g. SIMUL Instrument Prices)
Try adding an = filter on a string column (e.g. ric)
You'd notice that the typeahead is not working, you can see the error in your browser console

Expected Result:
Typeahead should work as for any other not joined in-memory table.
Actual Result:
It throws an exception on the Vuu server when creating the InMemColumnValueProvider. There we check if the passed table is of type InMemDataTable, if it is then we go ahead and create it otherwise we throw an exception.
But on the server side JoinTable extends DataTable directly, which means JoinTable is not of type InMemDataTable (not sure what's the context behind this) and hence, the exception.
Additional Context:
JoinTable: https://github.com/finos/vuu/blob/71700fde0dc2431ebf98022f04753e14940c3a67/vuu/src/main/scala/org/finos/vuu/core/table/JoinTable.scala#L660
InMemColumnValueProvider:
https://github.com/finos/vuu/blob/11b94a05c20b576f179cc4f15dbe2bb099406e66/vuu/src/main/scala/org/finos/vuu/core/table/ColumnValueProvider.scala#L24-L29

Will work on this and discuss with @naleeha about it when she's free.


Should the predictor variables really be unique?

The code currently has this safety check:
https://github.com/DataDog/piecewise/blob/3a15a1c3113cbbecf979bb318f19f2c7fbdc9408/piecewise/regressor.py#L215-L216
This is a problem for me, as I have several occurences of each predictor variable.
I tried to remove this check and run the following code, where each predictor variable is repeated 10 times:
import numpy
from piecewise.plotter import plot_data_with_regression
nbrep=10
t = numpy.concatenate([numpy.arange(10)]*nbrep)
v = numpy.array((
    [2*i for i in range(5)] +
    [10-i for i in range(5, 10)])*nbrep
) + numpy.random.normal(0, 1, 10*nbrep)
plot_data_with_regression(t, v)

It gives this result, which seems great to me:

So, do you think that we could safely remove this safety check, or is there any corner case I didn't think of?

Based on the way it's currently written, I believe it would be possible for different points at the same t to be modeled with two (or more) different linear segments. From a prediction perspective, this obviously makes no sense. However, it seems possible to prevent this problematic case by making the initialization a little smarter, so that it initializes all points with the same ts as modeled by the same linear segment. Since it does seem useful to handle this case, I'll take a pass at changing the initialization and creating a test to cover this case. PR to follow....

@Ezibenroc - I've created a PR for this: https://github.com/DataDog/piecewise/pull/6
How does that look?

I've merged the PR, so closing this issue. Feel free to follow-up if I missed anything.

It looks good, thank you!


Grid demo not working as described

On the Picnic documentation page, there's a section titled "Responsive column count" with a demo grid that should have 3 columns when screen size is between 600px and 1200px. I tried this and it doesn't appear to be working (I get 6 columns above 1200px and 2 columns everywhere else).
Screenshot below:

I've never used Picnic CSS before, just reading the documentation for the first time and discovered this issue. I'm assuming it's a bug rather than me not reading the description properly - do let me know if I've assumed incorrectly.

Note: give it a bit so the CDN updates.


Release 0.3/0.2.1

Would it be possible to have another release of go-vex published? We're relying on some changes to main for https://github.com/guacsec/guac/ and would ideally use a released version.

It looks like v0.2.1 is cut, does it include the change you are using? @lulf

Indeed it does, looks like I missed that announcement :)


Move to using a static version number

Using a static version will make it easier to promote binary
artifacts. However, it comes with the cost of having to managing
bumping the version as part of the normal development cycle.

@chef/lob @schisamo The goal of moving to a static version is so that the we promote a binary artifact.  Since the version is set at build-time, we need a way for an unreleased package to have the correct version if it is promoted.
The challenge here is that we will have to manage this version number as part of the development flow rather than just as part of the release cycle.
Thoughts? Alternatives?

One other point of clarification, all builds will also have a timestamp attached:
MAJOR.MINOR.PATCH+YYYYMMDDHHMMSS

This approach will also do away with any git describe data. This data has mostly proven frivolous and the SHA of the Omnibus project used to build an artifact is now contained in the version-manifest.

:+1:


obasic.com save command

obasic.com
"save" command gives error

I managed to reproduce it when saving with a file name with lowercase letters:
44531 Bytes free
BASIC Rev. 4.51
[CP/M Version]
Copyright 1977 (C) by Microsoft
Ok
10 print 1
save "UPPER"
[[BDOS command 26: F_DMAOFF(4725)]][[BDOS command 19: F_DELETE(4700)]][[Delete file UPPER.BAS]][[=>00]]
[[BDOS command 22: F_MAKE(4700)]][[Create file UPPER.BAS]][[=>00]]
[[BDOS command 26: F_DMAOFF(4725)]][[BDOS command 21: F_WRITE(4700)]][Write record 0 from 4725][[=>00]]
[[BDOS command 26: F_DMAOFF(4725)]][[BDOS command 16: F_CLOSE(4700)]][[=>00]]
Ok

save "lower"
[[BDOS command 26: F_DMAOFF(4725)]][[BDOS command 19: F_DELETE(4700)]][[Delete file lower.BAS]][[=>ff]]
[[BDOS command 22: F_MAKE(4700)]][[Create file lower.BAS]][[=>00]]
[[BDOS command 26: F_DMAOFF(4725)]][[BDOS command 21: F_WRITE(4700)]][Write record 0 from 4725][[=>01]]
Disk I/O error[[BDOS command 26: F_DMAOFF(4725)]][[BDOS command 21: F_WRITE(4700)]][Write record 1 from 4725][[=>01]]
Disk I/O error[[BDOS command 26: F_DMAOFF(4725)]][[BDOS command 21: F_WRITE(4700)]][Write record 2 from 4725][[=>01]]
Disk I/O error[[BDOS command 26: F_DMAOFF(4725)]][[BDOS command 21: F_WRITE(4700)]][Write record 3 from 4725][[=>01]]
Disk I/O error[[BDOS command 26: F_DMAOFF(4725)]][[BDOS command 21: F_WRITE(4700)]][Write record 4 from 4725][[=>01]]

OBASIC does not convert to uppercase (as CCP and most programs do) and can generate files that are hard to delete. See https://groups.google.com/g/comp.os.cpm/c/BZGKhIFNWjo
CP/M file directories can be upper and lower case, but for iz-cpm to be convenient to deal with host files that are usually in lowercase, i internally transform to uppercase. See https://github.com/ivanizag/iz-cpm/blob/master/src/fcb.rs#L213
Removing that line, the problem is solved for OBASIC.COM. But it turns less convenient with most other programs.
I'll have to think what is the best solution. Meanwhile, saving works when using all uppercase file names.

the error kills source code of basic program!


Cleanup code and spelling errors

based on #43
general bug fixes, nothing breaking

I reverted changes made in here


replace float:left and clear:left by display: flex (much easier)

Colorbox still supports legacy browsers, so I doubt I'll be doing this unless it's part of a larger rewrite.


i7-7700k + ASUS GTX 1080Ti ROG Strix + 16GB RAM = 0% improvement

I have the same FPS (45 - 75) after this "patch" as before at 1080p... 😢🤷♂

As written on the nexus mod page "If you are GPU bound you will not see any performance increase on your average FPS but you should see less FPS drops."


Add Unit tests.

Split out from: https://github.com/smithfield-studio/acf-svg-icon-picker/issues/1

I will try to port some of the test setup from https://github.com/timber/starter-theme to this!

#25 will also close this issue.


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2024-05-30T14:12:35+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.22
7d0316f5a1ab0b43d7b7aa62cefe26fd0121a043
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/test unit-tests-arm64


fix(vue-app): allow passing Props from Layout to Pages

In this PR we allow passing props from layouts to NuxtChild / Pages.
When you create a layout that defines a prop in the Nuxt component, as an average layperson, you would assume that the page receives the prop but it doesn't.
Example Sandbox

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

The attributes of the Nuxt component are added as props to the NuxtChild.

This PR resolves https://github.com/nuxt/nuxt.js/issues/7274, resolves https://github.com/nuxt/nuxt.js/issues/1502, resolves https://github.com/nuxt/nuxt.js/issues/1894
Bugreport: https://cmty.app/nuxt/nuxt.js/issues/c10571
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[x] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #7275 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #7275   +/-   ##
=======================================
  Coverage   62.87%   62.87%
=======================================
  Files          84       84
  Lines        3410     3410
  Branches      932      932
=======================================
  Hits         2144     2144
  Misses       1018     1018
  Partials      248      248

Flag
Coverage Δ

#unittests
62.87% <ø> (ø)

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 e089ac6...97b4d48. Read the comment docs.

Hi there! This is my first PR to nuxt and I don't get why my page is not able to be routed in the tests. When running the fixtures in my development environment this page is rendered properly. What am I missing here?

Codecov Report

Merging #7275 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #7275   +/-   ##
=======================================
  Coverage   70.26%   70.26%
=======================================
  Files          88       88
  Lines        3696     3696
  Branches     1008     1008
=======================================
  Hits         2597     2597
  Misses        892      892
  Partials      207      207

Flag
Coverage Δ

#unittests
70.26% <ø> (ø)

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 5c348c0...9b080ac. Read the comment docs.

@dariusbepunkt The error is probably because the page is named /with-prop but the test requests /with-props
Btw, I think we can simplify the nuxt component a little bit. We can remove all props here, check for this.$attrs.nuxtChildKey instead of this.nuxtChildKey in routerViewKey and only pass props: this.$attrs to nuxt-child instead of creating a new object with both props & attrs.

I am closing this PR since it won't be added in Nuxt 2 but available in Nuxt 3


feat: Updating AthenaSQL stage to handle multiple queries

Feature or Bugfix

Feature

Detail
The feature enhancement takes into consideration a new prop variable called parallel. This will determine if user wants to add "n" number of Athena query step sequentially or parallel and then use the logic accordingly. For this change the queryString variable had to be updated from a string to array which will determine total number of steps. Also ensured that queryStringPath functionality still works. Overall the stage would handle 3 patterns.
Relates
#316
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: DDKIntegrationTestsD8472609-5SCHWjldmWcf
Commit ID: d2b65ee37a48151b98a154ac7abaca5eb2c22b26
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: DDKIntegrationTestsD8472609-5SCHWjldmWcf
Commit ID: c8805d4fb2ef09d278946457e264a48a38606e10
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


feat: add support for tweetdeck client

Add support for TweetDeck client (tweetdeck.twitter.com)

Thanks for the contribution.
This project uses develop as its default branch, could you create a pull request base on develop branch?
The master branch is a bit older, which does not contain the commits of the manifest upgrade.

I couldn't get the develop branch's v3 manifest to work, there were errors when loading the extension on both Chrome & Firefox (something about the service worker).

Use ./watch to build the background.js

@larvata I've updated the PR to merge into develop.
However, I couldn't get the popup component to work on this branch.

@akiroz do you mean this? It works on my side.

Your PR looks good to me, I'll merge it in the next few days.

Yes, the icon didn't appear on my address bar so I wasn't able to test color changing functions on this branch. Anyway thanks for testing it out!

version 1.3.0 has been published.
Thank you for your contribution.


xaxis update

Change x axis to only show year, closes #704

can't visually validate this right now because Augur is down. will check another time

@JamesKunstle Sounds good, augur is back up whenever you get a chance to check it out


Add the create App feature

Add the features for creating an app :

creating an app from a pre-configured app (mlflow, jupyter....)
creating a custom app by entering docker information

Apparently Saagie uses the createJobMutation to create an App, see example below :
`{
"operationName":"createJobMutation",
"variables":{
"job":{
"projectId":"8321e13c-892a-4481-8552-5be4b6cc5df4",
"name":"Test MLflow",
"description":"",
"category":"",
"technology":{
"id":"d4beece1-3069-42bb-956f-4388c20fda78"
},
"isStreaming":false,
"isScheduled":false,
"storageSizeInMB":128
},
"jobVersion":{
"dockerInfo":{
"image":"saagie/mlflow-server:1.16-1.0"
},
"exposedPorts":[
{
"basePathVariableName":"SAAGIE_BASE_PATH",
"isRewriteUrl":true,
"isAuthenticationRequired":true,
"port":5000,
"name":"Mlflow Server"
}
],
"storagePaths":[
     ],
     "releaseNote":"",
     "resources":{
        "cpu":0.3,
        "disk":512,
        "memory":512
     }
  }

},
"query":"mutation createJobMutation($job: JobInput!, $jobVersion: JobVersionInput!) {\n  createJob(job: $job, jobVersion: $jobVersion) {\n    id\n    name\n    __typename\n  }\n}\n"
}`

With the App refactoring there are 2 versions of the API. Either we install an pre configured app :
{
   "operationName":"installAppMutation",
   "variables":{
      "projectId":"60d4133b-c0bc-4e7e-bc5e-ea8ccce91122",
      "technologyId":"d4beece1-3069-42bb-956f-4388c20fda78",
      "contextId":"mlflow-server"
   },
   "query":"mutation installAppMutation($projectId: UUID!, $technologyId: UUID!, $contextId: String!) {\n  installApp(projectId: $projectId, technologyId: $technologyId, contextId: $contextId) {\n    id\n    name\n    __typename\n  }\n}\n"
}

or we create a custom app from scratch
{
   "operationName":"createAppMutation",
   "variables":{
      "app":{
         "projectId":"60d4133b-c0bc-4e7e-bc5e-ea8ccce91122",
         "name":"zet",
         "description":"",
         "technologyId":"36912c68-d084-43b9-9fda-b5ded8eb7b13",
         "version":{
            "dockerInfo":{
               "image":"repo/image:v1"
            },
            "ports":[
               {
                  "number":"80",
                  "isRewriteUrl":true,
                  "basePathVariableName":null,
                  "scope":"PROJECT"
               }
            ],
            "releaseNote":"",
            "volumesWithPath":[

            ]
         }
      }
   },
   "query":"mutation createAppMutation($app: AppInput!) {\n  createApp(app: $app) {\n    id\n    __typename\n  }\n}\n"
}

Trello card: [#50] Add the create App feature

For the pre-configured app, the endpoint is available, but should wait the deployment of app v2 in order to use it.


Simplified Linux clipboard support

According to the README,

Reading images from clipboard works only on Windows and macOS, on Linux you should read from a directory instead.

Checking the logic, there's two distinct ways manga_ocr handles clipboard:
On Wayland, the pyperclip library  is used to get clipboard contents
https://github.com/kha-white/manga-ocr/blob/b0cac91777d6143c178329986302576adf49f9de/manga_ocr/run.py#L72
while on Windows and MacOS it's through Pillow's PIL.ImageGrab.grabclipboard
https://github.com/kha-white/manga-ocr/blob/b0cac91777d6143c178329986302576adf49f9de/manga_ocr/run.py#L93
Although the documentation for grabclipboard() says

Only macOS and Windows are currently supported.

it's undocumented but the Pillow source shows that Linux support was added in 9.4.0 (using wl-paste and xclip).
Thus one can change the existing code from
https://github.com/kha-white/manga-ocr/blob/b0cac91777d6143c178329986302576adf49f9de/manga_ocr/run.py#L66-L86
to
        # if sys.platform not in ('darwin', 'win32') and write_to == 'clipboard':
        #     # Check if the system is using Wayland
        #     import os
        #     if os.environ.get('WAYLAND_DISPLAY'):
        #         # Check if the wl-clipboard package is installed
        #         if os.system("which wl-copy > /dev/null") == 0:
        #             pyperclip.set_clipboard("wl-clipboard")
        #         else:
        #             msg = 'Your session uses wayland and does not have wl-clipboard installed. ' \
        #                 'Install wl-clipboard for write in clipboard to work.'
        #             raise NotImplementedError(msg)

        if read_from == 'clipboard':

            # if sys.platform not in ('darwin', 'win32'):
            #     msg = 'Reading images from clipboard works only on macOS and Windows. ' \
            #         'On Linux, run "manga_ocr /path/to/screenshot/folder" to read images from a folder instead.'
            #     raise NotImplementedError(msg)

            from PIL import ImageGrab
            logger.info('Reading from clipboard')

and preserve support for Wayland while adding support for Linux systems running X.
This requires Pillow>=9.4.0 and pyperclip can probably be removed.
(see https://github.com/kha-white/manga-ocr/compare/master...stephen-huan:manga-ocr:master for an implementation)
If neither wl-paste nor clip are available, the NotImplementedError should propagate through the try catch.
Note that the implementation of grabclipboard() on Linux is currently a bit wonky, see https://github.com/python-pillow/Pillow/issues/7147.
In particular the current implementation will (a) generate a lot of unnecessary temporary files and (b) raise either ChildProcessError from xclip reporting Error: target image/png not available or UnidentifiedImageError from when Pillow tries to parse non-image data, e.g. plaintext when the other operating systems return None on invalid data.
It may be worthwhile to temporarily patch
            try:
                img = ImageGrab.grabclipboard()
            except OSError:
                logger.warning('Error while reading from clipboard')

to
           try:
               img = ImageGrab.grabclipboard()
           except (ChildProcessError, UnidentifiedImageError) as error:
               logger.trace(error)
           except OSError:
               logger.warning('Error while reading from clipboard')

since the default level of loguru is debug, using trace for the parse errors that occur every delay_secs prevents the user from being spammed when running with the default command manga_ocr. However, using the LOGURU_LEVEL  environmental variable allows seeing every logging message, for debugging reasons.
LOGURU_LEVEL=TRACE manga_ocr

It may also be worthwhile to temporarily maintain a patched implementation of grabclipboard() from Pillow, at least until the next quarterly release of Pillow comes out (on July 1st). One should also be sure to copy the license.

I see; I misunderstood the purpose of that code.
I also figured out that one can use clipnotify to block until the clipboard is updated, instead of polling.
Replacing
            time.sleep(delay_secs)

with
        has_clipnotify = os.system("which clipnotify > /dev/null") == 0
        ...

            # on linux with the clipnotify package
            if sys.platform == "linux" and has_clipnotify:
                # block until clipboard updates
                with subprocess.Popen(
                    ["clipnotify", "-s", "clipboard"],
                    stdout=subprocess.PIPE,
                ) as process:
                    process.stdout.read()  # type: ignore
            # poll in a platform-agonistic manner
            else:
                time.sleep(delay_secs)

is all that is necessary. The solution is Linux and X11 specific however, and requires clipnotify as a runtime dependency.

Addressed by https://github.com/kha-white/manga-ocr/pull/44.


[WIP] docs(guide): add Dart intro

@wardbell Here's my work in progress. We still need to get the examples ready, but I think the text is close. You can see the diffs between the TS & Dart versions here:
https://github.com/kwalrath/angular.io/commit/f0fa8e8ff180736fbd7a34b06004429362279b52#diff-d402be7e11ce92b900f4020006d7c80d

@tomplusplus when https://github.com/angular/angular.io/pull/448 is in (or at least when the docregions are in that PR) I'll update this PR to use the real Dart code.

#448 is updated with the docregions

Great. Thanks!

FWIW,  I recently split intro into intro and architecture. You may want to follow suit

Will do. Thanks for the heads up.


doc: update conventions with repl examples and function (in|out)puts

Description of changes
Closes #276829.
This PR adds two more items to the nixpkgs documentation conventions as discussed in the documentation meetings and PR reviews.
This is related to #278769, where I'm going through the nixpkgs manual to make it more consistent (and also update its content). I'm documenting the conventions I'm applying as I do this work in an effort to:

Let other people know the conventions I'm using, should they want to help with this work.
Let other people start using the same conventions for future work.
Have conventions clearly documented, because they weren't (or didn't even exist) before this work started.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Regarding function inputs/outputs. I am not so sure about ditching f :: A -> b -> C style notation.
Some reasons why it's nice (to me):

it's very consistent in style, which is helpful when trying to quickly find info
it's easy to auto-generate (given some doc strings conventions as current RFCs are pushing, or possibly, eventually: a type system)
it helps communicate that it is very possible to partially evaluate functions. So for instance, if repeatN :: String -> Integer -> String, and "hello" :: String, we have repeatN "hello" :: Integer -> String. Put differently, there are three, equally valid ways to look at repeatN:

repeatN is a function which takes String as input and produces another function Integer -> String as output, or
repeatN is a function which takes a function String -> Integer as input and produces a String as output, or
repeatN is a function which takes a String and Integer as input and produces a String as output

it is already a convention used by some Nix IDE tools and documentation
it is already a convention used by some languages (e.g. Haskell), and similar-ish to the BNF convention more generally

Some reasons why its not nice (to me):

it takes some getting used to (but its knowledge that is helpful in other languages too)
nothing is said immediately about what String is required (having some way to include variable names along with the types would be helpful...)

But I can imagine a convention where we write along with runCommand :: String -> AttrSet -> String -> Derivation something like runCommand :: name -> drvAttr -> buildCommand -> Derivation so that we have overall:
runCommand :: name -> drvAttr -> command -> Derivation
                        :: String -> AttrSet -> String -> Derivation

Returns a derivation realizing a binary executing specified shell commands.

name :: String
    The name that Nix will append to the store path in the same way that stdenv.mkDerivation uses its name attribute.

drvAttr :: AttrSet
    Attributes to pass to the underlying call to stdenv.mkDerivation.

buildCommand :: String
    Shell commands to run in the derivation builder.

Note
    You have to create a file or directory $out for Nix to be able to run the builder successfully.

I do not think your changes address the issue I raised. I tried to express this once, but made mistakes. So here's the second try.
In most places, there's a "headerless" style of documentation: e.g. nixdoc generates output like so.
There might be other styles, including headered ones. It's worth understanding which "local styles" are already in use. The nixos-render-doc work might have had a list of existing styles in use (not easy to (re)find).
To fix potential issues around misunderstanding of colloquialisms, note that "to ditch" means:

To discard or abandon.

I will stick to discard from now on for clarity.
The convention as it stands now---even though a work in progress---has the potential to influence new PRs which choose to change existing f :: A -> B -> C information to relying on the Input/Output header structure alone to convey information. (There was already one discussion in the matrix around this.) This is the sense in which f :: A -> B -> C information may be discarded.
Therefore:

Please provide an explanation of why headers will help readers. Most function documentation styles, in most languages (e.g. Rust or Haskell) do not use headers. They rely on type information alone. to act as headers.

Some third party Python documentation has subtle headers, but the official ones do not. MDN Javascript documentation does. However, the MDN documentation also has an explicit/separate page for each function (not one long scroll-through document, for example like the current Nixpkgs manual).

Please do not select a convention, without a summary of existing function input/output styles, and some analysis of their pros and cons. (In particular, of headerless styles.)

Please consider improving the structure of headerless styles, as an alternative to headers.

Please consider choosing to be clear about what ought to be included under each header. Why leave what goes under each header up for grabs?

Please consider marking clearly that this document is a work in progress that does not endorse any convention yet and does not encourage its use as inspiration just yet.

@bzm3r
In my opinion, the changes I made do address the issue. From what I understood, this seems to be your problem with this PR:

...has the potential to influence new PRs which choose to change existing f :: A -> B -> C information to relying on the Input/Output header structure alone to convey information

And I have already addressed in the PR that the headers are there for navigational purposes, and they don't force the content to be written in any specific way. Anyone can still write f :: A -> B -> C in the documentation, but when talking about the inputs specifically, it would be best if they put it under a header. This doesn't mean f :: A -> B -> C shouldn't exist. These aren't conflicting things. f :: A -> B -> C is content, the headers exist to help navigate the manuals.
The need for headers is way less pronounced in the nixpkgs.libs section of the manual (because those are usually short), but it's definitely needed in many other places in the manual, because there are a lot of utility functions that handle a lot of stuff behind the scenes.

Please provide an explanation of why headers will help readers

Headers have anchor links that people can use to jump to a specific part of the document, and use in links for other people as well.
Manuals are used more as a consultation device. This section of the diataxis framework page helps here. Because of that, making things consistent by having headers that are standardised across the manual helps people quickly consult it.

Most function documentation styles, in most languages (e.g. Rust or Haskell) do not use headers.
The Rust documentation uses headers. I picked a random page there for std::string::String. Note how it has "Examples", "Errors", "Panics" (and so on...) headers for a lot of the functions:

Examples for new()
Examples for with_capacity()
Errors for from_utf8()
Examples for from_utf8()

Notably, the Rust documentation doesn't have an "Inputs" header because most of its functions don't have arguments with 10+ different attributes like dockerTools.buildImage, for example. Python's functions sometimes have a large number of arguments, and in my opinion it's a really bad experience to navigate their docs. Note that even then, they also have some similarities with the conventions in this PR, for example how they move examples to be the last part of each unit they're documenting: examples for regular expressions, examples for datetime.

Please clarify, in the first post of this PR, what precisely was agreed upon in team meetings as to what the goal of this document is.
Done.

Please consider improving the structure of headerless styles, as an alternative to headers.
As I wrote already, these are not conflicting things.

Please do not select a convention, without a summary of existing function input/output styles, and some analysis of their pros and cons. (In particular, of headerless styles.)
I believe I've already addressed that what you're calling a "headerless style" relates to the content of the documentation, not to its structure, and those are not conflicting things.

As for selecting a convention, this is exactly what we're doing in the documentation team meetings. I find an opportunity to standardise something in the nixpkgs manual, then I bring it to the meeting, we discuss it there, and we decide on something. And that is then documented in a PR just like this one.
We shouldn't be spending this much time arguing over the structure of stuff in the nixpkgs manual (and I'm only spending this much time replying to all of this because I think you're being genuine in your concerns and I respect that).
The good thing about the structure of something is that it can easily be changed later (as opposed to the content, which is way harder). If we find out later that some other structure is better, we can quickly move to it. We might need to adapt some text in the process, but we won't need to completely change the content of the documentation.
(and I already know this is inefficient and so on, all of this is already addressed in #278769 - I'm the one doing the work to bring consistency to the manuals, so I'll work this way, even though it seems inefficient, because it's what helps me make progress on the content as quickly as possible, which is ultimately what matters)

Please consider choosing to be clear about what ought to be included under each header. Otherwise, please explain why this document would prefer to leave what goes under each header up for grabs?
It should be up to whoever's writing the documentation to figure out what content goes where, and even whether they should stick to the convention or not. I can't predict all the possible content that goes in the manuals, but I do know that a good part of it documents utility functions, and those follow roughly the same documentation structure. Hence why a convention for following the same structure (if it makes sense, which is also to be determined by whoever's writing the documentation).

I thought this was kind of implied in the conventions, but I'll update the PR to make it clear that the conventions are about keeping a consistent structure in the manuals.

Please consider marking clearly that this document is a work in progress that does not endorse any convention yet and does not encourage its use as inspiration just yet.
This document does encourage people to use the conventions laid out in it. That's (also) why they're being documented.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2024-01-15-documentation-team-meeting-notes-104/38305/1

@DanielSidhion Okay: I will step out of your way then, and not raise further issues here on this point.
If in the future I feel that that the Input/Output header style does not work well, I'll create a separate PR to address that.

We shouldn't be spending this much time arguing over the structure of stuff in the nixpkgs manual (and I'm only spending this much time replying to all of this because I think you're being genuine in your concerns and I respect that).

Could you update the first post of this PR again with what was discussed in this documentation team meeting?
For now, some final thoughts that are mostly just notes for myself:

structure of content is essential, while cosmetics of their presentation is secondary (and malleable)
linkability and searchability of documentation are key in making it "map-like"
headers are not necessary for content to be linkable
content can be a header (for example: in Rust, the function signature)
if a header does not promise content of certain value, and consistently deliver on that promise, it will become useless to readers

Reviewed in the docs team meeting, looking good!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2024-01-25-documentation-team-meeting-notes-106/38792/1


Failed install on TYPO3 6.2.13/4

Hi
I am trying to install  (TYPO3 6.2.13, +) t3onepage distribution and I fail.
I get to the Configure/View/start /Start to edit page, hence everything should be ok, but indeed the page tree is still empty.
Is there anything I need to set or check?
Thanks Regards
mario

hi @mario6097,
please import the data.t3d manually from extension root.


Remove boilerplate images from LFS

[AUTOMATED] Latest version has been pushed to preview.
Preview deployed at https://preview-e560554.dev.nextjs-boilerplate.jam3.cloud.jam3.net

I don't know why the images don't appear in the PR diff.
They exist in the branch and in the preview, though:
https://github.com/Jam3/nextjs-boilerplate/blob/fix-lfs/docs/logo.jpg


Dio won't catch errors and get stuck on request when server throws an error

New Issue Checklist

[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
flutter

Platform Version
2.0.3

Dio Version
4.0.0-prev3

Issue Description and Steps
I am making a request (of any type: get,post,delete,patch) and it works if the server return a value, but when the server throws an error it get stuck and won't catch the error.
This issue occurred when upgrading from 4.0.0-beta7 to 4.0.0-prev3.
This is how I make the request:
 try {
      final Response<dynamic> rs = await dio.get<dynamic>(
        requestInfo.path,
        queryParameters: requestInfo.queryParams,
        cancelToken: cancelToken,
      );

      // It won't reach this code if there is an error.

      return rs.data;
    } catch (error) {
      // And most surprisingly it won't reach this code if there is an error.

      _onError(error);
    }

Please help,
Thanks

Hey @wendux can you please take a look at this? It is still happening in stable version 4.0.0.

to anyone also having this problem, the solution is (for me at least) is to not forget to put return handler.next(error); at the end of the onError interceptor.

@naamapps Thank you!!! That new way of writing interceptors is not as trivial as I thought at first.


UnhandledPromiseRejectionWarning: Error: spawn EACCES

Steps to reproduce
Tell us about your environment:

Puppeteer version: v1.0.0
Platform / OS version: macos 10.13.2
URLs (if applicable):
Node.js version: v9.4.0

What steps will reproduce the problem?
Please include code that reproduces the issue.
const puppeteer = require('puppeteer');

 puppeteer.launch({
      executablePath: './chromium/Chromium.app'
  }).then(async browser => {
      const page = await browser.newPage();
      await page.goto('https://y.qq.com');
      await page.screenshot({path: 'yqq.png'});
      await browser.close();
  });

What is the expected result?
What happens instead?
(node:73106) UnhandledPromiseRejectionWarning: Error: spawn EACCES
at _errnoException (util.js:1003:13)
at ChildProcess.spawn (internal/child_process.js:340:11)
at Object.exports.spawn (child_process.js:499:9)
at Function.launch (node_modules/puppeteer/lib/Launcher.js:101:40)
at

Same here, tried with different versions of node, npm, installing puppeteer globally, checking chromium is installed as root, permission issues, etc, nothing worked. In windows everything works fine, (im trying it in ubuntu), and seems its a problem with the chromium that puppeteer installs.

Does ./chromium/Chromium.app exist? Does pptr's bundled version work for you?

In my case, considering i am in ubuntu, ./chromium/Chromium.app does not exists, but it exists  ./node_modules/chromium/lib/chromium/chrome-linux/chrome, either specifiying it in executablePath option or not specifiying it, is the same result. For @Lone a solution could be check ./chromium/Chromium.app permissions as said here: issues/1649

Dupe of #1649.

For people arriving here from Google and looking for a Linux (Debian for me) solution: The locally installed version of Chrome had very restrictive executable permissions for me (700), so if your not the only person running the tool or if your process switches to another user during runtime (process.setuid) you may have to chmod/chown your permissions correctly.

It's about the permission issue.
In my case, when I tried to run with ubuntu user, it works;
when run with www-data, it failed
Thus I changed the permission to the chrome folder, i.e.
$ chmod 777 node_modules/puppeteer/.local-chromium/linux-536395/chrome-linux/*

P/S: Change all files in that folder, because initially I only change chrome file, but it still doesn't work.

Also same problem On ubuntu 16
Chmod 777 same user as the node process
still not working..
this must be one of the most frustrating issues come across so far
(node:5062) UnhandledPromiseRejectionWarning: Error: spawn EACCES
    at ChildProcess.spawn (internal/child_process.js:330:11)
    at Object.exports.spawn (child_process.js:500:9)
    at Function.launch (/srv/us/node_modules/puppeteer/lib/Launcher.js:98:40)
    at <anonymous>
(node:5062) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:5062) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

I tried running chmod 777 node_modules/puppeteer/.local-chromium/linux-536395/chrome-linux/* as well but for me the folder linux-536395 doesn't exist. Mine is a different version linux-543305. And it worked on this folder. Maybe you should check whether the folder exists. Hopefully that helps.

Run npm without sudo

this article solved it for me - https://jmfeurprier.com/2015/10/02/how-to-install-node-js-with-ubuntu-and-vagrant-in-a-synced-folder/
I use Puppeteer on Vagrant, so had permission issues. The trick was to create a dir into vagrant home folder  (/home/vagrant/node_modules/) and then create a symlink for the project in vagrant. worked like a charm!


Set font-size for table to 7.0 for the RVV C intrinsics specification

Hi Bill @wmat  , this changes is needed for the intrinsics specification.

Looks good


Twilio SMS notifications

Addresses #12
This PR implements basic Twilio notifications from a single Twilio number to multiple recipients. I have update README.md to include Twilio instructions and relevant environment variables that would need to be included to use this notify method. Open to any tweaks, I tried to follow the style / conventions of other examples in the repo.
Thanks!

Lint fails: https://github.com/pcarn/vaccine-watch#lint
If you don't want to install pre-commit, I can commit the changes to your fork if you'd rather.

Thanks for the review. Linting should pass now.

@djsiroky I sent an invite for write access, in case there's anything else you want to change.
You should have access to merge when you're ready.


Error in binary semisupervised classification

I got the below error calling the automl.score() method in https://github.com/VIDA-NYU/alpha-automl/blob/devel/examples/semi_supervised_classification.ipynb
ValueError: Target is multiclass but average='binary'. Please choose another average setting, one of [None, 'micro', 'macro', 'weighted'].

The error was raised because the test dataset didn't have labels. I updated the test dataset and it was fixed.


Make color a per-stream property rather than per channel

ngscopeclient import bin with multiple channel shall have their own colors
During import of bin (it is probably the same for other import with multiple channel) there is only one color for all channels
Improvement:

Add one color R/G/B per channel instead of a global color R/G/B for all channels

Example of today import with 3 channels on a bin file (with all the same colors which is quite confusing and hard to use)

The implementation of this is straightforward, the UI side is more of an open question.
The channel color is used in, for example, the filter graph editor to distinguish filters from each other. We also do want to allow related channels to be visually tied together (e.g. mag and angle of the same S-parameter import).

Having RGB color for each stream seems the best in worst case all color will be identical and that will work for all cases


Add liana version in gui settings

close #243

I prefer to show both, because we may have to release a patch for the gui without touching the daemon for example.

Makes sense. Let's reuse the struct definition at least then? Feel free to add a third leg to the existing one.
------- Original Message -------
Le jeudi 2 février 2023 à 12:35 PM, Édouard @.***> a écrit :

I prefer to show both, because we may have to release a patch for the gui without touching the daemon for example.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


Image embedded in email contains full path to image

New to all this so any help (or graciousness on errors in implementation) appreciated!
I am using endroid qr-code to generate qr codes to be used to check in will-call list folks at an event.
I have the qr-code generating and saving to a private folder (qr) one level above my webroot on my server.
I'm creating a temporary symlink to the private folder on the fly in the code (then destroying the symlink after its no longer needed).
The saved qr code filename is generated for each based on the individual - like: qr---.png (where  are variable data).
My line in the page that sends the email using PHPMailer is:
<img style="width:200px;height:auto;" alt="PHPMailer" src="cid:qr-code">
But when saving the embedded qr code from the resulting email the filename (in Gmail - using the download icon) the filename shows as:
home__qr_qr-1-Mary-Jones.png
I see the "/" are being replaced by "" but can anyone tell me if it's possible to remove the homeqr when sending the email so it just has a filename of qr-1-Mary-Jones.png ?
THANKS!

Hi @craigkendall I am not familiar with PHPMailer so I can not answer this question. This page mentions some AddEmbeddedImage method where you can pass a filename. Maybe you can use that. Good luck!

Thanks @endroid - I realized it wasn't PHPMailer but my use of temporary symlinks that was failing. I changed that up and it's working mostly as I intended now. Appreciate the response and the great library.


fix: allow formatting of named mapping params

The v0.8.18 version of solidity includes supported for named mapping parameters. The 1.1.2 version of prettier-plugin-solidity resolves the issue.
Fixes #408.

Codecov Report

Merging #409 (308a057) into development (d34fe2e) will decrease coverage by 0.05%.
The diff coverage is n/a.

:exclamation: Current head 308a057 differs from pull request most recent head 8abd153. Consider uploading reports for the commit 8abd153 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@               Coverage Diff               @@
##           development     #409      +/-   ##
===============================================
- Coverage        55.73%   55.68%   -0.05%
===============================================
  Files              175      175
  Lines             4464     4464
  Branches           735      735
===============================================
- Hits              2488     2486       -2
- Misses            1744     1745       +1
- Partials           232      233       +1

Impacted Files
Coverage Δ

src/utils/index.ts
80.00% <0.00%> (-6.67%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Rancher deployment

Research how k8s was deployed via rancher to adopt for future Prometheus k8s scrappers. For example, I can't see kube-proxy pods and etc.

It seems that Rancher deploys K8S core components as containers, not as pods: https://rancher.com/docs/rke/latest/en/config-options/services/.
Need ssh access to instances to check. Wasn't able to login to Hetzner vm's.


update week10

I´m not 100% sure about the last change, but I think you relate to the second assignment within week 10, not the one to come up in january...

Thanks a lot!


Shorter text on buttons

Made text labels shorter so they wouldn't overblow the main panel buttons

Thanks again! Every addition regarding language support is appreciated! I'd also take and link to your translated README once it's finished.


[BUG]: Compare model generates nonsensical summary of two models

https://app.staging.terarium.ai/projects/33f364c1-1da2-4cf7-a176-9870ef1a3ab6/workflow/a7a94429-6bd3-457a-a8c9-a57c2f3525e9?operator=77a5de63-a58f-4a9a-9456-aabd7a135623
Compare an SEIRHD model with an SIR model, the summary starts to talk about image classifiers, pronoun biases, and training cost. None of these are relevant to the actual model inputs, which are not image classifiers nor require any sort of training.
Likewise, comparing the same model against itself (SIR vs SIR) generated garbage, it doesn't know the models are identical and hallucinates.
Example of SEIRHD vs SIR
To provide a detailed comparison of the metadata for multiple models, I will focus on several key aspects:
model architecture, training data, performance metrics, intended use cases, ethical considerations, and limitations.
This will help domain experts understand the nuances and trade-offs between the models.

### Model Architecture
  **Model A:**
  - **Type:** Transformer-based
  - **Layers:** 12
  - **Parameters:** 110 million
  - **Special Features:** Incorporates a novel attention mechanism to improve long-range dependencies.

  **Model B:**
  - **Type:** Convolutional Neural Network (CNN)
  - **Layers:** 20 - **Parameters:** 50 million
  - **Special Features:** Utilizes depthwise separable convolutions to reduce computational complexity.

### Training Data
  **Model A:**
  - **Dataset Size:** 1 billion tokens
  - **Sources:** Diverse text corpora including books, articles, and web pages.
  - **Preprocessing:** Tokenization, lowercasing, removal of special characters.
  **Model B:**
  - **Dataset Size:** 10 million images - **Sources:** Publicly available image datasets such as ImageNet.
  - **Preprocessing:** Normalization, resizing, and data augmentation techniques like rotation and flipping.

### Performance Metrics
  **Model A:**
  - **Accuracy:** 92% on benchmark text classification tasks.
  - **F1 Score:** 0.89
  - **Latency:** 50ms per inference on a standard GPU.
  **Model B:**
  - **Accuracy:** 95% on image recognition tasks.
  - **F1 Score:** 0.93
  - **Latency:** 30ms per inference on a standard GPU.

 ### Intended Use Cases
  **Model A:**
  - **Primary Applications:** Text classification, sentiment analysis, and language translation.
  - **Secondary Applications:** Named entity recognition (NER), summarization.

  **Model B:**
  - **Primary Applications:** Image classification, object detection, and facial recognition.
  - **Secondary Applications:** Image segmentation, style transfer.

  ### Ethical Considerations
  **Model A:**
  - **Bias:** Potential for gender and racial bias due to imbalanced training data. For example, if the training data contains more male pronouns, the model may exhibit a bias towards male entities in tasks like NER.
  - **Mitigation Strategies:** Implementing bias detection algorithms and augmenting the dataset with more balanced examples.

  **Model B:**
  - **Bias:** Risk of reinforcing stereotypes in image recognition tasks. For instance, if the dataset contains more images of certain ethnic groups in specific contexts, the model may develop biased associations.
  - **Mitigation Strategies:** Using diverse and representative datasets, and applying fairness-aware algorithms during training.

  ### Limitations
  **Model A:**
  - **Scalability:** High computational requirements for training and inference, making it less suitable for edge devices.
  - **Generalization:** May struggle with domain-specific jargon or highly specialized texts not represented in the training data.

  **Model B:**
  - **Scalability:** While more efficient than Model A, it still requires significant computational resources for training.
  - **Generalization:** Performance drops significantly on images that differ from the training data, such as those with unusual lighting or occlusions.

  ### Conclusion Both models have their strengths and weaknesses, making them suitable for different types of tasks. Model A excels in natural language processing applications but requires substantial computational resources and careful bias mitigation. Model B is highly effective for image-related tasks and is relatively more efficient but still faces challenges in generalization and bias. Understanding these details can help domain experts make informed decisions about which model to deploy based on their specific needs and constraints.

Needs to review the test, and we need to have a better model card without document and add a new feature to link a document to a model

Ask @j2whiting to see how we can improve relevances of answers.