Skip to content

Commit

Permalink
Merge pull request #158 from eseiler/misc/timer
Browse files Browse the repository at this point in the history
[MISC] Split timer into two classes
  • Loading branch information
smehringer authored Nov 7, 2023
2 parents b6d288e + 7846d5e commit 737a231
Show file tree
Hide file tree
Showing 10 changed files with 351 additions and 118 deletions.
8 changes: 4 additions & 4 deletions include/hibf/build/build_data.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,10 @@ struct build_data
std::vector<double> fpr_correction{};

// Timers do not copy the stored duration upon copy construction/assignment
mutable timer<concurrent::yes> index_allocation_timer{};
mutable timer<concurrent::yes> user_bin_io_timer{};
mutable timer<concurrent::yes> merge_kmers_timer{};
mutable timer<concurrent::yes> fill_ibf_timer{};
mutable concurrent_timer index_allocation_timer{};
mutable concurrent_timer user_bin_io_timer{};
mutable concurrent_timer merge_kmers_timer{};
mutable concurrent_timer fill_ibf_timer{};

size_t request_ibf_idx()
{
Expand Down
2 changes: 1 addition & 1 deletion include/hibf/build/insert_into_ibf.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ void insert_into_ibf(robin_hood::unordered_flat_set<uint64_t> const & kmers,
size_t const number_of_bins,
size_t const bin_index,
seqan::hibf::interleaved_bloom_filter & ibf,
timer<concurrent::yes> & fill_ibf_timer);
concurrent_timer & fill_ibf_timer);

//!\overload
void insert_into_ibf(build_data const & data,
Expand Down
4 changes: 2 additions & 2 deletions include/hibf/build/update_parent_kmers.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@ namespace seqan::hibf::build
*/
inline void update_parent_kmers(robin_hood::unordered_flat_set<uint64_t> & parent_kmers,
robin_hood::unordered_flat_set<uint64_t> const & kmers,
timer<concurrent::yes> & merge_kmers_timer)
concurrent_timer & merge_kmers_timer)
{
timer<concurrent::no> local_merge_kmers_timer{};
serial_timer local_merge_kmers_timer{};
local_merge_kmers_timer.start();
parent_kmers.insert(kmers.begin(), kmers.end());
local_merge_kmers_timer.stop();
Expand Down
8 changes: 4 additions & 4 deletions include/hibf/hierarchical_interleaved_bloom_filter.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -249,10 +249,10 @@ class hierarchical_interleaved_bloom_filter
* \brief Only contains values after the HIBF has been constructed.
* \{
*/
timer<concurrent::yes> index_allocation_timer{};
timer<concurrent::yes> user_bin_io_timer{};
timer<concurrent::yes> merge_kmers_timer{};
timer<concurrent::yes> fill_ibf_timer{};
concurrent_timer index_allocation_timer{};
concurrent_timer user_bin_io_timer{};
concurrent_timer merge_kmers_timer{};
concurrent_timer fill_ibf_timer{};
//!\}
//!\endcond
};
Expand Down
Loading

1 comment on commit 737a231

@vercel
Copy link

@vercel vercel bot commented on 737a231 Nov 7, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Successfully deployed to the following URLs:

hibf – ./

hibf.vercel.app
hibf-seqan.vercel.app
hibf-git-main-seqan.vercel.app

Please sign in to comment.