Fix HEAppE prepare job operator
Hi, thanks for your great work on the LEXIS/HEAppE Airflow components! While trying out the new HEAppEPrepareJobOperatorFullContext
, I encountered two issues (that were also easily resolved):
- The
HEAppEPrepareJobOperatorFullContext
was passing aWalltimeLimit
argument to theHEAppEPrepareJobParametersAPISpec
, but this did not exist in theHEAppEPrepareJobParametersAPISpec
class. I replaced it with theWaitingLimit
argument, as that seemed to be used in other places and theWalltimeLimit
argument can be set per individual task. - The
HEAppEPrepareJobParametersAPISpec
always replaced theMaxCores
argument with the static number 128. This meant you could never allocate more nodes than this static number. I simply removed this override and that seemed to do the trick.
Please let me know if these fixes are in line with how the classes are meant to be used!