Skip to content
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

Update titler-xarray-endpoint to production #700

Closed
wants to merge 1 commit into from

Conversation

abarciauskas-bgse
Copy link
Contributor

There are now 2 deployments of titiler-xarray: one production endpoint and one dev. Dev will be reserved for new feature development and testing, so we should use the production endpoint in VEDA dashboard services. I will make an equivalent PR to veda-config.

@netlify
Copy link

netlify bot commented Oct 13, 2023

Deploy Preview for veda-ui ready!

Name Link
🔨 Latest commit 30fd35d
🔍 Latest deploy log https://app.netlify.com/sites/veda-ui/deploys/6529697f48f16b000861839f
😎 Deploy Preview https://deploy-preview-700--veda-ui.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.

@hanbyul-here
Copy link
Collaborator

While we are at it, I want to flag that the example data you put (oco2-geos-l3-daily) is missing spatial and temporal extent. which leads to the error on the page. (I noticed this issue a while ago, so I believe it is unrelated to changing the endpoint to production.)

  • Are spatial and temporal extent intentionally missing?
  • If so, is there a better example zarr data to put? since oco2-geos-l3-daily is missing the values that dashboard needs?

@abarciauskas-bgse
Copy link
Contributor Author

@hanbyul-here thanks for looking, tagging @slesaad who published that dataset. @slesaad should we fix or remove this dataset? I can look into doing so if it would help.

@danielfdsilva
Copy link
Collaborator

@abarciauskas-bgse Should we change this in the veda-ui, given that this is never used directly? Wouldn't it make more sense to keep the dev endpoint here? Is it unstable?

@abarciauskas-bgse
Copy link
Contributor Author

🤔 The intention to have dev and prod deployment is that dev might be unstable. I do think it could be useful to leave the configuration here pointing to dev so we can test new features' in titiler-xarray are still working with VEDA UI. So yes I think it makes sense to close this for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants