-
-
Notifications
You must be signed in to change notification settings - Fork 307
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
[REFACTOR] function to copy spectrum meta data / settings #3913
Conversation
output.setRT(input.getRT()); | ||
output.setMSLevel(input.getMSLevel()); | ||
output.setName(input.getName()); | ||
output.setType(SpectrumSettings::CENTROID); |
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.
output.setType(SpectrumSettings::CENTROID);
output.getFloatDataArrays().clear();
are effectively removed now. But it seems like they are needed.
src/openms/source/ANALYSIS/XLMS/OPXLSpectrumProcessingAlgorithms.cpp
Outdated
Show resolved
Hide resolved
output[scan_idx].setRT(input[scan_idx].getRT()); | ||
output[scan_idx].setMSLevel(input[scan_idx].getMSLevel()); | ||
output[scan_idx].setName(input[scan_idx].getName()); | ||
output[scan_idx].setType(SpectrumSettings::CENTROID); |
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.
information of type lost.
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.
hmm I see, this is not just copying over the settings but over-writing it.
src/openms/include/OpenMS/TRANSFORMATIONS/RAW2PEAK/PeakPickerIterative.h
Show resolved
Hide resolved
otherwise +1 |
what about the |
MSSpectrum does not inherit directly from MetaInfoInterface, so that should be covered by SpectrumSettings: OpenMS/src/openms/include/OpenMS/KERNEL/MSSpectrum.h Lines 67 to 70 in 1c5cf83
|
tests passed according to cdash |
extract single function to copy spectral settings (but not data) from one spectrum to another