-
Notifications
You must be signed in to change notification settings - Fork 7
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
[FEAT] query viz func add #63
base: master
Are you sure you want to change the base?
Conversation
"def query_viz(query, n_rows, n_cols, image_mode = False):\n", | ||
" items = [set() for _ in range(n_cols)]\n", | ||
" if image_mode:\n", | ||
" # image mode\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분이 에러가 있는데 if문의 실행이 정의되어 있지 않습니다.
의도하신 부분일까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저 부분이 image 단위로 출력하는 부분인데, 구현에 어려움이 있어서 도움을 요청하였습니다 ~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
도움을 요청한 부분이었군요. 확인해보겠습니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네, 감사합니다!
@ppskj178 주피터 파일은 +를 누르면 바로 저렇게 열리고 해당 이미지처럼 되지 않는데 다른 방법이 있는걸까요? |
주피터 파일을 보는중에 해당 버튼을 눌러서 코드형태로 출력하고 동일하게 진행할 수는 있는데 그냥 이슈를 수동으로 등록하고 아래처럼 링크 걸고 코드일부를 복붙해오는방식도 좋아보입니다. |
네 앞으로는 그렇게 진행해야겠네요 |
query마다 visualize하는 부분이 재활용되는 것 같아서 이 부분을 함수로 추가해보았습니다.
또한, #40에서 이야기 나왔던 image 단위로 출력하는 부분을 작성하려했으나, 어려움이 있어서 다른 분들이 도움 주셨으면 좋겠습니다.