Skip to content

Rename annulus functions #34

Description

@astrofrog

I would like to suggest the following renames:

annulus_circular -> aperture_circular_annulus
annulus_elliptical -> aperture_circular_annulus

I've had comments that the current names are a bit confusing, and it would be better for all the functions that do aperture photometry to start with aperture.

An alternative would be to combine annulus_circular and aperture_circular into a single function that takes an optional r_in argument. If not specified, the aperture is cir cular, otherwise it's a circular annulus.

@kbarbary @cdeil - any thoughts on this?

Metadata

Metadata

Assignees

No one assigned

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions