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

Add StarAcq event type and infrastructure for emailed alerts #94

Closed
wants to merge 8 commits into from

Conversation

taldcroft
Copy link
Member

@taldcroft taldcroft commented Mar 19, 2017

This adds a new kadi event that has the number of acq stars ID'd and which ones, along with the size of one-shots. This is slightly messy and might not end up being merged. However, it will be run in the short term on the side at 2 hour intervals to provide visibility into metrics related to gyro bias variations and mitigation. This might belong elsewhere (like attitude_error_mon), but decided the existing infrastructure in kadi was useful for this.

Note also the new flag to use MAUDE instead of CXC as the telemetry data source.

To do:

  • Fix docstring
  • Change email address
  • Update task_schedule to only update this model and use MAUDE
  • Install and run on the side

@jeanconn - any opinion on the difference between using localhost vs. head.cfa.harvard.edu for the SMTP server?

@jeanconn
Copy link
Contributor

I don't really have an opinion on the sending mail choice. It looks like our original requirement during GAG migration was that local "sendmail" and direct connections to head.cfa.harvard.edu would continue to work. It looks like direct SMTP connections to localhost are doing the right thing too and we've been moving to that in more and more code.

@jeanconn
Copy link
Contributor

For sot/attitude_error_mon#9 I was also planning to add the one shot pitch/yaw breakdown (or a delta quaternion), and I figured it made sense to do it in kadi, so nice to have it done. Where is this running on the side?

@taldcroft
Copy link
Member Author

It's running on /proj/sot/ska/dev-kadi every 2 hours (except not at 2 and 4am, no point).

I'll change alerts to aca_alert, just wanted to make sure it didn't blow up.


alerts = []

if abs(event['one_shot_pitch']) > 60 or abs(event['one_shot_yaw']) > 60:
Copy link
Contributor

@jeanconn jeanconn Mar 23, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is 60 a little low? I thought the warning on 50220 wasn't that helpful with estimated maneuver error of ~53 arcsec. (Granted, anything over the maneuver error estimate seems worth a trending-style warning, but not sure if we need an alert at 60 unless all the boxes are 80).

One shot at 2017:081:20:16:31.008
Roll  : 2.11
Pitch : 63.38
Yaw   : 7.40

@taldcroft
Copy link
Member Author

I purposely set these limits pretty low for increased awareness in the near term. But note that 60 is not that low from a historical perspective since fewer than 70 of 9000 one-shots in the last 5 years exceed that. So seeing 60 means something slightly unusual has happened and if we see it every week that means one shots are worse.

The alert on n_id <= 5 is definitely too jumpy for production.

@jeanconn
Copy link
Contributor

jeanconn commented May 1, 2017

I don't immediately see why this kicked off two different notes for the 19504 acquisition:

One shot at 2017:121:04:14:26.632
Roll  : 1.65
Pitch : -63.57
Yaw   : 27.50

One shot at 2017:121:04:14:25.898
Roll  : 1.65
Pitch : -63.57
Yaw   : 27.50

@jeanconn
Copy link
Contributor

jeanconn commented May 5, 2017

I'm seeing events_staracq in /proj/sot/ska/data/kadi/events.db3, so did this end up in flight or was that a previously-existing thing?

@taldcroft
Copy link
Member Author

Will leave this as a branch but closing since we have no plans to merge to master.

@taldcroft taldcroft closed this May 16, 2017
@taldcroft taldcroft deleted the acq branch December 7, 2019 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants