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

Belle.invdyn.gravity compensation [WIP] [skip ci] #25

Open
wants to merge 27 commits into
base: master
Choose a base branch
from

Conversation

BelleL
Copy link

@BelleL BelleL commented Aug 8, 2019

cleaned up branch of inverseDynamics
adds callback for inverse dynamics control and include comments/doxygen

@BelleL BelleL requested review from dmsj and tarquasso August 8, 2019 15:01
@tarquasso
Copy link
Contributor

tarquasso commented Aug 8, 2019

Please resolve the conflicts of this branch with master

tried to modify franka

setup structure for callback

finished most structure

run franka is messy af

moved stuff from dracula

need to change urdf and yaml

need to test compilation

gtg

gtg

passes tests

passes tests pt2

recreated segfault locally

modifying things in two dockers

tested compiling the code in dracula

file not found...

commented out multibody dracula since we aren't using it

make sure it's actually calling inversedynamics callback

added debugging flags

does enter while loop

fixed external force error

trying to see cause of segfault is ext force

seeing if eigen::map is giving problem

moved out of function

added debug lines

remapped tau

the problem is in the last two lines

added two more debug flags

runs find all the way to the return statement

tried to merge manual

give up that

still getting eigen errors

refactored franka_plan_runner

callback

callback

debug print statements

testing tau init

testing tau init

testing tau init

making Eigen::VectorXd integral_error a member variable

removing prints to make it faster

removing CalcInverseDynamics to see if loop can meet comms requirements

removing CalcInverseDynamics to see if loop can meet comms requirements

delete print statements add chrono timgin

fixed chrono errors

cast chrono

comment out momap logs

trying to make new variable

added time flags

added time flags

made derivative outside

find segfault

find segfault

updated context

updated context

commented deriv added exception

added urdf masses

log tau

trying tau_ext filtered

tau_ext didn't work

tau_ext didn't work

tau_ext didn't work

tau_ext didn't work

franka_Test.urdf checked

testing gravity calcs

testing gravity calcs

testing gravity calcs

testing gravity calcs

testing gravity calcs

testing gravity calcs

testing gravity calcs

testing gravity calcs

testing gravity calcs

testing gravity calcs

added comments

including urdf and making new branch to clean up commits

checkedout master's franka_0.sh

uncommented include headres

resolve manual diffs between franka_test.urdf

deleted unused comments
@BelleL BelleL force-pushed the belle.invdyn.gravity_compensation branch from 379b62f to cdc5225 Compare August 8, 2019 18:48
Copy link
Contributor

@tarquasso tarquasso left a comment

Choose a reason for hiding this comment

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

Please let me know once it works

@@ -0,0 +1,474 @@
<?xml version="1.0" ?>
Copy link
Contributor

Choose a reason for hiding this comment

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

is this urdf at all used? What is useful content in it?

Copy link
Contributor

@tarquasso tarquasso left a comment

Choose a reason for hiding this comment

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

check if urdf in this commit is needed?

@dmsj dmsj changed the title Belle.invdyn.gravity compensation Belle.invdyn.gravity compensation [WIP] [skip ci] Oct 13, 2019
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