search

psu-efd / pyHMT2D

Python Hydraulic Modeling Tools - 2D
MIT License
73 stars 16 forks source link

Cross-platform support? #4

Closed mdbartos closed 1 year ago

mdbartos commented 3 years ago

Greetings,

Is this a Windows-only package?

When trying to import the module I get the following traceback:

>>> import pyHMT2D
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.7/site-packages/pyHMT2D/__init__.py", line 2, in <module>
    from .Hydraulic_Models_Data import *
  File "/usr/local/lib/python3.7/site-packages/pyHMT2D/Hydraulic_Models_Data/__init__.py", line 4, in <module>
    from .RAS_2D import *
  File "/usr/local/lib/python3.7/site-packages/pyHMT2D/Hydraulic_Models_Data/RAS_2D/__init__.py", line 2, in <module>
    from .HEC_RAS_Model import HEC_RAS_Model
  File "/usr/local/lib/python3.7/site-packages/pyHMT2D/Hydraulic_Models_Data/RAS_2D/HEC_RAS_Model.py", line 5, in <module>
    import win32com.client as win32
ModuleNotFoundError: No module named 'win32com'
psu-efd commented 3 years ago

Greetings,

Is this a Windows-only package?

When trying to import the module I get the following traceback:

>>> import pyHMT2D
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.7/site-packages/pyHMT2D/__init__.py", line 2, in <module>
    from .Hydraulic_Models_Data import *
  File "/usr/local/lib/python3.7/site-packages/pyHMT2D/Hydraulic_Models_Data/__init__.py", line 4, in <module>
    from .RAS_2D import *
  File "/usr/local/lib/python3.7/site-packages/pyHMT2D/Hydraulic_Models_Data/RAS_2D/__init__.py", line 2, in <module>
    from .HEC_RAS_Model import HEC_RAS_Model
  File "/usr/local/lib/python3.7/site-packages/pyHMT2D/Hydraulic_Models_Data/RAS_2D/HEC_RAS_Model.py", line 5, in <module>
    import win32com.client as win32
ModuleNotFoundError: No module named 'win32com'

Thanks for testing the package. Yes, this package is intended only to work on Windows as of now because both SRH-2D and HEC-RAS are only supported in Windows. Maybe I should add this to the README.