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

Proper queued, progress, failed states #42

Open
4 tasks
aseemk opened this issue Oct 6, 2014 · 6 comments
Open
4 tasks

Proper queued, progress, failed states #42

aseemk opened this issue Oct 6, 2014 · 6 comments
Milestone

Comments

@aseemk
Copy link
Member

aseemk commented Oct 6, 2014

Thanks @nein09 for implementing some handling of not-ready state in the embed!

Not urgent, but at some point, we should revisit that and round it out:

  • We should use proper accessible HTML etc., rather than a DZI.
  • Ideally, it'd be a proper progress bar, requiring us to actually track progress on the back-end.
  • Either way, the queued/progress UI should auto-update.

Most importantly, the view page needs this feature too. To achieve that, we should:

  • Make a shared "viewer" component that both the embed and the view page use.

From memory, Zoom.it did that. There was a shared component that was a thin wrapper around the Seadragon combo viewer, which took the content info and showed either an error message, queued/progress state, or the actual DZI, depending on the content info.

@aseemk aseemk added this to the Backlog milestone Oct 6, 2014
@nein09
Copy link
Member

nein09 commented Oct 6, 2014

It seems to me that the most elegant solution to this is to implement some kind of progress callback in the DeepZoomTools, but maybe there's a better way to do it in Node that I don't know about.

@aseemk
Copy link
Member Author

aseemk commented Oct 6, 2014

That would work (perhaps the more "Node" way is to emit a progress event instead), but I think the trickier thing might be seeing if VIPS conveys progress at all.

@nein09
Copy link
Member

nein09 commented Oct 6, 2014

If VIPS won't do it, we could look at the number of levels that need to be written versus what's actually on disk at the moment.

@aseemk
Copy link
Member Author

aseemk commented Oct 6, 2014

Nice idea! Could even look at the actual generated tiles.

So this'd be setInterval (+ clearInterval when done) + fs.readdir.

@aseemk aseemk changed the title Proper queued/progress state Proper queued, progress, failed states Oct 13, 2014
@aseemk
Copy link
Member Author

aseemk commented Oct 13, 2014

Expanded the title to cover that we need to display a "failed" state too.

E.g. http://zoomhub.net/v1/content/Yev6 reports failed: true, but http://zoomhub.net/Yev6 doesn't.

@aseemk
Copy link
Member Author

aseemk commented Feb 22, 2015

Opened zoomhub/node-deepzoomtools#10 to emit progress there.

@aseemk aseemk mentioned this issue Feb 22, 2015
5 tasks
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