-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Add skeleton code of fgviewer #8303
Conversation
Format the file
Add comment to fgviewer related lines Fix cmake commands
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's a pretty good start. please address powei's comments and then as long as the build's not broken, we can commit this
|
||
CivetServer* mServer; | ||
|
||
tsl::robin_map<FrameGraphInfoKey, FrameGraphInfo> mViews; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know we use robin_map in a lot of places, but we're thinking about moving away from it enventually. So for this, I would just suggest to stuck to the std::unordered_map, it's probably fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
BTW, I'm wondering what's the different between using these two classes? And why do we want to use robin_map at the beginning?
Address the comments
ff0230c
to
8e58dff
Compare
fgviewer
being built before the feature is readyfgviewer
folder tolibs
and create some skeleton codes of debug server