-
Notifications
You must be signed in to change notification settings - Fork 4
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
v1.2.0 development PR #20
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
Updated implementation of unit tests to capture messages and errors and split testing into smaller units within single file
Added external script to generate example data for each class and method used in the package for quick loading and use in testing and documented examples
Updated plotActivities to set input as NULL if not specified
Added unit testing for plotActivites to meet reasonable code coverage
No code just file to contain the tests for plotSegment
Added testing for getSamples and getSamplefeatures
Added getSampleByComponent testing using testthat
Added tests for user-facing "get" methods
Added testthat testing for scaleByModel using independent data, testing for clinPredictionPlatinum and depreciated clinPredictionDenovo as it does not function as intended.
Implemented testing for ClinPredictDenovo and depreciated. Added testing for getPloidyfeat, getSegCounts, and avoidMeasurementErrors. Added testing for addSampleFeatures and improved data.frame merging to return only samples present in object rather than all available rows in sample.feature data. Fixed requirement for all samples to match between data sets to only require one match.
Improved and fixed data input where header does not match expected input or includes columns in addition to the required columns. Adds reordering to ensure columns are in the correct order.
Completed depreciation of clinPredictionDenovo function and updated associated examples, vignette calls, and return value.
Added testing units for CNQuant and SigQuant subsetting functions
Added test for idSmoothingTargets function
Fixed a bug where specifying a single core resulted in a different computation of components and signatures due to a missing argument on an internal function. Calculation now correctly removes normal segments - rerunning signature extraction is recommended.
Added error catch to warn and prevent errors when no samples have sufficient seg counts to calculate copy number signatures
Computation of nSamp was using full sample set not filtered sample set - impact is minimal as step is partially redundant but caused edge case bug in GEL implementation.
Added functionality to report and add metadata to sample features for samples with too few non-diploid segments. Added parameter to specify non-diploid segment threshold. Removed unassigned use of SMOOTHINGFACTOR.
Fixed a bug in which CNSig class object initialisation date was fixed on package load rather than object instance creation. Added method to check and set column type for segTable input. Refactored createCNQuant to remove replicate code block.
Updated version number for dev to main PR
Refactored code comparing class identity to resolve build check warning. Commented out redundant class comparison lines related to QDNAseq class objects which are handled beforehand.
Updated the description to remove unnecessary import of default packages `base` and `parallel`, the namespaces of which were not called, to resolve a R build warning.
Regenerate example class objects to remove activity errors caused by inclusion of normal segments due to bug in single vs multicore implementation.
Added additional testing line in example_data.R file used for development testing.
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.
PR for dev branch