Inconsistency in patsy requirement (README vs requirements file))

alan-turing-institute / mogp-emulator

Package for fitting Gaussian Process Emulators to multiple output computer simulation results.

MIT License

47 stars 14 forks source link

Inconsistency in patsy requirement (README vs requirements file)) #241

Open krishnakumarg1984 opened 2 years ago

krishnakumarg1984 commented 2 years ago

In the README, it says patsy is strongly recommended and talks about the optional requirements text file.

However, the requirements.txt file implies that patsy is a requirement and not just a strong recommendation.

It would be helpful to newcomers if this discrepancy can be resolved suitable, and the files made consistent.

edaub commented 2 years ago

Yes, this is correct. patsy is now a requirement and the readme need to be updated. When installing from pip this should automatically be handled as pip uses the requirements.txt file for installation.