Skip to content

fix: CPU compatibility, missing deps, and deprecated AMP API#31

Open
sumitKts3d wants to merge 1 commit intoferdous-alam:mainfrom
sumitKts3d:fix/cpu-compat-and-deps
Open

fix: CPU compatibility, missing deps, and deprecated AMP API#31
sumitKts3d wants to merge 1 commit intoferdous-alam:mainfrom
sumitKts3d:fix/cpu-compat-and-deps

Conversation

@sumitKts3d
Copy link
Copy Markdown

  • inference_gencad.py: auto-detect CPU when CUDA is unavailable instead of hardcoding cuda:0; accept single image file in addition to directory; load CAD checkpoint with map_location='cpu'; print export paths; improve exception messages
  • utils/model_utils.py: replace .cuda(device=device) with .to(device) in logits2vec so inference works on CPU
  • model/cond_ldm.py: replace deprecated torch.cuda.amp autocast/GradScaler with torch.amp equivalents
  • trainer/ldm_trainer.py: same AMP deprecation fix
  • trainer/diffusion_prior_trainer.py: same AMP deprecation fix
  • requirements.txt: add matplotlib (was missing)
  • environment.yml: add matplotlib to pip section (was missing)

- inference_gencad.py: auto-detect CPU when CUDA is unavailable instead
  of hardcoding cuda:0; accept single image file in addition to directory;
  load CAD checkpoint with map_location='cpu'; print export paths; improve
  exception messages
- utils/model_utils.py: replace .cuda(device=device) with .to(device) in
  logits2vec so inference works on CPU
- model/cond_ldm.py: replace deprecated torch.cuda.amp autocast/GradScaler
  with torch.amp equivalents
- trainer/ldm_trainer.py: same AMP deprecation fix
- trainer/diffusion_prior_trainer.py: same AMP deprecation fix
- requirements.txt: add matplotlib (was missing)
- environment.yml: add matplotlib to pip section (was missing)
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