e2e: add environment for kind

This commit adds a new implementation of the Environment interface
implemented using kind, ie Kubernetes in Docker. The motivation is that
you might want to manually run some complex configurations in the e2e
tests using Kubernetes resources and still manage the tests simply using
the e2e package.
Users who need the escape hatch of deploying things to a Kubernetes
cluster manually, e.g. workloads that are not a simple deployment, can
use the kubeconfig file located in the environment's shared directory.
Signed-off-by: Lucas Servén Marín<EMAIL_ADDRESS>

Despite what CI says, the tests on macOS are actually failing: https://github.com/efficientgo/e2e/actions/runs/4491794039/jobs/7900760164#step:6:194
Mounting the volumes is working seamlessly across all of the layers! However, there seems to be an issue with connecting to the NodePort service for pods. I am not familiar with how Colima/Lima forward ports and don't have a mac to test this. This works out-of-the-box on Linux, but it is also jumping one layer less:
on Linux:
host->"node"->iptables->container
on macOS:
host->Lima->"node"->iptables->container
I'm guessing, since I don't have a mac to test, but I image the issue is with getting packets from the host to the Lima VM, since the discovered IP address is the address of the "node" and not of the Lima VM, so the packets destined to the "node" can't be routed. Does anybody have a mac to help debug this? Maybe we can pair?

@squat let me try to run the tests on my Mac and report back 👍

Thanks @matej-g ⭐
We should also make sure that CI actually fails when the tests fail

Oh cool that you got that too work! I was reading that exact same blog last nicht at 3am hahaha
I have a suspicion that for a typical Lima/Colima setup we might be able to get away with only one (max two) things:

Add routes to the macOS host so that all packets destined for and IP in the CIDR of the Docker bridge in the Lima node go through the IP of the Lima guest
Not certain if it will be required, but if necessary, configure the Lima guest to forward packets (sysctl and/or iptables). This would mean issuing one single lima iptables ... command after colima start

It would be great to pair on this to see if it could be this simple.

I got the tests to pass on macOS! However, I was wrong, not only were those two steps necessary, but a third one was necessary as well!
PS: I was finally able to test this by running a macOS VM on my Linux machine, so the entire stack is:
Linux host -> macOS guest -> Lima guest -> "node" -> container :smiling_imp:

oki everything should be ready for now! I'll make a follow-up with some more documentation, if that works


Updated whitelist definition.

Signed-off-by: RaulSanchez<EMAIL_ADDRESS>

Build status:

Build status:

Build status:


Gateway API: no validation of RegularExpression in HTTPRoute

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
Here is my configuration. Istio doesn't reject it but RegularExpression makes HTTPRoute take no effect.
apiVersion: gateway.networking.k8s.io/v1
kind: HTTPRoute
metadata:
  name: hr
spec:
  parentRefs:
  - name: back
    kind: Service
    group: ''
    port: 80
  rules:
  - matches:
    - headers:
      - name: host
        type: RegularExpression
        value: "*.example.com"
    backendRefs:
    - name: ratings
      port: 8080

According to #49607, it is better to add validation for RegularExpression in HTTPRoute instead of only add warn in log to avoid confusion. For example, if I add RegularExpression in existing HTTPRoute, it won't trigger update and will keep on the old behavior.
Version
client version: 1.21.0
control plane version: 1.21.0
data plane version: 1.21.0 (2 proxies)

Client Version: v1.28.2
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.27.3

Additional Information
No response

This is what the gatewayapi should handle, you should raise this up there.
IIRC, gatewayapi use CEL for validatingwehbook.

I know. Thank you.

What exactly is invalid about this regex?


Nick: Watch coloriagewk.com

Nick requests the watch of the watch_keyword coloriagewk\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

coloriagewk\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


[0.74.0-rc.5] - Memory leak when using fetch with new arch

Description
Use the Fetch API and see memory consumption increase without the automatic cleaning of the garbage collector.
Related #42374 #39100 https://github.com/facebook/hermes/issues/1147
Steps to reproduce

Open Xcode
yarn install
yarn ios
Click on start
See the memory profile as in the demo video in the player

React Native Version
0.74.0-rc.5
Affected Platforms
Runtime - iOS
Areas
TurboModule - The New Native Module System
Output of npx react-native info
System:
  OS: macOS 14.3
  CPU: (8) arm64 Apple M1
  Memory: 103.22 MB / 8.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.11.0
    path: ~/.nvm/versions/node/v20.11.0/bin/node
  Yarn:
    version: 3.6.4
    path: ~/.nvm/versions/node/v20.11.0/bin/yarn
  npm:
    version: 10.2.4
    path: ~/.nvm/versions/node/v20.11.0/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.14.3
    path: /Users/migueldaipre/.rbenv/shims/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK:
    API Levels:
      - "33"
      - "34"
    Build Tools:
      - 30.0.3
      - 33.0.0
      - 33.0.1
      - 34.0.0
    System Images:
      - android-33 | Google APIs ARM 64 v8a
      - android-34 | Google Play ARM 64 v8a
    Android NDK: Not Found
IDEs:
  Android Studio: 2023.1 AI-231.9392.1.2311.11330709
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.10
    path: /usr/bin/javac
  Ruby:
    version: 3.3.0
    path: /Users/migueldaipre/.rbenv/shims/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.74.0-rc.5
    wanted: 0.74.0-rc.5
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: true

Stacktrace or Logs
no crash log. Run the reproduction demo, if you watch enough time, you will see it.

Reproducer
https://github.com/migueldaipre/RN74FetchLeak
Screenshots and Videos
https://github.com/facebook/react-native/assets/21174859/02dcdbdc-5edf-421c-b4b9-429ce55e2163

TurboModule - The New Native Module System

Is this related to the New Architecture?
Seems like a duplicate of https://github.com/facebook/react-native/issues/42374 https://github.com/facebook/react-native/issues/39100 no?

TurboModule - The New Native Module System

Is this related to the New Architecture? Seems like a duplicate of #42374 #39100 no?

Yes, this is happening with the old and the new arch.
Could you help us find a fix for the problem?
In apps that make many requests we have out of memory problems.

hello we got same here

hello we got same here on Android

Both platforms have been fixed in 0.75-rc3


Crash fixes

Crash fixes.

Coverage decreased (-0.5%) to 96.199% when pulling 2bb96b4df434d7059713e13adfc467be233f15d7 on tmp into b68c25884d6a880af7c6939d9bd09de7662659fa on master.

Coverage decreased (-0.5%) to 96.199% when pulling 2bb96b4df434d7059713e13adfc467be233f15d7 on tmp into b68c25884d6a880af7c6939d9bd09de7662659fa on master.

Coverage decreased (-0.5%) to 96.199% when pulling 2bb96b4df434d7059713e13adfc467be233f15d7 on tmp into b68c25884d6a880af7c6939d9bd09de7662659fa on master.


Failed to create new KafkaAdminClient

Im getting the following error when i try to run the kafka_spark_db_dag    in airflow
Traceback (most recent call last):
  File "/workspaces/sales-forecast-mlops-at-scale/services/airflow/dags/spark_streaming.py", line 110, in <module>
    stream_kafka_to_db()
  File "/workspaces/sales-forecast-mlops-at-scale/services/airflow/dags/spark_streaming.py", line 107, in stream_kafka_to_db
    write_df_to_db(processed_df)
  File "/workspaces/sales-forecast-mlops-at-scale/services/airflow/dags/spark_streaming.py", line 97, in write_df_to_db
    return query.awaitTermination()
  File "/workspaces/sales-forecast-mlops-at-scale/venv/lib/python3.10/site-packages/pyspark/sql/streaming/query.py", line 221, in awaitTermination
    return self._jsq.awaitTermination()
  File "/workspaces/sales-forecast-mlops-at-scale/venv/lib/python3.10/site-packages/py4j/java_gateway.py", line 1322, in __call__
    return_value = get_return_value(
  File "/workspaces/sales-forecast-mlops-at-scale/venv/lib/python3.10/site-packages/pyspark/errors/exceptions/captured.py", line 185, in deco
    raise converted from None
pyspark.errors.exceptions.captured.StreamingQueryException: [STREAM_FAILED] Query [id = 568bfb7c-0611-4ed0-9c8c-d5efb7a22068, runId = 54a32339-a883-4985-b4f8-11d1d58f4f0e] terminated with exception: Faile
```d to create new KafkaAdminClient`

First of all, after you have started up the system, can you go to kafka-ui at port 8800 and check if there are messages publishing to the main topic?
If no, there are some networking issues. Check the logs of kafka, kafka-ui, and data-producer.
If yes, then can you try to empty the directory /services/airflow/spark_streaming_checkpoints and rerun this DAG? I've just noticed that I didn't remove those checkpoint files. They might be the cause.

The messages are being published to the main topic and i am able to view the data in the rossman_sales in the database , i have deleted everything in this directory /services/airflow/spark_streaming_checkpoints the eror still remains  however i think there are files that are suppose to be generated in the /services/airflow/spark_streaming_checkpoints but the folder is remaining the same , maybe its the error

I guess it's about the file/directory permission then. If you're suspecting spark_streaming_checkpoints/ you can disable checkpointing by simply commenting lines 92-94 . It won't affect other parts, except when you rerunning this DAG, it will reread all the messages in the topic instead of where it left off last time. But it's fine for debugging.

I'm closing the issue since it seems inactive now. Please feel free to reopen again or open a new one if you have more problems.


Update of english documentation

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

There're a lot of issues, but I'll fix 'em separately before publishing.


Xiao C6 does not reapply Cooldown to skill.

Xiao's C6 resets the cooldown of his skill when triggered. Then he gets his 3 free skills. But after that, the cooldown is not reset, so he can use his skill a 4th time for free.
https://gcsim.app/v3/viewer/share/bcec1400-1723-4415-a954-9ac2bee1506f

This is working correctly.


Flutter Web: Opacity 0% does not always make its children disappear

There seems to be a rendering bug in Flutter Web:
In some scenarios, setting the opacity of the Opacity widget to 0 does not make its children disappear. Setting debugShowCheckedModeBanner to true or adding a Text widget like in the following code fixes this issue somehow.
Steps to Reproduce
import 'package:flutter/material.dart';
import 'package:flutter/widgets.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: false, // <--- true makes the bug disappear
      home: HiderDemo(),
    );
  }
}

class HiderDemo extends StatefulWidget {
  @override
  _HiderDemoState createState() => _HiderDemoState();
}

class _HiderDemoState extends State<HiderDemo> {
  bool _visible = true;

  @override
  Widget build(BuildContext context) {
    return GestureDetector(
      onTap: () => setState(() => _visible = !_visible),
      child: Column(
        mainAxisAlignment: MainAxisAlignment.center,
        children: [
          Opacity(
            opacity: _visible ? 1.0 : 0.0,
            child: Center(
              child: Container(
                width: 100,
                height: 100,
                color: Colors.white,
              ),
            ),
          ),
          //Text('asdf') // <--- Uncommenting this also makes the bug disappear
        ],
      ),
    );
  }
}

flutter run -d chrome
Click on the white box

Expected results: The box disappears and reappears when clicking it multiple times.
Actual results: Nothing happens.
One additional observation is, that the box is no longer correctly centered when resizing the window while the box is in the "hidden" state. That makes sense, because it should technically not be rendered now anyway.
Now set debugShowCheckedModeBanner to true and try again. Now everything works as expected.
Alternatively you can also uncomment the Text widget in the code sample. This will fix the rendering bug, too. Or you could add a Material widget around the HiderDemo widget.
But these workarounds don't always work. I have a more complex app where the bug appeared and there neither of these workarounds helped. I'm yet to find the reason for this.

Flutter Doctor
[✓] Flutter (Channel stable, 2.2.1, on Linux, locale de_DE.UTF-8)
    • Flutter version 2.2.1 at /home/marcus/flutter
    • Framework revision 02c026b03c (vor 3 Wochen), 2021-05-27 12:24:44 -0700
    • Engine revision 0fdb562ac8
    • Dart version 2.13.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /home/marcus/Android/Sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: /home/marcus/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /home/marcus/android-studio
    • Flutter plugin version 54.1.1
    • Dart plugin version 201.9335
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.53.2)
    • VS Code at /usr/share/code
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (3 available)
    • AFTB (mobile)               • <IP_ADDRESS>:5555 • android-arm    • Android 5.1.1 (API 22)
    • sdk google atv x86 (mobile) • emulator-5554      • android-x86    • Android 10 (API 29) (emulator)
    • Chrome (web)                • chrome             • web-javascript • Google Chrome 89.0.4389.72

• No issues found!

Hi @MarcusWichelmann T
I can't reproduce on web chrome on Linux and windows
Here I updated the code sample to make it easier to see white box against different color

minimal code sample
import 'package:flutter/material.dart';
import 'package:flutter/widgets.dart';
void main() {
  runApp(MyApp());
}
class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: false, // <--- true makes the bug disappear
      home: HiderDemo(),
    );
  }
}
class HiderDemo extends StatefulWidget {
  @override
  _HiderDemoState createState() => _HiderDemoState();
}
class _HiderDemoState extends State<HiderDemo> {
  bool _visible = true;
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.pink,
      body: GestureDetector(
        onTap: () => setState(() => _visible = !_visible),
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: [
            Opacity(
              opacity: _visible ? 1.0 : 0.0,
              child: Center(
                child: Container(
                  width: 100,
                  height: 100,
                  color: Colors.white,
                ),
              ),
            ),
            //Text('asdf') // <--- Uncommenting this also makes the bug disappear
          ],
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.2, on Linux, locale en_US.UTF-8)
    • Flutter version 2.2.2 at /home/taha/Code/flutter_stable
    • Framework revision d79295af24 (4 days ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /home/taha/Code/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /home/taha/Code/SDK
    • Java binary at: /home/taha/Code/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+0-b944-P17168821)
    • All Android licenses accepted.
[✓] Chrome - develop for the web
    • Chrome at google-chrome
[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1
[✓] Android Studio (version 4.2)
    • Android Studio at /home/taha/Code/android-studio
    • Flutter plugin version 57.0.2
    • Dart plugin version 202.8488
    • Java version OpenJDK Runtime Environment (build 11.0.8+0-b944-P17168821)
[✓] VS Code (version 1.56.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.23.0
[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Linux
    • Chrome (web)    • chrome • web-javascript • Google Chrome 91.0.4472.101
• No issues found!

Can you please details of your Linux distro? neofetch output

Hi @TahaTesser,
adding a Scaffold, like you did, also makes the rendering bug disappear. At least in this specific minimal sample. Have you run the code exactly as posted by me?
I'm on Fedora 33, but this is reproducable on any OS and in different browsers (Chrome, Firefox, Safari).

Also reproducable with Flutter 2.2.2.

I am able to reproduce in 2.2.2 on stable, Windows 10. master is currently blocked by #84671
The only comment I have, is the background in latest Chrome is already white, so it is better to have something other than that, e.g.

Code
import 'package:flutter/material.dart';
import 'package:flutter/widgets.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: false, // <--- true makes the bug disappear
      home: HiderDemo(),
    );
  }
}

class HiderDemo extends StatefulWidget {
  @override
  _HiderDemoState createState() => _HiderDemoState();
}

class _HiderDemoState extends State<HiderDemo> {
  bool _visible = true;

  @override
  Widget build(BuildContext context) {
    return GestureDetector(
      onTap: () => setState(() => _visible = !_visible),
      child: Column(
        mainAxisAlignment: MainAxisAlignment.center,
        children: [
          Opacity(
            opacity: _visible ? 1.0 : 0.0,
            child: Center(
              child: Container(
                width: 100,
                height: 100,
                color: Colors.red,
              ),
            ),
          ),
          //Text('asdf') // <--- Uncommenting this also makes the bug disappear
        ],
      ),
    );
  }
}

HI @MarcusWichelmann
I had to Scaffold to provide some background (my browser isn't grey as yours). and your widgets don't a single Scaffold, a normal app would at least one Scaffold in the entire widget tree, thanks for catching this issue.
Now I changed the container's color itself from white to red, I can see the same behavior without Scaffold and red container with debugShowCheckedModeBanner: false. It works if `debugShowCheckedModeBanner: true.
This also reproduces on the master

stable
master

❌
❌

Check flutter doctor -v outputs for each channel below

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.2, on Linux, locale en_US.UTF-8)
    • Flutter version 2.2.2 at /home/taha/Code/flutter_stable
    • Framework revision d79295af24 (5 days ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /home/taha/Code/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /home/taha/Code/SDK
    • Java binary at: /home/taha/Code/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+0-b944-P17168821)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 4.2)
    • Android Studio at /home/taha/Code/android-studio
    • Flutter plugin version 57.0.2
    • Dart plugin version 202.8488
    • Java version OpenJDK Runtime Environment (build 11.0.8+0-b944-P17168821)

[✓] VS Code (version 1.56.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.23.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Linux
    • Chrome (web)    • chrome • web-javascript • Google Chrome 91.0.4472.101

• No issues found!

[✓] Flutter (Channel master, 2.3.0-17.0.pre.366, on Pop!_OS 20.04 LTS
    5.11.0-7614-generic, locale en_US.UTF-8)
    • Flutter version 2.3.0-17.0.pre.366 at /home/taha/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db930ba32f (32 hours ago), 2021-06-14 18:29:03 -0700
    • Engine revision e1d2d4b264
    • Dart version 2.14.0 (build 2.14.0-210.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /home/taha/Code/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /home/taha/Code/SDK
    • Java binary at: /home/taha/Code/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+0-b944-P17168821)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 4.2)
    • Android Studio at /home/taha/Code/android-studio
    • Flutter plugin version 57.0.2
    • Dart plugin version 202.8488
    • Java version OpenJDK Runtime Environment (build 11.0.8+0-b944-P17168821)

[✓] VS Code (version 1.56.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.23.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Pop!_OS 20.04 LTS
      5.11.0-7614-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 91.0.4472.101

• No issues found!

✅ : No Issue ❌: Issue reproduced

Ah, you're right. I forgot that I had changed the body background in my index.html manually to gray. So the sample provided by @asashour is more fitting to reproduce this.
Good to hear that you could reproduce this.
Thanks!

Looking into this now. I have tried to repro the issue on other platforms (using the original repro code except changing the container to red):

On Android emulator the code works as expected
On Flutter desktop (-d macos) the bug is present, but in the opposite way: clicking the red container causes the entire screen to turn red

I think we can fix this bug in the Flutter Web side by just not painting at all when the opacity is 0, but it's odd that this is necessary.


Download button is enabled for failed uploads

I uploaded NaN which failed as expected.
The Download button is enabled in both the collection view and on the landing page, but the Explore button is correctly not available in both cases.

Actually this is a quick fix! I'll work on that now. Thanks Brian 🎉!

Actually this is a quick fix! I'll work on that now. Thanks Brian 🎉!


tox + GitHub actions

Another great addition. Thanks!
I've never used Github actions...

Github actions is slowly replacing good old travis ;)

From an integration standpoint, it's probably best to use the services that come the platform (Github)...


Component interface has "modifiedtime" capitals: bug? [fix is in 2005 backend only]

Component interface has "modifiedtime" field but Hybris sends us a field "modifiedTime". Is it a typo?

We generate the data interface from swagger. The ComponentWsDTO has modifiedtime, but the ComponentAdaptedData changes it to modifiedTime.
The best approach is to change the ComponentAdaptedData in cmsoccaddon. Or, we can change the SPA component model.

I created this ticket: https://jira.hybris.com/browse/CMSX-8953. If "modifiedTime" is changed back to "modifiedtime" in cmsoccaddon, then in SPA, we also need to change it back.

PR is here: https://github.com/SAP/cloud-commerce-spartacus-storefront/pull/2875

Backend changes were released in wcms-module-20.05.0-M20


quic-client fuzz: Assertion failed: !h->done_implicit

With the fix from #22615 applied, this file
crash-023364fbccd9d4db290b1a30e33461ea5c8b55e8.gz
still produces:
ssl/quic/quic_txp.c:294: OpenSSL internal error: Assertion failed: !h->done_implicit

This appears to be because we decide to use implicit length for stream data, but then continue to try to send more stream related frames for subsequent streams in the same packet. Fix on the way.


Remove OGV debug line

majorshill at slack, drew my attention to this.

Ah I have seen this, I have reject his PR as it will be in the next (big) one that I do which is coming soon

Thank you though :-)


Plugin is not pulling data from Strapi when there is a timestamp on the cache.

https://github.com/strapi/gatsby-source-strapi/blob/88d1fd57320bf89ee3b068a14f637aae770ed7a8/src/gatsby-node.js#L104
So I am having this issue when I run my app and the cache does have a value for the gatsby-source-strapi plugin.
The response returned from Strapi when a request with the updatedAt param is
{     "data": null,     "error": {         "status": 404,         "name": "NotFoundError",         "message": "Not Found",         "details": {}     } }.
In that scenario the newOrExistingEntries would be [ [] ] and the expression const data = newOrExistingEntries  || allResults;
will assign data a value of [ [] ], and this consists on not pulling any data from Strapi.
I think there must be a deep check for the existence of newOrExistingEntries when evaluating the value of data to except this case of newOrExistingEntries that passes the JS conversion to Boolean with the value [[]].

Thanks for your interest in this project. This plugin is moving into the Gatsby User Collective and this repo will be archived. Please open an issue in that repository, submit a PR if you'd like to see this implemented, or join us on Discord if you have questions!


After flutter upgrade throwing warning

The plugin advance_pdf_viewer uses a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to see if this plugin supports the Android V2 embedding. Otherwise, consider removing it since a future release of Flutter will remove these deprecated APIs.
If you are plugin author, take a look at the docs for migrating the plugin to the V2 embedding: https://flutter.dev/go/android-plugin-migration.

@pradeep-gc , please do a quick search before filing a new issue:
https://github.com/lohanidamodar/pdf_viewer/issues/83


Package manager

Tasks:

Package Manager UI:

[x] #1344: Package Manager Xaml
[x] #1342: Package List Xaml
[x] #1341: Package Details Xaml
[x] #1343: Available Packages ViewModel
[x] #1346: Installed Packages ViewModel
[x] #1347: Loaded Packages ViewModel
[ ] #1345: Specify global list of package repos
[x] #1348: Add Search Textbox
[x] #1406: Wire up Install/Uninstall buttons
[x] #1412: Always update Installed Packages and Loaded Packages lists

Closing since majority of items are complete. We can open another epic in 0.4 for #1488 and friends


chore: update go-to-protobuf usage

Related: https://github.com/akuity/kargo/pull/2046

This PR updates protobuf codegen to use the latest go-to-protobuf (v0.30.1).
The new codegen process is done by

Creating a dummy module in a temporary directory to provide isolated directory to the go-to-protobuf.
Use vendored proto files instead of generating from Kubernetes types

Reference:

https://github.com/kubernetes/code-generator/compare/v0.29.5...v0.30.1#diff-028a4953abcbbe704c55b50d01ff8001a791acb61504f409fcd748cf93c3658e

After looking at this, it's surfacing a concern I've had for some time -- that this process is becoming too complex. I don't believe that the creators of the various tools we're using ever intended for anyone to have to jump through this many hoops, so what is it that we are overlooking?

@hiddeco were the problems potentially solved by this PR solved differently through your recent work on managing tools better?

Not in full, but I have a local copy somewhere that incorporates what's being attempted in this PR on top of the changes from #2449, and I am planning to pick this up as part of #2573.

This has been addressed by #2573. (Thanks for the work nonetheless, @devholic 🙇)


Translated long text breaks the way the input component is displayed.

Overview
The long texts do need two lines to render and that pushes the number for the amount to be lower than it should be.

Fixed in #368


Possible Issue with TR64?

Is there a change within the Fritzbox TR64?
This Script stopped working for me.
I get the 404 Error Message from the Fritzbox but the log say:

time="2023-11-20T14:52:08Z" level=warning msg="Failed to poll WAN IPv4 from router" error="Post \"http://<IP_ADDRESS>:49000/igdupnp/control/WANIPConn1\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)"

Sure it can reach the FritzBox? That isn't a 404, but a connection timeout. Can you try with curl or so?

mh you are right, my bad id did only check it with my pc not my server. pc can reach but server not... they are on the same network and the server only cant reach the fritzbox... strange but not related to this script.
Thanks for the reply

No problem


Does this also control / remove Windows inbox apps?

Will a clean also remove Windows inbox apps such as Solitaire or Duolingo?

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

ID: f38c1219-3096-f435-13ae-1c325f9d2b82
Version Independent ID: 8f0002e6-00e0-eb00-b6b9-dcd16034c949
Content: CleanPC CSP
Content Source: windows/client-management/mdm/cleanpc-csp.md
Service: unspecified
GitHub Login: @nickbrower
Microsoft Alias: maricia

My experience with CleanPC has also been mixed. I think better documentation is needed to understand exactly what it is doing when triggered. By observation I believe it is going through a "Restore PC" routine but I can't find any documentation to back that claim.

It seems to execute the "reset this PC"... but after it does other provisioning tasks, so they get wiped out.  I can see it joins the machine to my domain (pc account appears) and runs some other tasks... then resets the PC and bam, you're back to OOBE and it's like nothing ever happened.

@officedocsbot assign @mypil

@domcote - thank you for submitting feedback.
I think the best way forward is if you can open a service ticket so this can get resolved ASAP. Based on outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for Windows 10 products:  https://support.microsoft.com/en-us/hub/4338813/windows-help?os=windows-10
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.

@officedocsbot close


DSD-949 - Heading add noSpace prop

Fixes JIRA ticket DSD-949
This PR does the following:

add a noSpace prop to the Heading component where if set, it sets margin-bottom to 0

How has this been tested?

Accessibility concerns or updates

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] I have added relevant accessibility documentation for this pull request.
[ ] All new and existing tests passed.

Front End Review:

[ ] View the example in Storybook

The usage of passing noSpace to the useStyleConfig function is the right approach. As it currently stands, passing this variable to that function does nothing because it's not set up in heading.ts.
You'll find that in heading.ts, the function is declared as
const Heading = {
  baseStyle: {
    // This is to help target custom anchor elements
    // passed as children to the Heading component.
    a: baseLinkStyles,
  },

This needs to be updated as
const Heading = {
  baseStyle: ({ noSpacing }) =>{
    // This is to help target custom anchor elements
    // passed as children to the Heading component.
    a: baseLinkStyles,
  },

and then you can add the mb conditional logic. This is so that we can keep the logic for styles in the them file rather than in the component file.
One minor nitpick is to use marginBottom instead of mb because not everyone might be use to the Chakra-style shorthand. I prefer the clarity over the brevity.

What seemed to work was removing "marginBottom" from the "margins" object on the top in order for it to be recognized in baseStyle function

This is good to go but the merge conflicts should be fixed.

Tugboat has finished building the preview for this pull request!
Link:

https://pr988-o1drzhobwiiuqaft8ukh0y6ekdlszrwi.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/62749b502e91387b1710f523

Tugboat has finished building the preview for this pull request!
Link:

https://pr988-o1drzhobwiiuqaft8ukh0y6ekdlszrwi.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/62749b502e91387b1710f523

Tugboat has finished building the preview for this pull request!
Link:

https://pr988-o1drzhobwiiuqaft8ukh0y6ekdlszrwi.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/62749b502e91387b1710f523


Is Crowdin still working?

Is-it me or it's been a long time since I've seen a Crowdin PR?

You are correct @g123k , I fixed a bug I introduced in July in the config file. Sync pending.


upgrade guava to latest version

What's the purpose of this PR
upgrade guava to latest version, the use of android flavor is to support for JDK 7
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Read the Contributing Guide before making this pull request.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit tests to verify the code.
[x] Run mvn clean test to make sure this pull request doesn't break anything.

Codecov Report

Merging #3277 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3277      +/-   ##
============================================
+ Coverage     51.48%   51.50%   +0.01%
- Complexity     2305     2307       +2
============================================
  Files           442      442
  Lines         13739    13738       -1
  Branches       1398     1398
============================================
+ Hits           7074     7076       +2
+ Misses         6176     6175       -1
+ Partials        489      487       -2

Impacted Files
Coverage Δ
Complexity Δ

...o/spring/annotation/ApolloAnnotationProcessor.java
93.93% <0.00%> (-0.18%)
9.00% <0.00%> (ø%)

.../framework/apollo/spring/property/SpringValue.java
89.47% <0.00%> (+1.75%)
21.00% <0.00%> (+1.00%)

...rk/apollo/spring/property/SpringValueRegistry.java
88.88% <0.00%> (+5.55%)
11.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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


template st2client conf

This moves the /root/.st2/config template (a bash heredoc) to values so that it can be adjusted based on install requirements. This also makes secrets in envFromSecrets available to the generate-st2client-config initContainers so that they can be used within the bash heredoc template. The template value is also passed through tpl to support helm templating for any non-secret config such as injecting the timezone.
Relevant st2 docs on the st2client config file: https://docs.stackstorm.com/reference/cli.html#configuration-file
Closes #233
Closes #256
Closes #261
Note: This PR is an alternate implementation of #256 and #261 based on @armab's feedback in #261.
I want to provide the ST2_API_KEY via envFromSecrets, but the /root/.st2/client config with a username takes precedence over the ST2_API_KEY env var. Templating the file allows me to either inject the api_key in the config file, or remove the credentials so that the ST2_API_KEY var is used.

@armab RFR


Autoremove empty rows from grid

When user removes all the cells from a grid row, it leaves a ghost 👻 row that's invisible in UI.
It should be removed since that was likely user's intent.

Hi @zvonimirfras ,
I would like to take this up. can you assign me this please ?

i would like to take a stab at this! would anyone be kind enough to point me to the right component?

Sure, thing, thanks for doing this, @JoelLau! 😃

i would like to take a stab at this! would anyone be kind enough to point me to the right component?

I just realized I haven't pointed you to the component 😅
It's a grid component. Grid component uses several components to function - Grid, Row, and Column.
Removing empty rows will likely be easiest to do in the AGrid component when the state is updated, but you'll know more when you poke around a bit.
Will this be enough to get you started, @JoelLau?
Feel free to reach out to me here, on discord (Zvonimir#9989), or anywhere else (I'm easy to find 🙂)

sorry! it sounds like enough. i'll start working on it tomorrow (:

took a while, but i'm not sure if there're edge cases i'm not considering or if i'm conforming to the code style. lmk!


[FEATURE REQ]  Backends Credentials Specified in Config Publisher Yaml file

Please describe the feature.
Can we specify the backend's headers within the publisher configuration yaml files? Similar to how the example displays defining the credentials for the loggers?

@zcarroll4 - the publisher configuration YAML should support most options in the REST API.
Have you tried something like this?
backends:
 - name: mybackend
   properties:
     credentials:
       ...etc

Here is the credentials object that the API expects.

@zcarroll4 - the publisher configuration YAML should support most options in the REST API.
Have you tried something like this?

backends:

 - name: mybackend

   properties:

     credentials:

       ...etc

Here is the credentials object that the API expects.

This is exactly what I was looking for, I'll take a look at that wiki page tomorrow and test again. And yeah I tried something similar to how the logger syntax was but didn't have any luck yet.

@zcarroll4 please remember to close the issue if the suggestions solves your issue.

@zcarroll4 it would be great if you can post a scrubbed sample here for future references once you get  a successful run.

backends:
  - name: APP-TX
    properties:
      url: "https://****************.azurewebsites.us/api"
      resourceid: "https://***********************/subscriptions/******************/resourceGroups/********/providers/Microsoft.Web/sites/*************"
      credentials:
        header:
          "HEADER-NAME-1": [
            "**************************************"
          ]
          "HEADER-NAME-2": [
            "{{INSERT SAVED NAME VALUE}}"
          ]


Wrong image size

Describe the bug
iPhone 10x not displaying images correctly
it is somehow cut off, that with a preview, that after opening
can you check?
Reproduction
Iphone X
v. 14.4
Browser: Chrome\Safari
Logs
There is not

System Info
There is not

Additional Information

Preview

After open

Android works fine

That's interesting.
I have only Android phone, so I have not tested on actual iPhone.
I just tried https://www.browserstack.com/test-on-iphone.
iPhone 11/12 works fine.
hmm... and I noticed some of the older version iPhones do have issues displaying the site.
I currently have no solution.


Make bestMatches case insensitive

Thank you for your great work on this plugin. One suggestion: Make it so that the completion object returns results regardless of the case (upper or lower) of the user's input.
Changing regpat.test( listChoice[i] )  to regpat.test( listChoice[i].toLowerCase() )  within completion.js changed the behavior for me.
Thanks!

Yes. I'll add some configuration in next release with case sensitive/insensitive choice.
My option is to change
var regpat = RegExp( this._regExpEscape(pattern));
to
var regpat = RegExp( this._regExpEscape(pattern), "i");
so as to have a case insensitive search.

That works perfectly. Cheers!


Wrapped Oracle packages leave endDelimiter in compiled code with createProcedure change type

Environment
Liquibase Version: 4.17.2
Liquibase Integration & Version: CLI
Database Vendor & Version: Oracle Database 19c EE High Perf Release <IP_ADDRESS>.0 - Production
Description
When installing a wrapped package using the change set type of createProcedure, end delimiter is not removed from the compiled code.
Steps To Reproduce
Package spec:
create or replace package lb_demo
as
    function get_name(n in varchar2) return varchar2;
end lb_demo;
/

Liquibased components for this test case:
source/plsql/lb_demo_wrapped.sql:
create or replace package body hr.lb_demo wrapped
a000000
369
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
b
f3 e7
C7uNqX4GMBimM/tcik3DZlp+HHIwg3lK7csVfHRGWE6OHAUN9hQa5FOhovwOITXgGEZwHQj5
/aHV8ltILXM69TOTTbd852V91XogQ/kw7Vh1iNblf7fy0S8EWQLbRqQ371PjTdjjhJ11Zuh2
qrg4Kg6U9NSzqz1dYKVaGzFKI1cVpvpmeQVMbwjwUU8IZGwSncf34ik0DR6+Obyux9SDfQGz
V0Q/zqbhcIIz

;

db/liquibase/lb_demo_wrapped.sql.yml
databaseChangeLog:
- changeSet:
    author: trent
    id: 55
    runOnChange: true
    changes:
    - createProcedure:
        dbms: oracle
        encoding: utf8
        endDelimiter: ;
        path: source/plsql/lb_demo_wrapped.sql
    rollback:
    - empty: null

liquibase.properties:
liquibase.hub.mode=off
changeLogFile=db/liquibase/lb_demo_wrapped.sql.yml
url=jdbc:oracle:thin:@connect_string
username=user
password=password
classpath=/home/trent/Oracle/ojdbc11.jar
logLevel=info

Actual Behavior
Liquibase CLI output
$ liquibase update
Starting Liquibase at 14:26:58 (version 4.17.2 #5255 built at 2022-11-01 18:07+0000)
Liquibase Version: 4.17.2
Liquibase Community 4.17.2 by Liquibase
[2022-12-06 14:27:09] INFO [liquibase.lockservice] Successfully acquired change log lock
[2022-12-06 14:27:37] INFO [liquibase.changelog] Reading from ACDC.DATABASECHANGELOG
Running Changeset: db/liquibase/lb_demo_wrapped.sql.yml::55::trent
[2022-12-06 14:27:45] INFO [liquibase.changelog] Stored procedure created
[2022-12-06 14:27:45] INFO [liquibase.changelog] ChangeSet db/liquibase/lb_demo_wrapped.sql.yml::55::trent ran successfully in 766ms
[2022-12-06 14:27:47] INFO [liquibase.lockservice] Successfully released change log lock
Liquibase command 'update' was executed successfully.

Package after this operation

And cannot use my functional package until I recompile having removed the semicolon at the end

Work Arounds

In the source file, ommit the end delimiter character (;)
In the yml config, specifal the changelog type as sqlFile instead of createProcedure

Reproduced on liquibase 4.17.2. Adding it to the list of issues to fix. Thanks!

Hi @MalloD12
In the snippet above for the body - if you take out the semi-colon does it work? On reflection, I think that a mistake in my bug report including the semi-colon at the end of the reference source file.
Ive tested with end delimiter as / i.e.
create or replace package body hr.lb_demo wrapped
a000000
369
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
abcd
b
f3 e7
C7uNqX4GMBimM/tcik3DZlp+HHIwg3lK7csVfHRGWE6OHAUN9hQa5FOhovwOITXgGEZwHQj5
/aHV8ltILXM69TOTTbd852V91XogQ/kw7Vh1iNblf7fy0S8EWQLbRqQ371PjTdjjhJ11Zuh2
qrg4Kg6U9NSzqz1dYKVaGzFKI1cVpvpmeQVMbwjwUU8IZGwSncf34ik0DR6+Obyux9SDfQGz
V0Q/zqbhcIIz
/

And that works fine using SQLcl or SQL*Plus (command line interface).
󱐌 localhost:1521/freepdb1: HR> @src/spec.sql

Package LB_DEMO compiled

󱐌 localhost:1521/freepdb1: HR> @src/body.sql

Package Body HR.LB_DEMO compiled

󱐌 localhost:1521/freepdb1: HR> select lb_demo.get_name('Trent') from dual;

LB_DEMO.GET_NAME('TRENT')
____________________________
Trent

Just getting up to speed on the latest version, I'm unsure if this is still an issue. Testing with XML
<?xml version="1.0" encoding="UTF-8"?>
<databaseChangeLog
        xmlns="http://www.liquibase.org/xml/ns/dbchangelog"
        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
        xmlns:ora="http://www.oracle.com/xml/ns/dbchangelog-ext"
        xsi:schemaLocation="http://www.liquibase.org/xml/ns/dbchangelog http://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-4.17.xsd"
>
  <changeSet id="spec" author="tschf" runOnChange="true">
    <createProcedure
      dbms="oracle"
      encoding="utf8"
      path="src/spec.sql"
    >
    </createProcedure>

  </changeSet>

  <changeSet id="body" author="tschf" runOnChange="true">
    <createProcedure
      dbms="oracle"
      encoding="utf8"
      path="src/body.sql"
    >
    </createProcedure>

  </changeSet>
</databaseChangeLog>

Seems working okay:
Liquibase Version: 4.27.0
Liquibase Open Source 4.27.0 by Liquibase
Running Changeset: lb_demo.xml::spec::tschf
Running Changeset: lb_demo.xml::body::tschf

UPDATE SUMMARY
Run:                          2
Previously run:               0
Filtered out:                 0
-------------------------------
Total change sets:            2

Liquibase: Update has been successful. Rows affected: 2
Liquibase command 'update' was executed successfully.

󱐌 localhost:1521/freepdb1: HR> @status

OBJECT_NAME                 OBJECT_TYPE     STATUS
___________________________ _______________ _________
DATABASECHANGELOG           TABLE           VALID
DATABASECHANGELOGLOCK       TABLE           VALID
PK_DATABASECHANGELOGLOCK    INDEX           VALID
LB_DEMO                     PACKAGE         VALID
LB_DEMO                     PACKAGE BODY    VALID

󱐌 localhost:1521/freepdb1: HR> select lb_demo.get_name('Trent') from dual;

LB_DEMO.GET_NAME('TRENT')
____________________________
Trent

(this is with a final forward slash in the source files).
For good measure, also testing with yml as per OP, and that too seems working okay
databaseChangeLog:
- changeSet:
    author: tschf
    id: spec
    runOnChange: true
    changes:
    - createProcedure:
        dbms: oracle
        encoding: utf8
        endDelimiter: ;
        path: src/spec.sql
    rollback:
    - empty: null
- changeSet:
    author: tschf
    id: body
    runOnChange: true
    changes:
    - createProcedure:
        dbms: oracle
        encoding: utf8
        endDelimiter: ;
        path: src/body.sql
    rollback:
    - empty: null

Also thought I'll test sqlFile changeset type while I'm here - this throws an error
  <changeSet id="body" author="tschf" runOnChange="true">
    <sqlFile
      path="src/body.sql"
      dbms="oracle"
    />
  </changeSet>

     Reason: liquibase.exception.DatabaseException: ORA-00900: invalid SQL statement
 [Failed SQL: (900) aHV8ltILXM69TOTTbd852V91XogQ/kw7Vh1iNblf7fy0S8EWQLbRqQ371PjTdjjhJ11Zuh2
qrg4Kg6U9NSzqz1dYKVaGzFKI1cVpvpmeQVMbwjwUU8IZGwSncf34ik0DR6+Obyux9SDfQGz
V0Q/zqbhcIIz]

Hi @tschf, I'm closing this issue for now. If you are still experiencing any issue, please reopen this issue or open a new one with any additional detail you can provide to the team.
Thanks,
Daniel.


reproducing paper results with different typo probabilities

Hi!
I'm trying to reproduce results from your ACL22 paper for MLRC2022.
In your paper, you show that LOVE+FastText and LOVE+BERT achieve more robustness in OOV, with respect to their respective baselines.
However, I found no instructions on how to produce typos/a corrupted dataset, except for details provided in Appendix B.3, where you talk about simulating post-OCR errors.
Would you be able to provide more insights on simulated typos?
I also found these paths in extrinsic/rnn_ner/gen_vocab.py:
    train_path = 'input/train.txt'
    dev_path = 'typo_data/typo_dev.txt'
    test_path = 'typo_data/typo_test.txt'
    out_path = 'typo_data/typo_vocab.txt'

which make me believe that some folders or script for simulating typos are missing.
Thank you!

Hi! I'm trying to reproduce results from your ACL22 paper for MLRC2022. In your paper, you show that LOVE+FastText and LOVE+BERT achieve more robustness in OOV, with respect to their respective baselines. However, I found no instructions on how to produce typos/a corrupted dataset, except for details provided in Appendix B.3, where you talk about simulating post-OCR errors.
Would you be able to provide more insights on simulated typos? I also found these paths in extrinsic/rnn_ner/gen_vocab.py:
    train_path = 'input/train.txt'
    dev_path = 'typo_data/typo_dev.txt'
    test_path = 'typo_data/typo_test.txt'
    out_path = 'typo_data/typo_vocab.txt'

which make me believe that some folders or script for simulating typos are missing. Thank you!

Hi,
Thanks for asking!
To add typos, you can use the function "get_random_attack" in the file attacks.py;
To simulate post-OCR, you can use the function "ocr_aug" in the file attacks.py
This paper also describes about how to add attacks: Combating Adversarial Misspellings with Robust Word Recognition
Note that there is a real-world dataset that contains OCR errors, CLEF-HIPE.

Got it, thank you for answering!


Add ALL_SESSION_COOKIE_SAMESITE

For some cases, we need to set "samesite" in all cookies. Especially useful when we don't know cookie names that third party package created.

@jotes All fixed. Please check this again.
I set flag name as SESSION_COOKIE_SAMESITE_FORCE_ALL


Polish for Home Page

Context: We're still seeing lots of text after the user initially selects the occupation. Instead, the user should see the intro text shrink to two lines and end with an ellipsis (...) . Then if the user wants, she can expand the full text again with a caret.
Here's the part of the page we're describing:

And this would disappear on selection of an occupation:

Here are two screenshots of the page conditions.

Here is the link to the Figma
https://www.figma.com/file/RXkmrydtGk66Z3X0AbbKNZ/JoHo-Interface-3?node-id=2%3A40


Crashplan fails to start when a volume is encrypted

Current Behavior
When rebooting the NAS CrashPlan fails to start as one of the volumes is encrypted and the mounting process seems to take a bit longer than the container starting up.  Manually starting CrashPlan after the volume is mounted is fine but I have to remember to do it.
Expected Behavior
Is there a way of delaying the start of the CrashPlan container or delaying the start of the container manager in Synology?  I have read you can edit docker.service with the following but I don't seem to have one or know if it would work on the Synology implementation of docker.
[Unit]
#ExecStartPre=/bin/sleep 30
Any help is much appreciated
Steps To Reproduce
No response
Environment

OS:
OS version:
CPU:
Docker version:
Device model:
Browser/OS:

Container creation
Composer file
Container log
Start container crashplan-pro failed: {"message":"Bind mount failed: '/volumex/xxxx' does not exists"}.

Container inspect
No response
Anything else?
No response

This is a generic question that would apply to any container. You will probably have better answers by looking at the Synology forums.

Ok thanks


Timetable doesn't get returned in order

Hey,
When I get my Timetable using untis.getOwnTimetableForToday();, it gets returned in a completely random order.
Is there a way to sort it according to startTime or endTime?
If so please let me know. :)

You can sort with js itself. However, as far as I know there is no way to do it server side.

Hey, a little follow up to how i fixed it:
var sortedInput = timetable.slice().sort((a, b) => a.startTime- b.startTime);


Allows Membership date to be found by find function

MembershipPredicate uses Enum to compare and Membership getValue returns toString value

Codecov Report

Merging #150 (6af0c5f) into master (4135b58) will increase coverage by 0.04%.
The diff coverage is 90.00%.

@@             Coverage Diff              @@
##             master     #150      +/-   ##
============================================
+ Coverage     67.59%   67.63%   +0.04%
- Complexity      703      704       +1
============================================
  Files           115      115
  Lines          2271     2274       +3
  Branches        236      237       +1
============================================
+ Hits           1535     1538       +3
  Misses          648      648
  Partials         88       88

Impacted Files
Coverage Δ

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...address/logic/parser/ListMembersCommandParser.java
100.00% <100.00%> (ø)

...in/java/seedu/address/model/person/Membership.java
93.65% <100.00%> (+0.31%)
:arrow_up:

...person/util/PersonContainsMembershipPredicate.java
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Uvm

Updated riscv/sim/sv_verif

Corrected 1_compile.do (tested on different PCs);
Cleaned up;
Added .gitignore.

the ideia was to keep the original tb with the uvm tb. So I suggest to change the tb directory like this:

https://github.com/amamory/vfsd-utopia/tree/master/src/tb/sv  <== the original tb from the book
https://github.com/amamory/vfsd-utopia/tree/master/src/tb/uvm  <== your tb

please recall to update the do files, test the original and the uvm tb considering this new paths.
also, update the readme files accordingly.


Check exertion of chunked transfers in large deploy transfer test

The test found in join_with_big_trie.rs has recently been fixed to actually test the condition it is supposed to test, it was giving false positives before. However, ultimate verification still relies on manual log reading.
The functionality should be enhanced to determine the outcome from the logs automatically, similar to how it is done in network_soundness.py.

PR: https://github.com/casper-network/casper-test/pull/145


 We should use different function names like GoToLua and GoToLuaProxy.

func LuaToGo(L *lua.State, t reflect.Type, idx int) interface{}

t and interface{} force the use of reflection and type switching. We should store the result in the value pointed by the interface{} passed as argument, just like json.Unmarshal.

The new functions would be:
func GoToLua(L *lua.State, v interface{})

func GoToLuaProxy(L *lua.State, v interface{})

func LuaToGo(L *lua.State, v interface{}, idx int) error

This is an easy change but would break backward-compatibility.
This might not be such a big deal considering the low usage or Luar, but this is bad practice nonetheless.
Shall we move to a Luar2 repo?
Or shall we ignore this change?

Go versioning of packages used to be a ... tricky business. But people keep their own forks when they want to isolate themselves from any changes.  Your proposal makes much more sense - de-emphasize reflection (which is an implementation detail here). I wonder if the old klunky API could be deprecated but available against the new API for a while.

As you suggest, we could opt for a transition period: Leave the deprecated functions for a while, then remove them. Then we need to new names. Not sure what can be as simple as GoToLua and LuaToGo...

Other changes:

Remove the Copy* functions as they are redundant with GoToLua and LuaToGo.
Replace map2table, slice2table and struct2table with a single luar.totable.
Remove luar.type?

@stevedonovan What's your preference?
I think a transition period would add more complexity than solve problems. I'd rather leave a "Compatibility notice" in the readme.
Let's decide on something. My preferences, in order:

Create a new repo
Change directly on top of the current repo and leave a compatibility notice.
No change at all
Progressive changes.

I've worked on the API rewrite: it was much more than I expected. I've discovered many more bugs and design issues in the process. The result has been pushed on branch v2. Here is the change log taken from the README:
Change log
The main differences with the previous version:

The function prototypes of GoToLua and LuaToGo are simpler and do not
require the use of reflection from the callers. The dontproxify argument is
gone, use GoToLuaProxy to control proxification.

The Copy* functions and GoLuaFunc are gone, use GoToLua and LuaToGo instead.

Use Register instead of RawRegister.

InitProxies is gone since it was not needed.

The LuaObject and LuaTableIter structure fields are unexported.

LuaObject methods not only work on Lua functions but also on anything with a
__call metamethods. Idem for tables and the __index/__newindex
metamethods.

Use NewLuaObjectFromName(L, "_G") instead of Global.

Lookup and Geti gone. Instead the Get and GetObject functions are
variadic: each subfield argument can be any valid Lua key (string, integer...).

Use (*LuaObject) Call instead of (*LuaObject) Callf. The protoype of
(*LuaObject) Call has changed in a fashion similar to GoToLua and LuaToGo.
Types is gone as it is no longer needed.

Register ProxyIpairs and ProxyPairs instead of calling LuarSetup.

Register and use Unproxify instead of ArrayToTable, MapToTable,
ProxyRaw, SliceToTable and StructToTable.

ComplexReal and ComplexImag have been replaced by the proxy attributes
real and imag, respectively.

SliceSub and SliceAppend have been replaced by the proxy methods
slice and append, respectively.
Slice proxies have the cap metamethod alongside append and slice.

String proxies have a slice method just like slice proxies. They can be
looped rune-by-rune over with ipairs.

The range of supported conversion has been extended:

LuaToGo can convert to interfaces and pointers with several levels of indirection.

LuaToGo can convert to non-empty maps and structs.


Backports 0.18 pr2

13899 should be dropped (it was merged in https://github.com/dashpay/dash/pull/4196) + pls see b774f74847049c5dfc00b7895f42fd4c7028b215 and 8aefd624d0944924e7310a753b1f980e59eeeed1


Azure Service Bus support is available now

Announcement: https://docs.microsoft.com/en-us/azure/service-bus-messaging/service-bus-managed-service-identity
cc @clemensv

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

ID: 726adac9-1848-01a6-9586-842c81f820f7
Version Independent ID: 249db438-8c95-8e69-f69e-8e48fcd0b359
Content: Azure Services that support managed identities for Azure resources
Content Source: articles/active-directory/managed-identities-azure-resources/services-support-managed-identities.md
Service: active-directory
Sub-service: msi
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@DanielLarsenNZ Thank you for your feedback. We will have this reviewed and updated as necessary after checking with the content authors.

@MarkusVi Could you please look into this and have the documentation updated as pointed by the customer. The managed Identity authentication is available natively for Azure Service bus resource now. It would be great if you could look into this one.
Thank you

Fixed...
#please-close


[sourcekitd-test] Add option to count the number of instructions executed during a request

The number of instructions executed gives a pretty stable measurement that's a good proxy for the time an operation took.
To use this in sourcekitd-test, we need to use an in-process sourcekitd (since sourcekitd-test cannot by itself inspect the instruction counter) of an XPC service.
I'm putting this behind an additional flag for now because whoever uses it, should do so deliberately and understand the limitation that this counter only gives meaningful results when used together with an in-process sourcekitd.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please test

While using the locally, I found an overflow bug. Apparently 32 bits are not enough to store the number of instructions executed 🤷
@swift-ci Please smoke test


Typesafe marshal

Notes:

Adds typesafe marshaling as described in https://github.com/ghcjs/ghcjs/issues/419

Adds some typesystem cleverness:

Removes the list specific conversion methods, in favor of an auxiliary class

Adds a EnsureNotMaybe constraint to make the marshaling of Maybe well behaved

Renames JSType and JSONType to JSTypeOf and JSONTypeOf, to avoid collision with the new type family.

Things left to do:

[ ] Add instances of the marshaling classes to all of the reference types in ghcjs-base.  This means that lots of modules in ghcjs-base would import GHCJS.Marshal.  Is that ok?

[ ] Document the behavior of generic deriving marshaling

[ ] Address these FIXMEs: https://github.com/ghcjs/ghcjs-base/pull/34/files#diff-6dc88c79ebb178104238eb0e8385d068R164

[ ] Consider whether all these instances should be in GHCJS.Marshal - it might be better to have the other parts of ghcjs-base import this marshaling stuff.  That would allow the stuff here to turn into normal deriving: https://github.com/mgsloan/ghcjs-base/blob/bd055d746b0c38c3d104fca7f89fa137e59d6111/GHCJS/Marshal.hs#L190 .

[ ] Consider whether we should recommend deriving NFData / Data as standard practice for newtype wrappers around references (documented here)

What is the reason for having two JSRef conversion approaches, one “pure” and one in IO?

Thanks for the suggestions!  I've added a commit updating the laws.

It makes sense to have pure marshaling for immutable things. So, in javascript this applies to the boolean / number / string types. Pure marshaling doesn't make sense for objects, because they can change.

Okay, now I understand the differentiation. Thanks. Follow-up question: Is pure marshaling a necessity or an optimization or something else?

What is the reason for having two JSRef conversion approaches, one “pure” and one in IO?

With pure conversions you can operate directly on immutable (either guaranteed by the underlying type, like JavaScript string or number, or by the typesystem) data types from pure Haskell code. Normally the impure conversions should be a strict superset of the pure conversions, since you should be able to get an impure one from a pure one by adding return.
I don't really like the fact that it's two typeclasses. Can we do better? Since purity is really a property of an instance, it would be better to indicate for every instance whether the conversion is pure, eliminating the second class.
I played with the following code earlier, but without injectivity of the Purity type family it doesn't work (GHC 8 lets you add an injectivity annotation, but it (correctly) rejects the family as being noninjective). Is there a way to fix this?
{-# LANGUAGE TypeFamilies, DataKinds, OverloadedStrings #-}
module Test where

import           Data.Text (Text)
import qualified Data.Text as T

newtype JSRef = JSRef Int -- dummy

ref0, ref1 :: JSRef
ref0 = JSRef 0
ref1 = JSRef 1

type family Purity (a :: Bool) b where
  Purity True  b = b
  Purity False b = IO b

class FromJSRef a where
  type IsPureFromJSRef a :: Bool
  fromJSRef :: JSRef -> Purity (IsPureFromJSRef a) a

instance FromJSRef Int where
  type IsPureFromJSRef Int = True
  fromJSRef (JSRef x) = x

instance FromJSRef Bool where
  type IsPureFromJSRef Bool = True
  fromJSRef (JSRef x) = x /= 0

instance FromJSRef Text where
  type IsPureFromJSRef Text = False
  fromJSRef (JSRef x) = return (T.replicate x "text")

An alternative route that actually does work:
Using GADTs, and keeping the pFromJSRef and fromJSRef things separate, still eliminating one of the typeclasses (change the Bool kind to a custom Purity kind to get better error messages). Can we simplify this further?
{-# LANGUAGE ScopedTypeVariables, GADTs, DataKinds, KindSignatures, TypeFamilies, ConstraintKinds, OverloadedStrings #-}
module Test where

import           Data.Text (Text)
import qualified Data.Text as T

newtype JSRef = JSRef Int -- dummy

ref0, ref1 :: JSRef
ref0 = JSRef 0
ref1 = JSRef 1

fromJSRef :: forall a. FromJSRef a => JSRef -> IO a
fromJSRef x = let (r :: Result (IsPureFromJSRef a) a) = fromJSRef_ x
              in  case r of Pure   r -> pure r
                            Impure r -> r

pFromJSRef :: forall a. (FromJSRef a, IsPureFromJSRef a ~ True) => JSRef -> a
pFromJSRef x = let (r :: Result True a) = fromJSRef_ x
               in  case r of Pure r -> r

data Result (p :: Bool) a where
  Pure   :: a    -> Result True  a
  Impure :: IO a -> Result False a

class FromJSRef a where
  type IsPureFromJSRef a :: Bool
  fromJSRef_ :: JSRef -> Result (IsPureFromJSRef a) a

instance FromJSRef Int where
  type IsPureFromJSRef Int = True
  fromJSRef_ (JSRef x) = Pure x

instance FromJSRef Bool where
  type IsPureFromJSRef Bool = True
  fromJSRef_ (JSRef x) = Pure $ x /= 0

instance FromJSRef Text where
  type IsPureFromJSRef Text = False
  fromJSRef_ (JSRef x) = Impure $ return (T.replicate x "text")

Bikeshedding opportunity: rename JSNumber, JSBool to Number, Boolean respectively?
Looks prettier, but will probably require the use of more qualified imports. I don't really like having half the world prefixed with JS.

With pure conversions you can operate directly on immutable (either guaranteed by the underlying type, like JavaScript string or number, or by the typesystem) data types from pure Haskell code. Normally the impure conversions should be a strict superset of the pure conversions, since you should be able to get an impure one from a pure one by adding return.

So this makes me think that the motivation is convenience? If so, is it really worth the effort to differentiate between these types?

So this makes me think that the motivation is convenience? If so, is it really worth the effort to differentiate between these types?

The only realistic option would be to drop pure conversions, which would force much more code to be in IO, unless we have a separate way to handle all the pure conversions. The pure conversions were originally introduced for ghcjs-ffiqq, which needed a pure way to convert to/from arbitrary JSRef values.
for example things like this get much less nice if you need IO everywhere, and IO adds considerable overhead, preventing many optimizations:
f :: SomeObject -> Bool -> Int
f o p
  | [js| `o.doSomePureCheck(`p) |] = 1
  | otherwise                      = 2

And I also think it's quite usful to operate on JSRef values in pure code, without converting beforehand.

Fair enough. Thanks for the responses.
I hope you don't mind me continuing to flesh out my understanding on this topic here.
What are the scenarios in which we need to convert types? Let me propose the ones that come to mind:

Converting any JS type polymorphically
Converting a specific JS type monomorphically

Since the most general form of conversion for all types is via IO, I can clearly see the need for that in 1.
We can always convert via IO for 2, but that is, as has been said, inconvenient and inefficient for some types, specifically the primitives. For those types, we want a non-IO conversion.
Now, this prompts the question of whether we need to treat all primitives in the same way. Do we need to convert any primitive JS type polymorphically? (This would be 3 on the list above, if true.)
If the answer is yes, we should have a type class (or some other ad-hoc mechanism) for that. If the answer is no, then I would question the use of an ad-hoc mechanism for something we don't need.
The ghcjs-ffiqq package is an example that uses the primitive JS type polymorphism. Looking at the source and example code, I wonder whether this is a good idea. It appears to me that code using the quasiquoter does not improve type inference and could lead to places requiring type annotations or signatures.
The use of the primitive JS type polymorphism in ghcjs-ffi appears to be a way to reduce the number of symbols one needs to know to use the interface. That is, you can use the overloaded js for Int, Bool, etc. rather than different functions for each (e.g. jsint, jsbool, etc.). Granted, there could be a lot of different functions, but I wonder if there is a better way.
What if, instead of using a type class to select the conversion instance, you used a parameter to the quasiquoter function that specialized the type? You could, for example, use the TH Name as a parameter, allowing you to use name quoting of the type, e.g. ''Int. (Alternatively, you could use an enumeration to specify a closed world of types.) You could end up with the same number of quasiquoter functions you have now, but just give some of them a parameter. (For example, [js'| 'x + 'y |] becomes [js' ''Int| 'x + 'y |].)
This approach removes any concern with type inference and avoids polluting the namespace with type-specific functions. Finally, you wouldn't need to define a class of pure JS type conversions.
Feel free to shoot holes in my understanding or ideas.

I do welcome comments. And we should definitely steer towards simplicity unless we have a good reason for more type classes. I'm really short on time though, with preparations for Haskell eXchange taking most of my time at the moment, so I'll come back to this later.

I think this is a good way to get rid of the pure marshaling classes:
class PureMarshal a

instance PureMarshal JSRef
instance PureMarshal Text
-- etc

pFromJSRef :: PureMarshal a => JSRef -> a
pFromJSRef = unsafePerformIO . fromJSRefUnchecked
{-# INLINE pFromJSRef #-}

pToJSRef :: PureMarshal a => a -> JSRef
pToJSRef = unsafePerformIO . toJSRef
{-# INLINE pToJSRef #-}

Inlining should make this perform just as well for the common case, right?

Bikeshedding opportunity: rename JSNumber, JSBool to Number, Boolean respectively?

I initially had it this way, but then backpeddled because it seemed weird to have Bool be a haskell type and Boolean be a javascript ref.  I like dropping the JS prefixes, though, so I'll do this.

The ghcjs-ffiqq package is an example that uses the primitive JS type polymorphism. Looking at the source and example code, I wonder whether this is a good idea. It appears to me that code using the quasiquoter does not improve type inference and could lead to places requiring type annotations or signatures.

At some point, I'm hoping to have a typescript FFI quasiquoter - see this https://github.com/mgsloan/ghcjs-typescript#alternative-approach-to-typescript-ffi .

Inlining should make this perform just as well for the common case, right?

Nope, at least not with unsafePerformIO
And I really don't like this approach, it's too fragile. You basically insert unsafePerformIO everywhere by creating a empty instances. It would only be acceptable if user's code was never expected to add pure marshalling instances, and we could hide it.

Hmm, good point. Darn, I'd like something really simple!
Your GADTs approach is pretty reasonable, but it would obviously be quite a breaking change.   I'd like to bundle such a change with the change to using exceptions instead of Maybe.  Maybe this is being overly performance sensitive, but would having these result constructors introduce overhead?  They should probably have strictness annotations on the fields.  Since there's only one constructor available for a given type, one would hope it'd be equivalent.

I do welcome comments. And we should definitely steer towards simplicity unless we have a good reason for more type classes. I'm really short on time though, with preparations for Haskell eXchange taking most of my time at the moment, so I'll come back to this later.

Of course! No problem.

I've rebased this atop master.
Does this look good?  I'm waiting on review of this before addressing the remaining items in the checklist.  Next up after this is switching marshaling over to using exceptions instead of Maybe.

It seems you guys want to break and rewrite Marshalling.
I'm going to document FFI and what I know about Marshalling and the relationship between FFI and Marshalling on https://github.com/ghcjs/ghcjs/wiki/GHCJS-User-Guide
I already documented a basic project setup with stack on it.
Since FFI is not documented, it was difficult for me to figure out how it worked. I had to ask people and consult ghcjs-dom.
Coding is not finished until it is documented for users.


stop capture shortcut key request

macOS version:    19.5.0 (darwin)
Kap version:      3.2.2

It's difficult to find Cap stop icon. A shortcut key would help.

Hey, you can configure a global shortcut in the preferences window. The same shortcut that triggers Kap can be used to stop the recording

Awesome！Thanks a lot.


Use training framework package

Removed MAX training framework code
Updated training README
Closes #20

Passed all tests.

@ptitzler if the suggestions look good, you can merge it.


indexes refactoring

indexes atifacts partially duplicted in the langchain and in langchain_community packages.
Regarding #17832 and following @baskaryan recommendations, indexes is moved into the langchain_community.
All ut-s in langchain work without changes.
Missed ut-s from langchain were added into the langchain_community ut-s. Only ut-s that test the "all" symbol were not added, because "allinlangchain_community` is not used.

Close temporarily till the dependency rules are clear.

Created this from referencing the refactor https://github.com/langchain-ai/langchain/pull/20667


KakuteF7: enable SPI DMA for SPI 4

DMA for SPI 1 left disabled because the SD card does not work when enabling it.

Out of curiosity you might want to take a quick look at the load with and without SPI DMA at 2 and 4 kHz.

@ 2khz:

DMA: 24.4% idle
no DMA: 12.8% idle

@ 4khz: no chance of running in both cases, CPU is maxed out.
BTW it's not easy for an end-user to recover from this, as you don't get MAVLink anymore.

Interesting, I thought a simplified kakutef7 might have a chance.
On v5 with only the icm20602 running it's ~ 12% idle (IMU_GYRO_RATEMAX=4000 SYS_AUTOSTART=4001).
 PID COMMAND                   CPU(ms) CPU(%)  USED/STACK PRIO(BASE) STATE FD
   0 Idle Task                   61288 12.972   244/  512   0 (  0)  READY  3
   1 hpwork                          0  0.000   344/ 1260 249 (249)  w:sig  3
   2 lpwork                          0  0.000   344/ 1612  50 ( 50)  w:sig  3
   3 init                          905  0.000  2200/ 2668 100 (100)  w:sem  3
   4 wq:manager                      1  0.000   408/ 1252 255 (255)  w:sem  3
 435 mavlink_rcv_if0              1400  0.257  2376/ 3908 175 (175)  READY  4
  16 wq:hp_default                5614  1.202  1160/ 1900 241 (241)  READY  3
  18 dataman                       365  0.085   760/ 1204  90 ( 90)  w:sem  4
  20 wq:lp_default                 542  0.171   812/ 1700 205 (205)  READY  3
  29 wq:I2C1                      3577  0.773   820/ 1396 246 (246)  READY  3
 127 wq:SPI1                    129868 27.319  1968/ 2364 253 (253)  w:sem  3
 152 wq:I2C3                      2880  0.601   820/ 1396 244 (244)  READY  3
 154 wq:SPI4                      1607  0.343   904/ 2364 250 (250)  w:sem  3
 176 wq:att_pos_ctrl             84270 17.783  5024/ 7196 242 (242)  READY  3
 177 wq:rate_ctrl               100583 21.219  1052/ 1596 255 (255)  w:sem  3
 182 commander                    5162  1.116  1280/ 3212 140 (140)  READY  6
 183 commander_low_prio              9  0.000   716/ 2996  50 ( 50)  w:sem  6
 189 mavlink_if0                 16116  3.264  1832/ 2572 100 (100)  READY  4
 235 gps                           913  0.171  1016/ 1676 208 (208)  w:sem  4
 270 mavlink_if1                  5912  1.202  1888/ 2484 100 (100)  READY  4
 271 mavlink_rcv_if1              1573  0.343  2384/ 3908 175 (175)  READY  4
 290 px4io                       14391  3.006   936/ 1484 240 (240)  w:sem  4
 422 navigator                     697  0.171  1040/ 1764 105 (105)  w:sem  5
 453 log_writer_file                 0  0.000   368/ 1164  60 ( 60)  w:sem  3
 444 logger                       1788  0.343  2184/ 3644 233 (233)  w:sem  3
 456 top                         13710  2.920  1552/ 2028 248 (248)  RUN    3

Processes: 26 total, 12 running, 14 sleeping, max FDs: 15
CPU usage: 82.30% tasks, 4.73% sched, 12.97% idle
DMA Memory: 5120 total, 1024 used 1024 peak
Uptime: 476.641s total, 61.289s idle

BTW it's not easy for an end-user to recover from this, as you don't get MAVLink anymore.

Yes, and it can even be bad enough you the serial console isn't usable. At the moment IMU_GYRO_RATEMAX is limited to 2 kHz with param metadata. Earlier on it was also limited to 2 kHz in each driver. I've thought about another watchdog at various levels (each driver, WQ, system wide), but I'm not sure if it's worth it. Thoughts?

Yes it's DShot that uses a bit more. It's worth having another look at that.

I've thought about another watchdog at various levels (each driver, WQ, system wide), but I'm not sure if it's worth it. Thoughts?

Probably something on the system level, but it could easily do more harm than good. We can see if it becomes an issue in practise, and add something if really needed.


Transform origin?

I have a Gatsby app that the landing page looks like:
  return (
    <Layout location={location} title={siteTitle}>
      <SEO title="All posts" />
      <Bio />
      <Goo>
        <svg width="192" height="192">
          <g style={{ animation: 'left 4s linear infinite' }}>
            <circle cx="37%" cy="37%" fill="orchid" r="32" style={{ animation: 'right 1s linear infinite' }}/>
            <circle cx="63%" cy="63%" fill="mediumorchid" r="32"/>
          </g>
        </svg>
      </Goo>
    </Layout>
  )
}

Just exactly as the tutorial But it appears that the continuous transformation is about the upper left hand corner of the rectangle rather than the middle. Basically I want to shift the SVG over so that it appears like the turtorial. Right now I only see what appears when the transform takes the display to the 'lower right quadrant'
Thank you

Good catch, I've added it to https://gooey-react.netlify.app/tutorial/. Let me know if that solves it for you.


Auto teleport signing does not popup wallet

https://github.com/kodadot/nft-gallery/assets/90852205/26a62884-7630-48cc-aba5-0b7c4de6516c
when I click buy now, or finish purchase from the cart, in the signing step, there is no wallet popup

tried refreshing
tested here: https://canary.kodadot.xyz/ahp/explore/items?listed=true&collections=63&sort=price_ASC

it does not work for me on any nft

seems to be working let me know if the issue still persists
https://github.com/kodadot/nft-gallery/assets/44554284/74960ceb-8668-4217-a3f6-3278edd59f64
https://github.com/kodadot/nft-gallery/assets/44554284/1b5ea94d-9e67-4e1e-b3e6-65e5dab2eb58

@hassnian will try later today

did not try, but looks like I am not the only one

for this user, the popup probably did not even display

he was clicking on the circle next to the heading.... :D

it should also probably rotate the circle when it's waiting for the signature - as it's done in the simple signing

cc @hassnian

anything we can do?
I just cant sign the autoteleport, seems like issue of ed or something like that

https://github.com/kodadot/nft-gallery/assets/90852205/268ca4ec-f6e2-4f5c-9f1b-3affad36768b
can we solve it?

anything we can do?
I just cant sign the autoteleport, seems like issue of ed or something like that
Screen.Recording.2024-01-25.at.15.18.50.mov
can we solve it?

will check

@hassnian  any findings?

@hassnian any findings?

got busy with other issues, wasn't able to check
feel free to pick it up.

@exezbcz  please confirm issue is resolved
i have tried about 15 times on differant NFT's in the "Genera" collection
the auto-teleport sign modal opened every single time


Show warning about invalid values (instead of rejecting input)

In metadata template CSV, warn user about invalid value(s) instead of rejecting the invalid input. This would allow the user to override the warning and enter an invalid value.
NF use case: our schema is always evolving, and we sometimes have new data types that are not yet represented in our schema. We want to allow our contributors to enter these values, even if they're invalid, if need be (after they see a warning).
Example:

cc: @allaway

As a side note: for our use case, I think that it is sufficient to "warn" at the google sheets stage of data entry. I don't think that our use case requires any changes to validation in the app or schematic (e.g. if someone enters invalid metadata, it should still fail in the app the same way it does now, and only pass once we've assessed the new values and modified the underlying schema if appropriate).

Also, I think since schematic actually generates the google sheet this issue should probably get moved there

Good points, @allaway! Re: transfer to schematic, will do now.


Inline mode (probably) doesn't work exactly as people expected.

Here's an example:
https://github.com/el-hoshino/QuickshaRe/pull/11
With danger-swiftlint, it can put a review comment right onto the exact line of that warning/error when inline-mode is turned on, which is very easy to find where is wrong, and I guess probably most people would expect this kind of comment when they turn inline-mode on.
https://github.com/el-hoshino/QuickshaRe/pull/11#discussion_r368292012
But unfortunately danger-xcode_summary's inline-mode can only leave a normal warning with a link to that file/line along with all other warnings/errors that danger produces. So if there're a number of danger's warnings it's easy to miss the warnings produced by Xcode.
https://github.com/el-hoshino/QuickshaRe/pull/11#issuecomment-576002087
So it would be very helpful if danger-xcode_summary plugin can also support putting a comment onto the exact line 😃
danger-swiftlint's inline-mode implementation is here:
https://github.com/ashfurrow/danger-ruby-swiftlint/blob/38ff6e27123565e30660724877ead279f7cd87e4/lib/danger_plugin.rb#L140

We just merged #51, let me know if we can close this issue.

@diogot Thanks for merging!
Unfortunately I tried again with the PR above, and seems nothing changed...
The version is latest 0.5.2
+ gem install danger-xcode_summary
Successfully installed danger-xcode_summary-0.5.2
Parsing documentation for danger-xcode_summary-0.5.2
Done installing documentation for danger-xcode_summary after 0 seconds
1 gem installed

@diogot if I understand correctly, #51 adds the full file path (https://github.com/diogot/danger-xcode_summary/pull/51/files#diff-b922e84caed4009276a6db5646cef94bR140) whereas danger-swiftlint trims the directory portion (https://github.com/ashfurrow/danger-ruby-swiftlint/blob/38ff6e27123565e30660724877ead279f7cd87e4/lib/danger_plugin.rb#L307)
I may be wrong, and have never worked on a plugin so just commenting here if it helps. Let me know if you want me to attempt to fix it instead (may take longer :p)

Hi @thomasjoulin, thanks for the notes.
I'm not able to dedicate much time to this project lately, so it would be great if could help on this!


Make project work with Docker and Docker-compose

See:

https://github.com/altixco/django-postgres-dokku

Any update on this?.

Not jet we lost the morning because of the internet issues. I pretend to
work on th Mac in the office cause there is installed Docker. im installing
it on my old pc and trying to learn how to do it - did u have some
recommendation of were I can find Docker information?
El mié., 26 de agosto de 2020 6:21 p. m., Edison Arango <
<EMAIL_ADDRESS>escribió:

Any update on this?.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/altixco/altixco.github.io/issues/2#issuecomment-681173277,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQWI6TP65CJPDONIUNUSGG3SCWKJLANCNFSM4QJ63LZQ
.

Yes, it's in the Knowledge site:
https://knowledge.altix.co/stack/docker.html
Basically the official docker docs is the best starting point, and there is a lot of resources out there about Docker, most developers use it therefore it's not hard to find solutions.

Hi Edison, i have a truble cause i don't know the requirements for the dockerfile. to be more  spesificl what should i put in de "from". else if there is a way to know that from the file?

@Camilandia20 This is done, but please review the result of the pull request that Juan did:
https://github.com/altixco/altixco.github.io/pull/6


Use ByteBuf instead of DirectBuffer

Replaces the dependency on Agrona with netty-buffer, which provides similar functionality but allows us to avoid copying frame data for netty based transports and properly implement frame pooling.
New vs old allocations in TcpPing, for example, shows the effect this has on allocation pressure:

LGTM cc @lexs


Immediate disconnect on connection issue

Hello!
I'm currently working with the Nordic XIAO BLE SENSE for a school project, and I'd like to use your project to implement Midi over BLE. I added an overlay file for my board in order to have 4 buttons, and some debugging messages.
However, I'm running into the following issues:

Only my computer running under Ubuntu 23.10 and my Android smartphone can detect the board, but my friends running under Windows and MacOS can't. Do you know how to fix this?

When, using either my computer or my phone, I'm trying to connect to the board over BLE, the connection seems to be rejected, and the board prints the following error:
 Do you know what's going on, and how I could fix this problem?

Thanks!

Interesting. Lately, I've been developing with nRF Connect SDK and haven't checked vanilla Zephyr compatibility in a while. So I just tried building the sample app with Zephyr 3.3.0 for my nRF52840dk and it seems to work fine. I tried both with Bluetooth MIDI connect iOS app and the macOS Audio MIDI Setup app. Here's the log when connecting to the iOS app:
*** Booting Zephyr OS build zephyr-v3.3.0-1122-g29ac5d43817a ***
[00:00:00.380,310] <inf> bt_hci_core: hci_vs_init: HW Platform: Nordic Semiconductor (0x0002)
[00:00:00.380,340] <inf> bt_hci_core: hci_vs_init: HW Variant: nRF52x (0x0002)
[00:00:00.380,371] <inf> bt_hci_core: hci_vs_init: Firmware: Standard Bluetooth controller (0x00) Version 3.3 Build 99
[00:00:00.381,072] <inf> bt_hci_core: bt_dev_show_info: Identity: F0:BE:77:15:29:08 (random)
[00:00:00.381,103] <inf> bt_hci_core: bt_dev_show_info: HCI: version 5.4 (0x0d) revision 0x0000, manufacturer 0x05f1
[00:00:00.381,134] <inf> bt_hci_core: bt_dev_show_info: LMP: version 5.4 (0x0d) subver 0xffff
[00:00:00.381,164] <inf> ble_midi: ble_midi_init: Initialized BLE MIDI
[00:00:17.158,996] <inf> ble_midi: mtu_updated: MTU updated: tx 23, rx 23 (actual 23), setting tx_buf_max_size to 20
[00:00:17.159,057] <inf> ble_midi: on_connected: tx_running_status 0, tx_note_off_as_note_on 0
[00:00:17.159,088] <inf> ble_midi: on_connected: Got conn. interval 30 ms, requesting interval 7 ms with error 0
[00:00:17.577,148] <inf> ble_midi: mtu_updated: MTU updated: tx 23, rx 23 (actual 23), setting tx_buf_max_size to 20
[00:00:18.147,277] <inf> ble_midi: midi_read_cb: Got read request, responding with empty payload
[00:00:18.537,261] <inf> ble_midi: midi_ccc_cfg_changed: I/O characteristic notification enabled: 1
[00:00:18.537,292] <inf> ble_midi: on_service_availability_changed: BLE MIDI service available: 1
[00:00:18.745,269] <inf> ble_midi: le_param_updated: Conn. params changed: interval: 15 ms, latency: 0, timeout: 200
[00:00:22.182,037] <inf> ble_midi: le_param_updated: Conn. params changed: interval: 15 ms, latency: 0, timeout: 200

I just pushed a logging improvement so you can see the disconnect reason code in the log. Might give you a clue. And if you happen to have an nRF52840dk, it would be interesting to know if the unmodified sample app runs properly on that.
Just a guess, but judging from your log, the disconnect might be related to the central dropping the connection because it doesn't like the requested connection interval range (which currently sets both min and max to 7.5 ms). It would be interesting to see if using a larger max interval helps, for example setting INTERVAL_MAX to 0xc here.

Only my computer running under Ubuntu 23.10 and my Android smartphone can detect the board, but my friends running under Windows and MacOS can't. Do you know how to fix this?

The device won't advertise when connected, which means centrals won't be able to detect it. Did you perhaps forget to disconnect the device?


Added repo highlights

Changes proposed

Structuring README.md. Will be adding more, on merge.
Screenshots

Note to reviewers

Hey @syed-m-osama , Thanks for those changes. But I some suggestions for you.

I think the color contrast could be better. Could you make the color contrast stronger so that it looks fantastic in my opinion?

Hey @syed-m-osama , Thanks for those changes. But I some suggestions for you.

I think the color contrast could be better. Could you make the color contrast stronger so that it looks fantastic in my opinion?

surely, can I know what colors you have in your mind? (Meanwhile, I'll go ahead and do some changes.)
PS - thanks for taking time out in-advance.


fix: set page size 1000 till filter is implemented in org list API

Set organisation API page_size to 1000 till filters are supported on the BE.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 9.651%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
10836

Relevant Lines:
112283

💛  - Coveralls


add unit testing and CI pipeline

At least one test with Jest

Issue on hold — Jest definitions conflict with Figma definitions


Logic for merging checker

Hi there,
I see when you're merging in your CheckerBase you take 'Entailment' as a precedence?
https://github.com/amazon-science/RefChecker/blob/64e7c34b5fd4f6af7a5227473458619a3d92ad5b/refchecker/checker/checker_base.py#L6C1-L23C21
I see there is a TODO there, but I'd have thought maybe the default would be any contradition would indicate a problem.
Just curious on the thought process to make sure I understand your approach.
Thanks!

Hi, @Padarn. Thanks for raising the question! That part is for merging checking results with different segments as references. Ideally, there should be no difference between selecting "Entailment" or "Contradiction" first, because we take a simplified assumption that the whole reference should be self-consistent. However, conflicts in reference do happen in real-world applications. It is an open research question for how to handle it. If you have any ideas, welcome to discuss in this thread.
As a temporary workaround, we might consider expose the option outside to let users choose precedence when conflicts happen. Thanks again!

Hey @rudongyu thanks for your response.
I have two rough thoughts on this:

What about providing a more nuanced score of 'agreement' rather than a binary classification? Probably having the LLM classify and then scoring in during aggregation would be better (to avoid calibration problems with the LLM score)

Perhaps just providing an 'Inconsistent' class when it is mixed would be easier to understand?


How to get feature importance scores or attention heatmap

Hi,
I am trying to get some visualizations of interpretable results from FT-Transformer, such as feature importance or attention heatmaps. I find some discussions about feature importance in paper section 5.3, but I don't know how to achieve it.  Is there a way to achieve these based on the source codes you published? Or can you make an example of implementation?
Thank you very much!

Please, see this issue
P.S. The implementation of the paper is now located here: https://github.com/yandex-research/tabular-dl-revisiting-models

If you need further help, feel free to create a new issue in that repository or continue the discussion in the issue I mentioned


Added support for hotkeys

@crhampton1 bump for visibility

This looks like a good idea to me.
Here's the note in the code with an explanation: "When enabled, you will be able to quickly swap to the simplefog control by using Ctrl+S and toggle the opacity using the hotkey 'T'"


Integrate Panel for UI

The (semi) working notebook demos we have for this project both use ipyleaflet for the draw-a-box UI control.  ipyleaflet doesn't integrate well with JupyterLab or Panel as a display/dashboard tool.
We have @pmagana's panel.ipynb as an example of basic controls for UI to pass to an ERDDAP query.
If the project decides to use HoloViz and Panel for UI/plotting, we should look into swapping out the ipyleaflet bounding box draw tool for the examples in this notebook and connect the pieces together.

Hello @mwengren,
First of all, I hope everything is going well at these strange times.
I'm glad to hear that the project seems alive again, and will be continued in this edition of OceanHackWeek 2020. Please let me know if I can help in any way.
Regards,
Pedro

@pmagana Likewise!  A few of us are helping to bring it back to life again this year.   I had to drop it for a bit and never quite got going on it again.  But a new OWH is good motivation.
Hope all is well and you have no more laptop difficulties!  Feel free to jump in anywhere you like.


feat: Create Starfield App

Description
This PR adds an app that simulates flying through a starfield. This one looks so cool on an actual device! The gif doesn't do it justice.
The generated webp is 20fps for 15 seconds and is typically ~85 KB, but if star count and tail length both get raised to higher values, the output can be larger than 128 KB, causing some frame drops. I limited these to try to keep frame drops to a minimum, but with the current maximums, the output can max at ~170 KB. Let me know if you'd like me to further restrict these or change anything else to keep file size down!
I also heavily optimized this one. Current profile output takes 280ms. Is this acceptable?
Screenshot

Schema

Dropdown: Background Color
Dropdown: Star Colors
Toggle: Use Custom Star Colors?
Text: Custom Star Colors (accepts a single hex code or a comma-separated list)
Dropdown: Number of Stars
Dropdown: Star Speed
Dropdown: Star Tail Length

I had been thinking that some old-school screen savers could work well and this looks really cool. Can't wait to try it out in my actual TidByt!

@btjones Haha, thank you! I recently purchased my Tidbyt and pretty quickly started thinking the same thing. I plan to add more, so feel free to ping me on the Discord or Discourse if you have any other ideas!

I had a couple thoughts on colors options that could be interesting:

"random" which would select a different random color every time the app runs
"rainbow" which would have each star be a different color

I had a couple thoughts on colors options that could be interesting:

"random" which would select a different random color every time the app runs
"rainbow" which would have each star be a different color

@btjones Great ideas! I've added both options.

Random will choose a random color each time the starfield is rendered
Rainbow will choose a random color for each star. It's pretty chaotic, but I think I like it? Either way, doesn't hurt to have 😂

Now that it supports multiple colors at once, it might be cool to add color schemes? Or let users enter multiple comma-separated colors for the custom colors? I don't want to make the settings overwhelming, but that could be cool.

I went ahead and added support for palettes, along with adding a Christmas palette! The Christmas palette doesn't really look like a starfield, but it still looks cool 🤷

@gabe565 amazing.

@matslina Thanks for the feedback! I attempted to optimize as much as possible, but was unclear on some of the MQTT image push details. Does the backend continually regenerate the image and skip the MQTT push if the image is unchanged? If so, then I agree that an even longer window would make sense.
Only thing to be aware of in this case is since each star gets placed randomly, sometimes a burst of stars can happen in a similar area. These look cool, but if the same burst happens too many times in a loop, then it starts to become obvious. Otherwise, I'm not opposed to a longer timespan that will give the same deterministic output!
 Let me know what you think!

@matslina Thanks for the feedback! I attempted to optimize as much as possible, but was unclear on some of the MQTT image push details. Does the backend continually regenerate the image and skip the MQTT push if the image is unchanged? If so, then I agree that an even longer window would make sense.

There are a few layers of caching. Devices have a local cache of graphics. The backend also checks the widget tree against a cache before drawing and encoding webps. This change should be especially helpful for the latter of the two.


[MINOR][DOCS] Remove Java 8/11 at IgnoreUnrecognizedVMOptions description

What changes were proposed in this pull request?
This PR aims to remove old Java 8 and Java 11 from IgnoreUnrecognizedVMOptions JVM option description.
Why are the changes needed?
From Apache Spark 4.0.0, we use IgnoreUnrecognizedVMOptions JVM option to be robust, not for Java 8 and Java 11 support.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual review.
Was this patch authored or co-authored using generative AI tooling?
No.

Could you review this, @LuciferYang ?

Thank you, @bjornjorgensen .

Thank you, @LuciferYang !

Merged to master.