Skip to content

Update packet logging functionality#10

Open
chiaradistefano wants to merge 4 commits into
HLC-Lab:mainfrom
chiaradistefano:main
Open

Update packet logging functionality#10
chiaradistefano wants to merge 4 commits into
HLC-Lab:mainfrom
chiaradistefano:main

Conversation

@chiaradistefano

Copy link
Copy Markdown
Collaborator

This PR extends the recently introduced packet logging feature to the UEC protocol, allowing for detailed simulation analysis via CSV files. Additionally, it includes a minor fix in the Dragonfly+ configuration file.

Feedback and discussion

The implementations should not create any kind of conflict with existing ones.

Description of changes

  • Packet Logging Expansion: Extended the packet logging feature to the UEC protocol.
    • Modified the queue files to check if packet logging is enabled and handle writing the data to the CSV file accordingly.
    • Updated main_uec.cpp to accept the packet logging flag as a simulation parameter.
  • Dragonfly+ Fix: Included a minor fix in the Dragonfly+ file in the constructor method.

How to run

To enable packet logging in UEC simulations, use the -log packets flag:

./htsim_uec -log packet [other_args...]

PR Impact Assessment

Interaction with CMS specification

None

Impact on default behavior

Modify existing features or parameters

Merging instructions

PR dependencies

DRAFT

Should the PR commit history be kept (make sure it's clean)

No ("squash and merge")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant