Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove % from dials.index % indexed column. #2603

Merged
merged 2 commits into from
Feb 20, 2024

Conversation

toastisme
Copy link
Contributor

Changes dials.index output from

+------------+-------------+---------------+-------------+
|   Imageset |   # indexed |   # unindexed | % indexed   |
|------------+-------------+---------------+-------------|
|          0 |         204 |             1 | 99.5%       |
+------------+-------------+---------------+-------------+

Saving refined experiments to indexed.expt
Saving refined reflections to indexed.refl

to

+------------+-------------+---------------+-------------+
|   Imageset |   # indexed |   # unindexed |   % indexed |
|------------+-------------+---------------+-------------|
|          0 |         204 |             1 |        99.5 |
+------------+-------------+---------------+-------------+

Saving refined experiments to indexed.expt
Saving refined reflections to indexed.refl

Copy link
Member

@dagewa dagewa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the example the heading (% indexed) also went from left- to right-justified, but I don't see a change that does that? Could that also be added? Otherwise, yes, this is nicer.

@toastisme
Copy link
Contributor Author

In the example the heading (% indexed) also went from left- to right-justified, but I don't see a change that does that? Could that also be added? Otherwise, yes, this is nicer.

From the looks of it the columns are by default right-justified and the added % undid that.

@toastisme toastisme merged commit 9a39143 into dials:main Feb 20, 2024
9 of 11 checks passed
@toastisme toastisme deleted the remove_unit_from_indexed_column branch February 20, 2024 14:59
benjaminhwilliams pushed a commit that referenced this pull request Feb 27, 2024
* Removed % from % indexed column.
benjaminhwilliams pushed a commit that referenced this pull request Feb 27, 2024
* Removed % from % indexed column.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants