fix bug in dice rolling: highest value was never rolled

ssciwr · Feb 14, 2024 · df2ff4c · df2ff4c
1 parent 32ea1cb
commit df2ff4c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/calculate-liam/pyproject.toml b/calculate-liam/pyproject.toml
@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "calculate-liam"
-version = "0.0.6"
+version = "0.0.7"
 dependencies = ["click", "numpy"]
 authors = [
   { name="Liam Keegan", email="liam@keegan.ch" },

diff --git a/calculate-liam/src/calculate_liam/cli.py b/calculate-liam/src/calculate_liam/cli.py
@@ -16,4 +16,5 @@ def flip_coin_cli():
 def roll_dice_cli(n_dice, n_sides):
     click.echo("")
     for d in roll_dice(n_dice, n_sides):
-        click.secho(f" {d} \n", bold=True, bg="red")
+        click.secho(f" {d} ", bold=True, bg="red")
+        click.echo("")
diff --git a/calculate-liam/src/calculate_liam/stats.py b/calculate-liam/src/calculate_liam/stats.py
@@ -7,4 +7,4 @@ def flip_coin() -> str:
 
 
 def roll_dice(n_dice: int, n_sides: int):
-    return np.random.randint(1, n_sides, n_dice)
+    return np.random.randint(low=1, high=n_sides + 1, size=n_dice)