Skip to content

add action yml to populate issue#7807

Merged
williambdean merged 1 commit into
pymc-devs:mainfrom
williambdean:add-slow-test-yaml
Feb 4, 2026
Merged

add action yml to populate issue#7807
williambdean merged 1 commit into
pymc-devs:mainfrom
williambdean:add-slow-test-yaml

Conversation

@williambdean
Copy link
Copy Markdown
Contributor

@williambdean williambdean commented Jun 3, 2025

Description

Adding the forgotten yml workflow file to populate the issue here: #7686

CC: @ricardoV94

Related Issue

  • Closes #
  • Related to #

Checklist

Type of change

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

📚 Documentation preview 📚: https://pymc--7807.org.readthedocs.build/en/7807/

Comment on lines +18 to +19
- name: Checkout code
uses: actions/checkout@v4

Check warning

Code scanning / zizmor

credential persistence through GitHub Actions artifacts Warning

credential persistence through GitHub Actions artifacts
@williambdean williambdean mentioned this pull request Jun 3, 2025
11 tasks
@williambdean
Copy link
Copy Markdown
Contributor Author

Good to go?

@ricardoV94 ricardoV94 requested a review from maresb June 16, 2025 20:14
@williambdean williambdean removed the request for review from maresb February 3, 2026 17:58
@williambdean
Copy link
Copy Markdown
Contributor Author

Friendly ping @maresb

@williambdean williambdean requested a review from maresb February 3, 2026 17:59
Copy link
Copy Markdown
Contributor

@maresb maresb left a comment

Choose a reason for hiding this comment

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

I would have just used pytest-split, which defines pytest --store-durations

But this is a really interesting solution! I'm really intrigued by your idea of collecting the timings in a GH Issue! It solves the DB problem, at least for now.

I suspect you'll hit a character limit at some point.

I have a billion things going on at the moment, but I'm hoping to set up a server sometime soon to receive telemetry like this. As I've mentioned, I'm really interested in using PyMC to do inference on the test times. 😉

@williambdean
Copy link
Copy Markdown
Contributor Author

I would have just used pytest-split, which defines pytest --store-durations

Yes, definitely some pros there. We were having that the test_durations didn't stay synced with changing repo and / or the test_durations didn't reflect the times in the CI/CD runs (large imbalance of run times still)

This is more manual and gets the github issue interface.

Related to pymc-labs/pymc-marketing#2249

But this is a really interesting solution! I'm really intrigued by your idea of collecting the timings in a GH Issue! It solves the DB problem, at least for now.

I suspect you'll hit a character limit at some point.

Possible, I have looked through many of the logs and I have not hit that yet.

I have a billion things going on at the moment, but I'm hoping to set up a server sometime soon to receive telemetry like this. As I've mentioned, I'm really interested in using PyMC to do inference on the test times. 😉

That'd be cool

@williambdean
Copy link
Copy Markdown
Contributor Author

@williambdean williambdean merged commit 24e9f49 into pymc-devs:main Feb 4, 2026
14 checks passed
@williambdean williambdean deleted the add-slow-test-yaml branch February 4, 2026 11:59
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.

3 participants