New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

implemented eigen_values function #1319

Merged
merged 2 commits into from Sep 2, 2017

Conversation

Projects
None yet
4 participants
@ranjithkumar007
Contributor

ranjithkumar007 commented Aug 5, 2017

No description provided.

@ranjithkumar007

This comment has been minimized.

Show comment
Hide comment
@ranjithkumar007
Contributor

ranjithkumar007 commented Aug 12, 2017

@ranjithkumar007

This comment has been minimized.

Show comment
Hide comment
@ranjithkumar007

ranjithkumar007 Aug 12, 2017

Contributor

Error message - cat: write error: No space left on device
Should we redirect the output of make -j8 install of FLINT to /dev/null?

Contributor

ranjithkumar007 commented Aug 12, 2017

Error message - cat: write error: No space left on device
Should we redirect the output of make -j8 install of FLINT to /dev/null?

@srajangarg

This comment has been minimized.

Show comment
Hide comment
@srajangarg

srajangarg Aug 14, 2017

Contributor

yes, we can try that. push a commit here to see if it works.

Contributor

srajangarg commented Aug 14, 2017

yes, we can try that. push a commit here to see if it works.

@srajangarg

This comment has been minimized.

Show comment
Hide comment
@srajangarg

srajangarg Aug 15, 2017

Contributor

@isuruf what's eating up all the disk space? Afaik, travis gives us 20GB per job.

Contributor

srajangarg commented Aug 15, 2017

@isuruf what's eating up all the disk space? Afaik, travis gives us 20GB per job.

@isuruf

This comment has been minimized.

Show comment
Hide comment
@isuruf

isuruf Aug 17, 2017

Member

Coverage info

Member

isuruf commented Aug 17, 2017

Coverage info

@certik

certik approved these changes Aug 31, 2017

This looks good to me. I have restarted the failing build.

Show outdated Hide outdated bin/install_travis.sh
@certik

This comment has been minimized.

Show comment
Hide comment
@certik

certik Aug 31, 2017

Contributor

Now the test fails with:

+++conda config --set always_yes yes --set changeps1 no

+++'[' -x /usr/lib/command-not-found ']'

+++/usr/lib/command-not-found -- conda

conda: command not found

So something broke on Travis.

Contributor

certik commented Aug 31, 2017

Now the test fails with:

+++conda config --set always_yes yes --set changeps1 no

+++'[' -x /usr/lib/command-not-found ']'

+++/usr/lib/command-not-found -- conda

conda: command not found

So something broke on Travis.

@ranjithkumar007

This comment has been minimized.

Show comment
Hide comment
@ranjithkumar007

ranjithkumar007 Aug 31, 2017

Contributor

Upon restart, the error message is :

 ./#home#travis#build#symengine#symengine#symengine#polys#upolybase.h.gcov bytes=857652
 ./#usr#include#c++#4.8#bits#stl_map.h.gcov bytes=487270
 ./#usr#include#c++#4.8#ext#new_allocator.h.gcov bytes=10103812
 ./#usr#include#c++#4.8#future.gcov bytes=7946924
cat: write error: No space left on device

@certik , this PR has the same problem that #1305 had. I got it fixed by moving piranha to a different build. The problem was with coverage info as pointed out by @isuruf. Should I wait for #1305 to get merged in and then update this PR or add the commits that fixed the error here?

Contributor

ranjithkumar007 commented Aug 31, 2017

Upon restart, the error message is :

 ./#home#travis#build#symengine#symengine#symengine#polys#upolybase.h.gcov bytes=857652
 ./#usr#include#c++#4.8#bits#stl_map.h.gcov bytes=487270
 ./#usr#include#c++#4.8#ext#new_allocator.h.gcov bytes=10103812
 ./#usr#include#c++#4.8#future.gcov bytes=7946924
cat: write error: No space left on device

@certik , this PR has the same problem that #1305 had. I got it fixed by moving piranha to a different build. The problem was with coverage info as pointed out by @isuruf. Should I wait for #1305 to get merged in and then update this PR or add the commits that fixed the error here?

@certik

This comment has been minimized.

Show comment
Hide comment
@certik

certik Aug 31, 2017

Contributor

@ranjithkumar007 does not matter to me, but #1305 can't be merged without more work it seems. So just concentrate on one PR, and let's get it merged, then the other one.

Contributor

certik commented Aug 31, 2017

@ranjithkumar007 does not matter to me, but #1305 can't be merged without more work it seems. So just concentrate on one PR, and let's get it merged, then the other one.

@ranjithkumar007

This comment has been minimized.

Show comment
Hide comment
@ranjithkumar007

ranjithkumar007 Sep 1, 2017

Contributor

Thanks @certik , I updated the PR.
Travis should pass now.

Contributor

ranjithkumar007 commented Sep 1, 2017

Thanks @certik , I updated the PR.
Travis should pass now.

@ranjithkumar007

This comment has been minimized.

Show comment
Hide comment
@ranjithkumar007

ranjithkumar007 Sep 1, 2017

Contributor

@certik , Travis Passed 😃

Contributor

ranjithkumar007 commented Sep 1, 2017

@certik , Travis Passed 😃

@certik certik merged commit f6549e9 into symengine:master Sep 2, 2017

3 checks passed

codecov/project 83.27% (+0.59%) compared to dd5f26e
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@certik

This comment has been minimized.

Show comment
Hide comment
@certik

certik Sep 2, 2017

Contributor
Contributor

certik commented Sep 2, 2017

@ranjithkumar007 ranjithkumar007 deleted the ranjithkumar007:eigen branch Sep 2, 2017

@ranjithkumar007

This comment has been minimized.

Show comment
Hide comment
@ranjithkumar007

ranjithkumar007 Sep 2, 2017

Contributor

Thanks a lot @certik. 😃

Contributor

ranjithkumar007 commented Sep 2, 2017

Thanks a lot @certik. 😃

isuruf pushed a commit to isuruf/symengine that referenced this pull request Aug 4, 2018

Merge pull request #1319 from ranjithkumar007/eigen
implemented eigen_values function
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment