-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat(data): Expanding Alpha Centauri, and rearranging to be circumstellar rather than circumbinary #73
feat(data): Expanding Alpha Centauri, and rearranging to be circumstellar rather than circumbinary #73
Conversation
Thanks for reposting. Just as an aside for myself and anyone curious in the future, the wikipedia article on Alpha Centauri: |
I should be able to test this at some point this week, sorry for taking a bit to get to it. Could you update it please? If it's OK with you I can just hit the "update branch" button, but I don't want to risk breaking anything if you're using this branch elsewhere too. |
I hadn't noticed the update branch button before, so thank you for pointing it out. |
You are welcome! And thank you for updating. |
Sorry it's taken me longer than expected to get to this. Thanks for updating! |
@Zitchas what's the status on this? |
The status is "waiting for me to have the capability and time to sit down and play it." First I didn't have time, then I didn't have the capability to build it due to software failures. And then didn't have the capability due to hardware failures. So the delay is all on me at this point. |
No worries, these things take as long as they take. And, feel free to hit the update branch button yourself when you've got time rather than waiting on me. |
OK, this will be tested within the next 24h. For sure this time. sigh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've retested. It certainly makes Alpha Centauri feel a lot bigger, and the player's (and others') ships a lot smaller.
and no, I definitely didn't test by demanding tribute
OK, I've finally tested it. Thanks for your patience, Quantumshark. My thoughts: All in all, I like it. |
One thing in regards to the asteroid belt... It's very sparse and hard to really notice. I had trouble locating it. edit: I eventually added a x10 to the strength of asteroid scanners, sold my HD, and loaded up my shuttle with 8 of them in order to see where the ring was. It's probably in a good location, but probably needs to be a bit more dense and/or more valuable, or the generic asteroids need to be removed. Right now, the generic ones are just dense enough to render the meaningful asteroids hard to distinguish a pattern. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can tweak it more in the future, but I'm satisfied it's good enough to merge and include alongside other experiments.
Thanks!
Content
Same changes as #66, but not broken.
Summary
Larger, semi-real scaled Alpha Centauri (obviously, it's not possible to get it 100% accurate as there isn't support for elliptical orbits), with the planets orbiting the stars individually rather than both stars in the middle.
Ramscoop and solar collection will work weirdly until someone fixes the code to make them work from each star separately rather than the current approach of summing them across all stars and emanating all that from the centre of the system.
Screenshots
Testing Done
Went to the system and looked at it to check that the planets were in the right places
Save File
This save file can be used to test these changes:
Any; just visit Alpha Centauri
Performance Impact
Asteroid belt feels like it might be an issue, but it's less dense than the one you've got in Sol and I'm not aware of that one being a problem.