search

ccp4 / DUI

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

Rename "reindex" button #56

Closed rjgildea closed 6 years ago

rjgildea commented 6 years ago

The button between "index" and "refine" is currently named "reindex" which I think is potentially confusing. The most important thing about this step is not the reindexing, but the determination of the bravais/lattice symmetry. The labelling of this button should somehow emphasise this.

rjgildea commented 6 years ago

Perhaps the "reindex" tab should be removed altogether, and simply automatically run RBS and reindex immediately after indexing in P1 (if space_group has not been set)?

dagewa commented 6 years ago

I think always running RBS automatically after indexing makes sense. Then the lattice table becomes the result of indexing. Further steps can be greyed out until a choice is made. If space_group was set then the choice is automatic, but it is still useful to have run RBS so that later the user can go back and pick a different option for another branch of processing.

dagewa commented 6 years ago

Closing as the button is now renamed, but running RBS automatically is still in discussion