Add named Cells

[MicahGale]

This idea came from @tjlaboss.

The idea is that rather than just referring to a cell by its number, which is mutable, could we refer to it and request it by a string?
Something like cell.name possibly?

This is already somewhat clunkily possible through comments, however it is clunky.

My main concern with this is: how do you preserve those names through saving to and reading from a file? My thought would be to add them as a preceding comment. However, how do you know it's a name and not just a general comment? We might need to introduce a special comment syntax. This is starting to sound like Mo.py and that concerns me.

[MicahGale]

In GitLab by @tjlaboss on May 25, 2022, 13:23

Syntactically, there's no way to tell if a comment is a name. I think that is a one-way street. Nevertheless, the .name attribute will be pretty useful for working in Python and will be helpful for compatibility with things like the OpenMC Python API.

[MicahGale]

So you think it should only be user set and valid while in memory? Should it be added to the comments?

What's the OpenMC way of doing this?

[MicahGale]

In GitLab by @tjlaboss on May 25, 2022, 14:30

OpenMC objects have a .name attribute which does not need to be set, does not need to be unique, but is carried with the object and written to XML (except universes).

IMO if an object doesn't have a natural name, like the SCn or FCn cards, then the name should be written as a comment. No preference as to whether it's prepended as a comment card or added as an inline $ comment.

[MicahGale]

In GitLab by @tjlaboss on May 25, 2022, 15:00

Another thought: if a name is written as a comment, MCNPy could potentially check if it just wrote that string, to avoid doubling and tripling up on identical comments through multiple passes.

[MicahGale]

Yeah so $ comments aren't support right now. You're right some thought should be given to avoid repeated entries on the comment. I'm thinking if a name is set add it to the comments unless a similar comment already exists. Once the file gets read back in the name isn't parsed.

[MicahGale]

Are you going to do this or should I boot it to the next release?