Open
Conversation
figrecipe.colors hid BGR/str2* converters behind _prefixed names. Import from private names for backward compat, expose only public API in __all__. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove all re-exports of hidden BGR/str2* functions from plt.color - Update internal callers: get_colors_from_conf_matap → get_colors_from_cmap - Update internal callers: str2hex → to_hex Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…sher Aligns workflow filename with PyPI OIDC trusted publisher config. Follows figrecipe pattern: trigger on release published, two-job build+publish. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace all internal figrecipe._* imports with public API equivalents: - figrecipe._utils._units → figrecipe.utils - figrecipe._utils._crop → figrecipe.crop - figrecipe._api._notebook → figrecipe.utils - figrecipe._api._seaborn_proxy → figrecipe.utils - figrecipe._api._style_manager → figrecipe.utils - figrecipe._composition → figrecipe - figrecipe._graph._presets → figrecipe - figrecipe._wrappers → figrecipe.utils - figrecipe._diagram (single import) → figrecipe Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…igration Remove ~190 files of the old AxisWrapper/FigWrapper/SubplotsWrapper infrastructure. figrecipe RecordingAxes + stx_* methods (added in figrecipe commit dac5b47) replace all functionality. Deleted: - _subplots/ (AxisWrapper, FigWrapper, SubplotsWrapper, CSV formatters) - ax/_style/ and ax/_plot/ (30+ style and plot functions) - styles/ dead code - All associated tests Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…gration Replace export_as_csv() AxisWrapper calls with figrecipe RecordingFigure proxy: - _figure_utils.py: New _RecordingFigureDataProxy reads from figrecipe recorder - _image_csv.py: Remove dead SigmaPlot export block - _plot_bundle.py: Simplify (CSV from explicit data only) - _plot_scitex.py: Remove AxisWrapper fallback, keep matplotlib line extractor - _save.py: Remove AxisWrapper CSV block - bundle/_mpl_helpers.py: _get_scitex_axes() → stub returning None CSV data still exported for figrecipe RecordingFigure objects via proxy. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…etion The RecordingFigureDataProxy now handles both states: - Pre-savefig: reads _array from in-memory recorder args - Post-savefig: reads from figrecipe-saved <stem>_data/*.csv files Pass spath through _export_csv_data → get_figure_with_data so the proxy can locate the data directory. Also removed dead _export_sigmaplot_csv(). Result: stx.io.save(fr_fig, "plot.png") now creates plot.csv alongside the figrecipe plot.yaml and plot_data/ directory. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…hase 4 - test_imports.py: Replace 'ax' submodule assertion with figrecipe API checks (ax submodule deleted in AxisWrapper migration) - diagram/__init__.py: Fix broken figrecipe._diagram._compile import → figrecipe._diagram._graphviz + figrecipe._diagram._mermaid Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Three files had try-except blocks importing format_record from the deleted _subplots._export_as_csv module. They already had correct fallback paths, so remove the dead try blocks and go directly to the pattern-based column name generation. - plt/utils/_metadata/_csv.py - plt/utils/metadata/_data_linkage.py - plt/utils/_collect_figure_metadata.py Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
Summary
Test plan
🤖 Generated with Claude Code