New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Correct parameters name in ParameterLoader #2458
Correct parameters name in ParameterLoader #2458
Conversation
ed7603f
to
a2be4e6
Compare
const paramValueText = printParam(parameter) | ||
const paramUnitsText = parameter?.units ? `[${parameter.units}]` : '' | ||
|
||
return paramValueText + paramUnitsText |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return paramValueText + paramUnitsText | |
return paramValueText + ' ' + paramUnitsText |
I don't remember the correct syntax, but the idea is to have something like "10 [A]" instead of "10[A]".
edit: it might be better to just prepend a space on the paramUnitsText, otherwise we would have a space appended to the strings that don't have units.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not against the proposal, but I'd prefer to have a separate column with the units instead of having it appended to each value.
I used this way to keep concise with the already existing implementation in
But is actually easy to change if you guys think its better |
@JoaoMario109 can you rebase over master ? |
* Change to use correct parameters values and units instead of just showing the raw enum value
a2be4e6
to
7ed4ce2
Compare
Change to use correct parameters values and units instead of just showing the raw enumeration value
Closes #2447