🐛 [firebase_messaging] Neither getInitialMessage or onMessageOpenedApp is working

Bug report
Based on the documentation I called getInitialMessage() for when the app is opened from a terminated state and listened to the stream from onMessageOpenedApp for when the app is brought to foreground from background. I tested both the case but neither seems to be working as the getInitialMessage() returns null when the app is opened and no RemoteMessage is received in onMessageOpenedApp when the app resumes.
Steps to reproduce
Steps to reproduce the behavior:

Send data message
Tap on the local notification
App opens (RemoteMessage is null when app is opened/No RemoteMessage received in onMessageOpenedApp when app resumes)

Expected behavior
When app is opened : RemoteMessage should not be null.
When app is resumed : Stream from onMessageOpenedApp should receive the RemoteMessage.

Additional context
I am using flutter_local_notification to create the local notification when the app receives the data message.

Flutter doctor
Click To Expand
[✓] Flutter (Channel dev, 1.26.0-12.0.pre, on Linux, locale en_US.UTF-8)
    • Flutter version 1.26.0-12.0.pre at /home/ishwor/Workspace/Sdk/Flutter
    • Framework revision a706cd2112 (3 weeks ago), 2021-01-14 18:20:26 -0500
    • Engine revision effb529ece
    • Dart version 2.12.0 (build 2.12.0-224.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /home/ishwor/Workspace/Sdk/Android
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /home/ishwor/Workspace/Sdk/Android
    • ANDROID_SDK_ROOT = /home/ishwor/Workspace/Sdk/Android
    • Java binary at: /usr/lib/jvm/java-8-oracle/bin/java
    • Java version Java(TM) SE Runtime Environment (build 1.8.0_201-b09)
    • All Android licenses accepted.

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).

[✓] Connected device (1 available)
    • Redmi 5A (mobile) • a1868107d840 • android-arm64 • Android 8.1.0 (API 27)

! Doctor found issues in 1 category.

Flutter dependencies
Click To Expand
dependencies:
- firebase_analytics 7.0.1 [meta flutter firebase_core firebase_analytics_web firebase_analytics_platform_interface]
- firebase_core 0.7.0 [firebase_core_platform_interface flutter quiver meta firebase_core_web]
- firebase_crashlytics 0.4.0+1 [flutter stack_trace firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface]
- firebase_messaging 8.0.0-dev.14 [meta flutter firebase_core firebase_core_platform_interface firebase_messaging_platform_interface firebase_messaging_web]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_local_notifications 3.0.3 [flutter platform flutter_local_notifications_platform_interface timezone]
- khalti_resources 1.0.0 [flutter google_fonts i18n_extension nepali_utils]
- utils 1.1.0 [animations biometric_auth camera connectivity crypto device_info firebase_ml_vision flutter flutter_web_browser image_picker khalti_crypt khalti_plugins location logger package_info path_provider pretty_dio_logger shared_preferences tuple url_launcher]

dev dependencies:
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]
- mockito 4.1.3 [analyzer build code_builder collection dart_style matcher meta source_gen test_api]

transitive dependencies:
- _fe_analyzer_shared 7.0.0 [meta]
- analyzer 0.39.17 [_fe_analyzer_shared args charcode cli_util collection convert crypto glob html meta package_config path pub_semver source_span watcher yaml]
- animations 1.1.2 [flutter]
- args 1.6.0
- asn1lib 0.8.1
- async 2.5.0-nullsafety.3 [collection]
- biometric_auth 0.1.0 [flutter flutter_plugin_android_lifecycle intl platform]
- boolean_selector 2.1.0-nullsafety.3 [source_span string_scanner]
- build 1.4.0 [analyzer async convert crypto logging meta path glob]
- built_collection 4.3.2 [collection quiver]
- built_value 7.1.0 [built_collection collection fixnum quiver]
- camera 0.5.8+7 [flutter]
- characters 1.1.0-nullsafety.5
- charcode 1.2.0-nullsafety.3
- cli_util 0.2.0 [path]
- clock 1.1.0-nullsafety.3
- code_builder 3.4.1 [built_collection built_value collection matcher meta]
- collection 1.15.0-nullsafety.5
- connectivity 3.0.0-nullsafety.3 [flutter meta connectivity_platform_interface connectivity_macos]
- connectivity_macos 0.2.0-nullsafety [flutter]
- connectivity_platform_interface 2.0.0-nullsafety.1 [flutter meta plugin_platform_interface]
- convert 2.1.1 [charcode typed_data]
- crypto 2.1.5 [collection convert typed_data]
- csslib 0.16.2 [source_span]
- dart_style 1.3.6 [analyzer args path source_span]
- device_info 2.0.0-nullsafety.2 [flutter device_info_platform_interface]
- device_info_platform_interface 2.0.0-nullsafety.2 [flutter meta plugin_platform_interface]
- dio 3.0.10 [http_parser path]
- fake_async 1.2.0-nullsafety.3 [clock collection]
- ffi 0.1.3
- file 6.0.0-nullsafety.4 [meta path]
- firebase 7.3.0 [http http_parser js]
- firebase_analytics_platform_interface 1.0.3 [flutter meta]
- firebase_analytics_web 0.1.1 [flutter flutter_web_plugins firebase firebase_analytics_platform_interface meta]
- firebase_core_platform_interface 3.0.1 [flutter meta plugin_platform_interface quiver]
- firebase_core_web 0.2.1+3 [firebase_core_platform_interface flutter flutter_web_plugins meta js]
- firebase_crashlytics_platform_interface 1.1.6 [flutter meta collection firebase_core plugin_platform_interface]
- firebase_messaging_platform_interface 1.0.0-dev.10 [flutter meta firebase_core plugin_platform_interface]
- firebase_messaging_web 0.1.0-dev.5 [flutter flutter_web_plugins meta firebase_core firebase_core_web firebase_messaging_platform_interface js service_worker]
- firebase_ml_vision 0.10.0 [flutter]
- fixnum 0.10.11
- flutter_local_notifications_platform_interface 2.0.0+1 [flutter plugin_platform_interface]
- flutter_plugin_android_lifecycle 1.0.11 [flutter]
- flutter_web_browser 0.13.1 [flutter]
- flutter_web_plugins 0.0.0 [flutter js characters collection meta typed_data vector_math]
- glob 1.2.0 [async collection node_io path pedantic string_scanner]
- google_fonts 1.1.1 [flutter http path_provider crypto pedantic]
- html 0.14.0+3 [csslib source_span]
- http 0.12.2 [http_parser path pedantic]
- http_parser 3.1.4 [charcode collection source_span string_scanner typed_data]
- i18n_extension 1.4.6 [sprintf args flutter]
- image_picker 0.6.7+21 [flutter flutter_plugin_android_lifecycle image_picker_platform_interface]
- image_picker_platform_interface 1.1.0 [flutter meta http plugin_platform_interface]
- intl 0.17.0-nullsafety.2 [clock path]
- js 0.6.3-nullsafety.3
- khalti_crypt 1.0.0 [asn1lib pointycastle]
- khalti_plugins 1.0.0 [flutter]
- location 3.2.1 [flutter location_platform_interface location_web meta]
- location_platform_interface 1.1.0 [flutter meta plugin_platform_interface]
- location_web 1.0.1 [flutter flutter_web_plugins http_parser js location_platform_interface meta]
- logger 0.9.4
- logging 0.11.4
- matcher 0.12.10-nullsafety.3 [stack_trace]
- meta 1.3.0-nullsafety.6
- nepali_utils 2.2.1
- node_interop 1.1.1 [js]
- node_io 1.1.1 [node_interop path]
- package_config 1.9.3 [path charcode]
- package_info 0.4.3+2 [flutter]
- path 1.8.0-nullsafety.3
- path_provider 1.6.27 [flutter path_provider_platform_interface path_provider_macos path_provider_linux path_provider_windows]
- path_provider_linux 0.0.1+2 [path xdg_directories path_provider_platform_interface flutter]
- path_provider_macos 0.0.4+3 [flutter]
- path_provider_platform_interface 1.0.3 [flutter meta platform plugin_platform_interface]
- path_provider_windows 0.0.4+3 [path_provider_platform_interface meta path flutter ffi win32]
- pedantic 1.9.0
- platform 3.0.0-nullsafety.4
- plugin_platform_interface 1.1.0-nullsafety.1 [meta]
- pointycastle 1.1.1
- pretty_dio_logger 1.1.1 [dio]
- process 4.0.0-nullsafety.4 [file path platform]
- pub_semver 1.4.4 [collection]
- quiver 2.1.3 [matcher meta]
- service_worker 0.2.4 [js]
- shared_preferences 0.5.12+4 [meta flutter shared_preferences_platform_interface shared_preferences_linux shared_preferences_macos shared_preferences_web shared_preferences_windows]
- shared_preferences_linux 0.0.2+2 [file flutter meta path path_provider_linux shared_preferences_platform_interface]
- shared_preferences_macos 0.0.1+10 [shared_preferences_platform_interface flutter]
- shared_preferences_platform_interface 1.0.4 [meta flutter]
- shared_preferences_web 0.1.2+7 [shared_preferences_platform_interface flutter flutter_web_plugins meta]
- shared_preferences_windows 0.0.1+3 [shared_preferences_platform_interface flutter ffi file meta path path_provider_platform_interface path_provider_windows]
- sky_engine 0.0.99
- source_gen 0.9.6 [analyzer async build dart_style glob meta path pedantic source_span]
- source_span 1.8.0-nullsafety.4 [charcode collection path term_glyph]
- sprintf 5.0.0
- stack_trace 1.10.0-nullsafety.6 [path]
- stream_channel 2.1.0-nullsafety.3 [async]
- string_scanner 1.1.0-nullsafety.3 [charcode source_span]
- term_glyph 1.2.0-nullsafety.3
- test_api 0.2.19-nullsafety.6 [async boolean_selector collection meta path source_span stack_trace stream_channel string_scanner term_glyph matcher]
- timezone 0.5.9 [path]
- tuple 1.0.3 [quiver]
- typed_data 1.3.0-nullsafety.5 [collection]
- url_launcher 6.0.0-nullsafety.4 [flutter url_launcher_platform_interface url_launcher_linux url_launcher_macos url_launcher_windows]
- url_launcher_linux 0.1.0-nullsafety.3 [flutter]
- url_launcher_macos 0.1.0-nullsafety.2 [flutter]
- url_launcher_platform_interface 2.0.0-nullsafety.1 [flutter plugin_platform_interface]
- url_launcher_windows 0.1.0-nullsafety.2 [flutter]
- vector_math 2.1.0-nullsafety.5
- watcher 0.9.7+15 [async path pedantic]
- win32 1.7.4 [ffi]
- xdg_directories 0.1.2 [meta path process]
- yaml 2.2.1 [charcode collection string_scanner source_span]

