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

Price some decouplers/bases/nosecones. #732

Closed
wants to merge 2 commits into from
Closed

Price some decouplers/bases/nosecones. #732

wants to merge 2 commits into from

Conversation

ahmedcharles
Copy link
Contributor

The prices are based primarily on similar parts or the old tech tree prices.

@ahmedcharles
Copy link
Contributor Author

I'm not listing them as being RP-0 configured because I suspect having the entry cost be 1 for everything will be contentious and all I really want to do is improve the costs of parts that aren't currently configured rather than spend lots of time debating. Hopefully that means this can just get merged and I won't have to keep local changes that do this.

@NathanKell
Copy link
Member

I believe these changes should be made to @pap1723 's xls file?

@ahmedcharles
Copy link
Contributor Author

How do I do that?

The prices are based primarily on similar parts or the old tech tree
prices. For the tanks, the prices are based on procedural tanks with the
same volume, with some adjustment for complexity of design.

The entry costs are all 1 to make them competitive with procedural
parts.
@pap1723
Copy link
Contributor

pap1723 commented Aug 4, 2017

These are included with the new developmental updates. HOWEVER, there is no pricing adjustments for the fuel tanks. NathanKell has changed the costs of the PP Tanks to be more in line for the Launch Cost major changes that are coming. Therefore, these fuel tanks should be priced by using PP Tanks and determining their cost. That will have to come at a later time when these changes are fully complete.

@pap1723 pap1723 closed this Aug 4, 2017
@ahmedcharles ahmedcharles deleted the tree branch August 5, 2017 04:03
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.

None yet

3 participants