Skip to content

Conversation

@SimonHeybrock
Copy link
Member

@celinedurniak were there others that are not needed that you had in mind?

@celinedurniak
Copy link
Collaborator

What are RunType and ScatteringRunType?

@SimonHeybrock
Copy link
Member Author

SimonHeybrock commented Sep 26, 2024

What are RunType and ScatteringRunType?

These a type-variables, which will (in the case of powder) take values such as SampleRun or VanadiumRun. Any data-reduction step that performs identical computation for several run types uses these. Here is an example:

def convert_to_dspacing(
data: MaskedData[RunType],
graph: ElasticCoordTransformGraph,
calibration: CalibrationData,
) -> DspacingData[RunType]:

This function will/can be used multiple times in the reduction workflow graph, typically for sample and vanadium runs.

You are right though that ScatteringRunType was unused, removed it now!

@celinedurniak
Copy link
Collaborator

And 'NexusSource' is the input raw file?

@SimonHeybrock
Copy link
Member Author

And 'NexusSource' is the input raw file?

It is the NXsource group loaded from a Nexus file. See https://scipp.github.io/essreduce/generated/modules/ess.reduce.nexus.generic_types.html for full list of types and explanations.

@celinedurniak
Copy link
Collaborator

Thanks for the explanations.
So, to answer the first question, at the moment, I do not see any more terms to remove.

@SimonHeybrock SimonHeybrock merged commit a0f8015 into main Sep 26, 2024
@SimonHeybrock SimonHeybrock deleted the remove-unused-domain-types branch September 26, 2024 09:11
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.

3 participants