Skip to content
This repository has been archived by the owner on Aug 19, 2023. It is now read-only.

Aer needs its own documentation page #1508

Closed
1ucian0 opened this issue Jun 2, 2022 · 4 comments · Fixed by Qiskit/qiskit-aer#1589
Closed

Aer needs its own documentation page #1508

1ucian0 opened this issue Jun 2, 2022 · 4 comments · Fixed by Qiskit/qiskit-aer#1589
Labels
documentation Something is not clear or error in description

Comments

@1ucian0
Copy link
Member

1ucian0 commented Jun 2, 2022

With the removal of Aer from the metapackage, a documentation page for its own is needed.

Once Aer is removed from the metapackage, the module will be at the same level of other application modules and it will require its own https://qiskit.org/documentation/aer/ in the same way that https://qiskit.org/documentation/nature/ exists, for example.

I suggest creating https://qiskit.org/documentation/aer/ silently with no change in the metapackage documentation first.

@1ucian0 1ucian0 added the documentation Something is not clear or error in description label Jun 2, 2022
@mtreinish
Copy link
Member

mtreinish commented Jun 2, 2022

Just for clarification that aer pr does not remove aer from the metapackage. It is creating a new python namespace. The term package there is in the strict python terminology which is confusing for most people: https://docs.python.org/3/glossary.html#term-package

As for the issue this is kinda premature. Aer is still part of the metapackage for now. This issue is also primarily not something we fix in the metapackage repository as publishing self contained docs is done in the aer repository.

@mtreinish
Copy link
Member

I apologize "premature" probably wasn't the correct way to describe this. Upon reflection renaming the qiskit-aer package from qiskit.providers.aer -> qiskit_aer is going to have some complications for link preservation. Since when we merge Qiskit/qiskit-aer#1526 the aer docs no longer refer to the old location. We'll probably need to use sphinx-reredirects to redirect from the old path to the new one (which likely means indexing everything and redirecting manually, see: https://github.com/Qiskit/retworkx/blob/main/docs/source/conf.py#L125-L128 and https://github.com/Qiskit/retworkx/blob/main/docs/source/sources.txt for what I did in retworkx to go from retworkx/stubs/* -> retworkx/apiref/*). I think having aer publish a standalone documentation page for it's next release sounds like the best course of action. That's primarily an aer issue, but doing the cleanup for the metapackage side of the docs we can track here.

mtreinish added a commit to mtreinish/qiskit that referenced this issue Aug 31, 2022
As part of the upcoming aer 0.11.0 release the project is moving to a
separate package/namespace and will no longer be part of `qiskit.*`.
This means the old documentation pages that existed for aer will no
longer be built because autodoc for the path will no longer be from
qiskit.providers.aer. To address this in the next metapackage release
this commit adds redirects from the existing aer documentation pages to
the new aer documentation.

Related to Qiskit#1508
@mtreinish
Copy link
Member

I've pushed up Qiskit/qiskit-aer#1589, Qiskit/qiskit#1595, and Qiskit/qiskit#1596 which should address this. Aer will move to a standalone docs build and the main qiskit docs will have redirects from the old aer pages to the new build

jakelishman pushed a commit that referenced this issue Sep 15, 2022
* Redirect aer docs to standalone aer page

As part of the upcoming aer 0.11.0 release the project is moving to a
separate package/namespace and will no longer be part of `qiskit.*`.
This means the old documentation pages that existed for aer will no
longer be built because autodoc for the path will no longer be from
qiskit.providers.aer. To address this in the next metapackage release
this commit adds redirects from the existing aer documentation pages to
the new aer documentation.

Related to #1508

* Switch to redirect in docs tree

To ease the migration of docs around the aer release this commit shifts
the redirect from the old import path to the new import path in the same
docs tree. This is needed for two reasons, while aer is still part of the
metapackage we should document it as such, and secondly the current state
of the Aer documentation isn't really for standalone hosting the docs
aren't really sufficient on their own yet. So until both situations are
resolved we should keep aer as part of the metapackage docs.
@1ucian0
Copy link
Member Author

1ucian0 commented Mar 16, 2023

closed by Qiskit/qiskit-aer#1589

@1ucian0 1ucian0 closed this as completed Mar 16, 2023
jakelishman pushed a commit to jakelishman/qiskit-terra that referenced this issue Aug 11, 2023
* Redirect aer docs to standalone aer page

As part of the upcoming aer 0.11.0 release the project is moving to a
separate package/namespace and will no longer be part of `qiskit.*`.
This means the old documentation pages that existed for aer will no
longer be built because autodoc for the path will no longer be from
qiskit.providers.aer. To address this in the next metapackage release
this commit adds redirects from the existing aer documentation pages to
the new aer documentation.

Related to Qiskit/qiskit-metapackage#1508

* Switch to redirect in docs tree

To ease the migration of docs around the aer release this commit shifts
the redirect from the old import path to the new import path in the same
docs tree. This is needed for two reasons, while aer is still part of the
metapackage we should document it as such, and secondly the current state
of the Aer documentation isn't really for standalone hosting the docs
aren't really sufficient on their own yet. So until both situations are
resolved we should keep aer as part of the metapackage docs.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Something is not clear or error in description
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants