openclimatefix / PVNet

PVnet main repo
MIT License
15 stars 3 forks source link

Force using CPU to get e2e test working locally with macs/mps #169

Closed Sukh-P closed 3 months ago

Sukh-P commented 3 months ago

Pull Request

Description

When running the PVNet e2e test locally on a mac it fails with a MPS tensor datatype error, this PR forces the CPU to be used to avoid this issue (means the local test mirrors what runs in Github/through the CI)

How Has This Been Tested?

By running tests locally and checking the local error was fixed

Checklist:

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 61.39%. Comparing base (f1534d8) to head (1e8ab1d).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #169 +/- ## ======================================= Coverage 61.39% 61.39% ======================================= Files 28 28 Lines 1790 1790 ======================================= Hits 1099 1099 Misses 691 691 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.