Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug in array syntax in ice_itd.F90 #400

Merged
merged 1 commit into from
Oct 29, 2015

Conversation

susburrows
Copy link
Contributor

Fixes an error encountered when compiling with intel DEBUG mode on
edison. In this bug, the code fails at runtime in ice_itd.F90 with a
message stating:

"Subscript #1 of the array ATRCR has value 1 which is greater than the
upper bound of 0"

This is fixed as described on DiscussCESM
(https://bb.cgd.ucar.edu/error-running-compset-fsdwsf-ert), by
removing the explicit array syntax in the call to compute_tracers().

Fixes #339

[BFB]

Fixes an error encountered when compiling with intel DEBUG mode on
edison.  In this bug, the code fails at runtime in ice_itd.F90 with a
message stating:

"Subscript #1 of the array ATRCR has value 1 which is greater than the
upper bound of 0"

This is fixed as described on DiscussCESM
(https://bb.cgd.ucar.edu/error-running-compset-fsdwsf-ert), by
removing the explicit array syntax in the call to compute_tracers().

Fixes #339
singhbalwinder added a commit that referenced this pull request Oct 28, 2015
#400)

Fix bug in array syntax in ice_itd.F90

Fixes an error encountered when compiling with intel DEBUG mode on
edison. In this bug, the code fails at runtime in ice_itd.F90 with a
message stating:

"Subscript #1 of the array ATRCR has value 1 which is greater than the
upper bound of 0"

This is fixed as described on DiscussCESM
(https://bb.cgd.ucar.edu/error-running-compset-fsdwsf-ert), by
removing the explicit array syntax in the call to compute_tracers().

Fixes #339
[BFB]
@singhbalwinder
Copy link
Contributor

Pushed to next. Waiting for nightlies.

@jgfouca
Copy link
Member

jgfouca commented Oct 28, 2015

BFB?

@susburrows
Copy link
Contributor Author

should be BFB. I thought we were only adding a flag if it is non-BFB...?

@jgfouca
Copy link
Member

jgfouca commented Oct 29, 2015

Nope, you should always add [BFB]

@susburrows
Copy link
Contributor Author

OK, @singhbalwinder , can you make sure this is added when you merge to master? thank you...

@singhbalwinder
Copy link
Contributor

Okay. I will do that.

@singhbalwinder
Copy link
Contributor

I just checked and for some reason I have included BFB in my commit to the next branch.

@rljacob
Copy link
Member

rljacob commented Oct 29, 2015

That's ok as long as its on the master merge commit.

@singhbalwinder singhbalwinder merged commit ab10ca1 into master Oct 29, 2015
singhbalwinder added a commit that referenced this pull request Oct 29, 2015
Fix bug in array syntax in ice_itd.F90

Fixes an error encountered when compiling with intel DEBUG mode on
edison. In this bug, the code fails at runtime in ice_itd.F90 with a
message stating:

"Subscript #1 of the array ATRCR has value 1 which is greater than the
upper bound of 0"

This is fixed as described on DiscussCESM
(https://bb.cgd.ucar.edu/error-running-compset-fsdwsf-ert), by
removing the explicit array syntax in the call to compute_tracers().

Fixes #339

[BFB]
@singhbalwinder singhbalwinder deleted the susburrows/atm/bugfix-ice_itd-array-syntax branch October 29, 2015 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding a DEBUG=TRUE parameter to a case on Edison causes the case to crash.
4 participants