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

ORM Fixes #44

Open
5 tasks
KinanBab opened this issue Sep 3, 2024 · 0 comments
Open
5 tasks

ORM Fixes #44

KinanBab opened this issue Sep 3, 2024 · 0 comments
Labels
bug Something isn't working open source

Comments

@KinanBab
Copy link
Collaborator

KinanBab commented Sep 3, 2024

  • Debug implementation for BBoxes. Currently uses derive, need to write our own, depends on whether NoPolicy or not at runtime to determine how to debug.
  • PartialEq for BBoxes ---> Need some sort of BBoxPartialEq that returns a BBox or something like it, then our ORM wrappers turn it into a bool before calling ORM code
  • Check ORMPolicy trait. Currently it enforces an empty() function so that we can implement TryFromU64 trait
  • Derive macros for ORM modules rely on things have PartialEq, Debug, etc, we need to replace those with our versions..
  • Add ORM unit tests
@KinanBab KinanBab added bug Something isn't working open source labels Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working open source
Projects
None yet
Development

No branches or pull requests

1 participant