Previously, #812 only dedups after syn_generic, but I found that sometimes, syn_map will also unnessarily uniquify ILMs. This adds the dedup_ilm step after both syn_generic and syn_map.
Related PRs / Issues
Type of change:
[ ] Bug fix
[ ] New feature
[ ] Other enhancement
Impact:
[ ] Change to core Hammer
[ ] Change to a Hammer plugin
[ ] Other
Contributor Checklist:
[ ] Did you set master as the base branch?
[ ] Did you state the type-of-change/impact?
[ ] Did you delete any extraneous prints/debugging code?
[ ] (If applicable) Did you add documentation for the feature?
[ ] (If applicable) Did you update the poetry.lock file if you updated the requirements in pyproject.toml?
[ ] (If applicable) Did you add a unit test demonstrating the PR?
[ ] (If applicable) Did you run this through the e2e integration tests?
[ ] (If applicable) Did you update the submodules in e2e/ if this feature depends on updated plugins?
Previously, #812 only dedups after
syn_generic, but I found that sometimes,syn_mapwill also unnessarily uniquify ILMs. This adds thededup_ilmstep after bothsyn_genericandsyn_map.Related PRs / Issues
Type of change:
Impact:
Contributor Checklist:
masteras the base branch?poetry.lockfile if you updated the requirements inpyproject.toml?e2e/if this feature depends on updated plugins?