diff options
author | Joel Fernandes <joelaf@google.com> | 2017-07-13 21:48:09 -0700 |
---|---|---|
committer | Joel Fernandes <joelaf@google.com> | 2017-07-13 21:48:39 -0700 |
commit | effd7b5ba93f752abe87c0def558d64e9b88b2ef (patch) | |
tree | fcae6b21749329c8b2881e187bbd8df7f3eecf83 | |
parent | 49f4c42b6750e96c668840a7d54e2cc2f322ab09 (diff) | |
download | trappy-effd7b5ba93f752abe87c0def558d64e9b88b2ef.tar.gz |
trappy: optimize merge_df by removing try/exceptoreo-mr1-dev
Change-Id: I238bb50ed1907def19b23b0610eec87234ef4d51
Signed-off-by: Joel Fernandes <joelaf@google.com>
-rw-r--r-- | trappy/utils.py | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/trappy/utils.py b/trappy/utils.py index 57b4ccd..47ef4ab 100644 --- a/trappy/utils.py +++ b/trappy/utils.py @@ -147,11 +147,9 @@ def merge_dfs(pr_df, sec_df, pivot): for key, value in data.iteritems(): if key == pivot: continue - try: - if np.isnan(value): - data[key] = pivot_map[data[pivot]][key] - except: - pass + # Fast check for if value is nan (faster than np.isnan + try/except) + if value != value and pivot_map.has_key(data[pivot]): + data[key] = pivot_map[data[pivot]][key] # Get rid of primary/secondary labels data['Time'] = data['Time'][1] |