search

pnlbwh / CNN-Diffusion-MRIBrain-Segmentation

CNN based brain masking
Other
14 stars 10 forks source link

py2 to py3 conversion #1

Closed tashrifbillah closed 4 years ago

tashrifbillah commented 4 years ago

Follow up from work group on 1/29/2020:

@tashrifbillah suggested building the packages in requirements.txt using py3 compiler.

Unused packages such as:

opencv-python>=3.4.1.15
nilearn>=0.5.0

can be removed.

If there are only a few keras incompatibility of the current software against py3, we can work on to remove them. If there are many (unlikely), we shall devise a plan.

cc: @yrathi @sbouix

tashrifbillah commented 4 years ago

Hi @SenthilCaesar , any update on building the packages in requirements.txt using py3 compiler?

senthilcaesar commented 4 years ago

Hi @tashrifbillah , Yes the model seems to work fine with py36 compiler . I am still testing few other things. Will keep up posted on the progress.

senthilcaesar commented 4 years ago

py2 to py3 conversion completed

tashrifbillah commented 4 years ago

See commit https://github.com/pnlbwh/CNN-Diffusion-MRIBrain-Segmentation/commit/761b8d005b8775fef32cbc1c350da4fcde291775