fix: Ensure that ndex size is considered when setting autofilter range.

feat: Throw an error when autofilter set for 'odf' engine.
feat: Shows warning when autofilter is set together with merge_cells.
chore: Added tests for index=True, startrow/startcol, multindex with merge_cells.

Author: jarent-nvidia

pandas/io/excel/_odswriter.py

@@ -104,6 +104,10 @@ def _write_cells(
         """
         Write the frame cells using odf
         """
+
+        if autofilter_range:
+            raise ValueError("Autofilter is not supported with odf!")
+
         from odf.table import (
             Table,
             TableCell,

pandas/io/formats/excel.py

@@ -949,8 +949,26 @@ def write(
             )
 
         if self.autofilter:
+            if num_cols == 0:
+                indexoffset = 0
+            elif self.index:
+                if isinstance(self.df.index, MultiIndex):
+                    indexoffset = self.df.index.nlevels - 1
+                    if self.merge_cells:
+                        warnings.warn(
+                            "Excel filters merged cells by showing only the first row."
+                            "'autofiler' and 'merge_cells' should not "
+                            "be used simultaneously.",
+                            UserWarning,
+                            stacklevel=find_stack_level(),
+                        )
+                else:
+                    indexoffset = 0
+            else:
+                indexoffset = -1
             start = f"{self._num2excel(startcol)}{startrow + 1}"
-            end = f"{self._num2excel(startcol + num_cols)}{startrow + num_rows + 1}"
+            autofilter_end_column = self._num2excel(startcol + num_cols + indexoffset)
+            end = f"{autofilter_end_column}{startrow + num_rows + 1}"
             autofilter_range = f"{start}:{end}"
         else:
             autofilter_range = None

pandas/tests/io/excel/test_style.py

@@ -353,16 +353,84 @@ def test_format_hierarchical_rows_periodindex(merge_cells):
 
 
 @pytest.mark.parametrize("engine", ["xlsxwriter", "openpyxl"])
-def test_autofilter(engine, tmp_excel):
+@pytest.mark.parametrize("with_index", [True, False])
+def test_autofilter(engine, with_index, tmp_excel):
     # GH 61194
     df = DataFrame.from_dict([{"A": 1, "B": 2, "C": 3}, {"A": 4, "B": 5, "C": 6}])
 
     with ExcelWriter(tmp_excel, engine=engine) as writer:
-        df.to_excel(writer, autofilter=True, index=False)
+        df.to_excel(writer, autofilter=True, index=with_index)
 
     openpyxl = pytest.importorskip("openpyxl")  # test loading only with openpyxl
     with contextlib.closing(openpyxl.load_workbook(tmp_excel)) as wb:
         ws = wb.active
 
         assert ws.auto_filter.ref is not None
-        assert ws.auto_filter.ref == "A1:D3"
+        assert ws.auto_filter.ref == "A1:D3" if with_index else "A1:C3"
+
+
+@pytest.mark.parametrize("engine", ["xlsxwriter", "openpyxl"])
+def test_autofilter_with_startrow_startcol(engine, tmp_excel):
+    # GH 61194
+    df = DataFrame.from_dict([{"A": 1, "B": 2, "C": 3}, {"A": 4, "B": 5, "C": 6}])
+    with ExcelWriter(tmp_excel, engine=engine) as writer:
+        df.to_excel(writer, autofilter=True, startrow=10, startcol=10)
+
+    openpyxl = pytest.importorskip("openpyxl")  # test loading only with openpyxl
+    with contextlib.closing(openpyxl.load_workbook(tmp_excel)) as wb:
+        ws = wb.active
+        assert ws.auto_filter.ref is not None
+        # Autofiler range moved by 10x10 cells
+        assert ws.auto_filter.ref == "K11:N13"
+
+
+def test_autofilter_not_supported_by_odf(tmp_path):
+    # GH 61194
+    # odf needs 'ods' extension
+    tmp_excel_ods = tmp_path / f"{uuid.uuid4()}.ods"
+    tmp_excel_ods.touch()
+
+    with pytest.raises(ValueError, match="Autofilter is not supported with odf!"):
+        with ExcelWriter(str(tmp_excel_ods), engine="odf") as writer:
+            DataFrame().to_excel(writer, autofilter=True, index=False)
+
+
+@pytest.mark.parametrize("engine", ["xlsxwriter", "openpyxl"])
+def test_autofilter_with_multiindex(engine, tmp_excel):
+    # GH 61194
+    df = DataFrame(
+        {
+            "animal": ("horse", "horse", "dog", "dog"),
+            "color of fur": ("black", "white", "grey", "black"),
+            "name": ("Blacky", "Wendy", "Rufus", "Catchy"),
+        }
+    )
+    mi_df = df.set_index(["animal", "color of fur"])
+    with ExcelWriter(tmp_excel, engine=engine) as writer:
+        mi_df.to_excel(writer, autofilter=True, index=True, merge_cells=False)
+
+    openpyxl = pytest.importorskip("openpyxl")  # test loading only with openpyxl
+    with contextlib.closing(openpyxl.load_workbook(tmp_excel)) as wb:
+        ws = wb.active
+
+        assert ws.auto_filter.ref is not None
+        assert ws.auto_filter.ref == "A1:C5"
+
+
+def test_autofilter_with_multiindex_and_merge_cells_shows_warning(tmp_excel):
+    # GH 61194
+    df = DataFrame(
+        {
+            "animal": ("horse", "horse", "dog", "dog"),
+            "color of fur": ("black", "white", "grey", "black"),
+            "name": ("Blacky", "Wendy", "Rufus", "Catchy"),
+        }
+    )
+    mi_df = df.set_index(["animal", "color of fur"])
+    with ExcelWriter(tmp_excel, engine="openpyxl") as writer:
+        with tm.assert_produces_warning(
+            UserWarning,
+            match="Excel filters merged cells by showing only the first row."
+            "'autofiler' and 'merge_cells' should not be used simultaneously.",
+        ):
+            mi_df.to_excel(writer, autofilter=True, index=True, merge_cells=True)