search

AllenInstitute / ecephys_etl_pipelines

Pipelines and modules for processing extracellular electrophysiology data
Other
1 stars 2 forks source link

#18 monitor lag solution for visual behavior neuropixels #22

Closed ns706 closed 2 years ago

danielsf commented 2 years ago

@ns706 I apologize if I am being pedantic, but I assume that you noticed this PR is failing the automatic linter and unit test checks run by CircleCI.

Re: linting -- we aren't doing anything beyond standard pep8 standards. If you just install flake8 (I use conda install flake8) and run flake8 ecephys_etl_pipelines/src/ you should see all of the failures. You'll need to do the same for the tests directory.

ns706 commented 2 years ago

@ns706 I apologize if I am being pedantic, but I assume that you noticed this PR is failing the automatic linter and unit test checks run by CircleCI.

Re: linting -- we aren't doing anything beyond standard pep8 standards. If you just install flake8 (I use conda install flake8) and run flake8 ecephys_etl_pipelines/src/ you should see all of the failures. You'll need to do the same for the tests directory.

Thanks. I've fixed the linter issues.