Skip to content

Add high-level abstract coupling restriction#101

Merged
pjaap merged 10 commits intomasterfrom
feature/high-level-coupling
Jan 28, 2026
Merged

Add high-level abstract coupling restriction#101
pjaap merged 10 commits intomasterfrom
feature/high-level-coupling

Conversation

@pjaap
Copy link
Copy Markdown
Member

@pjaap pjaap commented Jan 20, 2026

This may be much more user friendly

@chmerdon
Copy link
Copy Markdown
Member

Nice, how about using the normal vectors on the boundary regions to define the opposite function (assuming that the boundary regions are hyperplanes not necessarily orthogonal to an axis)?

@pjaap
Copy link
Copy Markdown
Member Author

pjaap commented Jan 21, 2026

Uh, this is a brilliant idea. For this high-level restriction I assume that the boundary regions are hyperplanes and that the axis is orthogonal to the boundary regions.

For the general case, the concrete constructor with the coupling matrix is still there and should stay.

I'll push an update today :)

This may be much more user friendly
@pjaap pjaap force-pushed the feature/high-level-coupling branch from 00f19fd to cb2e992 Compare January 21, 2026 11:22
@pjaap
Copy link
Copy Markdown
Member Author

pjaap commented Jan 21, 2026

Alright. Now there is no need for an axis any more.

The give_opposite! function is pretty tricky from a mathematical POV, but it can be implemented efficient. This does not seem to slow down the computation of the matrix.

@pjaap
Copy link
Copy Markdown
Member Author

pjaap commented Jan 27, 2026

I am happy now. New example, new features :)

@chmerdon
Copy link
Copy Markdown
Member

Nice! I think it can be merged. (I am in favor of renaming PeriodicBoundary to PeriodicElasticity in Example212 and Example312, though)

@pjaap
Copy link
Copy Markdown
Member Author

pjaap commented Jan 27, 2026

like this?

@pjaap
Copy link
Copy Markdown
Member Author

pjaap commented Jan 28, 2026

damn, classic mistake with the filename. We should definitely have an "exclude" list an parse all Example*.jl files in the examples folder.

But now merge?

@chmerdon
Copy link
Copy Markdown
Member

Ok, ready to merge I think

@pjaap pjaap merged commit 351b311 into master Jan 28, 2026
8 checks passed
@pjaap pjaap deleted the feature/high-level-coupling branch January 28, 2026 10:18
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

Successfully merging this pull request may close these issues.

2 participants