Duy Nguyen
Duy Nguyen
Hi @msdemlei! > I need to remove rows from registry query results. That's nasty because RegistryResults reads from the astropy.VOTable instance, and that doesn't support dropping rows so far for...
Thanks @msdemlei! I wouldn't look too closely at the implementation, as that was just a proof a concept; it was mainly a shortcut to "get a table". I was definitely...
Hi @camipacifici! (Starting to move the conversation to GitHub) I'm starting to address your review comments, and specifically regarding your request to block data loading unless WCS linking is enabled,...
> What is the objection of pixel linking anyway? I don't personally have one, but I'm following through with a request from @camipacifici to block users from loading VO data...
@camipacifici I just pushed up [10d712d](https://github.com/spacetelescope/jdaviz/pull/2872/commits/10d712de17f87e2de22763d04e933744baf7b19f) which provides a warning when: 1. WCS linking is not enabled and 2. There is pre-existing data in the data collection On point 2,...
Hi Cami, On the smaller points, I pushed up a change that should fix the "hanging" issue. I forcibly stopped the plugin whenever a load data error occurred, but forgot...
Hi Cami! Now that SkyView is back up I was able to start debugging your aforementioned issue. In short, I did get an error, but it was a download timeout...
Hi @camipacifici! I've implemented basic versions of the two requests you had: 1. https://github.com/spacetelescope/jdaviz/pull/2872/commits/03a00e39995ac2b7f9fb310d56f9f6eb5b829e0e implements the "recalculate center" button for the viewer coordinate select (which only shows when the user...
Hi @camipacifici! Another quick update. So... I know we were concerned about the performance of auto-updating coordinates during pan-zoom, but I was curious and went ahead and implemented it anyways...
Taking a quick look at the screenshots, if the two hover texts are for the different layers, they look the same to me; it appears somehow the two copies got...