Skip to content

Feature: IVEA projection#3

Draft
felixpalmer wants to merge 1 commit into
felix/polyhedral-iseafrom
felix/polyhedral-ivea
Draft

Feature: IVEA projection#3
felixpalmer wants to merge 1 commit into
felix/polyhedral-iseafrom
felix/polyhedral-ivea

Conversation

@felixpalmer

Copy link
Copy Markdown
Owner

Overview

This PR is stacked on top of the root PR: OSGeo#4758 - please see there before reviewing

Projecting with DSEA onto a icosahedral net yields a better result (less cusps) that using ISEA. This approach has been referred to as IVEA in some places, for example https://dggal.org/

The purpose of this PR is to show how an alias can be added for an existing projection to make it easier for end users - some of whom may find it confusing that a dodecahedral projection can be unfolded onto an icosahedral net.

Change list

  • Register new projection
  • Include in test render scripts

Example renders IVEA

ivea_net isea_net

Note cusps compared to IVEA

@felixpalmer

Copy link
Copy Markdown
Owner Author

Leaving open for reference, but will require reworking to be compatible with new approach that uses conway meta to cut up the polyhedron and a definition for the parents net


polyhedral::load_triangles(Q, decakis_dodecahedron::SPH_TRI, nets::dsea::icosahedron::FACE_TRI);
polyhedral::set_orient_from_angles(Q, ISEA_STD_LAT_DEG, ISEA_STD_LON_DEG, IVEA_AZ_DEG);

@jerstlouis jerstlouis Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@felixpalmer Curious about this (IVEA_AZ_DEG) here...

Is that the azimuth from DSEA's perspective?

If so, that might be problematic if trying to override +proj=ivea with +azi from ISEA/IVEA's perspective?

Is that positioning the vertices of the icosahedron at ISEA_STD_LAT_DEG, ISEA_STD_LON_DEG, or the vertices of the dodecahedron?

Also wondering if you updated this PR already or it still needs refactoring (it shows no conflict)?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No this PR is out of date, no point keeping it as the other branch has diverged too much

@felixpalmer felixpalmer marked this pull request as draft June 29, 2026 21:55
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