GEOS-ESM / GEOS_OceanGridComp

GEOS Ocean Gridded Component
Apache License 2.0
3 stars 2 forks source link

House cleaning of configurations #47

Closed sanAkel closed 10 months ago

sanAkel commented 11 months ago

This PR:

mathomp4 commented 11 months ago

The coupled tests are not happy. Do I need to change things at low res in my BCs set?

zhaobin74 commented 11 months ago

The coupled tests are not happy. Do I need to change things at low res in my BCs set?

Maybe because "woa18_decav_t01_04.nc" file is not present? It needs to be linked/copied in gcm_run.j.

mathomp4 commented 11 months ago

The coupled tests are not happy. Do I need to change things at low res in my BCs set?

Maybe because "woa18_decav_t01_04.nc" file is not present? It needs to be linked/copied in gcm_run.j.

Ah. Okay. I'm on leave until next week, so I'll try and get things fixed up then. @zhaobin74 where is this file located on discover so I can include it in TinyBCs?

@mathomp4 I saw your message and trying to fix it-- I have no idea where Yury kept the files I gave him. What he has in "oceanval" directory ( /home/yvikhlia/oceanval/verification/WOA18 ) is useless (because data in there goes to 1500 m only).

Just finished recreating the files and uploaded, but discover is having hiccups. Enjoy your break and see you week after.

sanAkel commented 11 months ago

@mathomp4, @zhaobin74 and all,

Please do following in your gcm_run.j (for now). Later, I'll coordinate with @biljanaorescanin to have these files populated in the BCs path .and. patch the App; see below(*).

set             FILE = linkbcs
/bin/rm -f     $FILE 
cat << _EOF_ > $FILE
#!/bin/csh -f
...
...
...

cp /discover/nobackup/sakella/woa18/1-deg/WOA18_decav_TPOTS_monthly_fulldepth_01.nc INPUT

 /bin/ln -sf $OBCSDIR/cice/kmt_cice.bin .
 /bin/ln -sf $OBCSDIR/cice/grid_cice.bin .

_EOF_

That gets a _fulldepth WOA18 T and S file into scratch/INPUT.

(*) Reason for not patching the App at this moment is so as to get @biljanaorescanin's help (as needed) in order to copy files from ⬆️ path and purge whats not needed.

I can successfully run with my WOA18_decav_TPOTS_monthly_fulldepth_01.nc, so I expect you can as well.

@zhaobin74 For the 540x458 configuration you are using, I assume you are using woa18_04_initial_conditions.nc in which case, you can copy from /discover/nobackup/sakella/MOM6-GFDL/MOM6-testing/Gustavo.Marques/datasets/woa18_04_initial_conditions.nc

mathomp4 commented 11 months ago

@sanAkel I can't see the file (rw only for you), but... 900MB. Ouch.

Maybe once I can see inside it, I can work with you to prune it down for TinyBCs (i.e., if it's a daily file or something, we can pull out just a few days).

sanAkel commented 11 months ago

That (cut size) can be done. See in about an hour. Needs to be trimmed a lot!

sanAkel commented 10 months ago

@mathomp4 I have undrafted this PR (again)! I don't want to hold off this clean up anymore (until I get a coarser WOA initial conditions. When we spoke last week, you said that the file size is ok.)

@zhaobin74 can you please review this PR.

Thanks!

mathomp4 commented 10 months ago

Yay it's working!

mathomp4 commented 10 months ago

Yay it worked! :)

zhaobin74 commented 10 months ago

Yay it worked! :) @mathomp4, you mean it worked for c12x5 setup, right?

mathomp4 commented 10 months ago

Yay it worked! :) @mathomp4, you mean it worked for c12x5 setup, right?

Yes. Mainly that the CI didn't crash like before. The ocean could be boiling for all I know, but it ran for 6 hours! 😄

zhaobin74 commented 10 months ago

Yay it worked! :) @mathomp4, you mean it worked for c12x5 setup, right?

Yes. Mainly that the CI didn't crash like before. The ocean could be boiling for all I know, but it ran for 6 hours! 😄

Thanks, @mathomp4. I can test the other two configs, 1440x1080 and 540x458.

zhaobin74 commented 10 months ago

First, 1440x1080 didn't work out of the box.

cp /discover/nobackup/sakella/woa18/1-deg/WOA18_decav_TPOTS_monthly_fulldepth_01.nc INPUT

failed with permission denied error.

I see there is another (smaller) file /discover/nobackup/sakella/woa18/1-deg/WOA18_decav_TPOTS_mon01_fulldepth_01.nc. Using this file worked.

Is this one supposed to be used? In that case, 1440x1080/MOM_input should be updated to this new file.

zhaobin74 commented 10 months ago

540x458 config works out of the box with /discover/nobackup/sakella/MOM6-GFDL/MOM6-testing/Gustavo.Marques/datasets/woa18_04_initial_conditions.nc

sanAkel commented 10 months ago

/discover/nobackup/sakella/woa18/1-deg/WOA18_decav_TPOTS_monthly_fulldepth_01.nc

@zhaobin74 thanks for testing both configurations:

  1. 1440x1080 I fixed the file permission issue. Sorry, somehow I overlooked it! Whenever I scp files from local mac ➡️ discover, this happens.

-rw-r--r-- 1 sakella s2640 908M Oct 4 18:24 /discover/nobackup/sakella/woa18/1-deg/WOA18_decav_TPOTS_monthly_fulldepth_01.nc

Reg:

I see there is another (smaller) file /discover/nobackup/sakella/woa18/1-deg/WOA18_decav_TPOTS_mon01_fulldepth_01.nc. Using this file worked. Is this one supposed to be used?

NO. I want to use for this coarse- resolution tests only.

In that case, 1440x1080/MOM_input should be updated to this new file.

⬆️ Not needed since I fixed the file permissions.

  1. 540x458

config works out of the box 👍 Thanks!

sanAkel commented 10 months ago

@zhaobin74 can You please retry 1440x1080?

sanAkel commented 10 months ago

@mathomp4 can you approve the PR since it seems to have worked for you.

mathomp4 commented 10 months ago

@mathomp4 can you approve the PR since it seems to have worked for you.

Approved. Though without Yury, once @zhaobin74 approves I might have to use superpowers to get it in.

zhaobin74 commented 10 months ago

@zhaobin74 can You please retry 1440x1080?

After the permission fix, 1440x1080 config is working:smiley:

sanAkel commented 10 months ago

@mathomp4 can you approve the PR since it seems to have worked for you.

Approved. Though without Yury, once @zhaobin74 approves I might have to use superpowers to get it in.

Yes @mathomp4 please use them! 😄