Modifying adduct type textbox for small molecules crashes Skyline

support
Modifying adduct type textbox for small molecules crashes Skyline Chris Ashwood  2017-12-20 20:27
 
Dear Skyline support,

I'm using Windows 7 64-bit and Skyline daily (4.0.9.11664).

I've come across a crash that closes Skyline unexpectedly, likely due to my unintention methods. When I modify the adduct of an already-specified small molecule transition using the text box, e.g. removing the - from [M-2H] and replacing with +, Skyline instantly crashes (stack overflow, please find full log below).

When using the intended method (using the arrow on the side of the text box), it doesn't crash so I think I've performed adduct modification in an unexpected way.

I've attached the Skyline assay that caused the crash and please find the error log below:

Problem signature:
  Problem Event Name:    CLR20r3
  Problem Signature 01:    Skyline-daily.exe
  Problem Signature 02:    1.0.0.0
  Problem Signature 03:    5a3856df
  Problem Signature 04:    System.Windows.Forms
  Problem Signature 05:    4.7.2102.0
  Problem Signature 06:    59371148
  Problem Signature 07:    0
  Problem Signature 08:    ffffffff
  Problem Signature 09:    System.StackOverflowException
  OS Version:    6.1.7601.2.1.0.256.1
  Locale ID:    3081
  Additional Information 1:    1c98
  Additional Information 2:    1c98c898ba4042187feba63d99e9d7e6
  Additional Information 3:    ba93
  Additional Information 4:    ba935b4e6adfbd97b792a80604cf6fb1

Cheers,
Chris
 
 
Nick Shulman responded:  2017-12-21 14:10
Thanks, Chris,

We actually ran into this bug independently yesterday.
Whenever you make a change to that textbox, we have some code that tries to format what you typed into the standard format (by, for instance, putting square brackets around the whole thing if you removed them).
Unfortunately, the way we wrote the code it can easily get into an infinite loop of trying to reformat the formula.
This will be fixed in the next release candidate of Skyline 4.1.
 
Chris Ashwood responded:  2017-12-21 14:55
Hi Nick,

Awesome, good to hear it's been noted. Thanks to yourself and the rest of the Skyline team for all the support and great improvements to Skyline this year, Merry Christmas.

Cheers,
Chris