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

In config file, name must be the same as aedt file name #226

Open
hui-zhou-a opened this issue Mar 13, 2023 Discussed in #223 · 4 comments
Open

In config file, name must be the same as aedt file name #226

hui-zhou-a opened this issue Mar 13, 2023 Discussed in #223 · 4 comments
Assignees

Comments

@hui-zhou-a
Copy link
Collaborator

Discussed in #223

Originally posted by ring630 March 9, 2023
In config file, name must be the same as aedt file name. Otherwise, it cannot find the results.

image
@hui-zhou-a hui-zhou-a self-assigned this Mar 13, 2023
@hui-zhou-a
Copy link
Collaborator Author

@beliaev-maksim I created a issue from the discusson 😉. Validation check is a quick fix, but I prefer another solution. The root cause is that simulation_data.py grabs project name to name the .prof file. aedt_test_runner.py look for .prof by the aedt name.

@beliaev-maksim
Copy link
Collaborator

That is the architectural change

I will leave it to @Samuelopez-ansys and @boyang2022 to decide))

@Samuelopez-ansys
Copy link
Member

@ring630 We can change it, but I would prefer to keep it like it, let's discuss later about it

@boyang2022
Copy link
Collaborator

@beliaev-maksim I created a issue from the discusson 😉. Validation check is a quick fix, but I prefer another solution. The root cause is that simulation_data.py grabs project name to name the .prof file. aedt_test_runner.py look for .prof by the aedt name.

@ring630 , what is the problem with the project name?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants