Very tiny issue.
I think the phrasing is a little confusing. In particular the "unfix" part, because the default numeraire happens specifically when no variable is fixed.
It would also be great to destinguish between reporting the fixed/numeraire values as set, and reporting the variable used when no variable is fixed.
I also would go with "numeraire" over "normalization", or succinctly use both for clarity.
I might also add the 'set_silent(model) option.
current version
"\n\nDefault price normalization using income for $consumer - This value is fixed to $(value(consumer)).\n"*
"Unfix with unfix($consumer)."
I should probably just do this myself, and I will if @mitchphillipson don't get to it first.
EliLazarus
commented
1 month ago
Very tiny issue. I think the phrasing is a little confusing. In particular the "unfix" part, because the default numeraire happens specifically when no variable is fixed. It would also be great to destinguish between reporting the fixed/numeraire values as set, and reporting the variable used when no variable is fixed. I also would go with "numeraire" over "normalization", or succinctly use both for clarity. I might also add the 'set_silent(model) option.
current version "\n\nDefault price normalization using income for $consumer - This value is fixed to $(value(consumer)).\n"* "Unfix with unfix($consumer)."
I should probably just do this myself, and I will if @mitchphillipson don't get to it first.