ccp4 / DUI

CCP4 DIALS User Interface
GNU General Public License v2.0
5 stars 3 forks source link

Refactor img_viewer #161

Closed dagewa closed 4 years ago

dagewa commented 4 years ago

Changes intended to improve readability and maintainability of this long module.

There's still much to do in here, but I'd like to request review now and continue later if this is merged. Testing should mainly concentrate on the mask generation as this is where most changes were made.

dagewa commented 4 years ago

This is intended to address #122

dagewa commented 4 years ago

For future reference it isn't necessary to merge into master for testing. You could have just run git checkout refactor-img_viewer to switch to the branch, and git checkout master when you are done.