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

Support Floquet periodic boundary conditions #314

Merged
merged 83 commits into from
Feb 11, 2025
Merged

Conversation

simlapointe
Copy link
Contributor

@simlapointe simlapointe commented Dec 13, 2024

Add support for Floquet periodic boundary conditions and improve flexibility of periodic boundary conditions specifications.

  • Support Floquet boundary conditions by incorporating the phase delay into Maxwell's equations and solving the modified PDE with periodic boundary conditions.
  • Automatically detect periodic boundary transformations.
  • Support both translation and rotation transformations.

@simlapointe simlapointe marked this pull request as ready for review December 31, 2024 01:55
@simlapointe simlapointe requested a review from hughcars December 31, 2024 01:55
Copy link
Collaborator

@hughcars hughcars left a comment

Choose a reason for hiding this comment

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

Looking pretty good, I think there are a few larger changes to take care of first

  1. FP should be part of K, given it has no omega dependence in its construction unlike A2. This will simplify loads of things
  2. The muinv kx should be in the MaterialOperator, (doing this neatly will mean adding the ability to use transpose coefficients in the ceed context)
  3. The geodata.cpp can be shortened and simplified some, I have some initial work on doing this in hughcars/periodic-bc, but I'm sure once you get started you'll probably see even more things to do. The basic thing is a mixing assumptions on sdim and no assumptions on sdim.

Comment on lines +266 to +268
distance between them is given by the `"Translation"` vector in mesh units. In `floquet.json`,
an additional `"FloquetWaveVector"` specifies the phase delay between the donor and receiver
boundaries in the X/Y/Z directions.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Given the relative simplicity of this example, it would be great if we can derive the expected modes analytically. I suspect it'll be possible to map them to an equivalent set of modes on a full length cylinder without phase shift, at which point analytic results can be compared against like for the waveguide case.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I compared the cylindrical waveguide modes (as well as a rectangular waveguide and a nail/pillar) against Comsol and they all matched so I feel confident about it, but I'll see if I can derive analytical modes for the Floquet case.

@@ -74,7 +74,7 @@ Periodic boundary conditions on an existing mesh can be specified using the
["Periodic"](../config/boundaries.md#boundaries%5B%22Periodic%22%5D) boundary keyword. This
boundary condition enforces that the solution on the specified boundaries be exactly equal,
and requires that the surface meshes on the donor and receiver boundaries be identical up to
translation. Periodicity in *Palace* is also supported through meshes generated
translation or rotation. Periodicity in *Palace* is also supported through meshes generated
Copy link
Collaborator

Choose a reason for hiding this comment

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

How have you tested the rotational case? Might make for a cool example, thinking plausibly the waveguide cylinder could be split into wedges maybe and analyzed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tested the pure rotation with an annulus wedge and the rotation + translation case with a helical case.

Copy link
Collaborator

@hughcars hughcars left a comment

Choose a reason for hiding this comment

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

Some minor typos, and one further simplification of the coefficient handling for the floquet hermitian term then we're good to go.

@simlapointe simlapointe merged commit 0fead03 into main Feb 11, 2025
20 checks passed
@simlapointe simlapointe deleted the simlapointe/periodic-bc branch February 11, 2025 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Periodic boundary conditions
2 participants