case insensitive mode moved to start of regex

BCHSI / philter-ucsf

Open source clinical text de-identification

BSD 3-Clause "New" or "Revised" License

111 stars 49 forks source link

case insensitive mode moved to start of regex #22

Open landiisotta opened 1 year ago

landiisotta commented 1 year ago

Since Python 3.6 the inline flags not at the start of regular expression throw an error. Modified line 226 of philter.py:

added exception to catch the re.error;
moved case insensitive regex (?i) to the start of the regular expression.

Keetan commented 1 year ago

I also ran into the re.error when following the instructions in the readme, and was able to run philter successfully (steps 0, 2a, and 2d) using the changes in this PR.