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

Streaming filter #771

Closed
wants to merge 2 commits into from
Closed

Conversation

c2xu
Copy link

@c2xu c2xu commented Dec 10, 2024

The filters and their target frequencies are no longer hard-coded. Instead, multiple filters
with tidal frequencies or arbitrary frequencies as their target frequencies can be turned on.
The filter names are specified in MOM_input and must consist of two letters/numbers. If the
name of a filter is the same as the name of a tidal constituent, then the corresponding tidal
frequency will be used as its target frequency. Otherwise, the user must specify the target
frequency. In either case, the target frequency is specified by "TIDE_${FILTER_NAME}_FREQ".

The restarting capability has also been implemented. Because the filtering is a point-wise
operation, all variables are considered as fields, even if they are velocity components.

The filters and their target frequencies are no longer hard-coded.
Instead, up to 10 filters with tidal frequencies as their target
frequencies and an unspecified number of filters with arbitrary
target frequencies can be turned on. The filter names are specified
in MOM_input and must consist of two letters/numbers. If a filter
name is the same as the name of a tidal constituent, then the
corresponding tidal frequency will be used as its target frequency.
Otherwise, the user must provide the target frequency. In either
case, the target frequency is specified by "TIDE_${FILTER_NAME}_FREQ"
in MOM_input.
@c2xu c2xu changed the title C2xu/streaming filter Streaming filter Dec 10, 2024
The restarting capability has been implemented.
@c2xu c2xu force-pushed the c2xu/streaming_filter branch from ab07004 to b5756ee Compare December 10, 2024 11:51
@c2xu c2xu marked this pull request as draft December 10, 2024 12:56
@c2xu c2xu closed this Dec 10, 2024
@c2xu
Copy link
Author

c2xu commented Dec 10, 2024

This PR has been reopened as PR #772

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.

1 participant