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

Zero images #233

Merged
merged 5 commits into from
May 22, 2017
Merged

Zero images #233

merged 5 commits into from
May 22, 2017

Conversation

allegroLeiden
Copy link
Contributor

This allows the user to run LIME with zero images specified, and thus introduces the facility to run LIME in two separate sessions (desirable for CASA integration): the first to solve the RTE and save the results to file, the second to read the file and create raytraced images from it. A new parameter par.doSolveRTE has been added to compensate for the loss of information when zero images are specified.

-Removed malloc of (*gp)[i_ui].mol in gridio.c.readGrid(). This overwrote (*gp)[].mol[].abun values set after .mol is malloc'd in the earlier call to readGridTable().

-Keyword datatypes in writeGridIfRequired() and readOrBuildGrid() changed to lime_DOUBLE from TDOUBLE.
  - This allows the user to run LIME in two parts (necessary for the migration to CASA): the first part solves the raytracing equations for the model, then stores the solution at all grid points in a file; the second reads the file and performs all desired raytracing on it to produce image output.

Also:
  - Added new optional parameter doSolveRTE, default 0. This allows the user to perform the full solition without requesting any line images.

  - Function goodnight() broken into two: reportOutput() (called now only if par.nImages>0) and goodnight().
@allegroLeiden
Copy link
Contributor Author

This also fixes #231.

@allegroLeiden allegroLeiden merged commit fb1b371 into lime-rt:master May 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants