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 tanh function to ParamObj #798

Merged
merged 3 commits into from
Feb 4, 2025
Merged

Add tanh function to ParamObj #798

merged 3 commits into from
Feb 4, 2025

Conversation

vytautas-a
Copy link
Collaborator

Add tanh function to paramobj

@vytautas-a vytautas-a requested a review from HGSilveri February 4, 2025 15:01
@vytautas-a vytautas-a self-assigned this Feb 4, 2025
vytautas-a and others added 2 commits February 4, 2025 17:43
Co-authored-by: Henrique Silvério <29920212+HGSilveri@users.noreply.github.com>
Copy link
Collaborator

@HGSilveri HGSilveri left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@HGSilveri HGSilveri changed the title Add tanh function to paramobj Add tanh function to ParamObj Feb 4, 2025
Copy link
Collaborator

@a-corni a-corni left a comment

Choose a reason for hiding this comment

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

LGTM as well ;)

@HGSilveri HGSilveri merged commit ba23444 into develop Feb 4, 2025
9 checks passed
a-corni added a commit that referenced this pull request Feb 13, 2025
* Add tanh function to ParamObj (#798)

* added tanh function to paramobj

* Update tests/test_parametrized.py

Co-authored-by: Henrique Silvério <29920212+HGSilveri@users.noreply.github.com>

* fixed test

---------

Co-authored-by: Henrique Silvério <29920212+HGSilveri@users.noreply.github.com>

* Create extended usage doc

* Fix doc

* Post-rereading correction

* Allow switch_device to work when Channel uses BaseEOM (#804)

* Allow switch_device to work when Channel uses BaseEOM

* Bump to v1.2.2

* Bump to v1.3dev2

* Addressing the confusion btw Weighted Analog and XY

* Add more specific links, descr of digital qc

* Address review comments

* Testing with :

* Try with #-

* Fix reference

* Fix links to sections

* Fix hamiltonian

* Fix apidoc ref

---------

Co-authored-by: Vytautas Abramavicius <145791635+vytautas-a@users.noreply.github.com>
Co-authored-by: Henrique Silvério <29920212+HGSilveri@users.noreply.github.com>
Co-authored-by: HGSilveri <henrique.silverio@tecnico.ulisboa.pt>
HGSilveri added a commit that referenced this pull request Feb 14, 2025
* Setting up Markdown support

* Convert Conventions page to markdown

* Attempt to use auto-generated labels

* Introduce section about hardware

* Redirect api core

* Add pulse section

* Direct to API core instead of pulse

* Add some elements about pulser.devices

* Create getting started notebook (#776)

* Create initial notebook

* Add temporary links

* Add programming section

* Fix installation of mermaid

* Fix figure caption

* Taking into account review comments

* Fix doc

* Try mermaid with mkdown

* Address review comments

* Fix doc

* Add picture

* Addressing review comments

* Add figure, vector on the bloch sphere, complement to the Device

* Make introduction

* Add programming.md in ref

* Add label

* delete ref

* Fix programming

* Create tutorial Excitation to rydberg state

* Address review comments

* Add AFM state prep

* Fix programming.md

* Address review

* Delete docs, add hbar

* Fix picture

* Fixing programming

* Transfer cell as raw

* Fix sequence

* 4-hand fixing

* Final fixes

* Add paragraph along br

* Address review comment

* Fix referencing

* Fix the notebook

* Add note about neighbouring atoms

* Creating notebook for hardware section, reorganising specs display in Device, adding the presence of EOM in Channel display

* Fix typing

* Fix build

* fix typing

* Fix flake8

* Fix reference

* Fix ref

* Fix figure

* Re-reading myself

* Fix rendering

* Fix rendering

* Fix build

* Available devices

* Address review comments

* Add page on Register creation and Rydberg-Atom interactions + Retire old tutorials (#790)

* Deleting tutorials

* Adding the Register user manual

* Adding links

* Undo deleting the tutorials for the repo

* Retire tutorials instead

* Do the same for the tutorials deleted in #776

* Address review comments

* Fix notebook reference

* Clear all outputs

* Adressing review comments

* Address review comments

* Add section in conventions on indexed operator notation

* Small fixes

* Improve formula rendering

* Shuffle the description of the arguments

* Trying new raw cells

* Add remove-input tag to last cell

* Fix typing

* Add no-index

* Delete no-members

* Dlete no-index

* Reintroduce :noindex:

* Improve description of device

* Try fixing link to

* Fix device and channel description, try hidding the input

* Try fixing build

* Fix tags to hide_input

* Change device descr to "Args", stop trying to mask input

* Add link to backend

* Move Attributes to Args in Device

* Add mention to API doc

* Apply some Pasqal branding to the HTML documentation

* Use Pulser logo :)

* Remove obsolete file

* Add seealso sections in programming

* Deleting the-channels

* Tag The-Channels

* referencing to html instead

* Going for hardware.ipynb

* Add page on Pulses and Waveforms (#793)

* Add tutorial on Pulses and Waveforms

* Address review comments

* Reference fundamentals pages in Programming

* Remove signatures from autosummary

* API Reference Rework (#722)

* Improving autodoc typehints generation

* Improving the Register API doc

* WIP: New API reference for pulser-core

* Try to pass RTD build

* Adding `pulser-simulation` module to the API ref

* Adding pulser_pasqal ref

* Import sorting

* Fix UTs

* Fix links in new pages

* Adding the Sequence page (#802)

* WIP

* Getting links to work at top-level pulser API

* Writing Sequence page

* Address review comments

* Fix typo

* Merge remote-tracking branch 'origin' into docs-v2

* Modifications to the TOC tree (#806)

* Reorder pages in Extended Usage

* Retire tutorial on AFM state preparation

* Slight modifications to the Backend page

* Move and update the XY Hamiltonian page

* Fix broken links

* Fix typo

Co-authored-by: Antoine Cornillot <61453516+a-corni@users.noreply.github.com>

---------

Co-authored-by: Antoine Cornillot <61453516+a-corni@users.noreply.github.com>

* Create introduction to Extended Usage (#800)

* Add tanh function to ParamObj (#798)

* added tanh function to paramobj

* Update tests/test_parametrized.py

Co-authored-by: Henrique Silvério <29920212+HGSilveri@users.noreply.github.com>

* fixed test

---------

Co-authored-by: Henrique Silvério <29920212+HGSilveri@users.noreply.github.com>

* Create extended usage doc

* Fix doc

* Post-rereading correction

* Allow switch_device to work when Channel uses BaseEOM (#804)

* Allow switch_device to work when Channel uses BaseEOM

* Bump to v1.2.2

* Bump to v1.3dev2

* Addressing the confusion btw Weighted Analog and XY

* Add more specific links, descr of digital qc

* Address review comments

* Testing with :

* Try with #-

* Fix reference

* Fix links to sections

* Fix hamiltonian

* Fix apidoc ref

---------

Co-authored-by: Vytautas Abramavicius <145791635+vytautas-a@users.noreply.github.com>
Co-authored-by: Henrique Silvério <29920212+HGSilveri@users.noreply.github.com>
Co-authored-by: HGSilveri <henrique.silverio@tecnico.ulisboa.pt>

---------

Co-authored-by: a_corni <antoine.cornillot@pasqal.com>
Co-authored-by: Antoine Cornillot <61453516+a-corni@users.noreply.github.com>
Co-authored-by: Benjamin Becquet <benjamin.becquet@pasqal.com>
Co-authored-by: Benjamin Becquet <benjamin.becquet@gmail.com>
Co-authored-by: Vytautas Abramavicius <145791635+vytautas-a@users.noreply.github.com>
@HGSilveri HGSilveri mentioned this pull request Feb 14, 2025
HGSilveri added a commit that referenced this pull request Feb 14, 2025
**Main Changes:**
- Add tanh function to ParamObj (#798)
- Upgrades to AnalogDevice specs (#807) 
- **Defining the new Backend API classes** (#764)
- **Documentation Upgrade** (#811)
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