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

Test tabular batch widget 'GUI' #201

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

olihammond
Copy link

This is a very preliminary attempt at having a table-based multiple run reduction using the LoKI workflows and locally stored nxs , mask, and direct beam datafiles. I think there is a lot that can be done to make this 'smarter' and more dynamic by integrating some of the previous workflow widget features. However, this could be seen as a step towards a more representative ess sans "GUI".

This is a very preliminary attempt at having a table-based multiple run reduction using the LoKI workflows and locally stored data.
@olihammond olihammond marked this pull request as draft February 28, 2025 13:17
@SimonHeybrock
Copy link
Member

SimonHeybrock commented Mar 6, 2025

Thanks for bringing a concrete proposal! Is this in any way related to what @jokasimr has put together for reflectometry? Can we look into finding a unified approach?

@olihammond
Copy link
Author

olihammond commented Mar 6, 2025

Thanks for bringing a concrete proposal! Is this in any way related to what @jokasimr has put together for reflectometry? Can we look into finding a unified approach?

Excellent question. I'd be very happy to see more unification; certainly the tabular input was strongly inspired by the reflectometry work by @jokasimr. Perhaps there are components both could use from the other, or some smart refactoring and condensing that could be done. I'm not sure how much we (or at least, I and my limited talents) can achieve this, because we run into a lot of technique-specific quirks quite quickly, namely, plotting and the manner of interaction with the underlying Scipp workflows.

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.

2 participants