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

Get rid of agentdata system all together #34

Open
4 tasks
ensiform opened this issue Dec 19, 2021 · 4 comments
Open
4 tasks

Get rid of agentdata system all together #34

ensiform opened this issue Dec 19, 2021 · 4 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@ensiform
Copy link
Member

This extra entity system is very wasteful and we can actually (re)use a lot of fields within the entitystate and playerstate if needed, as ET has a ton of extra free unused fields on player types and in general in ETF they are not used at all.

@ensiform ensiform added enhancement New feature or request help wanted Extra attention is needed labels Dec 19, 2021
@ghost
Copy link

ghost commented Jan 3, 2022

Hello, please excuse me if I'm totally wrong, but wouldn't be UEIDs perfect for fixing this?
Tremulous uses UEIDs.
I assume it would be hard to completely rewrite ETF's entity system to match the Tremulous style of how to deal with entities. Anyways, would it be worth at all?
I have to admit I don't have the skills, to make that happen. It just crossed my mind and I followed Tremulous for a while and saw that they really spent a lot of time to improve the entity system and memory managemant.

@ensiform
Copy link
Member Author

ensiform commented Jan 3, 2022

I do not see the relevance to this "problem". There is no need to have extra entities at all.

@ghost
Copy link

ghost commented Jan 3, 2022

Ah, ok, it seems I didn't even understand the problem. Sorry for that.

@ensiform
Copy link
Member Author

ensiform commented Jan 3, 2022

I have not currently pushed the changes yet, but I have already fixed the corpse task and the gas hallucination task. Haven't pushed because not yet confident I have done it right without more clients available to testing.

ensiform added a commit that referenced this issue Jul 4, 2022
Also transition away from fake client side agentdata for gas effect.

This is a work in progress for #34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant