CODAIT / text-extensions-for-pandas

Natural language processing support for Pandas dataframes.
Apache License 2.0
217 stars 34 forks source link

NLP Integration notebook review #120

Closed kmh4321 closed 4 years ago

kmh4321 commented 4 years ago

This PR addresses #108

review-notebook-app[bot] commented 4 years ago

Check out this pull request on  ReviewNB

Review Jupyter notebook visual diffs & provide feedback on notebooks.


Powered by ReviewNB

kmh4321 commented 4 years ago

@frreiss my changes are now done. I am yet to verify on Watson Studio but with #128 fix in, it should work smoothly. I can verify once the changes done in #128 are pushed to PyPi.

kmh4321 commented 4 years ago

Out of the 31 merge conflicts shown above, only 2 are actual conflicts. Rest are Jupyter metadata related.

frreiss commented 4 years ago

Notebook changes look great! I've fixed the merge conflicts using vscode. Waiting on CI to complete.