search

ercius / openNCEM

A collection of packages and tools for electron microscopy data analysis supported by the National Center for Electron Microscopy facility of the Molecular Foundry
GNU General Public License v3.0
58 stars 28 forks source link

Change genfire import line #23

Closed ercius closed 4 years ago

ercius commented 4 years ago

https://github.com/ercius/openNCEM/blob/1e93ed6f4383d81a4b332145281d7fa085084b91/ncempy/edstomo/DoGenfire.py#L1

@ZGainsforth Can you change the line referenced above to be something like import genfire.reconstruct You will probbaly have to change gf.reconstruct to genfire.reconstruct later.

I would try it but I figure you would be able to test it and confirm it works.

This should avoid importing the GUI portion of the genfire package. Here is what the genfire.__init__ imports:

from . import utility
from . import reconstruct
from . import fileio
from . import gui

I dont think you need utility, fileio or gui and then ncempy will build even on headless machines. I think this will fix #22.

ercius commented 4 years ago

@ZGainsforth Thanks for getting that done. Merged.