unable to bridge -  'bool' object is not subscriptable

What happened?
I am testing out Gefyra as an alternative to telepresence but hit an error in my initial testing.
I have an EKS cluster already configured and heavily used.
I can install gefyra using homebrew without issue.
I can start gefyra without issue after opening UDP port 31820
gefyra up --endpoint <my node IP>:31820
[INFO] Installing Gefyra Operator
[INFO] Created network 'gefyra' (de529461f820)
[INFO] Container image "quay.io/gefyra/operator:0.13.4" already present on machine
[INFO] Pulling image "quay.io/gefyra/stowaway:0.13.4"
[INFO] Successfully pulled image "quay.io/gefyra/stowaway:0.13.4" in 6.651035992s
[INFO] Operator became ready in 15.2111 seconds
[INFO] Deploying Cargo (network sidecar) with IP <IP_ADDRESS>

I can start a local container using gefyra via:
[INFO] Container image 'myapp:latest' started with name 'myLocalContainer' in Kubernetes namespace 'test-namespace' (from --namespace argument)
/docker-entrypoint.sh: /docker-entrypoint.d/ is not empty, will attempt to perform configuration
/docker-entrypoint.sh: Looking for shell scripts in /docker-entrypoint.d/
/docker-entrypoint.sh: Launching /docker-entrypoint.d/10-listen-on-ipv6-by-default.sh
10-listen-on-ipv6-by-default.sh: info: Getting the checksum of /etc/nginx/conf.d/default.conf
10-listen-on-ipv6-by-default.sh: info: /etc/nginx/conf.d/default.conf differs from the packaged version
/docker-entrypoint.sh: Launching /docker-entrypoint.d/20-envsubst-on-templates.sh
/docker-entrypoint.sh: Launching /docker-entrypoint.d/30-tune-worker-processes.sh
/docker-entrypoint.sh: Configuration complete; ready for start up
2022/12/09 22:59:45 [notice] 17095#17095: using the "epoll" event method
2022/12/09 22:59:45 [notice] 17095#17095: nginx/1.21.3
2022/12/09 22:59:45 [notice] 17095#17095: built by gcc 8.3.0 (Debian 8.3.0-6)
2022/12/09 22:59:45 [notice] 17095#17095: OS: Linux 5.10.76-linuxkit
2022/12/09 22:59:45 [notice] 17095#17095: getrlimit(RLIMIT_NOFILE): 1048576:1048576
2022/12/09 22:59:45 [notice] 17095#17095: start worker processes
2022/12/09 22:59:45 [notice] 17095#17095: start worker process 17176
2022/12/09 22:59:45 [notice] 17095#17095: start worker process 17177
2022/12/09 22:59:45 [notice] 17095#17095: start worker process 17178
2022/12/09 22:59:45 [notice] 17095#17095: start worker process 17179
2022/12/09 22:59:45 [notice] 17095#17095: start worker process 17180
2022/12/09 22:59:45 [notice] 17095#17095: start worker process 17181
2022/12/09 22:59:45 [notice] 17095#17095: start worker process 17182
2022/12/09 22:59:45 [notice] 17095#17095: start worker process 17183

I can then docker exec into the running container using gefyra in a separate shell:
docker exec -it myLocalContainer bash
However, when I try to intercept traffic with a bridge via:
gefyra bridge -N myLocalContainer -n test-namespace --port 80:8000 --target deployment/test-deployment/myapp-container
It fails with:
[INFO] Creating bridge for Pod test-namespace-myapp-659c598bbd-nptk5
[CRITICAL] There was an error running Gefyra: 'bool' object is not subscriptable

What did you expect to happen?
I expected a bridge to start and be able to forward traffic to my local container.
How can we reproduce it (as minimally and precisely as possible)?
These are the steps I ran^
What Kubernetes setup are you working with?

$ kubectl version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.4", GitCommit:"872a965c6c6526caa949f0c6ac028ef7aff3fb78", GitTreeState:"clean", BuildDate:"2022-11-09T13:28:30Z", GoVersion:"go1.19.3", Compiler:"gc", Platform:"darwin/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"23+", GitVersion:"v1.23.13-eks-fb459a0", GitCommit:"55bd5d5cb7d32bc35e4e050f536181196fb8c6f7", GitTreeState:"clean", BuildDate:"2022-10-24T20:35:40Z", GoVersion:"go1.17.13", Compiler:"gc", Platform:"linux/amd64"}
WARNING: version difference between client (1.25) and server (1.23) exceeds the supported minor version skew of +/-1

OS version

macOS Big Sur Version 11.6.8

Anything else we need to know?
I was just trying to go through https://gefyra.dev/getting-started/aws-eks/#running-gefyra
any help would be appreciated!

The reason you're seeing this error message is probably this line:
https://github.com/gefyrahq/gefyra/blob/main/client/gefyra/local/bridge.py#L28
The calling function expects an InterceptRequest instead of a boolean. However, I am afraid that this will not fix your problem, since in this case an exception occurs, which we are probably not able to handle. I don't really know what can happen here except 409 Conflict - which is already handled though.
Could you provide the logs of the operator?
kubectl logs -n gefyra <gefyra-operator-pod-name>

I suspect there is something wrong with the CRD of the InterceptRequest.

