Skip to content

Add get_by_comment method to NumberedObjectCollection and correspondi… - #1006

Draft
digvijay-y wants to merge 1 commit into
idaholab:developfrom
digvijay-y:feat/search_by_comment
Draft

Add get_by_comment method to NumberedObjectCollection and correspondi…#1006
digvijay-y wants to merge 1 commit into
idaholab:developfrom
digvijay-y:feat/search_by_comment

Conversation

@digvijay-y

@digvijay-y digvijay-y commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

…ng tests

  • Implemented a method to yield objects matching a comment string or regex pattern.
  • Added unit tests to validate functionality and error handling for the new method.

Pull Request Checklist for MontePy

Description

Please provide a summary of the change, referencing the issue it fixes, if applicable. Include relevant context and motivation.

Fixes # (issue number)


General Checklist

  • I have performed a self-review of my own code.
  • The code follows the standards outlined in the development documentation.
  • I have formatted my code with black version 25 or 26.
  • I have added tests that prove my fix is effective or that my feature works (if applicable).

LLM Disclosure

  1. Are you?

    • A human user
    • A large language model (LLM), including ones acting on behalf of a human
  2. Were any large language models (LLM or "AI") used in to generate any of this code?

  • Yes
    • Model(s) used:
  • No

Documentation Checklist

  • I have documented all added classes and methods.
  • I have added type hints to all functions as needed.
  • I have marked all changes with the .. versionchanged:: or .. versionadded:: directives.

Infrastructure Changes

  • For infrastructure updates, I have updated the developer's guide.

Significant features or Behavior changes

  • For significant new features, I have added a section to the getting started guide.

First-Time Contributor Checklist

  • If this is your first contribution, add yourself to pyproject.toml if you wish to do so.

Additional Notes for Reviewers

Ensure that:

  • This PR fully addresses and resolves the referenced issue(s).
  • The submitted code is consistent with the merge checklist outlined here.
  • The PR covers all relevant aspects according to the development guidelines.
  • 100% coverage of the patch is achieved, or justification for a variance is given.

📚 Documentation preview 📚: https://montepy--1006.org.readthedocs.build/en/1006/

…ng tests

- Implemented a method to yield objects matching a comment string or regex pattern.
- Added unit tests to validate functionality and error handling for the new method.
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