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

Warning when battery and buffer initial levels are incompatible with their capacities #203

Closed
LorenzzoQM opened this issue Oct 2, 2024 · 0 comments · Fixed by #204
Closed
Assignees
Labels
bug Something isn't working enhancement New feature or request modeling Satellite and environment capabilities

Comments

@LorenzzoQM
Copy link
Contributor

Is your feature request related to a problem? Please describe.
When passing an initial battery charge larger than the battery capacity the satellite will start with a full battery but does not provide a warning about the value being larger than it should. For the case where a initial buffer level larger than the buffer capacity, the buffer will initialize at zero, without any warning.

Describe the solution you'd like
The system provides a warning when the initial values for battery and buffer level are incompatible with storage sizes.

Additional context
This would help identify bugs when randomizing the environment for testing and training.

@LorenzzoQM LorenzzoQM added enhancement New feature or request triage Needs to be reviewed by a maintainer labels Oct 2, 2024
@LorenzzoQM LorenzzoQM self-assigned this Oct 2, 2024
@Mark2000 Mark2000 added bug Something isn't working modeling Satellite and environment capabilities and removed triage Needs to be reviewed by a maintainer labels Oct 3, 2024
LorenzzoQM added a commit that referenced this issue Oct 4, 2024
LorenzzoQM added a commit that referenced this issue Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request modeling Satellite and environment capabilities
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants