fix item one indexing error: first item on the CSV was getting excluded

DininduSenanayake commented 2 months ago

The main changes are:

Now use the csv module to read the CSV file correctly. This ensures that all rows, including the first one, are read properly.
Removed the strip() method when reading soil names, as the csv module handles whitespace correctly.
I have removed config = config.replace('run', "") for the moment as it wasn't using the base_config variable.

Assumption : soil names are in the first column of the CSV file. If they're in a different column, you'll need to adjust the index in row[0] accordingly.

DininduSenanayake commented 2 months ago

@Ollehar I think we can close this PR without merging in favour of the R script you wrote last week.https://github.com/DininduSenanayake/APSIM-eri-mahuika/pull/7

Ollehar commented 2 months ago

@Ollehar I think we can close this PR without merging in favour of the R script you wrote last week.#7

Agreed, I think this is solved.

nesi / APSIM-HPC

fix item one indexing error: first item on the CSV was getting excluded #16