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

Fix 3rd #70

Merged
merged 18 commits into from
Jun 23, 2023
Merged

Fix 3rd #70

merged 18 commits into from
Jun 23, 2023

Conversation

lsh918
Copy link
Contributor

@lsh918 lsh918 commented Jun 20, 2023

수정사항

  • submodule로 들고 다니는 eigen의 태그 변경(3.4로 변경) 및 source 위치 변경(gitlab) 했습니다.
  • poetry 기반 설치에서 setuptools 기반 설치가 되도록 변경했습니다.
  • 일부 Cython code들에서 필요없는 import 들을 걷어냈습니다.
    • numpy C-API 관련 deprecated warning이 뜨는것도 제거했습니다.
  • CUDA 버전 컴파일시 발생하는 문제(nvcc가 있음에도 제대로 잡지못하는 문제)를 해결했습니다.
  • fire 및 이를 사용하는 CLI 를 제거했습니다.
  • ipynb_checkpoints 디렉토리를 제거했습니다.

@lsh918 lsh918 requested a review from ita9naiwa June 20, 2023 08:49
@hanmanhui hanmanhui self-requested a review June 20, 2023 09:00
Copy link
Contributor

@hanmanhui hanmanhui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isort에서 체크실패하고 있어서 수정필요해보입니다. 주석처리한 라인들은 남겨두는 이유가 있나요?

@lsh918
Copy link
Contributor Author

lsh918 commented Jun 20, 2023

@hanmanhui isort들은 모든 수정사항 적용 후 한번에 처리하려고 했습니다. 주석 처리된건 없애도 될듯한데, 제가 짠 코드가 아니긴 해서 혹시 작성할때 일부러 남겨둔 의도가 있는게 아닐까 싶어서 바로 삭제하진 않았습니다. 삭제하는게 더 괜찮으실까요?

hanmanhui
hanmanhui previously approved these changes Jun 21, 2023
Copy link
Contributor

@hanmanhui hanmanhui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

주석 처리한 변수들이 따로 쓰이진 않아 보여서 지워도 될 것 같습니다. 그외에도 이슈 없어보입니다.

@lsh918 lsh918 merged commit 6128e20 into dev Jun 23, 2023
2 checks passed
@lsh918 lsh918 deleted the fix-3rd branch June 23, 2023 02:55
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

Successfully merging this pull request may close these issues.

None yet

2 participants