Merge pull request #40 from beautiful-code/course_learn_suggest_expectations_refactor

app/ui/course_learn_suggest_expectations.py

@@ -62,10 +62,10 @@ def course_suggester_main():
     default_content = {
         "course": "SQL",
         "module": "Query Optimization Techniques",
-        "tasks": [
-            "Watch this video https://youtu.be/BHwzDmr6d7s?si=sfFYnd73y9w0EjGB to understand SQL execution order and some optimization techniques.",
-            "Watch this video https://youtu.be/FoznjTU929c?si=6M3xUIUwAxE6EbKS to understand SQL explain command usage.",
-            "Go over these articles https://intellipaat.com/blog/sql-optimization-techniques/ & https://www.thoughtspot.com/data-trends/data-modeling/optimizing-sql-queries  to understand various query optimization techniques."
+        "concepts": [
+            "SQL execution order and some optimization techniques.",
+            "SQL explain command usage.",
+            "Various query optimization techniques."
         ],
         "expectation": "Should understand the importance of indexing in query optimization.",
         "check_question": "How does indexing improve query performance?"
@@ -80,8 +80,8 @@ def course_suggester_main():
                                help="Enter the name of the course", value=st.session_state_inputs["course"])
         module = st.text_input("##### Module Name", key="module",
                                help="Enter the name of the module", value=st.session_state_inputs["module"])
-        tasks = st.text_area("##### Tasks", key="tasks", value="\n".join(
-            st.session_state_inputs["tasks"]), help="Enter the tasks to be covered in the module")
+        concepts = st.text_area("##### Concepts", key="concepts", value="\n".join(
+            st.session_state_inputs["concepts"]), help="Enter the concepts to be covered in the module")
         existing_expectation = st.text_input(
             "##### Existing Expectation", key="existing_expectation", value=st.session_state_inputs["expectation"])
         existing_check_question = st.text_input(
@@ -90,7 +90,7 @@ def course_suggester_main():
         suggester = SuggestExpectations()
         inputs = {
             "course": course, "module": module,
-            "tasks": tasks.split('\n'),
+            "concepts": concepts.split('\n'),
             "existing_expectations": [
                 Expectation(
                     expectation=existing_expectation,
@@ -140,7 +140,7 @@ def course_suggester_main():
                             feedback_inputs = {
                                 "expectation": response["expectation"], "check_question": response["check_question"], "request": feedback,
                                 "course": course, "module": module,
-                                "tasks": tasks.split('\n'),
+                                "concepts": concepts.split('\n'),
                             }
                             suggestions = rework.kickoff(
                                 inputs=feedback_inputs)

app/workflows/courses/expectation_revision.py

@@ -9,7 +9,7 @@ import os
 class Inputs(BaseModel):
     course: str
     module: str
-    tasks: List[str]
+    concepts: List[str]
     expectation: str
     check_question: str
     request: str
@@ -26,7 +26,7 @@ class ExpectationRevision:
         self.request = inputs["request"]
         self.course = inputs["course"]
         self.module = inputs["module"]
-        self.tasks =  inputs["tasks"]
+        self.concepts =  inputs["concepts"]
         llm_response = self._get_suggestion()
         return {
             "run_id": self.run_id,
@@ -51,7 +51,7 @@ class ExpectationRevision:
         with callbacks.collect_runs() as cb:
             response = chain.invoke({
                 "learning_outcome": self.learning_outcome, "check_question": self.check_question, "request": self.request,
-                "course": self.course, "module": self.module, "tasks": "* " + ("\n* ".join(self.tasks)),
+                "course": self.course, "module": self.module, "concepts": "* " + ("\n* ".join(self.concepts)),
                 "format_instructions": parser.get_format_instructions()
               })
             self.run_id = cb.traced_runs[0].id

app/workflows/courses/suggest_check_question.py

@@ -10,7 +10,7 @@ import os
 class Inputs(BaseModel):
     course: str
     module: str
-    tasks: List[str]
+    concepts: List[str]
     expectation: str
 
 
@@ -30,7 +30,7 @@ class SuggestCheckQuestion:
         self.course = inputs["course"]
         self.module = inputs["module"]
         self.learning_outcome = inputs["expectation"]
-        self.tasks = inputs["tasks"]
+        self.concepts = inputs["concepts"]
         llm_response = self._get_check_quesiton()
         return {
             "run_id": self.run_id,
@@ -44,7 +44,7 @@ class SuggestCheckQuestion:
 
         with callbacks.collect_runs() as cb:
             llm_response = chain.invoke({
-                "course": self.course, "module": self.module, "tasks": "* " + ("\n* ".join(self.tasks)),
+                "course": self.course, "module": self.module, "concepts": "* " + ("\n* ".join(self.concepts)),
                 "format_instructions": parser.get_format_instructions(),
                 "learning_outcome": self.learning_outcome
             })

app/workflows/courses/suggest_expectations.py

@@ -27,7 +27,7 @@ class Response(BaseModel):
 class Inputs(BaseModel):
     course: str
     module: str
-    tasks: List[str]
+    concepts: List[str]
     existing_expectations: Optional[List[Expectation]]
 
 
@@ -36,7 +36,7 @@ class SuggestExpectations:
         self.course = inputs["course"]
         self.module = inputs["module"]
         self.existing_expectations = inputs["existing_expectations"]
-        self.tasks = inputs["tasks"]
+        self.concepts = inputs["concepts"]
         llm_response = self._get_suggestions()
         return {
           "run_id": self.run_id,
@@ -63,7 +63,7 @@ class SuggestExpectations:
 
         with callbacks.collect_runs() as cb:
             llm_response = chain.invoke({
-                "course": self.course, "module": self.module, "tasks": "* " + ("\n* ".join(self.tasks)),
+                "course": self.course, "module": self.module, "concepts": "* " + ("\n* ".join(self.concepts)),
                 "format_instructions": parser.get_format_instructions(),
                 "existing_expectations": existing_expectations_str
             })
@@ -73,7 +73,7 @@ class SuggestExpectations:
 
     def _build_chain(self):
         parser = JsonOutputParser(pydantic_object=Expectations)
-        prompt = hub.pull("course_learn_suggest_expectations_from_learner")
+        prompt = hub.pull("course_learn_suggest_expectations")
         llm = ChatOpenAI(model=os.environ['OPENAI_MODEL'], temperature=0.2)
         chain = (prompt | llm | parser).with_config({
             "tags": ["course_learn", "suggest_expectations"], "run_name": "Suggest Module Expectations",
@@ -93,7 +93,7 @@ class SuggestExpectations:
 # response = suggester.kickoff(inputs={
 #     "course": "SQL",
 #     "module": "Query Optimization Techniques",
-#     "tasks": [
+#     "concepts": [
 #         "Watch this video https://youtu.be/BHwzDmr6d7s?si=sfFYnd73y9w0EjGB to understand SQL execution order and some optimization techniques.",
 #         "Watch this video https://youtu.be/FoznjTU929c?si=6M3xUIUwAxE6EbKS to understand SQL explain command usage.",
 #         "Go over these articles https://intellipaat.com/blog/sql-optimization-techniques/ & https://www.thoughtspot.com/data-trends/data-modeling/optimizing-sql-queries  to understand various query optimization techniques."

ui_main.py

@@ -23,7 +23,7 @@ def common_sidebar():
     st.sidebar.markdown("### Pages")
     st.sidebar.button("Home", on_click=lambda: router.redirect(*router.build("show_main_page")))
     st.sidebar.button("TIL Feedback", on_click=lambda: router.redirect(*router.build("feedback_main_wrapper")))
-    st.sidebar.button("Course Suggester", on_click=lambda: router.redirect(*router.build("course_suggester_main_wrapper")))
+    st.sidebar.button("Course Suggester", on_click=lambda: router.redirect(*router.build("course_builder_main_wrapper")))
 
 
 
@@ -57,7 +57,7 @@ def show_main_page():
 
     card_info = [
     {"title": "TIL Feedback", "description": "Provide your valuable feedback.", "key": "feedback_main_wrapper", "image": "📝"},
-    {"title": "Course Suggester", "description": "Get suggestions for courses", "key": "course_suggester_main_wrapper", "image": "📚"},
+    {"title": "Course Builder", "description": "Get suggestions for building courses", "key": "course_builder_main_wrapper", "image": "📚"},
 
 ]
 
@@ -139,7 +139,7 @@ def feedback_main_wrapper():
     common_sidebar()
     feedback_main()
 
-def course_suggester_main_wrapper():
+def course_builder_main_wrapper():
     common_sidebar()
     course_suggester_main()
 
@@ -149,7 +149,7 @@ router = sr.StreamlitRouter()
 
 router.register(show_main_page, '/')
 router.register(feedback_main_wrapper, '/feedback', methods=['GET'])
-router.register(course_suggester_main_wrapper, '/course_suggester')
+router.register(course_builder_main_wrapper, '/course_builder')
 
 
 router.serve()