forked from wtsi-npg/npg_tracking
-
Notifications
You must be signed in to change notification settings - Fork 0
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
merge from devel to master #18
Merged
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
prep release 92.8.1
pull from devel to master to create release 92.8.1
Update test data as well.
Add a new instrument format record for NovaSeqX.
This instrument type is no longer classified as a sequencing instrument. It is recognised as a cBot. This instrument type is added as a new type to the graphical listing of instruments.
Correct handling of the 'cBot 2' type instrument
Parse RunParameters.xml and RunInfo.xml to retrieve necessary information. Added a new method - platform_NovaSeqX. Added tests for this new method and some existing methods.
Ability to retrieve info about a NovaSeqX run.
Reimplemented the 'Instrument Types' list on the tracking runs web page so that new NovaSeq models apear there automatically. Excluded very old instrument models.
Tracking server - Novaseqx UI representation
Removed provisions for cif files.
No wash pending status for NovaSeqX
Identification of run completion for NovaSeqX
Updated cycle and tile counting
Record all recent changes
Previous values were causing issues in Jammy. New values use more resources but service is more stable. Originally found at: https://dev.to/joetancy/php-fpm-with-apache2-2mk0 https://thecustomizewindows.com/2023/04/how-to-install-apache-mpm-worker-and-php-fpm-on-ubuntu-server/
Single change event cfg so we can bugfix release
Update devel with bugfix release from master
Limit DRAGEN analysis to human samples only.
pin dbd::mysql to 4.050
Moved tests for aggregation across lanes to a new test scripts. Tests for a new merge by library method will be added to this new test script.
Also refactored the method itself.
We use the name of the Application in RunParameters.xml to detect that the instrument belongs to the NovaSeq type. Newer NovaSeqX instrument we have has the following definition for the application: <Application>NovaSeqXSeries Control Software</Application> This pull request fixes the reg exp we use to ensure that the above definition does not result in the instrument being classified as NovaSeq.
Distinguish between NovaSeq and NovaSeqX RunParams.
Ensure insert size is returned for compositions.
Added a method for aggregation by library
Added records for undocumented changed.
Fixed incorrect string comparison
Name variables consistently. Source dependencies from the branch that is the target.
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.
No description provided.