Skip to content

[WIP] Refactor QuesPM code for OOP and efficiency#1

Merged
TETRAWasTaken merged 1 commit into
mainfrom
copilot/refactor-quespm-to-python
Oct 10, 2025
Merged

[WIP] Refactor QuesPM code for OOP and efficiency#1
TETRAWasTaken merged 1 commit into
mainfrom
copilot/refactor-quespm-to-python

Conversation

Copilot AI commented Oct 10, 2025

Copy link
Copy Markdown
Contributor
  • Analyze the existing QuesPMupdate.ipynb notebook code structure
  • Design OOP architecture with appropriate classes
  • Create QuestionPaperMaker.py with the following classes:
    • QuestionLoader class (handles loading questions from files)
    • QuestionSelector class (handles random selection of questions)
    • PDFGenerator class (handles PDF creation and formatting)
    • QuestionPaperMaker class (main orchestrator)
  • Refactor code to be cleaner and more efficient:
    • Remove code duplication (question selection logic repeated 3 times)
    • Improve file handling with context managers
    • Make file paths configurable instead of hardcoded
    • Better separation of concerns
    • Add error handling
  • Test the new Python file
  • Update README.md with usage instructions for the new Python file
Original prompt

On QuesPMupdate.ipynb, Rewrite this code to be more clean and efficient, use OOPS concept and make it a python file rather than notebook


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@TETRAWasTaken TETRAWasTaken marked this pull request as ready for review October 10, 2025 05:01
@TETRAWasTaken TETRAWasTaken merged commit 1becc72 into main Oct 10, 2025
1 of 2 checks passed
@TETRAWasTaken TETRAWasTaken deleted the copilot/refactor-quespm-to-python branch October 10, 2025 05:03
Copilot AI requested a review from TETRAWasTaken October 10, 2025 05:04
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.

2 participants