This repository has been archived by the owner on Aug 15, 2023. It is now read-only.

Tallies names should not be overwritten by ODW #115

Open

RemDelaporteMathurin opened this issue Jun 21, 2022 · 1 comment

Member

RemDelaporteMathurin commented Jun 21, 2022

Currently, if users set a tally name on instanciation, it gets overwritten:

my_tally = odw.CellTally(
    "(n,Xa)", target="copper", materials=materials, name="my_custom_name"
)

I think it can be set afterwards, but that's confusing.

Member

shimwell commented Jun 22, 2022

Yep agreed we shouldn't overwrite names, perhaps it could check if there is a name and if not then set it

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.