daquinterop / Py_DSSATTools

A Python library for crop modeling using DSSAT
GNU General Public License v3.0
45 stars 16 forks source link

Simulation fails with new release (2.1.2) #25

Closed daquinterop closed 7 months ago

daquinterop commented 7 months ago
          Hi @daquinterop!

Thanks for merging it. I am having an error running the model now. For some reason, when I try to run any simulations, I get this error attached in the screenshot below. Any idea why this might be happening? The ERROR.OUT file isn't being outputted.

image

Thank you again!

_Originally posted by @osherboudara-work in https://github.com/daquinterop/Py_DSSATTools/issues/19#issuecomment-1912651577_

osherboudara-work commented 7 months ago

https://github.com/daquinterop/Py_DSSATTools/issues/19#issuecomment-1912879326

@daquinterop, replying to your message from the link above. I have tested the edited library that you provided through the git clone command and the simulations are working for me across the existing crops (tested tomato and soybean and both work). The simulation for wheat also worked but I am getting an error on the available_cultivars() function. I am still able to set the Crop function to wheat which allows the simulation to execute but it doesn't allow me to view the cultivars via available_cultivars(). I have attached a screenshot of the error.

image
daquinterop commented 7 months ago

Hi @osherboudara-work, I just fixed that problem. You can install it from the repo again. Please let me know if it works to push the latest changes to PyPI. Thanks for reporting this issue

osherboudara-work commented 7 months ago

Hi @daquinterop!

I just reinstalled the library from the repo and the simulations for wheat (including the available_cultivars() function) work! I would say it is safe to push the changes to PyPI.

Thank you for all your hard work on this library! Let me know when you make the update to PyPI.

daquinterop commented 7 months ago

Thanks for letting me know. I just made the update to PyPI