Skip to content

Refactor: Restructure qBraid-Algorithms, update docs, and fix functionality - #66

Merged
TheGupta2012 merged 7 commits into
mainfrom
vinayswamik-Bug-Fixes
Oct 31, 2025
Merged

Refactor: Restructure qBraid-Algorithms, update docs, and fix functionality#66
TheGupta2012 merged 7 commits into
mainfrom
vinayswamik-Bug-Fixes

Conversation

@vinayswamik

@vinayswamik vinayswamik commented Sep 16, 2025

Copy link
Copy Markdown
Contributor

Motivation

This PR focuses on improving the qBraid-Algorithms library by addressing structural issues, enhancing documentation, and correcting functionality.

Summary

  • Restructured code for better organization and readability

    • Updated function names from load_program to generate_program and generate_subroutine to save_to_qasm for consistency across the documentation and implementation.
    • Adjusted example usages in the README and Jupyter notebooks to reflect the new function names.
    • Removed obsolete QASM files related to the algorithms to streamline the codebase.
  • Updated and clarified documentation

    • Enhanced documentation in various modules, including detailed descriptions for available algorithms.
    • Added formulation and algorithm steps to improve clarity and understanding.
    • Updated CSS for documentation styling and added custom styles for better presentation.
    • Included new features and key points in the overview section of the documentation.
  • Fixed functionality errors in existing implementations

  • Update Jupyter notebooks and improve HHL algorithm implementation

    • Refactored the HHL algorithm to enhance input validation and error handling.
    • Updated the QPE notebook to include improved result processing and visualization.
    • Added new example notebook for HHL algorithm.

Purpose

These changes aim to improve maintainability, usability, and overall consistency within the qBraid-Algorithms library.

Notes

Further changes and improvements will continue to be added to this PR.

@vinayswamik vinayswamik changed the title Update README.md Refactor: Restructure qBraid-Algorithms, update docs, and fix functionality Sep 16, 2025
@vinayswamik vinayswamik self-assigned this Sep 16, 2025
Comment thread docs/index.rst
Comment thread qbraid_algorithms/bernstein_vazirani/bernvaz.py
Comment thread qbraid_algorithms/qft/qft.py
Comment thread qbraid_algorithms/iqft/__init__.py
Comment thread qbraid_algorithms/qpe/phase_est.py
Comment thread qbraid_algorithms/qtran/qasm_builder.py
Comment thread qbraid_algorithms/qtran/qasm_builder.py

@TheGupta2012 TheGupta2012 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good

@TheGupta2012
TheGupta2012 merged commit e89f4ad into main Oct 31, 2025
6 of 7 checks passed
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