V1.1, Optimizing update

Assets/Scenes/Game.unity

@@ -38,7 +38,7 @@ RenderSettings:
  m_ReflectionIntensity: 1
  m_CustomReflection: {fileID: 0}
  m_Sun: {fileID: 0}
- m_IndirectSpecularColor: {r: 0.37661862, g: 0.34757808, b: 0.32425174, a: 1}
+ m_IndirectSpecularColor: {r: 0.46172708, g: 0.4317774, b: 0.3634297, a: 1}
  m_UseRadianceAmbientProbe: 0
 --- !u!157 &3
 LightmapSettings:

Assets/Scripts/ItemRenderer.cs

@@ -14,6 +14,15 @@
 public class ItemRenderer : MonoBehaviour
 {
 
+ // Optimize update 1.1 (Item thumbnails caching)
+ // The majority of the items in the game do not have random colors.
+ // For all items that do not have random colors we can save the rendered image so that
+ // we do not need to render it again every time the market or inventory is loaded.
+ // Rendering the thumbnails is the heaviest part of the game and > 100 renders at one time will
+ // crash on iPhone 8.
+
+ public Dictionary<int, Texture> cachedRenders = new Dictionary<int, Texture>();
+
  public Camera itemCamera;
  public Transform container;
  public RenderTexture texture;
@@ -29,6 +38,47 @@ public class ItemRenderer : MonoBehaviour
 
  Quaternion containerDefaultRotation;
 
+ public IEnumerator RenderItem(Item item, IndexedItem origin, RawImage destination) {
+ WaitForEndOfFrame endOfFrame = new WaitForEndOfFrame();
+
+ InitiateRenderItem(item);
+
+ Texture2D render = new Texture2D(itemCamera.targetTexture.width, itemCamera.targetTexture.height);
+
+ RenderTexture.active = texture;
+
+ itemCamera.targetTexture = texture;
+ itemCamera.Render();
+
+ render.ReadPixels(new Rect(0, 0, itemCamera.targetTexture.width, itemCamera.targetTexture.height), 0, 0);
+ render.Apply();
+
+ destination.texture = render;
+
+ // Check if this item type has random colors. If it has we can't cache the render
+ // because all items of this type has diffrent colors.
+ if (!origin.pattern) {
+ cachedRenders.Add(origin.id, render);
+ }
+
+ yield return endOfFrame;
+ }
+
+ public void SetItemThumbnail(Item item, RawImage destination) {
+ IndexedItem origin = online.GetIndexedItem(item.item);
+
+
+ // Check if this item render is cached (if it has unique pattern it will never be cached so
+ // we don't have to check that)
+ if (cachedRenders.ContainsKey(origin.id)) {
+ destination.texture = cachedRenders[origin.id];
+ Debug.Log("Used cached render");
+ } else {
+ Debug.Log("Rendered new");
+ StartCoroutine(RenderItem(item, origin, destination));
+ }
+
+ }
 
  void Start() {
 
@@ -73,27 +123,7 @@ public GameObject InitiateFinishedItem(Item item, Transform parent) {
  return model;
  }
 
- public IEnumerator RenderItem(Item item, RawImage destination) {
-
- WaitForEndOfFrame endOfFrame = new WaitForEndOfFrame();
-
- InitiateRenderItem(item);
-
- Texture2D render = new Texture2D(itemCamera.targetTexture.width, itemCamera.targetTexture.height);
-
- RenderTexture.active = texture;
-
- itemCamera.targetTexture = texture;
- itemCamera.Render();
-
- render.ReadPixels(new Rect(0, 0, itemCamera.targetTexture.width, itemCamera.targetTexture.height), 0, 0);
- render.Apply();
-
- destination.texture = render;
-
- yield return endOfFrame;
-
- }
+
 
 
 }

Assets/Scripts/MarketManager.cs

@@ -48,6 +48,11 @@ public class MarketManager : MonoBehaviour
  public int activeItemListing = -1;
 
 
+ /**
+ * OnListings is when the user has clicked on an item on the left side of the market
+ * This will load all items of that type and present them on the right side of the market
+ * view.
+ */
  public void OnListings(string data) {
  ItemListings listings = JsonConvert.DeserializeObject<ItemListings>(data);
  IndexedItem origin = connection.GetIndexedItem(listings.items[0].item.item);
@@ -61,24 +66,15 @@ public void OnListings(string data) {
 
  ClearTransform(listingsContent);
 
- RawImage firstThumb = null;
-
  for(int i = 0; i < listings.items.Length; i++) {
 
  ItemListing listing = listings.items[i];
 
  Transform listingObject = Instantiate(listingPrefab, listingsContent).transform;
 
  RawImage thumbnail = listingObject.Find("Thumbnail").GetComponent<RawImage>();
- if (i == 0) firstThumb = thumbnail;
-
- if(!origin.pattern && i > 0) {
- thumbnail.texture = firstThumb.texture;
- } else {
- StartCoroutine(renderer.RenderItem(listing.item, thumbnail));
- }
 
-
+ renderer.SetItemThumbnail(listing.item, thumbnail);
 
  listingObject.Find("Seller").GetComponent<Text>().text = listing.seller;
  listingObject.Find("ListingPrice").GetComponent<Text>().text = listing.item.price.ToString();
@@ -88,6 +84,10 @@ public void OnListings(string data) {
  }
  }
 
+ /**
+ * Market front is the left side of the market. This is where all items of the same type
+ * are grouped together.
+ */
 
  public void OnMarketFront(string data) {
 
@@ -99,7 +99,7 @@ public void OnMarketFront(string data) {
  Transform entry = Instantiate(marketFrontEntryPrefab, marketFrontScrollContent).transform;
 
  RawImage thumbnail = entry.Find("Thumbnail").GetComponent<RawImage>();
- StartCoroutine(renderer.RenderItem(frontEntry.item, thumbnail));
+ renderer.SetItemThumbnail(frontEntry.item, thumbnail);
 
  entry.Find("ItemTitle").GetComponent<Text>().text = origin.name;
  entry.Find("ItemBorder").GetComponent<Image>().color = origin.rarity.color;
@@ -111,6 +111,7 @@ public void OnMarketFront(string data) {
  });
  }
 
+ // Open the first or last opened item listings
  if(itemFrontData.Length > 0) {
  if (activeItemListing == -1 || !IsItemIndexListed(activeItemListing)) activeItemListing = itemFrontData[0].item.item;
  OpenMarketListings(activeItemListing);
@@ -140,15 +141,4 @@ void ClearTransform(Transform parent) {
  public void UpdateMarketFront() {
  connection.Send("get_market_front");
  }
-
- void Start()
- {
-
- }
-
- // Update is called once per frame
- void Update()
- {
-
- }
 }

Assets/Scripts/OnlineConnection.cs

@@ -1,7 +1,6 @@
 /**
  * This script manages the websocket connection
  * between the client and server.
- * 
  */
 
 using System.Collections;
@@ -67,6 +66,8 @@ public class OnlineConnection : MonoBehaviour
  async void Start() {
 
  previousGoldAmount = PlayerPrefs.GetInt("gold");
+
+
  token = PlayerPrefs.GetString("token");
 
  ws = new WebSocket("wss://dungeon.ygstr.com");
@@ -272,7 +273,7 @@ void LoadInventory() {
  // Set color of item border
  itemSlot.transform.Find("Border").GetComponent<Image>().color = origin.rarity.color;
 
- StartCoroutine(renderer.RenderItem(inventory[i], itemSlot.GetComponentInChildren<RawImage>()));
+ renderer.SetItemThumbnail(inventory[i], itemSlot.GetComponentInChildren<RawImage>());
  }
 
 

Assets/Scripts/OnlineRaidManager.cs

@@ -145,7 +145,7 @@ public void OnRaidEnd(string json) {
  im.Inspect(item, true);
  });
 
- StartCoroutine(renderer.RenderItem(item, earnedLootSlot.Find("Thumbnail").GetComponent<RawImage>()));
+ renderer.SetItemThumbnail(item, earnedLootSlot.Find("Thumbnail").GetComponent<RawImage>());
  }
 
  goldDispalyed = 0;