scripts: Fix remote option in Windows (#14102)

[pqnet]

Make sure to read the contributing guidelines before submitting a PR

[pqnet]

fixes #14102

[CISC]

The model arg should probably not be a Path, but only assigned to dir_model as a path when not remote.

[pqnet]

The model arg should probably not be a Path, but only assigned to dir_model as a path when not remote.

Probably. I do not know enough of the codebase to do this kind of change so I just fixed the errors I got while running the script

[CISC]

The model arg should probably not be a Path, but only assigned to dir_model as a path when not remote.

Probably. I do not know enough of the codebase to do this kind of change so I just fixed the errors I got while running the script

Would you be willing to give it a go? It's fairly simple.

Change this:

llama.cpp/convert_hf_to_gguf.py

Line 6352 in 1caae7f

"model", type=Path,

Change this to args.model:

llama.cpp/convert_hf_to_gguf.py

Line 6461 in 1caae7f

repo_id=str(dir_model),

Move this assignment into an else clause, converting it to Path:

llama.cpp/convert_hf_to_gguf.py

Lines 6456 to 6464 in 1caae7f

	dir_model = args.model
	if args.remote:
	from huggingface_hub import snapshot_download
	local_dir = snapshot_download(
	repo_id=str(dir_model),
	allow_patterns=["LICENSE", "*.json", "*.md", "*.txt", "tokenizer.model"])
	dir_model = Path(local_dir)
	logger.info(f"Downloaded config and tokenizer to {local_dir}")

No str needed any more:

llama.cpp/convert_hf_to_gguf.py

Line 6489 in 1caae7f

fname_out = Path("./" + str(args.model).replace("/", "-") + "-{ftype}.gguf")

[CISC]

Suggested change

	if args.remote:
	if args.remote:

[CISC]

@pqnet Nice, will merge once you fix linting failure.