Small bug in ad_v2dbc_im.F

Bug reports, work arounds and fixes

Moderators: arango, robertson

Post Reply
Message
Author
kaihc
Posts: 3
Joined: Fri Mar 06, 2009 4:14 pm
Location: Norwegian Met Office R&D

Small bug in ad_v2dbc_im.F

#1 Unread post by kaihc »

I've found what appears to be a small bug in ad_v2dbc_im.F (svn revision 758). When using the ADM_DRIVER option and periodic boundary conditions I get segmentation fault. Changing

Code: Select all

ELSE IF (tl_LBC(isouth,isVbar,ng)%Shchepetkin) THEN
to

Code: Select all

ELSE IF (ad_LBC(isouth,isVbar,ng)%Shchepetkin) THEN
on line 1817 seems to fix this.

All the best,
Kai

User avatar
arango
Site Admin
Posts: 1347
Joined: Wed Feb 26, 2003 4:41 pm
Location: DMCS, Rutgers University
Contact:

Re: Small bug in ad_v2dbc_im.F

#2 Unread post by arango »

Yes, thank you.

leon
Posts: 78
Joined: Mon Mar 03, 2008 4:14 am

Re: Small bug in ad_v2dbc_im.F

#3 Unread post by leon »

I found this problem too after extensive debugging. But the bug has not been fixed yet. Hope it can be fixed soon, so people like me won't speed too much time on debugging. Thanks a lot.

kaihc wrote:I've found what appears to be a small bug in ad_v2dbc_im.F (svn revision 758). When using the ADM_DRIVER option and periodic boundary conditions I get segmentation fault. Changing

Code: Select all

ELSE IF (tl_LBC(isouth,isVbar,ng)%Shchepetkin) THEN
to

Code: Select all

ELSE IF (ad_LBC(isouth,isVbar,ng)%Shchepetkin) THEN
on line 1817 seems to fix this.

All the best,
Kai

User avatar
arango
Site Admin
Posts: 1347
Joined: Wed Feb 26, 2003 4:41 pm
Location: DMCS, Rutgers University
Contact:

Re: Small bug in ad_v2dbc_im.F

#4 Unread post by arango »

I don't know what are you talking about. I checked the code and it already have that correction! I think that you need to update. As you see below, ad_v2dbc_im.F already have the change as shown on :arrow: trac for Revision 767:
Attachments
Trac_Changeset_767.png

leon
Posts: 78
Joined: Mon Mar 03, 2008 4:14 am

Re: Small bug in ad_v2dbc_im.F

#5 Unread post by leon »

Hi Arango,

Thanks a lot for the update. My post (Jul. 9) is before the update (Jul.10). That's the reason. Thanks.

arango wrote:I don't know what are you talking about. I checked the code and it already have that correction! I think that you need to update. As you see below, ad_v2dbc_im.F already have the change as shown on :arrow: trac for Revision 767:

Post Reply