Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HLA-1208 Removing TEAL GUI #1759

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

s-goldman
Copy link
Collaborator

@s-goldman s-goldman commented Feb 29, 2024

Resolves HLA-1208

Closes #1738

This PR removed the TEAL GUI interface for drizzlepac. This PR does not make changes to teal.load calls which are still being used to load and parse ".cfg" configuration files. Much of this PR is removing documentation and adding comments that certain options are deprecated or no longer supported. The initial print statement of the available drizzlepac teal tasks has also been removed.

An entry to the ADRs was also added, and a number of unused imports were removed.

Checklist for maintainers

  • added entry in CHANGELOG.rst within the relevant release section
  • updated or added relevant tests
  • updated relevant documentation
  • added relevant milestone
  • added relevant label(s)
  • ran regression tests, post a link to the Jenkins job below.
    How to run regression tests on a PR

@mcara
Copy link
Member

mcara commented Feb 29, 2024

🎆

Copy link

codecov bot commented Mar 13, 2024

Codecov Report

Attention: Patch coverage is 75.00000% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 31.47%. Comparing base (a3421b6) to head (b7e6627).

Files Patch % Lines
drizzlepac/createMedian.py 50.00% 4 Missing ⚠️
drizzlepac/mdriz.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##           main    #1759       +/-   ##
=========================================
+ Coverage      0   31.47%   +31.47%     
=========================================
  Files         0      159      +159     
  Lines         0    35105    +35105     
=========================================
+ Hits          0    11051    +11051     
- Misses        0    24054    +24054     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate removing TEAL
2 participants