Repository navigation
Replace R tools and modernize Python companions - #298
Open
Alexander-Mitrofanov wants to merge 6 commits into
Open
Alexander-Mitrofanov wants to merge 6 commits into
Alexander-Mitrofanov wants to merge 6 commits into
Conversation
Member
|
thanks, please revise: implement respective tasks with at least one commit per task or major step rebase
homogenize python tools
|
Alexander-Mitrofanov
force-pushed
the
feat/295-python-companion-scripts
branch
from
October 23, 2026 22:52
b4a9c99 to
7b1412b
Compare
Alexander-Mitrofanov
changed the base branch from
master
to
feat/294-python-intarna-fuse
October 23, 2026 22:52
This branch has not been deployed
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.
Replace the auxiliary R programs with Python output tools and fix the existing CopomuS and IntaRNApvalue workflows. The tools now share checked subprocess execution, RNA/FASTA and CSV validation, executable lookup, and atomic text output while retaining their distinct scientific models and existing command-line interfaces.
This PR depends on #296 and targets
feat/294-python-intarna-fuse. The R migration, shared helpers, CopomuS revisions, IntaRNApvalue revisions, and packaging/documentation each have a separate commit.python/DEVELOPMENT.md.Validation on Linux:
make tests -j2: all 9 test groups pass, including 119 API cases / 462,689 assertions.python3 -m unittest discover -s python/tests -v: all 46 tests pass, including native sampling and mutation workflows.tests/testIntaRNAFuse.pywith native IntaRNA/RNAfold: all 10 tests pass, including independent fusion-energy checks.automake --warnings=portability,error,make install, andmake distpass. Distribution contents include shared modules, packaging metadata, documentation and regressions, and exclude the removed R/Perl directories and Python caches.b4a9c99produced identical mutation order, ranks and scores after newline normalization. Three one-thread runs had median wall times of 2.94 s before / 2.23 s after, with peak RSS about 17.3 MiB in both. A fixed-seed long-sequence shuffle comparison was byte-identical; timings and commands are recorded in the audit.git diff --checkpass.Out-of-tree C++ validation used the existing source-include/pkg-config/CLI-fixture workarounds. No C++ predictor behavior is changed. A platform-specific PyInstaller executable was not built; the builder's invocation, path handling and failure propagation are covered by tests.
Fixes #295.