Skip to content
Snippets Groups Projects

Catch signal error in dynesty

Merged Moritz Huebner requested to merge catch_signal_error_dynesty into master
All threads resolved!

This was preventing dynesty from running on Windows systems because these signal attributes don't exist on Windows systems.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Before merging this, I'd like to see it verified that this has no impact on running on an LDG cluster. While on the face of it I can't see any reason it should, if it does it effects a lot of people.

  • I can have a look into that.

  • Matthew David Pitkin mentioned in merge request !755 (merged)

    mentioned in merge request !755 (merged)

  • Colm Talbot changed milestone to %0.6.7

    changed milestone to %0.6.7

  • changed milestone to %0.6.8

  • Moritz Huebner resolved all threads

    resolved all threads

  • Moritz Huebner added 53 commits

    added 53 commits

    • 1ef62581...0140d1e7 - 51 commits from branch master
    • fdc17676 - Merge remote-tracking branch 'origin/master' into catch_signal_error_dynesty
    • 8ca3af9c - Changed the log-level of the signal attribute check to be debug

    Compare with previous version

  • Moritz Huebner added 1 commit

    added 1 commit

    Compare with previous version

  • Colm Talbot approved this merge request

    approved this merge request

  • Colm Talbot unapproved this merge request

    unapproved this merge request

  • Gregory Ashton approved this merge request

    approved this merge request

  • Colm Talbot approved this merge request

    approved this merge request

  • Gregory Ashton mentioned in commit 19d44d79

    mentioned in commit 19d44d79

  • Please register or sign in to reply
    Loading