Skip to content

Commit

Permalink
Merge pull request #53 from Pennycook/relax-column-filter
Browse files Browse the repository at this point in the history
Add "date" to application_efficiency filter
  • Loading branch information
Pennycook authored Jul 22, 2024
2 parents 0e6ad88 + 2b0e701 commit 26358b9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion p3/metrics/_efficiency.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ def application_efficiency(df, foms="lower"):
if foms not in ["lower", "higher"]:
raise ValueError("FOM interpretation must be 'lower' or 'higher'")

result = df.filter(required_columns)
result = df.filter(required_columns + ["date"])

# Identify the best FOM for each (problem, platform) triple
key = ["problem", "platform"]
Expand Down

0 comments on commit 26358b9

Please sign in to comment.