-
Notifications
You must be signed in to change notification settings - Fork 0
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
Docs: promoting to production #162
Merged
Merged
Changes from 5 commits
Commits
Show all changes
10 commits
Select commit
Hold shift + click to select a range
20c3deb
POC data promotion form
anayeaye 5a21cb5
docs: add publication process diagram and clean up the descriptions o…
anayeaye fe57986
remove test issue template
anayeaye 38f7b14
add link to dashboard dataset configuration docs
anayeaye 7464ab8
updated but not yet tested geoglam ingest template
anayeaye 35afe1a
readme edits
smohiudd 30411dc
lint
anayeaye 4a3ff89
lint
anayeaye 05a2aee
remove geoglam ingest example; it is now included in veda-docs
anayeaye 2dcc81b
fix bad link
anayeaye File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -76,10 +76,15 @@ | |
"source": [ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should this notebook be moved to veda-docs? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Agree that this should move to veda-docs |
||
"from cognito_client import CognitoClient\n", | ||
"\n", | ||
"STAGING_CLIENT_ID = \"4rhmpnmnk3rgd9qtiuarllppau\"\n", | ||
"STAGING_USERPOOL_ID = \"us-west-2_0G3VRilt1\"\n", | ||
"STAGING_IDENTITY_POOL_ID = \"us-west-2:ad6647b6-b410-4e73-8205-28a066c290fb\"\n", | ||
"\n", | ||
"# Obtain a token from the cognito client \n", | ||
"client = CognitoClient(\n", | ||
" client_id=\"o8c93cebc17upumgstlbqm44f\",\n", | ||
" user_pool_id=\"us-west-2_9mMSsMcxw\",\n", | ||
" identity_pool_id=\"us-west-2:40f39c19-ab88-4d0b-85a3-3bad4eacbfc0\",\n", | ||
" client_id = STAGING_CLIENT_ID,\n", | ||
" user_pool_id = STAGING_USERPOOL_ID,\n", | ||
" identity_pool_id = STAGING_IDENTITY_POOL_ID\n", | ||
")\n", | ||
"_ = client.login()\n", | ||
"\n", | ||
|
@@ -135,7 +140,7 @@ | |
"metadata": {}, | ||
"outputs": [], | ||
"source": [ | ||
"API = \"https://ig9v64uky8.execute-api.us-west-2.amazonaws.com/staging/\"\n", | ||
"STAGING_WORKFLOWS_API = \"https://staging.openveda.cloud/api/workflows\"\n", | ||
"\n", | ||
"LOCAL_FILE_PATH = \"CropMonitor_2023_06_28.tif\"\n", | ||
"YEAR, MONTH = 2023, 6\n", | ||
|
@@ -340,7 +345,7 @@ | |
" \"content-type\": \"application/json\",\n", | ||
" \"accept\": \"application/json\",\n", | ||
"}\n", | ||
"response = requests.post((API + \"dataset/validate\"), json=dataset, headers=headers)\n", | ||
"response = requests.post((STAGING_WORKFLOWS_API + \"dataset/validate\"), json=dataset, headers=headers)\n", | ||
"response.raise_for_status()\n", | ||
"print(response.text)" | ||
] | ||
|
@@ -377,7 +382,7 @@ | |
} | ||
], | ||
"source": [ | ||
"response = requests.post((API + \"dataset/publish\"), json=dataset, headers=headers)\n", | ||
"response = requests.post((STAGING_WORKFLOWS_API + \"dataset/publish\"), json=dataset, headers=headers)\n", | ||
"response.raise_for_status()\n", | ||
"print(response.text)" | ||
] | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
@anayeaye as part of the validation should we also create json schema that includes our minimum requirements for collection metadata to maintain consistency across the production catalog? For example, all collections in VEDA must include:
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.
Ooof I just included those there in an optional section in the updated geoglam notebook that is now runnable in the hub https://github.com/NASA-IMPACT/veda-docs/blob/c9dc33a8f2960fa29a59db2974f979b12055f8ca/contributing/dataset-ingestion/example-template/example-geoglam-ingest.ipynb
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 don't disagree on a firm nudge to create renders and thumbnails but I don't think we should require them until we have great docs on how to set them up + a way for users to upload thumbs (which are both doable)