Conversation
md_template/model.md
Outdated
| import numpy as np | ||
| import matplotlib.pyplot as plt | ||
| xmin = np.linspace(0.01, 3, 30) | ||
| # TODO changing 0.01 to 0 above results in an error -- suspected GPkit bug |
There was a problem hiding this comment.
You have to remove_unbounded_zeros to use a <= 0 constraint.
|
responded to code review. |
| with open("gp.generated.tex", "w") as f: | ||
| f.write(m._repr_latex_()) | ||
| # TODO wish model name didn't get printed in the above | ||
| f.write("$%s$" % m.latex(excluded=["models", "units"])) |
There was a problem hiding this comment.
@bqpd note the need for dollar signs here
|
Looks good to me! Do we want any tests? |
|
@whoburg, I think this is ready to merge? |
|
@mjburton11, you've been using markdown a lot. Do you see any improvements that could be made to this template before we merge? (e.g. maybe your Let's all start new projects from this template and hopefully thereby make it better over time. |
|
Just added |
| \begin{figure}[h!] | ||
| \label{fig:sweep} | ||
| \begin{center} | ||
| \includegraphics{simple_sweep.pdf} |
|
@mjburton11 is tackling convexengineering/gpkit#890, and then this should be good to merge |
3 participants
@bqpd, please review and add additional commits if you see ways to improve