Add new skew hadamard matrices#34985
Conversation
|
The branch from #34848 was merged into 10.0.beta0. Now there are merge conflicts that must be resolved. |
|
@wdjoyner - you might look at the freshly built docs here: https://deploy-preview-35059--sagemath-tobias.netlify.app/ |
|
by the way, for |
this is a rebase over sagemath#34985
|
PDF docs don't build |
|
our usual unicode hiccup: |
|
So this needs yet another |
|
I don't know how to fix this in a regular way; i.e. I added --- a/src/sage_docbuild/conf.py
+++ b/src/sage_docbuild/conf.py
@@ -620,6 +620,7 @@ latex_elements['preamble'] = r"""
\DeclareUnicodeCharacter{256E}{+}
\DeclareUnicodeCharacter{256F}{+}
\DeclareUnicodeCharacter{2570}{+}
+ \DeclareUnicodeCharacter{030C}{\v{c}} % č
\fi
\let\textLaTeX\LaTeXbut it had no effect. Am I missing something, @vbraun ? |
|
hmm, |
|
trying to use |
|
@jhpalmieri - have you seen this sort of trouble before? How about switching to |
|
with the last change, it works. |
It sounds like this has been resolved, but I'll reply anyway. I've never used |
|
Documentation preview for this PR is ready! 🎉 |
gh-35059: Add Hadamard matrices up to order 1000 <!-- ^^^^^ Please provide a concise, informative and self-explanatory title. Don't put issue numbers in there, do this in the PR body below. For example, instead of "Fixes #1234" use "Introduce new method to calculate 1+1" --> ### 📚 Description Added new construction for (non skew) Hadamard matrices of order up to 1000. The only orders <1000 for which no construction is known are 668, 716, 892. In particular, the following has been changed: - Added more Williamson type matrices (the way these matrices are stored has been changed to be more compact) - Implemented a function to construct non-skew Supplementary difference Sets, as well as a function to construct Hadamard matrices from these sets - Added Miyamoto construction Note that only commits starting from 15e1b50 are new, the previous ones where created in #34985. <!-- Describe your changes here in detail --> <!-- Why is this change required? What problem does it solve? --> <!-- If it resolves an open issue, please link to the issue here. For example "Closes #1337" --> ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> - [x] I have made sure that the title is self-explanatory and the description concisely explains the PR. - [x] I have linked an issue or discussion. - [x] I have created tests covering the changes. - [x] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open pull requests that this PR logically depends on --> <!-- - #xyz: short description why this is a dependency - #abc: ... --> URL: #35059 Reported by: MatteoCati Reviewer(s): david joyner, Dima Pasechnik
7 participants
Fixes #34848.