-
Notifications
You must be signed in to change notification settings - Fork 15
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
Try to implement "generic astro image API" with ipyastroimage #1
Comments
On my list for sprinting next week 😀. This is mostly javascript and acts as a drop-in replacement for the 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. |
Let me know which days, happy to stand by to help.
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?).
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. |
Did this go anywhere? I am revisiting the idea with |
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
The text was updated successfully, but these errors were encountered: