Skip to content

Conversation

@alpi182
Copy link

@alpi182 alpi182 commented Nov 17, 2025

This Expands the HUD that shows the Players Position, Speed & Angle to also optionally show these values for the Dummy.

Checklist

  • Tested the change ingame
  • Provided screenshots if it is a visual change
  • Tested in combination with possibly related configuration options
  • Written a unit test (especially base/) or added coverage to integration test
  • Considered possible null pointers and out of bounds array indexing
  • Changed no physics that affect existing maps
  • Tested the change with ASan+UBSan or valgrind's memcheck (optional)
DummyHUD0

@alpi182 alpi182 closed this Nov 17, 2025
@alpi182 alpi182 reopened this Nov 17, 2025
}
else
{
DummyClientId = GameClient()->m_aLocalIds[1 - (g_Config.m_ClDummy ? 1 : 0)];
Copy link
Collaborator

Choose a reason for hiding this comment

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

same as g_Config.m_ClDummy ? 0 : 1, 1 - g_Config.m_ClDummy also works, im not sure what's best so don't change it

@SollyBunny
Copy link
Collaborator

SollyBunny commented Nov 19, 2025

also fix CI and squash

@alpi182
Copy link
Author

alpi182 commented Nov 19, 2025

also fix CI and squash

i dont even know what this is supposed to mean. i have never worked with github

}
const float LineSpacer = 1.0f; // above and below each entry
const float Fontsize = 6.0f;

Copy link
Collaborator

Choose a reason for hiding this comment

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

Removed line for no reason?

@SollyBunny
Copy link
Collaborator

i dont even know what this is supposed to mean. i have never worked with github

The CI is the checks at the bottom (which aren't running automatically for some reason), they should be all green (or blue depending on theme), you can click on the red ones to see why they failed

By squash I mean combine commits since it doesn't make sense to have one of them but not the others, you can do this with

git reset tater master # tater should be whatever your remote for tater is
git commit -m "new commit msg"
git push origin master --force # push over changes

if you fuck up, always know git reflog exists and the commit b3dd49a is always stored on GitHub

@alpi182
Copy link
Author

alpi182 commented Nov 19, 2025

The CI is the checks at the bottom (which aren't running automatically for some reason), they should be all green (or blue depending on theme), you can click on the red ones to see why they failed

By squash I mean combine commits since it doesn't make sense to have one of them but not the others, you can do this with

git reset tater master # tater should be whatever your remote for tater is
git commit -m "new commit msg"
git push origin master --force # push over changes

if you fuck up, always know git reflog exists and the commit b3dd49a is always stored on GitHub

Can´t i run the Checks myself or do I have to wait until you initiate them.

Also I don´t understand while building there are like 4000 lines of code with hundreds of Errors in files I have never touched. Do I need to edit those files aswell or just the Errors appearing at hud.cpp

Also I am doing all this inside Github and can´t really test it on my end because cloning TClient on windows is pain in the ass

@SollyBunny
Copy link
Collaborator

I'll just fix it

@SollyBunny
Copy link
Collaborator

SollyBunny commented Nov 20, 2025

Ah there's unrelated CI problems, I'll fix it after merge

@SollyBunny SollyBunny merged commit e0eb9d9 into TaterClient:master Nov 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants