-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Version 0.8.0 #124
Merged
Version 0.8.0 #124
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
Remove versioneer
Update build system to flit_core
* Update graph.py to support set_backend function I also made a fuction to see the available backends and get the current backend * Apply private _available_backends suggestions from code review This commit contains the suggested changes regarding making the available_backends variable private, and refactoring the get_available_backends method to available_backends Co-authored-by: Rocco Meli <[email protected]> * Print warning when backend is already set + make sure we use _available_backends everywhere where possible * remove reliance on environment variables * make _validate_backend function print warning when a certain backend isn't installed * Use precommit hooks add __all__ back refactor _alias_backendDict to _alias_to_backend * Update ValueError message Co-authored-by: Rocco Meli <[email protected]> * Don't return the backend when setting it Co-authored-by: Rocco Meli <[email protected]> * Add dummy function to make mypy happy remove __all__ assignment * fist play with rustworkx backend * add documentation for backend selection * add cli backend selection * cleanup * changelog * apply @Jnelen suggestion and add warning filter * make molecule test also more robust * add rx to ci * add rustworkx to all backends tests * add back graphtool * add rx to test all backends and add verbose mode * Update test_molecule.py --------- Co-authored-by: jnelen <[email protected]> Co-authored-by: Jochem Nelen <[email protected]>
* Update .readthedocs.yml * rm duplicate key
* move backend tests * avoid cached parametrization * remove conda configs * make all tests work * remove mols in favour of fixtures * make molecules fixtures * avoid graph-tool on windows * simplify ci * mention pip explicitly * reset backend for good measure * try no space * windows * changelog * reset backend for every test * add warning test * Apply suggestions from code review * format
This reverts commit 78effb8.
* install rustworkx by default with pip * update readme * docs and pyproject toml
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.
Version 0.8.0
Date: 28/05/2024
Contributors: @RMeli
Added
Improved
Changed
pip
installations torustworkx
[PR#122 | @RMeli]pre-commit
versions [PR Update GitHub actions and pre-commit configuration #120 | @RMeli]Removed
tests/molecule.py
in favour of fixtures [PR Test all backends together in CI #118 | @RMeli]