Replies: 4 comments 4 replies
-
Hi Dominic, Well spotted! The start time doesn't have to 0.0, but the BMI documentation for |
Beta Was this translation helpful? Give feedback.
-
@dorchard To solve the problem of a reference date we had discussed a couple of options. One was to add a new BM function (I liked I don't think we ever had a vote but the consensus seemed to be to go with the second option. Either way, I think the bmi-tester should be updated to allow for non-zero start times. |
Beta Was this translation helpful? Give feedback.
-
Thanks @mcflugen - I could make a PR re non-zero start time (it would basically just remove one line). I do also wonder whether negative time steps are allowed? i.e., should it be possible to have models that start in the future and work backwards? At the moment it seems that this would probably be rejected by the bmi-tester due to the check that the 'end' is later than the 'start': https://github.com/csdms/bmi-tester/blob/master/src/bmi_tester/_tests/stage_1/time_test.py#L60-L67 |
Beta Was this translation helpful? Give feedback.
-
Following this interesting discussion I made a couple of PRs to the bmi-tester to generalise the current situtation:
|
Beta Was this translation helpful? Give feedback.
-
I've been looking into the expected properties of models implementing the BMI interface and looking at what is described in the bmi-tester. I noticed that the tester expects the start time to be 0.0 (https://github.com/csdms/bmi-tester/blob/c4f5c994a13f6fd010ef5ba25fdcaffa893222be/src/bmi_tester/_tests/stage_1/time_test.py#L16-L21) I wondered if that's universal and if so, why is a start time needed? I can imagine that maybe some models might want to start somewhere in real history, perhaps at the beginning of a sensor record or at the start of the industrial revolution or 1970 or something like this. In which case the bmi-tester would not like it! I was wondering if anyone had any more info, thoughts, or knowledge they could share with me (a newbie) (maybe @mdpiper or @mcflugen?)
Beta Was this translation helpful? Give feedback.
All reactions