Skip to content

Conversation

@kwynn03
Copy link

@kwynn03 kwynn03 commented Jan 13, 2026

Changes included in this PR:

  • Reformatted some of the elements in the galaxy tool tip (RA, DEC, Score, Distance). This is just a small change (in function.py) to the rounding of each element in the pop up that occurs when a user clicks on a galaxy in the visualization for a particular event.

  • Added a link to the NED search page for the particular galaxy into the tool tip (see function.py).

  • Sorted the list of galaxies by rank, so the first galaxy in the list is rank 0 (see the changes in ajaxrequests.py).

  • Added a field under the Galaxies tab that can limit the number of galaxies shown on the visualization to the user input value. Similar to before, the 'Get' button does load in all galaxies available at the beginning, but there is now a user-input field to change what is shown on the page. This allows for quick updates to the visualization and list based on user input (either by hitting 'enter' or by focusing on a different field). The default number of galaxies to visualize is now 100, but a user can still see all galaxies on the visualization by making the user input field blank. This required a major re-write of the alert_event_galaxies function, and is the bulk of the work done here. Note that the refreshing of aladin markers was a little tricky, and was solved by setting a global galaxy list that is updated on a field change. I'm sure there is a better way of doing this, so please do let me know (see changes in alert_info.html).

These changes were all tested on a recent local copy of TM, by selecting different events (ex: S251112cm, S251021u, and others), loading the galaxy list for each event, and checking the tool tips and galaxy list for different field inputs. Various field inputs were tested including scientific notation, no input, numbers out of bounds, and non-integers or strings.

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.

1 participant