Copy link
Copy Markdown
Contributor
- Resolves some TODOs by adding missing methods, etc.
- Moves the implementation of various
__repr__to the C++ side. - Makes Correspondence a data class to benefit from automatic
__copy__method creation. - Makes Database object compatible with context manager interface to enable
with pycolmap.Database("path.db") as dbusage. - Some backwards compatibility breakage to achieve consistent method names in C++ and Python.
Copy link
Copy Markdown
Contributor
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. LGTM
Copy link
Copy Markdown
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice improvements, thanks!