jlingema / uGMTScripts

Script collection for upgraded GMT
1 stars 4 forks source link

Wishlist for testfiles #9

Closed dinyar closed 9 years ago

dinyar commented 9 years ago

Hi Joschka,

here a preliminary wishlist for the testbench pattern file. The current format (found in https://github.com/jlingema/uGMTScripts/tree/master/ugmt_patterns/data/patterns/emu) is more or less what the testbench accepts now, but the following additions/changes would be great:

Let me know if something can't be changed easily, it may be a fast fix in the testbench..

jlingema commented 9 years ago

Is position of XTRK important? I.e. before / after muons (before / after energies)?

dinyar commented 9 years ago

No, i only use the "internal ordering", so muons should be ordered from fwd+ to fwd- and tracks as well, but they could be interlaced.

On Wed, Nov 5, 2014, 13:19 Joschka Lingemann notifications@github.com wrote:

Is position of XTRK important? I.e. before / after muons (before / after energies)?

— Reply to this email directly or view it on GitHub https://github.com/jlingema/uGMTScripts/issues/9#issuecomment-61799036.

jlingema commented 9 years ago

A first version of this has been added to the repo: https://github.com/jlingema/uGMTScripts/tree/master/ugmt_patterns/data/patterns/testbench

Please check and let me know if something should be changed.

dinyar commented 9 years ago

This is working well with the first version of the testbench. Would it be possible though to remove the two newlines at the end of the pattern files?

jlingema commented 9 years ago

The new version does not have newlines anymore.

dinyar commented 9 years ago

Great, seems to work now! Closing.

dinyar commented 9 years ago

fyi: I just checked - at least one newline doesn't seem to crash the testbench.