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

modify.F: fix bug overwriting crucial parameters from comm.1 #16

Merged
merged 1 commit into from
Jun 29, 2023

Conversation

scaedufax
Copy link
Contributor

modify.F containsa bug overwriting N from the namelist input.
This leads to errors if a runs is continued, but N has changed during the run (escapers, binaries, ...).

KZ(16) & KZ(17) allow auto-adjustment of values which are overwritten by the namelist input on reruns as well.

This MR changes in modify.F:

  • force N from comm.1 on restart.
  • Check for auto-adjustment settings KZ(16) and KZ(17) and
    conditionally restore RMIN, DTMIN, ECLOSE, ETAI, ETAR and ETAU from
    comm.1 file.
  • moved restart parameters WRITE statement below restore segment
    for correct display of restart parameters.

 - force N from comm.1 on restart.
 - Check for auto-adjustment settings KZ(16) and KZ(17) and
   conditionally restore RMIN, DTMIN, ECLOSE, ETAI, ETAR and ETAU from
   comm.1 file.
 - moved restart parameters WRITE statement below restore comm.1 values
   for correct display of restart parameters.
@spurzem spurzem merged commit 3b16f9a into nbody6ppgpu:dev Jun 29, 2023
@spurzem
Copy link
Member

spurzem commented Jun 29, 2023

Thanks Uli for finding and fixing this.

@scaedufax scaedufax deleted the modify-namelist-bug branch June 29, 2023 09:25
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.

2 participants