Heatmap with non-rectangular cells

Hi Scott,
I hope you're fine. I am working on a heatmap application. I have a reference example in matlab. I wasn't sure how to integrate this reference example into scottPlot. I can draw normal heatmap. When I put a double[,] variable in it, everything is great and good. But I need to compress these values to certain angle.
Is this possible? can you help me? There is a function called pcolor in matlab. When you perform a process like the one below, you get the image below. Compresses the given values into a field.
Matlab Code :
[X,Y] = meshgrid(-3:6/17:3); XX = 2*X.*Y; YY = X.^2 - Y.^2; colorscale = [1:18; 18:-1:1]; C = repmat(colorscale,9,1);
Output:

I can share more detailed information if needed.

Hi @saklanmaz! Right now heatmaps are actually implemented as pixels of a Bitmap, and zooming/panning is just resizing the Bitmap before it's rendered to the screen. Because of this implementation, non-rectangular cells are not possible.
However, ScottPlot does support adding arbitrary polygons to plots. https://scottplot.net/cookbook/4.1/#polygon If you're inspired to figure-out the dimensions of each of those squares, color them individually, and add them as polygons to the chart, you can achieve the effect in your screenshot with ScottPlot
If you end-up doing this it would be awesome if you could follow-up here with a code example demonstrating how to do it so that others with a similar question can have a strong starting point if they attempt something similar in the future
Hope it helps!
Scott

Thank you for the information, I will try it quickly. If I can I will share. :)


[RFC] Inject route params as signal

We recently talked about the injectParamsMap function on 𝕏, It may be a good idea to add them here as well.
Here's the code I'm talking about:
export function injectParamsMap(injector?: Injector): Signal<ParamMap> {
  return assertInjector(injectParamsMap, injector, () => {
    const route = inject(ActivatedRoute);
    return toSignal(route.paramMap, {
      initialValue: route.snapshot.paramMap,
    });
  });
}

It can include three functions, injectParamsMap, injectQueryParams, and injectRouteData.
Let me know what you think!

Let's just wait a bit more to see if we hear anything about Signal Input after v17 release

Okay, sure


Documentation doesn't have an api reference

I think it will be easier to use this library if there was an api reference like how DRF has.
thank you for this library it made my life easier

Hi, thanks!
Yeah, absolutely that's a good idea!

i would love something like this http://www.cdrf.co/3.1/rest_framework.viewsets/ModelViewSet.html
should i close this issue

Just leave it open.
Feel free to submit a PR if you wish =)

I will try to get it done this week end

Not pretty, but here's something...
http://drf-haystack.readthedocs.io/en/latest/apidoc/modules.html

thank you 👍  sorry i never got to it

Np ;)


Set up Sunwell on HearthSim CDN

I removed sunwell.min.js from the repo so I'll be moving it to build automatically off our build server.

I'd like to keep it in the repo so people can use it as a direct download and drop-in source. Thats why I put it there in the first place. Having a single CDN for every library in your project is bad practice.

@Paratron I'm not sure what you mean; it'll be autogenerated on our build server now, and it'll be made available on the CDN.
Take Joust for example: https://joust.hearthsim.net/branches/master/joust.js
I'll be doing this ASAP (already working on it); but we can't ask contributors to always regenerate sunwell.min.js. It'll go out of date all the time and increase the size of the repository (git doesn't deal well with big blobs).

I mean that we should have the minified and up to date version of the sunwell.js stored in the repo for people who dont want to use the CDN or dont want to look for a different download source than github. Please put it back into the repo as I did it originally.

I already gave you a heads up about this on skype before blizzcon and it wasn't a problem then, why is it a problem now?
Asking contributors to regenerate sunwell.min.js all the time is not reasonable. The CDN will provide something that is automatically built and compiled on push so that we don't ever have to deal with it again. People who want a drop-in source can use that.
If you want downloads off Github then we can start doing releases, that sounds reasonable.

The minified file is now available here. It should update automatically.


Exception Details: This operation is not permitted on a non-empty directory.

Trying to write data into SQl DW through databricks stream data frame. process is trying to delete the temp folder in the BLOB storage and throwing below. In the documentation i see that process will not automatically cleanup tempdir. Is it true? if true, then why is this error? Using below query in python
df1.writeStream
.format("com.databricks.spark.sqldw")
.option("url", sqlDwUrlSmall)
.option("tempDir", tempDir)
.option("forwardSparkAzureStorageCredentials", "true")
.option("dbTable", "SampleTable")
.option("checkpointLocation", "/tmp_checkpoint_location1")
.option("numStreamingTempDirsToKeep", -1)
.start()
ERROR AzureNativeFileSystemStore: Encountered Storage Exception for delete on Blob: https://savupputest1.blob.core.windows.net/container1/tempDirs/2019-12-20/21-27-29-347/adca2ed6-a705-4274-8c24-0f0e3d7c64a7/batch0, Exception Details: This operation is not permitted on a non-empty directory. Error Code: DirectoryIsNotEmpty
19/12/20 21:27:32 ERROR AzureNativeFileSystemStore: Failed while attempting to delete key tempDirs/2019-12-20/21-27-29-347/adca2ed6-a705-4274-8c24-0f0e3d7c64a7/batch0

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

ID: d863ab39-0b6a-eb81-9dde-5eb25b46cc33
Version Independent ID: 655abadf-5f9e-a65b-0109-c5166f956c4d
Content: Tutorial - Perform ETL operations using Azure Databricks
Content Source: articles/azure-databricks/databricks-extract-load-sql-data-warehouse.md
Service: azure-databricks
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

@sandeep8530 ,  Thanks for raising the issue.We will investigate the issue and update you.

@sandeep8530  Since this issue is not related directly to the doc, I would recommend you to create a thread on the forums - Azure Databricks
Once you post your issue on forums, it will have visibility across the community which is a better suited audience for such types of issues.


Authentication still available in Registry 2.0.0 ?

Hi all,
I'm using Registry 0.9.1 with HTTPd or NGinx in front to handle authentication.
I tried Registry 2.0.0 with docker 1.6.0 but docker client (or registry) didn't works well and client fall back to v1
<IP_ADDRESS> - - [17/Apr/2015:15:41:07 +0200] "GET /v2/ HTTP/1.1" 401 1233 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - - [17/Apr/2015:15:41:07 +0200] "GET /v1/_ping HTTP/1.1" 200 1541 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - - [17/Apr/2015:15:41:07 +0200] "GET /v1/_ping HTTP/1.1" 200 1541 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - my-deployer [17/Apr/2015:15:41:07 +0200] "PUT /v1/repositories/teama1/teama-java5/ HTTP/1.1" 200 2 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - my-deployer [17/Apr/2015:15:41:07 +0200] "GET /v1/images/4f903438061c7180cf99485b42f7709f5268bfb4732fe885f9104ed3bb66fd3c/json HTTP/1.1" 200 1510 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"

When I remove Authentication requirement on HTTPd front side, all operations works in /v2/
Nginx/Compose sample don't show authentication anymore too.
Did Registry 2.0.0 support authentication schemes ?
Thanks

Yes registry 2.0.0 supports authentication.

  ProxyPass        /v1 http://localhost:5000/v1
  ProxyPassReverse /v1 http://localhost:5000/v1

  ProxyPass        /v2 http://localhost:5002/v2
  ProxyPassReverse /v2 http://localhost:5002/v2

  ProxyPass        /_ping http://localhost:5000/_ping
  ProxyPassReverse /_ping http://localhost:5000/_ping

  # Authentication require for push
  <Location /v1>
    Order deny,allow
    Allow from all
    AuthName "Registry Authentication"
    AuthType basic
    AuthUserFile "/etc/apache2/htpasswd/registry-htpasswd"

    # Read access to authentified users
    <Limit GET HEAD>
      Require valid-user
    </Limit>

    # Write access to ecd-deployer only
    <Limit POST PUT DELETE>
      Require user docker-deployer
    </Limit>

  </Location>

  # Allow ping to run unauthenticated.
  <Location /v1/_ping>
    Satisfy any
    Allow from all
  </Location>

  # Authentication require for push
  <Location /v2>
    Order deny,allow
    Allow from all
    AuthName "Registry Authentication"
    AuthType basic
    AuthUserFile "/etc/apache2/htpasswd/registry-htpasswd"

    # Read access to authentified users
    <Limit GET HEAD>
      Require valid-user
    </Limit>

    # Write access to ecd-deployer only
    <Limit POST PUT DELETE>
      Require user docker-deployer
    </Limit>

  </Location>

  # Allow ping to run unauthenticated.
  <Location /_ping>
    Satisfy any
    Allow from all
  </Location>

cc @moxiegirl

Do you need more details, I could provides HTTPd access log (stock and forensic).

@hgomez From the logs, it's pretty clear that the v2 ping returns a 401 and falls back to v1. Are you able to hit https://registry-ancy.teama.mycorp.int/v2/ with curl or your browser using basic auth? Do you have tls configured on apache?

no problem with curl
curl -L -v --user my-deployer:mypassword  https://registry-ancy.teama.mycorp.int/v2/

* Hostname was NOT found in DNS cache
*   Trying <IP_ADDRESS>...
* Connected to registry-ancy.teama.mycorp.int (<IP_ADDRESS>) port 443 (#0)
* successfully set certificate verify locations:
*   CAfile: none
  CApath: /etc/ssl/certs
* SSLv3, TLS handshake, Client hello (1):
* SSLv3, TLS handshake, Server hello (2):
* SSLv3, TLS handshake, CERT (11):
* SSLv3, TLS handshake, Server key exchange (12):
* SSLv3, TLS handshake, Server finished (14):
* SSLv3, TLS handshake, Client key exchange (16):
* SSLv3, TLS change cipher, Client hello (1):
* SSLv3, TLS handshake, Finished (20):
* SSLv3, TLS change cipher, Client hello (1):
* SSLv3, TLS handshake, Finished (20):
* SSL connection using teamaHE-RSA-AES256-GCM-SHA384
* Server certificate:
* 	 subject: C=FR; ST=mycorp; L=mycorp; O=mycorp; OU=mycorp; CN=*.teama.mycorp.int<EMAIL_ADDRESS>* 	 start date: 2013-11-12 12:29:23 GMT
* 	 expire date: 2015-11-12 12:29:23 GMT
* 	 common name: *.teama.mycorp.int (matched)
* 	 issuer: DC=int; DC=mycorp; CN=mycorpRootCA
* 	 SSL certificate verify ok.
* Server auth using Basic with user 'my-deployer'
> GET /v2/ HTTP/1.1
> Authorization: Basic ZWNkLWRlcGxveWVyOmRvY2tlcnJvY2tzaW4yMDE1
> User-Agent: curl/7.35.0
> Host: registry-ancy.teama.mycorp.int
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Tue, 21 Apr 2015 08:56:44 GMT
* Server Apache is not blacklisted
< Server: Apache
< Content-Length: 2
< Content-Type: application/json; charset=utf-8
< Docker-Distribution-Api-Version: registry/2.0
< Host: registry-ancy.teama.mycorp.int
<
* Connection #0 to host registry-ancy.teama.mycorp.int left intact

Side note, 401 came back with Apache HTTPd 401 error page

Authentication required!

    This server could not verify that you are authorized to access
    the URL "/v2/".
    You either supplied the wrong credentials (e.g., bad password), or your
    browser doesn't understand how to supply the credentials required.

Could it be the problem ?

Questions

Should /v2/ (ping) return 401 without credentials ?
Do you have a working sample of v2 with httpd or nginx using httpd/nginx authentication ?

I have the some problem.
Without basic auth on nginx all works well. docker pull/push work.
My configuration: https://gist.github.com/Lispython/36306d55bad098c2969e
Also I have chef cookbook: https://github.com/wh13/docker_registry_2
You can make VM this problem config via kitchen converge & kitchen login. docker-compose.yml you can find in /var/app/
This is my logs and tests.
https://gist.github.com/Lispython/ff10680b9e20ddd48fba

@hgomez

Should /v2/ (ping) return 401 when no credentials are provided ?

The /v2/ endpoint is a general auth check. It requires credentials to work correctly. There are some subtleties around fallback that need to be taken care of. @dmcgowan

Do you have a working sample of v2 with httpd or nginx using httpd/nginx authentication ?

Not that I know of. Let's have @moxiegirl or @dmp42 take a look.
@Lispython Do you have daemon logs, as well? It looks like the initial login is failing in the daemon and is falling back to v1.

Sorry, I never tried httpd so I've got nothing.

Did you have a working nginx configuration instead, it should be easy to adapt

@stevvooe I think that docker login doesn't send Basic auth header on login.
https://gist.github.com/Lispython/ff10680b9e20ddd48fba#file-logs-output-L29 this is response from nginx, not from registry.
Please, show working nginx configuration if my is wrong.

Same behaviour here, Apache HTTPd send 401, not registry.
Did there is plans to implement authentication on registry side, so we could use nginx/httpd only as relay ?

@hgomez There is now a PR to implement Basic auth directly in the registry using .htpasswd files https://github.com/docker/distribution/pull/409

I have Basic Auth working in nginx, the problem was that the docker client expects the 'Docker-Distribution-Api-Version: registry/2.0' header from the /v2/ endpoint. Setting that header in the auth response solved the problem. See example ngix config.
    location /v2/ {
        # Do not allow connections from docker 1.5 and earlier
        # docker pre-1.6.0 did not properly set the user agent on ping, catch "Go *" user agents
        if ($http_user_agent ~ "^(docker\/1\.(3|4|5(?!\.[0-9]-dev))|Go ).*$" ) {
              return 404;
        }

        proxy_pass http://registry;

        proxy_read_timeout 900;

        more_set_headers 'Docker-Distribution-Api-Version: registry/2.0';
        #more_set_headers 'Content-Type: application/json; charset=utf-8';

        auth_basic "Restricted";
        auth_basic_user_file /etc/nginx/custom/dockerpasswd;
    }

@kuhnroyal Thank you @kuhnroyal!
@moxiegirl Could you add this to the nginx setup examples with auth?

Just updated my Apache HTTPd configuration and it works too
Header set "Docker-Distribution-Api-Version" "registry/2.0"

all operations performed using Registry 2.0
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:26 +0200] "GET /v2/ HTTP/1.1" 200 2 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - - [22/Apr/2015:22:05:31 +0200] "GET /v2/ HTTP/1.1" 401 1233 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - - [22/Apr/2015:22:05:32 +0200] "GET /v2/ HTTP/1.1" 401 1233 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - - [22/Apr/2015:22:05:32 +0200] "GET /v2/ HTTP/1.1" 401 1233 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:33 +0200] "HEAD /v2/teama1/teama-java5/blobs/sha256:a3ed95caeb02ffe68cdd9fd84406680ae93d633cb16422d00e8a7c22955b46d4 HTTP/1.1" 200 - "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:33 +0200] "HEAD /v2/teama1/teama-java5/blobs/sha256:a3ed95caeb02ffe68cdd9fd84406680ae93d633cb16422d00e8a7c22955b46d4 HTTP/1.1" 200 - "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:34 +0200] "HEAD /v2/teama1/teama-java5/blobs/sha256:b6c77cf05f323e226c196cf9c0c0011ca781d84d5005b25df8e092d5dc780ca4 HTTP/1.1" 200 - "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:35 +0200] "HEAD /v2/teama1/teama-java5/blobs/sha256:cdf9bbc08a6bf9b5641961839ee9fd8beeaaacee4de55e2e360a1db364d0ba91 HTTP/1.1" 200 - "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:35 +0200] "HEAD /v2/teama1/teama-java5/blobs/sha256:a3ed95caeb02ffe68cdd9fd84406680ae93d633cb16422d00e8a7c22955b46d4 HTTP/1.1" 200 - "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:36 +0200] "HEAD /v2/teama1/teama-java5/blobs/sha256:a3ed95caeb02ffe68cdd9fd84406680ae93d633cb16422d00e8a7c22955b46d4 HTTP/1.1" 200 - "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:36 +0200] "HEAD /v2/teama1/teama-java5/blobs/sha256:a3ed95caeb02ffe68cdd9fd84406680ae93d633cb16422d00e8a7c22955b46d4 HTTP/1.1" 200 - "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:37 +0200] "HEAD /v2/teama1/teama-java5/blobs/sha256:153220a76b6c452aab6ce6d6874841e49f5365e2e1a242cbf37f2205db5b410a HTTP/1.1" 200 - "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:37 +0200] "HEAD /v2/teama1/teama-java5/blobs/sha256:a3ed95caeb02ffe68cdd9fd84406680ae93d633cb16422d00e8a7c22955b46d4 HTTP/1.1" 200 - "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"
<IP_ADDRESS> - teama-deployer [22/Apr/2015:22:05:38 +0200] "PUT /v2/teama1/teama-java5/manifests/latest HTTP/1.1" 202 - "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.13.0-37-generic os/linux arch/amd64"

Thanks
PS: Seems to be a bug, will it be fixed in 2.0.1
REPS: If you want Apache HTTPd configuration too, just ping me :)

@hgomez Great to hear this worked!
This seems like a client bug, rather than a registry bug. I'll need to track down why we put this in exactly. I had submitted #99 but am uncertain about requiring this in the specification.
Would you be willing to contribute an example to the docs?
We'll close this when we've worked out the missing documentation issues.
cc @moxiegirl

@stevvooe np, that one line took me 4 hours on sunday ;)

Confirm that setting header 'Docker-Distribution-Api-Version: registry/2.0' solved problem on my nginx configuration. Thanks!

@kuhnroyal @Lispython Sorry about that. I don't know about getting back your Sunday but we'll get this documented.

@stevvooe It is on my todo list. If you need it bumped up in priority let me know. I have a few things ahead of ya.

Just send a PR for HTTPd config sample
https://github.com/docker/distribution/pull/423

I have the same problem. Without basic auth everything works just fine, but it stops working with basic auth enabled on the nginx side.
GET /v2/ HTTP/1.1" 401

Update your nginx configuration with
more_set_headers 'Docker-Distribution-Api-Version: registry/2.0';

It should fix problem

Should we close this issue now we have a workaround ?

Unfortunately your workaround didn't work. Let me explain my registry deployment.

Registry is started (docker run -p 5000:5000 registry:2.0)
Nginx server is started as proxy server

Here is my nginx config
server {
    listen 443 ssl;
    server_name registry.my.server.com;

    ssl on;
    ssl_certificate /etc/nginx/ssl/my.server_bundled.crt;
    ssl_certificate_key /etc/nginx/ssl/my.server.key;

    add_header Docker-Distribution-Api-Version registry/2.0;

    proxy_set_header Host $host;
    proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
    proxy_set_header X-Real-IP $remote_addr;
    proxy_set_header X-Forwarded-Proto $scheme;
    proxy_set_header X-Original-URI $request_uri;
    proxy_set_header Docker-Distribution-Api-Version registry/2.0;

    client_max_body_size 0; # disable any limits to avoid HTTP 413 for large image uploads

    chunked_transfer_encoding on;

    location / {
      auth_basic "Restricted";
      auth_basic_user_file /etc/nginx/htpasswd/users.htpasswd;
      proxy_pass http://docker-registry;
    }
}

I can access the path /v2/ with my username and password
curl  --user admin:admin https://registry.my.server.com/v2/
{}%

But I can't login
docker login -u admin -p admin -e<EMAIL_ADDRESS>registry.myserver.com

Response
FATA[0001] Error response from daemon: (Code: 404; Headers: map[Date:[Thu, 23 Apr 2015 14:15:32 GMT] Content-Type:[text/plain; charset=utf-8] Content-Length:[19] Connection:[keep-alive] Docker-Distribution-Api-Version:[registry/2.0] Server:[nginx/1.7.12]])

I'm not a nginx specialist but tweak was to use
more_set_headers 'Docker-Distribution-Api-Version: registry/2.0';

Your configuration contains :
add_header Docker-Distribution-Api-Version registry/2.0;
...
proxy_set_header Docker-Distribution-Api-Version registry/2.0;

@hgomez standard nginx distribution doesn't include this module. That is why I've used
add_header Docker-Distribution-Api-Version registry/2.0;
...
proxy_set_header Docker-Distribution-Api-Version registry/2.0;

which sets request and response headers

ok :)
proxy_set_header Docker-Distribution-Api-Version registry/2.0; is not enough ?

@hgomez As far as I understand it, proxy_set_header only will set the header if there is an actual response from the proxy target, in this case there is not, because it is being blocked by the auth request.

Oups, right out and in required

exactly. That means also, that registry server doesn't even get any requests because all they are blocked by nginx. The question is why does the docker client makes the requests without authorisation?

Closing issue

OMG I lost probably a day to this over the last several days.  Someone on IRC (stevvooee) pointed me at this issue, and the more_set_headers bit did it.  Thanks all!  I normally wait for the x.0.1 versions before I dive into a major release for just these reasons.

Still having this issue despite the response headers being sent back to the client. I'm using a client with the same sha as @andrexus
*** - - [04/May/2015:11:50:03 -0500] "GET /v2/ HTTP/1.1" 401 479 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.19.5-100.fc20.x86_64 os/linux arch/amd64" 694
*** - - [04/May/2015:11:50:03 -0500] "GET /v1/_ping HTTP/1.1" 401 479 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.19.5-100.fc20.x86_64 os/linux arch/amd64" 782
*** - - [04/May/2015:11:50:03 -0500] "POST /v1/users/ HTTP/1.1" 401 479 "-" "Go 1.1 package http" 713
*** - pkill [04/May/2015:11:50:03 -0500] "GET /v1/users/ HTTP/1.1" 404 19 "-" "docker/1.6.0 go/go1.4.2 git-commit/4749651 kernel/3.19.5-100.fc20.x86_64 os/linux arch/amd64" 93176

With or without the Header set directive in httpd, the response headers reflected in the error message indicate that Docker-Distribution-Api-Version is being properly set.
FATA[0006] Error response from daemon: (Code: 404; Headers: map[Date:[Mon, 04 May 2015 16:56:40 GMT] Content-Type:[text/plain; charset=utf-8] Docker-Distribution-Api-Version:[registry/2.0] Content-Length:[19] X-Content-Type-Options:[nosniff]])

I'm using httpd version 2.2.15.

@pkill Could you please ask your question on IRC, Google Groups or another issue? If you've set the header and are still having fallback problems, it might not be the same issue.

@stevvooe it turns out it was the same root cause: the Docker-Distribution-Api-Version isn't being set in the 401 response. I think this issue was closed prematurely and should probably be left open for until the next minor version of the registry is released. Others might run into working out the minor implementation details of this fix because of the permutations of different proxy server suites, their various configuration and versions. While the comments thus far seems to solve the problem for Nginx users.
As an httpd 2.2.15 user, I had to set two directives to get this to work.
    Header always set Docker-Distribution-Api-Version "registry/2.0"
    Header onsuccess set Docker-Distribution-Api-Version "registry/2.0"

According to http://httpd.apache.org/docs/2.2/mod/mod_headers.html#Header

The optional condition argument determines which internal table of responses headers this directive will operate against. Despite the name, the default value of onsuccess...

and later

The default value of onsuccess may need to be changed to always under the circumstances similar to those listed below. Note also that repeating this directive with both conditions makes sense in some scenarios because always is not a superset of onsuccess with respect to existing headers:

You're modifying or removing a header generated by some piece of the server but that header is not being found by the default onsuccess condition.

This is because httpd keeps an internal table of response headers it could send depending on a certain condition. By default, headers set with the Header directive will be sent when the onsuccess condition is met. However, I'm guessing that a 401 response does not count as a success condition. Setting the always condition places this header in a table of responses that will be sent on a 401 response.
I will submit a pull request.
Note to users of httpd 2.4 and later, you can refine the expression to only send the response on a 401 HTTP_STATUS condition but I have not tested that.

I used HTTPd 2.4

May be HTTPd 2.2 behave differently

@hgomez yes, that's what I'm thinking and illustrates why I think this issue should be left open until this header is no longer required.
I'm going to open a new MR to increase the compatibility of that directive. Do you have time to test my change on Apache 2.4? I'll link the MR in a bit.

@pkill I'm sorry I assumed this was a different issue. As you stated:

the response headers reflected in the error message indicate that Docker-Distribution-Api-Version is being properly set.

This is really a documentation issue but it was closed by @hgomez, who reported it.
I'll be awaiting your PR.

@stevvooe No worries. The Docker client was a bit misleading when it reflected those headers in the error message because it was reflecting response headers from the /v1/ endpoint, not on the original /v2/ endpoint 401 response but as you said, it's a documentation issue.
@hgomez if you could be so kind as to test https://github.com/docker/distribution/pull/490 and comment on the issue, I would appreciate your time.

I don't have apache 2.4.x anymore (moved to a new company) so cannot test it :-(

I have 2.4.10 on my workstation. I could probably proxy to the existing registry temporarily.

How odd. I can't get @hgomez's MR to work on a local instance of httpd v2.4.10.
I didn't actually need to proxy anywhere since the initial basic auth response headers are all we care about. I also setup a host override for <IP_ADDRESS> registry.example.com.
Using
Header set "Docker-Distribution-Api-Version" "registry/2.0"
pkill@balance in {/etc/httpd/conf} at 2015-05-04 16:30:25
$> curl -v -s "http://registry.example.com/v2/"  1> /dev/null
* Adding handle: conn: 0x96e8d0
* Adding handle: send: 0
* Adding handle: recv: 0
* Curl_addHandleToPipeline: length: 1
* - Conn 0 (0x96e8d0) send_pipe: 1, recv_pipe: 0
* About to connect() to registry.example.com port 80 (#0)
*   Trying <IP_ADDRESS>...
* Connected to registry.example.com (<IP_ADDRESS>) port 80 (#0)
> GET /v2/ HTTP/1.1
> User-Agent: curl/7.32.0
> Host: registry.example.com
> Accept: */*
>
< HTTP/1.1 401 Unauthorized
< Date: Mon, 04 May 2015 21:30:33 GMT
* Server Apache/2.4.10 (Fedora) OpenSSL/1.0.1e-fips is not blacklisted
< Server: Apache/2.4.10 (Fedora) OpenSSL/1.0.1e-fips
< WWW-Authenticate: Basic realm="Registry Authentication"
< Content-Length: 381
< Content-Type: text/html; charset=iso-8859-1
<
{ [data not shown]
* Connection #0 to host registry.example.com left intact

Using
Header always set "Docker-Distribution-Api-Version" "registry/2.0"
Header onsuccess set "Docker-Distribution-Api-Version" "registry/2.0"

pkill@balance in {/etc/httpd/conf} at 2015-05-04 16:32:08
$> sudo systemctl restart httpd

pkill@balance in {/etc/httpd/conf} at 2015-05-04 16:32:15
$> curl -v -s "http://registry.example.com/v2/"  1> /dev/null
* Adding handle: conn: 0x13e58d0
* Adding handle: send: 0
* Adding handle: recv: 0
* Curl_addHandleToPipeline: length: 1
* - Conn 0 (0x13e58d0) send_pipe: 1, recv_pipe: 0
* About to connect() to registry.example.com port 80 (#0)
*   Trying <IP_ADDRESS>...
* Connected to registry.example.com (<IP_ADDRESS>) port 80 (#0)
> GET /v2/ HTTP/1.1
> User-Agent: curl/7.32.0
> Host: registry.example.com
> Accept: */*
>
< HTTP/1.1 401 Unauthorized
< Date: Mon, 04 May 2015 21:32:16 GMT
* Server Apache/2.4.10 (Fedora) OpenSSL/1.0.1e-fips is not blacklisted
< Server: Apache/2.4.10 (Fedora) OpenSSL/1.0.1e-fips
< Docker-Distribution-Api-Version: registry/2.0
< WWW-Authenticate: Basic realm="Registry Authentication"
< Content-Length: 381
< Content-Type: text/html; charset=iso-8859-1
<
{ [data not shown]
* Connection #0 to host registry.example.com left intact

¯\_(ツ)_/¯

This info should be right there in the Docker Registry 2.0 documentation. Quite a ride to get this working with basic authentication and SSL. Maybe even pre-package a container that houses these components together. I know it's against the Docker ethos, but you kinda always need a proxy besides some locat test as far as I can see.

@tnolet Where exactly in the documentation should such a header requirement be placed? The registry sets it appropriately, but the proxies seem to handle header configuration differently in each case. To put it another way, where would you have found it, had it been where it was supposed to be?
@dave-tucker @moxiegirl

@stevvooe just a little extra mention in the install docs at https://docs.docker.com/registry/deploying/ would suffice. Maybe put it in one of those alert/info boxes. Maybe add the part for basic authentication to the same page would really help.

Docker 1.6.1 and Distribution 2.0.1 have been released and I'm wondering if nginx/httpd tricks are still required ?
If so, which version of Docker and Distribution should fix header issues ?
Thanks

@hgomez don't expect Docker 1.6.x to be able to connect to a v2 registry without the  "Docker-Distribution-Api-Version".  We are trying to remove this requirement for 1.7 but we need to be able to get the new registry discovery process into the engine (as defined here #303). We have no solution which we can define on the distribution server which would mitigate the need for this header.

This issue seems to be reintroduced in docker 1.8.2 onwards. With docker registry V2 fronted by nginx, it works fine with clients <= 1.7 but with 1.8.2 it gives the error. Also tried the same with 1.9 rc but issue persists.

@gauravbhandare This issue has been long closed. Generally, commenting on closed issues is considered bad etiquette unless you're 100% sure that the issue is the same. In this case, you are describing an issue that has similar behavior but the root cause is likely very different.
Please open a new issue or consult mailing list to get help with your issue. If you believe this issue is related, you should reference it in your new issue and we can compare the symptoms.


V2 telemetry emits durations in millis instead of seconds

V2 telemetry in 1.3.0.rc1 emits latency numbers in milliseconds.
Mixer telemetry emits them in secods, which is what the dashboard assumes.
Update the dashboard so that it works both with the istio_requests_total and envoy_requests_total and scales envoy_requests_total by /1000.0

consider updating the metric name to istio_request_duration_milis

is it feasible to have it output the same metrics? it would be nice to be able to smoothly transition to v2 without having to change the dashboards. And for kiali, custom dashboard, etc

I agree with @howardjohn .... I suggest you add a new metric with the new time unit rather than re-using the same metric name but changing to ms instead of seconds - at least as a temporary bridge. Otherwise, there will be breakage in ANY client using telemetry (it isn't just dashboards and kiali - its anyone that is looking at telemetry - I bet there are people out there with scripts that query prometheus and obtain metrics for their own reporting and statistical analysis).
@jshaughn @jotak this has impact on Kiali - we need to keep on top of this and change the name of the metric Kiali looks for and change Kiali's algorithms since the time unit is changing.

Before we call this “ready” it will be identical to the current metric.
This issue is so that the preview can be used.
My 1st comment is what you are asking for, add a new _milisecond metric.

@jmazzitelli maybe in kiali we'll want to keep backward compatibility? (e.g. with some setting flag to disable telemetry v2). Or once 1.3 is supported we drop support for all previous versions?

The Grafana dashboard approach to this issue should be to use an updated prom query for places that require latency info. The query will use the or operator with the _millis value(s) converted to seconds. We should have a PR soon that demonstrates this approach.
I imagine that something similar can be done in Kiali.


🐛 [firebase_notification] "Notifications are not allowed for this application" - how fix it?

I have flutter app. It is working for iOS. now I try to run it on macos - I have problems:

I added GoogleService-Info.plist to my macos/runner

my app uses firebase_messaging(for push notification)
final settings = await FirebaseMessaging.instance.requestPermission(
announcement: true,
carPlay: true,
sound: true,
criticalAlert: true,);

this code returns an error:
[firebase_messaging/unknown] Notifications are not allowed for this application
Found this discussion
but I don't see my app there. i installed it via TestFlight - but that didn't help either, the app is not listed.
how do i solve this problem? any advice - I will be grateful
also i don't know how to configure my app properly. I found this document but there are no details (what permissions are needed and other details)

I found this document but there are no details (what permissions are needed and other details)

The document you referred is archived. You should refer to https://firebase.google.com/docs/cloud-messaging/flutter/client going forward. Although it also doesn't have specific steps related to macOS desktop in general.
I think you should refer to https://github.com/invertase/flutterfire_desktop for desktop related plugin usage and see if it helps in your case.
Regarding the issue you reported, I don't think there's any action required from flutterfire client side on this. You can refer the discussion you linked.
Closing for now. If you disagree, write in comments and I'll reopen it.


Adding support for 128x96 (1.27") Adafruit ssd1351

Raspberry Pi Zero W
Linux Kernel version
Linux raspberrypi 4.19.118+ #1311 Mon Apr 27 14:16:15 BST 2020 armv6l GNU/Linux
Expected behaviour
Display should function with the ssd1351 driver
Actual behaviour
Nothing.
Details
I am trying to get luma.oled working for this device
It seems like there is good support for the 128x128 variant however even after adding the 128x96 resolution here the screen does not respond.
I have tested this screen with the Adafruit reference library so I am confident everything is wired correctly and am able to talk to the device. The device specific parameters can be found here
I also noticed some minor differences in the init process. I attempted to replicate the exact init sequence in luma.oled but with no luck.
This all is rather new to me. I am happy to author support for this screen, but I am looking for some pointers on where I may be going wrong and how to further diagnose this.

I spoke too soon.
issue was the GPIO pin placement and the defaults this library uses (learned from this thread https://github.com/rm-hull/luma.oled/issues/205)
Ill post a PR later to add the new resolution.

I can post a PR with the new resolution support.

Pleaso do, we can discuss approach there.

Done. see: https://github.com/rm-hull/luma.oled/pull/291


Validate linting on CI on parallel

Coverage decreased (-0.6%) to 89.938% when pulling c84aba5b9c8b7003833b1e23e55f00e8385ea808 on validate-formatting-parallel into 8174f9dd42a0bdc57870bb1e33a56f2cffd4ab0a on master.


Existentially abstract unbound subtype parameters

That happens when the subtype of a sealed trait has more type parameters than its parent. When those extra type parameters are covariant they are replaced by their upper bounds, otherwise they are existentially quantified.
Fixes #51

Thank you for this! I hadn't expected a solution so fast (with so little work required from myself)! :)
I'll try to review and merge later today, but if all the old tests still pass, it's looking good.


Make e2e tests run with Podman

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages (more info).
[ ] Added or updated documentation.
[ ] Tests for new functionality and regression tests for bug fixes.
[ ] Screenshots or videos attached (for UI changes).
[ ] All your commits have a Signed-off-by line in the message (more info).

Included in https://github.com/nordeck/element-web-modules/pull/183


Continued Conversation

Recently, I got a Pixel 2XL, rooted it, and wanted the new Google Assistant 2.0 UI and  continued conversation. Found the XDA thread for it, and then found P4XLprops, flashed if, and I have 2 Errors...
1- Whenever I open Camera, It starts lagging like hell. Then it crashes after what, 5 seconds?
2- Google Assistant CC (Continued Conversation) Switch isn't available, but the UI shows immediately.
Camera Solved, 2 still needs answering

1.It's a known Issue  with stock google camera there are only Workarounds which maybe adopted .
2.Continued Conversation is available for English(US) .
Issue: The module overrides existing prop values to spoof device to be Pixel4XL. Google camera uses these prop values but since the camera hardware is different in pixel2XL and 4XL it causes failure communicating with the hardware and thereby leading it  to crash.
Workarounds:

Use cstark's Google Camera Port - https://www.celsoazevedo.com/files/android/google-camera/dev-cstark27/

If you use Edxposed Fraework on you device, you may make use of this module to fix stock Google Camera crash - https://github.com/ikws4/WeiJu

Here's how-

Hello, The Module fixed the error, as I was already using Camera PX port as it is superior.
About the Continued Conversation...

There is no switch, as I said.

Nice(though i still wonder what you mean  by saying module fixed it), now about the continued conversation, after changing the language to US English also clear the app data for toggle to appear.
You may check the video below (The toggle is not visible when my language is English (India) and on changing the language to English(US) it doesn't appear until google app's data is cleared).

Hello, Even though I do exactly that, no phone appears, I'm also wondering why it doesn't say downloading additional resources and just Setting up... The UI is working just fine but even with my assistant on English (United States) Still no phone, Does it depend on the country?

It doesn't depend on the country although you should too check what you have set as you system language in settings.

I have seen multiple comments online it is only available in the US, Now I've turned the nav bar on and there is still no setting for GA 2.0 CC

I have seen multiple comments online it is only available in the US, Now I've turned the nav bar on and there is still no setting for GA 2.0 CC

It works for me in India , I hope this puts rest to all those comments And what do you mean by putting navbar on?

As of Nav bar, I mean the bar at the bottom, I AM using a root adblocker so I might disable that and try again

As of Nav bar, I mean the bar at the bottom, I AM using a root adblocker so I might disable that and try again

Both those things i.e the choice of  system navigation or adblocker won't have anything to do with it rather retry clearing data for the google app retrigger the assitant from home screen and visit the assistant settings from google app to check if it appears or not.

This is starting to get annoying,

It still isn't showing up, Do they not let CC where I am?

No it doesn't depend on region as long as you choose to communicate with assistant in US english.
Try enrolling in google app beta testing but honestly that shouldn't be the case because continued conversation is there too in stable versions of google app still in case you want to enroll you can enroll by clicking join here - https://play.google.com/apps/testing/com.google.android.googlequicksearchbox
and then updating google app from playstore.

I have tried everything, and still. No GA 2.0.

I still don't know, what is going on.

I have tried everything, and still. No GA 2.0.

I still don't know, what is going on.

Sorry, I do get the 2.0 UI, but for me the entire 2.0 is the CC because of its amazing features, while I only get the UI.
This is very annoying, as CC would help do a lot of things, this isn't your problem anymore, I'm going to ask Google Support for help.

Also PS, I am on Pixel 2 XL, not any custom roms

Also PS, I am on Pixel 2 XL, not any custom roms

Are you getting the updated google assistant post this module's installation ?

I am. What I am NOT getting is the continued Conversation, but I get the UI and everything else

i try to use latest cstark , it crashed and i also tried your framework but it still getting crashed, suggest me something and i for cc i have option but it's not working for me so i install march prop then cc start working. help me for cstark gcam

i try to use latest cstark , it crashed and i also tried your framework but it still getting crashed, suggest me something and i for cc i have option but it's not working for me so i install march prop then cc start working. help me for cstark gcam

cstark port is for pixel devices you need to find a port for your device that will work for you

i have redmi 5 plus, it's latest version is not working,:(


[DOCS] introduction.md: add consistent information for 3 strategies (MACD, RSI, StochRSI).

The current documentation found here (https://gekko.wizb.it/docs/strategies/introduction.html) is outdated for 3 strategies (MACD, RSI, StochRSI). A better text can be found here (https://app.gekkoplus.com/strategies). I propose this PR to update this information.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
docs update

What is the current behavior? (You can also link to an open issue here)
The informations found here (https://gekko.wizb.it/docs/strategies/introduction.html) for the MACD, RSI and StochRSI are outdated

What is the new behavior (if this is a feature change)?
Updated the information (https://gekko.wizb.it/docs/strategies/introduction.html) to match the information found here (https://app.gekkoplus.com/strategies)

Other information:

thanks!


Timeout exceeding adding symbol

After successfully logging in, ATAT attempts to add the first alert and fails with the following error. This is on MacOS 10.15.7
$ npx cross-env node -r source-map-support/register dist/add-alerts.js
 Current Log level: 3 (you can set env variable ATAT_LOG_LEVEL=5 for verbose logging)                        14:59:28
 Base delay: 1000 (you can set env variable BASE_DELAY=1000 to increase/decrease speed)                      14:59:28
 Using config file:  config.yml                                                                              14:59:28
 Press Ctrl-C to stop this script                                                                            14:59:28
 Adding symbol: BINANCEUS:BTCUSD  ( BTC priced in USD )                                                      14:59:55

 ERROR  waiting for XPath //div[@id="header-toolbar-symbol-search"] failed: timeout 20000ms exceeded          15:00:17

  at new WaitTask (node_modules/puppeteer/src/common/DOMWorld.ts:776:28)
  at DOMWorld.waitForXPath (node_modules/puppeteer/src/common/DOMWorld.ts:671:22)
  at Frame.waitForXPath (node_modules/puppeteer/src/common/FrameManager.ts:1189:47)
  at Page.waitForXPath (node_modules/puppeteer/src/common/Page.ts:1982:29)
  at src/service/tv-page-actions.ts:14:20
  at Generator.next (<anonymous>)
  at dist/service/tv-page-actions.js:7:71
  at new Promise (<anonymous>)
  at __awaiter (dist/service/tv-page-actions.js:3:12)
  at fetchFirstXPath (src/service/tv-page-actions.ts:11:82)

$ npm list
<EMAIL_ADDRESS>/Users/tedwardd/git-work/add-tradingview-alerts-tool
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

hi there - does this happen consistently for you? or only sometimes? do you have a fast/slow internet connection?

This is consistently repeatable. I have a fast, 1Gbps, fiber internet connection.
I've attached a screen recording of what happens. Let me know if there's any other information I can provide to assist.

Nevermind. I solved the issue. I did not realize I had to change the value for chartUrl in config.yml. I changed that and now it works. Might want to make this more apparent in the docs. Cheers.

Hi! Thanks for pointing out the bad UX - I've added an error that will show up if people forget to change the chartUrl!
Cheers,
Michael


Unable to give auto or 100% in the template container

I am not able to enter 100% or auto width in the following picture.

Hello @madanpragada4,
Thanks for submitting a GitHub issue! We are very sorry that you are running into this problem. In order to better serve you, as this does not present itself as a library specific issue, we would like to ask that you reach out to our support team at https://support.sendgrid.com.
Thank you!
SendGrid DX Team


[Merged by Bors] - fix: comment more cache

The previous fix was an improvement, but was not aggressive enough in commenting the failing step.
This step used to use cache to download the cache for Mathlib.Init.  If this was successful, then it would continue trying to download all of the available cache.  Otherwise, it would start with building.  The underlying reason is that if even the cache for Mathlib.Init is not there, there is not much of an available cache and there is no need to spend a couple of minutes realizing as much.
However, changes to lake build --no-buld (possibly) have made this step always fail to download any cache, so we revert to the old behaviour of trying to get the cache no matter what.

Thanks!
bors r+

cc: @tydeu so he is aware of this issue

@digama0 This was discussed on Zulip and a fix was implemented in https://github.com/leanprover/lean4/pull/5641.


Generate symbols file for test plan

build: generate symbols file
ztest: generate symbols for tests
cmake: binutils: find binutils nm for host compiler

@nashif I assume, with this, we will no longer need an abomination like the following, is that correct?
https://github.com/zephyrproject-rtos/zephyr/blob/735ddeb4a8e26c6ce07d6aa5a0b77fffc559b2a8/tests/lib/newlib/thread_safety/testcase.yaml#L7-L14

@nashif I assume, with this, we will no longer need an abomination like the following, is that correct?

yes


Trino throws error: init() got an unexpected keyword argument 'auth'

I just hit this error, probably because something was recently changed/broken.
/opt/app-root/lib64/python3.8/site-packages/pyhive/trino.py in cursor(self)
     53     def cursor(self):
     54         """Return a new :py:class:`Cursor` object using the connection."""
---> 55         return Cursor(*self._args, **self._kwargs)
     56
     57

StatementError: (builtins.TypeError) __init__() got an unexpected keyword argument 'auth'
[SQL: show schemas in osc_datacommons_dev]

@HumairAK @redmikhail @caldeirav

This might be my fault...due to SQLAlchemy versioning.  PUDL needs 1.4 (it chokes with 1.3) and I think I remember that our instance needs 1.3.  So we have a major skew problem we need to fix.


Android Basics: classes-and-inheritance

Classes and inheritance in Kotlin
5. Modify classes in the hierarchy
Implement floorArea() for RoundHut
(...)
The floor area of a circular dwelling is 2 * PI * radius.
(...)
area is wrong: instead 2 * PI * radius, is PI * radius * radius

@Daravolo
Thank you for reporting.
This issue be fixed now.


Review needed

Need to adjust background image ,need help
Any other suggestions are welcome

Fixed Linters error & Updated README.md file


Deleting files

How can I delete files on PMS directly from Serenity? Long press on a video does not give me the option to delete a file.

Closing.  As the vendors web server should be used for this.


Adding new penalty terms

Hi,
I came across this package and am really impressed! I was wondering how difficult it would be to add new penalty terms.
I know that pyGAM has constraints for the curvature being larger (convex) or smaller (concave) than zero. Would it be possible to constrain the curvature to be larger than a certain pre-specified value? https://github.com/dswah/pyGAM/blob/b57b4cf8783a90976031e1857e748ca3e6ec650b/pygam/penalties.py#L170
I have the same question for monotonicity. https://github.com/dswah/pyGAM/blob/b57b4cf8783a90976031e1857e748ca3e6ec650b/pygam/penalties.py#L99
In general, is there an intuitive answer as to why the penalty is computes in the same way, but with a different derivative for both cases? Would it be possible specify any rejection criteria for the mask and still have the pyGAM code work?
D = sparse_diff(sp.sparse.identity(n).tocsc(), n=derivative) * mask
penalty = D.dot(D.T).tocsc()

wow thats a cool idea.
i think youre right!
i need to think about it a bit :thinking:
what do you think?

i think it would be incredibly useful, but I don't know the math well enough to know how to do this.

let's do a test!
can you design a simple dataset for a case where:

perfect solution requires a model with curvature >1 on a certain X interval
but we add a constraint were all curvatures are forced to be less than 1
using the method you suggested

if the fitted model fits poorly on the high curvature interval then we know the method works.
we could test analogously for monotonicity/slope.


B disp textview readonly wrap fix

Description
Please include a terse summary of the change or which issue is fixed.
If an Issue was fixed, write "Fixes #issuenumber" as this will help Github automatically close that issue once the PR is merged.
Temrminal.Gui Pull Request checklist:

[ ] My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
[ ] Mhy code follows the Terminal.Gui library design guidelines
[ ] I ran dotnet test before commit
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the API documentation (using /// style comments)
[ ] My changes generate no new warnings
[ ] I have checked my code and corrected any poor grammar or misspellings
[ ] My pull request generate no conflicts with develop branch
[ ] I tested my code on at least Windows and Linux
[ ] I conducted basic QA to assure all features are working

I'm still having issues with the read only copy on TextView. I'll try to fix it asap. Can I reopen this pr or is better create another one? Maybe the fix is already fine on others TextView fixes on another pr's, but I'm not certain sure.

Create a new PR please.


Document possible Skip actions.

@yunhaoling easing your confusion. Thanks for bringing this up.

There is no build associated with this changeset, going to override and merge.

/check-enforcer override


don't allow failures anymore

allowing something to break is bad

I don't sign anything.

This was an intentional change in our Travis setup. Closing.

It was intentional as it was for node-sass on io.js.Those issues are fixed now. Please discuss something before closing the ticket :)

It still would end up being closed since you are not willing to sign the CLA. This is required for anyone contributing to a Google Open Source project. Without a signed CLA, no contribution can be accepted.


Signature Base does not include port when provided / non-default

Testing on a local oauth provider instance, if I run on a non-standard port and provide it in the url, TDOAuth removes the port when generating the base. This is not correct looking at the OAuth 1.0 protocol: http://tools.ietf.org/html/rfc5849#section-<IP_ADDRESS>
ie
GET on http://localhost:8080/myService
generates a signature base of: GET&http%3A%2F%2Flocalhost%2FmyService ...
when it should be: GET&http%3A%2F%2Flocalhost%3A8080%2FmyService ...
As a result, the signature is invalid to the provider. Trivial issue, but I found it annoying.

Sorry about that. Patches welcome. Though I think the patches will need to be against my fork, seeing as I do not have access to this repo anymore.
Though I will ask for access.

Fixes in my fork. Feel free to cherry pick from it:
https://github.com/puug/TDOAuth

I'll merge this, this week. Thanks.


Can't be compiled on CTRUlib v1.0.0

Before anyone else opens this: this can't be compiled on CTRUlib 1.0.0 yet, the reason being that ctrcommon is not updated yet. I'm considering using Citrus or trying to replicate all functionality with CTRUlib only - whatever is easier. Ideas are welcome!

I'd go with ctrulib but whatever is easier on you :P

Well, that's the point... it is much easier to rewrite with Citrus, and it can be assumed that @SteveIce10 will keep updating this library. In the long run, relying on CTRUlib only would be easier, of course.
Anyway, I've almost decided I will go with Citrus by this point.

Alright, just checked with the latest ctrcommon + ctrulib 1.0.0. Compiles fine. Closing this for now.

:+1:


Change nickname to username, add door name field

The nickname field is actually a username. Change the nickname field to be called username, and add a new nickname/door name field to use for the door, etc.

We could make it similar to the two emails where the first (say username) will autofill the second (door name/nickname) until they explicitly change it.

Good idea, will do :)


fix: add region argument to glue extractor boto3 client instantiation

Summary of Changes
glue extractor does not get region_name argument. It might be needed in different scenarios. For example, while using Databricks for extracting data from AWS Glue, it gives NoRegionError: You must specify a region.
You might be questioning why cant we use aws configuration and set region with .aws/config $HOME directory. Databricks is using instance profile for associating cluster with AWS IAM role. It is not a good practice to install aws cli to databricks cluster and configure aws credentials. So, typically, there is no .aws/config file stored in databricks cluster, and it is not needed.
In a nutshell, it might be better to have an option to provide region_name parameter while creating glue extractor.
Tests

Documentation

CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

any update on this?

closed in favor of https://github.com/amundsen-io/amundsen/pull/1687


CB-6660 Added NIFI_REGISTRY_REST as an exposed service

Updated displayName for the NIFI and NIFI_REGISTRY exposed services for proper stylization
Updated knoxService value for the NIFI_REGISTRY exposed service to be consistent with other exposed services
Updated apiIncluded to be false for the NIFI_REGISTRY exposed service due to the addition of the NIFI_REGISTRY_REST (NiFi Registry API) exposed service
Closes #CB-6660

I guess this should go to rc-2.20 right ?

If possible, I think it would be best to put this in rc-2.20.

@jtstorck can you please send the pull request to 2.20 branch ?


Q: Messing with sensors make them disappear from Home

Not sure how to diagnose.
If I move a sensor to a different room or use them in automation, they disappear from my Home app. Everything else stays, and it even stays in the Homebridge Config UI, yet they don't appear in my Home app again until I reboot the server.
Do you have any suggestions for how I can go about resolving this?

It looks like the cause was the Thermostat was in a different "Home". We deleted the home and it was able to work successfully. Now the issue is we can't add the thermostat via the HomeKit section of the app. It tries to connect but it never adds. Is it taking revenge?

Hello Olivia,
I used to have my doubts about thermostats taking revenge. But who knows with advanced AIs these days. Ours is all the time too or too cold so I'm starting to suspect something...
Anyway, I'd like to know why do you actually need and use this plugin. I was under the impression that it was relevant for about a year before Ecobee released an update which published all the room temperature sensors into the Home app. This effectively replaced all the features this plugin provides so I'm a bit puzzled by it's popularity. Before the updated the thermostat showed only an average value in the Home app so it wasn't possible to see temperature in each room.
Cheers,
Vojta

I couldn't figure out how to add sensors through the ecobee app. The thermostat is there as in the readme, but my choices are limited when it comes to sensors.

Hm... That's strange as far as my understanding of things go. I'm probably playing role of ecobee support here, but my Home app on the iPhone shows all the sensors (6 or so) connected to the thermostat. And I can assign them to their respective rooms and they then thay also report occupancy.
I googled it around and it shows the same way on their website If you look closely at this picture it shows the individual rooms. This should work out of the box.
What thermostat do you have is it Ecobee 3 or later? I also remember I got the functionality in an over the air update about a year after I bough the thermostat. So maybe your updates are disabled in some setting somewhere?

That is interesting. We currently have ecobee3 (I'm not entirely sure they make these anymore? We don't have the lite) with version <IP_ADDRESS>. I can't find any way to update manually nor any way to disable it.

Well. I have the same version and also ecobee3. I think they make ecobee4 now which is ecobee3 + voice control. So that shouldn't be an issue.


Add exclusion_condition to expect_row_values_to_have_data_for_every_n_datepart

This PR addresses #140 by adding an exclusion_condition the the row_values_to_have_data_for_every_n_datepart test.
This condition allows the user to specify a criterion for excluding dates from the test output, thereby allowing the test to pass if there are dates that are known to be missing.
exclusion_condition, which defaults to None, is a SQL clause that is appended to the where closing where statement.
Example(s):
tests:
  - dbt_expectations.expect_row_values_to_have_data_for_every_n_datepart:
      date_col: date_day
      date_part: day
      exclusion_condition: date_day not in ('2019-12-25', '2020-12-25', '2021-12-25')

# or equivalently

tests:
  - dbt_expectations.expect_row_values_to_have_data_for_every_n_datepart:
      date_col: date_day
      date_part: day
      exclusion_condition: not(date_part(month, date_day) = 12 and date_part(day, date_day) = 25)

@clausherther in the absence of #139, I'm not 100% of the procedure in place for merging into main. E.g., does this need to incur a version bump, etc.?
Can you advise? I'll then update the remaining items (e.g., docs, version) as required.

Hi @gofford ! I'll take a more detail look on Monday, but this is great, thanks! Please also update README.md with documentation about the new parameter. I will deal with versioning and the CHANGELOG etc. once it's merged to main.

@gofford we'll also need an integration test for this parameter somewhere here, as a new instance of the test.


Underscore not optional when using require.js

Omitting underscore works fine if I include exoskeleton inline in a page:
<script src="/js/vendor/exoskeleton.min.js"></script>
<script>
console.log(Exoskeleton)
</script>

But if I require exoskeleton, then it tries to load underscore and dies when that fails.
Is underscore required when using require?

Adding underscore.js with just define([], function(){}) tricks require into loading Exoskeleton correctly, but this seems like a hack.


fix: Drop carousel bug on reload

PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #10005

Needs Design check

@exezbcz please review

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Screenshot 📸

[x] My fix has changed something on UI;

Ready for review?


Ordered lists appear as unordered lists

Even when numbered lists used in the documentation, there is no numbering used for the rendered lists. The HTML gets the right tag (ol) but the wrong list-style is set for ol and/or li in CSS.

Hey @elitan Can I fix this please?

Hey @elitan Can I fix this please?

Sure!


.net?

Are you planning a .net implementation?

@markhorsell We are, just due to the bandwidth of our team we haven't had time to write it or a Go server yet. We would be happy to accept PRs from the community of different implementations if you have the time, otherwise we will update this ticket when we have our own ready.


Fix os.symlink 3rd argument

From python doc,
os.symlink(src, dst, target_is_directory=False, *, dir_fd=None)
which means that target_is_directory is a positional argument.
Related to #1175

Oh lol, good timing.

LGTM, thank you for contributing!

Thank you for reviewing this!


Question (not a bug)

I want to know if animejs is the right library that can accomplish something like this ->

1. There are cards on screen.
2. Cards can be dismissed in 2 ways. Swipe up, or Swipe down. Swipe down to dismiss a card by _skipping_. Swipe up to dismiss a card by _promoting_ it.
3. If you do not _swipe up_ all the way, the card snaps back to original position. If you do not _swipe down_ all the way, the card snaps back to original position.
4. If i Swipe up quickly, I will dismiss the card with 1 promotion point. However,
   If i swipe up slowly, it gives the user a chance choose how many points they will dismiss the card with.
5. The swipe up color is yellow. The higher up you swipe, the yellow color fades into a lighter yellow color.

Is animejs a library that help with something like this? Or maybe something else..?

these link might helps
https://github.com/davidkpiano/flipping
https://hammerjs.github.io
a repository example which use hammer js
https://github.com/ericktatsui/Touch-Menu-Like-Android
http://ericktatsui.github.io/Touch-Menu-Like-Android/
close your issue after this

ok i guess animejs cant do it?
will close this then

anime can do this such thing but you have to integrate it with touch libs or native touchmove or mousemove or pointer events
https://tympanus.net/codrops/
look this website most of example are using anime js


Add support for fetching total members from discord server

Feature details?
We currently only have metrics from Reddit, it would be nice to have similar or some kind of data from our discord server as well.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I would love to work on this, please assign it to me.

I would love to work on this, please assign it to me.

You got it!
LMK if any help/suggestions is required.

Hey @Bhupesh-V
a couple of questions.

How exactly is this to be designed currently, the Reddit script is writing to a file in data/, we want to have another script that writes to a new file or wants to modify the same file.
What all data from discord is needed for this, do we have any experience building with discord and what all APIs should I explore?

We can modify the same file, just add a new JSON Key like discordTotalMembers.
No, we haven't researched about the API. We only need the total members count for now. If using the API is not possible, you can look at this hack to fetch data using an invite link https://stackoverflow.com/questions/71566064/how-to-get-total-member-count-of-any-discord-server.

PR is ready for review, LMK if we need to change anything.


Templates in client_definitions

https://ansible-sensu.readthedocs.io/en/latest/dynamic_checks/
The docs show templates in the client_definitions folder. I have
$ tree data/static
data/static
`-- sensu
    |-- client_definitions
    |   |-- sensu_rabbitmq_servers
    |   |   `-- check_users.json.j2
    |   `-- webservers
    |       `-- check_uptime.json.j2

The loop for client_definitions copies the directory over, not individual files. template is needed if you want to us j2 templates. But, template is not supported in this context.
- name: Deploy client definitions
  copy:
    src: "{{ static_data_store }}/sensu/client_definitions/{{ item }}/"
    dest: "{{ sensu_config_path }}/conf.d/{{ item | basename | regex_replace('.j2', '') }}"
    owner: "{{ sensu_user_name }}"
    group: "{{ sensu_group_name }}"
  when:
    - sensu_available_client_definitions is defined
    - sensu_available_checks is not skipped
    - item in sensu_available_client_definitions.stdout_lines
  loop: "{{ group_names|flatten }}"
  notify: restart sensu-client service

I myself have put static checks in the client_definitions for now and used Check tokens in a custom client.json.j2 template
client.json.j2
{
  "client": {
    "name": "{{ sensu_client_name }}",
    "address": "{{ ansible_default_ipv4['address'] }}",
    "subscriptions": {{ sensu_client_subscriptions | to_nice_json(indent=6) }},
    {% if sensu_client_slack_channel %}
    "slack_channel": "{{ slack_channel }}",
    {% endif %}
    "keepalive": {
      "handlers": {{ sensu_client_keepalive_handlers | to_nice_json(indent=8) }},
      "thresholds": {
        "warning": {{ sensu_client_keepalive_threshold_warning }},
        "critical": {{ sensu_client_keepalive_threshold_critical }}
        }
    },
    "global": {
        "domain": "{{ checks_global_domain }}",
        "check_disk_flags": "{{ sensu_disk_mount }}"
    },
    "es_node": {
        "user": "{{ checks_es_node_user }}",
        "password": "{{ checks_es_node_password }}"
    },
    "mysql": {
        "user": "{{ checks_mysql_user }}",
        "password": "{{ checks_mysql_password }}",
        "sock": "{{ checks_mysql_sock }}"
    },
    "redact": ["password", "user"],
    "safe_mode": {{ sensu_client_safe_mode | bool | lower }},
    "environment": "{{ checks_environment }}"
  }
}

client_definitions/metrics_es_cluster.json
{
    "checks": {
        "es_cluster_metrics": {
            "command": "metrics-es-cluster.rb -h :::address::: -u :::es_node.user::: -P :::es_node.password::: -a -s elasticsearch",
            "subscribers": [
                "roundrobin:es"
            ],
            "ttl": 1800,
            "source": "es-cluster",
            "interval": 60,
            "timeout": 30,
            "refresh": 1800,
            "handler": "relay",
            "slack_channel": "#dev-alerts",
            "collection": "elastic_cluster",
            "type": "metric"
        }
    }
}

This might be easier to update the docs then convert to template use.

Hi again @michaelpporter,
This is a good catch,  as far as I can tell, that logic was never used template: https://github.com/sensu/sensu-ansible/commit/488a9a727bac913a9f2744ca297156b4d8c599c3#diff-2be5f2ebe724642f1c79fbf55ef84f47R98
I went back through the issue/PR and reviewed https://github.com/sensu/sensu-ansible/commit/bb18e42a0cf5afd9865702ff7c04fee534a52b74, I think that this is a documentation issue and that it was never intended to actually template out Jinja files. So, that all  said, using check token substitution is a great way to work around this role not support native Jinja templates for the client_definitions.
I'd be willing to see this support native Jinja templates though as well. Unless I haven't had enough coffee today, I think we just need to change https://github.com/sensu/sensu-ansible/blob/45b93849fa162936f21f82032de6ca4dd4dca174/tasks/plugins.yml#L105 to template.

I tested template with files and it did not work. It fails on the folder as it’s not a template file. The TravisCI tests does not have files so the task is skipped.

FYI, the output using template:
TASK [external/sensu-ansible : Deploy client definitions] ***********************************************************************************
failed: [sensu-checks.######.com] (item=sensu_checks) => {"changed": false, "item": "sensu_checks", "msg": "could not find src=/Users/michaelporter/######/awx-plays/plays/data/static/sensu/client_definitions/sensu_checks, Could not find or access '/Users/michaelporter/######/awx-plays/plays/data/static/sensu/client_definitions/sensu_checks' on the Ansible Controller.\nIf you are using a module and expect the file to exist on the remote, see the remote_src option"}
failed: [sensu-checks.######.com] (item=sensu_clients) => {"changed": false, "item": "sensu_clients", "msg": "could not find src=/Users/michaelporter/######/awx-plays/plays/data/static/sensu/client_definitions/sensu_clients, Could not find or access '/Users/michaelporter/######/awx-plays/plays/data/static/sensu/client_definitions/sensu_clients' on the Ansible Controller.\nIf you are using a module and expect the file to exist on the remote, see the remote_src option"}

Partial listing of that folder:
± |master U:2 ✗| → ll /Users/michaelporter/######/awx-plays/plays/data/static/sensu/client_definitions/sensu_checks
total 296
drwxr-xr-x  39 michaelporter  staff  1248 Feb 13 06:12 ./
drwxr-xr-x  10 michaelporter  staff   320 Feb 23 09:46 ../
-rw-r--r--@  1 michaelporter  staff   565 Feb 13 11:51 check_elasticsearch_snapshot.json

This could become a new function
- name: Ensure local directories exist
  file:
    state: directory
    dest: "{{ static_data_store }}/sensu/{{ item }}"
  delegate_to: localhost
  become: no
  loop:
    - checks
    - filters
    - handlers
    - mutators
    - definitions
    - client_definitions
    - client_templates # add templates

- name: Register available client templates
  command: "ls {{ static_data_store }}/sensu/client_templates"
  delegate_to: localhost
  register: sensu_available_client_templates
  changed_when: false
  become: false

- name: Ensure directory structure exists
  file:
    path: '{{ sensu_config_path }}/conf.d/{{ item.path | dirname }}'
    state: directory
  with_filetree: '{{ static_data_store }}/sensu/client_templates'
  when:
    - item.path | dirname in group_names
    - sensu_available_client_templates is defined
    - sensu_available_client_templates is not skipped

- name: Deploy client templates
  copy:
    src: "{{ static_data_store }}/sensu/client_templates/{{ item }}/"
    dest: "{{ sensu_config_path }}/conf.d/{{ item.path | dirname }}/{{ item.path | basename | regex_replace('.j2', '') }}"
    owner: "{{ sensu_user_name }}"
    group: "{{ sensu_group_name }}"
  with_filetree: "{{ static_data_store }}/sensu/client_templates"
  when:
    - item.path | dirname in group_names
    - sensu_available_client_templates is defined
    - sensu_available_client_templates is not skipped
  notify: restart sensu-client service

Ah @michaelpporter, good catch! I was going to test that PR some more locally (had to run some errands after opening it). So, with that,  perhaps we do just move templates into their own setup.
Perhaps, I'm just missing it, but in your example above, where do the .j2 templates get templated out? Or are you saying that's the task we should right? I think that seems like a totally fair addition in general though.

@jaredledvina the last one should be template, not copy. I edited it. I have not tested it with live templates. I did run though it with `debug. I can do more testing tomorrow or Monday.

PS: I got the idea and basic structure from https://stackoverflow.com/questions/41667864/can-the-templates-module-handle-multiple-templates-directories

PSS: Because with_filetree loops over file. if you have a folder with 5 templates it will try to make the directory 5 times. There might need to be a step to build the array so the remote connections are limited.

This seems to work.
- name: Register available client templates
  command: "ls {{ static_data_store }}/sensu/client_templates"
  delegate_to: localhost
  register: sensu_available_client_templates
  changed_when: false
  become: false

- name: Deploy client template folders
  file:
    path: '{{ sensu_config_path }}/conf.d/{{ item | basename }}'
    state: directory
    owner: "{{ sensu_user_name }}"
    group: "{{ sensu_group_name }}"
  when:
    - sensu_available_client_templates is defined
    - sensu_available_client_templates is not skipped
    - item in sensu_available_client_templates.stdout_lines
  loop: "{{ group_names|flatten }}"
  notify: restart sensu-client service

- name: Deploy client templates
  template:
    src: "{{ static_data_store }}/sensu/client_templates/{{ item.path | dirname }}/{{ item.path | basename }}"
    dest: "{{ sensu_config_path }}/conf.d/{{ item.path | dirname }}/{{ item.path | basename | regex_replace('.j2', '') }}"
    owner: "{{ sensu_user_name }}"
    group: "{{ sensu_group_name }}"
  with_filetree: "{{ static_data_store }}/sensu/client_templates"
  when:
    - item.state == 'file'
    - item.path | dirname in group_names
  notify: restart sensu-client service

Locally I updated molecule for Ubuntu. 14.04 is failing the inspec tests on the ports.
platforms:
  - name: ubuntu-14.04
    image: dokken/ubuntu-14.04
    command: /sbin/init
    capabilities:
      - SYS_ADMIN
    groups:
      - sensu_checks
  - name: ubuntu-16.04
    image: dokken/ubuntu-16.04
    command: /bin/systemd
    capabilities:
      - SYS_ADMIN
    groups:
      - sensu_checks
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
  - name: ubuntu-18.04
    image: dokken/ubuntu-18.04
    command: /bin/systemd
    capabilities:
      - SYS_ADMIN
    groups:
      - sensu_checks
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
# .....
  inventory:
    group_vars:
      all:
        sensu_master: true
        sensu_include_dashboard: true
        sensu_rabbitmq_server: true
        sensu_redis_server: true
        sensu_rabbitmq_host: "{{ ansible_hostname }}"
        sensu_redis_host: "{{ ansible_hostname }}"
        sensu_api_host: "{{ ansible_hostname }}"
        ansible_default_ipv4:
          address: <IP_ADDRESS>
        sensu_remote_plugins:
          - sensu-plugins-disk-checks
        sensu_check_interval: 60

# Ensure disk check exists
describe file('/etc/sensu/conf.d/sensu_masters/check_disk_usage.json') do
  it { should exist }
  its('content') { should match(/check-disk-usage/) }
end

# Ensure disk metrics exists
describe file('/etc/sensu/conf.d/sensu_checks/metrics_disk_usage.json') do
  it { should exist }
  its('content') { should match(/metrics-disk-usage/) }
  its('content') { should match(/60/) }
end

# Ensure not_used does not exist
describe file('/etc/sensu/conf.d/not_used/not_a_check.json') do
  it { should_not exist }
end

tree
.
|____sensu
| |____filters
| |____client_templates
| | |____not_used
| | | |____not_a_check.json.j2
| | |____sensu_checks
| | | |____metrics_disk_usage.json.j2
| |____checks
| |____client_definitions
| | |____sensu_masters
| | | |____check_disk_usage.json
| |____mutators
| |____definitions
| |____handlers

PR merged and released in 5.1.0!
https://github.com/sensu/sensu-ansible/releases/tag/5.1.0
Thanks!!


Design RYR OpenAPI 1.0.0

Types of changes

Code cleanup / Refactoring
Documentation

Description

This is an attempt to define the current API using the OpenAPI 3.0.1
specification.
Motivation and Context

Using modern tools to generate server stubs and documentation from one
well defined document.
How Has This Been Tested?

Checklist:

[X] I have updated the documentation accordingly
[] I have written unit tests

Fixes request-yo-racks/api#57

Codecov Report

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

@@           Coverage Diff           @@
##           master      #59   +/-   ##
=======================================
  Coverage   98.34%   98.34%
=======================================
  Files           6        6
  Lines         241      241
=======================================
  Hits          237      237
  Misses          4        4

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 72bf673...9a310db. Read the comment docs.


Add CloudCover's pledge to manifesto

Added in https://github.com/opentffoundation/manifesto/pull/609.


Support changing system default ProxySelector

The ProxySelectorRoutePlanner class which got deprecated in favor of the SystemDefaultRoutePlanner could deal with:

null as default ProxySelector
a change in the default ProxySelector

This change ports that behavior to the SystemDefaultRoutePlanner.

Committed to SVN trunk and 4.5.x branch. Please review and close.


$attachment problem

Type

[*] Functions: $attachment
[ ] Callbacks: <Callback Name>
[ ] Handlers: <Handler Type>
[ ] Others: ______

Code used:
Your Code Here
const aoijs = require("aoi.js")

const bot = new aoijs.Bot({
token: "TOKEN", //Discord Bot Token
prefix: "?", //Discord Bot Prefix
intents: ["GUILDS", "GUILD_MESSAGES"] //Discord Intents
})

//Events
bot.onMessage()

//Command Example
bot.command({
name: "ping",
code: `Pong! $pingms`
})
bot.command({
  name: "dog",
  code: `$attachment[https://cdn.discordapp.com/attachments/773357374328012840/780585674541105152/20201116_133035.jpg] Take a pic of Kuba's dog!`
});

//Ready Event
bot.readyCommand({
    channel: "",
    code: `$log[Ready on $userTag[$clientID]]`
})

More:

Aoi.js version: 5.0.1
Node.js version: 16.13
Operating system: Debian 11

Screenshots (If any): Enabled Status
Ready on NodeJSTest#8390
Initialized on aoi.js || v5.0.2
Our Official Discord Server: https://aoi.js.org/invite
TypeError: Cannot read properties of undefined (reading 'addBrackets')
at Function.module.exports [as code] (/home/andrelaptop/Scrivania/DiscordBotJS/node_modules/aoi.js/package/functions/Funcs/calling/attachment.js:8:73)
at Interpreter (/home/andrelaptop/Scrivania/DiscordBotJS/node_modules/aoi.js/package/interpreter.js:70:108)
at module.exports (/home/andrelaptop/Scrivania/DiscordBotJS/node_modules/aoi.js/package/Handler/guildMessages/commands.js:74:13)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at async Client. (/home/andrelaptop/Scrivania/DiscordBotJS/node_modules/aoi.js/package/classes/Bot.js:22:13)

you need to give the attachment a name

How does data part works though?

How does data part works though?

attachment doesnt have any data part tho

you need to give the attachment a name

Where? I copy the command as write as docs

$attachment[url;name]

No response.


fix(canary-web-components): update date input calendar and clear to update appearance

Summary of the changes
Update date input calendar and clear to update appearance when focused to light variant
Related issue
N/A
Checklist

[x] Relevant unit tests and visual regression tests added.
[x] Visual testing against Figma component specification completed.
[x] All acceptance criteria reviewed and met.
[x] Accessibility Insights FastPass performed.
[x] A11y unit test added and yields no issues.
[x] A11y plug-in on Storybook yields no issues.
[x] Manual screen reader testing performed using NVDA and VoiceOver.
[x] Page can be zoomed to 400% with no loss of content.
[x] Screen magnifier used with no issues.
[x] Text resized to 200% with no loss of content.
[x] Text spacing increased as per the WCAG 1.4.12 success criterion with no loss of content.
[x] Browser setting 'prefers reduced motion' tested. No animations or motion visible whilst this setting is on.
[x] Windows High Contrast mode tested with no loss of content.
[x] System light and dark mode tested with no loss of content.
[x] Manual keyboard testing for keyboard controls and logical focus order.
[x] Min/max content examples tested with no loss of content or overflow.
[x] Browser support tested (Chrome, Safari, Firefox and Edge).
[x] Correct roles used and ARIA attributes used correctly where required.
[x] Logical heading structure is maintained, and the HTML elements used for headings can be changed to fit within the wider page structure.
[x] All prop combinations work without issue.
[x] Changes to docs package checked and committed.
[x] Tested for FOUC (Flash of Unstyled Content) in both SSR (Server-Side Rendering) and SSG (Static Site Generation) settings.

have tested in all browsers on windows & all good


builds are really big

I think some node_modules are slipping in. Probably don't need to ship the reference screenshots either... unless they are needed in the docs.
The next release needs to strip these out.

[ ] fix MANIFEST.in

IIRC, I also saw that in you setup.py you are including data_files, and IIRC that put all the .git repo under the package... check if that if still happening...

I thought this would handle that:
https://github.com/Anaconda-Server/nbpresent/blob/master/MANIFEST.in#L17

I did not check recently... this is just something I saw, let say, around 3 weeks ago... I will check later again...

This has been fixed... there are perhaps still some issues when building locally, but the binstar build is clean.


Make the HDMI2Ethernet SoC stream proper RTP packets that can be received via gstreamer

This can be used as a basis:
https://github.com/timvideos/HDMI2USB-misoc-firmware/blob/master/hdl/streamer/rtp.py
I'm not well aware of RTP, so I think some metadata are missing (sync, ect...)

FYI A summer of code student looked at doing RTP stuff as part of GSoC. His blog can be found at http://hdmi2ethernet.blogspot.de/
There is a bunch of information at

http://hdmi2ethernet.blogspot.de/2014/05/tools-for-rtp-packet-creation.html
http://hdmi2ethernet.blogspot.de/2014/05/rtp-packet-crafting-using-dpkt-180.html
http://hdmi2ethernet.blogspot.de/2014/07/debugging-gstreamer-and-testing.html

@aomtoku - Please take a look at this github issue.


chore(deps): upgrade compiler dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-compiler-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2220

Questions ?
Please refer to the Backport tool documentation


Quiz

@yatulearn is the pull request has been accepted or not i already submited it

same issue has been resolved by some else, and you have not responded since 5 days I asked to make changes, kindly check it once


[PDI-13796] - can use Dimension Lookup and Combination Lookup steps a…

…s the data provider for a Shared Dimension

@bytekast @DFieldFL @pminutillo please review.
Sorry about all the noise with the spaces.  Existing code had lots of checkstyle violations


How to pronunce it

How to pronunce "capsule"?
It should be pronounced as /ˈkæpsjuːl/ with a bit of french accent :)

What I really love from this pronounce is the presence of the k... That's a Kubernetes reference!


Addition of a build system generator

I suggest to reuse a higher level build system than your current make script so that powerful checks for software features will become easier.

CMake
Autotools

They are just tools to 'meta' the final makefile. I think for this small project the current makefile is okay so there is no need to meta it and introduce addtional complexity. I am also not very familiar with them...
There is a cmake makefile contribute by others, I didnt update it recently, not sure if it still works.

Does a tool like “autoscan” show any software dependencies that you find worth for further considerations?
How do you think about to support more development environments by the reuse of build system generators?


Fix a broken link to troubleshooting doc in README

What does this PR do?
Closes #828.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #829 (f60bbcf) into master (07b45e7) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #829      +/-   ##
==========================================
- Coverage   80.55%   80.52%   -0.04%
==========================================
  Files         254      254
  Lines        8552     8552
  Branches     1453     1453
==========================================
- Hits         6889     6886       -3
- Misses       1319     1323       +4
+ Partials      344      343       -1

Impacted Files
Coverage Δ

...ain/java/com/datadog/opentracing/PendingTrace.java
58.12% <0.00%> (-0.85%)
:arrow_down:

...rc/main/java/com/datadog/opentracing/DDTracer.java
55.60% <0.00%> (-0.83%)
:arrow_down:

...android/rum/internal/ndk/DatadogNdkCrashHandler.kt
87.35% <0.00%> (-0.60%)
:arrow_down:

...droid/rum/tracking/ActivityViewTrackingStrategy.kt
88.46% <0.00%> (+1.92%)
:arrow_up:


Added GCD Calculator

Adds Calculator🛠
fixes #155
Description👨💻
GCD Calculator calculates greatest common divisor of two values or multiple values
Type of change📄

✅ Adds calculator

Screenshots📷

Pls add a Readme.md in your calculator folder @rishiiiidha

@Rakesh9100 I have a doubt do I have to create another pull request or I can add here itself,
If that's possible please do let me know.

You can do commit in this PR only, no need to open another PR @rishiiiidha
Just add Readme file in your fork local and then commit and push the changes.

Nice work, Merged!! ✨
Congrats on getting your PR merged to this repo under SWOC S4 🥳🎉 @rishiiiidha


If linearGradient angle == TWO_PI the gradient will be fully opaque

I hope you don't find these small issues annoying... I came across a minor bug I think. When you give the gradient an angle of TWO_PI the gradient will be opaque (first colorstop).
peasyGradients.linearGradient(gradient, TWO_PI);

Angle == TWO_PI

versus angle == PI

Interesting find... I'll look into it.


[VQC-CHALLENGE] [TheQuantumSquad007] [26.10.2022]

Team members
Zentz#5399
Tyy#5960
Terkasaurus#6388
fcodelrio#2239
Gehad Salem#7458
saraok#9644
Submission summary
Following the implementation from the paper: https://arxiv.org/pdf/1804.11326.pdf
More explanations are given in the ipynb file.
First Submission
Additional comments

Congrats! Your model achieved 100% accuracy on the judging data set and is currently tied for 1st on the leaderboard.
Thank you for pointing out the RX/RZ typo in the feature map of gen_binary.py. This was not intentional, and we apologize if it created confusion. I'll make an announcement on Discord later today.
In addition, we will soon release an optional second leg of the challenge, which will involve solving the same classification task, but this time using a kernel estimator instead of a VQC. This will act as the tie-breaker between teams whose VQC models achieve 100% accuracy.


Vim9: crash when specifying type in unpack notation in ":for" loop

Describe the bug
In Vim9 script, specifying a type when using the unpack notation in a :for loop makes Vim crash.
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    vim9
    def Func()
        for [n: number, s: string] in [[1, 'a'], [2, 'b']]
        endfor
    enddef
    defcompile
EOF
)

Vim crashes.
Expected behavior
Vim doesn't crash.
Environment

Vim version: 8.2 Included patches: 1-2366
OS: Ubuntu 16.04.7 LTS
Terminal: xterm(363)

Additional context
Regression introduced in 8.2.2034.

backtrace
valgrind log
asan log

I have another question regarding this snippet.
Consider this other one, which is very similar:
vim9
var l: list<number> = [1, 2]
for  [x, y, _]
in  [[0, 1, ''], [0, 1, '']]
    l[x] = l[x] + y
endfor

no error

It doesn't raise any error, which is expected.
Now, same code in a :def function:
vim9
def Func()
    var l: list<number> = [1, 2]
    for  [x, y, _]
    in  [[0, 1, ''], [0, 1, '']]
        l[x] = l[x] + y
    endfor
enddef
defcompile

E39: Number expected

This time, E39 is raised.  This seems inconsistent.
I thought that the issue came from the fact that the items were inferred with the type any, because the lists contain a mix of values, and that Vim wants a value with the type number to index a list.  But that's not the explanation:
vim9
def Func()
    var l: list<number> = [1, 2]
    for  [x, y, _]
    in  [[0, 1, ''], [0, 1, '']]
        echo l[x]
    endfor
enddef
Func()

1
1

Here, Vim doesn't complain that x has the type any when indexing l.
Workaround: Use an intermediate variable.
vim9
def Func()
    var l: list<number> = [1, 2]
    for  [x, y, _]
    in  [[0, 1, ''], [0, 1, '']]
        var n: number = x
        l[n] = l[n] + y
    endfor
enddef
defcompile

no error

Is this another bug?  If it's working as intended, is there any other way to avoid the intermediate variable, which looks a bit weird here?
I thought about specifying the type directly in the :for statement:
for  [x: number, y, _]
       ^------^

That's when I found the crash.
I think it's not supposed to work, because of this comment:

I don't like it, too complicated.  Either type inferrence should work,
or the types should be declared separately.

Although, the current case is a bit different.  Here, we're talking about specifying types in the unpack notation in a :for loop.  There, we were talking about specifying types in the unpack notation in a :var assignment.  Does that make any difference?
If we still won't allow specifying the types right in the :for statement, I thought about declaring the type earlier:
vim9
def Func()
    var l: list<number> = [1, 2]
    var x: number
    for  [x, y, _]
    in  [[0, 1, ''], [0, 1, '']]
        var n: number = x
        l[n] = l[n] + y
    endfor
enddef
Func()

But this raises E1017:
E1017: Variable already declared: x, y, _]

I have another question regarding this snippet.
Consider this other one, which is very similar:
vim9
var l: list<number> = [1, 2]
for  [x, y, _]
in  [[0, 1, ''], [0, 1, '']]
    l[x] = l[x] + y
endfor

no error

It doesn't raise any error, which is expected.
Now, same code in a :def function:
vim9
def Func()
    var l: list<number> = [1, 2]
    for  [x, y, _]
    in  [[0, 1, ''], [0, 1, '']]
        l[x] = l[x] + y
    endfor
enddef
defcompile

E39: Number expected

This time, E39 is raised.  This seems inconsistent.
I thought that the issue came from the fact that the items were inferred with the type any, because the lists contain a mix of values, and that Vim wants a value with the type number to index a list.  But that's not the explanation:
vim9
def Func()
    var l: list<number> = [1, 2]
    for  [x, y, _]
    in  [[0, 1, ''], [0, 1, '']]
        echo l[x]
    endfor
enddef
Func()

1
1

Here, Vim doesn't complain that x has the type any when indexing l.
Workaround: Use an intermediate variable.
vim9
def Func()
    var l: list<number> = [1, 2]
    for  [x, y, _]
    in  [[0, 1, ''], [0, 1, '']]
        var n: number = x
        l[n] = l[n] + y
    endfor
enddef
defcompile

no error

Is this another bug?  If it's working as intended, is there any other way to avoid the intermediate variable, which looks a bit weird here?
I thought about specifying the type directly in the :for statement:
for  [x: number, y, _]
       ^------^

That's when I found the crash.
I think it's not supposed to work, because of this comment:

I don't like it, too complicated.  Either type inferrence should work,
or the types should be declared separately.

Although, the current case is a bit different.  Here, we're talking about specifying types in the unpack notation in a :for loop.  There, we were talking about specifying types in the unpack notation in a :var assignment.  Does that make any difference?
If we still won't allow specifying the types right in the :for statement, I thought about declaring the type earlier:
vim9
def Func()
    var l: list<number> = [1, 2]
    var x: number
    for  [x, y, _]
    in  [[0, 1, ''], [0, 1, '']]
        var n: number = x
        l[n] = l[n] + y
    endfor
enddef
Func()

But this raises E1017:
E1017: Variable already declared: x, y, _]

Here, Vim doesn't complain that x has the type any when indexing l.

So, why does Vim complain in the previous snippet?

Here, Vim doesn't complain that x has the type any when indexing l.

So, why does Vim complain in the previous snippet?


Updating to Terraform version 13

The secure foundation deployment is shifting to version 13. You might want to update to align with the foundation.

great idea.  I'll check it out and run some tests.


Build error on windows: access to extern crates through prelude is experimental

I'm trying to build on windows 10 with msys and I get the following error:
$ rustc --version
rustc 1.29.2

$ cargo build --release --features "cairo-backend"
   Compiling resvg v0.5.0 (file:///D:/Adrian/node-svg/resvg)
error[E0658]: access to extern crates through prelude is experimental (see issue #44660)
   --> src\geom.rs:119:30
    |
119 |     fn transform(&self, ts: &usvg::Transform) -> Self;
    |

[... more of the same ...]

error: aborting due to 7 previous errors

For more information about this error, try `rustc --explain E0658`.
error: Could not compile `resvg`.

To learn more, run the command again with --verbose.

You should use the latest stable Rust. Which is 1.32.
Will fix it.


QuickAppLauncher- How to launch\Auto Launch?

Great App! Thank you.
Sideloaded the app using Sidequest, can launch from SideQuest when connected to my PC fine.
1.) Is there a way to launch it from within Quest native UI? For example: QuickLaunch don't show up as a Quest TV Channel, even though I have debugging turned on...
2.) Is there a way to autoload this app by default?
3.) How to I disable Login checking for ALL/Any Google Play apps, so we can get them to run on a Quest?
Is there any way to do that auto when executing specific apps via QuickAppLauncher?
Example, I am trying to run Tello Drone app natively on my Quest, and it won't run because of Auth checking.
Can you guys help?
Thanks in advance.

Wish there were clear instructions on getting the latest to run on Quest 2.
I only get a Settings Gear icon and my background I put on .
The actual app icons don't show up like they used in prior releases.
Or better yet, a Tut video


Using config in Plugins module

Firstly, great work on the module. It's really helpful!
I'm having trouble following how I'm supposed to use the config parameter in the playbook. I've tried many different variations of syntax with no success. I want to enable the key-auth plugin with a custom key_names value — the following is how I thought it might work:
...
    - name: Add key authentication
      kong_plugin:
        kong_admin_uri: "{{kong_admin_base_url}}"
        api_name: "test"
        plugin_name: "key-auth"
        config:
            key_names: "key"
        state: present

But when I try to run this I get the following error when checking the plugin was enabled properly:
TASK [Verify key auth was added] ***********************************************
fatal: [localhost]: FAILED! => {"changed": false, "connection": "close", "content": "Cannot GET /\n", "content_length": "13", "content_type": "text/html; charset=utf-8", "date": "Mon, 24 Oct 2016 16:22:24 GMT", "failed": true, "msg": "Status code was not [401]: HTTP Error 404: Not Found", "redirected": false, "status": 404, "url": "http://gateway.app/", "via": "kong/0.9.3", "x_content_type_options": "nosniff", "x_kong_proxy_latency": "0", "x_kong_upstream_latency": "26", "x_powered_by": "Express"}

As it turns out, the plugin wasn't enabled (probably due to the syntax / data type issues with the config).
Have you got any examples of how I can use this config bit? Thanks!

A bit more playing around today with a packet capture revealed the problem. I was getting an error during the plugin create HTTP request {"key_names":"key_names is an unknown field"} so I changed my config to the following and it worked:
    - name: Add key authentication
      kong_plugin:
        kong_admin_uri: "{{kong_admin_base_url}}"
        api_name: "test"
        plugin_name: "key-auth"
        config:
            config.key_names: "key"
        state: present

Apparently I needed to prefix my config list with config.. Could you add this to your documentation please?


Cant change  damage color

changing damage color makes it invisible and shows it as white in settings

CC @rizecookey, any updates?

im still having this issue

The color code for setting the damage color is a bit unintuitive as it works a bit differently from your regular color codes because it also encodes opacity, which means:

the first two hexadecimal digits encode the opacity (00 being completely transparent, FF being fully opaque)
the following 6 hexadecimal digits work like a regular color code (two digits each for red, green, and blue)

So if you want to change the damage color but not the opacity, I'd recommend keeping the first two digits the same as the default and only changing the last six (for which you can use a regular HTML color code picker). I might add some sliders in a future update that could make this easier to use, though.

Thank you! This is kind of a weird way to encode data into a hex code, would definitely make more sense to use a format like HSL or include sliders as you mentioned for opacity stuff.

Yeah, the config screen library I use sadly doesn't provide a better color picking component. I've added some sliders myself in release 1.7.8, though.


Notedown fork: Knitty

I forked Notedown to knitty. And there's neither entry points nor module names conflicts. Fixed docs accordingly.

does knitty do everything this does? It looks like this project is inactive

Knitty should do everything that project does as I forked it and made notedown tests work (even fixed some R bugs on Windows). But at the same time I no longer use notedown as latest versions of pandoc + jupytext made it obsolete.

ok thanks