cugraph icon indicating copy to clipboard operation
cugraph copied to clipboard

Split edge_partition_src_dst_property.cuh to .hpp and .cuh files.

Open seunghwak opened this issue 3 years ago • 2 comments

Partially addresses #2479 Code clean-up before adding edge_property_t and update_edge_property (to support edge property in addition to edge weights).

  • [x] Split edge_partition_src_dst_property.cuh to edge_src_dst_property.hpp (can be complied with g++) and edge_partition_endpoint_device_view.cuh (requires nvcc)
  • [x] Rename edge_partition_src|dst_property_t to edge_src|dst_property_t
  • [x] Update update_edge_partition_src_dst_property.cuh to update_edge_src_dst_property.cuh and update_edge_partition_src|dst_property to update_edge_src|dst_property.
  • [x] Rename dummy_property_t to edge_src_dummy_property_t and edge_dst_dummy_property_t
  • [x] Replaced edge_src|dst_property_t's fill() with fill_edge_src|dst_property (to allow including `edge_src_dst_property.hpp to be compiled with g++).
  • [x] Created edge_src|dst_property_view_t classes (host side, in addition to the device_view objects which can be trivially copied to the device side); so the view() member function can be compiled with g++.

seunghwak avatar Aug 03 '22 23:08 seunghwak

Codecov Report

:exclamation: No coverage uploaded for pull request base (branch-22.10@cc05758). Click here to learn what that means. The diff coverage is n/a.

@@               Coverage Diff               @@
##             branch-22.10    #2503   +/-   ##
===============================================
  Coverage                ?   61.19%           
===============================================
  Files                   ?      106           
  Lines                   ?     5638           
  Branches                ?        0           
===============================================
  Hits                    ?     3450           
  Misses                  ?     2188           
  Partials                ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov-commenter avatar Aug 09 '22 21:08 codecov-commenter

rerun tests

seunghwak avatar Aug 11 '22 19:08 seunghwak

rerun tests

seunghwak avatar Aug 16 '22 16:08 seunghwak

@gpucibot merge

ChuckHastings avatar Aug 23 '22 01:08 ChuckHastings