Hi @Ishwor-Shrestha
Does onMessage and onBackgroundMessage work for you?
I assume this is happening on Android?
Thank you

Could you post your code on how you're setting up the listeners?

@Ehesp I've followed FlutterFire's documentation
    final initialMessage = await FirebaseMessaging.instance.getInitialMessage();

    _subscription = FirebaseMessaging.onMessageOpenedApp.listen((message) {
      // body
    });

And I am accessing these in initState.

@markusaksli-nc Yes onMessage and onBackgroundMessage works for me. And yes I am testing this in android. I was trying to access RemoteMessage when the app is opened after tapping on notification created using flutter_local_notification.
The push notification only contains data message, if this makes any difference

Could you provide a minimal complete reproducible code sample?
I'm not seeing this with the official example.

@Ishwor-Shrestha Can you also try removing any conflicting messaging packages for now, confirm it works, and then try again with local notifications. I suspect there is some native battles for the event going on.

@Ehesp I don't follow. You mean I try without local notifications to see if it works?

Yeah, strip it out of the native side of things and see if it works.

Hello,
I can confirm, since the new method "getInitialMessage" I can't deal with local notification anymore. The RemoteMessage returned is null.
Without Local notification it works as expected. Somebody has a workaround?


Cluster owners can't scale nodepool if the node template has been created by a different user

Bug
Steps to reproduce (least amount of steps as possible):

As user A, create nodetemplates, and create a cluster with nodepool using this nodetemplates
As user A, add user B as cluster owner
As user B, try to scale up the node pool

Result:
Error updating node pool scale: unable to find node template [cattle-global-nt:nt-u-t2k5pklxw3-nt-f9sw5]: undefined
Other details that may be helpful:
Environment information

Rancher version (2.3.6 and 2.4.2)
Installation option (single install/HA): HA
Kubernetes 1.17

gzrancher/rancher#10033

The bug is reproduced with Rancher server rancher\rancher:v2.3.6
Steps:

As user A, create node template. Create a cluster with node pool using new node template.
As user A, add user B as cluster owner
As user B, try to scale up the node pool

Result:
User B is unable to scale up node pool
unable to find node template [cattle-global-nt:nt-7s2qq]: undefined

The bug fix is verified with Rancher server rancher\rancher:v2.4-head version fb2c5fb2d
Reopening for a fix in master-head

https://github.com/rancher/rancher/pull/27959

The bug fix is verified with Rancher server rancher\rancher:master-head version 16616de9c


MLflow model cannot be served if storageUri doesn't end in "/"

/kind bug
What steps did you take and what happened:
[A clear and concise description of what the bug is.]
I set the storageUri (without /) as per the official documentation and tried to deploy the mlflow model, but it failed.
The error log is as follows:
2024-07-10 05:23:14,432 [mlserver.parallel] ERROR - An error occurred processing a model update of type 'Load'.
Traceback (most recent call last):
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/mlserver/parallel/worker.py", line 158, in _process_model_update
    await self._model_registry.load(model_settings)
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/mlserver/registry.py", line 299, in load
    return await self._models[model_settings.name].load(model_settings)
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/mlserver/registry.py", line 150, in load
    await self._load_model(new_model)
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/mlserver/registry.py", line 167, in _load_model
    model.ready = await model.load()
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/mlserver_mlflow/runtime.py", line 158, in load
    self._model = mlflow.pyfunc.load_model(model_uri)
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/mlflow/tracing/provider.py", line 237, in wrapper
    is_func_called, result = True, f(*args, **kwargs)
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/mlflow/pyfunc/__init__.py", line 1017, in load_model
    model_impl = importlib.import_module(conf[MAIN])._load_pyfunc(data_path, model_config)
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/mlflow/pytorch/__init__.py", line 718, in _load_pyfunc
    return _PyTorchWrapper(_load_model(path, device=device), device=device)
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/mlflow/pytorch/__init__.py", line 612, in _load_model
    pytorch_model = torch.load(model_path, **kwargs)
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/torch/serialization.py", line 997, in load
    with _open_file_like(f, 'rb') as opened_file:
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/torch/serialization.py", line 444, in _open_file_like
    return _open_file(name_or_buffer, mode)
  File "/opt/mlserver/envs/environment/lib/python3.8/site-packages/torch/serialization.py", line 425, in __init__
    super().__init__(open(name, mode))
FileNotFoundError: [Errno 2] No such file or directory: '/mnt/models/data/model.pth'

When I add / to the end of the storageUri set in the InferenceService yaml,  the inferenceService is able to start normally.
Example:
InferenceService yaml
storageUri: "s3://<uri>/model/"

However, this doesn't match what the official documentation says.
The official documentation: https://kserve.github.io/website/master/modelserving/v1beta1/mlflow/v2/#deploy-with-inferenceservice
What did you expect to happen:
I expected to be able to download the model files in the correct path without adding / to the end of the storageUri.
If this is not possible, I request that the official documentation be modified to inform users that they need to add / after the storageUri to get it to work properly.
What's the InferenceService yaml:
[To help us debug please run kubectl get isvc $name -n $namespace -oyaml and paste the output]
apiVersion: "serving.kserve.io/v1beta1"
kind: "InferenceService"
metadata:
  name: "<name>"
  namespace: "<namespace>"
  labels:
    <labels>
spec:
  predictor:
    serviceAccountName: <serviceAccountName>
    runtimeClassName: <runtimeClassName>
    model:
      modelFormat:
        name: mlflow
      protocolVersion: v2
      storageUri: "s3://<uri>/model"
      resources:
        limits:
          nvidia.com/gpu: 1

Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
I used kubectl exec -it <podname> -c kserve-container -n <namespace>  to get inside the container and viewed the contents of the downloaded model files.
I noticed that the path of the model file download did not match the path I had saved on MLflow.
Path to the model file saved in mlflow: <uri>/model/data/model.pth
model
|-data
| |-model.pth
|-<other files>

Path to the model file I actually found in the container: /mnt/models/model.pth
bash-5.1$ ls /mnt/models
data  *model.pth  <other files>
bash-5.1$ ls /mnt/models/data
<other files>
exit

When I add / to the end of the storageUri set in the InferenceService yaml, the path to the model file in the container becomes:
bash-5.1$ ls /mnt/models/
data  <other files>
bash-5.1$ ls /mnt/models/data
model.pth  <other files>

I think the key is in this code below.
https://github.com/kserve/kserve/blob/master/python/kserve/kserve/storage/storage.py#L226-L259
Changing this code above to the following might solve the problem.
Sample code:
diff --git a/python/kserve/kserve/storage/storage.py b/python/kserve/kserve/storage/storage.py
index fb9b8d30..8d2e1c02 100644
--- a/python/kserve/kserve/storage/storage.py
+++ b/python/kserve/kserve/storage/storage.py
@@ -233,36 +233,8 @@ class Storage(object):
             # Skip where boto3 lists the directory as an object
             if obj.key.endswith("/"):
                 continue
-            # In the case where bucket_path points to a single object, set the target key to bucket_path
-            # Otherwise, remove the bucket_path prefix, strip any extra slashes, then prepend the target_dir
-            # Example:
-            # s3://test-bucket
-            # Objects: /a/b/c/model.bin /a/model.bin /model.bin
-            #
-            # If 'uri' is set to "s3://test-bucket", then the downloader will
-            # download all the objects listed above, re-creating their subpaths
-            # under the temp_dir.
-            # If 'uri' is set to "s3://test-bucket/a", then the downloader will
-            # add to temp_dir: b/c/model.bin and model.bin.
-            # If 'uri' is set to "s3://test-bucket/a/b/c/model.bin", then
-            # the downloader will add to temp dir: model.bin
-            # (without any subpaths).
-            # If the bucket path is s3://test/models
-            # Objects: churn, churn-pickle, churn-pickle-logs
-            bucket_path_last_part = bucket_path.split("/")[-1]
-            object_last_path = obj.key.split("/")[-1]
-
-            if bucket_path == obj.key:
-                target_key = obj.key.rsplit("/", 1)[-1]
-                exact_obj_found = True
-            elif bucket_path_last_part and object_last_path.startswith(
-                bucket_path_last_part
-            ):
-                target_key = object_last_path
-            else:
-                target_key = obj.key.replace(bucket_path, "").lstrip("/")
-
-            target = f"{temp_dir}/{target_key}"
+            target_key = os.path.relpath(obj.key, bucket_path)
+            target = os.path.join(temp_dir, target_key)
             if not os.path.exists(os.path.dirname(target)):
                 os.makedirs(os.path.dirname(target), exist_ok=True)
             bucket.download_file(obj.key, target)

Environment:

Istio Version: v1.21.0
Knative Version: v1.13.1
KServe Version: v0.12.0
Kubeflow version:
Cloud Environment: k8s_istio
Minikube/Kind version:
Kubernetes version: v1.29.3

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."


is it necessary to do sanitization of user input? and when?

I am new to nosql. When I looking at the generated resolver, none of the user input is sanitized.
When I search google for a "NoSQL injection", some results came back, but all of the articles are not clear enough.
When I open the AWS support ticket, they said :

I understand you are trying to utilise resolver mapping templates to sanitise inputs to prevent nosql injection. Please let me know if my understanding is incorrect.

Currently, AppSync hasn’t a feature to automatically identify, mitigate or avoid SQL injections. As you may know, the only means that we have now is sanitise the inputs via regex or use procedure and parameterised inputs.  Please see the documentation [1] for a few examples and suggestions to sanitise inputs.
As it shows you can use regex expressions to fine-tune the inputs. However this is a limited data massaging technique and does not really ensure prevention nosql injection.

You may use Lambda resolver as a proxy to detect any injection to DyanmoDB but that does add a layer of complexity, cost and possibly latency. It does sound like a good adition to the service therefore I have gone ahead and created a feature request on your behalf to the AppSync team on your behalf for AppSync built-in NoSQL injection detection.   However, we are not able to provide you an ETA or ensure that this feature will be implemented.

[1] Tutorial: Aurora Serverless  - Input Sanitization - https://docs.aws.amazon.com/appsync/latest/devguide/tutorial-rds-resolvers.html#input-sanitization

Since the link support team provide is for the RDS, it really doesn't answer my question.
Is it necessary to do sanitization of the user input? if so when?

@monaye What sort of input sanitization are you looking for?

Closing this issue due to lack of user response. Please feel free to comment on this if you're still stuck with this issue and we'll re-open this issue.


Replace 'Neutron' with 'Netron'

[Enter feedback here]
In the Explore your model section, replace Neutron with Netron, which is the correct name for the service netron.app

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

ID: b236fa85-23be-f5ec-5c5f-332937187f04
Version Independent ID: 781552b8-a796-a963-05d3-ed98f509dbd4
Content: Convert your PyTorch model to ONNX
Content Source: docs/windows-ml/tutorials/pytorch-convert-model.md
Product: machine-learning
GitHub Login: @QuinnRadich
Microsoft Alias: quradic

Fixed, thanks!


Fanout failure

Recently Fanout on-chain transaction started to always fail with the following reason:
ErrScriptExecutionFailed (RdmrPtr Spend 0,ValidationFailedV1 (CekError An error has occurred:  User error:
The provided Plutus code called 'error'.) ["fannedOutUtxoHash /= closedUtxoHash","PT5"])

I suspect that I somehow messed up my privnet setup, but could not identify any changes. So this is rather a call for help.
I have a docker compose spinning up a cardano-node instance and 2 hydra-nodes. It is the same setup as in hydra-poc demo reduced to just Alice and Bob.
The API interactions proceed like this:

Init is sent to Alice's node
Commit is sent to Alice's node
Commit is sent to Ben's node
Close is sent to Alice's node
ReadyToFanout is received
Fanout is sent to Alice's node, which causes the aforementioned PostTxOnChainFailed

Here are full logs from both nodes:
hydra-fanout-failure-logs.tar.gz
Docker images used:
inputoutput/cardano-node@sha256:3bffeccf986986cece4d21c1901b564a3180880e136b0e221caa13e2757c751d
ghcr.io/input-output-hk/hydra-node@sha256:7a117b734e3a8af6fff7372b1d361a9d0d968e558d943999908e15a63bcbcf1c

What version of hydra-node do you run? The provided docker image hash is not one of the released tags, so likely you have been using the latest docker image?

After updating to ghcr.io/input-output-hk/hydra-node@sha256:4ea632938973394c057ef297ff794de318cf123436e33bcb314cd05d9ab2af27 fan-out validation still fails but the error is now lower bound validity before contestation deadline which is very suspicious because the Fanout was issued after receiving ReadyToFanout from API server.

Okay. sha256:4ea632938973394c057ef297ff794de318cf123436e33bcb314cd05d9ab2af27 is the current latest from our current master version.
If you want to have a working setup, the latest tagged version 0.5.0 should work.
Besides that, we can try to solve your issue as the latest error with a negative remainingContestationPeriod sounds interested!

Yes, I think it was the latest as of this Monday. I'll retry with the current latest and also 0.5.0, and report the full logs.
The reason for using latest instead of 0.5.0 is unrelated to this issue: I experimented with the new NewTx CBOR submission, so this image was just lying around.

With 0.5.0 fan-out works as expected.
Here are the node logs for sha256:4ea632938973394c057ef297ff794de318cf123436e33bcb314cd05d9ab2af27 with negative remainingContestationPeriod:
hydra-fanout-failure-logs-4ea63293.tar.gz

Do you have also the cardano-node logs? I'm afraid we do not log the current observed slot on the chain so it's a bit tricky to debug.
On alice's node:

2022-06-03T13:35:36.384661119Z the CloseTx gets posted with notAfter: 1441, that means slot 1441 + 10 seconds should be the deadline
2022-06-03T13:35:36.41938441Z we observe the same transaction as OnCloseTx with a remainingContestationPeriod":-6.5426332541904499e8
2022-06-03T13:35:36.419808352Z the node instantly decides it it ShouldPostFanout leading to a ReadyToFanout output just after
2022-06-03T13:36:02.778774773Z a client tries to Fanout
2022-06-03T13:36:02.811756172Z the corresponding FanoutTx fails with [\"lower bound validity before contestation deadline\",\"PT5\"]

My interpretation:
The time tracking in the hydra-node seems to be off in this scenario. If we would log slot information on posting/observing transactions that would be useful. We do a +100 slots when posting a close tx so that should happen around slot 1341, the node observes that tx and thinks it slot 1441 is -6.5426332541904499e8 seconds away, that is.. waaaay in the past.
Is this easily reproducible on your end? If yes, could you post your reproduction steps?

Waiting for 30 seconds didn't help.
Reproduction steps are quite awkward and not fully automatic. All I could come up with looks like this (mostly stolen from hydra-poc demo):
mre.tar.gz
tar xzf mre.tar.gz
cd hydra-fanout-failure
docker-compose --profile tui pull
./start-from-scratch

Then in separate terminals from the same hydra-fanout-failure directory invoke docker-compose --profile tui run hydra-tui-1 and docker-compose --profile tui run hydra-tui-2, and perform the above interaction (init, two commits, close, and fanout).

We have been trying to reproduce this after fixing #395 and we are unable to do so: we can correctly open/fanout a head within the demo using latest master's images (built from source tree).

@azaretsky Could you try with latest master? Doing a docker-compose --profile tui build in the demo/ directory.

Still the same failure along with the large negative value for remaining contestation period (-655477168.3246627). I can provide the logs, but they look the same.

Turns out this was caused by the major field of genesis-shelley.json being set to 0: It should be 6.


Game refuses to boot

Hey, I have installed JUST this mod, literally nothing else apart from SSE Engine Fixes and it's requierment. Using the latest build form March 16 and the game works fine WITHOUT the mod. But when I turn it on, black screen for like 1-2 seconds and then a instant crash to desktop. Am I missing something?

Here is the log if it helps, all tho nothing useful seems to be in it
ImprovedCameraSE.log

nvm i'll just fiddle with it more, i see a lot of people are having this issue too

It is not a lot of people it is just a random event occuring on certain PC setups for some odd reason and it has to be some external factor I am unaware of running in the background screwing with the hook. I'll be looking at it more intently once the initial NG release is out of the way as I need to anyhow!
All you can do for now is use iMenuMode=1 or iMenuMode=0

I would like to confirm that MenuMode=0 has worked (1 did not)

That basically confirms one of two things either Skyrim is inside of a protected folder or another app you have running in the background is doing DX11 hooking and screwing things up. As iMenuMode=1 it should be ignored. I know discord doesn't ignore it and injects itself into that window without doing a re-check what it is injecting into!
I'll have to do some research how to block other apps hooking it.


Updated AlienSpy.py with new variants

Added support for config.pl-version of 2.2, added amarillo-variant.

Sorry, new commit fixes stuff.


Add Goat Cash (GOAT) and VolAir (VOL) ERC20 Tokens

Hi BRD team,
Just wondering if we can have Goat Cash and VolAir wallets? What do you require?
Here are the details of each token.
Goat Cash (GOAT)
Contract: https://etherscan.io/token/0x9f452e458b024e82d6e3ff50a07b8de74c988523
Decimals: 18
Image: https://cdn.discordapp.com/attachments/482114378779590656/482150420207108096/goatcash-logo512x512.png
VolAir (VOL)
Contract: https://etherscan.io/token/0x33c52ee4bd563a26071619bc6f70146295febe8d
Decimals: 18
Image: https://cdn.discordapp.com/attachments/502222665751330826/502222842784514068/volair-image1024x1024-padding.png
Many thanks.

Please send requests to add tokens to<EMAIL_ADDRESS>


Turn on coverage reporting

we should be showing coverage deltas and striving to keep them in a range.

No comments. Closing. Re-open if needed.


Create the id generator capability for Denmark

Proposed Changes

Add a new capability: Generate identifiers based on citizen data
Add the generator for Denmark

Denmark Generator

All digits, expect the 8th and 9th digits, can be directly defined based on the citizen data
For the 8th and 9th digits, we must find combinational sum (taking into account the multiplication factors) that makes the id a valid one.

@AlexOlival @JoaoFSCruz Please consider if this change should be brought to the php version as well

@AlexOlival @JoaoFSCruz Please consider if this change should be brought to the php version as well


Support for Ensto's own thermostat and boost function to lower temperature when nobody is home

Hi,
First of all, thank you for the excellent work you've already done.
I'm currently using the Ensto mobile app with the schedule function and when I know that my kids are not home, I have manually lowered the room temperature using the boost function for a set period (negative boost). Basically I have fixed boost values (-2deg / 2hrs)) that I activate when I know that the kids are not at home and in return I deactivate it when the kids come back home. Of course as the BT cannot carry 3 stories, this means that I always have to walk next to the Enstos to do this.
I wish I could automate this function from Home Assistant as I already have the kids location known (home / away) in HA. I believe that this could be done if I would have access from Home Assistant to the boost function (on/off, target temp. and boost time) or the HA's thermostat card would use Ensto's room temperature. This function would also be very useful for lowering the inside temperature when we're not at home for awhile.

Hi,
That is already possible. I improved instructions to explain how thermostats boost offset can be set. See Example 1.
Unfortunately that doesn't work from Home Assistant directly, but you need to have ESP32 board with ESPHome and this esphome-ensto.yaml between Home Assistant and Ensto's thermostat to control it.


Bluetooth: Mesh: various fixes

Cherry-picks various fixes, among others:

suspend/resume functionality
removing 20ms tx delay in adv bearer
support for randomly delaying publications
docfixes and refactoring

Is this commit missing: https://github.com/zephyrproject-rtos/zephyr/commit/3d37549bac9?

Is this commit missing: zephyrproject-rtos/zephyr@3d37549bac9?

Hmm, shouldn't it be the first one in the list?
How I see it in zephyr's git log:

362924a6939 Bluetooth: Mesh: Use ATOMIC_DEFINE instead of atomic_t
3d37549bac9 Bluetooth: Mesh: allocate mesh max required buffer number
c3b7159cdf7 tests/bluetooth/mesh: Enable for native_sim
07848a4456b Bluetooth: Mesh: Fix integer overflow


Release Google.Cloud.Kms.V1 version 3.8.0

Changes in this release:
Documentation improvements

Update comments (commit 38d35cd)
Minor formatting (commit 5f3c5d6)
Minor formatting (commit 21481f5)

Created release for Google.Cloud.Kms.V1-3.8.0


slurmctld charm should configure a sane value for theTaskPlugin slurm configuration parameter

Bug Description
The TaskPlugin configuration is not handled by the charms and requires a custom user-supplied configuration to configure.
To Reproduce
na
Environment
na
Relevant log output
na

Additional context
The TaskPlugin is a configuration parameter we set as custom user supplied config for each deployment.
The slurm charms could configure TaskPlugin=task/cgroup,task/affinity if is_container() is False and TaskPlugin=task/affinity otherwise to provide a sane default value that corresponds with theProctrackType configuration.

@jamesbeedy question - does task/cgroup require us to ensure the hybrid cgroups are disabled on the nodes?

@jamesbeedy question - does task/cgroup require us to ensure the hybrid cgroups are disabled on the nodes?

no


[suggestion] add text sub-field in subtitles field of JSON

Hello,
Some sites (kuwo, neteasemusic, qqmusic which youtube-dl supports) doesn't have a url to download subtitles/lyrics but the subtitles/lyrics's content is just part of the HTML page or a field of API response.
Those 3 extractors parsed the subtitles/lyrics of the 3 sites but put it into the description field as subtitles property only have url field rather than text field or similar things.
Adding a new sub-field text (which can be exclusive with url sub-field) allow extractors to put parsed subtitles directly into the data if subtitles content is located in a field of the API response rather than another URL.
If the owner believes it's OK, I'll do some development and file a PR to update those three extractors.

This is already implemented. Provide concrete example URLs when subtitles are not extracted.


Consider allowing custom messages when started as an actor.

One of the improvements mist made a while back with its web socket processes was the ability to send custom messages to the socket's actor. This meant you could properly do pubsub things and have some other process push events to clients.
I think it would be really great if filespy had similar functionality. Concretely, I'm experimenting using filespy in lustre's dev tools to add hot reloading to its development server. I have things working (pending that PR you merged recently being published) but I have to set up a separate actor that sits between filespy and connected web socket clients. This is because although the filespy actor can have custom state, there's no way to communicate to the actor besides change events. I currently have something like:
|---------|              |-------------|                 |--------------|
|         |              |             |<-- connection --|              |
| filespy |--- events -->| coordinator |                 | websocket(s) |
|         |              |             |---- events ---->|              |
|---------|              |-------------|                 |--------------|

I could cut out this coordinator actor entirely if I had a way to tell filespy that a new client has connected. In mist they support it like this:
pub type WebsocketMessage(custom) {
  Text(String)
  Binary(BitArray)
  Closed
  Shutdown
  Custom(custom)
}

I think the easiest change would be to just expand the Change type with a second Custom variant – it'd make the actor stuff much more useful!

So in effect this'd allow you to send a custom message to the same actor that handles file changes, right? Sorry, as you can probably tell I haven't touched this in a while so I'm a bit rusty on the details.
I have no opposition to this, I made filespy more as a proof of concept to see if I could hack hot code reloading into gleam, and if this is useful for that goal then great :)
PS: Life has gotten in the way (I started a new job + I'm trying to finish my thesis at the same time) and I haven't touched this in a while, and I don't want to have to be blocking your progress with changes I'm going to take forever to get around to, so if you'd like I can transfer ownership of the repo to you (especially since you helped me set this up in the first place :p)

I don't want to have to be blocking your progress

No worries at all! We're all doing this as a hobby after all ^.^ Good luck on your thesis 💕

Fixed by #4. I've created a release @hayleigh-dot-dev, hope it's useful!


Nicknaming others is temporary

When a player who receives a nickname from staff reconnects they lose said nickname.

no longer an issue, closing


Enhancements to WebcamVideoStream (VideoCapture Properties and Release)

Address a few limitations with WebcamVideoStream:
1.) Set resolution of the VideoCapture, or use a capture object with custom properties already set.
Based on feedback provided in #68 , the constructor has been improved to allow optional args for a VideoCapture object and a resolution tuple.
https://github.com/jrosebr1/imutils/pull/68#issuecomment-543759112
2.) Address #35 by releasing the VideoCapture when the WebcamVideoStream is stopped.
Thank you for the time you've already spent developing this awesome library.

@jrosebr1 Please consider this PR when you have time. Let me know if you have any feedback on it. Thank you!

Not the maintainer, but thanks for this! I would suggest default setting the resolution to something like (10000, 10000), as this will force OpenCV to use the highest possible resolution of the camera, which it doesn't do by default.

Not the maintainer, but thanks for this! I would suggest default setting the resolution to something like (10000, 10000), as this will force OpenCV to use the highest possible resolution of the camera, which it doesn't do by default.

@makeworld-the-better-one I like your idea, but I'll leave it up to the maintainer to decide how they want this to work. I defaulted the resolution here to (320, 240) because that is the default in the other existing classes VideoStream and PiVideoStream.

Thanks for the changes!

@ariG23498 please consider this PR to fix a couple issues that are frequently reported by other users.


[z3] Binary with dependency to source

Hi,
for my usecase, I prebuilt binaries of Z3 and distribute them among my team members using vspkg export.
We now noticed that these binaries include references the actual source directory, which is not included in the export.
Consequently, some components of Z3 do not work as expected.
For instance, we are getting the following error because the buildtree is not present at this machine.
ASSERTION VIOLATION
File: /home/pgrulich-ldap/dependencies/vcpkg/buildtrees/z3/src/50ecd99ceb-ccf6d18521.clean/src/ast/bv_decl_plugin.cpp
Line: 863
m.has_plugin(symbol("bv"))

Proposed solution
I tried to investigate the problem and noticed that the CMAKE of Z3 is setting the Z3_COMPONENT_EXTRA_INCLUDE_DIRS variable.
In my case its set to:
Z3_COMPONENT_EXTRA_INCLUDE_DIRS:  /home/pgrulich-ldap/dependencies/vcpkg/buildtrees/z3/x64-linux-dbg/src; /home/pgrulich-ldap/dependencies/vcpkg/buildtrees/z3/src/50ecd99ceb-ccf6d18521.clean/src
We could maybe solve the problem by including the source dir in the final vcpk package and passing setting Z3_COMPONENT_EXTRA_INCLUDE_DIRS correctly. However, I'm not sure how to do these changes to the z3 port as I just tried vcpkg for the first time.
I hope the issue description is clear enough.
Thanks a lot for your help,
Philipp

Hi @PhilippGrulich, can you please test PR https://github.com/microsoft/vcpkg/pull/18104, and if this issue still exist, please provide a sample project to reproduce it.
Thanks.

We don't have enough information to solve this issue, unfortunately; if you have any more information to help us solve this, please reopen!


Add AOT Compatibility

We've been using UFX.Relay and have a technical spike where we've created a tunnel which runs on a Linux distro. We created a second technical spike to see if we could use AOT to create a native compilation and our solution generated some warnings pointing back to UFX.Relay. We realised that the the same fixes we applied to our solution also solve the issue for UFX.Relay.
Adding <IsAotCompatible>true</IsAotCompatible> (see docs) to UFX.Relay.csproj causes the following errors to be generated:
TunnelBuilderExtensions.cs(33,16,38,11): error IL3050: Using member 'Microsoft.AspNetCore.Builder.EndpointRouteBuilderExtensions.MapGet(IEndpointRouteBuilder, String, Delegate)' which has 'RequiresDynamicCodeAttribute' can break functionality when AOT compiling. This API may perform reflection on the supplied delegate and its parameters. These types may require generated code and aren't compatible with native AOT applications.

TunnelBuilderExtensions.cs(33,16,38,11): error IL2026: Using member 'Microsoft.AspNetCore.Builder.EndpointRouteBuilderExtensions.MapGet(IEndpointRouteBuilder, String, Delegate)' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. This API may perform reflection on the supplied delegate and its parameters. These types may be trimmed if not directly referenced.

These errors are resolved by adding: <EnableRequestDelegateGenerator>true</EnableRequestDelegateGenerator> to UFX.Relay.csproj

That's great thanks @HowardvanRooijen I'll review and merge in accordingly.

Serendipitously this video was released this week, which gave me the clue I needed: https://www.youtube.com/watch?v=N-MrQeZ1enY


About HMDB51 Dataset

Thank you for your great work.
Can you provide HMDB51 Dataset，because I can't download from official channels. Thanks.

Sorry for the very late reply. We upload the JHMDB dataset here

Sorry for the very late reply. We upload the JHMDB dataset here.

I am glad to see your reply.


Install ruby-augeas from RubyGems on Debian for Ruby >= 2.0.0

Ubuntu's PPAs don't include a ruby-augeas package for newer versions of Ruby. This patch installs ruby-augeas from RubyGems and the necessary dependencies.

:beers:

puppe-homes isn't the place for this - this problem is too generic.
I would suggest that we need to move this to puppet-puppetversion as it is a problem the puppet installation itself not with this specific module

That's fair. I'll reimplement this there.


llvm4: rc4 -> release!

Motivation for this change
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Wrapping up some of the testing currently, but almost identical to rc4 to submitting optimistically.  Will update checklist above as jobs complete.

(LGTM on Linux and NixOS)


Using 204 instead of 200

To save bandwidth/serverload/etc. as well as just generally be more semantically correct, divolte should return a 204 response code rather than a 1x1 pixel GIF. At the very least, this should be a configurable option.

I find it hard to estimate the possible effects of changing the response. I'd expect browsers not to have any issues with this, but I'm not sure. At this point it seems that all major trackers (including GA) use a HTTP 200 1x1 pixel response instead of HTTP 204, but I haven't been able to figure out why.

Google does do 204's, both with DoubleClick and just on their search home page (when I go to the Google home page it actually calls out to https://www.google.com/gen_204 ;-), as does ScoreCard Research, Krux, Addoox, NavEgg, LiveRail, etc.
It's actually pretty broadly employed at least in the DMP/HTTP data platform industry. When it isn't used, it is generally because a) someone didn't think about what the right response code would be, b) 204's didn't make as sense back in the old days when the response actually had to be rendered, c) misguided concerns about REST client frameworks that expect a 200 and content/content-type.
In divolte's case, it is controlling both sides of the protocol, so I would think it'd make sense for 204 to be the default behaviour, but I'll go for just having it as an option.

When trying a site with GA, I am seeing this:

I'm not against using a 204, but I'd like to research the situation a bit more. It wouldn't surprise me if the client side actually makes the decision on which endpoint to call. For example, some trackers will use a POST request through XHR if available and GET otherwise.
Forcing it all onto a 204 as a configurable option would be a trivial enough change, but it would be a shame to have to deprecate it at some point if it turns out to be not entirely right.
Is the lack of this option a pressing issue for you or blocking production deployment?

GA is a different story, because they've got backwards compatibility needs from hell (all those "utm" parameters date back from when it was Urchin!).
This isn't blocking deployment. It's just making for a reason to prefer a different solution.
I wasn't sure if the divolte implementation had some need to get content back or if it expected a content-type or some such. If it doesn't, I'll just patch it to do the 204 and be done with it.

Right, got it.
We're currently working on a major overhaul of both the way configuration is implemented and the internal threading + event passing (on dev/many-to-many). The idea is that we'll support multiple sources and sink location in one collector instance for use cases like multi-tenancy and eventually server side events. Picking this change up right now would probably make it quite painful to merge at later stage. Would you mind if we push it back a bit?

Makes sense to me. I was going to implement it as another handler that was available and let the path from the client side dictate the response.

I just ran some tests with a 204 response against all the browsers we test against, and it turns out this basically works fine. The main sharp edge is that this typically triggers the 'onerror' handler instead of 'unload' for images.


Unable to ssh with tailscale

Please ask, I have enabled tailscale ssh on one of the headscale clients, and headscale is also configured with ACL. I don’t know why it still prompts Permission denied (tailscale)
acls:
  - action: accept
    src:
      - "frank"
    dst:
      - "*:*"
      - "frankxin:*"
  - action: accept
    src:
      - "frankxin"
    dst:
      - "frankxin:*"
ssh:
  - action: check
    src:
      - "frank"
    dst:
      - "*:*"
      - "frankxin:*"
    user:
      - "some-allowlisted-user"
    checkPeriod: 8h
  - action: check
    src:
      - "frankxin"
    dst:
      - "*:*"
      - "frank:*"
    user:
      - "some-allowlisted-user"
    checkPeriod: 8h
disableIPv4: false
randomizeClientPort: false

Version of headscale used: v0.21.0
Version of tailscale client: 1.38.2
OS (e.g. Linux, Mac, Cygwin, WSL, etc.) and version: macOS Ventura

Hi, I don't think that Headscale supports the ssh like Tailscale now, just use ACLs and normal ssh login should work. Let me know !

This is related to #661 :)


Build step error for helloworld example

Bug Report
Version
examples v0.1.0 (tonic/examples)
├── tonic v0.2.0 (tonic/tonic)
├── tonic-health v0.1.0 (tonic/tonic-health)
│   └── tonic v0.2.0 (tonic/tonic) (*)
│   └── tonic-build v0.2.0 (tonic/tonic-build)
└── tonic-build v0.2.0 (tonic/tonic-build) (*)

Platform
Linux arch0 5.6.7-arch1-1 #1 SMP PREEMPT Thu, 23 Apr 2020 09:13:56 +0000 x86_64 GNU/Linux

Linux Distribution: Arch Linux
System protoc version: libprotoc 3.11.4
Description
Had build.rs issues when using tonic in an application. Wanted to check if it is my code issues or system setup issues, hence tried with helloworld example in tonic which had the same build step error.
$ cargo run --bin helloworld-server
   Compiling tonic-health v0.1.0 (tonic/tonic-health)
   Compiling examples v0.1.0 (tonic/examples)
error: failed to run custom build command for `examples v0.1.0 (tonic/examples)`

Caused by:
  process didn't exit successfully: `tonic/target/debug/build/examples-8e69c115720c8c30/build-script-build` (exit code: 1)
warning: build failed, waiting for other jobs to finish...
error: failed to run custom build command for `tonic-health v0.1.0 (tonic/tonic-health)`

Caused by:
  process didn't exit successfully: `tonic/target/debug/build/tonic-health-0f64809ad5dfe82e/build-script-build` (exit code: 1)

Attempts

Looked at couple of issues (#102 ), OUT_DIR variable is being set right, dir exists with writable perms.
Tried setting PROTOC, PROTOC_INCLUDE to system packages which didn't work.
Able to compile using prost-build in build step (Just messages obviously).
Ran build multiple times with RUST_BACKTRACE=full, RUST_LOG=trace - absolutely no extra output.

I expected to see: Some error message on what is going wrong, apart from build command failed with exit code 1.

Never mind
rustup component add --toolchain stable rustfmt solved it. Toolchain had to be specified explicitly, I thought it would pick the default one


[HTML5] Fetch API now passes credentials.

Used default value before, i.e. "same-origin", now uses "include" (i.e. include for cross-origin if cross-origin is allowed).
Fixes #54447

Thanks!

Cherry-picked for 3.4.3.

This is causing problems for me. Is there a way to change it back to the default?

This is causing problems for me. Is there a way to change it back to the default?

This will be reverted soon, see https://github.com/godotengine/godot/issues/58615.


Is it possible to close InAppWebView ?

Is there a method that allows the closure of the InAppWebView widget ?

I resolved inserting, in InAppWebViewOptions, the option clearCache setted to true.


[nanoflann]Add new port.

Related issue: #8784. There are no features of this port need to test.

/azp run

LGTM, thanks for the PR!


Add simple serialization for datatypes

@vyzo more work inc on actually serializing to bytes, but this is a start

This approach is reasonable, and handles Artefacts and Entities fine.
I guess for that for the chian cells we create subclasses that have the correct CBOR type set -- we can add the subclasses in Types I guess, otherwise they will have to live at the interface barrier.

It is a little problematic with the Chain cells, as the Index constructs new instances directly to add the chain pointer on top of the metadata and the artefact/entity reference.
Perhaps we can have a constructor method in ChainCells to create the appropriate type from the subtypes.

I reverted the merge to let you work out the rest of the details, don't kill the branch!


feature request for adding snow condition

After slack chat discussion at udacity lyft-challenge channel. Look like it is a good feature if CARLA can support snowy day condition.

I think we should be adding general road conditions such as: rain, snow, ice, etc. I am adding it to our roadmap to do more realistic control simulation. Thanks!

how about effect of rain, snow and fog on lidar returns?

Added to 0.9.12 sprint.

@berkerlogoglu, @hc167 Hi, we keep in mind to do that. But now we are working to set the new Sky atmosphere in CARLA. As German said we will add to 0.9.12. sprint.

Hi, as 0.9.12 released, just wanted to ask if the weather features (rain, snow, ice, etc) are added? I am asking because I couldn't find any information about weather features update on the release page. Thanks.

I also wanted to ping this issue, since affected lidar/radar measurements would be really useful

I also wanted to ping this issue, since affected lidar/radar measurements would be really useful

as 0.9.13 released, Has it been resolved？

Still no snowy weather yet - CARLA 0.9.15


add index on builds.created

Used for reporting - e.g. where created > $some_epoch to quickly and cheaply limit querying recent build metrics.

i think all your comments were addressed @wass3r


onScrolledToBottom called just once.

Thank you for sharing your addon.
In my implementation, which is basically a copy of the example at http://offirgolan.github.io/ember-light-table/, "onScrolledToBottom" is called just once when the component first loads, but it doesn't get called again on scroll (although "canLoadMore" is true after the first fetch of records).
Are there any more configurations needed for the infinite scroll to work?
I didn't see any in the example or docs.
Thank you very much!

onScrolledToBottom should be called irregardless of canLoadMore. The action originates here and is propagated through the body to your controller / component.
You have to implement the canLoadMore behavior yourself, as it is done in the example:
onScrolledToBottom() {
  if(this.get('canLoadMore')) {
    this.incrementProperty('page');
    this.fetchRecords();
  }
}

Having said that, are you sure that onScrolledToBottom is only called once? It should be called every time the {{lt-infinity}} component scrolls into the viewport.
Can you maybe share your code so we can step through it? :)

@buschtoens thanks for your response. That was my understanding. My code is here: https://gist.github.com/betaSocialCode/1d5703746a096ccec8f571ec84745151
Thanks again!

Sorry for being so persistent, but sometimes it's just the most trivial stuff going wrong. Did you add a console.log('pizza'); right in line 102 to verify that onScrolledToBottom actually isn't called again?
If so, something with lt-infinity has to be going wrong. Maybe play around with the scrollBuffer? Are you doing anything funky with CSS?
Can you maybe isolate a complete working (or not working for that matter :smile: ) example in a repository? Otherwise it's pretty much impossible for us to track this bug down. :)

I sure did try to consol.log. onScrolledToBottom is definitely not called again. I'll look into it more or create an example you can step through. Thanks!

@betaSocialCode if you are able to create a simple reproduction app, please share it here and Ill look into it more.
Also, are you on version 1.x?

@offirgolan , yeah, I'm on 1.0.1. Thanks!


feat: generate components using graphql

Description of changes

Issue #, if available

Description of how you validated changes
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing this PR since the changes landed in https://github.com/aws-amplify/amplify-cli/pull/12973


[tcgc] clean up deprecation exports of previous version

resolve: https://github.com/Azure/typespec-azure/issues/1080

All changed packages have been documented.

:white_check_mark: @azure-tools/typespec-client-generator-core

Show changes
@azure-tools/typespec-client-generator-core - breaking ✏

clean up deprecation exports of previous version

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


Work item type change can create invalid link

The enforcement rules for setting up links do not cover workitem type changes. For example:

Create a workitem of type user story
Create a workitem of type bug
Set a link whereby the bug work item blocks the user story work item
Change the type of the bug work item to user story
The link remains in place

Sorry - this link is deleted when the work item is saved.
Can we add a warning message when the user changes the work item to a type that will invalidate the links?

That would be a great idea. A couple of solutions come to mind.
1.Check the link when the user clicks on the checkmark button?

We provide an in line error directly under the field, that remains until
the field is cleared or a valid link is entered.

If we can detect an invalid link on typing into the field, we don't
enable the checkmark button until the link is valid and provides an inline
message to that effect under the field.

I have a password entry model in mind here, for an example...
-Monica

On Mon, Dec 5, 2016 at 10:49 AM, Len DiMaggio<EMAIL_ADDRESS>wrote:

Sorry - this link is deleted when the work item is saved.
Can we add a warning message when the user changes the work item to a type
that will invalidate the links?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/almighty/almighty-ui/issues/433#issuecomment-264889772,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARTwq8wCLAXjge_-dl5fBF88NRycMTytks5rFDKmgaJpZM4LEYSM
.

Solving this on the database layer might also be an option so that a trigger function will be called BEFORE the update to the WI is made. The trigger function would then check if the update of the WIT violates any existing links.

@ldimaggi What would you like to see? Shall an already existing link be automatically deleted, invalidated, rendered as invalid? See also https://github.com/almighty/almighty-core/issues/550

I would block the user from making a work item type change that would create a situation where either links are deleted or where links that cannot be created elsewhere in the UI are created. Can we warn the user and then have him/her delete the links before changing the work item type?

I would agree with Len that prevention is a better path. We can use the
inline messaging, under the field to communicate out these message to the
user.
On Mon, Dec 5, 2016 at 12:59 PM, Len DiMaggio<EMAIL_ADDRESS>wrote:

I would block the user from making a work item type change that would
create a situation where either links are deleted or where links that
cannot be created elsewhere in the UI are created. Can we warn the user and
then have him/her delete the links before changing the work item type?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/almighty/almighty-ui/issues/433#issuecomment-264927060,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARTwq4FnTgKc3nDJ7tbu14MVQ38ben24ks5rFFDvgaJpZM4LEYSM
.

"First, do no harm"   ;-)
Seriously, let's prevent the user from creating a broken link.
-- Len
On Tue, Dec 6, 2016 at 8:07 AM, MonicaGran<EMAIL_ADDRESS>wrote:

I would agree with Len that prevention is a better path. We can use the
inline messaging, under the field to communicate out these message to the
user.
On Mon, Dec 5, 2016 at 12:59 PM, Len DiMaggio<EMAIL_ADDRESS>wrote:

I would block the user from making a work item type change that would
create a situation where either links are deleted or where links that
cannot be created elsewhere in the UI are created. Can we warn the user
and
then have him/her delete the links before changing the work item type?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/almighty/almighty-ui/issues/433#
issuecomment-264927060>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
ARTwq4FnTgKc3nDJ7tbu14MVQ38ben24ks5rFFDvgaJpZM4LEYSM>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/almighty/almighty-ui/issues/433#issuecomment-265145327,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnOPbR8n4VXA2QJCx0vJ1hL0bsQ6v0Jks5rFV4hgaJpZM4LEYSM
.

--
Len DiMaggio<EMAIL_ADDRESS>JBoss by Red Hat
314 Littleton Road
Westford, MA 01886  USA
tel:<PHONE_NUMBER>
cell:<PHONE_NUMBER>
http://www.redhat.com
http://community.jboss.org/people/ldimaggio

@kwk I think that the only way to truly protect the system is on the backend. We only improve the user experience by preventing the user from making mistakes in the UI. You can't actually protect anything in the UI.

@joshuawilson I'm working on this already in the backend.

#633 fixes this. @ldimaggi @naina-verma please verify.

@nimishamukherjee , is that bug fixed by disabling the WI state drop down?

@nimishamukherjee from my understanding yes.

@naina-verma, @kwk yes.

Change the type of the bug work item to user story

The second last step is not possible now.
The user can select the work item type only when creating a work item via the dialogue box.
After that, there is no option to change the work item type.
As the work item link types depend on the type, it would have the same options (associated with a work item type)

@ldimaggi please close this bug if you also find this bug is no longer possible.

@kwk ,
just a thought , Disabling the WI state is temporary solution . By, this we could not verify the solution is good to resolve the purpose of the bug.
For example , when the WI state feature would be enabled in future we need to track this issue again.
Thx !

@naina-verma the state (e.g. open, closed) of a work item has nothing to do with this bug IMHO. We will probably never want to enable the type switching feature for work items (if that is what you meant). Instead we more likely will write be creating workflows for particular movement from one type to another.


Gateway message channels

Reconsidering Gateway Architecture: Gateway Messaging
Defines GatewayMessageChannel for sending and responding to GatewayRequest messages between gateway components.
The message channel is bi-directional, consisting of a long lived sender for sending messages to a receiver (messaging hub) within the gateway. Responses are sent back via one-shot messages to the originator of the messaging sequence.
Application: We apply the message sender to webserver component of the gateway

@justinmoon, could you rerun this for me, if necessary? I see it failed to upload codecov results

Codecov Report
Base: 58.99% // Head: 58.97% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (ebc2b51) compared to base (da0dc1e).
Patch coverage: 75.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #768      +/-   ##
==========================================
- Coverage   58.99%   58.97%   -0.03%
==========================================
  Files         100      101       +1
  Lines       16019    16056      +37
==========================================
+ Hits         9451     9469      +18
- Misses       6568     6587      +19

Impacted Files
Coverage Δ

ln-gateway/src/lib.rs
41.55% <ø> (ø)

ln-gateway/src/messaging.rs
66.66% <72.72%> (ø)

ln-gateway/src/webserver.rs
63.15% <100.00%> (+2.04%)
:arrow_up:

fedimint-server/src/consensus/interconnect.rs
87.50% <0.00%> (-8.34%)
:arrow_down:

core/api/src/server.rs
39.95% <0.00%> (-0.61%)
:arrow_down:

modules/fedimint-mint/src/lib.rs
90.90% <0.00%> (-0.38%)
:arrow_down:

fedimint-server/src/net/peers.rs
91.20% <0.00%> (-0.30%)
:arrow_down:

core/api/src/encode.rs
13.64% <0.00%> (+0.13%)
:arrow_up:

client/client-lib/src/query.rs
94.73% <0.00%> (+3.15%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

This is much easier to review!

Just had to figure out what sequence I could have pushed these changes. Starting with messaging was not so obvious, smh


Make acoular_demo.py matplotlib independent + call in tests

modify acoular_demo.py such that it can also be executed if matplotlib cannot be imported (e.g. print mic geom as array, calculate max or sum of array), add it to tests

This would probably require complicated code in the demo. Maybe just check if matplotlib available and print out a meaningful message. There could also be another matplotlib-less demo.

Related to #233


Local run with gitlab.com working across the (Canonical/CloudBase/FINOS) team

This is completed, the finos-legend-bundle can be reliably deployed locally. We've also updated the local deployment guide with new information: https://github.com/finos/legend-integration-juju/blob/1de2c7b60289b0ff03a23f5e6a9a57c3e1be6bea/docs/deploy/local.md


close sidebar when opening full screen

Description
the sidebar will now close when the full screen is opened, trying to open it while full screen is up will take you to fullscreen, it will reopen when the fullscreen is closed.
Testing
tested on vscode

is there any interest in making the sidebar reopen to the current session?


Update README in the docs

[ ] Additional function parameters, change functionality or change defaults?
[ ] New estimator in cr.tl.estimators?
[ ] New kernel in cr.tl.kernels?
[ ] New gene trend model in cr.ul.models?
[ ] New plotting function in cr.pl?
[ ] Other?

Should reflect recent changes I made to the github README

amazing, thanks.


[project] Stock Prediction

Stock Prediction System through AI

Assign me with this issue

@Rajiv-paul definitely go ahead.

@Rajiv-paul I hope you are working on the project. Please let me know if you need help :)
Happy Hacktoberfest! 🎃

@Rajiv-paul I hope you are working on the project. Please let me know if you need help :)
Happy Hacktoberfest! 🎃

Started 😊


Sending Group Invites

Good morning everyone,
I made a small bot to enhance the size of our group.
My problem is,
If I use the steamGroups.inviteUserToGroup() function, it will invite a single person when I set the info for 1 player.
(i haven't invited 200 ppl yet.)
So i then started adding ' to the front and end of the SteamID-variable. This made me not crashing anymore but it wont invite.
steamGroups.inviteUserToGroup('doesntcarehere','76561198013129088'); <- works
steamGroups.inviteUserToGroup('doesntcarehere',inviteid); <- does nothing.
output inviteid -> '76561198013129088'
if i use the inviteid variable with having no ' at the end/beginning let's me crash the bot.
error 'TypeError: str.indexOf is not a function'

nvm fixed it.

All IDs need to be treated as strings (just like node-steam does) and hence you need to enclose them in '.


s3_hook_fixes_and_backward_compatibility

support airflow v1.9.0

Thanks @codeinvain! Assigning to myself for review.

LGTM


Code Completion in xml comments

XML tag Code Completion in XML Comments
Feature Request!
Add some code completion/intellisense for the supported XML tags inside of the /// comment blocks.
Hopefully also in other multi-line comment blocks the C# compiler recognizes as valid a XML documentation comment block
(supported comment blocks according to the documentation), but with the variability in what those can be & the infrequent usage they get I can understand skipping out on those.
Right now the intellisense only comes up with suggestions when in a cref=".." attribute or the like when you start typing out the name of a namespace/class/method/attribute/etc..
In fact the default code user-snippets in vscode seem to be disabled when in a comment line all together, which is odd since most language extensions still allow for user defined snippets to work in comments, but it's all in the implementation I suppose.
I have yet to see a request for this, the closest being: #8 .

This should already be implemented and fixed when the next version of the extension is published. For example, this test shows what will be available.

This is what it will look like:

You can try it out by installing the beta version of the extension and setting your omnisharp-path to latest: https://github.com/OmniSharp/omnisharp-vscode/wiki/Installing-Beta-Releases

Has this been released by now? It's still not there.

Yes, this was released several years ago.

Okay, so I'll have to create a new bug for this as it somehow got lost again.

Followup Question [this seems a place to anchor it... if it is already done, and I just mussed it in my search, perhaps an answer here will help others.... If not, then I can certainly create and appropriate issue...
If there any way for CoPilot to detect INCORRECT XML code comments... For example..

Create a Property with Getter; Setter;
Generate XML Doc Comments
Remove the Setter.

The summary comment is now wrong...


in Rest Framework integration, Requesting an access token API is not there in code

reference latest/rest_framework.html#requesting-an-access-token
this snippet will help you in tracking the bug.
response = requests.post(
    "https://adfs.example.com/adfs/oauth2/token",
    data=payload,
    verify=False
)

also, i have checked in the code itself.  especially urls.py  in that I did not find this URL for the token
django_auth_adfs/drf_urls.py

What?

@jamilnoyda "https://adfs.example.com/adfs/oauth2/token" is just an example of a url value. You should use the URL for your own ADFS server.


ICN001 and N817 don't play with each other

List of keywords searched

ICN001
N817

Minimal code snippets
test1.py
import xml.etree.ElementTree as Et

test2.py
import xml.etree.ElementTree as ET

Invoked commands
ruff check test1.py test2.py

Current ruff settings
select = ["ICN001", "N817"]

Current ruff version
ruff 0.6.0
Description
It seems like N817 wants me to import it as Et while ICN001 wants me to import it as ET. Perhaps it is ok that rules conflict in ruff. If so I guess this is not a bug.

I also prefer 1.
% echo "from xml.etree import ElementTree" | ruff check --select=ALL --ignore=D100,F401 -
-:1:23: ICN001 `xml.etree.ElementTree` should be imported as `ET`
  |
1 | from xml.etree import ElementTree
  |                       ^^^^^^^^^^^ ICN001

% echo "import xml.etree.ElementTree as ET" | ruff check --select=ALL --ignore=D100,F401 -
-:1:8: N817 CamelCase `ElementTree` imported as acronym `ET`
  |
1 | import xml.etree.ElementTree as ET
  |        ^^^^^^^^^^^^^^^^^^^^^^^^^^^ N817

(1) sounds good to me too

Wow! You are so responsive in this repo. Hats off!

@MichaReiser I still see N817 CamelCase ElementTree imported as acronym ET on v0.6.1. If I understand https://github.com/astral-sh/ruff/pull/12922 correctly, it should now be import ElementTree as XET? However, the autofix of ICN001 still autofixes to import ElementTree as ET. Do I miss anything?

I have never seen XET in the wild and the docs/tutorial recommends ET.
https://docs.python.org/3/library/xml.etree.elementtree.html#tutorial

@hofbi / @cclauss: We are definitely not recommending XET in this rule! 😄
The tests added in #12922 are specifically testing that the "conventional aliases" setting can be used to configure this rule. One fixture is being run with the "conventional aliases" setting being set to "xml.etree.ElementTree": "XET", and the snapshot you see as a result shows you what would happen if you ran Ruff with that (strange) configuration

Ok makes sense for XET. Then do I have to configure anything so that both check play nicely together. Right now I still see the N817 error after the autofix adding ET.

There's a separate fix for from ... import that has not been released yet https://github.com/astral-sh/ruff/pull/12946
If the incompatibility that you're seeing isn't related to from ... imort, please open a new issue and share a code example with us. Happy to take a look.

Oh I missed that one. My issue is definitely https://github.com/astral-sh/ruff/issues/12940. So just waiting for the next release to get this fixed, thanks.

No worries. I missed that one too when implanting the fix 🥲


Reorder sheet (and list) rows

In my usage of Depot, I use the ordering of the rows in sheets and lists.
For this use-case, it would be desireable to add the capability to reorder the rows.
This could be achieved by:

(simplest) "move up" and "move down" button in the row, near the "delete row" one.
(probably more difficult) Some drag&drop support.

At the moment, the only way I can reorder the rows is by opening the file as a text file and editing it.
A nice extra would be to have in the Depot Specification something specifying that the order of elements in the file match the order that they are presented in the extension. This would formalize the contract.

This is a great idea! I think in scope for now is definitely some sort of move up/down arrow near the "X" for removing a line. An issue right now in "enforcing" order in the VSCode extension context is that when the data travels to the extension host and back during the serialization process, the data seems to get arbitrarily rearranged. I tried to do some order enforcement stuff before but could never really get it to work properly, with Code's side of things being a bit of a black box.
Regardless, getting some movement arrows in would be great!


[Final] 6 digits car number is not supported in registration

i have an old car with 6 digits in the license plate and i am not able to register.

Although this is not the definition of the project, as written on the user manual
we will change this
@dshames  good luck :)

thanks.

@OmriBS we support 6-7 digit cars. Can you send a picture with the details you tried to register with?

sorry, i've accidentally put 5 digits number. my bad.


Allow click at sub header to zoom to second if min zoom is set

JIRA: PLAS-3092
Overview of PR
as a developer, i want sub header to be clickable to zoom in to second unit
and also show the date format at the main header as 'dddd, LL, HH:mm:00' format.

c


multiArchiveTag is not passed along to api when specified during archive creation.

It looks like multiArchiveTag and streamMode are not passed correctly to the api when creating an archive, due to valid_opts removing them. This does not match the documentation that says these are valid options.
We currently monkey patch OpenTok to solve this issue for us, but would like to fix the upstream issue here.
I've opened an associated PR that fixes this issue.

Hi @DropsOfSerenity. Thanks for opening this issue, and for the PR. I'll review the PR and hopefully should be able to get it merged soon.

Hi @DropsOfSerenity, this has been fixed in https://github.com/opentok/OpenTok-Ruby-SDK/releases/tag/v4.8.1, and is available on RubyGems.
Note: I made some additional ammendments after merging your PR which can be seen in this PR.
Specifically, I changed multi_archive_tag and stream_mode in valid_opts to snake case from camel case, since this is more idiomatic and also in keeping with the code style of the library (although the API expects the properties in camel case, the top-level keys get automatically converted here before the request is made by the Client object. I just thought I should mention this, since your PR used the camel cased versions, I assume your monkey-patch does too, so if you want to remove your monkey patch in order to use 4.8.1, you'll need to update the keys in the options hash to the snake cased versions wherever you call Archives#create. (I've also updated the docs comments accordingly, which I'm guessing is where you initially got the key names from for your PR).
Thanks again for raising the issue, and for submitting the PR.


Preserve user settings in universal variables

I think it's better to skip erasing user settings in universal variables, as it's a expected behaviour for the most software: https://github.com/gazorby/fish-exa/blob/19cc781c2f4e332093f0bf5e870a0e04ba57e08c/conf.d/fish-exa.fish#L69

We should only erase private variables prefixed with _.
What do you think?

Another thing, it's probably better to check if a user setting in universal variable exist before rewriting it: https://github.com/andreiborisov/sponge/blob/main/conf.d/sponge.fish Otherwise, you can possibly write on top of existing user configuration (for example, when the user transferred universal variables via dotfiles and setting up a new machine, variables will exist before the plugin installation in that case).

I have the same setup on my own plugin and was surprised by its behaviour😅 https://github.com/andreiborisov/sponge/commit/2ff7d9941b2095311771bb29942958e571e7080e

Hi @andreiborisov!

I think it's better to skip erasing user settings in universal variables

The main reason user settings are deleted on uninstall is to allow reseting the plugin to a clean state, from which you can start tweaking settings again if something went wrong. Also, why would user settings from an absent plugin would be useful if you explictly uninstalled it?
One the other hand, i agree that persisting them after plugin update should be the expected behaviour, which is not the case right now (and it might be the reason you opened this, sorry for that if it's the case :/):
https://github.com/gazorby/fish-exa/blob/19cc781c2f4e332093f0bf5e870a0e04ba57e08c/conf.d/fish-exa.fish#L64
Moreover, missing documentation about this doesn't help.

it's probably better to check if a user setting in universal variable exist before rewriting it

Sure. An option would be to use global scoped variables for default configuration (when no user configuration exists). This way users can use universal variables to override defaults and they would persist after plugin removal.

One the other hand, i agree that persisting them after plugin update should be the expected behaviour, which is not the case right now.

Yes, I believe fisher fires uninstall and install events on update and there is no way to distinguish an update from a manual uninstall.

An option would be to use global scoped variables for default configuration (when no user configuration exists). This way users can use universal variables to override defaults and they would persist after plugin removal.

That what we were using in pure prompt, but at the end we decided that it's not the best approach because:

It's often being confusing for the user that they can use universal variables, but cannot use global variables. User might wanna use different settings for different shell sessions.
If you'll check for existence of a universal variable in conf.d/fish-exa.fish and set a global variable in case there is no a universal one, the user will be required to use set with a -U flag to set their value, which they often forget (see 1). But even more important, it will require shell restart to take effect, which goes against description of universal variables in fish docs (instant effect across all fish sessions). So it's kinda counterintuitive.

tldr: Keeping in my mind these point, in my opinion, it's easier both from the point of implementation and the usage to check if a universal variable is set, and assign it the default value, if it's not (in conf.d/fish-exa.fish).

The main reason user settings are deleted on uninstall is to allow reseting the plugin to a clean state, from which you can start tweaking settings again if something went wrong.

This is something I'm worried about too, but I'm leaning towards pointing users to the documentation, where there is a clearly defined list of used variables. Perhaps a section with an instruction on how to clean all of them (set -e EXA_*) would be sufficient?

there is no way to distinguish an update from a manual uninstall

In fact, fisher emit a separate event for update, so you can have different logic for both update and uninstall.
Currently, __fish_exa_update is overriden with the same behaviour as default (uninstall then install), but we could fix it by skipping uninstall and just removing aliases.

perhaps a section with an instruction on how to clean all of the variables (set -e EXA_*) would be sufficient?

Your're right that common behaviour is to left user settings untouched, so mentioning this in the readme should be enough. Maybe reseting plugin state is a missing feature of fisher, it reminds me the reset command of the scoop package manager (Windows)

User might wanna use different settings for different shell sessions (so have both global and universal variables configured)

But even more important, it will require shell restart to take effect

Another pattern would be to just seperate "runtime configuration" from user configuration.
For example, we can use private, universal variables (starting with _) to store default configuration, and replace their value at runtime, with the equivalent public variables's values if they exists:
The plugin would comes with default settings, stored in private variables:
set -U _EXA_DEFAULT_OPTIONS "--group" "--header" "--group-directories-first"
set -U _EXA_LL_OPTIONS "--long"

And resolve user config at runtime:
function __fish_exa_config
    set -l options
    for a in $argv
        if set -q $a
            set -a options $$a
        else
            set a (string join "" "_" $a)
            set -a options $$a
        end
    end
    echo $options
end

alias --save "$name" "exa (string split ' ' -- (__fish_exa_config EXA_STANDARD_OPTIONS EXA_LL_OPTIONS)"

In addition, we could change the uninstall function to erase global variables only, this way one can decide to configure the plugin only using global vars in config.fish (let's say, for testing purposes) so uninstalling it would reset the state, but only for the current shell.
Combined with runtime configuration this approach would have the following advantages:

User have full control over configuration scoping (can be either global, universal or a mix of both)
Any change takes effect immediately
Using global variables only, it's possible to reset the plugin back to its initial state (only in the current shell instance) just by reinstalling it.

What do you think?

In fact, fisher emit a separate event for update, so you can have different logic for both update and uninstall.

Ah, so it was fixed, nice, I wasn't aware!

Maybe reseting plugin state is a missing feature of fisher, it reminds me the reset command of the scoop package manager (Windows)

Knowing Jorge's love for simplicity, we shouldn't hold our breath on this one😁

Another pattern would be to just separate "runtime configuration" from user configuration.

That's absolutely a valid option. I think it comes down to personal preferences here, since it would be behave in a similar way for the user.
One thing to note however, this approach seem to require more maintenance from the plugin author compared to simplistic "just read the variable". So it wouldn't be my first option.

Using global variables only, it's possible to reset the plugin back to its initial state (only in the current shell instance) just by reinstalling it.

I'm not sure why would someone want to reset the plugin to its initial state for the current shell only. If you worried about validity of user configuration state, you can use your runtime approach to check if the configuration is valid and set the default values for invalid parameters, possibly with a message for the user.

What do you think?

I will be happy with either approach🙌🏻 (as long as the configuration stored in universal variables is persisted, so I can throw it in my dotfiles).

Hi @andreiborisov!
A bit late but user-defined variables are finally preserved as of ad73006f309e2455fa140049cc23fcf6be28f34d, enjoy customization!

Not late at all — still using your plugin. Thank you for the update!☺


Add Whalebird

unofficial Mastodon client
They have a deb available from their website.
https://github.com/h3poteto/whalebird-desktop/releases/download/4.5.4/Whalebird-4.5.4-linux-x64.deb

Closed via #144


sync from master.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@wangfakang: PR needs rebase.

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

Hi @wangfakang. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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


New releases clarification

Cher Arnaud.
Unless I'm mistaken, there is no more updates of Kiwi source code on this github since September 25th.
The last update on Google Play today announce the integration of Axie Marketplace and other updates as integrated telemetry to GeometryOU that is not in the code here.
My question: is Kiwi still open source and free?
From where comes the code for Google Play ?
What is the roadmap?
Thank you.

Hello,
There is no telemetry added ? I'm not sure what made you think so.
Perhaps what you have seen are the remainders of the translated text inside the app where the word Google was replaced by Geometry OU ?
Just to clarify, there is no reason (and most of all no use) to have data collected and it could be just a remainder of the original Chromium UI, since the business model of Kiwi is just to convince you to use Bing or Yahoo search.
About Axie Marketplace, Ronin Wallet and others, there was a bug when opening new windows from a background page that affected many extensions, and it's essentially to explain with super simple words what extensions APIs have been fixed.
If you want to practice to build your own Kiwi, you can take a look the original /src repo, it even has builds done directly on GitHub publicly and a full transparent build process (it was even built directly from GitHub to Play Store) and you'll get a really good idea of how it works.
/src.next is built on a private server, and occasionally code pushed here.
From time to time work is done directly on the repo here (when I sync with d3ward for example) but it's still very rare as it takes a bit of extra time and there is very little amount of devs interested in helping unless they are paid.
For the 2022 roadmap, it's still being defined right now.

Thank for your answer and your work.
Please, can you tell which version is currently available on Google Play and where can we check the source code with same commits to the released version?


Build is failing due to Received status code 502 from server: Bad Gateway

Build passed successfully now.


Pass by reference to const

What is gained from passing bool, int and float by reference? Isn't it actually worse?

Probably related: https://stackoverflow.com/questions/11932614/in-c-is-it-bad-to-pass-a-const-bool-by-reference

For anyone interested, here's a simple regex to try to find pass by const but not reference

Thanks!


Overpass Area ID isn't read from File in Map.tsx

Fix as discribed in title

fixed with rework of geocoding in next pull


Fix HTTP/2 test cleanup with LocalChannel

Motivation:
When a LocalChannel is closed it is responsible to ensure all queued objects are released. When a LocalChannel is closed it will also close its peer channel. However in HTTP/2 unit tests we may not wait until all channels have completed the shutdown process before destroying the threads and exiting the test. This may mean buffers are GCed before they are released and be reported as a leak.
Modifications:

In HTTP/2 tests when we use LocalChannel we should wait for all channels to close before exiting the test and cleaning up the associated EventLoopGroups.

Result:
More correct usage of LocalChannel in HTTP/2 tests.

this should help with #775

@normanmaurer - FYI

LGTM

4.1 (782b7bc)


Workflow to build, tag, and publish release assets

Workflow to build, tag, and publish release assets

Build and tag images with the release tag name
Update Helm chart files for Traction and Tenant-UI
Package Traction and Tenant-UI Helm charts
Push changes to the branch
Add assets to the Release

Need to clean up the PR


Subscription event payloads need to implement stubs for every field

Suppose you have a Subscription type in your schema like so:
type Subscription {
  fieldA: String!
  fieldB: String!
}

The type-erased object::Subscription interface will have resolvers that call field accessors for getFieldA and getFieldB. If you generate the schema with schemagen --stubs, it will implement placeholder stubs that throw std::runtime_error if you try to access either of those fields without defining the field accessor. However, if you don't specify --stubs, it will static_assert that all of the field accessors have been implemented.
When subscribing, the selection set can only ever have 1 field per event; see line 5 in Source Stream. So, the object::Subscription type needs to support subscribing to any field, but the payloads of the events can only ever access 1 of the fields. If you use the same type for multiple field payloads, essentially a std::variant of the fields, that's OK, but otherwise you either need to use --stubs (which makes it easier to forget/mismatch a field accessor signature anywhere else in the schema), or implement your own stubs for the fields that you are not using in the subscription event payload.
The current suggestion is to emit an additional event payload type for each Subscription field which only requires that field accessor to be implemented, or possibly a single payload type that includes stubs but has static_asserts that at least 1 field accessor has been implemented unless you pass --stubs for the global stub generation.

Fixed in the next branch for v5.


Title for login page is not localized

Describe the bug
We hardcode a value of "Login" for the page title here.
The assumption may have been that this page would only be shown prior to language selection. However, an applicant can:

Continue as guest
Choose another language
Navigate back to the front page (maybe by logging out).
The login page is rendered with localized text

We should likely localize this text, similar to the other applicant-facing page titles.
Noticed as part of #3545.

Currently the front page is translated, and the language selection page is not.
Should we even translate the front page? In my opinion, it makes sense for the front page + language selection page to both be in English. My expectation is that when I click logout, the website is returned to its default state (English).

It think we can best-guess user language even before they select language based on accept-language header. It provides languages user likely uses. For example here mine: accept-language: be,en-GB;q=0.9,en;q=0.8,en-US;q=0.7,ru;q=0.6,la;q=0.5 Which states that my 2 preferred languages are Belarusian and English GB (no idea why GB over US but ok). If do that than localizing first screen makes more sense.

https://user-images.githubusercontent.com/30369272/220974946-f3a5d781-cd58-41c4-b559-6968c318d7e9.mp4
It looks like we already pre-select the correct language for the user based on the accept-language header. This video demonstrates that with the ModHeader chrome extension.
Changing the content of the log in and language page based on this header (even before the language is confirmed) seems reasonable to me. Will have to check with PMs on whether we should proceed. #4269 just adds the log in title translation for now.


Fix typo

Overview

Fix typo in test_adapter.py file.
Checklist

[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[ ] README.md updated and added information about my change
[ ] I have run changie new to create a changelog entry

Please change commit message to Fix typo in test_adapter.py file


Update installation.md to correctly link to the quick tour file

Fixing link to the quick tour file in the installation instructions.
What does this PR do?
Update installation.md to correctly link to the quick tour file

Before submitting

[x] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

The current link works well I think, from trying in https://huggingface.co/docs/text-generation-inference/installation

Hi @Narsil & @osanseviero, sorry for the confusion, I was checking directly on github, not on the website. The link that gives a 404 without the .md is this one: https://github.com/huggingface/text-generation-inference/blob/main/docs/source/quicktour. But as this is the source for the website, adding the extension would break that link.


Improve upgrades

Description

Added upgrade action to splunk_installer resource
Fixed chef-splunk::upgrade recipe to actually upgrade splunk
Fixed issue #122 removed initial_captain reference from comment

Issues Resolved
#122
Check List

[X] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[X] New functionality includes testing.
[X] New functionality has been documented in the README if applicable
[X] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

@tas50 Can this be merged?

@tas50 @lamont-granquist can this be merged?

Shipped as 3.0. Thanks for bringing this cookbook back to life.


[Docs] Write Schema Object Guide

Description
Right now, we focus on field validation, but it would be nice to show how to use an object schema of Zod, like you might do with Formik, React Hook Form, or even TRPC.
This discussion of this question is shown here with an answer:
https://github.com/crutchcorn/houseform/issues/48
Describe the solution you'd like
No response
Describe alternatives you've considered
No response

In which document this issue should be resolve?

@Yasir761 this should be a new MD file that lives within docs/guides.
Then, once this MD file is written, you can add a link in the sidebar here: https://github.com/crutchcorn/houseform/blob/main/docs/.vitepress/config.ts#L70-L73


PUBKEY_SSH_FIRST_USER parameter

Would it be possible to add PUBKEY_SSH_FIRST_USER parameter like it is described here?
https://github.com/RPi-Distro/pi-gen/blob/f1c166a2833950a7c44fe19b01780723635a7aa3/README.md?plain=1#L185
Thank you...

Thanks for reaching out!
Definitely possible, some of the bits and pieces are already in place:
https://github.com/usimd/pi-gen-action/blob/6f631e5a090466b6b1c54a7b92e45699d9f00b85/src/pi-gen-config.ts#L25-L26
These would need to be exposed as inputs in the action.yaml and be mapped accordingly here https://github.com/usimd/pi-gen-action/blob/6f631e5a090466b6b1c54a7b92e45699d9f00b85/src/configure.ts#L46
Following the spirit of this action, some input validation should be added to fail fast. The syntax of the pubkey should be checked. The cipher used could be with a regex against a whitelist, to verify the public key, there is hopefully some dependency (like https://www.npmjs.com/package/ssl-validator, although at first sight there's no matching function).
Are you interested in creating a PR for this?

I'm really sorry, but I do not know the TypeScript :-(


NTR 'positive regulation of gene expression, epigenetic'

To be the parent of
'positive regulation of gene expression via CpG island demethylation'
'transcription initiation-coupled chromatin remodeling'
Since ''transcription initiation-coupled chromatin remodeling'' is  incorrectly the child of ''positive regulation of gene expression via CpG island demethylation''

+[Term]
+id: GO:0141137
+name: positive regulation of gene expression, epigenetic
+namespace: biological_process
+def: "An epigenetic process that increases gene expression at specific genomic regions through chromatin remodeling either by modifying higher order chromatin fiber structure, nucleosomal histones, or the cytosine DNA demethylation." [PMID:36618446]
+synonym: "activation of gene expression, epigenetic" EXACT []
+synonym: "up regulation of gene expression, epigenetic" EXACT []
+synonym: "up-regulation of gene expression, epigenetic" EXACT []
+synonym: "upregulation of gene expression, epigenetic" EXACT []
+is_a: GO:0010628 ! positive regulation of gene expression
+is_a: GO:0040029 ! epigenetic regulation of gene expression
+property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/26694" xsd:anyURI
+created_by: pg
+creation_date: 2023-12-22T13:13:03Z


test: put hubble tests on quarantine in GKE

Note for reviewers - review without whitespace changes, I wrapped one context in hubble test to not run BeforeAll and AfterAll blocks.

test-gke

Coverage increased (+0.002%) to 36.653% when pulling 0141c0ed88b26da9f96040a870bd64335346c934 on pr/quarantine-gke-hubble-tests into 66eb29ef800d92e308b40b37cb883bcb754f590b on master.

test-gke

test-4.9

Hubble tests got skipped, gke failure was caused by failure that got fixed by merging https://github.com/cilium/cilium/pull/12743 in the meantime, if reviews are good let's merge.

Merging this as this will unblock GKE pipeline.