sunpy icon indicating copy to clipboard operation
sunpy copied to clipboard

[map branch] Reproject tidy

Open CyclingNinja opened this issue 8 months ago • 8 comments

Change reproject_to to use the reproject functionality in NDCube

Remove old reproject import

Remove footprint addition and rely on metadata for information

CyclingNinja avatar May 08 '25 15:05 CyclingNinja

@ayshih do you know of a more efficient way to compare rsun values than creating this outmap and comparing it to the one on the object?

CyclingNinja avatar May 08 '25 16:05 CyclingNinja

Contrary to what I wrote here, you'd be fine replacing outmap.rsun_meters with target_wcs.wcs.aux.rsun_ref. That said, it's negligibly more efficient since the reprojected map is made anyway.

ayshih avatar May 08 '25 16:05 ayshih

thanks @ayshih thanks for clearing up. initially I was looking for a solution with what you'd said previously in mind. I've spoken to Tom and Danny and I think I'm now at the best solution, let me know if you have any further thoughts

CyclingNinja avatar May 09 '25 13:05 CyclingNinja

Looks like we have a few places now where the warning is raised.

Is the CI passing on the main branch?

nabobalis avatar May 09 '25 15:05 nabobalis

@nabobalis true, by the main branch I assume you mean map_ndcube_migration?

CyclingNinja avatar May 09 '25 15:05 CyclingNinja

Yeah that branch

nabobalis avatar May 09 '25 15:05 nabobalis

okayy, i've hit a slight block currently the new implementation of reproject is giving nans (perhaps resultant coord is off limb?) when pixel_to_world is used on the result of a reproject_to call, @ayshih and @Cadair do you have any ideas on this one?

CyclingNinja avatar Jun 12 '25 16:06 CyclingNinja

The CI is now passing on the map_ndcube_migration branch, so we need to fix this.

Cadair avatar Jun 20 '25 10:06 Cadair