Skip to content

Conversation

@PabloRoque
Copy link
Contributor

@PabloRoque PabloRoque commented Jul 13, 2025

Description

Minor cleanup of test_Elemwise. Carryover from #1535.

  1. Removes a constant exec param since it never changes
  2. Introduce ids to improve test readability
  3. Removes a dangling context manager

Related Issue

Checklist

Type of change

  • New feature / enhancement
  • Bug fix
  • Documentation
  • Maintenance
  • Other (please specify):

📚 Documentation preview 📚: https://pytensor--1536.org.readthedocs.build/en/1536/

@PabloRoque PabloRoque marked this pull request as ready for review July 13, 2025 15:00
@PabloRoque PabloRoque changed the title Clean test_Elemwise Clean test_Elemwise Jul 13, 2025
@codecov
Copy link

codecov bot commented Jul 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.86%. Comparing base (7efd1c5) to head (dd2c41c).
⚠️ Report is 124 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@ Coverage Diff @@ ## main #1536 +/- ## ======================================= Coverage 81.86% 81.86% ======================================= Files 230 230 Lines 52508 52508 Branches 9339 9339 ======================================= Hits 42987 42987 Misses 7089 7089 Partials 2432 2432 
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
Copy link
Member

@ricardoV94 ricardoV94 left a comment

Choose a reason for hiding this comment

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

Thanks

@ricardoV94 ricardoV94 merged commit 21218d7 into pymc-devs:main Jul 13, 2025
74 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment