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

Documentation Language Modules #633

Closed
JanWittler opened this issue Sep 1, 2022 · 5 comments
Closed

Documentation Language Modules #633

JanWittler opened this issue Sep 1, 2022 · 5 comments
Labels
PISE-SS22 Tasks for the PISE practical course (SS22)
Milestone

Comments

@JanWittler
Copy link
Contributor

JanWittler commented Sep 1, 2022

Referenced files:
https://github.com/robinmaisch/JPlag/tree/master-remote/docs/frontend-design

This issue is used to collect and discuss feedback regarding the documentation of the language frontends.

Relates to #345.
@robinmaisch @jplag/maintainer

@JanWittler JanWittler added the PISE-SS22 Tasks for the PISE practical course (SS22) label Sep 1, 2022
@JanWittler JanWittler added this to the v4.0.0 milestone Sep 1, 2022
@JanWittler
Copy link
Contributor Author

Overall looks good to me.

FrontendDesign.md

--

FrontendStructure.md

Not much to complain about here neither. The tables are a bit hard to read as they are so close by each other but have different column widths.

TokenExtraction.md

  • Maybe drop approach 3?

TokenSelection.md

  • I would disagree that readability is a necessary property of the token extraction. The tokens should have a debug description but one does not have to optimise the token set for readability.
  • Also I wouldn't say it has to be consistent with other language frontends. If we detect empirically that some tokens should always be extracted, we can require this as a baseline, but languages can differ fundamentally which even may make it impossible for tokens to be consistent (e.g. Haskell doesn't have loops)
  • Tip 2 is written a bit confusing.
  • Tip 5 code example is a bit confusing without any reference code.

@robinmaisch
Copy link
Contributor

Sorry for the long delay. Before I push another version, just a few questions back to you:

  • I see your point about the readability. Should I reframe it as a 'nice to have' feature that will aid the design process, or cut it altogether? I realize, what I really meant by 'readable tokens' is to choose readable String representations for them.

  • Also about the consistency, should I try to emphasize the limitations to that point that come with different programming paradigms, or leave it out? Using the experience from existing frontends that we know to work well – to the extent possible – surely is not an absolute requirement, but it did play a major role for my contribution.

@JanWittler
Copy link
Contributor Author

I see your point about the readability. Should I reframe it as a 'nice to have' feature that will aid the design process, or cut it altogether? I realize, what I really meant by 'readable tokens' is to choose readable String representations for them.

No hard opinions regarding this. As the String representations are enforced anyways by the interface specifications, I don't see the necessity to document this here additionally.

Also about the consistency, should I try to emphasize the limitations to that point that come with different programming paradigms, or leave it out? Using the experience from existing frontends that we know to work well – to the extent possible – surely is not an absolute requirement, but it did play a major role for my contribution.

I agree, so I would go with your first mentioned option, i.e. suggest to reuse knowledge from existing frontends if the language paradigms match to some extent.

@robinmaisch
Copy link
Contributor

Thanks for the feedback! The docs files are updated now.

@tsaglam tsaglam changed the title Documentation Language Frontends Documentation Language Modules Sep 15, 2022
@tsaglam tsaglam moved this to Done in v4.0.0 Release Sep 16, 2022
@tsaglam
Copy link
Member

tsaglam commented Sep 16, 2022

✅ Documentation incorporated into the wiki and the repo!

@tsaglam tsaglam closed this as completed Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PISE-SS22 Tasks for the PISE practical course (SS22)
Projects
No open projects
Status: Done
Development

No branches or pull requests

3 participants