Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[android] Remove MathUtils #11182

Closed
Guardiola31337 opened this issue Feb 13, 2018 · 2 comments
Closed

[android] Remove MathUtils #11182

Guardiola31337 opened this issue Feb 13, 2018 · 2 comments
Labels
Android Mapbox Maps SDK for Android
Milestone
android-v6.0.0-boba

Comments

@Guardiola31337
Copy link
Contributor

We should remove MathUtils once mapbox/mapbox-java#719 lands and a new beta of MAS 3.0 is available.

Need to keep this in mind in future releases of Mapbox Maps SDK as MAS moves forward.

cc @tobrun @cammace @osana @electrostat
@Guardiola31337 Guardiola31337 added the Android Mapbox Maps SDK for Android label Feb 13, 2018
@Guardiola31337 Guardiola31337 added this to the android-v6.0.0 milestone Feb 13, 2018
@Guardiola31337 Guardiola31337 mentioned this issue Feb 13, 2018
Merged
@tobrun
Copy link
Member

tobrun commented Feb 13, 2018 •
edited
Loading

I'm only now noticing the impact what this would have, the issue is that MathUtils will be included in mas-services, while the SDK doesn't use that dependency (only telem + geojson). Adding such a dependency just for an utils class seems like overkill to me.

@lilykaiser
Copy link

Can this be closed?

@tobrun tobrun closed this as completed Feb 13, 2018
@Guardiola31337 Guardiola31337 mentioned this issue Feb 13, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Milestone
android-v6.0.0-boba
Development

No branches or pull requests
3 participants
@Guardiola31337 @tobrun @lilykaiser