Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add screen clear when 'top' feature updates screen #227

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jehunseo
Copy link

  • guider/guider.py top -a 명령어 사용시 시스템 자원 표시가 화면 갱신 없이 기록되는 방식으로 나타나고 있습니다
  • 이를 os.system() 함수로 데이터 갱신이 일어날 때마다 화면이 clear되도록 기능을 추가하였습니다.

@iipeace
Copy link
Owner

iipeace commented Jul 31, 2020

@jehunseo,
Guider는 windows에서 top 명령어를 지원하지 않고 있습니다.
혹시 windows에서 동작하도록 구현하실 계획이신지요?

@jehunseo
Copy link
Author

WSL 환경에서 코드를 동작하였는데 top 명령어 동작이 화면 갱신 없이 동작하는 부분을 수정할 수 있을까 하여 해당 수정사안을 제안하게 되었습니다.
리눅스 외에 윈도우는 제한적이지만 혹시나 추후 구현될까 하여 os를 구분하는 구문을 추가하였습니다.

Windows상에서 top 명령어와 같이 동작하도록 구현하는 일에 관심은 있지만 시간을 내기는 힘들 것 같습니다. 이후 해당 기능에 대해 작성할 수 있다면 다시 request하도록 하겠습니다.

@jehunseo jehunseo closed this Jul 31, 2020
@iipeace
Copy link
Owner

iipeace commented Aug 6, 2020

@jehunseo ,
WSL은 제가 생각하지 못했네요!
좋은 제안 감사합니다 :)

@iipeace
Copy link
Owner

iipeace commented Aug 6, 2020

@jehunseo ,
제가 Windows10 WSL Ubuntu 환경에서 python3 guider.py top 명령을 실행해보니 화면갱신이 잘 되는데요.
혹시 화면갱신이 되지 않았던 환경 정보룰 공유해 주실 수 있을까요?

@jehunseo
Copy link
Author

jehunseo commented Aug 7, 2020

@jehunseo ,
제가 Windows10 WSL Ubuntu 환경에서 python3 guider.py top 명령을 실행해보니 화면갱신이 잘 되는데요.
혹시 화면갱신이 되지 않았던 환경 정보룰 공유해 주실 수 있을까요?
image

Windows 10 Home edition에 Ubuntu 20.04 LTS 환경입니다. WSL 2버전으로 사용하고 있었구요.

@iipeace iipeace self-requested a review August 8, 2020 14:43
@iipeace iipeace reopened this Aug 8, 2020
@iipeace
Copy link
Owner

iipeace commented Oct 19, 2020

제 Windows10 WSL2 Ubuntu 20 환경에서는 screen clear가 잘 되는데요,
사용하시는 환경과 어떠한 차이점이 있는지 궁금하네요 ^^;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants