-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
base: master
Are you sure you want to change the base?
Conversation
affemitkaraffe
commented
Aug 8, 2022
- 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> |
There was a problem hiding this comment.
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 ?
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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 ?
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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').
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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'
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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...