hayabhay / frogbase

Transform audio-visual content into navigable knowledge.
https://frogbase.dev
MIT License
781 stars 95 forks source link

Disabled Streamlit telemetry & removed segment expander #24

Closed hayabhay closed 1 year ago

hayabhay commented 1 year ago

Two small changes

  1. Disabled Streamlit's default telemetry in .streamlit/config.toml
  2. Removed st.expander for detail view
    • Streamlit's column creation is very slow and when the number of segments range in the ~100s, creating this has a pretty big compute cost
    • Streamlit's expander triggers the above creation & deletion every time the html is collapsed/expanded instead of simply hiding visibility. This entails the repetition of this expensive column creation. So for now, at the very least, expander is removed so the play button & segment text rendering is a one time operation per detail view.