Formatting

Author: Tkaixiang

lib/cadet/assessments/assessments.ex

@@ -2290,7 +2290,8 @@ defmodule Cadet.Assessments do
     base_query =
       Answer
       |> where(submission_id: ^id)
-      |> join(:inner, [a], q in assoc(a, :question))  # [a] are bindings (in SQL it is similar to FROM answers "AS a"), this line's alias is INNER JOIN ... "AS q"
+      # [a] are bindings (in SQL it is similar to FROM answers "AS a"), this line's alias is INNER JOIN ... "AS q"
+      |> join(:inner, [a], q in assoc(a, :question))
       |> join(:inner, [_, q], ast in assoc(q, :assessment))
       |> join(:inner, [..., ast], ac in assoc(ast, :config))
       |> join(:left, [a, ...], g in assoc(a, :grader))
@@ -2794,11 +2795,11 @@ defmodule Cadet.Assessments do
   end
 
   def get_llm_assessment_prompt(question_id) do
-     from(q in Question,
-    where: q.id == ^question_id,
-    join: a in assoc(q, :assessment),
-    select: a.llm_assessment_prompt
-  )
-  |> Repo.one()
+    from(q in Question,
+      where: q.id == ^question_id,
+      join: a in assoc(q, :assessment),
+      select: a.llm_assessment_prompt
+    )
+    |> Repo.one()
   end
 end

lib/cadet/courses/course.ex

@@ -100,6 +100,7 @@ defmodule Cadet.Courses.Course do
     if llm_api_key = get_change(changeset, :llm_api_key) do
       if is_binary(llm_api_key) and llm_api_key != "" do
         encrypted = encrypt_llm_api_key(llm_api_key)
+
         case encrypted do
           nil ->
             add_error(
@@ -111,7 +112,6 @@ defmodule Cadet.Courses.Course do
           encrypted ->
             put_change(changeset, :llm_api_key, encrypted)
         end
-
       else
         # If empty string or nil is provided, don't encrypt but don't add error
         changeset

lib/cadet/jobs/xml_parser.ex

@@ -98,7 +98,8 @@ defmodule Cadet.Updater.XMLParser do
         reading: ~x"//READING/text()" |> transform_by(&process_charlist/1),
         summary_short: ~x"//WEBSUMMARY/text()" |> transform_by(&process_charlist/1),
         summary_long: ~x"./TEXT/text()" |> transform_by(&process_charlist/1),
-        llm_assessment_prompt: ~x"./LLM_ASSESSMENT_PROMPT/text()" |> transform_by(&process_charlist/1),
+        llm_assessment_prompt:
+          ~x"./LLM_ASSESSMENT_PROMPT/text()" |> transform_by(&process_charlist/1),
         password: ~x"//PASSWORD/text()"so |> transform_by(&process_charlist/1)
       )
       |> Map.put(:is_published, false)

lib/cadet_web/admin_views/admin_grading_view.ex

@@ -8,7 +8,7 @@ defmodule CadetWeb.AdminGradingView do
       assessment:
         render_one(assessment, CadetWeb.AdminGradingView, "assessment.json", as: :assessment),
       answers: render_many(answers, CadetWeb.AdminGradingView, "grading_info.json", as: :answer),
-      enable_llm_grading: course.enable_llm_grading,
+      enable_llm_grading: course.enable_llm_grading
     }
   end
 
@@ -22,8 +22,7 @@ defmodule CadetWeb.AdminGradingView do
       number: assessment.number,
       story: assessment.story,
       reading: assessment.reading,
-      llm_assessment_prompt: assessment.llm_assessment_prompt,
-
+      llm_assessment_prompt: assessment.llm_assessment_prompt
     }
   end
 
@@ -157,7 +156,7 @@ defmodule CadetWeb.AdminGradingView do
       team: &extract_team_data(&1.submission.team),
       question: &build_grading_question/1,
       solution: &(&1.question.question["solution"] || ""),
-      grade: &build_grade/1,
+      grade: &build_grade/1
     })
   end
 
@@ -166,8 +165,10 @@ defmodule CadetWeb.AdminGradingView do
   end
 
   defp extract_ai_comments_per_answer(question_id, ai_comments) do
-    matching_comment = ai_comments
-    |> Enum.find(&(&1.question_id == question_id)) # Equivalent to fn comment -> comment.question_id == question_id end
+    matching_comment =
+      ai_comments
+      # Equivalent to fn comment -> comment.question_id == question_id end
+      |> Enum.find(&(&1.question_id == question_id))
 
     case matching_comment do
       nil -> nil

lib/cadet_web/controllers/courses_controller.ex

@@ -60,7 +60,14 @@ defmodule CadetWeb.CoursesController do
       llm_api_key(:body, :string, "OpenAI API key for this course", required: false)
       llm_model(:body, :string, "LLM model to be used for this course", required: false)
       llm_api_url(:body, :string, "LLM API URL to be used for this course", required: false)
-      llm_course_level_prompt(:body, :string, "LLM course level prompt to be used for this course", required: false)
+
+      llm_course_level_prompt(
+        :body,
+        :string,
+        "LLM course level prompt to be used for this course",
+        required: false
+      )
+
       source_chapter(:body, :number, "Default source chapter", required: true)
 
       source_variant(:body, Schema.ref(:SourceVariant), "Default source variant name",
@@ -106,7 +113,11 @@ defmodule CadetWeb.CoursesController do
             llm_api_key(:string, "OpenAI API key for this course", required: false)
             llm_model(:string, "LLM model to be used for this course", required: false)
             llm_api_url(:string, "LLM API URL to be used for this course", required: false)
-            llm_course_level_prompt(:string, "LLM course level prompt to be used for this course", required: false)
+
+            llm_course_level_prompt(:string, "LLM course level prompt to be used for this course",
+              required: false
+            )
+
             source_chapter(:integer, "Source Chapter number from 1 to 4", required: true)
             source_variant(Schema.ref(:SourceVariant), "Source Variant name", required: true)
             module_help_text(:string, "Module help text", required: true)
@@ -125,7 +136,8 @@ defmodule CadetWeb.CoursesController do
             llm_api_key: "sk-1234567890",
             llm_model: "gpt-4",
             llm_api_url: "https://api.openai.com/v1/chat/completions",
-            llm_course_level_prompt: "You are a helpful teaching assistant for an introductory programming course",
+            llm_course_level_prompt:
+              "You are a helpful teaching assistant for an introductory programming course",
             source_chapter: 1,
             source_variant: "default",
             module_help_text: "Help text",