-
Notifications
You must be signed in to change notification settings - Fork 54
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
Integrate pev2 as a component to vizualize execution plan #1431
Conversation
Could we add some text to explain what this does, what tool is used and maybe a link to PEV2? (not sure for the link) |
I tried a sample plan coming from explain.dalibo.com (Example 5) and the visualisation seems broken. looks like js or css is missing. |
8c75446
to
6637940
Compare
6637940
to
e63ab39
Compare
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.
C'est bon pour moi, à terminer avec pierre.
@marion, pourras-tu ajouter une entrée dans changelog ? |
@MarionGiusti Can you please rebase your branch? |
e63ab39
to
5295a72
Compare
Maybe we should tell users what to do. There's no explanation about what this tool does and how to use it. No introduction text, nor placeholders in the textareas. For people who have already used explain.dalibo.com it will be straight forward, for others it will not. In the pev2 projects, the release process is a bit random and there may be several versions released during one year. How will we make sure that we are up-to-date in temBoard? |
5a7a417
to
ce4bed1
Compare
ce4bed1
to
7831fe9
Compare
7831fe9
to
e206bf3
Compare
No description provided.