Skip to content
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

fix: minor patches #374

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

fix: minor patches #374

wants to merge 10 commits into from

Conversation

edkerk
Copy link
Member

@edkerk edkerk commented May 8, 2024

Main improvements in this PR:

  • Fixes:
    • loadDatabases clarifies what to do if download from UniProt fails.
    • calculateFfactor sometimes failed to properly read paxDB.tsv files.
    • runDLKcat should parse params.path as absolute paths to docker (solves runDLKcat() cannot find path to data #378).
    • applyKcatConstraints can handle when all kcat are zero.
    • getStandardKcat should also assign standard kcat values for reactions that do have a non-empty grRules field, but are not in model.ec due to the genes not matching any of the enzymes in the uniprot.tsv file.
  • Features:
    • New removeStandardKcat function removes all traces of getStandardKcat having modified a model.
  • Chores:
    • Update dependencies in GitHub Actions.

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 with a merge commit.

Copy link

github-actions bot commented May 8, 2024

This PR has been automatically tested with GH Actions. Here is the output of the tests:

 
[�Warning: Cannot find RAVEN Toolbox in the MATLAB path, or the version is too old (before v2.8.3). Make sure you have installed RAVEN in accordance to the following instructions,
including running 'checkInstallation()': https://github.com/SysBioChalmers/RAVEN/wiki/Installation]�
[�> In GECKOInstaller.checkRAVENversion (line 79)
In GECKOInstaller.install (line 13)]�
GECKO installed, unknown version (cannot find version.txt)
Running geckoCoreFunctionTests
Done geckoCoreFunctionTests
__________

Note: In the case of multiple test runs, this post will be edited.

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.

None yet

1 participant