-
Notifications
You must be signed in to change notification settings - Fork 86
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
added MANIFEST.in file to include requirements.txt #140
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
/azp run |
Azure Pipelines successfully started running 2 pipeline(s). |
guenp
approved these changes
Sep 2, 2021
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @AnudeepGunukula for the contribution!
guenp
pushed a commit
that referenced
this pull request
Oct 19, 2021
Add async support for Workspace.get_targets, merge in changes from main: * Add environment variable to set User-Agent prefix (#133) * Add deprecation notice to 3rd party QIO namespaces (#132) * Revert "Support fleet management jobs (#127)" (#134) * Revert "Support fleet management jobs (#127)" * Changing schema of grouped terms and separating term types (#131) * Remove lingering fleet-mgmt (#137) * Update installation instructions in readme and example notebook (#139) * added MANIFEST.in file to include requirements.txt (#140) * Rename env vars to start with AZURE_QUANTUM_ (#141) * Make UT for Workspace.get_targets more robust against provider changes (#144) * Serialize problem name (#64) * Workspace.get_targets() returns generic Target for unknown targets (#148) * Add Qiskit and Cirq plugins for IonQ and Honeywell targets (#149) * fix bug in case insensitive provider lookup (#150) * Update README, add Azure Quantum logo (#154) * Refactor CI/CD steps, add tests against cirq beta (#151) * Update README to use logo file from main branch (#155) Co-authored-by: XField <58103249+vxfield@users.noreply.github.com> Co-authored-by: ashwinmayya <ashwinm@microsoft.com> Co-authored-by: adelebai <adele.bai@live.com.au> Co-authored-by: Andres Paz <anpaz@microsoft.com> Co-authored-by: Anudeep Gunukula <55506841+AnudeepGunukula@users.noreply.github.com>
vxfield
added a commit
that referenced
this pull request
Oct 21, 2021
* Add an async workspace API (#95) * Add an async workspace * Add an async job, async solver and extend the workspace APIs to use the correct types. * Get linked storage has to be async * Add aiohttp requirement * Update conda env as well * refactor into submodule * Add the authentication submodule * Revert some sync job changes leftover * Add async storage and problem interfaces * Refactor the async uses and add an online and streaming problem that removes the need for threading * await problem uploader. * Put the clients into async context managers so that the client sessions are closed. * Workaround for Azure/azure-sdk-for-python#19943 * create client once per workspace * Add async tests and integrate all the refactorings to the job classes and add aio targets. * refactoring async tests * aio streaming problem cant accept terms because that needs to be awaited. Update some of the tests * update the common aio class * update recordings * Add a missing attribute in the recordings and fix a missing await in the job * regex the response URL in the custom response handler * await remaining tests * Fix a bug in the streaming problem causing an infinite loop. Add some test recordings. * clone the sync recordings * add the recordings * refactor the test base to be shared between sync and async * fix broken import * fix UTs * inherit Async classes from Sync classes * add missing targets, add async workspace.get_targets Co-authored-by: XField <58103249+vxfield@users.noreply.github.com> Co-authored-by: Guen Prawiroatmodjo <guenp@microsoft.com> * Add async support for Workspace.get_targets (#135) Add async support for Workspace.get_targets, merge in changes from main: * Add environment variable to set User-Agent prefix (#133) * Add deprecation notice to 3rd party QIO namespaces (#132) * Revert "Support fleet management jobs (#127)" (#134) * Revert "Support fleet management jobs (#127)" * Changing schema of grouped terms and separating term types (#131) * Remove lingering fleet-mgmt (#137) * Update installation instructions in readme and example notebook (#139) * added MANIFEST.in file to include requirements.txt (#140) * Rename env vars to start with AZURE_QUANTUM_ (#141) * Make UT for Workspace.get_targets more robust against provider changes (#144) * Serialize problem name (#64) * Workspace.get_targets() returns generic Target for unknown targets (#148) * Add Qiskit and Cirq plugins for IonQ and Honeywell targets (#149) * fix bug in case insensitive provider lookup (#150) * Update README, add Azure Quantum logo (#154) * Refactor CI/CD steps, add tests against cirq beta (#151) * Update README to use logo file from main branch (#155) Co-authored-by: XField <58103249+vxfield@users.noreply.github.com> Co-authored-by: ashwinmayya <ashwinm@microsoft.com> Co-authored-by: adelebai <adele.bai@live.com.au> Co-authored-by: Andres Paz <anpaz@microsoft.com> Co-authored-by: Anudeep Gunukula <55506841+AnudeepGunukula@users.noreply.github.com> * set user agent * append user agent info rather than replace it * test if user_agent is set * append with dash instead of space * add test * test long user agent, test from env var * close client sessions before returning * close client for download_blob * close client for download_blob * Fixes to feature/aio (#166) * Async fixes on storage * Fixes on Async workspace * Fixes on Async Workspace * Fixes in the async _authentication * Linting fixes * Fixes on Async job * Fixes in the Async Optimization * Fixes on Async Target * Update azure-quantum/azure/quantum/job/base_job.py Co-authored-by: Guen Prawiroatmodjo <guenp@microsoft.com> * Revert Async Target changes Co-authored-by: Guen Prawiroatmodjo <guenp@microsoft.com> * Async oneqbit.py File rename (#167) * Async fixes on storage * Fixes on Async workspace * Fixes on Async Workspace * Fixes in the async _authentication * Linting fixes * Fixes on Async job * Fixes in the Async Optimization * Fixes on Async Target * Update azure-quantum/azure/quantum/job/base_job.py Co-authored-by: Guen Prawiroatmodjo <guenp@microsoft.com> * Revert Async Target changes * Typo while renaming the file * Typo renaming oneqbit file Co-authored-by: Guen Prawiroatmodjo <guenp@microsoft.com> * fix circular import * update environment yml files * update env name * Async PR fixes (#170) * Port UserAgent changes from Sync to Async * Fix async streaming problem * Fix Async tests * add async mark * Async PR fixes (#171) * Port UserAgent changes from Sync to Async * Fix async streaming problem * Fix Async tests * Minor fixes * Update recordings * add missing awaits and methosd to streaming problem * rerecord tests, update env var name * copy aio tests * asyncio mark * Remove .aio from azure-quantum async conda env Co-authored-by: Anthony Shaw <anthony.p.shaw@gmail.com> Co-authored-by: XField <58103249+vxfield@users.noreply.github.com> Co-authored-by: ashwinmayya <ashwinm@microsoft.com> Co-authored-by: adelebai <adele.bai@live.com.au> Co-authored-by: Andres Paz <anpaz@microsoft.com> Co-authored-by: Anudeep Gunukula <55506841+AnudeepGunukula@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi , this pull request will close the issue #57
i had added the MANIFEST.in in
{root}/azure-quantum/
folder which will include therequirements.txt
fileafter creating source distribution.
i had also tested it by running cmd
python setup.py sdist
. its working fine.Kindly let me know if any changes required
Thankyou.