-
Notifications
You must be signed in to change notification settings - Fork 4
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
Handling sentences without place names #5
Comments
I tested it with some text. It seems my test cases were not complete. I will add the exception to it. Thanks
I tested it with some text. It seems my test cases were not complete. I will add the exception to it. Thanks |
What's the expected behavior when Profile does not find a place name (perhaps because there is none)? Right now it gives an IndexError, which is really easy to catch and replace with maybe
""
, but this is making me wonder if this is the version of the code that was run on XCEDE, since this issue surely would have come up. Do you suggest I just put an exception around it and return""
? And is there another version of the code somewhere?The text was updated successfully, but these errors were encountered: