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

Major change in code internals #8

Open
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

PBarmby
Copy link
Collaborator

@PBarmby PBarmby commented Jul 2, 2014

Hi Sophia -- I was trying to use imagecube with Sahar's data and not being very successful. For what I needed to do, a different file structure was necessary, so I rewrote the code to have 2 output files: a multi-extension FITS file which contains the processed images, and a separate 3D datacube file (see OUTPUT in the help for the main imagecube file). This makes it easier to run one task at a time and also allowed me to simplify the code somewhat. I put in a few more bells and whistles, like adding information to HISTORY cards in the header.

You may want to have a look at the changes in detail (you could clone my file_struct branch if you like) before merging. We can also talk about it over Skype if you like..

@sophiathl
Copy link
Owner

Hi Pauline, how about skype this Friday, about London's morning (our time
difference now is 6 hours)? Or over the weekend, when I am in Athens (7
hours)? Or perhaps when you are back, end of July? Anything is OK with me.

On Wed, Jul 2, 2014 at 3:26 AM, Pauline Barmby [email protected]
wrote:

Hi Sophia -- I was trying to use imagecube with Sahar's data and not being
very successful. For what I needed to do, a different file structure was
necessary, so I rewrote the code to have 2 output files: a multi-extension
FITS file which contains the processed images, and a separate 3D datacube
file (see OUTPUT in the help for the main imagecube file). This makes it
easier to run one task at a time and also allowed me to simplify the code
somewhat. I put in a few more bells and whistles, like adding information
to HISTORY cards in the header.

You may want to have a look at the changes in detail (you could clone my
file_struct branch if you like) before merging. We can also talk about it

over Skype if you like..

You can merge this Pull Request by running

git pull https://github.com/PBarmby/imagecube file_struct

Or view, comment on, or merge it at:

#8
Commit Summary

  • starting to work on new file structure
  • Initial change to hdulist file structure, untested.
  • Fixed indentation; cleanup still works.
  • construct_mef now works
  • convert_image now works
  • switched cleanup function
  • register_image in progress, not yet working
  • fixes to resample_image (as yet untested)
  • switched over convolve_image, untested
  • convolve_image seems to work
  • resample_image appears to work
  • create_datacube nearly but not quite working
  • resample_image works
  • still problems with register_image, create_datacube
  • merge_headers was overwriting input: fixed. This fixes
    register_image and resample_image too, I think.
  • create_datacube works
  • test_imagecube now passes
  • added comments to code, more info to output headers
  • tiny changes
  • fixed check on imagecube history
  • minor bug fix
  • Existing imagecube.fits not included in input
  • changed datacube creation to use dstack and swapaxes
  • changed im_regrid to im_resamp (less confusion with im_reg)
  • changed introductory info

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#8.

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