[0.25] ATC - Alternative Tree Configurator [15.10. - V0.5.1]

Atrius129 · October 28, 2014

Hello. I wanted to drop in and say I have released the TechManager mod. It has the ability to add new tech nodes and is licensed MIT, so you can use it any way you like. There appears to be a lot of interest in adding this functionality to ATC (and I mention you guys by name in my initial posting), so I hope to hear back. Thanks.
TechManager

This worked perfectly for me! Thank you.

Ixonal · October 28, 2014

I think I'm about | | this close to getting this scat working, bare with me...

Edit 1: I'm still getting this exception the first time the tech tree loads,



Exception in NEWNODE processing - System.NullReferenceException: Object reference not set to an instance of an object

  at RDNode.Setup () [0x00000] in <filename unknown>:0 

  at RDNode.DrawArrow (UnityEngine.Material mat) [0x00000] in <filename unknown>:0 

  at RDNode.ShowArrows (Boolean show, UnityEngine.Material mat, .Parent[] list) [0x00000] in <filename unknown>:0 

  at RDNode.InitializeArrows () [0x00000] in <filename unknown>:0 

  at RDNode.Setup () [0x00000] in <filename unknown>:0 

  at ATC.TechChanger.processNewNodes (.ConfigNode tree) [0x00000] in <filename unknown>:0

And there are a couple weird rendering issues that I have to iron out, but I am able to create a proper looking node right now...

Edited October 28, 2014 by Ixonal

anonish · October 28, 2014

I think I'm about | | this close to getting this cat working, bare with me...

I'd be happy to answer questions if you have any, assuming you are trying to roll TechManager into ATC. Not sure which cat you are working on.

Edit 1: I'm still getting this exception the first time the tech tree loads,

The simplicity of tm can be a little misleading. Things have to happen at certain times to work out correctly. For example, when and how exactly you pick a node to gen prefabNode from. The allocated GameObject must be deactivated before adding RDNode and RDTech components, RDTech must invoke Start() before activation, etc.

EDIT: nvm, that's not Setup.

Got something I can look at?

Edited October 28, 2014 by anonish

Ixonal · October 28, 2014

I'd be happy to answer questions if you have any, assuming you are trying to roll TechManager into ATC. Not sure which cat you are working on.
The simplicity of tm can be a little misleading. Things have to happen at certain times to work out correctly. For example, when and how exactly you pick a node to gen prefabNode from. The allocated GameObject must be deactivated before adding RDNode and RDTech components, RDTech must invoke Start() before activation, etc.
EDIT: nvm, that's not Setup.
Got something I can look at?

I'm mainly trying to follow the process flow in TM and compare that to the flow in ATC to see what may need to be done differently. I'll post what I have to Github tomorrow, I'm right about to hit the hay here.

Edit 1: Here's my random techyness, just because.

http://imgur.com/bjfPSlC

Edited October 28, 2014 by Ixonal

anonish · October 28, 2014

Well, here's a couple observations on the process differences compared to what is in gitHub at the moment.

nodePrefab = new GameObject("newnode", typeof(RDNode), typeof(RDTech));

That is right out. It will create an *active* GameObject and the RDNode and RDTech unity script init functions Awake and Start will get called. But those functions expect the correct values to already be set.

You have to new GameObject("newnode"), then inactivate it, then AddComponent<RDNode> and AddComponent<RDTech>. Then set at least the prefab field. Now you have a working mostly blank GameObject to instantiate off of for your new nodes and you are free to setup those classes before calling setActive(true) on the instantiated objects. Also, because the RDNode init relies on RDTech being initialized first (a ksp bug, imo), you have to call the RDTech's start function before activating the GameObjects.

Again, the classes are *clean* if you follow this procedure. You don't have to worry about what is hiding behind a half used object. All that is set is what you set, and you get to set it before unity inits the object. This is a major departure between TechManager and TreeLoader and is a big part of why my code is short and to the point.

return Array.Find<RDNode> (AssetBase.RnDTechTree.GetTreeNodes(), x => x.gameObject.name == "node0_start");

This never worked for me. Had to switch to

stockNodes = GameObject.FindObjectsOfType<RDNode>();

RDNode startNode = Array.Find<RDNode>(stockNodes, x => x.gameObject.name == "node0_start");

KSP wipes and re instantiates the nodes each time you enter the RD building, so this has to be redone each time as well. I don't remember if I made that change before or after figuring out the clean node creation procedure, it might not be necessary. The start node (or any stock node) is really only needed for the prefab and transform fields. If those have the same value both ways, then either way would be fine.

Consider extending TechManager to derive the relevant properties from your ConfigNode system rather than extending ATC to create new nodes. I know I had a lot of 'almost works' moments until everything fell in place.

SirJodelstein · October 28, 2014

Hey everyone,

this is just a short notice to let you all know that I am still very interested in this project, and that i am following the current developments regarding the new nodes from afar. Unfortunately, real life constraints prevent me from doing more (haven't played KSP for over a month now). Anonish, Ixonal, i am really impressed by your continued efforts and wish you all the best. You rock! I'll gladly assist with integration/branch merging when you are ready.

Ixonal · October 28, 2014

Consider extending TechManager to derive the relevant properties from your ConfigNode system rather than extending ATC to create new nodes. I know I had a lot of 'almost works' moments until everything fell in place.

Not really my call, as it's not my project. I'm just helping figure this part out.

Got rid of the exception, but it's still not acting right. The node won't show up the first time I load the research center, but it will the second time.

Also, if I use GameObject.FindObjectsOfType<RDNode>(); to find nodes, my new node doesn't show up at all, but it does if I use AssetBase.RnDTechTree.GetTreeNodes()

Edit: There's also this weird issue that has to do with zooming. The node and half the arrow to it start hidden, but then become visible as I zoom out...

Edit 2: Also, seen this error before?



ATC: Error Loading tree - System.NullReferenceException: 

  at (wrapper managed-to-native) UnityEngine.Component:InternalGetTransform ()

  at UnityEngine.Component.get_transform () [0x00000] in <filename unknown>:0 

  at ATC.TechChanger.setupAnchors (.RDNode target, .Parent& connection) [0x00000] in <filename unknown>:0 

  at ATC.TechChanger.LoadTree () [0x00000] in <filename unknown>:0 

  at ATC.TechChanger.Update () [0x00000] in <filename unknown>:0

Edited October 28, 2014 by Ixonal

anonish · October 29, 2014

The two calls to render properly i've seen, the zooming and exception i have not. But those all sound like what i would expect when starting with dirty GameObjects. Did you switch to the clean instantiation i suggested?

Ixonal · October 29, 2014

The two calls to render properly i've seen, the zooming and exception i have not. But those all sound like what i would expect when starting with dirty GameObjects. Did you switch to the clean instantiation i suggested?

I updated my repo, here's what I'm currently using to spawn new tech nodes (I separated it from the main class) https://github.com/Ixonal/KSPAlternativeTechConfigurator/blob/master/RDNodeFactory.cs

Ixonal · October 30, 2014

hmmmm.... anonish, why do you deactivate all of the stock nodes when prepping for creation?

anonish · October 30, 2014

hmmmm.... anonish, why do you deactivate all of the stock nodes when prepping for creation?

I like clean breaks. If you are going to create one node, might as well make them all. Keeps the logic simpler as I don't have to determine what is new and what is a rewrite. Also I don't have to implement a REMOVENODE cfg entry type if someone *doesn't* want a stock node -- in that case, they just don't put it in the file.

Ixonal · October 30, 2014

I like clean breaks. If you are going to create one node, might as well make them all. Keeps the logic simpler as I don't have to determine what is new and what is a rewrite. Also I don't have to implement a REMOVENODE cfg entry type if someone *doesn't* want a stock node -- in that case, they just don't put it in the file.

ahh, well spoot... I'm having a hell of a time finding out why the new nodes don't load the first time, but they do subsequent times. Thought that might have something to do with it.

Edit 1: seems like that null reference exception is stemming from obtaining the new node's parent node using "RDNode source = connection.parent.node;" and then trying to reference the transform. If I try to access an RDNode with the same name as the node it's looking for, it seems to be able to access the transform, but the anchors go screwy. This seems to suggest there are two versions of the new node somehow, even though the creation process has only run once... ehh, I'll keep lookin at it this weekend I guess...

Edited October 31, 2014 by Ixonal
Edit 1

G2000 · October 31, 2014

This mod/plugin looks very cool, and I downloaded it, but I was just wondering. How the hell does it work . I'm a bit of an idiot when it comes to working out what I'm actually looking at and meant to be doing. I would really like to help except I only just started learning code so I don't really think I can but if you need any testing or the suchlike, I'd be happy to help as I'm using this for my "Modpack" of sorts. Hope that this thing comes together soon. Oh, and that GUI looks really cool! I'd be happy to alpha test that too!

Dr. Jet · October 31, 2014

Any news about progress in implementing new_node function? I want to make something like this.

Ixonal · November 1, 2014

Any news about progress in implementing new_node function? I want to make something like this.

virtually all my posts in the past several pages have been about my exploits in trying to implement it... I'm so close, yet so far.

Furious1964 · November 3, 2014

How do you get this to work? There's no GUI for me.

Tested this on a fresh install and it just doesn't work for me, can't change the tree or even get any GUI. What am I doing wrong?

Edited November 3, 2014 by Furious1964

Ixonal · November 4, 2014

How do you get this to work? There's no GUI for me.
Tested this on a fresh install and it just doesn't work for me, can't change the tree or even get any GUI. What am I doing wrong?

as in the RnD tree isn't showing up, or what?

Furious1964 · November 4, 2014

as in the RnD tree isn't showing up, or what?

Yes, not showing up. The tree shows only the tech I unlocked.

Ixonal · November 4, 2014

Yes, not showing up. The tree shows only the tech I unlocked.

Out of curiosity, if you leave the research department and come back, does the problem persist?

Furious1964 · November 4, 2014

Out of curiosity, if you leave the research department and come back, does the problem persist?

Yes, even started with a fresh install with only this mod.

Ixonal · November 5, 2014

Yes, even started with a fresh install with only this mod.

That sounds like a bug I'm getting with my fork (though, only with new nodes, as far as I've seen). As far as I know of, nobody else using the official release version is having this issue. At least, I haven't seen it reported. Did you build the mod from source or just download the zip?

Furious1964 · November 5, 2014

That sounds like a bug I'm getting with my fork (though, only with new nodes, as far as I've seen). As far as I know of, nobody else using the official release version is having this issue. At least, I haven't seen it reported. Did you build the mod from source or just download the zip?

Downloaded from ZIP.

Ixonal · November 5, 2014

Downloaded from ZIP.

that's.... weird... got an output log for it?

Furious1964 · November 6, 2014

that's.... weird... got an output log for it?

Here you go. As I said, I have to run the game in OpenGL because of all the mods I run. Will be installing Linux this weekend.

http://www./view/4hfxnwdvsgh9f9n/output_log.txt

Ixonal · November 6, 2014

Here you go. As I said, I have to run the game in OpenGL because of all the mods I run. Will be installing Linux this weekend.
http://www./view/4hfxnwdvsgh9f9n/output_log.txt

I don't suppose you have one from an install with only ATC in it? not sure which messages are from ATC and which are from other mods. Also, do you have a copy of the cfg file that's being used to alter the tech tree?

[0.25] ATC - Alternative Tree Configurator [15.10. - V0.5.1]

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation