Maximilian_Krahn

Results 24 comments of Maximilian_Krahn

we have now implemented all changes from the master and this branch is also ready to review/to be merged πŸ™ :octocat: πŸ₯‡

this branch is now based on the master and ready for review/to be merged 🦸 πŸ¦Έβ€β™‚οΈ πŸ¦Έβ€β™€οΈ

> you need to fix some conflicts I have fixed those. The problem was just, that the series types were introduced in the file. πŸ€–

@selimelawwa I am sorry, I saw your push so late. Would tomorrow evening CET suit you as well? It looks like you forgot to format your files with the format.sh...

I got this idea from this [post](https://ljvmiranda921.github.io/notebook/2018/06/21/precommits-using-black-and-flake8/). Flake8 is basically a code linter, which will do some basic static checks. This will help us to prevent basic bugs, like using...

Hej @jbesomi I used style guides in other projects and languages before. As I am running short on time this July, as my BT takes more time than expected I...

As described in #162 , dask is not feasible from a UX perspective. Here are our results from experimenting with tokenize. See [THE ATTACHED PDF](https://github.com/jbesomi/texthero/files/5125891/ComparisonTokenisation.pdf) for a notebook of the...

@jbesomi We implemented the suggested changes. Big functions `visualize_topics` and `top_words_per_document` are now split up into multiple smaller ones, so the user of the library now has to write the...

@jbesomi we created a [short](https://colab.research.google.com/drive/15U0XW_dfr05mB3ATIktH7P4s3Mz0Z3qg?usp=sharing) notebook, where we display the functionality of those two pipelines. 🐰 We think those two will be the main use cases of the implemented functions....

we also have updated this branch, so it is now sourced from the master πŸ₯³ it is now ready to be reviewed or merged πŸ¦€ 🀞