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

Fix histogram output for single datapoint in dials.ssx_index #2156

Merged
merged 3 commits into from
Jun 29, 2022

Conversation

jbeilstenedmands
Copy link
Contributor

If you try to create a histogram with one datapoint, numpy can sometimes crash; if it doesn't the output is still not useful as the histogram bin width defaults to 1 which is far too large for this case.

@codecov
Copy link

codecov bot commented Jun 29, 2022

Codecov Report

Merging #2156 (54ac131) into main (d43c47f) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2156      +/-   ##
==========================================
- Coverage   80.59%   80.58%   -0.01%     
==========================================
  Files         585      585              
  Lines       66209    66211       +2     
  Branches     9340     9341       +1     
==========================================
- Hits        53359    53355       -4     
- Misses      10802    10804       +2     
- Partials     2048     2052       +4     

@jbeilstenedmands jbeilstenedmands merged commit 6cefa4f into main Jun 29, 2022
@jbeilstenedmands jbeilstenedmands deleted the ssx_index_histogram_bugfix branch June 29, 2022 12:28
ndevenish pushed a commit that referenced this pull request Jul 12, 2022
ndevenish pushed a commit that referenced this pull request Jul 12, 2022
This was referenced Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants