Maximilian_Krahn
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 π¦ π€