GridOPTICS / GridPACK

https://www.gridpack.org/
47 stars 22 forks source link

Python module: RuntimeError: Cblock is not a transfer function #198

Open wperkins opened 9 months ago

wperkins commented 9 months ago

With the current develop branch (0edc74ec), the Python module unit tests fail with

======================================================================
ERROR: hadrec_test (tests.gridpack_test.GridPACKTester)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/d3g096/Projects/GridPACK-Wind/src/GridPACK/python/src/tests/gridpack_test.py", line 63, in hadrec_test
    hadapp.initializeDynSimu(busfaultlist)
RuntimeError: Cblock is not a transfer function
Check the value of the parameters

-------------------- >> begin captured stdout << ---------------------
NoPrint status: False
NoPrint status: True

--------------------- >> end captured stdout << ----------------------

This appears to be independent of the pybind11 version, which was suspected in #197 (I tried 2 versions).

wperkins commented 9 months ago

The source of the error is here: https://github.com/GridOPTICS/GridPACK/blob/0edc74eceb6cecb9cc5b6e590519bcf8a67b7c1e/src/applications/modules/dynamic_simulation_full_y/cblock.cpp#L24C1-L34C2 This was added by @abhyshr. I would guess the unit test input needs to be updated to account for this change.
@abhyshr, can you check the input for the Python unit test here: https://github.com/GridOPTICS/GridPACK/blob/0edc74eceb6cecb9cc5b6e590519bcf8a67b7c1e/python/src/tests/input_tamu500_step005.xml

wperkins commented 9 months ago

Apparently, Github's permalinks don't work for me.

wperkins commented 9 months ago

This should not have been closed. I put the wrong issue on #200.

wperkins commented 7 months ago

My solution to this is to simply remove the test from the Python API unit tests. In #209, I've commented it out. Eventually, this test should be fixed or the inputs (and hadrec.py script) deleted.