Katie Bottenhorn, PhD comments

Results 7 comments of


                                            Katie Bottenhorn, PhD

trafficstars

Meta-analytic clustering workflow

I'm happy to take a look. Last I remember it was _almost_ ready, but hung up in a few places (i.e., `sleuth_to_dataset`, I think). I'll take a look and fix...

[PRE REVIEW]: TLViz: Visualising and analysing tensor decomposition models with Python

Hi! Sorry, I don't have the bandwidth to review right now.

Going black (i.e. adopting an _opinionated_ python code formatter)

I feel the same! (both about `"` vs `'` and about `black`)

What happened to image metadata?

Related, in trying to edit images' meta-data today, I was greeted with this (same on Firefox v113.0.1, Chrome v113.0.5..., & Safari v15.2): ![Screen Shot 2023-05-26 at 12 40 59 PM](https://github.com/NeuroVault/NeuroVault/assets/14095475/866dd975-2eef-454a-a4b8-6e3df652fa5b)

More options with NK

Hey @DominiqueMakowski, Thanks for taking a look! I think you're right, that seems like the best way to integrate our peak-finding methods. I think an optional NK dependency in peakdet...

Issues in Ex 12.2

Previous comment was outdated, that's my bad. LGTM!

Issues in Ex 12.2

Yes to `peakdet.modalities`, but the only thing in `peakdet.analytics` is heart rate variabilty computations. Are those used for denoising? (genuine question, as my knowledge of HR-based denoising has some gaps)