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

Double Sponson Weight #304

Closed
Maingunnery opened this issue Jun 9, 2019 · 1 comment · Fixed by MegaMek/megamek#1374
Closed

Double Sponson Weight #304

Maingunnery opened this issue Jun 9, 2019 · 1 comment · Fixed by MegaMek/megamek#1374
Assignees
Labels

Comments

@Maingunnery
Copy link

Environment

megameklab-windows-0.45.4
Java version 1.8.0_211

Description

The current version of MML effectively doubles the weight of Sponson turrets.
The 10% of total capacity is for the L&R pair, not each individually:
https://bg.battletech.com/forums/index.php?topic=33987.msg792520#msg792520

An possible related issue that could influence any solution is if either Vehicular Sponson Turrets can be pod-mounted or have a fixed capacity on Omni-Vehicles (like all turrets). My suggestion for the later case would be to move the Sponson turrets to the first tab with Chassis options so that it is with all the other turret options.

Files

test-example.zip

@Maingunnery Maingunnery changed the title Bug: Sponson Weight Double Sponson Weight Jun 9, 2019
neoancient added a commit to MegaMek/megamek that referenced this issue Jun 9, 2019
@neoancient
Copy link
Member

While working on support vehicle construction I recognized the need for omnis to determine the weight of sponson and turret mounts as well as basic/advanced fire control. I haven't done the sponson and pintle weights yet, but when I do I'll be adding the sponson section to combat vehicles.

@neoancient neoancient added the Bug label Jun 9, 2019
@neoancient neoancient self-assigned this Jun 9, 2019
neoancient added a commit to MegaMek/megamek that referenced this issue Jun 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants