ci/Add Windows pipeline

[Luni-4]

Pull Request Template

Checklist

• Confirm that run-checks script has been executed.

Related Issues/PRs

Provide links to relevant issues and dependent PRs.

Changes

This PR adds Windows CI to burn

Testing

Describe how these changes have been tested.

[Luni-4]

@nathanielsimard

The Windows CI error is related to a part of burn's code I do not know. It seems there is a missing files problem https://github.com/burn-rs/burn/actions/runs/6735375454/job/18308404823#step:9:2067

If I'm not wrong we can fix the problem using https://doc.rust-lang.org/std/env/fn.temp_dir.html

[nathanielsimard]

@nathanielsimard

The Windows CI error is related to a part of burn's code I do not know. It seems there is a missing files problem https://github.com/burn-rs/burn/actions/runs/6735375454/job/18308404823#step:9:2067

If I'm not wrong we can fix the problem using https://doc.rust-lang.org/std/env/fn.temp_dir.html

Yeah, we should probably update the tests before merging this PR, using temp_dir is better than actually targeting the directory /tmp which may not exist on Windows.

[antimora]

Saw the hack for windows. Please add the context and when this hack should be removed. Preferably we should have an issue or tondo fix the hack. The commit messages are invisible after a merge so we can't rely on them.

[Luni-4]

Saw the hack for windows. Please add the context and when this hack should be removed. Preferably we should have an issue or tondo fix the hack. The commit messages are invisible after a merge so we can't rely on them.

Thanks @antimora for your review! Yep, I will open an issue as soon as the CI for Windows will be finished. For now I'm just investigating which issues will be arising from the execution of these CI runs

[Luni-4]

I cannot detect where the problem is in the burn-wgpu crate, as you can see from the commits histroy I have tried different approaches, but I always obtain the same error: No adapter found for graphics API AutoGraphicsApi

My attempts:

1. Using the same CI for Windows implemented in wgpu project, but it failed
2. Update wgpu version to 0.18
3. Tried to set up only the WGPU_BACKEND: dx12 environment variable

Perhaps the main problem could be in the burn code, or in the wgpu-hal crate which comunicates directly with the device. I do not know. I do not have a Windows machine ready and usable to test burn code, so I can only use this CI machine. If someone is more expert than me with Windows and is willing to help me, I'm saying thanks in advance!

[syl20bnr]

Going to look at this one.

[Luni-4]

@nathanielsimard
Approved for me

[nathanielsimard]

LGTM 🙏