Is this ready to go?

Worked for me,

Suggestions:

• Add a few loops to example.gramps
• Add a help button ( https://gramps-project.org/wiki/index.php?title=Gramps_5.0_Wiki_Manual_-_Tools#Find_Possible_Loop ) See https://github.com/gramps-project/gramps/blob/master/gramps/plugins/tool/notrelated.py#L58

The following warning is output:
(Gramps.py:2378): Gtk-WARNING **: Could not lookup object notrelated on signal clicked of object close

Minor editorial issue:

• 5 trailing whitespace need to be removed.
  eg;

$ git apply ~/Downloads/102.patch
/home/102.patch:263: trailing whitespace.
        self.progress.set_pass(_('Looking for possible loop for each person'), 
/home/102.patch:271: trailing whitespace.
            GObject.TYPE_STRING)    # 4==family gid 
/home/102.patch:332: trailing whitespace.
            for pth in range(len(self.model)): 
/home/102.patch:349: trailing whitespace.
            if not family: 
/home/102.patch:419: new blank line at EOF.
+
warning: 5 lines add whitespace errors.

What is the best way to add a loop example to example.gramps ?
I created the wiki tool page for gramps 5.0.
Need to have a test case in example.gramps to make a screenshot of the tool window.
work in progress...

I have not tried to run this but just from a quick glance I see that
you have tool_modes = [TOOL_MODE_GUI, TOOL_MODE_CLI]
but you are doing things like importing the ProgressMeter, so if
you really want this tool to be run from the CLI you'll need to
change that to use the various progress methods in the User
class instead (self.progress = user.progress(...)). Or else you
can remove TOOL_MODE_CLI from the .gpr.py file.

I am just guessing, since I haven't tried to run it, but when you ask
"What is the best way to add a loop example to example.gramps"
I would suppose adding one or two new people, say a great-grandchild
of the main person and maybe a great-grandparent too, depending on
what your tool checks for? But that's just a guess.

I was thinking to add a new testloop.gramps in example/gramps
Do you think it's a good idea ?

I suppressed ", TOOL_MODE_CLI" in tools.gpr.py

What is the best way to remove the three last patchs ? OK finaly done with :
$ git undo 3
$ git push --force
I don't understand what the errors are. The file import and works correctly in gramps.
perhaps it is better to add another test file.

I propose to add two example files :
child-father-child-loop.gramps
test_complex_loop.gramp

The help file for wiki 5.0 is done. perhaps we need some corrections.

If you agree, it's OK for me.
This is ready to go.

Looks good.

The rest of the tool and report pages link to the user wiki using the name of the report/tool as shown on the menu, can we keep it consistent?

eg: I suggest either change the menu entry to "Find possible loop in the database" which I believe is to long, or rename the menu entry to say something more meaningful (in other languages also) like "Find Ancestral loops" and update the link to the wiki ?

The following warning is output(only while using the tool - one for each example database) :

$ ./Gramps.py 
(Gramps.py:3071): Gtk-WARNING **: Could not lookup object notrelated on signal clicked of object close
(Gramps.py:3071): Gtk-WARNING **: Could not lookup object notrelated on signal clicked of object close

Gramps Settings:
----------------
 python    : 3.4.3
 gramps    : 5.0.0-23ab7eb

 gtk++     : 3.10.8
 pygobject : 3.12.0
 pango     : 1.36.3

PS:[Fixed 20160412] I'll fix the wiki page (looks like you copied the 4.0 page instead of the 4.2 so it does not have all my changes)

As far as names, I'd suggest something more generic,
such as "Find Database Loops" etc. That assumes that
the tool might be enhanced some time in the future. But
I would not say "ancestral" as that seems limiting to me.

Also, as long as I'm offering opinions, I'd say that instead of
being in Utilities it would better belong in Family Tree Repair
(where things like Check and Repair Database are) or else in
Family Tree Processing (where things like Find Possible
Duplicate People are). But I find the whole Tools menus and
submenus somewhat inconsistent generally, since after all
Utilities has Verify the Data and all three of those seem to
me to be roughly in the same area. Oh well. In the final
analysis I doubt any of it really matters, but certainly before
5.0.0 is released would be the time to shuffle things IF that
was going to happen.

PS:[Fixed 20160412] I'll fix the wiki page (looks like you copied the 4.0 page instead of the 4.2 so it does not have all my changes)

Sorry. I saw you made modifications. Do I need to restaure some data ? is it correct for you ?

Gtk-WARNING **: Could not lookup object notrelated on signal clicked of object

I think this is related to the glade file. Done.

As far as names, I'd suggest something more generic, such as "Find Database Loops"

Done.

@SNoiraud Sorry. I saw you made modifications. Do I need to restaure some data ? is it correct for you ?

No I've already fixed it.

Tested with the each of the example files and it works 👍

Thanks for all your efforts