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

Lofar/add pipeline plots #765

Closed
wants to merge 5 commits into from

Conversation

karenterveer
Copy link
Collaborator

The module modules/LOFAR/pipelineVisualization_LOFAR.py creates some nice plots at the end of the pipeline, i.e. the polarisation of antennas, the reconstructed arrival directions for all stations and also the fluence/timing/direction plot. Unless otherwise specified, it saves the plots in the current working directory. Furthermore it raises warnings if the polarisations in single antennas are very off or if the event time was before the outer station clocks were synced with the core station clocks.

class pipelineVisualizer:
"""
Creates debug plots from the LOFAR pipeline -
This is the pipelineVisualizerTM for LOFAR.
Copy link
Collaborator

Choose a reason for hiding this comment

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

*glorious pipelineVisualizerTM :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

thank you, Philipp

# plot absolute station positions
if station.get_parameter(stationParameters.triggered):
station_pos = detector.get_absolute_position(station.get_id())
ax.scatter(station_pos[0], station_pos[1], color=cmap(norm(i)), s=20, label=f'Station CS{station.get_id():03d}')
Copy link
Collaborator

Choose a reason for hiding this comment

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

maybe a different marker would be good to differentiate between station positions and antenna positions later (square or star?)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good point! (pun intended)

Copy link
Collaborator

Choose a reason for hiding this comment

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

don't use points!

@MijnheerD
Copy link
Collaborator

I merged these changes locally, so I will close this PR. Great work!

@MijnheerD MijnheerD closed this Nov 22, 2024
@MijnheerD MijnheerD deleted the lofar/add_pipeline_plots branch November 22, 2024 15:49
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

Successfully merging this pull request may close these issues.

3 participants