WIP: change the setup for path_diversity project

berkeley-stat222 / mousestyles

2016 final project

http://berkeley-stat222.github.io/mousestyles/

BSD 2-Clause "Simplified" License

2 stars 33 forks source link

WIP: change the setup for path_diversity project #190

Closed jJasonWang closed 8 years ago

jJasonWang commented 8 years ago

See the last checklist in #122.

In this pull request:

I changed the way to import the function from the package. The way to call the function will be path_diversity.function. (import the module by from mousestyles import path_diversity)
I created a pytest.ini to avoid the UnusedImport error for style checking.
I also changed the tests and function name to make it more robust and consistent. (for path_index and path_filter)

@aksam-ahmad, @alannaiverson, @jcpalumbo3, @sicihuang, @togawa28. Please help me review the file you created. All the files which used the functions in path_diversity project should have the from mousestyles import path_diversity. Also, all the function should be called by path_diveristy.function. Thanks.

@peterasujan, can you help me double check the setup for pytest.ini is correct? Thanks.

coveralls commented 8 years ago

Coverage increased (+0.08%) to 77.42% when pulling 279f7754e86574d86d7491adb58ae402a1623788 on jJasonWang:path_diversity-issue122 into 8089c170e5c631a139c2c8f920d1073d3c5923c9 on berkeley-stat222:master.

coveralls commented 8 years ago

Coverage increased (+0.08%) to 77.42% when pulling abbc93c287bb5b28e47a5cae43a9b8a3f2e59c7b on jJasonWang:path_diversity-issue122 into dfdf5c339c7cc2b965f2bb2354c1b9a3b1a2ab51 on berkeley-stat222:master.

coveralls commented 8 years ago

Coverage increased (+0.08%) to 78.262% when pulling e51b1ac5b8f3eba34d81c3155e1b774ce0bd5fa1 on jJasonWang:path_diversity-issue122 into 9a6534ee7f84e28241b63a77e9d45b51fd3dbf4c on berkeley-stat222:master.

jJasonWang commented 8 years ago

@jarrodmillman. Can you help me check this pull request? I might not be able to work on this until 5 pm today. I think that if travis-CI and testing pass, we are ready to merge it.

It is just the setup for path diversity project. I changed it to the way the behavior project setup.

jarrodmillman commented 8 years ago

@jJasonWang Sure, I'll take a look.

coveralls commented 8 years ago

Coverage increased (+0.08%) to 78.262% when pulling 9e05bd4bcba2f3f6664ed53287b71bb0afe2d9d3 on jJasonWang:path_diversity-issue122 into 7c257a82cb04cf60fc3cdb11ab8de4a0ab861f58 on berkeley-stat222:master.