[serialization][test] failures in serialization causing regressions

newer
Re: [boost] Asynchronous library...

Barend Gehrels

26 Nov 2016 26 Nov '16

9:31 p.m.

The regression matrix in develop of Boost.Geometry contains many new compilation failures due to:

1) : D:\t08\run\boost_root\boost/serialization/array.hpp(25) : error C2039: 'array' : is not a member of 'std'

2): ../boost/numeric/ublas/storage.hpp: In member function ???void boost::numeric::ublas::unbounded_array::serialize(Archive&, unsigned int)???: ../boost/numeric/ublas/storage.hpp:331:18: error: ???make_array??? is not a member of ???boost::serialization??? ar & serialization::make_array(data_, s);and a

The master is still OK.

Can this be solved?

Ping! This is still the case! Robert, can you explain this? It is (of course) not only geometry failing, also ublas or (if there are) other libraries with ublas dependencies: http://www.boost.org/development/tests/develop/developer/numeric-ublas.html All yellow! Thanks, Barend

Show replies by date

Robert Ramey

27 Nov 27 Nov

1:22 a.m.

New subject: [serialization][test] failures in serialization causing regressions

On 11/26/16 1:31 PM, Barend Gehrels wrote:

...

...
The regression matrix in develop of Boost.Geometry contains many new > compilation failures due to:

1) : D:\t08\run\boost_root\boost/serialization/array.hpp(25) : error C2039: 'array' : is not a member of 'std'

2): ../boost/numeric/ublas/storage.hpp: In member function ???void boost::numeric::ublas::unbounded_array::serialize(Archive&, unsigned int)???: ../boost/numeric/ublas/storage.hpp:331:18: error: ???make_array??? is not a member of ???boost::serialization??? ar & serialization::make_array(data_, s);and a

The master is still OK.

Can this be solved?

Ping! This is still the case! Robert, can you explain this?

It is (of course) not only geometry failing, also ublas or (if there are) other libraries with ublas dependencies: http://www.boost.org/development/tests/develop/developer/numeric-ublas.html

All yellow!

Thanks, Barend

I'll look into this asap Robert Ramey

Robert Ramey

1:23 a.m.

New subject: [serialization][test] failures in serialization causing regressions

On 11/26/16 1:31 PM, Barend Gehrels wrote:

...

...
The regression matrix in develop of Boost.Geometry contains many new > compilation failures due to:

1) : D:\t08\run\boost_root\boost/serialization/array.hpp(25) : error C2039: 'array' : is not a member of 'std'

2): ../boost/numeric/ublas/storage.hpp: In member function ???void boost::numeric::ublas::unbounded_array::serialize(Archive&, unsigned int)???: ../boost/numeric/ublas/storage.hpp:331:18: error: ???make_array??? is not a member of ???boost::serialization??? ar & serialization::make_array(data_, s);and a

The master is still OK.

Can this be solved?

Ping! This is still the case! Robert, can you explain this?

It is (of course) not only geometry failing, also ublas or (if there are) other libraries with ublas dependencies: http://www.boost.org/development/tests/develop/developer/numeric-ublas.html

All yellow!

It looks to me that these test are using std::array and the compiler is not supporting C++11+ I don't think this has anything to do with the serialization library.

...

Thanks, Barend

Robert Ramey

2:03 a.m.

New subject: [serialization][test] failures in serialization causing regressions

On 11/26/16 5:23 PM, Robert Ramey wrote:

...

On 11/26/16 1:31 PM, Barend Gehrels wrote:

...
...
The regression matrix in develop of Boost.Geometry contains many new > compilation failures due to:

1) : D:\t08\run\boost_root\boost/serialization/array.hpp(25) : error C2039: 'array' : is not a member of 'std'

2): ../boost/numeric/ublas/storage.hpp: In member function ???void boost::numeric::ublas::unbounded_array::serialize(Archive&, unsigned int)???: ../boost/numeric/ublas/storage.hpp:331:18: error: ???make_array??? is not a member of ???boost::serialization??? ar & serialization::make_array(data_, s);and a

The master is still OK.

Can this be solved?

Ping! This is still the case! Robert, can you explain this?

It is (of course) not only geometry failing, also ublas or (if there are) other libraries with ublas dependencies: http://www.boost.org/development/tests/develop/developer/numeric-ublas.html

All yellow!

It looks to me that these test are using std::array and the compiler is not supporting C++11+

I don't think this has anything to do with the serialization library.

OK - I've looked at this a little bit. I made adjustments to more easily support all C++ compilers from C++98 to C++14 + and to factor common code from vector, valarray, native arrays, and who knows what else. One of the things this entailed was to put the implementation of an "array_wrapper" in a separate file. This is more general than the concept of an "array". It also includes the definition of "make_array" which perhaps should be called make_array_wrapper. So I think the best would be for ublas / storage to replace #include boost/serialization/array.hpp with #include boost/serialization/array_wrapper.hpp A word of caution - I'm actually not quite done with this but I don't think the array_wapper concept/definition will change so I think it is safe to make this change. Unfortunately, this will mean that serialization and ublas will have to be merged to the master at the same time. I realize that this is not ideal - but still ... Robert Ramey

Barend Gehrels

9:24 p.m.

New subject: [serialization][ublas][test] failures in serialization causing regressions

Hi Robert, Op 27-11-2016 om 3:03 schreef Robert Ramey:

...

On 11/26/16 5:23 PM, Robert Ramey wrote:

...
On 11/26/16 1:31 PM, Barend Gehrels wrote:

...
...
The regression matrix in develop of Boost.Geometry contains many new > compilation failures due to:

1) : D:\t08\run\boost_root\boost/serialization/array.hpp(25) : error C2039: 'array' : is not a member of 'std'

2): ../boost/numeric/ublas/storage.hpp: In member function ???void boost::numeric::ublas::unbounded_array::serialize(Archive&, unsigned int)???: ../boost/numeric/ublas/storage.hpp:331:18: error: ???make_array??? is not a member of ???boost::serialization??? ar & serialization::make_array(data_, s);and a

The master is still OK.

Can this be solved?

Ping! This is still the case! Robert, can you explain this?

It is (of course) not only geometry failing, also ublas or (if there are) other libraries with ublas dependencies: http://www.boost.org/development/tests/develop/developer/numeric-ublas.html

All yellow!

It looks to me that these test are using std::array and the compiler is not supporting C++11+

I don't think this has anything to do with the serialization library.

OK - I've looked at this a little bit.

I made adjustments to more easily support all C++ compilers from C++98 to C++14 + and to factor common code from vector, valarray, native arrays, and who knows what else. One of the things this entailed was to put the implementation of an "array_wrapper" in a separate file. This is more general than the concept of an "array". It also includes the definition of "make_array" which perhaps should be called make_array_wrapper.

So I think the best would be for ublas / storage to replace

#include boost/serialization/array.hpp

with

#include boost/serialization/array_wrapper.hpp

A word of caution - I'm actually not quite done with this but I don't think the array_wapper concept/definition will change so I think it is safe to make this change. Unfortunately, this will mean that serialization and ublas will have to be merged to the master at the same time. I realize that this is not ideal - but still ...

Thanks for investigating. Your advice to replace it in ublas looks good - however, I don't know who is responsible for it and if it will be done on short term. Maybe we can prepare a PR for them. And indeed, then we have the merge problem. Are there people from Ublas on the list? I modified the subject to draw more attention. Regards, Barend

Robert Ramey

10:58 p.m.

New subject: [serialization][ublas][test] failures in serialization causing regressions

On 11/27/16 1:24 PM, Barend Gehrels wrote:

...

Hi Robert,

Op 27-11-2016 om 3:03 schreef Robert Ramey:

...
On 11/26/16 5:23 PM, Robert Ramey wrote:

...
On 11/26/16 1:31 PM, Barend Gehrels wrote:

...
...
The regression matrix in develop of Boost.Geometry contains many new > compilation failures due to:

1) : D:\t08\run\boost_root\boost/serialization/array.hpp(25) : error C2039: 'array' : is not a member of 'std'

2): ../boost/numeric/ublas/storage.hpp: In member function ???void boost::numeric::ublas::unbounded_array::serialize(Archive&, unsigned int)???: ../boost/numeric/ublas/storage.hpp:331:18: error: ???make_array??? is not a member of ???boost::serialization??? ar & serialization::make_array(data_, s);and a

The master is still OK.

Can this be solved?

Ping! This is still the case! Robert, can you explain this?

It is (of course) not only geometry failing, also ublas or (if there are) other libraries with ublas dependencies: http://www.boost.org/development/tests/develop/developer/numeric-ublas.html

All yellow!

It looks to me that these test are using std::array and the compiler is not supporting C++11+

I don't think this has anything to do with the serialization library.

OK - I've looked at this a little bit.

I made adjustments to more easily support all C++ compilers from C++98 to C++14 + and to factor common code from vector, valarray, native arrays, and who knows what else. One of the things this entailed was to put the implementation of an "array_wrapper" in a separate file. This is more general than the concept of an "array". It also includes the definition of "make_array" which perhaps should be called make_array_wrapper.

So I think the best would be for ublas / storage to replace

#include boost/serialization/array.hpp

with

#include boost/serialization/array_wrapper.hpp

A word of caution - I'm actually not quite done with this but I don't think the array_wapper concept/definition will change so I think it is safe to make this change. Unfortunately, this will mean that serialization and ublas will have to be merged to the master at the same time. I realize that this is not ideal - but still ...

Thanks for investigating. Your advice to replace it in ublas looks good - however, I don't know who is responsible for it and if it will be done on short term. Maybe we can prepare a PR for them. And indeed, then we have the merge problem. Are there people from Ublas on the list? I modified the subject to draw more attention.

Regards, Barend

Actually I really need to look at this a little more carefully. Originally boost/serialization/array.hpp was for C++ native arrays. Now it's morphed to be std::array. Over the years stuff got added: fast serialization of bit serializable arrays, c++11 <array>, array_wrapper concept, std::vector, valarray and who knows what else. I'm generally against modification of the interface but do refactor from time to time to deal with all this. And of course this is what happens when people actually start to depend upon the library. As I said, I need to look at this some more - perhaps it's possible / better to adjust the contents of boost/serialization/array.hpp.

Barend Gehrels

6 Dec 6 Dec

8:22 p.m.

New subject: [serialization][ublas][test] failures in serialization causing regressions

Hi Robert, Op 27-11-2016 om 23:58 schreef Robert Ramey:

...

On 11/27/16 1:24 PM, Barend Gehrels wrote:

...
Hi Robert,

Op 27-11-2016 om 3:03 schreef Robert Ramey:

...
On 11/26/16 5:23 PM, Robert Ramey wrote:

...
On 11/26/16 1:31 PM, Barend Gehrels wrote:

...
...
The regression matrix in develop of Boost.Geometry contains many new > compilation failures due to:

1) : D:\t08\run\boost_root\boost/serialization/array.hpp(25) : error C2039: 'array' : is not a member of 'std'

2): ../boost/numeric/ublas/storage.hpp: In member function ???void boost::numeric::ublas::unbounded_array::serialize(Archive&, unsigned int)???: ../boost/numeric/ublas/storage.hpp:331:18: error: ???make_array??? is not a member of ???boost::serialization??? ar & serialization::make_array(data_, s);and a

The master is still OK.

Can this be solved?

Ping! This is still the case! Robert, can you explain this?

It is (of course) not only geometry failing, also ublas or (if there are) other libraries with ublas dependencies: http://www.boost.org/development/tests/develop/developer/numeric-ublas.html

All yellow!

It looks to me that these test are using std::array and the compiler is not supporting C++11+

I don't think this has anything to do with the serialization library.

OK - I've looked at this a little bit.

I made adjustments to more easily support all C++ compilers from C++98 to C++14 + and to factor common code from vector, valarray, native arrays, and who knows what else. One of the things this entailed was to put the implementation of an "array_wrapper" in a separate file. This is more general than the concept of an "array". It also includes the definition of "make_array" which perhaps should be called make_array_wrapper.

So I think the best would be for ublas / storage to replace

#include boost/serialization/array.hpp

with

#include boost/serialization/array_wrapper.hpp

A word of caution - I'm actually not quite done with this but I don't think the array_wapper concept/definition will change so I think it is safe to make this change. Unfortunately, this will mean that serialization and ublas will have to be merged to the master at the same time. I realize that this is not ideal - but still ...

Thanks for investigating. Your advice to replace it in ublas looks good - however, I don't know who is responsible for it and if it will be done on short term. Maybe we can prepare a PR for them. And indeed, then we have the merge problem. Are there people from Ublas on the list? I modified the subject to draw more attention.

Regards, Barend

Actually I really need to look at this a little more carefully.

Originally boost/serialization/array.hpp was for C++ native arrays. Now it's morphed to be std::array. Over the years stuff got added: fast serialization of bit serializable arrays, c++11 <array>, array_wrapper concept, std::vector, valarray and who knows what else. I'm generally against modification of the interface but do refactor from time to time to deal with all this. And of course this is what happens when people actually start to depend upon the library. As I said, I need to look at this some more - perhaps it's possible / better to adjust the contents of boost/serialization/array.hpp.

For your information: Because it is only in develop and not in master, it is currently not a problem for users. It was inconvenient for us (checking regressions) but luckily there were some columns not affected, and in the meantime I merged to master again and that looks good. Per release 1.64 we, as Boost.Geometry, will stop using Ublas. We currently use it only internally for matrix inversion and products, and will do that with Boost.QVM. That is basically already applied (but not yet submitted), but it is too late for this release, and not necessary because master is OK. Regards, Barend

Robert Ramey

8:49 p.m.

New subject: [serialization][ublas][test] failures in serialization causing regressions

On 12/6/16 12:22 PM, Barend Gehrels wrote:

...

For your information:

Because it is only in develop and not in master, it is currently not a problem for users. It was inconvenient for us (checking regressions) but luckily there were some columns not affected, and in the meantime I merged to master again and that looks good.

Per release 1.64 we, as Boost.Geometry, will stop using Ublas. We currently use it only internally for matrix inversion and products, and will do that with Boost.QVM. That is basically already applied (but not yet submitted), but it is too late for this release, and not necessary because master is OK.

I've checked in a fix into the develop branch. I havn't yet seen the results of the ublas tests but I'm hopeful that this problem will disappear. For me it's not really an ublas problem but a serialization library backwards compatibility problem. The only problem now is that the fix breaks the test_array in the serialization library due to some behavior of the build system which we haven't been unable to pin down. Very annoying. I'm curious as to why you replaced ublas with QVM. Why was one superior to the other?

...

Regards, Barend

_______________________________________________ Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost

Edward Diener

27 Nov 27 Nov

2:49 p.m.

New subject: [serialization][test] failures in serialization causing regressions

On 11/26/2016 8:23 PM, Robert Ramey wrote:

...

On 11/26/16 1:31 PM, Barend Gehrels wrote:

...
...
The regression matrix in develop of Boost.Geometry contains many new > compilation failures due to:

1) : D:\t08\run\boost_root\boost/serialization/array.hpp(25) : error C2039: 'array' : is not a member of 'std'

2): ../boost/numeric/ublas/storage.hpp: In member function ???void boost::numeric::ublas::unbounded_array::serialize(Archive&, unsigned int)???: ../boost/numeric/ublas/storage.hpp:331:18: error: ???make_array??? is not a member of ???boost::serialization??? ar & serialization::make_array(data_, s);and a

The master is still OK.

Can this be solved?

Ping! This is still the case! Robert, can you explain this?

It is (of course) not only geometry failing, also ublas or (if there are) other libraries with ublas dependencies: http://www.boost.org/development/tests/develop/developer/numeric-ublas.html

All yellow!

It looks to me that these test are using std::array and the compiler is not supporting C++11+

The config macro 'BOOST_NO_CXX11_HDR_ARRAY' determines whether or not std::array is available.

...

I don't think this has anything to do with the serialization library.

...
Thanks, Barend

2912

Age (days ago)

2922

Last active (days ago)

List overview

Download

8 comments

3 participants

participants (3)

Barend Gehrels
Edward Diener
Robert Ramey