Skip to content
This repository has been archived by the owner on Apr 16, 2024. It is now read-only.

Only include non-empty cells in download package outputs #61

Open
cjgrady opened this issue Nov 1, 2021 · 1 comment
Open

Only include non-empty cells in download package outputs #61

cjgrady opened this issue Nov 1, 2021 · 1 comment
Assignees
Labels

Comments

@cjgrady
Copy link
Contributor

cjgrady commented Nov 1, 2021

We are currently returning all cells in the geojson outputs but the cells without data are not really useful. Filter those cells out to reduce the size of the response and avoid errors caused by having too many cell values in the output package pages.

@cjgrady cjgrady self-assigned this Nov 1, 2021
@cjgrady
Copy link
Contributor Author

cjgrady commented Nov 1, 2021

GitHub won't let me attach a test file (maybe it is too big?) But you can see the problem when there are too many sites in the "package/sitesObserved.js" file in the download package. I am not sure what that number is but it seems to work fine with 50k sites and not when there are more than 250k sites

@cjgrady cjgrady added the bug label Nov 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant