refactor: archive quantity package to autolens_workspace_developer/legacy#528
Merged
Conversation
…gacy Removed autolens/quantity/, test_autolens/quantity/, the FitQuantity / AnalysisQuantity re-exports (including the duplicate DatasetQuantity re-export from autogalaxy), the subplot_fit_quantity plot re-export, config and docs stanzas, and the conftest fixtures that exercised the deleted tests. Frozen copy lives at autolens_workspace_developer/legacy/quantity/. Co-Authored-By: Claude Opus 4.7 (1M context) <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
quantitypackage —FitQuantity,AnalysisQuantity,VisualizerQuantity,ResultQuantity— that has no real users and was blocking JAX porting work.autolens_workspace_developer/legacy/quantity/(separate PR on that repo).Changes
Removed:
autolens/quantity/— source tree (6 files)test_autolens/quantity/— test tree (4 files)from .quantity.fit_quantity import FitQuantity+... AnalysisQuantityfromautolens/__init__.pyfrom autogalaxy.quantity.dataset_quantity import DatasetQuantity(was imported twice at lines 74 and 95)subplot_fit_quantityfrom thefrom autogalaxy.plot import (...)block inautolens/plot/__init__.pydataset_quantity_7x7_array_2d+fit_quantity_7x7_array_2d@pytest.fixtureentries fromtest_autolens/conftest.pyfit_quantity:stanzas fromautolens/config/visualize/plots.yamlandtest_autolens/config/visualize.yamlAnalysisQuantityautosummary entry fromdocs/api/modeling.rstFitQuantityautosummary fromdocs/api/fitting.rstAnalysisQuantitymention fromautolens/analysis/analysis/lens.pymodule docstringquantity/directory row fromCLAUDE.mdTest plan
pytest test_autolens/ -x -q— 294 passed in 38sFitQuantity|AnalysisQuantity|DatasetQuantity|subplot_fit_quantity|fit_quantity:|quantity_7x7python -c "import autolens as al; assert not hasattr(al, 'FitQuantity')"passes for all three classes +subplot_fit_quantityShip order
Pairs with the PyAutoGalaxy PR (also
feature/archive-quantity-package). PyAutoLens has no remaining cross-references to removed autogalaxy symbols, so the two can review in parallel — but both libraries should merge before the workspace PRs.🤖 Generated with Claude Code