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

seed as arguments of Repeated Models #135

Open
pedro-andrade-inpe opened this issue Feb 26, 2016 · 4 comments
Open

seed as arguments of Repeated Models #135

pedro-andrade-inpe opened this issue Feb 26, 2016 · 4 comments
Assignees
Labels
Milestone

Comments

@pedro-andrade-inpe
Copy link
Member

seed is a reserved parameter for Models. Whenever it is a parameter, it means that this value will be used as a seed. This way, models with repeated simulations must have this parameter explicitly. Think more about this.

@pedro-andrade-inpe
Copy link
Member Author

Models without a parameter seed should not be able to use repeated simulations. Stop with an error in this case. Implement new tests for it.

@agojunior
Copy link
Collaborator

In the current version repeated models cannot have a seed parameter or all results will be the same, why would you want to make the seed parameter compulsory?

@pedro-andrade-inpe
Copy link
Member Author

The Model should have seed, but not MultipleRuns. If one wants to have repeated MultipleRuns using a Model that does not have a seed as argument, it should stop with an error. That's the point.

@agojunior
Copy link
Collaborator

(this is really easy to implement, but I'm not sure i understand what to do. I will ask you abot this in today reunion)

@agojunior agojunior changed the title seed as arguments of Models seed as arguments of Repeated Models Jun 2, 2016
@pedro-andrade-inpe pedro-andrade-inpe modified the milestones: 0.1.2, 0.2 Jun 2, 2016
@pedro-andrade-inpe pedro-andrade-inpe modified the milestones: 0.2, 0.3 Jul 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants