Add 3 examples for Parametric Workflows #250

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.

Sign up for GitHub

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

Jump to bottom

Merged

sujal-tipnis merged 21 commits into main from doc/parametric_new

Apr 6, 2022

Contributor

sujal-tipnis commented Mar 31, 2022

No description provided.


          Add 3 examples for Parametric Workflows

db9ad63

sujal-tipnis requested review from dnwillia-work, mkundu1, seanpearsonuk, smf2001, wescott2 and ypatel-qa

March 31, 2022 22:17


          Add 3 examples for Parametric Workflows indentation change

309a986

dnwillia-work reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_1.py Outdated Show resolved Hide resolved


          remove pandas dataframe display

0355bf4

sujal-tipnis commented

View reviewed changes

examples/01-parametric/parametric_static_mixer_1.py Outdated Show resolved Hide resolved

Contributor Author

sujal-tipnis commented Mar 31, 2022

@mkundu1 @dnwillia-work @seanpearsonuk I tried building the documentation locally for these scripts and that completed successfully. But while building the documentation on Github, the build fails. Any possible reason that you think this might fail on GitHub?

akaszynski reviewed

View reviewed changes

doc/source/conf.py Show resolved Hide resolved

akaszynski reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_1.py Show resolved Hide resolved

akaszynski requested a review from PipKat

April 1, 2022 11:12

Contributor

akaszynski commented Apr 1, 2022

Pinging @PipKat for review.

PipKat reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_1.py Outdated Show resolved Hide resolved

PipKat reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_1.py Outdated Show resolved Hide resolved

PipKat reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_1.py Outdated Show resolved Hide resolved

PipKat reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_1.py Outdated Show resolved Hide resolved

PipKat reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_1.py Outdated Show resolved Hide resolved

PipKat reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_2.py Outdated Show resolved Hide resolved

PipKat reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_2.py Outdated Show resolved Hide resolved

PipKat reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_2.py Outdated Show resolved Hide resolved

PipKat reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_3.py Outdated Show resolved Hide resolved

PipKat reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_3.py Outdated Show resolved Hide resolved

seanpearsonuk reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_2.py Outdated Show resolved Hide resolved

seanpearsonuk reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_3.py Outdated Show resolved Hide resolved

seanpearsonuk reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_3.py Outdated Show resolved Hide resolved

seanpearsonuk reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_3.py Outdated Show resolved Hide resolved

seanpearsonuk reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_3.py Outdated Show resolved Hide resolved


          Disable archive operation as it is failing in server

78ce69d

Contributor

mkundu1 commented Apr 4, 2022

@sujal-tipnis I've disabled the ParametricProject.archive call as it is failing in the docker server. Opened #254 to investigate the issue.

There is also an error from ParametricSession.__del__ which is ignored in the CI (#255).

The CI is now passing in the current branch. You can now merge it.

mkundu1 approved these changes

View reviewed changes

sujal-tipnis and others added 15 commits

April 4, 2022 12:17


          Update examples/01-parametric/parametric_static_mixer_1.py

f5731d0

Co-authored-by: Kathy Pippert <[email protected]>


          Update examples/01-parametric/parametric_static_mixer_1.py

b0e0cc8

Co-authored-by: Kathy Pippert <[email protected]>


          Update examples/01-parametric/parametric_static_mixer_1.py

0c5f771

Co-authored-by: Kathy Pippert <[email protected]>


          Update examples/01-parametric/parametric_static_mixer_1.py

87e8fb4

Co-authored-by: Kathy Pippert <[email protected]>


          Update examples/01-parametric/parametric_static_mixer_1.py

17362ee

Co-authored-by: Kathy Pippert <[email protected]>


          Update examples/01-parametric/parametric_static_mixer_2.py

8b7a951

Co-authored-by: Kathy Pippert <[email protected]>


          Update examples/01-parametric/parametric_static_mixer_2.py

90d47d1

Co-authored-by: Kathy Pippert <[email protected]>


          Update examples/01-parametric/parametric_static_mixer_3.py

5f9abe3

Co-authored-by: Kathy Pippert <[email protected]>


          Update examples/01-parametric/parametric_static_mixer_2.py

ab0f965

Co-authored-by: Kathy Pippert <[email protected]>


          Update examples/01-parametric/parametric_static_mixer_1.py

6759a77

Co-authored-by: Dan Williams <[email protected]>


          made formatting changes to the script

027c028


          fixed style

a910289


          minor variable name changes

3ac1c3e


          minor style change

87ec17c


          Re-enable project archive (#259)

b297bac

seanpearsonuk reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_3.py Outdated Show resolved Hide resolved

seanpearsonuk reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_3.py Outdated Show resolved Hide resolved

seanpearsonuk reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_3.py Outdated Show resolved Hide resolved

seanpearsonuk reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_3.py Outdated Show resolved Hide resolved

seanpearsonuk reviewed

View reviewed changes

examples/01-parametric/parametric_static_mixer_1.py Show resolved Hide resolved

U-ANSYS\stipnis added 2 commits

April 5, 2022 13:56


          make variable name changes

2722eab


          make variable name changes(2)

44ae87f

seanpearsonuk approved these changes

View reviewed changes

ypatel-qa approved these changes

View reviewed changes

sujal-tipnis merged commit e6900ed into main

sujal-tipnis deleted the doc/parametric_new branch

April 6, 2022 13:39

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

dnwillia-work dnwillia-work left review comments

PipKat PipKat left review comments

seanpearsonuk seanpearsonuk approved these changes

mkundu1 mkundu1 approved these changes

smf2001 Awaiting requested review from smf2001

wescott2 Awaiting requested review from wescott2

+2 more reviewers

akaszynski akaszynski left review comments

ypatel-qa ypatel-qa approved these changes

Labels

None yet