tweaking auth again

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


pages: admin not showing content column

I guess after the introduction of this commit:
https://github.com/inveniosoftware/invenio/commit/47e2031f6d9b95eb13d5fc0bbe0df3cf7b1db09d
The Flask-Admin for the pages module does not show the Content anymore (which is quite important ;) )

Any idea why this might be? @hachreak

It seems that the problem is when you use with_variant().
db.Text().with_variant(db.Text(length=2**32-2), 'mysql')

I asked directly to Mike for this solution.
Maybe Flask-Admin can't undestand it because it returns a Variant Object?
See: https://bitbucket.org/zzzeek/sqlalchemy/issue/3370/text-and-length#comment-17513621

Maybe you can create an issue in https://github.com/flask-admin/flask-admin @hachreak ?

The issue is still open https://github.com/flask-admin/flask-admin/issues/912 for the moment.

This should help: form_overrides = dict(content=TextAreaField)
https://github.com/zenodo/invenio/pull/7/files#diff-b83bf3bae0fb93452518c475004ffe97R89

@bouzlibop can you please submit a patch for Invenio-Pages?


3.5.0 PHP version constraints

https://packagist.org/packages/onelogin/php-saml if you compare tags 3.4.1 and 3.5.0 the PHP version constraints are downgraded.
Was this intended?
I am using 3.4.1 on PHP 7.2 - 7.4 without any issues.

+1

It's much funnier #449

@arokettu suggest to close that to avoid duplication 👍

I'm closing this as the issue is now resolved.
https://github.com/onelogin/php-saml/releases/tag/3.5.1


How may I tell the formatter, not to send my comments to a new line?

Originally requested VSCode to implement this feature.
I put comments on ending divs like the following

Some heading

Some text for the card.

I have also configured my emmet to put the closing div tag comments on the same line as the closing tag.
The problem:
When I format the document inside VSCode (my editor of choice) which uses js-beautify under the hood, it sends all of my closing div comments to the new line like the following,

Some heading

Some text for the card.

Which is a problem for me, and I have to manually backspace them back to the closing div tags.
I requested VSCode team to provide a setting like "html.format.commentOnNewLine": true, which I set to false to get the desired effect, but they replied that they're using js-beautify under the hood which is doing the work, so I should file the feature here, which I did. :)
Thank You for the nice beautifier and the awesome work!

I agree with pillaiindu. I have the same problem. I would like to see this implemented.

@gunther3613
Please review the CONTRIBUTING.md and submit a PR.

I stopped using js-beautify because of this issue.

@pillaiindu Same, I came from Netbeans, where the only formatting it did was (mainly) alter indentation, but it seems like this just tries to do too much, and altering my newlines is as annoying as it gets. It's too bad Netbeans just isn't as all around a good editor as VS Code, but since then I've just manually been formatting my code and I haven't looked back to either

#426

I still have the problem. I've been looking for a solution for two hours because it's driving me nuts, but still no luck. Below are my version specs.

@gkarapeev
Not sure why that would be.  Please reproduce on beautifier.io and/or show examples of your inputs.

Using VSCode (1.37.1) with Beautify plugin installed (1.5.0) and set to default formatter on Linux it only works properly for me if I rewrite the comments next to the tag I want to have them.
If I open a file having previously written comments and try to format it, HTML one-line comments move to the next line which is absolutely undesirable and annoying cause it "breaks" my already formatted file =(

@gcjuan Please reproduce this on https://beautifier.io using the HTML beautifier. If it does happen there, please not the settings here.

I cannot reproduce it on https://beautifier.io the way I do it in VSCode. It should mean there's a problem in the plugin implementation on the code editor cause it works time to time and depending if you've opened a file having already written comments or you write them afterwards.
Thanks for all!


xcode ios build says plugin missing

It gives error that plugin is missing under product=> archive

try full restart


[Question] ctr fails to pull images from insecure-registry.

Description
I've set inscecure registry in /etc/containerd/config.toml like below:
      bin_dir = "/opt/cni/bin"
      conf_dir = "/etc/cni/net.d"
      conf_template = ""
    [plugins.cri.registry]
      [plugins.cri.registry.mirrors]
        [plugins.cri.registry.mirrors."registry.k8s.my.io"]
          endpoint = ["http://registry.k8s.my.io:5000"]
        [plugins.cri.registry.mirrors."docker.io"]
          endpoint = ["https://registry-1.docker.io"]

Then restart containerd.
systemctl stop containerd; systemctl start containerd
But still fail when I try to pull image via ctr
ctr image pull registry.k8s.my.io:5000/coredns/coredns:1.2.2
elapsed: 0.1 s                                     total:   0.0 B (0.0 B/s)
ctr: failed to resolve reference "registry.k8s.my.io:5000/coredns/coredns:1.2.2": failed to do request: Head https://registry.k8s.my.io:5000/v2/coredns/coredns/manifests/1.2.2: http: server gave HTTP response to HTTPS client

What should I do to let ctr/kubernetes pull images from insecure-registry.
Completely containerd config
root = "/var/lib/containerd"
state = "/run/containerd"
oom_score = 0

[grpc]
  address = "/run/containerd/containerd.sock"
  uid = 0
  gid = 0
  max_recv_message_size = 16777216
  max_send_message_size = 16777216

[debug]
  address = ""
  uid = 0
  gid = 0
  level = "debug"

[metrics]
  address = ""
  grpc_histogram = false

[cgroup]
  path = ""

[plugins]
  [plugins.cgroups]
    no_prometheus = false
  [plugins.cri]
    stream_server_address = "<IP_ADDRESS>"
    stream_server_port = "0"
    enable_selinux = false
    sandbox_image = "k8s.gcr.io/pause:3.1"
    stats_collect_period = 10
    systemd_cgroup = false
    enable_tls_streaming = false
    max_container_log_line_size = 16384
    [plugins.cri.containerd]
      snapshotter = "overlayfs"
      no_pivot = false
      [plugins.cri.containerd.default_runtime]
        runtime_type = "io.containerd.runtime.v1.linux"
        runtime_engine = ""
        runtime_root = ""
      [plugins.cri.containerd.untrusted_workload_runtime]
        runtime_type = ""
        runtime_engine = ""
        runtime_root = ""
    [plugins.cri.cni]
      bin_dir = "/opt/cni/bin"
      conf_dir = "/etc/cni/net.d"
      conf_template = ""
    [plugins.cri.registry]
      [plugins.cri.registry.mirrors]
        [plugins.cri.registry.mirrors."registry.k8s.my.io"]
          endpoint = ["http://registry.k8s.my.io:5000"]
        [plugins.cri.registry.mirrors."docker.io"]
          endpoint = ["https://registry-1.docker.io"]
    [plugins.cri.x509_key_pair_streaming]
      tls_cert_file = ""
      tls_key_file = ""
  [plugins.diff-service]
    default = ["walking"]
  [plugins.linux]
    shim = "containerd-shim"
    runtime = "runc"
    runtime_root = ""
    no_shim = false
    shim_debug = false
  [plugins.opt]
    path = "/opt/containerd"
  [plugins.restart]
    interval = "10s"
  [plugins.scheduler]
    pause_threshold = 0.02
    deletion_threshold = 0
    mutation_threshold = 100
    schedule_delay = "0s"
    startup_delay = "100ms"

Steps to reproduce the issue:

insecure-registry
restart containerd
pull image via ctr

Describe the results you received:
ctr: failed to resolve reference "registry.k8s.my.io:5000/coredns/coredns:1.2.2": failed to do request: Head https://registry.k8s.my.io:5000/v2/coredns/coredns/manifests/1.2.2: http: server gave HTTP response to HTTPS client
Describe the results you expected:
ctr/kubernetes can pull images from insecure-registry.
Output of containerd --version:
containerd github.com/containerd/containerd v1.2.0 c4446665cb9c30056f4998ed953e6d4ff22c7c39

@Alvin-Lau please try this
ctr image pull --plain-http=true registry.k8s.my.io:5000/coredns/coredns:1.2.2

@Alvin-Lau Can you try crictl pull registry.k8s.my.io:5000/coredns/coredns:1.2.2?
plugins.cri.registry is a config for the cri plugin, and the cri plugin should use http if you specify that in the config.
If it doesn't work, we should fix it.

It doesn't work. "crictl pull registry.k8s.my.io:5000/coredns/coredns:1.2.2"

"crictl pull registry.k8s.my.io/coredns/coredns:1.2.2" doesn't work, neither.

@Alvin-Lau the method, which is ctr image pull --plain-http=true registry.k8s.my.io:5000/coredns/coredns:1.2.2 can work? could you try it? Thanks

ctr image pull --plain-http=true registry.k8s.my.io:5000/coredns/coredns:1.2.2  works.

"crictl pull registry.k8s.my.io/coredns/coredns:1.2.2" doesn't work, neither.

This should work. Could you put the error message here?

crictl pull registry.k8s.my.io/coredns/coredns:1.2.2
Only oneline output
FATA[0010] failed to connect: failed to connect: context deadline exceeded

"crictl pull registry.k8s.my.io/coredns/coredns:1.2.2" doesn't work, neither.

This should work. Could you put the error message here?

crictl pull registry.k8s.my.io/coredns/coredns:1.2.2
Only oneline output:
FATA[0010] failed to connect: failed to connect: context deadline exceeded

@Alvin-Lau could you mind to provide more detail from backend, not just client side?

crictl pull registry.k8s.my.io/coredns/coredns:1.2.2
FATA[0010] failed to connect: failed to connect: context deadline exceeded

Hm, this just look like either of the following:

you containerd is not running
you didn't config /etc/crictl.yaml with runtime-endpoint: unix:///run/containerd/containerd.sock.
you are not running crictl with sudo.

crictl pull registry.k8s.my.io/coredns/coredns:1.2.2
FATA[0010] failed to connect: failed to connect: context deadline exceeded

Hm, this just looks like one of the following:

you containerd is not running

containerd is running.

you didn't config /etc/crictl.yaml with runtime-endpoint: unix:///run/containerd/containerd.sock.
I didn't config '/etc/crictl.yaml'. Let me try it.
you are not running crictl with sudo.
I did run 'crictl' with 'sudo'

@Alvin-Lau could you mind to provide more detail from backend, not just client side?

I'm fresh with crictl. Would you mind to point me about how to get backend log?

If this is the case, this seems to be a bug to me. Let me see.
Are you sure you are using the config from #2758 (comment)?
Can you run crictl info and paste the result here? Thanks!

I'm sorry. I get right output after i restart containerd.
sudo crictl pull registry.k8s.my.io/coredns/coredns:1.2.2
DEBU[0000] PullImageRequest: &PullImageRequest{Image:&ImageSpec{Image:registry.k8s.my.io/coredns/coredns:1.2.2,},Auth:nil,SandboxConfig:nil,}
DEBU[0000] PullImageResponse: &PullImageResponse{ImageRef:sha256:367cdc8433a4581c3c9d01501b92ecd79666d23c41232013a7b11e5f7acb7eed,}
Image is up to date for sha256:367cdc8433a4581c3c9d01501b92ecd79666d23c41232013a7b11e5f7acb7eed

Yeah, you need to restart containerd. Then at least we don't have a bug. :) Thanks!

Yeah, you need to restart containerd. Then at least we don't have a bug. :) Thanks!
crictl works, but ctr does not work.

sudo ctr image pull  registry.k8s.my.io:5000/coredns/coredns:1.2.2
ctr: failed to resolve reference "registry.k8s.my.io:5000/coredns/coredns:1.2.2": failed to do request: Head https://registry.k8s.my.io:5000/v2/coredns/coredns/manifests/1.2.2: http: server gave HTTP response to HTTPS client

sudo ctr image pull registry.k8s.my.io:5000/coredns/coredns:1.2.2
ctr: failed to resolve reference "registry.k8s.my.io:5000/coredns/coredns:1.2.2": failed to do request: Head https://registry.k8s.my.io:5000/v2/coredns/coredns/manifests/1.2.2: http: server gave HTTP response to HTTPS client

add --plain-http and it will work~

sudo ctr image pull registry.k8s.my.io:5000/coredns/coredns:1.2.2
ctr: failed to resolve reference "registry.k8s.my.io:5000/coredns/coredns:1.2.2": failed to do request: Head https://registry.k8s.my.io:5000/v2/coredns/coredns/manifests/1.2.2: http: server gave HTTP response to HTTPS client

add --plain-http and it will work~

Yes, add '--plain-http' works. Does ctr use the registry config in /etc/containerd/config.toml .

ctr is client side tools and it doesn't use the config from daemon. 😄

ctr is client side tools and it doesn't use the config from daemon. 😄

OK. I got it. Thanks for your help.

Thanks for your kindly replies. Let me close this.

@Random-Liu
I get the same error,
Output of containerd --version:
containerd github.com/containerd/containerd v1.2.2<PHONE_NUMBER>f7fe2f4e74d43e2fc7ccd237edcbce
Output of crictl --version:
crictl version v1.13.0
Output of ctr --version:
ctr github.com/containerd/containerd v1.2.2
Output of cat /etc/containerd/config.toml:
root = "/var/lib/containerd"
state = "/run/containerd"
oom_score = 0

[grpc]
  address = "/run/containerd/containerd.sock"
  uid = 0
  gid = 0
  max_recv_message_size = 16777216
  max_send_message_size = 16777216

[debug]
  address = "/run/containerd/debug.sock"
  uid = 0
  gid = 0
  level = "debug"
[metrics]
  address = ""
  grpc_histogram = false
[cgroup]
  path = ""
[plugins]
  [plugins.cgroups]
    no_prometheus = false
  [plugins.cri]
    stream_server_address = "<IP_ADDRESS>"
    stream_server_port = "0"
    enable_selinux = false
    sandbox_image = "k8s.gcr.io/pause:3.1"
    stats_collect_period = 10
    systemd_cgroup = false
    enable_tls_streaming = false
    max_container_log_line_size = 16384
    [plugins.cri.containerd]
      snapshotter = "overlayfs"
      no_pivot = false
      [plugins.cri.containerd.default_runtime]
        runtime_type = "io.containerd.runtime.v1.linux"
        runtime_engine = "/usr/local/bin/runc"
        runtime_root = "/var/lib/containerd"
      [plugins.cri.containerd.untrusted_workload_runtime]
        runtime_type = ""
        runtime_engine = ""
        runtime_root = ""
    [plugins.cri.cni]
      bin_dir = "/opt/cni/bin"
      conf_dir = "/etc/cni/net.d"
      conf_template = ""
    [plugins.cri.registry]
      [plugins.cri.registry.mirrors]
        [plugins.cri.registry.mirrors."registry.bc.**.**"]
          endpoint = ["http://registry.bc.**.**"]
        [plugins.cri.registry.mirrors."docker.io"]
          endpoint = ["https://registry-1.docker.io"]
    [plugins.cri.x509_key_pair_streaming]
      tls_cert_file = ""
      tls_key_file = ""
  [plugins.diff-service]
    default = ["walking"]
  [plugins.linux]
    shim = "/usr/local/bin/containerd-shim"
    runtime = "/usr/local/bin/runc"
    runtime_root = "/var/lib/containerd"
    no_shim = false
    shim_debug = true
  [plugins.opt]
    path = "/opt/containerd"
  [plugins.restart]
    interval = "10s"
  [plugins.scheduler]
    pause_threshold = 0.02
    deletion_threshold = 0
    mutation_threshold = 100
    schedule_delay = "0s"
    startup_delay = "100ms"

Output of crictl info:
{
  "status": {
    "conditions": [
      {
        "type": "RuntimeReady",
        "status": true,
        "reason": "",
        "message": ""
      },
      {
        "type": "NetworkReady",
        "status": true,
        "reason": "",
        "message": ""
      }
    ]
  },
  "config": {
    "containerd": {
      "snapshotter": "overlayfs",
      "defaultRuntime": {
        "runtimeType": "io.containerd.runtime.v1.linux",
        "runtimeEngine": "/usr/local/bin/runc",
        "runtimeRoot": "/var/lib/containerd",
        "options": null
      },
      "untrustedWorkloadRuntime": {
        "runtimeType": "",
        "runtimeEngine": "",
        "runtimeRoot": "",
        "options": null
      },
      "runtimes": null,
      "noPivot": false
    },
    "cni": {
      "binDir": "/opt/cni/bin",
      "confDir": "/etc/cni/net.d",
      "confTemplate": ""
    },
    "registry": {
      "mirrors": {
        "docker.io": {
          "endpoint": [
            "https://registry-1.docker.io"
          ]
        },
        "registry.bc.**.**": {
          "endpoint": [
            "http://registry.bc.**.**"
          ]
        }
      },
      "auths": null
    },
    "streamServerAddress": "<IP_ADDRESS>",
    "streamServerPort": "0",
    "enableSelinux": false,
    "sandboxImage": "k8s.gcr.io/pause:3.1",
    "statsCollectPeriod": 10,
    "systemdCgroup": false,
    "enableTLSStreaming": false,
    "x509KeyPairStreaming": {
      "tlsCertFile": "",
      "tlsKeyFile": ""
    },
    "maxContainerLogSize": 16384,
    "containerdRootDir": "/var/lib/containerd",
    "containerdEndpoint": "/run/containerd/containerd.sock",
    "rootDir": "/var/lib/containerd/io.containerd.grpc.v1.cri",
    "stateDir": "/run/containerd/io.containerd.grpc.v1.cri"
  },
  "golang": "go1.10.7"
}

Output of crictl -D pull registry.bc.**.**/master/ui-incident:latest:
DEBU[0000] PullImageRequest: &PullImageRequest{Image:&ImageSpec{Image:registry.bc.**.**/master/ui-incident:latest,},Auth:nil,SandboxConfig:nil,}
DEBU[0000] PullImageResponse: nil
FATA[0000] pulling image failed: rpc error: code = Unknown desc = failed to resolve image "registry.bc.**.**/master/ui-incident:latest": no available registry endpoint: failed to do request: Head https://registry.bc.**.**/v2/master/ui-incident/manifests/latest: dial tcp <IP_ADDRESS>:443: connect: no route to host

Hi,
I'm also facing this currently,:
Extra info, I'm using KinD and that is the node where I want to pull the images to.
System information:
containerd --version
containerd github.com/containerd/containerd 1.2.6-0ubuntu1

ctr --version
ctr github.com/containerd/containerd v1.2.0-531-gd71c7ada

I see that there's a discrepancy between the containerd version and ctr version, could this be the case?
crictl --version
crictl version v1.14.0

cat /etc/containerd/config.toml
disabled_plugins = ["aufs", "btrfs", "zfs"]
root = "/var/lib/containerd"
state = "/run/containerd"
oom_score = 0

[grpc]
  address = "/run/containerd/containerd.sock"
  uid = 0
  gid = 0
  max_recv_message_size = 16777216
  max_send_message_size = 16777216

[debug]
  address = ""
  uid = 0
  gid = 0
  level = ""

[metrics]
  address = ""
  grpc_histogram = false

[cgroup]
  path = ""

[plugins]
  [plugins.cgroups]
    no_prometheus = false
  [plugins.cri]
    stream_server_address = "<IP_ADDRESS>"
    stream_server_port = "0"
    enable_selinux = false
    sandbox_image = "k8s.gcr.io/pause:3.1"
    stats_collect_period = 10
    systemd_cgroup = false
    enable_tls_streaming = false
    max_container_log_line_size = 16384
    [plugins.cri.containerd]
      snapshotter = "overlayfs"
      no_pivot = false
      [plugins.cri.containerd.default_runtime]
        runtime_type = "io.containerd.runtime.v1.linux"
        runtime_engine = ""
        runtime_root = ""
      [plugins.cri.containerd.untrusted_workload_runtime]
        runtime_type = ""
        runtime_engine = ""
        runtime_root = ""
    [plugins.cri.cni]
      bin_dir = "/opt/cni/bin"
      conf_dir = "/etc/cni/net.d"
      conf_template = ""
    [plugins.cri.registry]
      [plugins.cri.registry.mirrors]
        [plugins.cri.registry.mirrors."docker.io"]
          endpoint = ["https://registry-1.docker.io"]
        [plugins.cri.registry.mirrors."docker.wdf.sap.corp:50000"]
          endpoint = ["http://docker.wdf.sap.corp:50000"]
    [plugins.cri.x509_key_pair_streaming]
      tls_cert_file = ""
      tls_key_file = ""
  [plugins.diff-service]
    default = ["walking"]
  [plugins.linux]
    shim = "containerd-shim"
    runtime = "runc"
    runtime_root = ""
    no_shim = false
    shim_debug = false
  [plugins.opt]
    path = "/opt/containerd"
  [plugins.restart]
    interval = "10s"
  [plugins.scheduler]
    pause_threshold = 0.02
    deletion_threshold = 0
    mutation_threshold = 100
    schedule_delay = "0s"
    startup_delay = "100ms"

I've tried not to use the port in the mirror, but it also wasn't working, so trying to follow this I tried this approach, with no success as well.
crictl info
{
  "status": {
    "conditions": [
      {
        "type": "RuntimeReady",
        "status": true,
        "reason": "",
        "message": ""
      },
      {
        "type": "NetworkReady",
        "status": true,
        "reason": "",
        "message": ""
      }
    ]
  },
  "config": {
    "containerd": {
      "snapshotter": "overlayfs",
      "defaultRuntime": {
        "runtimeType": "io.containerd.runtime.v1.linux",
        "runtimeEngine": "",
        "runtimeRoot": "",
        "options": null
      },
      "untrustedWorkloadRuntime": {
        "runtimeType": "",
        "runtimeEngine": "",
        "runtimeRoot": "",
        "options": null
      },
      "runtimes": null,
      "noPivot": false
    },
    "cni": {
      "binDir": "/opt/cni/bin",
      "confDir": "/etc/cni/net.d",
      "confTemplate": ""
    },
    "registry": {
      "mirrors": {
        "docker.io": {
          "endpoint": [
            "https://registry-1.docker.io"
          ]
        },
        "docker.wdf.sap.corp:50000": {
          "endpoint": [
            "http://docker.wdf.sap.corp:50000"
          ]
        }
      },
      "auths": null
    },
    "streamServerAddress": "<IP_ADDRESS>",
    "streamServerPort": "0",
    "enableSelinux": false,
    "sandboxImage": "k8s.gcr.io/pause:3.1",
    "statsCollectPeriod": 10,
    "systemdCgroup": false,
    "enableTLSStreaming": false,
    "x509KeyPairStreaming": {
      "tlsCertFile": "",
      "tlsKeyFile": ""
    },
    "maxContainerLogSize": 16384,
    "containerdRootDir": "/var/lib/containerd",
    "containerdEndpoint": "/run/containerd/containerd.sock",
    "rootDir": "/var/lib/containerd/io.containerd.grpc.v1.cri",
    "stateDir": "/run/containerd/io.containerd.grpc.v1.cri"
  },
  "golang": "go1.10.4"
}

Last, the output of the pull commands:
crictl pull  docker.wdf.sap.corp:50000/kubernetes-helm/tiller:v2.11.0
FATA[0000] pulling image failed: rpc error: code = Unknown desc = failed to resolve image "docker.wdf.sap.corp:50000/kubernetes-helm/tiller:v2.11.0": no available registry endpoint: failed to do request: Head https://docker.wdf.sap.corp:50000/v2/kubernetes-helm/tiller/manifests/v2.11.0: x509: certificate signed by unknown authority

ctr image pull --plain-http=true docker.wdf.sap.corp:50000/kubernetes-helm/tiller:v2.11.0
ctr: failed to resolve reference "docker.wdf.sap.corp:50000/kubernetes-helm/tiller:v2.11.0": unexpected status code http://docker.wdf.sap.corp:50000/v2/kubernetes-helm/tiller/manifests/v2.11.0: 400 Bad Request

It seems that in crictl it's not using the image mirror specs and ctr can't find the manifest for the image, but in my host machine it works when using docker and simply adding the registry as insecure:
docker pull docker.wdf.sap.corp:50000/kubernetes-helm/tiller:v2.11.0
v2.11.0: Pulling from kubernetes-helm/tiller
c67f3896b22c: Pull complete
c12cb9af1ab6: Pull complete
c1946e757e66: Pull complete
9f15637d9e6c: Pull complete
Digest: sha256:f6d8f4ab9ba993b5f5b60a6edafe86352eabe474ffeb84cb6c79b8866dce45d1
Status: Downloaded newer image for docker.wdf.sap.corp:50000/kubernetes-helm/tiller:v2.11.0


spirv-val: Add GLCompute to VUID 04644 message

Based on https://github.com/KhronosGroup/SPIRV-Tools/pull/4212 but the Vulkan spec now has GLCompute but realize the message was never updated

141/401    TestTargetEnvEqOpenglCompatWithOpenGlCompatShader -Failed-
/tmp/shaderc/build/glslc/glslc --target-env=opengl_compat -c /tmp/shaderc/build/glslc/test/tmp90eotpe_/tmpx0f2nmef.frag
Non-zero return code: -11

Cannot find file: /tmp/shaderc/build/glslc/test/tmp90eotpe_/tmpx0f2nmef.frag.spv

Seems like a CI fluke


[npm] Upgrade the dev dependencies

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

The main upgrade is eslint fixing an issue with the RestElement.

Looks like all PRs on the next branch fails with a

Bad response: 422 {"message":"Couldn't find a repository matching this job.","error":true}.


guess subcommand raise an exception when header name contains null name field.

Embulk: 0.9.5
The following data contains null name field.
The following data generate embulk example and changed header name.
null,account,time,purchase,comment
1,32864,2015-01-27 19:23:49,20150127,embulk
2,14824,2015-01-27 19:01:23,20150127,embulk jruby
3,27559,2015-01-28 02:20:02,20150128,"Embulk ""csv"" parser plugin"
4,11270,2015-01-29 11:54:36,20150129,NULL

% embulk guess test/seed.yml -o config.yml
2018-04-05 19:25:28.583 +0900: Embulk v0.9.5
2018-04-05 19:25:29.296 +0900 [INFO] (main): Started Embulk v0.9.5
... snip
org.jruby.exceptions.RaiseException: (NoMethodError) undefined method `strip' for nil:NilClass
	at org.jruby.RubyArray.map(org/jruby/RubyArray.java:2487)
	at RUBY.guess_lines(uri:classloader:/gems/embulk-0.9.5-java/lib/embulk/guess/csv.rb:165)
	at RUBY.guess(uri:classloader:/gems/embulk-0.9.5-java/lib/embulk/guess_plugin.rb:121)
	at RUBY.guess(uri:classloader:/gems/embulk-0.9.5-java/lib/embulk/guess_plugin.rb:24)

Error: (NoMethodError) undefined method `strip' for nil:NilClass

Thanks for reporting. It looks coming from "null" is processed specially.
https://github.com/embulk/embulk/blob/v0.9.5/embulk-standards/src/main/ruby/embulk/guess/csv.rb
Hmm, we may want to disable this special processing for headers, but I'm afraid it can have side effects. If it's not a quite urgent issue, let us leave it as a backlog to consider how to fix it.


need help regarding ubuntu

Hi please I have an issue, I wanted to scrap Google results exactly to find the number of search results in excel, I found this source here https://github.com/RomainFallet/keyworder it is what I want, but it didn't work because of the commands (sudo npm install
, sudo npm install<EMAIL_ADDRESS>--unsafe-perm=true --allow-root --no-save, npm run build).
Please let me know how to solve this problem.

Hi,
Ummm... I'm not sure what this has to do with this arduino library. Imma close this. Cheers.


Scan scripts without classes

Hi,
sorry for spamming issues ;-)
I'm currently working on a legacy system that still contains lots of scripts with procedural non-OO code. Violations of rules in these scripts are currently undetected, and it would be great if deptrac could find those. Scripts could be added to a layer e.g. by using the DirectoryCollector. Not sure if it's feasible to reliably detect includes/requires though.
I'm aware that the inner workings are tailored to classes, so this might be a bit hard to do.

Well, technically they are scanned and even analyzed (uses will be tracked) ;-)
I need to think about if it's worth it 🤔

FYI: It is possible to detect includes and requires with following expression https://github.com/nikic/PHP-Parser/blob/master/lib/PhpParser/Node/Expr/Include_.php

Hello, is there a hack momentarily to be able to scan non-OOP files?

@danielspk and @umulmrum if this is still relevant for you and if we can hammer out here a more precise definition of what exactly you are expecting, I will take a stab at it. :smile:

Hi @patrickkusebauch
I have a global functions file within a defined namespace. That file contains functions, not classes. Example:
<?php

declare(strict_types=1);

namespace App\Domain;

function demoA(): bool {
    return true;
}

I wish I could validate that those functions can only be called from certain namespaces (like I do with DDD layers of classes).

@danielspk I looked at the code at it does not look unreasonably difficult to treat functions the same way classes are treated. I expect to do it either over the weekend or next week.

@DanielBadura It would be immensely helpful to me if you could provide a sample repo on GitHub that I could test the changes against.


io.containerd.snapshotter.v1.overlayfs -> takes all space

Version (k3OS / kernel)
k3os version v0.20.7-k3s1r0
Architecture
x86_64
Describe the bug
Since upgrading from 0.19-dev5 to v0.20.7-k3s1r0, the file /var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs grows pretty fast, which causes to fail the node with disk pressure and a lot of evicted pods.
Expected behavior
Do not takes so much disk, which causes "/" to be full
Actual behavior
After a few minutes, this file grows, node is under disk pressure and removes the pods, which then reduces this file to a few hundred MB.
Additional context
Maybe, it happend fist, after upgrading Rancher to 2.5.9 to support new k3s (otherwise, Rancher isn't able to get status of the cluster / nodes / ... )
Diskspace from every node is 20GB.

@linuxmail /var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs is where containerd sets up overlayfs working directories. This means that your workload(s) are likely filling this up. If you are running into disk pressure with this location "filling up" with ~20GB of data it would seem that your root disk is under-allocated.

@linuxmail /var/lib/rancher/k3s/agent/containerd/io.containerd.snapshotter.v1.overlayfs is where containerd sets up overlayfs working directories. This means that your workload(s) are likely filling this up. If you are running into disk pressure with this location "filling up" with ~20GB of data it would seem that your root disk is under-allocated.

That said. If you are only seeing this behavior post-upgrade, considering manually pruning the rancher/k3os images on your node(s).


linalg.solve: Unclear error when using assume_a='her' with real matrix

My issue is about an obscure error that happens when using scipy.linalg.solve(a,b, assume_a='her') with a being a real matrix.
While raising an exception seems logical, the one that is raised makes it harder to diagnose the problem in user code.
(personally, when I first encountered the error, I went to check if the lapack/blas libraries were correctly installed/linked)
Reproducing code example:

import numpy as np
import scipy.linalg as spl
a = np.array([[0,1,2],[1,3,4],[2,4,6]])
a2 = np.asarray(a, dtype=complex)
b = np.array([7,8,9])

spl.solve(a,b, assume_a='sym')   # success: expected behavior
spl.solve(a2,b, assume_a='her')  # success: expected behavior
spl.solve(a2,b, assume_a='sym')  # success: expected behavior: this is a different assumption
spl.solve(a,b, assume_a='her')  # error, see below

Observed behavior (Error message) :

Traceback (most recent call last):
File "", line 1, in
File "path/python3.9/site-packages/scipy/linalg/basic.py", line 221, in solve
hecon, hesv, hesv_lw = get_lapack_funcs(('hecon', 'hesv',
File "path/python3.9/site-packages/scipy/linalg/blas.py", line 399, in getter
value = func(names, arrays, dtype, ilp64)
File "path/python3.9/site-packages/scipy/linalg/lapack.py", line 964, in get_lapack_funcs
return _get_funcs(names, arrays, dtype,
File "path/python3.9/site-packages/scipy/linalg/blas.py", line 358, in _get_funcs
raise ValueError(
ValueError: LAPACK function dhecon could not be found
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  ...

"Expected" behavior(s)
The exception that is raised is (to me, at least) not clear enough about what the user's blunder was.
I see two (mutually exclusive) possibilities:

assume_a is silently replaced by 'sym', because it is the same assumption as 'her' in the case of of a real matrix.
an exception is raised earlier, clearly pointing out the dtype mismatch

I attached two (draft-state) patches, one for each possibility (at the end of this report)
(personally, I would go with the former solution, which feels more coherent with the documentation of the spl.pinv and spl.pinvh functions, although I do not know what would truly be preferred for the project)
Scipy/Numpy/Python version information:
scipy : master branch, d98e31ec32b293da156d2c41fc8c09eda345ecff   (same behavior at version 1.6.1).
numpy: 1.20.1
python: 3.9.4 (releaselevel='final", serial=0)
blas/lapack linkage info (in case it's relevant):
lapack_mkl_info:
    libraries = ['mkl_rt', 'pthread']
    library_dirs = ['/opt/intel/mkl/lib/intel64']
    define_macros = [('SCIPY_MKL_H', None), ('HAVE_CBLAS', None)]
    include_dirs = ['/opt/intel/mkl', '/opt/intel/mkl/include', '/opt/intel/mkl/lib']
lapack_opt_info:
    libraries = ['mkl_rt', 'pthread']
    library_dirs = ['/opt/intel/mkl/lib/intel64']
    define_macros = [('SCIPY_MKL_H', None), ('HAVE_CBLAS', None)]
    include_dirs = ['/opt/intel/mkl', '/opt/intel/mkl/include', '/opt/intel/mkl/lib']
blas_mkl_info:
    libraries = ['mkl_rt', 'pthread']
    library_dirs = ['/opt/intel/mkl/lib/intel64']
    define_macros = [('SCIPY_MKL_H', None), ('HAVE_CBLAS', None)]
    include_dirs = ['/opt/intel/mkl', '/opt/intel/mkl/include', '/opt/intel/mkl/lib']
blas_opt_info:
    libraries = ['mkl_rt', 'pthread']
    library_dirs = ['/opt/intel/mkl/lib/intel64']
    define_macros = [('SCIPY_MKL_H', None), ('HAVE_CBLAS', None)]
    include_dirs = ['/opt/intel/mkl', '/opt/intel/mkl/include', '/opt/intel/mkl/lib']

notes

both patches were tested with the minimal example given
I did not (yet) check other functions for similar problems

two (mutually exclusive) patches:
patch 1 (draft): silently correct the user's mistake
diff --git a/scipy/linalg/basic.py b/scipy/linalg/basic.py
index d673d823c..db0a9ee93 100644
--- a/scipy/linalg/basic.py
+++ b/scipy/linalg/basic.py
@@ -169,6 +169,10 @@ def solve(a, b, sym_pos=False, lower=False, overwrite_a=False,
     if assume_a not in ('gen', 'sym', 'her', 'pos'):
         raise ValueError('{} is not a recognized matrix structure'
                          ''.format(assume_a))
+
+    # for a real matrix, describe it as "symmetric", not "hermitian" (lapack doesn't know what to do with real hermitian matrices)
+    if assume_a == 'her' and not np.iscomplexobj(a1):
+        assume_a = 'sym'

     # Deprecate keyword "debug"
     if debug is not None:

patch 2 (draft): explicitly raise a meaningful exception
diff --git a/scipy/linalg/basic.py b/scipy/linalg/basic.py
index d673d823c..6db5c4b1e 100644
--- a/scipy/linalg/basic.py
+++ b/scipy/linalg/basic.py
@@ -218,6 +218,10 @@ def solve(a, b, sym_pos=False, lower=False, overwrite_a=False,
         rcond, info = gecon(lu, anorm, norm=norm)
     # Hermitian case 'hesv'
     elif assume_a == 'her':
+        if not np.iscomplexobj(a1):
+            raise ValueError('input a, containing real values, '
+                             'cannot be assumed to be hermitian ("her"). '
+                             'did you mean to assume it is symmetric ("sym")?')
         hecon, hesv, hesv_lw = get_lapack_funcs(('hecon', 'hesv',
                                                  'hesv_lwork'), (a1, b1))
         lwork = _compute_lwork(hesv_lw, n, lower)

I was hoping to get to this before it bites anyone but apparently not. Indeed patch 1 should be the behavior but if I can find enough time for finishing #12824 then probably nobody would bother specifying structure. A PR would be much appreciated if you have the time.

sorry I didn't see your answer immediately.
I will send a PR after I've slept, if that's OK with you.

alright, here's the PR.
sorry it took so long, I wanted to make sure that it would be sufficiently high-quality (code formatting, tests, etc…)


[WIP] validate x-envoy-expected-rq-timeout-ms header value

Signed-off-by: Kateryna Nezdolii<EMAIL_ADDRESS>For the case when respect_expected_rq_timeout is enabled and not valid (for example negative) value is set in x-envoy-expected-timeout-ms header, Envoy will use default timeout (15s) as upstream timeout (in case per try timeout is not specified). With this fix Envoy will fall back to timeout value from x-envoy-upstream-rq-timeout-ms header instead.
Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Fixes #8586

@asraa since you had suggestions on the linked issue, mind doing first pass?
@nezdolik thanks for picking this up.  Do you mind adding a unit test as well, just so we can check the exact value?

@alyssawilk thanks for reminding about unit test. I will mark PR as WIP and add a test.

@asraa just realised that falling back is not possible. Whenx-envoy-expected-timeout-ms header is present in request for ingress Envoy, it means that egress Envoy has set it up and sanitised x-envoy-upstream-rq-timeout-ms header. As x-envoy-upstream-rq-timeout-ms is no longer present (from ingress Envoy perspective), it cannot be used for fallback. Do you agree?

Hmm I think I see what you're saying. Envoy sets the x-envoy-expected-timeout-ms from x-envoy-upstream-rq-timeout-ms (which it then removes), so it's not the case that ingress Envoy's request would still have x-envoy-upstream-rq-timeout-ms?

@asraa right, that's what i see in code
cc @snowp

Hmm, gotcha. I think it's fine to resolve this PR with adding a comment in router.cc that if x-envoy-expected-timeout-ms is set, it was validated and sanitized by ingress Envoy.

I think an attempt to fallback still somewhat makes sense since there is no guarantee that the downstream is another Envoy, so we can't know anything about what sanitizing was done. It's probably no super important to handle though, WDYT?

@snowp, my reasoning was that if Envoy sees x-envoy-expected-rq-timeout-ms header, it's highly likely that that header has been set by downstream Envoy. According to doc x-envoy-expected-rq-timeout-ms is set by Envoy for internal requests. There is of course a possibility that nonEnvoy client could have set x-envoy-expected-rq-timeout-ms header, nothing prevents client to do so, but is not a recommended way for client to express timeout expectations.
Please correct me if I am wrong. Should we add extra fallback for case which should not ideally happen (nonEnvoy client setting up x-envoy-expected-rq-timeout-ms) ? Isn't default value (15s) good enough?

Unless there are any further suggestions/discussions, i suggest to close this PR


runtime: add mul intrinsic for overflow checks

Motivation
Profiling over production programs reveals that growslice and makemap are hot functions (even outside the malloc part). I am working to improve their performance and correctness for go1.10.
One part of these functions is at least one check for len > _MaxMem/elemSize which contains a costly division and needs a previous check for elemSize != 0.
We have resorted to switches with constant divisions in growslice to improve performance for common cases. This introduces branch mis predicts and a larger function size which is bad for icaches.
Elsewhere we use maxSliceCap with precomputed values (_MaxMem/elemSize) which introduces cache misses, uses additional memory and only works for small element sizes.
An alternative to the above optimizations that avoids data cache lookups and a division is to check len*elemSize > _MaxMem. Care needs to be taken that  len*elemSize does not overflow.
To be able to check overflow quickly and safely i propose the following:
Proposal
Add a new runtime function runtime.mul:
func mul(a, b uint) (uint, bool) {
	return a * b, b != 0 && a > ^uint(0)/b
}

The function can be used like:
if n, overflow := mul(len, elemSize); overflow || n > _MaxMem {
   panic("Does not fit into Memory")
}

Let the compiler recognize runtime.mul as an intrinsic and replace it with optimized inline instructions where possible.
On architectures that provide overflow/carryflags this can be a mul and setting overflow according to the flag. Otherwise use the provided generic implementation. (Note that a*b is always returned also in the generic implementation to match optimizations in machine code.)
Thereby we can eliminate maxSliceCap and shrink the switch cases without loss of (much) performance but better branch prediction and cache use and shrinking the functions away from special cases again.
possible future directions and proposals

add a similar function for add (the name is already taken in the runtime namespace) or just optimize overflow checks for adds better if this is not already the case
if runtime.mul provides nice code and performance improvements after evaluation of the implementation and use cases we could add such a function to math (and just let the compiler do the existing optimization to that name). Note that math.Bits are already intrinsics.
add detection of generic overflow checking in the ssa backend such that any overflow checking code is optimized

If the introduction of the above function and optimization looks ok i would like to work on implementing it and then replace existing runtime codepaths that would benefit from the new function.
/cc @khr @josharian @ianlancetaylor @stemar94

Related: #6815

Independent of the merits of this proposal, why does this need to be in the runtime? Why not math/bits (or analogous)?

Also related: #18616 (search for carry)

@griesemer: As far as i know currently runtime can only import runtime, unsafe and runtime/internal/... not math. Since everything depends on runtime we would create an import cycle if we use math.mul in runtime. (https://github.com/golang/go/blob/master/src/cmd/go/internal/load/pkg.go)
Not using math in runtime e.g. avoids the issue of introducing a math import which could have inits now or in the future that do not play well with runtime not having fully initialized.
Being runtime only also avoids introducing a new public api for now and runtime.mul can be easily deleted again or changed since it is private to the runtime.
If we find it works well with runtime we can use the then existing compiler support to discuss a public api in another proposal.

Why not put the _Maxmem check in there as well?

@robpike i think keeping mul simple leaves it more generally applicable to other cases of overflow checking at the same time as being used for _Maxmem checks.
Not all checks in runtime are against _Maxmem but can also be _Maxmem-someAmount. Where someAmount is know to be smaller than _Maxmem. Some uses round up len*elemSize before checking against _Maxmem. These cases could be easier to cover by the mul even when not directly comparing against _Maxmem.
if n, overflow := mul(len, elemSize); overflow || n > _MaxMem-someAmount {
   panic("Does not fit into Memory")
}

Seems unobjectionable to add such a runtime function, although it should probably go in runtime/internal/sys, with the other runtime intrinsics.
If we decide to add something like it to math/bits as well, we might decide to give the runtime flavor looser semantics for performance (like we have done with other runtime intrinsics that also appear in math/bits). FWIW, I recently wanted math/bits to have something like Mul128(x, y int64) (lo, hi int64) for use with math/rand.
Longer term, though, it seems perhaps better to fix this either via:

adding [u]int128 to the language (happy to write more about this if you want)
adding arbitrary precision integers to the language (#19623)

Note that with either of these, we should be able to get equivalent performance, with enough compiler help. (1) In the arbitrary precision case, the compiler could in theory recognize that the integer is in use only in one expression, which bounds its range, and replace it under the hood with a 128 bit version. (2) Working with a 128 bit multiply, the compiler can recognize that the top half of a multiply is being checked against 0, so it can substitute a pure overflow check there.

Sounds ok to me.  I think our experience with using CTZ and friends in the runtime first before providing it generally in math/bits worked well.  It would be nice to replicate that experience.


Update the descriptions of "optical", "opticalData", "calorimetric", "calorimetricData", ...

... and so forth to differ clearly between metadata and data.
This issue is based on the second remark of https://github.com/ise621/building-envelope-data/issues/42#issuecomment-700067196 .

@simon-wacker Can you please update the descriptions of "optical", "opticalData" (and "calorimetric", "calorimetricData", ...) so that software developers understand the difference between both?


[Feature Request] Set table sort order run time

Existing Component
Yes
Component Name
el-table
Description
Imagine a use case of e - table component usage with server side pagination and sorting. I use vuex store to remember table layout before leaving the route with an intention to see the same layout next time I visit the view.I wonder how can I set the column sorting order programatically so visually I can see the active column with its sort order on.

Duplicate of #10446


[Feature] Add lidar frame to wf_simulator

Status
DEVELOPMENT
Description
I added /sim_lidar tf frame to wf_simulator node, mainly for using lidar_fake_perception #1431 same time.
Todos

[x] Tests

Steps to Test or Reproduce

Launch wf_simulator and set initial pose
You can see /sim_base_link and /sim_lidar frames on /map coordinate

@yn-mrse :+1:


Fix License type of dnsinfo

Motivation:
dnsinfo uses Apple Public Source License 2.0 not Apache License 2.0.
Modification:
Changed Apache License 2.0 to Apple Public Source License 2.0
Result:
Fixes #10772

@netty-bot test this please


Question on S3 support of LAPKC71

Hi, just came across this amazing project, thanks for sharing!
One quick question, did you notice that the NUC kit does not resume properly from deep sleep (S3)? My LAPKC71E never resumes properly from S3, and some googling pointed to bunch of guys patching their DSDT tables to fix this.
Thanks again for sharing!

Yes and no, long long ago, when I was using Windows OS, someone told me S3 was killed to support S0/S2idle, but I remember I can S3 sleep with bios37 or so(judging by power meter, power less than 1w, S0 on windows far more than that). Later, I switched to Linux completely, once I dived into sleep, I tried to switch to deep, then I can't back anymore.
$ cat /sys/power/mem_sleep
[s2idle] deep

But I don't care about this, S2idle is ok for me on Linux. So I quit at that time.
Back to now, so, have you ever tested on Windows?
I have a glance, I think issues with wake or sleep method in DSDT
...
    Method (_PTS, 1, NotSerialized)  // _PTS: Prepare To Sleep
    {
        If (Arg0)
        {
            \_SB.TPM.TPTS (Arg0)
            \_SB.PC00.NPTS (Arg0)
            RPTS (Arg0)
            \_SB.PC00.LPCB.SPTS (Arg0)
        }
    }

    Method (_WAK, 1, NotSerialized)  // _WAK: Wake
    {
        \_SB.PC00.LPCB.SWAK (Arg0)
        RWAK (Arg0)
        \_SB.PC00.NWAK (Arg0)
        If ((Arg0 == 0x04))
        {
            \_SB.PC00.LPCB.EC0.XSEC = \_SB.PC00.LPCB.EC0.OSEC
        }

        Return (AM00) /* \AM00 */
    }
...

_PTS  does not handle correctly, so that we can't resume
It is just _WAK's fault.

If you can offer me one copy of DSDT from a TongFang machine(Mechrevo, XMG/Schenker, etc) which is shipped with Intel gen11 or so and whose S3 works well, then I may fix it.

Thanks for the prompt reply!
The annoying part is, although the function is broken, the BIOS still advertises support for S3:
(base) hl@hl-lapkc71e:~$ sudo dmesg|grep S3 [sudo] password for hl:  [    0.237345] ACPI: PM: (supports S0 S3 S4 S5)
So nothing stops me from setting mem_sleep to deep and having the laptop freeze upon resume. Default behaviour out of the box for ubuntu2404. Annoying.
For testing with Windows - I can try using Win10 LTSC2015/2019 which should be well before M$ enforcing modern standby, or later versions - which would you recommend?
For grabbing the DSDT from a working laptop - I'll try my best to source one of those but no promises here....
Thanks again for your time!

Also if you could find the old BIOS you mentioned, could you please send me the file or mention a source that still hosts such files? The official site only hosts the latest 2 versions.

The annoying part is, although the function is broken, the BIOS still advertises support for S3:
(base) hl@hl-lapkc71e:~$ sudo dmesg|grep S3
[sudo] password for hl:
[    0.237345] ACPI: PM: (supports S0 S3 S4 S5)

The driver is written against ACPI, if specific sections presented, then they would be reported as supported.
    Name (_S0, Package (0x04)  // _S0_: S0 System State
    {
        Zero,
        Zero,
        Zero,
        Zero
    })
    Name (_S3, Package (0x04)  // _S3_: S3 System State
    {
        0x05,
        Zero,
        Zero,
        Zero
    })
    Name (_S4, Package (0x04)  // _S4_: S4 System State
    {
        0x06,
        Zero,
        Zero,
        Zero
    })
    Name (_S5, Package (0x04)  // _S5_: S5 System State
    {
        0x07,
        Zero,
        Zero,
        Zero
    })

For testing with Windows - I can try using Win10 LTSC2015/2019 which should be well before M$ enforcing modern standby, or later versions - which would you recommend?

No, I don't use it on bare metal anymore. I have no idea.


Add completed goals to the feed

To go along with the completion of goals functionality #12 add the recent completion events to the community feed.

Also added badges and profile images to feed.  Need to test  speed  review pagination.

all set.


[Bug]: position of x and instagram.

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
position of x and instagram are not looking good.
Expected behavior
interchange the position
Add ScreenShots

On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

please assign this to me.

@dnyandeepchute  where will you keep the position send the screenshot

@hardiksharma02 already mentioned in Expected behavior that i will interchange the position as it looks good.

@hardiksharma02 instagram is not present there i will add it and fix the position, please increase it to level2, thank you

Hi @dnyandeepchute, we've made some changes to our code, so please make sure to incorporate the latest updates from the main branch into your own. We've decided to remove the Instagram functionality as it's not essential for our project. This change isn't related to any specific issue, so feel free to close it and focus on other tasks. Apologies for any inconvenience, and thank you for your time.


Add lack of cooperation by Nvidia

One of the two largest discrete GPU vendors does not support GBM and EGLStreams has poor or no support on most compositors. Given that Nvidia users form the majority of the gaming and machine learning userbase, this should get a mention.

That's fair enough, I will add a section about missing stuff then. Hopefully Nvidia support will be the only thing.

Done


Investigate means of resetting EPS without restarting car (Serial data comm enable pin)

This would be great.  A hard reset should never be required.


Inconsistent behaviour by kpt if there are 2 repository CR pointing to same git repo but different directory inside the repoo

Issue:
kpt alpha rpkg init is working incorrectly when I apply 2 Repository CR in the system which are pointing to the same private repository.
Details of my 2 CRs
$ cat test_ct_repo.yaml
apiVersion: config.porch.kpt.dev/v1alpha1
kind: Repository
metadata:
  name: test-catalogue
  namespace: test
spec:
  description: test-catalogue
  content: Package
  deployment: false
  type: git
  git:
    repo: https://github.com/********/**********************.git
    directory: /
    branch: main
    createBranch: true
    secretRef:
      name: cred

$ cat test_ct_repo_out.yaml
apiVersion: config.porch.kpt.dev/v1alpha1
kind: Repository
metadata:
  name: test-catalogue-out
  namespace: test-out
spec:
  description: test-catalogue-out
  content: Package
  deployment: false
  type: git
  git:
    repo: https://github.com/********/**********************.git
    directory: test-pkg-out
    branch: main
    createBranch: true
    secretRef:
      name: cred

Notes:

In both of the above CR, repo path is same. Since, its a private repo, I have replaced the username and repo name with *.
Namespace and CR name is different.
directory is different.
secret was created in both the namespaces.
test-catalogue was applied first. And then test-catalogue-out

$ kubectl get repositories -A
NAMESPACE   NAME                 TYPE   CONTENT   DEPLOYMENT   READY   ADDRESS
test-out    test-catalogue-out   git    Package   false        True    https://github.com/******/*********************.git
test        test-catalogue       git    Package   false        True    https://github.com/******/*********************.git

Below are my observations after Creating the above CRs.

Incorrect revision creation.

$ kpt alpha rpkg init test-pkg11 --workspace=v1 --repository=test-catalogue --namespace=test
NAMESPACE   NAME                                                           PACKAGE     WORKSPACENAME   REVISION   LATEST   LIFECYCLE   REPOSITORY
test-out    test-catalogue-out-233a4ca4388204556f5bceff99a9799d1a86ed1a   test-pkg11    v1              v1         false     Draft   test-catalogue-out

As you can notice in the above command, the namespace and repository passed are test and test-catalogue respectively. Yet, the revision which was created had the details of the repository and namespace of the test-catalogue-out Repository CR.
After facing this issue, I deleted and re-created the porch. And recreated the 2 repo CR as above, this time I was getting a different issue, as below.

Not able to init the package.

$ kpt alpha rpkg init test-pkg-repo1 --workspace=v1 --repository=test-catalogue --namespace=test
Error: Internal error occurred: create not allowed while custom resource definition is terminating

@Cbkhare Thanks for reporting this and the detailed description. I am able to reproduce weird behavior here, but I haven't been able to fully understand what is going on yet.
I see that one of the repositories point to the root of the git repo, while the other point to a subdirectory. This probably isn't a good way to do this, since Porch assumes each package is a direct subdirectory of the directory provided for a repository. So for the test-catalogue repository, you should probably set the directory property to something like test-pkg. However, I'm still seeing odd behavior when I do this, so I will look into it.

So dug deeper into this. This is actually a rather significant bug in Porch, where some parts of the code doesn't account for the fact that it might be multiple Repository objects pointing to the same repository, but with different settings (which probably is a pretty common case).
The cache that keeps track of repositories and sets up the sync between git/oci and Porch, uses the repository URL as the key for the inventory of repositories:
https://github.com/GoogleContainerTools/kpt/blob/239a679abe979e058e53fbdcb60a456060cda81c/porch/pkg/cache/cache.go#L120-L125
This is reasonable, as we don't want to poll the same repository multiple times in parallel. But the internal representation of a repository, which has a one-to-one mapping to a git Repository due to the repo url being the key, also includes information specific to the KRM Repository CR:
https://github.com/GoogleContainerTools/kpt/blob/239a679abe979e058e53fbdcb60a456060cda81c/porch/pkg/cache/cache.go#L133-L141
This is something we need to fix.


How to make the tooltip stay in the same position horizontally across multiple elements in a container?

Hi flowbite-vue development team, and community 👋
Recently, I tried to achieve a particular piece of functionality while using your fwb-tooltip component, which I will first describe for context:

when rendering the fwb-tooltip component with a few span elements, for example, the tooltip seems to jump horizontally.

My goal is to make sure that where I am trying to use the tooltip, and when I have a container with a collection of elements inside, and then hover over any of them, the position of the tooltip will be the same horizontally for all of them.  To make it a bit more clear, I want it to hug the right hand side of the parent element (or in the example screenshot, hug the right hand side of the table/column).

For extra context, I am trying to use the right placement in this scenario.
I found a couple of places that could potentially be affecting this in the component library, but I have a particular intuition that I need to make changes in the floating-vue.js file.
My question is whether it is possible to make the position of the tooltip the same horizontally across different elements/hug the tooltip to the right hand side of the parent element/table/column, regardless of the length of a string, for example, without having to make changes to the component library, or if there is a way of specifying the position of the tooltip in such a scenario with a different approach? Has anyone faced this issue/ran into such a scenario before while using fwb-tooltip? If there are changes needed in the original component library, how, where, and which changes should I be making?
I hope to hear back from the community!

Hey, can you please create a simple example to show what you mean... I would like to see the code and it's behavior...  so anything like code + gif or some online sandbox would be good.

Hey, can you please create a simple example to show what you mean... I would like to see the code and it's behavior... so anything like code + gif or some online sandbox would be good.

Hi, thank you for your reply! I will be posting a finished code sandbox link here next week, or so, and then I will await your feedback on it 🙂


cli: failed to stop workflow

When trying to stop a running workflow:
$ # roofit example
$ git diff                                                                                [5:00:56]
diff --git a/reana.yaml b/reana.yaml
index e1355ca..51be33c 100644
--- a/reana.yaml
+++ b/reana.yaml
@@ -19,7 +19,7 @@ workflow:
       - name: fitdata
         environment: 'reanahub/reana-env-root6:6.18.04'
         commands:
-        - root -b -q 'code/fitdata.C("${data}","${plot}")'
+        - sleep 1000
 outputs:
   files:
     - results/plot.png
$ reana-client run
[INFO] Creating a workflow...
workflow.19
[INFO] Uploading files...
File code/gendata.C was successfully uploaded.
File code/fitdata.C was successfully uploaded.
[INFO] Starting workflow...
workflow.19 is running

Then first workflow job finished (14e3e86f-31ac-4990-9405-1f888d 5adb98) and the second one (70c6e3eb-6142-43fb-8169-aa90c060023c-4r2dn ) starts to run:
$ kubectl get pods                                                                                [5:01:49]
NAME                                                            READY   STATUS    RESTARTS   AGE
70c6e3eb-6142-43fb-8169-aa90c060023c-4r2dn                      1/1     Running   0          9s
reana-batch-serial-dcac5ed9-1d93-4a91-8631-ce518d4f8be2-l2gcg   2/2     Running   0          34s
reana-cache-88b76b854-85ptz                                     1/1     Running   0          36m
reana-db-5cff5946c7-xwmtk                                       1/1     Running   0          36m
reana-message-broker-65bbb5956f-7mgss                           1/1     Running   0          36m
reana-server-6487dc6d74-mkrcc                                   2/2     Running   0          32m
reana-traefik-86556f5678-h9nqk                                  1/1     Running   0          36m
reana-ui-597887756f-hhhsd                                       1/1     Running   0          36m
reana-wdb-574d66ff44-6kdhk                                      1/1     Running   0          36m
reana-workflow-controller-64d7cd56b4-4cq9q                      2/2     Running   0          34m

User tries to stop the workflow execution:
$ reana-client stop -w workflow.19 --force                                                                                [5:01:45]
Workflow could not be stopped:
(404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'Cache-Control': 'no-cache, private', 'Content-Type': 'application/json', 'Date': 'Fri, 07 Feb 2020 16:01:49 GMT', 'Content-Length': '262'})
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"jobs.batch \"14e3e86f-31ac-4990-9405-1f888d5adb98\" not found","reason":"NotFound","details":{"name":"14e3e86f-31ac-4990-9405-1f888d
5adb98","group":"batch","kind":"jobs"},"code":404}

We should:

[ ] Not try to delete a job that was already finished, or at least not fail and continue with the rest to finilise the stop
[ ] Give better error messages for stop

This is also happening when jobs are stuck in ImagePullBackoff , see conversation and complete issue about ImagePullBackoff  problem.


make: add goal to install required tools

Just a little convenience feature to install the required tools via makefile.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "makefile/add-get-tools-goal" git@github.com:jalateras/infrakit.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Thanks for the patch!  I'll merge this shortly.

Current coverage is 67.61% (diff: 100%)

Merging #322 into master will decrease coverage by 0.06%

@@             master       #322   diff @@
==========================================
  Files            30         30
  Lines          1550       1550
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           1049       1048     -1
- Misses          398        399     +1
  Partials        103        103

Powered by Codecov. Last update f57fa10...3510096

@GordonTheTurtle even easier
git commit --amend
git push -f origin


Support unique id fields

PageProperty doesn't support a new unique_id field property type.
API doc: https://developers.notion.com/reference/page-property-values#unique-id

@seratch hi, do you need help with this?
I'm not good at Kotlin, but I can try to make a PR, it looks not so hard!

Pull requests for "good first issue" ones are always welcomed!

I will take it then!


[upgrade-ert] does not check for the version of ERT already in use first

The pipeline should first check which ERT version is in use with PCF prior to downloading the latest from PivNet and doing any work. If the in-use ERT version is equal to latest on PivNet, do nothing.

I'm going to close this and leave the story unstarted in Tracker.


Use autoconfigure module

Alternative to #36220
cc @lmolkova
EDIT this approach doesn't allow user to add "span exporter customizer" for az mon span exporter. going to try one more approach...

/azp run java - core - ci

@trask it looks great! I remember we used to have autoconfigure module in core samples, but I removed it because it complained about missing otlp dependency,  let me see if it still complains on your core changes commit.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-monitor-opentelemetry-exporter

/azp run java - spring-monitor - tests

azure-monitor-opentelemetry-exporter code version (not released) is 1.0.0-beta.12.
spring-cloud-azure-starter-monitor uses the 1.0.0-beta.11 version of the OTel exporter, published on Maven Central.
It would be interesting to run the tests of spring-cloud-azure-starter-monitor-test with the 1.0.0-beta.12 version of the OTel exporter.

azure-monitor-opentelemetry-exporter code version (not released) is 1.0.0-beta.12.
spring-cloud-azure-starter-monitor uses the 1.0.0-beta.11 version of the OTel exporter, published on Maven Central.
It would be interesting to run the tests of spring-cloud-azure-starter-monitor-test with the 1.0.0-beta.12 version of the OTel exporter.

ya, CI did this for us, which is how I caught the problem with filtering the _OTELRESOURCE_ metric (which I need to add comment to explain still)

@trask I would like to have more info about this

@trask I would like to have more info about this

sorry, I didn't have a chance to test this out further, wanted to merge and unblock @heyams


Setting of PIDs and Rates on KISS FC does not work with KISS 24RE ESC telemetry connected

The KISS FC will detect lipo voltage if it receives telemetry data from the KISS 24RE ESCs and prevent the setting of the PIDs and/or Rates from the OSD via serial (see SendFCSettings() in KISS_OSD.ino). Would be great if there would be a way to send the PIDs and Rates to the FC - which would then store them in the EEPROM instead of using them directly (safety issue). The user would have to reconnect the lipo to use the new values.

i started on config rewrite today. ;)))

Awesome Alex! Let me know how I can be of any help.

Check your rcg messages :)

Again: THANKS A TON!
 No motor even moved. Even though I had some bugs in SendFCSettings() and it partially send crap data - all fixed now :)
I have two requests:

Can you please make the ESCs beep if there is no buzzer configured (after saving)? Some people do not use a buzzer or it is not connected to the FC.

In the long run, it would be great to use a separate code to just save the PIDs and Rates. Using the settings could create havoc in the future: What if you make a major change in the byte stream for the settings, but the OSD is not updated? I do have everything updated until 104, and I will keep monitoring what you and Felix are doing inside protocol.js. But you cannot expect the end user to keep the OSD up to date. Let me know what your thoughts are on that, or if you maybe have a better idea.

I need to do more testing, but I have a suspicion that retrieving the settings through 0x30 does not work any more after telemetry has been requested through 0x20. Did you run into a similar issue with your GUI?

Will do more testing in the field on both of my quads. If there is no major accident in the next two weeks of testing - I think we should talk about releasing it to the community :)
Thanks,
The other Alex :)

I switched frames on my Acro quad recently and gave KISSFC-1.1RC4-unsafe a try. Tweaking the PIDs has never been that much fun before - just land, change them in the menu - beep - take off again. No more fiddling around with the cable etc.
While it was a lot of fun while it lasted, at some point the OSD stopped working. Or better say: The OSD was still running, however the KISS FC just decided to stop sending the settings at some point. My OSD FW won't display anything if it does not receive any settings from the KISS FC. My other quad (with same OSD and KISS FC FW) did the same from the get go. So I flashed back to T107xx and voila - OSD works again (no change in OSD FW).
Very strange.

Any ways to reproduce it?

I will try to reproduce it tomorrow.
But what I can already say: On one of my quads I just updated the KISS FC FW to KISSFC-1.1RC4-unsafe, loaded my backup through your GUI and the OSD would not work (because it would not receive any settings).

well.. format changed probably?

On 18 Nov 2016, at 03:32, awolf78<EMAIL_ADDRESS>wrote:
I will try to reproduce it tomorrow.
But what I can already say: On one of my quads I just updated the KISS FC FW to KISSFC-1.1RC4-unsafe, loaded my backup through your GUI and the OSD would not work (because it would not receive any settings).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/awolf78/KISS_OSD/issues/3#issuecomment-261432790, or mute the thread https://github.com/notifications/unsubscribe-auth/ACARCqDbeCjjJqEf3rk-gEZtVNyVWe2Nks5q_Q5BgaJpZM4KZCEq.

From current protocol.js:
`if (obj.ver > 104) {
data.setUint8(128, obj.NotchFilterEnable);
data.setUint16(129, obj.NotchFilterCenter,0);
data.setUint16(131, obj.NotchFilterCut,0);
data.setUint8(133, obj.YawCfilter);
            blen=142;
        }`

Why is blen=142 while you only set 133 bytes in the buffer?

Ok, I think I found the issue - shame on me, had a bug in my protocol code :(
I have been saving away many times on both of my quads and have not found any issue so far. Thanks for providing the FW to test and debug my code!
Before I close this:
How are we going to convince Flyduino that saving with a Lipo attached does not cause any issues?

Its noted ;)


version 1.0.7 - EWS call gets 401 Unauthorized response

Describe the bug
Starting with ad2image version 1.0.7 HTTP Requests to EWS API for higher resolutions fail due to "Unauthorized" / 401 response of EWS.
Using same EWS system & credentials and ad2image version 1.0.6 all works as expected.
To Reproduce

rollout 1.0.7
GET avatar?uid=&size=504

an update of Apache HTTP Client (5.2 -> 5.3) seems to have removed NTLM auth but only marked the classes as "deprecated": https://stackoverflow.com/questions/77784888/apache-http-client-5-3-ntcredentials-no-longer-working
quick fix could be a downgrade of the used Apache HTTP Client version

Also we should add an integration test (with wiremock) that checks that outgoing requests to EWS api contain a proper authorization header for NTLM like
Authorization: NTLM .......


Add link to TSG Wiki on the github repo

Add a link to https://wiki.thescriptersguild.com/main on the sidebar of the github repo

done

closed


The library can support python pickle format ?

The library can support python pickle format ? support encode to python pickled format or decode from python pickled format ?
Thanks

It would need to be added, perhaps.  Can you show us what python pickle format looks like or link to a doc?


2 Speed pump

My "pump 1" has 2 speed options.
When I click on the pump in the official intouch.2 app, it turns on the pump in slow mode.
When I click on the pump again, it changes it to fast mode.
When I click the pump a 3rd time, it turns off.
It seems that the "pump 1" home assistant switch operates in a similar way. It's a little bit confusing though. When the switch is already on (pump running slow) and I go to toggle the switch off, it turns itself back on (pump running fast). Now that I understand this operation, I'm ok with this.
However, I'm wondering if we could expose a binary sensor that shows whether the pump is in slow/fast mode please? Or perhaps it would be best to have a 3 state switch of sorts?

Version v0.0.7 ought to handle this now


ModuleNotFoundError: No module named 'unsloth'

Hello,
First of all, my gpu is rtx3090. I've installed like this
pip install "unsloth[cu118_ampere] @ git+https://github.com/unslothai/unsloth.git"
But the error
ModuleNotFoundError: No module named 'unsloth'
Can anyone tell me the reason?

@BBaekdabang Did the install fully succeed?

when trying to install, this error occured

@BBaekdabang Apologies just fixed it a few hours back - it seems like I accidentally made a typo in a late night PR request!!
You might have to reinstall via -no-cache-dir to get the latest one

thanks for your reply. But this error occurred,,

@BBaekdabang I think upgrading pip will work! pip install --upgrade pip

this time, this error

@BBaekdabang Oh that seems like some sort of python installation errors - hm thats a bummer. How about if you try installing say pandas pip install pandas does that work?

It doesn't work... I think that python is something wrong

@BBaekdabang Oh my if Pandas does not install, there is a big issue - you might have to reinstall all of pip and python :(

@BBaekdabang You can also try installing Miniconda / Anaconda separarely or use a new pip venv. Anyways I'll close this issue for now - reopen if you need anymore help!


No update since end 2022?

How can I update pikvm on my pcie Version?
The image is from November 2022 and when trying to follow the steps on the pikvm website with the update script, the system fails to come up again.
I am very disappointed in this device and having to rely on 3rd party modifications to run pikvm.
Whenever a device has network access, there should be the possibility to run up to date software on it.

Got to work the official v3-hdmi-rpi4-latest.img.xz PiKVM version.
This should be mentioned here I think: https://wiki.blicube.com/blikvm/en/modify_pikvm_image/
Otherwise people also might use the outdated one from the Google share, which also to be honest can feel fishy.


PluralSight : ERROR: unable to download video data: [WinError 10054] An existing connection was forcibly closed by the remote host

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.07.16. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.07.16

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', '--username', 'PRIVATE', '--password', 'PRIVATE', 'https://app.pluralsight.com/library/courses/low-polygon-texturing-mari-786', '--playlist-start', '16']
[debug] Encodings: locale cp1252, fs mbcs, out cp1252, pref cp1252
[debug] youtube-dl version 2016.07.16
[debug] Python version 3.4.4 - Windows-10-10.0.10586
[debug] exe versions: ffmpeg N-80256-g0a9e781, ffprobe N-80256-g0a9e781
[debug] Proxy map: {}
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', '--username', 'PRIVATE', '--password', 'PRIVATE', 'https://app.pluralsight.com/library/courses/texturing-low-polygon-sci-fi-weapon-mari-1070']
[debug] Encodings: locale cp1252, fs mbcs, out cp1252, pref cp1252
[debug] youtube-dl version 2016.07.16
[debug] Python version 3.4.4 - Windows-10-10.0.10586
[debug] exe versions: ffmpeg N-80256-g0a9e781, ffprobe N-80256-g0a9e781
[debug] Proxy map: {}
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', '--username', 'PRIVATE', '--password', 'PRIVATE', 'https://app.pluralsight.com/library/courses/creature-texturing-mari-1458']
[debug] Encodings: locale cp1252, fs mbcs, out cp1252, pref cp1252
[debug] youtube-dl version 2016.07.16
[debug] Python version 3.4.4 - Windows-10-10.0.10586
[debug] exe versions: ffmpeg N-80256-g0a9e781, ffprobe N-80256-g0a9e781
[debug] Proxy map: {}
ERROR: unable to download video data: [WinError 10054] An existing connection was forcibly closed by the remote host
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\YoutubeDL.py", line 1648, in process_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\YoutubeDL.py", line 1590, in dl
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\downloader\common.py", line 350, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\downloader\http.py", line 162, in real_download
  File "C:\Python\Python34\lib\http\client.py", line 537, in read
  File "C:\Python\Python34\lib\http\client.py", line 576, in readinto
  File "C:\Python\Python34\lib\socket.py", line 378, in readinto
ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host

[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', '--username', 'PRIVATE', '--password', 'PRIVATE', 'https://app.pluralsight.com/library/courses/texturing-prehistoric-reptiles-mari-812']
[debug] Encodings: locale cp1252, fs mbcs, out cp1252, pref cp1252
[debug] youtube-dl version 2016.07.16
[debug] Python version 3.4.4 - Windows-10-10.0.10586
[debug] exe versions: ffmpeg N-80256-g0a9e781, ffprobe N-80256-g0a9e781
[debug] Proxy map: {}
ERROR: texturing-prehistoric-reptiles-mari-812: Failed to parse JSON  (caused by ValueError('Expecting value: line 1 column 1 (char 0)',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\extractor\common.py", line 545, in _parse_json
  File "C:\Python\Python34\lib\json\__init__.py", line 318, in loads
  File "C:\Python\Python34\lib\json\decoder.py", line 343, in decode
  File "C:\Python\Python34\lib\json\decoder.py", line 361, in raw_decode
ValueError: Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\extractor\common.py", line 545, in _parse_json
  File "C:\Python\Python34\lib\json\__init__.py", line 318, in loads
  File "C:\Python\Python34\lib\json\decoder.py", line 343, in decode
  File "C:\Python\Python34\lib\json\decoder.py", line 361, in raw_decode
ValueError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\YoutubeDL.py", line 676, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\extractor\common.py", line 343, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\extractor\pluralsight.py", line 272, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\extractor\common.py", line 539, in _download_json
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpdy42tioh\build\youtube_dl\extractor\common.py", line 549, in _parse_json
youtube_dl.utils.ExtractorError: texturing-prehistoric-reptiles-mari-812: Failed to parse JSON  (caused by ValueError('Expecting value: line 1 column 1 (char 0)',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue required an account credentials please provide them or explain how one can obtain them.

Duplicate of #10091.


It doesn't seem to work with IE

Can you provide example user agent string?

My bad, this was a terrible case and it's actually working fine


TST: stats.ortho_group: improve determinant distribution test

Reference issue
Closes gh-18272
What does this implement/fix?
gh-18272 reported an occasional failure of test_multivariate.TestOrthoGroup.test_det_and_ortho. This improves the test and makes it deterministic.

Original CI failure was due to existing blank line whitespace. Fixed in second commit.

Once #18225 in, you can actually use the SciPy linalg.det. Not a blocker but would be nice to have some real use case for stacked arrays :)

Sure. I'll mark this as draft until then.

Would you be willing to review it @ilayn?

Sure will do. No worries.

The updated test is fine, and is an improvement over the existing test.
The following comments are probably just overanalysis...
Testing the distribution of the determinant is the same problem as testing whether a random bit generated by a PRNG is truly random, i.e. it is the same as determining whether a coin flip is fair.  This is a fundamental problem for random number generators, and I don't think we want to bring in the heavy duty tests normally associated with this problem into the test suite. We can't really expect to have a very strong test here, because we don't want to spend a lot of CI time on it. (More rigorous validation of the rvs generated by the SciPy distributions is something that could be considered as a separate task that would be run outside of the normal CI.)  Instead it makes sense to use a simple test that is not too computationally expensive but that should catch egregious mistakes.  In this PR, that means a seed is set, and a single collection of 5000 variates is checked to ensure that the proportion of +1 determinants is, in some sense, reasonable.
That check is to apply the binomial test and verify that 0.5 is in the confidence interval of the proportion with confidence level 0.95.  Note that, under the null hypothesis that +1 and -1 are equally likely, the 0.95 confidence interval for the proportion would fail to contain 0.5 5% of the time.  This means that the seed is important: if the seed is changed, there is a 1 in 20 chance that the test will fail.  It also means that checking the confidence interval is really no better than checking that the p-value of the binomial test is greater than 0.05. It isn't wrong to use the confidence interval, but it doesn't really provide any benefit in this test.
Also note that if the seed was not set, this test would fail far too often to include in the test suite.  In fact, if there is any change in the values returned by ortho_group.rvs() for the given seed (which could happen because of variations in floating point operations on different platforms with different compilers, or because of a change in the implementation in ortho_group.rvs(), or because of a change in the underlying NumPy PRNG), there is a 5% chance that the test will fail.  Perhaps the probability of such a change, multiplied by 0.05, results in the probability of such a failure being small enough that we can ignore it for now, and adjust the test if it ever occurs.

Oh interesting. by the way, is there any relationship between the determinant distribution and the ortho_group randomization? Determinant is highly dependent on the LU factorization and I don't think you can make that connection while pivoting precision is involved with random pivots.

@ilayn since np.linalg.det has been replaced, I think this is ready.

No deep issues. Warren's other comment is about whether it is ok for there to be a 5% chance of false positive failures when there is a change the random stream. It's similar to a question of test tolerances - pick a tight tolerance that's more sensitive to real issues, or a loose tolerance that's more robust against false positive failures. This is set to a 5% false positive rate. We can preemptively adjust that (e.g. to 1%) or make an adjustment should the need arise.

I'll take a look at this today.

@steppi, thanks. Actually, I think it is fine as it is, so I'll go ahead and merge.

Thanks @WarrenWeckesser


make some kubelet args configurable

Fixes: #1788
As long as user specified args does not conflict with automatically specified.

This change is

@redhat-k8s-bot test please

I signed it!

Running locally via Vagrant on Fedora box. Tested both use cases when the kubelet_args is empty and set to --host-network-sources=*. Cluster is deployed, nodes are registered.

Hi, @ingvagabund. I also have an proposed change for it.
https://github.com/cheyang/contrib/commit/662e124d7c7fe71d32544ab7171bb8b0b7f7ca8f
Then we can define kubelet_options as below.
kubelet_options:
  - "--pod-infra-container-image={{kube_regitry}}/google_containers/pause:2.0"
  - "--max-pods=3"

There are two benefits:

It can handle long kubelet options elegant
It can avoid the space in the end of kubelet command if kubelet_args is empty.

If you feel we can leverage my change, I can send merge request. If not, it's also fine.

@cheyang it crossed my mind as well. Just did not have much time to extend the idea. I like yours, it is elegant. Be my guest :) I will review your PR.

Thank you! @ingvagabund , I've submitted the changes, please review it.

Closing in favor of https://github.com/kubernetes/contrib/pull/1859