emo-bon / governance-data

Holds the governance content for the emo-bon data management
0 stars 0 forks source link

emo-bon/governance-data comments #24

Open cymon opened 3 months ago

cymon commented 3 months ago

a) There's a column in ./observatories.cvs name "contect person" should be "contact_person" with underscore b) "contact person email" should be "contact_person_email" with underscores c) Column that have the same data but are in different sheets should have the same column names: e.g EMOBON_observatory_name in observatories.csv and operating_observatory_id in organisation.csv d) consider combining observatories.csv and organistations.csv to same on duplicating data. e) there a quite a few cells with blanks strings in them " "... rather than being NULL/NaN

kmexter commented 2 months ago

a) and b) I changed as they are typoes, but I will also need to pass this on to Bram so they can check if this breaks anything in the harvesting/qc code c) I agree, but before changing one or the other I need to check with Bram on that also d) not sure why ioulia wanted these two separate, but agree could be combined - need to check first where this is used before doing that

kmexter commented 1 month ago

I will change A and B and then tell bram