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.

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

Review of ATL #11

Open
rene-schoene opened this issue Jul 5, 2019 · 0 comments
Open

Review of ATL #11

rene-schoene opened this issue Jul 5, 2019 · 0 comments

Comments

@rene-schoene
Copy link
Contributor

The presented tool is basically the same as the reference solution, however performance has been optimized by

  • more sophisticated data structures (i.e. maps) for faster access
  • usage of ATL helpers with caches
  • refined initialization

The paper is extremely short, with the actual presentation of the optimizations spanning ~1/3rd of a page. It assumes, that the reader is already familiar with the reference solution, and with internals of the ATL runtime. A general introduction, how the reference solution works should be added. With this, the optimizations could be explained better, as mentioned helpers like getTree or getPartition would be in context of their usage.
Another interesting point would be, if those optimizations are general, i.e., if they are applicable to a wider range of transformation tasks.
Furthermore, in the paper unsolved "issues that are in the transformation code" are mentioned but not explained in detail.

Metrics

  1. Does the solution use the provided benchmark structure, i.e., is a batch execution supported?
    → Yes
  2. Can all models be transformed within 10 minues?
    → Yes
  3. What kind of result is produced? BDT, BDD, with fix order
    → not described
  4. Is the transformation correct?
    → not described nor checked
  5. Computed Metrics (like number of decision or assignment nodes)
    → Only execution time
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

No branches or pull requests

1 participant