Fix for duplicate row IDs in `log_parsing` output #2031

dagardner-nv · 2024-10-31T22:30:06Z

Description

Use the index of the incoming DF as the source of the doc field not the first column of the sequence_ids
Fix typeo in docstring for TableInfo::num_indices (unrelated)
Remove declaration for unimplemtned/unused function in python/morpheus/morpheus/_lib/include/morpheus/stages/deserialize.hpp (unrelated)

Closes #2029

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

dagardner-nv · 2024-11-01T00:57:58Z

/merge

dagardner-nv added 3 commits October 31, 2024 14:29

Remove declaration of unused/unimplemented function

d2975e6

Fix typeo in docstring

b75bdf1

Use the inxoming index as the 'doc' field not the sequence ids

e09deac

dagardner-nv requested review from a team as code owners October 31, 2024 22:30

dagardner-nv changed the title ~~Fix for duplicate row id~~ Fix for duplicate row IDs in log_parsing output Oct 31, 2024

dagardner-nv self-assigned this Oct 31, 2024

dagardner-nv changed the base branch from branch-25.02 to branch-24.10 October 31, 2024 22:31

dagardner-nv added bug Something isn't working non-breaking Non-breaking change labels Oct 31, 2024

yczhang-nv approved these changes Oct 31, 2024

View reviewed changes

dagardner-nv marked this pull request as draft October 31, 2024 23:27

dagardner-nv added 2 commits October 31, 2024 16:43

Fix applying seq_ids to start/stop cols

7b5f54a

Fix test

9fb9983

dagardner-nv marked this pull request as ready for review October 31, 2024 23:45

rapids-bot bot merged commit 986abdb into nv-morpheus:branch-24.10 Nov 1, 2024
12 of 13 checks passed