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

Add input data, measurements, benchmarking documentation #274

Merged
merged 8 commits into from
Mar 22, 2023

Conversation

debermudez
Copy link
Contributor

No description provided.

@debermudez debermudez marked this pull request as draft March 21, 2023 17:30
@debermudez debermudez force-pushed the dbermudez-more-docs branch 2 times, most recently from a4f177a to ad3915b Compare March 21, 2023 18:46
@debermudez debermudez changed the base branch from pa-docs to matthewkotila-cli March 21, 2023 18:47
@debermudez debermudez marked this pull request as ready for review March 21, 2023 20:05
src/c++/perf_analyzer/docs/README.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/README.md Show resolved Hide resolved
src/c++/perf_analyzer/docs/benchmarking.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/benchmarking.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/benchmarking.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/data_guide.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/data_guide.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/measurements_metrics.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/README.md Show resolved Hide resolved
src/c++/perf_analyzer/docs/input_data.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/input_data.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@tgerdesnv tgerdesnv left a comment

Choose a reason for hiding this comment

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

Was this PR mostly a cut-and-paste from some other area?

src/c++/perf_analyzer/docs/benchmarking.md Show resolved Hide resolved
@matthewkotila matthewkotila force-pushed the matthewkotila-cli branch 3 times, most recently from 68a0de1 to 340c197 Compare March 22, 2023 16:56
@debermudez debermudez changed the title Add data guide documentation Add input data, measurements, benchmarking documentation Mar 22, 2023
@debermudez
Copy link
Contributor Author

It was mostly cut and paste. Matt and I have been reorganizing and rewording parts of it as we go tho.

@debermudez debermudez force-pushed the dbermudez-more-docs branch from f228871 to 195e356 Compare March 22, 2023 17:26
Base automatically changed from matthewkotila-cli to pa-docs March 22, 2023 20:46
@debermudez debermudez force-pushed the dbermudez-more-docs branch from 6f86cb7 to 2d0fddd Compare March 22, 2023 21:06
@debermudez debermudez merged commit 9afd990 into pa-docs Mar 22, 2023
@debermudez debermudez deleted the dbermudez-more-docs branch March 22, 2023 21:09
matthewkotila pushed a commit that referenced this pull request Apr 12, 2023
* Add data guide documentation

* Add measurement doc

* Add benchmarking doc

* Add more to measurements and metrics doc and new link in readme

* Fix some comments and add more to metrics

* Move data_guide to input_data

* Adjusted header size

* Update all links
matthewkotila pushed a commit that referenced this pull request Apr 13, 2023
* Add data guide documentation

* Add measurement doc

* Add benchmarking doc

* Add more to measurements and metrics doc and new link in readme

* Fix some comments and add more to metrics

* Move data_guide to input_data

* Adjusted header size

* Update all links
matthewkotila pushed a commit that referenced this pull request Apr 18, 2023
* Add data guide documentation

* Add measurement doc

* Add benchmarking doc

* Add more to measurements and metrics doc and new link in readme

* Fix some comments and add more to metrics

* Move data_guide to input_data

* Adjusted header size

* Update all links
matthewkotila added a commit that referenced this pull request Apr 20, 2023
* Add quick start documentation page (#261)

* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments

* Add installation documentation page (#266)

* Add installation documentation page

* Add quick start to docs readme

* Add CLI documentation page (#271)

* Add CLI documentation page

* Addressed comments

* Addressed comments

* Clean up

* Clean up

* Rewrite top-level readme (#273)

* Rewrite top-level readme

* Addressed comments

* Add input data, measurements, benchmarking documentation (#274)

* Add data guide documentation

* Add measurement doc

* Add benchmarking doc

* Add more to measurements and metrics doc and new link in readme

* Fix some comments and add more to metrics

* Move data_guide to input_data

* Adjusted header size

* Update all links

* Add back communication protocol docs, fix links, add bls composing models option (#287)

* Fix various things in docs (#288)

* Fix various things in docs

* Addressed comments

* Proof-read all docs (#290)

* Proof-read all docs

* Addressed comments

* Fix docs

---------

Co-authored-by: Matthew Kotila <matthew.r.kotila@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants