[torchlib] [ci] Fix failing external data tests for Windows

[shubhambhokare1]

Fix failing external data tests for Windows due to file permission errors

[codecov]

❌ 22 Tests Failed:

Tests completed	Failed	Passed	Skipped
12052	22	12030	2021

View the top 3 failed tests by shortest run time

onnxscript.converter_test.TestConverter test_eager_op

Stack Traces | 0.001s run time

No failure message available

onnxscript.converter_test.TestConverter test_loops_break

Stack Traces | 0.001s run time

No failure message available

onnxscript.converter_test.TestConverter test_loops_while

Stack Traces | 0.001s run time

No failure message available

To view individual test run time comparison to the main branch, go to the Test Analytics Dashboard

[github-actions]

Test Results

     24 files  ±    0      24 suites  ±0   2h 46m 17s ⏱️ - 1m 4s
 14 073 tests  - 2 860  11 699 ✅  - 2 857    2 338 💤  - 1   32 ❌ ± 0   4 🔥  -  2 
354 978 runs  +    7  81 137 ✅ +   71  273 651 💤 ±0  162 ❌  - 50  28 🔥  - 14 

For more details on these failures and errors, see this check.

Results for commit 9b3e82e. ± Comparison against base commit 684b5b5.

This pull request removes 2861 and adds 1 tests. Note that renamed tests count towards both.

tests.ir.serde_roundtrip_test.SerdeTest ‑ test_serialization_deserialization_produces_same_model_0001_attn_stable_diffusion_unet_0_attn_stable_diffusion_unet_0_onnx
tests.ir.serde_roundtrip_test.SerdeTest ‑ test_serialization_deserialization_produces_same_model_0002_ln_llama2_2_ln_llama2_2_onnx
tests.ir.serde_roundtrip_test.SerdeTest ‑ test_serialization_deserialization_produces_same_model_0003_attn_stable_diffusion_unet_without_encoder_hidden_states_1_attn_stable_diffusion_unet_without_encoder_hidden_states_1_onnx
tests.ir.serde_roundtrip_test.SerdeTest ‑ test_serialization_deserialization_produces_same_model_0004_geglu_stable_diffusion_unet_3_geglu_stable_diffusion_unet_3_onnx
tests.ir.serde_roundtrip_test.SerdeTest ‑ test_serialization_deserialization_produces_same_model_0005_geglu_stable_diffusion_unet_0_geglu_stable_diffusion_unet_0_onnx
tests.ir.serde_roundtrip_test.SerdeTest ‑ test_serialization_deserialization_produces_same_model_0007_attn_phi_1_5_0_attn_phi_1_5_0_onnx
tests.ir.serde_roundtrip_test.SerdeTest ‑ test_serialization_deserialization_produces_same_model_0008_geglu_stable_diffusion_unet_2_geglu_stable_diffusion_unet_2_onnx
tests.ir.serde_roundtrip_test.SerdeTest ‑ test_serialization_deserialization_produces_same_model_0009_gelu_phi_1_5_0_gelu_phi_1_5_0_onnx
tests.ir.serde_roundtrip_test.SerdeTest ‑ test_serialization_deserialization_produces_same_model_0010_attn_phi_1_5_1_attn_phi_1_5_1_onnx
tests.ir.serde_roundtrip_test.SerdeTest ‑ test_serialization_deserialization_produces_same_model_0011_gelu_phi_1_5_3_gelu_phi_1_5_3_onnx
…

onnxscript.ir._core_test.ExternalTensorTest ‑ test_release_does_not_invalidate_tensor

♻️ This comment has been updated with latest results.

[justinchuby]

Could you add a test to verify that calling numpy() etc. on a released external tensor will still work (e.g. we can release and load again at will)? Thanks!

[justinchuby]

Thank you!

[justinchuby]

Windows crashes: Windows fatal exception: access violation

https://github.com/microsoft/onnxscript/actions/runs/10693258903/job/29643192895?pr=1834

[shubhambhokare1]

Windows crashes: Windows fatal exception: access violation

https://github.com/microsoft/onnxscript/actions/runs/10693258903/job/29643192895?pr=1834

Failures seem to be unrelated.