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

Actually award reputation for landing on the runway correctly #2479

Closed
wants to merge 15 commits into from

Conversation

Clayell
Copy link
Contributor

@Clayell Clayell commented Jan 13, 2025

As far as I know, "@/rewardReputation" is not a valid name.

Potentially fix #2401

@Capkirk123
Copy link
Member

Has this been tested?

@Clayell
Copy link
Contributor Author

Clayell commented Jan 14, 2025

No, but I can't think of any reason why it would be "@/rewardReputation" instead of "@rewardReputation", right?
I will try to test if I have some time.

@njits23
Copy link
Contributor

njits23 commented Jan 14, 2025

Landing reputation gets paid out correctly. The problem in issue #2401 is that confidence also shows up as a reward in the contract completion screen. This is unexpected, as confidence is not listed as a reward in mission control when accepting the contract. It only shows reputation. What issue #2401 actually requires to fix it is that confidence no longer shows up in the contract completion screen. Unless it is actually intended that landing on the runway also awards confidence, which this PR wouldn't fix anyways. The relevant CC documentation on the current configs being a valid syntax is over here: https://github.com/jrossignol/ContractConfigurator/wiki/Expressions#special-identifiers

In this case the reference to the root node is also quite important, as it is specifically referring to the @/rewardReputation in the root node, and not to @/OptVesselGroup/rewardReputation.

@Clayell Clayell closed this Jan 14, 2025
@Clayell Clayell deleted the rep branch January 14, 2025 20:15
@Clayell
Copy link
Contributor Author

Clayell commented Jan 14, 2025

Thanks for clearing that up!

Unless it is actually intended that landing on the runway also awards confidence

I believe that the contracts are only meant to provide reputation, but these contracts were originally written long before confidence was added. In my opinion, they should give some extra confidence, as reputation only serves to mildly increase your subsidy.

description = <b>Program: X-Plane Research<br>Type: <color=blue>Optional</color></b><br><br>Design, build, and launch a crewed rocketplane to achieve an altitude of @/altitudeKm km, a velocity of @/velocity m/s and return home safely. Optionally, landing back at the runway will award extra reputation. If playing from the Cape, the Space Shuttle runway provided by the RSS-CanaveralHD mod is also a valid landing target.<br><br>This is a series of @maxCompletions contracts, of which @index have been completed.<br><br><b><color="red">This contract will no longer be offered once you complete 'X-Planes (High Altitude)'!</color></b>

Optionally, landing back at the runway will award extra reputation. If playing from the Cape, the Space Shuttle runway provided by the RSS-CanaveralHD mod is also a valid landing target.

Where is confidence awarded in contracts anyway? Is it just an automatic calculation based on the amount of reputation/science that the contract gives?

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.

Landing on runway bonus confidence
3 participants