Skip to content

feature/remove_mapper_valued#381

Merged
Jammy2211 merged 1 commit intomainfrom
feature/remove_mapper_valued
Jan 28, 2026
Merged

feature/remove_mapper_valued#381
Jammy2211 merged 1 commit intomainfrom
feature/remove_mapper_valued

Conversation

@Jammy2211
Copy link
Owner

This pull request introduces several improvements and cleanups across the documentation and codebase. The most significant changes include updating community and contribution guidelines, simplifying user support instructions, modernizing plotting and inversion code by removing deprecated interfaces, and clarifying installation instructions.

Source Science Examples:

Specific examples showing how to perform the source science calculations which previously used MapperValued, but now just use scipy directly, are available on the autolens workspace:

https://github.com/Jammy2211/autolens_workspace/blob/main/notebooks/imaging/source_science.ipynb
https://github.com/Jammy2211/autolens_workspace/blob/main/notebooks/imaging/features/pixelization/source_science.ipynb

Documentation and Community Updates:

  • Added a new "Community Guidelines" section to CONTRIBUTING.md to set expectations for contributor behavior and reference the Code of Conduct.
  • Updated the README.rst to:
    • Replace the Binder notebook link with a Google Colab link for easier access.
    • Clarify and restructure the support section, emphasizing Slack for community support and GitHub Issues for bug reports and feature requests.
    • Add new sections on citations and contributing, with direct links to relevant pages.
  • Improved installation instructions in the documentation to specify pip install autolens.

Codebase Simplification and Cleanup:

  • Removed all usage and imports of the deprecated MapperValued class from the codebase, replacing related logic with new inversion methods (e.g., max_pixel_centre, max_pixel_list_from). This affects both the main code (autolens/analysis/result.py, autolens/__init__.py) and plotting utilities for imaging and interferometer fits. [1] [2] [3] [4] [5]
  • Removed the interpolate_to_uniform plotting parameter and all related logic from imaging and interferometer plotters, streamlining the plotting interface. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]

These changes improve code maintainability, modernize documentation, and provide clearer guidance for both users and contributors.

@Jammy2211 Jammy2211 merged commit d514d4b into main Jan 28, 2026
4 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant