-
Notifications
You must be signed in to change notification settings - Fork 14
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
v4.1: Elmer, M3DC1 fusion-io, docker clones, docker staged builds #36
Conversation
noticed bug in GUI mode with new logfiles. need to patch. only effects GUI mode (not TUI) |
@wingena you should download the v4.1 container, and download the v4.1 branch, and see if you can run your 3d plasmas. once you have verified everything is working as intended, approve this PR or comment with the error. thanks. |
@plasmapotential I have three comments:
Otherwise, looks okay and passes the test. |
One more thing: |
@Wingen thanks for the suggestions i did the following:
The speed during plasma3D is quite strange. Can you review the MAFOT build process I am using in the Dockerfile and buildMAFOT bash script to make sure everything looks ok? how many nodes / cores are you running this on (so i can scale your times up/down to my machine)? Dockerfile line that builds MAFOT: Line 86 in 03a6e91
buildMAFOT bash script that is called inside container to build: A couple of things i need from your side:
|
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.
reviewed all files
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.
Reviewed and approved. Removed D3D files until we learn if they are IP approved.
This PR is not only to merge the v4.1 branch into main, but also corresponds to a new docker container release, v4.1. The notable changes in this PR are: