SunAltitude stopped working about coordinatesharp HOT 4 CLOSED

I 100% agree! I have some test projects, but they are starting to become too manual, and if I don't test every aspect, I miss stupid things on occasion as we just saw. I will start working on something better as the library is starting to grow faster.

from coordinatesharp.

@dsherret #^%$* Thanks! I found the issue immediately. I passed an incorrect value after I made adjustments to the solar time calculations. I'm fixing it now! I'll message back in a few when it's pushed.

from coordinatesharp.

@dsherret It's pushed to GitHub, Nuget and the Website. I unlisted the versions effected by the issue. The only properties effected were sun alt/az. I had set the function that handles them to calculate the stripped GeoDate with no time by accident.

Silly mistake. Thank you for bringing it to my attention, let me know if you find anything else. It should work now.

from coordinatesharp.

Thanks for the quick fix and once again for the awesome project!

By the way, I highly recommend adding a test project to ensure the public api is returning values within a certain epsilon (working on it slowly with bug fixes is a good time to start). It will allow you to do changes with confidence and catch mistakes like this :)

from coordinatesharp.