Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

move out iqmc sweep counter #117

Merged
merged 2 commits into from
Sep 5, 2023
Merged

move out iqmc sweep counter #117

merged 2 commits into from
Sep 5, 2023

Conversation

ilhamv
Copy link
Collaborator

@ilhamv ilhamv commented Sep 5, 2023

No description provided.

@ilhamv ilhamv requested a review from spasmann September 5, 2023 21:55
@spasmann
Copy link
Collaborator

spasmann commented Sep 5, 2023 •
edited
Loading

This works for source iteration, but we also need to add mcdc["technique"]["iqmc_sweep_counter"] += 1 to the linear operator kernel functions. So that's AxV(), RHS(), HxV(), FxV(), and Preconditioner()

Not sure how to make the changes on the PR myself.

@ilhamv
Copy link
Collaborator Author

ilhamv commented Sep 5, 2023

Ah I didn't know about that. I added the counter into the functions.

@ilhamv ilhamv merged commit 7b04daf into CEMeNT-PSAAP:main Sep 5, 2023
2 checks passed
clemekay pushed a commit to clemekay/MCDC that referenced this pull request Nov 2, 2023
@ilhamv
Merge pull request CEMeNT-PSAAP#117 from ilhamv/main
5151f48 
move out iqmc sweep counter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spasmann spasmann spasmann approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ilhamv @spasmann