This PR adds new options to the sce_to_anndata() function to allow for compression of the exported HDF5 files.
I added a compression option and set the default to gzip. I like the idea of having a separate argument rather than just using ..., partially as a reminder. I did not use the same default of none that the zellkonverter::writeH5AD function uses. Instead, I set it to gzip since I think we want to compress by default. I'm on the fence on whether that was a good decision.
I added ... as an option to pass any additional arguments to the function.
I updated the tests to test that compression fails with an invalid option and a test to make sure using additional arguments works. Let me know if I should create any additional tests.
Closes #244
This PR adds new options to the
sce_to_anndata()
function to allow for compression of the exported HDF5 files.compression
option and set the default togzip
. I like the idea of having a separate argument rather than just using...
, partially as a reminder. I did not use the same default ofnone
that thezellkonverter::writeH5AD
function uses. Instead, I set it togzip
since I think we want to compress by default. I'm on the fence on whether that was a good decision....
as an option to pass any additional arguments to the function..h5ad
extension.