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

Multi-rotation and multipin plans don't support certain combinations that are implied by the parameter model #563

Open
rtuck99 opened this issue Oct 10, 2024 · 0 comments
Labels

Comments

@rtuck99
Copy link
Contributor

rtuck99 commented Oct 10, 2024

Currently the plans only support a fixed number of frames per rotation, whereas the MultiRotationScan RotationScanPerSweep allows variable rotation width which implies variable frames if you have fixed exposure per frame. Also there's a fixed omega_start in the DetectorParams which we currently assume to be the same as the first.

We should limit the flexibility allowed by the model if we decide not to implement this functionality, or at least validate that these constraints are maintained.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Candidates for Next Sprint
Development

No branches or pull requests

2 participants