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

The amsCd component is wrongly capitalized #335

Closed
dpvc opened this issue Sep 4, 2019 · 6 comments
Closed

The amsCd component is wrongly capitalized #335

dpvc opened this issue Sep 4, 2019 · 6 comments
Milestone

Comments

@dpvc
Copy link
Member

dpvc commented Sep 4, 2019

The amsCd component has the wrong capitalization, so doesn't load properly with \require{amsCd} or via autoloading.

@nasosev
Copy link

nasosev commented Feb 11, 2020

@dpvc is this really fixed in 3.0.1? The file still is capitalised "amscd.js" at https://cdn.jsdelivr.net/npm/mathjax@3.0.1/es5/input/tex/extensions/amscd.js .

@dpvc
Copy link
Member Author

dpvc commented Feb 11, 2020

@nasosev, Argh! It is fixed in the source repository (the es5 directory that gets built and saved as the npm package has the properly renamed file); but when the es5 directory gets copied to the main MathJax repository, the old version needed to be removed from the repository and the new one added. I did add the new assistive-mml extension, for example, but missed this one because on MacOS, which is not case sensitive, both are the "same" name, and so they didn't show up as differences in git status. Argh!

We will need to do a v3.0.2 release to fix that. There is another issue as well, so we may make another release soon to fix those two items.

@dpvc
Copy link
Member Author

dpvc commented Feb 11, 2020

I've opened mathjax/MathJax#2326 to make sure this gets done in the main MathJax repository.

@dpvc
Copy link
Member Author

dpvc commented Feb 11, 2020

@nasosev, In the meantime, you can use

MathJax = {
  loader: {
    source: {
      '[tex]/amsCd': '[tex]/amscd'
    }
  }
};

to remap the correct capitalization to the incorrect one. This will allow you to use \require{amsCd} or load: ['[tex]/amsCd'] with the current CDN setup.

@nasosev
Copy link

nasosev commented Apr 10, 2020

@nasosev, Argh! It is fixed in the source repository (the es5 directory that gets built and saved as the npm package has the properly renamed file); but when the es5 directory gets copied to the main MathJax repository, the old version needed to be removed from the repository and the new one added. I did add the new assistive-mml extension, for example, but missed this one because on MacOS, which is not case sensitive, both are the "same" name, and so they didn't show up as differences in git status. Argh!

We will need to do a v3.0.2 release to fix that. There is another issue as well, so we may make another release soon to fix those two items.

@dpvc
FYI, the problem persists in v3.0.5: https://cdn.jsdelivr.net/npm/mathjax@3/es5/input/tex/extensions/amsCd.js does not exist but amscd.js is there.

@dpvc
Copy link
Member Author

dpvc commented Apr 10, 2020

@nasosev, oh no, not again! I thought I had taken care of that! Apparently I still didn't get it right.

We will be making a 3.1.0 release later this month or early next month, and I will see if I can't get it taken care of during that. I will rename the file right now to make sure that it is correct when the next version is released.

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

No branches or pull requests

2 participants