andersen-lab / Freyja

Depth-weighted De-Mixing
BSD 2-Clause "Simplified" License
102 stars 29 forks source link

freyja plot doesn't deal with empty pandas df #166

Closed PlushZ closed 8 months ago

PlushZ commented 1 year ago

When attempting to apply the --mincov parameter for the 'freyja plot' and no samples meet the minimum coverage value, Freyja throws a pandas error. I wanted to inquire whether you are aware of this issue and if there are any plans to address it in the near future? Thanks.

Polina

joshuailevy commented 1 year ago

Hi @PlushZ, this is a good point. While I was aware of the issue, I hadn't gotten around to incorporating a more specific warning/error message to guide the user. I'll make sure to add this into the next release. Josh