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

Updated Some Links in 10_qgan_option_pricing of Finance Section #1121

Merged
merged 2 commits into from
Jan 29, 2021
Merged

Updated Some Links in 10_qgan_option_pricing of Finance Section #1121

merged 2 commits into from
Jan 29, 2021

Conversation

divshacker
Copy link
Contributor

Summary

This PR is Clearly made to update a broken link in [10_qgan_option_pricing] (https://qiskit.org/documentation/tutorials/finance/10_qgan_option_pricing.html)
There was a link which is not correct that's why it was broken now I have fixed that .
I have also run this on Latest Qiskit version

Details and comments

Please Let me know if any Changes required from my end .
Thank You

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@divshacker
Copy link
Contributor Author

@SooluThomas @nonhermitian Please have a look and let me know .
Thank you

@nonhermitian nonhermitian self-requested a review January 28, 2021 16:38
@divshacker
Copy link
Contributor Author

I think you need to merge this also as auto merge is blocked somehow . @nonhermitian
Thank You

@SooluThomas SooluThomas merged commit 33ffde2 into Qiskit:master Jan 29, 2021
manoelmarques pushed a commit to manoelmarques/qiskit-finance that referenced this pull request Mar 10, 2021
Updated Some Links in 10_qgan_option_pricing of Finance Section
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants