Skip to content

Feature/643 fix 'Model' > 'Simulate Network' also instantiates the model#647

Merged
ddelpiano merged 3 commits intodevelopmentfrom
feature/643
Jan 30, 2023
Merged

Feature/643 fix 'Model' > 'Simulate Network' also instantiates the model#647
ddelpiano merged 3 commits intodevelopmentfrom
feature/643

Conversation

@aranega
Copy link
Copy Markdown
Member

@aranega aranega commented Jan 5, 2023

Fixes #643

@aranega aranega requested a review from ddelpiano January 5, 2023 19:37
@SimaoBolota-MetaCell SimaoBolota-MetaCell linked an issue Jan 12, 2023 that may be closed by this pull request
@SimaoBolota-MetaCell
Copy link
Copy Markdown
Contributor

It works correctly now
By following these steps (and similar):

  • Load tutorial 3b
  • Create Network using 'Model > Create Network' at the top bar
  • Simulate Network using 'Model > Simulate Network' at the top bar

an error is no longer displayed and the model is correctly simulated.

Screen.Recording.2023-01-12.at.15.50.06.mov

Additionally, the loader text now has the expected 'Simulating the model'

'Instantiating and Simulating' works properly as well

Screen.Recording.2023-01-12.at.15.59.18.mov

Comment thread webapp/components/topbar/menuConfiguration.js
@ddelpiano ddelpiano merged commit f0edbbc into development Jan 30, 2023
@ddelpiano ddelpiano deleted the feature/643 branch April 14, 2023 22:00
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

Successfully merging this pull request may close these issues.

'Model' > 'Simulate Network' also instantiates the model

3 participants