Skip to content

Multithreaded HTTPRequests#47

Closed
wilhelmberg wants to merge 27 commits intodevelopfrom
BergWerkGIS-multithread-filesource
Closed

Multithreaded HTTPRequests#47
wilhelmberg wants to merge 27 commits intodevelopfrom
BergWerkGIS-multithread-filesource

Conversation

@wilhelmberg
Copy link
Copy Markdown

/cc @david-rhodes

Pulls in multithreaded HTTPRequests: mapbox/mapbox-sdk-cs#75

Same limitations apply:

wilhelmberg and others added 20 commits April 24, 2017 20:00
…om/mapbox/mapbox-unity-sdk into BergWerkGIS-multithread-filesource

# Conflicts:
#	sdkproject/Assets/Mapbox/Core/Unity/Utilities/HTTPRequest.cs
…into BergWerkGIS-multithread-filesource

# Conflicts:
#	sdkproject/Assets/Mapbox/Core/Unity/MapboxAccess.cs
#	sdkproject/Assets/Mapbox/Examples/SlippyVectorTerrain/TerrainVisualization.asset
#	sdkproject/ProjectSettings/EditorBuildSettings.asset
@david-rhodes
Copy link
Copy Markdown

In ThreadedSlippyTest.unity, these are the only tiles that get loaded.

screen shot 2017-05-05 at 8 56 34 am

If I stop playing and try again, Unity hangs.

@wilhelmberg
Copy link
Copy Markdown
Author

In ThreadedSlippyTest.unity, these are the only tiles that get loaded.

Oh boy, that looks bad.

Tiles are fine on my computer but I noticed up to 170 Threads during loading and almost 6GB RAM when all tiles are displayed:

image

image

I already started implementing a HTTPRequestSimple and a will add a factory next week.
Then we will be able to easily compare and profile.

@wilhelmberg wilhelmberg mentioned this pull request May 9, 2017
@wilhelmberg
Copy link
Copy Markdown
Author

Superseded by #58

@wilhelmberg wilhelmberg closed this May 9, 2017
@wilhelmberg wilhelmberg deleted the BergWerkGIS-multithread-filesource branch May 9, 2017 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants