This is the load script for the peak annotations and edited test files.
Note that the format detection was implemented before what was implemented in parallel in main. The version in this PR is flexible in that it doesn't require all sheets or all headers. It's not a perfect implementation, but it currently works just as well. It could be made to be better by trying to use optional columns/sheets to identify format.
Affected Issues/Pull Requests
Resolves #825
Merges into #998
Next PR: #912 (a parallel PR series, but it needs merged before the PRs above this one)
This completes this PR series. PR #1000 isn't branched off this PR's branch, but it does have some commented code that depends on code implemented somewhere in this series, that will be uncommented once this series is merged and it is rebased.
Review Notes
See comments in-line.
Checklist
This pull request will be merged once the following requirements are met. The
author and/or reviewers should uncheck any unmet requirements:
Review requirements
Minimum approvals: 1
No changes requested
All blocking issues resolved by reviewers
Specific reviewers: @add_username_here
Review period: 2 days
Associated issue/pull request requirements:
[x] All requirements in affected issues marked "resolved" are satisfied
[x] All required pull requests are merged (or none)
hepcat72
commented
2 months ago
Summary Change Description
This is the load script for the peak annotations and edited test files.
Note that the format detection was implemented before what was implemented in parallel in main. The version in this PR is flexible in that it doesn't require all sheets or all headers. It's not a perfect implementation, but it currently works just as well. It could be made to be better by trying to use optional columns/sheets to identify format.
Affected Issues/Pull Requests
This completes this PR series. PR #1000 isn't branched off this PR's branch, but it does have some commented code that depends on code implemented somewhere in this series, that will be uncommented once this series is merged and it is rebased.
Review Notes
See comments in-line.
Checklist
This pull request will be merged once the following requirements are met. The author and/or reviewers should uncheck any unmet requirements:
changelog.md(or no change)