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

Typechecker not called on GEMOC official example #26

Closed
dvojtise opened this issue Jun 13, 2019 · 5 comments · Fixed by #144
Closed

Typechecker not called on GEMOC official example #26

dvojtise opened this issue Jun 13, 2019 · 5 comments · Fixed by #144

Comments

@dvojtise
Copy link
Contributor

when installing the GEMOC ALE official example I was expecting it to be configured as a nominal configuration and to be able to raise typechecking errors

but introducing errors in the ALE code does not raise any error. (both trying to change Ecore attribute or local attribte for example)

might be related to #4

note: using current ale distributed with GEMOC

@dvojtise dvojtise added the bug label Jun 13, 2019
@dvojtise
Copy link
Contributor Author

dvojtise commented Jun 13, 2019

Actually, what are the correct keys to be used in the dsl file ?

the ALE example uses:

  • syntax=
  • behavior=

and the GEMOC ALE example uses:

  • ecore=
  • ale=

for the same information

syntax/behavior allows to have some type-checking in the ale editor, and ecore/ale allows to start the GEMOC ALE engine 🤔

@dvojtise
Copy link
Contributor Author

dvojtise commented Jul 2, 2019

Need to harmonize the keys of the dsl file: one purpose = one keyword

@dvojtise
Copy link
Contributor Author

dvojtise commented Aug 9, 2019

Decision: must change to ecore and ale keywords instead of syntax and behavior

@ebousse
Copy link
Contributor

ebousse commented Nov 15, 2019

Same issue here: when using ALE in GEMOC (with TFSM official example) I have no auto-completion and it can barely detect any error.

And I tried to make a tiny group of students code an interpreter in ALE in the GEMOC Studio, and they really struggled because of this :D

@dvojtise
Copy link
Contributor Author

the current workaround is to duplicate the key to use both ecore and syntax and tio use both ale and behavior (referencing the same files)

echebbi added a commit that referenced this issue Jun 25, 2020
…in .dsl files (fix #26)

Because:
 - "ecore" and "ale" are the keys understood by the GEMOC Studio and using others make some ALE features unavailable

Signed-off-by: Emmanuel Chebbi <emmanuel.chebbi@outlook.fr>
echebbi added a commit that referenced this issue Jun 25, 2020
…in .dsl files (fix #26)

Because:
 - "ecore" and "ale" are the keys understood by the GEMOC Studio and using others make some ALE features unavailable

Signed-off-by: Emmanuel Chebbi <emmanuel.chebbi@outlook.fr>
echebbi added a commit that referenced this issue Jun 27, 2020
…in .dsl files (#144)

Because:
 - "ecore" and "ale" are the keys understood by the GEMOC Studio and using others make some ALE features unavailable (#26)

Signed-off-by: Emmanuel Chebbi <emmanuel.chebbi@outlook.fr>
echebbi added a commit that referenced this issue Jun 27, 2020
…in .dsl files (#144)

Because:
 - "ecore" and "ale" are the keys understood by the GEMOC Studio and using others make some ALE features unavailable (#26)

Signed-off-by: Emmanuel Chebbi <emmanuel.chebbi@outlook.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants