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

DGS Reduction stuck if reduce with no monitor set #11602

Closed
lottiegreenwood opened this issue Dec 11, 2014 · 7 comments · Fixed by #15386
Closed

DGS Reduction stuck if reduce with no monitor set #11602

lottiegreenwood opened this issue Dec 11, 2014 · 7 comments · Fixed by #15386
Assignees
Labels
Direct Inelastic Issues and pull requests related to direct inelastic Low Priority Things that you don't ever want to be done.
Milestone

Comments

@lottiegreenwood
Copy link

This issue was originally TRAC 10760

When filling in DGS Reduction form, if you enter nothing into the Monitor Spectrum IDs box, then click reduce, the Reduce button will grey out and not become clickable again until you exist the DGS Reduction form and re-open it.

@lottiegreenwood
Copy link
Author

@NickDraper (2015-04-27T08:10:35):
Moved to R3.5 at the R3.4 code freeze

@lottiegreenwood lottiegreenwood added Low Priority Things that you don't ever want to be done. Direct Inelastic Issues and pull requests related to direct inelastic labels Jun 3, 2015
@lottiegreenwood lottiegreenwood added this to the Release 3.5 milestone Jun 3, 2015
@NickDraper NickDraper modified the milestones: Release 3.5, Release 3.6 Sep 14, 2015
@NickDraper NickDraper modified the milestones: Release 3.6, Release 3.7 Jan 22, 2016
@NickDraper NickDraper changed the title Inconsistency in way DGS Reduction handles non-entry of information DGS Reduction stuck if reduce with no monitor set Feb 19, 2016
@NickDraper
Copy link
Contributor

One for you to assign on

@AndreiSavici
Copy link
Member

@lottiegreenwood could you send me an example for inputs? Instead of clicking reduce, please click export. Attach the xml file to this ticket. If the file you want to reduce is not in the repository, please attach that one as well.

@AndreiSavici
Copy link
Member

@lottiegreenwood I think I see how you produced the error. Did you delete the inputs in the monitor id boxes?

@NickDraper
Copy link
Contributor

Actually, I can help with this one.

Just start up the DGS reduction interface and delete the default monitor 1 and 2, so it is monitors "empty box" and "empty box", then click reduce. Then the reduce button left left disabled.

@NickDraper
Copy link
Contributor

Yes

@lottiegreenwood
Copy link
Author

@AndreiSavici Yes. I tried to export the example but (I'm assuming because it's just an entirely empty dialogue) it wouldn't save the file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Direct Inelastic Issues and pull requests related to direct inelastic Low Priority Things that you don't ever want to be done.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants