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

Remove unused scatter plot code #978

Open
3 tasks
ian-noaa opened this issue May 18, 2023 · 2 comments
Open
3 tasks

Remove unused scatter plot code #978

ian-noaa opened this issue May 18, 2023 · 2 comments
Assignees
Labels
epic A piece of functionality directly related to requirements similar to an epic user story needs requirements Status for project issues that are being discussed Stale

Comments

@ian-noaa
Copy link
Collaborator

Problem

In dtcenter/MATScommon#27, it was discovered that there is a good amount of unused scatter plot code in MATScommon.

Solution

The scatter plot code should be removed so that we don't need to maintain dead code.

No Go's

Describe any features or behaviors that have been considered and rejected as out of scope for this project

Tasks

Describe discrete tasks that need to be done to complete this project

  • Task 1
  • Issue link
  • etc...
@ian-noaa ian-noaa added epic A piece of functionality directly related to requirements similar to an epic user story needs requirements Status for project issues that are being discussed labels May 18, 2023
@mollybsmith-noaa
Copy link
Collaborator

All of Molly's in-use scatter code is for the plot type matsTypes.PlotTypes.simpleScatter

@github-actions
Copy link

This issue is stale because it has been open 90 days with no activity.

@github-actions github-actions bot added the Stale label Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic A piece of functionality directly related to requirements similar to an epic user story needs requirements Status for project issues that are being discussed Stale
Projects
None yet
Development

No branches or pull requests

3 participants