Skip to content

Updating dm_parse_faces onset timestamps#312

Merged
DESm1th merged 8 commits intoTIGRLab:masterfrom
benselby:working
Oct 14, 2021
Merged

Updating dm_parse_faces onset timestamps#312
DESm1th merged 8 commits intoTIGRLab:masterfrom
benselby:working

Conversation

@benselby
Copy link
Copy Markdown
Contributor

Correcting the trial onset times to be relative to the sync time, and reported in seconds instead of milliseconds.

Please pardon the polluted history - the git is me.

@pep8speaks
Copy link
Copy Markdown

pep8speaks commented Sep 30, 2021

Hello @benselby, Thank you for updating!

Line 234:81: E501 line too long (90 > 80 characters)
Line 235:81: E501 line too long (81 > 80 characters)
Line 240:30: W291 trailing whitespace

To test for issues locally, pip install flake8 and then run flake8 datman.

Comment last updated at 2021-10-04 19:58:24 UTC

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 30, 2021

Codecov Report

Merging #312 (0fb389e) into master (9db754f) will decrease coverage by 0.06%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #312      +/-   ##
==========================================
- Coverage   31.53%   31.46%   -0.07%     
==========================================
  Files          62       62              
  Lines        9218     9238      +20     
==========================================
  Hits         2907     2907              
- Misses       6311     6331      +20     
Impacted Files Coverage Δ
bin/dm_parse_faces.py 0.00% <0.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 9db754f...0fb389e. Read the comment docs.

@DESm1th DESm1th merged commit a0cd1ff into TIGRLab:master Oct 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants