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

Single shot routines run automatically upon all manually loaded shots #9

Closed
philipstarkey opened this issue Feb 4, 2015 · 4 comments
Labels
bug Something isn't working major

Comments

@philipstarkey
Copy link
Member

Original report (archived issue) by Russell Anderson (Bitbucket: rpanderson, GitHub: rpanderson).


Active single-shot routines run upon manually loading shots into the GUI. Formerly, single-shot routines would only run on new shots sent from BLACS, or those which were "Mark[ed] as not done". This is a problem when loading a lot of shots in, and/or when those shots have already been analysed by the single-shot routine. In the latter case, the routine may have changed (e.g. use different analysis parameters), and the routine will undesirably override previous analysis results.

@philipstarkey
Copy link
Member Author

Original comment by Chris Billington (Bitbucket: cbillington, GitHub: chrisjbillington).


Could you test if this was fixed by 4558c2f ?

@philipstarkey
Copy link
Member Author

Original comment by Chris Billington (Bitbucket: cbillington, GitHub: chrisjbillington).


Actually you don't have to test, I did. Closing, but let me know if for some reason I've overlooked something.

@philipstarkey
Copy link
Member Author

Original comment by Chris Billington (Bitbucket: cbillington, GitHub: chrisjbillington).


  • changed state from "new" to "resolved"

@philipstarkey
Copy link
Member Author

Original comment by Russell Anderson (Bitbucket: rpanderson, GitHub: rpanderson).


My description wasn't wholly accurate; it's a bit more subtle.

It's actually that new shots (manually loaded or network-submitted) are not marked as done (analysis complete) when being loaded into lyse when there are no active single-shot routines.

Thus the check-box next to the single-shot routines "didn't work", in the sense that if you deactivated all single-shot scripts, then loaded shots in, they wouldn't be marked as done. Then reactivating any single shot routine would result in it running on all the shots you just loaded.

This is exactly what 4558c2f fixed.

Note: for those following along at home, lyse does not distinguish manually loaded shots from those submitted over the network at present. This may the subject of a future feature, whereby the user is prompted to mark all manually loaded shots as done upon loading them in.

@philipstarkey philipstarkey added major bug Something isn't working labels Apr 5, 2020
Loki27182 pushed a commit to Loki27182/lyse that referenced this issue Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working major
Projects
None yet
Development

No branches or pull requests

1 participant