Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Venn diagrams with word clouds for set elements #29

Closed
paulbrodersen opened this issue Mar 29, 2017 · 6 comments
Closed

Venn diagrams with word clouds for set elements #29

paulbrodersen opened this issue Mar 29, 2017 · 6 comments

Comments

@paulbrodersen
Copy link

paulbrodersen commented Mar 29, 2017

I made a thing. Super-derivative of your amazing work and the equally amazing wordcloud (https://github.com/amueller/word_cloud). Essentially, it plots the set elements as a word cloud on top of the corresponding subset patches.

At the moment, the code can be found on my github: https://github.com/paulbrodersen/venn_wordcloud.
However, to prevent the further balkanisation of python packages, I would love to properly integrate my code bits into your package -- if you are interested. It would essentially add two functions venn2_wordcloud and venn3_wordcloud, and one more dependency (wordcloud).

@konstantint
Copy link
Owner

Hey, this is totally awesome!

However, I personally feel that "balkanization" is a good thing, because it limits the dependencies and allows everyone to use exactly the thing they need without having to pull in too much extra stuff.
In addition, it is easier to maintain things when each author has to be responsible for the code they are most familiar with.

I therefore think that you should publish your package as matplotlib_venn_wordcloud in pip, and I'll simply add the reference to it in the README here.

@paulbrodersen
Copy link
Author

Then we will do it that way. I will figure out how to put stuff on pypi this weekend. I will let you know when it's live.

@konstantint
Copy link
Owner

Hope this helps:

  1. Put your current python files into a subdirectory for a more packag-y shape.
  2. Add setup.py so that your package would be properly installable. (You can copy-paste matplotlib_venn's version and adapt slightly. E.g. drop the PyTest part there)
  3. Make sure python setup.py install works on your machine.
  4. Register an account at PyPI
  5. python setup.py register sdist upload

And you're done.

@konstantint
Copy link
Owner

konstantint commented Apr 4, 2017

Btw, one thing I would warmly suggest you consider is changing your current license from GPL to something less restrictive (like MIT, BSD, Unlicense or, at the very least, LGPL). In fact even if I wanted to include your project to be part of matplotlib_venn, its licensing terms would conflict with those of matplotlib_venn's in my understanding.

I never stop being amused by how the flagship "free as in freedom" license is actually pretty "freedom-limiting" in its terms.

@paulbrodersen
Copy link
Author

  1. Thanks for the pointers. I think I just managed to upload the thing to pypi about 30 seconds ago. pip install into a fresh virtualenv seems to work. Can you give it a spin, too? Package name is matplotlib_venn_wordcloud, as you suggested.

  2. Hadn't thought about licensing issues. I can change it to MIT. Give me 5 min.

@paulbrodersen
Copy link
Author

Ok, version 0.2 with an MIT license should be live on pypi. Thanks for the help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants