Allow node params on links#736
Conversation
| ): | ||
| self._raise_error( | ||
| self._raise_warning( | ||
| "Cannot define transmission technology data over the `nodes` dimension" |
There was a problem hiding this comment.
Shall we change the message to something like this?
"Trying to define transmission technology data over the nodes dimension. Be sure that you know what you are doing."
There was a problem hiding this comment.
We probably want a more descriptive message, stating the possible error that this might create.
|
Thanks for the contribution @jnnr. My only concern is that this was in there for a reason. If you define transmission data at a node in YAML, what happens with this change? It would require some tests to check for what happens. |
|
Thanks for your comments, @brynpickering and @irm-codebase. Apparently, we need to first better understand how link data is treated in preprocessing to understand the scenarios of failure we are opening up. I will take some time with to study this with @irm-codebase at the whiteboard and come back to this soon. |
|
Any progress on identifying the link @jnnr ? |
Fixes the following issue:
I am not allowed to set a constraint on
flow_outof transmission links. I will get this error:I am trying to provide the model with a data table like this:
And a constraint like this:
Summary of changes in this pull request
I convert the error message into a warning.
Reviewer checklist