akoyabio / phenoptrReports

Create analysis reports from inForm data.
https://akoyabio.github.io/phenoptrReports
zlib License
15 stars 7 forks source link

Merge seg files bug #55

Closed JpGuegan closed 2 years ago

JpGuegan commented 2 years ago

With the new version of PhenoptrReport (3.1), the merge_cell_seg_file function stops on files with no detected cells (empty cell seg file with just the column names) and we have to stop the command because it lasts forever. However, when we processed the same batch of files with an older version (v2.4), the script runs smoothly and a merge_cell_seg_file is created. How to solve this?

AB-Kent commented 2 years ago

That's odd, the merge function has changed very little since version 0.2.4. Can you share a set of files that fail? (Email a link to files.) Also please show the output of the command devtools::session_info()

JpGuegan commented 2 years ago

Here is a link to some files https://explicyte.transfernow.net/dl/20211208q7dRXnGK

and below is the session info

AB-Kent commented 2 years ago

Hmm, I'm able to merge that data without problem. My system is very similar to yours, though Windows 10 vs Windows Server.

Is there anything else you can tell me about the failure? What versions of readr and vroom do you have, they are not in the list above.

JpGuegan commented 2 years ago

Ok, that's weird because I have tested it on two different computers (Windows10 & server). Here are the versions of readr (v2.0.2) and vroom (1.5.5).

AB-Kent commented 2 years ago

Please update both readr and vroom to the latest versions; you seem to have run into this readr bug: https://github.com/tidyverse/readr/issues/1297

JpGuegan commented 2 years ago

Thanks! The updates fixed the problem. Sorry for the inconvenience!