InsightSoftwareConsortium / InsightSoftwareConsortium/ITK

Address Todo items in AzimuthElevationToCartesianTransform

Open
#987 3 comments 0 reactions 0 assignees View on GitHub
Good first issue type:Design
Dominant language
C++
Stars
1.7k
Forks
748
Avg merge
1d 1h
Merged PRs (30d)
64

Description

### Description

AzimuthElevationToCartesianTransform

* \todo Is there any real value in allowing the user to template
* over the scalar type? Perhaps it should always be double, unless
* there's a compatibility problem with the Point class.
*
* \todo Derive this class from TransformBase class.
* Currently, this class derives from AffineTransform, although
* it is not an affine transform.
*
* \todo Add a figure in the documentation that informs the formulas used in this class
* that are used to transform Cartesian to azimuth-elevation-radius

### Expected behavior

Noted todo items should be removed from code base.

### Actual behavior

### Versions

ITK version 5.0.0

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.