Skip to content

[WIP] Implementing Univariate Series - #810

Closed
char-chen wants to merge 128 commits into
symengine:masterfrom
char-chen:univariateseries
Closed

char-chen wants to merge 128 commits into
symengine:masterfrom
char-chen:univariateseries

Conversation

@char-chen

Copy link
Copy Markdown
Contributor

Started making changes with series-generic. Implemented skeleton methods/functions.

Charles Chen and others added 19 commits January 26, 2016 22:32
…t/symengine into series-generic-changes

UnivariateSeries now inherits from SeriesBase
… series-generic-changes

Wrong commit please ignore
@isuruf

isuruf commented Feb 14, 2016

Copy link
Copy Markdown
Member

Let me know if you need any help here or in gitter

@irislq

irislq commented Feb 14, 2016

Copy link
Copy Markdown
Contributor

Right, for the convert functions what would be the outputs? Right now they return either SymEngine::Integer or SymEngine::Rational but there are no operator= that can perform the operation in series_visitor.h

@isuruf

isuruf commented Feb 14, 2016

Copy link
Copy Markdown
Member

Output should be of Coefficient type, in your case int.

Btw, you can have the Coefficient type as Expression instead of int.

Comment thread symengine/expand.cpp

void bvisit(const Pow &self) {
RCP<const Basic> _base = expand(self.get_base());
if(is_a<const UnivariatePolynomial>(*self.get_base())) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you take this functionality out of this function and call the function?

@irislq

irislq commented Mar 8, 2016

Copy link
Copy Markdown
Contributor

Anyone know where this "std::bad_alloc" error is coming from? This is when I run test_series_generic.
error.txt

…ric, commented out map_uint_mpz univariate_series func
@irislq irislq mentioned this pull request Mar 11, 2016
@isuruf

isuruf commented Mar 13, 2016

Copy link
Copy Markdown
Member

Now that #857 is merged, can you make a new pull request with the changes to UnivariateSeries

@irislq

irislq commented Mar 13, 2016

Copy link
Copy Markdown
Contributor

Any particular reason? We already have this PR.

@isuruf

isuruf commented Mar 13, 2016

Copy link
Copy Markdown
Member

This PR has too many commits that are now irrelevant.

@char-chen

Copy link
Copy Markdown
Contributor Author

Edit: ok, will set up new one.

@char-chen char-chen closed this Mar 13, 2016
@char-chen
char-chen deleted the univariateseries branch March 13, 2016 06:14
char-chen pushed a commit to char-chen/symengine that referenced this pull request Mar 13, 2016
@char-chen char-chen mentioned this pull request Mar 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants