Marco Mangano

@bernardopacini @sseraj as you mentioned you are working on testing, there might also be an opportunity to fix and test this issue (I also call myself in to work on...

What is the status of this PR? @ArshSaja are you going to address @anilyil comments about additional checks?

I don't have much to add to what @ewu63 said. I did some benchmarking at the beginning of my PhD using the Rosenbrock, Sellar, and [the scalable function on Sec.5.4...

Check the table at the top of this page with the tested dependencies: https://mdolab-mach-aero.readthedocs-hosted.com/en/latest/installInstructions/install3rdPartyPackages.html I believe both the Python and PETSc versions might be too new, type errors are very...

@sanjan98 and I will take a look soon. I believe it is time to make a new release for ADflow. There are no breaking changes here, right @sseraj ? Likely...

Ok, so we should go for a minor release overall, given all these changes since the latest one. We can wait for #362 to be merged since it's minor and...

Chiming in just to say that this is some incredible detective work! Updating the Wolfe condition here is a no-brainer. I wonder if we should look at other codes and...

Specific test errors aside, I have mixed feelings about moving the actual pyOptSparse wrapper to the parOpt repo. On the one hand, outsourcing it could make things a bit easier...

[This test](https://github.com/gjkennedy/pyoptsparse/blob/d36d430a85e3f48b641528ceacb3e33486e9c9b9/tests/test_require_mpi_env_var.py#L45-L76) is failing because the docker images are not updated to the latest `paropt` version, but I believe we could remove this test altogether at this point?

The changes make sense, but I wonder if it is more elegant to have something like: ``` if nDV == 0: pass ```