Open matthiaskoenig opened 5 years ago
Just as a note: The new SBML parser supports reading compressed files from paths, but not yet from file handles. But support for compression on writing is missing.
I suppose libsbml doesn't support writing to a Python file stream, i.e., it can only either take a filename or create a string?
Unfortunately not. Libsbml only supports reading or writing SBML to strings or paths. So the support for file handles and file streams will be implemented in python (via temporary files and reading the SBML strings via read()
).
Is this still valid? I'm asking because cobra/sbml.py seems to work well with gz and bz2, both reading and writing
I implemented the compression support via libsbml for SBML. But there is no compression support for the other input formats.
Hi Would you like me to use python modules gzip and bz2 to implement reading/writing of compressed files for the other formats? It would be slightly different from libsbml, which does it internally, but then the other formats could use them as well. I'm not sure it makes sense for matlab, but I might implement it for completeness.
On Wed, Jul 13, 2022 at 3:36 AM Matthias König @.***> wrote:
I implemented the compression support via libsbml for SBML. But there is no compression support for the other input formats.
— Reply to this email directly, view it on GitHub https://github.com/opencobra/cobrapy/issues/812#issuecomment-1182871303, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACQYYZXUQGMQXNFO7GUA42LVTZWWFANCNFSM4G3YOIHA . You are receiving this because you commented.Message ID: @.***>
Yes, this was basically the idea. Especially the JSON can be compressed very efficiently.
Okay - if you can review and hopefully merge #1245, which allows I/O to use Paths, I can build upon that to get all formats to deal with compressed files. Probably a helper function/file in the io directory, and all the formats will call it.
On Wed, Jul 13, 2022 at 11:58 AM Matthias König @.***> wrote:
Yes, this was basically the idea. Especially the JSON can be compressed very efficiently.
— Reply to this email directly, view it on GitHub https://github.com/opencobra/cobrapy/issues/812#issuecomment-1183401665, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACQYYZTTKOPOJLVLI5G4323VT3RTPANCNFSM4G3YOIHA . You are receiving this because you commented.Message ID: @.***>
In the discussion with @Midnighter it came up that there should be a generic support for compressed files for the various io modules. I.e. cobrapy should support the reading and writing of compressed files to the various formats (JSON, MAT, SBML, YAML). To avoid code duplication there should be a single implementation of the compression support.
Compression becomes important for the genome-scale models which are very large (uncompressed).
Just as a note: The new SBML parser supports reading compressed files from paths, but not yet from file handles. But support for compression on writing is missing.