Z question

135 views
Skip to first unread message

joel silvestre

unread,
Jul 11, 2018, 1:03:34 PM7/11/18
to OpenPnP
Hi,

first many thanks for this great software and all that work!

if I have understand correctly board Z value must be set in the Job Boards Z cell and Feeders Z value in Feeders Configuration Pick Location Z cell?

In this case during the fiducial checking the nozzle goes down, up, down again and then the fiducial seeking moves are made nozzle down.

Also positioning the camera at feeders pick location moves the nozzle down and that's look quite dangerous, anything can be at this place...

Do I miss something?

Joël

Jason von Nieda

unread,
Jul 11, 2018, 1:29:43 PM7/11/18
to ope...@googlegroups.com

--
You received this message because you are subscribed to the Google Groups "OpenPnP" group.
To unsubscribe from this group and stop receiving emails from it, send an email to openpnp+u...@googlegroups.com.
To post to this group, send email to ope...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/openpnp/70682f48-c546-4374-8a03-4916e6977ce9%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

jdlv

unread,
Jul 11, 2018, 3:06:22 PM7/11/18
to ope...@googlegroups.com

ma...@makr.zone

unread,
Jul 12, 2018, 3:21:10 AM7/12/18
to OpenPnP
Hi Jason

On Wednesday, July 11, 2018 at 7:29:43 PM UTC+2, Jason von Nieda wrote:
 
as this keeps popping up again and again (including myself stepping into that one), couldn't proper Axis Mapping just be made the default for Z and C?

This seems feasible with the new human-readable/deterministic "N*" nozzle ids (vs. the old UUIDs). Also I tried to imagine a machine, where this axis mapping would not apply and that is still supported by the OpenPNP model and I can't. Even if there is such a machine: if only those few machine owners need to adjust their machine.xml, much is gained for the rest of us.

Or am I missing something?

_Mark

Jason von Nieda

unread,
Jul 12, 2018, 10:35:26 AM7/12/18
to ope...@googlegroups.com
Yep, I think this would be a good idea. The defaults are set up in GcodeDriver.createDefaults, and we have access to the Machine object, so we can just iterate through the heads and nozzles and set the IDs. 

Jason


--
You received this message because you are subscribed to the Google Groups "OpenPnP" group.
To unsubscribe from this group and stop receiving emails from it, send an email to openpnp+u...@googlegroups.com.
To post to this group, send email to ope...@googlegroups.com.

Mark

unread,
Jul 12, 2018, 12:06:25 PM7/12/18
to ope...@googlegroups.com

Yeah but wouldn’t the generic “N*” wildcard be better? So when they add nozzles later, they would still match?

 

_Mark

ma...@makr.zone

unread,
Jul 12, 2018, 12:15:51 PM7/12/18
to OpenPnP
Ah I see, this isn't a real wildcard matchup:

public Axis getAxis(HeadMountable hm, Axis.Type type) {
for (Axis axis : axes) {
if (axis.getType() == type && (axis.getHeadMountableIds().contains("*")
|| axis.getHeadMountableIds().contains(hm.getId()))) {
return axis;
}
}
return null;
}

https://github.com/openpnp/openpnp/blob/0bfe717a8a0bf8fd34b88edba95f314c72cddb7a/src/main/java/org/openpnp/machine/reference/driver/GcodeDriver.java#L387

That would have to be added...

_Mark

Jason von Nieda

unread,
Jul 13, 2018, 12:34:44 AM7/13/18
to ope...@googlegroups.com
Hi Mark,

Yep, you found it :) 

Easy enough to make that a more useful wildcard, or probably just use a regex to make it even more flexible.

I've filed an issue for this: https://github.com/openpnp/openpnp/issues/752

Jason


--
You received this message because you are subscribed to the Google Groups "OpenPnP" group.
To unsubscribe from this group and stop receiving emails from it, send an email to openpnp+u...@googlegroups.com.
To post to this group, send email to ope...@googlegroups.com.

Jason von Nieda

unread,
Jul 26, 2018, 10:56:44 PM7/26/18
to OpenPnP
This is fixed now: https://github.com/openpnp/openpnp/issues/752

I just made it to so that the default is the current list of Nozzle IDs. It's a bit of a cheat, but I think it will cover most cases and we can revisit if people still run into problems.

Thanks,
Jason


On Thursday, July 12, 2018 at 11:34:44 PM UTC-5, Jason von Nieda wrote:
Hi Mark,

Yep, you found it :) 

Easy enough to make that a more useful wildcard, or probably just use a regex to make it even more flexible.

I've filed an issue for this: https://github.com/openpnp/openpnp/issues/752

Jason


On Thu, Jul 12, 2018 at 11:15 AM ma...@makr.zone <ma...@makr.zone> wrote:
Ah I see, this isn't a real wildcard matchup:

public Axis getAxis(HeadMountable hm, Axis.Type type) {
for (Axis axis : axes) {
if (axis.getType() == type && (axis.getHeadMountableIds().contains("*")
|| axis.getHeadMountableIds().contains(hm.getId()))) {
return axis;
}
}
return null;
}

https://github.com/openpnp/openpnp/blob/0bfe717a8a0bf8fd34b88edba95f314c72cddb7a/src/main/java/org/openpnp/machine/reference/driver/GcodeDriver.java#L387

That would have to be added...

_Mark

--
You received this message because you are subscribed to the Google Groups "OpenPnP" group.
To unsubscribe from this group and stop receiving emails from it, send an email to openpnp+unsubscribe@googlegroups.com.

Mark

unread,
Jul 27, 2018, 5:13:13 AM7/27/18
to ope...@googlegroups.com

Great :)

Reply all
Reply to author
Forward
0 new messages