Closed
Description
Here, I don't understand
module_height
; does it not change with tracker rotation? Maybeaxis_height
and an internal calculation would producemodule_height
as an attribute, not an input parameter. InFixedMount
,module_height
is merely awkward, as the measurement that seems more likely to be at hand is the height of the bottom of the module above ground.
I'm OK merging this as is and improving in later PRs.
Originally posted by @cwhanse in #1176 (comment)