search

AdvancedPhotonSource / xdesign

Tools for designing x-ray phantoms and experiments.
https://xdesign.readthedocs.io
Other
24 stars 16 forks source link

Make demos work #65

Closed jahomax closed 5 years ago

carterbox commented 5 years ago

Hi @jahomax, thanks for fixing these demos. I found an additional demo that needs to be fixed, so could you please check the box on the right side of this pull request that says, "Allow edits from maintainers"?

jahomax commented 5 years ago

Hi @carterbox , yes there was an additional demo, but without Anaconda or MKL I couldn't install Tomopy so couldn't fix that one. The "Allow edits from maintainers" box is ticked already it seems.

carterbox commented 5 years ago

I figured it out! Thanks for checking on that. Is there a reason that you don't want to use 'conda' for managing your packages and environments?

jahomax commented 5 years ago

The reason I use pip is that it comes from python.org's python installer. It's a non-profit organization with no paid components, which is the way I like to operate. I build it from source and it has no propietary-licensed stuff like Intel's MKL.

carterbox commented 5 years ago

@jahomax Would you be interested in a build of TomoPy without MKL? It would have reduced functionality.

jahomax commented 5 years ago

In principle that would appeal to me, but my tomography work is rather geometrically different, meaning I can't use standard packages and methods. But I know of other researchers who like their work to not depend on non-free licenses.

On Fri, 29 Mar 2019 at 04:44, Daniel Ching notifications@github.com wrote:

@jahomax https://github.com/jahomax Would you be interested in a build of TomoPy without MKL? It would have reduced functionality.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/tomography/xdesign/pull/65#issuecomment-477865305, or mute the thread https://github.com/notifications/unsubscribe-auth/Arzx1tD6bSXuGZiwVghUNhR7FfZwIMEJks5vbZosgaJpZM4cOf7K .