Bug/fix target angle rate

[LorenzzoQM]

Description

Closes #306

Target angle rate used v_BN_P instead of r_BN_P.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

How should this pull request be reviewed?

• By commit
• All changes at once

How Has This Been Tested?

Pytest tests passed.

Checklist

• I have performed a self-review of my code
• I have commented my code in hard-to-understand areas
• I have made corresponding changes to the documentation and release notes
• My changes generate no new warnings
• [] I have added tests that prove my fix is effective or that my feature works

[Mark2000]

Simple fix, thanks!

[Copilot]

Pull request overview

Fixes an error in the target angle rate observation where the spacecraft position vector was incorrectly taken from the velocity state, aligning the implementation with the intended target-rate computation (Issue #306).

Changes:

• Update _target_angle_rate to use sat.dynamics.r_BN_P instead of sat.dynamics.v_BN_P for r_BN_P.
• Document the fix in the Development release notes.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/bsk_rl/obs/observations.py	Corrects the state vector used in _target_angle_rate computation.
docs/source/release_notes.rst	Adds an entry describing the target angle rate observation fix.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.