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.

Sign up for GitHub

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

Jump to bottom

In memory caching #189

Merged
merged 3 commits into from
Nov 5, 2023
Merged

In memory caching #189

merged 3 commits into from
Nov 5, 2023

Conversation

stationeros
Copy link
Contributor

@stationeros stationeros commented Nov 4, 2023 •
edited
Loading

Description

Adding in memory caching in dataset search so that we minimize api calls . This has the following two benefits

  1. Faster page load times. Please refer to the demo below
streamlit_cache.mov
  1. Making lesser downstream calls thereby reducing the chances of getting throttled.

Thanks for contributing this Pull Request. Make sure that you submit this Pull Request against the master branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

  • Relevant Issues : (compulsory)
  • Relevant PRs : (optional)
  • Type of change :
    • New feature
    • Bug fix for existing feature
    • Code quality improvement
    • Addition or Improvement of tests
    • Addition or Improvement of documentation

@stationeros stationeros merged commit 70ae6b0 into master Nov 5, 2023
2 checks passed
@stationeros stationeros mentioned this pull request Nov 5, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@stationeros