Skip to content

Conversation

@skoranda
Copy link

Description of the Change

Integration of the Astrodash API backend into the blast framework as another Django application. This commit includes the necessary migrations, fixtures, data file manifest, and code changes to initialize database tables and download the required data files.

Checklist

Please check all that apply to your proposed changes

  • HTML code change
  • Added package dependency
  • Added data
  • Changed django model
  • Documentation change
  • Added or changed TaskRunner

Additional context

This commit does not address the Astrodash front-end application.

Integration of the Astrodash API backend into the blast
framework as another Django application. This commit includes
the necessary migrations, fixtures, data file manifest,
and code changes to initialize database tables and download the
required data files.
@manning-ncsa
Copy link
Collaborator

Acknowledging that I tested commit fe3046e from the feature/astrodash_frontend branch instead of from astrodash_integration, I was able to demonstrate local operation using the sn2002er example we have been using as a test of basic functionality. The data initialization routine worked perfectly, and I saw no errors in the container logs when generating the spectrum shown below.

As we discussed today in person, we have not yet decided whether this will be merged into the Blast repo, a compatible fork, or a repo containing only the Astrodash Django app itself (i.e. for installation as a "plugin" of Blast alongside for example rest_framework and mozilla_django_oidc).

image

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.

2 participants