jmbreda / Sanity

Filtering of Poison noise on a single-cell RNA-seq UMI count matrix
GNU General Public License v3.0
69 stars 11 forks source link

Added progress prints, and targeting print messages at stderr instead of stdout to avoid buffering #21

Closed dhdegroot closed 11 months ago

dhdegroot commented 11 months ago

The memory and time estimate was not printed when Sanity was run on our computing cluster. This is because the messages to stdout are buffered. One can either solve this by flushing the messages, or by just printing to stderr. I decided that it was cleaner to send those messages to stderr. As such, stdout could in principle be used for already outputting results from the computation, maybe the fitted v_g or even the fitted deltas.

In addition, I added a print statement done by the first thread once it has done 2.5% of its alotted genes. After that it will print after 5%, 10% and so on. I think this keeps the print statements limited, and it is nice for users to see progress.

Let me know if I can help further, it is not hard to do flushes instead of cerr, if you prefer that.