-
Notifications
You must be signed in to change notification settings - Fork 1
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
Remove date time internals #3
Remove date time internals #3
Conversation
…into Remove_DateTime_Internals
…into Remove_DateTime_Internals
Very nice @jmurphy6895 ! To fix tests, we need to add ForwardDiff UUID in |
Hi @jmurphy6895 ! I was testing here and I am seeing a lot of type instabilities in the interpolations functions. See the Cthulhu.jl output of the current version: ![]() against the one in this PR: ![]() This behavior is caused probably due to something in the new definition of the types. |
I could fix it! It was a type definition in the structure. |
Replaced Internal Representation of Current Time with JD instead of DateTime to allow for AutoDiff Capabilities