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

feat: more transducers #110

Merged
merged 14 commits into from
Feb 15, 2022
Merged

feat: more transducers #110

merged 14 commits into from
Feb 15, 2022

Conversation

erdos
Copy link
Owner

@erdos erdos commented Feb 7, 2022

improve performance by the following changes

  • use transducers where possible
  • for dfs walk, find the initial node with DFS so zipper is not initialized where not needed

@codecov
Copy link

codecov bot commented Feb 7, 2022

Codecov Report

Merging #110 (b587461) into master (3aefbda) will increase coverage by 0.01%.
The diff coverage is 97.29%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #110      +/-   ##
==========================================
+ Coverage   93.07%   93.08%   +0.01%     
==========================================
  Files          25       25              
  Lines        1876     1894      +18     
  Branches      106      107       +1     
==========================================
+ Hits         1746     1763      +17     
  Misses         24       24              
- Partials      106      107       +1     
Impacted Files Coverage Δ
src/stencil/merger.clj 99.08% <80.00%> (-0.92%) ⬇️
src/stencil/cleanup.clj 93.61% <100.00%> (ø)
src/stencil/infix.clj 98.34% <100.00%> (-0.01%) ⬇️
src/stencil/model.clj 96.93% <100.00%> (ø)
src/stencil/postprocess/table.clj 80.18% <100.00%> (-0.18%) ⬇️
src/stencil/util.clj 98.71% <100.00%> (+0.50%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3aefbda...b587461. Read the comment docs.

@erdos erdos added enhancement New feature or request performance labels Feb 13, 2022
@erdos erdos merged commit cb2029e into master Feb 15, 2022
@erdos erdos deleted the feat-more-transducers branch February 15, 2022 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant