Skip to content
Snippets Groups Projects

make multinest temporary files optional and work with MPI/resuming

Merged Colm Talbot requested to merge multinest-temporary-directory-optional into master

This wound up being a bigger change than I expected, but I've tested that this now

  • allows users to turn off the temporary directory usage
  • correctly returns the files to the expected directory when interrupted/finished
  • clears up the temporary directory when interrupted/finished
  • disables the temporary directory usage when running with MPI

I've generally used shutil.move rather than copying and removing as I had all kinds of issues with directories properly being removed.

The other kind of subtle thing is that pymultinest already has a signal handling function which can only be accessed by using importlib rather than a standard import.

Closes #485 (closed)

Edited by Colm Talbot

Merge request reports

Pipeline #124996 passed

Pipeline passed for b3d5eebe on multinest-temporary-directory-optional

Test coverage 58.00% (0.00%) from 1 job
Approved by

Merged by Gregory AshtonGregory Ashton 4 years ago (May 9, 2020 2:59am UTC)

Merge details

  • Changes merged into master with fec351e2 (commits were squashed).
  • Deleted the source branch.

Pipeline #125678 failed

Pipeline failed for fec351e2 on master

Test coverage 58.00% (0.00%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading