Skip to content

demo app update streamlit caching version#374

Open
cceyda wants to merge 7 commits into
salesforce:mainfrom
cceyda:streamlit_update
Open

demo app update streamlit caching version#374
cceyda wants to merge 7 commits into
salesforce:mainfrom
cceyda:streamlit_update

Conversation

@cceyda

@cceyda cceyda commented Jun 21, 2023

Copy link
Copy Markdown

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
Copy Markdown

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

@cceyda

cceyda commented Jun 21, 2023

Copy link
Copy Markdown
Author

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

dxli94 commented Jun 21, 2023

Copy link
Copy Markdown
Contributor

@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.

2 participants