Menu

#98 dta2ms2.py needs fixing

Crux v2.0
closed
sjoemac
None
2014-01-14
2014-01-13
No

Python script dta2ms2.py in src/python produces ms2 files where the mass on the Z line is one proton too low (i.e. is for precursor neutral mass, rather than M+H). The mass on the S line should be checked for correctness also when the Z line mass is fixed.

Related

Issues: #98

Discussion

  • sjoemac

    sjoemac - 2014-01-13
    • assigned_to: sjoemac
     
  • sjoemac

    sjoemac - 2014-01-13

    Could you try this patch?

    Thanks,
    Sean

     
    • Jeff Howbert

      Jeff Howbert - 2014-01-13

      Hi Sean,

      I didn't receive any attachment. ???

      Jeff

      On Mon, 13 Jan 2014, sjoemac wrote:

      Could you try this patch?

      Thanks,
      Sean

      Attachment: dta2ms2.diff (903 Bytes; application/octet-stream)


      ** [issues:#98] dta2ms2.py needs fixing**

      Status: open
      Created: Mon Jan 13, 2014 09:21 PM UTC by Jeff Howbert
      Last Updated: Mon Jan 13, 2014 09:25 PM UTC
      Owner: sjoemac

      Python script dta2ms2.py in src/python produces ms2 files where the mass on the Z line is one proton too low (i.e. is for precursor neutral mass, rather than M+H). The mass on the S line should be checked for correctness also when the Z line mass is fixed.


      Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/cruxtoolkit/issues/98/

      To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/

       
      • William S Noble

        William S Noble - 2014-01-13

        To get it you have to go to the sourceforge issue tracker. It's attached
        there.

        Bill

        On Mon, Jan 13, 2014 at 1:40 PM, Jeff Howbert howbert@users.sf.net wrote:

        Hi Sean,

        I didn't receive any attachment. ???

        Jeff

        On Mon, 13 Jan 2014, sjoemac wrote:

        Could you try this patch?

        Thanks,
        Sean

        Attachment: dta2ms2.diff (903 Bytes; application/octet-stream)

        Status: open
        Created: Mon Jan 13, 2014 09:21 PM UTC by Jeff Howbert
        Last Updated: Mon Jan 13, 2014 09:25 PM UTC
        Owner: sjoemac

        Python script dta2ms2.py in src/python produces ms2 files where the mass
        on the Z line is one proton too low (i.e. is for precursor neutral mass,
        rather than M+H). The mass on the S line should be checked for correctness
        also when the Z line mass is fixed.


        Sent from sourceforge.net because you indicated interest in
        https://sourceforge.net/p/cruxtoolkit/issues/98/

        To unsubscribe from further messages, please visit
        https://sourceforge.net/auth/subscriptions/


        Status: open
        Created: Mon Jan 13, 2014 09:21 PM UTC by Jeff Howbert
        Last Updated: Mon Jan 13, 2014 09:28 PM UTC
        Owner: sjoemac

        Python script dta2ms2.py in src/python produces ms2 files where the mass
        on the Z line is one proton too low (i.e. is for precursor neutral mass,
        rather than M+H). The mass on the S line should be checked for correctness
        also when the Z line mass is fixed.


        Sent from sourceforge.net because you indicated interest in
        https://sourceforge.net/p/cruxtoolkit/issues/98/

        To unsubscribe from further messages, please visit
        https://sourceforge.net/auth/subscriptions/

         

        Related

        Issues: #98

  • Jeff Howbert

    Jeff Howbert - 2014-01-13

    The Z line mass in the ms2 file doesn't quite agree with the M+H mass in dta (off in the 3rd or 4th decimal place). This is because the script first subtracts mass_h, then adds back mass_p. Wouldn't it be preferable to use mass_p throughout, and get rid of mass_h? This would be arguably closer to physical reality.

    I checked a couple of S line masses, and they look right.

     
  • sjoemac

    sjoemac - 2014-01-13

    Ok, how about this one?

     
  • Jeff Howbert

    Jeff Howbert - 2014-01-14

    That works perfectly on the particular set of 10 dta files I wanted to convert. I didn't test it on anything else.

    I can't say how it would do if there were multiple dta files with different charge states per scan. As I read the code, it looks like it would only handle up to 2 dta files/charge states per scan. Is this correct?

     
  • Jeff Howbert

    Jeff Howbert - 2014-01-14

    How does it work from here? Will you (Sean) commit the patched version?

     
  • sjoemac

    sjoemac - 2014-01-14

    Yes, it can handle only up to two charges per scan. Since we only observed that with the dta files that we were given, that is what we stuck with. We can extend it to handle multiple charges but I think that there may be a converter in proteowizard that we should check out. If you okay the current version, I'll take that as a code review pass and commit the new version. Then I'll work on the next dependent issue.

     
  • Jeff Howbert

    Jeff Howbert - 2014-01-14

    OK, consider it officially code-reviewed and approved. Thanks for fixing it.

     
  • sjoemac

    sjoemac - 2014-01-14
    • status: open --> closed
     
  • sjoemac

    sjoemac - 2014-01-14

    Fixed as of rev. 16288

     

Log in to post a comment.