Skip to content
This repository was archived by the owner on Oct 14, 2021. It is now read-only.

add tools and services query class#69

Open
sliu008 wants to merge 4 commits intojddeal:masterfrom
sliu008:toolsandservices
Open

add tools and services query class#69
sliu008 wants to merge 4 commits intojddeal:masterfrom
sliu008:toolsandservices

Conversation

@sliu008
Copy link
Copy Markdown

@sliu008 sliu008 commented Oct 7, 2021

  • Add in tools and service cmr query classes to query umm-t and umm-s.
  • Change the base query class to be more generic and created a GranuleCollectionBaseQuery and ToolServiceBaseQuery classes.
  • Add in more parameters to query by provider, native id, and name.
  • Made concept id query more generic for each classes.
  • Added test cases for new tools query and service query classes.

@frankinspace
Copy link
Copy Markdown

@sliu008

This is great! I was able to checkout your PR and run the unit tests successfully.

Were you able to get live results after applying your changes? I've installed your PR changes locally but when I try doing:

import cmr
query = cmr.queries.ServiceQuery()
query.provider("POCLOUD").get_all()

I get back 0 results. Also, maybe a quick addition of examples to the readme would help

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants