fix: catch pp wrong due to both acc and judgements specified

app/objects/score.py

@@ -324,8 +324,6 @@ def calculate_performance(self, beatmap_id: int) -> tuple[float, float]:
             mode=mode_vn,
             mods=int(self.mods),
             combo=self.max_combo,
-            # prefer to use the score's specific params that add up to the acc
-            acc=self.acc,
             ngeki=self.ngeki,
             n300=self.n300,
             nkatu=self.nkatu,

app/usecases/performance.py

@@ -18,6 +18,7 @@ class ScoreParams:
     combo: int | None = None
 
     # caller may pass either acc OR 300/100/50/geki/katu/miss
+    # passing both will result in a value error being raised
     acc: float | None = None
 
     n300: int | None = None
@@ -60,21 +61,24 @@ def calculate_performances(
     osu_file_path: str,
     scores: Iterable[ScoreParams],
 ) -> list[PerformanceResult]:
+    """\
+    Calculate performance for multiple scores on a single beatmap.
+
+    Typically most useful for mass-recalculation situations.
+
+    TODO: Some level of error handling & returning to caller should be
+    implemented here to handle cases where e.g. the beatmap file is invalid
+    or there an issue during calculation.
+    """
     calc_bmap = Beatmap(path=osu_file_path)
 
     results: list[PerformanceResult] = []
 
     for score in scores:
-        # assert either acc OR 300/100/50/geki/katu/miss is present, but not both
-        # if (score.acc is None) == (
-        #     score.n300 is None
-        #     and score.n100 is None
-        #     and score.n50 is None
-        #     and score.ngeki is None
-        #     and score.nkatu is None
-        #     and score.nmiss is None
-        # ):
-        #     raise ValueError("Either acc OR 300/100/50/geki/katu/miss must be present")
+        if score.acc and (
+            score.n300 or score.n100 or score.n50 or score.ngeki or score.nkatu
+        ):
+            raise ValueError("Must not specify both accuracy and geki/katu/50s/100s.")
 
         # rosupp ignores NC and requires DT
         if score.mods is not None:

tools/recalc.py

@@ -66,7 +66,6 @@ async def recalculate_score(
     calculator = Calculator(
         mode=GameMode(score["mode"]).as_vanilla,
         mods=score["mods"],
-        acc=score["acc"],
         combo=score["max_combo"],
         n_geki=score["ngeki"],  # Mania 320s
         n300=score["n300"],