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

Implement the octonions #35630

Merged
merged 4 commits into from
May 28, 2023
Merged

Implement the octonions #35630

merged 4 commits into from
May 28, 2023

Conversation

tscrim
Copy link
Collaborator

@tscrim tscrim commented May 9, 2023

📚 Description

The octonions are a nonassociative normed division algebra, one of only 4 over the reals, and used to create the 7 dimensional cross product. We implement a slight generalization with 3 parameters over an arbitrary commutative ring of characteristic not equal to 2, which allows us to also define the split-octonions as another special case.

Fixes #32940.

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

@fchapoton
Copy link
Contributor

They do not seem to appear in the documentation. For instance under the setting "non-associative algebras".

@tscrim
Copy link
Collaborator Author

tscrim commented May 15, 2023

I thought I had fixed that. Well, now it is done. (With a force push since I had to rebase it anyways...)

@fchapoton
Copy link
Contributor

There are several typos "octonian".

@fchapoton
Copy link
Contributor

another typo : "quaterions"

@tscrim
Copy link
Collaborator Author

tscrim commented May 16, 2023

Good catch(es). Thank you.

non-commutative unital 8-dimensional `R`-algebra that is a deformation
of the usual octonions, which are when `a = b = c = -1`. The octonions
were originally constructed by Graves and independently discovered by
Cayley (who due to first publishing them, they are sometimes called
Copy link
Contributor

Choose a reason for hiding this comment

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

"who due to ..., they are" does not parse

r"""
Test that ``self`` is an Hurwitz algebra.

An algebra `A` is *Hurwitz* if there exists a quadratic form `N`
Copy link
Contributor

Choose a reason for hiding this comment

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

nondegenerate? Otherwise this is vacuous

Copy link
Contributor

@fchapoton fchapoton left a comment

Choose a reason for hiding this comment

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

ok, good to go

@tscrim
Copy link
Collaborator Author

tscrim commented May 17, 2023

Thank you!

@github-actions
Copy link

Documentation preview for this PR (built with commit 4370091) is ready! 🎉

@vbraun vbraun merged commit 82a4509 into sagemath:develop May 28, 2023
@tscrim tscrim deleted the algebras/octonians branch May 29, 2023 00:20
vbraun pushed a commit that referenced this pull request Jun 21, 2023
    
<!-- Please provide a concise, informative and self-explanatory title.
-->
<!-- Don't put issue numbers in the title. Put it in the Description
below. -->
<!-- For example, instead of "Fixes #12345", use "Add a new method to
multiply two integers" -->

### 📚 Description

<!-- Describe your changes here in detail. -->
<!-- Why is this change required? What problem does it solve? -->
<!-- If this PR resolves an open issue, please link to it here. For
example "Fixes #12345". -->
<!-- If your change requires a documentation PR, please link it
appropriately. -->

"The" exceptional Jordan algebra is a 27 dimensional algebra defined as
the $3 \times 3$ self-adjoint matrices over the octonions. This is an
important Jordan algebra as it is used to construct the simple Lie
group/algebra of type $F_4$.

Fixes #32940.

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. It should be `[x]` not `[x
]`. -->

- [x] The title is concise, informative, and self-explanatory.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [x] I have created tests covering the changes.
- [x] I have updated the documentation accordingly.

### ⌛ Dependencies

- #35630 used in the construction

<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
    
URL: #35629
Reported by: Travis Scrimshaw
Reviewer(s): Frédéric Chapoton, Matthias Köppe, Travis Scrimshaw
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement octonions and split octonions
4 participants