-
Notifications
You must be signed in to change notification settings - Fork 16
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
Set default metrics for all datasets #229
Set default metrics for all datasets #229
Conversation
✅ Deploy Preview for ghg-demo ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SwordSaintLancelot there seem to be indentation errors that caused the preview build to fail. See the CI job logs for details, please.
@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) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also don't push .veda/ui changes?
@SwordSaintLancelot don't add exclude to all the datasets, only to those that already had it |
Reverted them to their original state. |
package.json
Outdated
@@ -1,7 +1,7 @@ | |||
{ | |||
"name": "veda-config", | |||
"description": "Configuration for Veda", | |||
"version": "1.0.0-beta.2", | |||
"version": "0.1.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this branch here should be rebased to current develop
or something? Where you did you branch off from, @SwordSaintLancelot?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did branch it from develop. I will rebase it and push again. Also, this package was updated when I tried to view the changes in local netlify using the command yarn serve.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. I was just worried about the "reverting" commits above. If these changes are already in develop
by now and meant to be there, your commits here would undo them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there were a few changes that happened when I ran the yarn serve and that's what modified the /.veda/ui folder along with package.json but I changed those to original package.json which was in the develop branch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, great work getting this ready!
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.