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

Made python interface installable via pip #143

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

AlreadyTakenJonas
Copy link

Hi, HarisIqbal88!

My boss told me to use your package for my master thesis and I really like it, but it was bugging me, that it didn't have an installer. So I made it installable via pip. I had to rearrange your directories a little bit, but I hope you like it.

I also added CITATION.cff, this will show how to cite your software directly in the repos sidebar.

You might want to add your email or ORCID in pyproject.toml and check CITATION.cff for errors.

I wish you a happy New Year!

Kind regards,
Jonas Eichhorn

@AlreadyTakenJonas
Copy link
Author

I almost forgot: After installing you can run python3 -m PlotNeuralNetPy.pyexamples.test_simple to run your test script.

@@ -0,0 +1,4 @@
#!/usr/bin/env python3
# -*- coding: utf-8 -*-
from .test_simple import main, arch
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These imports conflict. No need to import the individual example namespaces into the top __init__ namespace.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Didn't catch that!

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

Successfully merging this pull request may close these issues.

3 participants