search

US-GHG-Center / veda-config-ghg

Veda config for GHG
https://ghg-demo.netlify.app
Other
3 stars 15 forks source link

Set default metrics for all datasets #229

Closed SwordSaintLancelot closed 8 months ago

SwordSaintLancelot commented 8 months ago

Why are you creating this Pull Request?

The default metric has been added to all the layers of every dataset for the US GHG center.

netlify[bot] commented 8 months ago

Deploy Preview for ghg-demo ready!

Name Link
Latest commit a76f0201552dc24bb84c0621e4f072a28b940049
Latest deploy log https://app.netlify.com/sites/ghg-demo/deploys/655d21d9fad85d0009d5206e
Deploy Preview https://deploy-preview-229--ghg-demo.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

SwordSaintLancelot commented 8 months ago

@j08lue I did look into the logs and it mentioned that the issue is with duplicate mapping keys for all the dataset files. I checked the indentation and it seems to be correct (at least similar to the example that you did) I removed the analysis key from a the datasets that showed the error.

slesaad commented 8 months ago

@SwordSaintLancelot don't add exclude to all the datasets, only to those that already had it

SwordSaintLancelot commented 8 months ago

Also don't push .veda/ui changes?

Reverted them to their original state.