Merged
Conversation
Added 'accelerator="cpu"' to pipeline initialization in all relevant notebooks for consistent device selection. Updated pyproject.toml to require matchcake>=0.1.2 and added jupyter and notebook to dev dependencies. Also replaced SptmfRxRx and SptmFHH with CompRxRx and CompHH in the deep learning notebook, and updated execution metadata and outputs.
Removed execution counts, outputs, and execution metadata from all code cells in nif_deep_learning.ipynb. Also reduced AutoML iterations and max time for faster runs.
…nvention Add CPU accelerator option and update dependencies
The workflow now runs 'uv lock' after bumping the package version and adds 'uv.lock' to the commit. This ensures the lock file stays in sync with version changes. Also, the PyPI publish step was moved to the end of the workflow.
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified FilesNo covered modified files...
|
This file contains hidden or 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
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.
Description
This pull request updates several Jupyter notebooks and the project dependencies to improve compatibility, reproducibility, and hardware specification. The most significant changes include explicitly setting the computation accelerator to CPU in training pipelines, updating the
matchcakepackage version, and refactoring quantum operation imports and usage in the deep learning notebook.Dependency updates:
matchcakepackage requirement inpyproject.tomlfrom>=0.0.4to>=0.1.2to ensure compatibility with newer features and fixes.jupyterandnotebookas development dependencies inpyproject.tomlfor improved local notebook support.Notebook improvements:
accelerator="cpu"in the pipeline initialization fornotebooks/nif_deep_learning.ipynb,notebooks/ligthning_pipeline_tutorial.ipynb, andnotebooks/automl_pipeline_tutorial.ipynbto ensure CPU usage for training. [1] [2] [3] [4]notebooks/nif_deep_learning.ipynbfromSptmfRxRxandSptmFHHtoCompRxRxandCompHHfor improved clarity and compatibility with the updatedmatchcakepackage. [1] [2]Minor notebook adjustments:
automl_iterationsandinner_max_timeparameters innotebooks/nif_deep_learning.ipynbto enable faster tutorial runs.outputsandexecution_countfields to several notebook cells for consistency with Jupyter notebook standards. [1] [2] [3]Checklist
Please complete the following checklist when submitting a PR. The PR will not be reviewed until all items are checked.
Make sure that the tests passed and the coverage is
sufficient by running
pytest tests --cov=src --cov-report=term-missing.You can do this by running
black src tests.You can do this by running
isort src tests.You can do this by running
mypy src tests.