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

Incorporate Metadata to the Maintenance response #246

Closed
chadell opened this issue Oct 21, 2023 · 2 comments
Closed

Incorporate Metadata to the Maintenance response #246

chadell opened this issue Oct 21, 2023 · 2 comments
Labels
type: minor feature

Comments

@chadell
Copy link
Collaborator

chadell commented Oct 21, 2023

Environment

  • circuit_maintenance_parser version:

Proposed Functionality

Besides the data representing the Maintenance, the response could incorporate metadata to understand things like:

  • Which Provider has been used to parse it?
  • Which Parsers have been used?
  • If the parsing consumed some resources, a summary of the amount of them (for instance, Openai tokens)

Use Case
@chadell
Copy link
Collaborator Author

chadell commented Nov 15, 2023

working on #249

chadell added a commit that referenced this issue Nov 20, 2023
@chadell @glennmatthews
Add Metadata to every Maintenance, addresses issue #246 (#249)
9ba38f0 
* feat: ✨ OpenAI parser

* refinement in logic

* logic refinement

* Update README.md

Co-authored-by: Glenn Matthews <[email protected]>

* Update circuit_maintenance_parser/parser.py

Co-authored-by: Glenn Matthews <[email protected]>

* improve question

* make more explicit the text parsing

* Automate token management for local tests

* Make openai library an extra

* Adopt OpenAI library changes

* fix mypy

* feat: ✨ Metadata for every Maintenance

* fix tests

* update readme

* Add a boolean in Metadata to reflect LLM parsing

* Update circuit_maintenance_parser/parser.py

Co-authored-by: Glenn Matthews <[email protected]>

---------

Co-authored-by: Glenn Matthews <[email protected]>
@chadell
Copy link
Collaborator Author

chadell commented Nov 21, 2023

Completed by #249

@chadell chadell closed this as completed Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: minor feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chadell