Skip to content

Deprecate MeanModelWorkflow in favor of OSPSuite.ReportingFramework#1371

Draft
KatrinCoboeken with Copilot wants to merge 5 commits into
Switch-to-OSPSuitePlotsfrom
copilot/add-lifecycle-management-meanmodelworkflow
Draft

Deprecate MeanModelWorkflow in favor of OSPSuite.ReportingFramework#1371
KatrinCoboeken with Copilot wants to merge 5 commits into
Switch-to-OSPSuitePlotsfrom
copilot/add-lifecycle-management-meanmodelworkflow

Conversation

Copilot AI commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

MeanModelWorkflow is being deprecated and replaced by OSPSuite.ReportingFramework. Any attempt to instantiate the class now fails immediately with a clear error message directing users to the new framework.

Changes

  • Deprecated MeanModelWorkflow: Modified the initialize method to throw an error immediately upon instantiation
  • Migration guidance: Error message directs users to OSPSuite.ReportingFramework with a GitHub link
  • Updated test coverage: Tests now verify the deprecation error is thrown correctly
  • Skipped dependent tests: Added skip statements to test files that rely on MeanModelWorkflow functionality

The initialize method now stops execution immediately:

initialize = function(...) {
  stop(
    "MeanModelWorkflow is no longer available. ",
    "Please use OSPSuite.ReportingFramework instead. ",
    "For more information, visit: https://github.com/Open-Systems-Pharmacology/OSPSuite.ReportingFramework",
    call. = FALSE
  )
}

Impact

This is a breaking change. Users can no longer instantiate or use MeanModelWorkflow. All existing workflows using this class must be migrated to OSPSuite.ReportingFramework.

Original prompt

This section details on the original issue you should resolve

<issue_title>Add lifecycle management to MeanModelWorkflow</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits February 3, 2026 16:18
Co-authored-by: KatrinCoboeken <28564329+KatrinCoboeken@users.noreply.github.com>
Co-authored-by: KatrinCoboeken <28564329+KatrinCoboeken@users.noreply.github.com>
Co-authored-by: KatrinCoboeken <28564329+KatrinCoboeken@users.noreply.github.com>
Copilot AI changed the title [WIP] Add lifecycle management to MeanModelWorkflow Add finalize method to MeanModelWorkflow for lifecycle management Feb 3, 2026
Copilot AI requested a review from KatrinCoboeken February 3, 2026 16:24
@KatrinCoboeken

Copy link
Copy Markdown

@copilot Thats not what I wanted. I want to have a stop as soon as possible if somebody tries to start the MeanModelWorkflow, with a message that this is not longer available, and a reference to OSPSuite.ReportingFramework.

Co-authored-by: KatrinCoboeken <28564329+KatrinCoboeken@users.noreply.github.com>

Copilot AI commented Feb 4, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Thats not what I wanted. I want to have a stop as soon as possible if somebody tries to start the MeanModelWorkflow, with a message that this is not longer available, and a reference to OSPSu...

Updated the implementation to deprecate MeanModelWorkflow instead of adding lifecycle management. The initialize method now throws an error immediately with a message directing users to OSPSuite.ReportingFramework.

Changes in commit 2f1b1b9:

  • Modified initialize to throw deprecation error with reference to OSPSuite.ReportingFramework
  • Updated tests to expect the error message
  • Added skip statements to test files that rely on MeanModelWorkflow functionality

Copilot AI changed the title Add finalize method to MeanModelWorkflow for lifecycle management Deprecate MeanModelWorkflow in favor of OSPSuite.ReportingFramework Feb 4, 2026
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.

2 participants