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

Large difference in number of predicted species with ED Observatory #57

Open
rster2002 opened this issue May 26, 2024 · 4 comments · May be fixed by #70
Open

Large difference in number of predicted species with ED Observatory #57

rster2002 opened this issue May 26, 2024 · 4 comments · May be fixed by #70
Assignees
Milestone

Comments

@rster2002
Copy link
Owner

The current implementation of exobiology predictions return a lot more species than elite observatory. Maybe this is caused by incorrect spawn conditions?

ED Observatory:
image

Journals:
image

@Somfic
Copy link
Contributor

Somfic commented May 26, 2024

Happy to pick up this issue if you'd like.

I found EDMC-BioScan, which claims:

With the latest updates, BioScan can be slightly more accurate at ruling out certain bios with overlapping criteria. In a few edge cases, BioInsights may be slightly more accurate. Determining region boundaries and proximity to nebulae can be hard.

However I can't find the source of BioInsights anywhere.

@rster2002
Copy link
Owner Author

If you would that would be amazing! Best to branch from the state branch as I changed how spawn source is constructed a bit, but predictions logic is still the same.

@rster2002
Copy link
Owner Author

This is the code I bodged together to test a bit: https://gist.github.com/rster2002/3ef65973bcf9fbb89809797ace4861b5

Looking at this there should also be a check for whether the planet is landable, which it currently doesn't check.

@Somfic
Copy link
Contributor

Somfic commented May 26, 2024

That sounds.. like a good condition to check for xd

@rster2002 rster2002 linked a pull request Jun 17, 2024 that will close this issue
@rster2002 rster2002 added this to the Version 0.6.0 milestone Jul 6, 2024
@rster2002 rster2002 self-assigned this Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants