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

0.50.0(/0.49.20?) Bug: Armor Tech Rating reverting to B #1596

Open
DynamicWorlds opened this issue Aug 22, 2024 · 5 comments
Open

0.50.0(/0.49.20?) Bug: Armor Tech Rating reverting to B #1596

DynamicWorlds opened this issue Aug 22, 2024 · 5 comments
Labels

Comments

@DynamicWorlds
Copy link

Environment

v0.50.0-SNAPSHOT (2024-08-20 evening's nightly and earlier) Unsure if pre 0.50.0 or not.
Win 10
Java Vendor: Eclipse Adoptium
Java Version: 17.0.11

Description

When trying to make a support vehicle with armor tech rating C, it keeps saving the unit as having armor tech rating B, which makes the unit invalid due to being overweight.
To reproduce, load unit, change to armor tech rating to C, save, and then reload unit.

Files

Buggy (Dune)FC.zip

@DynamicWorlds
Copy link
Author

Update: manually editing the .blk file does work

@DynamicWorlds
Copy link
Author

Did some more testing, and found that it occurs for all tech levels, saving as tech rating B no matter which tech rating is selected.

I'm also now 95% certain that this bug came into being with 0.49.20 as I remember needing to go back to an earlier version of megameklabs because of a bug I think was this one and I'm sure that was pre 0.50.00. Unfortunately, I can't currently check it as I switched my Java version for the nightlies.

@DynamicWorlds DynamicWorlds changed the title 0.50.0 Bug: Armor Tech Rating reverting to B 0.50.0(/0.49.20?) Bug: Armor Tech Rating reverting to B Aug 24, 2024
@johnboyjjb
Copy link

johnboyjjb commented Aug 28, 2024

MY Units modified in MekHQ in 49.20 give the error when loaded into MML or MM. MM throws an invalid because the unit becomes very overweight but keeps the armor points as defined. MML throws an invalid error on load "Bar 9 exceeds maximum for armor tech rating X." The unit then reverts to Bar 8 Tech A. Putting it back to Bar 9 Tech F validates the design. Which then fails on the next load.

@HammerGS HammerGS added the Bug label Sep 5, 2024
@DynamicWorlds
Copy link
Author

MY Units modified in MekHQ in 49.20 give the error when loaded into MML or MM. MM throws an invalid because the unit becomes very overweight but keeps the armor points as defined. MML throws an invalid error on load "Bar 9 exceeds maximum for armor tech rating X." The unit then reverts to Bar 8 Tech A. Putting it back to Bar 9 Tech F validates the design. Which then fails on the next load.

I'm curious under what conditions the units are reverting to tech level A. Can you provide an example file of a unit that is happening to and/or more information?

BTW, I'm Gryphon on Discord if you feel inclined to @ me there in addition.

@johnboyjjb
Copy link

Merkava Heavy Tank BS2.zip
This was kosher in 19.18(?) and then the start date for armor changed.
Now it shows invalid in MM. So I loaded it into MML and fixed it so it shows valid. But it won't show as valid in MM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants