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

script_args should be an optional argument on run_discovery_script_file argument #870

Closed
2 tasks done
XIAO-GUO-commint opened this issue Dec 4, 2023 · 2 comments · Fixed by #873
Closed
2 tasks done
Assignees
Labels
bug Something isn't working

Comments

@XIAO-GUO-commint
Copy link

🔍 Before submitting the issue

  • I have searched among the existing issues
  • I am using a Python virtual environment

🐞 Description of the bug

I want to run a script in SpaceClaim using run_discovery_script_file, but I'm unsure what to include in script_args. I have reviewed the original documentation, but it did not provide guidance on how to manage this.

from ansys.geometry.core import launch_modeler_with_spaceclaim

modeler_discovery = launch_modeler_with_spaceclaim()

modeler_discovery.run_discovery_script_file(file_path=r'E:\2023project\nncx\muti-optimize\Mesh_Gem\1.scscript',script_args=script_args)
image

📝 Steps to reproduce

I want to run a script in SpaceClaim using run_discovery_script_file, but I'm unsure what to include in script_args. I have reviewed the original documentation, but it did not provide guidance on how to manage this.

💻 Which operating system are you using?

Windows

📀 Which ANSYS version are you using?

23r2

🐍 Which Python version are you using?

3.10

📦 Installed packages

numpy
pandas
ansys-geometry...
@XIAO-GUO-commint XIAO-GUO-commint added the bug Something isn't working label Dec 4, 2023
@RobPasMue
Copy link
Member

Hi @XIAO-GUO-commint - if you do not want to pass in any script arguments then you do not need to pass any. For now, pass in an "empty dictionary" (i.e. {}).

We have to change the method to make script_args as optional. @AlejandroFernandezLuces can you take care of this? It is an easy fix. We will do a patch release right afterwards

@RobPasMue RobPasMue changed the title Bug located in ... script_args should be an optional argument on run_discovery_script_file argument Dec 4, 2023
@AlejandroFernandezLuces
Copy link
Contributor

Hi @XIAO-GUO-commint - if you do not want to pass in any script arguments then you do not need to pass any. For now, pass in an "empty dictionary" (i.e. {}).

We have to change the method to make script_args as optional. @AlejandroFernandezLuces can you take care of this? It is an easy fix. We will do a patch release right afterwards

Sure! I'll take a look.

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
Development

Successfully merging a pull request may close this issue.

3 participants