Commit 868e5892 authored by Podhorszki, Norbert's avatar Podhorszki, Norbert
Browse files

bpls -l for a single scalar (not over time) for a BP5 file showed 0. Fixed to... 

bpls -l for a single scalar (not over time) for a BP5 file showed 0. Fixed to use variable->MinMax() but also fixed in BP5Deserializer to not segfault if the scalar is not present in every step of the file.
parent 9b13a08f

source/adios2/toolkit/format/bp5/BP5Deserializer.cpp

+6 −1
Original line number Diff line number Diff line
@@ -1812,8 +1812,11 @@ bool BP5Deserializer::VariableMinMax(const VariableBase &Var, const size_t Step, 
                writer_meta_base =

                    GetMetadataBase(VarRec, RelStep, WriterRank++);

            }

            if (writer_meta_base)

            {

                ApplyElementMinMax(MinMax, VarRec->Type, writer_meta_base);

            }

        }

        else if (VarRec->OrigShapeID == ShapeID::LocalValue)

        {

            for (size_t WriterRank = 0; WriterRank < writerCohortSize;
@@ -1822,10 +1825,12 @@ bool BP5Deserializer::VariableMinMax(const VariableBase &Var, const size_t Step, 
                void *writer_meta_base =

                    GetMetadataBase(VarRec, RelStep, WriterRank);

                if (writer_meta_base)

                {

                    ApplyElementMinMax(MinMax, VarRec->Type, writer_meta_base);

                }

            }

        }

    }

    return true;

}

source/utils/bpls/bpls.cpp

+2 −1
Original line number Diff line number Diff line
@@ -1232,7 +1232,8 @@ int printVariableInfo(core::Engine *fp, core::IO *io, 
        if (longopt && !timestep)

        {

            fprintf(outf, " = ");

            print_data(&variable->m_Value, 0, adiosvartype, false);

            auto mm = variable->MinMax();

            print_data(&mm.second, 0, adiosvartype, false);

        }

        fprintf(outf, "\n");