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

[nit] Less structures exposed in Cython #15

Merged

blefaudeux merged 1 commit into main from ben/less_public_fields

Oct 6, 2024

Contributor

blefaudeux commented Oct 6, 2024

Keeping more structures internal

no benefit in exposing them to python and augments the cython code
we're about to make this more modular, to expose loading from local folders or object storage, so anything DB related is not generic enough anyway


          nit, less structures exposed in Cython

71eb181

Contributor Author

blefaudeux commented Oct 6, 2024

cc @DeltaEvo, waiting for the CI but instaland after that, feel free to chime in anytime

blefaudeux merged commit 83bb17c into main

2 checks passed

blefaudeux deleted the ben/less_public_fields branch

November 6, 2024 22:56

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet