Skip to content
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

[BUG]: Minor typos in variables or missing variables #4873

Open
2 tasks done
mcordery opened this issue Jan 17, 2025 · 1 comment · May be fixed by #4875
Open
2 tasks done

[BUG]: Minor typos in variables or missing variables #4873

mcordery opened this issue Jan 17, 2025 · 1 comment · May be fixed by #4875
Assignees
Labels
bug Something isn't working

Comments

@mcordery
Copy link

Version

25.02

Which installation method(s) does this occur on?

Source

Describe the bug.

During a debug build, the following issues cropped up. All these are easily fixable. I think the first one may be fixable by the example of the previous function. I've noted that these are actually still in the repo versions.

Problem 1:
src/prims/vertex_frontier.cuh
in retrieve_vertex_list_from_bitmap()
line 227, no local scope comm or root variable declared as in function previous
Also typo in vertex_ragne_first, should be range
assert((comm.get_rank() != root) ||
(bitmap.size() >= packed_bool_size(vertex_range_last - vertex_ragne_first)));

Problem 2:
Typo in variable
/home/mcordery/Projects/cugraph/cpp/src/prims/detail/per_v_transform_reduce_e.cuh:1613:14: error: use of undeclared identifier 'sorted_uniue_nzd_key_last'; did you mean 'sorted_unique_nzd_key_last'?
1613 | assert(sorted_uniue_nzd_key_last == sorted_unique_key_first + (*key_segment_offsets).back());

Problem 3:
Typo in variable
/home/mcordery/Projects/cugraph/cpp/src/prims/fill_edge_src_dst_property.cuh:973:28: error: no member named 'local_edge_partition_dst_range_sizse' in 'cugraph::graph_view_t<int64_t, int64_t, false, false, void>'
973 | : graph_view.local_edge_partition_dst_range_sizse()));

Minimum reproducible example

Relevant log output

Environment details

Other/Misc.

No response

Code of Conduct

  • I agree to follow cuGraph's Code of Conduct
  • I have searched the open bugs and have found no duplicates for this bug report
@mcordery mcordery added ? - Needs Triage Need team to review and classify bug Something isn't working labels Jan 17, 2025
@ChuckHastings
Copy link
Collaborator

For a long time (a couple of years) we were unable to compile debug builds (OOM in the compiler). We have recently made changes in our C++ TUs that have made it possible... but we are out of the habit. These bugs are being corrected in 25.02.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants