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

Investigate whether we can create a second MASP epoch tracker #2833

Closed
Tracked by #2841 ...
cwgoes opened this issue Mar 7, 2024 · 9 comments · Fixed by #3318
Closed
Tracked by #2841 ...

Investigate whether we can create a second MASP epoch tracker #2833

cwgoes opened this issue Mar 7, 2024 · 9 comments · Fixed by #3318
Assignees
Labels

Comments

@cwgoes
Copy link
Contributor

cwgoes commented Mar 7, 2024

h/t @gavinly

Investigate whether we can create a second MASP epoch, where masp_epoch = regular_epoch % n (mod n), where n could be, for example, 4 - the objective being to allow faster epochs without letting the conversion table grow too much.

@cwgoes
Copy link
Contributor Author

cwgoes commented Apr 24, 2024

@grarco any idea of how difficult this is?

@grarco
Copy link
Contributor

grarco commented Apr 24, 2024

We are still waiting for a confirmation on whether this is actually needed

@cwgoes
Copy link
Contributor Author

cwgoes commented Apr 24, 2024

A confirmation from whom?

@grarco
Copy link
Contributor

grarco commented Apr 24, 2024

@gavinly

@gavinly
Copy link
Contributor

gavinly commented Apr 24, 2024

i'm ambivalent about prioritizing an epoch time decrease: it doesn't have to be a priority, but i'm concerned that it could be stuck like this forever, once it's the default (politically and/or dev priority). is there some in-between option that wouldn't require creating a second MASP epoch?

@cwgoes
Copy link
Contributor Author

cwgoes commented Apr 24, 2024

I don't think there's an in-between option. I also don't think this change is particularly complicated in principle, but it might be annoying in practice - @grarco do you have a rough idea (hours? days? weeks?)

@grarco
Copy link
Contributor

grarco commented Apr 24, 2024

I agree there's no other option. I'd say it takes a few days (but hard to say for certain until I start to work on this)

@cwgoes
Copy link
Contributor Author

cwgoes commented Apr 24, 2024

Let's revaluate this again next week after other definitely necessary work is completed.

@cwgoes
Copy link
Contributor Author

cwgoes commented May 27, 2024

@grarco to evaluate this in more detail today.

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

Successfully merging a pull request may close this issue.

4 participants