Skip to content

Resolve Arviz redirect link in Pymc3 Plot page #4333

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

CloudChaoszero
Copy link
Contributor

@CloudChaoszero CloudChaoszero commented Dec 14, 2020

Description of your problem

#4319 
Links on this page are broken. They should be changed to link to the correct arviz plotting documentation

https://docs.pymc.io/api/plots.html

The directory re-direct path originally goes to https://arviz-devs.github.io/arviz/. However, now it's https://arviz-devs.github.io/arviz/api,
based on the following comparisons, respectively

Example 1: https://arviz-devs.github.io/arviz/

plot_trace: https://arviz-devs.github.io/arviz/generated/arviz.plot_trace.html

Example 2: https://arviz-devs.github.io/arviz/api

plot_trace: https://arviz-devs.github.io/arviz/api/generated/arviz.plot_trace.html

@codecov
Copy link

codecov bot commented Dec 14, 2020

Codecov Report

Merging #4333 (b6052b2) into master (b386d94) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4333   +/-   ##
=======================================
  Coverage   87.67%   87.68%           
=======================================
  Files          88       88           
  Lines       14240    14240           
=======================================
+ Hits        12485    12486    +1     
+ Misses       1755     1754    -1     
Impacted Files Coverage Δ
pymc3/distributions/distribution.py 95.28% <0.00%> (+0.24%) ⬆️

@MarcoGorelli MarcoGorelli self-requested a review December 14, 2020 07:52
Copy link
Contributor

@MarcoGorelli MarcoGorelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just tried building the docs locally with this change, and they work - thanks @CloudChaoszero !

EDIT

see below

@MarcoGorelli
Copy link
Contributor

Actually, I just tried building the docs locally without this change, and the link to the ArviZ docs still works 🤔

@CloudChaoszero
Copy link
Contributor Author

CloudChaoszero commented Dec 14, 2020

@MarcoGorelli

Hmm, that's interesting. The original links should not have worked, and with the change, it should have fixed the redirects.

#4319 confirms the redirected links for certain functions reach an error.


I can check it out again sometime soon.

I did a make html in the docs directory.

@CloudChaoszero
Copy link
Contributor Author

CloudChaoszero commented Dec 15, 2020

[Updated w/o >]

Thanks @MarcoGorelli. I found the same thing (w/screenshot below).

Do you think the docs should be re-built?

  • Latest prod push in arviz-dev was 2 days ago, but the changes are irrelevant to this prod issue haha

If a re-built did work, should I close this PR?

@MarcoGorelli
Copy link
Contributor

Yes, rebuilding the PyMC3 docs may indeed solve the issue

If a re-built did work, should I close this PR?

Perhaps let's wait for the PyMC docs to be rebuilt?


Generic note - > on GitHub is usually used when quoting other's messages

@MarcoGorelli
Copy link
Contributor

MarcoGorelli commented Dec 21, 2020

Hey @CloudChaoszero

Looks like the links work fine with the latest version of the docs built, so I'm going to go ahead and close this. Thanks anyway for your PR, and there's lot's of other open issues you could work on if you like - any help would be much appreciated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants