HTTPERROR: 500 Server Error (CoreNLPDependencyParser) [CoreNLP ver: 4.3.2]

[JingliSHI0206]

Hi,

I try to use CoreNLPDependencyParser to extract POS and dependency structure. but httperror will come out if percentage sign (%) is included in the sentence. a simple demo is as below.

from nltk.parse import CoreNLPDependencyParser
words_processed = ['it', "'s", 'because', 'when', 'you', 'buy', 'it', '%']
depparser = CoreNLPDependencyParser(url='http://localhost:9001')
temp_parser_res = depparser.parse(words_processed)
print(temp_parser_res)

[Output]:

    HTTPError                                 Traceback (most recent call last)
    <ipython-input-28-301966b0519c> in <module>()
          5 words_processed = ['it', "'s", 'because', 'when', 'you', 'buy', 'it', '%']
          6 depparser = CoreNLPDependencyParser(url='http://localhost:9001')
    ----> 7 temp_parser_res = depparser.parse(words_processed)
          8 print(temp_parser_res)
    3 frames
    /usr/local/lib/python3.7/dist-packages/nltk/parse/api.py in parse(self, sent, *args, **kwargs)
         43         
         44         if overridden(self.parse_sents):
    ---> 45             return next(self.parse_sents([sent], *args, **kwargs))
         46         elif overridden(self.parse_one):
         47             return (tree for tree in [self.parse_one(sent, *args, **kwargs)] if tree is not None)
    
    /usr/local/lib/python3.7/dist-packages/nltk/parse/corenlp.py in raw_parse_sents(self, sentences, verbose, properties, *args, **kwargs)
        288                 yield iter([tree])
        289         """
    --> 290         parsed_data = self.api_call('\n'.join(sentences), properties=default_properties)
        291         for parsed_sent in parsed_data['sentences']:
        292             tree = self.make_tree(parsed_sent)
    
    /usr/local/lib/python3.7/dist-packages/nltk/parse/corenlp.py in api_call(self, data, properties)
        249         )
        250 
    --> 251         response.raise_for_status()
        252 
        253         return response.json()
    
    /usr/local/lib/python3.7/dist-packages/requests/models.py in raise_for_status(self)
        939 
        940         if http_error_msg:
    --> 941             raise HTTPError(http_error_msg, response=self)
        942 
        943     def close(self):
    
    HTTPError: 500 Server Error: Internal Server Error for url: http://localhost:9001/?properties=%7B%22outputFormat%22%3A+%22json%22%2C+%22annotators%22%3A+%22tokenize%2Cpos%2Clemma%2Cssplit%2Cdepparse%22%2C+%22ssplit.ssplit.eolonly%22%3A+%22true%22%7D

any help about this issue is greatly appreciated.

[AngledLuffa]

NLTK doesn't set the content type to text and doesn't escape the % symbol. The result is a change I made to make the server more compatible with one library stops working when I try to url-decode the incoming text. I suppose we could check for % without an escape sequence or text that doesn't decode properly rather than failing. I'll put that in the dev branch for now. The next release should be in a week or less. If you're comfortable with working around %, or rebuilding from the dev branch, great, and if not, I can put a temporary patch together for you.

[AngledLuffa]

20fe1e9

[AngledLuffa]

Would you try this? I think it should compensate for that issue without breaking the stanza integration

https://nlp.stanford.edu/software/stanford-corenlp-4.3.2b.zip

[JingliSHI0206]

i fix this issue by replacing % with %20.
thank you so much for your suggestion.

[AngledLuffa]

Please note that %20 is a space, and %25 is the percent sign in the original text.

…

On Sun, Dec 5, 2021 at 3:36 PM Jingli Shi ***@***.***> wrote: i fix this issue by replacing % with %20. thank you so much for your suggestion. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1226 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA2AYWIUIRF46I5B7LVKZMLUPPZQJANCNFSM5JG2C3ZA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[JingliSHI0206]

thank you for correcting me. it's a typo.

Please note that %20 is a space, and %25 is the percent sign in the original text.
…
On Sun, Dec 5, 2021 at 3:36 PM Jingli Shi @.***> wrote: i fix this issue by replacing % with %20. thank you so much for your suggestion. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1226 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2AYWIUIRF46I5B7LVKZMLUPPZQJANCNFSM5JG2C3ZA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.