Skip to content

Re-implement field write api from #506, but template view directly (no auto in function parameter)#523

Open
aliemen wants to merge 1 commit into
masterfrom
506-V2-test-print-func-without-auto
Open

Re-implement field write api from #506, but template view directly (no auto in function parameter)#523
aliemen wants to merge 1 commit into
masterfrom
506-V2-test-print-func-without-auto

Conversation

@aliemen
Copy link
Copy Markdown
Collaborator

@aliemen aliemen commented May 24, 2026

See description of #506.

I want to re-run the jülrich ci and see if this was actually the problem. Here is what I changed. From this:

template <unsigned Dim>
void write_as_list_impl(const auto view, std::ostream& out = std::cout) {
    auto N = view.extent(0);
    ...

to this:

template <unsigned Dim, typename View>
void write_as_list_impl(const View& view, std::ostream& out = std::cout) {
    auto N = view.extent(0);
    ...

I think that the Jülich cuda/12.6 + gcc/13 combination didn't like this one. Newer cuda versions don't seem to have a problem with that (that's why the cscs ci passed).

@aliemen aliemen self-assigned this May 24, 2026
@aliemen
Copy link
Copy Markdown
Collaborator Author

aliemen commented May 24, 2026

Looks like Jülich CI passes now

@aliemen aliemen added enhancement New feature or request feature request Used to label a feature request in the issue tracker. labels May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request feature request Used to label a feature request in the issue tracker. gitlab-mirror

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants