Comments for "Astronomy"

» Submit Your Comment
Please log in to enter your comment. If you are not a registered PEAR developer, you can comment by sending an email to
» Comments
  • Till Klampaeckel  [2011-08-15 21:55 UTC]

    I think it's a great idea to spin off the code, etc..

    Do you need feedback yet on the implementation?
  • Bill Shupp  [2011-08-18 17:44 UTC]

    I think this is an awesome idea. I recently ported the SGP4/SDP4 code from the Gpredict program for real time satellite tracking and predictions to php, which you can view here:

    This code is GPL though, so I couldn't put it in PEAR. (I'm also not modifying the original coding style either, which wouldn't fit into PEAR). But there is a bit of overlap with your package, and perhaps you'll find looking through it useful. Comments are certainly welcome.

    A few comments on the current state of Science_Astronomy:

    * The code quality looks very good
    * I don't think it's necessary to support PHP4, and would like to see the use of PEAR errors removed - use exceptions instead.
    * I'd rather see class constants than define statements
    * I've gotten in the habit of using locations (America/Los_Angeles) to get the current gmt offset, which should account for daylight savings time. Perhaps you could add that as an option as well to your rise/set calculations. Better yet, maybe you could accept a DateTime() object that already has the timezone set.

    Let me know if you'd like any assistance with the package, or help working through any of the above.


    Bill Shupp
  • Guilherme Elias  [2012-01-20 15:06 UTC]

    I think it is a great idea to include your astronomy code contribution in PEAR ! It will be useful to many astronomers or people who need this kind of info for their applications.