Gene/disease conflations in KG2C-6-3

I have noticed a number of KG2C-6-3 disease/gene conflations:
match (n:biolink:Disease{id:"MONDO:0042979"}) return n Gene CACNA1S conflated with the disease Hypokalemic periodic paralysis.
match (n:biolink:Disease{id:"MONDO:0011309"}) return n Gene TSHR conflated with disease familial gestational hyperthyroidism.
match (n:biolink:Disease{id:"MONDO:0012276"}) return n Gene KCNMA1 conflated with disease eneralized epilepsy - paroxysmal dyskinesia
Some/all of these look to be due to mutations in the gene causing the disease.

Okay I think we've made the problem different, but not necessarily better?
on kg2.6.4
MATCH (d:`biolink:Disease`)
MATCH (g:`biolink:Gene`)
WHERE d.name = g.name
RETURN d.id as disease,g.id as gene, d.name as name;

returns 3616 records (which is more than before). Here are some examples, chosen randomly to try to get a spread of the different provided by combos:

disease
gene
name

"UMLS:C1838854"
"HGNC:7497"
"MT-TS1"

"UMLS:C3809434"
"OMIM:609799"
"NEK8"

"UMLS:C3501265"
"OMIM:600970"
"MYO6"

"UMLS:C3715165"
"OMIM:600958"
"MYBPC3"

"UMLS:C3809346"
"OMIM:600958"
"MYBPC3"

"UMLS:C3810194"
"OMIM:609369"
"ATL3"

"UMLS:C3805411"
"OMIM:193060"
"VIM"

"UMLS:C1859252"
"OMIM:614123"
"TMCO1"

"UMLS:C1856603"
"ENSEMBL:ENSG00000001084"
"GCLC"

"UMLS:C1861786"
"ENSEMBL:ENSG00000001631"
"KRIT1"

"UMLS:C1861785"
"ENSEMBL:ENSG00000001631"
"KRIT1"

"UMLS:C1366911"
"ENSEMBL:ENSG00000001631"
"KRIT1"

"UMLS:C3280729"
"ENSEMBL:ENSG00000003137"
"CYP26B1"

Here is the provided by breakdown:
MATCH (d:`biolink:Disease`)
MATCH (g:`biolink:Gene`)
WHERE d.name = g.name
RETURN distinct(d.provided_by+" \\ "+ g.provided_by), count(*) order by count(*) DESC;

(d.provided_by+" \ "+ g.provided_by)
count(*)

"identifiers_org_registry:umls \ identifiers_org_registry:ensembl"
2015

"identifiers_org_registry:umls \ umls_source:OMIM"
1565

"OBO:mondo.owl \ ORPHANET:"
9

"OBO:doid.owl \ ORPHANET:"
9

"EFO:efo.owl \ ORPHANET:"
8

"identifiers_org_registry:umls \ umls_source:HGNC"
7

"umls_source:MSH \ ORPHANET:"
3

I think the problem might be with UMLS rather than with the data sources:
MATCH (d:`biolink:Disease`)
MATCH (g:`biolink:Gene`)
WHERE d.name = g.name
AND NOT (d)-[:`biolink:close_match`]-(g)
RETURN distinct(d.provided_by+" \\ "+ g.provided_by), count(*) order by count(*) DESC;

(d.provided_by+" \ "+ g.provided_by)
count(*)

"identifiers_org_registry:umls \ identifiers_org_registry:ensembl"
2015

"OBO:mondo.owl \ ORPHANET:"
9

"OBO:doid.owl \ ORPHANET:"
9

"EFO:efo.owl \ ORPHANET:"
8

"identifiers_org_registry:umls \ umls_source:HGNC"
7

"umls_source:MSH \ ORPHANET:"
3

You'll noticed all of the UMLS-OMIM equivalencies are wrong. This likely means that the UMLS CUI nodes are receiving the wrong category, likely from a different source.

You'll noticed all of the UMLS-OMIM equivalencies are gone. This likely means that the UMLS CUI nodes are receiving the wrong category, likely from a different source.

Ok, that is not the case. (For example, C3280729 is a disease not a gene, see https://www.ncbi.nlm.nih.gov/medgen/C3280729) I think the CUI nodes are getting the right category and (perhaps) we should ignore has_cui relationships from OMIM.
See: match (n:biolink:Disease {id: 'UMLS:C3280729'})-[e {relation: 'oboFormat:xref'}]-(m) return n, e, m as a reference. Both the genes and phenotypes from OMIM contribute to their respective CUI nodes. This is problematic because they have different names. Per
https://github.com/RTXteam/RTX-KG2/blob/e6275ee3413b19edda1ee42cf7d7bb14d3beaa22/kg2_util.py#L538-L546
and
https://github.com/RTXteam/RTX-KG2/blob/e6275ee3413b19edda1ee42cf7d7bb14d3beaa22/kg2_util.py#L496-L505
the category and name for a node have different criteria for which one is picked in a merge. This is likely leading to an odd merge, but I haven't confirmed that.

This is substantially better in KG2.6.5. I added an extra column in the below table (to the query) to note if this synonymization was already occurring. There were only 29 results to the query below:
MATCH (d:`biolink:Disease`)
MATCH (g:`biolink:Gene`)
WHERE d.name = g.name
RETURN d.id as disease,g.id as gene, d.name as name;

disease
gene
name
already occurring?

"MESH:C537910"
"ORPHANET:118982"
"Bardet-Biedl syndrome 2"
yes

"MESH:C537912"
"ORPHANET:118985"
"Bardet-Biedl syndrome 4"
yes

"MESH:C537909"
"ORPHANET:118975"
"Bardet-Biedl syndrome 1"
yes

"ORPHANET:2473"
"ORPHANET:123245"
"McKusick-Kaufman syndrome"
yes

"EFO:0009023"
"ORPHANET:137646"
"Bardet-Biedl syndrome 12"
yes

"EFO:0009024"
"ORPHANET:118985"
"Bardet-Biedl syndrome 4"
yes

"EFO:0009021"
"ORPHANET:118975"
"Bardet-Biedl syndrome 1"
yes

"EFO:0009022"
"ORPHANET:118978"
"Bardet-Biedl syndrome 10"
yes

"EFO:0009027"
"ORPHANET:118994"
"Bardet-Biedl syndrome 9"
yes

"EFO:0009025"
"ORPHANET:118987"
"Bardet-Biedl syndrome 5"
yes

"EFO:0009026"
"ORPHANET:118990"
"Bardet-Biedl syndrome 7"
yes

"MONDO:0014440"
"ORPHANET:137646"
"Bardet-Biedl syndrome 12"
yes

"MONDO:0014435"
"ORPHANET:118990"
"Bardet-Biedl syndrome 7"
yes

"MONDO:0014434"
"ORPHANET:118987"
"Bardet-Biedl syndrome 5"
yes

"MONDO:0014433"
"ORPHANET:118985"
"Bardet-Biedl syndrome 4"
yes

"MONDO:0014432"
"ORPHANET:118982"
"Bardet-Biedl syndrome 2"
yes

"MONDO:0014438"
"ORPHANET:118978"
"Bardet-Biedl syndrome 10"
yes

"MONDO:0014437"
"ORPHANET:118994"
"Bardet-Biedl syndrome 9"
yes

"MONDO:0009367"
"ORPHANET:123245"
"McKusick-Kaufman syndrome"
yes

"MONDO:0008854"
"ORPHANET:118975"
"Bardet-Biedl syndrome 1"
yes

"DOID:0110124"
"ORPHANET:118982"
"Bardet-Biedl syndrome 2"
yes

"DOID:0110123"
"ORPHANET:118975"
"Bardet-Biedl syndrome 1"
yes

"DOID:0110129"
"ORPHANET:118990"
"Bardet-Biedl syndrome 7"
yes

"DOID:0110126"
"ORPHANET:118985"
"Bardet-Biedl syndrome 4"
yes

"DOID:0110127"
"ORPHANET:118987"
"Bardet-Biedl syndrome 5"
yes

"DOID:0110131"
"ORPHANET:118994"
"Bardet-Biedl syndrome 9"
yes

"DOID:0110132"
"ORPHANET:118978"
"Bardet-Biedl syndrome 10"
yes

"DOID:0110134"
"ORPHANET:137646"
"Bardet-Biedl syndrome 12"
yes

"DOID:0111255"
"ORPHANET:123245"
"McKusick-Kaufman syndrome"
yes

You will see that all of these gene/disease conflations present in KG2.6.5 were already present in KG2.5.2. Thus, it seems unnecessary to not roll out KG2.6.5 based on these 29 cases. I'm going to close out this issue for now.


i dont know what is the problem

when I ran it shows
C:\Users\vcvis\Desktop\pytorch-0.4-yolov3-master>python train.py -d cfg/voc.data -c cfg/yolo-voc.cfg -w yolov3.weights
Training for (1935,311)
please help
thank you

@vcvishal  Try to add -r to the command. yolov3.weights is already trained many epochs. Thus, the training is stopped max epoch is reached. -r means resetting epoch and starts from 0.

thank you
it works
again thank you for code


Unable to install using brew on Mac M1. Download error

Summary
I am unable to install microk8s using brew on Mac M1
What Should Happen Instead?
Should install
Reproduction Steps
brew install ubuntu/microk8s/microk8s
(base) ➜  ~ brew install ubuntu/microk8s/microk8s

==> Fetching dependencies for ubuntu/microk8s/microk8s: ca-certificates, sqlite and kubernetes-cli
==> Fetching ca-certificates
==> Downloading https://ghcr.io/v2/homebrew/core/ca-certificates/manifests/2024-03-11
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/c431e0186df2ccc2ea942b34a3c26c2cebebec8e07ad6abdae48447a52c5f506--ca-certificates-2024-03-11.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/ca-certificates/blobs/sha256:cab828953672906e00a8f25db751977b8dc4115f021f8dfe82b644ade03dacdb
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/2c0842d836eaa991deac09d2f933bf5ff5b214d7c5d225c7fe042f6262ca21a9--ca-certificates--2024-03-11.all.bottle.tar.gz
==> Fetching sqlite
==> Downloading https://ghcr.io/v2/homebrew/core/sqlite/manifests/3.45.2
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/52aeccef7dfe87a5156de420a9e1f4b5b62f61b6c2b57633a5e6f04518b50edf--sqlite-3.45.2.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/sqlite/blobs/sha256:40d4422804074ca098583526923f1d3c5a3710f57c5dc5525f493e6a17121c8b
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/21304c055e73f68e51f89f5f487ee0a200876efa684a679a33bb1a9d3a7be766--sqlite--3.45.2.arm64_sonoma.bottle.tar.gz
==> Fetching kubernetes-cli
==> Downloading https://ghcr.io/v2/homebrew/core/kubernetes-cli/manifests/1.29.3
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/f849422f297c8c88f7a790d467820e41426719217c771feeff4196037435a83c--kubernetes-cli-1.29.3.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/kubernetes-cli/blobs/sha256:bec474887cbeb601422ba44ecdc63dbce2e1ace2b21019ac2d544cc7c7ffd912
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/91daa9a8cf96ab50454bc72b7fb68552357d5d4f640cce109bbb26441a7c280f--kubernetes-cli--1.29.3.arm64_sonoma.bottle.tar.gz
==> Fetching ubuntu/microk8s/microk8s
==> Downloading https://github.com/canonical/microk8s/archive/refs/tags/installer-v2.3.4.tar.gz
==> Downloading from https://codeload.github.com/canonical/microk8s/tar.gz/refs/tags/installer-v2.3.4
######                                                                                2.7%curl: (56) Recv failure: Connection reset by peer

Error: microk8s: Failed to download resource "microk8s"
Download failed: https://github.com/canonical/microk8s/archive/refs/tags/installer-v2.3.4.tar.gz

Introspection Report

Can you suggest a fix?

Are you interested in contributing with a fix?

Show us the output of brew install ubuntu/microk8s/microk8s --debug

/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading go
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading kubernetes-cli
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading kubernetes-cli
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading ca-certificates
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading sqlite
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading kubernetes-cli
==> Fetching dependencies for ubuntu/microk8s/microk8s: ca-certificates, sqlite and kubernetes-cli
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading ca-certificates
==> Fetching ca-certificates
==> Downloading https://ghcr.io/v2/homebrew/core/ca-certificates/manifests/2024-03-11
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/c431e0186df2ccc2ea942b34a3c26c2cebebec8e07ad6abdae48447a52c5f506--ca-certificates-2024-03-11.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/ca-certificates/blobs/sha256:cab828953672906e00a8f25db751977b8dc4115f021f8dfe82b644ade03dacdb
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/2c0842d836eaa991deac09d2f933bf5ff5b214d7c5d225c7fe042f6262ca21a9--ca-certificates--2024-03-11.all.bottle.tar.gz
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading sqlite
==> Fetching sqlite
==> Downloading https://ghcr.io/v2/homebrew/core/sqlite/manifests/3.45.2
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/52aeccef7dfe87a5156de420a9e1f4b5b62f61b6c2b57633a5e6f04518b50edf--sqlite-3.45.2.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/sqlite/blobs/sha256:40d4422804074ca098583526923f1d3c5a3710f57c5dc5525f493e6a17121c8b
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/21304c055e73f68e51f89f5f487ee0a200876efa684a679a33bb1a9d3a7be766--sqlite--3.45.2.arm64_sonoma.bottle.tar.gz
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FromAPILoader): loading kubernetes-cli
==> Fetching kubernetes-cli
==> Downloading https://ghcr.io/v2/homebrew/core/kubernetes-cli/manifests/1.29.3
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/f849422f297c8c88f7a790d467820e41426719217c771feeff4196037435a83c--kubernetes-cli-1.29.3.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/kubernetes-cli/blobs/sha256:bec474887cbeb601422ba44ecdc63dbce2e1ace2b21019ac2d544cc7c7ffd912
Already downloaded: /Users/RJoseph3/Library/Caches/Homebrew/downloads/91daa9a8cf96ab50454bc72b7fb68552357d5d4f640cce109bbb26441a7c280f--kubernetes-cli--1.29.3.arm64_sonoma.bottle.tar.gz
==> Fetching ubuntu/microk8s/microk8s
/usr/bin/env /opt/homebrew/Library/Homebrew/shims/shared/curl --disable --cookie /dev/null --globoff --show-error --user-agent Homebrew/4.2.17\ \(Macintosh\;\ arm64\ Mac\ OS\ X\ 14.4\)\ curl/8.4.0 --header Accept-Language:\ en --retry 3 --fail --location --silent --head https://github.com/canonical/microk8s/archive/refs/tags/installer-v2.3.4.tar.gz
==> Downloading https://github.com/canonical/microk8s/archive/refs/tags/installer-v2.3.4.tar.gz
==> Downloading from https://codeload.github.com/canonical/microk8s/tar.gz/refs/tags/installer-v2.3.4
/usr/bin/env /opt/homebrew/Library/Homebrew/shims/shared/curl --disable --cookie /dev/null --globoff --show-error --user-agent Homebrew/4.2.17\ \(Macintosh\;\ arm64\ Mac\ OS\ X\ 14.4\)\ curl/8.4.0 --header Accept-Language:\ en --fail --progress-bar --retry 3 --remote-time --output /Users/RJoseph3/Library/Caches/Homebrew/downloads/281c9d82705923ca3136ff18904081c5aa47bd1c6cd754f6562810a7d68f9af0--microk8s-installer-v2.3.4.tar.gz.incomplete --location https://codeload.github.com/canonical/microk8s/tar.gz/refs/tags/installer-v2.3.4
curl: (56) Recv failure: Connection reset by peer

/usr/bin/env /opt/homebrew/Library/Homebrew/shims/shared/curl --disable --cookie /dev/null --globoff --show-error --user-agent Homebrew/4.2.17\ \(Macintosh\;\ arm64\ Mac\ OS\ X\ 14.4\)\ curl/8.4.0 --header Accept-Language:\ en --retry 3 -V
Error: microk8s: Failed to download resource "microk8s"
Download failed: https://github.com/canonical/microk8s/archive/refs/tags/installer-v2.3.4.tar.gz

Seems like you're using an unstable network connection.
Make sure you're on the latest release of Homebrew:


Update Makefile to handle the content-scope-scripts bundle-trackers

Recently, a new build step was added to the content-scope-scripts
repository to generate a tracker-lookup.json file. Update the Makefile
to generate that when necessary.

Reviewer: @sammacbeth
CC: @jonathanKingston , @jdorweiler
Steps to test this PR:

Test building the extension works as before when a release version of content-scope-scripts dependency is used.
Test that building the extension with a locally linked dev copy of content-scope-scripts works OK, even when content-scope-scripts/build/tracker-lookup.json doesn't already exist.
Test that running touch content-scope-scripts/build/tracker-lookup.json triggers content-scope-scripts and the extension to be rebuilt.

Automated tests:

[ ] Unit tests
[ ] Integration tests

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR Author Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications

Closing out, in favour of https://github.com/duckduckgo/content-scope-scripts/pull/477.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1612 ms

Resolved: Bofrost.at performance has improved in db3efbe after 6 minutes.


Downloaded images not showing in gallery and not open directly

This solved the issue for me. Can you guys test this patch?
After debugging the app the reason that images don't show is that Android does not know about them till you reboot or index is rebuilt. This patch force android's MediaScanner to index the download image.
Notification and gallery working after this.

Replaced by #35


Speed up lambda capture handling

Updated GDScriptLambdaCallable::call and GDScriptLambdaSelfCallable::call to use alloca instead of Vector when using captures, to avoid extra allocation/copy_on_write calls on each lambda function call.
This makes functions like Array.map and Array.filter around 60% faster when using simple lambda functions with captures, compared with gdscript below:
func _ready() -> void:
	time("test_no_capture")
	time("test_map_multiply")
	time("test_filter")
	time("test_self_lambda")

func test_no_capture():
	var a := range(1000)
	for i in 1000:
		a.map(func(n): return n * 2)

func test_map_multiply():
	var a := range(1000)
	var multiplier := 5
	for i in 1000:
		a.map(func(n): return n * multiplier)

func test_filter():
	var a := range(1000)
	var min := 500
	for i in 1000:
		a.filter(func(n): return n >= min)

var max := 750
func test_self_lambda():
	var a := range(1000)
	var min := 500
	for i in 1000:
		a.filter(func(n): return n >= min && n <= max)

func time(test_name : String):
	var start := Time.get_ticks_msec()
	call(test_name)
	var end := Time.get_ticks_msec()
	print("%s: %dms" % [test_name, end - start])

Old:
test_no_capture: 111ms
test_map_multiply: 212ms
test_filter: 215ms
test_self_lambda: 265ms

New:
test_no_capture: 111ms
test_map_multiply: 129ms
test_filter: 131ms
test_self_lambda: 180ms

Using alloca does require additional stack space, which has a small impact on how many recursive calls can be made when p_argcount + captures_amount is large.  The first example below (5 captures + 22 arguments) crashes for me at 567 recursive calls with the new code, and at 574 with old code.  The second example (1 capture + 4 arguments) crashes at 606 recursive calls in both versions.  I think the difference is small enough to be ok, but I can look into avoiding alloca for larger sizes if needed.
func test_recursive_call_large():
	var c1 := 1
	var c2 := 1
	var c3 := 1
	var c4 := 1
	var c5 := 1
	var recursive_func := func(x, f, p1, p2, p3, p4, p5, p6, p7, p8, p9, p10, p11, p12, p13, p14, p15, p16, p17, p18, p19, p20):
		if x > 0:
			print(c1 + c2 + c3 + c4 + c5)
			f.call(x-1, f, p1, p2, p3, p4, p5, p6, p7, p8, p9, p10, p11, p12, p13, p14, p15, p16, p17, p18, p19, p20)

	# 566 max new, 573 old
	recursive_func.call(573, recursive_func, 1, 2, 3, 4, 5, 6, 7, 8, 9, 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 0)

func test_recursive_call_small():
	var c1 := 1
	var recursive_func := func(x, f, p1, p2):
		if x > 0:
			print(c1)
			f.call(x-1, f, p1, p2)

	recursive_func.call(605, recursive_func, 1, 2) # 605 max both

Thanks!


Added bit about calendar item maintance

@markjjo - for your review.

@mahage-msft PR merged. I will ping you in Teams when revised topic has been published.


Update README.md

Swift completed

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

Hi @NikolaiIOSDev, this change to the README appears to remove large parts of the badging and introduction text. We cannot accept this PR as is, is it possible that you meant to update some other piece of documentation?

I'm closing this out. Please reopen it if this was done in error.


Can not login when use Routes.php

October build 419
in my routes.php
Route::get('/test-login', function () {
    $user = \RainLab\User\Models\User::findByEmail('youpp@126.com');
    \RainLab\User\Facades\Auth::login($user);
    return \Illuminate\Support\Facades\Redirect::to('/');
});

it not work.
but used in  page or component, it work well
title = 'test'
url = '/test'
layout= 'default'
==
function onStart(){
$user = \RainLab\User\Models\User::findByEmail('youpp@126.com');
\RainLab\User\Facades\Auth::login($user);
}
==

<div class="container my-5">
    Lorem ipsum dolor sit amet, consectetur adipisicing elit. Aliquid culpa dignissimos molestias nobis, possimus praesentium quae quam voluptatum?  Because this is something that affects Laravel 5.5 (i.e. Build 420+). There is a note in the Release Note for Build 420 about it, essentially it boils down to the fact that Laravel stopped session handling all routes and you now have to add the web middleware explicitly to any route that requires session handling.

@LukeTowers ok

if set edgeUpdates' => true of config/cms.php, then run php artisan october:util set build, it will show show newest build version of build 424

Yes, that's an artifact of how the set build command works (basically just polls the gateway server to get the latest build number)


refactor: remove useless DenomMetadata key function

Make DenomMetadata substore only with DenomMetadataPrefix

Description

closes: #714
Motivation and context

How has this been tested?

Screenshots (if appropriate):
Checklist:

[x] I followed the contributing guidelines and code of conduct.
[ ] I have added a relevant changelog to CHANGELOG.md
[ ] I have added tests to cover my changes.
[ ] I have updated the documentation accordingly.
[ ] I have updated API documentation client/docs/swagger-ui/swagger.yaml

Changes in https://github.com/cosmos/cosmos-sdk/pull/9890/files x/bank/migrations/v043/keys.go does not reflected here. I'm not sure about this, is it okay not reflect this?

yeah, it's legacy migration. So we don't need it


Latest version of bioepic on anaconda is 0.0.6

Installed bioepic through bioconda and latest version is 0.0.6.

Thanks.
I have made a PR, but is not updated for some reason. Will ask about being added to the core so I can make PRs myself.

Has there been any update on this?

I am busy with other stuff unfortunately :)
Can you pip install bioepic? That should give you the latest version.

@c-guzman I uploaded the latest bioepic to bioconda. Enjoy.


Update API to better match Promise spec and add the 'status' prop (closes #35)

Breaking change: Async.Pending was renamed to Async.Waiting
Added the status prop, which can be one of initial, pending, fulfilled or rejected
Added isInitial, isPending, isFulfilled (with alias isResolved), isRejected and isSettled boolean props. isLoading is now an alias for isPending.
Added separate TypeScript types for each status, to make various props non-optional.

The pending and fulfilled statuses were chosen over loading and resolved because they better match the terminology in the Promise specification. The Pending helper component was renamed to Initial accordingly, causing a breaking change.

This branch is available as a pre-release on npm:
npm install react-async@next

or:
npm install<EMAIL_ADDRESS>

@ghengeveld Hello, before you create release 6.x, I wondered if you have considered simplifying some of the properties?
For example, I think it would be nice to just have a single promise property instead of promise and promiseFn. The updated promise property could be a function that returns a Promise or it could be a promise.
Maybe do something similar for watch and watchFn or for deferFn.

Thanks for the suggestion. I've considered this when I introduced watchFn. The reason it's a separate property is that there could also be a legitimate need to watch a function reference itself (rather than invoke it). As for promiseFn, I prefer to mirror deferFn. deferFn itself can't accept a Promise instance as that defeats the purpose, so renaming it to defer would be very confusing.

@all-contributors please add @dhurlburtusa for ideas


Cannot connet to boltz from RTL

I'm deployng lnd, boltz and RTL as docker containers, and cannot get RTL to connect to the bolt rest.
Seems that despite setting the correct host name, the tls certificate of the boltz instance would not generate to the host.
I can access boltz only if I set the container to a fixed IP, but I'd like to avoid that.
I'm using the container's names to access each other, and it works to access "lnd" container from both RTL and boltz.
However, I keep getting this error when trying to connect from RTL:
authentication handshake failed: tls: failed to verify certificate: x509: certificate is valid for 8a3de803d473, localhost, unix, unixpacket, bufconn, not boltz
In lnd, I can easily set as many "tlsextradomain" parameters in lnd.conf to generate new tls certificates.
Didn't find any documentation to generate the certificate in boltz, can it be done?

I struggled many days with this, and only 1 hour after opening this issue I figured a workaround.
boltzd will consider the hostname in the domains to generate the certificate, so I just changed the hostname by passing the directive "--hostname=boltz" to docker.


Sync to upstream after QA has passed successfully

I checked and can confirm that the role is ansible-lint clean also with 5.3.2.


Include dependencies for Github Action concerning report generation

Description of changes
This PR intends to solve the issues in context of the groovydoc report generation:

Add the Maven Site plugin and maven-project-info-reports-plugin to the parent module pom.xml to ensure that the build runs in Github Actions.

Enable the correct groovydoc generation during the site generation by adding the site execution setting to GMavenPlus plugin in the parent module pom.xml .

Allow correct assignment of the generated groovydoc to the individual child module(qOffer-domain & qOffer Infrastructure) by having the groovydocs generated seperately for each module.
This is achieved by introducing the gmavenplus plugin to the child module pom.xml.

Additionally the GroovydocOutput property has to set seperately dependent on the project.build directory for each child module

Finally adding the groovydoc-maven-plugin to the reporting section of the pom.xml of each child modules to enable the linkage from the project summary page to the generated gapidocs.

Additional Information
As of now since GMavenPlus doesn't allow for an aggregate report generation during the site generation,
The final site has to be generated with the mvn site site:stage command and can be seen in the index.html
file of the target/staging/ folder of the parent module.
You can check out a video of the final docs below:
https://user-images.githubusercontent.com/29627977/105975455-43433580-608f-11eb-8910-d973a6c4e124.mov

I updated the PR to also implement the correct report linkage and generation for each child module.

I updated the PR to also implement the correct report linkage and generation for each child module.


include docs PDF in release assets

Currently we only use the Quarto HTML output which we host online. For offline use it would be good to build and include the PDF output of our documentation.
https://quarto.org/docs/output-formats/pdf-basics.html

[ ] get PDF output working, I believe https://github.com/Deltares/iMOD-Documentation already has it
[ ] add it to CI for our nightly and release builds
[ ] update the release process docs

Having this would alleviate #665, giving users a way to read old documentation.

I tried adding a quarto-render-pdf task in pixi.toml, which is the same as quarto-render but with --to pdf.
First it complained that no TeX installation was detected, so I ran quarto install tinytex. After that it the amsmath package failed on some equations on allocations.qmd: "Package amsmath Error: Erroneous nesting of equation structures;"

With #735 and #737 we are closer to PDF output. However, there is a bug that is fixed upstream in Quarto for including remote images, that was discussed in #737.
It's easiest to put this on hold until Quarto 1.4 is released, that is already available as a pre-release, but not in conda-forge / pixi.

With the update to Quarto 1.4 in #1139 this is no longer Paused.

This is currently blocked by https://github.com/conda-forge/quarto-feedstock/issues/44

This is currently blocked by https://github.com/Deltares/Ribasim/pull/1298#issuecomment-2085709572


"You need to include froogaloop for vimeo to work"

When using the experimental Vimeo feature, I get the console message "You need to include froogaloop for vimeo to work".
When I include froogaloop.js (as found here: http://developer.vimeo.com/player/js-api), the message disappear.
Is this necessary? Should MediaElement handle this automatically?

I still see the message in the console with cureent version but all seems to work... Not sure why we see this.

Use the 3.x-dev version. That has been fixed there


Update log_system_information.sh

PR #256, despite changing shell scripts, also introduced a broader system gathering script.
This PR contains only this updated script (as #256 was closed and shell migration was not good in our case)

It would be better to replace log_system_information.sh with a python script, for the same reasons mentioned in #256 .
Currently we have 3 shell scripts left over:
./install_requirements.sh
./package/create_package.sh
./log_system_information.sh

package/create_package.sh can be deleted.
log_system_information.sh should include as much things as possible.
It's actually  very simple to call a subprocess:
Some examples:
https://github.com/luxonis/depthai/blob/main/depthai_profiler.py#L21
https://github.com/luxonis/depthai/blob/main/model_compiler/model_compiler.py#L72
install_requirements.sh should be a python script too (with python shebang and executable permissions) so it can be called as ./install_requirements.py

Sure, changing to draft and will work in this further, thanks @szabi-luxonis!


[supply] Add support for internal test

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Google introduced internal test as a new way to distribute apks to testers.
So I tried to setup supply to upload my apk to intern test track, but supply doesn't allow track which are not in this array.
Maybe it is enough to expand the array and add the right track name?
I couldn't find the correct track name in the google developer api docs , but the docs was last updated on 15 February 2018.
It would be great if someone could find a way to bring internal testing support to supply :)

I could find the correct track name in the google developer api docs ,

Where? I see only exactly what fastlane has in its code :/
Is there an announcement post or something about that change from Google?

@janpio Ups. Sorry for the typo. I couldn't find it. So im not 100% sure whether it is possible.
sources: https://support.google.com/googleplay/android-developer/answer/3131213#internal_test
https://android-developers.googleblog.com/2018/03/our-big-bet-on-mobile-games-at-game.html

https://support.google.com/googleplay/android-developer/answer/3131213

What's the difference between an alpha, beta, and internal test?

Internal test: You should create an internal test if you want to quickly distribute your app for internal testing and quality assurance checks.
Alpha: You should create a closed alpha release to test pre-release versions of your app with a larger set of testers. Once you've tested with a smaller group of employees or trusted users, you can expand your test to an open beta release.

Note: As of March 2018, you can no longer create open alpha tests. You'll retain access to any existing open alpha tests that are already in progress.

Beta: You should create a beta release after you've created an alpha release. Your beta release can include a wider range of users for testing, before your app goes live in production.

+1.
Internal test has several advantages comparing with Alpha, for example, it doesn't take into distribution limits.
Our development team is based in Russia, but app distribution is the US only, so we're actively using internal test releases to avoid Alpha limits and expect fastlane to support that. Let me know if I can help somehow.

PR created for this ^


invalid literal for int() with base 10: '1307=1' and UserWarning: Problem parsing transcript with ID 'transcript/10670'

Hi,
I am getting issues like below when I run TranscriptClean
"Correcting transcripts...
invalid literal for int() with base 10: '1307=1'
invalid literal for int() with base 10: '15=1'
invalid literal for int() with base 10: '1094=1'
invalid literal for int() with base 10: '1094=1'
invalid literal for int() with base 10: '1093=1'
invalid literal for int() with base 10: '1509=1'
invalid literal for int() with base 10: '511=1'
invalid literal for int() with base 10: '91=15588'
invalid literal for int() with base 10: '77=19737'
invalid literal for int() with base 10: '77=19737'
.."
Also,
"/data_disk2/software/TranscriptClean-2.0.3/TranscriptClean.py:339: UserWarning: Problem parsing transcript with ID 'transcript/10670'
warnings.warn("Problem parsing transcript with ID '" +
/data_disk2/software/TranscriptClean-2.0.3/TranscriptClean.py:339: UserWarning: Problem parsing transcript with ID 'transcript/10345'
warnings.warn("Problem parsing transcript with ID '" +
/data_disk2/software/TranscriptClean-2.0.3/TranscriptClean.py:339: UserWarning: Problem parsing transcript with ID 'transcript/11633'
warnings.warn("Problem parsing transcript with ID '" +
/data_disk2/software/TranscriptClean-2.0.3/TranscriptClean.py:339: UserWarning: Problem parsing transcript with ID 'transcript/11869'
warnings.warn("Problem parsing transcript with ID '" +
/data_disk2/software/TranscriptClean-2.0.3/TranscriptClean.py:339: UserWarning: Problem parsing transcript with ID 'transcript/23980'
warnings.warn("Problem parsing transcript with ID '" +
/data_disk2/software/TranscriptClean-2.0.3/TranscriptClean.py:339: UserWarning: Problem parsing transcript with ID 'transcript/224'
warnings.warn("Problem parsing transcript with ID '" +"
Can you please help me to fix the issue?
Thanks
Philge

Hi,
I'm pretty sure that the "problem parsing transcript..." warnings are being caused by the "invalid literal..." errors, but prior versions of TranscriptClean have buried the stack trace of the thrown errors in a try / except block. If you install the latest version, I am certain it will still throw an error but it will be more informative. Would you be able to run it with the latest commits and copy the output from that run here? Should make it easier to debug.

Thanks for running the commits, I now know what line is throwing the error but am still not entirely sure what's causing it. If you would be able to send me a snippet of your input SAM file (that still causes an error when you try to run it), that would be really helpful.
Alternatively, you can send me the cigar string of one of your transcripts that you know is causing the error.


AddingPowerSupport_For_golang-github-ryanuber-columnize

Adding Power Support.
Adding power support ppc64le
This is part of the Ubuntu distribution for ppc64le. This helps us simplify testing later when distributions are re-building and re-releasing. For more info tag @gerrith3.
The build is successful on both arch: amd64/ppc64le, Please find the Travis Link: https://travis-ci.com/github/santosh653/columnize
Please Let me know if you need any other details or If I am missing any..
Thank You !!

@ryanuber If all the checks are successful, Can you please process this PR?

Hello, In case there are no further checks to be verified, Can we have this PR approved? It is open from quite sometime now.

If ll the checks are successful, Can you please Approve?

Hello, this library is pure Golang which should be portable to any platform. Adding architectures to run the tests on for every single build does not seem necessary to me. I'm unsure what the urgency is for here. If you have any clarifying remarks feel free to send them my way but for now I'm just going to close this. Thank you.

@ryanuber Thanks for looking at the issue.  We have a porting center that ports applications and dependencies to Power for customers and ISVs.  We wind up porting a lot of architecture indendent (supposedly) code in python, go, java, node.js, etc because all of these have the ability to either natively contain architecture dependent code, call out to archtecture dependent code, or depend on a compiler/interpreter which runs on Power.  We run into an incredible number of these which fail to port.  Unfortunately, not being the project maintainer, we don't know wihch projects were supposed to be architecture independent and which are not.  We are actively ensuring that projects that are part of the ppc64le distributions already (and obviously can be built for Power) are maintaining architecture independence (or ported to...) at top of tree to simplify things across the board.  This provides a record of which projects top of tree is actively validated to be architecture independent, ensures that any changes that would create dependencies are caught immediately, and shows downstream builders (e.g. distros, customers, ISVs, etc.) that the package is tested on ppc64le.
Apologies for the seeming urgency, we have borrowed some extra people to help out with this for as many as we can out of the active 28,000+ ubuntu projects.  I realize it may not seem like much to you, but it saves people like our team hours when we are asked to port and validate if we can see that this is already tested.  And, we provide the hardware resources, the builds happen in parallel to your x86-64 builds and should have no negative impact on your tree.

@ryanuber , If you have all the clarifications in place, Can you re-open this PR so as to Merge/Approve later on?

@ryanuber Checking if you have any further questions & also can this PR be re-opened & approved?


steamtinkerlaunch: 11.11 -> 12.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/sonic2kk/steamtinkerlaunch/releases.
meta.description for steamtinkerlaunch is: "Linux wrapper tool for use with the Steam client for custom launch options and 3rd party programs"
meta.homepage for steamtinkerlaunch is: "https://github.com/sonic2kk/steamtinkerlaunch"
meta.changelog for steamtinkerlaunch is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 12.0 with grep in /nix/store/rbryp80sp3c9xqm7g1490vb3m0z296mf-steamtinkerlaunch-12.0
found 12.0 in filename of file in /nix/store/rbryp80sp3c9xqm7g1490vb3m0z296mf-steamtinkerlaunch-12.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
steamtinkerlaunch

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rbryp80sp3c9xqm7g1490vb3m0z296mf-steamtinkerlaunch-12.0 \
  --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 steamtinkerlaunch https://github.com/r-ryantm/nixpkgs/archive/7f64e371c182a8018fde2eda5f1b44bf118a5e8f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rbryp80sp3c9xqm7g1490vb3m0z296mf-steamtinkerlaunch-12.0
ls -la /nix/store/rbryp80sp3c9xqm7g1490vb3m0z296mf-steamtinkerlaunch-12.0/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 built:

steamtinkerlaunch

Maintainer pings
cc @urandom2 for testing.

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


Update readme

Codecov Report

Merging #1 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #1   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          34     34
=====================================
  Hits           34     34

Impacted Files
Coverage Δ

admin_ip_restrictor/__init__.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Enable AA at HMP Ranby

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

Project coverage is 84.57%. Comparing base (2b06d56) to head (635c63f).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #1204   +/-   ##
=========================================
  Coverage     84.57%   84.57%
  Complexity     3757     3757
=========================================
  Files           385      385
  Lines         12767    12767
  Branches       1098     1098
=========================================
  Hits          10798    10798
  Misses         1459     1459
  Partials        510      510

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


[Doubt-Hand-Python] Is it possible to use provided tflite models with TensorFlow?

Found that hand landmark detection is not possible with the Python API. But the hand landmark detection model provided is [dot]tflite. I was wondering whether it is possible to use the model with TensorFlow.

Hi, we will release Python support for more MediaPipe solutions in early Q4, e.g., Oct.
tflite models can work with tflite inference engine. See https://www.tensorflow.org/lite for more details.


Add DbUp - library for deployment of changes to SQL Server databases

DbUp is a .NET library that helps you to deploy changes to SQL Server databases. It tracks which SQL scripts have been run already, and runs the change scripts that are needed to get your database up to date. http://dbup.github.io

@JakeGinnivan FYI...


Description of what Wiby does and how it achieves this needs improvement

I had not been contributing to Wiby as I had other commitments during 2020 and so I came a fresh perspective ( meaning I did not really understand it at first).
What I think is needed is refresh of the readme so that it is more obvious what wiby does for a prospective user and how that is achieved.
I know it is early days at the moment but the readme file is the window into the tool and is equally as important as the code itself.
What we need to highlight is

Wiby's purpose and why you would want to use it
How it works, equally important so that folks want to use it
Many examples

I am going to work on this.
I think small updates that make it easy to review would be the best approach.

I am taking a first pass on the documentation, after a bit of hiatus, making it a bit more digestible for someone without knowledge of the project. Update to the readme are here in my clone. As I go through the exercise important points are being raised.

We are github centric and we should call this out. Meaning, we make assumptions about where the repositories are.
In general dependencies usually refer to npm modules. but we could have repo dependencies of other repo dependents, and this may need documenting
a significant section on the .wiby.json format is needed coupled with the command line options
address complex repo dependents that use modules created from build pipelines that have typescript or other transpiled steps.

Thanks for doing this!

We are github centric and we should call this out. Meaning, we make assumptions about where the repositories are.

This is true - but I'd love to reduce that, e.g. we could replace direct Github API calls when creating commits with raw git commands.

In general dependencies usually refer to npm modules. but we could have repo dependencies of other repo dependents, and this may need documenting

Not following this - can you please elaborate?

a significant section on the .wiby.json format is needed coupled with the command line options

I think one word is not necessary here, but I'm not sure which one 😂

address complex repo dependents that use modules created from build pipelines that have typescript or other transpiled steps.

This should all be happening in CI regardless? Unless you mean we need to take into account folks who commit the build artifacts into git?

Scanning through the README, some notes:

The wiby program exists to create a notification to the dependent package maintainers of changes made in a dependency.

This is not exactly true. We want to minimize the disruption for dependent package maintainers. We want the maintainer of the dependency to have a way to figure out if they might be introducing an unintended breaking change. This is especially relevant for things like fastify, where a change in the core framework might unintentionally result in regressions when used with plugins, etc.
By running wiby, the authors of a dependency will know whether they need to bump major (if dependents break) or alternatively they might have to introduce some fixes to avoid the breaking changes.

example-dependency-id-a, example-dependent-id-a

This has been bothering me for a long time... But I find the dependent vs dependency wording hard to understand at a glance. I know these are industry wide terms, but I wonder if we can find some synonyms or generally a better way to express this? Parent/child is not exactly the right thing either, although we have parent/dependency scattered across the codebase... Maybe there's a way to come up with more concrete examples, as opposed to using abstract terminology? Are there any well known packages that we could use as examples? Even if it's e.g. wiby and tap? Or possibly lodash as the dependency, and pretty much anything as a dependent? Or better yet, something that we have in this org - detect-node-support depends on @pkgjs/nv?
Alternatively, maybe using application vs library as a more abstract concept? Or framework vs plugin? Just that if we go half-way abstract, it might not be obvious enough that the tool itself is more than that...

I think that the thing i run wiby on is a library. It has dependencies (irrelevant here) and dependents (what wiby tests).

I have raised a PR 112 and am still working on it.


GEODE-10291: Add ubuntu 22.04 (jammy) to CI

I don't have ami credentials nor concourse credentials to test any of this, so I will need some help if this should be wanted in CI

Hi @moleske & @pivotal-jbarrett, it seems to me that @moleke addressed all the comments, so I am not sure is this can be merged already?


Bloom Credit

http://www.bloomcredit.io/

Fixed in https://github.com/jplock/awesome-fintech/commit/8abc238bbf2378cde9a322dc6ad164f0d5cbea70


Update sbt-sonatype to 3.4

Updates org.xerial.sbt:sbt-sonatype from 3.2 to 3.4.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.xerial.sbt", artifactId = "sbt-sonatype" } ]

Passed by #180


Add a new TPKeyboardAvoiding_setAdditionalContentOffset to UIScrollView.

I needed to apply an offset to the applied content offset of TPKeyboardAvoiding and thus created this patch.
The additional offset is added to the originally computed offset of TPKeyboardAvoiding. The additional offset is to be set on each UIScrollView on which it is desired.

Can you share an example of how you're using this?

Hi
Does this modification also works with tableview (not scrollview) ?
Thanks

Hi @cbowns. The thing is I don't use it anymore because I refactored the part of my app that was using it.
However I can tell you why I needed this: I wanted the textfield (or the text view) to be placed on the screen at a specific position after the keyboard opens. Applying an offset makes this possible.
@jeromeDms: the change is made in UIScrollView, of which UITableView inherits so it should work, yes.


How to import / use node module package inside src/js file

Let's say I have a javascript file named EpubReader.js inside my src directory and I want to use an already installed node module package (ex : require-dir, lodash etc) in my js file. What is the appropriate way to do this ? I have been trying something like this -
var dir = require("require-dir")
and I received this error "no require-dir" . Although it is installed in my node module directory.
But, if I copy this simple line in a sample index.js file and run it using node command, it works without any error.

Duplicate: https://github.com/readium/readium-js-viewer/issues/638#issuecomment-321926683


Section 3-chroot/072-grub - need check mess in re-link (needed as.gnu also)

In section 3-chroot/072-grub
Build still tries to use clang. Temporarily switch to GCC
mv -v /usr/bin/{gcc,gcc.clang}
ln -sv gcc.gnu /usr/bin/gcc
There a little mess.

It shows "/usr/bin/gcc - no such file"
Need to add  ln -sv gcc.gnu /usr/bin/gcc
Reg. this instruction, configure fails with "wrong assembler" - it looking at "as from clang", not GNU
I added ln -sv /usr/bin/as.gnu /usr/bin/as
After these fixes configuration and compile OK

Read https://github.com/dslm4515/CMLFS/issues/23


Fix build badge on README to point to travis

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.

LGTM


Document env-var configuration options.

Solves #26

Hi @hamishfagg, thanks for contributing this documentation!
Your documentation will make a fantastic addition to this project. I think it adds a lot of helpful information for those trying to connect to LocalStack that aren't able to use the default configuration, and #26 mentions, it's not so discoverable at the moment!
As you can see by #45 we are slightly changing the format of LOCALSTACK_HOST, and your documentation will still be relevant. I'm going to merge this, and rebase my changes (which should then be much more focused!).
Congratulations on your first contribution to this project, and we hope you are getting on ok with LocalStack! 🎉


Remove insight module for now

This is a quick solution to fix #753
Purpose of this pull request?

[x] Bug fix

What changes did you make?
Completely removed the insight module for now. So user tracking will be gone once this version is published.
I’ll be happy to bring back insight when the related work is done. Quoting from #753 to give context:

It’s not as easy as I thought 🙈
Because we have to update sindresorhus/os-name to v5.0.1 in yeoman/insight.
BUT: os-name is a pure ESM since 5.0.0. which means we have to change the module system in insight as well as described over here: https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c
… which means we have to refactor yeoman/yo as well to make this work 😔

Is there anything you'd like reviewers to focus on?
@SBoudrias Do you think this is valid approach to get this bugfix out as soon as possible?

I wonder what the failing (Node 16) test is about. Guess it was just a hick-up 🤔
Pull request test runs are green (compared to the push one).
Plus it runs fine locally with the exact same Node version:


[WIP] ModelState/DataAnnotation library

I've written a form validation library that use the standard ComponentModel.DataAnnotation and I've added a test page in StandaloneApp.
the library it's very simple and allow sharing models with dataannotations between blazor and backend (ref #17).
all the logic is in the ModelStateDictionary<T> class (so isn't in the form, as #970) and allow code like this
    var ModelState = new ModelStateDictionary<TestModel>(new TestModel())

            <input type="text" bind="@ModelState[x => x.TextValue]" class="form-control" />
            @ModelState.ValidationFor(x=> x.TextValue)

        <button onclick="@(() => this.UpdateModel())">Update model</button>

    protected virtual void UpdateModel()
    {
        this.ValidateModel();
        if (this.ModelState.IsValid)
        {
            this.ModelState.Update();
        }
    }

Since <input> tag values are always strings, ModelStateDictionary does an automatic type conversion of all values using TypeConverter (since it's available in netstandard2.0): putting an invalid value (like a non convertible string into number) will show an error automatically.
For now it's support short, int, long, datetime, float, double and nullable
ModelStateDictionary can update the Model via Update(T model) function.
The library support custom validation (with virtual OnCustomValidateModel and AddModelError).
It's a starting point (no xml comments or test cases), but seems work.

I'm making a pass through some of the older PRs to see what we can clear up.
This one certainly looks interesting, but our hope would be that this type of validation library could be implemented as an external package rather than having to be embedded in the core. If you're still interested in proceeding with it, would you be able to see what would be needed to make it a separate package? If additional extensibility APIs are needed we can certainly consider adding them. Hope that's OK!


Remove windows warnings in CameraTracking

Signed-off-by: ahcorde<EMAIL_ADDRESS>🦟 Bug fix
Fixes #
Summary
Remove windows warnings in INTEGRATION_CameraTracking generated by pose.pb.h
Checklist

[x] Signed all commits for DCO
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[x] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Related windows build with failures


add generation audit from docuemnt

Coverage decreased (-3.09%) to 82.762% when pulling 09268e10f46bb31bdc0f578cea91583aec73deea on audit_generation_from_document into 02ef72c7c29b1b754232859615660fa0ba051d9b on sandbox.

Coverage decreased (-3.09%) to 82.762% when pulling 153b8c08c0753a0e08db49d02e6ed9814b7cdbaf on audit_generation_from_document into 02ef72c7c29b1b754232859615660fa0ba051d9b on sandbox.


Force comparison of MBID

ISRC turns out to be a minefield of inconsistency. MBID is generally a good indicator for tracks, so if that's present on both tracks, default to using that.

See https://github.com/music-assistant/hass-music-assistant/issues/2316 for discussion

Nice, easy and clean. Maybe add a link to the issue report in the comment as a reference for the future why we have this early mbid check in the code

I did already in the comment just above yours!

Nice, easy and clean. Maybe add a link to the issue report in the comment as a reference for the future why we have this early mbid check in the code

Done.


S9 [Task] Adds role description to API and to tests

Task
Adds role description to API and to tests

Task Title
Adds role description to API and to tests

User Story
In support of #913

Description
Adds role description to API and to tests

Business Need (the why):

Applications or Systems impacted

Dependencies

superceded by #933


Incorrect alias redirect and incorrect language url alias choice in a navbar on mobile.

Description of bug
When multiple aliases and languages are setup on a website desktop works perfectly, but mobile redirects to a different alias (not by language but totally different alias) or generates incorrect links in a navbar (to pages and language switch). These two different behaviours occur at different urls.
Steps to reproduce
List the precise steps to reproduce the bug:

Setup your website locally with any url you want (for instance website.com.ua)
Add another alias local.website.com.ua
Theoratically should be optional, but it's done on my local: I've deleted website.com.ua url from my local iis and it's used for production so that I can access production and local websites at the same time from different urls. Instead locally local.website.com.ua is set up in IIS
Go to PersonaBar => Settings => Languages
Add Ukrainian language and set it as default language
Go to PersonaBar => Settings => Site Settings => Site Aliases
Edit your aliases in such way that you have these settings:

website.com.ua:

language: Ukrainian
browser: normal
theme: none
is primary

website.com.ua/en:

language: English-US
browser: normal
theme: none
is primary

local.website.com.ua:

language: Ukrainian
browser: normal
theme: none

local.website.com.ua/en:

language: English-US
browser: normal
theme: none

Edit your page code in such way that you have language switch added:
<dnn:LANGUAGE runat="server" ID="LANGUAGE1" ShowMenu="False" ShowLinks="True"                                          SelectedItemTemplate=' '                                          AlternateTemplate='    '                                          ItemTemplate='   ' />
Go to local.website.com.ua in browser - should open it and language url is local.website.com.ua/en
Go to local.website.com.ua/en in browser - should open it and language url is local.website.com.ua
Open dev tools and switch to mobile view (any should work, but you can choose Iphone SE)
Go to local.website.com.ua/en in browser while still in a mobile view
See error: language url at flags and pages urls in a navbar lead to the website.com.ua alias, which leads to production instead of localhost
Go to local.website.com.ua in browser while still in a mobile view
See 301 redirect to website.com.ua

Current behavior
Mobile view redirects to a different alias for some reason and generates incorrect url for language.
Expected behavior
local.website.com.ua should always lead to local.website.com.ua and all the urls both in navbar and for all languages should be generated for the same alias. Same applies for website.com.ua.
Screenshots
Let me know if you need screenshots. It seems to me that the description should be enough as it's not a design issue.
Error information
I suspect that it has to do with those settings for browser type in aliases. We don't have separate urls for mobile browser as our website has responsive interface. But it seems that DNN looks that it's a mobile browser, doesn't find any alias for mobile browser and redirects to a primary alias. However it doesn't explain why does it open local.website.com.ua/en and doesn't redirect that one and generates primary alias url for language change only.
Additional context
Site Alias Mapping Mode was left Canonical (default one). I've tried to switch it to none. Didn't change current behaviour. I've found one discussion at DNN forum that setting it to redirect could cause unwanted redirect between aliases. But it's not the case right here.
The only solution I've found here is to set url as primary separately for each server (local, test and production). Meaning on local local.website.com.ua and local.website.com.ua/en would be primary urls and then it works as expected. But it's very inconvenient thing to do regularly on deploy and I'd expect it to work in a correct way without these settings.
Affected version

[ ] 9.10.02 latest supported release

Affected browser

[ ] Chrome Version 106.0.5249.119 (Official Build) (64-bit)
[ ] Windows 10 Enterprise LTSC (x64)

There is also a similar discussion at the DNN forum as well here.
And desktop works as expected, but mobile has not logical behaviour from a user perspective.

I cannot make this happen as described on the current codebase. Which means either:

It was somehow fixed since this was reported.
Or there is something else third party or environmental to make this happen

I am closing this issue but if you have any other insights to make it reproducible, just comment and I'll reopen this.


Metadata updates to title do not populate when title is used for match

Version Number
1.19.1
What branch are you on?
nightly
Describe the Bug
Using title as the primary attribute within match to perform metadata updates will not allow metadata updates to title itself. Any other metadata updates specified, such as sort_title will still populate however.
Relevant Collection/Overlay/Playlist Definition
metadata:
  Nomadland:
    match:
      edition: Test1
    sort_title: NomadlandTEST1
    title: NomadlandTEST1

  581734:
    match:
      edition: Test2
    sort_title: NomadlandTEST2
    title: NomadlandTEST2

  Nomadland3:
    match:
      title: Nomadland
      edition: Test3
    sort_title: NomadlandTEST3
    title: NomadlandTEST3

Logs
https://gist.github.com/Kevin2kkelly/231c73494c5d69f19c07596d2eb2014b

We are looking into this

This should be fixed in nightly51 which has now released - any issues please let us know


Fix HiSpeed link for "Wire CD Ray"

Each Wiremod HiSpeed device has an output labelled "Memory". This output is often used with the bus device for the ZCPU.
I noticed that the Wire CD Ray device lacks this convention and instead users can use any of the 6 outputs ("Data", "Sector", .., "Address") to link to the bus. This is confusing.
I have added the memory output and renamed the underlying "Command" variable to match other HiSpeed devices' "Memory" variable to improve code readability.
Pros:

Uses the Memory output convention like other HiSpeed devices.
Backwards compatible.

Cons:

As @Divran pointed out, this doesn't fix the underlying issue that the bus system and corresponding devices could do with a rewrite to use Wirelinks.

Unfortunately, I can't merge this. I'm not going to accept such a large refactor to one entity, to fix the issues with a very small, completely unrelated entity. The addressbus entity (https://github.com/wiremod/wire/blob/master/lua/entities/gmod_wire_addressbus.lua) tells people to wire to the output called "Memory", but then it doesn't actually use it at all. All it does is use the input to retrieve the entity that you wired it to. This could therefore be solved much better by refactoring the addressbus entity. For example we could make it have 4 wirelink inputs instead of number inputs. Or 4 entity inputs. The issue then is backwards compatibility, but that can probably be solved somehow. I could check if my aliasing system is up to the task.

That is how the hispeed system works - you can use any output you want, it only looks at the entity that's connected, not which output. The same applies to any other hispeed device that has multiple outputs. "Memory" is only a convention if there are no other outputs.
Also I don't think the name of the variable has anything to do with the name of the output.

no, I think that was how it worked before wirelinks were created. In fact, I think that's how it worked before any other wire type was added at all. Back when only numbers existed. Because if other types had existed, whoever invented hispeed would've used entity types instead. Wirelinks are one of the newest types of wires to be added though so that's why old shit like cpu and its related entities doesn't use it.

@immibis
That may have been the case but there are hispeed devices with Memory and other outputs such as the keyboard.
So you've got:

Memory output, no other outputs
Memory output, has other outputs
No memory output, has other outputs

Surely you'd deal with the last case which is an outlier.
This would be better off with an entity or wirelink refactor. I issued this PR because I found no information online about why CD Ray deviates from the rest of the hispeed devices with no Memory output; I wanted to save some other people the trouble I experienced.

@Gavxn As I just said it literally doesn't matter. Maybe just for consistency, but other than aesthetically it has no impact whatsoever.
@Divran That's how it worked before wirelinks were created and, unless there's been a recent change I'm unaware of (quite possible as I haven't played in ages), that's how it still works because it can't be changed without breaking compatibility. E2 is the exception here by using wirelinks.

Yes so the options are

Do nothing. You can still use the address bus just fine just by wiring it to any random number output of the entity you want.
Edit the address bus to use wirelinks and try to preserve backwards compatibility by adding some checks.


fix bug with missing pid file

Ran into a problem running blue pill where if a pid file is missing for any running bluepill application, we wind up with an error when trying to run any other bluepill.
In the case pasted below, testworker-husky was being run by bluepill but the expected pid file was not found.

thanks to @sferik and @lrajlich - last update was applied to master branch.


[Bug]: Dispensers can't place shulker boxes

Bug Description
The latest version won't let dispensers place shulker boxes, even in unclaimed chunks.
How to Reproduce?

Place dispenser
Insert shulker box
Activate dispenser

Expected Behavior
The dispenser should place a shulker box
Version
1.1.0
Mod Loader Version
1.20.2 - 0.14.24
Mod Loader
Fabric
Logs or additional context
https://pastebin.com/fcjANdyX
Code of Conduct

[X] I have searched the issue tracker and confirmed that the issue has not been reported
[X] I have checked the FAQ (if one is present) and my issue is not listed
[X] I have verified that I am using the latest version of the mod
[X] I have verified that I have installed all the required dependencies for the mod
[X] I have verified that I do not have Optifine installed. We will close issues if we detect Optifine is in your pack. We cannot fix Optifine issues. Please do not report issues with Optifine present.

Fixed in 1508e0b


build: use nox for tests and docs

Fix #239
In this PR I added a few configurations to build unit tests and documentation pages using nox. It creates isolated environments to ensure that the build will work when send to RDT. It's also the first step toward unit testing.

create 2 nox sessions: "lint" to run the pre-commits (assuming it will be merged) and "docs" to create the documentation in a docs/_build folder.
remove makefiles as they are nicely replaced by nox -s docs
add the nox-produced files to .gitignore
create a doc set of extra_requires to have everything needed for the documentation
set up a .readthedocs.yaml file so that the documentation is built using the correct env

As I need to check the RDT build there will surely be test commits, I assume you will just squash my PR so I'm just sorry for the noise.

@maartenbreddels, @mariobuikhuizen I would like the doc to be build on main (at least latest) but when I install in dev mode there are many npm command that start. Of course it wasn't an issue before because RDT was build from the pipy version.
Would you have any guidance ?

I never build on rtd, because there wasn’t enough memory. Why do you want to do that?

If I want to work on the API build in #238 I would like to make sure latest is on track with main. So I would need to run: pip install .[doc]

I think you should install Jupyter lab to be able to build

thanks @maartenbreddels for the tips. The complete solution was "install jupyterlab before the lib to unable the build of jupyter extentions when npm install will be called from the js folder to execute webpack scripts".
with the pre-install job, it works like a charm. This is then ready for review

sorry for the noise. I made many tests from RDT side. I hope they will find a solution from their side: https://github.com/readthedocs/readthedocs.org/issues/10223

This PR now depends on the fix in: https://github.com/widgetti/ipyvuetify/pull/259

@mariobuikhuizen I rebased on the current master and solved the remaining RDT related issue. I think this one is also ready to go.


capacity does not work on often reload

capacity setted to 10
cache.keys().length = 135
on every page reload it requests resource and put into cache
this code does not work because $$lruHeap.size() equals to 1
 if ($$lruHeap.size() > this.$$capacity) {
	            this.remove($$lruHeap.peek().key);
	          }

so capacity does not remove old items if they does not expire

What does often reload mean?

Page reload. So every time cache initialized again.

I need to see your entire cache configuration. Code examples please. Version please. Even better, a plunker that demonstrates the issue. Details!

capacity:10
maxAge: week
storageMode:localstorage
on every page load http get: request?random puts on the cache
so if i make 1000 loads it make 1000 entries in localstorage

Could you please fork this plunker and demonstrate the issue? http://plnkr.co/edit/RXWGpYqRAfKPS6R38MXs?p=preview

Refresh result and see as keys count exceed capacity
http://plnkr.co/edit/6zyAvLYTDeHJSbCvUllF?p=preview

Indeed it does.

Can you try 4.4.0 and see if it works better?

You can try the same plunker. Result the same.


fix firebase query returns 2 sets for #263

Fixing #263
I saw that the __queryChanged is being called thrice, making creation of event listeners called thrice. Currently wrapped them in a checker if both query and oldQuery exists with this idea

__queryChanged is called in the attached phased
__queryChanged is called when any time a new query is created on calculated in the __computeQuery

Can anyone give me some feedback on any bugs I missed?
Thanks

@tjmonsi - thanks a lot for this - I 'll try to test it over during the we!

@christophe-g give me a heads-up when you're done. Thanks :)

Using this commit I'm consistently getting duplicates.
I agree with the idea of getting rid of "on-value", any idea why it was used in the first place?
I don't have time to explore your code, but here is the test case I was using: https://github.com/tbeattysk/firebase-duplicate-test

@tbeattysk did you manage to use my branch for your test case?

@tbeattysk I used your repo as a test case using a fix that I added. It's working now (I think).
As for using on-value: see merged PR: https://github.com/firebase/polymerfire/pull/230

@tjmonsi I have tested the latest commit (https://github.com/firebase/polymerfire/pull/270/commits/80ee04a32397270b1fbaa78291dd338c6438413e), and it's still not working 100% correctly based on my test case https://github.com/firebase/polymerfire/issues/263#issuecomment-328293934. Now I don't get the duplicated result, but if you click the "Add query" fast enough, you'll get empty query result on the first few query before the result is "eventually" return in subsequent query.
I also have another <firebase-query> in my project, which doesn't return any expected result at all.

I think the original idea of using on instead of once for the value event was a good take. Because the once event is not being called. I don't know why the promise is not being resolved. I think I would try to create a simple repo for it and file a bug in firebase for it.

@andrewspy can you check it again?

@tjmonsi I am happy to report that it is finally working on both my test case and my project.
Thanks!

@christophe-g any report?

@tjmonsi - looks good! thanks and sorry for the delay (... quite busy ...)

@tjmonsi Please release the new version 2.2.1.

@FluorescentHallucinogen just released 2.2.1... sorry for the delay


added catenary curve calculator

Fixes Issue🛠

Closes #1702
Description👨💻
A calculator that takes user inputs to visualise and calculate the catenary curve
Type of Change📄

[ ] Bug fix (non-breaking change which fixes a bug)
[x] New feature (non-breaking change which adds functionality)
[ ] Style (non-breaking change which improves website style or formatting)
[ ] Documentation (non-breaking change which updates or adds documentation)
[ ] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

Please review my pull request @Rakesh9100


Forgot to add the section number per neurite in the init

Current coverage is 100.00%

Merging #275 into master will not affect coverage as of 3532b5d

@@            master    #275   diff @@
======================================
  Files           42      42
  Stmts         1797    1797
  Branches         0       0
  Methods
======================================
  Hit           1797    1797
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 3532b5d

Powered by Codecov. Updated on successful CI builds.

Can we make this nsections or number_of_sections? section_number means something else to me. Like the number of a section.

Sure.number_of_sections is good and clear I think.

If this is ok, can you merge it? I have another pull request waiting on the line. :P

Done. You should probably use branches for separate features so you don't get blocked by pending merges. It is more involved than with gerrit, but it is workable.


Post Win 10 Reinstall: bash: /root/.bashrc: Input/output error

Version:
Microsoft Windows [Version 10.0.15063]

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
Trying to reinstall bash after having reformatted my HDD and reinstalled Windows.

What's wrong / what should be happening instead:

So I figured when I got the .bashrc input/output whatever, "OK, this might be related to restoring something from my backup that I shouldn't have because this is a fresh install of Windows."  So I uninstall the Linux subsystem, reboot, reinstall it, reboot, and try and run bash and I get the error listed in the title.
"Well, that's annoying," I figure, but decided it might be a permissions issue, so I open an administrative command prompt and just type in bash.  See below for results.

Strace of the failing command, if applicable:  (If <cmd> is failing, then run strace -o strace.txt -ff <cmd>, and post the strace.txt output here)

C:\Windows\system32>bash
bash: /root/.bashrc: Input/output error
root@JBox401:/mnt/c/Windows/System32#

I did some cursory searching, but I wasn't able to find anything that pertained to my problem.  I don't have any NTFS compacting or encrypting on, nothing like that, and the HDD is a standard run-of-the-mill spinning platter HDD from Seagate...I think...  Maybe Western Digital.

Do not change Linux files using Windows apps and tools
https://blogs.msdn.microsoft.com/commandline/2016/11/17/do-not-change-linux-files-using-windows-apps-and-tools/
If you restored files from Windows software, that action is in violation of the above.
When restoring a customized .bashrc and/or other files it has to be done from bash via /mnt.
If so, let's do lxrun /uninstall /full.

In my defense, I wasn't trying to backup that stuff; I was just copying my Users folder to get all my documents and music and app settings in one go...  ^^;;;  Thank you for the link!  It showed what folder I need to NOT copy in future backups! ^^


Better error message for payment in multi-checkout

The error message should have a better text than saying InvalidField: card_expirationMonth and card_accountNumber. The output is not proper for usage.

let's catch this manually in Spartacus, and i created a ticket in Stingrays to improve


[KOGITO-771] - Upgrade GraalVM to 19.3.1

Signed-off-by: spolti<EMAIL_ADDRESS>

once the jenkins job gets finished I'll push the rc2 tag.

Images availagle under tag rc2
@sutaakar @radtriste please test it.

Started BDD tests with rc2

tests ran ok with rc2 images
Solved:

native (with update to Graalvm 19.3.1) => OK
persistence (https://github.com/kiegroup/kogito-runtimes/pull/346) => OK

@radtriste cool, thanks.


Unknow error trying to log in

Attempted to use Kia integration for the first time.  Canada User.
This error originated from a custom integration.
Logger: aiohttp.server
Source: custom_components/ha_kia_hyundai/api_cloud_ca.py:56
Integration: Kia/Hyundai (documentation, issues)
First occurred: 8:09:08 PM (3 occurrences)
Last logged: 8:09:41 PM
Error handling request
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/aiohttp/connector.py", line 980, in _wrap_create_connection
return await self._loop.create_connection(*args, **kwargs)  # type: ignore[return-value]  # noqa
File "/usr/local/lib/python3.10/asyncio/base_events.py", line 1103, in create_connection
transport, protocol = await self._create_connection_transport(
File "/usr/local/lib/python3.10/asyncio/base_events.py", line 1133, in _create_connection_transport
await waiter
File "/usr/local/lib/python3.10/asyncio/sslproto.py", line 534, in data_received
ssldata, appdata = self._sslpipe.feed_ssldata(data)
File "/usr/local/lib/python3.10/asyncio/sslproto.py", line 188, in feed_ssldata
self._sslobj.do_handshake()
File "/usr/local/lib/python3.10/ssl.py", line 975, in do_handshake
self._sslobj.do_handshake()
ssl.SSLError: [SSL: WRONG_SIGNATURE_TYPE] wrong signature type (_ssl.c:1007)
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 433, in _handle_request
resp = await request_handler(request)
File "/usr/local/lib/python3.10/site-packages/aiohttp/web_app.py", line 504, in _handle
resp = await handler(request)
File "/usr/local/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line 117, in impl
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 85, in security_filter_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 80, in ban_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 235, in auth_middleware
return await handler(request)
File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 146, in handle
result = await result
File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 180, in post
return await super().post(request, flow_id)
File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 72, in wrapper
result = await method(view, request, data, *args, **kwargs)
File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 110, in post
result = await self._flow_mgr.async_configure(flow_id, data)
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 271, in async_configure
result = await self._async_handle_step(
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 367, in _async_handle_step
result: FlowResult = await getattr(flow, method)(user_input)
File "/config/custom_components/ha_kia_hyundai/config_flow.py", line 141, in async_step_auth
await api_cloud.login()
File "/config/custom_components/ha_kia_hyundai/api_cloud_ca.py", line 56, in login
self._access_token, _ = await self.api.login(self.username, self.password)
File "/usr/local/lib/python3.10/site-packages/kia_hyundai_api/ca.py", line 147, in login
await self._post_request_with_logging_and_errors_raised(
File "/usr/local/lib/python3.10/site-packages/kia_hyundai_api/ca.py", line 32, in request_with_logging_wrapper
response = await func(*args, **kwargs)
File "/usr/local/lib/python3.10/site-packages/kia_hyundai_api/ca.py", line 141, in _post_request_with_logging_and_errors_raised
return await self.api_session.post(url=url, json=json_body, headers=headers, ssl=self.ssl_context)
File "/usr/local/lib/python3.10/site-packages/aiohttp/client.py", line 536, in _request
conn = await self._connector.connect(
File "/usr/local/lib/python3.10/site-packages/aiohttp/connector.py", line 540, in connect
proto = await self._create_connection(req, traces, timeout)
File "/usr/local/lib/python3.10/site-packages/aiohttp/connector.py", line 901, in _create_connection
_, proto = await self._create_direct_connection(req, traces, timeout)
File "/usr/local/lib/python3.10/site-packages/aiohttp/connector.py", line 1206, in _create_direct_connection
raise last_exc
File "/usr/local/lib/python3.10/site-packages/aiohttp/connector.py", line 1175, in _create_direct_connection
transp, proto = await self._wrap_create_connection(
File "/usr/local/lib/python3.10/site-packages/aiohttp/connector.py", line 984, in _wrap_create_connection
raise ClientConnectorSSLError(req.connection_key, exc) from exc
aiohttp.client_exceptions.ClientConnectorSSLError: Cannot connect to host kiaconnect.ca:443 ssl:<ssl.SSLContext object at 0x7fdc11789040> [[SSL: WRONG_SIGNATURE_TYPE] wrong signature type (_ssl.c:1007)]

Sorry, I couldn't reproduce this issue, it is based on the kia api servers using old ssl cert standards that newer python is identifying as inadequate.
What platform are you running home assistant on?

I am running it on an old intel laptop.
Home Assistant 2023.5.2Supervisor 2023.04.1Operating System 10.1Frontend
20230503.3 - latest
On Thu, May 18, 2023 at 9:38 PM Brendan Dahl @.***>
wrote:

Sorry, I couldn't reproduce this issue, it is based on the kia api servers
using old ssl cert standards that newer python is identifying as inadequate.
What platform are you running home assistant on?
—
Reply to this email directly, view it on GitHub
https://github.com/dahlb/ha_kia_hyundai/issues/73#issuecomment-1553890855,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUPNFW6H6GGIMVJGIA7J4VDXG3FJPANCNFSM6AAAAAAXWORADA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Leo Vander Burgt
416-988-7835

have you tried https://github.com/Hyundai-Kia-Connect/kia_uvo they have a maintainer that has access to CA accounts, I appreciate you letting me know the CA implementation here isn't working but as I can't address it; I'll just remove it and stick with the implementation for USA wrote and drop the port I did of the CA implementation.


Request Body Is Not Sent

Describe the bug
During a POST/CREATE/PUT/PATCH request the request body is not actually being sent.
To Reproduce
Steps to reproduce the behavior:

Go to postwoman.io
Switch to POST
Enter: https://jsonplaceholder.typicode.com as URL and /users as path
Add a field "name" with value "foo"
Send the request
Check the response and see missing field "name" with value "foo"

Expected behavior
The response should be { "name":"foo", "id": 11}
Desktop (please complete the following information):

OS: Ubuntu 18.04
Browser: Chrome (Incognito Tab)
Version: 76

Additional context
I’ve tested this both with a local development project as well as the JSONPlaceholder API and the results are both the same. This problem did not yet occur 20 hours ago, so perhaps it’s a regression from one of the more recent commits.

Looking into it now. I suspect this is after the axios migration.

I can confirm it is working again on my end as well! :+1:  Thanks for the quick response & fix.


Install on Debian/Ubuntu

Dear all,
For your information, I have also encountered a few issues when installing on Debian/Ubuntu and here are my results so far.
I have written down some notes here:
https://github.com/cfenell/rucio-1/blob/install-doc/doc/gfal2.md
My notes on installing Rucio on Ubuntu are also here:
https://github.com/cfenell/rucio-1/blob/install-doc/doc/rucio-server.md

Debian stable with gfal2 library and plugins from Debian unstable packages (version 2.20.5-1): gfal2-python and gfal2-util from develop branch compile without problems and the gfal CLI utils seem to work flawlessly.

Ubuntu 20.04 with gfal2 version 2.20.3: gfal2-python and gfal2-util from develop branch fail to compile. Master branch compiles but I get
gfal-ls gsiftp://..../
gfal-ls error: 70 (Communication error on send) - globus_l_ftp_control_send_cmd_cb: gss_init_sec_context failed to generate output token
Is this expected or have I missed something else?

Best regards Carl-Fredrik Enell, EISCAT

Hello,
Thanks for reporting this!
It's definitely not expected, but at the same time, Debian and/or Ubuntu are not our main platforms so the result is not entirely surprising.
I went through the installation guide [1]. If I understand correctly, you've installed a tagged release of the gfal2 library and later tried to compile gfal2-python from the develop branch. This wouldn't work as we're transitioning Gfal2 projects to the next major release series. This means the develop branches are on gfal2-util-1.8.0 / gfal2-python-1.12.0 / gfal2-2.21.0. Having gfal2-python from develop branch (1.12.x series) won't compile without gfal2 2.21.x series.
I would recommend to always build either the develop branch on all projects or the tagged release (master branch) on all projects. Mixing a develop branch with a tagged dependency won't work between major releases.
Can you try again having all of them compiled from develop branch of the latest tag branch?
(gfal2-util-1.7.1 / gfal2-python-1.11.1 / gfal2-2.20.5)
Cheers,
Mihai
[1] https://github.com/cfenell/rucio-1/blob/install-doc/doc/gfal2.md

Hello,
Glad to hear it worked in the end.
I will close the ticket then.
Cheers,
Mihai


[feat]: each control is an island (initial components)

Create an Astro component for each of the following input control types:

Text
Checkbox
Button

Yes indeed! Preferably one for each of the possible values for FormControl.type, though not expected a full implementation for all of them. More on just having initial components for each.
There's another bug issue where I mention a clean up on the Field.astro component by creating a new RadioGroup.astro component. But if you can do that for as well on this issue (I think it fits the scope) then that issue can focus on the bug. :)


Assassin's Creed II issues found

[ ] Blood suspended in air

[ ] Hair not rendering right

[ ] Black squared textures falling from sky

[ ] Low poly hay

Software information
All high settings; MSAA x2
System information

GPU: GTX 960
Driver: 418.56
Wine version: 4.6
D9VK version: bb776e5e3583b2ced1da06e32c32f90b5e850c75

Apitrace file(s)

https://drive.google.com/file/d/1qavOf6brbxieRQQl747rK08cuU63XSiA/view

Log files

https://github.com/Joshua-Ashton/d9vk/files/3294855/AssassinsCreedIIGame_d3d9.log

Result of alpha test regression
Fixed by https://github.com/Joshua-Ashton/d9vk/commit/e98ceff97f721d4ca8f1d8c10742ce78dbffd76f


fix linux keybinding conflicts

Alt-I is opening the devtools on linux instead of focusing input
Alt-P no longer works for codeedit preview (as Alt-P is used for switch tab)
copy/paste keybinding is not working for copying command strings from previously executed commands in tabs

Just found another conflict with Cmd-R in (when you're in Ctrl-R history search) and Cmd-R for restart line.


weekly ZAP CRM export is failing sporadically

link to failed run
dcp_projects appears to fail with Exception: data has current_milestone of 1cb27780-be0d-ed11-82e3-001dd8068d18 but expanded of ***
not sure why dcp_projectmilestones fails. usually if dcp_projects fails, others do

haven't seen this happen in the many runs during recent dev, but can't be sure it was fixed
gonna lower the priority of this issue and move it to Backlog

closing since it hasn't happened in a while. may have been caused by trying to append to a sql table from a previous run rather than deleting and recreating it


[SR-337] hasPrefix and hasSuffix not implemented in Swift

Previous ID
SR-337

Radar
None

Original Reporter
@alblue

Type
Improvement

Additional Detail from JIRA

Votes
0

Component/s

Labels
Improvement

Assignee
None

Priority
Medium

md5: 7139ff1a5de95a9c013b0ae4cb94233c

Issue Description:
In stdlib/public/core/StringLegacy.swift there is a fixme to implement hasPrefix and hasSuffix without Objective-C
// FIXME: Implement hasPrefix and hasSuffix without objc
// rdar://problem/18878343
I'd like to give this a go.

Fixed in https://github.com/apple/swift/commit/f9b3e14137ac2f50e93aa2c1db28511e05f88e75.


vscode.dev PWA Icon size on Windows 11 is too small

The icon comes with a transparent frame which will take some space in Windows 11 when installed as an PWA by Edge. Removing the transparent frame could fix it, but I'm not sure if it will impact other platforms. You can find the size difference from the screenshot below (left vscode.dev, right installer.exe).

Version: 1.76.0
Commit: 92da9481c0904c6adfe372c12da3b7748d74bdcb
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/110.0.1587.63
Embedder: vscode.dev

I tried to take a look at the issue, but the vscode SVGs all render wrongly with a white background in many applications including Windows explorer.


Travis CI fails intermittently due to memory issues

Capturing these failures in a ticket
https://api.travis-ci.org/v3/job/429869920/log.txt
Java HotSpot(TM) 64-Bit Server VM warning: INFO: os::commit_memory(0x00000000c1980000, 285736960, 0) failed; error='Cannot allocate memory' (errno=12)

Another issue reported by @leletan is
https://api.travis-ci.org/v3/job/429910904/log.txt
java.io.IOException: Filesystem closed
    at org.apache.hadoop.hdfs.DFSClient.checkOpen(DFSClient.java:808)
    at org.apache.hadoop.hdfs.DFSInputStream.close(DFSInputStream.java:710)
    at java.io.BufferedInputStream.close(BufferedInputStream.java:483)
    at java.io.FilterInputStream.close(FilterInputStream.java:181)
    at com.uber.hoodie.common.table.log.HoodieLogFileReader$1.run(HoodieLogFileReader.java:104)


Agree on project vision

We need to have some additional value to make a fork. The most obvious one is androidX migration. Any further ideas a welcome

One of the most obvious reasons for is to rewrite core part and dsl in Kotlin. But before that we need to found out the benefits of moving to Kotlin.

The main benefits that I see are

dropping the support of Java DSL (less work for us to do)
better Kotlin support with DslMarkers, context-bound extension functions, nullable values and so on
party tricks with infix and inline functions, reified generics

Another one reason to use Kotlin is that Google claimed its libraries as Kotlin-first. Seems than native android development is moving towards this direction.

So, draftly features list looks like this:

[ ] AndroidX support (is it already merged to original project?).
[ ] Kotlin-only library (no Java API support).
[ ] DslMarkers, better nullability control, inline/infix functions.
[ ] (?) Serializable view hierarchies (e.g. for server-side UI). Hello, Compose.
[ ] (?) Extend DSL to other UI frameworks (e.g. JavaFx) - can be useful as a research and/or fast prototyping approach.

cc: @Mishkun @sgrekov @r4zzz4k

My five cents:

(is it already merged into original project?).
On one hand, it's already merged into ours. On the other hand, the way things go really depends on how we would coordinate our fork with the previous maintainer -- would this project be official continuation of Anvil or a usual fork.

(?) Extend DSL to other UI frameworks (e.g. JavaFx)
I'd probably postpone discussion on other UI frameworks as there is a long way to go: @sgrekov has correctly noticed that DSL is tied to Android right now, so the first step would be to setup multiplatform project structure and start abstracting out and pulling parts of current API into common module.

Yes, serializable view hierarchies and DSLs for other UI frameworks are definitely quite far from the current point. However, sometimes it's nice to keep such things in the head while making various decisions.

Another Kotlin feature that we can use is inline classes. We can use them instead of Int, where we are awaiting for specific Int, like dp, sp, ResInt, etc

Clearly, Issue is resolved. To sum up conversations held here and in other places, the main idea of making this project as a fork of Anvil is to provide easy-to-use Kotlin-based performant View-DSL.
Main points to use Inkremental vs Anvil:

Kotlin-first
(future) Multiplatform support

Main points to use Inkremental vs Jetpack Compose%

Easy migration
Flat learning curve
Intermediate step before full Compose migration


Fix step measure types

The step of the Hz Units and Bytes per Second Units is 1000 and not 1024. For instance:
3 MHZ = 3 * 1000 HZ (step = 1000)

@amaurygonzalez Is this change for one of the blocker BZ? If so, please add the BZ link to the description. Thanks!

@amaurygonzalez Can you let me know if there is a BZ or issue associated with this PR? I need to know before I can merge it. Thanks!

Hi @gtanzillo, this PR don't correspond with any BZ. I noticed the error when I was reviewing the code

Thanks @amaurygonzalez.  👍


Events with times do not show up in the demo

Here. Is the time format wrong, or could it be a localisation issue? (I'm in UK)

use moment.js
https://fullcalendar.io/docs/lang


nydus-test framework supports fscache mode

Improve current framework to support start nydusd as fscache daemon
add test case for nydusd as fscache daemon

Excellent!


Re-organize files

This PR makes some organizational changes based on the previous PR.

Updated the yml file to address a warning created by a missing package
Removed the scripts and data files associated with the "Experiment 1" because I jumped ahead in the analysis prematurely and we will return to this analysis later
Add a description in the readme about what the goals are for Experiment 0
Created a main NB to run the scripts for Experiment 0 in order using parameters from a config file.  This structure may be useful for future analysis if we are changing different variables between analyses.

Thank you for the comments.  I have made the minor adjustment to my scripts as suggested.  Perhaps we can meet to discuss what you had in mind for next re-organization of this repo, it looks like I got a little lost.