Skip to content

Update Fermi Catalog and adapt feature name in plotting function #88

Open
azegarelli wants to merge 5 commits into
mainfrom
upd-fermi-catalog
Open

Update Fermi Catalog and adapt feature name in plotting function #88
azegarelli wants to merge 5 commits into
mainfrom
upd-fermi-catalog

Conversation

@azegarelli

Copy link
Copy Markdown
Contributor

Hi,

After the release of a new catalog by Fermi (https://fermi.gsfc.nasa.gov/ssc/data/access/lat/fl16y/), and after discussion among the Real-time group and ROC (@blaufuss ), we would like to change the catalog name in skyreader/constants.py..
In this way, the new FIT file (stored in /cvmfs/icecube.opensciencegrid.org/users/azegarelli/realtime/catalogs) is used to produce zoomed plots around the track-alert reconstructions with Skymist.

Consequently, I propose to change here plot_4fgl to plot_fermi_sources, to make the boolean more general to the usage of every Fermi catalog name that can be changed in skyreader/constants.py..
This implies also a similar change in SkyMist (see the following pull request https://github.com/icecube/skymist/pull/272)

We take profit of this merge request, to fix a failing check for mypy(>=3.11), appeared after January 2026 (

already_filled_uniqs_array = np.concatenate(already_filled_uniqs)
already_filled_indeces = np.array(
[np.where(uniqs == uni)[0][0] for uni in already_filled_uniqs_array]
)
)

@azegarelli azegarelli requested review from G-Sommani and ric-evans June 9, 2026 09:39
@azegarelli

Copy link
Copy Markdown
Contributor Author

I forgot to mention that I have already tested that the current plotting functions work for the new catalog

This is the plot for the last track alert
Screenshot 2026-06-09 alle 11 41 57

As you can see, the F16Y sources are now overplotted.

Just for comparison, this was the plot produced at the time of the alert with the Fermi 4FGL catalog
Screenshot 2026-06-09 alle 11 43 08

@blaufuss blaufuss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks @azegarelli

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