cabouman / mbircone

BSD 3-Clause "New" or "Revised" License
11 stars 9 forks source link

preprocess tested and ready to merge #99

Closed dyang37 closed 1 year ago

dyang37 commented 2 years ago

This is a PR to merge preprocess module into master branch. See below for preprocess API docstrings: mozilla.pdf

I tested on various NSI datasets to make sure that the default parameters work on all corner cases we have encountered so far (landscape vs. portrait mode, horizontal/vertical flip of the radiograph, etc.).

dyang37 commented 2 years ago
dyang37 commented 1 year ago

Closing this PR for now. We can reopen this once we reach consensus on the interface design of preprocess module.