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

Fossil Solver #1156

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Fossil Solver #1156

wants to merge 15 commits into from

Conversation

olim88
Copy link
Contributor

@olim88 olim88 commented Feb 1, 2025

Helps find fossils in the Fossil Excavator

shows heat map of the most probable tile to be a fossil

image

adds extra information to tootips

image

@LifeIsAParadox LifeIsAParadox added the reviews needed This PR needs reviews label Feb 1, 2025
olim88 added 12 commits February 1, 2025 22:37
nearly working state however lots of the code will need refactoring once i understand what it needs to do
still needs refactor but works now
needs a refactor next and a few small feature
hopefully good code now. ready to finally make a pr
@AzureAaron AzureAaron added the new feature This issue or PR is a new feature label Feb 2, 2025
Copy link
Collaborator

@kevinthegreat1 kevinthegreat1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this is fine but I can't test.

Comment on lines 176 to 178

@SerialEntry
public boolean fossilSolver = true;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is in the wrong place. I have fixed it.

@LifeIsAParadox LifeIsAParadox added merge me please Pull requests that are ready to merge and removed reviews needed This PR needs reviews labels Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge me please Pull requests that are ready to merge new feature This issue or PR is a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants