Jump to content

[1.12.x] IndicatorLights Community Extensions v1.6.2: Compatibility patches for IndicatorLights


Snark

Recommended Posts

@UnanimousCoward, @Snark, there's a small graphics bug for the ReStock version of the RA-100, in ILCE v1.6.  I believe it's leftover from a previously reported error in the ReStock GitHub Issues page here:

https://github.com/PorktoberRevolution/ReStocked/issues/608

I'm seeing the same glitch: the light is hanging in mid-air, above the antenna.

EDIT: It looks like both the 1.25m and compact antenna part switch variants of the light may be getting loaded, regardless of the variant selected OR the smaller tip light is just a legacy light that needs to be blocked off from loading.  The relatively large light on the compact variant seems to be positioned correctly, but the tip light on the 1.25m variant gets placed about a meter high.  I think the compact variant light is partially sticking out the bottom of the antenna tip (looks like a ring) on the 1.25m variant, which may have been by design, and supports my theory that the hanging light wasn't supposed to load.

I'm using KSP 1.7.3 x64 on Win10.  Tried on a clean install with just ReStock v0.1.4, IL v1.5, ILCE v1.6, and MM v4.0.2.

Edited by KSPrynk
Link to comment
Share on other sites

  • 2 months later...

I noticed that the Indicator Lights patch for ReStock was changing the scale of the small radial battery.  As nice as it is to be able to fit loads of them on a small probe, I don't think it's the intent of this mod to change the underlying models.  I've submitted a Pull request on GitHub to patch this, hope you find it useful!

https://github.com/KSPSnark/IndicatorLightsCommunityExtensions/pull/4

Link to comment
Share on other sites

Hey @Snark, I was checking several mods yesterday if I missed some important one that would fit my game play and came along the Curvus mod (another 2 Kerbal pod), seems it doesn't have indicator lights for it, and I couldn't find a config in the community extensions either. Do you by chance know if there's already a config for it? And if not: would it be helpful if I create one? The mod is still maintained, afaik, should I contact the mod author and ask if he's fine with me creating a config for IL? Just asking to avoid any potential hazard (I came along another thread with kind of heated discussion regarding modding, you know that particular thread, I want to avoid such "disturbances", naturally) :) 

Edited by VoidSquid
Link to comment
Share on other sites

  On 10/21/2019 at 4:41 AM, VoidSquid said:

Curvus mod (another 2 Kerbal pod), seems it doesn't have indicator lights for it, and I couldn't find a config in the community extensions either. Do you by chance know if there's already a config for it?

Expand  

There's certainly not a config for it in this mod (yet).

FYI, there's a list of supported mods in the OP of this thread, which I make a point of keeping up to date. So if you're ever wondering whether a particular mod is supported or not, you can just check that list.

Anyway, that's the answer for this mod. I have no way of knowing whether someone may have created such config elsewhere, of course, but I'm guessing not.

  On 10/21/2019 at 4:41 AM, VoidSquid said:

And if not: would it be helpful if I create one?

Expand  

Absolutely! The more, the merrier. If you make a config for it, please provide it to me so I can incorporate it here.

  On 10/21/2019 at 4:41 AM, VoidSquid said:

The mod is still maintained, afaik, should I contact the mod author and ask if he's fine with me creating a config for IL?

Expand  

No, that shouldn't be necessary. It's just a 3rd party compatibility patch and therefore shouldn't discommode the mod author in any way.

  On 10/21/2019 at 4:41 AM, VoidSquid said:

Just asking to avoid any potential hazard (I came along another thread with kind of heated discussion regarding modding, you know that particular thread, I want to avoid such "disturbances", naturally) :)

Expand  

Yes, and kudos for your caution. ;)

However, that topic regards people forking and releasing the mod itself (which does affect the modder, which is the source of the kerfuffle). You're not doing that; you're just talking about producing a 3rd party thing that happens to work with it, which doesn't interfere with it in any way and is released separately. Therefore, not a concern in this case.

Link to comment
Share on other sites

@Snark

This should do it, I think, but please, check for proper syntax:

  Reveal hidden contents

Screenshot:

screenshot0-png-Windows-Photo-Viewer-201

 

Question: despite having all science from the crew report at the launchpad, a freshly launched pod still has the lights blinking as if there'd be some science to get. After I do a crew report (collecting 0.0 science), the lights act normal. Any idea what's going on here?

Edited by VoidSquid
Link to comment
Share on other sites

  On 10/21/2019 at 5:10 PM, VoidSquid said:

This should do it, I think, but please, check for proper syntax:

Expand  

Looks pretty good, but can you please include, 1. a link to the release thread for that mod, and 2. a "NEEDS" directive so that this config won't be active until/unless the relevant mod is installed?

Link to comment
Share on other sites

Stupid me, sorry, forgot, here you go, ok now?

  Reveal hidden contents

 

Link to comment
Share on other sites


 

@VoidSquidHave several mods that are screaming for the IndicatorLights treatment

MoreHitchhikers for example - and many others. Are you game?

also have several pods coming out shortly as well (Yarbrough08, DuoPods, LanderTek, etc).

Then there is MoreCupolas, and MoreGreenhouses... :P

Edited by zer0Kerbal
won't let me enter anything after saying Voidsquid three times....
Link to comment
Share on other sites

Pretty sure you're overestimating my abilities, I've never done any proper programming (just some scripting), I can't model, never used Blender, I'm not very familiar with MM syntax, and so on.

That said, sure, if I can help, I'll happily do so, provided I can find time for it. Deal?

Edited by VoidSquid
Link to comment
Share on other sites

  On 7/7/2019 at 5:53 PM, Beetlecat said:

Alright then.

After a few sessions fiddling with Blender and getting used to MM syntax issues, I've managed to throw together a quick set of patches for the Kerbonov (continued) mod. This covers adding crew indicators to the Kn-2 and Kn-7 cockpits, the 1-person passenger can, and a transmitting indicator to the VHF antenna. I basically copied patches for other similar components, and changed part names and position values. That's how we do it, right? ;)

The Kerbonov antenna has *two* similar lighted parts on the model, so I suppose a second indicator light could be added to perform a different function, but I didn't pursue that *yet*.

https://pastebin.com/V332R5tq

  Reveal hidden contents

The biggest bit of learning curve was Blender itself, but once I got used to the controls, it was pretty simple to navigate around. I can imagine that trying this in the versions before the 2.8 beta would have been far more "interesting."

Also, much respect for @UnanimousCoward for cranking through this process for so many of the the restock(+) parts.

Expand  

 

Mini *ping* for @Snark -- the above configs for the Kn-2 pods are effectively complete.

 

Link to comment
Share on other sites

  On 10/21/2019 at 8:25 PM, VoidSquid said:

Pretty sure you're overestimating my abilities, I've never done any proper programming (just some scripting), I can't model, never used Blender, I'm not very familiar with MM syntax, and so on.

That said, sure, if I can help, I'll happily do so, provided I can find time for it. Deal?

Expand  

All good and no programing needed. Just someone with the patience to stick a few lights on here and there. :D

Link to comment
Share on other sites

Hey @zer0Kerbal, if you can give me a raw MM script that *somehow* (your job, hehe) romoves the cloned ILs, I'll be happy to adjust the individual IL indicators (my part), ok?

The two pods I made IL configs for were stand-alone parts, not cloned/copied, much easier for someone (me) who's not proficient with MM syntax, I just spent some time to position the ILs properly. This beast aka More Hitchhikers starts with

+PART[crewCabin]:NEEDS[KGEx/MoreHitchhikers]:FOR[MoreHitchhikers]

read: cloning, and I seriously don't know how to proceed with my part.

Link to comment
Share on other sites

  On 10/25/2019 at 3:46 PM, VoidSquid said:

Hey @zer0Kerbal, if you can give me a raw MM script that *somehow* (your job, hehe) romoves the cloned ILs, I'll be happy to adjust the individual IL indicators (my part), ok?

The two pods I made IL configs for were stand-alone parts, not cloned/copied, much easier for someone (me) who's not proficient with MM syntax, I just spent some time to position the ILs properly. This beast aka More Hitchhikers starts with

+PART[crewCabin]:NEEDS[KGEx/MoreHitchhikers]:FOR[MoreHitchhikers]

read: cloning, and I seriously don't know how to proceed with my part.

Expand  

PPD-05-Indicator-Lights.png

 

seems I need to change how several models are scaled, and release an update for MHH first., however:
this works (once I update the base model).

 

  Reveal hidden contents

 

Link to comment
Share on other sites

  • 2 weeks later...

Looks like when installing the latest Restock mod, having this also installed causes the Science Experiment container geometry to vanish. The indicator lights are still there, floating in space. Removing either Restock or this mod causes it to to return. Having both installed causes it to bug out. IndicatorLights itself, is fine.

Link to comment
Share on other sites

  On 11/9/2019 at 7:37 PM, AmpCat said:

Looks like when installing the latest Restock mod, having this also installed causes the Science Experiment container geometry to vanish. The indicator lights are still there, floating in space. Removing either Restock or this mod causes it to to return. Having both installed causes it to bug out. IndicatorLights itself, is fine.

Expand  

Was just coming here to say this...

 

@Beetlecat found the issue here: 

 

  Quote

 

Yup -- ILCE has a typo when defining the storage unit; it's missing a "-1" at the end of the name. It should look like here in IndicatorLights_Restock.cfg (at line #857)

//Science Box
@PART[ScienceBox]:AFTER[ReStock]:NEEDS[IndicatorLights]
{
	-MODEL,* {}
	MODEL
	{
		model = ReStock/Assets/Science/restock-sciencebox-radial-1
	}


 

Expand  

I've tested this and it works

Edited by Bombaatu
Link to comment
Share on other sites

  • 3 months later...

@Beetlecat thanks for the fix on the Science box.  Just installed this mod in my save and that is exactly what I needed.  

I think there may be another bug, In my save all the z-100 batteries were shrunk down in size by about half.  They are about the size of a pressure sensor now.  I am  looking through the code, but haven't found the error yet.

 

 

Link to comment
Share on other sites

  On 2/14/2020 at 6:34 PM, gunt3rgam3r said:

@Beetlecat thanks for the fix on the Science box.  Just installed this mod in my save and that is exactly what I needed.  

I think there may be another bug, In my save all the z-100 batteries were shrunk down in size by about half.  They are about the size of a pressure sensor now.  I am  looking through the code, but haven't found the error yet.

 

 

Expand  

wow! Thanks for that hint/clue. I was playing through some Unkerbaled Start early missions, and noticed that the batteries were tiny and able to fit into a sounding rocket-sized bay no problem -- not realizing this was totally in error.

Just to check -- are you also using Restock?

Edited by Beetlecat
Link to comment
Share on other sites

  On 2/14/2020 at 8:59 PM, gunt3rgam3r said:

Yep using restock, but not using restock+.  Indicator lights and the patches mod are the only thing that chnged the error is most likely in the community patches mod.

Expand  

There's an odd chance of that, for sure. and possibly a weird interaction w/tweakscale. I usually have that installed, too.

Link to comment
Share on other sites

  On 2/14/2020 at 9:14 PM, Beetlecat said:

There's an odd chance of that, for sure. and possibly a weird interaction w/tweakscale. I usually have that installed, too.

Expand  

I don't have tweakscale installed in my save.

Found a fix:

  Reveal hidden contents

If you reset the scale to 1,1,1 in the ReStock.cfg file for the community patches the battery size is now correct. 

Link to comment
Share on other sites

This thread is quite old. Please consider starting a new thread rather than reviving this one.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...