Skip to content
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

[Improving Defaults] times argument default should extract information on timeframe from modules, not start at 0 #274

Open
tati-micheletti opened this issue Apr 15, 2024 · 0 comments

Comments

@tati-micheletti
Copy link
Contributor

tati-micheletti commented Apr 15, 2024

Currently, the default behavior of SpaDES is to set the default start$time to 0 and end$time to 1. I would suggest we recover the times proposed in the modules' metadata, such as:

Module A: timeframe = as.POSIXlt(c(2010, 2020))
Module B: timeframe = as.POSIXlt(c(2015, 2050))

out$times
$start
[1] 2010
$end
[2] 2050

This way times is not starting at a non-sensical default and the user doesn't have to worry about scheduling it wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant