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

Difference in Flux scheduler configuration max_shift #10675

Open
dxqbYD opened this issue Jan 28, 2025 · 2 comments
Open

Difference in Flux scheduler configuration max_shift #10675

dxqbYD opened this issue Jan 28, 2025 · 2 comments
Labels
bug Something isn't working contributions-welcome good first issue Good for newcomers help wanted Extra attention is needed

Comments

@dxqbYD
Copy link

dxqbYD commented Jan 28, 2025

Describe the bug

Could you please check if the value of 1.16 here...

...is intentional or maybe a typo?

max_shift is 1.15 both in the model configuration...
https://huggingface.co/black-forest-labs/FLUX.1-dev/blob/main/scheduler/scheduler_config.json
...and in the original inference code by BFL:
https://github.com/black-forest-labs/flux/blob/d06f82803f5727a91b0cf93fcbb09d920761fba1/src/flux/sampling.py#L214

Reproduction

Logs

System Info

Who can help?

@yiyixuxu @DN6

@dxqbYD dxqbYD added the bug Something isn't working label Jan 28, 2025
@yiyixuxu
Copy link
Collaborator

oh yeah not intentional! do you want to help us change?

the config in scheduler is correct, so have should not affect the results https://huggingface.co/black-forest-labs/FLUX.1-dev/blob/main/scheduler/scheduler_config.json#L7

@yiyixuxu yiyixuxu added good first issue Good for newcomers help wanted Extra attention is needed contributions-welcome labels Jan 29, 2025
@dxqbYD
Copy link
Author

dxqbYD commented Jan 29, 2025

the value is here, too:
https://github.com/huggingface/diffusers/blob/658e24e86c4c52ee14244ab7a7113f5bf353186e/src/diffusers/pipelines/flux/pipeline_flux.py#L852C18-L852C27

Might still not get used as you say, if config.get() prefers the scheduler config

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working contributions-welcome good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants