Skip to content

Fixes for KSP 1.5.0#29

Open
Flupster wants to merge 36 commits intothewebbooth:masterfrom
Flupster:master
Open

Fixes for KSP 1.5.0#29
Flupster wants to merge 36 commits intothewebbooth:masterfrom
Flupster:master

Conversation

@Flupster
Copy link

No description provided.

@MateoCB
Copy link

MateoCB commented Oct 17, 2018

This is working nicely, thanks @Flupster

@CptBoBy
Copy link

CptBoBy commented Oct 18, 2018

compiled the mod with the modified files ,still doesn't work.

@Flupster
Copy link
Author

I'll upload the compiled files later tonight and link them here, compiled fine for me and worked

@DonCorleone-NL
Copy link

That would be awesome Flupster, thx alot mate!

@new01DAY
Copy link

new01DAY commented Oct 18, 2018

I'm not sure, but I think it doesn't work for some because of the changes to [x] Science!.csproj <HintPath>E:\SteamLibrary\steamapps\common\Kerbal Space Program\KSP_x64_Data\Managed\UnityEngine.dll</HintPath> from <HintPath>..\..\..\..\Games\KSP\KSP_Real\KSP_x64_Data\Managed\UnityEngine.dll</HintPath>.

@Flupster
Copy link
Author

You need to add the references to the DLLs,mine was in another location,however I reverted it back in the second commit

@new01DAY
Copy link

Yeah, sorry. I just realized why/what it was. I actually don't know much about compiling and I'm trying to figure it out.

@CptBoBy
Copy link

CptBoBy commented Oct 18, 2018

compiled without problems having set the proper references , but game still says incompatible and the x science window appears completely empty.

@Flupster
Copy link
Author

Flupster commented Oct 18, 2018

zip of the compiled mod can be found in my fork releases over at https://github.com/Flupster/KSP-X-Science/releases/

@Flupster
Copy link
Author

compiled without problems having set the proper references , but game still says incompatible and the x science window appears completely empty.

Just did a full reinstall of KSP dropped in the mod from my forks release page and works, i'm not sure why yours isn't working but you could try mine?

https://i.imgur.com/qSoSuF9.jpg

@CptBoBy
Copy link

CptBoBy commented Oct 18, 2018

compiled without problems having set the proper references , but game still says incompatible and the x science window appears completely empty.

Just did a full reinstall of KSP dropped in the mod from my forks release page and works, i'm not sure why yours isn't working but you could try mine?

https://i.imgur.com/qSoSuF9.jpg

tried with my old install , not even the empty science window appeared. completely reinstalled ksp ,and same thing ,at startup game says mod compatible only with 1.4.5 ,and game pretends mod is not even installed

@Flupster
Copy link
Author

Thanks. But your 5.17.2 release zip that includes this change is broken - only 22 bytes.

My bad forgot to actually include in the mod within the zip, should be fixed now!

@Oeddish
Copy link

Oeddish commented Oct 20, 2018

FWIW I came here to submit a PR with more or less your exact changes, which work for me (with minor differences I'll comment on in the diff). Would suggest also updating CHANGES.md to keep the project consistent.

{
// Set the music volume to what it was before
MusicLogic.SetVolume(oldVolume);
MusicLogic.SetVolume(oldVolume, oldVolume);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if it matters, but i tested this with 0f rather than oldVolume for the parameter. Seemed to work as expected.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have to be honest, I'm not entirely sure what the second parameter of SetVolume is, it just required a float so I used the first parameter as the second, If you know what the second parameter is let me know and if 0f is the better option for the second parameter

@Tekaoh2
Copy link

Tekaoh2 commented Oct 22, 2018

This works on my machine. +1 vote for merge so it can go on CKAN

@Flupster
Copy link
Author

Updated for v1.6.0 of KSP, no actual changes were made just the version numbers, tested a little all seems to be working.

@SilkDev
Copy link

SilkDev commented Jan 17, 2019

Hey @Flupster, it seems there's no maintainers left here, maybe you could push it to CKAN as a new package ?

Or maybe a little ping to @thewebbooth could work ?

@SilkDev SilkDev mentioned this pull request Feb 5, 2019
This was broken, requests coming from `%255Bx%255D%2520Science%2521/%255Bx%255D%2520Science%2521.version`
siimav and others added 26 commits May 12, 2019 00:04
Performance improvements & DMagic API fixes
Perhaps these science combinations were not possible at one time, but they are definitely possible now. I've achieved them using the Field Science contract pack to help me with the waypoints. But the contract pack is not required if you know where they are.
Comment out an outdated assumption
… science value in brackets. Improved performance.
In KSP 1.8.0, some parts throw a NullReferenceException when doing this.
Interestingly, the parts that throw this are the kerbalEVA* parts from
the DLCs when you don't have the DLCs.
Ignore null exception when gathering experiments
Science threshold feature for Here and Now window.
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

Successfully merging this pull request may close these issues.