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

ENH: make wradlib aware of its version #9

Merged
merged 16 commits into from Mar 3, 2016
Merged

Conversation

kmuehlbauer
Copy link
Member

make wradlib aware of its version ,adapt setup.py, wradlib/init.py and doc/source/conf.py from pyart, add version and date to doc

This can go with issue #8 , its somehow related to python2/3. Adapted setup.py with versioning applied.

kmuehlbauer and others added 2 commits February 24, 2016 14:24
@kmuehlbauer kmuehlbauer mentioned this pull request Feb 24, 2016
1 task
@wradlib wradlib added this to the Release 0.8.0 milestone Mar 3, 2016
@kmuehlbauer
Copy link
Member Author

This needs to be merged to test deployment and versioning.

wradlib pushed a commit that referenced this pull request Mar 3, 2016
ENH: make wradlib aware of its version, add pypi deployment
@wradlib wradlib merged commit 9d2f761 into wradlib:master Mar 3, 2016
@kmuehlbauer kmuehlbauer mentioned this pull request Mar 9, 2016
3 tasks
@heistermann
Copy link
Collaborator

This PR involved to modify __init__.py, adding

from .version import git_revision as __git_revision__

When I try to import wradlib, this line throws an exception:

ImportError: No module named version

I don't have any idea about this module...is this some Python2/3 issue?

@jjhelmus
Copy link
Contributor

jjhelmus commented Mar 9, 2016

You need to trigger wradlib's build step which creates the version.py file. Either use python setup.py install and changing directory so you do not pick up the source directory, or use python setup.py build_ext --inplace to do an in-place build.

@kmuehlbauer
Copy link
Member Author

@jjhelmus Thanks, I just started to write...

@heistermann
Copy link
Collaborator

Thanks!

kmuehlbauer pushed a commit that referenced this pull request Feb 28, 2018
ENH: make wradlib aware of its version, add pypi deployment
ElmerJeanpierreLopez added a commit to ElmerJeanpierreLopez/wradlib that referenced this pull request Sep 15, 2019
Documentation mentions elevation angle in [rads] ,but it should be [deg]. I already discussed in Issue wradlib#9.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants