-
Notifications
You must be signed in to change notification settings - Fork 1
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
Simplify K definitions #38
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure about this...
The modules were split , with some repetition, in order to get a good course progression in a tutorial. It enables talking about only some of the concepts isolated in their individual file and running an exercise on that ifle without giving out too much unknown other code .
We can of course depart from this approach and make things more compact but we would lose this work and re-do it when/if someone runs a basic tutorial again.
I did this refactoring in preparation for a tutorial I made last week. I tried to keep modularity, but wanted to reduce the number of unintuitive concepts to be explained (i.e. make K look user friendly). In particular, I inlined
So the course progression can still be:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After discussing this, maybe we merge your new structure and the next person holding a tutorial can again modify it.
Or you re-instate the intermediate step without statements as a file?
I added an intermediate definition Does this variant enable the course progression you have in mind? If not, I'll restore the original module structure. |
30d0958
to
72545bf
Compare
Yes, that works, thank you! BTW We could collect links to tutorial slides in the repo itself if you think this is useful. https://docs.google.com/presentation/d/1mx34lFnJ8niRPM7flcV_0sdSZ81BaTUV4YhiAacnbAY/edit#slide=id.p |
calls.k
: can be integrated later with a corresponding targetvariables.k
bycalc.k
: defines sortStmt
, no need to redefine variable rules that wayEXPR-SYNTAX
: reduce the number of concepts to be explained during a demoimp-semantics.expr