You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The main link between solcore and sunglass is in the sunglass.run_calculation.run_solar_cell_model, which is an extremely long function that does too many things and is rather fragile. Luckily, it is not that difficult to break it into smaller chunks each doing only one thing, since it is very sequential.
and running until the end of the file should be separated into smaller functions. This umbrella issue will be dealt with combining all those smaller functions to reproduce the existing functionality, but first those functions need to be created. Suggested functions to split things into are:
Appropriate tests to check that those individual building blocks work should be added, too. Once the above are done, then the existing run_solar_cell_model code can be refactored to use them.
The text was updated successfully, but these errors were encountered:
Problem
The main link between
solcore
andsunglass
is in thesunglass.run_calculation.run_solar_cell_model
, which is an extremely long function that does too many things and is rather fragile. Luckily, it is not that difficult to break it into smaller chunks each doing only one thing, since it is very sequential.Proposed solution
The current code starting here:
sunglass/sunglass/run_calculation.py
Line 21 in f51044b
and running until the end of the file should be separated into smaller functions. This umbrella issue will be dealt with combining all those smaller functions to reproduce the existing functionality, but first those functions need to be created. Suggested functions to split things into are:
Appropriate tests to check that those individual building blocks work should be added, too. Once the above are done, then the existing
run_solar_cell_model
code can be refactored to use them.The text was updated successfully, but these errors were encountered: