Skip to content

Conversation

@Devin-Crawford
Copy link
Contributor

@Devin-Crawford Devin-Crawford commented Feb 6, 2026

Description

  • Fix a defect when trying to use a local copy of example-data.
  • Move a leftover print statement for debugging into the logger.

Issue linked

#7202

Checklist

  • I have tested my changes locally.
  • I have added necessary documentation or updated existing documentation.
  • I have followed the coding style guidelines of this project.
  • I have added appropriate tests (unit, integration, system).
  • I have reviewed my changes before submitting this pull request.
  • I have linked the issue or issues that are solved by the PR if any.
  • I have agreed with the Contributor License Agreement (CLA).

- Fix #7202 (incorrect handling of file)
- Remove incorrectly handled debug statements.
- Add unit test when using local repo to test example data.
@Devin-Crawford Devin-Crawford requested a review from a team as a code owner February 6, 2026 17:50
@github-actions github-actions bot added the bug Something isn't working label Feb 6, 2026
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 21.14%. Comparing base (3592774) to head (319eceb).

❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (85.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #7212       +/-   ##
===========================================
- Coverage   83.47%   21.14%   -62.33%     
===========================================
  Files         254      253        -1     
  Lines       76596    76517       -79     
===========================================
- Hits        63938    16180    -47758     
- Misses      12658    60337    +47679     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

 cleans up its own directory after teardown, preventing race conditions when pytest-xdist workers tear down module-scoped file_tmp_root while function-scoped fixtures still need the directory.
@github-actions github-actions bot added the testing Anything related to testing label Feb 7, 2026
@Devin-Crawford Devin-Crawford mentioned this pull request Feb 7, 2026
2 tasks
@Devin-Crawford Devin-Crawford force-pushed the fix/local_example_download branch from 3c4c529 to dd53d4e Compare February 7, 2026 20:33
- Add while loop in test_analyze.py test_3dl_analyze_setup()
@Devin-Crawford Devin-Crawford marked this pull request as draft February 8, 2026 19:10
@Devin-Crawford Devin-Crawford marked this pull request as ready for review February 8, 2026 19:27
@Devin-Crawford Devin-Crawford marked this pull request as draft February 10, 2026 12:01
Devin-Crawford and others added 2 commits February 10, 2026 09:24
Co-authored-by: Sébastien Morais <146729917+SMoraisAnsys@users.noreply.github.com>
@Devin-Crawford Devin-Crawford marked this pull request as ready for review February 10, 2026 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working testing Anything related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug located in CI/CD Bug located in downloads.py

2 participants