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.

Sign up for GitHub

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

Jump to bottom

Home-Based Config Paths #318

Closed
ccoffrin opened this issue Jun 16, 2019 · 3 comments · Fixed by #386
Closed

Home-Based Config Paths #318

ccoffrin opened this issue Jun 16, 2019 · 3 comments · Fixed by #386
Labels
bug

Comments

@ccoffrin
Copy link

It seems that at least on OS X the config file paths cannot include home-based paths. For example, ~/foo/my.conf or $HOME/foo/my.conf. If it is easy to support these it would provide some added convenience.
@randomir
Copy link
Member

Isn't ~/Library/Application Support/dwave supported?

You can double check with:

$ dwave config ls -m

@ccoffrin
Copy link
Author

Let me clarify. In this library the call load_config("~/foo/my.conf") does not seem to work. The problem seems to be that the special character ~ is not expanded before trying to open the file.

@randomir
Copy link
Member

Oh, that is a bug. Will fix. Thanks!

@randomir randomir added the bug label Jun 19, 2020
randomir added a commit to randomir/dwave-cloud-client that referenced this issue Jun 19, 2020
@randomir
Expand user home and env vars in config_file path
a122a34 
Closes dwavesystems#318.
@randomir randomir mentioned this issue Jun 19, 2020
Merged
randomir added a commit to randomir/dwave-cloud-client that referenced this issue Jun 19, 2020
@randomir
Expand user home and env vars in config_file path
e3a0be7 
Closes dwavesystems#318.
randomir added a commit that referenced this issue Jul 8, 2020
@randomir
Release version 0.7.3
a178c2f 
Changes from 0.7.2:

New Features
---

- Hybrid solvers support (#378):
  - `dwave.cloud.hybrid.Client` is a new client type specialized
    for SAPI hybrid solvers
  - `Solver.hybrid` is a new derived property, also available for
    feature-based solver filtering
- Client certificate SAPI authentication supported via `client_cert`
  and `client_cert_key` config/kwarg parameters (#382)
- `dwave install` now supports `--upgrade` option to (re)install
  partially installed contrib packages, or upgrade stale
  requirements (#381)

Fixes
---

- `qpu`/`sw`/`hybrid` derived solver properties refactored to use
  the new SAPI solver property, `category`
- Nested exception properly propagated by `Future.exception()`
- `dwave install` now enforces the full requirement specifier for
  each contrib dependency
- User home and environment variables are now expanded in
  config file path (#318)

Changes
---

- `Solver.is_*` properties scheduled for removal in 0.8.0
- `Future.error` deprecated in favor of `Future.exception()` and
  scheduled for removal in 0.9.0
- Removed most of Python 2 constructs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bugfix/expand config path tildas/issue 318 randomir/dwave-cloud-client
2 participants
@randomir @ccoffrin