Materials and Goo after 1.4.1
irineus opened this issue · 7 comments
Hello.
It seems there's a problem with "SC-9001 Science Jr" and "Mystery Goo™ Containment Unit" science readings after 1.4.1 (at 1.4 it got the same problem). You need to go EVA and reset it manually in order to grab the science.
For other parts, seems works fine in my tests:
- Double-C Seismic Accelerometer
- PresMat Barometer
- GRAVMAX Negative Gravioli Detector
- 2HOT Thermometer
Didn't made extensive tests on "Atmospheric Fluid Spectro-Variometer", but it seems to work in a couple of flight I've made.
Try the cwizard branch. I don't think this branch is being maintained regularly. If the issue persists with my branch let me know there and Ill look into it.
Note that the goo and materials science is only collected if there is a scientist on board.
The mod is still being maintained, as it has been for years.
It it usually a mod conflict or installation error that causes an issue. So I do not usually patch unless I get more than one report.
So I've made some tests to see if there's any mod conflict. Report and prints below (sorry for the pt-br language in the prints).
-
Started a new career mode, edited funds and science to upgrade and unlock all science tree, and made a small science rover with all science equip available and with Bob as crew:
-
Moved the rover outside the runway and at the biome change, got science for (in order):
- Crew report
- Atmospheric Fluid Spectro-Variometer
- 2HOT Thermometer
- GRAVMAX Negative Gravioli Detector
- Double-C Seismic Accelerometer
- PresMat Barometer
Got no science for:
- SC-9001 Science Jr
- Mystery Goo™ Containment Unit
- The only way to grab science for "Mystery Goo™ Containment Unit" is manually trigger it and, of course, manually restore it.
So, it seems, as stated at my first report, the mod works for every science experiment available, but "SC-9001 Science Jr" and "Mystery Goo™ Containment Unit".
If you need any other help with some tests, just let me know.
@irineus I attempted to duplicate your bug using KSP 1.4.2.2110 (WindowsPlayer x64) en-us and ForScience 1.5.0, but was unable to do so. Using your same procedure, all 8 experiments were triggered upon entering the various biomes around KSP.
I tried several attempts, but the only variation was that occasionally upon reverting to launch, the mod would trigger collecting experiments on the runway itself, while it would sometimes require me to leave the runway and return to collect runway experiments.
Given that the experiments you're having issues with are those that require a Scientist on board and that the only difference between our versions is the language, I'm lead to believe that this may be a localization issue. In particular, though I can't find confirmation in the API docs, I think that the issue may be that at line 257, if (kerbal.experienceTrait.Title == "Scientist") return true;
, the Title being returned is now localized to "Cientista", and therefore the mod does not believe a scientist is on board. During my googling, I found but lost a similar sounding issue on a different mod that appeared to be this same issue.
Again, the docs aren't super helpful, but checking other mods, it looks like the localization-independent way to do that is either:
if (kerbal.protoCrewMember.experienceTrait.Config.Name == "Scientist") return true;
or
if (kerbal.protoCrewMember.experienceTrait.TypeName == "Scientist") return true;
I'm not setup to compile and test at the moment and I'm loathe to do a dry-coded PR, but hopefully @confusingbits can take that and poke at it a bit to confirm/deny my suspicions.
@dtparr you nailed it.
Made the same test with en-us as the language and it worked perfectly.
If @confusingbits could change the lines of code as @dtparr pointed, I can test the DEV version to confirm it.
Very nice report @dtparr , thank you.
I will see about fixing this as soon as I can.