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

New York State SUN #4701

Draft
wants to merge 91 commits into
base: master
Choose a base branch
from

Conversation

alisonspencer1
Copy link

Draft fixes for issue#4646

@PavelMakarchuk PavelMakarchuk changed the title Alisonspencer1/issue4646 NY-SUN New York State Geothermal Energy System Credit Jul 1, 2024
Copy link
Collaborator

@PavelMakarchuk PavelMakarchuk left a comment

Choose a reason for hiding this comment

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

Generally confused about the purpose of this PR,

Comment on lines 1 to 4
from ... policyengine_us.model_api import *
from ... policyengine_us.entities import *
from ... policyengine_us.tools.general import *
from ... kj_alison_issues.files import * # do we need this one??? #wont hurt?
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
from ... policyengine_us.model_api import *
from ... policyengine_us.entities import *
from ... policyengine_us.tools.general import *
from ... kj_alison_issues.files import * # do we need this one??? #wont hurt?
from policyengine_us.model_api import *

Copy link
Collaborator

Choose a reason for hiding this comment

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

delete duplicate files and move everything out of the this folder

Copy link
Collaborator

Choose a reason for hiding this comment

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

I am not sure if this variable belongs to this PR

Copy link
Collaborator

Choose a reason for hiding this comment

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

format everything to relate to NY and add referencing to display the origins of the inputs

@PavelMakarchuk PavelMakarchuk changed the title New York State Geothermal Energy System Credit New York State SUN Jul 1, 2024
label = "Past year or projected year home energy use in annual kilowatt hours"
unit = "kWh/year"
definition_period = YEAR
reference = ""
Copy link
Collaborator

Choose a reason for hiding this comment

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

add a ref here. (manual)

alisonspencer1 and others added 29 commits July 2, 2024 14:50
2010-06-07: 0.4

CON_ED:
0000-01-01: .inf
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
0000-01-01: .inf

2010-06-07: 0.8

UPSTATE:
0000-01-01: .inf
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
0000-01-01: .inf



LONG_ISLAND:
0000-01-01: .inf
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
0000-01-01: .inf

period: year
label: New York max size PV system multiplier
reference:
- title: NY-Sun Residential & Nonresidential Incentive Program #add detail after
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- title: NY-Sun Residential & Nonresidential Incentive Program #add detail after
- title: NY-Sun Residential & Nonresidential Incentive Program # Page 4

@@ -0,0 +1,16 @@
description: NY SUN maximum size PV system size multiplier (does not exceed 110%)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
description: NY SUN maximum size PV system size multiplier (does not exceed 110%)
description: New York caps the PV system size at the following percentage of projected energy use under the SUN incentive program.

Comment on lines +24 to +25
multiplier = p.max_pv_system_size
max_kwh = ny_sun_current_solar_energy_use * multiplier
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
multiplier = p.max_pv_system_size
max_kwh = ny_sun_current_solar_energy_use * multiplier
max_kwh = ny_sun_current_solar_energy_use * p.max_pv_system_size

max_kwh = ny_sun_current_solar_energy_use * multiplier
#Conversion factor of 1 kW=1 kWh / 3600 (for number of seconds in an hr)
max_kw = max_kwh / 3600
approved_kw = min(max_kw, ny_sun_new_pw_system_size)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
approved_kw = min(max_kw, ny_sun_new_pw_system_size)
capped_kw = min_(max_kw, ny_sun_new_pw_system_size)

@@ -0,0 +1,25 @@
description: NY gives regional incentives for residential solar projects based on solar system size
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
description: NY gives regional incentives for residential solar projects based on solar system size
description: New York provides the following percentage of the past or projected solar energy usage under the SUN incentive program, based on region.

period: year
breakdown:
- ny_sun_region_status
label: New York regional incentive on residential solar projects
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
label: New York regional incentive on residential solar projects
label: New York SUN incentive rate

Copy link
Collaborator

Choose a reason for hiding this comment

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

rename to rate.yaml

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

Successfully merging this pull request may close these issues.

4 participants