Thank you for the reply @SteinRobert.
After getting the error:
[CRITICAL] There was an error running Gefyra: 'bool' object is not subscriptable
I see the operator logs as:
[2022-12-12 17:16:47,184] gefyra               [INFO    ] Gefyra Operator startup
[2022-12-12 17:16:47,185] gefyra               [INFO    ] Loaded in-cluster config
[2022-12-12 17:16:47,220] kopf.activities.star [INFO    ] Ensuring Gefyra components with the following configuration: {'NAMESPACE': 'gefyra', 'STOWAWAY_IMAGE': 'quay.io/gefyra/stowaway', 'STOWAWAY_IMAGE_PULLPOLICY': 'Always', 'STOWAWAY_TAG': '0.13.4', 'WIREGUARD_EXT_POR
[2022-12-12 17:16:47,252] kopf.activities.star [INFO    ] Gefyra CRD InterceptRequest created
[2022-12-12 17:16:47,270] kopf.activities.star [INFO    ] Gefyra Stowaway Serviceaccount created
[2022-12-12 17:16:47,275] kopf.activities.star [INFO    ] Stowaway proxy route configmap created
[2022-12-12 17:16:47,309] kopf.activities.star [INFO    ] Stowaway deployment created
[2022-12-12 17:16:47,334] kopf.activities.star [INFO    ] Stowaway nodeport service created
[2022-12-12 17:16:47,343] kopf.activities.star [INFO    ] Stowaway rsync service created
[2022-12-12 17:16:47,358] gefyra.stowaway      [INFO    ] Waiting for Stowaway to become ready
[2022-12-12 17:16:48,369] gefyra.stowaway      [INFO    ] Waiting for Stowaway to become ready
[2022-12-12 17:16:49,378] gefyra.stowaway      [INFO    ] Waiting for Stowaway to become ready
[2022-12-12 17:16:50,387] gefyra.stowaway      [INFO    ] Waiting for Stowaway to become ready
[2022-12-12 17:16:51,398] gefyra.stowaway      [INFO    ] Waiting for Stowaway to become ready
[2022-12-12 17:16:52,407] gefyra.stowaway      [INFO    ] Waiting for Stowaway to become ready
[2022-12-12 17:16:53,417] gefyra.stowaway      [INFO    ] Waiting for Stowaway to become ready
[2022-12-12 17:16:54,427] gefyra.stowaway      [INFO    ] Waiting for Stowaway to become ready
[2022-12-12 17:16:55,436] gefyra.stowaway      [INFO    ] Waiting for Stowaway to become ready
[2022-12-12 17:16:56,472] gefyra.stowaway      [INFO    ] Stowaway ready: gefyra-stowaway-88967bcb5-xhmjq
[2022-12-12 17:16:56,487] gefyra.stowaway      [INFO    ] Copy Peer1 connection details from Pod gefyra-stowaway-88967bcb5-xhmjq:/config/peer1/peer1.conf
[2022-12-12 17:16:56,591] gefyra.stowaway      [INFO    ] Creating Cargo connection secret
[2022-12-12 17:16:56,598] gefyra.stowaway      [INFO    ] Cargo connection secret created
[2022-12-12 17:16:56,617] kopf.activities.star [INFO    ] Gefyra components installed/patched
[2022-12-12 17:16:56,617] kopf.activities.star [INFO    ] Activity 'check_gefyra_components' succeeded.
[2022-12-12 17:16:56,618] kopf.activities.star [INFO    ] Activity 'configure' succeeded.
[2022-12-12 17:16:56,619] kopf._core.engines.a [INFO    ] Initial authentication has been initiated.
[2022-12-12 17:16:56,620] kopf.activities.auth [INFO    ] Activity 'login_via_client' succeeded.
[2022-12-12 17:16:56,620] kopf._core.engines.a [INFO    ] Initial authentication has finished.

with no added log lines whne I repeat the (and fail) the bridge command
looking at the stowaway pod, it's logs include a QR code and then:
[custom-init] no custom files found exiting...
[cont-init.d] 99-custom-scripts: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
2022/12/12 17:16:54 [notice] 382#382: using the "epoll" event method
2022/12/12 17:16:54 [notice] 382#382: nginx/1.20.2
2022/12/12 17:16:54 [notice] 382#382: built by gcc 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04)
2022/12/12 17:16:54 [notice] 382#382: OS: Linux 5.4.209-116.367.amzn2.x86_64
2022/12/12 17:16:54 [notice] 382#382: getrlimit(RLIMIT_NOFILE): 1048576:1048576
2022/12/12 17:16:54 [notice] 382#382: start worker processes
2022/12/12 17:16:54 [notice] 382#382: start worker process 410
2022/12/12 17:16:54 [notice] 382#382: start worker process 411
2022/12/12 17:16:54 [notice] 382#382: start worker process 412
2022/12/12 17:16:54 [notice] 382#382: start worker process 413
2022/12/12 17:16:54 [notice] 382#382: start worker process 414
2022/12/12 17:16:54 [notice] 382#382: start worker process 415
2022/12/12 17:16:54 [notice] 382#382: start worker process 416
2022/12/12 17:16:54 [notice] 382#382: start worker process 417
2022/12/12 17:16:54 [notice] 382#382: start worker process 418
2022/12/12 17:16:54 [notice] 382#382: start worker process 419
2022/12/12 17:16:54 [notice] 382#382: start worker process 421
2022/12/12 17:16:54 [notice] 382#382: start worker process 422
2022/12/12 17:16:54 [notice] 382#382: start worker process 423
2022/12/12 17:16:54 [notice] 382#382: start worker process 424
2022/12/12 17:16:54 [notice] 382#382: start worker process 425
2022/12/12 17:16:54 [notice] 382#382: start worker process 426
[#] sysctl -w net.ipv4.ip_forward=1
net.ipv4.ip_forward = 1
[#] ip link add wg0 type wireguard
Error: Unknown device type.
[!] Missing WireGuard kernel module. Falling back to slow userspace implementation.
[#] wireguard-go wg0
┌──────────────────────────────────────────────────────┐
│                                                      │
│   Running wireguard-go is not required because this  │
│   kernel has first class support for WireGuard. For  │
│   information on installing the kernel module,       │
│   please visit:                                      │
│         https://www.wireguard.com/install/           │
│                                                      │
└──────────────────────────────────────────────────────┘
.:53
CoreDNS-1.10.0
linux/amd64, go1.19.1, 596a9f9
[#] wg setconf wg0 /dev/fd/63
[#] ip -4 address add <IP_ADDRESS> dev wg0
[#] ip link set mtu 8921 up dev wg0
[#] ip -4 route add <IP_ADDRESS>/32 dev wg0
[#] ip -4 route add <IP_ADDRESS>/16 dev wg0
[#] iptables -A FORWARD -i wg0 -j ACCEPT; iptables -A FORWARD -o wg0 -j ACCEPT; iptables -t nat -A POSTROUTING -o eth0 -j MASQUERADE

I see no errors in either of these logs.

Could you please try again with the executable of this build?
I made it a bit more verbose.
https://github.com/gefyrahq/gefyra/actions/runs/3678419836

Thanks again @SteinRobert.
I downloaded the binary and replaced the version I had in /usr/local/bin with it. Now when I run the bridge command:
gefyra bridge -N myLocalContainer -n test-namespace --port 80:8000 --target deployment/test-deployment/myapp-container
it says:
```[CRITICAL] There was an error running Gefyra: Cannot bridge pod test-namespace-myapp-659c598bbd-nptk5 since it has a command defined.

Is that a deal breaker?

Hey @grant30mad thank you for trying this build. Something must have changed in your cluster since your previous logs indicate that this wasn't a problem before:
[CRITICAL] There was an error running Gefyra: Cannot bridge pod

happens before
[INFO] Creating bridge for Pod test-namespace-myapp-659c598bbd-nptk5

Running a command is indeed a limitation. We do patch most of the shell commands, however other custom commands are not supported at the moment.
Could you go back to the old cluster state, so that we can investigate the problem you opened the issue for?

Hmm. @SteinRobert  as my app is very reliant on a command, this lack of support is reason enough for me to not use gefyra.
to your comment about something changing, the only thing that changed was the binary being used.
steps I took:

find a node ip - kubectl get nodes -o wide | awk {'print $1" " $2 " " $7'} | column -t
gefyra up command now failed as --endpoint removed - [WARNING] --endpoint/-ehas been removed. Please consider--hostand--port instead.
per new error message, I switched to use --host and --port. Then gefyra up suceeds:

 gefyra up --host <IP_ADDRESS> --port 31820
[INFO] Installing Gefyra Operator
[INFO] Created network 'gefyra' (cb0d0a557c2e)
[INFO] Pulling image "quay.io/gefyra/operator:0.13.4"
[INFO] Successfully pulled image "quay.io/gefyra/operator:0.13.4" in 3.492932236s
[INFO] Pulling image "quay.io/gefyra/stowaway:0.13.4"
[INFO] Successfully pulled image "quay.io/gefyra/stowaway:0.13.4" in 6.376640852s
[INFO] Operator became ready in 20.0732 seconds
[INFO] Deploying Cargo (network sidecar) with IP <IP_ADDRESS>

I built the docker image for myapp - docker build -f Dockerfile . -t myapp
I use gefyra to start the container locally:

gefyra run -i myapp -N myLocalContainer -n test-namespace
[INFO] Container image 'myapp:latest' started with name 'myLocalContainer' in Kubernetes namespace 'test-namespace' (from --namespace argument)
/docker-entrypoint.sh: /docker-entrypoint.d/ is not empty, will attempt to perform configuration
/docker-entrypoint.sh: Looking for shell scripts in /docker-entrypoint.d/
/docker-entrypoint.sh: Launching /docker-entrypoint.d/10-listen-on-ipv6-by-default.sh
10-listen-on-ipv6-by-default.sh: info: Getting the checksum of /etc/nginx/conf.d/default.conf
10-listen-on-ipv6-by-default.sh: info: /etc/nginx/conf.d/default.conf differs from the packaged version
/docker-entrypoint.sh: Launching /docker-entrypoint.d/20-envsubst-on-templates.sh
/docker-entrypoint.sh: Launching /docker-entrypoint.d/30-tune-worker-processes.sh
/docker-entrypoint.sh: Configuration complete; ready for start up
2022/12/15 17:32:03 [notice] 40250#40250: using the "epoll" event method
2022/12/15 17:32:03 [notice] 40250#40250: nginx/1.23.3
2022/12/15 17:32:03 [notice] 40250#40250: built by gcc 10.2.1 20210110 (Debian 10.2.1-6)
2022/12/15 17:32:03 [notice] 40250#40250: OS: Linux 5.10.76-linuxkit
2022/12/15 17:32:03 [notice] 40250#40250: getrlimit(RLIMIT_NOFILE): 1048576:1048576
2022/12/15 17:32:03 [notice] 40250#40250: start worker processes
2022/12/15 17:32:03 [notice] 40250#40250: start worker process 40330
2022/12/15 17:32:03 [notice] 40250#40250: start worker process 40331
2022/12/15 17:32:03 [notice] 40250#40250: start worker process 40332
2022/12/15 17:32:03 [notice] 40250#40250: start worker process 40333
2022/12/15 17:32:03 [notice] 40250#40250: start worker process 40334
2022/12/15 17:32:03 [notice] 40250#40250: start worker process 40335
2022/12/15 17:32:03 [notice] 40250#40250: start worker process 40336
2022/12/15 17:32:03 [notice] 40250#40250: start worker process 40337

I then authenticate a second shell and try the bridge command:

gefyra bridge -N myLocalContainer -n test-namespace --port 80:8000 --target deployment/test-deployment/myapp-container
[CRITICAL] There was an error running Gefyra: Cannot bridge pod cove-5181-cove-659c598bbd-6l4sf since it has a `command` defined.

I just re-did these steps again just now in order to paste them to you. Nothing has changed in the cluster but myapp pods do have a command set.
my deployment spec:
apiVersion: apps/v1
kind: Deployment
spec:
  replicas: 2
  template:
    spec:
      containers:
      - command:
        - /bin/sh
        - -c
        - find /var/www -type f | xargs sed -i -e s@\/\/demo@//$SUBDOMAIN@g && nginx
          -g "daemon off;"
        envFrom:
        - configMapRef:
            name: myapp
        image: ecr-url/myapp
        imagePullPolicy: Always
        ports:
        - containerPort: 8000
          protocol: TCP
        resources:
          limits:
            cpu: "1"
            memory: 1Gi
          requests:
            cpu: 100m
            memory: 200Mi

the cluster state has not changed during this entire gefyra experiment.

Hey @grant30mad thank you very much for your feedback. I'm sorry I forgot to mention the --endpoint parameter changed.
Concerning the command - we do indeed intend to support every common shell command (including you /bin/bash). This is simply a bug in the gefyra client (just checking for sh not /bin/sh. It really helped, that you provided us with your deployment file!
I'd be happy patch that and provide you with another binary tomorrow.

Here's another binary which allows the shell path - so it should work with your command:
https://github.com/gefyrahq/gefyra/actions/runs/3713559062
Could you please check? Thank you so much for you time and effort in helping improve Gefyra!

@grant30mad any chance you got to try this one again with version 1.0.0? It should include the fix for your problem.

Sorry, thought this was included in 1.0.0 - but it is actually included in 1.0.1 (which was just released). Could you check again please @grant30mad ?

I'll try to take a look at this in a few days. (I work with @grant30mad)

@drewfish any chance you found the time to look at this?

Closed due to inactivity.


⚠ Bofrost.fr has degraded performance

In acdcec7, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 998 ms

Resolved: Bofrost.fr performance has improved in ccaf917.


Multiple event sources

Is there any way to add multiple calendars/event sources? I have two different event types and they can overlap. I would like to have them in different colors as well. I'm using week view only, and they would show as two different columns in one day.

@akuljana Yes, it's possible. You still need to merge the two event sources into a single array and pass it to the calendar. But you could generate different css class based on a field of your event, then set different color for each css class. You could take a look at weekviewNormalEventTemplateUrl option in README. It allows you to custom the event view in weekview, you could add some ng-class logic inside your own template.


feat: align-deps command

Describe your changes
Align dependencies command - this will be important when upgrading to a new supported version of React Native.
Type of change

[x] New feature (non-breaking change which adds functionality)

Test Plan

yarn test
cd apps/example
yarn flagship-code align-deps -p 0.73
Verify changes are shown
Verify changes not made to package.json
yarn flagship-code align-deps -p 0.73 --fix
Verify changes are shown
Verify changes made to package.json

Checklist before requesting a review

[x] A self-review of my code has been completed
[x] Tests have been added / updated if required
[x] Documentation has been updated to reflect these changes

Notes

Separate PR will be made for updated documentation

going to move to RN73 feature branch since it ties into the idea of multiple versions and aligning dependencies to react-native version


Specify extra headers to include in runner

When using parametrized tests with Ceedling, I often run into situations something like the following:
//error.h

typedef enum {
    NO_ERROR = 0,
    ERROR_FOO,
    ERROR_BAR,
} error_t;

// module.h

#include "error.h"

error_t module_func(int val);

// test_module.c

#include "unity.h"
#include "module.h"
#include "error.h"

TEST_CASE(0, NO_ERROR)
TEST_CASE(5, ERROR_FOO)
TEST_CASE(8, ERROR_BAR)
void test_module_func(int val, error_t error)
{
    TEST_ASSERT_EQUAL(error, module_func(val));
}

Ceedling will generate a runner something like this:
/* AUTOGENERATED FILE. DO NOT EDIT. */

/*=======Test Runner Used To Run Each Test Below=====*/
#define RUN_TEST_NO_ARGS
#define RUN_TEST(TestFunc, TestLineNum, ...) \
{ \
  Unity.CurrentTestName = #TestFunc "(" #__VA_ARGS__ ")"; \
  Unity.CurrentTestLineNumber = TestLineNum; \
  Unity.NumberOfTests++; \
  if (TEST_PROTECT()) \
  { \
      setUp(); \
      TestFunc(__VA_ARGS__); \
  } \
  if (TEST_PROTECT()) \
  { \
    tearDown(); \
  } \
  UnityConcludeTest(); \
}

/*=======Automagically Detected Files To Include=====*/
#include "unity.h"
#include <setjmp.h>
#include <stdio.h>

int GlobalExpectCount;
int GlobalVerifyOrder;
char* GlobalOrderError;

/*=======External Functions This Runner Calls=====*/
extern void setUp(void);
extern void tearDown(void);
extern void test_module_func(int val, error_t error);

/*=======Test Reset Option=====*/
void resetTest(void);
void resetTest(void)
{
  tearDown();
  setUp();
}

/*=======MAIN=====*/
int main(void)
{
  UnityBegin("test_module.c");
  RUN_TEST(test_module_func, 8, 0, NO_ERROR);
  RUN_TEST(test_module_func, 8, 5, ERROR_FOO);
  RUN_TEST(test_module_func, 8, 8, ERROR_BAR);

  return (UnityEnd());
}

This lacks an inclusion of error.h, and so error_t and associated constants are missing.
I think asking Ceedling to figure out this dependency is a little much. However, providing a mechanism to hint this information (maybe a TEST_HEADER macro, like the TEST_FILE hint) would increase the utility of parameter testing a great deal.
Is there an existing way to cause Ceedling to do this? I haven't found it yet. If not, would you be open to a PR to add this functionality?
Versions and config
Versions
Welcome to Ceedling!
  Ceedling::   0.28.1
  CMock::      2.4.4
  Unity::      2.4.1
  CException:: 1.3.1

Config
---

# Notes:
# Sample project C code is not presently written to produce a release artifact.
# As such, release build options are disabled.
# This sample, therefore, only demonstrates running a collection of unit tests.

:project:
  :use_exceptions: FALSE
  :use_test_preprocessor: TRUE
  :use_auxiliary_dependencies: TRUE
  :build_root: build
#  :release_build: TRUE
  :test_file_prefix: test_
  :which_ceedling: vendor/ceedling
  :default_tasks:
    - test:all

#:release_build:
#  :output: MyApp.out
#  :use_assembly: FALSE

:environment:

:extension:
  :executable: .out

:paths:
  :test:
    - +:test/**
    - -:test/support
  :source:
    - src/**
  :support:
    - test/support

:defines:
  # in order to add common defines:
  #  1) remove the trailing [] from the :common: section
  #  2) add entries to the :common: section (e.g. :test: has TEST defined)
  :commmon: &common_defines []
  :test:
    - *common_defines
    - TEST
  :test_preprocess:
    - *common_defines
    - TEST

:unity:
  :use_param_tests: TRUE

:cmock:
  :mock_prefix: mock_
  :when_no_prototypes: :warn
  :enforce_strict_ordering: TRUE
  :plugins:
    - :ignore
    - :callback
  :treat_as:
    uint8:    HEX8
    uint16:   HEX16
    uint32:   UINT32
    int8:     INT8
    bool:     UINT8

:gcov:
    :html_report_type: basic

#:tools:
# Ceedling defaults to using gcc for compiling, linking, etc.
# As [:tools] is blank, gcc will be used (so long as it's in your system path)
# See documentation to configure a given toolchain for use

# LIBRARIES
# These libraries are automatically injected into the build process. Those specified as
# common will be used in all types of builds. Otherwise, libraries can be injected in just
# tests or releases. These options are MERGED with the options in supplemental yaml files.
:libraries:
  :placement: :end
  :flag: "${1}"  # or "-L ${1}" for example
  :common: &common_libraries []
  :test:
    - *common_libraries
  :release:
    - *common_libraries

:plugins:
  :load_paths:
    - vendor/ceedling/plugins
  :enabled:
    - stdout_pretty_tests_report
    - module_generator
...

The test runners have there own config. Add this to your project.yml:
:test_runner:
  :includes:
    - error.h

By the way, I've personally been fighting parameterized testing in my repo but, keep running into small problems.  So I'm curious how  you solved the issue of TEST_CASE not being defined anywhere. I can hack around it by defining it in the test file, but only AFTER the runner is created. If it is defined before the runner creates the wrong  function calls for the parameterized testing.

is there also the possibility to include files only for specific runners? if i add this option, the header is included in all testrunners...
greets peter

Would also really appreciate the possibility of having an include file for just a specific runner.
Having generic headers like an "error.h" included in the project.yml is fine.
But having headers specific to a single runner defined there is rather ugly when using parameterised testing and wanting to use defines for test parameters.


How to get separate multiple query data?

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Context
I'm tying to get multiple query data. I came up with this before redirect:
Router.push('/acceptance/procurement/RejectedInfo?procurementUUID=0ae56414-0699-4866-898d-be41ab146523?acceptanceUUID=24ecb437-8e52-48f1-82f8-d5d7fb1d22b7');
which mean there is 2 query param:
procurementUUID=0ae56414-0699-4866-898d-be41ab146523 and
acceptanceUUID=24ecb437-8e52-48f1-82f8-d5d7fb1d22b7'
but in my props, both query is merged with procurementUUID

How can I separate procurementUUID and  acceptanceUUID value?
Thanks in advance.

This is how you do it:
Router.push({ pathname: '/acceptance/procurement/RejectedInfo', query: { procurementUUID: '0ae56414-0699-4866-898d-be41ab146523', acceptanceUUID: '24ecb437-8e52-48f1-82f8-d5d7fb1d22b7' } });

It's clearly written in the docs. :p

This is basic HTML query resource manipulation. You learned how to do it by passing a object to Router, but it is worth pointing out that you were close with the string solution also.
'/acceptance/procurement/RejectedInfo?procurementUUID=0ae56414-0699-4866-898d-be41ab146523&acceptanceUUID=24ecb437-8e52-48f1-82f8-d5d7fb1d22b7'
You only have 1 question mark in query params, and that denotes the start of the query. Every property then follows key=value with a & joining multiple into the same query.
domain.com?key1=valueA&key2=valueB&key3=valueC
The object you found in the docs is the preferred way when using the Router for sure. But it's good to know what it is actually building behind the scenes.

Thank you. It's really helpful. :)


Set esbuild target as node12

Currently, target version is not specified so it's esnext.
https://esbuild.github.io/api/#target
As a result, generated file is not valid for node12, the node version which is used by theia editor.
https://github.com/eclipse-theia/theia/blob/02261581de60f5221de6264fab4b011c51e5ab91/package.json#L7
$ wget https://github.com/rust-analyzer/rust-analyzer/releases/download/2021-08-23/rust-analyzer.vsix
$ unzip rust-analyzer.vsix -d ext
$ node --version
v12.22.5
$ node --check ./ext/extension/out/main.js
...
void 0;var qd=_n(),Di=Fr(),aw=kd(),Od=Nr(),cw="Content-Length: ",Id=`\r

SyntaxError: Unexpected token '?'
    at wrapSafe (internal/modules/cjs/loader.js:915:16)
    at checkSyntax (internal/main/check_syntax.js:66:3)
    at internal/main/check_syntax.js:39:3

$ node --version
v14.17.5
$ node --check ext/extension/out/main.js

We just removed a workaround for Node 12 in #10053, as the official policy is to support the latest two VS Code releases.
Is Theia planning to rebase on top of a newer Code version?

@lnicola Thank you for pointing out.
It seems theia is currently updating its node version.
https://github.com/eclipse-theia/theia/pull/9936

Do we really want to commit to supporting Node 12 indefinitely?

Probably the PR will be released on the end of Sep. It may be postponed though, I think we don't need to support indefinitely.
https://github.com/eclipse-theia/theia/milestone/24

Any opinion, @matklad? Doing this would mean reverting #10053.

Yeah, I'd rather put the pressure on upstream to update, rather than maintain compatibility downstream. We do need to say --target=node14 though explicitly!

As a remainder, in this kinds of situation there's always an option of using an older version -- it's not that rust-analyzer won't support theia at all, it's just that theia users would have to use an older version of rust-analyzer.

Yeah, I'd rather put the pressure on upstream to update, rather than maintain compatibility downstream. We do need to say --target=node14 though explicitly!

OK, I sent another PR to set target explicitly.
https://github.com/rust-analyzer/rust-analyzer/pull/10062
As a personal (and for theia users) memo, probably 2021-08-09 is the last version which supports nodejs because previously tsc is used and it's target is es2019
https://github.com/rust-analyzer/rust-analyzer/commit/463860439f2c02187793c2d43b649c184a270a9f
https://github.com/rust-analyzer/rust-analyzer/releases/tag/2021-08-09

@mtsmfm I think #7284 would be a large improvement for Theia users, perhaps you'd like to look into it.

@lnicola I'm not familiar with Open VSX though, I don't think it's related to Theia.

It seems theia is currently updating its node version.

Note that we are only removing unnecessary constraints on the upper bound Node version: we concluded that we should limit ourselves to using Node.js 12 JS API only (at least for now), this way adopters should be able to use any Node.js version >=12 thanks to backward compatibility of the Node.js APIs.
But what this issue shows me is that we can't really expect VS Code extension developers to limit themselves to older Node.js APIs, it only makes sense to align with VS Code's runtime (node 14 here).
Regarding your current configuration, as VS Code runs on Node.js 14 I don't think esnext is a great target: According to node.green esnext features aren't fully supported by node 14 so you might want to change it to something like es2020?

I assume Theia will have to move on in April, when Node 12 goes EOL.
As for esnext, we've already switched to node14, which seems both reasonable and explicit enough.


chore(main): release sandworm-mocha 1.1.0

:robot: I have created a release beep boop
1.1.0 (2022-09-27)
Features

initial commit (05fdd62)
initial release (5e4db06)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sandworm-hq/sandworm-mocha/releases/tag/sandworm-mocha-v1.1.0 :sunflower:


Test PointMass model with other Simulation models.

We should be able to test Sampled PointMass with Raytrace and standard Roulette, and straight PointMass with Raytrace.

There is a test in Test/pointmass, but this is outdated.
It should be updated following the pattern from Test/SIE, and possibly extended.


Replace Neat with CSS grid layout

Fixes #105
Removes Neat and replaces it with a 12-column grid using CSS grid layout grid-column styles (see mixins).
The grid-column mixin can be used flexibly to specify the starting column and number of columns that the item spans on the page. By default it starts at the first column and spans all 12 columns.
Updates the footer layout with this approach, which is the only component/layout currently using Neat. However, I'm hoping this is flexible enough once we get into implementation to replace Neat or other 12-column grid approaches.

Ok, I've adjusted this a little to hopefully make it clearer how this is working. The grid-column property is a short-hand for grid-column-start and grid-column-end, so I've separated these out in the mixin.

Although there are only 12 columns, the col-start and span values add up to 13. Thinking about this, it kind of makes sense but also is a little confusing at first. I'm not sure if there's anything we can do about that?

I adjusted the code comments to better explain this. The numbers are referring to grid-lines, not column numbers. If you inspect in the browser with the grid view, it illustrates this pretty clearly:

I'm seeing things like @include grid-column(col-start / span 6); which I'm assuming is the equivalent of @include grid-column(col-start 1 / span 6);. Assuming I'm right, is there a way to NOT have to pass those variables when we're using the default value, so something like @include grid-column(span 6);?

Yep, good call! This is now possible.

I also want to note that grid isn't supported by Internet Explorer, so this would be a definitive move away from support for that browser.


ALSA-related fixups

A typo in one of snd_pcm_open() call. According to alsa-lib source, SND_PCM_NO_AUTO_RESAMPLE is a bit flag, it's tested internally on pcm->mode. So it makes more sense to pass it as mode parameter, leaving stream only for stream type.
Suspend recovery. After suspend, ALSA could return -ESTRPIPE error code, which also should be handled. Fortunately, snd_pcm_recover() knows how to deal with it. But it doesn't always call snd_pcm_prepare() after that, which seems to be required on my hardware. Anyway, it shouldn't hurt to call it one more time.
Event demangling. ALSA API requires calling of snd_pcm_poll_descriptors_revents(). Raw revent values from fds have no meaning as-is.

Thanks for the patches! I'm happy to merge 12f5e08 as-is, but I need to understand some of the changes in the other commits better before merging them; questions above.

I merged 12f5e08 and the non-snd_pcm_prepare parts of ba4017a.
Looking at http://www.alsa-project.org/alsa-doc/alsa-lib/pcm.html#pcm_errors, the correct way to handle -ESTRPIPE involves a snd_pcm_resume loop which snd_pcm_recover handles internally.
If we really need the additional snd_pcm_prepare call, we can do that, but it's easier to take the obviously correct changes first and layer on stuff we're less sure about later.

Sorry for disappearing. :-(
I've tested again with libcubeb version from the master branch, and now 44100 audio is playing fine, while suspend-resume still stops audio completely. (As expected).
At this moment I see too many data callback calls, each for about 3 to 4 frames. For every poll wakeup alsa_set_stream_state() is called, which in turn calls poll_wake() and causes another wakeup. But I don't know how to fix this without causing that regression again. I need more time to think about it.

I think, I found what caused resume recovery failure on my machine. I have alsa-lib 1.1.1 with default configuration (i.e. no configuration at all), so dmix is used. And here is what snd_pcm_resume() backend code looks like for dmix:
int snd_pcm_direct_resume(snd_pcm_t *pcm)
{
        snd_pcm_direct_t *dmix = pcm->private_data;
        int err;

        snd_pcm_direct_semaphore_down(dmix, DIRECT_IPC_SEM_CLIENT);
        err = snd_pcm_resume(dmix->spcm);
        if (err == -ENOSYS) {
                /* FIXME: error handling? */
                snd_pcm_prepare(dmix->spcm);
                snd_pcm_start(dmix->spcm);
                err = 0;
        }
        snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
        return err;
}

For some reason snd_pcm_resume(dmix->spcm) returns -ENOSYS. Perhaps, operation is not implemented in kernel driver at all. Nonetheless, that resets error code to 0. And while snd_pcm_prepare() is called for dmix->spcm, it's not called for dmix itself. Returned 0 also prevents snd_pcm_recover() from calling snd_pcm_prepare().
Latest git version of alsa-lib had changes to /src/pcm/pcm_direct.c. Now snd_pcm_direct_resume() always returns -ENOSYS, and therefore snd_pcm_prepared() is also called.
I've tried to run with alsa-lib built from the master branch, and resume now works with unchanged version of libcubeb.

Thanks for the patches, and thanks for digging in to the resume behaviour and confirming that it's fixed upstream in ALSA!

At this moment I see too many data callback calls, each for about 3 to 4 frames. For every poll wakeup alsa_set_stream_state() is called, which in turn calls poll_wake() and causes another wakeup. But I don't know how to fix this without causing that regression again. I need more time to think about it.

Hmm, that's something I was worried about when I made the changes for bug 1279125 and it's really unfortunate... I think ultimately we need to find an alternative way to fix that bug, but I don't yet know what we need to do either.


live555: 2024.05.15 -> 2024.05.30

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

As always, live555 deletes older releases...

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4205

As always, live555 deletes older releases...

Please upload them to https://web.archive.org

Please upload them to https://web.archive.org

This is being done already - the second url template is a software archive that is saving older releases.


M1 Pro After installing TF based on Official apple guide gives ERROR for even importing TF

SYSTEM:

MacBook Pro 14 (M1 Apple Silicon)
MacOS 12.0.1

DONE:

https://developer.apple.com/metal/tensorflow-plugin/
I have followed this for ARM M1 apple silicon for my 14"
(I had anaconda installed before, that may causing the error but I DO NOT want to delete my anaconda all together)

CODE
import tensorflow as tf

ERROR
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
~/miniforge3/lib/python3.9/site-packages/numpy/core/__init__.py in <module>
     21 try:
---> 22     from . import multiarray
     23 except ImportError as exc:

~/miniforge3/lib/python3.9/site-packages/numpy/core/multiarray.py in <module>
     11
---> 12 from . import overrides
     13 from . import _multiarray_umath

~/miniforge3/lib/python3.9/site-packages/numpy/core/overrides.py in <module>
      6
----> 7 from numpy.core._multiarray_umath import (
      8     add_docstring, implement_array_function, _get_implementing_args)

ImportError: dlopen(/Users/ps/miniforge3/lib/python3.9/site-packages/numpy/core/_multiarray_umath.cpython-39-darwin.so, 0x0002): Library not loaded: @rpath/libcblas.3.dylib
  Referenced from: /Users/ps/miniforge3/lib/python3.9/site-packages/numpy/core/_multiarray_umath.cpython-39-darwin.so
  Reason: tried: '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/python3.9/site-packages/numpy/core/../../../../libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/python3.9/site-packages/numpy/core/../../../../libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/bin/../lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/bin/../lib/libcblas.3.dylib' (no such file), '/usr/local/lib/libcblas.3.dylib' (no such file), '/usr/lib/libcblas.3.dylib' (no such file)

During handling of the above exception, another exception occurred:

ImportError                               Traceback (most recent call last)
/var/folders/yp/mq9ddgh54gjg2rp7mw_t015c0000gn/T/ipykernel_95218/3793406994.py in <module>
----> 1 import tensorflow as tf

~/miniforge3/lib/python3.9/site-packages/tensorflow/__init__.py in <module>
     39 import sys as _sys
     40
---> 41 from tensorflow.python.tools import module_util as _module_util
     42 from tensorflow.python.util.lazy_loader import LazyLoader as _LazyLoader
     43

~/miniforge3/lib/python3.9/site-packages/tensorflow/python/__init__.py in <module>
     39
     40 from tensorflow.python import pywrap_tensorflow as _pywrap_tensorflow
---> 41 from tensorflow.python.eager import context
     42
     43 # pylint: enable=wildcard-import

~/miniforge3/lib/python3.9/site-packages/tensorflow/python/eager/context.py in <module>
     28
     29 from absl import logging
---> 30 import numpy as np
     31 import six
     32

~/miniforge3/lib/python3.9/site-packages/numpy/__init__.py in <module>
    138     from . import _distributor_init
    139
--> 140     from . import core
    141     from .core import *
    142     from . import compat

~/miniforge3/lib/python3.9/site-packages/numpy/core/__init__.py in <module>
     46 """ % (sys.version_info[0], sys.version_info[1], sys.executable,
     47         __version__, exc)
---> 48     raise ImportError(msg)
     49 finally:
     50     for envkey in env_added:

ImportError:

IMPORTANT: PLEASE READ THIS FOR ADVICE ON HOW TO SOLVE THIS ISSUE!

Importing the numpy C-extensions failed. This error can happen for
many reasons, often due to issues with your setup or how NumPy was
installed.

We have compiled some common reasons and troubleshooting tips at:

    https://numpy.org/devdocs/user/troubleshooting-importerror.html

Please note and check the following:

  * The Python version is: Python3.9 from "/Users/ps/miniforge3/bin/python"
  * The NumPy version is: "1.19.5"

and make sure that they are the versions you expect.
Please carefully study the documentation linked above for further help.

Original error was: dlopen(/Users/ps/miniforge3/lib/python3.9/site-packages/numpy/core/_multiarray_umath.cpython-39-darwin.so, 0x0002): Library not loaded: @rpath/libcblas.3.dylib
  Referenced from: /Users/ps/miniforge3/lib/python3.9/site-packages/numpy/core/_multiarray_umath.cpython-39-darwin.so
  Reason: tried: '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/python3.9/site-packages/numpy/core/../../../../libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/python3.9/site-packages/numpy/core/../../../../libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/bin/../lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/lib/libcblas.3.dylib' (no such file), '/Users/ps/miniforge3/bin/../lib/libcblas.3.dylib' (no such file), '/usr/local/lib/libcblas.3.dylib' (no such file), '/usr/lib/libcblas.3.dylib' (no such file)

@stromal Could you please let us know the TF version you are using ?Thanks!

Problem resolved by

not installing TF to the base environment
uninstall numpy
reinstall numpy

I have the same issue. I tried installing both TF 2.7.0 and 2.6.0. Neither worked. The resolution suggested by @stromal did not work for me.

I think TF 2.7

@stromal I have been using Anaconda with Rosetta. I guess that might be creating trouble. I can remove it and try this again.


API's Available For Insights Data?

Is the Search and Intelligence > Insights data available via the MS Graph API?  And if not, is there a roadmap item to indicate when it will be added?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 37967c2e-89bc-9a30-062f-5ced811e9c46
Version Independent ID: 0f4f3219-9b95-2dfb-8de3-9daaf8447766
Content: Search Usage Reports
Content Source: MicrosoftSearch/usage-reports.md
Service: mssearch
GitHub Login: @jeffkizn
Microsoft Alias: ankmis

Search usage data is not available via API. It is an item that is under investigation but nothing concrete to add to the roadmap yet.


How to run eslint each time file changes

Hello.
I'm wondering if it's possible to execute eslint each time file changes ?

Any update?

Use tslint  plugin https://github.com/microsoft/typescript-tslint-plugin ?


How to collect a list of Unis?

What is the best way to collect a list of Unis? For example I have a List<Uni> and would like to collect those and produce Uni<List>. Another task might be to sum all the integers and result in Uni

Did you try with:
Uni<String> uni1 = Uni.createFrom().item("hello");
Uni<String> uni2 = Uni.createFrom().item("world");
Uni<String> uni3 = Uni.createFrom().item("!");

String res =  Uni.combine().all().unis(uni1, uni2, uni3)
         .combinedWith((s1, s2, s3) -> s1 + " " + s2 + " " + s3).await().indefinitely();

// Or with a list
List<Uni<String>> list = Arrays.asList(uni1, uni2, uni3);
Function<List<?>, String> combinator = l -> l.get(0) + " " + l.get(1) + " " + l.get(2);
Uni.combine().all().unis(list).combinedWith(combinator).await().indefinitely();

Damned :-) Just found a generic type issue on this example. When using a list, you will need to cast the result. I'm on it.

FYI: https://github.com/smallrye/smallrye-mutiny/pull/90

@cescoffier thank you


about section with placeholder graphics

Screenshots
iPad

iPhone X

Laptop with HiDPI

closes #13

Amazing! :heart:
Some minor nitpicks :stuck_out_tongue: :

Maybe there could be a tiny bit of more space under the "Annual Software Engineering Conference" header, especially on smallest mobile?
I'm not sure but it looks like the cards aren't horizontally centered by a really small amount when the screen width is constraining the card. It's barely noticeable so dw about it if a fix isn't obvious

In the screenshot I'm selecting the div that has all the cards in it and they go past its right edge. Maybe the max-width: 100% isn't behaving as expected?

Also I noticed a52051162167d688bb3d1d5c47bd7213039db4bd , good thinking! If you want to, we can cherry pick that into master/splash page when this gets merged :smile:

Maybe there could be a tiny bit of more space under the "Annual Software Engineering Conference" header, especially on smallest mobile?

Oh yes, definitely!

I'm not sure but it looks like the cards aren't horizontally centered by a really small amount when the screen width is constraining the card. It's barely noticeable so dw about it if a fix isn't obvious
In the screenshot I'm selecting the div that has all the cards in it and they go past its right edge. Maybe the max-width: 100% isn't behaving as expected?

I didn't even notice, thanks! I'll check it out!

Also I noticed a520511 , good thinking! If you want to, we can cherry pick that into master/splash page when this gets merged 😄

Yes!

Is it sketch if I reduced the max-width to 98% and it looks centred to me?

What do you think?

Is it sketch if I reduced the max-width to 98% and it looks centred to me?

Tbh it's a bit sketchy but if it works, it works :shrug: :joy:
I'd check with the common browsers to see if it's showing up ok for them

I just checked with Firefox and Safari, and it looks good to me!


Can we combine this profitably with Schedule-Free?

https://github.com/facebookresearch/schedule_free
This looks compatible with the prodigy approach, but not entirely sure.

Working on that!

I've created an experimental implementation of schedule-free Prodigy. First tests are going well:

https://github.com/konstmish/prodigy/pull/26


kconfig: Shield experimental features under a Kconfig option

This makes all features marked with an "experimental" tag in their
prompts depend on a CONFIG_EXPERIMENTAL option that defaults to "y".
Defaulting to "y" is consistent with the current behavior and
will not break anything.
The rationale is similar to the one available in Linux, and is
succintly explained in the help text for CONFIG_EXPERIMENTAL.
Signed-off-by: Leandro Pereira<EMAIL_ADDRESS>

Codecov Report

Merging #8485 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8485      +/-   ##
==========================================
+ Coverage   64.61%   64.61%   +<.01%
==========================================
  Files         421      421
  Lines       40296    40296
  Branches     6803     6803
==========================================
+ Hits        26037    26038       +1
+ Misses      11126    11125       -1
  Partials     3133     3133

Impacted Files
Coverage Δ

samples/philosophers/src/main.c
98.43% <0%> (+1.56%)
:arrow_up:

Continue to review full report at Codecov.

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

Another option would be to have the experimental stuff default n (which is the default if you don't have a default) and note in the help text or the like that it's experimental.

Help text will be forgotten and completed features will marked as such forever as we will see based on the list that was modified in this PR, maybe adding the option is a good thing.

@nashif
Yeah, I have no strong opinions either way here to be honest.
Would probably be pretty easy to go back if we discover problems at least.

Typo here ("EXPERIMENtAL") :)
Error: Found references to undefined Kconfig symbols:

EXPERIMENtAL
============

- Referenced at subsys/bluetooth/host/Kconfig:522:

config BT_AVDTP
	bool
	prompt "Bluetooth AVDTP protocol support" if EXPERIMENtAL && BT_BREDR && BT_HCI && BT
	depends on EXPERIMENtAL && BT_BREDR && BT_HCI && BT
	help
	  This option enables Bluetooth AVDTP support

Typo here ("EXPERIMENtAL") :)

cool, is that your first victim :)

cool, is that your first victim :)

Might be. :)

Might want to check that no configuration files enable any of those experimental symbols too. They'd need to enable CONFIG_EXPERIMENTAL now as well.

CONFIG_EXPERIMENTAL is declared as "default y" precisely so that, at this moment, no changes need to be made in any configuration file.  I'm more worried about LTS releases and people building products with unstable features; this configuration option makes it easier to find which features are safe for such usages.  Given that 1.14 will be the LTS release, reviewing this early in the 1.13 cycle gives us a good head start.

stale PR, please reopen or submit a new one when you have the time.


[WIP] Hot reloading

Awesome project btw. I kinda started on https://github.com/loco-rs/loco/issues/920 and wanted your input.

Semi working hot reloading without unsafe.
Need to also attach the tower-livereload to make it fully automated.

It looks like the tera member in the TeraView struct needs to be made private, to not expose the leaky details about it being either in a Mutex in debug vs not wrapped in release builds.
Perhaps we could add 2 methods,

fn with_tera<'s, A>(&'s mut self,  f: impl FnOnce(&'s tera::Tera) -> A ) -> A
fn with_tera_mut<'s, A>(&'s mut self,  f: impl FnOnce(&'s mut tera::Tera) -> A ) -> A

Bit also unsure about wiring up the tower-livereload without breaking existing interfaces. I did a roughed up patching within my own code and it works without issues.

Hi! happy you decided to contribute!
I'm wondering if we can split this into:

reloading content from disk
live-reload (triggering browser refresh) -- which will probably be solved by some kind of middleware glue (maybe..?)

There is a good chance a simplification can be made for reloading from disk - by just reading a file, and then rendering using a template string (instead of arc/mutex/ etc.). It can make a simpler (yet less clever) code, since we don't much care about perf in debug builds. WDYT?

Moving this one to 0.14, if you can refactor it into a middleware that only does live-reload that would be perfect!

Just back from holidays, I'll try to see if I can come up with something simpler this or next week. Think most of the reload logic could probably live in the user side, if we somehow expose methods like full_reload and paths in the TeraView


Return a Range object instead?

Return a Range object with a @@iterator on it so we can add some helper functions on the range object like range.includes()
Possible methods on the Range object

includes(x)
@@iterator
@@slice (https://github.com/tc39/proposal-slice-notation/issues/19 )
toLocaleString(...) (https://github.com/Jack-Works/proposal-Number.range/issues/12#issuecomment-593772905)

Does that also means introducing new Range type with possible static methods on it? Should there be Number.Range and Bigint.Range than?

yes, I think it will be useful but I'm not quite clear about what will the API looks like
maybe some investigation on other languages

Some possible instance and static methods:

Number.Range.isFinite()
Number.Range.prototype.toLocaleString([locales [, options]])
BigInt.Range.prototype.toLocaleString([locales [, options]])

Maybe not making a Range object, but add useful methods on %RangeIteratorPrototype%.
There're complains that said Number.range is too long, and make Range a class will even longer ((new Number.range()).includes())
By adding methods on the %RangeIteratorPrototype%, we can use it like Number.range(a, b).isFinite().

For the convenience of future polyfill, IMO it should also expose 3 getters (or non-writable values) start, end and step to expose its internal slot.

Yes, we're going to have helper methods. I'm closing this issue. If anyone wants to suggest any helpers, please open a new issue. That will make the discussion easier to track.


Smal typo

Please squash your commits.

Please squash your commits.

Closing as stale. Please feel free to reopen when you have finished this. Thanks!


additional sources for contrast colors

to keep the color definitions DRY... being able to pull the colors from

config setting
another field using query language

would be awesome.

That's a good idea, especially using the query language. But I have no idea how to do this actually.

you could try to get inspiration from here:
https://github.com/sylvainjule/kirby-color-palette#dynamic-options

If you have figured out a good syntax, it's not too difficult, @nilshoerrmann:

Create the prop in the PHP field setup. Can be very simple

'yourProp' => function (string $prop = null) {
    return $prop;
},

Create a computed prop, that will take the normal prop and run it as a query:

'yourProp' => function () {
    if ($this->yourProp !== null) {
        return $this->model()->toString($this->yourProp);
    }
},

Hi, is there a way today to configure contrast options with a query  from an other field? (if I want to pick colors for background AND text in my panel and check accessibility) Thank you :)

No, I don't think so. But we should finally add this option. Can you provide me with the desired field setup in YAML you were thinking of?

So I noticed something that certainly is the reason why we have not implemented this yet: it's one thing to set up the relationship between two field via queries in the blueprint – that's purely on the server. But we also have to deal with live updates inside the panel, in the browser. E. g. your two fields are on the same page and thus changing the one should automatically update the other on the fly. So somehow the Vue components have to listen to other fields as well and not just the PHP backend.
Are there other examples of fields doing this already?

We had Sylvain Jule make us a plugin that we can get colours directly from the site settings. It is still online and can be found here. It's in heavy use every day and with cache function.
https://github.com/sylvainjule/kirby-color-palette/tree/custom-classes

Thanks @Werbschaft!
I'm thinking about a good syntax. So we have this pattern for server side rendered options already:
contrast:
  type: query
  query: site.colors

What about this for live field updates?
contrast:
  type: watch
  field: nameoffield

Alternatively (or additionally), it could also be an idea to add an option to edit the contrast color directly, allowing users to switch the color they are editing:
contrast: editable

/cc @johannahoerrmann

Yes @nilshoerrmann , making the contrast field editable at first would personally allow me to set up what I want. Otherwise the syntax you suggest seems really good to me.

Please have a look at https://github.com/hananils/kirby-colors/releases/tag/1.7.0.
Feedback welcome :)

Wow thank you SO MUCH, it's a great great feature. Web accessibility is really important to me and thanks to your work I can offer it very simply in the panel. I'll mention it on the Kirby forum. Thanks again!

Just so you know: The new version only includes the blueprint queries, not contrast: editable. This will require more time and consideration because the field markup and CSS is already quite complex.


Enhance removal logic incl. soft-delete

Not all modules are currently removed after deployment. Especially the ones, that have soft- delete enabled (they can be deleted but not recreated).
The current removal logic should be enhanced to cover for those cases.

Working on it as part of another PoC

Aside from KeyVault we have to also handle RSV (pre-removal), cognitive services & API management


Settings view

Allow user to set period time, rest time, long rest time, etc.

Present view using this:
Do not call this method unless you want to use a custom animator. To use one of the standard animators to present another view controller, instead call one of the dedicated presentation methods:
presentViewController(:asPopoverRelativeTo:of:preferredEdge:behavior:)
presentViewControllerAsModalWindow(:)
presentViewControllerAsSheet(_:)
Each of these methods calls this method in turn. User interaction is blocked during presentation and dismissal.


[DPG] Support new RPaaS generation from TypeSpec

See https://github.com/Azure/typespec-azure/issues/3591

MPG generate diff for azure-mgmt-sphere: https://apiview.dev/Assemblies/Review/f48fd53f674c4657a064361ee8dd0859?revisionId=c635cb1ae6ee4ef991528b33ea20042c&diffOnly=False&doc=False&diffRevisionId=880ae55cfe1742649ef8f9ddf4cda63b

[ ] AzureSphereMgmtClient -> AzureSphereClient (could be renamed in client.tsp)
[ ] lro operation are not recognized (across language, need TCGC optimize)
[ ] expose send_request for client
[ ] expose new kwargs stream
[ ] hide maxpagesize
[ ]

We do support it now


chore: prepare release 2024-09-19 [skip-bc]

Summary
This PR has been created using the apic release script. Once merged, the clients will try to release their new version if their version has changed.
Version Changes

csharp: 7.4.0 -> patch (e.g. 7.4.1)
dart: 1.24.1 -> patch (e.g. 1.24.2)
go: 4.3.2 -> patch (e.g. 4.3.3)
java: 4.3.2 -> patch (e.g. 4.3.3)
javascript: 5.5.1 -> patch (e.g. 5.5.2)
kotlin: 3.3.2 -> patch (e.g. 3.3.3)
php: 4.4.2 -> patch (e.g. 4.4.3)
python: 4.4.2 -> patch (e.g. 4.4.3)
ruby: 3.3.2 -> patch (e.g. 3.3.3)
scala: 2.3.2 -> patch (e.g. 2.3.3)
swift: 9.3.2 -> patch (e.g. 9.3.3)

Skipped Commits
(None)

will re-open once https://github.com/algolia/api-clients-automation/pull/3774 is merged


Refactor unit tests to permit stored properties in common test scaffolding

Description
Currently KituraNetTest that provides the common scaffolding for all the tests is a protocol that XCTestCase extends. But since protocols and extensions cannot have direct stored properties, the functionality this can provide is constrained. This PR refactors this such that KituraNetTest extends XCTestCase and the other tests extend from it and adds stored properties to leverage this.
Motivation and Context
IBM-Swift/Kitura#970
How Has This Been Tested?

Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

I reviewed this PR and it can be merged once CI has run.

Current coverage is 81.94% (diff: 100%)

Merging #168 into master will decrease coverage by 1.57%

@@             master       #168   diff @@
==========================================
  Files            27         27
  Lines          2348       2348
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           1961       1924    -37
- Misses          387        424    +37
  Partials          0          0

Powered by Codecov. Last update a5cf88f...2a22acf


"~/... and /Users/raxod502/... are the same file"

I know this. You don't need to tell me, Emacs.

And surprisingly, it wasn't even VC's fault.


Add support for export options

One should be able to export the data set presented on the screen to a file, pdf or csv, etc. While the server may handle that portion, the screen should present options for exports.

Another function that can be accomplished using DataTables. The dataTables buttons provide export functionality
https://datatables.net/extensions/buttons/


Added support for configuring array values with environment variables

resolves https://github.com/distribution/distribution/issues/625
copied from https://github.com/distribution/distribution/issues/625#issuecomment-916392527

@milosgajdos let me know what do you think, should i add tests right?

@andriisoldatenko can you please sign the commit?

@andriisoldatenko can you please rebase and sign your commit.

Sure will do!

@milosgajdos done. Let's me think how to write proper test.

Codecov Report
Patch coverage has no change and project coverage change: -0.04 :warning:

Comparison is base (d5c1b39) 55.76% compared to head (f8f35cf) 55.73%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3511      +/-   ##
==========================================
- Coverage   55.76%   55.73%   -0.04%
==========================================
  Files         108      108
  Lines       10969    10976       +7
==========================================
  Hits         6117     6117
- Misses       4165     4172       +7
  Partials      687      687

Impacted Files
Coverage Δ

configuration/parser.go
81.76% <0.00%> (-3.52%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@andriisoldatenko can you please sign your commit https://github.com/distribution/distribution/blob/main/CONTRIBUTING.md#sign-your-work

@milosgajdos done :)

So, I played around with this quickly on your branch but I don't think this will address the issue 🤔
Once you've added some tests you'll see this hits a hurdle pretty soon.

@andriisoldatenko I've taken the liberty and updated the PR with some code and a test if that's ok with you? I'm happy to revert if you would like to take a stab at it again.

@milosgajdos no problem, thanks a lot for help and for review!

@milosgajdos do we need to update any release notes/docs/manuals?

I suppose we could add an example to https://github.com/distribution/distribution/blob/main/docs/configuration.md#override-specific-configuration-options 🤔


Someone just broke the 2.1

My workflow did this successfully yesterday night:
    - name: Deploy ApiReview.Client
      uses<EMAIL_ADDRESS>      with:
        app-name: apireviews
        package: './artifacts/ApiReview.Client'

which fails with this:
An action could not be found at the URI 'https://api.github.com/repos/azure/webapps-deploy/tarball/v2.1'

hacking the URL reveals that this works:
https://codeload.github.com/Azure/webapps-deploy/legacy.tar.gz/v2.1.0
Is that intentional?

Please use<EMAIL_ADDRESS>in the workflow. In future, please use major version of the action which will include all minor and patch updates.

Please use "azure/webapps-deploy@v2" in the workflow. In future, please use major version of the action which will include all minor and patch updates.

Is that specific to webapps-deploy or is that generally a thing for GitHub actions?

In general based on Github actions versioning guidelines, actions should follow this behavior, but individual actions releases may vary.
For webapps-action, major tag will have all minor and patch updates.

Closing this issue, please feel free to re-open if issue persists.


Rector: enable Code-Quality Rules

Codecov Report

Merging #6 (f57dccf) into main (9085625) will increase coverage by 0.23%.
The diff coverage is 81.25%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@             Coverage Diff              @@
##               main       #6      +/-   ##
============================================
+ Coverage     67.44%   67.68%   +0.23%
  Complexity      287      287
============================================
  Files            22       22
  Lines           980      981       +1
============================================
+ Hits            661      664       +3
+ Misses          319      317       -2

Files Changed
Coverage Δ

src/CodeClone.php
0.00% <0.00%> (ø)

src/CodeCloneMap.php
0.00% <0.00%> (ø)

...SuffixTree/ApproximateCloneDetectingSuffixTree.php
94.17% <100.00%> (+0.05%)
:arrow_up:

...tector/Strategy/SuffixTree/SuffixTreeHashTable.php
96.22% <100.00%> (ø)

src/Detector/Strategy/SuffixTreeStrategy.php
95.12% <100.00%> (+0.12%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Whitescreen issue

I'm getting whitescreen suddenly while using brackets. I tried clear cache folder, uninstall and reinstall and tried different version but the issue is still coming. I'm using windows 10 and brackets version is 1.7

Possible duplicate of #12617

I close as a dupe of #12617 for now.


Wording correction for notification modal

As mentioned in #386 i changed the wording a little in the notification modal.

As the meaning is as a abit changed, wondering should we gloabl replace all keys in all languages.

I think so yea, unfortunately the meaning might be wrong in some languages then.


[WIP][SubPR] 🍰 Implement Enum Backend Errors – A Draft To Discuss

🍰 Pullrequest

As sub PR to #1858
!!! May we have to change the parent PR number and the compared branch if there come more sub branches inbetween !!!
Implement enum backend errors – a draft to discuss.
Issues

fixes #1850

Todo

[x] have subs in your eyes
[ ] is locale identifiers:

[ ] settings.password.reset still needed?
[ ] why user-already-exists is still in the master?
[ ] message.activateEmail seems unused
[ ] message.unsetPassword seems unused

We close this one in favor of #2141


Development server

Motivation
Wasm-bindgen provide all requirements to generate Wasm application. Including the startup with the #[wasm_bindgen(start)] attribute.
It miss a very little portion of code to:

start a server
(re-)build the wasm application

Today, it requires an external tool, hot reloading is not provided etc.
It can be with WebPack:
Paint example
Or using the wasm-bindgen-cli
GFX example
One of the paint/complex work is to manage cross target libraries.
For example the Quad example from GFX is a main as it can run on all platforms.
But the same code can also run for Web with WebAssembly.
I have tried to manage that in that issue: https://github.com/gfx-rs/gfx/pull/3475
It works using 2 Cargo manifests, requires to duplicate dependencies and do some tricks.
Proposed Solution
Improve the CLI crate to add a mode to serve the application.
maybe to provide a command like:
wasm-bindgen --server
Or maybe add a server target.

Thanks for the report! This is something I'd be happy to help design and take a PR for, but I don't really know of a great design or how it would fit into wasm-bindgen myself. There's already basically a server for wasm-bindgen-test so that can probably be reused.
This would be a great feature to have though!

Hello,
I wanna to work on it, @alexcrichton do you have some advices ?
Do you have idea what I need to check first ?
For me it means:

call cargo build command - maybe with watch option
call wasm-bindgen-cli
serve over HTTP (I plan to use https://crates.io/crates/https)

Thank you,
Marc-Antoine

Yeah that's the rough shape of ideally what this would look like. I think it'd be good though to make a proposal for the interface for this and how it's expected to be used, though.


Enhancing the usability of custom metric tags in the Istio environment for WASM.

Describe the bug / error
In an Istio environment, custom metric tags can be automatically configured using extraStatTags. The default stats_config format is label=.=value;.;. It is necessary to enhance the ease of use of custom metric tags in Istio environment for WASM.
What is your Envoy/Istio version?
What is the SDK version?
What is your TinyGo version?
URL or snippet of your code including Envoy configuration
Additional context (Optional)

Please keep in mind this is the repository for the Go SDK, and I don't think your description is not actionable here.

Istio provides some default stats_tags in stats_config, such as:
{
"tag_name": "reporter",
"regex": "(reporter=\.=(.?);\.;)"
},
{
"tag_name": "source_namespace",
"regex": "(source_namespace=\.=(.?);\.;)"
},
{
"tag_name": "source_workload",
"regex": "(source_workload=\.=(.*?);\.;)"
}
In this way, we can provide a function to automatically organize metrics in the corresponding format, which improves usability in the Istio environment without the need to write a xDS code snippet.


10.0.22621.3527

termsrv.zip

Zero size termsrv.dll file in the zip?
Try again and let us know if it is x86 or x64, thanks

termsrv (2).zip

Okay, not sure what is going on with your zip files, but they wer 156 bytes and 120 bytes respectively -- no termsrv.dll in there.


SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data

When running the following code, I get a SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data error. The issue is likely the writeResponse method trying to infer the content type. This instead should render this with content type text/plain
  router.get('/version', (Request request) async {
    return Response.Ok('Version 1.0');
  });

I am looking into this.

Fixed in v0.2.8


2-arity version of child+ not working

(child+ (object-named "Main Camera")
        (GameObject.)
        false)
;; works

(child+ (object-named "Main Camera")
        (GameObject.))
;; clojure.lang.ArityException: Wrong number of args (2) passed to: arcadia/core/eval--30607/fn--30619--30623
;;   at clojure.lang.AFn.invoke (System.Object arg1, System.Object arg2) [0x00000] in <filename unknown>:0
;;   at arcadia/core$eval__30380$fn__30429$G__30343__30451__30454.invoke (System.Object , System.Object ) [0x00000] in <filename unknown>:0
;;   at user$eval__34092__34097.invoke () [0x00000] in <filename unknown>:0

closed as of 7493e4590e46b923860602ed1cc3aed5062979e6


How to mock view.measure() ?

💬 Questions and Help
So, I'm trying to test my component. In my component there a function to measure from reference of View.
when I debug, the ref value is exists. but the measure method is undefined.
this is the screenshot of the testing.

I need help how to mock the .measure or other function from component reference

Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.


What about a constructor from an existing std::array of key value pairs?

What do you think about adding an additional constructor for the case where one already has the data array created?
diff --git a/include/constexpr_hash_map/constexpr_hash_map.hpp b/include/constexpr_hash_map/constexpr_hash_map.hpp
index c1b9d50..68d740b 100644
--- a/include/constexpr_hash_map/constexpr_hash_map.hpp
+++ b/include/constexpr_hash_map/constexpr_hash_map.hpp
@@ -42,6 +42,15 @@ public:
         static_assert(N == sizeof...(elements), "Elements size doesn't match expected size of a hash-map");
     }

+    /// @brief The only other construction that might be used, array must be provided in the constructor.
+    /// @param std::array<std::pair<K,V>, N>, cannot be empty
+    explicit constexpr hash_map(data_type arr) noexcept
+    : data{std::move(arr)}
+    {
+        static_assert(N > 0, "N should be positive");
+        static_assert(N == data.size(), "Array size doesn't match expected size of a hash-map");
+    }
+
     /// @brief Searches map for a given key and returns iterator.
     /// @param key key to be searched for
     /// @return constant iterator to an element (cend, if not found)

Someone with more experience template programming may find a way to deduce the N, K and V parameters from the std::array itself.

This version fixes an issue in the static assert and provides a deduction guide.
diff --git a/include/constexpr_hash_map/constexpr_hash_map.hpp b/include/constexpr_hash_map/constexpr_hash_map.hpp
index c1b9d50..2a01401 100644
--- a/include/constexpr_hash_map/constexpr_hash_map.hpp
+++ b/include/constexpr_hash_map/constexpr_hash_map.hpp
@@ -42,6 +42,14 @@ public:
         static_assert(N == sizeof...(elements), "Elements size doesn't match expected size of a hash-map");
     }

+    /// @brief The only other construction that might be used, array must be provided in the constructor.
+    /// @param std::array<std::pair<K,V>, N>, cannot be empty
+    explicit constexpr hash_map(data_type arr) noexcept
+    : data{std::move(arr)}
+    {
+        static_assert(N > 0, "N should be positive");
+    }
+
     /// @brief Searches map for a given key and returns iterator.
     /// @param key key to be searched for
     /// @return constant iterator to an element (cend, if not found)
@@ -170,6 +178,9 @@ protected:
 private:
     data_type data;
 };
+
+template <std::size_t N, typename K, typename V>
+hash_map(std::array<std::pair<K, V>, N>) -> hash_map<N, K, V>;
 }  // namespace burda::ct

 #endif // CONSTEXPR_HASH_MAP_CONSTEXPR_HASH_MAP_HPP


FAS-1 Linters

Nice work!
However, I would suggest a couple of changes:

use poetry for managing packages
add black and isort to dev-dependencies via poetry and pin their versions - we don't want unexpected surprises due to changes that might bring new versions


wrong answer on this case

{3, 6, 6, 5, 0},
{1, 2, 1, 8, 0},
{2, 1, 0, 4, 0},
{1, 9, 5, 9, 0},
{0, 5, 9, 5, 0}

Can I ask what is the main idea under the code, any document explains these squares and marked zeros?

Can I ask what is the main idea under the code, any document explains these squares and marked zeros?

It's been a couple of years when I wrote this code. I used the German version of the Wikipedia article: https://de.wikipedia.org/wiki/Ungarische_Methode
and http://ccp.uchicago.edu/khetarpal/code/edit-distance/HungarianAlgorithm.java (does not exist anymore)
The marked zeroes are the 0' of the Wikipedia article and the squares are only used as markers: "Draw a square around the element to mark it".... as described in the article.


RecvPayload inconsistency

Hello again,
I've continued my testing (still a side project, not much time available) and found that Server.RecvPayload using client indexes makes it pretty hard to use SendPayloadToClient which uses client ids.
How can I match those? I'm missing a translate step for clientIndex <-> clientId.
EDIT: Having two different client identifiers feels like a bad idea. Ofc I've seen Server.getClientIndexByClientId which does the translation I want, but that's private. Making it public would solve this issue BUT doesn't seem like the way to go. Everything but SendPayloadToClient uses clientIds, let's use those everywhere.

My advice is to please tend to prefer clientIndex when identifiying clients, not client Id. Then, have a nice function to get the client id for a particular client index.

Temporarily I can make the getClientIndexById method public. I'd really like to get this thing rewritten though :/.

@toqueteos Are you able to hop on as a maintainer? It seems like @wirepair could use a hand

@gafferongames I wouldn't mind being a maintainer but there's no rush. I'm making progress, although veeeeery sloooooow on making this more idiomatic and close to the C library 💪

No rush at all. Marathon not a sprint!

I made GetClientIndexById public so i'm going to close this one.


HashTag and see more functionality

Is there an existing issue for this?

[X] I have searched the existing issues

Use case
How can we handle HashTag (@, #) functionality in Quill Editor.
How can we add see more options if text is more than specific lines
Proposal
Please add HashTag tag functionality to editor as it mostly used in social Media Apps.
Anyone please suggest me as I have these new requirements in my flutter app using flutter_quill editor as already it is handling a lot of functionality.

Thank you for the report, we currently do have a lot of issues to process as such we're unable to add this feature, it also requires some changes to different places.
We plan to separate such features as an extension package since they often conflict with other features, and they are not standard for all use cases.
An example in Quill JS is quill-mention.

Though this is not a feature yet, it seems like it's possible to implement. See #1330.


Removes extra docker call for the service name

The service name can be extracted directly from docker service ls without the need for a second call for every service.
This PR drops the service variable in favor for using the name variable consistently.

LGTM. Thanks!


Package "xxx" isn't found at NPM registry error with private npm packages

@ruscon sorry for being silent for too long. I've fixed all 3 issues that you've created.


Fail to upload package when description rendering is invalid.

This branch prevents a package upload if the rendered content for the
description_content_type is invalid. If no description_content_type is
set, 'text/x-rst' is assumed as the type.
I'm sure the error message could point to something more informative that explains why failed rendering blocks an upload. Since such a page doesn't exist for now, I went with a somewhat generic error message. I'm happy to push more commits when a specific thing is in place. I'm a bit ignorant about how docs are handled for warehouse.
Fixes #3966

@di Please let me know if you need any more for the docs or the error message. I did my best, however, PyPI might have some guidelines or voice that I'm unaware of, and I can tweak the wording until it feels right.

Woohoo! Thanks for the help @di! I'm glad I was able to fix a PyPI issue that has bugged me for ages.


Changes to conform to the internal guidelines

The following changes were done to conform to the internal documentation:

remove Maintainer from RHEL Dockerfile
add architecture label
change Component label to BZComponent

lgtm.


Feature request: Move spawn kill/kick time period out of code and into config file

Hi AK,
I am having some issues with spawn killers.
I was wondering if you could make the spawn protection time configurable by getting the value from the config file.
I think this is the bit:
Spawn Protection time between players deaths in seconds to issue a warning
warn_time = 3
Keep up the good work :-)
--G

Yes correct, this is the time within a player gets a warning, when a new respawned player is killed again.
Yes of course, this can easily be implemented and added to the config file and allow the operator to define an own value.

Hi AK,
I was thinking about this last night.
The problem I have is that if the team balance is out,  one team will be
pinned in their spawn area by the other team (in TDM).
If I increase the spawn protection too much, the opposite will happen.  A
good player could cross half the map while still being protected.
I was wondering if it would be possible to protect based on distance
travelled from spawn ?
This would solve the problem because you could say protection is over after
5 seconds or X distance whichever is first.
That way slow players could be protected and fast players would not get an
advantage.
This may be too hard/not possible, but an interesting idea :-)
--G
On Wed, Apr 22, 2020 at 11:09 PM Alexander Kress<EMAIL_ADDRESS>wrote:

Yes correct, this is the time within a player gets a warning, when a new
respawned player is killed again.
Yes of course, this can easily be implemented and added to the config file
and allow the operator to define an own value.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SpunkyBot/spunkybot/issues/69#issuecomment-617768953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJV5CETFIYTVG63SFDCXRZTRN3UAZANCNFSM4MN57W7A
.

no that is impossible, it is used pinned to the respawn time.
Of course, this value is an impact if an admin does not know what he did.
Per default all UrT servers using the respawn protection of 2 sec, within this time the enemy cannot even kill you.
If the spawnkill counter is set e.g. higher than 5 sec, this would be a big disadvantage for the opponent team.
Therefore I hardcoded the value to 3 sec.
From my point of view, this value makes only seen between 2-5 secs.
2 sec : no function
3 sec: best practice value
4 sec: maybe only for noob servers
5 sec: already too long, player can reach half of map

Sweet, thanks for checking it out :-)
--G
On Sun, Apr 26, 2020 at 12:14 AM Alexander Kress<EMAIL_ADDRESS>wrote:

no that is impossible, it is used pinned to the respawn time.
Of course, this value is an impact if an admin does not know what he did.
Per default all UrT servers using the respawn protection of 2 sec, within
this time the enemy cannot even kill you.
If the spawnkill counter is set e.g. higher than 5 sec, this would be a
big disadvantage for the opponent team.
Therefore I hardcoded the value to 3 sec.
From my point of view, this value makes only seen between 2-5 secs.
2 sec : no function
3 sec: best practice value
4 sec: maybe only for noob servers
5 sec: already too long, player can reach half of map
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SpunkyBot/spunkybot/issues/69#issuecomment-619386049,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJV5CEWYRTG4A2DAC65LYIDROLV45ANCNFSM4MN57W7A
.


some nitpicks on SequenceSampler PR

cc @robintibor @hubertjb  what do you think?

So how about this solution:

target_transform as part of BaseConcatDataset but always transforms targets per item, independent of if we get sequences or individual items
What was seq_target_transform we could implement the way data augmentations are implemented, once https://github.com/braindecode/braindecode/pull/254 is merged? I think then everything could resolve very nicely?

Also @hubertjb @cedricrommel


Changing fe.conf and be.conf should trigger pod restart when using the Kubernetes Operator

Feature request
Is your feature request related to a problem? Please describe.

Currently, the fe.conf and be.conf is located inside 2 ConfigMap.
When we changed the values, pods are not being restarted gracefully. We had to manually perform rolling restart to pick up the changes.
It would be amazing if the operator performs restart correctly when config file is changed.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

@yandongxiao
@didip I suppose you are using operator to deploy the cluster using CR instead of helm chart, in helm chart, the fe.conf/be.conf configmaps are managed by the chart and will change the annotation whenever the content is changed, and hence triggers the rolling restart.


Usability Question

"Keep in mind that your entire Vectra index is loaded into memory".
However, the index can be large and if you have enough RAM (say 32 GB or 64 GB) it is ok if index is loaded, right?
Alternatively, each document or topic could be stored in a separate index, if RAM is not enough, right?

Right on the money @securigy. Indexes can get pretty large, and yeah 32GB ram would be more than enough. Besides, you can have some sort of logic (or a "sorting" index) that routes to the correct index.

Well, I definitely will try your solution for local vector DB. For some clients of mine it is a must. I already implemented Pinecone in C# and I can't say I like it too much - ingested data is available after unacceptable delay like anything else there...


Global TLS is disabled when an KIngress is configured with a TLS secret

Serving Issue: https://github.com/knative/serving/issues/12377
Global TLS is configured here:
https://github.com/knative-sandbox/net-kourier#setup-tls-certificate
When you create a KIngress with it's own TLS secret (ie. provided by autoTLS) it elides the configured global tls cert from the envoy's filter chain.
Looks like it's elided here when the SNI listener is created
https://github.com/knative-sandbox/net-kourier/blob/87c1085d407b1a5026cd231eba64c90ca91ec7f4/pkg/generator/caches.go#L237-L256

cc @nak3 unsure who I should assign this to?

Hello :wave:
I have tested with a modified version of kourier (https://github.com/knative-sandbox/net-kourier/compare/main...norbjd:fix-tls-disabled-on-main-endpoint-when-domain-mapping-configured) where if len(sniMatches) > 0 and if useHTTPSListenerWithOneCert() are not mutually exclusive, and it works correctly; at least, it solves issue https://github.com/knative/serving/issues/12377#issuecomment-1005019057.
Please tell me this fix looks correct to you, and if so, I will happily make a pull request.

Hello :wave:
I just wanted to add to https://github.com/knative/serving/issues/12377 that if I create multiple ksvc and if I apply a domain mapping to any of these ksvc, then all ksvc's https default urls return OpenSSL SSL_connect: SSL_ERROR_SYSCALL.

It looks like that when configuring a domain mapping my.great.domain.tld, the whole listener_8443's filter_chains (in Envoy config) is overriden and become a single object corresponding only to my.great.domain.tld:
"filter_chains": [
 {
  "filter_chain_match": {
   "server_names": [
    "my.great.domain.tld"
   ]
  },
  "filters": [
   {
    "name": "envoy.filters.network.http_connection_manager",
    "typed_config": {
     "@type": "type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager",
     "stat_prefix": "ingress_http",
     "rds": {
      "config_source": {
       "ads": {},
       "initial_fetch_timeout": "10s",
       "resource_api_version": "V3"
      },
      "route_config_name": "external_tls_services"
     },
     "http_filters": [
      {
       "name": "envoy.filters.http.router"
      }
     ],
     "access_log": [
      {
       "name": "envoy.file_access_log",
       "typed_config": {
        "@type": "type.googleapis.com/envoy.extensions.access_loggers.file.v3.FileAccessLog",
        "path": "/dev/stdout"
       }
      }
     ]
    }
   }
  ],
  "transport_socket": {
   "name": "envoy.transport_sockets.tls",
   "typed_config": {
    "@type": "type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.DownstreamTlsContext",
    "common_tls_context": {
     "tls_certificates": [
      {
       "certificate_chain": {
        "inline_bytes": "<certificate in base64>"
       },
       "private_key": {
        "inline_bytes": "W3JlZGFjdGVkXQ=="
       }
      }
     ],
     "alpn_protocols": [
      "h2",
      "http/1.1"
     ]
    }
   }
  }
 }
]

Because of this, all https requests (other than https://my.great.domain.tld) coming to this listener are not handled because this config misses the "default" case. If we had this "default" case, we obtain something like:
"filter_chains": [
 {
  "filter_chain_match": {
   "server_names": [
    "my.great.domain.tld"
   ]
  },
  "filters": [
   // same as above
  ]
 },
 // possibly other filter chain objects (with different filter_chain_match)
 // and below, the "default" filter chain
 {
  "filters": [
   {
    "name": "envoy.filters.network.http_connection_manager",
    "typed_config": {
     "@type": "type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager",
     "stat_prefix": "ingress_http",
     "rds": {
      "config_source": {
       "ads": {},
       "initial_fetch_timeout": "10s",
       "resource_api_version": "V3"
      },
      "route_config_name": "external_tls_services"
     },
     "http_filters": [
      {
       "name": "envoy.filters.http.router"
      }
     ],
     "access_log": [
      {
       "name": "envoy.file_access_log",
       "typed_config": {
        "@type": "type.googleapis.com/envoy.extensions.access_loggers.file.v3.FileAccessLog",
        "path": "/dev/stdout"
       }
      }
     ]
    }
   }
  ],
  "transport_socket": {
   "name": "envoy.transport_sockets.tls",
   "typed_config": {
    "@type": "type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.DownstreamTlsContext",
    "common_tls_context": {
     "tls_certificates": [
      {
       "certificate_chain": {
        "inline_bytes": "<default certificate (defined in CERTS_SECRET_NAMESPACE/CERTS_SECRET_NAME secret) in base64>"
       },
       "private_key": {
        "inline_bytes": "W3JlZGFjdGVkXQ=="
       }
      }
     ],
     "alpn_protocols": [
      "h2",
      "http/1.1"
     ]
    }
   }
  }
 }
]

With that said, I have forked the repo and created a version that solves this issue (see https://github.com/knative-sandbox/net-kourier/compare/main...norbjd:fix-tls-default-filter-chains). It works (without domain mapping, with 1 domain mapping, and with 2 domain mappings), but I have not created a PR because I don't know if my modification is correct and if it impacts or not the rest of kourier's behavior.
Feel free to check my commit and tell me if it's ok, I will happily create a PR then.

Thank you @norbjd ! I guess your approach should be fine. Could you please send the PR? It would be great if you could add the test case as well.

@nak3 I have created the PR https://github.com/knative-sandbox/net-kourier/pull/743 and added some tests covering the fix! :tada:


OS X 15 introduces breaking changes; C++ headers not found by cmake

Building with cmake 3.30.3:
simdutf % cmake --build build
[  1%] Building CXX object src/CMakeFiles/simdutf.dir/simdutf.cpp.o
In file included from /tmp/simdutf/src/simdutf.cpp:1:
/tmp/simdutf/include/simdutf.h:3:10: fatal error: 'cstring' file not found
    3 | #include <cstring>
      |          ^~~~~~~~~
1 error generated.
make[2]: *** [src/CMakeFiles/simdutf.dir/simdutf.cpp.o] Error 1
make[1]: *** [src/CMakeFiles/simdutf.dir/all] Error 2

cstring is a standard C++ header. Your system is broken. It is likely that you need to remove Xcode and reinstall it. Sorry.
Cheers!!!

Reinstalling XCode command-line tools fixed it, thank you!


Vaccum map into HomeAssistant

Hello,
would it somehow be possible to get the Vaccum map via the API into home assistant? Or toggle the cleaning of certain areas via the api?
Thanks and best wishes,
Timo

Currently, there is no API available to retrieve the map or specify cleaning areas.
It would be helpful to vote for the following issue:
https://github.com/OpenWonderLabs/SwitchBotAPI/issues/325

Thanks I will do that 👍


Can we please have functionality for ticket type:Billed/Non-Billed and ticket status changes.

It will be great if it has feature of ticket type "Billed/Non-Billed" and ticket status like::"open, in-progress, closed"

You can use "Custom Queries" on RTT's Issues > Search Options.

If you want to know how to create Custom queries, Please refer Custom queries.


⚠ Spryr has degraded performance

In 1059968, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 17416 ms

Resolved: Spryr performance has improved in 4a99db7 after 13 minutes.


Getting black screen when setVisibility GONE -> VISIBLE or INVISIBLE -> VISIBLE

I have the same question. For example, use Fragment, show () and or ()

and what about issue? I have same problem.

the same problem

Getting the same issue when I hide the cameraView on a button click and show it again on another click. Getting black screen and camera preview is not loaded. Is it a memory issue or device specific issue?
I am trying in Lenovo Tab, OS 5.1

Same here!!! Any plans to fix this?

the same problem

Same here!


Implement CloseNotifier interface

Hi! :smile:
Is it trivial to implement a CloseNotify method on h2quic.ResponseWriter? I'm asking in reference to this issue: https://github.com/mholt/caddy/issues/1643
If so, I could potentially submit a pull request, I just need to know if this would be a no-op like Flush... or if it's more complicated maybe I should leave it to you too; I'm not super well versed in quic's operational requirements.

Thanks for raising this! I think a "real" implementation might be a bit tricky, but definitely not impossible. But, as you're saying, I think it should be fine to just throw it in as a NOP for now (returning a new chan each time). We can then figure out the proper implementation later.

Sounds good to me, PR is currently finishing up tests. Kinda sorry to add to your list of things to do. 😇 Thanks for all the great work you do though!


Orientation change

In case that app could have different orientations supported we need to
prevent changing video orientation to support exclusive orientations
during 1 segment recording.
Also reset it after pausing segment.

+1 for this as this would prevent Video Preview Layer from rotating when recorder is in VC's locked to a particular orientation . But Instead of  [self currentOrientationSupportedByApplication]  I would have it ViewController specific.

the same problem !
When I changed the orientation of the device, the video will be paused !


Remove all the pre-1.0 Readme content

This removes all the pre-1.0 readme content, bringing the focus of the docs to "how to contribute". The official user/usage docs will live at buildkite.com/docs

:shipit:


Specify browser in env files

Apparently it seems that .env files are ignored when launching storybook.
Investigate if there are some known issues.

We are not lucky here @masmau .
This is a StoryBook bug, it simply ignores local env settings.
There have been several requests to implement this without luck, see for instance this and also this.
The only option is to try with this flag when starting storybook --ci                          CI mode (skip interactive prompts, don't open browser) and run it in the preferred browser using for instance a bookmark.

Storybook chooses the browser to use with better-opn package.
Set the BROWSER env variable with the name of the browser of your choice.
Set BROWSER=none to avoid opening in any browser.


It works but : Failed to mount component: template or render function not defined.

Sorry i'am new in vuejs and js ^^
I have the popup with payment that works but in console i have this error :
vue.esm.js?a026:591 [Vue warn]: Failed to mount component: template or render function not defined. found in ---> <VueStripeCheckout>        <Stripe> at src\components\Stripe.vue          <Shop> at src\components\Shop.vue            <App> at src\App.vue              <Root>
I have a file "Stripe.vue" with an integration of vue-stripe-checkout

And a basic implementation in a "Shop.vue" :

Do you have any idea why its works but i have this error ?
Thank you for the help and the package :)

My application works anyways, and yes i use vue-cli3.
I will try to understand your link ^^
Thank you for help and rapidity !

@SebastienGras no problem. I will try to solve this as soon as possible.

And just by the way you forgotten an ',' in your readme before closed function :)

Thanks! @SebastienGras


fixed mixed args issue and updated specs

This should allow mixed arguments in methods such as {{foo "bar" fred="wilma"}}. I have added the relevant spec but haven't tested all/extreme cases. All specs are passing 100%.


Update Node support for app and generated repos

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #431
Summary:
This commit updates the generator to not only use the current Node version setting in the package.json to populate the template files with the appropriate node version, but update to support the next LTS version using the new abstractNodeVersionNext variable.
The updated template files are done to reflect this policy https://auro.alaskaair.com/node-support
This PR also updates dependencies and some MAJOR updates were not addressed that are described in this issue

https://github.com/AlaskaAirlines/WC-Generator/issues/203

the packages for chalk, inquirer and latest-version packages have been updated to the most current version supported by the app.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[ ] Revision of an existing capability
[x] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[ ] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 4.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Compatibility with new version required~~~

^_^

Sorry for the delay, had a crunch to finish school work before spring break ended heh. I just pushed an update! :)


Release Event Hubs Track 2 Library Preview 4 for JavaScript

For general summary & goal, please see: Release Event Hub Track 2 Library (#442)
The work items linked to this issue herein represent the tasks needed to produce the 4th preview for the "track two" Event Hubs client library.
API Design Review with the Board

[x] Create and send batch

Features

[x] Act on feedback from API review board for Partition Processor #5103
[x] Last Enqueued Event Info for Event Processor users #5126
[x] Update Partition Manager to take namespace to ensure uniqueness #5145
[x] Remove explicit support for IotHub connection string

Diagnostics

[ ] 🏃Improved Logging story
[x] 🏃Tracing (non-EPH)

Stress Testing

[x] Set up basic send, receive operations in long running process across days to flush out bugs
[x] Refer to Track 1 Stress Test framework in Python, .Net and Java to come up with proposal

@azure/event-hubs version 5.0.0-preview.4 has been released:
<EMAIL_ADDRESS>View the changelog for user-facing changes.