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

[18' 최상희] 최종보고서 #84

Open
pengin7384 opened this issue Oct 25, 2018 · 0 comments
Open

[18' 최상희] 최종보고서 #84

pengin7384 opened this issue Oct 25, 2018 · 0 comments

Comments

@pengin7384
Copy link

pengin7384 commented Oct 25, 2018

Issue

image

  • 인자의 순서가 아닌 인자명으로 추적하는 기능을 제안하였으나 부적합하다고 판단

PullRequest

image

image

image

image

Conversation

image

  • info.py에 정보가 너무 부족하다고 생각하여 elapsed_time정보를 추가한 후 PullRequest를 전송했다.

image

  • 홍규님께서 record 타임 도중에는 해당 정보를 얻을 수 없을 것이다 라고 커멘트를 남겨주셨다.

image

  • record중일경우 자동으로 null이 반환되어 문제가 생길것 같지 않다고 커멘트를 남겼다.

image

  • 남형님께서 elapsed_time의 타입을 지정하여 저장할것을 제안하셨다.

image

  • 해당부분을 수정한 후 커멘트를 남겼다.

image

  • 남형님께서 double타입보다는 int(long) in nano second unit을 사용하면 더 나을것 같다고 제안해주셨다.

image

  • 3초가 넘을 경우 오버플로우가 발생할것 같아서 지속적으로 커뮤니케이션중이다.

기존

$ uftrace script -S info.py
False
v0.8.3-537-g0836 ( dwarf python tui perf sched )
()

수정후

$ uftrace script -S info.py 
False
v0.8.3-537-g0836 ( dwarf python tui perf sched )
()
0.06264812

후기

처음 컨트리뷰톤을 시작할때는 깃조차 제대로 쓰지 못하였는데 멘토님들 덕분에 이제는 기초적인 깃사용법부터 시작해서 이슈 등록하기, PullRequest전송, 코드 리딩법, 디버깅 등 다양한 기술을 배워서 사용할 수 있게 되었습니다.

uftrace의 기본 원리를 이해하기 어려웠는데 정기모임 시간이 끝난 후에도 끝까지 이해를 도와주신 한범 멘토님!
하루종일 질문해도 항상 웃으며 답변해주시는 송태웅 멘토님! 감사합니다!

1540143982928
1540143989685

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

No branches or pull requests

1 participant