cbirdferrer / collatrix

this tool collates the outputs of the MorphoMetriX photogrammetry tool
MIT License
4 stars 2 forks source link

Output is not a valid CSV file #7

Closed devarops closed 4 years ago

devarops commented 4 years ago

The header in column 1 of the output CSV file is blank. Please add a header to the first column of the output to ensure that it is a valid CSV file.


cbirdferrer commented 4 years ago

Thank you for this suggestion! Header added! I've also gone through and edited the headers of the demo files to be valid as well.

devarops commented 4 years ago

Thank you for addressing this issue, but I'm still getting an empty header in the first column.

image

You could also just remove the first column. In either case, I strongly recommend that the output be a valid CSV file.

If you believe that the problem is fixed and I'm missing something, please use an automated test to show this.

cbirdferrer commented 4 years ago

I apologize, I only changed one of the instances of .to_csv in the main function, I have double-checked that all instances (2 in the main function, 1 each in the other 2 functions) of df.to_csv now include an index_label: .to_csv(outcsv,sep = ',',index_label = 'IX')

devarops commented 4 years ago

That looks good! :+1: :100: