Skip to content

MAINT: Simplify optimizers#74

Merged
derb12 merged 6 commits intodevelopmentfrom
simplify_optimizers
Apr 11, 2026
Merged

MAINT: Simplify optimizers#74
derb12 merged 6 commits intodevelopmentfrom
simplify_optimizers

Conversation

@derb12
Copy link
Copy Markdown
Owner

@derb12 derb12 commented Apr 11, 2026

Description

This PR simplifies optimizer-type methods by filtering based on the required parameters rather than on modules. The benefit is more granular filtering of methods, better error messages when methods are not supported by the optimizing method, and now optimizers can also be implemented as ND mixins.

The granular filtering allows optimize_extended_range to use beads and jbcd as methods, addressing issue #64.

Type of Pull Request

  • Bug Fix
  • New Feature
  • Miscellaneous Changes (refactor, code improvements, etc.)
  • Documentation or Example Programs

Pull Request Checklist

  • New code and/or documentation is valid for use with the BSD 3-clause license.
  • New code is fully documented with docstrings that follow Numpy style,
    if applicable.
  • New code follows PEP 8 standards as closely as possible, if applicable.
  • Added/updated tests and ensured they pass locally, if applicable.
  • Verified that documentation builds locally, if applicable.

@derb12 derb12 merged commit 6cbfdf0 into development Apr 11, 2026
11 checks passed
@derb12 derb12 deleted the simplify_optimizers branch April 11, 2026 16:18
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