Skip to content

Fix extracting layers from a nc cube containing more that one variable.#9095

Merged
joa-quim merged 1 commit into
masterfrom
fix-slice-extraction-when-2vars
Jul 9, 2026
Merged

Fix extracting layers from a nc cube containing more that one variable.#9095
joa-quim merged 1 commit into
masterfrom
fix-slice-extraction-when-2vars

Conversation

@joa-quim

@joa-quim joa-quim commented Jul 8, 2026

Copy link
Copy Markdown
Member

Before, this cube file with two variables: 'z' and 'bathymetry' alway…s returned the 'bathymetry' layer.

grdinterpolate tsu.nc?z -T570 -Glay570.grd

Now it works, (note the 'tsu.nc' file was created with the new nswing supplement).

Assisted-by: Sonnet 5.0

…s returned the 'bathymetry' layer.

grdinterpolate tsu.nc?z -T570 -Glay570.grd

Now it works, (note the 'tsu.nc' file was created with the new nswing supplement).

Assisted-by: Sonnet 5.0
@joa-quim joa-quim requested a review from a team July 8, 2026 17:16
@joa-quim joa-quim changed the title Fix extracting layers from a nvc cube containing more that one variable. Fix extracting layers from a nc cube containing more that one variable. Jul 9, 2026
@joa-quim joa-quim merged commit fb6ce45 into master Jul 9, 2026
10 of 14 checks passed
@joa-quim joa-quim deleted the fix-slice-extraction-when-2vars branch July 9, 2026 12:28
@Esteban82 Esteban82 added add-changelog Add PR to the changelog AI-assisted All (or most) of the code was written by Artificial Intelligence. labels Jul 9, 2026
@Esteban82

Copy link
Copy Markdown
Member

Let me know if you want me to test it.

@joa-quim

joa-quim commented Jul 9, 2026

Copy link
Copy Markdown
Member Author

I've merged this already. I did test it with the situation where the bug showed up (the nc file created by nswing).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add-changelog Add PR to the changelog AI-assisted All (or most) of the code was written by Artificial Intelligence.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants