Skip to content

Allow the client to get barometer/pressure data (if available)#20

Closed
pieterbartsmit wants to merge 2 commits intomasterfrom
f/baro-mic-sst
Closed

Allow the client to get barometer/pressure data (if available)#20
pieterbartsmit wants to merge 2 commits intomasterfrom
f/baro-mic-sst

Conversation

@pieterbartsmit
Copy link
Copy Markdown
Contributor

This adapts the client to include barometer/pressure data in requests following the patterns already established in the code. Minimum working example (requires access to api and Spotter):

from pysofar.spotter import Spotter

# You will need a valid access token setup for the pysofar library for this to
# work. We refer to the pysofar documentation how to set that up.
spotter = Spotter('SPOT-010149','SPOT-010149')

data = spotter.grab_data(limit=20,start_date='2021-03-22T00:00:00.000Z',end_date='2022-03-23T00:00:00.000Z',
                         include_wind=True,
                         include_barometer_data=True,
                         include_surface_temp_data=True,
                         include_microphone_data=True
                         )

print(data['barometerData'])

- Barometer Data
- Pressure Data
@pieterbartsmit pieterbartsmit requested a review from evanShap March 28, 2022 22:30
@pieterbartsmit pieterbartsmit marked this pull request as ready for review March 28, 2022 22:56

if status != "complete":
raise CouldNotRetrieveFile(f"File creation not yet complete. Try {file_url} in a little bit")
raise CouldNotRetrieveFile(
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pieterbartsmit just checking — were the various additions of new lines throughout this commit intentional?

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.

2 participants