search

oss2024hnu / coursegraph-py

BSD 2-Clause "Simplified" License
5 stars 71 forks source link

show_dot.py에서 비효율적인 루프 구조 개선 #513

Open awc0596 opened 3 weeks ago

awc0596 commented 3 weeks ago

이전 코드에서는 노드가 없는 경우를 처리하는 로직이 주석 처리되어 있습니다. 'subGdict' 및 'nodedict' 를 딕셔너리 컴프리헨션을 사용해 초기화하고, 모든 학기별 노드를 먼저 생성한 뒤, 노드 연결 처리를 개선했습니다. 각 과목과 관련된 스타일 적용 또한 한 곳에서 처리되어, 노드 생성 로직과의 중복을 제거

kyagrd commented 3 weeks ago

충돌이 나고 있습니다. 해결해 주세요

kyagrd commented 3 weeks ago

충돌나고 있네요 아마 여러 PR이 같은 파일에 대해 올라와서 다르 먼저 머지된 거 때문에 그런 거 같은데 충돌 해결해 주세요

kyagrd commented 3 weeks ago

그러니까 저런 설명을 이슈에다 적는거라니끼요. 이슈에다 안적어놓아서 점수르 더 받을 수 있는 걸 못받았네요

awc0596 commented 3 weeks ago

지금 다시 이슈에 수정 작성해서 올려도 될까요

kyagrd commented 3 weeks ago

아뇨 이미 이슈는 반려했습니다. 지금 코드까지 다 작성했는데 ... 순서가 반대라서 다음에 다른 이슈를 올려주세요

kyagrd commented 3 weeks ago

이미 PR이 올라온 내용은 중복 이슈죠