Make traker optional for LESS/NICE installation#41
Make traker optional for LESS/NICE installation#41luojiyin1987 wants to merge 6 commits intoOpenDCAI:mainfrom
Conversation
|
Thanks for working on this and for thinking about the installation experience. I took a closer look at the dependency side, and I think the situation here may be a bit different from what the PR description assumes. traker itself only depends on torch, numpy, and tqdm, and DataFlex already depends on packages like transformers, accelerate, peft, and trl, which already bring in PyTorch. So even if we remove traker from the base requirements, users would still end up installing the PyTorch/CUDA stack anyway. Because of that, I don't think making traker optional would really make the base installation lighter in practice. At the same time, it would add one more step for LESS/NICE users and make those selectors fail at runtime if the extra dependency is missing. So for now I’d prefer to keep traker in the default requirements. Still, thanks again for putting this together and for trying to improve the onboarding flow. If you see another way to simplify installation that would have a more concrete effect, I’d be very happy to discuss it. |
|
I tried to install it several times, but it failed. I use the Tsinghua source |
Summary
This PR makes the
trakerdependency optional for first-time users and documents separate base vs optional installation paths.Changes
trakerfrom the baserequirements.txtpyproject.tomlfortraker,less, andnicetrakerinless_selector.pyandnice_selector.pyLESSorNICEis selected withouttrakerREADME.mdandREADME-zh.mdwith bothuvandpipinstallation pathsWhy
trakercurrently pulls in large PyTorch/CUDA wheels during base installation, even for users who do not intend to useLESSorNICE. On slower or unstable networks this can fail early in the onboarding flow.Making it optional keeps the minimal install path lighter and more reliable while preserving explicit installation for advanced selectors.
Closes #40
Validation
python3 -m compileall srcpyproject.tomlsuccessfullyNotes
This is intentionally a small change focused on installation UX. It does not alter training logic beyond dependency loading behavior for
LESSandNICE.