Error Decoding x509 Certificate

Describe the bug:
The Certificate (which is automatically generated by ingress-shim) is having issues decoding x509 certificates.
cert-manager.logs
I0829 11:00:41.315362       1 logger.go:38] Calling CreateOrder
E0829 11:00:41.315406       1 pki.go:128] cert-manager/controller/certificates/certificates "msg"="error decoding x509 certificate" "error"="error decoding cert PEM block" "related_resource_kind"="Secret" "related_resource_name"="childchain-stress-test-cloud-tls" "related_resource_namespace"="default" "secret_key"="tls.crt"
I0829 11:00:41.315675       1 issue.go:169] cert-manager/controller/certificates/certificates "level"=0 "msg"="Order is not in 'valid' state. Waiting for Order to transition before attempting to issue Certificate." "related_resource_kind"="Order" "related_resource_name"="childchain-stress-test-cloud-tls-2580951430" "related_resource_namespace"="default"
E0829 11:00:41.315719       1 pki.go:91] cert-manager/metrics "msg"="error decoding x509 certificate" "error"="error decoding cert PEM block" "related_resource_kind"="Secret" "related_resource_name"="childchain-stress-test-cloud-tls" "related_resource_namespace"="default" "secret_key"="tls.crt"
I0829 11:00:41.319303       1 base_controller.go:193] cert-manager/controller/certificates "level"=0 "msg"="finished processing work item" "key"="default/childchain-stress-test-cloud-tls"
E0829 11:00:41.551677       1 base_controller.go:189] cert-manager/controller/orders "msg"="re-queuing item  due to error processing" "error"="error creating new order: acme: urn:ietf:params:acme:error:rateLimited: Error creating new order :: too many new orders recently: see https://letsencrypt.org/docs/rate-limits/" "key"="default/childchain-stress-test-cloud-tls-2580951430"
I0829 11:00:41.693989       1 base_controller.go:187] cert-manager/controller/orders "level"=0 "msg"="syncing item" "key"="default/childchain-stress-test-cloud-tls-2580951430"
E0829 11:00:41.694081       1 controller.go:139] cert-manager/controller/orders "msg"="order in work queue no longer exists" "error"="order.certmanager.k8s.io \"childchain-stress-test-cloud-tls-2580951430\" not found"
I0829 11:00:41.694104       1 base_controller.go:193] cert-manager/controller/orders "level"=0 "msg"="finished processing work item" "key"="default/childchain-stress-test-cloud-tls-2580951430"
I0829 11:00:41.693989       1 base_controller.go:187] cert-manager/controller/certificates "level"=0 "msg"="syncing item" "key"="default/childchain-stress-test-cloud-tls"
E0829 11:00:41.694289       1 pki.go:128] cert-manager/controller/certificates/certificates "msg"="error decoding x509 certificate" "error"="error decoding cert PEM block" "related_resource_kind"="Secret" "related_resource_name"="childchain-stress-test-cloud-tls" "related_resource_namespace"="default" "secret_key"="tls.crt"
E0829 11:00:41.704102       1 pki.go:91] cert-manager/metrics "msg"="error decoding x509 certificate" "error"="error decoding cert PEM block" "related_resource_kind"="Secret" "related_resource_name"="childchain-stress-test-cloud-tls" "related_resource_namespace"="default" "secret_key"="tls.crt"
I0829 11:00:41.704144       1 base_controller.go:187] cert-manager/controller/orders "level"=0 "msg"="syncing item" "key"="default/childchain-stress-test-cloud-tls-2580951430"
I0829 11:00:41.704272       1 base_controller.go:193] cert-manager/controller/certificates "level"=0 "msg"="finished processing work item" "key"="default/childchain-stress-test-cloud-tls"
I0829 11:00:41.704318       1 base_controller.go:187] cert-manager/controller/certificates "level"=0 "msg"="syncing item" "key"="default/childchain-stress-test-cloud-tls"
I0829 11:00:41.704552       1 sync.go:89] cert-manager/controller/orders "level"=0 "msg"="creating Order with ACME server as one does not currently exist" "resource_kind"="Order" "resource_name"="childchain-stress-test-cloud-tls-2580951430" "resource_namespace"="default"
E0829 11:00:41.704595       1 pki.go:128] cert-manager/controller/certificates/certificates "msg"="error decoding x509 certificate" "error"="error decoding cert PEM block" "related_resource_kind"="Secret" "related_resource_name"="childchain-stress-test-cloud-tls" "related_resource_namespace"="default" "secret_key"="tls.crt"
I0829 11:00:41.704598       1 sync.go:376] cert-manager/controller/orders "level"=0 "msg"="order URL not set, submitting Order to ACME server" "resource_kind"="Order" "resource_name"="childchain-stress-test-cloud-tls-2580951430" "resource_namespace"="default"
I0829 11:00:41.704819       1 sync.go:382] cert-manager/controller/orders "level"=0 "msg"="build set of domains for Order" "resource_kind"="Order" "resource_name"="childchain-stress-test-cloud-tls-2580951430" "resource_namespace"="default" "domains"=["childchain-stress-test.omg.network"]
I0829 11:00:41.704848       1 logger.go:38] Calling CreateOrder
I0829 11:00:41.705099       1 issue.go:169] cert-manager/controller/certificates/certificates "level"=0 "msg"="Order is not in 'valid' state. Waiting for Order to transition before attempting to issue Certificate." "related_resource_kind"="Order" "related_resource_name"="childchain-stress-test-cloud-tls-2580951430" "related_resource_namespace"="default"
E0829 11:00:41.705184       1 pki.go:91] cert-manager/metrics "msg"="error decoding x509 certificate" "error"="error decoding cert PEM block" "related_resource_kind"="Secret" "related_resource_name"="childchain-stress-test-cloud-tls" "related_resource_namespace"="default" "secret_key"="tls.crt"
I0829 11:00:41.705303       1 base_controller.go:193] cert-manager/controller/certificates "level"=0 "msg"="finished processing work item" "key"="default/childchain-stress-test-cloud-tls"

Expected behaviour:
The certificate is decoded successfully.
Steps to reproduce the bug:
I followed the quickstart - https://docs.cert-manager.io/en/latest/tasks/issuing-certificates/ingress-shim.html
Anything else we need to know?:
Environment details::

Kubernetes version (e.g. v1.10.2):

Client Version: version.Info{Major:"1", Minor:"12+", GitVersion:"v1.12.9-gke.7", GitCommit:"b6001a5d99c235723fc19342d347eee4394f2005", GitTreeState:"clean", BuildDate:"2019-06-24T19:37:31Z", GoVersion:"go1.10.8b4", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"12+", GitVersion:"v1.12.8-gke.10", GitCommit:"f53039cc1e5295eed20969a4f10fb6ad99461e37", GitTreeState:"clean", BuildDate:"2019-06-19T20:48:40Z", GoVersion:"go1.10.8b4", Compiler:"gc", Platform:"linux/amd64"}

Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): GKE
cert-manager version (e.g. v0.4.0): v0.9.1
Install method (e.g. helm or static manifests): static manifests

ingress.yaml
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  annotations:
    certmanager.k8s.io/cluster-issuer: letsencrypt-staging
    ingress.kubernetes.io/backends: <redacted>
    ingress.kubernetes.io/forwarding-rule: <redacted>
    ingress.kubernetes.io/target-proxy: <redacted>
    ingress.kubernetes.io/url-map: <redacted>
    kubernetes.io/ingress.class: gce
    kubernetes.io/ingress.global-static-ip-name: <redacted>
    kubernetes.io/tls-acme: "true"
  creationTimestamp: 2019-08-27T07:46:45Z
  generation: 13
  labels:
    component: test-ingress
    heritage: Tiller
    release: stress-test
  name: stress-test
  namespace: default
spec:
  rules:
  - host: test.domain.com
    http:
      paths:
      - backend:
          serviceName: test
          servicePort: 80
        path: /*
  tls:
  - hosts:
    - subdomain.domain.com
    secretName: test-cloud-tls
status:
  loadBalancer:
    ingress:
    - ip: <redacted>

clusterissuer.yaml
apiVersion: certmanager.k8s.io/v1alpha1
kind: ClusterIssuer
metadata:
  annotations:
  creationTimestamp: 2019-08-28T09:05:34Z
  generation: 1
  name: letsencrypt-staging
  resourceVersion: "16092254"
spec:
  acme:
    email: <redacted>
    http01: {}
    privateKeySecretRef:
      name: letsencrypt-staging
    server: https://acme-staging-v02.api.letsencrypt.org/directory
status:
  acme:
    lastRegisteredEmail<EMAIL_ADDRESS>    uri: https://acme-staging-v02.api.letsencrypt.org/acme/acct/10776649
  conditions:
  - lastTransitionTime: 2019-08-28T09:05:35Z
    message: The ACME account was registered with the ACME server
    reason: ACMEAccountRegistered
    status: "True"
    type: Ready

/kind bug

This is a benign log message and only happens because the Certificate has not been issued yet - the order is still in progress and you should see certificates issued shortly afterwards providing you've configured your cluster correctly.
If you are having issues actually obtaining certificates, can you join the #cert-manager channel on slack.k8s.io as there's a larger body of people there able to help debug/dig in, and we find interactive back-and-forth helps uncover issues a lot quicker 😄

@munnerz Hello, struggling to generate wildcard certs, config below:
Certificate:
apiVersion: cert-manager.io/v1alpha2
kind: Certificate
metadata:
  name: darwincontent
  namespace: ingress-nginx
  labels:
    use-gcloud-solver: "true"
spec:
  secretName: wildcard-cert
  issuerRef:
    name: letsencrypt-production
  commonName: '*.domain.com'
  dnsNames:
  - '*.domain.com'

ClusterIssuer:
apiVersion: cert-manager.io/v1alpha2
kind: ClusterIssuer
metadata:
  name: letsencrypt-production
spec:
  acme:
    # You must replace this email address with your own.
    # Let's Encrypt will use this to contact you about expiring
    # certificates, and issues related to your account.
    email<EMAIL_ADDRESS>    server: https://acme-v02.api.letsencrypt.org/directory
    privateKeySecretRef:
      # Secret resource used to store the account's private key.
      name: letsencrypt-prod

    solvers:
    - selector: {}
      http01:
        ingress:
          class: nginx

    - selector:
        matchLabels:
          use-gcloud-solver: "true"

      dns01:
        clouddns:
          # The ID of the GCP project
          # reference: https://docs.cert-manager.io/en/latest/tasks/issuers/setup-acme/dns01/google.html
          project: 'stacksoft-managed'

          # This is the secret used to access the service account
          serviceAccountSecretRef:
            name: gcloud-dns-service-account
            key: key.json

Results:
Stuck on :
Normal  Requested     8m23s  cert-manager  Created new CertificateRequest resource

Logs:
E1018 01:52:23.885758       1 pki.go:128] cert-manager/controller/certificates "msg"="error decoding x509 certificate" "error"="error decoding cert PEM block" "related_resource_kind"="Secret" "related_resource_name"="darwincontent-cert" "related_resource_namespace"="ingress-nginx" "resource_kind"="Certificate" "resource_name"="darwincontent" "resource_namespace"="ingress-nginx" "secret_key"="tls.crt"
I1018 01:52:23.886027       1 controller.go:135] cert-manager/controller/certificates "level"=0 "msg"="finished processing work item" "key"="ingress-nginx/darwincontent"
E1018 01:52:23.887055       1 controller.go:131] cert-manager/controller/certificaterequests-issuer-venafi "msg"="re-queuing item  due to error processing" "error"="Operation cannot be fulfilled on certificaterequests.cert-manager.io \"darwincontent-3947332722\": the object has been modified; please apply your changes to the latest version and try again" "key"="ingress-nginx/darwincontent-3947332722"
I1018 01:52:23.887092       1 controller.go:129] cert-manager/controller/certificaterequests-issuer-venafi "level"=0 "msg"="syncing item" "key"="ingress-nginx/darwincontent-3947332722"
I1018 01:52:23.887176       1 controller.go:135] cert-manager/controller/certificaterequests-issuer-venafi "level"=0 "msg"="finished processing work item" "key"="ingress-nginx/darwincontent-3947332722"

I1018 01:52:28.875595       1 controller.go:129] cert-manager/controller/certificaterequests-issuer-selfsigned "level"=0 "msg"="syncing item" "key"="ingress-nginx/darwincontent-3947332722"
I1018 01:52:28.877219       1 controller.go:135] cert-manager/controller/certificaterequests-issuer-selfsigned "level"=0 "msg"="finished processing work item" "key"="ingress-nginx/darwincontent-3947332722"
I1018 01:52:28.883473       1 controller.go:129] cert-manager/controller/certificaterequests-issuer-ca "level"=0 "msg"="syncing item" "key"="ingress-nginx/darwincontent-3947332722"
I1018 01:52:28.884491       1 controller.go:135] cert-manager/controller/certificaterequests-issuer-ca "level"=0 "msg"="finished processing work item" "key"="ingress-nginx/darwincontent-3947332722"
I1018 01:52:28.884904       1 controller.go:129] cert-manager/controller/certificaterequests-issuer-vault "level"=0 "msg"="syncing item" "key"="ingress-nginx/darwincontent-3947332722"
I1018 01:52:28.884999       1 controller.go:135] cert-manager/controller/certificaterequests-issuer-vault "level"=0 "msg"="finished processing work item" "key"="ingress-nginx/darwincontent-3947332722"
I1018 01:52:28.888785       1 controller.go:129] cert-manager/controller/certificaterequests-issuer-venafi "level"=0 "msg"="syncing item" "key"="ingress-nginx/darwincontent-3947332722"
I1018 01:52:28.888903       1 controller.go:135] cert-manager/controller/certificaterequests-issuer-venafi "level"=0 "msg"="finished processing work item" "key"="ingress-nginx/darwincontent-3947332722"

Hello @arianitu ,
My error same u...
Can u fix this? Pls tell me how to fix this.
Thanks

Hey folks,
Anybody had any luck finding a fix for this issue yet
please do update us,
thanks and regards,

Hey folks,
Anybody had any luck finding a fix for this issue yet
please do update us,
thanks and regards,

I found issue in my Issuer config. There was changed apiVersion
from:
certmanager.k8s.io/v1alpha1
to:
cert-manager.io/v1alpha2
It solved my issue

@manhvc @rush2subbu
Hello everyone, almost always the issue is a configuration issue on your end and not cert-manager, but cert-manager is horrible with error messages, the documentation probably to blame and the authors assuming we know how it works.
The way I solved this issue was realizing that errors don't only live in the cert, they also live in all the sub objects and you should check each of them.
For example, if you go to https://docs.cert-manager.io/en/latest/reference/certificates.html, notice on the left menu there are:

Certificate
CertificateRequests
Orders
Challenges,
etc.

You can actually find error messages in each of these, like so:
kubectl get certificaterequest
kubectl describe certificaterequest X
kubectl get order
kubectl describe order X
kubectl get challenge
kubectl describe challenge X
If you do it for all of them, you will more than likely find the real cause of your issue. In my case, it was a config issue in the ingress where it was pointing to a ClusterIssuer that did not exist (because I named it incorrectly.)
I would have never known that unless I looked at the messages in the other objects since the error messages in cert-manager were just making me look in the wrong place. Like @munnerz said, the message above is just a benign message that is pointing you in the wrong direction.
cert-manager needs some serious changes in the way it handles errors, it's been horrible for years now and I think they just need to not log any message that isn't an error, or have a guide on how to debug a cert that isn't being generated.

Actually, I had the exact same problem and the advice from @ivanovUA solved the problem. It was not crystal clear following the cert-manager docs ;-)

same for me ^ 💯

Just to document what had happened to me..
I had created a new cluster and for some reason (probably because I've been using a NodePort service instead of LoadBalancer for the nginx-ingess) the firewall rules had not automatically been set. Make sure port 80 and 443 is open!
Figured it out embarrassingly late.. :)

I had this issue when setting commonName to a wildcard but not including this wildcard in dnsNames field. I got to the root of the problem by describing the CertificateRequest:
$ kubectl describe certificaterequest.cert-manager.io

In my case describing the certificate request as advised above by @arianitu  pointed me in the right direction:
`1. Run: kubectl get certificaterequest -n test

Using the output from previous command run: kubectl describe certificaterequest -n test wild-test-domain-example-2164310232`
Events:
Type    Reason          Age                From          Message

Normal  IssuerNotFound  13m (x5 over 13m)  cert-manager  Referenced "Issuer" not found: issuer.cert-manager.io "letsencrypt-test" not found

Since the CRD for certificates changed in the new cert-manager release. I had to describe it to see why the ClusterIssuer was not being found. I was clearly missing some piece of configuration. I did this running: kubectl describe crd certificaterequests.cert-manager.io

I learned that my CRD for my certificaterequests was missing the kind attribute under spec.issuerRef as in:

apiVersion: cert-manager.io/v1alpha2
kind: Certificate
metadata:
name: wild-test-domain-example
namespace: test
spec:
secretName: wild-test-domain-example-tls
issuerRef:
kind: ClusterIssuer
name: letsencrypt-test
dnsNames:

'test.com'

Thank you @arianitu for the pointers.

Spent a lot of time trying to resolve this issue.
This is what I have installed:
Rancher | v2.4.2
Helm | v2.16.3-rancher1
Cert Manager | v0.12.0 (helm ls --all-namespaces)
In my experience, I just needed to create an issuer that was valid. This is my Issuer yaml file. I was using Cloudflare.
staging_issuer.yaml
apiVersion: cert-manager.io/v1alpha2
kind: ClusterIssuer
metadata:
  name: letsencrypt-staging
  namespace: cert-manager
spec:
  acme:
    server: https://acme-staging-v02.api.letsencrypt.org/directory
    email<EMAIL_ADDRESS>
    # Name of a secret used to store the ACME account private key
    privateKeySecretRef:
      name: letsencrypt-staging

    solvers:
    - selector:
        dnsNames:
        - mygelb.com
      dns01:
        cloudflare:
          email<EMAIL_ADDRESS>          # !! Remember to create a Rancher secret before setting up
          apiTokenSecretRef:
            name: cloudflare-api-key-secret // Secret name
            key: api-key // Secret key

Make sure your issuer is valid and working. You can deploy a new issuer with these commands
Create new:
kubectl create --namespace=cert-manager -f staging_issuer.yaml

Edit an existing cluster issue yaml file:
kubectl apply -f staging_issuer.yaml

Lastly in my ingress, this was my config.

Whenever the certificate is generated, it will show up in the SSL/TLS Certificates. It will be greyed out when you first configure your ingress.
Once the issuer and your ingress controller is configured, your ingress controller will validate certificates. That's when you can run @arianitu commands to debug what the issue is.


I want to add Computer Graphics Book

Hey,
Experienced Web Developer here.
Please assign this issue to me
Thank you


Increase flexibility of isPartOf

The isPartOf method should be able to accept an eUriType so that the user can specify MODEL or LOCAL uri's in addition to identifiers.org uri's.

implemented by #15


"Page took too long to render" - Cannot view wiki

I'm getting a wiki error when I clicked a link on your wiki page about the cathode script editing:
Loading a script
To edit a script, launch the editor by going to "Experimental Tools" > "Edit Cathode Scripts". You can then select a level from the game in the dropdown in the top left and press "Load".
All flowgraphs (scripts) will be displayed in the hierarchy on the left hand side of the toolset.
Select a script and you will then see all of the nodes contained within that flowgraph. Nodes are displayed with their name, and their type in brackets after. To view all available node types,  click here.
You can select each node to load up its parameters and links. Imagine the UE4 Blueprint interface when thinking of node child/parent links - parent links come in to the node on the left hand side, and child links go out on the right.
This link generates a page to large to render wiki error:
click here.

Yeah this is a known issue, pages are rendered locally so if your PC isn't beefy enough it'll time out. If you refresh a few times it should eventually load :)
To solve this issue, I'll look at adding the node info to the opencage.co.uk website.

Thank you.  I'm using firefox on a year old I7-9700F with 16GB RAM and a GTX 1660 GPU.
Should be beefy enough for anything in a browser.
It's loading OK now.
If you did that check to the node info you seem to have fixed it.
Thank you again.


anndata obs should be recorded as well as schema

How to keep track of extracted obs?

The AnnDataExtractSchema  (or whatever we want to call it) must allow specifying arbitrary obs columns from BQ to include, e.g.:
BQ_OBS_COLUMN,EXTRACT_OBS_COLUMN
cell_type,cell_type
cell_barcode,original_cell_barcode
...


JSON API reaches final review phase.

Due to the massive number of changes to the specification in the past
two weeks, we are providing an extra week for final review before
tagging 1.0.
This release candidate, technically RC4, exists solely to provide time
for the specification to be carefully reviewed for inconsistencies and
defects. Do not expect any changes to the specified document structure.
Thanks to the supportive (and prolific) group of contributors who have
helped us reach this milestone. Let's ensure that the final spec is
as good as can be!

This should be merged by the end of today.

Yay! I know 1.0’s taken forever, but its also astonishing how much more rigorous the design has become lately. Let's remember that, less than four months ago, we were still using PUT; we had the option of responses without a data key and resource objects without a type; we recommended URIs that used comma-separated ids; the equivalent of the related resource url typically pointed to the resources linked at the time the client received the URI, rather than at the time it requested it; we didn't even have a first draft of extension negotiation; and we had no attributes, relationships, or resource identifier objects. Great work everyone!

Yeah, a couple more release candidates, and we are there! :)

:confetti_ball:

:shipit:

:+1:


refactor: migrate SqlEditor component to FC & tsx

SUMMARY
Migrated SqlEditor to TypeScript to apply direction outlined in #18100 .

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[x] Has associated issue: #18100
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@ad-m @eschutho I have a issue with <Menu onClick={this.handleMenuClick} style={{ width: 176 }}>
SqlEditor class component doesn't have handleMenuClick. What should I do?

I am now running the CI. WE should spin up a test env as soon as it passes


0.13.0 build 1 is broken on linux

Bug report at https://github.com/pyFFTW/pyFFTW/issues/294#issuecomment-1024595535
I can recreate the issue locally. If I force install of build 0 which was not using the static linking, then it works okay for me.
Specifically the following version works, but build "1" does not
mamba install "pyfftw=0.13.0=py39h51d1ae8_0"
The PyPI packages also work as expected.

not sure if the same issue is also present on OS X

Build 1 was where we switched back to static linking, so apparently something went wrong there. We can expand the test case a bit and see if the issue is reproduced on CI.

I also experienced this bug. It would be nice to find a solution :slightly_smiling_face: because this bug should affect a lot of packages and users for "simple installations".
For example on Linux, conda create -n env_pyfftw pyfftw leads to a broken environment.

Should we just revert the static linking?

That would be easiest. I didn't try looking at the logs to see what could be going wrong. We should also add a minimal test case like in the linked issue.

The error is missing symbols that seem related to pthreads e.g. from this run
  threads.c:(.text.fftw_threads_register_planner_hooks+0x40): undefined reference to `sem_init'
  collect2: error: ld returned 1 exit status

that results in a single-threaded build being run. There is perhaps an upstream bug in this single-threaded case, but that is not what it should be building. We have set the recipe to use pthreads multithreading (as opposed to OpenMP). I tried manually adding -pthread to CFLAGS in the build script, but it did not help (this should not be necessary as setup.py is already supposed to add it).
Also, there are RuntimeErrors during run_tests.py but this surprisingly does not cause the build to fail?
When I remove the STATIC_FFTW_DIR argument, the build succeeds again and no errors are run during testing. The log confirms that pthreads version is being built:
2022-02-20T01:49:10.6692980Z   INFO:__main__:Build pyFFTW with support for FFTW with
2022-02-20T01:49:10.6694033Z   INFO:__main__:double precision + pthreads
2022-02-20T01:49:10.6695354Z   INFO:__main__:single precision + pthreads
2022-02-20T01:49:10.6696310Z   INFO:__main__:long precision + pthreads

This makes me wonder if there is an issue with the static FFTW libs from fftw-feedstock

I don't know if it is related but I am still seeing the original message about undefined plan with nthreads.

Should we just revert the static linking?

Is there a reason not to do so? build 0 seems to work fine.

I don't know if it is related but I am still seeing the original message about undefined plan with nthreads.

Same here.

Should we just revert the static linking?

Is there a reason not to do so? build 0 seems to work fine.

Will reverting the static linking on linux not cause the bug fixed in #5 to resurface, if numpy is linked against MKL and numpy is imported first?

Currently the last build with static linking is not usable on Linux. Coming back to dynamic linking might lead to a bug for numpy linked with MKL but conda-forge users won't see that by default. It seems to me that it is better to have something working for the simple case that something not working at all.


pdngen: is there a way to short the VSS of two voltage domains?

I have a design from OpenFASOC with two voltage domains (CORE and TEMP_ANALOG), and I'm trying to generate its power delivery network with the new tcl commands. When generating their voltage rings, both ground tracks are completely isolated (i.e. they're not being connected):

Preferably, what I would like to achieve is for a metal strap to connect them together, like so (pardon my dreadful drawing):

Is there a way to do this in pdngen?
To reproduce the case:
Download reproducible_pdngen.tar.gz and source script.tcl in openroad (or openroad gui)
I'm on OpenROAD commit no. 5ca26d8 from Aug 4 2022.

@maliberty @luccareinehr looking at it, it appears there is no way to connect the two (that would be an oversight).
There are two things here:

need be able to connect two two grids (I think I know how to handle that)
At present, PDNGEN will not put straps into other grids, so for how we would need to move the met5 part of the ring down to met3 until that is fixed. This I think would require changing the order of the builds in the grids, so that will probably take longer to fix since it can come with lots of unintended changes.

@arlpetergadfort Can you explain how to connect the two grids?
By the way, thanks for the help

@luccareinehr I initially thought I would need to modify the code.
If I use the script below I can connect VSS between the two grids. The key here (and this could be a future improvement for PDNGEN) is to move the top layer of the ANALOG_TEMP grid to met4, so the CORE grid can place straps on met5 over that area. The second item needed is to tell the ANALOG_TEMP grip to connect from met4 to met5. Please try this out and let me know if there is anything else.
# Read odb file
read_db design_before_pdn.odb

# Add global connections
add_global_connection -net VDD -inst_pattern {temp_analog_1.*} -pin_pattern VPWR -power
add_global_connection -net VDD -inst_pattern {temp_analog_1.*} -pin_pattern VPB
add_global_connection -net VIN -inst_pattern {temp_analog_0.*} -pin_pattern VPWR -power
add_global_connection -net VIN -inst_pattern {temp_analog_0.*} -pin_pattern VPB
add_global_connection -net VSS -inst_pattern {.*} -pin_pattern VGND -ground
add_global_connection -net VSS -inst_pattern {.*} -pin_pattern VNB

# Set voltage domains
# TEMP_ANALOG region created with the create_voltage_domain command
set_voltage_domain -name CORE -power VDD -ground VSS
set_voltage_domain -region TEMP_ANALOG -power VIN -ground VSS

# Standard cell grids
define_pdn_grid -name stdcell -pins met5 -starts_with POWER -voltage_domains CORE

add_pdn_stripe -grid stdcell -layer met1 -width 0.49 -pitch 6.66 -offset 0 -extend_to_core_ring -followpins
add_pdn_ring -grid stdcell -layer {met4 met5} -widths {5.0 5.0} -spacings {2.0 2.0} -core_offsets {2.0 2.0}
add_pdn_stripe -grid stdcell -layer met4 -width 1.2 -pitch 56.0 -offset 2 -extend_to_core_ring
add_pdn_stripe -grid stdcell -layer met5 -width 1.6 -pitch 56.0 -offset 2 -extend_to_core_ring

add_pdn_connect -grid stdcell -layers {met4 met5}
add_pdn_connect -grid stdcell -layers {met1 met4}

define_pdn_grid -name stdcell_analog -pins met4 -starts_with POWER -voltage_domains TEMP_ANALOG

add_pdn_stripe -grid stdcell_analog -layer met1 -width 0.49 -pitch 6.66 -offset 0 -extend_to_core_ring -followpins
add_pdn_ring -grid stdcell_analog -layer {met4 met3} -widths {5.0 5.0} -spacings {2.0 2.0} -core_offsets {2.0 2.0}
add_pdn_stripe -grid stdcell_analog -layer met4 -width 1.2 -pitch 56.0 -offset 2 -extend_to_core_ring

add_pdn_connect -grid stdcell_analog -layers {met4 met3}
add_pdn_connect -grid stdcell_analog -layers {met1 met4}
add_pdn_connect -grid stdcell_analog -layers {met4 met5}

# Run pdngen
pdngen

# Save resulting design
write_db design_after_pdn.odb

@arlpetergadfort That gives a nice result, thank you!
May I ask why pdngen works this way? Does it check for intercepting geometries of the same layer when connecting straps to a ring? So, for example, if a ring and a strap are both made of met4, they wouldn't get connected (like was happening in the original case)

the macro grids generate a keepout area around them which is what prevents the met5 strap if you have the ring in met5. Otherwise, yes, it connects on the overlapping shapes based on the connect layers defined.


Decouple Conversion Patch and CRD Reconciler

Instead of using the CRD conversion webhook patching process
to add the osm reconciler label, use Go templates to conditionally
render the label on initial install

Description:
Fixes #4608

Testing done:
Unit tests added for the new installCRDs function
The demo also succeeds

Affected area:

Functional Area

New Functionality
[ ]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[x]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Did you notify the maintainers and provide attribution? no

Is this a breaking change? no

Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)? N/A

@nojnhuh I was going to write a bash script initially that looked similar to what you suggested, but @shashankram said that the team tends to favor Go binaries

@nojnhuh I was going to write a bash script initially that looked similar to what you suggested, but @shashankram said that the team tends to favor Go binaries

@keithmattix There seems to be a misunderstanding here. What I meant was that for the reconciliation part, it makes sense to keep the code within the Go application, and not script such functionality. If it's just a matter of running a few commands, doing so from the shell should work just fine. In fact, we do such things in the cleanup hooks and control plane init containers.

Ah, yes I misunderstood; I wasn't planning on re-writing the reconciliation stuff in shell. I'll adjust the PR to modify the labels via shell

Codecov Report

Merging #4612 (45b54c8) into main (aaa7f30) will increase coverage by 0.04%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##             main    #4612      +/-   ##
==========================================
+ Coverage   69.14%   69.18%   +0.04%
==========================================
  Files         216      216
  Lines       15000    15024      +24
==========================================
+ Hits        10372    10395      +23
- Misses       4576     4577       +1
  Partials       52       52

Flag
Coverage Δ

unittests
69.18% <92.85%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cmd/osm-bootstrap/osm-bootstrap.go
45.59% <0.00%> (-0.72%)
:arrow_down:

pkg/crdconversion/crdconversion.go
75.75% <100.00%> (+0.94%)
:arrow_up:

pkg/crdconversion/framework.go
74.56% <100.00%> (+6.88%)
:arrow_up:

pkg/metricsstore/metricsstore.go
92.17% <100.00%> (+0.27%)
:arrow_up:

pkg/messaging/workqueue.go
89.28% <0.00%> (-10.72%)
:arrow_down:

pkg/k8s/client.go
93.09% <0.00%> (+0.07%)
:arrow_up:

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 cd2c815...45b54c8. Read the comment docs.

@shashankram Done

Looks like my review didn't get in before merging. We can discuss and create a new issue/PR if changes are needed.


SVG motion path: offset problem

I am experimenting with anime and I am very impressed! Thank you for sharing this animation engine!
I run into a problem when tweening an svg along a path. If both, the element and the path are part of an SVG, there is an x/y offset for the element.
http://codepen.io/redon77/pen/vxJZee
I can not figure out, how to center the element along the path.
Is there a possibility to set the wanted offset (x, y, rotation)?

Hi
for
translateX: path,
translateY: path,
to
( to <
}  to >
(svg id="red")
(rect id="myCircle" width="24" height="16" fill="red"/)
(svg)
and in css
#red {
position: absolute;
top:  8px;
left: 8px;
}
for lotate:path,
to#myCircle {
transform-origin: 50% 50% 0;
}
Good luck,

@htakaoka thank you
As you can see, css position and rotation for the element inside the svg does not work
http://codepen.io/redon77/pen/vxJZee
only
transform-origin: -50% 50%;
has an effect
That seams logical for me, because anime is tweening top, left and rotation accordingly to the motionPath
So still looking for a solution to offset the element

Hi
rect id="myCircle" width="24" height="16" fill="red"/
don't set inside sgv for path.
make 2th sgv and set in.
<sgv
<path------
</sgv
<sgv id="red
<rect id="myRect"----
</sgv
Good luck,

… but I would like to have it inside the same svg
I found a solution:
The element to be moved has to be placed at the registerpoint of the svg with the desired offset (and not for example at the starting point of the path to follow)
http://codepen.io/redon77/pen/vxJZee

However, it would be nice if I could set the offset-values dynamically in anime.js

other things that I would like

move along path: from, to
I can now use
var path = anime.path('#motionPath path', 35);
to move from 0 to 35% of the path length
something like
var path = anime.path('#motionPath path', 12, 35);
move from 12% to 35%

and similar for svg line drawing
draw from 12% to 35%

at rect inside same svg
<rect id="myRect" x="-12" y="-8"  width="24" height="16" fill="red"/ >
it's work
Good luck.


cannot get all of available resolution in HLS url

Bug
i tried to get all of available resolution in HLS url, but i only got one resolution
Sample HLS url : https://bitdash-a.akamaihd.net/content/MI201109210084_1/m3u8s/f08e80da-bf1d-4e3d-8899-f0f6155f6efa.m3u8
When i tried to get all of available resolution via onLoad, i only got this list of videoTracks :
[{"trackId":"1","codecs":"","bitrate":0,"height":720,"width":1152}]

but when i tried in DASH url, it is working fine
Sample DASH url : http://dash.edgesuite.net/envivio/EnvivioDash3/manifest.mpd
[{"trackId":"v1_257","codecs":"avc1.4D401E","bitrate":1200000,"height":432,"width":768},
{"trackId":"v2_257","codecs":"avc1.4D401E","bitrate":1850000,"height":576,"width":1024},
{"trackId":"v3_257","codecs":"avc1.4D401E","bitrate":2850000,"height":720,"width":1280},
{"trackId":"v4_257","codecs":"avc1.4D401E","bitrate":200000,"height":180,"width":320},
{"trackId":"v5_257","codecs":"avc1.4D401E","bitrate":300000,"height":180,"width":320},
{"trackId":"v6_257","codecs":"avc1.4D401E","bitrate":4300000,"height":720,"width":1280},
{"trackId":"v7_257","codecs":"avc1.4D401E","bitrate":5300000,"height":1080,"width":1920},
{"trackId":"v8_257","codecs":"avc1.4D401E","bitrate":480000,"height":288,"width":512},
{"trackId":"v9_257","codecs":"avc1.4D401E","bitrate":750000,"height":360,"width":640}]

Platform
Android Exoplayer
Environment info
React native info output:
System:
    OS: macOS 10.15.1
    CPU: (8) x64 Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz
    Memory: 62.67 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 13.1.0 - /usr/local/bin/node
    Yarn: 1.19.1 - /usr/local/bin/yarn
    npm: 6.12.1 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
    Android SDK:
      API Levels: 22, 23, 24, 25, 26, 27, 28, 29
      Build Tools: 23.0.1, 23.0.2, 23.0.3, 25.0.0, 25.0.2, 26.0.2, 26.0.3, 27.0.3, 28.0.1, 28.0.2, 28.0.3, 29.0.2
      System Images: android-25 | Google APIs ARM 64 v8a, android-25 | Google APIs ARM EABI v7a, android-25 | Google APIs Intel x86 Atom, android-25 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>77832
    Xcode: 11.2.1/11B500 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.1 => 0.61.1
  npmGlobalPackages:
    react-native-cli: 2.0.1

Library version: 5.0.2
Steps To Reproduce

pass url via source={{uri: "https://bitdash-a.akamaihd.net/content/MI201109210084_1/m3u8s/f08e80da-bf1d-4e3d-8899-f0f6155f6efa.m3u8"}}
log the onLoad payload onLoad={(payload) => console.log("PAYLOAD : "+JSON.stringify(payload.videoTracks))}
...

Expected behaviour

get all of the available resolution

Reproducible sample code
<Video
       source={{uri: "https://bitdash-a.akamaihd.net/content/MI201109210084_1/m3u8s/f08e80da-bf1d-4e3d-8899-f0f6155f6efa.m3u8"}}
       controls
       style={{flex: 1}}
       onLoad={(payload) => console.log("PAYLOAD : "+JSON.stringify(payload.videoTracks))}
     />

Video sample
https://bitdash-a.akamaihd.net/content/MI201109210084_1/m3u8s/f08e80da-bf1d-4e3d-8899-f0f6155f6efa.m3u8

is there any solution or workaround about this?

hello @mabudiman , the problem arise because i implement IMA SDK with preroll ads, i think it will work normally, if you use standard react-native-video

Hello, @RobbyWH is it possible to use hls videoTracks with IMA SDK?

hello @arslan-akh , it is possible if you implement IMA SDK without preroll ads
but if you implement IMA SDK with preroll ads, you must use third party library like https://github.com/shinnn/parse-m3u8
because if the ads play in the preroll, the react-native-video cannot load the original videoTracks data

Hello, @RobbyWH is it possible to use hls videoTracks with IMA SDK? I have same problem with exoplayer, I get all data from onLoad event except videoTracks

Facing the same issue, how can i resolve it?


Interest

Hey, I'm interested in this project but have no experience with custom pcbs. Are you planning on having a run of these manufactured? Is that up to someone else? Can I take these plans to someone and have a one-off made? Is it still considered work-in-progress and or is it production-ready?
Would love to get my hands on one. Thanks for all the work on it.

Hey!
I was (and still am) planning to get a batch manufcatured and sell them - Covid and the following chip crysis however hit the market really hard, there's currently no chance to get a reasonable amount (even 1 is already hard) of the chips on this PCB for a decent price (less than 10-20 bucks per chip, they should be ~ 1.5$ MSRP).
Once the market recovers, I'm definitely still planning to do a Groupbuy!

That's great! What can I follow/subscribe/check to be sure I'm notified once you start taking pre-orders?
Also, what would it take to add a split backspace option to the board? I use all compact boards and my muscle memory has the backspace on R2 so I make a lot of \\\ typos on this board. I'd be happy to sponsor that feature if you're open to it (and I can afford it).

I'm currently thinking about creating a discord server, not sure if I'll do that though. My instagram (nick: ebastler) should be useful, if you have instagram. Guess I'll make it a geekhack post too.

In case anyone stumbles upon this: https://discord.gg/VG8xeKzdr7
I actually made a discord server.


fix(rgaa): remove all fieldset

:tada: This PR is included in version 1.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Websocket server fails to send lots of small messages (or one large message)

Testing with master + pull request #420 and standard HTML5 JS WebSocket API.
var str = "" for (i <- 1 to 51) {   str += "x"   println(s"$i-${str.length}")   sendMessage(s"$i-$str") }
The client will stop receiving messages beyond i=50. This corresponds to the output buffer size set within the BaseWebsocketHandler. This is a contrived example but the real problem is that you can't send large messages e.g. a JSON string to the client at all.

Thanks for bringing this up, obviously we haven't done too much with the websocket api yet so I'm glad to get some feedback.  I think there's a few different issues happening here.  First, the output buffer size of 50 is entirely arbitrary and definitely way too small.  It should definitely not be hardcoded like that and can easily have a much larger default value.  I've already fixed this in another branch targeted for colossus 0.9, but I'll work to backport to master it so we can publish a fixed build sooner.
Second, the websocket handler doesn't have a good mechanism to deal with output backpressure.  Normally when pushing messages you can attach a callback function to run once the message has been sent, but that's been hidden in the websocket API.  I will look at improving this.
Lastly, I suspect there's a bug in the websocket frame encoding that is causing problems when sending large messages.  I will do what I should have done before and write more tests to figure out what's going on :)  If I can figure this out quickly enough I'll just merge them into #420


Can't query SQL db, can't pickle 'module' object

Describe the bug
Can't connect to redshift database
Browser and Version

Chrome 112

To Reproduce
Steps to reproduce the behavior:
Make the flow as in the screenshot using the SQL Database utility, and ask a question about the DB.
I get a "cannot pickle 'module' object" error.
Screenshots

Additional context
Found this issue in Langchain, which was fixed on their side already: https://github.com/hwchase17/langchain/issues/1721
Maybe this will be fixed if we use a more recent Langchain version in Langflow build.

Can you upload your flow document so we can reproduce the error


If we are able to load folder contents from cache but not from the network, don't switch into error state.

Verified that @RicoYao has signed the CLA. Thanks for the pull request!

+2 some comments

+2 lgtm
:on:


Use len(tkz) instead of tkz.vocab_size to estimate vocabulary size

vocab_size may not be changed when new tokens added
ex: that's what happening when using microsoft/phi-1_5
Comment with explanation

Thanks @EgorBu ! LGTM. Will merge after tests finish running assuming all looks good.


Release Marathon 1.4.0-RC8

Cherry picks:

Improve performance of dependency graph computations https://phabricator.mesosphere.com/D476
Performance improvements in 1.1 branch: 334734390f0c22ea10e5b9746f5430168dcd43cb (see also https://phabricator.mesosphere.com/D481)
[ ] NetworkSpec plugin API (https://phabricator.mesosphere.com/D490)
[x] Generate DiscoveryInfo for pod container endpoints (#5087) -- cherry picked with https://github.com/mesosphere/marathon/commit/593ca148d2a6063de8b769b34356e30c9e8d92d3
[x] Clarify rexray documentation (#5117) -- cherry picked with https://github.com/mesosphere/marathon/commit/f997313e5c77f6f16390a53c25c7eb2bb4bde8c6
[ ] Define network-scope label for ipaddress.discovery.ports (#5144)
[x] Increase queue length for storage operations, helps large migrations (#5083) -- cherry picked with https://github.com/mesosphere/marathon/commit/cbdadb2c0944537e2d090678a9dc6a44b0448365

Hi tim, what about this PR?? https://github.com/mesosphere/marathon/pull/4107

[ ] D496 pods allow duplicate endpoint ports

#4107 is already in the releases/1.4 branch, thanks
On Mon, Feb 6, 2017 at 8:46 AM, Alexis Petalas<EMAIL_ADDRESS>wrote:

Hi tim, what about this PR?? #4107
https://github.com/mesosphere/marathon/pull/4107
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/marathon/issues/5110#issuecomment-277684982,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACPVLMZOLUIf316364ruDOWBFZFhnXSRks5rZyQhgaJpZM4L29hO
.

unless something crazy comes up tomorrow .. we should try to land RC8 ASAP because others in the org are waiting on it

https://github.com/mesosphere/marathon/issues/5144 depends on https://github.com/mesosphere/marathon/commit/1c9f0cd2236707538623a52bf0aedea95ac2d2b1 which introduces PortDiscovery class.

@alexispetalas the PR should have landed on master with https://github.com/mesosphere/marathon/pull/4418.

Release is currently blocked by https://github.com/mesosphere/marathon/pull/5157.

dcos/dcos PR: https://github.com/dcos/dcos/pull/1231


Fix hyperlinks in Documentation Framework

Notes for Reviewers
This PR fixes #7479

**[Signed commits](https://github.com/meshery/meshery/blob/master/CONTRIBUTING.md#signing-off-on-commits-developer-certificate-of-origin)**
- [x] Yes, I signed my commits.

🚀 Preview for commit 5be05bd5665cde90df0be0dada26b6bc31eb8292 at: https://642d95b29a77225a1dbdfc66--meshery-docs-preview.netlify.app

🚀 Preview for commit a8cd66e2a03496eaeeeafba5dbd1f9888d612504 at: https://642d97a992354e005b693193--meshery-docs-preview.netlify.app

🚀 Preview for commit 5e3141fb38c4cb0feeee753845cc7de23b93ca33 at: https://642ddfbfefd06e00abe962c1--meshery-docs-preview.netlify.app

🚀 Preview for commit 4dadcaf469907654b2409a5798cbe441365f32e2 at: https://642edb9b6b210a0a3a95b2a1--meshery-docs-preview.netlify.app


Feature Request: Collapsible Admonition blocks.

It would be interesting to enable admonitions to be collapsible.
Perhaps we could draw inspiration from the following:
https://squidfunk.github.io/mkdocs-material/reference/admonitions/#collapsible-blocks
Instead of defining 3 colons ::: we could define 3 question marks ??? and then they are collapsible (collapsed by default)
Only the title would be shown, and an arrow indicator on the right of the admonition shows the user that it is possible to expand the admonition.

Yes please - this would really assist me in migrating documentation sites from mkdocs-material to Docusaurus!


Manpage Format is Incorrect

Describe the bug
I set my default manpager to neovim by adding this to my .bashrc or .zshrc: export MANPAGER='nvim +Man!'
When opening a man page through a neovim terminal buffer with flatten.nvim loaded, the man page expectedly loads in the current neovim instance, but the format is messed up (see the screenshots below).
To Reproduce
Steps to reproduce the behavior:

Add export MANPAGER='nvim +Man!' to shell config
Open a terminal buffer in neovim
man ls
See error

Screenshots

man ls from the shell

man ls from neovim terminal

Desktop (please complete the following information):

macOS Monterey 12.5 (M1 Macbook Air)
NVIM v0.9.0-dev-1326+g7c8c15507

Additional Context
After looking around, the simple issue is that the Man! command isn't being run after flatten opens the man page in the current neovim instance. Could allowing nvim +<cmd> work with flatten?

This is something I've been looking into - definitely Manpage support is something I could hard code as an edge case, but in terms of supporting command pass-through I haven't yet figured out a way to do that. I am actively looking though and will absolutely implement that when I'm able.
I'll look for a solution for manpages in the short term though, thanks for reporting!
Until then, probably best to just use Man <topic> from the parent instance's command-line.

in terms of supporting command pass-through I haven't yet figured out a way to do that

This is such a great coincidence, I was about to post this in a discussion thread, but I think it's relevant enough to post it here.
I'm looking to solve a very similar problem, and I found a way to read all the arguments passed to the nested nvim instance.
I'm using the v:argv variable, which sounds the same as the argv function, exept that v:argv includes way more data.
For example, when open neovim in diff mode:
nvim -d /tmp/a/README.md /tm/b/README.md

I get the following through vim.v.argv:
{ "<HOME>/.nix-profile/bin/nvim",
  "--cmd", "let g:loaded_node_provider=0 | let g:loaded_python_provider=0 | let g:python3_host_prog='/nix/store/2j5i0n338vrw9w1jwhmfgfxhnbncr27h-neovim-0.8.1/bin/nvim-python3' | let g:ruby_host_prog='/nix/store/2j5i0n338vrw9w1jwhmfgfxhnbncr27h-neovim-0.8.1/bin/nvim-ruby'",
  "--cmd", "set packpath^=/nix/store/vncf2y1mpg3m8iii6dngmjvsinb732sg-vim-pack-dir",
  "--cmd", "set rtp^=/nix/store/vncf2y1mpg3m8iii6dngmjvsinb732sg-vim-pack-dir",
  "-d", "file-a.txt", "file-b.txt" }

Which gets lets me know that -d was trying to start a diff mode for the passed files.
I haven't figured out how to get only the commands actually passed by the user. But for the time being, that does what I need.
See my fork: https://github.com/davidmh/flatten.nvim/commit/311b4a2f22f0e7d9fbb5659adb65d4c3a232ee83
And this is how I use it: https://github.com/davidmh/dot-files/blob/main/nvim/fnl/own/plugin/flatten.fnl#L20-L30
Some things I haven't worked through, which I like to address before sending a PR:

[ ] Get the actual commands passed by the user, not the extra commands shown in the output
[ ] Have options like -b automatically enable blocking.
In my use case, the diff mode is meant to be used with the approvals

What do you think?

Thank you so much! That's super helpful, will look through your fork and working v:argv tomorrow.
As far as a PR goes, that would be very welcome, but I would like to avoid adding argument flags that aren't already included, so that using flatten is as much like using default neovim as possible. I would consider adding args for specific use cases such as blocking if enough people seem to prefer that to setting a global though, so if you submit a PR it would be ideal for the functionality to be enabled by a global or module-local var instead of an argument directly, so that I/we can decide on which direction to go in based on how the general user base responds to the idea.
Thanks!

That makes sense, maybe a better way to enable blocking programmatically would be to introduce a callback function to determine the blocking before the files are sent from within the nested session.

As far as a PR goes, that would be very welcome, but I would like to avoid adding argument flags that aren't already Neovim args, so that using flatten is as much like using default Neovim as possible

I'm just re-reading this. The flag I mentioned (-b) already exists in neovim, what I was proposing was to opt out of flatten through a config option in the flatten config. The user would start neovim as they always do.
I'll tune up the fork to include a new callback: config.callbacks.should_block(vim.v.argv) to be used as a fallback to vim.g.flatten_wait.
I would be happy to also work on dispatching other commands, like the +Man something in this issue.

Yes, sorry, I should've been more clear - I meant that I'd like to keep all arguments the same functionally as well as semantically, but it seems like opting out through a config option would work very well. Thank you for contributing, I'll review the PR today!


Show people and their roles in the project in search results

This will help, for the SPARQL side of things:
https://github.com/ESIPFed/science-on-schema.org/blob/master/guides/Dataset.md#roles-of-people
Once we get this information into our SearchResult object, we can show it in the search result UI.

We have this for DataONE right now but we can show them for Gleaner results, too

On the DataONE side, their results contain lists under authorLastName and investigator.


[WIP] Packaging: prepare building for RHEL 8 (Python3)

Make possible building of leapp and snactor utilities for Python3.
We plan to support Python2 only no RHEL 7. For all other systems,
build only for Python3.
This is now just untested POC for testing purposes
Expected command for building now:
PR=678 COPR_REPO=leapp9-poc DIST_VERSION=8 make copr_build

Can one of the admins verify this patch?

We are kind of pretty moved with the stuff since the time I created this draft. I am closing it as we will do much more changes in close future that are beyond this PR.


Running EthTx against a local mainnet fork?

It would be super-useful to run EthTx client against a local mainnet fork, eg. doable via Hardhat. I tried that, however Hardhat client does not support tracers yet https://github.com/NomicFoundation/hardhat/issues/1668
Erigon apparently does not support local forks https://github.com/ledgerwatch/erigon/issues/1613
Is there still any option to run EthTx on a local mainnet fork at all?

@pokrovskyy tried Foundry Anvil?
https://github.com/mds1/forge-trace-and-script-issue#issue-1-more-verbose-failure-messages-in-scripts


Mixins need to be moved to libs or higher level

when moving to libsass we need to provide standard mixins and move layout/_mixins.scss accordingly.
the option is to use /libs or create a separate folder /mixins - open for discussion.

As for me better is separate folder /mixins

/mixins folder makes more sense. Custom mixins will then be put in that folder as well, so that all mixings were in one place. /libs won't work like that.


Add more protos to the repo following the spec

This is still a work in progress. So far, this PR:

Add endpoints for new bsp endpoints
Add ScalaPlatform to build target
Add scala test classes protos
Add clarification to the docs that compile is intransitive

Ping, any update on this?

Just addressed all feedback, feel free to refine the introduction and motivation. We're still far from having something as convincing as I'd like them to be.
Did you have a look at the addition buildTarget/run and buildTarget/scalaMainClasses?

@jastice Feel free to have a look at the changes, your input is appreciated :smile: I haven't yet implemented in the bloop side but I think that the sbt language extension has all we need to have better build.sbt file support in IntelliJ, could you confirm?

@jvican From the description it's not clear to me: is SbtBuildTarget supplied for any target used by sbt? Is parent supposed to be the meta build of that target or the other way around?
Minimum information missing: sbt version (required for correct highlighting)
probably helpful: sbt project uri/id, configuration, project base dir. these could be used to merge targets back to intellij modules where necessary.

@jastice Thanks for having a look!
The child (a normal build) has a reference to the parent (meta-build), and so on. The idea is that there will be an sbt build target per sbt project (including meta projects) so that IntelliJ can import them correctly.
You're right that the sbt version is missing. The project URI is already in the BuildTarget. The way the structure works is that the BuildTarget has a data field that contains the language specific information, in this case SbtBuildTarget.
What I don't understand is why you need configurations. Isn't the configuration always going to be Compile?

Is the BuildTarget uri the exact project uri? how is the sbt project id included?
In bloop you have separate name and name-test targets that I combine back into a single IDEA module with compile and test sources/dependencies/etc respectively. How do you intend to do this for sbt?

Is the BuildTarget uri the exact project uri? how is the sbt project id included?

The build target URI will be the one of the sbt project URI, like /data/rw/code/scala/foo/project. What do you mean by sbt project id?

In bloop you have separate name and name-test targets that I combine back into a single IDEA module with compile and test sources/dependencies/etc respectively. How do you intend to do this for sbt?

As there's only one configuration per sbt project, I was thinking of just having one IDEA module for the sbt build. Is that possible?

This is ready for review again @olafurpg

Whoop, accidentally closed the PR, sorry.
I am ok with keeping BuildTarget.kind if we make it a List[String], I'm convinced it's useful information.

Feedback addressed, thanks for your feedback @olafurpg :pray: I'm merging.


Replace SQL DB with Azure Storage

Azure Storage general-purpose v2 supports tables, would be much cheaper than SQL or Cosmos, and therefore seems better suited for this small project.
Could also simplify az functions through function bindings (and [Table] attribute).
However, it seems there isn't a nice way to emulate it in Docker yet, so i'll wait for Azurite to support it.

Table storage in azurite still seems far off. But, switching the Azure SQL Database to serverless should make it practically free for this little thing :smile:


Journalism: WorstAttribute feature

Feature whose value is an int, indicating the value of the lowest attribute the player has.
Includes one subfeature:

Names : A list of the names of all attributes that had the lowest value (e.g. a player could have both Tech and Trust at 2, and all others >= 3).

Only runs at the session and player level.

71233f9b777c747c86d47bc667bd7979281841ef


undefined reference to `demo_yolo'

Hi,
I receive the following error after I try to build the ros package:
Errors     << dn_object_detect:make /home/markob/any_ws/logs/dn_object_detect/build.make.001.log
/home/markob/any_ws/src/dn_object_detect/lib/libyololib.so: undefined reference to `demo_yolo'
collect2: error: ld returned 1 exit status
make[2]: *** [/home/markob/any_ws/devel/.private/dn_object_detect/lib/dn_object_detect/dn_object_detect] Error 1
make[1]: *** [CMakeFiles/dn_object_detect.dir/all] Error 2
make: *** [all] Error 2
cd /home/markob/any_ws/build/dn_object_detect; catkin build --get-env dn_object_detect | catkin env -si  /usr/bin/make --jobserver-fds=6,7 -j; cd -

Can you help me with this error?
In addition, I disabled the GPU line in the Makefile of darknet by setting GPU=0. Is your code also working on the CPU without the CUDA dependencies?
Thank you in advance.
Best,
Marko

I have the same problem too! do you get the answer? Thanks

If you use the patched darknet code included as a submodule in this repository, you should not encounter this error. I haven't tried to compile without CUDA. I should when I get some spare time.

I used the patched darknet code and I followed the instructions in the readme.

I am unable to reproduce your error. I use the following commands to do a successful build.
Assume a working catkin workspace called /home/kunle12/dev/catkin_ws
cd  /home/kunle12/dev/catkin_ws
git clone --recursive https://github.com/kunle12/dn_object_detect
cd dn_object_detect/darknet/
mkdir build;cd build
cmake -DCMAKE_INSTALL_PREFIX=/home/kunle12/dev/catkin_ws/src/dn_object_detect ..
make install
cd ../../..
catkin_make dn_object_detect


feat(BindingSystem): add binding system API

@EisenbergEffect this is a work in progress.  Let me know what you think.
The current design would require us to add the following somewhere in aurelia framework:
import {initializeBindingSystem} from 'aurelia-binding';

initializeBindingSystem(container);

To use this outside of an Aurelia application, initializeBindingSystem() would be required although we could refactor the BindingSystem class to make the static methods lazy initialize when they're called to support the external or unit-test scenarios more cleanly.
My thinking was this would preserve our ability to have one instance of services like the TaskQueue and EventManager in the container while still enabling external and unit-testing scenarios.
Remaining work:

remove DI dependency in package.json
fixup templating where there is a dep on the old BindingExpression.create API

related to https://github.com/aurelia/binding/pull/171

@EisenbergEffect how does it look now?  Any closer?  The method names are verbose.  My intent was to clear up the confusion around observing property-changes vs observing collection-mutation.
Still need to tidy this up and sync up the unit-tests.

Refactored and fixed up the tests- ready for merge if it looks good.  What do you think of the method names?  Too verbose?

Looks good except one thing. I think the assertIntialize function has a bug. It's trying to call initialize which doesn't exist in the module scope...unless I'm seeing something incorrectly. Once that is fixed, please merge.

good catch- added tests for that as well.


Update GeoLite db URL as logstash 5.x doesn't support old database fo…

As defined in Logstash 5.x no longer supports .dat format Geo Databases

Thanks!


Issue with field desc

Hi,
First, thx for your provider, it is very useful!
The issue I have, is a template with a desc field (Notes inside proxmox ui).
During the clone process, the value is set to something temporary value clone and then set back to the template value.
Everytime, I launch terraform apply, it shows a diff because of this field.
I do not know if you need additional informations, if yes ping me ;)

hi @DblK
I find that Proxmox like to add a new line at the end of the notes created via API, which may be the source of your diff.
Try adding \n to your template (may need escaping for terraform to handle it)
Example from my use case with Ansible groups:
  description = "{\"groups\": [\"consul\"]}\n"

hi @DblK
I find that Proxmox like to add a new line at the end of the notes created via API, which may be the source of your diff.
Try adding \n to your template (may need escaping for terraform to handle it)
Example from my use case with Ansible groups:
  description = "{\"groups\": [\"consul\"]}\n"


Wheel Handler Gamepad Connection

This code makes some minor changes to chaos monkey.
It also changed some AHHHHHH
And inverted the y-sticks in the gamepad class because the FTC people have up as negative one for some reason?!
On a more important note, it also adds the WheelDriver component, which connects to the gamepad to yknow.
Drive.
Please send help I cannot write anymore

Looks good to me


[FLINK-33874][runtime] Support resource request wait mechanism at DefaultDeclarativeSlotPool side for Default Scheduler

What is the purpose of the change
Support resource request wait mechanism at DefaultDeclarativeSlotPool side for Default Scheduler
Brief change log

introduce slot.request.max-interval
implement the mechanism.

Verifying this change
This change is already covered by added corresponding test cases.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure


changes to obsidian filename to not change Page title in Confluence

It seems that whatever the filename is when you first publish, is permanent.  Changing the filename later and republishing does not result in the Page name being changed.
While fixing this, I would suggest introducing a frontmatter setting, perhaps something like connie-publish-title to follow your existing convention.  If set,  the filename in Obsidian would not be used at all, and the frontmatter string would be used instead of the filename.  Of course, changing the value of the frontmatter should result in page renaming in Confluence.

At the moment publish looks up the pages at publish time to find if there is a matching page under the parent page id to update if not then create a new page.
To allow for page renames you would need to track the page id in the yaml frontmatter after publishing. Would this be acceptable for this to be supported? It would mean all pages published to confluence would have their page id written into the page.
What would you expect to happen if you had a page id that was deleted and tried to be published again or a page id that didn't exist on that site?

To allow for page renames you would need to track the page id in the yaml frontmatter after publishing.

I certainly think this is a good approach.  Sure, it means you have to publish once, and then copy the ID, but it's a one time operation.  Since I would also like to be able to publish pages individually, and independently of their location in the confluence page structure.

What would you expect to happen if you had a page id that was deleted and tried to be published again or a page id that didn't exist on that site?

Throw a helpful error: confluence page id's are "Boss" - Obsidian can learn them, and use them, but cannot assign them.  I don't see another solution.

Additionally, I see that I was wrong in my original comment - it does not refuse to change, it - as you said - publishes another child page with the updated title.

Initial PR for support https://github.com/obsidian-confluence/obsidian-confluence/pull/53 :)
This will add title support, pageid support and preventing moving files once they are moved.

Please test in v1.3.0


Cleanup SignalR Windows Auth with browsers

cc @davidfowl
Maybe we should check all the browsers listed in https://docs.microsoft.com/en-us/aspnet/core/signalr/supported-platforms?view=aspnetcore-3.1#javascript-client and put their status here?

Yep

@BrennanConroy is that better or worse?


Build overcloud host and IPA images using packages from Ark

Override package mirrors for both Ubuntu Jammy and Rocky Linux 9 overcloud images so that they are built from package snapshots defined in pulp-repo-versions.yml.
Also build IPA from the same mirrors, although this fails currently because python3.10-dev supplied by Ark in the snapshot defined in pulp-repo-versions.yml is too old. Updating the package snapshot to a newer version means dependencies can be solved and the image is able to be built.
Use the authenticating Pulp proxy for everything that touches packages on Ark, to prevent credentials accidentally leaking into built artefacts.
Requires: https://github.com/stackhpc/stackhpc-image-elements/pull/67

TODO: Some automation to trigger a host/IPA image build when pulp-repo-versions.yml is updated, ?


[CORE][Minor]:remove unused import in SparkContext.scala

What changes were proposed in this pull request?
Found one unused import in SparkContext.scala while auditing ML code.
Remove it.
How was this patch tested?
Remove the unused import and compile successfully

Test build #68494 has started for PR 15846 at commit f14fba5.

hm thanks for submitting but we also want to discourage change this small without any user facing benefit.

OK. I will close it. Thanks!

Test build #68494 has finished for PR 15846 at commit f14fba5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68494/
Test PASSed.


No execution started when the WPS ID is more than 42 characters

When trying to start an execution for an application where the WPS ID (<workflow_id>_<s:softwareVersion> is longer than 42 characters, nothing happens. Deploying the process is fine, but when the execute request is sent, nothing happens. No Calrissian pods are created, no error is shown in the ADES logs.
k8s version: 1.22.5
ADES: 1.1.2

Waiting for input from @bbrauzzi

Hi @JannesLathouwers
I am sorry but I was not able to reproduce the issue. Could you please provide more details of your job submission?
Note that the process id (<workflow_id>_<s:softwareVersion>) differs from the job namespace name in k8s.

Hi @bbrauzzi, sorry for the late reply.
The issue occurred for me when providing a CWL with the following values:
#!/usr/bin/env cwl-runner
$graph:
- class: Workflow
  id: this_identifier_is_too_long_for_some_reason  # <-- this seems to be an issue
...
- class: CommandLineTool
  baseCommand: python3
...

cwlVersion: v1.0

$namespaces:
  s: https://schema.org/
s:softwareVersion: 3.0.7
. . . .

The ADES concatenates the id and the s:softwareVersion into the identifier. When this result is more than 42 characters, the ADES fails silently for me. I also tried with a shorter id but a longer s:softwareVersion (while the total length is still over 42 characters). This also fails.
When this identifier generated by the ADES is shorter than 42 characters it works as expected.

Thank you for the details, I was finally able to reproduce the issue.
It has been addressed internally ( EOEPCA-733 ) and we'll keep you updated.

The fix has been implemented and has been included in the following Ades releases:

eoepca/proc-ades:2.0.4
eoepca/proc-ades:1.2.4


Add community sections to the main ReadMe

Motivation and Context

Found it hard to find documents related to Chat Copilot and Semantic Kernel from the repo.
Description

Add links and guides to the main ReadMe so that people can find things more easily.
Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

This is great, thank you for this update!


FIX avoid width and hight check for objectgroup in TiledMap

The inner class ObjectGroup still checks for hight and width on an object layer and fails if not available. This attributes are no longer supported by Tiled as they make no sense.
Useage needs to be removed
Offering a PullRequest

Hi, Slick hasn't really kept up with Tiled changes so it's indeed quite possible that there are inconsistencies like that.
Happy to take a pull request, but I guess it would need to cater for old Tiled format vs. new to be backwards compatible?

I believed Slick was dead and abandoned? If not, then it would be great to use this Repo to keep it up to date.

Oh yeah it is abandoned. I setup this repo initially to publish Slick on Maven Central, but since then a few people have been submitting pull requests with bug fixes. Usually I merge them and publish a new version on Central.

Fair enough. I've switched to https://github.com/mini2Dx

No worries, looks like a cool library!

Yes, it's quite a "slick"  ;) library from a very active and responsive dev


lxd/main_activateifneeded: Clarify 'No DB' debug statements

Specify the database name in each debug statements.
Signed-off-by: Michael Jeanson<EMAIL_ADDRESS>

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"


Use tags for flux repository

What does this PR do?
Currently Flux is using the tag with the branch naming, and this PR should fix
🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted. Consult the CONTRIBUTING guide for submitting pull-requests.
Motivation

More

[x] Yes, I have tested the PR using my local account setup  (Provide any test evidence report under Additional Notes)
[x] Yes, I ran pre-commit run -a with this PR
[x] Yes, I have added a new example under examples to support my PR (when applicable)
[x] Yes, I have updated the Pages for this feature

Note: Not all the PRs required examples and docs.
For Moderators

[x] E2E Test successfully complete before merge?

Additional Notes

Hotfixing this as main is broken


Improve automated tests

[ ] use react-native-testing-library
[ ] setup on circle ci

follow this: https://github.com/entria/ReactNavigationRelayModern/pull/32

if/when we set up on CircleCI, it's possible to do ios builds. We need to make sure we do debug and release builds I think as it's clear that they can have completely different header, framework, and link search paths. I've seen the same in other libraries where debug is good and release doesn't work...

@MateusAndrade ☝

[ ] it should run build on ci

It should run debug and release build on CI :-). That's the key. Note that lots of repos (like netinfo) have the ios build working well, but I don't think even they do the release version. Not sure it is possible (with all the cert management required...ugh iOS) but it should be a goal. That's what got us with v1.2.0 - debug was fine but iOS release build blew up


User can switch theme

Context
Describe / explain why we should do this: motivations, context or other info. Keep it brief and simple - PM
User can click on this menu item to switch theme:

User flow
Describe the user flow using user stories so the end result is super clear - PM
How to
List the step-by-step to get it do if needed - PM
Technical details
Give the technical insights so anyone in the team can tackle the tasks - Dev
QA testing
Does this task require some QA tests ?
If yes, explain how to validate it

@SlnPons maybe it's not in milestone Wallet#2?

It's it "super easy" to do this thanks to the UI kits and all composants that are already set in both themes?
I was under the impression that it was one of the strength of such a ui-kit.
cc @mazmassa maybe I misunderstood. Let me know if it's not trivial.

You are tight, all components exists in both themes. We just need to create the system to switch from one theme to another. I think it's "quite easy"

We just need to create the system to switch from one theme to another. I think it's "quite easy"

Indeed! All was done to support the switch between themes. However, we must give the users the way to do. That's what @Thykof is mentioning. This will happen when clicking on the menu option... and this was not done yet.
BTW, @Thykof we have a good example done by Lilyan (http://localhost:6006/?path=/story/components-theme-mode--theme-mode). The way we setup in the localStorage and etc...
Let me know if you need help to do that.

less than a day

Ok thanks a lot to both of you! It's defo a nice-to-have then. Not a priority for now, I would say:

send coins
settings
receive coins
teaser screens (for non-available features)
switch mode light / dark

Done


Run: Onenote notes search (feature request)

It would be super great to be able to search Onenote notes.
Open search with ALT+SPACE -> type: "random search" and it would show from which Onenote keyword is found and clicking the item opens it using Onenote app.

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue #3603
Thanks for helping us to make PowerToys a better piece of software.

Duplicate of #3603


Add SortNameCommand

Describe your changes
Add SortCommand and SortNameCommand so that the list can be sorted by name
Issue ticket number and link
Closes #125
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.

Codecov Report
Attention: Patch coverage is 75.67568% with 9 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

...rc/main/java/seedu/address/model/ModelManager.java
70.58%
2 Missing and 3 partials :warning:

...rc/main/java/seedu/address/ui/PersonListPanel.java
0.00%
2 Missing :warning:

.../seedu/address/logic/parser/SortCommandParser.java
88.88%
0 Missing and 1 partial :warning:

src/main/java/seedu/address/ui/MainWindow.java
0.00%
1 Missing :warning:

Files with missing lines
Coverage Δ
Complexity Δ

...java/seedu/address/logic/commands/SortCommand.java
100.00% <100.00%> (ø)
1.00 <1.00> (?)

.../seedu/address/logic/commands/SortNameCommand.java
100.00% <100.00%> (ø)
2.00 <2.00> (?)

.../seedu/address/logic/parser/AddressBookParser.java
100.00% <100.00%> (ø)
16.00 <0.00> (+1.00)

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)
3.00 <0.00> (ø)

...edu/address/model/person/PersonNameComparator.java
100.00% <100.00%> (ø)
2.00 <2.00> (?)

.../seedu/address/logic/parser/SortCommandParser.java
88.88% <88.88%> (ø)
4.00 <4.00> (?)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...rc/main/java/seedu/address/ui/PersonListPanel.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
91.93% <70.58%> (-8.07%)
25.00 <3.00> (+2.00)
:arrow_down:


Publish agenda for HTTP sub-team kick-off

@swift-server/stakeholders @LoganWright has organised the HTTP sub-team kick-off for Monday November 21st. Details of the time, agenda and webex are in this PR.

Merging as the meeting has happened.


Search functionality is making lot's of request to the server

Is your feature request related to a problem? Please describe.
Search functionality is making lot's of request to the server instead some search engines can be used to improve the performance of web-app
Describe the solution you'd like
Add some search engines like elastic search
Describe alternatives you've considered
Or we can cache the result to decrease the search request to the server
Additional context
Add any other context or screenshots about the feature request here.

@Tushar19varshney Let me know your view

@Rupeshiya in my opinion elastic search is useful for a very large data pool whereas in this app a user may not have to search through a very large data set also  elastic search has some disadvantages that can’t be easily bypassed when it is used as a storage. First of all, it isn’t ACID. It doesn’t support transactions so it isn’t possible to rollback first stored document if saving of a second fails.


feat: ETH connector migration.

@paouvrard Sorry for jumping in, but I'd like to invite you to participate in the Race of Sloths - a fun and gamified open-source contributions program. Consider mentioning @race-of-sloths user in your github comment or PR description to join!
See how the flow works here: https://github.com/near/nearcore/pull/11778


125 potcar metada is not parsed

Adding parsing of the metadata of the potential directly to a potcar parser so that we do not have to depend on pymatgen.
Closes #125.

Thanks a lot @JPchico.

Great! Thanks for implementing this!

No worries! Feel free to review it and merge it when you can :) I cannot do it myself

I think @espenfl needs to approve the PR to be able to merge.

Hi,
looks good to me upon a quick glance. One thing I would change is the testing. I think there is nothing against using one of the potentials that are publicly available on the VASP wiki for the examples. E.g. the Si potential here
But this is not necessarily needed.
I am struggling this week with a migraine, but I am happy to take a closer look next week and add more tests maybe?
Cheers, M

Thanks @MichaelWolloch. Then we merge this one now and improve downstream. All the best with the recovery.


”Isverna” is listed in the drop-down menu

Description:
"Isverna" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Isverna" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Isverna, MEHEDINȚI".


Grid auto re-register

If a grid hub was restarted, then SelendroidStandaloneServer will re-register itself again.
Commit details:
a task created for self registration - first check grid if the node already registered, and re-register if not.
then the task is scheduled for execution (recurrence is configured via -registerCycle argument)
A nice feature that I found useful when having a farm of selendroid-servers. :-)

I don't know too much about grid. It would be great if someone with more experience could review this as well.
/cc @selendroid/contributers

Since Selendroid add/remove hardware device dynamically, you could not use same logic as Selenium Grid for "isAlreadyRegistered()" check.

there seems to be possible race condition with small registerCycle.

As @koichirok said:

Since Selendroid add/remove hardware device dynamically, you could not use same logic as Selenium Grid for "isAlreadyRegistered()" check.

Well, I'd say that actually you could. :-)
See that the reference to instance of SelendroidConfiguration is passed as a first parameter to SelfRegisteringRemote constructor.
So just plug/unplug your devices and call performRegistration method to notify grid hub.
(I actually use it for notifying grid hub if one of emulators crashed and that selendroid node has less free browsers while crashed emulator is rebuilding and restarting (this code is not published yet. not sure if it could be of use)).

@frolix wrote:

As @koichirok said:

Since Selendroid add/remove hardware device dynamically, you could not use same logic as Selenium Grid for "isAlreadyRegistered()" check.

Well, I'd say that actually you could. :-)
See that the reference to instance of SelendroidConfiguration is passed as a first parameter to SelfRegisteringRemote constructor.
So just plug/unplug your devices and call performRegistration method to notify grid hub.
(I actually use it for notifying grid hub if one of emulators crashed and that selendroid node has less free browsers while crashed emulator is rebuilding and restarting (this code is not published yet. not sure if it could be of use)).

I've got it. then, you will make another PR for this in the future, don't you?

@asm89, @koichirok, @DominikDary, please take a look. Tried to fix all the findings.

:+1:


Roadmap: Task 3b: Visual layout to Python workflow

Lay out workflow visually as configurable objects and return Python workflow

Box flow was investigated for this task. The need for it has been superseded by other capabilities. This has been fulfilled.


docs: switch homepage example to GCS

Fixes #615
I think a GCS example might more exciting to the Node developer.
Also tweaked the style a bit to make it pop out a little more.

Preview: http://stephenplusplus.github.io/gcloud-node-gh-pages-again/

grape-spaceship-123 lives on!


"Method VerumConsilium\Browsershot\PDF::SetTitle() does not exists"

Laravel: 5.6.38
Voyager: v1.0.11
PHP: 7.2.2
Database: Mysql 10.1.30
I've been trying to integrate browser-shot into laravel voyager and I really don't know how I would do this. I have little knowledge about facades, node.js and puppeteer. As of now, I've successfully downloaded browsershot to my laravel project, using composer require verumconsilium/laravel-browsershot.
I've also installed puppeteer in my project folder using npm install puppeteer. In my Controller, Here are some of the contents of my Controller:
margins(20, 0, 0, 20)
            ->download();
    }
   ...
}
?>
I've also downloaded the files from this repository to my project folder(but not all). What am I missing?
P.S. I am deeply sorry for those inconvenienced by my little knowledge about this things. Please understand me.

Hi, the package doesn't provide a SetTitle method. If you want to set the title that is displayed in the browser tab, simply put a <title> tag inside the <head>  in your html.
I'm closing this issue since it's not really a problem with the package but feel free to keep commenting if you face any more issues


Update README.md

Description
added "The Verge" website in the news section of the README.md

Checklist

[ ] My change follows the Contributing Guidelines yes
[ ] I have added only one new link to the list. yes
[ ] I have sorted the link alphabetically under the related section. yes

Please consider my changes Thank You


[Android] Fix the crash issue when launch cordova.

Reserved object was created, but it's content is empty.
The condition of judgment prevents the initialization for embedded mode.
BUG=XWALK-4257

@lincsoon @axinging PTAL, thanks.

LGTM

CordovaActivity inherits from XWalkActivity. If the developers use XWalkActivity, they must follow the rule that invokes embedding API in onXWalkReady(). So the usage described in XWALK-4257 is a wrong way, not a bug.
On the other side, initEmbeddedMode() is for the class which doesn't use XWalkActivity to keep the backward compatibility with old code. It can't coexist with XWalkActivity, so the check you concern is to make sure initEmbeddedMode() won't be invoked under XWalkActivity. Your modification will violate this intention.
My suggestion is that this commit is not necessary, and we should use CordovaActivity with the recommended way just like the default template.

Thanks for the explanation, I will close XWALK-4257.


[dt-rfc] Non existing draft or RFC raises attribute error

Describe the issue
In dt-rfc branch, the API call /api/rfcdiff-latest-json/ for a non-existing draft/RFC generates an attribute error.
Example:

https://dt-rfc.dev.ietf.org/api/rfcdiff-latest-json/rfc9999
https://dt-rfc.dev.ietf.org/api/rfcdiff-latest-json/draft-ftie-foo-bar-00

Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

addressed by #6681


bug: wrong identation

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Auto identation in one-line do loop is wrong:
  do ic=1,2
    c(ic) = 0
  end do
  call xxx        ! normal identation

  do ic=1,2; c(ic) = 0; end do
    call xxx       ! additional wrong spaces in this case

Screenshots
No response
Expected Behaviour
I expect the formatter to recognize the end of the loop, even if it is on the same line
  do ic=1,2; c(ic) = 0; end do
  call xxx       ! beginning in the same position

Version of Modern Fortran
v3.4.2022112801
Version of Visual Studio Code
v1.73.1
Platform and Architecture
Ubuntu 20.04 Linux x64 5.15.0-53-generic
Additional Information
I use findent with following parameter:
"fortran.formatting.findentArgs": [
"-i2"
]

Hi @as-92, please report bugs for findent to the findent repo: https://github.com/wvermin/findent


[dashmachine] Remove dashmachine or change to fork

Considering the steady increase of issues, a huge backlog of PR's without any activity and no commits for months:
This project is clear vaporware/abandonware.
The consequences are potential security risks.
There are also great viable alternatives to this, like Heimdall or Organizr.
Taking into account this is already a VERY niche application: Either the used container needs to be changed to an active fork, or the application needs to be removed.

I would agree. If the upstream developers are no longer building the app it makes sense to deprecate / remove.


Doesn't properly check invalid input in tnetstring.parse

Problem Description
In the condiction branch data_type == ord(b'}')of tnetstring.parse(), it doesn't properly check invalid data in the parse loop.
Steps to reproduce the behavior:
from mitmproxy.io import tnetstring
t = b'51:11:12345678901#4:this,8:true!0:~,4:true!0:]4:\\x00,~}'
tnetstring.loads(t)

TypeError: unhashable type: 'list'

System Information
Mitmproxy: 7.0.0
Python:    3.8.2
OpenSSL:   OpenSSL 1.1.1k  25 Mar 2021
Platform:  Linux-5.4.0-kali3-amd64-x86_64-with-glibc2.29

I'm not sure if I understand what's the problem here. Is there any way how you can trigger this issue from mitmproxy? Lists aren't supported as dictionary keys in our tnetstring implementation.

I'm not sure if I understand what's the problem here. Is there any way how you can trigger this issue from mitmproxy? Lists aren't supported as dictionary keys in our tnetstring implementation.
It's somewhat expected that tnetstring.loads borks on invalid input, but I don't understand where that is a Denial-of-Service issue.

sorry for the misunderstood description, after testing now I do, I found this issue won't make the program down.

Thanks, that clears things up! 😃


Cannot honor AWS_PROFILE env var

Hello, first of all, thanks for providing this great tool!
I was trying to find a way to not to have to enter credentials for the KMSConfig, and it seems that if I don't enter the credentials key, the default local access/secret are applied. I wanted to do that so my team doesn't mistakenly push credentials to the project repository, since the idea was to version control config.json.
Having the default local value applied to access/secret makes it impossible for aws-sdk to pick the credentials from the local aws-cli configuration (i.e. ~/.aws) when about to consume the KMS service. If the tool wouldn't apply those values, a standard env var such as AWS_PROFILE could be honored.
Maybe cognito-local could allow one extra config in KMSConfig, probably "omitDefaultCredentials": true/false or similar naming, and then don't apply those defaults if that's true?

Hey @gfzabarino, I don't personally use this bit of functionality but I think the intent was for it to be combined with a local KMS emulator so committing credentials wouldn't be a problem.
Just so I understand how your use case: you have cognito-local with local lambda functions but want to use the real KMS service to encrypt/decrypt the code?

Just so I understand how your use case: you have cognito-local with local lambda functions but want to use the real KMS service to encrypt/decrypt the code?

hey @jagregory, that’s exactly what I’m doing, we have a dev kms that seems to be easier to get via serverless vs setting up a local KMS emulator.


[release-1.8] Add release note for duplicated probe rewrite headers

Adds release note for http probe rewrite fix backported earlier (#30955)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@Monkeyanator: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.8
2704fae9835c9c3671c94b681d565866c757d1ef
link
/test unit-tests_istio_release-1.8

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

/retest


Test linux and osx in travis

The upside is that we can see that exercises work on multiple platforms. The downside is that it seems to take a very long time for the OSX builds to start...

The EXERCISM env var has been set in travis so it's no longer needed as part of the command.

Leaving this for now until significant delay on osx builds is resolved.


Tp 929 remove automatic triggering of workbasket export

Removing code calling upload_workbaskets task from WorkBasket model approve method
Updating existing tests
Adding WorkBasket to admin
Adding custom button Workbasket list page. This button calls the upload_workbaskets task. (See screenshot below)
Adding tests for custom button

A general comment about commit messages: I think a good convention to follow is to have a commit message complete the sentence "When merged, this commit will ...".
Your commit messages are good, but the -ings are a bit superfluous.
Of course when you merge this PR you will squash all the commits - right? 😉

@simonwo I was already adding a WorkBasket admin view, so now I'm waiting for this PR to merge, otherwise we'll have conflicts

Probably out of scope for this ticket: We should probably be tracking the upload tasks somewhere in the admin.
It's fine that we schedule new upload_workbaskets tasks - if there are none to upload it will do nothing.
It would be nice to see if there already was one and show that.
But again:  not really needed for this.

Codecov Report

Merging #375 (346544c) into master (b4fe066) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #375      +/-   ##
==========================================
+ Coverage   91.44%   91.46%   +0.02%
==========================================
  Files         260      262       +2
  Lines       14525    14560      +35
  Branches     1089     1088       -1
==========================================
+ Hits        13282    13317      +35
  Misses       1044     1044
  Partials      199      199

Impacted Files
Coverage Δ

workbaskets/models.py
93.63% <ø> (-0.12%)
:arrow_down:

conftest.py
95.25% <100.00%> (+0.05%)
:arrow_up:

workbaskets/admin.py
100.00% <100.00%> (ø)

workbaskets/tests/test_admin.py
100.00% <100.00%> (ø)

workbaskets/tests/test_models.py
100.00% <100.00%> (ø)

workbaskets/tests/test_views.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b4fe066...346544c. Read the comment docs.


The undo Button should be Optional

A lot of users complaining about The undo Button that is slowing down the game and it is making it less challenging.  It would be optimal to make the undo button optional.

I will try to add this feature as my first OS contribution.


Signing certificate expired

Not sure if this causes any immediate problems, but the software signing certificate for UTM has expired. My copy is from the App Store.
https://developer.apple.com/support/expiration/

I don't believe this should be an issue for users as the timestamp for when the app was modified was before the expiration date. Let me know if you get an error from macOS.


same_payer for singletons

Seems that same_payer isn't working with singletons?
It's always requesting the authority of the previous payer after the initial payer assumes row.
This works fine
SingletonTable(_self, scope).set(something, caller_account);
After the above, this would not.
SingletonTable(_self, scope).set(something, same_payer);

eosio::same_payer is defined in multi_index.hpp.  If you want to use it with singleton, include multi_index.hpp.

I've already got that included since I'm using multi_index in other places.
The issue isn't it looking for the same_payer variable ( or i'd just replace it with name("") ), it's that it complains about wanting the authority of the original row owner when trying to modify the row from a secondary user.
"{"code":500,"message":"Internal Service Error","error":{"code":3090004,"name":"missing_auth_exception","what":"Missing required authority","details":[{"message":"missing authority of eosio","file":"apply_context.cpp","line_number":151,"method":"require_authorization"},{"message":"pending console output: ","file":"apply_context.cpp","line_number":72,"method":"exec_one"}]}}"

The contract can set _self (contract owner) or an authorizer of action as ram payer. If you deployed your contract to alice and eosio paid singleton RAM, bob's action can change its ram payer to bob or alice only. (not eosio)

I don't want it to change the payer. I want it to stay the same payer as it already is, and have it modified by other accounts in the same way that a multi_index does.
bob does action which creates singleton row in bob's name -> alice does action, and modifies bob's row without changing payer.

Hmm, after reading your comment, I looked for the source code handling ram payer.
https://github.com/EOSIO/eos/blob/master/libraries/chain/apply_context.cpp#L490-L500
https://github.com/EOSIO/eos/blob/master/libraries/chain/apply_context.cpp#L395-L404
It seems that if you not change the occupied RAM size by updating singleton data, update_db_usage() is not called, so evaluation for ram payer authorization doesn't occur. Did your update on singleton data change the occupied RAM size?

Indeed, when doing something that wouldn't change ram size ( adding to existing vector ) it doesn't throw errors, but when doing something like adding to a vector it throws.

It seems this is the offending line https://github.com/EOSIO/eos/blob/master/libraries/chain/apply_context.cpp#L400
Which would occur both in cases of lower or higher RAM consumption.

Yes, I think this limitation was introduced after the "RAM Kidnapping" attacks started happening and it's no longer possible to grow rows without some sort of permissioned system. Shame, as that's incredibly limiting for dynamic rows that should grow. For instance, with this contract I'm working on part of the design is that users assume full costs of that RAM and get tokens in return for it, which makes this assertion counter-productive.

I'm going to close this ticket out though as it seems this is "working as intended".


Display messages for csslint.

Simplify patterns and specify compact format for csslint cli.
This fixes #55 (or at least makes it functional).

Merged, cheers :)


file not found

Hi, I'm trying to build the code but I'm getting this error.
"Cannot find source file:
bin/singlewire.pio.h" what am I doing wrong?

Change /bin to /build in cmake and the source.


Desk research of navigation discussions (Mia)

https://docs.google.com/document/d/1WMMgbnxriRekaDPfrudXDOyyCMgFBky6ymbK_vsNn7c/edit#heading=h.pe1zr62pm8n8

This is completed and in the above linked document


Added WCF app context switch for 4.7

Added WCF app context switch for 4.7
Suggested Reviewers
//cc @jimcarley, @mairaw

@rpetrusha there are some merge conflicts here


[Bug]: The execution of data archiving tasks requires confirmation interaction

ODC version
odc432
OB version
。
What happened?
The execution of data archiving tasks requires confirmation interaction

What did you expect to happen?
。
How can we reproduce it (as minimally and precisely as possible)?
。
Anything else we need to know?
No response

Fixed In Commit fcd5430ebbd9cf2f6445dee12800f2f3d181ff22
------
畅晚
Fri Sep 20 15:57:39 2024 +0800

pass