Remove indents from mdu file

[veenstrajelmer]

Describe the bug
Indents are written in mdu file

To Reproduce

import hydrolib.core.dflowfm as hcdfm
file_mdu = 'with_optional_sections.mdu'
fm = hcdfm.FMModel(file_mdu)
fm.save('test.mdu')

Expected behavior
No indents

Screenshots

# written by HYDROLIB-core 0.4.1

[General]
    fileVersion           = 1.09           # File format version (do not edit this)
    fileType              = modelDef       # File type. Do not edit this.
    program               = D-Flow FM      # Program
    version               = 1.2.143.140737 # Version number of computational kernel
    autoStart             = 0              # Autostart simulation after loading MDU (0: no, 1: autostart, 2: autostartstop)
    pathsRelativeToParent = 0              # Default: 0. Whether or not (1/0) to resolve file names (e.g. inside the *.ext file) relative to their direct parent, instead of to the toplevel MDU working dir.

[Geometry]
    netFile                    =
    bathymetryFile             =        # Removed since March 2022. See [geometry] keyword BedLevelFile.
    dryPointsFile              =
    structureFile              =

Version info (please complete the following information):

• Version: main branch

Additional info:
Comment by @arthurvd:
For me it's fine to change this with a SerializerConfig, just like the (new) ext file does it, for example:

    serializer_config: INISerializerConfig = INISerializerConfig(
        section_indent=0, property_indent=0
    )

Points of attention:

• Should be done in a nice way such that all MDU fields that are an INIBasedModel get this setting, without too much code duplication (e.g. Geometry, Output, Numerics, etc., etc.
• there is one SerializerConfig in FMModel itself already, keep it.

[MRVermeulenDeltares]

@veenstrajelmer do you mean that it will look as followed:

[General]
fileVersion           = 1.09           # File format version (do not edit this)
fileType              = modelDef       # File type. Do not edit this.
program               = D-Flow FM      # Program
version               = 1.2.143.140737 # Version number of computational kernel
autoStart             = 0              # Autostart simulation after loading MDU (0: no, 1: autostart, 2: autostartstop)
pathsRelativeToParent = 0              # Default: 0. Whether or not (1/0) to resolve file names (e.g. inside the *.ext file) relative to their direct parent, instead of to the toplevel MDU working dir.

[Geometry]
netFile                    =
bathymetryFile             =        # Removed since March 2022. See [geometry] keyword BedLevelFile.
dryPointsFile              =
structureFile              =```

[veenstrajelmer]

Indeed

[MRVermeulenDeltares]

I tried the following:
493.zip

from that it appears that property_indent= 0 has effect, but section_indent= 0 has no effect.

Defaults of the two are as followed:
section_indent: int = 0
property_indent: int = 4

This makes it clear why section_indent= 0 has no effect

[veenstrajelmer]

It appears that the default for property_indent is set to 4. I also noticed indenting of another ini file during your demo. I would suggest to set the default for property_indent to 0 for all ini files (via INISerializerConfig). @arthurvd do you agree?

[veenstrajelmer]

Test shows the expected results