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

Normalizing a multivector is currently dealing with weights on degenerate blades as if they were not degenerate. Shouldn't work that way. #21

Open
adiffer opened this issue Jun 5, 2024 · 2 comments
Assignees

Comments

@adiffer
Copy link
Member

adiffer commented Jun 5, 2024

Normalization is handled at the level of the weights and probably shouldn't be because weights for blades that square to zero shouldn't be included in the coefficient sum.

@adiffer adiffer self-assigned this Jun 5, 2024
@adiffer
Copy link
Member Author

adiffer commented Jun 5, 2024

I have renamed the method to .normalizeOnVS() since it IS a kind of normalization.
I have started on another .normalize() method that makes use of what most people seem to do. They divide by the square root of the product of a multivector with its reverse. Mine will take U -> U/ sqrt(<reverse(U) * U>_0) for now. I have to take a peek at what Shirokov would say on this, though.

@adiffer
Copy link
Member Author

adiffer commented Jun 6, 2024

Unit tests for new method are passing now. Still gotta check Shirokov.

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

No branches or pull requests

1 participant