ACHMartin / seastar_project

4 stars 0 forks source link

LookDirection or AntennaAzimuthImage #204

Closed ACHMartin closed 1 year ago

ACHMartin commented 1 year ago

In Doppler (perhaps other section) is LookDirection or AntennaAzimuthImage we should be used?

ACHMartin commented 1 year ago

Using "LookDirection" in "test_seastar_doppler_package_adrien_v0.ipynb" it raises an error. We need to correct all files + documentation where "LookDirection" is mentioned, including modification of the notebook. It is a standalone notebook without the local path, so I think we could remove the "adrien".

DavidMcCann-NOC commented 1 year ago

I agree its confusing semantics. LookDirection as the variable Metasensing have provided denotes the port/starboard pointing direction of the arrays and is a single letter L or R (always L in our case). So I agree, perhaps we need to find a different phrase for the documentation in other functions. Perhaps 'Antenna direction' or 'Antenna location'?

The only place we actually use it in Doppler is the exception: if len(L1.LookDirection.shape) > 2: raise Exception('L1.LookDirection need to be a 2D field. \n' 'Use e.g. L1.sel(Antenna="Fore") to reduce to ' 'a 2D field.') Which looks to me to be expecting IncidenceAngleImage?

ACHMartin commented 1 year ago

Thank you for the explanation. Indeed, either IncidenceAngleImage or AntennaAzimuthImage.

ACHMartin commented 1 year ago

link to #211

DavidMcCann-NOC commented 1 year ago

Closed with #211