Modifications to make STEMMUS-SCOPE Octave compliant

[BSchilperoort]

Description

This PR will implement the changes required to make STEMMUS-SCOPE compliant with Octave. Functionally the model has not changed at all.

Note: the source input_data.xlsx file needs to have the empty column B in the sheet options removed, for Octave compatibility.

Closes #118 and #119

Update executable file

Please re-generate exe file if matlab codes are changed. About how to create an exe file, see exe readme.

Checklist

• Add a refernce to related issues.
• @mentions of the person or team responsible for reviewing proposed changes.
• This pull request has a descriptive title.
• Code is written according to the [guidelines].(http://cnl.sogang.ac.kr/cnlab/lectures/programming/matlab/Richard_Johnson-MatlabStyle2_book.pdf)
• Documentation is available.
• Model runs successfully.

[SarahAlidoost]

Let's implement loading the packages in STEMMUS_SCOPE.m as we discussed, see here.

[BSchilperoort]

I'll remove this file

[SarahAlidoost]

Could you please add loading packages to STEMMUS_SCOPE.m in this PR?

[BSchilperoort]

Ah I did not realize you wanted to do that in this PR. I have updated the code and tested it in Octave. Once I've verified it does not break Matlab I'll request your review.

[SarahAlidoost]

@BSchilperoort great job 👍 I left some suggestions. The instructions can be later improved and can be split for run and development purposes.

[BSchilperoort]

@BSchilperoort great job 👍 I left some suggestions. The instructions can be later improved and can be split for run and development purposes.

Thanks for the nice review @SarahAlidoost, you made it really easy with the ready-to-commit suggestions! 😄
I committed all of those suggestions, and removed that redundant file.

[SarahAlidoost]

@BSchilperoort great job +1 I left some suggestions. The instructions can be later improved and can be split for run and development purposes.

Thanks for the nice review @SarahAlidoost, you made it really easy with the ready-to-commit suggestions! smile I committed all of those suggestions, and removed that redundant file.

Just one more thing, see my comment. :)

[BSchilperoort]

The package loading is now performed in STEMMUS_SCOPE.m. The code has been verified on Octave (local machine) and MATLAB (Snellius).
I did spot a missing change to the bin_to_csv.m code. This has now also been fixed.

I could not build a new executable file, due to issues with the STEMMUS_SCOPE_run. file (calls to Python).

[SarahAlidoost]

The package loading is now performed in STEMMUS_SCOPE.m. The code has been verified on Octave (local machine) and MATLAB (Snellius). I did spot a missing change to the bin_to_csv.m code. This has now also been fixed.

Great 👍

I could not build a new executable file, due to issues with the STEMMUS_SCOPE_run. file (calls to Python).

I will remove the file, re-create the exe file, and push it to this PR.

[SarahAlidoost]

@BSchilperoort well done! great improvements.