Add linear algebra by thejoeyluu · Pull Request #76 · ChicoState/cheat-sheet

thejoeyluu · 2026-05-06T05:43:06Z

level 1 and 2 linear algebra is now in play

…ayout

Copilot

Pull request overview

This PR expands the cheat-sheet formula catalog by adding Linear Algebra I/II content, and also introduces an orientation layout option that flows from the frontend layout controls through backend LaTeX generation/normalization.

Changes:

Add new Linear Algebra I and II formula datasets and register them in the formula catalog.
Add orientation as a persisted layout option in the frontend hook/UI and backend API/model.
Update LaTeX generation/normalization to include paper/orientation options, and adjust backend/frontend tests accordingly.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
frontend/src/hooks/latex.test.jsx	Extends hook tests to cover the new `orientation` default and custom value.
frontend/src/hooks/latex.js	Adds `orientation` state, persistence (localStorage), and API payload wiring for compile/normalize/generate calls.
frontend/src/components/CreateCheatSheet.jsx	Adds an Orientation selector to layout controls and plumbs `orientation` state into the UI.
backend/api/views.py	Accepts/validates `orientation` in generate/compile endpoints and returns it in normalize-only responses.
backend/api/tests.py	Updates/extends tests for new layout comment and updated LaTeX header expectations.
backend/api/models.py	Adds `orientation` field to `CheatSheet`.
backend/api/migrations/0008_cheatsheet_orientation.py	DB migration to add the `orientation` column with a default.
backend/api/latex_utils.py	Threads `orientation` through LaTeX header/layout-comment generation and normalization logic.
backend/api/formula_data/linear_algebra_i.py	Adds Linear Algebra I formulas.
backend/api/formula_data/linear_algebra_ii.py	Adds Linear Algebra II formulas.
backend/api/formula_data/init.py	Registers Linear Algebra I/II into available classes and formula data mappings.

Comments suppressed due to low confidence (1)

backend/api/views.py:376

When compiling via cheat_sheet_id, the view overwrites columns/font_size/margins/spacing from the saved CheatSheet but does not overwrite orientation. This means a saved landscape sheet will still compile as portrait unless the client redundantly sends orientation. Set orientation = cheatsheet.orientation (and consider re-running validate_layout_params after loading the model values) so the stored layout is consistently respected.

    if cheat_sheet_id:
        cheatsheet = get_object_or_404(CheatSheet, pk=cheat_sheet_id, user=request.user)
        columns = cheatsheet.columns
        font_size = cheatsheet.font_size
        margins = cheatsheet.margins
        spacing = cheatsheet.spacing
        content = cheatsheet.build_full_latex()

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-05-06T18:28:51Z

    setSpacing,
    margins,
    setMargins,
+    orientation,
+    setOrientation,
    pdfBlob,


orientation is now part of the layout state, but it is not included in the onSave(...) payloads (both the autosave snapshot and the explicit Save handler). As a result, the new orientation setting won’t persist across reloads/restores even though the UI exposes it. Include orientation (and, if applicable, in compileSnapshot) wherever layout settings are saved.

    columns = models.IntegerField(default=4)
    margins = models.CharField(max_length=20, default="0.15in")
    font_size = models.CharField(max_length=10, default="9pt")
    spacing = models.CharField(max_length=10, default="small")
+    orientation = models.CharField(max_length=20, default="portrait")
    # Stores selected formulas with user-defined order: [{"class": "...", "category": "...", "name": "..."}]


+    # 1. Force the PDF driver to rotate by passing landscape and letterpaper to the document class
+    doc_options = f"{doc_class_size},fleqn,letterpaper"
+    if orientation == "landscape":
+        doc_options += ",landscape"
+
+    # 2. Also pass them to the geometry package
+    geometry_options = f"letterpaper,margin={margins}"
+    if orientation == "landscape":
+        geometry_options += ",landscape"


Copilot · 2026-05-06T18:28:51Z

 @api_view(["POST"])
 def generate_sheet(request):
    """
-    POST /api/generate-sheet/
-    Accepts { "formulas": [...], "columns": 4, "font_size": "9pt", "margins": "0.15in", "spacing": "small" }
+    Accepts { "formulas": [...], "columns": 4, "font_size": "9pt", "margins": "0.15in", "spacing": "small", "orientation": "portrait" }
    Each formula: { "class": "ALGEBRA I", "category": "Linear Equations", "name": "Slope Formula" }
    Or for special classes (like UNIT CIRCLE): { "class": "UNIT CIRCLE", "name": "Unit Circle (Key Angles)" }
    Returns { "tex_code": "..." }


PR title/description focus on adding Linear Algebra content, but this PR also introduces a new layout feature (orientation) and changes LaTeX paper options (letterpaper) across compile/generate flows. Please update the PR description/title to reflect this additional user-facing behavior, or split the layout work into a separate PR for clearer review/release notes.

Davictory2003 · 2026-05-06T18:31:49Z

@copilot apply changes based on the comments in this thread

…, and landscape test Agent-Logs-Url: https://github.com/ChicoState/cheat-sheet/sessions/feaa4cd4-4709-4685-85f7-54a23f49946e Co-authored-by: Davictory2003 <68972845+Davictory2003@users.noreply.github.com>

Copilot · 2026-05-06T18:38:29Z

@copilot apply changes based on the comments in this thread

Applied all four changes from the review thread in commit 8a449ba:

orientation in onSave payloads — Added orientation to both the autosave snapshot (including compileSnapshot) and the explicit handleSave payload in CreateCheatSheet.jsx, and added orientation to the useEffect dependency array.
orientation in serializer — Added "orientation" to CheatSheetSerializer.Meta.fields so clients can read and write the value via the API.
build_full_latex() uses orientation — Updated models.py to build doc_options and geometry_options with letterpaper + landscape when orientation is landscape, making it consistent with build_dynamic_header().
Landscape test — Added test_build_dynamic_header_landscape_includes_landscape_options in tests.py that verifies \documentclass[9pt,fleqn,letterpaper,landscape]{extarticle} and letterpaper,margin=0.15in,landscape in the generated header.

All 99 backend and 53 frontend tests pass.

thejoeyluu added 8 commits May 5, 2026 17:17

Save landscape orientation backend logic

fc671ea

Resolve merge conflicts and keep incoming changes

6e9624c

feat: add landscape orientation support and fix layout regex

664489c

test: update backend tests to match new orientation and letterpaper l…

de70d22

…ayout

feat: add orientation field to CheatSheet model

7a7af9e

fix: resolve syntax errors in backend tests

5948d56

test: update remaining generate sheet tests for letterpaper

096616a

feat: add linear algebra I and II formulas and included videos

8b349dd

Davictory2003 requested a review from Copilot May 6, 2026 18:23

Copilot started reviewing on behalf of Davictory2003 May 6, 2026 18:23 View session

impxcts approved these changes May 6, 2026

View reviewed changes

Copilot AI reviewed May 6, 2026

View reviewed changes

Copilot started work on behalf of Davictory2003 May 6, 2026 18:31 View session

fix: add orientation to onSave payloads, serializer, build_full_latex…

8a449ba

…, and landscape test Agent-Logs-Url: https://github.com/ChicoState/cheat-sheet/sessions/feaa4cd4-4709-4685-85f7-54a23f49946e Co-authored-by: Davictory2003 <68972845+Davictory2003@users.noreply.github.com>

Copilot finished work on behalf of Davictory2003 May 6, 2026 18:39

Copilot AI requested a review from Davictory2003 May 6, 2026 18:39

Davictory2003 approved these changes May 6, 2026

View reviewed changes

Davictory2003 merged commit c39a5c1 into main May 6, 2026
5 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add linear algebra#76

Add linear algebra#76
Davictory2003 merged 9 commits intomainfrom
add-linear-algebra

thejoeyluu commented May 6, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI May 6, 2026

Uh oh!

Copilot AI May 6, 2026

Uh oh!

Davictory2003 commented May 6, 2026

Uh oh!

Copilot AI commented May 6, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

thejoeyluu commented May 6, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI May 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI May 6, 2026

Choose a reason for hiding this comment

Uh oh!

Davictory2003 commented May 6, 2026

Uh oh!

Copilot AI commented May 6, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants