Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@wbruna
Copy link

@wbruna wbruna commented Oct 20, 2025

Clean up the diff, and include a few missing changes, mainly from the upscaler and model weight type statistics.

Clean up the diff, and include a few missing changes, mainly from
the upscaler and model weight type statistics.
@LostRuins
Copy link
Owner

Will take a look in a bit

@wbruna
Copy link
Author

wbruna commented Oct 23, 2025

I could include the newer changes right now, but since master-334-d05e46c makes lots of small changes all over the place, I think it's best if we do this step-by-step.

#include "tae.hpp"
#include "vae.hpp"

using std::isfinite;
Copy link
Owner

Choose a reason for hiding this comment

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

i wonder why main sd.cpp doesnt need this but i think its ok to have

Copy link
Author

Choose a reason for hiding this comment

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

I think it comes from C-style includes added by stb headers.

Copy link
Owner

@LostRuins LostRuins left a comment

Choose a reason for hiding this comment

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

most just need to check on the \033[K terminator, everything else seems good to merge

@LostRuins LostRuins merged commit fef7391 into LostRuins:concedo_experimental Oct 23, 2025
@LostRuins
Copy link
Owner

all good now, merged

@wbruna wbruna deleted the kcpp_sync_sd_90ef5f8 branch October 23, 2025 14:19
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