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

[FEA] User tool should pass --platform option to Profiling tool #678

Closed
cindyyuanjiang opened this issue Dec 6, 2023 · 0 comments · Fixed by #679
Closed

[FEA] User tool should pass --platform option to Profiling tool #678

cindyyuanjiang opened this issue Dec 6, 2023 · 0 comments · Fixed by #679
Assignees
Labels
feature request New feature or request user_tools Scope the wrapper module running CSP, QualX, and reports (python)

Comments

@cindyyuanjiang
Copy link
Collaborator

cindyyuanjiang commented Dec 6, 2023

Is your feature request related to a problem? Please describe.
User tool is not passing any platform information to the Profiling core tool. Therefore no matter which platform the user tool runs on, the Profiling tool is always running on the default onprem platform.

Describe the solution you'd like
User tool should pass --platform option/argument to Profiling tool.

@cindyyuanjiang cindyyuanjiang self-assigned this Dec 6, 2023
@cindyyuanjiang cindyyuanjiang changed the title [FEA] Add platform option to Profiling tool [FEA] User tool should pass --platform argument to Profiling tool Dec 6, 2023
@cindyyuanjiang cindyyuanjiang changed the title [FEA] User tool should pass --platform argument to Profiling tool [FEA] User tool should pass --platform option to Profiling tool Dec 6, 2023
@mattahrens mattahrens added the user_tools Scope the wrapper module running CSP, QualX, and reports (python) label Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request user_tools Scope the wrapper module running CSP, QualX, and reports (python)
Projects
None yet
2 participants