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

Revert "Ports Bloodstains And Footprints from EE" #2264

Closed
wants to merge 1 commit into from

Conversation

deltanedas
Copy link
Member

who couldve seen this coming

@deltanedas deltanedas requested review from IamVelcroboy and a team as code owners November 22, 2024 08:50
@github-actions github-actions bot added S: Needs Review Changes: YML Changes any yml files Changes: C# Changes any cs files Changes: Sprite Changes any png or json in an rsi labels Nov 22, 2024
Copy link
Contributor

RSI Diff Bot; head commit 70f8e8d merging into 2f47a36
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/_EE/Effects/footprints.rsi

State Old New Status
dragging-1 Removed
dragging-2 Removed
dragging-3 Removed
dragging-4 Removed
dragging-5 Removed
dragging-test Removed
footprint-left-bare-diona Removed
footprint-left-bare-dwarf Removed
footprint-left-bare-human Removed
footprint-left-bare-lizard Removed
footprint-left-bare-slime Removed
footprint-left-bare-spider Removed
footprint-right-bare-diona Removed
footprint-right-bare-dwarf Removed
footprint-right-bare-human Removed
footprint-right-bare-lizard Removed
footprint-right-bare-slime Removed
footprint-right-bare-spider Removed
footprint-shoes Removed
footprint-suit Removed

@Emily9031
Copy link
Contributor

Guess since I made the comment about performance in the first place, I wanna say a couple things. I do think that while performance from this was definitely impacted quite a bit, I still think it was at least usable. I'd guess it was somewhere in the range of feeling like playing at 20-30fps, which while it sucks for high intensity combat moments, isn't a complete dealbreaker for me. And I kinda worry that if this feature is just reverted and removed, it will end up not being added back at all.

I'd personally prefer if the feature is kept in, but maybe just have an issue made about it needing some optimization. At least unless more players than just me start reporting performance issues from it.

@deltanedas
Copy link
Member Author

it will end up not being added back at all.

it can probably be fixed by checking for stains on the same tile then not making more, but effort

@MilonPL
Copy link
Member

MilonPL commented Nov 22, 2024

it can probably be fixed by checking for stains on the same tile then not making more

i'll look at it later, but if it modifies too much code or takes too much effort to implement, we'll just revert

@dotCatshark
Copy link
Contributor

Guess since I made the comment about performance in the first place, I wanna say a couple things. I do think that while performance from this was definitely impacted quite a bit, I still think it was at least usable. I'd guess it was somewhere in the range of feeling like playing at 20-30fps, which while it sucks for high intensity combat moments, isn't a complete dealbreaker for me. And I kinda worry that if this feature is just reverted and removed, it will end up not being added back at all.

I'd personally prefer if the feature is kept in, but maybe just have an issue made about it needing some optimization. At least unless more players than just me start reporting performance issues from it.

I agree that it should be kept in rather than removed. I don't think it's enough of a big deal for a feature to cause lag that is clearly not gamebreaking outside of what is seemingly just edge cases for it to be completely removed. If it gets worse later or if more people complain about it actually making the game unplayable we can remove it but in the meantime it's kind of just removing a cool feature that has apparently not caused very many problems up to this point. And yeah, if it gets removed it probably won't be added back in, going based on the track record of things that are """pending rework.""" Also, the coders themselves have said themselves that optimization IS possible.

Please do not merge this PR.

@Unkn0wnGh0st333
Copy link
Contributor

Please optimize... don't 1984 this gangsta addition my super amazing awesome team of cool people.

@Proxysseia
Copy link

runs fine on my machine even with high amount of decals and my pretty iffy specs. makes combat spaces look very pretty and was overall a great addition to DV. would be sad to see it go.

@HTMLSystem
Copy link
Contributor

I'd say my computer isn't the best either, hell I can't even run replays, but I've had no problems with the bloodstains or footprints at all.

@deltanedas
Copy link
Member Author

there is a fix pr open itll be used instead when its done

@deltanedas deltanedas closed this Nov 24, 2024
@deltanedas deltanedas deleted the revert-2186-blood branch December 23, 2024 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files S: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants