LAAC-LSCP / ChildProject

Python package for the management of day-long recordings of children.
https://childproject.readthedocs.io
MIT License
13 stars 5 forks source link

detect and load additional metadata recursively #186

Closed lucasgautheron closed 3 years ago

lucasgautheron commented 3 years ago

Description

Currently, recordings and children metadata are assumed to be entirely stored in metadata/recordings.csv and metadata/children.csv respectively. However, in case users spread their metadata across several files (e.g. to achieve multi-tier access), then there is no standard.

I suggest the standard should be that additional metadata belong as extra CSVs in metadata/recordings and metadata/children respectively. These folders are optional.

If the package detects them, it will list all CSVs from these folders recursively, sort them alphabetically (currently based on their basename, which might be ambiguous), then merge them sequentially.

For instance, if recordings/0.csv and recordings/1.csv exist, the package will :

etc.

Status

alecristia commented 3 years ago

sort them alphabetically (currently based on their basename, which might be ambiguous)

I didn't see in your code a check for ambiguity (ie without path, the names are identical).

I may have also missed a set of messages for the user:

and/or error:

(my messages are super cryptic, but you see the idea!)