Allow to set NLTK options via TabPy config

tableau / TabPy

Execute Python code on the fly and display results in Tableau visualizations:

https://tableau.github.io/TabPy/

MIT License

1.56k stars 598 forks source link

Allow to set NLTK options via TabPy config #471

Open ElPincheTopo opened 3 years ago

ElPincheTopo commented 3 years ago

Create a convenience function to expose the settings file to scripts, added NLTK options to the example script and updated SentimentAnalysis.py to use the download path and proxy settings defined in the nltk parameters in the config.

0golovatyi commented 3 years ago

Looks good. Could you update one of the integration tests for models with the new settings (https://github.com/tableau/TabPy/tree/master/tests/integration)?

ElPincheTopo commented 3 years ago

Sure, I'm having a busy week but I hope to do this before the weekend.

cgore1 commented 3 years ago

@ElPincheTopo would appreciate if you could address integration test comment so that we can merge your change! thanks!

salvuswarez commented 2 years ago

Hello, Has this feature been applied yet? Running into this issue currently and could really use a solution like this. Thanks,