Point [SOURCE] documents to github - #20673

This looks good. I'm surprised that there ins't a built in sphinx option for this though....

Sorry for the delay, was caught up with the non-digital world. I have addressed the comments by @QuLogic. I think now it should be fine hopefully, otherwise I can make changes np. @jklymak

Many of these links don't seem to go to the right place, though some of them do...

Are these the ones that are deprecated? While I tested it, I saw those would go to the deprecation github page. Aside from that, also add_callback confuses the function with the docstring add_callback. Any other ones? In that case, I think its the inspect functions themselves, but yeah not sure.

I only did a spot check, but starting from https://62590-1385122-gh.circle-artifacts.com/0/doc/build/html/api/axes_api.html barh works, but bar does not. fill_between and fill_betweenx do not work, for me at least (Safari). i.e. fill_between links to: https://github.com/matplotlib/matplotlib/blob/300e9cdb/lib/matplotlib/__init__.py#L5259-L5263

So now those ones you spotted are fixed. I also checked around ~20 more of those, and they look ok as well.

Edit: lib/mpl_toolkits links were not generated, now it is. I guess 3rd time the charm?

Great. Do you know why they were different?

I noticed inspect.getsourcefile(obj) was returning the filename to be lib/matplotlib/__init__.py instead of the python file name in which it was defined. Thus, I added fn.endswith('__init__.py') to get the filename like shown below and then it shows the proper filename.

if not fn or fn.endswith('__init__.py'):
    fn = inspect.getsourcefile(sys.modules[obj.__module__])

@QuLogic I added some extra Exceptions and added the rest of your suggestions.