Store CrystalStructure in Sample #14055
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This addresses #13905.
Sample now holds a CrystalStructure. I have not included it in the Nexus reading/writing of Sample yet, because it's still unclear how exactly the scatterers would be stored (ongoing discussion with Tobias Richter). However I do not think that this is urgent, because typically this information would come from a CIF-file and added to sample after loading the raw data into a workspace. This will be addressed in #11268, which is also the reason for exposing Sample::setCrystalStructure to Python - this way it's possible to use either a C++ or Python library for CIF-parsing.
Testing information
Code review. I marked it Core because Sample is present in all workspaces and it would be good to make sure nothing has been overlooked.