search

STScI-Citizen-Science / MTPipeline

Pipeline to produce CR rejected, astrodrizzled, png's of HST WFPC2 solar system data.
6 stars 1 forks source link

Added most of the docstrings to build_master_finders_table.py, what rema... #140

Closed acviana closed 10 years ago

acviana commented 10 years ago

...ins unupdated needs to be refactored. For Ticket #136. Local tests failing due to a missing dependency. Should still pass on Travis.

This is a refactor of the code as well as a complete reworking of all the major docstrings. There is still more work to be done but this can be done in the parent update-master-finders branch.

acviana commented 10 years ago

@walyssonBarbosa Once Travis has finished running these tests please review my changes and let me know your comments.

acviana commented 10 years ago

@walyssonBarbosa Please let me know when you have had a chance to look over my changes.

acviana commented 10 years ago

Just to confirm, I am going to merge this into your branch now. Is that OK?

walyssonBarbosa commented 10 years ago

Okay. Then I will have to do the changes again, right?

acviana commented 10 years ago

Yes, you can just copy them out and then copy them back in after the merge.

walyssonBarbosa commented 10 years ago

Ok.