CI Fix Windows permission error on merge test

[BenjaminBossan]

For some BS reason, Windows CI suddenly started throwing permission errors on test_merge_layers. These errors occur when using the TempDirectory() context manager, which raises a PermissionError on Windows when it tries to clean up after itself. Therefore, this context manager is now avoided in favor of manual clean up.

More context:

I investigated this issue first in #1947. My suspicion that this could be caused by a new pytest version was not confirmed. Maybe the reason is that GH rolled out a new Windows worker, not sure.

Also note that this is not the first time that this workaround is required, e.g. also here:

peft/tests/test_custom_models.py

Lines 1448 to 1467 in e6cd24c

	# note: not using the context manager here because it fails on Windows CI for some reason
	tmp_dirname = tempfile.mkdtemp()
	try:
	model.save_pretrained(tmp_dirname)
	model = ModelEmbConv1D(emb_size=105)
	# first check that this raises
	with pytest.raises(RuntimeError) as exc:
	PeftModel.from_pretrained(model, tmp_dirname)
	msg = exc.value.args[0]
	assert "size mismatch" in msg and "100" in msg and "105" in msg
	# does not raise
	PeftModel.from_pretrained(model, tmp_dirname, ignore_mismatched_sizes=True)
	finally:
	try:
	shutil.rmtree(tmp_dirname)
	except PermissionError:
	# windows error
	pass

[HuggingFaceDocBuilderDev]

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

[sayakpaul]

Windows :(