Skip to content

Conversation

melisasvr
Copy link

This PR enhances the syllabus generator feature by:

  • Implementing a hybrid model in tools.py with sequential chaining (Course Information → Learning Outcomes → Course Content → Assessment Criteria → Course Schedule) and parallel processing (Learning Resources, Policies & Procedures).
  • Improving prompt templates in tools.py for better clarity, consistency, and structured content generation.
  • Updating schemas.py to include unit_time, unit_time_value, and start_date fields.
  • Modifying core.py to pass new fields to the syllabus generator.
  • Expanding test_core.py with tests for new sections (e.g., learning_outcomes, assessment_criteria) and dependencies.

Notes:

  • Tests in test_core.py may require a Google API key for GoogleGenerativeAI to run successfully. Please advise on how to configure this (e.g., .env setup).
  • Awaiting feedback on whether sandbox or Develop is the correct base branch.

@stevenrayhinojosa-gmail-com
Copy link
Contributor

Thank you for contributing it ran well, no issues.

@AaronSosaRamos AaronSosaRamos self-requested a review March 9, 2025 20:15
Copy link
Contributor

@AaronSosaRamos AaronSosaRamos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good work! Please, solve the given comments. Thank you!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good approach. Although I suggest to use Pydantic Schemas to guarantee deterministic results with a given structure. This enhances responses based in few-shot prompting.

@melisasvr melisasvr force-pushed the update-tools-folder branch from 4267b47 to 9ca7f60 Compare March 13, 2025 22:22
@AaronSosaRamos
Copy link
Contributor

Hey @melisasvr, don't forget to pull the latest changes from Develop to your branch. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants