cpmodel / FTT_StandAlone

Future Technology Transformation models
GNU General Public License v3.0
9 stars 1 forks source link

Update folder name for Linux compatibility (case-sensitive filepaths) #187

Closed ct-camecon closed 1 day ago

ct-camecon commented 6 days ago

Amended a filepath in 'SourceCode/initialise_csv_files.py', when generating input CSV files:

Linux is case sensitive (whereas Windows and macOS aren't) so this difference matters. The lowercase 'f' failed to match any data and generate input CSV files. This led to a problem when running the model because the (non-existent) input files weren't read in, leading to missing arrays during solution.

Other than this now working on Linux, all other behaviour should be unchanged.

Otherwise, the file's been tidied slightly (stripped of trailing whitespace) and uses pathlib for the globbing.

ct-camecon commented 5 days ago

Great. All fine with me (extra comments included). I'm not able to merge the pull request myself. Can I leave that to you/colleagues?