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

added previewer for shp and geotiff #36

Merged
merged 7 commits into from
Jul 12, 2023
Merged

Conversation

schorcht-ioer
Copy link

Dear community,

These are previewers for Esri Shapefiles and Geotiffs.

All previewers have a spinner and a file size limit. If the file size limit is exceeded, an error is thrown.
Before the file is loaded, the file size is queried first. The file is loaded only when the file size falls below the threshold value (file_size_limit).

Esri Shapefiles

Parameters

file_size_limit = 20 [in MB]

For Esri Shapefiles, there is only the file size threshold parameter (file_size_limit). If the application should run too slow, it is recommended to reduce this value.

Geotiffs

Parameters

file_size_limit = 15 [in MB]
row_col_limit = 50000 [number of columns or rows]
load_timeout = 30 [in seconds]

With geotiffs it is also possible to define a maximum number of rows and columns (since compressed tiffs can have a small data size but still be too large to view). Since no special mimetype was available for geotiffs (only tiff), an error is thrown in case of missing projection information (or unsupported tiff variants). Also, there is a problem with the geotiff reader used that an error cannot be caught :\ Therefore, a timeout was defined in which the raster should be loaded (if not, an error is thrown).

cheers,
martin

@qqmyers
Copy link
Member

qqmyers commented Jul 11, 2023

@schorcht-ioer - can you add information about the hardcoded size/row/time limits that you have here in the pull request to the comments in 5.2curlcommands.md and mention how they can be changed (someone would have to download these previewers locally and edit their local copy rather than using the hosted versions)?

With that change I'll go ahead and merge.

@schorcht-ioer
Copy link
Author

ok, @qqmyers , I have added additional information about the limits in 5.2curlcommands.md.

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.

2 participants