[WIP] Pickle TypeMap

[ajtritt]

Motivation

Currently, it takes about 2.5 seconds to PyNWB. Pickling the TypeMap on build/install will help bring this down.

This is WIP to make sure it works with testing/deployment

@dorukozturk @jcfr Your thoughts/input on this would be greatly appreciated.

How to test the behavior?

Run test suite

Checklist

• Have you checked our Contributing document?
• Have you ensured the PR description clearly describes problem and the solution?
• Is your contribution compliant with our coding style ? This can be checked running flake8 from the source directory.
• Have you checked to ensure that there aren't other open Pull Requests for the same change?
• Have you included the relevant issue number using #XXX notation where XXX is the issue number ?

[codecov]

Codecov Report

Merging #1050 into dev will decrease coverage by 3.77%.
The diff coverage is 77.41%.

@@            Coverage Diff            @@
##              dev   #1050      +/-   ##
=========================================
- Coverage   71.17%   67.4%   -3.78%     
=========================================
  Files          37      37              
  Lines        2793    2807      +14     
  Branches      554     557       +3     
=========================================
- Hits         1988    1892      -96     
- Misses        679     789     +110     
  Partials      126     126

Impacted Files	Coverage Δ
src/pynwb/legacy/__init__.py	0% <0%> (-86.96%)	⬇️
src/pynwb/__init__.py	70.67% <82.75%> (-1.5%)	⬇️
src/pynwb/legacy/io/__init__.py	0% <0%> (-100%)	⬇️
src/pynwb/legacy/io/misc.py	0% <0%> (-83.34%)	⬇️
src/pynwb/legacy/io/image.py	0% <0%> (-66.67%)	⬇️
src/pynwb/legacy/io/base.py	0% <0%> (-35.9%)	⬇️
src/pynwb/legacy/io/epoch.py	0% <0%> (-35.3%)	⬇️
src/pynwb/legacy/io/behavior.py	0% <0%> (-33.34%)	⬇️
src/pynwb/legacy/io/icephys.py	0% <0%> (-31.25%)	⬇️
... and 9 more

---

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 0843e24...853e84f. Read the comment docs.

[codecov]

Codecov Report

Merging #1050 into dev will decrease coverage by 3.77%.
The diff coverage is 77.41%.

@@            Coverage Diff            @@
##              dev   #1050      +/-   ##
=========================================
- Coverage   71.17%   67.4%   -3.78%     
=========================================
  Files          37      37              
  Lines        2793    2807      +14     
  Branches      554     557       +3     
=========================================
- Hits         1988    1892      -96     
- Misses        679     789     +110     
  Partials      126     126

Impacted Files	Coverage Δ
src/pynwb/legacy/__init__.py	0% <0%> (-86.96%)	⬇️
src/pynwb/__init__.py	70.67% <82.75%> (-1.5%)	⬇️
src/pynwb/legacy/io/__init__.py	0% <0%> (-100%)	⬇️
src/pynwb/legacy/io/misc.py	0% <0%> (-83.34%)	⬇️
src/pynwb/legacy/io/image.py	0% <0%> (-66.67%)	⬇️
src/pynwb/legacy/io/base.py	0% <0%> (-35.9%)	⬇️
src/pynwb/legacy/io/epoch.py	0% <0%> (-35.3%)	⬇️
src/pynwb/legacy/io/behavior.py	0% <0%> (-33.34%)	⬇️
src/pynwb/legacy/io/icephys.py	0% <0%> (-31.25%)	⬇️
... and 9 more

---

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 0843e24...853e84f. Read the comment docs.

[rly]

The above code will not work with extensions that call register_class. This will add kwargs to __rct_kwargs but they are not registered into the __TYPE_MAP. Same for the register_map method.

[ajtritt]

Good catch... back to the drawing board.