Replies: 1 comment
-
Hi Tom, Thanks for pointing out this issue. It should have been resolved now, those lines of code have been added to the simIncludeRW.py file. Best, |
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
-
Hi all,
I noticed a small bug when using the rw factory for creating a custom reaction wheel config: the minimum torque parameter can be set in the function rwFactory.create(... u_min=0.1, useMinTorque=True), but the code does not set the u_min parameter of the created rw object. So u_min stays at 0.0 and the useMinTorque does not have any effect.
u_max is set in line 179 in simIncludeRW.py, u_min is missing:
if 'u_min' in kwargs:
varu_min = kwargs['u_min']
if not isinstance(varu_min, (float)):
print('ERROR: u_min must be a FLOAT argument')
exit(1)
RW.u_min = varu_min
if RW.u_min <= 0.0 and varUseMinTorque:
print('ERROR: RW is being setup with non-positive u_min value with varUseMinTorque set to True')
exit(1)
Unfortunately I don't have rights for the bitbucket repo, so I can't issue a pull request.
Cheers
Tom
Beta Was this translation helpful? Give feedback.
All reactions