Volodymyr Agafonkin
Volodymyr Agafonkin
This is the only feature preventing me from switching from Rollup to ESBuild as the author of [50+ open source JS libraries](https://github.com/mourner/projects). Gently bringing this to your attention @evanw 🙏
You're right, I used south-based azimuth whereas north-based azimuth is usually used for sun position...
Probably, but I'd suggest just adding a PI instead of waiting for it to be merged.
Yeah, hesitating to break backwards compatibility, leading to even more confusion :) I'll deal with this eventually, just didn't get my hands on it yet.
@jhiesey Sorry for a late response, and thanks for catching the bug! Would you mind submitting a PR with a fix?
@fchristant while I'm going to fix the problem, please don't complain — this is open source, if something's not working for you, I'm happily accepting pull requests. There's little activity...
OK, now I see what's going on here — SunCalc calculates all the times after it gets the time of the nearest solar transit. But solar transit isn't strictly 12am...
@emcjh it shows you times in your local time zone, you need to convert them according to the time zone of the location if you want precise local times and...
If your timezones are good, why is sunrise happening at 7:44pm?
@Malvoz maybe you could take a look? Not sure what's going on there.