CIRDLES / ET_Redux

EARTH-TIME.org's flagship data-processing and workflow automation software.
http://cirdles.org/projects/et_redux/
Apache License 2.0
12 stars 16 forks source link

Bug in sample manager #199

Closed bowring closed 3 years ago

bowring commented 3 years ago

Noted by @noahmclean : Thanks for looking into this. No need for files or data. Just do this. I’ve tried it on three computers – two Macs and One PC. The same thing happens every time.

Open Redux and create a new ID-TIMS sample. So, go to “Sample” -> “New Sample Analysis for ID-TIMS”. You can keep all the default sample names, values, etc for the new sample. Just create two new fractions and name them z2 and z1, in that order. Save the Sample (SaveAs Sample, SaveAs for Live Workflow, it doesn’t matter). This brings you to the main Redux window, looking at the blank data table. The fraction names will now be arranged in alphabetical order. Click Sample -> Manage Sample. The fractions are now in alphabetical order here. Click Close and Save. You’re now back at the main Redux window with the blank data table. Fraction names have now been duplicated. If you entered two fractions, z2 and then z1, they are now both labelled z1. If you entered five fractions, there are probably two z1s and two z3s.

Some other observations:

If you create new fractions (by default named F-1 through F-n), as long as the new fraction names are in alphabetical order everything is fine. If any one new fraction is out of order, unpredictable fraction duplication results. So: z1, z2, z3, z4, z5 is fine, and clicking Manage Sample has no effect, but ordering the fractions z5, z1, z2, z3, z4 reproducibly results in the error. The duplicate fraction names persist when you close and re-open Redux and then re-open a corrupted file. Sometimes new fractions get created. You might start with five fractions, hit Manage Sample, and end up with seven or eight. No error or exception is logged to the console when running from the terminal. The duplicate fraction naming (plus other hijinks) also occurs if you go to New Sample Analysis for ID-TIMS, then hit the Import Fraction Files button and import some existing fraction xml files, again so that they appear out of alphabetical order.

bowring commented 3 years ago

Fixed in release 3.6.93 April 2021