Skip to content

Resolve #140 snlinverter night tare power issue #141

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

Merged
merged 5 commits into from
Apr 12, 2016

Conversation

ejmiller2
Copy link
Contributor

Fixing #140 by changing the start power (Pso) comparison to use the DC input power (p_dc) instead of the calculated AC power (ac_power).

Fixing pvlib#140 by changing the start power (Pso) comparison to use the DC input power (p_dc) instead of the calculated AC power (ac_power).
@wholmgren
Copy link
Member

Great. One easy request and one slightly harder request:

Easy: Add a brief note to the "Bug fixes" section of the pvlib/docs/sphinx/source/whatsnew/v0.3.1.txt file and add your name or username to the "Contributors" list.

Slightly harder: Add a new test to pvlib/test/test_pvsystem.py that requires this new behavior in order for it to pass. You basically want a copy of test_snlinverter but with one of the inverters and the input parameters that were giving you the wrong results.

Thanks!

@wholmgren wholmgren added the bug label Mar 23, 2016
@wholmgren wholmgren added this to the 0.3.1 milestone Mar 23, 2016
@ejmiller2
Copy link
Contributor Author

I'll give it a shot a bit later today.

Added note and contribution for fix to issue pvlib#140.
@ejmiller2
Copy link
Contributor Author

All done (I think).

@@ -15,8 +15,10 @@ Enhancements
Bug fixes
~~~~~~~~~

* Fixes snlinverter Pnt problem (:issue:`140`)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know I asked for brief, but maybe something more like "Fixes night tare issue in snlinverter. The night tare was erroneously activated when the AC output power, rather than the DC input power, was less than the inverter's self-consumption" or similar. I had to look up Pnt, so I'm sure others would need to as well.

Improved the night tare issue description.
Added carriage returns for improved readability.
@wholmgren
Copy link
Member

@ejmiller2 thank you for fixing this.

@wholmgren wholmgren merged commit 566a1e1 into pvlib:master Apr 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants