-
Notifications
You must be signed in to change notification settings - Fork 279
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
1.2 Update #1436
Comments
Is anything needed of me for TF or do you guys got it? |
Every RO part that utilizes a TACLS converter will need to be updated to the new "TacGenericConverter" module. Example follows. Before:
After:
|
@jwvanderbeck I don't think we know right now. Probably not, but we'll now for sure once we've got RSS & RO rolling again :) |
I think all the important changes have been made in AJE and RF. @NathanKell I know you mentioned some improvements that could be made to the ullage code with KSP 1.2's resource flow changes, and I haven't looked at that. |
RT has a KSP 1.2.1-compatible version available: https://github.com/RemoteTechnologiesGroup/RemoteTech/releases/tag/RemoteTech-1.8.2 |
Checked the ones that are either released or (RF) ready to be. @jwvanderbeck as Strato says, sorry, dunno yet. :\ |
RT released 1.8.3 now which is 1.2.2 compatible: https://github.com/RemoteTechnologiesGroup/RemoteTech/releases/tag/1.8.3 |
@lamont-granquist thanks for checking RT. I personally don't play with it, so I wouldn't know how to test it. Have you personally used it in 1.2.2, ideally with RSS? |
@stratochief66 no, i'm not playing RSS in 1.2.2, shaving other yaks... |
RT works, if i can say, better in KSP 1.2 than in KSP 1.1.3. There are some major bug fixes (like the science transmittion bug) that have been fixed and they were backported to the 1.1.3 version. From the list:
The rest of the mods need a rework. |
I know it's pretty minor, but PParts really could be updated to get that annoying popup to not popup. There will be people who continually say 'it's not updated', when while it works the popup shows says it's not. I've submitted a PR for it, though not entirely sure that's all that needs to happen. |
RF and AJE can be checked off now |
@NathanKell So RealHeat is checked off...is this now done internally with KSP or is there an actual updated version floating around somewhere? |
Some minor notes: numbers should now be properly formatted (0.01 vs 0.010), and booleans start with a capital (True vs true) This was done in response to the comment made by @PhineasFreak here: KSP-RO#1436
pushed up a PR for TACLS, anything else that I can help with? |
For TestFlight: KSP-RO/TestFlight#154 |
https://github.com/KSP-RO/TestFlight/releases/tag/1.8.0.0.beta-1 Needs to be tested |
Looks like there's a pre-release for DRE: https://github.com/Starwaster/DeadlyReentry/releases/tag/v7.5.0 |
Note for future reference: there have been some changes to SmokeScreen to add a new particle system (shuriken). Any RealPlume configs that do some patching using a "final" pass (i.e. zzRealPlume) will need to be updated: Before:
After:
None of the above are required right now but when RealPlume implements this then it will. |
@PhineasFreak do you know if there was a functional reason to change the node name? Will plumes require tweaking in size, shape, speed etc along with the new particle effect? If so, that makes sense to distinguish between old and new configs. Is there any reason to not just run a script searching for PARICLE in plumes and changing it to SHURIKEN ? If that would work, @rsparkyc is good with scripts :) |
@stratochief66 i think that sarbian intends the new system to be a drop-in replacement without any required config changes. The script solution would be ideal. I think that Nhawks also did it that way for his RealPlume PR (or did a Find/Replace in Notepad++). |
More info on It is intended to be a drop-in replacement, the main benefit being less memory allocation while playing (i.e. gc alloc). Additional optimization can be done by reducing I have implemented this change in B9, and everything seems to be working as it did before. I think it's safe to implement in RF's RealPlume configs as well. |
Cool, I can run through those later and make the change. EDIT: Just looked at the memory usage graph. This looks much better, and should help if people are experiencing stuttering (which I was a lot) EDIT 2: ok, made a PR here: #1483 Note: this changes all instances of the text, and doesn't do any checks for |
CLS released a version that claims to support 1.2: https://github.com/codepoetpbowden/ConnectedLivingSpace/releases/tag/1.2.4.0 |
@diego-treitos Not until I or somebody else tests it in game (with an RO-ish modded install) thoroughly and reports back. :) |
Trying to test stuff, I see realheat is marked as done, but it looks like the latest version is still for 1.1.3 (https://github.com/KSP-RO/RealHeat/releases/tag/v4.3). Is there a 1.2.2 recompile somewhere, I should I do it myself? |
@rsparkyc i have pushed an update for RealHeat to the official repository (KSP-RO/RealHeat#2). Seems like that a simple recompile does the job! |
Yeah, I'm an idiot, didn't think to look at open PRs |
for anyone interested, i'm going to track my testing progress in this spreadsheet: https://docs.google.com/spreadsheets/d/19_dhwSioZODCCtMFD0kZOtbmbg--ryoE0nzoWJL2cRc/edit?usp=sharing Right now i'm just trynig to get the "Mandatory" mods, along with RSS, and see what works. I'm linking to the git commits or releases for all the dependencies, along with any notes regarding them |
@PhineasFreak the thing is rcsfx DOES use a different plume. It has it's own fx file that needs to be defined per config. I did this in my own mod files, but just changing the name won't do anything. It would either have no plume, or you'd see the new fx plume over the old particles. So basically if you do a check for modulercs* and the mod is using the new rcsfx module and you DON'T define the plume, it will be invisible. |
Is KSCSwitcher a listed RO dependency? It probably needs KSP-RO/KSCSwitcher#2 and a release. |
TestFlight is no longer pre-release |
Pinging the members for the RSSTimeDateFormatter. A removal of the "KSPUtil" reference and a recompile is all that is needed. I can also open a PR if you want. |
That is mine. If you can submit a PR to fix it that would be great otherwise I will look at it this weekend. EDIT: Re-read what you said and realized it is real simple :p I'll try to sneak that in today while I am at work. |
@jwvanderbeck Yep, i just forked the repo and i saw that the only thing required was a recompile. I had recompiled it locally when KSP 1.2.0 was originally released but i had forgotten that it did not require any changes to the code. |
Having some problems with Xamarin but will get it recompiled soon as I can. |
Consider MandatoryRCS mod in place of Persistant Rotation? |
Why? Dare to elaborate eberkain? Is Persistent Rotation out of development and not working in 1.2.2? |
No, Persistent Rotation is fine. I only suggested looking at what both mods offer and considering which one would be better to include. Didn't mean to make anyone mad. |
Who got mad? I just wanted to know if you knew more than I did and might enlighten me, that was without any mad connotation! |
Considering that RO basically nukes Reaction Wheels and makes it so that RCS is essentially required I see no reason to include Mandatory RCS. |
@jwvanderbeck @PhineasFreak Any luck on RSS Time Date Formatter? |
@RedAV8R I think that @jwvanderbeck will handle the required recompile. I can also open a PR to do so if he is busy. |
The problem I had was that I couldn't get it to build. My Xamarin project was being stupid and kept insisting that .NET was missing and I couldn't fix it :( I spent like a day trying to figure out what was wrong (TF builds fine) but couldn't figure it out. I'll try just wiping what I have and download it fresh from git and see if that works. |
@jwvanderbeck No probs. I also remember that one time that Xamarin acted stupidly after an update and would refuse to compile any of the projects that i had. Proper head banging stuff! It turned out that i had to uninstall and reinstall it in order for the required Microsoft Build Tools to also install correctly... |
Possibly unimportant, but I found that the previous version simply seems to work for me in KSP 1.2.2 ; maybe just needs a CKAN metadata change rather than recompile? |
RealHeat, ConnectedLivingSpaces and Deadly Reentry are all installable on 1.2.2 off of CKAN |
The Ferram Aerospace Research (FAR) mod is updated and live! |
@NathanKell @pap1723 @rsparkyc @SirKeplan So, all the pre-reqs for RO now appear to be available on CKAN now. Am I wrong? Is RO ready to go for a full release? If no objects, I am thinking of making a release either tonight or tomorrow. Thoughts? |
Both RO and RP-0? |
RO yay! But I would say RO only?
…On May 23, 2017 4:13 PM, "Pap" ***@***.***> wrote:
Both RO and RP-0?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1436 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AE5-ItSYcjzSKDXytKcq9zgvfPW3txKjks5r82gXgaJpZM4LDdSi>
.
|
KCT is still in beta develop so that rules out RP-0 for official CKAN / release |
KCT is listed as a recommendation. |
@Olympic1 this should probably be changed to a dependency, as RP-0 pretty much requires it for a balanced career gameplay. |
oh and RE: CLS - I didn't run into any issues with the latest 1.2.2 release. |
KCT just released for KSP 1.2. That makes all dependencies available for RP-0 |
Released |
The text was updated successfully, but these errors were encountered: