Skip to content

Conversation

@SimonPorr
Copy link

Automatic usage of Tripyview in ESM_tools FESOM 2.6 model, including setting up the necessary environments and the installation of Tripyview tool.

@SimonPorr SimonPorr requested a review from JanStreffing March 17, 2025 13:13
@JanStreffing JanStreffing requested a review from pgierz March 17, 2025 13:16
Copy link
Member

@pgierz pgierz left a comment

Choose a reason for hiding this comment

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

Looks great. I'll try this out and see how it goes, if there are no strange errors, I'll approve.

We might consider also adding a section in the handbook. What do you think, @mandresm?

save_path: "${base_dir}/${expid}/analysis/Tripyview"

which_clim: 'phc3'
clim_path: '/work/ab0246/a270092/input/fesom2/hydrography/phc3.0/phc3.0_annual.nc'
Copy link
Member

@pgierz pgierz Mar 17, 2025

Choose a reason for hiding this comment

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

Not sure about this one, it is a hard-coded path from a specific user on Levante. Maybe this should instead be something like:

Suggested change
clim_path: '/work/ab0246/a270092/input/fesom2/hydrography/phc3.0/phc3.0_annual.nc'
clim_path: '${fesom.pool}/hydrography/phc3.0/phc3.0_annual.nc'

Not sure about the exact path, but something in the fesom configuration...

Copy link
Contributor

Choose a reason for hiding this comment

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

correct for awicm3 on levante

@JanStreffing
Copy link
Contributor

looks good from my side

@JanStreffing
Copy link
Contributor

Can we go ahead with this one? @SimonPorr has also done more work for OpenIFS automatic plots, which should also be merged.

@mandresm
Copy link
Contributor

mandresm commented Sep 29, 2025

Just so @JanStreffing knows, I have this in my radar, to take care of it soon.

@JanStreffing
Copy link
Contributor

It would be nice to have this one merged for testing the fesom 2.7 release candidate today and tomorrow.

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.

5 participants