search

ericmjonas / pybm3d

Python wrapper around bm3d
GNU General Public License v3.0
133 stars 29 forks source link

import pybm3d into namespace #2

Closed ericmjonas closed 7 years ago

timmeinhardt commented 7 years ago

@ericmjonas What was your intention behind this issue?

ericmjonas commented 7 years ago

I think at the time the cython wrapper had to be manually imported in an awkward fashion, and the thought was to get it into the top-level module namespace for easier use?

timmeinhardt commented 7 years ago

The lates commit of the develop branch https://github.com/ericmjonas/pybm3d/commit/9a49a2491ed4058cf35fae3e79b6718c5a058193 changes the interface and imports the bm3d function directly into the package's namespace. Is this what you had in mind?

ericmjonas commented 7 years ago

Yes that’s awesome !

On Wed, Nov 8, 2017 at 8:46 PM Tim Meinhardt notifications@github.com wrote:

The lates commit of the develop branch 9a49a24 https://github.com/ericmjonas/pybm3d/commit/9a49a2491ed4058cf35fae3e79b6718c5a058193 changes the interface and imports the bm3d function directly into the package's namespace. Is this what you had in mind?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ericmjonas/pybm3d/issues/2#issuecomment-342794116, or mute the thread https://github.com/notifications/unsubscribe-auth/AACPEtMoH7smtMBSMVh-y6pQadq3JNg-ks5s0ZSrgaJpZM4FgGNz .