Skip to content

Commit

Permalink
fix: remove fire ported from Hari's PR #303
Browse files Browse the repository at this point in the history
Signed-off-by: Mehant Kammakomati <mehant.kammakomati2@ibm.com>
  • Loading branch information
kmehant authored and HarikrishnanBalagopal committed Aug 30, 2024
1 parent 6cfb2ff commit eb8d6c3
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
1 change: 0 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ dependencies = [
"trl>=0.9.3,<1.0",
"peft>=0.8.0,<0.13",
"datasets>=2.15.0,<3.0",
"fire>=0.5.0,<1.0",
"simpleeval>=0.9.13,<1.0",
]

Expand Down
5 changes: 2 additions & 3 deletions tuning/sft_trainer.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@
)
from transformers.utils import is_accelerate_available
from trl import SFTConfig, SFTTrainer
import fire
import transformers

# Local
Expand Down Expand Up @@ -508,7 +507,7 @@ def parse_arguments(parser, json_config=None):
)


def main(**kwargs): # pylint: disable=unused-argument
def main():
parser = get_parser()
logger = logging.getLogger()
job_config = get_json_config()
Expand Down Expand Up @@ -629,4 +628,4 @@ def main(**kwargs): # pylint: disable=unused-argument


if __name__ == "__main__":
fire.Fire(main)
main()

0 comments on commit eb8d6c3

Please sign in to comment.