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.

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

Fix settings and low distance warning #18

Merged
merged 3 commits into from
Nov 14, 2022
Merged

Fix settings and low distance warning #18

merged 3 commits into from
Nov 14, 2022

Conversation

tautomer
Copy link
Collaborator

@tautomer tautomer commented Nov 8, 2022

Some diffs are because of black...

Apart from that, here are the changes.

  1. Local rc file was incorrectly parsed.
  2. Documentation on the rc files is now consistent with the code.
  3. Incorrect error message in array padding.
  4. Disable the low distance warning from sensitivity plotting.
  5. Update the change log (including the changes in the restart part).

tautomer and others added 3 commits November 6, 2022 23:24
@tautomer
2 minor fixes
074f8f6 
1. Local rc file is not correctly parsed.
2. Error message in tools.pad_np_array_to_length_with_zeros missing "f"
@tautomer
Fix a false low distance warning due to plotting
5262b8f
@tautomer
[GLOBAL] -> [GLOBALS] in the documentation
bb0f695
@lubbersnick
Copy link
Collaborator

Thank you, looks good

@lubbersnick lubbersnick merged commit fbbffb1 into lanl:development Nov 14, 2022
@tautomer tautomer deleted the fix_settings branch November 14, 2022 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tautomer @lubbersnick