lkorczowski / Tinnitus-n-Sleep

Detecting events in sleeping tinnitus patients
MIT License
1 stars 0 forks source link

Refactor Notebooks with new preprocess #82

Closed lkorczowski closed 4 years ago

lkorczowski commented 4 years ago

closes #78 closes #79 closes #80 (obsolete) closes #81 (required)

review-notebook-app[bot] commented 4 years ago

Check out this pull request on  ReviewNB

Review Jupyter notebook visual diffs & provide feedback on notebooks.


Powered by ReviewNB

codecov[bot] commented 4 years ago

Codecov Report

Merging #82 into master will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #82   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           11        11           
  Lines          635       655   +20     
=========================================
+ Hits           635       655   +20     
Impacted Files Coverage Δ
tinnsleep/data.py 100.00% <100.00%> (ø)
tinnsleep/reports.py 100.00% <100.00%> (ø)
tinnsleep/signal.py 100.00% <100.00%> (ø)
tinnsleep/utils.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 63d4d68...ab53081. Read the comment docs.

review-notebook-app[bot] commented 4 years ago

View / edit / reply to this conversation on ReviewNB

RobinGuillard commented on 2020-05-18T14:01:36Z ----------------------------------------------------------------

WARNING : you are not follwing the parameters set that was originally there!! We used to do a n_adaptative of 480 epochs (which is of 2 min) and now you have only 1 min of n_adaptative!