Skip to content

Latest commit

 

History

History
29 lines (21 loc) · 855 Bytes

POEM_026.md

File metadata and controls

29 lines (21 loc) · 855 Bytes
Raw

POEM ID: 026
Title: Remove support for factorial function in ExecComp
authors: [swryan]
Competing POEMs: N/A
Related POEMs: N/A
Associated implementation PR: OpenMDAO/OpenMDAO#1483

Status:

  • Active
  • Requesting decision
  • Accepted
  • Rejected
  • Integrated

Motivation

With version 1.5.0, SciPy enforces the integer datatype for arguments to the factorial function.

ExecComp is intended for continuous functions and relies on this assumption to calculate derivatives via complex step. The factorial function does not satisfy this requirement and thus is not supported.

Description

Use of the factorial function in an ExecComp is deprecated for SciPy versions up to 1.5.0 and will raise a RuntimeError for SciPy versions 1.5.0 and up, with an appropriate error message.