Skip to content

Windows Compatibility - working on linux - #170

Merged
nhsavage merged 6 commits into
mainfrom
Windows
Jul 24, 2026
Merged

Windows Compatibility - working on linux#170
nhsavage merged 6 commits into
mainfrom
Windows

Conversation

@Eleanor-Dean

Copy link
Copy Markdown
Contributor

Windows compatibility -->
Removes ":" and swaps for "."
creates a windows lock file (not currently met office compatible)
adds a OS finder in the shell and deals with pycpt as needed with that.
Adds a environment directory for ease

Before Merge:
This change was done on linux so needs a test on Windows. --> assigned to me

Post Merge:
instructions need updating as needed

@Eleanor-Dean Eleanor-Dean self-assigned this Jul 23, 2026
@Eleanor-Dean
Eleanor-Dean requested a review from nhsavage July 24, 2026 10:57
@Eleanor-Dean

Copy link
Copy Markdown
Contributor Author

Lock file is still not MO compatiable.

However if used windows run is working inc. pycpt.

micro instructions:
 Donwload miniforgge
 Download gitbash
 Add ~./bashrc file with: ". "/c/Users/<username>/miniforge3/etc/profile.d/conda.sh"
conda activate base"
 Download zip of osop or clone
 Unzip (if needed)
 Move to env dir of osop
 Install environment with conda env create –name osop -f osop-win-64-lock.txt
 Go to home dir i.e. c:users/username
 Nano .cdsapirc
 Add creds
 Pip install cdsapi
 Grab a test download from cds climate store
 Save as test.py
 Python test.py
 Check working
 Export SCRATCH = “$HOME/scratch”
 Mkdir -p “$SCRATCH”
 ./run_all.sh -t

@nhsavage
nhsavage marked this pull request as ready for review July 24, 2026 13:21
@github-actions

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
lib/osop
   compare_terciles.py76760%9–336
   compute_products_func.py1341340%8–449
   compute_scores_func.py1541540%17–414
   ens_plotting.py1425462%112–148, 240, 250, 318–337, 357–418
   plot_verify.py17314516%89–126, 158–249, 276–309, 339–434, 458–535
   pycpt_convert.py39428727%76–99, 116–122, 140, 163–169, 189–205, 219–225, 242–270, 290, 311–313, 340–531, 554, 569–571, 663, 736–744, 759–765, 776–791, 796–799, 804–808, 816–827, 843–897, 917–971, 1009–1044
   run_pycpt.py1261260%11–362
   util.py591280%89, 103, 131, 159–172, 179–184
TOTAL157498837% 

Tests Skipped Failures Errors Time
45 0 💤 0 ❌ 0 🔥 13.923s ⏱️

@nhsavage nhsavage left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to add rioxarray to new windows env see #176

@nhsavage

Copy link
Copy Markdown
Contributor

need to add windows runner to github actions runners. See #177

@nhsavage
nhsavage merged commit dc075db into main Jul 24, 2026
4 checks passed
@nhsavage
nhsavage deleted the Windows branch July 24, 2026 14:01
@nhsavage nhsavage linked an issue Jul 24, 2026 that may be closed by this pull request
This was referenced Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create lock file for windows

2 participants