[NLP] bug with thread trimming

Common-SenseMakers / sensemakers

Sensemakers infrastructure for developing AI-based tools for semantic annotations of social posts. Cross-poster app to publish your semantic posts on different networks.

GNU General Public License v3.0

1 stars 2 forks source link

[NLP] bug with thread trimming #117

Open ronentk opened 2 months ago

ronentk commented 2 months ago

See in tests/test_multi_chain_app_interface.py:158

quote tweet urls get trimmed by char trimming

ronentk commented 1 month ago

Added a fix for this bug, but trimming needs to be tested more exhaustively ( #119). Currently set limit to 40 posts so in practice this won't apply for the majority of posts. https://github.com/Common-SenseMakers/sensemakers/blob/575a8db7cf6955f751a32239ecd2b6c307f5bc2c/nlp/desci_sense/shared_functions/interface.py#L23