Skip to content

Conversation

@Shane-Stevenson
Copy link
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Example
  • Documentation

Description

Initial PR had an error in the plot title string. This fixes it.

Related Issues & Documents

QA Instructions, Screenshots, Recordings

Build and run this example's binary

Added/updated tests?

_We encourage you to test all code included with MOLE, including examples.

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

Read Contributing Guide and Code of Conduct

[optional] Are there any post deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

Copy link
Collaborator

@jbrzensk jbrzensk left a comment

Choose a reason for hiding this comment

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

While you are at it, can you fix the legend.
Something like line 59 should be:

plot_script << "plot '-' using 1:2 with lines title \"Estimated Solution\" \n";

@Shane-Stevenson
Copy link
Contributor Author

Shane-Stevenson commented Dec 4, 2025

I've fixed the legend, documentation, used arma::vec, and added the exact solution to the graph

@jbrzensk jbrzensk merged commit 01d05dc into csrc-sdsu:main Dec 5, 2025
6 checks passed
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