Recommendations for setting Reactor.p.time for calendar time?

[drewj-usnctech]

I can't seem to find anything inside the ARMI framework that sets the calendar time for a simulation. There's this Reactor.p.time parameter

armi/armi/reactor/reactorParameters.py

Lines 78 to 83 in 6603c1d

	pb.defParam(
	"time",
	units=units.YEARS,
	description="Time of reactor life from BOL to current time node",
	categories=["depletion"],
	)

with a depletion description

but it doesn't seem to be set inside ARMI. It looks like the cycle and node integers are serialized, but then something must set Reactor.p.time in order for it to be useful in e.g., report generation

armi/armi/physics/neutronics/reports.py

Line 98 in 6603c1d

report[NEUTRONICS_SECTION][KEFF_PLOT].add(labels[0], r.p.time, r.core.p.keff, None)

And it seems to me that this is something that the main Operator or a shipped interface could handle relatively easily? The Operator handles marching from (cycle, node) to (cycle, node), so it could conceivably also set Reactor.p.time

We have a kind of complex routine that re-constructs the time from the (cycle, node) value at interactEveryNode and it's got me thinking there's got to be a better way.

So the question(s) are

1. What is the recommended way to set Reactor.p.time in a depletion rune?
2. Is this something that ARMI could standardize such that the field is set as the simulation marches in time, prior to calls to Interface.interact*?

[keckler]

@drewj-usnctech Your analysis is spot-on. Check out this issue, it has some context that probably answers your question: #1100

Long story short, right now this time parameter is advanced by our internal depletion plugin. I don't think there is any particularly good reason for that.

[drewj-usnctech]

Thanks @keckler. Question isn't really answered, but I've got more threads to pull now

[john-science]

@keckler At some point, I would like to pick your brain about this.