Skip to content

Improve exception handling in host matching task and avoid creating duplicate hosts#402

Draft
manning-ncsa wants to merge 30 commits intomainfrom
400-host-matching
Draft

Improve exception handling in host matching task and avoid creating duplicate hosts#402
manning-ncsa wants to merge 30 commits intomainfrom
400-host-matching

Conversation

@manning-ncsa
Copy link
Copy Markdown
Collaborator

@manning-ncsa manning-ncsa commented Apr 27, 2026

Fixes #400
Fixes #403

Added

  • Added three new fields to the Host mode (object_id, catalog_name, catalog_release) to capture the
    source catalog information provided by the host matcher.
  • Added support for input arguments to custom functions invoked by the blast_admin Django custom
    management command. See blast_admin.py module docstring for details.

Changed

  • The HostMatch._run_process() function was altered to treat any unhandled exception as a failure with a
    corresponding "failed" status message. Previously, an unhandled exception would result in a misleading
    "no host match" status.
  • Additional logic was added to the host matching task to avoid creating duplicate Host objects: The host
    information returned by Prost is compared against existing Host objects by cone search and by catalog information.
  • Updated the host information card displayed on result pages to include the new catalog information.
  • Removed redundant call to initialize_all_tasks_status() in reprocess_transient()

Removed

  • Removed the obsolete "Initialize transient task" periodic task. Workflows for new transients ingested
    from TNS are now triggered upon discovery. Thus, all pathways for adding new transients now automatically
    initialize and trigger workflows immediately, eliminating the need for this periodic task.
  • Removed unused TransientImportForm
  • Removed unused CatalogManage

Fixed

  • Fixed a bug in the dataset archive importer to support importing transients lacking host information.

@manning-ncsa manning-ncsa force-pushed the 400-host-matching branch 2 times, most recently from ec382cc to 21700f3 Compare April 27, 2026 22:18
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.

Deleting Host or User object deletes Transient object and cascades Improve exception handling in host matching task

1 participant