Skip to content

Conversation

@ballPointPenguin
Copy link
Member

No description provided.

@jucor jucor requested a review from Copilot November 27, 2025 09:24
Copilot finished reviewing on behalf of jucor November 27, 2025 09:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR optimizes the Python CI workflow to run only when Delphi-related files change and improves coverage comment management by updating existing comments instead of creating new ones.

Key Changes:

  • Added path filters to trigger the workflow only for changes to Delphi Python files, requirements, Dockerfile, or the workflow itself
  • Modified coverage reporting to update existing comments instead of creating duplicates

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@jucor jucor left a comment

Choose a reason for hiding this comment

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

Awesome, thanks for this. Just one minute change, let me know if you want me to edit the branch to factor it in?

@github-actions
Copy link

github-actions bot commented Dec 1, 2025

Delphi Coverage Report

File Stmts Miss Cover
init.py 3 0 100%
main.py 55 55 0%
components/init.py 2 0 100%
components/config.py 165 133 19%
components/server.py 116 72 38%
conversation/init.py 2 0 100%
conversation/conversation.py 1031 363 65%
conversation/manager.py 131 42 68%
database/init.py 1 0 100%
database/dynamodb.py 387 234 40%
database/postgres.py 298 199 33%
pca_kmeans_rep/init.py 5 0 100%
pca_kmeans_rep/clusters.py 234 8 97%
pca_kmeans_rep/corr.py 98 17 83%
pca_kmeans_rep/pca.py 266 71 73%
pca_kmeans_rep/repness.py 273 32 88%
pca_kmeans_rep/stats.py 107 22 79%
poller.py 224 188 16%
regression/init.py 4 0 100%
regression/comparer.py 399 175 56%
regression/datasets.py 56 19 66%
regression/recorder.py 36 27 25%
regression/utils.py 140 38 73%
run_math_pipeline.py 258 238 8%
system.py 85 55 35%
utils/general.py 62 40 35%
Total 4438 2028 54%

@ballPointPenguin ballPointPenguin merged commit 1ce8660 into edge Dec 1, 2025
4 checks passed
@ballPointPenguin ballPointPenguin deleted the br/python-ci branch December 1, 2025 21:29
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.

4 participants