fix: adjust alignment of values to soil headers #33
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR address the following issue: #32
In the screenshot below, I compare the soil.SOL files from the original (left) to the one that this PR generates (right). The alignment of the variables to the headers are improved, albeit not perfect. However, I also verified that the results from the DSSAT simulation were the same.
DSSAT run with the original soil.SOL file from DSSAT.
DSSAT run with newly created soil.SOL file from this
fix-soil-write
branch.Second test-case
Separately, I also verified that the writing of the weather file was not negatively impacted. I was able to verify that the results of a simulation were the same. In this test case, I did not change the SOL file, I used the one that DSSAT provided. I opened up a weather file and re-wrote it with this branch, and re-ran a simulation. The results were the same.