Problem with base classes for coordinate-frame classes needing to have `name`
See original GitHub issueA recent code change in #11090 is causing errors in SunPy:
We have a base class for a frame that does not have a name
attribute, so the base.name
access produces an error. It’s easy enough to add a dummy name
attribute to “fix” this on our side, but presumably the Astropy code needs some protection?
Issue Analytics
- State:
- Created 3 years ago
- Comments:6 (6 by maintainers)
Top Results From Across the Web
Using and Designing Coordinate Frames
Customizing Frame Component Names¶. First, as mentioned in the point (1) above, some frame classes have special names for their components. For example,...
Read more >Astropy: Inititializing a user-defined Coordinate Frame
I calculate the transformatin matrix and give it as an argument for the new frame. But I get an error. Here is the...
Read more >Source code for sunpy.coordinates.metaframes
So each type of frame needs its own distinct rotated-Sun frame class. ... A frame that applies solar rotation to a base coordinate...
Read more >Chapter 3. Coordinates and Transformations
Assuming a coordinate frame is represented with respect to an incorrect base frame. Applying a coordinate transform to a quantity not represented with...
Read more >Chapter 3 – The Classical Orbital Elements (COEs)
So far, we have learned to describe where satellites are in orbit by it ... We will first introduce the coordinate frame used...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Ah, yes, an oversight on my part - oops! I’ll submit a follow-up PR now, and add a regression test.
fixed by #11101