OxWearables / biobankAccelerometerAnalysis

Extracting meaningful health information from large accelerometer datasets
https://biobankaccanalysis.readthedocs.io/en/latest/index.html
Other
179 stars 61 forks source link

Extract light from ax3 #218

Open angerhang opened 1 year ago

angerhang commented 1 year ago

Test plan:

python accelerometer/accProcess.py data/hang.cwa --deleteIntermediateFiles False

Output: Screenshot 2022-12-01 at 19 44 53

Screenshot 2022-12-02 at 14 16 17

This PR might break some of the regression tests as it adds a new column to the extracted files but won't know until the tests are finished. Oh well, will have to regenerate the other reference data format but can anyone review this PR before I regenerate the reference extraction files? :D

aidendoherty commented 1 year ago

Would it be better to introduce this enhancement in Actipy rather than BBA @angerhang @chanshing ?

angerhang commented 1 year ago

Would it be better to introduce this enhancement in Actipy rather than BBA @angerhang @chanshing ?

Great idea Aiden. I am not familiar with the Actipy codebase so not sure how to do that exactly. It is also not clear that for new feature, which repository to contribute to because Actipy doesn't have many features that BBA has. Having to duplicate code in two repository just double the efforts for any new changes unfortunately :(

Maybe, let's get merged in BBA first, then create an issue on Actipy to add the light extraction is the a good first step?

chanshing commented 1 year ago

Thanks, Hang. I'll put it on my todo list to implement it in actipy also.

chanshing commented 1 year ago

All looks good. I'll think about how to rename ax3_light and to distinguish it from light. Thanks a lot Hang!

angerhang commented 1 year ago

This feature is put on hold until the actipy backend migration finishes sometime next month.

Will revisit this end of January.