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.

Sign up for GitHub

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

Jump to bottom

coz: Causal profiler #3282

Closed
wants to merge 3 commits into from
Closed

coz: Causal profiler #3282

wants to merge 3 commits into from

Conversation

klimkin
Copy link
Contributor

@klimkin klimkin commented Oct 20, 2020 •
edited
Loading

coz/0.2.2

Requires libelfin/0.3 #3283

  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.

@conan-center-bot
Copy link
Collaborator

config.yml syntax error in build 1:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

@klimkin klimkin changed the title Add libelfin and coz coz: Causal profiler Oct 20, 2020
@conan-center-bot
Copy link
Collaborator

config.yml syntax error in build 2:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

@uilianries
Copy link
Member

@klimkin Please, separate libelfin from this PR.

@conan-center-bot
Copy link
Collaborator

config.yml syntax error in build 3:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

@klimkin
libelfin: reading ELF binaries and DWARFv4 debug information
f190114 
C++11 library for reading ELF binaries and DWARFv4 debug information.
@conan-center-bot
Copy link
Collaborator

config.yml syntax error in build 4:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

@klimkin
Copy link
Contributor Author

klimkin commented Oct 23, 2020 •
edited
Loading

@klimkin Please, separate libelfin from this PR.

@uilianries This recipe requires libelfin. For libelfin, please see #3283

@uilianries
Copy link
Member

@klimkin This PR still contains 2 different packages. CCI does allow it, please, remove one of these packages.

@klimkin
Copy link
Contributor Author

klimkin commented Oct 26, 2020

@klimkin This PR still contains 2 different packages. CCI does allow it, please, remove one of these packages.

@uilianries coz requires libelfin which is submitted as a separate PR. This PR is rebased on #3283. It won't build is libelfin is not available.

@uilianries
Copy link
Member

@klimkin if it won't build, please close this PR.

@klimkin @SpaceIm
Apply suggestions from code review
3ad8018 
Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
@conan-center-bot
Copy link
Collaborator

config.yml syntax error in build 5:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

This was referenced Oct 28, 2020
Closed
Closed
@klimkin
Copy link
Contributor Author

klimkin commented Oct 28, 2020

@emeryberger I noticed coz mentions coz-profiler as the package name, but the repo name is coz. Which name for the package would you prefer?

@klimkin
coz: Causal profiler
fbae9df 
Coz is a new kind of profiler that unlocks optimization opportunities
missed by traditional profilers. Coz employs a novel technique we call
causal profiling that measures optimization potential. This measurement
matches developers' assumptions about profilers: that optimizing
highly-ranked code will have the greatest impact on performance. Causal
profiling measures optimization potential for serial, parallel, and
asynchronous programs without instrumentation of special handling for
library calls and concurrency primitives. Instead, a causal profiler
uses performance experiments to predict the effect of optimizations.
This allows the profiler to establish causality: "optimizing function X
will have effect Y," exactly the measurement developers had assumed they
were getting all along.

https://github.com/plasma-umass/coz
@conan-center-bot
Copy link
Collaborator

config.yml syntax error in build 6:

Only one library can be changed in the same PR: [coz/all, libelfin/all]|

@emeryberger
Copy link

Hi - I'd prefer coz if that's possible!

@ericLemanissier ericLemanissier mentioned this pull request Nov 12, 2020
Closed
@ghost ghost mentioned this pull request Nov 16, 2020
Closed
@stale
Copy link

stale bot commented Dec 1, 2020

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Dec 1, 2020
@klimkin
Copy link
Contributor Author

klimkin commented Dec 3, 2020

Will resubmit once #3283 is merged.

@klimkin klimkin closed this Dec 3, 2020
@emeryberger
Copy link

Noticed that #3283 was merged, FWIW.

@klimkin
Copy link
Contributor Author

klimkin commented Jan 14, 2021

@emeryberger See re-opened #3794. This is now waiting for merging CMake build for coz, since it will simplify the recipe: plasma-umass/coz#178

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Failed stale
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@klimkin @conan-center-bot @uilianries @emeryberger