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

add fits reader #64

Merged
merged 13 commits into from
Jun 28, 2016
Merged

add fits reader #64

merged 13 commits into from
Jun 28, 2016

Commits on Jun 22, 2016

  1. Cosmetic changes to indexReferenceTask

    A few PEP8 tweaks, plus remove linefeeds from a config field doc
    string and standardize the indentation for one block of code.
    r-owen committed Jun 22, 2016
    Configuration menu
    Copy the full SHA
    3cebe20 View commit details
    Browse the repository at this point in the history
  2. Import numpy as np

    Use standard alias for numpy and move the import
    to the standard location
    r-owen committed Jun 22, 2016
    Configuration menu
    Copy the full SHA
    23f20a2 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    8ec9cc4 View commit details
    Browse the repository at this point in the history
  4. Make default name of TextReaderTask more general

    Change TextReaderTask._DefaultName to 'catalogReader'
    a name sufficiently generic to be used by variant readers.
    r-owen committed Jun 22, 2016
    Configuration menu
    Copy the full SHA
    c89bdd4 View commit details
    Browse the repository at this point in the history

Commits on Jun 28, 2016

  1. Add FitsReaderTask

    Add a new catalog reader FitsReaderTask and associated config
    to read catalogs from FITS files.
    Add a test for the new task in a new test file
    tests/testIndexReference.py (which presently on has the one test)
    r-owen committed Jun 28, 2016
    Configuration menu
    Copy the full SHA
    e81a2d8 View commit details
    Browse the repository at this point in the history
  2. Fix flake8 warnings in unit tests

    Apply autopep8, remove unused imports,
    and stop setting unused variables.
    r-owen committed Jun 28, 2016
    Configuration menu
    Copy the full SHA
    ff8dfe4 View commit details
    Browse the repository at this point in the history
  3. Standardize imports in unit tests

    import numpy -> import numpy as np
    import testFindAstrometryNetDataDir ->
        from . import testFindAstrometryNetDataDir
    standardize grouping of imports
    r-owen committed Jun 28, 2016
    Configuration menu
    Copy the full SHA
    784c489 View commit details
    Browse the repository at this point in the history
  4. Rename catalog reader tasks and run method

    Rename TextReaderTask/Config to ReadTextCatalogTask/Config
    Rename FitsReaderTask/Config to ReadFitsCatalogTask/Config
    Rename "readFile" method to "run"
    Update unit tests accordingly
    r-owen committed Jun 28, 2016
    Configuration menu
    Copy the full SHA
    48dad41 View commit details
    Browse the repository at this point in the history
  5. Refactor catalog reader tasks

    Move catalog reader tasks to new module readCatalog.py
    Rename tests/testIndexReference.py to tests/testReadCatalog.py
    since it only tests a reader
    r-owen committed Jun 28, 2016
    Configuration menu
    Copy the full SHA
    2c8c67c View commit details
    Browse the repository at this point in the history
  6. Add missing from __future__ import

    Add missing
      from __future__ import absolute_import, division, print_function
    to all three index catalog reader modules
    r-owen committed Jun 28, 2016
    Configuration menu
    Copy the full SHA
    05c56c8 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    2bf808c View commit details
    Browse the repository at this point in the history
  8. Break out testReadFitsCatalog into its own file

    Also put table columns into a more reasonable order,
    since the code no longer rearranges them.
    r-owen committed Jun 28, 2016
    Configuration menu
    Copy the full SHA
    d786527 View commit details
    Browse the repository at this point in the history
  9. Add unit test for text reader

    r-owen committed Jun 28, 2016
    Configuration menu
    Copy the full SHA
    d0a6e89 View commit details
    Browse the repository at this point in the history