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

Update the widget to research-app 0.12 #342

Merged
merged 3 commits into from
Dec 1, 2022

Conversation

pkgw
Copy link
Contributor

@pkgw pkgw commented Dec 1, 2022

Overview

Here we'll try to update the widget to the latest release of the research app, 0.12, which contains the upgrade to Vue 3.

This may take some futzing because the new app contains the new engine which has some fixes to FITS rendering, so the test images may need updating.

Checklist

  • Changed code has some test coverage: N/A
  • Changes in functionality documented: N/A

I recently updated the `builtin-image-sets.wtml` file to match the
`imagesets6.xml` managed through the new-ish `wwt-core-catalogs`
repository. I created `builtin-image-sets.wtml` back when we couldn't
maintain the WWT servers in any reasonable way, so there was no
mechanism to update the imageset list. Now we have much better systems
for that and the `imagesets6` file was the one that was more recently
maintained.

Anyway, as part of that maintenance, the builtin SFD map switched from
the virtualearth.net data source to the WWT tiles.aspx data source. And
it turns out that they are slightly rotated relative to one another!
Fortunately, the new version (tiles.aspx) is more correct. The imagery
that we were using in our tests up until now used the older version, so
we need to update the reference images here.

Unrelated to this change, but it looks like we should check out the
other datasets that are served off of virtualearth.net and see whether
any of them also have astrometric issues.
We have been tracking down more pixel-level positioning/sizing offsets
in the FITS display stack, which has led these reference images to
change significantly enough to need updating.
@pkgw
Copy link
Contributor Author

pkgw commented Dec 1, 2022

Hmm, discovered something interesting here. I'll copy/paste from my commit message:

I recently updated the builtin-image-sets.wtml file to match the imagesets6.xml managed through the new-ish wwt-core-catalogs repository. I created builtin-image-sets.wtml back when we couldn't maintain the WWT servers in any reasonable way, so there was no mechanism to update the imageset list. Now we have much better systems for that and the imagesets6 file was the one that was more recently maintained.

Anyway, as part of that maintenance, the builtin SFD map switched from the virtualearth.net data source to the WWT tiles.aspx data source. And it turns out that they are slightly rotated relative to one another! Fortunately, the new version (tiles.aspx) is more correct. The imagery that we were using in our tests up until now used the older version, so we need to update the reference images here.

Unrelated to this change, but it looks like we should check out the other datasets that are served off of virtualearth.net and see whether any of them also have astrometric issues.

@codecov
Copy link

codecov bot commented Dec 1, 2022

Codecov Report

Merging #342 (c46cbe6) into master (47e19f6) will not change coverage.
The diff coverage is n/a.

❗ Current head c46cbe6 differs from pull request most recent head fcfaf9e. Consider uploading reports for the commit fcfaf9e to get more accurate results

@@           Coverage Diff           @@
##           master     #342   +/-   ##
=======================================
  Coverage   55.68%   55.68%           
=======================================
  Files          23       23           
  Lines        2753     2753           
=======================================
  Hits         1533     1533           
  Misses       1220     1220           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@pkgw pkgw merged commit b0feb93 into WorldWideTelescope:master Dec 1, 2022
@pkgw pkgw deleted the app-update branch December 1, 2022 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant