-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TimeManager interface module #209
Conversation
82a0e33
to
c8d4a95
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job! Feel free to merge after addressing the minor comments.
src/TimeManager.jl
Outdated
- `cs`: [CoupledSimulation] or [CouplerSimulation] containing info about the simulation | ||
- `t`: [Integer] number of seconds since simulation began | ||
""" | ||
current_date(cs, t) = cs.dates.date0[1] + Dates.Second(t) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please specify the argument types here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a better way I should specify the types than [CoupledSimulation] or [CouplerSimulation]
and [Integer]
? I wasn't sure how to be more specific since there are multiple valid types.
0217214
to
b1660d7
Compare
b1660d7
to
7949bd4
Compare
bors r+ |
Purpose
Move functions from coupler_utils/calendar_timer.jl to src/CallbackManager.jl module, and ensure that they are thoroughly commented and tested.
Part of SDI #134
To-do
PHASE 1 - moving and documentation
src/
ClimaCoupler.jl
docs/src
,docs/make.jl
PHASE 2 - testing
test/
, add totest/runtests.jl
PHASE 3 - cleanup