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

Polyaxon CLI should raise an error for invalid input/ouputs names with dots . #1515

Open
polyaxon-team opened this issue Oct 12, 2022 · 0 comments

Comments

@polyaxon-team
Copy link
Contributor

Current behavior

The CLI currently allows users to pass inputs/outputs with dots ., also the platform allows the run to be scheduled. However the interpolation engine does not allow reusing the param's variable name, especially when using DAGs or Joins, since the parser uses the dot . get extract the variables required.

Enhancement

There are two options:

  • Add a validation on the parsing level to show an error to the user before they submit the operation to the platform, to prevent any confusion.
  • Allow using [] as an alternative solution to getting params/inputs/outputs values instead of . and update the documentation to show how it can be used.
@polyaxon-team polyaxon-team changed the title Polyaxon CLI should raise an error for invalid input/ouputs names Polyaxon CLI should raise an error for invalid input/ouputs names with dots . Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant