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.

Sign up for GitHub

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

Jump to bottom

X and y not of same length #14

Closed
RK900 opened this issue Jun 8, 2017 · 8 comments
Closed

X and y not of same length #14

RK900 opened this issue Jun 8, 2017 · 8 comments
Labels
bug encoding

Comments

@RK900
Copy link
Collaborator

RK900 commented Jun 8, 2017 •
edited
Loading

  • Actual length should be 29971
  • Features of 6892348 - length 30092
  • Labels of 6892348 - length 29830
  • Difference of 262
RK900 added a commit that referenced this issue Jun 15, 2017
@RK900
Encode pasting and resetting, fix y (labels) element of #14
fcf78e8
@RK900
Copy link
Collaborator Author

RK900 commented Jun 15, 2017

30092 - 29971 = 121
121 total movesets
Adding 1 extra feature for every moveset, thus the extra 121

@RK900
Copy link
Collaborator Author

RK900 commented Jun 16, 2017

removed extra feature at the end of each puzzle
some puzzles have 0 moves, so has extra 2 features

@RK900 RK900 mentioned this issue Jun 16, 2017
Closed
@RK900 RK900 closed this as completed in 3d5f886 Jun 16, 2017
@RK900
Copy link
Collaborator Author

RK900 commented Jun 19, 2017

Need to fix this for pasting and resetting sequences

  • Should be 29973
  • Actual is 29796
  • Difference of 177

@RK900 RK900 reopened this Jun 19, 2017
@RK900
Copy link
Collaborator Author

RK900 commented Jun 19, 2017

Lengths equal for encoding pasting & resetting; closed by #10.

@RK900 RK900 closed this as completed Jun 19, 2017
@RK900
Copy link
Collaborator Author

RK900 commented Jun 22, 2017

Occurring for 6502998

@RK900 RK900 reopened this Jun 22, 2017
@RK900
Copy link
Collaborator Author

RK900 commented Jun 22, 2017

Due to error in data: Length of puzzle is 80, but base change is at position 101

@RK900
Copy link
Collaborator Author

RK900 commented Jun 23, 2017

Added moveset6-22a.txt, which only contains non-faulty/corrupted data

@RK900 RK900 closed this as completed Jun 23, 2017
@RK900
Copy link
Collaborator Author

RK900 commented Jul 19, 2017 •
edited
Loading

Occurring for expert movesets on puzzle IDs 6502966,6502976,6502984

@RK900 RK900 reopened this Jul 19, 2017
@RK900 RK900 removed the priority label Jul 20, 2017
@RK900 RK900 closed this as completed Jan 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug encoding
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@RK900