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

Proper motions: modeling #9

Open
mcdittmar opened this issue Mar 4, 2021 · 7 comments
Open

Proper motions: modeling #9

mcdittmar opened this issue Mar 4, 2021 · 7 comments

Comments

@mcdittmar
Copy link
Collaborator

There are currently 2 use cases whose data includes proper motions.
* Precise Astrometry: includes ra, dec, pm_ra, pm_dec, radial velocity, parallax
* Column Groupings: includes ra, dec, 'total' pm, radial velocity

My question has to do with the modeling of proper motion.
The Measurements model currently has ProperMotion( longitude, latitude ); which corresponds to the Precise Astrometry case. The other appears to be ProperMotion( magnitude, position_angle ) where the position angle is missing (perhaps because it is not relevant to the Column Groupings use case).

Questions:

  1. am I interpreting this correctly?
  2. are we going to want to support both representations of ProperMotion?
@lmichel
Copy link
Collaborator

lmichel commented Mar 4, 2021

  1. The column_grouping case is focused on (RV, q_RV, o_RV, r_RV)
    PMs are out of the case scope.

  2. In column_grouping, the pm is given as a simple angular velocity, without direction.

    <FIELD name="pm" ucd="pos.pm" datatype="short" width="4" unit="mas/yr"><!-- ucd="POS_EQ_PM" -->

    Not sure it worth to implement it

@msdemlei
Copy link
Contributor

msdemlei commented Mar 4, 2021 via email

@mcdittmar
Copy link
Collaborator Author

mcdittmar commented Mar 4, 2021 via email

@lmichel
Copy link
Collaborator

lmichel commented Mar 5, 2021

Mark
I agree, it is not required to map PM in this case, but the issue of the exotic data serializations must be kept open.

Not sure that in our sample the angle is available somewhere. People might be only interested in PM modulus which is enough to compute an error cone.

@lmichel
Copy link
Collaborator

lmichel commented Mar 5, 2021

Markus,

The way to support different representations for a particular measure must be well documented in the context of that measure.

Managing cos(delta) is a real a trick. It has been added and then removed from Mango on the behalf that almost all catalogues have PM.RAs natively including this correction. This assumption must however by confirmed by data providers.

In my opinion, it would be wise to raise a flag when cos(delta) has been applied.

About the combination of pos/pn/vr/parallax, we kept focused on the cross-match use case for wich we need to combine errors. This is the purpose of MANGO error extension. The error combination is based on the associatedParameters UCDs(see mango:exterrors package).
X-match is a big challenge for out exercise because this is the case where we need a perfect interoperability.

@mcdittmar
Copy link
Collaborator Author

Laurent:

> added then removed from Mango

  • the Mango model doesn't include ProperMotion, right?, that is in Measurements. There, we have have not (yet) added any indication whether or not the cos(delta) is applied, leaving it to the usage thread to specify the expectations.

the x-match case is sounding very interesting!

@lmichel
Copy link
Collaborator

lmichel commented Mar 10, 2021

Mango do use meas:ProperMotion actually.
The question of tagging the use of cos(delta) would have imply to extend that class.

added then removed from Mango:
This was in the description of the complex errors which is not par of meas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants