generated from ansys/template
-
Notifications
You must be signed in to change notification settings - Fork 7
feat: add thermic source #651
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
Draft
pluAtAnsys
wants to merge
28
commits into
main
Choose a base branch
from
feat/thermicsource
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 18 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
0f11fee
Pushed draft branch for thermic source
ansmcailler 87864cb
Merge branch 'main' into feat/thermicsource
9debcf0
Merge branch 'main' into MCR_thermicsource
pluAtAnsys ddf82a6
ci: auto fixes from pre-commit.com hooks.
pre-commit-ci[bot] a12a993
Merge remote-tracking branch 'origin/MCR_thermicsource' into feat/the…
5f00348
Re-adding full project.py script
ansmcailler 5466a49
ci: auto fixes from pre-commit.com hooks.
pre-commit-ci[bot] 6c6d0d7
chore: adding changelog file 651.added.md [dependabot-skip]
pyansys-ci-bot 3a9263d
run pre-commit to add missing docstring
dcbb88a
refactor the test code into example as temp location.
fd1f665
Merge remote-tracking branch 'origin/feat/thermicsource' into feat/th…
7de324e
add key to the Intensity class to be filled
ee721b9
remove no need to check if it is temperature field
76274b5
Fix issue with intensity template
echambla e5f8132
property and setter method
81f0f61
add unittest
5ebe14b
add example
ff25390
improve the unittest coverage
8092f7e
Merge branch 'main' into feat/thermicsource
pluAtAnsys 54501ef
Merge branch 'main' into feat/thermicsource
pluAtAnsys a0c6f78
Merge branch 'main' into feat/thermicsource
pluAtAnsys 47f7a83
Merge branch 'main' into feat/thermicsource
pluAtAnsys 2e8be76
Merge branch 'main' into feat/thermicsource
pluAtAnsys 9b60e42
Merge branch 'main' into feat/thermicsource
pluAtAnsys ff5d345
Finalized thermic source basic development (incl unit test)
ansmcailler 2855c91
ci: auto fixes from pre-commit.com hooks.
pre-commit-ci[bot] ae44b32
chore: adding changelog file 651.added.md [dependabot-skip]
pyansys-ci-bot 4e60a56
Merge branch 'main' into feat/thermicsource
pluAtAnsys File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| Add thermic source |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| from pathlib import Path | ||
|
|
||
| import ansys.speos.core as core | ||
| from ansys.speos.core.launcher import launch_local_speos_rpc_server | ||
| import ansys.speos.core.source as source | ||
|
|
||
| speos = launch_local_speos_rpc_server(version="252") | ||
| speos_file = Path.cwd() / "Speos Bench" / "Inverse.1.1.speos" / "Inverse.1.1.speos" | ||
| project = core.Project(speos=speos, path=speos_file) | ||
| mysource = project.find(name=".*", name_regex=True, feature_type=source.SourceThermic) | ||
| mysource[0].set_emissive_faces_temp(value=100) | ||
| mysource[0].commit() | ||
| mysimu = project.find(name=".*", name_regex=True, feature_type=core.simulation.SimulationInverse) | ||
| mysimu = mysimu[0] | ||
| new_source = project.create_source(name="test", feature_type=source.SourceThermic) | ||
| new_source.commit() | ||
| print(project) | ||
|
|
||
| # mysimu.compute_CPU() | ||
| # print(mysimu.result_list) | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please consider refactor this into unittest or example.
if needed, add the speos file used into the tests/assert folder