Skip to content

Usability Discussion #195

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

Closed
reyang opened this issue Oct 2, 2019 · 3 comments
Closed

Usability Discussion #195

reyang opened this issue Oct 2, 2019 · 3 comments
Assignees
Labels
discussion Issue or PR that needs/is extended discussion. usability
Milestone

Comments

@reyang
Copy link
Member

reyang commented Oct 2, 2019

With the alpha v0.1 release we've managed to have a hard-to-use SDK 🥇 , which has been demonstrated in #191.

This is a little bit deviated from the Zen of Python, as well as most of the collaborators' intention ❤️ .

Let's see if together we can do something to make the API/SDK more friendly to users:

  1. Be Python idiomatic.
  2. Make it work by default (this is already arguable).
  3. Make it as simple as possible, but no simpler.
  4. Make it self explanatory (stretch goal).
@reyang reyang added the discussion Issue or PR that needs/is extended discussion. label Oct 2, 2019
@reyang reyang added this to the Beta Release milestone Oct 2, 2019
@lzchen
Copy link
Contributor

lzchen commented Oct 2, 2019

The PR link does not work.

@codeboten
Copy link
Contributor

This update to the ruby implementation went in last week:
open-telemetry/opentelemetry-ruby#171

@c24t
Copy link
Member

c24t commented Mar 18, 2020

#466 went a long way to solving several usability problems in the library and #327 will make similar changes to integrations. There's still more work to do to make the library easy to use, but we'll track that work in more specific issues.

@c24t c24t closed this as completed Mar 18, 2020
srikanthccv pushed a commit to srikanthccv/opentelemetry-python that referenced this issue Nov 1, 2020
* fix(basic-tracer): add timestamp for event

* fix: use performance.now()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Issue or PR that needs/is extended discussion. usability
Projects
None yet
Development

No branches or pull requests

6 participants