You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, the C++ samples can (and do) contain use of deprecated methods. While these samples are compiled and run as part of scons test, we do not force deprecation warnings to be errors in these samples, and the output written to stdout and stederr is not included in the test comparisons for most of these tests, which means the warnings printed to the screen go unnoticed
Steps to reproduce
Run scons test
Run ./build/samples/cxx/bvp/blasius (may require fiddling with LD_LIBRARY_PATH)
See DeprecationWarning in the output
Behavior
Step 1 passes with no errors. Step 2 produces output starting with:
DeprecationWarning: OneDim::OneDim(vector<Domain1D*>): To be removed after Cantera 3.0; superseded by OneDim::OneDim(v
ector<shared_ptr<Domain1D>>).
DeprecationWarning: OneDim::addDomain(Domain1D*): To be removed after Cantera 3.0; superseded by OneDim::addDomain(shared_ptr<Domain1D>).
DeprecationWarning: Sim1D::Sim1D(vector<Domain1D*>&): To be removed after Cantera 3.0; superseded by Sim1D::Sim1D(vector<shared_ptr<Domain1D>>&).
..............................................................................
Attempt Newton solution of steady-state problem... failure.
Take 10 timesteps 0.0003844 0.7784
Attempt Newton solution of steady-state problem... success.
Problem solved on [6] point grid(s).
I already have a fix prepared for the specific warnings here that will be included in an upcoming PR, but thought it was helpful to track this gap in our testing separately.
The text was updated successfully, but these errors were encountered:
This issue does not affect the sample programs in the C++ tutorial (https://cantera.org/dev/userguide/cxx-tutorial.html) which are now included in the test suite as of #1657, because the regression tests for these make comparisons of the combined stdout and stderr of the samples.
However, nothing has changed for the other C++ sample programs.
Problem description
Currently, the C++ samples can (and do) contain use of deprecated methods. While these samples are compiled and run as part of
scons test
, we do not force deprecation warnings to be errors in these samples, and the output written tostdout
andstederr
is not included in the test comparisons for most of these tests, which means the warnings printed to the screen go unnoticedSteps to reproduce
scons test
./build/samples/cxx/bvp/blasius
(may require fiddling withLD_LIBRARY_PATH
)DeprecationWarning
in the outputBehavior
Step 1 passes with no errors. Step 2 produces output starting with:
System information
Additional context
I already have a fix prepared for the specific warnings here that will be included in an upcoming PR, but thought it was helpful to track this gap in our testing separately.
The text was updated successfully, but these errors were encountered: