Bugfix/event issues

[Mark2000]

Description

Closes #269
Closes #291
Closes #279
Closes #287

Fixes a variety of event system issues.

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?

Please describe how tests have been updated to verify your changes.

Future Work

#287 fix is not robust.

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

[Copilot]

Pull Request Overview

This PR fixes multiple issues in the event system by replacing string-based event definitions with function-based callbacks, improving time-limit handling in the Gym environment, and adding proper event cleanup when satellites are removed.

• Replaced legacy string conditions/actions with conditionFunction/actionFunction in simulator and dynamics modules
• Refactored satellite event handling to support callable extra_actions and improved image event callbacks
• Rounded time_limit to match sim_rate and remove stale events for dead satellites in the Gym

Reviewed Changes

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

Show a summary per file

File	Description
src/bsk_rl/sim/simulator.py	Switched max-step-duration event to use lambda callbacks
src/bsk_rl/sim/dyn/relative_motion.py	Converted conjunction/range events to use conditionFunction and actionFunction
src/bsk_rl/sats/satellite.py	Refactored update_timed_terminal_event to accept callable actions
src/bsk_rl/sats/access_satellite.py	Updated image event creation to use a named side_effect function
src/bsk_rl/gym.py	Aligned time_limit to sim_rate and cleaned up terminal/image events on agent death

Comments suppressed due to low confidence (1)

src/bsk_rl/sim/dyn/relative_motion.py:102

• [nitpick] If r_BN_N is already a NumPy array, avoid wrapping it in np.array on every tick to reduce allocations. Use the native arrays directly if possible.

                    )

[LorenzzoQM]

Nice changes. The code looks much cleaner and easier to debug