search

solarmonitor / solarmonitoridl

4 stars 7 forks source link

Updated solarmonitoridl to enable CHIMERA functionality #46

Closed GartontT closed 7 years ago

dpshelio commented 7 years ago

Don't forget to push the latest changes! Great job you've done!

eoincarley commented 7 years ago

Hi guys, sorry for the late input here. David is right, having a code in multiple small functions is standard practice. One long procedure is defo not the way to go. Small functions run by a master procedure improves readability and debugging. Tadhg, you could probably think about this for future development of the code, along with David's other suggestions.

David, I hope it's ok to merge this pull request into the master branch on solarmonitoridl.