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

Architecture docs: add arch diagram and lingo updates #2158

Merged
merged 2 commits into from
May 5, 2017

Conversation

ebidel
Copy link
Contributor

@ebidel ebidel commented May 5, 2017

No description provided.

Copy link
Member

@paulirish paulirish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm % nits

* **Gatherers** - Requests data from the browser or collects information from the page (and maybe post-processing)
* **Artifacts** - output of a gatherer
* **Audits** - Tests for something. Either a pass/fail check or calculates a score.
* **Categories** - Grouping audit results into a user-facing section of the report (eg. `Best Practices`). Appies weighting and overall scoring to the section.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Applies

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done


* **Driver** - Interfaces with [Chrome Debugging Protocol](https://developer.chrome.com/devtools/docs/debugger-protocol) ([API viewer](https://chromedevtools.github.io/debugger-protocol-viewer/))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nuke the API viewer link since these two things now redirect to the same place

Copy link
Member

@brendankenny brendankenny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, you committed the diagram to master but made a PR for this? LGTM2, I GUESS


`npm install -g js-vd; vd --exclude "node_modules|third_party|fs|path|url|log" lighthouse-core/ > graph.html`
* **Driver** - Interfaces with [Chrome Debugging Protocol](https://developer.chrome.com/devtools/docs/debugger-protocol) ([API viewer](https://chromedevtools.github.io/debugger-protocol-viewer/))
* **Gatherers** - Requests data from the browser or collects information from the page (and maybe post-processing)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requests data from the browser or collects information from the page

is there another way to phrase this? (collecting information from the page is also data requested from the browser). Should it mention that it's collected via the driver?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requests data from the browser, via the driver. Minimal post-processing.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Uses Driver to collect information about the page. Minimal post-processing."

* **Driver** - Interfaces with [Chrome Debugging Protocol](https://developer.chrome.com/devtools/docs/debugger-protocol) ([API viewer](https://chromedevtools.github.io/debugger-protocol-viewer/))
* **Gatherers** - Requests data from the browser or collects information from the page (and maybe post-processing)
* **Artifacts** - output of a gatherer
* **Audits** - Tests for something. Either a pass/fail check or calculates a score.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Tests for something using the artifacts as inputs" or something?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using the artifacts as input, the audits do core evaluation and pass/fail/scoring.

let's also add this guy.. (probably as a sub-bullet to audits?):

  • Computed Artifacts - Generated on-demand from artifacts, these add additional meaning, and are often shared amongst multiple audits.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Using the Artifacts as input, Audits evaluate a test and assign pass/fail/scoring."

Done

@paulirish
Copy link
Member

paulirish commented May 5, 2017

image 👍 👍 👍 👍 👍

@ebidel ebidel merged commit e46f540 into master May 5, 2017
@ebidel ebidel deleted the ebidel-patch-1-1 branch May 5, 2017 18:43
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.

None yet

3 participants