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

Deprecate Future.{eta_min,eta_max} #468

Merged
merged 3 commits into from
May 3, 2021

Conversation

randomir
Copy link
Member

@randomir randomir commented May 1, 2021

The underlying feature has already been removed from SAPI, but let's
provide a grace period for Ocean users. Although the return value has
been None for a while now.

The underlying feature has already been removed from SAPI, but let's
provide a grace period for Ocean users. Although the return value has
been None for a while now.
@codecov-commenter

This comment has been minimized.

Also, fix the value to `None`, as it has effectively been since SAPI
removed the answer time estimates.
@randomir randomir force-pushed the remove-problem-eta-support branch from 509411d to bc90fb6 Compare May 3, 2021 10:18
Get/set keys differed, resulting in python being installed every time.
@randomir randomir merged commit e499c11 into dwavesystems:master May 3, 2021
@randomir randomir deleted the remove-problem-eta-support branch May 3, 2021 10:37
randomir added a commit that referenced this pull request Jun 28, 2021
New Features
---

- Annotated `Future.sampleset` with `.wait_id` method (#377)
- Added `dispatches_events` decorator that can be used to simplify event
  dispatch on method entry and exit (#443/#455)
- CLI: added `--client` option to override the client type used (#458)

Fixes
---

- Dispatch sampling events also from unstructured solvers (#454)
- CLI: fixed solver by client filtering (#457)
- Docs: added link checks (#464), updated sphinx/conf (#470)

Changes
---

- CLI: deprecated `-c` flag (short for `--config-file`) in favor
  of `-f` (#458)
- Dropped Python 3.5 support (#467)
- Deprecated `Future.{eta_min,eta_max}` (#468)
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