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

Update DecisionDiffuser Algorithm to omnisafe #300

Closed
wants to merge 14 commits into from
Closed

Update DecisionDiffuser Algorithm to omnisafe #300

wants to merge 14 commits into from

Conversation

Aidilele
Copy link

Update DecisionDiffuser Algorithm to omnisafe

@Aidilele
Copy link
Author

Pull from HongyiZhou

@Gaiejj
Copy link
Member

Gaiejj commented Jan 19, 2024

Thank you for your contribution! We will merge your code following a meticulous code review. 😊

@LiuTaowen-Tony
Copy link

Hey, I also implemented this algorithm. here

I mainly implemented actor, algorithm, and dataset. The model architecture is adapted from the original repo.

Maybe we can collaborate a bit

@Aidilele
Copy link
Author

Hey, I also implemented this algorithm. here

I mainly implemented actor, algorithm, and dataset. The model architecture is adapted from the original repo.

Maybe we can collaborate a bit

Wow, I'm sorry that I didn't notice your reply. In fact, I am little confuse about the code with condition setting in the original paper. Maybe we can discuss it together. email -> zhou2429060789@hotmail.com

@LiuTaowen-Tony
Copy link

Hey, I also implemented this algorithm. here
I mainly implemented actor, algorithm, and dataset. The model architecture is adapted from the original repo.
Maybe we can collaborate a bit

Wow, I'm sorry that I didn't notice your reply. In fact, I am little confuse about the code with condition setting in the original paper. Maybe we can discuss it together. email -> zhou2429060789@hotmail.com

Sure, can do. So the author of the original codebase didn't include condition free guidance. I implemented according to the mathematical formula. Please check my pull request, I attached a pdf file explaining the way how I implemented it.

@Aidilele Aidilele closed this by deleting the head repository Mar 5, 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.

None yet

3 participants