OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

fix(style,frontend): Print terminal output character by character #187

Open Yashwanth-Chandrakumar opened this issue 11 months ago • 9 comments

Added a 50ms delay between printing of commands in OpenDevin's Terminal

Yashwanth-Chandrakumar avatar Mar 28 '24 09:03 Yashwanth-Chandrakumar

Hi, I noticed that you removed some comments in your pull request. I believe that the comments you removed were necessary or helpful. I'm wondering why you did that.

808vita avatar Mar 28 '24 10:03 808vita

Sorry for that issue i have added the comments again.

Yashwanth-Chandrakumar avatar Mar 28 '24 10:03 Yashwanth-Chandrakumar

Hi @rbren i have a small doubt when i create a pull request here and commit to the branch in my forked repo, why does it show like 9 commits ahead of OpenDevin/Opendevin:main

Yashwanth-Chandrakumar avatar Mar 28 '24 17:03 Yashwanth-Chandrakumar

OK I think I miscommunicated what I want to see here 😄

I want it to look like the AI is typing live, like it does with ChatGPT typing

rbren avatar Mar 28 '24 17:03 rbren

i have a small doubt when i create a pull request here and commit to the branch in my forked repo, why does it show like 9 commits ahead of OpenDevin/Opendevin:main

This means that you've made 9 new commits that aren't merged into the main branch--it's expected

rbren avatar Mar 28 '24 17:03 rbren

Would be good to have this behavior for both chat messages and for the terminal

rbren avatar Mar 28 '24 17:03 rbren

Ok thanks for the comfirmation so when you merge my pull request the 9 commits head of would disappear right

Yashwanth-Chandrakumar avatar Mar 28 '24 17:03 Yashwanth-Chandrakumar

Would be good to have this behavior for both chat messages and for the terminal

Ok Sure

Yashwanth-Chandrakumar avatar Mar 28 '24 17:03 Yashwanth-Chandrakumar

Ok thanks for the comfirmation so when you merge my pull request the 9 commits head of would disappear right

yup exactly

rbren avatar Mar 28 '24 19:03 rbren