Consider reorganizing file layout to shorten path lengths
@rino20 ci sta già lavorando.
Dal 11/1/2023.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Describe the bug
The current package includes path names upwards of 150 characters such as default_8bit_cluster_preserve_quantize_scheme.py which are in turn relatively short files deep in a hierarchy. On Windows, the default limitation for paths is 260 characters as defined by MAX_PATH. While it is possible to modify this on a per-call basis, by default the Python ecosystem's tooling does not make this process simple for end users and there are a variety of tools still limited to the 260 character MAX_PATH. When you combine the path lengths of this repository with a location in a users' profile such as C:\Users\person-fullname\AppData\Local\Python\environments\deep-learning-environment\Lib\site-packages the limit can be exceeded.
Motivation
I work in a context where users can create environments in arbitrary locations and we want to use this package, the workaround we've found for the time being is to repack this repository as a Python egg and inject that into the environment at runtime, but would prefer to be able to use it as a direct Python dependency. Note that this limitation is specific to Windows, fortunately MacOS defaults to a MAXPATHLEN of 1024 and Linux values are typically 1024 or higher.
System information
TensorFlow version (installed from source or binary): All versions
TensorFlow Model Optimization version (installed from source or binary): All versions, specifically found working with v0.7.3.
- Lingua principale
- Python
- Stelle
- 1.6k
- Fork
- 349
- Merge medio
- 3g 2h
- PR unite (30g)
- 1
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di tensorflow/model-optimization
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
tensorflow/model-optimization#1301 · 1 commento ·
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
tensorflow/model-optimization#1272 ·
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 50/100
tensorflow/model-optimization#1270 · 2 commenti ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 48/100
tensorflow/model-optimization#1241 ·
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
tensorflow/model-optimization#1182 · 1 commento ·
Tutte le issue di tensorflow/model-optimization
Issue simili
-
agent-ready documentation needs-triage
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
-
instance instance add
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
searxng/searx-instances#939 · 1 commento ·
-
area-deployment area-integrations triage:bot-seen
Difficoltà 2/5 Mezza giornata Idoneità per principianti 86/100