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

To many gunners for aerospace (small craft, dropships, jumpships) due to AMS #6217

Closed
7 tasks done
NexiraElimus opened this issue Sep 16, 2024 · 5 comments · Fixed by #6218
Closed
7 tasks done

To many gunners for aerospace (small craft, dropships, jumpships) due to AMS #6217

NexiraElimus opened this issue Sep 16, 2024 · 5 comments · Fixed by #6218
Labels
Bug Severity: Medium Issues described as medium severity as per the new issue form

Comments

@NexiraElimus
Copy link

Prerequisites and Pre-Issue Checklist

  • I'm reporting the issue to the correct repository:

  • MegaMek

  • MegaMekLab

  • MekHQ

  • I've tested the issue against at least the latest MILESTONE version

  • I've asked on the MegaMek Discord about the error

  • I've reviewed the BattleTech rules and MekHQ documentation, and I've confirmed that something isn't working as intended.

  • I've searched the Github tracker and haven't found the issue listed

Severity *

Medium (Gameplay Limitation): Non-core functionality is impaired, providing a suboptimal but playable experience.

Brief Description *

Per the rules AMS arent supposed to need gunners. Megameklab shows the correct number of gunners needed by a craft, however in Megamekhq the game requires gunners as though it needed gunners for AMS's as they dont need gunnery skill (tac ops 132, and more specifically on 255 TO it very specifically says not for AMS's.)

"gunners are required only for any item that requires a Gunnery Skill roll
to use in combat, and which has a range of more than 1 hex.
(Automatic defensive armaments like Anti-Missile Systems,
A-Pods, and the like require no gunners, nor do non-weapon
items listed on the Heavy Weapons and Equipment Tables, like
Active Probes and ECM gear.)"

255-TO

Steps to Reproduce

easily reproducable by

1: open megameklab andadd 2 normal guns like er large lasers, and 6 laser AMSs to a small craft. Mekameklab will only require 1 gunner correctly.

2: then in mekhq us gm mode to add the craft and minimum crew and see that its added 2 gunners. It is possible equipment other than AMS is also requiring crew as it shouldnt, but i have not encountered that yet.

Operating System *

Windows 11

Java Version *

Java Version: 17.0.12

MekHQ Suite Version *

v0.50.0

Custom MekHQ Version

No response

Attach Files

logs.zip

Final Checklist

  • I've checked to make sure that this issue has not already been filed
  • I'm reporting only one issue in this ticket for clarity and focus
@NexiraElimus
Copy link
Author

NexiraElimus commented Sep 16, 2024

image
image

further pictures, both from mekhq showing that despite what the sheet says, it gets 2 gunners.
edited: corrected the image

@repligator repligator added the Bug label Sep 16, 2024
@IllianiCBT IllianiCBT transferred this issue from MegaMek/mekhq Sep 27, 2024
@IllianiCBT
Copy link
Collaborator

I believe we pull this data from mml, so shifted this to the mml repo

@pavelbraginskiy
Copy link
Collaborator

pavelbraginskiy commented Sep 27, 2024

I believe we pull this data from mml, so shifted this to the mml repo

It was specified in the issue description that the information shows up correctly in MML but not MHQ. Also, crew requirements are defined in MM.

In general, construction rules are implemented in MM, not MML.

@IllianiCBT
Copy link
Collaborator

Ah, that's my bad, my eyes must have jumped that line! D:

@IllianiCBT IllianiCBT transferred this issue from MegaMek/megameklab Sep 27, 2024
@IllianiCBT
Copy link
Collaborator

And back to MekHQ it goes

@HammerGS HammerGS added the Severity: Medium Issues described as medium severity as per the new issue form label Oct 7, 2024
@HammerGS HammerGS transferred this issue from MegaMek/mekhq Nov 24, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in Bug Catcher: Dropships Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Severity: Medium Issues described as medium severity as per the new issue form
Projects
Development

Successfully merging a pull request may close this issue.

5 participants