[vault-secrets-webhook] Webhook don't mutate pods

Hello we have external vault server and vault-secrets-webhook 1.4.0
Describe the bug:
We have added annotations in deployment
apiVersion: apps/v1
kind: Deployment
...
spec:
...
  template:
    metadata:
      annotations:
        vault.security.banzaicloud.io/vault-addr: https://vault.external.com/
        vault.security.banzaicloud.io/vault-path: kubernetes-stage
        vault.security.banzaicloud.io/vault-role: some-role
        vault.security.banzaicloud.io/vault-skip-verify: "true"
containers:
    - name: <some-name>
      image: <some-image>
      env:
        - name: DATABASE_URL
          value: 'vault:path/data/dev/backend/databases/phprotect#url

After deploy, pods are creating without init vault containers
Expected behaviour:
vault-secrets-webhook mutates out pods
Steps to reproduce the bug:

Install chart v1.4.0 to namespace vault with values.yaml:

# Default values for vault-secrets-webhook.
# This is a YAML-formatted file.
# Declare variables to be passed into your templates.

replicaCount: 2

debug: false

certificate:
  useCertManager: false
  generate: true
  server:
    tls:
      crt:
      key:
  ca:
    crt:
image:
  repository: banzaicloud/vault-secrets-webhook
  # tag: ""
  pullPolicy: IfNotPresent
  imagePullSecrets: []

service:
  name: vault-secrets-webhook
  type: ClusterIP
  externalPort: 443
  internalPort: 8443

serviceAccount:
  labels: {}
  #  useful: label
  annotations: {}
  # Enables GKE workload identity
  #  iam.gke.io/gcp-service-account<EMAIL_ADDRESS>
env:
  # VAULT_IMAGE: vault:1.5.0
  VAULT_ENV_IMAGE: banzaicloud/vault-env:1.4.0
  # VAULT_CAPATH: /vault/tls
  # used when the pod that should get secret injected does not specify
  # an imagePullSecret
  # DEFAULT_IMAGE_PULL_SECRET:
  # DEFAULT_IMAGE_PULL_SECRET_NAMESPACE:
  # VAULT_CLIENT_TIMEOUT: 10s

metrics:
  enabled: false
  port: 8443
  serviceMonitor:
    enabled: false
    scheme: https
    tlsConfig:
      insecureSkipVerify: true

securityContext:
  runAsUser: 65534

volumes: []
# - name: vault-tls
#   secret:
#     secretName: vault-tls

volumeMounts: []
# - name: vault-tls
#   mountPath: /vault/tls

podAnnotations: {}

labels: {}
  #  team: banzai

resources: {}

nodeSelector: {}

tolerations: []

affinity: {}

## Assign a PriorityClassName to pods if set
priorityClassName: ""

rbac:
  enabled: true
  psp:
    enabled: false

# A list of Kubernetes resource types to mutate as well:
# Example: ["ingresses", "servicemonitors"]
customResourceMutations: []

customResourcesFailurePolicy: Ignore

# This can cause issues when used with Helm, so it is not enabled by default
configMapMutation: false

configMapFailurePolicy: Ignore

podsFailurePolicy: Ignore

secretsFailurePolicy: Ignore

apiSideEffectValue: NoneOnDryRun

namespaceSelector:
  # matchExpressions:
  #   - key: name
  #     operator: NotIn
  #     values:
  #       - kube-system
  matchLabels:
    sidecar.vault.io/inject: "true"

# In case of the K8s cluster version is above 1.15 objectSelector is usable
objectSelector: {}
  # matchExpressions:
  # - key: security.banzaicloud.io/mutate
  #   operator: NotIn
  #   values:
  #   - skip

podDisruptionBudget:
  enabled: true
  minAvailable: 1

Create namespace

kubectl create namespace project
kubectl label namespace project sidecar.vault.io/inject=true

Apply deployment

Additional context:
Add any other context about the problem here.
Environment details:

Kubernetes version (e.g. v1.17.3):
Cloud-provider/provisioner (e.g. run on EC2):
Install method (e.g. helm):
Logs from the misbehaving component (and any other relevant logs): vault-secrets-webhook pod logs

time="2020-08-05T09:55:48Z" level=warning msg="no tracer active" app=vault-secrets-webhook
time="2020-08-05T09:55:48Z" level=warning msg="no tracer active" app=vault-secrets-webhook
time="2020-08-05T09:55:48Z" level=warning msg="no tracer active" app=vault-secrets-webhook
time="2020-08-05T09:55:48Z" level=warning msg="no tracer active" app=vault-secrets-webhook
time="2020-08-05T09:55:48Z" level=info msg="Listening on https://:8443" app=vault-secrets-webhook

When i set podsFailurePolicy: "true" then i see error
Error creating: Internal error occurred: failed calling webhook "pods.vault-secrets-webhook.admission.banzaicloud.com": Post https://vault-secrets-webhook.vault.svc:443/pods?timeout=30s: context deadline exceeded

Have you checked this @javdet ?
https://github.com/banzaicloud/bank-vaults/blob/b2fefa92005166b6084728a0d8be90d64682c309/charts/vault-secrets-webhook/README.md#about-gke-private-clusters

I don't have use gke, but i added allow port 8443. And i see the same error - timeout

Can you check that the service is having the pods as endpoints? Also that you don't have some special firewall settings?

Hello.
I am having a similar issue. Looking into the API server logs, I got:
W0812 16:57:43.619928 1 dispatcher.go:168] Failed calling webhook, failing open pods.vaultenv-webhook.admission.banzaicloud.com: failed calling webhook "pods.vaultenv-webhook.admission.banzaicloud.com": expected webhook response of admission.k8s.io/v1, Kind=AdmissionReview, got /, Kind=
E0812 16:57:43.619957 1 dispatcher.go:169] failed calling webhook "pods.vaultenv-webhook.admission.banzaicloud.com": expected webhook response of admission.k8s.io/v1, Kind=AdmissionReview, got /, Kind=
W0812 16:57:43.621440 1 dispatcher.go:168] Failed calling webhook, failing open secrets.vaultenv-webhook.admission.banzaicloud.com: failed calling webhook "secrets.vaultenv-webhook.admission.banzaicloud.com": expected webhook response of admission.k8s.io/v1, Kind=AdmissionReview, got /, Kind=
E0812 16:57:43.621459 1 dispatcher.go:169] failed calling webhook "secrets.vaultenv-webhook.admission.banzaicloud.com": expected webhook response of admission.k8s.io/v1, Kind=AdmissionReview, got /, Kind=

This started to happen after upgrading my Kubernetes version from 1.15 to 1.16. I performed some tests using kubectl port-forward and I was able to get a response:
$ k port-forward vaultenv-webhook-78c94445cb-cnjl8 -n vaultenv 8443:8443
Forwarding from <IP_ADDRESS>:8443 -> 8443
Forwarding from [::1]:8443 -> 8443

$ curl -k https://localhost:8443/pods -i
HTTP/2 400
content-type: text/plain; charset=utf-8
x-content-type-options: nosniff
content-length: 14
date: Wed, 12 Aug 2020 17:44:59 GMT

no body found

$ curl -k https://localhost:8443/secrets -i
HTTP/2 400
content-type: text/plain; charset=utf-8
x-content-type-options: nosniff
content-length: 14
date: Wed, 12 Aug 2020 17:45:04 GMT

no body found

After digging a bit, I found this issue that looks related: https://github.com/kubernetes/kubernetes/issues/85681
Additional info
Kubernetes version (e.g. v1.17.3): v1.16.13-eks-2ba888
Cloud-provider/provisioner (e.g. run on EC2): EKS
Install method (e.g. helm): Terraform (static manifests)
What do you think @bonifaido?

First of all, I think we have two separate issues here:

I have submitted #1078 to resolve the issue of @hugomcfonseca, where the webhook registration is defeated by the Kubernetes version.
The original issue of @javdet is some kind of firewall issue in my opinion, and I haven't got back an answer for that one.

I have updated to vault-webhook 1.12.0 and when I updated and restarted any other pod, the mutation webhook didn't created the container: copy-vault-env
When I set podsFailurePolicy to Fail it started adding again.
Is there any reason for this?
Thanks :)

API log errors:
I0515 09:28:11.485423       1 trace.go:205] Trace[179036938]: "Call mutating webhook" configuration:vault-secrets-webhook,webhook:pods.vault-secrets-webhook.admission.banzaicloud.com,resource:/v1, Resource=pods,subresource:,operation:CREATE,UID:7a0e2f55-acd3-47ca-b875-4a39682ff604 (15-May-2021 09:28:01.485) (total time: 10000ms):
Trace[179036938]: [10.000376401s] [10.000376401s] END

W0515 09:28:11.485461       1 dispatcher.go:170] Failed calling webhook, failing open pods.vault-secrets-webhook.admission.banzaicloud.com: failed calling webhook "pods.vault-secrets-webhook.admission.banzaicloud.com": Post "https://vault-secrets-webhook.vault.svc:443/pods?timeout=10s": context deadline exceeded

Seems like your webhook pod is not accessible in the time of the mutation, can you please give more details about your cluster, k8s version, etc...?

Of course:
vault version: 1.7.0
cluster version v1.19.6-eks-49a6c0
HELM Chart Version: 1.12.0
From containainer in the same cluster:
/ # curl -k https://vault-secrets-webhook.vault.svc/pods
no body found
/ # curl -k https://vault-secrets-webhook.vault.svc/pods -LIS
HTTP/2 400
content-type: text/plain; charset=utf-8
x-content-type-options: nosniff
content-length: 14
date: Tue, 18 May 2021 12:30:50 GMT

I have this webhook deployed in other 3 clusters, and it works well (it has different version). It worked in this cluster until I upgrade it. I have tried to downgrade and it still fails. I have also tried to recreate the entire cluster with no effect.
Endpoint:
kubectl get endpoints -n vault                                                                                INT  at production/kong kube
NAME                    ENDPOINTS                           AGE
vault-secrets-webhook   <IP_ADDRESS>:8443,<IP_ADDRESS>:8443   2d5h

Service
NAME                    TYPE        CLUSTER-IP     EXTERNAL-IP   PORT(S)   AGE
vault-secrets-webhook   ClusterIP   <IP_ADDRESS>   <none>        443/TCP   2d5h

If you need more details tell me.
Thanks!

Are the webhook pods running correctly, how many instance you have of those?

Yes, and when service starts, it appears the log of the image cache
NAME                                     READY   STATUS    RESTARTS   AGE
vault-secrets-webhook-77586c8d85-d4xjx   2/2     Running   0          5h28m
vault-secrets-webhook-77586c8d85-pdx9p   2/2     Running   0          5h29m

+ vault-secrets-webhook-77586c8d85-d4xjx › vault-secrets-webhook
+ vault-secrets-webhook-77586c8d85-pdx9p › vault-secrets-webhook
vault-secrets-webhook-77586c8d85-pdx9p vault-secrets-webhook time="2021-05-18T07:37:41Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-77586c8d85-pdx9p vault-secrets-webhook time="2021-05-18T07:37:41Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-77586c8d85-pdx9p vault-secrets-webhook time="2021-05-18T07:37:41Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-77586c8d85-pdx9p vault-secrets-webhook time="2021-05-18T07:37:41Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-77586c8d85-pdx9p vault-secrets-webhook time="2021-05-18T07:37:41Z" level=info msg="Listening on https://:8443" app=vault-secrets-webhook
vault-secrets-webhook-77586c8d85-d4xjx vault-secrets-webhook time="2021-05-18T07:38:00Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-77586c8d85-d4xjx vault-secrets-webhook time="2021-05-18T07:38:00Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-77586c8d85-d4xjx vault-secrets-webhook time="2021-05-18T07:38:00Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-77586c8d85-d4xjx vault-secrets-webhook time="2021-05-18T07:38:00Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-77586c8d85-d4xjx vault-secrets-webhook time="2021-05-18T07:38:00Z" level=info msg="Listening on https://:8443" app=vault-secrets-webhook

Could you please try with debug log level? Like LOG_LEVEL=debug applied to the webhook deployment.

Debug log when kong is started:
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:21:11Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:21:11Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:21:11Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:21:11Z" level=warning msg="no tracer active" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:21:11Z" level=info msg="Listening on https://:8443" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:00Z" level=debug msg="reviewing request 87e8b1c9-472c-4c78-9e22-83a9d0187ad0, named: kube-system/" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:01Z" level=debug msg="Successfully connected to the API" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:01Z" level=debug msg="No pod init containers were mutated" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:01Z" level=debug msg="No pod containers were mutated" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:03Z" level=debug msg="json patch for request 87e8b1c9-472c-4c78-9e22-83a9d0187ad0: []" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:35Z" level=debug msg="reviewing request 496a802a-85ae-4a63-80f1-c8f688a330c0, named: kong/" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:35Z" level=debug msg="Successfully connected to the API" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:35Z" level=debug msg="No pod init containers were mutated" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:35Z" level=debug msg="Successfully mutated pod containers" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:35Z" level=debug msg="Successfully appended pod init containers to spec" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:35Z" level=debug msg="Add generic volumes to podspec" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:35Z" level=debug msg="Successfully appended pod spec volumes" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:42Z" level=debug msg="json patch for request 496a802a-85ae-4a63-80f1-c8f688a330c0: [{\"op\":\"add\",\"path\":\"/spec/volumes/3\",\"value\":{\"name\":\"vault-env\",\"emptyDir\":{\"medium\":\"Memory\"}}},{\"op\":\"add\",\"path\":\"/spec/initContainers\",\"value\":[{\"name\":\"copy-vault-env\",\"image\":\"ghcr.io/banzaicloud/vault-env:1.12.0\",\"command\":[\"sh\",\"-c\",\"cp /usr/local/bin/vault-env /vault/\"],\"resources\":{\"limits\":{\"cpu\":\"250m\",\"memory\":\"64Mi\"},\"requests\":{\"cpu\":\"50m\",\"memory\":\"64Mi\"}},\"volumeMounts\":[{\"name\":\"vault-env\",\"mountPath\":\"/vault/\"}],\"imagePullPolicy\":\"IfNotPresent\",\"securityContext\":{\"allowPrivilegeEscalation\":false}}]},{\"op\":\"remove\",\"path\":\"/spec/containers/0/command/2\"},{\"op\":\"remove\",\"path\":\"/spec/containers/0/command/1\"},{\"op\":\"replace\",\"path\":\"/spec/containers/0/command/0\",\"value\":\"/vault/vault-env\"},{\"op\":\"add\",\"path\":\"/spec/containers/0/args\",\"value\":[\"/bin/sh\",\"-c\",\"kong migrations up\"]},{\"op\":\"add\",\"path\":\"/spec/containers/0/env/28\",\"value\":{\"name\":\"VAULT_ADDR\",\"value\":\"https://vault.example.com\"}},{\"op\":\"add\",\"path\":\"/spec/containers/0/env/29\",\"value\":{\"name\":\"VAULT_SKIP_VERIFY\",\"value\":\"false\"}},{\"op\":\"add\",\"path\":\"/spec/containers/0/env/30\",\"value\":{\"name\":\"VAULT_AUTH_METHOD\",\"value\":\"jwt\"}},{\"op\":\"add\",\"path\":\"/spec/containers/0/env/31\",\"value\":{\"name\":\"VAULT_PATH\",\"value\":\"production-kubernetes\"}},{\"op\":\"add\",\"path\":\"/spec/containers/0/env/32\",\"value\":{\"name\":\"VAULT_ROLE\",\"value\":\"kong\"}},{\"op\":\"add\",\"path\":\"/spec/containers/0/env/33\",\"value\":{\"name\":\"VAULT_IGNORE_MISSING_SECRETS\",\"value\":\"false\"}},{\"op\":\"add\",\"path\":\"/spec/containers/0/env/34\",\"value\":{\"name\":\"VAULT_ENV_PASSTHROUGH\",\"value\":\"VAULT_ADDR\"}},{\"op\":\"add\",\"path\":\"/spec/containers/0/env/35\",\"value\":{\"name\":\"VAULT_JSON_LOG\",\"value\":\"false\"}},{\"op\":\"add\",\"path\":\"/spec/containers/0/env/36\",\"value\":{\"name\":\"VAULT_CLIENT_TIMEOUT\",\"value\":\"10s\"}},{\"op\":\"add\",\"path\":\"/spec/containers/0/env/37\",\"value\":{\"name\":\"VAULT_LOG_LEVEL\",\"value\":\"debug\"}},{\"op\":\"add\",\"path\":\"/spec/containers/0/volumeMounts/3\",\"value\":{\"name\":\"vault-env\",\"mountPath\":\"/vault/\"}}]" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:47Z" level=debug msg="reviewing request e0d6fbda-f1d4-4905-bebb-ddb5a9427786, named: kong/" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:47Z" level=debug msg="Successfully connected to the API" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:47Z" level=debug msg="Successfully mutated pod init containers" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:49Z" level=debug msg="reviewing request 6de965f1-b5f8-4634-8d8f-430874261142, named: kong/" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:49Z" level=debug msg="Successfully connected to the API" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:49Z" level=debug msg="No pod init containers were mutated" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:49Z" level=debug msg="Successfully mutated pod containers" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:49Z" level=debug msg="Successfully appended pod init containers to spec" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:49Z" level=debug msg="Add generic volumes to podspec" app=vault-secrets-webhook
vault-secrets-webhook-5455458b9d-9h77k vault-secrets-webhook time="2021-05-18T13:22:49Z" level=debug msg="Successfully appended pod spec volumes" app=vault-secrets-webhook

Kong Pods:
NAME                                      READY   STATUS      RESTARTS   AGE
kong-kong-5d54679874-sp6kx                0/2     Init:0/2    0          4m2s
kong-kong-5f74b78676-8bxph                2/2     Running     0          5h20m
kong-kong-5f74b78676-xlwmj                2/2     Running     0          5h20m
kong-kong-post-upgrade-migrations-dmwxg   0/1     Completed   0          4m2s
kong-kong-pre-upgrade-migrations-jgg88    0/1     Completed   0          4m17s

Here, the migrations have mutated:
kubectl get pods kong-kong-post-upgrade-migrations-dmwxg -n kong -o jsonpath="{..image}" | tr -s '[[:space:]]' '\n' | sort | uniq                                                                        ok  at production/kong kube
ghcr.io/banzaicloud/vault-env:1.12.0
kong:2.4-alpine

But the kong pod hasn't mutated
kubectl get pods kong-kong-5d54679874-sp6kx -n kong -o jsonpath="{..image}" | tr -s '[[:space:]]' '\n' | sort | uniq                                                                                ok  at production/kong kube
docker.io/istio/proxyv2:1.9.5-distroless
kong:2.4-alpine

I have upgrade Vault webhook to 1.13.0 in all clusters, and all the clusters have failed with the same error.

Any help with this?
Can be a problem related with EKS or Istio? It worked before without any problem.
Thanks

I have found the error. It was the resource limits. I quit this and it start working again.
Thanks!

currently having this issue. @ichasco-heytrade do you mean resource limits on your app or resource limits on the webhook deployment. or just not enough resources in your clusteR?

I have set resource limits to webhook pod. And because of this it didn't mutate the secrets (there isn't any error log  in webhook pod or in the cluster referring to this). It has been hard to troubleshoot.
The limits I set were:
  resources:
    requests:
      memory: 24Mi
      cpu: 5m
    limits:
      memory: 48Mi
      cpu: 10m

hmm tried setting that on the webhook. still getting

Error creating: Internal error occurred: failed calling webhook "pods.vault-secrets-webhook.admission.banzaicloud.com": Post "https://vault-secrets-webhook.vault-infra.svc:443/pods?timeout=10s": dial tcp <IP_ADDRESS>:8443: i/o timeout

I do have multiple nodes with tolerations. maybe that has something to do with it?

My problem was when I set those values. When I unset them, it started working again.

I tried this on a fresh cluster in GKE all default settings. 1.19 still same behavior. when I added a new nodepool to the cluster the webhook was stopping containers from starting up. and the nodepool never got healthy it wasn't untill I deleted the mutatingwebhook that the cluster came up.  it also interfered with my gitlab runners. and none of them have the annotations.
wierdly enough this works perfectly on my local k3d cluster.

Change the timeoutSeconds from 10s to 30s in the vault-secrets-webhook mutatingwebhookconfiguration (kubectl edit mutatingwebhookconfiguration xxx -n yyyy) and see if that helps to fix the issue.
Note: There would be multiple timeoutSeconds in the webhook config so edit accordingly.

Hi @bonifaido  , I got the same case where webhook don't mutate pods, but the error in my kube-api-logs, looks like this:
I0609 10:47:58.158990      20 trace.go:205] Trace[418003165]: "Call mutating webhook" configuration:vswh-vault-secrets-webhook,webhook:pods.vault-secrets-webhook.admission.banzaicloud.com,resource:/v1, Resource=pods,subresource:,operation:CREATE,UID:0f382606-0f12-43a8-831c-2926a90af30f (09-Jun-2022 10:47:45.163) (total time: 12995ms):
Trace[418003165]: [12.995896231s] [12.995896231s] END
W0609 10:47:58.159070      20 dispatcher.go:182] Failed calling webhook, failing closed pods.vault-secrets-webhook.admission.banzaicloud.com: failed calling webhook "pods.vault-secrets-webhook.admission.banzaicloud.com": Post "https://vswh-vault-secrets-webhook.vswh.svc:443/pods?timeout=30s": context canceled

any idea, why is this happening ?

Closing this, as the original issue has been solved.


Cannot use no_learning method "prox_l2" with Downsampling operator through Trainer.test

I am setting up experiments to do super-resolution, with the Downsampling physics operator.
When wanting to use Trainer.test(...) we can specify a no_learning method string. I have tried to change it to "prox_l2".
Trainer.test calls : physics.prox_l2(y)
but the signature for Downsampling.prox_l2 is : prox_l2(self, z, y, gamma, use_fft=True), so I have the following error:
Downsampling.prox_l2() missing 2 required positional arguments: 'y' and 'gamma'

thanks for spotting this bug - I'll fix asap.


3 Controllers in SteamVR and some games but there are other like PaintLab and Waltz of Wizard that dont, this is not issue #136

Hello,
I have been using the code very succesfully, i have 3 controllers PSMOVE and  2 cameras with the latest version of PSmove Service, i am using FreePIE to do the head tracking with the controller which works just fine but only for some games there are other that take the head controller and 1 of the hand controllers and other other disapears, if press start to go back to steamVR your controller comes back and its available.
If I configure the head tracking with Opentrack and Kinect, and use only 2 controllers all games work just fine. Is there any known issues with FreePIE? I tried with FreePieBridge alpha 4.5 and 5 with same results.
thank you

I was going to write this at the same time. It would be awesome, if there was an option to hide on of the controller. For example controller which is used by PSMoveFreepieBridge, isn't seen by SteamVR.  It will be a really good option, beacuse many games, like for example Budget Cuts demo use 2 of 3 connected controllers randomly.

Kinect and Opentrack solved the problem but the problem with kinect is that you use your real head and you cant place it in the calibration mat XD so it always in a wrong position.
If try to develope with SteamVR SDK using this solution, you will notice that the same thing happen when you launch your game to test, even if you remap the controllers. Not sure if it matters or not but in the SDK you can choose which one will be game pad and which one will be HMD, when you hit play that is ignore and you need to remap during game play.

Hello. Please post in the google group first before posting in GitHub, especially for issues related to the SteamVR driver and 3rd party programs.
https://groups.google.com/forum/#!forum/psmoveservice


Pull request validator rework

Make ipaddrport supprt ipv4 and ipv6, add ipv4addrport and add option ipv4only for hostport which allows hostport to accept only ipv4 IP addresses when an IP address rather than host name is specified (for apps and protocols that don't support ipv6).
Also update affected consumers of host and hostport datatype.

merged, thanks!


ipython-kernel: use cryptonite instead of SHA

This might make a difference to #865, and cryptonite is elsewhere in the dependency tree for IHaskell anyway!

Hmm, it seems I was mistaken about cryptonite (or at least I can't find where it is depended on). I know we've had issues with it in the past so I assumed it was a dependency: #636, #659, #655

Okay, it looks like tls is what depends on cryptonite.


[Frontend] Home page of the Community-Website

In this issue, we want to revamp the home page having better header, footer and home page content by following this color pallet.
Deliverables:

[ ] Better Header { Just keep home, Contact Us, Broadcasts, Resource Share, About Us, FAQ  in the header section along with admin? on the right side of the header }

[ ] Better Footer { Use better logo for social media handles, add a line All right reserved by @Hitk-Tech-Communty  or Made with :heart: by HITK Tech Community in the footer.}

[ ] Better Home page Content { This page should include a brief about who are are (put dummy text as of now) along with our motive(put dummy text as of now) in most attractive way possible }

I want to work on this one. Can you assign it to me?

I want to work on this one. Can you assign it to me?

Hey @reetpriye please go ahead with this issue.


Support optional path parameters

See example here.

See example here.

@jordanekay did you want to merge this PR after the feedback? Otherwise I've got some time this weekend an can get this functionality merged in 😄

Let’s close this—I’ll let you take it from here. Thank you!

Cool - also FWIW the Alamofire dependency is dropped on main as well; will cut a release with that and the optional path parameters soon.
Please do let me know if there's any more features needed / blockers for using this library for parts of your project!


SplitLayout: Make secondary pane optional

With this PR the secondary part is optional.
If it is put as undefined, it will be removed. The splitter / handle will not be visible anymore.
I adjusted the example.
Here is a short demo on how it works with both situations
https://github.com/grafana/scenes/assets/255404/a39f59d7-a342-4e08-93cd-c3f1f1865041

@kaydelaney
If any changes to required on the modifications done, please let me know, then I can change the code in the PR.

@cbos  - sorry for a long wait. I've pushed a cosmetic change so that the resize handle and secondary panes are not rendered when there is no secondary pane at all.

@dprokop
Thanks for finishing the pull request!


Some way to conditionally apply directives

So I encountered this issue while using AngularUI Dropdown Toggle: I was trying to set a directive (dropdown-toggle) using ngClass so it can be applied conditionally, but turned out not working because ngClass adds the classnames into the DOM only after the HTML compilation is over.
Perhaps we can create another directive which works similar with ngClass but is being run before the compilation phase, so we have an easy way to conditionally apply directives?

very informative and thanks for sharing
Angularjs training in chennai


Added support for Edgeware.

Hi MEW team! Happy user of your products for several years so very excited about this.

Many of our users also belong to the ETH ecosystem, and we are compatible with the EVM so it is great to see an extension that can manage all tokens.
The addition is very minimal and follows the one of Kusama.
I think ledger feature won't be supported since our ledger app format is old. But I cannot test it. Transfers, information, addresses, all seem to work out of the box. I noticed there is some lag (sometimes even a couple of minutes) before the transactions show of in the activity, but it seems to be the same in subscan so not related to the extension.

@remzrn Thanks for the PR! As it stands this looks good to me. If you support the EVM similar to how Moonbeam and Astar do you may want to check out how they are implemented and make another PR for the "EVM version" of the chain.
Also do make a PR to this repo to add some dapps available on Edgeware.

@remzrn Thanks for the PR! As it stands this looks good to me. If you support the EVM similar to how Moonbeam and Astar do you may want to check out how they are implemented and make another PR for the "EVM version" of the chain.
Also do make a PR to this repo to add some dapps available on Edgeware.

Thank you so much @brettkolodny. I'll try to submit the PR for the EdgeEVM within a week.


Adding swagger for Microsoft.OperationsManagement RP

Also ran the Autorest tool to make sure that there is not Error.
PR information

[ x] The title of the PR is clear and informative.
[ x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x ] I have read the contribution guidelines.
[x ] My spec meets the review criteria:

[ x] The spec conforms to the Swagger 2.0 specification.
[x ] The spec follows the guidelines described in the Swagger checklist document.
[ x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

@dashimi16,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

There are some linter warnings, see the linter log.  We wouldn't hold up this PR due to them however you might want to fix them now so you don't accrue clean-up debt.

This change has introduced new errors, see the following.
https://travis-ci.org/Azure/azure-rest-api-specs/jobs/270116255
https://travis-ci.org/Azure/azure-rest-api-specs/jobs/270116256

I would recommend running the validation tools locally before committing changes, see this link for instructions.

Getting closer.  There are some semantic validation errors that must be fixed, see log https://travis-ci.org/Azure/azure-rest-api-specs/jobs/270509581

This looks good, once we have ARM sign-off from @ravbhatnagar we can merge.

@ravbhatnagar The missing GET has been added.  Based on @dashimi16 answers to your other questions do you approve the change?

@ravbhatnagar I'd like to merge this today however you're review is still marked as "changes requested".

@ravbhatnagar, can you please let us know? Thanks.

Signing off!

No modification for AutorestCI/azure-sdk-for-python

No modification for AutorestCI/azure-sdk-for-ruby

No modification for AutorestCI/azure-sdk-for-node


[Bug Report] Fabric Server Crash Minecraft 1.18.2

Describe the bug
Server crash
To Reproduce
Steps to reproduce the behavior:

Start server

Expected behavior
I want server to start and plugin to work)
Screenshots
If applicable, add screenshots to help explain your problem.
Server Software and Version:

Vendor: Fabric
Minecraft Version: 1.18.2
Output of /version command: fabric don`t have this command

Additional context
Fabric API v0.47.10
latest.log

Having same issue as listed.

You'll need to use the snapshot builds for 1.18.2 until a new release is made https://jenkins.jpenilla.xyz/job/MiniMOTD/


Implement Vision Grid Transformer for Document Layout Analysis

Implement Vision Grid Transformer for Document Layout Analysis
AlibabaResearch recently  published a new model for Document Layout Analysis which sets a new benchmark in the task of Document Layout Analysis.
Introduction - To fully leverage multi-modal information and exploit pre-training techniques to learn better representation for DLA, in this paper, we present VGT, a two-stream Vision Grid Transformer, in which Grid Transformer (GiT) is proposed and pre-trained for 2D token-level and segment-level semantic understanding
https://arxiv.org/abs/2308.14978
Effect on LLM usage - VGT can dissect the page into different portions (headers, subheaders, titles, etc.) which can then be OCRed and passed to an LLM for RAG.

I'm working on a similar project and am excited to see that you have already started. I'm curious about your progress. If needed, I can offer my help.

That would be great, I have started looking at Advanced Literate Machinery.
I was not able to obtain the weight to test the model, but it does looks very good.


CoApp

I try to create a NuGet package using CoApp and *.autopkg
I know it is a dead project, but it is the best alternative for C++ projects (that I know of)
I have installed CoApp on the agent machine, and try to use a PowerShell script to genereate the NuGet.
If I run the PowerShell script localy on the agent machine, everything works. No errors and the nuget package is generated. But if I let TFS 2015 Build run the PowerShell script, I get a error that it do not found Write-NuGetPackage (the cmdlet from CoApp).
How do I do to use the cmdlet installed on the agent machine? Or is it any beter way to generate a NuGet package for C++?

Where is CoApp installing Write-NuGetPackage?  My guess is it is wiring it up through your profile load or user account as you logon the machine, but you are running the Build Agent as a service (likely Network Service) so you aren't going to see your user hooks.
Since the question is focused on getting a powershell task to call powershell cmdlets installed on a machine, and not something about packaging specifically, I'm clearing the Area:Packaging label also.

It is installed in C:\Program Files (x86)\Outercurve Foundation\Modules\CoApp
Is it any way to add the Module to TFS so that all build agents are getting the module?

Removing packaging again, not a packaging question really (about agent manipulation)

I have solved the problem.
According to the installation instructons, following must be run after installation:
[System.Environment]::SetEnvironmentVariable("PSMODULEPATH", [System.Environment]::GetEnvironmentVariable("PSMODULEPATH", "User") , "User")

In addition, the installation must be performed in a separate PowerShell instance.


fix(sys): update commit of webview-official

Update the commit of webview-official which fix #51.
Now it can compile without problem on macOS.

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] New Binding Issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes. Issue #___
[x] No

The PR fulfills these requirements:

[x] It's submitted to the dev branch and not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

I kind of forget how covector works btw. Should I setup anything?

I kind of forget how covector works btw. Should I setup anything?


chore(deps): bump golang dependencies

Proposed changes

Checklist

[ ] Pull request is created against the dev branch
[ ] All checks passed (lint, unit, build tests etc.) with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 18.666%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
375

Relevant Lines:
2009

💛  - Coveralls


Opus sound files?

I extracted a music file from Ark that claimed to be type "OPUS". However, it is not a standard Opus file -- it doesn't seem to use Ogg framing. Instead, it starts with the string "UE4OPUS", and then looks to be entirely binary from there on out.
Any ideas what this format might be / how to decode it? I imagine it's some alternative framing format for Opus packets so ought to be an easy translation, but I couldn't find any information on the internet about it.
I may be willing to contribute some code if I can figure this out.
(Note: I've never worked with the Unreal Engine, but I'm experienced with C++ and serialization formats.)

OPUS is used only for streamed sounds. And for instance, Fortnite uses pure ogg format for all sounds.
At the moment I'm not planning to spend any time on sounds formats, I have many other "next to do" things.

I've added support for UE4OPUS in vgmstream (a library/plugins to play video game music formats, see here: https://github.com/kode54/vgmstream).
It works fine for ARK and Fortnite, but it only plays raw files extracted from the .uasset/ubulk/uexp. By "raw files" I mean the data starting from "UE4OPUS" string, de-chunked from ubulk+uexp (using the uexp table to remove blank padding) if needed. Here are a couple of examples: https://www.sendspace.com/file/t6o1n8
While I can extract them with some effort, I hope you can add raw .ue4opus extraction support since it'd be more future-proof and (I assume) easier for you.
(at least I wasn't able to extract them with the current version, I always got an error).

I could explain later how if you need it but it's a bit long, as you need to create valid Ogg pages which is kinda involved.
Even if you add a conversion option, I really, really hope you also add raw extraction, because I like to save the original UE4OPUS files, and avoid converted data whenever possible.
(IMO a UE4OPUS to OPUS converter would work better as an external program).

I've added possibility to save streamed sounds in internal engine's format. For Fortnite, you'll get ".ue4opus" files as output. I didn't test results (I don't know how), but I think it should work.
Anyway, it would be much better if I'd be able to convert them into "standard" OPUS files.

Thanks for adding the option.

As for converting UE4OPUS to Ogg Opus, here is a quick explanation. I may add more detail later but Ogg is stupidly complex and it's very time consuming.
If you add this conversion I hope you still keep the option to extract it raw, though.
UE4OPUS starts with a simple header: string, sample rate, total samples, channels (nothing in the UE4 classes is needed). Then Opus packets (packet size + packet data) until file end.
You need to take the header info and recreate a OggS "opus header" page, make a fake OggS "comment" page, then put Opus packets inside many OggS "data" pages.
Ogg is divided into "OggS pages", that can contain certain fixed fields (page type, position/sequence, stream id, crc) and a variable "lacing values" table (that has the number of packets and their sizes, in a weird format).
Inside a "data page" you must fill the lacing table and put N opus packets inside (without UE4OPUS packet size). N can be any number but not too much nor too little, as it'd be inefficient.
Ogg pages need position, which means number of samples up to that point, you must manually extract and count samples in opus packets (this can be found in the opus src), though maybe all packets share the same number of samples (not sure). Ogg pages are also CRC-protected.
Note that in the "opus header" you should also put 'encoder delay', an extra field not in UE4OPUS that is equal to (samples in first packet / 8).
Anyway, getting any one field wrong may result in an invalid Ogg, so as you can see it's all quite complex.
The Ogg page formats are explained in Xiph's docs, though it's a bit hard to follow, you may have to check their sources.

Thank you for long and very detailed explanation. Hopefully I'll return to "true OPUS" support one day. However, a question: what's the point for supporting "UE4OPUS" if I'll support "OPUS"?

Main reason is data preservation, I prefer to store original files only.
If I have the original data, I can play it as-is (w/ vgmstream) or convert to Ogg Opus if I wanted. As you can see Ogg is rather complex, so if umodel somehow gets the Ogg conversion wrong one could extract the original UE4OPUS and check it.
Or if Epic changes the format or adds features that Ogg Opus can't use (like looping, common in videogames formats) having the original files around would be useful for me.

Shouldn't "UE4OPUS" differ from Ogg or OPUS with just data layout? I feel you afraid of transcode data with losing quality.

I understand data quality remains the same. But as a principle, and for various tiny details, unmodified data is only one I can trust to play 100% like the game.
Generally, most "format to other format" converters don't understand videogame audio needs (like looping, encoder delay skip, proper sample counts, non-standard sample rates, seek tables, etc) and end up destroying valuable (to me) metadata. Some just want playable audio but I look for 100% accurate audio, so that metadata is worth keeping.
For example, if don't make the Ogg Opus very carefully, you may end up with a file +-0.2 secs longer than the original UE4OPUS. Which may seem small, but for accurate looping/gapless sound it's enough to feel off.
Anyway, it's your tool and you can do what you feel right. Just remember some appreciate the option to keep original data. :)

Sure :)


BME280 Pressure is not correct

In: devices/adafruit/bme280.go
Need to investigate why the pressure value seems to be wrong (which affects the altitude calculation)

Resolved


Fix Regexp expectation on request with nil base_url

At the moment, Typhoeus throws an exception when stubbing using Regexp and trying to run a Request with nil as base url.
Usually a Request with nil as url simply returns an error Response
Typhoeus::Request.new(nil).run
# => #<Typhoeus::Response:0x007f2c35d3d728 ...>

Creating a Regexp stub and running a Request with nil:
Typhoeus.stub(/.*/).and_return(Typhoeus::Response.new(code: 200, body: "Typhoeus rocks"))
Typhoeus::Request.new(nil).run
# NoMethodError: undefined method `match' for nil:NilClass

Thank you very much for your work @fnordfish!

You are very welcome!


upgrade to cassandra 3.4

Could you please upgrade to cassandra 3.4?
Tried to fork your repo and do it myself, but the built image won't start and doesn't log anything.
Tried with 3.3 and 3.4

@tobert Are there any plans to push another tag with Cassandra 3.x?

+1 & maybe to the latest version...


Stumbling blocks for Aqueduct data object authors

A list of common stumbling blocks for data object authors, in case anyone wants to brainstorm ideas for improving the ergonomics of the framework:

Failures during a data object first init permanently wedges the document.
Hangs/races when accessing map keys are difficult to debug.
Creating data object/DDSes with duplicate ids results in corruption.
Data object registration is easy to forget, crashes the runtime, and is "non-local" to the component.

@skylerjokiel - is this something we need to account for in 1.0 for 3rd parties?


Options and flags for setting file permissions

Added more shorthands to work with file permissions of the transfered files:

chmod(value): --chmod=VALUE (accumulative)
hardLinks(): -H
perms(): -p
executability(): -E
owner(): -o
group(): -g
acls(): -A
xattrs(): -X
devices(): --devices
specials: --specials
times(): -t

that's awesome, can we also add to the readme all the different option flags?
great job!!!

All the available options are already in the README, look under option shorthands. All rsync options can be set using the set method so this list is not limiting you from using any rsync option (except maybe the accumulative ones).
If you want to know more on the rsync options I suggest you look them up in the rsync man pages, or use this online version.

great tx!

Checked the readme but I don't see the ones that you added today, like chmod()

@born2net the changes were not merged into the master branch yet. Have another look 😄

tx!


link: bind_at_load is deprecated on macOS

What version of Go are you using (go version)?

1.21.1

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
macOS ARM64
go env Output
GO111MODULE=''
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/stan/Library/Caches/go-build'
GOENV='/Users/stan/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/stan/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/stan/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/opt/homebrew/Cellar/go/1.21.1/libexec'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/opt/homebrew/Cellar/go/1.21.1/libexec/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.21.1'
GCCGO='gccgo'
AR='ar'
CC='cc'
CXX='c++'
CGO_ENABLED='1'
GOMOD='/dev/null'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/6p/9w5bw1p13v12yw5x27j5hfnc0000gn/T/go-build490684525=/tmp/go-build -gno-record-gcc-switches -fno-common'

ld: warning: -bind_at_load is deprecated on macOS warning is issued when building the project on macOS.
Quick search shows that -bind_at_load parameter is passed here: https://github.com/golang/go/blob/master/src/cmd/link/internal/ld/lib.go#L1421
Guess it's one of the side effects of upgrading to XCode 15 as it outputs more warnings than its predecessor.

already fixed here: https://github.com/golang/go/commit/040dbf9c181a0e3ea9f7bd3ebe3f75acdc878aaf


No Module named "dotenv" / No Module named "openai"

Problem
When trying to run the code with poetry run task start after running poetry install, I received the following errors

No Module named "dotenv"
No Module named "openai"

Expected Outcome
The Discord bot should have been successfully launched as I installed all dependencies using the poetry install command.
How did I fix it
I ran the following commands to install the packages

python -m pip install python-dotenv
python -m pip install openai

What can we do next
OpenAi was added to the dependency file pyproject.yaml, but it doesn't work correctly. Need to fix/find out why it wasn't installed correctly when I ran poetry install.
python-dotenv is missing from the file, so it should be added.
Image 1:

Image 2

Seems to be a MINGW or Windows issue. The setup currently works fine with Mac OS and Linux(Debian)


feat(@kampus-apps/pano): initial role system

Description
Initial version for role system

I'll close this for now, let's have a brief discussion again to start planning and implementation. @usirin


Camara does not work after remove-reCreate the canvas object

Hello, I've tried to use your code in a offline webapp. The camera is used in a form where the user can enter and exit as many times as he need. When the user enter in the form, the application inject the html code of the new form and init the decoder object. It runs very well the first time! However, when the user come back (the application clean the DOM) and enter again in the form, where the camera is used, the canvas object does show nothing. However, my phone notify me that Chrome is using the camera.
I've tried to stop the camera before exit from the form, but it did not work... I attach a code where you can test the issue. I have use the "simple_jquery.html" file and modify some things. So you can test easily.
When you open the example you can follow next steps:

Push New. It will create a canvas HTML element, init the decoder and show the camera.
Push Delete. It will stop the camera and delete the canvas HTML element.
Push New again:
· What it should happen: the application shoud create a new canvas HTML element, init the decoder and show the camera.
· What happen: the application create a new canvas HTML element, init the decoder BUT DOES NOT SHOW THE CAMERA. However if you test it using a mobile, it will notify you that the camera is being used!

Thanks in advance!!
simple_jquery.html.TXT

Hello,
Add destroy function to webcodecamjquery.js:
        isInitialized: function() {
            return initialized;
        } ,
        destroy: function() {
            stop();
            $(video).off();
            sucessLocalDecode = false,
            localImage = false,
            isStreaming = false,
            delayBool = false,
            initialized = false,
            localStream = null;
        }
    });
    $.fn[pluginName] = function(options) {

Usage:
$("#cmdDelete").click(function(ev){
	objDecoder.destroy();
	$("#canvas2").detach();
});

Regards

Hello, thanks very much for your help. I've just tried that and it worked fine!
Regards!

I have implemented Destroy in WebCodeCamJs.js, but the memory in the browser is getting bigger, every time I open the webcam, because DecoderWorker.js more and more in my browser memory integrates with angular 5
how do i fix it ?, thanks
sorry English is not your native language, if there are errors in the question

I think we need to terminate the decode worker as well.
added this to the previous solution
destroy: function() {             stop();             $(video).off();             DecodeWorker.terminate();             sucessLocalDecode = false,                 localImage = false,                 isStreaming = false,                 delayBool = false,                 initialized = false,                 localStream = null;         },


Add classifier for CUDA 11.6

Request to add a new Trove classifier.
The name of the classifier(s) you would like to add:

Environment :: GPU :: NVIDIA CUDA :: 11.6

Why do you want to add this classifier?
CUDA 11.6 was out recently and our product releases cannot be uploaded to PyPI without this classifier added.
cc: @jakirkham

cc: @m3vaz

Thanks Leo! 😄
LGTM FWIW

Friendly ping @di 🙂

Thanks, Dustin!

Hi @di, is it normal that this new classifier does not appear on the list https://pypi.org/classifiers/?

@leofang We need to merge https://github.com/pypa/warehouse/pull/10994 before this is available.


PUT /api/triggeredjobs/{jobname} removes existing files

Reviewing the documentation here https://github.com/projectkudu/kudu/wiki/WebJobs-API
These two should be equivalent
PUT /api/zip/site/wwwroot/App_Data/jobs/triggered/{job name}/
which uses this ZIP API https://github.com/projectkudu/kudu/wiki/REST-API
and
PUT /api/triggeredwebjobs/{job name}
What I've noticed is that the ZIPAPI will PUT the contents on top of the webjob, but will not delete any existing files. However, the /api/triggeredwebjobs/ API will overwrite the entire directory and delete existing files.
For context: I was trying to figure out how to update only a single set of files in a webjob, such as adding newer versions of DLL's to an existing web job without updating its app.config.
The workaround for me is to use the ZIP API, but I would have expected these APIs to behave the same given the same use of the PUT verb.

Indeed, the behavior is different. The code in the /api/triggeredwebjobs/ does attempt to delete existing files.
I added a note in the wiki.


Training accuracy falling down after some epoches since upgrading to 0.8.0rc0

I'm trying to use a simple network to classify some 64*64 images. in v0.7 it worked fine and consistent. The training phase was showing a consistent  average growth in accuracy. But since I'm trying to use 0.8, the accuracy fall down occasionally from ~0.9 to ~0!
I'm using cpu  version for python 2.7
is it a bug in TF or I'm doing something wrong?
#############################################
Environment info
Operating System: Ubuntu 14.04
I've installed pip with the following command : sudo apt-get install python-pip python-dev

The output from python -c "import tensorflow; print(tensorflow.version)".
0.8.0rc0
#############################################

my learning code is as below:
sess = tf.InteractiveSession()
x = tf.placeholder(tf.float32, shape=[None, 64,64])
y_ = tf.placeholder(tf.float32, shape=[None, 31])
input = tf.reshape(x,shape=[-1,6464])
w1 = tf.Variable(tf.random_uniform([6464,128],minval=-0.1,maxval=0.1),dtype=tf.float32)
b1 = tf.Variable(tf.random_uniform([128],minval=-0.1,maxval=0.1),dtype=tf.float32);
y1_ = tf.nn.softmax(tf.matmul(input, w1) + b1)
keep_prob = tf.placeholder(tf.float32)
y1 = tf.nn.dropout(y1_, keep_prob)
w2 = tf.Variable(tf.random_uniform([128,31],minval=-0.1,maxval=0.1),dtype=tf.float32)
b2 = tf.Variable(tf.random_uniform([31],minval=-0.1,maxval=0.1),dtype=tf.float32);
y2 = tf.nn.softmax(tf.matmul(y1, w2) + b2)
cross_entropy = -tf.reduce_sum(y_*tf.log(y2))
learn_rate = tf.placeholder(tf.float32)
train_step = tf.train.AdamOptimizer(learn_rate).minimize(cross_entropy)
correct_prediction = tf.equal(tf.argmax(y2,1), tf.argmax(y_,1))
accuracy = tf.reduce_mean(tf.cast(correct_prediction, tf.float32))
sess.run(tf.initialize_all_variables())
d = Dataset()
dat = d.load()
for i in range(9000):
if i%200==0:
a = accuracy.eval(feed_dict={x: dat.data, y_: dat.label,keep_prob: 1.0,learn_rate:1e-2})
print("step %d, training accuracy %g" % (i, a))
train_step.run(feed_dict={x: dat.data, y_: dat.label,keep_prob: 0.8 ,learn_rate:1e-2})
for i in range(5000):
if i%100==0:
a = accuracy.eval(feed_dict={x: dat.data, y_: dat.label,keep_prob: 1.0,learn_rate:1e-3})
print("step %d, training accuracy %g" % (i, a))
train_step.run(feed_dict={x: dat.data, y_: dat.label,keep_prob: 0.8,learn_rate:1e-3 })
==========================
some line of the output is as below:
step 7400, training accuracy 0.870539
step 7600, training accuracy 0.875406
step 7800, training accuracy 0.877677
step 8000, training accuracy 0.882868
step 8200, training accuracy 0.884491
step 8400, training accuracy 0.887735
step 8600, training accuracy 0.888709
step 8800, training accuracy 0.893251
step 0, training accuracy 0.894549
step 100, training accuracy 0.895198
step 200, training accuracy 0.894873
step 300, training accuracy 0.895198
step 400, training accuracy 0.894873
step 500, training accuracy 0.895198
step 600, training accuracy 0.895198
step 700, training accuracy 0.895522
step 800, training accuracy 0.895522
step 900, training accuracy 0.895847
step 1000, training accuracy 0.896171
step 1100, training accuracy 0.896171
step 1200, training accuracy 0.896496
step 1300, training accuracy 0.89682
step 1400, training accuracy 0
step 1500, training accuracy 0
step 1600, training accuracy 0
step 1700, training accuracy 0

Can you try reducing your learning rate? If your original optimization was close to being numerically unstable, then a small numerical change in the upgrade could cause training to diverge consistently.

The case is not about divergence of the learning accuracy. the first loop is using 0.01 as learning rate and the second one is using 0.001. In both cases it's not divergent (as you can see the accuracy is increasing). My problem is the accuracy of learning is falling to zero! And I'm experiencing this since I'm using 0.8rc0 in 0.7 I had no such problem.

I guess the issue is whether this is due to a bug, or part of normal training procedure. Accuracy suddenly falling to zero is a common occurrence in neural network training. Typically it happens when your neural network weights overflow or underflow. Can you see what the norm of your parameters looks like over time? Also, another suggestion is to try retuning your hyper-parameters? (ie, trying different learning rate, training algorithm, etc) There might be some numeric difference between 0.7 and 0.8, but it's not clear which one is the "correct" one.

I've printed the value of W matrices in case the accuracy falls down And surprisingly all values was nan!
So I think it's a bug and is not about my code!

BTW cross entropy is nan and it makes other elements nan

It's common to get NaN during training. For instance you have an expression y_*tf.log(y2). If y2 is 0 and y_ is 0, then tf_log(y2) is -infinity, and 0*(-infinity) is NaN.

Yeah I got that. Thanks for your help @yaroslavvb . Since I've find out that cross entropy is causing the NaN problem I'm trying to use other cross entropies. but sadly non is as fast as this.

One workaround is to do tf.log(y2+1e-8) instead of tf.log(y2)

@yaroslavvb This is awesome, thank you!


Large diffs do not show

Describe the bug
vitest does not render large object diffs. The config options outputDiffLines: Infinity, and outputTruncateLength: Infinity seem to have no effect, it shows this unhelpful message with no diff visible at all:
AssertionError: expected [ Array(500) ] to deeply equal [ Array(500) ]

Reproduction
https://github.com/silverwind/vitest-diff
Clone, npm install and npx vitest yields this unhelpful output:
 RUN  v0.25.8 /Users/silverwind/git/vitest-diff

 ❯ test.js (1)
   × diff

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Failed Tests 1 ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯

 FAIL  test.js > diff
AssertionError: expected [ Array(500) ] to deeply equal [ Array(500) ]
 ❯ test.js:6:13
      4|   const a = JSON.parse(readFileSync(new URL("a.json", import.meta.url), "utf8"));
      5|   const b = JSON.parse(readFileSync(new URL("b.json", import.meta.url), "utf8"));
      6|   expect(a).toEqual(b);
       |             ^
      7| });
      8|

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/1]⎯

 Test Files  1 failed (1)
      Tests  1 failed (1)
   Start at  13:09:01
   Duration  1.28s (transform 339ms, setup 0ms, collect 3ms, tests 56ms)

System Info
System:
    OS: macOS 13.0.1
    CPU: (8) x64 Intel(R) Core(TM) i7-1060NG7 CPU @ 1.20GHz
    Memory: 153.65 MB / 16.00 GB
    Shell: 5.9 - /usr/local/bin/zsh
  Binaries:
    Node: 19.2.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 8.19.3 - /usr/local/bin/npm
  npmPackages:
    vitest: 0.25.8 => 0.25.8

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Thanks for the new options. I updated the repro and it does at least now render part of the diff, although it is pretty slow, took around 20 seconds to produce the output on my machine.
Still, it's only rendering part of the diff, and it's hiding 4951 lines, is it expected?
  - Expected  - 1500
  + Received  + 1500

      Object {
        "address": Object {
  -       "city": "Sandoná",
  -       "street": "35539 Nolan Lights",
  +       "city": "Vatican City",
  +       "street": "95452 Davion Grove",
        },
  -     "full name": "Hilda Labadie",
  +     "full name": "Corey Maggio",
      },
      Object {
        "address": Object {
  -       "city": "Benkovac",
  -       "street": "1515 Dietrich Lights",
  +       "city": "Thaton",
  +       "street": "8704 Trey Motorway",
        },
  -     "full name": "Stacy Lakin",
  +     "full name": "Kelvin Jaskolski",
      },
      Object {
        "address": Object {
  -       "city": "Pizitou",
  -       "street": "84570 Jay Fall",
  +       "city": "Iriba",
  +       "street": "89217 Hyatt Dale",
        },
  -     "full name": "Lana Klocko",
  +     "full name": "Connie Considine",
      },
      Object {
        "address": Object {
  -       "city": "Grenville",
  -       "street": "73389 Stamm Land",
  +       "city": "As Samawah",
  +       "street": "2630 Sincere Avenue",
        },
  -     "full name": "Louis Jones",
  +     "full name": "Tabitha Nikolaus",
      },
      Object {
        "address": Object {
  -       "city": "Quruqsoy",
  -       "street": "674 Kreiger Avenue",
  +       "city": "La Fermina",
  +       "street": "760 Gillian Prairie",
        },
  -     "full name": "Gerard Mills",
  +     "full name": "Claire D'Amore",
  ... 4951 more lines

Yes, look into the new options to display more.

We are comparing objects as strings, and it is very slow, yes. This is why it’s nor enabled by default.

The new options are already all set to Infinity, so that seems to not be enough to show all lines.
Regarding text diff performance, I had good experience with jsdiff, maybe worth looking into.

The new options are already all set to Infinity, so that seems to not be enough to show all lines.

I forgot to pass them down 🤦🏻

#2828 removes these options - please, let me know if it works for you

Does it mean that diffs will not be cut down by any arbitrary limits any more and will always show in full? If so I'm happy to accept that, as it reduces my config.
Imho, the default should be unopinionated and just show everything. If users have specific cases where diff gets too large, they can limit it themselves.


add the dirty harry sicilian

Adding the dirty harry variation in the sicilian defense, most famous because of the following chessable course: https://www.chessable.com/the-dirty-harry-sicilian/course/47232/

I think this is more the marketing name of a course, then the name of the opening. I did not find any other autorative sources calling it the dirty harry. In fact, even the owner of chessable, chesscom, just lists the line as " B33 Sicilian Defense: Open " in their analysis board.
If I search for the opening in google, the summary is essentially :  4 yt videos ( 3 of which are from the author of the course ) , the chessable course, this PR and other non-chess related results.
This does not mean it's wrong to add it, just that IDK where the " line " goes. But I would prefer to at least one more publication using the name, before it's added as an opening name. Let me know if there are some notable sources that I am not aware of.


Added legacy preferGlobal flag

This is an attempt to make this show npm i -g now instead:

From the docs:

This option used to trigger an npm warning, but it will no longer warn. It is purely there for informational purposes.

Codecov Report

Merging #1722 into canary will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           canary    #1722   +/-   ##
=======================================
  Coverage   70.17%   70.17%
=======================================
  Files          12       12
  Lines         342      342
=======================================
  Hits          240      240
  Misses        102      102

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 17362c0...b69a024. Read the comment docs.


Add support for visualizing electron density maps

Hi Marco,
Would be great if PV could show electron density maps (as some kind of mesh?). Having such a visualization would make it much more clear how well e.g. ligands fit the experimental data.
Thanks,
Andreas

Hi  Andreas,
I've added support for rendering of volume data in a separate branch a while ago. It's not much more than an implementation of the marching cube algorithm that operates on a scalar 3d volume. It doesn't support non-rectangular coordinate systems, nor does it support anything related to symmetry. I'd assume that these features would need to be added to be useful to you?
There are a few other things on my todo list that I'd like to tackle first, so I'm not sure when I have time to work on it.
Best,
Marco

sure, no rush with this. I will prob have some data for this later this year and can ping you again then. Btw. just upgraded to latest pv and looks great!

Hi Marco,
Just another voice for volumetric data. I'm not using electron density maps though. I've got PMF maps I'd like to display, DX volumetric data format. Currently they're all rectangular coordinate systems and nothing symmetry based, so the current implementation you metion above would work for me.
This is good stuff! Keep on it!
Thanks,
Tom.

Hi Marco,
Any progress on the implimentation of the volume data? I'm thinking about developing some interactive figures for a paper/thesis that would contain some protein stuctures and some volumetric data. I'm using the openDX format, but that can change. I don't require any symmetry elements and all the data is retangular, no tricky coordinate systems.
I see you said you had implemented a version of the marching cube algorithim in an earlier, seperate branch. I'd be willing to give it a test if you'd like.
Tom.

Hi Tom,
I'll have to check on my old laptop, I don't remember in what state the branch is. I'll let you know as soon as possible if I think the branch is in any shape to be taken out for a test-drive.
Best,
Marco

Awesome, Thanks! I've been playing around with the viewpointChanged event and syncronsed viewers to allow comparison of structures. It's looking real nice!
Tom.


Upgrade Linq2db

Creating this issue to follow up the problems with sqlite in upgrading linq2db.
All information is in the comments from https://github.com/akkadotnet/Akka.Persistence.Sql/pull/163#issuecomment-1475507465
We want to upgrade linq2db, but can't get past 3.5.0 because then sqlite explodes with some DateTime related issues.

Hello, I would like to know when the 'Linq2db' with dependencies to Akka.Net v1.5.1 will be released. We would like to used it with Sql Server.

Hello, I would like to know when the 'Linq2db' with dependencies to Akka.Net v1.5.1 will be released. We would like to use it with Sql Server.

@Arkatufus is working on a project plan today that should address this: https://github.com/orgs/akkadotnet/projects/9 <-- first beta release


Don't know how to generate gelmap.npz file

Hi all,
Thanks for sharing the simulator.
I am trying to calibrate the simulator with another sensor (the DIGIT), but it seems we require a gelmap.npz file to reproduce the shape of the gel. In this repo there's the file related to the gelsight but I was wondering how can I create this kind of file for my sensor.
Thanks

Running into the same issue, any updates?

Hi, thanks for bringing this up! If you can find a DIGIT sensor soft elastomer's mesh model, you can output it as a point cloud or a depth map from a virtual camera. I believe the gelmap5.npy is a depth map.

Hi @gabrielecaddeo ,
I am also interested in including Digit simulation in Taxim. Did you manage to do it already?
Thanks in advance!

@Jcastanyo @gabrielecaddeo @vocdex
Do you have any updates about the gelmap of the DIGIT sensor?


Add support for filtering tags in k8s workloads.

What does this PR do?
Just add the anotation tags.datadog.com/disable: "comma,separated,tags,to,be,disabled"
Add support in taglist to remove tags.
The idea is to use this before calling the tags.Compute() method The current implementation is done for k8s workloads, but it could be expanded to other places.
(ps: I am still learning go, please forgive any mistakes. )
Motivation
Wanted to exclude tags from the kubernetes workload for specific pods, this pr allows to do that during runtime.
Example: pod_name is high cardinality, I want it enabled for some services, but not for others.
Additional Notes
It reuses an existing annotation and is compatible with previous use cases of the annotation (only kube_service was supported before my change.)
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes
Tested using unit testing and planning to rollout the agent with the changes in a subset of our infrastructure.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Can someone provide a feedback on this PR ?

@lmello maybe you need to contact support to get them to send engineers here to look at it? I'm interested in your feature being added, so I hope they'll expedite the process.

upport to get them to send engineers here to look at it? I'm interested in your feature being added, so I hope they'll expedite the process.

@2rs2ts I gave up, contacted support, TAM and it didn't move up. We end up building our own pipeline before shipping metrics and logs to datadog using vector, to sanitize and clean tags.

I gave up, contacted support, TAM and it didn't move up. We end up building our own pipeline before shipping metrics and logs to datadog using vector, to sanitize and clean tags.

Boy is that a common story. Sorry to hear it, but I'm not surprised; it feels like engineering must be a skeleton crew over there


The implementation of coverage mechanism seems to be wrong.

The author of pointer-generator propose a method called coverage mechanism.
Coverage vector, which is the sum of attention distributions over all previous decoder timesteps, but the coverage vector in this repository seems to sum the attention of encoder!
Please help me find the correct method to implementation the mechanism or tell me where is my fault.

According to the paper "Get-to-the-Point", eq10, coverage vector is the sum of previous encoder attentions.

According to the paper "Get-to-the-Point", eq10, coverage vector is the sum of previous encoder attentions.

Thanks, you're right.
I want to ask more about the mechanism. When the amount of data is very small (I have 5000 pairs data which consists of content and headline), the model in this repository still likes to repeat itself.
My paremeters settings (in params.py) about the coverage mechanism:
enc_attn_cover = True
cover_func= 'max'
cover_loss: float = 1
show_cover_loss = False
I would appreciate it if you could help me !

Sorry, I have tried some experiments using this repo but I can not reproduce the result on CNNDM, So, maybe there are still some errors in the repo. Additionally, Pointer-generator uses "sum" as cover_func.

Sorry, I have tried some experiments using this repo but I can not reproduce the result on CNNDM, So, maybe there are still some errors in the repo. Additionally, Pointer-generator uses "sum" as cover_func.

Thanks for your help.


⚠ Melody has degraded performance

In c01ca56, Melody (https://melody.triza.dev) experienced degraded performance:

HTTP code: 404
Response time: 5743 ms

Resolved: Melody performance has improved in dc60679.


[DRAFT / DISCUSSION] Depend between entry points

Type of Issue
[ ] Bug Report
[x] Feature Request

Description

I wonder how to handle the dependencies here. I have a @org/components root package, and I would like to have @org/components/shared for shared utilities and stuff.
The root package can import stuff from shared without generating duplicated compile code?
Is there a way to set up this kind of "dependency"?

Motivation
Allow dependencies between entry points of the same package.
Expected Behaviour
By TypeScript AST Traversal, detect whether @org/components depends on another entry point within the same package, e.g. @org/components/shared.
Then create a build graph, i.e. @org/components/shared is required to be built before @org/components.
Then, we run the transformation process for packages w/o inter-entry-point deps first. When running the transformation process for dependent packages, configure tsc compilerOptions.paths to point to a FESM bundle of its dependency.
Version Information
ng-packagr: 2.0.0-rc.x

What happens if we detect a cyclic dependency?
My view: error and non-zero exit code.

I realize this discussion was just started 6 hours ago, but any ideas as to when this would be released? I could absolutely use it now! Also, sorry for the meta-question, is there a better place to ask something like this?

No, no idea. No, there is no good place for such questions.
Continue on the topic!
Cheers, David

On 8. Dec 2017, at 02:24, Daniel<EMAIL_ADDRESS>wrote:
I realize this discussion was just started 6 hours ago, but any ideas as to when this would be released? I could absolutely use it now! Also, sorry for the meta-question, is there a better place to ask something like this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

From my point of view, if you have have something that can be shared/re-used I'd say create a separate package rather than a separate entry point. But that's just my opinion.
And yeah with compilerOptions.paths it is possible to be done, I am doing something similar for a mono-repo

So I started a first draft in #419. Ideas and thoughts?


chore(Std/Lean/Meta): fix repeat'

Per https://github.com/leanprover/std4/commit/ee3bcfce030ae6ae41016286402947c33faa9cca#r84122462

Oops, Mario did this in the meantime.


Fix .gl and .state access

It seems like a recent change "broke" ArcGISDeckExternalRenderer, the 3D integration between deck.gl and ArcGIS.
It's tricky to see the breakage because there is still no sample app, nor test, in the codebase. Plus, 3D integration was WIP, and things did not look correct, but at least it was possible to write apps that did not crash.
I am fine with merging this; to see the positive impact of this PR I am working on some pens that I will share once ready. Once things are looking good I will work on adding automated testing and a pure-js sample app in the codebase, specific for 3D.

Thank you! Added 2D/3D test app. Feel free to merge.

Lint is still failing

Coverage increased (+2.6%) to 82.841% when pulling 20da1d036d9543c71819ddb784cf15eb2aaba0f2 on damix911:ddamico/fix-arcgis-xtren into 2982aaceb499923095c25140bcfd4b7cc95e292e on uber:master.

This is ready for merge.


Analytics (new): "Open in CellGuide" in marker genes panel

Overview
Tool: Gene Expression

Feature: Marker Genes Panel

Description: We'd like to track if Gene Expression is driving traffic to CellGuide to see the overlap in use between our products. We'd like to add analytics to the "Open in CellGuide" button in the Marker Genes panel in Gene Expression.
Expectations: Unsure, maybe 20% of users who use marker genes click on this button? It is prominently displayed but users may be familiar with the cell type.
New events to add
<suggested event name>: <information that should be stored in the payload (if applicable)>

<Description of what causes the event to fire>

Checklist

[ ] Add events to the custom goals section of both the staging and prod Plausible spaces
[ ] If rdev environment is created for this new feature, once it has been deployed, ping @ainfeld to check analytics
[ ] Once feature is deployed to Staging ping @ainfeld to check analytics again
[ ] Verify events are showing up in the Staging space in Plausible correctly
[ ] Once feature is deployed to Prod ping @ainfeld
[ ] Confirm Plausible event names in Prod look the same as Staging and are showing up in the Prod space in Plausible correctly

People to cc
PM: @signechambers1 @niknak33
DS: @ainfeld
Privacy policy: https://cellxgene.cziscience.com/privacy

Closed, completed in #5735


Touch screen not supported for Extensions List Pane (left side)

VSCode Version: 1.3.1
OS Version: Windows 10 Enterprise

Steps to Reproduce:

Click the extensions icon, and in the left it will show all the extensions I've installed
But when I want to scroll the extensions list with my touch screen, it doesn't work. (While it works for explorer, search etc.,)

Duplicate of #9338


Fix language identification of .ts files

This is a nitpicky fix for Github misidentifying index.ts as a javascript file:
.
I tested with linguist locally (https://github.com/github/linguist#application-usage) to confirm that this change fixes the issue.

Pull Request Test Coverage Report for Build 87

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 10.56%

Totals

Change from base Build 86:
0.0%

Covered Lines:
93

Relevant Lines:
658

💛  - Coveralls

Pull Request Test Coverage Report for Build 87

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 10.56%

Totals

Change from base Build 86:
0.0%

Covered Lines:
93

Relevant Lines:
658

💛  - Coveralls

Pull Request Test Coverage Report for Build 87

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 10.56%

Totals

Change from base Build 86:
0.0%

Covered Lines:
93

Relevant Lines:
658

💛  - Coveralls

Pull Request Test Coverage Report for Build 87

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 10.56%

Totals

Change from base Build 86:
0.0%

Covered Lines:
93

Relevant Lines:
658

💛  - Coveralls

Pull Request Test Coverage Report for Build 87

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 10.56%

Totals

Change from base Build 86:
0.0%

Covered Lines:
93

Relevant Lines:
658

💛  - Coveralls

Pull Request Test Coverage Report for Build 87

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 10.56%

Totals

Change from base Build 86:
0.0%

Covered Lines:
93

Relevant Lines:
658

💛  - Coveralls

Thanks!
The reason why the project was ID'd as Node was because of the shebang (#!/usr/bin/env node).


[Story] Check that Agora web app is up to date

What product(s) is this story for?
Agora
As a user, I want
No response
Description
See tasks
Cc: @sagely1
Acceptance criteria
No response
Tasks

[ ] check why agora-server appears in the code of the API client for Angular
[ ] check that the agora-app project is up to date by comparing it to model-ad-app
[x] check if MODEL-AD app has hot reloading
[ ] check that hot reloading is available when serving the app with nx serve agora-app

Anything else?
No response
Have you linked this story to a GitHub Project?

[X] I have linked this story to a GitHub Project and set its metadata.

MODEL-AD web app has live reloading and it worked today, but then it no longer work when I wanted to give a demo to Lawrence. Something to investigate. But the discussion today enabled Lawrence to test changes more rapidly.


[stable/prometheus-operator] Release "prometheus-operator" failed: rpc error: code = Canceled

Describe the bug
When I try to install prometheus operator on AKS with helm install stable/prometheus-operator --name prometheus-operator -f prometheus-operator-values.yaml I am getting this error:

prometheus-operator" failed: rpc error: code = Canceled

I checked with history:
helm history prometheus-operator -o yaml
- chart: prometheus-operator-6.3.0
  description: 'Release "prometheus-operator" failed: rpc error: code = Canceled desc
    = grpc: the client connection is closing'
  revision: 1
  status: FAILED
  updated: Tue Jul 30 12:36:52 2019

Version of Helm and Kubernetes:
Client: &version.Version{SemVer:"v2.14.3", GitCommit:"0e7f3b6637f7af8fcfddb3d2941fcc7cbebb0085", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.14.3", GitCommit:"0e7f3b6637f7af8fcfddb3d2941fcc7cbebb0085", GitTreeState:"clean"}
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.4", GitCommit:"5ca598b4ba5abb89bb773071ce452e33fb66339d", GitTreeState:"clean", BuildDate:"2018-06-06T08:13:03Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"windows/amd64"}
Server Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.7", GitCommit:"4683545293d792934a7a7e12f2cc47d20b2dd01b", GitTreeState:"clean", BuildDate:"2019-06-06T01:39:30Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}
Which chart:
[stable/prometheus-operator]
Additional Info
I am using below configurations to deploy a chart:
kubectl apply -f https://raw.githubusercontent.com/coreos/prometheus-operator/master/example/prometheus-operator-crd/alertmanager.crd.yaml
kubectl apply -f https://raw.githubusercontent.com/coreos/prometheus-operator/master/example/prometheus-operator-crd/prometheus.crd.yaml
kubectl apply -f https://raw.githubusercontent.com/coreos/prometheus-operator/master/example/prometheus-operator-crd/prometheusrule.crd.yaml
 kubectl apply -f https://raw.githubusercontent.com/coreos/prometheus-operator/master/example/prometheus-operator-crd/servicemonitor.crd.yaml

In values file: createCustomResource is set to false,

This reads like an issue with Helm rather than the chart

@rnkhouse run helm using --max and give it a smaller number like for example 5

@mariusv You want me to run this command: helm install stable/prometheus-operator --name prometheus-operator -f prometheus-operator-values.yaml --max 5

@mariusv
helm history prometheus-operator --max=4
REVISION        UPDATED                         STATUS  CHART                           DESCRIPTION
1               Tue Jul 30 12:36:52 2019        FAILED  prometheus-operator-6.3.0       Release "prometheus-operator" failed: rpc error: code = C...

Just came across this error, also linked here: https://github.com/helm/helm/issues/6130


Telegram bot c# download

I need help.
In you example on github was delete art of download file on telegram bot.
In my dream create bot, and he can claim photo users, and save that on google disk, but users can see this photo or video / download its

Please follow issue template
If you want to chat, feel free to join our discussion group.


Fixes ByteBuddy AbstractMethodError which could occur when running tests from IDE

We have now had for a time the strange behaviour that certain tests would fail when run from the IDE with the following error:
java.lang.AbstractMethodError: Receiver class co.elastic.apm.agent.bci.bytebuddy.MinimumClassFileVersionValidator does not define or inherit an implementation of the resolved method 'abstract net.bytebuddy.jar.asm.ClassVisitor wrap(net.bytebuddy.description.type.TypeDescription, net.bytebuddy.jar.asm.ClassVisitor, net.bytebuddy.implementation.Implementation$Context, net.bytebuddy.pool.TypePool, net.bytebuddy.description.field.FieldList, net.bytebuddy.description.method.MethodList, int, int)' of interface net.bytebuddy.asm.AsmVisitorWrapper.
	at net.bytebuddy.asm.AsmVisitorWrapper$Compound.wrap(AsmVisitorWrapper.java:746) ~[byte-buddy-1.14.8.jar:?]

I've been able to find the root cause of the issue and fix it with this PR.
Our MinimumClassFileVersionValidator implements AsmVisitorWrapper from byte-buddy-dep, which means it uses the unshaded org.objectweb.asm.ClassVisitor as parameter type.
The error message shows that at runtime the AsmVisitorWrapper interface is picked up from byte-buddy instead, which expects the shaded net.bytebuddy.jar.asm.ClassVisitor.
I assume that when the tests are run from maven the classpath order is slightly different, so that byte-buddy-dep wins over byte-buddy. In the IDE the order seems to be the other way around, causing the failure.
I've fixed it by excluding byte-buddy from the dependencies which pull it in transitively: assertJ and mockito.
What does this PR do?

Checklist

[x] This is something else

[ ] ~I have updated CHANGELOG.asciidoc~

I think before merging, it's worth doing a manual test using a snapshot and just run through to see all the normal things expected are displayed in the UI

The shanges should (in theory) only have altered the testing classpath. However, I agree that it makes still sense to do a manual test.


error: Error: Data still available after parsing

when
run
pool.connect();
 Error: Data still available after parsing logger.js:67 [2019-06-21T05:43:09.199Z] error: Error: Data still available after parsing     at Object.checkFinished (/home/changliang/dash/dashcore-node/node_modules/@dashevo/dashcore-p2p/lib/messages/utils.js:20:13)     at VersionMessage.setPayload (/home/changliang/dash/dashcore-node/node_modules/@dashevo/dashcore-p2p/lib/messages/commands/version.js:71:9)     at Function.exported.commands.(anonymous function).fromBuffer (/home/changliang/dash/dashcore-node/node_modules/@dashevo/dashcore-p2p/lib/messages/builder.js:110:15)     at Messages._buildFromBuffer (/home/changliang/dash/dashcore-node/node_modules/@dashevo/dashcore-p2p/lib/messages/index.js:105:41)     at Messages.parseBuffer (/home/changliang/dash/dashcore-node/node_modules/@dashevo/dashcore-p2p/lib/messages/index.js:75:15)     at Peer._readMessage (/home/changliang/dash/dashcore-node/node_modules/@dashevo/dashcore-p2p/lib/peer.js:219:31)     at Socket.<anonymous> (/home/changliang/dash/dashcore-node/node_modules/@dashevo/dashcore-p2p/lib/peer.js:167:10)
`Peer.prototype.connect = function() {
this.socket = this._getSocket();
this.status = Peer.STATUS.CONNECTING;
var self = this;
this.socket.on('connect', function() {
self.status = Peer.STATUS.CONNECTED;
self.emit('connect');
self._sendVersion();
});
this._addSocketEventHandlers();
this.socket.connect(this.port, this.host);
return this;
};`
https://github.com/bitpay/bitcore/issues/2183
https://forum.bitcore.io/t/bitcore-p2p-error-parsing/772/5
but
There's no real solution.
why?

It seems the version message you are receiving from the pool you are trying to connect to is not formatted correctly. The error starts here:
https://github.com/dashevo/dashcore-p2p/blob/master/lib/peer.js#L219. If you can print out this.dataBuffer just before, we might know what is the incorrectly formatted message you get from your peer.

Hi @changliang0931!
It seems the version message you are receiving from the pool you are trying to connect to is not formatted correctly. The error starts here:
https://github.com/dashevo/dashcore-p2p/blob/master/lib/peer.js#L219. If you can print out this.dataBuffer just before, we might know what is the incorrectly formatted message you get from this peer.

@Cofresi  Thank you for your answer.
this.dataBuffer    is  []    Empty array
but I changed the code.
https://github.com/dashevo/dashcore-p2p/blob/1c6f4af689da4dbea4865b632f0b42f0466273c2/lib/messages/builder.js#L23
to
options.protocolVersion = options.protocolVersion || 70215

this.dataBuffer is [] Empty array
empty array shouldn't cause problems as it is handled here:
https://github.com/dashevo/dashcore-p2p/blob/master/lib/messages/index.js#L44-L45

but I changed the code to options.protocolVersion = options.protocolVersion || 70215

So does it work now with your change?

So does it work now with your change?

Unfortunately, I still can't work.
my  dashcore is
dashcore-0.14.0

Hi @changliang0931 ! I finally found the problem. It was a new optional field in the version message. Fixed in https://github.com/dashevo/dashcore-p2p/pull/14
Once again thank you for reaching out and making our libraries a little better!

嗨@ changliang0931！我终于找到了问题。它是版本消息中的新可选字段。固定在＃14
再次感谢您伸出援手，让我们的图书馆更好一点！

Thank you very much for your reply.

Closing this as it has been fixed in #14. Please let us know if there are more issues with unsupported p2p messages.


[Proof] Improve arb_accumulator_proof

Motivation
The existing way of implementing Arbitrary is not very nice. Do it properly.
Have you read the Contributing Guidelines on pull requests?
Probably.
Test Plan
Existing tests.
Related PRs
None.

@bors-libra delegate+

:v: @wqfish can now approve this pull request

@bors-libra r=msmouse

:pushpin: Commit dc4ff72 has been approved by msmouse

:hourglass: Testing commit dc4ff72811aa28beee68a8f0f8a5182d9fa98ba3 with merge 2fad74b0fbfee77be68ea49745dee98617a50c39...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: msmouse
Pushing 2fad74b0fbfee77be68ea49745dee98617a50c39 to master...


Use company details from supplier for agreement details view

Trello: https://trello.com/c/zBkCpdEs/594-wrong-supplier-registered-details-showing-in-admin-framework-agreement-page

WFM - could maybe do with a test case that covers the non-companies-house path but otherwise 👍

Added

Woop woop woop


User command application

AcreTools currently assumes all applications are user command apps and includes the .dyalog file in the application build - which does not hurt. However, it should only include this file if certain functions exist.

Done


Advertisement section

There is only one image of advertisement we have to add more info that with a good UI also, So that when we refresh a new ad comes on the page.

Advertisements are temporary and static rn. We will change that later. No need to design that now.


add bubble sort main and tests

Fixes #
Review of colleague's PR #
Changes proposed in this PR:
-

Don't forget to be descriptive with your commit messages and PR. Currently your PR does not inform the reviewer what has been done, nor the issue being resolved. Also remember to use the appropriate tags.


[SPARK-38837][PYTHON] Implement dropna parameter of SeriesGroupBy.value_counts

What changes were proposed in this pull request?
Implement dropna parameter of SeriesGroupBy.value_counts to exclude counts of NaN.
It also fixes the behavior of self._dropna in the context of SeriesGroupBy.value_counts.
Why are the changes needed?
To reach parity with pandas.
Does this PR introduce any user-facing change?
Yes. dropna parameter of SeriesGroupBy.value_counts is supported.
>>> psdf = ps.DataFrame(
...             {"A": [np.nan, 2, 2, 3, 3, 3], "B": [1, 1, 2, 3, 3, np.nan]}, columns=["A", "B"]
...         )

>>> psdf.groupby("A")["B"].value_counts(dropna=False).sort_index()
A    B
2.0  1.0    1
     2.0    1
3.0  3.0    2
     NaN    1
Name: B, dtype: int64

>>> psdf.groupby("A", dropna=False)["B"].value_counts(dropna=False).sort_index()   # self.dropna=False
A    B
2.0  1.0    1
     2.0    1
3.0  3.0    2
     NaN    1
NaN  1.0    1
Name: B, dtype: int64

>>> psdf.groupby("A")["B"].value_counts(dropna=True).sort_index()
A    B
2.0  1.0    1
     2.0    1
3.0  3.0    2
Name: B, dtype: int64

How was this patch tested?
Unit tests.

Merged to master and branch-3.3.


docs(functions): fix StackOverflow url in README.md

The StackOverflow URL in our functions README.md was incorrect.

👍  nice catch!


Slug not support all allowed chars

I want to slug some string and when my string contains special chars e.g.: ? or ! this chars are removed.
Allowed chars (RFC 3986):
ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789-._~:/?#[]@!$&'()*+,;=
this chars not need to be percent-encoded, but they are removed.
How to reproduce:
echo str_slug('Some string ?');

Should be:
some-string-?

Now:
some-string

Use \Illuminate\Support\Str::ascii('Some string ?') instead of str_slug().

Our slug method intentionally drops punctuation since it doesn't add any real meaning for slugs.


feat(server): add code to 404 exception

Adding a code to this exception make it easier to handle in the exception handler.

Coverage remained the same at 95.707% when pulling b8f68eae007cdf20c06db4238627f124fe79fbce on RomainForks:develop into 2de3833fd082cfe6f464914f3dd2bea2fe5014b1 on adonisjs:develop.

Thanks 😄


Feature Suggestion: pre-fetching surrounding (extra) tiles

Hi,
How about adding an option to allow pre-fetching extra tiles?
I believe it will improve user experience with fast network, since the user will not see white space ultil loading done. With slow network, it will spoil performance though.
I'd apologize if this had been considered already.
--- leaflet-src.js      2017-05-28 10:30:08.435964020 +0900
+++ leaflet-src.js_tilemargin   2017-05-29 19:32:16.399501929 +0900
@@ -5486,9 +5486,10 @@

        _pxBoundsToTileRange: function (bounds) {
                var tileSize = this.getTileSize();
+               var margin = this.options.tileMargin;   // default: 0
                return new L.Bounds(
-                       bounds.min.unscaleBy(tileSize).floor(),
-                       bounds.max.unscaleBy(tileSize).ceil().subtract([1, 1]));
+                       bounds.min.unscaleBy(tileSize).floor().subtract([margin, margin]),
+                       bounds.max.unscaleBy(tileSize).ceil().add([margin - 1 , margin - 1]));
        },

        _noTilesToLoad: function () {

https://github.com/TolonUK/Leaflet.EdgeBuffer

I was missing it, thanks!

I tried the plugin with a GeoJSON Layer, but it seems that GeoJSON Layer are not Grid-based Layers. It would be great to have such a feature for geoJSON Layer too. Any chance to prefetch tiles with GeoJSON Layers?
The Performance would not be affected that much, because the GeoJSON Data is loaded upfront and there is no reloading of data while panning or zooming.

@SvenLauterbach You could create a PR for @TolonUK - it shouldn't be that hard to achieve your feature request.

@SvenLauterbach Have you tried the padding option of the renderer you use? http://leafletjs.com/reference-1.0.3.html#renderer.

@SvenLauterbach in addition to @geoloep's suggestion, you can also set noClip: true if you really don't want geometries to be clipped outside the viewport.
Note however, that clipping is done for performance reasons: even if you don't have to load anything in for example a GeoJSON layer, rendering a lot of data is expensive even if it's not visible. Of course, this depends on the complexity of your data, but in general it's wise to have clipping enabled and keep the padding to a reasonably small value.

Closing this, since nothing more to do as far as I can tell.

@geoloep does work like a charm, thank you.


Display a table of collected sales

Sort by price.
Include columns:

Model
Year
TTAF
SMOH

Related to #1

I began Flask development as a subpackage of the planefinder application. Crawling is separate from the webpage. The webpage can take some control of the program. Now it is mostly for display.
I'm basing the structure on the O'Reilly book Flask Web Development
Next steps are to create a basic CSS file, a base template, a route, etc.

Next step, define a function in trade_a_plane to get the model year. Current function name is model_year(...).

Created initial test of the web app. Next, add data to the table.


use new package versions

ClimaAtmos v0.16.3 (cache)
SF v0.8 (interface)

#480 changes included here

ClimaLSM v0.3.4

closes #475 and #490
replaces #500, #498, #473

Can we remove UnPack package? Its functionality can be replaced by destructuring syntax supported in native Julia!

Can we remove UnPack package? Its functionality can be replaced by destructuring syntax supported in native Julia!

I want to get this PR merged today, so I'll remove UnPack separately. I opened an issue for it here https://github.com/CliMA/ClimaCoupler.jl/issues/520

Plots do look slightly different, but I think it's within acceptable bounds given there are no overall trend changes and we've updated for a new SurfaceFluxes in this PR. I'll merge now


[master branch] "tile_image" generate tiles with different sizes when the input image dimension is not the multiple of input tile size

proposed fix:
change
for row in range(0, n_rows, step_size[0]):         if row + tile_size[0] > n_rows:             row = check_in_range(row, n_rows, tile_size[0])         for col in range(0, n_cols, step_size[1]):             if col + tile_size[1] > n_cols:                 col = check_in_range(col, n_cols, tile_size[1])
to
for row in range(0, n_rows - tile_size[0] + step_size[0]+ 1, step_size[0]):         if row + tile_size[0] > n_rows:             row = check_in_range(row, n_rows, tile_size[0])         for col in range(0, n_cols - tile_size[1] + step_size[1]+ 1, step_size[1]):             if col + tile_size[1] > n_cols:                 col = check_in_range(col, n_cols, tile_size[1])

This issue is fixed in master. But it is a major refactor so it might not work to rebase to your branch.

Thanks Jenny! I saw the if statement has been fixed, but I think the range for the row and column indices need to be fixed too, otherwise the function will generate the last tile multiple times as long as row < n_rows-step_size[0]

Ah yes, you're right. I'll submit a PR soon.

Solved by #42


Brave label on the wrong line

https://github.com/Installomator/Installomator/blob/ca17fa3c6b5d619af2ed441fad037d831e5ea1e3/Installomator.sh#L1232
just wanted to point out that the brave label is on the wrong line

I don't know what you saw or if things fell i to the right place by themselves, but I don't see the problem. At least not in our updated 0.7beta.