-
Notifications
You must be signed in to change notification settings - Fork 51
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
First pass at setting up documentation. #58
Conversation
Not everything is in place quite yet...
TODO:
Once everything is in place, we can run |
… into mb/documentation
Not everything is in place quite yet...
ee1975a
to
e6521e5
Compare
mb/ar/documentation
Looks great: http://uwescience.github.io/pulse2percept The one thing that's still a bit weird is that this page has nothing on it: http://uwescience.github.io/pulse2percept/reference/pulse2percept.html It looks like it's not picking up the functions in that module :-/ |
Well, there is no such module, right? That is, we don't have a high-level Now the Python 3 tests seem to get stuck somehow and then error out... Python 2 tests run fine in my repo. Not sure what's going with Python 3... |
mb/ar/documentation v2
Yeah - got it. But it's weird that there's this empty page then.
I guess if we put a top-level docstring here:
https://github.com/uwescience/pulse2percept/blob/master/pulse2percept/__init__.py#L1
it would get rendered in here...
…On Wed, Feb 15, 2017 at 11:47 AM, Coveralls ***@***.***> wrote:
[image: Coverage Status] <https://coveralls.io/builds/10170667>
Coverage decreased (-1.0%) to 87.312% when pulling *cc780a8
<cc780a8>
on arokem:documentation* into *1b86d0a
<1b86d0a>
on uwescience:master*.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#58 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAHPNvYaFa61ZRsl0ZL9t9pQ7dMWjxz2ks5rc1ZUgaJpZM4Le2aX>
.
|
I think this is ready to go! |
Not everything is in place quite yet...