-
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
Temporary Commit at 8/23/2024, 3:24:47 PM #2701
base: main
Are you sure you want to change the base?
Conversation
This pull request was exported from Phabricator. Differential Revision: D61730570 |
This pull request was exported from Phabricator. Differential Revision: D61730570 |
635293c
to
db057b0
Compare
Summary: `MapData` has an attribute `df` that takes only the last row from each of `map_df`'s trial-arm-metric groups (when sorted by map_key values), and drops `map_key` columns. This diff makes it so that `map_key` columns are still present in the `df` attribute, showing their values for each kept row. This will make it easier to understand and model partially complete trials in the future. [RfC] I've patched up a few tests that assert these dataframes have a specific form, but there don't seem to be any tests failing for functional reasons (since the unused map_key columns would just go unused if not needed). Putting this up as an RfC though in case others feel strongly that these columns should be dropped. Differential Revision: D61730570
This pull request was exported from Phabricator. Differential Revision: D61730570 |
Summary: `MapData` has an attribute `df` that takes only the last row from each of `map_df`'s trial-arm-metric groups (when sorted by map_key values), and drops `map_key` columns. This diff makes it so that `map_key` columns are still present in the `df` attribute, showing their values for each kept row. This will make it easier to understand and model partially complete trials in the future. [RfC] I've patched up a few tests that assert these dataframes have a specific form, but there don't seem to be any tests failing for functional reasons (since the unused map_key columns would just go unused if not needed). Putting this up as an RfC though in case others feel strongly that these columns should be dropped. Reviewed By: Balandat Differential Revision: D61730570
db057b0
to
3d4100b
Compare
This pull request was exported from Phabricator. Differential Revision: D61730570 |
3d4100b
to
3726b14
Compare
Summary: `MapData` has an attribute `df` that takes only the last row from each of `map_df`'s trial-arm-metric groups (when sorted by map_key values), and drops `map_key` columns. This diff makes it so that `map_key` columns are still present in the `df` attribute, showing their values for each kept row. This will make it easier to understand and model partially complete trials in the future. [RfC] I've patched up a few tests that assert these dataframes have a specific form, but there don't seem to be any tests failing for functional reasons (since the unused map_key columns would just go unused if not needed). Putting this up as an RfC though in case others feel strongly that these columns should be dropped. Reviewed By: Balandat Differential Revision: D61730570
This pull request was exported from Phabricator. Differential Revision: D61730570 |
Summary: `MapData` has an attribute `df` that takes only the last row from each of `map_df`'s trial-arm-metric groups (when sorted by map_key values), and drops `map_key` columns. This diff makes it so that `map_key` columns are still present in the `df` attribute, showing their values for each kept row. This will make it easier to understand and model partially complete trials in the future. [RfC] I've patched up a few tests that assert these dataframes have a specific form, but there don't seem to be any tests failing for functional reasons (since the unused map_key columns would just go unused if not needed). Putting this up as an RfC though in case others feel strongly that these columns should be dropped. Reviewed By: Balandat Differential Revision: D61730570
3726b14
to
896e146
Compare
This pull request was exported from Phabricator. Differential Revision: D61730570 |
Summary: `MapData` has an attribute `df` that takes only the last row from each of `map_df`'s trial-arm-metric groups (when sorted by map_key values), and drops `map_key` columns. This diff makes it so that `map_key` columns are still present in the `df` attribute, showing their values for each kept row. This will make it easier to understand and model partially complete trials in the future. [RfC] I've patched up a few tests that assert these dataframes have a specific form, but there don't seem to be any tests failing for functional reasons (since the unused map_key columns would just go unused if not needed). Putting this up as an RfC though in case others feel strongly that these columns should be dropped. Reviewed By: Balandat Differential Revision: D61730570
896e146
to
0b4d4fe
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2701 +/- ##
==========================================
+ Coverage 95.27% 95.28% +0.01%
==========================================
Files 493 493
Lines 47903 47903
==========================================
+ Hits 45641 45646 +5
+ Misses 2262 2257 -5 ☔ View full report in Codecov by Sentry. |
Summary: `MapData` has an attribute `df` that takes only the last row from each of `map_df`'s trial-arm-metric groups (when sorted by map_key values), and drops `map_key` columns. This diff makes it so that `map_key` columns are still present in the `df` attribute, showing their values for each kept row. This will make it easier to understand and model partially complete trials in the future. [RfC] I've patched up a few tests that assert these dataframes have a specific form, but there don't seem to be any tests failing for functional reasons (since the unused map_key columns would just go unused if not needed). Putting this up as an RfC though in case others feel strongly that these columns should be dropped. Reviewed By: Balandat Differential Revision: D61730570
This pull request was exported from Phabricator. Differential Revision: D61730570 |
0b4d4fe
to
8cb9d3c
Compare
Summary: `MapData` has an attribute `df` that takes only the last row from each of `map_df`'s trial-arm-metric groups (when sorted by map_key values), and drops `map_key` columns. This diff makes it so that `map_key` columns are still present in the `df` attribute, showing their values for each kept row. This will make it easier to understand and model partially complete trials in the future. [RfC] I've patched up a few tests that assert these dataframes have a specific form, but there don't seem to be any tests failing for functional reasons (since the unused map_key columns would just go unused if not needed). Putting this up as an RfC though in case others feel strongly that these columns should be dropped. Reviewed By: Balandat Differential Revision: D61730570 fbshipit-source-id: 0f09b357e0382b48a42b61d5c0f5a63206736196
Differential Revision: D61730570