Sbromberger/fix templates #263
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
LLNL#201) * Updating Github Actions to run on pull requests against our new dev branch names * Adding quotes around dev branch name
* Updating versions of GCC and Ubuntu in Github Actions * Updating versions of checkout and cache actions
Co-authored-by: Keita Iwabuchi <[email protected]>
…splitting and local caches on each process (LLNL#205) * Rewrites ygm::container::disjoint_set to use union by rank with path-splitting and local caches on each process * Removing unused code * Removing unused code * Clears static structures used in disjoint_set::all_compress() * Updating disjoint_set cache size * Updating disjoint_set cache size
…le_next_receive to not require an MPI_Status object (LLNL#206)
Cereal Boost vector Co-authored-by: Keita Iwabuchi <[email protected]>
* Create cereal_boost_container.hpp Cereal Boost vector * Add test for cerealizing boost vector * Add test_cereal_boost_container in cmake --------- Co-authored-by: Keita Iwabuchi <[email protected]>
* Add parquet-tools * Add dump command in parquet-tools * Enable cmake to find Arrow and Parquet installed by pip - Add a cmake option `PIP_PYARROW_ROOT` to use Arrow and Parquet installed along with pyarrow by pip. - Use pip to install Arrow and Parquet in the CI Test. * Skip testing parquet on ubuntu-20.04. * User shorter names --------- Co-authored-by: Keita Iwabuchi <[email protected]>
* Add parquet-tools --------- Co-authored-by: Keita Iwabuchi <[email protected]>
* Enable Doxygen & RTD --------- Co-authored-by: Keita Iwabuchi <[email protected]>
Overhaul and standardization of containers using CRTP base classes. --------- Co-authored-by: Lance Fletcher <[email protected]> Co-authored-by: Trevor Steil <[email protected]>
* Enable Doxygen & RTD * Update Doxygen & RTD (merge docs branch) * Update Doxygen & RTD * (RTD) Set OS and Python versions --------- Co-authored-by: Keita Iwabuchi <[email protected]>
* Updated bag, first pass. * Renamed container traits. Renamed base_async_insert. * Added move constructors. * Renamed old partitioner. * Wrote the initial boiler plate for set * Got async_insert for ygm::container::set working. * Working on map. * Added move constructor to set class * Got async_exe_if_contains working * Changed async_contains interface slightly * Added base_count class, and added bag::count functionality * Added base_async_insert_contains class and integrated it into ygm::container::set * Using tuple concepts. * Added a few tests to bag * Split base classes. * First attempt * Updating local repo with Roger's changes * Temporarily commenting out broken tests * Got multiset converted to CRTP, also added some test cases * filter, map, flatten * Updated test_set.cpp * map, filter, flatten * base_iteration * Updates array to use CRTP base classes and adds more tests * Fixes bug in copy constructor concerning order of initialization of member variables * Adds ygm::container::array constructors to construct from YGM and STL containers * adds tests for concepts * Fixes partitioner initialization in ygm::container::array --------- Co-authored-by: Roger Pearce <[email protected]> Co-authored-by: Lance Fletcher <[email protected]>
… a rank that has completed a barrier() and processed on a different rank that has not completed the same barrier() (LLNL#224)
* Adds ability to handle headers in CSV files. Requires allowing ygm::io::line_parser to read the first line of the first file independently and skip over the first line of files in for_all operations. * Makes ygm::io::detail::csv_line::size() const * Adds ability to check if ygm::io::csv_parser headers contain a particular label * Fixes use of ygm::comm::mpi_bcast() in ygm::io::line_parser::read_first() * Makes skipping first line in line parser controlled by a member variable
* Updates container documentation * Fixes links in rtd/getting_started page
* Adds batch erase functionality for keys from an STL container * Adds batch erase keys to map and multimap * Adds base_batch_erase_key_value * Adds more batch erase functionality
Added missing cache initialization.
* Fixed reduce() and added test.
* Removes old files. * renamed transform & added test.
…can create docs (LLNL#250) * Removing all clever tricks and introducing duplicate code so doxygen can create docs * Fixing ndjson_parser inheritance from base_iteration_value * Adds back in missing comm() functions to ygm::io classes * ygm::container::detail::base_batch_erase_key_value no longer inherits from ygm::container::detail::base_batch_erase_key in order to make inherited member functions to show up in docs properly
…wer version of Doxygen (LLNL#252)
* Adds concepts for async_reduce operations * Fixes reducing_adapter's use of underlying container's partitioner
* Makes lambdas that take user-provided lambdas/functors mutable so captured objects can call non-const functions * Fixes reduce_by_key lambda
…er() functions to properly use overloaded for_all() operations, such as when calling gather on the output of a filter(). (LLNL#257)
* Adds async_reduce functionality to ygm::container::array * Removes extra newline
* Makes locations in YGM_CHECK_ASYNC_LAMBDA_COMPLIANCE macro appear as string literals and adds missing check for std::is_standard_layout * Update base_async_visit.hpp * Update base_async_visit.hpp * Update comm.ipp
Co-authored-by: Keita Iwabuchi <[email protected]>
* Find pyarrow module automatically for Arrow and Parquet - Find pyarrow module executing python script if A) PIP_PYARROW_ROOT is not set and B) find_package can't find Parquet or Arrow * Install pyarrow if not found * Install pyarrow if not found * Install pyarrow v16.1.* in venv * Upgrade pip in venv before install pyarrow * Update FindArrowParquet.cmake Small typo --------- Co-authored-by: Keita Iwabuchi <[email protected]> Co-authored-by: Trevor Steil <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This minor change fixes the build errors I reported this morning. Built with gcc 13.2.0 and clang++ 17.0.6.
Test Results