Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Reading E4 should account to find directories inside each given directory, each representing a Segment #34

Closed
jomy-kk opened this issue Jun 23, 2022 · 0 comments
Assignees
Labels
fix Fix a feature

Comments

@jomy-kk
Copy link
Owner

jomy-kk commented Jun 23, 2022

Describe the bug
The way E4._read is implemented it expects csv files to be right on the given directory, but it should not.
Inside the given directory, it should expect subdirectories (probably named after the start timestamp), and each should be considered a Segment in time.

Example of tree:
--- given_path
    | ---- 1655923195_A0200B
        | ---- ACC.csv
        | ---- BVP.csv
    | ---- 1656324824_A0200B
        | ---- ACC.csv
        | ---- BVP.csv
    | ...

Modules
biosignals

To Reproduce
Instructions that reproduce the behavior:

E4._read(given_path, EDA)

Terminal/Console Error

Files were not found in path given_path

Expected behavior
To read files inside subdirectories, and consider each subdirectory as a Segment.

Python Version
3.10.4

@jomy-kk jomy-kk added the fix Fix a feature label Jun 23, 2022
@jomy-kk jomy-kk added this to To do in Develop Biosignals Framework via automation Jun 23, 2022
@jomy-kk jomy-kk closed this as completed Jun 24, 2022
Develop Biosignals Framework automation moved this from To do to Done Jun 24, 2022
jomy-kk added a commit that referenced this issue Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fix a feature
Development

No branches or pull requests

2 participants