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

Fixes in ddpm.py and main.py #851

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

silky1708
Copy link

@silky1708 silky1708 commented May 16, 2024

  1. first move self.logvar to self.device, and then index using t (line 1030, ddpm.py)
    the current verison of the code gives the following error:
RuntimeError: indices should be either on cpu or on the same device as the indexed tensor (cpu)
  1. remove extraneous argument outputs from on_train_epoch_end in main.py
    the current version of the code gives the following error:
TypeError: on_train_epoch_end() missing 1 required positional argument: 'outputs'

first move `self.logvar` to `self.device`, and then index on line 1030
remove extraneous argument outputs (L403, main.py)
@silky1708 silky1708 changed the title Fix in ddpm.py, move logvar to self.device first Fixes in ddpm.py and main.py May 16, 2024
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.

1 participant