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

gapreport #372

Closed
3 tasks done
zahraazimzadeh opened this issue Oct 20, 2021 · 4 comments
Closed
3 tasks done

gapreport #372

zahraazimzadeh opened this issue Oct 20, 2021 · 4 comments

Comments

@zahraazimzadeh
Copy link

Description of the issue:

*Try to be as clear as possible, e.g.:

  • Is there a functionality missing?
  • Did an existing function fail? yes gapreport
  • What is the exact error message, and other output that you got? Gap analysis for iMT1026v3 - Torres2019 - Contextualized P. pastoris GSMM under glucose growth and various oxygenation levels

Index exceeds the number of array elements (0).

Error in haveFlux (line 52)
J(abs(sol.x(mixIndexes))>cutOff)=true;

Error in gapReport (line 64)
I=haveFlux(model);

  • What have you tried yourself to fix it? I tried the gapreport for the small yeast model in the tutorial folder and didn't get any error. Actually I want to check the latest model of pichia pastoris so I tried gapreport for the xml file of it.

Reproducing this issue:

*If applicable, please attach the problematic code.

model=importModel('iMT.xml',true,false,true);
gapReport(model);

System information

  • Please report:
  1. RAVEN version (stabile release, devel branch?) RAVEN 2
  2. Operating system (Windows/Mac/Linux; include version) Windows

I hereby confirm that I have:

Note: replace [ ] with [X] to check the box. PLEASE DELETE THIS LINE

@edkerk
Copy link
Member

edkerk commented May 4, 2022

This is solved in #408 (haveFlux), but the underlying problem is probably that the model is not functional to start with (if this model is the same as the one downloaded from BioModels).

@edkerk
Copy link
Member

edkerk commented May 13, 2022

Now in main #417

@edkerk edkerk closed this as completed May 13, 2022
@zahraazimzadeh
Copy link
Author

Thanks

@zahraazimzadeh
Copy link
Author

👌

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

No branches or pull requests

2 participants