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

Add some error checking to optimiser to fix issues #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AlexCPU
Copy link

@AlexCPU AlexCPU commented Mar 30, 2021

I've added a little bit of error handling to try and work out some of the issues I've been seeing.

For example I am consistently getting the error that the pool has too many crew, and it's always Niners Ezri, so a little more debug in assignCrewToPools which seems to give the below debug output implying it can't count properly, e.g. in this example pool.assignedCrew.length shows 11 with only 10 seats, however the debug view shows length 8.
image

Also, I'm always getting an error in sortCrewToTrain where highestContribingTrainee is blank.
image

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.

1 participant