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

Move timeInfo() out of the package #93

Open
spencerpease opened this issue Aug 6, 2019 · 0 comments
Open

Move timeInfo() out of the package #93

spencerpease opened this issue Aug 6, 2019 · 0 comments
Labels
Idea refactor Make the code base cleaner with minimal change to functionality
Milestone

Comments

@spencerpease
Copy link
Contributor

Motivation

The function timeInfo() doesn't have any inherent dependence on the ws_monitor data PWFSLSmoke was created to work with. If we ever needed to use this function in another context (let's say working with raw air-sensor data), it would be convenient if was available from a smaller, more generalized package.

Idea

timeInfo() should exist in some *utils package, either MazamaSpatialUtils, MazamaCoreUtils, or some other "time utils" package.

Potential Difficulties

This function does have a dependency on MazamaSpatialUtils and the maptools package. If this function gets moved somewhere, it still needs to have access to these packages.

@spencerpease spencerpease added refactor Make the code base cleaner with minimal change to functionality Idea labels Aug 6, 2019
@spencerpease spencerpease added this to the future ideas milestone Aug 6, 2019
@jonathancallahan jonathancallahan added this to To do in future ideas via automation Jun 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Idea refactor Make the code base cleaner with minimal change to functionality
Projects
future ideas
  
To do
Development

No branches or pull requests

1 participant