-
Notifications
You must be signed in to change notification settings - Fork 9
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
base_model #59
Comments
Hi Basazin, could you add the rest of the code in your notebook to this issue so that I can track down the exact ensemble and reproduce the error? |
Hello, Greg
I added the screenshot of the code both before and after the error. Let me know if you get the updated version of the Issues on Github or have any questions.
Best
Basazin
…________________________________
From: Greg Medlock <[email protected]>
Sent: Tuesday, July 28, 2020 2:56 PM
To: opencobra/Medusa <[email protected]>
Cc: belhub <[email protected]>; Author <[email protected]>
Subject: Re: [opencobra/Medusa] base_model (#59)
Hi Basazin, could you add the rest of the code in your notebook to this issue so that I can track down the exact ensemble and reproduce the error?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#59 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AMSQE5XY4F46EO2BYE4TBKTR55CSDANCNFSM4PGOM7LA>.
|
Looks like this will require updating all of the test models to account for COBRApy updates. To prevent future changes from breaking test models, all test models should be encoded as SBML (rather than pickled ensembles or COBRApy json objects). I will make these updates to the test models. |
Hello, Greg
Okay,
Did you send me the zoom invitation for a meeting we have at this time?
Best
Basazin
…________________________________
From: Greg Medlock <[email protected]>
Sent: Tuesday, August 4, 2020 8:49 AM
To: opencobra/Medusa <[email protected]>
Cc: belhub <[email protected]>; Author <[email protected]>
Subject: Re: [opencobra/Medusa] base_model (#59)
Looks like this will require updating all of the test models to account for COBRApy updates. To prevent future changes from breaking test models, all test models should be encoded as SBML (rather than pickled ensembles or COBRApy json objects). I will make these updates to the test models.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#59 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AMSQE5WQHQ3A3VW4RQQTLN3R7AU2FANCNFSM4PGOM7LA>.
|
There is a problem with the base_model attribute of the medusa module. It generates the following error when I try to call base_model method on the loaded ensembles. This is the screenshot that contains the code before and after that error occurs.
c.png">
The text was updated successfully, but these errors were encountered: