Skip to content

fixing_auto_uncomp_uncompatible_notebooks#1307

Merged
Adamg-classiq merged 3 commits intomainfrom
Fixing_adam_notebooks_uncomp
Nov 20, 2025
Merged

fixing_auto_uncomp_uncompatible_notebooks#1307
Adamg-classiq merged 3 commits intomainfrom
Fixing_adam_notebooks_uncomp

Conversation

@Adamg-classiq
Copy link
Contributor

PR Description

Some notes

  • Please make sure that the notebook runs successfully with the latest Classiq version.

  • Please make sure that you placed the files in an appropriate folder

    • And that the file names are clear, descriptive, and match the notebook content.
      • Note that we require the file names of .ipynb and .qmod to be unique across this repository.
    • Plus, please make sure that all required files are included: .qmod, .synthesis_options.json, .metadata.json
    • And that images are embedded inside the notebook, not added as external files
  • If applicable, please include link to the paper on which the notebook is based, in the notebook itself.

  • Please use rebase on your branch (no merge commits)

  • Please link this PR to the relevant issue

  • Please make sure to run pre-commit when commiting changes

    • If you're using git in the terminal, make sure to install pre-commit via running pip install pre-commit followed by pre-commit install
    • Note that Classiq runs automatic code linting. Meaning that one of the tests verifies the output of pre-commit.
    • Also note that pre-commit may minorly alter some files. Make sure to git add the changes done by pre-commit

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@Adamg-classiq Adamg-classiq changed the title fix fixing Nov 18, 2025
@Adamg-classiq Adamg-classiq changed the title fixing fixing_auto_uncomp_uncompatible_notebooks Nov 18, 2025
@Adamg-classiq Adamg-classiq force-pushed the Fixing_adam_notebooks_uncomp branch from 583f20f to f96a44d Compare November 19, 2025 15:21
Copy link
Member

@TomerGoldfriend TomerGoldfriend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please run notebook to see outputs.

@Adamg-classiq Adamg-classiq merged commit fb284f2 into main Nov 20, 2025
6 checks passed
@Adamg-classiq Adamg-classiq deleted the Fixing_adam_notebooks_uncomp branch November 20, 2025 09:07
@github-actions
Copy link

🏆 Incredible, @Adamg-classiq! You've merged your 15th PR! 🎯🎊

Your ongoing commitment to classiq-library is truly remarkable. You're a driving force in our community! 🚀
Your contributions are helping to shape the future of quantum computing! What exciting features or improvements do you envision next? 🔮

We are grateful for your dedication! 💫

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants