feat(python)!: Change default engine for read_excel to "calamine"

[stinodego]

Closes #17177

Changes

• Set "calamine" as the default engine for read_excel for all file types, instead of only for .xlsb and .xls.
  • Note that the "calamine" engine does not accept the engine_options parameter (see example below).
• Added a common columns parameter that can constrain which columns are returned (ref: Combine engine_options and read_options into a single parameter in read_excel #17265).

Example

Before:

>>> pl.read_excel("data.xlsx", engine_options={"skip_empty_lines": True})

After:

>>> pl.read_excel("data.xlsx", engine_options={"skip_empty_lines": True})
Traceback (most recent call last):
...
TypeError: read_excel() got an unexpected keyword argument 'skip_empty_lines'

Instead, explicitly specify the xlsx2csv engine or omit the engine_options:

>>> pl.read_excel("data.xlsx", engine="xlsx2csv", engine_options={"skip_empty_lines": True})

[codecov]

Codecov Report

Attention: Patch coverage is 78.12500% with 7 lines in your changes missing coverage. Please review.

Project coverage is 80.71%. Comparing base (d444b79) to head (5c6151e).
Report is 1 commits behind head on main.

Files	Patch %	Lines
py-polars/polars/io/spreadsheet/functions.py	86.20%	3 Missing and 1 partial ⚠️
py-polars/polars/selectors.py	0.00%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #17263      +/-   ##
==========================================
- Coverage   80.72%   80.71%   -0.01%     
==========================================
  Files        1475     1475              
  Lines      193162   193181      +19     
  Branches     2751     2756       +5     
==========================================
- Hits       155922   155919       -3     
- Misses      36732    36752      +20     
- Partials      508      510       +2     

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

[stinodego]

This part of the logic wasn't being hit anymore since engine cannot be None. Not sure if we still need this.

[stinodego]

@alexander-beedie for the columns parameter, what happens when multiple sheets are read? Shouldn't it accept a mapping of sheet to column names?