You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Capped plotly version to <=5.24.1 in pyproject.toml
Plotly Express introduced binary data encoding ("bdata") in plotly 6.4.0+ when serializing figures. The frontend cannot decode this binary format, causing figures created with px.histogram(), px.box(), and px.density_contour() to not display in the UI, even though they were correctly generated and saved to S3.
How to test
Run rag_documentation_demo.ipynb shows figures from embeddings and rag tests correctly.
What needs special review?
Dependencies, breaking changes, and deployment notes
Release notes
Checklist
What and why
Screenshots or videos (Frontend)
How to test
What needs special review
Dependencies, breaking changes, and deployment notes
This PR makes several dependency and version updates in the lock file and project metadata. The key changes are:
The lock file (poetry.lock) has been regenerated using Poetry 2.1.3. Several new wheel entries have been added for the Brotli package, targeting additional architectures and musllinux variants, which should improve support on various platforms.
The narwhals package entry has been removed from the lock file along with its extras. This change removes the dependency on a lightweight compatibility layer between dataframe libraries.
The Plotly package has been downgraded from version 6.3.0 to 5.24.1. The change includes updates to the package description and associated file hashes. In addition, the dependency constraint in pyproject.toml has been adjusted to "plotly (>=5.0.0,<6.0.0)" to ensure compatibility.
Finally, the project version has been bumped from 2.10.2 to 2.10.3 in pyproject.toml and in the version.py file.
These changes primarily affect dependency management and package distribution, and they do not introduce any new application logic.
Test Suggestions
Run the full test suite to ensure that changes in dependencies do not break existing functionality.
Verify that the application runs correctly on all supported platforms, especially where new Brotli wheels have been added.
Test installation from the generated lock file in a clean environment to ensure that dependency resolution works as expected.
Check for any issues due to the removal of the narwhals package if it was implicitly expected elsewhere in the project.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bugSomething isn't workinginternalNot to be externalized in the release notes
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request Description
What and why?
<=5.24.1inpyproject.tomlplotly 6.4.0+when serializing figures. The frontend cannot decode this binary format, causing figures created withpx.histogram(),px.box(), andpx.density_contour()to not display in the UI, even though they were correctly generated and saved to S3.How to test
Run
rag_documentation_demo.ipynbshows figures from embeddings and rag tests correctly.What needs special review?
Dependencies, breaking changes, and deployment notes
Release notes
Checklist