Skip to content

WIP: Compiled on rhel7 cluster #1

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
Open
AreWeDreaming wants to merge 3 commits into
base: master
Choose a base branch
from
Open

WIP: Compiled on rhel7 cluster #1

AreWeDreaming wants to merge 3 commits into from

Conversation

@AreWeDreaming
Copy link

@AreWeDreaming AreWeDreaming commented Nov 2, 2022

  • Changed MPI script from python2 to python3
  • Made compliant with GIZA
  • Added makefile for gnu rhel system
  • Changed how ADC is linked
  • Added makefile for separated ADC lib

@AreWeDreaming
Compiled on rhel7 cluster
4ef0df9 
- Changed MPI script from python2 to python3
- Made compliant with GIZA
- Added makefile for gnu rhel system
- Changed how ADC is linked
- Added makefile for separated ADC lib
@AreWeDreaming
Copy link
Author

AreWeDreaming commented Nov 2, 2022

I ran cqlinput_eoved_.16_multi-flux-surface_test1.0 and compared it against the result in the zipped folder test_results_201026/eoved_.16_multi-flux-surface.test1.0.nc. Results from nccmp below. There are some large differences, but these occur only for small quantities. An exception is restp which shows large differences.

 DIFFER : VARIABLE : curra : POSITION : [0,0] : VALUES : 2.98139e-15 <> 5.23442e-15 : PERCENT : 43.0426
DIFFER : VARIABLE : curra : POSITION : [0,1] : VALUES : 1.99748e-14 <> 6.4017e-15 : PERCENT : 67.9511
DIFFER : VARIABLE : curra : POSITION : [0,2] : VALUES : 1.24587e-14 <> 1.27942e-14 : PERCENT : 2.62209
DIFFER : VARIABLE : curra : POSITION : [0,3] : VALUES : 4.3725e-14 <> 2.58402e-14 : PERCENT : 40.9029
DIFFER : VARIABLE : curtor : POSITION : [0,0] : VALUES : -1.96502e-13 <> 2.99871e-13 : PERCENT : 165.529
DIFFER : VARIABLE : curtor : POSITION : [0,1] : VALUES : -3.17091e-13 <> 2.51748e-13 : PERCENT : 179.393
DIFFER : VARIABLE : curtor : POSITION : [0,2] : VALUES : 7.60512e-14 <> 1.8743e-13 : PERCENT : 59.4242
DIFFER : VARIABLE : curtor : POSITION : [0,3] : VALUES : -7.82152e-14 <> 3.68704e-13 : PERCENT : 121.214
DIFFER : VARIABLE : ccurtor : POSITION : [0,0] : VALUES : -2.77137e-10 <> 4.22923e-10 : PERCENT : 165.529
DIFFER : VARIABLE : ccurtor : POSITION : [0,1] : VALUES : -4.8046e-09 <> 4.01742e-09 : PERCENT : 183.616
DIFFER : VARIABLE : ccurtor : POSITION : [0,2] : VALUES : -7.24626e-10 <> 1.40726e-08 : PERCENT : 105.149
DIFFER : VARIABLE : ccurtor : POSITION : [0,3] : VALUES : -1.61357e-08 <> 8.67199e-08 : PERCENT : 118.607
DIFFER : VARIABLE : curpol : POSITION : [0,0] : VALUES : -1.46767e-19 <> 2.23972e-19 : PERCENT : 165.529
DIFFER : VARIABLE : curpol : POSITION : [0,1] : VALUES : -2.53623e-17 <> 2.01359e-17 : PERCENT : 179.393
DIFFER : VARIABLE : curpol : POSITION : [0,2] : VALUES : 7.77409e-18 <> 1.91594e-17 : PERCENT : 59.4242
DIFFER : VARIABLE : curpol : POSITION : [0,3] : VALUES : -3.39906e-18 <> 1.60231e-17 : PERCENT : 121.214
DIFFER : VARIABLE : ccurpol : POSITION : [0,0] : VALUES : -3.28578e-15 <> 5.01424e-15 : PERCENT : 165.529
DIFFER : VARIABLE : ccurpol : POSITION : [0,1] : VALUES : -1.40309e-12 <> 1.11636e-12 : PERCENT : 179.565
DIFFER : VARIABLE : ccurpol : POSITION : [0,2] : VALUES : -7.31057e-13 <> 2.77259e-12 : PERCENT : 126.367
DIFFER : VARIABLE : ccurpol : POSITION : [0,3] : VALUES : -2.43011e-13 <> 4.71958e-13 : PERCENT : 151.49
DIFFER : VARIABLE : currm_e : POSITION : [0,0] : VALUES : -1.96502e-13 <> 2.99871e-13 : PERCENT : 165.529
DIFFER : VARIABLE : currm_e : POSITION : [0,1] : VALUES : -3.17091e-13 <> 2.51748e-13 : PERCENT : 179.393
DIFFER : VARIABLE : currm_e : POSITION : [0,2] : VALUES : 7.60512e-14 <> 1.8743e-13 : PERCENT : 59.4242
DIFFER : VARIABLE : currm_e : POSITION : [0,3] : VALUES : -7.82152e-14 <> 3.68704e-13 : PERCENT : 121.214
DIFFER : VARIABLE : curr : POSITION : [0,0] : VALUES : -1.96896e-13 <> 3.00471e-13 : PERCENT : 165.529
DIFFER : VARIABLE : curr : POSITION : [0,1] : VALUES : -4.09356e-13 <> 3.25001e-13 : PERCENT : 179.393
DIFFER : VARIABLE : curr : POSITION : [0,2] : VALUES : 1.31719e-13 <> 3.24625e-13 : PERCENT : 59.4242
DIFFER : VARIABLE : curr : POSITION : [0,3] : VALUES : -3.40836e-13 <> 1.60669e-12 : PERCENT : 121.214
DIFFER : VARIABLE : restp : POSITION : [0,0] : VALUES : -0.174091 <> 0.11408 : PERCENT : 165.529
DIFFER : VARIABLE : restp : POSITION : [0,1] : VALUES : -0.0864816 <> 0.108928 : PERCENT : 179.393
DIFFER : VARIABLE : restp : POSITION : [0,2] : VALUES : 0.300486 <> 0.121924 : PERCENT : 59.4242
DIFFER : VARIABLE : restp : POSITION : [0,3] : VALUES : -0.230664 <> 0.048932 : PERCENT : 121.214
DIFFER : VARIABLE : restnp : POSITION : [0,0] : VALUES : -0.174091 <> 0.11408 : PERCENT : 165.529
DIFFER : VARIABLE : restnp : POSITION : [0,1] : VALUES : -0.0864816 <> 0.108928 : PERCENT : 179.393
DIFFER : VARIABLE : restnp : POSITION : [0,2] : VALUES : 0.300486 <> 0.121924 : PERCENT : 59.4242
DIFFER : VARIABLE : restnp : POSITION : [0,3] : VALUES : -0.230664 <> 0.048932 : PERCENT : 121.214

@AreWeDreaming AreWeDreaming changed the title Compiled on rhel7 cluster WIP: Compiled on rhel7 cluster Nov 2, 2022
smithsp
smithsp approved these changes Nov 9, 2022
View reviewed changes
mpi/doparallel.py
if c=="'":
state = 0
if string.strip(s)=='':
if s.strip=='':
Copy link

@smithsp smithsp Nov 9, 2022

Choose a reason for hiding this comment

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

Suggested change
if s.strip=='':
if s.strip()=='':

smithsp
smithsp reviewed Jul 18, 2023
View reviewed changes
pltinit.f
! CALL PGQINF('USER', PG_VAL, PG_L)
! WRITE (*,*) 'PGPLOT user: ', PG_VAL(1:PG_L)
! CALL PGQINF('NOW', PG_VAL, PG_L)
! WRITE (*,*) 'PGPLOT time now: ', PG_VAL(1:PG_L)
Copy link

@smithsp smithsp Jul 18, 2023

Choose a reason for hiding this comment

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

@AreWeDreaming Was this causing the problems of @wdeshazer?

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

Reviewers

1 more reviewer

@smithsp smithsp smithsp approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AreWeDreaming @smithsp