Skip to content

1888/public temporary conversion - #2066

Open
MarcelKoch wants to merge 6 commits into
1888/precision-handlingfrom
1888/public-temporary-conversion
Open

1888/public temporary conversion#2066
MarcelKoch wants to merge 6 commits into
1888/precision-handlingfrom
1888/public-temporary-conversion

Conversation

@MarcelKoch

Copy link
Copy Markdown
Member

This PR makes the temporary_conversion class public. It also adds new create functions for:

  • types convertible to the desired type (both raw and unique_ptr)
  • temporary conversion of a derived type
  • temporary conversion of a base type

Later the original temporary_conversion::create<T1, T2, T3, ...>(op) will be removed, since it will not be used anymore.

@MarcelKoch MarcelKoch added this to the Ginkgo 2.0 milestone Aug 17, 2026
@MarcelKoch MarcelKoch self-assigned this Aug 17, 2026
@MarcelKoch MarcelKoch mentioned this pull request Aug 17, 2026
9 tasks
@ginkgo-bot ginkgo-bot added reg:build This is related to the build system. reg:testing This is related to testing. mod:core This is related to the core module. labels Aug 17, 2026
@yhmtsai

yhmtsai commented Aug 18, 2026

Copy link
Copy Markdown
Member

is there any need for making it public?

@MarcelKoch

Copy link
Copy Markdown
Member Author

Yes, it is a return value of a public function in #2067 (as_precision)

@MarcelKoch
MarcelKoch force-pushed the 1888/public-temporary-conversion branch from d5dec4d to a7ccae7 Compare August 18, 2026 14:37
@MarcelKoch
MarcelKoch force-pushed the 1888/precision-handling branch from cb39513 to 8e4015d Compare August 18, 2026 14:37
@MarcelKoch
MarcelKoch force-pushed the 1888/public-temporary-conversion branch from a7ccae7 to 9d3847a Compare August 18, 2026 14:57
@MarcelKoch
MarcelKoch force-pushed the 1888/precision-handling branch from 8e4015d to 55551dd Compare August 18, 2026 14:57
@MarcelKoch
MarcelKoch force-pushed the 1888/public-temporary-conversion branch from 9d3847a to c410e89 Compare August 19, 2026 12:38
@MarcelKoch
MarcelKoch force-pushed the 1888/precision-handling branch from 55551dd to 560109c Compare August 19, 2026 12:38
@MarcelKoch
MarcelKoch force-pushed the 1888/precision-handling branch from 560109c to ebe527a Compare August 19, 2026 12:56
@MarcelKoch
MarcelKoch force-pushed the 1888/public-temporary-conversion branch from c410e89 to 8445ef7 Compare August 19, 2026 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1:ST:ready-for-review This PR is ready for review 1:ST:skip-full-test mod:core This is related to the core module. reg:build This is related to the build system. reg:testing This is related to testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants