Skip to content

Conversation

@yuchongzhang
Copy link
Collaborator

@yuchongzhang yuchongzhang commented Mar 27, 2024

PR Type

[Feature | Fix ]

Short Description

  1. A refactor of the BasicClient class to support the model taking in a dictionary of tensors as its input. This is useful, for example, if one wants to train a language model on text data. In this case, in addition to the tensors that represent the tokenized sentence, we may also need to pass in the attention masks.
  2. An extension of the label-based samplers to support a dictionary-based dataset.
  3. A refactor of the dynamic layer exchange example to use hugging face models and datasets. The new example also shows how to use collate functions to dynamically pad the batches as opposed to padding everything to the maximum length, which is what we did before.
  4. The code for the same AG news experiment using the sparse tensor exchanger.
  5. Moved the original dynamic layer exchanger example to the research folder and replaced it with a typical example on non i.i.d. CIFAR-10.
  6. Some small fixes here and there.

Tests Added

N/A

@emersodb emersodb self-requested a review April 15, 2024 17:33
Copy link
Collaborator

@emersodb emersodb left a comment

Choose a reason for hiding this comment

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

Good to go from me

@yuchongzhang yuchongzhang merged commit a771bb3 into main Apr 15, 2024
@yuchongzhang yuchongzhang deleted the AG_partial_exchange branch April 15, 2024 17:44
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.

3 participants