Consider surface constants and coordinates to be a Sequence, not an Iterable

[tjlaboss]

Pull Request Checklist for MontePy

Description

Previously, the type checker verified that surface coordinates and coefficients were instances of Iterable. This was overly permissive, as it would allow objects like sets and generators to be used. Now, they only allowed to be instances of Sequence, which has a definite order and length.

Fixes #914

---

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

2. Were any large language models (LLM or "AI") used in to generate any of this code?

• No

---

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--926.org.readthedocs.build/en/926/

[MicahGale]

Looks pretty good. Changelog isn't necessary. Probably should go through the entire code base too.

[tjlaboss]

Not sure why black is failing:

$ python -m black tests/
All done! ✨ 🍰 ✨
33 files left unchanged.
$ python -m black --version
python -m black, 25.1.0 (compiled: yes)
Python (CPython) 3.13.2

---

Probably should go through the entire code base too.

Every other use of Iterable was appropriate. Only these needed to be Sequence specifically.

[tjlaboss]

Perhaps it is time to revise the black behavior in CI; black 25.1 is now too far out of date to pass the test.

[MicahGale]

Perhaps it is time to revise the black behavior in CI; black 25.1 is now too far out of date to pass the test.

Yeah let's make an issue to make the CI job just commit the changes if needed.

[MicahGale]

LGTM.

[MicahGale]

FYI #914 won't auto-close. You should keep an eye on this @tjlaboss