-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Testing #10
Comments
Just a reminder, if you work on tests at all, please mark them out so we don't duplicate work. Feel free to work either through a fork or the |
I have written a few tests for the asset pricing module, but need some help. I am trying to think of a very simple example that we can "know" the answer to so that I can check the accuracy of the |
#45 Test Branch Merged.I have one more item before this issue is closed. For the
|
@sanguineturtle Thanks for looking into this. Just a runtime test for examples is completely fine. No need to do anything beyond that. In fact the ar1sim.py file was something I threw together that doesn't actually get used anywhere. |
I have pushed a new I have posted a stackoverflow question on this topic: http://stackoverflow.com/questions/25215477/supress-matplotlib-figures-when-running-py-file-via-python-or-ipython-terminal Any thoughts would be most welcome. |
@sanguineturtle Can we close this? |
Ping |
@mmcky is this ready to close? |
Closing as we have some basic test infrastructure in place to run the examples. |
We have talked about writing tests, so I thought we should open up an issue to organize this effort. Below is a todo list of modules that are currently (as of 7/9/14) being imported into the main init.py file for the package. My thoughts for this issue are to organize test writing efforts so that we can track which tests have been written, which are being worked on right now, and who is working on them. This way we can write all the tests, but not duplicate work (duplicating work is already unpleasant, but when it is duplication of writing tests, the pain is magnified 😡)
I suggest that as someone starts working on tests for a particular module that they say so in a comment on this issue so that we don't end up duplicating work.
Any other suggestions for how we should proceed are encouraged.
The text was updated successfully, but these errors were encountered: