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

[Feature]: Auto-detect temporal frequency for MoV analysis (not sure if this is a popular feature) #1227

Open
ShihengDuan opened this issue Jan 8, 2025 · 1 comment

Comments

@ShihengDuan
Copy link

Is your feature request related to a problem?

I'm trying to calculate the monthly modes of variability with some daily sea level pressure data. It shows the error when checking the time axis:

check_monthly_time_axis
    raise ValueError(
ValueError: DATA ERROR: time is not correct!

I guess the error is because daily data has more time steps than the desired monthly data?

Describe the solution you'd like

It would be convenient to automatically convert daily (or even sub-daily? I don't know if anyone would use such simulations) into monthly data (or seasonal/annual, as we have the option to calculate these temporal frequencies in the PMP config file).

Describe alternatives you've considered

Or is it encouraged to use monthly data because most climate models output PSL at monthly frequency?

Additional context

No response

@lee1043
Copy link
Contributor

lee1043 commented Jan 8, 2025

@ShihengDuan Hi, thank you for the feature suggestion. At the moment the modes of variability code expects monthly time series as its input. If your data is daily, you will need to convert it to monthly. But I do agree that it would be convenient to have that function. I will see what I can do.

During the mean time, I recommend using xcdat or xarray to convert daily to monthly mean dataset.

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

2 participants