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
Uh oh!
There was an error while loading. Please reload this page.
-
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