Dev by ThorstenSuckow · Pull Request #305 · ThorstenSuckow/helios

ThorstenSuckow · 2026-04-26T21:28:35Z

No description provided.

chore(docs): initial commit for master-thesis

Ecs rendering

Copilot

Pull request overview

Adds a new LaTeX “helios_master” documentation bundle under docs/latex/helios_master, including master documents (DE/EN), bibliography sources, and vendored LNCS class/BibTeX style files to support publication-style builds.

Changes:

Added LNCS llncs.cls and splncs04.bst (including DOI handling) plus upstream history.txt.
Added German and English master TeX entrypoints configured with biblatex and a shared .bib database.
Added initial German content structure (index, appendix chapter scaffolding, and a concurrency-paper appendix section).

Reviewed changes

Copilot reviewed 19 out of 20 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
docs/latex/helios_master/splncs04.bst	Adds LNCS BibTeX style (with DOI formatting).
docs/latex/helios_master/llncs.cls	Adds LNCS LaTeX class file (vendored).
docs/latex/helios_master/index_en.tex	Adds English index include file (currently empty).
docs/latex/helios_master/index_de.tex	Adds German index include file (`content/de/index`).
docs/latex/helios_master/history.txt	Adds LNCS class/style version history.
docs/latex/helios_master/helios_master_en.tex	Adds English master document (KOMA-Script, `biblatex`, listings, appendix config).
docs/latex/helios_master/helios_master_de.tex	Adds German master document (KOMA-Script, `biblatex`, listings, appendix config).
docs/latex/helios_master/helios_master.bib	Adds bibliography entries used by the master documents.
docs/latex/helios_master/content/en/2. introduction.tex	Adds initial English section stub.
docs/latex/helios_master/content/en/1. abstract.tex	Adds initial English abstract stub.
docs/latex/helios_master/content/de/testsysteme.tex	Adds unnumbered “Testsysteme” chapter stub.
docs/latex/helios_master/content/de/notation.tex	Adds unnumbered notation chapter stub.
docs/latex/helios_master/content/de/konventionen.tex	Adds conventions text snippet (not yet wired into index).
docs/latex/helios_master/content/de/index.tex	Adds German content include list (currently mainly appendix include).
docs/latex/helios_master/content/de/glossar/index.tex	Adds glossary chapter stub.
docs/latex/helios_master/content/de/einleitung/index.tex	Adds German introduction chapter stub.
docs/latex/helios_master/content/de/danksagung.tex	Adds unnumbered acknowledgements chapter stub.
docs/latex/helios_master/content/de/appendix/concurrency_paper/index.tex	Adds appendix chapter wrapper + include for the concurrency section.
docs/latex/helios_master/content/de/appendix/concurrency_paper/einleitung.tex	Adds initial German write-up for concurrency/ECS appendix content.
docs/latex/helios_master/content/de/abstract/index.tex	Adds German abstract stub.

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

Copilot · 2026-04-26T21:31:59Z

+
+\section{Eine erste Annäherung an die inhärente Nebenläufigkeit von ECS-Programmen}
+
+Bevor wir weiter auf die in der Arbeit vorgestellten Formalisieren eingehen, wollen wir zunächst die ``ìnhärente Nebenläufigkeit`` verstehen, die die Autoren einführend anmerken, aber nicht weiter ausführen.


The quoted term "ìnhärente" contains an unexpected accented character and appears to be a typo for "inhärente". Correcting it avoids confusing readers and prevents potential copy/paste/search issues.

Suggested change

Bevor wir weiter auf die in der Arbeit vorgestellten Formalisieren eingehen, wollen wir zunächst die ``ìnhärente Nebenläufigkeit`` verstehen, die die Autoren einführend anmerken, aber nicht weiter ausführen.

Bevor wir weiter auf die in der Arbeit vorgestellten Formalisieren eingehen, wollen wir zunächst die ``inhärente Nebenläufigkeit`` verstehen, die die Autoren einführend anmerken, aber nicht weiter ausführen.

Copilot · 2026-04-26T21:32:00Z

+            to develop a \textit{Geometry Wars} clone in the distance learning program in Computer Science,
+            Trier University of Applied Sciences, Department of Computer Science.
+            English version of the original German report
+            \textit{``helios: Konzeption und prototypische Umsetzung eines C++ Game Frameworks``, DOI:  \href{https://doi.org/10.5281/zenodo.xxxxx}{10.5281/zenodo.xxxxxx}


The DOI placeholder is inconsistent between the link target and the displayed DOI (different number of 'x' characters). This can easily lead to a wrong DOI being published; make the DOI string consistent in both places.

Suggested change

\textit{``helios: Konzeption und prototypische Umsetzung eines C++ Game Frameworks``, DOI: \href{https://doi.org/10.5281/zenodo.xxxxx}{10.5281/zenodo.xxxxxx}

\textit{``helios: Konzeption und prototypische Umsetzung eines C++ Game Frameworks``, DOI: \href{https://doi.org/10.5281/zenodo.xxxxxx}{10.5281/zenodo.xxxxxx}

Copilot · 2026-04-26T21:32:00Z

+%\begin{appendices}
+%    \input{content/de/glossar/index}
+    \input{content/de/appendix/concurrency_paper/index}
+%\end{appendices}


The section is labeled "Anhang", but the appendices environment is commented out while an appendix chapter is still included. If this content is intended to be in the appendix, uncomment the environment (or otherwise adjust the structure) so numbering/TOC formatting matches the intent.

Suggested change

%\begin{appendices}

% \input{content/de/glossar/index}

\input{content/de/appendix/concurrency_paper/index}

%\end{appendices}

\begin{appendices}

% \input{content/de/glossar/index}

\input{content/de/appendix/concurrency_paper/index}

\end{appendices}

Copilot · 2026-04-26T21:32:00Z

+% ----- Anhang
+\usepackage[titletoc,title,page,header]{appendix}
+\renewcommand{\appendixname}{Anhang}
+\renewcommand{\appendixtocname}{Anhänge}
+\renewcommand{\appendixpagename}{Anhänge}


In the English master document, the appendix-related names are overridden to German ("Anhang/Anhänge"). If this file is intended as the English version, consider using the English defaults (or conditionally overriding based on language) so headings/TOC entries are localized correctly.

Suggested change

% ----- Anhang

\usepackage[titletoc,title,page,header]{appendix}

\renewcommand{\appendixname}{Anhang}

\renewcommand{\appendixtocname}{Anhänge}

\renewcommand{\appendixpagename}{Anhänge}

% ----- Appendix

\usepackage[titletoc,title,page,header]{appendix}

% Use the English names provided by babel for appendix headings/TOC entries.

Copilot · 2026-04-26T21:32:00Z

+    Zur Identifikation von Komponenten steht die Menge $I$ zur Verfügung, wobei $i \in I$ ein Label zur Identifizierung einer Komponente darstellt, realisiert über die Abbildung
+    \[
+        \tau: I \rightarrow K
+    \]Í


The line ending the displayed equation contains a stray non-ASCII character ("Í") right after \]. This will likely break compilation depending on engine/encoding; remove the character so the math environment ends cleanly.

Suggested change

\]Í

\]

Copilot · 2026-04-26T21:32:01Z

+Darüber hinaus sorgt der dem Paradigma innewohnende datenorientierte Ansatz durch eine flache Datenhierarchie für eine stärkere Entkopplung von Datenhaltung sowie der Systeme, die auf diesen Daten arbeiten.\\
+
+Auf diese Vorteile gehen auch Redmond et al. ein.
+Dabei betonen Sie direkt in der Einführung, dass ECS-Systeme inhärent nebenläufig seien (``inherently concurrent`) und sich diese Eigenschaft für die parallele Bearbeitung unterschiedlicher Aufgaben nutzen lässt.


This sentence contains a mismatched LaTeX opening/closing quote sequence: it starts with `` but closes with a single backtick. Use the proper closing quotes ('') so the quoted text renders correctly.

Suggested change

Dabei betonen Sie direkt in der Einführung, dass ECS-Systeme inhärent nebenläufig seien (``inherently concurrent`) und sich diese Eigenschaft für die parallele Bearbeitung unterschiedlicher Aufgaben nutzen lässt.

Dabei betonen Sie direkt in der Einführung, dass ECS-Systeme inhärent nebenläufig seien (``inherently concurrent'') und sich diese Eigenschaft für die parallele Bearbeitung unterschiedlicher Aufgaben nutzen lässt.

Copilot · 2026-04-26T21:32:01Z

+\subsection*{Abgrenzung Nebenläufigkeit und Paralellität}
+Wie Oechsle in~\cite{Oec22} anmerkt, wird in der Literatur oft nicht genau zwischen \textit{Nebenläufigkeit} und \textit{Paralellität} unterschieden.


"Paralellität" is misspelled (should be "Parallelität") in both the subsection title and the following sentence. Fixing this improves correctness and searchability in the document.

Suggested change

\subsection*{Abgrenzung Nebenläufigkeit und Paralellität}

Wie Oechsle in~\cite{Oec22} anmerkt, wird in der Literatur oft nicht genau zwischen \textit{Nebenläufigkeit} und \textit{Paralellität} unterschieden.

\subsection*{Abgrenzung Nebenläufigkeit und Parallelität}

Wie Oechsle in~\cite{Oec22} anmerkt, wird in der Literatur oft nicht genau zwischen \textit{Nebenläufigkeit} und \textit{Parallelität} unterschieden.

Copilot · 2026-04-26T21:32:01Z

+
+\begin{itemize}
+    \item \textbf{Abfragen} (\textit{query}/ \textit{queries}, $q$): Abfragen dienen dazu, auf bestimmte Komponentenkonstellationen gebündelt Zugriff zu erhalten.
+    Zur gezielten Steuerung von Abfragen können diese so gestaltet werden, dass entweder nur solche Komponentenbündel zurückgegeben werden, die der Schnittmenge von Komponenten entsprechen ()


This sentence ends with empty parentheses "()", which reads like a placeholder and makes the explanation incomplete. Either remove the parentheses or replace them with the intended set/intersection expression and complete the sentence.

Suggested change

Zur gezielten Steuerung von Abfragen können diese so gestaltet werden, dass entweder nur solche Komponentenbündel zurückgegeben werden, die der Schnittmenge von Komponenten entsprechen ()

Zur gezielten Steuerung von Abfragen können diese so gestaltet werden, dass entweder nur solche Komponentenbündel zurückgegeben werden, die der Schnittmenge von Komponenten entsprechen.

ThorstenSuckow added 3 commits April 26, 2026 21:09

chore(docs): initial commit for master-thesis

7661ada

Merge pull request #303 from ThorstenSuckow/master_thesis

521cc83

chore(docs): initial commit for master-thesis

Merge pull request #304 from ThorstenSuckow/ecs_rendering

1bbda5a

Ecs rendering

Copilot AI review requested due to automatic review settings April 26, 2026 21:28

ThorstenSuckow merged commit b10d8a3 into main Apr 26, 2026
1 of 2 checks passed

Copilot started reviewing on behalf of ThorstenSuckow April 26, 2026 21:28 View session

Copilot AI reviewed Apr 26, 2026

View reviewed changes

Navigation Menu

Dev - #305

Dev#305
ThorstenSuckow merged 3 commits into
mainfrom
dev

ThorstenSuckow commented Apr 26, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 26, 2026

Uh oh!

Copilot AI Apr 26, 2026

Uh oh!

Copilot AI Apr 26, 2026

Uh oh!

Copilot AI Apr 26, 2026

Uh oh!

Copilot AI Apr 26, 2026

Uh oh!

Copilot AI Apr 26, 2026

Uh oh!

Copilot AI Apr 26, 2026

Uh oh!

Copilot AI Apr 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants


		\section{Eine erste Annäherung an die inhärente Nebenläufigkeit von ECS-Programmen}

		Bevor wir weiter auf die in der Arbeit vorgestellten Formalisieren eingehen, wollen wir zunächst die ``ìnhärente Nebenläufigkeit`` verstehen, die die Autoren einführend anmerken, aber nicht weiter ausführen.

	\textit{``helios: Konzeption und prototypische Umsetzung eines C++ Game Frameworks``, DOI: \href{https://doi.org/10.5281/zenodo.xxxxx}{10.5281/zenodo.xxxxxx}
	\textit{``helios: Konzeption und prototypische Umsetzung eines C++ Game Frameworks``, DOI: \href{https://doi.org/10.5281/zenodo.xxxxxx}{10.5281/zenodo.xxxxxx}

	Dabei betonen Sie direkt in der Einführung, dass ECS-Systeme inhärent nebenläufig seien (``inherently concurrent`) und sich diese Eigenschaft für die parallele Bearbeitung unterschiedlicher Aufgaben nutzen lässt.
	Dabei betonen Sie direkt in der Einführung, dass ECS-Systeme inhärent nebenläufig seien (``inherently concurrent'') und sich diese Eigenschaft für die parallele Bearbeitung unterschiedlicher Aufgaben nutzen lässt.

		\subsection*{Abgrenzung Nebenläufigkeit und Paralellität}
		Wie Oechsle in~\cite{Oec22} anmerkt, wird in der Literatur oft nicht genau zwischen \textit{Nebenläufigkeit} und \textit{Paralellität} unterschieden.

	Zur gezielten Steuerung von Abfragen können diese so gestaltet werden, dass entweder nur solche Komponentenbündel zurückgegeben werden, die der Schnittmenge von Komponenten entsprechen ()
	Zur gezielten Steuerung von Abfragen können diese so gestaltet werden, dass entweder nur solche Komponentenbündel zurückgegeben werden, die der Schnittmenge von Komponenten entsprechen.

Conversation

ThorstenSuckow commented Apr 26, 2026

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants