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

Add additional timeframes and make them consistent across backward looking graphs / tables #263

Open
2 tasks
ckoopmann opened this issue Jan 10, 2023 · 7 comments

Comments

@ckoopmann
Copy link
Collaborator

ckoopmann commented Jan 10, 2023

Currently the website supports different timeframes:

Eth-Supply Graph:
5m, 1h, 1d, 7d, 30d, since merge

All other graphs / tables:
5m, 1h, 1d, 7d, 30d, since burn

Suggestion:

  • Make this consistent by adding "since burn" / "since merge" to all graphs
  • Add another timeframe such as 1 year to all graphs (optional)
@ckoopmann
Copy link
Collaborator Author

ckoopmann commented Jan 10, 2023

Would love to give this a shot if it is judged a desirable change.
(Although I guess it would also require backend changes maybe ? Any feedback on the expected complexity / scope of this would be appreciated).

@ckoopmann
Copy link
Collaborator Author

Noticed that these suggestions are partially contained in:
#244
#241

@thegoodwei
Copy link

I love this dashboard. I only suggest implementing 90-day (and eventually 1-year) time frames for all dashboard stats, with snapshots per quarter. Users want to view the burnt supply of eth in usd value per annum and quarter. That would make an easier comparison with corporate earnings reports from traditional saas companies over that same time period.

@ckoopmann
Copy link
Collaborator Author

I started hacking on this a little bit but then was blocked by the grouped-analysis-1 endpoint of the api not returning data for the merge timeframe and also being unable to find the implementation of that endpoint in the backend codebase.
See: ultrasoundmoney/eth-analysis-rs#8

@alextes
Copy link
Collaborator

alextes commented Feb 2, 2023

@thegoodwei noted. 1y is already on our wishlist. Didn't know people would like 90-day too. Can you provide more details? Do you monitor it, copy it, do you use this data for anything? and hypothetically, would this even ideally be a dashboard for your use-case. e.g. would it be more useful still if it came as a neat CSV?

@ckoopmann apologies for the slow reply, the backend part is indeed missing! and it's what I'm working on right now. Once I have it I'll likely implement the frontend myself but if I could use the help can I ping you here?

@ckoopmann
Copy link
Collaborator Author

ckoopmann commented Feb 4, 2023

@thegoodwei noted. 1y is already on our wishlist. Didn't know people would like 90-day too. Can you provide more details? Do you monitor it, copy it, do you use this data for anything? and hypothetically, would this even ideally be a dashboard for your use-case. e.g. would it be more useful still if it came as a neat CSV?

@ckoopmann apologies for the slow reply, the backend part is indeed missing! and it's what I'm working on right now. Once I have it I'll likely implement the frontend myself but if I could use the help can I ping you here?

No problem, that sounds great.
I'd love to do the frontend adjustments, which seem pretty straight forward, or help in any way possible. Feel free to ping me if I can be of help🙏
I'm also more than happy to help on the backend side btw. \

Just out of curiosity: Where is the api/fees/grouped-analysis-1 api endpoint implemented ? I couldn't find the implementation anywhere in the open sourced backend code 🤔 (also opened an issue for this in the backend repo )

@alextes
Copy link
Collaborator

alextes commented Feb 5, 2023

@ckoopmann awesome!

That API is in the older typescript repo. Sent you an invite. Happy to take PRs there too.

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

3 participants