Update forward TSN to be fragment aware

When picking a new TSN value only take fragments that are at the start
or end of DATA. Taking the middle value creates a situation where we
expect the other side the re-assumable data, but we have forced them
to evict half the cache on that data already.
Relates to #104

@mturnshek if you get a chance I would try this out. I for sure need a review though, this might effect things negatively I am not aware of.

Codecov Report

Merging #105 into master will decrease coverage by 7.95%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #105      +/-   ##
==========================================
- Coverage   78.47%   70.52%   -7.96%
==========================================
  Files          43       43
  Lines        2723     2724       +1
==========================================
- Hits         2137     1921     -216
- Misses        454      653     +199
- Partials      132      150      +18

Flag
Coverage Δ

#go
?

#wasm
70.52% <66.66%> (-0.03%)
:arrow_down:

Impacted Files
Coverage Δ

association.go
67.06% <66.66%> (-15.72%)
:arrow_down:

association_stats.go
69.23% <0%> (-30.77%)
:arrow_down:

error_cause_protocol_violation.go
63.63% <0%> (-18.19%)
:arrow_down:

chunk_forward_tsn.go
80.85% <0%> (-14.9%)
:arrow_down:

error_cause_header.go
89.47% <0%> (-10.53%)
:arrow_down:

error_cause.go
24.44% <0%> (-8.89%)
:arrow_down:

rtx_timer.go
89.53% <0%> (-8.14%)
:arrow_down:

stream.go
83.67% <0%> (-6.81%)
:arrow_down:

chunk_abort.go
53.12% <0%> (-6.25%)
:arrow_down:

... and 3 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 7219116...585060f. Read the comment docs.

@Sean-Der PR #107 should address this issue as discussed in #104.

Closing for the better PR #107


Update ruby version for security fix

See. https://www.ruby-lang.org/en/news/2015/08/18/ruby-2-2-3-released/

Merged as 50cb7aee44a95f6242c596ff00d8af68c72c219c. Thanks!


Unbreak HKModal

When we added transitions, it broke our handling of onClick events emanating from the buttons passed into the footer. This actually exposed the fact that we had become uncontrolled.
Solution, make it controlled.
There's a nice doc on the state flow in the HKModal component for your reading pleasure.
Also update stories to properly consume the button events.

I tested in my DHC branch at it seems to have solved the issue I was having


[FEATURE] Hover color for links on open source roadmap

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.
On the open source roadmap page, it is not immediately clear to a sighted user that the roadmap steps are links to subsequent pages.
This issue is to add a color on hover (dark and light mode) to enhance discoverability of these pages.
Page: https://www.biodrop.io/docs/open-source-roadmap
Description
add hover CSS to the links as described above - they should remain consistent with the style of hover colors throughout the project
Screenshots
No response
Do you want to work on this issue?
No
If you want to work on this issue...
No response

Great suggestion Amanda 👍

Please assign this issue to me

please assign me this issue.

Hey, can you please assign this to me?

I would like to address the issue too. @amandamartin-dev

@eddiejaoude I have raised a PR on this issue. Kindly Review it.

@amandamartin-dev I wanted to work on this issue, so I mentioned this in discord too.....please assign this to me😶, this would be my first contribution😀

What is going on here? This is not collaboration 😢 - please read the contributing guide before commenting (note: that tagging maintainers or commenting in Discord or raising a pull request before it is assigned does not improve the chances of getting assigned)
@HardikChoudhary24 you were the first to request the issue plus as this looks like your first contribution to the project and this issue is a "good first issue" I have assigned it to you


Update EmojiPrinter.php

StyleCI

A new inspection was created.

Thanks!
Unless I'm mistaken, the common convention is function() rather than function () and StyleCI has this one wrong.
I tried to find the setting for that, or what to add to .styleci.yml, but was unsuccessful.
I'd prefer to go that route, I think.
If you're game to investigate, that'd be awesome.
Either way, I appreciate the interest!


Tooltip on send form has broken styles

Did we need all those tippy styles I removed in the global-styles? I'll check ...it appeared they were unused but I'll put them back if needed.

That could well be the issue 😂


hostname - fully qualified?

Will there be any change to the hostname format foo.postgres.database.azure.com?  It has been our observation that for PITR recoveries the hostname is not enough for the reverse proxy and connections attempted to bar.postgres.database.azure.com result in a connection to the original foo.postgres.database.azure.com unless the host portion of the admin user (admin@foo) is altered to (admin@bar). Now that there is no host portion (just "admin", say) can we rely upon the correct server being the subject of the connection?
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 3e4deba6-4073-0174-e4cf-ee02458fb764
Version Independent ID: 13b2c9e1-283b-41b7-99f2-9008e53ee7f3
Content: Compare Azure Database for PostgreSQL - Single Server and Flexible Server
Content Source: articles/postgresql/flexible-server/concepts-compare-single-server-flexible-server.md
Service: postgresql
Sub-service: flexible-server
GitHub Login: @AwdotiaRomanowna
Microsoft Alias: alkuchar

@mckenzm
Thanks for your feedback! We will investigate and update as appropriate.

Hi @mckenzm , Single and Flex are built on totally different architecture. The behavior you observed in single, i.e. relying more on the portion of user name than on the host name was the consequence of the gateway in front of each server. The Flexible Server does not use the gateway, so host name is fully respected and user name does not decide any more to which host to connect to.

#please-close


test: Tweaked output for deploy_marathon_app

A marathon app is considered deployed when all instances are running and healthy. To simplify debugging, we should check those cases separately and print different debug messages.

LGTM

needs the 1 comment and code quality fix (you can run tox -e py34-syntax to check the code quality checks locally.

Updated the patch, addressed Cody's comment.

Replaced with #780 which fixes the syntax errors.


Look into changing request comment to TEXT

So, it's currently a LONGTEXT, which is 2^32 -1 bytes (4 GB max of text). This is excessive.
Can we fit all existing data in a TEXT (2^16 -1 = 65535 bytes)?

I'm looking at reducing this to 3000 chars - which is approximately equivalent to the highlighted text:

Does this seem long enough?
Note that there are 36 requests in the production system which exceed this length, which will be truncated by this change. All of them appear to fall broadly into these categories:

Full page (main page, request form, account creation form, block notice) copy/paste into comment box
Spam (potato potato potato ..., "our services include...")
Essay about what's wrong with Wikipedia, biography
Foreign-language content which I can't read

I'm looking at reducing this to 3000 chars - which is approximately equivalent to the highlighted text:

Does this seem long enough?
Note that there are 36 requests in the production system which exceed this length, which will be truncated by this change. All of them appear to fall broadly into these categories:

Full page (main page, request form, account creation form, block notice) copy/paste into comment box
Spam (potato potato potato ..., "our services include...")
Essay about what's wrong with Wikipedia, biography
Foreign-language content which I can't read

(that's 36 / 103885  == 0.03%)

(that's 36 / 103885  == 0.03%)

I'd like to see those requests. :p

I'd like to see those requests. :p

3000 is more than enough.  I'd argue 1000 is perfectly sufficient.  I don't want to read essays for requests.

1k would truncate 185 requests, including useful data in some of those comments (blocked username, for instance).


improve logging in ci mode: at least know which test is executing and how many remain

now we have:
[STATUS] [0:12] [9x pending, 1x running, 1x completeSuccess, 1x completeFailureOrError] [Test=simple test group deliberately flaky test]


Pointcloud

Is there a way to get a point cloud out of the depthai-core interface?  None of the examples showed how to do that.

So DepthAI does not generate a point cloud by default.  We have examples of using Open3D to take the depth data and project this into a point cloud, but nothing native (at least at this time).

Brandon - the Open3D example would be great. Is it on Github?
Bob Davies

Looking.  I think we only have it for Python.  Not sure if C++ would be similar and/or easily portable.  CC: @saching13

Using Open3D code would be pretty much similar.
And we don't have an official example that converts the depth to pointcloud but I can point you to some examples.

You can do it easily using opencv function here
Use PCL Library
Here is the raw code which converts depth to PointCloud using Eigen that I had written for some internal tests.

Sachin - thanks for the response.  I will try to use the OpenCV function
reprojectImage that you suggested.
Bob Davies

Actually, the Eigen solution looks better.  I will give that one a try first....

Actually, the Eigen solution looks better. I will give that one a try first....

Sounds good. Just a heads up. Make sure to Cherry Pick the DepthAlignment class and use it on top of gen2 pipeline.
The link contains old gen1 version of depthai.

Sachin - what is the best way to "get_right_homography" using the latest
depthai-core API's?
Bob Davies

Sachin - what is the best way to "get_right_homography" using the latest depthai-core API's? Bob Davies
…

It needs to be done similar to the H_left here

We are adding on-device pointcloud generation support in the near future. More information here.
Thanks, Erik


Add minAllowed resource limits for hvpa controller

Prevent hvpa controller to be OOM killed.
As this is deployed per Seed the resource consumption widely varies. We tried to find a good trade-off between availability and cost.
Very large Seeds can use up to 600 Mib of memory and 2 CPU but we think that this is the resource usage under load and with this PR we just want to provide enough resources for the container to start up.
cc @vpnachev
/area control-plane
/area auto-scaling
/area availability
/kind enhancement
Add minimum resource limit for the hvpa controller vpa

/invite @ggaurav10 @amshuman-kr


Auto SuggestBox in PageHeader?

is it possible to add an Auto SuggestBox in the PageHeader just like on the News app?

It seems that in news app the autosuggestion box is right of the page header. So if you split the grid, you could put the autosuggest box to the second column. Otherwise you could put it in the page headers content

I want to point you in a direction different than where you are likely headed. There is 0% value in putting this inside the CommandBar. There's no reason not to simply layout the control to the right of the CommandBar/PageHeader instead. Does this make sense?


Add PowerShell

Previous PR: https://github.com/sindresorhus/awesome/pull/764
https://github.com/janikvonrotz/awesome-powershell
**A curated list of delightful PowerShell modules and resources **
ALL THE BELOW CHECKBOXES SHOULD BE DONE AND CHECKED AND THE ABOVE PLACEHOLDERS SHOULD BE FILLED IN BEFORE SUBMITTING A PULL REQUEST!

[x] I have read and understood the contribution guidelines and the instructions for creating a list.
[x] This pull request has a descriptive title. (For example: Add Node.js)
The list I added

[x] has been around for at least 20 days,
[x] is a non-generated Markdown file in a GitHub repo,
[x] is not a duplicate,
[x] only has awesome items with descriptions (Awesome lists are curations of the best, not of everything),
[x] contains the awesome badge on the right side of the list heading,
[ ] has a Table of Contents section named Contents as the first section,
[x] has an appropriate license (Meaning something like CC0, NOT a code licence like MIT, BSD, Apache, etc),
[x] has contribution guidelines,
[ ] has been checked for inconsistent formatting, spelling, and grammar (This includes starting each link description with an uppercase character and ending it with a period ., and consistent naming),
[x] has been added at the bottom of the appropriate category.

Feedback:

I've unchecked some checkboxes that are clearly not done. E.g. some descriptions are missing . at the end.
Drop the Inspired by the awesome list thing. text.
contribute.md => contributing.md
Drop all the A / An prefixes in the descriptions.
There should be a succinct summary at the top about the project.
In A curated list of delightful PowerShell packages and resources., PowerShell should be linkified to https://en.wikipedia.org/wiki/PowerShell
Use the SVG version of the CC0 license badge.
Misc section should be at the bottom.
Put the SharePoint and SQL Server sections at the bottom (above Misc).

ping

Merge conflict also needs to be fixed.

yes, I submitted all your feed back to the original repo. I don't think there is any news.
I am filled up with course work and my own stuffs. I will close this and reopen a PR later.


Domain changed

I'm getting 343.6 now with this update as well BTW: https://tools.pingdom.com/#600f3dbec9800000

You're right. I've updated the size.


Adding Unit tests to AdServiceTest.java

Adding Unit tests to AdServiceTest.java, aimed to improve test coverage

test


Any way to reload the database and make all observables to re-query?

Reload SQLiteOpenHelper and re-query
In my app I have a import/export functionality, by importing, the SQLite database file is replaced by one on the sdcard, so I need to reload the SQLiteOpenHelper and then notify all the Queries.
Something like the following method would be really handy!
public void reloadDatabase(SQLiteOpenHelper helper, Set<String> tables){
   // Replace the SQLiteOpenHelper
   synchronized (databaseLock) {
      readableDatabase = null;
      writeableDatabase = null;
      this.helper.close();
      this.helper = helper;
   }
   // Make all Queries to reload
   sendTableTrigger(tables);
}

Or prehaps BriteDatabase can save a Set of tables, filled as we query them, to be notified on a potential database reload?
I guess I could make the user restart the app, but I rather not.
SQLBrite is amazing as it is very simple and allow us to get rid of loaders, and IMO notifying on tables is far better than the Uri system, thanks for creating SQLBrite!
Re-query only
Let's say my app is a calendar, I have an upcoming event list, starting by today's events.
In every event list I want to mark each yesterday's event as missed.
Then I need to reload event's Queries every day at 00:00.
In this particular case it would make sense to make the sendTableTrigger() public and call it every day.
I could also use a Observable.combineLatest with a Query and a PublishSubject and call this Subject every day to run the Query again, but since table triggers are managed the same way, it seems simple to switch sendTableTrigger() to public.
To support this idea; ContentResolver's notifyChange(Uri uri, null) is public.

I needed the same thing, so I just exposed the triggers.
public PublishSubject<Set<String>> getTriggersSubject() {
     return triggers;
}

Would be great if this would be integrated, in the main repos.
But I am not sure, whether this is is the smartest solution.
https://github.com/xMikeTx/sqlbrite/commit/baffb9cdb7ef8243257db8522a6311dc2825baa7

As to the re-query case, you can run a query more than once so you can use something like combineLatest with another stream that emits a new dummy value every day at midnight.
Observable.combineLatest(queryObservable, midnightObservable, (q, m) -> q)
   .subscribe(query -> /* .. */);

In this instance midnightObservable could be a PublishSubject that sent out an item when an AlarmManager fires.
I'm not super sure about how widespread the use of reloading helpers is. One solution would just be to replace you BriteDatabase with an Observable<BriteDatabase> and flatMap to create the query observable.
Observable<BriteDatabase> dbObservable = /* .. */
dbObservable.flatMap(db -> db.createQuery("SELECT * FROM table", "table"))
    .subscribe(query -> /* .. */);

By using Observable<BriteDatabase> instead of a BriteDatabase I ended up duplicating createQuery:
static Query createQuery(Iterable<String> tables, String sql, String... args){
   return dbObservable.switchMap(db -> db.createQuery(tables, sql, args);
}

It's not working in my app due to some weird SQLiteDatabaseCorruptException: database disk image is malformed (code 11), probably unrelated, so it's a fine solution, thank you.
About the re-query, I keep thinking making sendTableTrigger public is better than rewriting a PublishSubject to trigger a reload, by anyway you're the boss.
If you feel these are the right solutions, then this issue is closed for me.
Thank you!

It's actually not that easy to accept an Observable<SQLiteOpenHelper> in SqlBrite because of the synchronous APIs that it offers. I'll have to think about it more to see if there's anything we can do.


dependencies.yaml: Update verify tooling to latest versions (part 1)

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
dependencies.yaml: Update verify tooling to latest versions

Update k/repo-infra to v0.2.2
Update zeitgeist to v0.3.0

/assign @saschagrunert @cpanato @puerco
cc: @kubernetes-sigs/release-engineering
Which issue(s) this PR fixes:

Special notes for your reviewer:
Continued in https://github.com/kubernetes-sigs/k8s-container-image-promoter/pull/394.
Peeling these commits out, as the initial PR led to more linting than I initially expected.
Does this PR introduce a user-facing change?

NONE

/test pull-cip-unit-tests

Second flake I've seen due to reqcounter.
The first is here.

/check-cla

/hold since https://github.com/kubernetes-sigs/k8s-container-image-promoter/pull/394 was approved and contains these commits
We can close this out if/when https://github.com/kubernetes-sigs/k8s-container-image-promoter/pull/394 passes tests and merges successfully.

(That's a cool feature! GH auto-closed this PR because it contained a subset of the commits within https://github.com/kubernetes-sigs/k8s-container-image-promoter/pull/394.)


main.dart问题

lib/main.dart:1: Warning: Interpreting this as package URI, 'package:flutter_weather/main.dart'.
file:///G:/SOFT/Flutter/flutter/packages/flutter/lib/src/rendering/box.dart:6:8: Error: Not found: 'dart:ui'
import 'dart:ui' as ui show lerpDouble;

你这是flutter环境没装好吧，报错显示你dart库里面都没有ui包


Please explain how to configure a keyboard shortcut

OK, I feel stupid, but can you please tell me exactly how to configure a keyboard shortcut to insert the ISO date in Sublime Text 3?
I have added:
// iso date (YYYY-MM-DD)   { "keys": ["ctrl+shift+f5", "ctrl+shift+d"],     "command": "insert_date",     "args": {"format": "%Y-%m-%d"} }
To my Default (Linux) sublime-keymap - User file, but "ctrl+shift+f5" does not do anything.
What exactly am I doing incorrectly?
Thank you,
GG

You have to press both key chords in succession. First, Ctrl+Shift+F5 and then Ctrl+Shift+D.

Ahhh I see said the blind man - thank you.


Add storage mirror firestore

Add storage mirror firestore source code. Credit goes to @jsdt and @kevinthecheung

[x] The new extension's id roughly follows the pattern
[firebase-product]-[action]-[noun], like storage-resize-images
[x] The following match the extension's id:

[x] The folder that contains the extension (my-new-extension/)
[x] The name field of my-new-extension/functions/package.json
[x] The name field of my-new-extension/extension.yaml
[x] A new entry in the packages array of the lerna.json file in the
repository's root

[x] I've added a generate-readme script to
my-new-extension/functions/package.json that matches the other
extensions in the repository
[x] I've run the following npm commands at the root of the repository:

[x] npm install
[x] npm run generate-readmes
[x] npm run format

@dackers86 @jhuleatt thanks for the feedback. All addressed and PTAL

overriding CLA bot - it got confused by an alternate email for @dackers86


Add Support for RabbitMQ Stream Java Client

New listener container for the stream API to support invoking @RabbitListener etc.
https://github.com/rabbitmq/rabbitmq-stream-java-client

Is there a tentative date for 2.4 being release with this being present? I see the last Milestone was 25 days ago.

Next milestone is Sept 20; with release candidate in October and GA in November, aligned with the Boot 2.6 RC and GA.
https://calendar.spring.io/


Test HTTP support

Currently, I don't really know whether the HTTP server actually works and have no client to test with.

you might consider writing a simple Python testbed client or trying to make it work with coc.nvim or other semi-universal language clients.


SOR | Docker compose#23

Summary
Added a docked-compose file to start the SOR with KUPO backend.
Related to:

#23

Type of Change

New feature (non-breaking change which adds functionality)
Documentation

Checklist
N / A
Testing
I have tested the docker-compose file locally.

TODO

Add the configuration repo as submodule.
Clean up
Add documentation

@Haluk-GeniusYield
I took care of the remaining TODO points. Now the PR is ready to be reviewed.

There is a missing backslash in the existing Kupo backed instance command that prevents it to work as is.
Maybe this could be fixed in this PR to avoid an issue+PR just for that:
https://github.com/geniusyield/smart-order-router/blob/230c97969760cb7610c0846688bbe4de5268592c/README.md?plain=1#L190

Maybe this could be fixed in this PR to avoid an issue+PR just for that:

Nice catch. Thanks for the comment.
I took care of this as well, as you proposed it:

8cec30189e42ede4b4612e32cdbb6e527dcf77a2


Install directory durable

Resolved mistake https://github.com/intersystems-community/zpm/issues/200


Extract error raising into separate class

Coverage increased (+0.02%) to 99.355% when pulling 2e36f7b45e607e5de39d6943d6554c5298b94d44 on better_errors into 8f84e40de4df1f942511d0fd4c013bb6185d4909 on master.

Coverage increased (+0.02%) to 99.355% when pulling 2e36f7b45e607e5de39d6943d6554c5298b94d44 on better_errors into 8f84e40de4df1f942511d0fd4c013bb6185d4909 on master.


feat: port List.length_injective_iff and List.length_injective

Replace commented-out Lean 3 List.length_injective_iff and List.length_injective lemmas, port them to Lean 4.
Compare to the current mathlib3 versions: https://github.com/leanprover-community/mathlib/blob/6f1ad8258079beb8437c2e402d4218f4e6b7f9f6/src/data/list/basic.lean#L256-L266

I tried translating induction l1 generalizing l2; cases l2 to induction l1 generalizing l2 <;> cases l2, but then I didn't know how to get access to the induction hypothesis. In the Lean 3 version, it gets the automatically-generated name l1_ih, but in Lean 4 it doesn't seem to have a name, and I don't know how to give it one. The translation I'm proposing here is more verbose and explicit, which is maybe not a bad thing.

The error that I get when I keep @[simp] on length_injective is
/- The `simpNF` linter reports:
SOME SIMP LEMMAS ARE NOT IN SIMP-NORMAL FORM.
see note [simp-normal form] for tips how to debug this.
https://leanprover-community.github.io/mathlib_docs/notes.html#simp-normal%20form -/
-- Mathlib.Data.List.Basic
#check List.length_injective.{u_1} /- Left-hand side simplifies from
  Function.injective List.length
to
  Subsingleton α
using
<EMAIL_ADDRESS>Try to change the left-hand side to the simplified term!
 -/

I tried translating induction l1 generalizing l2; cases l2 to induction l1 generalizing l2 <;> cases l2, but then I didn't know how to get access to the induction hypothesis.

I found a way:
induction l1 generalizing l2 with | nil => ?_ | cons _ _ ih => ?_ <;> cases l2

The latest commit has it this way. It's debatable whether this is good style, but it does have the advantage of closely matching the mathlib3 proof.

The error that I get when I keep @[simp] on length_injective is

That's because Lean 4 tries simp lemmas in a different order.  Lean 3 tried the last simp lemma first.  I'm not sure if we want to guarantee any particular order in Lean 4 (as this is not entirely trivial because the discrimination tree indexing shuffles the lemmas a bit).  Here's a short demo that illustrates the difference:
def f := 0
def g := 0

@[simp] theorem a : f = g := rfl
@[simp] theorem b : f = 0 := rfl

example (p : _ → Prop) : p f := by simp
-- in Lean 3: p 0
-- in Lean 4: p g

fyi @digama0
We can get the Lean 3 behavior by manually specifying a priority on the simp lemma using @[simp default+1] theorem length_injective.  But in this case I would suggest to add the following simp lemma instead:
@[simp] theorem subsingleton_of_subsingleton [Subsingleton α] : Subsingleton α := ‹_›

Instead of ·, you can use (guessed) case cons ih => to select a case by (suffix) name and at the same time rename inaccessible variables.

Instead of ·, you can use (guessed) case cons ih => to select a case by (suffix) name and at the same time rename inaccessible variables.

Done. Do there exist or-patterns, so that i could collapse these two cases?
    case nil.cons => cases hl
    case cons.nil => cases hl

We can get the Lean 3 behavior by manually specifying a priority on the simp lemma using @[simp default+1] theorem length_injective.

Done.

But in this case I would suggest to add the following simp lemma instead:
@[simp] theorem subsingleton_of_subsingleton [Subsingleton α] : Subsingleton α := ‹_›

This sounds to me like something that should be backported to mathlib3 before we put it in mathlib4.


[RISCV] NotImplementedError when using mpt2c with branch fixing

When trying to use the mpt2c tool to generate a C source file from an MPT file (generated using chopstix) a NotImplementedError exception is thrown, corresponding to the line 838 of src/microprobe/passes/branch/__init__.py
The MPT file in question is this one. The command to generate the C source file is as follows: mp_mpt2c.py -T riscv_v22-riscv_generic-riscv64_linux_gcc --fix-indirect-branches --fix-branch-next --fix-memory-registers --endless -t bt.mpt -O /tmp/temp.c
The last instruction in the BBL is JALR (JALR_V0) Description: Jump and Link Register: Jump to rs1 plus the 12-bit signed immediate while saving PC+4 into rd.

Problem detected and solved: the implementation of the corner case where the last instruction is a branch without a target set was missing. For now, if that's the case the instruction is converted to a nop since branch to next does not make sense (no next instructions) and branch to the first instruction creates an endless loop without the user expecting it.
Fixes will be released in the following repo update.


Is stream join support subquery？

two streams，Take the latest value within the window time，then add latest values。
error message is：

No reason can be found according to the error message
Is the syntax of SQL statement wrong?

@nickscut Subquery is not supported. We recommend to use "rule pipeline" which you can extract your subquery as a rule and sink the result to a specific mqtt topic. Then your target rule can retrieve those "medium" results from that mqtt topic with  another stream. In the next release 1.4.0, we will introduce an inMemory source to bridge between rules, check rule pipeline for more detail.


Refs #5039. Fixed max_sample at datareader

Fixed an issue with the Delivery control with samples unlimited value.

Build status:

Linux
Windows


Adding setting documentation for initialOffset in the Kafka components

Docs for dapr/components-contrib#1120

/cc @orizohar The PR for this is merged


Support detail parameter of deploy

Thank you for providing a useful GitHub Action!! :)
Overview
This PR is to support detail parameter of deploy API.
This action uses speedcurve 2.0.5, and  the speedcurve package supports detail parameter (ref1, ref2).
Test Result
I tested the updated action on my repository.
https://github.com/negibokken/speedcurve-test-action/actions/runs/3546695233/jobs/5956003537

@josephwynn-sc Could you review this PR?


Add unit tests for token embedder and seq2labels model

This PR adds tests for the token embedder in GECToR, the Seq2Labels model and updates the tokenization tests so that they conform to AllenNLP tests.
To test, please set up the environment and run the following command from the project root directory i.e. gector/.
pytest -v tests

LGTM!!! Great work!

Tried running the CI plan with Python versions 3.9 and 3.10 as well.
Since currently, this repo uses allennlp=0.9.0 which in turn uses spacy=2.1.9, it is incompatible with Python v3.9 and v3.10. This will be added to the CI plan in subsequent PRs once allennlp is unpinned.


the Directory plugin is not working in DS Langflow

Bug Description
I am trying to read files from a directory, using the Data>Directory plugin, in Datastax Astra Langflow.
The Directory plugin doesn't do anything when the user clicks on the Path box.
In theory, one expects that a filw browser opens and allows the user to choose a directory on the local filesystem, like it is the OSS Langflow.
Can you please comment if this plugin should work like in the OSS Langflow or not?

Reproduction

open Langflow on Datastax Astra portal.
open a generic flow or create a new one
drag in the Data>Directory plugin and try to read files from a directory.

Expected behavior
the Directory plugin should allow users to import files from a dir
Who can help?
No response
Operating System
ASTRA
Langflow Version
1.0.9-post1
Python Version
None
Screenshot

Flow File
No response

@fanimali this is a great point - I'm working on improvements to the File Component, that allow uploading a zip and recursively parsing the zip. I think this is a good workaround right now (this is already merged into GitHub).
The directory component will likely be retired... the basic issue is it requires a path, but the path is a string - and relative to the machine Langflow is running on. As you noted. The file component will take its place. Now, you're right however that the ideal UI would not be uploading a zip, but allowing selection of a directory. With that in mind, there still needs to be UI work to allow that - the zip solution is somewhat of a workaround.
TLDR:

Current workaround - upload a zip with the new file component, and all files will be recursively processed.
Pending improvement - allow multiselect / uploading of a directory in the file component

Once #2 is complete, i think we can deprecate / remove the directory component.


Revert "Revert "unix,fs: fix for potential partial reads/writes" (and fix it)

This is a replacement for #640, which needed to be reverted (https://github.com/libuv/libuv/pull/640#issuecomment-359026693) due to incorrect handling of partial-reads. It's very tempting to think that read and write should be symmetric with respect to signals and error handling, but they aren't (and fortunately the nodejs test suite noticed that before getting too far).
The key distinction is that writes get placed in a queue, while reads do not. And, as a corollary to that, partial reads can be useful for interactive usage, while omitting the middle of a write was simply a bug.
Note: I tried to approximately split my commits into a revert of the revert and then my fixes, but I wanted to keep test-eintr-handling.c (which was deleted in the original PR), so it's not a complete revert. These should probably be squashed in the final commit. It would also be nice to recognize Ben Wijen's contributions and authorship (14bfc27e641aff178c431083c0c0eada4d6f02dd) in the merge commit.

bump. Can I get a CI run here?

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/767/

New CI since the last one was cancelled - https://ci.nodejs.org/view/libuv/job/libuv-test-commit/770/

Sorry, I forgot to push my attempted fix for that test (for machines where iovmax > "alot of bufs") after the last CI run. Hopefully this will test correctly on Windows now.

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/771/

Looks like the zOS fs_write_alotof_bufs failure could be legit.

Yes, but I don't have z/OS to determine why readv is not returning the expected amount of data.

Yes, but I don't have z/OS to determine why readv is not returning the expected amount of data.

/cc @libuv/zos

Will look into this in a bit

Use this line from src/unix/core.c in test/test-fs.c
#include <limits.h> /* INT_MAX, PATH_MAX, IOV_MAX */

bump.

Request reviewer's attention to this PR: a node.js unit test case revealed silent partial write issue. Here is a minimal reproduce and some relevant info:
#cat 16601.js
const fs = require('fs');
const ksm = process.binding('buffer').kStringMaxLength;
const stream = fs.createWriteStream('./big.txt')
const a = Buffer.alloc(ksm, 'a');

stream.write(a, () => {
  stream.end();
})
stream.on('finish', () => {
  const len = fs.readFileSync('./big.txt').length
  console.log(`attempt: ${ksm}, real: ${len}`)
})

#ulimit -f 1024
#node 16601.js
attempt:<PHONE_NUMBER>, real: 1048576
strace showed that only one attempt was made to write, the failure was neither noticed nor propagated.
[pid  9464] write(11, "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"...,<PHONE_NUMBER>) = 1048576
[pid  9464] write(8, "\1\0\0\0\0\0\0\0", 8) = 8
[pid  9464] futex(0x21b5704, FUTEX_WAIT_PRIVATE, 8, NULL <unfinished ...>
[pid  9457] <... epoll_wait resumed> {{EPOLLIN, {u32=8, u64=8}}}, 1024, 7999) = 1
[pid  9457] read(8, "\1\0\0\0\0\0\0\0", 1024) = 8
[pid  9457] futex(0x21b5704, FUTEX_WAKE_OP_PRIVATE, 1, 1, 0x21b5700, {FUTEX_OP_SET, 0, FUTEX_OP_CMP_GT, 1}) = 1
[pid  9465] <... futex resumed> )       = 0
[pid  9465] futex(0x21b56c0, FUTEX_WAKE_PRIVATE, 1) = 0
[pid  9465] close(11 <unfinished ...>

libuv in Node CI: https://ci.nodejs.org/view/libuv/job/libuv-in-node/19/
@gireeshpunathil have you verified that this PR fixes the issue?

yes, I have verified, it works - i.e, throws error (which is expected when writing too big data under too low ulimit).

@santigimeno If it's the difference between approving this PR and not, would you be willing to fix up the nits yourself, push directly to the branch, and land this? It would help us close a bug in Node.js.

If it's the difference between approving this PR and not, would you be willing to fix up the nits yourself, push directly to the branch, and land this? It would help us close a bug in Node.js.

The PR LGTM and I can fix up the nits, but would like some other @libuv/collaborators to look at it first.

Bump again.

bump bump bump

bump x4. I've rebased and updated it to address all comments.

Desperately trying to get some reviews here.../ping @bnoordhuis @saghul @ronkorving @santigimeno @tniessen @cjihrig

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/1040/
Libuv in node: https://ci.nodejs.org/view/libuv/job/libuv-in-node/64/

CI looks good. The failures are known. I'm landing

Landed in https://github.com/libuv/libuv/commit/19a341919546140c37834488e1f7c46721b0d2d5 and https://github.com/libuv/libuv/commit/60abdbaed6c20a41678cd9af1d6f40a1d11bf5e2. Thanks @vtjnash and sorry it took so long.

Thanks! I'll need to review my patch list now and see what this resolves :) Hopefully this attempt doesn't trigger more nodejs issues—ping me if it does.

This might have introduced a minor bug in AIX, please check #2023 for a possible fix.


k+1 - sized group members get assigned twice

When minimum group size is set to k it often happens that some groups will have size k+1. Some members in the k+1 - sized group are assigned twice.
This error should occur very early when construction "valid" groups, i.e. sublists from the list of participants.

Bug could not be replicated after adding 'id' column to faulty csv file.


Update demo to add AVIF test images and AVIF animated images

Some users seems has issues on AVIF/AVIFS codec integration, here is the example and show it's functions.

This close #291


When will the new version be released for UVR?

When will the new version be released for UVR?

What exactly will the new version include?

Please download the latest version of UVR. The change log details all the changes made.
Link:
https://github.com/Anjok07/ultimatevocalremovergui#installation


Add Direction Feature when mode is a modal

Default Direction vertical

other values direction (vertical ,horizontal, leftToRight ,topToBottom ,none, zoomIn)

a fade more would be much better, which is one of the animationType of Modal

When submitting or proposing new features, please update the documentation to allow users to easily implement the feature and help project maintainers understand your goals for the proposed feature. Thanks for contributing!

@matthamil sorry i forget to update documentation

Would love to see this feature merged before 1.0! Is it just documentation preventing the merge?

This looks like a reasonable approach, but I'm not sure it will solve a lot of people's requests. Instead of configuring the whole navigator, I've noticed that a lot of people are requesting to configure one screen at a time.

@ericvicenti I think this it reasonable no matter it's for the stack or screen, the workaround for screen modal is actually a stack too, right now I have to implement my own modal navigator for each project only for a fade transition

I agree with @ericvicenti that it should work on per-screen basis, setting animation type on entire navigator doesn't solve much of my pains.
Also, PR needs to be rebased.

I will rebuilt it with per-screen ;)
@ericvicenti @grabbou

👍 Looking for this as I'm currently implementing a filter modal.

Any news on this?

Looks like this PR currently needs some revision, there are conflicting files.

thank you for opening this PR! at the moment we are in the process of re-thinking how we do modals in react-navigation. you can comment on https://github.com/react-navigation/rfcs/issues/15 with ideas or open up a RFC if you have a plan for how you would like this to work!
unfortunately we can't ship the code in this PR in its current form because it doesn't merge cleanly with master. I apologize that there wasn't enough follow-up on this PR to begin with, we're trying to do better at handling pull requests in a more timely manner going forward. apologies to anybody who hoped this would land -- please feel free to open a feature request on http://react-navigation.canny.io/ or participate in the RFC if this is important to you!


Scene.enablePhysics() needs gravity set as optional

Hello,
First PR here, please tell me whatever's wrong.
Fix description :
The first argument for gravity in Scene.enablePhysics() isn’t set as optional, yet this guide from the documentation demonstrates how this parameter isn’t required at all.
In fact, not giving any argument does work, but as I’m using Typescript, I noticed it throws a warning when no arguments are given (“1-2 expected”)

Thanks !!!


Reconstructing signal with the nfft result

I have a signal r evaluated at time z which looks like the below graph for each time step z.
r = np.array([119.75024144,<PHONE_NUMBER>3,<PHONE_NUMBER>6,<PHONE_NUMBER>8,
<PHONE_NUMBER>1,<PHONE_NUMBER>3,<PHONE_NUMBER>8,<PHONE_NUMBER>7,
<PHONE_NUMBER>8,<PHONE_NUMBER>3,<PHONE_NUMBER>7,<PHONE_NUMBER>5,
<PHONE_NUMBER>7,<PHONE_NUMBER>5,<PHONE_NUMBER>8,<PHONE_NUMBER>6,
<PHONE_NUMBER>3,<PHONE_NUMBER>1,<PHONE_NUMBER>6,<PHONE_NUMBER>2,
<PHONE_NUMBER>9,<PHONE_NUMBER>7,<PHONE_NUMBER>9,<PHONE_NUMBER>2,
<PHONE_NUMBER>8,<PHONE_NUMBER>4,<PHONE_NUMBER>7,<PHONE_NUMBER>2,
<PHONE_NUMBER>7,<PHONE_NUMBER>4,<PHONE_NUMBER>8,<PHONE_NUMBER>1,
<PHONE_NUMBER>4,<PHONE_NUMBER>2,<PHONE_NUMBER>3,<PHONE_NUMBER>9,
<PHONE_NUMBER>9,<PHONE_NUMBER>8,<PHONE_NUMBER>3,<PHONE_NUMBER>8,
<PHONE_NUMBER>6,<PHONE_NUMBER>8,<PHONE_NUMBER>2,<PHONE_NUMBER>8,
<PHONE_NUMBER>7,<PHONE_NUMBER>1,<PHONE_NUMBER>7,<PHONE_NUMBER>1,
<PHONE_NUMBER>2,<PHONE_NUMBER> ,<PHONE_NUMBER>5,<PHONE_NUMBER>2,
<PHONE_NUMBER>9,<PHONE_NUMBER>6,<PHONE_NUMBER>2,<PHONE_NUMBER>7,
<PHONE_NUMBER>7,<PHONE_NUMBER>3,<PHONE_NUMBER>9,<PHONE_NUMBER>9,
<PHONE_NUMBER>7,<PHONE_NUMBER>1,<PHONE_NUMBER>1,<PHONE_NUMBER>7])

z = np.array ([-422.05230434, -408.98182253, -395.78387843, -382.43143962,
       -368.92341485, -355.26851343, -341.47780372, -327.56493425,
       -313.54536462, -299.43740189, -285.26768576, -271.07676026,
       -256.92098157, -242.86416227, -228.95449427, -215.207069  ,
       -201.61590575, -188.17719265, -174.89201262, -161.75452196,
       -148.74812279, -135.85126854, -123.04093538, -110.29151714,
        -97.57502515,  -84.86119278,  -72.1145478 ,  -59.2947726 ,
        -46.36450604,  -33.29821629,  -20.08471733,   -6.72030326,
          6.80047849,   20.48309726,   34.32320864,   48.30267819,
         62.393214  ,   76.56022602,   90.76260159,<PHONE_NUMBER>1,
<PHONE_NUMBER>9,<PHONE_NUMBER>9,<PHONE_NUMBER>9,<PHONE_NUMBER>9,
<PHONE_NUMBER>3,<PHONE_NUMBER>9,<PHONE_NUMBER>5,<PHONE_NUMBER>3,
<PHONE_NUMBER>6,<PHONE_NUMBER>2,<PHONE_NUMBER>1,<PHONE_NUMBER> ,
<PHONE_NUMBER>8,<PHONE_NUMBER>2,<PHONE_NUMBER>8,<PHONE_NUMBER>4,
<PHONE_NUMBER>1,<PHONE_NUMBER>2,<PHONE_NUMBER>4,<PHONE_NUMBER>1,
<PHONE_NUMBER>4,<PHONE_NUMBER>6,<PHONE_NUMBER> ,<PHONE_NUMBER>7])

Since the sampling period is not constant, I'm using the nfft from which I should get the frequency components with complex numbers that represent a magnitude and phase.
rf = nfft.nfft(z, r)
Then I want to reproduce the signal with the nfft result:
N   = rf.shape[0]
amp = np.abs(rf)/N
phi = np.angle(rf)

s = 0
for i in range(N//2):

    f = i*fs/N
    s += amp[i]*np.cos(2*np.pi*f *z    +phi[i])

plt.plot(z, s, label='nfft result')

But it looks not at all like the original signal.

#7 has info on this.

In particular, unlike the FFT, the NFFT cannot exactly reconstruct a given signal, because in general the effective Nyquist frequency of an unevenly-sampled signal is very large. Your code is working as expected, and if you evaluate the NFFT at a denser frequency grid, your reconstruction will have more fidelity. For example:
N = 100000
k = -N / 2 + np.arange(N)
r_hat = nfft.nfft_adjoint(z, r, N)

r_reconstructed = nfft.nfft(z, r_hat) / N
plt.plot(z, r, '.', color='blue')
plt.plot(z, r_reconstructed, '-', color='red')

If we use original fft for example and assume the sampling period is constant 13 ( which is not true), I reproduce the signal like the following:
from scipy.fftpack import fft

Ts = 13.00
fs = 1/Ts

rf = fft(r)
amp = np.abs(rf)/r.shape[0]
phi = np.angle(rf)

s = amp[0]
for i in range(n//2):
    if i>0:

        k   = i*fs/n
        s += 2*amp[i]*np.cos(2*np.pi*k  *(z)  +phi[i])

plt.plot(z, s, label='fft result')
plt.plot(z, r, label='data')
plt.legend()

which because of the non-uniform sampling period, the reconstruction is wrong.
But if I use nfft:
N = 100000
r_hat = nfft.nfft_adjoint(z, r, N)
r_reconstructed = nfft.nfft(z, r_hat) / N

I haven't been able to do the same. What doesn't make sense to me is the amplitudes first. For example I don't find the DC constant amplitude which is approx. 110.
np.abs(r_reconstructed)

array([119.05166608,<PHONE_NUMBER>9,<PHONE_NUMBER>4,<PHONE_NUMBER>9,
119.23368786,<PHONE_NUMBER>5,<PHONE_NUMBER>1,<PHONE_NUMBER>2,
114.24173965,<PHONE_NUMBER>3,<PHONE_NUMBER>4,<PHONE_NUMBER>3,
100.463555  ,  99.78755809,  99.97843652,<PHONE_NUMBER>1,
100.13686047,<PHONE_NUMBER>9,<PHONE_NUMBER>5,<PHONE_NUMBER> ,
101.24062906,<PHONE_NUMBER>7,<PHONE_NUMBER>1,<PHONE_NUMBER>4,
111.67608816,<PHONE_NUMBER>2,<PHONE_NUMBER>5,<PHONE_NUMBER> ,
118.81533953,<PHONE_NUMBER>2,<PHONE_NUMBER>2,<PHONE_NUMBER>4,
119.61643871,<PHONE_NUMBER>9,<PHONE_NUMBER>7,<PHONE_NUMBER>8,
118.7799233 ,<PHONE_NUMBER>2,<PHONE_NUMBER>1,<PHONE_NUMBER>8,
104.48122313,<PHONE_NUMBER>7,<PHONE_NUMBER>3,<PHONE_NUMBER> ,
100.09855226,<PHONE_NUMBER>9,<PHONE_NUMBER>4,<PHONE_NUMBER>6,
100.38244013,<PHONE_NUMBER>4,<PHONE_NUMBER>4,<PHONE_NUMBER>2,
103.42187277,<PHONE_NUMBER>3,<PHONE_NUMBER> ,<PHONE_NUMBER>7,
114.65794209,<PHONE_NUMBER>2,<PHONE_NUMBER>3,<PHONE_NUMBER>4,
119.15335268,<PHONE_NUMBER>3,<PHONE_NUMBER>1,<PHONE_NUMBER>3])

np.abs(r_hat)

array([<PHONE_NUMBER>4,<PHONE_NUMBER>5, 1514.53901906, ...,<PHONE_NUMBER>9,
1514.53901906,<PHONE_NUMBER>5])

The DC offset is here:
>>> r_hat[len(r_hat) // 2] / len(z)
(110.40382390478732+0j)

If you want to understand what's going on mathematically, I'd suggest starting with the two formulas in the documentation: https://github.com/jakevdp/nfft#about
All this package does is compute those sums in ~O[log(N)] rather than ~O[N] if you have problems that you can express in terms of those sums, this package will help you. Otherwise, it won't.

In the equation of forward transform is the f_k the result of nfft_adjoint?
This is so far what I came up with to reproduce the signal with this equation:
N = 100000
K = -N / 2 + np.arange(N)
r_hat = nfft.nfft_adjoint(z, r, N)
rf = r_hat/ len(z)

amp = np.abs  (rf)
phi = np.angle(rf)

Ts = 0.01
fs =1/Ts

s = 0
for i, k in enumerate(K):

    ### f   = k *fs/N
    s += amp[i]*np.cos(2*np.pi*k *z  +phi[i])

plt.plot(z, s, label='nfft result')
plt.plot(z, r, label='data')
plt.legend()


chore: Upgrade Prometheus from v0.39.1 to v2.40.7

Update Prometheus to v2.40.7

[x] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
This version renames the wal package from "wal" to "wlog"

Ok.. we need first to update Vulcan: Aka: promql-engine

Closing: https://github.com/thanos-io/thanos/pull/5896


Add license and contributing files

Add license, authors and contributing files.

@maarten-litmaath could you please kindly review this, especially the AUTHORS file?

Bonjour Baptiste,
the contents of the AUTHORS file are incorrect.  Also see my next comment.
The contents of the file should be as follows:
Maintainers
Baptiste Grenier<EMAIL_ADDRESS>Original Authors
Andrea Sciaba<EMAIL_ADDRESS>Contributors
Laurence Field<EMAIL_ADDRESS>Maarten Litmaath<EMAIL_ADDRESS>Maria Alandes Pradillo<EMAIL_ADDRESS>

Hi again,
upstream is 2 bugfix versions ahead:
http://linuxsoft.cern.ch/wlcg/centos7/x86_64/  -->
lcg-info-1.12.3-1.noarch.rpm
lcg-info-1.12.4-1.el7.noarch.rpm

Hi again,
upstream is 2 bugfix versions ahead:
http://linuxsoft.cern.ch/wlcg/centos7/x86_64/ -->
lcg-info-1.12.3-1.noarch.rpm
lcg-info-1.12.4-1.el7.noarch.rpm

In fact this are also the versions that are in UMD: http://repository.egi.eu/sw/production/umd/4/centos7/x86_64/updates/
The fact is that this GitHub repository is expected to be the new home and so the upstream, it was moved there as part of the original migration of BDII components from CERN to EGI...
Are you still maintaining it somewhere in an accessible repo?
We can also look at the RPM content to see the changes on the perl file and man page but would be cleaner to be able to check the repo if any.
I pushed an update for the AUTHORS, thanks for being our Indiana Jones with those components :)

Hi again,
I can look into what changed and commit the fixes here.
I had applied them ad-hoc, there is no other source repository.
How urgent is this business?

So we would like to merge #2 and release a new version that would address #1 that was opened quite some time ago in GGUS: https://ggus.eu/index.php?mode=ticket_info&ticket_id=140478.
Would be good if we could finalise this task soon. We can quite easily make a diff of the files too if you don't have much time now for making a proper PR.

Hi again,
if you could have a quick looks at the diffs, that would be great, merci!
IIRC they were fairly trivial...

I ingested a diff with the last version from UMD (seems the same as from the CERN repo) and finally it builds!
So it's ready for your final validation and merge, thanks!
I will send tag a new version 1.12.4 and then we will merge #2 and release 1.12.5 to be submitted to UMD.

Salut,
what would you want me to validate and merge still?

No specific need @maarten-litmaath, if you are fine with the change EGI colleagues can make the formal review approving it so that we can merge it.


Threshold on 16 bits images is too slow

The calculations of threshold based on the histogram is too slow if the images are 16 bits.
https://github.com/image-js/image-js-typescript/blob/main/src/operations/threshold.ts#L84
We should at least have an options to specify the number of slots to use for the histogram calculation and for me the default value should be 2**8.
The result by itself should barely change.

The result by itself should barely change.

This seems wrong to me intuitively. We should discuss it in-person.

@moonayyur You may add this option but by default it can stay the same value as today.


Game crash when loading dungeon maps with an index higher than in the base game (>= 306)

01636199774_01003d200baa2000.log


FIX: Make find_ecg_events work even if ECG peak detection doesn't yield anything

Fixes #9225

Ok this is ready for review.

I think this should (and can easily) be backported too, any objections?

yes and we should release a new mne release too soon....

One CI run times out, the others finished. I think this can safely be merged.

Thanks @hoechenberger, backported

Awesome, thank you!


Update Readme

Sir, I put your new Logo/Icon in Readme :) Thanks!

Thanks again for the new logo!

your most welcome Sir!


Fix English sidebar link to "Use CPM.cmake"

The English link for "Use CPM.cmake" in the docs actually points to the Chinese version of "Use CPM.cmake".

The wiki at https://github.com/drogonframework/drogon/wiki also has the same issue in the sidebar.

@aloisklink thanks.


Support with new version of Leetcode

As I see, the auto commit to github repo after successful submit does not work with new version of Leetcode.
It would be good to have this feature added so that users don't have to switch to old UI of leetcode for this extension to work.

This is a dead project, forget

For new LeetCode support, you can use LeetHub v2 (https://github.com/arunbhardwaj/LeetHub-2.0).


Allocation of parts

Please provide following functions ohh great Master!

The ability to be able to select multiple line items and allocate them from the same location. Instead of doing individually.

Not currently able to access the "About page" while in the home screen.

The option to be able to partial build or split locations when closing out a "Build"

@Marbles87 Which version of InvenTree are you on?

The ability to be able to select multiple line items and allocate them from the same location. Instead of doing individually.

Does the "Auto Allocate" option satisfies this need? If all stock items default location point to the same location, it should allocate just fine.

Not currently able to access the "About page" while in the home screen.

Works for me, which browser are you using?

The option to be able to partial build or split locations when closing out a "Build"

Split locations should work fine, you'll need to allocate parts accordingly.
Partial build completion is currently not implemented, I think you can split your builds in more instances as a workaround?


The mwc-select component not exist

Where I can found the mwc-select component ?

It is not yet implemented. See https://github.com/material-components/material-components-web-components/issues/315. Closing as dupe.


Update blog page template to display a warning on blog posts older than year

ref: https://github.com/kubernetes/website/issues/30771, but this is also just a general feature!
Blog posts by their nature go stale over time. This becomes particularly problematic when we decide to deprecate functionality in the project, as those blog posts are forever a part of the docs and forever searchable via Google.
Update the blog post's templates (probably https://github.com/kubernetes/website/blob/main/layouts/blog/baseof.html, but you may need to copy over single.html from the Docsy theme) to include a warning for blog posts over 1 year old:

Warning: This blog post was published over a year ago. Use caution when following any steps published, unless otherwise noted in the post.

Some notes:

Use standard docsy warning block styles
Make sure strings are in localization files so the localization teams know where to look

/triage accepted
/priority important-soon
/area website

/assign

/area web-development

Sorry y'all, I hadn't seen this issue before I created #31043. It turns out that there's a clean solution that doesn't involve changes to the blog template.
Anyhow, this has been closed by #31594. /cc @sftim @nate-double-u

/close
Thanks @chalin!


Change the How to Documentation

I noticed in the how to, that is not mentioned that I need to fork the project, commit my changes in my branch and then make a PR.
Because if I just make a branch from main I will not have permission to make a PR, at least I tried and get an error.
An explication on how to contribute using a fork is here: https://www.dataschool.io/how-to-contribute-on-github/
Thanks.

I think you could put this part before the "Add new card" part.

@wbalbo good idea. Do you want to submit a PR?


Insufficient method and member summary in API docs

Is your feature request related to a problem? Please describe.
Every time I refer to the .NET API Documentation, I always find it quite frustrating that I have to click a link and go to a separate page to find out if a method is static or abstract or to find out its return type. When the site lists the methods or properties of a type, it only includes the name and a short description, not even the return type. Static, instance and abstract methods are all listed together.
For instance, the following is the list of methods for the System.Uri class:

From: https://learn.microsoft.com/en-us/dotnet/api/system.uri?view=net-8.0
There is a combination of static and instance methods all listed together. There is no way I can tell which methods are static or which ones are abstract. For that, I'll have to visit the link for each method. I have to also do this to find out what type each method returns. This is extremely inconvenient. Especially when I'm trying to study a new API and also when there are static and non-static versions of the same method.
Describe the solution you'd like
Introduce filters to list out either all/static/instance/abstract methods. A good example of this is how Java's API docs does it:

From: https://docs.oracle.com/en/java/javase/20/docs/api/java.base/java/util/stream/Stream.html
As seen in the image, I can easily view only static, instance, abstract methods. Moreover, it can be observed that the modifiers and return types are also specified for each method in the summary table.
Describe alternatives you've considered
A possible alternative would be to introduce an additional section for static methods as done by Dart's API docs:

From: https://api.dart.dev/stable/3.0.1/dart-core/int-class.html
Also, note the specification of modifiers via labels like override. The return type is also specified here.

cc @kexugit - general feedback above the method summary section for a type.

This issue was moved to dotnet/dotnet-api-docs#9020


SceneFlexLayout: Min width/height option was ignored

Discovered that the SceneFlexLayout component was not respecting the minWidth and minHeight options if it doesn't have a parent, this pr fixes so that they are applied without having to have a parent.
The reason why it works when there is a parent is that applyItemStyles function is called with both the own state and parentState and there the minWidth and minHeight options are applied.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Models Reference

The link to Models Reference isn't there. The .rst file is missing.

Duplicate of #720


max_index defined as data_size?

https://github.com/martinmoene/variant-lite/blob/8a5f0665f76160b6d038537b37771bd708d45de0/include/nonstd/variant.hpp#L1475
Was this line meant to be data_size, or instead the maximum number of types in the variant?
It seems that the code below will cause a bad_access, because int is at index 4, which is also the data_size of the variant:
template<size_t i>
class N{int k;};

using MyVar = nonstd::variant<N<0>, N<1>,N<2>,N<3>, int>;

int main()
{
    MyVar a(2);

    nonstd::visit([](const auto& a) {
        std::cout << "Val: " << sizeof(decltype(a)) << std::endl;
    }, a);

    return 0;
}

Thanks @KarimLUCCIN ,
Good catch.
max_index() means max_alternative_index().
Its implemention is wrong, and it is used erroneously and superfluously in this and this check for invalid access.
The bad access can be revealed more directly via get<type>(var):
using nonstd::get;
using variant_t = nonstd::variant<N<0>,N<1>,N<2>,N<3>, std::int32_t>;

try
{
    (void) get<std::int32_t>( variant_t( 13 ) ) );
}
catch( std::exception const & e )
{
    std::cout << "Error: '" << e.what() << "'\n";
}

yielding: Error: 'bad variant access'

Thanks for the fix!


[feature] Add support for vue files

⭐ Suggestion
Vue is support by the language server volar, I'm not sure how hard it would be to add support for vue to ast-grep but I can try if there's interest. It's basically a way to write css, html and typescript/javascript in a single file.
💻 Use Cases
Lots of people use vue I guess.

Please see https://ast-grep.github.io/advanced/language-injection.html#more-possibility-to-be-unlocked


[SRVKS-1017] Clean up net-istio main branch like serving's main

This patch clean up net-istio main branch like serving main.
The patch removes a lot of files so please refer to https://github.com/nak3/net-istio/tree/cleanup-main for the change.

/cc @mgencur @skonto @ReToCode

/lgtm
/approve
/hold for other reviewers

/hold cancel


Show Component subclasses in menu

Hey, I'm jsut wondering if its possible to show the component subclass I've created in the dropdown, inside the engine? When you create a C++ component it becomes available in the dropdown.

Hi, not sure to understand what you mean. By default subclasses of UActorComponent should appear in the components list:
from unreal_engine.classes import ActorComponent

class NewComponent(ActorComponent):
    pass

Hmmm, is there any special trick I need to do to trigger this?
The dropdown I'm referrign to is the + Add Component one, in the details panel of an actor.

I've been able to successfully use the Python component, but only as a proxy. I'm probably missing something on how to create an actual component subclass.
My code looks like this:
import unreal_engine as ue
from unreal_engine.classes import ActorComponent

import debug; debug.listen()

class PyPositionReport(ActorComponent):

    def __init__(self):
        pass

    def begin_play(self):
        name = self.uobject.get_owner().get_name()
        location = self.uobject.get_owner().GetTransform().translation
        print('%s is at %s' % (name, location))

The debug.listen() just calls settrace() and lets me debug the code in PyCharm, and it's not interfering with te component.

Hey, got any tips on this?

Hi, when you execute the code of a python UE subclass you should get a yellow warning in the logs. Can you paste it ? Thanks


Why gohbase rowkey is string type, not []byte

It is []byte for everything besides hrpc.Mutate requests. Feel free to send a PR to add non-*Str mutate request constructors.


Validation stops on first invalid field

I noticed validation stops on first invalid field, that is perfect for most use cases.
However there is one use case where validation should not stop on first invalid field.
In my apps I try to keep structure of documents consistent all the time ( what a suprise ;-) ) so on every startup app is validating all documents in all collections.
It may happen that doing some changes manually on db admin simply broke structure.
It is better to see upfront how much db was damaged.
Let me know if sth like that could be easy to add.
Again, thanks for your hard work ! :)

Hmm I was talking about validation in Astronomy with the friend from the work like few days ago. He pointed out that sometimes it would be great to validate all fields of the object, not just stop after the first error :). It shouldn't be hard to implement. Probably I would have to introduce new method like validateAll(), what do you think?

validateAll would be very useful for me.
As I wrote I'm already using js-schema for doing that, so for me this is another must-have feature ;-)

Ok, no problem :). I will implement it soon, maybe tomorrow :)

Done in version 0.10.5 :)


Add type checker changs for client resource access action

Purpose
$title
Part of: #36024
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@rdulmina The commit messages and PR heading is not according to the best practices. Please correct them.

[1] https://github.com/ballerina-platform/ballerina-lang/blob/master/CONTRIBUTING.md#submit-your-contribution

The build is failing will fix


Add focus tracking support for shadow dom

Part of https://github.com/adobe/react-spectrum/issues/1472
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

Thanks for the PR, I see it's failing Typescript right now. I also started to do an initial review, however, I don't understand the changes in this commit:

The commit replaces the use of getOwnerDocument with getRootNode function in the FocusScope.tsx file. This change modifies how the active element is retrieved and how event listeners are added and removed, leading to better management of the focus scope. Additionally, getRootNode function has been expanded to handle Shadow DOM.

Can you explain more about the changes in this commit? What was going "wrong" before the commit? Do we need tests for it?

@snowystinger Thanks for reviewing. You can pause the review for now. We will get back to working on this PR or create a new one this week.


Question about backward scanning

Hi,
Thanks for sharing this amazing work. May I know where backward scan is implemented? Looking at here, it seems that only forward scanning is done while Vim does the backward scan by flipping the residuals and hidden states.

Hi! It is done here.

That's much better 🙂 Thanks!


Property in Controller class can't used in the functions which are register in Routes

I want to add a property in the Controller class. Like in the HeroController, I add a 'test' property, and I want to use it in the create function. But when create is invoke by the route, it show [TypeError: Cannot read property 'test' of undefined].
I think it's because typescript is simulate class but Javascript didn't have a class until ES6. I have use c# before and use Javascript and Typescript recently but not full understand it.
class HeroController {

    test = "test";

    create(req: express.Request, res: express.Response): void {
        console.log(this.test);
    }
}

I found I got a incorrect understanding about keywords this before, and solved the problem by change the way of register controller in the Routes.
Before:
router.post("/heroes", controller.create);
After:
router.post("/heroes", (req, res) => controller.create(req, res));

I found 'this' in TypeScript is a common problem, here is some useful wiki about how to avoid it.

glad you got the resolution :)


Production Setup

Hello,
Besides memory is or will there there be any other storage option available in 1.4 or 1.5?
I would like to have the same setup described here https://camo.githubusercontent.com/57ddedea3c60f776f5634474e70da232b64efee1/687474703a2f2f6e6574666c69782e6769746875622e696f2f61746c61732f696d616765732f77696b692f61746c61735f73746f726167655f61745f6e6574666c69782e737667
obviously the retentions will be different - we have a smaller volume. We would probably be able to keep 1w of data in memory.
Is EMR in the diagram used to roll up to coarser time units? Any other options for that? Ideally I don't want to deal with that at this stage, I think with our volume we should be okay with a simple rollup algorithm running directly on the S3 objects.
I can't find any documentation at all for this kind of setup.
More context on my project:
We are currently using spring-boot and metrics3 directly to collect metrics. metrics3 forwards them to Graphite. We are moving to Amazon ECS and autoscaling is increasingly being used so the fact that we have the instanceId(containerID) in the metric names is not a very good match for Whisper.
Now I'm looking to migrate that to spring-cloud-netflix with Spectator and Atlas. In order to be able to run Atlas in production I need to be able to also store and make historical data available.
Thank you

Besides memory is or will there there be any other storage option available in 1.4 or 1.5?

For 1.4 it will only get bug fixes at this point. For 1.5, there is no plan to support storage that serves from a location other than memory, but it should have the portion that loads from s3 merged. So it should be relatively easy to have it backed by a persistent source so it can be recovered.
1.5 is currently planned to be released by end of Q1 2016.

Is EMR in the diagram used to roll up to coarser time units?

We use EMR for a lot of processing. In terms of what we keep in memory for quick serving we don't consolidate (rollup to coarser time granularity) though we have in the past. One of the main uses we have is doing time shifts like day over day and week over week to get a baseline for comparison. The consolidated datapoints made it difficult to reason about how the current behavior really relates to the past behavior.
The rollups we do now are for the set of tags available. For example, in the < 16d ago cluster we drop the instance dimension for almost all metrics which greatly reduces the overall volume. Due to auto-scaling and frequent deployments, instances are short lived and looking at a graph over several weeks or doing a time shift is useful for the aggregate at the cluster level, but not for the individual nodes.
We also keep the data long term in hive and we do create a copy that is rolled to daily datapoints to make it more efficient for trending queries over a long time period.

I can't find any documentation at all for this kind of setup.

Documentation is something we know needs to be improved and plan to make progress on that this quarter. One of the reasons this has been slow to come out is that some of the internal tooling we rely on heavily for this setup isn't open source.
There is also a todo item for documenting how much data can be stored on a given node with a specific configuration. Like most data systems though things like number of metrics depends a lot on the data characteristics.

K, so there will be support for backing up the in-memory data to S3. Any idea how the recovery process will look like?
What about the redundant setup? Will all the parts be available in 1.5 to build the architecture you describe describe where a "Publish" cluster handles sharding and replication?

Any idea how the recovery process will look like?

We treat recovery then is the same as deploying a new version. In general, we don't trust processes that are not run regularly. So there isn't a special recovery setup. When the process comes up and while running it will check and keep itself in sync with the S3 copy.

Will all the parts be available in 1.5 to build the architecture you describe describe where a "Publish" cluster handles sharding and replication?

That is the current plan.


Added policies for 1803 and 1809 (1903 not out yet)

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/3912

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

Other than my 2 comments above, the rest of the PR content seems to be technical information,
so no grammar or syntax mistakes to look for there. Nothing further for me to add.
Then again, it may be useful to verify that the content is correct and that there are no typos in it.

@dansimp The PR introduces details on policies for 1803 and 1809 for New policies for Windows 10 page. Please review and merge as the time permits. Thanks.


Contact Custom Field Page - Issue while type conversion from text to date

Describe the bug

After conversion from text to date type if we edit filled customfield value it shows old value
After conversion if we edit blank value and we added date then also it shows blank

https://www.berrycast.com/conversations/63d81fd9-e449-52ee-8525-db6a2ac827df

I believe this is fixed, but not moved

Revert back if still error


Kernels for other languages are not found with store python jupyter

The directories we search do not include the actual path for the roaming kernels if they're installed in a store based python.
We need to use the python involved to get the real path to %APPDATA%\jupyter

Seems to be working now.


SpiceDB Spanner queries use different+worse index than e.g. same query in Spanner Studio

What platforms are affected?
others
What architectures are affected?
others
What SpiceDB version are you using?
1.25.0 - 1.27.0
Not sure if this issue is present in earlier versions as well.
Steps to Reproduce
TLDR:

Use Spanner Datastore
Run a LookupResources call
Compare query performance and query plan from SpiceDB issued Spanner queries vs. ones ran in e.g. Spanner Studio
Observe significant performance deficit

Details:
A LookupResource will issue multiple SQL queries targeting the Spanner datastore that are very similar, but will use different indices for the lookup, which depending on the amount of relations present will result in large performance degradations.
First query, using the ix_relation_tuple_by_subject index and returning in <20ms:
SELECT
  namespace,
  object_id,
  relation,
  userset_namespace,
  userset_object_id,
  userset_relation,
  caveat_name,
  caveat_context
FROM
  relation_tuple
WHERE
  ((userset_namespace = '<USERSET_NAMESPACE>'
      AND userset_object_id IN (('<OBJECT_ID>'))
      AND userset_relation = '...'))
  AND namespace = '<NAMESPACE>'
  AND relation = '<RELATION>'
ORDER BY
  userset_namespace,
  userset_object_id,
  userset_relation,
  namespace,
  object_id,
  relation
LIMIT
<PHONE_NUMBER>854775807

Later query, using ix_relation_tuple_by_subject_relation index, returns in 2000-4000ms:
SELECT
  namespace,
  object_id,
  relation,
  userset_namespace,
  userset_object_id,
  userset_relation,
  caveat_name,
  caveat_context
FROM
  relation_tuple
WHERE
  ((userset_namespace = '<USERSET_NAMESPACE>'
      AND userset_object_id IN (('<OBJECT_ID>'))
      AND userset_relation = '...'))
  AND namespace = '<NAMESPACE>'
  AND relation = '<RELATION>'
  AND ((object_id > '<SOME OBJECT ID>'))
ORDER BY
  userset_namespace,
  userset_object_id,
  userset_relation,
  namespace,
  object_id,
  relation
LIMIT
<PHONE_NUMBER>854775807

This only occurs with queries issued by SpiceDB. Running the EXACT SAME query inside e.g. Spanner Studio will always use the correct index and return in <20ms as well, yielding the same rows.
This issue was investigated by adding the QueryWithStats() Spanner call in spanner.go and verifying the planning and execution times as well as which indices where used.
The used Query optimizer version and Statistics package are the same between the queries in Spanner Studio and issued by SpiceDB.
Expected Result
SpiceDB issued Spanner calls use the correct/fastest index for SQL queries
Actual Result
ix_relation_tuple_by_subject_relation is used instead of ix_relation_tuple_by_subject, leading to a severe performance degradation.
We have stopped using LookupResources calls for now for cases with many relations, as the performance penalty was too severe.

Thanks for the detailed report @kruegener! 👏🏻 ⭐

I suspect the best way forward is actually to force the index. This is often discouraged for other databases, but it looks like the Spanner documentation actually recommends it: https://cloud.google.com/spanner/docs/secondary-indexes#index-directive

If you'd like you can try out #1692 which should fix this issue.

Thx for the quick MR on this! I hope you also had some time off in the recent weeks.
I just got to testing your MR for our use-case, but unfortunately with the same result.
There was a small typo in your MR, the index is called ix_relation_tuple_by_subject, not idx_relation_tuple_by_subject. But fixing that also didn't help.
I can confirm that the slow query is altered (what seems correct to me) with the forced index and now looks like this:
SELECT namespace,
       object_id,
       relation,
       userset_namespace,
       userset_object_id,
       userset_relation,
       caveat_name,
       caveat_context
FROM relation_tuple /*@ FORCE_INDEX = ix_relation_tuple_by_subject */
WHERE ((userset_namespace = @p1 AND userset_object_id IN (@p2) AND userset_relation = @p3))
  AND namespace = @p4
  AND relation = @p5
  AND ((object_id > @p6))
ORDER BY userset_namespace, userset_object_id, userset_relation, namespace, object_id,
         relation LIMIT<PHONE_NUMBER>854775807

But QueryWithStats still reveals that one step of the query plan involves a full scan of the ix_relation_tuple_by_subject_relation index, which takes up 99% of the time.
Running the same query once again in Spanner Studio does not use that index, similar to the original situation without the forced index.
I've got no idea why this happens and the forced index is ignored, but the result and timings are exactly the same as before.

I've found some documentation interleaved indexes which is another lead to explore for making these queries perform.

Spanner stores index data in the same way as tables, with one row per index entry. Many of the design considerations for tables also apply to indexes. Non-interleaved indexes store data in root tables.

Totally spitballing: it's possible the full scan is of this table is worse than using a less-specific, but interleaved index.

@kruegener Would you be willing to try modifying one of your Spanner indexes manually?
The current index being used is defined as CREATE INDEX ix_relation_tuple_by_subject ON relation_tuple (userset_object_id, userset_namespace, userset_relation, namespace, relation), which does not contain the object_id (likely because it wasn't used previously).
Now that LR has cursors, it might be worth adding another index of the form:
CREATE INDEX ix_full_relation_tuple_by_subject ON relation_tuple (userset_object_id, userset_namespace, userset_relation, namespace, relation, object_id)

If you're willing to add that manually and try on your slow query, I'd appreciate it


ts-script.exe Error

This error occurs after running ts-script.exe
ts-script.exe:42 (a nil value)
LuaForWindows is installed,
start as admin does not fix it
changing folder which contains .exe does not work

This error occurs when you dont edit the file paths in "config.txt" to match the paths on your computer.

i did edit the path, maybe i did something wrong syntax wise?
here is my confix.txt line
Path to data Folder:d:\Spiele\Steamcmd\steamcmd\steamapps\common\GarrysModDS\garrysmod\data;

d:\Spiele\Steamcmd\steamcmd\steamapps\common\GarrysModDS\garrysmod\data
You have to escape the "" with another ""

i am not even using ""? am i missing something?

i jsut did, but git commented it out. i did double backslash

Yup, I noticed it when writing my comment. Contact me on Steam so we can figure this out: http://steamcommunity.com/id/Cr4zyPi3t

Closed
Scripts werent installed correctly and server query didnt have sufficient permission


layer compose bug？

it seems like some bug in effects.py
File "C:\Users\heyingjie\AppData\Local\Programs\Python\Python38-32\lib\site-packages\psd_tools\api\layers.py", line 524, in compose
return compose(self, **kwargs)
File "C:\Users\heyingjie\AppData\Local\Programs\Python\Python38-32\lib\site-packages\psd_tools\composer_init_.py", line 134, in compose
image = compose_layer(layer, **kwargs)
File "C:\Users\heyingjie\AppData\Local\Programs\Python\Python38-32\lib\site-packages\psd_tools\composer_init_.py", line 163, in compose_layer
return apply_layer_ops(layer, image, force=force, **kwargs)
File "C:\Users\heyingjie\AppData\Local\Programs\Python\Python38-32\lib\site-packages\psd_tools\composer_init.py", line 192, in apply_layer_ops
image = apply_effect(layer, image, effect_base)
File "C:\Users\heyingjie\AppData\Local\Programs\Python\Python38-32\lib\site-packages\psd_tools\composer_init.py", line 311, in apply_effect
for effect in layer.effects:
File "C:\Users\heyingjie\AppData\Local\Programs\Python\Python38-32\lib\site-packages\psd_tools\api\layers.py", line 426, in effects
self._effects = Effects(self)
File "C:\Users\heyingjie\AppData\Local\Programs\Python\Python38-32\lib\site-packages\psd_tools\api\effects.py", line 42, in init
if not item.get(Key.Enabled):
AttributeError: 'Integer' object has no attribute 'get'
it is worked before i use photoshop open the psd, and rename another layer.(not the some layer i try to compose).the version of photoshop is 21.0.0, any idea please?

@shadowshien What is your psd-tools version? Can you provide an example that reproduces this error?

1.8.30，trying to upload a sample

don't know if was success to upload a sample,first day use github, sorry

@shadowshien We can't see the attachment ;)

@jfthuong Here ya go...
chad test.psd.zip
# super hacky example
from psd_tools import PSDImage
psd = PSDImage.open('chad test.psd')

for layer in psd.descendants():
    if layer.kind != 'group':
        if layer.name !='inverter_sma_1' and layer.name != 'inverter_solaredge_1':
            layer.visible = True
        else:
            layer.visible = False

image = psd.compose( layer_filter=lambda x: x.is_visible() )
image.save( 'chad_test_1.png' )

I found similar problem when parsing shape layer.
Normally, Effects variable in ShapeLayer has a value of Effects(), but sometime error string ('Integer' object has no attribute 'get') is exist inside the layer.


http.client.RemoteDisconnected: Remote end closed connection without response

When I run python3 -m chyoa download 'https://chyoa.com/story/Aaralon%27s-Discoveries.276' aaralons_discoveries the program errors out with
Traceback (most recent call last):
  File "/usr/lib/python3.5/runpy.py", line 184, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/wil/code/chyoa-scraper/chyoa/__main__.py", line 13, in <module>
    chyoa.main(sys.argv)
  File "/home/wil/code/chyoa-scraper/chyoa/__init__.py", line 36, in main
    dl.download(url, dest, debug=debug)
  File "/home/wil/code/chyoa-scraper/chyoa/download.py", line 21, in download
    scraper.scrape(url)
  File "/home/wil/code/chyoa-scraper/chyoa/scraper.py", line 32, in scrape
    self._scrape_urls(self.to_visit)
  File "/home/wil/code/chyoa-scraper/chyoa/scraper.py", line 48, in _scrape_urls
    fields = self.parser.get_chapter_fields(url)
  File "/home/wil/code/chyoa-scraper/chyoa/parser.py", line 36, in get_chapter_fields
    response = urlopen(url)
  File "/usr/lib/python3.5/urllib/request.py", line 163, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python3.5/urllib/request.py", line 466, in open
    response = self._open(req, data)
  File "/usr/lib/python3.5/urllib/request.py", line 484, in _open
    '_open', req)
  File "/usr/lib/python3.5/urllib/request.py", line 444, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.5/urllib/request.py", line 1297, in https_open
    context=self._context, check_hostname=self._check_hostname)
  File "/usr/lib/python3.5/urllib/request.py", line 1257, in do_open
    r = h.getresponse()
  File "/usr/lib/python3.5/http/client.py", line 1197, in getresponse
    response.begin()
  File "/usr/lib/python3.5/http/client.py", line 297, in begin
    version, status, reason = self._read_status()
  File "/usr/lib/python3.5/http/client.py", line 266, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response

https://docs.python.org/3/library/http.client.html#http.client.RemoteDisconnected
This sounds like either a problem with CHYOA's servers. Maybe Cloudflare is rate limiting you?


Crash

File: https://cdn.download.clearlinux.org/releases/33510/clear/clear-33510-live-desktop.iso
CRC32: 13abfa78
MD5: 6dad4b6e0916812a01b8b119dfc0e789
Sha256: f9e3a3261ce4d86dcd12a80e1a26fb5a8f16cbc29c720c52164b62aa54db2a6d

