8000 [Issue] AC/5 Ammo being considered a 'weapon'/'other' instead of ammo · Issue #6775 · MegaMek/mekhq · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
< 8000 h1 class="gh-header-title mb-2 lh-condensed f1 mr-0 flex-auto wb-break-word"> [Issue] AC/5 Ammo being considered a 'weapon'/'other' instead of ammo #6775
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.

Sign up for GitHub

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

Jump to bottom
Closed
5 tasks done
PsyKatik opened this issue Apr 26, 2025 · 4 comments · Fixed by #6859
Closed
5 tasks done

[Issue] AC/5 Ammo being considered a 'weapon'/'other' instead of ammo #6775

PsyKatik opened this issue Apr 26, 2025 · 4 comments · Fixed by #6859
Assignees
@psikomonkie
Labels
Bug Parts/Repair/Refit Severity: Low Issues described as low severity as per the new issue form

Comments

@PsyKatik
Copy link

Brief Description *

in an updated campaign (from 0,50.04, to 0.50.05) i found that setting the auto-restock defaults in the campaign options menu, considers the AC/5 Ammo as a weapon or 'other', and setting its default to what i have the 'other' parts set to, instead of the 100% i set ammo to.

3. Steps to Reproduce

unknown, unable to reproduce

Attach Files

File for MekHQ peeps (say Hi).cpnx.zip

Severity *

Low (Minor/Nuisance): Minor glitches or cosmetic issues that don't affect gameplay and occur rarely.

MekHQ Suite Version *

0.50.05

Operating System *

Windows 11

Java Version *

17.0.13

Final Verification

  • I confirm this is a single, unique issue that hasn't been reported before
  • I have filled and provided all necessary information above
  • I have included any and all logs, custom units, and CPNX (save) files
  • I have asked on MegaMek Discord about this issue
  • I have confirmed this issue is being opened on the correct repository: MegaMek, MegaMekLab, or MekHQ
@psikomonkie psikomonkie self-assigned this Apr 26, 2025
@psikomonkie
Copy link
Collaborator
psikomonkie commented Apr 26, 2025 •
edited
Loading

@PsyKatik could you upload your 0.50.04 campaign save too? Thanks!
8000

@PsyKatik
Copy link
Author

Secomd File for MekHQ peeps (say hi).zip

@psikomonkie
Copy link
Collaborator

@PsyKatik Can you please upload your custom units? Campaigns cannot be opened without them.

Image

@PsyKatik
Copy link
Author

here you go.
Centurion CN9-A Mk II.zip

psikomon 996B kie added a commit to psikomonkie/mekhq that referenced this issue May 3, 2025
@psikomonkie
Issue MegaMek#6775: Campaign.getDefaultStockPercent(Part) should no…
07379ea 
…w handle `AmmoStorage` parts with `CampaignOptions::getAutoLogisticsAmmunition` instead of `CampaignOptions::getAutoLogisticsOther`
@psikomonkie psikomonkie mentioned this issue May 3, 2025
Merged
psikomonkie added a commit to psikomonkie/mekhq that referenced this issue May 3, 2025
@psikomonkie
Issue MegaMek#6775: Some JUnit tests for AutoLogistics
f61f941
@psikomonkie psikomonkie added Bug Severity: Low Issues described as low severity as per the new issue form Parts/Repair/Refit labels May 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@psikomonkie psikomonkie

Labels
Bug Parts/Repair/Refit Severity: Low Issues described as low severity as per the new issue form
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix: #6775 AutoLogistics properly considers AmmoStorage as ammo and not other psikomonkie/mekhq
2 participants
@psikomonkie @PsyKatik
0