Skip to content

Conversation

@mennetob
Copy link
Contributor

Loading from sorted ark files, speeds up the data loading significantly. It should not change the outcome significantly, since the segments are later reordered by sequence length for batching anyways.

Implementation structure leaves the option open to make this configurable later

@TParcollet
Copy link
Collaborator

@mravanelli ok to merge that? Seems ok to me.

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.

2 participants