improving bounds efficency (#130)

* improving bounds efficency

* typi

* adding test for loading stack

* adding tilebound call and changing getting sectionId from z

* typo

* typo

Author: fcollman

integration_tests/test_stack_integrated.py

@@ -221,6 +221,15 @@ def teststack(request, render, render_example_tilespec_and_transforms):
     yield stack
     render.run(renderapi.stack.delete_stack, stack)
 
+@pytest.fixture(scope="module")
+def teststack_loading(request, render, render_example_tilespec_and_transforms):
+    (tilespecs, tforms) = render_example_tilespec_and_transforms
+    stack = 'test_insert_loading'
+    r = render.run(renderapi.stack.create_stack, stack, force_resolution=True)
+    render.run(renderapi.client.import_tilespecs, stack, tilespecs,
+               sharedTransforms=tforms)
+    yield stack
+    render.run(renderapi.stack.delete_stack, stack)
 
 def test_stack_bounds(render, teststack):
     # check the stack bounds
@@ -415,12 +424,15 @@ def test_get_tile_specs_from_stack(render, teststack,
     assert len(ts) == len(tilespecs)
 
 
-def test_get_sectionId_for_z(render, teststack,
+def test_get_sectionId_for_z(render, teststack, teststack_loading,
                              render_example_tilespec_and_transforms):
     (tilespecs, tforms) = render_example_tilespec_and_transforms
     sectionId = render.run(
         renderapi.stack.get_sectionId_for_z, teststack, tilespecs[0].z)
     assert (sectionId == tilespecs[0].layout.sectionId)
+    sectionId = render.run(
+        renderapi.stack.get_sectionId_for_z, teststack_loading, tilespecs[0].z)
+    assert (sectionId == tilespecs[0].layout.sectionId)
 
 
 def test_get_resolvedtiles_from_z(render, teststack,

renderapi/stack.py

@@ -649,6 +649,40 @@ def get_stack_bounds(stack, host=None, port=None, owner=None, project=None,
     return get_json(session, request_url)
 
 
+@renderaccess
+def get_tilebounds_for_z(stack, z, host=None, port=None, owner=None,
+                        project=None, session=requests.session(),
+                        render=None, **kwargs):
+    """returns the bounds for each tile associated with a particular z value
+
+    :func:`renderapi.render.renderaccess` decorated function
+
+    Parameters
+    ----------
+    stack : str
+        stack to look within
+    sectionId : str
+        sectionId to find z value
+    render : renderapi.render.Render
+        render connect object
+    session : requests.sessions.Session
+        session object (default start a new one)
+
+    Returns
+    -------
+    list
+        list of dictionaries with tilebounds
+
+    Raises
+    ------
+    RenderError
+
+    """
+
+    request_url = format_preamble(
+        host, port, owner, project, stack) + '/z/{}/tileBounds'.format(z)
+    return get_json(session, request_url)
+
 @renderaccess
 def get_sectionId_for_z(stack, z, host=None, port=None, owner=None,
                         project=None, session=requests.session(),
@@ -678,10 +712,12 @@ def get_sectionId_for_z(stack, z, host=None, port=None, owner=None,
     RenderError
 
     """
-    sectionData = get_stack_sectionData(
-        stack, host, port, owner, project, session)
+  
+
+    bounds = get_tilebounds_for_z(stack, z, host, port, owner, project, session)
+
     try:
-        return next(sd['sectionId'] for sd in sectionData if sd['z'] == z)
+        return bounds[0]['sectionId']
     except Exception as e:
         logger.error(e)
         raise RenderError('Could not find z value %f in stack %s' % (z, stack))