[pull] master from polakowo:master#56
Merged
Merged
Conversation
- Define FigureProtocol with Capability flags for backend-neutral plotting - Add plot_line, plot_markers, plot_area, plot_ohlc, plot_histogram, plot_bars, plot_hline, and plot_zone methods to FigureMixin - Implement backend registry (register_backend, get_backend, list_backends) with Plotly registered as the default built-in backend - Add create_figure() factory that routes through the registry and respects settings['plotting']['default_backend'] - Add resolve_backend/resolve_backend_for_fig helpers for downstream migration of existing plot methods - Add assert_plotly_only_kwargs/assert_plotly_only_method guards for graceful non-Plotly backend error reporting - Expose new API surface on vbt.plotting.* and top-level vbt.* - Add default_backend='plotly' to plotting settings - Add comprehensive test suites for protocol conformance and backend registry
- Add hover_text sequence length validation in plot_markers to raise ValueError on mismatched lengths - Add plot_zone tests: minimal, color/opacity, None-param omission, subplot domain resolution - Add subplot column routing tests (1x2 and 2x2 layouts) to catch regressions that ignore the col dimension - Add use_widgets_setting fixture to parametrize subplot tests over both Figure and FigureWidget - Add create_figure tests for rows-only, cols-only, and figure= kwarg routing through make_subplots - Add backend registry tests for kwarg forwarding and unregistered default backend error handling - Add to_html protocol conformance test - Add None-param omission tests for plot_histogram and plot_area - Fix test_unknown_backend_raises_keyerror to assert correct key name - Compare add_hline compat test against go.Figure reference instead of hardcoded Plotly internals
- Rename `default_backend` setting to `default_renderer` and update all references - Rename `register_backend`, `get_backend`, `list_backends` to `register_renderer`, `get_renderer`, `list_renderers` - Rename `resolve_backend`, `resolve_backend_for_fig` helpers and `backend_name` class attribute to `renderer_name` - Rename `_BACKEND_REGISTRY` to `_RENDERER_REGISTRY` and `BackendFactory` to `RendererFactory` - Rename `tests/test_plotting_backend.py` to `tests/test_plotting_renderer.py` - Clarify in `create_figure` docstring that vectorbt's `renderer=` selects the plotting library, distinct from Plotly's own `fig.show(renderer=...)` output-format kwarg
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )