Improve copyright POS tagging #930

 * minor changes mostly from scanning several npms

Signed-off-by: Philippe Ombredanne <[email protected]>

Author: pombredanne

src/cluecode/copyrights.py

@@ -226,13 +226,13 @@ def detect(location):
     (r'^(Send|It|Mac|Support|Confidential|Information|Various|Mouse|Wheel'
       r'|Vendor|Commercial|Indemnified|Luxi|These|Several|GnuPG|WPA|Supplicant'
       r'|TagSoup|Contact|IA64|Foreign|Data|Atomic|Pentium|Note|Delay|Separa.*|Added'
-      r'|Glib|Gnome|Gaim|Open|Possible|In|Read|Permissions?|New'
+      r'|Glib|Gnome|Gaim|Open|Possible|In|Read|Permissions?|New|MIT'
       r')$', 'NN'),
 
     # Various non CAPS
     (r'^(OR)$', 'NN'),
 
-    # Various rare non CAPS but NNP
+    # Various rare non CAPS but NNP, treated as full names
     (r'^(FSF[\.,]?)$', 'NAME'),
 
     # Windows XP
@@ -252,6 +252,9 @@ def detect(location):
      r'[Ff]unctionality|bgcolor|F+|Rewrote|Much|remains?,?|Implementation|earlier'
      r'|al.|is|laws|url|[Ss]ee)$', 'JUNK'),
 
+    # Some mixed case junk
+    (r'^LastModified$', 'JUNK'),
+
     # Some font names
     (r'^Lucida$', 'JUNK'),
 
@@ -277,8 +280,8 @@ def detect(location):
 
     (r'^\$?LastChangedDate\$?$', 'YR'),
 
-    # Misc corner cases
-    (r'^Software,\',|\(Royal|PARADIGM|nexB|Antill\',$', 'NNP'),
+    # Misc corner cases that are NNP
+    (r'^Software,\',|\(Royal|PARADIGM|nexB|okunishinishi|yiminghe|Antill\',$', 'NNP'),
 
     # rarer caps
     # EPFL-LRC/ICA
@@ -347,7 +350,8 @@ def detect(location):
     (r'^HOLDER\(S\)$', 'JUNK'),
     (r'^([Hh]olders?|HOLDERS?)$', 'HOLDER'),
 
-    (r'^([Rr]espective)$', 'NN'),
+    # not NNPs
+    (r'^([Rr]espective|JavaScript)$', 'NN'),
 
     # affiliates or "and its affiliate(s)."
     (r'^[Aa]ffiliate(s|\(s\))?\.?$', 'NNP'),
@@ -495,11 +499,8 @@ def detect(location):
     # all CAPS word, all letters including an optional trailing single quote
     (r"^[A-Z]{2,}\'?$", 'CAPS'),
 
-    # email eventually in parens or brackets. The closing > or ) is optional
-    (r'[\<\(][a-zA-Z0-9\+_\-\.\%]+(@|at)[a-zA-Z0-9][a-zA-Z0-9\+_\-\.\%]*\.[a-zA-Z]{2,5}?[\>\)]?', 'EMAIL'),
-
-    # email
-    (r'[a-zA-Z0-9\+_\-\.\%]+(@|at)[a-zA-Z0-9][a-zA-Z0-9\+_\-\.\%]*\.[a-zA-Z]{2,5}?', 'EMAIL'),
+    # email eventually in parens or brackets with some trailing punct.
+    (r'^[\<\(]?[a-zA-Z0-9]+[a-zA-Z0-9\+_\-\.\%]*(@|at)[a-zA-Z0-9][a-zA-Z0-9\+_\-\.\%]+\.[a-zA-Z]{2,5}?[\>\)\.\,]*$', 'EMAIL'),
 
     # URLS such as <(http://fedorahosted.org/lohit)>
     (r'[<\(]https?:.*[>\)]', 'URL'),
@@ -526,6 +527,7 @@ def detect(location):
 
     # comma as a conjunction
     (r'^,$', 'CC'),
+
     # .\" is not a noun
     (r'^\.\\\?"?$', 'JUNK'),
 
@@ -538,7 +540,7 @@ def detect(location):
     # communications
     (r'communications', 'NNP'),
 
-    # Code variable names, snake case
+    # Code variable names including snake case
     (r'^.*(_.*)+$', 'JUNK'),
 
     # nouns (default)

tests/cluecode/data/copyrights/junk_trailing.txt

@@ -0,0 +1 @@
+Otherwood (c) 2011 note this implementation is heavily based/inspired from the dictionary implementation

tests/cluecode/data/copyrights/junk_trailing.txt.yml

@@ -0,0 +1,4 @@
+what:
+    - copyrights
+copyrights:
+    - Otherwood (c) 2011

tests/cluecode/data/ics/chromium-chrome-common-extensions-docs-examples-apps-hello-python-httplib2/__init__.py.yml

@@ -6,8 +6,12 @@ copyrights:
     - Copyright 2006, Joe Gregorio contributors Thomas Broyer ([email protected])', James Antill
 holders:
     - Joe Gregorio contributors
-    - Thomas Broyer James Antill
+    - Thomas Broyer
+    - ([email protected])
+    - James Antill
 holders_summary:
     - Joe Gregorio contributors
-    - Thomas Broyer James Antill
+    - Thomas Broyer
+    - ([email protected])
+    - James Antill
 notes: extra trailing contribution should not be detected