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

Hardpoint UI Fixes #416

Merged
merged 11 commits into from
Mar 9, 2023
Merged

Hardpoint UI Fixes #416

merged 11 commits into from
Mar 9, 2023

Conversation

x3Karma
Copy link
Contributor

@x3Karma x3Karma commented Jul 10, 2022

TL;DR:

Methodology:

When using SetHardpointState(CAPTURE_POINT_STATE_AMPING), the game will say that it is "CONTESTED" even though there's no enemy. The bar will still increase even though it said that its being contested.
image
This is replaced with an arbitrary number of 8 which removes the "CONTESTED" UI bug.
image


This shouldn't affect the actual contested UI from being displayed as the code checks for contested zones first before displaying amping UI.

@ASpoonPlaysGames
Copy link
Contributor

Is there a chance that this is an underlying problem with sh_consts or something? Maybe we are missing another CAPTURE_POINT_STATE somehow?

@x3Karma
Copy link
Contributor Author

x3Karma commented Jul 12, 2022

Is there a chance that this is an underlying problem with sh_consts or something? Maybe we are missing another CAPTURE_POINT_STATE somehow?

Could be, otherwise I have no clue how this works. Past the number 8, every subsequent even number would put it back to normal "Amping _%" but every odd number puts it back to "CONTESTED".

Wouldn't surprise me if its just a random number since the playerHardpointID also needs a random number to remove the HUD from being displayed on your screen.

@GeckoEidechse GeckoEidechse added needs testing Changes from the PR still need to be tested needs code review Changes from PR still need to be reviewed in code labels Jul 25, 2022
@x3Karma x3Karma requested a review from EladNLG February 26, 2023 11:29
Copy link
Contributor

@NoCatt NoCatt left a comment

Choose a reason for hiding this comment

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

Fixes work on test server, in a all contesting situations.

@GeckoEidechse GeckoEidechse added almost ready to merge Apart from any small remaining other issues addressed by other labels, this would be ready to merge and removed needs testing Changes from the PR still need to be tested labels Mar 9, 2023
@GeckoEidechse
Copy link
Member

Merging as soon as I get a recent code review ^^

for like one line the rest of the file is kinda oof tho but not related to my pr
Copy link
Member

@EladNLG EladNLG left a comment

Choose a reason for hiding this comment

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

I like when the playerHardpointID is 69 :)

Anyways, took a look over everything, and it looks good - no real complaints.

We should, however, turn the SetHardpointState( hardpoint, 8 ) thing into an issue because I will not accept hacks in my Titanfall 2 Northstar mod >:(((

@EladNLG
Copy link
Member

EladNLG commented Mar 9, 2023

(gecko I forgot to remove the conversations i cannot delete them please dont hurt me)

@GeckoEidechse GeckoEidechse removed needs code review Changes from PR still need to be reviewed in code almost ready to merge Apart from any small remaining other issues addressed by other labels, this would be ready to merge labels Mar 9, 2023
Copy link
Member

@GeckoEidechse GeckoEidechse left a comment

Choose a reason for hiding this comment

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

I did not test / code review PR, simply giving sign-off approval based on other reviews.

@GeckoEidechse GeckoEidechse merged commit 28d444c into R2Northstar:main Mar 9, 2023
GeckoEidechse pushed a commit that referenced this pull request Apr 18, 2023
* fixed not showing Auto-Titans death in killfeed

* bug fix

* Hardpoint Fixes
Copy link
Member

@pg9182 pg9182 left a comment

Choose a reason for hiding this comment

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

Crashed when applied on 1.12 and testing with just myself in private match / cp / mp_colony02:

[22:34:11] [NATIVE SV] [info] Player pg9182 fully connected at time 5.73333
[22:34:11] [SCRIPT SV] [info] Player client script initialization complete: entity (1: player pg9182 [1])
[22:34:11] [SCRIPT SV] [info] ############################
[22:34:11] [SCRIPT SV] [info] CommandString: ClientStatus was not added via AddClientCommandCallback but is being called in CodeCallback_ClientCommand
[22:34:11] [SCRIPT SV] [info] ############################
[22:34:11] [SCRIPT SV] [info] ############################
[22:34:11] [SCRIPT SV] [info] CommandString: AllDialogueFinished was not added via AddClientCommandCallback but is being called in CodeCallback_ClientCommand
[22:34:11] [SCRIPT SV] [info] ############################
[22:34:11] [NATIVE SV] [info] Setting player "player pg9182 [1]" predicted first person proxy model to "" (class "spectator")
[22:34:11] [SCRIPT SV] [info] done waiting!
[22:34:12] [SCRIPT SV] [info] started intro!
[22:34:12] [SCRIPT SV] [info] starting dropship intro!
[22:34:12] [SCRIPT SV] [info] SCRIPT ERROR: [SERVER] Entity class "CNPC_Dropship" doesn't match required class for this index or function call
[22:34:12] [SCRIPT SV] [info]  -> player.SetPlayerNetInt( "playerHardpointID", hardpoint.hardpoint.GetHardpointID() )
[22:34:12] [SCRIPT SV] [info]
CALLSTACK
*FUNCTION [OnHardpointEntered()] gamemodes/_gamemode_cp.nut line [654]

[22:34:12] [SCRIPT SV] [info] LOCALS
[hardpoint] unknown struct
[player] ENTITY (npc_dropship [128] (NPC "npc_dropship" at <933.302 -1869.51 314.585>))
[trigger] ENTITY (trigger_capture_point trigger_hardpoint_c [203] at <1404 -1940 476>)
[this] TABLE

DIAGPRINTS

Doesn't crash before this PR.

GeckoEidechse added a commit that referenced this pull request Apr 19, 2023
This reverts commit 28d444c.

As it causes crash as explained in #416 (review)
GeckoEidechse added a commit that referenced this pull request Apr 19, 2023
This reverts commit 28d444c.

As it causes crash as explained in #416 (review)
GeckoEidechse added a commit that referenced this pull request Apr 19, 2023
This reverts commit 28d444c.

As it causes crash as explained in #416 (review)
GeckoEidechse pushed a commit that referenced this pull request Apr 22, 2023
* fixed not showing Auto-Titans death in killfeed

* revert indents

* revert indents again

* bug fix

* Hardpoint Fixes

* "fixed" spacing

for like one line the rest of the file is kinda oof tho but not related to my pr

* 69 spacing

* SetPlayerNetInt is placed inside a player entity check
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

6 participants