Skip to content

Resolved Exporting Slowdown#667

Open
Krowe-moh wants to merge 10 commits into4sval:devfrom
Krowe-moh:ExportingFix
Open

Resolved Exporting Slowdown#667
Krowe-moh wants to merge 10 commits into4sval:devfrom
Krowe-moh:ExportingFix

Conversation

@Krowe-moh
Copy link
Copy Markdown
Contributor

If you export using search menu it'll spam your console with "saved ######", at a point your whole extraction will be slowed down and requires a console clear. this pr prevents the spam and does
image

I've also pushed some engine shipping search that a certain game had and cherry-picked some commits that give more right click options to the search entries (Show metadata and ect)

@4sval
Copy link
Copy Markdown
Owner

4sval commented Apr 4, 2026

exporting from search window works fine
image

logs are grouped by folder, it's gonna print out how many stuff was exported for each folder
if there's a single package in a folder it's gonna print that package

now of course if logs are getting spammed at some point it's gonna slow everything down, it's not related to the export, and it has always been an issue related to a feature that needs a refactor. The only source of logs should be serilog with some context and a custom sink to send stuff to the UI.

@Krowe-moh
Copy link
Copy Markdown
Contributor Author

correct it's not a problem, but this pr still resolves special cases and would be nice to be merged. you don't really lose anything besides "exported 6 files from #### folder", tbh kinda useless

there are special cases where only one file exists in one folder and it'd be nice that Fmodel doesn't just slowdown to an insane amount, no real loss with this pr

@LongerWarrior
Copy link
Copy Markdown
Collaborator

we could limit logging for assetsGroups, maybe show only first 100-500 entries(as too many entries in log are just useless) and for the rest just show summary :
"Successfully exported {0}, failed to export{1} in other folders."

@SternI
Copy link
Copy Markdown
Contributor

SternI commented Apr 4, 2026

What's up

@Krowe-moh
Copy link
Copy Markdown
Contributor Author

The sky

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.

6 participants