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

Additional Enhancement to solarSunda #348

Closed
dpinney opened this issue Jun 4, 2015 · 7 comments
Closed

Additional Enhancement to solarSunda #348

dpinney opened this issue Jun 4, 2015 · 7 comments
Assignees

Comments

@dpinney
Copy link
Owner

dpinney commented Jun 4, 2015

TODO

  • Change tilt code to 1 box, "-" makes tilt equal the latitude formula from Sunda spreadsheet. Otherwise it's the degrees the user specifies.
  • Put in dash logic for "System Array Size (kW-DC)". If dash then use SUNDA multiple of system output size (kw-AC) (it's like 1.3809. it's in the excel). Otherwise use number value in there.
  • Port the tilt "-" logic to the other solar models.

QUESTIONS

  • Collapsible sections for construction costs?
  • Inspection and permits - make it 20*panelSize instead of fixed amount? See Doug's notes? We need Paul and Doug to agree on a formula.
@dpinney dpinney added this to the GEARED milestone Jun 4, 2015
@drdanley
Copy link

drdanley commented Jun 4, 2015

Re: Inspection and permits - make it 20*panelSize instead of fixed amount.

This cost is definitely not linear. Maybe partially linear. Could be try something like:

Permit cost = x (fixed cost) + y*panelsize (variable cost)

Paul could probably figure out what the fixed and variable components should be to make sure it matches with the spreadsheet. Maybe 3/4 fixed and 1/4 variable for the 1 MW template sized system.

Not a critical issue -- just an idea.

@dpinney
Copy link
Owner Author

dpinney commented Jun 12, 2015

Mannan is working on the TODO section. More clarity needed on questions.

@dpinney dpinney removed this from the GEARED milestone Jun 12, 2015
@mannanj
Copy link
Contributor

mannanj commented Jun 15, 2015

A few minutes away from having this imported to all models.

Future TODO: Use a regex that makes sure a degree < 90 is entered; currently a degree > 90 crashes the Sunda model. After testing PVwatts, I expect all the models will similarly crash.

@dpinney
Copy link
Owner Author

dpinney commented Jun 15, 2015

Inside the tag use

pattern="^(90|[1-8]\d|\d)?$"

@mannanj
Copy link
Contributor

mannanj commented Jun 15, 2015

Worked like a charm. Was there a reason solarRates currently doesn't have a tilt selection?

@dpinney
Copy link
Owner Author

dpinney commented Jun 15, 2015

Nah. Tilt=latitude is good for that one. Manual tilt is overkill there.

@mannanj
Copy link
Contributor

mannanj commented Jun 15, 2015

Done.

@mannanj mannanj closed this as completed Jun 15, 2015
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