Skip to content

Question: Why is executeCoupling() commented out in olbMixing.hh? #72

@SiyonaAgarwa

Description

@SiyonaAgarwa

Hi! I was going through olbMixing.hh in the CFD handlers and noticed that this line is commented out:

// this->getLattice().executeCoupling();

Here’s the surrounding loop for context:

for (int iT = 0; iT < int(maxIter); ++iT) {
    writeVTK(this->getStep());
    for (auto& [speciesId, adLattice] : adLattices) {
        // this->getLattice().executeCoupling();
        adLattice->collideAndStream();
        adConverges.at(speciesId)->takeValue(
            adLattice->getStatistics().getAverageRho(), true
        );
    }
    this->getStep() += 1;
}

When I tried running the simulation, I needed to uncomment executeCoupling() for it to work correctly on my end.

Also, I wanted to ask about the concentration-related CFD examples. They are not working for me out of the box. When I tried running them, I encountered the following error:

terminate called after throwing an instance of 'std::invalid_argument'
what(): Continuous simulations are currently not supported for this platform.

I had to modify/write additional code to get them running properly.

Could you please clarify:

  • Is executeCoupling() intentionally commented out (e.g., for specific workflows or solver configurations)?
  • Why are the concentration examples not working as expected?
  • Is the above error related to platform limitations or missing configuration?

Just want to make sure I’m not missing something before proceeding further. Thanks!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions