python311Packages.bandcamp-api: 0.2.2 -> 0.2.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.bandcamp-api is: Obtains information from bandcamp.com
meta.homepage for python311Packages.bandcamp-api is: https://github.com/RustyRin/bandcamp-api
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.3 with grep in /nix/store/7w8ivpi2fyqgnm9j0iylq29566bjqjs5-python3.11-bandcamp-api-0.2.3
found 0.2.3 in filename of file in /nix/store/7w8ivpi2fyqgnm9j0iylq29566bjqjs5-python3.11-bandcamp-api-0.2.3

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.bandcamp-api
python311Packages.bandcamp-api
spotdl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/7w8ivpi2fyqgnm9j0iylq29566bjqjs5-python3.11-bandcamp-api-0.2.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.bandcamp-api https://github.com/r-ryantm/nixpkgs/archive/41b534b40ff8ef3a79720b4e7a8b1e04bfa1c20b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/41b534b40ff8ef3a79720b4e7a8b1e04bfa1c20b#python311Packages.bandcamp-api

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/7w8ivpi2fyqgnm9j0iylq29566bjqjs5-python3.11-bandcamp-api-0.2.3
ls -la /nix/store/7w8ivpi2fyqgnm9j0iylq29566bjqjs5-python3.11-bandcamp-api-0.2.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

spotdl

2 packages built:

python310Packages.bandcamp-api
python311Packages.bandcamp-api

Maintainer pings
cc @dotlambda for testing.

spotdl is already broken on master.
https://hydra.nixos.org/build/239672092


Add percentile function for p-quantile

Description
Add percentile function for datasets with p-quantile based from an array of numbers. percentile value is 0 to 100.
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?

[ ] If this is a new feature, did you add files to the examples directory to demonstrate the feature?
[x] If this is a new feature, did you add documentation to the docs directory for the feature?

Hi @lulunac27a,  could you please add/update test files in the example vault that both demonstrates this function being used but also works as a way to test that it is working? You can either add a new file or update https://github.com/pyrochlore/obsidian-tracker/blob/master/examples/TestExpression.md to include usage of the new function and then update a few of the files in the diary directory to use as the dataset

I added the tests


feat Support compressed snapshot meta and job info

What problem does this PR solve?
Related PR: https://github.com/selectdb/ccr-syncer/pull/223
The backup meta and snapshot info are larger and might exceed the thrift max message size limitation. This PR compresses the backup meta and job info in both the GetSnapshot and RestoreSnapshot methods to avoid this.
To keep compatibility, the field compressed is added to TGetSnapshotResult to indicate whether the meta and job info are compressed; config enable_restore_snapshot_rpc_compression indicates whether the FE support reads the compressed meta and job info, the ccr-syncer should read this config before issuing a restore snapshot request with compressed meta and job info.
Release note
Support compressing large backup meta and snapshot job info, to avoid exceeding the thrift server limitation.
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[x] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall


Cursor shape not getting updated after transition to new scene

I'd like to report some cursor shape effects that I encountered while testing.
When transitioning to a new scene using a WalkHotspot and the new scene does not have a script attached to it the cursor still shows the same shape in the new scene and is only getting updated when moving it.
I could change the behavior by introducing a script for the new scene and by adding EgoVenture.check_cursor() in the _ready function. The behavior changed that the new scene first showed the neutral cursor and then switched to the right cursor when moving the mouse.
I could also reproduce wrong cursor shapes in the Carol example game in some locations.
For example when doing the transition from man22 to man 22a_cl (while slightly moving the mouse) it still shows the eye cursor from the previous scene. Once moving the mouse in the new scene it switches to the correct cursor shape (see screenshot below). It's not easy to reproduce, but it happened from time to time.

Another example which might be easier to reproduce is scene man12f_lift0. When I open the bag without any mouse movement it shows the neutral cursor shape. When moving the mouse by one pixel it switches to previous hand cursor shape of the old scene. When moving it another pixel it switches to the neutral cursor again.

That's sadly a known glitch currently because of godotengine/godot#45936. Sorry.

Would it be possible (as a mitigation) to embed the EgoVenture.check_cursor() function logic into the EgoVenture.change_scene() function?
This would require to loop through the children of the upcoming scene instead of the current scene.
It might be a bit tricky from an offset point of view for four_room scenes.
I've experimented a bit (for non-four-room scenes) and it seemed to work.
(However I'm still trying to get familiar with the Godot framework, so I might've overlooked something :-)
Benefit: It would no longer be required to call the check_cursor() explicitly.

fyi - here's how I changed the function change_scene as a workaround.
As the check cursor is done for four side rooms in another function I excluded them:
func change_scene(path: String):
get_tree().change_scene_to(_scene_cache.get_scene(path))
# run check cursor on new scene
var target_shape = Input.CURSOR_ARROW
var mousePos = get_viewport().get_mouse_position()
var scene = _scene_cache.get_scene(path).instance()
for child in scene.get_children():
	# not needed for four side rooms
	if child.filename == "res://addons/egoventure/nodes/four_side_room.tscn":
		return
	if "mouse_default_cursor_shape" in child and child.visible:
		var rect = child.get_rect()
		if rect.has_point(mousePos):
			if child is TriggerHotspot:
				child.on_mouse_entered()
			target_shape = child.mouse_default_cursor_shape
Speedy.set_shape(target_shape)

I'll reopen this issue though to keep it in mind, because the last change is still only a workaround.


Doc native params on mediaTypes (incl. cta)

Scope of work here is:

add info on cta field
update example code to use mediaTypes: { native ... API
add some links to our list of bidders with native support

Let me know what you think - thanks!

Code changes here for reference: https://github.com/prebid/Prebid.js/pull/1625

@mjacobsonny I think my latest commits address your feedback - let me know what you think


RuntimeError: DataLoader worker (pid(s) 1448, 8988) exited unexpectedly

When executing 09_dataloader.py got this error.
Currently running on Pytorch 1.5.

Try to set the num_workers to 0. This is a known issue, especially on windows. I added a comment in this file


make kompose go get-able

Fix #216

I was implying the CLI be go-getable in #216

@ericchiang  you mean go get github.com/kubernetes-incubator/kompose/cli ?

@ngtuna I was implying renaming the package main to kompose so go get would install the binary correctly. For users familiar with Go this is what they expect for most CLI go projects.
So at minimum
go get github.com/kubernetes-incubator/kompose/cli/kompose

It's obviously not critical. But it's pretty standard for Go projects :)

Okay thanks @ericchiang for pointing it out. It's pretty easy to rename the package. I will consider more on removing experimental tag of bundlefile.

@ericchiang I see it would be better to move main/main.go to the parent directory but not sure if it violates the standard of kubenertes-incubator or not. That is a bit of weird in path of the package when doing go get github.com/kubernetes-incubator/kompose/cli/kompose. It would be better with:
$ go get github.com/kubernetes-incubator/kompose

$ go get github.com/kubernetes-incubator/kompose

I'm for this (but that's just a preference).

Works for me but I can't officially review so it's not worth much.

ping @ericchiang @kadel Could you add your review on this ?

lgtm but I'll let @kadel add the review.

I found that make binary doesn't work on HEAD (not broken by this PR)
$ make binary
CGO_ENABLED=1 ./script/make.sh binary
---> Making bundle: binary (in .)
# command-line-arguments
./main.go:35: cannot use command.BeforeApp (type func(*"github.com/kubernetes-incubator/kompose/vendor/github.com/urfave/cli".Context) error) as type func(*"github.com/skippbox/kompose/vendor/github.com/urfave/cli".Context) error in assignment
./main.go:36: cannot use append(command.CommonFlags()) (type []"github.com/kubernetes-incubator/kompose/vendor/github.com/urfave/cli".Flag) as type []"github.com/skippbox/kompose/vendor/github.com/urfave/cli".Flag in assignment
./main.go:40: cannot use command.ConvertCommandDummy() (type "github.com/kubernetes-incubator/kompose/vendor/github.com/urfave/cli".Command) as type "github.com/skippbox/kompose/vendor/github.com/urfave/cli".Command in array or slice literal
./main.go:44: cannot use command.UpCommand() (type "github.com/kubernetes-incubator/kompose/vendor/github.com/urfave/cli".Command) as type "github.com/skippbox/kompose/vendor/github.com/urfave/cli".Command in array or slice literal
./main.go:45: cannot use command.DownCommand() (type "github.com/kubernetes-incubator/kompose/vendor/github.com/urfave/cli".Command) as type "github.com/skippbox/kompose/vendor/github.com/urfave/cli".Command in array or slice literal
make: *** [binary] Error 2

I found that make binary doesn't work on HEAD (not broken by this PR)

Could you re-run it ? I just run make binary successfully on HEAD.

I found that make binary doesn't work on HEAD (not broken by this PR)

Could you re-run it ? I just run make binary successfully on HEAD.

From the error log it seems that it's because I'm still using skippbox/kompose/
./main.go:35: cannot use command.BeforeApp (type func(*"github.com/kubernetes-incubator/kompose/vendor/github.com/urfave/cli".Context) error) as type func(*"github.com/skippbox/kompose/vendor/github.com/urfave/cli".Context) error in assignment

make binary works after I change my local directory from skippbox/kompose to kubernetes-incubator/kompose.

Merging


Account has insufficient balance for requested action.

Hello!
I love this bot, great work!
I run into some issues tho, please see below if you have any solution for my problem?
["2020-3-3 0:00:21"] [32mINFO [39m: [36mAttempting to execute USDT-ETC-ETH with an age of -646 ms and expected profit of 0.2931%[39m ["2020-3-3 0:00:21"] [32mINFO [39m: [36mBuying 4.86 ETCUSDT @ market price[39m ["2020-3-3 0:00:25"] [32mINFO [39m: [36mSuccessfully bought 4.86000000 ETCUSDT @ a quote of 40.77637200[39m ["2020-3-3 0:00:25"] [32mINFO [39m: [36mSelling 4.86 ETCETH @ market price[39m ["2020-3-3 0:00:27"] [31mERROR[39m: [36mAccount has insufficient balance for requested action.[39m

Thank you for your kind words and interest in the project!
I recommend running the app with the DEBUG log level as much more information is displayed. That really should be the default setting going forward.. #noteToMyself
But anywho, looking at that log snippet it appears that you are trading with a large latency from the market. Your last tick was 646 ms (which is HUGE) when it comes to automated trading. That should definitely be less than 100ms and honestly less than 10ms.
Now I'm looking closer, and your log reads an age of -646.. Negative? Oh buddy, never seen that before. I'll have to look into what scenario would cause that log.

What does the ping read when you startup the bot?


[webview_flutter_wkwebview] Fix crash when pop from a webview page in second engine which is make by FlutterEngineGroup

请问iOS WKWebview  WKUIDelegate 的以下方法：

(void)webView:(WKWebView *)webView runJavaScriptTextInputPanelWithPrompt:(NSString *)prompt defaultText:(nullable NSString *)defaultText initiatedByFrame:(WKFrameInfo *)frame completionHandler:(void (^)(NSString * _Nullable result))completionHandler;
如何在Flutter webview_flutter.dart中实现

@HuajingZeng Thanks for pointing this bug out! I created a full PR with tests and a version bump at: https://github.com/flutter/plugins/pull/6131. I'm going to close this PR in favor of the other.


Codepen example breaks with latest vugel

If you try any vugel version >= 1.0.13, Codepen example mentioned in readme gets broken

Good catch! https://github.com/Planning-nl/vugel/commit/faa068fb08f2e4b3fe8ec62fab282f4b68fb19bd#diff-e0996ff8a423b594eb5ad38891f650a8L17 has broken it, I am not quite sure why. For now you can use https://pastebin.com/raw/mpL94HkF if you want to use it with the codepen


Tabs for open files like Sublime/Notepad ++

It's good if you can add tabs on the top for the open files similar to Notepad++/Sublime Text. At present if we hide the file navigation side bar, we only have shortcuts to shuffle between all the open files, which means we directly can not go to a specific file without showing the sidebar, tabs will serve the true purpose and will give us more monitor real estate to be used. Thanks for this awesome editor. :)

Already a thread here for the feature request hence closing.


Ambiguous compare URL results in 404

Gitea version (or commit ref): 1.13.2
Git version: 2.26.2
Operating system: docker
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (https://try.gitea.io/our-org/custom-project/compare/main...company-org:payed-feature)
[ ] No

Log gist:

2021/03/07 18:51:43 ...s/context/context.go:139:HTML() [D] Template: status/404
	/go/src/code.gitea.io/gitea/modules/context/context.go:139 (0x166ce41)
	/go/src/code.gitea.io/gitea/modules/context/context.go:168 (0x166d275)
	/go/src/code.gitea.io/gitea/modules/context/context.go:155 (0x2016f75)
	/go/src/code.gitea.io/gitea/routers/repo/compare.go:359 (0x2016f47)
	/go/src/code.gitea.io/gitea/routers/repo/compare.go:531 (0x2019dcf)
	/usr/local/go/src/reflect/value.go:476 (0x4a6586)
	/usr/local/go/src/reflect/value.go:337 (0x4a5a78)
	/go/src/code.gitea.io/gitea/vendor/gitea.com/macaron/inject/inject.go:177 (0xd905d9)
	/go/src/code.gitea.io/gitea/vendor/gitea.com/macaron/inject/inject.go:137 (0xd8ffaa)
	/go/src/code.gitea.io/gitea/vendor/gitea.com/macaron/macaron/context.go:121 (0xd91afc)
	/go/src/code.gitea.io/gitea/vendor/gitea.com/macaron/macaron/context.go:112 (0x167a813)
	/go/src/code.gitea.io/gitea/modules/context/repo.go:593 (0x167a7fc)
	/usr/local/go/src/reflect/value.go:476 (0x4a6586)
	/usr/local/go/src/reflect/value.go:337 (0x4a5a78)
	/go/src/code.gitea.io/gitea/vendor/gitea.com/macaron/inject/inject.go:177 (0xd905d9)
	/go/src/code.gitea.io/gitea/vendor/gitea.com/macaron/inject/inject.go:137 (0xd8ffaa)
	/go/src/code.gitea.io/gitea/vendor/gitea.com/macaron/macaron/context.go:121 (0xd91afc)
	/go/src/code.gitea.io/gitea/vendor/gitea.com/macaron/macaron/context.go:112 (0x1678034)
	/go/src/code.gitea.io/gitea/modules/context/panic.go:39 (0x1678025)

Description
There is a specific structure of repositories and organizations we need in order to complete our work. First I'll explain our use case. In Steps to reproduce I'll show how I could create the organization/repository structure, exactly how I was expecting it to be. However when trying to create a specific PR the feature-branch is not listed, shown in Steps to show that feature branch is not listed. Finally Steps to show 404 and how to adapt compare URL shows how I got the 404 page and how I adapted the compare URL to get the desired PR.
The repository structure is available at https://try.gitea.io. If you'd like me to create this structure somewhere else, just tell me.
Use case

A company develops a open source project
we mirror the company's repo to our gitea server inside company-org organization
we fork their repository and we push to our customization we/custom-project repo
we create PR from the mirror of the company's public repo company-org/project to our customization our-org/custom-project to merge upstream changes
we pay the company to develop custom features based on our customized code and thus the company gets access to their company-org organization
company pushes to a fork company-org/custom-project of our customized repo
we create PR from company's custom repo company-org/custom-project to our customization our-org/custom-project

At the end we would like to have the following fork chain:
company-org/project (mirror) <- our-org/custom-project <- company-org/custom-project
Steps to reproduce

Created mirror from public repo: https://github.com/flozzone/project.git -> company/project
Created fork from company/project to our-org/custom-project
Created fork from our-org/custom-project to this-is-me/custom-project [1]
Changed ownership from this-is-me/custom-project to company-org [1]
Push changes on branch payed_feature to company-org/custom-project repo
Goto company-org/custom-project

Steps to show that feature branch is not listed
7a. Select main branch
8a. Click Create Pull Request
9a. Open pull from dropdown
10a. payed-feature is not listed
Steps to show 404 and how to adapt compare URL
7b. Select payed-feature branch
8b. Click Create Pull Request -> 404
9b. Change compare URL from https://try.gitea.io/our-org/custom-project/compare/main...company-org:payed-feature to https://try.gitea.io/our-org/custom-project/compare/main...company-org/custom-project:payed-feature -> Compare is correctly shown
[1] Steps 3. and 4. were required in order to create a fork in the same organization in which the mirror is located. Creating
the fork directly was not possible since the owner was not listed when forking.
Thoughts and questions
Is this 404 occurring because we have the mirror and a fork of same repository inside one organization? Why is the adapted compare URL working. IMHO this is a popular use case (despite the fact that no one reported this before) or is there another way to accomplish what I want?

the simple issue is that the shortcut url: https://try.gitea.io/our-org/custom-project/compare/main...company-org:payed-feature is looking for the payed-feature branch in company-org/project not company-org/custom-project.
How would gitea know that you meant company-org/custo-project?

This is what I thought, but gitea should now that I need a PR from company-org/custom-project since I selected it (see step 6) and clicked Create Pull Request. I am not constructing the compare URL myself.
For me it seems that the syntax which is used in the compare URL/view is ambiguous: ORG:BRANCH is not enough to identify a branch in my situation. The current syntax works well when the forked-from repo is owned by another organization with the assumption that it has the same name. With my structure I am hitting the limit to have just one fork of a repo in the same organization, this is why I needed step 3,4 to move the repo.
But I think this could be work around by first checking for ambiguity and in case use the full syntax: ORG/REPO:BRANCH. This is actually done in case I select some branch from company-org/project (See the screenshot below)

But somehow not all branches are shown. The payed-feature is not in the list:

while it should be there:

So the simplifying code is getting things wrong in this case. The trouble is I can't remember where I put it.

OK there is no way we can ever use the simplified url in this case - however, that seems like an overkill in the general case - because most people will not be doing this kind of naughty behaviour...
Aha. The trick is if the baserepo is a fork then that is the only way you could need the full repo id.
diff --git a/templates/repo/home.tmpl b/templates/repo/home.tmpl
index 4d8d28921..b77f28f92 100644
--- a/templates/repo/home.tmpl
+++ b/templates/repo/home.tmpl
@@ -64,7 +64,7 @@
                        {{if eq $n 0}}
                                {{if and .CanCompareOrPull .IsViewBranch (not .Repository.IsArchived)}}

-
+
                                                        <button id="new-pull-request" class="ui compact >

sorts out the initial 404

This should make 1.14 but likely rc2

This won't be merged until v1.14.0 (or a release candidate) is released.
You can get the template files from: https://github.com/CL-Jeremy/gitea/tree/backport-fix-14921/templates/repo. Download these two raw files: home.tmpl and diff/compare.tmpl, put them in $GITEA_CUSTOM/templates/repo and you are good to go.
These files are generated with:
git checkout v1.13.4
git remote add zeripath https://github.com/zeripath/gitea.git
git fetch zeripath fix-14921-compare-urls-on-forked-forks
git cherry-pick zeripath/fix-14921-compare-urls-on-forked-forks


Dark/Light mode switch is placed weirdly on mobile

Describe the bug
Currently the dark/light mode switch is placed below the navigation on mobile. That is rather unexpected.
To Reproduce
Steps to reproduce the behavior:

Go to the website
Scale it to mobile
See error

Expected behavior
Usually the dark/light switch is near or at the top.

can I work on this issue?

hi @Hbrehman , feel free to do this! If you need help, let me know.


[Enhancement] support load struct from parquet

What type of PR is this:

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

Which issues of this PR fixes:

Fixes #

support load struct from parquet
support map/struct on aggreagte key model

MySQL td> select * from s2;
+---------+------------+----------------------------------------------------------------------------------------------------------------------------------------------------------+
| col_int | col_string | col_struct                                                                                |
+---------+------------+----------------------------------------------------------------------------------------------------------------------------------------------------------+
| 2       | 2-test     | {"c_int":2,"c_float":2.1,"c_double":2.2,"c_char":"char2","c_varchar":"varchar2","c_date":"2020-02-02","c_timestamp":"2020-02-02 20:00:00","c_boolean":1} |
| 1       | 4-test     | {"c_int":4,"c_float":4.1,"c_double":4.2,"c_char":"char4","c_varchar":"varchar4","c_date":"2020-04-04","c_timestamp":"2020-04-04 20:00:00","c_boolean":1} |
| 3       | 3-test     | {"c_int":3,"c_float":3.1,"c_double":3.2,"c_char":"char3","c_varchar":"varchar3","c_date":"2020-03-03","c_timestamp":"2020-03-03 20:00:00","c_boolean":1} |
+---------+------------+----------------------------------------------------------------------------------------------------------------------------------------------------------+

Problem Summary(Required):

Checklist:

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

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

Bugfix cherry-pick branch check:

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

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:heart_eyes: pass : 2 / 2 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/ast/ColumnDef.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/catalog/AggregateType.java
1
1
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 2 / 2 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/ast/ColumnDef.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/catalog/AggregateType.java
1
1
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 2 / 2 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/ast/ColumnDef.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/catalog/AggregateType.java
1
1
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 2 / 2 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/ast/ColumnDef.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/catalog/AggregateType.java
1
1
100.00%
[]


Compilation error if variable isn't initialized outside if statement

#pythran export f (float64 [][], int)
def f (array, x):
    case = 0
    if x == 1:
        case = 1
    else:
        case = 2

    array[0][0] = case

Commenting out "case = 0" line causes the compilation error.

That's a funny one. The issue actually comes from case which is a C++ reserved identifier. We try hard to protect these, But I forgot one. That's an easy fix, thanks for reporting nevertheless!

That thought crossed my mind but for some reason I didn't try just changing the name of the variable. Ugh...
Thanks for your work man.


masks not detected properly

Hi there,
the results seem weird, even after changing MIN_SIZE_TEST.
is this normal:
Mask-RCNN-FBNetV3A:

mask_rcnn_fbnetv3a_dsmask_C4.yaml:

mask_rcnn_fbnetv3g_fpn.yaml

I couldn't reproduce your problem.
Did you use trained=True when loading the model from model_zoo?
This is my script:
# Get the model
from d2go.model_zoo import model_zoo
model = model_zoo.get('mask_rcnn_fbnetv3a_C4.yaml', trained=True)
# Download the image
import cv2
from matplotlib import pyplot as plt
!wget http://images.cocodataset.org/val2017/000000439715.jpg -q -O input.jpg
im = cv2.imread("./input.jpg")
plt.imshow(im)
# Make prediction
from d2go.utils.demo_predictor import DemoPredictor
predictor = DemoPredictor(model)
outputs = predictor(im)
# Visualize
from detectron2.utils.visualizer import Visualizer
from detectron2.data import MetadataCatalog, DatasetCatalog

v = Visualizer(im[:, :, ::-1], MetadataCatalog.get("coco_2017_train"))
out = v.draw_instance_predictions(outputs["instances"].to("cpu"))
plt.imshow(out.get_image()[:, :, ::-1])

This is my output:

A

yes, i used trained=True as in d2go_beginner.ipynb
i am currently looking into the cause, meanwhile send the environment.yml
conda env export > environment.yml

alright, i found the solution.
it was caused by the version of the pytorch and numpy used.
using the latest nightly version of pytorch from conda , and the latest stable numpy have solved my issue.


I did not really want to merge my work in the codeschool branch, sorry. Did not know, how to push, but this has already worked with commiting in my own master branch. Thanks.

Tasks completed

Great work! Try to keep the pull request's heading as precise as possible.


Add links to statistics third-parties

On author profile page

On project detail page

Heads up @andrew. :) Thanks for libraries.io.

Awesome! Thanks for the heads up!


docs: improve CONTRIBUTING.md

#392  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


Native submit not being prevented

Hi braintree team. Thanks for your excellent hosted fields solution. What I running into is that Im manually submitting the form with the hosted field containers via
document.getElementById(this.ccFormId).submit();

But the native submit handler is running. In your docs you say to bypass the braintree submission handler you can do the following
var form = document.getElementById('my-form');
HTMLFormElement.prototype.submit.call(form);

I'm not using that exact call, and I do have the OnPaymentMethodRecived callback in place but the form is still submitting natively instead of providing the nonce or validation errors.
Here is the form(in jade)
form(action="", method="POST").grid-block#cc-input-form
    .grid-content.small-6
      #card-number.hosted-input
    .grid-content.small-3
      #expiration-date.hosted-input
    .grid-content.small-3
      #cvv.hosted-input

The reason I have no submit input is that I'm using Angular and this form is actually a part of a larger and modular ngForm(Which is not a form at all, but a construct that sort of acts like one, therefore this is NOT a <form> inside a <form> which is invalid html)
I'm wondering if you could provide some feedback.
UPDATE: For kicks I tried putting in a submit input, but the same result.
Do we actually need a click event to that submit button to get the braintree nonce goodness?
UPDATE: I overcame this with a hidden submit input that I then had to run .click() on.
See #151

Using .submit() directly on a form element will bypass all javascript submit handlers. This may be why you are not seeing onPaymentMethodReceived being fired.
If you want to programmatically dispatch a submit event on a form that has no button to click, you can do this:
var form = document.getElementById('my-form');
var e = document.createEvent('Event');

e.initEvent('submit', true, true);
form.dispatchEvent(e);

See MDN's Creating and triggering events

@mrak Thanks so much. I didn't think about doing it that way.
Thanks I think we can close this.

@mrak perhaps what you said would be a helpful addition to the docs.


A problem occurred evaluating project ':react-native-camera'. > Could not find method compileOnly() for arguments [com.facebook.react:react-native:+] on object of type org.gradle.api.interna l.artifacts.dsl.dependencies.DefaultDependencyHandler.

Replace "compileOnly" with provided in /project/node_modules/react-native-camera/android/build.gradle

in /android/app/build.gradle, replace compile with implementation

Hi, I have many problems to implement it, where you have to replace that or any line. Thanks


GetIngestionStatusBySourceId always returns the status of Queued

Hello,
I am having trouble getting the GetIngestionStatusBySourceId method to work (https://docs.microsoft.com/en-us/azure/data-explorer/kusto/api/netfx/kusto-ingest-client-examples#ingest-from-local-files-and-report-status-to-a-table). It always seem to return a status of Queued, even after the ingestion is completed as per the data explorer (using .show operations). Am I missing anything? I updated the method to break the loop based on previous and current row counts for the table in question and as expected, it works. But its not ideal, since we would be ingesting numerous files and this method won't be reliable.
Thanks!

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

ID: 49e16209-5d32-cebb-cec1-8c5be30a25aa
Version Independent ID: 781d5ac3-83e4-9aa2-dfd7-35ace0fcd691
Content: Kusto.Ingest code examples - Azure Data Explorer
Content Source: data-explorer/kusto/api/netfx/kusto-ingest-client-examples.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

Sorry, it does work perfectly. I missed setting the ReportLevel and ReportMethod properties for the KustoQueuedIngestionProperties instance. Closing this one out. Thanks!

@karthik25 Glad that you were unblocked. Thanks for sharing your findings here 😊


Move dj to its own directory and rename

Summary
What do we think about the following changes for better directory structure?

Move dj to its own directory under this repo, so that it fits the same pattern of a typical monorepo (and is parallel in setup to the query and reflection services)
Move each of these services into directories with names that match the Python package, so dj -> datajunction-server, djqs -> datajunction-query, and djrs -> datajunction-reflection

The directory structure would look like this:
├── datajunction-server
│   ├── dj
│   │   ├── ...
│   ├── alembic
│   ├── tests
│   ├── pyproject.toml
├── datajunction-query
│   ├── djqs
│   │   ├── ...
│   ├── alembic
│   ├── tests
│   ├── pyproject.toml
├── datajunction-reflection
│   ├── djrs
│   │   ├── ...
│   ├── alembic
│   ├── tests
│   ├── pyproject.toml

Test Plan

[ ] PR has an associated issue: #
[ ] make check passes
[ ] make test shows 100% unit test coverage

Deployment Plan

This makes sense to me. What will this ultimately look like from a pip install perspective?

@CircArgs it would still just be pip install datajunction-server, and then for each of the other services, pip install datajunction-reflection and pip install datajunction-query. Only the directory structure changes


Swift-Format hangs on swift 6.0.2

Environment:

WSL2 Ubuntu 20.04
Swift 6.0.2

I saw #861 and created a .swift-format file in the project dir (also tried on layer above), but I'm still having issues with a hanging swift-format.

is there a way to enable verbose diagnostics for swift-format or is there any other method to find out, where its getting stuck?
Do I maybe just have to wait for 6.0.3?

Running swift-format without any arguments will wait for input from stdin. You need to pass a few flags to have it format files in place. What you might want is something like swift-format -r -i ..
There's some good discussion about the default behavior being less-than-intuitive here: https://forums.swift.org/t/default-behavior-of-swift-format/74314. It's something we'd like to improve in the future.

Synced to Apple’s issue tracker as rdar://139283088

Running swift-format without any arguments will wait for input from stdin. You need to pass a few flags to have it format files in place. What you might want is something like swift-format -r -i ..
There's some good discussion about the default behavior being less-than-intuitive here: https://forums.swift.org/t/default-behavior-of-swift-format/74314. It's something we'd like to improve in the future.

Thank you, working like a charm when you do it right. I feel dumb now. :)

Thank you, working like a charm when you do it right. I feel dumb now. :)

You shouldn't! It's good signal that we can do more to smooth out the on-boarding process.
I'll go ahead and close this since it's resolved.

Let’s keep this issue open to consider a potential default behavior change of swift-format without any arguments.

That's fine. I've updated the title to better reflect what we want to do.


Added Omega Radiance

I've added omega radiance

i'll close this one, as https://github.com/hk-modding/modlinks/pull/1446 is still open, but there we have to probably wait for a modcommon to be put on modlinks


restore demand paging PR + PC fixes

This PR:

Restores all the demand paging commits that were reverted on Friday (the root cause of the sentinel failure is #31546)
Adds CONFIG_ARCH_MAPS_ALL_RAM. This signals to the kernel that all RAM has a bootup mapping, and not just the Zephyr image. Page frame management is unaffected, but this will be taken into account when establishing new memory mappings, as to not disturb the existing mapping for all RAM.
Enable CONFIG_ARCH_MAPS_ALL_RAM for ACPI targets. Various PC-like targets had their VM size increased.

Thanks @andrewboie! I've started HW testing, will post details soon.

Thanks @andrewboie! I've started HW testing, will post details soon.

This unfortunately breaks EHL booting with SBL. Reverting to commit 3746ebc305f5ae581665cd033637a11caee5b836 (i.e. right before this PR went in) makes it work again.

This unfortunately breaks EHL booting with SBL. Reverting to commit 3746ebc305f5ae581665cd033637a11caee5b836 (i.e. right before this PR went in) makes it work again.

This unfortunately breaks EHL booting with SBL. Reverting to commit 3746ebc (i.e. right before this PR went in) makes it work again.

Using git bisect to narrow this down gives the following:
5c47bbc5010d98c6f6d795aa6d268138c3463bbd is the first bad commit
commit 5c47bbc5010d98c6f6d795aa6d268138c3463bbd
Author: Andrew Boie<EMAIL_ADDRESS>Date:   Thu Dec 10 11:23:21 2020 -0800

    x86: only map the kernel image

    The policy is changed and we no longer map all page frames.

    Signed-off-by: Andrew Boie<EMAIL_ADDRESS>
 arch/x86/gen_mmu.py | 27 ++++++++++++++++++---------
 1 file changed, 18 insertions(+), 9 deletions(-)

This unfortunately breaks EHL booting with SBL. Reverting to commit 3746ebc (i.e. right before this PR went in) makes it work again.

Using git bisect to narrow this down gives the following:
5c47bbc5010d98c6f6d795aa6d268138c3463bbd is the first bad commit
commit 5c47bbc5010d98c6f6d795aa6d268138c3463bbd
Author: Andrew Boie<EMAIL_ADDRESS>Date:   Thu Dec 10 11:23:21 2020 -0800

    x86: only map the kernel image

    The policy is changed and we no longer map all page frames.

    Signed-off-by: Andrew Boie<EMAIL_ADDRESS>
 arch/x86/gen_mmu.py | 27 ++++++++++++++++++---------
 1 file changed, 18 insertions(+), 9 deletions(-)

I added some debug logs to the ACPI table code with the suspicion that something is outside, but it looks to me like everything is still within 2MB that's declared for the board:
Starting MB Kernel ...
find_rsdp search 0xf0e0
find_rsdp rsdp 0xfff80
z_acpi_find_table rsdt 0x78854030
z_acpi_find_table returning sdt 0x788545b0
*** Booting Zephyr OS build zephyr-v2.4.0-3554-g3746ebc305f5  ***
z_acpi_find_table rsdt 0x78854030
z_acpi_find_table returning sdt 0x78854530
Hello World! ehl_crb_sbl

Btw, I seem to be unable to increase the declared RAM amount from 2MB (even if I also increase KERNEL_VM_SIZE) - it results in some build errors.

I added some debug logs to the ACPI table code with the suspicion that something is outside, but it looks to me like everything is still within 2MB that's declared for the board:
Starting MB Kernel ...
find_rsdp search 0xf0e0
find_rsdp rsdp 0xfff80
z_acpi_find_table rsdt 0x78854030
z_acpi_find_table returning sdt 0x788545b0
*** Booting Zephyr OS build zephyr-v2.4.0-3554-g3746ebc305f5  ***
z_acpi_find_table rsdt 0x78854030
z_acpi_find_table returning sdt 0x78854530
Hello World! ehl_crb_sbl

Btw, I seem to be unable to increase the declared RAM amount from 2MB (even if I also increase KERNEL_VM_SIZE) - it results in some build errors.


chore(deps): update dependency @sentry/cli to v1.51.0

This PR contains the following updates:

Package
Type
Update
Change

@sentry/cli (source)
devDependencies
minor
1.50.0 -> 1.51.0

Release Notes

getsentry/sentry-cli
v1.51.0
Compare Source

feat: Add dist option to react-native appcenter command (#653)
ref: Notify user about missing sudo command instead of incorrect "No such file or directory" when updating/uninstalling sentry-cli (#656)
fix: Remove redundant Closing connection 0 warnings after every HTTP request (#657)
fix: Update release structure for XCode React Native calls (#660)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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

v3.0.44
GitHub release

Your semantic-release bot :package::rocket:


tox.ini missing

I am trying to implement some new functionality which requires me to run the tests.
However, it seems like the Tox configuration file is not committed.

When I run pyscript tests I do it from a clone of homeassistant core, which has a tox.ini file.  I thought it was easier doing it there since it has all the scaffolding, including via script/setup and source venv/bin/activate etc.


DOC: Add Index.difference to API doc

Add Index.difference to api doc rather than deprecated Index.diff  (#8226)
Or is it OK to remove Index.diff because #6581 lists #8227 (which closes #8226)?

yep no problem with changing this in the API docs (but move as @jorisvandenbossche mentions)

The deprecation of Index.diff is actually not explicitely mentioned in #6581, we should add that. Also, we should clean up that list for 0.17, as not all things listed there should already be removed in 0.17

@jorisvandenbossche yep, that list really should be for 'next-major-release' then separate section for 0.17.0

I think it is natural under the same category as intersection and union. How about moving difference and sym_diff to "Combining / joining / merging" category, renaming it to "Combining / joining / set operations"?

yes, sounds good to me

agreed, making a new sub-section for setops would be gr8.

OK, did the suggested change. Pls check.


Provide a better serialization format for protocol commands

Protocol commands are right now serialized via Java serialization.

A faster, more compact, serialization format may be used instead, such as
ProtocolBuffers or Avro.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 13 Dec 2009 at 2:35

Original comment by<EMAIL_ADDRESS>on 17 Apr 2010 at 10:44

Added labels: Project-Terrastore

Currently evaluating:
* Avro (http://hadoop.apache.org/avro/)
* Fudge (http://www.fudgemsg.org)
* Kryo (http://code.google.com/p/kryo/)

Original comment by<EMAIL_ADDRESS>on 31 Dec 2009 at 2:59

Added labels: Priority-Medium
Removed labels: Priority-Low

Original comment by<EMAIL_ADDRESS>on 16 Nov 2010 at 1:00

Added labels: Milestone-0.8.0

Node communication protocol is now based on MessagePack (http://msgpack.org/),
and so faster and more compact.
Moreover, for even lower memory and bandwidth consumption, communication can be
compressed by enabling the --compressCommunication server switch.

Original comment by<EMAIL_ADDRESS>on 16 Nov 2010 at 3:05

Changed state: Fixed

Moved to high priority due to discovered bottlenecks in node-to-node
communication when large commands (i.e. as a result of a huge map request) are
sent over the wire.

Original comment by<EMAIL_ADDRESS>on 16 Nov 2010 at 1:00

Changed state: Started
Added labels: Priority-High
Removed labels: Priority-Medium


React SDK

Build React-Celo SDK Docs

Follow CLI format

Revisit in future after larger SDK discussion.


When using the @mui/material library, there are no issues during development, but problems arise after packaging.

System Info
System:
OS: macOS 13.5
CPU: (10) arm64 Apple M1 Pro
Memory: 64.55 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.16.1 - ~/Library/Caches/fnm_multishells/50124_1692009561527/bin/node
Yarn: 1.22.19 - ~/Library/pnpm/yarn
npm: 9.5.1 - ~/Library/Caches/fnm_multishells/50124_1692009561527/bin/npm
pnpm: 8.6.12 - ~/Library/pnpm/pnpm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Browsers:
Chrome: 115.0.5790.170
Safari: 16.6
npmPackages:
@rspack/cli: ^0.2.11 => 0.2.11
Details
When using the @mui/material library, there is no problem with rspack serve, but there is a problem with rspack build. The issue lies in the file<EMAIL_ADDRESS>'use client';

import * as React from 'react';
import { useTheme as useThemeSystem } from '@mui/system';
import defaultTheme from './defaultTheme';
import THEME_ID from './identifier';
export default function useTheme() {
  const theme = useThemeSystem(defaultTheme);
  if (process.env.NODE_ENV !== 'production') {
    // eslint-disable-next-line react-hooks/rules-of-hooks
    React.useDebugValue(theme);
  }
  return theme[THEME_ID] || theme;
}

The line theme[THEME_ID] || theme causes an error during build. However, there is no problem when printing THEME_ID on the previous line (using this variable beforehand).
Reproduce link
https://github.com/suxiaoshao/rspack-mui-bugs
Reproduce Steps
pnpm install

pnpm dev

# Everything is normal.

pnpm build
pnpm preview

# This is where the problem arises.

try nightly, this issue has been fixed, IIRC.

feel free to reopen this issue if you have any further question.

try nightly, this issue has been fixed, IIRC.

怎么使用nightly,有版本号吗

try nightly, this issue has been fixed, IIRC.

怎么使用nightly,有版本号吗

@rspack/cli@nightly


New Domain :p

I recently found another phishing link that is to scam people by making them think they get free Discord Nitro.

[x] Yes, I have checked both the domain-list.json and suspicious-list.json files, it is not there.
[x] Yes this is a Discord-related scam.
[x] Yes, it is in alphabetical order.

Proof:

A log showing the link that has been deleted by one of the moderators

Hey, tysm for your report!
I run a quick test of the domain and It looks like it is a ip grabber
see https://urlscan.io/result/e4a4f208-ba1f-4479-b607-feb9dddfb68e/
ip grabbers should not get on this list pretty sure

Ah alrighty! Just wanted to make sure


Should include author information

I noticed that we don't currently include author information (making it impossible to search for libraries by a given person).

It is now possible to specify this in the impact.json file.  If that fails, it pull the information from crawling GitHub.  But this typically points to modelica-3rdparty, for example.  So it would be good for users to start using the impact.json approach.


Is there any way to update cookie jar using REST API?

To automate burp suite active scanning process, we are using your rest-api extension. However, in our use case, we have to update the cookies in the cookie jar via REST API. I dont think it is possible using the currently provided APIs.
Is there any way where we can update the cookies in each and every HTTP request of proxy history using REST API? I have tried with Burp Macro but it is not working as expected.
Thanks.

The BurpExtender API seems to provide this method
void	updateCookieJar(ICookie cookie)
This method is used to update the contents of Burp's session handling cookie jar.

so it's a matter of trying implementing the endpoint. We will do it.

This is merged, and available in master!

Thanks. I will check it out.


(Postgres): We show schemas/tables we can't query

Context
When a pg data source is added, we show the system schemas pg_toats, pg_catalog, information_schema along with their tables. However, they cannot be queried:

TBD if GlareDB is supposed to be able to query them
Expected
We either don't show data source system tables, or make it obvious that GlareDB can't read from them
Actual
Use can double-click a table in a system schema, then run and get an error ---> confusiuon
In other cases, someone might be surprised to see them (perms thing)
Impact
Confusion, possibly more

Wrong repo, my bad


In mobile view the opening video does not occupies entire screen

Also, a problem with the last image of the video.

Assign this to me @QAZIMAAZARSHAD

I want to work on this.
Please do assign this to me.

I want to work on this.
Please do assign this to me.

@purohitmansi first complete other assigned issues then I will assign you.

Thank you for assigning me these. Please give me 2 days to solve out these
issues. How many days are given to solve an issue?
On Sun, 11 Jul 2021 at 13:18, Qazi Maaz Arshad @.***>
wrote:

I want to work on this.
Please do assign this to me.
@purohitmansi https://github.com/purohitmansi first complete other
assigned issues then I will assign you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/QAZIMAAZARSHAD/Voters-Registration-Portal/issues/298#issuecomment-877756917,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARMXVKTAJUVWSYK2INWHKKDTXFEC7ANCNFSM47SWA35A
.

Thank you for assigning me these. Please give me 2 days to solve out these issues. How many days are given to solve an issue?
…
On Sun, 11 Jul 2021 at 13:18, Qazi Maaz Arshad @.***> wrote: I want to work on this. Please do assign this to me. @purohitmansi https://github.com/purohitmansi first complete other assigned issues then I will assign you. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#298 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARMXVKTAJUVWSYK2INWHKKDTXFEC7ANCNFSM47SWA35A .

Depends on the level of the issue.
Importance of issue.
The number of other contributors in line to solve that particular issue.

Assign this to me Please.

Assign this to me Please.

Under DCP or LGMSoC?

Under LGMSoC .


first release of Security Summary information

Initial version of the Security Summary information page to support the Security Summary report that will be included as SW products are published

closing to adjust registered symbol issue


Add queries for extra check

Checklist

[ ] Issue(s) linked

Let's combine this with other branch queries for extra checks


[WIP] Include module sums

Main updates:

[x] Include modules sums skimming
[x] Add simple progress information to skimming
[ ] Include color mapping for module sums
[ ] Deal with columns having a different number of elements per event

Unrelated updates:

[x] Add genpart_pos* variables (x, y and z: gen particle position per layer) as a vector of vectors of floats: one vector (currently of length 50) per generated particle
[x] Throw error if an event (default or chosen by the user) does not exist in the dataset

@isehle @mchiusi Is there any update on module sums?
This PR is so outdated that we might simply create a new one when time comes.

@isehle @mchiusi Is there any update on module sums?
This PR is so outdated that we might simply create a new one when time comes.

Yes, that's a good idea. Thank you for cleaning up the list of outdated PRs.


Web: GestureRecognizers in nested TextSpans do not work.

If I have nested TextSpans in a RichText widget, adding a GestureRecognizer to the top level TextSpan will work fine, but adding one to a nested TextSpan will not. This is only an issue with Flutter for web, not standard Flutter.
This works:
RichText(
  text: TextSpan(
    text: 'TOP',
    recognizer: TapGestureRecognizer()..onTap = _incrementCounter,
    children: <TextSpan>[
      TextSpan(
        text: 'CHILD',
        // recognizer: TapGestureRecognizer()..onTap = _incrementCounter,
      ),
    ],
  ),
),

This does not:
RichText(
  text: TextSpan(
    text: 'TOP',
    // recognizer: TapGestureRecognizer()..onTap = _incrementCounter,
    children: <TextSpan>[
      TextSpan(
        text: 'CHILD',
        recognizer: TapGestureRecognizer()..onTap = _incrementCounter,
      ),
    ],
  ),
),

If both are present, the parent will win (I assume this is intended behaviour).
My use-case is nested spans to be able to colour separators in a slide credit, where I need different links. Or maybe I don't understand how best to work with RichText and TextSpan for something like this?

Found the same issue while porting  flutter_linkify to flutter web.
Here is an example of it:
https://github.com/dbacinski/flutter_linkify/tree/flutter_web/example
Lib is essentially doing the same thing as described above:
return TextSpan(
    children: elements.map<TextSpan>(
      (element) {
        if (element is LinkableElement) {
          return TextSpan(
            text: element.text,
            style: linkStyle,
            recognizer: onOpen != null
                ? (TapGestureRecognizer()..onTap = () => onOpen(element))
                : null,
          );
        } else {
          return TextSpan(
            text: element.text,
            style: style,
          );
        }
      },
    ).toList(),

Thank you for sample. We reproduced issue. Working on a fix.

Hi. I am experiencing some issues related to this topic.
I have the code:
RichText(
            text: TextSpan(
              text: 'Package goes from ',
              style: TextStyle(fontSize: 36, color: Colors.black),
              children: <TextSpan>[
                TextSpan(
                  text: _originValue ?? 'here', // _originValue is a String variable (not initially defined)
                  style: TextStyle(
                      color: Colors.deepPurple,
                      fontWeight: FontWeight.bold,
                      decoration: TextDecoration.underline),
                  recognizer: TapGestureRecognizer()
                    ..onTap = () async {
                      _originValue = await Navigator.push(context,
                          MaterialPageRoute(builder: (context) => Cities()));
                    },
                ),
                TextSpan(
                  text: ' to ',
                ),
                TextSpan(
                  text: _destinationValue ?? 'there', // _destinationValue is a String variable (not initially defined)
                  style: TextStyle(
                    color: Colors.deepPurple,
                    fontWeight: FontWeight.bold,
                    decoration: TextDecoration.underline,
                  ),
                  recognizer: TapGestureRecognizer()
                    ..onTap = () async {
                      _destinationValue = await Navigator.push(context,
                          MaterialPageRoute(builder: (context) => Cities()));
                    },
                ),
              ],
            ),
          )

All of this is in a StatefulWidget (in case it's important)

It works fine on mobile (both iOS and Android). Recently I added web support, but it does not always work on web.
The steps I used to add web support from beta channel
flutter channel beta
flutter upgrade
flutter config --enable-web

After running flutter run -d chrome it renders like this:

Where only there button works, and the state of it changes.

However, the button here doesn't work. And it doesn't matter if I first click on here or there only there button works.
I think I should have the patch of this issue, as I upgraded to beta just a few hours ago. I think it did not cover my problem.
Any further help will be greatly appreciated, thank you.

PS
The output from my flutter doctor -v

I find the same problem in mobile devices. I commit it here.
https://github.com/flutter/flutter/issues/48231.
@ferhatb  @yjbanov

Same problem here. I can't show the a code snippet in the moment but I can say that it's not working properly. Its stange that it recognizes when I tap the widget fast and strongly.

We're tracking the issue with gesture recognition in text spans in https://github.com/flutter/flutter/issues/46975

I just find out that TapGestureRecognizer is not working in proper way when RichText has textAlign with TextAlign.center at least in web. It's like the tap still recognized when align is TextAlign.left.

This is still broken on Flutter v1.17.0-3.4.pre  not sure why it is closed...

Has been fixed in latest flutter build. Remember to add _textKey to your RichText like so.

@Dennis-Mwea your example is not using a TextAlign other than TextAlign.left. It is still broken on Flutter 1.18.0-11.1.pre now


The Geometric  parser is not working correctly

To see this problem please create this svg file

Now create an application with Win2D and create same path. The image you get is not the same.
You write beautiful code (I can't write code like you do) and started to look your code and parsed my path and realized there is bug in your code. In this example you placed first 2 element and then third element is wrong.  I experimented with many complex svg files and saw the output image is not correct.
Thank you for sharing code with me. Good luck
best regards
aghaDotKhanAtHotmailDotCom

example.pdf

Thanks for reporting this. Let me have a look.

Thank you my friend Ratish Philip:
Thank you for sharing code with me.
https://1drv.ms/f/s!AtjIdnVOoJXtluRlhqxfQT3_QUPxrg
When you have time please download my applications.
Please look at TimeMachine.
ThemeClock and double click on it.
Rubik’s cube.
Similar work I would like to do under Win2D.
Best regards
Agha Khan
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Ratish<EMAIL_ADDRESS>Sent: Monday, August 7, 2017 11:56 AM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [ratishphilip/CompositionProToolkit] The Geometric parser is not working correctly (#11)
Thanks for reporting this. Let me have a look.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ratishphilip/CompositionProToolkit/issues/11#issuecomment-320750261, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFEm2yYi9bsbVNA7ScxhMfdeRAXipoJZks5sV13lgaJpZM4Ouhrv.

@AghaKhan I had a look at the svg defined in example.pdf file.
It seems that there is an extra line break in the path data which is causing the issue. (after 1-, see image below)

Closing this as the problem was with the svg data.


test(GUI): fix non-running test in DrivesModel test suite

We have a test to ensure a drive is not auto-selected when its
protected, however the real functionality of this test was not being
tested given that the size the fake drive had was not enough to hold the
image.
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Related to this: presumably Etcher should also not auto-select system drives, when you've enabled 'dangerous mode'? Currently it does auto-select the system drive (when there's only one available), which IMHO doesn't "feel right"?

Good point. I'll send a PR for it.

See https://github.com/resin-io/etcher/pull/1069


Feature: Better control code and file extension support

Features
This request is for a few things that are all related...

Show escape and other control characters on display, should look similar to how the old DOS EDIT.COM would display these characters.

Maybe with a toggle that will show space/placeholders for space, tab and other normally hidden characters.

Allow specifying features not just by type, but by file extension (this part may be a duplication)

codepage
line ending
etc.

Example
[.ans, .asc, .msg, .cfg]
{
  ...
}

Effectively, a section beginning with a . as a file extension, optionally with /,\s*/ separators...
Reasoning:
I'd rather not have to create a new file type in order to specify/align a file extension to have a cleaner editing experience.  I'd also like to be able to hand edit files that have ANSI escape codes, and in the case of CP437 or even UTF line/block characters.
The Remote SSH plugin is amazing.  I have a classic BBS setup on a system that is remote to me, and a lot of the files are DOS line endings and CP437 (IBM DOS US code page), And while the Remote SSH gives me incredibly easy access, I really cannot effectively make tweaks or edits to these files.

@sandy081 Both...

"Allow specifying features by file extension" - I want to be able to specify the codepage and the line endings etc by file extension.  Right now, I can only do it by supported "file type" ...  I'm working with text files, but these files are codepage 437, and I only want .ans, .asc, etc.  It's not a programming language, so there's no highlighting... just certain file extensions.

"Show escape and other control characters" - if you load a .ans (ansi) file in the old EDIT.com, you would see characters represented for the control characters (byte values under 32) [0]

I cannot be the only person that wants to edit files with control characters in a specific code page and be able to see the control characters in something I can reasonably copy/paste/modify.
[0] https://www.ascii-codes.com/

Right now, I can set something specific to a file type, like [css], but that isn't a file extension, just a type... I want to make settings by file extension.
The second half, and the reason I want this, is that currently, VS Code lacks any display for control characters in a file.

By duplication, I meant that there may be another issue already requesting said feature, but with tens of thousands of issues, it's hard to grok through.

My mistake, it looks like, when I have the "files.encoding": "cp437", it does show the control and line/block art characters... however, I really would like to set this by file extension.. going to close and create a new issue just for the configuration option.


cairomm_1_16: fix darwin build

Motivation for this change
ZHF: #122042
cc @NixOS/nixos-release-managers
Hydra failure: https://hydra.nixos.org/build/142578519/nixlog/1
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 123020 run on x86_64-darwin 1

3 packages failed to build:

cairomm_1_16
gtkmm4
pangomm_2_48

end of the log: https://gist.github.com/rmcgibbo/5f534f91b91fab01abff03da7ee58dd4

Hmm, could this be related to the darwin sandbox? My local build was without sandboxing.
If so, is it crude if we just disable tests on darwin?

Hmm, could this be related to the darwin sandbox? My local build was without sandboxing.
If so, is it crude if we just disable tests on darwin?

It looks like sandboxing related to me as well. There is no policy for this that I know of (but it would be nice if someone could file an issue to start the discussion). If it builds without sandbox, I would not disable tests and leave it with a broken sandboxed build for now.

Leaving stuff broken on Hydra feels a bit wrong, though. It inhibits efforts like ZHF.

Leaving stuff broken on Hydra feels a bit wrong, though. It inhibits efforts like ZHF.

Hydra doesn't enable sandboxing. If you want to disable tests please leave a comment that this is a due to sandboxing, so that we can revisit this later.

Alright, I pushed a commit that disables tests on Darwin only, with a comment.

Result of nixpkgs-review pr 123020 run on x86_64-darwin 1

1 package failed to build:

gtkmm4

2 packages built:

cairomm_1_16
pangomm_2_48


ERROR:pytest_commander.api:Exception on /api/v1/result-tree [GET]

Hi,
i get the following error when starting pytest_commander. I work on a conda env.
ERROR:pytest_commander.api:Exception on /api/v1/result-tree [GET] Traceback (most recent call last):   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\flask\app.py", line 2447, in wsgi_app     response = self.full_dispatch_request()   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\flask\app.py", line 1952, in full_dispatch_request     rv = self.handle_user_exception(e)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\flask\app.py", line 1821, in handle_user_exception     reraise(exc_type, exc_value, tb)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\flask\_compat.py", line 39, in reraise     raise value   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\flask\app.py", line 1950, in full_dispatch_request     rv = self.dispatch_request()   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\flask\app.py", line 1936, in dispatch_request     return self.view_functions[rule.endpoint](**req.view_args)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\pytest_commander\api.py", line 45, in tree     return branch_schema.dump(test_runner.result_tree)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\marshmallow\schema.py", line 547, in dump     result = self._serialize(processed_obj, many=many)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\marshmallow\schema.py", line 515, in _serialize     value = field_obj.serialize(attr_name, obj, accessor=self.get_attribute)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\marshmallow\fields.py", line 310, in serialize     value = self.get_value(obj, attr, accessor=accessor)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\marshmallow\fields.py", line 239, in get_value     return accessor_func(obj, check_key, default)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\marshmallow\schema.py", line 472, in get_attribute     return get_value(obj, attr, default)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\marshmallow\utils.py", line 239, in get_value     return _get_value_for_key(obj, key, default)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\marshmallow\utils.py", line 253, in _get_value_for_key     return getattr(obj, key, default)   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\pytest_commander\result_tree.py", line 187, in short_id     return self.nodeid.short_id   File "c:\users\lto\miniconda3\envs\selide\lib\site-packages\pytest_commander\nodeid.py", line 125, in short_id     return self.fragments[-1].val IndexError: list index out of range

Hi, thanks for raising this issue. Could you please give more details on the tests you are running and exactly which args you are passing to pytest_commander on the command line?

Hi, thanks for raising this issue. Could you please share some more details about the tests you are running and what args you are passing to pytest_commander on the command line?

I have tried pycharm, cmd and anaconda prompt with following args in my working directory:
pytest_commander ./
I have a python file in this directory with tests as functions inside.

Can you please try updating to the latest 2.6.0 release and let me know if you still have the issue?
If so, are you by any chance running from the root directory /?

Hi, after some refactoring work on my script, pytest_commander now starts without an error.
But i cant see any name of my test in the browser ui. when i press the play button though, the test starts normally.
Please refer to the screenshot.
https://pasteboard.co/K6Aa4N8.jpg

That issue should be resolved in the latest release, please can you run pip install -U pytest-commander and try again?

Does work now, thank you very much! Can be closed.


[APIM 4.0.0] Page not found for links in Reference

Description:

Page not found error occurs when navigating to links in the Reference.

As an example:
Reference > Customize product > extending WSO2 API manager > extending-workflows > configuring-workflows-for-tenants

Fixed with https://github.com/wso2/docs-apim/pull/2762


question re homebridge.APIs

Hello Guys,
apologies for flooding into the issues thread, I've got few dev questions and would greatly appreciate if you could answer. Thanks!

I am updating AccessoryInformation when it was already deployed to HomeBridge and reflected in the Home.app, if I do: Accessory.getService(Service.AccessoryInformation).setCharacteristic(Characteristic.Manufacturer, "smth") and then that.api.updatePlatformAccessories I don't get changes reflected in Home until after HomeBridge restarts. Any hints how to force update Hope.app data without restart?

when reachability is false, does it mean that all Homekit requests for getValue are taken from the accessories cache, and whenever I update it to true, I have to support Get/Set events for instantaneous values read/set form the Homekit apps?

Unlikely possible, but will give it a go, all new homebridge accessories are getting into the room where homebridge itself was assigned, could I have a way to push new accessories to the designated room?

Thanks again!

Question 1 - I don't know the answer to this, but I don't believe this is a valid use case.  Please check the Apple HAP-Spec
Question 2 - Reachability was deprecated a while back by Apple.  Take a look at this pull request, where updateValue was updated to create 'not responding' status.  https://github.com/KhaosT/HAP-NodeJS/pull/556
Question 3 - Nope, all new accessories go to the default room.
If you want to chat more, I'm frequently available on Slack


[PDAL] Update to v2.2.0

Update PDAL to v2.2.0 (from v1.7.2)
Fixed issues mentioned here:
#14055

@GeorgeR Thanks for your PR. Could you resolve conflicts?

Pinging @GeorgeR for response.

@GeorgeR Now pdal build failed on x64-windows-static with the following error:
-- Configuring done
CMake Error: install(EXPORT "PDALTargets" ...) includes target "pdalcpp" which requires target "pdal_arbiter" that is not in any export set.
CMake Error: install(EXPORT "PDALTargets" ...) includes target "pdalcpp" which requires target "pdal_kazhdan" that is not in any export set.
-- Generating done
CMake Warning:
  Manually-specified variables were not used by the project:
    CMAKE_INSTALL_BINDIR
CMake Generate step failed.  Build files cannot be regenerated correctly.

Could you continue to fix this?

Pinging @GeorgeR for response.

@LilyWangL no problem, will continue to work on this shortly (had to work over weekend so didn't get much time)

@GeorgeR Now pdal build failed on x64-windows-static with the following error:
-- Configuring done
CMake Error: install(EXPORT "PDALTargets" ...) includes target "pdalcpp" which requires target "pdal_arbiter" that is not in any export set.
CMake Error: install(EXPORT "PDALTargets" ...) includes target "pdalcpp" which requires target "pdal_kazhdan" that is not in any export set.
-- Generating done
CMake Warning:
  Manually-specified variables were not used by the project:
    CMAKE_INSTALL_BINDIR
CMake Generate step failed.  Build files cannot be regenerated correctly.

Could you continue to fix this?

I'm really at a loss as to how to fix this.
I almost got PDAL to build on it's own as static (but got some CURL errors) and noticed that the output PDALTargets.cmake file didn't reference the two offending libraries (arbiter or kazhdan) in INTERFACE_LINK_LIBRARIES for pdalcpp, but when built from vcpkg it did.
I reverted some of the changes in the vcpkg patch, for example the export of arbiter and kazdhan, but this didn't change the fact that both are expected in INTERFACE_LINK_LIBRARIES in the form of:
$<LINK_ONLY:pdal_kazhdan>

I also enabled the PDAL_USE_STATIC_RUNTIME flag in the port file:
if(VCPKG_LIBRARY_LINKAGE STREQUAL "dynamic")
  set(VCPKG_BUILD_SHARED_LIBS ON)
  set(VCPKG_BUILD_STATIC_LIBS OFF)
else()
  set(VCPKG_BUILD_SHARED_LIBS OFF)
  set(VCPKG_BUILD_STATIC_LIBS ON)
  set(PDAL_USE_STATIC_RUNTIME TRUE)
endif()

So i'm still getting the same errors as you've posted, but i'm unsure why vcpkg is creating a different PDALTargets.cmake than the what i've got via regular cmake.

@NancyLi1013 are you looking into this? could you make sure that this gets worked through?

Yes, I arm looking into this. Thanks for your kindly reminder @strega-nil .
@GeorgeR
Sorry for the delay. Could you please resolve the conflicts?

I tried your changes, but get the following error:
Building package pdal[core]:x64-linux...
Could not locate cached archive: /home/fgawrilo/.cache/vcpkg/archives/da/da6419805d08e65d688e0e4cba52c509042f6d69.zip
-- Using cached /media/data/Code/repos/3rd/vcpkg/downloads/PDAL-PDAL-2.2.0.tar.gz
-- Cleaning sources at /media/data/Code/repos/3rd/vcpkg/buildtrees/pdal/src/2.2.0-1c3e5bb51e.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /media/data/Code/repos/3rd/vcpkg/downloads/PDAL-PDAL-2.2.0.tar.gz
-- Applying patch reimplement-patch-172-in-220.patch
-- Using source at /media/data/Code/repos/3rd/vcpkg/buildtrees/pdal/src/2.2.0-1c3e5bb51e.clean
-- Configuring x64-linux-dbg
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:108 (message):
Command failed: /media/data/Code/repos/3rd/vcpkg/downloads/tools/cmake-3.18.4-linux/cmake-3.18.4-Linux-x86_64/bin/cmake /media/data/Code/repos/3rd/vcpkg/buildtrees/pdal/src/2.2.0-1c3e5bb51e.clean -DPDAL_BUILD_STATIC:BOOL=ON -DWITH_TESTS:BOOL=OFF -DWITH_COMPLETION:BOOL=OFF -DCMAKE_MAKE_PROGRAM=/home/fgawrilo/repos/3rd/vcpkg/downloads/tools/ninja-1.10.1-linux/ninja -DCMAKE_SYSTEM_NAME=Linux -DBUILD_SHARED_LIBS=OFF -DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=/home/fgawrilo/repos/3rd/vcpkg/scripts/toolchains/linux.cmake -DVCPKG_TARGET_TRIPLET=x64-linux -DVCPKG_SET_CHARSET_FLAG=ON -DVCPKG_PLATFORM_TOOLSET=external -DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON -DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON -DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON -DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE -DCMAKE_VERBOSE_MAKEFILE=ON -DVCPKG_APPLOCAL_DEPS=OFF -DCMAKE_TOOLCHAIN_FILE=/home/fgawrilo/repos/3rd/vcpkg/scripts/buildsystems/vcpkg.cmake -DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON -DVCPKG_CXX_FLAGS= -DVCPKG_CXX_FLAGS_RELEASE= -DVCPKG_CXX_FLAGS_DEBUG= -DVCPKG_C_FLAGS= -DVCPKG_C_FLAGS_RELEASE= -DVCPKG_C_FLAGS_DEBUG= -DVCPKG_CRT_LINKAGE=dynamic -DVCPKG_LINKER_FLAGS= -DVCPKG_LINKER_FLAGS_RELEASE= -DVCPKG_LINKER_FLAGS_DEBUG= -DVCPKG_TARGET_ARCHITECTURE=x64 -DCMAKE_INSTALL_LIBDIR:STRING=lib -DCMAKE_INSTALL_BINDIR:STRING=bin -D_VCPKG_ROOT_DIR=/media/data/Code/repos/3rd/vcpkg -D_VCPKG_INSTALLED_DIR=/media/data/Code/repos/3rd/vcpkg/installed -DVCPKG_MANIFEST_INSTALL=OFF -G Ninja -DCMAKE_BUILD_TYPE=Debug -DCMAKE_INSTALL_PREFIX=/media/data/Code/repos/3rd/vcpkg/packages/pdal_x64-linux/debug
Working Directory: /media/data/Code/repos/3rd/vcpkg/buildtrees/pdal/x64-linux-dbg
Error code: 1
See logs for more information:
/media/data/Code/repos/3rd/vcpkg/buildtrees/pdal/config-x64-linux-dbg-out.log
/media/data/Code/repos/3rd/vcpkg/buildtrees/pdal/config-x64-linux-dbg-err.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:328 (vcpkg_execute_required_process)
ports/pdal/portfile.cmake:25 (vcpkg_configure_cmake)
scripts/ports.cmake:136 (include)
Error: Building package pdal:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with ./vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: pdal:x64-linux
Vcpkg version: 2020.11.12-unknownhash
Additionally, attach any relevant sections from the log files above.
config-x64-linux-dbg-err.log
config-x64-linux-dbg-out.log

@GeorgeR
The current changes in this PR cannot work fine. Please update vcpkg to the latest version to rebuild this port on vcpkg master branch.

PDAL failed on the x64-windows-static with the following errors:
CMake Error: install(EXPORT "PDALTargets" ...) includes target "pdalcpp" which requires target "pdal_arbiter" that is not in any export set.
CMake Error: install(EXPORT "PDALTargets" ...) includes target "pdalcpp" which requires target "pdal_kazhdan" that is not in any export set.

I started on this fresh yesterday, using a commit which fixes VS2019 build errors:
https://github.com/PDAL/PDAL/commit/0f319a51cb3cb67b10328176ebbefcbe6ca4fc9b
I will get it building outside of vcpkg (but using the vcpkg toolchain) first, then patch in the relevant changes. The issue I'm facing at the moment are differences between how vcpkg is configuring the project vs what I'm doing via cmake-ui.

I started on this fresh yesterday, using a commit which fixes VS2019 build errors:
https://github.com/PDAL/PDAL/commit/0f319a51cb3cb67b10328176ebbefcbe6ca4fc9b
I will get it building outside of vcpkg (but using the vcpkg toolchain) first, then patch in the relevant changes. The issue I'm facing at the moment are differences between how vcpkg is configuring the project vs what I'm doing via cmake-ui.

Hi @GeorgeR
What's progress on this PR?

Converting to draft for now until this PR can work fine. Please ping me if it is ready for review.

Closing this PR since it seems that no progress is being made. Please reopen if work is still being done.


Firebase.Core.App.Configure() Second Call don't work

Hi
My Problem is  when i first start my App everything works fine but when i start the app the second time i receive no notification.  So i delete the App reinstall it and on the second run i comment out the Line  Firebase.Core.App.Configure(); and all works fine. So my Workaround is to check if i have a token then i dont  call Configure() if not i call the method but when i look in all examples this is not the standard way so is it a Bug? Or maybe a misconfiguration in Firebase or in the Apple Settings . What does the  methode do exactly? Can you help me to Understand the method and what i am doing wrong.
Source
` [Export("messaging:didReceiveRegistrationToken:")]
public void DidReceiveRegistrationToken(Messaging messaging, string fcmToken)
{
    }

    [Export("messaging:didRegisterForRemoteNotificationsWithDeviceToken")]
    public void didRegisterForRemoteNotificationsWithDeviceToken(UIApplication application)
    {

    }
    [Export("messaging:didReceiveMessage:")]
    public void DidReceiveMessage(Messaging messaging, RemoteMessage remoteMessage)
    {

    }

    public override void ReceivedRemoteNotification(UIApplication application, NSDictionary userInfo)
    {

    }

    public override void DidReceiveRemoteNotification(UIApplication application, NSDictionary userInfo, Action<UIBackgroundFetchResult> completionHandler)
    {

    }

    public override bool FinishedLaunching(UIApplication app, NSDictionary options)
    {

        Firebase.Core.App.Configure();

        #region Register your app for remote notifications.
        if (UIDevice.CurrentDevice.CheckSystemVersion(10, 0))
        {

            // For iOS 10 display notification (sent via APNS)
            //UNUserNotificationCenter.Current.Delegate = this;

            var authOptions = UNAuthorizationOptions.Alert | UNAuthorizationOptions.Badge | UNAuthorizationOptions.Sound;
            UNUserNotificationCenter.Current.RequestAuthorization(authOptions, (granted, error) =>
            {
                Console.WriteLine(granted);
            });
        }
        else
        {
            // iOS 9 or before
            var allNotificationTypes = UIUserNotificationType.Alert | UIUserNotificationType.Badge | UIUserNotificationType.Sound;
            var settings = UIUserNotificationSettings.GetSettingsForTypes(allNotificationTypes, null);
            UIApplication.SharedApplication.RegisterUserNotificationSettings(settings);
        }

        UIApplication.SharedApplication.RegisterForRemoteNotifications();
        Messaging.SharedInstance.Delegate = this;
        Messaging.SharedInstance.ShouldEstablishDirectChannel = true;
        #endregion
        global::Xamarin.Forms.Forms.Init();
        LoadApplication(new App());

        return base.FinishedLaunching(app, options);
    }`

https://github.com/firebase/firebase-ios-sdk/issues/2438
Downgrading iOS.Core and iOs.InstnaceId to 5.1.3 and 3.2.0 solved the same issue for me

Is this still an issue?

We have not received the requested information. If you are still experiencing this issue, please, open a new issue related to this. Thanks!


ci: skip compiling Java with Saxon 10

This pull request skips Java compile test on Saxon 10, because of #852.
a08d60a49c13925d497b2044163ef09becc0e310 is a temporary commit to verify this change.

Inspecting AppVeyor, Azure Pipelines and Travis CI logs, I verified that Compile Java was performed on Java 8 + Saxon 9.x and not on the other cases.


Azure SQL Server delete behavior in transaction

Problem
Say a transaction with only one delete operation is executed on SQL server:
delete k1

If k1 is not present in the DB already, this transaction will fail with the error message: delete affected only 0 rows, expected 1.
Is this an expected behavior?
It is not reproducible with other state stores. If the item is not available, they do not error out.
Proposal
If the above behavior is unexpected, I see two easy ways to fix this:

We can use individual deletes instead of bulk delete in our transactions (performance will take a hit), OR
Ignore this type of error in executeMulti during bulk delete invocation (should be better)

Background
Transactions in Azure SQL Server use the bulk delete function to execute all delete operations at once.
func (s *SQLServer) executeMulti(sets []state.SetRequest, deletes []state.DeleteRequest) error {
        // ...
	if len(deletes) > 0 {
		err = s.executeBulkDelete(tx, deletes)
		if err != nil {
			tx.Rollback()

			return err
		}
	}
        // ...
}

This is the important part from the bulk delete snippet:
func (s *SQLServer) executeBulkDelete(db dbExecutor, req []state.DeleteRequest) error {
	// ...
	rows, err := res.RowsAffected()
	if err != nil {
		return err
	}

	if int(rows) != len(req) {
		err = fmt.Errorf("delete affected only %d rows, expected %d", rows, len(req))

		return err
	}
        // ...
}

Added a PR for fix with 2nd proposal.

/assign

Closed by #1516


Connecting redstone through SU Areas crashes

"Never let DSM test ur mod, he'll crash it multiple times and tell you to get good" - Sun Tzu the Art of War -
crash-2021-03-08_20.25.21-client.txt
latest.log

"That Sun Tzu guy is a ducking lying piece of underwater trash, this has been fixed, definitely let DSM test your mod so he can fix things and as well as telling you to get good." - Sun Tzu the Art of War -
Fixed.


adds KMS region tags

Adds region tags for all relevant samples/functions.
However, the quickstart for datastore looks a little bit different (just the name of the function, and the fact it's in a dedicated file). Can we get this refactored for Java? I can try to do it but it might take a minute.

I get the following error when I try to run the EncryptFileCommand:
# java -cp target/kms-samples-1.0.0-jar-with-dependencies.jar com.example.CryptFileCommands
Error: Main method not found in class com.example.CryptFileCommands, please define the main method as:
   public static void main(String[] args)

nm, I was able to figure it out and fix the corresponding index error! PTAL.

I got ambitious and added Quickstart.java. I compiled and it worked. I tried to match it to the other quickstarts, but LMK if I missed something!

@lesv no because the README is accurate - I updated the code to match :D. I can update it if you'd like, however.
Follow-up question, are the Quickstart instructions accurate? I was able to run without modifying pom.xml, so can this and/or the gradle instructions be removed?

QS Instructions doesn't even come close to matching what's in the pom.xml with the exclude of guava and bringing in a good version.  We should fix.

I originally had it so that the Quickstart was in a separate file, but then when I went to test it, it seemed weird to me that you had a quickstart to list keyrings, without any ability to create keyrings. Which is why I put it in with the rest in Snippets. Seemed to me that it can be included in the docs from the Snippets just as easily as from the Quickstart, and it didn't seem to make that much sense as a standalone file... no?

It also seems weird to me that we're cluttering up our source code with region tags just to surround a function, but I seem to have lost that battle.
Added a test for Quickstart - LGTM. Thanks for catching that command index error! :)


[Feature][Project] Dev0.1.0 Project management test cases and functional tests

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Problem Description

Project  test cases
Project functional tests

Description
No response
Use case
No response
Solutions
No response
Anything else
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

fix


Distance - Receipt briefly shows placeholder after receipt is generated

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

Version Number: 9-0-56-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to workspace chat
Submit a manual expense (if there is no other expense)
Go offline
Submit a distance expense
Go to transaction thread
Click Distance
Go online
Wait for the receipt in the background to generate completely
Click outside the RHP to dismiss it

Expected Result:
There will be no changes to the distance receipt and report header
Actual Result:
The distance receipt shows placeholder thumbnil and header changes to "0.00 for Pending..." briefly
Workaround:
Unknown
Platforms:

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

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/7aab32db-1702-4a61-9ac5-1eb42c9ccd89
View all open jobs on GitHub

We think that this bug might be related to #wave-collect - Release 1

This isn't breaking enough functionality to be considered a deploy blocker but it should be fixed.

However, most likely coming from https://github.com/Expensify/App/pull/51519 cc: @Gonals @ikevin127 @wildan-m

Indeed, this is not a blocker and we discussed it in the PR (https://github.com/Expensify/App/pull/51519#discussion_r1817955794) and we decided to ship it like this because the fix for it was out of scope for the issue.
Therefore even though this is technically a regression of our PR, we decided not to count it as regression when it comes to compensation.

Ok, marking this as a new issue then

@NikkiWines The issue is likely internal and should be addressed in the backend to resolve discrepancies between transactionBackup and transaction data during online updates. Please refer to this link for an initial investigation into the missing fields. https://github.com/Expensify/App/issues/48630#issuecomment-2438017630.

This falls under #quality since the PR it stems from (https://github.com/Expensify/App/pull/51519#discussion_r1817955794) was also under #quality.
I'll mark as internal for now and raise a discussion in Slack around prioritization.

No updates.

No updates

This issue occurs, because the receipt data isn't added into queuedOnyxUpdates array. After clicking outside of RHP, the page is navigated from modal to distance request details and then openReport is activated to flush queuedOnyxUpdates array to update. So w/o the receipt data former data is updated.

@sakluger Sure. Since I haven't had an access to BE API, I think the internal team would take this issue.


Use hap-nodejs 0.4.0

This bumps the version of HAP to 0.4.0 so that Homebridge has the new IP camera support.
This is a super naive pull and hasn't  been fully tested. There may be other incompatibilities not taken into consideration, but it looks like 0.4.0 was just additions.
/cc @KhaosT
I also only bumped Homebridge's version to 0.3.5 since nothing should actually break here.

I'll merge this first and then make changes later when adding proper support for camera.


Remove unneeded console line

🐞 Bug report
Description
In the watchmode file we have an unneeded console.log This line can be removed as there is no reason to leave it there.

Can you assign it to me?

I would like to work on this

Hi everybody, I would love to assign one of you to the job but somebody ready made a PR #1437

@Jefiozie given that the PR was closed by @SanderElias because the CLA was not signed will you be assigning this issue out. If so could you assign it to me please?

Hi @TomJWilliams somebody already made a pr for this issue. But thank for willing to fix it.
#1447


feat(docker): integrate cuda/no-cuda jobs into single job

https://github.com/softprops/action-gh-release/issues/243


Support Arch Linux / packer-io binary

On Arch Linux (and possibly some other distros??) packer.io is packaged as packer-io not packer, because of a conflict with a pre-existing packer binary.
We should support this, by finding the actual binary, and trying to find packer-io if packer doesn't exist.
I'm working on some code to do this.

You can overwrite the packer command by passing in the bin_path to the packer binary that you want to be called.
I need to document this but you'd use it like:
packer = PackRb::Packer.new(tpl: template, machine_readable: true, bin_path: '/usr/bin/packer-io')
packer.build(debug: true)

But then I'd need the binary detection logic in whatever calls pack_rb... And if I use pack_rb in 5 projects, I'd need to duplicate that logic in all of them...


Fix bottom navbar on iphone

Fix https://www.pivotaltracker.com/story/show/102424602. When the width of the view is below the width of a landscape-mode iPhone 6 Plus, unlock the footer, and vice versa. This works dynamically as well -- if the user resizes the window above and below the iPhone 6 Plus landscape width threshold, the footer will lock and unlock accordingly.

Not ready yet. Nothing to see here.


Remove Web UI from README

It seems that the Web UI was removed, but it's still mentioned in the README. To avoid confusion, I've removed it from there.

Thanks!


x/tools/gopls: use RelativePatterns for file watching if RelativePatternSupport is available

In order to fix problems with Windows paths, we should use RelativePatterns if they are supported:
https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#relativePattern
Because these patterns allow specifying a directory URI, they are a better solution for watching absolute directories, and don't suffer from windows drive casing issues.
See https://github.com/microsoft/vscode/issues/104387 for more background.

This is definitely necessary: I discovered that e.g. absolute patterns do not work in coc.nvim.


Make the default colours be the 'default' colours

Prompts should match the user's terminal unless explicitly styled.
(For instance, the default Ubuntu terminal uses a maroon background instead of black.)

Many thanks!


Disable TestURLSession pending SR-3464.

@swift-ci please test linux

This is a very large set of tests to disable. Can we be more specific instead?

I only know that the first test fails; the others didn't run in CI after that. Narrowing this would take either a few hours watching CI or a local reproducer, and I didn't have either.


(#3) added button module first draft

First draft of the button module.
@gazhands needs to have a look and together we can figure out what we do with the tertiary hover/focus state. Right now it wiggles :)
I try to reframe from using box-shadow or outlines in this case as the button still needs to work in IE8 where those features are not supported. border it must be.
@klepas go ahead and have a look at my html. I’m sure the button needs some special markup that I can’t think of right now. :)
Also: yay, description.

staging: https://124-78596482-gh.circle-artifacts.com/0/tmp/circle-artifacts.0Zh2pYm/packages/buttons/tests/site/index.html


BIGNUM error

After make process, I get this error (several times)
bignum.h:57:24: error: invalid use of incomplete type 'BIGNUM {aka struct bignum_st}'  class CBigNum : public BIGNUM
any idea what could be wrong, I am using noobs 2.4.4 raspbian installation.

As per the guide it's required to use Raspbian Jessie, I'm assuming NOOBS will have installed Raspbian Stretch.  Please check the version and confirm, you can do that with

cat /etc/os-release

If it is Stretch, I did some research and it may be the open SSL libraries, try this

sudo apt-get install libssl-dev

make -f makefile.unix clean

Then try make again

I was wrong about installing libssl-dev, you will need to downgrade it (back to the Jessie version)
Remove the current version of libssl-dev ( 1.1.0f-3 )

sudo apt-get remove libssl-dev

Set your repository list to point to "jessie" instead of "stretch", save and exit.

sudo nano /etc/apt/sources.list

Then do sudo apt-get update to download the packages for jessie
Then do

sudo apt-get install libssl-dev

package, it should be version 1.0.1t-1

make -f makefile.unix clean

Do the make process again and finish the installation.
When complete, type

sudo apt-mark hold libssl-dev

to make the package to not upgrade in the future
Switch back your sources, by changing 'jessie' back to 'stretch' in number 2
Do a

sudo apt-get update

and

sudo apt-get upgrade

and make sure it doesn't try and install libssl-dev (it will say it has been kept back)

Or install Jessie on your SD card, or wait a few weeks for the latest version of SolarCoin core that won't have these issues, and you will probably need to upgrade to anyway.

this part was fixed, but then another error came up:
net.cpp:64:1: error: reference to 'array' is ambiguous  array<int, THREAD_MAX> vnThreadsRunning;  ^~~~~ In file included from net.h:9:0,                  from main.h:10,                  from db.h:8,                  from net.cpp:7: /usr/include/boost/array.hpp:60:11: note: candidates are: template<class T, unsigned int N> class boost::array      class array {            ^~~~~ In file included from /usr/include/c++/6/tuple:39:0,                  from /usr/include/c++/6/bits/stl_map.h:63,                  from /usr/include/c++/6/map:61,                  from serialize.h:11,                  from bignum.h:9,                  from main.h:8,                  from db.h:8,                  from net.cpp:7: /usr/include/c++/6/array:90:12: note:                 template<class _Tp, unsigned int _Nm> struct std::array      struct array             ^~~~~ makefile.unix:172: recipe for target 'obj/net.o' failed make: *** [obj/net.o] Error 1
I will follow your advise and install jessie, do you have a binary to share?

Its the first bit in the guide '1 - THIS GUIDE ONLY WORKS ON RASPBIAN JESSIE, LITE DOWNLOADS AVAILABLE FROM https://downloads.raspberrypi.org/raspbian_lite/images/ make sure and use the version dated 2017-07-05'

Thanks!


DatePicker window should jump to first available date when using minDate()

Package
filament/forms
Package Version
v2.12.8
Laravel Version
v9.12.2
Livewire Version
v2.10.5
PHP Version
8.1.5
Bug description
DatePicker window does not jump to first available date when using the minDate() function
Steps to reproduce
Create two DatePicker fields, set the first reactive and the second to minDate(fn(callable $get) => $get('first_date')). Assign a date a few months in the future in the first field, then open the second field. The dates will be greyed out, but the will still remain on the current month, rather than jumping to the month with the first available selectable date.
Relevant log output
No response

Fixed by #2489