-
Notifications
You must be signed in to change notification settings - Fork 0
NC code to dos #19
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
NC code to dos #19
Conversation
…igh Level API prep
…updated the high-level api
…ut the object type casting
…and an include list for the error mechanisms.
se-jaeger
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a few TODO statements in the codebase. Can you make sure they still make sense. No need to fix them now but please double check that they are still make sense and are not already solved.
Other than that, I think it's good to go! Good job!
se-jaeger
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Finished several TODOs scattered throughout the code base
Added default behavior for some error types
Refactored the permutate error type
Built the basis of the high level API
Updated the getting started notebook with High Level API information
Added tolerance of datetime64 objects for the numerical error types