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

Minimal mdu writing #495

Open
veenstrajelmer opened this issue Mar 30, 2023 · 0 comments
Open

Minimal mdu writing #495

veenstrajelmer opened this issue Mar 30, 2023 · 0 comments
Labels
type: enhancement Improvements to existing functionality

Comments

@veenstrajelmer
Copy link
Collaborator

veenstrajelmer commented Mar 30, 2023

When reading and saving an mdu file, the output file is much larger since all keywords are written to the file. Among modellers, the preference is to only write the non-default keywords to the mdu file (and maybe some extra ones). This enables you to easily see the non-default model settings. However, one could also argue to want a full list of enabled his/mapoutput, even if some of these values are 1 by default. Either way, this is worth a discussion.

There are also some other formatting alignments with the kernel (dia-file) possible that improve readability:

  • align all comments with max spacing between = and comment
  • keywords start with capital letter?
  • headers like [wind] should be lowercase?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Improvements to existing functionality
Projects
None yet
Development

No branches or pull requests

2 participants