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

Integer number of inverters per transformer #45

Closed
blthayer opened this issue Feb 19, 2022 · 2 comments
Closed

Integer number of inverters per transformer #45

blthayer opened this issue Feb 19, 2022 · 2 comments

Comments

@blthayer
Copy link

On a whim, I changed my configured number of transformers (num_transformers) from 1 to 3. When attempting to run the case, I get the following error: Error loading and verifying case: There must be an integer number of inverters per transformer!

This seems like a bit of a nuisance error IMO. It seems well within the realm of possibility (though maybe not typical) that a different number of inverters might be "under/downstream" of the different transformers.

Perhaps pvrpm could round and emit a warning, or model transformers having different numbers of inverters beneath them.

@brandons209
Copy link
Contributor

I think we could round, but from the way PVRPM is built and my understanding an even number of inverters to transformers is needed. Rounding and emitting a warning that it was rounded can work although I am of the belief that the user should have a proper configuration before running rather then having a possibly incorrect configuration and assuming by rounding.

@blthayer
Copy link
Author

blthayer commented Mar 6, 2022

Fair enough!

@blthayer blthayer closed this as completed Mar 6, 2022
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

2 participants