Skip to content
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

Update links #779

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update links #779

wants to merge 3 commits into from

Conversation

tinyboxvk
Copy link
Contributor

  • Use HTTPS where possible
  • Fix broken links

Signed-off-by: tinyboxvk <13696594+tinyboxvk@users.noreply.github.com>
Signed-off-by: tinyboxvk <13696594+tinyboxvk@users.noreply.github.com>
Signed-off-by: tinyboxvk <13696594+tinyboxvk@users.noreply.github.com>
# This tag requires that the tag USE_MATHJAX is set to YES.

MATHJAX_RELPATH = http://cdn.mathjax.org/mathjax/latest
MATHJAX_RELPATH = https://cdn.jsdelivr.net/npm/mathjax@2
Copy link
Member

Choose a reason for hiding this comment

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

NB: Functional change, potentially.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

USE_MATHJAX is currently set to NO, so I think we should be good.

Please also see below.
https://www.mathjax.org/cdn-shutting-down/
doxygen/doxygen@2c7bba0

Copy link
Contributor

Choose a reason for hiding this comment

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

https://docs.mathjax.org/en/latest/web/start.html#using-mathjax-from-a-content-delivery-network-cdn
It looks like jsdelivr is the mathjax recommended way. However, I couldn't find any recommendation from Doxygen on which version to use. The latest version seems to be 3.2 which is a rewrite with updated everything.

Which MathJax version is required by Doxygen?

Assuming Ubuntu is the most legacy-ish distro and we would potentially start to use MathJax, would MathJax 3 work with the Ubuntu 22.04 doxygen default?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ubuntu 22.04 currently ships doxygen (1.9.1-2ubuntu2). The support for MathJax 3 was added in 1.9.4 in doxygen/doxygen#9218.

Looking at the changelogs https://changelogs.ubuntu.com/changelogs/pool/universe/d/doxygen/doxygen_1.9.1-2ubuntu2/changelog, the support was not backported.

include/volk/volk_avx_intrinsics.h Show resolved Hide resolved
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.

3 participants