Refactor/latex docker downsize api#8
Merged
macite merged 32 commits intomacite:chore/update-gemsfrom Apr 1, 2025
Merged
Conversation
* solves current permission issues that prevented the working folder from being rm -rf'd from the api * this ensures that input.pdf and input.log still exist when the script successfully completes
…atex-docker-downsize-api
macite
pushed a commit
that referenced
this pull request
Oct 15, 2025
* chore: create CreateMarkingSessions migration and MarkingSession model (#4) * chore: create CreateSessionActivities migration and SessionActivity model, migration ran successfully (#5) * feature/session-tracker-service (#6) - implement session tracker service with integration test - amend marking session update_session_details method to initalize date time before assigning * feat: implement service track in endpoints (#7) - implement test * feat: add analytics endpoints for marking sessions, session activities and and aggregated analytics (#8) * chore: fix robocop * chore: migrate marking sessions schema * chore: revert schema * chore: bump session migrations * chore: migrate session activities * chore: fix rubocops * refactor: rename marker to user * fix: get task before logging action * chore: record activity after action is complete * chore: whitelist valid actions * test: correct activity action type * refactor: move statistics to entity models * wip * fix: add unit id to route * refactor: end session once there hasnt been any new activity * feat: download tutor times summary * chore: serialise date filters * feat: seed marking sessions * chore: aggregate activity actions count * refactor: expose endpoint to retrieve all marking sessions for a unit * fix: ensure end date is inclusive * chore: add permission for tutor times * refactor: fix timezone issues * refactor: clean up marking sessions api * refactor: clean up session tracker * refactor: add unit tests * chore: get last created activity * test: test all routes recording assessment activities * refacto: remove duration minutes from schema - calculate it based on start/end time * chore: update default ip * chore: remove todo * chore: set end_time to default marking session * chore: test permissions for tutor summary csv * chore: use get tutor times permission * test: ensure correct number of sessions are returned * chore: remove newline * chore: remove newline * feat: split sessions for allocated for tutorial times * test: fix tutorial time * test: fix wrong unit * feat: option to exclude sessions during tutorials * refactor: remove old code * test: ensure ignore tutorial sessions has different duration * fix: add missing parameter * chore: remove todo * chore: avoid redundancy --------- Co-authored-by: Joseph K Saji <jjbrothersdelhi@hotmail.com> Co-authored-by: Josh Talev <139172317+jtalev@users.noreply.github.com> Co-authored-by: Martin Dolores <77220115+martindolores@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR moves Latex install to a separate contaner, and uses Docker to execute the PDF generation on that container. In doing this, we reduce the build time and size of the docker images.