Skip to content

Commit

Permalink
Misc LLM setup fixes
Browse files Browse the repository at this point in the history
Signed-off-by: Anuradha Karuppiah <[email protected]>
  • Loading branch information
AnuradhaKaruppiah committed Aug 19, 2024
1 parent 0a4b8b5 commit ae5431f
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 116 deletions.
2 changes: 1 addition & 1 deletion python/morpheus_llm/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ list(APPEND CMAKE_MESSAGE_CONTEXT "morpheus_llm")

# Save the root of the python for relative paths
# Where is this used, fixme
set(MORPHEUS_PY_ROOT ${CMAKE_CURRENT_SOURCE_DIR}/..)
set(MORPHEUS_LLM_PY_ROOT ${CMAKE_CURRENT_SOURCE_DIR}/..)


# #################################
Expand Down
3 changes: 2 additions & 1 deletion python/morpheus_llm/MANIFEST.in
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
recursive-include morpheus *.cpython*.so py.typed *.pyi
include morpheus_llm/_version.py
recursive-include morpheus_llm *.cpython*.so py.typed *.pyi
2 changes: 1 addition & 1 deletion python/morpheus_llm/morpheus_llm/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
list(APPEND CMAKE_MESSAGE_CONTEXT "morpheus_llm")

# Save the root of the python for relative paths
set(MORPHEUS_PY_ROOT ${CMAKE_CURRENT_SOURCE_DIR}/..)
set(MORPHEUS_LLM_PY_ROOT ${CMAKE_CURRENT_SOURCE_DIR}/..)

add_subdirectory(_lib)

Expand Down
Empty file.
112 changes: 0 additions & 112 deletions python/morpheus_llm/setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -24,115 +24,3 @@ versionfile_build = morpheus_llm/_version.py
tag_prefix = v
parentdir_prefix = morpheus_llm-

# ===== isort Config =====
[isort]
line_length=120
multi_line_output=3
include_trailing_comma=True
force_grid_wrap=0
combine_as_imports=True
order_by_type=True
force_single_line=True
src_paths=morpheus_llm,tests
known_dask=
dask
distributed
dask_cuda
known_rapids=
nvstrings
nvcategory
nvtext
cudf
cuml
cugraph
dask_cudf
known_first_party=
morpheus
morpheus_llm
_utils
default_section=THIRDPARTY
sections=FUTURE,STDLIB,THIRDPARTY,DASK,RAPIDS,FIRSTPARTY,LOCALFOLDER
skip=
__init__.py
# Skip _version.py as it is auto-generated
_version.py
.eggs
.git
.hg
.mypy_cache
.tmp
.tox
.venv
buck-out
build
dist
models
thirdparty

# ===== flake8 Config =====
[flake8]
filename = *.py, *.pyx, *.pxd

exclude =
__pycache__,
.git,
.tmp/,
*.egg,
build/,
cpp,
docs,
models/,
thirdparty
max-line-length = 120
max-doc-length = 120

extend-ignore =
# Ignore missing docstrings __init__ methods as we document those on the class
D107,
# Don't require quotes to be placed on the same line as a one-line docstring, useful when the docstring is close
# to the line limit
D200,
# Allow a blank line between a docstring and the code
D202,
# D204: 1 blank line required after class docstring
D204,
# D205: 1 blank line required between summary line and description (allow for a paragraph)
D205,
# D400: First line should end with a period (only works if we're adhering to D205)
D400,
# D401: First line should be in imperative mood
D401

# Cython Rules ignored:
# E999: invalid syntax (works for Python, not Cython)
# E225: Missing whitespace around operators (breaks cython casting syntax like <int>)
# E226: Missing whitespace around arithmetic operators (breaks cython pointer syntax like int*)
# E227: Missing whitespace around bitwise or shift operator (Can also break casting syntax)
# W503: line break before binary operator (breaks lines that start with a pointer)
# W504: line break after binary operator (breaks lines that end with a pointer)

per-file-ignores =
# imported but unused
__init__.py: F401, E402
# Ignore additional deps needed for examples
examples/*.py: F821
# Cython Exclusions
*.pyx: E999, E225, E226, E227, W503, W504
*.pxd: E999, E225, E226, E227, W503, W504
# Remove some documentation requirements for tests
tests/*.py:
# D100: Missing docstring in public module
D100,
# D101: Missing docstring in public class
D101,
# D102: Missing docstring in public method
D102,
# D103: Missing docstring in public function
D103

[yapf]
based_on_style = pep8
column_limit = 120
split_all_top_level_comma_separated_values = true
join_multiple_lines = true
indent_dictionary_value = true
2 changes: 1 addition & 1 deletion python/morpheus_llm/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
],
author="NVIDIA Corporation",
include_package_data=True,
packages=find_packages(include=["morpheus*"], exclude=['tests']),
packages=find_packages(exclude=['tests']),
install_requires=[],
license="Apache",
python_requires='>=3.10, <4',
Expand Down

0 comments on commit ae5431f

Please sign in to comment.