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

Dedupe builtin/vX/util/* #315

Open
rvagg opened this issue Sep 27, 2024 · 0 comments
Open

Dedupe builtin/vX/util/* #315

rvagg opened this issue Sep 27, 2024 · 0 comments

Comments

@rvagg
Copy link
Member

rvagg commented Sep 27, 2024

Similar to #286, almost everything in builtin/vX/util/* is duplicated and identical (apart from minor import reorgs) from v8 to v15. So we should just move them to the top level. The exception to this is the adt directory which can stay.

I'm not sure we even need to leave aliases in place, this could be a breaking change for the next major release. If a future state change adjusts any of these (e.g. FilterEstimate might grow a new field?) then we could do what we do with ActorV4 and ActorV5 or introduce a new package with a version, e.g. smoothing2.

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

No branches or pull requests

1 participant