afids / afids-validator

Validator for the anatomical fiducial placement protocol
https://validator.afids.io
GNU General Public License v3.0
2 stars 4 forks source link

Strip old flexx visualization from project #84

Closed tkkuehn closed 4 years ago

tkkuehn commented 4 years ago

Proposed changes

Remove all of the prototype flexx visualization that will be superseded by a forthcoming visualization from Jak.

Types of changes

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!

Notes

All PRs will undergo the unit testing before being reviewed. You may be requested to explain or make additional changes before the PR is accepted.

_PR template was adopted from appium_

kaitj commented 4 years ago

Code looks good, everything appears to be working locally on my end. Just curious what breaks with the removal or do you just mean that the existing visualization functions will no longer work?

tkkuehn commented 4 years ago

Just curious what breaks with the removal or do you just mean that the existing visualization functions will no longer work?

Yep, I just mean existing point cloud visualization will be gone with this change. I'll go ahead and merge!