-
Notifications
You must be signed in to change notification settings - Fork 3
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
PR for pspec_graduation
#4
Conversation
…function, and in_units function
modified: hera_pspec/tests/test_version.py
…_pspec into pspec_graduation
modified: .travis.yml
* Placeholder comments added. * Some methods deprecated (to be removed or split out into utility library). * Warnings added to unsupported branches of the code * Code-style tidy-up, including more whitespace * Deprecate the redundant set_data() function * Rename 'Ctrue' -> 'C_empirical'
…to pspec_graduation
modified: hera_pspec/conversions.py new file: hera_pspec/tests/test_conversions.py
modified: hera_pspec/conversions.py modified: hera_pspec/tests/test_conversions.py
…pspec into pspec_graduation
until we completely graduatea utils module modified: hera_pspec/dataset.py
…to pspec_graduation
I'll review the code that @philbull committed, which was the @acliu if you also want to sign off on this that might be good too (I know you are busy so no pressure to actually review it). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This passes tests and in going into a dev branch, so I am automatically accepting it.
this is a PR to graduate some of the basic code for
hera_pspec
, includingdataset.py
,version.py
andconversions.py
with tests. Thesetup.py
installation and travis testing environment are also graduated, and the oldpspec.py
code is also graduated for the purpose of backwards compatibility, although future PRs may eliminate this frommaster
and move it to a branch.