CenterForTheBuiltEnvironment / mrt

http://centerforthebuiltenvironment.github.io/mrt
GNU General Public License v3.0
4 stars 8 forks source link

2 azimuths are confusing #8

Open dheinzerling opened 9 years ago

dheinzerling commented 9 years ago

There are 2 azimuths for the user to enter. One is under the Occupant heading and the other is under the SolarCal heading. The one under the solarcal heading controls the actual position of the sun while the one under the occupant heading controls the occupant azimuth as used in the solarcal calculation (0 = front facing window). This is somewhat confusing. Personally, I would get rid of the occupant heading and move occupant azimuth and posture down to the SolarCal heading and do one of 2 options:

  1. leave solar azimuth, but call it solar azimuth, not az
  2. or create a new heading called "Sun" and move solar azimuth and altitude to that heading.
thoyt commented 9 years ago

@dheinzerling I think the occupant heading is sort of needed. I don't want to force people to think about SolarCal if they're just doing longwave mrt calcs.

Definitely agree that having two azimuths is confusing and that it should be clarified. I think changing the names should be a good start.

praftery commented 6 years ago

Entering lat/long and datetime might be more intuitive for solar positioning than alt/azimuth. Would also neatly resolve this issue of the two azimuths.