radiasoft/rswarp

Instantiating the Assembly class with non-zero centroid values breaks installation.

ncook882 opened this issue · 1 comments

As part of the STLconductor class initialization, the Assembly class must first be instantiated.

Currently, the following code is executed on line 120:

Assembly.__init__(self, voltage, xcent, ycent, zcent, condid, kwlist, self.conductorf, self.conductord, self.intercept, self.conductorfnew, kw=kw)

where xcent, ycent, and zcent are user-provided coordinates for the desired centroid position of the conductor.

However, providing non-zero values to the Assembly class results in the conductor sometimes not being installed. The code should be modified to provide dummy values, since the actual translation of the conductor is handled by #15 .

Commit 8c57ee5 adjusts the STLconductor class to provide dummy values of the centroid position. Tests suggest that the solver no longer suffers from inconsistent installation, and is much less sensitive to variations in aspect ratio and resolution.