8000 Get IMGSTAT into the maude decom aca packets / images · Issue #84 · sot/chandra_aca · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

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.

Sign up for GitHub

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

Jump to bottom
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
@javierggt

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 javierggt self-assigned this Nov 25, 2019
@jeanconn jeanconn mentioned this issue Nov 25, 2019
Merged
@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 •
edited
Loading

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 •
edited
Loading

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
Assignees

@javierggt javierggt

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@javierggt @jeanconn
0