rinongal / textual_inversion

MIT License
2.9k stars 279 forks source link

question about 'def next_step(...)' #140

Closed fido20160817 closed 1 year ago

fido20160817 commented 1 year ago

'timestep, next_timestep = min(timestep - self.scheduler.config.num_train_timesteps // self.scheduler.num_inference_steps, 999), timestep' should be: 'timestep, next_timestep = timestep, min(timestep + self.scheduler.config.num_train_timesteps // self.scheduler.num_inference_steps, 999)' am I right? I have tested both and the results are all normal. But I think the latter coding is right.

rinongal commented 1 year ago

This part is not our code. If you have questions about the original LDM code you should hop on to their own repository

fido20160817 commented 1 year ago

Thanks. It's my mistake. I mix your code with others.