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

Overmap Manoeuvrability Tweak #20292

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Geevies
Copy link
Contributor

@Geevies Geevies commented Dec 23, 2024

  • Standardized and reduced burn delays for ships and shuttles. Ships now have a 1 second burn delay, down from 2. Shuttles now have a 0.5 second burn delay, down from 1. The only exception are the racer shuttles, which have a 0.25 burn second delay.
  • Standardized all ships and shuttles to have a 0.1 second turn delay.
2024-12-23.15-03-23.mp4

@github-actions github-actions bot added 🗺️ Mapping - Runtime The PR touches the Runtime map files. 🗺️ Mapping - Horizon The PR touches the Horizon map files. 🗺️ Mapping - Away Ship/Away Site The PR touches away ship and/or away site map files. 🗺️ Mapping - Random Ruins The PR touches random ruins map files. labels Dec 23, 2024
@Geevies
Copy link
Contributor Author

Geevies commented Dec 23, 2024

!review

Copy link
Contributor

@DreamySkrell DreamySkrell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there should be defines for burn and turn delays, instead of using magic numbers

BURN_DELAY_SHIP
BURN_DELAY_SHUTTLE
BURN_DELAY_SHUTTLE_RACER
// etc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗺️ Mapping - Away Ship/Away Site The PR touches away ship and/or away site map files. 🗺️ Mapping - Horizon The PR touches the Horizon map files. 🗺️ Mapping - Random Ruins The PR touches random ruins map files. 🗺️ Mapping - Runtime The PR touches the Runtime map files. Review Required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants