feathr-ai/feathr

Path concatenation without forward slash

Open

#297 aperta il 23 mag 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Scala (244 fork)batch import
featuregood first issue

Metriche repository

Star
 (1929 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

This is reported by Manjunath from Slack channel. For work_dir: "dbfs:/feathr_getting_started" The user was running jupyter notebook from windows. without forward slash('dbfs:/feathr_getting_started') the job configs were coming as '--join-config', 'dbfs:/feathr_getting_started\feature_join.conf'

To fix this, the user need to add foward slash: work_dir: "dbfs:/feathr_getting_started"

We should add validation or automatically append a forward slash. The best way would be using Path utils that can join files/folders properly.

Guida contributor

Direzione di ricerca
Utilizza os.path.join o pathlib di Python per unire correttamente i percorsi invece di semplici concatenazioni di stringhe. Aggiungi convalida per assicurarti che work dir termini con una barra.
Tech stack
pythonscala
Dominio
backenddata
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonBasic file path knowledge
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70