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

[ENH] save jobs as m file #482

Merged
merged 24 commits into from
Jan 21, 2022
Merged

Conversation

Remi-Gau
Copy link
Contributor

@Remi-Gau Remi-Gau commented Jan 21, 2022


MISC

  • reorganize subfolders for src and tests
  • update variable names of default values in functions using argparse

@Remi-Gau Remi-Gau changed the base branch from main to dev January 21, 2022 09:21
@codecov
Copy link

codecov bot commented Jan 21, 2022

Codecov Report

Merging #482 (d64d799) into dev (9d0275f) will increase coverage by 0.19%.
The diff coverage is 72.16%.

❗ Current head d64d799 differs from pull request most recent head d5fcb7e. Consider uploading reports for the commit d5fcb7e to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #482      +/-   ##
==========================================
+ Coverage   73.17%   73.36%   +0.19%     
==========================================
  Files         186      189       +3     
  Lines        3869     3924      +55     
==========================================
+ Hits         2831     2879      +48     
- Misses       1038     1045       +7     
Flag Coverage Δ
unittests 73.36% <72.16%> (+0.19%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/IO/cleanCrash.m 100.00% <ø> (ø)
src/IO/createDerivativeDir.m 100.00% <ø> (ø)
src/IO/getData.m 100.00% <ø> (ø)
src/IO/loadAndCheckOptions.m 91.66% <ø> (ø)
src/IO/onsetsMatToTsv.m 100.00% <ø> (ø)
src/IO/overwriteDir.m 100.00% <ø> (ø)
src/IO/regressorsMatToTsv.m 100.00% <ø> (ø)
src/IO/renameSegmentParameter.m 0.00% <ø> (ø)
src/IO/renameUnwarpParameter.m 0.00% <ø> (ø)
src/IO/unzipAndReturnsFullpathName.m 57.14% <ø> (ø)
... and 29 more

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 9d0275f...d5fcb7e. Read the comment docs.

@Remi-Gau Remi-Gau merged commit be9ba3f into cpp-lln-lab:dev Jan 21, 2022
@Remi-Gau Remi-Gau deleted the remi-job_mat_to_mfile branch January 21, 2022 18:52
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.

use cfg_util to create a .m version of the job.mat`files that contain matlabbatches
1 participant