Closed doneachh closed 4 months ago
@energyLS i deleted a line in prepare_energy_totals, which dropped all lines with nan values as well to keep the keys. But in the next line na values get replaced by 0. Considering whether thats ok or to delete the line with replacing na by 0s as well. What do you think? :)
@energyLS i deleted a line in prepare_energy_totals, which dropped all lines with nan values as well to keep the keys. But in the next line na values get replaced by 0. Considering whether thats ok or to delete the line with replacing na by 0s as well. What do you think? :)
@doneachh so basically we get zeros everywhere where you have introduced nans?
@energyLS i deleted a line in prepare_energy_totals, which dropped all lines with nan values as well to keep the keys. But in the next line na values get replaced by 0. Considering whether thats ok or to delete the line with replacing na by 0s as well. What do you think? :)
@doneachh so basically we get zeros everywhere where you have introduced nans?
Yes
Closes # (if applicable).
Changes proposed in this Pull Request
When no sector data is given for a country, we write nan values into base_energy_totals.csv instead of ignoring it.
Checklist
envs/environment.yaml
andenvs/environment.docs.yaml
.config.default.yaml
,config.tutorial.yaml
, andtest/config.test1.yaml
.doc/configtables/*.csv
and line references are adjusted indoc/configuration.rst
anddoc/tutorial.rst
.doc/release_notes.rst
is amended in the format of previous release notes, including reference to the requested PR.