Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

spec.yaml is not updated correctly #105

Open
hvgazula opened this issue Nov 9, 2023 · 5 comments · May be fixed by #109
Open

spec.yaml is not updated correctly #105

hvgazula opened this issue Nov 9, 2023 · 5 comments · May be fixed by #109
Assignees
Labels
bug Something isn't working

Comments

@hvgazula
Copy link
Contributor

hvgazula commented Nov 9, 2023

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.

@hvgazula hvgazula added the bug Something isn't working label Nov 9, 2023
@gaiborjosue
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Contributor

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

@hvgazula
Copy link
Contributor Author

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

@gaiborjosue
Copy link
Contributor

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants