Updated CE spectra. Corrected QTOF data, removed extra data. #104
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@nilshoffmann, Thanks a lot for your submission. I have a question reagarding the re-use of record accessions. Some of the records changed completely. You shall avoid to re-use accessions due to trackability of the record history. Someone may used your records for identification and replacement is thus not useful. To deprecate a record, add the deprecation tag to the record and just push it. The record will remain in your repository, but will disapear on MassBank.
Example: |
|
Thanks for the clarification. I will fix this and update my pull request!
… Am 22.11.2019 um 22:06 schrieb Tobias Schulze ***@***.***>:
@nilshoffmann, Thanks a lot for your submission. I have a question reagarding the re-use of record accessions. Some of the records changed completely. You shall avoid to re-use accessions due to trackability of the record history. Someone may used your records for identification and replacement is thus not useful.
To deprecate a record, add the deprecation tag to the record and just push it. The record will remain in your repository, but will disapear on MassBank.
A deprecated record is marked with the tag DEPRECATED: followed by date and reason on the second line. The content of a deprecated record may not be valid.
Example:
ACCESSION: XY000010
DEPRECATED: 2019-05-03 considered noisy
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Merged
|
Closing this PR. An updated PR with a corrected history is here: #105 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Unfortunately, the original submission contained mislabeled QTOF data and also contained some extra files that have now been removed. If we need to reassign the record numbers somehow, please advise, how this should be done.
Also, for some of the mediators, I have corrected the CHEBI / PUBCHEM links to point to the non-labeled compounds, while the rest of the record still follows the regular MassBank conventions.
Validation checks run and only report InChI warnings concerning omission of undefined stereo.