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.

Sign up for GitHub

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

Jump to bottom

Initial code review for DLWMLS #2

Merged

euroso97 merged 1 commit into main from spiros-dev

Nov 14, 2024

Member

spirosmaggioros commented Nov 10, 2024 •

edited

Loading

Added pre-commit bot checker
nnunet_ subfolders shouldn't be there as hugging face downlad them and in order to actually push these folders the user must have git LFS configured, which is not a good idea
A minor change in setup.py, the iteration on requirements.txt is not ideal as the requirements are meant for the devs, the user doesn't need to have all the packages installed.


          Added pre-commit configuration and pre-commit bot | Some small fixes …

e088660

…on setup.py(nothing special) | Remove nnunet_* subfolders

spirosmaggioros requested a review from euroso97

November 10, 2024 00:26

spirosmaggioros self-assigned this

spirosmaggioros added the code review label

euroso97 merged commit 1cdf459 into main

3 checks passed

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

Labels