aws / aws/sagemaker-python-sdk

Add support for multiple requirements.txt files

Aperta
#2,690 1 commento 0 reazioni 1 assegnatario Rivendicata da @nargokul Vedi su GitHub
component: pysdk-team type: feature request
Lingua principale
Python
Stelle
2.3k
Fork
1.3k
Merge medio
1g 22h
PR unite (30g)
35

Descrizione

**Describe the feature you'd like**

Right now, its expected that `requirements.txt` will be found (**With that name**) in the script directory. In some cases, multiple requirements.txt files are needed to install the correct dependencies. Unfortunately, right now you'd have to edit your code in order to merge to one unified requirements.txt file if you have multiple.

**How would this feature be used? Please describe.**

Allow `dependencies` kwarg to pick up any `.txt` files as requirements files when they're added to the list.

```python
estimator = HuggingFace(
...
dependencies=['./requirements.txt', '../_test_requirements.txt']
)
```

**Describe alternatives you've considered**
Right now, I'm:
- merging all requirements files a user specifies in a CLI tool I made
- Writing the merged requirements to a new tmp directory
- Adding all python files to the tmp directory as well
- Launching training runs out of the temp directory

I've also seen folks installing requirements directly in their python files, but that just feels wrong to me.

**Additional context**
...

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.