Skip to content
This repository was archived by the owner on Oct 16, 2023. It is now read-only.

polish code and add setup script#207

Open
juncongmoo wants to merge 1 commit intohpcaitech:mainfrom
juncongmoo:release
Open

polish code and add setup script#207
juncongmoo wants to merge 1 commit intohpcaitech:mainfrom
juncongmoo:release

Conversation

@juncongmoo
Copy link
Copy Markdown
Contributor

  • polish code
  • add setup script
  • upgrade version to 0.0.2

@juncongmoo
Copy link
Copy Markdown
Contributor Author

juncongmoo commented Mar 7, 2023

It is safe to approve. Please let me know if there is anything I need to change. Just want to help...

@binmakeswell
Copy link
Copy Markdown
Member

Hi @juncongmoo Thanks so much for your contribution!

@binmakeswell binmakeswell requested a review from ver217 March 20, 2023 06:04
colossalai
torchaudio
omegaconf
hiq-python
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why we need hiq?

Copy link
Copy Markdown
Contributor Author

@juncongmoo juncongmoo Mar 20, 2023

Choose a reason for hiding this comment

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

Oh. I was using hiq to benchmark the fastapi servers in examples folder and planning to submit several more PRs after this one were merged. One example is:

image

I tried to revert to remove hiq but found I need to write many tedious functions to do the same thing, so I kept it there, also considering it is a very friendly library :-).

If you think we should remove it, I will come back two days later to remove it because I am really busy on something else right now. Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants