Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

spontaneous cloaking device #59

Open
yowshi opened this issue Feb 17, 2025 · 3 comments
Open

spontaneous cloaking device #59

yowshi opened this issue Feb 17, 2025 · 3 comments

Comments

@yowshi
Copy link

yowshi commented Feb 17, 2025

i've seen it on my llama.begin and now my plowshare where i spontaneously get a cloaking device.

@royfalk
Copy link
Contributor

royfalk commented Feb 17, 2025

Please specify the following:

  • branches
  • ship model
  • screenshot if possible
  • json of ship from save game if possible

@yowshi
Copy link
Author

yowshi commented Feb 19, 2025

Image
this particular one is running sgtugg's Vega-Strike-Engine-Source-fix-FTL_0.9.x, assuming this is an engine issue and not an asset issue.
assets are latest deb and i have seen this on the the 0.9.0beta2 version of the engine.
i dont know what actually causes it i think i missed it's initial happening and just kind of noticed it during one of my play sessions.

Llama.begin.csv

Plowshare.stock.csv

@royfalk
Copy link
Contributor

royfalk commented Feb 20, 2025

If I had to guess, cloak.cpp

   if(UnitCSVFactory::GetVariable(unit_key, "Can_Cloak", false)) {
        status = CloakingStatus::ready;
    } else {
        status = CloakingStatus::disabled;
    }

is where things are going wrong. I don't see the issue but the old assets used 0, false, FALSE interchangeably. It's possible one of these if read incorrectly as true. It's almost certain.

Either fix the GetVariable for boolean to support all or fix the assets.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants