openvax / gtfparse

Parsing tools for GTF (gene transfer format) files
Apache License 2.0
108 stars 25 forks source link

deleted util module, which provided UNIX-specific access to memory usage #15

Closed iskandr closed 5 years ago

iskandr commented 5 years ago

Fixes https://github.com/openvax/gtfparse/issues/5 and https://github.com/openvax/varcode/issues/234

Our whole stack is broken on Windows because gtfparse is logging free memory before/after loading a GTF. There's apparently no platform-independent way to get free memory within the Python standard library and, while we could theoretically use wmi to access the Windows API, this is already a lot of complexity for a set of logging statements unrelated to functionality.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 107


Files with Coverage Reduction New Missed Lines %
gtfparse/read_gtf.py 1 80.0%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 105: 0.1%
Covered Lines: 139
Relevant Lines: 155

💛 - Coveralls
julia326 commented 5 years ago

why delete? may be worth keeping and only supporting it for certain OSs