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

Add in the Pei (1992) shape model #26

Merged
merged 3 commits into from
Oct 26, 2017
Merged

Add in the Pei (1992) shape model #26

merged 3 commits into from
Oct 26, 2017

Conversation

karllark
Copy link
Owner

@karllark karllark commented Oct 26, 2017

Addresses #14.

@karllark karllark self-assigned this Oct 26, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling a46b7de on add_p92 into d50cb62 on master.

@karllark karllark merged commit 06f5a45 into master Oct 26, 2017
@karllark karllark deleted the add_p92 branch October 26, 2017 11:13
@pllim
Copy link
Contributor

pllim commented Oct 26, 2017

Wow, 100% coverage! 👏

@karllark
Copy link
Owner Author

Thanks. I am trying to be "good" with my tests for this package. Now, if I can get the documentation in better shape, then I will be happy and will do an official 1st release. Fun learning how to do a well supported package. Hope others will find it useful. I know I already do find this package useful for my research, so a win already!

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

3 participants