Skip to content
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

Add requirements.txt to MANIFEST.in #493

Merged
merged 1 commit into from
Aug 23, 2021
Merged

Add requirements.txt to MANIFEST.in #493

merged 1 commit into from
Aug 23, 2021

Conversation

xhochy
Copy link
Contributor

@xhochy xhochy commented Aug 23, 2021

Thanks @xadupre for uploading the sdist. Sadly I cannot build from it as the requirements.txt file is not part of it and the setup.py requires it. The following PR should fix it.

Signed-off-by: Uwe L. Korn <uwe.korn@quantco.com>
@janjagusch
Copy link
Contributor

@xadupre, would be great if you could release another tag after this is released. 🙂

@xadupre
Copy link
Collaborator

xadupre commented Aug 23, 2021

Sure I'll do that. Sorry for the mistake.

@xadupre xadupre merged commit 0da9cab into onnx:master Aug 23, 2021
@xadupre
Copy link
Collaborator

xadupre commented Aug 23, 2021

I published a new release https://github.com/onnx/onnxmltools/releases/tag/1.9.1, let me know if this one works for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants