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

dials.scale, catch all possible edge cases where no reflections left after reflection selection #2146

Merged
merged 2 commits into from
Jun 21, 2022

Conversation

jbeilstenedmands
Copy link
Contributor

Triggered a stacktrace for me during testing on very sparse data.

… be left

for a given dataset after reflection selection.
@jbeilstenedmands jbeilstenedmands enabled auto-merge (squash) June 21, 2022 10:36
@jbeilstenedmands jbeilstenedmands merged commit ab88cef into main Jun 21, 2022
@jbeilstenedmands jbeilstenedmands deleted the scaling_dataset_removal_bugfix branch June 21, 2022 11:56
ndevenish pushed a commit that referenced this pull request Jul 12, 2022
…after reflection selection (#2146)

Very much an edge case only triggered for very sparse data
ndevenish pushed a commit that referenced this pull request Jul 12, 2022
…after reflection selection (#2146)

Very much an edge case only triggered for very sparse data
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