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

General-use utils should have its own namespace #437

Open
JoelPasvolsky opened this issue Oct 30, 2020 · 2 comments
Open

General-use utils should have its own namespace #437

JoelPasvolsky opened this issue Oct 30, 2020 · 2 comments

Comments

@JoelPasvolsky
Copy link
Contributor

Generally useful decorator like cloud-client's @dwave.cloud.utils.retried or @dwave_networkx.utils.decorators.binary_quadratic_model_sampler that aren't limited in use to the particular repo that were written in should be under a generic dwave.utils or dwave.decorators namespace.

@arcondello
Copy link
Member

I know that @randomir has been suggesting a dwave.common package or similar. Unfortunately, there is no nice way to populate a namespace from multiple packages (or at least none that I know of, though I haven't re-checked recently). dwavesystems/dwave-ocean-sdk#21 is an attempt to create one using entry points.

@randomir
Copy link
Member

Yes, I still have dwave.common on my todo list. It would include:

  • logging stuff
  • profiling (like tictoc)
  • testing utils, like various assertX
  • misc commonly used utils like: @retried, @cached, @deprecated, etc.

Basically, utilities we use across multiple Ocean packages.

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

No branches or pull requests

3 participants