-
Notifications
You must be signed in to change notification settings - Fork 4
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
V2 #21
Closed
Closed
V2 #21
Changes from 20 commits
Commits
Show all changes
98 commits
Select commit
Hold shift + click to select a range
341d130
remove assets, rename folders
rileyhales 4d358ef
organize versions into subpackages
rileyhales c630e12
dates default return format as csv, json still available
rileyhales 1dc7199
option to not send index in csv responses
rileyhales 7b48b61
remove aws token
rileyhales 2f58d4d
remove aws token
rileyhales b444802
put configs in ignored file
rileyhales e5aa0f7
reimplement getreachid and organize module names
rileyhales 128f11b
use pygeoglows for historical, drop forecast anomalies
rileyhales 1b48ecd
uncomment logging
rileyhales ecaff05
correct forecast data structures
rileyhales 0518ecd
Merge branch 'byumaster'
msouff bc5cfe0
postponed v2 updates
msouff 4dbd27e
updated helm-docker
msouff cd3ae4b
build helm only on merge to master
msouff da3f45c
build helm only on merge to master
msouff 5dad9a7
build helm only on merge to master
msouff 3a679a1
Merge branch 'helm-docker_update' into 'master'
msouff 3a3ca0f
build commit only on merge to master
msouff 859a89a
Merge branch 'helm-docker_update' into 'master'
msouff 258a48c
reversed ci triggers
msouff 040c3a5
Merge branch 'helm-docker_update' into 'master'
msouff 5c464ce
include unified tethys ci
msouff dc62afb
include unified tethys ci
msouff 980be3b
include unified tethys ci
msouff ac17d34
updated helm-check credentials
msouff 1e094a7
added ECCR credentials
msouff 87e1763
added ECCR credentials
msouff 8274d4f
test ECCR user
msouff ff6381d
test all ECCR credentials
msouff 5ef941a
test all ECCR credentials
msouff ce00863
test all ECCR credentials
msouff 8ab9245
test all ECCR credentials
msouff 18f0fe5
test all ECCR credentials
msouff 35f19a9
wrap variables in double quotes
msouff d0a7179
wrap variables in double quotes
msouff b83d46b
escape '$' in HELM_REPO_USERNAME with backslash
msouff dbac99d
escape '$' in HELM_REPO_USERNAME with backslash
msouff 3be8136
escape '$' in HELM_REPO_USERNAME with backslash
msouff 776f175
escape '$' on CI
msouff 6ef7a28
testing user variable
msouff d1fc32b
testing user variable
msouff 15d804e
escape '$' on CI
msouff 8037b40
hardcode user for helm check and upload
msouff c7ac71d
hardcode user for helm check and upload
msouff f473709
hardcode user for helm check and upload
msouff f122d14
revert to using CI var
msouff 5ad7e5a
revert to using CI var
msouff cc490d4
revert to using CI var
msouff 45d0204
revert to using local check_helm_chart
msouff ff0e534
Helm docker update
msouff c4afd82
Merge branch 'helm-docker_update' into 'master'
msouff d1cea8c
upload helm
msouff 713ec27
Merge branch 'master' into 'helm-docker_update'
msouff 29422d5
upload helm
msouff 9e66a9b
Helm docker update
msouff c7e0ffe
Merge branch 'helm-docker_update' into 'master'
msouff 36b21c6
upload helm
msouff 6a2ddc3
Merge branch 'helm-docker_update' into 'master'
msouff cbbb93f
upload helm
msouff bc4055a
Merge branch 'helm-docker_update' into 'master'
msouff a858069
updated endpoints
msouff 8ec48c2
Merge branch 'endpoints' into 'master'
msouff 6826b58
updated license to cc; updated geoglows spelling
msouff 7bb3a30
correct historical retrieval and static pages
rileyhales 117e692
update publication list
rileyhales ece148c
re-enable analytics on v2
rileyhales 14e2155
update capitalization of GEOGLOWS
rileyhales c71f3b1
consolidate logging
rileyhales 9cff8f2
remove logging
rileyhales 9fd9591
Merge branch 'creative_commons' into 'master'
msouff d83421e
Revert "consolidate logging"
rileyhales f2ed65f
update geographic properties table
rileyhales 769c9fd
remove assets, rename folders
rileyhales 169323b
rebased v2 to latest aquaveo master
msouff f9a3578
dates default return format as csv, json still available
rileyhales c7b2d38
option to not send index in csv responses
rileyhales 93fe459
remove aws token
rileyhales e191960
remove aws token
rileyhales a2b4ba0
put configs in ignored file
rileyhales 7645dd8
add missing imports
rileyhales 19e0436
reimplement getreachid and organize module names
rileyhales d34209a
use pygeoglows for historical, drop forecast anomalies
rileyhales faf96a1
uncomment logging
rileyhales 6914bbc
correct forecast data structures
rileyhales ea71075
correct historical retrieval and static pages
rileyhales d389e0e
re-enable analytics on v2
rileyhales a4f0589
update capitalization of GEOGLOWS
rileyhales ef79400
consolidate logging
rileyhales 729b63e
remove logging
rileyhales 48cd563
Revert "consolidate logging"
rileyhales 5dd0c44
update geographic properties table
rileyhales d969ea5
Merge branch 'v2' of https://github.com/BYU-Hydroinformatics/geoglows…
msouff 0ee848c
fixes from rebases
rileyhales f72b731
fixed lincense and docs
msouff fe78543
correct retrospective bucket names
rileyhales 21813ac
correct path for forecast datasets
rileyhales 3c3b657
select first not last date available
rileyhales 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -184,3 +184,4 @@ dmypy.json | |
|
||
## Misc | ||
sample_data/** | ||
/aws_config.json |
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
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
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.
The updates to the Dockerfile correctly reflect the changes in environment file naming and application directory structure. Ensure that the
environment.yaml
file contains all necessary dependencies for the application. Additionally, consider optimizing the Dockerfile by combiningRUN
commands where possible to reduce the number of layers in the final image, improving build performance and image size.