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

Get IMGSTAT into the maude decom aca packets / images #84

Open
jeanconn opened this issue Nov 25, 2019 · 4 comments
Open

Get IMGSTAT into the maude decom aca packets / images #84

jeanconn opened this issue Nov 25, 2019 · 4 comments
Assignees

Comments

@jeanconn
Copy link
Contributor

Get IMGSTAT into the maude decom aca packets / images

@jeanconn
Copy link
Contributor Author

Need at least IMGSTAT. Nicer to have them in English.

@javierggt
Copy link
Contributor

Question... what do you prefer?

  1. one column called IMGSTAT
  2. one column for each flag
  3. both

(as far as I understand, we want at least No. 2)

Any preference on names? Well, I guess you will tell me this during the PR.

@javierggt
Copy link
Contributor

javierggt commented Nov 25, 2019

IMGSTAT columns:

  • IMGSTAT (unsigned)
  • SAT_PIXEL (bool)
  • DEF_PIXEL (bool)
  • QUAD_BOUND (bool)
  • COMMON_COL (bool)
  • MULTI_STAR (bool)
  • ION_RAD (bool)

@javierggt
Copy link
Contributor

javierggt commented Nov 26, 2019

I just pushed a change for this and forgot to mention it in the git log: a42f931f5...

Note that, as we discussed, IMGNUM, FID, IMGFUNC appear twice in this code (in header 1 and header 2). This code ignores the values in header 2.

Also, there is a flag for invalid checksum in command groups. It seems to appear twice, in the 3rd and 4th word of the ACA packet. I didn't ignore any because I am not sure.

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

2 participants