ColorFiltered.matrix does not work properly on Mac and Android

Steps to Reproduce
Using a Color matrix in the example the image should be duotone green and red, instead it only produces a green image.
This is the flutter demo code with a matrix I created. Using the same matrix in the web example produces the proper green and red image. Web example: https://api.flutter.dev/flutter/widgets/ColorFiltered-class.html
Expected results:
On Mac, Android, and Web, I expected to see a duotone green and red image.
Actual results:
On Mac and Android, I see a green image. On Web I see the proper green and red image.

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

void main() => runApp(const MyApp());

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

  static const String _title = 'Flutter Code Sample';

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: _title,
      home: Scaffold(
        appBar: AppBar(title: const Text(_title)),
        body: const MyStatelessWidget(),
      ),
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    return SingleChildScrollView(
      child: Column(
        children: <Widget>[
          ColorFiltered(
            colorFilter: const ColorFilter.matrix([
              -1.0,
              0.0,
              0.0,
              0.0,
              1.0,
              1.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              1.0,
              0.0,
            ]),
            child:
                // For mac: update permissions following: https://stackoverflow.com/a/65866640/4433852
                Image.network(
                    'https://flutter.github.io/assets-for-api-docs/assets/widgets/owl.jpg'),
          ),
        ],
      ),
    );
  }
}

Logs

% flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.0.5, on macOS 12.5 21G72 darwin-arm (Rosetta), locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.2)
[✓] VS Code (version 1.69.2)
[✓] Connected device (3 available)
[✓] HTTP Host Availability

% flutter -version
Flutter 3.0.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision f1875d570e (2 weeks ago) • 2022-07-13 11:24:16 -0700
Engine • revision e85ea0e79c
Tools • Dart 2.17.6 • DevTools 2.12.2

Web Version
Mac Version

Reproducible using the code sample provided above.

macOS
web (canvaskit)
web (html)
ios
android

Interestingly, on the master channel, canvaskit renders the filter as seen below. This seems like a regression in the canvaskit behavior on master, but on other platforms, they maintain their behavior from stable on master.

Labeling for further investigation.

flutter doctor -v
[✓] Flutter (Channel stable, 3.0.5, on macOS 12.4 21F79 darwin-arm, locale en-GB)
    • Flutter version 3.0.5 at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (3 weeks ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • Java binary at: /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8609683/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • 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.12+0-b1504.28-7817840)

[✓] Android Studio (version 2021.2)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8609683/Android Studio.app/Contents
    • 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.12+0-b1504.28-7817840)

[✓] VS Code (version 1.69.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.44.0

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 12 (API 31) (emulator)
    • iPhone 13 Pro (mobile)      • 6E074BA5-191C-47D4-892F-FD96AFC34148 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-5 (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64   • macOS 12.4 21F79 darwin-arm
    • Chrome (web)                • chrome                               • web-javascript • Google Chrome 103.0.5060.134

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 3.1.0-0.0.pre.2009, on macOS 12.4 21F79 darwin-arm, locale en-GB)
    • Flutter version 3.1.0-0.0.pre.2009 on channel master at /Users/nexus/dev/sdks/flutters
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9d96c7b8be (4 hours ago), 2022-08-01 00:39:06 -0400
    • Engine revision cf0db3e546
    • Dart version 2.19.0 (build 2.19.0-49.0.dev)
    • DevTools version 2.16.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • Java binary at: /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8609683/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13F100
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • 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.12+0-b1504.28-7817840)

[✓] Android Studio (version 2021.2)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8609683/Android Studio.app/Contents
    • 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.12+0-b1504.28-7817840)

[✓] VS Code (version 1.69.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.44.0

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 12 (API 31) (emulator)
    • iPhone 13 Pro (mobile)      • 6E074BA5-191C-47D4-892F-FD96AFC34148 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-5 (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64   • macOS 12.4 21F79 darwin-arm
    • Chrome (web)                • chrome                               • web-javascript • Google Chrome 103.0.5060.134

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hi, @joelnagy , only green here is as expected. If you set m[4] of the sample code to 255 instead of 1, you can get the result you want.
c.f.
https://api.flutter.dev/flutter/dart-ui/ColorFilter/ColorFilter.matrix.html

The matrix is in row-major order and the translation column is specified in unnormalized, 0...255, space.

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

void main() => runApp(const MyApp());

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

  static const String _title = 'Flutter Code Sample';

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: _title,
      home: Scaffold(
        appBar: AppBar(title: const Text(_title)),
        body: const MyStatelessWidget(),
      ),
    );
  }
}

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

  @override
  Widget build(BuildContext context) {
    return SingleChildScrollView(
      child: Column(
        children: <Widget>[
          ColorFiltered(
            colorFilter: const ColorFilter.matrix([
              -1.0,
              0.0,
              0.0,
              0.0,
              255.0,
              1.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              0.0,
              1.0,
              0.0,
            ]),
            child:
                // For mac: update permissions following: https://stackoverflow.com/a/65866640/4433852
                Image.network(
                    'https://flutter.github.io/assets-for-api-docs/assets/widgets/owl.jpg'),
          ),
        ],
      ),
    );
  }
}

ColdPaleLight Are the Web versions not working as expected then, based on the fact that they present a different image colorization? Should this still not be a bug for consistency across platforms?
For my code, I based my matrix math on how this appears when used with an SVG feColorMatrix in HTML. It seems that this is what the Web version in doing here with Flutter as well.

ColdPaleLight Are the Web versions not working as expected then, based on the fact that they present a different image colorization? Should this still not be a bug for consistency across platforms?
For my code, I based my matrix math on how this appears when used with an SVG feColorMatrix in HTML. It seems that this is what the Web version in doing here with Flutter as well.

Yes，looks like the web (html) is not working as expected.


 Tried to allocate 130.00 MiB (GPU 0; 15.11 GiB total capacity; 11.09 GiB already allocated; 11.23 GiB reserved in total by PyTorch)

Hi @aprilhu01 ,
This is a normal issue caused by too heavy workloads on limited GPU memory resource. I tried to reproduce it with the same parameters and the request sample here, but no OOM happens on a single 16G A770 card.
Therefore, I suggest to:

Check the prompts in ShareGPT_V3_unfiltered_cleaned_split.json, where too long ones will overconsume memory footprints;

Tune max-num-batched-token and max-num-seq in the script, whose explanations can be found here. Lower values can help to reduce the memory consumption.

Thanks. Will tune script.


Add different language

It would be nice to track your mons in your native language (or if you want to trade but don‘t know which one the other player is asking fore).
Willing to provide for german

thanks for reaching out @Lutenus! closing this issue as it is a duplicate of #146


Implement brute-force finding best filter combination which maximizes correctness in evaluation

(Only for evaluation): Given the unifiers defined in config, what is the best combination of filters with the highest resulting correctness? Go through the search space of combinations and find the best combination. To reduce the search time, define a max amount of filters and skip combinations where a filter does not modify the data (except for DropDuplicates, it is always useful). Store the n best combinations as a Dataframe.
Notes: Choose a good function which has to be maximized. Possible objectives are: Correctness, a * Correctness + (1 - a) * Completeness, a * Correctness + (1 - a) * Completeness - b * amount_of_filters_in_combination.

Because evaluation is done after typegen, implementing this would disrupt the current workflow. Evaluation will only evaluate with the used filter list from typegen.


rtl is not supported!

Just added reverse support in 5.1.5 https://github.com/swiftcarrot/react-input-slider#props


Improvements

Explanations:

Rename __output_file__() to _output_file() in favor of PEP 8.
Delete LICENSE.txt because it is a duplicate of LICENSE


Initial exploration around JIT

An initial standalone test for using JIT compilation of the EVM bytecode.
In this test, I've implemented few opcodes that I felt characteristic of the needs of the real EVM bytecodes, hopefully not missing any of the real difficulties with implementation of

PC opcode where it required to read from the return return fp - 3 to get the pointer to the beginning of the segment
JUMP like opcode where one skips the next n opcode
PUSH like opcode where one read value from the next opcodes and jump to the next opcode
RET like opcode where the execution halts
CALL like opcode where the opcode does jit internally to trigger a subcall and returns its value

Overall, I feel that with very few changes to the opcodes as they are, and a lot of simplification for the syscall and at the interpreter level, we can migrate to jit, which will also save steps (to be estimated) but at least all the input/output argument copying + control flow management

blocked because this needs a fix in the cairo vm


Grant owner access to hammerlab on pypi

So that we can implement uniform deploying strategies in accordance with the rest of the pygenomics tools.

We should use the office account

Closing this one as it looks like our office account already had the owner status for this project. My bad not catching this before.


ProjectTypeGuids unification

Split off from #165
ProjectTypeGuids unification (all to upper-case) can be scheduled for 3.1.

@moh-hassan raised good point and I think it is better to preserve case as is.


docs(best-practices): Update testing section

This updates our docs with a note on the differences between enzyme and @testing-library/react, and adds a note about testing react hooks.

@artsyjian - I've moved the segment destination stuff into its own file as that's a little less concerned with day-to-day product dev in force.

@artsyjian - I've moved the segment destination stuff into its own file as that's a little less concerned with day-to-day product dev in force.

sounds good. thanks for the heads up.


build: bump critools to 1.28

What problem does this PR solve?:
bump critools to k8s 1.28
Which issue(s) does this PR fix?:

https://jira.d2iq.com/browse/D2IQ-99586
Special notes for your reviewer:

Does this PR introduce a user-facing change?:

running vsphere e2e tests.

#996 is a superset of this.

closing in favor of https://github.com/mesosphere/konvoy-image-builder/pull/996


update AEM build for dev env: update-pipeline

Expected Behaviour
$ aio cloudmanager:update-pipeline --programId=1234 123456 --updateAemBuild
Since Dev environments don't autoupdate, is it possible to get an option for update-pipeline to update to the latest AEM build? It seems that update-pipeline command can only update the git branch attached to the pipeline. This is a nice feature in the UI that would be helpful in this tool.

This functionality is not exposed through the public API, so can't be added to the CLI.

FTR: https://github.com/AdobeDocs/cloudmanager-api-docs/issues/29


Propolsal for MAINTAINERS.md and GOVERNANCE.md

As I'm still not official member this is just a propolsal to stream line the repository acording to CNCF standarts
Needed updates per file:
MAINTAINERS.md

add real names for each maintainer
add company affiliation

GOVERNANCE.md

maintainers team in github needs update - @external-secrets/maintainers
ESO dev mailing list needs update - ESO dev mailing

Propolsal template is optional but at some point is good to have a formal process for new features!
Orlix
Signed-off-by: OrlinVasilev<EMAIL_ADDRESS>

@OrlinVasilev we have our bot configuration file that also holds this kind of information.
https://github.com/external-secrets/external-secrets/blob/main/.github/PAUL.yaml
I'm a bit worried about the redundancy and having to update both files.
@gusfcarvalho @moolen what do you think?

@knelasevero , not saying you don't have it I took it from there :D
But the project now is under CNCF so is good to follow the CNCF guides and practices e.g.
https://clomonitor.io/projects/cncf/external-secrets
just search for maintainers and you will get the idea
other resource is https://github.com/cncf/project-template
Hope that helps!

or maybe we can do maintainers.md to point to the paul.yaml as reference and single point of truth!

I see! Yeah, if we can have a way to avoid having to update both files, that would be great! Or maybe have a Makefile command that keeps files in sync and maybe CI checking that.
But I don't want to block the PR now with this nit :)
LGTM, I will let @moolen and @gusfcarvalho have a look as well

@moolen thank you!

LGTM :)

Thanks @gusfcarvalho @knelasevero @moolen :)

gonna merge this then with a supermajority vote :tada: :wink:

Thank you folks super happy to contribute! Can I have "hacktoberfest-accepted" label as well so we can show that the repository was full on hacktoberfest!


[CI] Use latest chart-testing image

Updates CI to use the chart-testing:v1.0.3.
I already pushed the new image (gcr.io/kubernetes-charts-ci/test-image:v2.0.3).

/assign @prydonius

cc @helm/charts-maintainers

/hold

I just figured I used jq instead of yq in my change. The Docker image only has yq. I should have tested it with the Docker image too. Will fix it probably tomorrow.

Doh, of course, jq is also there since yq is only a wrapper around it. Anyways, I'll do some tests with the Docker image.

/lgtm


BibEdit: fix page footer display

Originally on 2012-06-13
BibEdit does not show page footer containing language selection etc at all.  This should be fixed.

No need to fix this as it was a request by catalogers and there is no request for it to be changed.


added format constants that follow HTML5 input types

This is my first contribution to an open source project.
I've added the requested constants from #3928

@nimshory  Why did you close this PR?


Responsive Column Control - Specify styling on generic class instead of specific column classes

Look at: https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/7956e740008d09d753351aec6f238993ef55f25e/content/src/main/content/jcr_root/apps/acs-commons/components/content/column-control/clientlib/css/colctrl.less
There was an issue to add a generic class to the HTML output: https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/154
I believe we can now transfer all the styling (position, border-box, etc etc) to that generic class, instead of putting it on every column.
We believe only the width should be on the styles per specific column.
Right now, it's giving our designer a headache. :1234:

Ah yes - agree. It looks anything non-dynamic could be bubbled up to a generic class.
@hkuiyo any way you can make a PR for this (and any other responsive col tickets)

@hkuiyo it doesnt appear you ever PR'd this back in. Ill close this since Responsive Columns are quasi-irrelevant now in light of AEM Responsive Grid. If you have the fix feel free to PR it though.


Add migration to fix "processing_failed"-image files

REDMINE-16706

In the other migration we copied all attachment attributes. Is there are reason why we only copy the file name here?

Coverage remained the same at 99.295% when pulling 110ad3a21306fe1254c4e49ce3e23a324e21cf82 on DanielKuhn:fix/processing-failed-image-files into 12ec0d6468bfcda8a9a9116f7e3f300c87890c46 on codevise:master.

@tf no reason, good catch

I updated the migration so that it also fixes the records which were manipulated via production console. Ready to merge now


Creating a Mat or UMat from Vec

I have no idea where else to ask this question. But I figure it could be somewhat related to the api wrapper.
I have a byte buffer created by gstreamer pipeline which is 'width * height * 3' bytes in size. In my appsink I get a slice &[u8] and I can copy it to a vec if needed.
I cannot easily see how to create a UMat or Mat from this. There isn't a variant of 'from_slice' that accepts the correct image dimensions for example. And the implementation of from_slice/from_slice_2d seem to create a Mat with totally undesirable setup for treating the buffer as an image.
Certainly part of my problem is just ignorance. :D
Any suggestions, tips, or clarifications would really appreciated!

So you want to create a 2D Mat 3 channels deep from a 1D slice? So your target would be Mat with type Vec3b::typ() (CV_8UC3 in OpenCV terms) with the rows() == height and cols() == height. It looks like a something like this should work:
// quick and dirty untested code
let m = Mat::from_slice(source_slice).unwrap(); // will give type=CV_8UC1, rows=1, cols=width*height*3
let m = m.reshape(3, height).unwrap(); // should give you type=CV_8UC3, rows=height, cols=width

Ah, thank you for clarifying this for me. I understand more clearly why I was segfaulting when creating a mat from a pointer as well.


Synchronized blocks around the code causes thread blocks

We are using config-server + git as a storage for properties.
Time to time we are facing the case when git provider becomes turtle - config service stucks on giving properties to clients.
Some info regarding this:

can't say about particular count of nodes using config-service, but it's around 100+ and we expect more ...
here is the threaddump from config-server

one of the 201 threads stuck on fetch:

 Thread 30550: (state = IN_NATIVE)
  - java.net.SocketInputStream.socketRead0(java.io.FileDescriptor, byte[], int, int, int) @bci=0 (Compiled frame; information may be imprecise)
  .........
  - org.springframework.cloud.config.server.environment.JGitEnvironmentRepository.fetch(org.eclipse.jgit.api.Git, java.lang.String) @bci=35, line=398 (Compiled frame)
  - org.springframework.cloud.config.server.environment.JGitEnvironmentRepository.refresh(java.lang.String) @bci=18, line=217 (Compiled frame)
  - org.springframework.cloud.config.server.environment.JGitEnvironmentRepository.getLocations(java.lang.String, java.lang.String, java.lang.String) @bci=11, line=194 (Compiled frame)
  - org.springframework.cloud.config.server.environment.MultipleJGitEnvironmentRepository.getLocations(java.lang.String, java.lang.String, java.lang.String) @bci=220, line=146 (Compiled frame)
  - org.springframework.cloud.config.server.environment.AbstractScmEnvironmentRepository.findOne(java.lang.String, java.lang.String, java.lang.String) @bci=17, line=42 (Compiled frame)

slowest responses:

The problem: if count of request (includes healthchecks and configs retrievings itself) will be huge and response time from git repo will be not that slow but bigger than expected we will have all thread stuck and config service will be unavailable due to absence of free threads on tomcat, it can cause all nodes DOWN on discovery server, so whole system would be unavailable.
Possible solutions:

I was thinking to add some fail fast strategy and look at local repo instead on getting stuck for several minutes in case if response time will be huge, and made little refactoring as a first step - https://github.com/spring-cloud/spring-cloud-config/pull/952 but it solves not much, since we can't just fail in the middle of git flow, so we can break git in such case and such cases would be hard to cover by unit-tests.
also I have  doubts regarding healthcheck flow, since it's harm flow of retrieving properties (which is main responsibility), but does not so much (if service started - properties should be read once and it won't get to config service another time except being refreshed)
have concurrent git repos for each JGitRepository and each repo will have its own lock. It also solves not that much, but throughput could be increased in X times, although it requires refactoring not only JGitRepository, but higher classes, since synchronized blocks are all around the code. (once it could lead to deadlock, since its very hard to track all synchronized blocks)

So the question to members: what is the best solution to solve this in your opinion?

I would think this problem could be mitigated by scaling out the config server with a couple of instances.  Are your above findings based on a single instance?

Clustering of config service may help, but is it not excessive?
Each node would be just stuck and resources won't be used at all. We would have huge overhead on java/tomcat but useful processing time would be like <10%

I dont understand why it would be excessive.  This is one of the typical solutions to handle a lot of load.  With the additional instances the app should not experience the problems you described.

See also #6 (suggested alternative implementation)

We ran into the similar issue and as by Ryan, were able to scale out to get around the limitation.
Wanted to share what we noticed is that this is not simply a latency issue. The affected instance of the cloud config server doesn't recover once they get overwhelmed with requests unless restarted.
Below are 2 thread dumps showing all thread waiting on a monitor held by a thread doing a native file-system i/o which never returns.
Threads waiting for the lock

Thead hold the monitor

Environment:

3 instance of Spring Cloud Config running on PCF (each AI with 3Gb memory & 1Gb disk)
64 µservice scaled at 4 to 5 instances each
All config purely yaml files and about 5-10 k in size

We ran into the similar issue and as by Ryan, were able to scale out to get around the limitation.
Wanted to share what we noticed is that this is not simply a latency issue. The affected instance of the cloud config server doesn't recover once they get overwhelmed with requests unless restarted.
Below are 2 thread dumps showing all thread waiting on a monitor held by a thread doing a native file-system i/o which never returns.
Threads waiting for the lock

Thead hold the monitor

Environment:

3 instance of Spring Cloud Config running on PCF (each AI with 3Gb memory & 1Gb disk)
64 µservice scaled at 4 to 5 instances each
All config purely yaml files and about 5-10 k in size

We ran into the similar issue and as by Ryan, were able to scale out to get around the limitation.
Wanted to share what we noticed is that this is not simply a latency issue. The affected instance of the cloud config server doesn't recover once they get overwhelmed with requests unless restarted.
Below are 2 thread dumps showing all thread waiting on a monitor held by a thread doing a native file-system i/o which never returns.
Threads waiting for the lock

Thead hold the monitor

Environment:

3 instance of Spring Cloud Config running on PCF (each AI with 3Gb memory & 1Gb disk)
64 µservice scaled at 4 to 5 instances each
All config purely yaml files and about 5-10 k in size

And you have 250+ services hitting config server at the same time?

Actually we have same issue, config service just stuck and cannot process any requests w\o restart.
And yes, there can be a case that even more service at the same time trying to get properties especially if clients healthcheck is enabled.
In our case problem occurs during network spike between git and config server and all requests just standing in a line waiting the first one to fail on connection timeout, but event in this case retry return it to the line.

@tdanylchuk do you have a way to re-create the connection timeout issue? I'm not sure what you mean by your last sentence ("retry return it to the line"). Can you explain a bit? I guess maybe you just mean that retry is never successful because of some pathology in the network connection to git? Removing locks and making the retry faster wouldn't necessarily help, right?

@pankesh I don't know why you think it's anything other than latency at the end of the day. Your screenshots only demonstrate what we know: that only one request can be serviced at a time. Your assertion that the filesystem call "never returns" is hard to justify.

@dsyer Agreed about the one request at a time in the current design.
Regarding the screenshot - those thread analysis consists of 3 dumps collected 30 s apart and the particular thread that is holding the monitor never returned. Our latency back from our enterprise github is about 4s-6s so there should have been no reason for that to be held that long.
We know how to reproduce this in our environment so if there is anything additional that you'd like me to collect/capture, i can provide it here. Tx!

30s isn't all that long if there are network timeouts. You don't need hundreds of clients to see this behaviour - only 2 - if the git server is unresponsive and the network is in a pathological state. Your 4s "normal" latency is really sad, and it will inevitably lead to issues if many clients at once all need to do something with that level of delay.


Support for a more compressed scene file format

Handle scenes and templates in a compressed format
I confirm I have signed the Contributor License Agreement.

Looks good, simpler than I expected.

Hi @zachpeterpaul, can you give a slightly expanded description to this PR please? It'll help the open source community understand what this PR is and the benefits it will bring (include some numbers too ideally).


Queue agent init script creates tables in the default bundle

This initialization script should be parametrized with a tablet_cell_bundle to create tables in, which should default to sys.

Added in https://github.com/ytsaurus/ytsaurus/pull/167.
Should we consider updating tables we've already created with the correct tablet cell bundle in this script?


Clip Loaders dont work for sd3+ models anymore.

 I tried moving the gguf files to the text_encoders directory like mentioned on some issues here and my files get picked up whether they are in the clip or the text_encoders folder. The clip loader does work without problems on the flux workflows too but all the sd3+ just dont. Whether i try to run them with the dual or the trippple clip loader. Exchanging the gguf clip loader for the one in the comfy core makes the workflows work again. No errors on the console are produced.

Pushed a possible fix, could you verify if that fixes it?


Create PRC process page

[ ] Create page
[ ] Add description of what is being covered (general process including what happens to preprints when they are imported to eLife, submitting from eJP to preprint servers, checking RPs ahead of publication, updates to RPs, update to VoR for RPs, controlled vocab, DOIs)
[ ] blocked until elifebot is set up
[ ] Fred approved page
[ ] James approved page
[ ] Naushin approved page
[ ] Rebecca approved page
[ ] Share page with Exeter and invite feedback

Definition of done

[ ] Page revised after team comments
[ ] Team reviewed and approved page?

Blocked as process is still in flux


chore: add event bus implementation

closes: [CDP] iOS Native Refactor: Creation of journeys module#11651
Overview
This PR introduces the EventBus framework, a flexible and scalable solution for handling event-driven communication across various modules of our SDK. The EventBus framework is designed to manage and dispatch events, providing a decoupled and efficient way of handling inter-module communication.
Changes

EventBus Framework: A central mechanism for event handling, enabling modules to send and receive events without direct dependencies.
EventRepresentable Protocol: Defines the structure for events that can be transmitted through the EventBus.
Test Suite for EventBus: Comprehensive tests covering a variety of scenarios, ensuring the EventBus behaves as expected under different conditions.

Key Features

Decoupling of Modules: Reduces direct dependencies between modules, leading to cleaner and more maintainable code.
Event Management: Streamlines the process of event creation, dispatch, and handling.

Impact
The introduction of the EventBus framework is a significant step towards a more modular and scalable architecture. It simplifies inter-module communication and opens the door for more dynamic and flexible event-driven programming within our application. Removes the need for hooks and cater for use-cases like independent of SDK initializations

Creating new broken down PRs with updated implementations


[Feature] Foveation support

https://github.com/alvr-org/alvr-visionos

I think, this problem can be with hardcoded tangents
let tangents: simd_float4 = simd_float4(1, 1, 1, 1)


half=True - causing crashes?

Sorry to be  a bother, but I've found that if I have load(r"C:\VapourSynth\vsgan_models\BSRGAN.pth", half=True) it crashes. I tested it with load(r"C:\VapourSynth\vsgan_models\BSRGAN.pth", half=False) and no crash. Tested with a few other ESRGAN models and same deal with "True" crashing and "False" not crashing.

Fixed in v1.6.3, yet another blunder from 1.6.0. This time it was because the rejoined tiling system would create the new merged tile under the default device (99% of setups would have this to be the CPU).


Added all the files

Pull Request for ML-Crate 💡
Issue Title: Parameters of Cricket Analysis #723

Info about the related issue (Aim of the project) :  To perform Parameters of Cricket Analysis of "Indian Premier League"
Name:  Somnath Shaw
Email ID for further communication<EMAIL_ADDRESS>GitHub ID:  https://github.com/SOMNATH0904
Idenitfy yourself:  Student

Closes: #723
Describe the add-ons or changes you've made 📃
It is a comprehensive guide for conducting EDA on sports-related datasets. It aims to equip users with the skills necessary to uncover insights and patterns from raw data, leveraging various statistical and graphical techniques. This project covers a range of essential EDA steps, including data cleaning, data visualization, and summary statistics. Users will learn how to handle missing values, identify outliers, and understand the distribution and relationships within the data.
Type of change ☑
New Project added.
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙
The files are updated.
Describe how it has been tested
Describe how have you verified the changes made
Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

@abhisheks008 Please Check this !!

Hi @SOMNATH0904 please follow the project structure and update the project files accordingly.

@abhisheks008 I have updated the files according to the project structure. Can you please take a review ? ...

@abhisheks008 Please check now!!


Native MPI Interoperability with PAMI machine layer

Original author: Nikhil Jain
Original issue: https://charm.cs.illinois.edu/redmine/issues/17

Suport interoperation of Charm and MPI on PAMI.

Original date: 2013-05-30 21:00:35

The  bits described "here":https://wiki.alcf.anl.gov/parts/index.php/PAMI#Using_multiple_clients may be helpful, especially usage of the PAMI_CLIENTS environment variable.

Original author: Nikhil Jain
Original date: 2013-05-31 06:31:36

Yeah, I am an aware of these methods and PAMI_CLIENTS. We have some sort of interop working using it in one of the branches.

Original author: Nikhil Jain
Original date: 2013-10-04 03:31:41

Suport for time division based interoperation has been added. Other features will be dealt in a separate issue.

Original date: 2014-08-13 03:39:26

Retitled for ease of comprehension in release-notes/roadmap usage of Redmine.


Code coverage and multiple python3 version matrix

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@25efb05). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #136   +/-   ##
=========================================
  Coverage          ?   53.02%
=========================================
  Files             ?      341
  Lines             ?    25739
  Branches          ?     3334
=========================================
  Hits              ?    13649
  Misses            ?    11402
  Partials          ?      688

Continue to review full report at Codecov.

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


[Snyk] Security upgrade golang from 1.20.2-alpine3.17 to 1.20.5-alpine3.17

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

Dockerfile.client

We recommend upgrading to golang:1.20.5-alpine3.17, as this image has only 0 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

300
Out-of-bounds Read SNYK-ALPINE317-OPENSSL-5438697
No Known Exploit

300
Out-of-bounds Read SNYK-ALPINE317-OPENSSL-5438697
No Known Exploit

400
Allocation of Resources Without Limits or Throttling SNYK-ALPINE317-OPENSSL-5661570
No Known Exploit

400
Allocation of Resources Without Limits or Throttling SNYK-ALPINE317-OPENSSL-5661570
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Allocation of Resources Without Limits or Throttling

@dependabot rebase


Fix log when mount in foreground failed

Codecov Report

Merging #1052 (aaff277) into main (245f149) will increase coverage by 0.04%.
The diff coverage is 0.00%.

:exclamation: Current head aaff277 differs from pull request most recent head f083828. Consider uploading reports for the commit f083828 to get more accurate results

@@             Coverage Diff              @@
##               main    #1052      +/-   ##
============================================
+ Coverage     46.11%   46.15%   +0.04%
  Complexity      149      149
============================================
  Files           146      146
  Lines         23782    23782
  Branches        422      422
============================================
+ Hits          10967    10977      +10
+ Misses        11468    11459       -9
+ Partials       1347     1346       -1

Impacted Files
Coverage Δ

cmd/mount_unix.go
0.00% <0.00%> (ø)

pkg/vfs/writer.go
78.64% <0.00%> (-1.02%)
:arrow_down:

pkg/fuse/fuse.go
29.77% <0.00%> (-0.65%)
:arrow_down:

pkg/meta/sql.go
69.89% <0.00%> (+0.09%)
:arrow_up:

pkg/meta/tkv.go
77.25% <0.00%> (+0.65%)
:arrow_up:

Continue to review full report at Codecov.

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

This helps me a lot.


Re-draw

HI,
How can i re-draw the scrachable layer?

Recreating scratch layer is not available in the current version. Thanks for the suggestion. Will include in the next version.

When will this be added? This is most need feature

hi guys, I had to re-draw the scrachable layer.
I override all class ScratchTextView and added this method:
public void repaint() {

       if (this.mDrawable != null && this.mGradientBgPaint != null && mScratchBitmap != null) {
           this.mScratchBitmap = Bitmap.createBitmap(this.getWidth(), this.getHeight(), Config.ARGB_8888);
           this.mCanvas = new Canvas(this.mScratchBitmap);
           Rect rect = new Rect(0, 0, this.mScratchBitmap.getWidth(), this.mScratchBitmap.getHeight());
           this.mDrawable.setBounds(rect);
           int startGradientColor = ContextCompat.getColor(this.getContext(), color.scratch_start_gradient);
           int endGradientColor = ContextCompat.getColor(this.getContext(), color.scratch_end_gradient);
           this.mGradientBgPaint.setShader(new LinearGradient(0.0F, 0.0F, 0.0F, (float) this.getHeight(), startGradientColor, endGradientColor, TileMode.MIRROR));
           this.mCanvas.drawRect(rect, this.mGradientBgPaint);
           this.mDrawable.draw(this.mCanvas);

           this.invalidate();
       }
   }

that's fine for me.
Please add this code and update de library
Thanks!!!

Ability to redraw, reveal fully and change image will be added soon. Apologies for the delay

Ability to redraw, reveal fully and change image will be added soon. Apologies for the delay

can you tell me when this update is available. Thanks for this nice library

is the update available??


Warning: path /(.*) cannot be used with pathType Prefix

Describe the bug
Fix GitHub Action Warning: Warning: path /(.*) cannot be used with pathType Prefix
To Reproduce
Example: https://github.com/microsoft/contosotraders-cloudtesting/actions/runs/5147171216/jobs/9267224090#step:52:164
Screenshots

Solution:
https://github.com/kubernetes/ingress-nginx/issues/10200#issuecomment-1673141202

Switching from PathPrefix to ImplementationSpecific for any path failing the validation makes the warnings go away for me.


[CI][Bugfix] Skip chameleon for transformers 4.46.1

Chameleon test only specifically checked for 4.46.0, however the newly released 4.46.1 did not address this model
Failing CI https://buildkite.com/vllm/ci-aws/builds/10430#0192d917-3982-44f7-8c97-0a3e2472c7b1/159-4366

Should this be changed in the chameleon model test as well (here)?


[FEATURE] Support HTTPStatus

Is your feature request related to a problem
We typically use HTTPStatus in our code. When used as a value for status_code in path functions this results in the string literal in the documentation and when "trying" the response is shown as undocumented because it doesn't match the actual response code
The solution you would like
I want to use HTTPStatus enum values and have it work exactly like the starlette.status pseudo enum values

For now you can create your own HTTPStatus class (you can't subclass Enums) and add a __str__ method to convert it to the string representation of the actual status code, for example:
class MyHTTPStatus:
    OK = 200
    ...
    def __str__(self):
        return str(int(self))

or use int(HTTPStatus) in the endpoint decorator.

Thanks for the help @retnikt ! :bow:
Yeah, @hmvp you can probably use that for your use case.

I am sorry to disagree...
It is indeed quite easy to workaround this. For example by just importing status from starlette and using that in the decorators.
However HTTPStatus is part of the standard lib and since we use it extensively in the rest of our code I would like to use that instead of the starlette status. The second reason is that I expected it to work (since its part of stdlib) and it somewhat did but gave unexpected results.. This is a papercut/pitfall/surpising behavior of fastapi and given the high standard of the rest of the library this should not be there
I am not sure if @retnikt  understood what I am trying to do here.. Adding another class is a weird suggestion given that both HTTPStatus and starlette.status already exist

The ASGI spec uses ints for status codes, not enums. That's why it takes ints.
Also, @retnikt has been helping a lot here, answering a lot of questions, trying to help others like you, for free. Just out of being a cool person. Please try to be more respectful to the community that is here trying to help.
If you really want to use HTTPStatus enums you can use the int value, it's quite simple, e.g. HTTPStatus.OK.value.

I did not want to be disrespectful to @retnikt and I value his contribution for other people that might have a similar but different issue. I just don't see how it is relevant for my issue. I am sorry if my wording to strong... As a non-native English speaker I might miss some subtleties.
With regards to the ASGI spec, I did not know that, but I also was not aware that the status_code code argument followed the ASGI spec in that regard, especially since putting in a HTTPStatus code enum value just works, but will give a weird result in the docs. It should not be to difficult to add some code along the line of:
if isinstance(HTTPStatus, status_code):
     status_code = status_code.value

or  just status_code = int(status_code) to the path functions, which would solve a papercut and would still be valid ASGI.  Otherwise I would expect the path functions to be noisy about wrong input..
On the other hand, this is indeed not a big issue and if you don't want to change anything, thats fine with me. I just wanted to signal that this is a papercut in fastapi

Cool, thanks!
Yeah, I would accept a PR checking if a status code is an enum to get its value first. :nerd_face: :heavy_check_mark:

Working on it!

This was added on https://github.com/tiangolo/fastapi/pull/1534 by @retnikt :rocket: :tada:
It will be available in FastAPI version 0.57.0 (released in a couple of hours).

Works as expected :+1:


Ocasional 404 while calling /token or other endpoints

Would you be so kind to investigate the reason, why sometimes on working urls 404 is returned. I know the urls are valid as getting the exception I try it in browser -- and result is returned successfully. One example is getting client_credentials token returns 404.

Are you sure these are 404 and not 400? Any other affected endpoints?

I'm pretty sure, as I get exactly 404. Unfortunately, I haven't logged urls until recently, but I did get quite a few 404 exceptions even though resources are available.

We haven't had any similar reports, would need detailed request/response logs to help debug. I usually use mitmproxy (https://mitmproxy.org) to debug stuff like this or wireshark etc.

I'll try to reproduce, but this url https://www.deviantart.com/oauth2/token?grant_type=client_credentials&client_id=ID&client_secret=SECRET causes 404 definitely.

I did see occasional 4xx issue to various endpoints. Reloading the same endpoint seemd to work OK. It is a tricky issue to reproduce in order to fix it.

That url is working for me. I actually have tests that run ever 15 minutes all using that endpoint for their tokens, I would expect those to fail also if there was an issue.
Could you be hitting the clients rate limit maybe and getting a 429 or 403 error?

I'm sure it's not that kind of errors. As far as I see, it may be the framework problem to show 404 when the internet connection is not available. But I've also seen the behavior while having connection but with other urls, like curated tags. As for now I'll close the issue then. Thanks for the help.


Provide a neat way to get date strings from arrays with date units.

✨ Feature Request
Following [this comment] https://github.com/SciTools/iris/pull/4499#issuecomment-1012947187  (and preceding ones).
If you know what you're doing this isn't actually that hard.
E.G. coord.units.num2date(coord.bounds).astype('str')
But it isn't so easy to work that out.
How about Coord.date_strings() ?
We can make it cheekily default to the Coord's own points, like Connectivity.indices_by_src

I wanted this again today.  I think the implementation is easy, and the hard bit is deciding what the interface should be.  So marking this for discussion.

One option which seemed favourable in peloton discussions: adding a __format__ method to Coord, such that format(tcoord, "%Y-%m-%d") could be used to create a string with all points formatted with the given format string. This is also used by f-strings so f"{tcoord:%Y-%m-%d}".
Actual implementation might look something like
def __format__(self, fmt):
    return str([format(point, fmt) for point in self.units.num2date(self.points)])

Just needs a bit more thought into points vs bounds (always both? extra symbol in the fmt string to choose one or the other?)

As one immediately upvoting this feature I am really looking forward to this, and I would very much favour the possibility to choose only points, only bounds, or both. Many thanks /L

I wanted this again today so I rolled my own
def print_coord_times(coord, points=True, bounds=True):
    """
    Given a time type coordinate (e.g. 'time' or 'forecast_reference_time'),
    print string representations of the points/bounds.

    Set *points* or *bounds* to False to skip printing those.
    """
    if not coord.units.is_time_reference():
        raise ValueError(f"{coord.name()} is not a time type coordinate")

    if points:
        print("Points:")
        print(coord.units.num2date(coord.points).astype('str'))

    if bounds:
        print("Bounds:")
        print(coord.units.num2date(coord.bounds).astype('str'))


JShell is not working with Java 15 (StringIndexOutOfBoundsException)

Summary
A brief summary of the problem.
Since Java 15.0.0 jshell seems to broken. I tried every Java 15 release. JDK 15.0.0, 15.0.1 and 15.0.2 with HotSpot as well as with J9.
When using one of these releases and trying to call the "jshell" command, I get an excrption (see actual result section.
Steps to reproduce
I downloaded sdkman to switch between java versions easily.

sdk use java  15.0.0.hs-adpt
call "jshell" in the console
get the stacktrace above

Expected results
I expect jshell to behave as Java 14.0.2 does. So that I can just use jshell without exceptions.
When using java 14 by calling: sdk use java 14.0.2.hs-adpt, the jshell command behaves as expected and I can use it without getting exceptions.
Actual results
Exception in thread "main" java.io.IOError: java.lang.StringIndexOutOfBoundsException
at jdk.internal.le/jdk.internal.org.jline.utils.Curses.tputs(Curses.java:62)
at jdk.internal.le/jdk.internal.org.jline.utils.Curses.tputs(Curses.java:45)
at jdk.internal.le/jdk.internal.org.jline.keymap.KeyMap.key(KeyMap.java:243)
at jdk.internal.le/jdk.internal.org.jline.reader.impl.LineReaderImpl.key(LineReaderImpl.java:6095)
at jdk.internal.le/jdk.internal.org.jline.reader.impl.LineReaderImpl$$Lambda$369/0x0000000064833110.apply(Unknown Source)
at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:195)
at java.base/java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:177)
at java.base/java.util.Spliterators$ArraySpliterator.forEachRemaining(Spliterators.java:948)
at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:497)
at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:487)
at java.base/java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:150)
at java.base/java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:173)
at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:239)
at java.base/java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:497)
at jdk.internal.le/jdk.internal.org.jline.reader.impl.LineReaderImpl.bindKeys(LineReaderImpl.java:6103)
at jdk.internal.le/jdk.internal.org.jline.reader.impl.LineReaderImpl.emacs(LineReaderImpl.java:5813)
at jdk.internal.le/jdk.internal.org.jline.reader.impl.LineReaderImpl.defaultKeyMaps(LineReaderImpl.java:5789)
at jdk.internal.le/jdk.internal.org.jline.reader.impl.LineReaderImpl.(LineReaderImpl.java:293)
at jdk.jshell/jdk.internal.jshell.tool.ConsoleIOContext$2.(ConsoleIOContext.java:133)
at jdk.jshell/jdk.internal.jshell.tool.ConsoleIOContext.(ConsoleIOContext.java:133)
at jdk.jshell/jdk.internal.jshell.tool.JShellTool.start(JShellTool.java:978)
at jdk.jshell/jdk.internal.jshell.tool.JShellToolBuilder.start(JShellToolBuilder.java:254)
at jdk.jshell/jdk.internal.jshell.tool.JShellToolProvider.main(JShellToolProvider.java:120)
Caused by: java.lang.StringIndexOutOfBoundsException
at java.base/java.lang.String.charAt(String.java:1334)
at jdk.internal.le/jdk.internal.org.jline.utils.Curses.doTputs(Curses.java:359)
at jdk.internal.le/jdk.internal.org.jline.utils.Curses.tputs(Curses.java:60)
... 22 more
Triaging info
Java version:
Every Java 15 version
What is your operating system and platform?
My OS is Manjaro Linux x64.
How did you install Java?
via sdkman.https://sdkman.io/
Did it work before?
It worked with Java 14 and below.
Did you test with other Java versions?
Yes.

Is it https://bugs.openjdk.java.net/browse/JDK-8261450?

@aahlenst I think it's a similar issue but not the same issue. I don't even get into the jshell in my case.

I just did a comparison with a colleague. He is using Mac OS and I'm on Manjaro Linux. The jshell of OpenJDK 15.0.2 is working fine on his notebook while I'm still getting the exceptions above.

@bennetelli Can you check with 15.0.2 from https://adoptopenjdk.net/?variant=openjdk15&jvmVariant=hotspot? Looks like you're still using 15.0.0. If it doesn't work, test with https://adoptopenjdk.net/nightly.html?variant=openjdk16&jvmVariant=hotspot. And please do the same with 15 and 16 from https://jdk.java.net/.

@aahlenst We got it. There is a problem with the rxvt-unicode terminal. Seems like rxvt-unicode and jshell don't work nicely together. Using gnome-terminal works as expected.

If it happens with a build from jdk.java.net, it might make sense to report it to https://bugreport.java.com/bugreport/ by only mentioning the build from jdk.java.net. Please post a link here once the report was processed.

@bennetelli Are you able to try with the latest 17.0.3?

you were faster than me. Thanks for taking a look at it :)


Add MN scrape

https://mn.gov/deed/programs-services/dislocated-worker/reports/
PDFs

When I try to scrape using utils.get_urls or requests.get(url) or requests.get(url, verify=False), the website demands that I prove that I'm not a bot: this contains the page it displays. The other error I get is "max retries exceeded ... unable to get local issuer certificate".
The urls of each pdf ends with some seemingly random numbers, so getting the urls of each individual pdf seems impossible.
Is there a way to bypass this?


Charts & Stats page needs design

Currently, the page is just a list of charts. The page will grow, and ideally, we want to have some structure and navigation there. Meanwhile, this page is too valuable to hide, so we display it as is.
@corwinharrell there are currently two other pages that are in the redesign queue, which have higher priority: #370 (Validators Dashboard is sitting there half-baked, so I strongly believe we should finish it before jumping into something new) and #512 (Transaction details page will have receipts with execution results and logs on it soon, so we might want to revisit its design)

Sounds good @frol. I'll keep this in mind as we prioritize and move forward with design 👍

node design #549
receipts list #512
stats page #525

If I can suggest a small modification, I would identify the epoch with a link from the block number instead of the block hash. This makes it easier to remember. from @jimmy3dita
@corwinharrell


How exactly could I use Complex numbers?

Hello. I was trying to create an ndarray and realized that numdot has complex number support. The problem is that I couldn't find any function that returns an imaginary number (like conj or imag). Is that not added yet? I do realize that maybe the imaginary results where more of a way to deal with the fft calculations.

Hi! Imaginary numbers are planned for the next release (and mostly implemented by now), but not available yet in the latest release. I'm planning to get the next version out within the week though :)

OK, thank you. Sincerely this project seems fantastic. You work pretty fast lol!

Thank you! ☺
The functions I currently have implemented for the upcoming release are listed in the latest changelog, by the way.
I don't work with complex numbers in NumPy often - let me know if I'm missing anything you consider important, and I should be able to add it before the next release!

It seems fine the way it is. I was trying to make a 2d game which flips in a third dimension. I wanted to represent the flipped node (with two children, one for each side) as a ndarray with complex numbers. I didn't want to use quartenions as I only need an extra axis for flipping. I temporarily solved it using the default godot array with a pair of elements for each element. It doesn't work as well because I can't multiply by a complex number to change orientations. I am trying to make symmetry operators that do basically that ($e^{i\theta}$). As I want to create the smallest pattern possible that fills the entire space using those transformations. I thought using complex numbers would create simpler operators. So, I think that if dot and cross products work with complex arrays this is totally fine for my use case!

Release v0.7 is available now, and it includes support for complex numbers (and cross products, which I didn't actually have before).
Let me know if anything isn't working as expected!

Thank you!


feat: update ast for supporting generics

This PR is a follow-up to reverted PR https://github.com/google/wire/pull/360.

Add a testcase for Generic.

@toddtreece Could you help me review this PR 🙇

@aqaurius6666  why was the PR reverted, you have any clue?

@stytchiz it seems the test issue referenced here is solved, why not approve this PR and let us have fun with generics and wire? :)


no rule for default values

ISSUE TYPE

Documentation Report

ROLE VERSION

N/A

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

we have no rule for default values.
a rule for default values should be created. see the discussion for detail.
https://github.com/reallyenglish/ansible-role-monit/pull/16#discussion_r136019347
STEPS TO REPRODUCE

N/A

N/A

EXPECTED RESULTS

we have a rule for default values.
ACTUAL RESULTS

we do not have one.

"default values" for what?


Description of logic for choosing between multiple available debuggee-side DLLs

From my testing, if there are multiple target framework debuggee DLLs -- netcoreapp, netstandard2.0, net2.0:

For .NET Framework -- If the debugged process runtime supports netstandard2.0, and there is a netstandard2.0 debuggee-side DLL, that will be used; otherwise net2.0 will be used.
For .NET Core and .NET 5+ -- netcoreapp will always be used, even if the runtime supports netstandard2.0 and there is a netstandard2.0 debuggee-side DLL available.

Not sure how to describe this in the documentation, or even if it should be described.

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

ID: c0470059-27e3-58a0-155e-84fbaefae158
Version Independent ID: 95c9afb2-32f3-945b-9381-9f711179c77d
Content: Install a Visualizer - Visual Studio
Content Source: docs/debugger/how-to-install-a-visualizer.md
Product: visual-studio-windows
Technology: vs-ide-debug
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@zspitz I'm a little concerned about losing clarity and creating more confusion here. Let me know if you have a suggestion you feel strongly about.


ValueError: max() arg is an empty sequence - after upgrade to 2024.8.1

What type of installation are you running?
Server
Which Linux OS are you using?
Ubuntu 22.04
Which version of LNXLink has the issue?
2024.8.1
Describe the problem
Error message in logs upon service start:
2024-08-09 16:53:01,600 [9181:MainThread.__main__.__init__.46] [INFO]  LNXlink 2024.8.1, Python 3.10.12
2024-08-09 16:53:01,700 [9181:MainThread.__main__.start.84] [INFO]  Loaded addons: inference_time, bash, mounts, sys_updates, required_restart, memory, restart, statistics, update, cpu
2024-08-09 16:53:01,708 [9181:Thread-2 (_thread_main).__main__.on_connect.241] [INFO]  MQTT connection: Connection Accepted.
2024-08-09 16:53:01,763 [9181:Thread-2 (_thread_main).__main__.run_module.135] [ERROR]  Error with addon Inference Time: max() arg is an empty sequence, Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/lnxlink/__main__.py", line 130, in run_module
    pub_data = method()
  File "/usr/local/lib/python3.10/dist-packages/lnxlink/modules/inference_time.py", line 17, in get_info
    "max": max(
ValueError: max() arg is an empty sequence

Additional context
Error was not present in 2024.7.0
Configuration
No response
Logs
Aug 09 16:53:01 stuff lnxlink[922858]: INFO:lnxlink:LNXlink 2024.8.1, Python 3.10.12
Aug 09 16:53:01 stuff lnxlink[922858]: INFO:dasbus.connection:Connecting to the system bus.
Aug 09 16:53:01 stuff lnxlink[922858]: INFO:dasbus.connection:Connecting to the system bus.
Aug 09 16:53:01 stuff lnxlink[922858]: INFO:lnxlink:Loaded addons: inference_time, bash, mounts, sys_updates, required_restart, memory, restart, statistics, update, cpu
Aug 09 16:53:01 stuff lnxlink[922858]: INFO:lnxlink:MQTT connection: Connection Accepted.
Aug 09 16:53:01 stuff lnxlink[922858]: ERROR:lnxlink:Error with addon Inference Time: max() arg is an empty sequence, Traceback (most recent call last):
Aug 09 16:53:01 stuff lnxlink[922858]:   File "/usr/local/lib/python3.10/dist-packages/lnxlink/__main__.py", line 130, in run_module
Aug 09 16:53:01 stuff lnxlink[922858]:     pub_data = method()
Aug 09 16:53:01 stuff lnxlink[922858]:   File "/usr/local/lib/python3.10/dist-packages/lnxlink/modules/inference_time.py", line 17, in get_info
Aug 09 16:53:01 stuff lnxlink[922858]:     "max": max(
Aug 09 16:53:01 stuff lnxlink[922858]: ValueError: max() arg is an empty sequence
Aug 09 17:08:03 stuff lnxlink[922858]: INFO:lnxlink:Sending statistics data: {"uuid": "xxxxxxxxxxxxxxxxxxxxxx", "version": "2024.8.1"}

Inference data appears to still be sent to mqtt fine.

This should only show once it starts and doesn't affect the normal operation of the app.
I found the issue and it should be fixed on the next release.

Yeah confirmed, it's only logged once and no problems with the data.

I've released the 2024.9.0 version of LNXlink which fixes this issue.


Azure Kubernetes cluster autoscaler switch issue

Azure documentation issue guidance
Hello,
Firstly Issue doc Link : https://docs.microsoft.com/en-us/azure/aks/cluster-autoscaler#create-an-aks-cluster-and-enable-the-cluster-autoscaler
Following the doc to create a cluster using the following command :
az aks create
--resource-group myResourceGroup
--name myAKSCluster
--node-count 1
--vm-set-type VirtualMachineScaleSets
--enable-cluster-autoscaler
--min-count 1
--max-count 3
Get issue screenshot as the following :

However, I've already created the autoscaler cluster a number of times by using the following command :
az aks create --resource-group myResourceGroup
--name myAKSCluster
--location northeurope
--kubernetes-version 1.15.3
--generate-ssh-keys
--enable-vmss
--enable-cluster-autoscaler
--min-count 1
--max-count 3

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

ID: 077f1bb5-2ba9-490c-2c6e-0de292266539
Version Independent ID: 0fd87b4f-88a4-c2eb-3ebc-f2b113ac203c
Content: Use the cluster autoscaler in Azure Kubernetes Service (AKS)
Content Source: articles/aks/cluster-autoscaler.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the feedback! We are currently investigating and will update you shortly.

@MicahMcKittrick-MSFT
This flag is available in Azure cli version 2.0.71 but not available in cloud shell Azure cli version 2.0.74
I believe this was removed in the latest CLI to favor the default value.

@cloudmelon I updated the doc to use the --enable-vmss command.
Once the PR merges the changes will go live after a few hours.


After migrating to Calico from Flannel, X-Forwarded-For headers set by the front proxy container are invalid

My on-prem cluster uses MetalLB and a front proxy (Envoy) within a pod. Previously, when using Flannel, the front proxy was able to see the original source IP address because externalTrafficPolicy: Local is set on the Kubernetes service for the front proxy. Since switching to Calico, the front proxy instead reports either a <IP_ADDRESS>/16 or a <IP_ADDRESS>/16 IP in its X-Forwarded-For header.
Expected Behavior
The Flannel migration should work the same as it did before.
Current Behavior
Calico does not appear to properly report the originating IP address to the Envoy container.
Possible Solution
I'm not sure how to fix this; I'm not familiar enough to know which Calico setting I even need to use here, despite reading docs — I can't make heads or tails of ipipMode and vxlanMode.
Steps to Reproduce (for bugs)
Sample service:
apiVersion: v1
kind: Service
metadata:
  name: switchboard
spec:
  type: LoadBalancer
  loadBalancerIP: <IP_ADDRESS>
  externalTrafficPolicy: Local # "pods can see the real source IP address of incoming connections."
  ports:
  - name: http
    port: 80
    targetPort: 80

And here is my pool:
apiVersion: projectcalico.org/v3
items:
- apiVersion: projectcalico.org/v3
  kind: IPPool
  metadata:
    creationTimestamp: "2023-07-10T20:54:28Z"
    name: default-ipv4-ippool
    resourceVersion: "123754290"
    uid: d11ad29c-938e-4719-acfd-c6dc4627e762
  spec:
    allowedUses:
    - Workload
    - Tunnel
    blockSize: 26
    cidr: <IP_ADDRESS>/16
    ipipMode: Always
    natOutgoing: true
    nodeSelector: all()
    vxlanMode: Never
kind: IPPoolList
metadata:
  resourceVersion: "123865121"

Context
This prevents me from using IP-based security in my network. Previously I had whitelisted the internal <IP_ADDRESS>/24 range to allow my home network secure access to login to certain selfhosted services.
Your Environment

Calico version: latest (?) just installed
Orchestrator version (e.g. kubernetes, mesos, rkt): 1.27.3
Operating System and version: Linux

Hi @zaneclaes , based on my understanding and the IPPool you shared, I would expect your pods to be in the <IP_ADDRESS>/16 range so I would have thought that those IPs would have been expected. It might be that I am misunderstanding the requests you are making when you see the logs in your front proxy. Can you also give us details on how you made those requests with the IPPool and service provided? You have also mentioned the <IP_ADDRESS>/16  and <IP_ADDRESS>/24 ranges, what do those represent?

The point is that the IPs I see should NOT be anything at all related to Calico. The goal is that the front-proxy properly sets X-Forwarded-For to the IP of the computer which originated the request, so that I may use the header in downstream services which depend on it as a mode of authentication. The 10.244.x.x is the internal IP range used by Kubernetes/Calico (as shown by the pods themselves), the 169.254.x.x is apparently related to Calico (the docs say its a router?) and did not exist before I migrated from Flannel to Calico. The 192.168.x.x is my home network.
In short, if I am on my home WiFi and open a website served by the front proxy, the X-forwarded-for should be set to 192.168.x.x and not one of those other two ranges created by Calico. And if I open it from my phone outside the house (since I do expose my house to the outside world), it should be set to whatever IP my cell provider assigns. Thus I can properly use the <IP_ADDRESS>/16 subnet to authenticate local on-prem traffic.

Can you also give us details on how you made those requests with the IPPool and service provided?

In case this wasn't clear...
I just open up a web browser and type the URL. When on my home network, the router responds to the DNS lookup with the IP <IP_ADDRESS> (when outside the home network, it port-forwards 80 and 443 to this internal IP address). Thus the request is received MetalLB (per the service definition in the post) and sent to the front-proxy pod. The front proxy (envoy) is running as a DaemonSet, but nodeAffinity restricted to those machines which serve as ingress...

@zaneclaes could you share some more details about your setup? Say, a simple diagram of the topology (how many nodes, how they connect to each other and your router, with the IPs each one has, the routes on each node and how you do routing on your cluster/network), the exact calico version (you can get this from kubectl describe pod on a calico-node pod), etc...
If you could share your flannel config and how exactly you migrated from calico (did you just create a new "blank" cluster with calico and spun up your services on it? something else?), that could help.
Re: IPIP vs. VXLAN, do you have any specific questions? Did you use this doc https://docs.tigera.io/calico/latest/networking/configuring/vxlan-ipip? We'd definitely appreciate pointers on what this doc might fail to explain clearly...
Thanks!

Hi @zaneclaes, sorry, for the late reply. In addition to the details @coutinhop shared above, what is a bit confusing about your use case is that envoy is seeing the Calico pod CIDR at all. I would have expected that envoy would be the entry point into your cluster and so your source IP should be preserved in the X-Forward-For header since Calico should not be modifying HTTP headers. The only way I would have expected you to see any other IP is if the node that your request landed on had none of the envoy pods on it, so traffic is sent to a separate node and NAT'ed but even in this case, you would be seeing an IP from your node CIDR, not your pod CIDR. Is it possible that traffic is landing at a separate pod before it hits your envoy pods?

@coutinhop I can try to diagram it... but I think this is severely overcomplicating things. There are only 2 relevant pieces AFAICT: (1) MetalLB (2) Envoy. When a request hits Kubernetes MetalLB and forwards it to Envoy, the IP is incorrect. Everything else is superficial. The upstream (WAN, etc.) has no impact, and the downstream (Home Assistant) happens after the problem:

There are exactly 2 nodes, each running on the "infra" VLAN (10.0.0.x) with static IPs:

The master node (conductor) runs Envoy
The second node (nixie-beast) runs the Home Assistant application

Here is kubectl get nodes -owide:
conductor       Ready      control-plane,master   2y59d   v1.27.3   <IP_ADDRESS>      <none>        Ubuntu 20.04.6 LTS                        5.4.0-153-generic   containerd://1.6.21
nixie-beast     Ready      <none>                 2y27d   v1.27.3   <IP_ADDRESS>      <none>        Ubuntu 20.04.6 LTS                        5.15.0-76-generic   containerd://1.6.21

Here is the Envoy process and Home Assistant process:
NAME                READY   STATUS    RESTARTS   AGE   IP             NODE        NOMINATED NODE   READINESS GATES
switchboard-ct2vv   1/1     Running   3          16d   <IP_ADDRESS>   conductor   <none>           <none>
home-assistant-0   1/1     Running   1          11d     <IP_ADDRESS>    nixie-beast     <none>           <none>

Here's the full describe on a Calico node.
As for flannel, I had originally installed it when I created the cluster by doing an install of https://github.com/coreos/flannel/raw/master/Documentation/kube-flannel.yml and the flag --pod-network-cidr=<IP_ADDRESS>/16 during kubeadm init. I did no other customization, and didn't think about Flannel at all until I tried to upgrade to Calico via the instructions here.
Finally, IPIP vs. VXLAN, I'm simply unfamiliar with these two terms and have not taken the time to find some external guide as to how they each behave. Perhaps this is out of scope for the documentation, but all the "encapsulation types" section really tells me is that there is a difference in overhead. It vaguely sounds like encapsulation might be related to my issue, but I'm not sure... I don't have a good sense of what problems the wrong method would create.
@mgleung I think MetalLB is what you are looking for (see the first sentence of the bug report). Here is the service description, which is patched with this layer2 reservation for the <IP_ADDRESS>-200 range on the home VLAN for this traffic. Again, this worked fine with Flannel... no changes were made to MetalLB when switching to Calico.

@zaneclaes sorry for the long delay. Are you still seeing this issue or have you moved on from it? If you still need help, we'd need more info...

Yes, this is still happening @coutinhop ... I have been awaiting a reply to the additional info which I already provided... so I'm not sure what you're referring to.

Calico does not appear to properly report the originating IP address to the Envoy container.

Calico doesn't report anything to the Envoy container, the Envoy container should be setting this based on the source IP address on the connection that it received, so we should follow the path of the connection to see where along that path the source IP is being modified.

Finally, IPIP vs. VXLAN, I'm simply unfamiliar with these two terms and have not taken the time to find some external guide as to how they each behave.

I don't think IPIP vs VXLAN is going to be the issue here, because assuming externalTrafficPolicy: Local is behaving correctly, then the traffic in question will never actually be encapsulated (it arrives on the same node it needs to be one, and encap is only used for cross node comms).
More likely we're looking for some sort of NAT. For example, if externalTrafficPolicy: Local isn't behaving correctly, then you will see packets arrive on nodes and be forwarded to another node, at which point SNAT will occur and you would see behavior like you're describing.
Just to confirm, you're using MetalLB's L2 service implementation and not Calico's BGP service IP advertisement? I'm not very familiar with MetalLB's L2 implementation, but it's possible that something there is interacting with how Calico functions.
One place to look would be to monitor if any SNAT or MASQ` rules are being hit in iptables, via a command like this on your ingress nodes while they are receiving traffic:
iptables-save -c | grep SNAT

iptables-save -c | grep MASQ

the 169.254.x.x is apparently related to Calico (the docs say its a router?)

That range is the IPv4 link-local range, which Calico does "use" in some very limited capacity, but should actually shows up on any packets. I am struggling to think how this IP showing up in the header would be related to Calico.

@zaneclaes any update?

Closing this issue. Feel free to reopen if there is any further update.


Inconsistent test execution results using MSTest V2

Description

We have a unit test that relies on the fact that the assembly under test will be copied to the TestDeploymentDir.
A second unit test expects a deployment item to be present in the TestDeploymentDir
Executing the first test alone produces a failure, however executing all the tests in the test container leads to no failures.

Steps to reproduce

UnitTestSample.zip contains the minimal code, unit tests, and the runsettings that elucidate the scenario.
To execute both the tests, use "C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe" Bin\Debug\UnitTestSampleTests.dll /Settings:my.runsettings /Platform:x86
To observe the failure, use "C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe" Bin\Debug\UnitTestSampleTests.dll /Settings:my.runsettings /Platform:x86 /Tests:AssemblyLoadDirectoryShouldBeTheSameAsTestDeploymentDir
Reproducible using the TestRunner of VS2019 as well.

Expected behavior
AssemblyLoadDirectoryShouldBeTheSameAsTestDeploymentDir test in the attached sample should pass when executed individually.
Actual behavior
AssemblyLoadDirectoryShouldBeTheSameAsTestDeploymentDir test in the attached sample does not pass when executed individually.
Environment

OS: Win10
vstest.console: 15.0.28307.421
MSTest framework and adapter: 2.1.2

Misc
We tried running with vstest.console 16.0.29506.90, but the results were the same.

The tests have been combined under a single TestClass for the sake of demonstration, in reality they can reside in different test classes and the behavior should be consistent(we expect both the tests to pass) in both the cases.

@jayaranigarg Any thoughts? I found an issue with similar symptoms https://github.com/microsoft/testfx/issues/490, maybe this defect is correlated?

@nohwnd Could you please provide your comments here? This issue has blocked our migration to MSTest v2. Thanks.

Hi @ruhullahshah,
I don't find a lot of documentation about the exact scope of DeploymentItem so I cannot really say if that's a bug or not. Basically, tests marked with DeploymentItem are being copied to the TestDeploymentDir location which isn't the case for other tests. When running the 2 tests in parallel, then the dll is copied upfront because one of the test is marked with DeploymentItem so the 2nd test can work properly. You can move the DeploymentItem to the class level, this way all tests of the class, even run in isolation are going to be run from the TestDeploymentDir.


[Bug]: Mobs not spawning naturally, nor trees growing. in game ticks are fucked?

General Info

[X] I didn't added any mods
[X] I get this issue consistently in single-player
[X] I get this issue consistently in multi-player
[ ] I am crashing and can provide my crash report(s)
[X] I am using the latest version of the modpack

Your launcher
Curseforge
Modpack version
1.8.9
Describe your issue
no mobs spawn naturally nor do trees grow.
Steps to reproduce the issue
No response
Additional Information
a

Yeah same for me pack is so broken better downgrade

The v1.5.9 fixed that issue.


Add info about bulding the project using Visual Studio

Added info about building as I've seen some people struggling with it
Also I'm not sure about the point 5, is the /Release/ there because some old configuration does that?
Also, maybe i should move this into an issue, but i get error while building
error LNK2019: unresolved external symbol "__declspec(dllimport) public: __cdecl QTextCharFormat::QTextCharFormat(class QTextCharFormat &&)" (__imp_??0QTextCharFormat@@QEAA@$$QEAV0@@Z) referenced in function "public: void __cdecl S2Plugin::CPPSyntaxHighlighter::addRule(class QString const &,enum S2Plugin::HighlightColor)" (?addRule@CPPSyntaxHighlighter@S2Plugin@@QEAAXAEBVQString@@W4HighlightColor@2@@Z)
And i really don't get this, it's in CPPSyntaxHighlighter.cpp the mRules.append lines

I think explaining on how to set cmake vars in Visual Studio is a bit beyond the scope of the project though. It implies VS is the recommended way of building, which I did not intend, as I use VSCode mostly. Also, I wouldn't recommend going the way you explain it, but rather use cmake-gui to set the vars and have it create the VS project for you. I haven't tried that with VS myself, but it should work out of the box. Looks like the latest VS does support cmake natively, but it doesn't hurt to try with cmake-gui.
Basically the instructions would be to use cmake gui to set the two vars and generate the project, then compile, or look at the github workflow file and follow those commands to compile. Both these options seemed obvious, so that's why I didn't include any build instructions.
The Release prefix might be a VSCode default, I didn't put in a CMAKE_RUNTIME/LIBRARY/ARCHIVE_OUTPUT_DIRECTORY_RELEASE or similar in there, but that will explicitly influence where the library ends up. You could try adding that if VS complains about it, see if it stops complaining with that setting included.
About the linker error: are you sure you're pointing towards the correct Qt version? You mention 5.3.2 in your instructions, but it needs 5.6.3
Qt5ROOT also gets cached, so it might be referring to an older test you did. You can double-check with cmake-gui.
As you can see, the github workflow can link it fine, so it's not something specific on my pc that I forgot to include in the project.


Initializing room storage with invalid data should not return a 500

Discord feedback: https://discord.com/channels/913109211746009108/913158542565994606/957288610028220497

Shipped in api v2


@immutable should be moved to flutter sdk?

Use case
import 'package:flutter/material.dart';

@immutable
class Resolution {
  final int width;
  final int height;
  const Resolution(this.width, this.height);
  double get aspectRatio {
    return width / height;
  }
}

immutable is not related to material app. I think the immutable annotation should be located somewhere else. Maybe it can be part of the dart SDK? I don't see how this is Flutter-specific.
Proposal
move @immutable
https://github.com/flutter/flutter/issues/160820

The annotation is defined in package:meta/meta.dart, which is in the sdk repository. It is exported by package:flutter/foundation.dart.


Is extension's contentScript needed?

Thanos Wallet will be use tzip-10 for communicate with dApp. That means we don't need to inject something for dApp to user page.
What else might Thanos Wallet need in extension's contentScript?

Solved.


When is javax.servlet a required dependency?

It seems javax.servlet is an optional dependency of the Vonage Java SDK by design (https://github.com/Vonage/vonage-java-sdk/commit/a39be5f998741862c1aa7d29661f0f6aebf3fda1). I can not find any documentation mentioning that it is an optional dependency, nor when it is necessary to have it installed when using the Vonage Java SDK. It seems to not be required for sending SMS with request signing, but it has to be installed anyway due to a (presumably accidental) hard dependency.
With version 5.6.0 of the Vonage Java SDK, I could send SMS with request signing just fine without javax.servlet. After updating to version 6.4.0, sending SMS with request signing will fail unless javax.servlet is installed:
java.lang.NoClassDefFoundError: javax/servlet/ServletInputStream
	at com.vonage.client.auth.SignatureAuthMethod.apply(SignatureAuthMethod.java:47)
	at com.vonage.client.AbstractMethod.applyAuth(AbstractMethod.java:127)
	at com.vonage.client.AbstractMethod.execute(AbstractMethod.java:73)
	at com.vonage.client.sms.SmsClient.submitMessage(SmsClient.java:74)
	...
Caused by: java.lang.ClassNotFoundException: javax.servlet.ServletInputStream
	at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:418)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:352)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:351)
	... 6 more

I am fairly certain that com.vonage.client.auth.RequestSigning has a hard dependency on javax.servlet due to the runtime class verification of the JVM. More specifically, an implicit cast from javax.servlet.ServletInputStream to java.io.InputStream is causing this hard dependency which was added here, which is a change that was introduced after 5.6.0.
Having a hard dependency on javax.servlet from RequestSigning may be perfectly reasonable, but it does not appear to be necessary given that I was able to send SMS with request signing without javax.servlet prior.
For now, it seems the safest option is to always install javax.servlet even if it should not be required.

Thanks for pointing this out. Should be fixed in #445 (v7.3.0) but feel free to reopen if this is not the case :)


Move Kanalictl

This MR lifts and shifts kanalictl under this project and makes any necessary updates for v2.0.0.

Coverage increased (+0.1%) to 75.9% when pulling 60d6fb1f242a4bb36e4824394d5c3a463f56e56b on kanalictl into 2707eaab6823c32fd887010c97a90efe69d7c45b on validation.


The new update does not work with credentials true

Use App
Expected outcome
App response
Actual outcome
"error": "You need to sign in or sign up before continuing."
Versions
App: <IP_ADDRESS>-stable
Chrome: 50.0.2661.75
Platform: Windows (10)
Steps to reproduce

Duplicate of #487


feat: 🎸 Bump SDK version to v22

JIRA Link
NA
Changelog / Description
Bump SDK version to v22
Checklist -

[ ] New Feature ?
[ ] Updated swagger annotation (if API structure is changed) ?
[ ] Unit Test (if possible) ?
[ ] Updated the Readme.md (if required) ?

Waiting for https://github.com/PolymeshAssociation/signing-managers/pull/44 to release a new version

:tada: This PR is included in version 4.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Does it support MIDI channels?

Hello there. Maybe I'm too new in C++ world, but by looking at source code I haven't found MIDI channelling anywhere. Can you help me do this?

Hi @damirka and thanks for reaching out!
So I personally didn't need to control the MIDI channel and just used channel 0 hard coded.
This could be easily changed though.
All there needs to be done is propagate a variable all the way to send() method in midi.cpp instead of the hard coded value (https://github.com/levosos/Midier/blob/master/src/midi/midi.cpp#L17).
Let me know if you want to open a pull request with these changes or if I can help you doing that!

Okay @levosos, let's do it this way: I'll make my first PR on C++, and if there's something wrong you'll help me fix this. 😆

No problem :-)


MkDocs deploy has been failing recently

Failed jobs: https://github.com/cpp-linter/cpp-linter-action/actions/workflows/mkdocs-deploy.yml
Run git config user.name 'github-actions'
INFO     -  Cleaning site directory
INFO     -  Building documentation to directory: /home/runner/work/cpp-linter-action/cpp-linter-action/site
WARNING  -  Both index.md and readme.md found. Skipping readme.md from /home/runner/work/cpp-linter-action/cpp-linter-action/docs
ERROR    -  Error reading page 'API Reference/cpp_linter.clang_format_xml.md': No module named 'mkdocstrings.handlers.python'
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/mkdocstrings/handlers/base.py", line 563, in get_handler
    module = importlib.import_module(f"mkdocstrings_handlers.{name}")
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line [9](https://github.com/cpp-linter/cpp-linter-action/runs/6997858571?check_suite_focus=true#step:5:10)92, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line [10](https://github.com/cpp-linter/cpp-linter-action/runs/6997858571?check_suite_focus=true#step:5:11)50, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1004, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'mkdocstrings_handlers'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.10.5/x64/bin/mkdocs", line 8, in <module>
    sys.exit(cli())
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/click/core.py", line [11](https://github.com/cpp-linter/cpp-linter-action/runs/6997858571?check_suite_focus=true#step:5:12)30, in __call__
    return self.main(*args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/mkdocs/__main__.py", line 215, in gh_deploy_command
    build.build(cfg, dirty=not clean)
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/mkdocs/commands/build.py", line 292, in build
    _populate_page(file.page, config, files, dirty)
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/mkdocs/commands/build.py", line 174, in _populate_page
    page.render(config, files)
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/mkdocs/structure/pages.py", line 175, in render
    self.content = md.convert(self.markdown)
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/markdown/core.py", line 264, in convert
    root = self.parser.parseDocument(self.lines).getroot()
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/markdown/blockparser.py", line 90, in parseDocument
    self.parseChunk(self.root, '\n'.join(lines))
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/markdown/blockparser.py", line 105, in parseChunk
    self.parseBlocks(parent, text.split('\n\n'))
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/markdown/blockparser.py", line [12](https://github.com/cpp-linter/cpp-linter-action/runs/6997858571?check_suite_focus=true#step:5:13)3, in parseBlocks
    if processor.run(parent, blocks) is not False:
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/mkdocstrings/extension.py", line 121, in run
    html, handler, data = self._process_block(identifier, block, heading_level)
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/mkdocstrings/extension.py", line [17](https://github.com/cpp-linter/cpp-linter-action/runs/6997858571?check_suite_focus=true#step:5:18)5, in _process_block
    handler = self._handlers.get_handler(handler_name, handler_config)
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/site-packages/mkdocstrings/handlers/base.py", line 565, in get_handler
    module = importlib.import_module(f"mkdocstrings.handlers.{name}")
  File "/opt/hostedtoolcache/Python/3.10.5/x64/lib/python3.10/importlib/__init__.py", line 1[26](https://github.com/cpp-linter/cpp-linter-action/runs/6997858571?check_suite_focus=true#step:5:27), in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 10[27](https://github.com/cpp-linter/cpp-linter-action/runs/6997858571?check_suite_focus=true#step:5:28), in _find_and_load
  File "<frozen importlib._bootstrap>", line 1004, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'mkdocstrings.handlers.python'
Error: Process completed with exit code 1.

funny I just noticed that. It seems the mkdocstring plugin recently updated in a breaking way. I didn't know it was a beta plugin.
I'm still looking into it. So far I've fixed the ModuleNotFoundError, but the mkdocstring behavior seems to have changed also (need to document the module name instead of the associated __init__ module). Now I'm just fixing the cross-references in the dev docs.

So far I've fixed the ModuleNotFoundError

Btw, do you find the reason why our last test (in this run https://github.com/cpp-linter/test-cpp-linter-action/actions/runs/2527084295) did not show ModuleNotFoundError? We'd better prevent it from happening again

Those CI workflows (on the test repo) aren't testing the docs build. The ModuleNotFoundError error was happening when running mkdocs build because the mkdocs plugin, mkdocstring, changed how their plugin is distributed via pip install; I think they're planning to add more than just python syntax support (hopefully).

Sorry, my bad. Actually, I want to ask the question about FileNotFoundError in the issue #67


Edit

@jessicaskl, the behavior is spot-on! Just need a couple of visual and a11y tweaks. Here's the stuff we talked about:

Arrange contents w/ CSS grid:

display: inline-grid;
grid-template-columns: auto auto;
grid-template-rows: auto auto;
grid-auto-flow: column;
grid-row property on the button

Replace img tag with restyled button + image tag
Add aria-controls attribute pointing to id of input
Style changes to input in read-only mode


[HDInsight] Support the latest azure monitor api

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

This PR supports GET, PUT and DELETE  of the API "/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.HDInsight/clusters/{clusterName}/extensions/azureMonitor"
Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
Now it is.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
As soon as possible, ETA is 5/27/2021
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[x] SDK of Python
[x] SDK of Java
[x] SDK of Js
[ ] SDK of Go
[x] PowerShell
[x] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[ ] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[x] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Will merge once ARM sign off.

Thanks~

Hi @RamyasreeChakka , Thanks for your review. Our backend only returns 202 for the put request without any schema.
I add this 200 response code just in order to satisfy the validation rule.

/azp run


Handling Port Conflicts

Running on my default machine (port 80 seems to be locked to Apache)
I have adjusted docker-compose.yml to create "alias" ports
        ports:
           - "2222:22"
           - "33060:3306"
           - "8080:80"

But still receive the following errors
ERROR: for nginx  Cannot start service nginx: driver failed programming external connectivity on endpoint laradock_nginx_1 (230a90054690dc1dc7bf6e964d9f575fd98201fe078c6f0e28dc1cd3576b97dd): Error starting userland proxy: Bind for <IP_ADDRESS>:80: unexpected error (Failure EADDRINUSE)

ERROR: for mysql  Cannot start service mysql: driver failed programming external connectivity on endpoint laradock_mysql_1 (ee1d5503b571b18a6ae08d7cde591f4117dc253397ca79301389ebd2f73fb270): Error starting userland proxy: Bind for <IP_ADDRESS>:3306 failed: port is already allocated
ERROR: Encountered errors while bringing up the project.

@mikeerickson What commands are you using to bring the project up?

@philtrep I figured it out.... I set the port in the wrong section (didn't set in the ngnix section) I set it in the workspace section (now I need to figure out what the workspace section does)


Parameters to set FPS in video input file

Hello everyone,
I'm using opendatacam on desktop with gtx1080 gpu.
I do drag and drop files in the browser, then the video runs at 35-40 Fps.
how to set the config setting to change the Fps in 20-25 fps in video input file
Thank you

Hi,
This is not possible currently
In order to implement this, would need to set a limit into the darknet code to process the frame not more than 30 FPS ..
Not planning to add this for now, but you are welcome to do so and contribute a pull request
Have a nice day

Another possible work around could be: Upload the file onto the machine using SSH and then use the gst filesource and videorate filters