neuronets / trained-models

Trained TensorFlow models for 3D image processing
https://neuronets.dev/trained-models
22 stars 15 forks source link

spec.yaml is not updated correctly #105

Open hvgazula opened 1 year ago

hvgazula commented 1 year ago

https://github.com/neuronets/trained-models/actions/runs/6762690478/job/18378961087#step:8:1

The input arguments don't match the order of arguments the function is expecting.

gaiborjosue commented 1 year ago

Hello Harsha, yes. It seems it is a matter of removing "docker" from the default path in the workflow when calling the Python file. I will create a branch and solve this.

gaiborjosue commented 1 year ago

Hello @hvgazula, I am trying to solve this issue right now. However, the current workflow is the one separated in multiple jobs (not the one you refactored). Thus, do you think that this can be solved directly within the new refactored workflow? Or should I go ahead and solve it in the old workflow in a separate branch?

gaiborjosue commented 1 year ago

I checked your refactored workflow, and it seems this issue is already solved there.

hvgazula commented 1 year ago

I don't think it is resolved. Which workflow did you check?

gaiborjosue commented 1 year ago

https://github.com/hvgazula/trained-models/actions/runs/6831844963

It didn't fail, that is why I assume you fixed it.