Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: MCQ Generation Page Break and Missing Options #110

Closed
wants to merge 1 commit into from

Conversation

Naman-B-Parlecha
Copy link
Contributor

@Naman-B-Parlecha Naman-B-Parlecha commented Jan 5, 2025

This PR fixes the issue where the MCQ generation process caused the page to break (#109). It also ensures all generated MCQs display their options correctly.

Note: In order to generate this PR on a local GPU, make use of the changes introduced in PR #107.

Closes #109

Before Fix
https://github.com/user-attachments/assets/329bab79-9f9e-4cb1-a1e2-aeda46ee0712

After Fix
Screencast from 2025-01-07 11-59-44.webm

@manthanabc
Copy link

I believe the question statement must be changed also to question_statement like in #86, other wise it does'nt show correct question statement.

@Naman-B-Parlecha
Copy link
Contributor Author

Naman-B-Parlecha commented Jan 5, 2025

@manthanabc i believe we are getting the output as before, hence making a small change on Frontend would work in this case

@Aditya062003
Copy link
Contributor

@Naman-B-Parlecha pls ensure that everything is working now after these changes. If possible, share a webcast with the working solution.

@Naman-B-Parlecha
Copy link
Contributor Author

Naman-B-Parlecha commented Jan 7, 2025

Screencast.from.2025-01-07.11-59-44.webm

@Aditya062003 there is no breaking, after the fix

@Aditya062003
Copy link
Contributor

I think this issue has already been fixed with the PDF generation, so we don't need a separate PR or issue for this. Closing for now.

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.

Bug: MCQ Generation Causing Page Break and Missing Options
3 participants