updated biochar logic as CDR technology#258
Conversation
|
The test expects 7 rows for biochar pyrolysis but the code now produces 9. I also updated the parameters name to the standard e.g. "biomass input" -> "biomass-input" |
|
the output files were added and code detaisl fixed |
|
Thanks for the PR and sorry for keeping you waiting for a review. Can you elaborate what you mean by "as biochar pyrolysis is not considered as CDR technology"? Is your goal maybe to view pyrolysis as a CDR technology and change the units to reflect the costs and inputs per carbon sequestered? |
|
your comment is spot on, it was a typo (now rather than not). I have improve the text explainng the scope and linking it to the pypsa-eur feature. it should also explain why we changed basis for the paramaters and the reason for deviating from standard technology-data unit. however it will be consistent with the pypsa-eur implementation |
Closes # (if applicable).
Changes proposed in this Pull Request
The technology biochar pyrolysis in compile_cost_assumption.py was updated with change of basis for units and additional parameters. The changes is in line with the planned implementation in pypsa-eur PyPSA/pypsa-eur#2141. Biochar pyrolysis will be implemented considered a CDR technology rather than a biomass conversion technology, and all the parameters are expressed per tonne of CO2 sequestered.
Checklist
doc.environment.yaml(if applicable).doc/release_notes.rstof the upcoming release is included.