-
Notifications
You must be signed in to change notification settings - Fork 33
example(plate-balancing) : Add working plate balancing example with external c3 repo #376
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
base: stephen/add_c3_repo_for_sampling_c3
Are you sure you want to change the base?
example(plate-balancing) : Add working plate balancing example with external c3 repo #376
Conversation
d4f321a
to
fe05cec
Compare
Hi. Thanks for open sourcing your impressive work. I am working on this branch to reproduce the plate-balancing work, and was wondering if you accept fixes here? These are just a few missing dependencies juanmed@3d90389 |
Hi, Yes we accept fixes. Feel free to file bugs if you find any. I think you find the .yaml files for the plate balancing are included in this branch,. You should take a look at https://github.com/DAIRLab/c3 for our work related to C3 and https://github.com/DAIRLab/dairlib/tree/sampling_based_c3_PR2_copy I believe has the latest work for sampling based C3. |
d50a4b4
to
80568f2
Compare
80568f2
to
41d8eed
Compare
268e67c
to
9571fcd
Compare
d88901a
to
5a72ca5
Compare
* Integrate new C3 algorithm * Tune new C3 parameters * Fix bug that wrongly computes the size of z and delta vectors. * Remove unnecessary use of epsilon in new c3 projection step; update logging script to accept folder path as argument and modify log directory creation; add progress parameters YAML file for new c3 * Add log directory to .gitignore to prevent logging files from being tracked * Minor changes * Add documentation and address comments * Refactor C3 solver structure: - Introduce BaseC3 class, replace C3 with BaseC3 in related files. - Enhances modularity and maintainability of all functions in BaseC3 class. - Change name of C3NextGen class to C3Plus * Update C3 solver implementation: - Modify BaseC3 constructor to accept a function for calculating z size. - Adjust C3Plus to disable parallelization and utilize the new z size calculation function. - Clean up .gitignore by removing the logs directory entry. - Improve documentation in C3Plus header for clarity on C3+ formulation. * Clean up codes * Replace all NEXTGEN name by C3+ * Minor changes * Add C3+ parameters to jacktoy and push-anything examples * Add missing placeholders in sampling_c3_options.yaml of push-anything example * Change class name BaseC3 to C3Base * Remove unnecessary small functions and remove obsolete SolveRobustSingleProjection functions * Modify C3Base constructor to accept z_size and add overloaded constructor for default z_size. * Use fully vectorized code in c3plus projection step * Throw error if any weights of the cost matrix U are negative.
5a72ca5
to
6a81a2d
Compare
30ff460
to
9e969e1
Compare
* Resolve merge conflicts with main * Remove obsolete codes * Remove accidentally commited files * Add external force tracking result to lcm * Add back the deleted files in Cassie example * Add back deleted files in pydairlib/cassie * Revert code formatting * Revert code formatting * Revert code formatting
6a81a2d
to
a922a8f
Compare
@Meow404 Hi, I was trying out your work in this branch, looks awesome. As of the latest commit here, this is the behavior I get and was wondering if your get the same? slide-tray-2025-08-28_01.58.40.mp4Asking because I expected the robot arm to 1) slide the tray out of the supports, 2) lift the tray and 3) slide the tray again onto the supports, but seems to do only 1) and 2). |
Hi @juanmed, Nice to hear from you. We are discovering that the example does work a little differently based on your computer specs. I've tried the same example on three different computers with different results, but this can be overcome by tuning the parameters a little in this file |
@Meow404 Aha that makes sense, will play with the params there. Thanks for the advice! (Btw I had to focus on some background work before retaking plate balancing again) |
9e969e1
to
99cb601
Compare
Fixes DAIRLab/c3#11
Dependency DAIRLab/c3#10
This change is