Skip to content

Missileguidance - Improve JDAM coordinate finding#11231

Merged
PabstMirror merged 12 commits intoacemod:masterfrom
SpicyBagpipes:patch-6
Feb 21, 2026
Merged

Missileguidance - Improve JDAM coordinate finding#11231
PabstMirror merged 12 commits intoacemod:masterfrom
SpicyBagpipes:patch-6

Conversation

@SpicyBagpipes
Copy link
Contributor

When merged this pull request will:

  • Fix JDAM TOO not working for copilot turrets
  • UCAV Sentinel is covered by this in base game

IMPORTANT

  • If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

@SpicyBagpipes
Copy link
Contributor Author

I don't like using focusOn here however ace_player does not update for UAVs

@SpicyBagpipes
Copy link
Contributor Author

No idea why hemtt check has failed there either

@SpicyBagpipes
Copy link
Contributor Author

SpicyBagpipes commented Jan 13, 2026

GVAR(gps_weapons) seems to be empty after this so the JDAM settings are no longer appearing... wut...

happens on master too

Caching got me confused, cache is done when the JDAM appears on a pylon, so there's something up with _player there. Will fix.

_player was ace_player, whereas you will need to be able to get UAVs too.
@PabstMirror
Copy link
Contributor

iirc focusOn may be wrong when direct controlling from zeus
_player should be either ace_player or the UAV AI and work all cases

@SpicyBagpipes
Copy link
Contributor Author

iirc focusOn may be wrong when direct controlling from zeus _player should be either ace_player or the UAV AI and work all cases

_player was giving me my player while remote controlling the UAV. With direct control in zeus, it is still giving me the UAV AI, which is what we want.

@SpicyBagpipes
Copy link
Contributor Author

focusOn still required for old handling for UAV purposes

SpicyBagpipes and others added 3 commits January 13, 2026 19:27
Co-authored-by: Jouni Järvinen <rautamiekka@users.noreply.github.com>
Co-authored-by: Jouni Järvinen <rautamiekka@users.noreply.github.com>
Co-authored-by: Jouni Järvinen <rautamiekka@users.noreply.github.com>
@PabstMirror PabstMirror added this to the 3.20.3 milestone Feb 12, 2026
@PabstMirror PabstMirror added the kind/enhancement Release Notes: **IMPROVED:** label Feb 12, 2026
@SpicyBagpipes
Copy link
Contributor Author

I'm not entirely sure that all JDAMs have the ability to be dropped dumb - if we're fine with that capability being invented then that's fine by me (most of them can guide to a CCIP point anyway)

@PabstMirror
Copy link
Contributor

I'm open to anything, but I think this is better than trying to fly to a random corner of the map
I guess we could block firing on no-target, but this seems like the simplest solution for now

@Drofseh
Copy link
Contributor

Drofseh commented Feb 19, 2026

I'm not entirely sure that all JDAMs have the ability to be dropped dumb - if we're fine with that capability being invented then that's fine by me (most of them can guide to a CCIP point anyway)

I'm pretty sure they do

@SpicyBagpipes
Copy link
Contributor Author

I'm not entirely sure that all JDAMs have the ability to be dropped dumb - if we're fine with that capability being invented then that's fine by me (most of them can guide to a CCIP point anyway)

I'm pretty sure they do

SME says no. CCIP points are still guidance in real life, they're just loaded onto the ordnance before release

@SpicyBagpipes
Copy link
Contributor Author

I'm open to anything, but I think this is better than trying to fly to a random corner of the map I guess we could block firing on no-target, but this seems like the simplest solution for now

Blocking firing probably more realistic but agreed, GBU-31 or something once that's in going to [0,0,0] (which is a movement it's never going to complete) would probably be disastrous for some units

@Drofseh
Copy link
Contributor

Drofseh commented Feb 19, 2026

SME says no.

SME says they can't be dropped dumb?

@SpicyBagpipes
Copy link
Contributor Author

SME says no.

SME says they can't be dropped dumb?

Some of them can't be, and those that can just load the coordinates of CCIP anyway so end up still guiding

@PabstMirror PabstMirror enabled auto-merge (squash) February 21, 2026 15:55
@PabstMirror PabstMirror merged commit 2f1b149 into acemod:master Feb 21, 2026
2 of 3 checks passed
@SpicyBagpipes SpicyBagpipes deleted the patch-6 branch February 21, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement Release Notes: **IMPROVED:**

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants