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

_reinitialize method in floris_model.py is slower than it needs to be for minor changes #885

Open
achenry opened this issue Apr 20, 2024 · 1 comment
Assignees

Comments

@achenry
Copy link

achenry commented Apr 20, 2024

I'm using Floris in a controller in which the model is updated at each time-step with new wind speeds and directions, and it looks like _reinitialize is a bottleneck, specifically the from_dict call at the end of the method. Would it be possible to set a flag to not update the parameters of floris unless necessary? I only need to update wind speed and direction.

Code profile:
image

and the rest of the code profile...
image

@paulf81
Copy link
Collaborator

paulf81 commented Apr 29, 2024

hi @achenry , let's you and I and @misi9170 find some time to chat about this, maybe Tuesday or Thursday?

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

3 participants