From 05248e0ecd05923ccebd5e1081bb260448c25df4 Mon Sep 17 00:00:00 2001
From: Pauline Ribeyre <4224001+paulineribeyre@users.noreply.github.com>
Date: Thu, 27 Aug 2026 14:15:36 -0500
Subject: [PATCH 1/2] Gen3Submission: error check fixes

---
 gen3/submission.py | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/gen3/submission.py b/gen3/submission.py
index 1e82039c..b610dbe0 100755
--- a/gen3/submission.py
+++ b/gen3/submission.py
@@ -247,11 +247,11 @@ def delete_records(self, program, project, uuids, batch_size=100):
             uuids_to_delete = uuids[batch_size * i : batch_size * (i + 1)]
             if len(uuids_to_delete) == 0:
                 break
-            output = requests.delete(
-                "{}/{}".format(api_url, ",".join(uuids_to_delete)),
-                auth=self._auth_provider,
-            )
             try:
+                output = requests.delete(
+                    "{}/{}".format(api_url, ",".join(uuids_to_delete)),
+                    auth=self._auth_provider,
+                )
                 raise_for_status_and_print_error(output)
             except requests.exceptions.HTTPError:
                 print(
@@ -347,7 +347,9 @@ def export_record(self, program, project, uuid, fileformat, filename=None):
         api_url = "{}/api/v0/submission/{}/{}/export?ids={}&format={}".format(
             self._endpoint, program, project, uuid, fileformat
         )
-        output = requests.get(api_url, auth=self._auth_provider).text
+        response = requests.get(api_url, auth=self._auth_provider)
+        raise_for_status_and_print_error(response)
+        output = response.text
         if filename is None:
             if fileformat == "json":
                 try:
@@ -383,7 +385,9 @@ def export_node(self, program, project, node_type, fileformat, filename=None):
         api_url = "{}/api/v0/submission/{}/{}/export/?node_label={}&format={}".format(
             self._endpoint, program, project, node_type, fileformat
         )
-        output = requests.get(api_url, auth=self._auth_provider).text
+        response = requests.get(api_url, auth=self._auth_provider)
+        raise_for_status_and_print_error(response)
+        output = response.text
         if filename is None:
             if fileformat == "json":
                 try:

From baedd4d81caca12dcb92f71bda7220a1ab4ed1f7 Mon Sep 17 00:00:00 2001
From: paulineribeyre <paulineribeyre@users.noreply.github.com>
Date: Thu, 27 Aug 2026 19:16:31 +0000
Subject: [PATCH 2/2] Apply automatic documentation changes

---
 docs/_build/html/_modules/gen3/submission.html | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/docs/_build/html/_modules/gen3/submission.html b/docs/_build/html/_modules/gen3/submission.html
index b82fd483..f0ef32c9 100644
--- a/docs/_build/html/_modules/gen3/submission.html
+++ b/docs/_build/html/_modules/gen3/submission.html
@@ -314,11 +314,11 @@ <h1>Source code for gen3.submission</h1><div class="highlight"><pre>
             <span class="n">uuids_to_delete</span> <span class="o">=</span> <span class="n">uuids</span><span class="p">[</span><span class="n">batch_size</span> <span class="o">*</span> <span class="n">i</span> <span class="p">:</span> <span class="n">batch_size</span> <span class="o">*</span> <span class="p">(</span><span class="n">i</span> <span class="o">+</span> <span class="mi">1</span><span class="p">)]</span>
             <span class="k">if</span> <span class="nb">len</span><span class="p">(</span><span class="n">uuids_to_delete</span><span class="p">)</span> <span class="o">==</span> <span class="mi">0</span><span class="p">:</span>
                 <span class="k">break</span>
-            <span class="n">output</span> <span class="o">=</span> <span class="n">requests</span><span class="o">.</span><span class="n">delete</span><span class="p">(</span>
-                <span class="s2">"</span><span class="si">{}</span><span class="s2">/</span><span class="si">{}</span><span class="s2">"</span><span class="o">.</span><span class="n">format</span><span class="p">(</span><span class="n">api_url</span><span class="p">,</span> <span class="s2">","</span><span class="o">.</span><span class="n">join</span><span class="p">(</span><span class="n">uuids_to_delete</span><span class="p">)),</span>
-                <span class="n">auth</span><span class="o">=</span><span class="bp">self</span><span class="o">.</span><span class="n">_auth_provider</span><span class="p">,</span>
-            <span class="p">)</span>
             <span class="k">try</span><span class="p">:</span>
+                <span class="n">output</span> <span class="o">=</span> <span class="n">requests</span><span class="o">.</span><span class="n">delete</span><span class="p">(</span>
+                    <span class="s2">"</span><span class="si">{}</span><span class="s2">/</span><span class="si">{}</span><span class="s2">"</span><span class="o">.</span><span class="n">format</span><span class="p">(</span><span class="n">api_url</span><span class="p">,</span> <span class="s2">","</span><span class="o">.</span><span class="n">join</span><span class="p">(</span><span class="n">uuids_to_delete</span><span class="p">)),</span>
+                    <span class="n">auth</span><span class="o">=</span><span class="bp">self</span><span class="o">.</span><span class="n">_auth_provider</span><span class="p">,</span>
+                <span class="p">)</span>
                 <span class="n">raise_for_status_and_print_error</span><span class="p">(</span><span class="n">output</span><span class="p">)</span>
             <span class="k">except</span> <span class="n">requests</span><span class="o">.</span><span class="n">exceptions</span><span class="o">.</span><span class="n">HTTPError</span><span class="p">:</span>
                 <span class="nb">print</span><span class="p">(</span>
@@ -423,7 +423,9 @@ <h1>Source code for gen3.submission</h1><div class="highlight"><pre>
         <span class="n">api_url</span> <span class="o">=</span> <span class="s2">"</span><span class="si">{}</span><span class="s2">/api/v0/submission/</span><span class="si">{}</span><span class="s2">/</span><span class="si">{}</span><span class="s2">/export?ids=</span><span class="si">{}</span><span class="s2">&amp;format=</span><span class="si">{}</span><span class="s2">"</span><span class="o">.</span><span class="n">format</span><span class="p">(</span>
             <span class="bp">self</span><span class="o">.</span><span class="n">_endpoint</span><span class="p">,</span> <span class="n">program</span><span class="p">,</span> <span class="n">project</span><span class="p">,</span> <span class="n">uuid</span><span class="p">,</span> <span class="n">fileformat</span>
         <span class="p">)</span>
-        <span class="n">output</span> <span class="o">=</span> <span class="n">requests</span><span class="o">.</span><span class="n">get</span><span class="p">(</span><span class="n">api_url</span><span class="p">,</span> <span class="n">auth</span><span class="o">=</span><span class="bp">self</span><span class="o">.</span><span class="n">_auth_provider</span><span class="p">)</span><span class="o">.</span><span class="n">text</span>
+        <span class="n">response</span> <span class="o">=</span> <span class="n">requests</span><span class="o">.</span><span class="n">get</span><span class="p">(</span><span class="n">api_url</span><span class="p">,</span> <span class="n">auth</span><span class="o">=</span><span class="bp">self</span><span class="o">.</span><span class="n">_auth_provider</span><span class="p">)</span>
+        <span class="n">raise_for_status_and_print_error</span><span class="p">(</span><span class="n">response</span><span class="p">)</span>
+        <span class="n">output</span> <span class="o">=</span> <span class="n">response</span><span class="o">.</span><span class="n">text</span>
         <span class="k">if</span> <span class="n">filename</span> <span class="ow">is</span> <span class="kc">None</span><span class="p">:</span>
             <span class="k">if</span> <span class="n">fileformat</span> <span class="o">==</span> <span class="s2">"json"</span><span class="p">:</span>
                 <span class="k">try</span><span class="p">:</span>
@@ -462,7 +464,9 @@ <h1>Source code for gen3.submission</h1><div class="highlight"><pre>
         <span class="n">api_url</span> <span class="o">=</span> <span class="s2">"</span><span class="si">{}</span><span class="s2">/api/v0/submission/</span><span class="si">{}</span><span class="s2">/</span><span class="si">{}</span><span class="s2">/export/?node_label=</span><span class="si">{}</span><span class="s2">&amp;format=</span><span class="si">{}</span><span class="s2">"</span><span class="o">.</span><span class="n">format</span><span class="p">(</span>
             <span class="bp">self</span><span class="o">.</span><span class="n">_endpoint</span><span class="p">,</span> <span class="n">program</span><span class="p">,</span> <span class="n">project</span><span class="p">,</span> <span class="n">node_type</span><span class="p">,</span> <span class="n">fileformat</span>
         <span class="p">)</span>
-        <span class="n">output</span> <span class="o">=</span> <span class="n">requests</span><span class="o">.</span><span class="n">get</span><span class="p">(</span><span class="n">api_url</span><span class="p">,</span> <span class="n">auth</span><span class="o">=</span><span class="bp">self</span><span class="o">.</span><span class="n">_auth_provider</span><span class="p">)</span><span class="o">.</span><span class="n">text</span>
+        <span class="n">response</span> <span class="o">=</span> <span class="n">requests</span><span class="o">.</span><span class="n">get</span><span class="p">(</span><span class="n">api_url</span><span class="p">,</span> <span class="n">auth</span><span class="o">=</span><span class="bp">self</span><span class="o">.</span><span class="n">_auth_provider</span><span class="p">)</span>
+        <span class="n">raise_for_status_and_print_error</span><span class="p">(</span><span class="n">response</span><span class="p">)</span>
+        <span class="n">output</span> <span class="o">=</span> <span class="n">response</span><span class="o">.</span><span class="n">text</span>
         <span class="k">if</span> <span class="n">filename</span> <span class="ow">is</span> <span class="kc">None</span><span class="p">:</span>
             <span class="k">if</span> <span class="n">fileformat</span> <span class="o">==</span> <span class="s2">"json"</span><span class="p">:</span>
                 <span class="k">try</span><span class="p">:</span>