search

evalf / nutils

The nutils project
http://www.nutils.org/
MIT License
88 stars 48 forks source link

fix Multipatch.refined #794

Closed joostvanzwieten closed 1 year ago

joostvanzwieten commented 1 year ago

This PR fixes two issues with Multipatch.refined:

  1. In commit 7e29a2838 the apply_annotations decorators were removed from the topology module, including Multipatch.__init__, which wants a tuple of patches, but Multipatch.refined, which passed a generator of patches, was not changed accordingly.

  2. In commit 64d5f6f86 the apply_annotations decorators were removed from the transformseq module, including PlainTransforms.__init__, which wants a tuple of canonical transform chains, but Multipatch.interfaces, which passed a tuple of uncanonical transform chains when refined, was not changed accordingly.

In addition, this PR adds a test for refined multipatch topologies.

joostvanzwieten commented 1 year ago

Rebased. No changes otherwise.

joostvanzwieten commented 1 year ago

Apologies for causing both issues!

The reviewer didn't catch this either...