Skip to content

Fix max load factor2 - #853

Merged
kbenoit merged 4 commits into
masterfrom
fix-max_load_factor2
Jul 15, 2017
Merged

Fix max load factor2#853
kbenoit merged 4 commits into
masterfrom
fix-max_load_factor2

Conversation

@koheiw

@koheiw koheiw commented Jul 14, 2017

Copy link
Copy Markdown
Collaborator

Optimize hash tables by setting max_load_factor in C++. In some cases, hash table construction becomes 10x faster, making lead time of tokens_*() much shorter.

@codecov

codecov Bot commented Jul 14, 2017

Copy link
Copy Markdown

Codecov Report

Merging #853 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #853      +/-   ##
==========================================
- Coverage   77.39%   77.38%   -0.02%     
==========================================
  Files         103      103              
  Lines        7787     7782       -5     
==========================================
- Hits         6027     6022       -5     
  Misses       1760     1760

@kbenoit

kbenoit commented Jul 15, 2017

Copy link
Copy Markdown
Collaborator

On Windows I'm still getting

* checking compiled code ... NOTE
File 'quanteda/libs/x64/quanteda.dll':
  Found no calls to: 'R_registerRoutines', 'R_useDynamicSymbols'

It is good practice to register native routines and to disable symbol
search.

See 'Writing portable packages' in the 'Writing R Extensions' manual.

Can you make sure all functions are registered in src/init.c?

@kbenoit
kbenoit merged commit 9c69e73 into master Jul 15, 2017
@kbenoit
kbenoit deleted the fix-max_load_factor2 branch August 10, 2017 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants