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

✨ Add download-study-logs-fcn #163

Merged
merged 152 commits into from
Jun 26, 2024
Merged

✨ Add download-study-logs-fcn #163

merged 152 commits into from
Jun 26, 2024

Conversation

bisgaard-itis
Copy link
Collaborator

@bisgaard-itis bisgaard-itis commented Jun 25, 2024 •
edited
Loading

What do these changes do?

  • Add a function for iterating over all studies.
  • Add a function for downloading study job logs.
  • Update openapi.json to latest server version

Related issue/s

How to test

For internal developers

bisgaard-itis and others added 30 commits October 23, 2023 11:33
@bisgaard-itis
update workflow before publishing python package
9c15998
@bisgaard-itis
Merge branch 'master' of github.com:bisgaard-itis/osparc-simcore-clients
ab1bc1e
@bisgaard-itis
Merge branch 'ITISFoundation:master' into master
60c4f60
@bisgaard-itis
fix dependency issue and bump version
3aa7d70
@bisgaard-itis
Merge branch 'master' of github.com:bisgaard-itis/osparc-simcore-clients
a9810d4
@bisgaard-itis
point to website in project description
5d2ceb2
@bisgaard-itis
Merge branch 'ITISFoundation:master' into master
185195c
@bisgaard-itis
Merge branch 'ITISFoundation:master' into master
f58c10f
@bisgaard-itis
Merge branch 'ITISFoundation:master' into master
fdd75b7
@bisgaard-itis
fix broken dependency
11e369d
@bisgaard-itis
improve doc
e844983
@bisgaard-itis
Merge branch 'ITISFoundation:master' into master
c798687
@bisgaard-itis
Merge branch 'ITISFoundation:master' into master
55bdba4
@bisgaard-itis
add github token to download artifacts
9ce0911
@bisgaard-itis
ensure only read-access @wvangeit
39210ef
@bisgaard-itis
yet another attempt at downloading artifacts
50c487f
@bisgaard-itis
resolve merge conflicts
128a1a7
@bisgaard-itis
make sure to use repo that ran the trigger wf
3d8d1eb
@bisgaard-itis
Merge branch 'ITISFoundation:master' into master
3f2b07a
@bisgaard-itis
another attempt at fixing
48cf475
@bisgaard-itis
change owner
77d1e81
@bisgaard-itis
change repo owner
21b446b
@bisgaard-itis
Merge branch 'ITISFoundation:master' into master
10b5ce0
@bisgaard-itis
allow publishing to testpypi also when pr
d90cccc
@bisgaard-itis
minor change
b1561a4
@bisgaard-itis
Merge branch 'ITISFoundation:master' into master
2db2d81
@bisgaard-itis
revert minor (but breaking) change
a22c0c4
@bisgaard-itis
Merge branch 'ITISFoundation:master' into master
9a6a1e0
@bisgaard-itis
minor fix
a83b349
@bisgaard-itis
Merge branch 'master' of github.com:bisgaard-itis/osparc-simcore-clients
e4e07d7
@bisgaard-itis bisgaard-itis self-assigned this Jun 25, 2024
@bisgaard-itis bisgaard-itis added enhancement New feature or request add test Add more tests labels Jun 25, 2024
@bisgaard-itis bisgaard-itis added this to the South Island Iced Tea milestone Jun 25, 2024
@bisgaard-itis bisgaard-itis marked this pull request as ready for review June 25, 2024 14:20
pcrespov
pcrespov reviewed Jun 25, 2024
View reviewed changes
clients/python/client/osparc/_studies_api.py Outdated Show resolved Hide resolved
clients/python/client/osparc/_studies_api.py Outdated Show resolved Hide resolved
clients/python/client/osparc/_studies_api.py Show resolved Hide resolved
clients/python/client/osparc/_studies_api.py Outdated Show resolved Hide resolved
clients/python/client/osparc/_studies_api.py Show resolved Hide resolved
clients/python/client/osparc/_studies_api.py Outdated Show resolved Hide resolved
clients/python/client/osparc/_studies_api.py Outdated Show resolved Hide resolved
clients/python/test/e2e/conftest.py Show resolved Hide resolved
clients/python/test/e2e/conftest.py Outdated Show resolved Hide resolved
clients/python/test/test_osparc/test_utils.py Outdated Show resolved Hide resolved
pcrespov
pcrespov approved these changes Jun 25, 2024
View reviewed changes
Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

Cool, thx! left some suggestions for your consideration. Otherwise all ok!

pcrespov
pcrespov approved these changes Jun 26, 2024
View reviewed changes
Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

cool! thx

@bisgaard-itis bisgaard-itis merged commit 8a210f3 into ITISFoundation:master Jun 26, 2024
8 checks passed
@bisgaard-itis bisgaard-itis deleted the 161-add-download-study-logs-fcn branch June 26, 2024 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pcrespov pcrespov pcrespov approved these changes

@wvangeit wvangeit Awaiting requested review from wvangeit

Assignees

@bisgaard-itis bisgaard-itis

Labels
add test Add more tests enhancement New feature or request
Projects
None yet
Milestone
South Island Iced Tea
Development

Successfully merging this pull request may close these issues.

Add function for downloading study logs
2 participants
@bisgaard-itis @pcrespov