Skip to content

Performance improvements #21

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
Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Performance improvements #21

wants to merge 17 commits into from

Conversation

@daniel-statsig
Copy link
Contributor

@daniel-statsig daniel-statsig commented Jan 24, 2024 •
edited
Loading

  • Remove Sorbet
  • Add get_all_evaluations method
  • Reduce throw away class usage
  • Avoid iteration where possible
  • Freeze string literals

daniel-statsig and others added 17 commits January 19, 2024 13:45
@daniel-statsig
Remove Sorbet (#267)
dd4d9dd
@tore-statsig @daniel-statsig
symbolize (#268)
678e8c6 
Co-authored-by: Daniel Loomb <[email protected]>
@sroyal-statsig
Setup Get All Evals (#273)
0658ab7
@sroyal-statsig
update
d545fcc
@sroyal-statsig
Clean up client Init Helper (#277)
9eba87c
@daniel-statsig
Update client_initialize_helpers.rb
fb62326
@daniel-statsig
Remove all Sorbet comments. Update classes to not use Sorbet (#283)
38726a1
@daniel-statsig
Enable GH Actions for Perf branch
2d49ec9
@daniel-statsig
Merge branch 'main' into performance-improvements
232169c
@daniel-statsig
Fix Tests, revert breaking change
9c22980
@daniel-statsig
Fix gcir
70ba51a
@daniel-statsig @tore-statsig
Remove inline strings v2 (#278)
2aee3bf 
Co-authored-by: tore-statsig <[email protected]>
@tore-statsig @daniel-statsig
Pass evaluation result by reference through evaluator (#282)
206148b 
Co-authored-by: Daniel Loomb <[email protected]>
@tore-statsig
Classes not hashes (#275)
c9229cb
@tore-statsig
Find unsupported configs up front (#286)
711f91c
@daniel-statsig
Rm eval details 4 (#290)
bd32404
@tore-statsig
Add eval_gate helper for nested gate checks (#289)
e5c1cb6
@github-actions
Copy link

Code Coverage

Package Line Rate Health
ruby-sdk 84% ➖
Summary 84% (1558 / 1850) ➖

Minimum allowed line rate is 80%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@daniel-statsig @tore-statsig @sroyal-statsig