[GitHub] [incubator-sdap-nexus] RKuttruff commented on a diff in pull request #185: SDAP-399: Quickstart update - Docker

2022-09-29 Thread GitBox
RKuttruff commented on code in PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#discussion_r984136730 ## docs/quickstart.rst: ## @@ -18,43 +18,59 @@ This quickstart guide will walk you through how to install and run NEXUS on your Prerequisites

[GitHub] [incubator-sdap-nexus] RKuttruff commented on a diff in pull request #185: SDAP-399: Quickstart update - Docker

2022-09-29 Thread GitBox
RKuttruff commented on code in PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#discussion_r984136329 ## CHANGELOG.md: ## @@ -41,9 +41,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 - Fixed issue where satellite to

[GitHub] [incubator-sdap-nexus] RKuttruff commented on a diff in pull request #185: SDAP-399: Quickstart update - Docker

2022-09-29 Thread GitBox
RKuttruff commented on code in PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#discussion_r984136094 ## docker/jupyter/Dockerfile: ## @@ -29,7 +29,8 @@ ENV CHOWN_HOME_OPTS='-R' ENV REBUILD_CODE=true ARG

[GitHub] [incubator-sdap-nexus] RKuttruff commented on pull request #202: SDAP-402: Update matchOnce logic

2022-09-29 Thread GitBox
RKuttruff commented on PR #202: URL: https://github.com/apache/incubator-sdap-nexus/pull/202#issuecomment-1262998052 I actually don't know exactly how many matches L2 -> Saildrone _should_ be returning. I based the current test case on how many were returning from the AWS deployment.

[GitHub] [incubator-sdap-nexus] skorper commented on a diff in pull request #185: SDAP-399: Quickstart update - Docker

2022-09-29 Thread GitBox
skorper commented on code in PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#discussion_r984133710 ## docker/jupyter/requirements.txt: ## @@ -1,4 +1,3 @@ -shapely -requests -numpy -cassandra-driver==3.9.0 Review Comment: I'm curious why we were able

[GitHub] [incubator-sdap-nexus] skorper commented on a diff in pull request #185: SDAP-399: Quickstart update - Docker

2022-09-29 Thread GitBox
skorper commented on code in PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#discussion_r984133683 ## docs/quickstart.rst: ## @@ -18,43 +18,59 @@ This quickstart guide will walk you through how to install and run NEXUS on your Prerequisites

[GitHub] [incubator-sdap-nexus] skorper commented on pull request #202: SDAP-402: Update matchOnce logic

2022-09-29 Thread GitBox
skorper commented on PR #202: URL: https://github.com/apache/incubator-sdap-nexus/pull/202#issuecomment-1262983746 Note, L2 -> Saildrone is returning more matches than expected for both my changes and the version deployed to AWS (Riley correct me if I'm wrong) so this discrepancy is maybe

[GitHub] [incubator-sdap-nexus] kevinmarlis opened a new pull request, #203: SDAP-403

2022-09-29 Thread GitBox
kevinmarlis opened a new pull request, #203: URL: https://github.com/apache/incubator-sdap-nexus/pull/203 On the AQ deployment we found a redirect bug that caused the AQ `/nexus/list` to crash when attempting to connect to a remote SDAP. The redirect code was catching

[GitHub] [incubator-sdap-nexus] RKuttruff commented on pull request #202: SDAP-402: Update matchOnce logic

2022-09-28 Thread GitBox
RKuttruff commented on PR #202: URL: https://github.com/apache/incubator-sdap-nexus/pull/202#issuecomment-1261347211 As a followup: All the (currently implemented) test cases are passing with the exception of L2 -> Saildrone (returning more matches than expected) -- This is an automated

[GitHub] [incubator-sdap-nexus] RKuttruff commented on pull request #202: SDAP-402: Update matchOnce logic

2022-09-28 Thread GitBox
RKuttruff commented on PR #202: URL: https://github.com/apache/incubator-sdap-nexus/pull/202#issuecomment-1261314820 Testing to see if this affects my regression tests. It seems that the same query is returning different numbers of matches. The query I'm using:

[GitHub] [incubator-sdap-nexus] skorper opened a new pull request, #202: SDAP-402: Update matchOnce logic

2022-09-27 Thread GitBox
skorper opened a new pull request, #202: URL: https://github.com/apache/incubator-sdap-nexus/pull/202 [SDAP-402](https://issues.apache.org/jira/browse/SDAP-402) - Fixed matchOnce logic. Previously, only a single secondary point was matched to a primary point. Now, we are allowing

