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

Buff reward of far side landing optional and sample return contracts #2398

Merged
merged 4 commits into from
Jul 13, 2024

Conversation

Clayell
Copy link
Contributor

@Clayell Clayell commented Jul 13, 2024

Currently, the reward for the lunar landing optional contracts are 200 reputation each. However, the sample return contract, which is far more difficult, gives the exact same reward, and furthermore the far side landing optional contract gives less of a reward?
I've fixed this problem by scaling the rewards roughly on difficulty. (for reference, Luna 16 (sample return) weighed 5.7t wet, while Luna 13 (lander) weighed 1.6t wet)

@siimav
Copy link
Contributor

siimav commented Jul 13, 2024

400 for Return contract sounds good.
Not so sure about the 300 for optional far side landing though. Especially considering that you left the required contract at the same 150. Honestly I'd just set both of those at 200.

@Clayell
Copy link
Contributor Author

Clayell commented Jul 13, 2024

Honestly I'd just set both of those at 200.

Perhaps 250 would be more accurate? It was only accomplished in 2019, and it requires a relay and partial orbit before landing. Setting it to the same reward as the normal lunar landing contract doesn't make sense in my opinion.

Especially considering that you left the required contract at the same 150.

Woops, that's my bad.

@siimav
Copy link
Contributor

siimav commented Jul 13, 2024

Ehhhh, whatever.

@siimav siimav merged commit 78b0c44 into KSP-RO:master Jul 13, 2024
3 checks passed
@Clayell Clayell deleted the rep-changes branch July 13, 2024 20:49
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.

2 participants