Page 1 of 1

Blackhole script

Posted: 12.09.2018, 00:27
by Anthony_B_Russo10
This is script I had made for one of my addons and when I try and run it, Celestia gives me an error.
CelestiaError.png

I don't know were the error is but if you do please reply.

Posted: 12.09.2018, 08:59
by Goofy
Hi Anthony_B_Russo10.
Sorry but without your addon I cannot check it.
But your script is very short, so it will be easy to try this way:
at half of the script put a new line with a single
" } "
This will stop the execution at that point.
So launch the script and check if the error is still there or not.
If yes, you know that it's in the first half of the script, so apply the same procedure there.
And so on.
At the end you'll have the exact script line that gives error.

And now my question to the authors:
Celestia knows exactly what is and where is the error, so why the error message is generic and doesn't point us to the wrong script line?
Is it possible to add this feature, that would save lot of time in scripting, please?
Thanks a lot.
Bye
Goofy :smile:

Added after 2 minutes 21 seconds:
Anyway, I think the error is this:

setfaintestautomag45deg{ magnitude 8.5 }

Celestia has not the command

setfaintestautomag45deg{

Bye
Goofy :smile:

Posted: 12.09.2018, 11:29
by Anthony_B_Russo10

Posted: 14.09.2018, 04:43
by gironde
setfaintestautomag45deg{ magnitude 8.5 }

this command is on https://en.wikibooks.org/wiki/Celestia/Celx_Scrip ... ommand_setfaintestautomag45deg


and a space between deg and { ? setfaintestautomag45deg { magnitude 8.5 }

Added after 4 hours 42 minutes:
I got this error message with a script I had just translated into French.
I found the translation site the text sequence '!' by "!".

That's what made the mistake (").


Check also the presence of the pointstars parameter in renderflags {".........."}
renderflags {clear "pointstars | nightmaps | galaxies | markers"}
This setting is no longer used in Celestia 1.6.1, but do not cause an error if it is present, it is simply ignored.

:hi:

Posted: 14.09.2018, 10:29
by Goofy
...and a space between deg and { ? setfaintestautomag45deg { magnitude 8.5 }

Gironde, probably you didn't read my message #2, where I told exactly the same thing.
Goofy :smile:

Posted: 15.09.2018, 10:15
by gironde
Yes Goofy, not viewed :smile:

"the great spirit meets " :smile:

Posted: 15.09.2018, 13:19
by Goofy
Gironde, no problem at all.
BTW, do you agree on what I said?
Celestia knows exactly what is and where is the error, so why the error message is generic and doesn't point us to the wrong script line?
I think that a solution to this nonsense could save us a lot of time, wasted to find errors in scripts that may be thousands lines long (my Mars script is more than 7000 lines ...) :cyclop:
Bye.
Goofy :smile:

Posted: 17.09.2018, 19:47
by Anthony_B_Russo10
If either of you want you can make script to go with the addon.