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.

Sign up for GitHub

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

Jump to bottom

Solar elevation time doesn't match the solar elevation #35

Open
Swamp-Ig opened this issue Jun 5, 2019 · 1 comment
Open

Solar elevation time doesn't match the solar elevation #35

Swamp-Ig opened this issue Jun 5, 2019 · 1 comment
Assignees
@sffjunkie
Labels
released
Projects
Refactor code base

Comments

@Swamp-Ig
Copy link

Swamp-Ig commented Jun 5, 2019

There seems to be an issue with the solar elevation calculation, it doesn't seem to match the results given by time_at_elevation.

It can be off by up to six mins at times. I have submitted a test case that demonstrates the issue.

Possibly related to #7 and #34.
@Swamp-Ig Swamp-Ig mentioned this issue Jun 5, 2019
Open
@sffjunkie sffjunkie added this to Issues in Refactor code base Jun 19, 2019
@sffjunkie sffjunkie self-assigned this Jun 27, 2019
@sffjunkie
Copy link
Owner

Times and elevation and time_at_elevation match in version 2.0

@hudcap hudcap mentioned this issue Nov 3, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sffjunkie sffjunkie

Labels
released
Projects
Refactor code base
Issues
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sffjunkie @Swamp-Ig