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

demo app update streamlit caching version #374

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

cceyda
Copy link

@cceyda cceyda commented Jun 21, 2023

streamlit st.cache is deprecated in favor of st.cache_data & st.cache_resource.
This PR updates that and removes invalid params.
And fixes the type error with gradcam by passing numpy instead of torch

@salesforce-cla
Copy link

Thanks for the contribution! Before we can merge this, we need @cceyda to sign the Salesforce Inc. Contributor License Agreement.

@cceyda
Copy link
Author

cceyda commented Jun 21, 2023

Already signed the cla getting message You already signed the CLA on 2023-06-21... maybe needs a few minutes to reflect or other type of error

@dxli94
Copy link
Contributor

dxli94 commented Jun 21, 2023

@cceyda thanks for the contribution. I'll take a look. Kindly stay patient before I can merge. There are several others in the pipeline.

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

Successfully merging this pull request may close these issues.

None yet

2 participants