-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feat/aut789] - Walkthrough: Targeted flux analysis with GC-MS SIM Agilent #474
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking quite nice.
I forgot to mention this on the other LC_MS/MS method, but I will mention it here: it would be great to have a screenshot of the chromatograms for a complete MDV (i.e., all of the transitions for e.g., lactate or some other comonent_group) to help visualize what an MDV is and how it can be inspected. Also for the HeatMap, I would recommend using the "Peak_apex_int" instead of LogSN.
It looks like there is a mismatch between the TraML file transitions, and the components defined in the quantitation methods and FeatureFilters/QCs.
I have updated the screenshots. Regarding the mismatch, I have compared the file structures in other examples and had an initial thought that the transition_group_id was swapped with the transition_name column so I tried that out and it has caused SmartPeak to crash. Could you please elaborate with an excerpt from both files? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice updated.
The issue is that the compounds between the TraML and FeatureFilter... or FeatureQC... files are not consistent. The names provided in the transition_group_id names in the traML should be provided in the ...ComponentGroups... files and the transition_name names in the traML should be provided in the ...Components... files. Please let me know if that makes sense.
The |
OK great. Thanks for updating that. I must have just missed it then. Does the workflow run or are you still encountering errors? |
I just reran it with the latest version without any issues. |
Objective