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

CUDASimulation::simulate(RunPlan plan) #678

Closed
ptheywood opened this issue Sep 2, 2021 · 0 comments · Fixed by #753
Closed

CUDASimulation::simulate(RunPlan plan) #678

ptheywood opened this issue Sep 2, 2021 · 0 comments · Fixed by #753

Comments

@ptheywood
Copy link
Member

ptheywood commented Sep 2, 2021

It might be nice to provide a mechanism to run simulations using individual RunPlan objects?

There is overlap between a RunPlan and various parts of a simulations's Config object however, which would need to be clear.

I.e. how many steps would be used below:

flamegpu::CUDASimulation simulation(model);
simualtion.Config().steps=10;
RunPlan plan1(model);
plan1.setSteps(20);
RunPlan plan2(model);
simulation.simulate(); // This would do 10 steps
simulation.simulate(plan1); // This would do 20 steps.
simulation.simulate(plan2); // this would do 1 step
Robadob added a commit that referenced this issue Dec 14, 2021
CUDASimulation can be simulated with a RunPlan

Closes #678
mondus pushed a commit that referenced this issue Dec 14, 2021
CUDASimulation can be simulated with a RunPlan

Closes #678
@ptheywood ptheywood added this to the v2.0.0-alpha.3 milestone Dec 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant