[samaraseeri]

I'm not familiar with openfoam and trying to compile a new solver which outputs the following error. Can anyone give me hint on what needs to be fixed. best


modeFFTuw.C:81:12: error: no viable conversion from 'dimensioned<Foam::Vector<double>>' to 'Foam::vector' (aka 'Vector<double>')
vector cornerCellCentre = ::Foam::max(mesh.C().internalField());
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/OpenFOAM/OpenFOAM-v2112/src/OpenFOAM/lnInclude/Vector.H:91:9: note: candidate constructor not viable: no known conversion from 'dimensioned<Foam::Vector<double>>' to 'const Foam::Vector<double> &' for 1st argument
Vector(const Vector&) = default;
^
/Users/OpenFOAM/OpenFOAM-v2112/src/OpenFOAM/lnInclude/Vector.H:100:16: note: candidate constructor not viable: no known conversion from 'dimensioned<Foam::Vector<double>>' to 'const Foam::zero' for 1st argument
inline Vector(const Foam::zero);
^
/Users/OpenFOAM/OpenFOAM-v2112/src/OpenFOAM/lnInclude/Vector.H:104:16: note: candidate template ignored: could not match 'VectorSpace' against 'dimensioned'
inline Vector(const VectorSpace<Vector<Cmpt2>, Cmpt2, 3>& vs);
^
/Users/OpenFOAM/OpenFOAM-v2112/src/OpenFOAM/lnInclude/Vector.H:110:25: note: explicit constructor is not a candidate
inline explicit Vector(Istream& is);
^
modeFFTuw.C:41:9: warning: 'sprintf' will always overflow; destination buffer has size 5, but format string expands to at least 13 [-Wfortify-source]
sprintf(filename,"%1s_sans_mode%1u",champ ,mode);
^
modeFFTuw.C:44:9: warning: 'sprintf' will always overflow; destination buffer has size 5, but format string expands to at least 8 [-Wfortify-source]
sprintf(filename,"%1s_mode%1u",champ ,mode);
^
2 warnings and 1 error generated.
make: *** [Make/darwin64ClangDPInt32Opt/modeFFTuw.o] Error 1

[LuckyTran]

Your units (dimensions) don't match and that's why it's refusing to build.

[samaraseeri]

I found that solver on the forums site https://www.cfd-online.com/Forums/at...ft-modefftuw.c. Can you guide me on how to match the units. Thanks

[samaraseeri]

Originally Posted by LuckyTran

Your units (dimensions) don't match and that's why it's refusing to build.

I found that solver on the forums site https://www.cfd-online.com/Forums/at...ft-modefftuw.c. Can you guide me on how to match the units. Thanks

[LuckyTran]

Check out the OpenFOAM guide on basic rules regarding dimensions.

In your case the problems start at

    vector cornerCellCentre = ::Foam::max(mesh.C().internalField());

You are taking the maximum cell centroid which is a dimensioned vector and trying to store it in a vector called cornerCellCentre. A (non-dimensioned) vector is not the same class as a dimensioned vector.

You cannot assign a dimensioned anything to a dimensionless anything because they are different classes. It is equivalent to trying to take an int and store it in a float. Your compiler can clearly see this upfront and that's why it refuses to build. There are simple workarounds such as gutting the dimensions or arbitrarily adding dimensions or recasting but if you managed to arrive here then it usually means you overlooked the basic I/O patterns in OpenFOAM in the first place.

[samaraseeri]

Originally Posted by LuckyTran

Check out the OpenFOAM guide on basic rules regarding dimensions.

In your case the problems start at

    vector cornerCellCentre = ::Foam::max(mesh.C().internalField());

You are taking the maximum cell centroid which is a dimensioned vector and trying to store it in a vector called cornerCellCentre. A (non-dimensioned) vector is not the same class as a dimensioned vector.

You cannot assign a dimensioned anything to a dimensionless anything because they are different classes. It is equivalent to trying to take an int and store it in a float. Your compiler can clearly see this upfront and that's why it refuses to build. There are simple workarounds such as gutting the dimensions or arbitrarily adding dimensions or recasting but if you managed to arrive here then it usually means you overlooked the basic I/O patterns in OpenFOAM in the first place.

I think the code compiled and attached above was correct, maybe some headers were missing or there are libraries that needs to be linked. It is not my code I'm just examining it to learn from it.

[LuckyTran]

I mean you clearly show an error from wmake which shows the code not compiling

[samaraseeri]

Originally Posted by LuckyTran

I mean you clearly show an error from wmake which shows the code not compiling

ok thanks for your reply. I'll check how to match the units then.