Skip to content

Conversation

@hkjeldsberg
Copy link
Collaborator

  • Visualize resulting landmarks using vmtkrenderer

  • Updated docstrings with example image for case C0001 from Aneurisk and both landmarking algorithms

  • Builds on Restructure Landmarking #55

- Updated tests
- Updated documentation
- Fixed typos in docs
- Set llvm version to 3.3 for Travis
- Separated find_interface methods for algorithms
- Added missing docstrings
- Use of numpy integration of vtk
…i algorithms.

- Updated documentation with visualization image
@hkjeldsberg hkjeldsberg added the enhancement New feature or request label Aug 27, 2020
@codecov-commenter
Copy link

Codecov Report

Merging #57 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #57   +/-   ##
=======================================
  Coverage   98.98%   98.98%           
=======================================
  Files          14       14           
  Lines         888      889    +1     
=======================================
+ Hits          879      880    +1     
  Misses          9        9           
Impacted Files Coverage Δ
test/test_automated_landmarking.py 100.00% <100.00%> (ø)
test/test_common.py 100.00% <100.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 0b73948...e91cbae. Read the comment docs.

Copy link
Contributor

@aslakbergersen aslakbergersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No comments, looks good!

@aslakbergersen aslakbergersen merged commit d7636b6 into master Aug 27, 2020
@aslakbergersen aslakbergersen deleted the visualize-landmarks branch August 27, 2020 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants