Skip to content
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

Update hpmor-chapter-001.tex #54

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update hpmor-chapter-001.tex #54

wants to merge 2 commits into from

Conversation

Odina14
Copy link
Collaborator

@Odina14 Odina14 commented Nov 20, 2024

Ich habe einige kleinere Zeichenfehler korrigiert (Vor und nach Auslassungspunkten muss ein Leerzeichen stehen, sofern sie nicht den fehlenden Teil eines Worts markieren, vor und nach Gedankenstrichen steht ebenfalls ein Leerzeichen und wenn nach dem Einschub durch Gedankenstriche ein Nebensatz steht, der ein Komma verlangt, bleibt dieses nach dem Gedankenstrich erhalten.) Außerdem habe ich die Nutzung der Auslassungspunkte und Gedankenstriche an die deutschen Regeln angepasst und vereinheitlicht. In 153 könnte man überlegen, ob man "Vaters" durch "Dads" ersetzt. Anders als an anderen Textstellen, sind wir hier in Harrys Gedanken, der seinen Vater Dad nennt, und "Vaters" wird wie ein Eigenname gebraucht (anders als an anderen beschreibenden Stellen: sein Vater etc.).

Ich habe einige kleinere Zeichenfehler korrigiert (Vor und nach Auslassungspunkten muss ein Leerzeichen stehen, sofern sie nicht den fehlenden Teil eines Worts markieren, vor und nach Gedankenstrichen steht ebenfalls ein Leerzeichen und wenn nach dem Einschub durch Gedankenstriche ein Nebensatz steht, der ein Komma verlangt, bleibt dieses nach dem Gedankenstrich erhalten.) Außerdem habe ich die Nutzung der Auslassungspunkte und Gedankenstriche an die deutschen Regeln angepasst und vereinheitlicht. In 153 könnte man überlegen, ob man "Vaters" durch "Dads" ersetzt. Anders als an anderen Textstellen, sind wir hier in Harrys Gedanken, der seinen Vater Dad nennt, und "Vaters" wird wie ein Eigenname gebraucht (anders als an anderen beschreibenden Stellen: sein Vater etc.).
@entorb
Copy link
Owner

entorb commented Nov 21, 2024

Hi @Odina14, vielen lieben Dank für die vielen Verbesserungsvorschläge.
Bezüglich der Auslassungspunkte und Gedankenstriche:
Ich habe ein Qualitäts-Checker Skript aufgesetzt, dass solche DInge vereinheitlicht. Aktuell ist die hinterlegte Regel für "…": um Leerzeichen diese entfrenen, es sei denn es folgt ein Satzzeichen.
https://github.com/entorb/hpmor-de/blob/main/scripts/check_chapters.py#L180

Für Gedankenstriche habe ich einige mehr Regeln hinterlegt:
https://github.com/entorb/hpmor-de/blob/main/scripts/check_chapters.py#L428

Die Regeln kann ich gerne anpassen/aufweichen, wenn Du Dir da sehr sicher bist.


% “I told them you weren’t to know,” Petunia whispered. “But it’s true. I’ve seen things—”
„Ich wollte nicht, dass du es erfährst“, flüsterte Petunia. „aber es ist wahr. Ich habe Sachen gesehen…“
„Ich wollte nicht, dass du es erfährst“, flüsterte Petunia. „Aber es ist wahr. Ich habe Sachen gesehen …“
Copy link
Owner

Choose a reason for hiding this comment

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

Hier ist doch ein Satz nur unterbrochen, ich denke es wäre korrekt "aber" klein zu lassen, oder?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hi Torben! Sehr gerne :). Ich höre gerade das deutsche Hörbuch und da gab es den Hinweis auf dieses tolle Projekt! So cool, dass du das ins Leben gerufen hast! Ich bin mir absolut sicher, was die Zeichensetzung betrifft. Ich habe Germanistik mit einem Schwerpunkt in Interpunktion studiert, arbeite seit 10 Jahren als Lektorin in Verlagen und habe meine Korrekturen vor dem Absenden mit dem Duden abgeglichen. Die Regeln anzupassen, ist eine tolle Option, anstatt alles händisch zu korrigieren. Ich kenne mich gut mit der deutschen Schriftsprache aus, aber leider so gar nicht mit Python ;).

Zu deiner zweiten Frage. Das "aber" darf klein bleiben, aber dann müsste der Punkt davor durch ein Komma ersetzt werden. Liebe Grüße und nochmal Danke!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ein Nachtrag noch: Bei den Auslassungspunkten und Gedankenstrichen müsste die Regel dann so lauten, dass ein Leerzeichen vorangeht und folgt, es sei denn es folgt ein Satzzeichen. Denn wenn ", . oder , folgen steht dahinter kein Leerzeichen. (Streng genommen steht kein Leerzeichen davor, wenn eine Wortunterbrechung markiert wird, z.B. "Das finde ich schei...", aber das ist mir in HPMOR bisher nicht begegnet."

Copy link
Owner

Choose a reason for hiding this comment

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

Na da ergänzen wir uns großartig ;-)

Ich schlage vor wir gehen so vor:
Für die neue Regel für die Auslassungspunkten und Gedankenstriche lege ich eine neues "Issue" an und dort definieren wir die Regel und ich erstelle einen Entwurfs PR dazu.

Daher bitte vorerst diese beiden Themen nicht händisch korrigieren, da sonst doppelte Arbeit. Andere Korrekturen natürlich weiter gerne.

@entorb
Copy link
Owner

entorb commented Nov 23, 2024

Bitte schau mal in diesen Entwurf rein, ob das so den korrekten Regeln entspricht.

@Odina14
Copy link
Collaborator Author

Odina14 commented Nov 23, 2024

Sieht perfekt aus! Danke! Händisch hatte ich das Thema in Ruhe gelassen ;)

@entorb
Copy link
Owner

entorb commented Nov 24, 2024

Hi @Odina14, kannst Du bitte mal probieren ob diese Option für Dich in diesem PR sichtbar ist?
https://github.blog/changelog/2022-02-03-more-ways-to-keep-your-pull-request-branch-up-to-date/

@Odina14
Copy link
Collaborator Author

Odina14 commented Nov 28, 2024

Hey Torben, sorry für die späte Rückmeldung. War krank. Leider nein. Bei mir sieht es so aus:
Screenshot 2024-11-28 155049

@entorb
Copy link
Owner

entorb commented Nov 28, 2024

Ok, dann musst du leider die Konflikte (durch meine große Änderung) händisch lösen oder einen neuen PR erstellen der dann auf dem neuen Stand basiert. Gute Besserung!

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.

2 participants