improve docstrings for autoarray/inversion package and update fit log…#229
Merged
improve docstrings for autoarray/inversion package and update fit log…#229
Conversation
…_evidence inversion package: - abstract.py: fix wrong Parameters section in AbstractInversion.__init__ (was listing data/noise_map; now correctly lists dataset/linear_obj_list/settings/xp); remove spurious dict_values param from has() method - imaging/abstract.py: fix same wrong Parameters section (data/noise_map → dataset/...) - interferometer/abstract.py: fix wrong Parameters section (noise_map/transformer → dataset/...); add docstring to fast_chi_squared property - dataset_interface.py: remove spurious over_sampler/border_relocator params; fix "imaigng" typo - inversion_interferometer_util.py: fix wrong param name image → visibilities - inversion_util.py: add missing params add_to_curvature_diag/no_regularization_index_list/ settings/xp to curvature_matrix_via_mapping_matrix_from; fix "transfomed" typo - linear_obj.py: fix "piels" → "pixels" typo; fix "regularizaiton" typo - mappers/abstract.py: fix wrong Parameters in Mapper.__init__ (was listing source_plane_data_grid/source_plane_mesh_grid/adapt_data/mesh_weight_map; now lists actual params interpolator/regularization/settings/image_plane_mesh_grid/xp); fix "unfamliar"/"pxelization"/"annd" typos - imaging/inversion_imaging_util.py: fix "PSf" → "PSF" typo fit package (retrospective improvements using inversion context): - fit_util.py: expand log_evidence_from docstring with full mathematical formula (χ² + s^T H s + ln det(F+H) - ln det(H) + Σ ln 2πσ²) and paper references - fit_dataset.py: expand log_evidence property with same full formula - fit_interferometer.py: expand log_evidence with full formula; mention fast_chi_squared 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.
…_evidence
inversion package:
fit package (retrospective improvements using inversion context):