Skip to content

Conversation

@HealthyPear
Copy link
Member

Closes #11.

Here is an example:

Crab Nebula_skymap_with_wobbles

Not super configurable, but a first version.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@HealthyPear
Copy link
Member Author

One thing that is missing here is the possibility to add an extended source, but I would prefer to do it after #33 so we have the gammapy config file with the sky models

@HealthyPear
Copy link
Member Author

One thing I need to fix is the position angle of the wobbles: I just realized that I am using astropy's convention (0 deg is North and then increase counterclockwise) rather than the IACT convention (0 deg is West and then increase clockwise)

easy fix

astropy_position_angle = (90 * u.deg - out_position_angle) % (360 * u.deg)

We were using astropy's convention (0 deg is North and then increase counterclockwise) rather than the IACT convention (0 deg is West and then increase clockwise)
@HealthyPear
Copy link
Member Author

OK wobble position angle is now fixed

@HealthyPear
Copy link
Member Author

I need to bump the minimum version of python to 3.10 as now starplot requires it due to some upstream dependencies.

Anyway Python 3.9 is about to die soon (see https://devguide.python.org/versions/)

A-posteriori,
- updated wobble skymap function for new version of starplot
- moved plot_observability_constraints_grid and create_observability_heatmap to plots.observability
- removed save testing figure in docs
@HealthyPear HealthyPear merged commit 90d0d12 into main Oct 15, 2025
8 checks passed
@HealthyPear HealthyPear deleted the wobble_skymap branch October 15, 2025 15:02
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.

Add support to see wobble positions and stars in the FoV

1 participant