Setup style guide

Implements base styles according to layout, using Tailwind CSS

According to 501d94a063f66cfa491642ade76cab0078e52194
There's an error when installing the last version of auto-prefixer due to a wrong version of postcss that it's configured on Next. The fix it's on the backlog so the dependency should be updated here as well according to the release.


Pin Java dependencies

Description
Store all Java dependencies required for builds for faster downloads.
To pin all Java projects:
./install_dependencies.sh --pin-all
Workflow

⚠⚠ Create your PR as draft (we're receiving lot of PR, it saves us lot of time) ⚠⚠
Follows the style guidelines of this project (See how to easily lint the code)
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

NOTE:  By default in PR only default scenario tests will be launched. Please refer to the documentation to run all scenarios in your PR if needed.

Once your PR is reviewed, you can merge it ! :heart:

Hi @smola is this PR still relevant ?

@cbeauchesne I'll have to check again. the speed up was big under some circumstances, but I'm not sure it was significant enough in CI to justify the added complexity.

Hi @smola is this PR still relevant ?

@smola, same question ? 😄


useless import in monorepo

Clear and concise description of the problem

If some packages in the monorepo compiled out the same variable as plugin imported(such as h that imported from vue), when i use this packages as workspace plugin will import this variable too and cause error.

Uncaught SyntaxError: Identifier 'h' has already been declared

Although i can set exclude those packages, but i donot know which package will compiled out h or something that imported by plugin unless i got error.
exclude: [/node_modules/, /\.git/, /core\/pinia/, /ui\/block-plugin-template/],

if i want auto import @poto/pinia like this

AutoImport({
      imports: [
        'vue',
        '@vueuse/core',
        'pinia',
        {
          '@poto/pinia': [
            'useActionsStore',
            'useCustomBlocksStore',
            'useDesignerStore',
            'useWidgetMenuStore',
          ],
        },
      ],
      dts: 'src/auto-imports.d.ts',
      vueTemplate: true,
      resolvers: [ElementPlusResolver()],
      exclude: [/node_modules/, /\.git/, /ui\/block-plugin-template/],
    }),

plugin will import itself to this package

Suggested solution
Maybe transform should have a dynamic filter
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

https://github.com/antfu/unplugin-auto-import/blob/2c37ab86fcfda773be366a97e2967a9790a63bf8/src/types.ts#L117-L121

https://github.com/antfu/unplugin-auto-import/blob/2c37ab86fcfda773be366a97e2967a9790a63bf8/src/types.ts#L117-L121

This should not be the reason. This issues is about the filter of transform

I got it, should exclude anything out of project.
exclude: [/node_modules/, /\.git/, /^(?!.*ui\/playground\/)/],


fix(system_monitor): multithreading support for boost::process

Signed-off-by: v-nakayama7440-esol<EMAIL_ADDRESS>Description
boost::process::pipe internally creates file descriptor. However, this file descriptor does not support multithreading because O_CLOEXEC is not set. Because of this, when each thread of SystemMontor uses boost::process to create a child process, the file descriptors of boost::process::pipe conflict between child processes. As a result, the child process waits for the file descriptor to be released.
In this PR, the SystemMonitor application creates file descriptor with O_CLOEXEC set and passes it to boost::process::pipe to eliminate the wait for child process to release file descriptor.
Related links

The issue https://github.com/autowarefoundation/autoware.universe/issues/939 was still observed after the fix https://github.com/autowarefoundation/autoware.universe/pull/948.
This is related to the boost::process issue and to apply the workaround described in https://github.com/boostorg/process/issues/169#issuecomment-679170651.

Tests performed
Process Monitor's /diagnostics topic message timestamp interval is essentially about 1 second (diagnostic_updater event execution cycle is 1hz). However, before the fix, the interval would occasionally be about 2 seconds.
Graph before fix (vertical axis unit is msec, horizontal axis unit is message order):

After the fix, Process Monitor confirmed that the timestamp interval for messages on the /diagnostics topic was not always reaching 2 seconds. Monitors other than Process Monitor were also checked in the same way.
Graph after fix (vertical axis unit is msec, horizontal axis unit is message order):

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@v-nakayama7440-esol Could you please attach some measurements such as visualized interval of before and after the fix in Tests performed above?

@ito-san
The graphs of Process Monitor before fix and Process Monitor and CPU Monitor after fix are attached to "Tests performed".

Verified the issue was gone after 1 hour test in cargo project.

Verified in bus project with humble.


No Chrome installations found

Config / CLI options
pwmetrics https://example.com/
Steps to reproduce.

Fresh install in project using npm install pwmetrics
Run pwmetrics https://example.com/
See error in console output

Stack trace
Launching Chrome
{ Error
    at new LauncherError (/usr/local/lib/node_modules/pwmetrics/node_modules/chrome-launcher/dist/utils.js:35:22)
    at new ChromeNotInstalledError (/usr/local/lib/node_modules/pwmetrics/node_modules/chrome-launcher/dist/utils.js:66:9)
    at Launcher.<anonymous> (/usr/local/lib/node_modules/pwmetrics/node_modules/chrome-launcher/dist/chrome-launcher.js:145:27)
    at Generator.next (<anonymous>)
    at fulfilled (/usr/local/lib/node_modules/pwmetrics/node_modules/chrome-launcher/dist/chrome-launcher.js:9:58)
    at process._tickCallback (internal/process/next_tick.js:68:7)
    at Function.Module.runMain (internal/modules/cjs/loader.js:745:11)
    at startup (internal/bootstrap/node.js:282:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:743:3)
  message: 'No Chrome installations found.',
  code: 'ERR_LAUNCHER_NOT_INSTALLED' }
  ✘ Error: Unable to complete run 1 of 1 due to connect ECONNREFUSED <IP_ADDRESS>:9222.

Environment

pwmetrics version: ^4.2.3
Chrome version: Version 80.0.3987.116 (Official Build) (64-bit)
OS version: macOS Catalina 10.15.3

Potential Solution
Update chrome-launcher dependency to a more recent version. Lighthouse root repo uses 0.11.2 currently.
"chrome-launcher": {
      "version": "0.11.2",
      "resolved": "https://registry.npmjs.org/chrome-launcher/-/chrome-launcher-0.11.2.tgz",
      "integrity": "sha512-jx0kJDCXdB2ARcDMwNCtrf04oY1Up4rOmVu+fqJ5MTPOOIG8EhRcEU9NZfXZc6dMw9FU8o1r21PNp8V2M0zQ+g==",
      "requires": {
        "@types/node": "*",
        "is-wsl": "^2.1.0",
        "lighthouse-logger": "^1.0.0",
        "mkdirp": "0.5.1",
        "rimraf": "^2.6.1"
      }
    }

Any updates here? Have same issue!

Possible solution is to set path to Chrome instance directly using flag chromePath
chromePath: "/Applications/Google Chrome Canary.app/Contents/MacOS/Google Chrome Canary"
But hope there is better solution


zproject/wsgi.py has incorrect permissions

In the 2.1.4 release, zproject/wsgi.py did not have proper permissions for the SSO remote auth provider to access the script when running under apache. after changing the permissions to be 0755 the auth began to work again.

@timapril is it possible you've been changing its permissions in after every release before?
Its permissions at 0644 in Git, and appear to always have been: and it has no shebang line, so I'm not sure why 0755 would be preferable.

What did you see the permissions as when it wasn’t working? And what does umask show?
(We override umask to 022 in install and upgrade-zulip-stage-2 but not in upgrade-zulip or upgrade-zulip-from-git, so maybe we need more overrides.)


feat: support rootless containers via home-manager

Here's a PR that pulls in home-manager and adds a new virtualisation.user.quadlet home-manager option.
fixes #2

Thanks! Looks reasonable in general. Will take a closer look over the weekend.

Pushed some changes as discussed in the comments.
I've also included a patch to add the DNS* options to container.nix


Desktop keyboard interaction with RTL or mixed TextDirection is wrong

There seem to be many inconsistencies with how native desktop platforms handle RTL and Flutter.  Navigating through the text in Flutter happens in string order, but on native it happens in visual order.  This is me just repeatedly pressing the right arrow on both platforms:

Native Mac TextEdit
Flutter

Example app
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: Scaffold(
        appBar: AppBar(
          title: Text('TextField Demo'),
        ),
        body: Center(
          child: TextField(),
        ),
      ),
    );
  }
}

Example string: "Cairo" in Arabic is: القاهرة and it's written right to left.
Cause
A lot of our selection manipulation code seems to not consider the RTL case, for example _getRightByWord.  We'll have to update these methods to consider RTL and mixed direction text, and make sure the user experience matches native.
Adding TODOs for this issue in https://github.com/flutter/flutter/pull/86736.

Hi
I've updated the code sample to already add some mixed RTL and LTR text for easier reproduction and to increase the font size for recordings.
Here is what I discovered
Desktop
(macOS, Windows, Linux all have the issue on stable to master) -  (recording from Windows)

stable
beta
dev
master

❌
❌
❌
❌

https://user-images.githubusercontent.com/48603081/129362097-b0bd25ed-0ffe-452f-b465-d27c4d3b861c.mp4
Web in HTML
(macOS, Windows, Linux has no issue stable  but issue reprpduceds from beta  to master) -  (recording from Windows)

stable

✅

https://user-images.githubusercontent.com/48603081/129362923-d25633f8-7cc0-4e53-8ad1-c34700f53b80.mp4

beta
dev
master

❌
❌
❌

https://user-images.githubusercontent.com/48603081/129362924-eb167294-7a8e-4000-a776-80f45bdf79aa.mp4
Web in Canvakasit
(macOS, Windows, Linux all have the issue on stable to master) - -  (recording from Windows)

stable
beta
dev
master

❌
❌
❌
❌

https://user-images.githubusercontent.com/48603081/129362935-5fe320ae-25fe-4add-bebd-06ad6211c358.mp4
Check flutter doctor -v outputs for each channel below

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on Microsoft Windows [Version 10.0.22000.120], locale en-US)
    • Flutter version 2.2.3 at C:\Users\Taha\Code\flutter_stable
    • Framework revision f4abaa0735 (6 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\Taha\Code\android-sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_SDK_ROOT = C:\Users\Taha\Code\android-sdk
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31515.178
    • Windows 10 SDK version 10.0.19041.0

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

[✓] VS Code (version 1.59.0)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.25.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22000.120]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 92.0.4515.131
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 92.0.902.67

! Doctor found issues in 1 category.

[✓] Flutter (Channel beta, 2.5.0-5.1.pre, on Microsoft Windows [Version 10.0.22000.120], locale en-US)
    • Flutter version 2.5.0-5.1.pre at C:\Users\Taha\Code\flutter_beta
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 65cf7de5f4 (2 days ago), 2021-08-11 15:48:01 -0700
    • Engine revision fbeb7e22bd
    • Dart version 2.14.0 (build 2.14.0-377.4.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\Taha\Code\android-sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_SDK_ROOT = C:\Users\Taha\Code\android-sdk
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31515.178
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2020.3)
    • Android Studio at C:\Users\Taha\Code\android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.59.0)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.25.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22000.120]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 92.0.4515.131
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 92.0.902.67

• No issues found!

[✓] Flutter (Channel dev, 2.5.0-6.0.pre, on Microsoft Windows [Version 10.0.22000.120], locale en-US)
    • Flutter version 2.5.0-6.0.pre at C:\Users\Taha\Code\flutter_dev
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 225a43d941 (8 days ago), 2021-08-05 11:32:48 -0700
    • Engine revision 07ec4b82c7
    • Dart version 2.14.0 (build 2.14.0-383.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\Taha\Code\android-sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_SDK_ROOT = C:\Users\Taha\Code\android-sdk
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31515.178
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2020.3)
    • Android Studio at C:\Users\Taha\Code\android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.59.0)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.25.0

[✓] Connected device (4 available)
    • Windows (desktop)       • windows • windows-x64     • Microsoft Windows [Version 10.0.22000.120]
    • Windows (UWP) (desktop) • winuwp  • windows-uwp-x64 •
    • Chrome (web)            • chrome  • web-javascript  • Google Chrome 92.0.4515.131
    • Edge (web)              • edge    • web-javascript  • Microsoft Edge 92.0.902.67

• No issues found!

[✓] Flutter (Channel master, 2.5.0-7.0.pre.65, on Microsoft Windows [Version 10.0.22000.120], locale en-US)
    • Flutter version 2.5.0-7.0.pre.65 at C:\Users\Taha\Code\flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4bd8b288bb (12 hours ago), 2021-08-12 18:02:02 -0700
    • Engine revision 670a681c1d
    • Dart version 2.15.0 (build 2.15.0-6.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\Taha\Code\android-sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_SDK_ROOT = C:\Users\Taha\Code\android-sdk
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31515.178
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 2020.3)
    • Android Studio at C:\Users\Taha\Code\android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] VS Code (version 1.59.0)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.25.0

[✓] Connected device (4 available)
    • Windows (desktop)       • windows • windows-x64     • Microsoft Windows [Version 10.0.22000.120]
    • Windows (UWP) (desktop) • winuwp  • windows-uwp-x64 •
    • Chrome (web)            • chrome  • web-javascript  • Google Chrome 92.0.4515.131
    • Edge (web)              • edge    • web-javascript  • Microsoft Edge 92.0.902.67

• No issues found!

✅: No Issue ❌: Issue reproduced

I believe TextEdit is wrong, and the HTML rendered used to be wrong but @mdebbar fixed it. The way Flutter edits bidi text (i.e. in "string order") is intentional behavior that we consider to be correct. Unless I'm wrong, we can close this bug.

That's right @yjbanov. The web html renderer on stable is buggy, and it was fixed some time ago in master. If you try this on a regular input field in Chrome, you'll get the same behavior as Flutter (try here).
But Mac seems to be in favor of the behavior described in this issue. I tried in multiple apps (including Safari) and they behave as @justinmc showed.
I'm not sure what the right answer here is. Maybe we should change the behavior based on the platform?

Ah I had no idea this was how some platforms work, I should have tried more native apps.  Thanks for pointing that out and thanks @TahaTesser for the great repro platform info.
It seems that the address bar in Chrome on Mac also behaves as I showed, even though actual HTML inputs don't.  More evidence that it's a Mac platform thing.
I guess I'll go ahead and close this issue and keep Flutter's behavior as-is for now, since I'm unsure whether changing it per-platform would really benefit any native RTL language users right now.  I'd love to hear from anyone that has a strong opinion on this though, so feel free to continue the discussion or open a new issue and tag me.


GitHub: wrong url to fontspec on CTAN

(Seems slightly silly to open an issue for this...)
The url in the repo's description should now be something like
http://ctan.org/pkg/fontspec
rather than
http://tug.ctan.org/pkg/fontspec

Thanks :)
Fixed.


Add Microsoft Edge on the list of vendor

It would be nice to also add Microsoft Edge. :)

Thanks @robertpmansion for your feedback. I'll add this feature with the next release.


Convert Client Key Data to supported format on the fly

I have something like
export async function getPatchedLocalKube() {
    const config = await KubeConfig.getDefaultConfig();
    const ctx = config.fetchContext();
    const tlsAuth = await ctx.getClientTls();
    if (tlsAuth?.userKey.includes("BEGIN EC PRIVATE KEY")) {
        console.log(btoa(await convertEcKeyToPKCS8(tlsAuth?.userKey!)));
        ctx.user[ 'client-key-data' ] = btoa(await convertEcKeyToPKCS8(tlsAuth?.userKey!));
    }
    return KubeConfigRestClient.forKubeConfig(config);
}

async function convertEcKeyToPKCS8(key: string) {
    const derStream = new Deno.Command("openssl", {
        args: [ "ec", "-inform", "PEM", "-outform", "DER" ],
        stdin: "piped",
        stdout: "piped",
        stderr: "null"
    }).spawn();

    await new Response(key).body?.pipeTo(derStream.stdin!);

    const pkcs8Stream = new Deno.Command("openssl", {
        args: [ "pkcs8", "-topk8", "-nocrypt", "-outform", "PEM" ],
        stdin: "piped",
        stdout: "piped",
        stderr: "null"
    }).spawn();

    await derStream.stdout?.pipeTo(pkcs8Stream.stdin!);

    return await new Response(pkcs8Stream.stdout).text();
}

Would be could if it could be native + portable :D
After i wrote this code snippet i noticed that i can just modify my kubeconfig. it would be awesome anyway to have some logs or something to notice that it could work but just that the private key format is unsupported

Hmm, it sounds good to report on this sort of quirk. I've done similar messaging a fair bit with other kubeconfig things, saying that e.g. deno can't handle a specific field. And I recently ran into wanting to use a pkcs2 key with webcrypto, was pretty opaque and annoying to fix up.
It's unfortunate that the conversion is easiest with executing openssl. Otherwise this sounds good. I don't actually know exactly what formats Deno supports, so some more research is needed here.


VSCode extension to convert SDL

Like Markdown Preview Enhanced shows Markdown Previews on the side. This would output SDL conversion to Nexus :)
I'm talking about this icon which comes built-in with VSCode for Markdown support. Something like that for SDL would be fun 😜

Sounds neat! Though I'm not sure of the utility - since you should generally only need to do the SDL -> Nexus conversion once, I'm not sure if it would be really needed as a VSCode plugin?
If someone wants to build it though, go for it!

True that but as SDL conversion is already built; it'd be cool to have it as an extension.
Idk a lot of Typescript yet (learning now) so I might make it in future. Till then if anyone wants to take it, happily do so.
An NPM module extracted for SDL conversion might've been easy to make an extension with just JS.
Also, an extension might be good to convert multiple projects. Keep up the good work. Love Nexus ❤

Cool - glad to hear you love Nexus so far! If you (or someone else) takes this on at some point let me know and I can reopen, for now I'm going to close just to clean up the issue log.
Also, convertSDL is exported, and all internal classes etc should be available under the exported core namespace so if you do want to create a converter, go for it!


UML Cardinality Updates

Resulting from test tool implementation the following changes are suggested:

SyntaxTemplate --[:dictionary]--> Dictionary 1:0..1 (May not need a dictionary)
Range--[:unit]-->Code 1:0..1 (May not need a unit, e.g. person count)
Quantity--[:unit]-->Code 1:0..1 (May not need a unit, e.g. person count)
PopulationDefinition--[plannedEnrollmentNumber]-->Range 1:0..1 (May not be needed in both StudyPopulation and Cohort classes, only one needs defining)
PopulationDefinition--[plannedCompletionNumber]-->Range 1:0..1 (May not be needed in both StudyPopulation and Cohort classes, only one needs defining)
PopulationDefinition--[plannedMaximumAge]-->Quantity 1:0..1 (May not be needed in both StudyPopulation and Cohort classes, only one needs defining)
PopulationDefinition--[plannedMinimumAge]-->Quantity 1:0..1 (May not be needed in both StudyPopulation and Cohort classes, only one needs defining)
GeographicScope--[code]-->AliasCode (not needed if type is Global)

@dih-cdisc:
Changed and uploaded for 2.7.1
Changed in 2.8: will upload today after review CT and updates of potential changes.


High Count Of Leaked Connection Count For Data source for the managed server where Conductor is deployed.

I have deployed conductor v 3.9.2 war on one of my managed server on weblogic and that is pointing to PLATO-O data source.
And for that particular managed server I can see increasing leaked connection day by day for that particular data source.
Other managed server which are also pointing to that data source are not showing any leaked connection.
Can anyone help me with this?

I am getting following message:
_####<Sep 6, 2023 8:14:17,865 AM MMT> <Warning> <JDBC> <fcubsprodapp01> <WLS_CONDUCTOR_1> <[ACTIVE] ExecuteThread: '300' for queue: 'weblogic.kernel.Default (self-tuning)'> <<WLS Kernel>> <> <01c23b6c-6303-44c7-9aaa-df93830da3bd-0025e0fa> <1693964657865> <[severity-value: 16] [rid: 0] > <BEA-001592> <Forcibly releasing inactive connection "weblogic.jdbc.wrapper.PoolConnection_oracle_jdbc_driver_T4CConnection@392ebaa0" back into the data source connection pool "PLATO-O".>_

Ensure that you are properly closing database connections using a try-with-resources block or finally block to guarantee closure, like this:
try (Connection connection = dataSource.getConnection()) {     // Use the connection for database operations } catch (SQLException e) {     // Handle exceptions }
Utilize the try-with-resources statement in Java to automatically close resources (like connections) at the end of a block of code. This ensures proper resource cleanup.
try (Connection conn = dataSource.getConnection();      Statement stmt = conn.createStatement();      ResultSet rs = stmt.executeQuery(sql)) {     // Use the connection, statement, and result set } catch (SQLException e) {     // Handle the exception }


Add test-data to the path for failsafe tests.

Move the BPD for the test setup to under target so that it is cleared when running mvn clean.
Description
This makes it so mvn verify can successfully run with the changes to how test data is stored.
Type of change
Please delete options that are not relevant.

[X ] Bug fix (non-breaking change which fixes an issue)

Checklist before requesting a review

[X ] I have performed a self-review of my code
[X ] My code follows the style guidelines of this project
[X ] I have commented my code, particularly in hard-to-understand areas
[X ] My changes generate no new warnings
[X ] New and existing unit tests pass locally with my changes

Nice, I realized that I was only running the surefire tests with mvn test and not the failsafe tests with mvn verify so I was totally missing this issue! Thanks for the fix.


refactor: onClick can use useCallback hook

The onClick event handler creates a new function on every render, which may negatively affect performance. To avoid this, you can use the useCallback hook.
onClick can now use useCallback hook to trigger changes only when there's change wrt to its concerned state (if there's no change in the state then useCallback hook in react returns the same memoized version of function from previous render)

Thank you for your contribution. Just a quick note, I'm not sure if memoizing () => {setIsModalOpen(true);} and () => {_setApiEndpoint(endpoint); setDropDown(false);} is necessary since these functions do not have a significant impact on performance, especially since the components they are used in are not frequently rerendered like the MessageContent component.

Thank you for your contribution. Just a quick note, I'm not sure if memoizing () => {setIsModalOpen(true);} and () => {_setApiEndpoint(endpoint); setDropDown(false);} is necessary since these functions do not have a significant impact on performance, especially since the components they are used in are not frequently rerendered like the MessageContent component.

Yes that's correct, i think its still better, since browser would have lesser bottleneck as the application grow, we could have a common click handler for a component.
But if you want to keep this part simple i totally understand, let me know your thoughts.


feat(styles): Tag component but .tag class name is also used for code examples

The code examples also use a class called .tag and when I named my class .tag as well it overwrote the styles for the code examples.

closed as decision was reached in Roundtable meeting of 25.01.2024


Mermaid diagram script update demo

update script
update script

Plan Result (sandbox-enterprise)
No changes. Your infrastructure matches the configuration.

Plan Result (prod-video-hearings)
No changes. Your infrastructure matches the configuration.

Plan Result (prod-enterprise)
No changes. Your infrastructure matches the configuration.


capture metrics from arbitrary file names

Currently, metrics can capture from stdout/stderr as in this example from https://buildtest.readthedocs.io/en/devel/writing_buildspecs/performance_checks.html#defining-metrics
buildspecs:
  metric_regex_example:
    executor: generic.local.sh
    type: script
    description: capture result metric from output
    run: echo "HPCG result is VALID with a GFLOP/s rating of=63.6515"
    tags: tutorials
    metrics:
      hpcg_rating:
        type: float
        regex:
          exp: '(\d+\.\d+)$'
          stream: stdout

      hpcg_state:
        type: str
        regex:
          exp: '(VALID)'
          stream: stdout

We need to add support for arbitrary file names during performance checks. Therefore we can add another field such as file_regex as a property  so it could be something like this
buildspecs:
  metric_regex_example:
    executor: generic.local.sh
    type: script
    description: capture result metric from output
    run: echo "HPCG result is VALID with a GFLOP/s rating of=63.6515" > hpcg.txt
    tags: tutorials
    metrics:
      hpcg_rating:
        type: float
        file_regex:
          exp: '(\d+\.\d+)$'
          file: hpcg.txt

      hpcg_state:
        type: str
        file_regex:
          exp: '(VALID)'
          file: hpcg.txt

if there is a regex mismatch or file doesn't exist or is not a file then metrics is undefined. This can be later used in any of the status check such as assert_ge.

@wyphan @prathmesh4321
now that i am thinking about this i think we can remove file_regex and just use regex but have a property file that is a sub-property of regex. Since the logic of regex and file_regex is same just which file gets read we can have file or stream be required property but only one can be specified with this change we can see changes to metrics as follows
buildspecs:
  metric_regex_example:
    executor: generic.local.sh
    type: script
    description: capture result metric from output
    run: echo "HPCG result is VALID with a GFLOP/s rating of=63.6515" > hpcg.txt
    tags: tutorials
    metrics:
      hpcg_rating:
        type: float
        regex:
          exp: '(\d+\.\d+)$'
          file: hpcg.txt

      hpcg_state:
        type: str
        regex:
          exp: '(VALID)'
          file: hpcg.txt

Similarly we can have the same test capture metrics using stream
buildspecs:
  metric_regex_example:
    executor: generic.local.sh
    type: script
    description: capture result metric from output
    run: echo "HPCG result is VALID with a GFLOP/s rating of=63.6515"
    tags: tutorials
    metrics:
      hpcg_rating:
        type: float
        regex:
          exp: '(\d+\.\d+)$'
          stream: stdout

      hpcg_state:
        type: str
        regex:
          exp: '(VALID)'
          stream: stdout

Currently, we have a slight difference in regex and file_regex under status, the regex is a single match whereas file_regex is an array of regex match.  see https://buildtest.readthedocs.io/en/devel/buildspecs/buildspec_overview.html#passing-test-based-on-regular-expression and https://buildtest.readthedocs.io/en/devel/buildspecs/buildspec_overview.html#specify-regular-expression-on-arbitrary-filenames

buildspecs:
  status_regex_pass:
    executor: generic.local.bash
    type: script
    tags: [system]
    description: Pass test based on regular expression
    run: echo "PASS"
    status:
      regex:
        stream: stdout
        exp: "^(PASS)$"

  status_regex_fail:
    executor: generic.local.bash
    type: script
    tags: [system]
    description: Pass test based on regular expression
    run: echo "FAIL"
    status:
      regex:
        stream: stdout
        exp: "^(123FAIL)$"

buildspecs:
  regex_on_multiple_files:
    type: script
    executor: generic.local.bash
    description: Test regex on multiple files
    run: |
      echo "Hello" > hello.txt
      buildtest --help > buildtest_help.txt
    status:
      file_regex:
        - file: hello.txt
          exp: '^(Hello)$'
        - file: buildtest_help.txt
          exp: '^(usage: buildtest)'

  regex_on_directory_not_supported:
    type: script
    executor: generic.local.bash
    description: Test regex on directory is not supported
    run: |
      mkdir -p hello
      echo "Hello" > hello/hello.txt
    status:
      file_regex:
        - file: hello
          exp: '^(Hello)$'

  file_expansion_supported:
    type: script
    executor: generic.local.bash
    description: Test regex with variable and shell expansion
    run: |
      echo "Hello" > $BUILDTEST_ROOT/hello.txt
      echo "Hello" > $HOME/hello.txt
    status:
      file_regex:
        - file: $BUILDTEST_ROOT/hello.txt
          exp: '^(Hello)$'
        - file: ~/hello.txt
          exp: '^(Hello)$'

We could potentially leverage the features of file_regex into regex for multiple status check and not have to release a new key to perform file check by stream so we could have something like this
buildspecs:
  regex_on_multiple_files:
    type: script
    executor: generic.local.bash
    description: Test regex on multiple files
    run: |
      echo "Hello" > hello.txt
      buildtest --help > buildtest_help.txt
    status:
      regex:
        - file: hello.txt
          exp: '^(Hello)$'
        - stream: stdout
          exp: '^(usage: buildtest)'

Note that under metrics the regex won't be a list that is similar to how it behaves under status so it would not be correct to have the following
buildspecs:
  metric_regex_example:
    executor: generic.local.sh
    type: script
    description: capture result metric from output
    run: echo "HPCG result is VALID with a GFLOP/s rating of=63.6515" > hpcg.txt
    tags: tutorials
    metrics:
      hpcg_rating:
        type: float
        regex:
            - file: hpcg.txt
              exp: '(\d+\.\d+)$'
            - stream: stdout
              exp: '(\d+\.\d+)$'
          ```


Add new boundary layers for leg districts

Follow on from as-yet unfunded Vision Zero work (eg #94), they are looking to add state senate and assembly districts as boundary types.
Looks like NYC open data has the boundaries we need:
https://data.cityofnewyork.us/City-Government/State-Assembly-Districts/pf5b-73bw
https://data.cityofnewyork.us/City-Government/State-Senate-Districts/h4i2-acfi

State Senate
State Assembly

Current pattern is to load these into CARTO like this:
https://chekpeds.carto.com/tables/nyc_city_council
and then looks like a few places where new boundaries would need to be set up:
https://github.com/search?q=org%3AGreenInfo-Network+nyc_city_council&type=Code
A few details would be determining naming convention and any constraints around adding to the lists/UIs in various spots
Looks like as have integer fields for assem_dist and st_sen_dist. Probably want to preprend these with "Assembly District " and "Senate District " respectively.
Does this need to become part of the ETL script assignment process?

OK, we have a budget for this, CHEKPEDS:Crashmapper LegDistricts in tracking. Could be fill in work before Thanksgiving. I think this will actually be relatively straightforward. (Famous last words!)

Client-Side: Map View
Preliminary list of likely places to change, thanks to searching in Github: https://github.com/GreenInfo-Network/nyc-crash-mapper/search?q=city_council&unscoped_q=city_council
Client-Side: Chart App
Preliminary list of likely code changes, from Github search: https://github.com/GreenInfo-Network/nyc-crash-mapper-chart-view/search?q=city_council&unscoped_q=city_council

Part 3: ETL Script
The ETL script has functions update_neighborhood() et al, which are all the same UPDATE SET WHERE structure. For NYPD Precincts, Borough, Community, etc. these all run in about 1 second. This is despite the fact that some 100k-200k rows fall outside of any area and are thus null.
Assembly and Senate have the expected performance: After 5 seconds, a timeout.
 It gets this right for "city council" and I seem to recall some hardcoded behaviors about how the word is generated from the layer name...

The next day (16 hours later, some 20 hours after creating the tables and columns) the same thing s still happening. The query planner is giving very different and far less efficient results for these 2 new tables and columns, than it does for the previously-existing ones. There are no indexes shown in the pg_indexes table beyond the 3 standard ones for every table. There may be an index out of date that a ANALYZE could help, but we can't do those through the UI.
I have emailed<EMAIL_ADDRESS>since CHEKPEDS has a paid account, and we'll see if they have insight from their side.
Email as follows:
Hello, my name is Greg Allensworth and I'm working with CHEKPEDS. We have a paid account, and I'm hoping you can help me with optimizing a query. Specifically, the query itself is identical to other we have written but the query planner is giving me some strange results.
The following query executes in about 1 second, and we get similar times for some 5 other queries identical to this one for NYPD precincts, boroughs, neighborhoods, ...
UPDATE crashes_all_prod
SET city_council = a.identifier
FROM nyc_city_council a
WHERE crashes_all_prod.the_geom IS NOT NULL AND ST_Within(crashes_all_prod.the_geom, a.the_geom)
AND (crashes_all_prod.city_council IS NULL)
However, I have two new tables nyc_senate and nyc_assembly, and that same query on either of these will time out after 3 seconds:
UPDATE crashes_all_prod
SET senate = a.identifier
FROM nyc_senate a
WHERE crashes_all_prod.the_geom IS NOT NULL AND ST_Within(crashes_all_prod.the_geom, a.the_geom)
AND (crashes_all_prod.senate IS NULL)
General record counts etc. for the "nyc_senate" and "nyc_city_council" are not too dissimilar, and tallies for crashes_all_prod with geom NOT null and field IS null are similar. Also, there is little-to-no variation in the timing of the 5 other tables where we do this same query for precincts, neighborhoods, boroughs, etc. It is just these 2 new tables.
The EXPLAIN output is puzzling, showing that these two new tables get a very different query plan.
The fast query has very sensible planner output. The first pass is a scan over crashes_all_prod where geom is NOT null and precinct IS null; this excludes some 99% of records. It then uses the spatial index to relate point to polygon. Great.
-- FAST query for nypd_precinct et al
Update on crashes_all_prod  (cost=0.14..351969.43 rows=8563 width=386)
->  Nested Loop  (cost=0.14..351969.43 rows=8563 width=386)
->  Seq Scan on crashes_all_prod  (cost=0.00..99236.23 rows=174755 width=364)
Filter: ((the_geom IS NOT NULL) AND (nypd_precinct IS NULL))
->  Index Scan using cartodb_query_the_geom_idx on nyc_nypd_precinct a  (cost=0.14..1.44 rows=1 width=1108)
Index Cond: (the_geom ~ crashes_all_prod.the_geom)
Filter: _st_contains(the_geom, crashes_all_prod.the_geom)
This new query, though, very different output. It loops over polygons, does the geometry null in the first pass along with the bbox, and so on. Clearly less efficient.
-- SLOW query for Assembly and Senate
Update on crashes_all_prod  (cost=56.26..323460.12 rows=3663 width=386)
->  Nested Loop  (cost=56.26..323460.12 rows=3663 width=386)
->  Seq Scan on nyc_assembly a  (cost=0.00..29.65 rows=65 width=23883)
->  Bitmap Heap Scan on crashes_all_prod  (cost=56.26..4975.15 rows=56 width=368)
Recheck Cond: ((a.the_geom ~ the_geom) AND (the_geom IS NOT NULL))
Filter: ((assembly IS NULL) AND _st_contains(a.the_geom, the_geom))
->  Bitmap Index Scan on etl_test_copy_the_geom_idx  (cost=0.00..56.25 rows=1313 width=0)
Index Cond: ((a.the_geom ~ the_geom) AND (the_geom IS NOT NULL))
Do you have any insight from your end, as to why the query planner would behave so differently for these two new tables? Their structure is identical to these other tables. I checked the pg_indexes table as well, and all tables have just CARTO's 3 standard indexes. Could it be because these tables are new, or that the "assembly" and "senate" columns in crashes_all_prod are new? Would a VACUUM or ANALYZE help?

Hmm. I just did a local checkout and run of this code and it seems to be working fine for me in terms of speed -- about the same as other geom type queries. Maybe whatever database weirdness was happening has resolved itself through some maintenance process we don't see?
But I did find a new-to-me bug that I see now is more generally about selected geog ID mismatches between map and chart app. Will file that separately.

New ETL script is in place. After 2 weeks of email exchanges with CARTO, they couldn't figure out why the query planner was going the way it was, thus why it wasn't able to run in <3 seconds. So I gave up and instead have it run in 10 "slices" using AND cartodb_id % 10 = X so Senate and Assembly are now 10 queries at 0.2 sec apiece instead of 1 query that times out at 10 seconds.
About 1 hour from now, the ETL should run and https://dashboard.heroku.com/apps/nyc-crash-mapper-etl/logs should indicate successful update of these new senate and assembly fields, in 20 reasonably-timed queries.


Does the Azure SQL Database ledger support video, audio, image, pdf?

According to the row of "Restricted data types" in the "Limitations" table of the Azure SQL Ledger database, does the Azure SQL Ledger database support video, audio, image, and pdf datatypes?
If the Azure SQL Ledger database is a SQL database, how do the supported data types of SQL database differ from the SQL Ledger database, which "xml" and "sql_variant" is supported in Azure SQL Database but not supported in Azure SQL Ledger Database?

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

ID: f9f12685-ec4a-67d6-32a3-ce2e86a99ca1
Version Independent ID: c067a930-785f-a7d5-0654-6e387ec49913
Content: Limitations for Azure SQL Database ledger - Azure SQL Database
Content Source: articles/azure-sql/database/ledger-limits.md
Service: sql-database
Sub-service: security
GitHub Login: @VanMSFT
Microsoft Alias: vanto

Hi @AminAvan - The ledger feature is part of Azure SQL Database, and there should be no difference to the supported data types other than the ones listed in the doc. Whether it supports audio/video is dependent on the data type used.
@Pietervanhove - Can you please confirm?

That is correct yes. All data types are supported in a ledger table except for:

XML
SqlVariant
User-defined data type
FILESTREAM

Thanks @Pietervanhove!
@AminAvan - Since this question has been addressed, we will go ahead and close this Issue. Feel free to provide additional comments.
#please-close


Add Safari & iOS Safari 13.1

the obligatory issue 0:-)
found no changelog so far, but at the latest TP 91 suggests there is a delta to 13 (via #5067)…

Added!

Safari (for macOS) 13.1, unlike iOS Safari, does not exist yet. The latest version of Safari (for macOS) is 13.0.2 beta released in Sep 30, and there is no presence of "Safari 13.1" in WebKit's release trackers either.
@Fyrd 9c7e28284550c51728f897340f830abe6487c586 needs to be partially reverted.

Safari TP 93 from yesterday still reports as 13.1 tho, just like 91…

(Changelog: https://webkit.org/blog/9600/release-notes-for-safari-technology-preview-93/)

I think it’d be better if we stick to the ‘official beta version strings’ for macOS Safari, just like iOS Safari, not only for consistency but also an issue I’ve stumbled upon while preparing a PR: Some features are dependent on the OS version. Although we know the current TP is only available on Mojave and Catalina, there is no assurance 13.1+ would not support High Sierra.
For example, while TP 84 started dropping the support for High Sierra,, its eventual stable release "Safari 13" supports High Sierra. Info.plist in TP 88 says it's 13.0 and TP 90 says it's 13.1, but we know 13.0.2 is currently in the beta stage. TP 89 can't be both 13.0.1 and 13.0.2. The bottom line is, we can't really trust Safari version string in a TP.
On top of that, some recent changesets are frequently cherry-picked to be backported into the "old" version strings, which makes this more complicated. These uncertainties add guesswork or other patches to correct the record afterwards.


save_model on regressor results in crash (Windows 10)

I get this error, causing python to crash, when I invoke the save_model method after a model run.  Trees are about 10+ deep, and 200 trees to save.   Any ideas?

We couldn't reproduce your issue. Would you please try to run ThunderGBM using command-line? Our command-line version also saves the trained model. This should help you identify whether the error is from ThunderGBM or other parts of your code.

we are a python shop so am not sure even how to get those arrays outside python. let me know....
also, we are attempting to migrate from scikit learn’s random forest, which has an option ‘ min_samples_leaf’ ...
Is it possible to have this option or guidance on how to replicate that parameter in thundergbm?   Also can thundergbm apply a weighting vector to put more weight on recent samples?
The goal is to attempt to replicate the process into making comparable output,  but run faster.
Also thoughts in the xgboost 1.0 random forest gpu implementation?  seems the speed gains are about 2x for that versus 6x on yours, but it is hard to tell if i am making a fair comparison because the parameter menu is a bit different.
Michael Krause, CFA
Counterpoint Mutual Funds, LLC
12760 High Bluff Drive, Suite #280
San Diego, CA  92130x-apple-data-detectors://1/0
Phone:<PHONE_NUMBER>tel:(858) 792-0104
mkrause<EMAIL_ADDRESS>The person sending this e-mail is an employee of Counterpoint Mutual Funds, LLC, a registered investment adviser. This message, including any attachments, is intended only for the use of the individual or entity to which it is addressed and may contain information that is privileged, confidential, and exempt from disclosure under applicable law. Any dissemination, distribution, copying, or other unauthorized use of or reliance upon the information contained herein in whole or in part is strictly prohibited. If you have received this communication in error, please immediately notify us by e-mail, telephone, or facsimile, and delete this message from your system.
On Dec 15, 2019, at 6:13 PM, Zeyi Wen<EMAIL_ADDRESS>wrote:

We couldn't reproduce your issue. Would you please try to run ThunderGBM using command-line? Our command-line version also saves the trained model. This should help you identify whether the error is from ThunderGBM or other parts of your code.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Xtra-Computing/thundergbm/issues/35?email_source=notifications&email_token=AGBMC36UWENHLBYV5CQX5W3QY3P3RA5CNFSM4J26T622YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG5JVNQ#issuecomment-565877430, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGBMC33MP5GXAUNLLAWPZILQY3P3RANCNFSM4J26T62Q.

How to best get the arrays outside python for command line execution?
also, we are attempting to migrate from scikit learn’s random forest, which has an option ‘ min_samples_leaf’ ...
Is it possible to have this option or guidance on how to replicate that parameter in thundergbm?   Also can thundergbm apply a weighting vector to put more weight on recent samples?

Also, where to download the package for command line execution (for Windows 64 environments)?  I just installed the python wheel for Windows...

You need to build the executable first. Please refer to the document for more details.


chore(tech-debt): whisper device and model should be set during initialization

Describe what should be investigated or refactored
The device and model variables for the Whisper model are getting reinitialized every time make_transcribe_request is called.
These fields should be initialized during the initialization of the class Whisper instead of every time the function is called.

Context
I believe this was put in there for handling multiple concurrent requests. We load a new instance of our whisper model on to the GPU per transcription that comes from the API to the backend. @CollectiveUnicorn might have a better memory than me on this.
Take a look at the faster-whisper documentation for GPU inferencing. It only uses CUDA or CPU, and we shouldn't make that a runtime decision, maybe at best a deploy time decision which is the current state of this code/charts.


[BUG] Play some .wav file have tail noise

cocos2d-x version: latest
devices test on: windows
developing environments

NDK version:
Xcode version:
VS version: vs2019
browser type and version:

Steps to Reproduce:
see: Roll.zip

Caused by the workaround code on non-apple platforms: https://github.com/cocos2d/cocos2d-x/issues/16938


[AVCe] fix for SW BRC

Issue: MDP-50659
Test: manually

@ogolikov : this one also needs rebase on top the latest master.


Training on a custom dataset possible?

Hello team, thanks for this repo! I was wondering if your model could be trained on a different dataset than you used in your paper - for example the Colon Nuclei dataset.
To be brief, this dataset has info about HE images, the segmented nuclei and their classification into 6 types.
Thanks!

Hello, thank you for the interest for our framework :) . Absolutely! You can train the model with a different dataset with different cell classes.

Hi, thanks for this info. Is there a preprint or a paper I can have a look at to see the result metrics?

Hello, I updated the README and I added the link for the preprint version of the article.


[$1000] Web - Clicking browser back button takes user either to home page or does nothing while on magic code page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to sign in page on ND
Paste an email and click continue
While on the magic code page click on the browser back button

Expected Result:
App should take user back to the get started page
Actual Result:
App either takes user to the browser home page or does nothing
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.3.57-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/115492554/8a40195a-a646-4ed9-ad79-739fe9f88d68
https://github.com/Expensify/App/assets/115492554/e7a590b8-262f-45dd-b066-43f77e29eff2
Expensify/Expensify Issue URL:
Issue reported by: @Nathan-Mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1692046450739489
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0169f5f9316f0edd34
Upwork Job ID:<PHONE_NUMBER>001395200
Last Price Increase: 2023-08-27

@johnmlee101 @NikkiWines this feels like a regression?

Proposal
Please re-state the problem that we are trying to solve in this issue.
When click browser back button the Apps return to browser home page
What is the root cause of that problem?
Because you maintain on one page SignInPageLayout

Page input for email or LoginForm component
Page input validate code form or ValidateCodeForm component
and two other things again UnlinkLoginForm and EmailDeilveryFailurePage component

You doesn't set this to separate route, only for check state after submit form. After that, showing component based on current state. So when you click browser back button your problem its happen because in the fact you not move to new route after submit form.
What changes do you think we should make in order to solve the problem?
If still stay with one page SignInPageLayout we can check based on state, I saw you have condition to check it for showing component. After that, do preventif back browser listener with popstate native event browser and add function for disabled move history back browser and add handle check state for showing component

Proposal
Please re-state the problem that we are trying to solve in this issue.
When click browser back button the Apps return to browser home page
What is the root cause of that problem?
Because you maintain on one page SignInPageLayout

Page input for email or LoginForm component
Page input validate code form or ValidateCodeForm component
and two other things again UnlinkLoginForm and EmailDeilveryFailurePage component

You doesn't set this to separate route, only for check state after submit form. After that, showing component based on current state. So when you click browser back button your problem its happen because in the fact you not move to new route after submit form.
What changes do you think we should make in order to solve the problem?
If still stay with one page SignInPageLayout we can check based on state, I saw you have condition to check it for showing component. After that, do preventif back browser listener with popstate native event browser and add function for disabled move history back browser and add handle check state for showing component

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/fl/hendrifaisalhidayat

Proposal
Please re-state the problem that we are trying to solve in this issue.
When click browser back button the Apps return to browser home page
What is the root cause of that problem?
Because you maintain on one page SignInPageLayout

Page input for email or LoginForm component
Page input validate code form or ValidateCodeForm component
and two other things again UnlinkLoginForm and EmailDeilveryFailurePage component

You doesn't set this to separate route, only for check state after submit form. After that, showing component based on current state. So when you click browser back button your problem its happen because in the fact you not move to new route after submit form.
What changes do you think we should make in order to solve the problem?
If still stay with one page SignInPageLayout we can check based on state, I saw you have condition to check it for showing component. After that, do preventif back browser listener with popstate native event browser and add function for disabled move history back browser and add handle check state for showing component

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/hendrifaisalhidayat

@Nathan-Mulugeta This looks like ideal behaviour, as the back button handles only navigational changes, but there were no navigational changes in this case.

I think users expect browser back button would have the same use as the go back button IMO.

Proposal
Please re-state the problem that we are trying to solve in this issue.
When the user presses the back button on their browser, they may return to the home page or remain on the same page if they are on the magic code page.
What is the root cause of that problem?
I believe it's designed this way. The verification code page and sign-in page are combined into one page with the same URL. Consequently, clicking the browser's back button will lead to the browser's home page.
What changes do you think we should make in order to solve the problem?
We need to add following function in componentDidMount lifecycle using useEffect.

https://github.com/Expensify/App/blob/main/src/pages/signin/ValidateCodeForm/BaseValidateCodeForm.js

useEffect(() => {
    window.history.pushState({}, '');
    const handlePopstate = (event) => {
      event.preventDefault();
      clearSignInData();
    };
    window.addEventListener('popstate', handlePopstate);

    return () => {
      window.removeEventListener('popstate', handlePopstate);
    };
}, []);

https://github.com/Expensify/App/blob/main/src/pages/signin/EmailDeliveryFailurePage.js

useEffect(() => {
  window.history.pushState({}, '');

  const handlePopstate = (event) => {
      event.preventDefault();
      redirectToSignIn();
  };
  window.addEventListener('popstate', handlePopstate);

  return () => {
      window.removeEventListener('popstate', handlePopstate);
  };
}, []);

What alternative solutions did you explore? (Optional)
We also have the option to set up these pages as two distinct routes.

Yeah, I think this is expected behavior. There isn't any navigation performed and there's an in-product go back link if the user wants to go back to the login page.
I don't think this is a bug cc: @laurenreidexpensify

Yeah it was never in the design to allow for browser navigation to dictate the internal flow for sign ins, I agree with @NikkiWines I don't think its a bug, more like a feature proposal if anything


[WIP] [DO NOT MERGE] feat(atomic-commerce): spike atomic for commerce

The goal of this PR is to get a feel for what the work needed in atomic to support the new headless controllers looks like.
It doesn't build, as we are missing key headless controllers to support displaying results. For example, we're missing result list and results per page controllers. Tasks for these have been created in CAPI-566 and CAPI-567.
This work was done as part of the CAPI-230 epic, and findings are detailed in this document.

Closing this. Feel free to comment in the document linked in the PR description if you have any questions/comments!


Texture bug fix: flip_t didn't flip the result t derivative.

Fixes #1428
Added a FIXME about an unrelated issue, I will investigate soon but did not want to garble this quick fix review.

LGTM. I agree this is necessary - can you think of a test case (maybe in OSL?) to show that it works?

Yes, it will be easy to add a test.

Changed the existing testsuite/texture-flipt to also check the derivatives.


Route53 DNS service fails to set DNS records if host has custom labels

If an instance is created with one or more custom labels, the Route53 catalog service fails to set DNS records with the error:

json: cannot unmarshal array into Go value of type string

We can consistently replicate this:
Create a host without any labels, output of Route53 DNS service:

15/10/2016 17:55:14time="2016-10-15T16:55:13Z" level=info msg="Starting Rancher External DNS service"
15/10/2016 17:55:19time="2016-10-15T16:55:19Z" level=info msg="Configured Route 53 with hosted zone 'foo.com.' in region 'eu-west-1' "
15/10/2016 17:55:19time="2016-10-15T16:55:19Z" level=info msg="Healthcheck handler is listening on :1000"
15/10/2016 17:55:19time="2016-10-15T16:55:19Z" level=info msg="DNS records to add: [{test-app.test-app.testing.foo.com. [SECRET] A 300}]"
15/10/2016 17:55:19time="2016-10-15T16:55:19Z" level=info msg="Adding dns record: {test-app.test-app.testing.foo.com. [SECRET] A 300}"

Create a host with one label (foo=bar), output of Route53 DNS service:

15/10/2016 18:02:04time="2016-10-15T17:02:04Z" level=info msg="Starting Rancher External DNS service"
15/10/2016 18:02:11time="2016-10-15T17:02:11Z" level=info msg="Configured Route 53 with hosted zone 'foo.com.' in region 'eu-west-1' "
15/10/2016 18:02:11time="2016-10-15T17:02:11Z" level=info msg="Healthcheck handler is listening on :1000"
15/10/2016 18:02:11time="2016-10-15T17:02:11Z" level=info msg="json: cannot unmarshal array into Go value of type string"
15/10/2016 18:02:12time="2016-10-15T17:02:12Z" level=info msg="json: cannot unmarshal array into Go value of type string"

Create a host with two labels (foo=bar, bar=foo), output of Route53 DNS service:

15/10/2016 18:07:41time="2016-10-15T17:07:41Z" level=info msg="Starting Rancher External DNS service"
15/10/2016 18:07:47time="2016-10-15T17:07:47Z" level=info msg="Configured Route 53 with hosted zone 'foo.com.' in region 'eu-west-1' "
15/10/2016 18:07:47time="2016-10-15T17:07:47Z" level=info msg="Healthcheck handler is listening on :1000"
15/10/2016 18:07:47time="2016-10-15T17:07:47Z" level=info msg="json: cannot unmarshal array into Go value of type string"
15/10/2016 18:08:14time="2016-10-15T17:08:14Z" level=info msg="json: cannot unmarshal array into Go value of type string"

All hosts are AWS EC2 instances with RancherOS (v0.6.1 (4.4.19)), and we haven't tested it with other DNS services.

Useful
Info

Versions
Rancher v1.2.0-pre4-rc2 Cattle: v0.170.1 UI: v1.2.13 Route53 Service: v0.5.0

Access
localauth admin

Orchestration
Cattle

Route
host.containers

@adammwood Can not reproduce this. Could you get the hosts metadata from within a container and paste the output? ie. curl --header 'Accept: application/json' 'http://rancher-metadata/2015-12-19/hosts'

@janeczku Thanks, I think this shows the problem. I've just created a new host with a custom label to get this output.
[
  {
    "agent_ip": "SECRET",
    "hostId": 29,
    "hostname": "test--1",
    "labels": {
      "foo": [
        "bar"
      ],
      "io.rancher.host.agent_image": "rancher/agent:v1.1.0-rc1",
      "io.rancher.host .docker_version": "1.12",
      "io.rancher.host.linux_kernel_version": "4.4"
    },
    "memory":<PHONE_NUMBER>,
    "name": "test--1",
    "uuid": "b757ae0d-0089-47d8-9aef-acd7f9b78679"
  }
]

I'm guessing that the label's value should be string, but it's in an array.

@adammwood Yes there is the problem indeed. Was the host added using custom host method or with the EC2 integration?

@janeczku All our hosts are added through the EC2 integration. This probably explains some other strange behaviour that we've seen recently when using labels... The Route53 service was the only issue that we noticed was directly affected by using them.
I've created another host with a few labels and see the following in the API:
{
  "labels": {
    "foo1": "bar1",
    "foo2": "bar2",
    "foo3": "bar3"
  }
}

... but when I get the host metadata from a container I get this:
[
  {
    "labels": {
      "foo1": [
        "bar1"
      ],
      "foo2": [
        "bar2"
      ],
      "foo3": [
        "bar3"
      ],
      "io.rancher.host.agent_image": "rancher/ agent:v1.1.0-rc1",
      "io.rancher.host.docker_version": "1.12",
      "io.rancher.host.linux _kernel_version": "4.4"
    }
  }
]

Should I close this issue and open one specifically about the labels?

@adammwood I have created an issue for this here https://github.com/rancher/rancher/issues/6287.
Could you add your observation about label values showing as strings in API but as arrays in Metadata there?


Sevenzip support

Description
There is implementation of different archive type to store. Why 7z? Because it is more efficient than zip, compressor use more than one CPU core and utilize them better. Archives become smaller, approximately by 30% percent.
I used library and passed archive type in CommonOptions, because i think that it is common parameter to control whole Rugby
Also added max-connections option to control the number of concurrent connections to HTTP server. It increased performance in our case up to 40%.
References
There is difference for archives sizes between zip and 7z. For test i uploaded binaries with both types

Checklist (I have ...)

[x] 🧐 Followed the code style of the rest of the project
[x] 📖 Updated the documentation, if necessary
[x] 👨🏻🔧 Added at least one test which validates that my change is working, if appropriate
[x] 👮🏻♂ Run make lint and fixed all warnings
[x] ✅ Run make test and fixed all tests

❤ Thanks for contributing to the 🏈 Rugby!

Hello @dimones!
Thank you for your request. I will look into it soon.

@dimones This performance improvement looks quite interesting.
I was wondering what you use instead of the "zip -r" command in your build agents.
(https://github.com/swiftyfinch/Rugby/blob/main/Docs/remote-cache.md#example-of-uploading-to-s3)

@dimones Thank you for your contribution!


Import recent research articles elifesciences/issues#8938

@scottaubrey it might be useful to show me how to roll this out to test but not prod.

@scottaubrey it might be useful to show me how to roll this out to test but not prod.

Should have a bit of time tomorrow PM


Add self:phpcs task

A self:php task should also be present, that does at least self:phpcs, but also maybe self:phpmd when that exists.

Sorry - this doesn't make sense - the toolkit has no PHP!


Tidy inconsistent indenting

Take for example anevent96.fpp. The indenting is inconsistent throughout the file
Fix this and other instances

calculate_f2
do_all
isr_calculate
isr_cuts
isr_histos
isr_init
readdat
steer


Push docker images to docker hub

The current server deploy process:

On commit to main...
Run build.sh to generate medplum-server.tar.gz
Run deploy-server.sh which calls docker build and docker push

Before: Only pushed to AWS ECR, which is not publicly available
After: Push to both AWS ECR and to Docker Hub
In the future, we want to also experiment with using Docker Hub for production deploys as well, and removing AWS ECR entirely.  That will provide more transparency, and make it easier for developers to setup self-hosting.

Coverage remained the same at 94.475% when pulling c6eaaa7f51d66aec2818a2c3641a4199c0409453 on cody-push-to-docker-hub into 7b378f684ad76da846ddb12b2047c9e47a61f231 on main.


How to obtain notifications

how-to-work-with-notifications article added

facelifted WebSockets and billing pages. Added examples, descriptions, fixed little mistakes.

Will see next week.
@Astarsys resolve conflict, please.

Will see next week.
@Astarsys resolve conflict, please.


Panic when despawning entity with child scene just after creation

Bevy version
main branch (71bf07f5c085b9c02cbe5337040337c3ed8b0002)
Operating system & version
OS: Windows 10
Rust toolchain: stable-x86_64-pc-windows-gnu 1.52.1
What you did
Despawning an entity with a GLTF scene child one update after its creation leads to a panic.
Here is minimal code reproducing the issue.
use bevy::{
    prelude::*,
};

#[derive(Default)]
struct State {
  entity: Option<Entity>,
  delay: usize,
}

fn update(
    mut commands: Commands,
    assets: Res<AssetServer>,
    mut state: Local<State>,
) {
    if let Some(id) = state.entity {
        if state.delay == 0 {
            commands.entity(id).despawn_recursive();
            state.entity = None;
        } else {
            state.delay -= 1;
        }
    } else {
        let id = commands
            .spawn()
            .with_children(|parent| {
                let handle: Handle<Scene> = assets.load("alien.glb#Scene0");  // no panic with "alien.glb"
                parent.spawn_scene(handle);
            }).id();
        state.entity = Some(id);
        state.delay = 0;  // no panic when set to 1
    }
}

fn main() {
    App::build()
        .add_plugins(DefaultPlugins)
        .add_system(update.system())
        .run();
}

What you expected to happen
Entity is despawned, no panic.
Sample program above will spawn/despawn entities indefinitely.
What actually happened
After the first despawn, program panics with an Entity does not exist error and the following backtrace (truncated):
thread 'main' panicked at 'Entity does not exist', C:\Users\user\bevy\crates\bevy_ecs\src\world\mod.rs:215:37
stack backtrace:
   0: rust_begin_unwind
             at /rustc/9bc8c42bb2f19e745a63f3445f1ac248fb015e53\/library\std\src/panicking.rs:493:5
   1: core::panicking::panic_fmt
             at /rustc/9bc8c42bb2f19e745a63f3445f1ac248fb015e53\/library\core\src/panicking.rs:92:14
   2: core::option::expect_failed
             at /rustc/9bc8c42bb2f19e745a63f3445f1ac248fb015e53\/library\core\src/option.rs:1321:5
   3: core::option::Option<T>::expect
             at /rustc/9bc8c42bb2f19e745a63f3445f1ac248fb015e53\library\core\src/option.rs:349:21
   4: bevy_ecs::world::World::entity_mut
             at C:\Users\user\bevy\crates\bevy_transform/C:\Users\user\bevy\crates\bevy_ecs\src\world/mod.rs:215:9
   5: <bevy_ecs::system::commands::Insert<T> as bevy_ecs::system::commands::Command>::write
             at C:\Users\user\bevy\crates\bevy_transform/C:\Users\user\bevy\crates\bevy_ecs\src\system/commands.rs:356:9
   6: bevy_ecs::system::commands::CommandQueue::apply
             at C:\Users\user\bevy\crates\bevy_ecs/C:\Users\user\bevy\crates\bevy_ecs\src\system/commands.rs:27:13
   7: <bevy_ecs::system::commands::CommandQueue as bevy_ecs::system::system_param::SystemParamState>::apply
             at C:\Users\user\bevy\crates\bevy_ecs/C:\Users\user\bevy\crates\bevy_ecs\src\system/system_param.rs:489:9
   8: <(P0,P1,P2,P3) as bevy_ecs::system::system_param::SystemParamState>::apply
             at C:\Users\user\bevy\crates\bevy_transform/C:\Users\user\bevy\crates\bevy_ecs\src\system/system_param.rs:1107:19
   9: <bevy_ecs::system::function_system::FunctionSystem<In,Out,Param,Marker,F> as bevy_ecs::system::system::System>::apply_buffers
             at C:\Users\user\bevy\crates\bevy_transform/C:\Users\user\bevy\crates\bevy_ecs\src\system/function_system.rs:338:9
  10: <bevy_ecs::schedule::stage::SystemStage as bevy_ecs::schedule::stage::Stage>::run
             at C:\Users\user\bevy\crates\bevy_ecs/C:\Users\user\bevy\crates\bevy_ecs\src\schedule/stage.rs:830:25
  11: bevy_ecs::schedule::Schedule::run_once
             at C:\Users\user\bevy\crates\bevy_ecs/C:\Users\user\bevy\crates\bevy_ecs\src\schedule/mod.rs:208:13
  12: <bevy_ecs::schedule::Schedule as bevy_ecs::schedule::stage::Stage>::run
             at C:\Users\user\bevy\crates\bevy_ecs/C:\Users\user\bevy\crates\bevy_ecs\src\schedule/mod.rs:226:21
  13: bevy_app::app::App::update
             at C:\Users\user\bevy\crates\bevy_app/C:\Users\user\bevy\crates\bevy_app\src/app.rs:62:9
<snipped>

Additional information

alien.glb is from bevy's assets, but I reproduced with other .glb files
It does not panic when using alien.glb asset path instead of alien.glb#Scene0.
It does not panic when delaying the despawn_recursive() to the next system call (initialize state.delay to 1 after spawn in sample code).
I'm not familiar with entity parenting, nor if it could be a known limitation of GLTF scenes. May I have misused something?

In the error message at this line:

<bevy_ecs::system::commands::Insert as bevy_ecs::system::commands::Command>::write

It appears that it's the Insert command that's panicking.

I added some traces, and there is an extra Insert command for the panic case.
Without delay, two Insert commands are added then written:

spawn parent 0v0
despawn parent 0v0
insert PreviousParent(0v0) to 1v0
insert Children([1v0]) to 0v0
write Insert<PreviousParent> for 1v0
write Insert<Children> for 0v0 (panic)

With delay, there is only one Insert command:

spawn parent 0v0
insert PreviousParent(0v0) to 1v0
write Insert<PreviousParent> for 1v0
despawn parent 0v0

Both commands are added by parent_update_system().
The Insert<Children> command is added to re-attach the child (1v0) to its despawned parent (0v0), after detecting a parent change.

The Insert command is added to re-attach the child (1v0) to its despawned parent (0v0), after detecting a parent change.

Ahh interesting haha.
So despawning is triggering the Change<> filter, which is not what we always want because you can't insert() on a despawned entity haha.

it goes like this:

stage Update

your system spawns entity 0, attach the scene to it

frame change
stage PreUpdate

scene spawner spawn the scene, adding Parent(0) to it's root (which is entity 1)

stage Update

despawn entity 0

stage PostUpdate

system parent_update_system kicks in, notice that entity 1 has a Parent(0) component and try to sinc everything. This crashes as you despawned the entity in the previous stage


ERR PuppetServiceImpl grpcError() messageSendMiniProgram() rejection

1. Versions

What is your wechaty version?

Answer:  with docker  wechaty/wechaty:0.63

Which puppet are you using for wechaty? (puppeteer/padlocal/service...)

Answer: padlocal
2. Describe the bug
Not Sure what is going wrong, I'm try to send miniprogram to chatroom. it appeared suddenly after 2022-01-08 17:30, before this time, the miniprogram can be send successful.
3. Full Output Logs
09:46:23 VERB PuppetServiceImpl messageSendMiniProgram()
09:46:23 ERR PuppetServiceImpl grpcError() messageSendMiniProgram() rejection: [tid:c4a6e148] [173ms] download file failed:{"host":{"host":"<IP_ADDRESS>","port":80},"payload":"qwAAA3tOIHBwqjsAAAAAAAAAAAAAAAADYgAAAAN2ZXIAAAABMQAAAAl3ZWl4aW5udW0AAAAKMTAwMTAyNTY0OAAAAANzZXEAAAACNDQAAAANY2xpZW50dmVyc2lvbgAAAAkzODU4NzgwNDgAAAAMY2xpZW50b3N0eXBlAAAADGlQYWQgaU9TMTIuMgAAAAdhdXRoa2V5AAAAQDA+AgEBBDcwNQIBAQIBAQIEO6pwcAIBAAIBAAIEXPjAKgIDPRS8AgRvwfRtAgStyoRyAgRkCvwVAgTBbijsBAAAAAAHbmV0dHlwZQAAAAExAAAADGFjY2VwdGR1cGFjawAAAAExAAAABnJzYXZlcgAAAAExAAAACHJzYXZhbHVlAAAAgB2Kpl0HkpyKR9sJwp6FnFeJin5I5YtJjomJfj190pwmwUXcjuoVjqEf3YccyeunP0A6fWCcXTgUJKHhNjl1Vy1LIZ1/fpxIoL20nzdD0a/12ojwxwr7h8TgDrtBvq6ggz02Lzsir5Oy11X6Iu/QxXIpShG8vTmRtnnqhYqqO2KZAAAACGZpbGV0eXBlAAAAATMAAAAKd3hjaGF0dHlwZQAAAAEwAAAABmZpbGVpZAAAAQIzMDdmMDIwMTAwMDQ3ODMwNzYwMjAxMDAwMjA0MDgwOWZhYjcwMjAzMmY1NmMxMDIwNDVkZTVlNzczMDIwNDYxNGU4ZjQ4MDQ1MTc3Nzg3NTcwNmM2ZjYxNjQ1Zjc3Nzg2OTY0NWY3NjM5NjI3MTc5NzEzNTMxNmU3ODYxNmEzMjMyMzczNzMzMzkzMzVmMzEzNjMzMzIzNTMzMzkzMjMwMzQ1ZjY0MzQzNDY1MzQzNTY0MzcyZDMyNjU2NDMxMmQzNDM3Mzc2MzJkNjEzMjMwNjQyZDY0Mzk2MzY1MzczNjMxMzg2MTMwMzEzMzAyMDQwMTE0MDAwMzAyMDEwMDA0MDAAAAALbGFzdHJldGNvZGUAAAABMAAAAAVpcHNlcQAAAAEwAAAADWNsaS1xdWljLWZsYWcAAAABMAAAAAl3eG1zZ2ZsYWcAAAAAAAAAC3d4YXV0b3N0YXJ0AAAAATEAAAANZG93bnBpY2Zvcm1hdAAAAAExAAAABm9mZnNldAAAAAEwAAAAC2xhcmdlc3ZpZGVvAAAAATAAAAAKc291cmNlZmxhZwAAAAEw","unpackaeskey":"UT6NXpemlZe/PoICyoqoVQ=="}, retcode: -5103059
09:46:24 VERB PuppetServiceImpl roomPayload()
09:46:24 VERB Puppet roomPayload(19489980059@chatroom)

it caused by wrong the MiniProgram data format, issue close.


Publish Messages from file

Dear Team, I have download Service Bus extension and integrated with JMeter.
I would like to know, how can we publish messages by reading file. Message contains body and user properties.
Our use case demands publishing 10000 messages for one hour. Or publish 1/2/4/5 message per second.
Kindly support me.

Hi maheshmahadevaiah,
Do you want to know how to use the Service Bus extension?
Here is a tutorial for it, could you please read this first?
https://blog.pnop.co.jp/jmeter-azure-service-bus_en/

Hi Kuniteru, I was able to publish messages to ServiceBus from above article.
My use-case is to publish message by reading a file and control the flow by sending 1/2/4/5 messages per second.
Just guide me how to make use of your extension to implement above use-case.

How about reading messages from a file using the CSV Data Set Configuration Element, _CSVRead function, _StringFromFile function, etc.?
It can be used in the same way as it is used for HTTP requests.

I'm not a part of this request, but it seem like a simple JMeter usage issue rather than a valid request. Might be wise to close.

Thanks, @gbroner2000
As you said, I think this is a simple JMeter usage issue, so I close this.


github: Remove issue template and use central one

Remove the GitHub issue template from this repository. We already have a
central set of templates [1] that are being used so the template in this
repository is redundant.
[1] - https://github.com/kata-containers/.github/tree/master/.github/ISSUE_TEMPLATE/
Fixes: #2952.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

/test-ubuntu

/test-ubuntu-qemu-metrics


Clone detection for python code snippets?

Hi,
Can we do clone detection on python codes also using clone task? Description in the README mentioned clone detection on Java Data specifically.
But after looking at the code in run_clone.py etc, looks like same clone task can be used for clone detection of python codes also, since the internal models being used are same for any task. Am I correct?

Hi, yes, you can finetune CodeT5 models on clone detection with Python code, as their pretraining dataset includes Python language.


Files are not stored in the path mentioned in pre-create hook

Describe the bug
Uploaded file is not stored in the path mentioned in pre-create hook, only info file is stored in the correct path.
To Reproduce
Steps to reproduce the behavior:

Start tusd with http hooks enabled and pre-create hook set up as per documentation (eg below)
Upload a file
Check the path where the uploaded file is stored on disk

Expected behavior
According to tusd documentation -

if the pre-create hook returns the following hook response, an upload with ID project-123/abc is created, the info file is saved at ./uploads/project-123/abc.info, and the file content is saved at ./uploads/project-123/abc/presentation.pdf

{
    "ChangeFileInfo": {
        "ID": "project-123/abc",
        "Storage": {
            "Path": "project-123/abc/presentation.pdf"
        }
    },
}

But setting the exact same values results in info file stored as project-123/abc.info and  uploaded file stored as project-123/abc. As per documentation shouldnt the uploaded file be stored as project-123/abc/presentation.pdf? Not sure if im missing some other setting needed to achieve this.
Setup details
Please provide following details, if applicable to your situation:

Operating System: Linux
Used tusd version: v2.4.0
Used tusd data storage: disk store
Used tusd configuration: tusd -hooks-http http://backend:3001/upload -hooks-http-forward-headers Authorization
Used tus client library: tus-js-client

This feature is only available in the 2.5.0 release candidates (v2.5.0-rc1 and v2.5.0-rc2). The documentation is misleading in that it's already in the latest regular release v.2.4.0, which it isn't. Apologies for that.

thanks for clarifying. works as expected on the release candidates.

https://github.com/tus/tusd/releases/tag/v2.5.0 is now released, so the documentation is accurate again. Thanks for reporting this!


[MS-348] Update Lionguard in Moonshot

Description
Update Lionguard in Moonshot
Motivation and Context
Update singapore-toxicity-classifier to Lionguard Classifier.
Used onnxruntime to read the models, included more classifier.
To score, metrics_json will select the classifier to grade
Type of Change

feat: A new feature

How to Test
Run a recipe that has lionguardclassifier as the metric
Checklist
Please check all the boxes that apply to this pull request using "x":

[x]  I have tested the changes locally and verified that they work as expected.
[x]  I have added or updated the necessary documentation (README, API docs, etc.).
[ ]  I have added appropriate unit tests or functional tests for the changes made.
[x]  I have followed the project's coding conventions and style guidelines.
[ ]  I have rebased my branch onto the latest commit of the main branch.
[ ]  I have squashed or reorganized my commits into logical units.
[ ]  I have added any necessary dependencies or packages to the project's build configuration.
[x]  I have performed a self-review of my own code.
[x]  I have read, understood and agree to the Developer Certificate of Origin below, which this project utilises.

Screenshots (if applicable)
[If the changes involve visual modifications, include screenshots or GIFs that demonstrate the changes.]
Additional Notes
[Add any additional information or context that might be relevant to reviewers.]

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
   have the right to submit it under the open source license
   indicated in the file; or

(b) The contribution is based upon previous work that, to the best
   of my knowledge, is covered under an appropriate open source
   license and I have the right under that license to submit that
   work with modifications, whether created in whole or in part
   by me, under the same open source license (unless I am
   permitted to submit under a different license), as indicated
   in the file; or

(c) The contribution was provided directly to me by some other
   person who certified (a), (b) or (c) and I have not modified
   it.

(d) I understand and agree that this project and the contribution
   are public and that a record of the contribution (including all
   personal information I submit with it, including my sign-off) is
   maintained indefinitely and may be redistributed consistent with
   this project or the open source license(s) involved.

Link to Jira Ticket: https://imda-dsl.atlassian.net/browse/MS-348?atlOrigin=eyJpIjoiMzYzNWU5Y2NhNzE1NDZjOGE3NWQyMTYzZGZlNGQzZWYiLCJwIjoiaiJ9

Tested. Working as expected.
runner-2.json


What's the intention of checkStyleDevDependencies

@lygstate it only runs linter-js-standard if the package.json of the project contains one of the linters as a dependency.
See https://github.com/ricardofbarros/linter-js-standard/blob/master/lib/init.js#L13-L18

Thank you @tonyghita for the explanation 😃


Uncaught TypeError: Assignment to constant variable (Chrome)

Can`t build for production mode with 10.17 version (dev version is well), got error:
Uncaught TypeError: Assignment to constant variable.
Using Laravel mix with Webpack (Laravel mix using  "webpack": "^3.11.0")

8.15 version is well in production and dev mode

Hi @somethingideally,
could you please copy and paste the error details so that I can know on which const it throws the error?
I’m on vacation now and I can’t check myself.
Thanks in advance

lazyload.es2015.js:238 Uncaught TypeError: Assignment to constant variable.
at B (lazyload.es2015.js:238)
at H (lazyload.es2015.js:307)
at q (lazyload.es2015.js:275)
at W._manageIntersection (lazyload.es2015.js:339)
at Array.forEach ()
at W._onIntersection (lazyload.es2015.js:351)
there are logs of error

Thank you.
I’m afraid those logs won’t be of much help though, since it seems your code has been minified.
Do you have the chance to execute unminified code and get the logs from there?

It looks like it’s webpack configuration fault.
To avoid any, could you check if the error occurs also in this very simple repo which leverages webpack to bundle LazyLoad?
https://github.com/verlok/lazyload-es2015-webpack-test
Let me know what happens.
Thanks

@Somethingideally could you answer my question? Thanks

I've just tried the es2015 version loaded as is in the browser, using this repo which has LazyLoad 8.15.2 as a dependency, and it doesn't throw any errors.
So I really need your help @Somethingideally and @wuxiuran to know what goes wrong in your cases.

@wuxiuran, @Somethingideally,
I've just check if the error occurs in the repo which leverages webpack to bundle LazyLoad.
I've tried:

LazyLoad version 8.15.2 build in development mode
LazyLoad version 8.15.2 build in production mode
LazyLoad version 10.18.0 build in development mode
LazyLoad version 10.18.0 build in production mode

And I wasn't able to replicate the error.
Maybe you can find the solution to your specific problem in that repo?

If not, I need your help to know what goes wrong in your cases.
If yes, please share the solution here to others which might have the same problem.

Thanks

I have the same error with the production build (v. 10.19.0):

If that helps, this issue is not present on 10.4.2. I've just discovered it after making an upgrade.

I'm still waiting for an answer to this question I've asked before.
I've just checked if the error occurs in the repo which leverages webpack to bundle LazyLoad.
I've tried the following:

LazyLoad version 8.15.2 build in development mode
LazyLoad version 8.15.2 build in production mode
LazyLoad version 10.18.0 build in development mode
LazyLoad version 10.18.0 build in production mode

But I wasn't able to reproduce the error.
Why do you think that WebPack build works?
Thanks
@adriandmitroca, @grund3g, @wuxiuran, @Somethingideally

Also could you post the version of WebPack and Babel you're using, and your babel and webpack configuration file? Thanks
@adriandmitroca, @grund3g, @wuxiuran, @Somethingideally

Same problem here:

<EMAIL_ADDRESS>"webpack": "3.10.0",
"babel-loader": "8.0.0-beta.0",
"@babel/core": "7.0.0-beta.34"

ugflifyjs config:
new UglifyJsPlugin({       parallel: 4,       sourceMap: true,       uglifyOptions: {         ie8: false,         safari10: true,         compress: {           drop_console: false         }       }     }),

The problem seems to reside inside UglifyJS in my opinion, given that with WebPack 4 everything works fine (see this repo).
Anyone is having the same problem on WebPack 4?

Adding those lines resolved my issue:
mix.options({
uglify: {
uglifyOptions: {
compress: { inline: false }
}
}
});


Remove componentjs mentions, update Readme format

Component.js is no longer maintained (see componentjs/component#639).

Thanks


unauthorized_client

I tried using this example with an existing AAD app I'd already created and I'm getting the unauthorized_client response.  I replaced the client id and secret, as well as the application id and the redirect url was modified correctly.  Is there something I'm missing here?

I see now that this sample is only good when the app is created using the app dev portal.


JSON example

Problem
It is a bit hard to handle JSON. (I tried with Raptor and almost broke my monitor.)
Solution
Please create an example JSON file. (You already added a Raptor asset for that.)

Hi, we are only trying to copy what is in the official spine runtime repo. If you need a specific example made please add your issue to the main project:
https://github.com/EsotericSoftware/spine-runtimes

I already find the solution so could you add it to your example files if i pass it to you?

I already find the solution so could you add it to your example files if i pass it to you?

Do you have the one? Can you share please ?

Do you have the one? Can you share please?

this is the working version
this handles the main part
 const attachmentLoader = new AtlasAttachmentLoader(Assets.get(atlas));
    const spineJsonParser = new SkeletonJson(attachmentLoader);

    const skeletonData = spineJsonParser.readSkeletonData(Assets.get(skeleton));

    this.spine = new Spine(skeletonData);
    ```

Do you have the one? Can you share please?

this is the working version
this handles the main part
 const attachmentLoader = new AtlasAttachmentLoader(Assets.get(atlas));
    const spineJsonParser = new SkeletonJson(attachmentLoader);

    const skeletonData = spineJsonParser.readSkeletonData(Assets.get(skeleton));

    this.spine = new Spine(skeletonData);
    ```

thanks!!


changes in image-to-string

There is some conflicting issue with 'freshlybuiltimagebol/OCR_Printed_Text.py'. Please update this file, so that I can merge the pull request.


[Slider] Dragging slider with onChangeCommitted prop changes other value

Continuing  #27800,

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Dragging slider with onChangeCommitted prop changes other value when mouseup happens near the other thumb.

Expected Behavior 🤔
It should only change the value of the thumb which is being dragged.

Steps to Reproduce 🕹

https://codesandbox.io/s/material-demo-forked-vxsmr?file=/demo.js

Context 🔦

Trying to drag and change first value near to second value but when doing mouseup second value moves up and It gets difficult to select the value.
If the 2nd value is 80 and I want to change 1st value 20 to 60, 2nd value moves up, Which is not required.

Does anybody know if you can make the Slider onChangeCommitted slide like the onChange slider?


When you have Atom running on Mac OS X Mavericks or Sierra and choose Restart, it will prevent the restart

I am using Atom 1.12.7
On Mavericks, it will block the Mac's restart by showing a dialog that says Cancel or Try Again.
On Sierra, it will show that same dialog (about cannot restart), but will restart anyway
(note that nothing was edited, meaning there is no "unsaved content" so restart should not be blocked).

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


fix(preset-classic): throw if preset finds GA options in theme config

Motivation
To prevent the legacy config from ever silently passing. Reported in https://github.com/facebook/docusaurus/pull/5832#issuecomment-1007122465.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Tested locally

Thanks for this @Josh-Cena!

I got bitten by this recently when upgrading my websites. My sites had much lower traffic for a few days before I realized why. Looking forward to the next version!

yes, we have a few features merged already, let's release this next week


Delay event connection 5 frames on client

This change is important to support events that are fired the frame the client joins the game. Without the delay, the client won't have enough time to connect to the events.

Closed after heavy discussion in favour of the preferred solution of implementing a queue.


Addressal of issue #2 by adding support for Colab

This is regarding issue #2 . After merging user will be able to run import opendatasets without getting the Attribution Error.
The network.py file's try except block was written to handle Exception whereas Colab was throwing AttributionError
Link to screenshot of Colab giving error on running import opendatasets

How would this fix the issue? AttributeError is inherited from Exception. The actual error is happening inside the except Exception block.

Thanks for your PR @VedantMadane . This issue has been fixed here: https://github.com/JovianML/opendatasets/commit/662056c04b7612f75ca46148c5fefdbea168aa9f


Add axes titles

Add axes titles to plots.
https://github.com/skane88/simplebeam/blob/4a2b473368b007de23645b035de23474f729c785/simplebeam/beam.py#L740

Fixed by #9


Full page cached pages don't appear to honor Redirect to Base URL

Enable full page caching on your site.
Enable a canonical URL on your site.
Make it so that other URLs can render your site. So going to http://otherurl.com should redirect to http://www.canonicalurl.com

It doesn't appear to – if the page is in full page cache it renders the cache before it redirects, ignoring the redirect to canonical url setting.

The only way I can think of fixing this without the db is using the canonical url as part of the cache-key. (also see: https://hackerone.com/reports/200444)

mm, that's an interesting idea.
On Mon, Jan 23, 2017 at 3:27 PM, Michael Krasnow<EMAIL_ADDRESS>wrote:

The only way I can think of fixing this without the db is using the
canonical url as part of the cache-key. (also see:
https://hackerone.com/reports/200444)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/concrete5/concrete5/issues/4987#issuecomment-274651577,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgNwWqKSzrx-yveP0719HTk4jYQRv9Sks5rVTd2gaJpZM4LqHF8
.

I can try implementing that (only if that checkbox is checked?), but I want to see what we are going to do with that H1 report as that can impact this.

I'd include the full request URL, (including GET parameters), so that pagination would work too

Including get parameters gets tricky, for example, you have /test?ccm_paging=3 and /test?ccm_paging=3&utm=... they produce the same exact content, but it would be a cache miss. Also, if the get parameters are re-ordered, it would be a cache miss. Also, with pagination:
User A goes to page 1 -> Page is cached
Articles are added
User B goes to page 1 -> Sees old page  User A saw (cached)
User B goes to page 2 -> Sees some duplicated articles from page 1

Including get parameters gets tricky, for example, you have /test?ccm_paging=3 and /test?ccm_paging=3&utm=... they produce the same exact content, but it would be a cache miss. Also, if the get parameters are re-ordered, it would be a cache miss.

What about adding a new method to the block controllers, asking them their potential GET parameters?
Once we've retrieve the list of GET parameters, we can sort them alphabetically, thus having a normalized URL.

Also, with pagination:
User A goes to page 1 -> Page is cached
Articles are added
User B goes to page 1 -> Sees old page User A saw (cached)
User B goes to page 2 -> Sees some duplicated articles from page 1 which is now cached

If someone sets this page to be in the cache, we can assume he knows what he is doing (for instance, he adds new articles very rarely, and he manually clears the cache when that happens).

@mlocati sounds interesting but a bit off topic for this particular issue.