-
Notifications
You must be signed in to change notification settings - Fork 19
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
Record all recent changes #773
Merged
Merged
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
@@ -1,5 +1,35 @@ | ||||||
LIST OF CHANGES | ||||||
|
||||||
- Removed the use of 'xml' lims driver from all tests. | ||||||
- Deleted 'xml' lims driver class and all classes, which supported this | ||||||
functionality, together with all related tests and test data. | ||||||
- In release 97.0.0, ihe default LIMS driver type in samplesheet generation | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
was changed to 'ml_warehouse'. Following this change, the code for | ||||||
auto-generation of MiSeq default samplesheets is now simplified to exclude | ||||||
an explicit creation of LIMS objects. | ||||||
- Changes to st::api::lims, which are not related to the 'xml' lims driver | ||||||
removal: | ||||||
- deleted all previously deprecated methods ('seq_qc_state', | ||||||
'associated_lims', 'associated_child_lims', 'associated_child_lims_ia'); | ||||||
- deleted methods, for which we do not have data in ml warehouse, our | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
primary source of LIMS data ('request_id', 'project_id', 'project_name'); | ||||||
- deleted all code for computing inline tag indexes and inferring | ||||||
tag sequences and library types from sample description since | ||||||
definitions like this have not been in use for the last six years; | ||||||
- reimplemented the 'reference_genome' method to exclude a fallback to | ||||||
a study reference for tag zero and lane-level objects when samples | ||||||
have different references. | ||||||
- Restricted the 'all_lanes_mergeable' flag in the 'lonf_info' role to | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
NovaSeq Standard workflow, ie dropped an additional case of HiSeq Rapid Run | ||||||
since HiSeq instruments are no longer used. | ||||||
- Updated DBIx classes from the prod database. Changes are due to the MySQL | ||||||
database server upgrade to v8.+ | ||||||
- Finding the runfolder path - stopped errors being raised when a runfolder | ||||||
is found in 'analysis' or 'outgoing' in the presence of a duplicate runfolder | ||||||
in 'incoming' (the duplicates in 'incoming' are sometimes created by | ||||||
instruments well after the run was mirrored and the runfolder moved to | ||||||
'analysis'). The duplicate runfolder in 'incoming' is disregarded'. | ||||||
|
||||||
release 97.0.0 | ||||||
- Change CI workflow to standard file name | ||||||
- Change to Perl versions from Perlbrew | ||||||
|
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.
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.