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

Misc ammo housekeeping and fixes #2908

Merged
merged 5 commits into from
Jan 7, 2024
Merged

Misc ammo housekeeping and fixes #2908

merged 5 commits into from
Jan 7, 2024

Conversation

SamaelGray
Copy link
Contributor

#2874 reopened, the original fork needed to be removed.

Changes

  • Various small changes and housekeeping for mechanoid ammo types and their generic ammo variants.

References

  • Closes [Bug]: Wrong similarTo tags on some calibers #2873

    Despite trying various changes and cross-referencing the Demo mech shell with other generic ammo types and making sure things are consistent, none of the mechanoid ammo types get converted into their generic ammo type correctly. 80mm and 66mm ones stay as is and other ammo types like 5x35mm and 12x64mm get converted into manmade generic ammo types instead of the correct mech charged generic ammo type despite the similarTo tag being assigned correctly.
    The issue described in # 2873 is apparently caused by a fundamental problem with the system since none of the various tried xml solutions could fix the issue. Simply replacing the generic ammo type to a working one does not fix the problem properly.

    -None of the generic mech ammo types except the Demo shells appear ingame inside the dev tools either, it's like they are not being recognized by the ammo injector or something.

Testing

Check tests you have performed:

  • Compiles without warnings
  • Game runs without errors
  • Playtested a colony (specify how long)

@SamaelGray SamaelGray requested review from a team as code owners December 1, 2023 10:00
@SamaelGray SamaelGray marked this pull request as draft December 1, 2023 10:01
Copy link

github-actions bot commented Dec 1, 2023

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-7058503030.zip

@github-actions github-actions bot added the Download in Comments This PR has a zipfile download available. label Dec 1, 2023
@SamaelGray
Copy link
Contributor Author

While this PR does not fix the attached issue, the changes still are worthy of merging into the dev branch.

Copy link
Contributor

@N7Huntsman N7Huntsman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything is working except the mech incendiary shell, which is not replacing the 66mm TB or the 80x256mm FC. No errors and I haven't been able to determine why.

Additionally, we could do with adding a crafting recipe and tags to the 164x284mm Demo rounds, for player-controlled Termites. If we can include that in this PR while poking at other bits of mech ammo, that's just as well.

The recipe is roughly based around the 80mm fuel cells and what has been used so far in the VFE-Mechs patch with appropriate adjustments.
Copy link

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-7299709341.zip

Copy link

github-actions bot commented Jan 6, 2024

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-7434663604.zip

@N7Huntsman N7Huntsman marked this pull request as ready for review January 7, 2024 00:09
Copy link

github-actions bot commented Jan 7, 2024

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-7434732089.zip

Copy link
Contributor

@N7Huntsman N7Huntsman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With some other tweaks/fixes to mech generic ammo, it's good to go.

@N7Huntsman N7Huntsman merged commit 320668a into CombatExtended-Continued:Development Jan 7, 2024
2 checks passed
@SamaelGray SamaelGray mentioned this pull request Jan 7, 2024
2 tasks
@SamaelGray SamaelGray deleted the SamaelGray-AmmoHousekeeping branch January 12, 2024 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Download in Comments This PR has a zipfile download available.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Wrong similarTo tags on some calibers
2 participants