Spatial view rewrite#1290
Open
adkinsrs wants to merge 19 commits into
Open
Conversation
…ver tools and dynamic layout handling
- Updated create_spatial_plot to improve NaN handling and hover tooltips. - Modified create_umap_plot to include dynamic category display and adjusted plot settings. - Simplified panel_app_expanded.py by removing unnecessary template wrapping. - Enhanced BaseSpatialViewer with new properties and improved opacity handling. - Added zoom panel update functionality in ExpandedSpatialViewer.
- Added 'mode' parameter to create_spatial_plot for different tool configurations. - Updated default tools for spatial plots based on mode. - Removed redundant code in panel_app and panel_app_expanded. - Implemented a loading overlay in tilegrid.js for improved user experience during data loading.
…olbar adjustments; improve CSS for overflow handling in spatial plots
…ected layer; adjust unselected_alpha and color for better visibility in spatial viewer
…tions for disabling PrivateTmp
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
1 participant
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.
This pull request introduces several improvements to the spatial viewer backend and frontend, focusing on modernizing dependencies, improving the Panel app export and embedding workflow, and enhancing layout and styling for spatial visualizations. The changes streamline the process of generating and embedding spatial visualizations, improve maintainability, and address some UI/UX issues related to layout and resource loading.
Backend and API improvements:
SpatialPanelAPI resource to return a script tag for embedding the Panel viewer app usingbokeh.server_document, with support for passing configuration arguments and toggling between standard and zoomed-in views. This replaces the previous filename-based workflow and enables more flexible embedding. [1] [2] [3] [4]get_viewer_scriptfor generating the embedding script, including environment-based domain URL selection and argument passing.Panel app and export enhancements:
ExpandedSpatialViewer, improving export reliability and presentation by stripping interactive controls and embedding all resources into a single HTML file. [1] [2]BOKEH_RESOURCES=cdnenvironment variable in Docker, systemd, and compose templates to ensure Bokeh resources are loaded from CDN, reducing resource loading issues in offline exports. [1] [2]Dependency and environment updates:
panelto 1.9.3, addedhvplotandspatialpandas) in bothrequirements.txtand developer documentation to support new features and improve compatibility. [1] [2] [3] [4]PYTHONPATHexplicitly for the spatial service. [1] [2]Frontend and styling improvements:
Codebase cleanup:
arg_mappingand manual settings construction in the download plugin, to reduce complexity and improve maintainability.These changes collectively modernize the spatial viewer infrastructure, improve user experience for exporting and embedding visualizations, and ensure the system is ready for future enhancements.