Skip to content

fix: 404 errors from celestial.js on page load#57

Open
eatyourgreens wants to merge 1 commit into
genghisken:masterfrom
eatyourgreens:fix-celestialjs-404
Open

fix: 404 errors from celestial.js on page load#57
eatyourgreens wants to merge 1 commit into
genghisken:masterfrom
eatyourgreens:fix-celestialjs-404

Conversation

@eatyourgreens

@eatyourgreens eatyourgreens commented Jul 28, 2025

Copy link
Copy Markdown
Collaborator

Fix the missing data files error popup for the ATLAS sky map by adding the data/ directory to celestial.js.

Constellations and the Milky Way are shown by default, but can be turned off in the configuration script.

The ATLAS sky map, showing transients (as green dots), constellation stick figures, and the Milky Way.

Fix the missing data files error popup for the ATLAS sky map by adding the `data/` directory to `celestial.js`.

Constellations and the Milky Way are shown by default, but can be turned off in the configuration script.
@genghisken

Copy link
Copy Markdown
Owner

Can we hold off on this one please. I'm not sure what triggered the fix here. Which issue is this related to?

The files changed are deep inside the celestial.js infrastructure, which is actually outside our control. When we download a new version of celestial.js and data, all the changes here will be overwritten.

@eatyourgreens

Copy link
Copy Markdown
Collaborator Author

@genghisken this is a fix for the error popup that Celestial shows, because its data files are missing.

A popup on the home page saying: Your browser doesn't support local file loading or the file doesn't exist.

I copied the JSON files over from https://github.com/ofrohn/d3-celestial

@eatyourgreens

Copy link
Copy Markdown
Collaborator Author

Looking at the code, only mw.json is needed in order to remove that error popup.

//Milky way outline
d3.json(path + "mw.json", function(error, json) {
if (error) {
window.alert("Your Browser doesn't support local file loading or the file doesn't exist. See readme.md");
return console.warn(error);
}

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