Make the alert components modular

Alert messages and descriptions should be passed through the constructor

@Roshaen I would like to work on it. And can you tell me a little more about the changes, I've to perform.

What you have to do is you have to make the component dynamic means if someone is using this component then he/she only have to pass the properties or attributes and the desired widget would be delivered. So you have to take properties example title, description, color, etc from constructor and then have to use those values in your code.

Oh sure, @Roshaen I want to work under JWOC on this issue.

Ok @sarkartanmay393
Assigning you
Go through the Contribution guidelines to get started

@Roshaen Close the issue and set point label on my pr please.


Establish basic UX flow to handle balance update

blockchain address related to avatar should alway be accessible.
one balance per token?

sorry accidentally closed.

The overall consensus among the team is that users should have the ultimate say on where their tokens remain staked after an balance update. That is, when there is balance update/mismatch/change between what is recorded in our DB and what the publicAddress possesses at the moment, the app can do some updating automatically, show pending status where appropriate, and ultimately let user decide and confirm. More specifically, this will possibly require some changes regarding:

UI updates to indicate pending status
Sync/connection with daemon as appropriate (#361)
What is learned from user journeys (#379) and other user research and test, it's ongoing learning

These should be translated into specific issues as needed.


Error: 'isComplete' can have side effects

Can't compile.
code:
import scorper
const port{.intdefine.} = 8888
when isMainModule:
  proc cb(req: Request) {.async.} =
    let headers = {"Content-type": "text/plain"}
    await req.resp("Hello, World!", headers.newHttpHeaders())
  let address = "<IP_ADDRESS>:" & $port
  waitFor serve(address, cb)

error:
PS C:\Users\\NimProjects\scorpertests> nim r .\scorpertestss.nim
Hint: used config file 'C:\Users\\Nim\nim-1.4.8\config\nim.cfg' [Conf]
Hint: used config file 'C:\Users\\Nim\nim-1.4.8\config\config.nims' [Conf]
..............................................................................................................
C:\Users\\.nimble\pkgs\scorper-1.0.10\scorper\http\streamserver.nim(912, 6) Error: 'isComplete' can have side effects

it's fixed in devel, now published to v1.0.11


Support guru's archive format for modified files

Closes #4

Please do not submit yet, I'm still looking into a bug.

Never mind my previous comment. No (known) bugs, ready to submit.


doc: remove irritating tripple dots from install instructions example

I am probably not the first one to stumble over that when copy&paste the code into the project

Never mind. The 3 dots indicate that this is the didFinishLaunchWithOptions method which already exists and needs to be extended.


the-powder-toy: 97.0.352 -> 98.2.365

Description of changes
Changelog: https://github.com/The-Powder-Toy/The-Powder-Toy/compare/v97.0.352...v98.2.365
Additional (nix) changes:

moved package from pkgs/games to pkgs/by-name

for that purpose, restructured the parameter list to be backwards compatible with what was overrideable from all-packages.nix

drop python dependency, it is just a dependency of meson and does not need to be specified explicitly
make use of more modern nix tooling to install the desktop file
add missing preInstall and postInstall run hooks
drop unnecessary patches that were previously needed to compile on gcc13: the patch was merged upstream
bump lua dependency from 5.1 to the current lua (5.2)
ordered parameters and build inputs to be alphabetical
ran nixpkgs-fmt

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Note: I do not own any darwin machines, i am waiting for ofborg to build this but can't actually run the binaries either. Do not merge if darwin tests didn't succeed yet. And feedback from darwin users is welcome.

Ok, ofborg logs look fine. Quick heads-up by a darwin user would still be appreciated but it seems to work as intended.

Addressed the feedback. Though i can't say i quite agree with it staying in all-packages.nix seeing as i tried keeping the override interface consistent, it is probably easier to just leave it in all-packages instead of arguing over it.


change names of calls to StorageHostsMappingApi

The name of a class and a model in the oas client has changed, so when calling to the old name, an error is raised.
StorageHostVolumeMappingApi -> StorageHostsMappingApi
StorageHostVolumeMapping -> StorageHostsMapping
StorageHostVolumeMappingCreate -> StorageHostsMappingCreate
I changed the calls to the api and models to fix it in the two files where it is relevant

.

@Lauferin what is the new gem version?  This is a breaking API change and the gem should have a major version bump

@agrare We created a new gem and changed version to 1.2, everything should be fine now.

Okay, we want to follow semantic versioning (https://semver.org/) so any breaking change should be a major version bump not a minor one as you have here.
I'm going to merge this but follow semver going forward

@Lauferin you also need to yank the "bad" version (1.1.35 thru 1.1.40) as they are breaking release branch builds

@agrare , yanked and taggedto 1.1.34.


Accessibility Change

I thiiiiink this should introduce a very rudimentary accessibility for blind players to use Monarch. Adds each card's Play button to the tab index, allowing the player to tab through their hand one at a time. Also adds an aria label that names the card and indicates that the button may be pressed to play it.

What I've added, is a string to the English language file to handle the label. Rather than composing the card.name with CARD.Play in the template, the card name is being inserted into a localizable string. This would allow languages with different rules for word order and punctuation to translate this label as appropriate.
I also added the same label to card piles, since they also have play buttons.


Unable to find library, no such file or directory

I'm attempting to compile this simple bounce.ino example with ino, but it's giving me the error:
Bounce2.h: No such file or directory

I've placed it in the same directory as bounce.ino. I still get this error even if I use arduino to add the library to my ~/sketchbook/libraries folder.
Why can't it find the header? How do I specify that the local directory should also be checked for headers? Why can't ino see the libraries in the standard ~/sketchbook/libraries` folder?

You can fix this issue by adding the following to environment.py at line 73:
lib_dir_extra = os.path.expanduser('~/sketchbook/libraries')

and changing the line in commands/build.py at 249 to:
lib_dirs = [self.e.arduino_core_dir] + list_subdirs(self.e.lib_dir) + list_subdirs(self.e.arduino_libraries_dir) + list_subdirs(self.e.lib_dir_extra)

After putting the library directory in both the lib directory as well as the /usr/share/arduino/libraries/ this was still not working. Did ino clean and then ino build and it worked.


Implement the initial structure of the Project Template of the todo template (#1793 )

this closes #1793

@MeysamBahadori please update this PR based on our conversation :)

Ok

Ready for review...

BlazorMode and WebAppDeploymentType symbols removed.

@MeysamBahadori
plz resolve the conflicts

Plz review again...


Superfluous use of finalAttrs.{pname,version} introducing overriding traps

Describe the bug
There are many code adopting finalAttrs to reuse variables, rather than provide an overridable option. Specifically, using finalAttrs.version and finalAttrs.pname in src =  is incorrect since overriding them will NOT cause it to automatically use the source from new name/version, because the hash literal is still unchanged. It will be a silent incorrect behavior when the FOD is cached locally (reusing the old one), and will be a hash-mismatch error if it's not, both causing a hard-to-debug trap.
There are also usages in build script like cp -r ${finalAttrs.pname}-${finalAttrs.version}/* $out, which is still incorrect for .overideAttrs { pname = "foo-patched"; } use case.
mkDerivation (finalAttrs: {
  version = "1.2.3";
  src = fetchFromGitHub {
    repo = finalAttrs.pname; # so adding a `-patched` suffix would try to use another repo?
    rev = "v${finalAttrs.version}"; # oops, changing version will not fetch the new src, because of unchanged hash.
    hash = "..some literal hash..";
  };
  # So overriding either `pname` or `version` makes it fail to build.
  installPhase = ''
    cp -r ${finalAttrs.pname}-${finalAttrs.version}/* $out
  '';
})

A quick search shows that there are even more incorrect usages than correct usages on the first glance:

finalAttrs.version: https://github.com/search?q=finalAttrs.version&type=code
finalAttrs.pname: https://github.com/search?q=finalAttrs.pname&type=code

Expected usage
Users should always override src manually, thus making a overridable illusion via finalAttrs is more misleading than not supporting it at all.
mkDerivation rec {
  pname = "some-pkg";
  version = "1.2.3";
  src = fetchFromGitHub {
    repo = "some-pkg"; # or `pname`
    rev = "v1.2.3"; # or `"v${version}"`
    hash = "..some literal hash..";
  };
  installPhase = ''
    cp -r some-pkg-*/* $out
    # Or cp -r ${some-pkg}-*/* $out
  '';
}

It seems the even the good old rec is more correct for the pname usages because it's fixed at parse time and will not be overridden anyway, which is the intended behavior for repo and installPhase. :thinking:
Inspired by #277994. cc @AndersonTorres

Add a :+1: reaction to issues you find important.

@oxalica in description
.overide { pname = "foo-patched"; }
i think you mean
.overrideAttrs { pname = "foo-patched"; }

repo = pname; is an anti-pattern, and finalAttrs.pname is another.
Reuse variables to avoid rec anti-pattern is a legitimate use of finalAttrs.
src needs to be overridden. The other solution is to pass src as an input, but I don't know why this is not popular enough in Nixpkgs.

It will be a silent incorrect behavior when the FOD is cached locally (reusing the old one), and will be a hash-mismatch error if it's not, both causing a hard-to-debug trap.

This is more of a design flaw in hash name itself. It will happen during override and update. See https://github.com/NixOS/rfcs/pull/171

I would be interested in some kind of function based src attribute.
Something like this:
stdenv.mkDerivation {
  pname = "foo";
  version = "1.2.3";

  srcHash = "sha256-AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA="

  srcFor = { version, hash, ... }: fetchFromGitHub {
    owner = "foobar";
    repo = "foo";
    rev = "refs/tags/${version}";
    inherit hash;
  };
};

stdenv.mkDerivation {
  pname = "foo";
  version = "1.2.3";

  srcRev = "f853c074c86ce58df79fab3cfb80ae69133a31e1";
  srcHash = "sha256-AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA="

  srcFor = { rev, hash, ... }: fetchFromGitHub {
    owner = "foobar";
    repo = "foo";
    inherit rev hash;
  };
};

Where mkDerivation will assign src like this:
  src = attrs.src or srcFor = {
    inherit (finalAttrs) version;
    inherit (stdenv) system;
    rev = finalAttrs.srcRev or null;
    hash = finalAttrs.srcHash;
  };

A bigger shortcoming of this would be that it only works for derivations that need a single source.

in my humble opinion, src should be an input:
{lib, stdenv, src, . . .}:

I think the root cause for this issue is that maintainers do not want to repeat the version number for src and version attributes. But src.rev and version are usually not identical either (i.e. refs/tags/x.y.z vs. x.y.z).
Maybe we can try to embed version numbers into src directly, use src as an input and then set inherit (src) version; in the derivation.
Example:
{
  stdenv,
  fetchFromGitHub,
  src ? fetchFromGitHub {
    owner = "foobar";
    repo = "foo";
    version = "1.2.3";
    tagPrefix = "v";
    hash = "sha256-AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA=";
  },
}:
stdenv.mkDerivation {
  pname = "foobar";
  inherit (src) version;

  inherit src;
}

Obviously we would have to implement version for all common fetchers, while some will need some extras such as tagPrefix.
In the case of fetchFromGitHub:
rev = if rev == null then
  "refs/tags/${toString tagPrefix}${version}"
else rev;
```

I think this could be a viable way forward, as it makes overriding sources much easier, while directly tying `version` to `src`.

This should be a bit more polished.
As an example, I gathered all relevant sources for CBQN in one single file, sources.nix:
https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/interpreters/bqn/cbqn/sources.nix

@getchoo

  version ? "0.1.0",
  src ? fetchFromGitHub {
    owner = "foobar";
    repo = "foo";
    # here we can reuse `version` like before - or not! `src` and `version` are no longer tied
    # but both are still reusable, overridable, and accessible
    rev = "v${version}";
    hash = "...";

But then, when you override the version, the hash will break it.
I agree with @Scrumplex's examples above, putting src into the inputs this way is superior. I would totally support an RFC wanting to allow doing that.
Also my https://github.com/NixOS/nixpkgs/pull/316668 seems very relevant here, given it introduces a bunch of lib.sources functions that can be useful here.

This isn't caused nor exacerbated by finalAttrs at all. If you override version and not src, even without finalAttrs, you'll also silently use the existing sources. That's just how FODs work.
The finalAttrs pattern in fact makes it easier to override src, as you may only need to set outputHash:
packageName.overrideAttrs (oldAttrs: {
  version = "0.1.1";

  src = oldAttrs.src.overrideAttrs {
    outputHash = "...";
  };
})

Without finalAttrs, you'd still need to set src, and more verbosely at that:
packageName.overrideAttrs (oldAttrs: {
  version = "0.1.1";

  src = fetchFromGitHub {
    repo = "...";
    owner = "...";
    rev = "...";
    hash = "...";
  };
})

Quoted from @eclairevoyant, original link https://github.com/NixOS/nixpkgs/issues/293452#issuecomment-1979152319
So it's still simpler to use finalAttrs.version.

I agree with @Scrumplex's examples above, putting src into the inputs this way is superior. I would totally support an RFC wanting to allow doing that.

Well I am doing some experiments, and indeed it is way superior:
https://github.com/NixOS/nixpkgs/pull/322093
https://github.com/NixOS/nixpkgs/pull/321056

Well I am doing some experiments, and indeed it is way superior:
#322093 #321056

Be careful. You don't know when the gnome people will come up with a project naming "sources".

Then let's add version to the name of the FOD, then this shouldn't be an issue.

Well I am doing some experiments, and indeed it is way superior:

#322093 #321056

Be careful. You don't know when the gnome people will come up with a project naming "sources".

In all seriousness a problem of this nature is a genuine possibility, but prefixing simply named non-package arguments with an underscore, e.g. _src or _sources, would probably be a sufficient guard.

In all seriousness a problem of this nature is a genuine possibility, but prefixing simply named non-package arguments with an underscore, e.g. _src or _sources, would probably be a sufficient guard.

Yeah, we discussed in this issue https://github.com/NixOS/nixpkgs/issues/309892

Well I am doing some experiments, and indeed it is way superior:
#322093 #321056

Be careful. You don't know when the gnome people will come up with a project naming "sources".

Famous last words...

This isn't caused nor exacerbated by finalAttrs at all. If you override version and not src, even without finalAttrs, you'll also silently use the existing sources. That's just how FODs work.
The finalAttrs pattern in fact makes it easier to override src, as you may only need to set outputHash:
packageName.overrideAttrs (oldAttrs: {
  version = "0.1.1";

  src = oldAttrs.src.overrideAttrs {
    outputHash = "...";
  };
})

Without finalAttrs, you'd still need to set src, and more verbosely at that:
packageName.overrideAttrs (oldAttrs: {
  version = "0.1.1";

  src = fetchFromGitHub {
    repo = "...";
    owner = "...";
    rev = "...";
    hash = "...";
  };
})

Quoted from @eclairevoyant, original link #293452 (comment)
So it's still simpler to use finalAttrs.version.

How does this work when using srcs as opposed to src?

Overriding lists doesn't work very well.


Running basic-key or clicker-key results in babel/register error

I'm getting this following error:
Failed to load external module @babel/register
Requiring external module babel-register
fs.js:27
const { Math, Object } = primordials;
^
ReferenceError: primordials is not defined
at fs.js:27:26

Adding this to package.json solved this issue:
  "resolutions": {
    "graceful-fs": "4.2.3"
  }


Feature Request: @AfterConstructor

Proposal
Add a hook method after Constructor.
It can help developer to validate/cache/compute.
Hope Implementations
  public class SomeClass {
  private final LocalDate birthDate;
  private int age;

  @AfterConstructor
  private void afterConstructor() {
    compute();
    valid();
    cache();
  }

  private void compute() {
    this.age = LocalDate.now().getYear()-birthDate.getYear();
  }

  private void valid() {
    Preconditions.checkArgument(age<150);
  }

  private void cache() {
    //do cache...
  }
}

Notes:
1.I hope the feature is a new method annotation.
Because i guess that developer hope the after constructor method excute after all generated constructors will covers 90% of cases.
In this case,a new method annotation is better than @XArgsConstructor attribute.
2.Ignore or not allow explicitly written constructor.Of course,add an option in lombok.config is perfect practice.
Explicitly written constructor can do anything without lombok.
Lombok should not interfere explicitly written code and inject explicitly written constructor is very difficult.
3.For some cases that method only need excute after some constructors,@AfterConstructor can have a attribute such as excludeConstructs,param type is @XArgsConstructor annotation class array.
I think this is the most important feature that lombok should have, but it does not have.

There is already a [[Wiki page|FEATURE-IDEA: @After]] for this idea.


incoming requests with content-length >=<PHONE_NUMBER> are aborted

Trying to receive a request that has a content-length >=<PHONE_NUMBER> (1gb) instantly closes the connection and calls onAborted.
I've been looking for some limit / max magic number in the source of uWebSockets and uWebSockets.js, but can't find anything anywhere.
Is this configurable somehow, or can this limit be removed?
Here's a sample setup:
import fs from 'fs'
import uws from 'uWebSockets.js'

const app = uws.App()

app.post('/*', (res, req) => {
  console.log('length', req.getHeader('content-length'))
  res.onAborted(() => console.log('aborts instantly'))
  res.onData((data, isLast) => {
    console.log('ondata never fires')
  })
})

app.listen(1234, x => x && console.log('listening'))

Then
mkfile -n 1953125b bigfile.mb // precisely<PHONE_NUMBER>
curl -v --data-binary @bigfile.mb http://localhost:1234

If you send it as chunked it can have any length.

That's unfortunately non trivial when uploading from browsers. Is there any chance we could sponsor this fix ? :)

I started looking at it, it should be simple (mainly changing from unsigned int to uint64_t) but it's cluttered all over, maybe best to begin by replacing all unsigned int to some typedef, then experimentally change that typedef to uint64_t but requires a few changes to masks

Sounds great, I did take a short peek back when creating the issue, but I unfortunately don't have the time to take a stab at it myself.
P.s. off topic, but now that I've got you. I wrote you an email late last year regarding paid work on adding http client support. Don't know if it got caught up in spam, but I'm still interested, so if it's something you'd want to talk about you can catch me on<EMAIL_ADDRESS>

I probably missed it, ignored it, or some combination of both. Right now I have no time. But we will see what happens in the future.

That's unfortunately non trivial when uploading from browsers. Is there any chance we could sponsor this fix ? :)

no its not, its incredibly easy. const chunks = Math.ceil(fileSize/chunkSize); then send a payload like {chunk: n, bytes: data}; and reassemble on the server. why is this hard, we've been doing this for years.

Ideally it should just work


Adjust text font size

Is it possible to add a second parameter to the "text" element that controls the font size for that text paragraph?

See https://github.com/crobi/rpg-cards/issues/44

Hey, thanks!


what int8 average pooling round mode?

I use the int8 average include pooling, and compare the result with my own C naive code, some times the error is +-1, I change my naive code with int/int and round away form zero, the results are also not exactly right. I find the source code in mkl-dnn, find _mm_cvtss_si32 , but I also don't know the round mode.

Hi @chenqy4933,
You can find reference pooling implementation here. DNNL respects the rounding mode set by MXCSR register. Unless redefined by application the rounding is performed to nearest even integer (NRE).

Thanks very much for your answer @vpirogov ，I get it.


feat(aws): Adds support for AWS SSM

Objective: closes #251
Adds support for ssm_parameter ssm_activation
  NAME                                         MONTHLY QTY  UNIT              PRICE   HOURLY COST  MONTHLY COST

  aws_ssm_activation.advanced
  └─ On-prem managed instances (advanced)              730  hours             0.0069       0.0069        5.0735
  Total                                                                                0.0069        5.0735

  aws_ssm_parameter.advanced
  ├─ Parameter storage (advanced)                      730  hours             0.0001       0.0001        0.0511
  └─ API interactions (advanced)                         -  10k interactions  0.0500            -             -
  Total                                                                                0.0001        0.0511

Thanks @robh007! I tweaked a few things in https://github.com/infracost/infracost/pull/338/commits/9c2f1fbac44501f4147b4bf5d8951ea2a9fd3af4 (the commit message explains).

Thanks @robh007! I tweaked a few things in https://github.com/infracost/infracost/pull/338/commits/9c2f1fbac44501f4147b4bf5d8951ea2a9fd3af4 (the commit message explains).


Can't Access Google Account Using Huawei

Hello, i can't sign in using my google account using the last version of the ReVanced GmsCore v<IP_ADDRESS>.240913 on my Huawei without google services.
I was using the last version of Vanced v0.2.24220220 before updating Youtube  to the last version. Vanced had an option for Huawei without google services was working fine.
Now i can't use my Google Account with Youtube, can you fix it
Thanks

i'm having the same issue here, it was working fine with the older microG, not with the GmsCore it's not signing in to google.

This problem is already resolve https://github.com/ReVanced/GmsCore/issues/6#issuecomment-2020340735

@rawen90 , Many thanks this saved alot of my time searching for solution. this work like charm

As a suggestion, you could put an option when starting the application to start in Huawei just like Vanced did, but the method mentioned above works.
Thank you


Ability to register package routes in required place of the app

At the moment all package endpoints are registering automatically after package installation.
It good, but we can't wrap it into middlewares or modify prefixes.
Good solution to resolve it - will be implement some kind of helper which will register routes in required place.
As a reference - we can implement the same logic as Laravel providing in Auth::routes() method.
It also will be very helpful if the package will automatically determine whether he should register routes automatically (if a helper didn't call) or not.

@DenTray, done. Please check

@RGO230 as discussed, please check my case described in the similar task in media package and make the same changes here

Released in 0.0.2-beta


How do I use hasMany through with where statement

How do I achieve this:
tracks: function () {
		return this.hasMany('track').through('album')->where('is_publish',1);
	},

Does it not work if you change the -> to a .?

No, it doesnt work :s, would have been awesome if I could just add .where('is_publish',1); straightly from model

when I add .where('is_publish',1) it shows me a msg like "tracks is not defined on the model."

How did u intend to handle such cases?

You need to put the where first if it's on the track and not the album.

@bendrucker how do you mean, where in the code the condition should be added?

this.hasMany('track').through('album').where('is_publish',1) is not the same as this.hasMany('track').where('is_publish',1).through('album'). The former expects is_publish` on album, the latter on track.

@bendrucker
error: Cannot call method 'through' of undefined" after I replaced with this.hasMany('track').where('is_publish',1).through('album') in channel model

Actually I am doubting if bookshelfjs let me do such things like adding where clause into the model

what does Cannot read property 'parentFk' of undefined mean?? i get this error after I changed where(''is_publish',1') to .where({'is_publish': 1})

well guys, this did the job return this.hasMany('track').through('album').query('where', 'is_publish', '=', 1); it works now....


Fixed exception when adding duplicated entries to a dictionary

Adressing #407.
Guard clause added by calling the ContainsKey prior to calling Add.

This is looking good to me, and the build-release.sh scripts runs fine on my machine :+1:

Thank you for your contribution! It's now live as AutoFixture 3.34.1.


Add set function to system status

Similar to https://github.com/mavlink/MAVSDK/pull/1718

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 9 (0.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 33.853%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/core/mavsdk.cpp
0
3
0.0%

src/core/mavsdk_impl.cpp
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/core/mavsdk.cpp
1
29.63%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
6011

Relevant Lines:
17756

💛  - Coveralls


Amazon functionality validation faild due to video play in background in sleep mode in FireTV stick

Hello
I have submitted my react-native-tvos app to the Amazon store but they give a functionality validation error.
Here I attached a screenshot of the error. please help me to fix this issue.
I have used below version of tvos and react native video.
"react-native"<EMAIL_ADDRESS>"react-native-video": "^5.2.1"
Platform

Amazon (Android)

Device

Amazon Fire TV Cube
Amazon Fire TV Stick 4K
Amazon Fire TV Stick 4K

Environment info

React native info output:
System:
OS: macOS 13.2.1
CPU: (8) x64 Apple M1
Memory: 31.08 MB / 8.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 19.4.0 - /usr/local/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 9.2.0 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
Android SDK: Not Found
IDEs:
Android Studio: 2022.1 AI-221.6008.13.2211.9514443
Xcode: 14.2/14C18 - /usr/bin/xcodebuild
Languages:
Java: 19.0.1 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.0.0 => 18.0.0
react-native: Not Found
react-native-macos: Not Found
react-native-tvos: 0.69.8-1
npmGlobalPackages:
react-native: Not Found

Steps To Reproduce

Install and launch the app
Play any video
Long press Home button to invoke HUD (Heads UP Display) overlay
Select Sleep to put the device to Sleep mode
Select any button to come out of sleep mode and relaunch the app
Actual Result: observe that app audio continues to play in background

Expected behaviour
Returning back from sleep mode should not result in inconsistent behavior of the app
Reproducible sample code
here is my react-native-video code.
  <Video
    ref={setPlayer}
    subtitle={true}
    source={{
      uri: configuration.url,
    }}
    title={title}
    subTitleText={subtitle}
    style={[styles.playerLoaded]}
    paused={pausedRef.current}
    onLoad={handleLoad}
    resizeMode={'contain'}
    rate={playerRate}
    onProgress={handleProgress}
    onError={onError}
    onLoadStart={onLoadStart}
    onSeek={handleSeek}
    onReadyForDisplay={onReadyForDisplay}
    onEnd={onEndVideo}
    playInBackground={false}
    playWhenInactive={false}
  />

I think you should be able to handle this is case with following lib: https://github.com/eliasrodeloso/react-native-lock-detection
You should be able to stop playback when screen is off

No update since long time, let's close.
Ping me if reproduced on 6.0.0


Multi-line textbox support

It would be nice to add multi-line textbox support, highly needed for comment section in osu and when reporting a user.

Duplicate to #1618. Don't mind me being assigned there though. You can give it a shot.


SignGUIResult does not exist

Hello,
I'm getting the following error when my plugin starts:
Plugin SimpleTrade v1.0-SNAPSHOT has failed to register events for class com.zayatv.simpletrade.listeners.InventoryClickListener because de/rapha149/signgui/SignGUIResult does not exist.

And this is the code I have with this api:
private void openSign(Player player)
{
    SignGUI gui = SignGUI.builder().setLine(0, "Type amount below").setHandler((p, result) -> {
        String input = result.getLineWithoutColor(1);
        plugin.tradeInv.openTradeInventory(player);

        return Collections.emptyList();
    }).build();
    inEconomyMenu.add(player);
    gui.open(player);
}

I don't know what might have caused this since my plugin is the only one on the server right now.
Here is my pom.xml:
<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0"
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
    <modelVersion>4.0.0</modelVersion>

    <groupId>com.zayatv</groupId>
    <artifactId>simpletrade</artifactId>
    <version>1.0-SNAPSHOT</version>
    <packaging>jar</packaging>

    <name>SimpleTrade</name>

    <properties>
        <java.version>17</java.version>
        <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
    </properties>

    <build>
        <plugins>
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-compiler-plugin</artifactId>
                <version>3.8.1</version>
                <configuration>
                    <source>${java.version}</source>
                    <target>${java.version}</target>
                </configuration>
            </plugin>
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-shade-plugin</artifactId>
                <version>3.2.4</version>
                <executions>
                    <execution>
                        <phase>package</phase>
                        <goals>
                            <goal>shade</goal>
                        </goals>
                        <configuration>
                            <createDependencyReducedPom>false</createDependencyReducedPom>
                        </configuration>
                    </execution>
                </executions>
            </plugin>
        </plugins>
        <resources>
            <resource>
                <directory>src/main/resources</directory>
                <filtering>true</filtering>
            </resource>
        </resources>
    </build>

    <repositories>
        <repository>
            <id>spigotmc-repo</id>
            <url>https://hub.spigotmc.org/nexus/content/repositories/snapshots/</url>
        </repository>
        <repository>
            <id>sonatype</id>
            <url>https://oss.sonatype.org/content/groups/public/</url>
        </repository>
        <repository>
            <id>jitpack.io</id>
            <url>https://jitpack.io</url>
        </repository>
        <repository>
            <id>dmulloy2-repo</id>
            <url>https://repo.dmulloy2.net/repository/public/</url>
        </repository>
    </repositories>

    <dependencies>
        <dependency>
            <groupId>org.spigotmc</groupId>
            <artifactId>spigot</artifactId>
            <version>1.20.4-R0.1-SNAPSHOT</version>
            <scope>provided</scope>
        </dependency>
        <dependency>
            <groupId>com.github.MilkBowl</groupId>
            <artifactId>VaultAPI</artifactId>
            <version>1.7</version>
            <scope>provided</scope>
        </dependency>
        <dependency>
            <groupId>de.rapha149.signgui</groupId>
            <artifactId>signgui</artifactId>
            <version>2.3.2</version>
        </dependency>
    </dependencies>
</project>

Could you send me your plugin's jar file? If you don't want to send it in here, you can add me on Discord, my username is rapha149

So I took a look at your jar file and my api was not compiled into it. You can check that yourself if you open it with a program like JD-GUI. At the moment the only package is com.zayatv.simpletrade, but for it to work there must also be the package de.rapha149.signgui.
Now, your pom.xml looks good so if you were to compile it with Maven from the command line it should work, but I guess you use your IDE's tools to compile it. You would need to look at the settings in your IDE to extract my API into the generated jar file.

Thank you. It works when I compile it with maven.


PR: Set min and max width for the file switcher

Fixes #4692
Preview:

Also adds a validation to avoid an error when going to previous row for instances with a parent that is not a plugin

Good jobs. My only complaint is that with so much space, perhaps we should stop shorten paths (unless they take more than the minimal width).
I mean, as can be seen in your screenshot, it's strange for us to shorten both file paths when there's ample space to show the full path.
@dalthviz, please see if you can do something about it.


Function to enable the NM service

Write a function that uses the systemd DBus API to enable the NM service.
It should return a result.
Questions:

Should the function return instantly or once the service has been successfully enabled, should this be optional?

https://github.com/resin-io/hq/pull/453


Remove leading space from keywords.txt identifier

The Arduino IDE requires the use of a single true tab separator between the keyword name and identifier. When spaces are used rather than a true tab the keyword is not highlighted.
Reference:
https://github.com/arduino/Arduino/wiki/Arduino-IDE-1.5:-Library-specification#keywords

thanks @per1234, I didn't know that! Thanks for picking up the tabs v space issue and sending the PR.


Can we do Index operations by Alias names?

Update: From the code - https://github.com/olivere/elastic/blob/release-branch.v6/delete.go#L176, it seems that Index and Alias are interchangeable just like ES API, could you please confirm?
Which version of Elastic are you using?
[X] elastic.v6 (for Elasticsearch 6.x)
Please describe the expected behavior
How could I operate on aliases directly without using index names? Basically I want to delete a given Id by an aliase name. The only solution I could see is to get all the aliases and then operate on them in a loop like below
	a,err := es.Aliases().Alias(alias).Do(context.Background())
	idx := a.IndicesByAlias(alias)

But this will require multiple ES delete operations.
Please describe the actual behavior
Is there any API where I could delete in all the indices. For e.g.
a,err := es.Aliases().Alias(alias).Delete().Do(context.Background())
Any steps to reproduce the behavior?
N.A.

The documentation states:

An index alias is a secondary name used to refer to one or more existing indices.
Most Elasticsearch APIs accept an index alias in place of an index name.

I can confirm that indexing documents, removing documents, updating documents etc. works fine when you specify the name of an alias instead of the index name. In fact, it's one of the best features of Elasticsearch, as you can easily do index management with aliases.
What you couldn't do until lately is to post a document to an alias and expect it to update multiple indices. With ES 7.x, you can do something like that.
By reading the official documentation and a bit trial and error, you will sort it out.


[bug] The column selection (Alt + select) is incorrect (offset by 3 characters)

VSCode Version: 1.37.0-insider
OS Version: Win10 1903

Steps to Reproduce:

Try to select something with column selection (alt + left mouse click)
There is an error in the selected line - offset by 3 characters

Gif example below:

Does this issue occur when all extensions are disabled?: Yes
Information that may come in handy:

I have set "editor.multiCursorModifier": to "ctrlCmd"
I have set "editor.insertSpaces": to false (i prefer tabs)
I don't use zoom
Keybindings are default

/confirmed
Looks like there is problem when you use Tabs for indentation and only for selection with a mouse.
Offset between selection cursor and mouse is basically (Tab size - 1) * Indent level.
VS Code version: Code - Insiders 1.37.0-insider (0bd57ac9efae79f9eb1aedc3818ebbb570b9bf6b, 2019-08-02T21:22:13.578Z)
OS version: Windows_NT x64 10.0.18950

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4650U CPU @ 1.70GHz (4 x 2295)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.91GB (2.33GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

So can we talk about it like about a bug?

Yes, looks like it is bug specific to indentation with Tabs. At least I highly doubt that it is expected behavior.
In meantime you can use keyboard for precise multi-line selection. I also haven't noticed this problem when file is indented with spaces.

Duplicate of #78262


Moving network (2) resources to v1beta1 version

Moving network (2) resources to v1beta1 version

[ ] google_network_connectivity_hub
[ ] google_network_connectivity_spoke

Needs permissions for Hub creation.


[swift-stdlib-tool] Add support for back deployed concurrency

This tool was mostly already compatible with copying
libswift_Concurrency.dylib based on whether or not the executables it
was scanning linked it. The only thing missing was that it can now end
up copying from different source directories, the older swift-5.0
directory, and the new swift-5.5 directory.
This change allows users to pass --source-libraries multiple times
(or picks a default with both) in order to have it scan both directories
for any libraries the app links. The biggest part of this change is
instead of storing just the library name throughout this logic we now
store the actual discovered path, so that we don't have to loop through
all potential source directories each time we need to find the library,
only the first time.
https://forums.swift.org/t/swift-concurrency-back-deployment/51908/36

Unlike Xcode's internal implementation of this I didn't put this behind another flag (depending on if folks are using the default --source-libraries or not) because I didn't see a huge reason to do that. I'd be curious if I'm missing some obvious case though

FYI @DougGregor @allevato

@swift-ci please test

@swift-ci please smoke test

thanks! once this lands I'll submit a cherry pick for 5.5

⛵

Thanks all!


minor panel fix and new scrollbar

I agree with your minor panel fixes
I really like the choice of color for the side scroller
One thing, when I open the 'comments slide', I get two side scrollers:

I don't think you have to necessarily get rid of one,
but I wonder if there's a way for them to be farther apart.
what if the scroller for 'contributors' was on the left of the panel?
What are your thoughts?

Yeah your right mate i didn't see that on mine, i think the panel looks better without the scroll bar anyway so i have removed it altogether and just kept the main scroll bar as is.

you think this looks better? just testing this out atm, the white circle is repos, im thinking i might remove it because people wont know what it is aint it,
just have followers what ya think?

Thanks for fixing the 'two side scrollers' issue. I think showing 'Followers' and 'Repos' looks cool, however it's not clear what the 'Repo' indicator is for. If you want to keep the 'Repo' indicator for now, that's alight because it looks cool 😄. What do you think?

We'll figure those indicators out for sure. Otherwise, the changes are good!

Yeah im just trying some other things out cuz that repo thing is a bit random lol

ya think this looks better?

or maybe we use roles, people can give themselves a roll if they want

or


Update ReadMe to reflect personal picture and responsibilities

Add my own picture into AboutUs.md and details of my roles

Codecov Report

Merging #33 (59eb286) into master (4c141e5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #33   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 4c141e5...59eb286. Read the comment docs.


Need API reference for web5.js - JSDoc

We have a high priority request for API reference docs for the web5 js sdk. Something like this generated by jsdoc would be sufficient.
DevRel is happy to assist if needed.

Will sync up with the team and get this queued up.

Request by @csuwildcat - JSDoc by Tuesday, 18th April. "SGTM" - @ALRubinger @angiejones

Moving to Phase 3 per @csuwildcat guidance on readiness.

Assigning to @csuwildcat for triage.

@frankhinek Could you update on whether this is available in 0.6.0?

We have a high priority request for API reference docs for the web5 js sdk. Something like this generated by jsdoc would be sufficient.
DevRel is happy to assist if needed.

@angiejones Docs now being automatically published to https://tbd54566975.github.io/web5-js/
Would be great for DevRel to assist in writing the inline docs as the coverage is very minimal at the moment.

@frankhinek Thank you!
When you say "automatically published", is that for the latest release, or for main?

Identical to how it was handled with dwn-sdk-js: main
https://github.com/TBD54566975/dwn-sdk-js/blob/ec94bcde2d06f84ad8a2af4ea2512201a132db30/.github/workflows/document-gen.yml#L3-L6
https://github.com/TBD54566975/web5-js/blob/ee0107c7b32b9685a6665af753226b5591d83f24/.github/workflows/docs-publish.yml#L1-L8
IMHO These are very simple, temporary solutions that should be replaced with something more robust hosted on developer.tbd.website

Thank you @frankhinek!
Agree with you - as part of some proposals we discussed around delivery/versioning norms, will draft one for OSE to review and provide input on to see if that works. For now this is phenomenal - and we need to be aware that as breaking changes enter main, docs will be ahead of released versions we have in Quickstart 0.6.0, etc.

We have a high priority request for API reference docs for the web5 js sdk. Something like this generated by jsdoc would be sufficient.
DevRel is happy to assist if needed.

@angiejones API reference now being automatically published to https://tbd54566975.github.io/web5-js/
Would be great for DevRel to assist by contributing to the inline JSDoc as the coverage is very minimal at the moment.

@frankhinek this one is pretty barren compared to the NPM one. How is the NPM one generated?


emba_updater without branch - question

Describe the bug
I checked the emba_updater script on Kali (using emba) and on Ubuntu Server (using EMBArk) and on Kali the main repository is used:

but on Ubuntu Server I have to define the branch manually:

To Reproduce
Steps to reproduce the behavior:

EMBArk installation (default mode)
Start EMBArk update script: sudo ./emba_updater
See git messages.

Expected behavior
EMBArk update script should work without manual hit configuration.
Screenshots
See above.
Desktop (please complete the following information):

OS: Kali Linux 2022.4 and Ubuntu Server 22.04.1

Could you try changing the following line:
git pull | tee -a "$LOG_DIR"/emba_update.log

to
git pull origin master | tee -a "$LOG_DIR"/emba_update.log

After the modification it was working on Ubuntu Server:

Thank you!


#haq-gest | Nav Bar reordering, editing

Nav bar should have the following three options

Datasets with 3 sub menus - Summary, Schemes, All Datasets
Data Stories
About Us

Currently the nav bar is not editable via strapi. May not be necessary for the current iteration.

Fixed


Plethora (for Fabric) and Kotlin do not work together

Mod Name
Plethora
Mod Homepage
https://modrinth.com/mod/plethora-peripherals
Minecraft version
1.20.1
Describe the bug
when you start the game crashes
Steps to reproduce

Download CC: Tweaked (for Forge), Connector, Connector Extra, Kotlin for Fabric, Plethora for Fabric, Trinkets for Fabric and API Forgified Fabric .
Start the game

Logs
https://pastebin.com/CPPaTb6R
Additional context
everything starts without Plethora

Caused by: java.lang.IllegalStateException: ComputerCraft has not initialised yet?

This is a conflict between Plethora and ComputerCraft. You can try disabling the integration using Connector's "hidden mods" feature. https://sinytra.org/docs/connector/configuration#hiding-forge-mods-presence


too long time to update the total pod number in dashboard

In GitLab by @tmon546596046 on Mar 16, 2018, 16:49

In GitLab by @liyanq528 on Nov 8, 2019, 07:47
this reason is the collector container restart always;

In GitLab by @tmon546596046 on Nov 8, 2019, 07:47
PR #1081 fix collector restart.

In GitLab by @tmon546596046 on Nov 8, 2019, 07:47
sometime it longer than 5s, it always less than 1s

In GitLab by @sokril on Nov 8, 2019, 08:19
closed

In GitLab by @sokril on Nov 8, 2019, 08:19
assigned to @tmon546596046


[bug] The fee field is not respected when using stx_signTransaction

Is there an existing issue for this?

[X] I have searched the existing issues

SATS-CONNECT Version
3.0.1
Current Behavior
When using unsigned stx transfers, wallet opens a pop-up for the user to sign the transaction, but it doesn't respect the fee set by the developer.
Expected Behavior
Wallet would open the pop-up to sign a transaction with the fee set by the developer.
Steps To Reproduce
Create an unsigned token transfer transaction:
   const txOptions: UnsignedTokenTransferOptions = {
      network: Network,
      amount,
      recipient,
      fee,
      nonce,
      publicKey,
    };
    const serializedTx = serializeTransaction(
      await makeUnsignedSTXTokenTransfer(txOptions)
    );

    const signedTransaction = await xverseRequest("stx_signTransaction", {
      transaction: serializedTx,
      pubkey: publicKey,
    });

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
Usign unsigned contract calls works as expected.

Hi @vini-btc ,Thank you for bringing this to our attention. We are aware of the inconsistencies in the fee settings, where the wallet currently prompts the user to sign the transaction but does not honor the fee set by the developer. We currently updating the fee handling logic and will provide an update as soon as the fix is implemented


Update protos with account get/update operations

What changed?

Manually updated latest submodule for proto/api-cloud
Ran make which updated the account update/get operations in addition to migration from string values to enums

Why?

How did you test it?

Potential risks

Not super up-to-date with the status of Cloud protos. @anekkanti / @cretz, do you know if this is safe to upgrade? Is the change from string to enums implemented?

@bergundy thanks for the approval! Would you mind cutting a new release?

Hey, I'm just verifying with our Cloud team that we're good to release and will get back to it.

https://github.com/temporalio/api-go/releases/tag/v1.40.0


Update prometheus dependency to v2.12.0 to fix validation failure for…

Fixing admission webhook failure of .externalLabels during rulefmt.Parse. Resolves this issue (#2774)

Just realized that go mod vendor sets the version of prom back to v2.9.2...how embarrasing. Didn't even check that. I'm gonna add a unit test to make sure this doesn't get missed.

Looks good, thank you! :)

is it going to be released anytime soon ? mentioned PR by @hatemosphere has issues becase of that validation ( humanizePercentage filter )


Introduce general purpose executors to act as workers (reduces startup time)

Providing a worker pool to cover scheduled and one-off executor processes will reduce the length of time it takes to initialise the application. It could be provided by SafeExecutors.

I don't think this is worth doing.
I just put in a timer check 'before and after' in SafeExecutors.
Timings:
[2015-04-19 16:01:34,160] DEBUG [main] o.m.h.c.c.SafeExecutors - Creation of core-services took 21 milliseconds
[2015-04-19 16:01:36,764] DEBUG [main] o.m.h.c.c.SafeExecutors - Creation of uri-listener took 1 milliseconds
[2015-04-19 15:01:37,580] DEBUG [main] o.m.h.c.c.SafeExecutors - Creation of environment took 4 milliseconds
[2015-04-19 15:01:37,842] DEBUG [main] o.m.h.c.c.SafeExecutors - Creation of transaction-checking took 1 milliseconds

All the others were created in 0 or 1 millisecond.
Not much improvement to be made there so closing.

Pushed some optimisations to start up for your perusal

Seeing definite improvements. On my slow machine I see on repeat start, splash in 2s and Unlock screen in further 4s total.


Fix broken nl2br filter example

This example attempts to handle both Windows-style and Unix-style line endings, but does it wrong; as noted at https://stackoverflow.com/q/12521127/1709587, the regex here will match a single \r\n as if it were a double line-break, which results in all single Windows-style line-breaks getting converted into <p> breaks instead of <br> breaks as intended.
This patch fixes that by applying Alan Moore's proposed fix from https://stackoverflow.com/a/12521544/1709587 of using a negative lookahead so that a \r cannot be counted as a line break if it is followed by a \n (while still allowing the \r\n combination to count as a line break).

Thanks! We should probably be advising people to use CSS white-space: pre-line; instead, but we can still fix the filter.


Fix depwarn on 0.5 for ByteString

Requires JuliaLang/Compat.jl#192 and a new tag.

Update for the slightly incompatible change in JuliaLang/Compat.jl#197. Doesn't need anything from JuliaLang/Compat.jl#198

Thanks!


Added project + online version

This is a link to online version https://codeforse.github.io/ML_Project/cross-val.html the main code is Cross_Validation.ipynb
Adding remark - the project was actually submitted in monday (04.12.2023) and then i've verified from if i submitted correctly from TA's, but then i found out that you did not recieve the PR
For this reasons, I ask for not adding penalty terms ;)

Edit: actually it was submitted on Tuesday, not Monday with delay of 1 day.

This is a link to online version https://codeforse.github.io/ML_Project/cross-val.html the main code is Cross_Validation.ipynb
Adding remark - the project was actually submitted in monday (04.12.2023) and then i've verified from if i submitted correctly from TA's, but then i found out that you did not recieve the PR For this reasons, I ask for not adding penalty terms ;)

Edit: actually it was submitted on Tuesday, not Monday with delay of 1 day.

Merging to master is prohibited, use your branch


when using material 3 with NestedScrollView the app stops

When using NestedScrollView with Material 3, the application crashes. I tried using WidgetsBinding.instance!.addPostFrameCallback and didn't get any results. I'm attempting to have the native scroll of Android 12 and use this widget for the app bar, but I've been without finding a solution for 4 days.
VIDEO
https://github.com/flutter/flutter/assets/82860569/cd15bed2-10fc-4582-83ac-ceac2642e591
CODE

ERROR

FLUTTER DOCTOR -V

I'm experiencing the same problem on 3.13.0.
NestedScrollView with  headerSliverBuilder no longer scroll. This happens on an actual device and emulator. Downgrading to 3.10.6 fixes the problem.

Hi @ANGEEEL7U7. Thanks for filing this issue. I am able to reproduce this issue but there is a currently open issue describing this case. I'd be closing this in favor of https://github.com/flutter/flutter/issues/104798, kindly upvote, comment and follow up on this issue for updates. If you disagree, please comment and I'd reopen this.
Also, kindly refrain from posting pictures/screenshots if not necessary, copy and paste code samples and output of flutter doctor -v.
Thank you!


Write a README

We need a well-written document that details what the project is, our goals, and (eventually) how to use it. Other background information would be cool, but is not required.

/*
R A N D S T R U C T
By: Jordan Cantrell
    Nikk Forbus
    James Foster
    Connor Kuehl
    Cole Nixon
    Tim Pugh
    Jeff Takahashi

/
/////////////////////////////////////////////////////////////////////////////////////////////
/
Details:
Randstruct is a GCC compiler plugin that was ported from grsecurity to upstream.
- This randomizes the layout of manually/automatically selected C structures
- This makes flaw exploitation less deterministic, requiring significantly more flaws
before an attacker can detect and target the layout of sensitive kernel structures
in memory
Kees Cook, our sponsor, wants this functionality to be made usable in Clang/LLVM.
/
/////////////////////////////////////////////////////////////////////////////////////////////
/
Goals:
Develop full randomization:
- All structures marked with "__randomize_layout" have field positions randomized,
incl. bit fields
Develop a 'performance-sensitive' mode:
- Best-effort limited randomization to cache-line (64-byte) size region, keeping
adjacent bit fields together
Develop an automatic structure selection method (e.g. all functions pointers):
- Disabled with "__no_randomize_layout"
Develop regression tests
- Randomization seed needs to be externally recorded
Ultimate goal to be upstreamed into LLVM and Clang
- Submit work for upstream
- Adjust per feedback
- Rinse and repeat until acceptance in upstream
/
/////////////////////////////////////////////////////////////////////////////////////////////
/
How to use:
1. Use the thing.
2. Good job!
*/


Changes on README

We need changes on README file to better communicate:
What is this project: A django application for managing job postings. Forked from django-job-portal repository
Screenshots
How to run it: How to run it on dev (local) environment. How to run it on production environment.
How to contribute: Link to CONTRIBUTE file. Explaining this is a collaborative effort promoted by GeeksCat association for Hacktoberfest 2020 edition

That would be great !!
I was thinking on contributing, however, by not being a pythonist found no way to run the app locally. I would like to contribute somehow, maybe enabling CI e2e tests via actions.
Anyway, I'm feeling blocked until:

README contains a getting started info
or #63 reaches master so I can run the app via Docker & the info provided
or a stable environment is available remotely

Best 😉

Good to see you here @danifornells !!! :)
I think we do already have the docker compose in place, let me work on the README with some simple instructions for spinning up a dev env

Thank you @victormartingarcia, much appreciated. It's now up & running on my local environment. Will try to collaborate somehow 😘


Consider more intuitive table view scrolling behavior

Currently, the last file is cut off smoothly when the window overflows, thereby making it seem like there are no more files to scroll to. The file explorer should compute a file height such that the last file is cut off, making it clear that there are more files below.

Somewhat related to, but distinct from, #657.

This is neither necessary nor beneficial. You can tell that you can scroll from the scrollbar, and it is awkward and problematic to cut rows off.


Update DoctrineToTypesenseTransformer.php

Only DateTime object is checked.
DateTimeImmutable isn't
It's better to check the interface instead of the class.

More information.
Without this modification, I can't import my immutable datetime property.
The console says: "it should be an int64".
In fact it fallback as a string.


DOCS: Update readme to make interface flows into a table

Are you looking to convert this python codeblock from the readme into a markdown table?
#The following map is used to map datafile external interface names and those on the website
external_tflows_map = {'SCH - HQ - NY': 'HQ CHATEAUGUAY',
                       'SCH - HQ_CEDARS': 'HQ CEDARS',
                       'SCH - HQ_IMPORT_EXPORT': 'SCH - HQ_IMPORT_EXPORT', #subset of HQ Chateauguay…it includes only scheduled import/export flows over Chateauguay, and excludes wheel-through interchange
                       'SCH - NE - NY':  'NPX NEW ENGLAND (NE)',
                       'SCH - NPX_1385': 'NPX 1385 NORTHPORT (NNC)',
                       'SCH - NPX_CSC':  'NPX CROSS SOUND CABLE (CSC)',
                       'SCH - OH - NY':  'IESO',
                       'SCH - PJ - NY':  'PJM KEYSTONE',
                       'SCH - PJM_HTP':  'PJM HUDSON TP',
                       'SCH - PJM_NEPTUNE':'PJM NEPTUNE',
                       'SCH - PJM_VFT': 'PJM LINDEN VFT'}

Or something else?

Yes!


Integration of Koukaam Netio4(all) switches

Hi, here it is again, but correctly rebased and cleaned (at least I hope so..:))
I would still like to add a throttle on the update function, but If I do so, when the switch is turned on or off from the interface, the update function is called and thus, if throttled may return the wrong state. So is there a way to avoid the call to the update function when the state is set ? (I tried returning True or False)

I had a couple comments and so did Travis CI. Also, you'll want to add this module to the omit list in .coveragerc.

Closing this as it has been stale for a month. You can re-open it when you plan on finishing it.

Sure, sorry got wrapped up into something else.
I will rebase cleanly and push again.

On 27 Feb 2016, at 03:01, Paulus Schoutsen<EMAIL_ADDRESS>wrote:
Closing this as it has been stale for a month. You can re-open it when you plan on finishing it.
—
Reply to this email directly or view it on GitHub https://github.com/balloob/home-assistant/pull/1030#issuecomment-189555043.


Add require statement to Readme

Nice to have so one does not have to guess the module name when going along with the Readme.

Looking good, thanks!


How can I set right bouding box?

Thank you for your dataset, but how can I set a right bouding box for training? Is there some additional configuration file？Thank you!

The near and far depth of each scene:
sphere: 2.0 6.0
scalar: 0.4 1.4
double_m: 8.0 20.0
ym: 5.0 9.0
fire: 12.0 24.0
double2c: 6.0 17.0
build: 16.0 34.0
fan: 12.0 20.0
car: 12.0 20.0

Thank you for your answer, but in PINF we have a config for bbox to init nerf/TiNeuVox position, like this:

Will it provide ? Thank you!


Building tensorflow wheel met error clang: error: no input files

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Darwin Kernel Version 18.0.0, Mac OS 10.14
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from:
build from source
TensorFlow version:
Master branch
Python version:
2.7
Installed using virtualenv? pip? conda?:
No
Bazel version (if compiling from source):
0.23.2
GCC/Compiler version (if compiling from source):
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include/c++/4.2.1
CUDA/cuDNN version:
without it
GPU model and memory:
without GPU.

Describe the problem
I want to build tensorflow from source, use command:
bazel build --config=opt //tensorflow/tools/pip_package:build_pip_package

And here is result:
Target //tensorflow/tools/pip_package:build_pip_package up-to-date:
  bazel-bin/tensorflow/tools/pip_package/build_pip_package
INFO: Elapsed time: 5926.109s, Critical Path: 623.43s
INFO: 10670 processes: 10670 local.
INFO: Build completed successfully, 11295 total actions

Then I try to build the wheel, use command:
./bazel-bin/tensorflow/tools/pip_package/build_pip_package /tmp/tensorflow_pkg

But got error:
warning: no files found matching '*.pyd' under directory '*'
warning: no files found matching '*.pd' under directory '*'
warning: no files found matching '*.dll' under directory '*'
warning: no files found matching '*.lib' under directory '*'
warning: no files found matching '*.h' under directory 'tensorflow/include/tensorflow'
warning: no files found matching '*' under directory 'tensorflow/include/Eigen'
warning: no files found matching '*.h' under directory 'tensorflow/include/google'
warning: no files found matching '*' under directory 'tensorflow/include/third_party'
warning: no files found matching '*' under directory 'tensorflow/include/unsupported'
clang: error: no input files
error: command 'clang' failed with exit status 1

@a6802739 Apologies for the delay in response. Is this still an issue? Thanks!

@ymodak, it's fixed now.

That's great to hear. Can you please share your solution? It may help other users coming across this issue in the future. Thanks!


Add support for some more missing settings

What I got merged in https://github.com/ansible/eda-server/pull/1059 was necessary, but didn't process everything that we needed. This should have been obvious to me, but the way I was testing didn't make everything immediately break at the time.
I'm trying to use a new method with settings.exists here, which I think it way way nicer than catching NameError. I haven't tested that it works with env vars (I'm pretty sure it will), but I'll try to post a confirmation of that soon.

if "FOO" in settings:
    FOO = settings.FOO

Suggested by @rochacbruno should work, and might be preferable

@AlanCoding I have confirmed that works with envvars.


Server-side PHP Example

Issue Type (delete the irrelevant ones)

[x] Question
Hello,
Super plugin, congratulations.
However I have a question, would you have an example of sending parameters to php server? Let me explain:
From the documentation here : https://xaksis.github.io/vue-good-table/guide/advanced/remote-workflow.html
I'm using Symfony 4 in back and VueJs in front

Currently we have the code on the client side, but I do not see any call to an url (something like: axios.post for example) to send the data to the server.

Also how must the server be configured (to receive the data and process correctly).
Thank you for your reply

Specs
What version are you using?
VueJS 2.5.4
What browser?
Google Chrome

@AndrianD, I am new to this plugin as well. The plugin only offer you will the data that is ready to be sent to the server side. This is explained under the section What we send to the server.
How you send to the server is left to the developer and is not dependent on a specific plugin.
The 3rd example on this section (https://xaksis.github.io/vue-good-table/guide/advanced/remote-workflow.html#provide-handlers-for-user-events) has the method getItems which invokes a method getFromServer. This is your own implementation of what you send to the server.
This could definitely be abstracted within the plugin so the developer is only responsible of accepting the parameters on the server and provide the response back to the client transformed in the required format. I am hoping we can expect this in future releases. But I hope this answers your question.

@abishekrsrikaanth thank you for your detailed answer! @AndrianD, I believe the above answers your question.
closing.


Rubygems caching with docker

Any examples how i can cache rubygems for specific job?

Check this: https://github.com/ayufan/gitlab-ci-multi-runner#how-to-configure-runner-for-gitlab-ci-integration-tests
By default /cache folder is added and you can bundle install into cache.

@ayufan thanks, looks like i need to register second worker if i want to get matrix builds?

Nope. Just up limits. The same worker will process multiple jobs, each will be in separate container with separate caches.

Look at concurrent in global section and limit in runner's section.

Great, thanks, this is works for me


Zooz temp/humidity sensors stop reporting humidity on 9.16.0

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Z-Wave JS UI version
9.16.0
ZwaveJS version
1.37.0
Describe the bug
My ZSE40 and ZSE44 sensors stopped reporting humidity after updating to 9.16.0 from 9.15.0.
After excluding and re-including the ZSE40 to troubleshoot, the humidity was not even listed as a sensor value in the UI. Only temperature was listed.
Reverting to 9.15.0  resolved the issue. You can see when the upgrade and downgrade took place looking at these graphs.

To Reproduce
Upgrade zwave-js-ui to 9.16.0 with a ZSE44 or ZSE40 sensor.
Expected behavior
Humidity values are reported.
Additional context
No response

I noticed for a ZW100 sensor weird behavior since upgrade 9.15 to 9.16. Temperature in HA gets wrongly interpreted (it shows -5 °C instead of 21 °C). In Zwave-ui-js the value of this device is flapping between °C and °F without clear pattern to me. I'm still investigating, and perhaps file a bug report, but it could be related to this issue. I reverted back to 9.15, so far HA  shows correct 21 °C again.

I can reproduce, fix is coming

Yes indeed, the issue I faced seems to be gone. Many thanks @AlCalzone 👍


chore(main): release 0.9.0

:robot: I have created a release beep boop
0.9.0 (2024-02-01)
Features

access systemconfig from the api (#259) (6b7a045)
allow heartbeat expiration to be configured from db (#253) (26363f5)
changes in ros2 relay configuration (#264) (2deddd1)
heartbeat colour coding the netapp status heartbeat responses (#266) (789fa5a)
heartbeat data storage implemented with influxdb (#263) (29c46d4)

Bug Fixes

incorrect check if policy exists (ab2247b)
missing function names in RedisInterfaceClient logging (01141a6)
task response when requesting a plan (1cef9c0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/5G-ERA/middleware/releases/tag/v0.9.0 :sunflower:


Synapse Linked Service Creation Error

The 2021-06-01-preview for Synapse Linked Service returns response code 200 during create/update though the resource actually fails due to a bad payload. You can test this by passing in a bad type into the payload:
{
	"properties": {
		"type": "LinkedService",
		"typeProperties": {
			"connectionString": "someconnectionstring"
		}
	}
}

The API will report this as 200 even though it's a bad request and should return 400

@mbfrahry We are looking into it and get back to you for any additional information.


Add a method to specify phantomjs path

I'm using spiderling in conjunction with jakoch/phantomjs-installer so I have phamtomjs binary somewhere in my project (say _PRJ_/bin/ or _PRJ_/vendor/bin/).
As it would be nice not having to put project's dir into the PATH it would be a great thing to have the possibility of programmatically specify the path to the binary (now it is hardcoded to "phantomjs" in the command() method)
As putting . in the PATH is not a secure option, symlinking it in the current dir is not an option too ..
What do you think? Adding a static method making the user able to specify the bin full path is IMHO the right approach ..
Hope you can give a solution ASAP :) Thank you in advance

A solution for me could be to add my bin dir to path programmatically as follow:
putenv('PATH="%s:%s"', __DIR__.'/vendor/bin', getenv('PATH'));

Evaluate by yourself if an explicit path setter could be a feature or not ..

I read on another ticket that you might be looking at this issue - is that right? A :+1: from me also, though I am happy with my inline PATH amendment for now.


Syntax errors include line number in message

This is just a minor cosmetic issue, but since there is ongoing work on error messages for 1.4.0 already, I figured now would be a good time to address it.  A syntax error currently includes a line number as part of the error text:
SyntaxError: empty expression not allowed (line 30)

Since the SyntaxError object already includes the relevant filename and line number in its trace data or directly as own properties if those are disabled, this seems redundant.

On the other hand sometimes people will just print a string coerced version of the error which only includes the name and the message (.fileName and .lineNumber are non-standard):
((o) Duktape 1.3.99 (v1.3.0-301-g9a26d9d)
duk> try { eval('foo=') } catch (e) { print(e); }
SyntaxError: parse error (line 1)

This is one reason why syntax error line and JSON error offset were added to the message, so that they'd be trivial to spot:
duk> JSON.parse('{"foo" "bar"}')
SyntaxError: invalid json (at offset 8)
	duk_bi_json.c:213
	parse  native strict preventsyield
	global input:1 preventsyield

These have both been requested explicitly in the past, to be contained directly within the error message (one liner).
I guess one could make the same argument for other kinds of errors, so current behavior is not maybe very consistent. Going more into that direction, it'd be relatively simple to either:

add the file/line to the .message property directly; or
add the file/line to the .toString() coercion of an error (but keep .message intact)

Both are non-compliant to some extent however:

Modifying the message given to e.g. new Error() before storing it as .message is non-compliant. So one could only automatically add file/line to errors thrown from within Duktape.
The .toString() algorithm is specified explicitly in terms of combining .name and .message only.

The current behavior is compliant because Duktape can freely decide the messages of its own errors (syntax errors in this case).

For what it's worth I think having the line number in JSON errors does make sense.  For code though it's a little jarring since runtime errors don't include it.
On the other hand having the line number for syntax errors in eval() code does make sense, so... I guess keep the current behavior?
Whatever the case I don't think line numbers should be added to other errors.  The properties are non-standard but available in most extant engines, so it's reasonable I think for a user to treat them as a given.

Yeah, I'm not sure what the best behavior would be - there are competing concerns. Just quickly checking other engines for SyntaxError behavior:
V8 doesn't seem to blame syntaxerror on the source code (maybe I'm doing something wrong?):
> try { eval('\n\nfoo='); } catch (e) { console.log(e); console.log(e.fileName, e.lineNumber); console.log(e.stack); }
[SyntaxError: Unexpected end of input]
undefined undefined
SyntaxError: Unexpected end of input
    at repl:1:7
    at REPLServer.defaultEval (repl.js:132:27)
    at bound (domain.js:254:14)
    at REPLServer.runBound [as eval] (domain.js:267:12)
    at REPLServer.<anonymous> (repl.js:279:12)
    at REPLServer.emit (events.js:107:17)
    at REPLServer.Interface._onLine (readline.js:214:10)
    at REPLServer.Interface._line (readline.js:553:8)
    at REPLServer.Interface._ttyWrite (readline.js:830:14)
    at ReadStream.onkeypress (readline.js:109:10)

Rhino uses an interesting .fileName (no .stack):
js> try { eval('\n\nfoo='); } catch (e) { print(e); print(e.fileName, e.lineNumber); print(e.stack); }
SyntaxError: Unexpected end of file
<stdin>#2(eval) 3

I guess I'm inclined to keep the current behavior until a clearly better approach is evident :)

JSON.parse() syntax error on V8:
> try { JSON.parse('{ "foo": 123') } catch (e) { console.log(e); console.log(e.fileName, e.lineNumber); console.log(e.stack); }
[SyntaxError: Unexpected end of input]
undefined undefined
SyntaxError: Unexpected end of input
    at Object.parse (native)
    at repl:1:12
    at REPLServer.defaultEval (repl.js:132:27)
    at bound (domain.js:254:14)
    at REPLServer.runBound [as eval] (domain.js:267:12)
    at REPLServer.<anonymous> (repl.js:279:12)
    at REPLServer.emit (events.js:107:17)
    at REPLServer.Interface._onLine (readline.js:214:10)
    at REPLServer.Interface._line (readline.js:553:8)
    at REPLServer.Interface._ttyWrite (readline.js:830:14)

Rhino:
js> try { JSON.parse('{ "foo": 123') } catch (e) { print(e); print(e.fileName, e.lineNumber); print(e.stack); }
SyntaxError: Unterminated object literal
 0

By the way I found a bug related to this - when syntax errors are rethrown (I think...) the line number is stripped.  I discovered this by deliberately causing an error in an eval() and letting the debugger intercept it - the Throw notify is missing the line number in the message.

I think what happens is that the Throw notify shows the Error before the line number is added to the message. Right now that happens in the compiler: it catches the error (after it's initially thrown), appends the line number, and rethrows it.
Now that the error augmentation process understands syntax errors, I could move that into the augmentation code, so it'd happen before the throw notify. It would also be simpler.

Ok, so #462 should have that change and you should see the modified message in a Throw notify.

Somewhat related from a code-structure standpoint: Compilation errors can be rethrown multiple times on their way out of the compiler, leading to sometimes 3-4 identical "fatal" Throw notifys in a row.  How hard would that be to fix so that only the last one halts execution?

I guess I see catching and rethrowing an ordinary part of error handling (like we discussed re: the Throw notify) so it's very difficult for the Throw notify to provide a meaningful indication of whether an error will be ultimately caught or not - what does that mean if there are e.g. conditional rethrows along the catch stack?
One would need to classify all catch sites, both C and Ecmascript, as either truly catching the error or just "rethrowing" the error (or a related error). Ultimately that information would need to be given by the code setting up the catch point because it depends on what the catching code really does. For C code it'd be possible to provide such a flag when doing a protected call, but Ecmascript catchers would need some custom syntax which seems like an overkill.
Given all that, I think trying to fix any one case might be possible but the problem will never go away because the "will not be caught permanently" (or something similar) information is simply not always available.
But regading the compilation errors: eliminating all rethrow sites would be fixable by moving all the stuff those rethrow sites do either into the original throw site or into the shared error throw handling. Both would at least initially seem awkward from a modularity standpoint. I'm also not very excited about reducing rethrows here and there because they're useful for modularity and the process will never be complete because the problem is unsolvable :-)
Another approach entirely would be to avoid trying to make that "will this be caught" judgement call, and instead provide more information about the active catchers to the debug client so that it could make that call with whatever heuristics are useful. There isn't always much information about a catch site though if it's in C code, so I'm not sure how well that would work.

The root of the problem in this case is that protected call sites don't count as a catch clause, which is the criteria used to determine whether the error is fatal.
I didn't mean to suggest restructuring the whole compiler to minimize rethrows, that would be insane.  And I certainly appreciate modularity. :)  A good general-purpose fix I think would be to add a mechanism where certain protected sections could be marked as "catching" so that the debugger doesn't intercept them.  You'd still get 3-4 Throw notifys this way, but only the final one, the one that propagates out of the compiler, would cause execution to stop,

To clarify, the complaint in this case isn't about the duplicate notifys - the same would be true even in Ecmascript code with catch-and-rethrow logic.  I'm fine with that.  The issue is that, unlike with an Ecmascript catch, the compiler-internal catches aren't seen by the Throw notify check and so it is sometimes necessary to send several Resume commands to power through all the rethrows.
As mentioned in a code comment, the fact that protected calls are ignored by the catch check is usually desirable, but in certain cases (like here), it would be useful to optionally be able to make a protected call which is considered "catching".

Ah ok, so the problem is that execution pauses on a supposedly uncaught error multiple times.
We could add an internal (?) duk_rethrow() which would indicate that you're conceptually rethrowing a caught exception and such a rethrow shouldn't be caught. That would make the error propagate to an actual error site without throw notify / pause handling.

Oops, "shouldn't be caught" above should be "shouldn't cause pausing".

That would probably work.  We could also use that when rethrowing an error at the end of a finally clause, to prevent a second pause on the same (uncaught) error.


chore: optimized images

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #5105

Details of Changes
Optimized images with tinypng.

@Zerotask Does this affect the quality or sharpness in any way?

@yaichenbaum No, the quality is the same.

@Zerotask Can you solve the build issue?

@yaichenbaum I don't think I caused the problem. Can you check that?

The problem is because certain icon files need to match exactly, it looks like those files were compressed differently.

What exactly has to match and why? I did not change the width or height and if 2x the same images will be copied to the same destination, as the pipeline error says, this looks like a mistake to me.

I guess icons in Files/Assets should match icons in Files.Package?

I guess icons in Files/Assets should match icons in Files.Package?

But why 2x the same images and what is the criteria? The file size? Looks pretty strange for me such a behaviour. I could understand it, if it checks width and height, but I didn't change that.

I guess icons in Files/Assets should match icons in Files.Package?

But why 2x the same images and what is the criteria? The file size? Looks pretty strange for me such a behaviour. I could understand it, if it checks width and height, but I didn't change that.

The packaging behavior is very specific, it's possible they need the same hash.

@Zerotask Yes, it will fix this issue. Just update all icons in FIles.Package/Assets from Files.Assets.

Okay, but this is pretty strange to me. The error also indicates that the root of the problem is something else.
I recommend having a centralized folder for images instead of duplicate folders.
Optimizing images shouldn't really cause such problems.

@Zerotask Yes, it will fix this issue. Just update all icons in FIles.Package/Assets from Files/Assets.

I copied the smaller amount of images from FIles.Package/Assets to Files/Assets. Hope this will fix the CI/CD problem.

@Zerotask Thanks for the PR, would you happen to know how much space was saved with these optimizations?

@yaichenbaum approx. 6,55 MB saved.


Discussion: Bump the minimal required PHP version?

Currently, this library's PHP version requirements align to those of the WordPress project (i.e., PHP >= 5.6 as a hard requirement according to https://wordpress.org/support/article/requirements/).
I've done some googling around and found contradicting information on when the PHP version requirements for WordPress will change, and which PHP version will then be required. @westonruter Do you have more (or official) information on this which you could share?

There's an in-depth proposal for dropping support for old PHP versions via a fixed schedule: https://make.wordpress.org/core/2020/08/24/proposal-dropping-support-for-old-php-versions-via-a-fixed-schedule/
Matt Mullenweg's response to the proposal can be seen here: https://make.wordpress.org/core/2020/08/24/proposal-dropping-support-for-old-php-versions-via-a-fixed-schedule/#comment-39736
In short:

The right policy for WordPress core is to have users drive what we support, measured by usage, not other projects with their own priorities and stakeholders.

It seems that once usage for PHP 5.x goes below 5% that this will then allow for WordPress to bump the minimum PHP requirement to PHP 7.
At the moment, PHP 5.6 accounts for 9.2% of WordPress installs: https://wordpress.org/about/stats/

Version
Usage

8.0
0.50%

7.4
30.80%

7.3
29.08%

7.2
16.42%

7.1
4.17%

7.0
5.53%

5.6
9.20%

5.5
0.96%

5.4
1.43%

5.3
1.35%

5.2
0.56%

So we're waiting on hosts to bump the PHP versions.
There are 93 plugins in the WP directory that use PHP-CSS-Parser: https://wpdirectory.net/search/01F79GKCFAQRTSWMTQ07H7KVQS
I checked the ones with 20,000+ active installs and there's a split between the ones that require PHP 5 vs PHP 7:

Plugin
Requires PHP

AMP
5.6

Admin Menu Editor
5.x

WooCommerce PDF Invoices & Packing Slips
7.1

MailPoet – emails and newsletters in WordPress
7.1

Advanced Contact form 7 DB
n/a

Jupiter X Core
5.6

wpDataTables – Tables & Table Charts
5.6

Ultimate FAQ – WordPress FAQ Plugin
n/a

WordPress Email Template Designer – WP HTML Mail
7.2

Web Stories
5.6

WordPress Real Cookie Banner
7.2

WooCommerce PDF Invoices, Packing Slips, Delivery Notes & Shipping Labels
5.6

Done in #420. Closing.