Revision: 9408
          
http://languagetool.svn.sourceforge.net/languagetool/?rev=9408&view=rev
Author:   dominikoeo
Date:     2013-02-17 19:36:35 +0000 (Sun, 17 Feb 2013)
Log Message:
-----------
[fr] fixed false positive in "C'est la fin." caused by
     disambiguator rule which was not robust enough.

Modified Paths:
--------------
    
trunk/languagetool/languagetool-language-modules/fr/src/main/resources/org/languagetool/resource/fr/disambiguation.xml
    
trunk/languagetool/languagetool-language-modules/fr/src/main/resources/org/languagetool/rules/fr/grammar.xml
    
trunk/languagetool/languagetool-language-modules/fr/src/test/java/org/languagetool/tagging/disambiguation/rules/fr/FrenchRuleDisambiguatorTest.java

Modified: 
trunk/languagetool/languagetool-language-modules/fr/src/main/resources/org/languagetool/resource/fr/disambiguation.xml
===================================================================
--- 
trunk/languagetool/languagetool-language-modules/fr/src/main/resources/org/languagetool/resource/fr/disambiguation.xml
      2013-02-17 18:56:18 UTC (rev 9407)
+++ 
trunk/languagetool/languagetool-language-modules/fr/src/main/resources/org/languagetool/resource/fr/disambiguation.xml
      2013-02-17 19:36:35 UTC (rev 9408)
@@ -776,15 +776,38 @@
     </pattern>
     <disambig><match no="2" postag_regexp="yes" postag="V avoir ind pres 
.*"/></disambig>
   </rule>
-  <rule name="RP : Être + adj ambigu" id="RP-ETRE_ADJ_AMBIG">
-    <pattern>
-      <token postag="V etre .*" postag_regexp="yes" skip="2"/>
-      <marker>
-        <token postag="J.*" 
postag_regexp="yes"><exception>torse</exception></token>
-      </marker>
-    </pattern>
-    <disambig><match no="2" postag_regexp="yes" postag="J.*"/></disambig>
-  </rule>
+  <rulegroup name="RP : Être + adj ambigu" id="RP-ETRE_ADJ_AMBIG">
+    <rule>
+      <pattern>
+        <token postag="V etre .*" postag_regexp="yes"/>
+        <marker>
+          <token postag="J.*" 
postag_regexp="yes"><exception>torse</exception></token>
+        </marker>
+      </pattern>
+      <disambig><match no="2" postag_regexp="yes" postag="J.*"/></disambig>
+    </rule>
+    <rule>
+      <pattern>
+        <token postag="V etre .*" postag_regexp="yes"/>
+        <token postag="A"/>
+        <marker>
+          <token postag="J.*" postag_regexp="yes"/>
+        </marker>
+      </pattern>
+      <disambig><match no="3" postag_regexp="yes" postag="J.*"/></disambig>
+    </rule>
+    <rule>
+      <pattern>
+        <token postag="V etre .*" postag_regexp="yes"/>
+        <token postag="A"/>
+        <token postag="A"/>
+        <marker>
+          <token postag="J.*" postag_regexp="yes"/>
+        </marker>
+      </pattern>
+      <disambig><match no="4" postag_regexp="yes" postag="J.*"/></disambig>
+    </rule>
+  </rulegroup>
   <rule name="RB : le la les" id="RB-LE_LA_LES">
     <pattern>
       <marker>

Modified: 
trunk/languagetool/languagetool-language-modules/fr/src/main/resources/org/languagetool/rules/fr/grammar.xml
===================================================================
--- 
trunk/languagetool/languagetool-language-modules/fr/src/main/resources/org/languagetool/rules/fr/grammar.xml
        2013-02-17 18:56:18 UTC (rev 9407)
+++ 
trunk/languagetool/languagetool-language-modules/fr/src/main/resources/org/languagetool/rules/fr/grammar.xml
        2013-02-17 19:36:35 UTC (rev 9408)
@@ -1168,7 +1168,7 @@
         <message>Voulez-vous écrire <suggestion>À</suggestion> ?</message>
         <example type="incorrect"><marker>A</marker> quelle heure arrive-t-il 
?</example>
         <example type="correct"><marker>À</marker> quelle heure arrive-t-il 
?</example>
-        <example type="correct">A priori, il devrait arriver à 
l'heure.</example>
+        <example type="correct">A priori, il devrait arriver à 
l’heure.</example>
       </rule>
       <rule>
         <pattern>
@@ -3377,6 +3377,7 @@
         <example type="correct">La plus <marker>grande fait</marker> dix 
centimètres.</example>
         <example type="correct">Les équations <marker>différentielles 
non</marker> linéaires.</example>
         <example type="correct">Une des multiples affaires.</example>
+        <example type="correct">C’est la fin.</example>
       </rule>
       <rule name="nom féminin suivi du masculin">
         <pattern>

Modified: 
trunk/languagetool/languagetool-language-modules/fr/src/test/java/org/languagetool/tagging/disambiguation/rules/fr/FrenchRuleDisambiguatorTest.java
===================================================================
--- 
trunk/languagetool/languagetool-language-modules/fr/src/test/java/org/languagetool/tagging/disambiguation/rules/fr/FrenchRuleDisambiguatorTest.java
 2013-02-17 18:56:18 UTC (rev 9407)
+++ 
trunk/languagetool/languagetool-language-modules/fr/src/test/java/org/languagetool/tagging/disambiguation/rules/fr/FrenchRuleDisambiguatorTest.java
 2013-02-17 19:36:35 UTC (rev 9408)
@@ -54,12 +54,6 @@
   }
 
   public void testChunker() throws IOException {
-    TestTools.myAssert("Je ne suis pas la seule.",
-        "/[null]SENT_START Je/[je]R pers suj 1 s  /[null]null ne/[null]A  
/[null]null suis/[être]V etre ind pres 1 s  /[null]null pas/[pas]A  /[null]null 
la/[le]D f s  /[null]null seule/[seul]J f s ./[null]null", 
-        tokenizer, sentenceTokenizer, tagger, disambiguator);
-    TestTools.myAssert("Je ne suis pas la seule.",
-        "/[null]SENT_START Je/[je]R pers suj 1 s  /[null]null ne/[null]null  
/[null]null suis/[suivre]V imp pres 2 s|suis/[suivre]V ind pres 1 
s|suis/[suivre]V ind pres 2 s|suis/[être]V etre ind pres 1 s  /[null]null 
pas/[pas]N f sp|pas/[pas]N m sp  /[null]null la/[la]N m sp|la/[la]R pers obj 3 
f s|la/[le]D f s  /[null]null seule/[seul]D f s|seule/[seul]J f s|seule/[seul]N 
f s ./[null]null", 
-        tokenizer, sentenceTokenizer, tagger, disamb2); 
     TestTools.myAssert("Il a enfin publié son livre.",
         "/[null]SENT_START Il/[il]R pers suj 3 m s  /[null]null a/[avoir]V 
avoir ind pres 3 s  /[null]null enfin/[enfin]A  /[null]null publié/[publier]V 
ppa m s  /[null]null son/[son]D e s  /[null]null livre/[livre]N e s 
./[null]null", 
         tokenizer, sentenceTokenizer, tagger, disambiguator);

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
The Go Parallel Website, sponsored by Intel - in partnership with Geeknet, 
is your hub for all things parallel software development, from weekly thought 
leadership blogs to news, videos, case studies, tutorials, tech docs, 
whitepapers, evaluation guides, and opinion stories. Check out the most 
recent posts - join the conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Languagetool-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/languagetool-commits

Reply via email to