achim1 / muonic

A python gui for QNet DAQ card experiments
http://achim1.github.io/muonic/
1 stars 9 forks source link

Is it wanted that the trigger is "off" (so not displayed) if coincidence is set to single? #55

Closed aschulzdesy closed 9 years ago

aschulzdesy commented 9 years ago

If one sets the setting to single, the trigger is not displayed. Which does not make sense to me, I thought that the trigger is then set to any event, so should have a much higher rate. I think this was different before.

hpbretz commented 9 years ago

I agree, it also helps to demonstrate how the trigger works. I removed the check that disables the trigger in case of single coincidence. Needs test with a real detector, the simulation mode ignores the settings apparently.

achim1 commented 9 years ago

Yes it was explicitely requested and we implemented it. There was also a long discussion at that time what trigger even means and how to demonstrate what a trigger is to students.. We probably should not change it, as people are used to the behaviour - but that's only my 2 cents

hpbretz commented 9 years ago

I wasn't part of that discussion, and I don't want to start it now. So someone just decide how we should do it, I can quickly change it back.

aschulzdesy commented 9 years ago

I also was not part of the discussion. But just spoke to Caro and especially since we changed "coincidence" to "trigger condition" she agrees with having it as I proposed. So trigger also give a value when set tot single. So I think it is fine like it is implemented in version 2.0.20.