Radial Build #3312
jonmaddock
started this conversation in
Ideas
Radial Build
#3312
Replies: 1 comment
-
|
Migrated here from an issue as this is a larger issue. Since radial positions are calculated both absolutely and relatively at several different parts of the code and within different modules, this seems like a larger issue. The presence of This may be better to implement when the code is fully Pythonised and a concurrent data structure can be set up. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
In GitLab by @stuartmuldrew on Oct 25, 2019, 17:25
We calculate the distance up to a component in the radial build at multiple points in the code. Each time we just list everything in the calculation and there is a high risk of an error being introduced if we make changes to the order.
As discussed with @jonmaddock and @skahn, there should be reference for the positions, either with a function call or a set of variables, so this calculation is only done once.
Ref: @jmorris-uk
Beta Was this translation helpful? Give feedback.
All reactions