Skip to content

Commit

Permalink
Remove pulseheight fits.
Browse files Browse the repository at this point in the history
Pulseheight monitor service is not enabled for any station.
Model is not very efficient (separate object per detector).
Fit routines should be in SAPPHiRE.
Should use integrals instead of pulse heights.
  • Loading branch information
153957 committed Jul 22, 2018
1 parent 7509838 commit 219adda
Show file tree
Hide file tree
Showing 15 changed files with 54 additions and 861 deletions.
115 changes: 0 additions & 115 deletions provisioning/roles/nagios/files/check_pulseheight_mpv.py

This file was deleted.

8 changes: 0 additions & 8 deletions provisioning/roles/nagios/tasks/main.yml
Expand Up @@ -103,14 +103,6 @@
become: true
notify: restart nagios

- name: copy mpv check script
copy:
src: check_pulseheight_mpv.py
dest: /usr/lib64/nagios/plugins
mode: 0755
become: true
notify: restart nagios

- name: ensure apache is started
service:
name: httpd
Expand Down
6 changes: 0 additions & 6 deletions publicdb/api/urls.py
Expand Up @@ -45,10 +45,4 @@
url(r'^station/(?P<station_number>\d+)/num_events/(?P<year>\d{4})/(?P<month>\d{1,2})/(?P<day>\d{1,2})/(?P<hour>\d+)/$', views.num_events, name="num_events"),

url(r'^station/(?P<station_number>\d+)/trace/(?P<ext_timestamp>\d+)/$', views.get_event_traces, name="event_traces"),

url(r'^station/(?P<station_number>\d+)/plate/(?P<plate_number>\d+)/pulseheight/fit/$', views.get_pulseheight_fit, name="pulseheight_fit"),
url(r'^station/(?P<station_number>\d+)/plate/(?P<plate_number>\d+)/pulseheight/fit/(?P<year>\d{4})/(?P<month>\d{1,2})/(?P<day>\d{1,2})/$', views.get_pulseheight_fit, name="pulseheight_fit"),
url(r'^station/(?P<station_number>\d+)/plate/(?P<plate_number>\d+)/pulseheight/drift/(?P<year>\d{4})/(?P<month>\d{1,2})/(?P<day>\d{1,2})/(?P<number_of_days>\d+)/$', views.get_pulseheight_drift, name="pulseheight_drift"),
url(r'^station/(?P<station_number>\d+)/plate/(?P<plate_number>\d+)/pulseheight/drift/last_14_days/$', views.get_pulseheight_drift_last_14_days, name="pulseheight_drift_!4"),
url(r'^station/(?P<station_number>\d+)/plate/(?P<plate_number>\d+)/pulseheight/drift/last_30_days/$', views.get_pulseheight_drift_last_30_days, name="pulseheight_drift_30"),
]

2 comments on commit 219adda

@153957
Copy link
Member Author

@153957 153957 commented on 219adda Jul 22, 2018

Choose a reason for hiding this comment

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

Wat denken jullie (@davidfokkema @tomkooij @kaspervd) van deze aanpassing?

Het wordt nu niet gebruikt, geen station heeft die Nagios service aan staan. We zouden het meer kunnen gebruiken nu meer van de status van een station op data gebaseerd wordt, maar ook dan zou ik zeggen dat de fit routines in SAPPHiRE moeten (HiSPARC/sapphire#32) en dat het model nu niet handig is opgezet. We zouden eenvoudig de MPV die uit data processing komt kunnen ergens opslaan, nu matcht dat namelijk ook niet, wat ook vreemd is.

@davidfokkema
Copy link
Member

Choose a reason for hiding this comment

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

Ik ben vóór HiSPARC/sapphire#32, en vóór het verwijderen van deze Nagios-achtige code.

Please sign in to comment.