remove unused precomp variables and calculations
the products of precompQuantum are not actually used anywhere, so just remove the entire function.
also remove unused variables in precompPower that could confuse memoization
Merge request reports
Activity
fSQL is used in squeeze.py, so shouldn't we move it in there?
Indeed, almost everything in precompIFO is used at most 1-2 places, and very lightweight to compute. So we can lift the dependency on precompIFO by just move the code where it's needed.
The only real exceptions that I'm aware of are the coating design (getCoatDopt), and the suspension TFs. Those require a noticeable amount of computation, so they should probably be memoized.
@christopher.wipf yes fSQL is used in squeeze.py, but actually nothing in squeeze.py is used anywhere else in the code. Should we just remove that whole module?
Otherwise I'm happy to just mvoe that computation into that module.
added 1 commit
- 7a2b6fd3 - remove unused precomp variables and calculations
@christopher.wipf i moved the SQL code into squeeze.py