Opened 10 years ago
Closed 9 years ago
#1288 closed Bug (fixed)
nn_ice_lim declared string but compared as integer
| Reported by: | anders | Owned by: | cbricaud |
|---|---|---|---|
| Priority: | high | Milestone: | |
| Component: | OCE | Version: | v3.6 |
| Severity: | Keywords: | BDY | |
| Cc: |
Description
nn_ice_lim is declared in OPA_SRC/BDY/bdy_oce.F90 as an array of strings (violating variable naming conventions) but compared against integer in OPA_SRC/BDY/bdydta.F90.
This makes the model impossible to compile with ice model (tested with LIM3 and r4605 of trunk) and BDY.
Commit History (1)
| Changeset | Author | Time | ChangeLog |
|---|---|---|---|
| 4608 | davestorkey | 2014-03-31T14:39:07+02:00 | Bug fixes for BDY with LIM, see ticket #1288. |
Attachments (1)
Change History (5)
comment:1 Changed 10 years ago by davestorkey
Changed 10 years ago by anders
comment:2 Changed 10 years ago by anders
I need to change a few more things in bdyice_lim.F90 to make it compile. See attached patch.
Note, with this it compiles. I am still far from running this configuration.
comment:3 Changed 9 years ago by clevy
- Owner changed from NEMO team to cbricaud
comment:4 Changed 9 years ago by cbricaud
- Resolution set to fixed
- Status changed from new to closed
in rev 5084 of trunk, nn_ice_lim has been replaced by cn_ice_lim and nn_ice_lim doesn't exist.
Note: See
TracTickets for help on using
tickets.
This is a bug. I don't have a configuration with LIM and BDY so I was unable to test this combination. I have put a fix in the code (change set 4608). Could you test it again and confirm that it works?
Thanks,
Dave