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

Photovoltaics Changes #70

Merged
merged 4 commits into from
Jun 9, 2016
Merged

Photovoltaics Changes #70

merged 4 commits into from
Jun 9, 2016

Conversation

nmerket
Copy link
Contributor

@nmerket nmerket commented Dec 1, 2015

  • Need definition for AnnualOutput.
  • Need definition for LevelizedCostOfElectricity.
  • Need another set of eyes on PV ownership enumerations.

baseelements_pvsystem

@RebHudson
Fixes #53
Fixed #54

@@ -1732,6 +1732,9 @@
<xs:restriction base="xs:string">
<xs:enumeration value="leased"/>
<xs:enumeration value="owned"/>
<xs:enumeration value="power purchase agreement"/>
<xs:enumeration value="utility owned"/>
<xs:enumeration value="other"/>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Gavinhastings @RebHudson Do these enumerations for PVSystem/Ownership looks reasonable?

Choose a reason for hiding this comment

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

Those enumerations work for me

@nmerket
Copy link
Contributor Author

nmerket commented Dec 1, 2015

From @RebHudson


Annual Output could be “actual” or “projected”. These would need to be enumerations for the value. Actual would be a measured value of the full year’s energy output in kWh/year. It would be great to have a year associated with that value.
The projected value requires a calculation/estimation. Does this need to be a set calculation? I could see that different solar installers might use different formulas.

There is a simplified version of an output calculator:
http://pvwatts.nrel.gov/

For levelized cost of electricity, it also involves a calculation that could be done in different ways. A good definition from SAM (NREL created a SAM model that has a method for calculating LCOE https://www.nrel.gov/analysis/sam/help/html-php/index.html?mtf_lcoe.htm) is: The LCOE is the total cost of installing and operating a project expressed in dollars per kilowatt-hour of electricity generated by the system over its life.
Again – does this need a specific formula associated with it? Or can it be up to the contractor’s discretion?

http://www.nrel.gov/analysis/tech_lcoe.html seems to have a simpler version.

@nmerket
Copy link
Contributor Author

nmerket commented Dec 3, 2015

I added the following definitions

AnnualOutput

[kWh] Projected Annual Output for a typical meterological year as determined by PVWatts.

I decided to go with the projected because that's what more people are likely to have. It's fairly simple to calculate too.

Levelized Cost of Electricity

[$] The LCOE is the total cost of installing and operating a project expressed in dollars per kilowatt-hour of electricity generated by the system over its life. Can be calculated with System Advisor Model or through a simplified calculation at http://www.nrel.gov/analysis/tech_lcoe.html.

<xs:element minOccurs="0" name="AnnualOutput"
type="RatedAnnualkWh">
<xs:annotation>
<xs:documentation>[kWh] Projected Annual Output for a typical meterological year as determined by PVWatts. </xs:documentation>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

add "or similar"

@nmerket
Copy link
Contributor Author

nmerket commented Feb 4, 2016

Review: @brandongallagher @GamalielL

@GamalielL
Copy link

Looks good to me. I've asked Andy to review as well.

@brandongallagher
Copy link

works for me. i'm looking forward to the AnnualOutput field. that will help us.

@nmerket nmerket merged commit a4ae899 into master Jun 9, 2016
@nmerket nmerket deleted the photovoltaics branch June 9, 2016 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants