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

Z homming hangs the machine, after commit "Implement accurate acceleration profiles" #3

Open
casainho opened this issue Dec 24, 2011 · 10 comments

Comments

@casainho
Copy link
Collaborator

When I home the machine with G28 command, the Z homing hangs. If I revert this commit "Implement accurate acceleration profiles - 4bef47f" all works ok.

I started to wrote a message to dev group with subject "Issue on firmware -- Homing Z hanging":

I found that the Z homing sequence doesn't happen in totality. Here
is the current code:

static void zero_z(void)
{
// hit endstops, no acceleration- we don't care about skipped steps
SpecialMoveZ(startpoint.z - 250, config.homing_feedrate_z);
<-------------------- This works

synch_queue();

// move forward a bit
SpecialMoveZ(startpoint.z + 1, config.search_feedrate_z);
<-------------------- This works

// move back in to endstops slowly
SpecialMoveZ(startpoint.z - 6, config.search_feedrate_z);
<-------------------- This doesn't happen!!! Machine hangs.

synch_queue();

// this is our home point
//startpoint.Z = current_position.Z = config.home_pos_z *
config.steps_per_mm_z;
tTarget new_pos = startpoint;
new_pos.z = config.home_pos_z;
plan_set_current_position (&new_pos);

// move back in to endstops slowly
SpecialMoveZ(startpoint.z - 6, config.search_feedrate_z);
<-------------------- This doesn't happen!!! Machine hangs.

I found that this works, moving Z only in positive way. Maybe the
problem is a negative value of Z axis...

// move back in to endstops slowly
//SpecialMoveZ(startpoint.z - 6, config.search_feedrate_z);
SpecialMoveZ(startpoint.z + 6, config.search_feedrate_z);

@bobc
Copy link
Collaborator

bobc commented Dec 24, 2011

There is nothing special about negative values, all coords are signed and moves are converted to a positive delta plus a direction. Also, the first negative move works?

I am wondering if there is some rounding issue that occurs with different config values. It could mean for a low feed rate the calculated rate ends up being zero, although there is supposed to be a minimum feed rate to stop that.

@bobc
Copy link
Collaborator

bobc commented Dec 26, 2011

I changed my config file to the config_reprap_mendel one, and I hit the same problem. Printer hangs after second Z move.

X and Y homing seems to work ok though.

@casainho
Copy link
Collaborator Author

If you git revert your commit, the firmware will work well -- current stable firmware don't have your patch.

@bobc
Copy link
Collaborator

bobc commented Dec 26, 2011

Yeah, I know that :) I am trying to find the bug and fix it!

bobc added a commit that referenced this issue Dec 28, 2011
@bobc
Copy link
Collaborator

bobc commented Dec 28, 2011

The printer hangs after the second Z move, adding a synch_queue() prevents the hang. I need to look into this a bit further to see why the decel of the second move does not work correctly, just in case it could happen at other times.

@casainho
Copy link
Collaborator Author

Ok. I will test ASAP and I would like to release V1.0. Thanks!

@bobc
Copy link
Collaborator

bobc commented Dec 28, 2011

That's funny, I seem to have accidentally reverted your last 2 commits, not
sure how that happened. Not sure how to fix it either...

On , casainho
reply@reply.github.com
wrote:

Ok. I will test ASAP and I would like to release V1.0. Thanks!


Reply to this email directly or view it on GitHub:

#3 (comment)

@bobc
Copy link
Collaborator

bobc commented Dec 28, 2011

No wait, my bad. Your commits are there ok, for some reason it came through
as a branch merge.

I still don't quite understand git!

On , bobcousins42@googlemail.com wrote:

That's funny, I seem to have accidentally reverted your last 2 commits,
not sure how that happened. Not sure how to fix it either...

On , casainho
reply@reply.github.com>
wrote:

Ok. I will test ASAP and I would like to release V1.0. Thanks!


Reply to this email directly or view it on GitHub:

#3 (comment)

@casainho
Copy link
Collaborator Author

2011/12/28 Bob Cousins
reply@reply.github.com:

No wait, my bad. Your commits are there ok, for some reason it came through
as a branch merge.

I still don't quite understand git!

Branch merge?

Since I understand, a good way to work is you having your local master
equal to github master. You can locally create your branches were you
work... finally, merge them with local master. Finally commit and push
the changes you did in your master.

You can just keep your branches locally and private or uploaded them
to github so others can see them. Looks like Triffid Hunter did that.

@casainho
Copy link
Collaborator Author

Picasa Web Albums

Recent Uploads

Jorge Pinto added 1 photos to 6/22/12

Jun 22, 2012 8:11:05 AM

Unsubscribe from this user.

To share your photos or receive notification when your friends share
photos, get your own free Picasa Web Albums account.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants