[Bug Report] datepicker onchange event cannot emit when use :value to bind value instead use v-model

Element UI version
2.13.0
OS/Browsers version
MacOS10.15 / Chrome 78.0.3904.108
Vue version
2.6.10
Reproduction Link
https://codepen.io/zhongqc/pen/BayBBKO
Steps to reproduce
open browser console panel
change datepicker 1 value & watch console print
change datepicker 2 value & watch console print
What is Expected?
can emit change method when change datepicker value
can use :value & @change method to listen datepicker value change
What is actually happening?
when use :value to bind value cannot emit change method

use input method to resolve this issue, close this issue
but could update the document let user know can use the input method


Blend mode

Guys, what about it?

Guys, what about it?

Not sure what your asking?

Not sure what your asking?

I would like to know what the fate of this Issue is. Will this feature be done?)

I would like to know what the fate of this Issue is. Will this feature be done?)


Code of Conduct

I'm assuming there's no reason we'd want a different CoC for this repo, and that Contributor Covenant would be the way to go.
We could either add the file in-repo similar to https://github.com/badges/shields/pull/5236 in the main repo, though I suppose we could also consider leveraging default community health files for the badges org to apply across all repos?

Ooh, I didn't know about default community health files! That's cool! We could start with funding.yml which is already in place. We're still finalizing the contacts for badges/shields#5236 but we could add that in soon after.

I'm going to go ahead and add our same CoC files from shields to this repo. I'd rather get it in sooner rather than later, especially with the cutover, and we'll always have the ability to remove the in-file CoC file if/when we setup a community health repo and files.

Straying a bit off topic here, but..
Your .github repo can now also include a README.md for the organisation too https://github.blog/changelog/2021-09-14-readmes-for-organization-profiles/ which is cool.
One thing we do need to be slightly mindful of if we start using a .github repo is impact on other projects under the badges org (really just poser tbh - I think it is the only active non-shields project). For example, if we put a funding.yml in a .github repo, I think that then automatically makes the funding CTA for all repos that are already created under the badges org without its own funding.yml "give money to shields.io". In the case of funding.yml specifically, poser actually have their own funding CTA already https://github.com/badges/poser/blob/master/.github/FUNDING.yml which would override the default but we need to consider the fact that @badges !== shields.io sometimes.

Agree with that… might be another reason to spin off projects that aren't part of shields.io.

might be another reason to spin off projects that aren't part of shields.io

I wonder if the path of least resistance would actually be to transfer the active shields.io projects:

https://github.com/badges/shields
https://github.com/badges/squint
https://github.com/badges/daily-tests
https://github.com/badges/shields-ops

to a new org e.g: https://github.com/shields_io and all of the core/ops team can be allowed to create repos under the new org, etc.
The advantage of doing it that way round is we could just do it

Hmm, interesting…

Sorry, forgot I'd used the close keyword. Think it's probably worth revisiting the org-level pieces in a separate/focused thread anyway, maybe a topic for next ops meeting?

Yes for sure :+1:  I did note in the first post I was going a bit off-topic


Add delegated authority. Remove wallet while creating new authority.

Description
Current wallet may be deleted when add delegated authority for the wallet, which already has another delegated authority
Steps to Reproduce

Add delegated authority for the wallet
Waiting update
Add second delegated authority for the wallet

Expected Behavior
Added second delegated authority
Actual Behavior
Current wallet was deleted
Environment
https://github.com/novasamatech/nova-spektr/commit/432167ed6b6979d57bd411da75e199bf4d9e501a
Screenshots
log -
nova-spektr.log

Example
https://github.com/novasamatech/nova-spektr/assets/40560660/dfaddf9b-6287-4485-a123-906a8c886ead

Status verification: ✅


[Question]: “FileNotFoundError: No such file or directory” and can not download file from dataset

Describe your problem
I uploaded a txt file to the knowledge base, but the parsing failed, and a FileNotFoundError occurred.

When I try to download the file from the dataset, I get a file containing error information.

It seems the file is not uploaded successfully.

me too


node version fix

changes in node version

First of all, I would much prefer it is fixed to work with the latest version of node.
Secondly, if for some reason that can't happen then I would prefer the value of the NODE_TAG ARG is updated and not inline FROM directive, because ARG is there to allow building the image with different versions of the base image at build time by supplying --build-arg NODE_TAG=<some-node-version> argument.
Finally, I would not specify the patch version (the third point of the version number) to receive any security fixes in the near future and use something like 11.11 instead as the default value.

Please test these changes and let me know when it is ready to be merged. If the version 11.11 does not work then change it to 11.11.0 or any other version that seems to work for now.

I just tested the changes and 11.11 works fine, thank you!

The quick fix is deployed, you should check if everything is working as expected on the live site.


Cannot upgrade past besu 22.7.2

Something in https://github.com/hyperledger/besu/releases/tag/22.7.3
is breaking this ethsigner AT: https://app.circleci.com/pipelines/github/ConsenSys/ethsigner/2104/workflows/628d606e-0ded-45a0-87c5-0a6c18f3f87b/jobs/5654/tests#failed-test-0

Fixed by #487


Updated the brew install command in README

@AbhishekBiswal This update is no more valid. The current correct command is:
brew install --cask jpadilla-redis
Ref: https://formulae.brew.sh/cask/jpadilla-redis


Hostname validate missing and UI confusion around private key and passcode

Describe the bug

I made the mistake and entered<EMAIL_ADDRESS>as the hostname for a server connection instead of foo.bar.com.  There was no validation on the hostname form and it just would not connect.  Some sort of invalid hostname message would have been more helpful.

If I enter a bad passcode for my ssh secret key a red "Connection error" dialog appears.  It seems like this information gets cached for a while because when I try to click on the server name again, it doesn't prompt me but immediately shows the "Connection error" again.  It shouldn't.  It should prompt me immediately for a new passcode.

When there is a "Connection error", the dialog comes down and it has a red X image on the left.  I keep on trying to click that X to close the dialog because of iOS.  Either remove the X, add a cancel button to the dialog, or let it be clickable and swipe-able to remove it.  I vote for the click option.

Informations

Device : iPhone 11
OS : iOS 15.3.1
WebSSH Pro version : 17.0 (861)

Screenshots
If applicable, add screenshots to help explain your problem.
To Reproduce
Steps to reproduce the behavior:

Go to servers
Press + in the upper right
Enter hostname with @ sign in it.
Try to connect to it.

2nd problem:

Add a private key with a passcode.
Try to connect to the server with that key.
Enter an invalid passcode.  Red dialog should come down.
Try to connect to that server immediately afterwards.  No passcode prompt.

3rd problem:

Add a private key with a passcode.
Try to connect to the server with that key.
Enter an invalid passcode.  Red dialog should come down.
Try to click on the red X to get rid of the dialog.

Hello @j256 👋
Thank you a lot for making this issue about WebSSH' UX!
I'm planning it for the upcoming milestone
Have a nice day ☀


Kube-proxy replacement not working when running without privileges

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I'm running this setup:

Single node Kubernetes cluster based on Talos Linux (Talos Linux 1.2.3 with Kubernetes 1.25.0, installed without kube-proxy and without CNI)

Install Cilium using helm (this needs to be this complex as SYS_MODULE is not available on Talos Linux):
$ cat <<PR > postrend.sh
#!/bin/sh

set -e

$ cat <&0 > base.yaml
kubectl kustomize .
PR

$ cat <<PATCH > caps-patch.yaml
apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: cilium
  namespace: kube-system
spec:
  template:
    spec:
      containers:
        - name: cilium-agent
          securityContext:
            capabilities:
              add:
                - CHOWN
                - KILL
                - NET_ADMIN
                - NET_RAW
                - IPC_LOCK
                - SYS_RESOURCE
                - PERFMON
                - BPF
                - DAC_OVERRIDE
                - FOWNER
                - SETGID
                - SETUID
                - SYS_ADMIN
      initContainers:
        - name: clean-cilium-state
          securityContext:
            capabilities:
              add:
                - NET_ADMIN
                - SYS_RESOURCE
                - PERFMON
                - BPF
PATCH

$ cat <<KUST > kustomization.yaml
resources:
- base.yaml
patchesStrategicMerge:
- caps-patch.yaml
KUST

$ cat <VALUES > values.yaml
k8sServiceHost: <your loadbalancer IP>
k8sServicePort: 6443
kubeProxyReplacement: strict
operator:
  replicas: 1
securityContext:
  extraCapabilities:
  - PERFMON
  - BPF
  privileged: false
VALUES

$ helm upgrade --install -n kube-system --version 1.12.2 -f values.yaml --post-renderer ./postrend.sh cilium cilium/cilium

In my case, coredns is reliably crashlooping or otherwise unhappy about not being able to reach the kubernetes API. This can further be debugged by launching a pod that does something like curl https://<IP_ADDRESS>:443/ (the ClusterIP Service default/kubernetes).
Funny thing is that when I change securityContext.privileged to true and upgrade the chart, the problem vanishes.
Inter-Pod communication works in both situations, even between nodes, as long as you connect to Pod IP and not through a service.
Cilium Version
v1.12.2
Kernel Version
5.15.68-talos
Kubernetes Version
v1.25.0
Sysdump
cilium-sysdump-20221006-140816.zip
Relevant log output
No response
Anything else?
I've upgraded to 1.13.0-rc1 without effect.
Code of Conduct

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

I use cilium ctl to install cilium 1.12.2
cilium install --helm-set-string=kubeProxyReplacement=strict,k8sServiceHost=<your loadbalancer IP>,k8sServicePort=6443

have a try on your os Talos Linux 1.2.3.
I think this is dependency  on os

This effectively deploys the helm chart (like I did), with the following values:
kubeProxyReplacement=strict
k8sServiceHost=<your loadbalancer IP>
k8sServicePort=6443

But it will also include the SYS_MODULE capability, which does not exist on Talos, so this install will fail.
I appreciate the help, but I think it's more useful to focus on debugging the kube proxy replacement while running unprivileged.

In the meanwhile, I've discovered there are two scenario's to get it working:

run privileged
deploy the kube-proxy DaemonSet

I think both are not really acceptable, IMO

It seems that it's a distribution issue since, as you said, the distribution doesn't provide SYS_MODULE?

@aanm That's certainly one way to look at it.
The other way would be "Is it possible to do strict KPR without SYS_MODULE?"

@DWSR can you get capsh --print while running Cilium in privileged mode? Thanks

@DWSR can you get capsh --print while running Cilium in privileged mode? Thanks

root@talos-v7n-kwk:/home/cilium# capsh --print
WARNING: libcap needs an update (cap=40 should have a name).
Current: =ep
Bounding set =cap_chown,cap_dac_override,cap_dac_read_search,cap_fowner,cap_fsetid,cap_kill,cap_setgid,cap_setuid,cap_setpcap,cap_linux_immutable,cap_net_bind_service,cap_net_broadcast,cap_net_admin,cap_net_raw,cap_ipc_lock,cap_ipc_owner,cap_sys_module,cap_sys_rawio,cap_sys_chroot,cap_sys_ptrace,cap_sys_pacct,cap_sys_admin,cap_sys_boot,cap_sys_nice,cap_sys_resource,cap_sys_time,cap_sys_tty_config,cap_mknod,cap_lease,cap_audit_write,cap_audit_control,cap_setfcap,cap_mac_override,cap_mac_admin,cap_syslog,cap_wake_alarm,cap_block_suspend,cap_audit_read,38,39,40
Ambient set =
Securebits: 00/0x0/1'b0
 secure-noroot: no (unlocked)
 secure-no-suid-fixup: no (unlocked)
 secure-keep-caps: no (unlocked)
 secure-no-ambient-raise: no (unlocked)
uid=0(root) euid=0(root)
gid=0(root)
groups=
Guessed mode: UNCERTAIN (0)

It seems that it's a distribution issue since, as you said, the distribution doesn't provide SYS_MODULE?

I don't believe so, as required modules are compiled in, so one does not need to load modules. Also, when running privileged, everything works, so the kernel functionality seems there.

@jonkerj @DWSR what is the error that you get when installing Cilium without being privileged and with the SYS_MODULE capability?

On a moduleless-kernel, there is no SYS_MODULE capability (see #21506 for more info on that), so I can't tell. Other than that, there is no error message at all, it is simply not possible to reach services, it quietly fails.

@jonkerj can you get the output of capsh --print when running with privileged and unprivileged w/ sys_module and without sys_module deployed via helm chart?

@jonkerj can you get the output of capsh --print when running with privileged and unprivileged w/ sys_module and without sys_module deployed via helm chart?

I cannot test it with SYS_MODULE, as this capability does not exist. Trying to include it yields an error from the container runtime:
error: failed to create containerd task: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: unable to apply caps: operation not permitted: unknown

So the container is not getting created and therefore there is no capsh-ing possible. If it would start, it would probably show the same as below, but with sys_module included.
Privileged (without SYS_MODULE):
WARNING: libcap needs an update (cap=40 should have a name).
Current: =ep cap_sys_module,cap_sys_boot-ep
Bounding set =cap_chown,cap_dac_override,cap_dac_read_search,cap_fowner,cap_fsetid,cap_kill,cap_setgid,cap_setuid,cap_setpcap,cap_linux_immutable,cap_net_bind_service,cap_net_broadcast,cap_net_admin,cap_net_raw,cap_ipc_lock,cap_ipc_owner,cap_sys_rawio,cap_sys_chroot,cap_sys_ptrace,cap_sys_pacct,cap_sys_admin,cap_sys_nice,cap_sys_resource,cap_sys_time,cap_sys_tty_config,cap_mknod,cap_lease,cap_audit_write,cap_audit_control,cap_setfcap,cap_mac_override,cap_mac_admin,cap_syslog,cap_wake_alarm,cap_block_suspend,cap_audit_read,38,39,40
Ambient set =
Securebits: 00/0x0/1'b0
 secure-noroot: no (unlocked)
 secure-no-suid-fixup: no (unlocked)
 secure-keep-caps: no (unlocked)
 secure-no-ambient-raise: no (unlocked)
uid=0(root) euid=0(root)
gid=0(root)
groups=
Guessed mode: UNCERTAIN (0)

Unprivileged (without SYS_MODULE):
WARNING: libcap needs an update (cap=40 should have a name).
Current: = cap_chown,cap_dac_override,cap_fowner,cap_kill,cap_setgid,cap_setuid,cap_net_admin,cap_net_raw,cap_ipc_lock,cap_sys_admin,cap_sys_resource,38,39+ep
Bounding set =cap_chown,cap_dac_override,cap_fowner,cap_kill,cap_setgid,cap_setuid,cap_net_admin,cap_net_raw,cap_ipc_lock,cap_sys_admin,cap_sys_resource,38,39
Ambient set =
Securebits: 00/0x0/1'b0
 secure-noroot: no (unlocked)
 secure-no-suid-fixup: no (unlocked)
 secure-keep-caps: no (unlocked)
 secure-no-ambient-raise: no (unlocked)
uid=0(root) euid=0(root)
gid=0(root)
groups=
Guessed mode: UNCERTAIN (0)

By the way: 38,39 and 40 are permon, bpf and checkpoint_restore according to kernel source.

@frezbo it looks that Cilium is unable to function properly with Talos even if users deploy Cilium without SYS_MODULE. Is Talos loading these modules by default? https://docs.cilium.io/en/stable/operations/system_requirements/#requirements-for-l7-and-fqdn-policies

@frezbo it looks that Cilium is unable to function properly with Talos even if users deploy Cilium without SYS_MODULE. Is Talos loading these modules by default? https://docs.cilium.io/en/stable/operations/system_requirements/#requirements-for-l7-and-fqdn-policies

All of those are =Y (see source). Also, I wonder: if they were not there, how could it work privileged? Running privileged or not should not influence kernel config.

@jonkerj Given this comment you made on another issue, would making a note in Cilium's KPR documentation be sufficient to close this issue out?

apologies, I was on PTO, still catching up, will take a look into this next week

@jonkerj Given this comment you made on another issue, would making a note in Cilium's KPR documentation be sufficient to close this issue out?

I would rather understand (or better: fix) the underlying problem, rather than documenting that it does not work.

apologies, I was on PTO, still catching up, will take a look into this next week

I would rather understand (or better: fix) the underlying problem, rather than documenting that it does not work.

The linked comment appears to indicate that you have Cilium with KPR working correctly on Talos by passing a custom list of capabilities to the Helm chart. Is that not the case?

I would rather understand (or better: fix) the underlying problem, rather than documenting that it does not work.

The linked comment appears to indicate that you have Cilium with KPR working correctly on Talos by passing a custom list of capabilities to the Helm chart. Is that not the case?

Aaah now I understand. No, it was not the case unfortunately. In that issue I got Cilium running unprivileged, but did not know yet it was actually (silently) misbehaving KPR wise.

Ah, OK.

@frezbo it looks that Cilium is unable to function properly with Talos even if users deploy Cilium without SYS_MODULE. Is Talos loading these modules by default? https://docs.cilium.io/en/stable/operations/system_requirements/#requirements-for-l7-and-fqdn-policies

Quick update: So I've built a custom Talos image to debug which allows SYS_MODULE capability and this is the output of capsh --print from the cilium agent pod (manually installed libcap2-bin) from privileged and non-privilged pod:
privileged
root@talos-default-worker-1:/home/cilium# capsh --print
WARNING: libcap needs an update (cap=40 should have a name).
Current: =ep cap_sys_boot-ep
Bounding set =cap_chown,cap_dac_override,cap_dac_read_search,cap_fowner,cap_fsetid,cap_kill,cap_setgid,cap_setuid,cap_setpcap,cap_linux_immutable,cap_net_bind_service,cap_net_broadcast,cap_net_admin,cap_net_raw,cap_ipc_lock,cap_ipc_owner,cap_sys_module,cap_sys_rawio,cap_sys_chroot,cap_sys_ptrace,cap_sys_pacct,cap_sys_admin,cap_sys_nice,cap_sys_resource,cap_sys_time,cap_sys_tty_config,cap_mknod,cap_lease,cap_audit_write,cap_audit_control,cap_setfcap,cap_mac_override,cap_mac_admin,cap_syslog,cap_wake_alarm,cap_block_suspend,cap_audit_read,38,39,40
Ambient set =
Securebits: 00/0x0/1'b0
 secure-noroot: no (unlocked)
 secure-no-suid-fixup: no (unlocked)
 secure-keep-caps: no (unlocked)
 secure-no-ambient-raise: no (unlocked)
uid=0(root) euid=0(root)
gid=0(root)
groups=
Guessed mode: UNCERTAIN (0)

non-privileged
root@talos-default-worker-1:/home/cilium# capsh --print
WARNING: libcap needs an update (cap=40 should have a name).
Current: = cap_chown,cap_dac_override,cap_fowner,cap_kill,cap_setgid,cap_setuid,cap_net_admin,cap_net_raw,cap_ipc_lock,cap_sys_module,cap_sys_admin,cap_sys_resource+ep
Bounding set =cap_chown,cap_dac_override,cap_fowner,cap_kill,cap_setgid,cap_setuid,cap_net_admin,cap_net_raw,cap_ipc_lock,cap_sys_module,cap_sys_admin,cap_sys_resource
Ambient set =
Securebits: 00/0x0/1'b0
 secure-noroot: no (unlocked)
 secure-no-suid-fixup: no (unlocked)
 secure-keep-caps: no (unlocked)
 secure-no-ambient-raise: no (unlocked)
uid=0(root) euid=0(root)
gid=0(root)
groups=
Guessed mode: UNCERTAIN (0)

Also the output of checking the compiled in Kernel modules required as mentioned in Cilium Docs:
❯ talosctl -n <IP_ADDRESS> read /proc/config.gz | zgrep \
      -e "CONFIG_BPF" \
      -e "CONFIG_BPF_SYSCALL" \
      -e "CONFIG_NET_CLS_BPF" \
      -e "CONFIG_BPF_JIT" \
      -e "CONFIG_NET_CLS_ACT" \
      -e "CONFIG_NET_SCH_INGRESS" \
      -e "CONFIG_CRYPTO_SHA1" \
      -e "CONFIG_CRYPTO_USER_API_HASH" \
      -e "CONFIG_CGROUPS" \
      -e "CONFIG_CGROUP_BPF" \
      -e "CONFIG_NETFILTER_XT_SET" \
      -e "CONFIG_IP_SET" \
      -e "CONFIG_IP_SET_HASH_IP" \
      -e "CONFIG_NETFILTER_XT_TARGET_TPROXY" \
      -e "CONFIG_NETFILTER_XT_TARGET_CT" \
      -e "CONFIG_NETFILTER_XT_MATCH_MARK" \
      -e "CONFIG_NETFILTER_XT_MATCH_SOCKET"
CONFIG_BPF=y
CONFIG_BPF_SYSCALL=y
CONFIG_BPF_JIT=y
CONFIG_BPF_JIT_ALWAYS_ON=y
CONFIG_BPF_JIT_DEFAULT_ON=y
# CONFIG_BPF_UNPRIV_DEFAULT_OFF is not set
# CONFIG_BPF_PRELOAD is not set
# CONFIG_BPF_LSM is not set
CONFIG_CGROUPS=y
CONFIG_CGROUP_BPF=y
CONFIG_NETFILTER_XT_SET=y
CONFIG_NETFILTER_XT_TARGET_CT=y
CONFIG_NETFILTER_XT_TARGET_TPROXY=y
CONFIG_NETFILTER_XT_MATCH_MARK=y
CONFIG_NETFILTER_XT_MATCH_SOCKET=y
CONFIG_IP_SET=y
CONFIG_IP_SET_MAX=256
CONFIG_IP_SET_BITMAP_IP=y
CONFIG_IP_SET_BITMAP_IPMAC=y
CONFIG_IP_SET_BITMAP_PORT=y
CONFIG_IP_SET_HASH_IP=y
CONFIG_IP_SET_HASH_IPMARK=y
CONFIG_IP_SET_HASH_IPPORT=y
CONFIG_IP_SET_HASH_IPPORTIP=y
CONFIG_IP_SET_HASH_IPPORTNET=y
CONFIG_IP_SET_HASH_IPMAC=y
CONFIG_IP_SET_HASH_MAC=y
CONFIG_IP_SET_HASH_NETPORTNET=y
CONFIG_IP_SET_HASH_NET=y
CONFIG_IP_SET_HASH_NETNET=y
CONFIG_IP_SET_HASH_NETPORT=y
CONFIG_IP_SET_HASH_NETIFACE=y
CONFIG_IP_SET_LIST_SET=y
# CONFIG_BPFILTER is not set
CONFIG_NET_SCH_INGRESS=y
CONFIG_NET_CLS_BPF=y
CONFIG_NET_CLS_ACT=y
CONFIG_BPF_STREAM_PARSER=y
CONFIG_CRYPTO_SHA1=y
# CONFIG_CRYPTO_SHA1_SSSE3 is not set
CONFIG_CRYPTO_USER_API_HASH=y
CONFIG_BPF_EVENTS=y
# CONFIG_BPF_KPROBE_OVERRIDE is not set

So the issue seems to be not related to the SYS_MODULE capability. In both cases cilium service list shows the default kubernetes ClusterIP service to be active:
root@talos-default-worker-1:/home/cilium# cilium service list
ID   Frontend           Service Type   Backend
1    <IP_ADDRESS>:443      ClusterIP      1 => <IP_ADDRESS>:6443 (active)

Anything else I can look into to debug?

Attaching the cilium agent logs in both privileged and unprivileged modes and also the output of cilium debuginfo
cilium-agent-privileged.log
cilium-agent-privileged-debuginfo.log
cilium-agent-unprivileged.log
cilium-agent-unprivileged-debuginfo.log

@aanm does the logs provided above provide any hint? Is there any other information needed?

Hello Everyone!
I'm new to Cilium and wanted to give it a shot by running on RPi4 & Talos OS.
Just to mention that I'm experiencing the same problem running Talos 1.3.2 and Cilium 1.12.5 (kubeProxyReplacement=strict ). E.g.: pod2pod comms are working but service IPs are not reachable. I was following the official guidance from Talos documentation - https://www.talos.dev/v1.3/kubernetes-guides/network/deploying-cilium
I wonder if it was ever working in this mode?
I've also noticed some errors from the Cilium apply-sysctl-overwrites init container:

can't stat sysctl.d dir '/etc/sysctl.d/': stat /etc/sysctl.d/: no such file or directory

Which might be correct with Talos but not sure if this can cause some troubles?

@dimatha kubeProxyReplacement: strict works correctly on Talos, but only when securityContext.privileged is true.

@DWSR Thanks! Confirmed.

Strict mode fails for me on that setup (rpi4, talos 1.3.2) but with cilium 1.13.0-rc4 and ipv6 enabled. The agent fails with
could not load module ip_tables: exit status 1

Disabling ipv6 makes it work again.

Strict mode fails for me on that setup (rpi4, talos 1.3.2) but with cilium 1.13.0-rc4 and ipv6 enabled. The agent fails with

could not load module ip_tables: exit status 1

Disabling ipv6 makes it work again.
Edit: agent is running in privileged mode.

This will be fixed by https://github.com/cilium/cilium/pull/23953

Excellent. But that still does not solve the issue that Cilium is not working in unprivileged mode under Talos, without giving any error or whatsoever.

@jonkerj this seems to have fixed it, could you try with these:
ipam:
  mode: kubernetes
hostServices:
  hostNamespaceOnly: true
socketLB:
  hostNamespaceOnly: true

and doing a cilium install --helm-values values.yaml

@jonkerj this seems to have fixed it, could you try with these:

Sure, I will. It will take be a while, as my test cluster needs to be revived

okay, I've drilled this down to specifying the existing cgroup mount that talos uses, helm/cli values file used:
ipam:
  mode: kubernetes
kubeProxyReplacement: strict
securityContext:
  capabilities:
    ciliumAgent:
      - CHOWN
      - KILL
      - NET_ADMIN
      - NET_RAW
      - IPC_LOCK
      - SYS_ADMIN
      - SYS_RESOURCE
      - DAC_OVERRIDE
      - FOWNER
      - SETGID
      - SETUID
    cleanCiliumState:
      - NET_ADMIN
      - SYS_ADMIN
      - SYS_RESOURCE
cgroup:
  autoMount:
    enabled: false
  hostRoot: /sys/fs/cgroup
k8sServiceHost: <IP_ADDRESS>
k8sServicePort: 6443

The k8sServiceHost and k8sServicePort needs to be updated

okay, I've drilled this down to specifying the existing cgroup mount that talos uses, helm/cli values file used:
ipam:
  mode: kubernetes
kubeProxyReplacement: strict
securityContext:
  capabilities:
    ciliumAgent:
      - CHOWN
      - KILL
      - NET_ADMIN
      - NET_RAW
      - IPC_LOCK
      - SYS_ADMIN
      - SYS_RESOURCE
      - DAC_OVERRIDE
      - FOWNER
      - SETGID
      - SETUID
    cleanCiliumState:
      - NET_ADMIN
      - SYS_ADMIN
      - SYS_RESOURCE
cgroup:
  autoMount:
    enabled: false
  hostRoot: /sys/fs/cgroup
k8sServiceHost: <IP_ADDRESS>
k8sServicePort: 6443

My initial impression is that this works correctly on Talos. Thanks @frezbo

Scratch that, traffic using in-cluster service IPs are broken when using this, but external services are fine.

that's weird, all cilium connectivity test passes, and coredns comes up fine, this was tested on talos main (1.4.0.alpha)

Using @frezbo's values, I can confirm my symptom has been resolved:
some@testpod:$ wget -qO/dev/null https://<IP_ADDRESS>:443/
wget: note: TLS certificate validation not implemented
wget: server returned error: HTTP/1.1 401 Unauthorized

This shows a test pod at non-CP node is able to reach the API server at it's ClusterIP service. Also, CoreDNS is working again:
some@testpod:$ nslookup kubernetes.default.svc.cluster.local
[..]
Name:   kubernetes.default.svc.cluster.local
Address: <IP_ADDRESS>

So, inter-node traffic works, ClusterIP also works. LGTM, basically.


Fix signature threshold

Fixes issue #:
None
Description of the changes being introduced by the pull request:
Prior to this PR metadadata signature verification as provided by tuf.sig.verify() and used e.g. in tuf.client.updater counted multiple signatures with identical authorized keyids each separately towards the threshold.
This PR changes this to count identical authorized keyids only once towards the threshold.
It further clarifies the behavior of the relevant functions in the sig module, i.e. get_signature_status and verify in their respective docstrings. And adds tests for those functions and also for the client updater.
An alternative fix is outlined in the commit message of a0397c7 (including a patch).
Please verify and check that the pull request fulfills the following
requirements:

[ ] The code follows the Code Style Guidelines
[ ] Tests have been added for the bug fix or new feature
[ ] Docs have been added for the bug fix or new feature

Note, this was first reported to us by Erik MacLean at Analog Devices, Inc.
More information about that disclosure will be forthcoming.


[RFC] examples: split out "fedora/minimal-common.yaml" and define two archs

Tweak the example to be slightly more realistic and have two arches for fedora and a common yaml. This will probably see more refactor when an additonal fedora that is not minimal is added.
Tiny starting point, if this looks okay I will play a bit more with how fedora maybe in two arches and two flaviors could look like :)

This is nice; you can also take some inspiration from the last big branch to be merged: https://github.com/supakeen/otk/tree/targets/example/fedora

This is nice; you can also take some inspiration from the last big branch to be merged: https://github.com/supakeen/otk/tree/targets/example/fedora where I was trying out some similar organisation but also have a lot of hacky bits due to testing new features of that branch.

Cool, how do you want to do this, do you want to split this out from your tree and open separately or should we merge/keep this one and then rebase the other one on top of this?


Update v20.2 architecture docs re: range merges

Fixes #10577
Summary of changes:

Port range merges section from v21.1 Distribution Layer page to v20.2
page

Fix up all the links to point to the above

Delete the old standalone 'Range Merges' page

Finally, sneak in a little change to both the v20.2 and v21.1 pages:
add 'Range merges' to the list of things that cause range descriptor
updates

Hi Nathan
this is mostly a "backport" of range merges info to the v20.2 dist layer docs per the linked issue  except that I also added "range merges" to the list of things that causes a range descriptor to be updated in both the v20.2 and v21.1 docs (which ISTR you mentioned in another issue or I read somewhere?).  Anyway happy to make any updates as needed


Unable to run built-in commands using OS.execute

Sample Project
Godot version:
3.0.6
8d117b214f2bcd14015532fdfbed9b3f059da0e6
OS/device including version:
Windows 10 x64
Issue description:

var output = []
OS.execute('echo', ['somethingyo'], true, output)

Produces the following:
'"echo"' is not recognized as an internal or external command,
operable program or batch file.

It appears that the path argument is being automatically surrounded in quotes, per OS_Windows::execute.
As mentioned in this issue, this can currently be circumvented by calling CMD /K "<command>" - however this is not an elegant solution as it requires launching a second process. I propose one of the following solutions which doesn't require this workaround.
Solution 1:
Remove automatic quotes. This would have the downside of requiring the user to use quotes
when necessary.
Solution 2:
Enclose the path with quotes only when there is whitespace in the path.
Solution 3:
Check if the command is a DOS command, and enclose the path in quotes if it is not.

this is not an elegant solution as it requires launching a second process

AFAIK removing quotes won't help with internal commands. cmd is the only way to use them. Unlike Unix tools, Windows/DOS internal commands don't have separate executables and are all bundled into cmd.exe (or command.com in case of DOS) and should not spawn extra processes.

It works as long as we remove the quotes. Consider the following script:
func _ready():
    var output = []
    var res = OS.execute('echo somethingyo', [], true, output)
    for o in output:
        print(o)

Currently we are using
argss = "\"\"" + p_path + "\"";

which produces
'"echo"' is not recognized as an internal or external command,
operable program or batch file.

However if I change it to
argss = "\"" + p_path;

then we get

Hah, interesting, blocking/pipe version of OS::execute uses _popen call which seems to support external commands, but it have this note in the documentation:

If used in a Windows program, the _popen function returns an invalid file pointer that causes the program to stop responding indefinitely. _popen works properly in a console application.

I wonder if it is working at all in exported app or only in editor (which is console application)?
Non blocking version uses CreateProcess and should not work with internal commands (at least it was not in Windows 7 times).

Hm, it does concern me that it wouldn't work with non-blocking option. I suppose we can either:

Leave the current behavior intact, and allow the user to launch cmd /K for builtins
I honestly cannot think of any scenarios where this isn't at least acceptable, though it's unfortunate we're basically dealing with two cmd processes doing so
Figure out some sort of compatibility with CreateProcess (which I assume would involve us wrapping such commands in a cmd /K)
Replace CreateProcess with popen
I'm honestly not sure why we use CreateProcess for non-blocking/no-pipe branch - from what I gather _popen is non-blocking and we'd be free to simply open it and move on, though I haven't yet tested this idea.
Document somewhere that only blocking calls can directly call builtins
This just seems like a bad idea

I would assume it should work in an exported app, though I will test it tomorrow. If it doesn't, that's likely a larger concern since it would affect existing expected behavior.

The quotes should only be added if they are needed. Would that solve the issue?

Some tests.
Non blocking var pid = OS.execute('echo', ['somethingyo'], false) fails with:
ERROR: execute: Condition ' ret == 0 ' is true. returned: ERR_CANT_FORK
At: platform/windows/os_windows.cpp:2494

Blocking version works in exported app, but it's opening new console window each time, even when it's starting GUI app.
Also cmd /k is not auto-closing console after command execution and will hang in case you are using editor executable to run project from existing command line session, it's better to use cmd /c instead.

Tested in 3.2.4 beta3 and the provided code sample seems to do nothing, not even error appears.


Improve lesson modal appearance

The lesson modal looks a little dull.

We could spice up the design whilst keeping colour schemes consistent with the rest of the site. If you're having trouble finding stylistic inspiration maybe take a look at https://dribbble.com/tags/modal

Does it look good? If so, let me work on that 👍

That looks amazing @marcinwmichna. A massive improvement!  Current updates have been made to the modal title in #37, I think there is still scope for more improvements - someone may want to give this a go too.

I would like to work on this.

I would like to work on this.

Go for it! I'd recommend creating separate pull requests if you are tackling 2 different issues.

Go for it! I'd recommend creating separate pull requests if you are tackling 2 different issues.

sure!!

sure!!

Hey @medhavi11, are you still working on this?

Is it better than the above one? If so, let me know

Hey I want to contribute to this and I would like to improve the UI for the lesson modal. Is this issue still open ? .Please let me know..!


Define guided tutorial structure and schema

Describe the feature few words
We need to define file structure for guided tutorials as well as content formatting. This way front end will be able to use that content and show guided tutorials UI.
What will be the end result of this change?
Content for guided tutorials according to the format. We should provide one such example for others to reference.
Is there any previous work that needs to be completed before making this change?
No.
Which components will this change affect?
The frontend will need to implement this format.

Lets first start with Hello World tutorial

https://wiki.ubc.ca/Documentation:Mini-Lessons_Basics_-_BOPPPS_Model_for_Planning_Lessons

I would imagine that the files would be the same as they are currently, right?

I would keep the current structure of contracts, scripts, transactions. The one thing I would add is doc files separated into steps accompanied by a test script file that would tell us if the step was completed or not.

I don't have any context on this, so I'm not sure what this is actually accomplishing. The code for the tutorials is stored here: https://github.com/onflow/flow-playground-tutorials
and we've been just using a regular saved playground session via a shared link to provide a starting state for each tutorial.
I think it would be ideal for the playground to load the files from that (or another repo) directly so we can stop worrying about copying and saving playground links because that is no fun. Is that what this task is meant to accomplish or is it different than that?

Hey, yeah our goal is to pull the files directly from that repo. But that means the repo will have to follow some structure (and I feel the established structure of having contracts, scripts, transactions will do). But since this tutorials will become guided tutorials which means a tutorial will be broken into multiple steps and each step will have a check (I'm thinking a script testing the action was actually done). You can check the guided tutorial UI in the figma for you to have a better sense of what we are trying to do. Also the content of each tutorial will then somehow be broken in those steps. So we have to agree how we will denote that in the content or structure (steps into multiple markdown files or with certain titles or any other ideas). Do you have some input?

You can check the guided tutorial UI in the figma

Where do I find this?
I think keeping it in the established structure is good, but it is still unclear to me what other structure there needs to be. Are you saying that we'd break up the existing tutorial into multiple markdown files that are shown between each code step that the user takes? That seems fine to me if that is the case, but I'll defer to y'all. Y'all can break it up however you want, but I'd want to be consulted first if you're planning on changing the actual content of the explanations and such in the tutorials

we'd break up the existing tutorial into multiple markdown files that are shown between each code step that the user takes?

Yes, either multiple md files or maybe define the steps by title ("H1/H2.." level). The folder structure should stay as it is. But how about adding the test scripts validating each step? should that be part of the tests folder? I don't want to put that in the "scripts" folder as the scripts will all be showed as part of the tutorial script files then.

Those can probably just be in the scripts folder because they might be useful for the user to run on their own as well as the tutorial running it in the background

@joshuahannan the problem is that then we have to do some additional logic to not show those in the FE of playground. I feel we should define where tests will live as part of this standards, especially now with Cadence adding support for tests it would be wise to add that to the current standard/structure. I feel there are two ways to do this, one is to have a test folder where all tests live the other is to have test along with things they are testing (those are two established patterns I've seen in web2 development) but named with a prefix like test_. I don't mind either but I feel it's important we define this. What are your thoughts?

I'm saying that they should be visible in the FE of the playground. A user might want to see what is being run to verify their state is correct and run it themselves.

No, we don't want that.

No, we don't want that.
Can you elaborate a bit? I usually like giving the user more flexibility to explore and try stuff out if they're learning

As for where to put Cadence tests, I'm in favor of having them in separate directories

Of course, sorry. A product decision was made not to expose that, as far as my opinion goes, I agree, and the reason behind that is that purpose of the playground is to onboard new developers and make their learning curve as easy as possible. Exposing how we test whether a certain test passes for them to proceed with the tutorial will expose the mechanics behind guided tutorials and expose more scripts they need to understand why are there in the first place. If you take a look in the designs here you will see we have abstracted the process of detecting completion of each step for the purpose of hiding complexity. I normally would agree to show users as much as possible, but in the case of making onboarding as easy as possible, it's not probably the best. cc @srinjoyc if you want to elaborate further.

👍 we could however optionally expose that if the user decides to do that. But that is then also a bit out of the scope of this initial work.


Invoke-WebRequest to show expected download size and more human readable format (MB/GB instead of Bytes)

Summary of the new feature/enhancement
Currently, users of Invoke-WebRequest do not know how much they have downloaded because a big Byte number gets displayed and due to the digits not being mono-spaced, I cannot even attempt to count digits whilst the number changes every second. Also, I have no progress bar to tell me how much it needs to download.
Proposed technical implementation details (optional)
ByteSize is a small (14KB) library to better display human readable file sizes but as the problem is quite simple, it was decided to not introduce a dependency but rather just hand-code it instead. Because PowerShell treats 1 KB as 1024, it should probably use 1024 as the base for KB, MB, etc.
I suggest to have one significant digit after after the comma starting with KB and increase the number of significant digits by 1 for MB, GB, etc.
PR here: https://github.com/PowerShell/PowerShell/pull/14611

This would be really nice to have

A true progress bar with this information would be very helpful:

Download size as in (Invoke-WebRequest $url -Method Head).Headers.'Content-Length'
Already downloaded as described in OP
Percentage downloaded.

Some of my server setup and automation tasks rely on Invoke-Webrequest.  A progress bar would be nice, but I would be content with some human-readable "bytes downloaded" display.  "bytes" is just not a human-readable unit of measure when you are talking about MB-GB file sizes.  At least not for this human.
So this is me piling on.  Yes, please implement this request.


fix removed collider event not triggering

Fixes https://github.com/appsinacup/godot-rapier-physics/issues/290

Does this also fix collision when using the disabled property on collision shapes?

Hm, not sure, I only tested the repro project you uploaded and it works with that.

Mergin and you can test with main build to see for yourself. Locally it seems good.


XOutput crash when i plugin gamepad (HidGuardian hidden device) [3.25]

XOutput crash when i unplug and plugin my gamepad. Gamepad is hidden in HidGuardian driver, by XOutput affect button.
My settings:

When I plug gamepad it's start to stutter all input devices for about 3 sec. It's also crash xoutput and generate ViGEmClient dump file:
Dump file (zipped):
ViGEmClient.zip
Log file:
XOutput.log
Sorry for my english

This issue started at version 3.23.

This will be fixed in 3.26.


Practical sample .js file - how to use axios with example

Hi,
I want to use Axios as a tool to do REST API testing. I am new to this. I just use below code to access the google API, but it does not show any expected out put. where as if I paste the same URL in browser it gives JSON result. Could you please help me with sample file how to sue it.
Note that I have replaced the valid UR_API_KEY .
var axios = require('/usr/local/lib/node_modules/axios/');
xios.get('https://maps.googleapis.com/maps/api/place/nearbysearch/json?location=-33.8670,151.1957&radius=500&types=food&name=cruise&key=UR_API_KEY')
.then(function (response) {
console.log(response);
})
.catch(function (response) {
console.log(response);
});
Thanks,
Amiy

So what is output? BTW, you should log response.data, not response.

Thanks Nick for your quick reply..
Actually it is not authenticating, and loging, and thats why I think I am not getting response. If I user response.data, I get the html tag as out put.
So basically in our application, it has an authentication token which required to access the end point url

is Base url
is end point URI
is the token
user name and pasword (if required). I use below code, but bot getting desired response

var config = {
baseURL: 'https://auth-uat.corp.abc.com/WebAuth/classicLogin.html?appIdKey=d612fb79f12334578f0ee0f6db39043bf8153&sslEnabled=true',
responseType: 'json',
xsrfCookieName: 'myacinfo-uat',
xsrfHeaderName: 'd612fb79f12334578f0ee0f6db39043bf8153',
withCredentials: true,
auth: {
	username: 'user_test1',
	password: 'testuser'
}

Please help us.
Thanks in advance - Amiya
};

Can you look at the logs to see exactly what the URL is that is being requested? You are setting a full URL with query string and all for baseURL which may not be what you want to do. Any path or query that are specified in your actual call to axios will be appended to the baseURL.
axios.defaults.baseURL = 'https://api.example.com/';
axios.get('/auth/login');

The above will make a request to https://api.example.com/auth/login

Hi  Mzabriskie,
I checked the log, it is generating the comple URL. In our application we cant directly hit the end-point-URL directly. It has to go through the authentication process with Auth token. So to authenticate I am using base URL with token d612fb79f12334578f0ee0f6db39043bf8153. Could you please help us?
Thanks,
Amiya

@amiya-pattnaik Are you still having this problem?

Closing this due to inactivity. Please feel free to reopen if needed.


There is an extra step "Select subscription" when deploying changes by clicking "Yes" button from the prompt notification

OS: Win10
Build Version: 20240430.1
Regression: Not a regression
Repro Steps:

Make sure more than one subscription is selected.
Execute "Edit Scaling Range..." action for a container app.
Click "Yes" button in the prompt notification "Deploy changes now?".

Check whether there is no extra step "Select subscription".

Expect:
There is no extra step "Select subscription".
Actual:
There is an extra step "Select subscription".

Good catch.  When we call the deploy command we should probably attach the existing subscription so that we don't end up re-prompting the user for this info.


Chocolatey closing as soon as it opens. How to fix this?

It is like crashing. It closes as soon as I open it.

Chocolatey is a command line program, not a GUI program. So if you are double-clicking on choco.exe, then it is expected that it does not open up a GUI.
Have you tried opening an administrator command prompt, and running choco --help?

@neethunath I am going to go ahead and close this issue out, as I suspect what @TheCakeIsNaOH has said is what was happening.
If this is something else, then feel free to respond to this issue.


Prevent bundle to lookup groups/roles

Hey,
my customer reported some issues with his AD connection and after digging into the code, I've found out, that the bundle automatically is trying to load groups in the LdapRoleMapper on loadUserByUsername() ....even without any role mapping configured at all (edit: Which is the default value)!
Using a deep AD structure resulted in failed (timeout?) queries as the check is recursively by default, too.
The bundle shouldn't run the query if $roles is empty.
Cheers
Matthias

Thanks for helping resolve this. I've tagged a new minor version. Regarding the recursive check, that option can be toggled via configuration options for the bundle. The fact that it times out when querying recursively with (some of?) their user's really points toward poor group design / nesting unfortunately.


Combine multiple images / Sprite

Is this possible? For instance, creating a sprite from multiple images to save on HTTP requests.

I don't understand what you want to do
Maybe you're talking of merge ()?

For instance like https://github.com/Ensighten/grunt-spritesmith
Interesting, I didn't see merge before. It's not quite what I'm after as with this it seems you need to pass x, y, width, height, i was looking for more an automatic method for combing multiple images maybe one on-top of another. Have you seen the CSS Sprite technique? That's primary reason for wanting this

I supose thinking about it you would need a way of generating a CSS file as well with the x, y start coordinates for the merged/combined images, or at least a dedicated method that will return a keyed-filename array with the coordinates like:
$coordinates = Gregwar::createSprite('file1.jpg', 'file2.jpg');

'file1.jpg' => [0, 0],
'file2.jpg' => [0, 300]

Probably outside scope of this package itself though.

Indeed, but it's not a really hard problem though
I think there is already scripts to do this specifically


Default header and formatting in post_processing module

Lastly, I have been doing quite a lot of string parsing in PyMAPDL.
I found that the next combination is very useful for parsing some APDL command outputs (see notes).
mapdl.post1()
mapdl.header('OFF', 'OFF', 'OFF', 'OFF', 'OFF', 'OFF')
nsigfig = 10
mapdl.format('', 'E', nsigfig + 9, nsigfig)
mapdl.page(1E9, '', -1, 240)

Hence I would propose either:

Setting the previous options during the construction of the post_processing class. So those mapdl commandswill be included in the init method.
If we are interested in being very consistent on the format. We could do a wrapper for the PRXXXX commans (see notes) which will:

Read the previous format (mapdl.format('STAT')).
Store it in a tmp variable.
Change the format to our format.
Output the PRXXXX command.
Setting the previous format back from the tmp file.

Both options are doable. The second, although more complete, might pose some overload.
I'm implementing the first. If there is interest, I will implement the second too.
Notes
/FORMAT affects to PRNSOL, PRESOL, PRETAB, PRRSOL, PRPATH, and CYCCALC commands.

I think the best approach is to do nothing and then post-process the string output as needed in the suggested CommandOutput class. Best for us to not change the default output and instead create an intelligent parser for command output to convert to numpy arrays, but not change the default behavior of MAPDL.

This is my parsing function for APDL ascii outputs which automatically excludes headers. Please feel free to use or implement it, it you like.
import re
from typing import Optional, Union

import numpy as np
import pandas as pd

def read_string(string: str,
                return_type: Optional[str] = 'np.ndarray'
                ) -> Union[np.ndarray, pd.DataFrame]:
    """
    Reads the output strings of mapdls ``ETABLE`` or ``PRNSOL`` commands and converts
    them either to a `np.ndarray` or to `pd.DataFrame`.

    .. code-block:: python

        >>> read_string(mapdl.pretab('s', 'x'))
        array([[1.0000e+00, 2.1009e-03],
               [2.0000e+00, 2.5280e-03],
               [3.0000e+00, 2.8548e-03],
               ...
        >>> # Format: [['Element number', 'Value'], ...]

    Parameters
    ----------
    string : str
        Input string, e.g. from ``mapdl.pretab()`` or ``mapdl.prnsol()``
    return_type : str, optional
        Output type, one of ``['np.ndarray', 'pd.DataFrame']``, by default
        ``'np.ndarray'``.

    Returns
    -------
    Union[np.ndarray, pd.DataFrame]
    """
    array = []

    for line in string.splitlines():

        # if line starts with node or element num
        if re.match('(?=.*[1-9])^[0-9|\s]{8}', line[:10]):
            # find all trailing non-zero integers | scientific floats | fixed point floats
            matches = re.findall(
                '((?=.*[1-9])^[0-9|\s]{8})|(.?[0-9]\.[0-9]{5}E.[0-9]{3})|(\-?[0-9|\.]{6})',
                line,
            )

            # concatenate nested tuples of re.findall and convert to int/float
            array.append(
                [int(''.join(x)) if idx==0 else float(''.join(x))
                 for idx, x in enumerate(matches)]
            )

    if return_type == 'np.ndarray':
        return np.array(array)

    elif return_type == 'pd.DataFrame':

        for line in string.splitlines():

            if any((re.match('^\s{4}NODE', line), re.match('^\s{4}ELEM', line))):

                columns = line.split()
                break

        if not columns:
            raise ValueError('Could not identify any columns.\n'
                             + 'May the input string be invalid/empty?')

        return pd.DataFrame(np.array(array), columns=columns).set_index(columns[0])

Fixtures:

import pytest

@pytest.fixture
def etable_fixture() -> str:

    with open('test/test_modules/test_reference_files/pretabux.txt', 'r') as f:
        etable = f.read()

    return etable

@pytest.fixture
def prnsol_u_comp_fixture() -> str:

    with open('test/test_modules/test_reference_files/prnsolucomp.txt', 'r') as f:
        prnsol = f.read()

    return prnsol

@pytest.fixture
def prnsol_s_comp_fixture() -> str:

    with open('test/test_modules/test_reference_files/prnsolscomp.txt', 'r') as f:
        prnsol = f.read()

    return prnsol

Tests
import numpy as np

def test_read_string_etable_np(etable_fixture):
    """Asserts that read_string can handle etables correctly."""

    array = read_string(etable_fixture)

    assert np.shape(array) == (100, 2)

def test_read_string_etable_pd(etable_fixture):
    """Asserts that read_string can handle etables correctly."""

    df = read_string(etable_fixture, return_type='pd.DataFrame')

    assert df.shape == (100, 1)

def test_read_string_prnsol_u_np(prnsol_u_comp_fixture):
    """Asserts that read_string can handle nodal solutions correctly."""

    array = read_string(prnsol_u_comp_fixture)

    assert np.shape(array) == (341, 5)

def test_read_string_prnsol_u_pd(prnsol_u_comp_fixture):
    """Asserts that read_string can handle nodal solutions correctly."""

    df = read_string(prnsol_u_comp_fixture, return_type='pd.DataFrame')

    assert df.shape == (341, 4)

pretabux.txt
prnsolscomp.txt
prnsolucomp.txt

Hi @engineering-luke , this is a nice piece of work! I will use it as a base for the CommandOuput branch #735 . Thank you very much!

Closing issue in favor of #735 .


[Question]How to ignore node modules?

The npm run dist command also bundles node modules into the app.
How to ignore node modules?

Tried with
"excludes": [
      "./node_modules/*"
 ]

But no luck :(

What about this:
"excludes": [
      "./node_modules/**/*"
 ]

BTW, node_modules of devDependencies is ignored by default. If a module is used only in development, make it a devDependency.

Nope that doesn't work either.
And yes, modules that are used only during development are in devDependency. But I do have a bunch of dependencies which in turn have their dependencies. So it is a pretty big list of node_modules :P
Also, I am using webpack to extract all the dependencies and create one single bundle which is put in the dist along with the entry html. Hence I do not need the node_modules included

Code is located here.
I get it to work with:
"excludes": [
    "**/node_modules",
    "**/node_modules/**/*"
]

I have no idea why ./node_modules doesn't work and I will check it out later.

@evshiron Okay adding both "**/node_modules" & "**/node_modules/**/*" makes it ignore the node_modules folder. Thanks for the help. But still wonder why do I need to add both. Ideal thing to do would be just ./node_modules/* (like .gitignore)


Fixed playground not displaying docs if api is hidden behind reverse …

…proxy (#16810)
I want to merge this change because...
Issue described here:
https://github.com/saleor/saleor/issues/16810
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations

Docs

[ ] Link to documentation:

Pull Request Checklist

[ ] Privileged queries and mutations are either absent or guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migrations are either absent or optimized for zero downtime
[ ] The changes are covered by test cases
[ ] All new fields/inputs/mutations have proper labels added (ADDED_IN_X, PREVIEW_FEATURE, etc.)
[ ] All migrations have proper dependencies
[ ] All indexes are added concurrently in migrations
[ ] All RunSql and RunPython migrations have revert option defined

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Quick question for maintainers - where ideally these lines missing from coverage should be tested? Couldn't find relevant tests after short search

Quick question for maintainers - where ideally these lines missing from coverage should be tested? Couldn't find relevant tests after short search

Hey, IMO the best place would be graphql/core/tests/test_view.py.
Something like this should work:
from unittest.mock import patch
from django.shortcuts import render
import pytest

@patch("saleor.graphql.views.render", wraps=render)
def test_1(mocked_render, rf, settings):
    # given
    request = rf.get(path="/", )
    request.app = None
    request.user = None
    settings.PUBLIC_URL = ""

    # when
    view = GraphQLView.as_view(backend=backend, schema=schema)
    view(request)

    # then
    mocked_render.assert_called_once_with(
        request,
        "graphql/playground.html",
        {
            "api_url": 'http://<IP_ADDRESS>:8000/graphql/',
            "plugins_url": 'http://<IP_ADDRESS>:8000/plugins/',
        },
    )

Also, would be good to use pytest.mark.parametrize to test all different values for PUBLIC_URL.

Hey @jakub-borusewicz thank you for your changes. We have introduced ruff validation rules in the meantime. Could you address what ruff complains about? Thank you!


new lexer request: stylus (css)

(Original issue 936 created by krebo on 2013-12-10T04:05:57.644389+00:00)
Would be great to have support for the Stylus preprocessor in Pygments.
The docs are a little dense for me at the moment. Can someone point me in the right direction for how to implement this?

(Original comment by krebo on 2013-12-10T04:41:19.695659+00:00)
Started at pull request #264

(Original comment by tshatch on 2016-06-01T22:27:23.181221+00:00)
This is especially tricky, please send a new PR if you can fix the issues and conflicts.


Add a rule testing option (-t)

Add a -t command line option for rule testing, that would take just the RULES_FILE argument and test the given rule file(s) for rule syntax errors.
Use case: Quickly test rules for invalid syntax without needing to give a file argument or run on a file.

I would use yarac for this as @wxsBSD proposes. If you don't want yarac to generate the output file we can add a flag to yarac that prevents the compiled file to be written.

@plusvic That sounds good, thanks.

The syntax for yarac is:
yarac [OPTION]... [NAMESPACE:]SOURCE_FILE... OUTPUT_FILE

It accepts multiple source files and uses the last argument as the output file. The output file can't be made optional without changing the arguments format in a backward-incompatible way. Adding a flag for ignoring the output file while being forces to specify a file is a bit weird. Using /dev/null sounds like an acceptable solution (at least in OSes that have it):
yarac rules.yar /dev/null


Publish based on Access Control Lists

We're using Symfony for the application specific logic, and will implement Ratchet for notification messages. React/ZMQ will be used to implement server-push integration, but I'd like to combine it with the Symfony ACL without the client having to do an AJAX call whether or not it should eventually show the message.
I think it has something to do with the SessionProvider, but can't figure it out exactly. Where does my ACL logic has to come?

Unfortunately I have no experience with Symfony ACL so I'm unable to comment.
I'm closing this issue as there hasn't been any activity in over 3 months. If you're still looking for a solution you could try posting your question on our Mailing List


Connection lost when refresh page or go to another page

I build exe with pyinstaller and on some PC I have trouble with refresh page or going to another page in my app using eel. if I open link in another tab in browser and no closing main tab all is working but it isn't good solution i think

Is there any news or updates on this? I'm running into the same issue when attempting to travel to a different file that's available in my static web folder.
I have a login.html file and I want to move to index.html but application exits when doing so.

I just refresh connection but I think this is not good solution :)
Simply if page is open more than 15 - 20 secs connection lost

Try rewrite close_callback, eventhough do it only in develop?
Server will run after exit and same port may not working next run...
if devel:
      directory = 'gui/src'
      app = None
      page = {'port': 8080}

      def close_callback(page, sockets):
          pass

  else:
      directory = 'gui/web_builded'
      close_callback = None
      app = 'chrome-app'
      page = 'index.html'

      eel.start(page, mode=app, close_callback=close_callback, host='localhost', port=8686, disable_cache=True)

@igrG @becurrie @mivendors2020 this shouldn't be an issue any more - the latest version of Eel will try to reconnect when the connection is dropped.
If Eel is existing when you go to a new page in your app, you probably haven't included eel.js on that page.
I'm going to close this for now but let me know if there's any outstanding issues.


Investigate CTFd WORKER TIMEOUT

During CTFd service start, we collect a bunch of [CRITICAL] WORKER TIMEOUT. This naturally resolves its self fairly quickly, but it probably means we're doing something wrong. Something about service start must be slow (and if I remember right, locking).
Every 30s a bunch of workers timeout, with the number timing out slowly decreasing:
ctfd  | [2023-07-10 18:20:27 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:23)
ctfd  | [2023-07-10 18:20:27 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:24)
ctfd  | [2023-07-10 18:20:27 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:25)
ctfd  | [2023-07-10 18:20:27 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:26)
ctfd  | [2023-07-10 18:20:27 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:27)
ctfd  | [2023-07-10 18:20:27 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:28)
ctfd  | [2023-07-10 18:20:27 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:29)
ctfd  | [2023-07-10 18:20:58 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:99)
ctfd  | [2023-07-10 18:20:58 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:100)
ctfd  | [2023-07-10 18:20:58 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:101)
ctfd  | [2023-07-10 18:20:58 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:102)
ctfd  | [2023-07-10 18:20:58 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:103)
ctfd  | [2023-07-10 18:20:58 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:104)
ctfd  | [2023-07-10 18:20:58 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:105)
ctfd  | [2023-07-10 18:21:29 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:176)
ctfd  | [2023-07-10 18:21:29 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:177)
ctfd  | [2023-07-10 18:21:29 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:178)
ctfd  | [2023-07-10 18:21:29 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:179)
ctfd  | [2023-07-10 18:21:29 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:180)
ctfd  | [2023-07-10 18:21:29 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:181)
ctfd  | [2023-07-10 18:21:59 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:254)
ctfd  | [2023-07-10 18:21:59 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:255)
ctfd  | [2023-07-10 18:21:59 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:256)
ctfd  | [2023-07-10 18:21:59 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:257)
ctfd  | [2023-07-10 18:21:59 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:258)
ctfd  | [2023-07-10 18:21:59 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:259)
ctfd  | [2023-07-10 18:22:30 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:321)
ctfd  | [2023-07-10 18:22:30 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:322)
ctfd  | [2023-07-10 18:22:30 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:323)
ctfd  | [2023-07-10 18:22:30 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:324)
ctfd  | [2023-07-10 18:22:30 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:325)
ctfd  | [2023-07-10 18:23:01 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:386)
ctfd  | [2023-07-10 18:23:01 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:387)
ctfd  | [2023-07-10 18:23:01 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:388)
ctfd  | [2023-07-10 18:23:01 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:389)
ctfd  | [2023-07-10 18:23:31 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:444)
ctfd  | [2023-07-10 18:23:31 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:445)
ctfd  | [2023-07-10 18:23:31 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:446)
ctfd  | [2023-07-10 18:23:31 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:447)
ctfd  | [2023-07-10 18:24:02 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:495)
ctfd  | [2023-07-10 18:24:02 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:496)
ctfd  | [2023-07-10 18:24:02 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:497)
ctfd  | [2023-07-10 18:24:02 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:498)
ctfd  | [2023-07-10 18:24:32 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:546)
ctfd  | [2023-07-10 18:24:32 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:547)
ctfd  | [2023-07-10 18:24:32 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:548)
ctfd  | [2023-07-10 18:25:03 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:599)
ctfd  | [2023-07-10 18:25:03 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:600)
ctfd  | [2023-07-10 18:25:33 +0000] [7] [CRITICAL] WORKER TIMEOUT (pid:641)

Fixed by 9433d98.


Missing initContainer in PodSpecProps

Kubernetes has it https://kubernetes.io/docs/concepts/workloads/pods/init-containers/ but not cdk8s
When try to use it

/**
 * (experimental) Properties of a `PodSpec`.
 *
 * @experimental
 */
export interface PodSpecProps {
    /**
     * (experimental) List of containers belonging to the pod.
     *
     * Containers cannot currently be
     * added or removed. There must be at least one container in a Pod.
     *
     * You can add additionnal containers using `podSpec.addContainer()`
     *
     * @default - No containers. Note that a pod spec must include at least one container.
     * @experimental
     */
    readonly containers?: ContainerProps[];
    /**
     * (experimental) List of volumes that can be mounted by containers belonging to the pod.
     *
     * You can also add volumes later using `podSpec.addVolume()`
     *
     * @default - No volumes.
     * @see https://kubernetes.io/docs/concepts/storage/volumes
     * @experimental
     */
    readonly volumes?: Volume[];
    /**
     * (experimental) Restart policy for all containers within the pod.
     *
     * @default RestartPolicy.ALWAYS
     * @see https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/#restart-policy
     * @experimental
     */
    readonly restartPolicy?: RestartPolicy;
    /**
     * (experimental) A service account provides an identity for processes that run in a Pod.
     *
     * When you (a human) access the cluster (for example, using kubectl), you are
     * authenticated by the apiserver as a particular User Account (currently this
     * is usually admin, unless your cluster administrator has customized your
     * cluster). Processes in containers inside pods can also contact the
     * apiserver. When they do, they are authenticated as a particular Service
     * Account (for example, default).
     *
     * @default - No service account.
     * @see https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/
     * @experimental
     */
    readonly serviceAccount?: IServiceAccount;
}

keep this open plz


data_for not returning expected

using data_magic (0.21)
I'm observing that tag with value of 042 is not returned as expected.
given a default.yml
test_1:
  code: 042

When I run this
class MyPage
    include DataMagic
end

page = MyPage.new
puts page.data_for :test_1

Expected
{"code"=>042}

Actual
{"code"=>34}

That is correct. A number beginning with a 0 is an octal and octal 042 is
decimal 34.

Michael Hansen
On Mar 24, 2016 4:12 PM, "niartseoj"<EMAIL_ADDRESS>wrote:

using data_magic (0.21)
I'm observing that tag with value of 042 is not returned as expected.
given a default.yml
test_1:
code: 042
When I run this
class MyPage
include DataMagic
end
page = MyPage.new
puts page.data_for :test_1
Expected
{"code"=>042}
Actual
{"code"=>34}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/cheezy/data_magic/issues/22

Try putting quotes around the value and it will be returned as a string.

On Mar 24, 2016, at 4:11 PM, niartseoj<EMAIL_ADDRESS>wrote:
using data_magic (0.21)
I'm observing that tag with value of 042 is not returned as expected.
given a default.yml
test_1:
code: 042
When I run this
class MyPage
include DataMagic
end
page = MyPage.new
puts page.data_for :test_1
Expected
{"code"=>042}
Actual
{"code"=>34}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/cheezy/data_magic/issues/22

Closing based on knowledge that quotes '042' returns as string.


stm32F303: opamp?

Dear Mr. Pařez,
are there plans to use the internal op amps of the stm32F303? I greped superficially through the sources and I'm not sure that they are in use. Perhaps it could improve input impedance.
Kind regards, Jochen

Oh just looked at the characteristics of an TI OPA2340 I've used earlier: 10^13 Ohm, 6 pF input impedance. Perhaps not comparable to the stm32F303 internal opamp.

R_AIN_max is 0.82 kΩ @ 72 MHz, 4.5 sampling cycles, 8 bit ADC ≙ 5.5 MSPS.
The internal opamp has R_LOAD ≥ 4 kΩ
→ does not match.


⚠ AVS (kty) has degraded performance

In 6dac19b, AVS (kty) (https://avus.com.br) experienced degraded performance:

HTTP code: 200
Response time: 8040 ms

Resolved: AVS (kty) performance has improved in 9f173ac.


Implement the @FileUpload decorator

Is there an existing proposal similar to this?

[X] I have searched the existing proposals

What are you proposing?
I propose implementing the @FileUpload decorator in ExpressoTS. This decorator would simplify the process of uploading files by reducing the amount of boilerplate code required to configure file uploads. It will streamline the process, making it easier for developers to handle file uploads in their projects.
Is there any specific group of users that will benefit from this?
Developers using ExpressoTS who frequently work with file uploads will benefit from this feature.
What problems are you trying to solve?
The current process for handling file uploads in ExpressoTS can be cumbersome, requiring developers to manually configure multer in either the global configuration or within each controller. This proposal aims to reduce redundancy and simplify the code by introducing a decorator that encapsulates this functionality, making it easier and quicker to set up file uploads.
Do you have any references or examples that can illustrate your idea?
@controller('/upload')
export class UploadController {

  @Post('/')
  @FileUpload('avatar', { dest: 'uploads/' })
  public uploadFile(req: Request, res: Response) { }

What type of idea is this?
Innovation: No similar idea exists

All the use cases are defined in the official multer doc: https://www.npmjs.com/package/multer


Add hover and focus events to QuesoField component

This pull request adds the hover and focus events to the QuesoField component. It also includes the necessary changes to emit the "update:active" and "update:hover" events when the corresponding states change.
This is necessary for some input based animations in some projects.

Added in the newest PR #132


TEA Human Resources and Finance DataImport templates.

*Adding TEA's DataImport templates (HumanResources and Finance).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@brian-pazos you need to sign the contributor license agreement as stated about, one you sign I will approve and merge the changes


Configure ember-cli-deploy for gh-pages

Fixes #2

ember deploy production will build and deploy the site to gh-pages branch

Living site https://jubar.github.io/ember-montevideo-app/

Thanks @marpo60 👏


Feature/dynamic apply button

Top button now says 'Go to dashboard' if you're logged in.
Also makes all those buttons go to the /apply/form route so that they won't end up at the dashboard unnecessarily.

LGTM


Upgrade to Python 3

With Python 2 being retired next year, it could be nice to upgrade the code to Python 3.
It will mainly be changes on the print function and unicode.

Hey @Luk3M,
I will be looking into this soon

Should be done now. (closing) feel free to reopen if I've missed things.
#37


.route file and tillable option

We need to make sure the content of the .route file is consistent with the routing while the tillable option is activated.

Now, the .route file will output the routing track ids correctly when tileable routing is enabled.
This improvement has been made on the VPR8 + OpenFPGA version.


Is it possible for RtAudio.js to support ASIO?

Hello, I think initially avoided it because I remember having hard time making it work, also it's not too common. But might add it, still need it?

Thank you. I don't need it for the time being.


Error: Cannot find module 'require("./server/main")'

I'm submitting a...

[x ] Bug report
[ ] Feature request
[ ] Documentation issue or request

In package

[ ] @ng-toolkit/init
[ ] @ng-toolkit/serverless
[x ] @ng-toolkit/universal
[ ] @ng-toolkit/pwa
[ ] @ng-toolkit/firebug

Current behavior

npm run server fails with an error
internal/modules/cjs/loader.js:583
throw err;
^
Error: Cannot find module 'require("./server/main")'
Expected behavior

should start the server.
Minimal reproduction of the problem with instructions
ng new myproject
cd myproject
npm install @angular/platform-server --save
npm install @nguniversal/express-engine --save
ng add @ng-toolkit/universal
npm run build:prod
npm run server
Example repository
https://github.com/openbridge/angular_ng_toolkit
What is the motivation / use case for changing the behavior?

We'd like the library to work.
Environment

Angular version: X.Y.Z
@angular-devkit/architect                  0.803.19
@angular-devkit/build-angular              0.803.19
@angular-devkit/build-optimizer            0.803.19
@angular-devkit/build-webpack              0.803.19
@angular-devkit/core                       8.3.19
@angular-devkit/schematics                 8.3.19
@angular/cli                               8.3.19
@ngtools/webpack                           8.3.19
@nguniversal/common                        8.1.0
@nguniversal/express-engine                8.1.1
@nguniversal/module-map-ngfactory-loader   8.1.1
@schematics/angular                        8.3.19
@schematics/update                         0.803.19
rxjs                                       6.4.0
typescript                                 3.5.3
webpack                                    4.39.2
- Node version: v12.13.1
- Platform: MacOS, and AlpineLinux via Docker container

Hello @thenetimp
Did you ran npm run build:ssr before running npm run server?

I hadn't run npm run build:ssr, but just did and got the same error.  I'll pull your repo and try it.

Yeah, I am pretty aware of some bugs on latest version due to Angular/Universal schematic changes, which I have addressed on my sample repo and expect to be applied on ng-toolkit/universal schematic too per #724.
Cheers!

Your sample repo is working.

Hello,
I have the same issue.
@KingDarBoja  i tried your repo and it works, can you tell please what are the changes you made?

@hymenoby Sure!
As the latest versions of ng-toolkit/universal (8.0.0 and above) were released to fix issues along with compatibility with Angular 8, the internal schematic @nguniversal/express-engine  got updated as well but unfortunately, I didn't check the changes being introduced on it, as I was too focused on fixing code generation and deployment steps.
So in order to ensure proper functionality, I read the entire documentation, made some sample repos using a fresh angular project and applied the same universal packages to spot differences between the old and new version.
Most of these changes were made on the package.json scripts, the webpack.config file (which I also changed to Typescript because why not?), the server and prerender files. Ah, don't forget to check out the main.server file because few import statements were moved from the server file into that, to avoid duplication of Angular instances on production.
Those files were once again updated to be ready for upcoming v9 of @nguniversal/express-engine, with few changes that I am too lazy to explain too but should be spotted on Angular Universal release commits.
The breaking change that caused the error message  Cannot find module 'require("./server/main")' was the externals option on the webpack.config file, which I have no idea why it was included. Also using that breaks the serverless deployment so better to comment it out for the meantime.
Cheers!

I'm having the same error in Angular 8. I run these on an existing Angular 8 App:
npm run ng add @ng-toolkit/universal
npm run build:prod
with no errors, but when I run this:
npm run server
I get this error:
Error: Cannot find module 'require("./server/main")'
is there any fix for that.
I don't want to go through all the generated files. It might be much easier to use Angular Universal instead.
Thanks.

I had the same error, to solve it i had to go back to basic angular-universal;
just a few tips i had to use :

In server.ts after the line
const DIST_FOLDER = join(process.cwd(), 'dist/browser');
i added

const domino = require('domino');
const fs = require('fs');
const path = require('path');
const template = fs.readFileSync(path.join(DIST_FOLDER, 'index.html')).toString();
const win = domino.createWindow(template);

global['window'] = win;
global['document'] = win.document;
global['DOMTokenList'] = win.DOMTokenList;
global['Node'] = win.Node;
global['Text'] = win.Text;
global['HTMLElement'] = win.HTMLElement;
global['navigator'] = win.navigator;
global['MutationObserver'] = getMockMutationObserver();

since i am using the window object in the code

had to inject the document object every time i use it like this
constructor(@Inject(DOCUMENT) document: Document) {}
and use it instead of just using the document object

had to exclude all the code that required the brower and that i know would not work on SSR :
First inject the platformID:
constructor(@Inject(PLATFORM_ID) private platformId: object) {}

Then in the code exclude all the code that would not work on SSR :
  if (isPlatformBrowser(this.platformId)) {
      if (_translateService.getBrowserLang()) {
        _translateService.use(_translateService.getBrowserLang());
      }
    } else {
      _translateService.use('en');
    }

In this case since i can't get the browser lang i excluded the code to get the browser lang and set a default lang on SSR.
Hope this helps

thanks, actually I fixed by commenting out the externals option (which is './dist/server/main': 'require("./server/main")') on the webpack.server.config.js file.
Now, I'm running into another error:
Warning: Flex Layout loaded on the server without FlexLayoutServerModule
ERROR TypeError: Cannot read property 'match' of undefined
any idea?

Thanks, everyone.  With your help I was able to get a fresh angular application (no additional components) running in on AWS lambda.
The fix for me was to remove the "externals option" from the webpack file.  I then had an issue with serverless part of ng-toolkit where I simply updated the webpack-cli module based on the version in the comment of this issue here.
https://github.com/maciejtreder/ng-toolkit/issues/483#issuecomment-433185899
Now I can systematically take our existing application and port it into this new repository and hope nothing breaks in the process.  You all rock thanks for the help.
PS I won't close the ticket, I'll leave that up to the decision of the maintainer.

I am also having same issue.   Kindly help.  Thanks in advance.

I am also having same issue.
Commenting out in webpack.config solve this issue but getting another issue.

Hi mate, can you post your issue here? Maybe we can help you to find out what's going on.

npm run build:prod
successfully built but
error while running npm run server
node local.js
internal/modules/cjs/loader.js:716
throw err;
^
Error: Cannot find module 'require("./server/main")'
Its really frustrating and showstopper for me.

Commenting out in webpack.config solve this issue but getting below error.
Error: The selector "app-root" did not match any elements
at platform_server_DefaultServerRenderer2.selectRootElement

That sounds like a bug in your code, I had something similar with a variable didn't have a function attached to it.  Look at that line in server.js and it should tell you what component the error is in and you should be able to debug it.

That sounds like a bug in your code, I had something similar with a variable didn't have a function attached to it. Look at that line in server.js and it should tell you what component the error is in and you should be able to debug it.

Okay! I'll check it. Thanks for your help and support.

That sounds like a bug in your code, I had something similar with a variable didn't have a function attached to it. Look at that line in server.js and it should tell you what component the error is in and you should be able to debug it.

I debugged it and now it is working fine :)  Thanks for your help and support.  :)

I am getting following error message  after deploying Angular Universal App in IIS
**iisnode encountered an error when processing the request.
HRESULT: 0x2
HTTP status: 500
HTTP subStatus: 1002
HTTP reason: Internal Server Error
You are receiving this HTTP 200 response because system.webServer/iisnode/@devErrorsEnabled configuration setting is 'true'.
In addition to the log of stdout and stderr of the node.exe process, consider using debugging and ETW traces to further diagnose the problem.
The node.exe process has not written any information to stderr or iisnode was unable to capture this information. Frequent reason is that the iisnode module is unable to create a log file to capture stdout and stderr output from node.exe. Please check that the identity of the IIS application pool running the node.js application has read and write access permissions to the directory on the server where the node.js application is located. Alternatively you can disable logging by setting system.webServer/iisnode/@loggingEnabled element of web.config to 'false'.**
Kindly help.

Hi @atul2210,
I do think it seems to be an issue related to the Angular Universal package but still I lack the knowledge on that service (IIS) so you should try to look for some help on some StackOverflow post.
I found a similar one already that could help you: Deploy Angular Universal to IIS.
Also, there is an issue on the Angular Universal repo too! Hosting on IIS.

For the IIS problem, you really should start your own support ticket instead of piggybacking off this one.

everything worked fine with me. I got rid of the error by correcting code in the 'server.js' file.
module.exports = require("require(\"./server/main\")");
to
module.exports = require('./server/main');
i used 'ng-toolkit/universal' and after running 'npm run server' i got same error.

Haha great idea @ankit5577 .
Its working with that change.
But why nobody fixes this issue? 😥

@patrickbussmann It's a compiler default behavior I guess.
as require("require(\"./server/main\")"); contains 2 double-quotes,
which will make ./server/main a non-string as it will be out of double-quotes.
and As you know backslash is an escape character. so it tries to remove quotes within predefined quotes.
"\"X\"" = "X".
moreover, I believe this uses angular-cli as 'ng' command is used for bundling which technically makes it right as in typescript, libraries are imported using direct paths.
but yeah I got your point regarding require() issue.
rest, let's see what they do. :/

Happening to me as well

everything worked fine with me. I got rid of the error by correcting code in the 'server.js' file.
module.exports = require("require(\"./server/main\")");
to
module.exports = require('./server/main');
i used 'ng-toolkit/universal' and after running 'npm run server' i got same error.

@nicholas-davis


Implement GROUP BY

I don't know when to quit.

This PR (dependent on #7695 and #7696) implements GROUP BY, this time with only about 15 or 16 changed files less changed files, although still a good number due to test changes and a very focused approach to getting it done without a lot of byproduct.

@tuespetre,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@tuespetre - Given the 2 dependent PRs are merged in. Can you update/rebase this PR?

@smitpatel sure can, but heads up, there are some test cases and scenarios I have thought of that still need to be addressed here. I wouldn't bother spending your time reviewing it or anything yet. Sorry ☹

Just change the prefix the title with [WIP] and update it once you are done working out. Or given there has not been any discussion here. You can close this and open a new PR when work is ready for review.

OK. Thanks

@smitpatel I am now going to state that this PR will depend on #7767 and #7771

:up::date: finished work on tests: split out separate AsyncGroupByQuery____ and GroupByQuery___ test classes, and added several new and complex test cases for grouping, plus made them actually work :trollface:

@tuespetre - The query you posted in PR #7771 , certainly it is a complex query for group join.
What is expected Sql for that?
In general what are the blockers for group by translation in current stack. (not just PR #7771 but other issues you faced which prohibits group by translations)

@smitpatel here is the query I was basing that on:
https://github.com/tuespetre/EntityFramework/blob/d14681e89b2b2cb95840abc460186167ddcb9a85/test/EFCore.SqlServer.FunctionalTests/GroupByQuerySqlServerTest.cs#L469
I don't see anything else blocking generally useful GROUP BY translations for now. One scenario I have not covered in this PR is aggregate subqueries that include WhereClauses (see the GroupBy_Sum_Where test in that same file for an example.) I viewed that as an additional enhancement beyond the basic ability to group and aggregate.

:up::date: ➕  ♻🅱

:up::date: ➕  ♻🅱 again

@tuespetre - Please remove any trailing code changes from any other PRs which are not merged. Also for reviewing purpose create 2 different commits. 2nd one containing only test changes (there are 2200 lines of addition just from tests 😄 nice.) And let us know when it is ready for review.

We are closing this pull request due to lack of response. If you decide to work on the issue again then feel free to reopen the PR and push more commits to it. We would be happy to review it.
Thank you helping us out in improving EF Core.

@tuespetre - I have taken tests from this PR. Thanks.


Add Platform tab to STO What's Supported page SMP DOC-3664

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Updates were implemented here: https://github.com/harness/developer-hub/pull/3157


docs: Notifier app link broken

When trying to access to the following link I get a 404. I guess users not owning the app are not allowed to see the page, right?

https://github.com/settings/installations/12807923

Maybe this link can be used? https://github.com/apps/octokit-release-notifier

I guess users not owning the app are not allowed to see the page, right

right 👍🏼 thanks for opening the issue

Maybe this link can be used? https://github.com/apps/octokit-release-notifier

could you send a PR and update the URL to this: https://github.com/apps/release-notifier-release-notifier

:tada: This issue has been resolved in version 1.2.11 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Spoiler Images

I can't seem to spoiler an image when using the theme
Screenshots

no spoiler button

oops just forgot a "&", thx for reporting the issue, should be fixed with a refresh :3


File Manager: Does not delete marked files not on current page

Hi there!
Thanks for the awesome theme!
I discovered a small bug (I hope it is not a duplicate - did a search but could not locate one, )though:
I have a folder with many files, about 15 pages
I wanted to delete some of the files
I found 3 that were to be deleted on page 2
Skimmed through the rest of the pages until I reached the last one and since I did not find any other useless files, while still in page 15,  I clicked on the delete icon, but nothing happened
On the top left corner I see that 3 files are already selected

As mentioned, I clicked the delete icon while on page 15 and those files were from page 2
If I go back on page 2 i see them still selected.
I click on delete icon and all work as expected.
If I go to page 15 and select at least one file from that page and click on the delete icon, the window does popup and asks for the confirmation for all the selected items

What is your server-side OS and its version?
Ubuntu server 12.04 LTS, webmin 1.831, authentic 18.32
What is your client-side OS, browser and its versions?
Windows 8.1 64bit, Chrome Version 56.0.2924.87
What are the steps to reproduce the issue
Go to a folder with many files separated in pages
Go to page 1 - select a file
Go to page 2 (notice that file is mentioned as selected)
Click delete -->nothing happens
Do back to page 1 - click delete --> all ok
don't delete anything, go again to page 2, select at least one file there- click delete--> popup appears normally and includes all selected files

I hope I explained well
Have a nice day and thanks again!

Hi,
Thank you for reporting this issue.
I have a question, does the file actually gets deleted? Did you try after failure deletion, to check if the file was deleted. It looks more like an interface problem.
It's ok, you can leave it as it is, if you don't have time to double check. I will manage it from here.


Rename types.py to apitypes.py to avoid lint errors with standard package named types

This is a breaking change for any module importing the types module from the api library.
Imports must be changed to apitypes instead.

changed with release 2.1.1


[WIP] Move detailed information to wiki pages

I've moved some of the detailed information to GitHub wiki pages.
@linusg @vn-ki Feel free to add any other information you guys find necessary to wiki pages. Also let me know if there is anything else we could remove from README. :)

I think we should remove the usage (--help) dump completely from the README, that's something the user can do on their own. Also the most important flags are explained in the README anyway.

I am not sure about this one. I personally like projects who display the help usage in README to give people a rough idea of what all the tool is capable of.

Maybe shorten the list of features and display it in a way so it takes up less space?

Yep, gotta look in this too.

I wouldn't say "Specify the Target Directory" should be in "Advanced options" but maybe stay in the README...

This seems a better idea!

What about we move sections on downloading by track name, spotify link, playlist, etc. to wiki and keep help usage in README?

FAQ + linking from the README sounds nice! Keeping the help output and moving all the detailed explanations to the Wiki is fine as long as the help output will not grow and grow and grow... 😉
When doing so I would say just having a "getting started" section with e.g. instructions on how to download a single song by name and a playlist by URL (features I personally use the most) should be sufficient.

What about we move sections on downloading by track name, spotify link, playlist, etc. to wiki and keep help usage in README?

I think these are better off being in the readme. Most people don't open wiki unless they really need to.

Also not sure about this but maybe we could have an FAQ section in wiki (for stuff like how to specify the target directory, etc) and link to it from README.

I'm all for an FAQ section.
I haven't taken a look at the wiki yet. I'm a bit busy right now.

I think these are better off being in the readme. Most people don't open wiki unless they really need to.

I think we could specify the most basic -s option in README and then provide a link to using other options (-f, -p, etc) in wiki.

(Also I want to thank you both for being so responsive :)

I am not sure about this one. I personally like projects who display the help usage in README to give people a rough idea of what all the tool is capable of.

We should probably reconsider this: The output of python3 spotdl.py --help is currently almost 70 lines, which indeed only giving a "rough" overview to first-time users.
I certainly understand what you mean, a raw -h/--help dump can be better than a ton of text, but in our case it seems overwhelming.
Even the first few lines. As a beginner, but avid Linux user/programmer I would love somthing like:
(-s SONG | -l LIST | -p PLAYLIST | -b ALBUM | -u USERNAME | -V)
[-f FOLDER] [--overwrite {force,prompt,skip}] [-o OUTPUT_EXT][-c CONFIG]

This is totally self explaining. The whole thing probably not:
 (-s SONG | -l LIST | -p PLAYLIST | -b ALBUM | -u USERNAME | -V)
[-m] [-nm] [-a] [-f FOLDER] [--overwrite {force,prompt,skip}]
[-i {.webm,.m4a}] [-o OUTPUT_EXT] [-ff FILE_FORMAT]
[-sf SEARCH_FORMAT] [-dm] [-d] [-mo] [-ns]
[-ll {INFO,WARNING,ERROR,DEBUG}] [-c CONFIG]

Because "what the hell is -m, -nm, -i, -dm, -d, -mo and -ns"? (I know, it's explained later.)
So does the Python argparse module offer something like a short help (-h/--help) and an extended/full help (sth. like --help-complete)? I may be wrong, but I think some GNU tools offer this...

Okay, I moved the help usage to wiki and added an FAQ section. The README looks super cool now. We could merge this right away unless you guys have any other suggestions!

@vn-ki Good point. I added some basic information in 5ee65826bd89f5cbce0afe396285f13c4248ce2c.

Okay then, there we go!


feat: add ability to provision oathkeeper rules

Motivation
This allows Turing, XP and Merlin services to integrate with Oathkeeper, by providing an option to generate Oathkeeper Rule CRDs when enabled.
Related PRs: https://github.com/caraml-dev/helm-charts/pull/318
Modification
The Oathkeeper rules is by default disabled, so this should be compatible with old app versions that is still using Keto 0.5.
Checklist

[x] Chart version bumped
[x] README.md updated

This changeset becomes a cross-cutting concern. we solved a similar task in routes chart where we generate VS and gateways. wdyt, if we move these rules to the routes chart and make app charts independent from authorization configuration?

This changeset becomes a cross-cutting concern. we solved a similar task in routes chart where we generate VS and gateways. wdyt, if we move these rules to the routes chart and make app charts independent from authorization configuration?

That works, I agree that if we want to generate the prefix automatically, the current approach will not be feasible. Oathkeeper allow us to define the rule configuration directly as well, so while I am preparing another PR that move these rules to route chart, we will configure the rules directly as opposed to using CRDs.
How about Oathkeeper itself? Do you think it should be part of the CaraML helm chart, or completely independent?

@khorshuheng for the umbrealla chart, we keep a list of dependencies so we can run tests (= install and uninstall chart) in the mock environment. my opinion: we only need to keep oathkeeper on the dependencies list as long as it is required for testing caraml in the mock env.


[Feature-Request] Log Redirect

Start with the why:
Adding the feature to redirect the log output in depthai would be useful for both developers and users of the library. It would provide greater flexibility in handling and managing log information, allowing users to customize the log output based on their specific needs. By redirecting the log output, developers can choose different destinations for logs, such as storing them in a file for later analysis or sending them to a remote logging service. This feature enhances the debugging and troubleshooting capabilities of depthai and improves the overall user experience.
Move to the what:
The feature that I would like to see added to depthai is the ability to redirect the log output. Currently, depthai generates log messages during its operation, which are typically printed to the console by default. However, it would be beneficial to have the option to redirect these log messages to alternative destinations, such as a file, GUI interface, or remote logging service.

@richard-xx
this is already possible by:

Setting DEPTHAI_LEVEL env variable
Programatically using device.setLogOutputLevel & device.setLogLevel and device.addLogCallback


A question about the prefix of only SFT

hi, thank your excellent job!
I would like to ask if the template of only SFT in your paper contains prefix, such as User/Assistant.
The code is in this [url] (https://github.com/imoneoi/openchat/blob/f528c2716f644777c5cbfa6c73aa167de50250ce/ochat/config/conversation_template.py#L91C1-L92C1)

Yes, it contains the role prefix. However, conditions such as GPT3 / GPT4 are removed. The resulting role prefixes are User:  and Assistant: