Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v1.1.0 Search API: Retain nesting of queries, Data API: add querying for all structures #51

Merged
merged 16 commits into from
Mar 12, 2025

Conversation

ivana-truong
Copy link
Collaborator

Search API

  • Changed __and__/__or__ method of SearchQuery and added group method in response to Issue in grouped attribute search #49. Now building queries with group function will cause the group to be preserved while constructing the rest of the query
from rcsbapi.search import group

query = group(q1 & q2) & q3 & q4

Data API

  • Added ALL_STRUCTURES to const.py/data.__init__.py. This can be passed into the input_ids parameter to make a query for all structures. Currently supports entries and chem_comps input_types
    • Will batch into lists of 5,000 and merge results into one dictionary object
  • Added progress_bar parameter to .exec. If set to True, there will be a progress bar for the executing query.
  • Added batch_size parameter to .exec. Defaults to 5,000.

Misc

  • Updated citation information

@ivana-truong ivana-truong requested a review from piehld March 12, 2025 17:37
Copy link
Collaborator

@piehld piehld left a comment

Choose a reason for hiding this comment

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

Thank you again @ivana-truong! This all looks excellent!

Linking to the original PR to staging here for reference: #50

@piehld piehld changed the title Search API: Retain nesting of queries, Data API: add querying for all structures v1.1.0 Search API: Retain nesting of queries, Data API: add querying for all structures Mar 12, 2025
@piehld piehld merged commit 4d09870 into master Mar 12, 2025
3 of 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