Fail fast if the user isn't logged in

functions.py

@@ -1,4 +1,3 @@
-import os
 
 import gradio as gr
 import pandas as pd
@@ -13,7 +12,11 @@ from huggingface_hub import (
 from huggingface_hub.repocard_data import eval_results_to_model_index
 from pytablewriter import MarkdownTableWriter
 
-BOT_HF_TOKEN = os.getenv("BOT_HF_TOKEN")
+COMMIT_DESCRIPTION = """This is an automated PR created with https://huggingface.co/spaces/T145/open-llm-leaderboard-results-to-modelcard
+
+The purpose of this PR is to add evaluation results from the Open LLM Leaderboard to your model card.
+
+Please report any issues here: https://huggingface.co/spaces/T145/open-llm-leaderboard-results-to-modelcard/discussions"""
 
 
 def search(df, value):
@@ -157,28 +160,18 @@ def get_edited_yaml_readme(df, repo, token: str | None):
 
 def commit(
     repo,
-    pr_number=None,
+    pr_number=None, # specify pr number if you want to edit it
     message="Adding Evaluation Results",
     oauth_token: gr.OAuthToken | None = None,
-):  # specify pr number if you want to edit it, don't if you don't want
-    data = load_dataset("open-llm-leaderboard/contents", split="train")
-    df = pd.DataFrame(data)
-
-    desc = """
-  This is an automated PR created with https://huggingface.co/spaces/T145/open-llm-leaderboard-results-to-modelcard
-
-  The purpose of this PR is to add evaluation results from the Open LLM Leaderboard to your model card.
-
-  Please report any issues here: https://huggingface.co/spaces/T145/open-llm-leaderboard-results-to-modelcard/discussions
-  """
-
+):
     if not oauth_token:
-        raise gr.Warning(
-            "You are not logged in. Click on 'Sign in with Huggingface' to log in."
-        )
+        raise gr.Warning("You are not logged in. Click on 'Sign in with Huggingface' to log in.")
     else:
         token = oauth_token
 
+    data = load_dataset("open-llm-leaderboard/contents", split="train")
+    df = pd.DataFrame(data)
+
     if repo.startswith("https://huggingface.co/"):
         try:
             repo = RepoUrl(repo).repo_id
@@ -198,7 +191,7 @@ def commit(
             else:
                 print(f"Something went wrong: {e}")
 
-        liste = [
+        ops = [
             CommitOperationAdd(
                 path_in_repo="README.md", path_or_fileobj=readme_text.encode()
             )
@@ -206,9 +199,9 @@ def commit(
         commit = create_commit(
             repo_id=repo,
             token=token,
-            operations=liste,
+            operations=ops,
             commit_message=message,
-            commit_description=desc,
+            commit_description=COMMIT_DESCRIPTION,
             repo_type="model",
             **edited,
         ).pr_url