Release nbconvert constraint

[edoaltamura]

Summary

Releases constraint in the nbconvert version due to jupyter/nbconvert#2092, which nbconvert v7.16.4 may have fixed.

Details and comments

Closes #735
Closes OkuyanBoga#51

[coveralls]

Pull Request Test Coverage Report for Build 11406103785

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 91.681%

---

Totals
Change from base Build 11253691893:	0.0%
Covered Lines:	4640
Relevant Lines:	5061

---

💛 - Coveralls

[edoaltamura]

Before merging, I just wanted to check how Nbconvert might behave as soon as we re-instate the stable tutorials in

qiskit-machine-learning/.github/workflows/main.yml

Lines 232 to 261 in 2bc61a3

	# - name: Run stable tutorials
	# env:
	# QISKIT_PARALLEL: False
	# QISKIT_DOCS_BUILD_TUTORIALS: 'always'
	# run: |
	# # clean last sphinx output
	# make clean_sphinx
	# # get current version
	# version=$(pip show qiskit-machine-learning | awk -F. '/^Version:/ { print substr($1,10), $2-1 }' OFS=.)
	# # download stable version
	# wget https://codeload.github.com/qiskit-community/qiskit-machine-learning/zip/stable/$version -O /tmp/repo.zip
	# unzip /tmp/repo.zip -d /tmp/
	# # copy stable tutorials to main tutorials
	# cp -R /tmp/qiskit-machine-learning-stable-$version/docs/tutorials/* docs/tutorials
	# # run tutorials and zip results
	# echo "earliest_version: 0.1.0" >> releasenotes/config.yaml
	# # ignore unreleased/untagged notes
	# tools/ignore_untagged_notes.sh
	# make html
	# cd docs/_build/html
	# mkdir artifacts
	# tar -zcvf artifacts/tutorials.tar.gz --exclude=./artifacts .
	# if: ${{ matrix.python-version == 3.9 && !startsWith(github.ref, 'refs/heads/stable') && !startsWith(github.base_ref, 'stable/') }}
	# shell: bash
	# - name: Run upload stable tutorials
	# uses: actions/upload-artifact@v4
	# with:
	# name: tutorials-stable${{ matrix.python-version }}
	# path: docs/_build/html/artifacts/tutorials.tar.gz
	# if: ${{ matrix.python-version == 3.9 && !startsWith(github.ref, 'refs/heads/stable') && !startsWith(github.base_ref, 'stable/') }}

These were commented out temporarily when Algorithms were merged (#817) because of the removed dependency. What do you think @woodsp-ibm?

[woodsp-ibm]

This was done to make sure that when we went from version to version there were no breaking changes - at least as far as testing with the tutorials from the last (stable) minor version was concerned. The nbconvert change you did here does revert what was done earlier. I am not sure with things in this case - to some degree its a breaking change here since now we would expect users to use the parts that did come from algorithms from ML here now instead - so if you don't install qiskit-algorithms, since ML not longer depends/needs it then those stable tutorials are not going to work right. It would seem once a release was done with this self-contained ML, ie the next version, then that would be a point after which perhaps re-enabling that would make more sense?

[edoaltamura]

Indeed, we should reinstate the stable tutorials after 0.8 is released. I was wondering if the nbconvert>7.14 install could give errors on the stable tutorials as soon as they are reactivated, but I don't see issues with the tutorials running in this round of checks. So, stable tutorials in 0.8 should also run with nbconvert>7.14 successfully.

I'll merge this for now, and we can revisit this after the 0.8 release if nbconvert breaks the stable tutorials.