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

Optimize memory in span_map #36

Open
callegarimattia opened this issue Jan 16, 2024 · 0 comments
Open

Optimize memory in span_map #36

callegarimattia opened this issue Jan 16, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@callegarimattia
Copy link
Contributor 

          @callegarimattia I identified one place we could optimise. In Span Map instead of storing all "HotCharacters" in the Span, we can only store the str values. This way we will save some memory.

Can you create an issue after merging this PR and we could work on it?

The function in question is: extract_spans - which you already flagged to be deprecated

Originally posted by @krishnasism in #34 (comment)
@callegarimattia callegarimattia added the enhancement New feature or request label Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@callegarimattia