POI-Bewertung: Live-Praesenz zaehlt als GPS-Beleg (Rene stand am Ort, wurde abgewiesen)
Vorher zaehlte NUR eine aufgezeichnete Tour der letzten 48h (<=50m am POI, >=2 Punkte) — die aktuelle Geraeteposition floss gar nicht ein. - DogFriendlyIn: optional user_lat/user_lon (Frontend schickt _userPos mit) - Beleg-Weg a): Geraetestandort <= 75m am POI (50m Radius + GPS-Toleranz) -> route_id=None markiert Live-Beleg; Weg b) Tour-Beleg unveraendert - Fehlermeldung nennt jetzt beide Wege (hin gehen ODER Standort aktivieren) - pytest 39 passed Bump v1239
This commit is contained in:
parent
abd7447d29
commit
6a06c9be7e
7 changed files with 61 additions and 42 deletions
|
|
@ -35,6 +35,8 @@ GPS_RADIUS_M = 50 # max. Abstand POI ↔ nächster Track-Punkt
|
|||
DWELL_MIN_POINTS = 2 # mind. so viele Track-Punkte im Radius (Verweil-Proxy)
|
||||
ROUTE_RECENCY_H = 48 # Tour darf max. so alt sein
|
||||
POI_NEAR_M = 80 # eingereichte Position muss so nah am POI sein
|
||||
LIVE_NEAR_M = 75 # Live-Präsenz: Gerätestandort ≤ so nah am POI (René 2026-06-08:
|
||||
# ~50 m Radius + GPS-Toleranz — er stand am Ort, ohne Aufzeichnung)
|
||||
DAILY_CAP = 20 # max. Beiträge pro Tag/User
|
||||
|
||||
# --- Gamification-Schwellen ---
|
||||
|
|
@ -49,6 +51,9 @@ class DogFriendlyIn(BaseModel):
|
|||
lat: float
|
||||
lon: float
|
||||
welcome: bool = True # True → dog=yes, False → dog=no (Pächterwechsel)
|
||||
# Aktuelle Gerätekoordinate (Live-Präsenz-Beleg) — optional, vom Frontend mitgeschickt
|
||||
user_lat: Optional[float] = None
|
||||
user_lon: Optional[float] = None
|
||||
|
||||
|
||||
def _verified_count(conn, uid: int) -> int:
|
||||
|
|
@ -154,33 +159,42 @@ async def mark_dog_friendly(body: DogFriendlyIn, user=Depends(get_current_user))
|
|||
if today_n >= DAILY_CAP:
|
||||
raise HTTPException(429, "Tageslimit erreicht — morgen geht's weiter.")
|
||||
|
||||
# 3) GPS-Beleg: kürzliche Tour, die am POI vorbeiführt (+ Verweil-Proxy)
|
||||
routes = conn.execute(
|
||||
"SELECT id, gps_track FROM routes "
|
||||
"WHERE user_id=? AND created_at > datetime('now', ?) ORDER BY created_at DESC",
|
||||
(uid, f'-{ROUTE_RECENCY_H} hours')
|
||||
).fetchall()
|
||||
best = None # (route_id, min_dist, points_near)
|
||||
for r in routes:
|
||||
try:
|
||||
track = json.loads(r['gps_track'])
|
||||
except Exception:
|
||||
continue
|
||||
near, mind = 0, float('inf')
|
||||
for p in track:
|
||||
d = haversine_m(body.lat, body.lon, p['lat'], p['lon'])
|
||||
if d < mind:
|
||||
mind = d
|
||||
if d <= GPS_RADIUS_M:
|
||||
near += 1
|
||||
if mind <= GPS_RADIUS_M and near >= DWELL_MIN_POINTS:
|
||||
if best is None or mind < best[1]:
|
||||
best = (r['id'], mind, near)
|
||||
# 3) GPS-Beleg — ZWEI Wege (René 2026-06-08: stand physisch am Ort und wurde
|
||||
# abgewiesen, weil nur aufgezeichnete Touren zählten):
|
||||
# a) LIVE-PRÄSENZ: aktuelle Gerätekoordinate ≤ LIVE_NEAR_M am POI
|
||||
# b) TOUR-BELEG: kürzliche Tour, die am POI vorbeiführt (+ Verweil-Proxy)
|
||||
best = None # (route_id|None, dist_m, points_near)
|
||||
if body.user_lat is not None and body.user_lon is not None:
|
||||
live_d = haversine_m(body.user_lat, body.user_lon, body.lat, body.lon)
|
||||
if live_d <= LIVE_NEAR_M:
|
||||
best = (None, live_d, 0) # route_id=None markiert Live-Beleg
|
||||
if not best:
|
||||
routes = conn.execute(
|
||||
"SELECT id, gps_track FROM routes "
|
||||
"WHERE user_id=? AND created_at > datetime('now', ?) ORDER BY created_at DESC",
|
||||
(uid, f'-{ROUTE_RECENCY_H} hours')
|
||||
).fetchall()
|
||||
for r in routes:
|
||||
try:
|
||||
track = json.loads(r['gps_track'])
|
||||
except Exception:
|
||||
continue
|
||||
near, mind = 0, float('inf')
|
||||
for p in track:
|
||||
d = haversine_m(body.lat, body.lon, p['lat'], p['lon'])
|
||||
if d < mind:
|
||||
mind = d
|
||||
if d <= GPS_RADIUS_M:
|
||||
near += 1
|
||||
if mind <= GPS_RADIUS_M and near >= DWELL_MIN_POINTS:
|
||||
if best is None or mind < best[1]:
|
||||
best = (r['id'], mind, near)
|
||||
if not best:
|
||||
raise HTTPException(
|
||||
422,
|
||||
"Kein GPS-Beleg: In deinen letzten Touren ist kein Besuch an diesem Ort. "
|
||||
"Geh mit deinem Hund dorthin, dann kannst du ihn eintragen."
|
||||
"Kein GPS-Beleg: Du bist gerade nicht an diesem Ort und in deinen letzten "
|
||||
"Touren ist kein Besuch dort. Geh mit deinem Hund hin (Standort aktiviert), "
|
||||
"dann kannst du ihn eintragen."
|
||||
)
|
||||
|
||||
# 4) Positions-Sanity gegen die bekannte POI-Koordinate
|
||||
|
|
@ -223,8 +237,9 @@ async def mark_dog_friendly(body: DogFriendlyIn, user=Depends(get_current_user))
|
|||
except Exception as e:
|
||||
logger.warning("OSM-Upload später erneut (contrib %s): %s", contrib_id, e)
|
||||
|
||||
logger.info("dog=%s erfasst: user %s, osm %s, Tour %s (%.0fm, %d Pkt), submitted=%s",
|
||||
value, uid, body.osm_id, best[0], best[1], best[2], submitted)
|
||||
logger.info("dog=%s erfasst: user %s, osm %s, Beleg %s (%.0fm, %d Pkt), submitted=%s",
|
||||
value, uid, body.osm_id,
|
||||
f"Tour {best[0]}" if best[0] else "Live-Präsenz", best[1], best[2], submitted)
|
||||
return {
|
||||
"status": "erfasst", "value": value, "verified": True, "submitted": submitted,
|
||||
"verified_count": total, "badge": total >= BADGE_AT,
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue