Skip to content
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

add project specific abatement potential and weighted cost #250

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

elpamart
Copy link
Contributor

This pull request includes updates to:

  • the data/notebooks/High_level_overview.ipynb file to correct the country name from "Caribbean" to "The Bahamas", add country size information to the dataset, add the country abatement potential, the project abatement potential, weighted total cost npv and weighted total cost to the calculations.
  • the update to the data_ingestion_WIP.xlsm to rename the "abatement_potential" column to "country_abatement_potential", and the addition of the "project_abatement_potential", "country_size_ha" (used for the weighted cost), and the "weighted_total_cost_npv" and "weighted_total_cost".

These new data will be used for updating the overview page as follow:

  • the map view should show the average by country of the "country_abatement_potential" vs the average "weighted_total_cost_npv" or "weighted_total_cost".
  • the table view should show the "project_abatement_potential" instead of the "country_abatement_potential".

@elpamart elpamart added the data label Jan 24, 2025
@elpamart elpamart requested a review from alexeh January 24, 2025 11:51
@elpamart elpamart self-assigned this Jan 24, 2025
Copy link

vercel bot commented Jan 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tnc-blue-carbon-cost-tool-ghps ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 24, 2025 11:51am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant