SurveyHumanGlomerulonephritis

Project short name:
SurveyHumanGlomerulonephritis
Primary Wrangler:
Ida
Secondary Wrangler:

Link to Ingest
https://contribute.data.humancellatlas.org/projects/detail?uuid=0057c36c-06ce-4cdf-bff4-533ad13f090c&tab=project
Associated files

Google Drive:

Published study links

Paper: A single-cell survey of the human glomerulonephritis

Accessioned data: On demand from authors

Key Events

[ ] Convert published metadata to HCA spreadsheet
[ ] Manually curate dataset to meet HCA metadata standard
[ ] Collect any matrix and cell-type annotation files
[ ] Upload sheet to validate metadata
[ ] Transfer raw files to ingest to validate data files
[ ] Check linking using ingest graph validator
[ ] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[ ] Submit dataset to Production
[ ] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

Data is available on demand form the authors --> I sent an email to ask for anonymised cell count matrices (thread)
If they don't reply to our request we should get in touch with Peng and discuss this

I've emailed the authors about available data, I'm closing the ticket now. This can be re-opened if the authors reply

I've emailed the authors about available data, I'm closing the ticket now. This can be re-opened if the authors reply


Popover dismiss on next click failing on iOS

Your example does not work on ios http://getbootstrap.com/javascript/#dismiss-on-next-click
I have seen that there was a discussion about it before (https://github.com/twbs/bootstrap/issues/15935#issuecomment-76694397), however it seems to be not resolved

Was just looking in to this issue as well. Have been unable to get this working in Mobile Safari, and then realized even the GetBootstrap demo was failing as well.

@jholt1 I would suggest changing the name of this to something like "Popover dismiss on next click failing on iOS", the current title seems too generic.

Possible duplicate of #16028.

@RyanZim Indeed. Closing as duplicate.


top_mosaic_09cm_area37.png can not be found in Vaihingen/gt/ directory

mask_top_mosaic_09cm_area11.tif  mask_top_mosaic_09cm_area15.tif  mask_top_mosaic_09cm_area28.tif  ,only these file could be found.
the other file could be found in Vaihingen/image/.
when I run 'CUDA_VISIBLE_DEVICES=3 python pretrain_seg.py --model 'fcn8s' --dataID 1 --root_dir /home/zgt/projects/rs/UAE-RS-main/datasets/' in my bash, it tells me
'''
/home/zgt/projects/rs/UAE-RS-main/segmentation/pretrain_seg.py:22: DeprecationWarning: np.str is a deprecated alias for the builtin str. To silence this warning, use str by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use np.str_ here.
Deprecated in NumPy 1.20; for more details and guidance: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
name_classes = np.array(['impervious surfaces','buildings','low vegetation','trees','cars'], dtype=np.str)
Traceback (most recent call last):
File "/home/zgt/projects/rs/UAE-RS-main/segmentation/pretrain_seg.py", line 259, in
main(parser.parse_args())
File "/home/zgt/projects/rs/UAE-RS-main/segmentation/pretrain_seg.py", line 127, in main
for batch_index, src_data in enumerate(src_loader):
File "/home/zgt/anaconda3/envs/env_1/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 521, in next
data = self._next_data()
File "/home/zgt/anaconda3/envs/env_1/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 1203, in _next_data
return self._process_data(data)
File "/home/zgt/anaconda3/envs/env_1/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 1229, in _process_data
data.reraise()
File "/home/zgt/anaconda3/envs/env_1/lib/python3.9/site-packages/torch/_utils.py", line 434, in reraise
raise exception
FileNotFoundError: Caught FileNotFoundError in DataLoader worker process 0.
Original Traceback (most recent call last):
File "/home/zgt/anaconda3/envs/env_1/lib/python3.9/site-packages/torch/utils/data/_utils/worker.py", line 287, in _worker_loop
data = fetcher.fetch(index)
File "/home/zgt/anaconda3/envs/env_1/lib/python3.9/site-packages/torch/utils/data/_utils/fetch.py", line 49, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/home/zgt/anaconda3/envs/env_1/lib/python3.9/site-packages/torch/utils/data/utils/fetch.py", line 49, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "/home/zgt/projects/rs/UAE-RS-main/segmentation/dataset/seg_dataset.py", line 63, in getitem
label = Image.open(datafiles["label"])
File "/home/zgt/anaconda3/envs/env_1/lib/python3.9/site-packages/PIL/Image.py", line 3227, in open
fp = builtins.open(filename, "rb")
_FileNotFoundError: [Errno 2] No such file or directory: '/home/zgt/projects/rs/UAE-RS-main/datasets/Vaihingen/gt/top_mosaic_09cm_area37.png'
'''

Should I move the files under /home/zgt/projects/rs/UAE-RS-main/datasets/Vaihingen/image/ to /home/zgt/projects/rs/UAE-RS-main/datasets/Vaihingen/gt/? If I do this, Have some effects?

Hi, please convert the annotation files into the .png format by yourself, where the pixel values range from 0 to num_classes-1.


Add cron job to update dwolla tokens

Dwolla tokens expire. We have support right now to update them, but we don't actually have a cronjob to run the command that does it.

Done!


Don't display 'fatal: Not a git repository (or any of the parent directories): .git' if git package vendored

[with bundler 1.2.0.rc on local & production machines]
If an application's Gemfile uses git repos, such as https://github.com/gitlabhq/gitlabhq/blob/master/Gemfile, and I run bundle package --all, and then in production run bundle install --local --deployment --without development test it gives the following error:
> /var/vcap/packages/ruby/bin/bundle install --local --deployment --without development test
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
fatal: Not a git repository (or any of the parent directories): .git
Some gems seem to be missing from your vendor/cache directory.
Could not find libv8-<IP_ADDRESS> in any of the sources

have the same issue with bundler-1.3.5. @subelsky's suggestion worked for me.

i am getting problem in Git bash
yesterday i run my server and shout down my laptop and open again it's showing error
HolySpirit@HolySpirit-PC MINGW64 ~
$ cd / D:\project files\globex\globex -root directory
HolySpirit@HolySpirit-PC MINGW64 /
$ git checkout master
fatal: Not a git repository (or any of the parent directories): .git


Is it Working for human images?

Transferred to human mesh generation may require a better initial mesh and more vertices points. In theory, it is feasible, while the implementation may be designed more specific for the task of the human body.


[BUG] "Bloxstap does not support running in administrator mode. Relaunch or reinstall Bloxstrap with lower privileges."

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?

Randomly had an issue that said 'Bloxstrap does not support running in administrator mode. Relaunch or reinstall Bloxstrap with lower privileges.'
This has never showed up before. So what I did was uninstalled it and reinstalled it to see if it fixed the problem. It has not. Now I can not install it AT ALL and I think it is because I am on a Windows Administrator account. But I have no idea how that would affect the compatibility of Bloxstrap.

now?

Yeah I still cant install it

It was an oversight by Matt, a patch to remove the admin check is being pushed soon.

Alright, any idea how long that might take?

The check is being removed with v2.6.1. Please be patient, it should release soon.

can i just download old version of bloxstrap to avoid error until v2.6.1?

Yeah, I never had that issue until 2 minutes ago. I checked compatibility and it doesn't show run with administrator so I reinstalled it at least 3 times, even moving the app out of the downloads folder yet it wouldn't budge.
This bug is really annoying and I hope they patch it because it was not being run in administrator mode and I can not find a way to limit the privileges.

Yeah, I never had that issue until 2 minutes ago. I checked compatibility and it doesn't show run with administrator so I reinstalled it at least 3 times, even moving the app out of the downloads folder yet it wouldn't budge. This bug is really annoying and I hope they patch it because it was not being run in administrator mode and I can not find a way to limit the privileges.
Edit: Small edit, sometimes it works at first launch, desktop app only, but you will be signed out and its hard to trick bloxstrap to work so we can only wait.

i downgraded to v2.5.4 and it works well, just dont use checkbox on "Automatically update Bloxstrap"

Yeah, I never had that issue until 2 minutes ago. I checked compatibility and it doesn't show run with administrator so I reinstalled it at least 3 times, even moving the app out of the downloads folder yet it wouldn't budge. This bug is really annoying and I hope they patch it because it was not being run in administrator mode and I can not find a way to limit the privileges.
Edit: Small edit, sometimes it works at first launch, desktop app only, but you will be signed out and its hard to trick bloxstrap to work so we can only wait.

i downgraded to v2.5.4 and it works well, just dont use checkbox on "Automatically update Bloxstrap"

Thanks for the suggestion, this worked out thanks!

Duplicate of #2003


useScript API Reference

useScript API Reference
Description
The useScript function is designed to help developers inline scripts into a web page with minimal payload. It receives a function and its arguments as parameters and returns the stringified, minified version of the function. This is particularly useful for inlining event handlers and other scripts directly into HTML, optimizing performance by reducing the amount of JavaScript sent over the network. It integrates seamlessly with the hx-on: handlers of HTMX.
Importing
import { useScript } from "apps/utils/useScript.ts";

Parameters

fn: Function

The function to be stringified and minified.

args: ...any[]

The arguments to be passed to the function when it is called.

Return Value

string

A stringified, minified version of the function, ready to be inlined into HTML.

Usage Examples
Example 1: Inline Script with dangerouslySetInnerHTML
In this example, useScript is used to inline a simple script that logs a message when the window loads.
import { useScript } from "apps/utils/useScript.ts";

const onLoad = () => {
  console.log("Window loaded");
};

function ExampleComponent() {
  return (

Hello, World!


release on supermarket

Hi,
just discovered the #24 bug when using this cookbook. Looks like there was no release of this cookbook on chef supermarket since June, 24th - any plans to cut a new release?
best

Sorry for the delay, Christmas and New Years has a harsh effect on productivity ;)
We are wrapping up a couple of issues ( #64 - #66 ) and will have the release ready. If the issues take longer to resolve, I'll push the update at latest on Monday.
Thanks for posting!

We'll improve version updates in the future. Sorry it took so long.


Fuzzy search does not work on string with . or special characters

Hi, I've tried searching for string with say m.a.d. and it will return nothing but the list of options has a string with m.a.d.
Is there a fix to this?
Thanks

same issue, doesnt work with hyphens either


Wrong information in homework calender - 2 kids same school

The problem
I have two children from the same school.
In the homework calendar, daughter 1 shows daughter 2's homework.

Version of Home Assistant?
Core 2024.10.4 Supervisor 2024.10.3 Operating System 13.2 Frontend 20241002.4
What type of installation are you running?
Home Assistant OS
Version of WebUntis
v1.3.2
Last working WebUntis version
No response
Traceback/Error logs
No response
Additional information
No response

Do you have 2 separat logins (usernames and passwords)?
What happens if you temporary deactivate on of the configs?

No I am using the parents account. It might be possible that it is the same issue as #168. At the moment the exam calender is empty at both kids.
This week the kids have holiday, I will monitor it next week when school starts.

Although holiday you should see homework in the Calendar tab from the past 30 days

Yes you are right. I see for all the days for both kids the same homework content.
For Salome it is the right content for Eva it is the wrong content.

Ok I think the problem is a little bit different as I wrote at the beginning. If kid 1 has homework I se it also at kid 2 and reversed.

so homework from both kids is displayed in both calendars
this is probably what the webapp shows.
do you see any Information on this site that could keep the homework apart (like Klasse or Schüler)?

Yes, in the Hausaufgaben table I find both Schüler Eva and Salome.
If it would be possible to differentiate the calender by the Schüler name that would be perfect.

could you test version v1.3.3.beta
i added a student filter for the homework if you set up the integration via "Schüler"

I get a failure message in the log:
Logger: homeassistant.setup
Quelle: setup.py:269
Erstmals aufgetreten: 17:24:39 (1 Vorkommnisse)
Zuletzt protokolliert: 17:24:39
Setup failed for 'webuntis': Integration not found.

Here are some more details:
2024-10-31 17:24:37.649 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration webuntis which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-10-31 17:24:37.649 ERROR (SyncWorker_0) [homeassistant.loader] The custom integration 'webuntis' does not have a valid version key (v1.3.3.beta) in the manifest file and was blocked from loading. See https://developers.home-assistant.io/blog/2021/01/29/custom-integration-changes#versions for more details
2024-10-31 17:24:38.240 WARNING (MainThread) [homeassistant.helpers.translation] Failed to load integration for translation: Integration 'webuntis' not found.
2024-10-31 17:24:39.865 ERROR (MainThread) [homeassistant.setup] Setup failed for 'webuntis': Integration not found.
homeassistant.loader.IntegrationNotFound: Integration 'webuntis' not found.
homeassistant.loader.IntegrationNotFound: Integration 'webuntis' not found.
homeassistant.loader.IntegrationNotFound: Integration 'webuntis' not found.

try updating again

Now Webuntis is working 👍
At first glance it looks correct.
I deleted both old integrations and add the with the Option "Schüler".
Now I see only the Homework of each kid. Currently I am in holiday and I have only my iPhone available.
Next week I am back home then I will do a more detailed check.
Thank you very much 💪🏼 great work!

Hi @JonasJoKuJonas, today I tested all the homework data and all is available and correct.
You could close that issue.
Thanks

Closing the issue


Selected Dates (css-styles) not being cleared

GIF:

Not sure if this is the desired behaviour, but the <td> cells aren't having their css classes cleared when a new selection is made. If I understand the code (and bug) correctly, this only happens to dates that are selected in the <div class="CalendarMonth .. " data-visibile="false"> ... </div> wrapper.
It's almost like any cell in that data-visible="false" wrapper doesn't get any CSS classes removed when a new selection is made.
Let me know if I can help in any way 👍
Thanks for all your hard work, this lib is awesome 😄

Hmmmm, that's a pretty serious bug. I thought that I had adjusted it so that the invisible months were included in the recalculation but maybe not.
https://github.com/airbnb/react-dates/blob/master/src/components/DayPickerRangeController.jsx#L235-L243 is the code I would expect to clear that range. Obviously it's not quite doing the right thing. I can look into this EOD.

@majapw Appreciate the fast response! I'll also take a look at this and if I get lucky and solve the bug I'll make a PR.
👍

got it working @majapw  - I don't have permissions to push my branch, but here's the rundown if you're interested:
The code you linked was working properly, but its this clause in the deleteModifier() function that was being evaluated to true:
https://github.com/airbnb/react-dates/blob/master/src/components/DayPickerRangeController.jsx#L648-L650
so, it was just returning updatedDays without actually clearing any styles on the invisible months. (Note - I'm calling the 'invisible months' the ones before and after the actual visual set, since this bug only affects those).
So all I did was change the util function isDayVisible to account for those months, and its working nicely. I changed the variables fistDatOfFirstMonth and lastDayOfLastMonth to these:
let firstDayOfFirstMonth = month.clone().subtract(1, 'month').startOf('month');
...
let lastDayOfLastMonth = month.clone().add(numberOfMonths, 'months').endOf('month');

and of course had to fix some tests:
Hope this helps 👍

Awesome! Thank you! I will create a branch and push that up (and probs tag you @acr13).

Closing in favor of #575 which I just merged and am releasing in v12.2.0


[red-knot] Panics on generic type alias

Red knot currently panics when encountering PEP-695 generic type aliases:
type ListOrSet[T] = list[T] | set[T]

Backtrace:
thread '<unnamed>' panicked at crates/red_knot_python_semantic/src/types/infer.rs:872:37:
no entry found for key
stack backtrace:
[…]
   6: red_knot_python_semantic::semantic_index::SemanticIndex::definition
             at ./crates/red_knot_python_semantic/src/semantic_index.rs:210:33
   7: red_knot_python_semantic::types::infer::TypeInferenceBuilder::infer_definition
             at ./crates/red_knot_python_semantic/src/types/infer.rs:872:26
   8: red_knot_python_semantic::types::infer::TypeInferenceBuilder::infer_type_parameters
             at ./crates/red_knot_python_semantic/src/types/infer.rs:4036:50
   9: red_knot_python_semantic::types::infer::TypeInferenceBuilder::infer_type_alias_statement
             at ./crates/red_knot_python_semantic/src/types/infer.rs:1805:13
  10: red_knot_python_semantic::types::infer::TypeInferenceBuilder::infer_statement
             at ./crates/red_knot_python_semantic/src/types/infer.rs:851:53
  11: red_knot_python_semantic::types::infer::TypeInferenceBuilder::infer_body
             at ./crates/red_knot_python_semantic/src/types/infer.rs:831:13
  12: red_knot_python_semantic::types::infer::TypeInferenceBuilder::infer_module
             at ./crates/red_knot_python_semantic/src/types/infer.rs:793:9
  13: red_knot_python_semantic::types::infer::TypeInferenceBuilder::infer_region_scope
             at ./crates/red_knot_python_semantic/src/types/infer.rs:417:17
  14: red_knot_python_semantic::types::infer::TypeInferenceBuilder::infer_region
             at ./crates/red_knot_python_semantic/src/types/infer.rs:405:46
  15: red_knot_python_semantic::types::infer::TypeInferenceBuilder::finish
             at ./crates/red_knot_python_semantic/src/types/infer.rs:4044:9

Fixing this probably involves understanding type … = … statements as having their own annotation scope (https://docs.python.org/3/reference/simple_stmts.html#the-type-statement).


cmd/compile: dev.typeparams: struct with generic parameter triggers 0xb01dfacedebac1e panic on macos

Sorry, I'm not exactly sure how to categorize this because the panic happens at runtime but the issue seems to be the result of something funky that happened during compilation:

What version of Go are you using (go version)?

$ go version
go version devel go1.18-df778e6fd9 Sat Jul 17 01:52:18 2021 +0000 darwin/amd64

Does this issue reproduce with the latest release?
N/A
What operating system and processor architecture are you using (go env)?

Ran this code with go run -gcflags="-G=3" .:
package main

import (
	"errors"
	"fmt"
)

type Option[T any] struct {
	ok  bool
	val T
}

func (o Option[T]) String() string {
	if o.ok {
		return fmt.Sprintf("Some(%v)", o.val)
	}
	return "None"
}

func Some[T any](val T) Option[T] { return Option[T]{ok: true, val: val} }
func None[T any]() Option[T]      { return Option[T]{ok: false} }

type Result[T, E any] struct {
	ok  bool
	val T
	err E
}

func (r Result[T, E]) String() string {
	if r.ok {
		return fmt.Sprintf("Ok(%v)", r.val)
	}
	return fmt.Sprintf("Err(%v)", r.err)
}

func Ok[T, E any](val T) Result[T, E]  { return Result[T, E]{ok: true, val: val} }
func Err[T, E any](err E) Result[T, E] { return Result[T, E]{ok: false, err: err} }

func main() {
	a := Some[int](1)
	b := None[int]()
	fmt.Println(a, b)

	x := Ok[int, error](1)
	y := Err[int, error](errors.New("test"))
	fmt.Println(x, y)
	// fmt.Println(x)
	_, _, _, _ = a, b, x, y
}

What did you expect to see?

$ go run -gcflags="-G=3" .
Some(1) None
Ok(1) Err(test)

What did you see instead?

$ go run -gcflags="-G=3" .
Some(1) None
unexpected fault address 0xb01dfacedebac1e
fatal error: fault
[signal SIGSEGV: segmentation violation code=0x1 addr=0xb01dfacedebac1e pc=0x106eee5]
goroutine 1 [running]:
runtime.throw({0x10a2e1e, 0x401})
/Users/jmccollum/go/src/github.com/golang/go/src/runtime/panic.go:1003 +0x71 fp=0xc000072608 sp=0xc0000725d8 pc=0x102eb31
runtime.sigpanic()
/Users/jmccollum/go/src/github.com/golang/go/src/runtime/signal_unix.go:743 +0x2f6 fp=0xc000072658 sp=0xc000072608 pc=0x10425f6
reflect.(*rtype).Kind(...)
/Users/jmccollum/go/src/github.com/golang/go/src/reflect/type.go:820
reflect.unpackEface(...)
/Users/jmccollum/go/src/github.com/golang/go/src/reflect/value.go:150
reflect.Value.Elem({0x1096460, 0xc000010250, 0x726c0})
/Users/jmccollum/go/src/github.com/golang/go/src/reflect/value.go:1154 +0xc5 fp=0xc000072698 sp=0xc000072658 pc=0x106eee5
fmt.(*pp).printValue(0xc000078b60, {0x1096460, 0xc000010250, 0x113e6c0}, 0x76, 0x0)
/Users/jmccollum/go/src/github.com/golang/go/src/fmt/print.go:810 +0x50a fp=0xc000072880 sp=0xc000072698 pc=0x108764a
fmt.(*pp).printArg(0xc000078b60, {0x1096460, 0xc000010250}, 0x76)
/Users/jmccollum/go/src/github.com/golang/go/src/fmt/print.go:712 +0x74c fp=0xc000072920 sp=0xc000072880 pc=0x10870ac
fmt.(*pp).doPrintf(0xc000078b60, {0x10a30db, 0x7}, {0xc000072aa0, 0x5, 0x7600000005})
/Users/jmccollum/go/src/github.com/golang/go/src/fmt/print.go:1026 +0x288 fp=0xc000072a18 sp=0xc000072920 pc=0x10898a8
fmt.Sprintf({0x10a30db, 0x7}, {0xc000072aa0, 0x1, 0x1})
/Users/jmccollum/go/src/github.com/golang/go/src/fmt/print.go:219 +0x59 fp=0xc000072a70 sp=0xc000072a18 pc=0x1083f79
main..inst.Result[int,error].String(0x10094f4, {0xe0, 0x102d20a, {0x10c0680, 0xc000010250}})
/Users/jmccollum/go/src/local/other/main.go:112 +0xd1 fp=0xc000072ac0 sp=0xc000072a70 pc=0x108ae11
main.(*Result[int,error]).String(0xc000062040)
:1 +0x45 fp=0xc000072af8 sp=0xc000072ac0 pc=0x108b065
fmt.(*pp).handleMethods(0xc000078c30, 0x11a7108)
/Users/jmccollum/go/src/github.com/golang/go/src/fmt/print.go:626 +0x322 fp=0xc000072d48 sp=0xc000072af8 pc=0x1086422
fmt.(*pp).printArg(0xc000078c30, {0x109b580, 0xc000062040}, 0x76)
/Users/jmccollum/go/src/github.com/golang/go/src/fmt/print.go:709 +0x693 fp=0xc000072de8 sp=0xc000072d48 pc=0x1086ff3
fmt.(*pp).doPrintln(0xc000078c30, {0xc000072ef0, 0x2, 0x109b580})
/Users/jmccollum/go/src/github.com/golang/go/src/fmt/print.go:1169 +0x149 fp=0xc000072e58 sp=0xc000072de8 pc=0x108a789
fmt.Fprintln({0x10c06e0, 0xc00000e018}, {0xc000072ef0, 0x2, 0x2})
/Users/jmccollum/go/src/github.com/golang/go/src/fmt/print.go:264 +0x4f fp=0xc000072ea8 sp=0xc000072e58 pc=0x108404f
fmt.Println(...)
/Users/jmccollum/go/src/github.com/golang/go/src/fmt/print.go:274
main.main()
/Users/jmccollum/go/src/local/other/main.go:125 +0x210 fp=0xc000072f80 sp=0xc000072ea8 pc=0x108abf0
runtime.main()
/Users/jmccollum/go/src/github.com/golang/go/src/runtime/proc.go:255 +0x227 fp=0xc000072fe0 sp=0xc000072f80 pc=0x10311a7
runtime.goexit()
/Users/jmccollum/go/src/github.com/golang/go/src/runtime/asm_amd64.s:1571 +0x1 fp=0xc000072fe8 sp=0xc000072fe0 pc=0x105a321
exit status 2
Apparently related to 0xb01dfacedebac1e
Notes:

Replacing the line fmt.Println(x, y) with fmt.Println(x) avoids the above fault
Replacing the line y := Err[int, error](errors.New("test")) with y := Err[int, string]("test") avoids the fault

This seems to be related to instantiating the generic Result[T,E any] with an E of error and then trying to use reflection on that value from fmt.Sprintf("Err(%v)", <val of type E>)

This runs successfully with unified IR:
$ GOEXPERIMENT=unified go run main.go
Some(1) None
Ok(1) Err(test)

CC @randall77 @danscales @mdempsky
In general it is too early to report bugs against the dev.typeparams branch.  The branch is under very active development.  Fixes are welcome and useful, bug reports are something of a distraction.  I'll let this one stand since you already filed it.  Thanks.

Ah sorry, I made a bad assumption based on the fact that gopls has some support for type parameters. I was doing some experimentation and ran into this. I'll hold off on filing anything else related to typeparams

FWIW, I think the issue reports are still useful. Ideally if we can get them as minimized, standalone test cases (i.e., no imports). It's easy enough now to add regress tests that are known to be currently failing, and then we can fix them as we get to them.

Thanks for the issue, and sorry for the delay in triaging.  This is a good example.  Some of our dictionary code is still missing cases where type args are interfaces.  We probably think of this as not so common, but here's a good example where a type arg is 'error'.
I'm working on this, and looking for other bugs related to  type args which are interface.

Fixed by b7b790a


bundled with 0.12.x timeline?

when will electron be bundled with 0.12.x?
the documentation states that electron is still using the unstable version 0.11.13
thanks.

please label as question

Are you talking about the node version?
Electron uses io.js (a fork of node). The version used in Electron v0.31.0 is io.js v3.1.0.
This information is available in the releases page.

The incorrect documentation in question is https://github.com/atom/electron/blob/master/docs/tutorial/using-native-node-modules.md. At least that's the only reference to Node.js bundling mentioned in conjunction with version information.
And as @etiktin has mentioned, Electron is currently shipping with io.js v3.1.0.

OK, this also propagates to the web site.


linux: linux-generic: remove config PLT for relocation

Patch in meta-96boards, c0d4c09ec75f ("linux-generic: direct branch
instr modules need PLT for relocation") enables config PLT for
relocation on arm/arm64.  It is now enabled by default in the upstream
kernel from release v4.18
We shouldn't enable config PLT for relocation for arm64 just because
there may be a chance to run into the issue.  Then we can 'fix' the
issue upstream if we often hit this issue like we did for ARM.
Signed-off-by: Anders Roxell<EMAIL_ADDRESS>

@rafaeldtinoco fyi

Nothing against it BUG I don't like much the fact that we are removing this to be honest. Mainly because we are opening a windows of opportunity for it to happen again (since the branch instructions are 32 bit even for ARM64). It is not quite clear what is the performance impact of having an indirect branch for external calls in a module (http://bit.ly/2wiVL96), I really don't think is much.
Considering this is a really hard issue to debug (when it happens), we are not doing performance benchmarks, or even if we do we would not be close to what the delta of performance - by having this option enabled - would mean, but functional tests, is this change really necessary ? I would get if upstream told us it is not, but in our case of functional tests, is it ?

As discussed, this can be merged now.


Call a method when swiping horizontally to change the page

Hello, I want to call some method when user swipes horizontally to change the month. I think overriding a method viewDidScroll in JTHorizontalCalendarView.m could do that. How can it be done?

The delegate have calendarDidLoadPreviousPage and calendarDidLoadNextPage methods, if you want something more you will have to inherit from JTHorizontalCalendarView


The Economist style guide link in README is broken

Seems the economist changed the website and the existing link points to nowhere.

Thanks for pointing this out.
Unless I'm missing it, it doesn't appear that there's an online version of the guide anymore.

You could at least link to one of their books: https://www.amazon.de/Economist-Style-Guide-12th-ebook/dp/B01N9TXC27/ref=sr_1_4?ie=UTF8&qid=1513677620&sr=8-4&keywords=the+economist+style+guide
or archive.org?

I've added a link to their store: https://shop.economist.com/products/style-guide.


API?

@jthielen proposed a rough accessor API in pint/#849, to which I've added a couple of things:
DataArray:

[x] da.pint.to(...): return dataarray with converted units (#11)
[ ] da.pint.to_base_units(): return dataarray with base units
[x] da.pint.units: units of quantity (as a Unit)
[x] da.pint.magnitude: magnitude of quantity
[x] da.pint.quantify(unit_registry=None, unit=None): create DataArray wrapping a Quantity based on string unit attribute of DataArray or specified unit
[x] da.pint.dequantify(): replace data with the Quantity's magnitude, and add back string unit attribute from Quantity's unit
[ ] da.pint.sel(): wrap da.sel to handle indexing with Quantities (by casting to magnitude in the coordinate's units similar to how MetPy does it, since true unit-aware indexing is not available yet in xarray)
[ ] da.pint.loc: wrap da.loc likewise
[ ] da.pint.to_system(system): convert all variables to be expressed in the (base?) units of a different system. Might require upstream additions to pint.

Dataset:

[x] ds.pint.to(...): return Dataset with converted units (#11)
[ ] ds.pint.to_base_units(): return dataset with base units
[x] ds.pint.quantify(unit_registry=None): convert all data variables to quantities
[x] ds.pint.dequantify(): convert all data variables from quantities to magnitudes with units as an attribute
[ ] ds.pint.sel(): wrap ds.sel to handle indexing with Quantities
[ ] ds.pint.loc: wrap ds.loc likewise
[ ] ds.pint.to_system(system): convert all variables to be expressed in the (base?) units of a different system.

(this may be modified as things change on xarray's and pint's end, especially involving Dask arrays (xref #883))

Anything else?
At some point when the integration is more solidified (but before official release) we should change the accessor from pint to units, to get a interface more like what's described here.
This would be:
a) More intuitive
b) Units-library agnostic
c) A good fit for potentially using an entrypoint to choose which units library you want to use. There's already an entrypoint for plotting backends in xarray, and plans to add one for storage backends too.

One additional thing I've realized is needed since writing https://github.com/hgrecco/pint/issues/849#issuecomment-579992247 is a helper for coordinate unit conversion. Not sure what the best name for this would be though. In MetPy, the tentative name is .convert_coordinate_units, but that really only makes sense since the MetPy accessor's version of .to is .convert_units. Maybe .coodinate_to?
See https://github.com/Unidata/MetPy/pull/1325 / https://github.com/Unidata/MetPy/compare/v0.12.0...jthielen:0-12-patch-xarray-0-15-1?expand=1

we could also provide work-arounds for operations like rolling or ffill that, due the functions and libraries they use (i.e. bottleneck, numbagg, scipy, numpy.lib, etc.), won't be able to support duck arrays in the near future.

a helper for coordinate unit conversion.

So a da.pint.coords_to, that gets called within da.pint.to?
Also what about UnitRegistry().wraps? Do we need a xarray equivalent for that? Would that help with providing workarounds for rolling etc?

a helper for coordinate unit conversion.

So a da.pint.coords_to, that gets called within da.pint.to?

Not quite, it would be separate. I'm thinking of something to change the units on a coordinate of da without changing anything else on da, which needs to follow different logic given that Indexes are immutable, and, for now, not supporting units directly...so something like https://github.com/Unidata/MetPy/blob/06589346707a35e3c56699b75149bec793f9da34/src/metpy/xarray.py#L158-L162.

how about also accepting a dict / kwargs dict (like assign / assign_coords) to DataArray.pint.to and then using different branches for data and coordinates? That way, converting the data and multiple coordinates to (different) units would be possible with one function call.
Obviously, we need something really similar for Dataset.pint.to, so we could implement it by using _to_temp_dataset and to_dataarray.

how about also accepting a dict / kwargs dict (like assign / assign_coords) to DataArray.pint.to and then using different branches for data and coordinates? That way, converting the data and multiple coordinates to (different) units would be possible with one function call.

So long as we don't lose the easy ability to just convert data units (e.g., da.pint.to('degC')), that sounds great.

So as a to-do list, that would mean one positional arg for the data and multiple keyword args for the coords in da.pint.to(), then all keyword args for ds.pint.to():

[ ] da.pint.to(unit) - convert data
[ ] da.pint.to(new_unit1, coord1=new_unit2, coord2=new_unit3, ...) - convert data and multiple coords simultaneously
[ ] ds.pint.to(var1=new_unit1, var2=new_unit2, coord1=new_unit3, ...) - convert multiple data vars and coords simultaneously

Agreed for points 1 (DataArray only?) and 3 (Dataset only). For 2 there might be a few more options (nothing wrong with this one), but I think we should discuss them in a separate issue / when we implement it to keep this issue as a high level API overview.

something else that is not a part of the API but in scope for this package (I think?): maybe we could monkeypatch the repr of DataArray, Variable and Dataset (or register some sort of repr function for pint arrays (via entrypoints?)) to make it more readable (i.e. make sure the unit is always visible). See also pydata/xarray#2773

I definitely think that's in scope for this package, at least until some more general solution for reprs of all wrapped arrays is implemented upstream.
Monkey-patching actually seems like a reasonable temporary solution here I think? The worst that can happen is that some other method which uses the repr fails to display the units right

I definitely agree that some kind of repr fix is in order. The current default is...less than ideal...

from #11:

[ ] pint_xarray.testing: for testing purposes
[ ] conversion / extraction functions (from pint_xarray.conversions)

if we can get the reprs to work (pydata/xarray#2773, hgrecco/pint#1133), improve the documentation and maybe also expose the pint_xarray.testing module, we should be close enough to release a initial development version (0.1?)

the HTML repr works (pint got released today), and there are #20 (docs), #22 (inline repr) and #24 (testing), so once those are merged and the placeholder functions we have right now are temporarily removed, we should be able to release 0.1

Sounds great! Right now though it looks like the only placeholders are sel and loc. If you're good with waiting a few days, I should be able to get around to adding those in time for 0.1...I should be able to implement them the same way I did in MetPy.

I was thinking of plus_minus, to_base_units and to_system, which don't have docstrings, work only on the data or raise NotImplementedError.

from #61: try to dequantify automatically before plotting (see xarray.plot.plot.label_from_attrs)


Drag&Drop causes browser freeze

Hi,
I just started the project demo-dashboard without changes. I had the same behavior on iPhone 6, iPad (iOS 10.2.1 ) and on Android Smartphone:
Drag and drop the grid items several times make Safari and Chrome freeze. As far as I can tell this occures by dragging the item and scrolling the site at same time...to reproduce the behavior drag the top item and move tham to the botom of the view and reversed.
Thank you in advance!

+1
Same problem here.

I don't have an iOS device to test this with, but I've just tried it with the version I have online on my android phone and I don't see this issue. But it does seem to have other issues on mobile. Fantastic...


Accept the Webcodec VideoFrame interface

Hi,
This is regarding the videoframe interface.
https://w3c.github.io/webcodecs/#videoframe-interface
https://web.dev/mediastreamtrack-insertable-media-processing/
Currently faceDetection.send({image: videoElement});
Or similar javascript api does not accept video frame directly.
It would be good to have.
System information (Please provide as much relevant information as possible)
Chrome 94 on supported OS

MediaPipe Solution (you are using):
Face detection,  javascript,  chrome browser
Programming language
javascript,  browser
Are you willing to contribute it (Yes/No): Do not know how to. Willing to.

Describe the feature and the current behavior/state:
Send currently accepts canvas, image and video
Will this change the current api? How?
Yes, api will now accept videoframe
Who will benefit with this feature?
Anyone using the Mediastream insertable streams API
Please specify the use cases for this feature:
Natively support Mediastream insertable streams API

What does "video frame directly" mean to you?

Please See this hands demo, with the MediaStream Insertable Streams API,
" VideoFrame is a CanvasImageSource. A VideoFrame can be passed to any method accepting a CanvasImageSource"
it would be great to use this directly rather than have to draw it on a canvas first,
this is applicable to all javascript supported solutions
<html>
<head>
    <script<EMAIL_ADDRESS>crossorigin="anonymous"></script>
    <script<EMAIL_ADDRESS>crossorigin="anonymous"></script>
    <script<EMAIL_ADDRESS>crossorigin="anonymous"></script>
    <script<EMAIL_ADDRESS>crossorigin="anonymous"></script>

</head>
<body>

<button onClick='demostart()'>demostart</button>
<video id='video-after' autoPlay></video>

<script>
  let globalController = null;
  let timestamp = null;
  let stream = null;
  let canvasElement2 = null;
  let canvasCtx2 = null;

  const hands = new Hands({locateFile: (file) => {
    return `https://cdn.jsdelivr.net/npm/@mediapipe/hands/${file}`;
  }});

  hands.setOptions({
    maxNumHands: 2,
    modelComplexity: 1,
    minDetectionConfidence: 0.5,
    minTrackingConfidence: 0.5
  });

  function onResults (results) {
    if (!canvasCtx2) {
      return;
    }
    canvasElement2.width = results.image.width;
    canvasElement2.height = results.image.height;
    canvasCtx2.save();
    canvasCtx2.clearRect(0, 0, canvasElement2.width, canvasElement2.height);
    canvasCtx2.drawImage(
      results.image, 0, 0, canvasElement2.width, canvasElement2.height);
    if (results.multiHandLandmarks) {
      for (const landmarks of results.multiHandLandmarks) {
        drawConnectors(canvasCtx2, landmarks, HAND_CONNECTIONS,
          {color: '#00FF00', lineWidth: 5});
        drawLandmarks(canvasCtx2, landmarks, {color: '#FF0000', lineWidth: 2});
      }
    }
    canvasCtx2.restore();
    globalController.enqueue(new VideoFrame(canvasElement2, { timestamp, alpha: 'discard' }));
  }

  hands.onResults(onResults);
  hands.initialize();

  async function demostart () {
    canvasElement2 = new OffscreenCanvas(1280, 720);
    canvasCtx2 = canvasElement2.getContext('2d');
    const canvasElement = new OffscreenCanvas(1280, 720);
    const canvasCtx = canvasElement.getContext('2d');
    stream = await navigator.mediaDevices.getUserMedia({ video: { width: 1280 } });
    const videoTrack = stream.getVideoTracks()[0];
    const trackProcessor = new MediaStreamTrackProcessor({ track: videoTrack });
    const trackGenerator = new MediaStreamTrackGenerator({ kind: 'video' });

    const transformer = new TransformStream({
      async transform (videoFrame, controller) {
        const width = videoFrame.displayWidth;
        const height = videoFrame.displayHeight;
        canvasElement.width = width;
        canvasElement.height = height;
        canvasCtx.drawImage(videoFrame, 0, 0);
        timestamp = videoFrame.timestamp;
        globalController = controller;
        await hands.send({ image: canvasElement }); // using videoFrame does not work, have to draw it on to a canvas first
        // would be good to have results as a response to async .send, instead a callback
        videoFrame.close();
      },
    });

    trackProcessor.readable.pipeThrough(transformer).pipeTo(trackGenerator.writable);

    const videoAfter = document.getElementById('video-after');
    const streamAfter = new MediaStream([trackGenerator]);
    videoAfter.srcObject = streamAfter;
  }
</script>
</body>
</html>

VideoFrame may be a CanvasImageSource, but that doesn't mean it can be used as a texture source for a WebGLRenderingContext (and I believe it can't, although I haven't tried it yet myself).  Try videoFrame.createImageBitmap() instead.

Hi,
videoFrame.createImageBitmap is not a function

Hi,
As per the sample implementation here, it looks to be used in a webgl context directly.
https://github.com/webrtc/samples/blob/gh-pages/src/content/insertable-streams/video-processing/js/webgl-transform.js#L185
for Chrome 90+

That's even better then!  I didn't see that capability mentioned in the W3 specs, but if it can be used for texImage2d, then you should be able to just pass the VideoFrame through directly.  Just add and fill in a "width" and "height" field to your VideoFrame, and cast it as ImageBitmap if you need to (i.e. if TS compiler complains), and then send it directly in the .send() call.

Hi,
That worked, supplied custom (width, height) property to videoFrame.
so for chrome version <=90 use videoFrame.createImageBitmap()
and chrome 90+ use videoFrame but add width, height to the object

Is there no way we can make this solution from @tyrmullen official?

That's even better then! I didn't see that capability mentioned in the W3 specs, but if it can be used for texImage2d, then you should be able to just pass the VideoFrame through directly. Just add and fill in a "width" and "height" field to your VideoFrame, and cast it as ImageBitmap if you need to (i.e. if TS compiler complains), and then send it directly in the .send() call.

So instead of await selfieSegmentation.send({ image: { ...videoFrame, height, width } });, we have to use
videoFrame.width = videoFrame.displayWidth;
videoFrame.height = videoFrame.displayHeight;
await selfieSegmentation.send({ image: videoFrame });


Opening "Glamorous Toolkit - a tour" and clicking on the hamburger menu raises an exception

Instance of BrAbstractListModel did not understand #itemsProvider:
16 August 2022 2:38:32.658405 pm

VM: unix - x86_64 - linux-gnu - CoInterpreter * VMMaker-tonel.1 uuid: 70e0a4f8-739f-0d00-858c-38070d0dc4a6 Jul  5 2022
StackToRegisterMappingCogit * VMMaker-tonel.1 uuid: 70e0a4f8-739f-0d00-858c-38070d0dc4a6 Jul  5 2022
v9.0.13-4-ge0b19ce - Commit: e0b19ce - Date: 2022-03-30 09:53:45 +02:00

Image: Pharo10.0.0 [Build information: Pharo-10.0.0+build.521.sha.14f541319d443f4261f84f4fa19fbb34460a5edb (64 Bit)]

BrAbstractListModel(Object)>>doesNotUnderstand: #itemsProvider:
	Receiver: BrAbstractListModel {
	Selection {}
}
	Arguments and temporary variables:
		aMessage: 	itemsProvider: a BrSequenceableCollectionItemsProvider
		exception: 	Instance of BrAbstractListModel did not understand #itemsProvider:
		resumeValue: 	nil
	Receiver's instance variables:
		widget: 	BrGridList @ 386483200 (nil)
		actors: 	an Array(Selection {})
		eventDispatcher: 	Direct event dispatcher for BrAbstractListModel {
	Selection {...etc...
		model: 	nil
		whenAttachedActions: 	an OrderedCollection()
		itemTypeStencil: 	a BrListItemTypeStencilBuilder
		itemIdStencil: 	nil

BrGridList(BrUngroupedList)>>itemsProvider:
	Receiver: BrGridList @ 386483200 (nil)
	Arguments and temporary variables:
		aBrAbstractListItemsProvider: 	a BrSequenceableCollectionItemsProvider
	Receiver's instance variables:
		spaceReference: 	a BlNullSpaceReference
		parent: 	GtPresenterSlideShowPreviewElement @ 559655680 (nil)
		children: 	Children array (1)
		bounds<EMAIL_ADDRESS>corner<EMAIL_ADDRESS>		measuredBounds<EMAIL_ADDRESS>corner<EMAIL_ADDRESS>		boundsCache: 	a BlBoundsCache
		eventDispatcher: 	Bubbling event dispatcher for BrGridList @ 386483200 (nil)
		constraints: 	a BlLayoutCommonConstraints
		layout: 	a BlFrameLayout
		transformation: 	a BlElementCachedIdentityTransformation
		taskQueue: 	<<error during printing>>

BrGridList(BrUngroupedList)>>items:
	Receiver: BrGridList @ 386483200 (nil)
	Arguments and temporary variables:
		aCollectionOfValues: 	an OrderedCollection(a GtPresenterSlideViewModel a GtPrese...etc...
	 I thought I fixed that a few days ago.
See Issue 2721.

I'm sure I have an old image :-)
Thanks!


refactor: rename take children function

rename function to remove_with_children because this also removes and returns the target block itself.

🤖 Generated by Copilot at 79dea58
Renamed a method in the BlockBuffer struct and updated its usage in the BlockchainTree struct. This improves the readability and clarity of the code that handles buffered blocks in the blockchain tree.

Codecov Report

Merging #2795 (79dea58) into main (7d36dea) will decrease coverage by 25.56%.
The diff coverage is 40.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##             main    #2795       +/-   ##
===========================================
- Coverage   71.40%   45.85%   -25.56%
===========================================
  Files         505      505
  Lines       65794    65794
===========================================
- Hits        46982    30168    -16814
- Misses      18812    35626    +16814

Flag
Coverage Δ

integration-tests
17.44% <0.00%> (-0.04%)
:arrow_down:

unit-tests
39.18% <40.00%> (-27.14%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

crates/blockchain-tree/src/block_buffer.rs
50.16% <25.00%> (-48.15%)
:arrow_down:

crates/blockchain-tree/src/blockchain_tree.rs
21.94% <100.00%> (-62.17%)
:arrow_down:

... and 244 files with indirect coverage changes


Fix typo

@googlebot I signed it!

Sorry, this repo is just a mirror of the Skia repo. Please see https://skia.org/dev/contrib/submit for how to submit patches to Skia.
Thanks,
-joe


Error NamedStructField should be rewritten in OperatorToFunction with subquery if query is wrapped in view

Describe the bug
When selecting from a view that selects fields from structs, this error is thrown:
Internal error: NamedStructField should be rewritten in OperatorToFunction.

To Reproduce
In datafusion 38.0.0 cli:
> create or replace view v as select struct(1, 'b')['c1'];
0 row(s) fetched.
Elapsed 0.001 seconds.

> select * from v;
Internal error: NamedStructField should be rewritten in OperatorToFunction.
This was likely caused by a bug in DataFusion's code and we would welcome that you file an bug report in our issue tracker

Expected behavior
+--------------------------------+
| struct(Int64(1),Utf8("b"))[c1] |
+--------------------------------+
| b                              |
+--------------------------------+
1 row(s) fetched.

Additional context
It errors the same when a subquery is viewed: create view v as select (select struct(1, 'b')['c1']);.
This error appeared once as a regression in 37.0: https://github.com/apache/datafusion/issues/10029

It appears to be successful on the current main branch.
DataFusion CLI v38.0.0
> create or replace view v as select struct(1, 'b')['c1'];
0 row(s) fetched.
Elapsed 0.005 seconds.

> select * from v;
+--------------------------------+
| struct(Int64(1),Utf8("b"))[c1] |
+--------------------------------+
| b                              |
+--------------------------------+
1 row(s) fetched.
Elapsed 0.006 seconds.

Also hit a similar error in a different scenario and can confirm that master version fixed it for me.

Can confirm that it's gone for our more complicated use case in 8fcb3e4b, thus also probably in 0.39.
Was or should it be covered in a test? Otherwise feel free to close.


vpr is so slow with large circuit

I am implementing a large circuit on OpenFPGA and when in VPR step, it tooks a lot of time (particularly 222590 seconds) to run. In vtr documents, I see that there is a option to optimize my circuit but it is still experimental (--router_algorithm parallel). I want to ask that how long is this option completed? Or could you give me some advices for me to improve my circuits?
Thanks,

There are some options which you wrote in your documents but it is still not good enough. I have read your c++ code but it is difficult to change your routing algorithm. Please, can you give me some advice or some hints for this issue?

please comment if you have any concern
@tangxifan

@ducminhnguyen123
VPR has been tested on many benchmarks where number of LUTs is up to 1M. See details in the VTR8 research paper.
https://www.eecg.utoronto.ca/~kmurray/vtr/vtr8_trets.pdf
You may find details in the Table 28 (See a screenshot below). It can accomplish in 1 day even for the largest. Most of them can be finished in 2 hours. The QoR is continuously monitored on VTR master branch.

So, if you are experiencing such issue, it mostly is due to some wrong settings. My suggestion is to share your log file or architecture XML. It helps to find out why.

thanks a lot with your answer, can you tell me how many luts per clb you config, pls?

@ducminhnguyen123 try nextpnr


Created messaging subscribers for Institute contact events

For ticket: https://eccbc.atlassian.net/browse/GRAD2-2644

Hi @cditcher , please address the maintainability issues in the code.

There remains one issue but with the inheritance chain, changing to contstructor injection forces child classes to implement same. It will have to remain field injection.


feat: build autoqasm program directly from main decorator

Issue #, if available:
#805, #806, #813
Description of changes:

Update aq.main to build AutoQASM program directly.
Add support for typed input parameters and auto-capture of inputs.

usable in qubit indices

Various typing simplifications

Wrapped testing programs in fixtures so they don't get built during other tests.
Testing done:
Tests updated and passing everywhere except test_nested_function and test_main_from_main.
A few ambiguities are called out with todos, which should be examined for how we want to handle the behavior with the new interface.
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[x] I have read the CONTRIBUTING doc
[x] I used the commit message format described in CONTRIBUTING
[x] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[x] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[x] I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Two tests failing in test_api: test_nested_function and test_main_from_main

main from main issue root caused and captured in #809 to be fixed in a follow-up PR

Updated all tests and incorporated all currently available feedback. Top of mind remaining to get this merged:

test_transpiler.py could use review, particularly test_partial_function and test_classmethod.
example notebooks need to be updated

Will the other example notebooks need to be updated?
If so, can you keep a running TODO list in the PR description to help us stay in sync with you?

Yes, I highlighted the remaining work in the comment above, but I agree the description may be easier to reference. I expect all the examples will need updates, I've only gone through the first two yet though

@ajberdy what's the current status of this PR... seems like it's getting close? If you could please just ping @laurencap and me when all of the TODOs/comments addressed (or have issues opened) and you are ready for a final PR review pass.

@ajberdy what's the current status of this PR... seems like it's getting close? If you could please just ping @laurencap and me when all of the TODOs/comments have been addressed (or have issues opened) and you are ready for a final PR review pass.

@rmshaffer Yes, looks like it's getting close! There are 3 TODOs on this PR, all tagged with issues and to be fixed in later PRs (#809, #814, #816). I believe @laurencap has a couple example notebooks left to review and possibly some remaining test files.
I've responded and acted on all the comments, so there aren't currently any action items for the PR on my end. Once all the files are reviewed, I'd say we're good for a final pass.


Mounting Encoder for iPower Motor GM4108H-120T

Hi, I bought the iPower Motor GM4108H-120T and the encoder AMT103 CUI you used, I found there is no way to mount the encoder to the motor. Any advices?

I just bought the AS5048A Magnetic Encoder for the motor


Bump version of lib/sql submodule to 0.37.1

I think this didn't cause trouble because there were changes in CI recently
cc/ @alberhander

Example of failures in CI tests:
Step #9:   2236) Admin::OrganizationUsersController security #create fails if password is not strong
Step #9:      Failure/Error: Unable to find matching line from backtrace
Step #9:      Sequel::DatabaseError:
Step #9:        PG::InvalidParameterValue: ERROR:  extension "cartodb" has no installation script nor update path for version "0.37.1"
Step #9:        CONTEXT:  SQL statement "CREATE EXTENSION cartodb VERSION '0.37.1' CASCADE"
Step #9:        PL/pgSQL function inline_code_block line 10 at SQL statement
Step #9:      # ./app/models/user/db_service.rb:674:in `block (2 levels) in upgrade_cartodb_postgres_extension'
Step #9:      # ./app/models/user/db_service.rb:668:in `block in upgrade_cartodb_postgres_extension'
Step #9:      # ./app/models/user.rb:594:in `in_database'
Step #9:      # ./app/models/user/db_service.rb:667:in `upgrade_cartodb_postgres_extension'
Step #9:      # ./app/models/user/db_service.rb:208:in `load_cartodb_functions'
Step #9:      # ./app/models/user/db_service.rb:106:in `new_non_organization_user_main_db_setup'
Step #9:      # ./app/models/user.rb:1242:in `setup_user'
Step #9:      # ./app/models/user.rb:295:in `after_create'
Step #9:      # ./spec/support/factories/users.rb:100:in `create_user'
Step #9:      # ./spec/support/factories/users.rb:137:in `create_test_user'
Step #9:      # ./spec/support/factories/users.rb:127:in `create_owner'
Step #9:      # ./spec/factories/organizations_contexts.rb:61:in `block (2 levels) in <top (required)>'

Example of failures in CI tests:
Step #9:   2236) Admin::OrganizationUsersController security #create fails if password is not strong
Step #9:      Failure/Error: Unable to find matching line from backtrace
Step #9:      Sequel::DatabaseError:
Step #9:        PG::InvalidParameterValue: ERROR:  extension "cartodb" has no installation script nor update path for version "0.37.1"
Step #9:        CONTEXT:  SQL statement "CREATE EXTENSION cartodb VERSION '0.37.1' CASCADE"
Step #9:        PL/pgSQL function inline_code_block line 10 at SQL statement
Step #9:      # ./app/models/user/db_service.rb:674:in `block (2 levels) in upgrade_cartodb_postgres_extension'
Step #9:      # ./app/models/user/db_service.rb:668:in `block in upgrade_cartodb_postgres_extension'
Step #9:      # ./app/models/user.rb:594:in `in_database'
Step #9:      # ./app/models/user/db_service.rb:667:in `upgrade_cartodb_postgres_extension'
Step #9:      # ./app/models/user/db_service.rb:208:in `load_cartodb_functions'
Step #9:      # ./app/models/user/db_service.rb:106:in `new_non_organization_user_main_db_setup'
Step #9:      # ./app/models/user.rb:1242:in `setup_user'
Step #9:      # ./app/models/user.rb:295:in `after_create'
Step #9:      # ./spec/support/factories/users.rb:100:in `create_user'
Step #9:      # ./spec/support/factories/users.rb:137:in `create_test_user'
Step #9:      # ./spec/support/factories/users.rb:127:in `create_owner'
Step #9:      # ./spec/factories/organizations_contexts.rb:61:in `block (2 levels) in <top (required)>'


Pretrained Weights

Are there any pre-trained checkpoints available?

Are there any pre-trained checkpoints available?

@sunyi123 Hi! We've released the checkpoints for MIT-Adobe 5K and HDR+ for your convenience. You can find them here: https://github.com/CXH-Research/FilmNet/releases/tag/weights.


Fix displaying false vsphere errors

Summary
Fixes #10460

Technical notes summary

hostsystem and folder can be null or "", this PR adds this into consideration before displaying error banners.
Areas or cases that should be tested

Provisioning vSphere Clusters
Screenshot/Video

https://github.com/rancher/dashboard/assets/135728925/de64f2e2-058c-46f8-9379-3de0aceb94a6

/backport v2.8.next1 release-2.8


Some suggestions

Hi Bea
I really like the library, I've been having a look at the code and I had a few requests that I've put in this pull request.

First, for me I needed to convert the filetype str into a pypdb PDBFileType enum object.
Second, it would be convenient to return the filename along with the file data from the fetcher
Third, for some PDB entries there isn't a PDB file and only a cif file. Perhaps it would be possible to check what file is present and then download the appropriate one? I haven't implemented this but if it's not possible to get that information then I could just look over PDB and CIF files as follows:

import profet

def get_id(pdb):

    fetcher = profet.Fetcher("pdb")

    filedata = None
    filename = None
    for filetype in [ "pdb", "cif" ]:
        filename, filedata = fetcher.get_file(pdb, filetype=filetype)
        if filedata is not None:
            break
    if filename is not None and filedata is not None:
        with open(filename, "w") as outfile:
            outfile.write(filedata)
    else:
        raise RuntimeError("No PDB or CIF file found")

if __name__ == '__main__':

    get_id("4v1w") # lower case
    get_id("4V1W") # upper case
    get_id("4V5D") # no pdb file (only cif)
    get_id("Q8W3K0") # From alphafold

Hi James! Thank you so much for this!
I addressed the filename return, which should happen on the fetcher level and not on the pdb, so it should work now for both alphafold and pdb :)
I tried implementing the cif on the pdb side (as alphafold always has both the pdb and the cif), so it also should work ok.
I didn't encounter the filetype problem, but I'll see if it still happens with the new pypdb version, and will let you know!

Hi @mooniean
Thanks for updating this! I pulled the repo and tried to test it but for some reason I've been having an issue with the Fetcher constructor hanging. I don't think the issue is a problem with your code but the alphafold id link seems to hang forever.
If I run the following code
import profet
print("Get fetcher")
fetcher = profet.Fetcher("pdb")
print("Get PDB")
result = fetcher.get_file("4v1w")
print(result)

It hangs after printing "Get fetcher" and if I interrupt the process, it seems to be hanging here:
^CTraceback (most recent call last):
  File "/home/upc86896/Software/profet/jmp_test/test_get.py", line 3, in <module>
    fetcher = profet.Fetcher("pdb")
  File "/home/upc86896/Software/env/dev/lib/python3.9/site-packages/profet/profet.py", line 9, in __init__
    self.alpha = Alphafold_DB()
  File "/home/upc86896/Software/env/dev/lib/python3.9/site-packages/profet/alphafold.py", line 17, in __init__
    self.df = pd.read_csv("http://ftp.ebi.ac.uk/pub/databases/alphafold/accession_ids.txt",

KeyboardInterrupt

I tried to do "wget http://ftp.ebi.ac.uk/pub/databases/alphafold/accession_ids.txt" and this also seems to hang so I think it is not your fault but maybe something to do with this link!
I thought that maybe it was something to do with my internet connection so I created a github workflow for profet which builds the package and runs the tests here (https://github.com/jmp1985/profet/pull/1) If you like I can submit this as a pull request to your repo. Currently the workflow seems to be hanging during running pytest so I think the issue is not just on my computer. Does running the tests work for you?

I think there must be an issue on their side. I tried again today and I still have the same issue so it may be worth sending them an email to let them know that their side is having issues!


ARROW-17359: [Go][FlightSQL] Create Example with SQLite in-mem and use to test FlightSQL server

It also looks like the integration tests have been failing due to JS issues, we should probably dig into that

@lidavidm I was considering looking into the JS integration failure, I'm just not particularly familiar with the arrow JS library. Though it looks like it's a gulp issue on build. I'll take a brief look at it and see if i can discover anything obvious.

@lidavidm I was considering looking into the JS integration failure, I'm just not particularly familiar with the arrow JS library. Though it looks like it's a gulp issue on build. I'll take a brief look at it and see if i can discover anything obvious.

See ARROW-17410
@domoritz do you have any idea what's going on? The JIRA links some related issues but I'm not quite sure what the actual resolution is (do we just need to upgrade a package?)

@lidavidm @domoritz  It looks like the remedy is to downgrade zlib to v1.2.11 until the fix in the linked issue gets released. I'll try testing it locally


after unmount the app, con't mount

Version
3.0.0-beta.14
Reproduction link
https://github.com/doter1995/vue3-test.git
Steps to reproduce
app.mount('#main');
app.unmount('#main');
app.mount('#main');

[Vue warn]: App has already been mounted. Create a new app instance instead
What is expected?
mount the app
What is actually happening?
con't mount app

i find the unmount not reset  isMounted=false, maybe need reset isMounted=false at unmount
unmount() {
        if (isMounted) {
          render(null, app._container)
        } else if (__DEV__) {
          warn(`Cannot unmount an app that is not mounted.`)
        }
      },

fix the bug at pull #1296


Adding copy and move constructor for tensor

In my understanding, class tensor does not has an explicitly written copy constructor. This leads to the use of the default copy constructor for itself and its elements. The default copy constructor for a std::vector<> copies all the elements inside the vector. As std::vector<float> data in tensor exists, a new vector is created and copied with each copy constructor use.
The repository code uses the copy constructor with tensor at many places (with direct uses such as Tensor x = y). This leads to unintentional data copy.
The same applies to move constructor.

:octocat: cibot: Thank you for posting issue #185. The person in charge will reply soon.

Right. I agreed.

I am gonna be making copy assignment operator and move assignment operator

I am gonna be making copy assignment operator and move assignment operator

If we have shared_ptr, IMO we do not need to write the copy and move assignment operator again.
@zhoonit close this once shared_ptr PR merges.

I am gonna be making copy assignment operator and move assignment operator

If we have shared_ptr, IMO we do not need to write the copy and move assignment operator again.
@zhoonit close this once shared_ptr PR merges.

I think we need a consensus on this topic.
You know in python family tensor library, shared_ptr kind of implementation is used. for example in numpy:
>> import numpy as np
>>
>> a = np.arange(5)
>> b = a
>> b[2] = -1
>> print(a)
array([ 0,  1, -1,  3,  4])

If you want to copy, you need to explicitly call copy like...
>> import numpy as np
>>
>> a = np.arange(5)
>> b = a.copy()
>> b[2] = -1
>> print(a)
array([ 0,  1, 2,  3,  4])

However, in c++ we have reference operator and move assignment operator in c++ which can do such job
for example
>> Tensor a = {1, 2, 3, 4, 5} // just an illustration.
>> Tensor &b = a;
>> b.getData()[2] = -1

>> std::cout << a;
1, 2, -1, 4, 5

or if you need to change the scope of a certain Tensor....
>> Tensor a = {1, 2, 3, 4, 5}
>> Tensor b = std::move(a);
>> b.getData()[2] = -1;

Which python can't do.
So I think it is more natural to use copy assignment operator to copy the inner data structure as well (or use flag sth like copy-on-write?)
Another concern is that changing the semantics of copy operator as noted

So I think it is more natural to use copy assignment operator to copy the inner data structure as well. (would be even better if we use flag sth like copy-on-write and check before *_i() operation happens?)

IMO, using these semantics is confusing as the major constituent of the tensor is a shared_ptr. I think the copy/move constructor/assignment operators should follow the semantics of the shared_ptr itself.
Further, if we are going to create a copy of the internal data structure with copy assigment, then IMO, there is no major advantage with using shared_ptr compared with directly vector.

Another concern is that changing the semantics of copy to not copy inner data changes the interface. Then we need to check every code other than tensor as @kparichay noted #235.

Yes, it will change the interface. However, this removes all the unnecessary memory copies in the code.
I think @jijoongmoon @myungjoo @wooksong @dongju-chae can provide some comments on which way is more appropriate.

After the discussion with @wooksong, let's convert std::vector<T> to std::shared_ptr<T *>, where the memory allocation will be done manually for the T *. tensor.cpp does not heavily rely on std::vector<> functionalities, but the overhead of vector and memory management of std::vector<> creates issues for expansion of tensor for other backends and merging with nnfw.

I guess this can now be closed if @kparichay  is okay.
We need to fix the copy constructor but not directly related to this issue.

Let's close this and make a new issue for fix memcopy in copy constructor


Remove unconfimed email user accounts

Hey there,
This pull request is essentially here to setup a documentation workflow base on jsDoc, and also to setup the Database Cleanup System ( issue #109 ).
Please go and check it out, and let me know your thoughts
Thanks in advance
Massiles.

This pull request is the version 1.0.0 for the Database Cleanup System #109. Please refer to the git commits and the code for more info.
Typos in git commits : thought not tought
Thanks.
Massiles.

Hey there !
I've just removed the setInterval and replaced it with the npm/@node-sechduler job scheduler.
I set the schedule so as the job executes on the first day of every month at 00:00 .
Check out the git commit & code for more info.
Thanks.
Massiles.

Hey there @supernova350,
Thanks for the great feedback. I really appreciate it.
After some research, I found that even this solutions does not solves the problem that you've pointed to. The problem is that our scheduling is an in-process scheduling, i.e. these job are scheduled but will only fire as long as our script is running.
How can we fix that ?
-Well, what we can do is to set our cron job not based on the script state, but on the infrastructural state. This means that we will use the server cron deamon. But, this will only work if the server is using linux and that we have only one server. Otherwise, it will be very difficult to set this up.
-Another way around, is to use a database/store, to have a persistent scheduling, like npm/@agenda with mongoDB or npm/@bull with redis. With that, we can remember that last time the job is executed.
-Also, what we can do is to create a row or a table in our database to store related information about our job, and recreate that behavior by ourselves.
Is it worth it ?
Well, with the first solution definitely not !. The reason is that behavior will make us dependent on our infrastructure, and we do not know it, since we do not own the server in which our code is running.
Some disadvantages are shown here, source on this link

So the remaining way to go is the second and the third, I'am joining them since they are very similar, the whole point is to set a database/store to memorize the last time our job executed.
My thoughts on the problem
I really understand the problem, but I do not think that this is big enough to go and spend development time creating a solution for it. The reason is, even if we redeploy and the scheduling restarts, the worst thing that will happen it that some users can still access our platform for one more month before we delete their account from our database.
This is my thoughts on the current problem, this does not mean that we do not need to solve it, but this means that it is not urgent.
Ideas from this
-We need to build a system that will remind the user to confirm their email address before deleting their account on the database.
Please let me know your thoughts on this, and if something is wrong.
I look forward to hearing from you.
Thanks in advance.
Massiles.

What you said is correct, but is too much for a project of this scale. The simpler the better.
Let's focus on making the project go live before thinking about the size of the database. If it is extremely important to you, I'd recommend storing a created_at timestamp for each user account and use that in an sql script to manually purge inactive accounts (ex. if current - created > 30d, purge).

hey @supernova350,
Thanks for your great feedback as always.
Yeah, I totally agree with you, and I want to say that that's is what I am currently developing, in other words this pull request is just to set an alpha version of the system. The current version is basically fetching the users with email_confirmed set to false and purge them.
The next version, that I am currently developing, will try to use sequelize to retrieve the age of each row in our user table. Which in fact is the same thing as what you said. I just need to search a bit, since I am an express/mongodb developer and I never dealt with sequelize before XD.
This is my current plan, please tell me if something is wrong.
Thanks in advance.
Massiles.

Hey @supernova350 ,
Guess what, I've just finished it and the trick was really simple XD. Please let me know what you think about it.
Thanks in advance.
Massiles.

Otherwise, looks good! The cron job will do its work, but if the process is not running, it will not execute the method. It may help to make the cron interval 1d, so there's a better chance of the method being called.

hey @supernova350,
About the cron interval we made it more then 20 days for better UX. In other words, if a user did not confirmed his email address within the first 30 days, we can be sure at 99% that he wont do it after that. But, we can not affirm that after just 1 day.
Also, it is more of a convention to do it like that.
Hope that is clear.
Massiles.

The cron interval does not really matter, since we're checking if the account is unconfirmed AND older than 30 days. Theoretically, we could have an async while(true) and check unconfirmed accounts there. However, that would be inefficient. The interval is just to reduce load on the server.

Exactly, the cron is just to avoid checking for unwanted users at every second. Which is really inefficient. Also the cron schedule can be really helpful since it is a way of planning events and job, we can use it to create a chronological sequence of those. For example, we can use it to create the system that I've suggested, which will remind the user to confirm his/her email address.
Massiles.

About the cron interval we made it more then 20 days for better UX.
So, this doesn't really make sense. No problem though!

Well, yes it does !
We do not want to remove user accounts everyday as you suggested.
Imagine that we do that and a user registers today, if he do not confim his email address today, then his account will be purged  tomorrow. Which will resault in a worse UX.
Hope that is clear.
Massiles

About the cron interval we made it more then 20 days for better UX.

So, this doesn't really make sense. No problem though!

Yeah, I see what you mean.
I should of said the unconfirmed_user_account_max_age instead of cron.
The cron is used to lower the server load, and the  unconfirmed_user_account_max_age= 30 days   is for better UX.
Hope that is clear.
Massiles.

That’s what I was trying to say. My bad if I was unclear.

Well I think with that been said, we can agree that this first version of the system is on point.
Aren't you?
Massiles

I agree. It should be ready.

Looks good- Thanks @MassiGy @supernova350!

Thanks  for the great feedback as always.
Massiles.


kernel: irq: Add support for dynamic direct IRQs

The kernel has support for installing ISRs dynamically at runtime with irq_connect_dynamic.
Installation of dynamic direct interrupts is currently unsupported.
Add support to declare direct IRQs as dynamic.
This would allow direct interrupts to be installed dynamically at runtime.
For example two radio protocols can use the RADIO peripheral as long
as the protocols are correctly initialized and uninitialized.
These interrupts would have the same restrictions as normal direct
interrupts except:

Small overhead compared to non-dynamic.
Can be given an argument.

Example:
IRQ_DIRECT_DYNAMIC(RADIO_IRQn, CONFIG_BT_CTLR_WORKER_PRIO,
		   IRQ_CONNECT_FLAGS);

irq_connect_dynamic(RADIO_IRQn, CONFIG_BT_CTLR_WORKER_PRIO,
                    radio_nrf5_isr, NULL, 0);

irq_enable(RADIO_IRQn);

Signed-off-by: Joakim Andersson<EMAIL_ADDRESS>

Only added z_arch_irq_line_get for ARM architecture for now. I need help in adding it for other architectures.

Add support to declare direct IRQs as dynamic. This would allow to have
low latency interrupts be shared between different users.

Not sure I undertand how A leads to B here.
What about this requires runtime dynamic registration?

Add support to declare direct IRQs as dynamic. This would allow to have
low latency interrupts be shared between different users.

Not sure I undertand how A leads to B here.
What about this requires runtime dynamic registration?

Maybe I did not explain myself properly. I updated the description to try to be more clear.
I want to use the kernels support for dynamic interrupts to disable the Bluetooth driver and release all its occupied resources, this includes the interrupt. Then I can allow a different protocol to use the same HW peripheral as the Bluetooth driver.
This works except for the RADIO ISR which is declared as direct, which does not support dynamic installation.
While I could go through this function: https://github.com/zephyrproject-rtos/zephyr/blob/master/subsys/bluetooth/controller/ll_sw/nordic/hal/nrf5/radio/radio.c#L67
to acheive what I want I prefer to have this ISR multiplexing done through the kernel instead.
It seems like a bad architecture to me to have to include internal bluetooth header.
And instead of creating a new layer for the direct ISR I'd prefer the kernel to provide me this functionality.

The goal of direct interrupts, when we added them to Zephyr, is that they were ISRs that were programmed directly in the vector table/IDT/whatever and jumped to directly by the CPU.
Here we have something that relies on the indirection of the sw_isr_table, at this point is there really that much more overhead with the regular IRQ mechanism?

The overhead of the regular IRQ mechanism might not necessarily have a known Worst case execution time. And I'm in need to know this on order to verify that my interrupts are serviced in time. Maybe I have misunderstood the original intent of what direct IRQs are. My assumption is that they are there to have ISR that does not invoke kernel overhead so that we can make assumption about WCET.

Have you considered making the vector table read/write and installing in there at runtime?

This is basically what I'm trying to achieve, but I cannot modify the vector table because it is located in flash, so I need to modify something that is in RAM. To me it seemed quite convenient to re-use the SW ISR table, and just skip the kernel overhead.

To be clear: I am not a fan of this patch.
The goal of direct interrupts, when we added them to Zephyr, is that they were ISRs that were programmed directly in the vector table/IDT/whatever and jumped to directly by the CPU.
Here we have something that relies on the indirection of the sw_isr_table, at this point is there really that much more overhead with the regular IRQ mechanism?

@andrewboie
Radio ISRs on a hard real-time controller have extremely tight timing requirements. We really do not want to use ZLI, but regular interrupts with the pre and post ISR overhead is simply not an option. This allows a hard real-time ISR to be shared sequentially by two different radio protocols with minimum overhead (1 pointer indirection).

@andrewboie
Radio ISRs on a hard real-time controller have extremely tight timing requirements. We really do not want to use ZLI, but regular interrupts with the pre and post ISR overhead is simply not an option. This allows a hard real-time ISR to be shared sequentially by two different radio protocols with minimum overhead (1 pointer indirection).

Given how obscure this use-case is (having low-latency interrupts shared by completely different drivers which don't talk to each other and are loaded/unloaded at runtime), can't you install a direct interrupt at build time and then demux in the ISR? Seems like the overhead would be about the same.
void my_radio_isr(void) __attribute__("interrupt")
{
	ISR_DIRECT_HEADER()
	if (driver1_in_use != 0) {
		driver1_isr();
	} else {
		driver2_isr();
	}
	ISR_DIRECT_FOOTER();
}

Given how obscure this use-case is (having low-latency interrupts shared by completely different drivers which don't talk to each other and are loaded/unloaded at runtime), can't you install a direct interrupt at build time and then demux in the ISR?

Not if one of the users of the ISR is downstream and not upstream. This would force us to carry a patch forever with the extra change to point to the downstream subsystem that needs the ISR. Also, this would be pretty ugly because it doesn't allow us to arbitrarily switch to any subsystem without hardcoding its ISR handler in the "common" ISR. It can be done, but it's not pretty.

Given how obscure this use-case is (having low-latency interrupts shared by completely different drivers which don't talk to each other and are loaded/unloaded at runtime), can't you install a direct interrupt at build time and then demux in the ISR? Seems like the overhead would be about the same.

void my_radio_isr(void) __attribute__("interrupt")
{
	ISR_DIRECT_HEADER()
	if (driver1_in_use != 0) {
		driver1_isr();
	} else {
		driver2_isr();
	}
	ISR_DIRECT_FOOTER();
}

@andrewboie
That is exactly what I'm doing. I'm installing a direct interupt and then
demuxing the ISR. The problem is that I wish to use this for the Bluetooth driver
that is part of zephyr. Starting from your example, as @carlescufi explained
hardcoding the ISRs in the common ISR would be very inflexible. So instead it would be better to implement something like how the isr_radio is implemented:
static radio_isr_cb_t isr_cb;
static void           *isr_cb_param;

void isr_radio(void)
{
	isr_cb(isr_cb_param);
}

void radio_isr_set(radio_isr_cb_t cb, void *param)
{
	irq_disable(RADIO_IRQn);

	isr_cb_param = param;
	isr_cb = cb;

	irq_enable(RADIO_IRQn);
}

But we would need to expose the radio_isr_set for the application to use.
And also it would be nice to have a solution that would work for any ISR, not
just the radio ISR.
Which is why I thought it would be sensible to re-use the sw-table. That gives
generic ISR demuxing. It would also provide us the register function, since it
already exists (irq_connect_dynamic).
I found it quite easy to support this for the 2 architectures that use the SW
ISR table, ARC and ARM. And for native_posix it seems it also works without
much problems. X86 however seems to present problems. But from your comments I guess
you wouldn't miss the support for X86.
I would therefore suggest to continue with this PR, but limit it in number of
architectures that support it, looks like ARC ARM and Native_posix (+nrf52_bsim),
but we could also limit it to just ARM.
If you still disagree with that at least I would like to move it to an NRF SOC
specific part of the zephyr kernel so that it would be available in the upstream
for use in the Bluetooth driver.
Please advise.

Starting from your example, as @carlescufi explained
hardcoding the ISRs in the common ISR would be very inflexible

Where do you see this mechanism getting re-used?
My point is that the whole reason why you are having to implement this are due to some very unusual combination of constraints which are specific to your project.
So just declare the radio ISR in the BT stack __weak and do the rest in your application.

Starting from your example, as @carlescufi explained
hardcoding the ISRs in the common ISR would be very inflexible

Where do you see this mechanism getting re-used?
My point is that the whole reason why you are having to implement this are due to some very unusual combination of constraints which are specific to your project.
So just declare the radio ISR in the BT stack __weak and do the rest in your application.

I will be re-using it, which is why I made this PR to begin with.
We have had support for this use-case in our SDK for many year, and people do use it which is another reason for why I want it.
I did however reduce the amount of infrastructure. It is much simpler now. So please have another look.
As I already stated I'm fine with reducing the number of supported architectures, but as far as I can tell the only architecture with any issue is x86. All implementations of irq_connect_dynamic support re-assignment except for x86 as well. So having that limitation for only x86 seems acceptable to me.
Also I don't see why this feature is not welcome. We do support dynamic interrupts, we do support direct interrupts. Why not dynamic direct interrupts?
Maybe it is a difference of perspective but from my perspective this seems like a perfectly reasonable feature to have in an RTOS designed for microcontrollers.

@andrewboie

If you still disagree with that at least I would like to move it to an NRF SOC
specific part of the zephyr kernel so that it would be available in the upstream
for use in the Bluetooth driver.

You didn't comment to this suggestion. But this is my preferred alternative I have created an RFC PR here: #20387
I can close this PR and proceed with that instead.


Allow running towncrier API without its own ini file

My project setup is a bit more complicated and the project is part of a namespace package ... so the whole part in which towncrier tries to be helpful (and opinionated) and auto-detect the settings just gets in the way of running towncrier for my project.
It would be great to have an API or entry_point which can be called with all input data already computed
Thanks!
To have it reach the newsfragments dir I had to put
[towncrier]
; Your project name
package = chevah/server

After butchering the current towncrier so that I can use it for my project I ended up with this entry point
@task
@needs('update_setup')
def release_notes():
    """
    Update the release notes.
    """
    from pkg_resources import load_entry_point
    sys.argv = [
        'towncrier',
        '--yes',
        '--version=%s' % (SETUP['product']['version'],),
        '--package=chevah.server',
        '--filename=chevah/server/static/documentation/release-notes.rst',
        ]
    return sys.exit(
        load_entry_point('towncrier', 'console_scripts', 'towncrier')()
        )

with so many option, is hard to run it without a config file... so I think that this is no longer valid


Postolache update

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
0026081324e14da0ec174a485f65bd2eaef17f62

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/6637abf757e9d50008c0ff8d


[WIP] Http response thread pool

Change Description

Thanks go to @bspark8 #6678 for testing and verifying the improvement of processing/sending http request on a different thread.
This PR adds a thread pool to the http_plugin used for processing/sending http requests.
Remove auto load of http_plugin in nodeos main.cpp since not all nodeos need the http_plugin and the extra threads it spawns.
Application logic is still processed on the application thread.
See #6678 for additional info

Consensus Changes
None
API Changes
None
Documentation Additions

http_plugin has new option:
--http-threads arg (=2)               Number of worker threads in http thread pool

I missed that you are passing the new io_context to init. Thanks for pointing that out.

@bspark8 I've added a new io_context for the websocketpp server as you suggested.

@heifner Thank you for additional threading for the part that receives the HTTP requests.
In addition, I did the same test that I did in the previous PR(#6678) in the current patch. Below is the result.
Current patch

About 1137 TPS (22% performance improvement)
Each thread CPU usage
main: 88%
receiving HTTP requests: 52%
responding to the HTTP requests: 14%


CWD goes back 1 directory when opening a file from Most Recent files

Describe the bug
I have the following code structure:
   src/
     - child 1
         + file 1
         + file 2
     - child 2
     - child 3

I enter "child 1" directory, enter nvim and access file 1 from MRU in dashboard. When I check the CWD (using :lua print(vim.fn.getcwd()), it points to the src directory
To Reproduce
Steps to reproduce the behavior:

Go to a subdirectory
Access a file in that subdirectory from MRU list on dashboard

Expected behavior
When printing out CWD, it should point to "child 1" directory.
Screenshots

I believe that it has something to do with typescript, as the bug doesn't occur with my other directories.

because it find the root is src folder.

No, if you look at the screenshots, you'll notice that I'm in folder /mnt/HDD/umass/umass-projects/cs220/03-more-image-processing-with-hof (1st screenshot), but when I enter the file (2nd screenshot) and print out the cwd again, it shows /mnt/HDD/umass/umass-projects, meaning 2 folders back

you in  /mnt/HDD/umass/umass-projects/cs220/03-more-image-processing-with-hof this folder open a file then the pwd is /mnt/HDD/umass/umass-projects . this folder has vcs right ? so the hyper found it and set it to root.

In my case, CWD changes to /home/ after I open a recent project from Dashboard and select a file with the automatically opened Telescope plugin. I don't know if this is normal behavior or not

nope dahshboard only change the dir to root if find .git .svn when change_to_vcs_root option is true now. if not found dashboard won't change your pwd .


Getting an error when trying to run main.py

Hi, I have set this up with a pi zero w with a Waveshare 2.13 inch SPI E-Ink E-Paper (B) Display - 3 Colors; followed the instructions to a T, but I am getting the following:
jared7118@raspberrypi:~ $ python3 ~/zero-btc-screen/main.py
Traceback (most recent call last):
File "/home/jared7118/zero-btc-screen/main.py", line 60, in
main()
File "/home/jared7118/zero-btc-screen/main.py", line 40, in main
builder.bind(data_sink)
File "/home/jared7118/zero-btc-screen/config/builder.py", line 23, in bind
screen_class(observable=observable, **kwargs)
File "/home/jared7118/zero-btc-screen/presentation/screens/epd2in13v2.py", line 23, in init
self.epd = self._init_display()
File "/home/jared7118/zero-btc-screen/presentation/screens/epd2in13v2.py", line 31, in _init_display
epd.init(epd.FULL_UPDATE)
File "/home/jared7118/.local/lib/python3.9/site-packages/waveshare_epd/epd2in13_V2.py", line 132, in init
if (epdconfig.module_init() != 0):
File "/home/jared7118/.local/lib/python3.9/site-packages/waveshare_epd/epdconfig.py", line 76, in module_init
self.SPI.open(0, 0)
FileNotFoundError: [Errno 2] No such file or directory
Any ideas what is wrong? SPI is enabled (I've checked 2x)
Thanks in advance!!!!

faulty screen


Security - Axios version

When running npm audit, it reports a high severity issue in the current version of axios.
Axios needs to be updated to a minimum version of 0.21.1 (The current version uses 0.19.2)

Fixed on 2c0b5e4

@ferferga I'll leave this up to you as to whether to reopen this. But there is no new version on npm, so clients using the package will still be affected.

@Camc314 Yes, I'm waiting for @anthonylavado to provide the token to the repo secrets.
As long as that's done, CI will be working  and a new version (both for stable and unstable) will be published.

Token added. Try now?

Version live now on npm.
Unpinning.


Hello, is the possible do retarget automatically for mixamo or other skeletron?

Hello, is the possible do retarget automatically for mixamo or other skeletron?

have u used SMPL-to-FBX to get a fbx for a mixamo skeleton successfully? I tried but failed, the result is weired.


Error: Module parse failed: Unexpected character '@'

Which @angular/* package(s) are the source of the bug?
compiler-cli, Don't known / other
Is this a regression?
Yes
Description
When i run ng build, i get this error "Error: Module parse failed: Unexpected character '@' "
I used "stylePreprocessorOptions"
In Angular version  14. Everything worked fine.
Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
./src/assets/css/styles.scss?ngGlobalStyle:18:0 - Error: Module parse failed: Unexpected character '@' (18:0)
File was processed with these loaders:
 * ./node_modules/resolve-url-loader/index.js
 *<EMAIL_ADDRESS>You may need an additional loader to handle the result of these loaders.
|     100% {background: #ffe6e6;}
| }*/
> @keyframes rotation {
|   from {
|     transform: rotate(0deg);

Please provide the environment you discovered this bug in (run ng version)
Angular CLI: 15.2.4
Node: 14.20.0
Package Manager: npm 8.3.0
OS: win32 x64

Angular: 15.2.5
... animations, cdk, common, compiler, compiler-cli, core, forms
... material, platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1502.4
@angular-devkit/build-angular   15.2.4
@angular-devkit/core            15.2.4
@angular-devkit/schematics      15.2.4
@angular/cli                    15.2.4
@angular/flex-layout            15.0.0-beta.42
@schematics/angular             15.2.4
rxjs                            6.6.7
typescript                      4.8.4

Anything else?
"architect": {
"build": {
"builder": "@angular-builders/custom-webpack:browser",
"options": {
"customWebpackConfig": {
"path": "./extra-webpack.config.js"
},
"allowedCommonJsDependencies": [
"hash.js",
"web3-bzz",
"web3-core",
"web3-eth",
"web3-eth-personal",
"web3-net",
"web3-shh",
"web3-utils"
],
"outputPath": "dist/",
"index": "src/index.html",
"main": "src/main.ts",
"polyfills": "src/polyfills.ts",
"tsConfig": "tsconfig.app.json",
"aot": true,
"assets": ["src/assets"],
"styles": ["src/assets/css/styles.scss"],
"stylePreprocessorOptions": {
"includePaths": ["src/assets/css/", "src/app"]
},
"scripts": []
},

Does this happen if you use use the official builder instead of @angular-builders/custom-webpack:browser?

Are you trying to import src/assets/css/styles.css in your JS like an ECMA module by any chance?

I had import src/assets/css/styles.css and import '../assets/css/styles.scss'; in my app.component.ts. When i removed it it started working. Thank you for your idea.


Docs confusing/wrong

I think the web docs might be wrong.
https://reacttraining.com/react-router/web/api/Route/exact-bool
The grid of examples are identical but with different results.  Neither are an exact match.

That table is correct. The exact column indicates whether the exact prop is true or false.


monitor droplet move

Hey there,
We have some good news! We are upgrading your Droplet to the latest version of our cloud, so we can bring you the latest features on the DigitalOcean platform.
This message is to inform you on December 30, 2014 between 12:30 and 18:30 UTC, your Droplet’s hypervisor will begin the migration process. During a period of approximately 6 hours all of the Droplets on the hypervisor where your Droplet resides will be migrated to a new host. We spread out the migrations to ensure that each runs in a timely manner and completes successfully.
When your Droplet is queued for migration it will be powered off and you can expect about 1 minute of downtime per GB of your Droplet’s disk size. We understand that you may be running services that are mission critical, so of course you have the option to upgrade your Droplet on your own at a time that is more convenient.
You can read more about how to complete this upgrade on your own by following the link below:
https://www.digitalocean.com/community/tutorials/how-to-manually-initiate-droplet-upgrade-to-latest-digitalocean-cloud
Once the upgrade is complete, please check that all your services are running as expected. If you have any issues please check your configuration as specific services you are running may need to be restarted.
Please feel free to reply to this email if you have any questions, and we’ll make sure this process is as simple as possible.
Thank you,
The DigitalOcean Team
Affected Droplets:
grtp.co

https://gratipay.freshdesk.com/helpdesk/tickets/1514

Closing since this has already happened.


Inputs per denomination in GUI not honoured

Hi,
When setting non default values for denoms_goal and denoms_hardcap in GUI, dash-qt still uses default values.
Steps to reproduce:
1.) Fresh installation of dashcore-20.1.1-x86_64-linux-gnu.tar.gz (no configuration files)
2.) Start dash-qt
3.) Set "Inputs per denomination" to Target = 10, Maximum = 16
4.) Check used values in console with getcoinjoininfo: 10/16, this is ok.
5.) Restart dash-qt
6.) Check values in GUI: 10/16, this is ok
7.) Check used values in console with getcoinjoininfo: 50/300, this is NOT ok.
Workaround: Set values in ~/.dashcore/dash.conf.
Cheers, Peter

More than one user reporting this issue.  Can a dev have a look into it and see if you can reproduce and maybe suggest a fix?

More than one user reporting this issue.

Can you cite the others?

Resolved in https://github.com/dashpay/dash/pull/6208; will be released in v22 / v21.2 / v21.1.1 whichever comes first :)


Bump surelog

This PR applies patch to opentitan in order to workaround issue in Surelog: https://github.com/chipsalliance/Surelog/issues/2724

Moved to #597


Feature/jenkins migrations tests

Added execute permissions to jenkins.sh...

Coverage decreased (-0.009%) to 75.899% when pulling c2cc3e30130d5faf05dceb5b9665b6611946f6e8 on feature/jenkins-migrations-tests into 4862ce3566f2ade9a64fcff69d149c49c55b3ac5 on develop.


Documentation: add link to Github repo

The Genkit repo isn't mentioned anywhere in the documentation.
We should add a links to the following:

[ ] The repo itself
[ ] The issue tracker
[ ] The discussions forum

repo link is there; other links added in #692


Add zap.Logger adapter for sarama

This adds:

A SaramaLogger struct that wraps a zap.Logger with the sarama.StdLogger interface
Configures sarama to use this new logger

SaramaLogger uses zap.SugaredLogger internally for its Print(...) and Printf(...) functions.
This results in log messages from sarama with the same structured format as the rest of the logging:
{"level":"info","ts":1567704918.2796876,"msg":"client/metadata fetching metadata for all topics from broker kafka-broker-1.internal.network:9092\n","name":"sarama"}

This was helpful for debugging SSL connection issues, ref #563.

Looks like the equivalent work was done in #548.