Final revision before thesis release - #7
Merged
Merged
Conversation
Took 2 minutes Took 3 minutes Took 1 minute
Took 44 minutes Took 13 seconds
Took 1 hour 36 minutes
Took 1 hour 56 minutes
Took 1 hour 50 minutes
Took 50 minutes
Took 13 minutes
Took 11 minutes
Took 5 minutes
Took 18 minutes
Took 3 hours 19 minutes
Took 8 seconds
Took 11 minutes
Took 35 minutes Took 4 minutes
Took 41 seconds
Took 20 hours 21 minutes Took 16 seconds
Took 6 seconds
Took 10 seconds
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 55dd160f0c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
MKrumnikl
reviewed
Apr 12, 2026
MKrumnikl
left a comment
MKrumnikl
left a comment
Collaborator
There was a problem hiding this comment.
review k "teoretické" části
Firestone82
changed the title
Firestone82
pushed a commit
that referenced
this pull request
Apr 12, 2026
- Fix typo 'můž' → 'muž' in LLM history section - Fix GPT-3 parameter formatting: '(2020, 175 miliard parametrů)' → '(2020 s 175 miliardami parametrů)' - Replace vague 'geometrically close' with cosine similarity explanation in history and concepts sections - Add BPE citation (Sennrich et al. 2016) in tokenization section - Add 'original transformer' citation (Vaswani et al.) in positional encoding section - Add DOLOS and MOSS citations in Kelvin architecture section - Add global comment feature description in Kelvin purpose section - Add TPU mention to hardware compute cost section - Add tokens/second metric to GPU inference description - Add CPU RAM offloading (hybrid GPU/CPU) explanation to compute cost section - Improve prompt injection motivation: explain student incentive to game automated grading - Add real-world cost example for 500-student UPR course in cloud deployment section - Add quantization (Q4_K_M) description for on-premise model choice - Add 'analýza a hodnocení kódu' to Czech and English keywords - Add bibliography entries for DOLOS, MOSS, and BPE (Sennrich et al.) https://claude.ai/code/session_01NZ2nEj9kSNX1iBDVhGw3rN
Firestone82
pushed a commit
that referenced
this pull request
Apr 12, 2026
- Add global comment mention in Kelvin features (2-1) - Emphasize Vue/Svelte deprecation status (2-2) - Add DOLOS and MOSS citations for plagiarism tools (2-2) - Fix 'geometricky blízko' to use cosine similarity metric (3-1-2, 3-1-3) - Fix 'můž' typo to 'muž' (3-1-2) - Fix parenthetical format: "(2020 s 175 miliardami parametrů)" (3-1-2) - Add BPE citation (Sennrich et al.) (3-1-3) - Rewrite token/word explanation with OpenAI tokenizer reference (3-1-3) - Add citation for original Transformer (3-1-3) - Add training compute citation (3-2-2) - Move figure reference earlier in training section (3-2-2) - Shorten static analyzers section to reduce digression (3-3-1) - Add TPU mention, tokens/s metric, RAM offloading in compute cost (3-4-2) - Add student motivation for prompt injection section (3-4-5) - Rename section "Rizika a omezení" to "Omezení" (3-LLM) - Shorten context window limitation section, reference prior parts (4-1-4) - Add real-world cost example (UPR course) and future work note (4-2-1) - Add RAM offloading argument for on-premise deployment (4-2-2) - Mention quantization in model choice section (4-5-3) - Add error logging detail in job processing (5-1-3) - Add 'hodnocení' to Czech abstract in main.tex - Add bibliography entries: DOLOS, MOSS, BPE, OpenAI tokenizer - Replace excessive em-dashes with conjunctions in prose https://claude.ai/code/session_013DdqQKYpis217tZVSrB6nn
Firestone82
temporarily deployed
to
github-pages
April 17, 2026 13:43 — with 
GitHub Actions
Inactive
GitHub Actions
Inactive
Firestone82
temporarily deployed
to
github-pages
April 17, 2026 13:53 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Firestone82
temporarily deployed
to
github-pages
April 21, 2026 18:51 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Firestone82
temporarily deployed
to
github-pages
April 21, 2026 19:18 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Firestone82
temporarily deployed
to
github-pages
April 21, 2026 20:02 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
Firestone82
temporarily deployed
to
github-pages
April 29, 2026 17:46 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
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
3 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.
No description provided.