joaquinanguera / aceR

An R package for processing ACE data
MIT License
3 stars 14 forks source link

Error in loading demographics.csv #51

Open joshvolponi opened 1 month ago

joshvolponi commented 1 month ago

Problem: "I found the aceR toolbox cannot load the demographic.csv and will generate error if I remove the demographic.csv. Attached please kindly find the R code and the output." This may be related to #49 as the columns N and Y are with the new name convention ACE_RawData_WPI2-20240531T173409Z-001.zip Attached is the file that they were trying to process

monicathieu commented 1 month ago

Ahh, thank you for uploading the example data! This data makes clear that the new naming convention is indeed coming out of ACE Nexus. This is the first time I'm learning about the new Nexus column naming convention.

I just checked the development branch and if load_ace_bulk() is called with app_type = "nexus", it SHOULD handle at least the new "Participant ID" column name. The "PID" column in the new demographics.csv appears to be a duplicate column of "Participant ID", so that shouldn't cause issues either. Is this person able to run this data through on the development branch with app_type = "nexus"? If not, please post their error message here and that will help me identify which column renames aren't handled yet.