Introduce shuffled index for samples

The `next_sample` function and corresponding methods have been updated to bring in a shuffled index for samples. This randomizes the order in which samples are presented to the user. The changes also included some refactoring that removed some lines for a cleaner code.

app.py

@@ -1,3 +1,4 @@
+import random
 import uuid
 
 import gradio as gr
@@ -31,16 +32,20 @@ def update_commit_view(sample_ind):
     return github_link_md, diff_json, commit_msg, repo_val, hash_val
 
 
-def next_sample(current_sample_ind):
+def next_sample(current_sample_ind, shuffled_idx):
     if current_sample_ind == n_samples:
         return None
 
     current_sample_ind += 1
-    updated_view = update_commit_view(current_sample_ind)
+    updated_view = update_commit_view(shuffled_idx[current_sample_ind])
     return (current_sample_ind,) + updated_view
 
 
 with gr.Blocks(theme=gr.themes.Soft()) as demo:
+    repo_val = gr.Textbox(interactive=False, label='repo', visible=False)
+    hash_val = gr.Textbox(interactive=False, label='hash', visible=False)
+    shuffled_idx_val = gr.JSON(visible=False)
+
     with gr.Row():
         current_sample_sld = gr.Slider(minimum=0, maximum=n_samples, step=1,
                                        value=0,
@@ -51,8 +56,6 @@ with gr.Blocks(theme=gr.themes.Soft()) as demo:
                                        container=False,
                                        scale=5)
         with gr.Column(scale=1):
-            repo_val = gr.Textbox(interactive=False, container=False, label='repo', visible=False)
-            hash_val = gr.Textbox(interactive=False, container=False, label='hash', visible=False)
             skip_btn = gr.Button("Skip the current sample")
     with gr.Row():
         with gr.Column(scale=2):
@@ -96,7 +99,6 @@ with gr.Blocks(theme=gr.themes.Soft()) as demo:
     ]
 
     feedback_form = [
-        current_sample_sld,
         session_val,
         repo_val,
         hash_val,
@@ -107,22 +109,26 @@ with gr.Blocks(theme=gr.themes.Soft()) as demo:
 
     saver.setup(feedback_form, "feedback")
 
-    skip_btn.click(next_sample, inputs=[current_sample_sld], outputs=[current_sample_sld] + commit_view)
+    skip_btn.click(next_sample, inputs=[current_sample_sld, shuffled_idx_val],
+                   outputs=[current_sample_sld] + commit_view)
 
 
-    def submit(*args):
-        saver.flag(args)
-        return next_sample(args[0])
+    def submit(current_sample, shuffled_idx, *args):
+        saver.flag([current_sample] + args)
+        return next_sample(current_sample, shuffled_idx)
 
 
-    submit_btn.click(submit, inputs=feedback_form, outputs=[current_sample_sld] + commit_view)
+    submit_btn.click(submit, inputs=[current_sample_sld, shuffled_idx_val] + feedback_form,
+                     outputs=[current_sample_sld] + commit_view)
 
 
-    def init_session(*args):
+    def init_session(current_sample):
         session = str(uuid.uuid4())
-        return (session,) + update_commit_view(*args)
+        shuffled_idx = list(range(n_samples))
+        random.shuffle(shuffled_idx)
+        return (session, shuffled_idx) + update_commit_view(shuffled_idx[current_sample])
 
 
-    demo.load(init_session, inputs=[current_sample_sld], outputs=[session_val] + commit_view)
+    demo.load(init_session, inputs=[current_sample_sld], outputs=[session_val, shuffled_idx_val] + commit_view)
 
 demo.launch()