Skip to content
This repository was archived by the owner on Sep 10, 2025. It is now read-only.

Conversation

@Gasoonjia
Copy link
Contributor

@Gasoonjia Gasoonjia commented Jul 30, 2024

This pull request enables the definition of multi-transformer models, including updates to ModelArgs and the Model class in the builder. These changes will serve as the primary entry point for both torchchat-defined models and imported models from torchtune.

@pytorch-bot
Copy link

pytorch-bot bot commented Jul 30, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/torchchat/966

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 11113ff with merge base 9dc9eff (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 30, 2024

import torch
import torch.nn as nn
from torch.distributed.device_mesh import DeviceMesh
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Some irrelevant updates come from lintrunner

@Gasoonjia Gasoonjia changed the title Enable multi transformer models in codebase [Llava][multimodal] enable Llava in torchchat Jul 30, 2024
@Gasoonjia Gasoonjia requested a review from Jack-Khuu August 1, 2024 18:18
@Gasoonjia Gasoonjia changed the title [Llava][multimodal] enable Llava in torchchat [Llava][multimodal] enable multi-transformer defination Aug 26, 2024
@Gasoonjia Gasoonjia changed the title [Llava][multimodal] enable multi-transformer defination enable multi-transformer defination Aug 26, 2024
Copy link
Contributor

@Jack-Khuu Jack-Khuu left a comment

Choose a reason for hiding this comment

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

Amazing work

Looks like export_et is a victim of rebasing though; that file got merged into export

@@ -0,0 +1,135 @@
# Copyright (c) Meta Platforms, Inc. and affiliates.
Copy link
Contributor

Choose a reason for hiding this comment

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

This file got merged into export itself, you'll want the changes over there

@Gasoonjia Gasoonjia merged commit d0d1105 into main Aug 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants