Disable auto mount service account token in the ACME HTTP01 pod

Fixes: https://github.com/cert-manager/cert-manager/issues/5254
Here are the failed HTTP01 tests, after adding the Kyverno automountServiceAccountToken policy:

https://prow.build-infra.jetstack.net/view/gs/jetstack-logs/pr-logs/pull/cert-manager_cert-manager/5754/pull-cert-manager-master-e2e-v1-26/1618585611369713664

Here are examples of failing attempts to create the HTTP01 solver Pod, from the cert-manager controller manager logs:
2023-01-26T12:34:05.286102563Z stderr F E0126 12:34:05.285984       1 controller.go:167] cert-manager/challenges "msg"="re-queuing item due to error processing" "error"="admission webhook \"validate.kyverno.svc-fail\" denied the request: \n\nresource Pod/cert-manager/cm-acme-http-solver-jtnj2 was blocked due to the following policies\n\nrestrict-automount-sa-token:\n  validate-automountServiceAccountToken: 'validation error: Auto-mounting of Service\n    Account tokens is not allowed. Rule validate-automountServiceAccountToken failed\n    at path /spec/automountServiceAccountToken/'\n" "key"="cert-manager/e2e-conformance-hdbsg-3383192779-894339866"
2023-01-26T12:34:05.286468946Z stderr F I0126 12:34:05.286371       1 pod.go:71] cert-manager/challenges/http01/ensurePod "msg"="creating HTTP01 challenge solver pod" "dnsName"="<IP_ADDRESS>" "resource_kind"="Challenge" "resource_name"="testcert-1-4146183125-3892227383" "resource_namespace"="e2e-tests-certificates-szxph" "resource_version"="v1" "type"="HTTP-01"

Here's an example of cert-manager components failing to connect to the K8S API server, after I set automountServiceAccountToken: false but before I manually added volumes and volume mounts to make the API token available in the Pod:
richard@LAPTOP-HJEQ9V9G:~$ kubectl -n cert-manager logs cert-manager-cainjector-79fdf966d4-4cj7g
I0125 20:49:39.356754       1 feature_gate.go:249] feature gates: &{map[]}
I0125 20:49:39.356935       1 start.go:136] "starting" version="v1.11.0-beta.0-64-g12618cd0ab606a" revision="12618cd0ab606a0d1069361890ceac2f26c03c52"
E0125 20:49:39.357268       1 config.go:133] cert-manager/controller-runtime/client/config "msg"="unable to load in-cluster config" "error"="open /var/run/secrets/kubernetes.io/serviceaccount/token: no such file or directory"
E0125 20:49:39.357296       1 config.go:177] cert-manager/controller-runtime/client/config "msg"="unable to get kubeconfig" "error"="invalid configuration: no configuration has been provided, try setting KUBERNETES_MASTER environment variable"

Disable automountServiceAccountToken in the ACME HTTP01 solver Pod

Thanks @tspearconquest for your feedback. I'm happy to make those changes. You originally described them "recommended" but are they documented anywhere? Or are there examples of other projects that use those defaults?

Thanks @tspearconquest for your feedback. I'm happy to make those changes. You originally described them "recommended" but are they documented anywhere? Or are there examples of other projects that use those defaults?

I can. Please see: https://www.b2-4ac.com/rabbitmq-metrics-in-elasticsearch/
Also you can find it mentioned in several different issues across github with a quick google. Use the query: kubernetes volume "defaultmode: 292" and ensure to include the double quotes around defaultmode: 292

Thanks @tspearconquest and @moolen  for reviewing these changes and the associated documentation changes. I'll ask one of the cert-manager maintainers to review it next.

/kind cleanup


internal: verify ds pods are updated

If a ds is updated and the current method is used to wait for the ds to be ready, the check will pass but in reality the ds pods will still need to be updated. this is because the check doesn't ensure the pods are updated.
Ensure that by adding the condition that the number of up to date pods under the daemoonset equals to the expected and ready number pods.

/assign @ffromani
as per comment: https://github.com/openshift-kni/numaresources-operator/pull/700#discussion_r1485816105

please rebase on top of main because we merged #853

/approve
/lgtm
thanks for splitting this out and fixing very quickly


Enhancements and cleanup in scipy.integrate (attempt #2)

ENH: Enable full_output option for nquad. Currently returns neval only. Convert scipy.integrate.dblquad and scipy.integrate.tplquad to thin wrappers around scipy.integrate.nquad, maintaining API compatibility. Replaces PR #5892, and handles LANL license requirements by hosting a SciPy fork at losalamos/scipy (the "official LANL version" referenced in the disclaimer).

@@            master   #5906   diff @@
======================================
  Files          238     238
  Stmts        43592   43600     +8
  Branches      8179    8185     +6
  Methods          0       0
======================================
+ Hit          34020   34023     +3
- Partial       2602    2604     +2
- Missed        6970    6973     +3

Review entire Coverage Diff as of 0bd577c

Powered by Codecov. Updated on successful CI builds.

I'm running this option by the appropriate people around here. Take a look and let me know what you think about it.

You mention in this PR description a disclaimer about an "official LANL version", so I was expecting to see it somewhere -- but I don't see "LANL" in the diff (or any of the changed files). Is that the idea, that you no longer need to have a disclaimer here at all?

Ahh, neat. Sounds good to me.

Any further comments/concerns?

Are there any holdups here that I should be addressing?

Also resolves #5452.

Hi @pv and @rgommers, would you mind looking at this? It's ready to go in.

Guys, seriously. It's been 3 months. What's the holdup here? This is not a complicated PR.

Based on a quick look:

No test coverage for full_output.
'message' and 'explain' are processed, but not returned. The for loop on line 760 is unnecessary since out is not returned, you can just use the returns from quad directly.
The docstring should not introduce something as "Preliminary", but describe what's there now. What the option actually does is not explained (and Returns section is not updated).
Conversion of tplquad and dblquad to nquad may have performance implications from Python overhead, maybe useful to check.
The full_output probably should not be processed in the inner integrations in nquad, if it is not needed (Python overhead).

I added brief test coverage for full_output, and I got rid of the extra full_output processing, and I fixed the docstring.
As far as the "feel" of the performance change, the two routines are quite similar. Since this change eliminates an otherwise redundant code path in quadpack (originally discussed in PR #2510), it reduces the overall maintenance of an (admittedly largely static) piece of scipy.
Neval actually counts the number of times the integrand function is evaluated, so it actually only increments at the innermost level of nquad. Not really any way to get away from that.

Also, do I need to worry about CircleCI? It looks like it's failing because it doesn't have things that numpy needs, which I don't think has anything to do with me.

Sorry, I think we forgot about it. Please keep pinging us until it gets in.
Sent from my mobile phone.
On Jun 3, 2016 9:28 PM, "Nathan Woods"<EMAIL_ADDRESS>wrote:

Guys, seriously. It's been 3 months. What's the holdup here? This is not a
complicated PR.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/scipy/scipy/pull/5906#issuecomment-223671928, or mute
the thread
https://github.com/notifications/unsubscribe/AABQWOeerWkmrqvwSRofKrb53rZwhumIks5qIIB7gaJpZM4Hl85c
.

Also, do I need to worry about CircleCI? It looks like it's failing because it doesn't have things that numpy needs, which I don't think has anything to do with me.

No, don't worry about CircleCI - its setup isn't correct yet. It will fail until we'll get it to build merge commits instead of the PR branch.

Yeah, I didn't love that test either. I made the recommended changes.

The full_output probably should not be processed in the inner integrations in nquad, if it is not needed (Python overhead).

This comment indicates that @pv expects neval to be the number of outer integrations (I think). The comment is not addressed anyway, full_output is used in all dimensions.
And what is returned now is not comepletely clear, a comment above self.out_dict['neval'] += infodict['neval'] would be useful.

Conversion of tplquad and dblquad to nquad may have performance implications from Python overhead, maybe useful to check.

This seems to be OK, impact is minor:
master:
In [126]: func = lambda t, x: np.exp(-x*t)/t**4

In [127]: %timeit nquad(func, [[1, np.inf],[0, np.inf]])
100 loops, best of 3: 8.45 ms per loop

In [128]: %timeit dblquad(func, 0, np.inf, lambda x: 1, lambda x: np.inf)
100 loops, best of 3: 8.27 ms per loop

PR 5906:
In [2]: func = lambda t, x: np.exp(-x*t)/t**4

In [3]: %timeit nquad(func, [[1, np.inf],[0, np.inf]])
100 loops, best of 3: 8.39 ms per loop

In [4]: %timeit dblquad(func, 0, np.inf, lambda x: 1, lambda x: np.inf)
100 loops, best of 3: 8.47 ms per loop

And for some very simple functions:
In [11]: %timeit nquad(lambda x,y: x**2 + y**2, [[0, 10], [-5, 5]])
1000 loops, best of 3: 295 µs per loop

In [12]: %timeit nquad(lambda x,y: x**2 + y**2, [[0, 10], [-5, 5]], full_output=True)
1000 loops, best of 3: 332 µs per loop

In [13]: %timeit nquad(lambda x,y,z: x**2 + y**2 + z, [[0, 10], [-5, 5], [-1, 1]], full_output=True)
100 loops, best of 3: 7.27 ms per loop

In [14]: %timeit nquad(lambda x,y,z: x**2 + y**2 + z, [[0, 10], [-5, 5], [-1, 1]])
100 loops, best of 3: 6.54 ms per loop

This PR looks OK to merge to me now. @pv if you don't agree with interpretation of neval as number of inner loop evals (https://github.com/scipy/scipy/pull/5906#issuecomment-225365720) please comment. Otherwise I'll hit the green button before 0.18.x branching.

I'll go ahead and re-squash the PR, then.
On Sun, Jun 12, 2016 at 6:39 AM -0600, "Ralf Gommers"<EMAIL_ADDRESS>wrote:
This PR looks OK to merge to me now. @pv if you don't agree with interpretation of neval as number of inner loop evals (#5906 (comment)) please comment. Otherwise I'll hit the green button before 0.18.x branching.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Allright, gave it 5 more days which should be enough - in it goes. Thanks @woodscn (for the PR and for the patience).

Thanks @rgommers for merging! Time for :beers:.


For all offers: update ref_armttk to latest, fix resulting breakage

I observed that the existing ref_armttk allows the current state of the offers to pass arm-ttk but when you update it to the latest ref of arm-ttk there are many failures.  These need to be addressed.

AB#1167684


Release v0.8.0

Issue tracker for the CAA release as part of CoCo v0.8.0
v0.8.0-alpha.1

[x] IBM Cloud amd64
[x] IBM Cloud s390x
[x] Azure, AKS, SEV-SNP, Ubuntu 22.04
[x] Libvirt amd64
[ ] AWS EKS

Kata Containers for CoCo 0.8.0 RC1 - commit 424de1cbfa4e1da9ecf9a56b1d1e1a11a4f339cd

guest-components for CoCo 0.8.0 RC1 - commit 615a46ff16ee8670014946d14e44e85cede82f01 (same commit pinned on kata's 424de1cbfa4e1da9ecf9a56b1d1e1a11a4f339cd)

I've raised PR https://github.com/confidential-containers/cloud-api-adaptor/pull/1559
To cover this sections:

Update the csi-wrapper and peerpod-ctrl go modules to use the tagged version of cloud-api-adapter, by running:
go get<EMAIL_ADDRESS>go mod tidy

in their directories and removing the local replace references if we needed to add them earlier.

of the release doc

I've created pre-release https://github.com/confidential-containers/cloud-api-adaptor/releases/tag/v0.8.0-alpha.1 now, which should trigger the podvm build process to test that.

As reported on Slack, I've manually tested the RC podvm and CAA code for IBM Cloud with amd64 and s390x clusters/peer pods

I tested azure podvm & caa images built on the v0.8.0-alpha.1 rev. looks good, also remote attestation seems to work.

hmm, when running the e2e test suite, other than in my manual test, i'm seeing snapshot errors:
Failed to pull image "nginx": rpc error: code = Unknown desc = failed to pull
  and unpack image "docker.io/library/nginx:latest": failed to prepare
  extraction snapshot "extract-218425411-EbOv
  sha256:ec983b16636050e69677eb81537e955ab927757c23aaf73971ecf5f71fcc262a":
  missing CRI reference annotation for snaposhot 8: unknown

Note the typo "snaposhot", that's from nydus apparently

@stevenhorsman managing to run the alpha.1 release with AWS

The simple pod failed on AWS (AKS) and remaining tests didn't run because it hit the timeout:
=== RUN   TestAwsCreateSimplePod
=== RUN   TestAwsCreateSimplePod/SimplePeerPod_test
    assessment_runner_test.go:190: timed out waiting for the condition
--- FAIL: TestAwsCreateSimplePod (900.53s)
    --- FAIL: TestAwsCreateSimplePod/SimplePeerPod_test (900.53s)

I will get more information.
FYI @bpradipt

@stevenhorsman managing to run the alpha.1 release with AWS

The simple pod failed on AWS (AKS) and remaining tests didn't run because it hit the timeout:
=== RUN   TestAwsCreateSimplePod
=== RUN   TestAwsCreateSimplePod/SimplePeerPod_test
    assessment_runner_test.go:190: timed out waiting for the condition
--- FAIL: TestAwsCreateSimplePod (900.53s)
    --- FAIL: TestAwsCreateSimplePod/SimplePeerPod_test (900.53s)

Hey Wainer, does your cluster have multiple worker nodes by any chance. I've just realised that my nydus verification tests just runs on the first CAA ds it finds, so wouldn't be reliable on a mutli node cluster. I'm looking to fix it soon, but can back out the test failure in the short-term if it's causing issues?

@stevenhorsman managing to run the alpha.1 release with AWS

The simple pod failed on AWS (AKS) and remaining tests didn't run because it hit the timeout:
=== RUN   TestAwsCreateSimplePod
=== RUN   TestAwsCreateSimplePod/SimplePeerPod_test
    assessment_runner_test.go:190: timed out waiting for the condition
--- FAIL: TestAwsCreateSimplePod (900.53s)
    --- FAIL: TestAwsCreateSimplePod/SimplePeerPod_test (900.53s)

@wainersm - Hey Wainer, does your cluster have multiple worker nodes by any chance. I've just realised that my nydus verification tests just runs on the first CAA ds it finds, so wouldn't be reliable on a mutli node cluster. I'm looking to fix it soon, but can back out the test failure in the short-term if it's causing issues?

Hi @stevenhorsman , it is a single-node cluster but good to know tests aren't reliable on a multi-node cluster!
The problem is actually the region the framework deployed the cluster does not support confidential VM, which is now the default since commit e4059a5223bf4a955391b7f87178af9b11809dc2 . I will disable CVM and see if it passes the simple tests.

it is a single-node cluster but good to know tests aren't reliable on a multi-node cluster!

I've created https://github.com/confidential-containers/cloud-api-adaptor/pull/1562 which I hope fixes the issue

@bpradipt @stevenhorsman the status of my tests for AWS is:

I deployed the AKS cluster in a region that doesn't support AMD SEV-SNP so I had to disable CVM (DISABLECVM=true)
Some tests started to pass, others not. The "simple pod" which checks the nydus snapshotter is used failed; it turned out the AKS cluster have containerd 1.6 installed, whereas CoCo 0.8.0 requires +1.7
I managed to install the operator with INSTALL_OFFICIAL_CONTAINERD=true which forced the operator to superseed the system's containerd with its own 1.7
Now the "simple pod" fails to start with a nydus related error:

  Warning  Failed     48m                    kubelet            Failed to pull image "nginx": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/library/nginx:latest": failed to prepare extraction snapshot "extract-101183212-RW23 sha256:ec983b16636050e69677eb81537e955ab927757c23aaf73971ecf5f71fcc262a": missing CRI reference annotation for snaposhot 3: unknown
  Warning  Failed     48m                    kubelet            Failed to pull image "nginx": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/library/nginx:latest": failed to prepare extraction snapshot "extract-967947483-KBeo sha256:ec983b16636050e69677eb81537e955ab927757c23aaf73971ecf5f71fcc262a": missing CRI reference annotation for snaposhot 4: unknown
  Warning  Failed     47m                    kubelet            Failed to pull image "nginx": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/library/nginx:latest": failed to prepare extraction snapshot "extract-758725951-sRqW sha256:ec983b16636050e69677eb81537e955ab927757c23aaf73971ecf5f71fcc262a": missing CRI reference annotation for snaposhot 5: unknown

Does it ring a bell?
My cluster was gone overnight and I am working to re-install it to obtain more information.

Now the "simple pod" fails to start with a nydus related error:

  Warning  Failed     48m                    kubelet            Failed to pull image "nginx": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/library/nginx:latest": failed to prepare extraction snapshot "extract-101183212-RW23 sha256:ec983b16636050e69677eb81537e955ab927757c23aaf73971ecf5f71fcc262a": missing CRI reference annotation for snaposhot 3: unknown
  Warning  Failed     48m                    kubelet            Failed to pull image "nginx": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/library/nginx:latest": failed to prepare extraction snapshot "extract-967947483-KBeo sha256:ec983b16636050e69677eb81537e955ab927757c23aaf73971ecf5f71fcc262a": missing CRI reference annotation for snaposhot 4: unknown
  Warning  Failed     47m                    kubelet

Oh, that's relieving to see 😅
but are we talking about AKS or EKS (on AWS)? AKS clusters should bundle containerd 1.7, I think?

Sorry, I meant AWS EKS :)

Yesterday I tried again on AWS EKS on two versions (1.26 and 1.28) of kubernetes; I got the same missing CRI reference annotation for snaposhot  error on both cases.
Then I deployed a kubeadm cluster via kcli on AWS with CentOS Stream 8 workers. This time I wasn't even able to get the podvm properly runnings (it fails on the "initializing" checks). This is the first time I tried that installation method it might be that I made some mistake.
@mkulke any luck with AKS?

@wainersm: not yet, I'm trying to establish a working baseline with CLOUD_PROVIDER=libvirt on a single-node cluster. kcli also gave me grief, so I aborted that approach and i'm trying to deploy libvirt on single-node cluster created with kubeadm. It will start VMs, but the podvm do not seem to come up properly.
Do you think it makes sense to reference the missing CRI reference annotation for snaposhot to the tracking issue for remote-snapshotter on CC, now that we have this somewhat confirmed?

@wainersm: not yet, I'm trying to establish a working baseline with CLOUD_PROVIDER=libvirt on a single-node cluster. kcli also gave me grief, so I aborted that approach and i'm trying to deploy libvirt on single-node cluster created with kubeadm. It will start VMs, but the podvm do not seem to come up properly.
Do you think it makes sense to reference the missing CRI reference annotation for snaposhot to the tracking issue for remote-snapshotter on CC, now that we have this somewhat confirmed?

Good idea. let me add an entry on that for the peer pod issue.

@wainersm: not yet, I'm trying to establish a working baseline with CLOUD_PROVIDER=libvirt on a single-node cluster. kcli also gave me grief, so I aborted that approach and i'm trying to deploy libvirt on single-node cluster created with kubeadm. It will start VMs, but the podvm do not seem to come up properly.
Do you think it makes sense to reference the missing CRI reference annotation for snaposhot to the tracking issue for remote-snapshotter on CC, now that we have this somewhat confirmed?

Good idea. let me add an entry on that for the peer pod issue.

Done :)

Marked AKS as passing, also manually tested image decryption with snapshotter pulling, works as intended.

I've created https://github.com/confidential-containers/cloud-api-adaptor/pull/1570 to bump the versions to the 0.8 release

After the release I re-tested everything from scratch locally. For IBM cloud on s390x every test passed. For libvirt on kcli on x86 they all passed except nydus pull:
=== RUN   TestLibvirtCreateSimplePodWithNydusAnnotation/SimplePeerPod_test/PodVM_is_created
    assessment_helpers_test.go:162: Called PullImage explicitly, not using nydus-snapshotter :2023/11/14 15:08:25 [adaptor/proxy] CreateContainer: calling PullImage for "docker.io/library/alpine:latest" before CreateContainer (cid: "697963694d8aed2171cd3ea83759e0893c9f752b63c0e4047ffba26fc0c8e97d")
    assessment_runner_test.go:370: Expected to pull with nydus, but that didn't happen
time="2023-11-14T07:08:34-08:00" level=info msg="Deleting pod alpine..."
time="2023-11-14T07:08:44-08:00" level=info msg="Pod alpine has been successfully deleted within 60s"
--- FAIL: TestLibvirtCreateSimplePodWithNydusAnnotation (117.37s)
    --- FAIL: TestLibvirtCreateSimplePodWithNydusAnnotation/SimplePeerPod_test (117.36s)
        --- FAIL: TestLibvirtCreateSimplePodWithNydusAnnotation/SimplePeerPod_test/PodVM_is_created (1.57s)

I've manually checked and we aren't using the nydus pull with libvirt, but given that we've called that experimental I'm not sure if that is a problem at this point?

After the release I re-tested everything from scratch locally. For IBM cloud on s390x every test passed. For libvirt on kcli on x86 they all passed except nydus pull:
=== RUN   TestLibvirtCreateSimplePodWithNydusAnnotation/SimplePeerPod_test/PodVM_is_created
    assessment_helpers_test.go:162: Called PullImage explicitly, not using nydus-snapshotter :2023/11/14 15:08:25 [adaptor/proxy] CreateContainer: calling PullImage for "docker.io/library/alpine:latest" before CreateContainer (cid: "697963694d8aed2171cd3ea83759e0893c9f752b63c0e4047ffba26fc0c8e97d")
    assessment_runner_test.go:370: Expected to pull with nydus, but that didn't happen
time="2023-11-14T07:08:34-08:00" level=info msg="Deleting pod alpine..."
time="2023-11-14T07:08:44-08:00" level=info msg="Pod alpine has been successfully deleted within 60s"
--- FAIL: TestLibvirtCreateSimplePodWithNydusAnnotation (117.37s)
    --- FAIL: TestLibvirtCreateSimplePodWithNydusAnnotation/SimplePeerPod_test (117.36s)
        --- FAIL: TestLibvirtCreateSimplePodWithNydusAnnotation/SimplePeerPod_test/PodVM_is_created (1.57s)

I've manually checked and we aren't using the nydus pull with libvirt, but given that we've called that experimental I'm not sure if that is a problem at this point?

Hi Steve, currently kcli deploys Ubuntu 20.04 nodes with containerd 1.6. As we are not setting INSTALL_OFFICIAL_CONTAINERD=true  it is not replaced with containerd 1.7, thus nydus-snapshotter doesn't work. Let me send a PR to update the scripts to use ubuntu 22.04... hopefully it will work out of box.

Let me send a PR to update the scripts to use ubuntu 22.04... hopefully it will work out of box.

@wainersm - FYI - I'm testing https://github.com/stevenhorsman/cloud-api-adaptor/tree/containerd-22.04-switch locally at the moment, but if you have already created this PR then let me know and I can switch to it

Let me send a PR to update the scripts to use ubuntu 22.04... hopefully it will work out of box.

@wainersm - FYI - I'm testing https://github.com/stevenhorsman/cloud-api-adaptor/tree/containerd-22.04-switch locally at the moment, but if you have already created this PR then let me know and I can switch to it

I didn't create the PR because yesterday I couldn't setup the cluster with Ubuntu 22.04 due to this bug: https://github.com/karmab/kcli/issues/615 . I still don't know how to test the fix, whether kcli has nightly builds or not. Do you know? Can you create the cluster with the kcli version you have installed?
Ah, let's use your PR ;)

Can you create the cluster with the kcli version you have installed?

So to start with the ubuntu 22.04 cluster seems correct:
ubuntu@peer-pods-worker-0:~$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.3 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.3 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

I'm guessing that my libvirt testing on rc1 was before the operator change that removed containerd from always being installed and that's why it worked then

I'm guessing that my libvirt testing on rc1 was before the operator change that removed containerd from always being installed and that's why it worked then

yup. containerd is 1.6 on ubuntu 22.04. We need to set that env.


Unable to choose account in flutter ios

I am unable to choose account in flutter ios but in android it is working fine, can anyone help me with that ?

Hi @kapilkarda
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself


List of Apps does not reload

When for example an app gets deleted, then the list of Apps does not get reloaded, therefore you can still search for it.

Fixed the bug here: commit


JUSIX enable / disable force deno caches to recompile

We should add the JUSIX option enable/disable behaviour to the compiler options to force deno to rebuild the module when app.dx parameter is changed and the app is restarted. Otherwise all files have to get a change to be recompiled with the new setting (JUSIX enabled or disabled).

Probably fixed by using the jsxImportSource property from the deno.config instead of the jusix flag in app.dx


bind_element_to_model() got an unexpected keyword argument 'field_name'

Following your readme on main.py there is the field_name parameter but generates:
bind_element_to_model() got an unexpected keyword argument 'field_name'
And the app doesn't start

So looking at the code and readme (that is not organized very well) the parameter field_name not exists anymore.
It is required to use fields also for a single field and now it works.


Update find.md (to.have.length doesn't works)

The current implementation doesn't seem to work.
Returns: TypeError: Cannot read property 'have' of undefined.

Yup right, this also seems to works, in fact, a better way to do it.


[IT-1101] remove sceptre cloudtrail setup

We are replacing the sceptre deployment of cloudtrail with org-formation
deployment.  The advantage of the ofn deployment is trails will get
automatically created for every account we add to the org.
depends on https://github.com/Sage-Bionetworks/aws-infra/pull/290

hmm, it looks like cloudtrail is setup by the shared essentials.yaml file. I guess one way we can manage this is to make strides use a custom essentials.yaml instead of using the shared one.

@ahayden we deploy the shared essentials.yaml without cloudtrail & config.  Then PR #128 will deploy cloudtrail & config into strides independently of essentials.yaml

@ahayden bridge was setup way before all other accounts were setup so it's a bit of a snowflake.  we need to keep it as-is due to legacy dependencies.
I've also added the removal of AWS config to this PR.


How to measure battery voltage?

uint16_t v1 = analogRead(34);
uint16_t v2 = analogRead(14);

float battery_voltage = ((float)v1 / 4095.0) * 2.0 * 3.3 * (1100 / 1000.0);
float other_voltage = ((float)v2 / 4095.0) * 2.0 * 3.3 * (1100 / 1000.0);

String voltage = String(battery_voltage) + "V " + String(other_voltage) + "V";
tft.setTextDatum(BR_DATUM);
tft.drawString(voltage, tft.width(), tft.height() - 5);

This gives me only USB power and constant 7.2V for pin 14 (whatever that is).
How can I track LiPo battery voltage/capacity?

Hello, i have tried to monitor voltage of an input solar panel to charge battery.
When there is an input, the GPIO34 displays the charge voltage (solar panel voltage).
When there is no input (only the battery), the GPIO34 displays the battery voltage (and you can check the capacity).
I don't know why you have a constant 7.2v on GPIO14, i will test and report it.

Idem issu Sakvojage, only 7.2 V
I Have try pin 35

uint16_t v1 = analogRead(34);
uint16_t v2 = analogRead(14);

float battery_voltage = ((float)v1 / 4095.0) * 2.0 * 3.3 * (1100 / 1000.0);
float other_voltage = ((float)v2 / 4095.0) * 2.0 * 3.3 * (1100 / 1000.0);

String voltage = String(battery_voltage) + "V " + String(other_voltage) + "V";
tft.setTextDatum(BR_DATUM);
tft.drawString(voltage, tft.width(), tft.height() - 5);

This gives me only USB power and constant 7.2V for pin 14 (whatever that is).
How can I track LiPo battery voltage/capacity?

Try This
`#include <TFT_eSPI.h> // Graphics and font library for ILI9341 driver chip
#include <SPI.h>
#include <Arduino.h>
const uint8_t blue = 2;
const uint8_t vbatPin = 34;
#define TFT_GREY 0x5AEB // New colour
TFT_eSPI tft = TFT_eSPI();  // Invoke library
float VBAT= 0;  // battery voltage from ESP32 ADC read
void setup(void) {
Serial.begin(115200);
tft.init();
tft.setRotation(3);
pinMode(blue, OUTPUT);
pinMode(vbatPin, INPUT);
//pinMode (16, OUTPUT);
}
void loop() {
VBAT = (float)(analogRead(vbatPin)) * 3600 / 4095 * 2;
Serial.println("Vbat = ");
Serial.print(VBAT); Serial.println(" Volts");
digitalWrite(blue, 0);
delay(2000);
Serial.println("Vbat2 = ");
Serial.print(VBAT); Serial.println(" Volts");
digitalWrite(blue, 1);
delay(5000);
tft.fillScreen(TFT_BLACK);
tft.setCursor(53, 50, 22);
tft.setTextColor(TFT_YELLOW,TFT_BLACK);tft.setTextFont(4);
tft.println(VBAT);
delay(3000);`

added VBATOK
#include <TFT_eSPI.h> // Graphics and font library for ILI9341 driver chip
#include <SPI.h>
#include <Arduino.h>
const uint8_t blue = 2;
const uint8_t vbatPin = 34;
#define TFT_GREY 0x5AEB // New colour
TFT_eSPI tft = TFT_eSPI();  // Invoke library
float VBAT= 0;  // battery voltage from ESP32 ADC read
void setup(void) {
Serial.begin(115200);
tft.init();
tft.setRotation(3);
pinMode(blue, OUTPUT);
pinMode(vbatPin, INPUT);
}
void loop() {
VBAT = (float)(analogRead(vbatPin)) * 3600 / 4095 * 2;
float VBATOK = VBAT / 1000;
Serial.println("Vbat = ");
Serial.print(VBATOK); Serial.println(" Volts");
digitalWrite(blue, 0);
delay(2000);
Serial.println("Vbat2 = ");
Serial.print(VBATOK); Serial.println(" Volts");
digitalWrite(blue, 1);
delay(5000);
tft.fillScreen(TFT_BLACK);
tft.setCursor(53, 50, 22);
tft.setTextColor(TFT_YELLOW,TFT_BLACK);tft.setTextFont(4);
tft.println(VBATOK);
delay(3000);

close #35

uint16_t v1 = analogRead(34);
uint16_t v2 = analogRead(14);

float battery_voltage = ((float)v1 / 4095.0) * 2.0 * 3.3 * (1100 / 1000.0);
float other_voltage = ((float)v2 / 4095.0) * 2.0 * 3.3 * (1100 / 1000.0);

String voltage = String(battery_voltage) + "V " + String(other_voltage) + "V";
tft.setTextDatum(BR_DATUM);
tft.drawString(voltage, tft.width(), tft.height() - 5);

This gives me only USB power and constant 7.2V for pin 14 (whatever that is).
How can I track LiPo battery voltage/capacity?

Can you explain why do we have to mutiply with 2 and what does  (1100 / 1000.0) mean? Thanks

Not sure about the multiply 2 yet but the 1100/1000.0 is just lazy copying from the factory test example.  the 1100 is the default vRef.  It looks like this code was taken from https://github.com/Xinyuan-LilyGO/TTGO-T-Display/blob/master/TFT_eSPI/examples/FactoryTest/FactoryTest.ino and vRef was just replaced with the default 1100.

Did anyone find a solution to this?

First I also read 7.2 volts when using the adc functions from the micropython port.
But then I used adc.atten(ADC.ATTN_11DB) to change the attenuation and then I read around 3.9 volts.
https://docs.micropython.org/en/latest/esp32/quickref.html#adc-analog-to-digital-conversion

float battery_voltage = ((float)v / 4095.0) * 2.0 * 3.3 * (1100 / 1000.0);
So 1100 is actually 1.1V in mV. 1.1V is the internal reference voltage that readings get compared to.
Then 2.0 * 3.0 are voltage divider parameters. You can't measure 5V on an ADC with a 1.1V reference voltage you can only measure from 0V to 1.1V. That's why they used voltage dividers to extend the range to 2.0 * 3.3 * 1.1 = 7.26V.
You actually have to set the reference voltage by setting the right attenuation adc1_config_channel_atten(ADCx_CHANNEL_x, ADC_ATTEN_0db); Something like this.
Then v / 4095.0 is there because ESP32 has a 12bit ADC and 12bits is 4096 steps, so 0V is 0 and 1.1V is 4095. When you multiply that with 7.26, you get a range where 0 on ADC is 0.0V and 4095 is 7.26V.

They also built in an option to disable the voltage divider, so you have to set GPIO 14 to HIGH before measuring voltage and set it to LOW when not using it to preserve power.
So there are two important things to do before reading the voltage:

setting the attenuation to 0dB on the correct ADC and ADC channel
setting the GPIO 14 to HIGH when measuring LiPo voltage

Isn't just better to post a code snippet that works for measuring battery voltage/ USB voltage and end this discussion?

I tested it.
This works:
#include <driver/adc.h>
#include "esp_adc_cal.h"
void setup() {
Serial.begin(115200);
esp_adc_cal_characteristics_t adc_chars;
esp_adc_cal_value_t val_type = esp_adc_cal_characterize((adc_unit_t)ADC_UNIT_1, (adc_atten_t)ADC_ATTEN_DB_2_5, (adc_bits_width_t)ADC_WIDTH_BIT_12, 1100, &adc_chars);
pinMode(14, OUTPUT);
}
void loop() {
digitalWrite(14, HIGH);
delay(1);
float measurement = (float) analogRead(34);
float battery_voltage = (measurement / 4095.0) * 7.26;
digitalWrite(14, LOW);
Serial.println(battery_voltage);
}

They also built in an option to disable the voltage divider, so you have to set GPIO 14 to HIGH before measuring voltage and set it to LOW when not using it to preserve power.

From where did you get this information ?

Quote from here:
 ADC_EN is the ADC detection enable port     If the USB port is used for power supply, it is turned on by default.     If it is powered by battery, it needs to be set to high level

Does anyone know how the schematic of battery measurement part is designed? I am not able to measure precise voltage of battery when I use code samples posted above. It shows ~0,2V less than it actually is when measured (with dedicated voltmeter) from battery terminals. Also plugging in USB cable then value changes right away to 4,22V when measures with esp32. But when measuring on battery terminals it starts slowly charging. 🤔

The ADC in esp32 is not accurate and the resistors in voltage divider are also not accurate. You could try measuring the resistors and calibrating ADC, but it will never be as accurate as a multimeter.

How can I read the battery voltage when usb is plugged in?   I can't find a way to know when the battery is charged.

How can I read the battery voltage when usb is plugged in? I can't find a way to know when the battery is charged.

The blue LED turns off. The battery voltage as measured by this reply while USB is plugged in ends up at about 4.78V when fully charged on my board

I tested it.
This works:
#include <driver/adc.h> #include "esp_adc_cal.h"

indeed it works. you can plug in a battery and you'll see a realistic discharge curve

documentation:
https://docs.espressif.com/projects/esp-idf/en/release-v3.3/api-reference/peripherals/adc.html
https://docs.espressif.com/projects/esp-idf/en/release-v3.3/api-reference/peripherals/adc.html#adc-calibration
PS: I believe this issue can be closed now.


switch to vue 3 compiler

I was with a colleague working on making a port of the sfc compiler from vue 3 to deno and I think we have already achieved it, here is the repository, I think all the current compiler problems will be a thing of the past

awesome, this is really great

awesome, this is really great

I have a branch where I have been doing integration tests of the vue sfc compiler, the isolation problems disappear almost completely, and also the error detection improves a lot, on the other hand I have also been able to successfully integrate the support for typescript and scss outside out of the box, also the ability to load imports and intermediate code within a component
here is the branch in which I am doing the tests https://github.com/buttercubz/vno/tree/vue3-compiler

I have a branch where I have been doing integration tests of the vue sfc compiler, the isolation problems disappear almost completely, and also the error detection improves a lot, on the other hand I have also been able to successfully integrate the support for typescript and scss outside out of the box, also the ability to load imports and intermediate code within a component
here is the branch in which I am doing the tests https://github.com/buttercubz/vno/tree/vue3-compiler

this looks very exciting, and a great step forward. I created a branch 'vue3Compiler', would you want to create a PR into that branch with this? I'd like to test it out

this looks very exciting, and a great step forward. I created a branch 'vue3Compiler', would you want to create a PR into that branch with this? I'd like to test it out

sure

sure

I'd love to get this merged and released, is there any other version for posscss in the vue_sfc_compiler repo that will not log depreciated errors to the console ?

I'd love to get this merged and released, is there any other version for posscss in the vue_sfc_compiler repo that will not log depreciated errors to the console ?

yes, I have pending to remove the deprecation warnings

yes, I have pending to remove the deprecation warnings

Just merged and published the release!

Just merged and published the release!


Update to 3.10

A new version of ply, version 3.10 has been released.

please feel free to submit a pr!


Allow order notes

Allow customers to add custom notes to the order.
Similar to:

Slack conversation: https://dintero.slack.com/archives/C02TVV8PMFB/p1660559516433189

Released in https://github.com/Dintero/Dintero.Checkout.WooCommerce.V2/releases/tag/1.2.0


Added Dockerfile

There were quite a number of issues with the other PR (no offense) so I just rewrote it.

Non-root application user for security.
Docker container labels.
ca-certificates in the event https connections need them.
Multi-stage build to scratch for the smallest image possible

Before Merging, you may wish to:

Update the Author label line, I'm not sure how you like to see your name or handle.
Remove ca-certificates lines if you are sure they are unused (I can never be sure).
Add RUN go test prior to the build (after go mod download).

We can merge this. cc @solsson, please don't spend time on the previous PR.


Revert "Stop the bleeding for #6059"

This fixes #6059.
(I wasn't able to reproduce the problem with the new etcd release.)
This reverts commit 6be09c5e9796ece6b968478e98aacd32af49d36a.
cc @lavalamp @fgrzadkowski @quinton-hoole

All the e2e tests are passing:
Ran 31 of 37 Specs in 1193.422 seconds
SUCCESS! -- 31 Passed | 0 Failed | 1 Pending | 5 Skipped I0408 14:01:53.929957   16770 driver.go:114] All tests pass

LGTM


Allow more than 16 pages in a Frame

By modifying the save-file you can get more than 16 pages, but it resets when editing it. Is there a good reason for this limit?

It was discussed some time ago: https://sourceforge.net/p/qlcplus/discussion/general/thread/883d0a80/
but the change was not committed yet. Please let's continue with the discussion there.

Increased to 100.
Closing this


Fix wording, remove breaking apostrophe

Removed the contraction so it doesn't break the single-quoted string.

👍

Is this a quick hit to merge?


Change default "find path" behavior and associated flag

See https://github.com/senzing-garage/garage-roadmap/issues/23

Fixed via https://github.com/senzing-garage/sz-sdk-python-abstract/issues/71
Will close once sz-sdk-python-abstract  v.0.1.4 is created.


Maintainer wanted

I have just updated Fable.Electron to support Electron 9.0. Unfortunately, I am unable to continue prioritizing keeping Fable.Electron up to date with new Electron versions. It's relatively little work, but since I'm not actively using it and since it's relatively simple for anyone to keep it up to date, my motivation for this is exclusively "because I'm that kind of guy who doesn't give up commitments", which means this is one of many small, but regular drains on my energy.
I have added the text below to the readme (quoting it here for reference), where I mention the work involved. Again, it's not much, so I hope someone who is using Fable.Electron is willing to step up.
Let me know if you have questions, if I should add more details to the description/readme, etc.
@alfonsogarciacaro Could a call for maintainers be communicated in relevant channels?

Maintainer wanted!
If you are actively using Fable.Electron or otherwise interested in seeing these bindings being kept up to date, please consider taking over this project!
I am not actively using Fable.Electron and will not be able to prioritize keeping it up to date with new Electron releases. I am therefore seeking a new maintainer.
Work involved: The only thing I have been doing so far is using a website scanner (I use Wachete, but use whatever you want) to monitor https://www.electronjs.org/docs/all for changes, and update the bindings accordingly. Minor releases and relevant documentation updates occur at most once a week (often less) and usually involve 15m to an hour of work (including deployment), while major Electron releases may take a several hours.
This documentation-driven approach is slightly complicated by the fact that the docs may have bugs, i.e. something should be an array but is documented as a single value. Use your best judgement and search the Electron source code if you’re unsure whether the docs are correct.

if nobody else volunteers - I think I'd adopt it

That would be awesome, @forki! I don't know whether you want to wait a bit or not.  It's fairly niche, after all. 😛
Are you interested in taking over cmeeren/fable-elmish-electron-material-ui-demo, too? That's even less work, just updating dependencies every now and then when you feel like it (e.g. for major Electron releases), and fixing any problems that arise. Unless you feel like making something more of it, of course.
If you're not interested, I'll be posting a call for maintainers for that repo, too.

I can't see anyone else stepping up - do you want to officially take over, @forki?

Ok let's do this
Christer van der Meeren<EMAIL_ADDRESS>schrieb am Mi., 3. Juni
2020, 07:15:

I can't see anyone else stepping up - do you want to officially take over,
@forki https://github.com/forki?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fable-compiler/fable-electron/issues/6#issuecomment-637958290,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAOAND3DKZ7QTQZAKV3OJLRUXL7FANCNFSM4NI4V2MQ
.

Thanks a lot! You're now a repo admin. Do you want to take over https://github.com/cmeeren/fable-elmish-electron-material-ui-demo, too? (See above, feel free to decline, then I'll post a call for maintainers there.)

I wonder if we should move the sample to fable org as well.
Christer van der Meeren<EMAIL_ADDRESS>schrieb am Mi., 3. Juni
2020, 08:11:

Thanks a lot! You're now a repo admin. Do you want to take over
https://github.com/cmeeren/fable-elmish-electron-material-ui-demo
http://cmeeren/fable-elmish-electron-material-ui-demo, too? (See above,
feel free to decline, then I'll post a call for maintainers there.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fable-compiler/fable-electron/issues/6#issuecomment-637977454,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAOANAVFPEXDCRTKUKBZXLRUXSP7ANCNFSM4NI4V2MQ
.

I'm fine with that, just let me know if you want admin access to the sample repo and you can arrange whatever you want with the Fable org guys.

Or perhaps let's move it first, to avoid too many URL changes/redirects.
@MangelMaxime (or anyone else relevant), what do you think? Should https://github.com/cmeeren/fable-elmish-electron-material-ui-demo be moved to the Fable org, just like Fable.Electron is?

Yes please make me admin. And I assume I also need to get nuget push
privileges
Christer van der Meeren<EMAIL_ADDRESS>schrieb am Mi., 3. Juni
2020, 08:16:

I'm fine with that, just let me know if you want admin access to the
sample repo and you can arrange whatever you want with the Fable org guys.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fable-compiler/fable-electron/issues/6#issuecomment-637979474,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAOANA76TUIYKWIKOT32LDRUXTDFANCNFSM4NI4V2MQ
.

Sure, what's your GH username?

Sure, what's your NuGet username?

https://www.nuget.org/profiles/sforkmann
Am Mi., 3. Juni 2020 um 08:21 Uhr schrieb Christer van der Meeren <
<EMAIL_ADDRESS>
Sure, what's your NuGet username?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fable-compiler/fable-electron/issues/6#issuecomment-637981568,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAOANDGGAWYN4IH7E5HE3LRUXTWBANCNFSM4NI4V2MQ
.

@alfonsogarciacaro @et1975 @MangelMaxime @Zaid-Ajaj
Do you think https://github.com/cmeeren/fable-elmish-electron-material-ui-demo should be part of the Fable org? Personally I don't care; I just need an answer so I know whether to transfer the repo to @forki or to this org.

Any member of elmish org can create/move a repo into elmish, but up to you folks.

Yes, I looking for approval, not action. :)

Sorry, I just see "elmish" in the name and assuming it is a layer above fable and thought you might want to move it elmish org instead.

@cmeeren Sorry for the slow answer, we can move it here and consider it the official sample for electron.

I tried to transfer ownership, but I "don't have permissions to create public repositories for fable-compiler"

@cmeeren Sorry for the delay, I deleted my fork of your repository can you please try again?

Done

Thank you I did the transform to fable-compiler.
I also invited you in the fable-electron team so you can still have write access to the repository if needed.

I am closing this issue as I don't think much more need to be done.

Thanks! Make sure to give @forki admin access to the transferred repo.


chronos@localhost / $ sudo apt-get install git sudo: apt-get: command not found

Please paste the output of the following command here: sudo edit-chroot -all

chronos@localhost / $ sudo edit-chroot -all
name: kali
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/kali...
crouton: version 1-20170124013033~c161e9c1
release: kali-rolling
architecture: armhf
xmethod: xiwi
targets: xfce,xiwi
host: version 8872.76.0 (Official Build) stable-channel nyan_blaze
kernel: Linux localhost 3.10.18 #1 SMP Fri Jan 6 23:44:58 PST 2017 armv7l GNU/Linux
freon: no
Unmounting /mnt/stateful_partition/crouton/chroots/kali...
name: precise
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/precise...
crouton: version 1-20161129162558~master:5f2f11f9
release: precise
architecture: armhf
xmethod: xorg
targets: xfce
host: version 8872.76.0 (Official Build) stable-channel nyan_blaze
kernel: Linux localhost 3.10.18 #1 SMP Fri Jan 6 23:44:58 PST 2017 armv7l armv7l armv7l GNU/Linux
freon: no
Unmounting /mnt/stateful_partition/crouton/chroots/precise...
Please describe your issue:apt-get not found
If known, describe the steps to reproduce the issue:chronos@localhost / $ sudo apt-get install git sudo: apt-get: command not found

It looks like you may be running the command from the ChromeOS shell, rather than the shell of the Linux install inside your chroot.

@r4d1n, Nice catch, I totally missed that...

Thank you so much its working

Awesome 🥇

hey  can you tell me why i cant download kali or that it keeps asking for a passwd or how to reset passwd

@gh4one,
See this related issue comment for a possible workaround - and please don't hijack someone else's issue/thread with an unrelated question -

https://github.com/dnschneid/crouton/issues/4153#issuecomment-560190450

-DennisLfromGA

Hello people of GitHub i need help trying to update using the developer console using ctrl+alt+f2 or foward i logged in then i try updating the system using sudo apt-get install or sudo apt-get update and even sudo apt-get upgrade it didn't wprk then i tried sudo edit-chroot -all and sudo enter-chroot and it didn't work can anyonee help me with this???

@ShamoneyX,
See this previous comment:

https://github.com/dnschneid/crouton/issues/3043#issuecomment-275457972

-DennisLfromGA

please i use chromebook with chrome os, and anytime that i enter command and type shell and want to use sudo account it ask of password whilst i have not st a password. Please can someone help me?

@cressat,
Please don't hijack topics that don't pertain to your issue.
Take a look at this https://github.com/dnschneid/crouton/issues/2050#issuecomment-348382431 for help with your issue.
Hope this helps,
-DennisLfromGA


Clean experiment file

Move ipynb experiment files to this repostiory
Identify results to be saved into this repository (or research repository)
Make them easier to call and name
Change to make figures automatically

About 2., the visibility of this repository cannot be changed to private, so results should be saved to other repository.


Encountered circular reference for allOf construction

Environment

Nuget Package: JsonSchema.Net
Nuget Version: 4.0.5
OS: Windows11
.Net Target: net7

Describe the bug
I built a schema for an array where the items have a type that's derived from another type. When I evaluate the schema on a correct json file, JsonSchema.Net says there is a circular reference. See below for the schema.
To Reproduce
schema:
{
  "$schema": "https://json-schema.org/draft/2020-12/schema",

  "type": "array",
  "items": { "$ref": "#/$defs/DerivedType" },

  "$defs": {

    "BaseType": {
      "type": "object",
      "properties": {
        "field1": { "type": "string" }
      }
    },

    "DerivedType": {
      "allOf": [
        { "$ref": "#/$defs/BaseType" },
        { "properties": { "field2": { "type": "string" } } }
      ]
    }
  }
}

test json:
[
  {
    "field1": "foo",
    "field2": "bar"
  }
]

Expected behavior
There is no circular reference as far as I can tell.
Additional context
Oddly enough, it seems json-everything.net/json-schema/ playground validates correctly. Here's the stacktrace I get from C#:
 Message:
Test method Tests.UnitTests.TestBugJson threw exception:
Json.Schema.JsonSchemaException: Encountered circular reference at schema location `file:///C:/redacted/bugschema2.json` and instance location `/0`

  Stack Trace:
RefKeyword.Evaluate(EvaluationContext context)
EvaluationContext.Evaluate()
AllOfKeyword.Evaluate(EvaluationContext context)
EvaluationContext.Evaluate()
RefKeyword.Evaluate(EvaluationContext context)
EvaluationContext.Evaluate()
ItemsKeyword.Evaluate(EvaluationContext context)
EvaluationContext.Evaluate()
JsonSchema.Evaluate(JsonNode root, EvaluationOptions options)
JsonSchemaExtensions.Evaluate(JsonSchema jsonSchema, JsonElement jsonElement, EvaluationOptions options)
UnitTests.TestBugJson() line 165
RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)

Thanks for reporting.  The playground is running .net 6.  I wonder if that's part of the cause.

Please see the PR ☝.  I haven't been able to replicate this locally.  Doesn't seem that .net 7 is the issue, either.

Yep... Reading from a file produces the error.  How odd.

Yup my test was reading from file. Fwiw, the target being an array is not a factor, it also throws if you change that to an object containing a derivedtype field


How can I train with multi GPU?

I executed "train_softmax.py" with 8 GPU machine but it only used GPU 0.
How can I execute with multi-GPU?
Thanks.

You will have to modify code and provide functionality like TensorFlow-Slim for multi GPU support.

ops


[BUGFIX] Split $objectDefinition into $schema and $properties

What

Split $objectDefinition into $schema and $properties (fixes bugs in #353 & #406);
Rename some stuff so that the names make sense.

Why
Object validation attempts to use a single variable to store both the object schema, and the definition of the object's properties. This causes validation to be incomplete where "properties" is not set, but "additionalProperties" is.

@o5 @shmax @mathroc may I have your help giving this another review?

Whoops, sorry guys--I'm still paying attention and still interested, but this one just slipped past my radar. Holler if you need anything.


Several minor improvements and typos in playground examples.

Wow! Thanks - this is a great improvement ❤


Revisions/translations pages for drafts are accessible only through the endpoints

When the kata is in a draft state, the only thing you can do is Preview it (see how it looks in the trainer). If one wishes to access the revisions (or translations) page, they have to go to https://www.codewars.com/kata/{kata_id}/revisions page directly, compared to simply pressing a button how it's done in the published-kata editor.

Done


How to pass angular selector as html option

I'm using latest angular (4.3.2) and would like to use my component (it's selector is app-ask-for-email) as the content of my modal e.g.
private askForEmailModalOptions: SweetAlertOptions = {
    type: 'info',
    html: `<app-ask-for-email></app-ask-form-email>`,
    showConfirmButton: false,
    allowOutsideClick: true,
    showCancelButton: false,
    showCloseButton: true,
    allowEscapeKey: true
  };

But the template is not compiled. Is there a way to make it work?

This question doesn't actually related to SweetAlert2. You should ask it in the Angular community, e.g. https://stackoverflow.com/questions/tagged/angularjs

Also, take a look on the Angular 2+ wrapper for SweetAlert2: https://github.com/toverux/ngsweetalert2 - it's great!

Thanks, https://github.com/toverux/ngsweetalert2 is great but it doesn't show me how to do the stuff I want to achieve. I already posted on stackoverflow: https://stackoverflow.com/questions/45500265/how-to-pass-angular-selector-as-html-option-of-sweetalert2

Well, amusing, someone just asked me the same question: https://github.com/toverux/ngsweetalert2/issues/16
That's not possible right now, but I'm considering adding it since it's a real use case.
And remember, html is just an option that takes an HTML string. That has just nothing to do with Angular template syntax. You HAVE to use some kind of  or transclusion.


FilesUploadV2Request does not return an actionable response

When using the FilesUploadV2Request, encountering an error does not return the cause or any actionable information in the response or exception.
Reproducible in:
The Slack SDK version
[INFO] +- com.slack.api:bolt:jar:1.32.0:compile
[INFO] |  +- com.slack.api:slack-api-model:jar:1.32.0:compile
[INFO] |  +- com.slack.api:slack-app-backend:jar:1.32.0:compile
[INFO] +- com.slack.api:bolt-socket-mode:jar:1.32.0:compile
[INFO] +- com.slack.api:slack-api-client:jar:1.32.0:compile

Java Runtime version
openjdk version "17.0.6" 2023-01-17 LTS
OpenJDK Runtime Environment Corretto-<IP_ADDRESS>.1 (build 17.0.6+10-LTS)
OpenJDK 64-Bit Server VM Corretto-<IP_ADDRESS>.1 (build 17.0.6+10-LTS, mixed mode, sharing)
OS info
Amazon Linux 2
Steps to reproduce:
try {
    FilesUploadV2Request req = FilesUploadV2Request.builder().channel(thisChannel).content(body).initialComment(headline).build();
    FilesUploadV2Response res = null;
    res = slack.methods(botToken).filesUploadV2(req);
    if (!res.isOk()) {
        logger.error("Response: Ok: {} Error: {} Warning: {}", res.isOk(), res.getError(), res.getWarning());
    }
    Thread.sleep(1000);
} catch (Exception e) {
    logger.info("Error {}", e);
    run(parseProductToWxBotManagerQueue);
}

In the case of this error, the variable thisChannel was a channel that had been deleted.
Expected result:
Some actionable response or exception stating 'Channel ### not found' or similar
Actual result:
2023-09-26 00:25:37.936  INFO 11176 --- [task-1] gov.noaa.nws.wxbot.slack.WxBotManager    : Error {}

com.slack.api.methods.SlackFilesUploadV2Exception: null
        at com.slack.api.methods.impl.FilesUploadV2Helper.<init>(FilesUploadV2Helper.java:33) ~[slack-api-client-1.32.0.jar!/:1.32.0]
        at com.slack.api.methods.impl.MethodsClientImpl.filesUploadV2(MethodsClientImpl.java:2173) ~[slack-api-client-1.32.0.jar!/:1.32.0]

Hi @matt-w-davis, thanks for writing in.
I've confirmed the API client's behavior to be certain, but if the specified channel does not exist, the filesUploadV2 method threw an exception with ample information for me. Could you reassess if the channel ID is causing your unexpected situation? Also, if the channel is still active, your file uploading operation may have been successful, which could explain why you didn't encounter any errors in this scenario.
I hope you'll figure the cause out soon!

It's the same issue as:
https://github.com/slackapi/python-slack-sdk/issues/1326
You have to use channel name, but this is not clear from the error message!


Undefined symbol error in ORQA; preprocess_wiki_extractor.py

Hi,
I'm trying to create TFR Blocks on my custom dataset using the language.orqa.preprocessing.preprocess_wiki_extractor module.
Running the command in the README throws the following error:
$ CUDA_VISIBLE_DEVICES=0,3 python -m language.orqa.preprocessing.preprocess_wiki_extractor --input_pattern=../splittext/part_* --output_pattern=../preprocess-wiki-extractor
Traceback (most recent call last):
  File "/projects/anaconda3/envs/condaorqa/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/projects/anaconda3/envs/condaorqa/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/projects/language/language/orqa/preprocessing/preprocess_wiki_extractor.py", line 39, in <module>
    from language.orqa.utils import bert_utils
  File "/projects/language/language/orqa/utils/bert_utils.py", line 20, in <module>
    import tensorflow_text as tf_text
  File "/projects/anaconda3/envs/condaorqa/lib/python3.7/site-packages/tensorflow_text/__init__.py", line 21, in <module>
    from tensorflow_text.python import metrics
  File "/projects/anaconda3/envs/condaorqa/lib/python3.7/site-packages/tensorflow_text/python/metrics/__init__.py", line 20, in <module>
    from tensorflow_text.python.metrics.text_similarity_metric_ops import *
  File "/projects/anaconda3/envs/condaorqa/lib/python3.7/site-packages/tensorflow_text/python/metrics/text_similarity_metric_ops.py", line 28, in <module>
    gen_text_similarity_metric_ops = load_library.load_op_library(resource_loader.get_path_to_datafile('_text_similarity_metric_ops.so'))
  File "/projects/anaconda3/envs/condaorqa/lib/python3.7/site-packages/tensorflow_core/python/framework/load_library.py", line 57, in load_op_library
    lib_handle = py_tf.TF_LoadLibrary(library_filename)
tensorflow.python.framework.errors_impl.NotFoundError: /projects/anaconda3/envs/condaorqa/lib/python3.7/site-packages/tensorflow_text/python/metrics/_text_similarity_metric_ops.so: undefined symbol: _ZN10tensorflow12OpDefBuilder4AttrESs

I did find a Github issue under tensorflow-text https://github.com/tensorflow/text/issues/192
The solution (as suggested by a user) is to update to tensorflow-text==2.2.0
However, this upgrades tensorflow to 2.2.0 as well.
The ORQA codebase requires 2.1.0 -- any thoughts would be helpful.
Thanks for releasing the codebase!

Nevermind.
Creating a new environment with the requirements.txt file solved the problem.
I'm guessing this was some version issue ...


SCPToolkit Deprecated

Since SCPToolkit has been deptrecated, a consideration should be made to use the new VIGEm framework, also developed by @nefarius, especifically ViGEmBus

I like the idea of it, but I could not find any documentation for the new VIGEm framework. If you can help me with that I might try it.

I think this serves as the doc for now
And this is are the samples
For reference here x360ce made the change to ViGEm

Added to v2.0


Incorrect Solution Passes Tests

Challenge Import a Google Font has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.100 Safari/537.36 Vivaldi/1.5.658.44.
Please describe how to reproduce this issue, and include links to screenshots if possible.
The instructions and tests for this challenge say that you should apply the Lobster font to the h2 element, but leave the p elements using Monospace.
So that implies that the correct solution is to create a selector that will target only the h2 elements:

However, this code passes even though it also changes the font of the p elements to Lobster.

CatPhotoApp

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.

Should we modify the tests so that there has to be a separate h2 selector?

Confirmed.

Wow! It also applies to the next challenge: [Specify How Fonts Should Degrade]

Good catch. Thanks @Margaret2 @Manish-Giri and @marhyorh.
Anyone up for fixing the tests on both challenges?

I'll take a crack at this. Been awhile since I made a contribution.


Added color output support

These changes are much more a matter of taste than my other fixes, so I'm giving them their own branch and pull request.
This pull request implements simple (and kind of poorly optimized) colorization for news output. A more robust solution would probably be to use termcolor (https://pypi.python.org/pypi/termcolor) but this was simple enough to do.
The full list of colors is here: http://ascii-table.com/ansi-escape-sequences.php

lets talk @wilt00  https://gitter.im/instantnews/Lobby


Refactor search from cloudsearch to elasticsearch

What does this PR do?
Refactors the API to search from elastic search
Description of Task to be completed?
Replace Cloudsearch on HealthTools API with ElasticSearch
How should this be manually tested?
run posttests --nocapture

@DavidLemayian ☝ don't know why I can't add you as a reviewer

@Gathondu Did you pull the changes before sending this?

@DavidLemayian yes I did. And I noticed there is no develop branch that's why I requested against master

@DavidLemayian let me refactor this to also allow to run from local instance of elastic search

@DavidLemayian We are good to go 👍


sv create with storybook is broken

Hi,
... at least to me ;
just want to report as i think, that I am not missing something .
Frank
my environment
os: Darwin Kernel Version 24.1.0: Thu Oct 10 21:06:23 PDT 2024; root:xnu-11215.41.3~3/RELEASE_ARM64_T8132'
node: v23.2.0
pnpm: 9.14.1

% npx sv create myapp

(node:58755) ExperimentalWarning: CommonJS module /Users/fb/.nvm/versions/node/v23.2.0/lib/node_modules/npm/node_modules/debug/src/node.js is loading ES Module /Users/fb/.nvm/versions/node/v23.2.0/lib/node_modules/npm/node_modules/supports-color/index.js using require().
Support for loading ES Module in require() is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
┌  Welcome to the Svelte CLI! (v0.6.4)
◇  Which template would you like?
│  SvelteKit minimal
◇  Add type checking with Typescript?
│  Yes, using Typescript syntax
◆  Project created
◇  What would you like to add to your project? (use arrow keys / spacebar)
│  storybook
◇  Which package manager do you want to install dependencies with?
│  pnpm
◇  Running external command (pnpm dlx storybook@latest init --skip-install --no-dev)
(node:58806) ExperimentalWarning: CommonJS module /Users/fb/.nvm/versions/node/v23.2.0/lib/node_modules/npm/node_modules/debug/src/node.js is loading ES Module /Users/fb/.nvm/versions/node/v23.2.0/lib/node_modules/npm/node_modules/supports-color/index.js using require().
Support for loading ES Module in require() is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
╭──────────────────────────────────────────────────────╮
│                                                      │
│   Adding Storybook version 8.4.4 to your project..   │
│                                                      │
╰──────────────────────────────────────────────────────╯
 • Detecting project type. ✓
 • Adding Storybook support to your "SvelteKit" app
  ✔ Getting the correct version of 9 packages
  ✔ Installing Storybook dependencies
. ✓

attention => Storybook now collects completely anonymous telemetry regarding usage.
This information is used to shape Storybook's roadmap and prioritize features.
You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL: https://storybook.js.org/telemetry

│   Storybook was successfully installed in your project! 🎉            │
│   To run Storybook manually, run pnpm run storybook. CTRL+C to      stop.                                                               │
│                                                                       │
│   Wanna know more about Storybook? Check out                          │
│   https://storybook.js.org/                                           │
│   Having trouble or want to chat? Join us at                          │
│   https://discord.gg/storybook/                                       │
◆  Successfully installed dependencies
◇  Project next steps │  1: cd myapp                                                             │
│  2: git init && git add -A && git commit -m "Initial commit" (optional)  │
│  3: pnpm dev --open                                                      │
│  To close the dev server, hit Ctrl-C                                     │
│  Stuck? Visit us at https://svelte.dev/chat                              │
└  You're all set!

% cd myapp
myapp % pnpm i
Lockfile is up to date, resolution step is skipped
Already up to date
Done in 368ms

myapp % pnpm run storybook

><EMAIL_ADDRESS>storybook /Users/fb/Documents/Github/myapp
> storybook dev -p 6006
@storybook/core v8.4.4

SB_CORE-SERVER_0002 (CriticalPresetLoadError): Storybook failed to load the following preset: ./.storybook/main.js.

Please check whether your setup is correct, the Storybook dependencies (and their peer dependencies) are installed correctly and there are no package version clashes.

If you believe this is a bug, please open an issue on Github.

SB_CORE-SERVER_0002 (CriticalPresetLoadError): Storybook failed to load the following preset: ./node_modules/.pnpm/@storybook+addon-svelte-csf@5.0.0-next.11_@storybook+svelte@8.4.4_storybook@8.4.4_svelte<EMAIL_ADDRESS>
Please check whether your setup is correct, the Storybook dependencies (and their peer dependencies) are installed correctly and there are no package version clashes.

If you believe this is a bug, please open an issue on Github.

Error [ERR_UNSUPPORTED_DIR_IMPORT]: Directory import './node_modules/.pnpm/@storybook+addon-svelte-csf@5.0.0-next.11_@storybook+svelte@8.4.4_storybook@8.4.4_svelte@5.2._vu6jifpvabysmtmzzp4bd6g5sy/node_modules/@storybook/addon-svelte-csf/dist/compiler/post-transform/story' is not supported resolving ES modules imported from ./node_modules/.pnpm/@storybook+addon-svelte-csf@5.0.0-next.11_@storybook+svelte@8.4.4_storybook@8.4.4_svelte<EMAIL_ADDRESS>    at finalizeResolution (node:internal/modules/esm/resolve:263:11)
    at moduleResolve (node:internal/modules/esm/resolve:860:10)
    at defaultResolve (node:internal/modules/esm/resolve:984:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:654:12)
    at #cachedDefaultResolve (node:internal/modules/esm/loader:603:25)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:353:53)
    at new ModuleJobSync (node:internal/modules/esm/module_job:341:34)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:413:11)
    at new ModuleJobSync (node:internal/modules/esm/module_job:341:34)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:413:11)

More info:

    at loadPreset (./node_modules/.pnpm/@storybook+core@8.4.4/node_modules/@storybook/core/dist/common/index.cjs:16477:13)

More info:

    at loadPreset (./node_modules/.pnpm/@storybook+core@8.4.4/node_modules/@storybook/core/dist/common/index.cjs:16477:13)
    at async Promise.all (index 1)
    at async loadPresets (./node_modules/.pnpm/@storybook+core@8.4.4/node_modules/@storybook/core/dist/common/index.cjs:16487:55)
    at async getPresets (./node_modules/.pnpm/@storybook+core@8.4.4/node_modules/@storybook/core/dist/common/index.cjs:16520:11)
    at async buildDevStandalone (./node_modules/.pnpm/@storybook+core@8.4.4/node_modules/@storybook/core/dist/core-server/index.cjs:37145:11)
    at async withTelemetry (./node_modules/.pnpm/@storybook+core@8.4.4/node_modules/@storybook/core/dist/core-server/index.cjs:35757:12)
    at async dev (./node_modules/.pnpm/@storybook+core@8.4.4/node_modules/@storybook/core/dist/cli/bin/index.cjs:2591:3)
    at async s.<anonymous> (./node_modules/.pnpm/@storybook+core@8.4.4/node_modules/@storybook/core/dist/cli/bin/index.cjs:2643:74)

WARN Broken build, fix the error above.
WARN You may need to refresh the browser.

WARN   Failed to load preset: {"type":"presets","name":"/Users/fb/Documents/Github/myapp/node_modules/.pnpm/@storybook+addon-svelte-csf@5.0.0-next.11_@storybook+svelte@8.4.4_storybook@8.4.4_svelte<EMAIL_ADDRESS>on level 1
Error [ERR_UNSUPPORTED_DIR_IMPORT]: Directory import './node_modules/.pnpm/@storybook+addon-svelte-csf@5.0.0-next.11_@storybook+svelte@8.4.4_storybook@8.4.4_svelte@5.2._vu6jifpvabysmtmzzp4bd6g5sy/node_modules/@storybook/addon-svelte-csf/dist/compiler/post-transform/story' is not supported resolving ES modules imported from ./node_modules/.pnpm/@storybook+addon-svelte-csf@5.0.0-next.11_@storybook+svelte@8.4.4_storybook@8.4.4_svelte<EMAIL_ADDRESS>    at finalizeResolution (node:internal/modules/esm/resolve:263:11)
    at moduleResolve (node:internal/modules/esm/resolve:860:10)
    at defaultResolve (node:internal/modules/esm/resolve:984:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:654:12)
    at #cachedDefaultResolve (node:internal/modules/esm/loader:603:25)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:353:53)
    at new ModuleJobSync (node:internal/modules/esm/module_job:341:34)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:413:11)
    at new ModuleJobSync (node:internal/modules/esm/module_job:341:34)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:413:11)
(node:58966) ExperimentalWarning: CommonJS module /Users/fb/Documents/Github/myapp/node_modules/.pnpm/@storybook+core<EMAIL_ADDRESS>is loading ES Module /Users/fb/Documents/Github/myapp/node_modules/.pnpm/@storybook+addon-svelte-csf@5.0.0-next.11_@storybook+svelte@8.4.4_storybook@8.4.4_svelte<EMAIL_ADDRESS>using require().
Support for loading ES Module in require() is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
? Would you like to help improve Storybook by sending anonymous crash reports? › (Y/n)

// .storybook/mail.ts
/** @type { import('@storybook/sveltekit').StorybookConfig } */
const config = {
  stories: ["../src/**/*.mdx", "../src/**/*.stories.@(js|ts|svelte)"],
  addons: [
    "@storybook/addon-svelte-csf",
    "@storybook/addon-essentials",
    "@chromatic-com/storybook",
    "@storybook/addon-interactions",
  ],
  framework: {
    name: "@storybook/sveltekit",
    options: {},
  },
};
export default config;

It's because you're using Node 23. Please use a stable version of Node.js

@benmccann wrong. broken on 22.11 also.

https://github.com/storybookjs/storybook/issues/29698

This is not happening for me on<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Additionally, i re-ran our tests on main and they are still passing.
But, you made the right move and opened an upstream issue, so that the storybook team can have a look.

@JReinhold You wanted to keep me tagging you if something arises 😆. This issue has been raised by at least 3 different people now.
This upstream issue also got linked: https://github.com/storybookjs/storybook/issues/30008. Please let me know if you need anything else


When posting duplicate event using client-go's Eventf() api server throws error "series.count: Invalid value: "": should be at least 2' (will not retry!)"

In our application, we are using high level APIs from client-go/tools/event/event_broadcaster.go.
When we post the same event again, we are getting the following error from api-server,
**E1220 17:40:34.360998 1980937 event_broadcaster.go:253] Server rejected event '&v1.Event{TypeMeta:v1.TypeMeta{Kind:"", APIVersion:""}, ObjectMeta:v1.ObjectMeta{Name:"testpod.17329128ddba5fde", GenerateName:"", Namespace:"default", SelfLink:"", UID:"a8f7410c-a383-4094-9abd-18c0b368cc0a", ResourceVersion:"18599666", Generation:0, CreationTimestamp:time.Date(2022, time.December, 20, 17, 40, 16, 0, time.Local), DeletionTimestamp:, DeletionGracePeriodSeconds:(*int64)(nil), Labels:map[string]string(nil), Annotations:map[string]string(nil), OwnerReferences:[]v1.OwnerReference(nil), Finalizers:[]string(nil), ManagedFields:[]v1.ManagedFieldsEntry{v1.ManagedFieldsEntry{Manager:"event_agent_app", Operation:"Update", APIVersion:"events.k8s.io/v1", Time:time.Date(2022, time.December, 20, 17, 40, 16, 0, time.Local), FieldsType:"FieldsV1", FieldsV1:(*v1.FieldsV1)(0xc000012c18), Subresource:""}}}, EventTime:time.Date(2022, time.December, 20, 17, 40, 16, 412565000, time.Local), Series:(v1.EventSeries)(0xc00040a3e0), ReportingController:"test", ReportingInstance:"test-F6s-nopci-VM-4013", Action:"&", Reason:"Published", Regarding:v1.ObjectReference{Kind:"Pod", Namespace:"default", Name:"testpod", UID:"_;();T_warn;array;array;Warning", APIVersion:"v1", ResourceVersion:"", FieldPath:""}, Related:(v1.ObjectReference)(nil), Note:"sd", Type:"Warning", DeprecatedSource:v1.EventSource{Component:"", Host:""}, DeprecatedFirstTimestamp:time.Date(1, time.January, 1, 0, 0, 0, 0, time.UTC), DeprecatedLastTimestamp:time.Date(1, time.January, 1, 0, 0, 0, 0, time.UTC), DeprecatedCount:0}': 'Event "testpod.17329128ddba5fde" is invalid: series.count: Invalid value: "": should be at least 2' (will not retry!)
In the event_broadcaster.go code, we see if the event is isomorphic, the event series is initialized with count as 1. If the event is posted two times only, then the series count would be 1 and we would get this error.
Instead of initializing the series to 1 when recordToSink() detects duplicate event, shouldn't we initializing to 2 taking into account of both original event and duplicate event? Otherwise, could you please let me know why api-server is throwing this error?
Looking forward to get response on this issue.
Thank you very much!

Thank you very much for your comment on this issue.
I have not yet tried this workaround. I would like to find when this would happen meaning when we would get this error. Is there anything wrong that leads to this scenario. I would like to find out.
I will also try this workaround and will make an update here.
In the meantime, could you please let me know how did you hit this error?
Thank you very much!

Hi @smuraligit I was using a python client, not a go client. Here is a simplified script to run on the pod.
from kubernetes import client
from kubernetes.client.rest import ApiException

if __name__ == '__main__':
    namespace = "example-ns"

    cr = {
        'apiVersion': 'events.k8s.io/v1',
        'kind': 'Event',
        'metadata': {
            'name': 'name-app1-a069xx',
            'namespace': 'example-ns',
            'ownerReferences': [{
                    'apiVersion': 'events.k8s.io/v1',
                    'kind': 'NameApp',
                    'name': 'name-app1',
                    'uid': 'c78b7fe0-5708-49da-8e51-3621358f4040',
                    'controller': True
            }]
        },
        'type': 'Normal',
        'eventTime': '2023-01-25T12:22:02.024897Z',
        'series': {
            'lastObservedTime': '2023-01-25T12:22:02.024897Z'
        },
        'reason': 'TestStarted',
        'action': 'TestStarted',
        'note': 'My nice event message here',
        'regarding': {
            'namespace': 'example-ns',
            'kind': 'NameApp',
            'name': 'name-app1',
            'uid': 'c78b7fe0-5708-49da-8e51-3621358f4040'
        },
        'reportingController': 'pod/' + os.environ['HOSTNAME'],
        'reportingInstance': 'name-app1'
    }

    events_api = client.EventsV1Api()
    try:
        events_api.create_namespaced_event(namespace, cr)
    except ApiException as e:
        print("Exception on creating Event: %s\n" % e)

And here is the error I got
Exception on creating Event: (422)
Reason: Unprocessable Entity
HTTP response headers: HTTPHeaderDict({'Audit-Id': '272c1f33-ce5a-4c88-a442-370219ef3fce', 'Cache-Control': 'no-cache, private', 'Content-Type': 'application/json', 'Strict-Transport-Security': 'max-age=31536000; includeSubDomains; preload', 'X-Kubernetes-Pf-Flowschema-Uid': 'd6363d35-55ce-41c8-91da-83ead4b8c8fc', 'X-Kubernetes-Pf-Prioritylevel-Uid': 'dc600c67-6df9-49a2-8d6a-2902a11351c9', 'Date': 'Wed, 25 Jan 2023 12:22:02 GMT', 'Content-Length': '378'})
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"Event \"name-app1-a069xx\" is invalid: series.count: Invalid value: \"\": should be at least 2","reason":"Invalid","details":{"name":"name-app1-a069xx","kind":"Event","causes":[{"reason":"FieldValueInvalid","message":"Invalid value: \"\": should be at least 2","field":"series.count"}]},"code":422}

hi @smuraligit I confirm that explicitly setting series.count to 2 in the CR used for the event creation worked for my application and fixed the error
Reason: Unprocessable Entity
Event \"name-app1-a069xx\" is invalid: series.count: Invalid value: \"\": should be at least 2

Here is the fix
'series': {
    'lastObservedTime': evtTime,
    'count': 2
}

Here is the complete example
from kubernetes import client
from kubernetes.client.rest import ApiException

if __name__ == '__main__':
    namespace = "example-ns"

    cr = {
        'apiVersion': 'events.k8s.io/v1',
        'kind': 'Event',
        'metadata': {
            'name': 'name-app1-a069xx',
            'namespace': 'example-ns',
            'ownerReferences': [{
                    'apiVersion': 'events.k8s.io/v1',
                    'kind': 'NameApp',
                    'name': 'name-app1',
                    'uid': 'c78b7fe0-5708-49da-8e51-3621358f4040',
                    'controller': True
            }]
        },
        'type': 'Normal',
        'eventTime': '2023-01-25T12:22:02.024897Z',
        'series': {
            'lastObservedTime': '2023-01-25T12:22:02.024897Z'
        },
        'reason': 'TestStarted',
        'action': 'TestStarted',
        'note': 'My nice event message here',
        'regarding': {
            'namespace': 'example-ns',
            'kind': 'NameApp',
            'name': 'name-app1',
            'uid': 'c78b7fe0-5708-49da-8e51-3621358f4040'
        },
        'reportingController': 'pod/' + os.environ['HOSTNAME'],
        'reportingInstance': 'name-app1'
    }

    events_api = client.EventsV1Api()
    try:
        events_api.create_namespaced_event(namespace, cr)
    except ApiException as e:
        print("Exception on creating Event: %s\n" % e)

Hello @tkrishtop
Thank you for your quick response on this issue.
Thank you for verifying the fix. I see that you are using python client and not client-go's high level API. In such case, you can initialize the count attribute of series to 2 and it would work fine. In such cases, you would not publish your fix for merging into the open source as you are making the change into your application. Please correct me if I'm wrong.
However, since I am using client-go high level api recordToSink() in client-go/tool/events/event_broadcaster.go, we need to make the change to the open source code and verify the fix and publish the PR.
Please let me know your thoughts. I am surprised how not too many people have observed this issue.

Hello @tkrishtop
One more question -
Why do you need to initialize series structure when you are generating only one event? We need series when we see duplicate events.
Thanks,
Srini

Why do you need to initialize series structure when you are generating only one event?

Hi @smuraligit, I apologize for oversimplifying. Actually, my application generates thousands of events, some of which are duplicates. I added the loop for _ in range(2): in the example.

I see that you are using python client and not client-go's high level API. In such case, you can initialize the count attribute of series to 2 and it would work fine.

You can also use this workaround with the Go client. I'm not familiar with the structure of your application. If you're unable to modify the CR directly like I did, you could add a decorator for the event object to manually overwrite the event initialization and force  series.count = 2.

In such cases, you would not publish your fix for merging into the open source as you are making the change into your application. Please correct me if I'm wrong.

Do you need help with the fix PR? To be honest, since we're encountering the same issue in both the Go and Python clients, I'm not certain if the series initialization is the correct fix. I'd rather test more and then discuss the issue with some community maintainers.

Hello @tkrishtop,
Thank you for your response.
Actually, we are using the high level API which initializes the event series for us. You can see the code here at https://github.com/kubernetes/client-go/blob/master/tools/events/event_broadcaster.go, line 186.
I agree with you that I also need to test and see under what circumstances this error is being hit before publishing the changes for review.
Thank you so much!


UI tests

Motivation and context

How has this been tested?

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added a description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

I see ESLint pipeline and CI / e2e_testing (actions_users) pipeline failed..


cirq.approx_eq is inconsistent for sets

Description of the issue
cirq.approx_eq(val1, val2) delegates to _approx_eq_iterables if the two values are iterable, which in turn compares the elements of each value in order. This is a problem if the values are sets, which are not guaranteed to be iterated in a fixed order
a downstream effect of this is that cirq.approx_eq is inconsistent for operations with interchangeable qubits, due to the comparison of the frozensets returned by GateOperation._group_interchangeable_qubits()
How to reproduce the issue
unfortunately this is difficult to reproduce consistently because the iteration order of sets is inconsistent. But some variation of the following will probably fail:
for n in range(10, 20):
    assert cirq.approx_eq(frozenset(cirq.LineQubit.range(n)), frozenset({*cirq.LineQubit.range(n)}))

Cirq version
1.4.0.dev20231205195308

csynkque discussion - options
(a) sort when comparing two sets
(b) raise exception on comparison of two unordered sequences
Needs investigation what are impacts of these choices on the rest of cirq functionality;
would there be backwards incompatible change?

(b) raise exception on comparison of two unordered sequences

wouldn't this break approximate comparison of operations constructed from InterchangeableQubitGates (because their _value_equality_approximate_values_ contains a frozenset)?
the downstream effect of this issue (which is perhaps more important than the example i posted) is that cirq.approx_eq is also inconsistent for these operations, e.g. when i run the following:
for i in range(10):
    op1 = cirq.CZ(cirq.q(i), cirq.q(10))
    op2 = cirq.CZ(cirq.q(10), cirq.q(i))
    if cirq.approx_eq(op1, op2):
        print(f"{op1} == {op2}")
    else:
        print(f"{op1} != {op2}")

i get
CZ(q(0), q(10)) == CZ(q(10), q(0))
CZ(q(1), q(10)) == CZ(q(10), q(1))
CZ(q(2), q(10)) == CZ(q(10), q(2))
CZ(q(3), q(10)) == CZ(q(10), q(3))
CZ(q(4), q(10)) != CZ(q(10), q(4))
CZ(q(5), q(10)) == CZ(q(10), q(5))
CZ(q(6), q(10)) == CZ(q(10), q(6))
CZ(q(7), q(10)) != CZ(q(10), q(7))
CZ(q(8), q(10)) == CZ(q(10), q(8))
CZ(q(9), q(10)) == CZ(q(10), q(9))

it does seem like sorting makes sense though, or even just falling back on exact (==) equality instead of _approx_eq_iterables when comparing unordered sequences?

I'm interested in looking into this issue. Will start getting more context on what's wrong.


More options for notifications' alarms

I would like to have many alarms options like "1 hour before", "1 day before", "as the event's time", "1 week before", "3 hours before", etc.
More than one alarm is selectable simultaneously. For example I can set "one day before" and "1 hour before". Both activated.

Done! I think I will release a new version of the application tomorrow


use .getHeader

response.header fails back to standard method response.getHeaders on Node >= 7.7 (i.e. on all live Node versions today), so, it overkill to to get all headers first, then do field.toLowercase (specially without typechecking of field).
response.getHeader is a standard method since Node 0.x and it's doing all lowercasing internally.
Additionally, coercing to empty string is invalid here, as headers may contain non-string values before being sent to network, for example, Content-Length may be a number value of 0.

it is related with https://github.com/koajs/koa/pull/1394
Because we change all headers to string when set, so get header default to empty string is reasonable.
I'm -0 on both change even in the next major version, the current implementation is also reasonable, the new implementation does not have much benefit but introduces some incompatible changes.

@dead-horse new implementation do have benefits, indeed:

Empty string is a valid value for HTTP header (see https://github.com/eclipse/jetty.project/issues/1116 for example), and return it in case where header wasn’t set at all is an error.

It’s rare and nowhere documented that ctx.response.set(‘foo’, 5) !== ctx.response.get(‘foo’). It’s even more confusing taking in account that ctx.res.setHeader(‘foo’, 5) === ctx.res.getHeader(‘foo’)

Node built in response.getHeader documentation directly says that return type is the same as response.setHeader. Koa documentation says nothing about conversion to string. So, any developer relying on a fact that ctx.response.get must always return a string is relying on a bug, that may be and should be fixed in patch/minor version.

More with current implementation:
ctx.response.header['content-length'] = 0;
ctx.response.get('Content-Length); // => ''

FYI, I think it's better to rebase the branch rather than to merge in changes (though if you rebase and squash the result will be equivalent). Just to omit some commit bloat I tend to stay clear from merge-commits if possible or squash. Others might disagree.

@fl0w that's the GitHub suggested way to Update branch, but anyway you can squash at merging, right?

@tinovyatkin koa team seems don't like merge, can we together fork koa and fix many problem? i think you are right.


does this work with limesdr mini ? its a transmit transive tool

Hi wolfyy59.  To my knowledge, this only works on a Raspberry Pi.  Maybe check with https://github.com/F5OEO/rpitx to see if they support other hardware.


tabs: responsive bug

Bug
Expected behavior
Arrows should disappear when resizing the tabs.
Actual behavior
If you resize the tabs bar and interact with the arrow buttons, when you resize the screen again, the arrow buttons do not go away.  This seems to only be an issue in Firefox.
Screen recording attached below.
Steps to repro

Go to the https://app.pluralsight.com/search in Firefox
Resize the browser to be smaller
Interact with the tabs arrow button that appears when you make the page rather small
Resize the browser to be bigger
Tabs arrow button does not disappear, even when you interact with it

Related packages
@pluralsight/ps-design-system-tab: "^7.6.20"
Environment

OS - MacOS 10.15.13
Browser version - Firefox v70.0.1

I am unable to attach the .mov file with the screen recording, but feel free to message me on slack and I can send it along!


[Question] How to change how ShowNonSerializedFields are rendered

So I'm doing a RPG game and i have bunch of stats and i would like to have the non serialized values to show at their places in inspector

This at this point is not possible, because NA will always render NonSerializedFields and buttons at last.
A workaround is to write a extended script. This is what i use in my project.
Delete all non-functional (because I have many other drawer extension as well).
This works on my project but because the extensions not included here, and many custom functions I used for myself, not guaranteed.
gist link
And add a [Ordered] so the inspector know which comes first.


refactor: use Public Sans and Reddit Mono on PAPI docs site

Overview
Updating the Python API docs to use our design system proportional and fixed-width fonts.
Test Plan

public sansbox
tested on macOS Safari and Chrome, on computers with and without the fonts installed locally

Changelog

Open Sans -> Public Sans
using Reddit Mono for monospace
other minor CSS tweaks

Review requests
is it pretty?
should we also get rid of Akko H1s?
is there a way to get rid of the 1px of white space between the light grey background box of <a><code> and the dotted border below it? i've tried every CSS kludge i know of.

also need testing assistance on

[ ] Windows Chrome
[ ] Windows Edge
[ ] Linux browsers

Risk assessment
low

Going ahead and merging this so I can continue on bigger style changes (new top and bottom nav). If there are any issues caused by this PR, I can address it as part of that or in a separate followup.


[textfield]: Cannot read property nativeElement of undefined

I'm very interested in v5.0.4 release, since I've migrated my app to angular-mdc v5, but the forms are not rendered due to this bug, so I cannot release.

@antidote2 Sorry to hear that! v5.0.4 was published earlier today, hopefully it resolved. If your app uses mdc-radio I plan to publish v5.0.5 in a bit to resolve another Ivy engine bug..

Thank you! You released faster than I could write my comment. The forms work well now. Like the new sass approach: it's cleaner.
Beste Grüße
Anatoli

From: Dominic Carretto<EMAIL_ADDRESS>Sent: Wednesday, February 19, 2020 11:11:46 PM
To: trimox/angular-mdc-web<EMAIL_ADDRESS>Cc: antidote2<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [trimox/angular-mdc-web] [textfield]: Cannot read property nativeElement of undefined (#2113)
@antidote2https://github.com/antidote2 Sorry to hear that! v5.0.4 was published earlier today, hopefully it resolved. If your app uses mdc-radio I plan to publish v5.0.5 in a bit to resolve another Ivy engine bug..
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/trimox/angular-mdc-web/issues/2113?email_source=notifications&email_token=AAHPC6Q2YXNOWM6RDO36VMDRDWVCFA5CNFSM4KXQFH42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMJ4MOQ#issuecomment-588498490, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAHPC6U6LDWA5AV2SQOO5PDRDWVCFANCNFSM4KXQFH4Q.


UMAP produces different results with the same parameters for a loop

hello，
why the results are inconsistent for each loop？ where the parameters and data are not changed.
The code is as follows：

The results are as follows：

Why are they different？ please.
Thanks a lot in advance.
@lmcinnes

Reference https://umap-learn.readthedocs.io/en/latest/reproducibility.html

Have a look at https://github.com/rapidsai/cuml/issues/5099#issuecomment-1396363524
Providing a random state and init="random" to UMAP (i.e. UMAP(n_neighbors=15, random_state=1, init="random")) generates deterministic behavior for me.


Can I get rid of fetch()?

Why does this source code do fetch() to fetch fonts and css? I want to eliminate fetch() so that it can be used in offline mobile apps.
https://github.com/ThatOpen/engine_components/blob/fc953a6fce8e2160b690a58333a65144a9ac1914/src/ui/UIManager/index.ts#L150-L163

Why does this source code do fetch() to fetch fonts and css?

It's used to style the UI.

I want to eliminate fetch() so that it can be used in offline mobile apps.

You could use https://www.npmjs.com/package/patch-package to load fonts and css from a local file, or better should be included in the repo.

Fixed!


Clean install Centos 8 release => yum / dnf issue

after installing i could login with no password, so I tried changing it, but passwd did not appear to be installed. Tried installing it:
dnf install passwd
Last metadata expiration check: 0:00:59 ago on Mon 13 Apr 2020 05:24:57 PM CEST.
Dependencies resolved.
========================================================================================================================
 Package                    Architecture              Version                           Repository                 Size
========================================================================================================================
Installing:
 passwd                     x86_64                    0.80-2.el8                        BaseOS                    114 k
Installing dependencies:
 libuser                    x86_64                    0.62-23.el8                       BaseOS                    417 k

Transaction Summary
========================================================================================================================
Install  2 Packages

Total size: 531 k
Installed size: 2.4 M
Is this ok [y/N]: y
Downloading Packages:
[SKIPPED] libuser-0.62-23.el8.x86_64.rpm: Already downloaded
[SKIPPED] passwd-0.80-2.el8.x86_64.rpm: Already downloaded
Running transaction check
Transaction check succeeded.
Running transaction test
Transaction test succeeded.
Running transaction
RPM: error: db5 error(-30969) from dbenv->open: BDB0091 DB_VERSION_MISMATCH: Database environment version mismatch
RPM: error: cannot open Packages index using db5 -  (-30969)
RPM: error: cannot open Packages database in /var/lib/rpm
The downloaded packages were saved in cache until the next successful transaction.
You can remove cached packages by executing 'dnf clean packages'.
Error: Could not run transaction.

Hmm after reinstalling it seems to work.


Enabling Registration of Jetty's MBeans

I did not see any documentation to enable registring jetty's mbeans. This is very useful for instrumentation purpose of the server. Is there a way to do it using sparkjava.

....let me rephrase the question to clarify my intention.....

I want to register JettyStatisticsHandler and expose the session statistics through JMX.
Also want to make sparkjava log access log

is there a way to do this today in sparkjava? If not i am happy to contribute.


No way of specifying path to the cabal file.

The cabal-add tool looks up the cabal file by searching the current working directory.
I want to add the cabal-add tool to the hls and it would be very beneficial to have a way of specifying the path to the cabal file.
To my understanding, the --project-file is not related to this request (and also I failed to make this option work in this scenario).

Thanks for the report, I've just pushed some fixes for --project-file. Is it any better now?
If you want to add cabal-add to HLS, I'd suggest using API instead of calling an executable: https://hackage.haskell.org/package/cabal-add-0.1/candidate/docs/Distribution-Client-Add.html. It allows to specify an explicit Cabal file, parseCabalFile.

Thank you for the fix!
Now --project-file works with both relative and absolute paths to the cabal.project file. I wonder, is there a way to specify just a cabal file by itself without using API.
I wanted to make a solution using just API at first, but had trouble cerating an input for the executeConfig. I will try this route once again.

I wonder, is there a way to specify just a cabal file by itself without using API.

This is not planned. The reason is that cabal-add is supposed to be used as Cabal external command and as such follows Cabal design decisions. Cabal does not allow you to pass *.cabal file, only cabal.project, so cabal-add behaves similarly.

I wanted to make a solution using just API at first, but had trouble creating an input for the executeConfig. I will try this route once again.

In a nutshell you can do something like this:
https://github.com/Bodigrim/cabal-add/blob/0cb197d7cf31630c2e61f59d9a14f1c0a00af971/app/Main.hs#L81-L97
Feel free to ask questions or ping me in HLS issue.
Once you get a prototype working, I'll be happy to make a Hackage release to simplify setup.

Cabal does not allow you to pass *.cabal file, only cabal.project, so cabal-add behaves similarly.
I didn't know that. Thank you for the explanation!

Thank you for the pointers on the Config usage, it will help.
I'll let you know once I have a working proof-of-concept solution.


Fixed the ForTaskExecutor, which was not properly running when the list to enumerate contained only 1 item

Many thanks for submitting your Pull Request :heart:!
What this PR does / why we need it:

Fixes the ForTaskExecutor, which was not properly running when the list to enumerate contained only 1 item

Fixes #462


For Fish: i am facing the bellow issue.

echo 'setenv SWIFTENV_ROOT "$HOME/.swiftenv"' >> ~/.config/fish/config.fish
-bash: /Users/macbook/.config/fish/config.fish: No such file or directory

This error would indicate you are using bash and not fish? Is that correct?
The error is saying you do not have a fish config, perhaps that is because you are using bash and not fish?