Skip to content

Commit

Permalink
Fix critical regression bug with AR_thr and threshold
Browse files Browse the repository at this point in the history
  • Loading branch information
jimmielin committed Apr 19, 2022
1 parent 9dcca08 commit 8b31bfb
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions KPP/fullchem/gckpp_Integrator.F90
Original file line number Diff line number Diff line change
Expand Up @@ -878,7 +878,7 @@ SUBROUTINE ros_yIntegrator (Y, Tstart, Tend, T, &
DO i=1,NVAR
! Short-circuiting using SKIP is very important here.
if (.not. keepSpcActive(i) .and. &
abs(LossY(i)).lt.threshold .and. abs(Prod(i)).lt.threshold) then ! per Shen et al., 2020
abs(LossY(i)).lt.AR_thr .and. abs(Prod(i)).lt.AR_thr) then ! per Shen et al., 2020
NRMV=NRMV+1
! RMV(NRMV) = i ! not needed unless in append version.
DO_SLV(i) = .false.
Expand All @@ -894,7 +894,7 @@ SUBROUTINE ros_yIntegrator (Y, Tstart, Tend, T, &
IF (.not. keepActive) THEN
DO i=1,NVAR
! Short-circuiting using SKIP is very important here.
if (abs(LossY(i)).lt.threshold .and. abs(Prod(i)).lt.threshold) then ! per Shen et al., 2020
if (abs(LossY(i)).lt.AR_thr .and. abs(Prod(i)).lt.AR_thr) then ! per Shen et al., 2020
NRMV=NRMV+1
! RMV(NRMV) = i ! not needed unless in append version.
DO_SLV(i) = .false.
Expand Down Expand Up @@ -1329,7 +1329,7 @@ SUBROUTINE ros_yIntegratorA (Y, Tstart, Tend, T, &
DO i=1,NVAR
! Short-circuiting using SKIP is very important here.
if (.not. keepSpcActive(i) .and. &
abs(LossY(i)).lt.threshold .and. abs(Prod(i)).lt.threshold) then ! per Shen et al., 2020
abs(LossY(i)).lt.AR_thr .and. abs(Prod(i)).lt.AR_thr) then ! per Shen et al., 2020
NRMV=NRMV+1
RMV(NRMV) = i
DO_SLV(i) = .false.
Expand All @@ -1345,7 +1345,7 @@ SUBROUTINE ros_yIntegratorA (Y, Tstart, Tend, T, &
IF (.not. keepActive) THEN
DO i=1,NVAR
! Short-circuiting using SKIP is very important here.
if (abs(LossY(i)).lt.threshold .and. abs(Prod(i)).lt.threshold) then ! per Shen et al., 2020
if (abs(LossY(i)).lt.AR_thr .and. abs(Prod(i)).lt.AR_thr) then ! per Shen et al., 2020
NRMV=NRMV+1
RMV(NRMV) = i
DO_SLV(i) = .false.
Expand Down

0 comments on commit 8b31bfb

Please # to comment.