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

Don't display engine rating for SV engines #4992

Merged
merged 2 commits into from
Jan 9, 2024

Conversation

SJuliez
Copy link
Member

@SJuliez SJuliez commented Jan 1, 2024

Removes the engine rating (but not the engine type) from the mech summary and the record sheet for support vehicles as they don't use engine ratings.

old:
image

new:
image

megamek/src/megamek/common/Engine.java Outdated Show resolved Hide resolved
Co-authored-by: neoancient <[email protected]>
@HammerGS
Copy link
Member

HammerGS commented Jan 3, 2024

Errata issued by CGL that could provide a better option.

https://bg.battletech.com/forums/index.php/topic,27319.msg1984570.html#msg1984570

@neoancient
Copy link
Member

We could change it to show the effective rating for the stackpole rule, but that's easily calculated from the information on the sheet and it only applies to fusion engines. I think it would be fine to leave it off.

@HammerGS
Copy link
Member

HammerGS commented Jan 4, 2024

It was originally a CGL request to add the number. They created the errata specifically to show the rating for SV's. So we should go ahead and show the new errata value and engine type for them.

@HammerGS HammerGS merged commit 1ac99a4 into MegaMek:master Jan 9, 2024
4 checks passed
HammerGS added a commit that referenced this pull request Jan 9, 2024
+ PR #5024: Mech summary invalid
+ PR #5023: Correct conversion exception causing search to not work
+ Fix #5015: Add advanced search for gyro
+ PR #5014: Fix CF warning sprite to work with isometric view
+ PR #5012: QuirkEntry: remove maximum slot number check (fixes
MegaMekLab#1364)
+ PR #4992: Don't display engine rating for SV engines
@SJuliez SJuliez deleted the print-engine-sv-without-rating branch February 4, 2024 22:00
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.

3 participants