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

Enlarge default coverage grid in order to ensure correct NCSS requests #302

Conversation

ricardogsilva
Copy link

@ricardogsilva ricardogsilva commented Nov 13, 2024

This PR makes the default coverages grid large enough to cover the full extent of NetCDF files and not just their relevant extent.

The grid now covers an area larger than the full extent of NetCDF files:

image

In the image above:

  • green rectangle shows the current extent of the grid
  • red rectangle shows the proposed change in this PR
  • below these two rectangles there are two sample coverages with both areas used in the system (ARPAV area and ARPAV+ARPAFVG area)

This makes NCSS requests work correctly.

Here is a sample request with the default bbox:

image

And the corresponding NetCDF file, loaded in QGIS after having been downloaded:

image

Another sample request (made with the frontend UI) that asks for a smaller spatial region:

image

and the corresponding NetCDF file, loaded in QGIS after having been downloaded:

image


@ricardogsilva ricardogsilva marked this pull request as ready for review November 14, 2024 12:58
@francbartoli francbartoli merged commit 898feab into geobeyond:main Nov 14, 2024
1 check passed
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

Successfully merging this pull request may close these issues.

smaller data than what is requested
2 participants