Skip to content

Fix failing windows test#169

Merged
jayqi merged 1 commit intomainfrom
168-failing-test
Nov 7, 2025
Merged

Fix failing windows test#169
jayqi merged 1 commit intomainfrom
168-failing-test

Conversation

@ejm714
Copy link
Collaborator

@ejm714 ejm714 commented Nov 6, 2025

Use non-interactive backend in CI for Matplotlib to resolve Windows error

FAILED tests/test_evaluate.py::test_calculate_all_and_save - _tkinter.TclError: Can't find a usable init.tcl in the following directories: 
    {C:\hostedtoolcache\windows\Python\3.10.11\x64\tcl\tcl8.6}

C:/hostedtoolcache/windows/Python/3.10.11/x64/tcl/tcl8.6/init.tcl: couldn't read file "C:/hostedtoolcache/windows/Python/3.10.11/x64/tcl/tcl8.6/init.tcl": No error
couldn't read file "C:/hostedtoolcache/windows/Python/3.10.11/x64/tcl/tcl8.6/init.tcl": No error
    while executing
"source C:/hostedtoolcache/windows/Python/3.10.11/x64/tcl/tcl8.6/init.tcl"
    ("uplevel" body line 1)
    invoked from within
"uplevel #0 [list source $tclfile]"


This probably means that Tcl wasn't installed properly.

Closes #168

@ejm714 ejm714 temporarily deployed to 168-failing-test - CyFi PR #169 November 6, 2025 23:44 — with Render Destroyed
@codecov
Copy link

codecov bot commented Nov 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.6%. Comparing base (768b622) to head (41ebca4).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##            main    #169     +/-   ##
=======================================
- Coverage   79.3%   74.6%   -4.7%     
=======================================
  Files         12      12             
  Lines        875     875             
=======================================
- Hits         694     653     -41     
- Misses       181     222     +41     

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ejm714 ejm714 requested a review from jayqi November 6, 2025 23:53
@jayqi
Copy link
Member

jayqi commented Nov 7, 2025

@jayqi jayqi merged commit 457ea6a into main Nov 7, 2025
10 checks passed
@jayqi jayqi deleted the 168-failing-test branch November 7, 2025 01:36
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.

Failed build: tests

2 participants