RA conversion fix

I failed. INDI had it right. Thank you to INDI for showing me how the centisecond conversion works.

Author: chimerasaurus

README.md

@@ -14,7 +14,10 @@ My goal in writing this is to eventually package it up and use it as a library f
 Some of this is written in a way that requires Python > 3.7. There are also Python requirements in the `requirements.txt` file.
 
 ## Running
-You will need to have your `PYTHONPATH` set up to run these right now, since it's not properly modularized.
+You will need to have your `PYTHONPATH` set up to run these right now, since it's not properly modularized. e.g.:
+
+  set PYTHONPATH=%PYTHONPATH%;x:\GitHub\ioptron-python
+  $Env:PYTHONPATH = "x:\GitHub\ioptron-python"
 
 ## Citations and sources
 

ioptron/utils.py

@@ -16,11 +16,13 @@ def convert_arc_seconds_to_degrees(seconds):
 
 def convert_arc_seconds_to_hms(seconds):
  """Converts arc seconds at 0.01 precision to arc HH:MM:SS"""
- hours = (((seconds*0.01)*(math.pi/648000))/(math.pi/12))
- minutes = Decimal(hours) % 1
- seconds = Decimal(minutes) % 1
- milliseconds = Decimal(seconds) % 1
- return (int(hours), int(minutes), int(seconds), round(float(milliseconds), 2))
+ hours = seconds / (15.0 * 60.0 * 60.0 * 100.0) #Thank you INDI.
+ print(hours)
+ minutes = (Decimal(hours) % 1) * 60
+ print(minutes)
+ seconds = (Decimal(minutes) % 1) * 60
+ print(seconds)
+ return (int(hours), int(minutes), int(seconds))
 
 def convert_j2k_to_unix_utc(sec, offset = 0):
  """Convert J2000 in 0.01 seconds to formatted UNIX in ms with offset if needed."""

tests/test_utils.py

@@ -26,9 +26,9 @@ def test_convert_arc_seconds_to_hms(self):
 
  # Run two tests
  with self.subTest():
- self.assertEqual(utils.convert_arc_seconds_to_hms(twelve_hours), (12, 0, 0, 0.0))
+ self.assertEqual(utils.convert_arc_seconds_to_hms(twelve_hours), (12, 0, 0))
  with self.subTest():
- self.assertEqual(utils.convert_arc_seconds_to_hms(twenty_four_hours), (24, 0, 0, 0.0))
+ self.assertEqual(utils.convert_arc_seconds_to_hms(twenty_four_hours), (24, 0, 0))
 
  #TODO: More tests to see if this actually works