Luke Zappia

I probably should have copied it here but from memory, something along the lines of `X is not a valid path`.

I don't know much about `SpatialExperiment` so I won't comment on that part but from my side I am happy to add this to **{zellkonverter}**. From this example it looks...

> > I do have a question for @giovp and @ivirshup. Do you plan to keep using this special "spatial" name in existing slots going forward or do you think...

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

You can find the release schedule here https://bioconductor.org/developers/release-schedule/. You may have just missed the deadline for acceptance already but @Bioconductor/core can confirm.

> * The package is built based on the Seurat object. The majority of functions use the Seurat objects as the input. If we would like to support the SpatialExperiment...

Hi @Nina-Song. It looks like the build is running without any issue now which is great. Please comment with the changes you have made when you are ready for me...

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

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

I think you probably need to prefix the function with the package name (eg. `bioc_parallel_obj = BiocParallel::MulticoreParam(1)`). A couple of other minor suggestions: * I would use `SerialParam()` as the...