Closed Jammy2211 closed 3 years ago
The implemented solution gives the following:
galaxies->lens: Galaxy (N=5)
galaxies->source: Galaxy (N=0)
Can we make it so it goes all the way down the Collection? e.g.:
galaxies->lens: Galaxy (N=5)
galaxies->lens->bulge: EllSersic (N=7)
galaxies->lens->disk: EllSersic (N=5)
galaxies->lens->mass: EllIsothermal (N=5)
galaxies->lens->shear: ExternalShear (N=2)
galaxies->lens->redshift: Redshift (N=0)
galaxies->source: Galaxy (N=0)
galaxies->source->pixelization: VoronoiBrightness (N=0)
galaxies->source->regularization:
If its not hard to format it may be worth mimicking the indentation of the model.info / model.results:
galaxies Collection(N=17)
lens Galaxy (N=5)
bulge EllSersic (N=7)
disk EllSersic (N=5)
mass EllIsothermal (N=5)
shear ExternalShear (N=2)
redshift Redshift (N=0)
source Galaxy (N=0)
pixelization VoronoiBrightness (N=0)
regularization Constant (N=0)
Could you also put the same text in the model.result file
Now looks like this:
(root) CollectionPriorModel (N=20)
galaxies CollectionPriorModel (N=20)
lens Galaxy (N=14)
bulge EllSersic (N=7)
centre TuplePrior (N=0)
elliptical_comps TuplePrior (N=0)
mass EllIsothermal (N=5)
centre TuplePrior (N=0)
elliptical_comps TuplePrior (N=0)
shear ExternalShear (N=2)
elliptical_comps TuplePrior (N=0)
source Galaxy (N=6)
bulge EllExponential (N=6)
centre TuplePrior (N=0)
elliptical_comps TuplePrior (N=0)
Can we remove the TuplePrior entries altogether?
How would you want it to look?
Replace (root)
with model
if it is (root)
just cause it looks a bit weird and remove tuples.
Also, the Galaxy
objects should have a Redshift
, which seems to have gone missing? This is a fixed float in this example, but can obviously be a prior in certain circumstances.
model CollectionPriorModel (N=20)
galaxies CollectionPriorModel (N=20)
lens Galaxy (N=14)
redshift Redshift (N=0)
bulge EllSersic (N=7)
mass EllIsothermal (N=5)
shear ExternalShear (N=2)
source Galaxy (N=6)
bulge EllExponential (N=6)
If the redshift is just a float then it won't have a parameterization. These entries are effectively one level up from the model.info entries which are any float or prior
A model.info file currently appears as follows:
It would be good if we could add a summary of the model parameterization at the top:
The N=? are the number of free parameters, so when they are zero is means all parameters associated with that prior are fixed values of an isntance.
We should add the same summary info to the
model.result
file, this can be a literal repeat of the stuff above, its just conveninent having everything in one place.