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.

Sign up for GitHub

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

Jump to bottom

Merge develop to release for v1.0.0 #291

Merged
merged 1,278 commits into from
Apr 10, 2024
Merged

Merge develop to release for v1.0.0 #291

merged 1,278 commits into from
Apr 10, 2024

Conversation

BECATRUE
Copy link
Member

This is a part of #289.

kangz12345 and others added 30 commits January 30, 2024 13:20
@kangz12345
Add use_client() decorator
a6b34ca
@kangz12345
Add description in use_client()
c939573
@kangz12345
Handle OSError in use_client()
3d3c179
@kangz12345
Add moveTo and moveBy
2128c3d
@kangz12345
Fix typo
d48343a
@kangz12345
Fix typo
afd2fc0
@kangz12345
Fix missing self
06da5bd
@kangz12345
Make StageManager a QObject
dd14c8c
@kangz12345
Add StageProxy for convenient signal connection
f723b80
@kangz12345
Add usage example docstring in StageProxy
8b1113f
@kangz12345
Remove widget parts for small PR
a6f631f
@kangz12345
Remove trailing whitespaces
d1f45e2
@kangz12345
Add module docstring
06120fa
@kangz12345
Move use_client() outside StageManager
aaf3fd3
@kangz12345
Add type annotation
009df58
@kangz12345
Add pylint disable comments
5dca72c
@kangz12345
Add consistent return value
4fd3ab8
@kangz12345
Change cache -> lru_cache(maxsize=8)
d861a5e
@kangz12345
Add pylint disable comment
fe1918c
@kangz12345
Remove default argument from use_client()
9f05f9b 
Wrong way to define a decorator with arguments.
@kangz12345
Make 2 blank lines between global definitions
617b668
@Aijuh
range-scan
bde9179 
1. Implement RangeScan widget.
2. Add codes for sending scan arguments to artiq proxy.
@Aijuh
format changed
2831641
@Aijuh
Format changed
b833b9a
@Aijuh
submit fixed
97ccc43
@Aijuh
Use _BaseEntry
4bcd10c
@Aijuh
Update builder.py
a12bdda
@Aijuh
Update builder.py
f03d368
@Aijuh
Update builder.py
f1e11f0
@Aijuh
test passed
7a001c4
BECATRUE and others added 25 commits April 6, 2024 01:52
@BECATRUE
Fancy TTL status (#282)
00d18ad 
This closes #281.


![image](https://github.com/snu-quiqcl/iquip/assets/76851886/f1cdca23-b08b-4751-9e80-7ef71a4ce8e3)
@BECATRUE
Remove visualization feature
649b0ba
@kangz12345
Revert "Reduce the diff for smaller PR"
8a801c2 
This reverts commit c9c2e94.
@kangz12345
Simpler variable names
e39e29c
@kangz12345
Make position reading period as an argument
160b526
@kangz12345
Connect the renamed signals of the widget properly
1154d4d
@kangz12345
Log the error reported by RPC clients
3ac673d
@kangz12345
Add stage app in the config template
b97fba3
@kangz12345
Break a too long line
8321570
@BECATRUE
Remove visualization feature (#284)
e3d639f 
This closes #283.

Now, the visualization checkbox in builder app is removed.

<img width=200
src="https://github.com/snu-quiqcl/iquip/assets/76851886/4dec444f-b85f-4bd3-8e2d-8cf24cc49549">
@BECATRUE
Fix a bug when changing the remote mode rapidly (#278)
4c3ccb0 
This resolved #277.

As I tested, there is a slight delay when scrolling rapidly.
However, it seems to be allowed and moderately efficient.
@kangz12345
Simplify the code
95b9985 
Applied the reivew.
@kangz12345
Add attributes docstring of StageControllerApp
2a5ed0f 
Applied the review.
@kangz12345
Fix typo
e426556 
Applied the review.
@kangz12345
Add missing argument
936fee3 
Applied the review.
@kangz12345
Stage controller app (#285)
b4422f0
@BECATRUE
Update background image and icon
0430bc6
@BECATRUE
Update resource file path
eab99f0
@BECATRUE
Update background image and icon (#288)
4fa74da 
This closes #287.

This PR is perfectly compatible to snu-quiqcl/qiwis#250, so in the
current QIWIS version, the background and icon may look strange (e.g.,
scale, ratio).

### Background

![image](https://github.com/snu-quiqcl/iquip/assets/76851886/c07f504f-aac0-4737-8e04-a4c44d9599ad)

### Icon
<img width=600
src="https://github.com/snu-quiqcl/iquip/assets/76851886/b08162c5-d834-4e7d-a46b-b8d17578be24">
@BECATRUE
Add requirements
cce9886
@BECATRUE
Install by requirements in actions
9267761
@BECATRUE
Add numpy dependency
31b25b4
@BECATRUE
Not support Python 3.8
44dd342
@BECATRUE
Remove QIWIS version
cb431ad
@BECATRUE
Use requirements.txt instead of direct installation (#290)
d8204e7 
This closes #286.

After merged, I will release the first official version, v1.0.0.
@BECATRUE BECATRUE added the repository About this git or GitHub repository label Apr 10, 2024
@BECATRUE BECATRUE self-assigned this Apr 10, 2024
@BECATRUE BECATRUE requested a review from kangz12345 as a code owner April 10, 2024 11:10
@BECATRUE BECATRUE merged commit 0ac9662 into release Apr 10, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kangz12345 kangz12345 kangz12345 approved these changes

Assignees

@BECATRUE BECATRUE

Labels
repository About this git or GitHub repository
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@BECATRUE @kangz12345 @Aijuh