-
Notifications
You must be signed in to change notification settings - Fork 18
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
for QC/QA, analysis regions should be stored on the chromatogram #72
Comments
We use a similar approach in the NMR part, each processing step is recorded with
so it is possible to exactly see what was done to get the final result. |
In my opinion, that's a fine approach for NMR and FTIR, NIR, ... . In
the aforementioned cases, we have one scan, which contains the needed
spectral information. In case of chromatography, we do have ~1000 to
100.000 scans, which could lead to performance issues when flipping back
and forth trough the analysis steps.
Am 15.10.19 um 17:28 schrieb Christoph Läubrich:
…
We use a similar approach in the NMR part, each processing step is
recorded with
* time
* processor
* settings
so it is possible to exactly see what was done to get the final result.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#72?email_source=notifications&email_token=AAFHUT4KRYBHQUFKUWJK3CLQOXOQ5A5CNFSM4JAZWFX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBJGFRY#issuecomment-542270151>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFHUT5AGH73XXIJF6YZAFLQOXOQ5ANCNFSM4JAZWFXQ>.
--
~~~~~~~~~~~~~~~~~~~~~~~~
OpenChrom - the open source alternative for chromatography / mass spectrometry
Dr. Philip Wenig » Founder » [email protected] » http://www.openchrom.net
~~~~~~~~~~~~~~~~~~~~~~~~
|
I don't think we need the flipping back and forth feature for QC/QA in Chromatogram. It would be enough to have the region where a certain filter or processor was applied registered/stored somewhere. |
We have been discussing something similar for NMR spectra. |
The concept of a edit history, change logbook exists already:
IEditHistory editHistory = chromatogram.getEditHistory();
Probably, additional information can be placed in the history.
Am 16.10.19 um 16:29 schrieb Kerner:
…
We have been discussing something similar for NMR spectra.
it should be possible to persist a spectra, chromatogram, ...
selection. Ideally, this would also be stored inside the ocb file.
This could also be used for things like bookmarking or copy a
selection to a new chromatogram.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#72?email_source=notifications&email_token=AAFHUT4LCE4LVU2MDIQHSJTQO4QMBA5CNFSM4JAZWFX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBMWKTI#issuecomment-542729549>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFHUT2BGRIUSPSL224KORLQO4QMBANCNFSM4JAZWFXQ>.
--
~~~~~~~~~~~~~~~~~~~~~~~~
OpenChrom - the open source alternative for chromatography / mass spectrometry
Dr. Philip Wenig » Founder » [email protected] » http://www.openchrom.net
~~~~~~~~~~~~~~~~~~~~~~~~
|
From what I can see in the source, the edit history is only a string based log, not really suitable to store numeric information that shall be reusable/accessible from other classes in a modular way. However, range selections stored in an appropriate class could certainly also trigger an entry in the edit history. |
If a processor/filter etc. is run on a zoomed chromatogram, the information of the analysis regions should be stored on the chromatogram for QC/QA reasons. Else, it will not be possible to reproduce which modifications where made to which part of the chromatogram.
The text was updated successfully, but these errors were encountered: