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.

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

Why use SideReal time? #287

Open
sambacha opened this issue Dec 24, 2022 · 3 comments
Open

Why use SideReal time? #287

sambacha opened this issue Dec 24, 2022 · 3 comments

Comments

@sambacha
Copy link

SideReal Time Problematic?

uint256 internal constant YEAR = 31556952; // 365.24219 ephemeris day = 1 year, in seconds

Why use SideReal time? Esp. post PoW, Ethereum slot times are deterministic, and sidereal time is meant to help telescopes, the application use is questionable unless I am not seeing something.

Side Real: 31_556_952
Gregorian: 31_556_926

Not implying that this use is insecure or wrong
The difference is small - I am more curious as to the motivation behind its intended use than anything

Cheers

@Alexangelj
Copy link
Contributor

You are right! I did not know about the specifics on this, so there was not an intentional motivation. It looks like the Gregorian would fit this application better so I'll be sure to update the next version

@sambacha
Copy link
Author

sambacha commented Jan 9, 2023

You are right! I did not know about the specifics on this, so there was not an intentional motivation. It looks like the Gregorian would fit this application better so I'll be sure to update the next version

To be sure this is more pedantic than a serious issue, I was more curious than anything 😄 so thanks for taking the time to answer this question. Thx

@Alexangelj
Copy link
Contributor

I appreciate details like this so the comment made my day

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants