-
Notifications
You must be signed in to change notification settings - Fork 22
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
Multi-record MassBank format #110
Comments
very, very interesting idea and I had a lot of thoughts on this while reading this paper (I was lucky enough to get a sneak peek) which is now finally out :-) Quick comment: why repeat all the duplicate information? This could be done much more compactly by having e.g. all compound/measurement specific data at the top and only repeating spectrum-specific parameters. This would obviously have to be defined carefully (and has pros and cons). I could say a lot more ... many thoughts and conversations on this .... |
@ChemConnector @meowcat I'm sure you also have thoughts :-) as will many others .. |
I like the idea of a merged record format. But, I would not merge information of different records into a header. The problem is not the general size of those spectral library files. There is enough space to story large files. The limitation of Merging of information from records ends in many problems of conventions, typos etc. And the vendors and users need specific programming programs reading headers, searching for the records, merge all and then write to the internal library. Importing one by one is much easier (see |
Currently, we see no point in introducing a multi-record format, because we are far away from file system limitations. Hence, closing |
Hi, we currently specify one record per file. If we have immense growth,
this might hit filesystem limitations. As an exchange format we could envision
to specify and allow multi-record MassBank files.
Fun exercise:
This might also be interesting in the light of https://github.com/HUPO-PSI/SpectralLibraryFormat
Yours,
Steffen
The text was updated successfully, but these errors were encountered: