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

Remove frame unwrapping and WFM code #592

Merged
merged 6 commits into from
Jan 28, 2025
Merged

Remove frame unwrapping and WFM code #592

merged 6 commits into from
Jan 28, 2025

Conversation

nvaytet
Copy link
Member

@nvaytet nvaytet commented Jan 22, 2025

This code is moving to essreduce: see scipp/essreduce#163

@nvaytet nvaytet marked this pull request as ready for review January 23, 2025 13:33
@nvaytet nvaytet requested a review from SimonHeybrock January 23, 2025 13:33
@nvaytet
Copy link
Member Author

nvaytet commented Jan 23, 2025

Don't know if we want to keep the fakes. Right now they were only used in the unwrap/wfm tests which are gone.

@SimonHeybrock
Copy link
Member

Don't know if we want to keep the fakes. Right now they were only used in the unwrap/wfm tests which are gone.

I think they can be removed? Where they also added in essreduce?

@nvaytet
Copy link
Member Author

nvaytet commented Jan 27, 2025

I think they can be removed? Where they also added in essreduce?

Not exactly. The FakeBeamlineEss was moved over, but not the FakeBeamline you initially created. Also, the choppers were not moved, I made some new ones which are using DiskChopper instead of tof.chopper_cascade.Chopper.

@SimonHeybrock
Copy link
Member

Do I understand correctly that your last commit does remove them now?

@nvaytet
Copy link
Member Author

nvaytet commented Jan 28, 2025

Yes, I removed them.
I'm of the opinion that it's ok to remove, but I thought we could revert if you prefered to keep them.

@nvaytet nvaytet merged commit 167d70f into main Jan 28, 2025
5 checks passed
@nvaytet nvaytet deleted the remove-unwrap branch January 28, 2025 13:24
@nvaytet
Copy link
Member Author

nvaytet commented Jan 31, 2025

I think I forgot to remove the dependency on tof...

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