QuantEcon / lecture-python

Source files for https://python.quantecon.org
BSD 3-Clause "New" or "Revised" License
61 stars 21 forks source link

[19_short_path] fix issue 61 mentioned in repo lecutre-python.myst #363

Closed shlff closed 3 years ago

shlff commented 3 years ago

Hi @jstac and @mmcky , as discussed with @mmcky in lecture-python.myst's issue #61, this PR fixes that issue by

github-actions[bot] commented 3 years ago

🚀 Deployed on https://6021c8faefcf273378535863--practical-lalande-b89631.netlify.app

mmcky commented 3 years ago

thanks @shlff this is a good summary of this issue in rst.

https://stackoverflow.com/questions/9645321/insert-a-link-into-bold-text-in-restructuredtext

We really do want the full text weighted directed graph to be of the same type so perhaps the "best" solution is to get rid of bold on weighted and just rely on a link to group this text together (rather than bold formatting).

shlff commented 3 years ago

Thanks for your advice, @mmcky . I modified this PR to get rid of bold on weighted and only attach a link (https://en.wikipedia.org/wiki/Directed_graph) to directed graph (there is no wiki or other appropriate links for weighted directed graph).

mmcky commented 3 years ago

thanks @shlff. It would be good to group the weighted in with the link. It should be:

thanks

I see what your saying -- thanks @shlff

mmcky commented 3 years ago

once the PDF check finishes I will merge this.