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

Investigate removing TEAL #1738

Open
stscijgbot-hstdp opened this issue Feb 15, 2024 · 9 comments · May be fixed by #1759
Open

Investigate removing TEAL #1738

stscijgbot-hstdp opened this issue Feb 15, 2024 · 9 comments · May be fixed by #1759

Comments

@stscijgbot-hstdp
Copy link
Collaborator

Issue HLA-1208 was created on JIRA by Steve Goldman:

This ticket would deprecate and then remove the TEAL interface from new versions of drizzlepac. After discussing this with Nadia, we agreed that this should be done as the current code is not maintainable. 

The change would also require altering some current regression tests that use TEAL to create configuration files. 

@stscijgbot-hstdp
Copy link
Collaborator Author

Comment by Matthew Burger on JIRA:

Asked on slack in #python, #research, and #hst_pipeline_meeting for people to let me know if they use TEAL.

Results:

  • Alyssa Pagan

@stscijgbot-hstdp
Copy link
Collaborator Author

Comment by Matthew Burger on JIRA:

Chris Sontag:  Awww.  I wrote that a bunch of years ago.  No idea who maintains it now.  Thought it was dead.

 
Marc Rafelski: I sometimes use it as a quick way to look at different options to change. I could use the documentation instead, but I'm used to that being a quick way. I wouldn't keep it around for my use cases.
 

@s-goldman
Copy link
Collaborator

As a part of this effort, the documentation will have to be updated #1439.

@stscijgbot-hstdp stscijgbot-hstdp changed the title Removing TEAL Investigate removing TEAL Feb 20, 2024
@stscijgbot-hstdp
Copy link
Collaborator Author

Comment by Michele De La Pena on JIRA:

This issue needs some initial investigation as the underlying code which reads the default configuration files, performs type checking, and controls which parameters work in tandem with one another (i.e., turn "off" parameter one which then disables parameters six and seven as they are no longer applicable) is valuable and would potentially require a lot of re-write.  Is it just the actual GUI which is not maintainable?

@stscijgbot-hstdp
Copy link
Collaborator Author

stscijgbot-hstdp commented Feb 21, 2024

Comment by Steve Goldman on JIRA:

Along the line of what Michele mentioned is that the teal.teal command could be removed, while we keep the teal.load commands for the backend parsing type checking, and rules within the cfg parameter files.

These would otherwise have to be converted to json, and then we would have to create new rules elsewhere in the code to compensate for the teal loads; this seems like more work than it is worth if teal.load can remain.

@s-goldman s-goldman linked a pull request Feb 29, 2024 that will close this issue
6 tasks
@stscijgbot-hstdp
Copy link
Collaborator Author

Comment by Michele De La Pena on JIRA:

@⁣sgoldman
When closing a ticket with Done or Cancel, please add comment regarding what was done or why it is being closed. You can reference a GitHub ticket as appropriate.

@stscijgbot-hstdp
Copy link
Collaborator Author

Comment by Steve Goldman on JIRA:

Thanks for the reminder! An initial attempt to remove TEAL has been created in PR 1759 more testing will be needed. 

In order to share this change with users, we should add documentation about the Legacy TEAL GUI, and share the change with something like a MAST newsletter or STAN. Mitchel Revalski also suggested providing a YML file or a list of the most current package versions that will work with TEAL for anyone that still wants to use the GUI. 

@stscijgbot-hstdp
Copy link
Collaborator Author

Comment by Michele De La Pena on JIRA:

Steve Goldman
Once the "hiding of the TEAL GUI" is fully tested, I can tell the ACS and WFC3 Teams and request they add this information to their STANs. Once we make a release of drizzlepac with this change, then we will define the last version of drizzlepac which worked with the TEAL GUI. Pardon, but I do no know...are there other packages which used TEAL?

Please write a ticket so we remember to notify the INS teams regarding the official deprecation of the TEAL GUI starting at version n.n.n of drizzlepac. It can be suggested they announce this in their STANs.

@stscijgbot-hstdp
Copy link
Collaborator Author

Comment by Steve Goldman on JIRA:

Sounds good. I know at least one package that still uses TEAL is stsci.skypac. Drizzlepac still prints out a shorter TEAL message when you import drizzlepac because stsci.skypac is a dependency.

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 a pull request may close this issue.

2 participants