fix(python): Throw exception if dataframe is too large to be compatible with Excel

[banflam]

fixes #20870

I have what I believe is a working solution and it is passing all the pre-existing tests. However, I don't know how to test whether my change is creating the expected behavior or not (when the dataframe is too large to be compatible with Excel, an exception should be thrown).

This is my very first commit to an open-source project, so I apologize in advance for any mistakes I might have made and I look forward to hearing any and all feedback.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.77%. Comparing base (7bf42dd) to head (f1ff63f).
Report is 7 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #20900      +/-   ##
==========================================
+ Coverage   79.76%   79.77%   +0.01%     
==========================================
  Files        1569     1569              
  Lines      222895   222911      +16     
  Branches     2573     2577       +4     
==========================================
+ Hits       177782   177827      +45     
+ Misses      44521    44492      -29     
  Partials      592      592              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[alexander-beedie]

Thanks for taking the time to contribute!
A couple of notes:

1. You need to account for the headers, which will also write a row, not just the data (so we will write +1 rows when include_headers is True).
2. The table can start from a cell other than A1, so you also need to account for a starting offset (see the position parameter).
3. While we're at it we should probably also check the max width too; Excel's max there is 16384 columns 🤔

[banflam]

Thank you for your feedback and guidance. I've made the changes.

With respect to the second point, it appears that position can be either a tuple representing the numerical offset from the first Excel cell, or a string representing the starting cell directly. I have accounted for the tuple but I'm not entirely sure how to parse the string to get an offset. Please let me know what to do.

[alexander-beedie]

I have accounted for the tuple but I'm not entirely sure how to parse the string to get an offset. Please let me know what to do.

The table_start variable holds this offset (zero-indexed), so you can use that; it will have already done the string conversion for you (if position was provided that way) 👍