Skip to content

Clone component for simulator#78

Merged
WassawRoki merged 9 commits intomainfrom
5-20-simulation-and-debugging
Nov 24, 2022
Merged

Clone component for simulator#78
WassawRoki merged 9 commits intomainfrom
5-20-simulation-and-debugging

Conversation

@APaludan
Copy link
Copy Markdown

@APaludan APaludan commented Nov 18, 2022

Løser følgende to problemer:

  1. Highlight initial state in simulation view
  2. Clones components for simulation view (so that changes to the components made in the simulation view is not reflected in the editor view/original model).

Solves part of #50.

jhbengtsson and others added 4 commits November 4, 2022 11:56
Components sent to simulation is now clones of the original
@APaludan APaludan linked an issue Nov 18, 2022 that may be closed by this pull request
1 task
@APaludan APaludan changed the title 5 20 simulation and debugging Clone component for simulator Nov 18, 2022
@WassawRoki WassawRoki marked this pull request as ready for review November 21, 2022 12:52
Copy link
Copy Markdown

@Sigurd00 Sigurd00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Det ser godt ud. Vi tænkte lidt på om orange var den bedste farve at bruge fordi den er tæt på rød der bruges til fejl og gul der bruges til warnings, men hvis i synes det er fint så bare brug den farve.

Comment thread src/main/java/ecdar/controllers/SimulatorController.java
Comment thread src/main/java/ecdar/controllers/SimulatorController.java
WassawRoki
WassawRoki previously approved these changes Nov 23, 2022
@jhbengtsson jhbengtsson dismissed WassawRoki’s stale review November 24, 2022 08:11

du må ikke kigge på det casper...

@WassawRoki WassawRoki merged commit 2b66254 into main Nov 24, 2022
@WassawRoki WassawRoki deleted the 5-20-simulation-and-debugging branch November 24, 2022 08:33
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.

(20) Simulation and debugging

5 participants