smistad / FAST

A framework for high-performance medical image processing, neural network inference and visualization
https://fast.eriksmistad.no
BSD 2-Clause "Simplified" License
433 stars 101 forks source link

Bug fix for View and PipelineHighlighter classes (2nd attempt) #140

Closed andreped closed 3 years ago

andreped commented 3 years ago

As discussed earlier. Needed to make a new, fresh pull request due to some issues during merge.

Also, regarding the View class (removeRenderer method) my approach can be explained from here: https://stackoverflow.com/questions/3385229/c-erase-vector-element-by-value-rather-than-by-position/3385251#3385251

There was also a comment stating that it is necessary to include the "algorithm" library as well, so I did that also. However, I did not see the mentioned errors if it was not included.

I've tested the fixes using FastPathology on my Windows10 laptop using the most recent commit of FAST.