Repository navigation
Fix genomic target accessibility boundaries - #7
Merged
martin-raden merged 4 commits intoOct 26, 2026
Merged
Conversation
Member
|
thanks, please extend: NA use in outputgiven the discussion in PatrickRWright#22 it is favorable to use empty strings instead of placeholding NA values within the final output tables to simplify reading by the user and importing into subsequent services DAVID email addressreplace the email address used to access the DAVID service with "rna@informatik.uni-freiburg.de" |
Collaborator
Author
|
Addressed the review feedback in 3adace6: published CopraRNA result and web tables now emit empty fields for unavailable values while internal scientific tables retain missing-value semantics, and DAVID authenticates with rna@informatik.uni-freiburg.de. Release-style validation passes with 254 tests and 1 optional browser skip; Ruff, diff checks, and source/wheel builds also pass. |
Fix formatting and punctuation in README.md regarding DAVID service requirements and report generation.
Updated email account for DAVID requests and clarified CSV import handling.
This file contains hidden or 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
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.
Genomic target extraction previously made the requested region's ends artificial folding boundaries. Add genomic context using IntaRNAsTar's target accessibility base-pair span, then restrict interaction prediction to the original region with
--tRegion. Report sequences and all target/seed/structure coordinates remain relative to the requested region.tAccL/accL, native zero/window semantics, circular wrapping and physical linear ends across both strands and all extraction modes. Preserve existing spliced CDS and short-truncated-UTR behavior.E_normbefore filtering and statistical processing.NAplaceholders in published result/web tables while preserving internal missing-value semantics, and authenticate DAVID withrna@informatik.uni-freiburg.de.Validation: 254 passed, 1 skipped, with
COPRARNA_PIPELINE_TESTS=1andCOPRARNA_REFERENCE_TESTS=1, using the release's pinned Python libraries. This includes real IntaRNAsTar 3.4.1 predictions, file/inline settings equivalence, web exports, and extraction from all 17 reference genomes. The skip is the optional browser integration. Independently captured scientific fixtures are unchanged. Source/wheel builds succeeded; 79 accessibility tests passed against the installed wheel outside the source tree, including native predictions. Ruff andgit diff --checkpassed.Targets
codex/issue-4-release-3as requested in the issue.Closes #6.