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 projection and geometry to utils/ #356

Open
matthewhanson opened this issue Sep 15, 2022 · 2 comments
Open

Move projection and geometry to utils/ #356

matthewhanson opened this issue Sep 15, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@matthewhanson
Copy link
Member

Is your feature request related to a problem? Please describe.
Suggestion for reorganizing the existing code. While most of the files in core/ deal with modifying or creating STAC items, the geometry and projection modules are utility functions that would be called from elsewhere.

Describe the solution you'd like
Move geometry.py and projection.py to the utils module

Describe alternatives you've considered
Leaving it as is, but it seems arbitrary in that other utility functions are in utils/

@matthewhanson matthewhanson added the enhancement New feature or request label Sep 15, 2022
@matthewhanson
Copy link
Member Author

Note this could have impact to various stactools submodules that may be using these functions.

@gadomski
Copy link
Member

Another alternative would be remove utils/ altogether and just have a flat module space in stactools.core.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants