Change latitude/longitude to float and remove rounding

Author: alisx255

backend/market/migrations/0005_sublet_true_latitude_sublet_true_longitude.py

@@ -13,15 +13,11 @@ class Migration(migrations.Migration):
         migrations.AddField(
             model_name="sublet",
             name="latitude",
-            field=models.DecimalField(
-                blank=True, decimal_places=6, max_digits=9, null=True
-            ),
+            field=models.FloatField(blank=True, null=True),
         ),
         migrations.AddField(
             model_name="sublet",
             name="longitude",
-            field=models.DecimalField(
-                blank=True, decimal_places=6, max_digits=9, null=True
-            ),
+            field=models.FloatField(blank=True, null=True),
         ),
     ]

backend/market/models.py

@@ -134,10 +134,8 @@ class Sublet(Listing):
     start_date = models.DateField()
     end_date = models.DateField()
 
-    latitude = models.DecimalField(
-        max_digits=9, decimal_places=6, null=True, blank=True)
-    longitude = models.DecimalField(
-        max_digits=9, decimal_places=6, null=True, blank=True)
+    latitude = models.FloatField(null=True, blank=True)
+    longitude = models.FloatField(null=True, blank=True)
 
 
     def clean(self):
@@ -151,8 +149,8 @@ def _calculate_approximate_location(self, latitude, longitude):
         if latitude is None or longitude is None:
             return None, None
 
-        lat_str = f"{float(latitude):.6f}"
-        lon_str = f"{float(longitude):.6f}"
+        lat_str = f"{float(latitude):.9f}"
+        lon_str = f"{float(longitude):.9f}"
         seed = hashlib.md5(f"{lat_str}{lon_str}".encode()).hexdigest()
 
         offset_factor = int(seed[:8], 16) / 0xFFFFFFFF
@@ -163,8 +161,8 @@ def _calculate_approximate_location(self, latitude, longitude):
         lat_offset = offset_distance * math.sin(angle)
         lon_offset = offset_distance * math.cos(angle)
 
-        approx_lat = Decimal(str(float(latitude) + lat_offset))
-        approx_lon = Decimal(str(float(longitude) + lon_offset))
+        approx_lat = float(latitude) + lat_offset
+        approx_lon = float(longitude) + lon_offset
         return approx_lat, approx_lon
 
     @property

backend/market/serializers.py

@@ -1,4 +1,3 @@
-from decimal import ROUND_DOWN, Decimal
 
 from django.contrib.auth import get_user_model
 from django.core.exceptions import ValidationError as ModelValidationError
@@ -110,12 +109,12 @@ class Meta:
 
     def get_latitude(self, obj):
         if obj.approximate_latitude:
-            return round(float(obj.approximate_latitude), 6)
+            return float(obj.approximate_latitude)
         return None
 
     def get_longitude(self, obj):
         if obj.approximate_longitude:
-            return round(float(obj.approximate_longitude), 6)
+            return float(obj.approximate_longitude)
         return None
 
 # Unified serializer for all listing types (Items and Sublets); used for CRUD operations
@@ -277,11 +276,9 @@ def _create_sublet(self, validated_data, additional_data):
 
 
         if latitude is not None:
-            latitude = Decimal(str(latitude)).quantize(
-                Decimal("0.000001"), rounding=ROUND_DOWN)
+            latitude = float(latitude)
         if longitude is not None:
-            longitude = Decimal(str(longitude)).quantize(
-                Decimal("0.000001"), rounding=ROUND_DOWN)
+            longitude = float(longitude)
 
         sublet = Sublet.objects.create(
             street_address=additional_data.get("street_address"),
@@ -348,15 +345,9 @@ def _update_sublet(self, instance, additional_data):
             if field in additional_data:
                 setattr(sublet, field, additional_data[field])
         if "latitude" in additional_data:
-            latitude = Decimal(str(additional_data["latitude"])).quantize(
-                Decimal("0.000001"), rounding=ROUND_DOWN
-            )
-            sublet.latitude = latitude
+            sublet.latitude = float(additional_data["latitude"])
         if "longitude" in additional_data:
-            longitude = Decimal(str(additional_data["longitude"])).quantize(
-                Decimal("0.000001"), rounding=ROUND_DOWN
-            )
-            sublet.longitude = longitude
+            sublet.longitude = float(additional_data["longitude"])
         sublet.full_clean()
         sublet.save()