duzenko Posted December 3, 2017 Report Share Posted December 3, 2017 Does anyone use the integrated idRenderWindow editor tool? This is the only place that seems to use additional renderwordls in TDM.If unused I would like to remove it which will allow to remove quite a number of linked pointer operations in front renderer. Quote Link to comment Share on other sites More sharing options...
stgatilov Posted December 3, 2017 Report Share Posted December 3, 2017 I think it is only used by in-game GUI editor.You can type "editGUIs" in console to launch it.It does not seem to work though. Quote Link to comment Share on other sites More sharing options...
grayman Posted December 3, 2017 Report Share Posted December 3, 2017 Is it a single block of code, or is it intermingled with a bunch of other useful stuff? If the former, I'd say to comment it out, in case someone not paying attention today wonders why he can't use it (or fix it) a month from now. Quote Link to comment Share on other sites More sharing options...
duzenko Posted December 3, 2017 Author Report Share Posted December 3, 2017 Yes, it's only the GUI editor, but I want to know if it is used by anyone? I know a number of tools is used, but that particular thing? Basically it's one editor control only, called idRenderWindow, but is it in use at all? Quote Link to comment Share on other sites More sharing options...
Springheel Posted December 3, 2017 Report Share Posted December 3, 2017 You're getting a small sample of people in the team forums. Quote TDM Missions: A Score to Settle * A Reputation to Uphold * A New Job * A Matter of Hours Video Series: Springheel's Modules * Speedbuild Challenge * New Mappers Workshop * Building Traps Link to comment Share on other sites More sharing options...
duzenko Posted December 3, 2017 Author Report Share Posted December 3, 2017 You're getting a small sample of people in the team forums.Could you move this thread to a more appropriate forum? Quote Link to comment Share on other sites More sharing options...
freyk Posted December 3, 2017 Report Share Posted December 3, 2017 (edited) I would like to use it, but the gui editor crashes from start.So, i create guis using other the gui editor from other idtech guis. Or do you point at renderDef? Edited December 3, 2017 by freyk Quote Info: My portfolio and darkmod graphical installer Amnesty for Bikerdude! Link to comment Share on other sites More sharing options...
stumpy Posted December 3, 2017 Report Share Posted December 3, 2017 (edited) "idRenderWindow" its used by the gui system to render 3d animations "renderDef A window that displays a rendered 3d scene" the built in gui editor is in the tools directory sub directory guied, if that isn't added to the game compile then editGUI wont work in the games command window. looking around the internet and one site says that idRenderWindow was used to show the 3D planet in the main menu of doom 3 or the gui editor never made it into the source as it was owned by raven software. Edited December 3, 2017 by stumpy Quote Link to comment Share on other sites More sharing options...
nbohr1more Posted December 4, 2017 Report Share Posted December 4, 2017 "idRenderWindow" its used by the gui system to render 3d animations "renderDef A window that displays a rendered 3d scene" the built in gui editor is in the tools directory sub directory guied, if that isn't added to the game compile then editGUI wont work in the games command window. looking around the internet and one site says that idRenderWindow was used to show the 3D planet in the main menu of doom 3 http://bugs.thedarkmod.com/view.php?id=4569 In v1.08 we removed "GUI renderDef lights" which were designed for this 3D menu behavior so this additional change would probably completely kill the option to have 3D GUI's. Since the Compass was affected by this change, we would probably need to redesign the compass if we removed idRenderWindow? Quote Please visit TDM's IndieDB site and help promote the mod: http://www.indiedb.com/mods/the-dark-mod (Yeah, shameless promotion... but traffic is traffic folks...) Link to comment Share on other sites More sharing options...
freyk Posted December 4, 2017 Report Share Posted December 4, 2017 (edited) Please dont remove this function, because others (like me) need this for design.For example the compass (as posted earlier), showing other 3d loot in the hud (like thief 1), putting a moving person in the mainmenu, etc. -I dont understand why people delete code of existing/working objects from the codebase.Please stop! (And repair/improve the object.)For example, i don't understand why the d3 PDA-system is removed.This could also been (optional) disabled/reskinned/re-imagned.(like it could be named garrets notebook) Edited December 4, 2017 by freyk 1 Quote Info: My portfolio and darkmod graphical installer Amnesty for Bikerdude! Link to comment Share on other sites More sharing options...
Obsttorte Posted December 4, 2017 Report Share Posted December 4, 2017 I dont understand why people delete code of existing/working objects from the codebase.Please stop! (And repair/improve the object.) +1 Quote FM's: Builder Roads, Old Habits, Old Habits Rebuild Mapping and Scripting: Apples and Peaches Sculptris Models and Tutorials: Obsttortes Models My wiki articles: Obstipedia Texture Blending in DR: DR ASE Blend Exporter Link to comment Share on other sites More sharing options...
Destined Posted December 4, 2017 Report Share Posted December 4, 2017 Well, I imagine that the code gets confusing after a while, if you just add stuff and comment out all the stuff you don't need anymore. Not sure how/if this affects speed, but I can imagine that working through more lines of code will slow down a program (from a laymans point of view this would make sense). And if I remember correctly someone here (most likely grayman) wrote some time back, that unnecessary code is usually commented out for some time and if it is not missed for a couple of years or so, it gets deleted. Quote Link to comment Share on other sites More sharing options...
peter_spy Posted December 4, 2017 Report Share Posted December 4, 2017 It makes sense, as you can't forsee every use and evey fork mod that might or might not be in the works. Noone used 3d menu gui or PDA system in TDM missions for so many years, so it got deleted. You can always refer to original Doom 3 code to reimplement this. Quote Misc. assets for TDM | Artstation stuff Link to comment Share on other sites More sharing options...
nbohr1more Posted December 4, 2017 Report Share Posted December 4, 2017 There's at least a few rationales for code removal, off the top o' my head: 1) Less features to test\fix when changing code infrastructure 2) Less dependencies to keep track of 3) More legible code, less code to review 4) Less resources to load 5) Shorter compile times That said, if this breaks 3D GUI's then I'd say it's probably unwise. We now have a GUI resizing option in 2.06 and I think many would like to see that extended to have a 3D scene where both GUI and Graphic options can be previewed. A 3D GUI would be perfect for that. Of course, I'm basing all this concern over 3D GUI on Stumpy's post. I haven't really validated whether 3D GUI really requires multiple render worlds (etc). 3 Quote Please visit TDM's IndieDB site and help promote the mod: http://www.indiedb.com/mods/the-dark-mod (Yeah, shameless promotion... but traffic is traffic folks...) Link to comment Share on other sites More sharing options...
freyk Posted December 4, 2017 Report Share Posted December 4, 2017 (edited) To me and newcomers, its is all about functionality on the frontend. We cannot read code. So understanable code at the backend is not importent to us. (but maybe performance) How about bringing back the pda menu, keep the 3d gui elements. And keep them as a feature. Edited December 4, 2017 by freyk Quote Info: My portfolio and darkmod graphical installer Amnesty for Bikerdude! Link to comment Share on other sites More sharing options...
peter_spy Posted December 4, 2017 Report Share Posted December 4, 2017 You want to make a total conversion for TDM without coding? Good luck with that... Quote Misc. assets for TDM | Artstation stuff Link to comment Share on other sites More sharing options...
HMart Posted December 4, 2017 Report Share Posted December 4, 2017 (edited) There's at least a few rationales for code removal, off the top o' my head: 1) Less features to test\fix when changing code infrastructure2) Less dependencies to keep track of3) More legible code, less code to review4) Less resources to load5) Shorter compile times That said, if this breaks 3D GUI's then I'd say it's probably unwise. We now have a GUI resizing optionin 2.06 and I think many would like to see that extended to have a 3D scene where both GUI and Graphic optionscan be previewed. A 3D GUI would be perfect for that. Of course, I'm basing all this concern over 3D GUI on Stumpy's post. I haven't really validated whether 3D GUIreally requires multiple render worlds (etc). Even tho i totally agree with that, for example i'm slowly removing the multiplayer code from my version of fhDoom, i personally don't like multiplayer games and i'm working on a single player game so no need to maintain that part of the code. But on 3D GUI's, IMO you guys didn't weighted well the benefits or not on maintaining or removing the functionality, IMO the problem with no one using it for TDM missions was, thief add no such thing so people looking for the thief look, don't turn to 3D GUI's, lack of a example and specially good tutorials, almost all GUI tutorials for idtech 4 are 2D based, what is unfortunate. IMO that ability is fantastic to have, with it you can do some really cool stuff, like being able to interact with 3D objects in the GUI's, rotating them or zooming in to see hidden messages on them or something like that, etc. Edited December 4, 2017 by HMart 4 Quote Link to comment Share on other sites More sharing options...
Oktokolo Posted December 4, 2017 Report Share Posted December 4, 2017 You want to make a total conversion for TDM without coding? Good luck with that...I would expect it to be possible to do a total conversion to another setting without touching the C code. It certainly would need its amount of scripting though. And, of course, the target game would be single player and feature mostly the same mechanics as TDM. Quote Link to comment Share on other sites More sharing options...
stumpy Posted December 4, 2017 Report Share Posted December 4, 2017 idRenderWindow is used by the gui editor, and by the game interface eg menu, players hud, gui readables, so the best method would be to remove the include idRenderWindow path from the gui editor then it would save loads of calls, as the gui editor doesn't work anyway and is called via some code in radiant. unless you were thinking about idRenderWorld, and not window, but I think that renders the game world and if you remove that then the game wouldn't work. Quote Link to comment Share on other sites More sharing options...
freyk Posted December 9, 2017 Report Share Posted December 9, 2017 (edited) Just dont do it.Improve the code to make it more functional.And make the gui editor work again.And for the future,please give the community the choice, when removing important code from the base. (which you did in this topic) (putting back the "mail" system and changing multiplayer to co-op, would be nice features of TDM)You want to make a total conversion for TDM without coding? Good luck with that...No, its just a wrong choice of set of words,to give coders a example that average TDM designer/player dont have the knowledge/time/permissions to put removed code back.(I have put down code for several TDM applications, like the tdm-launcher, thief 2 hash checker, app for android and scripted some installers.) Edited December 9, 2017 by freyk 1 Quote Info: My portfolio and darkmod graphical installer Amnesty for Bikerdude! Link to comment Share on other sites More sharing options...
stgatilov Posted December 9, 2017 Report Share Posted December 9, 2017 I agree with freyk.Removing features and proper architecture for the sake of some tiny optimizations (well, do they give +20% FPS?) is a simply the wrong way to go.If you do microoptimizations, please be sure that they don't break anything. Quote Link to comment Share on other sites More sharing options...
duzenko Posted December 9, 2017 Author Report Share Posted December 9, 2017 I agree with freyk.Removing features and proper architecture for the sake of some tiny optimizations (well, do they give +20% FPS?) is a simply the wrong way to go.If you do microoptimizations, please be sure that they don't break anything.On that subjectDoes anyone here even know how to make use of multiple render worlds in TDM?It's one thing when it is unused, but another when nobody even knows how to use it. Quote Link to comment Share on other sites More sharing options...
freyk Posted December 10, 2017 Report Share Posted December 10, 2017 sure, put some "renderdef" in your gui code. 1 Quote Info: My portfolio and darkmod graphical installer Amnesty for Bikerdude! Link to comment Share on other sites More sharing options...
stumpy Posted December 10, 2017 Report Share Posted December 10, 2017 its used by the players in game compass //// Compass HUD, original version code by Drumple, modifications by greebo//windowDef Desktop { rect 500, 290, 140, 180 backcolor 0 ,0 ,0 ,0 visible "gui::CompassVisible" nocursor 1 renderDef compassModel { rect 0, 0, 140, 180 visible 1 backcolor 0, 0, 0, 0 model "models/darkmod/player_equipment/compass.ase" needsRender 1 modelRotate ("gui::modelPitch"), ("gui::modelYaw"), ("gui::modelRoll"), 1 modelOrigin 0, 0, 0, 1 viewOffset -10, 0, 6, 1 lightOrigin -50, 0, 50, 0 lightColor 1, 1, 1, 1 }} // Desktop 1 Quote Link to comment Share on other sites More sharing options...
duzenko Posted December 10, 2017 Author Report Share Posted December 10, 2017 its used by the players in game compass Lesson learned Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.