I don't have this fancy 'orange' of which jim speaks
cull_nonmatching_trees method added

This gives a way to address:
https://github.com/OpenTreeOfLife/phylesystem-api/issues/135

If cull_nonmatching=true is passed into a call that would
previously return a NexSON tree fragment then the response
will now be a NexSON with the nonmatching trees omitted.

These trees will not be in the lists (and empty trees groups
will be culled). But there can be dangling reference to trees
in the full study in some of the annotations. So the client
code must be prepared to deal with some broken references if
it wants to traverse all of the annotations.
conversion to requested nexson version.

Unlike with other forms of slicing a study into pieces, when
we use the cull_nonmatching... feature, we can support
conversion to the user's requested version of the NexSON
syntax. This new code and test make sure that this works.
raise exception for unexpected cull_nonmatching value
resolved conflict
figure update
incorporating jfa comments
fixed textsize in figure
changed working on study import
this update of _study2shard_map should have had the mutex
some checks on the ID syntax
the int check was silly given the regex test
remove py2 print
Updated the maintainers phylesystem dummy repos

These are useful for testing phylesystem workflows, but they
were implemented before our id -> filepath mapping was
finalized.  This commit only affects testing code. It brings
the tests up to date with changes to
https://github.com/mtholder/mini_phyl
2d59ab892ddb3d09d4b18c91470b8c1c4cca86dc
and https://github.com/mtholder/mini_system
d43f6beb504a08b08b760f49589a54a00025a399

These changes make the test system a closer match for how
        we use the code in open tree web services.
New test which reveals an ID deletion bug.

Thanks to @jimallman for pointing out that this was a likely
problem spot!. The ID should not be removed from the
study2shard_map unless the commit merges to master! (bug not fixed
with this commit)
corrected version of the test
bug fix: only removing study ID from mapping when merged to master
More thorough tests

Makes sure that a study ID disappears from the index when
deleted, but you can still fetch an old version of it.
Reinstating it is still not tested.
bug fix for deleting study ID + getting old copy

If the study ID is not found in the study_id2shard map, then
a simple rule to look up the shard based on the prefix is used.

This works, because the shard does not actually need to list the study
in its study_id2file_path mapping - it knows the filesystem location
to check. If there is no content there, then we know that (for the
commit SHA requested) that study did not exist.

Implementing this lets us retrieve old versions of deleted studies
using the phylesystem-api.

WARNING: should we ever deploy multiple shards with the same prefix,
the fallback mechanism of using the prefix to designate the shard
will not work. We'll probably just move to a system of ranges
at that point (ot_0 - ot_3000 in this shard, ot_3001 - in the other
one). This fallback code in phylesystem_umbrella get_shard will
need to be tweaked based on what we decide to do then.

OUTSTANDING issue: recommitting a study does not put it back into
the study_id2shard mapping. So such studies won't appear in the
study_list web service call. We actually use the list from OTI, so this
is not a major problem. You'll still be able to GET a reinstated study,
it just won't be in the fast dict (or any routine that relies on that
such as iterating over all studies in a peyotl.phylesystem interaction).
This is not used in the phylesystem-api AFAIK, so not a huge deal...
test to reveal the reinstating the study bug mentioned in prev commit