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

issue with display names #59

Open
fbergmann opened this issue Sep 27, 2024 · 1 comment
Open

issue with display names #59

fbergmann opened this issue Sep 27, 2024 · 1 comment

Comments

@fbergmann
Copy link
Member

A bug at biosimulators mentions, that basico does not resolve display names with nested parenthesis:

biosimulators/Biosimulators_COPASI#74
@fbergmann
Copy link
Member Author

as it turns out, CDataModel::findObjectByDisplayName does not return a flux reference in those cases. This needs to be fixed in the main COPASI repo, until then I can add a workaround to basico.

fbergmann added a commit that referenced this issue Sep 27, 2024
@fbergmann
#59 workaround for reactions with )
c9639e8 
Ultimately the python-copasi backend should fix this, but for now I've added a workaround that manually resolves reaction references in the data handler if finding by display name fails
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@fbergmann