-
Notifications
You must be signed in to change notification settings - Fork 49
[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
Conversation
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.
Is this file required. License files are already available in LICENSES directory.
@@ -0,0 +1,12 @@ | |||
include LICENSE |
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.
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 |
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.
change 'ittapi/' to '../'
@@ -0,0 +1,109 @@ | |||
 |
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.
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 |
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.
will this change after publishing?
|
||
2. Clone the repository: | ||
|
||
git clone --recurse-submodules https://github.com/esuldin/pyitt.git |
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.
change repo url
|
||
3. Clone the repository | ||
|
||
git clone --recurse-submodules https://github.com/esuldin/pyitt.git |
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.
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' |
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.
should this be '../' ?
name = "pyitt" | ||
version = "1.1.0" | ||
authors = [ | ||
{ name="Egor Suldin", email="rd3tap@yandex.ru" }, |
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.
should we change non-intel email ids?
] | ||
|
||
[project.urls] | ||
"Homepage" = "https://github.com/esuldin/pyitt" |
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.
update repo urls
The changes from this pull request have been fetched to the |
No description provided.