Skip to content

Major rework #14

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

Open
wants to merge 22 commits into
base: devel
Choose a base branch
from
Open

Major rework #14

wants to merge 22 commits into from

Conversation

ppietikainen
Copy link
Member

Ok, this should take care of just about every issue out there. Ready for testing and further
cleanup

Major points:

IEEE citation style (#2)
Copyright page (More examples & maybe put CC logo in there too), can be commented out (#11)
Abstract page (citation style TBD) (#11)
Major rework of packages used, are the font sizes/spaces/etc. still what they should be? (#3)
A lot of cruft removed, and more can still be done (#4)

@ppietikainen ppietikainen requested a review from Copilot June 4, 2025 09:07
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR performs a major template and package rework, switching the thesis class, updating package lists, and modernizing abstract/appendix handling.

  • Switched from a custom dithesis class to the standard report class and added CC licensing and abstract commands.
  • Updated the TeX Live package list (added appendix, csquotes, doclicense; removed lastpage).
  • Removed several custom formatting blocks (list labels, environment wrappers) and introduced new \printabstract, \printtiivistelma, and appendix handling.

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
texlive-basic-packages.txt Added/removed packages (appendix, csquotes, doclicense; dropped lastpage)
main.tex Switched document class; added CC license, abstract/tiivistelma macros
Gallery/protocolheader.tex Added example TCP header figure using bytefield
Chapters/tiivistelma.tex Stripped out tiivistelma environment wrapper
Chapters/implementation.tex Removed custom list‐label length adjustments
Chapters/experiments.tex Tweaked citation syntax for page numbers
Chapters/appendices.tex Reformatted appendix metadata and keywords
Chapters/abstract.tex Stripped out abstract environment wrapper
Comments suppressed due to low confidence (6)

Chapters/tiivistelma.tex:1

  • Removal of the tiivistelma environment wrapper (\begin{tiivistelma}/\end{tiivistelma}) means the summary will no longer be formatted as intended. Reintroduce the environment or ensure the new \printtiivistelma macro is applied consistently.
Näissä ohjeissa opastetaan valmistumisvaiheessa olevaa opiskelijaa diplomityön tekemisessä. Ohjeissa selvitetään työn asema diplomi-insinööritutkinnossa, kerrotaan toimenpiteet, joihin työn tekijän on ryhdyttävä työn eri vaiheissa, sekä määritellään yksityiskohtaisesti diplomityön kirjallinen rakenne. Ohje on muotoiltu noudattaen näitä kirjoitusohjeita ja toimii siten esimerkkinä, miltä diplomityön tulisi näyttää.

Chapters/abstract.tex:1

  • The abstract content is no longer enclosed in the abstract environment, which may break formatting. Restore the environment or confirm that the \printabstract command applies proper styling.
This document provides general guidance for a degree student of Computer Science and Engineering in preparing his/her master's thesis. This guide defines the role of a thesis in the M.Sc. Degree, presents the actions to be taken in different phases of the thesis procedure and introduces the way that master’s thesis is written. The document has been formatted based on these guidelines to serve as an example how the thesis should look like.

texlive-basic-packages.txt:16

  • Removed the lastpage package despite its comment indicating it's still useful. If page-of-N footers are needed, consider re-adding it or updating the comment to reflect the removal.
-lastpage              # Enables 'Page X of Y' in headers/footers; still useful.

main.tex:12

  • Switching to the standard report class removes all custom thesis macros (e.g., \firstname, \maketitlepage, \printabstract) provided by dithesis. Consider either retaining the custom class or updating these macro usages to match the report class or a suitable package.
\documentclass[a4paper, 12pt, titlepage]{report}

Chapters/appendices.tex:23

  • The macro \lskip is undefined (likely meant \vskip or similar) and the closing brace may not match an opening brace. Update the spacing command and ensure all braces are balanced.
\lskip \textbf{Keywords: Ubiquitous computing, mobile computing, task-based computing, context-awareness, QoS.}

Gallery/protocolheader.tex:8

  • Using \widthof requires the calc package (or similar) to be loaded. Ensure you include \usepackage{calc} in main.tex to prevent compilation errors.
\begin{bytefield}[bitwidth=\textwidth/32, bitheight=\widthof{~Data offset~}]{32}

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.

1 participant