Skip to content

Add ActionSequenceFinder for PCM#2

Merged
Nicolas-Boltz merged 29 commits intomainfrom
analysis-impl
Sep 9, 2022
Merged

Add ActionSequenceFinder for PCM#2
Nicolas-Boltz merged 29 commits intomainfrom
analysis-impl

Conversation

@sebinside
Copy link
Copy Markdown
Member

@sebinside sebinside commented Aug 9, 2022

This PR adds the new implementation of the ActionSequenceFinder to extract sequences of actions (i.e., data flows) out of Palladio models. This implementation is based on (and translated from the Xtend sources) of this prototypical implementation.

Some notable changes:

  • Translated from Xtend to plain Java
  • Refactored and cleaned code whenever possible
  • Isolated instanceof instead of dynamic dispatch in Xtend
  • Uses immutable classes and lists as well as records whenever possible
  • Moved the standalone initialization into the analysis
  • Removed workflows
  • Added some initial tests

This PR does not contain the complete logic to perform data flow-based confidentiality analysis. The propagation of labels is still missing and shall be added in a consecutive PR.

sebinside and others added 29 commits June 20, 2022 11:35
@sebinside sebinside marked this pull request as ready for review September 8, 2022 14:58
Copy link
Copy Markdown
Member

@Nicolas-Boltz Nicolas-Boltz left a comment

Choose a reason for hiding this comment

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

LGM
We need to look into where the dependencies to the dataflow model are defined, as the target platform is missing those?!

@Nicolas-Boltz Nicolas-Boltz merged commit 2a827b4 into main Sep 9, 2022
@Nicolas-Boltz Nicolas-Boltz deleted the analysis-impl branch September 9, 2022 13:57
Nicolas-Boltz pushed a commit that referenced this pull request May 28, 2025
Nicolas-Boltz pushed a commit that referenced this pull request May 28, 2025
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.

3 participants