These are chat archives for symengine/symengine

27th
Jun 2015
Shivam Vats
@shivamvats
Jun 27 2015 06:51
@certik I am sorry I've been inactive in symengine for too long. I will discuss with @Sumith1896 on ways to design poly/series classes. This is relevant to Sympy too as we now need to design a ring series class for it .
Can we have a meeting sometime today @Sumith1896 ?
Sumith Kulal
@Sumith1896
Jun 27 2015 07:04
I'll get back after some time.
Isuru Fernando
@isuruf
Jun 27 2015 14:35
@Sumith1896 can you fix #484 ?
Francesco Biscani
@bluescarni
Jun 27 2015 14:50
@Sumith1896 @certik I have written a blog post about the integer class. It explains the underlying ideas of the implementation, and I thought it might be useful to you in case you want to implement something similar in symengine
Sumith Kulal
@Sumith1896
Jun 27 2015 15:38
@isuruf Could you have a look at this Sumith1896/csympy@b1c94e6 ?
Sorry about being late on this
Isuru Fernando
@isuruf
Jun 27 2015 15:39
Looks good
Sumith Kulal
@Sumith1896
Jun 27 2015 15:40
Cool
Sumith Kulal
@Sumith1896
Jun 27 2015 15:45
@bluescarni Thanks, I'll give a read. This series will be very interesting.
Sumith Kulal
@Sumith1896
Jun 27 2015 15:59
@shivamvats Anytime now. I am free.
If you cannot now, anytime tomorrow(IST) would also do.
Isuru Fernando
@isuruf
Jun 27 2015 16:32
@Sumith1896 catch program is not in SOURCE_DIR/symengine/catch
Sumith Kulal
@Sumith1896
Jun 27 2015 16:33
Isn't that were libcatch generates
and the .cpp and .hpp present?
Isuru Fernando
@isuruf
Jun 27 2015 16:34
Nope. It's built in the build directory in out of tree builds
Sumith Kulal
@Sumith1896
Jun 27 2015 16:34
Ohh okay
Can you correct me, I am not familiar with this?
Isuru Fernando
@isuruf
Jun 27 2015 16:37
Try it in your computer and you will figure it out. What you've got to do is clean the repo. Change directory into build folder and call cmake with cmake ${SOURCE_DIR}
Sumith Kulal
@Sumith1896
Jun 27 2015 16:46
We are concerned of the catch.cpp.o is it?
Isuru Fernando
@isuruf
Jun 27 2015 16:51
nope libcatch.a
Sumith Kulal
@Sumith1896
Jun 27 2015 16:54
out of source builds give libcatch.a location to here /home/sumith/github/csympy/build/symengine/catch
Isuru Fernando
@isuruf
Jun 27 2015 16:57

Yes, so you'll have to define a variable $BUILD_DIR here, https://github.com/sympy/symengine/blob/master/bin/test_travis.sh#L15
like https://github.com/sympy/symengine/blob/master/bin/install_travis.sh#L12

and then use -L$BUILD_DIR/symengine/catch

Sumith Kulal
@Sumith1896
Jun 27 2015 16:58
Ohh
Isuru Fernando
@isuruf
Jun 27 2015 17:01
Isuru Fernando
@isuruf
Jun 27 2015 17:21
@Sumith1896, you can turn on travis-ci for your fork, so that you can test in your fork first.
Shivam Vats
@shivamvats
Jun 27 2015 17:56
ping @Sumith1896
Sumith Kulal
@Sumith1896
Jun 27 2015 17:57
Give me a minute
Is it possible to set-up travis-ci only for my fork?
Locally doesn't seem to be working
@shivamvats I'd like to push to 484 first, hope not a problem
Isuru Fernando
@isuruf
Jun 27 2015 18:00
You can enable travis-ci for http://github.com/Sumith1896/csympy
Shivam Vats
@shivamvats
Jun 27 2015 18:00
@Sumith1896 Cool. We can meet tomorrow.
Sumith Kulal
@Sumith1896
Jun 27 2015 18:01
@shivamvats If I am done with this quickly, even today is fine.
Shivam Vats
@shivamvats
Jun 27 2015 18:01
Great. Take your time.
Sumith Kulal
@Sumith1896
Jun 27 2015 18:01
I'm just worried, other PRs are waiting because of me
Thanks @isuruf Build created here: https://travis-ci.org/Sumith1896/csympy/builds/68622348
Sumith Kulal
@Sumith1896
Jun 27 2015 18:06
@shivamvats There still?
Sumith Kulal
@Sumith1896
Jun 27 2015 18:55
@isuruf Two clang failures
Sumith Kulal
@Sumith1896
Jun 27 2015 19:02
Isn't the error reported in catch.hpp
How is that possible?
Francesco Biscani
@bluescarni
Jun 27 2015 20:08
@Sumith1896 it would seem like the <type_traits> header is not included
Sumith Kulal
@Sumith1896
Jun 27 2015 20:23
I'll fix that for now
But it is a bug in Catch, you mean?
Francesco Biscani
@bluescarni
Jun 27 2015 20:28
it looks like the compiler cannot find std::underlying_type, which is defined in <type_traits>
so either <type_traits> is not included, which seems strange, or maybe underlying_type is not provided by the C++ standard library
but it would seem a problem in Catch, yes
Sumith Kulal
@Sumith1896
Jun 27 2015 20:31
Let me try adding the header.
Sumith Kulal
@Sumith1896
Jun 27 2015 21:00
Doesn't work.
Isuru Fernando
@isuruf
Jun 27 2015 21:00
I think it's an issue with clang linking against libstdc++
Sumith Kulal
@Sumith1896
Jun 27 2015 21:01
Any quick fix from our side?
I don't want to go to sleep without that PR fixed :smile:
Sumith Kulal
@Sumith1896
Jun 27 2015 21:10
@isuruf Something similar martinmoene/lest#10
Francesco Biscani
@bluescarni
Jun 27 2015 21:21
it does not look like a linking issue, the error happens in compilation phase
Sumith Kulal
@Sumith1896
Jun 27 2015 21:22
This message was deleted
The above issue was resolved by changing travis configs.
Some thing is wrong there perhaps
Francesco Biscani
@bluescarni
Jun 27 2015 21:55
it is some kind of mismatch between clang and libstdc++, it would help if you could reproduce it on your local machine
Sumith Kulal
@Sumith1896
Jun 27 2015 21:55
Let me revert the last commit where I add <type_trait>