search

byu-dml / metalearn

BYU's python library of useable tools for metalearning
MIT License
22 stars 6 forks source link

#174 - Refactor tests module structure #175

Closed epeters3 closed 5 years ago

epeters3 commented 5 years ago

Addresses issue #174. I collapsed all the files that were in test/metalearn/metafeatures into tests, to avoid an error I was getting related to conflicting module names (tests/metalearn vs. metalearn). Also, I excluded the tests package from being installed during the top-level total package install.

epeters3 commented 5 years ago

I have verified that all the methods inside our run_tests.py are executing successfully with these changes.

bjschoenfeld commented 5 years ago

@epeters3 FYI if you use this syntax, the pull request will close the issue automatically: https://help.github.com/en/articles/closing-issues-using-keywords

epeters3 commented 5 years ago

Thanks, I didn’t know that!

On Wed, May 29, 2019 at 7:05 PM Brandon Schoenfeld notifications@github.com wrote:

@epeters3 https://github.com/epeters3 FYI if you use this syntax, the pull request will close the issue automatically: https://help.github.com/en/articles/closing-issues-using-keywords

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/byu-dml/metalearn/pull/175?email_source=notifications&email_token=AFBLEZANAZYE7I7CJTX4CUTPX4R5HA5CNFSM4HPRFGT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWRBQAQ#issuecomment-497162242, or mute the thread https://github.com/notifications/unsubscribe-auth/AFBLEZBNXIIHKJVWS4AFA6LPX4R5HANCNFSM4HPRFGTQ .