Luke Zappia

> > * [x] 🚨 The vignetted should have an _Installation_ section that uses Bioconductor installation instructions (NOT evaluated) You can write this section as though the package has been...

Ok, I'm happy with this. I will mark it as approved.

Congratulations on getting **{SpotSweeper}** accepted into Bioconductor 🎉! It can take a few days to be picked up by the build system but then it will be available in Bioconductor...

Hi @BioinfoMonzino Thanks for submitting **PIUMA** :tada:! Below is my review of your package. Please reply here if anything is unclear or needs any further explanation. **What next?** Please address...

> > * [x] ⚠️ I got some notes when checking the packages. The important ones should be mentioned below but please check the build report after making changes to...

@grimbough Do you have an idea whether it would be possible to add this relatively soon? At the moment it is blocking a project I was hoping to submit soon.

Ok, thanks. This is for reading/writing H5AD files from R. The **anndata** spec says a "boolean valued field" but I think you might be correct that it is actually written...

Thanks! I tried this and I think it works for our use (with minimal testing). Here is a helper function I wrote based on your code in case that's useful:...

Thanks, this should make things much easier! I finally got around to testing it. Has anything else changed in `h5writeAttribute()`/`h5readAttribute()` that would change how things are stored/read into R? I...

You can ignore my last comment (and the one I deleted). Turns out there was a bug in my code and `asScalar` wasn't been set correctly. When you do set...