Skip to content

correct api endpoint from search_summary_docs -> search#495

Merged
minhsueh merged 2 commits intomaterialsproject:mainfrom
minhsueh:ctk_app
Dec 1, 2025
Merged

correct api endpoint from search_summary_docs -> search#495
minhsueh merged 2 commits intomaterialsproject:mainfrom
minhsueh:ctk_app

Conversation

@minhsueh
Copy link
Copy Markdown
Collaborator

@minhsueh minhsueh commented Nov 5, 2025

Error message:

Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/crystal_toolkit/components/search.py", line 167, in update_results
    results = get_human_readable_results_from_search_term(search_term)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask_caching/__init__.py", line 899, in decorated_function
    rv = self._call_fn(f, *args, **kwargs)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask_caching/__init__.py", line 185, in _call_fn
    return ensure_sync(fn)(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/crystal_toolkit/components/search.py", line 123, in get_human_readable_results_from_search_term
    entries = mpr.summary.search_summary_docs(
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'SummaryRester' object has no attribute 'search_summary_docs'

Root cause:
mpr.summary.search_summary_docs is deprecated

Fix:

- entries = mpr.summary.search_summary_docs(formula=search_term, fields=fields)
+ entries = mpr.summary.search(formula=search_term, fields=fields)

@minhsueh minhsueh requested a review from tschaume November 5, 2025 17:47
@minhsueh minhsueh enabled auto-merge December 1, 2025 17:09
@minhsueh minhsueh disabled auto-merge December 1, 2025 17:10
@minhsueh minhsueh merged commit 8977097 into materialsproject:main Dec 1, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants