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

Immutability of Scalar Fields II #30311

Closed
mjungmath opened this issue Aug 7, 2020 · 17 comments
Closed

Immutability of Scalar Fields II #30311

mjungmath opened this issue Aug 7, 2020 · 17 comments

Comments

@mjungmath
Copy link

Immutability of scalar fields, see #30261.

Pass immutability to chart functions bound to scalar fields.

Depends on #30266
Depends on #30310
Depends on #30181

CC: @egourgoulhon @tscrim @mkoeppe

Component: manifolds

Keywords: immutability

Author: Michael Jung

Branch/Commit: 6f39d6e

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/30311

@mjungmath mjungmath added this to the sage-9.2 milestone Aug 7, 2020
@mjungmath
Copy link
Author

@mjungmath
Copy link
Author

Commit: 958ad16

@mjungmath
Copy link
Author

Author: Michael Jung

@mjungmath
Copy link
Author

Changed keywords from none to immutability

@mjungmath
Copy link
Author

Last 10 new commits:

055d226Trac #30266: Merge branch 't/30116/test_comparison_tensorfield' into t/30266/immutability_for_scalar_fields
8ca8b75Trac #30266: hash compatibility check wrt #30116
0bf9f36Trac #30266: Merge branch 'develop' into t/30266/immutability_for_scalar_fields
a5f6a19Trac 30266: hash function slightly improved
37b0042Trac #30266: remove dependency of #30116
2f72e19Trac #30266: if statement missing
8600c2dMerge branch 't/30181/immutable_elements_of_freemoduletensor' into t/30310/immutability_of_chart_functions
76229bdTrac #30310: immutability of chart functions
48a35b1Merge branch 't/30310/immutability_of_chart_functions' into t/30311/immutability_of_scalar_fields_ii
958ad16Trac 30311: apply set_immutable to chart functions

@mjungmath

This comment has been minimized.

@mjungmath
Copy link
Author

Dependencies: #30266, #30310

@mjungmath
Copy link
Author

Changed dependencies from #30266, #30310 to #30266, #30310, #30181

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 21, 2021

Changed commit from 958ad16 to 6f39d6e

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 21, 2021

Branch pushed to git repo; I updated commit sha1. This was a forced push. Last 10 new commits:

7247620Merge branch 'develop' into t/30310/immutability_of_chart_functions
5ecbf3aTrac #31181: return added
8b20fc4Merge branch 't/31181/mutability_class_does_not_return_is_immutable' into t/30310/immutability_of_chart_functions
4c33935Trac #31196: minor code improvements, py3 compatibility, documentation improved
e5228d3Trac #31196: cpdef require methods + example added
d957f73Trac #31196: unnecessary line in docstring removed
d6d6ba4Trac #31182: `__getstate__` and __setstate__
6cbd1fdTrac #31182: doctests added for `__setstate__` and __getstate__
848e96bMerge branch 't/31182/mutability_class_and_pickling' into t/30310/immutability_of_chart_functions
6f39d6eTrac #30311: apply set_immutable to all chart functions

@mjungmath
Copy link
Author

comment:11

Should be a green patchbot (startup time fails).

By the way: Why does the startup time fail so often? What's going on there?

@tscrim
Copy link
Collaborator

tscrim commented Jan 21, 2021

comment:12

That is a bit fickle of a test. I wouldn't pay too much attention to it, but it is good to check it quickly to see if there is something causing a major slowdown on startup.

@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 15, 2021

comment:13

Setting new milestone based on a cursory review of ticket status, priority, and last modification date.

@mkoeppe mkoeppe modified the milestones: sage-9.3, sage-9.4 Mar 15, 2021
@mjungmath
Copy link
Author

comment:14

Can we consider this ticket as done? Travis, Eric?

@tscrim
Copy link
Collaborator

tscrim commented Apr 10, 2021

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Apr 10, 2021

comment:15

Yep, LGTM.

@vbraun
Copy link
Member

vbraun commented May 27, 2021

Changed branch from u/gh-mjungmath/immutability_of_scalar_fields_ii to 6f39d6e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants