Skip to content

FEAT: add geometry-mechanical-dpf workflow #11

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

Merged
merged 25 commits into from
Jun 24, 2024
Merged

Conversation

dipinknair
Copy link
Contributor

@dipinknair dipinknair commented Jun 20, 2024

  • pyansys geometry
  • pymechanical
  • pydpf
  • cicd - geo
  • cicd - pymechanical
  • cicd - dpf

To verify , download output folder from cicd and check if geometry (pcb.pmdb) mechanical files (pcb.mechdb) and images are present. For dpf workflow, the model information is printed in cicd.

dipinknair and others added 4 commits June 20, 2024 11:22
Co-authored-by: Roberto Pastor Muela <37798125+RobPasMue@users.noreply.github.com>
@RobPasMue
Copy link
Member

Can you try and replicate the existing CI/CD workflows? I will help you out as needed.

@dipinknair dipinknair force-pushed the feat/pymechanical branch 2 times, most recently from 9451bde to bccbc3d Compare June 21, 2024 16:20
@dipinknair dipinknair requested review from RobPasMue and MaxJPRey June 22, 2024 00:07
Co-authored-by: Maxime Rey <87315832+MaxJPRey@users.noreply.github.com>
@RobPasMue
Copy link
Member

@dipinknair - next time, please do not ignore our suggestions. You resolved @MaxJPRey's comments without implementing the fixes.

Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor suggestions but overall it looks good!

Co-authored-by: Roberto Pastor Muela <37798125+RobPasMue@users.noreply.github.com>
@RobPasMue
Copy link
Member

@dipinknair - please add a description to the main README as well with this new workflow. Look at the existing ones for other workflows

Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Let's merge it

@RobPasMue RobPasMue merged commit d0bb357 into main Jun 24, 2024
7 checks passed
@RobPasMue RobPasMue deleted the feat/pymechanical branch June 24, 2024 14:56
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

Successfully merging this pull request may close these issues.

3 participants