Skip to content

[WIP] Add pyitt 1.1.0 #143

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

Closed
wants to merge 1 commit into from
Closed

[WIP] Add pyitt 1.1.0 #143

wants to merge 1 commit into from

Conversation

esuldin
Copy link
Contributor

@esuldin esuldin commented Apr 23, 2024

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this file required. License files are already available in LICENSES directory.

@@ -0,0 +1,12 @@
include LICENSE
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it required to remove LICENSE from pyitt folder and refer license files from LICENSE folder in base directory

include setup.py
recursive-include pyitt *.py
recursive-include pyitt.native *.cpp *.hpp
recursive-include ittapi/include *.h
Copy link
Contributor

Choose a reason for hiding this comment

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

change 'ittapi/' to '../'

@@ -0,0 +1,109 @@
![PyPI - Python Version](https://img.shields.io/pypi/pyversions/pyitt)
Copy link
Contributor

Choose a reason for hiding this comment

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

python 3.11 and 3.12 required?


pyitt package is available on PyPi and can be installed in the usual way for the supported configurations:

pip install pyitt
Copy link
Contributor

Choose a reason for hiding this comment

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

will this change after publishing?


2. Clone the repository:

git clone --recurse-submodules https://github.com/esuldin/pyitt.git
Copy link
Contributor

Choose a reason for hiding this comment

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

change repo url


3. Clone the repository

git clone --recurse-submodules https://github.com/esuldin/pyitt.git
Copy link
Contributor

Choose a reason for hiding this comment

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

change repo url

IS_64_ARCHITECTURE = sys.maxsize > 2 ** 32

# Check if custom location for ITT API source code is specified
ITT_DEFAULT_DIR = 'ittapi'
Copy link
Contributor

Choose a reason for hiding this comment

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

should this be '../' ?

name = "pyitt"
version = "1.1.0"
authors = [
{ name="Egor Suldin", email="rd3tap@yandex.ru" },
Copy link
Contributor

Choose a reason for hiding this comment

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

should we change non-intel email ids?

]

[project.urls]
"Homepage" = "https://github.com/esuldin/pyitt"
Copy link
Contributor

Choose a reason for hiding this comment

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

update repo urls

@eparshut
Copy link
Contributor

The changes from this pull request have been fetched to the dev_python_ittapi development branch. All other development to add python support to ittapi will be done there. To track the progress see: #145

@eparshut eparshut closed this May 14, 2024
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.

3 participants