iiasa / iiasa/message-ix-models
Work around iiasa/ixmp#425
@khaeru is already working on this.
Since Mar 4, 2022.
- Dominant language
- Python
- Stars
- 32
- Forks
- 57
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 5
Description
@Jihoon reports that when calling apply_spec() with a spec that adds unit "" (dimensionless), one runs into iiasa/ixmp#425 (i.e. only when connecting via ixmp JDBCBackend to a Oracle remote database).
For example, this is triggered by message_data.model.transport via entries in its set.yaml config file:
https://github.com/iiasa/message_data/blob/5f572d2a50f69aa90663d2c1aa97d4895ef32b94/data/transport/set.yaml#L130-L135
unit:
add:
"": dimensionless
km: kilometre
...
Since the ixmp bug is in the Java code, it is unlikely to get fixed, and message-ix-models must work around it. The workaround will be for apply_spec() to detect when a unit is "" and skip adding it.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.