-
Notifications
You must be signed in to change notification settings - Fork 6
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
Api part 2 #98
Api part 2 #98
Conversation
Just a quick reminder to keep track of the changes and maybe start a manual CHANGELOG.md for now until I get back to #85 and automate this? |
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 just had one suggestion that is a result of what I added being out of sync
@micahjohnson150 are we missing some commits? I don't see what you're referencing with 'Added in some support for rasters.' |
@micah-prime you were correct. I hadnt pushed everything apparently and got my edits and commits mixed up lol. This should be everything I have. |
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.
Looking good! Couple of minor asks
.all_*
from_unique_entries
which allows users to find options for a query theyre working on . Example: what dates did specific surface area get collected.