-
Notifications
You must be signed in to change notification settings - Fork 669
Closed
Labels
better engineeringTasks which help improve eng productivity e.g. building tools, cleaning up code, writing docsTasks which help improve eng productivity e.g. building tools, cleaning up code, writing docscommunity help wantedWe would love the community's help completing this issueWe would love the community's help completing this issue
Description
Currently, if you print a Message
object of a list of Message
s it just shows the memory address of the class, which is lame.
[<torchtune.data._messages.Message at 0x7fc0e5f6cd30>,
<torchtune.data._messages.Message at 0x7fc0e67f2470>]
We should implement a proper __str__
method to make the output legible. Ideally, something like:
Message(role="user", content="Hello world!")
SalmanMohammadi and joecummings
Metadata
Metadata
Assignees
Labels
better engineeringTasks which help improve eng productivity e.g. building tools, cleaning up code, writing docsTasks which help improve eng productivity e.g. building tools, cleaning up code, writing docscommunity help wantedWe would love the community's help completing this issueWe would love the community's help completing this issue