[Enhancement] Change test framework from Nose to pytest #413
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mdenker
approved these changes
May 13, 2021
|
Seems all ok, and in fact, we do need pytest capabilities in fixing a bug currently in the travis testing: The package is installed using which makes sure that the installed version of elephant is used, not local version. |
mdenker
added a commit
to INM-6/elephant
that referenced
this pull request
May 13, 2021
* Added cibuildwheel action * Added Python requirements to wheel build * Build only on 64bit machines, otherwise overflow * Removed Windows for testing, as vc is not available * Removed MacOS for testing, as -fopenmp is not available * Removed pp- (pypy) builds since they lack C. * Fixed removing pp- (pypy) builds since they lack C. * Put Macos back in. * Windows Hack * Remove vcpython alltogether, ignore 2.7 Python * Removed extra compile option, which breaks on Windows * Removed more extra compile options, which breaks on Windows * Try C++ instead of Gnu++. * Try C++ instead of Gnu++ Windows style argument. * Remove linux build while testing windows. * Remove libraries. * Differentiate Windows and Linux. * Added missing import. * Last mile: MacOS * Remove openMP lib * Remove openMP lib * Add openMP lib * More brew installs * Mac is called mac on github * Make sure C is reinstalled. * Multilib * Next try, new options * Ignore warning about void type * Update newsest fim package * Revert "Ignore warning about void type" This reverts commit 3ff6b62 * Revert to prior fim, new compiler argument. * Revert "Update newsest fim package" This reverts commit f321f77 * Definitely, gnu++17, but new try. * Try C++ * Warning message * llvm maybe? * Added apple in source * Small fixes for MacOS, but not comprehensive * Limit to Windows and Linux for now * Remove MacOS entry * Fix fix from mindlessness * Testrun * Trying to include fim.so, despite its renaming by wheels * Added newest version of original module * Reverted previous breaking change commited by accident. * Reverted package name from testing. * Test focal as CI build * Test bionic as CI build * Understand installation issue on CI -- is importing elephant importing the installed version? * Spelling error only * Try to make sure travis loads the installed elephant, not the cwd. * One step further -- which version will nosetests use? * Switch to pytest as of PR NeuralEnsemble#413
mdenker
added a commit
that referenced
this pull request
Aug 13, 2021
* bug fixed dealing with units * fixed same unit error in trial shuffling * added Bielefeld fim.so and adapted filtering and added window param to fpgrowth * removed max_occ test * further unit stuff * debugging for new fim version * enabled multithreading in fpgrowth * less verbose in spade * set equal number of bins in bin shuffling wrt spade * added tolerance to binning in spade everywhere * Added accelerated FIM algorithm sources by Florian Porrmann. * Enh/accelerated spade build (#82) * Added cibuildwheel action * Added Python requirements to wheel build * Build only on 64bit machines, otherwise overflow * Removed Windows for testing, as vc is not available * Removed MacOS for testing, as -fopenmp is not available * Removed pp- (pypy) builds since they lack C. * Fixed removing pp- (pypy) builds since they lack C. * Put Macos back in. * Windows Hack * Remove vcpython alltogether, ignore 2.7 Python * Removed extra compile option, which breaks on Windows * Removed more extra compile options, which breaks on Windows * Try C++ instead of Gnu++. * Try C++ instead of Gnu++ Windows style argument. * Remove linux build while testing windows. * Remove libraries. * Differentiate Windows and Linux. * Added missing import. * Last mile: MacOS * Remove openMP lib * Remove openMP lib * Add openMP lib * More brew installs * Mac is called mac on github * Make sure C is reinstalled. * Multilib * Next try, new options * Ignore warning about void type * Update newsest fim package * Revert "Ignore warning about void type" This reverts commit 3ff6b62 * Revert to prior fim, new compiler argument. * Revert "Update newsest fim package" This reverts commit f321f77 * Definitely, gnu++17, but new try. * Try C++ * Warning message * llvm maybe? * Added apple in source * Small fixes for MacOS, but not comprehensive * Limit to Windows and Linux for now * Remove MacOS entry * Fix fix from mindlessness * Testrun * Trying to include fim.so, despite its renaming by wheels * Added newest version of original module * Reverted previous breaking change commited by accident. * Reverted package name from testing. * Test focal as CI build * Test bionic as CI build * Understand installation issue on CI -- is importing elephant importing the installed version? * Spelling error only * Try to make sure travis loads the installed elephant, not the cwd. * One step further -- which version will nosetests use? * Switch to pytest as of PR #413 * Added authors of new FIM module and reference in new docs. * Added authors of new FIM module and reference in new docs. * Small text clarifications. * Test if entry for fim.so/pyd in MANIFEST is now redundant. * Update elephant/spade.py Co-authored-by: Alexander Kleinjohann <[email protected]> * Update elephant/spade.py Co-authored-by: Alexander Kleinjohann <[email protected]> * Added SPADE tutorial * Prevent wheel building on every push, and limit scipy version workaround * Pushed tutorial, removed file added in error * New attempt to make mybinder install requirements. * New attempt, dropping viziphant. * Avoid recursive elephant installation by viziphant in postBuild * Removed unit test that is fragile as it depends on the implementation of surrogate methods * Add viziphant to RTD environment * Typo in tutorial * Add viziphant to travis doc tests Co-authored-by: pbouss <[email protected]> Co-authored-by: stellalessandra <[email protected]> Co-authored-by: Alessandra Stella <[email protected]> Co-authored-by: Alexander Kleinjohann <[email protected]>
Merged
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The unit tests are run using
pytest(https://pytest.org) in replacement for the deprecatednose.This PR addresses #408.
Travis CI settings were updated to use the new framework.