Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

environment: transfer and adapt Env and SenseEnv from physic into Weather and SenseWeather #378

Closed
maruel opened this issue Jan 14, 2019 · 2 comments
Closed

environment: transfer and adapt Env and SenseEnv from physic into Weather and SenseWeather #378

maruel opened this issue Jan 14, 2019 · 2 comments
Milestone
v4.0.0

Comments

@maruel
Copy link
Contributor

maruel commented Jan 14, 2019

As part of this work, make it use context.Context.

The addition of the new packages can be done seamlessly in v3.5.0, but the removal of physic.Env and physic.SenseEnv is a breaking change.
@maruel maruel added this to the v4 milestone Jan 14, 2019
@maruel
Copy link
Contributor Author

maruel commented Jan 17, 2019

If the new methods are named SenseWeather() and SenseWeatherContinuous(), we can have the driver supports both for v3.5.0 which will be less disruptive for users.

@maruel maruel mentioned this issue Sep 18, 2023
Open
@maruel
Copy link
Contributor Author

maruel commented Sep 18, 2023

Ported to periph/conn#28.

@maruel maruel closed this as completed Sep 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v4.0.0
Development

No branches or pull requests
1 participant
@maruel