bokeh__bokeh-13289

{
  "status": "ok",
  "stdout": "src/bokeh/models/renderers/glyph_renderer.py:170:    def construct_color_bar(self, **kwargs: Any) -> ColorBar:\nsrc/bokeh/models/renderers/glyph_renderer.py:223:            raise ValueError(f\"construct_color_bar does not handle glyph type {type(self.glyph).__name__}\")\nsrc/bokeh/models/renderers/contour_renderer.py:100:    def construct_color_bar(self, **kwargs) -> ContourColorBar:\n",
  "stderr": "",
  "exit_code": 0
}

{
  "status": "ok",
  "filepath": "src/bokeh/models/renderers/glyph_renderer.py",
  "content": "    def construct_color_bar(self, **kwargs: Any) -> ColorBar:\n        ''' Construct and return a new ``ColorBar`` for this ``GlyphRenderer``.\n\n        The function will check for a color mapper on an appropriate property\n        of the GlyphRenderer's main glyph, in this order:\n\n        * ``fill_color.transform`` for FillGlyph\n        * ``line_color.transform`` for LineGlyph\n        * ``text_color.transform`` for TextGlyph\n        * ``color_mapper`` for Image\n\n        In general, the function will \"do the right thing\" based on glyph type.\n        If different behavior is needed, ColorBars can be constructed by hand.\n\n        Extra keyword arguments may be passed in to control ``ColorBar``\n        properties such as `title`.\n\n        Returns:\n            ColorBar\n\n        '''\n        from ...core.property.vectorization import Field\n        from ..annotations import ColorBar\n        from ..glyphs import (\n            FillGlyph,\n            Image,\n            LineGlyph,\n            TextGlyph,\n        )\n        from ..mappers import ColorMapper\n\n        if isinstance(self.glyph, FillGlyph):\n            fill_color = self.glyph.fill_color\n            if not (isinstance(fill_color, Field) and isinstance(fill_color.transform, ColorMapper)):\n                raise ValueError(\"expected fill_color to be a field with a ColorMapper transform\")\n            return ColorBar(color_mapper=fill_color.transform, **kwargs)\n\n        elif isinstance(self.glyph, LineGlyph):\n            line_color = self.glyph.line_color\n            if not (isinstance(line_color, Field) and isinstance(line_color.transform, ColorMapper)):\n                raise ValueError(\"expected line_color to be a field with a ColorMapper transform\")\n            return ColorBar(color_mapper=line_color.transform, **kwargs)\n\n        elif isinstance(self.glyph, TextGlyph):\n            text_color = self.glyph.text_color\n            if not (isinstance(text_color, Field) and isinstance(text_color.transform, ColorMapper)):\n                raise ValueError(\"expected text_color to be a field with a ColorMapper transform\")\n            return ColorBar(color_mapper=text_color.transform, **kwargs)\n\n        elif type(self.glyph) is Image:\n            return ColorBar(color_mapper=self.glyph.color_mapper, **kwargs)\n\n        else:\n            raise ValueError(f\"construct_color_bar does not handle glyph type {type(self.glyph).__name__}\")\n\n#-----------------------------------------------------------------------------\n# Dev API\n#-----------------------------------------------------------------------------\n\n#-----------------------------------------------------------------------------\n# Private API",
  "start_line": 170,
  "end_line": 230,
  "total_lines": 235,
  "is_truncated": false
}

{
  "status": "error",
  "error_type": "CommandError",
  "error_message": "grep: src/bokeh/glyphs: No such file or directory\n",
  "details": {
    "stdout": "",
    "stderr": "grep: src/bokeh/glyphs: No such file or directory\n",
    "exit_code": 2
  }
}

{
  "status": "ok",
  "stdout": "src/bokeh/models/glyphs.py:834:class ImageStack(ImageBase):\n",
  "stderr": "",
  "exit_code": 0
}

{
  "status": "ok",
  "stdout": "718:class ImageBase(XYGlyph):\n768:class Image(ImageBase):\n819:class ImageRGBA(ImageBase):\n834:class ImageStack(ImageBase):\n862:class ImageURL(XYGlyph):\n",
  "stderr": "",
  "exit_code": 0
}

{
  "status": "ok",
  "filepath": "src/bokeh/models/glyphs.py",
  "content": "class ImageBase(XYGlyph):\n\n    # explicit __init__ to support Init signatures\n    def __init__(self, *args, **kwargs) -> None:\n        super().__init__(*args, **kwargs)\n\n    x = NumberSpec(default=field(\"x\"), help=\"\"\"\n    The x-coordinates to locate the image anchors.\n    \"\"\")\n\n    y = NumberSpec(default=field(\"y\"), help=\"\"\"\n    The y-coordinates to locate the image anchors.\n    \"\"\")\n\n    dw = DistanceSpec(default=field(\"dw\"), help=\"\"\"\n    The widths of the plot regions that the images will occupy.\n\n    .. note::\n        This is not the number of pixels that an image is wide.\n        That number is fixed by the image itself.\n    \"\"\")\n\n    dh = DistanceSpec(default=field(\"dh\"), help=\"\"\"\n    The height of the plot region that the image will occupy.\n\n    .. note::\n        This is not the number of pixels that an image is tall.\n        That number is fixed by the image itself.\n    \"\"\")\n\n    image_props = Include(ImageProps, help=\"\"\"\n    The {prop} values for the images.\n    \"\"\")\n\n    dilate = Bool(False, help=\"\"\"\n    Whether to always round fractional pixel locations in such a way\n    as to make the images bigger.\n\n    This setting may be useful if pixel rounding errors are causing\n    images to have a gap between them, when they should appear flush.\n    \"\"\")\n\n    origin = Enum(ImageOrigin, default=\"bottom_left\", help=\"\"\"\n    Defines the coordinate space of an image.\n    \"\"\")\n\n    anchor = Anchor(default=\"bottom_left\", help=\"\"\"\n    Position of the image should be anchored at the `x`, `y` coordinates.\n    \"\"\")\n\nclass Image(ImageBase):\n    ''' Render images given as scalar data together with a color mapper.\n\n    In addition to the defined model properties, ``Image`` also can accept\n    a keyword argument ``palette`` in place of an explicit ``color_mapper``.\n    The value should be a list of colors, or the name of one of the built-in\n    palettes in ``bokeh.palettes``. This palette will be used to automatically\n    construct a ``ColorMapper`` model for the ``color_mapper`` property.\n\n    If both ``palette`` and ``color_mapper`` are passed, a ``ValueError``\n    exception will be raised. If neither is passed, then the ``Greys9``\n    palette will be used as a default.\n\n    '''\n\n    def __init__(self, *args, **kwargs) -> None:\n        if 'palette' in kwargs and 'color_mapper' in kwargs:\n            raise ValueError(\"only one of 'palette' and 'color_mapper' may be specified\")\n        elif 'color_mapper' not in kwargs:\n            # Use a palette (given or default)\n            palette = kwargs.pop('palette', 'Greys9')\n            mapper = LinearColorMapper(palette)\n            kwargs['color_mapper'] = mapper\n\n        super().__init__(*args, **kwargs)\n\n    _args = ('image', 'x', 'y', 'dw', 'dh', 'dilate')\n\n    # a hook to specify any additional kwargs handled by an initializer\n    _extra_kws = {\n        'palette': (\n            'str or list[color value]',\n            'a palette to construct a value for the color mapper property from',\n        ),\n    }\n\n    image = NumberSpec(default=field(\"image\"), help=\"\"\"\n    The arrays of scalar data for the images to be colormapped.\n    \"\"\")\n\n    color_mapper = Instance(ColorMapper, default=InstanceDefault(LinearColorMapper, palette=\"Greys9\"), help=\"\"\"\n    A ``ColorMapper`` to use to map the scalar data from ``image``\n    into RGBA values for display.\n\n    The name of a palette from ``bokeh.palettes`` may also be set, in which\n    case a ``LinearColorMapper`` configured with the named palette wil be used.\n\n    .. note::\n        The color mapping step happens on the client.\n    \"\"\").accepts(Enum(Palette), lambda pal: LinearColorMapper(palette=pal))\n\nclass ImageRGBA(ImageBase):\n    ''' Render images given as RGBA data.\n\n    '''\n\n    # explicit __init__ to support Init signatures\n    def __init__(self, *args, **kwargs) -> None:\n        super().__init__(*args, **kwargs)\n\n    _args = ('image', 'x', 'y', 'dw', 'dh', 'dilate')\n\n    image = NumberSpec(default=field(\"image\"), help=\"\"\"\n    The arrays of RGBA data for the images.\n    \"\"\")\n\nclass ImageStack(ImageBase):\n    ''' Render images given as 3D stacked arrays by flattening each stack into\n    an RGBA image using a ``StackColorMapper``.\n\n    The 3D arrays have shape (ny, nx, nstack) where ``nstack` is the number of\n    stacks. The ``color_mapper`` produces an RGBA value for each of the\n    (ny, nx) pixels by combining array values in the ``nstack`` direction.\n\n    '''\n\n    # explicit __init__ to support Init signatures\n    def __init__(self, *args, **kwargs) -> None:\n        super().__init__(*args, **kwargs)\n\n    _args = ('image', 'x', 'y', 'dw', 'dh', 'dilate')\n\n    image = NumberSpec(default=field(\"image\"), help=\"\"\"",
  "start_line": 718,
  "end_line": 850,
  "total_lines": 1835,
  "is_truncated": false
}

{
  "status": "ok",
  "filepath": "src/bokeh/models/glyphs.py",
  "content": "class ImageStack(ImageBase):\n    ''' Render images given as 3D stacked arrays by flattening each stack into\n    an RGBA image using a ``StackColorMapper``.\n\n    The 3D arrays have shape (ny, nx, nstack) where ``nstack` is the number of\n    stacks. The ``color_mapper`` produces an RGBA value for each of the\n    (ny, nx) pixels by combining array values in the ``nstack`` direction.\n\n    '''\n\n    # explicit __init__ to support Init signatures\n    def __init__(self, *args, **kwargs) -> None:\n        super().__init__(*args, **kwargs)\n\n    _args = ('image', 'x', 'y', 'dw', 'dh', 'dilate')\n\n    image = NumberSpec(default=field(\"image\"), help=\"\"\"\n    The 3D arrays of data for the images.\n    \"\"\")\n\n    color_mapper = Instance(StackColorMapper, help=\"\"\"\n    ``ScalarColorMapper`` used to map the scalar data from ``image``\n    into RGBA values for display.\n\n    .. note::\n        The color mapping step happens on the client.\n    \"\"\")\n\nclass ImageURL(XYGlyph):\n    ''' Render images loaded from given URLs.\n\n    '''\n\n    # explicit __init__ to support Init signatures\n    def __init__(self, *args, **kwargs) -> None:\n        super().__init__(*args, **kwargs)\n",
  "start_line": 834,
  "end_line": 870,
  "total_lines": 1835,
  "is_truncated": false
}

{
  "status": "ok",
  "stdout": "Caught expected error: construct_color_bar does not handle glyph type ImageStack\n",
  "stderr": "",
  "exit_code": 0
}

{
  "status": "ok",
  "filepath": "src/bokeh/models/renderers/glyph_renderer.py",
  "occurrences": 1,
  "strategy": "unescaped",
  "diff": "--- a/src/bokeh/models/renderers/glyph_renderer.py\n+++ b/src/bokeh/models/renderers/glyph_renderer.py\n@@ -193,6 +193,7 @@\n         from ..glyphs import (\n             FillGlyph,\n             Image,\n+            ImageStack,\n             LineGlyph,\n             TextGlyph,\n         )\n",
  "is_truncated": false
}

{
  "status": "ok",
  "filepath": "src/bokeh/models/renderers/glyph_renderer.py",
  "occurrences": 1,
  "strategy": "unescaped",
  "diff": "--- a/src/bokeh/models/renderers/glyph_renderer.py\n+++ b/src/bokeh/models/renderers/glyph_renderer.py\n@@ -217,7 +217,7 @@\n                 raise ValueError(\"expected text_color to be a field with a ColorMapper transform\")\n             return ColorBar(color_mapper=text_color.transform, **kwargs)\n \n-        elif type(self.glyph) is Image:\n+        elif isinstance(self.glyph, (Image, ImageStack)):\n             return ColorBar(color_mapper=self.glyph.color_mapper, **kwargs)\n \n         else:\n",
  "is_truncated": false
}

{
  "status": "ok",
  "stdout": "Success\n",
  "stderr": "",
  "exit_code": 0
}

{
  "status": "ok",
  "stdout": "Success\n",
  "stderr": "",
  "exit_code": 0
}

{
  "status": "ok",
  "filepath": "src/bokeh/models/renderers/contour_renderer.py",
  "content": "            line_data = data.line_data.asdict()\n            # Copy line properties from old to new data source\n            old_line_data = self.line_renderer.data_source.data\n            for name in old_line_data.keys():\n                if name not in (\"xs\", \"ys\", \"levels\"):\n                    line_data[name] = old_line_data[name]\n            self.line_renderer.data_source.data = line_data\n        else:\n            self.line_renderer.data_source.data = dict(xs=[], ys=[], levels=[])\n\n    def construct_color_bar(self, **kwargs) -> ContourColorBar:\n        ''' Construct and return a new ``ContourColorBar`` for this ``ContourRenderer``.\n\n        The color bar will use the same fill, hatch and line visual properties\n        as the ContourRenderer. Extra keyword arguments may be passed in to\n        control ``BaseColorBar`` properties such as `title`.\n        '''\n        from ..annotations import ContourColorBar\n        from ..tickers import FixedTicker\n        return ContourColorBar(\n            fill_renderer=self.fill_renderer,",
  "start_line": 90,
  "end_line": 110,
  "total_lines": 127,
  "is_truncated": false
}

{
  "status": "ok",
  "stdout": "Caught expected error: construct_color_bar does not handle glyph type ImageRGBA\n",
  "stderr": "",
  "exit_code": 0
}

{
  "status": "ok",
  "stdout": " M src/bokeh/models/renderers/glyph_renderer.py\ndiff --git a/src/bokeh/models/renderers/glyph_renderer.py b/src/bokeh/models/renderers/glyph_renderer.py\nindex 4172018..1da7aa8 100644\n--- a/src/bokeh/models/renderers/glyph_renderer.py\n+++ b/src/bokeh/models/renderers/glyph_renderer.py\n@@ -193,6 +193,7 @@ class GlyphRenderer(DataRenderer):\n         from ..glyphs import (\n             FillGlyph,\n             Image,\n+            ImageStack,\n             LineGlyph,\n             TextGlyph,\n         )\n@@ -216,7 +217,7 @@ class GlyphRenderer(DataRenderer):\n                 raise ValueError(\"expected text_color to be a field with a ColorMapper transform\")\n             return ColorBar(color_mapper=text_color.transform, **kwargs)\n \n-        elif type(self.glyph) is Image:\n+        elif isinstance(self.glyph, (Image, ImageStack)):\n             return ColorBar(color_mapper=self.glyph.color_mapper, **kwargs)\n \n         else:\n",
  "stderr": "",
  "exit_code": 0
}

{
  "status": "error",
  "error_type": "CommandError",
  "error_message": "/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/pytest_asyncio/plugin.py:208: PytestDeprecationWarning: The configuration option \"asyncio_default_fixture_loop_scope\" is unset.\nThe event loop scope for asynchronous fixtures will default to the fixture caching scope. Future versions of pytest-asyncio will default the loop scope for asynchronous fixtures to function scope. Set the default fixture loop scope explicitly in order to avoid unexpected behavior in the future. Valid fixture loop scopes are: \"function\", \"class\", \"module\", \"package\", \"session\"\n\n  warnings.warn(PytestDeprecationWarning(_DEFAULT_FIXTURE_LOOP_SCOPE_UNSET))\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/pytest_asyncio/plugin.py:208: PytestDeprecationWarning: The configuration option \"asyncio_default_fixture_loop_scope\" is unset.\nThe event loop scope for asynchronous fixtures will default to the fixture caching scope. Future versions of pytest-asyncio will default the loop scope for asynchronous fixtures to function scope. Set the default fixture loop scope explicitly in order to avoid unexpected behavior in the future. Valid fixture loop scopes are: \"function\", \"class\", \"module\", \"package\", \"session\"\n\n  warnings.warn(PytestDeprecationWarning(_DEFAULT_FIXTURE_LOOP_SCOPE_UNSET))\n",
  "details": {
    "stdout": "\nno tests ran in 0.29s\n\n==================================== ERRORS ====================================\n__________ ERROR collecting tests/integration/embed/test_json_item.py __________\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/pytest_asyncio/plugin.py:667: in _patched_collect\n    module = collector.obj\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/python.py:284: in obj\n    self._obj = obj = self._getobj()\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/python.py:546: in _getobj\n    return importtestmodule(self.path, self.config)\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/python.py:538: in importtestmodule\n    config.pluginmanager.consider_module(mod)\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/config/__init__.py:824: in consider_module\n    self._import_plugin_specs(getattr(mod, \"pytest_plugins\", []))\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/config/__init__.py:831: in _import_plugin_specs\n    self.import_plugin(import_spec)\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/config/__init__.py:868: in import_plugin\n    self.register(mod, modname)\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/config/__init__.py:512: in register\n    self.consider_module(plugin)\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/config/__init__.py:824: in consider_module\n    self._import_plugin_specs(getattr(mod, \"pytest_plugins\", []))\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/config/__init__.py:831: in _import_plugin_specs\n    self.import_plugin(import_spec)\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/config/__init__.py:868: in import_plugin\n    self.register(mod, modname)\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/_pytest/config/__init__.py:501: in register\n    plugin_name = super().register(plugin, name)\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/pluggy/_manager.py:168: in register\n    self._verify_hook(hook, hookimpl)\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/pluggy/_manager.py:360: in _verify_hook\n    _warn_for_function(argname_warning, hookimpl.function)\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/pluggy/_manager.py:41: in _warn_for_function\n    warnings.warn_explicit(\nE   pytest.PytestRemovedIn9Warning: The (startdir: py.path.local) argument is deprecated, please use (start_path: pathlib.Path)\nE   see https://docs.pytest.org/en/latest/deprecations.html#py-path-local-arguments-for-hooks-replaced-with-pathlib-path\n____________ ERROR collecting tests/unit/bokeh/models/test_plots.py ____________\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/xyzservices/lib.py:45: in __getattr__\n    return self.__getitem__(key)\nE   KeyError: 'Stamen'\n\nThe above exception was the direct cause of the following exception:\ntests/unit/bokeh/models/test_plots.py:335: in <module>\n    (\"STAMEN_TERRAIN\", xyz.Stamen.Terrain),\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/xyzservices/lib.py:47: in __getattr__\n    raise AttributeError(key) from err\nE   AttributeError: Stamen\n___________ ERROR collecting tests/unit/bokeh/test_tile_providers.py ___________\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/xyzservices/lib.py:45: in __getattr__\n    return self.__getitem__(key)\nE   KeyError: 'Stamen'\n\nThe above exception was the direct cause of the following exception:\ntests/unit/bokeh/test_tile_providers.py:52: in <module>\n    'STAMEN_TERRAIN':          xyz.Stamen.Terrain.build_url(),\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/xyzservices/lib.py:47: in __getattr__\n    raise AttributeError(key) from err\nE   AttributeError: Stamen\n=========================== short test summary info ============================\nERROR tests/integration/embed/test_json_item.py - pytest.PytestRemovedIn9Warn...\nERROR tests/unit/bokeh/models/test_plots.py - AttributeError: Stamen\nERROR tests/unit/bokeh/test_tile_providers.py - AttributeError: Stamen\n!!!!!!!!!!!!!!!!!!! Interrupted: 3 errors during collection !!!!!!!!!!!!!!!!!!!!\n10821 deselected, 3 errors in 6.55s\n",
    "stderr": "/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/pytest_asyncio/plugin.py:208: PytestDeprecationWarning: The configuration option \"asyncio_default_fixture_loop_scope\" is unset.\nThe event loop scope for asynchronous fixtures will default to the fixture caching scope. Future versions of pytest-asyncio will default the loop scope for asynchronous fixtures to function scope. Set the default fixture loop scope explicitly in order to avoid unexpected behavior in the future. Valid fixture loop scopes are: \"function\", \"class\", \"module\", \"package\", \"session\"\n\n  warnings.warn(PytestDeprecationWarning(_DEFAULT_FIXTURE_LOOP_SCOPE_UNSET))\n/opt/miniconda3/envs/testbed/lib/python3.10/site-packages/pytest_asyncio/plugin.py:208: PytestDeprecationWarning: The configuration option \"asyncio_default_fixture_loop_scope\" is unset.\nThe event loop scope for asynchronous fixtures will default to the fixture caching scope. Future versions of pytest-asyncio will default the loop scope for asynchronous fixtures to function scope. Set the default fixture loop scope explicitly in order to avoid unexpected behavior in the future. Valid fixture loop scopes are: \"function\", \"class\", \"module\", \"package\", \"session\"\n\n  warnings.warn(PytestDeprecationWarning(_DEFAULT_FIXTURE_LOOP_SCOPE_UNSET))\n",
    "exit_code": 2
  }
}

{
  "status": "ok",
  "stdout": "diff --git a/src/bokeh/models/renderers/glyph_renderer.py b/src/bokeh/models/renderers/glyph_renderer.py\nindex 4172018..1da7aa8 100644\n--- a/src/bokeh/models/renderers/glyph_renderer.py\n+++ b/src/bokeh/models/renderers/glyph_renderer.py\n@@ -193,6 +193,7 @@ class GlyphRenderer(DataRenderer):\n         from ..glyphs import (\n             FillGlyph,\n             Image,\n+            ImageStack,\n             LineGlyph,\n             TextGlyph,\n         )\n@@ -216,7 +217,7 @@ class GlyphRenderer(DataRenderer):\n                 raise ValueError(\"expected text_color to be a field with a ColorMapper transform\")\n             return ColorBar(color_mapper=text_color.transform, **kwargs)\n \n-        elif type(self.glyph) is Image:\n+        elif isinstance(self.glyph, (Image, ImageStack)):\n             return ColorBar(color_mapper=self.glyph.color_mapper, **kwargs)\n \n         else:\n",
  "stderr": "",
  "exit_code": 0
}

{
  "status": "ok",
  "patch_size": 877,
  "files_changed": 1
}