Skip to content

fix: predictLocalization, setParam and copyToComps#596

Merged
edkerk merged 9 commits intodevelopfrom
fix/localization
Jan 15, 2026
Merged

fix: predictLocalization, setParam and copyToComps#596
edkerk merged 9 commits intodevelopfrom
fix/localization

Conversation

@edkerk
Copy link
Member

@edkerk edkerk commented Dec 10, 2025

Main improvements in this PR:

  • fix:
    • predictLocalization and parseScores to correctly handle output from CELLO
    • setParam report reactions not found in model
    • mergeModels will only make new metFrom, rxnForm and geneFrom fields if none of the initial models have these fields
    • copyToComps does not add or modify metFrom, rxnForm or geneFrom fields
    • addRxns makes sure that subSystems are nested cell arrays
    • fitTasks parses correct arguments to fillGaps
  • refactor:
    • exportModel speed-up subSystems handling
  • feat:
    • runRAVENtests runs RAVEN unit tests
  • doc:
    • parseTaskList clarifies that it can handle boht *.txt and *.xlsx as input

Instructions on merging this PR:

  • This PR has develop as target branch, and will be resolved with a squash-merge.
  • This PR has main as target branch, and will be resolved as descriped here.

@edkerk
Copy link
Member Author

edkerk commented Jan 15, 2026

Tests passed locally

@edkerk edkerk merged commit c246045 into develop Jan 15, 2026
1 check failed
@edkerk edkerk deleted the fix/localization branch January 15, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant