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

WIP: trx integration #2884

Closed
wants to merge 15 commits into from
Closed

WIP: trx integration #2884

wants to merge 15 commits into from

Conversation

arokem
Copy link
Contributor

@arokem arokem commented Aug 31, 2023

This is like #2759 but rebased on master. Just trying to see whether we still have version issues.

@codecov
Copy link

codecov bot commented Aug 31, 2023

Codecov Report

Merging #2884 (e1130b1) into master (9bbdda0) will increase coverage by 0.00%.
Report is 60 commits behind head on master.
The diff coverage is 92.46%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2884   +/-   ##
=======================================
  Coverage   81.61%   81.61%           
=======================================
  Files         146      146           
  Lines       20363    20398   +35     
  Branches     3224     3233    +9     
=======================================
+ Hits        16619    16648   +29     
- Misses       2924     2927    +3     
- Partials      820      823    +3     
Files Coverage Δ
dipy/__init__.py 75.00% <ø> (ø)
dipy/core/geometry.py 91.19% <ø> (ø)
dipy/core/onetime.py 89.47% <ø> (ø)
dipy/core/profile.py 30.00% <ø> (ø)
dipy/core/rng.py 66.66% <ø> (ø)
dipy/core/sphere_stats.py 58.06% <ø> (ø)
dipy/core/wavelet.py 88.34% <ø> (ø)
dipy/data/fetcher.py 39.51% <ø> (ø)
dipy/info.py 100.00% <100.00%> (ø)
dipy/io/stateful_tractogram.py 76.96% <100.00%> (+0.43%) ⬆️
... and 19 more

@pep8speaks
Copy link

Hello @arokem, Thank you for updating !

Line 39:81: E501 line too long (81 > 80 characters)
Line 123:81: E501 line too long (81 > 80 characters)

@arokem
Copy link
Contributor Author

arokem commented Oct 9, 2023

Closing in favor of #2759

@arokem arokem closed this Oct 9, 2023
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.

None yet

3 participants