[GitHub] [incubator-sdap-nexus] RKuttruff opened a new pull request, #201: Bumped ingress timeouts to reflect increased AWS gateway timeout

2022-09-26 Thread GitBox
RKuttruff opened a new pull request, #201: URL: https://github.com/apache/incubator-sdap-nexus/pull/201 Potential fix for duplicate request issue -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [incubator-sdap-nexusproto] dependabot[bot] opened a new pull request, #10: Bump protobuf from 3.2.0 to 3.18.3 in /src/main/python/nexusproto

2022-09-23 Thread GitBox
dependabot[bot] opened a new pull request, #10: URL: https://github.com/apache/incubator-sdap-nexusproto/pull/10 Bumps [protobuf](https://github.com/protocolbuffers/protobuf) from 3.2.0 to 3.18.3. Release notes Sourced from

[GitHub] [incubator-sdap-nexusproto] dependabot[bot] closed pull request #9: Bump protobuf from 3.2.0 to 3.15.0 in /src/main/python/nexusproto

2022-09-23 Thread GitBox
dependabot[bot] closed pull request #9: Bump protobuf from 3.2.0 to 3.15.0 in /src/main/python/nexusproto URL: https://github.com/apache/incubator-sdap-nexusproto/pull/9 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-sdap-nexusproto] dependabot[bot] commented on pull request #9: Bump protobuf from 3.2.0 to 3.15.0 in /src/main/python/nexusproto

2022-09-23 Thread GitBox
dependabot[bot] commented on PR #9: URL: https://github.com/apache/incubator-sdap-nexusproto/pull/9#issuecomment-1256645505 Superseded by #10. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [incubator-sdap-nexus] skorper merged pull request #198: SDAP-400 Added NCAR insitu api to matchup

2022-09-20 Thread GitBox
skorper merged PR #198: URL: https://github.com/apache/incubator-sdap-nexus/pull/198 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] RKuttruff commented on pull request #200: SDAP-401: Added JSON output formatting for COVERAGE plotting

2022-09-19 Thread GitBox
RKuttruff commented on PR #200: URL: https://github.com/apache/incubator-sdap-nexus/pull/200#issuecomment-1251679363 Added implementation for `/domsresults` endpoint. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-sdap-nexus] RKuttruff opened a new pull request, #200: SDAP-401: Added JSON output formatting for COVERAGE plotting

2022-09-19 Thread GitBox
RKuttruff opened a new pull request, #200: URL: https://github.com/apache/incubator-sdap-nexus/pull/200 Added output formatter for COVERAGE plotting to be used by the `/match_spark` endpoint. Outputs support the following plots: - Time series (primary + secondary) - Scatter

[GitHub] [incubator-sdap-nexus] dependabot[bot] opened a new pull request, #199: Bump mako from 1.0.7 to 1.2.2 in /integrations/python-client

2022-09-16 Thread GitBox
dependabot[bot] opened a new pull request, #199: URL: https://github.com/apache/incubator-sdap-nexus/pull/199 Bumps [mako](https://github.com/sqlalchemy/mako) from 1.0.7 to 1.2.2. Release notes Sourced from https://github.com/sqlalchemy/mako/releases;>mako's releases. 1.2.2

[GitHub] [incubator-sdap-in-situ-data-services] briansaildrone commented on pull request #9: EKS IRSA Clarification

2022-09-08 Thread GitBox
briansaildrone commented on PR #9: URL: https://github.com/apache/incubator-sdap-in-situ-data-services/pull/9#issuecomment-1240931286 I've also added further clarification for enabling EKS IRSA -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-sdap-in-situ-data-services] briansaildrone opened a new pull request, #9: EKS IRSA Typo Fix

2022-09-06 Thread GitBox
briansaildrone opened a new pull request, #9: URL: https://github.com/apache/incubator-sdap-in-situ-data-services/pull/9 Accidentally copied/pasted the existing credential-based s3 auth provider into the EKS IRSA documentation. This PR fixes that by adding the correct auth provider. --

[GitHub] [incubator-sdap-nexus] skorper merged pull request #197: Matchup domsresults bugfix

2022-09-02 Thread GitBox
skorper merged PR #197: URL: https://github.com/apache/incubator-sdap-nexus/pull/197 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] skorper commented on pull request #197: Matchup domsresults bugfix

2022-09-02 Thread GitBox
skorper commented on PR #197: URL: https://github.com/apache/incubator-sdap-nexus/pull/197#issuecomment-1235907553 Added provider info to openapi spec: ![Screen Shot 2022-09-02 at 2 20 49

[GitHub] [incubator-sdap-nexus] skorper opened a new pull request, #197: Matchup domsresults bugfix

2022-09-02 Thread GitBox
skorper opened a new pull request, #197: URL: https://github.com/apache/incubator-sdap-nexus/pull/197 * Fixed bug introduced in the last PR where `domsresults` no longer worked. This was because the `details` field name were updated, but the code that inserts results into the database was

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #196: Address matchup errors and integrate SDAP Proxy changes

2022-08-22 Thread GitBox
ngachung merged PR #196: URL: https://github.com/apache/incubator-sdap-nexus/pull/196 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] skorper opened a new pull request, #196: Address matchup errors and integrate SDAP Proxy changes

2022-08-22 Thread GitBox
skorper opened a new pull request, #196: URL: https://github.com/apache/incubator-sdap-nexus/pull/196 Prior to this branch, I have been cherry-picking out the SDAP Proxy changes, as they were breaking things in matchup. This PR addresses those changes, as well as implements feedback

[GitHub] [incubator-sdap-ingester] MrFok opened a new pull request, #60: Preliminary Zarr Implementation + Tests

2022-08-19 Thread GitBox
MrFok opened a new pull request, #60: URL: https://github.com/apache/incubator-sdap-ingester/pull/60 - Generated separate files for Zarr implementation (without disturbing existing pipeline) - Local test generated for Lazy Load Zarr Aggregation (LLZA) - Minor dependency changes, bug

[GitHub] [incubator-sdap-nexus] RKuttruff commented on pull request #185: SDAP-399: Quickstart update

2022-08-14 Thread GitBox
RKuttruff commented on PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#issuecomment-1214321592 TODO: Check if AVHRR migration to Earthdata Cloud will affect granule download step. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-sdap-nexus] RKuttruff commented on pull request #185: SDAP-399: Quickstart update

2022-08-12 Thread GitBox
RKuttruff commented on PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#issuecomment-1212779246 Jupyter image verified to be working. Now awaiting validation & runtime observations from others. PR will be ready when this happens. -- This is an

[GitHub] [incubator-sdap-nexus] RKuttruff commented on pull request #185: SDAP-399: Quickstart update

2022-08-11 Thread GitBox
RKuttruff commented on PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#issuecomment-1212307051 I instead plan to have the k8s deployment in another PR. Once I get this one sorted & validated, I will make it ready for review. -- This is an automated message from the

[GitHub] [incubator-sdap-nexus] RKuttruff commented on pull request #185: SDAP-399: Quickstart update

2022-08-10 Thread GitBox
RKuttruff commented on PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#issuecomment-1211551329 ``` --- TypeError Traceback (most recent call last) Input In

[GitHub] [incubator-sdap-nexus] dependabot[bot] commented on pull request #190: Bump lxml from 4.6.5 to 4.9.1 in /tools/cdms

2022-08-10 Thread GitBox
dependabot[bot] commented on PR #190: URL: https://github.com/apache/incubator-sdap-nexus/pull/190#issuecomment-1211342374 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor

[GitHub] [incubator-sdap-nexus] dependabot[bot] commented on pull request #192: Bump numpy from 1.20.1 to 1.22.0 in /tools/cdms

2022-08-10 Thread GitBox
dependabot[bot] commented on PR #192: URL: https://github.com/apache/incubator-sdap-nexus/pull/192#issuecomment-1211342387 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor

[GitHub] [incubator-sdap-nexus] dependabot[bot] commented on pull request #191: Bump pillow from 8.1.2 to 9.0.1 in /tools/cdms

2022-08-10 Thread GitBox
dependabot[bot] commented on PR #191: URL: https://github.com/apache/incubator-sdap-nexus/pull/191#issuecomment-1211342379 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor

[GitHub] [incubator-sdap-nexus] dependabot[bot] commented on pull request #193: Bump protobuf from 3.14.0 to 3.15.0 in /tools/cdms

2022-08-10 Thread GitBox
dependabot[bot] commented on PR #193: URL: https://github.com/apache/incubator-sdap-nexus/pull/193#issuecomment-1211342395 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor

[GitHub] [incubator-sdap-nexus] ngachung closed pull request #193: Bump protobuf from 3.14.0 to 3.15.0 in /tools/cdms

2022-08-10 Thread GitBox
ngachung closed pull request #193: Bump protobuf from 3.14.0 to 3.15.0 in /tools/cdms URL: https://github.com/apache/incubator-sdap-nexus/pull/193 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [incubator-sdap-nexus] ngachung closed pull request #191: Bump pillow from 8.1.2 to 9.0.1 in /tools/cdms

2022-08-10 Thread GitBox
ngachung closed pull request #191: Bump pillow from 8.1.2 to 9.0.1 in /tools/cdms URL: https://github.com/apache/incubator-sdap-nexus/pull/191 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [incubator-sdap-nexus] ngachung closed pull request #190: Bump lxml from 4.6.5 to 4.9.1 in /tools/cdms

2022-08-10 Thread GitBox
ngachung closed pull request #190: Bump lxml from 4.6.5 to 4.9.1 in /tools/cdms URL: https://github.com/apache/incubator-sdap-nexus/pull/190 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [incubator-sdap-nexus] ngachung closed pull request #189: Bump mistune from 0.8.4 to 2.0.3 in /tools/cdms

2022-08-10 Thread GitBox
ngachung closed pull request #189: Bump mistune from 0.8.4 to 2.0.3 in /tools/cdms URL: https://github.com/apache/incubator-sdap-nexus/pull/189 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [incubator-sdap-nexus] ngachung closed pull request #192: Bump numpy from 1.20.1 to 1.22.0 in /tools/cdms

2022-08-10 Thread GitBox
ngachung closed pull request #192: Bump numpy from 1.20.1 to 1.22.0 in /tools/cdms URL: https://github.com/apache/incubator-sdap-nexus/pull/192 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [incubator-sdap-nexus] dependabot[bot] commented on pull request #189: Bump mistune from 0.8.4 to 2.0.3 in /tools/cdms

2022-08-10 Thread GitBox
dependabot[bot] commented on PR #189: URL: https://github.com/apache/incubator-sdap-nexus/pull/189#issuecomment-1211342368 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor

[GitHub] [incubator-sdap-nexus] ngachung closed pull request #188: Bump notebook from 6.4.11 to 6.4.12 in /tools/cdms

2022-08-10 Thread GitBox
ngachung closed pull request #188: Bump notebook from 6.4.11 to 6.4.12 in /tools/cdms URL: https://github.com/apache/incubator-sdap-nexus/pull/188 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #194: SDAP-390 Updated CDMS reader tool requirements.txt

2022-08-10 Thread GitBox
ngachung merged PR #194: URL: https://github.com/apache/incubator-sdap-nexus/pull/194 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-in-situ-data-services] dependabot[bot] opened a new pull request, #8: chore(deps): bump pyspark from 3.1.2 to 3.1.3

2022-08-10 Thread GitBox
dependabot[bot] opened a new pull request, #8: URL: https://github.com/apache/incubator-sdap-in-situ-data-services/pull/8 Bumps [pyspark](https://github.com/apache/spark) from 3.1.2 to 3.1.3. Commits

[GitHub] [incubator-sdap-nexus] skorper merged pull request #195: Fix changelog

2022-08-10 Thread GitBox
skorper merged PR #195: URL: https://github.com/apache/incubator-sdap-nexus/pull/195 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] RKuttruff opened a new pull request, #195: Fix changelog

2022-08-09 Thread GitBox
RKuttruff opened a new pull request, #195: URL: https://github.com/apache/incubator-sdap-nexus/pull/195 Minor formatting error + entry moved to correct section. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #183: SDAP-398 - CDMS Regression Test Script

2022-08-09 Thread GitBox
ngachung merged PR #183: URL: https://github.com/apache/incubator-sdap-nexus/pull/183 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] ngachung opened a new pull request, #194: SDAP-390 Updated CDMS reader tool requirements.txt

2022-08-09 Thread GitBox
ngachung opened a new pull request, #194: URL: https://github.com/apache/incubator-sdap-nexus/pull/194 GitHub dependabot is flagging a lot of the specific dependencies. To run the tool we really only need netCDF4 library. -- This is an automated message from the Apache Git Service. To

[GitHub] [incubator-sdap-nexus] dependabot[bot] opened a new pull request, #193: Bump protobuf from 3.14.0 to 3.15.0 in /tools/cdms

2022-08-09 Thread GitBox
dependabot[bot] opened a new pull request, #193: URL: https://github.com/apache/incubator-sdap-nexus/pull/193 Bumps [protobuf](https://github.com/protocolbuffers/protobuf) from 3.14.0 to 3.15.0. Release notes Sourced from

[GitHub] [incubator-sdap-nexus] dependabot[bot] opened a new pull request, #192: Bump numpy from 1.20.1 to 1.22.0 in /tools/cdms

2022-08-09 Thread GitBox
dependabot[bot] opened a new pull request, #192: URL: https://github.com/apache/incubator-sdap-nexus/pull/192 Bumps [numpy](https://github.com/numpy/numpy) from 1.20.1 to 1.22.0. Release notes Sourced from https://github.com/numpy/numpy/releases;>numpy's releases. v1.22.0

[GitHub] [incubator-sdap-nexus] dependabot[bot] opened a new pull request, #190: Bump lxml from 4.6.5 to 4.9.1 in /tools/cdms

2022-08-09 Thread GitBox
dependabot[bot] opened a new pull request, #190: URL: https://github.com/apache/incubator-sdap-nexus/pull/190 Bumps [lxml](https://github.com/lxml/lxml) from 4.6.5 to 4.9.1. Changelog Sourced from https://github.com/lxml/lxml/blob/master/CHANGES.txt;>lxml's changelog. 4.9.1

[GitHub] [incubator-sdap-nexus] dependabot[bot] opened a new pull request, #188: Bump notebook from 6.4.11 to 6.4.12 in /tools/cdms

2022-08-09 Thread GitBox
dependabot[bot] opened a new pull request, #188: URL: https://github.com/apache/incubator-sdap-nexus/pull/188 Bumps [notebook](http://jupyter.org) from 6.4.11 to 6.4.12. [![Dependabot compatibility

[GitHub] [incubator-sdap-nexus] dependabot[bot] opened a new pull request, #191: Bump pillow from 8.1.2 to 9.0.1 in /tools/cdms

2022-08-09 Thread GitBox
dependabot[bot] opened a new pull request, #191: URL: https://github.com/apache/incubator-sdap-nexus/pull/191 Bumps [pillow](https://github.com/python-pillow/Pillow) from 8.1.2 to 9.0.1. Release notes Sourced from https://github.com/python-pillow/Pillow/releases;>pillow's

[GitHub] [incubator-sdap-nexus] dependabot[bot] opened a new pull request, #189: Bump mistune from 0.8.4 to 2.0.3 in /tools/cdms

2022-08-09 Thread GitBox
dependabot[bot] opened a new pull request, #189: URL: https://github.com/apache/incubator-sdap-nexus/pull/189 Bumps [mistune](https://github.com/lepture/mistune) from 0.8.4 to 2.0.3. Release notes Sourced from https://github.com/lepture/mistune/releases;>mistune's releases.

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #178: SDAP-390 Update NetCDF reader tool for data match-up

2022-08-09 Thread GitBox
ngachung merged PR #178: URL: https://github.com/apache/incubator-sdap-nexus/pull/178 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #186: Fixed error causing timeSeriesSpark to fail

2022-08-04 Thread GitBox
ngachung merged PR #186: URL: https://github.com/apache/incubator-sdap-nexus/pull/186 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] RKuttruff commented on pull request #185: CDMS-159: Quickstart update

2022-08-04 Thread GitBox
RKuttruff commented on PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#issuecomment-1205848293 Currently investigating deployment with kubernetes. Will write second guide for that and add to this PR -- This is an automated message from the Apache Git Service. To

[GitHub] [incubator-sdap-nexus] tloubrieu-jpl opened a new pull request, #187: add a script to generate collection configuration from a remote SDAP

2022-08-04 Thread GitBox
tloubrieu-jpl opened a new pull request, #187: URL: https://github.com/apache/incubator-sdap-nexus/pull/187 This pull request adds a small script to generate a piece of collection-config configmap to present remote SDAP collections in the local SDAP server. -- This is an automated

[GitHub] [incubator-sdap-nexus] RKuttruff commented on pull request #185: CDMS-159: Quickstart update

2022-08-04 Thread GitBox
RKuttruff commented on PR #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185#issuecomment-1205649728 Currently uses an older version of the nexus webapp image. Changes made to sdap after this image was made causes the `/timeSeriesSpark` endpoint to fail (This endpoint is

[GitHub] [incubator-sdap-nexus] RKuttruff opened a new pull request, #186: Fixed error in timeSeriesSpark

2022-08-04 Thread GitBox
RKuttruff opened a new pull request, #186: URL: https://github.com/apache/incubator-sdap-nexus/pull/186 It appeared to have been caused by an import issue in RedirectHandler.py Updated import statements and tested with local SDAP. Verified results against SDAP image preceding the

[GitHub] [incubator-sdap-nexus] RKuttruff opened a new pull request, #185: CDMS-159: Quickstart update

2022-08-03 Thread GitBox
RKuttruff opened a new pull request, #185: URL: https://github.com/apache/incubator-sdap-nexus/pull/185 Updated quickstart guide for standalone SDAP deployment with Docker. Draft PR since this guide still needs to be verified and tested. -- This is an automated message from the

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #165: Fix matchup bugs

2022-08-02 Thread GitBox
ngachung merged PR #165: URL: https://github.com/apache/incubator-sdap-nexus/pull/165 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] ngachung commented on a diff in pull request #178: SDAP-390 Update NetCDF reader tool for data match-up

2022-08-02 Thread GitBox
ngachung commented on code in PR #178: URL: https://github.com/apache/incubator-sdap-nexus/pull/178#discussion_r935888956 ## tools/cdms/cdms_reader.py: ## @@ -0,0 +1,250 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements.

[GitHub] [incubator-sdap-nexus] ngachung commented on pull request #166: CDMS - 150 NETCDF Reader Updates

2022-08-02 Thread GitBox
ngachung commented on PR #166: URL: https://github.com/apache/incubator-sdap-nexus/pull/166#issuecomment-1202991601 This PR has been superseded by https://github.com/apache/incubator-sdap-nexus/pull/178 -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-sdap-nexus] ngachung closed pull request #166: CDMS - 150 NETCDF Reader Updates

2022-08-02 Thread GitBox
ngachung closed pull request #166: CDMS - 150 NETCDF Reader Updates URL: https://github.com/apache/incubator-sdap-nexus/pull/166 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #181: Preserve insitu schema

2022-07-28 Thread GitBox
ngachung merged PR #181: URL: https://github.com/apache/incubator-sdap-nexus/pull/181 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #182: Fix NetCDF output

2022-07-28 Thread GitBox
ngachung merged PR #182: URL: https://github.com/apache/incubator-sdap-nexus/pull/182 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] dependabot[bot] opened a new pull request, #184: Bump urllib3 from 1.26.2 to 1.26.5 in /data-access

2022-07-27 Thread GitBox
dependabot[bot] opened a new pull request, #184: URL: https://github.com/apache/incubator-sdap-nexus/pull/184 Bumps [urllib3](https://github.com/urllib3/urllib3) from 1.26.2 to 1.26.5. Release notes Sourced from https://github.com/urllib3/urllib3/releases;>urllib3's releases.

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #173: Add metadata to remote coll

2022-07-27 Thread GitBox
ngachung merged PR #173: URL: https://github.com/apache/incubator-sdap-nexus/pull/173 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] tloubrieu-jpl commented on pull request #173: Add metadata to remote coll

2022-07-25 Thread GitBox
tloubrieu-jpl commented on PR #173: URL: https://github.com/apache/incubator-sdap-nexus/pull/173#issuecomment-1194753764 Hello @nga, @skorper, Can you review this pull request ? I am afraid it gets buried under the other pull requests which came after. I need the pull request to

[GitHub] [incubator-sdap-nexus] RKuttruff opened a new pull request, #183: CDMS-151 - Regression Test Script

2022-07-25 Thread GitBox
RKuttruff opened a new pull request, #183: URL: https://github.com/apache/incubator-sdap-nexus/pull/183 Regression test script that tests a number of SDAP endpoints + the insitu endpoint. SDAP host and insitu endpoint can be configured through environment variables. Tests can also be

[GitHub] [incubator-sdap-nexus] RKuttruff opened a new pull request, #182: Fix NetCDF output

2022-07-25 Thread GitBox
RKuttruff opened a new pull request, #182: URL: https://github.com/apache/incubator-sdap-nexus/pull/182 Latitude and longitude were switched. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [incubator-sdap-nexus] RKuttruff opened a new pull request, #181: Preserve insitu schema

2022-07-20 Thread GitBox
RKuttruff opened a new pull request, #181: URL: https://github.com/apache/incubator-sdap-nexus/pull/181 Matchup makes repeated calls to `query_insitu_schema()` in the insitu module, which results in a severe performance penalty due to repeatedly hitting the insitu API for the same data.

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #179: SDAP-396: saildrone insitu

2022-07-20 Thread GitBox
ngachung merged PR #179: URL: https://github.com/apache/incubator-sdap-nexus/pull/179 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #175: Fix CSV/NetCDF bug

2022-07-20 Thread GitBox
ngachung merged PR #175: URL: https://github.com/apache/incubator-sdap-nexus/pull/175 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #180: SDAP-397: Requested matchup updates

2022-07-20 Thread GitBox
ngachung merged PR #180: URL: https://github.com/apache/incubator-sdap-nexus/pull/180 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #176: Fix satellite to satellite matchup

2022-07-20 Thread GitBox
ngachung merged PR #176: URL: https://github.com/apache/incubator-sdap-nexus/pull/176 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] ngachung commented on pull request #179: SDAP-396: saildrone insitu

2022-07-20 Thread GitBox
ngachung commented on PR #179: URL: https://github.com/apache/incubator-sdap-nexus/pull/179#issuecomment-1190587866 Thank you! That's expected as I filtered out only matches that contained mass_concentration_of_chlorophyll_in_sea_water. -- This is an automated message from the Apache

[GitHub] [incubator-sdap-nexus] skorper commented on pull request #179: SDAP-396: saildrone insitu

2022-07-20 Thread GitBox
skorper commented on PR #179: URL: https://github.com/apache/incubator-sdap-nexus/pull/179#issuecomment-1190504757 @ngachung Ran this same request locally using this branch -- it seemed to match to more than 6 points. See output below. I ran this through a script I wrote to verify

[GitHub] [incubator-sdap-nexus] skorper commented on pull request #180: SDAP-397: Requested matchup updates

2022-07-20 Thread GitBox
skorper commented on PR #180: URL: https://github.com/apache/incubator-sdap-nexus/pull/180#issuecomment-1190486320 Latest update: - Query insitu schema endpoint to get units for insitu observations - Use "meta" field to populate "id" field for insitu - Moved depth to top level

[GitHub] [incubator-sdap-nexus] skorper commented on pull request #180: SDAP-397: Requested matchup updates

2022-07-20 Thread GitBox
skorper commented on PR #180: URL: https://github.com/apache/incubator-sdap-nexus/pull/180#issuecomment-1190433207 Thanks for your review @ngachung. - I wondered that as well. The depth field should probably be at the root level alongside lat/lon/time. - Thank you! - Thanks!

[GitHub] [incubator-sdap-nexus] ngachung commented on pull request #180: SDAP-397: Requested matchup updates

2022-07-20 Thread GitBox
ngachung commented on PR #180: URL: https://github.com/apache/incubator-sdap-nexus/pull/180#issuecomment-1190348199 Thank you, @skorper. - We can discuss at our next team meeting, but I wonder if depth should just be added as an attribute similar to lat, lon for both primary and

[GitHub] [incubator-sdap-nexus] ngachung commented on pull request #179: SDAP-396: saildrone insitu

2022-07-20 Thread GitBox
ngachung commented on PR #179: URL: https://github.com/apache/incubator-sdap-nexus/pull/179#issuecomment-1190328943 Thank you, @skorper. This is a call we used for the notebook demo that returned 6 matches.

[GitHub] [incubator-sdap-nexus] skorper opened a new pull request, #180: SDAP-397: Requested matchup updates

2022-07-20 Thread GitBox
skorper opened a new pull request, #180: URL: https://github.com/apache/incubator-sdap-nexus/pull/180 https://issues.apache.org/jira/browse/SDAP-397 - Added depth to matchup insitu output - Added in the "data" field (see example below). Is this a desirable location for this

[GitHub] [incubator-sdap-nexus] skorper opened a new pull request, #179: SDAP-396: saildrone insitu

2022-07-20 Thread GitBox
skorper opened a new pull request, #179: URL: https://github.com/apache/incubator-sdap-nexus/pull/179 https://issues.apache.org/jira/browse/SDAP-396 Added saildrone insitu API. I haven't had much luck finding a request that results in a matchup. I'm trying to find overlap with

[GitHub] [incubator-sdap-nexus] JordanGethers opened a new pull request, #178: SDAP-390 Update NetCDF reader tool for data match-up

2022-07-18 Thread GitBox
JordanGethers opened a new pull request, #178: URL: https://github.com/apache/incubator-sdap-nexus/pull/178 cdms_reader.py is the original doms_reader.py code with additional functionality provided for the user to make each run of the program more customizable. The user may now select,

[GitHub] [incubator-sdap-nexus] RKuttruff opened a new pull request, #177: Fix domslist count

2022-07-14 Thread GitBox
RKuttruff opened a new pull request, #177: URL: https://github.com/apache/incubator-sdap-nexus/pull/177 Count equalling insitu datasets + satellite datasets Confirmed working with quick test on local SDAP instance -- This is an automated message from the Apache Git Service. To

[GitHub] [incubator-sdap-nexus] skorper opened a new pull request, #176: Fix satellite to satellite matchup

2022-07-14 Thread GitBox
skorper opened a new pull request, #176: URL: https://github.com/apache/incubator-sdap-nexus/pull/176 - Nga was seeing a 500 error when doing satellite to satellite matchup - Fixed satellite to satellite matchup failure -- was due to ` 'platform': None,` line - Nga's request was

[GitHub] [incubator-sdap-nexus] tloubrieu-jpl commented on a diff in pull request #174: CDMS-122 - Limited Zarr Proxy

2022-07-14 Thread GitBox
tloubrieu-jpl commented on code in PR #174: URL: https://github.com/apache/incubator-sdap-nexus/pull/174#discussion_r921396416 ## analysis/webservice/algorithms_spark/Matchup.py: ## @@ -218,12 +218,17 @@ def calc(self, request, **args): self.log.debug("Querying for

[GitHub] [incubator-sdap-nexus] ngachung commented on pull request #175: Fix CSV/NetCDF bug

2022-07-13 Thread GitBox
ngachung commented on PR #175: URL: https://github.com/apache/incubator-sdap-nexus/pull/175#issuecomment-1183782964 Lgtm. Update CHANGELOG.md please? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [incubator-sdap-nexus] skorper opened a new pull request, #175: Fix CSV/NetCDF bug

2022-07-13 Thread GitBox
skorper opened a new pull request, #175: URL: https://github.com/apache/incubator-sdap-nexus/pull/175 CSV and NetCDF requests were failing for AWS Insitu endpoints because there is no entry for those datasets in the `METADATA_LINKS` dict. Updated the code so that is optional. -- This

[GitHub] [incubator-sdap-in-situ-data-services] ngachung merged pull request #7: feat: SDAP-395: CDMS JSON Schema Endpoint

2022-07-13 Thread GitBox
ngachung merged PR #7: URL: https://github.com/apache/incubator-sdap-in-situ-data-services/pull/7 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] RKuttruff opened a new pull request, #174: CDMS-122 - Limited Zarr Proxy

2022-07-12 Thread GitBox
RKuttruff opened a new pull request, #174: URL: https://github.com/apache/incubator-sdap-nexus/pull/174 Implementation of a proxy to fetch Nexus data stored in an AWS S3 bucket as a Zarr array. Currently only supports the MUR25-JPL-L4-GLOB-v04.2 dataset and gridded data and is only

[GitHub] [incubator-sdap-nexus] ngachung commented on pull request #166: CDMS - 150 NETCDF Reader Updates

2022-07-12 Thread GitBox
ngachung commented on PR #166: URL: https://github.com/apache/incubator-sdap-nexus/pull/166#issuecomment-1181956713 Sorry for the confusion @JordanGethers. Yes, the github repository is public, but the CDMS JIRA is restricted. Please push all your changes to your forked repo, but where

[GitHub] [incubator-sdap-nexus] tloubrieu-jpl opened a new pull request, #173: Add metadata to remote coll

2022-07-12 Thread GitBox
tloubrieu-jpl opened a new pull request, #173: URL: https://github.com/apache/incubator-sdap-nexus/pull/173 This pull request completes ticket https://issues.apache.org/jira/browse/SDAP-388 by providing metadata of the remote collections in the /list end-point. The remote metadata

[GitHub] [incubator-sdap-nexus] JordanGethers commented on pull request #166: CDMS - 150 NETCDF Reader Updates

2022-07-12 Thread GitBox
JordanGethers commented on PR #166: URL: https://github.com/apache/incubator-sdap-nexus/pull/166#issuecomment-1181826386 Hello Nga, I made the adjustments you mentioned for SDAP-390, but I don't exactly understand where to put the code since you say that JIRA is private and not

[GitHub] [incubator-sdap-nexus] ngachung commented on pull request #166: CDMS - 150 NETCDF Reader Updates

2022-07-08 Thread GitBox
ngachung commented on PR #166: URL: https://github.com/apache/incubator-sdap-nexus/pull/166#issuecomment-1179182763 Thank you, @JordanGethers! Please reference https://issues.apache.org/jira/projects/SDAP/issues/SDAP-390 as CDMS Jira is restricted and not public. Please also

[GitHub] [incubator-sdap-nexus] ngachung merged pull request #172: SDAP-371: Fix domssubset API

2022-07-07 Thread GitBox
ngachung merged PR #172: URL: https://github.com/apache/incubator-sdap-nexus/pull/172 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [incubator-sdap-nexus] skorper opened a new pull request, #172: SDAP-371: Fix domssubset API

2022-07-07 Thread GitBox
skorper opened a new pull request, #172: URL: https://github.com/apache/incubator-sdap-nexus/pull/172 https://issues.apache.org/jira/browse/SDAP-371 ### Summary of changes - Renamed endpoint `/domssubset` -> `/cdmssubset` - Updated openapi spec with "cdms" terminology

[GitHub] [incubator-sdap-nexus] skorper merged pull request #171: Fix for null count field in return for matchup_spark request

2022-07-06 Thread GitBox
skorper merged PR #171: URL: https://github.com/apache/incubator-sdap-nexus/pull/171 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

<    1   2   3   4   5   6   7   8   9   10   >