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

Some small performance improvements #140

Merged
merged 1 commit into from
Jul 18, 2023
Merged

Some small performance improvements #140

merged 1 commit into from
Jul 18, 2023

Conversation

charleskawczynski
Copy link
Owner

  • Avoid materializing array
  • Use let with anonymous func over loop
  • Reorder bool

Avoid materializing array

Use let with anonymous func over loop

Avoid materialize, reorder bool
@charleskawczynski
Copy link
Owner Author

bors r+

@codecov
Copy link

codecov bot commented Jul 18, 2023

Codecov Report

Merging #140 (f6eb072) into main (ff60127) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #140      +/-   ##
==========================================
+ Coverage   88.38%   88.40%   +0.02%     
==========================================
  Files          10       10              
  Lines        1093     1095       +2     
==========================================
+ Hits          966      968       +2     
  Misses        127      127              
Impacted Files Coverage Δ
src/table.jl 98.78% <100.00%> (+<0.01%) ⬆️
src/transactions.jl 91.89% <100.00%> (+0.04%) ⬆️

@bors
Copy link
Contributor

bors bot commented Jul 18, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 567dc78 into main Jul 18, 2023
19 checks passed
@bors bors bot deleted the ck/perf3 branch July 18, 2023 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant