search

bjornaa / ladim1

LADIM - Lagrangian Advection and DIffusion Model
MIT License
12 stars 6 forks source link

Change/add "Pyyaml" as a requisite for ladim #5

Open mcarvajalino opened 4 years ago

mcarvajalino commented 4 years ago

Even if the "yaml" module is intalled in the environment, ladim fails to run showing a "cannot find yaml module" error (Python 3.7.6 in Miniconda 3). This can be corrected by installing the additional "pyyaml" parsing module. Should this be included in the model requisites?

https://github.com/bjornaa/ladim/blob/c0129614afd89aaafce9badc3e417f137f04d499/setup.py#L13

bjornaa commented 3 years ago

Hi Marcos,

You are correct. The package name is pyyaml in anaconda and pypi. The reason for the confusion is that the import statement is import yaml, import pyyaml gives a ModuleNotFoundError. The "yaml" will be replaced with "pyyaml" in the requirements.

Bjørn