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

Support vehicle construction #339

Merged
merged 77 commits into from
Oct 16, 2019
Merged

Support vehicle construction #339

merged 77 commits into from
Oct 16, 2019

Conversation

neoancient
Copy link
Member

Adds an option to construct or modify support vehicles: all vehicle types currently supported by MegaMek with the addition of rail. Requires support_vee branch of MegaMek.

This is ready for testing and early review, but I want to hold it back until I write the validation code, which may reveal some things that need to change.

Currently populated with combat vehicle views.
Checks for compatibility with installed mods.
Requires prop mod for fixed wing with electric engine.
Disallows external power pickup mod with maglev engine.
Added filtering of equipment.
Renamed to UnallocatedView to make the purpose clearer, made more generic and moved from vehicle package to general view package.
Moved armor methods into a separate interface.
On further reading of the rules, only small should allocate by point. Medium and large allocate by half ton as other units.
Fixed wing with electric engine -> Prop
Rail with external engine -> external power pickup.
Units that allocate by point (e.g. small support vehicles) show "armor factor" and unit that allocate by weight show "armor tonnage"
Switch pod-mounted equipment to fixed when omni chassis mod is removed.
There are too many canon units that don't follow the dates.
@neoancient neoancient marked this pull request as ready for review October 13, 2019 21:48
@neoancient neoancient merged commit f7113dc into master Oct 16, 2019
@Windchild292 Windchild292 deleted the support_vee branch March 5, 2021 17:10
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.

1 participant