neuralmagic / guidellm

Evaluate and Enhance Your LLM Deployments for Real-World Inference Needs
Apache License 2.0
158 stars 11 forks source link

Adapt build system for dynamic versioning and package naming based on build settings #19

Closed markurtz closed 3 months ago

markurtz commented 3 months ago

Summary

Enhances the build system by introducing dynamic versioning and package naming based on the build type and build number, which are loaded from env variables.

Details

Test Plan

markurtz commented 3 months ago

Why is the black removed from the pre-commit configuration?

I had missed the cleanup on the pre-commit previously with the tox.ini rework. Black is stripped out completely across the system now along with isort and flake8 since ruff has full parity with those and runs much faster

parfeniukink commented 3 months ago

Why is the black removed from the pre-commit configuration?

I had missed the cleanup on the pre-commit previously with the tox.ini rework. Black is stripped out completely across the system now along with isort and flake8 since ruff has full parity with those and runs much faster

alrigh. but in that case ruff can replace isort (probably ruff not that good, but anyway) and flake8 as well.