Update/dash avs explorer - #806
Open
budescode wants to merge 2 commits into
Open
Conversation
- Declare the local dash_avs_ui-0.0.1.tar.gz as a path source, since the component was never published to PyPI - Regenerate requirements.txt from uv.lock so pip users still work - Pin dash 4.4.1, dash-bootstrap-components 2.0.4 and gunicorn 26.2.0, none of which had a version before - Require Python 3.12 or newer; raise runtime.txt from 3.7.6 to 3.12.11
- Replace dash_core_components and dash_html_components imports with from dash import dcc, html - Replace app.run_server with app.run - Replace dbc.FormGroup, removed in dash-bootstrap-components 1.0, with html.Div - Point the KITTI dataset at uber/xviz-data; the plotly/xviz-data-kitti repository no longer exists - Disable the KITTI scenes whose data is no longer hosted and default to drive 0005, the one that is - Add uv install and run instructions to the README and update its Python version
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Issue for app: none (dependency update, no linked issue)
App pull request
About
Updates dash-avs-explorer to current package versions, makes it installable
with uv or pip from a fresh clone, and restores the KITTI dataset, whose
data source is not available.
The dash_avs_ui component itself is unchanged. Its 2020 build mounts
correctly under Dash 4.
Verified in a headless browser on Python 3.12: the page loads with no Dash
error and no failed requests, the unavailable scenes show as disabled
with drive 0005 selected, and the 3D viewer mounts and fetches its
frames.
Workflow
elements necessary for successful deployment are in place.
existing gallery app, I've summarized the changes requested in the
appropriate Streambed issue and confirm that they have been applied.
the GitHub repository for the source code in the portal description.
DashR gallery, the app in this PR mimics, as closely as possible,
the style and functionality of the existing app.
assets/folder.The pre-review review
I have addressed all of the following questions:
any dead and/or irrelevant code.)
readable and, where it isn't, it has been commented appropriately.)
lessen the volume of code that needs to be maintained.)
Post PR (at merge time)
#patch- An app has been updated or fixed#minor- A new app has been added, or an app has been significantly reworked#major- Breaking changes, make sure to discuss with dash-core before using this tag