bqplot-image-gl icon indicating copy to clipboard operation
bqplot-image-gl copied to clipboard

Try to implement "generic astro image API" with ipyastroimage

Open eteq opened this issue 6 years ago • 3 comments

As part of some work trying to build a python-friendly API for image viewers, I built this "concept" document: https://gist.github.com/eteq/bb625148969db51b2b03b71e92ca5231

It would be an interesting experiment to see if this image viewer can meet that API. The target case I had in mind with this was indeed notebooks, so this might be a perfect simple test case for it...?

cc @mwcraig @maartenbreddels

eteq avatar Nov 29 '18 04:11 eteq

On my list for sprinting next week 😀. This is mostly javascript and acts as a drop-in replacement for the Image mark in bqplot. I had started trying to work up a bqplot-based version of the image viewer over the summer but the default Image in bqplot is a little limited, and it can take a while for PRs to get reviewed and merged.

My understanding from a conversation on slack earlier this summer with @astrofrog is that this (ipyastroimage) will be used as part of glue-jupyter but that they don't intend to wrap it in a separate API for standalone use.

@maartenbreddels kept this piece of the glue-jupyter project separate from the rest of the glue repository so that we can use it in astrowidgets with minimal dependencies.

mwcraig avatar Nov 29 '18 14:11 mwcraig

On my list for sprinting next week 😀

Let me know which days, happy to stand by to help.

and it can take a while for PRs to get reviewed and merged.

Yes, and I think also it makes more sense to be outside of bqplot so we can move faster, break the api when needed and put in really astro specific parts (do I hear WCS?).

will be used as part of glue-jupyter but that they don't intend to wrap it in a separate API for standalone use.

First part yes, second part is up to you/@astrofrog I think. It could be that the API as suggested by @eteq lives outside the repo, and this repo only provides the minimum required to make that happen. On the other hand it may be too scattered, and live in this repo as well. I think it's better for @astrofrog to decide on that part.

I personally would not mind this repo to move or merge, the main reason this Image mark is here is so we can move faster.

maartenbreddels avatar Nov 29 '18 14:11 maartenbreddels

Did this go anywhere? I am revisiting the idea with HeatMap (because I forgot this repo existed) but I find that the mouse hover/click events are not giving me cursor info.

xref astropy/astrowidgets#72 and astropy/astrowidgets#121

pllim avatar Jan 29 '21 20:01 pllim