Telecommunication-Telemedia-Assessment / bitstream_mode3_p1204_3

Open source reference implementation of ITU-T P.1204.3
Other
58 stars 15 forks source link

Update Dockerfile #26

Closed plebreton closed 2 years ago

plebreton commented 2 years ago

Currently the docker does not seems to work anymore as spicy and sklearn are not available. I am not sure if you want to specify a version with the requirements file

slhck commented 2 years ago

I'm not sure this is the correct way to go, as the main project uses poetry to handle all dependencies. I think that either:

  1. the requirements.txt file should be updated to install everything, or
  2. the Dockerfile should use poetry to install all dependencies

I will try an alternative, hold on please :)

slhck commented 2 years ago

I made a fix, please test recent master!

plebreton commented 2 years ago

Thanks, it is fixed!