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

User Interactive Mode #70 #76

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

masfiyan
Copy link

I have successfully implement that how user adjust with time window a/c to it outlined in issue #70

Changes Made:

  • I have created Sootty Configuration where every user made configuration will be done over there, in which also I add 3 functions in it, like - set_user_start, set_user_end and get_time_window
  • Implemented Sootty in visualizer by main code

Remaining Work:
May include adjustment of visible wires and other parameters, further more complex enhancement which make sootty more user-friendly

@masfiyan
Copy link
Author

masfiyan commented Feb 18, 2024

Successfully Implement to just displaying the result of user-requested visible wires

Changes Made:

  • Initialize visible wires list in Sootty Configuration, and update Get & Set Methods
  • Implement logic in Visualizer to just displaying the required waveform
  • Do setting up in main

Remaining Work:
Now moving forward to implement this via CLI as query lang

@masfiyan
Copy link
Author

Successfully implement user interactive mode in query language in CLI

Syntax:

sootty "example/example1.vcd" -p "start=10:end=20:visible_wires=D1,D2,D0,Data,clock" -o > image.svg

Changes Made:
I have parse another argument for user parameter as -p where until now there are only 3 parameters handle like
start defines the starting time window
end defines the ending time window
visible_wires just enable to displaying the mention wire in waveform

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