Skip to content

ParticleContainer: Split #108

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.

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

Merged
merged 3 commits into from
Feb 13, 2023
Merged

ParticleContainer: Split #108

merged 3 commits into from
Feb 13, 2023

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Feb 10, 2023

Split specializations for faster, more resource efficient compiles that can be parallelized.

@ax3l ax3l added the enhancement New feature or request label Feb 10, 2023
Comment on lines +327 to +338
// ParticleTileType& DefineAndReturnParticleTile (int lev, int grid, int tile)
// {
// m_particles[lev][std::make_pair(grid, tile)].define(NumRuntimeRealComps(), NumRuntimeIntComps());
// return ParticlesAt(lev, grid, tile);
// }
// template <class Iterator>
// ParticleTileType& DefineAndReturnParticleTile (int lev, const Iterator& iter)
// {
// auto index = std::make_pair(iter.index(), iter.LocalTileIndex());
// m_particles[lev][index].define(NumRuntimeRealComps(), NumRuntimeIntComps());
// return ParticlesAt(lev, iter);
// }

Check notice

Code scanning / CodeQL

Commented-out code

This comment appears to contain commented-out code.
// const Vector<std::string>& int_comp_names) const;
// void WritePlotFilePre ();

// void WritePlotFilePost ();

Check notice

Code scanning / CodeQL

Commented-out code

This comment appears to contain commented-out code.
// Long NumberOfParticlesAtLevel (int level, bool only_valid = true, bool only_local = false) const;
.def("NumberOfParticlesAtLevel", &ParticleContainerType::NumberOfParticlesAtLevel,
py::arg("level"), py::arg("only_valid")=true, py::arg("only_local")=false)
// Vector<Long> NumberOfParticlesInGrid (int level, bool only_valid = true, bool only_local = false) const;

Check notice

Code scanning / CodeQL

Commented-out code

This comment appears to contain commented-out code.
.def("ByteSpread",&ParticleContainerType::ByteSpread)
.def("PrintCapacity", &ParticleContainerType::PrintCapacity)
.def("ShrinkToFit", &ParticleContainerType::ShrinkToFit)
// Long NumberOfParticlesAtLevel (int level, bool only_valid = true, bool only_local = false) const;

Check notice

Code scanning / CodeQL

Commented-out code

This comment appears to contain commented-out code.
// void InitFromAsciiFile (const std::string& file, int extradata,
// const IntVect* Nrep = nullptr);

// void InitFromBinaryFile (const std::string& file, int extradata);

Check notice

Code scanning / CodeQL

Commented-out code

This comment appears to contain commented-out code.
Comment on lines +164 to +165
// ParticleContainer ( ParticleContainer && ) = default;
// ParticleContainer& operator= ( ParticleContainer && ) = default;

Check notice

Code scanning / CodeQL

Commented-out code

This comment appears to contain commented-out code.
Comment on lines +161 to +162
// ParticleContainer ( const ParticleContainer &) = delete;
// ParticleContainer& operator= ( const ParticleContainer & ) = delete;

Check notice

Code scanning / CodeQL

Commented-out code

This comment appears to contain commented-out code.
>;
using ParticleInitData = typename ParticleContainerType::ParticleInitData;
using ParticleTileType = typename ParticleContainerType::ParticleTileType;
//using AoS = typename ParticleContainerType::AoS;

Check notice

Code scanning / CodeQL

Commented-out code

This comment appears to contain commented-out code.
Split specializations for faster, more resource efficient compiles that
can be parallelized.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants