docs: fix readthedocs timeouts

[rchowell]

Changes Made

Working on a readthedocs build atm, last successful one took 23 mins and we have a 30 min timeout. These changes got the build just under 8 mins.

• Removed I/O documents (sad)
• Removed ExpressionVisitor API docs.
• Added back requirements-doc.txt only for RTD
• Added back "DataFrame Creation"
• Removed RTD htmlzip

Timeline

• May 12, 2:25 pm — build for #f4da379e — success 11m 17s
• May 13, 11:25 am — readthedocs.org 13.1.1 release
• May 13, 4:20 pm — build for #fa946cfb — failure timeout > 30m
• May 14, 11:07 am — build for #f4da379e — success 10m 52s

Worklog

Just kicked off a build with redirects removed and typos addressed, https://app.readthedocs.org/projects/getdaft-docs/builds/28174757/

Seems HTMLZip takes a long time, and it isn't required (?). I don't believe we expose the ability to download this zip so it's unused afaik, but please correct me if I'm wrong!

It was introduced here, then PDF caused issues and was removed.

• [DOCS] Build PDF and htmlzip formats of docs for offline consumption #2538
• [DOCS] Do not generate PDFs #2539

It appears our builds were taking ~10 mins (+/- 5) but then jumped to 23 mins after this change. Then builds starting timing out and failing after this change.

Attempts

• I've kicked off a readthedocs build for a version before both changes, https://app.readthedocs.org/projects/getdaft-docs/builds/28175216/
• Now I've removed the visitor to see if this was the culprit. https://app.readthedocs.org/projects/getdaft-docs/builds/28175794/ (timeout!)
• This time I'm removing redirects. https://app.readthedocs.org/projects/getdaft-docs/builds/28176283/
• Trying with the visitor commit BUT removed changes to docs. https://app.readthedocs.org/projects/getdaft-docs/builds/28176466/ (success)
• Trying with the visitor and i/o docs BUT removed mkredirects. https://app.readthedocs.org/projects/getdaft-docs/builds/28176771/ (timeout)
• Trying to build latest, but with feat: adds user-defined dataframe source apis #4254 and feat: adds expression visitor #4278 removed, https://app.readthedocs.org/projects/getdaft-docs/builds/28177214/ (success)

Checklist

• Documented in API Docs (if applicable)
• Documented in User Guide (if applicable)
• If adding a new documentation page, doc is added to docs/mkdocs.yml navigation
• Documentation builds and is formatted properly (tag @/ccmao1130 for docs review)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.40%. Comparing base (6b855de) to head (1d90f4a).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4350   +/-   ##
=======================================
  Coverage   78.40%   78.40%           
=======================================
  Files         823      823           
  Lines      110068   110070    +2     
=======================================
+ Hits        86301    86304    +3     
+ Misses      23767    23766    -1     

Files with missing lines	Coverage Δ
daft/catalog/__init__.py	71.12% <100.00%> (ø)
daft/io/__shim.py	86.48% <100.00%> (ø)
daft/io/pushdowns.py	94.53% <ø> (ø)
daft/session.py	65.88% <ø> (ø)

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.