Issue 666: The "Add small molecule test nodes" checkbox on SkylineTester is obsolete and confusing, should be removed

Status:closed
Assigned To:Guest
Type:Defect
Area:Skyline
Priority:3
Milestone:4.3
Opened:2019-07-24 17:58 by Brian Pratt
Changed:2022-02-28 15:27 by Brian Pratt
Resolved:2019-09-25 13:32 by Brian Pratt
Resolution:Fixed
Closed:2022-02-28 15:27 by Brian Pratt
2019-07-24 17:58 Brian Pratt
Title»The "Add small molecule test nodes" checkbox on SkylineTester is obsolete and confusing, should be removed
Assigned ToGuest»Brendan MacLean
Type»Defect
Area»Skyline
Priority»3

2019-07-24 18:08 Brendan MacLean
Assigned ToBrendan MacLean»Brian Pratt
Yep. Agreed. Go for it.

2019-07-25 08:44 Brian Pratt
The core issue is that not all recent tests are set up to anticipate this extra document node, so a new developer might be surprised when using this option by accident. We don't really need it anymore - we now have actual small molecule data we can work with.

2019-09-25 13:32 Brian Pratt
resolve as Fixed
Statusopen»resolved
fixed in SHA-1: 532ed586a0416b7ef32f4ca3daa5e47efb28edea

* Skyline: when we first started supporting non-peptide data we didn't … (#594)

Skyline: when we first started supporting non-peptide data we didn't have a lot of examples. So we devised a special test mode that can add a non-peptide node to any test document on the fly. We don't really use it any more since we now have actual examples and tests, so now it just serves to confuse new Skyline developers. Thus I'm tearing it out to de-cruft the code base.

2022-02-28 15:27 Brian Pratt
close
Statusresolved»closed
Assigned ToBrian Pratt»Guest