-
Notifications
You must be signed in to change notification settings - Fork 303
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
Make evaluation metadata accept iso formatted dates instead of ints in ms #1334
Conversation
This pull request was exported from Phabricator. Differential Revision: D42050451 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D42050451 |
…n ms (facebook#1334) Summary: Pull Request resolved: facebook#1334 Because ints in ms don't work with pd.Timestamp, which the models ultimately need (https://fburl.com/code/ci5ji3kr) and which render in data Differential Revision: D42050451 fbshipit-source-id: ef853ae8e5d42ef489cadd66ac6af5d36efdba15
8f980e3
to
43aae7b
Compare
Codecov ReportBase: 94.66% // Head: 94.66% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1334 +/- ##
=======================================
Coverage 94.66% 94.66%
=======================================
Files 420 420
Lines 38485 38526 +41
=======================================
+ Hits 36430 36471 +41
Misses 2055 2055
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
…n ms (facebook#1334) Summary: Pull Request resolved: facebook#1334 Because ints in ms don't work with pd.Timestamp, which the models ultimately need (https://fburl.com/code/ci5ji3kr) and which render in data Differential Revision: D42050451 fbshipit-source-id: 0bc5328dd64afe648e459c381856109a467195ef
43aae7b
to
02eafd2
Compare
This pull request was exported from Phabricator. Differential Revision: D42050451 |
…n ms (facebook#1334) Summary: Pull Request resolved: facebook#1334 Because ints in ms don't work with pd.Timestamp, which the models ultimately need (https://fburl.com/code/ci5ji3kr) and which render in data Differential Revision: D42050451 fbshipit-source-id: 1c52446c15d2900f3f5829fe72d05d1ab953210d
02eafd2
to
b099cb8
Compare
This pull request was exported from Phabricator. Differential Revision: D42050451 |
…ts in ms (facebook#1334) Summary: Pull Request resolved: facebook#1334 Because ints in ms don't work with pd.Timestamp, which the models ultimately need (https://fburl.com/code/ci5ji3kr) and which render in data Also fix interpretation of ints in ms in `Data.from_evaluations()` Differential Revision: D42050451 fbshipit-source-id: 17bfdb0f5725b5bff2e883f0f3ed303a975fc95c
b099cb8
to
eb154f4
Compare
This pull request was exported from Phabricator. Differential Revision: D42050451 |
…n ms (facebook#1334) Summary: Pull Request resolved: facebook#1334 Because ints in ms don't work with pd.Timestamp, which the models ultimately need (https://fburl.com/code/ci5ji3kr) and which render in data Differential Revision: https://internalfb.com/D42050451 fbshipit-source-id: 7f4a0fd488337dadd8ee452241364402af077405
…n ms (facebook#1334) Summary: Pull Request resolved: facebook#1334 Because ints in ms don't work with pd.Timestamp, which the models ultimately need (https://fburl.com/code/ci5ji3kr) and which render in data Differential Revision: https://internalfb.com/D42050451 fbshipit-source-id: 7ad8713c399d0bea81cab7eaa4a4233191e6c142
This pull request has been merged in c6c67d8. |
Summary: Because ints in ms don't work with pd.Timestamp, which the models ultimately need (https://fburl.com/code/ci5ji3kr) and which render in data
Differential Revision: D42050451