cloudy-astrophysics / bug-tracker-migration-test

Trial run for importing the nublado.org Trac tickets as GitHub issues
0 stars 0 forks source link

continuum lowering problems (trac #384) #386

Open cloudy-bot opened 7 years ago

cloudy-bot commented 7 years ago

reported by: @CloudyLex

In the C17 review we had to disale continuum lowering for the ionization to go to the correct Saha slope at very high densities. This is unphysical. the current treatment of continuum lowering is incomplete since the collisional and photoionization rates need to be extended down to the lowered continuum.

I disabled continuum lowering as follows:

===================================================================
--- iso.cpp (revision 11622)
+++ iso.cpp (working copy)
@@ -19,7 +19,7 @@
    for( long ipISO=ipH_LIKE; ipISO<NISO; ipISO++ )
    {
        /* option to disable continuum lowering */
-       lgContinuumLoweringEnabled[ipISO] = true;
+       lgContinuumLoweringEnabled[ipISO] = false;

        /* flag set by compile he-like command, says to regenerate table of recombination coef */
        lgCompileRecomb[ipISO] = false;

and the test suite was mostly clean but there problems in auto:

cloud9 auto:cat serious.txt
limit_lte_h_t50_coll.out:  BIG BOTCHED MONITORS!!!   Big Botched Monitors!!!
limit_lte_he1_nomole_ste.out:  W-Heating - cooling mismatch =3.41e+01%. What's wrong????
limit_lte_he1_nomole_ste_nocoll.out:  W-Heating - cooling mismatch =2.83e+01%. What's wrong????

 PROBLEM  ConvFail 1, Temp not converged itr 2 zone 0 fnzone 1.71 Te=4.9912e+04
 PROBLEM  ConvFail 2, Temp not converged itr 2 zone 0 fnzone 1.92 Te=5.0008e+04  PROBLEM  ConvFail 1, Temp not converged itr 2 zone 1 fnzone 4.33 Te=4.9955e+04
 ####  1  Te:4.995E+04 Hden:1.000E+01 Ne:3.292E+18 R:1.000E+30 R-R0:9.073E-12 dR

Migrated from https://www.nublado.org/ticket/384

{
    "status": "new",
    "changetime": "2019-02-04T12:10:21Z",
    "_ts": "1549282221449479",
    "description": "In the C17 review we had to disale continuum lowering for the ionization to go to the correct Saha slope at very high densities.  This is unphysical.  the current treatment of continuum lowering is incomplete since the collisional and photoionization rates need to be extended down to the lowered continuum.\n\nI disabled continuum lowering as follows:\n{{{\n===================================================================\n--- iso.cpp\t(revision 11622)\n+++ iso.cpp\t(working copy)\n@@ -19,7 +19,7 @@\n \tfor( long ipISO=ipH_LIKE; ipISO<NISO; ipISO++ )\n \t{\n \t\t/* option to disable continuum lowering */\n-\t\tlgContinuumLoweringEnabled[ipISO] = true;\n+\t\tlgContinuumLoweringEnabled[ipISO] = false;\n\n \t\t/* flag set by compile he-like command, says to regenerate table of recombination coef */\n \t\tlgCompileRecomb[ipISO] = false;\n}}}\n\nand the test suite was mostly clean but there problems in auto:\n{{{\ncloud9 auto:cat serious.txt\nlimit_lte_h_t50_coll.out:  BIG BOTCHED MONITORS!!!   Big Botched Monitors!!!\nlimit_lte_he1_nomole_ste.out:  W-Heating - cooling mismatch =3.41e+01%. What's wrong????\nlimit_lte_he1_nomole_ste_nocoll.out:  W-Heating - cooling mismatch =2.83e+01%. What's wrong????\n}}}\n\n{{{\n\n PROBLEM  ConvFail 1, Temp not converged itr 2 zone 0 fnzone 1.71 Te=4.9912e+04\n PROBLEM  ConvFail 2, Temp not converged itr 2 zone 0 fnzone 1.92 Te=5.0008e+04  PROBLEM  ConvFail 1, Temp not converged itr 2 zone 1 fnzone 4.33 Te=4.9955e+04\n ####  1  Te:4.995E+04 Hden:1.000E+01 Ne:3.292E+18 R:1.000E+30 R-R0:9.073E-12 dR\n}}}\n",
    "reporter": "gary",
    "cc": "",
    "resolution": "",
    "time": "2017-03-09T19:05:53Z",
    "component": "ionization convergence",
    "summary": "continuum lowering problems",
    "priority": "major",
    "keywords": "",
    "version": "trunk",
    "milestone": "c19 branch",
    "owner": "nobody",
    "type": "defect - wrong answer"
}
cloudy-bot commented 7 years ago

Milestone renamed