Skip to content

Conversation

@dfkapraun
Copy link
Collaborator

Created an option for writing error and warning messages (from the model translator) to standard output.

@dfkapraun dfkapraun requested a review from Copilot October 20, 2025 16:34
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 adds a verboseOutput option to control where translator messages are displayed. When enabled, error and warning messages from the MCSim model translator are written directly to standard output instead of being saved to temporary files.

Key changes:

  • Added a new verboseOutput parameter to createModel() and compileModel() functions with a default value of FALSE
  • Modified message handling logic in compileModel() to conditionally output translator messages based on the verboseOutput flag
  • Updated documentation across multiple files to describe the new parameter

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
man/createModel.Rd Added documentation for the new verboseOutput parameter
man/compileModel.Rd Added documentation for the new verbose_output parameter
man/Model-class.Rd Added documentation for the verboseOutput field and fixed a capitalization inconsistency
R/createModel.R Added verboseOutput parameter and passed it to the Model constructor
R/compileModel.R Implemented conditional logic to output translator messages based on verbose_output flag
R/MCSim_model.R Added verboseOutput field to the Model class and passed it to compileModel()
DESCRIPTION Incremented version number to 1.0.9000

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

dfkapraun and others added 5 commits October 20, 2025 12:37
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@dfkapraun dfkapraun requested a review from Copilot October 20, 2025 17:08
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

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@dfkapraun dfkapraun requested a review from Copilot November 17, 2025 17:22
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

Copilot reviewed 11 out of 12 changed files in this pull request and generated no new comments.


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

@dfkapraun dfkapraun merged commit 9bcd829 into main Nov 17, 2025
3 checks passed
@dfkapraun dfkapraun deleted the verbose branch November 17, 2025 17:27
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