Skip to content

Conversation

@Cadene
Copy link
Collaborator

@Cadene Cadene commented Jan 25, 2025

What this does

How it was tested

TODO

@Cadene Cadene changed the base branch from main to user/aliberts/2024_11_30_remove_hydra January 25, 2025 18:27
@Cadene Cadene requested a review from aliberts January 25, 2025 18:28
@aliberts aliberts mentioned this pull request Jan 25, 2025
15 tasks
@aliberts aliberts marked this pull request as ready for review January 25, 2025 18:31
Copy link
Collaborator

@aliberts aliberts left a comment

Choose a reason for hiding this comment

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

Shallow review but LGTM, thanks!

f"The policy you are trying to load has been trained with use_amp={train_cfg.use_amp} "
f"but you're trying to evaluate it with use_amp={self.use_amp}"
)
if self.device == "mps" and train_cfg.device == "cuda":
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we still warn in this case?

@aliberts aliberts merged commit 214083f into user/aliberts/2024_11_30_remove_hydra Jan 28, 2025
1 check passed
@aliberts aliberts deleted the user/rcadene/2025_01_25_robot_doc_draccus branch January 28, 2025 11:42
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.

3 participants