Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved Sun Sensors Model #288

Merged
merged 2 commits into from
Jan 15, 2021
Merged

Improved Sun Sensors Model #288

merged 2 commits into from
Jan 15, 2021

Conversation

kylekrol
Copy link
Member

@kylekrol kylekrol commented Jan 7, 2021

Improved Sun Sensors Model

Relates to #143.

Summary of changes

  • Added single spherical angle noise model. This model is currently employed by the filter as well but will have to be updated as the model breaks down when the sun vector is near the z axis.
  • Added a simple eclipse model. Note, the model must be enable for it to be active (see config file additions).

Testing

Some sample images taken with eclipse and without eclipse are included here.

With eclipse enabled:

eclipse_s
eclipse_s_error

Without eclipse enabled:

s
s_error

You can also see the more detailed behavior with a shorter time scale:

short_s
short_s_error

The above plots can be generated with the following command:

python -m psim -s 20000 -p sensors/sun_sensors -ps 1 -c sensors/base,truth/base,truth/deployment -v SingleAttitudeOrbitGnc

@kylekrol
Copy link
Member Author

kylekrol commented Jan 7, 2021

This needs to be rebased once #280 and #287 are merged.

@kylekrol kylekrol mentioned this pull request Jan 7, 2021
@kylekrol kylekrol force-pushed the feature/sun-sensor-noise branch 2 times, most recently from c0db90d to 8e6c3c6 Compare January 7, 2021 03:49
@kylekrol kylekrol requested review from shihaocao and afoarce January 14, 2021 23:50
@kylekrol kylekrol force-pushed the feature/sun-sensor-noise branch from f99ba8a to b063c76 Compare January 15, 2021 00:44
@kylekrol kylekrol marked this pull request as ready for review January 15, 2021 00:44
@kylekrol kylekrol merged commit d832485 into master Jan 15, 2021
@kylekrol kylekrol deleted the feature/sun-sensor-noise branch January 15, 2021 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants