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

Add FHM and TopoData.zip files to file download list #26

Open
volkcj opened this issue Jul 14, 2017 · 1 comment
Open

Add FHM and TopoData.zip files to file download list #26

volkcj opened this issue Jul 14, 2017 · 1 comment

Comments

@volkcj
Copy link
Contributor

volkcj commented Jul 14, 2017

This is an enhancement--
add existing model products to the dl list, e.g. FHM (FIS/HSI, NREI, and TopoData.zip).

@philipbaileynar
Copy link
Contributor

This is a big task that will need to be addressed formerly. My vision is that the existing FTP download feature is deprecated and replaced with an API download feature that works much the same way. Obviously it would get files from the API instead of the FTP.

I like the idea of keeping the existing UI and replacing the file list on the left with the types of files that we know are available (those mentioned by @volkcj in the original issue, plus others). This doesn't guarantee that the files will exist for every visit, but it will be the files that we know about a priori.

The other way to do this is go visit by visit and make call(s) to the API to discover what files are available before showing the list to the user for their selection. I don't like this because the mere act of calling the API multiple times to find out what files are available will be time consuming for lots of visits. And not all visits will have the same files and so we would need some complicated UI for letting the user pick what they want to download.

I prefer the first option where the user picks a set of visits on the main grid and then selects one or more file types. They then get all the files of the selected types for the visits selected. There might be some nuances, such as Delft where we won't know ahead of time what all the discharges are and therefore which files to download.

2017-07-13_203923

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants