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

Reorganize Euclidean Tensors #123

Open
cortner opened this issue Mar 11, 2022 · 3 comments
Open

Reorganize Euclidean Tensors #123

cortner opened this issue Mar 11, 2022 · 3 comments
Labels
good first issue Good for newcomers

Comments

@cortner
Copy link
Member

cortner commented Mar 11, 2022

This is to discuss whether we want to
(1) entirely move away from putting Euclidean tensors into the coupling matrix and instead just apply a transform at the end; or
(2) keep the euclidean tensors but replace the tables for the coco initial conditions with an implementation.

This is based on the following observation: there is a matrix A (easy to make explicit of course) such that x = r * A * Y1(x). This gives us in particular that a rotation matrix can be expanded as follows

Q_{ab} 
= (Q e^b)_a 
= sum_m A_am Y1m(Q e^b)
= sum_{m,j}  A_am D^1_mj(Q) Y1j(e^b)

I.e. we can explicitly express Q in terms of D^1 matrices and from here on the computation of the coupling coefficients could be automated. This can of course be extended to tensors of arbitrary order.

I think we should FOR SURE to (2), but even consider (1). What would the performance downside be? Is there a benefit to having less generality in the possible coupling coefficients?

CC @MatthiasSachs

@cortner cortner changed the title Remove the Euclidean Property Tables Reorganize Euclidean Tensors Mar 11, 2022
@MatthiasSachs
Copy link
Collaborator

I see! Is that the approach Ralf Drautz describes in his paper?

@cortner
Copy link
Member Author

cortner commented Mar 11, 2022

(1) is Ralf's approach. (2) is to stick with our approach but just a neater way to compute the coupling coefficients

@cortner
Copy link
Member Author

cortner commented Mar 11, 2022

could actually be a nice BSc or MSc project if you know somebody?

@cortner cortner added the good first issue Good for newcomers label Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants