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-shot analysis routines still run when analysis is paused #14

Open
philipstarkey opened this issue Jun 9, 2015 · 1 comment
Open
Labels
bug Something isn't working minor

Comments

@philipstarkey
Copy link
Member

Original report (archived issue) by Shaun Johnstone (Bitbucket: shjohnst, GitHub: shjohnst).


If a multishot analysis routine returns an error (e.g. due to a failed fit) it pauses the analysis queue. However, as new shots come in, the single shot routines do not run, as expected, but multishot routines continue. In the case where the mutishot requires results from the single shot the error is unlikely to go away. In the case where the multishot does not rely on results from a single shot routine, the error may go away (e.g. once enough shots are in to allow it to fit) meaning that there are now no scripts in an error state, but analysis is paused.

To be consistent, multishot should pause when single shot is paused.

@philipstarkey
Copy link
Member Author

Original comment by Jan Werkmann (Bitbucket: PhyNerd, GitHub: PhyNerd).


should be fixed if you replace the last break in analysis_loop() with a return

#!python

                else:
                    logger.info('analysis is paused')
                    break
            if self.multishot_required and not self.analysis_paused:
                logger.info('doing multishot analysis')
                self.do_multishot_analysis()

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working minor
Projects
None yet
Development

No branches or pull requests

1 participant