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 lunar loader link #6

Merged
merged 2 commits into from
Feb 15, 2024
Merged

Conversation

patudom
Copy link
Contributor

@patudom patudom commented Feb 14, 2024

No description provided.

@patudom
Copy link
Contributor Author

patudom commented Feb 14, 2024

In the console, there are a bunch of 404 (Not Found) proxy service errors, which maybe comes from the fact that I am running from localhost instead of the hosted url of the stories? In any case, the image tiles are served with no problems. @Carifio24, please let us know if there is anything we need to do to address this.

https://worldwidetelescope.org/webserviceproxy.aspx?targeturl=https%3A%2F%2Fprojects.cosmicds.cfa.harvard.edu%2Fcds-website%2Fwwt-content%2FJWST-Brick-Annotation%2F2%2F3%2F3_0.png 404 (Not Found)

@Carifio24
Copy link
Member

Carifio24 commented Feb 15, 2024

So I took a look at these 404s, and these pop up when WWT is looking for tiles that don't exist in our image. For example, in the error that you posted the target URL is https://projects.cosmicds.cfa.harvard.edu/cds-website/wwt-content/JWST-Brick-wostars/2/3/3_0.png, but that tile doesn't exist (see the 2 folder here).

I don't remember off the top of my head exactly how the tile indexing works, but my guess is that where that tile would be is blank on our image (the image is at a ~45 degree angle relative to the equator, so my guess is that this tile would be one of the corners). So I think this is fine.

@Carifio24 Carifio24 merged commit 42cf5c5 into cosmicds:main Feb 15, 2024
1 check passed
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.

2 participants