Skip to content
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

provide method to access parent streams #2

Closed

Conversation

mantepse
Copy link

I trust that only the last element of the cache may contain the variable. I couldn't find a counterexample. It would be great if you could try to construct one.

I do not understand substitution in polynomial rings. In particular, I added a currently failing test.

The tests in lazy_series_ring.py now take about 30s on my laptop, I think that's too much.

As I mentioned elsewhere, the signature of integral should probably follow multi_power_series_ring_element.MPowerSeries.integral.

@mantepse
Copy link
Author

Given sagemath#36575, I guess it makes sense to alert the user that define_implicitly only works if the coefficients of eqn are rational functions of the coefficients of self. Or can we do better?

@mantepse
Copy link
Author

I'm afraid I need help with the failing test:

            sage: L.<z> = LazyPowerSeriesRing(QQ)
            sage: f = L.undefined(1)
            sage: f.define_implicity(log(1+f) - ~(1 + f) + 1, [])
            sage: f[1]
            0

This fails (ValueError: generator already executing) when trying to get the first coefficient of log(1 + f) with f "undetermined". I don't exactly see why. If I do it "by hand", I have

sage: I.<a> = InfinitePolynomialRing(QQ)
sage: L.<z> = LazyPowerSeriesRing(I)
sage: f = L(lambda n: a[n], valuation=1)
sage: F = log(1+f) - ~(1 + f) + 1
sage: F[0]
0
sage: F[1]
2*a_1

so a_1 must be zero.

I don't see what's going wrong.

@mantepse
Copy link
Author

I think I understand now: replace does -- obviously -- not work with Stream_function.

Unfortunately, I don't see how it could.

@github-actions
Copy link

Documentation preview for this PR (built with commit 05376a6; changes) is ready! 🎉

@mantepse mantepse deleted the branch tscrim:lazy_series/integration March 1, 2024 19:05
@mantepse mantepse closed this Mar 1, 2024
@mantepse mantepse deleted the lazy_series/integration branch March 1, 2024 19:05
tscrim pushed a commit that referenced this pull request Aug 8, 2024
Remove excessive blank lines around INPUT, OUTPUT, EXAMPLES, and before
docstring ending """. (#2)
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.

2 participants