{"action":"closed","issue":{"url":"https://api.github.com/repos/dpinney/omf/issues/307","labels_url":"https://api.github.com/repos/dpinney/omf/issues/307/labels{/name}","comments_url":"https://api.github.com/repos/dpinney/omf/issues/307/comments","events_url":"https://api.github.com/repos/dpinney/omf/issues/307/events","html_url":"https://github.com/dpinney/omf/issues/307","id":56730136,"number":307,"title":"pvWatts upgrades as part of GOSED","user":{"login":"dpinney","id":2131438,"avatar_url":"https://avatars.githubusercontent.com/u/2131438?v=3","gravatar_id":"","url":"https://api.github.com/users/dpinney","html_url":"https://github.com/dpinney","followers_url":"https://api.github.com/users/dpinney/followers","following_url":"https://api.github.com/users/dpinney/following{/other_user}","gists_url":"https://api.github.com/users/dpinney/gists{/gist_id}","starred_url":"https://api.github.com/users/dpinney/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dpinney/subscriptions","organizations_url":"https://api.github.com/users/dpinney/orgs","repos_url":"https://api.github.com/users/dpinney/repos","events_url":"https://api.github.com/users/dpinney/events{/privacy}","received_events_url":"https://api.github.com/users/dpinney/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/dpinney/omf/milestones/5","html_url":"https://github.com/dpinney/omf/milestones/The%20Future","labels_url":"https://api.github.com/repos/dpinney/omf/milestones/5/labels","id":327854,"number":5,"title":"The Future","description":"Unscheduled issues.","creator":{"login":"dpinney","id":2131438,"avatar_url":"https://avatars.githubusercontent.com/u/2131438?v=3","gravatar_id":"","url":"https://api.github.com/users/dpinney","html_url":"https://github.com/dpinney","followers_url":"https://api.github.com/users/dpinney/followers","following_url":"https://api.github.com/users/dpinney/following{/other_user}","gists_url":"https://api.github.com/users/dpinney/gists{/gist_id}","starred_url":"https://api.github.com/users/dpinney/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dpinney/subscriptions","organizations_url":"https://api.github.com/users/dpinney/orgs","repos_url":"https://api.github.com/users/dpinney/repos","events_url":"https://api.github.com/users/dpinney/events{/privacy}","received_events_url":"https://api.github.com/users/dpinney/received_events","type":"User","site_admin":false},"open_issues":86,"closed_issues":31,"state":"open","created_at":"2013-05-06T15:48:03Z","updated_at":"2015-05-05T15:09:30Z","due_on":"2100-01-01T08:00:00Z","closed_at":null},"comments":1,"created_at":"2015-02-05T21:09:24Z","updated_at":"2015-05-05T15:09:30Z","closed_at":"2015-05-05T15:09:30Z","body":"ORIGINAL FEEDBACK\r\nhttps://www.dropbox.com/s/hn3nchmroq4erk4/GOSED%20Notes%2027%20Jan%2015%20drd.docx?dl=0\r\n\r\nTODO\r\n* ~~PVWatts is a copyrighted name, and it is one word: PVWatts©. Also need to give some credit to NREL along with some sort of hyperlink.~~\r\n* ~~Start Date / Run time – should default to Jan 1st and 8760 hours.~~\r\n* ~~Start date is not really valid with a year, since we are using TMY data which is a blend of various years and is intended to be representative, rather than an exact year.~~ We need to leave it since the chart library we use requires a year. Also it's handy if the user wants more than 8760 hours.\r\n* ~~Need a field to specify “tilt”, which should be editable when the user chooses “Fixed”, “one axis” or “azimuth” tracking. Note that there are two different types of one axis trackers – east-west axis (where tilt is meaningless), and north-south axis (where tilt is usually zero in modern grid-tied systems, but could be up to 90 degrees). I don’t know how PVWatts does these calculations, so it may be best to just not offer tilt editing for “one axis” tracking.~~\r\n* ~~NOCT, Ref Cell Temperature, Diffuse Fraction, Rating Condition Irradiance: I don’t think that any of these should be user variables.~~\r\n* ~~Max Power Temperature Coefficient default to 0.45.  It should be stated that this is for modules, and the datasheet value is typically specified as “% per degree C (or degree K)”. Note that this is 0.45%, which is decimal (fraction?) form is 0.0045.~~\r\n* Inverter Efficiency – This is important, but it is part of “Derate” in the main section in PVWatts. Does this number then automatically calculate the “Derate” number in the main section? Also... bigger more detailed derate section?\r\n* ~~I would really like to see an “Inverter AC Size” field so that we can use this program to look at the effects of clipping. Also add line to charts.~~ Added. Defaults to same size as system.\r\n* ~~This brings up the subject of percent as a number (0-100%) or percent as a fraction. “Derate” is in % while this number and others are in fractions. Need to decide on one or the other and be consistent.~~\r\n* ~~Min Required Irradiance for Operation – Inverter datasheets will typically specify a minimum voltage for startup and for MPPT operation. I don’t think this is a value that users could easily get, so I think it should be eliminated as a variable.~~\r\n* ~~Tracker rotation limit (degrees), wind stow speed (m/s). I personally think this is too detailed for the purpose of this screening tool, especially the wind stow speed. But if you must use it the first value (rotation limit) needs to be specified as degrees from horizontal (I think – check the code?).~~ Eh, I'll leave it for now. Advanced option. Fixed the rot limit label.\r\n\r\nFROM SOLARFINANCIAL FEEDBACK\r\n* ~~Bounds checking with regexes.~~\r\n* ~~Nameplate lines on output.~~\r\n* ~~Fix irradiance to global, put it on a separate chart.~~\r\n* ~~Charts should start at 0. Except for temperature.~~\r\n\r\nFINAL\r\n* ~~Update production.~~"}}