there's probably a better, more elegant way to refactor q.mrot.do in a more modular way.
Maybe – short of spinning out the whole deal #129 – q.mrot.do should just accept loadings, and, well, rotate those (similary to how, say q.flag work), and then to have q.mrot.choose call qmethod()with these provided loadings, if necessary.
Still pretty half-baked this idea, and should be ok for now.
or maybe, the straightforward way would be to modularize extraction #281 and rotation #282 completely, and to really have qmethod() only wrap all of those.
there's probably a better, more elegant way to refactor
q.mrot.do
in a more modular way.Maybe – short of spinning out the whole deal #129 –
q.mrot.do
should just accept loadings, and, well, rotate those (similary to how, sayq.flag
work), and then to haveq.mrot.choose
callqmethod()
with these provided loadings, if necessary.Still pretty half-baked this idea, and should be ok for now.