aremazeilles / beat_routine

Performance Indicators developed by BEAT consortium for the Eurobench project
0 stars 2 forks source link

Header in Platform data #10

Closed aremazeilles closed 4 years ago

aremazeilles commented 4 years ago

Platform data files should have a header, with the label of each column.

Also (maybe to be treated later on), it is good practice to define the column of interest based on the header labels. If not, any change in the column order (for any reason) makes your code not working anymore. If you verify the column of interest is in the the file, and get its idea, the column order can change, you are not affected.

AS an example, what we do in our sample:

juritaborri commented 4 years ago

I created new branch with modified files according this suggestion

aremazeilles commented 4 years ago

If you are done, can you go to the root of the repo, branches -> and create a pull request (indicating it would fix that issue), and assign it to me?