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

More cleanups to Lt #460

Merged
merged 4 commits into from
Sep 24, 2020
Merged

More cleanups to Lt #460

merged 4 commits into from
Sep 24, 2020

Conversation

eric-wieser
Copy link
Member

@eric-wieser eric-wieser commented Sep 23, 2020

This includes a deprecation of two properties that were probably intended to be private anyway,

Docs are at https://galgebra--460.org.readthedocs.build/en/460/generated/galgebra.lt.html#galgebra.lt.Lt

This is a lot easier to follow if we just handle `xreplace` manually.
Note that as a consequence, `mv_dict` now contains the scalar.
This doesn't affect behavior, and it's not clear that `mv_dict` is intended to be public anyway.
@eric-wieser eric-wieser added component: docs Documentation component: transforms Lt and Mlt Deprecation Precursor to removal of features labels Sep 23, 2020
@eric-wieser
Copy link
Member Author

Failure is #441

These attributes did not contain correct values if the non-string constructor was used.
This corrects the implementation, but also deprecates it because its not clear that it's useful anyway.
@utensil utensil merged commit 364b4e8 into pygae:master Sep 24, 2020
@utensil utensil added the state: needs changelog Needs a changelog entry before the next release. Remove this label when the changelog is done. label Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: docs Documentation component: transforms Lt and Mlt Deprecation Precursor to removal of features state: needs changelog Needs a changelog entry before the next release. Remove this label when the changelog is done.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants