Skip to content

Debugging broken search on esda docs#488

Merged
martinfleis merged 8 commits into
pysal:mainfrom
sjsrey:docs-fix-search
May 8, 2026
Merged

Debugging broken search on esda docs#488
martinfleis merged 8 commits into
pysal:mainfrom
sjsrey:docs-fix-search

Conversation

@sjsrey

@sjsrey sjsrey commented May 6, 2026

Copy link
Copy Markdown
Member

This will get search to work, however, it is unique to esda which feels fragile to me.

@jGaboardi

Copy link
Copy Markdown
Member

Do we know what is making esda a special case for this?

@knaaptime

Copy link
Copy Markdown
Member

this is weird! its on the javascript side, so its clearly the theme/its interaction with sphinx

maybe there is an esda dependency thats forcing an older version of the theme or something? its odd we're not seeing this in, e.g. tobler or lib (nor the alpha package i'm working on separately)

@martinfleis

Copy link
Copy Markdown
Member

I really don't like this solution.

@jGaboardi

jGaboardi commented May 6, 2026

Copy link
Copy Markdown
Member

I really don't like this solution.

The only way this comment could have been more Martin-esque is if "solution" was in quotes.

@sjsrey

sjsrey commented May 7, 2026

Copy link
Copy Markdown
Member Author

Just to be clear, this is intended more as a debugging record rather than a real fix. I will be focusing on the docs in upcoming sprints.

ref: #459

@martinfleis

Copy link
Copy Markdown
Member

It is an upstream bug that has been reported - jbms/sphinx-immaterial#485. We shall temporarily pin sphinx to <9, which is what I have in gwlearn, and what we coincidentally get in libpysal without a pin.

@codecov

codecov Bot commented May 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.8%. Comparing base (8c05bdb) to head (aab619c).

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #488   +/-   ##
=====================================
  Coverage   85.8%   85.8%           
=====================================
  Files         27      27           
  Lines       3625    3625           
=====================================
  Hits        3112    3112           
  Misses       513     513           
Files with missing lines Coverage Δ
esda/getisord.py 88.0% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@knaaptime knaaptime left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i thought the pin was <9?

Comment thread pyproject.toml Outdated
@sjsrey sjsrey marked this pull request as ready for review May 8, 2026 17:53
@jGaboardi

Copy link
Copy Markdown
Member

Are we OK with the errors for now?

https://github.com/pysal/esda/actions/runs/25570840856/job/75065855524?pr=488#step:5:628

@martinfleis martinfleis merged commit e878004 into pysal:main May 8, 2026
17 checks passed
@jGaboardi

Copy link
Copy Markdown
Member

I'm not sure this did the trick. Still nothing happens when attempting to search. Nothing clickable and no results.

Screenshot 2026-05-08 at 4 10 20 PM

@jGaboardi jGaboardi added the docs label May 8, 2026
@martinfleis

Copy link
Copy Markdown
Member

Of course... the pin needs to show up in the Conda environment, not just in pyproject.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants