snoyer

Most likely a `BRepTools_WireExplorer` vs `TopoDS_Iterator` question, see #864

@nmunnich thanks for the interest, I don't currently have a dev environment (let alone the hardware) to rebase and test/validate so I just eyeballed the small merge conflict on the...

@nmunnich I'm not set up to do that right now so it will have to wait... Alternatively, feel free to edit the PR if you've got write access and want...

@nmunnich I'm back at a proper computer but I'm not sure how to proceed... According to the guidelines you linked I think the commit message should be something like ```...

TBH I was hoping I could wriggle out of having to do the squash myself as I'm not super comfortable with `git` and was going to pull out the "but...

branch running on `nice_nano_v2` board with `boardsource3x4 nice_view_adapter nice_view` shield 

The necessary core changes have been merged separately in #2993 so this PR only affect the nice!view widgets code now. Could @Nicell and @petejohanson please have a look and decide...

> Could you just confirm for me that you re-tested to make sure everything is working as it was before, and then I'll merge? I'm not daily-driving it but it's...

> arguably __bool__ for .IsNull() would be nice too Actually it looks like that is already the case in `7.8.1`, does that constitute another argument in favor of using `__eq__`?

hey @adam-urbanczyk thanks for the reply, do you have any further input as the maintainer of the project? > For now not planned, but I'll leave this open What other...