From 105a3e9c0eb6812c63af3ad07a6d2be07cba66b1 Mon Sep 17 00:00:00 2001
From: Harold Erbin <harold.erbin@gmail.com>
Date: Mon, 14 Oct 2024 09:46:05 +0200
Subject: [PATCH] Fix normalisation in ring of atoms

The original formulas was normalized using np.tan(np.pi / L). However,
this is not correct, as can be checked by computing the normalization
between the distances between the atoms (one does not get
R_interatomic).
---
 tutorials/simulating_sequences.ipynb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tutorials/simulating_sequences.ipynb b/tutorials/simulating_sequences.ipynb
index 3c942b5ea..fd9565119 100644
--- a/tutorials/simulating_sequences.ipynb
+++ b/tutorials/simulating_sequences.ipynb
@@ -51,7 +51,7 @@
     "R_interatomic = pulser.MockDevice.rydberg_blockade_radius(U)\n",
     "coords = (\n",
     "    R_interatomic\n",
-    "    / (2 * np.tan(np.pi / L))\n",
+    "    / (2 * np.sin(np.pi / L))\n",
     "    * np.array(\n",
     "        [\n",
     "            (np.cos(theta * 2 * np.pi / L), np.sin(theta * 2 * np.pi / L))\n",