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

Time series service: if boundaryGeometry is not specified, by default it should be the entire available area of the dataset. #9

Open
Zodiase opened this issue Mar 10, 2018 · 2 comments

Comments

@Zodiase
Copy link

Zodiase commented Mar 10, 2018

No description provided.

@tmcphillips
Copy link
Member

Right now the boundaryGeometry.type property defaults to 'Point' because that is all we handle. Should we require that 'Point' be specified explicitly and return error if missing or other type is requested?

@Zodiase
Copy link
Author

Zodiase commented Mar 20, 2018

I wouldn't worry about it until the area averaging is supported, after which point this issue would make sense.

For now, the client should be expected to specify a boundary. If not, I'd think it'd be still fine to just crash and return 500.

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

No branches or pull requests

2 participants