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

offsets.size or offsets.size-1 #106

Closed
EthanHu29 opened this issue Jun 11, 2024 · 1 comment
Closed

offsets.size or offsets.size-1 #106

EthanHu29 opened this issue Jun 11, 2024 · 1 comment

Comments

@EthanHu29
Copy link

In line 272 actions.kt, t.offsets.size should be t.offsets.size-1?

line 272 val (aNew, msRem) = ts.zip(ts.scan(0) { acc, t -> acc + t.offsets.size }).foldRight(

As the size of the offsets is always the number of paths plus 1?

@EthanHu29
Copy link
Author

I changed one of the test cases like shown here
1
The actions generated by .build() are
2
After change line 272, the actions generated by .build() changed to
3
Based on the test case rewritten, last action should be a parrallel action as shown above.

@rbrott rbrott closed this as completed in fadff2e Jul 15, 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
Development

No branches or pull requests

1 participant