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.

Sign up for GitHub

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

Jump to bottom

Fix 673 #1264

Open
wants to merge 6 commits into
base: devel
Choose a base branch
from
Open

Fix 673 #1264

wants to merge 6 commits into from

Commits on Aug 10, 2022

  1. solving the problem in 673 and adding test

    uri.akavia committed Aug 10, 2022
    Configuration menu
    Copy the full SHA
    f291aec View commit details
    Browse the repository at this point in the history

  2. minimize usage of private attributes 

    added species.py functions to set ._reaction field
made metabolites a settable property, and used that to simplify bits of model (including copy and add_reaction)
    uri.akavia committed Aug 10, 2022
    Configuration menu
    Copy the full SHA
    3ed87af View commit details
    Browse the repository at this point in the history

  3. ran black

    uri.akavia committed Aug 10, 2022
    Configuration menu
    Copy the full SHA
    86f1c28 View commit details
    Browse the repository at this point in the history

  4. fixed some bugs I introduced

    uri.akavia committed Aug 10, 2022
    Configuration menu
    Copy the full SHA
    9baeaa6 View commit details
    Browse the repository at this point in the history

  5. minor fixes

    uri.akavia committed Aug 10, 2022
    Configuration menu
    Copy the full SHA
    7ea50ad View commit details
    Browse the repository at this point in the history

Commits on Aug 11, 2022

  1. fixed according to comments

    uri.akavia committed Aug 11, 2022
    Configuration menu
    Copy the full SHA
    63ce057 View commit details
    Browse the repository at this point in the history