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

Upgrade streamlit #1279

Merged
merged 2 commits into from Jul 13, 2021
Merged

Upgrade streamlit #1279

merged 2 commits into from Jul 13, 2021

Conversation

demarant
Copy link
Contributor

Proposed changes:

  • update streamlit and st-annotated-text to latest version
  • make use of latest st-annotated-test version to pass height argument for the iframe annotated text

Why?
stay up-to-date with streamlit features.
Context returned by haystack-api comes in various lenght and the annotated text library was using a fixed height, which sometime did not show entire context/answer and the layout was not optimal. With the latest st-annotated-text we can pass a dynamic height based on the result context lenght.

Breaking changes - None

Issue #1278

Copy link
Member

@tholor tholor left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks for the upgrade :)

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.

None yet

2 participants