Peter Hickey

Results 174 comments of Peter Hickey

Oh I now see https://github.com/Bioconductor/SummarizedExperiment/issues/29.

I've not looked at the package closely, but has the design and implications of this package been discussed/planned with the **DropletUtils** and **SpatialExperiment** authors, which already have `DropletUtils::read10xCounts()`, `DropletUtils::read10xMolInfo()`, and...

Hi @LiNk-NY, Thank you for submitting **TENxIO** to Bioconductor. The package is already in reasonably good shape. However, I do think **TENxIO** requires comparison to existing Bioconductor solutions for importing...

Hi @LiNk-NY, Thank you for making the above changes. There are still a few issues that I think require further changes or that I noticed following your recent changes. Regarding...

Thank you for addressing the requested changes and engaging with the review process, @LiNk-NY. I'm happy to accept **TENxIO** into Bioconductor! A couple of final things: - [ ] Please...

The warning in the macOS builder looks like something that needs to be addressed in your code: https://bioconductor.org/spb_reports/ccImpute_buildreport_20220429124256.html#merida1_check_anchor I'll see if it reproduces on my Mac. Are you able to...

Can confirm that I get the warning when checking the package on my MacBook Pro (Early-2015). ```sh # Trimmed output of running `R CMD check ccImpute_0.99.0.tar.gz` * checking compiled code...

I'm not sure of the solution, but I think this indicates it's something you will need to fix within **ccImpute**.

Please remember to push to `[email protected]:packages/ccImpute` to trigger a new build.

Thanks for your submission of **CITEViz**, @gartician. I'll be reviewing the package but won't be able to start the formal review until next week. > should I instruct the users...