Skip to content
This repository has been archived by the owner on Jan 5, 2021. It is now read-only.

Prioritization of regions is not obviously visible in editor and simulator. #475

Closed
nyssen opened this issue Mar 2, 2016 · 3 comments · Fixed by #1835
Closed

Prioritization of regions is not obviously visible in editor and simulator. #475

nyssen opened this issue Mar 2, 2016 · 3 comments · Fixed by #1835

Comments

@nyssen
Copy link
Contributor

nyssen commented Mar 2, 2016

Regions are executed according to their respective prioritization. When simulating the attached example as follows, this leads to the following (intended behavior):

sct_issuet

After starting the simulation:

  1. raise 'change': states B (region a) and D (region b) become active, i.e. the change2 event, raised by exiting state A, is directly consumed in region b (as this region has lower priority and is executed before).
  2. raise 'change3': state C (region b) becomes active, but not state A (region a), because it is contained in a region that has higher priority (and the internal event is raised after the region was executed, so here the raised event is not directly consumed).

While the behavior is consistent (the execution semantics is defined based on prioritization of regions), the user does not have any direct visual feedback about the prioritization in the UI, but only implicit through their orientation (priorities match the orientation). We should IMHO add a more specific visualization of the priorities.

Further, during simulation the execution of the region is never highlighted as a simulation step. As such, the fact that in step 2. region a is executed (while not doing anything) before region b is not visible to the user. We should IMHO highlight the active region and add support for visualization of such 'micro steps'.

@nyssen
Copy link
Contributor Author

nyssen commented Mar 2, 2016

The following models can be used to reproduce the issue.
default.sct.txt
default.sgen.txt

@nyssen nyssen changed the title Inconsistent execution semantics w.r.t. to internal event reactions. Prioritization of regions not obvious in UI (and simulation) Mar 2, 2016
@nyssen nyssen changed the title Prioritization of regions not obvious in UI (and simulation) Prioritization of regions is not obviously visible in editor and simulator. Mar 2, 2016
@andreasmuelder andreasmuelder added this to the Inbox milestone Mar 6, 2016
@andreasmuelder
Copy link
Member

andreasmuelder commented Jan 2, 2018

Added visual feedback for region priorities

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants