guilhermelowa / minj Goto Github PK

Currently we are prompted to enter a number for a given metric, e.g.: "attention: <input_number>". I've noticed that while logging a number, I want to attach an explanation to it. Ex:

"attention: 5 - today I was so focused while playing bass that nothing else existed".
"emotions: 5 - felt good today because playing was so great"

My suggestion is to do a regex to get the number and the comment. The comments could be written in a column called "comments" on the metrics.csv. Preferably attach a tag to the specific metric you were logging. So in the above example, the metrics.csv file would look like:

attention,emotions,comments
5,5,[attention] today I was so focused while playing bass that nothing else existed;[emotions] felt good today because playing was so great"

I also suggest that comments should always be the last item in the header. Hence, when adding a new metric it should always come before comments.

To solve this Issue, basically you have to change the metrics_log.py file. Add the regex to get the comment and save it to the metrics.csv file and also modify the add / remove metrics functions in order to keep the comments column always the last one

Add test to the functionalities we already have. They read/write to a file. We currently have the following functionalities:

• Add metric
• Remove metric
• Log sleep times

There are some I/O to files on sleep_log.py and metrics.py that do not use the Python csv module. We should change to use the csv module since we are performing I/O on csv files

Create a script that adds a shell asking for sleep and wake up times on system start up. Or else, find a way to do this and add to README, like the crontab instructions.

Currently when we add / remove a metric, we update the file with the active metrics and update the header of the metrics.csv file (where the recordings are stored). I can see two problems with it: metrics with same name and adding a previously removed metric.

• When adding a metric with the same name as an active one, we have to make sure to include only one.
• When adding a metric previously removed, we shouldn't change the header on the metrics.csv file. Just keep writing to the respective column

We should add perform validation on sleep_log.py, to validate the sleep times. This should include 12h format and 24h