Coord::bboxAround() works only on Earth
Open, LowPublic
Actions

Assigned To

None

Authored By

	MaxSem
	Mar 10 2017, 1:21 AM

Description

Math::wrapAround( $res->lon1, $res->lon2, -180, 180 );

Needs to take the globe into account.

	Subject	Repo	Branch	Lines +/-
	Fix distance/distanceTo to work with globes other than Earth	mediawiki/extensions/GeoData	master	+27 -17
	Fix Coord::bboxAround() not supporting non-Earth globes	mediawiki/extensions/GeoData	master	+21 -2

Gehel triaged this task as Low priority.May 6 2024, 3:37 PM

Gehel moved this task from needs triage to Geodata on the Discovery-Search board.

thiemowmde subscribed.

Change #1031548 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/GeoData@master] [WIP] Change Math::distance to accept Coord instances

Change #1040728 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/GeoData@master] Fix Coord::bboxAround() not supporting non-Earth globes

Change #1040728 merged by jenkins-bot:

[mediawiki/extensions/GeoData@master] Fix Coord::bboxAround() not supporting non-Earth globes