Thanks! Overlooked one of the cases in the latest commit which led to this bug (on large deflate streams that aren't processed in memory, but on disk).


Android Paymentflow doesn't create card on stripe customer

Platform

Android

Describe the bug
Hello,
When I use createPaymentFlow on the web, everything works perfectly. However, when I use it on Android, everything appears to work fine, and I get the cardNumber returned. But the card is not being added to my Stripe customer.
I've followed all the steps in the documentation, including configuring MainActivity.java, variables.gradle, and initializing Stripe.
Here are the package details:
"@capacitor-community/stripe": "^6.2.1",
"stripe-pwa-elements": "^2.1.0"
On the web, I can see requests being sent in the network console when I click the "Add" button, but no requests are being made on Android.
To Reproduce
        await Stripe.createPaymentFlow({
          setupIntentClientSecret: response["paymentIntentClientSecret"],
          customerEphemeralKeySecret: response["ephemeralKeySecret"],
          customerId: response["customerId"],
          merchantDisplayName : "MerchantNameTest"
        });

        let result = await Stripe.presentPaymentFlow();

Expected behavior
Add new card into stripe with android app
Using Function:

Payment Flow

My mistake i forgot :
await Stripe.confirmPaymentFlow();

My mistake :
I forgot await Stripe.confirmPaymentFlow();


Added AuthorizationRule and SharedAccessAuthoirizationRule classes

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: iTOYS sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


[ci.yaml] Mark roller flaky

https://github.com/flutter/flutter/issues/93225
We need to roll flutter/recipes forward to get the new compatible version from Fuchsia for auto-rolling (sets bot commit instead of CR+2)
Pre-launch Checklist

[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[X] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[X] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[X] I signed the [CLA].
[X] I listed at least one issue that this PR fixes in the description above.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is [test-exempt].
[X] All existing and new tests are passing.

I'm going to land of red since this is only a ci.yaml change and it will unblock the tree.


scrollTo() doesn't wait for any Scrollable to be on the screen - it fails right away

Eg. when transition between screens has no loader, but for a while there is just empty screen with no scrolling. scrollTo fails because there is no Scrollable widget (that's true), but I would expect it to pump until there is something to scroll and fail when pump times out.

I checked the code responsible for this and there is waiting for some scrollable already - so it should work like I described.
The problem that I have is that during this scrolling action, new frames are rendered and widget tree is changed, so the scrollable found at the start of the action is not there, when actual dragging gesture is performed.
Closing as this is problem with tested app, not patrol


CI for Node

https://github.com/KevinMcKeown/github-actions-for-ci/issues/1

Initializes with two players


[BUG]: (Error while installing solaris - pip install . - on Windows)

Thank you for helping us improve solaris!
Summary of the bug
Following README's instruction to install solaris I've found the same error several times. I'm using Windows. Because I didn't have problems on Mac
Steps to reproduce the bug
cd solaris
conda env create -n solaris -f environment.yml
conda activate solaris
pip install .

Paste output here
    ERROR: Command errored out with exit status 1:
     command: 'C:\Users\sdtorres\Anaconda3\envs\solaris\python.exe' -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\sdtorres\\AppData\\Local\\Temp\\pip-req-build-maxx2990\\setup.py'"'"'; __file__='"'"'C:\\Users\\sdtorres\\AppData\\Local\\Temp\\pip-req-build-maxx2990\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base 'C:\Users\sdtorres\AppData\Local\Temp\pip-pip-egg-info-k_88fa02'
         cwd: C:\Users\sdtorres\AppData\Local\Temp\pip-req-build-maxx2990\
    Complete output (1 lines):
    GDAL must be installed to use `solaris`. See the documentation for more info. We recommend installing GDAL within a conda environment first, then installing solaris there.
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

Hi @Servando1990, thanks for reporting this issue! The rtree and gdal packages can be problematic to install without Anaconda. As noted in the current README, try the following:
cd solaris
conda env create -n solaris -f environment.yml
conda activate solaris
conda install -c conda-forge rtree gdal=2.4.1
pip install solaris
Hope this helps!

Hi @roshanr11, I'm facing the same problem and unfortunately running the extra line conda install -c conda-forge rtree gdal=2.4.1 doesn't solve the problem. Also, if i try to install solaris via pip and not from the conda environment I get the following:
C:\Users\52442>pip install solaris
Collecting solaris
  Using cached solaris-0.3.0-py3-none-any.whl (17.8 MB)
Requirement already satisfied: opencv-python>=4.1 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (<IP_ADDRESS>)
Collecting pyyaml==5.2
  Using cached PyYAML-5.2-cp37-cp37m-win_amd64.whl (215 kB)
Collecting requests==2.22.0
  Using cached requests-2.22.0-py2.py3-none-any.whl (57 kB)
Requirement already satisfied: matplotlib>=3.1.2 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (3.2.2)
Requirement already satisfied: torchvision>=0.5.0 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (0.6.1)
Requirement already satisfied: fiona>=1.7.13 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (1.8.13)
Collecting tensorflow==1.13.1
  Using cached tensorflow-1.13.1-cp37-cp37m-win_amd64.whl (63.1 MB)
Requirement already satisfied: torch>=1.3.1 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (1.5.1)
Requirement already satisfied: tqdm>=4.40.0 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (4.47.0)
Requirement already satisfied: scipy>=1.3.2 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (1.5.0)
Requirement already satisfied: rtree>=0.9.3 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (0.9.4)
Requirement already satisfied: scikit-image>=0.16.2 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (0.16.2)
Requirement already satisfied: networkx>=2.4 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (2.4)
Requirement already satisfied: rasterio>=1.0.23 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (1.1.5)
Requirement already satisfied: urllib3>=1.25.7 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (1.25.9)
Requirement already satisfied: pip>=19.0.3 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (20.1.1)
Requirement already satisfied: pandas>=0.25.3 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (1.0.5)
Collecting pyproj>=2.1
  Using cached pyproj-2.6.1.post1-cp37-cp37m-win_amd64.whl (17.2 MB)
Requirement already satisfied: affine>=2.3.0 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (2.3.0)
Collecting albumentations==0.4.3
  Using cached albumentations-0.4.3.tar.gz (3.2 MB)
Requirement already satisfied: numpy>=1.17.3 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (1.18.5)
Requirement already satisfied: gdal>=3.0.2geopandas>=0.7.0 in c:\users\52442\anaconda3\lib\site-packages (from solaris) (3.1.2)
ERROR: Could not find a version that satisfies the requirement shapely>=1.7.1dev (from solaris) (from versions: 1.0a7, 1.0b1, 1.0b2, 1.0b3, 1.0b4, 1.0rc1, 1.0rc2, 1.0, 1.0.1, 1.0.2, 1.0.3, 1.0.4, 1.0.5, 1.0.6, 1.0.7, 1.0.11, 1.0.12, 1.0.13, 1.0.14, 1.0.15, 1.2b1, 1.2b2, 1.2b3, 1.2b4, 1.2b5, 1.2b6, 1.2b7, 1.2rc1, 1.2rc2, 1.2, 1.2.1, 1.2.2, 1.2.3, 1.2.4, 1.2.5, 1.2.6, 1.2.7, 1.2.8, 1.2.9, 1.2.10, 1.2.12, 1.2.13, 1.2.14, 1.2.15, 1.2.16, 1.2.17, 1.2.18, 1.2.19, 1.3.0, 1.3.1, 1.3.2, 1.3.3, 1.4.0, 1.4.1, 1.4.2, 1.4.3, 1.4.4, 1.5.0, 1.5.1, 1.5.2, 1.5.3, 1.5.4, 1.5.5, 1.5.6, 1.5.7, 1.5.8, 1.5.9, 1.5.10, 1.5.11, 1.5.12, 1.5.13, 1.5.14, 1.5.15, 1.5.16, 1.5.17, 1.6a1, 1.6a2, 1.6b1, 1.6b2, 1.6b3, 1.6b4, 1.6b5, 1.6.0, 1.6.1, 1.6.2, 1.6.2.post1, 1.6.3, 1.6.4, 1.6.4.post1, 1.6.4.post2, 1.7a1, 1.7a2, 1.7a3, 1.7b1, 1.7.0)
ERROR: No matching distribution found for shapely>=1.7.1dev (from solaris)

which is weird because the latest version of shapely is 1.7.0 and modifying the setup.py doesn't help either.
I'm working on windows 10 and my version of python is 3.7.7

Hi!
I solved the problem in With Ubuntu 2.0
thanks

hi @Servando1990, could u install it on windows? i face the make error " GDAL must be installed to use solaris" as you.


Banner Option

In GrLevel2, there is an option for a banner, this for example:

Do you think it's possible to implement this feature into the Supercell Wx application? Thanks!

Thanks for the suggestion. There are plans to add a similar feature. #292 has more details.


Added example using regionmask for regional skill

Uses Riley's new pull request for adding skipna=True as an optional
argument to xskillscore! Thanks to Arianna for the example code for
applying the regional masks!

A note on the xskillscore installation instructions. Once it is merged into master and uploaded to pip (which Ray usually does pretty quickly after a new feature like this), we can just enforce the updated xskillscore version in the yaml file. E.g., xskillscore >= 0.9 or whatever the new version will be.

Combined Southern American regions (rectangular land + ocean regions) with land mask to apply a mask for just the Southern American continent.
Also included a notebook for looking at skill for the climatological seasonal cycle (relative to long-term mean).

Addresses https://github.com/hdrake/cmip6hack-multigen/issues/17.


Confirm before onDrop handles the files

It would be useful to have a function that fires before onDrop
use case: Already have a file in the dropzone want confirmation from the user before override it with another file

Related to #321

Since you won't have the full information about your files before you drop them, this feature should be implemented in the onDrop callback as well.
The implementation is up to you but I imagine it's pretty straightforward since you're dealing with your own state.


🦆Need Help , Raise Comment

About

This issue is about working flow of Issues and PR for newcomers .
For any issues related to issues comment below

I cannot modify the splash screen which comes before the intro for the app like trust page and etc. can you help?


aurbuild: corrupted packages with --force

I don't know why, but on rebuilds pacman sometimes complains on corrupted packages in /var/cache/pacman/pkg.

This is due to the checksums in the database file not matching with the old packages in the cache. So, you'd have to first clear the cache, or bump pkgrel (as was done earlier with setconf).

I've added an option to repofind to remove cached repo packages: repofind -c.

I ran into this issue when repeating builds of the same package to test changes. This is how I dealt with the issue:
https://github.com/djmattyg007/aurutils-with-signing/blob/fdddc654aeffa33cfaa9aa4f400530d61c4c38aa/bin/aurbuild#L27

It's more granular than removing all cached repository packages in one go, but how do you deal with packages that were cached as part of makepkg dependency resolution? e.g.

build foo
remove foo
build bar, sync foo as makedepend
remove bar
foo remains in cache

I don't think that's an issue, because the cached version is always removed when the package is rebuilt. Maybe I'm misunderstanding?

Good point. If pacman installs a dependency, it would retrieve the previously rebuilt packages, rather than the old one from the cache.
Arguably this should be the default, as there's little point in keeping packages in the cache when they're already in the repository.

The above commit should fix this, thanks for the idea.


Missing tag for 0.29.17

Looks like 0.29.17 was released recently. Though it appears the git tag for 0.29.17 is missing. Would be good if that could be published.

Thanks Stefan! 😄


v1beta2 API planning

We've recently released the first API version, this means we'll be collecting feedback from users soon.  By creating a v1beta2 API we can address comments like renaming/adding API fields, possibly moving them.
After we merge kubebuilder related code into main it will be possible for us to start working on a new version of API. Kubebuilder supports an easy way of automatical conversion from v1beta1 to v1beta2 - https://book.kubebuilder.io/multiversion-tutorial/conversion.html
This issue is created to store feedback and planning future work.

not planned


User schema

I'd love to get a clear picture of the current User schema for requests made to https://api.thegrid.io/user.
The only documentation I can find is for https://passport.thegrid.io/api/user which has significantly less properties than the JSON response I'm seeing.
I'm hesitant to share what properties are present since I'm not sure if I have anything "special" due to being on staff at The Grid.
All I need for each property is its name, type, and if its required.
Please help!

Friendly ping ✨


Create a MD link to the reffered files in the exported MD file

Hi,
Would be great if you were to generate clickable links to the file with say a TO-DO point.
You know how to do it I guess :-) It will make it possible here on GitHub to click into the issue.

I'm not sure I understand what you mean here. You want the outputtet markdown File column to be absolute links to the file (e.g. [file.txt](file://Users/XXX/Documents/sample.c))? This would be a nice feature, but could result in some misunderstanding when sending the markdown todo list to others.

I would suggest relative links like file.txt

Relative links would only work when the file is saved to the project root. And as far as I know without the file:// prefix it wont work at all (github.com does some magic that is not part of the markdown spec to make relative links work).


3 days view

Is it possible to have a 3 days view instead of a full week view?
Thanks

fixed it, using syncfusion_flutter_calendar now.


runtime: require GetQueuedCompletionStatusEx in Windows netpoll code

In the runtime package, on Windows, if the function GetQueuedCompletionStatusEx is not available on the system, we use the simpler but slower GetQueuedCompletionStatus function.
According to https://docs.microsoft.com/en-us/windows/win32/fileio/getqueuedcompletionstatusex-func, GetQueuedCompletionStatusEx is available on Windows Vista, Windows Server 2008, and newer.
According to https://golang.org/wiki/MinimumRequirements, as of the Go 1.11 release we require at least Windows 7, or Windows 2008R2, or higher.
According to https://en.wikipedia.org/wiki/List_of_Microsoft_Windows_versions, Windows 7 is newer than Windows Vista.
Therefore, I believe that all supported Windows systems support GetQueuedCompletionStatusEx, and we can remove the code that uses GetQueuedCompletionStatus.
@alexbrainman @mattn Do you agree?

@ianlancetaylor I agree. We can remove all code that calls GetQueuedCompletionStatus.
Do you want me to do the honors?
Alex

(Same answer.)  I can do it, unless you would like to.  Thanks.

I agree with you