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

Make IS and Clan structure available to mixed tech units. #204

Merged
merged 1 commit into from
Sep 18, 2018

Conversation

neoancient
Copy link
Member

It's not really clear what a Clan or IS tech base means in a mixed tech unit. There are entries in TROs that specify IS/mixed or Clan/mixed, but the mixed tech rules (TacOps, p. 377) don't really make it clear whether that means anything, or whether they amount to the same thing. For the sake of simplifying the code and the available options, MML uses the IS/Clan tech base to filter the available structure types, but this causes problems when it comes to early Clan units by making IS endo steel unavailable when they should be able to use it. (For that matter, the Clans should be able to construct a mixed tech unit with IS endo steel at any point in their history.)

Given the choice between a hack to deal specifically with IS Endo-Steel and a more general solution that basically ignores the distinction between IS mixed and Clan mixed*, I went with the latter.

*There is actually a distinction between IS mixed and Clan mixed, which is the availability and tech level of mixed tech units based on whether they are built by a Clan faction or an IS faction. But no difference for the unit construction.

Fixes #203: Inner Sphere Endo Steel is unavailable to early clan mechs.

Fixes #203: Inner Sphere Endo Steel is unavailable to early clan mechs.
@neoancient neoancient merged commit 61107e4 into master Sep 18, 2018
@neoancient neoancient deleted the mixed_tech_structure branch September 18, 2018 02:43
neoancient added a commit that referenced this pull request Sep 18, 2018
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.

4 participants