Todo List

Global atom_level3

2 test on c checks whether collisions are possible at this temperature, should add photo excitation

2 these pops ARE NOT defined below

Global atom_oi_calc
2 following needed to get badbugs/bug5.in to work

Global AtomSeqBoron
2 use transition::Zero here

Global BREAK
2 this code looks very similar to the one in cont_pump.cpp, can they be combined?

Global cdTemp
2 this should have a last argument like cdIonFrac for whether or not weighting is wrt electron density

Global ChargTranEval

1 update ct to Kimura et al. (1996)

1 above rate not intended for very low temperatures - find ref for low-T rate, probably is 1e-9 like above

0 these should be values at 1e5 K

2 not currently used - include as deexcitation process

Global ContCreatePointers

2 this is redundant with contents of oxygen line arrays use them instead when removing this, make sure all line intensity predictions also go into oi line arrays

2 what are we trying to print here?

Global ContRate
2 - this routine is very important since it sets the pace for the calculation, and directly affects the convergence of the code. Most of the logic is very old and messy. 1) make sure all test cases have save dr 2) cat all these reasons together into one file and sort on the reason 3) discover what logic is the main pacesetter for the code 4) which are never triggered and so can be removed

Global ConvBase
0 PvH prefers test against err/10

Global CoolCarb

2 add term for protons from Rouef, E., & Le Bourlot, J. 1990, A&A, 236, 515

1 add neutral helium Staemmler, V., & Flower, D. R. 1991, J. Phys. B, 24, 2343

3 change to atom_level3

Global CoolIron
2 - ground term is actually a fix level system, the vectors are created, with pointers ipFe1_54m , ipFe1_111m, must add collision date, use larger model atom

Global CoolMagn
2 use AtomSeqBeryllium here

Global CoolNeon
2 transfer these lines

Global CoolNitr
1 update cs these to following reference: >>refer n2 cs Hudson, C.E. & Bell, K.L. 2004, MNRAS, 348, 1275 and A&A, 430, 725 they agree with Lennon & Burke >>chng 10 feb 24 ML: cs values updated with Hudson & Bell. Not sure why they were wrong.

Global CoolPhos
2 update to Tayal data, email of April 22 2003, must be published

Global CoolSili
2 - update to this reference for As >>refer Si7 As Galavis, M.E., Mendoza, C., * Zeippen, C.J. 1997, A&AS, 123, 159

Global CoolSulf
1 upgrade SV to more levels there is intercombination line at 0.7634 ryd upgrade to atomic data described in >>refer s5 cs Hudson, C.E> & Bell, K.L. 2006, A&A, 452, 1113

Global dBase_solve
0 this pointer should be cached one time, and the species removed from the list if it is not computed

Global diatomics::diatomics (const string &a, const double &e_star, const double *const abund, double(*fun)(double))
3 these should be const since cannot change, are flags

Global diatomics::init (void)

1 add this as a Lya excitation process

2 put supra thermal excitation into excitation of electronic bands

Global DoBeckert_etal
2 - insert reference

Global DoSatelliteLines
2 say where these come from

Global DoSutherland

2 - insert reference

2 - this check may also too liberal.

Global DrvContPump
2 if used, add damp as arg since calling routine probably evaluated it

Global DummyEmis
2 bring these two together.

Global Fe3_cs
2 put in temperature dependence

Global FeIILyaPump
2 change this number to speed of light.

Global ForbiddenAuls

2 find a transition probability for this 2^3P0 - 2^3P1 transition. It will require a bit of trickery to insert into the rate matrix, because of the fact that the lower level has a higher index. See discussion "Energy order within 2 3P" near the top of helike.c

2 find a transition probability for this 2^3P1 - 2^3P2 transition. It will require a bit of trickery to insert into the rate matrix, because of the fact that the lower level has a higher index. See discussion "Energy order within 2 3P" near the top of helike.c

Global FreeFreeGaunt
2 - These are liberal bounds, in final product, this ASSERT should be much more demanding.

Global GetStandardHeLines
2 - this structure is currently only used for helium itself... stuff numbers in for other elements, or drop the [nelem] dimension off of CaBLines

Global GrainCharge
2 remove gv.bin[nd]->lgChrgConverged, gv.bin[nd]->LowestPot, gv.bin[nd]->dstpotsav gv.bin[nd]->RateUp, gv.bin[nd]->RateDn; also gv.HighestIon??, HighestIonStage()??

Global GrainChargeTemp

2 this algorithm might be more efficient with Brent

2 should collisional heating/cooling be included here?

2 use something like log(ThermRatio) + log(delta) ????

2 this algorithm might be more efficient with Brent

Global GrainCollHeating
2 a self-consistent treatment for the heating by Compton recoil should be used

Global GrainElecEmis1
- add ionizations due to cosmic rays

Global GrainsInit
xray - StickElec depends on Te ???? use elec_esc_length(1.5*kTe,nd) ????

Global HeatSum

1 add part of hard heat to secondaries

2 find correct high-energy limit for these

Global HelikeTransProbSetup
2 this test is out of place, where should it go?

Global hydro_transprob
1 define quantum defects and use scqdri to calculate A's if levels are not exactly degenerate.

Global Hypergeometric2F1
2 - pick these interface values and stick with it...best results have been 0.4, 1.5

Global InitBinAugerData
xray - Compton recoil still needs to be added here

Global iso_create
2 this should be changed for j-resolved levels

Global iso_level
2 the indices for the two-photon rates must be changed for further iso sequences.

Global iso_photo
2 - hydro.lgHInducImp should depend on iso and nelem, even better - just call one gamnc and within that code check to see whether induced is important by looking at occnum near threshold

Global iso_satellite
2 say where these come from

Global iso_update_rates
2 the indices for the two-photon rates must be changed for further iso sequences.

Global lgElemsConserved
2 this assert is not passed if error made much smaller. This error should be related to a check on convergence of the molecular networks and their sum rules, with a criteria used here and there

Global lines_continuum
2 this block of lines should have nInu, InwT, InwC like main vector of continuum points

Global lines_setup

2 initialization already done at this point

1 streamline all of this, using transition::Zero and then setting dangerously large negative numbers.

Global mole_h2_grain_form
2 - put in actual composition dependent Tad - this is only valid for bare surfaces - not ice - for ice Tad is 555K hmi.Tad is binding energy expressed as a temperature note that hmi.Tad is set to 800. in zero tau_nu the first equation in section 2.5 equation one paragraph before equation 2 at low grain temperatures all end in para, J=0

Global mole_h_reactions

2 following always true, why? either remove test or use it - it is here to save time - this step routine is called very often

2 this should be done with new populations after converged soln

Global NewChargeData
2 should any of the following 3 statements be removed?

Global oi_level_pops
2 this must have all stimulated emission, pump by cont, etc

Global OpacityAddTotal

2 add charged heavy elements

1 stupid - why this test on opacity_abs ? - we only get here if we already passed above test on this very thing

Global ParseDynaWind

2 Need to include divergence terms in pressure balance if flux index is != 0

2 Need to include divergence terms in pressure balance if flux index is != 0

Global ParseMonitorResults
2 NB this is not used, should do both, and more molecules (H2 only for now)

Global ParseVLaw
2 is there a need to keep this negative?

Global PE_init
xray - add fluoresence in energy balance

Global PresTotCurrent

2 this is the total binding energy of the molecules, and is negative, the energy need to get back to free atoms never set and only appears in print statements

0 add this press term due to cosmic rays - hextra.cr_energydensity

0 code has variable magnetic energydensity and pressure, which are equal, as they must be - del one or the other

Global PrtComment
2 extend to all iso and elem

Global PrtLinePres
1 make this and eval rad pressure same routine, with flag saying to print contributors - copy code from other routine - this code has been left behind

Global rayleh
2 update to astro-ph/0308073, Lee, H-W, ApJ in press

Global rfield_opac_malloc

3 consider making the fine opacity array a double. with a float, the opacity itself often becomes a denormalized number, it then becomes significant when multiplied by dr - can cause numerical noise. this is why the coarse opacity array is a double

1 set temp of 1e4K will be too coarse a line for PDRs where H2 line overlap is very important

Global RT_DestProb
2 this min is because there are no calculations that show what to do for beta beyound this value

Global RT_diffuse

2 should use ConEmitLocal for all continua, but not followed by rfield.DiffuseEscape - put that at the end. Once continua all bundled this way, it will be easy to save them as a function of depth and then do exact rt

2 Replace this constant with the appropriate macro, if any

2 add fegrain to outward beams, but within main formalism by including grains in all x-ray processes

Global RT_LineWidth

1 dynamics; this test assumes that neg vel are subsonic, so that sobolev length would overestimate the optical depth, since ion is at most present over computed slab, and possibly more.

1 rewrite so that this checks on size not sign of windv

Global RT_OTS
2 should also check whether IonLo is in bounds - in func_set_ion test He0 is set to zero, so this does not do anything. as NISO grows this will become larger waste of time

Global RT_stark
2 - Stark is disabled for now since Lya escape causes density dependent feedback on the radiative transfer. Would need to redo the escape probs every time the electron density is updated - see blr89.in for an example

Global Save1LineData
1 define lifetime and collision rate for multi-level species so that the critical density is derived correctly in this routine. For now the flag lgCS_2 being true means to save critical den and is only true for two-level systems all places where this routine is called with lgCS_2 false need to be fixed

Global SaveNewContinuum
2 - NB - if continuum resolution changed the lines WILL NOT WORK

Class stellar_grid

- check rebinning of Tlusty models

- why was it necessary to change stars_tlusty.in? (change from r43 to r50?)

- check all interpolation modes of CoStar

- compare models with original code, dump atmospheres!

- check all Edges arrays...

- update Doxygen documentation

Global store_new_densities
2 renorm should == 1 when the molecules and ionization are in equilibrium. Should monitor this figure of merit in calling routine.

Global SumDensities

2 - should this include mass in grains?

2 - should this include mass in grain mantle ice deposits?

Global tbl_fun
2 include code for interpolating inv_att_len somewhere!!

Global TempInterp
2 use a canned interpolation routine, no need for special one here

Global TempInterp2
there is a virtually identical routine in helike_recom.cpp -> combine

Global UpdatePot1

xray - secondaries from incident electrons still need to be added in

xray - primary, secondary, auger electrons need to be added into suprathermals

Global zero

2 change this to CS_new

2 file opacities are disabled for now - reinstate this when arrays settle down

Global ZoneStart
2 is this the best place for this?
 All Data Structures Namespaces Files Functions Variables Typedefs Enumerations Enumerator Friends Defines

Generated on 15 Nov 2012 for cloudy by  doxygen 1.6.1