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

[de] improve NACHDEM_PRAETERITUM #7001

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

affemitkaraffe
Copy link
Collaborator

  • add / fix suggestions

<example correction="gekommen war|gekommen hatte">Nachdem der Brief <marker>kam</marker>, ging ich nach Hause.</example>
<example correction="gesagt warst|gesagt hattest">Nachdem du etwas <marker>sagtest</marker>, erstarrten ihre Mienen vollkommen vor Ehrfurcht.</example>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hier müsste man vlt. überlegen, ob man komische suggestions in Kauf nehmen will.
Bei Sätzen wie: "Nachdem ich das wusste, habe ich es geändert." kommen dann Vorschläge wie "gewusst war"
Wie siehtst du das @danielnaber ?

Copy link
Member

Choose a reason for hiding this comment

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

Wenn nur falsche Vorschläge kommen für einen Fehler ist das natürlich blöd, das sollten wir vermeiden. Ansonsten verlasse ich mich da auf dein Urteil, @St-ac-y

Copy link
Collaborator

Choose a reason for hiding this comment

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

@affemitkaraffe @danielnaber ich würde die Regel so aufteilen, dass Verben mit "haben" und Verben mit "sein" getrennt werden (so viele mit "sein" gibt es ja nicht, zb. alle Verben der Bewegung) damit nicht so komische Vorschläge dabei rauskommen. Der User könnte eben denken, dass es auch so richtig sein könnte, obwohl es nur ein generischer Vorschlag ist, der nicht in allen Situationen stimmt. Was denkst du, @AgnesKleinhans ?

Copy link
Contributor

Choose a reason for hiding this comment

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

@St-ac-y Ja, für diesen Ansatz benötigen wir eine Liste aller Verben, die ausschließlich mit haben oder sein funktionieren. Ich stimme dir zu, dass es etwas unglücklich ist, wenn wir einen Vorschlag liefern, der grammatikalisch komplett falsch ist. In der prem4 liegt die Entität verbenmitsein, vielleicht hilft das weiter?

@@ -61343,12 +61343,13 @@ Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
<token>,</token>
</pattern>
<message>'Nachdem' drückt standardsprachlich Vorzeitigkeit aus und sollte daher nicht mit dem Präteritum verwendet werden. Verwenden Sie das Perfekt (Präsens im Hauptsatz) oder Plusquamperfekt (Präteritum im Hauptsatz) oder 'als' zum Ausdrücken von Gleichzeitigkeit.</message>
<suggestion suppress_misspelled="yes"><match no="2" postag="VER:PA2.*" postag_regexp="yes"/> war</suggestion>
<suggestion suppress_misspelled="yes"><match no="2" postag="VER:PA2.*" postag_regexp="yes"/> hatte</suggestion>
<suggestion suppress_misspelled="yes"><match no="2" postag="VER:PA2.*" postag_regexp="yes"/> <match no="2" postag="VER:([123].*)PRT.*" postag_replace="VER:AUX:$1PRT" postag_regexp="yes">sein</match></suggestion>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ich bin mir nicht sicher, ob die Zeile verstehe. Warum wird zwei mal das token 2 gematcht?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Beim ersten Mal brauche ich einfach nur das Wort für Partizip II (z. B. 'wusste' -> 'gewusst'). Beim zweiten Mal brauche ich das Postag (z. B. 'wusste' -> 'war', 'hatte').

Copy link
Collaborator

Choose a reason for hiding this comment

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

ja ok, wusste nicht, dass man dafür matchen muss, aber ja man kann eine suggestion nicht wie ein token behandeln und ihn eigene postags geben.

<suggestion suppress_misspelled="yes"><match no="2" postag="VER:PA2.*" postag_regexp="yes"/> war</suggestion>
<suggestion suppress_misspelled="yes"><match no="2" postag="VER:PA2.*" postag_regexp="yes"/> hatte</suggestion>
<suggestion suppress_misspelled="yes"><match no="2" postag="VER:PA2.*" postag_regexp="yes"/> <match no="2" postag="VER:([123].*)PRT.*" postag_replace="VER:AUX:$1PRT" postag_regexp="yes">sein</match></suggestion>
<suggestion suppress_misspelled="yes"><match no="2" postag="VER:PA2.*" postag_regexp="yes"/> <match no="2" postag="VER:([123].*)PRT.*" postag_replace="VER:AUX:$1PRT" postag_regexp="yes">haben</match></suggestion>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Vlt. noch eine suggestion für Perfekt?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hast du ein Beispiel? Die Regeln matchen nur bei 'VER.*PRT'

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ein Satz wie "Das Kind darf mit seinen Freunden spielen, nachdem es seine Hausaufgaben machte." wäre im Perfekt auch richtig, oder? Also: "Das Kind darf mit seinen Freunden spielen, nachdem es seine Hausaufgaben gemacht hat."
Das heißt vlt. könnte man hier schauen, dass man die Sätze mit Präsens im Hauptsatz rausfiltert und damit eine eigene Regel macht, die dem User auch das Perfekt vorschlägt. Sonst haben wir die Aufforderung nur in der message und der User hat nichts zum Klicken, falls so ein Fall auftritt. Weiß aber nicht wie viel AUfwand das jetzt wäre.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Danke! Eigentlich wollte ich nur die falschen/fehlenden Suggestions fixen. Deshalb würde ich Perfekt einfach aufnehmen.

Nach Präsens / Präteritum prüfen sollte mithilfe des Kommas auch funktionieren. Allerdings würde die Zahl der Subrules auf 16 steigen...

@affemitkaraffe affemitkaraffe marked this pull request as draft August 29, 2023 07:56
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.

None yet

4 participants