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

Add the semconvgen Library to PyPi (semantic-conventions package) #253

Closed
Kevin-Ngo opened this issue Jan 5, 2024 · 4 comments
Closed
Labels
enhancement New feature or request semconv/model Related to the data model or YAML format of the semantic convention generator semconv Related to the semantic convention generator.

Comments

@Kevin-Ngo
Copy link

  • There are some cases where using the SemanticConvention model outside of a CLI would be useful
  • Third-party scripts that perform validation on Semantics on top of the standard syntax validation found in the semconvgen models
  • Adding the package to PyPi makes it easily available for other applications/services to reference it as a dependency
@Oberon00 Oberon00 added enhancement New feature or request semconv Related to the semantic convention generator. semconv/model Related to the data model or YAML format of the semantic convention generator labels Jan 8, 2024
@Oberon00
Copy link
Member

Are we really sure we need this? We have enough compatibility problems and under-documentation with what Jinja templates can access. If we expose this, even more changes will be breaking changes.

At the moment, the issue claims that there are "some cases" where it would be useful. Please list some concrete use cases.

@Kevin-Ngo
Copy link
Author

The cases that I can think of, and one that I want to use, is wanting access to the syntax validation that is present in the SemanticConvention model for the sake of making my own code generation script based off semantic conventions. I don't really need to use the CLI, so having access to the model via a library would make it so that users can use the model (SemanticConvention) and its out of the box syntax validation on the YAML files.

@jsuereth
Copy link
Contributor

I don't think we're "stable" enough in semconv definition or YAML to expose this library. As part of the weaver project, I think we'll be formalizing the YAML a bit more and formalizing rules regarding, e.g. how to override attirbutes and such.

I'd be against folks depending on the state of things today outside of our hand-holding them through the binary we provide.

TL;DR: expect breaking python changes if you'r using it as a library.

@lmolkova
Copy link
Contributor

lmolkova commented Oct 9, 2024

closing this issue - we're sunsetting otel/semconvgen in favor of weaver #322 and won't make any improvements here.

@lmolkova lmolkova closed this as completed Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request semconv/model Related to the data model or YAML format of the semantic convention generator semconv Related to the semantic convention generator.
Projects
None yet
Development

No branches or pull requests

4 participants