Expose information about task-group usage to users by ychiucco · Pull Request #3383 · fractal-analytics-platform/fractal-server

ychiucco · 2026-06-30T12:54:54Z

Checklist before merging into `main`

I merged main into the current branch.
I added an appropriate entry to CHANGELOG.md
All GitHub checks completed successfully, or there are failures which are clearly unrelated to this Pull Request and I opened an issue about it.
The reported "PR coverage" is 100%, or we briefly discussed why this is not the case.
I added logging to new code - if appropriate.
I added type hints to new code - if appropriate - and maybe I ran this command to spot some naive errors:

ty check $(git diff main --name-only ./fractal_server)

github-actions · 2026-06-30T12:59:36Z

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)
fractal_server/app/routes/admin/v2
task_group.py	$This PR doesn't change the number of statements in fractal_server/app/routes/admin/v2/task_group.py, which is 126.$	$This PR doesn't change the number of statements missing coverage in fractal_server/app/routes/admin/v2/task_group.py, which is 0.$	$This PR doesn't change the coverage rate in fractal_server/app/routes/admin/v2/task_group.py, which is 100% (126/126).$	$In this PR, 14 new statements are added to fractal_server/app/routes/admin/v2/task_group.py, 14 of which are covered (100%).$
fractal_server/app/routes/api/v2
_aux_task_group_disambiguation.py	$This PR adds 5 to the number of statements in fractal_server/app/routes/api/v2/_aux_task_group_disambiguation.py, taking it from 54 to 59.$	$This PR doesn't change the number of statements missing coverage in fractal_server/app/routes/api/v2/_aux_task_group_disambiguation.py, which is 3.$	$This PR removes 0.33 percentage points from the coverage rate in fractal_server/app/routes/api/v2/_aux_task_group_disambiguation.py, taking it from 91.37% (51/54) to 91.04% (56/59).$	$In this PR, 11 new statements are added to fractal_server/app/routes/api/v2/_aux_task_group_disambiguation.py, 11 of which are covered (100%).$
task_group.py	$This PR adds 4 to the number of statements in fractal_server/app/routes/api/v2/task_group.py, taking it from 99 to 103.$	$This PR doesn't change the number of statements missing coverage in fractal_server/app/routes/api/v2/task_group.py, which is 0.$	$This PR adds 0.02 percentage points to the coverage rate in fractal_server/app/routes/api/v2/task_group.py, taking it from 99.2% (99/99) to 99.22% (103/103).$	$In this PR, 20 new statements are added to fractal_server/app/routes/api/v2/task_group.py, 20 of which are covered (100%).$
fractal_server/app/schemas/v2
task_group.py	$This PR doesn't change the number of statements in fractal_server/app/schemas/v2/task_group.py, which is 72.$	$This PR doesn't change the number of statements missing coverage in fractal_server/app/schemas/v2/task_group.py, which is 0.$	$This PR doesn't change the coverage rate in fractal_server/app/schemas/v2/task_group.py, which is 100% (72/72).$	$In this PR, 1 new statements are added to fractal_server/app/schemas/v2/task_group.py, 1 of which are covered (100%).$
Project Total

_{This report was generated by python-coverage-comment-action}

This reverts commit 6103a31.

…piv2task-group' into 3297-expose-information-about-task-group-usage-to-users

…ge-to-users

…e-for-get-apiv2task-group

…piv2task-group' into 3297-expose-information-about-task-group-usage-to-users

…ge-to-users

[skip ci]

add in_use

b09180f

ychiucco linked an issue Jun 30, 2026 that may be closed by this pull request

Expose information about task-group usage to users #3297

Open

This comment was marked as off-topic.

Sign in to view

tcompa reviewed Jun 30, 2026

View reviewed changes

Comment thread fractal_server/app/routes/api/v2/task_group.py Outdated

ychiucco added 6 commits June 30, 2026 16:08

remove slim from GET task-group

70bf8d7

fix test api

39c9e1d

changelog

0960771

try without inner model dump

6103a31

Revert "try without inner model dump"

e2fb07a

This reverts commit 6103a31.

Merge branch '3384-review-whether-any-client-uses-slimfalse-for-get-a…

eacc1b3

…piv2task-group' into 3297-expose-information-about-task-group-usage-to-users

ychiucco changed the base branch from main to 3384-review-whether-any-client-uses-slimfalse-for-get-apiv2task-group July 1, 2026 07:33

ychiucco added 6 commits July 1, 2026 09:34

unify read schemas

2f52ef8

remove import

ecb1ae5

Merge branch 'main' into 3297-expose-information-about-task-group-usa…

eb23e49

…ge-to-users

Merge branch 'main' into 3384-review-whether-any-client-uses-slimfals…

1adba89

…e-for-get-apiv2task-group

Merge branch '3384-review-whether-any-client-uses-slimfalse-for-get-a…

4b29d75

…piv2task-group' into 3297-expose-information-about-task-group-usage-to-users

Merge branch 'main' into 3297-expose-information-about-task-group-usa…

ee7b99f

…ge-to-users

ychiucco changed the base branch from 3384-review-whether-any-client-uses-slimfalse-for-get-apiv2task-group to main July 2, 2026 07:04

ychiucco added 2 commits July 2, 2026 09:19

query on taskgroups

303d0d6

changelog

ff00f20

[skip ci]

ychiucco marked this pull request as ready for review July 2, 2026 07:26

ychiucco added 3 commits July 2, 2026 09:43

remove distinct

affcb83

change aux function serialize_task_group

d41e69a

remove in-line type hint

ff80a1e

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Expose information about task-group usage to users#3383

Expose information about task-group usage to users#3383
ychiucco wants to merge 18 commits into
mainfrom
3297-expose-information-about-task-group-usage-to-users

ychiucco commented Jun 30, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Jun 30, 2026 •

edited

Loading

Uh oh!

This comment was marked as off-topic.

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

ychiucco commented Jun 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Checklist before merging into main

Uh oh!

github-actions Bot commented Jun 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage report

Uh oh!

This comment was marked as off-topic.

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ychiucco commented Jun 30, 2026 •

edited

Loading

Checklist before merging into `main`

github-actions Bot commented Jun 30, 2026 •

edited

Loading