Thanks for the correction, John. I had meant to say "float". At my end, I have temporarily changed all "floats" to "double" in the metis/parmetis files in libmesh, but this is a temporary fix.
It would be better to source it from the respective headers. But that raises another point: as of now, it seems like the metis.h in contrib has a hardcoded value of real width to be 32. I understand that this saves memory space, but would there be a motivation to keep this scalar type to be consistent with the rest of the library? -Manav On Fri, Mar 13, 2015 at 1:18 PM, John Peterson <[email protected]> wrote: > > > On Fri, Mar 13, 2015 at 11:54 AM, Manav Bhatia <[email protected]> > wrote: > >> Correction: I had meant to write "real as the scalar type" (instead of >> "scalar as the real type") in my previous message. >> > > Hmm... I don't see a single instance of Real in either metis_partitioner.h > or parmetis_partitioner.h. Perhaps you are referring to: > > std::vector<float> _tpwgts; > std::vector<float> _ubvec; > > which use float? I agree that should probably be changed, along with > using Metis' idx_t for indexing instead of hard-coding int. > > -- > John > ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ Libmesh-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/libmesh-users
