Skip to content

Rename output name fix#358

Merged
mathieudpnt merged 5 commits intoProject-OSmOSE:mainfrom
Gautzilla:rename-output-name-fix
Mar 26, 2026
Merged

Rename output name fix#358
mathieudpnt merged 5 commits intoProject-OSmOSE:mainfrom
Gautzilla:rename-output-name-fix

Conversation

@Gautzilla
Copy link
Contributor

Reverts #356

after all, I think the Project.rename_output() method name is the thing that is misleading.

Here's what I've done instead:

I guess this is a bit less confusing than the previous version, where output sometimes referred to a sole output dataset and sometimes to the whole run transform.

@Gautzilla Gautzilla self-assigned this Mar 26, 2026
@coveralls
Copy link
Collaborator

Coverage Status

coverage: 98.806%. remained the same
when pulling 3f904fb on Gautzilla:rename-output-name-fix
into 452b23c on Project-OSmOSE:main.

@Gautzilla Gautzilla marked this pull request as ready for review March 26, 2026 11:08
@mathieudpnt mathieudpnt merged commit 5befe2b into Project-OSmOSE:main Mar 26, 2026
2 checks passed
@Gautzilla Gautzilla deleted the rename-output-name-fix branch March 26, 2026 15:51
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