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

Fix franka sim quick start examples #17

Merged
merged 6 commits into from
Jan 30, 2024
Merged

Conversation

youliangtan
Copy link
Member

No description provided.

@Leo428
Copy link
Collaborator

Leo428 commented Jan 30, 2024

@youliangtan I fixed the bug by removing freeze() calls on the agent.state.params. Following the convention after Flax 0.7.1, we should always use and assume agent.state.params is dict instead of frozendict. It works on my end with flax==0.8.0 now.

@Leo428
Copy link
Collaborator

Leo428 commented Jan 30, 2024

@youliangtan, I also enabled the weight downloading from github function. It works fine on my end. Can you please test it and merge it? Thanks!

@youliangtan youliangtan marked this pull request as ready for review January 30, 2024 03:14
@youliangtan
Copy link
Member Author

Thanks for the quick fix. All works fine now. Cleanup a bit. Renamed serl_launcher/serl_launcher/utils/jaxrl_m_common.py -> serl_launcher/serl_launcher/utils/launcher.py

@youliangtan youliangtan merged commit be85c1b into main Jan 30, 2024
1 check passed
@youliangtan youliangtan deleted the fix/retry-franka-sim-examples branch January 30, 2024 03:15
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.

2 participants