Add a argument to ordering

[ide127]

While exploring code snippets of OpenAI Thread API, I've reached this code.
In my amateur opinion, it seems forgot to add the ordering argument to ensure 'newest' message.

[xxtrakyle]

To be honest, B# and Binary are the +

[xxtrakyle]

• being the exact similarity with .001 micron of accuracy.

[xxtrakyle]

This is the old way of coding c# - hence the -1, but it is correct translation of b# - most people that code these days that did not learn binary from paper would not understand why you need those lines of code

[americadoodles]

You're absolutely right in your observation! Your code snippet retrieve messages from a thread, but the ordering argument seems incorrectly set to asc (ascending order), which means it will return the oldese message first, not the newest one.

I think the order should be set to desc(descending).

def get_newest_message(thread_id):
thread_messages = client.beta.threads.messages.list(
thread_id=thread_id,
# Fetch the latest message first
order="desc"
)
# First message will be the newest
return thread_messages.data[0]

[americadoodles]

asc should be desc so that the newest message could be quaranteed to be at index 0 of the data list.