Skip to content

Conversation

@sneumann
Copy link
Owner

No description provided.

paul.stahlhofen and others added 30 commits October 14, 2020 13:00
Fix warning in loadRmbSettings('mysettings.ini') in issue #257
Updating to latest version of MassBank/RMassBank dev
…nce again to avoid a duplicate. (This was added to "Suggests" in the updated and merged version as well)
Introduced new slots `AC$MASS_SPECTROMETRY$MS_SCAN_LOWER_LIMIT` and `AC$MASS_SPECTROMETRY$MS_SCAN_UPPER_LIMIT`.
Scan range is now stored as a string in the subtag `SCAN_RANGE_M/Z` of `AC$MASS_SPECTROMETRY`.
Example: `AC$MASS_SPECTROMETRY: SCAN_RANGE_M/Z "50-210"`
…he entry prefix is not 2. This would lead to the length of the accession number being different from 8 which makes it unusable for the databse. Hence, the user is informed via the warning that the so-constructed accession numbers should be used for internal purposes only.
Arbitrary number of letters in accession number prefix
Inclusion of scan range in record format
Fix Travis error caused by R version
… to address #175

TODO: address warnings caused by nested list
Note: The unlist function behaves very unexpectedly. See the comment in buildRecord.R, line 165ff for details

Typo correction: This branch actually addresses #176
@sneumann See new function `.addGenericInfo`
Creating more flexible structure in AC$CHROMATOGRAPHY to allow for third SOLVENT
Fix an issue if no electronic noise was specified in settings.ini, closes #239
Paul Stahlhofen and others added 30 commits July 5, 2021 14:13
…esent

- Replaced 'cat' by 'warning'
- Refrained from writing outliers to an extra csv-file
Added check whether peaks are in scan range
Updated DESCRIPTION and NAMESPACE to include the logger package
- no internal variable in the RMassBank environment is used anymore
- instead, an RMassBank logging namespace is created immediately in `settings_example.R`, if a logging file was given
- Removed update_appender method (not necessary anymore)

Renamed log_xxx functions to rmb_log_xxx
Update email of Erik and addition of Paul as an author
Adjusted rounding of mass range
# Conflicts:
#	DESCRIPTION
…d so

Windows builds are single arch only (i.e. 64-bit only)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants