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

Autopickup lag when moving past tiles with many items #38340

Open
Funguss opened this issue Feb 25, 2020 · 11 comments
Open

Autopickup lag when moving past tiles with many items #38340

Funguss opened this issue Feb 25, 2020 · 11 comments
Labels
Code: Performance Performance boosting code (CPU, memory, etc.) (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@Funguss
Copy link
Contributor

Funguss commented Feb 25, 2020

Updated, issue was due to autopickup.

It seems that having many items in a tile causes lag when moving past it. This is unrelated to #37554 as there aren't any maps in my piles. I'm using 4x monster spawns and am trying to keep things tidy, but massive piles of dirty clothes and misc loot are causing me to hoard (as I don't want to even approach them!). I'm half inclined to think part of this is my rig being slow, but I doubt that's the full story, or even part of it.

Edit: There seemed to be a reduction in lag when my PC was nude and wandering through the mess. I'm not sure if this is actually the case or if it's my imagination.

Edit 2: Here's the save Edit 3: Here is the actual save. 😕 Bentonia.zip

Steps To Reproduce

  1. Dump thousands of litres of small junk items in one area.
  2. Try walking about in it.

Versions and configuration

  • OS: Windows
    • OS Version: MINGW/CYGWIN/MSYS2 on unknown Windows version
  • Game Version: 0.D-12504-g9015618 [64-bit]
  • Graphics Version: Tiles
  • Mods loaded: [
    Dark Days Ahead [dda],
    Aftershock [aftershock],
    C.R.I.T Expansion Mod [crt_expansion],
    Magiclysm [magiclysm],
    Makeshift Items Mod [makeshift],
    More Survival Tools [More_Survival_Tools],
    Mythological Replicas [nw_pack],
    Hydroponics [hydroponics],
    Mutant NPCs [mutant_npcs],
    No Rail Stations [No_Rail_Stations],
    Folding Parts pack [deoxymod],
    Vehicle Additions Pack [blazemod],
    Tanks and Other Vehicles [Tanks],
    Roadheader and other mining vehicles [Heavy miners],
    No Fungal Monsters [No_Fungi],
    Beta National Guard Camp [national_guard_camp]
    ]
@ZhilkinSerg
Copy link
Contributor

Do you have a savegame affected by this issue?

@Funguss
Copy link
Contributor Author

Funguss commented Feb 26, 2020

Sure thing, should have done that in the first place, shouldn't I?
Bentonia.zip
Edit: linked to the correct save. D'oh!

@ZhilkinSerg ZhilkinSerg added (S1 - Need confirmation) Report waiting on confirmation of reproducibility Code: Performance Performance boosting code (CPU, memory, etc.) labels Feb 26, 2020
@ZhilkinSerg
Copy link
Contributor

Where is that pile in question?

@Funguss
Copy link
Contributor Author

Funguss commented Feb 26, 2020

Character is standing in the middle of them, every pile generates lag, but the more items the worse it is. It seems to be related to the amount of pages there are. Walking east and west between those two central tiles should be noticeably slow, walking east especially. It takes about two seconds for me.

There's a fire ring in the centre of the zoo, too, with filthy armour and clothing nearby. West of that are gigantic pliles of potential laundry. All of these cause lag when walked past, too, especially the western laundry pile. The central tile has 41 pages of items bordering them, according to adanced inventory management.

Edit: It's not about walking through the tile but walking next to them, diagonally and horizontally. Walking through the centre of 9x9 tile sections filled with small, non-stacking items generates the most lag. Walking through them probably generates as much lag as walking beside them but I haven't tested this.

@foulman
Copy link
Contributor

foulman commented Feb 27, 2020

I notice this behavior all the time in my deathmobile storage areas. The more you have the slower it gets, I always chalked it up to something like temperature calculations for individual rags or something.

@Funguss
Copy link
Contributor Author

Funguss commented Feb 27, 2020

This behaviour particularly? I doubt it's temperature calculations, as it seems odd to process those when the PC walks past them. I also think it's related to the amount of item pages, not the amount of individual items. My filthy clothing piles have no stacking items, occupy fewer tiles and have many less items overall than my main loot pile, yet has equivalent pages and lag.

I'm wondering if the advanced inventory is buffering as you move, but this occurs even if 'a'll isn't selected on either column. The inventory lists for each tile may still be getting built, though. The amount of lag on move does seem equivalent to other item commands, so I imagine a list is being built for some reason.

@kevingranade
Copy link
Member

kevingranade commented Feb 27, 2020 via email

@Funguss
Copy link
Contributor Author

Funguss commented Feb 27, 2020

Yes!! Very nicely done! Turned autopickup off and it's no longer an issue. I didn't even realise I had it on, it's bound to "unknown key 7", whatever that is. I must have hit it by accident at some point.

Ok, should I close this then or update it for autopickup? A quick search shows there's no open issue at the moment and it seems it could do with some optimization.

@kevingranade
Copy link
Member

kevingranade commented Feb 27, 2020 via email

@Funguss Funguss changed the title Lag when moving past tiles with many items Autopickup lag when moving past tiles with many items Feb 27, 2020
@ZhilkinSerg ZhilkinSerg added (S2 - Confirmed) Bug that's been confirmed to exist and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Feb 27, 2020
@kevingranade
Copy link
Member

Can you provide your autopickup configuration? I'm not having any luck getting this to trigger.

@Funguss
Copy link
Contributor Author

Funguss commented Feb 28, 2020

Sure. It was "wheat", disabled, and "*arrow", disabled. So having autopickup on without anything defined, basically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code: Performance Performance boosting code (CPU, memory, etc.) (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

No branches or pull requests

4 participants