[Refactor] CombinedStatus and CommitStatus related functions & structs

as title

add limit for pagination on repoGetCombinedStatusByRef
fix swagger docs

Codecov Report

Merging #14026 (020f7fd) into master (069acf6) will decrease coverage by 0.03%.
The diff coverage is 58.00%.

@@            Coverage Diff             @@
##           master   #14026      +/-   ##
==========================================
- Coverage   42.23%   42.19%   -0.04%
==========================================
  Files         710      711       +1
  Lines       77281    77289       +8
==========================================
- Hits        32639    32615      -24
- Misses      39279    39299      +20
- Partials     5363     5375      +12

Impacted Files
Coverage Δ

modules/convert/convert.go
74.26% <ø> (-1.44%)
:arrow_down:

routers/api/v1/repo/status.go
48.64% <0.00%> (+5.27%)
:arrow_up:

routers/repo/blame.go
0.00% <0.00%> (ø)

routers/repo/commit.go
29.50% <0.00%> (ø)

services/pull/commit_status.go
4.47% <0.00%> (ø)

modules/convert/status.go
50.00% <50.00%> (ø)

routers/repo/pull.go
31.77% <50.00%> (ø)

models/commit_status.go
62.83% <100.00%> (+0.76%)
:arrow_up:

modules/gitgraph/graph_models.go
84.68% <100.00%> (ø)

routers/repo/view.go
38.11% <100.00%> (ø)

... and 16 more

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 069acf6...020f7fd. Read the comment docs.

I know this pull diff is not clear in the first place, just because it refactor a lot ...
But I tryed to split up tasks I did in seperate commits and name them as they do ...
hope this will help reviewing this

@lunny done - good idear "status" is to general


Implement ability to not attempt to coerce dynamic types to dates on serialize

Added a new option to prevent conversion to dates within dynamic objects during serialization

Delta looks so big as I changed a bunch of whitespace - can revert if required.


Error when compiling

I'm unable to build libnotcurses-sys because I'm getting the following errors:
$ cargo run
   Compiling libnotcurses-sys v3.9.0
error[E0432]: unresolved imports `crate::c_api::ffi::__syscall_slong_t`, `crate::c_api::ffi::__time_t`
 --> /Users/USERNAME/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libnotcurses-sys-3.9.0/src/time.rs:3:25
  |
3 | use crate::c_api::ffi::{__syscall_slong_t, __time_t, timespec};
  |                         ^^^^^^^^^^^^^^^^^  ^^^^^^^^
  |                         |                  |
  |                         |                  no `__time_t` in `c_api::ffi`
  |                         |                  help: a similar name exists in the module: `time_t`
  |                         no `__syscall_slong_t` in `c_api::ffi`

error[E0412]: cannot find type `mbstate_t` in this scope
    --> /Users/USERNAME/gitrepos/notcurses-c/test/notcurses-test/target/debug/build/libnotcurses-sys-d1fbad7168c446ec/out/bindings.rs:1542:16
     |
433  | pub union __mbstate_t {
     | --------------------- similarly named union `__mbstate_t` defined here
...
1542 |     arg5: *mut mbstate_t,
     |                ^^^^^^^^^ help: a union with a similar name exists: `__mbstate_t`

error[E0412]: cannot find type `mode_t` in this scope
    --> /Users/USERNAME/gitrepos/notcurses-c/test/notcurses-test/target/debug/build/libnotcurses-sys-d1fbad7168c446ec/out/bindings.rs:2770:37
     |
2770 |   pub fn devname(arg1: dev_t, arg2: mode_t) -> *mut core::ffi::c_char;
     |                                     ^^^^^^ not found in this scope
     |
help: consider importing one of these items
    -->  /Users/USERNAME/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libnotcurses-sys-3.9.0/src/bindings.rs:14:5
     |
14   |     use crate::c_api::libc::mode_t;
    -->  |/Users/USERNAME/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libnotcurses-sys-3.9.0/src/bindings.rs:14:5
     |
14   |     use libc::mode_t;
    -->  |/Users/USERNAME/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libnotcurses-sys-3.9.0/src/bindings.rs:14:5
     |
14   |     use std::os::macos::raw::mode_t;
     |

error[E0412]: cannot find type `mode_t` in this scope
    --> /Users/USERNAME/gitrepos/notcurses-c/test/notcurses-test/target/debug/build/libnotcurses-sys-d1fbad7168c446ec/out/bindings.rs:2775:11
     |
2775 |     arg2: mode_t,
     |           ^^^^^^ not found in this scope
     |
help: consider importing one of these items
    -->  /Users/USERNAME/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libnotcurses-sys-3.9.0/src/bindings.rs:14:5
     |
14   |     use crate::c_api::libc::mode_t;
    -->  |/Users/USERNAME/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libnotcurses-sys-3.9.0/src/bindings.rs:14:5
     |
14   |     use libc::mode_t;
    -->  |/Users/USERNAME/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libnotcurses-sys-3.9.0/src/bindings.rs:14:5
     |
14   |     use std::os::macos::raw::mode_t;
     |

Some errors have detailed explanations: E0412, E0432.
For more information about an error, try `rustc --explain E0412`.
The following warnings were emitted during compilation:

warning: Keeping vendored files…

error: could not compile `libnotcurses-sys` (lib) due to 4 previous errors

I've built notcurses using homebrew, and I'm using the latest version of rust. Is there something else I need to do?

Hi. Thanks for the report.

What's the output of uname -a?
What's the output of notcurses-info?
Can you try to compile it with --no-default-features and see if it throws the same errors?

fwiw, mbstate_t comes from wchar.h (uchar.h with c++11+), and mode_t comes from sys/types.h.

Oh, also, could you please try compiling the v3.8.0 release? I made some changes to the bindings after that version, just to make sure it has nothing to do with that.

This should be now fixed on the latest release v3.9.1.

I still get just one error btw:
warning: Deleting vendored files…
error[E0432]: unresolved imports `crate::c_api::ffi::__syscall_slong_t`, `crate::c_api::ffi::__time_t`
 --> src/time.rs:6:25
  |
6 | use crate::c_api::ffi::{__syscall_slong_t as c_long, __time_t as time_t};
  |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^  --------^^^^^^^^^^
  |                         |                            |
  |                         |                            no `__time_t` in `c_api::ffi`
  |                         |                            help: a similar name exists in the module: `time_t`
  |                         no `__syscall_slong_t` in `c_api::ffi`

For more information about this error, try `rustc --explain E0432`.
The following warnings were emitted during compilation:

warning: Deleting vendored files…

error: could not compile `libnotcurses-sys` (lib) due to previous error

Oops I was running it with cargo build --no-default-features and getting that error. If I don't use the --no-default-features flag I get no errors!

Presumably notcurses-rs also needs an upgrade?
The Cargo.toml still points to v3.9.0:
https://github.com/dankamongmen/notcurses-rs/blob/e7538418bde1eb2210563d2f01e65a0819259405/Cargo.toml#L33


the booster-task-analyser link is 404 - page not found when click

in Performance list the booster-task-analyser link is 404 - page not found when click

No longer supported


Additional date range presets

Description
Adds several new presets to the defaults included in the date range and a separator between groups.

@ItsMeBrianD what's going on w/ the netlify action?

I think we might have to specify archie's forked action as @v1.0.2 instead of @v1? Otherwise the output doesn't make any sense to me


Benchmark suite with LB for single-threaded servers

Based on @brunoborges idea in https://github.com/LesnyRumcajs/grpc_bench/pull/76
Create a benchmark suite (ideally using the existing images) that would provide single-threaded implementations with a LB (to be chosen) so that the we can compare e.g. rust_tonic_st x 2 + LB against single dotnet appliance, both running on e.g. 3 cores.

Initially my comment was that the total amount of CPU/RAM for Single-Threaded Workload X plus LB should equal Multi-Threaded Workload Y minus LB.
But in reality, any cloud-based deployment would have a LB anyways.
So the rule should probably be that the number of single-threaded instances should equal to # of CPUs given to one multi-threaded instance.
Benchmark results should only show (published) comparisons of 2 or more CPUs.
The use of a single CPU should be limited to CICD validation.

And regarding duration, anything below 10min IMO is not realistic enough for fair comparisons between languages/runtimes, because some have JIT compilers.

@brunoborges I agree, it makes sense to split those. We even added _mt and _st suffix, but it should be kind of a different category.

IMO nowadays in most cases load balancers come as an infrastructure, so I'm not sure N * ST + LB benchmarks will add much value...
Either you build a big service running on multiple cores, or you build a micro service targeting 1 CPU core, you can make your choice using the results we have at the moment...


Timeline: Boxes Not Aligned For 3rd Party Mods

I'm posting and posting to the timeline. Everything is aligned (thumbnail & post) when I use text, link, video, audio, sound.
However, when a post using a 3rd party module appears in the timeline, the thumbnail & post are misaligned (image below). Happens with Ultimate Articles, PKForum & all the mods I have.
Not sure how to fix.

https://www.boonex.com/forums/topic/Timeline-Boxes-Not-Aligned-For-3rd-Party-Mods.htm


Initial work to support connection to a local "region"

Addresses #199
Usage:
let provider = MockProvider { };
let mut http_client = hyper::Client::with_connector(hyper::net::HttpConnector::default());
http_client.set_redirect_policy(hyper::client::RedirectPolicy::FollowNone);
let region = Region::Local("localhost:8000");
let client = DynamoDbClient::new(http_client, provider, region);

I'm happy to do stuff like write tests for this if the direction seems good.

Looks like rustfmt had a breaking change in a non-breaking release that's breaking the Travis build here. 😕
Taking a look at how we can fix that.

I'm not super familiar with other APIs, actually being new to using DynamoDB myself, so unfortunately, I can't give much direction there. Basically, this works for me and seemed not incredibly hard to implement. I'll try to take a look at making that string not static.

I'm not sure where that string needs to be used later. You could consider using an 'Into<&'a str>' or an AsRef so that the API can take a str or String.

@insanitybit Using a conversion trait would mean making Region generic:
pub enum Region<T: AsRef<str>> {
    ...,
    Local(T)
}

That would mean that a Region<String> and a Region<&'a str> would not be the same type, and so everything using Region would also need to be generic over T the same way.
Cow<'a, str> is an option as well, but there is no advantage (and some disadvantages) to using it over an &'a str here.

Looking forward to getting this in!  Anything I can help with?

I think we should take a step back given that we're now talking about making Region, generic and question whether this is the right approach.  I think Region should remain a very primitive enum corresponding to Amazon's actual service regions, with some display-related traits for convenience.
I don't think creating a Local region is the right way to approach this. I'm pretty sure Dynamo DB is the only AWS service where a non-production endpoint can be meaningfully configured.  The java SDK provides a withEndpointConfiguration() constructor for Dynamo, and I think we should take that approach as well.

given that we're now talking about making Region generic

I was just giving that as an example of why we shouldn't use conversion traits in the enum. I'm definitely not advocating for making it generic (at least in type -- generic in lifetime is fine)! Just a String is good enough for me.
I'm good with just merging this once it take a String. We already have to re-think regions in general because not all regions support all services (see #708) and we'll have to revisit this once we want to tackle that. For now though, this solves a real issue without needing to rethink the entire Region setup. 😄

Hopefully I won't step on any toes, but I'd like to finish up this PR with the feedback requested in the next day or three.  I'd love to get this into the next release of Rusoto.

I have no problems with someone taking this over as long as it supports local services :)

I've modified this PR over in https://github.com/rusoto/rusoto/pull/759 to address some of the review comments.  😄

Closing this in favor of https://github.com/rusoto/rusoto/pull/759 which looks pretty close. 👍 Thanks for everyone's work on this!


Display popup with "logseq.provideUI()" instead of rendering

This is just one suggestion.  It always takes up space in my diary, so I thought I could do something more flexible by avoiding it. I have experience with other plugins, so I can help if needed.
"logseq.provideUI()" https://plugins-doc.logseq.com/logseq/provideUI

Heya sorry I've missed this entirely for many months, Logseq has kinda fallen out of use for me, but I'd be happy to implement this.
But I'm not entirely sure what the suggestion is.  I think I'm already calling logseq.provideUI to display the final result, though I could be wrong. https://github.com/matteron/logseq-ac-keijiban/blob/main/index.js

Please refer to this entry.
https://github.com/logseq/logseq/pull/3045


Documentation: Azure Link for Enterprise

https://docs.influxdata.com/enterprise_influxdb/v1.8/install-and-deploy/production_installation/
What products and version are you using? 1.8
Where did you look before opening the issue?

Thank you, @jrob2309! Resolved in #2820


Cleanup tf lock files

Terraform lock files get generated when you run terraform init so it shall not be included in source control.
https://www.terraform.io/docs/language/dependency-lock.html#lock-file-location

Thanks @felipecruz91 ! I will make a change to .gitignore as well

Thanks @felipecruz91 ! I will make a change to .gitignore as well


Introduce [legacy] branch

This branch contains the legacy layer version 1 implementation of meta-ros
that provides ROS Indigo Igloo in an OpenEmbedded Linux system. It is no longer
under active development, but will be retained until it is no longer needed by
the community.

How about just calling the branch indigo?

I assume even with superflore, you will need multiple branches for the different releases, right?

I assume even with superflore, you will need multiple branches for the different releases, right?

No -- see the first bullet item under https://github.com/ros/meta-ros/wiki/Superflore-OE-Recipe-Generation-Scheme#general .


Improve photo layout performance

While I have been working on some other PRs I noticed that some bottle necks in the layout routine for albums:

When we rearrange the photos of the layout, the layout should be hidden such that each relayouted child triggers a re-drawing
Stop querying the DOM for CSS selectors several times
When the DOM has been manipulated (i.e. the photos have been added), we must provide the engine a time slice to catch up with the new DOM otherwise querying CSS attributes is slow (--> make justify asynchronous, wrap it into setTimeout).

I had to revert one change in https://github.com/LycheeOrg/Lychee-front/pull/334/commits/c6bcfac33ae9cf0e159874c8a1a89d281a92849d. I also added a (long) comment what the problem is so that I (or someone else) remembers and understands what the root cause is.


Bug: Is not possible to search a second time

	<input type="text" name="schools" id="autocomplete" data-search="textinput" placeholder="hier suchen" />

I found the bug. The problem is a mix of requirements. You must define data-search="box", data-search="textinput" and id="autocomplete". Please clean up the code

fixed in 14880dc8228a5f9a2d4fa90e1d5ca0690ef0fc1e


Error while trying example

Hello!
Trying your example on Ubuntu 17.04 i386. gcc and g++ 6.3.0.
But after completing five steps I get this message:
root@ubuntu:/home/max/drAFL/build# ../afl/afl-fuzz -m none -i in -o out -- ./afl_test @@
afl-fuzz 2.52b by<EMAIL_ADDRESS>[+] You have 1 CPU core and 3 runnable tasks (utilization: 300%).
[] Checking core_pattern...
[] Setting up output directories...
[+] Output directory exists but deemed OK to reuse.
[] Deleting old session data...
[+] Output dir cleanup successful.
[] Scanning 'in'...
[+] No auto-generated dictionary tokens to reuse.
[] Creating hard links for all input files...
[] Validating target binary...
[*] Attempting dry run with 'id:000000,orig:seed'...
[-] PROGRAM ABORT : No instrumentation detected
Location : perform_dry_run(), afl-fuzz.c:2860
Any ideas what's wrong?
Thank you in advance.

do you have AFL_SKIP_BIN_CHECK=1 in your env variables?

Of course, step five requires it.

weird, anyway I currently support this project: https://github.com/mxmssh/manul (drAFL is legacy now)

you can fuzz blackbox binaries with Manul too


.

https://raw.githubusercontent.com/EXALAB/AnLinux-Resources/master/Scripts/Installer/Ubuntu/ubuntu.sh
^

Visual studio code
Sorry, you have internet access at the moment and I download the manual, but you still error code 00x90689
Not 404 the visual studio code

Error
Error sorry I don't have Microsoft teams and system error you system is the rest of your
Try again

Google classroom of the elite controller series x roblox Open you what time you to fix the vscode
exec consolerun /storage/emulated/0 python /data/data/com.alif.ide.python/files/alif/etc/samples/main.py
exec consolerun /storage/emulated/0 python /storage/emulated/0/Download/passwordwifi.pu.py
exec consolerun /storage/emulated/0 python
exec consolerun /storage/emulated/0 python /storage/emulated/0/Download/passwordwifi.pu.py
exec consolerun /storage/emulated/0 python

Vscode mobile
Install termux on
pkg update y-

Contributing translations
Translation Guidelines
Please stick to the intended meaning of the English text as much as possible.
Where a direct translation is not possible, use the text that makes the most sense for native readers of the language.
If you are in any doubt about a translation, please use the comments feature on PoEditor or send an email. Don't just copy the English.
Please use special characters (“…” rather than "...").
Any incomplete, doubtful or uncertain translations must be marked fuzzy.
Moderation
All strings are audited before release. This is to prevent vandalism. Anyone defacing the translations will be blocked from the project. Repeated inaccurate translations are also grounds for removal.
Applying to translate
Translations are hosted on PoEditor at https://poeditor.com/join/project/XE92nUJKqN.
Please request to join then send me an email so that I can approve you.


ImportError

Could anyone please fix the following import error? Thanks a lot.
ImportError: cannot import name 'apply_flat_fun' from 'jax.api_util' (/usr/local/lib/python3.7/dist-packages/jax/api_util.py)

This means you're importing a name that doesn't exist. apply_flat_fun is a private utility that, prior to JAX version 0.2.21, was available in jax.api_util. Commit https://github.com/google/jax/commit/e869e5e0f87713da57b94f81a547685429b29a55 moved it to jax._src.api_util.
Can you say more about what code you ran that led to this error?

What version of optax do you have? Is it the latest?

Thank you for your prompt reply. It looks like the version that I have is optax-0.1.0. So how to fix it? Thanks a lot.

I verified that this doesn't reproduce with optax 0.1.1 and current jax. Closing because it doesn't reproduce with current verssions (i.e., it's fixed).


publish latest version with cli too please

Can you please publish the latest version with the cli tool to rubygems? Thanks!

I will publish a new version to npm very soon. Thanks for your interest!


Hide regression line when source series is hidden

Is there a way that when a source series is hidden (by clicking on the series in the legend), that the regression line can be hidden?

This can be easily achieved in the standard highcharts options (assuming hideInLegend: true)
plotOptions: { 	series: { 		events: { 			//hide both the dots and lines 			legendItemClick: function () { 					var seriesArray = this.chart.series; 					if (seriesArray[0].visible) { 						seriesArray[0].hide(); 						seriesArray[1].hide(); 					} else { 						seriesArray[0].show(); 						seriesArray[1].show(); 					} 					return false; 			} 		} 	} }


Add NBT Library Dependency

Likely this one: https://github.com/tr7zw/Item-NBT-API
Will make it so I have to parse less strings to get the info I need

Unnecessary because we can use the PersistantDataHolder setup on 1.16+.


add license?

Great project!
I'd be hesitant to use it, as there is no license in the project. (i think, maybe i just can't find it 😅) Maybe add a license.txt?
I am interpreting the readme as pretty much equivalent to CC0, but I think it would be better to clearly state your intent with an appropriate license.
You probably know the creative commons license picker, in case you need some inspiration ;)
thanks!

Great idea! I've added the CC0 license :ok_hand:


Prepare 2023.11.0 release

Bump version to 2023.11.0
Pin to dask 2023.11.0

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (2f068f9) 85.55% compared to head (5d53518) 85.08%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1270      +/-   ##
==========================================
- Coverage   85.55%   85.08%   -0.47%
==========================================
  Files          77       77
  Lines        4257     4257
  Branches      758      758
==========================================
- Hits         3642     3622      -20
- Misses        446      461      +15
- Partials      169      174       +5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Review by Maxiausder4a

Content of the Repository:
The Repositorys includes the homework assignment of the first homework of the course Open Source Energy System Modeling, It contains the necessary files: LICENSE, README, code.py and test_code.py.  The Github Actions contain the linter Ruff and pytest.yaml. Moreover, it contains an approved open source license (Apache License 2.0).

The Pull Request Update README did not activate the Github Action pytest.yaml, and is now awaiting approval from the maintainer.

The Pull Request Update README did not activate the Linter ruff.yaml , and is now awaiting approval from the maintainer.

The Pull Request Testint the Linter Ruff by implementing Indentation Errors in code.py did not activate the the workflows pytest, and ruff, since it is awaiting approval by the maintainer.

=> It seems, that testing the workflows is not possible for me, since they are awainting approval from the maintainer to start testing.

Please review @danielhuppmann

After some time, all tests have failed as expected.

Correct, a repository-admin (in this case me) had to approve the GitHub Actions before they are started.


Support el/7

I'm not sure if I'm using the tool wrong. It fails when I try to upload an RPM file.
% packagecloud push tyk/tyk-gateway-unstable/el/7 tyk-gateway_4.0.0-rc21_linux_x86_64.rpm
pushing 1 files...
2022/01/21 18:58:15 push: tyk-gateway_4.0.0-rc21_linux_x86_64.rpm
2022/01/21 18:58:15 tyk-gateway_4.0.0-rc21_linux_x86_64.rpm rpc error: code = InvalidArgument desc = unknown distribution: el/7

However, the gem works.
% package_cloud push tyk/tyk-gateway-unstable/el/7 tyk-gateway_4.0.0-rc21_linux_x86_64.rpm
Using https://packagecloud.io with token:******50c1
/var/lib/gems/2.7.0/gems/json_pure-1.8.1/lib/json/common.rb:155: warning: Using the last argument as keyword parameters is deprecated
Looking for repository at tyk/tyk-gateway-unstable... /var/lib/gems/2.7.0/gems/json_pure-1.8.1/lib/json/common.rb:155: warning: Using the last argument as keyword parameters is deprecated
success!
Pushing tyk-gateway_4.0.0-rc21_linux_x86_64.rpm... success!

I have added support for RPM in v0.2.0.
I have not personally tested it with RPM, so please try it out.


Editor doesn't detect typings in "typings/globals"

VSCode Version: 1.3.1
OS Version: Windows 10, 64bit

Steps to Reproduce:

Create a regular typescript project with the files provided below.
Remove the "*.txt" file endings to make them json files again (github doesn't allow *.json endings as attachments, sorry)
Run "npm install"
Run "typings install"
Create a *.ts file that imports the 'react' module
Look at the typescript error you are getting from the editor.

In short: any typings file that ends up in "typings/globals" will NOT be detected by the editor. However, when running through the typscript compiler, e.g. via webpack, everything's fine. It's only the editor, which is really annoying.
tsconfig.json.txt
typings.json.txt
package.json.txt

@MartinHaeusler the problem is that most of the properties in the tsconfig.json file are not supported by the 'native' tsc compiler. I tried your example and compiling it with the native tsc compiler using the current version fails as well.
I will move the item to the TS team to comment on the future of these properties in the tsconfig.json and how they will be supported by the tsserver (which VSCode uses for its smartness).

Since you are using a special version of TS (the unofficial 2.0.0 version) you need to let VS Code know about this as well. You can do so by using the following setting:
	"typescript.tsdk": "./node_modules/typescript/lib"

This makes a simple ts file work for me:

@dbaeumer: Yes we are using typescript 2.0.0. I was wondering how to tell vscode which version to use but I could not find it anywhere in the documentation. I will try this as soon as I get back into the office, thanks a lot!

adding "typescript.tsdk": "./node_modules/typescript/lib" in .vscode/settings.json works :+1:


add overloads for custom floats

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 8 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 98.621%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
5547

Relevant Lines:
5547

💛  - Coveralls


User kill app, notification not remove

Hi i use this plugin in Android app now. When user or OS kill app notification does not disappear. Please help to solve this.

Some update?

This is valid at least on Android 10+.
When there is music playing by the Ionic app, the Music Controller won't be closed when killing the app.
However, when you PAUSE the music first (actually when issuing an updateIsPlaying(false) and then kill the app, the Controller is destroyed.
Also when issuing the destroy() command from the code, while the isPlaying setting is true, the Music Controller does not get dismissed.  It works however, when first explicitly setting updateIsPlaying(false)...
When a .destroy() is called, it shouldn't matter if isPlaying == true...

Faced this problem as well, i had to write a custom service to remove the persisting notification after app kill.

Faced this problem as well, i had to write a custom service to remove the persisting notification after app kill.
I am facing the same problem. I want to destroy the Music Controls when app is force closed when music is still playing.
@gondp @kpwa Can you please explain how to write custom service to achieve this?
Thank you.

I'm not clear on the description of this problem... but I'll add some things I've observed.
On my Android 12 (S21) device notifications are not visible in the notification pull down after I swipe away the app.
However, notifications do build up under App Info -> Notifications -> Notification Categories in that a new one is created each time I kill the app while the notification is active.  I've tracked this back to the random UUID assigned to the CHANNEL_ID in MusicControlsNotification.java.  If I assign a static value to this variable instead, then they stop multipling.
I've seen at least one other fork make a similar change.
I'm not sure why it should be different every time, unless it's to support multiple simultaneous notifications.  I think that's a very unusual situation, and also probably doesn't work well with the setting/clearing of ForegroundService anyway.  If anything, it should be based on the user's given NotificationId???

Additionally, when I call .destroy() from js, it does not remove the notification.
I tracked that back to probably some difference with the sdk26.  I added the following line to MusicControlsNotification.java from another fork and the problem was resolved: https://github.com/fenvalle/cordova-music-controls/blob/master/src/android/MusicControlsNotification.java#L129

@kpwa
Can you give me a sample custom service to remove the notification after app kill?
Any one else found any solution to this?


Flutter mac - Failed to establish connection with the application instance in Chrome

I got this error when running on mac machine. On my windows, it's working fine.
Failed to establish a connection with the application instance in Chrome. This can happen if the websocket connection used by the web tooling is unable to correctly establish a connection, for example due to a firewall.
I tried
flutter run -d chrome
flutter run -d chrome --release
Still not working in mac. Can you please help to resolve the issue

Please follow-up in https://github.com/flutter/flutter/issues/49482.
Closing this as duplicate.

I tried all steps in mac. Still, I stuck with this error.
[+1912 ms] "flutter run" took 5,915ms.
[   +6 ms] Failed to establish connection with the application instance in Chrome.
This can happen if the websocket connection used by the web tooling is unable to correctly establish a connection, for example due to a firewall.
[   +4 ms]
#0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
#1      _ResidentWebRunner.run (package:flutter_tools/src/isolated/resident_web_runner.dart:545:7)

#2      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:639:26)

#3      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1157:12)

#4      FlutterCommand.run. (package:flutter_tools/src/runner/flutter_command.dart:1009:27)

#5      AppContext.run. (package:flutter_tools/src/base/context.dart:150:19)

#6      AppContext.run (package:flutter_tools/src/base/context.dart:149:12)

#7      CommandRunner.runCommand (package:args/command_runner.dart:197:13)

#8      FlutterCommandRunner.runCommand. (package:flutter_tools/src/runner/flutter_command_runner.dart:278:9)

#9      AppContext.run. (package:flutter_tools/src/base/context.dart:150:19)

#10     AppContext.run (package:flutter_tools/src/base/context.dart:149:12)

#11     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:234:5)

#12     run.. (package:flutter_tools/runner.dart:64:9)

#13     run. (package:flutter_tools/runner.dart:62:12)

#14     AppContext.run. (package:flutter_tools/src/base/context.dart:150:19)

#15     AppContext.run (package:flutter_tools/src/base/context.dart:149:12)

#16     runInContext (package:flutter_tools/src/context_runner.dart:73:10)

#17     main (package:flutter_tools/executable.dart:90:3)


Fix clang-12 build

This issue is not reproducible with clang-10 or clang-11.
clang-12 invokes the linker with -L${SYSROOT}/lib which triggers an
issue because the arm-none-eabi-gcc sysroot has a multilib structure.
telling the linker to find libraries in that directory is
unfortunately wrong as it doesn't really take into account the
architecture of the platform.
This was causing the linker to link against an ARM library, inserting
blx instructions to transition to ARM from Thumb. Since the Cortex-M3
does not implement ARM, this triggers a fault.
Change-Id: Iadcea57727202858a00f8df5e8fbb038cfb25f2e

Relates to https://github.com/Javier-varez/Postform/pull/112


Fix-collectd-flakey-test

Proposed changes

Fix MQTT message assertions to avoid using a list and only assert the first message.
This fix is necessary as it is causing sporadic failures when running the integration tests.
Changes

Avoid using Should Contain with a list as it fails when the list length is greater than 1. Use the first message instead.
Add documentation and explicit arguments for MQTT message filtering based on payload in the  Should Have MQTT Messages keyword

Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

Avoid using Should Contain with a list as it fails when the list length is greater than 1

Are you saying that the Should Contain API itself doesn't support lists and hence should not be used or in this specific instance it should not be used? In this specific test where Should Contain is used to check for time and type fields, it should have been valid for all the messages in the list. So, I still didn't get what was wrong with that validation.

wrong with that valida

The problem is that Should Contain does not do a per item comparison, it only uses the python in operator on the list.
For example, the following python expression does not evaluate to True.
'time' in ['foo time','bar time']

The reason why it was sometimes working is that if the list only contained one item, then the RobotFramework was extracting the single item and doing a string operator
# A single item list was being interpreted by RF: ["foo time"] => "foo time"
"time" in "foo time"


Removing text.json.tests and text.formatting test projects from the build

Removing text.json.tests and text.formatting test projects from the build to unblock the repo

@krwq @KrzysztofCwalina


Questions about Ray implementation of PPO

Discussed in https://github.com/hpcaitech/ColossalAI/discussions/3606

Originally posted by cryoco April 19, 2023
I've seen 2 ray implementations of PPO in this repo, #3195  and #3309 . The former makes the experience maker and trainer decoupled, whereas the latter's models are still shared by experience maker and trainer. I'd like to know which implementation will be maintained in the future. Thank you!

Hi @cryoco Answered in Discussion.


⚠ Bofrost.at has degraded performance

In cb30935, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1012 ms

Resolved: Bofrost.at performance has improved in 8a5089b.


Add division 16/8 and 16/16.

Add two new macros for division of 16-bit dividend by 8-bit divisor and 16-bit dividend by 16-bit divisor.

Approved PR, and also closed and released 0.6.0
Good work!


[Feature Request]Can wavedrom support below datasheet like timing?

This MR might be helpful https://github.com/wavedrom/wavedrom/pull/337


Trace id problem when using Spring ThreadPoolTaskExecutor

Hi, I'm not sure if this is a problem with Spring Cloud Sleuth or I'm doing something wrong.
I found that when I use Spring ThreadPoolTaskExecutor the trace id of the parent process is not been used with the asynchronous task.
I have a small example with the problem here:
https://github.com/andcuevas/spring-cloud-sleuth-thread-pool-problem
Regards,
Andres

Since ThreadPoolTaskExecutor creates a separate thread it drops the MDC variables added by Sleuth and thus you see Trace Id and Span IDs being dropped.
Try to add a Decorator Pattern on top of ThreadPoolTaskExecutor class and override the execute method.
Also the Bean where you are configuring your TaskExecutor, initialize it to the new wrapper class.
Here is a sample implementation :
LogThreadWrapper.txt

I was hoping that Spring Cloud Sleuth and ThreadPoolTaskExecutor were compatible out the box since these are two tools from Spring.
It would be nice to know if a full compatibility will come in the future.

Well I hoped so too, may be that can be a question to @marcingrzejszczak if there is a plan to incorporate this change in the upcoming releases.

It should be working - currently I'm on holidays so will try to take a look at this once I'm back

Thanks. This is applicable for any multi-threaded method/class overridden. Best applicable for java 8 lambda functionality.


Feature/gc separate tissues

added/enabled tool to separate tissues by drawing some lines and splitting using a graph-cut
extended/cleaned up WidgetInterface in order to be able to implement the tool without dependencies on Mainwindow/SliceHandler, etc.

cleanup frenzy?

@oetiker it is difficult to add features to iseg without cleaning anything. the code is difficult to maintain/extend because of a history of copy-paste development.

@dyollb I totally feel your pain :-)


Reduce boilerplate in unit tests

I've done one or two PRs in this repo and felt that the unit tests had a bit of boilerplate.  Seeing #7 as well, I hoped you wouldn't mind a PR to reduce it a bit.
I have:

Pulled the error messages inside the test for most tests.
De-duplicated code which appears in multiple tests for rules into a rule-test-base file.
Split the config parser into a parser and a verifier, making it less effort to unit test the verifier (we do lose a small amount of coverage in the parser component by doing this, but would be happy to add in a new test).

There are no functional changes here, and only minimal changes to the tests themselves, it's mostly moving stuff around.

In v2.5.2


Will not work on functions with jmp/cmp/nearcall/ any relative offset in the first 14 bytes

Hey, while overall a great and simple library, it seem to break if a target function has a nearcall (E8 ? ? ? ?) in the first 14 bytes.
Possibly also will break on jmp and cmp if relative displacement is used.
Reason being this: https://github.com/SamuelTulach/LightHook/blob/fcdba7570fa564ae23c2a66cdf71b9d9334d579c/Source/LightHook.h#L186
But, considering not having to bring a fullass disassembler to use LH, not an issue per se, just letting you know ;)
Thanks for a great project!

Yeah it's not really fixable. I have modified the README to state that more clearly.


JSX rendering issue#2

hello,

initial output is:

0

after 2s timeout expires, rendered output is:

1

expected output is:

1
2

import Inferno from 'inferno';
import InfernoDOM from 'inferno-dom';
import InfernoComponent from 'inferno-component';

class MyComponent98 extends InfernoComponent.Component {
  constructor(props) {
      super(props);
      this.state = {
          rocks: [{id: 0}],
      };
  };

  componentDidMount() {
    setTimeout(function() {
        console.log("timeout");
        this.setState(
            Object.assign(
                {}, this.state,
                {rocks: [{id: 1}, {id: 2}]}))
    }.bind(this), 2000);
  };

  render() {
    return (
      <MyComponent99
            rocks={this.state.rocks} />
    )
  };
}

class MyComponent99 extends InfernoComponent.Component {
  constructor(props) {
    super(props);
  };

  render() {
    console.log("rocks in props=");
    console.log(this.props.rocks);

    return (

            {this.props.rocks.map((rock) => {
                return (

{rock.id}

                );
            })}

    )
  };
};

InfernoDOM.render(
  <MyComponent98 />,
  document.getElementById("app"));

regards,

@cvg1 Thanks for raising the issue, I'll try to take a look into it once I get a chance today (today is a very busy day for me however). Do you get the correct props passed through in your console.log?

yes, props are correct. and don't worry, it's not urgent at all.
take your time.
thanks

Have you tried using latest dev? Some similar issues have been fixed there (not sure about this)

hello,
yes, i can confirm that this issue exists for 0.5.22(npm) and is fixed in latest dev.
closing.


Support for absolute path links (without hostname?)

From the hal spec:
   {
     "_links": {
       "self": { "href": "/orders/523" },
       "warehouse": { "href": "/warehouse/56" },
       "invoice": { "href": "/invoices/873" }
     },
     "currency": "USD",
     "status": "shipped",
     "total": 10.20
   }

On similar representations, I'm getting unknown scheme errors when following links.

It would be great to have support for relative links.  I think we could use the #relative? and #+ of Addressable::URI to implement it pretty easily. @zeckalpha, i would merge a PR to that effect if you submitted one.

Fix in version 3.13.0


How to start using MJExtension?App crashed when I use MJExtension.unrecognized selector

NAME : NSInvalidArgumentException
REASON : +[MJUser mj_objectWithKeyValues:]: unrecognized selector sent to class 0x103c0f5e8
I add pod 'MJExtension' in my podfile
in my ViewControl.m file I define MJUser as the demo did,
and ,I import "MFExtension.h" as well,
in viewDidLoad, I write [MJUser mj_objectWithKeyValues:dic];
dic is defined as the demo did.
where I was wrong?
Is there something I lost?

试试手动导入 MJExtension 目录到项目中

可以了，难道pod下来的代码是不可用的？

@bqlin OK 了 已经用了3.0.8版本

我使用pod MJExtenstion 项目中使用mj_objectWithKeyValues 直接说找不到方法，手动导入到工程中就ok ，求解


Upload this to Open VSX (suggestion)

It will be really great if you can upload this extension on open-vsx  so people on VS Codium or Code OSS can easily access it.
As a side note- On linux I now use https://github.com/boocs/unreal-clangd Unreal Clangd extension which works flawlessly with clangd and is honestly better imo than Microsoft's Extension support. With these snippets it also fills some gaps tho.
This can also be closed if you dont want to put it as this extension does not need maintainance anymore I suppose and I am fine downloading vsix manually from the marketplace website.
Here is my simple blog as well on my workflow -
https://shuriken.sh/posts/unreal-clangd-workflow/

Honestly, I have not used UE4 in about 3 years now, I don't even know if any of these snippets still work (since you're still using them I assume at least some of them do).
So I'm not too sure about releasing this again, without making sure it's something functional and of high-quality.
Can you test how many of these snippets are still up-to-date (and maybe open a PR fixing those that aren't?). Then I will release it onto open-vsx for sure! 🙂
(also, I assume we should maybe rename it to ue5-snippets-vs, it seems to me like ue5 is just an updated version of ue4?)

These snippets are still used in UE5 tutorials so they clearly work. Interesting thing is that MS has similar snippets in Visual Studio now:
https://devblogs.microsoft.com/cppblog/unreal-engine-cpp-game-development-made-easy-visual-studio-2022/
The list above should provide a good baseline for any updates needed for this extension as the article dates back to end of last year :)


Amplify push is is stuck with message "Rolling back (1 of 3)" forever.

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v16.14.0
Amplify CLI Version
9.1.0
What operating system are you using?
AMZ Linux using Cloud9
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Amplify Categories
auth, custom, storage, function, api, hosting
Amplify Commands
push
Describe the bug
Pushing changes of one GraphQL type, in one index the sortKeyFields was changed and one attribute was added.
It started well and went to Update (3 of 3) without any error, but then without any error shown in shell log or elsewhere, the shell log went to Rolling back (1 of 3) and is still show that for more than an hour. I checked then the CloudFormation console and all stacks are in completed status, either CREATE_COMPLETE or UPDATE_COMPLETE. No stacks are in progress or in error state. No wonder that the rolling back is showing for ever, Rolling back (1 of 3), there is zero activity in CloudFormation.
Expected behavior
At least an error and a working roll back.
Reproduction steps
I have no time to do that.
GraphQL schema(s)

# Put schemas below this line

Project Identifier
No response
Log output

Could not open file: ~/.amplify/logs/amplify-cli-2022-07-23.log
Unknown error: Failed to retrieve resource from https://vfs-cloud9.eu-west-1.console.aws.amazon.com/vfs/3eb9a0d887d445adb83bf77e35a07ead/home/.amplify/logs/amplify-cli-2022-07-23.log with code 0

Additional information
The amplify command is still stuck, don't know what todo, control break? So long I can't access the log, as you can see above. Really bad user experience right know. I will wait some more time and then abort the command, hope that I will not get follow up errors like stack not in state update complete or similar, but that is probably only wishful thinking... it looks to me as if the amplify command hit a timeout right when the stack had completed the whole operation, then the rollback was never done, something like that.

I stopped the execution with control break, then I did a:
amplify pull --restore
which did well. Then I tried to push a change again with:
amplify push -y --allow-destructive-graphql-schema-updates --force
which failed with:
🛑 An error occurred during the push operation: /
Cannot iteratively rollback as the following step does not contain a previousMetaKey: {"status":"DEPLOYING"}
Hell continues...

Next step, I deleted the deployment-state.json in the amplify-xxx-xxx-xxx-deployment bucket, corresponding to the affected environment.
The push went through now, I hope it is ok to proceed like that.


Blog article highlight differences in expections in hires from commercial and academia

Had a conversation with @csnavely at SMC about this topic. Might make for a good blog article. Cory, if you have a moment to jot down some key points here in this issue, that might get the ball rolling.

Maybe I need to hear more of the content of this discussion from @markcmiller86 .  Are we really talking about hiring here?  Is that in scope for BSSw?  What is the connection to software productivity and sustainability types of things?

Hmmm...its been a few weeks now and I can't recall all the details but the comments had to do with software productivity and the thinking cap commercial vs. academic hires have. Commercial people tend to come with an issue ticket mentality w.r.t. software productivity...just give me the issues I am supposed to be responsible for. Academics come with a different mentality (which I can't recall now). This impacts the onboarding process as well as general expectations new hires have w.r.t. to the software development responsibilities.

Need more information on the scope of the actual topic, especially if it is a blog. Please put link to cc article if we plan to go with curated content here. I gave it a confused sign to indicate that I have reviewed  this

Closed due to insufficient interest


Move question on 'Purpose' to be first

This forces users to reflect on the purpose of their software and should help them when filling in the later anwers

Fixed by now


Inspect structural similarity between SEC filings with common properties.

👁 Inspect structural similarity between SEC filings with common properties

📚 Background
We have CSV files of 10-Q, 10-K, and 8-K filings made between 2003 and 2023. Each filing is represented by a row in the CSV file with the following properties:

ticker: Ticker symbol for the filing
currency: Currency type for the filing
companyName: Company name under which filing was made
location: Location associated with the filing
isUS: If US location or not
filedAt: Filing time string
filingYear: Filing year
market_cap_category: Company size
exchange: Exchange under which the company is listed
isDelisted: Is delisted from the stock exchange or not
category: Stock category
sicSector: Sector associated with the filing
sicIndustry: Industry associated with the filing

The URL to download the filing is available in linkToFilingDetails column.
(The plots  below show the distribution charts for the above properties)

The filings are available in the following Google Drive links:
🔗 10-Q Filings: https://drive.google.com/file/d/1XUBDNS1D52VUGV1B0BsGuY-XdA8IN4E4
🔗 10-K Filings: https://drive.google.com/file/d/1cC1Dd_OR9DRHNQ4eWJ6-2SbwEaevhNl0
🔗 8-K Filings: https://drive.google.com/file/d/1HhdQ7dZWZqqAUMG3HfgWnjFdPPxhNokX
Additionally, unlike 10-Q and 10-K there are multiple types of forms with different items for 8-K. They are represented in column names in the format Item:{item-id}. They are distributed as follows:

🦠 Problem
We need to figure out if different filing property values correspond to different structures of HTML filings.
For example, the figure below shows different possible sicSector values for 10-Q filings. You need to figure out if filings with different sicSector values are structurally dissimilar or not. One way to do this is to sample 10 HTML filings with sicSector=Mining and another 10 HTML filings with sicSector=Services and manually inspect if they are similar or not. If they are dissimilar, sicSector is a valid property for sampling a representative sample set.

Do the same for currency, location, isUS, market_cap_category, exchange, filingYear, isDelisted, category, sicSector, sicIndustry of 10-K, 10-Q and 8-K filings and figure out if their different values correspond to different structures of HTML filings.
🌟 Other Approaches
Automate this by using unsupervised clustering methods.

Hi,
I'd like to confirm my understanding of the issue. Taking a concrete example, It is to see if separating out the records on the basis of sicSector leads to records having different "distributions".  by "distributions", i mean for example that selecting sicSector=mining effects the frequency and correlations of the remaining properties including filingYear, stockcategory etc.
if so, let us call the property whose values we are segmenting by, as the segmenting property, and the remaining as other properties. If we train a classifier to predict the segmenting property from a vector consisting of other properties, in case the segmenting property indeed does separate out the data on the other properties, we should see high accuracy.
Let me know if my understanding, and approach seem sensible

Hi @tumble-weed 👋, thank you for showing interest in this issue.

Taking a concrete example, It is to see if separating out the records on the basis of sicSector leads to records having different "distributions". by "distributions", i mean for example that selecting sicSector=mining effects the frequency and correlations of the remaining properties including filingYear, stockcategory etc.

We are not interested in "distributions" in any way 😀

...  let us call the property whose values we are segmenting by, as the segmenting property (in this example sicSector), and the remaining as other properties ...  we train a classifier to predict the segmenting property from a vector consisting of other properties

That is an interesting idea you got there! You are very close in your understanding but not entirely correct. Instead of seeing how segmenting property affects all the other properties, we are only interested in seeing how segmenting property affects linkToFilingDetails.
linkToFilingDetails contains the URL for HTML filing. The purpose of this issue is to find out if these segmenting properties affect the HTML structure of filings. By "HTML structure" I mean anything that may affect the parsing of sec-parser. The easiest way to know it is by simply downloading the filing document and having a look at it.
Let me know if this clears your doubts!

Hi, @INF800.
Nice to meet you here.
As we discussed at the discord, I think the purpose of this issues is to find how sector differences differ when it comes to HTML structure.
But in terms of HTML structure, I think it can be the semantic structure of the HTML extracted by sec-parser.
Of course, the source of HTML is linkToFilingDetails from the CSV.

Hi @kameleon-ad 👋,

I think the purpose of this issues is to find how sector differences differ when it comes to HTML structure.

Yes, exactly.

But in terms of HTML structure, I think it can be the semantic structure of the HTML extracted by sec-parser.

Thank you for this important point. By HTML structure I mean the inherent structure of HTML filing available via linkToFilingDetails.

@kameleon-ad @tumble-weed Also, I just crafted a starter notebook to help you with some other things like the problems you may face while downloading the HTML file using python - This happens because SEC website accepts requests with specific headers.
The notebook code will help you fetch HTML files for each property type and put them in the following folder structure.
└── 10-Q
    ├── exchange
    │   ├── BATS_n50
    │   ├── NASDAQ_n50
    │   ├── Not Available_n50
    │   ├── NYSEARCA_n50
    │   ├── NYSEMKT_n50
    │   ├── NYSE_n50
    │   └── OTC_n50
    ├── isDelisted
    │   ├── False_n50
    │   ├── Not Available_n50
    │   └── True_n50
    ├── isUS
    │   ├── False_n50
    │   ├── Not Available_n50
    │   └── True_n50
    ├── market_cap_category
    │   ├── Large ($10-200B)_n50
    │   ├── Medium ($2-10B)_n50
    │   ├── Mega (>$200B)_n50
    │   ├── Micro ($50-300M)_n50
    │   ├── Nano ($0-50M)_n50
    │   ├── Not Available_n50
    │   └── Small ($0.3-2B)_n50
    ...
    ```

Hi, @INF800, nice to meet you
Maybe we just need to add some features in the CSV file containing information about the structure of the HTML Filling. But the first thing to do is define the "information structure for the HTML Filling", for example: the number of 'table of contents' item, number of tables in the HTML, etc.
There are two ways to do this:

As you said, do it manually, not to all HTML Filling, but using samples (ex: 20 doc from every unique values in the features: currency, location, isUS, market_cap_category, exchange, filingYear, isDelisted, category, sicSector, sicIndustry)
Using sec-parser to find the information structure

Then using clustering to cluster every HTML Filling, by the cluster we got from clustering, we calculate the correlation of cluster to every feature(currency, location, isUS, market_cap_category, exchange, filingYear, isDelisted, category, sicSector, sicIndustry) so we can know which features make a difference in HTML filling.
I am sorry it's just a raw idea, I haven't done it yet.

Hello @Risdan224, thank you for showing interest in this issue! You are right, we can definitely leverage unsupervided clustering methods for this task. Inspecting manually is not a viable option when dealing with hundreds of thousands of files, it is just a quick solution for now.
If you want to use clustering methods you can create your own feature sets and experiment with them. There are some interesting but a bit old projects like html-cluster based on page-compare project which can help you in feature engineering. Have a look at it and let me know what you think of it. I'd suggest starting with simple unsupervised methods first.

Hello @tumble-weed @Risdan224 @kameleon-ad, Please feel free to share your findings whenever you get a chance. I was actively working on it this past week and would like to discuss your perspectives on this.


Check if database can be encrypted

mongo supports this:
https://docs.mongodb.com/manual/core/security-encryption-at-rest/
though only in enterprise version


Little variance in sampled bit count

Whilst actual raw sample bytes appear to vary widely the count of "set bits in every 8 word sample" does not.
It hovers very close to 128 set bits for every sample, even when banging the board to create noise.

Samples written to wav file and contain recognisable audio, microphone is not very sensitive and noise ceiling is high


What is returned if no match found

What value is returned if no matching record is found. That should really be mentioned in the same sentence that mentins what is returned if a match occurs.

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

ID: 9b784f58-59eb-fb06-da8f-4747f6900db3
Version Independent ID: 9b784f58-59eb-fb06-da8f-4747f6900db3
Content: Filter, Search, and LookUp functions in Power Apps (contains video) - Power Platform
Content Source: power-platform/power-fx/reference/function-filter-lookup.md
Service: power-platform
Sub-service: power-fx
GitHub Login: @gregli-msft
Microsoft Alias: gregli

Thanks for your comment, @shinyion. This issue has been assigned to the doc author for review.


When running cucumber tests with webdriverio not getting code snippets as how Cucumberjs splits on console

Not sure if its not possible currently or if im doing something wrong.
I really liked how webdriverio works compared to selenium-webdriver. Im trying to use cucumberjs with webdriverio. I created a sample feature file and no step definitions. When i run using webdriverio config it just says as below but does not split the code skeleton snippets to console just how cucumberjs does. Could you help me if i had to set any flag for that?

Performing a search12 I am on the search page:
Step "I am on the search page" is not defined. You can ignore this error by setting
cucumberOpts.ignoreUndefinedDefinitions as true.
running chrome

@rameshrengasamy I am afraid we don't get this information in WebdriverIO from cucumber so we can't display it :-/.


Fix for broken backspace behaviour in iOS8-8.2

In iOS8 versions prior to iOS8.3 backspace button was not deleting tokens. This behaviour is a known iOS8 bug and this commit is a workaround that fixes it. Credits for this solution goes to stackoverflow user cnotethegr8 (http://stackoverflow.com/users/278629/cnotethegr8), it's originally posted here: http://stackoverflow.com/a/25862878/2189481.
Reportedly this bug is now fixed in latest iOS8 version, that is in iOS8.3 and code reflects that.

I can’t repro the issue on iOS 8.1. If you think it’s still an issue, please feel free to reopen the PR. Sorry for the delay, and thank you for the PR! :smiley:


Is this still Maintained ? Any update coming ?

Yes, a few bug fixes need to be addressed first (iOS errors mostly) and the current master branch could need a bit more community testing for the new isolate code. After that, V2.0 should be released - we also have some breaking API changes.


Reincorporate products with server components instead of client components

transfer eCommerce section from pages directory to app directory

Completed in branch Syed


Add Paging for Labs in front end.

🚀 Feature Proposal
Paging is already implemented for Patients list in Front end.
https://github.com/HospitalRun/hospitalrun-frontend/pull/2026
Paging for Labs list can be implemented similarly.

I can take this one.


Feature-gate test-programs

This commit feature-gates test-programs as a workaround for
cargo test not allowing for optional dev-dependencies.
This commit addresses #595.

I'm gonna go ahead and merge this PR, and close #595. If at a later date we decide we won't change things in a different direction, let's reopen #595 and take it from there!


Use generics

Now with generics we can have a more ergonomic api without reflection, what do you think about it?

It will get used then, and I don't want that.


Use "Remove display from desktop" for arbitrary Apps

Description of the new feature / enhancement
Windows 10 (and 11) have the following switch hidden in the extended display settings:

As far as I know, this makes a display that is connected to your computer available exclusively for certain apps. Unfortunately, I was not able to find any app making use of this, so I could not really test it. Apparently it needs some special API calls integrated into the applications code.
My idea would be to somehow make PowerToys able to "wrap" an application in order to make it able to use this feature.
Scenario when this would be used?
I myself have three normal monitors as well as a large wall display connected to my computer. I would like to switch my large display into this mode and have e.g. Edge use it in fullscreen mode to display a dashboard.
Supporting information
Maybe https://learn.microsoft.com/en-us/windows/win32/gdi/multiple-display-monitors, but I find it hard to find concrete information on how to make use of such displays.

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


Headless android support with external cameras

Support robots using the Dragonboard 410C or other development boards running a non Android Things image

This will require:

Support for external cameras, or using /dev/{camera}

Potential GPIO support, I personally have code for this somewhere

external cameras working, but building from source is required until code is added to autodetect


DCOS-11939: Pass update callback to ClusterHeader

This PR allows an onUpdate callback function to be passed to the ClusterHeader component. This is necessary for the Sidebar because the ClusterHeader's size will update when it receives the IP address of the cluster. When this happens, the height of the neighboring Gemini container needs to be updated.
A symptom of this was the Sidebar's dock icon rendering slightly off the viewport.
Before:

After:

Thanks!


Implement file staging provider API

Instead of dispatching inside the DFK explicitly based on a
hard-coded map between URI schemes and staging implementations,
go through each staging scheme in sequence, giving that scheme the chance
to handle staging
The concept of 'is_remote' being a property of files defined
by the URL scheme is also superceded by this, and so
is_remote is removed. For behaviour of "file:" URLs to mean
'no staging', then a no-op staging scheme is implemented
in this PR. Other  staging providers can implement different
semantics for file: URLs.
There's a user-facing API change here: if a configuration specifies
storage_access=[] rather than omitting it or specifying storage_access=None,
then the default staging mechanisms will be removed and
even no-op file access won't work.
Start reading at parsl/data_provider/staging.py, which defines the new API.
Existing staging code is modified to implement that API, and existing code that calls into staging does so via that API - defaulting to a set of staging providers giving equivalent behaviour to that  immediately before this PR.

even no-op file access won't work.
How does this look to the user?

even no-op file access won't work.

How does this look to the user?

The user gets this exception:

raise ValueError("Executor {} cannot stage file {}".format(executor, repr(file)))

if they try to stage any Files.
This is the exception that happens when no staging provider claims a particular File, and having no staging providers at all is a specific case of that.


Shall we have enable_adapter in PeftModel attributions?

Feature request
PeftModel supports disable_adapter, which disables the adapter modules; but I didn't see an enable_adapter func to revert this.
However, the base model implementation supports both disable_adapter_layers and enable_adapter_layers (taking lora as an example); and it seems to me that enable_adapter_layers is not called by any funcs.
May I ask why PeftModel does not support enable_adapter?  Is it intended?  Thanks!
Motivation
Try to have the flexibility to apply adapter or not.
Your contribution
NA

The disable_adapter function is a context manager. Within the context, adapters are disabled, but as soon as you leave it, it is enabled again. Therefore, there is no need for a separate function to enable the adapter.

Thanks for the explanation!


Link to the advertising page from advertisements themselves

This changes the info button next to the advertisements to link to the advertising page.

Hmm. I wonder if this code should also live in the donate app. I believe it's here because it's included in the main "on doc page" JS, but we could easily import it from donate, once we figure out how to properly set up our JS builds to work with it.
Will merge it for now, but plan to migrate it later.

It probably should live there.


Place a page's github link on the page

i believe providing a link to each page's location in github would greatly ease the ability to contribute and in turn increase the quality of the book

We would prefer contributions through the book's built-in feedback tools rather than Github. For this reason we don't have deeplinks into the Github.


timeit-profile

We use timeit to compare performance of similar code:
$ python -m timeit -s 'import foo' 'foo.egg()'
10000 loops, best of 3: 25.2 usec per loop
$ python -m timeit -s 'import foo' 'foo.spam()'
10000 loops, best of 3: 40.3 usec per loop

Now we know foo.spam() is slower.  But why?
profiling should provide timeit as a command.  It will be helpful to inquire into the cause of the performance loss.
$ python -m profiling timeit -s 'import foo' 'foo.spam()'
            ^^^^^^^^^
           Just add it.

Implemented at 9c5c5b355d4ef6b6b50f9575ff83c7bd95726eb9.


How to change the icon color of toolbar ?

also i cannot find this option you mentioned , the image to be exact
"
For the toolbar background, choose the provided image, otherwise, use the solid color RGB(25,25,25) which has the closest color aproximation to Windows's dark mode.
"

As far as I am aware, the QTTabBar Standard Buttons do not have a way to modify the text color aside from the icon colors. IMHO, the QTTabBar Standard Buttons are not useful and just add clutter to the windows file explorer. To disable the QTTabBar Standard Buttons, right click on the toolbar, unlock the toolbar, right click on the dots that appeared on the left, and disable the QTTabBar Standard Buttons, then lock the toolbar again.

ok , but i use groups and recently closed buttoms, is there a way to get use of those feature while having buttons disables ?

ok , checked it, if i right click on empty space i can access the groups and recently closed. thanks


[azurerm v3.31.0] error while creating APIM using Terraform

I am having an issue while creating APIM via terraform, using azurerm v3.31.0.
After 33m30s I get the error message - waiting for recovery of "Api Management" pollingTrackerPut#checkForErrors: the response did not contain a body: StatusCode=0.
Hashicorp have confirmed it's a problem with Azure API (https://github.com/hashicorp/terraform-provider-azurerm/issues/8579)

@samiko-nd Apologies for the late reply. Thanks for reaching out to us and reporting this issue. Could you please share the subscription-id, correlation-id and the time of the issue in UTC ?

@samiko-nd I wanted to do quick follow-up to check if you had a chance to look at my above comment. Please share the subscription-id, correlation-id and the time of the issue in UTC over email navba [ @ ] microsoft . (dot) com. Awaiting your reply.

@samiko-nd The action is currently pending on you to share the above requested details. Once you are ready to resume troubleshooting on this, please feel free to reopen this thread. We would be happy to help.


refresh a gem a little

remove jeweler
Gemfile added
fix dependencies
update rspec
nice and colorized output
-c option works when downloading subtitles
update README

This is very cool, thank you.

Can you bump version and release this ?

Will do, removed ancient hpricot too, working with nokogiri.
Download is working fine for you?

Yes, downloading works fine. Tested with nokogiri.

All done, thanks again! Plus some rubocop and rspec 3.


"use client" error from @emotion/react

I've setup my project for @mui/material following the example but I'm still getting the same error as before. I'm using PNPM if this changes anything.
Here the relevant logs:
- error node_modules\.pnpm\@emotion+react@11.11.1_@types+react@18.2.14_react<EMAIL_ADDRESS>(14:41) @ React
- error createContext only works in Client Components. Add the "use client" directive at the top of the file to use it. Read more: https://nextjs.org/docs/messages/context-in-server-component
    at eval (webpack-internal:///(sc_server)/./node_modules/.pnpm/@emotion+react@11.11.1_@types+react@18.2.14_react@18.2.0/node_modules/@emotion/react/dist/emotion-element-6bdfffb2.esm.js:34:92)
    at (sc_server)/./node_modules/.pnpm/@emotion+react@11.11.1_@types+react@18.2.14_react<EMAIL_ADDRESS>(D:\Programming\lq\.next\server\app\page.js:11938:1)
    at __webpack_require__ (D:\Programming\lq\.next\server\webpack-runtime.js:33:42)
    at eval (webpack-internal:///(sc_server)/./node_modules/.pnpm/@emotion+styled@11.11.0_@emotion+react@11.11.1_@types+react@18.2.14_react@18.2.0/node_modules/@emotion/styled/base/dist/emotion-styled-base.esm.js:8:72)
    at (sc_server)/./node_modules/.pnpm/@emotion+styled@11.11.0_@emotion+react@11.11.1_@types+react@18.2.14_react<EMAIL_ADDRESS>(D:\Programming\lq\.next\server\app\page.js:11982:1)
    at __webpack_require__ (D:\Programming\lq\.next\server\webpack-runtime.js:33:42)
    at eval (webpack-internal:///(sc_server)/./node_modules/.pnpm/@emotion+styled@11.11.0_@emotion+react@11.11.1_@types+react@18.2.14_react@18.2.0/node_modules/@emotion/styled/dist/emotion-styled.esm.js:5:95)
    at (sc_server)/./node_modules/.pnpm/@emotion+styled@11.11.0_@emotion+react@11.11.1_@types+react@18.2.14_react<EMAIL_ADDRESS>(D:\Programming\lq\.next\server\app\page.js:11993:1)
    at __webpack_require__ (D:\Programming\lq\.next\server\webpack-runtime.js:33:42)
    at eval (webpack-internal:///(sc_server)/./node_modules/.pnpm/@mui+styled-engine@5.13.2_@emotion+react@11.11.1_@emotion+styled@11.11.0_react@18.2.0/node_modules/@mui/styled-engine/node/index.js:44:38) {
  name: 'TypeError',
  digest: undefined
}
null

Is there anything I'm missing out? I've tried adding @emotion/react to the include array of the configuration, without success.

You are using Windows, did you try to include: ["@emotion\react"] rather than "@emotion/react"?

I fixed this problem in 1.0.4， you can use include: ["@emotion/react"] directly.

In the end, the update seems to have fixed my issue. I was indeed on Windows. My current setup looks a bit like this:
{
  // ...
  experimental: {
        swcPlugins: [
            [
                "use-client",
                {
                    include: [
                        "@mui/material",
                        "@mui/icons-material"
                    ]
                }
            ]
        ]
    }
}


equality for KORE terms

@dwightguth, the Att-less equals -- it also adds hashCode and equals for all KORE terms (this superseeds any case class generated equals and hashCode)

Looks good, but can you make hashCode be a lazy val? Everything is immutable, so that should be safe, and it will have a significant performance impact on large terms.


[Wallet] Add retry logic to verification reveal failure

Description
Add retry logic to verification reveal failure when No incomplete attestation found error returned by attestation service
Currently, when a reveal fails, we assume that either it was already revealed in the past (Verification Failure modal shown with option to enter codes) or it is an irrecoverable failure (Verification Failure modal with no option to enter codes). However @nambrot pointed out that a reveal can fail if an attestation service has not yet received the block where it is made responsible for the attestation, in which case the reveal should retry, as time will have passed for the attestation service to receive the relevant block.
More context: https://celo-org.slack.com/archives/CJXJ1EV0D/p1593590635047500
Other changes
Add reveal error message to analytics
Tested
Not yet tested
Backwards compatibility
Yes

Closing as these changes will go in with @jmrossy 's proxy setup for verification


The animation attribute of select is true, but there is no animation effect when adding tag

[ ] I'm sure this does not appear in the issue list of the repository

Basic Info

Package Name And Version<EMAIL_ADDRESS>Browser: chrome99.0.4844.51
Reproduction Link: https://arco.design/react/components/select

What is expected?
Repair animation. If you can, can you provide an interface that can customize animation?

Version 2.27.0 caused this bug.


Update shubhra-kar.md

Clyde Seepersad
Shubhra took on the impossible with a passion, an infectious optimism and a wonderful sense of humor and humility. My words fail so I'll reach for Andy Williams:
To fight for the right
Without question or pause,
To be willing to march
Into hell for a heavenly cause.
And I know if I'll only be true
To this glorious quest
That my heart will lie peaceful and calm
When I'm laid to my rest
And the world will be better for this
And one man, sore and covered with scars
Still strove with his last ounce of courage
To fight the unbeatable foe
To reach the unreachable star

added yours here Clyde: https://github.com/LFX-Engineering/memorials/commit/513f4c158a1d4fffacf81e81fa44641e30ed0fcb


Codeunit 235 "G/L Reg.-Gen. Ledger" - OnGLRegGenLedgOnBeforeRunPage

Can you please add the OnGLRegGenLedgOnBeforeRunPage event to the OnRun trigger in Codeunit "G/L Reg.-Gen. Ledger" (235)?
Actual Code:
trigger OnRun()
begin
    GLEntry.SetRange("Entry No.", "From Entry No.", "To Entry No.");
    PAGE.Run(PAGE::"General Ledger Entries", GLEntry);
end;

Suggestion:
trigger OnRun()
begin
    GLEntry.SetRange("Entry No.", "From Entry No.", "To Entry No.");
    OnGLRegGenLedgOnBeforeRunPage(GLEntry)
    PAGE.Run(PAGE::"General Ledger Entries", GLEntry);
end;

[IntegrationEvent(false, false)]
local procedure OnGLRegGenLedgOnBeforeRunPage(var GLEntry: Record "G/L Entry")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 53049.


json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Everything was fine until i got this error. What does it mean and how can i fix this?

I recieve the same error
Traceback:
Traceback (most recent call last):   File "/usr/lib/python3.5/socketserver.py", line 313, in _handle_request_noblock     self.process_request(request, client_address)   File "/usr/lib/python3.5/socketserver.py", line 341, in process_request     self.finish_request(request, client_address)   File "/usr/lib/python3.5/socketserver.py", line 354, in finish_request     self.RequestHandlerClass(request, client_address, self)   File "/usr/lib/python3.5/socketserver.py", line 681, in __init__     self.handle()   File "/usr/lib/python3.5/http/server.py", line 422, in handle     self.handle_one_request()   File "/usr/lib/python3.5/http/server.py", line 410, in handle_one_request     method()   File "language.py", line 36, in do_POST     result = translator.detect(text)   File "/usr/local/lib/python3.5/dist-packages/googletrans/client.py", line 249, in detect     data = self._translate(text, dest='en', src='auto')   File "/usr/local/lib/python3.5/dist-packages/googletrans/client.py", line 81, in _translate     data = utils.format_json(r.text)   File "/usr/local/lib/python3.5/dist-packages/googletrans/utils.py", line 62, in format_json     converted = legacy_format_json(original)   File "/usr/local/lib/python3.5/dist-packages/googletrans/utils.py", line 54, in legacy_format_json     converted = json.loads(text)   File "/usr/lib/python3.5/json/__init__.py", line 319, in loads     return _default_decoder.decode(s)   File "/usr/lib/python3.5/json/decoder.py", line 339, in decode     obj, end = self.raw_decode(s, idx=_w(s, 0).end())   File "/usr/lib/python3.5/json/decoder.py", line 357, in raw_decode     raise JSONDecodeError("Expecting value", s, err.value) from None json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

What version of googletrans do you have installed?
If it's latest from pypi then try the solution from #166 - change client parameter in request from 't' to either 'gtx' or 'webapp' in https://github.com/ssut/py-googletrans/blob/master/googletrans/utils.py file.

What version of googletrans do you have installed?
If it's latest from pypi then try the solution from #166 - change client parameter in request from 't' to either 'gtx' or 'webapp' in https://github.com/ssut/py-googletrans/blob/master/googletrans/utils.py file.

Ill try this, give me like an hour

Didnt work, but maybe its becuase my ip has been banned and i gotta wait

Getting the same error on running:
translator.translate('Đào Huy Tuấn')

What version of googletrans do you have installed?
If it's latest from pypi then try the solution from #166 - change client parameter in request from 't' to either 'gtx' or 'webapp' in https://github.com/ssut/py-googletrans/blob/master/googletrans/utils.py file.

'gtx' work fine


release: set aqaReference tag for 2022 Oct CPU release

to set default value of aqaRef to release tag, making sure we only use this for test
this PR should be reverted once we've done CPU release

@zdtsw @smlambert So interesting change, so basically once this re-gens, the pipelines nightly/release will use the specific AQA tag, rather than HEAD.. Would we want that ongoing...? ie.by default we use the latest and greatest AQA tag? So if someone wants to run a pipeline on HEAD, they'd need to specify..(blank) ?

@zdtsw @smlambert So interesting change, so basically once this re-gens, the pipelines nightly/release will use the specific AQA tag, rather than HEAD.. Would we want that ongoing...? ie.by default we use the latest and greatest AQA tag? So if someone wants to run a pipeline on HEAD, they'd need to specify..(blank) ?

I am thinking for the next week, we can lock to use the aqa release tag, once the build + test in public jenkins is done, we can revert the change, so aqa uses HEAD again.
The reason for this PR is because, i manually set the v0.9.5-release to all opendjkX-pipeline yesterday, expecting over the night we will start to use that one. But we have PRs got merged from outreachy which re-genereate pipelines. so if we do not code freeze next week (even just a *.md change) which can cause our pipeline keeps re-generate. so it runs back and forth between v0.9.5 and HEAD, which need to double check everytime when start a  new release build.

@zdtsw Eclipse CLA seems to be checking a different user name - is some git metadata getting mixed up here?

@zdtsw Eclipse CLA seems to be checking a different user name - is some git metadata getting mixed up here?

Thanks!
yeah, on a different laptop, not config to use working e-mail.
Will amend it

Makes one wonder whether the automatic regeneration of the build pipelines is a good feature, or if the regen could be tuned to be triggered only when critical pipeline code changes, to avoid some of the unnecessary churn described.
To have more controls over it and avoid churn, we chose to put a flag on the autogen of test pipelines.

what we could do is, to manually disable

https://ci.adoptopenjdk.net/job/build-scripts/job/utils/job/build-pipeline-generator/
https://ci.adoptopenjdk.net/job/build-scripts/job/utils/job/pipeline_jobs_generator_jdkXu/
after we declare "code freeze" and still allow trivial PRs (documentation)  to be merged to master

the risk of this is:  once we have emergency PR landed, we need to remember enable these jobs and all PRs between will be taken into automatically in that run.

My comment really should go into a release retrospective issue.  I am not opposed to this PR as a way to work around the currently overactive pipeline generation.  I was saying that perhaps after October release we consider making the pipeline generation 'smarter' (as in it won't trigger on doc changes or non-pipeline code changes).

My comment really should go into a release retrospective issue. I am not opposed to this PR as a way to work around the currently overactive pipeline generation. I was saying that perhaps after October release we consider making the pipeline generation 'smarter' (as in it won't trigger on doc changes or non-pipeline code changes).

I agree. I will close this PR and put in the retro see how we can improve this for the next CPU.


Add Poisson node

This PR addresses issue #129

Codecov Report

Merging #132 (dae3516) into master (d874509) will decrease coverage by 0.53%.
The diff coverage is 85.71%.

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

@@            Coverage Diff             @@
##           master     #132      +/-   ##
==========================================
- Coverage   75.27%   74.73%   -0.54%
==========================================
  Files         176      178       +2
  Lines        6163     6037     -126
==========================================
- Hits         4639     4512     -127
- Misses       1524     1525       +1

Impacted Files
Coverage Δ

src/ReactiveMP.jl
50.00% <ø> (ø)

src/nodes/poisson.jl
84.61% <84.61%> (ø)

src/rules/poisson/marginals.jl
100.00% <100.00%> (ø)

src/approximations/gausshermite.jl
37.50% <0.00%> (-6.95%)
:arrow_down:

src/nodes/bifm.jl
72.72% <0.00%> (-5.33%)
:arrow_down:

src/algebra/companion_matrix.jl
41.02% <0.00%> (-3.28%)
:arrow_down:

src/constraints/spec/form_spec.jl
58.33% <0.00%> (-3.21%)
:arrow_down:

src/distributions/normal.jl
81.48% <0.00%> (-1.86%)
:arrow_down:

src/nodes/flow/layers/permutation_layer.jl
74.57% <0.00%> (-1.62%)
:arrow_down:

src/nodes/autoregressive.jl
81.25% <0.00%> (-1.36%)
:arrow_down:

... and 36 more

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 d874509...5d49946. Read the comment docs.

Please look up the fixes


Unify password changing and invalidate auth tokens

Unify the password changing code
Invalidate existing auth tokens when changing passwords

Have to fight an import cycle in the auth service first...

There's an import cycle?

Yes, after fixing the CI error

It's recommended by the original authors (https://paragonie.com/blog/2015/04/secure-authentication-php-with-long-term-persistence#secure-remember-me-cookies)

Important: If the user should ever change their password, you should invalidate all existing long-term authentication tokens for that user.

I don't know if others do that but I see the security benefit. A password change requests a re-login on other devices.

It's recommended by the original authors (https://paragonie.com/blog/2015/04/secure-authentication-php-with-long-term-persistence#secure-remember-me-cookies)

Important: If the user should ever change their password, you should invalidate all existing long-term authentication tokens for that user.

I don't know if others do that but I see the security benefit. A password change requests a re-login on other devices. See the comments to this answer for example: https://security.stackexchange.com/a/112857

I agree that that it might make sense but I think we need to update the UI to make this clear to the user.

That should just be a transaction, then the order should be irrelevant.

@KN4CK3R please fix the merge conflicts. :tea:

@KN4CK3R why blocked?

Want to wait for https://github.com/go-gitea/gitea/pull/27625#discussion_r1477080995

One more question: if the current login uses an auth token, after deleting all auth tokens, should the current login gets a new auth token ?

Good question. Currently when you change your password in the account settings, you stay logged in because your session still exists but the "remember me" cookie is removed. So when your session times out, you have to login again. In that situation it may be valid to generate a new "remember me" session if it was present before changing the password.
I will have a look if there are some best practices out there.

I could not really find any info how to handle a password change combined with what to do with remember me infos for the local remember me session.  This PR does that to the remember me tokens but I think we can't destroy other (middleware) sessions? There is no consens what should be done with the local (normal) session. Some say the user should re-login, others say otherwise.
OWASP tells to regenerate the local session after a password change but I have enough knowledge if we have a potential problem here. https://cheatsheetseries.owasp.org/cheatsheets/Session_Management_Cheat_Sheet.html#renew-the-session-id-after-any-privilege-level-change
So at the moment, I think it's ok to destroy the local remember me session and require a login after the normal session timed out. It's no big deal to change this behaviour later.

@KN4CK3R please fix the merge conflicts. :tea:

First positive effect of #28733: Changed the changed lines in this PR from +44 -42 to +20 -1 🎉


Fix progress reporting for flycheck

bors r+

That's a great point! We definitely should track the set of open progresses
in the rust-analyzer crate at least
On Thu, 25 Jun 2020 at 12:28, Veetaha<EMAIL_ADDRESS>wrote:

@Veetaha commented on this pull request.
In crates/ra_flycheck/src/lib.rs
https://github.com/rust-analyzer/rust-analyzer/pull/5046#discussion_r445460987
:

           task_send.send(CheckTask::Status(Status::Being)).unwrap();

       }

       CheckEvent::End => {

           task_send.send(CheckTask::Status(Status::End)).unwrap();
       }

This has changed the behavior of the original PR. Progress is a hande,
overwriting it cancel currently active progress, e.g. when the user saved
the file and flycheck started executing cargo check but then she the file
again shortly we need to free the currently active progress resource.
@matklad https://github.com/matklad that was the main reason to extract
Progress handle, not to leak progress resources.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rust-analyzer/rust-analyzer/pull/5046#pullrequestreview-437365349,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANB3M5K4MJ37JRINEGVPG3RYMREXANCNFSM4OH5IZKA
.


nixos/hadoop: init HBase submodule

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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/)
22.05 Release Notes (or backporting 21.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
[x] (Module addition) Added a release notes entry if adding a new NixOS module
[x] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/858

Nice 😊👍

I'm just waiting on my laptop, I'm on phone app version GitHub.

👍

As per discussions on matrix, I'll find someone familiar with hadoop to review this PR

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/566


initial skeleton rust structure

Signed-off-by: Nell Shamrell<EMAIL_ADDRESS>

Forgot some things - closing this for now, will open a new one shortly


block.cc unused variable

https://github.com/NationalSecurityAgency/ghidra/blob/b9e67994b1f37512be3fe4d6d429e891bc8d530e/Ghidra/Features/Decompiler/src/decompile/cpp/block.cc#L826
I'm reporting this instead of just removing it incase it is supposed to be used.

fixed by 601757fb0932e041534b5894cf4b418c1b56c0ad


add setup.py

i want to install a git package with pip, but there is no setup.py, so  added it

it would be very nice to have release version control :D

Thanks! Looks great!
But I have no permission to control this repository any more. Sorry.


chore: rename open_range to isOpen_range, closed_range to isClosed_range

All these lemmas refer to the range of some function being open/range (i.e. isOpen or isClosed).

awaiting-review

Thank you for the review.
bors r+


Replace the BOM symbol when reading a file (closes #212)

/cc @inikulin

FPR

ping @inikulin

lgtm

thanks


Feat - Connect main functions

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add pathname to analytics.page()

Adds the pathname to analytics.page() call
Fix #13

@benjaminhoffman I'm not familiar with this codebase, but I think this is a good way to get the actual path for the segment payload

yeah im not sure the best way for this. i defer to @newhouse for official review now :)

Will have a look as soon as I can.

@xtellurian while this code looks simple enough, I'm struggling to understand the issue that it's solving, and when. Is this about canonical link issues, still?
More importantly, I wonder if this code change will actually break behavior for some users in some cases.
Some more explanation would be appreciated, and also any code samples that I can use to replicate the issue/scenario this is solving for would be great!

@newhouse I'm using this plugin in production. The paths I get in my segment events are all empty. My workaround is to manually call analytics.page()
I don't think this plugin correctly captures the path when analytics.page() is called.
But I can see that analytics.js theoretically adds the path to all page() calls - https://segment.com/docs/connections/sources/catalog/libraries/website/javascript/

@xtellurian we've seen issues in the past around page calls.  it's very strange and I wouldn't rule out segment itself (I worked there as an engineer and we all knew page calls were kinda shaky anyway... we used track calls instead).
see here: https://github.com/benjaminhoffman/gatsby-plugin-segment-js/issues?q=path

@newhouse I'm using this plugin in production. The paths I get in my segment events are all empty. My workaround is to manually call analytics.page()
I don't think this plugin correctly captures the path when analytics.page() is called.
But I can see that analytics.js theoretically adds the path to all page() calls - https://segment.com/docs/connections/sources/catalog/libraries/website/javascript/

Not that the effort is not appreciated, but I think that for now, unless you can provide a simple way to reproduce this that I can use to investigate and understand some more, it's probably not a good idea to merge this in.
And unless the plugin is definitely not allowing segment defaults to do their thing for 100% of users of this plugin, this would be a "breaking change", not a bugfix.
I suspect there is something unique to your setup/use-case that's causing this to be an issue. If you can figure out what that is so I can replicate and investigate, let me know...
Closing for now...


[Enhancement] Profile optimization(1)

Main works of this pr
Attach all the variables that differ from its default value to profile, in json format.
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:disappointed: fail : 25 / 38 (65.79%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/ProfileManager.java
0
1
00.00%
[148]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
0
2
00.00%
[303, 322]

:large_blue_circle:
com/starrocks/load/loadv2/LoadLoadingTask.java
0
2
00.00%
[233, 241]

:large_blue_circle:
com/starrocks/load/streamload/StreamLoadTask.java
0
6
00.00%
[1067, 1068, 1069, 1070, 1071, 1079]

:large_blue_circle:
com/starrocks/qe/SessionVariable.java
21
23
91.30%
[2587, 2588]

:large_blue_circle:
com/starrocks/common/util/RuntimeProfile.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
3
3
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 25 / 38 (65.79%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/ProfileManager.java
0
1
00.00%
[148]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
0
2
00.00%
[303, 322]

:large_blue_circle:
com/starrocks/load/loadv2/LoadLoadingTask.java
0
2
00.00%
[233, 241]

:large_blue_circle:
com/starrocks/load/streamload/StreamLoadTask.java
0
6
00.00%
[1067, 1068, 1069, 1070, 1071, 1079]

:large_blue_circle:
com/starrocks/qe/SessionVariable.java
21
23
91.30%
[2587, 2588]

:large_blue_circle:
com/starrocks/common/util/RuntimeProfile.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
3
3
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 25 / 38 (65.79%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/ProfileManager.java
0
1
00.00%
[148]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
0
2
00.00%
[303, 322]

:large_blue_circle:
com/starrocks/load/loadv2/LoadLoadingTask.java
0
2
00.00%
[233, 241]

:large_blue_circle:
com/starrocks/load/streamload/StreamLoadTask.java
0
6
00.00%
[1067, 1068, 1069, 1070, 1071, 1079]

:large_blue_circle:
com/starrocks/qe/SessionVariable.java
21
23
91.30%
[2587, 2588]

:large_blue_circle:
com/starrocks/common/util/RuntimeProfile.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
3
3
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 25 / 38 (65.79%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/ProfileManager.java
0
1
00.00%
[148]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
0
2
00.00%
[303, 322]

:large_blue_circle:
com/starrocks/load/loadv2/LoadLoadingTask.java
0
2
00.00%
[233, 241]

:large_blue_circle:
com/starrocks/load/streamload/StreamLoadTask.java
0
6
00.00%
[1067, 1068, 1069, 1070, 1071, 1079]

:large_blue_circle:
com/starrocks/qe/SessionVariable.java
21
23
91.30%
[2587, 2588]

:large_blue_circle:
com/starrocks/common/util/RuntimeProfile.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
3
3
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 25 / 38 (65.79%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/ProfileManager.java
0
1
00.00%
[148]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
0
2
00.00%
[303, 322]

:large_blue_circle:
com/starrocks/load/loadv2/LoadLoadingTask.java
0
2
00.00%
[233, 241]

:large_blue_circle:
com/starrocks/load/streamload/StreamLoadTask.java
0
6
00.00%
[1067, 1068, 1069, 1070, 1071, 1079]

:large_blue_circle:
com/starrocks/qe/SessionVariable.java
21
23
91.30%
[2587, 2588]

:large_blue_circle:
com/starrocks/common/util/RuntimeProfile.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
3
3
100.00%
[]

[FE PR Coverage Check]
:disappointed: fail : 39 / 52 (75.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/ProfileManager.java
0
1
00.00%
[148]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
0
2
00.00%
[302, 321]

:large_blue_circle:
com/starrocks/load/loadv2/LoadLoadingTask.java
0
2
00.00%
[233, 241]

:large_blue_circle:
com/starrocks/load/streamload/StreamLoadTask.java
0
6
00.00%
[1067, 1068, 1069, 1070, 1071, 1079]

:large_blue_circle:
com/starrocks/qe/SessionVariable.java
25
27
92.59%
[2592, 2593]

:large_blue_circle:
com/starrocks/common/util/RuntimeProfile.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/ExplainAnalyzer.java
10
10
100.00%
[]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
3
3
100.00%
[]

@Mergifyio backport branch-3.1


Make attributes case-insensitive

Any command that edits/deletes an attribute will work even if the case of the attribute name is different

LGTM we merge


Revamp mina book structure

Turn old RFCs into sections of the mina book
Editorial: try to improve weird structure, inconsistent notation, broken latex, vague language... where possible. No /content/ changes are included in this PR.

Note: this had to target master.

@dannywillems why do you suggest this should target master? I explicitly made it target develop.