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

Visualization merging #138

Closed
jpmorgan98 opened this issue Dec 8, 2023 · 4 comments
Closed

Visualization merging #138

jpmorgan98 opened this issue Dec 8, 2023 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@jpmorgan98
Copy link
Collaborator

I am trying to merge the visualizer into main but am having some difficulties

This was referenced Dec 8, 2023
@jpmorgan98
Copy link
Collaborator Author

jpmorgan98 commented Dec 8, 2023

Background

@RohanPankaj added a visualization capabilities as a REU project over summer 2023. I am trying to merge it and recon with the alterations in MC/DC and with some of the new dependency.

In the current visualization is handled via Netgen a full fat multiphyscis FEA solver that has x86 builds in conda. The problem is that Netgen currently requires a manual build on ARM MacOS and has no listed support for ppc64 (Power9). We don't need visualization as a strict requirement for MC/DC so I might make the configuration and testing of Netgen a non-standard option requiring additional flags to our current install script

Next Steps

  • Handle merge between new MC/DC and old MC/DC
  • Configure Netgen on my local machine validate visualizer with new MC/DC version
  • Understanding Netgen as a dependency and install requirements

@jpmorgan98 jpmorgan98 self-assigned this Dec 8, 2023
@jpmorgan98 jpmorgan98 added the enhancement New feature or request label Dec 8, 2023
@spasmann
Copy link
Collaborator

spasmann commented Dec 8, 2023

Curious do we know if the other dependencies,ngsolve and distinctipy, have similar issues?

@jpmorgan98
Copy link
Collaborator Author

The dependencies have all been added and seem to be working on MAC and windows. I had to do some bug hunting to make sure all tests pass but as of PR #143 things seem to be working. When that PR is approved I will close this issue

@jpmorgan98
Copy link
Collaborator Author

ngsolve is a pip only command for osz-arm64 (M1/2/3)

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

No branches or pull requests

2 participants