Skip to content

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

Merged
azegarelli merged 9 commits into
mainfrom
upd-fermi-catalog
Jun 15, 2026
Merged

Update Fermi Catalog and adapt feature name in plotting function #88
azegarelli merged 9 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

@azegarelli azegarelli closed this Jun 15, 2026
@ric-evans ric-evans reopened this Jun 15, 2026
@azegarelli azegarelli merged commit 8e328b2 into main Jun 15, 2026
16 checks passed
@azegarelli azegarelli deleted the upd-fermi-catalog branch June 15, 2026 16:08
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.

4 participants