at the top of the scripts it says what the script is

@@lorilongwell5451 not always, really got confused with the Update function script that didn't have a name

I did the same for a long time, now i can start a project and mess around a bit with ideas, it can be fun and useful at the same time watching these kind of stuff, just to expand your knowledge and way of thinking, you'll have a starting point if you ever decided to play around with game dev.

using System.Collections; using System.Collections.Generic; using UnityEditor; using UnityEngine; public class TakeScreenshot : MonoBehaviour { Camera cam; public string pathFolder; public List sceneObjects; public List dataObjects; private void Awake() { cam = GetComponent(); } [ContextMenu("Screenshot")] private void ProcessScreenshots() { StartCoroutine(Screenshot()); } private IEnumerator Screenshot() { for (int i = 0; i < sceneObjects.Count; i++) { GameObject obj = sceneObjects[i]; InventoryItemData data = dataObjects[i]; obj.gameObject.SetActive(true); yield return null; TakeShot($"{Application.dataPath}/{pathFolder}/{data.id}_Icon.png"); yield return null; obj.gameObject.SetActive(false); Sprite s = AssetDatabase.LoadAssetAtPath($"Assets/{pathFolder}/{data.id}_Icon.png"); if(s!= null) { data.icon = s; EditorUtility.SetDirty(data); } yield return null; } } public void TakeShot(string fullPath) { if(cam == null) { cam = GetComponent(); } RenderTexture rt = new RenderTexture(256, 256, 24); cam.targetTexture = rt; Texture2D screenShot = new Texture2D(256, 256, TextureFormat.RGBA32, false); cam.Render(); RenderTexture.active = rt; screenShot.ReadPixels(new Rect(0, 0, 256, 256 ), 0, 0); cam.targetTexture = null; RenderTexture.active = null; if (Application.isEditor) { DestroyImmediate(rt); } else { Destroy(rt); } byte[] bytes = screenShot.EncodeToPNG(); System.IO.File.WriteAllBytes(fullPath, bytes); #if UNITY_EDITOR AssetDatabase.Refresh(); #endif } }

@@christofstanits hi, i dont know why is keep saving as one sprite any solutions? same code *edit you know what, I'm dumb, I forgot to assign the id

@@christofstanits Thank you so much! This is an absolute life-saver!

@Eigulite I didn't (I have the same problem)

@@crazytm8942 it goes in the script InventorySystem in between void awake and void add.

@@archive5500 thank you!

There's a one-line screenshot method in unity which captures the game view and saves it at the location you pass as a parameter

@@mohammadsadeghlavaie5560 lol did you not watch the video?

.current is explained in his other video about events, basically go to the top and make put in "public static InventorySystem current;", then go to the private void Awake() and put "current = this;", as for UIInventoryItemSlot i'd like to know myself i'm going crazy over here

@@TactfulWaggle UIInventoryItemSlot is supposed to be the name of the slot script mentioned a few seconds earlier

Do you know where "ItemController" comes from!? at 7:40 I don't have itemController and it doesn't work if I do it with my script that has OnHandlePickupItem in it ;_;

@@TheCatMurgatroyd No Clue man, I never got around to putting this to work. Just went on and found another tutorial more in depth

@@brurrik Do you remember which one? I followed another one but was too stupid ;_;

did you solve how to get the on inventorychanged event i cant seem to think of how to do it?

@@gedjucius2418 solved yet ??

@@xxragingkillerx8357 just subdcribe to an event that is called every time you change the inventory

this. i mostly needed to see how to make a menu and it just gets zero attention, annoying.

change your InventorySystem script like this. public class InventorySystem : MonoBehaviour { public static InventorySystem current; private Dictionary m_itemDictionary; public List inventory; private void Awake() { current = this; inventory = new List(); m_itemDictionary = new Dictionary(); }

@@BANANADOMAU which part of the script do I replace this part with

@@koopee8577 just replace some part of the InventorySystem class script added this on variable declarations "public static InventorySystem current;" added this on Awake method "current = this;"

@@BANANADOMAU Thank you so much! I've been looking for this SINGLE piece of code forever and couldn't figure it out myself.

@@BANANADOMAU thank you so much! How'd you figure that out?

yeah, there are some hole in the tut.

yea exactly, like it's a good video except for the fact that there are huge holes in context and he only hearts comments that are positive, these comments go un-replied to and it gets infuriating to a point

.current is like public static InventorySystem current;

Maybe its a late answer, but in case anybody searchs for the same thing declare a "public static InventorySystem current;" variable inside the InventorySystem script and inside the "private void Awake" function instantiate the current variable (currently empty) using : "current = this;" if you do it like he did in the tutorial Hope its helpful for any dev wondering in the future

@@DaxVIII Veeery new to unity and yes, yes it was helpful. Searched for this for half an hour before coming to the comments here. Thank you

did you figure out how to use the items once they're in your inventory?

Ironic

@@martimking1craft *Iconic

how is camera not deprecated, why targettexture doesnt work ? help pls

@Elliott Childs Yhea thats the easy solution, I was thinking in doing that, but where is the fun part of programming? xd (if I cant solve the errors im going to do that)

As mentioned in the video, I turned it into a singleton. Lookup singletons in C# / Unity for more info.

@@GameDevGuide Yeah my bad on that part. What about the playlist?

@@GameDevGuide where did you mention it?

@@GameDevGuide still lost lmao

@@GameDevGuide Dude do you even know how "a tutorial" is work? Not everyone know what the fuck is a singleton, and this kind of scripts need to be completed so it can work.

Can you please explain how you did it? Thanks in advance :)

@@lucidjuice101 I followed his other tutorial "how to build an event system"

@@RoStepMusic Thanks for the info!

@@RoStepMusic i tried to follow that video but still seem to not know to do it

@@gedjucius2418 It is not that hard even for a beginner. You should not dive this deep if you don't have basic knowledge. Join his community discord server and ask for help. They will help you in no time.

Ah sad.... I commented the exact same question yet I see you asked this 5 months ago with no response

for me i changed the background type to uninitialized and it worked, make sure youre encoding to png not jpg

Check out the event system video here on the channel!

No

Im in the same place. Any info on this would be great.

I just wanted to circle back as I've learnt another way to solve the problem above; You can forcibly serialize using "[field: SerializeField]" and it achieves the same outcome as removing the getter/setter declaration.

@@Seriphis2200 I did both ways but still can't serialize that list

​@@HackEndLess Check that you have your InventoryItem class "[Serializable]", that you've used "[field: serializefield]" on each of the items that have the getters defined in line. In the InventorySystem class you will need to do the same field declaration for the "inventory" list defined within it... you cant serialise the dictionary so if you're expecting that to appear you're out of luck.

​@@Seriphis2200 I did everything you and others said but still can't serialize inventory list nothing worked for me, I even upgraded Unity version to 2020.3.27f1 from 2020.3.20f1. Anyway thanks for the help. It's not a big deal because system works. So I give up on that.

@@HackEndLess Thats a bummer, if its working then great but thats super frustrating. Best of luck with your project.

remove the {get set} from both the InventorySystem and the InventoryItem scripts :)

@@TVBUDDHAS Thank you- this helped! Why is that?

Same I wish he would show the whole thing

using System.Collections; using System.Collections.Generic; using UnityEditor; using UnityEngine; public class TakeScreenshot : MonoBehaviour { Camera cam; public string pathFolder; public List sceneObjects; public List dataObjects; private void Awake() { cam = GetComponent(); } [ContextMenu("Screenshot")] private void ProcessScreenshots() { StartCoroutine(Screenshot()); } private IEnumerator Screenshot() { for (int i = 0; i < sceneObjects.Count; i++) { GameObject obj = sceneObjects[i]; InventoryItemData data = dataObjects[i]; obj.gameObject.SetActive(true); yield return null; TakeShot($"{Application.dataPath}/{pathFolder}/{data.id}_Icon.png"); yield return null; obj.gameObject.SetActive(false); Sprite s = AssetDatabase.LoadAssetAtPath($"Assets/{pathFolder}/{data.id}_Icon.png"); if(s!= null) { data.icon = s; EditorUtility.SetDirty(data); } yield return null; } } public void TakeShot(string fullPath) { if(cam == null) { cam = GetComponent(); } RenderTexture rt = new RenderTexture(256, 256, 24); cam.targetTexture = rt; Texture2D screenShot = new Texture2D(256, 256, TextureFormat.RGBA32, false); cam.Render(); RenderTexture.active = rt; screenShot.ReadPixels(new Rect(0, 0, 256, 256 ), 0, 0); cam.targetTexture = null; RenderTexture.active = null; if (Application.isEditor) { DestroyImmediate(rt); } else { Destroy(rt); } byte[] bytes = screenShot.EncodeToPNG(); System.IO.File.WriteAllBytes(fullPath, bytes); #if UNITY_EDITOR AssetDatabase.Refresh(); #endif } }

@@aleks_gavs Thank you sooo much!

@@TheJakeblake I copied it from another comment)

@@aleks_gavs oh for real? I did not found that comment, thanks none the less, your comment is much more to the top I assume so this saves me time, which I'm glad, Have a Good Day.

two ways, one way just set the list to serializable, second way just use debug mode

@@huoguoji1354 I did set the list to serializable, but that did not make it show up.

@@raniem4368 maybe use the debug mode method， that usually works

@@raniem4368 the debug work for me

@@huoguoji1354 awesome. Thanks

Try adding "using System;" to the top.

same. some really good ideas, but some basics glossed over.

agreed. I ended up going to another tutorial that does things step by step explaining each step instead of: here code good luck.

Because this method supports saving from any camera, regardless of what's in the game view.

@@GameDevGuide that's cool 👌

Did you ever fix it cause same

You need to create a singleton with the inventory script

hey yea i have a same error like @Niksstuff so' how can i do it? @AleyisErussard

@@_Business.Guide_ I've said it, you need to create a singleton

This part was very glossed over, so I'll explain. A singleton is a kind of static object, which can be accessed from any script. To do that, you'll have to instantiate an instance of the singleton. Googling what "current" means in unity will only bring you to the one regarding Events, which isn't accurate. For my singletons, I use the name instance. So, at the top of the InventorySystem script, you should write: private static InventorySystem _instance; public static InventorySystem Instance => _instance; Then, inside your awake function (create one if you don't have one!), write some code that makes sure there is ONLY ONE instance of the singleton: if (_instance != null) { Debug.Log("there's already an instance of this singleton!") Destroy(_instance); return; } _instance = this; Now, when implementing this code, replace "current" with Instance when it's outside of the InventorySystem script, and _instance when it's inside.