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

bug: model loaded from YAML has no c-field #509

Closed
1 task done
edkerk opened this issue Aug 19, 2023 · 0 comments · Fixed by #508
Closed
1 task done

bug: model loaded from YAML has no c-field #509

edkerk opened this issue Aug 19, 2023 · 0 comments · Fixed by #508
Labels
bug Bug that needs fixing. fixed in develop This issue is fixed and pushed to develop branch. Will be closed when fix appears in master branch.

Comments

@edkerk
Copy link
Member

edkerk commented Aug 19, 2023

Description of the issue:

If a YAML model is loaded that has no objective function specified, no model.c field will be generated. However, this is a required field, otherwise other functions will fail. Instead, an all-zero model.c field should be generated.

I hereby confirm that I have:
@edkerk edkerk added the bug Bug that needs fixing. label Aug 19, 2023
@edkerk edkerk linked a pull request Aug 19, 2023 that will close this issue
fix: readYAMLmodel always make c-field #508
Merged
1 task
@edkerk edkerk added the fixed in develop This issue is fixed and pushed to develop branch. Will be closed when fix appears in master branch. label Oct 21, 2023
@edkerk edkerk closed this as completed Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Bug that needs fixing. fixed in develop This issue is fixed and pushed to develop branch. Will be closed when fix appears in master branch.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: readYAMLmodel always make c-field SysBioChalmers/RAVEN
1 participant
@edkerk