-
Notifications
You must be signed in to change notification settings - Fork 14
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
bidsConcatBetaTmaps. fix #425
Conversation
for more information, see https://pre-commit.ci
Codecov Report
@@ Coverage Diff @@
## main #425 +/- ##
==========================================
+ Coverage 60.60% 63.18% +2.58%
==========================================
Files 120 120
Lines 2000 2010 +10
==========================================
+ Hits 1212 1270 +58
+ Misses 788 740 -48
Continue to review full report at Codecov.
|
Will add tests to this workflow. |
Got the the following error from the test suite (so unrelated to the code):
This should not happen <--- I mean seriously I am still laughing. |
@CerenB Can you check that the minor changes I did, did not break anything? |
all good! |
OK will merge and apply those changes to the dev branch? |
should work on dev branch as well, yes. I did not do anything which would be problematic. |
I already applied the patch to I have some failing tests so I will see what is up with that and also update to use the latest version of bids-matlab everywhere: this means that CPP-ROI will also start diverging so I will make a realease so we know which version the |
minor fixes with: