-
Notifications
You must be signed in to change notification settings - Fork 65
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
Release v0.20.0 #732
Merged
Merged
Release v0.20.0 #732
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
* Associate states to bases * Add states to conventions * Use states in Hamiltonian * Add Channel.eigenstates, corespondance btw eigenstates and labels * Fixing widths in table * Revert changes to convetions, make table in Channels docstring * Add r""" * Fix indentation * Fix table in eigenstates docstring * Fix typo * Add multiple_bases_states, check for eigenstates * Sort imports * Move test on EIGENSTATES to unit tests * Change name of multiple_bases_states * Fix typo * Fix import of Collection
* First rework of the NoiseModel interface * Fix typing * Avoid resampling when amp_sigma=0 * Isolating code to find relevant noise parameters * Define a custom NoiseModel.__repr__() * Deprecating noise_types definition * Improving adjacent UTs * Complete NoiseModel UTs * Update NoiseModel JSON schema * Fix docstring indentation * Implementing review suggestions * Allowing temperature to be 0 * Disallow temperature to be null in JSON schema
* Allow modification of the EOM setpoint without disabling EOM mode * Adding abstract repr support
* Add leakage noise in NoiseModel * Modifying default value of with_leakage, taking out of SimConfig arguments * Deleting with_leakage from legacy, define bool as relevant if True * Fix failing tests * Fixing docstring for API doc * Test leakage in Simulation * Imrpove handling of operator's shape, delete argument of find_relevant_params * Fixing docs * Fixing nits * Delete condition on dephasing and depolarizing * Go back to previous schema * Fix typing
* Hide amplitude when drawing DMM channels * Ignore DeprecationWarning in legacy jsonschema
* Enable definition of effective noise operators in all basis * Modifying definition of collapse operators in dephasing * Fix tests * Fixing notebook * Revert abs --------- Co-authored-by: Henrique Silvério <[email protected]>
* Modifying test for relaxation * Include macos tests in CI * Take out dephasing_relaxation result * Revert ci to without macos tests
* Enable definition of leakage with all basis * Validate test results * Test simulation with leakage * fixing set_config * Fix failing tests * Fix set_config, add_config, coverage of qutip_result * Fix NoisyResults, Improve test_simresults * Fixing nits * Testing projections * Fix set_config, add_config documentation * Extend set of noise to fix tests * print deletion
* Allow specification of job IDs in RemoteResults * Rename submission_id to batch_id
* Defining pulser.math and AbstractArray * POC: Differentiable constant pulse amp Typing is still failing * Fix typing in waveforms * Fix all typing errors in POC * Pass all existing UTs * Pass all UTs without array support * Fix typing * All tests pass with torch installed * Add support for pulser-diff backend (#686) * works with basic features of pulser-diff * Fixed phase attribute setting; removed debugging code; reverted unnecessary changes * Modified register creation code to work with AbstractArray; register coordinates are differentiable with pulser-diff * Fixed type hints * Minor fixes and refactoring * Modified ParamObj code to work with quantum model training in pulser-diff * Minor refactoring; add possibility to ensure 0D AbstractArray is reshaped into 1D * Force array only for scalars * Fix UTs after pulser-diff changes * Avoid using AbstractArrayLike outside of pulser.math * Preserve gradient in EOM mode * Add torch as an optional requirement * Support waveform multiplication with abstract array * Explicitly marking the differentiable parameters * Remove __array_wrap__ * Pass relevant UTs without array support * Support new features * Using pm.Differentiable whenever possible * Simplifying Waveform.__getitem__() type hint * UTs for new features outside of pulser.math * Write torch UTs for registers * Write UTs for waveforms * UTs for pulse * UTs for EOM * UTs on internal functionality * UTs for Sequence with autograd * Implicitly cover math functions * Removing AbstractArray.__hash__() and differentiable phase shifts * Finish unit tests * Update CI to run tests with and without torch * Fix CI errors * Fix failing no-torch UT * Minor corrections * Include pulser[torch] installation in the README * Fix warning in UT after merge * Incorporating the latest changes * Fix typing * Addressing review comments * Including `detach()` in Differentiable protocol * Differentiable -> TensorLike * Tentatively allow waveform division by array * Full coverage --------- Co-authored-by: Vytautas Abramavicius <[email protected]>
* Loop over all possible matchings in switch_device * Decrease strictness on bottom_detuning and total_bottom_detuning, fix handling of add_dmm_detuning * Relax conditions on EOM configuration * Fix switching with parametrized sequence and EOM * Addressing review comments * Creating helpers class for _seq_str and _switch_device * Add more conditions on EOM in parametrized Seq * Fixing type * Handle having two controlled beams, * Fixing nits * Fix typing --------- Co-authored-by: Henrique Silvério <[email protected]>
* Add from_abstract_repr to Device and VirtualDevice * Adding warning section on the behaviour of from_abstract_repr in Device and VirtualDevice * Address review comment in test_abstract_repr, solve typing --------- Co-authored-by: Henrique Silvério <[email protected]>
* Introduce new method to return available results * mypy * review * Apply suggestions from code review Co-authored-by: Henrique Silvério <[email protected]> * [TEST] Add test for get_available_result and update existing tests * style * Add more assertions to tests * Move sequence declaration to dedicated function * style --------- Co-authored-by: Henrique Silvério <[email protected]>
* rework sdk to not require batch_id as an argument * rework sdk to not require batch_id as an argument * rework sdk to not require batch_id as an argument * rework sdk to not require batch_id as an argument * rework sdk to not require batch_id as an argument * change to context manager interface for open batches * fix rebase fix rebase, and linting fix rebase, and linting * fix rebase, and linting fix rebase, and linting fix rebase, and linting fix rebase, and linting fix rebase, and linting * fix type fix type * complete test coverage for method calls complete test coverage for method calls * context management class, update tests context management class, update tests * inside return is ignored with _ * mr feedback * boolean condition for open batch support boolean condition for open batch support boolean condition for open batch support boolean condition for open batch support * test coverage * flake8 * MR feedback MR feedback * comment on arg name * support complete -> open keyword change for batches * support complete -> open keyword change for batches * lint lint * Bump pasqal-cloud to v0.12 * Include only the new jobs in the RemoteResults of each call to submit() * Give stored batch ID to get available results * Submission -> Batch outside of RemoteResults * Including backend specific kwargs to RemoteConnection.submit() when opening batch * Fully deprecate 'submission' for 'batch' * Relax `pasqal-cloud` requirement * Consistency updates to the tutorial --------- Co-authored-by: oliver.gordon <[email protected]> Co-authored-by: HGSilveri <[email protected]> Co-authored-by: Henrique Silvério <[email protected]>
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
a-corni
approved these changes
Sep 20, 2024
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.
Let's go for v0.20 !
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.
Main changes: